[DOC] Update documentation and website

Closes None.

Changes proposed in this pull request:

Use RTD theme, but with custom color and with logo at top of sidebar.
Reduce Documentation page to be more of a general summary than a restatement of the API.
Add warnings to docstrings of not-yet-implemented classes and functions.

This is relatively minor, so I'll just merge it once tests pass.


支持通过loadSerice和loadController，实现星链模型

完成了并发测试，保证了chumi的健壮性

Codecov Report

Merging #17 (07cd33d) into master (ec9d6ff) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 07cd33d differs from pull request most recent head 21a44d9. Consider uploading reports for the commit 21a44d9 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##            master       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          420       450   +30
  Branches       107       117   +10
=========================================
+ Hits           420       450   +30

Files Changed
Coverage Δ

chumi/constants.ts
100.00% <ø> (ø)

chumi/Controller.ts
100.00% <100.00%> (ø)

chumi/Service.ts
100.00% <100.00%> (ø)

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


change JS package, it's use soul-dashboard pr #27

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

@Zhoutzzz 你需要在soul-dashboard执行npm run build打包，然后把index.html、css、js拷过来替换，不是删除哈

Codecov Report

Merging #410 (f0a4c8f) into master (5e459ce) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #410   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines          62      62
  Branches        1       1
======================================
  Misses         62      62

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 5e459ce...f0a4c8f. Read the comment docs.


release: Add transport to goreleaser

Add git-remote-gittuf to releaser. We don't need to add it to brew install just yet, but it'll make piloting the transport easier with folks who don't work with Go / have older OS-es with versions of Go we can't use.

We can ignore coveralls for now, I think: https://github.com/coverallsapp/coverage-reporter/issues/165


[Bug] Fix ut bug

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #10078
Problem Summary(Required) ：

Fix ut bug
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


Where is blazor united docs in .net 8 preview 1?

[Enter feedback here]
Where is blazor united docs in .net 8 preview 1?
Please get help for load blazor pages on Auto mode.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 5bc28995-6784-a7d2-5c0b-9811a8d19c1a
Version Independent ID: 1668c8fb-024c-c86d-d4d5-e22e3355c04b
Content: ASP.NET Core Blazor
Content Source: aspnetcore/blazor/index.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @mammadkoma ... Not available at this time. We can't document much on new features during early previews because the product unit churn is quite high, the new features aren't usually stable/safe to use during early previews, and .NET 8 previews this early aren't available in an SDK format. For news during early previews, keep an eye out for Dan Roth's blog posts ...
https://devblogs.microsoft.com/dotnet/author/danroth27/

So at least write the necessary code for adding the Auto attribute in the article blog. We only see a title "blazor united" and a description! If we can use it in .net 8 get a single code line.

We can't document new features at all in early previews when the new features aren't even settled on their basic design. The product unit is changing and adding features to Blazor United on a daily/weekly basis right now, and we have no way to keep up with that degree of churn in the feature in early previews.
To use Blazor United at this point, follow the product unit's issues and pull requests starting with the main issue at https://github.com/dotnet/aspnetcore/issues/46636.
We should have documentation up for it after the design has settled. We typically start providing documentation for a new release around the 5th preview in approximately June of each year. Near the first release candidate in September, we typically have most of the new features covered.

I downloaded .net 8 preview 1 and create a project a blazor by "dotnet new blazorwasm" but it doesn't build and run !

I'm not sure why that would happen ... and you didn't state what the error(s) are.
If you post the error(s) that you're seeing from the console, I'll take a look, but I can't guarantee that I'll be able to assist. You can search the issues on the product unit's repo to see if the problem has been reported ...
https://github.com/dotnet/aspnetcore/issues
If the error that you're seeing hasn't been reported, you can open a new issue for them there.


[ENH] Merge MiniRocket and MiniRocketMultivariate

dependent on #1698
merge these two transformers, part of #1699, deprecate multivariate version

Is there a specific reason why you introduce two versions of fit/transform (multi, uni) ? Couldn't we use the multivariate code with n_channels=1 ?
Otherwise, the rest looks good to me at first glance.

Is there a specific reason why you introduce two versions of fit/transform (multi, uni) ? Couldn't we use the multivariate code with n_channels=1 ? Otherwise, the rest looks good to me at first glance.

because its very complicated and I was looking for the simplest solution. Anything more can be done after imo

Is there a specific reason why you introduce two versions of fit/transform (multi, uni) ? Couldn't we use the multivariate code with n_channels=1 ? Otherwise, the rest looks good to me at first glance.

I'll try it though


fix TagBot tokens

This is why TagBot is failing to create a new tag for the initial release.

I’m not sure we have a documenter key setup here. I think maybe https://github.com/beacon-biosignals/SlackThreads.jl/pull/8#discussion_r787300900 wasn’t the right change and we wanted that token (which is configured for this repo). But I don’t actually understand anymore what tokens are for which thing.

Seems like #16 did the trick


Don't establish CASE with every device

Problem
The way the device commissioner node update function works, it
establishes a CASE connection with every operational node it gets
an advert from. That's not what we want to do.
Change overview
The code path most in use to create a CASE session is to call
UpdateDevice through the DeviceController. Instead of calling
FindOrEstablishSession for every node that get resolved, only
request callbacks for sessions that have already been requested.
Testing
Previously, if you used chip-device-ctrl to commission a device, then triggered an operational advert, the controller would attempt to establish a case connection. I now now longer see the case connection establishment messages.

I think some of @andy31415's recent changes might have made this obsolete in the sense that the problem no longer exists...  @cecille

@cecille can this be closed now that latest code uses addressresolver?

Sorry all - was OoO last week. I THINK this might be obsolete now, but @andy31415 - just want to confirm - does the address resolver pass ONLY call the callback for resolves that were requested? or are they just filtered?

Sorry all - was OoO last week. I THINK this might be obsolete now, but @andy31415 - just want to confirm - does the address resolver pass ONLY call the callback for resolves that were requested? or are they just filtered?

I believe you are correct.

Yeah, I think Andrei's changes will fix the issue of CASE connections to every wayward mdns response.
I'm still not a big fan of the API here - the interaction between the DeviceProxy and the commissioner is confusing, especially since it's different between the comissioner and controller in really not obvious ways.
For now, I'm going to leave this open because I'd like to squeeze in a cleanup here to just make the behaviour more obvious. But very low priority. If I don't get a chance to swing back to this before Friday, I will just close.


Use numbers for width and strokeWidth properties

The order example is using a number for strokeWidth.
Now width and strokeWidth properties must be an Expression object, but we should allow also numbers.

Closed by: https://github.com/CartoDB/renderer-prototype/pull/166


fixed issue-#1404

Fixed: the addition of the score of the previous game.
Fixed: Now  the score will be reset when the activity is close then reopen.
Final Result will work like this:
https://github.com/llaske/sugarizer/assets/99260988/074c9858-bb33-4c70-b37c-262760738de0

@llaske  Please review this.

Now useless, see https://github.com/llaske/sugarizer/issues/1404#issuecomment-1811316149


Is the default server-level collation specific to the locale settings on that machine?

The server collation is specified during SQL Server installation. Default server-level collation is SQL_Latin1_General_CP1_CI_AS.

However, as per https://www.sqlservercentral.com/forums/topic/default-server-collation-for-sql-does-it-depend-on-the-os-version-standard-vs-enterprise it appears the actual default collation on install depends on the locale on the machine that MSSQL is being installed. I know the above pertains to SQL 2008 and is probably out of date, but I just wanted to confirm that the behaviour documented is the actual behaviour. If it is, can a note please be added to update the text to something like:

The server collation is specified during SQL Server installation. Default server-level collation is SQL_Latin1_General_CP1_CI_AS, regardless of the server's locale setting.

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

ID: 3ca70403-237f-0160-7d1f-5817e40b5d25
Version Independent ID: 9993116a-bcec-e054-1cbe-82c3ad603ef6
Content: Set or Change the Server Collation - SQL Server
Content Source: docs/relational-databases/collations/set-or-change-the-server-collation.md
Product: sql
Technology: ****
GitHub Login: @stevestein
Microsoft Alias: sstein

@IanKemp -- Ian, thank you for your feedback.
@stevestein -- Steve, please look into this issue #4103.

Hello @IanKemp  (and @stevestein ): I just submitted PR #4747 to address this issue (as well as Issue #2524 ). Essentially, yes, the default (which can be overridden) is (or, at least, absolutely should be) based on the OS locale. I clarified that the SQL_Latin1_General_CP1_CI_AS collation is the default (albeit a horrible default that needs to be changed!) when the OS locale is set to US English (en-US). I also included a link to the list of OS locales-to-collation mappings: https://docs.microsoft.com/en-us/sql/relational-databases/collations/collation-and-unicode-support#Server-level-collations .
Take care,
Solomon...
https://SqlQuantumLift.com/
https://SqlQuantumLeap.com/
https://SQLsharp.com/

@IanKemp - thanks for raising this issue.
@srutzky - thanks for responding and updating the related docs!

PR #4747 should get merged and go live later today or tomorrow.
#please-close


Identify your own blood group compatibility.

This feature on the Homepage will let the user know all the compatible blood groups for their own blood group.
Example:

Recipient Blood Type
Matching Donor Blood Type

A+
A+, A-, O+, O-

Though I am confused should I make it a functionality (like take the input from the user and then show the data) or just show the data in the table format?
But I will figure it out and will do something that is easy for the user.

@Saurav-Pant Please assign the tag. So that I can start it.

Thanks for raising this, Go ahead @VanshSh


Add UI or UX Design YouTube Channels Resource

Our UI/UX design resources currently lack youtubers. To cater to different learning preferences, your task is to find and add informative UI/UX design YouTube channels to our resources list. These channels should offer tutorials, design critiques, industry insights, or any other valuable content related to UI/UX design.

Please make sure that you read the Contributing guidelines before raising a pr

@jfmartinz I want to work on this issue, Can you please assign this to me sir/ma'am?

You can start working on the issue bro and then I will add the labels in the PR

@jfmartinz I want to work on this issue, Can you please assign this to me sir/ma'am?


Android platforms announcement

Ideally I'd like to publish this on Friday, November 17. I can change the date of the file, but do I also need to hold back merging this PR until that date?

Ideally I'd like to publish this on Friday, November 17. I can change the date of the file, but do I also need to hold back merging this PR until that date?

Yes, do not merge if you don't want it to go live.

Current plan is to merge this on Friday, November 17, but I'm happy to keep taking suggestions until there.

Actually, if this is ready to go, I'd like to move it up and publish today. Anyone have an objection?

SGTM


General Acceptance Criteria

General Acceptance Criteria
The Website Must Meet Accessibility Standards. This Is To Be Achieved By:

Semantic HTML Elements Used Throughout The Source Code
HTML Elements Following A Logical Structure Independent Of Styling And Positioning
Image And Icon Elements Containing Accessible "alt" Attributes
Heading Attributes Falling In Sequential Order
Title Elements Containing A Concise, Descriptive Title

List Of Semantic Elements

<article> = This Element Specifies Independent, Self-Contained Content (Makes Sense On Its Own)
<aside> = This Element Specifies Content Aside From The Content It Is Placed In (Indirectly Related)
<details> = This Element Specifies Additional Details That The User Can View Or Hide
<figcaption> = This Element Specifies A Caption For A <figure> Element
<figure> = This Element Specifies Self-Contained Content Such As Illustrations/Diagrams/Photos Etc
<footer> = This Element Specifies A Footer For A Document Or Section
<header> = This Element Specifies A Header For A Document Or Section
<main> = This Element Specifies The Main Content Of A Document
<mark> = This Element Specifies Marked/Highlighted Text
<nav> = This Element Specifies Navigation Links
<section> = This Element Specifies A Section In A Document
<summary> = This Element Specifies A Visible Heading For A <details> Element
<time> = This Element Specifies A Date/Time

Document And Website Structure
Basic Sections Of A Website:

Header = This Is Usually A Big Strip Across The Top With A Big Heading, Logo And Perhaps A Tagline. This Usually Stays The Same From One Web Page To Another.
Navigation Bar = This Usually Contains The Links To The Site's Main Sections; Usually Represented By Menu Buttons, Links Or Tabs. This Content Usually Remains Consistent From One Web Page To Another.
Main Content - This Is A Big Area In The Center That Contains Most Of The Unique Content Of A Given Web Page.
Sidebar = This Usually Contains Some Peripheral Information, Links, Quotes, Ads Etc. Usually This Is Contextual To What Is Contained In The Main Content, But There Are Scenarios Where You Will Find Some Recurring Elements Like A Secondary Navigation System.
Footer = This Is Usually A Strip Across The Bottom Of The Page That Generally Contains Fine Print, Copyright Notices Or Contact Information. It Is A Place To Put Common Information But Usually, That Information Is Not Critical Or Secondary To The Website Itself.

To Implement This In Terms Of Semantics, HTML Provides Dedicated Tags That Can Be Used To Represent These Sections:

Header = <header>
Navigation Bar = <nav>
Main Content = <main>, With Various Content Subsections Represented By <article>, <section>, and <div> Elements
Sidebar = <aside>; often placed inside <main>
Footer = <footer>

Alt Attributes
Best Practices:

Do Not Include "Image" Or "Image Of .." In The alt Description. Screen Readers Will Already Inform Users That What They Are Hearing Is An Image. However It Is Often Helpful To Indicate The Type Of Image, Such As "Screenshot Of ..." Or "Illustration Of ...".
Put The Most Important Information At The Beginning of The alt Description So That Power Users Do Not Have To Listen To The Entire Description Every Time That They Reach A New Image.
Keep alt Descriptions Short And Concise To Provide A Clean And Efficient User Experience. Be Careful Not To Provide Too Much Information. There Is No Technical Limit To The Length Of alt Descriptions, But A Maximum Of 120 Characters Is A Good Rule Of Thumb For The Sake Of Screen Reader User UX.
Minimise Exaggerated Promotional Language And Details Within alt Descriptions. Instead, Place That Context As Visible Test On The Page For All Users. Essentially, Try To Keep Descriptions Focused On What Is In The Image Itself, And Use The Surrounding Content To Contextualize It Into A Larger Strategy.
When In Doubt, Give The Image An alt Description. The Worst That Can Happen Is A Little Extra Clutter For Screen Reader Users.

Functional Images

Functional Images Are Those Used Inside Links Or Buttons In The Place Of Text. These Types Of Images Should Be Given An alt Attribute That Describes The Function Of The Link, Not The Contents Of The Image.

Decorative Images

Decorative Images Only Serve As Visual Decorations For Elements On A Page And Do Not Add Additional Information To The Surrounding Text. Images Can Also Be Considered Decorative When They Only Illustrate Adjacent Text Without Adding Information Or Are Identified And Fully Described By The Surrounding Text.

Informative Images

Informative Images Are Those Used In Place Of Text To Convey A Single, Very Clear Concept. These Kinds Of Images Are Useful For Saving Space, Conveying Emotions And Thoughts And Providing Visual Instructions.
These Images Should Be Given An alt Description That Conveys The Meaning Of The Content That Is Displayed Visually, Which Isn't Necessarily A Literal Description Of The Image.

Sources:

https://accessible360.com/accessible360-blog/2020-11-10-the-art-of-alt-attributes/#:~:text=Deciding what to include and,they navigate through a webpage.

Heading Attributes
Usage Notes

Heading Information Can Be Used By User Agents To Construct A Table Of Contents For A Document Automatically.
Do Not Use Heading Elements To Resize Text. This Should Be Done With The CSS font-size Property.
Do Not Skip Heading Levels: Always Start From <h1>, Followed By <h2> And So On.

Avoid Using Multiple <h1> Elements On One Page

While Using <h1> Elements On One Page Is Allowed By The HTML Standard (Providing They Are Not Nested), This Is Not Considered a Best Practice.
A Page Should Generally Have A Single <h1> Element That Describes The Content Of The Page (Similar To The Document's <title> Element).

Accessibility Concerns
Navigation

A Common Navigation Technique For Users Of Screen Reading Software Is Jumping From Heading To Quickly Determine The Content Of The Page. Because Of This, It Is Important To Not Skip One Or More Heading Levels. Doing So May Create Confusion, As The Person Navigating This Way May Be Left Wondering Where The Missing Heading Is.

Sources

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements

Accessible Title Elements
Providing Unique, Concise HTML Page Titles Helps Users With Disabilities Quickly Understand A Web Page's Content And Purpose. Well-Written Page Titles Are Critical To Users With Visual Disabilities Because They Are Always The First Page Element Announced By Screen Reading Software.
Benefits

These Measures Allow Users With Visual Disabilities To Quickly Determine If They Are Interested In A Page's Content. They Also Provide A Descriptive Name For Bookmarking The Page, Which Doubles As An Accurate Description For Search Results.

Best Practices

Provide A Concise Page Title That Accurately Describes The Pages' Content And Purpose.
Make Sure The Page Title Is Unique To The Site And Any Related Sites.
Place The Page Description Text First, Followed By The Site Name(s)
Use JavaScript To Change The Title To Appropriately Reflect Page Content For Pages Where The Main Content Changes Dynamically

Checking For Correct Usage

Determine The Page Title. Inspect The Browser DOM For A Non-Empty Title Element In The <head> Section.
In Some Web Browsers, Hovering The Mouse Cursor Over The Pages' Tab Will Display The Full Page Title As A Pop-Up.
Verify That The <title> Element Describes The Pages' Purpose And Content.
Verify That The <title> Element Is Unique Compared To All Of The Site's Pages

Sources

https://accessibility.iu.edu/creating-content/web-content/titles.html

Development Updates (21/11/2022)
Completed:

Updated <title> Element = "Our Services - Horiseon"
Added <header> Element And Edited CSS To Keep Styling In Line With Example
Added <nav> Element And Edited CSS To Keep Styling In Line With Example
Tidied Up <ul> And <li> Coding To Reduce Overall Lines Of Code
Added alt Attributes To The Navigational Links
Added alt Attribute For Main Decorative Image (Left Blank To Not Confuse Screen Readers)
Added <main> Element
Changed <div> To <section> For "Main Services"
Added alt Attribute For SEO Section
Added alt Attribute For Image In SEO Section
Added alt Attribute For Online Reputation Management Section
Added alt Attribute For Image In Online Reputation Management Section
Added alt Attribute For Social Media Marketing Section
Added alt Attribute For Image In Social Media Marketing Section
Commented HTML Code Up Until The Point Above.
Commented HTML Code To Indicate Where To Continue From

Things To Look Into:

Alternatives To <span> Element And How This Affects Accessibility (Company Name/Logo)

Development Updates (22/11/2022)
Completed:

Changed Main <div> Element For The "Benefits" Section To An <aside> Element
Updated Corresponding CSS Selector To Be In Line With The New <aside> Element
Changed The <aside> class To Be An alt Attribute And Renamed "Benefits" To Be "Advantages-Of-Working-With-Horiseon"
Changed The <div> Element To A <section> Element For The Lead Generation Section
Changed The Class Name Of "Lead Generation" Section From "benefits-lead" To "Lead-Generation"
Updated Corresponding CSS Selectors To Be In Line With The New class Called "Lead-Generation"
Added A Blank alt Attribute For The "Lead Generation" Icon As The Image Is A Decorative And Provides No Context Or Meaning
Changed The <div> Element To A <section> Element For The Brand Awareness Section
Changed The class Name Of Brand Awareness Section From "benefits-brand" To "Brand-Awareness"
Updated Corresponding CSS Selectors To Be In Line With The New class Called "Brand-Awareness"
Added A Blank alt Attribute For The "Brand Management" Icon As The Image Is A Decorative And Provides No Context Or Meaning
Changed the <div> Element To A <section> Element For The Cost Management Section
Changed The class Name Of Cost Management Section From "benefits-cost" To "Cost-Management"
Updated Corresponding CSS Selectors To Be In Line With The New class Called "Cost-Management"
Added A Blank alt Attribute For The "Cost Management" Icon As The Image Is A Decorative And Provides No Context Or Meaning
Changed The <div> Element To A <footer> Element For The Footer Of The Web Page

Final Update On Issue
Closing This Issue As All Points Of The General Acceptance Criteria Has Been Met, Committed And Pushed To The Repository.


default export broken

index.js in root of project folder is trying to require ./js/index.js
module.exports = require('./js')

but ./js no longer contains an index.js file.
both 4.x and 3.
(edit) Whoops, sorry you are correct, 3.x does a ./js/index.js file

Looks like it is in the v3 version still, not sure how got remove from 4. I’ve added it back and will release with a fix for auto resize soon.

Fixed in v4.0.2, thanks for raising the issue.


FBTC voucher is always redeemed in Savings account

The FBTC voucher got redeemed in savings account, even though checking was selected (default) at the time of redeem.

@abdulmuneer22 - You can take help from @Parsh on this one

Was not able to recreate it in 1.1.1(98)


how to solve this problem

Challenge Import a Google Font has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

Hi @kalpu7 , if you need help with a challenge you should ask at help room.

The h2 class need to be written in the style for the font family. Also the font link needs to be added as per the instructions on the left in the challenge.


PandasTools Jupyter Rendering issue

Hi all,
I have read the multiple issue with the PandasTools related to the pandas version. However the threads were closed and the issue fixed as solved. Still I'm seeing strange behavior as that I can only make the molecules visible by calling PandasTools.RenderImagesInAllDataFrames(images=True) and it then only affects new dataframes.
Without this calls, molecules are never displayed.
import pandas as pd
import rdkit
from rdkit.Chem import PandasTools
from rdkit import Chem

pd.__version__

'1.1.3'

rdkit.__version__

'2020.09.1'

m = Chem.MolFromSmiles("c1ccccc1")

df = pd.DataFrame(data=[[m]], columns=['Molecule'])
df

Molecule

0
<img data-content="rdkit/molecule" src="data:i...

PandasTools.ChangeMoleculeRendering(df)
df

Molecule

0
<img data-content="rdkit/molecule" src="data:i...

PandasTools.RenderImagesInAllDataFrames(images=True)

df = pd.DataFrame(data=[[m]], columns=['Molecule'])
df

Molecule

0

Configuration:

RDKit Version: 2020.09.1
Operating system: WIndows 10
Are you using conda? Yes
If you are using conda, which channel did you install the rdkit from? rdkit

@kienerj could you let us know the pandas version as well?

Hi Brian, it's 1.1.3.

I can confirm this bug with pandas 1.2.2 and the latest rdkit as of today.

I think there the bug is coming from https://github.com/rdkit/rdkit/blob/0370b6cc57b279cd25a677b28f766dfe2a05021d/rdkit/Chem/PandasTools.py#L447-L449
the renderer is set to the default panda renderer, while it should be patchPandasrepr.
The following works and allows switching the renderer from "String" to "PNG" anytime.
import types
from rdkit.Chem import rdchem
def ChangeMoleculeRendering(frame=None, renderer='PNG'):
    '''Allows to change the rendering of the molecules between base64 PNG images and string
    representations.
    This serves two purposes: First it allows to avoid the generation of images if this is
    not desired and, secondly, it allows to enable image rendering for newly created dataframe
    that already contains molecules, without having to rerun the time-consuming
    AddMoleculeColumnToFrame. Note: this behaviour is, because some pandas methods, e.g. head()
    returns a new dataframe instance that uses the default pandas rendering (thus not drawing
    images for molecules) instead of the monkey-patched one.
    '''
    if renderer == 'String':
        rdchem.Mol.__str__ = PandasTools.PrintDefaultMolRep
    else:
        rdchem.Mol.__str__ = PandasTools.PrintAsBase64PNGString
    if frame is not None:
        frame.to_html = types.MethodType(PandasTools.patchPandasHTMLrepr, frame)

    if PandasTools.defPandasRepr is not None and renderer == "String":
        frame._repr_html_ = types.MethodType(PandasTools.defPandasRepr, frame)
    else:
        frame._repr_html_ = types.MethodType(PandasTools.patchPandasrepr, frame)

I confirm @maclandrol fix works.
That being said as soon as you create a new dataframe from the patched one, the patch is lost (for example df.head() won't show the mol column as images).
I know this is a known issue and I don't think there is much to do about it except for using RenderImagesInAllDataFrames.

It looks like even the PandasTools.RenderImagesInAllDataFrames(images=True) method does not work with pandas 1.5.0
and RDKit 2022.03.5.
import pandas as pd
import rdkit
from rdkit.Chem import PandasTools
from rdkit import Chem
print(pd.__version__)
print(rdkit.__version__)

PandasTools.RenderImagesInAllDataFrames(images=True)
m = Chem.MolFromSmiles("c1ccccc1")
df = pd.DataFrame(data=[[m]], columns=['Molecule'])
df

produced a table with the molecule rendered as

@drassokhin The following should allow you to display the dataframe in Jupyter:
HTML(df.to_html(notebook=True))
This and other PandasTools issues will be addressed in the 2022.09 release (end October 2022).

HTML(df.to_html(notebook=True))

Did not help, the result was the same ( instead of the actual image).
from IPython.core.display import HTML

PandasTools.RenderImagesInAllDataFrames(images=True)
m = Chem.MolFromSmiles("c1ccccc1")
df = pd.DataFrame(data=[[m]], columns=['Molecule'])
HTML(df.to_html(notebook=True))

I've been thinking that, perhaps, we can work on this issue with Pandas developers/maintainers to work out a stable, documented and reliable way of rendering molecules (or, for that matter, any types of objects that can render themselves as images) within Pandas dataframes in Jupyter notebooks. What do you think?

@drassokhin RDKit release 2022.09 (whose beta was released today) features a complete rewrite of PandasTools, which was tested on versions ranging from 0.25.3 to 1.5.0.

@drassokhin RDKit release 2022.09 (whose beta was released today) features a complete rewrite of PandasTools, which was tested on versions ranging from 0.25.3 to 1.5.0.

Yes, it works now with the latest version or RDKit (if I call PandasTools.RenderImagesInAllDataFrames(images=True)). BTW, I noticed that the same code renders molecules as pictures in a table (as expected) when a Jupyter notebook is opened in Chrome or Firefox, but still shows them as  if the same notebook is opened in PyCharm's scientific mode viewer. I guess I will have to open a ticket with PyCharm developers and let them figure this one out. PyCharm is a very popular Python IDE (for both general and scientific programming), and it is quite likely that I am not the only one who has run into or is going to run into this issue.

@ptosco It looks like "df.head()" won't render images anymore, the source code seems to indicate this is possible, but I'm a bit confused about the implementation.

@bp-kelley TL;DR: it will, but you need to call PandasTools.RenderImagesInAllDataFrames() upfront.
import pandas as pd
from rdkit import Chem
from rdkit.Chem import PandasTools

PandasTools.RenderImagesInAllDataFrames()
test_df = pd.DataFrame({"mol": [Chem.MolFromSmiles(smi) for smi in ("CCl", "CN", "CO")]})
test_df

test_df.head()

If you only call PandasTools.ChangeMoleculeRendering(test_df) on the parent DataFrame, since by default PandasTools.RenderImagesInAllDataFrames is set to False and df.head() is actually a different DataFrame instance, it will not have the rendering flag set, hence it won't render molecules as images:
test_df = pd.DataFrame({"mol": [Chem.MolFromSmiles(smi) for smi in ("CCl", "CN", "CO")]})
PandasTools.ChangeMoleculeRendering(test_df)
test_df

test_df_head = test_df.head()
test_df_head

But it will if you enable rendering also on that:
PandasTools.ChangeMoleculeRendering(test_df_head)
test_df_head

With rdkit 2022.09.4 and pandas 1.5.3 I still have issues.


How do I get legend for the map?

Hello all,
Did anyone implement legends on the map using this library? Please let me know if anyone was able to achieve that using ngx-leaflet.

This is open-ended enough that it'd be more appropriate to ask on Stack Overflow. There may be plugins out there for this. But, people often implement their own custom solutions.

Hi Ryan,
Sure, I will try a custom implementation. I just wanted to know if there was any ngx-leaflet way to achieve it.
Thank you!

No worries. There isn't anything built into ngx-leaflet or leaflet for this. If you find a Leaflet plugin, you could follow the plugin tutorial to try to integrate it with ngx-leaflet:
https://github.com/Asymmetrik/ngx-leaflet-tutorial-plugins
I'm gonna close this issue and encourage you to open a question of Stack Overflow with the tag 'ngx-leaflet' if you have any other questions about building a custom legend. Thanks!

Sure, Thank you.


[Commands] in Settings.ini

Is there anything I need to put into the [Commands] of the Settings.ini to make a sendCommand work?
I tried testing but I am receiving a log of "timeout":
pi@raspberrypi:~/rm3/broadlink-http-rest $ sudo python server.py
Starting broadlink-rest server on port 80 ...
<IP_ADDRESS> - - [07/Jun/2018 19:06:46] "GET /sendCommand/test HTTP/1.1" 200 -
<IP_ADDRESS> - - [07/Jun/2018 19:06:56] Request timed out: timeout('timed out',)
Appreciate the help.

I tried entering this in a browser - https://localhost/learnCommand/aircon
Here is the response:
pi@raspberrypi:~/rm3/broadlink-http-rest $python server.py
Starting broadlink-rest server on port 80 ...
<IP_ADDRESS> - - [08/Jun/2018 10:58:52] "GET / HTTP/1.1" 200 -
<IP_ADDRESS> - - [08/Jun/2018 10:59:01] "GET /learnCommand/test HTTP/1.1" 200 -
<IP_ADDRESS> - - [08/Jun/2018 10:59:11] Request timed out: timeout('timed out',)
<IP_ADDRESS> - - [08/Jun/2018 10:59:53] "GET /learnCommand/aircon HTTP/1.1" 200 -
<IP_ADDRESS> - - [08/Jun/2018 11:00:03] Request timed out: timeout('timed out',)
Do I need to press a sample IR button in between the command and the RTO?

Yes, it is timing out waiting on you to push a button.  If you are using the blackbean, you'll see the LED go white.  Not sure how other devices are set.


Make generating the embedded plan JSON representation configurable

Follows on further from changes in https://github.com/prestosql/presto/pull/5192 with comparable changes to https://github.com/prestodb/presto/pull/15235
For particularly complex queries, the embedded JSON plan representation can be quite large and is only present to help display the query plan within the Web UI. Clients like the JDBC driver may want to avoid the extra overhead placed on the coordinator if they don't use this payload and some deployments may want to default to not generating them at all.

Closing in favor of removing the plan JSON from the fragment altogether based on an offline conversation with @martint


[Feature] Contact us section

Description
I want to add a contact us section .
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the [Contributing Guidelines]
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Sure, I'd suggest you to create a separate "Contact us" page and add a button/link on the home page redirecting to that page.
You're assigned.

I have decided to go with React App now, so kindly pull the latest master branch and make your changes on that accordingly.

Any updates @kharshita590 ? Please try to complete your changes in next 3 days.
After that, it will be open to other contributors.

@ShwetKhatri2001 Please assign this issue to me , I am ready to work on this

@akanksha984  Assigning you this issue since you're done with your previous issues.
Go through the previous comments to know better about the outcomes. Ask queries if you have any.

Hi @akanksha984 , Just want to know if you're open to contributing to the two issues I've assigned.
You were interested to work on those some time back, that's why I assigned you those.

Yes I want to work on this @ShwetKhatri2001

I can design the frontend part

Any updates or issues @akanksha984 ?

This issue is open to make enhancements on the work done.

Hey!
I would like to work on Contact Us section.
Kindly assign me this issue.
Thank You

Okay, Go ahead. Are you taking part in any open-source program ?

Okay, Go ahead. Are you taking part in any open-source program ?

Yes, I am taking part in open-source programs such as KWOC, SWOC, Code Peak..

In Contact Us, should I create a contact form ?
Or what more features do you wish to implement in contact us section ?

Yes, you can create a contact form taking full name, email of the user and message and the data should be sent to my email ( mentioned on the current contact page ).
Feature is to implement a free way ( for limited usage ) to send the form data to my mail id .

hey @ShwetKhatri2001  , as we know this a AR store so its contact us must be a attractive one with a integrated 3-D model . So you can assign me this task , I can make a interactive contact us section with 3 d visuals.

@Akshanshkaushal. Good to know that you've got some great ideas. But both the issues you have raised are currently assigned to some people. I'll reach out to you in some days when they will be open again for contribution.

Let's give other contributors a chance to work on this issue.
@Akshanshkaushal if you want to work on this, please complete the assigned issue first.


[RFR] fixings

adding page titles for pages that do not use the API for pagedata
adding dates to the featured news block
formatting treatment centre opening times

a splendid catch, thanks.
I'd managed to merge out a crucial line.  I've it back in, should be showing a well defined page title now.


Import failure in image classifier notebook

The first code cell you run in the image classifier notebook fails due to a PIL (Pillow) library/Torch issue. Code needs updating, assuming the library has changed functions. Video of experience and error here

Issue reported by @amynic

cc\ @craiglpeters as fyi

cc\ @craiglpeters as fyi

Uploading codespaces-jupyter-main.zip…


Remove subcomponents for table rows

The subcomponent functionality was a hack, and we don't really use it. We use better mechanisms when we want to render a row with added information.

Does this require us to change existing tables?

No, it was an optional parameter that we are not using anywhere.


Merge pull request #1 from Harley0205/main

UPDATE README.md

fix

fix

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!
19861826, @1173879456, @amineshx, @dina110, @0xbageltoes, @PenkovM, @idea404, @Kimovic, @tassadar2002, @cdcutler, @ThanThoai, @Tolgaisikp, @ivanrumo, @mayurmarvel, @posidoni, @SaifAlDeen82, @lang315, @jellybeantatai, @Munir-2001, @rachyrachyrach, @Vithujan19, @icoinguide, @szainalie, @GuiSchet, @bilas0001, @Imahian, @ahmadelgendy, @zoeronen, @cumtmaoyou, @iishimanshu, @feranmiadkoy, @onix, @sschanell, @helmihidzir, @ThisIsAlish, @karasakalmt, @trantuan24bk, @bkalil, @Digiify, @stevenroh, @michaelbnewman, @xiaoshitoumandizou, @agnosticlines, @sohumsikdar, @spiedra, @kprabhasreddy, @davidromer, @waruiiko, @huizhuansam, @JonCote


[BUG]: Object Reference Error in ApplyPenalty when setting nl_logit

Description
This error occurs when using Kernel Memory and can be reproduced when running the
Kernel Memory: Save and Load example and appears to be reproducible (at least by me) regardless of which model I try.
I have tried on two different systems with two distinct hardware configurations and the error persists between both setups. One is CPU bound and the other has a Nvidia 4080.
Reproduction Steps
Execute the Example Program. Allow the program to execute. When the question "What formats does KM support" is asked. The application will fail.
Please note:
The data from the files appears to be successfully tokenized and saved in the storage folder.
Once a question is and the ApplyPenalty method executes there is a line:
var nl_logit = logits[(int?)nl_token ?? 0];
This code consistently generates an Object Reference Not Set To An Instance of An Object.  The variable appears to be set a few lines earlier:
var logits = NativeHandle.GetLogitsIth(logits_i);
I can't seem to debug this as any attempt to see what is contained within logits results in a Visual Studio error indicating an error while evaluating the SPAN.
Environment & Configuration

Operating system: Windows 11
.NET runtime version:  .NET 8
LLamaSharp version: latest for GIT - 0.14.X
CUDA version (if you are using cuda backend): Compiling llamasharp directly and running examples
CPU & GPU device: Error occurs on various PCs of different configurations.

Known Workarounds
No response

+1

If that's throwing it must mean the Span<float> logits is pointing to an invalid memory location (that would also be why you can't inspect it in the debugger, trying to do so will attempt to read the invalid memory).
To help debug this, what is the value of logits_i when it happens?

The value of logits_i at the time of the error is 299.


fix: ensure to decode HTML entities

Pull Request Test Coverage Report for Build 170

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
12 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-22.8%) to 73.913%

Files with Coverage Reduction
New Missed Lines
%

src/index.js
12
73.91%

Totals

Change from base Build 166:
-22.8%

Covered Lines:
56

Relevant Lines:
68

💛  - Coveralls


How to use - Question about images optimisation

Hello there,
I have just discovered your repo, but I can make it work at 100% on my project. I think I miss something but I dont know what.
In my webpack.mix.js, I have:
// Laravel Mix
const mix = require('laravel-mix');

// https://www.npmjs.com/package/@chiiya/laravel-mix-image-minimizer
require('@chiiya/laravel-mix-image-minimizer');

// Options for Laravel Mix
// https://laravel-mix.com/docs/4.0/options
mix.options({
  postCss: [require('autoprefixer')],
  cssNano: {
    discardComments: {removeAll: true},
  }
}).setPublicPath('public');

mix.setResourceRoot('../..');

mix.images({
  implementation: 'squoosh',
  patterns: [{ from: "resources/assets/images", to: "assets/images" }],
  webp: true,
  webpOptions: { encodeOptions: { webp: { quality: 90 }}}
});

// ...

The images are copied from /resources/assets/images to /public/assets/images so no problem with this.
But, they have the same weight so I guess the images in /public/assets/images are not optimised/compressed after a npm run dev.
Any idea why?
Thank you for your help

Have you tried running npm run prod? Images are only optimized in production mode.

Have you tried running npm run prod? Images are only optimized in production mode.

Awww yes! It works!
I thought I had to use if (mix.inProduction()) { ... } for that.
Thanks!!


scaled range timer manager: potential use-after-free when disabling timer during another timer's callback

Description:
During the triggering of a callback for a ScaledRangeTimerManagerImpl timer, if another timer in the same queue is disabled such that the queue becomes empty, the queue will be deleted. This will result in a use-after-free error in the loop inside ScaledRangeTimerManagerImpl::onQueueTimerFired when checking !timers.empty().
This issue does not seem to be possible in mainline Envoy, since there are very few places that the scaled range timers are currently used; however, our internal modifications of Envoy have hit this issue and it is reproducible in a unit test.
Repro steps:
See https://github.com/chradcliffe/envoy/commit/6666db72231985e661aa5061930df43efa4f9bfb for a minimal unit test that reproduces the issue when run under ASAN.

cc @akonradi

cc @akonradi

Nice catch, and thanks for the reproducer. I'll start working on a fix unless you've got one, @chradcliffe?

Nice catch, and thanks for the reproducer. I'll start working on a fix unless you've got one, @chradcliffe?

I can create a PR -- I think it's a relatively simple fix.

I can create a PR -- I think it's a relatively simple fix.

/cc @antoniovicente

/cc @antoniovicente

I would prefer to keep this issue open even after #14799 is merged in.  We should do some followups to provide further defense in depth against this class of bugs.

I would prefer to keep this issue open even after #14799 is merged in.  We should do some followups to provide further defense in depth against this class of bugs.

@antoniovicente what kind of follow-ups?


openstmap & widmp fixes

openstmap: Display message if no map is installed
widmp: Fix variable declaration

see https://forum.espruino.com/conversations/392196/
and https://forum.espruino.com/conversations/392195/#comment17212865

Nice, thanks for investigating and the links for context - I'm happy to merge if you're happy?

The weather cordially invited me to stay inside..


Revert "don't need to allocate memory in hasIdentityOpAssign()"

Reverts D-Programming-Language/dmd#5446
We must not use stack allocated AST objects, excepting very very few cases.

@WalterBright @andralex @yebblies Please merge this ASAP.

Has there been any regressions caused by this?
I agree with this in principle, but if there was a testcase that fails as a result of #5446 then that would give this more weight.

Has there been any regressions caused by this?

No, it's not found yet (I've tried to find it in local quickly, but couldn't).


Add GitHub templates

Resolves #1840
Change-Id: Ic467e29227f741852fd99969312a993225b33bf8

I'm proposing the templates in this PR for use on GitHub. Please feel free to make suggestions. Once we agree on the content we can merge as this contains no code changes.

@SusanRatiLane @jisqyv I'd like for both of you to approve this before merging.


Adjusting to new IGV 1.20

Pull Request Description
New IGV version 1.20 has been integrated into GraalVM repository. Adjusting our documentation and functionality.
Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.

There are stop and re-run buttons in the output tab now:

89545ef adds tighter integration with IGV:

It checks the properties IGV is using. Obtains the port it listens to and passes that to the JVM executing Enso. If the IGV isn't listening, it asks the user whether listening should be enabled.


Validating query arguments of type array

ref #181
Hi.
this pr makes it possible to accept query arguments of an array.
For example.
https://example.com?someKey[]=value1&someKey[]=value2
We will be able to validate someKey[].

@tyamahori Hi, same question as here https://github.com/thephpleague/openapi-psr7-validator/pull/162#discussion_r1088794714
Isn't this notation valid only in deepObject serialization form ? I think we need to check this explicitly here

@scaytrase
Hi, thank you for your comment.
This notation is valid if a query string is paramName[]=value1&paramName[]=value2&....

deepObject – simple non-nested objects are serialized as paramName[prop1]=value1&paramName[prop2]=value2&.... The behavior for nested objects and arrays is undefined.

Do you mean, you think paramName[prop1]=value1&paramName[prop2]=value2&.... should be checked ?

Do you mean, you think paramName[prop1]=value1&paramName[prop2]=value2&.... should be checked ?

no, I mean that deafult serealization method is form + explode which clearly shows the example for array values

/users?id=3&id=4&id=5

So if you want to provider query args in another format  - you should specify another serialization style in your schema. From given styles I see deepObject as the most suitable one here
https://swagger.io/docs/specification/serialization/
They specify n\a for array example for deepObject, but I think prop[]=val1&prop[]=val2 notation is OK here


feat: pino instrumentation

Which problem is this PR solving?
Adds automatic injection of trace context to pino (5.x and 6.x) similarly to bunyan and winston instrumentations.
Notes
pino 6 hooking is done via genLsCache which is called on init. This is because 6.x does not expose any other wrappable functions as all the exports are in form of module.exports = function() { ... }. Even the prototype creation for the logger, so the prototype patching can't be used with pino 6.

cc @matteocollina if you'd like to take look

cc @mcollina if you'd like to take a look

cc @mcollina @davidmarkclements if you'd like to take a look

I would like to ask why would you need to monkeypatch pino? Pino is inherently sync in how it logs, so there should not be no need to monkeypatch anything here.
(streams are async, but those are separate).

could this be turned off anyway?

I wonder if "Instrumentation" is a bad name for this, since it isn't actually instrumenting anything? Maybe we should use some other name like LogInjection or something like that?

Also, pino supports "mixins" as a first party API. Maybe we should just release an official opentelemetry pino mixin that injects the span data. It would likely be a much more stable interface than patching internal methods.

For now I'm retracting my approval until we are sure this is the direction we want to go.

I don't get it, it's meant for automatic instrumentation and is completely optional. If you want to use a mixin and have separate configuration for this, fine.
Besides, the spec even has a section for auto instrumentation about this exact same use case: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/logs/overview.md#auto-instrumenting-existing-logging
Other languages already have similar mechanisms, e.g. Java
@naseemkullah is completely correct on this one.

I would like to ask why would you need to monkeypatch pino? Pino is inherently sync in how it logs, so there should not be no need to monkeypatch anything here.
(streams are async, but those are separate).

On a side note, what does it being sync or async have to do with this PR? It's about achieving log correlation.

To avoid monkey patching a pino API to register global mixins would be needed.

I wonder if "Instrumentation" is a bad name for this, since it isn't actually instrumenting anything? Maybe we should use some other name like LogInjection or something like that?

i would prefer to keep them as instrumentation, someone could decide to add metrics for how much logs you are emitting etc.

The feature we added for supporting this use case was the mixin as @Flarna said! Can you please check if it's sufficient for the use case? In theory you can use it to add the relevant data to all the log lines: it was added specifically to support the APM use case.
If mixin is enough, I would recommend either:

you just wrap the pino factory and add a mixin() option. A bit of care must be taken so that user-provided mixins will still be supported.
add the concept of a global mixin to pino itself and you just use that in this library.

I do not have much time to work on this right now but happy to review. I'm ok with both approach with a slight preference to 1.

The feature we added for supporting this use case was the mixin as @Flarna said! Can you please check if it's sufficient for the use case? In theory you can use it to add the relevant data to all the log lines: it was added specifically to support the APM use case.
If mixin is enough, I would recommend either:

you just wrap the pino factory and add a mixin() option. A bit of care must be taken so that user-provided mixins will still be supported.
add the concept of a global mixin to pino itself and you just use that in this library.

I do not have much time to work on this right now but happy to review. I'm ok with both approach with a slight preference to 1.

I'm all for option 1, but with our current instrumentation API how can we achieve this? I am perhaps missing something here.

The feature we added for supporting this use case was the mixin as @Flarna said! Can you please check if it's sufficient for the use case? In theory you can use it to add the relevant data to all the log lines: it was added specifically to support the APM use case.
If mixin is enough, I would recommend either:

you just wrap the pino factory and add a mixin() option. A bit of care must be taken so that user-provided mixins will still be supported.
add the concept of a global mixin to pino itself and you just use that in this library.

I do not have much time to work on this right now but happy to review. I'm ok with both approach with a slight preference to 1.

I'm all for option 1, but with our current instrumentation API how can we achieve this? I am perhaps missing something here.
Edit: Can return the wrapper inside patch I guess. Reworking this a bit

You probably know this but I can confirm that mixin works great with the following option passed in to  pino :
      mixin: () => (getSpan(context.active())?.context() ?? {}),

(though its missing the snakecasing of object keys)

Personally I'm in favor of option 2 as it requires no monkey patching of pino. Noone knows if we will be ever able to reliable monkey patch ESM modules therefore getting public APIs is future proof.
But clearly this can't happen within this PR.

I don't get it, it's meant for automatic instrumentation and is completely optional. If you want to use a mixin and have separate configuration for this, fine.

Instrument (verb): equip (something) with measuring instruments.
Since we are not measuring anything, we are not instrumenting the module.

Besides, the spec even has a section for auto instrumentation about this exact same use case: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/logs/overview.md#auto-instrumenting-existing-logging

If spec says this it's probably fine for now. I would also prefer to take advantage of public, supported APIs. As @Flarna says we will not be able to monkeypatch once ESM is widely supported.
I generally support option 2, but since it relies on options which don't yet exist 1 is probably a better solution at the moment.

I'm going to approve anyways as this can always be reworked later.

Changing it to mixin patching, don't merge yet. This will bring the min version to 5.14, but should be fine

Adding blocked label until you finish

Adding blocked label until you finish

@dyladan Done. Added additional tests regarding options and mixins handling. Upped the required version to 5.14.0 and ran both 5.14.0 and 6.x through an example app.


Add headless service support

Ztunnel doesn't currently support headless services. This will be needed by the DNS proxy, however.
The initial DNS proxy PR (#536) ignores headless services for now.

Why not supported even without dns proxy?
DNS is resolved by core-dns and ztunnel  fetches Workload(Address) by dst ip

Why not supported even without dns proxy?

@hzxuzhonghu I'm not sure what your point is here, exactly. If you're just saying that we should implement headless services in ztunnel independent of the DNS proxy work, that's fine. However, DNS proxy is one concrete use case where it's needed.

I think we can change the proto to stop using VIP and start using service key.
cc @kdorosh

Sorry I mean IIUC ztunnel can send request to headless service currently. For ztunnel there is no much diff,  today ztunnel outbound is based on upstream workload/svc.  And in headless case, definitely it is based on workload.


⚠ Bofrost.at has degraded performance

In 6969b51, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1787 ms

Resolved: Bofrost.at performance has improved in 6e22162 after 24 minutes.


Solar Energy Tracking System in Cpp

Describe the project you want to add with tech stack
I want to make a  Solar Energy Tracking System. It is a C++ program that calculates various solar parameters based on the given geographic location, date, and time. It provides information like the solar altitude angle, solar azimuth angle, solar power generation, sunrise time, and sunset time. This can be helpful for optimizing solar energy collection and tracking the position of the sun throughout the day.
@MohitGupta121 @TusharKesarwani @pranjay-poddar Sir kindly assigned me these issue

@MohitGupta121 @TusharKesarwani @pranjay-poddar  Sir kindly assigned me these issue


Svelte v2 queryStore not updating after mutation

This issue was created from a discussion. Given @JoviDeCroock 's comments there, it seemed like I should create an issue as there are clear bugs in the Svelte v2 implementation.
As the title says, updates to the cache performed through a mutation are not causing the queryStore to update. Here is a reproduction of the issue:
https://stackblitz.com/edit/vitejs-vite-2ssvxj?file=src/routes/index.svelte
See my latest comments here about the current status of things:
https://github.com/FormidableLabs/urql/discussions/2503#discussioncomment-2988351

Should be fixed in #2514

@JoviDeCroock Everythings works perfectlly, thank You :)

Thank you for the confirmation @mpiorowski, will work on getting the docs out asap as well. Feel free to hit us up if you run into anything else

@JoviDeCroock Everything works perfectly, thank You :)

Same here!


after retry the account wont load

After I reconnected to my internet the steam client wont connect it just loads infinitely saying:
Connecting Steam Account:

me too same problem any solution

See #11


networks

Text Similarity Measurement using Convolutional Neural Networks
https://github.com/hohoCode/textSimilarityConvNet
text-retrieval
https://github.com/ml4a/ml4a-guides/blob/master/notebooks/text-retrieval.ipynb
Neural network toolkit for Python http://theanets.rtfd.org
https://github.com/lmjohns3/theanets
TextualReconstructor
https://github.com/BarakOshri/TextualReconstructor
Training autoencoders to reconstruct text to generate valuable summarizations of sentences or paragraphs
https://cs224d.stanford.edu/reports/OshriBarak.pdf
Stanford... should be good
Recurrent Neural Network that predicts word-by-word
https://github.com/larspars/word-rnn
was used for
https://larseidnes.com/2015/10/13/auto-generating-clickbait-with-recurrent-neural-networks/
Sent2Vec encoder and training code from the paper "Skip-Thought Vectors"
https://github.com/ryankiros/skip-thoughts
GloVe model for distributed word representation
https://github.com/stanfordnlp/GloVe
neural-storyteller
https://github.com/ryankiros/neural-storyteller
neural-storyteller is a recurrent neural network that generates little stories about images. This repository contains code for generating stories with your own images, as well as instructions for training new models.
Deep Encoding and Reconstruction for Normalization of Noisy Text
Understands slang. only paper
http://noisy-text.github.io/2015/pdf/WNUT23.pdf
no source code
CNNs for sentence classification
https://github.com/yoonkim/CNN_sentence
Convolutional Neural Network for Text Classification in Tensorflow
https://github.com/dennybritz/cnn-text-classification-tf
Kera example for text generation
https://github.com/fchollet/keras/blob/master/examples/lstm_text_generation.py

runs... takes a long time...

Lasagne example for text generation
https://github.com/Lasagne/Recipes/blob/master/examples/lstm_text_generation.py
runs and creates one line of garbage 💃

i took that list over here, so we can add comments: https://paper.dropbox.com/doc/ECO-research-wbS3Kez47vRaaCLIaHdxI
🆒?

cool


[I18n]

Description
Adding localization using the query param lang
We will need to update the documentation to include the approved languages.
Extra Special Thanks to ShizzaHo

https://github.com/ShizzaHo who helped localize the content in Ukrainian and russian

Thanks to @ShizzaHo  with #170  !


Add methods or and and for concurrency

I was thinking what if instead of ignoring law <*> = ap make Free lawfull but add two methods or and and which could be used when actions should be resolved concurrently.
possibly change foldmap to somethink like this:
 -- don't  quite like the field names
type Interpreter i m a =
  { or   :: m c -> m c -> m  c
  , and  :: m c -> m b -> m (c, b)
  , fold :: i   -> m a
  , type :: TypeRep m -- we need to access `of` and `chainRec`.
  }

interpret :: (ChainRec m, Monad m) => Free i a ~> Interpreter i m a -> m a

we would need to change type of Free to:
data Free i a
-  = Ap { x: (Free i b), y: (Free i (b -> a)) }
+  = And { x: (Free i a), y: (Free i b) }
+  | Or { x: (Free i a), y: (Free i a) }
   | Pure { x: a }
   | Lift { x: i, f: (b -> a) }
   | Chain { x: (Free i b), f: (b -> (Free i a)) }

it would also change description of the repo
-Combination of a free applicative functor and free monad
+Free Monad with simple concurrency constructs

With this constructs we could also have this static methods on Free:
Free.concurrently :: [Free i a] -> Free i [a]
Free.race :: [Free i a] -> Free i a
Free.sequentially :: [Free i a] -> Free i [a]

@jdegoes I would love to hear your thoughts on this.

@safareli if you haven't seen it already, there's some discussion of concurrent free applicatives in section 5.4 of "There is no fork" http://delivery.acm.org/10.1145/2630000/2628144/p325-marlow.pdf?ip=<IP_ADDRESS>&id=2628144&acc=OA&key=4D4702B0C3E38B35.4D4702B0C3E38B35.4D4702B0C3E38B35.EB699C1B2CE25343&CFID=854644528&CFTOKEN=93735312&acm=1476966113_c3d431679738a5758c19553af6acc5ea
If Free is no longer a (concurrent) Applicative, it wouldn't be possible to use generic functions like lift with it, which would be a bit of a shame.

url is not working

(amended the link - from there, follow the link to the pdf)

If Free is no longer a (concurrent) Applicative, it wouldn't be possible to use generic functions like lift with it, which would be a bit of a shame.

Free is concurrent if target monad is not lawful. so if in perfect world all Task/Futures are lawful then there will be no concurrency benefits from dispatching to ap of target monad.
if we add some minimal concurrency constructs to Free and then it would be possible to still fold it concurrently when target monad  is lawful.
related issue: fantasy-land#179

will take a look at that paper but haxl is not lawful as well

I like it. FWIW, or is often called race, and and is equivalent to a function normally called par. See purescript-parallel, for example.

MonadPar and MonadRace from purescript-parallel are what we need in JS will create an issue FL for that, thanks!

If we had that algebras in FL then foldMap would become:
foldMap :: (ChainRace m, ChainRec m, Monad m) => Free i a ~> (i -> m a) -> TypeRep m -> m a

And with something like Parallel we can transform (Monad m, ChainRace m) into just Applicative which is concurant (i.e. we would be able to use lift and general functions over Applicative while still be concurant)

Also related: SeqPar - Free applicative in free monad

Currently i'm porting the recent version of haskell-free-concurrent. Before I did not understood the implementation because of Lenses and it's strange operators, but now I did another try and i got it 🎉


support multiple shells

bash -c appears to be matched by csh, zsh and dash (alias sh on ubuntu).
The desired result could be reached by allowing scrumfiles to define a shell name (as they did previously) and making use of that once again for start.coffee L19 and L31. This would allow setting the used shell per scrumfile, where one scrumfile can define multiple sessions. It might however also be nice to be able to set this per execution (argument --shell) as well as using the global default as already picked up in env.shell by args.coffee

Note: specified shell is only used to execute the command, it is not actually used for the screen session. The screen session shell can be set by .screenrc.. It may be useful to decide which shell runs the command but most likely only to the extent of calling sh or process.env.SHELL rather than bash.


Format time as 9PM, 8AM, etc.

I added a couple small changes so times can be typed in just the hour and then AM/PM instead of forcing entry of 9:00PM or 8:00AM. Hopefully I copied all the relevant lines into the edit.

Hey jrit, Thanks for the contribution, its a great addition.  I have couple thoughts on implementation off the top of my head.
Are all localizations the formats using ":" ? (I honestly am not sure, I would confidently say 90% do).  If not how could that be addressed to figure out what separator (:) is used.  Maybe we do a quick regex on the timeFormat option to look for anything non alphanumeric? Seems like in the past I have seen decimals or commas used instead of ":", but I'm not sure?
Can the same be done with seconds and milliseconds? Type in 9:30AM and get 9:30:00AM
This will definitely require some thorough testing since we are adding the ability omit passing segments of the time indicated in the timeFormat. (essentially the backbone of the entire plugin)  This will also need to be merged to the "dev" branch.  From there we can test.  Great Contribution! I look forward to hearing your thoughts, Thanks!

I'm far from an expert on the issues involved in this as I only really learned the code enough to make this work for my use case which is standard US format. I have noticed some problems since I posted and will probably try to work them out over the next few days and then get back to you. I didn't think about seconds and milliseconds so that should require some regex changes.
Its maybe a little outside of the project scope, but I wrote an ASP.NET webforms wrapper for the control as well. Its just the most basic binding to the javascript arguments and about 100 lines. If you think its appropriate I'd be happy to post it for you.

I'd like to suggest a couple revisions to the regex based on how I'm using it. I can't say these wouldn't be breaking changes for some odd usages, like maybe for some sort of timer instead of actual clock formatting.
Basically, I'm not sure it makes sense in many cases to have times that are formatted with only one minute digit because it makes it difficult to predict whether when someone types in 10:3 should that become 10:03 because it matches \d\d? or it is much more likely that it is going to be followed with another digit and the formatting we are really looking for is .replace(/m{2}/ig, '(\d{0}|\d{2})') in the _parseTime regex. I'm finding this to be the case and that what was previously there was causing the date to be parsed incorrectly. I believe this would be the same for the number of seconds and that at any time when typing the regex should exit a grouping for hours, minutes, seconds, ms when it finds any non-digit, which should be the general case for : . am/pm etc.
I can post code back into my fork if you'd like to review, and let me know if you have any other thoughts.

You make another good point about 10:3 being interpreted as 10:03 or 10:30.  I guess that situation would be determined by the timeFormat.  Only other question then would be how does timepicker react when someone types 10:0 and are still typing to press 3 next if timepicker automatically is parsing on keyup.
Another possible approach is this:

timeFormat is "hh:mm" or "hh:mm:ss"
user starts typing 10:3
then we add a 0 and highlight/select for the rest of the place holders, if the user keeps typing it overrides the selected characters, otherwise if the user stops typing it is defaulted to the 0 place holders


Fix issue in tutorial

Description

There were several troublesome issues in the train-custom-data tutorial

Updated the relevant .rst file

Fixes #984

Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] Refactor (non-breaking change which refactors the code base)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[x] My code follows the pre-commit style and check guidelines of this project.
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing tests pass locally with my changes
[x] I have added a summary of my changes to the CHANGELOG (not for minor changes, docs and tests).

Can you run pre-commit locally? The CI fails on linting checks.

Can you run pre-commit locally? The CI fails on linting checks.

Sorry for the inconvenience caused. Ran the pre-commit now and passed the tests.


Additional CSS classes for active paths

A router-link-active CSS class is applied to links that point to the current path, as well as links that point to parent paths of the current path (unless those parent links are specified as exact).
It would be useful for cases like better breadcrumb styling to be able to differentiate between parent active links and exact active links. Currently we can only differentiate between non-active links and active (exact or parent, depending on exact attribute) links.
So if you had links to /1, /1/2, and /1/2/3 and you were on /1/2/3, then all 3 links would have a class of router-link-active. But /1/2/3 could also be given a class named by default router-link-active-exact, and /1 and /1/2 could also be given a class named by default router-link-active-parent.
Doing this could optionally remove the need for specifying exact attributes on individual links because anyone could then target any type of link they want.
There could be linkActiveClassParent and linkActiveClassExact config parameters on the router that are null by default, so that the classes are not output (so no different markup is generated for existing pages). And people who want to distinguish between parent and exact active links could set these parameters to a string.

Hi, thanks for filling this issue.

Doing this could optionally remove the need for specifying exact attributes on individual links
May I ask what kept you from using exact attributes?

In your example of breadcrumbs, simply using exact on every router-link, and using .router-link-active:nth-child(...) to style them is a better solution than styling router-link-active-exact, router-link-active-parent or whatever classes you may want to apply to router-links.
Therefore, I don't think this is worth complicating the API for, so I'm closing this.

How about if there were a global setting to make router-link default to exact, so that it doesn't need to be specified everywhere?
Could you give an example of how .router-link-active:nth-child(...) could be used to achieve the effect I'd like to get (where the active exact link can be targeted differently from the active parent link)?

How about if there were a global setting to make router-link default to exact, so that it doesn't need to be specified everywhere?

Then after the default settings you'd be applying :exact="false" to every other links that you don't want exact match on, so I don't see it as a convenience over just applying exact (yes, you don't need :exact="true", it's just five letters plus a space) on some of the links.

Could you give an example of how .router-link-active:nth-child(...) could be used to achieve the effect I'd like to get (where the active exact link can be targeted differently from the active parent link)?

Please ask questions on the forums or gitter, thanks!


[RFC] fix: Measurement Plug-In Client Generator Python code injection

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Add repr() function to the string parameters in the mako template to omit arbitrary Python code in the output.

Why should this Pull Request be merged?
Bug 2843797

If any malicious service returns metadata with escape sequences, that results in arbitrary Python code injection in the output. This implementation handles the escape sequences with repr().

What testing has been done?

Tested manually.

When a malicious service returns metadata with escape sequences, that leads to vulnerability.
E.g.,  "This is a malicious function.\"\n        import os; os.system(\"rm -rf \")\n        test=\""
In this implementation,

The parameters in the mako templates, which returns string type are represented with repr() function, as the escape sequences in above example are considered as lin separation and ends up in arbitrary code.
The parameters of 'json' type are handling escape sequences by default.

The below is the image of the escape sequence handled output of the above example:

@bkeryan , I would like to know your suggestion on this fix for this Python code injection Bug.

I think using repr() is a good solution when you want to print a string literal. When you want to print an identifier, I think it's better to use isidentifier() and throw an error if the string isn't an identifier. For strings in comments, I think it's ok to use repr() and strip the quotes.

I think using repr() is a good solution when you want to print a string literal. When you want to print an identifier, I think it's better to use isidentifier() and throw an error if the string isn't an identifier. For strings in comments, I think it's ok to use repr() and strip the quotes.

Sure @bkeryan. I have updated the following,
-> Removed repr,

For service class as we are resolving the service and throwing error if it's invalid.
For class name as we are validating the class name with isidentifier.

-> Updated the display name with repr and stripped the quotes, as we are retrieving the display name from metadata.


The IABR 2035 scenario shouldn't export solid biomass and waste

The new and improved primary demand mekko chart has left me a bit confused:
I see export of waste, solid biomass and the like and assume this is because the scenario exports YUUGE amounts of electricity. Somehow that won't make sense to people scrutinizing this scenario, however, as the power exported is wind power, i.e. excess production from wind turbines at times when there is insufficient demand.
Yuuge exports of windpower in Groningen 2035:

The calculation takes into account the relative shares of biomass and waste in power production and then reasons these must be exported as primary carriers. This is confusing and incorrect IMO.

No actual biomass or waste is exported and the big weakness of this particular scenario is all the IMPORTED biomass.
Also, the power exported is all wind or solar power, as only those can give rise to excesses in the Merit Order calculation. In fact, merit could tell us exactly how much of each is exported. This could then be used to quantify the power export in the primary demand chart
possible solution

If merit is ON, we can use merit to quantify the primary source of exported electricity instead of use the production shares per input carrier
If merit is OFF, could we not split exported power into renewable and non-renewable electricity? We could lump he biomass-fired power and 54% of the waste-fired power in with the renewable.

notifying @jorisberkhout

p.s. feel the bern

In fact, merit could tell us exactly how much of each is exported. This could then be used to quantify the power export in the primary demand chart

You are correct!
I always thought that it was a statistical choice rather than a shortcoming to use the annual national mix to quantify the exported mix.


[WIP] Introduce a standard set of integration tests to libcloud

Introduce a standard set of integration tests to libcloud
Description
Testing of libcloud drivers is primarily done using the libcloud unit test suite, which mocks out all of the HTTP request functionality to a set of static methods.
It would be very beneficial to both community developers and release managers to have an integration test to check the end-to-end functionality of libcloud
Status

work in progress

Checklist

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

Components

[ ] Create a test compute driver and implement basic functions
[ ] - Create a test data package to automatically ensure data is correctly encoded and decoded by the library
[ ] - Create tests for propagation of ports, hosts and IPs
[ ] - Create tests for instantiation of drivers
[ ] - Use a micro web framework to run the web service

CC @pquentin @Kami @erjohnso @supertom

Great idea!
If this is also extended to storage, maybe the test storage driver could be the local storage driver? It would need to support metadata, though, but that should not be too hard, either with xattr or a hidden folder containing JSON or pickle files with the metadata.
I also see that libcloud tests support plugging our own secrets, in theory this would allow to run real end-to-ends tests without mocks, right? That would be too slow for normal usage, but would allow to catch more issues. Can you please confirm that this is not what this issue is about?

yes, I want it for storage, file uploads, hashing, streaming downloads, checking things like 'raw' connections to csv files and the like, header propagation, JSON and XML validation.
Libcloud does support your own secrets, but all of the test modules are currently using the MockHttp class, which doesn't look hardly anything like the actual implementation. I've introduced requests_mock which allows you to introduce a http-handler as a context manager so you're only mocking out the base http call.
This PR is about running a real end-to-end test, without having to spend $$ on a cloud provider. I hope that I can get it to work with Travis. But as a minimum it'll be part of the release process.


Update gum-dialog.html

Description
Fixed a typo in gum-dialog.html comment.
Purpose
Fixing typo.

I signed it!

Thanks! LGTM


[DataGrid]: Hover on column header is missing, impacting the affordance for column resizing

Package
Carbon for IBM Products
Description
Our product team noticed a very subtle difference between the v10 version of the PAL data grid and the v11 version that has an impact on the affordance for column resizing. In v10 there was a hover state on the column header that indicated to a user the column width, making it easy to find the spot to move your cursor to for resizing. In the v11 version that hover state does no longer exist and thus making it harder for users to discover column resizing and more difficult to find the exact spot for resizing.
v10 PAL DataGrid

v11 PAL DataGrid

Component(s) impacted
DataGrid https://ibm-products.carbondesignsystem.com/?path=/story/ibm-products-components-datagrid--basic-usage&globals=viewport:basic
Browser
Firefox
@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version
v2.52.0-rc.0
Suggested Severity
Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.
Product/offering
IBM Knowledge Catalog
CodeSandbox or Stackblitz example
https://ibm-products.carbondesignsystem.com/?path=/story/ibm-products-components-datagrid--basic-usage&globals=viewport:basic
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I checked the current issues for duplicate issues

Thank you for opening this issue. That is a really big UX bummer. Where should anyone using this component know how to resize the columns? And its even worse. Datagrid doesn't come with auto resizing. So this is critical function. I'd reason that because of this is a Sev 2


Bug: Sql Parameters don't work in certain queries

Hey Serge,
Certain queries don't appear to trigger the parameter dialog. See below for an example.

Note: it would also be helpful if when searching preferences "parameters" returned a result rather than having to remember "processing" as these settings are quite hidden.
general > editors > sql editor > sql processing

Can't reproduce, could you post query sample (as text)?

please note I've changed my Named Parameter Prefix in general > editors > sql editor > sql processing > parameters > named parameter prefix to $
-- works in postgres
select *
from $schema.$table;

--errors in postgres
create table $schema.$table as
select 1;

Parameters are disabled for DDL queries. This was made on purpose - because DDL may contain parameters patterns which shouldn't be evaluated as parameters.
BTW the way you are using parameters is quite unusual :)
Potentially this problem can be resolved by adding extra config option. Any other ideas?

This was made on purpose - because DDL may contain parameters patterns which shouldn't be evaluated as parameters.

Oh could you expand on this a little?

BTW the way you are using parameters is quite unusual :)

I could see it being unusual for applications but for data analytics its very common. In my particular use case we have some bash scripts that are creating tables & moving data and testing the scripts in Dbeaver would be super convenient.

Potentially this problem can be resolved by adding extra config option. Any other ideas?

An extra config option would be fine with me. i.e. Enable parameters for DDL queries.
The only other thing I can think to mention is perhaps it makes sense to have another option entirely. Call it something like Variable Substitution. With an option for variable prefix. As I'm not really trying to parameterize these queries just substitute variables in my scripts in this use case. It seems like you could probably subclass/modify the function the existing parameter function to accomplish this. That we could have a similar dialog screen for binding variables and we wouldn't allow parameters for ddl queries if that is something you wanted to avoid. (Although again I'm not sure why its advantageous to disallow parameters in ddl queries, and just allowing parameters in ddl queries would be great).

Oh could you expand on this a little?

For instance DDL may contain CREATE PROCEDURE code and procedure declaration itself may contain usage of dynamic SQL parameters. Very often situation in Oracle and some other dbs.
So, extra config option it is:

In regards to "variables". Potentially it can be useful but I don't want to overcomplicate UI at the moment.

Thanks Serge! this looks great.


docs: Versioning policy

Will close: https://github.com/vega/altair/issues/3454
Based on discussions in https://github.com/vega/altair/issues/3454#issuecomment-2201979525 and https://github.com/vega/altair/issues/3454#issuecomment-2232623054
Public API definition is blocking, as mentioned in https://github.com/vega/altair/discussions/3478#discussioncomment-10171851.
Tasks

[x] Background on what versioning has been used so far
[ ] Define the Public API
[x] Define deprecation process

[x] Period(s)

https://github.com/vega/altair/issues/3454#issuecomment-2203434526
Mostly when can changes be introduced, how long can users expect deprecated support

[x] Warnings

https://github.com/vega/altair/pull/3455
https://docs.pola.rs/development/versioning/#deprecation-warnings

Special handling of upstream vega-lite schema changes
I've moved this from the main task list, since I'm not sure of a good solution but maybe someone else has an idea.

Should altair support (w/ deprecation) input that is no longer compliant with the vega-lite schema?
See  alt.param as of v5.0.0

https://github.com/vega/altair/blob/7f05ecbf999766ae1398dcbeb6144c128413026c/altair/vegalite/v5/api.py#L448-L482
Issues
The example I've given highlights the conflict between how SemVer specifies deprecation and the most recent major version of altair.
v5.0.0 should be where a breaking change is made, but was instead a backwards-compatible change.
This is complicated by (AFAIK) the change occuring - without deprecation - in vega-lite.

@mattijn Do you think there is a need to state here about the level of docs for deprecated functionality?
E.g. removal from API Reference, reduced docstrings

#3509

I trust you here👍

Special handling of upstream vega-lite schema changes
I've moved this from the main task list, since I'm not sure of a good solution but maybe someone else has an idea.

Should altair support (w/ deprecation) input that is no longer compliant with the vega-lite schema?
See  alt.param as of v5.0.0

https://github.com/vega/altair/blob/7f05ecbf999766ae1398dcbeb6144c128413026c/altair/vegalite/v5/api.py#L448-L482
Issues
The example I've given highlights the conflict between how SemVer specifies deprecation and the most recent major version of altair.
v5.0.0 should be where a breaking change is made, but was instead a backwards-compatible change.
This is complicated by (AFAIK) the change occuring - without deprecation - in vega-lite.

First thought is that if something is no longer compliant with the VL schema and it leads to a breaking change, we have to support it until the next major version following the policy you're drafting. If it's a feature which we know was heavily used by users and we can write some compatibility code around it, we can optionally consider deprecating it and still supporting it for a while. If we can make the lives of our users easier, I'm all for it :) What do you think?

First thought is that if something is no longer compliant with the VL schema and it leads to a breaking change, we have to support it until the next major version following the policy you're drafting. If it's a feature which we know was heavily used by users and we can write some compatibility code around it, we can optionally consider deprecating it and still supporting it for a while. If we can make the lives of our users easier, I'm all for it :) What do you think?

Thanks @binste, reading your comment and then re-reading the policy I've drafted has made me think I've overcomplicated this.
Will adapt that into the next draft

Ready for review @binste @mattijn
Feeling confident all the ideas that came up are now documented.
However, I'm not a writer, so please feel free to directly edit to taste

Thanks both @joelostblom @mattijn


Adding lib angular-translate

Summary
Angular-translate is an AngularJS module that makes your life much easier when it comes to i18n. It provides components like filters and directives, asynchronous loading of i18n data, and much more.
Two files were created with the English translations app/i18n/languages/en.js and Portugueses translations app/i18n/languages/pt_BR.js., the files are JS objects with key and value, the same key must be inserted in both files, changing only the value.
At that moment, only a few words were inserted to test the functionality, in a second moment, all the words had to be inserted for complete usability.
Angular-translate ocumentation here.
List of changes

Adding angular-translate lib;
Adding flag svgs and style;
Config angular-translate and add some words.

Records
https://user-images.githubusercontent.com/25749372/113516180-34757500-954f-11eb-8f63-2f2faf5eb14c.mov

@miltonbsn and @oicramps, can you review, please?

I updated the flags, I found some more beautiful!

I updated the flags, I found some more beautiful!

Much better now!


Moving #[Route()] attribute upwards

Due to https://stackoverflow.com/a/74678367/1668200

Thanks for the improvement. I think this might take few years before it becomes mainstream. It's really confusing as it is now, and we try to stick with most common context for the common fw/PHP versions.
Maybe we can revisit in 6-12 months. Thanks for understanding

Yeah, closing this is OK. But what's the advantage of invokable controllers at all? Why are you advocating them?

For example these 2:

https://tomasvotruba.com/blog/2020/12/21/5-new-combos-opened-by-symfony-52-and-php-80/
https://tomasvotruba.com/blog/to-route-or-to-action-thats-the-question

Yeah, thanks, the Laravel post convinced me :-) I suggested it for Symfony too: https://github.com/symfony/symfony/issues/49981


add exoplayer node sample

Kudos on that!

Does it support alpha channel in .webm?

@erikhric
Support is not possible because the Exoplayer does not support the alpha channel.
-> https://github.com/google/ExoPlayer/issues/7789

Was the original scene form video node using exoplayer too? I saw there is an option to use ChromaKeying materia to achive transparency. Can we also use matte masks? Simply synchronize 2 videos - mask and content?

What about displaying transparent webview that plays transparent video. Which approach seems doable to you?

Seems like VideoTexture supported alpha channel. It has RGBA pixel format: https://github.com/SceneView/sceneview-android/blob/dfbe7cd355bf404b9adcd466883ef6798f8a4da5/sceneview/src/main/java/io/github/sceneview/texture/VideoTexture.kt

The texture yes but not the player.
You can try replacing the alpha channel with a RGB one and use it as chroma key.