[SMALLFIX] remove unused method parameter

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7985/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7986/
Test PASSed.

LGTM. Thanks for the cleanup.


Improve error messages for validate

Improve error msgs from validate when file has bad name in trestle directory or is loaded from outside the directory.

currently if you ask it to validate an arbitrary file not in a trestle project, it says "Trestle project model not found at .  This message could be clarified.  It also says this if the file is valid e.g. catalog.json
If you ask it to validate a file not in the correct directory  such as catalogs\foo.json it will try to load it and complain if it is a bogus file.  But it shouldn't even try to load it.
If the file at catalogs/catalog.json is valid - it will validate it properly.  But it isn't in a valid location.
If you put a catalog.json in profiles it will complain that it is not a profile.  But it shouldn't load it in the first place.
Validate needs to determine file path relative to trestle root first and give warning accordingly before loading the file.  And the "trestle project not found" wording needs change.

Done.


feat: add directives page

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Check the title: https://github.com/nuxt/nuxt/pull/29203
This PR includes:

filter out directive imports in imports.vue page: Nuxt will use import.ts module to auto import and generate the dts
filter directive imports in directives.vue page
updated NSectionBlock.vue to allow add the directives when there are no directives (added expandable prop and the logic for mouse and expand icon)
add directives.md to the docs: will require a review

This PR can be merged, we only need to confirm that Nuxt will use the import.ts module, otherwise we will need to redo some logic here.

Some screenshots

toolbar_

directives page

Can we have it in the imports tab (as a badge or filter) instead? Would that be easier to maintain?

Something like this?

We need to:

make new filter responsive
change User composables to User directives and the icon when directives filter is checked.


Developing a project with large number of services

I am working on a project that has many different services (>50). It is not feasible for a developer to bring up the entire system on their dev machines. Today the system is IIS hosted, but we are starting the process of moving to .Net 5, self hosting, and docker containers.
Today, in order to test out a service, the developers will bring up some of the services on their dev machines, and using changes to the hosts file, will either route calls from the test environment to their machine, or from the dev to the test machine. This is of course very error prone, people step on each other, and makes development complex.
Is there a way to use project Tye (or some other solution), that will allow a developer to run a single service on their machine, and have his/her calls to the system route to dev machine, and all other calls stay on the test system?
Hope this makes sense. Thanks.

I achieved this by using tags, env vars, & config.
See local/cloud tags here: https://github.com/jongio/memealyzer/blob/main/pac/net/tye/tye.yaml
Then in config, I have env var overrides:
https://github.com/jongio/memealyzer/blob/main/src/net/Lib/Config.cs
Locally, I override via .env file in root so I don't have to set/unset env vars all the time
https://github.com/jongio/memealyzer/blob/main/.env
So, a dev could add any tags they want to tye.yaml, and then modify env vars to point the services that aren't started locally with the test endpoints.
Give this a try and then LMK if you want to chat for a few about your specific setup or have questions.
Jon


feat: [sc-23050] Add a description to each of the paths in the openapi spec of NameGuard

Story details: https://app.shortcut.com/ps-web3/story/23050

Most of the descriptions are copy-pasted from packages/sdk/src/index.ts  (with slight modifications).
I didn't add docstrings for /secure-primary-name, /fake-eth-name-check and /fake-eth-name-check-fields endpoints, because there were no descriptions for the corresponding functions in packages/sdk/src/index.ts.
Maybe someone could devise appropriate descriptions for these endpoints/fuctions and add them to sdk and api :))

I hope everything is addressed despite GitHub warning.


Prevent entire navigation stack from rebuilding when MediaQuery dependency changes

Fix:
https://github.com/flutter/flutter/issues/96044
The MediaQuery is a object that its data will change very frequently, especailly when keyboard animating. So I think we can avoid the invisible page from being rebuilt.
Preview
Before
All pages will rebuild.
https://user-images.githubusercontent.com/49340347/204210965-6179faf9-af85-4c1e-b106-a05c74582447.mp4
After:
Only visible pages will rebuild.
https://user-images.githubusercontent.com/49340347/204211137-77c06a0d-bf36-4690-9bbe-68b9371d38ce.mp4
Update when page becomes visible
https://user-images.githubusercontent.com/49340347/208226724-9fb4dd17-c9d0-4a31-91fa-7f6be4a87b2d.mp4
Pre-launch Checklist

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

I am not convinced that special casing MediaQuery this way is worth the effort. Widget rebuilds that this is triggering are usually very fast. Do you have a benchmark indicating that there is a performance problem? With https://github.com/flutter/flutter/issues/102060 in place I also suspect that this already cuts down the rebuilds by a lot because widgets can listen to mediaquery changes more targeted.

I have had the problem of the entire navigation stack being rebuilt for some time now. Of course #102060 helps to avoid some rebuilds, but there are some use-cases to avoid the rebuilds between the pages as well.
For example, Instagram: There is a TabBar with 5 different Pages. With Flutter, if I visit all pages once and then a MediaQuery rebuild is generated, then potentially thousands of widgets will be rebuilt. Especially with more complex apps with a lot of different content, this quickly leads to a performance bottleneck.
Currently I use custom widgets that avoid rebuilding when TickerMode is disabled and this has solved many janks in my app.

I think this change will not impact the multi tab scenario. Use CupertinoTabScaffold as example
I see it use a Stack to control the page show or hide instead of a Navigator or Overlay system.
See code:
https://github.com/flutter/flutter/blob/dbc9306380d8a72273b478b8fcc934a6014f946d/packages/flutter/lib/src/cupertino/tab_scaffold.dart#L493
So if one page in tab opens a textField, the other page will still rebuild. I think that's a different problem.
The PR https://github.com/flutter/flutter/pull/114459 solve problem is:
If just use MediaQuery.viewInsetsOf(context);, don't rebuild unless viewInsets changes. But if I use MediaQuery.viewInsetsOf(context) in every pages, it will still be rebuild when textField opens.
This PR solve is that:
No matter what, if current page (Or overlayEntry) is not visible, the widget in it should not rebuild until it becomes visible

Here is demo code to debug.

Demo code
void main() async {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const TextFieldPage(),
    );
  }
}

class TextFieldPage extends StatefulWidget {
  final int? id;

  const TextFieldPage({Key? key, this.id = 0}) : super(key: key);

  @override
  State<TextFieldPage> createState() => _TextFieldPageState();
}

class _TextFieldPageState extends State<TextFieldPage> {
  @override
  Widget build(BuildContext context) {
    print('${widget.id}page rebuild!');
    MediaQuery.viewInsetsOf(context);
    return GestureDetector(
      onTap: () {
        FocusScope.of(context).unfocus();
      },
      child: Scaffold(
        body: Stack(
          fit: StackFit.expand,
          children: <Widget>[
            Column(
              mainAxisAlignment: MainAxisAlignment.end,
              children: [
                const Spacer(),
                Text('Page ${widget.id}', style: const TextStyle(fontSize: 26)),
                CupertinoButton.filled(
                    child: const Text('push'),
                    onPressed: () {
                      Navigator.of(context).push(CupertinoPageRoute(builder: (ctx) {
                        return TextFieldPage(id: widget.id! + 1);
                      }));
                    }),
                const SizedBox(
                  height: 20,
                ),
                const Spacer(),
              ],
            ),
            Positioned(
              left: 0,
              right: 0,
              bottom: 0,
              height: 100,
              child: Padding(
                padding: const EdgeInsets.only(left: 26, right: 26),
                child: CupertinoTextField(
              decoration:
                  BoxDecoration(color: Colors.red, borderRadius: BorderRadius.circular(8)),
                ),
              ),
            )
          ],
        ),
      ),
    );
  }
}

Oh ok, maybe it's better to create a generic widget that disables the rebuild functionality? Something like TickerMode for rebuilds. If "RebuildMode" is turned off then all widgets in the subtree will not be rebuilt anymore.

@b3nni97
Emm I think maybe the RebuildMode is a very invasive and aggressive.....
That will break most current framework theory... Maybe not very feasible. You can have a try for that O(∩_∩)O

Either we go through the RebuildMode or not at all, It seems like MediaQuery is just one subset of the rebuild issue. I can also see thing may become unexpected when we do that though, if the build or didchangeDepedencies has some side effect, this will be a breaking change.

If irrc, the non visible Overlay will not be layout or paint, I am not sure how much of this change will help with performance

After the input from @goderbauer and @chunhtai, maybe this change is not very appropriate. It will have some side effect or other things unexpected, which is a little bit aggressive.
After some investigation, I think we have some options to optimize the rebuild from MediaQuery changes when writing code:

If we did want to avoid rebuilding when MediaQuery changes. We can use MediaQueryData.fromWindow(window) to get value, and using didChangeMetrics to update value and call setState() when it is current page, which will not break current theory and will works fine too.

Set Scaffold's resizeToAvoidBottomInset to false when you don't need adjust viewInsets, because when resizeToAvoidBottomInset  is true, the Scaffold will use MediaQuery.viewInsetsOf to subscribe the changes and rebuild when it changes. So set it to false when this page doesn't need to adjust viewInsets.

Using Builder to avoid rebuild all things in a page

class TextFieldPage extends StatefulWidget {
  const TextFieldPage({Key? key}) : super(key: key);
  @override
  State<TextFieldPage> createState() => _TextFieldPageState();
}

class _TextFieldPageState extends State<TextFieldPage> {

  @override
  Widget build(BuildContext context) {
    print("page build");
    return Scaffold(
      body: Column(
        mainAxisAlignment: MainAxisAlignment.end,
        children: [
          const Spacer(),
          const Text('Keyboard animation', style: TextStyle(fontSize: 26)),
          const Spacer(),
          Builder(builder: (BuildContext builderCtx) {
            /// If you use context, the whole page will be rebuild, you will see log "page build" and "builder build",
            /// but use builderCtx (after change you need to hot restart or recreate this page)
            /// you will only see log "builder build"
            MediaQuery.viewInsetsOf(builderCtx);
            print('builder build');
            return SizedBox(
                height: 60,
                child: const CupertinoTextField(
                  decoration: BoxDecoration(color: Colors.red),
                ));
          }),
        ],
      ),
    );
  }
}

@chunhtai Would you mind sharing your suggestions on this? since https://github.com/flutter/flutter/pull/116090#issuecomment-1356014347 is the suggestions for developers. It needs developer know more things to write code with higher performance.

I can't think of a systematic way to solve this problem from flutter side. If the developer has to have so many routes on the screen they should set https://api.flutter.dev/flutter/material/MaterialPageRoute/maintainState.html to false.
I am going to close this pr since I don't think this is the direction we want to go


Tests failing on Ubuntu 22.04

I am trying to build and run tests, but I am getting this list of failures. Seems to be some segfaults.
Summary of Failures:

 8/20 base_library / taskqueue-stress                 FAIL            0.08s   exit status 1
 9/20 mini-async-log-c / malc-stress-test-heap        FAIL            0.08s   exit status 1
10/20 mini-async-log-c / malc-stress-test-tls         FAIL            0.08s   exit status 1
11/20 mini-async-log-c / malcpp-smoke                 FAIL            0.09s   exit status 24
12/20 mini-async-log-c / malcpp-stress-test-heap      FAIL            0.08s   exit status 1
13/20 mini-async-log-c / malc-stress-test-queue       FAIL            0.09s   exit status 1
14/20 mini-async-log-c / malc-stress-test-queue-cpu   FAIL            0.09s   exit status 1
15/20 mini-async-log-c / malcpp-stress-test-tls       FAIL            0.10s   exit status 1
16/20 mini-async-log-c / malcpp-stress-test-queue     FAIL            0.09s   exit status 1
18/20 mini-async-log-c / malcpp-stress-test-queue-cpu FAIL            0.10s   exit status 1
19/20 base_library / bl-base                          FAIL            0.16s   exit status 1
20/20 mini-async-log-c / malc-smoke                   TIMEOUT        30.03s   killed by signal 15 SIGTERM

I have attached the test log as well
tests.tar.gz

I believe this line is the cause for the segfault in malc-smoke
https://github.com/RafaGago/base_library/blob/3d08c1ccbecc4ab26495999fc6692978b490e40f/include/bl/base/impl/thread_c11.h#L44
thrd_create takes a thrd_t, not a thrd_t*. The t argument needs to be dereferenced.

I forgot to write yesterday, but thanks for reporting, this is fixed on https://github.com/RafaGago/mini-async-log-c/commit/67fabe26d1f73ff75b0ee9e22dede4242a1c57c5


[TEP-114] Remove the feature flag custom-task-version to stop supporting v1alpha1 Run

Changes
This commit closes #5837. It removes the feature flag custom-task-version to stop supporting v1alpha1.Run as planed in TEP-114. There will be a separate PR to remove the v1alpha1.Run tracking by #6498.
/kind feature
Signed-off-by: Yongxuan Zhang<EMAIL_ADDRESS>
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
action required: Feature flag custom-task-version is removed, tekton pipeline will stop supporting v1alpha1 Run and only use v1beta1 CustomRun.

/test all

/assign @jerop

/retest

do we also need to remove the alpha custom task controller? or would that be in the follow-up PR removing v1alpha1 custom task? https://github.com/tektoncd/pipeline/tree/main/test/custom-task-ctrls/wait-task-alpha

Yes I will remove it in next PR.

/cancel kind feature

/remove kind feature

/kind feature cancel

/remove-kind feature

/lgtm


Auto cleaner of dev subscription.

We need some tool or solution that will clean up the mess our developers and tooling will make in dev subscription. The tool should be deployed to our staging subscription.

Obsolete task. Closing


Template of kafka server.settings to reduce log size

Add template for kafka config file server.settings with following code:
############################# Log Retention Policy #############################

# The following configurations control the disposal of log segments. The policy can
# be set to delete segments after a period of time, or after a given size has accumulated.
# A segment will be deleted whenever *either* of these criteria are met. Deletion always happens
# from the end of the log.

# The minimum age of a log file to be eligible for deletion
log.retention.hours=0
log.retention.minutes=10

# A size-based retention policy for logs. Segments are pruned from the log as long as the remaining
# segments don't drop below log.retention.bytes.
#log.retention.bytes=1073741824

# The maximum size of a log segment file. When this size is reached a new log segment will be created.
log.segment.bytes=1073741824

# The interval at which log segments are checked to see if they can be deleted according
# to the retention policies
log.retention.check.interval.ms=300000

# Allow topic delete
delete.topic.enable=true

working on solution in branch #83-kafka-template

Finished, merged in #86, closing.


Documentation uses old "Bar" class instead of "ProgressBar"

Hi,
The website / module docs use:
from progress.bar import Bar
Where the right import clause is nowadays:
from progress_bar import ProgressBar

I think it is me screwing up something and using different class...


Redirect to login page and network layer

I've tried to use network layer to redirect user to user page if status code is 401, but I dont know how to make redirect from callback:
import {
  RelayNetworkLayer,
  authMiddleware,
} from 'react-relay-network-layer';

Relay.injectNetworkLayer(new RelayNetworkLayer([

  authMiddleware({
    token: () => 'testtoken',
    tokenRefreshPromise(){
      // we are here if status code is 401
      // but how can I redirect user to login page here?
    }
  }),
]));

If I just remove token in tokenRefreshPromise and check it out in onEnter hook, user is able to open route because onEnter fires before tokenRefreshPromise.
What is the right way to redirect user to login page?
Thx a lot!

I've been wanting to migrate to Relay Modern for a while, but have been pretty busy. If we use that, then you'll be able to handle the network layer more manually.
However, given that you are using react-relay-network-layer more than anything else, you should ask it there. Please do not file an issue in multiple places.


Added default value for _squeeze

Fixed: UnboundLocalError: local variable '_squeeze' referenced before assignment

Thanks for the bugfix! I merged it.


add androidstudio beta version 2<IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

For future reference, https://developer.android.com/studio/archive
I recommend going for the full version, not just major.minor, (ie. 2<IP_ADDRESS> instead of 2020.3.1) as that'll cause trouble with new increments.

#19840

Are you in favour of this?

Are you in favour of this?

I am not, besides not being able to 'recognize' the currently installed package in case you haven't updated yet, there is also the matter of wanting to install specific versions due to bugs in newer versions you might want to avoid.
And probably a dozen other reasons I'm missing right now.

You are right. Keeping the latest Stable builds and then beta and canary builds. If you see builds more than a year old are present in this repository. I am talking about removing them.

You are right. Keeping the latest Stable builds and then beta and canary builds. If you see builds more than a year old are present in this repository. I am talking about removing them.

I'm not sure why we would want to remove any versions. They're all meticulously organized in folders, so from a manageability perspective it isn't a problem. What's there to gain from removing older versions, other than 'clean-up'?


Runtime error with ruby 1.9.1: implicit arg passing to super not supported

Getting this calling objectWithSerializedAttributes.attributes:
RuntimeError: implicit argument passing of super from method defined by define_method() is not supported. Specify all arguments explicitly.
vendor/plugins/serialized_attributes/lib/serialized_attributes/schema.rb:59:in `block in initialize'
ruby 1.9.1-p378

This is fixed in #7.


Implement oauth token access

https://help.github.com/articles/git-automation-with-oauth-tokens/

add documentation to use that


Machine/Node IP Copy icon is misaligned and compacted

Setup

Rancher version: 2.7-head (pre 2.7.2)

Describe the bug
Copy button is not correctly vertically aligned and is flush against value

To Reproduce

bring up an RKE2 cluster in DO
View the cluster's Machine Pool tab/table

✅ PASSED
Reproduction Environment

Component
Version / Type

Rancher version
v2.7.1

Installation option
Helm (high availability)

RKE binary version used
v1.4.0

If Helm Chart k8s cluster
v1.24.6

Cert Details
Let's Encrypt / nginx

Docker version
20.10.7, build f0df350

Helm version
v2.16.8-rancher2

Downstream cluster type
not applicable

Downstream K8s version
not applicable

Authentication providers enabled
local

Logged in user role
admin, standard user

Browser type
Google Chrome

Browser version
111.0.5563.110

Reproduction steps

Install Rancher on v2.7.1
From the default Rancher homepage /dashboard/home
Click --> hamburger menu --> local --> Nodes
Observe the UI list for External/Internal IP, specifically the copy button

Additional Info
RESULTS
✅ Expected
For the copy button not to be too close / crowding
❌ Actual
Too close / crowding

Validation Environment

Component
Version / Type

Rancher version
v2.7-7913e283ff90eebd1f8cc48860a6b410d72cd4dd-head

Rancher commit link
https://github.com/rancher/rancher/commit/7913e283ff90eebd1f8cc48860a6b410d72cd4dd

Installation option
Helm (high availability)

RKE binary version used
v1.4.0

If Helm Chart k8s cluster
v1.24.6

Cert Details
Let's Encrypt / nginx

Docker version
20.10.7, build f0df350

Helm version
v2.16.8-rancher2

Downstream cluster type
not applicable

Downstream K8s version
not applicable

Authentication providers enabled
local

Logged in user role
admin, standard user

Browser type
Google Chrome

Browser version
111.0.5563.110

Validation steps

Install Rancher on v2.7.1
From the default Rancher homepage /dashboard/home
Click --> hamburger menu --> local --> Nodes
Observe the UI list for External/Internal IP, specifically the copy button

Additional Info
RESULTS
✅ Expected
margin-left: 8px;
✅ Actual
margin-left: 8px;

📋 QA Test Plan

Scenario
Steps
Expected
Actual & Pass/Fail

Check light mode in addition to dark

Follow validation steps in light mode

Expected: Fix to also be present in light mode
Actual: Fix was present in light mode as wellPass/Fail: ✅PASS


deleting *.tmp WAL files on startup

Resolves https://github.com/prometheus/prometheus/issues/10245

Thanks!
A few initial notes, before our TSDB experts chime in:

I don't think this should be done in the main package
This should be logged
There are linting issues (see the "files changed tab" in this PR).


AWS is not free

AWS does have a free trial, but after a year its paid
https://github.com/dalisoft/awesome-hosting#web-services-platform

@89z Thank you for checking the correct information. Is #7 fix solved the wrong information? Thank you again

Yes, I think that is more accurate. Thank you!

Thank you. You welcome and happy to see your new issues related to information or pull requests if you want. Have a great day


Localization accuracy with different devices for mapping/localization

I would like to ask why localization accuracy (referred to tvec) is bad if I do mapping with a device (e.g., reflex camera Nikon D70s) and localization with another one (OAK D S2). In this case I get a tvec error of about 30 cm if I estimate intrinsics from EXIF with refinement. Even if I fix for the OAK D S2 intrinsics got from another 3D mapping did with this camera, the average tvec error is about 6 cm. What is the possible cause of this error?
When I do mapping and localization with the same device instead and I use for localization intrinsics (f, cx, cy, k) estimated from 3D mapping I get an error of about 3 cm (that is very good!).
Any ideas?
Many thanks to who can help me!

Intrinsics estimated & refined from a single image (localization) are much less accurate than when refined from multiple ones (mapping). Inaccurate intrinsics reduce the accuracy of the estimated poses.
Changes in the imaging parameters (resolution, blur, gamma response) can introduce a small noise in the keypoint detection. 3cm vs 6cm error is a pretty small difference. You could potentially reduce this with our PixSfM https://github.com/cvg/pixel-perfect-sfm


Is there a code example on how to use a custom trace ID using micrometer brave?

Check the usage of propagators here: https://docs.micrometer.io/tracing/reference/api.html and mostly in the Brave docs.
We highly recommend not using custom propagation formats but using highly accepted standards instead (like w3c or b3). Propagation format might be a harder problem than you think.


adding assets in sub directory breaks building

add and reference an image in assets/images and try to build/dev

Notice: Undefined offset: 1 in /Users/salam/Sites/paphper/paphper.github.io/vendor/react/filesystem/src/ChildProcess/Adapter.php on line 228

core wasn't up to date.


[Question] Detect annotation kind if few defined for a methon

Hi Team,
Thanks a lot to amazing product!
Does this possible to understand which action user did if the following solution implemented?
    @NeedsPermission(Manifest.permission.WRITE_EXTERNAL_STORAGE)
    @OnPermissionDenied(Manifest.permission.WRITE_EXTERNAL_STORAGE)
    @OnNeverAskAgain(Manifest.permission.WRITE_EXTERNAL_STORAGE)
    public void writeStorage() {
        // Detect a kind of annotation to handle
    }

Thanks in advance!

@neooleg Hi there now we don't support as such and no plan to do that~. We recommend to separate definitions to make them declarative~.

Got it, thanks for a quick answer


Create Letter_Combination_Phone_java.java

Java Solution of 17. Letter Combinations of a Phone Number implemented by same approach used by neetocode for python solution

Java Solution of 17. Letter Combinations of a Phone Number implemented by same approach used by neetocode for python solution

Duplicate


Add knowledge distillation loss schema

Changes
Add schema and docs
Related tickets
N/A
Tests
N/A

Hi @jpablomch , please take a look. Thanks!
(For consistency, I added a docstring under function _build_compression_loss_function. I can remove it if you think this is not necessary.)


Token Revocation doesn't work

Hi,
I get this stacktrace when I try to implement token revocation
Traceback (most recent call last):
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask_oauthlib/provider/oauth2.py", line 549, in decorated
    return f(*args, **kwargs)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask_oauthlib/provider/oauth2.py", line 524, in decorated
    uri, headers=headers, body=body, http_method=http_method)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/oauthlib/oauth2/rfc6749/endpoints/base.py", line 64, in wrapper
    return f(endpoint, uri, *args, **kwargs)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/oauthlib/oauth2/rfc6749/endpoints/revocation.py", line 76, in create_revocation_response
    request.token_type_hint, request)
  File "/Users/martin/Dev/Bright/api/lib/python3.4/site-packages/flask_oauthlib/provider/oauth2.py", line 951, in revoke_token
    if tok and tok.client_id == request.client.client_id:
AttributeError: 'NoneType' object has no attribute 'client_id'

Does anyone have ever implement this?

Bug found in utils.decode_base64

@mekza could you add an test case for it?

@lepture sure.

I also ran into this. I previously was using Flask-OAuthlib==0.7.0 for this application and just updated it to Flask-OAuthlib==0.9.2, and that's when this started. As noted by https://github.com/lepture/flask-oauthlib/pull/234#issuecomment-133856812 adding &grant_type=password to the request works around it.
My backtrace is pretty much the same, although I'm using python-2.7.

This change in behavior is actually not due to the Flask-OAuthlib exactly, it's due to a change of oauthlib between 0.6.3 to 1.0.3. (Downgrading Flask-OAuthlib didn't fix this, but downgrading oauthlib did fix this for me.)
Of course it's hard to say which lib is really in error, or simply exposed a bug in the other :)

@ploxiln could you open an issue on OAuthlib

:+1: Sorry I didn't find the time to work on it since this summer.

I haven't really looked into what exactly happens between Flask-OAuthlib and OAuthlib, and I'd want to do that and have a bit more evidence of a bug in OAuthlib before opening a bug there. After all, there wasn't a promise of backwards compatibility before 1.0
I'll likely do so in a few days :)

I looked into this more.
My suspicion about "revocation endpoint will now return empty string in response body" was wrong, that happens after where the exception / backtrace starts.
The change in oauthlib that causes this change in behavior is this:
oauthlib 0.7.2:
    def validate_revocation_request(self, request):
        """..."""
        if not request.token:
            raise InvalidRequestError(request=request,
                                      description='Missing token parameter.')

        if not self.request_validator.authenticate_client(request):
            raise InvalidClientError(request=request)
...

oauthlib 1.0.3:
    def validate_revocation_request(self, request):
        """..."""
        if not request.token:
            raise InvalidRequestError(request=request,
                                      description='Missing token parameter.')

        if self.request_validator.client_authentication_required(request):
            if not self.request_validator.authenticate_client(request):
                raise InvalidClientError(request=request)
...

So it's actually the fact that client_authentication_required() in Flask-Oauthlib is now being called, and that always returns False if grant_type is not included in the request, so then authenticate_client() is not called, so it doesn't set request.client.
I'm not sure if this is a good idea, but it looks like another possible fix is changing the line that throws the exception to if tok and tok.client_id == request.client_id: because it looks like (for my cases) client_id does exist in the request object (because it's in the POST body, for my application anyway).

I'm still running into this issue. Has this been fixed in a newer release / has anyone filed a bug report?

This seems to be fixed.

yes, just  pip install --upgrade flask-oauth
this bug seems to be fixed in release 0.9.4.  #290  : )

I am closing the issue then.


Reduce duplication in Haddock.hs

LGTM

@alanz thanks for the review. Just wanted to check with you : what's the process for reviewing / merging of PRs (now that I'm a contributor and have merge rights)? Should I wait for other reviews? Can I merge the PR after I get an approval from other contributors or should I wait for others to merge it?

@jhrcek We do not really have a clear policy.
Basically if at least one other person has said they are ok we can merge. I just held of a bit because the doc stuff had been done by @wz1000  / @bubba so I wanted to give them a chance to comment if they chose to.

A bit late to the party, but it's great to see this type of gardening. I'll make a tag for these types of PRs!

@bubba I'm glad you find it useful. It's my way of getting acquainted with the codebase in my free time. I'll create more PRs like these as I have time.


p2p: Bump chacha20poly1305 dependency

Fixes #875

Codecov Report

Merging #913 (1ebcb00) into master (1b3464e) will decrease coverage by 0.0%.
The diff coverage is n/a.

:exclamation: Current head 1ebcb00 differs from pull request most recent head 3270f39. Consider uploading reports for the commit 3270f39 to get more accurate results

@@           Coverage Diff            @@
##           master    #913     +/-   ##
========================================
- Coverage    70.7%   70.7%   -0.1%
========================================
  Files         203     203
  Lines       16214   16217      +3
========================================
  Hits        11466   11466
- Misses       4748    4751      +3

Impacted Files
Coverage Δ

testgen/src/vote.rs
83.7% <0.0%> (-0.9%)
:arrow_down:

light-client/src/types.rs
30.1% <0.0%> (-0.6%)
:arrow_down:

light-client/src/errors.rs
3.0% <0.0%> (-0.1%)
:arrow_down:

tendermint/src/node.rs
66.1% <0.0%> (+0.2%)
:arrow_up:

tendermint/src/block/commit_sig.rs
79.5% <0.0%> (+2.2%)
:arrow_up:

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 1b3464e...3270f39. Read the comment docs.


ORC-1299: Fix fetch data error in bench module

What changes were proposed in this pull request?
This PR aims to fix bench error when fetch data.
Why are the changes needed?
Fetch https://s3.amazonaws.com/nyc-tlc/ will get a 403 error.
How was this patch tested?
UT

cc @williamhyun

Fixed. @dongjoon-hyun thanks for your suggestion. It does look more concise

Thank you @deshanxiao for this PR!


Allow suppressing logs of management commands of django.contrib.auth by emitting to logger instead of stdout directly.

I've noticed that django.contrib.auth's management.create_permissions is too noisy on stdout without allowing any way to suppress it at runtime.
Loading environment 'development'
...
Running migrations:
  Applying contenttypes.0001_initial... OK
  Applying app.0001_initial... OK
  ...
  Applying app.0002_enable_user_permissions... OK
Adding content type 'app | user'
Adding content type 'app | foo'
Adding content type 'app | bar'
...
Adding permission 'app | user | Can add user'
Adding permission 'app | user | Can change user'
... // More logs of the same format.

I'm not sure if this was the intended behaviour (and please correct me if it was), but I'd imagine emitting to a logging.Logger instead would allow them to be suppressed with a custom logging configuration such as the following:
logging.config.dictConfig({
    ...,
    'loggers': {
        ...,
        'django.contrib.auth': {
            'level': 'ERROR',
            'handlers': ['console']
        }

    }
})

Welcome @aalekhpatel07 :)
PRs normally aren't accepted out of the blue like this unless they're trivial like small documentation updates or small style corrections etc.
You should start a discussion on the Django Internals category of the Django forum to garner feedback from other developers: https://code.djangoproject.com/wiki/DevelopersMailingList
If accepted then a ticket should be opened.
I won't close this as there may be some merit to avoiding print() calls, let's see what the other say. (Having said this another member may close it :))

oh wow! Really appreciate the quick feedback, @shangxiao. I can try to bring it up in the forum and see what others have to say about it.

Closing due to inactivity.


Browser version not supported error with new Rails

Rails added this feature which will block all unsupported browsers but this also is blocking the requests from turbo android apps
The only way I was able to get my android app the load the rails app was by commenting out the following line in the application controller
allow_browser versions: :modern

It's blocking this request but the device is running an older version of Chrome. I'd recommend updating the emulator to a more recent version of Android or modifying allow_browser to better fit your needs.


Error message when using invalid YAML config file

When I incorrectly indent one line in the batect.yml, I get the following error:
batect.yml (line 20, column 3): while parsing a block mapping
 in 'reader', line 1, column 1:
    project_name: international-tran ...
    ^
expected <block end>, but found BlockMappingStart
 in 'reader', line 20, column 3:
      database:
      ^

Caused by: com.fasterxml.jackson.dataformat.yaml.snakeyaml.error.MarkedYAMLException: while parsing a block mapping
 in 'reader', line 1, column 1:
    project_name: international-tran ...
    ^
expected <block end>, but found BlockMappingStart
 in 'reader', line 20, column 3:
      database:
      ^

 at [Source: (sun.nio.ch.ChannelInputStream); line: 20, column: 3]

Yep, that error message could be much better. Thanks for the feedback @ekamara!

This should be fixed with 99c45a7.
Thanks again for the feedback @ekamara.


What elements should be modifying global store objects

We are currently trying to keep all store access limited to pages and then send down these data to components via props or provide/inject calls (see #105). The idea behind this was that having only a few, central places from where we modify the store would make things easier to understand and debug (see also #64).
The downside is that a lot of information has to truck up and down the tree from these pages to the components where the information are actually displayed and where the user interacts with the app.
Now that we have worked with this pattern for a little bit, let's collect some observations on whether this actually makes things more clear as we had expected, or whether we could do without. Without here would mean, any place that needs to can directly talk to the global store, regardless of it's level in the tree hierarchy.

One example where we have recently run up against this is when we wanted to use v-model on a form element that directly affects a variable stored in the global store (#154). The recommended way of implementing this uses direct store calls from a computed property inside the component. So in this particular case, not allowing direct store access makes our live hard, because we need to write more complex logic to have v-model work.

Though cases of mapGetters, mapState, and provide/inject most often functionally equivalent (although we've seen a recent exception where an injected store field cannot be watched by a child component unless its store name is fully spelled out, e.g. $store.state.fieldName), it would be good to revisit how we want to consistently provide fields like this to child components.
For instance, looking at the current provide list in annotation.vue reveals something that may be too much. (It could very well be that after examining its uses by child components that this could be just fine too.)
        provide() {

            return {

                // Getters
                columnDescription: this.columnDescription,
                isMissingValue: this.isMissingValue,
                valueDescription: this.valueDescription,

                // Store fields
                columnToCategoryMap: this.columnToCategoryMap,
                dataDictionary: this.dataDictionary,
                dataTable: this.dataTable,
                missingColumnValues: this.missingColumnValues,
                missingValueLabel: this.missingValueLabel,
                toolGroups: this.toolGroups
            };
        },

Closing this as we have adopted the paradigm of allowing child components direct access to the store's getters, actions, and mutations. (Nobody except mutations and getters have direct access to Vuex state.)


update vcflib submodule

Fixes #8

Thank you so much. But I encountered a new problem.

I use the VCF file generated by sniffles as both base and comparison.
hap_eval -r ~/data/genome/GRCh38/GCA_000001405.15_GRCh38_no_alt_analysis_set.fna -b HG002_sniffles.vcf -c HG002_sniffles.vcf

Prompt the error:
ValueError: ("could not convert string to float: 'None'", 'chrM\t1\tSniffles2.DUP.37S18\tN\t\t59\tPASS\tPRECISE;SVTYPE=DUP;SVLEN=16569;END= 16570;SUPPORT=734;COVERAGE=None,None,896,None,None;STRAND=+-;STDEV_LEN=2.531;STDEV_POS=0.223\tGT:GQ:DR:DV\t./.:0:0:0' )

I use the VCF file generated by cuteSV as both base and comparison.
hap_eval -r ~/data/genome/GRCh38/GCA_000001405.15_GRCh38_no_alt_analysis_set.fna -b HG002_cuteSV_exclude.vcf -c HG002_cuteSV_exclude.vcf
Counter()
precision 0.0 recall 0.0 f1 0.0

The calculation results are all 0

For the first problem, please try the workaround in #10.

For the second problem, could you please share a small subset of the cuteSV VCF (the header and maybe 10 data lines) and I will take a look.


Update CodeEditor to allow JSON objects

Description
This updates the CodeEditor to adapt to the changes of the geostyler-mapbox-parser in v4.
The returned object is now transformed to a string.

Do you introduce a breaking change?

[ ] Yes
[x] No
[ ] I am unsure (no worries, we'll find out)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Impossible to build on Unity 2017.4.10f1 LTS 64bits

Hello,
I want to compile at least one of your example but I get on VS on building before deployment. I got:
Severity	Code	Description	Project	File	Line	Suppression State
Error		The command ""C:\Users\sal\Documents\Unity\HoloLensCameraStream-master\HoloLensVideoCaptureExample\App\Unity\Tools\AssemblyConverter.exe" -platform=uap -lock="C:\Users\sal\Documents\Unity\HoloLensCameraStream-master\HoloLensVideoCaptureExample\App\HoloLensVideoCaptureExample\project.lock.json" -bits=32 -configuration=Release -removeDebuggableAttribute=False -uwpsdk=10.0.17134.0 -path="." -path="C:\Program Files\Unity\Editor\Data\PlaybackEngines\MetroSupport\Players\UAP\dotnet\x86\Release"
followed by a long list of path.
I tried on 2017.4.9f1 and it was the same thing. I work on windows 10 version 1803 and compile with the sdk 10.0.1734.0 and tried on scripting runtime 3.5 or 4.6 and API compatibility 2.0 or 4.6.
Thanks in advance for your help.

Hi @nesifao ,
I am using unity 2017.4.9f1 and my project compiled to me. Probably you have not configured your project well, for that reason, the compilation process fails.
I introduced this plug-in for my project several months ago and it keeps working. I must say that I have used the emulator for running the project because at this moment I do not get HoloLens. I suppose that the project should work too on HoloLens.
Try to to configure again your project following all steps carefully. I hope you can deploy it with success.
Cristian.

Well in my case I got some problems when I was cloning the master branch it worked with unity 5.6 but not with the 2017.4.9 and 2017.4.13. It worked well if I import the package from the last release (3.0).

Try exporting the project as a package from unity 5.6 and importing it on unity 2017.4.9 or 2017.4.13, it might work. That's how I solved my problem with unity 2017.4.14f


add tmux support

When you use tmux, some of the functions not work (for example button clearing).
This is because tmux catches unknown escape sequences. By adding \ePtmux;\e[...]\e\\ you can tell tmux not to catch these, and the touchbar will work properly.

@msbence  Thank you for the PR, good job!


Windows 11 compatibility ?

Electron app with custom-electron-titlebar is not showing the titlebar.
Has anyone tested it in Windows 11 and could confirm works/not ?
Thanks for the module @AlexTorresSk  !!
Desktop (please complete the following information):

OS: Windows 11 Home
Electron version 16.0.4
Node Version 14.17.5

You can indicate how you are implementing the module.
I am currently using Windows 11 with the following versions and it works perfectly.

Windows 11 Pro
Electron 16.0.5
Node 16.9.1

Hi @AlexTorresSk ,
I did a code refactor in my project related with the topbar, and one of them solved this issue. I confirm is showing it fine in Windows 11, I close this.
Thanks


Error when using bash variables as args in fleetctl

I'm seeing an issue where I am unable to submit units to fleetctl where the unit name is a bash variable. As an example:
><EMAIL_ADDRESS>> fleetctl submit $NAME
> Error creating units: invalid character '{' in unit name

Whereas doing this directly works fine:
> fleetctl submit<EMAIL_ADDRESS>> Unit<EMAIL_ADDRESS>inactive
> Unit<EMAIL_ADDRESS>inactive

Would have expected the bash variable to work but strange that it doesn't that way.

$<EMAIL_ADDRESS>$ echo $NAME
<EMAIL_ADDRESS>$ NAME=`echo<EMAIL_ADDRESS>$ echo $NAME
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Gotcha - the {1..2} is just a convenient brace expansion rather than being a part of the argument to fleetctl


[3.0]fix ConfigManagerTest spelling-mistake

fix ConfigManagerTest spelling-mistake

Codecov Report

Merging #8990 (bed3ad8) into 3.0 (23a08d9) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.0    #8990      +/-   ##
============================================
- Coverage     63.31%   63.26%   -0.06%
+ Complexity      313      312       -1
============================================
  Files          1184     1184
  Lines         50295    50295
  Branches       7527     7505      -22
============================================
- Hits          31845    31819      -26
- Misses        14959    14973      +14
- Partials       3491     3503      +12

Impacted Files
Coverage Δ

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/remoting/exchange/support/DefaultFuture.java
88.39% <0.00%> (-5.36%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
58.69% <0.00%> (-4.35%)
:arrow_down:

.../exchange/support/header/HeaderExchangeServer.java
65.09% <0.00%> (-3.78%)
:arrow_down:

...bo/rpc/protocol/dubbo/DecodeableRpcInvocation.java
66.96% <0.00%> (-3.58%)
:arrow_down:

...he/dubbo/remoting/transport/netty/NettyServer.java
70.17% <0.00%> (-3.51%)
:arrow_down:

.../apache/dubbo/remoting/transport/CodecSupport.java
80.00% <0.00%> (-3.08%)
:arrow_down:

.../org/apache/dubbo/qos/legacy/LogTelnetHandler.java
30.30% <0.00%> (-3.04%)
:arrow_down:

.../dubbo/remoting/transport/netty4/NettyChannel.java
59.40% <0.00%> (-2.98%)
:arrow_down:

...pache/dubbo/registry/support/AbstractRegistry.java
76.72% <0.00%> (-2.91%)
:arrow_down:

... and 21 more

Continue to review full report at Codecov.

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


Error encountered resolving symbol values statically. Function calls are not supported.

This error is caused because of version mismatch of packages.
So, can you please add angular/common, angular/compiler, angular/core, angular/platform-browser, core-js, rimraf,  rxjs, reflect-metadata, typescript and zone.js as peerDependencies

+1


[POC] Simple azure sandbox with terraform

Summary
This is a proof of concept that a simple azure sandbox with MySQL, AKS, a VM, and a deployed nginx app can be created with terraform.
Changes

[ ] Client-facing API Changes
[ ] Internal API Changes
[ ] Bug Fixes
[x] New Features
[ ] Performance Improvements
[ ] Code Style
[ ] Refactoring
[ ] Documentation
[ ] Tests

Testing Done

[x] Manually Tested on local docker setup. Please include commands ran, and their output.
[ ] Added new tests for the changes made.
[ ] Updated existing tests to reflect the changes made.
[ ] No tests added or updated. Please explain why. If unsure, please feel free to ask for help.
[ ] Some other form of testing like staging or soak time in production. Please explain.

Navigate into the infra/recipes/azure-sandbox/config/ directory. Make sure you have the azure CLI installed and login with az login. Run terraform init and then terraform apply to deploy the changes for azure. Find the outputted nginx ip address and confirm that it works in the browser.
This is just a proof of concept. The next step is to deploy the actual azure storage integration using terraform and AKS.

https://github.com/linkedin/openhouse/tree/main/infra/recipes/k8s/environments

Terraform would be primarily used for k8s deployment and with that I tend to have directory structure like `infra/recipes/k8s/terraform/'. But as we evolve terraform be used solve other use cases beyond kubernetes deployment. I am fine the the below directory structure as proposed by Sumedh. We should leverage the existing helm templates and specs as part of terraform helm provider to deploy OH services.
infra/recipes/terraform/azure/
-- all the common configs
infra/recipes/terraform/azure/sandbox
-- sandbox cluster specific configs
-- leverage existing helm k8s templates and configs for deployment

For example,
environments/sandbox/main.tf will use,
terraform {
  required_version = ">= 0.12"
  backend "azurerm" {
    # Backend configuration here
  }
}

provider "azurerm" {
  features {}
}

module "mysql" {
  source       = "../../modules/mysql"
  environment  = "sandbox"
  db_name      = var.db_name
  db_username  = var.db_username
  db_password  = var.db_password
  # other variables
}

.... other features

and refer to mysql module defined in modules/mysql/main.tf
resource "azurerm_mysql_server" "mysql" {
  name                = var.db_name
  location            = var.location
  resource_group_name = var.resource_group_name
  admin_username      = var.db_username
  admin_password      = var.db_password
  # other configuration
}

output "mysql_server_name" {
  value = azurerm_mysql_server.mysql.name
}

and so on and so forth....

Also I would recommend break this PR into small PRs.
In PR1: Add the basic environment for sandbox, providers, etc that does no provision of actual resource.
In PR2: Add mysql module and respective update to sandbox environment.
In PR3: Add k8s module and respective update to sandbox environment.
In PR4: Add storage account module and respective update to sandbox environment.

This PR can be closed?


admob causing crashes on most of the device in android

Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: PlatformException(error, Attempt to invoke virtual method 'int java.lang.Integer.intValue()' on a null object reference, null, java.lang.NullPointerException: Attempt to invoke virtual method 'int java.lang.Integer.intValue()' on a null object reference
	at io.flutter.plugins.googlemobileads.h0.onMethodCall(Unknown Source:601)
	at w9.k$a.a(Unknown Source:17)
	at k9.c.l(Unknown Source:18)
	at k9.c.m(Unknown Source:40)
	at k9.c.i(Unknown Source:0)
	at k9.b.run(Unknown Source:12)
	at android.os.Handler.handleCallback(Handler.java:883)
	at android.os.Handler.dispatchMessage(Handler.java:100)
	at android.os.Looper.loop(Looper.java:227)
	at android.app.ActivityThread.main(ActivityThread.java:7822)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1026)
)
       at StandardMethodCodec.decodeEnvelope(message_codecs.dart:651)
       at MethodChannel._invokeMethod(platform_channel.dart:322)

Admob version
google_mobile_ads: ^4.0.0

this is causing lots of crashes

the crash is happeining when loading open add
that behaviour cannot visible on android 14 but on

android 10 and lower device you can see this issue
how to reproduce this
Open add admob Example

Hey @alihassan143, a few questions:
The Crashlytics screenshot doesn't offer much information. Can you share a more detailed screenshot?
Does the crash only happen on version 4.0.0?
Can you reproduce it using our Android  app open example?
Does only it occur with the App Open example? Does it happen on app launch? Ad display?
Can you please provide clear steps to reproduce? Thanks

Hi @malandr2, I've had the same experience the OP is having in my app for a couple of times.
I have two ads in my app:

The BannerAd in the home page loaded fine.
The NativeAd I load inside a ListView.builder crashed the app as soon as it was loaded

Package: google_mobile_ads: ^4.0.0
Device: OnePlus6T (OnePlus6T)
OS: Android 11 (SDK 30)
Type: java.lang.NullPointerException
Exception java.lang.NullPointerException:
  at com.sclh.sonar.ListTileNativeAdFactory.createNativeAd (ListTileNativeAdFactory.java)
  at io.flutter.plugins.googlemobileads.FlutterNativeAd.onNativeAdLoaded (FlutterNativeAd.java)
  at io.flutter.plugins.googlemobileads.FlutterNativeAdLoadedListener.onNativeAdLoaded (FlutterNativeAdLoadedListener.java)
  at com.google.android.gms.internal.ads.zzbsk.zze (zzbsk.java)
  at com.google.android.gms.internal.ads.zzbhb.zzbJ (zzbhb.java)
  at com.google.android.gms.internal.ads.zzaun.onTransact (zzaun.java)
  at android.os.Binder.transact (Binder.java:1054)
  at m.ajj.ba (:com.google.android.gms.policy_ads_fdr_dynamite@234310605@234310602057.578924991.578924991:8)
  at com.google.android.gms.ads.internal.formats.client.al.e (:com.google.android.gms.policy_ads_fdr_dynamite@234310605@234310602057.578924991.578924991:9)
  at com.google.android.gms.ads.nonagon.ad.nativead.af.run (:com.google.android.gms.policy_ads_fdr_dynamite@234310605@234310602057.578924991.578924991:63)
  at android.os.Handler.handleCallback (Handler.java:938)
  at android.os.Handler.dispatchMessage (Handler.java:99)
  at m.awy.a (:com.google.android.gms.policy_ads_fdr_dynamite@234310605@234310602057.578924991.578924991:1)
  at com.google.android.gms.ads.internal.util.f.a (:com.google.android.gms.policy_ads_fdr_dynamite@234310605@234310602057.578924991.578924991:2)
  at m.awy.dispatchMessage (:com.google.android.gms.policy_ads_fdr_dynamite@234310605@234310602057.578924991.578924991:1)
  at android.os.Looper.loop (Looper.java:233)
  at android.app.ActivityThread.main (ActivityThread.java:8068)
  at java.lang.reflect.Method.invoke
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:631)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:978)

Hope it helps
Thanks

Hey @stesancri, looking at your stack trace, your issue looks to be unrelated. Are you following https://codelabs.developers.google.com/codelabs/admob-inline-ads-in-flutter#7? Looks like your crash is happening in a ListTileNativeAdFactory class, which is not in the google_mobile_ads flutter plugin. It appears to be an implementation issue. As a next step I recommend debugging your createNativeAd method.
If you have any follow-ups, please create a new issue in GitHub as it's ultimately unrelated to OP's issue. Thanks!

Not sure at all, but could it be related to this?
As soon as I migrate my code following that guide, the google_mobile_ads package makes my app crash, so I wonder if there is a link or not...

Closing due to inactivity from OP


[FE] Cellxgene FE serves static assets from /static/cellxgene/*

Given the brand merge, both cellxgene and DP will now live on cellxgene.cziscience.com.
To avoid static assets from DP and cellxgene being accidentally served from the same route, we need to do the following:

Change cellxgene's Webpack config to serve all static files (fonts, images, js, css, etc.) from /static/* to /static/cellxgene/*

E.g.,
Current:

https://cellxgene.cziscience.com/static/main-90d61520926ca4c19056.js

https://cellxgene.cziscience.com/static/assets/RobotoCondensed-Regular-db6d916952fb988edac1e531d47ff83a.ttf

New:

https://cellxgene.cziscience.com/static/cellxgene/main-90d61520926ca4c19056.js

https://cellxgene.cziscience.com/static/cellxgene/assets/RobotoCondensed-Regular-db6d916952fb988edac1e531d47ff83a.ttf

@maniarathi @colinmegill @seve Tagging you for visibility 😄 Thank you!

Should this ticket be transferred to the cellxgene repo?

OH! Yes totally 🙏 Sorry for the confusion 😆 Please help me with that! Thank you!


Case-sensitive entity name creates duplicate menu item

There seems to be a case sensitive issue with uml generator. At some stage in the parser it thinks the entity name is case sensitive and hence treats it as a different "entity" when constructing the menu in navbar.html.
When I initially created the Address entity using "yo entity generator" I must have typed "address" as the entity name. Although it camelized the table name to "Address" in the Liquibase and Java, the .jhipster/address.js has kept it as "address" . When I re-defined "address" entity it again in my JHL script I typed it as "Address". That confused JHIPSTER-UML generator and created another menu entry in navbar.html called "Address". This is why I ended up with two identical menu items under "Entities".
My Entities dropdown menu script ended up like this:

                         address

class="glyphicon glyphicon-asterisk">
                         Address

You can see Address entity has been added twice to the menu. It did however, updated all other scripts and Java classes correctly including the Liquibase and Java entity.
You can re-create the situation by:
1- Create an entity named "address" using entity generator.
2- Update the entity but call it "Address" using JHL , you should see double menu item Address under Entities admin menu.

What version of JHipster UML are you using? I can't reproduce the issue.

I think the last one I downloaded was 1.6.1. What is the best way to tell the current version of JHipster UML?

The JHL parser expects an upper case as the first character of the entity name! Why is that?
This script is rejected by the parser:
entity address {...}
I get the following error message:
Expected a space or an entity name but "a" found.
SyntaxError: Expected a space or an entity name but "a" found.
    at peg$buildException (C:\Users\u382911\AppData\Roaming\npm\node_modules\jhipster-uml\lib\dsl\jhGrammar.js:362:14)
    at Object.peg$parse [as parse] (C:\Users\u382911\AppData\Roaming\npm\node_modules\jhipster-uml\lib\dsl\jhGrammar.js:3787:13)
    at DSLParser.parse (C:\Users\u382911\AppData\Roaming\npm\node_modules\jhipster-uml\lib\dsl\dsl_parser.js:36:27)
    at Object.<anonymous> (C:\Users\u382911\AppData\Roaming\npm\node_modules\jhipster-uml\jhipster-uml.js:77:27)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)

The following is what it expect:
entity Address {...}
Is this a separate issue?

Hello, sorry for the late answer, but I had a lot of work.
This is not an issue, it is intended (I believe there was a comment about that a few months ago). And no, this doesn't affect what's generated because we format everything from the JDL to the JSON.

I'm going to add a -version option... it may be helpful.

If you want, clone this repo and npm link it so that you may try with the latest version (it's stable!).


Agent 2.198.3 intermittently hanging on different release tasks

Agent Version and Platform
Version of your agent? 2.198.3
OS of the machine running the agent? Windows Server 2022 Datacenter Azure Edition
Azure DevOps Type and Version
dev.azure.com
If dev.azure.com, what is your organization name? https://dev.azure.com/bkycorp
What's not working?
Random tasks will intermittently hang and never complete. These tasks can be anything from doing an IIS Web App Management task to a simple PowerShell script task. A task that should only take a few minutes goes and goes for minutes on end with no console output of any kind. I have to cancel a release and redeploy it and then it succeeds.
Agent and Worker's Diagnostic Logs
I am unable to provide logs because the file has a 0 byte length.

Thanks for the issue-reporting in this repository, currently we are under issues with higher priority, we will take a look at this ticket once we will have enough capacity.

@stephajn could you please share agent logs to investigate for problematic runs? You can send them to my email -<EMAIL_ADDRESS>

Hey @stephajn, do you still experiencing this issue? I have tried the version of the agent that you have mentioned (v2.198.3) and the latest one (v2.200.2) and didn't see such behavior. Could you please attach logs from _diag folder to this ticket, if you still experience this problem?

@stephajn Is this issue still actual for you?

@anatolybolshakov and @EzzhevNikita
My apologies for the late response. This wasn't an issue once we downgraded the agent version to 2.193.0.
For the time being, we are holding fast on that version since it hasn't had any troubles.  When I looked at the logs back when the jobs would intermittently hang, the diagnostic log had a zero byte length and so I couldn't actually ship you anything to review.

@stephajn got it thanks! Any additional logs for this issue would help of cource, we will continue investigate changes between  2.193.0 - 2.198.3 for possible issues.

@stephajn Was thesystem.debug enabled in your pipelines?

@stephajn is this issue still actual for you?

@stephajn I'm closing this due to inactivity. Please re-open if you still have any problems.


Ionic 4 swipe to go back

Feature Request
In ionic 3 it's possible to swipe right to go back on ios when the back button is visible in the toolbar. This is not possible anymore on ionic 4 anymore. I also tried on the ionic-stencil-conference-app, same behavior.

it's not working on the stencil conference app either which doesn't use angular. This works on ionic 3 and it's the ios behavior, then it should be in ionic 4 without having to dev it by yourself

There is already an issue about it I didn't see #15018


Project still alive?

Is this just a mock of what an API could look like with the intent of having the education dept build their own?

Yes, this was a prototype I created on request from the White House to stimulate a series of conversations we were having with Dept of Ed. It jumpstarted 3 rounds of talks, but I have seen no other action. Honestly, it is something the industry should come together and to....and push Ed to accept the results.

Is there an API now in production given the new mobile app that just launched?

No that I know of -- here is where to tune in - https://github.com/usedgov/API-Program/issues/1


Use rustfmt

Just to see what this would look like. Let's wait for #437 to merge before moving forward with this so that we avoid merge conflicts. This PR is much easier to rebase.
I have not read through this carefully but I am making a PR so we can discuss the formatting and file rustfmt issues as necessary. I already filed https://github.com/rust-lang-nursery/rustfmt/issues/1094.
One thing that got much worse is nested aster builders. Before:
    let wrapper_ty = builder.path()
        .segment("__SerializeWith")
            .with_generics(wrapper_generics.clone())
            .build()
        .build();

After:
    let wrapper_ty = builder.path()
        .segment("__SerializeWith")
        .with_generics(wrapper_generics.clone())
        .build()
        .build();

Ref #438.

@dtolnay: Yeah I thought about those aster chains being a problem for rustfmt. One way of handling this we could change the aster api to use closures instead of doing my silly chaining. So your example before could look like:
    let wrapper_ty = builder.path()
        .segment("__SerializeWith", |b| {
            b.with_generics(wrapper_generics.clone()).build()
        })
        .build();

Which could look much nicer. Or we just lift these segments out into a variable like:
    let segment = builder.segment("__SerializeWith")
        .with_generics(wrapper_generics.clone())
        .build();
    let wrapper_ty = builder.path()
        .with_segment(segment)
        .build();

Or even just create the variable in place with:
    let wrapper_ty = builder.path()
        .with_segment(
            builder.segment("__SerializeWith")
                .with_generics(wrapper_generics.clone())
                .build()
        )
        .build();

:umbrella: The latest upstream changes (presumably #437) made this pull request unmergeable. Please resolve the merge conflicts.


Adds StandardMessageCodec benchmark

This is a benchmark to land before I land my performance fixes.
test exempt: is a test
Pre-launch Checklist

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

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

Please don't mark your own PRs as test-exempt, the whole point of the process is that every PR that doesn't have tests needs to be manually checked so as to denormalize the behaviour of writing PRs without tests.

Please don't mark your own PRs as test-exempt, the whole point of the process is that every PR that doesn't have tests needs to be manually checked so as to denormalize the behaviour of writing PRs without tests.

@Hixie everything that is landed is reviewed, so it is manually checked.  In the case of writing tests it seemed we could trust the author and the reviewer to make that decision and that your attention was only required in more nuanced situations.  In any case, I don't think there is harm communicating the explanation.  Let me know if you want to format it differently.

I think for this case we should teach the bot about tests under the dev/benchmarks folder....

test-exempt: is a test


feat: Support output.clean

Summary

[Feature Request]: Support output.clean
Related issue (if exists)

Types of changes

[ ] Docs change / Dependency upgrade
[ ] Bug fix
[x] New feature / Improvement
[ ] Refactoring
[ ] Breaking change

Checklist

[ ] I have added changeset via pnpm run changeset.
[ ] I have added tests to cover my changes.

 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.

LGTM. You might need resolve the conflicts. @h-a-n-a cc

done

It's missing in the docs 😢 I was lucky to read changelogs and notice that rspack supports it now out of the box. Otherwise I would still do rm -rf dist && rspack build 😢

It's missing in the docs 😢 I was lucky to read changelogs and notice that rspack supports it now out of the box. Otherwise I would still do rm -rf dist && rspack build 😢

Sorry for that. We have created https://github.com/web-infra-dev/rspack-website/issues/265 but haven't resolved it.


Import: fix skinning (and other node hierarchy problems)

Very large patch, but it's a massive improvement for all node hierarchy problems, most notably skinning.
Fixes every file I tested ＼（Ｔ∇Ｔ）／
Fixes #239.
Fixes #257.
Fixes #483.
Fixes #509.
Fixes #698.
Fixes #699.
Fixes #700.
Fixes #703.
Fixes #847.
Fixes #848.
Fixes #850.
Also fixes the model from #251.
Super big thanks to @fire btw for filing all the issues with test models <3 They were really invaluable, thanks!
Example before/after (#703)

Virtual nodes
The main piece of machinery added is an intermediate graph of "virtual" nodes, or vnodes. The glTF node graph is copied to a vnode graph, and a series of passes are done to transform it into a form we can import into Blender. This gives us a lot of flexibility in doing "global" transforms to the node graph before we commit to it by creating real Blender nodes for everything.
Vnodes are stored in a flat dict and referenced by keys just like glTF nodes are stored in a flat array and referenced by indices. Integer keys correspond to vnodes that came from glTF nodes. A root node is always inserted with key 'root'.
Summary of the vnode passes
Here's a visualization of how the graph gets transformed from "glTF form" into "Blender form"

A dummy root is added to turn the forest into a tree. This simplifies a lot of things. The dummy root doesn't actually get created in Blender though.
All nodes used by joints are turned into bones and their common ancestor is marked as an armature. (Assume the colored nodes were used by joints.)
The skinned mesh is moved to become a child of the armature that modifies it.
The node with both a mesh and camera has the camera split off into a separate node.

Joints, skins, and armatures
In master, the importer believes that every node is used by at most one joint, and it uses the inverse bind matrix of that joint to set the bone's local-to-parent. This is obviously not viable if there is not a unique joint that uses a node.
This PR changes our attitude about bones and joints. The importer now thinks each bone should have the same world transform it has in glTF (no dependence on the inverse binds). It's also no longer correct to think of skins as imported as armatures; instead a vnode pass marks nodes that should be armatures or bones for the sole purpose of satisfying the following Blender constraint

The nodes used to skin a mesh must all be bones, and must be proper descendants of a common node that is an armature.

There's no problem with overlapping skins if we think about it this way.
Skinned meshes
Where in the glTF node graph a skinned mesh is instantiated doesn't affect its transform; only the nodes used by its joints do. So we always move a skinned mesh from wherever it was in glTF to (a fresh child of) the armature that contains its bones. The fresh child has the identity transform, so the skinned mesh has the same world transform as the armature, and then there's a clear path bones-to-arma, then arma-to-world, where the mesh isn't affected by anything else.
We also need to calculate the positions of the vertices in the bind pose ie. the pose the edit bones are in. This might not be the same as the glTF, mainly because glTF nodes can be scaled but edit bones can't. Skinning code for the verts was added to primitive creation. This is the place the inverse bind matrices are now used.
Animation
Not much to say here, but it was updated to use vnode ids. The bone code has become simpler; there's a straight-forward formula for the TRS of the pose bones, which is nice.
We do have to be cognizant of objects parented to a bone (we need to translate them back to the head of the bone since they're at the tail by default).
Things I dropped
These are things I stripped out that I'd like to address later

All scene/collection handling was dropped. The glTF scenes are ignored and the whole node forest is imported into the current Blender scene. This is not actually a big difference from before; in 2.8 everything already went into the current scene.
We no longer make any attempt to get rid of the Yup2Zup node. It will be much simpler and more foolproof to convert coordinates whenever we read them out of the glTF file, which will get rid of it for good.
The stuff for selecting objects/setting the active object was dropped.
The stuff for making bone chains was dropped. You can't change the bone length after import because it has significance in the transform hierarchy (it affects objects parented to the bone). It should be possible to add a vnode pass to rotate/resize bones (see #324) after this.

Thanks for the callout. I was getting frustrated.
What's a good way to test this?

I tested https://github.com/KhronosGroup/glTF-Blender-IO/issues/850 and it worked. Thank you so much.
💯

Hello @scurest
Thanks for this huge PR.
I didn't go deeply into it yet, but here are some general thoughts (meanly in things you currently dropped):
First, I would like to include guys from sketchfab into testing process. @AurL Aurélien, can you please have a look into it, or forward it to the new responsible og glTF in Sketchfab?

All scene/collection handling was dropped. The glTF scenes are ignored and the whole node forest is imported into the current Blender scene. This is not actually a big difference from before; in 2.8 everything already went into the current scene.

One possible solution is to import all in current scene, and to create (if needed) some collection(s) to reflect glTF scene. That way, no issue with glTF node in multiple glTF scenes, because all Blender objects will be in same scene, but collections can be a good way to display glTF scenes. This can be done later, not directly linked to this PR.

We no longer make any attempt to get rid of the Yup2Zup node. It will be much simpler and more foolproof to convert coordinates whenever we read them out of the glTF file, which will get rid of it for good.

Using debug values to get rid of Yup2Zup is part of the production pipeline in Airbus DS. So this has to be restored, we can't drop it.

The stuff for selecting objects/setting the active object was dropped.

Same here, Airbus DS need to have the root of imported glTF active at end of import. This need to be restored.
The stuff for making bone chains was dropped. You can't change the bone length after import because it has significance in the transform hierarchy (it affects objects parented to the bone). It should be possible to add a vnode pass to rotate/resize bones (see #324) after this.
Yes, confirmed. We can drop it for now (and we need to think how we can manage it later).

Thanks for looking @julienduroure. I know it's huge ><
The active object is easy. Could you clarify what is supposed to be selected though? In master it depends on a lot of different things (like whether there are animations and what the debug_value is).
Should I just select all (and only) the imported objects?

I made the first object in the default glTF scene active and selected. Tell me if you wanted something else.
I also pushed the code for getting rid of the Yup2Zup node by hard-applying the conversion whenever we read. You'll notice that the direction bones point changes and correction nodes are inserted for cameras/lights (just like how the exporter works). I can change these in a later PR (I have the code for it written already).

Regarding active object:
At end of import, the active object must be the root object just imported. In most cases, this Blender Object is the glTF tree root (but can be different because of armature). This should be simpler to find it in vnode tree. If there are multiple root objects, pick (randomly) one to be active. Regarding scenes : if there is a scene property in glTF file, use this scene. If there is no scene property, use the scenes[0] scene.
Regarding Yup2Zup node : No selection is required at end of import.
Current rules :

We don't want to convert Yup to Zup:

Because user don't want it (debug = 100)

We want to convert Yup to Zup, and we will NOT remove it at end of process

Because there is some animation on object (check on gltf.animation_object in code), we can't remove it. So the process is to check all imported objected without any parent (most of the time these are root object in glTF tree, but it can be different because of armature), and set all these objects parent to Yup2Zup empty.

We want to convert Yup to Zup, and this empty will be removed at end of process

When there is no animation on object. In that case, We select all root objects (once again, this is most of the time root glTF nodes, but not always), and then apply transforms on all selected objects, and then remove Yup2Zup node

@scurest Seems we updated this thread in same time :)
I will have a look on your proposal for active/selection and for Yup2Zup removal.
First remark, as I wrote in my previous post that I wrote during your update: No selection is needed at end of import, only active object is really required. So I think you can remove the selection part (we will save some time).

We did :) I amended the last commit to set the active object like you said and not select anything.

Can you please also restore log on debug 101 ?

Sure! I have it print the vnode id now.

@scurest I've added another test case. https://github.com/KhronosGroup/glTF-Blender-IO/issues/862

#862 occurs when an armature object has both its loc/rot/scale animated, and also has pose bone animations, and two actions would get created. That didn't ever used to happen since the importer appears to create a new node for the armature that wasn't in the glTF node tree, so it's loc/rot/scale couldn't be animated. In this branch, an existing node is turned into the armature (except when that's not possible and the dummy root is turned into an armature).
So the animation_node code now uses the same cache as the animation_bone code to be sure only one action is created.
Here you can see the difference in the node tree, master/this branch. You can see the extra "6" node in master.

Thanks for catching that @fire :)

Is this targeted for 2.82?  https://developer.blender.org/project/view/89/

@fire No, BCon3 is tomorrow, I don't have time to test it heavily. This is a very good improvement (thanks again scurest :) ), but this is quite a big change, so we need time to test it carefully.
So this will go with 2.83

@julienduroure How's this coming?


How to get access remotely on ubuntu?

I have installed this Json server on ubuntu, on google cloud instance.
And I want to have access to this rest api from network (not only in local network in ubuntu)
What I have to do to make access from external network?
If I set up nginx then I go to http://<IP_ADDRESS>:PORT and its working but when I run json-server its not working.
https://i.stack.imgur.com/WOdzr.jpg
https://i.stack.imgur.com/myzFo.jpg
https://i.stack.imgur.com/xCJXs.jpg
What i have to do?

I believe you have to set the host when you run json-server. It is set as localhost by default.
You may try running the json-server with:
json-server -p 4000 -H <IP_ADDRESS> --watch db.json

I believe you have to set the host when you run json-server. It is set as localhost by default.
You may try running the json-server with:
json-server -p 4000 -H <IP_ADDRESS> --watch db.json

yep, -h real ip  ==> false, -h <IP_ADDRESS> ==> success


Added tasks 239-295

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

Comparison is base (4418a0f) 93.78% compared to head (c60a122) 93.78%.

:exclamation: Current head c60a122 differs from pull request most recent head 4a01c3b. Consider uploading reports for the commit 4a01c3b to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   93.78%   93.78%
=======================================
  Files          58       58
  Lines         772      772
  Branches      164      164
=======================================
  Hits          724      724
  Misses         17       17
  Partials       31       31

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


Ability to use the backups tool

It would be cool to be able to run the backup tool just like the compilation tool, indexation and such

Closing as I am pretty sure this project is completeley abandoned and I am not using the tool.


Doesn't work with Sketch 49

Hi there,
Thanks for amazing plugin! It saved so much time for us!
Not sure if you have plans to support this awesome plug in... But it would be great if you could take a look.
Thanks!

Hey @blynkkk ! Thanks for reporting this, something got broken after the last release. I have no idea what is causing this, but I'll take a look at it soon.

Great! Thanks a lot.

@blynkkk everything works well here (see gif below).

Are you sure your layers aren't grouped?
If you have time, could you please take a quick video of what's happening?

Closing this because of lack of info, feel free to reopen if needed.


Let an OS decide which Python version to use to run tests

Scripts are python 3 ready, therefore it's better not to hard code python 2 which may not even be available on some systems.

Btw. I also thought about adding a missing hashbang to gtest_parallel.py, which might be run without gtest_parallel_tests.py. What do you think?

Good cleanup, this matches gtest-parallel which imo is the "supported way" of running gtest_parallel.py, so I don't think we should add a shebang into gtest_parallel.py. All documentation mention gtest-parallel as the target to run.

Yeah, then it would be superfluous. OK, cool.


Refactor in preparation for the ORCRecordReader

Refactoring record reader to allow for ORC configuration during segment creation.

Couple of suggestions:

We can have init() present at all times - I don't see an issue with an empty implementation for most and overridable by the readers that need it.
We can make the reflection based support to be more general: If the users specify a record-reader class in the segment generation config, you can instantiate it and return. If only the input format is specified, we will instantiate the default readers for specific formats. Do capture the order in which these will be handled and what happens when both are specified (we probably just need one of them to be specified to avoid confusion).
In the recordreader factory, allocate the record reader and before returning call init() on it; ie, move the return to the end of the getReader method.

We can have init() present at all times - I don't see an issue with an empty implementation for most and overridable by the readers that need it.

I like this idea, but I am concerned because some of private "init" methods there already, and a lot of code (perhaps other open source users too?) directly instantiates these readers through the constructor rather than the init method. I also don't want people to be confused when they add a new record reader.

We can make the reflection based support to be more general: If the users specify a record-reader class in the segment generation config, you can instantiate it and return. If only the input format is specified, we will instantiate the default readers for specific formats. Do capture the order in which these will be handled and what happens when both are specified (we probably just need one of them to be specified to avoid confusion).

I made the reflection based support more general, I like that. However, for the order in which they are handled, ideally, it would be nice to throw an exception so users know they can't set both, however, in segment generator config, we default have avro format currently, and I don't want to break our users' behavior

In the recordreader factory, allocate the record reader and before returning call init() on it; ie, move the return to the end of the getReader method.

Since I wanted the record reader path because of the reason above to override, I moved it in front instead. Let me know if you have thoughts on this.

Does it need the entire segmentGenerationConfig? How can we make these record readers work for both realtime and offline.
Also, we should probably move these record reader implementations out of pinot-core

Does it need the entire segmentGenerationConfig?

Different record readers need different params, so thought segment generation config would be the best/most general. eg: csv record reader needs an extra reader config, json record reader needs a column sort order, etc)

How can we make these record readers work for both realtime and offline.

Not sure, will look into this.

Also, we should probably move these record reader implementations out of pinot-core

Sure, I can put them in their own module although my orc reader brings in hive, so I still would want the reflection instantiation option.

Codecov Report

Merging #3900 into master will increase coverage by <.01%.
The diff coverage is 29.03%.

@@             Coverage Diff              @@
##             master    #3900      +/-   ##
============================================
+ Coverage     67.14%   67.15%   +<.01%
  Complexity        4        4
============================================
  Files          1032     1032
  Lines         50878    50923      +45
  Branches       7107     7111       +4
============================================
+ Hits          34164    34198      +34
- Misses        14378    14393      +15
+ Partials       2336     2332       -4

Impacted Files
Coverage Δ
Complexity Δ

...inot/core/data/readers/GenericRowRecordReader.java
94.11% <0%> (-5.89%)
0 <0> (ø)

...data/readers/MultiplePinotSegmentRecordReader.java
72.72% <0%> (-0.67%)
0 <0> (ø)

.../pinot/core/minion/segment/MapperRecordReader.java
81.08% <0%> (-2.26%)
0 <0> (ø)

...pinot/core/minion/segment/ReducerRecordReader.java
90.19% <0%> (-1.81%)
0 <0> (ø)

...ealtime/converter/RealtimeSegmentRecordReader.java
58.33% <0%> (-2.54%)
0 <0> (ø)

...he/pinot/core/data/readers/ThriftRecordReader.java
80.3% <0%> (-1.24%)
0 <0> (ø)

...va/org/apache/pinot/core/minion/SegmentPurger.java
76.19% <0%> (-0.92%)
0 <0> (ø)

...ache/pinot/core/minion/BackfillDateTimeColumn.java
54.68% <0%> (-0.87%)
0 <0> (ø)

...ot/core/data/readers/PinotSegmentRecordReader.java
90.9% <0%> (-1.69%)
0 <0> (ø)

...ache/pinot/core/data/readers/AvroRecordReader.java
76.19% <0%> (-1.86%)
0 <0> (ø)

... and 35 more

Continue to review full report at Codecov.

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


Azure

[Enter feedback here]

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

ID: c1cce701-aeb0-07af-6961-4f3e5cf2ed44
Version Independent ID: 9c3e5bd6-7367-bfd8-f296-2f91f058a0fe
Content: Get started guide for developers on Azure
Content Source: articles/guides/developer/azure-developer-guide.md
Service: azure
GitHub Login: @ggailey777
Microsoft Alias: glenga

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

ID: c1cce701-aeb0-07af-6961-4f3e5cf2ed44
Version Independent ID: 9c3e5bd6-7367-bfd8-f296-2f91f058a0fe
Content: Get started guide for developers on Azure
Content Source: articles/guides/developer/azure-developer-guide.md
Service: azure
GitHub Login: @ggailey777
Microsoft Alias: glenga

@FACF302 Thank you for reaching out, can you please help give more context on specific feedback you have regarding the Azure documentation linked to this GitHub issue ? so that we can assist you accordingly.


Make it work on Symfony 3.0

Your behavior could not be installed on Symfony 3.0. Please, merge this PR so composer won't block it.

Sure thanks :)

Thank you as well :+1:


simple-user listener is never execute

this.session is a Proxy object, session is Inviter object, so "this.session !== session" is always true.

@john-e-riordan any idea why this check is even in there?

Guessing it may have been in there for a reason prior to c1a86a7c4 and could/should have been cleaned up then? Anyway, I don't see any harm in removing the guard - looks like it is not currently guarding anything.

Done in 258c5036beb31ae2f9d2d1ce5157955814e45f39


How do I use Semantic UI with NG2 Seed ?

IMPORTANT: This repository's issues are reserved for feature requests and bug reports. Do not submit support requests here.
Steps to reproduce and a minimal demo of the problem
I am trying to add ngSemantic to be able to use https://github.com/vladotesanovic/ngSemantic
I have tried to add
this.NPM_DEPENDENCIES = [
...this.NPM_DEPENDENCIES,
{src: 'jquery/dist/jquery.min.js', inject: 'libs'},
{src: 'ng-semantic/ng-semantic.js', inject: 'libs'},
];
Just adding with jquery does not fail, but adding the ng-semantic afterwards fails.
I have also tried adding it as
this.APP_ASSETS = [
...this.APP_ASSETS,
{src: ${this.APP_SRC}/node_modules/ng-semantic/ng-semantic.js, inject: true, vendor: true}
];
Setting the vendor to either true or false gives the same.
Current behavior
I keep getting this error :
ng-semantic.js?1475608741400:14 Uncaught ReferenceError: require is not defined
The server is having issues locating it.
WARN [web-server]: 404: /base/node_modules/ng-semantic.js
Chrome 53.0.2785 (Mac OS X 10.11.6) ERROR: '(SystemJS) XHR error (404 Not Found) loading http://localhost:9876/base/node_modules/ng-semantic.js
Error: XHR error (404 Not Found) loading http://localhost:9876/base/node_modules/ng-semantic.js
at XMLHttpRequest.wrapFn [as _onreadystatechange] (http://localhost:9876/base/node_modules/zone.js/dist/zone.js?f8afdc0a1fd11e453f00ee26da18050645299a9a:636:29)
I have also tried to change module in tsconfig.json
"module": "system",
That really gave a lot of error.
Expected/desired behavior
I would love to use Semantic UI with NG2 Seed, but as a bigtime backender I am rather baffled with all the loading and JS magic that happens and not quite sure where to start here.

Most likely you'd want to inject jquery and load ng-semantic ui path/package config with systemjs. This means keeping:
// notice that semantic ui is no here anymore
this.NPM_DEPENDENCIES = [
...this.NPM_DEPENDENCIES,
{src: 'jquery/dist/jquery.min.js', inject: 'libs'}
];

and add an entry in SYSTEM_DEV_CONFIG, or just require it with a regular import statement and probably it is going to work.


Unable to resize image. Most likely due to not enough memory.

I'm getting this error from react-native-image-resizer package.
Unable to resize image. Most likely due to not enough memory.

The package i believe doesn't work well with images taken with the camera, not due to not enough memory.
I think another package should be tried.

For those who face this annoying error.
in android/app/src/main/AndroidManifest.xml add android:largeHeap="true" to the application tag.
<application
      ...
      android:largeHeap="true"
      ...
>


Automatically deploy new tagged versions to PyPI

We should switch to automatically deploying the Python extension whenever we bump the Wasmtime version number, based on the whl packages our CI builds.
/cc @yurydelendik

Hi, I recently opened two PRs (#627 and #648) that touch the Python wheel generation, and since now the generated wheels have the right names (and version) I can look into using https://github.com/pypa/gh-action-pypi-publish to publish the wheels to PyPI.
Should I make a new PR?
I obviously don't have permissions to manage the repo secret here in GitHub nor create the PyPI token for upload, so someone with access to those would need to fill it in.

That'd be awesome if you could help us tackle this @luizirber, thanks!
If you want to test out on a local copy of the repo and let us know what secrets to configure, we can do that and then merge a PR!

@luizirber Are you still working on this? Is there a way I can help?

@OrangeTux feel free to take this one, I didn't do anything other than ask =]

@luizirber Cool, I'll take a look at the issue than.


Obfuscated apps?

Is it possible modify search_deobfuscator_class that it works on obfuscated apps? (not searching fixed names). Or maybe add some additional parameters where we can specify what to do? :)

It should already work with obfuscated apps, the matching algorithm it's pretty simple so it can produce a lot a false positives. I could add a parameter to specify the class to use like you said.
I haven't touch this project for like 9 months, I need to read it :sweat_smile:

:-) check email, i send you 21.5 some example

Looked into it and realized there is no support for multiple entries in the obfuscated String array, but only one obfuscated string.
Also rewrote the search for the obfuscated string and getString signature.
I'm gonna test it some more and when it's ready I'll push the fixes.

Can you tell me if it works now?

Ok, try now. It should work.

I try first release.zip but there was error. Then i find that in repository is probably newer version (difference in .py and .jar) I try this but error was the same :
Traceback (most recent call last):
  File "deobfuscator.py", line 212, in <module>
    main(args)
  File "deobfuscator.py", line 110, in main
    deobfuscator_class, encrypted_strings = deobfuscator.search_deobfuscator_class(path)
  File "deobfuscator.py", line 77, in search_deobfuscator_class
    if self.get_deobfuscator_signature(os.path.join(root, filename)):
  File "deobfuscator.py", line 90, in get_deobfuscator_signature
    method_class = file.split('smali')[1].split('/', 1)[1][:-1]
IndexError: list index out of range

I can send you decoded example for testing purposes (.rar, ~85MB)

Uh, that's strange. If you send me the rar I'll appreciate it.

Check your email

It seems like you are using Windows and I only tested it on Linux. Try replacing '/' with '\'

I don't send any '/' char to commandline. I execute: python.exe deobfuscator.py dir_name.
dir_name it's just name without path. If yo use full path to dir_name, also with replaced '/' - it's the same.

Yes, os.sep it's a good suggestion. Tried your app (2.2.0) on Fedora 34 and didn't have any problem. I'm now trying on Windows.

Ok, tested and should work now.

One small issue, check email ;-)


Kubernetes Clone

I am a bit lost on this project and related dependencies, because this project largely reinvents Kubernetes, but with a VPN setup to tie multiple nodes together. I also do not see the use case where a load balancer in one location would have backends in different countries, it adds to the latency and creates a poor user experience.

It's not a Kubernetes clone in any sense, it wasn't even inspired by it. And deploying a website in several countries was just an example, you just have to think a bit to see how many other examples can be there. For LittleJet it is irrelevant if you deploy an application in Argentina or Canada, you can deploy your application in the same organization but with different servers for redundancy and load balancing purposes. Also note that the VPN configuration is entirely optional.

This issue has been closed due to inactivity. You can reopen it if you wish.


Release on WebJars

It would be really helpful if jquery.scrollTo was available on WebJars:
http://www.webjars.org
(we're working on an application written in a combination of Clojure and ClojureScript, built with Leiningen. None of bower, npm or composer are particularly convenient to use from Leiningen. Right now, the only third-party JS library we're using that isn't on WebJars is jquery.scrollTo :-)
Thanks!

Hi @paulbutcher still interested in this? else I'll close

It would still be nice, but (obviously!) not something I've been able to find time to look at, so feel free to close.