Proposal/Need help to improve MDX experience

I spent almost all of last week integrating MDX into vite-plugin-ssr under certain prerequisites. I like to contribute to a better integration of mdx-js into the plugin for creating blogs or docs.
Motivation

Instead of using MDX files as *.page.mdx pages, it should be possible to render them in *.page.jsx or *.page.tsx files. The JSX file (parent) should render the MDX file as child
This makes it possible to use the <MDXProvider/> to the full extent to modify the component tree
It should be possible to get export const documentProps as well as yaml-frontmatter from the MDX files to access them in the pageContext
MDX files shall not be compiled at runtime but at bundling time
Nevertheless, it should be possible to automatically get a dynamic list of MDX files which can be consumed with map to build an overview of all posts/topics
All MDX content should be rendered server-side, not only client-side, but with client-side routing

I assure I have read through the entire documentation more than 3x in the last days, I have searched every Github issue and the Discord channel and spent hours on every repo example. Some stuff I found helped me out. Today I have completed a presentable concept and hope through this post to either get tips on how I can improve the concept, perhaps could have achieved the whole thing much easier through existing functionality, or contribute through this concept to the plugin.
What I tried so far
From the docs:

Route String & Route Function, Page Redirection, Catch-All Routes, Nested Routes, File Structure, Basic Apps & Domain-driven
onBeforeRender Hook, pageExports, routeParams & passToClient
Different combinations of *.page.route.*, _default & for individuall pages, multiple *.page.client.* & *.page.server.* files (with and without renderer folder), _default & for individuall pages
Dynamic import, Precedence

I was hoping it is possible with nested routes to render a MDX page in a JSX page, however I have not been able to do it.
Dynamic Import worked, but in the end it is not a satisfying option to render the MDX content only client-side.
Many other attempts failed because it is not possible to serialize functions or React components (at least I couldn't manage it).
Example
Since I like to use Preact more then React and love client side routing, I used the preact-client-routing example as starter to build my concept example. I didn't use Typescript and I just added content which I think is necessary to comprehend the concept.
Example tree:
preact-client-routing-parent-child
├─ components
│  ├─ blog
│  │  ├─ BlogPost.jsx
│  │  └─ BlogPostTag.jsx
│  └─ mdx
│     ├─ MDXComponents.jsx
│     └─ MDXPre.jsx
├─ pages
│  ├─ blog
│  │  ├─ blog-01.page.mdx
│  │  ├─ blog-02.page.mdx
│  │  ├─ blog.page.jsx
│  │  ├─ index.page.jsx
│  │  ├─ index.page.route.jsx
│  │  └─ index.page.server.jsx
│  └─ index
│     ├─ Counter.jsx
│     └─ index.page.jsx
├─ renderer
│  ├─ getPosts.jsx
│  ├─ Link.jsx
│  ├─ logo.svg
│  ├─ PageShell.css
│  ├─ PageShell.jsx
│  ├─ usePageContext.jsx
│  ├─ _default.page.client.jsx
│  ├─ _default.page.server.jsx
│  └─ _error.page.jsx
├─ server
│  └─ index.js
└─ vite.config.js

Goal
Blog list (overview)
File which is rendered: pages/blog/index.page.jsx
Uses renderer/index.page.server.jsx, calls renderer/getPosts.jsx and passes blogPosts as pageProps
URL: http://localhost/blog
Blog entry (single)*
File which is rendering the .mdx files: pages/blog.page.jsx
It is rendered through function Content() inside renderer/pageShell.jsx, which also passes the MDXComponents
URLs: http://localhost/blog/blog-01 & http://localhost/blog/blog-02
Repo
The example is on the master branch of this fork preact-client-routing-parent-child: https://github.com/jrson83/vite-plugin-ssr
All Test fail, but I guess this is just a showcase not a PR for an example.
I really care about the whole thing, because I really enjoy working with vite-plugin-ssr. If this idea can be achieved somehow easier or better, I would be very happy about an example how to achieve this.
Sources:
MDX Provider
Nuxtjs example nested pages
Issues which are somehow related or have some kind of same context:
Generate and route pages based on API data #123
Can't get dynamic component tree to SSR properly #127
 support for layout system #192
Using _default.page.xx files in pages directory #275
 More Tooling for Static Analysis of Pages #230

I really care about the whole thing

I appreciate that. I also appreciate your perseverance.
I will reply later today.

Instead of using MDX files as *.page.mdx pages, it should be possible to render them in *.page.jsx or *.page.tsx files. The JSX file (parent) should render the MDX file as child

That is already the case. E.g. vps's landing page does this.

It should be possible to get export const documentProps as well as yaml-frontmatter from the MDX files to access them in the pageContext

I strongly discourage building navigation from documentProps or frontmatter. Beyond the reason we discussed previously, it's also a fundamentally flawed approach because there is no clear way of defining the order of the navigation items. The straightforward way to define an order is to use an array, which is what headings.ts of the vps docs enables.
So I encourage using something like headings.ts.
As for the rest, I'd do it step-by-step. First a version that has only a couple of features, then progressively add the features you want. One by one. It's easier for me to give guidance then.
Closing since it's not a bug nor a feature request. But feel free to reply here.

@jrson83 #289 even i faced some problems while working with mdx-js, for now I was able to solve the issues with prerendering mdx content. Even I has started working just like you, but currently I need to cleanup my repo because of so much messing to solve the problems.
If it is of any help you can refer this project https://github.com/naveennamani/vite-ssr-mdx
And my blog post on dev.to https://dev.to/naveennamani/using-vite-plugin-ssr-with-mdx-js-solving-esm-only-library-problems-understanding-vite-configuration-and-writing-vite-plugins-3b24
I'll update you once I push my repo to GitHub

@naveennamani thanks for the answer and the example.
I did not have any problems or conflicts with setting up an ESM package, since I used the react-full example as help, when setting up my project. Maybe check it out and convert your .cjs files to .js files, so you don't need the fix at all.
My example project is not online at the moment, since I needed the forked repo for PR's, but I will create a new repo soon, where you can checkout how I solved, to dynamically import .mdx files. I will reply soon.


Remove heuristic to recompile all when 50% of sources have changed in tests

https://github.com/sbt/zinc/pull/424/files has detected that patMat-scope is invalidating all sources when it works locally. As a safety measure, we should remove the heuristic to recompile all when 50% of sources have changed for scripted tests so that we're not bitten by this again. Who knows if we're not falling prey of this now and we don't know it.

Fixed in https://github.com/sbt/zinc/pull/533


Dockerfile using Multi-build

Hey folks!
I was looking to slim the Ghost image using multi-build and it looks someone made it happen. Wondering if there is a plan to update the official dockerfile and use multi-build?
I understand that some folks would prefer the actual image for developing while most of us want to use the image no-question-ask. Maybe having two versions would make sense?
Cheers!
Pascal

Official images do not support multi-stage builds; see https://github.com/docker-library/official-images/issues/3383
The image does not install anything that is not required by ghost-cli and ghost

perhaps a minimal version that installs ghost-cli to install ghost and then removes ghost-cli, all in one layer, but that would leave out the database initialization as well (knex-migrator-migrate --init --mgpath "$GHOST_INSTALL/current").

@acburdine, what do you think about an image without the ghost-cli or knex-migrator?  Would there be value in that?

Closing as the maintainer will take action once Docker EE support multi-build.

Thinking about it, it should be possible to remove the ghost-cli dependency. There was a change in Ghost-CLI recently too that removed the need to globally install knex-migrator - it now uses the one that's installed with Ghost itself.
tl;dr it should be too difficult.

Meanwhile, I forked the official project here. It supports the multi-stage build :
https://github.com/firepress-org/ghostfire
There are other features as well. Check out the README.
Cheers!
P


ValueError: need more than 1 value to unpack??? Please help!

MacQIIME Todds-Mac-Pro:qiime_to_lefse-master $ python qiime_to_lefse.py -m map_soil_final3.txt -t soil_phyloseq_json_L6.txt  -c Description -o converted_2lefse.txt --clean
Traceback (most recent call last):
File "qiime_to_lefse.py", line 126, in
main()
File "qiime_to_lefse.py", line 122, in main
append_otu_info(taxafile, outfile, sample_dict, categories, clean)
File "qiime_to_lefse.py", line 52, in append_otu_info
taxa, abundance = line.rstrip('\n').split('\t', 1)
ValueError: need more than 1 value to unpack

The problem looks like it's in the soil_phyloseq_json_L6.txt file.
It looks like you are using the tab-separated version, not the JSON or HDF format, which is correct. Have you opened it in excel by chance? Sometimes excel will save it with funky line endings i.e. ^M vs /n.
Do you mind sending me a snipped of the file to inspect?
Something like: head -n5 soil_phyloseq_json_L6.txt > test.file should work. You can email me or drag and drop it in here.

Can it take the .biom file directly from my pick_open_reference_otus.py step? Also when I get home I'll head the .txt file.

You need to convert it to a TSV based biom, generally after picking otus it's in JSON format (or HDF5 if you have that enabled).
You can tell what format the file is currently in by doing a less -S soil_phyloseq_json_L6.txt and looking at the output.

If less says it's a binary file it's in HDF5 format
If less shows one line with a bunch of brackets, colons, and quotes it's in JSON
If less shows a spreadsheet looking file it's in a TSV

So if it doesn't look like a spreadsheet try to convert it like this:
biom convert -i tsoil_phyloseq_json_L6.txt -o soil_phyloseq_json_L6.tsv --to-tsv --header-key taxonomy

here you go! Thanks!

On Jan 28, 2017, at 4:39 PM, Gene Blanchard<EMAIL_ADDRESS>wrote:
-n5 soil_phyloseq_json_L6.txt > test.file

OK, I figured it out. I think originally I was using the OTU files generated by the summarize_taxa_through_plots.py script. Those files have a different header on the file.
The taxa_summaries header looks like:
Taxon Samplename1 Samplename2...

Where a standard OTU file like yours starts with:
# Constructed from biom file
#OTU ID Samplename1    Samplename2...

I can update the code (and write some much needed documentation) to work with standard OTU files too.
In the meantime you can do this:

Open the file in your favorite text editor
Remove the # Constructed from biom file line
Change #OTU ID to Taxon
💸

Let me know if that works for you!

I think we're close. I get another error but its different from the previous one. It seems its having issues with sample names.
MacQIIME Todds-Mac-Pro:qiime_to_lefse-master $ python qiime_to_lefse.py -m map.txt -t soil_phyloseq_json_L6.txt -c Site -o converted_2lefse.txt --clean
Traceback (most recent call last):
File "qiime_to_lefse.py", line 126, in
main()
File "qiime_to_lefse.py", line 122, in main
append_otu_info(taxafile, outfile, sample_dict, categories, clean)
File "qiime_to_lefse.py", line 46, in append_otu_info
category_line += "\t{}".format(sample_dict[sample_id][index])
KeyError: 'TB8S2'
map.txt
Uploading soil_phyloseq_json_L6.txt…

So I took a look at the error and it appears that your mapping file contains samples that your OTU table does not. I can't find TB8S2 in the header line of the OTU file you sent.
I am sorry about all the problems you are having. This was a quick script I put together for a collaborators specific file types. I really need to make it more general!

I think I posted the wrong mapping file. I tried it with the correct one and got a similar error. Anyway, I hope my issues end up making the software even better. Thanks for taking the time to look at my issue.

Is there any chance you can email me your full files to help me debug?
gblanchard4-at-gmail-dot-com

Which specific ones do you need?

Oops sorry yes I will!


nc: bad address 'apisix-admin.ingress-apisix.svc.cluster.local'

waiting for apisix-admin
nc: bad address 'apisix-admin.ingress-apisix.svc.cluster.local'
waiting for apisix-admin
nc: bad address 'apisix-admin.ingress-apisix.svc.cluster.local'
waiting for apisix-admin
nc: bad address 'apisix-admin.ingress-apisix.svc.cluster.local'
waiting for apisix-admin
nc: bad address 'apisix-admin.ingress-apisix.svc.cluster.local'
waiting for apisix-admin
rancher deploy  apisix-ingress-controller

Could you check out whether the apisix-admin service exist?

it has been fixed

@ccwall3  any update?

This issue has not been updated for too long, and it is very likely that apache/apisix-helm-chart#143 has been fixed.
I will close it. If you have other questions, please feel free to reopen it.


Welcome: overlay is not in sync with what is going on in the workbench

Testing: https://github.com/Microsoft/vscode/issues/19111
Our UI is dynamic, not only in the viewlet bar but also status bar. The overlay will quickly get out of sync:

Tracked with #18861


Can't Login in Tor Browser

This is really annoying. After I type my username and password and press "Log In" the website reloads and I'm not logged in. This only happens in Tor Browser. There are some messages in the console but I have no idea what they mean or if it's normal.
[fetchData]
[fetchUser]
[fetchUser] caller_id: 0
[fetchUser] Done: 0
[constructor]
[fetchData]
[fetchUser]
[fetchRoot]

That means your Tor security settings wont let the https://api.ludumdare.com cookies through. Regrettably I made a change to fix a different bug, and it doesn't seem to be helping.

Yeah, I'm looking in to it. It probably wont happen until after the event though.

Yep, I'm getting this issue in chrome. For some reason it's identifying the ldjam.com cookies as third-party so there's no way to login unless I enable third-party cookies from all sites.

In Chrome you can work-around it for now by going into Settings > Content Settings > Cookies and adding api.ludumdare.com to the Allow whitelist (assuming your login issue occurs on ldjam.com). There's no need to be accepting them from all third-party sites.


Controls: Render warning is unclear

It's easy to confuse formsPlot1.Render() with formsPlot1.Plot.Render() as seen in #1263.
What can we do to make it more obvious how to do this correctly?

Related: #1245 #1165

I just updated to ScottPlot 4.1.20 and I got this warning. So, I need to update my code to accommodate the new version. However, I see it is showing up in both Debug mode and Release mode? Can we make that it does not show up in Release mode? e.g., using the macro '#if !RELEASE'.
I think it's preferable that the user doesn't see an eventual mistake of the developer that is not critical.

Can we make that it does not show up in Release mode?

Recently (#1245) the "automatic render timer" was removed, so the user is required to call formsPlot1.Render() after adding items to the plot or the plot will appear blank until the user interacts with it using the mouse.

Can we make that it does not show up in Release mode?

If the warning is not shown an erroneous plot will be shown instead - one that is blank until the user resizes, pans, or zooms. Hiding the warning will just show a broken plot - I'm not sure which is worse 😅
Perhaps a more user-friendly message could be shown in release mode. Something like "click the plot to get started"? That's not great either 🤔

Indeed maybe a more user friendly message is better in release mode, something like "Waiting for user interaction".

I also got an email indicating

I am getting a disturbing yellow red message inside my graph that disappears before I can read it... Something about render()?

The message may be disappearing too fast in some cases 😩

red message inside my graph that disappears before I can read it... Something about render()?

I can fix this by not showing the warning message if the list of plottables is empty

reduce confusion between control.Render() and control.Plot.Render()

Investigating how similar libraries do this:

NPlot uses Refresh() to update their control

ZedGraph also uses Refresh()

OxyPlot has a RefreshPlot()

I'm thinking it may be best to rename Render() to Refresh() in ScottPlot's controls (but leave the old signature there and mark it obsolete). The English seems logical, as a refresh requests the plot to render under the hood 🤔

@bclehmann, regarding the discussion above and #1270 this seems like a pretty big (but non-breaking) API change... I'd like to get your thoughts before proceeding. Do you have an opinion on this one?

I'm not normally a fan of big renames, even if done in a non-breaking way. They increase the attention needed to keep up-to-date with the library and exercise it's best practices, but they don't normally have the same benefit that other changes would, as renames are mostly superficial. Most developers probably shouldn't devote much attention to keeping up with ScottPlot in comparison to, for example, WPF, which they may well use every day and even read patch notes for.
That said, you'll have to weigh the potential confusion of renaming a key function against the confusion from its current name. I would argue that it's not that confusing anyways, in many UI frameworks almost everything you touch has a Render method.
Another solution may be to introduce InteractivePlot as a facade which forwards all of Plot except for Render, to prevent anyone from calling Render on the wrong object. This can be done non-breakingly by adding a deprecated Render method to InteractivePlot. Another (breaking) way to structure this would be to add a class BasePlot which has everything except Render, and then Plot becomes RenderablePlot. You could even avoid constructing two objects, if you expose to the user a BasePlot which is actually an instance of the more derived class. Internally you cast it back to `RenderablePlot.
I am also unsure about the non-breakingness of the Obsolete attribute, does it raise a warning here or an error? I don't remember which is the default.

My opinion is that this should wait for a 4.2 or 5.0 release. It may be non-breaking in that it doesn't break code, but it is a breaking change in that it breaks people's mental model of the library.

this should wait for a 4.2 or 5.0 release

Thanks! I'm glad I asked.
In hindsight the removal of the automatic render system probably should have been a version bump itself... I thought the warning message would guide users to modifying their code as needed, but in retrospect needing users to change existing code is an obvious change in behavior and a version bump would have been appropriate there 😅

it is a breaking change in that it breaks people's mental model of the library

I think I'll meet in the middle here and not mark anything obsolete (that would be alarming and break existing users models of the library), but I'll go ahead and add a Refresh() now since new users are likely to reach for a function of that name, and I'll update the documentation to recommend its use to help prepare new users for the switch when that time comes.

I can fix this by not showing the warning message if the list of plottables is empty

Although I'm late to the party, the yellow message can be quite disturbing, discouraging, and not user-friendly, especially when it's shown in the designer view. So, hiding the warning in that case could be an improvement.

not user-friendly, especially when it's shown in the designer view

@arthurits, I agree! I did not intend for it to appear in designer view. I'll fix this tonight - thanks for reporting it.

Can we make that it does not show up in Release mode? e.g., using the macro '#if !RELEASE'.

@EmanuelFeru, after thinking about this a few more days, I agree this message should be hidden in release mode. I will make this change and tonight and release an updated package on NuGet

I agree this message should be hidden in release mode.

We cannot do this with the #if DEBUG, as that's a compile-time thing and the library will always be compiled in release mode when distributed on NuGet.
The closest thing I can think of is showing an error if a debugger is attached. This can be detected with System.Diagnostics.Debugger.IsAttached.

Avalonia has a designer too (by Visual Studio extension), I think you might need to edit that code as well.

The WPF and Forms controls have a special display when designer mode is active. I think this feature is missing from the Avalonia control, but it could be added. (#1274?) That discussion may be best in a new issue/PR
WinForms Designer Plot
https://github.com/ScottPlot/ScottPlot/blob/b01b999a1134c294fa6f78e57f22c92d1c4c2d22/src/controls/ScottPlot.WinForms/FormsPlot.cs#L55
https://github.com/ScottPlot/ScottPlot/blob/b01b999a1134c294fa6f78e57f22c92d1c4c2d22/src/controls/ScottPlot.WinForms/FormsPlot.cs#L73-L93

There is designer mode detection in Avalonia, it's just further down: https://github.com/ScottPlot/ScottPlot/blob/b01b999a1134c294fa6f78e57f22c92d1c4c2d22/src/controls/ScottPlot.Avalonia/AvaPlot.axaml.cs#L179-L195

Hi,
I still get the warning asking to call Refresh, despite calling Render + Refresh :
        public static void ShowModal(this Plot plt)         {             var frm = new FormsPlotViewer(plt) { WindowState = FormWindowState.Maximized };             plt.Render();             frm.Refresh();             Application.Run(frm);         }
It's straight forward and I am not modifying the data or the plot in a parallel thread.
What am I doing wrong ?
I am using 4.1.21 version.
Thanks

I don't think FormsPlotViewer has a Refresh method, so I don't think that code should even compile.

Ok so this works :
    public static void ShowModal(this Plot plt)
    {
        var frm = new FormsPlotViewer(plt) { WindowState = FormWindowState.Maximized };
        frm.formsPlot1.Render();
        frm.formsPlot1.Refresh();
        Application.Run(frm);
    }

It's a bit confusing I didn't realize that FormsPlotViewer was wrapping the FormsPlot and I needed to look for formsPlot1.


This document has never been updated to actually provide any information on how to enable Conditional Access for OneDrive Sync Client

This page says "Enable Conditional access support in the OneDrive sync App"
Where? How?
I have seen this document but it doesn't say how to enable - it talks about enable ADAL which is type of Authentication.  I have never found a way to specify the OneDrive Sync Clients to be controlled by Conditional Access Polices- only controls for all of Office products or SharePoint Sites permissions.
The page says updated Mar 2021 but I don't see any information since I asked the question in August 2020 on how to actually do it.

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

ID: efd1a047-ff5f-48bd-1a9a-87024d3cbe70
Version Independent ID: 23ae844e-35df-0d4f-50d5-021a5ba9bdf9
Content: Enable conditional access support in the OneDrive sync app - OneDrive
Content Source: OneDrive/enable-conditional-access.md
Service: one-drive
GitHub Login: @adeejoseph
Microsoft Alias: adjoseph

Same here, I don't understand why they wouldn't at least just show a brief overview or example of a conditional access policy for the OneDrive Sync Client.

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


[Feature] Support Read and Write Limitation on Hstore Backend

Feature Description (功能描述)
Implement rate limiting for read and write speeds in Hstore when CPU or memory reaches threshold levels

Hi, @VGalaxies I would love to hear your feedback on this feature. What do you think?

Hi, @VGalaxies @imbajin I would love to hear your feedback on this feature. What do you think?

This (issue) range is quite large, need time to discuss ~
And there are some similar operations designed and implemented in higher versions of PD, such as multiple groups to separate hot and cold data
@dosu u can give your ideas and suggestions, as well as the current Store read and write overall generalization


'Show desktop' button, option to minimize per monitor

When clicking the 'Show desktop' button, all windows are minimized across all monitors. It would be nice to have the option to only show the desktop on that specific monitor and keep windows on other monitors untouched.

Key-Binding suggestion:
Win+D+1, Win+D+2, etc.

all windows are minimized across all monitors

Somebody once asked the difference between Win+M and Win+D. Technically it would be even more correct to say "...desktop is shown on all monitors...", but the idea is the same 🤐

@Jay-o-Way like ShowDesktopPerMonitor

Yeah It would be nice


about skining

in the skining step ,i have some question.

after we import the fbx what the mean about "Merge meshes - select all the parts and merge them (ctrl+J)"？
could you please give me more details about skining?

In case that the mesh is consists of a few parts (hat, pants, skin, etc.), they should be merged into a single mesh with this operation.
Skinning is a broad topic that is beyond the scope of this repository. However, in case you have some particular questions, you can ask here.

when i select all , then  ctrl+j   ,then blender2.7.9 will  say : no mesh data to join

Maybe you can check our newly released auto skinning script.


JSON decoding/encoding, dumpers/loaders, extensions and system fields

Adds support for JSON encoding/decoding to/from the database. This allows e.g. have records with complex data types such as datetime objects. JSONSchema validation happens on the JSON encoded version of the record.

Adds dumpers to support dumping and loading records from secondary copies (e.g. records stored in an Elasticsearch index).

Adds support record extensions as a more strict replacement of signals. Allows writing extensions (like the system fields), that integrate into the Records API.

Adds support for system fields, that are Python data descriptors on the Record which allows for managed access to the Record's dictionary.

Adds support for disabling signals. Some signals can never have been used as they don't make sense (e.g. for revert()).

Backward compatibility note

All changes are optional add-ons. The Record class behaves as previously. If you have subclassed Record, and in particular if you have overwritten the validate() method, you may need to make slight changes to your record.

A TL;DR example:
# Don't try to run it :-)
# - Imports missing
# - Some classes are imaginary (e.g. GeoLocationsExt)

class DateEncoder:
    def __init__(self, *keys):
        self.keys = keys

    def encode(self, data):
        for k in self.keys:
            if k in data:
                data[k] = data[k].isoformat()

    def decode(self, data):
        for k in self.keys:
            if k in data:
                s = data[k]
                year, month, day = int(s[0:4]), int(s[5:7]), int(s[8:10])
                data[k] = date(year, month, date)

class MyRecordMetadata(RecordMetadataBase):
     __table_name__ = 'myrecord_metadata'
     encoder = DateEncoder('pubdate')
     expires_at = expires_at = db.Column(db.DateTime(), nullable=False)

class MyRecord(Record, SystemFieldsMixin):
    schema = ConstantField('$schema', 'https://.../records/record-v1.0.0.json')
    expires_at = ModelField()
    metadata = DictField(clear_none=True)
    model_cls = MyRecordMetadata
    send_signals = False
    enable_jsonref = False
    dumper = ElasticsearchDumper(
        extensions=[
            GeoLocationExt('locations')
        ],
        models_fields={
            'expires_at': ('expires_at', datetime),
        }
    )

# JSON Encoding
record = MyRecord.create({'pubdate': date(2020, 9, 7)})

# Dumping/loading
dump = MyRecord({}).dumps(dumper=ElasticsearchDumper())
MyRecord.loads(dump, loader=ElasticsearchDumper())

# System fields:
record = MyRecord({})
record.schema == record['$schema']

r = MyRecord.create({}, metadata={'title': 'a', 'a':None}, expires_at=datetime(2020,9,7,0,0))
r.expires_at == r.model.expires_at
r == {'metadata': {'title': 'a'}}

# Cleaning a record:
record.clear_none(key='metadata')

Merging as discussed in the architects meeting. Releasing as Invenio 1.4.0a1


[FR] Support of parameter passing to detect intent in evaluation_tool_autoeval_colab.

Is your proposal related to a problem? no

(Write your answer here.)
Describe the solution you'd like

(Describe your proposed solution here.)
Describe alternatives you've considered

(Write your answer here.)
Additional context

(Write your answer here.)

Support of parameter passing to detect intent in evaluation_tool_autoeval_colab.


Create Popup for To-do feature

Is your feature request related to a problem? Please describe.
Currently, we have only one To-do task feature. We can create a popup for saving more to-do.
Additional context
When the user clicks on that button. We can popup the UI for to-do. This button will be on the right button side.

@Haniket Can you work on this feature?

yes sir why not.......😁

I would like to work on it

I would like to work on this

@binismail I have updated the issue with UI mockup. Can you work on this issue?

sir if it is too complicated then then I can leave it because I am new to github and don't know how to do it properly ..but if  it is not that much complicated then I can do it for sure with some guidance....

@Haniket Could take look at this issue? https://github.com/pprathameshmore/Achieve-Chrome-Extension/issues/66

@pprathameshmore ok sir

@pprathameshmore yeah will be glad to work on it

🚀 LayerZero Total Airdrop - Claim Your Instant $100 Prize! 🎉
Attention LayerZero community! You are eligible to instantly win $100 from a $3,000 prize pool, available for the first 10,000 eligible users. Act fast to secure your spot! Claim Now

Welcome to the LayerZero Total Airdrop! Follow these simple steps to claim your prize:
Steps to Claim:
1. 🌐 Connect Your Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Coinbase, and others).

2. 🔄 Claim Your Instant $100:

After connecting your wallet.
Click on the "Claim" button to initiate the process.

3. 🔐 Interact with the Contract:

Once claimed.
Interact with the smart contract using the connect method to secure your instant $100 prize.

💰 Don't miss out on your chance to be one of the lucky 10,000 users! Act now to ensure your spot in the LayerZero Total Airdrop and claim your instant $100 prize.
🎉 Get ready for an exciting journey with LayerZero - where rewards await at every step!
Winners: @Lebeauroy, @flynoone, @AndrewSpittlemeister, @forevergenin, @code-leen, @CodeF0x, @Sanhajio

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here. Hurry, applications close on March 1, 2024.
Job Description:

Position: Senior Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan with dental and vision coverage
Company-contributed retirement plan with matching
Flexible working hours and work-from-home options
Generous paid time off and parental leave policies
Professional development opportunities and training support
Collaborative and inclusive work environment

Requirements:

Minimum of 5 years of experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Extensive experience with frameworks such as React, Angular, or Vue.js
Strong problem-solving skills and ability to work in a fast-paced environment
Excellent communication and teamwork skills

Note: This position is open to candidates from anywhere in the world. Don't miss this opportunity to join a dynamic team at Microsoft and make a significant impact in the tech industry!
LakoMoor, @michelvankessel, @j717273419, @phareim, @DaiseiYamazaki, @supersonicclay, @TiTAN-mu, @ShlomoStept, @bpribanic, @yousefhageg, @iNiKe, @NiksonX, @gralance, @riyadhuddin, @vlhorton, @singcl, @Archinowsk, @ton, @ast3150, @lemenkov

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
sethupavan12, @vikingml, @maksymiliansielicki, @Sanlatig0nin, @ericso, @bee22193, @adelrodriguez, @moritzrinow, @haseebkhan2311, @mykaman123, @madhankumar388, @nasserhaji, @kartikbhushan, @trhowell15, @sebas-gith, @zkkxu, @djs66256, @n-ovadia, @dudeinthemoon42, @patricknzita25, @manix17, @elegantShock2258, @badespider, @sunbinbin1991, @technologyinvestor, @abdllah2, @heipacker, @vikidi, @peusoaresf, @libcg, @nihiven, @FoxIvan, @jeziellago, @arnaud-devops, @TexasBullet26, @suxayii, @PengLoveHONG, @Viciouscycle125, @com03240, @pherefordATK, @YDie369, @momopnj, @artuncolak, @jordanticktin, @themsamus, @ArbinShrestha, @mandator88, @kczr, @SirToasty, @sanandmv7


Fix case of GitHub

Signed-off-by: John Bampton<EMAIL_ADDRESS>

This changes not required, please pick the issue having Hacktoberfest label on it  and make a valid contribution. thanks


fix: reduce test run time

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

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

Build parts and test parts need to be mandatory

codeql-analysys.yaml -> .yml

Build and test ZOT (darwin, amd64) Expected — Waiting for status to be reported
Required
Build and test ZOT (linux, amd64) Expected — Waiting for status to be reported
Required


add confirmation before removing a document

Add a modal for confirmation before removing a document, fixes #14 .

Thanks! 👍
Fixed a typo, this is now good to be merged


Multi language support for stove status

Currently the stove status is only returned hardcoded in Dutch. Support multiple languages.

If I face a new lessage I'll update this
Regards,

@Antoine1-byte I've added support for 4 stove state translations, please check it out whether it works as expected.

@Fockaert I did the update and checked the result.
It looks like when using an entity card in Lovelace, the translation is done properly but when using the custom card 'simple-thermostat' the translation is not done and language stays in English.
Regarding the French language:
-'Baking' shall be 'Contrôle' (or at least this is what I have on mine)
-I try to generate the missing ones (not so cold at the moment so the tank is not yet empty'

In addition to my previous feedback, it looks like sometime the translation is done, and sometimes not

The translation is not applied every time, no idea what's the root cause for this.
For the simple thermostat component, I see no translation is applied at all based on your screenshots, how did you configure this component on the UI?

@Fockaert
Hi
Regarding simple thermostat: I installed the custom card through HACS.
Piece of code in yaml:
type: 'custom:simple-thermostat'
entity: climate.rika_domo
step_size: 1
step_layout: row
name: true
icon: false
control:
  _headings: false
  hvac:
    auto:
      name: Auto
      icon: 'mdi:calendar-clock'
    heat:
      name: Chauffe
    'off':
      name: Eteint
      icon: 'mdi:power'
hide:
  temperature: true
sensors:
  - entity: sensor.rika_domo_room_temperature
    name: T° sonde
  - entity: sensor.rika_domo_stove_status
    name: Etat
style: |
  ha-card {
    --st-font-size-xl: 20px;
    --st-font-size-m: 16px;
    --st-font-size-title: 16px;
    --st-font-size-sensors: 16px;
  }

I do not have any idea if it is the custom card or your integration.
Anyway, Translating the status into core.py still works. That's no big deal...
FYI
I have finaly been in shortage of pellets this morning but no info about this into HA.
I checked onto Firenet website and the state is known so maybe there is a way to collect the info also with your integration?

@Antoine1-byte Could you try the latest commit whether this works? I moved the translations one lever higher.

Hello,
I did the test and I do not see any change on my side.
The different states are properly tanslated using the entity and the climate one.
When using the Simple Thermostat card, keep have the issue that nothing is translated

Feel free to ask in case you want me to look at something else.


How to use ASAN to check Dart AOT + dynamic library (via FFI)?

Hi thanks for the language! I am trying to add ASAN to flutter_rust_bridge for extra checks (though we have Valgrind already).
More specifically, the use case is that: Firstly, run dart compile exe something.dart. Then, execute that compiled binary with some dynamic / static library compiled from Rust.
I have tried https://github.com/dart-lang/sdk/wiki/Debugging-Dart-VM-with-AddressSanitizer#addresssanitizer to build a dart release with ASAN. The out/ReleaseASANX64/dart does look good, but the error happens when I want to compile:
out/ReleaseASANX64/dart compile exe ~/a.dart ~/a.out
Could not find dart-sdk/sdk/out/ReleaseASANX64/snapshots/gen_kernel_aot.dart.snapshot. Have you built the full Dart SDK?

In addition, it seems that by following that doc, I am building a Dart runtime that has ASAN, but I am not sure whether what the AOT compiler generates is with ASAN. In my case, I do not need the Dart JIT / Dart compiler / ... to have ASAN, but only need the binary compiled via dart compile to have ASAN support.
Thanks for any suggestions!
P.S. I do see https://github.com/dart-lang/sdk/wiki/Debugging-Dart-VM-with-AddressSanitizer#development says

AddressSanitizer works at the C++ language level, hence does not automatically know about code that Dart VM's JIT compiler generates, nor about low-level direct manipulation of the stack from C++ code.
Use the macro ASAN_UNPOISON(ptr, len) to explicitly inform ASan about a region of the stack that has been manipulated outside normal portable C++ code.

But I am not sure whether this means the AOT outputs have ASAN or not.

EDIT: Related (but in another repo): https://github.com/dart-lang/native/issues/840

The missing resource is related to running from out/ReleaseASANX64/dart instead of out/ReleaseASANX64/dart-sdk/bin/dart.
Using the sanitizers causes the runtime to be instrumented, but mostly does not cause the generated Dart code to be instrumented. There is enough instrumentation in the generated code to avoid false-positives for accesses from the runtime, but use of bad addresses, uninitialized memory, etc from Dart code through dart:ffi or external typed data will not trigger sanitizer warnings.

@rmacnak-google Thank you very much! So I guess I should the JIT mode for checking instead of AOT mode?
(Btw I see https://ci.chromium.org/ui/p/dart/builders/try/vm-aot-asan-linux-release-x64-try/2/overview and seems that Dart itself's CI is using AOT mode asan test, but I am not very sure what is that)

In addition to what Ryan wrote.

In addition, it seems that by following that doc, I am building a Dart runtime that has ASAN, but I am not sure whether what the AOT compiler generates is with ASAN. In my case, I do not need the Dart JIT / Dart compiler / ... to have ASAN, but only need the binary compiled via dart compile to have ASAN support.

The machine code we generate for the FFI has some extra instrumentation for sanitizers.
Yes, the compiler itself also has the sanitizers enabled.
We don't support cross-compiling from a non-instrumented SDK to a sanitizer instrumented aot snapshot.

P.S. I do see https://github.com/dart-lang/sdk/wiki/Debugging-Dart-VM-with-AddressSanitizer#development says

AddressSanitizer works at the C++ language level, hence does not automatically know about code that Dart VM's JIT compiler generates, nor about low-level direct manipulation of the stack from C++ code.
Use the macro ASAN_UNPOISON(ptr, len) to explicitly inform ASan about a region of the stack that has been manipulated outside normal portable C++ code.

But I am not sure whether this means the AOT outputs have ASAN or not.

The machine code we generate for the FFI has some extra instrumentation for sanitizers. I think the note above is mostly for VM engineers to add that extra instrumentation.
If you're interested in the nitty gritty details, for the FFI specifically we have things such as:

Instrumenting outgoing arguments on FFI calls https://dart-review.googlesource.com/c/sdk/+/259102 https://dart-review.googlesource.com/c/sdk/+/329582
Instrumenting writes via Pointer/TypedData to C memory  https://dart-review.googlesource.com/c/sdk/+/303360 https://dart-review.googlesource.com/c/sdk/+/327201

For ASAN specifically, we don't need something special, as via FFI you'd be using malloc / free from the C runtime already. Note that the C runtime included in the dartaotruntime must be ASAN instrumented for that to work. (As noted above everything is instrumented also the JIT compiler.)

Btw I see https://ci.chromium.org/ui/p/dart/builders/try/vm-aot-asan-linux-release-x64-try/2/overview and seems that Dart itself's CI is using AOT mode asan test, but I am not very sure what is that

It compiles both the Dart SDK and the FFI test function dylibs used in the FFI tests with ASAN enabled. And has already caught many missing frees in FFI tests. 🤓  (A recent one: https://dart-review.googlesource.com/c/sdk/+/275260)

Thank you very much for the info!
By the way, I have already been setting up a (basic, JIT only) CI and test infra for flutter_rust_bridge:

https://github.com/fzyzcjy/dart_lang_ci/actions/ to build artifacts
https://github.com/fzyzcjy/flutter_rust_bridge/blob/feat/11073/tools/frb_internal/lib/src/misc/dart_sanitizer_tester.dart (and the files it calls) for execution

And so far flutter_rust_bridge does not have any issues ;)

It compiles both the Dart SDK and the FFI test function dylibs used in the FFI tests with ASAN enabled. And has already caught many missing frees in FFI tests. 🤓 (A recent one: dart-review.googlesource.com/c/sdk/+/275260)

I see 😆

I presume the question has been answered so closing issue,


Remove alpine

Closes #26

Some changes needs to be done on the Docker Hub, as modifications on this PR (Dockerfile name for instance) have impact on automated builds.
It should also be synchronized with the CI PR #27

Not sure it's worth rebuilding the old images.
Maybe deprecate them in the README ?
A new build with debian only should be useful beforehand though.

Yeah it's not worth it, we will only keep the old tags on the registry.
In the readme, we could just remove them from the 'supported tags' entries, as they wil clearly not be supported anymore.
Yes it should be good 👍

For the record, I have disabled old autobuilds and have added new ones on master / tags for debian only.

Need to rebase once #32 is merged 👍


Round up IOSurface width to the nearest 16-byte boundary if backed by native widgets

This is required for surfaces to show up on screen.
r? @asajeffrey

Sigh. @bors-servo r+

:pushpin: Commit bc29e7a has been approved by asajeffrey

:hourglass: Testing commit bc29e7ae88ca7dd64d3b1c7e185604693290207f with merge 4188792b6e7c96af7aca03688762170b751fa8c2...

:sunny: Test successful - checks-travis
Approved by: asajeffrey
Pushing 4188792b6e7c96af7aca03688762170b751fa8c2 to master...


feature: presets for node and browser

@pollyjs/preset-node
@pollyjs/preset-browser
To abstract away setting up the persisters and adapters that are commonly available in each runtime.  Will simplify the documentation on getting started.

@rahilvora we would always love to have contributions :)

Great, I can try this.

@rahilvora Hey, we just published our preset we're using at Spotify https://github.com/spotify/polly-jest-presets. It's opinionated to our use-case: writing integration tests for web apps while abstracting them from the network communication.
Feel free to give it a shot and give us feedback!
We may add a browser one in the future, too.


[Feature Request] Light color scheme

Title says it, and the code is all there.
Maybe set cookie later so further requests gets served with desired scheme.

Have you seen #11?
Instead of cookies, use a query string (e.g. ?theme=light), as done with Units.


Syriac Resources

Ref: https://github.com/tesseract-ocr/tesseract/issues/654#issuecomment-274269267
syr - Syriac - http://crubadan.org/languages/syr

https://github.com/tesseract-ocr/tesseract/issues/654#issuecomment-274574951

@theraysmith commented 2 days ago
Update: after going back to the www to get fresh data, I believe that my corpus text is now good for:
chr
dzo
iku
snd
syr
tgk
tir
I have put a lot of time into cleaners/filters for languages that use 'virama' characters.
I am not convinced that they are perfect, but I will add the code to the github repo in due course, so experts/native speakers can offer suggestions/fixes to make them better.


Download Video From vidio.com

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.06.22 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Hello, can I request can do download Premium Video from vidio.com?
I've tried using it with Streamfab, it can't download premium video
Notification
https://prnt.sc/mdP-9B6TPIZk
Link to Download
https://www.vidio.com/watch/2102937-ep-03-perasaan-yang-dipaksa
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Show a log of an attempt to download it with yt-dlp

Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

If it's premium content, pass cookies by adding --cookies-from-browser chrome to your command (or replace chrome with the browser you are logged into the site with)


[cocoon] Migrate Flutter dashboard to support Wasm compilation

[x] https://github.com/flutter/flutter/issues/139167
[x] https://github.com/flutter/flutter/issues/139169
[x] https://github.com/codebird/flutter_app_icons/pull/3 - this package is tiny. Might be better to just inline it?

I updated flutter_app_icons last night too: https://github.com/codebird/flutter_app_icons/pull/3

Once dependabot updates google_sign_in_web: 0.12.3 here, the dashboard should compile to WASM like so:

YES!!!

https://pub.dev/packages/flutter_app_icons/changelog – published!

https://pub.dev/packages/flutter_app_icons/changelog – published!

Yes, and cocoon has already picked it up! It's google_sign_in_web that took me a little bit longer to merge because of a couple of CI issues, that is the last piece of the puzzle :P

firebase_core_web still doesn't support --wasm compilation:
Target dart2wasm failed: ProcessException: Process exited abnormally with exit code 64:
*NOTE*: Compilation to WasmGC is experimental.
The support may change, or be removed, with no advance notice.

/usr/local/google/home/dit/.pub-cache/hosted/pub.dev/firebase_core_web-2.11.5/lib/src/interop/utils/utils.
dart:14:1: Error: JS interop library 'package:js/js.dart' can't be imported when compiling to Wasm.
Try using 'dart:js_interop' or 'dart:js_interop_unsafe' instead.
import 'package:js/js.dart';
^
/usr/local/google/home/dit/.pub-cache/hosted/pub.dev/firebase_core_web-2.11.5/lib/src/interop/utils/utils.
dart:15:1: Error: JS interop library 'package:js/js_util.dart' can't be imported when compiling to Wasm.
Try using 'dart:js_interop' or 'dart:js_interop_unsafe' instead.
import 'package:js/js_util.dart' as util;
^

... and more ...

Should be easy to update the pin, no?

Should be easy to update the pin, no?

@kevmoo dependabot should be doing it (?)

https://github.com/flutter/cocoon/blob/9203992c85dc74a8aa79729dbcd4a98819af945e/dashboard/pubspec.yaml#L17-L18

(we need firebase_core: ^2.29.0, I think)

I haven't attempted this in a while!

google_sign_in might be broken on wasm.
I'll make a test repo tonight.

@rayliverified – yeah we're tracking https://github.com/firebase/flutterfire which might be related

Ah, I didn't find it earlier.
This one here: https://github.com/firebase/flutterfire/issues/12819
Do you know of the Firebase Storage issue with not being able to upload Uint8List data? I can file that one if it doesn't exist.

Ah, I didn't find it earlier.
This one here: firebase/flutterfire#12819

sounds related

Do you know of the Firebase Storage issue with not being able to upload Uint8List data? I can file that one if it doesn't exist.

I don't! Please CC me!

google_sign_in might be broken on wasm.
I'll make a test repo tonight.

@rayliverified did you end up creating an issue to google_sign_in_web and wasm?

Firebase storage is fixed, thank you all!
Google sign in Web is still broken. I think there's an issue tracking it
made by someone else maybe.
On Tue, 15 Oct 2024, 6:28 pm David Iglesias, @.***>
wrote:

google_sign_in might be broken on wasm.
I'll make a test repo tonight.
@rayliverified https://github.com/rayliverified did you end up creating
an issue to google_sign_in_web and wasm?
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/139170#issuecomment-2415337356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXZO3SSXK5ICVKL3YVAIFLZ3WQKHAVCNFSM6AAAAAA76LBTMCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJVGMZTOMZVGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ah, I thought it was a firebase problem, but it seems it's a Google Sign In issue, hmm... I'll dive deeper, thanks for bringing this to my attention @rayliverified, I totally had missed that github issue!


Company role change overlay - add osp information inside the role change section

Summary
Update the configuration of the company role change impact by adding OSP details.
If OSP is getting newly added:
Role: no change
Features:

Managed IdPs can get registers
OSP Management Board accessible (Overview of managed IdPs, registered customers & configuration of callback url)

Please also configure with the same information the unselection where those features are getting unassigned/disabled.
UI reference

@nidhigarg-bmw
  "ONBOARDING_SERVICE_PROVIDER": {
    "selected": {
      "roles": [
      ],
      "features": [
        {
          "title": "Managed IdPs",
          "description": "Company Administrator and IT Admins are able to create and manage Managed IdPs for 3rd party registrations."
        },
        {
          "title": "OSP Management Board accessible",
          "description": "Company Administrator and IT Admins can access the new Management Board for OSPs - including an overview of managed IdPs, registered customers & configuration of callback url."
        }
      ]
    },
    "deselected": {
      "roles": [
      ],
      "features": [
        {
          "title": "Managed IdPs",
          "description": "Company Administrator and IT Admins will loose the permission to create and manage Managed IdPs for 3rd party registrations. Note existing Managed IdP connections wont be editable anymore. Make sure you dont have active connections existing."
        },
        {
          "title": "OSP Management Board accessible",
          "description": "Company Administrator and IT Admins will loose the permission to access the Management Board for OSPs - including an overview of managed IdPs, registered customers & configuration of callback url."
        }
      ]
    }
  }
}


Add property: DataFrame.ndim

https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.ndim.html#pandas.DataFrame.ndim
>>> df = ks.range(10)
>>> df.ndim
2

>>> df = ks.DataFrame([[1, 2], [4, 5], [7, 8]],
...                   index=['cobra', 'viper', 'sidewinder'],
...                   columns=['max_speed', 'shield'])
>>> df.ndim
2

Codecov Report

Merging #947 into master will decrease coverage by 2.47%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #947      +/-   ##
==========================================
- Coverage   94.51%   92.03%   -2.48%
==========================================
  Files          34       34
  Lines        6449     6450       +1
==========================================
- Hits         6095     5936     -159
- Misses        354      514     +160

Impacted Files
Coverage Δ

databricks/koalas/missing/frame.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/frame.py
93.56% <50%> (-2.47%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.04% <0%> (-7.32%)
:arrow_down:

databricks/conftest.py
91.48% <0%> (-6.39%)
:arrow_down:

databricks/koalas/groupby.py
89.34% <0%> (-2.05%)
:arrow_down:

databricks/koalas/namespace.py
85.57% <0%> (-1.26%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0%> (-0.96%)
:arrow_down:

... and 3 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 caac367...3ced590. Read the comment docs.

@ueshin
Thanks for the comment!
i think Series.ndim is already existing.
>>> s
NaN    3.0
NaN    1.0
1.0    2.0
2.0    3.0
3.0    4.0
4.0    NaN
Name: 0, dtype: float64
>>> s.ndim
1

then am i going to index.ndim here is okay?


Add PeerId to response types used by {Block,Header}Clients

The response objects used by the client types should include the peer that sent this message, so it can be reported if the message was bad.

And add a on_reputation_change function on the Header/BlockClients traits.


Add workflow to lint the repo.

On a second look we did have it in the "test", but it's better to have as a separate workflow.


RFC: Deprecate make for ESP in favor of cmake

Problem
Currently many esp examples have build files for both make and cmake. The readme files all give directions for building with cmake but the presubmit checks all test make. This can lead to builds being broken if the build files are not kept in sync.

Summary of Changes
This CL attempts to move towards having just cmake.
Looking for feedback from espressif if this is all that's needed or if there are other items I'm unaware of.

Please also add a part of 'how this was tested' to the PR summary.

@rgoliver  Thanks for the PR. Few generic suggestions:

idf.sh files from all the ESP32 examples should be removed.
Documentation of ESP32 examples should be revisited to see if there's any mention of automake commands "idf make " etc.
The CI seems to fail (error - idf.py command not found) because now we need to use . ./export.sh along with ./install.sh from ESP-IDF.
I really appreciate all your help. If these things would be a bit too much, then @sweetymhaiske  can work on a PR and bring this to completion. I had already created an issue for this #6850 after our discussion and she was going to take up this today.

@rgoliver Thanks for the PR. Few generic suggestions:

idf.sh files from all the ESP32 examples should be removed.
Documentation of ESP32 examples should be revisited to see if there's any mention of automake commands "idf make " etc.
The CI seems to fail (error - idf.py command not found) because now we need to use . ./export.sh along with ./install.sh from ESP-IDF.
I really appreciate all your help. If these things would be a bit too much, then @sweetymhaiske  can work on a PR and bring this to completion. I had already created an issue for this #6850 after our discussion and she was going to take up this today.

Thanks for the feedback, sure that would be great if @sweetymhaiske can finish this up! Once this is in, I'll create a PR to compile the pigweed RPCs.

That works. Thanks @rgoliver ! @sweetymhaiske should already be on it.

#7034 is WIP for this


respect include header option

This PR fixes https://github.com/microsoft/azuredatastudio/issues/13217

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.001%) to 41.682%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/query/browser/gridPanel.ts
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/contrib/query/browser/gridPanel.ts
1
2.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
26570

Relevant Lines:
57765

💛  - Coveralls

@abist, are you still working on this one?

@alanrenmsft It should be mergeable, but I didn't make the change Charles was suggesting. I can either merge it now and use instantiation service later, or merge this later if it's not urgent because I'm working on other things right now


Add Bitcichain to Ledger

Description
Adding Bitcichain
Network Name: Bitcichain | BITCI
Chain ID: 1907
SLIP-0044 : https://github.com/satoshilabs/slips/pull/1622
Chainlist.org : https://chainlist.org/chain/1907
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[X] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[ ] Other (for changes that might not fit in any category)

@BitciTeknoloji please fix https://github.com/LedgerHQ/app-ethereum/actions/runs/6691713303/job/18292887350?pr=481#step:4:21

Already added in #516


(docs) 5.2.x release notes

Release notes PR wound up on master. This cherry-picks them onto 5.2.x.

Test PASSed

Can someone confirm that cherry-picking these commits from master is the right way to get them onto 5.2.x and that 5.2.x is the appropriate branch, and merge?

LGTM +1

CLA signed by all contributors.


Azure Language add scriptCode, and enums for script Language Detection Results

Data Plane API - Pull Request

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[x] a public preview
[ ] GA release

Change Scope
Adding a new enum scriptCode to Language Detection response to tell what script the input text was in.
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc: https://github.com/Azure/azure-rest-api-specs/tree/main/specification/cognitiveservices/data-plane/Language/preview/2023-04-15-preview
Updated paths: x.definitions.DetectedLanguage.properties.script, x.definitions.DetectedLanguage.properties.scriptCode

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

fix https://github.com/Azure/azure-rest-api-specs/issues/26778

Also, why is this targeting main? Main is for completed api-versions that should generally not be changed. If 2023-11-15-preview is in development, your team should be working in a feature branch - ideally created in https://aka.ms/openapihub so it gets a baseline commit (we are deprecating the dev/ folder).

What's the purpose of having both the script and scriptCode in a response? If there's a standard for the name mapping, is there much point is providing both? Responses would be smaller, and we don't need to add a bunch of partial names to the custom-words.txt shared by every service?

script was added in a previous version, with some discussion with some of our customers it was determined that mapping the text script to a standard like we did with iso6391Name would be beneficial so we added the new scriptCode. We are keeping script as to not break a customers that want to use the latest preview version that were using the previous preview version.

Also, why is this targeting main? Main is for completed api-versions that should generally not be changed. If 2023-11-15-preview is in development, your team should be working in a feature branch - ideally created in https://aka.ms/openapihub so it gets a baseline commit (we are deprecating the dev/ folder).

The development of this preview version was thought to be completed. This change was a last-minute request by some customers that we determined was important enough to push to this preview version.

/azp run

@maririos should the FalsePositive-BreakingChange label effectively "unrequire" the Swagger BreakingChange check? I'll apply the approval label as a workaround for now.

Requested re-run of the Automated merging requirements met check but doesn't seem like it's kicking off yet. Set to auto-merge if/when it does.

/azp run


make diagnostics startChar 0-based

Pull Request Test Coverage Report for Build 305156

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

Totals

Change from base Build 305038:
0.0%

Covered Lines:
19628

Relevant Lines:
33916

💛  - Coveralls

                var span = item.Span;

We really out to have a Contract.Assert here that the range is valid.
Or even better, factor this into a helper method:
static Range GetRange(string expression, Span span);
And then we could unit test that directly. Looks like there are some interesting case here, especially with multi-line expressions.

Refers to: src/libraries/Microsoft.PowerFx.LanguageServerProtocol/LanguageServer/LanguageServer.cs:432 in 685eee2. [](commit_id = 685eee29b5ef566749e1f51a6c5d186cec1e1dc4, deletion_comment = False)


Added deploy script and fixed package namespace issue

functions package renamed to upload-token.functions
added isHTTPS function to detect http calls from headers
added HSTS header

I'd also recommend waiting until https://github.com/covid19risk/covidwatch-cloud-functions/pull/47 goes in; it will make this PR somewhat simpler.


TOC Sidebar

First of all, thanks for the wonderful theme! I really appreciate your work.
Not sure if this has been asked already, search did not yield any results for me.
Is it possible to add a new sidebar on the right which includes the TOC of the current page (example: https://hugo-book-demo.netlify.app/) ?

Hi, thanks for your request. That's not possible, sorry. I don't like the approach of a three-column design, and removing the ToC sidebar was one of the reasons why I forked the hugo-book theme.
As this was a basic design decision, I'm closing this issue as won't fix and hope you understand that.

Sure, won't question this ;) Thanks for your quick response anyways!


Referencing addins that contain multiple assemblies throws an error

Example:
#addin "RestSharp"

Throws "An assembly with the same simple name 'RestSharp' has already been imported"

Hi believe this is a duplicate of  #373 , issue occurs when nuget is multi target, we need to fix identifying best match dll.


Sample name issues (empty field or record)

The sample name is always empty for AbSciex wiff files (a record is created but with an empty sample name field). Observed on version 0.9.8.
For Thermo raw files, the sample name information is not always retrieved (no sample record in the table) for raw files coming from the same acquisition sequence. This bug has been observed for raw files converted using the version 0.9.7. After verification it seems that this second problem has been observed for thermo raw files converted with an old version of PWIZ-mzDB (version 0.8). It seems to work since version 0.9.5.

Sample name is now returned when available
It is still not functional on Bruker data, maybe it would require an update of ProteoWizard...


Scalar Relations for Active Record

Lets imagine you need a report for the sales, employees, restockngs and aisles for all the shops in a determined city. The query would look something like this.
Shop::find()->select([
    'shop.*',
    'salesCount' => 'count(distinct sale.id)',
    'soldAmount' => 'sum(sale.amount)',
    'employeesCount' => 'count(distinct employee.id)',
    'aislesCount' => 'count(distinct aisle.id)',
    'restocksCount' => 'count(distinct restock.id)',
])->joinWith([
    'sales',
    'employees',
    'aisles',
    'restocks',
    'address',
])->andWhere(['address.city_id' => $city_id])
->groupBy('shop.id');

It will be a huge and slow query with a total of 6 simultaneous tables and consuming a lot of resources. On top of that 'salesAmount' will produce a wildly different result than expected due to the repetition of data implicit with every join. So that data can't be eagerloaded and needs to be calculated with a new query for every shop in the report.
To solve this issue the best approach is to segmentate the results into multiple relational queries per table. I propose a new type of relations that can take advantage of this.
public funciton getSalesCount()
{
    return $this->getSales()->asScalar('count(*)');
}

public funciton getSalesAmount()
{
    return $this->getSales()->asScalar('sum(amount)');
}

public funciton getRestocksCount()
{
    return $this->getRestocks()->asScalar('count(*)');
}

// etc

Which would be used very similar to the existing relations.
Shop::find()->with([
    'salesCount',
    'salesAmount',
    'employeesCount',
    'aislesCount',
    'restocksCount',
    'addressCount',
])->innerJoinWith('address')
->andWhere(['address.city_id' => $city_id]);

Which would produce queries
SELECT shop.*
FROM shop
  INNER JOIN address on shop.address_id = address.id
WHERE address.city_id = 1;

SELECT count(*), shop_id
FROM sales
WHERE shop_id in (1, 2, ..., 100)
GROUP BY shop_id;

SELECT sum(amount), shop_id
FROM sale
WHERE shop_id in (1, 2, ..., 100)
GROUP BY shop_id;

SELECT count(*), shop_id
FROM restock
WHERE shop_id in (1, 2, ..., 100)
GROUP BY shop_id;

/* etc */

The number of queries will increment but the execution time will be reduced drastically and the saleAmount parameter will be calculated properly.
The proposition would be to add a new method yii\db\ActiveQuery::asScalar() which would take an string parameter to determine how to calculate the scalar value. Relations marked as scalar won't produce instances of yii\db\ActiveRecord and will only save the scalar result.

You can achieve pretty much the same:
    public function getSalesCount()
    {
        return $this->getSales()->select(['shop_id', 'count(*)'])->groupBy('shop_id');
    }

Shop::find()->with(['salesCount', ...])
    ->innerJoinWith('address')
    ->andWhere(['address.city_id' => $city_id]);

I'm closing the issue for now because to can not find a good reason to introduce new API. The issue is not locked, so feel free to continue discussion if you have something on mind. Thank you

@SilverFire have you tried that snippet? it will simply not save the 'count(*)' anywhere on the ActiveRecord.
I would need to create a new class for each scalar relation to add it a getter/setter attribute.
And I would have to use it like this
$model->salesCount->count;
$model->salesAmount->sum;

So for the example I wrote above I would require at least 4 new classes just to make one report.

have you tried that snippet?

Nah, it was a concept that does something similar :octocat:

it will simply not save the 'count(*)' anywhere on the ActiveRecord

Well, you can say ->asArray() but then model does not look so shiny.

Then looks useful and could be implemented, as for me.
Let's wait for more thoughts from @yiisoft/core-developers

This topic has already been disscussion several times in several contexts.
See #2179, #8316, #10459

@klimov-paul i have read them all and none of them offers a solution for eager loading. On the example I provided its meant to find several shops and extra data for each shop which would be be executed by several individual queries per row or by one humongous query. Both approaches are not optimal and would consume lots of resources.
@SilverFire then the snippet would look something like this
public funciton getOrdersCount()
{
      return $this->getOrders()->select(['shop_id', 'count' => 'count(*)'])->asArray()->groupBy('shop_id');
      // usage $this->ordersCount['count']
}

Which is what I am doing at the moment but its not practical, intuitive or easy to maintain and loses some functionalities such as attributeLabels on gridviews.
My propsal is to use it like this
public funciton getOrdersCount()
{
      return $this->getOrders()->asScalar('count(*)');
      // usage $this->ordersCount
      // extra select and group by are added by array_keys($query->link) and only for eager-loading.
}

which can take advantage of attributeLabels, attributeHints and similar functionalities

i have read them all and none of them offers a solution for eager loading.

Is that really so? What about this comment then?
https://github.com/yiisoft/yii2/issues/10459#issuecomment-181971912
I will even re-post it here:
class WhateverCount extends \yii\db\ActiveRecord
{
    public function query()
    {
        return (new \yii\db\Query())
            ->select(['related_id', 'counted' => 'count(*)'])
            ->from('tablename')
            ->groupBy('related_id')
            ->asArray(true);
    }
}

class WhateverModel extends \yii\db\ActiveRecord
{
    public function tablename()
    {
        return 'sometable';
    }

    public function getWhateverCount()
    {
        $this->hasOne(WhateverCount::class,['id'=>'related_id']);
    }
}

$rows = WhateverModel::find()->with('whateverCount')->all();
foreach ($rows as $row) {
    echo $row->whateverCount['counted'];
}

Here is your eager loading without one humongous query.

@klimov-paul that approach is very similar to the one I am using already except that it produces the requirement for a class for every relational data. In the provided example I would need  to create 5 new classes just to handle one grid view.
The number of classes on the system would bloat and will be hard to mantain/debug. Will be specially difficult as I start to need more relation data such as classes like would need stuff like ShopSalesCount ShopTotalAmount, ShopBiggestSale, ShopAverageSaleAmount and it would lose functionalities like attributeLabels, attributeHints.
And finally it offers no benefit than making the query like this
public funciton getOrdersCount()
{
      return $this->getOrders()->select(['shop_id', 'count' => 'count(*)'])->asArray()->groupBy('shop_id');
      // usage $this->ordersCount['count']
}

similar this #12025
class OrderCount extends QueryRecord
{
    public static function query()
    {
        return (new Query())
            ->select(['shop_id', 'count'=>'count(*)'])
            ->from('order');
    }
}

class Shop extends ActiveRecord
{
    public function getOrderCount()
    {
        return $this->hasOne(OrderCount::className(), ['shop_id' => 'id']);
    }
}

foreach(Shop::find()->with('orderCount')->all() as $shop)
{
    echo $shop->orderCount->count;
}

@mdmunir the main issue i have with that approach is that it will increase the number of class models inmensely for every time you need new data.
Example  OrdersCount, MaxOrderAmount, TotalOrderAmount, AverageOrderAmount, etc just to give examples, most will be for a very particular use and not very reusable.
Plus its not intuitive/aesthetical to use $shop->orderCount->count instead of $shop->orderCount

Kind of workaround:
use yii\db\BaseActiveRecord;

/**
 * @property integer aggregate
 * @property string aggregate
 */
class DynamicAggregate extends \yii\db\ActiveRecord
{
    public static function find()
    {
        return new DynamicAggregateQuery(static::class);
    }

    public function attributes()
    {
        return ['id', 'aggregate'];
    }

    public static function primaryKey()
    {
        return ['id'];
    }

    public function __toString()
    {
        return $this->aggregate;
    }

    public static function populateRecord($record, $row)
    {
        BaseActiveRecord::populateRecord($record, $row);
    }
}

class DynamicAggregateQuery extends \yii\db\ActiveQuery
{
    public function forModel($modelClass, $alias = null)
    {
        $this->from = [$modelClass::tableName()];
        if ($alias !== null) {
            $this->alias($alias);
        }

        return $this;
    }
}

class Shop extends ActiveRecord
{
    public function getOrdersCount()
    {
        return $this->hasOne(DynamicAggregate::class, ['shop_id' => 'id'])
            ->forModel(Order::class)
            ->select(['shop_id', 'aggregate' => 'count(*)'])
            ->groupBy('shop_id');
    }
}

foreach(Shop::find()->with('ordersCount')->all() as $shop)
{
    echo $shop->ordersCount; // or $shop->ordersCount->aggregate;
}

The simpliest way is using a getter:
class WhateverModel extends \yii\db\ActiveRecord
{
    public function tablename()
    {
        return 'sometable';
    }

    public function getWhateverRelation()
    {
        $this->hasOne(WhateverRelation::class,['id'=>'related_id'])
            ->select(['related_id', 'counted' => 'count(*)'])
            ->from('tablename')
            ->groupBy('related_id')
            ->asArray(true);
    }

    public function getWhateverCount()
    {
        return $this->whateverRelation['counted'];
    }
}

$rows = WhateverModel::find()->with('whateverCount')->all();
foreach ($rows as $row) {
    echo $row->whateverCount;
}

:+1:
Enough ways to solve problem. Suggest to close the issue
@Faryshta ?

@SilverFire the last solution you wrote sounds interesting, can we have a shortcut or standarized way to use it?
public function getOrdersCount()
{
     return $this->agregateRelationalField($this->getOrders()->select('count(*)'));
}

or something like that to get it on the core.
@klimov-paul i think thats not intuitive and has many requirements but seems good enough. Can we get proper documentation on that proposal?

I think we can standarize it with a very small PR.

class ActiveQuery
{
     /**
       * @param string|array the added select
       * @return $this
       */
     public function asRelationalData($select)
     {
         $linkKeys = array_keys($this->link);
         $this->multiple = false;

         return $this->select(array_merge((array) $select, $linkKeys))
             ->groupBy($linkKeys)
             ->asArray(true);;
     }
}

// usage

class Shop extends ActiveRecord
{
     public function getOrders()
     {
          return $this->hasMany(Order::class, ['shop_id' => 'id']);
     }

     public function getOrdersData()
     {
          return $this->getOrders()->asRelationalData([
             'count' => 'count(*)',
             'totalAmount' => 'sum(amount)'
          ]);
    }
          /* this would be the equivalent of
         return $this->hasOne(Order::class, ['shop_id' => 'id'])
               ->select([
                    'count' => 'count(*)',
                    'totalAmount' => 'sum(amount)'
                    'shop_id',
               ])->groupBy('shop_id')
               ->asArray(true);
          */

     public function getOrdersCount()
     {
           return $this->ordersData['count'];
     }

     public function getOrdersTotalAmount()
     {
           return $this->ordersData['totalAmount'];
     }
}

@klimov-paul @SilverFire shall i make a pr?

Docs updated 2250612cb861947c3fcaab91f0d60d116978a88e

В документации ошибка, нужно менять getOrders на hasOne(Order::className(), ['customer_id' => 'id']), иначе на выходе двойной массив и getOrdersCount ничего не возвращает
public function getOrdersAggregation()
{
        return $this->hasOne(Order::className(), ['customer_id' => 'id'])
            ->select(['customer_id', 'counted' => 'count(*)'])
            ->groupBy('customer_id')
            ->asArray(true);
}

Или как вариант править  getOrdersCount

It is already fixed: 34aeb019e51978f8f9982d9dd8bd3ed8798e7217

Магия в запросе какая-то.
            ->select(['customer_id', 'counted' => 'count(*)'])
            ->groupBy('customer_id')

Я конечно понимаю, что 2 раза повторить customer_id нужно для того, что бы activeRecords подхватила и связала это внутри себя, но хотелось бы более явной команды на это.
Собственно именно из-за того что это встретил
return $this->ordersAggregation[0]['counted'];

Мне показалось, что здесь кроется какой-то костыль и поэтому я специально нашел этот баг что бы почитать в чем прикол
Что-то не то с API, слишком сложно. Мое мнение, что надо исправлять это.

@klimov-paul Требуется еще расширить документацию, так как данных может не найдено, тогда будет ошибка "Undefined offset: 0"
Правильнее всего.
return empty($this->ordersAggregation) ? 0 : $this->ordersAggregation[0]['counted'];

ЗЫ: не красивый код, хотелось бы скрыть подобное поведение за фассадом...

https://github.com/yiisoft/yii2/commit/93fbc55b23e816e906c667f84e7258c0719bc35c

Спасибо.


Document local set up and deployment steps

See https://github.com/Call-for-Code/Prometeo-Database/pull/10

Done


Idle screen dimming effect.

It animates for about 10s.
ubuntu
PC-1-screen0.webm

This is very cool. I don't know how to implement this, though.


Optimization of interventions

Hi
Once we have our models and we have determined the important causal drivers of our outcome, we may want to determine the optimal set of interventions. I imagine that this is the step 6 in your diagram:

However, in your paper I don't see any details on this step. Given the nonlinearity of your models I imagine there are interaction effects (in sense that the effect of X2 on the endpoint may change with the value of X1) as opposed of the effects being entirely additive. Therefore, we can't optimize on the interventions of multiple variables independently, they need to be jointly optimized.
Does your framework has a way to optimize the interventions of  multiple variables efficiently? How would you determine whether there is an interaction effect?
thanks
FKG

Hi @fkgruber,
I assume you have a known set of continuous treatment nodes and a continuous treatment effect node that you're interested in. In this case you need to pick a metric to optimise, ATE is probably a good place to start. There is no analytic way to do this for a general non-linear model, so you'll have to estimate it by sampling.
The SEM class doesn't currently have an rsample method, but you could add one by copying the sample method and replacing the sample calls with rsample ones. Both of the continuous noise distributions (spline and gaussian) have rsample methods.
You can then maximise the monte carlo estimate of the ATE wrt the treatments, since the gradients will flow through rsample.
To estimate the interaction effects, you can calculate the CATE, conditioning on different values of X1. General CATEs aren't supported by this version of the model, but for root nodes like X1, interventions and conditioning are equivalent.


Doubled Fields in string.xml-Files

Hello everybody,
i just checked out your code and i got errors in almoste every string.xml that there are doubled value names. Maybe there wend something wrong.
The doubled fields are:
menu_send
menu_delete
menu_move
menu_copy
menu_compress
menu_rename
menu_extract
menu_details
Hope this helps.
Btw, Great work :+1:

fixed by https://github.com/openintents/filemanager/commit/2f5e8c8ab926da94aaedfdc45f155d6601b621d0


Exclude examples from artifacts

I noticed that the examples from the main source tree appear in the release artifacts. Assuming that this is not intended, I have setup a patch that prevents the classes from the org.jsoup.examples package from being included in the *.jar and *-sources.jar artifacts.
I think an alternative approach would be to move the examples to a separate source tree (or different project). What do you think?

Thanks! Merged.


Thank you

Thank you for this repo.
I spent a solid 20 minutes searching for how to get started quickly with a typescript module. I eventually found this package, and it's freakin awesome. I'm writing code now.
To the authors and contributors, thank you kindly. You rock.

Agree - thank you much!


using GzDecoder in a loop

Hey I was wondering if my code is inefficient and if there is a way of reusing the GzDecoder instead of calling ::new() every iteration.
    loop {

        let bytes = client.read_message().into_bytes().unwrap();

        let mut msg = String::new();

        GzDecoder::new(bytes).read_to_string(&mut msg).unwrap();

        println("{}", msg);

    }

I tried instatiating a decoder outside of the loop but ran into multiple errors.
Something like this:
    let decoder = GzDecoder::new();
    loop {

        let bytes = client.read_message().into_bytes().unwrap();

        let mut msg = String::new();

        decoder.read_to_string(&mut msg).unwrap();

        println("{}", msg);

    }

I'd recommend profiling to see if it's sufficient for your use case. If it's not then helper methods can be added to GzDecoder to reuse it.


data binding to "is" attribute not working

I'm trying to dynamically via dom-repeat display a number of custom elements.  I have a data array similar to the one below called "names" where the el_name property contains the name of the custom element to display.
names = [ {el_name: 'tmpl-one'}, {el_name: 'tmpl-two'} ]

When I try to dynamically assign this variable to the "is" attribute it does not properly load the custom element but instead gives me an empty div.
<template is="dom-repeat" items="{{names}}">

</template>

I also tried this which didn't work

When I statically type in the name of the template it loads the custom element fine.
<template is="dom-repeat" items="{{names}}">

</template>

tmpl-one.html -
<dom-module id="tmpl-one">
<template>
  <paper-icon-item>...</paper-icon-item>
</template>

</dom-module>

names should have a JSON format try this out:
names = '[ {"el_name": "tmpl-one"}, {"el_name": "tmpl-two"} ]'

I assume this is not supported
http://w3c.github.io/webcomponents/spec/custom/#instantiating-custom-elements

After a custom element is instantiated, changing the value of the is attribute must not affect this element's custom element type.

@MeTaNoV that didn't work either.
@zoechi I guess that could be the problem.  Do you know of a way to insert a custom element via dom-repeat instead of via javascript?

@rhasson
I'm using Polymer from Dart. I build an element for that ;-)
https://github.com/bwu-dart/bwu_bind_html/blob/master/lib/bwu_bind_html.dart
Shouldn't be to hard to port it to JS (such an element probably already exists for JS)
This way you can add an HTML string in dom-repeat.

@zoechi after some experimentation I believe your approach is the best
option.
Thanks
Roy
On Wed, Dec 2, 2015 at 7:45 AM Günter Zöchbauer<EMAIL_ADDRESS>wrote:

@rhasson https://github.com/rhasson
I'm using Polymer from Dart. I build an element for that ;-)
https://github.com/bwu-dart/bwu_bind_html/blob/master/lib/bwu_bind_html.dart
Shouldn't be to hard to port it to JS (such an element probably already
exists for JS)
This way you can add an HTML string in dom-repeat.
—
Reply to this email directly or view it on GitHub
https://github.com/Polymer/polymer/issues/3122#issuecomment-161281053.

That's right. Changing is after the fact is not supported by the spec.


Mobile QQ browser browsing web failure

I created an application directly with create react app, and started the application.
The web page can run on PC, but the browser fails in the mobile browser

You are most likely using an ES6 runtime feature that's not polyfilled. You'll need to get the error somehow (maybe try running in IE 11 with IE 9 compatibility mode or something).
Unfortunately there's not much we can do to help otherwise.


Storm 2063 thread name in worker log improvements

@d2r did the original work for this.

+1
Also nice catch for storm-submit-tools!


Fixing broken example Makefile due to missing gpr dependency

grpc/examples/cpp/helloworld/Makefile seems to be missing -lgpr
make all seems to fail with the following errors:
/usr/bin/ld: greeter_async_client.o: undefined reference to symbol 'gpr_log'
//usr/local/lib/libgrpc.so.0: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
make: *** [greeter_async_client] Error 1

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

It also have no -lgpr in examples/cpp/route_guide/Makefile, but it can make right, why?


unprivileged user

Is this the same issue as the 'whoami' issue? Easy to fix the below for a particular user, but a more generic solution would be best.
TASK [Setup kubeconfig for vagrant user] ***************************************
changed: [k8s-master] => (item=mkdir -p /home/vagrant/.kube)
changed: [k8s-master] => (item=cp -i /etc/kubernetes/admin.conf /home/vagrant/.kube/config)
changed: [k8s-master] => (item=chown vagrant:vagrant /home/vagrant/.kube/config)
 [WARNING]: Consider using the file module with state=directory rather than
running 'mkdir'.  If you need to use command because file is insufficient you
can add 'warn: false' to this command task or set 'command_warnings=False' in
ansible.cfg to get rid of this message.

 [WARNING]: Consider using the file module with owner rather than running
'chown'.  If you need to use command because file is insufficient you can add
'warn: false' to this command task or set 'command_warnings=False' in
ansible.cfg to get rid of this message.

TASK [Install pod network] *****************************************************
changed: [k8s-master]

TASK [Generate join command] ***************************************************
changed: [k8s-master]

TASK [Copy join command to local file] *****************************************
fatal: [k8s-master]: FAILED! => {"msg": "Failed to set permissions on the temporary files Ansible needs to create when becoming an unprivileged user (rc: 1, err: chown: jasonpoley: illegal user name\n}). For information on working around this, see https://docs.ansible.com/ansible/become.html#becoming-an-unprivileged-user"}

RUNNING HANDLER [docker status] ************************************************

PLAY RECAP *********************************************************************
k8s-master                 : ok=15   changed=14   unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

closed


PR: Fix a couple of bugs in the Find plugin

Description of Changes

Catch any error when trying to create file items.

Prevent showing file entries without any match. This was happening sometimes when the maximum number of results is reached.
For example, searching for self in the main Spyder repo when the max results is 15 was giving this

Now it gives this

Issue(s) Resolved

Fixes #17443
Fixes #16256
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @ccordoba12

Good point. I thought about that but it's bit hard because this bug depends on the number of files and matches.
But I'll try to use the files we have for testing to see if I can come up with something.

@dalthviz, the test I added in commit 4fe2fc7 fails in 5.x, in the way described on issue #16256, and works here. I also made a small change in commit 7d1d7df to avoid generating a test directory inside the sourcecode tree.

Hello @ccordoba12! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/findinfiles/widgets/search_thread.py:

Line 292:21: E261 at least two spaces before inline comment
Line 292:22: E262 inline comment should start with '# '


Automate the deployment of metrics dashboard

1. Issue, if available:
Clean up the metrics dashboard.
Automate the deployment.
2. Description of changes:
3. How was this change tested?
4. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[X] No

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

Needs rebase


Scanner not detecting books in root folder

For files in the root folder (i.e. the folder where the top of the library is located), files not in folders are not being included in the scan.
I am unsure if this is intended or unintended behavior, but for someone with a non-standard file ordering, this is limiting for the utility of stump.
I would hope that the scanner would find all acceptable files in the library and display them as available.

@Chizi123 thanks for the report! This is unintended, I just didn't account for that library structure.

The fix for this went into the develop branch, I tested it briefly and it seemed to work. I'll test a little more thoroughly this weekend, but will keep this issue open until it is in main.


Don't #include <ostream> outside of test-only code.

On Android/ARM64, Release config, size of iree-run-module (after stripping):
Before: 794424
After: 591472

@benvanik substantial conflict with #10433 which expanded usage of <ostream> around this area, PTAL again.


feat: prompt for git

Feat - Prompt for Git
Issue #587 - Creating a project within an existing repository causes a fatal git error
Description
This pull request will prompt the user to choose a version control system.

There's two main reasons for this:

Using npx create-nuxt-app in an existing repository fails, as two .git directories exist. This is especially prevalent in our projects where we have a Django backend and Nuxt frontend in the same repo.
Several types of VCS exist (e.g. SVN), and users shouldn't be forced to delete unwanted directories in order to implement their preference.

This feature has been designed to easily allow another VCS to be added to the list.
Changes
The prompt.js asks for a VCS preference in the form of a list.

The saofile.js will now only run gitInit() if the respective choice is selected.

Tests pass on my machine 😣

Re-linting directly before pushing did the trick. Life of a Windows dev 😁

I'm now realising that these tests are auto-generated 😂

All good to review now 😜

thank you. this was my first open source contribution to a mainstream repo, so cheers for the good experience 😼


Example for user-defined temporal dataflow

Hello,
Thanks as always for open-sourcing and maintaining the great work. I'm wondering if there are any examples available for user to provide a fixed/constrained temporal mapping? Our use case would be that we want to provide some constraints on temporal loop orderings and allow Zigzag to tile/assign loop bounds as it sees fit based on the on-chip SRAM sizes, etc. Some accelerators do not have a lot of flexibility in case of temporal loop orders and we would like to evaluate those.
Is this achievable on Zigzag now? I saw a few pull requests #75 #69 that seem to be relevant but I'm not sure about the correct usage. Any help/guidance on this would be much appreciated.
Thank you!

Hello!
Your description sounds exactly like the feature that has been added in https://github.com/KULeuven-MICAS/zigzag/pull/75, but the examples and tests that use the constrained mapping seem to be missing from the master branch. I will get back to you with an example input ASAP!

Also, you can refer to https://github.com/KULeuven-MICAS/zigzag/pull/82, where we're working on integrating the tests using constrained temporal mapping.

A gentle ping on this thread. I tried the constrained mapping yaml in #82, but they are not working for me. Would appreciate any help!


Not rendering :::info sections correctly

It seems that the :::info sections in the docs aren't being handled correctly. For example the :::info section in the time docs is just being displayed as :::info Some info :::.

I'll make a PR for that. I updated a few pages and noticed that the search still has the old results. Do you know of anyway to reindex the pages or clear the search cache?

The PR deploy preview doesn't seem to have the weird search index issue so I guess I was just seeing old search cache stuff ¯\_(ツ)_/¯


chore(deps): bump

Applied mkpr on undefined
/tmp/mkpr.sh

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


”Rebricea” is listed in the drop-down menu

Description:
"Rebricea" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Rebricea" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Rebricea, VASLUI".


K8sVer check is invalid for kube-proxy

https://github.com/kubernetes-incubator/kube-aws/blob/master/core/controlplane/config/templates/cloud-config-controller#L3211
{{if ge .K8sVer "v1.9"}}kubeproxy.config.k8s.io{{else}}componentconfig{{end}}/v1alpha1

The syntax here does not work for 1.10 because of the string validation
This seems to work https://github.com/mcuadros/go-version
PR proposal on its way

/assign @kevtaylor

/kind bug

@mumoshu @c-knowles This was from testing a roll up to 1.10 and which breaks the kube-proxy if you attempt it as is. Once this was fixed I successfully rolled to 1.10 - so when this fix is merged (assuming you like it!), I'll put the changes in for a potential upgrade to 1.10


Add overlay methods like react-refresh-webpack-plugin

The "official" react-refresh webpack plugin supports apis to handleErrors during fast refresh.
See https://github.com/pmmmwh/react-refresh-webpack-plugin/blob/main/docs/API.md#module
It would be great if this option was available in the webpack plugin as well. Gatsby uses this to show error overlays. If someone installs gatsby-plugin-preact, fast refresh works great but they have to look in their terminal or devtools to see their errors.
Repro:
npx gatsby new preact-error-overlay https://github.com/gatsbyjs/gatsby-starter-default
yarn add gatsby-plugin-preact

// add gatsby-plugin-preact to your gatsby-config.
// make an error in your code

APIs fast-refresh supports:

handleRuntimeError
clearRuntimeErrors
showCompileError
clearCompileErrors

If you're open for contributors, let me know. I'm happy to create a PR for this

Open to contributions, if you don't get the time I might look at it later this or next week 👍


Takes additional tap to show keyboard on iPhone in web version.

The first tap focuses a textfield you can see the cursor becomes active. The second tap brings up the keyboard. I have noticed this problem in my own flutter web app.  See attached video. This is browsing the web version of the app on iOS.
I would expect the first tap where the form field gets focus and the cursor blinks to bring up the keyboard.
https://user-images.githubusercontent.com/638246/110120257-6d6ccf00-7d82-11eb-9afa-8dd35c0b5cbe.mov

Could you please log this here with a minimal reproduction case?
https://github.com/flutter/flutter/issues/
If you are able to reproduce it in your own code, it is likely not an issue with the app, but we'd love to track this issue against the main one.

I will add it once I am working on a web version of my flutter app again.  I am currently working on another project.  If I have some free time I will try to make a reproduction.


iPad : no cancel button after hitting the plus button

To reproduce :

Unlike the iphone, there is no cancel button after hitting the start a new chat"plus" button

iOS 9.3.1

Just press the + button again or tap anywhere else then on the menu that pops up.