NewEngineFragmentBuilder and NewEngineInGroupFragmentBuilder in FlutterFragment should respect shouldDestroyEngineWithHost when using provideFlutterEngine

Use case
We use local_auth plugin which used androidx Biometric on Android.
androidx biometric force use fragment Activity, so we change our super class to FlutterFragmentActivity.
Previously, we override getCachedEngineId to used cached fragment, but this result in blank screen after put app in background for over one night.
Then we  remove getCachedEngineId, and use override of provideFlutterEngine.
Recently, when using monkey test on Android, we found crash log for flutter fragment like this:
crash:
// CRASH: com.mini.myBMW.preseries.cn.univ (pid 28143) (dump time: 2024-01-05 00:34:21
// Version:
// Long Msg: java.lang.AssertionError: The internal FlutterEngine created by FlutterFragment{c10f699} (940cd33c-d79f-4781-adf4-f4001eed9003 id=0x1 tag=flutter_fragment) has been attached to by another activity. To persist a FlutterEngine beyond the ownership of this activity, explicitly create a FlutterEngine
// 	at io.flutter.embedding.android.FlutterActivityAndFragmentDelegate.detachFromFlutterEngine(Unknown Source:40)
// 	at io.flutter.embedding.engine.FlutterEngineConnectionRegistry.attachToActivity(Unknown Source:9)
// 	at io.flutter.embedding.android.FlutterActivityAndFragmentDelegate.onAttach(Unknown Source:37)
// 	at io.flutter.embedding.android.FlutterFragment.onAttach(Unknown Source:11)
// 	at androidx.fragment.app.Fragment.performAttach(Unknown Source:49)
// 	at androidx.fragment.app.FragmentStateManager.attach(Unknown Source:186)
// 	at androidx.fragment.app.FragmentStateManager.moveToExpectedState(Unknown Source:134)
// 	at androidx.fragment.app.FragmentManager.executeOpsTogether(Unknown Source:347)
// 	at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(Unknown Source:81)
// 	at androidx.fragment.app.FragmentManager.execPendingActions(Unknown Source:21)
// 	at androidx.fragment.app.FragmentManager.dispatchStateChange(Unknown Source:38)
// 	at androidx.fragment.app.FragmentManager.dispatchActivityCreated(Unknown Source:11)
// 	at androidx.fragment.app.FragmentController.dispatchActivityCreated(Unknown Source:4)
// 	at androidx.fragment.app.FragmentActivity.onStart(Unknown Source:20)
// 	at android.app.Instrumentation.callActivityOnStart(Instrumentation.java:1470)
// 	at android.app.Activity.performStart(Activity.java:8513)
// 	at android.app.ActivityThread.handleStartActivity(ActivityThread.java:4656)
// 	at android.app.servertransaction.TransactionExecutor.performLifecycleSequence(TransactionExecutor.java:235)
// 	at android.app.servertransaction.TransactionExecutor.cycleToPath(TransactionExecutor.java:215)
// 	at android.app.servertransaction.TransactionExecutor.executeLifecycleState(TransactionExecutor.java:187)
// 	at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:105)
// 	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2928)
// 	at android.os.Handler.dispatchMessage(Handler.java:117)
// 	at android.os.Looper.loopOnce(Looper.java:205)
// 	at android.os.Looper.loop(Looper.java:293)
// 	at android.app.ActivityThread.main(ActivityThread.java:9596)
// 	at java.lang.reflect.Method.invoke(Native Method)
// 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:586)
// 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1204)
// crashend

Proposal
I checked the start process of FlutterFragmentActivity and found this:
method ensureFlutterFragmentCreated in FlutterFragmentActivity will call createFlutterFragment,
when getCachedEngineId() != null it will use CachedEngineFragmentBuilder to build the Fragment, which provide destroyEngineWithFragment method to hook the engine.
But when using  provideFlutterEngine, It will use NewEngineFragmentBuilder, which config ARG_DESTROY_ENGINE_WITH_FRAGMENT to true in createArgs.
    @NonNull
    protected Bundle createArgs() {
      Bundle args = new Bundle();
      args.putString(ARG_CACHED_ENGINE_GROUP_ID, cachedEngineGroupId);
      args.putString(ARG_DART_ENTRYPOINT, dartEntrypoint);
      args.putString(ARG_INITIAL_ROUTE, initialRoute);
      args.putBoolean(ARG_HANDLE_DEEPLINKING, handleDeeplinking);
      args.putString(
          ARG_FLUTTERVIEW_RENDER_MODE,
          renderMode != null ? renderMode.name() : RenderMode.surface.name());
      args.putString(
          ARG_FLUTTERVIEW_TRANSPARENCY_MODE,
          transparencyMode != null ? transparencyMode.name() : TransparencyMode.transparent.name());
      args.putBoolean(ARG_SHOULD_ATTACH_ENGINE_TO_ACTIVITY, shouldAttachEngineToActivity);
      args.putBoolean(ARG_DESTROY_ENGINE_WITH_FRAGMENT, true);
      args.putBoolean(
          ARG_SHOULD_AUTOMATICALLY_HANDLE_ON_BACK_PRESSED, shouldAutomaticallyHandleOnBackPressed);
      args.putBoolean(ARG_SHOULD_DELAY_FIRST_ANDROID_VIEW_DRAW, shouldDelayFirstAndroidViewDraw);
      return args;
    }

The same issue with NewEngineInGroupFragmentBuilder.
In FlutterFragment shouldDestroyEngineWithHost method, it deal with the cached case and provided case in same process.
  @Override
  public boolean shouldDestroyEngineWithHost() {
    boolean explicitDestructionRequested =
        getArguments().getBoolean(ARG_DESTROY_ENGINE_WITH_FRAGMENT, false);//Since NewEngineInGroupFragmentBuilder always put true, we got true here.
    if (getCachedEngineId() != null || delegate.isFlutterEngineFromHost()) {
      // Only destroy a cached engine if explicitly requested by app developer.
      return explicitDestructionRequested;
    } else {
      // If this Fragment created the FlutterEngine, destroy it by default unless
      // explicitly requested not to.
      return getArguments().getBoolean(ARG_DESTROY_ENGINE_WITH_FRAGMENT, true);
    }
  }

But because NewEngineFragmentBuilder not respect to what we configured in FlutterFragmentActivity with shouldDestroyEngineWithHost method, we got this issue.
I think we should respect cached engine and provided engine in both creation of fragment and detachFromFlutterEngine in
FlutterActivityAndFragmentDelegate

Currently, we found this crash in monkey test, We still can't confirm the stable reproduce steps for this crash.
But I'm sure only the FlutterActivityAndFragmentDelegate's detachFromFlutterEngine method can throw this error, and the only call case is that there is another fragment calling 'onAttach'.

I'm not sure if this is caused by empty onAttachFragment override? since system sometimes kill the app and then relaunch the app, it will provide the fragment in this method. But it's hard to test this case.

Or should we reconsider, If it's reasonable to use NewEngineFragmentBuilder when we provide a engine?
@NonNull
  protected FlutterFragment createFlutterFragment() {
    final BackgroundMode backgroundMode = getBackgroundMode();
    final RenderMode renderMode = getRenderMode();
    final TransparencyMode transparencyMode =
        backgroundMode == BackgroundMode.opaque
            ? TransparencyMode.opaque
            : TransparencyMode.transparent;
    final boolean shouldDelayFirstAndroidViewDraw = renderMode == RenderMode.surface;

    if (getCachedEngineId() != null) {
      Log.v(
          TAG,
          "Creating FlutterFragment with cached engine:\n"
              + "Cached engine ID: "
              + getCachedEngineId()
              + "\n"
              + "Will destroy engine when Activity is destroyed: "
              + shouldDestroyEngineWithHost()
              + "\n"
              + "Background transparency mode: "
              + backgroundMode
              + "\n"
              + "Will attach FlutterEngine to Activity: "
              + shouldAttachEngineToActivity());

      return FlutterFragment.withCachedEngine(getCachedEngineId())
          .renderMode(renderMode)
          .transparencyMode(transparencyMode)
          .handleDeeplinking(shouldHandleDeeplinking())
          .shouldAttachEngineToActivity(shouldAttachEngineToActivity())
          .destroyEngineWithFragment(shouldDestroyEngineWithHost())
          .shouldDelayFirstAndroidViewDraw(shouldDelayFirstAndroidViewDraw)
          .build();
    } else {
      Log.v(
          TAG,
          "Creating FlutterFragment with new engine:\n"
              + "Cached engine group ID: "
              + getCachedEngineGroupId()
              + "\n"
              + "Background transparency mode: "
              + backgroundMode
              + "\n"
              + "Dart entrypoint: "
              + getDartEntrypointFunctionName()
              + "\n"
              + "Dart entrypoint library uri: "
              + (getDartEntrypointLibraryUri() != null ? getDartEntrypointLibraryUri() : "\"\"")
              + "\n"
              + "Initial route: "
              + getInitialRoute()
              + "\n"
              + "App bundle path: "
              + getAppBundlePath()
              + "\n"
              + "Will attach FlutterEngine to Activity: "
              + shouldAttachEngineToActivity());

      if (getCachedEngineGroupId() != null) {
        return flutterFragment
            .withNewEngineInGroup(getCachedEngineGroupId())
            .dartEntrypoint(getDartEntrypointFunctionName())
            .initialRoute(getInitialRoute())
            .handleDeeplinking(shouldHandleDeeplinking())
            .renderMode(renderMode)
            .transparencyMode(transparencyMode)
            .shouldAttachEngineToActivity(shouldAttachEngineToActivity())
            .shouldDelayFirstAndroidViewDraw(shouldDelayFirstAndroidViewDraw)
            .build();
      }

      return FlutterFragment.withNewEngine()
          .dartEntrypoint(getDartEntrypointFunctionName())
          .dartLibraryUri(getDartEntrypointLibraryUri())
          .dartEntrypointArgs(getDartEntrypointArgs())
          .initialRoute(getInitialRoute())
          .appBundlePath(getAppBundlePath())
          .flutterShellArgs(FlutterShellArgs.fromIntent(getIntent()))
          .handleDeeplinking(shouldHandleDeeplinking())
          .renderMode(renderMode)
          .transparencyMode(transparencyMode)
          .shouldAttachEngineToActivity(shouldAttachEngineToActivity())
          .shouldDelayFirstAndroidViewDraw(shouldDelayFirstAndroidViewDraw)
          .build();
    }
  }

cc @dnfield Dan would you take a look and see if you think the priority is correct?

Currently, we disable restore state by override:
    override fun onAttachFragment(fragment: Fragment) {
        if (fragment is FlutterFragment) {
            fragment.arguments?.putBoolean("enable_state_restoration", false)
        }
    }

We will keep testing and feedback later.

Are there any updates on this issue?
In my app, same issue has been occurred recently.


incorrect swagger file generated

Hi Abel,
I open another issue since it ieasier for me to communicate via github.
Indeed, the generated swagger file does not validate with http://editor.swagger.io
File attached.
swagger.json.txt
BR,
Tomasz

can you show me the signature for
POST /v1/rest/sots/{timestamp}

[OperationContract]
[WebGet(ResponseFormat = WebMessageFormat.Json, UriTemplate = "sots/{timestamp}")]
SOTObject[] GetSOTsForUser(string timestamp);

that's for the GET operation, don't you have another function for the POST ?
the swagger json shows another one (or maybe it's a bug when parsing functions)

    "/v1/rest/sots/{timestamp}": {
      "get": {
        "consumes": [ "application/xml" ],
        "produces": [ "application/json" ],
        "parameters": [
          {
            "name": "timestamp",
            "in": "path",
            "required": true,
            "type": "string"
          }
        ],
        "responses": {
          "default": {
            "description": "not available",
            "schema": {
              "type": "array",
              "items": { "$ref": "#/definitions/Company.SOT.DataMembers.SOTObject" }
            }
          }
        }
      },
      "post": {
        "consumes": [ "application/xml" ],
        "produces": [ "application/json" ],
        "parameters": [
          {
            "name": "sotObjects",
            "in": "body",
            "required": true,
            "schema": { "$ref": "#/definitions/Company.SOT.DataMembers.SOTObject[]" }
          },
          {
            "name": "timestamp",
            "in": "path",
            "required": true,
            "type": "string"
          }
        ],
        "responses": {
          "default": {
            "description": "not available",
            "schema": {
              "type": "array",
              "items": { "$ref": "#/definitions/Company.SOT.DataMembers.SOTObject" }
            }
          }
        }
      }
    },

I'm suspecting an issue when we have an array as a parameter in the body
        "parameters": [
          {
            "name": "sotObjects",
            "in": "body",
            "required": true,
            "schema": { "$ref": "#/definitions/Company.SOT.DataMembers.SOTObject[]" }
          },

the schema should probably be:
"type": "array",
"items": { "$ref": "#/definitions/Company.SOT.DataMembers.SOTObject" }

but first show me the POST function just to be sure

That is right, here is the POST method.
[OperationContract]
[WebInvoke(ResponseFormat = WebMessageFormat.Json, UriTemplate = "sots/{timestamp}", Method ="POST" )]
SOTObject[] SubmitSOTs(SOTObject[] sotObjects, string timestamp);

yes, that's it, the definition of the array is wrong.
after changing it and removing this definition from the json:
    "Company.SOT.DataMembers.SOTObject[]": {
      "type": "array",
      "items": { "$ref": "#/definitions/" }
    },

the json is valid in the swagger editor
I'll look into it, thanks for the help


Update README.md

I don't speak Chinese but I made it a bit more readable and added the LICENSE.

Hi, @duduainankai


Handle new transaction event types (placeholders)

The goal of this PR is to remove syntax errors introduced by the latest auto-generated API wrapper code in #888.
So this incorporated the new API code from that PR, and also the required fixes in other related code.
Please note that Handle new event types “consensus.Delegate”,  “consensus.Undelegate” is still required; this code only adds placeholders, marked with TODO comments, where the actual new stuff will have to go. (Icons, data fields, etc.)

I have updated this PR to contain only that which has been left out of #906.

This has been refreshed again.

Please note that now there is data on the nexus DB which use these new types, and trying to explore that data leads to a crash in the current explorer. This PR fixes that. (But still, this is not ready for review, since it's intentionally a placeholder only, to avoid syntax errors and crashes)


Better build script tooling

[ ] Use something like cross-env to support setting environment variables within build scripts on Windows
[ ] Use rimraf for clean script for Windows
[ ] Add script for generating release builds (building chrome, renaming folder with version number, zipping, repeating for firefox)
[ ] Create dedicated scripts for watch builds to avoid -- --watch messiness
[ ] ...maybe a github action to automate releases?

Actually, Firefox support dynamic import() in content scripts since 89 (https://bugzilla.mozilla.org/show_bug.cgi?id=1536094), so maybe I can get rid of the build process altogether now?

...maybe a github action to automate releases?

Authored this for Toolbox, https://github.com/toolbox-team/reddit-moderator-toolbox/blob/master/.github/workflows/publish.yml - can adapt for here easily enough. note to self to get the CWS token in chromium


feat: remove castError

THIS IS A BREAKING CHANGE
This PR changes the behavior of Solid's error handling, mainly on the concept of "casting errors". Solid has an internal mechanism wrapping thrown values if the value isn't an Error instance.

Why? It was this way before. We intentionally went to normalizing errors. I admit it was mostly for serialization so I could see the argument for reverting it now. But on the other hand I feel as much temptation to normalize farther to prevent serialization errors for things that random libraries pass us that can't be serialized.

@ryansolid it's not exactly that way before. We used to check if the error is truthy, and not if the error existed.


Migrate essential packages

This PR serves as a foundation change in the migration effort of Backplane CLI.
Notable changes:-

Migrate the essentials utility and client packages.
Globalize the getBackplaneClient() function to be more accessible. [Credits: @samanthajayasinghe]
Generate mocks for these packages.
Remove the sample hello command created during the initial stages of repo initialization.

Iterative changes/migration can be done over these introduced changes.

@wanghaoran1988 ready for a second round of review.
/label tide/merge-method-squash

/lgtm
/approve


Connect plantings with seeds.

Basics

[x] I added a line to changelog.md
[x] The PR is rebased with current master.

[x] References to issues, e.g. close #X, are in the commit messages and changelog.
[x] The buildserver is happy.

Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I fully described what my PR does in the documentation
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added automated tests or a manual test protocol
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
[ ] Code is consistent to our Design Decisions
[ ] Exceptions to any guidelines are documented

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] I've read through the whole documentation
[ ] I've checked conformity to guidelines

@markus2330 Plants are now showing their full plant name.
I added this feature mainly for debugging purposes.
It is easily removed if you want to display only want to display the common name, whether the seed is attached or not.

The hover text should be full name, but the label text via toolbar should be only the short name.
Please always write what we talk directly in issues, so that there are no extra round-trips.
See also #1006.
We now explain what short name is in more detail: #1021

https://github.com/ElektraInitiative/PermaplanT/issues/1006#issuecomment-1788974047

jenkins build please

jenkins build please

It doesn't yet seem to be fixed in the sidebar, it still doesn't show the additional name for me?

Sidebar is now also fixed.


Do you have a grid component in pipeline?

I like vue-material but I need a grid component to show tables. Do plan to implement something like this?

Yes. The md-table is a must have. But I have other components prioritized, like menus, cards, progress, toasts and dialogs. Then I will develop the data table component. I think that this will fit for the 0.5.0 release.
Thank you.

The component will be developed in the next minor version (v0.3.0). You can track the progress on the following issue: #32


Use new Jira users/search endpoint where possible

I approved before noticing what @glenn-barnett pointed out. I think that while True should be removed.


possible mix-use of str and unicode in python 2.7

There are multiple occurrence of string.format method. And when the method is called with unicode object, UnicodeEncodeError would be thrown.
Traceback (most recent call last):
File "resttest.py", line 918, in
command_line_run(sys.argv[1:])
File "resttest.py", line 914, in command_line_run
main(args)
File "resttest.py", line 856, in main
failures = run_testsets(tests)
File "resttest.py", line 657, in run_testsets
result = run_test(test, test_config=myconfig, context=context, curl_handle=curl_handle)
File "resttest.py", line 412, in run_test
body=body, headers=head, context=my_context)
File "/Users/baidu/workspace/git/pyresttest/pyresttest/validators.py", line 386, in validate
self.comparator_name, extracted_val, expected_val)
UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-1: ordinal not in range(128)
Sample cases:

test:

group: "ic-hui"
name: "facade/relatedCategory/get"
method: "POST"
headers: {'Content-Type': 'application/json'}
body: '{"kw":"液晶电视"}'
url: {template: "/facade/relatedCategory/get"}
expected_status: [200]
validators:

compare: {jsonpath_mini: "status", comparator: "eq", expected: 0}
compare: {jsonpath_mini: "data.result.0", comparator: "contains", expected: "电视"}

sample response:
{"status":0,"msg":"","data":{"result":[{"id":100972,"title":"PHILIPS 飞利浦 58PFL3740/T3 全高清LED液晶电视（黑色） 58英寸","subTitle":"","shortReason":"逐点高清技术，多接口多格式支持，性价比高。","price":"4099.0","priceHighlight":"历史最低","categoryId":1000569,"categoryName":"LED电视","merchantId":1,"merchantName":"京东商城","rejectReason":"","auditStatus":5}]}}

So I have a solution for this encode/decode error.
Edit your validators.py as below:
def validate(self, body=None, headers=None, context=None):

    reload(sys)
    sys.setdefaultencoding('utf-8')

    try:
        extracted_val = self.extractor.extract(
            body=body, headers=headers, context=context)

so
    reload(sys)
    sys.setdefaultencoding('utf-8')

is the key for output displayed correctly.
Reference: http://wangye.org/blog/archives/629/


Update links

Fixes from #170
Changes

Updated website link
Updated privacy policy link
Updated Twitter link
Updated support email
Updated readme by removing references to iOS

Closing due to wrongfull termination from the Foundation.

Closing due to wrongfull termination from the Foundation.

Closing due to wrongfull termination from the Foundation.

Closing due to wrongfull termination from the Foundation.

Closing due to wrongfull termination from the Foundation.


UNAUTHENTICATED(16): HTTP status code 401

I'm seeing the same thing as this

Does this happen always or ocasionally? Is this running with Cloud Run or Cloud Functions, or something else entirely?

For me at least it's always, I can't get cloud tasks to run at all using this package. The tasks are created fine but when they run they get the above error. My end point application is App Engine.

Thanks, I had not tried it with App Engine. Just fired up an example Laravel app with App Engine using the official tutorial, but there it's also working for me (tried with both a custom domain and without), so I'm not sure what causes this. Perhaps something with IAM permissions? Is there more detailed logging available? If you have more info to share that could possibly help, that would be much appreciated.

Thank you will close this for now


New release for eo-0.41.1

New release for eo-0.41.1

PR-Codex overview
This PR updates the version of the eo package from 0.41.0 to 0.41.1, reflecting changes across multiple files. It primarily involves adding tests and updating package metadata.
Detailed summary

Updated version from 0.41.0 to 0.41.1 in various files.
Added tests section in multiple test files under tests/org/eolang/.
Introduced new test cases for various functionalities.
Updated metadata in object files to reflect new version.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor release, tag is 0.41.1

@rultor release, tag is 0.41.1

@yegor256 OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.41.1

@yegor256 Done! FYI, the full log is here (took me 5min).


Require CSR encoding in X (Matrix Layers)

Context
Per conversation on the September 25 DP Call, @nayib-jose-gloria proposed:

CSC matrices–only found 2 in the corpus, and they are memory inefficient compared to CSR/COO. How do we feel about converting existing cases and codifying in the schema that we won’t take CSC? Or putting a size limit on CSC submissions?
“Backed” mode is prohibitively slow for validating CSC, requiring us to read into memory; we allocate processing resources relative to the worst case scenario, and we are quite close to making backed reads/writes happen in constant memory for CSR/COO matrices.

Also see continuing conversation in  cell-sci-platform.

I would think to also extend this to .raw.X, when present.

@jahilton @brianraymor My understanding is that for existing datasets with CSC, they should be converted (or in the process of converting) to CSR. For any new datasets the validator sees with CSC, the expectation is that the validator should raise a failure message rather than converting to CSR.
It also seems reasonable to me that this requirement should extend to .raw.X

Slight refinements but otherwise, agree:
My understanding is that for existing datasets with CSC, they should must be converted by curators ... And @jahilton is addressing that per earlier comments.
the expectation is that the validator should must raise a failure message rather than converting to CSR.
I will be updating the 5.3 draft soonish.

@brianraymor I think for layers with low sparsity we'll still want to support dense matrices as well; we just want to limit the options for sparse matrices to encode. That said, I can follow-up and see if our initial review of the corpus covered how many dense X/raw.X matrices actually exist.

@nayib-jose-gloria - would you review again? I've incorporated Jason's suggestion for raw.X and your feedback.

LGTM! Thanks! @brianraymor


OP-17098: serialisation issues are fixed.

https://devopsmx.atlassian.net/browse/OP-17098

This PR  triggered this build in this execution, please wait for it  and child pipelines to succeed before merging:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01GDADGZDWYFFHWDWH3AV58JYS

This PR  triggered this build in this execution, please merge to v4.0 branch only if this succeeds:  https://isd.prod.opsmx.net/deck/#/applications/isd40/executions/details/01GDADHHXCANPKKGKMZ8B8X7GK


Add Hit Sound Support

Really should have hitsound support as well.

would be interested in working on this! @abstrakt8 what's involved for this change?

would be interested in working on this! @abstrakt8 what's involved for this change?

First I want to integrate something like the BASS audio library (https://www.un4seen.com/) before working on the hitsounds.

hypothetically for those changes, would you want a pr basing off of develop
or restructure?
On Mon, 6 Jun 2022, 12:26 am abstrakt, @.***> wrote:

would be interested in working on this! @abstrakt8
https://github.com/abstrakt8 what's involved for this change?
First I want to integrate something like the BASS audio library (
https://www.un4seen.com/) before working on the hitsounds.
—
Reply to this email directly, view it on GitHub
https://github.com/abstrakt8/rewind/issues/25#issuecomment-1146843206,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACALSJ7RXEQ5RVT4BHADDCDVNTILPANCNFSM5HCNIBHQ
.
You are receiving this because you commented.Message ID:
@.***>

On develop (but after I merged my 100+ file changes restructure branch)

Any updates here? Would love this feature


Allow to change font/stroke colors of cells

Fixes #31 + α

Changes Summary

Allow to set/reset font color
Allow to reset background color of cell
Allow to set/reset stroke color of cell (#31)
Allow to set/reset border width of stroke color (from 0.1 px to 3.0 px)
Adjust layout using Gtk.StackSwitcher like the mockup

I think now it's possible to edit the style of the cell selected by default.


How to change the theme of the picker in iOS

How to change the theme of the picker in iOS
Hello! I'm currently using the date-time picker for iOS and if my simulator or device is in dark mode I can't see the text.
I tried using isDarkModeEnabled...it works in dark mode but if I change it back to light it's not working in light mode
I also tried using themeVariant but it still didn't work if you know how to fix this please let me know...Thanks!
(btw I'm having this issue only in spinner mode)

Under the hood react-native-modal-datetime-picker uses @react-native-community/datetimepicker. Did you already try swapping react-native-datetime-picker with @react-native-community/datetimepicker to see if the issue persists?

@mmazzarolo I tried changing packages an it worked Thank you so much!


fix#7 - rm GCM dependency

Hi @ludo,
As suggested in issue#7, this PR is to remove the Google Cloud Messaging dependency from the Android project.
Olivier

thanks @ludo !
It would be great if you were pushing a new version to npm.


added link validation in request

close #10
Added a function which checks if the link submitted has 'https://' at the 0th index on submission before it continues with the request. If the check fails the request returns ans an error message is displayed under the link input box.

Awesome! Thank you very much, merged and closing.


How did you get the params lidar_to_cam?

params_camera.yaml:
lidar_to_cam_tx: 0.05
lidar_to_cam_ty: -0.07
lidar_to_cam_tz: -0.07
lidar_to_cam_rx: 0.0
lidar_to_cam_ry: 0.0
lidar_to_cam_rz: -0.04
The lidar's coordinate system(z) is up, and the camera's coordinate system(z) is forward.
But in the yaml, rpy is 0 0 -0.04,
It's almost the same coordinate system.

@hitbg-yjm I have the same question, have you figure it out?

I don't think it's right, because the parameters are used in the point cloud conversion. Maybe the camera coordinate system is different.

@hitbg-yjm I have the same question, have you figure it out?

I don't think it's right, because the parameters are used in the point cloud conversion. Maybe the camera coordinate system is different.

yes, I read the code, and it seems that this transformation is only used as an offset (transform point to the center of imu)


Increase timeout

When computer is overloaded and has limited memory, it is easy to get timeout issues:
devnet:default>start
[Success] Starting node from directory : /clusters/default/node-spo1
[Success] Started submit api : http://localhost:8090
[Info] Swagger Url to interact with the cluster's node : http://localhost:10000/swagger-ui.html
Let's wait for the next block to make sure server started properly !!!
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
Waiting for next block...
[ERROR] Waited too long. Something is wrong. You may want to recreate the cluster or just reset data with 'reset' option ...
[OK] Server Started
devnet:default>tip
[Block#] 86
[Slot#] 125
[Block Hash] 14424d714e8553c9491297ea0f70d0ed9dbe28cf60533647008266c002120df7

perhaps increase timeout?

Thanks @matiwinnetou
If you face a similar issue again, could you please try checking if the chain is moving? Try tip more than once and check the block number.
This issue usually occurs when the chain gets stuck due to various reasons. One of the main reasons is downtime. For example, if the devnet is down for 1 minute, you need to reset it.
It's because of the low security parameter (k) value, which is currently set to 12. Don't ask me why :)
We are planning to set it to a higher default value and make it customizable, which will avoid this issue. Most probably, this will be part of the next release.


adding basic CI

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

In the file dataloader.py:

Line 55:80: E501 line too long (89 > 79 characters)

In the file mnist_bags_loader.py:

Line 49:80: E501 line too long (93 > 79 characters)
Line 101:80: E501 line too long (93 > 79 characters)


feat: SYNC_EVENT_KINDS config option

close #41

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.008%) to 95.547%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
901

Relevant Lines:
948

💛  - Coveralls


Missing cpu/memory request/limits from csi-init container

dynatrace-oneagent-csi-driver is missing cpu/memory request/limits for the cs-init init container
spec:
      initContainers:
      - name: csi-init
        image: docker.io/dynatrace/dynatrace-operator:v0.13.0
        imagePullPolicy: Always
        args:
        - csi-init
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        securityContext:
          allowPrivilegeEscalation: false
          privileged: false
          readOnlyRootFilesystem: true
          runAsNonRoot: false
          runAsUser: 0
          seLinuxOptions:
            level: s0
          seccompProfile:
            type: RuntimeDefault
        volumeMounts:
        - mountPath: /data
          name: data-dir
      containers:

To Reproduce
Steps to reproduce the behavior:

Deploy the helm chart

Expected behavior
cpu/memory request/limits should be defined for all the containers

Thank you for pointing out, indeed they should be defined for all containers.
Will look into it.

Hi, with operator v0.14.0, which should be released soon, the csi driver init-container has default limits/requests and they are configurable via the values file for helm. (see related PRs)
Thanks for pointing the issue out.


Add job to rebuild ES from specific point in time

Draft since not yet managed to test this thing locally

I'll add a comment to the file since I'm nut sure I can think of the best way to quickly validate this.
In any case in the event you use an unacceptable format the error will read like:
Wikimedia\Timestamp\TimestampException from line 240 of /var/www/html/w/vendor/wikimedia/timestamp/src/ConvertibleTimestamp.php: Wikimedia\Timestamp\ConvertibleTimestamp::setTimestamp: Invalid timestamp - 2023-03-30T11:45:46Zcat
#0 /var/www/html/w/vendor/wikimedia/timestamp/src/ConvertibleTimestamp.php(165): Wikimedia\Timestamp\ConvertibleTimestamp->setTimestamp('2023-03-30T11:4...')
#1 /var/www/html/w/extensions/CirrusSearch/maintenance/ForceSearchIndex.php(158): Wikimedia\Timestamp\ConvertibleTimestamp->__construct('2023-03-30T11:4...')
#2 /var/www/html/w/maintenance/doMaintenance.php(118): CirrusSearch\Maintenance\ForceSearchIndex->execute()
#3 /var/www/html/w/extensions/CirrusSearch/maintenance/ForceSearchIndex.php(686): require_once('/var/www/html/w...')
#4 {main}

The acceptable formats are actually more vague than specified in the maint script docs though since internally (as you see in the stack trace) it uses ConvertibleTimestamp. This means for example non-UTC timestamps can be used

I stole my assumptions about the format from here: https://github.com/wikimedia/mediawiki-extensions-CirrusSearch/blob/fc438bf13255981f87a14b862ef0dd68647b4604/maintenance/ForceSearchIndex.php#L83 - is that not the correct place to look?

that is indeed where to look; however the comment there is "actually" wrong. i.e. dates other than that format will also work


Use Travis CI

Add Travis CI configuration to the project to run tests on different Laravel/PHP versions automatically.

I did everything you requested


Test ignore in the annotation

Test the skipping of processing values within annotations.

Codecov Report

Merging #10 into master will increase coverage by 3.37%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #10      +/-   ##
============================================
+ Coverage     88.51%   91.89%   +3.37%
- Complexity       51       54       +3
============================================
  Files             6        6
  Lines           148      148
  Branches         16       16
============================================
+ Hits            131      136       +5
+ Misses            9        7       -2
+ Partials          8        5       -3

Impacted Files
Coverage Δ
Complexity Δ

...etterstrings/ast/InnerStringVarsAstTranslator.java
86.00% <0.00%> (+4.00%)
26.00% <0.00%> (+1.00%)

...ntkorwin/betterstrings/BetterStringsProcessor.java
100.00% <0.00%> (+13.04%)
11.00% <0.00%> (+2.00%)

Continue to review full report at Codecov.

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


Problem mentioned in Issue #144 solved. Always Pull most up to date dev before branching!

I think merge #143 solved the problem and got dev up to date with all changes made by SoSe2022 Group so far.
Everyone who added features up to this merge please check if your feature is implemented in current dev!
PLEASE MAKE SHURE TO PULL THE MOST UP TO DATE DEV BRANCH BEFORE BRANCHING AND ADDING FEATURE!

This issue was solved and exists only as documentation of the incident which occured. I will close it now for clean up purposes.


[shell] How to open a file using system default app in android?

I would like a button that can save a xlsx file to downloads and then open it using the default app in android.
edit: maybe relate to https://github.com/tauri-apps/plugins-workspace/issues/595?

Ineed it is duplicate of #595


maltego: init at 4.3.0

Description of changes

An open source intelligence and forensics application, enabling to easily gather information about DNS, domains, IP addresses, websites, persons, etc.
https://www.maltego.com/
Related to #81418
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Maltego has been merged on #276997. Closing.


Document Explicit/Implicit operators for System.Numerics.Complex.

Document all Explicit and Implicit operators for System.Numerics.Complex.
Summary
Pulled information from MSDN to document 2 Explicit and 10 Implicit operators for System.Numerics.Complex.
Suggested Reviewers
@rpetrusha

@rpetrusha  Thanks Ron. I changed int intValue; to uint value; and System.Numerics.Complex c1 = intValue; to System.Numerics.Complex c1 = value; as you suggested in your email.

Thanks, @tompratt-AQ. I'll review it first thing tomorrow.

@tompratt-AQ, this looks good. I'll merge now.


try commit

created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request


Add merge function

Is this more or less how you would've implemented this?

@thedufer Merge should always create a discrete signal -- it doesn't make sense to have a continuous merge. When you first create it, what's its value? The value of the last thing in the array, basically ignoring the other three. It makes sense to be able to merge continuous signals, to have a discrete signal-of-when-any-of-them-change, but I don't think it makes sense for that result to itself be continuous.

Okay, I see what you're saying.  I was basing it off combine, but that wasn't a great analogy.

(I'll fix it)


Add model loading possibility to IncQuantizedModel

Should be merged before optimum-intel next release as needed in #100

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Does this work for Intel based Macs?

Hi! Thanks for building this, seems really cool and I'd love to try it out. I haven't put much time into seeing the code and I was just trying to follow the instructions.
The tool itself builds and runs well, but when I try to create a new VM using the make build/vm/ubuntu command, it exits abruptly and I'm not sure if it finished

When starting the host, I can attach to the host, but it fails to start the SSH service. Once it reaches the login screen, I can't see the IP of the machine because the file $VMCTL/ubuntu/0.ipaddr doesn't exist.

Is there any additional configuration that I need to for getting it to run on x86 based machines. I do see the ubuntu.sh script checks for architecture before downloading, and I was hoping I would be able to run this on an Intel mac

I'm running Big Sur 11.4.  If you login into the serial console, and do lspci, does the network adapter show up?


Breaks production build after<EMAIL_ADDRESS>

Related issue: https://github.com/sveltejs/svelte/issues/13510
I'll create a reproduction later

It's working fine from what I tested with the scratch project with the latest Svelte 5 version.
Btw could you share your rendered HTML file?
Maybe some of the attributes are removed by the 'html-minifier-terser`.
You could also disable the config https://github.com/terser/html-minifier-terser#options-quick-reference.

That sounds like you're removing whitespace and/or comments via the minifier, which Svelte is sensitive to. Try turning those parts off.

Originally posted by @dummdidumm in https://github.com/sveltejs/svelte/issues/13510#issuecomment-2395163959

That sounds like you're removing whitespace and/or comments via the minifier, which Svelte is sensitive to. Try turning those parts off.

Originally posted by @dummdidumm in sveltejs/svelte#13510 (comment)

For me it was collapseWhitespace.

It seems many people have this issue, so I decided to remove the collapse whitespace by default.
It will fixed by v1.0.4
https://github.com/ntsd/sveltekit-html-minifier/pull/7


Contents of suggestion when executing configtest may not be correct

If you create a config file like the one below and run configtest, a suggestion will be output to warnings. However, I could not understand the meaning of this content. Is this correct behavior?
apikey = "aaa"

[plugin.metrics.aaa]
command = ["exit 1"]
[plugin.metrics.aaa.bbb] ← Target of warning
command = ["exit 2"]

$ mackerel-agent version
mackerel-agent version 0.73.2 (rev 3fd96af) [darwin amd64 go1.19]

$ mackerel-agent configtest -conf a.conf
[WARNING] plugin.metrics.aaa.bbb is unexpected key.
[WARNING] plugin.metrics.aaa.bbb.command is unexpected key. did you mean plugin.metrics.aaa.bbb.command ?

@ryuichi1208
As you pointed out, this suggestion in the error message is inappropriate. Thank you for letting us know! Your Pull Request will also be reviewed later.

@ryuichi1208 mackerel-agent v0.73.3  including your pull request has been released. Thank you for your contribution!

@wafuwafu13 Can you work on the refactoring for configtest as a separate issue? We close this issue.


Implemented Seq at without full rebuild

Some traversables (Vector) have effectively constant performance guarantees for most operations, avoiding full rebuild helps performance for them.
Moreover, I find this version much simpler and more readable.

Thanks!


Why there is no corresponding .cpp file for dcgm_agent.h?

https://github.com/NVIDIA/DCGM/blob/7e1012302679e4bb7496483b32dcffb56e528c92/dcgmlib/dcgm_agent.h#L1228
Hi,
I am looking for the specific implemetation method for dcgmGetPidInfo(),   but I can not find the corresponding .cpp file for dcgm_agent.h. Why is this?

The dcgm_agent.h is the public API header and is not directly related to the implementation part.
Take a look here and look for the tsapiEngineGetPidInfo in the DcgmApi.cpp file.

Hi @nikkon-dev , in the DcgmApi.cpp file, tsapiEngineGetPidInfo(dcgmHandle_t pDcgmHandle, dcgmGpuGrp_t groupId, dcgmPidInfo_t *pidInfo) ,  pidInfo is what users pass in instead of being obtained from some other sources, right?

Yes, that's what the user passes to the function setting two fields: version (dcgmPidInfo_version2 for now) and pid, which is the PID of a desired process created after the stats gathering was enabled.

Yes, that's what the user passes to the function setting two fields: version (dcgmPidInfo_version2 for now) and pid, which is the PID of a desired process created after the stats gathering was enabled.

Hi @nikkon-dev , how to enable stats gathering? Thanks!


Revert "[SPARK-35028][SQL] ANSI mode: disallow group by aliases"

What changes were proposed in this pull request?

Revert [SPARK-35028][SQL] ANSI mode: disallow group by aliases
Why are the changes needed?

It turns out that many users are using the group by alias feature.  Spark has its precedence rule when alias names conflict with column names in Group by clause: always use the table column. This should be reasonable and acceptable.
Also, external DBMS such as PostgreSQL and MySQL allow grouping by alias, too.
As we are going to announce ANSI mode GA in Spark 3.2, I suggest allowing the group by alias in ANSI mode.
Does this PR introduce any user-facing change?

No, the feature is not released yet.
How was this patch tested?

Unit tests

Test build #142533 has started for PR 33758 at commit 76c697e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47034/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47034/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47034/

Test build #142547 has started for PR 33758 at commit 8e0aec3.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47049/

Test build #142533 has finished for PR 33758 at commit 76c697e.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142533/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47049/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47049/

Merging to master/3.2

Test build #142547 has finished for PR 33758 at commit 8e0aec3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142547/

+1, LGTM.


Don't require the python interpreter in openvdb/python when unit tests are disabled

In trying to build the openvdb python module for vcpkg, I ran into a problem with openvdb/python requiring the python interpreter even when unit tests are disabled. By default, the openvdb vcpkg doesn't build unit tests and so it doesn't have a dependency that provides the python interpreter. Since I didn't want to add an extraneous dependency just for building I just patched it to remove this requirement. However, I think a worthwhile improvement here is to only require the python interpreter when the option OPENVDB_BUILD_UNITTESTS is enabled.
I encountered this while trying to help out issue #603.

I'll be looking as this as part of the some upcoming windows improvements

https://github.com/AcademySoftwareFoundation/openvdb/pull/646

Fixed as of #646

Thanks!


Remove Bytes from the public API

This should remove Bytes from the public API while it is unstable, and adds from_maybe_shared generic constructors that can allow improved performance (less copies) if the correct version of Bytes is used.
Closes #366

cc @LucioFranko @sfackler

Is the idea that the next release of http will be 1.0?

I'm strongly considering it, or otherwise a 0.2 meant as a preview of 1.0 just to check the new changes.

I'm also happy to go 1.0 with http


Simpler API in d3.geo.path().projection to turn off resampling

This seems a bit complicated to do a simple thing... What about accepting a second argument to turn off adaptive resampling?
d3.geo.path().projection(function(c){ return [c[0], -c[1]] }, resampleOrNot)

AFAICS resampling is useless for any linear transformation, yet it produces much longer path strings because of rounding.

To turn off resampling, you can simply use .precision(0).

Thanks, I had missed that.
Actually, the rounding errors were not due to resampling only:
var invertY = d3.geo.projection(function(λ, φ){
  return [λ, φ] //d3.geo.equirectangular()
}).scale(180/Math.PI).translate([0,0]).precision(0)
invertY([.42,0.42])
// [0.42000000000000004, -0.42000000000000004]

So a custom geometry stream is the only way to keep svg paths compact.


View Press Release

A press release, once published, should be viewable in full-screen format to people visiting the instance.

Done, in the current form (our "multi tenant"  version) you have to go to  http://host:3000/

Made it better. Once you hit the landing page (and there is only one company) You are automatically taken to that companies company page.


[AMBARI-24354] [Log Search UI] wrong filter label on open log tab

(cherry picked from commit 6917b7146437f66f00a2d52ff7d0da8c213da3df)
What changes were proposed in this pull request?
The time range interval value for the PAST and the CURRENT time range type is not applicable (it is always 1) so it has been removed from the params when we use those time range params. The params to filter sync has been modified to reflect this change.
How was this patch tested?
It was tested manually and by the unit tests:
PhantomJS 2.1.1 (Mac OS X 0.0.0): Executed 268 of 268 SUCCESS (7.53 secs / 7.455 secs)
✨  Done in 35.53s.

Identical with: https://github.com/apache/ambari/pull/1879
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/3288/
Test PASSed.


chore: remove JS defaults

Overview

Remove JS defaults from the bulk (or all) of components

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: [ABC-123]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut@55.6.2-alpha.b6eddc.0@codecademy/gamut-kit@0.6.348-alpha.b6eddc.0@codecademy/gamut-patterns@0.9.10-alpha.b6eddc.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6512fca06331cd00920518d4--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@55.6.2-alpha.eb46b5.0@codecademy/gamut-kit@0.6.348-alpha.eb46b5.0@codecademy/gamut-patterns@0.9.10-alpha.eb46b5.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://651343a023f2b8130927723c--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://65144665734c08047c53a6f9--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://65144dc064d3a60b37d2a7cb--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://651452b9d997730f7871e62e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@55.7.2-alpha.b78a22.0@codecademy/gamut-kit@0.6.351-alpha.b78a22.0@codecademy/gamut-patterns@0.9.10-alpha.b78a22.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://65148267ac0b9d354f402113--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@55.7.2-alpha.abf2c0.0@codecademy/gamut-kit@0.6.351-alpha.abf2c0.0@codecademy/gamut-patterns@0.9.10-alpha.abf2c0.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://651c62d8213588043bed917e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@55.7.3-alpha.204b96.0@codecademy/gamut-kit@0.6.352-alpha.204b96.0@codecademy/gamut-patterns@0.9.10-alpha.204b96.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://651da2594c6a00296fb55513--gamut-preview.netlify.appDeploy Logs


PythonGenerator with Pydantic  If a property is Optional not required the json expected a key None

Describe the bug
PythonGenerator with Pydantic
How to Reproduce
If a property is Optional not required the json expected a key None
it is possible change the Class Python pydantic generator and add a default value =None for not required properties
from:
optionalkey: Optional[str] = Field( )
to:
optionalkey: Optional[str] = Field( default= None )

Thanks for the issue @tornabene 👍
This should be an easy fix, it's rendered as part of the preset located here: https://github.com/asyncapi/modelina/blob/091f3a815728538da6e498cd0e414ccc28661308/src/generators/python/presets/Pydantic.ts#L28
Happy to review a PR and answer questions to fix this 🙂


web view shows demo todo list

running the web view appears to display the demo todo list instead of the .todos.json file in the current directory. restarts of the web view displays a new demo.

does not occur on mac osx. does occur on ubuntu

@asegner Sorry about not getting back sooner.  I can't seem to replicate this.  Please re-open if you're still having trouble.


daos.fun integration

We want daos.fun integration
This includes a basic service to interact with daos.fun and have a wallet and all that, as well as actions that mirror the swap actions but with the DAO wallet.

We have a daos.fun wallet for the ai16z DAO, we need to control it. Can we do this with a delegate? Or do we need the actual wallet private key? Let's figure it out. Can connect with @baoskee


Packed::::outline: Use provided styles for displaying entry

Fixes #5370.

In principle, it is intentional that we use StyleChain::default() instead of styles. Since outline is only called on queried elements, all the fields are already materialized and the styles won't be used anyway. The styles passed to the outline are only intended for displaying the counter. (All of this is not great, but that's how it's intended anyway.)
We do call elem.push_separator in Synthesize for FigureCaption, so it should also be populated in a language-aware manner, but something seems to be wrong there.

Closing in favor of https://github.com/typst/typst/pull/5550. Thanks still!


Support HotReload?

Thank you for developing this plugin, I like it very much. In practice, I encountered a bad experience: every time I modify the code, I have to go back to XCode to execute Common + R and re-run the simulator. Is there a better way to solve this experience problem, or can it support XCode Canvas Preview hot reload? Although this is indeed a bit difficult, it can greatly improve development efficiency. Thank you.

Have Cursor on the left on top of Xcode with only previews showing; any changes made in Cursor after saving will be reflected in previews.
For easily running on a simulator, use keyboard bindings. Here is an example:

Press Shift + Command + P
It will open the top bar, write: Preferences: Open Keyboard Shorcut (JSON)
In the JSON, add the following for building and running (Command + B and Control + Command + R) and Command + . for stopping/cancelling:

[
  {
    "key": "ctrl+alt+cmd+b",
    "command": "workbench.action.toggleSidebarVisibility"
  },
  {
    "key": "cmd+b",
    "command": "-workbench.action.toggleSidebarVisibility"
  },
  {
    "key": "cmd+b",
    "command": "runCommands",
    "args": {
      "commands": [
        {
          "command": "workbench.action.tasks.terminate",
          "args": "terminateAll"
        },
        {
          "command": "workbench.action.tasks.runTask",
          "args": "sweetpad: build"
        }
      ]
    }
  },
  {
    "key": "ctrl+cmd+r",
    "command": "runCommands",
    "args": {
      "commands": [
        {
          "command": "workbench.action.tasks.terminate",
          "args": "terminateAll"
        },
        {
          "command": "sweetpad.build.launch"
        }
      ]
    }
  },
  {
    "key": "cmd+.",
    "command": "runCommands",
    "args": {
      "commands": [
        {
          "command": "workbench.action.terminal.killAll"
        }
      ]
    },
    "when": "inDebugMode || taskRunning"
  }
]

I hope that will greatly improve development efficiency!

Thank you for your reply. I found this library:   which can hot reload SwiftUI. With your sweetpad, the development experience is greatly improved. Thank you for your work.

All credits for Sweetpad goes to @hyzyla!


Do not hardcode redis version in description

Now that redis 4 and 5 are supported, the 3.0.0 is no longer true.

Coverage remained the same at 85.62% when pulling de260c4fa09cb83754bb258fb99acacb5b5f4efe on jeffwidman:patch-2 into 8fe8b718a3f4b50ae0f4144205dc6d9f504e7ffc on Grokzen:unstable.


Get raw data from AFHTTPSessionManager responseObject

I just upgraded to AFNetworking 2.~ and realized that instead of NSData, I am getting a JSON dictionary. I do not want this behavior. Is there any way to globally prevent this serialization?

By default, the response serializer is set to AFJSONResponseSerializer.  Take a look at the migration guide, and read up on how response serializers work. Just create a new one or use one of the other built in response serializers.
Also, in the future please use StackOverflow for these types of questions. Github should be used for issue reporting and feature discussion.
:beers:

Thanks, @kcharwood!


[BUG] Documentation links in Dashboards security plugin don't deep link to the appropriate places

What is the bug?
Throughout the security plugin in OpenSearch Dashboards external links to the OpenSearch documentation site only link to the homepage and do not deep link to the appropriate sections. This is a poor experience for users and asks them to navigate the docs site to find the appropriate section that will help them out.
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to the Security Plugin in Dashboards and navigate to the Get Started section.
Click on the "Create config.yml" CTA
You are directed to the homepage of the docs site instead of the appropriate section under the security plugin

What is the expected behavior?
When a user clicks an external CTA in the security plugin meant to point them to the doc site it should deep link them to the appropriate documentation page not the homepage.
What is your host/environment?

OS: MacOS (Chrome)
Version: 2.3.0
Plugins: Security

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
This seems to be an issue with all the links to documentation within the security plugin. It looks like they are being redirected to the homepage because we changed the structure of the docs site but I could be wrong.

[Triage] Thanks for filing the issue @shanilpa ! This would make for a great first contribution.

Hi,
This issue was tagged for 2.6.0.
Can you close it if it was resolved or tag is to be tracked for next iteration?
Thanks!

Closing this issue. This was released in 2.6.0.


[BUG] Variables don't work in Body, values not extracted.

Variable templates don't expand into values in body, like this JSON:
{
"access_token": "{{token}}",
"clientVersion": "<IP_ADDRESS>"
}
The {{token}} variable exists, and works if placed in URL segment.
v 0.16.1

Windows 11, Update 2H2.
It works if I change body type from JSON to TEXT though.

It also works if I set body type to JSON and remove the Content-Type header.
But if I keep Content-Type = "application/json" it stops working

@Rzpeg Thanks for getting to the root of this. I have the fix ready. Will deploy a new version in 30min.

Fix has been released in v0.16.2
You can download and install the latest version.
Thanks again @Rzpeg !

Works perfectly! Thanks!
P.S. I was wondering why you had no XPath implemented for XML response, but then I found out I can install and use external npm packages, so I wrote the script to parse XPath. Just leaving it here for the case if you want to implement XPath natively, as this might be the feature that the Insomnia refugees will miss here:
const xpath = require('xpath');
const dom = require('xmldom').DOMParser;

const doc = new dom().parseFromString(res.body, 'text/xml');
const query = "//*[local-name()='AuthToken']/text()";
const token = xpath.select(query, doc)[0].nodeValue;

bru.setVar("token", token);

I found out I can install and use external npm packages

Isn't that beautiful?
That you can use any package of your choice at a collection level. Something that has been missing for an entire decade in the GUI API Clients ecosystem.

I was wondering why you had no XPath implemented for XML response

Currently Bruno supports these inbuilt libraries - https://docs.usebruno.com/scripting/inbuilt-libraries.html
I agree that xpath and xmldom are great package candidates to support as an inbuilt library.

Isn't that beautiful?
That you can use any package of your choice at a collection level. Something that has been missing for an entire decade in the GUI API Clients ecosystem.

Yep, that's a lot of possibilities! That is what allowed me to implement xpath :) And this is how I am planning to implement file upload, using fs-extra or something similar .
This also could also eventually opens ways for people to write 3rd party plugins, and export functions to be consumed in bru 👍

Cool. fs-extra will not work in current build.
File access is disabled for security reason.
I will release a patch for today that will allow to turn off this restriction at collection level
Something like this in bruno.json
{
  ""allowScriptFilesystemAccess" : true
}

Cool. fs-extra will not work in current build. File access is disabled for security reason.
I will release a patch later today that will allow to turn off this restriction at collection level
Something like this in bruno.json
{
  ""allowScriptFilesystemAccess" : true
}

Hey! Just wondering if the patch mentioned is included in the latest version? (v0.16.6)

@Rzpeg Can you share the way you solved your xpath and xmldom issue in Scriptmania- https://github.com/usebruno/bruno/discussions/385
It would benefit the community.

Hello guys, I have same problem here. Here is my json body:

This is what is created in database:

and here are mine env vars:

Do you have any idea why my var "login" doesnt want to go green like other vars that are working?
I tried also with regular var (collection var) and it resulted same:

Also here is my pre request script:
bru.setEnvVar("login", Math.round(Math.random() * 100000));
Bruno version 0.25.0

Hello,
I noticed a bug related to that.
I have a variable named "PublishedOn" which is an array of string

However, I get a parse error when putting it into a JSON body, like so:

The quick fix that I found was to "prettify" the JSON. It still shows as an error, yet still manages to send the request with the correct variable (an array of string):

Working for me. Which OS? Would you be able to create and share a sample collection ?
On my mac I see this parsing working Parsing body
My Env

What about if it's in the request vars?


Add import completion

I wish there was import completion like in LiteIDE.
I.e. you typed:
import "fm

and it offers fmt package to import.

This more of an issue with gocode. Also vim-go has the :GoImport command which offers autocomplete.

@nhooyr
LiteIDE has this feature without gocode, so I guess one can look at the implementation in it.
:GoImport isn't really useful when you don't know the head part of package's path.

@dobegor https://github.com/nsf/gocode/issues/257
It would make much more sense to let gocode handle it than do it ourselves.

@dobegor what do you mean it isn't useful? it works perfectly fine for me.

@dobegor @nhooyr Hi,
I was also planned. but we have goimport, so was thinking whether we need.
However, see https://github.com/zchee/deoplete-go/pull/19
If has been merged, the name of the Go stdlib can be retrieved.
and https://github.com/zchee/deoplete-jedi/blob/master/rplugin/python3/deoplete/sources/jedi.py#L41-L44
It is possible to display the name of the Go stdlib package if current input is import.
I think maybe performance reduced, but will not significantly affect because statically cache.
WDYT?

@dobegor @nhooyr or, If current input is import , call it.
https://gist.github.com/zchee/bdfe8b1967d3f958caee
It 's fmt, os, net, syscatll listed.
===
sorry.
I woke up earlier, so English is rough

@dobegor @nhooyr
I will explain in detail later...

@zchee looks great, good morning then :)
waiting for details on that feature.

@dobegor At first, I wrote the details of the flow.
Check it.
https://github.com/zchee/deoplete-go/pull/19#issue-131914074

At second, that is sloppy but this content.
    def gather_candidates(self, context):
        buf = self.vim.current.buffer
        pkgs = self.GetCurrentImportPackages(buf)
        parent = str(context['input']).strip('\t')
        pkg_data = os.path.join(self.data_directory, parent + 'json')

        if re.match(r'^\s*import \w*', parent):
            with open(os.path.join(self.data_directory,
                                   'packages.json')) as json_pkg_data:
                result = loads(json_pkg_data.read())

        elif parent not in pkgs and '.' in parent and os.path.isfile(pkg_data):
            with open(pkg_data) as json_pkg_data:
                result = loads(json_pkg_data.read())

        else:
            line = self.vim.current.window.cursor[0]
            column = context['complete_position']

            offset = self.vim.call('line2byte', line) + \
                charpos2bytepos(self.vim, context['input'][: column],
                                column) - 1
            source = '\n'.join(buf).encode()

            .
            .
            .

@dobegor May I ask if you understand what I mean?

@zchee i really think we shouldn't bother, much rather let gocode. Its not really a urgent feature, goimports works perfectly fine. Its autocomplete is nice. If you really want to, go ahead, there is no real harm. It might add some needless complexity though.

@nhooyr Yes.
It's possible. but it is also complicated.
I think it is the work of goimports. so I do not like the implements of this.
@dobegor WDYT?

@zchee so this will only allow to auto-complete a fixed set of import paths?
I guess we need to make a way to generate package.json for all packages in $GOPATH, for example, (imports-cache) command. It will parse $GOPATH and generate import paths for each package, then autocompletion will perform fast fuzzy matching along this list.
I have literally 2 days experience with Vim, but I can certainly write such utility in Golang (using third-party lib for fast fuzzy matching).

@dobegor why don't you just modify gocode to do this. That way every single editor that uses gocode will be able to take advantage of that functionality without changing anything.

@dobegor

I guess we need to make a way to generate package.json for all packages in $GOPATH, for example, (imports-cache) command. It will parse $GOPATH and generate import paths for each package, then autocompletion will perform fast fuzzy matching along this list.
I have literally 2 days experience with Vim, but I can certainly write such utility in Golang (using third-party lib for fast fuzzy matching).

Now, gocode are looking for collaborators https://github.com/nsf/gocode/issues/307.
If you said implements are realized, and faster than gocode, We will consider adoption it.
but at present, bad idea not to use the gocode.

I'm sorry if my understanding is wrong.

@zchee I'm gonna take a stab at implementing it in gocode. I'll update once I have it done.

@nhooyr I looking forward update.

@zchee nsf/gocode#332

@nhooyr Thanks!
I'll check that PR later.

Once its merged (likely tomorrow) we just need to update our regex for completion in strings, as well as add a new class import. Thats it.

@nhooyr I got it.

Its been merged!

@nhooyr Yeah!!!!
I'll update and test :)

we just need to update our regex for completion in strings, as well as add a new class import.

I would like to confirm, but it meaning is input_pattern?

yes.

@nhooyr OK.
I will update the input pattern corresponding to it. (Maybe tommorow..?)

@nhooyr I test import completion feature.
Not required modifying input_pattern. It worked perfectly :)
I small fixed for when align_class is enabled.

https://github.com/zchee/deoplete-go/commit/45d5c7ba70854ce6ec1a4e77baa1c8df75504b29

@dobegor This feature has been the implementation.
Close :)

@zchee https://asciinema.org/a/3xz29k70w2zmi27sg2zugde74
just a few slight problems.

@nhooyr Oh, Good catch. Thanks.
Actually, it seemed necessary fixes for self.input_pattern :(
(It is very difficult to understand (?:\b[^\W\d]\w*|[\]\)])\.(?:[^\W\d]\w*)? regex... :scream:)

haha. take a look at the previous PRs, its explained there
https://github.com/zchee/deoplete-go/pull/35
https://github.com/zchee/deoplete-go/pull/36

@nhooyr Yea.
When to implementing this feature, because I thought it was necessary to change the input_pattern, I had to try it. About 30 minutes...
However, was not able to completely understand :joy:
I will try again from now... (I do not know how long it takes. lol)

@nhooyr
Off topic: Could you tell me your twitter ID?
My TW: https://twitter.com/zchee

@zchee its the same, nhooyr. I'll change it if you want, I think I know whats wrong.


Increase test timeout duration for internal CI

gRPC master is currently timing out internally.

Known issues:
Performance - #9609 #9668

jenkins test this please


Ds/migrate to npm registry

Remove registry override and bump version to 0.0.10

Do we also need to update the publish workflow so the new packages are published to npmjs and not gh?

the default is npmjs
metrics-registry = "https://registry.npmjs.org/"
omit-lockfile-registry-resolved = false
**registry = "https://registry.npmjs.org/"**
replace-registry-host = "npmjs" ```


Builders compare new results to stored results from previous runs

There needs to be tools that compare results.json logs, taking flakiness into account. The bots needs to use these tools.

This works now.


Merge Dev ke UAT 15 July 2017

Coverage increased (+0.2%) to 66.046% when pulling b7a25b7c616bd30658eb7b7aa13e6a4c68f6eb10 on dev into ceb22824fb580c4ca2389d5a3a06492767c018eb on uat.


add toleration only for masters key "feature/kubespray-aurea-2.3.0" NoSchedule

Currently, the template sets toleration for NoSchedule with any key and value. This breaks users "NoSchedule" taints, as kubedns pods are scheduled there. This patch make kubedns possible to run only on key: "feature/kubespray-aurea-2.3.0" and effect: "NoSchedule" (like dashboard).

If this still is an issue, please open a new PR


MacOS Opening native tab toggles fullscreen

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
In MacOS when the "Prefer tabs when opening documents" is set to "Always", File->Open on a non-fullscreen session will toggle the fullscreen if atom remembers the new project as being fullscreen. This is really annoying when I open new tabs and get jerked out of my element into fullscreen. Atom should not toggle fullscreen when opening a new tab, only a new window.
Steps to Reproduce
First we set up the scenario:

Navigate to Dock in the macOS System Preferences
Change Prefer tabs when opening documents: to Always
Open up atom to a directory
File->Open another directory (You should now have 2 native tabs in the same window)
Enter the macOS native fullscreen mode
Close one of the native tabs [Taking note of that path] (So only one project is left open now)
Exit native fullscreen
Quit Atom

Now we can reproduce the bug:

Open up atom to the directory that was last open (Atom should open and NOT enter fullscreen)
Now File->Open the path you took note of in step 6

Expected behavior: [What you expect to happen]
The new native tab should open and the app should not enter native fullscreen.
Actual behavior: [What actually happens]
The app enters native fullscreen.
Reproduces how often: [What percentage of the time does it reproduce?]
100% of the time
Versions
Atom: 1.17.2
MacOS: 10.12.6

@seaside98 I don't even see the preferences option Prefer tabs when opening documents in atom 1.20.0... 😬 I really wanted it after I saw your issue!  I can't figure out how to get native tabs at all in this version...

@colllin It's in the macOS System Preferences app, not in atom. Open up the System Preferences and navigate to the Dock section to change the setting.

Ohhh.  Thanks for the clarification.  Either way, unfortunately, this capability seems to be blocked in Atom 1.20.0.  I’m not sure if this was an Atom decision, an Electron decision, or some combination.  If there’s any way to bring back native tabs, I would be happy to contribute a PR.
On Sep 18, 2017, 6:32 PM -0600, Addison<EMAIL_ADDRESS>wrote:

@colllin It's in the macOS System Preferences app, not in atom. Open up the System Preferences and navigate to the Dock section to change the setting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@seaside98 can you check with Atom 1.21-beta?
@colllin Native tabs have been re-enabled in Atom 1.21-beta.

@50Wliu I just tested with Atom 1.21-beta and the bug is still there.


Tunnel TCP connection (so we can expose SSH)

I'd like to know if it's possible to use Tunnelmole to connect to SSH. If not, I'd like to suggest this feature.
If we tunnel our port 22, we get both the http and https tunnelmole links
If we try to connect, such as with:
ssh user@<tunnelmole url>
We get this error:
user@<tunnelmole url>: Permission denied (publickey)

Nope, Tunnelmole only supports HTTP/HTTPs. Other protocols i.e. SSH are not supported. You might want to look into SSH  port forwarding instead: https://phoenixnap.com/kb/ssh-port-forwarding


Add migration to republish Attachable

This migration republishes all documents that are Attachable and have an HtmlAttachment.
This will cause the HtmlAttachments to be republished too allowing us to get an up to date view of any remaining issues with the migration to Publishing API before switching them over completely.
Republishing via the Attachable Edition's Document allows us to exercise the full republishing worker setup and avoids having to navigate the complicated state of HtmlAttachment within the migration.

:+1: Fire when ready


Baseline 15.0.995.28 missing

Hi guys
I've tried to run the script "CompareExchangeHashes.ps1" and I get an error, that baseline_15.0.995.28 is not found. In the folder with the extracted baselines, i have the folder 15.0.995.29, but noch the folder 15.0.995.28. Here's my powershell output:
[03/15/2021 12:55:59] Started...
Found exchange version: 15.0.1497.2
Can't find local baseline for 15.0.1178.4
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.1178.4.zip
Loaded baseline for 15.0.1178.4, hashes number 7452
Can't find local baseline for 15.0.1076.9
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.1076.9.zip
Loaded baseline for 15.0.1076.9, hashes number 7353
Can't find local baseline for 15.0.1497.12
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.1497.12.zip
Loaded baseline for 15.0.1497.12, hashes number 7464
Can't find local baseline for 15.0.1473.3
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.1473.3.zip
Loaded baseline for 15.0.1473.3, hashes number 7394
Can't find local baseline for 15.0.1497.8
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.1497.8.zip
Loaded baseline for 15.0.1497.8, hashes number 7464
Can't find local baseline for 15.0.1320.4
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.1320.4.zip
Loaded baseline for 15.0.1320.4, hashes number 7463
Can't find local baseline for 15.0.995.29
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.995.29.zip
Loaded baseline for 15.0.995.29, hashes number 11830
Can't find local baseline for 15.0.1497.0
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.1497.0.zip
Loaded baseline for 15.0.1497.0, hashes number 7464
Can't find local baseline for 15.0.995.28
Downloading baseline file from GitHub to C:\temp\hafnium_test_Script_Logs\baseline_15.0.995.28.zip
LoadFromGitHub : C:\temp\hafnium_test_Script_Logs\baseline_15.0.995.28.zip not found... please open issue on
https://github.com/microsoft/CSS-Exchange/issues, we will work on it
In C:\temp\hafnium_test_Script_Logs\CompareExchangeHashes.ps1:660 Zeichen:13

        LoadFromGitHub -url $zip_file_url -filename $zip_file -installed_ver ...

  + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
  + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,LoadFromGitHub

I think it's a typo somewhere in the script. Can you upload a fixed script? Thank you

Duplicate of #313 . This one is already on the list of missing baselines.


pythonPackages.pandas: fix evaluation

Motivation for this change
It seems as running the pythonPackages.pandas testsuite fails on NixOS.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

We won't just disable the whole test suite. Acceptable solutions are:

fixing the test suite. Likely upstream has done so.
disabling the specific tests.

fair point. Will have a deeper look at the test suite in the next time :-)

see https://github.com/NixOS/nixpkgs/pull/30956


Deviance residuals versus RMSE for example data

Currently, we're evaluating model fit for the example data using out-of-sample RMSE. As pointed out by the reviewers, RMSE seems to indicate that a model with just an intercept for each species seems to fit both the zooplankton community data and the Daphnia data set almost as well as all the models with seasonal effects, even through it's pretty apparent from the plots that the seasonal effect is important. As the reviewers had pointed out, that's likely because RMSE isn't that appropriate for models with a large mean-variance relationship like our data.
I had noticed this earlier before we submitted, and had wrestled with what to do with it. As an alternative, it makes sense to use deviance of the out-of-sample data. The down-side with that is it doesn't give the results in an understandable form (RMSE is at least interpretable). I was wondering what metric everyone thinks might be best here:

Deviance of all out-of-sample data for each model. Upside is this is connected to AIC, but it doesn't give any measure of absolute goodness of fit, and it can be hard to interpret for non-statisticians (or even for statisticians).
Deviance explained for out-of-sample data. This at least gives a sense of how well the model is working, but @dill, I know you're not wild about $R^2$-style metrics
RMSE of log-transformed data. This at least avoids the quadratically increasing variance issue, and might be a bit more interpretable (in that you can describe the effects as the number of orders of magnitude of variation around the mean), but I'm not convinced that it's that interpretable, and it is more ad-hoc.

Thoughts?

I'd not thought about out-of-sample deviance before, but I'm not against it if we can explain it well enough for readers to know what it is. If we do go ahead and use it I think it's a useful addition in itself to let folks know that RMSE is not always the right measure.
@eric-pedersen how big a task do you think it would be to try out 1. above?

It's pretty straight-forward. I figured out how to do it a couple months ago when playing around, then promptly lost the code (:anguished:). I can recreate it quickly, though.

@eric-pedersen Is using the metric the reviewer used to illustrate the deficiency in our use of RMSE not applicable or useful?

@gavinsimpson , sorry, didn't see this until just now. I think that could work, but how do we calculate that in mgcv?

get_mae <- function(y_pred, y_obs){
  stopifnot(length(y_obs)==length(y_pred))
  sum(abs(y_pred-y_obs))/length(y_pred)
}

?

I think this will likely work better than RMSE, but will still have issues with the variance-mean problem. I think figure 16 really highlights where the problem is coming from; both RMSE and MAE i all lakes are driven by the large variation around the high average densities in mid-spring and late fall; All three models actually capture this increase pretty well, but because of the nature of the year-to-year variability, all of the error is still coming from the large and small variations around this peak. As such, it looks like the models aren't doing that great predicatively compared to models with a simple lake-by-lake mean, even though they capture the dynamics of the peaks really well; the large points mid-spring are actually a sign of good model fit, as that's where the seasonal dynamics models are forecasting that there should be large observations.

I'd argue that, for this data, the natural scale to evaluate the error is actually the log-scale; a model is doing well if it's capturing the order-of-magnitude changes in population dynamics. The deviance for the Gamma distribution actually captures that concept pretty well: it's equal to the sum of -2*(ln(y/mu) - (y-mu)/mu), where y is the observed value and mu is the prediction.

I like this. Can you post both versions of tables 3 and 4 for the MAE and deviance? I'm intrigued to know if what the comparison is here...

Here's the table for MAE for Daphnia (ignore that it says deviance... the calculation is MAE):

And here's the same table for total deviance:

hmmm, not sure if it's just a feature of the rounding in the MAE table, but what's happening in Waubesa?

oh, is this a feature of that big outlier maybe?

And as a third comparison: here's MAE calculated on log-transformed ratios of fitted to observed values. I think this has the cleanest interpretation, in a lot of ways. Looking at say, Mendota, it can be interpreted as:
For the intercept-only model, observations are on average 5. 4 times larger or smaller than the fitted value. For models 1-3, the average observation is 3.8-3.9 times larger or smaller than the fitted value, which I'd count as a pretty substantial improvement (again, ignore the text, and this metric definitely does not have units of indivs per m^2...).  It also shows that the models fit for Waubesa does a much poorer job.

The more I think about this, the more I like it. It's interpretable, and it's in some sense scale free; in the case of untransformed variables, all models look like they fit better to low-density species, just because these species happen to be not as common (as both their mean and variance is lower than more abundant species).

I think Waubesa is just a harder to predict lake;  it shouldn't be any specific outlier on that plot that's driving it, as we don't actually have the training data plotted, just the test data (which, in retrospect, is something I should add to the plot...).

Thanks for doing the digging! It's good to know why 👍


Custom template using lazy loading?

Hi all,
I'm using this carousel, and trying to make a custom template for my image
I'm passing the carousel an array of images to get the images in lazy loading, but the array supports only the path property of the image to be shown.
Is there any way to use both - custom template and lazy loading?
as example, showing the image and label with the image name
Any help would be appreciated.

There is only one way for this - to change the internal template of the library.


cleaner interface for swapchain parameter of Graph::new()

Currently this is very error-prone and can lead to subtle sync bugs. Either add validation enforcing this or rework the API.

Swapchain resource name is assumed to be "swapchain" in ed2e5a6a3d55c427d8e8ef02f3e91497ac39cd4a


WIP: Added more logging to incoming transactions processing

Codecov Report

Merging #132 (464af27) into master (6159c14) will decrease coverage by 0.10%.
The diff coverage is 8.33%.

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
- Coverage   50.42%   50.32%   -0.11%
==========================================
  Files         109      109
  Lines       11716    11738      +22
==========================================
- Hits         5908     5907       -1
- Misses       5403     5424      +21
- Partials      405      407       +2

Flag
Coverage Δ

unittests
50.32% <8.33%> (-0.11%)
:arrow_down:

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

Impacted Files
Coverage Δ

model_sync_transactions.go
13.17% <0.00%> (-0.08%)
:arrow_down:

tasks.go
0.00% <0.00%> (ø)

model_incoming_transactions.go
48.61% <9.67%> (-4.38%)
:arrow_down:

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 6159c14...464af27. Read the comment docs.


[BUG] Required client entitlement is missing: MediaPlayback

Describe the bug
By every openning the player I have printed error in console like this:
2021-06-30 09:24:44.141532+0200 Media Creations[39379:12785401] [ProcessSuspension] 0x10c4dd0b0 - ProcessAssertion: Failed to acquire RBS MediaPlayback assertion 'WebKit Media Playback' for process with PID 39379, error: Error Domain=RBSAssertionErrorDomain Code=3 "Required client entitlement is missing" UserInfo={RBSAssertionAttribute=<RBSDomainAttribute| domain:"com.apple.webkit" name:"MediaPlayback" sourceEnvironment:"(null)">, NSLocalizedFailureReason=Required client entitlement is missing}
Mine YoutubePlayerController looks like this below:
final YoutubePlayerController _controller = YoutubePlayerController(
      initialVideoId: videoId,
      flags: YoutubePlayerFlags(
        enableCaption: false,
        hideThumbnail: true,
        forceHD: true,
        autoPlay: false,
        startAt: startAt,
      ),
    );

To Reproduce
Open YoutubePlayerFlutter on IOS
Expected behavior
Opens player without any errors
Screenshots
If applicable, add screenshots to help explain your problem.
Technical Details:

Device: Iphone 10
OS: iOS 14.6
Version Flutter 2.2.2

Additional context
Here's my code of YoutubePlayer:
return OrientationBuilder(
      builder: (_, __) => YoutubePlayer(
        key: UniqueKey(),
        showVideoProgressIndicator: true,
        onReady: _controller.play,
        controller: _controller,
        controlsTimeOut: const Duration(seconds: 2),
        actionsPadding: const EdgeInsets.only(bottom: 10),
        progressIndicatorColor: _theme.primaryColor,
        onEnded: (_) => _controller.pause(),
        bottomActions: [
          Padding(
            padding: const EdgeInsets.only(
              left: VideoFullScreenButton.paddingSize +
                  VideoFullScreenButton.size * 2,
              right: VideoFullScreenButton.paddingSize,
            ),
            child: CurrentPosition(),
          ),
          ProgressBar(
            colors: ProgressBarColors(
              playedColor: _theme.primaryColor,
              handleColor: _theme.primaryColor,
            ),
            isExpanded: true,
          ),
          GestureDetector(
            onTap: () => launch(
              Strings.youtubeUrlFromId(
                videoId,
                seconds: _controller.value.position.inSeconds,
              ),
            ),
            child: const Padding(
              padding: EdgeInsets.only(
                left: VideoFullScreenButton.paddingSize,
                right: VideoFullScreenButton.paddingSize,
              ),
              child: FaIcon(
                FontAwesomeIcons.externalLinkAlt,
                color: Colors.white,
                size: VideoFullScreenButton.size,
              ),
            ),
          ),
          VideoFullScreenButton(
            preloadedFullScreen: isFullScreen,
            key: UniqueKey(),
            onTap: (isFullScreen) => _onVideoFullScreenTap(
              isFullScreen,
              context,
              _controller,
            ),
          ),
        ],
      ),
    );

Is something missing in my project? In case of asks I'll respond immediately.

I have the same problem

Does this warning refer to Info.plist?

@PcolBP did you find any solution?

Unfortunately not for YoutubePlayerFlutter. For that project I switched to YoutubePlayerIframe since it offers better performance in compare to YoutubePlayerFlutter and no error is printing since then.


feat(shield): fetch service data api

Add fetch service data api for user and group entity.

will move from admin when upsert PR merged.


Mouse movement speeds up keyboard controls

Demonstrated in the gif below. My completely uninformed guess is that mouse movements cause the GUI to update rapidly, which resamples the keyboard faster resulting in a faster pan.  I'll fix this soon.


dsl: not platforms

Fixes/implements #5183

@indirect please share your thoughts on this PR / the linked-to issue

I would probably be okay with this as long as the option takes the form all_platforms_except: [:jruby], and if you pass both platforms: and all_platforms_except: it raises an InvalidOption exception. How do you feel about it, @segiddins?

I personally feel like it's DSL bloat, and the sort of thing that can cause bugs in combination with other options, just like how with/without can get confusing

The bugs when used in combination issue is why I'm only okay with this if combining :platform and :except_platform raises an exception. The unfortunate reality of gems that work on any platform except the one makes me think this is probably worth accepting.

@indirect @colby-swandale we should probably come to a decision on this

I'm on the fence about this. Is there a particular example that would highlight the need for this feature?

Do we want to make a decision on this? @indirect @segiddins


[WIP] Replace periodic authentication revocation with long polling

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test FAILed.

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


Removed the  pipe symbol in dockercfg-secret.yaml

Which problem is this PR solving?

solves-#48533

Description of the changes

removed pipe symbol in dockercfg-secret.yaml from  content/en/examples/secret/

Explicit hold until this PR only covers one language.
(Ok to unhold once that's done)
/hold

Thanks for the suggestion! I'll make a new commit that updates only the English content.

/remove lang/bn
/remove lang/zn

@priyanshsao we don't see that there is anything to fix here


Config tags not being passed through

Hiya,
It looks like there's maybe an issue with task tags not being unioned with config tags.
I'm not a python expert but this code snippet seems to not set the unioned result back into the t.tags variable.
loader.py (line 52)
if additional_tags:
t.tags.union(set(additional_tags))
Am I correct or have I missed something?

You are correct. It appears this should have been .update().


Update Scheduler name

This change is

I signed it!

/cc @gaocegege @richardsliu
As we sync up, we'd like to work together on the batch scheduler part of Volcano :)
We'll update related doc & code for more new features, e.g. Queue.

Some things that we need to know: Volcano scheduler is built on top of kube-batchd and maintained by @k82cn and his team. The Volcano scheduler will have some advanced features compared with kube-batchd, but share the same interface with kube-batchd. Thus I think we could use it.
/lgtm

/ok-to-test

@gaocegege , thanks; yes, I'm the owner of kube-batch and volcano; we'll make sure both of them share the same API :)

/assign @richardsliu

/lgtm
/approve


Release Dev to Test

Links to Jira Tickets
https://apps.nrs.gov.bc.ca/int/jira/browse/SIMSBIOHUB-541
Description of Changes
Release latest changes from Dev to Test
Testing Notes
n/a

Codecov Report
Attention: Patch coverage is 81.28755% with 218 lines in your changes are missing coverage. Please review.

Project coverage is 63.52%. Comparing base (e7e77e2) to head (2f5cd0d).
Report is 2 commits behind head on test.

Files
Patch %
Lines

...ministrative/security/submission/{submissionId}.ts
0.00%
35 Missing :warning:

api/src/repositories/security-repository.ts
55.81%
19 Missing :warning:

api/src/paths/submission/search-idx.ts
0.00%
17 Missing :warning:

...pi/src/paths/administrative/security/categories.ts
0.00%
16 Missing :warning:

api/src/paths/administrative/security/rules.ts
0.00%
16 Missing :warning:

...ministrative/submission/{submissionId}/messages.ts
66.66%
14 Missing :warning:

api/src/services/security-service.ts
59.37%
12 Missing and 1 partial :warning:

api/src/repositories/submission-repository.ts
91.22%
3 Missing and 7 partials :warning:

api/src/services/search-index-service.ts
80.76%
4 Missing and 6 partials :warning:

api/src/app.ts
0.00%
8 Missing :warning:

... and 27 more

Additional details and impacted files
@@            Coverage Diff             @@
##             test     #238      +/-   ##
==========================================
- Coverage   69.21%   63.52%   -5.70%
==========================================
  Files         211      267      +56
  Lines        5960     7470    +1510
  Branches      934     1039     +105
==========================================
+ Hits         4125     4745     +620
- Misses       1612     2470     +858
- Partials      223      255      +32

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


trojan

beautiful tool but it contains a trojan i dont recomment this tool to anyone

You can fix this problem by going to Windows Defender > Potential threat found > Allow this program. Once this is done then everything will work.

Above comment is how to run.

It might be dangerous but you might want to try it.

Yes, a very controversial project.
VT
Hybrid Analysis


Update Post “we-love-pets-earley/index”

Automatically generated by Decap CMS

👷 Deploy Preview for harmonious-valkyrie-ae7cbe processing.

Name
Link

🔨 Latest commit
a726fcc7a412d18142c7fd8743bd90c4cc7d274e

🔍 Latest deploy log
https://app.netlify.com/sites/harmonious-valkyrie-ae7cbe/deploys/66f5398ae1c7380008f6f4de


feat: New API endpoints

Concordance search in glossaries
Translation alignment
Concordance search in translation memory
fix: #74

Codecov Report

Merging #113 (2a305ca) into main (488d52c) will increase coverage by 0.01%.
Report is 3 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #113      +/-   ##
==========================================
+ Coverage   99.16%   99.17%   +0.01%
==========================================
  Files         140      141       +1
  Lines        4757     4788      +31
  Branches      111      111
==========================================
+ Hits         4717     4748      +31
- Misses         28       29       +1
+ Partials       12       11       -1

Files Changed
Coverage Δ

crowdin_api/__init__.py
100.00% <100.00%> (ø)

crowdin_api/api_resources/glossaries/resource.py
100.00% <100.00%> (ø)

...rces/glossaries/tests/test_glossaries_resources.py
100.00% <100.00%> (ø)

...din_api/api_resources/reports/requests/__init__.py
100.00% <100.00%> (ø)

..._api/api_resources/string_translations/resource.py
100.00% <100.00%> (ø)

...ations/tests/test_string_translations_resources.py
100.00% <100.00%> (ø)

...n_api/api_resources/translation_memory/resource.py
100.00% <100.00%> (ø)

..._memory/tests/test_translation_memory_resources.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes