Sound continues for a while after closing the tvOS player

Description of the problem
When playing a URN (not a URL) with the Pillarbox player on tvOS and the system user interface (what we will offer as UI on tvOS), sound continues for a while, then stops. This is annoying.
Some remarks:

The issue does not arise if VideoPlayer is replaced with VideoView.
The issue does not arise on iOS.
The Player is immediately deallocated but sound does not stop.
Investigation with Instruments shows the same memory profile at the end. Resources are properly cleaned up (at least those not persistent).

Relevant stack trace or log output
No response
Reproducibility
Always
Steps to reproduce

Open the Pillarbox tvOS demo in the simulator or on a physical device (you can e.g. use nightly 286).
Under Showcase, System player (using Pillarbox), play Apple Advanced example (a URL). Close the player and observe that sound immediately stops.
Under the same section play the URN example. Close the player and observe that sound only stops after ~10 seconds.

The only difference is that the second example is a URN. If replacing the SystemVideoView with a simple VideoView you can also observe that the problem vanishes.
The above also works fine on iOS.
Library version
0.5.0
Operating system
tvOS 16 (no issue on iOS)
Code sample
No response
Is there an existing issue for this?

[X] I have searched existing issues and found no similar bug report.

I suspect an issue with Apple tvOS player UI implementation. We should attempt to reproduce the issue in a sample app:

A simple AVPlayer using a VideoPlayer view, playing a URL.
A simple AVPlayer using a VideoPlayer view, playing a URL with a simple resource loader.

Current workaround: We can pause the player when SystemVideoView disappears. Alternatively (but a bit more intrusive) we can always pause the queuePlayer in Player.deinit.

I also discovered an issue on iOS and tvOS 17 beta 5 that I reported under iOS / tvOS 17 beta 5 regression: VideoPlayer displayed in a modal sheet makes the attached AVPlayer leak (FB12925347).
The issue above can be reproduced on iOS 16, though.

The issue can be reproduced in a project created from scratch as well, this is therefore not related to something we do in the demo.

The issue is related to the use of our LoadingResourceLoaderDelegate. I could namely produce a small project that reproduces the issue, as follows:

Load an AVPlayer with a loading item like what we have in Pillarbox (custom scheme + dummy resource loader delegate)-.
After a while, replace with an item (an URL-based item suffices).

After closing the player we can observe that audio continues for a short while.
Note that:

Using a proper resource loader delegate which asynchronously redirects to a valid URL works.
Replacing a proper item with another one after a delay works.

So really the issue is with our loading resource loader implementation.

I discovered that this issue also occurs on iOS with the player inlined and a model stored as a @StateObject. The fix will be made on a branch which should hopefully be soon submitted for review.


docs: Start populating KlayOracle documentation starting with: Architecture Page, Milestone Page. Draw inspiration from Redstone Finance, Witnet or Chainlinks docs

@pauloladimeji Respond with milestones and timelines to achieve the above with the current architecture setup and progress made. Setup a call with @alofeoluwafemi to get the information needed for developers to get up and running.


Update Excel.workbookquery.delete.md

Added parameter

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

File
Status
Preview URL
Details

api/Excel.workbookquery.delete.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 584d8d7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

api/Excel.workbookquery.md
:warning:Warning
View
Details

api/Office.MsoThemeColorSchemeIndex.md
:warning:Warning
View
Details

access/Concepts/Miscellaneous/in-operator-microsoft-access-sql.md
:warning:Warning
View
Details

api/Excel.workbookquery.delete.md
:white_check_mark:Succeeded
View

api/Excel.workbookquery.md

Line 20, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'Excel.workbookquery.refresh.md'.

api/Office.MsoThemeColorSchemeIndex.md

Line 33, Column 1: [Warning: multiple-h1s - See documentation] Multiple H1s(H1 'Remarks') are not allowed. You can only have one top-level heading.
Line 35, Column 302: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/office/vba/api/office.msothemecolorindex' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 39, Column 1: [Warning: multiple-h1s - See documentation] Multiple H1s(H1 'Example') are not allowed. You can only have one top-level heading.
Line 35, Column 302: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/office/vba/api/office.msothemecolorindex' will be broken in isolated environments. Replace with a relative link.

access/Concepts/Miscellaneous/in-operator-microsoft-access-sql.md

Line 21, Column 30: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/office/client-developer/access/desktop-database-reference/select-statement-microsoft-access-sql?redirectedfrom=MSDN' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 21, Column 30: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/office/client-developer/access/desktop-database-reference/select-statement-microsoft-access-sql?redirectedfrom=MSDN' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Can't rearrange the last card in your hand

You can drag it, but it doesn't move

Can't reproduce now. Guess it was a connection issue.


Update IAM service definitions

This is an auto-generated PR with IAM service updates.

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

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


reword installation, mention persistent SQLite, fix a typo

addresses #11

I hope this is acceptable @dainnilsson

Looks good, much clearer. Thanks!
I'll look into addressing #12 at some point as well.


Increase the timeout in HTTPConnectionPool()

Hello,
I am wondering where can I set the timeout parameter for the HTTPConnectionPool().
It occurred a ValidationError when I tried to add a model:

Here is the log of the backend:
[2022-07-16 21:36:15,918] [INFO] [werkzeug::_log::225] <IP_ADDRESS> - - [16/Jul/2022 21:36:15] "GET /health HTTP/1.1" 200 -
###【Loading model...】
###【Finish Loading model from label_studio_ml/examples/my_segment/mask_rcnn_R_50_FPN_1x.yaml label_studio_ml/examples/my_segment/mask_rcnn_R_50_FPN_1x.yaml on cpu 】
[2022-07-16 21:36:24,047] [INFO] [werkzeug::_log::225] <IP_ADDRESS> - - [16/Jul/2022 21:36:24] "POST /setup HTTP/1.1" 200 -

The model can be successfully loaded, but it takes a little longer time(about 7 seconds) because it is on cpu.
read timeout=3.0 is too short for loading, where can I change this parameter?
Thank you.

@makseq @KonstantinKorotaev
Could you please check the problem? Thank you.

Hi @pinecho
Add this environment variable ML_TIMEOUT_SETUP in seconds and restart Label Studio.

It works, thanks so much!


No issue just some praise!

No problem, just praise!
What I can see so far is really great, keep at it, it's going to be something really great!
Most of the forums that are still used relatively often these days are from the games industry, so I would advise you to focus on cool user profile pages and the ability to easily customize the style of the forum.
Gamers want it to be playful, with avatar and header images and a message feed in the profile, and game developers love to be able to customize forums to their corporate identity, i.e. their company colors, etc.
I wish you the best of luck and that you never get frustrated.
Kind regards, Andy :)

Thank you for this compliment.
Although the project development is slow, I pay a lot of attention to adding new features, stability and security. The commits I have sent to Github in recent months are especially related to stability and performance.

Stability, performance and security are very important I think it's good that you value this.
Just keep going, everything looks really good so far.


feature to control each image download to use different download operation

sometimes we hope to control some url(depend on url host) or some UIImageView (depend on different pages to show image view) to download image in a special way.
so i  add a feature to NSObject a method sd_setTag: to attach additional information so that we could have an opportunity to choose download operation class to perform file download.
how to use this feature?
1: UIImageView: call sd_setTag:  method to attach additional information before call  sd_setImageWithURL to show image.
2: for other situations, such as directly call [SDWebImageManager downloadImageWithURL.....] or [SDWebImageDownloader downloadImageWithURL....],  we could call [NSURL sd_setTag:] to attach additional informations before call those methods.

This doesn't sound like a core feature that we should add to the library. It's very hard to maintain a lot of marginal features. You can implement this in an extension to SDWebImage. I am going to close the PR. Thanks for the intention, but I hope you understand why we can't add a lot of features in.


request to join node-core-utils team

I'd like to add myself to the node-core-utils team.  I also notice there is a pending request from @lundibundi in the GitHub interface. Is it OK to add myself and them? Or is there a process?

+1. I'm not aware that a process was written down -- perhaps it should as there's also an implication that members of the team also be added as maintainers on npm: https://github.com/nodejs/node-core-utils/issues/494

+1 from me as well!

OK, added @lundibundi and myself.


python shebang

Current OS: Windows10 Creators update
Atom.io: 1.21.1
Python: 3.6.3
Scenario:

file.py file containing shebang: #!/usr/bin/env python3
running python file.py = OK
running with atom runner = error (see below)
disabling shebang and running with atom runner = OK

Error:
Unable to find command: /usr/bin/env
Are you sure PATH is configured correctly?

ENV PATH: C:\Program Files (x86)\Python36-32\Scripts\;C:\Program Files (x86)\Python36-32\;C:\Python27\;C:\Python27\Scripts;...

Error: spawn /usr/bin/env ENOENT
    at exports._errnoException (util.js:1022:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
    at onErrorNT (internal/child_process.js:359:16)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

The only fix here would be to have a switch to disable shebang processing in Atom runner.


Fix run_partially

Now will only run files that have changed
Bump to 0.0.4

I haven't thoroughly tested this or anything but it is doing what I want.

Actually hold off on merging until I get another commit up later so that this doesn't break run_all

Alright should be good to go if you would like to merge @mdeering

Master repository for guard-yardstick is here https://github.com/X0nic/guard-yardstick I have the same pull request ready to into there here: https://github.com/X0nic/guard-yardstick/pull/2
I just got permissions on that repository and on rubygems.org so I will be releasing it this week. Give that pull request a watch.

@backus Just released 0.1 to rubygems.org


How to get child text in parent charkey ?

I have the following example:
const parser = new xml2js.Parser({
    explicitArray: false,
});

parser.parseStringPromise(
    `<sentence>This library is <adjective>awesome</adjective>!</sentence>
`).then(console.log)

Which results to:
XML: {
  "sentence": {
    "_": "This library is !",
    "adjective": "awesome"
  }
}

You can see that the child tag adjective text is striped away in the charkey. But the problem is that there is no way to find where the awesome word originally was placed, making impossible to reconstruct the sentence by hand.
I would love to have a text key somewhere which would return the whole text This library is awesome!. Is that possible ?

You can try overload the ontext()method, for example"
    const parser = new xml2js.Parser({ explicitArray: false });

    const ontextFun = parser.saxParser.ontext;

    parser.saxParser.ontext = function (text) {
        console.log(text);
        ontextFun(text);
    }

    parser.parseStringPromise(`<sentence>This library is <adjective>awesome</adjective>!</sentence>`).then(console.log);

The output of parser.saxParser.ontext：
This library is
awesome
!

Im actually in need of doing this as well, none of the function parsers were able to help me achieve this. The example above by @Omega-Ariston did not work for a full xml document..
Does anyone have any ideas on how to do this?


Should we add another default SBOM importer?

We only have the RHT one now. If there was another big repo of SBOM's somewhere, we might explore the limits of trustify by importing them.

from @JimFuller-RedHat

I had a look around for sbom corpuses https://sbombenchmark.dev has a few k

the sbombenchmark.dev exposes an api
https://sbombenchmark.dev/api/data?draw=1&columns%5B0%5D%5Bdata%5D=file_url&columns%5B0%5D%5Bname%5D=&columns%5B0%5D%5Bsearchable%5D=true&columns%5B0%5D%5Borderable%5D=false&columns%5B0%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B0%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B1%5D%5Bdata%5D=target_name&columns%5B1%5D%5Bname%5D=&columns%5B1%5D%5Bsearchable%5D=true&columns%5B1%5D%5Borderable%5D=true&columns%5B1%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B1%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B2%5D%5Bdata%5D=score&columns%5B2%5D%5Bname%5D=&columns%5B2%5D%5Bsearchable%5D=true&columns%5B2%5D%5Borderable%5D=false&columns%5B2%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B2%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B3%5D%5Bdata%5D=spec&columns%5B3%5D%5Bname%5D=&columns%5B3%5D%5Bsearchable%5D=false&columns%5B3%5D%5Borderable%5D=true&columns%5B3%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B3%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B4%5D%5Bdata%5D=format&columns%5B4%5D%5Bname%5D=&columns%5B4%5D%5Bsearchable%5D=false&columns%5B4%5D%5Borderable%5D=true&columns%5B4%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B4%5D%5Bsearch%5D%5Bregex%5D=false&columns%5B5%5D%5Bdata%5D=creator&columns%5B5%5D%5Bname%5D=&columns%5B5%5D%5Bsearchable%5D=true&columns%5B5%5D%5Borderable%5D=true&columns%5B5%5D%5Bsearch%5D%5Bvalue%5D=&columns%5B5%5D%5Bsearch%5D%5Bregex%5D=false&order%5B0%5D%5Bcolumn%5D=1&order%5B0%5D%5Bdir%5D=desc&start=0&length=10&search%5Bvalue%5D=&search%5Bregex%5D=false&_=1727715463266

guessing the start and length  would allow one to get to the download urls

lastly - sqlite database with list of sboms and their file url - https://github.com/interlynk-io/sbomdb?tab=readme-ov-file


use react-timers throughout

This fixes a tonne of errors where these timeouts were being called when the component was no longer mounted.
To reproduce, just watch the console while moving around in the examples.

nice work :+1:


warning: profiles for the non root package will be ignored, specify profiles at the workspace root

Another little complication from #730; this warning now shows up when building the main project.
Moving this profile to the top level manifest should be fine, but maybe @vlovich can clarify why we were turning on lto for benchmarking?

I think I put it in cause, especially for micro-benchmarks like this, you want to test the most optimized code & not to waste time tweaking things from readable to less readable because you're optimization level wasn't high enough.  YMMV though - feel free to take it out if it doesn't make sense anymore.


speed up leader election

this helps with upgrade timing because save one minute on the lease.

infra
/retest


Another PR chart.yaml fix PR

Signed-off-by: jonahjon<EMAIL_ADDRESS>Adds change to do 2 Things

Modify Helm Chart.yaml version to be what the destination Tag will be be, re-package, and push to destination repository.
Override the Image Digest references to the images digests inside the helm chart so they will always be in sync.

/woof


Publishing API deployment refactor

This follows PR #150 in changing how we register new task
definitions. Rather than having a separate state file for
each task definition, we output the task definition from
the main govuk-publishing-platform module, and use the
task definition JSON to register the task definition using
the AWS CLI.
This should result in less boilerplate in the Terraform config
and fewer state refreshes when registering a task definition.
Trello card

The Concourse CI build is currently failing due to an issue affecting Concourse.


style: update header and drawer

Notion ticket link

Update Header and Drawer Component

Implementation description

add logos (unofficial logos, assets will be received soon according to binal)
update header/drawer to match mobile and desktop designs
create different views for logged in vs. logged out

Mobile Header

Mobile Drawer (Logged In)

Mobile Drawer (Logged Out)

Desktop Header (Logged In)

Desktop Header (Logged Out)

Steps to test

Run frontend and view the mobile/desktop versions for the logged out view by resizing the window
To view logged in view, hardcode a value for authenticatedUser in the code in Header.tsx 🙃

What should reviewers focus on?

Checklist

[x] My PR name is descriptive and in imperative tense
[x] My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
[x] I have run the appropriate linter(s)
[x] I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@hanlinc27 cleaned up the code a bit, thanks for the suggestions :D


Edit charity message and add charity list

The delivery fee is also optional, and up to you. All proceeds go to charity.

I'd like to slightly reword this. Perhaps something like the following:

All proceeds go in equal part to the Electronic Frontier Foundation, Partners in Health, and Health in Harmony. While we don't require a delivery fee, your contribution to our efforts to raise money for these charities is greatly appreciated.

Also use links to the websites of these organizations.

Fixed with 40b63ed


Merge devel to master, release 0.2.0

Towards minor release 0.2.0 -- changes include:

Added processing documents in bulk mode (endpoint: /procees_bulk)
Added OpenAPI+Swagger specs
Added tests
Integration with TravisCI (builds + tests)
Minor bugfixes

also was wondering if bulk processing of n=1 would work for single document processing?


Updated Dark Mode switch for your website

#34 added feature
added your cool website a new cool feature of switch theme

This button is looking too misplaced @STDYGIT Do some changes.After alterations please open a new PR for review.
Thank you!


Build attributes

Build attributes associated with files is something I'd like to generalize.
What are those, you might wonder?  Well....  we actually already have that already in some cases (_NO_INST keyword suffix), or lack them in others (keeping track of exactly what programs / scripts are installed where).  One is handled by Configure, the rest is hard coded in the build file templates.
Seeing that it's not terribly important for Configure to know the exact details, and having stuff hard coded in the build file templates is directly bad, we might want to see attributes as a message being passed from the build.info files to the build file templates.
So, I'd like to device some kind of build.info attribute syntax, which is always optional, but can help indicate to the build file templates what the intention is with a produced file.  For example:
# indicate that these programs should be installed
PROGRAMS{noinst}=test/foo
# indicate that these scripts are if the "misc" category
SCRIPT{misc}=tools/c_rehash

My intent  is for these attributes to be used primarily for end products, i.e. programs, scripts, libraries, ...  They would simply be bits of information in string form that get passed on to the build file templates, and most of all, Configure doesn't need to know anything about them other than the syntax and where to store them.

should be installed and PROGRAMS{noinst}= ?  One negative too many or few.

Thank you, corrected

Would you have some time to answer some of question sometime?
Samuel J.

Depends, Samuel J.  Is it related to this issue, then fire away.  If it's something other related to OpenSSL, please create another issue.


Unable to compute fixes

I run dart fix --dry-run command gives me options. one of them is dart fix --apply --code prefer_const_constructors . but the problem is when I run dart fix --apply --code prefer_const_constructors it gives me this error:  Unable to compute fixes: The lint 'prefer_const_constructors' is not enabled; add it to your analysis options and try again.
What should I do for fixing problem ? cause When I run dart fix --apply --code unused_import there was no problem and fixed unused imports.
my analysis_options.yaml file.
include: package:flutter_lints/flutter.yaml

linter:
  rules:

Hmmmm. This seems curious.
That lint should be available via your include of flutter_lints.
https://github.com/flutter/packages/blob/main/packages/flutter_lints/lib/flutter.yaml
Out of curiosity do any lints work for you?  (Interestingly unused_import is an analyzer diagnostic).
For example, what happens if you run:
dart fix --apply --code avoid_empty_else

I wrote this code in same .dart file
 if ("1" == "2")
     print("1");
      else
        ;
    print("2");

then run dart fix --dry-run
result :
To fix an individual diagnostic, run one of the following commands:
  dart fix --apply --code annotate_overrides
  dart fix --apply --code avoid_function_literals_in_foreach_calls
  dart fix --apply --code curly_braces_in_flow_control_structures
  dart fix --apply --code empty_statements
  dart fix --apply --code no_leading_underscores_for_local_identifiers
  dart fix --apply --code prefer_adjacent_string_concatenation
  dart fix --apply --code prefer_const_constructors
  dart fix --apply --code prefer_const_constructors_in_immutables
  dart fix --apply --code prefer_final_fields
  dart fix --apply --code prefer_interpolation_to_compose_strings
  dart fix --apply --code prefer_is_empty
  dart fix --apply --code sized_box_for_whitespace
  dart fix --apply --code sort_child_properties_last
  dart fix --apply --code unnecessary_brace_in_string_interps
  dart fix --apply --code unnecessary_this
  dart fix --apply --code unused_catch_stack
  dart fix --apply --code use_key_in_widget_constructors

then :
(base) mirzali@Mirzalis-MacBook-Pro ondan % dart fix --apply --code avoid_empty_else
Computing fixes in ondan...    39.6s
Unable to compute fixes: The lint 'avoid_empty_else' is not enabled; add it to your analysis options and try again.
(base) mirzali@Mirzalis-MacBook-Pro ondan % dart fix --apply --code empty_statements
Computing fixes in ondan...    40.9s
Unable to compute fixes: The lint 'empty_statements' is not enabled; add it to your analysis options and try again.

@pq yes I'm using flutter_lints, which is default.

@devoncarew, it looks like this is a duplicate of https://github.com/dart-lang/sdk/issues/51427?

I think so; this is fixed in main (and will be part of the next stable release). The issue hinged on having an analysis_options file in a sub-directory w/ different settings than that from one in a parent dir.

Thanks, @pq, and @devoncarew. I'll wait for the next stable release.

Sorry for the hassle and thanks for the thoughtful report @Shahmirzali-Huseynov!


Error when installing setup.py

Based on the instructions given I installed fiona, shapely etc, and now I' m trying to install pysim5g permanently with
python setup.py develop
Running the above, or the below
python setup.py install
I get the following error.
Any idea on how to fix it?

Hi @Pante9 - it looks like a syntax error from the f"string" in the last line there.
Can you try upgrading the python in your conda environment to 3.7? Try running:
conda activate pysim5g
conda install python=3.7

Then try again.
If that fixes it, we should change the README instructions, which use 3.5.

Thanks for your time @tomalrussell
I tried with python 3.7 and got the following error this time

Okay - the end of the error message there should be some help - are you running from a ZIP download of the code?
Two different things you could try:

clone the repository using git clone https://github.com/edwardoughton/pysim5g.git on the command line or use an application like GitHub Desktop, and then run python setup.py develop from that folder.
run pip install git+https://github.com/edwardoughton/pysim5g.git#egg=pysim5g to install the package directly from GitHub - this should work if you just want to run scripts using pysim5g, but is not so good if you want to make changes to the source code


Clearing form fields after publishing a comment

Repro:

Go to any post
Fill out a comment
Publish the comment

Expected:
Empty comment form
Actual:
Data from the just published comment still exist in the form

See PR #143


[ingest bottleneck 1] Fix Pandas joins in handroll_geometrize_routes() or find other way to unify needed information

This is taking place on branch actually_geometrize_and_ingest_to_postgres_transit_stations. The relevant function is handroll_geometrize_routes() in scripts/extract_scheduled_gtfs.py.
In order to ingest the TransitRoute data into the backend properly, we need to have all this information for each route* on a single row of a single GeoDataFrame:

shape_id (exists in GTFS files trips.txt and shapes.txt)
route_id (exists in GTFS files trips.txt and routes.txt)
The other relevant columns from routes.txt that go into the data model (service_id, route_type, route_long_name, and route_color)
A LineString representing the coordinates of the transit line as it curves around on the Earth's surface (each row in the original shapes.txt represents a single point on such a LineString; the gtfs_kit library's method geometrize_shapes() produces a GeoDataFrame with one row for each shape_id and its geometry).

Per the GTFS specification table relationships, the way to get this information joined is to join shapes with trips using shape_id as the key to join ON, and then to join trips with routes to get the route columns on. However, every attempt I have made to use pd.merge() to put the geometrized shapes together with the rest of the info has failed, resulting in a joined table where all columns from one of the source tables have every value replaced with NaN. This defeats the purpose of a join altogether.
If we can figure out how to make this merge work properly (or, alternately, just create a GeoDataFrame with the shapes and regular DataFrame with other info, that are guaranteed to have the same routes in the same order, and iterate through them both row-by-row at the same time to create TransitRoute objects on ingest), the main remaining bottleneck to us having route data for all three cities ingested into our backend will be cleared. Any help much appreciated!
*technically, each possible trip within the system. I think that for branching routes like the Green Line or Metra Electric, this results in several distinct LineStrings of the same color that would look unified when presented to user on frontend map.

Diagnostic screenshots:

Hypotheses:

Something about trying to merge a GeoDataFrame and a regular DataFrame gets rejected
some of the data that should be the same is subtly of different data type so join fails to find them
I did pd.merge() syntax wrong

working hypothesis: leading whitespace in routes_trips is absent in shapes, which means any join fails equality check on join key

It was that


SharePoint Framework -  React  JSX element type 'HotTable' is not a constructor

Description
Hello first of all, thank you for this wonderful library!
I having an issue when i try to import the component in a SharePoint Framework App(SPFX), the error is the following:

JSX element type 'HotTable' is not a constructor function for JSX elements. Property 'setState' is missing in type 'HotTable'.

I try with two version of SPFX one use react 15 and the other use react 16, with both i have the same error
Steps to reproduce

Create a sharepoint framework app (2016 or online version) Set Enviroment
Install the hot table component
Create a component like the following:

import * as React from 'react';
import './App.css';
import { HotTable } from '@handsontable/react';
import Handsontable from 'handsontable';
import 'handsontable/dist/handsontable.full.css';

export interface AppProps {
   data : any
}
export default class App extends React.Component < AppProps, {} > {

  constructor(props : AppProps){
    super(props);
  }

  public render():React.ReactElement<{}> {
    return (
      <div id="hot-app">
        <HotTable.HotTable data={[[],[]]} colHeaders={true} rowHeaders={true} width="600" height="300" />
      </div>
    )
  }

}

Your environment

React wrapper version:

For SPFX Online:     { "react": "16.7.0", "react-dom": "16.7.0" }
For SPFX 2016: {    "react": "15.4.2",    "react-dom": "15.4.2", }

Handsontable version: 7.0.0 "@handsontable/react": "^3.0.0"
Browser Name and version:
Operating System: Windows 10

SPFX 2016 Package.json
{
  "name": "hansone-table-testing",
  "version": "0.0.1",
  "private": true,
  "engines": {
    "node": ">=0.10.0"
  },
  "scripts": {
    "build": "gulp bundle",
    "clean": "gulp clean",
    "test": "gulp test"
  },
  "dependencies": {
    "react": "15.4.2",
    "react-dom": "15.4.2",
    "@types/react": "0.14.46",
    "@types/react-dom": "0.14.18",
    "@types/react-addons-shallow-compare": "0.14.17",
    "@types/react-addons-update": "0.14.14",
    "@types/react-addons-test-utils": "0.14.15",
    "@microsoft/sp-core-library": "~1.1.0",
    "@microsoft/sp-webpart-base": "~1.1.0",
    "@microsoft/sp-lodash-subset": "~1.1.0",
    "@microsoft/sp-office-ui-fabric-core": "~1.4.0-0",
    "@types/webpack-env": "1.13.1",
    "@types/es6-promise": "0.0.33",
    "handsontable": "^7.0.2",
    "@handsontable/react": "^3.0.0"
  },
  "resolutions": {
    "@types/react": "0.14.46"
  },
  "devDependencies": {
    "@microsoft/sp-build-web": "~1.1.0",
    "@microsoft/sp-module-interfaces": "~1.1.0",
    "@microsoft/sp-webpart-workbench": "~1.1.0",
    "gulp": "~3.9.1",
    "@types/chai": "3.4.34",
    "@types/mocha": "2.2.38",
    "ajv": "~5.2.2"
  }
}

Hey @GuillermoADC
thank you for sharing the issue, however, we do not support this framework. If you could share a demo (can be zip) we'd be able to test it out but I cannot mark it as a library bug.

Hey @AMBudnik
Thank you for the response, here is a link for the proyect https://drive.google.com/open?id=17vRyl63iJRso3Da7WR-v80sIwcSqB5nP, there are two files one is the zip for the proyect and the other is the screen capture of the error.
Steps To Run
1 - Extract the zip
2 - npm i
3 - gulp serve

Hi @GuillermoADC,
I haven't checked your example yet, but is there a chance that this is related to #123 #116 #106?

Hi @wojciechczerniak, how are you?
I Will check that

Hi @wojciechczerniak first of all thank you for the support.
I read the post related that you sendme and i manage to fix the issue.
First I tried only adding the following settings as you sugest on that posts
{
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
}

But throws the same error, so i tried some approach that i read there. I partially copied the tsconfig from the create-react-app, and test with each of the defirent properties. This are the properties that make the app works.
{
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "isolatedModules": true
}

Good finding, I wasn't aware of isolatedModules option. Thanks for your solution, might help someone with similar issue.

Hi @GuillermoADC,
We merged wrapper for React into our main repository, handsontable/handsontable. Now library and wrapper are available in monorepo.
We also transfer all issues into one common issue board, where you can follow work progress and request issues.


Can't host 2.2 in Azure App Service

I upgraded from 2.1.6 to 2.2.0, works nicely locally on IIS or Kestrel, but when deployed to Azure App Service, the API (it's an aspnetcore webapi app) won't start. I get
HTTP Error 500.21 - Internal Server Error
Handler "aspNetCore" has a bad module "AspNetCoreModuleV2" in its module list
Is this not supported yet? I can see the SDK is available on the machine.

The new ANCM is not yet on Azure App Service.
See Damian Edwards' comment in https://blogs.msdn.microsoft.com/webdev/2018/12/04/asp-net-core-2-2-available-today/

We will make an announcement when ANCMV2 is publicly available on Azure App Services. There isn't an easy way to get module information from IIS via command line.

Thanks. With previous updates I would just wait until I saw the SDK available on the machine, by going to the console on Azure App Services and doing dotnet --list-sdks. So this time I was surprised when I saw 2.2.0 was on the machine but it still didn't work. I guess this one went deeper! As far as feedback goes, though, it's quite hard to know when things that are released as nuget packages actually work on App Services...its a trial and error process, not the best.

Closing as ANCMV2 is now supported on App Service


Cleanup

removed baseurl setup (there are problems with multistore shops) for #6
removed check if it is live database or no (databases are not always named like that unfortunately)

@razbakov Why did you remove the live database check?

i know that this issue was only about baseurl,
but i just create a PR of my current master
why did I remove database check?
databases are not always named like that unfortunately:
for example my test database is named project_ar

@razbakov I extended the script to allow to configure (or to skip) base urls on store level. Thanks for your helpful input!


After Upgrade to 4.2.0 - got error

Describe the bug
Incrementally upgraded to version 4.2.0. After upgrade completed, tried to run application and then got error as shown below.
Expected behavior
After upgrade to 4.2.0 expected to work app without errors.
Screenshots

Environment (please complete the following information):

Lens Version: 4.2.0
OS: Mojave 10.14.5 (18F132)
Installation method (e.g. snap or AppImage in Linux): direct .dmg installation

Logs:
I have replaced apiUrl with [REDACTED] to hide info.
info: 📟 Setting Lens as protocol client for lens://
info: 📟 succeeded ✅
info: 🚀 Starting Lens from "/Users/samundra/Library/Application Support/Lens"
info: 🐚 Syncing shell environment
info: 💾 Loading stores
info: [STORE]: LOADED from /Users/samundra/Library/Application Support/Lens/lens-cluster-store.json
info: [STORE]: LOADED from /Users/samundra/Library/Application Support/Lens/lens-workspace-store.json
info: [STORE]: LOADED from /Users/samundra/Library/Application Support/Lens/lens-extensions.json
info: [STORE]: LOADED from /Users/samundra/Library/Application Support/Lens/lens-filesystem-provisioner-store.json
info: [STORE]: LOADED from /Users/samundra/Library/Application Support/Lens/lens-user-store.json
info: 🔑 Getting free port for LensProxy server
info: [CLUSTER-MANAGER]: init cluster {"id":"6afbf7d7-0b03-4f10-87b0-93adf7b1a9ab","name":"uat.test.io","initialized":false,"ready":false,"online":false,"accessible":false,"disconnected":true}
info: [CLUSTER-MANAGER]: init cluster {"id":"b1f38fb6-9cb6-44c9-aef3-a0f2c918f370","name":"qa.test.io","initialized":false,"ready":false,"online":false,"accessible":false,"disconnected":true}
info: [CLUSTER-MANAGER]: init cluster {"id":"d26e0914-3d7d-4818-84a7-4a8bd3e6e4ee","name":"prod.test.io","initialized":false,"ready":false,"online":false,"accessible":false,"disconnected":true}
info: 🔌 Starting LensProxy
info: [LENS-PROXY]: Proxy server has started at http://localhost:56830
info: 🔎 Testing LensProxy connection ...
info: ⚡ LensProxy connection OK
info: 🖥  Starting WindowManager
info: 🧩 Initializing extensions
info: [EXTENSION-DISCOVERY] loading extensions from /Users/samundra/Library/Application Support/Lens
info: [CLUSTER]: "uat.test.io" init success {"id":"6afbf7d7-0b03-4f10-87b0-93adf7b1a9ab","context":"uat.test.io","apiUrl":"[REDACTED]"}
info: [CLUSTER]: "qa.test.io" init success {"id":"b1f38fb6-9cb6-44c9-aef3-a0f2c918f370","context":"qa.test.io","apiUrl":"https://api.qa.test.io"}
info: [CLUSTER]: "prod.test.io" init success {"id":"d26e0914-3d7d-4818-84a7-4a8bd3e6e4ee","context":"prod.test.io","apiUrl":"https://api.prod.test.io"}
info: [EXTENSION-INSTALLER] installing dependencies at /Users/samundra/Library/Application Support/Lens
info: [WINDOW-MANAGER]: Loading Main window from url: http://localhost:56830 ...
info: [EXTENSION-INSTALLER] dependencies installed at /Users/samundra/Library/Application Support/Lens
info: [EXTENSION-DISCOVERY] watching extension add/remove in /Users/samundra/.k8slens/extensions
info: [EXTENSION]: enabled<EMAIL_ADDRESS>info: [STORE]: LOADED from /Users/samundra/Library/Application Support/Lens/extension-store/lens-survey/preferences-store.json
info: [EXTENSION]: enabled<EMAIL_ADDRESS>telemetry main extension activated
info: [STORE]: LOADED from /Users/samundra/Library/Application Support/Lens/extension-store/lens-telemetry/preferences-store.json
info: [EXTENSION]: enabled<EMAIL_ADDRESS>info: [WINDOW-MANAGER]: Main window loaded
info: 📡 Checking for app updates
info: Checking for update
info: [CLUSTER]: activate {"id":"d26e0914-3d7d-4818-84a7-4a8bd3e6e4ee","name":"prod.test.io","initialized":true,"ready":false,"online":false,"accessible":false,"disconnected":true}
info: [CLUSTER]: bind events {"id":"d26e0914-3d7d-4818-84a7-4a8bd3e6e4ee","name":"prod.test.io","initialized":true,"ready":false,"online":false,"accessible":false,"disconnected":true}
info: [CLUSTER]: reconnect {"id":"d26e0914-3d7d-4818-84a7-4a8bd3e6e4ee","name":"prod.test.io","initialized":true,"ready":false,"online":false,"accessible":false,"disconnected":true}
info: [KUBE-AUTH]: out-channel "kube-auth:d26e0914-3d7d-4818-84a7-4a8bd3e6e4ee" {"data":"Authentication proxy started\n","meta":{"id":"d26e0914-3d7d-4818-84a7-4a8bd3e6e4ee","name":"prod.test.io","initialized":true,"ready":false,"online":false,"accessible":false,"disconnected":true}}
info: Update for version 4.2.0 is not available (latest version: 4.2.0, downgrade is disallowed).
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus-server" not found
warn: PrometheusLens: failed to list services: services "prometheus-server" not found
warn: PrometheusLens: failed to list services: services "prometheus" not found
info: [STORE]: SAVING /Users/samundra/Library/Application Support/Lens/lens-cluster-store.json
(node:8048) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
this was working fine before upgrade. No change in kubeconfig.
``

Version 4.1.5 works fine without any issues.

I also am having problems post upgrade.  We have 4 clusters: dev works and our new eks cluster works, staging and production are producing an error.  dev, staging, production all on k8s 10.12.  Producing errors in ui:
Component stack:

    at Sidebar (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:8493742)
    at aside
    at div
    at IU (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:8498861)
    at AA (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:7676789)
    at t (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:221963)
    at kU (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:8500365)

and
typeError: Cannot read property '0' of undefined
    at yf.getVersion (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:7872056)
    at yf.getResourceApiBase (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:7871705)
    at http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:8494085
    at Array.map (<anonymous>)
    at http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:8494050
    at Array.map (<anonymous>)
    at BU.get crdSubMenus (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:8493932)
    at ce (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:12079)
    at e.computeValue (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:18380)
    at e.trackAndCompute (http://f807043b-bb69-4d27-acb7-7eee104018d3.localhost:52898/build/Lens.js:2:18206)

Just to confirm, is your CRD v1 or is it an earlier version?

I also am having problems post upgrade. We have 4 clusters: dev works and our new eks cluster works, staging and production are producing an error. dev, staging, production all on k8s 10.12

Update, post 4.2.1 upgrade, still happening (dev is ok, staging, prod are generating the errors noted in my original post).  Not sure I understand why -- the clusters really are nearly identical, generally speaking.  The configs are as well:
Dev: Working
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: LS0<redacted>S0t
    server: https://api.us-west-1.dev-2.clearbit.io
  name: us-west-1.dev-2
contexts:
- context:
    cluster: us-west-1.dev-2
    user<EMAIL_ADDRESS>  name: us-west-1.dev-2
current-context: us-west-1.dev-2
kind: Config
preferences: {}
users:
- name<EMAIL_ADDRESS>  user:
    auth-provider:
      config:
        client-id: 0oa22siuvyd8SNFJ42p7
        client-secret: tM<redacted>50
        id-token: eyJ<redacted>olQ
        idp-issuer-url: https://clearbit.okta.com/oauth2/k8s
        refresh-token: rIt<redacted>fCM
      name: oidc

Prod: Fails
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: LS0<redacted>S0t
    server: https://api.us-west-1.clearbit.io
  name: us-west-1.prod
contexts:
- context:
    cluster: us-west-1.prod
    user<EMAIL_ADDRESS>  name: us-west-1.prod
current-context: us-west-1.prod
kind: Config
preferences: {}
users:
- name<EMAIL_ADDRESS>  user:
    auth-provider:
      config:
        client-id: 0oa2h7xhp6y19QVKf2p7
        client-secret: smR<redacted>nFN
        id-token: eyJ<redacted>cew
        idp-issuer-url: https://clearbit.okta.com/oauth2/k8s
        refresh-token: 0yf<redacted>jzM
      name: oidc

Let me know if there's any tracing or logging or other stuff I might be able to provide.
Until then, I guess it's back to previous version :-).
** And hey: brilliant tool that I have totally come to depend upon -- thanks!!!!**

This will be fixed in 4.2.2 (not 4.2.1). The reason why it is crashing might be because you are using an older version of kubernetes (only a guess here).

I have Lens version 4.2.3. This issue has been fixed in 4.2.3.  I can access dashboard without any errors
Thank you for the quick solution. 🎉 🎉 🎉 🎉

You are very welcome.


gltf exporter: unwanted split of a large mesh

In Blender 2.8 Beta, I've imported Stanford dragon and exported it in ".gltf + bin" format. In the generated gltf file, geometry is split in multiple "primitive objects" such that indices are unsigned short, instead of keeping only one "primitive object" using unsigned int indices.

Has any progress been made on this issue? I still notice very large combined meshes getting split into multiple smaller meshes after exporting to .glb.

I also met this problem, how should solve excuse me?

It looks like the fix could be as simple as changing this
https://github.com/KhronosGroup/glTF-Blender-IO/blob/2fb64552718638d8d4a1b80ebd6f82c9d4b293fc/addons/io_scene_gltf2/blender/exp/gltf2_blender_extract.py#L1035
to<PHONE_NUMBER>?

Just for some history here:

glTF indicies come in 16-bit and 32-bit unsigned integer forms (and 8-bit, but 8-bit was deprecated on account of being mostly useless and unsupported in various graphics APIs).
The highest integer for a given number of bits (ie, 65535 for 16-bit) is disallowed due to "primitive restart" values used by various graphics APIs.
This exporter previously would ask the artist whether they prefer 16 or 32 bits, with a combo box on the UI.  Artists generally didn't like being asked such questions, so the UI for this was removed.
The intention behind removing the UI was originally to have Blender auto-select 16 or 32 bits per primitive, based on number of verts needed.  But, this logic was never written, so the choice of 16-bit became hard-coded.
If artists chose 16-bit for a mesh with more than 65k verts, there was some protective code that would split the mesh into parts, to keep the vertex count below the limit.  This code is still in use today, and affects all large meshes.

Ideally, we should get rid of the mesh-splitting code, and have the exporter intelligently auto-select 16 or 32 bits.  This would be based not on the number of vertices in Blender, but the actual number of vertices about to be exported to glTF for a given mesh/primitive.

The exporter does auto-select

Hmm, I haven't done any 32-bit sized meshes in a long time, maybe this is working already?
I don't know if Blender supports more than 2**32 verts, would the splitting code be useful there at all?  Seems like the memory might run out.  But I haven't tested large meshes in a while.

It looks like (from the bmesh source code) Blender can handle no more than 2**31 verts in a mesh and maybe less. A primitive can have more verts than that because of splitting (eg. when a vert is on a UV seam, etc.) though.
Exporting seem to work fine for me without the splitting code for both index sizes. I'll do a PR and see if the tests pass.

Just to make sure this is mentioned, uint32 indices are unsupported in the core WebGL 1.0 spec, per https://github.com/KhronosGroup/glTF/issues/570.  This requires OES_element_index_uint. But support for that extension is pretty good these days, 98% of devices, so I'm not particularly concerned with removing the splitting logic in this exporter.

^More specifically, the spec allows uint32 indices, so applications that need to support the remaining 2% of devices should be prepared to split primitives themselves at runtime or to preprocess the mesh some other way.

@donmccurdy Just to be clear, this removal of code doesn't appear to change what the exporter is currently doing in master.  It was already auto-upgrading to UINT, in spite of my misunderstanding.  So this is basically a removal of dead code.


Feat/new responsive design lab

@DavidScales Hey could you look through this pull quickly and lmk if it's ok? I removed a folder for one of the steps and made some changes to the code to match the new lab.

Looks good to me


Not updating

Not sure if it's just me, but I have to keep clearing and re-adding monkey madness 1 in order for the plugin to update.

Are you on Runelite or another client @iDznPB ? This has been a known issue for other clients, but not from our side.

Ah that might be my fault then, I was using it on OPRS, I wasn't aware there would be a difference, I'll jump back to RL. Thanks for getting back to me!


Duplicate cids when creating with bootstrapData

Steps to reproduce:

Create a fairly long form and store the generated schema (mine had 18 fields)
Instantiate FormBuilder with the schema on a new web page (akin to saving and loading the schema for editing)
Add a new field or 2 and check their cid values after saving. You'll likely find that their cid's already exist in the initial schema

In a nutshell, whenever a Bakcbone object (Collection/Model or View) is created, Backbone uses _.uniqueId to generate a new client id and is assigned as the objects cid.
Formbuilder uses the cid as the forms unique identifier. When you restore the schema from bootstrap data, the existing cid is ignored by Backbone and a new one is created. Since everytime a page is loaded, the count re-starts from 1 an already existing id will likely be generated and attached to a new field. Ideally, if the same steps used to create the schema are used to restore it, the ids should never duplicate but the problem is that while restoring, the fields are created all at once while when creating a field is created with a number of views so the cid is incremented further when creating than when restoring.
The fix that I propose (and I'm working on) to to generate a unique id specifically for each field and that formbuilder keeps track of. Whenever a field is created, we'll check the highest number from the existing id's and increment from that. The same will be done when bootstraping. This will however break any existing schemas since we cant use the cid attribute to store the unique id, because Backbone ignores it a creates its own.

@SongTanmay The issue does still exist if you still prefix your fields with a c because backbone will create models with the same prefix so if you already have a c2 in your schema a new c2 will be created and yours will be overridden. Th solution is to to always use custom server side ids if a field doesn't yet have an id

@larryweya I think client side IDs will also work, you will just have to change this line
model.attributes.cid = model.cid

in main.coffee, instead of using model.cid use a custom function.
This way you don't need to parse the formbuilder JSON at server side, just dump it in your database if you want. But if your database is storing the forms in completely normalize form, then server side IDs will work.

@larryweya I like the idea of using getNextFieldId and finding the nextId. I am looking forward to use that as well. But can't I just do the same thing with cids instead for f+id or model.id? OR in another words can I do same thing with cids ?

@larryweya I like the idea of using getNextFieldId and finding the nextId. I am looking forward to use that as well. But can't I just do the same thing with cids instead for f+id or model.id? OR in another words can I do same thing with cids ?

@SongTanmay I think the issue with that approach is that when using bootstrap data, when Backbone creates models, it assigns incrementing cids and if it comes across your bootstraped model with a cid it already assigned, your model will not be created, it will assume its a duplicate. You shoudl be able to use the model.id as long as its not prefixed with a c

Hi, @larryweya @SongTanmay
i have solve this issue buy providing UUID for each element once i load data from DB to keep it as ID for the field , The form Builder  on save event  will look if element has CID if not will create new CID in order inside the DOM.
Please verify.

success: function(data) {
                    var datum, _i, _len, _ref5;
                    _this.updatingBatch = true;
                    for (_i = 0, _len = data.length; _i < _len; _i++) {
                        datum = data[_i];
                        if ((_ref5 = _this.collection.get(datum.cid)) != null) {
                            _ref5.set({

                                id: datum.id
                            });
                        }
                        _this.collection.trigger('sync');
                    }

@KhaledSMQ
Can you provide lib code ??


Draggable is cropping the image

When I do the onMinimized(), the draggable is cropping the image.

for me Dragable is cropping the video and scaleFactor not working :(

Did anyone manage to find any solution for this?


(update): replace require with import

This PR requires more diligent testing to confirm all the New Relic features work as expected, especially Codestream & code-level metrics.


Docker

Hi mtobeiyf,
I updated docker file and readme.
It reduces the size of docker image file and processing time.
Best,
Namshik

Thanks, Namshik 👍
Gonna merge your PR.


Fixes #8236 - Initialize bots from JSON data files

Describe your changes :

See #8236 for more details

Type of change :

[x] New feature

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@nahuel please take look. This might be relevant to some of the work you are doing.

@pmbrull and @ShaileshParmar11, are the pytest and cypress failures because of this change?

Cypress didn't run here as a server did not start!


[SIL-opaque] Don't override arg value category.

Previously, when emitting block arguments, the value category of the SILType was overridden to be address for indirect arguments.  With opaque types, that distinction is made later during AddressLowering.  So only do that when opaque types are disabled.

@swift-ci please test


Impossible to debug app prod logs on e2e failure

Describe the bug
We on CI the app fails to start with some error, you cannot see app logs to fix it, cause it's currently suppressed using > prod.log 2>&1 in startup.ci.sh (e2e.Dockerfile script)
To Reproduce
Steps to reproduce the behavior:

Write smth that will crash your app on CI e2e
Observe "wait-for-it. sh: timeout occurred after waiting 15 seconds for localhost:7777"

Expected behavior
There is the way to see prod logs on e2e failure to fix the issue.
Screenshots
If applicable, add screenshots to help explain your problem.

Potential solutions

Remove > null 2>&1 at all.
Save to a log file and upload the Github artifact with logs.

@Shchepotin #1369 I've created a PR for 2nd potential solution, but I'm unsure why here is suppressing needed, so maybe 1st potential solution can also work


Correct travis badge to show master build

Changes the travis build status badge to point to the master develop builds rather than any branch.
DCO 1.1 Signed-off-by: Ben Griffith<EMAIL_ADDRESS>

:+1:


[webview_flutter] Better documentation of Android Platform Views modes

Fixes flutter/flutter#85215
Documentation improvement - documents the difference between the two modes more clearly, including the SDK requirement for the Virtual displays mode (which was previously missing, as described in flutter/flutter#85215)
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

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

@ludwiktrammer Sure. As to the changelog file: should I put literal "NEXT" or the next number ("2.0.12")?

@ludwiktrammer Sure. As to the changelog file: should I put literal "NEXT" or the next number ("2.0.12")?

You can put the literal "NEXT" as it is not necessary to release a new version just for the documentation change. So we will include it with the next version. The "NEXT" literal is a reminder for the next change to include this in the release notes.

@mvanbeusekom Ok, thanks. Then it's done :)

You can put the literal "NEXT" as it is not necessary to release a new version just for the documentation change.

For future reference, this is not our policy. See https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#faq

Do I need to update the version if I'm just changing the README? Yes. Most people read the README on pub.dev, not GitHub, so a README change is not very useful until it's published.

In this case it's not a big deal since we expect to release webview_flutter again soon and can get it then, but README fixes shouldn't rely on that any more than code fixes.

Apologies, my bad. Thank you for pointing that out @stuartmorgan.


[MISC] Update LICENSE file with correct copyright info

Issue
Substitute template in LICENSE file to accurately reflect Canonical's copyright.

@lucasgameiroborges please create PR for all Data Team repos to update License file. See https://warthogs.atlassian.net/browse/DPE-4423


[CERT-TEST-FAILURE] - TC-WASHERCTRL-2.1 - Test failed

Feature Area
Area:Dishwashers
Test Case
TC-WASHERCTRL-2.1
Summary
All-clusters-app has to be enabled with below mentioned commands :

[ ] spin-speeds
[ ] spin-speed-current

Bug prevalence
Everytime
GitHub hash of the SDK that was being used
d55f68526864b2986404d1da9c2ccd8b64735b6e - master
Platform
raspi
Platform Version(s)
RPI4 8GB RAM
Anything else?
No response

https://github.com/CHIP-Specifications/chip-test-plans/issues/2987

Issue resolved in 55327986b373b50a5aaa030deeaf05d5eef3317e


rabin error

dat-node 1.2.2 with both node v6.2.2 and node v7.4.0. Install seems successful, but upon running:
/root/dat-express/node_modules/bindings/bindings.js:91
  throw err
  ^

Error: Could not locate the bindings file. Tried:
 → /root/dat-express/node_modules/rabin/build/rabin.node
 → /root/dat-express/node_modules/rabin/build/Debug/rabin.node
 → /root/dat-express/node_modules/rabin/build/Release/rabin.node
 → /root/dat-express/node_modules/rabin/out/Debug/rabin.node
 → /root/dat-express/node_modules/rabin/Debug/rabin.node
 → /root/dat-express/node_modules/rabin/out/Release/rabin.node
 → /root/dat-express/node_modules/rabin/Release/rabin.node
 → /root/dat-express/node_modules/rabin/build/default/rabin.node
 → /root/dat-express/node_modules/rabin/compiled/7.4.0/linux/x64/rabin.node
    at bindings (/root/dat-express/node_modules/bindings/bindings.js:88:9)
    at Object.<anonymous> (/root/dat-express/node_modules/rabin/index.js:3:32)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/root/dat-express/node_modules/hyperdrive/archive.js:6:53)

Ended up working with 6.4.

Hmm that is weird about 6.2 vs 6.4. We have been wrestling with bindings lately and putting a lot of work into http://github.com/mafintosh/prebuild to make all these issues go away hopefully forever :D


Gaussian Mixture Models

Hello,
Are there any chance to see the implementations of a distribution-based model, like Gaussian Mixture Models, in the clustering group? https://scikit-learn.org/stable/modules/mixture.html#mixture

Thanks @IriscShih for the feature suggestion, but I don't think this is something we're going to implement. However I would welcome any community contribution for this feature.


[lbry.tv] Stop scanning my computer outside of web-cache space. It's illegal in my jurisdiction.

Bug
The scan required to determine this is a violation of numerous laws in my country. It is electronic intrusion and can be penalized by up to 20 years in prison per each verifiable occurrence.

To Reproduce
Steps to reproduce the behavior:

Go to any page at lbry.tv
Look at the bottom of your browser window.

Assumes you have ever had the desktop app installed and your config dir structure exists (I think? I don't actually know what the code is looking for, or how it does it. I just know it's snooping in my filesystem in places I did not give it permission to look.)

Disabling the browser function which allows this scan breaks the entire site and no content can be displayed. You are literally requiring that I permit you to snoop around my filesystem if I want to view content via your service.
Expected behavior
I expect LBRY to respect my privacy and personal electronic space. Just because modern web technology and default browser settings make this possible does not mean there is any possible justification for doing so without my explicit invitation (as when using one of those web-based virus-scanning tools.)
I strongly suspect this also happens at odysse.com but, because it hasn't yet reached UI-feature parity with lbry.tv, I have not verified.
I was unable to find any mention of this in the Terms of Service, or Privacy policies, or FAQ.

We don't snoop on your file system.
If you have a desktop app installed and choose to share data with us we make a record of it in our database. Then when you use the same email on one of the web apps, we show that message.

I used_to_have the desktop app installed (removed when I discovered it was -- without my knowledge or consent -- storing every bit content viewed and re-seeding.)
I never volunteered to share data with LBRY. There does not appear to be an option in preferences (lbry.tv; can't check desktop because I removed it) to either opt-in or opt-out of data sharing.

The first prompt when you run the Desktop app asks for permission to share data.
Lbry.tv does not have these options since you already are sharing by signing into the website.


chore(deps): update terraform terraform-ibm-modules/secrets-manager/ibm to v1.13.1

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/secrets-manager/ibm (source)
module
minor
1.12.4 -> 1.13.1

Release Notes

terraform-ibm-modules/terraform-ibm-secrets-manager (terraform-ibm-modules/secrets-manager/ibm)
v1.13.1
Compare Source
Bug Fixes

updated DA variable description (#124) (53f6cdd)

v1.13.0
Compare Source
Features

updated DA variable allowed_network to default to 'private-only' added new DA input variable existing_secrets_endpoint_type (#123) (a6b1c9e)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.3.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


What happens when there are multiple cookies of the same name? (universal-cookie)

Say there are two cookies stored for a domain.

Name "cookie" with value "parent" on domain parent.com
Name "cookie" with value "child" on domain child.parent.com

When using cookies.get("cookie") on child.parent.com, which value does the function return? Is there a default option, or is it selected based on the current domain?

The browser will use the child value first and if you're anywhere child.parent.com else the cookie value would be parent.


Yes, that would be great. Thanks a lot for volunteering...

Sure thing! List is created<EMAIL_ADDRESS>I demoted myself from owner to manager (feel free to demote me further!) and set the owners to the two email addresses used in your commits for the repo :-)


Change the default for xcodeproj.generation_mode to incremental

Fixtures still use legacy mode. I still have to determine what we will do in that space going forward.


Tomlinton/3pool multideposit

If you made a contract change, make sure to complete the checklist below before merging it in master.
Refer to our documentation for more details about contract security best practices.
Contract change checklist:

[ ] Code reviewed by 2 reviewers
[ ] Unit tests pass
[ ] Slither tests pass with no warning
[ ] Echidna tests pass (not automated, run manually on local)

We should probably split this into two PR's - one for the pricing change, and one for the multi depositing.

Good plan, will close this one in the interim and reopen when the pricing change is merged.


Update vpc-private.yaml

The comments of second paragraph were the same as it in vpc-privatepublic.yaml which is not correct.
*Issue #, if available: The comments
"####   - 2 public subnets for the load balancer that routes traffic to the instances
- An Internet gateway, NAT gateway, and route table to allow the instances
to communicate with the load balancer and the Internet."
does not comply with the vpc-private.yaml
*Description of changes: Delete the three lines from the file
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for catching this, @guanevan!  Merging this change.


Terminado hangs on stdin messages over the PTY buffer size for the server OS

We are dealing with an issue where large copy/pastes into Terminado (through open terminals created in the Jupyter UI) cause the Jupyter server to hang.
We diagnosed this issue and it seems to be one where Terminado is using PtyProcess.write on the entirety of stdin message, which for copy/pasted text is the entirety of the pasted text. PtyProcess is using blocking writes to a PTY file descriptor. When the size of the write exceeds the max PTY buffer size (which I believe is set at the OS level) the write blocks indefinitely. Because this is happening inside of the Tornado event loop, it hangs the server.
It's not clear to me if this should be considered a bug in Terminado or PtyProcess. I don't see PtyProcess offering a non-blocking write mechanism, so I'm not sure what the best immediate solution is. It seems like PtyProcess could offer non-blocking writes which could be leveraged by Terminado, or Terminado could split input into some reasonably small chunks before passing them through.

I suppose one other thing you could do is split up the data in stdin messages into multiple messages if it's over a certain size. This would likely prevent other issues like this associated with extremely large websocket messages, but would not address the underlying server issue, which could still be triggered by specially crafted connections by someone wishing to perform a denial-of-service on a Jupyter server they have access to, but this is likely not a concern since the attacker already has arbitrary code execution privileges on the server host through Terminado itself.

I've continued investigating this a bit and actually it seems like the problem, while hit when the PTY buffer size is surpassed is not caused just Terminado only using a single message, but I created a branch that splits the writes (on the server side) and still run into this issue.
This seems to indicate that the subprocess spawned by PtyProcess isn't reading and clearing the PTY buffer being written too.

Thanks for digging in to this @KoopaKing!  I'm happy to review any PRs you open here or in PtyProcess.  I don't have bandwidth to dive into the problem myself at the moment.

Sounds good, I actually just found https://github.com/jupyterlab/jupyterlab/issues/12952 which seems to be the same issue.
Currently testing a frontend fix in terminado.js by chunking the messages.

So verified that a frontend chunking does not work. It seems like we need to do something to tell the OS to send the information in the buffer. Still researching this a bit but I think it will either

happen automatically if we ensure chunking breaks on newlines, or
happen manually when we send the VEOF special character telling the OS to flush the buffer, although I'm still trying to figure out when that is legal (may need to be in conjunction with point 1)

So I have a working fix for this, after all that it seems to be fixed by switching the stdin write to the PtyProcess to using it's own thread. The working fix is using Tornado coroutines to do so.
It seems like maybe the PTY buffer that was filling up was sometimes the stdin write buffer, sometimes the stdout read buffer, both of which are using the same Tornado single threaded event loop. When sufficiently large blocking writes occur from Terminado to the subprocess and the subprocess is sending information back to Terminado, there are buffers both ways and Terminado is using the same event loop thread to both write and read.
This whole thing seems highly related to #162
Trying to clean up the fix here soon, but I don't normally a Python programmer, so it might be pretty rough.

Sorry about ghosting this for a while, was unexpectedly busy, and figuring out the right testing mechanism evaded me for a bit (again, not a Python programmer normally).
I've created a PR which should fix this issue along with a regression test. Feel free to lmk if there's anything you'd like updated in terms of the approach.


MacOS and VMware Fusion -

Thank you for the module. Seems promising!
Some of the commands do not work on my MacOS + VMware Fusion machine.
Version:
PS /Users/jan/Devel/Powershell> Get-Module vmxtoolkit

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Script     <IP_ADDRESS>               vmxtoolkit                          {Add-VMXScsiDisk, Connect-VMXcdromImage, Connect-VMXNetworkAdapter, Convert-VMXdos2unix…

PS /Users/jan/Devel/Powershell>

Sample issue:
PS /Users/jan/Devel/Powershell> Get-VMX -Path '/Users/jan/Virtual Machines.localized/'
InvalidOperation: /Users/jan/.local/share/powershell/Modules/vmxtoolkit/<IP_ADDRESS>/vmxtoolkit.psm1:2629
Line |
2629 |          (($cmdresult = &$vmrun List) 2>&1 | Out-Null)
     |                          ~~~~~~
     | The expression after '&' in a pipeline element produced an object that was not valid. It must result in a command name, a script block, or a
     | CommandInfo object.

VMXName         UUID                              State     Template Path
-------         ----                              -----     -------- ----
Kali            564dac602d2e9755-67a0cb262eac0f13 stopped            /Users/jan/Virtual Machines.localized/Kali.vmwarevm
Mikrotik        564dd13bcffa4eee-a1f453ab8c24a440 stopped            /Users/jan/Virtual Machines.localized/Mikrotik.vmwarevm
VMware ESXi 6.7 564daa906bdbd986-83483cc8a6f5638d stopped            /Users/jan/Virtual Machines.localized/VMware ESXi 6.7.vmwarevm
Windows 10      564d3b3caa16fbf7-78d37fa7a8cb7506 suspended          /Users/jan/Virtual Machines.localized/Windows 10.vmwarevm

PS /Users/jan/Devel/Powershell>

Where should be the $vmrun varibale set? I cannot find the place.

what version of fusion are you on ? need to validate the latest version (s)
the vmrun path is $GLOBAL:vmrun and will be evaluated with other variables during module initialization from vmxtoolkitinit.ps1:
            $VMware_Path = mdfind -onlyin /Applications "VMware Fusion"
            $Global:vmwarepath = $VMware_Path
            [version]$Fusion_Version = defaults read $VMware_Path/Contents/Info.plist CFBundleShortVersionString
            $VMware_BIN_Path = Join-Path $VMware_Path  '/Contents/Library'
            $preferences_file = "$HOME/Library/Preferences/VMware Fusion/preferences"
            $Global:VMware_vdiskmanager = Join-Path $VMware_BIN_Path 'vmware-vdiskmanager'
            $Global:vmrun = Join-Path $VMware_BIN_Path "vmrun"

can you run the commands with -verbose option ?


[Blog] Add GPU graph processing blog

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


Adding maxlength attribute to Text and Textarea input types.

Hi,
I'd like to add support for the maxlength attribute on Text and Textarea input types.
I know that maxlength on Textarea is an HTML5 attribute, but this will allow us to continue using a plugin to add maxlength support on browsers that don't support it (IE)
Thanks!

What is the status on this? I think it would be a great feature.


Add sw and ci for gemmx on narrow

In this PR, we add the software test for the GemmX accelerator(hooked up on the narrow ports, with normal data layout NHWC).

Ok!
Minor comment but maybe in the next PRs or so, but it looks like several are like "repeated" makefiles, codes and stuff, might be more elegant if we try to clean it soon. Before the number of items to test blows up!
Otherwise go!
How much more tests do you need to do?

Yes! I will remove the repeated Python golden model code in the next PR!
I plan to have one more pooling layer and one precision-scalable test.


Component instances

@LiamRiddell You told me that there was a reason for component classes and I did not listen.
Now I have realized that some components need the child count. Therefore I had to rewrite it to classes.
For component writers it is much easier now because you do not need Props anymore.
The performance has not changed.

Ahh yes, it's coming back to me now! The child count will be useful for the section component right now. Let's get this merged as soon as possible. It looks good! 👍

It is also needed for the group. I thought I could continue with that and then realized the problem.


Update docs about python docstrings

Describe what needs to be documented.
Following #3324, the sphinx docs need to be updated to reflect that changes should be made in the .rst files and not the MethodDoc.C files.

@rusu24edward making changes in this area, I'll go ahead and tackle can perhaps you can review?

hmm, well i just handled the functions for now. is the cli extractor still used for the atts and events?

Yes, I believe the extractor is still used for atts and events. I can definitely review

resolved in #4132


Remove version constraint on provider declaration

The provided example for terraform 0.13 include a constraint on version 0.3.0 which is very old (eg. no network data source). Removing this constraint allow the always use latest features.

Hey @olivier-monaco, apologies for the late reply! Thanks for this fix and I plan to be more responsive on provider support / development in the coming months!


Linear Perturbations update, Quality of Life and bug fixes

This update is divided in 2 sections:

The grid extension and the mcmc updates are related to what I was trying to do on the other project. I've seen that you were working on the planet rotation, so some of this may be redundant or even conflicting. I've left them separate so that you can easily choose not to add them to the master version

The Linear Perturbations update contains all the improvements I have performed on the reading and interpolation aspects of the linear perturbations, together with other improvements/bug fixes on the initial condition extraction and the burgers solution. I've seen that you changed something related to the interpolations, so we should be careful with this part

Here is a summary of the main changes:

Linear perturbations reading

Added option to choose which linear perturbation to use
Added dictionary to keep track of the linear perturbation type and grid (to be generalised)
If Cartesian grid, interpolate on annulus
Simplified interpolation notation
If Cylindrical grid, mask perturbations and grid

Interpolation

Simplified interpolation syntax
No cutting needed
Interpolation on an annulus segment with customisable dimensions

Major changes to initial condition extraction on annulus method

Using proper Chi transformation for the initial profile
Added planet mass dependence to t coordinate
Using vector Eta transform to speed up the process
Sorting Eta (and Chi) array to solve discontinuity due to 2\pi modulus
Generalised eta_tilde definition (and corrected in inner disc) to get rid of obnoxious signs
Simplified physical rescaling
Added a parameter limiting numerical solution to outer/"inner" edge of the disc (inner edge to be defined, currently r_p/5)

Transformations

Added non linear correction to the wake (currently set to 0 as it does not work)
Updated arguments of every function calling the wake to account for non liner correction (useful for the future only)
Added planet mass dependence in t coordinate
Corrected N wave definition, consistent with general definition of eta_tilde

Burgers Equation

Simplified physical rescaling (done once in non_linear_perts.py)
Eliminated eta extension of profile, initial profile taken on annulus now
Implemented cyclic boundary conditions
Changed limit for numerical computation, equal to the edge of the disc
Implemented automatic break condition when reaching asymptotic limit (wip, to be tested)
Added some nice debug plots

Hi @DanieleFasano, I have started having a look at these updates and you have done an awesome job here. Apologies again for taking so long. Since the updates are very extensive, I am hesitant to merge straight onto the master branch, since I think it is unlikely that after resolving conflicts there will not be any bugs. Based on this I propose the following:
Currently wakeflow is split across the master and developer branches. I will reconcile these two branches first such that the master branch is up to date with the developer branch. I will do this before managing your pull request, since these updates are smaller. This should not be too bad, although I do think I have one or two bugs on the dev branch to handle.
After this, I will get you to re-make you pull request onto the developer branch instead of the master. Both of these branches will be identical by then, and so this just allows me to do some testing on the developer branch after handling the code conflicts. Once we are both happy that wakeflow with all the updates is stable enough, I'll merge the developer onto the master branch once again, and make a new major release.
Let me know if you are happy with this plan!

Hi @DanieleFasano, I have finished reconciling the master branch with my most recent changes. I have also made a minor release  as a stepping-stone to the overhaul you have done here.
Additionally, I have create a new branch feb2023-overhaul specifically for merging this PR that is identical to the master branch. This is so that we don't edit the master branch until we are sure of stability (and I am scared of resolving the conflicts wrong haha).
The next step is for me to review your changes and resolve the conflicts :)
I am hoping to have everything done by July so that we have one (stable) version of wakeflow for when I come to Nice!

Hi @DanieleFasano thanks for addressing the requested changes so quickly! Everything looks good. I am curious though, I think when you've most recently merged your dev branch onto master, you've also pull across some changes that weren't present when I did the first review, for instance discminer_interface.py definitely wasn't present.
I'm happy to include the discminer_interface.py on the master branch and PyPI distribution though. I am wondering what the difference with wakefit_interface.py is in practice? They seem to be identical apart from the class being renamed. I would prefer not to have two copies of the same code.

Thanks @DanieleFasano I have now merged the PR. The conflicts in grid.py were very confusing and will probably require testing to reconcile, so I have kept both lots of code and commented the old (ie. my versions) out and labelled it. It is basically the code I added to rotate the planet location, but you have also changed the way the linear solution is read in. I'll work on reconciling these directly on the branch and once everything is good I'll merge onto the master branch.


Use link to site

A link is more useful than a code block for the URL of the website, then readers of the readme can click on it.

Verify it works: https://github.com/tcharding/rust-bitcoin.github.io/tree/04-03-use-link?tab=readme-ov-file

utACK abc5575


fix: Upgrade to babel 7

Codecov Report

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

@@           Coverage Diff           @@
##           master     #199   +/-   ##
=======================================
  Coverage   48.08%   48.08%
=======================================
  Files          21       21
  Lines         287      287
  Branches       57       57
=======================================
  Hits          138      138
  Misses        112      112
  Partials       37       37

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 cb90805...a15ccd4. Read the comment docs.

Coverage remained the same at 41.648% when pulling a15ccd4de5046c2500ae1baaac4b18e36cb980c1 on babel-7 into cb90805a93af182a669736c1535961b6766312b8 on master.


Add Tilt gen-dashboard-client local resource

Watching the internal/api/gen/http/openapi.json file it auto-generates
the dashboard TS client.

I kept the Makefile implementation in case it need to be executed manually without setting up Tilt.


UI: consortia and pipeline view bugs

Expected Behavior
Consortia users are not a part of should not be 'viewable' only joinable if they are not private.
Likewise, on the pipelines page, users should not see pipelines that are not part of their consortium at all unless they are shared pipelines.
Current Behavior
You can view the details of pipelines and consortia that are not your own.
Possible Solution

Remove view details from consortia list, make sure to pair this w/ api checks
Remove pipelines from list page, again use a filtering api call

the Consortia list page, I revised the comment because it was sort of nonsense


Update to use new Gitbook

This should be released next week.
References

Documentation
Redux website
Redux PR

Closing as revisit (waiting on GitBook...)


footer position

when there are lot of contexts(that need scolling), footer position might be a bug. It may cover/collide with other text

problem resolved.


Change discordapp.com links to discord.com

Summary
As per the announcement in the Discord Developers server; discord.com will be used in place of discordapp.com, with enforcement starting November 7th 2020.

I believe these are all the references to the discordapp domain, and brief testing shows everything to be functional.

This was merged via shardlab/discordrb@5f4fa9d0cce20addf1cab6fb13a59d2a743094a5 in the new repo and released in v3.4.0 and can be closed.


Chore/sp capture backups from follower web 11804

Second PR for using our follower DB for PGBackups.
This should handle the PgBackupsJob while the other (https://github.com/hightower/hightower/pull/10451) was for  one-off and deploy captures made from the CLI.
https://hightower.atlassian.net/browse/WEB-11804

@cspeper what's the thinking behind trying to parse out the follower from a pg:info versus passing in the database name as configuration?

Spoke about these changes with Niall IRL

@cspeper Can we get this new thing documented in the readme since this is an open source gem?

@BrentWheeldon I assumed you meant it should be documented if the follower db were another configuration.  Since it isn't configurable and shouldn't have any implications on the end result for the users of this gem, I wasn't sure what to document.    What do you think we should add to the Readme?

@cspeper At least make a note that it will be backing up the first (or an arbitrary?) follower, not the master.

@BrentWheeldon Done.  I figured it was safe to commit directly to master 😁 .  Also not worth republishing the gem I'm guessing.

@cspeper Nice, WFM.

Thanks Scott
On Thu, Nov 3, 2016 at 5:50 PM Brent Wheeldon<EMAIL_ADDRESS>wrote:

@cspeper https://github.com/cspeper Nice, WFM.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hightower/heroku_pg_backups_archive/pull/1#issuecomment-258284809,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbJl7QcBAZq39jUmMUtyuxqmd7Z3oM4ks5q6lchgaJpZM4Knshb
.

--
Niall Smart
CTO & Co-Founder
(206) 291-7578
https://www.gethightower.com/
176 Grand Street, 3rd Floor - New York, NY 10013
Product Vision http://htwr.co/sigvideo | Hightower Blog
http://htwr.co/1NpDuZH
https://www.linkedin.com/company/hightower-inc-
https://twitter.com/hightower https://www.facebook.com/gethightower


Switch metric value to int4

closes https://github.com/abap-observability-tools/abap-metrics-provider/issues/187

⚠ Datatype Int8 not available in 7.40. But Int4 should not be a problem as a metric type


Czech translation

Please, add czech translation.
{
"Azimuth": "Azimut",
"Dawn": "Svítání",
"Dusk": "Soumrak",
"Elevation": "Nadmořská výška",
"Noon": "Pravé poledne",
"Sunrise": "Východ",
"Sunset": "Západ"
}
Thanks

I am also waiting for the Czech translation. I checked last week and it was not added to the code.

It's merged on develop, will be available in next release


Re-enable app notifications

Expected Behavior
Re-enabled for alfajores network
Current Behavior
Disabled due to bug in app

For this we need to

Update the lastBlockNotified value in the alfajores firebase database. Set it to a recent block number (you can find this in blockscout)
Renable to notification polling and then deploy the service


Allow custom GitHub messages

This fixes #49.
Happy to hear and open to any feedback!
[Closes #49]

Looks like #51 introduced some conflicts 😕

Rebased and made some updates.  Let me know your thoughts.  👍

Hi @iamvery, after talking with some people smarter than me, I got introduced to the idea of allowing github_comment_template to be given a proc.
After some consideration, this seems like an awesome idea, because it opens up many more doors for users to customize the GitHub comment.  For example, it would be extremely useful if my colleagues and I were able to have lita-reviewme post a comment that includes a hyperlink to a Jira story if the Jira issue id is present in the PR title.
With the changes in https://github.com/iamvery/lita-reviewme/pull/50/commits/61b556dc2818101b6623a1791bc2e7454a8a5b28 users could now do something like this:
config.github_comment_template = lambda do |reviewer, pull_request|
  pr_title    = pull_request.title
  jira_ticket = str.scan(JIRA_TICKET_RGX).first
  return unless jira_ticket

  ":eyes: @#{reviewer} #{JIRA_BASE_URL}/#{jira_ticket}"
end

That just happens to be our use case, by allowing a proc, other users could come up with whatever they want... making lita-reviewme much more extensible. ❤
Are you open to this? If so, I will update the README.

This sounds great! I am spread a little thin right now. I'll try to get back to review this 🔜 . Thanks for making this better! ✨

Very cool! Updated README and ready to go! 👍

Feel free to send fixes for the small things if you have a chance 💫

Thanks for merging! 🙌  Will send the small update soon. 🎉

Couldn't figure out how to make the tests pass using lita's api for setting a default value. Mocking/stubbing seems to be rather difficult. 😢

@binarymason what do you think about #54?


KonfettiView class not found

Hi there.
I have xml-based project. After adding dependency
implementation 'nl.dionsegijn:konfetti-xml:2.0.4'
It's complaining:

Class referenced in the layout file, nl.dionsegijn.konfetti.KonfettiView, was not found in the project or the libraries.

Oh I got it.It's now named nl.dionsegijn.konfetti.xml.KonfettiView


Upgrade policy evaluation script to support Domestic Scheduled payment intent

Description

Add the implementation to support Domestic Scheduled payment intent here
Validate the request object with IDM object: domesticScheduledPaymentIntent, see initiation match method

Acceptance criteria

[ ] The policy evaluation script is deployed on am with no errors
[ ] The current functional tests pass ( to check nothin is broken with that changes)

Hey team! Please add your planning poker estimate with ZenHub @andra-racovita @mariantiris


None of the models are showing up after pnpm run dev

Describe the bug
None of the models are showing up after pnpm run dev
Link to the Bolt URL that caused the error
localhost
Steps to reproduce
pnpm install
pnpm run dev
Expected behavior
it should run with all the models available showing up
Screen Recording / Screenshot
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Thanks for detailed report, I will take a look at showing errors if this part fails and we will see then, does work for me on Mac


Feature: Save client height

As a user, I want to save a client's height, so that I can use it together with their weight to calculate BMI.

Resolved in #174


Reddit failed with error 403

Hello everybody,
has someone the same issue? It worked well before, but I guess reddit changed something. I did not find an issue for it. Is there a solution for it?
HttpException: 403 Forbidden in bridges/RedditBridge.php line 80

index.php(65): RssBridge->main()
lib/RssBridge.php(57): DisplayAction->execute()
actions/DisplayAction.php(72): DisplayAction->createResponse()
actions/DisplayAction.php(110): RedditBridge->collectData()
bridges/RedditBridge.php(80)

Query string: action=display&bridge=RedditBridge&context=single&r=Damnthatsinteresting&f=&score=30000&d=hot&search=&format=Atom
Version: 2023-09-24 (git.master.38e9c39)
Os: Linux
PHP version: 8.1.20

Duplicate of #3837, left a comment with some more details there.
But there really isn't much RSS-Bridge can do, Reddit added a bunch of IPs to their blocklists.


e_Concise description of proposed changes_rewind: Fix: Remove debug module completely

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #936   👈

This stack of pull requests is managed by Graphite.


RabbitMQ Compatibility

Hi @mostafa
I don't know if this is the channel to ask that but here I go:
Do you know, by any chance, if there is a way to use xk6/kafka to test RabbitMQ aswell as Kafka itself? Or, if there is any other k6 extension to do so?
Thanks in advance.
Regards!

Hey @ferrange,
RabbitMQ uses AMQP and MQTT protocols, which is not what this extension is built for. For AMQP, use xk6-amqp and for MQTT, use xk6-mqtt.


Add more android applications to help beginners understand app development

Add new helpful and informative apps which will be resourceful to the beginners of android development
Follow this template for Raising an issue
Title of an issue
Give related title to an issue or changes you will be fixing in the repository
Description
Write short description about your issue

Hello Aditya, I have added a Notes App and created a pull request. Please merge it.
Thank you in advance!


Add support universal translator

Issue, Question or Enhancement:
Enhancement
Please add support for your other project https://github.com/go-playground/universal-translator in order to translate errors

Hey @albertoforcato
I'm not sure translations would be the best fit for this package as a lot of what errors does or how the caller would use is based on dynamic input.
Did you have an idea of how in mind?

Closing due to inactivity


Include @jupyterlab/builder in this package

We should bring @jupyterlab/builder within this package and make it generic.
For now the code lives there: https://github.com/jupyterlab/jupyterlab/tree/main/builder

After the move, should the package still be named @jupyterlab/builder, or @jupyter/builder?
Since the Python package seems to be named jupyter_builder at the moment:
https://github.com/jupyterlab/jupyterlab-builder/blob/c04e4a2523df1ccbfc2d4bf9edb34cdd2bd25fb3/pyproject.toml#L9
Looks like @jupyterlab/builder may be preferable to avoid changing extensions that depend on it. Although there might be some ways to handle renames of npm packages without too much disruption.

@cronan03 to test this, you need to

Bring the Typescript files and the package.json
Build the Javascript assets; normally it should create a lib folder when you execute jlpm run build
You should check that the code is calling those Javascript assets in that Python function:

https://github.com/jupyterlab/jupyterlab-builder/blob/c04e4a2523df1ccbfc2d4bf9edb34cdd2bd25fb3/jupyter_builder/federated_extensions.py#L215

That implies defining properly the core_path to get the correct builder variable (it should point to the lib/build-labextension.js file)

@fcollonval
I’m interested in working on this and have already made some progress. I’d like to share what I’ve done so far and also have a few questions. I sent an email but haven’t received a response yet.

Hey @fcollonval,
I’ve gone through the issue as thoroughly as I could and have a basic understanding of it. However, I still have a few questions that I believe are crucial for me to move forward. I transitioned builder from @jupyterlab/builder to jupyter-builder.
Could you kindly clarify the following:

Why are we checking the compatibility of versions of @jupyterlab/builder of core and as of dependency? The required builder version is already specified in the extension’s package.json, so it will be downloaded as a dependency. How does the core’s builder version play into this?

Am I approaching this correctly by running build-labextension.js from the core jupyter-builder? I now think that we need to add jupyter-builder/builder as a dependency in the extension, install it in node_modules, and then run node_modules/jupyter-builder/builder/lib/build-labextension.js to build the extension. In this case, the --core-path would point to the actual core jupyter-builder folder, where we check for version compatibility between the core and the dependency (though I still don’t fully understand why this is necessary).

I’d really appreciate your insights on this. It will help me overcome these roadblocks and push the work forward.

@fcollonval
Sketched a simple chart of my understanding!
Am I on right track ?

Hey @Darshan808
Thanks for pushing on this.
The @jupyterlab/builder code contains the webpack configuration; see https://github.com/jupyterlab/jupyterlab/blob/main/builder/src/webpack.config.base.ts
This is the reason we are checking the core and an extension have matching version to ensure webpack config compatibility.
Regarding the interaction between the python code and the js code. As you found, it is python that executes the js code. You can see that more clearly looking at the extension building script:
https://github.com/jupyterlab/extension-template/blob/5696924f5b7adf07cf07ea9d3510c66914fbb770/template/package.json.jinja#L35
So about what will happen for extensions is the following:
The build dependency at
https://github.com/jupyterlab/extension-template/blob/5696924f5b7adf07cf07ea9d3510c66914fbb770/template/pyproject.toml.jinja#L2
should change to replace juputerlab by jupyter-builder.
Explicitly defining, @jupyterlab/builder in the package.json should not be needed.
But then the challenge is to get the core configuration, in particular we need to know which packages are provided by the core and should not be bundled within the extension. That information lives there
https://github.com/jupyterlab/jupyterlab/blob/84572d6cea7eb73f2cfb2a0cc0cec4387b3462be/dev_mode/package.json#L285
Unfortunately it is not published.
So I would say, the first step is to get the JavaScript code in. And ensure the python code is using the build-labextension file from jupyter-builder and not Jupyterlab.
Then we will test how to modify extension build settings to use jupyter-builder.
Finally, we will address the issue of getting the core packages list without requiring to install JupyterLab.

Do those comments make sense?

@fcollonval
Progress Update
Here is the progress I've made so far:

Moved the builder folder to jupyter-builder.
Updated the package.json in the extension template to include @jupyter-builder/builder as a development dependency.
Modified the template's scripts to use jupyter-builder on running jlpm run build, which calls build-labextension.js from the @jupyter-builder/builder package in the extension's node modules.

I believe the next step would be to remove jupyterlab as a build dependency and replace it with jupyter-builder in template's pyproject.toml. Could I get your thoughts on this approach?

Also
I have updated the extension's pyproject.toml to include jupyter-builder as a build dependency. And, I modified all the relevant commands to use jupyter-builder in place of jupyterlab/builder.

@fcollonval
I would appreciate it if you could review my message whenever you get a chance.

How should we proceed from here?

@Darshan808 would you mind opening a PR with your changes bringing in the builder TypeScript code?
Will see about the next step when the PR is merged. But yes it will mean changing some configuration in other repository; focusing first on JupyterLab core.


The other way around: Trigger/Send towards HA

What?
Use NHC2-connected buttons and link them to HA actions.
Use Case
Button Press on NHC2-button, would toggle Shelly Plug S through HA.
🙇 thank you 🙇

I've found it: this can be done using a virtual device and linking that to a button. HA can read it out (thanks to this project) and can be used in automation.


A tool to run DFA from the command line on a manifest file.

Thanks for the review, @csilvestrini. Addressed your comments. PTAL.