iOS 10.1 share sheet not appearing using WKWebview

I am attempting to implement the onepassword extension to fill in a WKWebview on iOS10.1, however the code provided does not call the share sheet to appear. The extension says its available and I'm using cocoapads to install.
I made a test project just to be sure there were no other factors. I loaded the test project
with the same code using a UIWebview and it worked great, when I switched the UIWebView to a WKWebview it stopped working. Any idea's?
Xcode 8.1
Min Target iOS 9
Device 7+
Test Project Provided
onepasswordTest.zip
Update on debugging. It is showing the below method does not exist:
[self fillItemIntoWKWebView:webView forViewController:viewController sender:(id)sender showOnlyLogins:yesOrNo completion:^(BOOL success, NSError *error)]
If I change the the method:
[[OnePasswordExtension sharedExtension] fillItemIntoWebView:_wkWebView forViewController:self sender:sender showOnlyLogins:YES completion:^(BOOL success, NSError *error)]
to the code below and manually add:
[self fillItemIntoWKWebView:webView forViewController:viewController sender:(id)sender showOnlyLogins:yesOrNo completion:^(BOOL success, NSError *error)]
the share sheet finally appears.
#pragma mark - Web View filling Support

(void)fillItemIntoWebView:(nonnull id)webView forViewController:(nonnull UIViewController *)viewController sender:(nullable id)sender showOnlyLogins:(BOOL)yesOrNo completion:(nonnull OnePasswordSuccessCompletionBlock)completion {
NSAssert(webView != nil, @"webView must not be nil");
NSAssert(viewController != nil, @"viewController must not be nil");
NSAssert([webView isKindOfClass:[UIWebView class]] || [webView isKindOfClass:[WKWebView class]], @"webView must be an instance of WKWebView or UIWebView.");

//#ifdef __IPHONE_8_0
if ([webView isKindOfClass:[UIWebView class]]) {
[self fillItemIntoUIWebView:webView webViewController:viewController sender:(id)sender showOnlyLogins:yesOrNo completion:^(BOOL success, NSError *error) {
if (completion) {
completion(success, error);
}
}];
}
//#if __IPHONE_OS_VERSION_MIN_REQUIRED >= __IPHONE_8_0 || ONE_PASSWORD_EXTENSION_ENABLE_WK_WEB_VIEW
else if ([webView isKindOfClass:[WKWebView class]]) {
[self fillItemIntoWKWebView:webView forViewController:viewController sender:(id)sender showOnlyLogins:yesOrNo completion:^(BOOL success, NSError *error) {
if (completion) {
completion(success, error);
}
}];
}
//	#endif
//#endif
}

Hi @mikeKane
I see you closed this, were you able to figure out what was happening or do you still need assistance?

@gks Thanks for responding. Using cocoapods this was not fixed. However manually dragging the .h and .m from the readme worked.
I do have some more questions but did not want to flood the issues area.

We are a VPN using localhost. Is there anyway to distinguish our app from other apps that use localhost.

The email/username portion is working great however the password box will not fill in. Im attempting to debug this but not coming up with anything noteworthy. Is there anything to look out for?

Hi @mikeKane No worries, happy to help here since this is already started. Or you can email in if you prefer<EMAIL_ADDRESS>

I assume you're using "localhost" for the URL then? If so, nope, we only filter by the URL provided us. The way around this would be to provide a real Cocoa interface and combine with use case #1 in which you could provide your actual website URL. This would allow you to have any login items created to have the URL of your site, and then when you request that URL you'll get the results that are only yours.

Naming the field and using the id to name the field would be a clue to our filling system. Odds are it's not seeing it as a password field. Or if you're doing anything fancy with the field that would do it. Third possibility I can think of is that maybe you've got 2 password fields, one hidden and one visible? It's possible we're filling into the hidden one.

I will change to email. Thank you very much so far.


Develop branch missing some commits from master

I'm not sure how this happened, but the develop branch is missing a couple of previous commits from the master branch of the project.  I noticed this issue because I was planning to write another issue for a null pointer exception in convertAppInterfaceUnregisteredReason.
Link to the file in the master branch:
https://github.com/smartdevicelink/sdl_android/blob/master/sdl_android_lib/src/com/smartdevicelink/proxy/rpc/enums/SdlDisconnectedReason.java
Link to the same file in the develop branch:
https://github.com/smartdevicelink/sdl_android/blob/develop/sdl_android_lib/src/com/smartdevicelink/proxy/rpc/enums/SdlDisconnectedReason.java
It looks like the following two commits never made it in to develop:
https://github.com/smartdevicelink/sdl_android/commit/3028ec4d47e6e7eb8fe0eaa27321d58ef17dda63
https://github.com/smartdevicelink/sdl_android/commit/0f7921545b168dd9e30839900d71a63cc26f79a9

Fixed with commits
https://github.com/smartdevicelink/sdl_android/commit/e1563e1c2b73f050fd69a075f13e6647f260cb7a
https://github.com/smartdevicelink/sdl_android/commit/47084a842f808436476d0b0a4be62c6cc203a363


cpu_profile: Unable to download traceconv for symbolizing profiles.

I followed the steps documented at: https://perfetto.dev/docs/quickstart/callstack-sampling
curl -LO https://raw.githubusercontent.com/google/perfetto/master/tools/cpu_profile
chmod +x cpu_profile
./cpu_profile -n "com.squareup"

Output:
Configured profiling for these processes:

com.squareup

Profiling active. Press Ctrl+C to terminate.
^CFinishing profiling and symbolization...
Unable to download `traceconv` for symbolizing profiles.

 If this is unexpected, please consider filing a bug at:
https://perfetto.dev/docs/contributing/getting-started#bugs.

This error is raised here: https://github.com/google/perfetto/blob/master/tools/cpu_profile#L326-L331
I removed soft_fail=True and this time got:
No prebuilts available for darwin-arm64

I have a Mac M1 laptop. Looks like I'm going to have to build Perfetto myself, wish me luck! Anyway, this error could probably be made a bit more clear, e.g. by removing soft_fail=True.

The upcoming release of Perfetto (coming some time next week) will have M1 binaries as well so this should disappear by then.

The new SDK has been released :)


How do you get these games?

I am trying to import Run 3 from coolmathgames on my website, but I don't know how to do it. Is it in an iframe? Should I copy every aspect of the code and put it all together? I need help pls

can you message me/friend request me on discord? (deleted#4522) ill give you instructions on how to do everything

Please email me at<EMAIL_ADDRESS>because my discord is blocked by my parents. I have a secret google account but you can email me there. I cannot say when I will read it, because my parents cannot be around to see me on my other account. I will try to see if I have enough time on my other acc to be on discord. I hope you understand :)


Bazel incompatible changes

Tensorflow doesn't build with Bazel incompatible flags.
Based on our CI (https://buildkite.com/bazel/bazel-at-release-plus-incompatible-flags/builds/95), the following flags will fail:

--incompatible_new_actions_api
--incompatible_no_support_tools_in_action_inputs
--incompatible_no_transitive_loads
--incompatible_disable_deprecated_attr_params
--incompatible_disallow_load_labels_to_cross_package_boundaries

From what I've seen, updating the dependency on rules_closure will fix some of the problems.
For --incompatible_no_transitive_loads, the simplest is to reexport some symbols explicitly (see https://github.com/bazelbuild/bazel/issues/5636).
If you run buildifier --lint=fix tensorflow/tensorflow.bzl, it will also fix some issues.
Let me know if you need help
(bug also filed inside Google: b/124051098)

I got error when converting tfmodel (.pb) to tflite (.lite): Unable to load package for '//tensorflow/tools/def_file_filter:def_file_filter_configure.bzl': BUILD file not found on package path
Is that because of incompatibility?

Add --incompatible_remove_old_python_version_api to that list. I'm flipping this flag to true now, for release in Bazel 0.25. The fix #26752 will bump TF's min required Bazel version to 0.22.

I am sorry for dropping this issue.
I saw that Yun has done many of these changes.

I am sorry to forget. I think this is complete. We even were able to build TF with bazel 3.0 now.


Remove unnecessary class variables

I removed unused class variables.

Thank you for suggestion, but perhaps these variables can be referenced outside of this class when it is instantiated. Trainer is such an example[https://github.com/chainer/chainer/blob/83d619f92efb6c842eb6fa283c1fa529b43392ab/chainer/training/trainer.py#L222].

Oops I'm sorry, I didn't notice it. I close this PR :bow:

Don't worry. Thank you, anyway.


Sync RHODS master branch with v1.7 ODH branch

This is sync up PR. List of commit been cherry-picked from ODH:

https://github.com/red-hat-data-services/kubeflow/commit/fb769e53ec58fa7ba6a27655e2949d8aa075a518
https://github.com/red-hat-data-services/kubeflow/commit/588142611a3c6cbbe72a5ad7ca66f2aedab3dc22

Fixes issue:

https://github.com/opendatahub-io/kubeflow/issues/145
https://github.com/opendatahub-io/kubeflow/issues/109

/retest-required


[BLE Mesh] How should I test the friendship and low power node functions? (IDFGH-2176)

I simply read your SDK code and think it should support the friendship and low power functions. But for these two functions I have the following problems:

Because I don't know much about these two functions, so when I use menuconfig to configure these two functions, I don't understand many parameter settings.
How should the friendship node be designed? Is it only necessary to enable the friendship function?
How should the low power node be designed? Does the device need to be designed with low power or sleep mode? Does your ESP32 support the above model?

Can you provide new examples for the above mentioned issues and explain in detail?
In which branch is the latest version of the ble mesh code? Is it in the master branch or the ble_mesh_release/esp-ble-mesh-v0.6.1 branch?

Hi @zhanlej
Sorry for the late reply.
Currently for Friend node, it only needs to enable the Friend functionality, and when the Low Power node enters the network, it will try to find proper Friend node and establish a friendship.
For Low Power node, after the friendship is established, the BLE Scan functionality will be disabled currenlty, and it will be re-enabled after a certain delay. The Low Power node is useful when the device is a sensor or some other things which need low power consumption.
We will provide corresponding examples to illustrate how to use the friendship functionality later.
Thanks & Regards

Hi @zhanlej
Sorry for the late reply.
Any updates about the testing progress?
Thanks.

Thanks for reporting, will close due to short of feedback, feel free to reopen with more details. Thanks.

Hi @Campou ,

Currently for Friend node, it only needs to enable the Friend functionality, and when the Low Power node enters the network, it will try to find proper Friend node and establish a friendship.
For Low Power node, after the friendship is established, the BLE Scan functionality will be disabled currenlty, and it will be re-enabled after a certain delay. The Low Power node is useful when the device is a sensor or some other things which need low power consumption.

Where this behaviour is described?
Is this behavior compliant with the standard?

We will provide corresponding examples to illustrate how to use the friendship functionality later.

Are these examples available to everyone?
Is it possible to have them?
Thanks & Regards

Hi, are there any ways/examples to test friendship node and LPN? How would i know in a mesh if they are working as expected according to the bluetooth mesh specifications

Are there examples for this yet? I'm similarly stuck.

@xiongweichao Hi Is there anyone who worked on LPN and friend node  Kindly provide the feedback on how to use those features...
Thank you in advance....


After Deployment getting 500 errors on some static routes

I deployed the application with terraform-aws-open-next with multi-zone approach:
module "deploy" {
  source = "./terraform-aws-open-next/modules/tf-aws-open-next-multi-zone"
  prefix = "iac"
  deployment = "SHARED_DISTRIBUTION_AND_BUCKET"

  zones = [
    {
      root        = true
      name        = "web-application"
      folder_path = "../.open-next"
    }
  ]

  website_bucket = {
    force_destroy = var.s3_force_destroy
  }

  providers = {
    aws.global          = aws.global
    aws.iam             = aws.iam
    aws.dns             = aws.dns
    aws.server_function = aws.server_function
  }
}

After succesfully deployment, some statically generated content fetch getting 500 error:
Request URL:
https://x1234.cloudfront.net/ka/%E1%83%A1%E1%83%A3%E1%83%9E%E1%83%94%E1%83%A0-%E1%83%9B%E1%83%A8%E1%83%9D%E1%83%91%E1%83%94%E1%83%9A%E1%83%98-1/%E1%83%9B%E1%83%A8%E1%83%9D%E1%83%91%E1%83%9A%E1%83%98%E1%83%A1-%E1%83%9B%E1%83%A8%E1%83%9D%E1%83%91%E1%83%94%E1%83%9A%E1%83%98-1/%E1%83%9B%E1%83%A8%E1%83%9D%E1%83%91%E1%83%94%E1%83%9A%E1%83%98-%E1%83%99%E1%83%90%E1%83%A2%E1%83%94%E1%83%92%E1%83%9D%E1%83%A0%E1%83%98%E1%83%90-1/%E1%83%A1%E1%83%90%E1%83%91-%E1%83%99%E1%83%90%E1%83%A2%E1%83%94%E1%83%92%E1%83%9D%E1%83%A0%E1%83%98%E1%83%90-2/%E1%83%A1%E1%83%90%E1%83%91-%E1%83%A1%E1%83%90%E1%83%91-%E1%83%99%E1%83%90%E1%83%A2%E1%83%94%E1%83%92%E1%83%9D%E1%83%A0%E1%83%98%E1%83%90/%E1%83%A1%E1%83%90%E1%83%91-%E1%83%A1%E1%83%90%E1%83%91-%E1%83%9E%E1%83%A0%E1%83%9D%E1%83%93%E1%83%A3%E1%83%A5%E1%83%A2%E1%83%98?_rsc=6gs74
Request Method: GET
Status Code: 500 Internal Server Error
Referrer Policy: strict-origin-when-cross-origin

and maybe I don't exactly know but, getting 500 errors while trying to access nextauth api request:
https://x1234.cloudfront.net/api/auth/providers
Request Method:GET
Status Code:500 Internal Server Error

Hi @Saba-Shavidze,
Are you able to share any logs from the lambda function (api route) or the cloudfront distribution? As I’m not quite sure why this is causing a 500 status code to be returned.
Are you also able to share the next js and open next version you are using?

nextjs: 14
open-next: 2.3.6
There is no logs enabled in cloudfront distribution and in server-function labmda I don't see any logs while trying to sign up with socials, to see some logs while getting 500 from nextauth api request.

ops, sorry, I can see this kind of errors while trying to sign up using nextuath:
{
  clientName: 'DynamoDBClient',
  commandName: 'BatchWriteItemCommand',
  input: { RequestItems: { 'iac-web-application-isr-tag-mapping': [Array] } },
  error: AccessDeniedException: User: arn:aws:sts::***:assumed-role/iac-web-application-server-function-role/iac-web-application-server-function is not authorized to perform: dynamodb:BatchWriteItem on resource: arn:aws:dynamodb:eu-central-1:***:table/iac-web-application-isr-tag-mapping because no identity-based policy allows the dynamodb:BatchWriteItem action
      at throwDefaultError (/var/task/cache.cjs:10:5073)
      at /var/task/cache.cjs:10:5239
      at de_CommandError (/var/task/cache.cjs:41:65230)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
      at async /var/task/cache.cjs:2:55466
      at async file:///var/task/index.mjs:11:1230
      at async file:///var/task/index.mjs:10:15749
      at async file:///var/task/index.mjs:2:20315
      at async Promise.all (index 0)
      at async Eh.batchWriteDynamoItem (/var/task/cache.cjs:60:62043) {
    '$fault': 'client',
    '$metadata': {
      httpStatusCode: 400,
      requestId: 'XXX',
      extendedRequestId: undefined,
      cfId: undefined,
      attempts: 1,
      totalRetryDelay: 0
    },
    __type: 'com.amazon.coral.service#AccessDeniedException'
  },
  metadata: {
    httpStatusCode: 400,
    requestId: 'XXX',
    extendedRequestId: undefined,
    cfId: undefined,
    attempts: 1,
    totalRetryDelay: 0
  }
}

Hey @Saba-Shavidze, which environment variable is this?
You can supply additional environment variables through env variables for the server function

Looking at that error, it looks like it is failing to write to the ISR mapping table. There might be permissions missing for ISR. Need to look at this further

Yes, it's not the problem I will update the policy.
Yep I noticed it later to pass additional environment variables to server function.
Thanks a lot for your quick reply every time I ask you something you're always trying to help me, really appreciate it.

I have just pushed out a new module version; this should resolve the permissions problem with DynamoDB. There were a few missing permissions on SQS, too.
I have updated it to be in line with what is in the CDK construct/ ion implementation
Can you use v2.2.2 of the module to see if this resolves your issue?

I'm happy to help.
Thanks for using the module and providing feedback/ raising any issues

One more question, Why the open-next alias versions does not update? next is updating but open-next does not

So the aliases are there to help with continuous deployment, do you have this enabled?
If you have continuous deployment enabled, the module will update the alias which is associated with the staging env I.e. next and then when the change is promoted the aliases flip I.e. open-next will update
If continuous deployment is disabled, the alias associated with the primary distribution is updated, the staging alias doesn’t need to update as it’s not used.
I have been meaning to write up a blog post about how that works but not had the time recently.
Hope that helps

How to enable it? I changed them manually to always point to $LATEST.
Can't really understand for now, but it's okay.
I'm working with cognito and have some problems, maybe u would give me some advices,
When the user authenticate via SOCIAL provider, in this case with google, everything works fine I'm creating the user with cognito, but when I'm trying to update the phone number it throws the 500 ERROR, on locally it works fine.
I pass the additional policy to the server-function:
data.tf
data "aws_caller_identity" "current" {}

data "aws_iam_policy_document" "cognito_policy" {
  statement {
    actions = [
      "cognito-idp:AdminCreateUser",
      "cognito-idp:AdminDeleteUser",
      "cognito-idp:AdminGetUser",
      "cognito-idp:AdminUpdateUserAttributes",
      "cognito-idp:UpdateUserAttributes",
      "cognito-idp:AdminSetUserPassword",
      "cognito-idp:AdminConfirmSignUp",
      "cognito-idp:InitiateAuth",
      "cognito-idp:SignUp"
    ]
    resources = [
      "arn:aws:cognito-idp:${var.cognito_region}:${data.aws_caller_identity.current.account_id}:userpool/${var.cognito_user_pool_id}"
    ]
  }
}

main.tf
  server_function = {
    additional_environment_variables = {
      "NEXTAUTH_URL"         = var.nextauth_url
      "NEXTAUTH_SECRET"      = var.nextauth_secret
      "COGNITO_DOMAIN"       = var.cognito_domain
      "COGNITO_REGION"       = var.cognito_region
      "COGNITO_USER_POOL_ID" = var.cognito_user_pool_id
      "COGNITO_ISSUER"       = var.cognito_issuer
      "AUTH_COGNITO_ID"      = var.auth_cognito_id
      "AUTH_COGNITO_SECRET"  = var.auth_cognito_secret
    },
    additional_iam_policies = [
      {
        name   = "cognito_policy",
        policy = data.aws_iam_policy_document.cognito_policy.json      }
    ]
  }

In cloudwatch logs I have this kind of error :
UnrecognizedClientException: The security token included in the request is invalid.

Hi @shavidze,
There is a readme for continuous deployment which explains the config, etc.
On the Cognito error, have you built a custom UI for login & user details management?
if so are you able to share the sdk call you use on the server to modify the phone number?

Thanks, I already fixed it.
Anyway thanks.

Hey @shavidze, are you able to share more about what the fix was?
Curious if I can do anything to improve the documentation or make any more examples.

https://gist.github.com/cmawhorter/3cf67164414c4a97fcd8#beware-env-credentials


Inactive user can no longer log in

Coverage increased (+0.05%) to 59.418% when pulling a1fd30887491fc062ac2bd95f8845ab84934e9e5 on login into a72795ba5b0efab4f34cb8ddd538489ca821dc1a on master.


Equinix Metal (Packet) Block Storage via Datera is EOL

This CSI provider is no longer useful as the service backing it has been deprecated and will be generally shut down, June 1, 2021.
Affected users have been notified. Speak with your Account Manager or the Success team about alternatives.
For more information, see:  https://metal.equinix.com/developers/docs/storage/elastic-block-storage/#deprecation
The API will generally return API errors upon attempted to list volumes when there are none or when creating new volumes.

[x] Close all issues
[x] Update the README https://github.com/packethost/csi-packet/pull/104
[x] Determine if there need for a final release.

Does this provider properly surface these errors?

[ ] Archive the project

Some general alternatives are offered at https://metal.equinix.com/developers/docs/kubernetes/kubernetes-on-equinix-metal/#storage.

No further releases will be made.


Update Readme to the latest version

Thanks! I'm afraid we can't release a new version for this improvement though, because it'll force us into a little recursive loop 😁


Support correlationId across fetch/HTTP calls and error/activity tracking

We would like to add support for generating/supporting CorrelationIds for applications.  Some elements we would like to see in any solution:

Easy ability to generate unique id for Fetch call kicking off the operation.
Built-in integration to usage tracking/client error tracking/logging
Support in HttpClient/JsonClient to pass through ids in requests/responses

Will require both hoist-react and hoist-core releases

Very much in favor of leveraging LoadSpec
Agree with everything else in here.
Am curious about what we mean by "always generate" a correlation id by default.  I don't think we would want these ids on our Hoist API fetches -- sending the header would be fine, but the various logging might just be noise.  Should we start with the requirement that you need to be explicit?
do I dare suggest a shorter name  -- 'corrId', cKey, corrKey, cid, 'ck', processKey?

Thanks for mentioning the logging! That's one thing I didn't capture here. Believe we want to have some out-of-the-box handling within our Hoist logging to log correlationId if present, similar to what we do with username.

But then yes, agree we will want some control over that to avoid it being too noisy - i.e. it will be essential to relay in machine-readable logs, but less so for humans reading logs in the admin console log viewer...

Wondering if we should have a convenience method in our BaseController/BaseService class to extract the correlationId from the configured header?

I do believe we want some way to have an ID always generated for requests, otherwise an app developer is going to need to remember to add this on each call to a business API, and will inevitably forget a few / do so inconsistently. Feel pretty strongly that the only way to get this into actual, widespread use is to make it easy to turn on globally in a "set and forget" fashion.

But agree 100% that forces careful choices around logging defaults - we could also consider some special handling to change the default behavior when calling our own Hoist endpoints, where having a correlationId is probably not useful.

Also feel strongly we need to stick with correlationId - it's the term of art and a strong convention in many, many other toolkits.

Could consider shortening to corrId but not really a fan of those super-compressed arg names  - with TS we will get auto-complete / support for catching typos, and then it's just that much more clear what's happening.
With support for defaults, I would expect many apps will have few to no references to this arg anyway.

Its fine to stick with correlationId -- not quite so convinced at the strength of the convention, but it seems safest.  On balance, if we are going to be seeing this everywhere, I would choose the shorter corrId version.  Would prefer not to have both, which we sometimes

If its in the header, I guess it won't be disturbing me when troublshooting the network traffic, so that's for sure a plus.

Not sure there is going to be an elegant way to turn it off in the logging for human readable logs -- think the idea that they both have the same data is a fairly important invariant, and we will often want it in our human readable logs.   I think we will probably need to be able to just turn off its generation at the cliientSource with corrId: false

Random-thought -- Maybe we could log just a small portion of it in our human-readable logs, and log the full id for every request to a seperate log?

yea, but was a little worried about the constraint that puts on what kind of correlationId you can use.

yea, but was a little worried about the constraint that puts on what kind of correlationId you can use.

Yes - I was looking for any mention of selecting some more meaningful corrId - e.g. one that started with an app name or included other human-readable info, presumably with some UUID tacked on at the end to ensure uniqueness.  Did not find any such examples.
Seemed like it could helpful, but in the end decided that this did not need to be an area where we got more clever than was required, especially if it makes it more difficult for us to take other optimizations like in the logging that we know we would like to take...

If you think about it - anything you bake into a corrId might just end up being more confusing than it is helpful as the ID gets passed around through a distributed system.  E.g.   But then imagine that you have TB > API-A > API-B, and API-A is misbehaving and calling API-B in a bad way, but passing along our ID.

HoistReact PR: https://github.com/xh/hoist-react/pull/3602
HoistCore PR: https://github.com/xh/hoist-core/pull/338
Toolbox PR: https://github.com/xh/toolbox/pull/704


Add filter mechanism in MAL core and fix some bugs

Also fixed some minor bugs

Fix concurrency bug in MAL increase-related calculation.
Fix a null pointer bug when building SampleFamily.

[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes https://github.com/apache/skywalking/issues/8145.
[x] Update the CHANGES log.

mal.md should be updated about filter expression.

filter is added to all MAL files now


allow dask in sorting functions

Right now, the sort_1dim and sort_2dim functions throw an error when trying to using the functions on dask arrays, which is only solved by loading the lazy data into memory with .load() or .compute(). This may pose a problem with large datasets, and it certainly slows down the computation. It seems that adding dask='allowed' to xr.apply_ufunc for sorting the indices would solve the issue, and allow for use of the functions with both dask arrays and normal DataArrays.

Hey Marlein,
Thanks for bringing this issue to our attention.
Having looked at the code and tested some things, I think we can simply remove xr.apply_ufunc and instead use np.digitize directly. This should work the same, since the result of np.digitize is not used in any situation where it would be necessary to have an xarray instead of a numpy array. Using np.digitize directly, without xr.apply_ufunc, also seems to work fine with Dask arrays.
Could you try this and let us know if this works for you, too, and solves the issue?
To try my solution, open the calc code and replace xr.apply_ufunc(np.digitize, constructorTEF.tracer, var_q) with np.digitize(constructorTEF.tracer, var_q) in sort_1dim. There are two more lines like this in sort_2dim that need to be modified in the same way.
Cheers,
Markus


Dependency updates

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

Project coverage is 65.72%. Comparing base (e1e636f) to head (1cc5e59).

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #3892    +/-   ##
========================================
  Coverage   65.72%   65.72%
========================================
  Files         515      515
  Lines       10001    10001
  Branches     2198     2357   +159
========================================
  Hits         6573     6573
+ Misses       3424     3180   -244
- Partials        4      248   +244

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


Optional dark theme

Is your feature request related to a problem? Please describe.
Some users, including me, prefer dark themes over light themes, because:

easier on the eyes, especially at night
oled screens use less energy
looks cool (subjective of course)

Describe the solution you'd like
I have implemented a dark theme on my fork of rss-bridge. You can test it here and view the source code here.

However, many people prefer light themes over dark themes. A possible solution is to add a button on the page somewhere that toggles between light and dark theme.

However, many people prefer light themes over dark themes. A possible solution is to add a button on the page somewhere that toggles between light and dark theme.

You could use @media (prefers-color-scheme: dark) CSS query to apply dark theme based on user's system/browser settings. https://css-tricks.com/a-complete-guide-to-dark-mode-on-the-web/

Oh, nice! Im gonna try that.
On Mar 12, 2021, at 12:11, Evo<EMAIL_ADDRESS>wrote:
However, many people prefer light themes over dark themes. A possible solution is to add a button on the page somewhere that toggles between light and dark theme.
You could use @media (prefers-color-scheme: dark) CSS query to apply dark theme based on user's system/browser settings. https://css-tricks.com/a-complete-guide-to-dark-mode-on-the-web/https://css-tricks.com/a-complete-guide-to-dark-mode-on-the-web/#dark-mode-at-the-operating-system-level
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/RSS-Bridge/rss-bridge/issues/2011#issuecomment-797422648, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEKI4I7W3M3PPQ2VEWYH4CDTDHZGPANCNFSM4YZMWXDA.

Oh, nice! Im gonna try that.

oled screens use less energy

As I know, OLED Screens only consume less energy if the Dark Mode is completely Black (which only looks good on OLED's, in my opinion).
But, if that's true, please remove that from the first Comment @t0stiman

As I know, OLED Screens only consume less energy if the Dark Mode is completely Black (which only looks good on OLED's, in my opinion).

I don't think that's true. It also doesn't really matter in terms of this issue/PR.


Review PR disposition

@psibi I reviewed the PR’s here to see which ones we should take to https://github.com/emacs-rustic/rustic
It looks like you have already merged several of these into your fork.
I am opening this issue here because it is easier to link to the PR’s.
Disposition Merge
#557 #554 #542 #540 #539 #536 #535 #531 #526  #525 #524  #523 #522 #499
Needs work
#538 #530 #527
More info
#559 #505 #458 #443
Don’t merge
#513 #455 #266 #203 #177 #97

I haven't gone through all the PR's, but it's a good start.

https://github.com/brotzeit/rustic/pull/538 https://github.com/brotzeit/rustic/pull/530 https://github.com/brotzeit/rustic/pull/527

We can skip 538 and 527 since tree sitter is supported in rustic.

I have created PR's for all the one the we would probably want to merge. Still interested in #530 and the other ones in more info.


Add new FileSystem related messages to LivebookProto

This will help to test in Teams

Just to confirm one thing: do we need an ID for the file system?

The type is going to be s3, github, and so on
The name is going to be a readable string
The value is going to be encrypted credentials

However, I could easily have two file systems with the same readable name and different credentials, no? This is different to secrets, where the name identifies the secret.
If I am correct, the question is who generates the ID. We could easily make it so the IDs are generated on the server, we just need to include them on the messages. This means that Created and Updated should include the ID. Deleted only needs the ID.
Thoughts?

@josevalim it is probably safe to assume that within a particular hub, there will only be one file system pointing to the same data source (bucket url, or in case of GitHub the repo name). I can see the same data source across multiple hubs, that's why we want to add hub id to file system id on the Livebook side. But using the readable name as the key on teams side doesn't sound ideal, perhaps we can just pass the id from Livebook side (unique within hub)?

I talked to @jonatanklosko and we decided to not couple the server and client IDs. So the server will generate an ID which we will include the message. But this ID is different from the s3.id. If we need to store it in the filesystem in the future, we could call it S3.external_id but it may not be necessary.

@josevalim Will we need to identify, in the client side, the hub id in the FileSystem, as we do with secrets?

@aleDsz yes, we will. but we should let Livebook generate the client-side ID, and not the server. This means we can change how either the server or the client encode IDs in the future, without breaking each other!


Error on nest_asyncio.apply()

Problem Description
I'm trying to set up a new Mac and get some existing projects running.  Specifically, I'm trying to use nest_asyncio in Spyder and getting an error on nest_asyncio.apply() as well as inability to restart the kernel after applying.
What steps reproduce the problem?
import asyncio
import nest_asyncio
nest_asyncio.apply()

Paste Traceback/Error Below (if applicable)
And I get the following error message:
[SpyderKernelApp] ERROR | Exception in message handler:
Traceback (most recent call last):
  File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/spyder_kernels/comms/frontendcomm.py", line 164, in poll_one
    asyncio.run(handler(out_stream, ident, msg))
  File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/nest_asyncio.py", line 33, in run
    task = asyncio.ensure_future(main)
  File "/Users/me/opt/anaconda3/lib/python3.9/asyncio/tasks.py", line 677, in ensure_future
    raise TypeError('An asyncio.Future, a coroutine or an awaitable is '
TypeError: An asyncio.Future, a coroutine or an awaitable is required
[SpyderKernelApp] ERROR | Exception in message handler:
Traceback (most recent call last):
  File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/spyder_kernels/comms/frontendcomm.py", line 164, in poll_one
    asyncio.run(handler(out_stream, ident, msg))
  File "/Users/me/opt/anaconda3/lib/python3.9/site-packages/nest_asyncio.py", line 33, in run
    task = asyncio.ensure_future(main)
  File "/Users/me/opt/anaconda3/lib/python3.9/asyncio/tasks.py", line 677, in ensure_future
    raise TypeError('An asyncio.Future, a coroutine or an awaitable is '
TypeError: An asyncio.Future, a coroutine or an awaitable is required

The error appears to be unique to Spyder.  I don't see any errors when I run it in the terminal, Jupyter notebook, QT console, etc.
Versions

Spyder version:  5.1.5
Python version:  3.9.12

Hey @jhogg11, thanks for reporting. I think this error is fixed in our latest version (5.3.0).
Since it's still not available in Anaconda, you have two options:

Use our Mac installer, which you can find here.

Create a new environment with conda-forge packages. For that, please close Spyder, open the Anaconda Prompt and run the following commands there:
 conda create -n spyder-cf -c conda-forge spyder
 conda activate spyder-cf
 spyder

Let us know if that works for you.

Thanks for the quick response.  Nest_asyncio seems to be working after using the Mac installer.
Any idea on when 5.3.0 will be available in Anaconda?

Nop, we don't have any control over that.
I'm going to close this issue because the bug is solved on our side.

Hey @jhogg11, thanks for reporting. I think this error is fixed in our latest version (5.3.0).
Since it's still not available in Anaconda, you have two options:

Use our Mac installer, which you can find here.
Create a new environment with conda-forge packages. For that, please close Spyder, open the Anaconda Prompt and run the following commands there: conda create -n spyder-cf -c conda-forge spyder
 conda activate spyder-cf
 spyder

Let us know if that works for you.

One other question - is it currently possible to install Spyder on an arm64 environment?  I want to run tensorflow, which I understand requires Miniforge3 arm64, however, it seems I can't get both tensorflow and Spyder in the same environment.  The error I'm getting when trying to open Spyder (in a new conda environment) is:
No module named 'PyQt5.QtWebKitWidgets'
But then I run into other issues trying with PyQt5.  Are you aware of this issue or any way to get around it?

Found the answer.  Instead of installing pyqtwebengine through pip, it needed to be installed through conda-forge:
conda install -c conda-forge pyqtwebengine
From one of the comments here:  https://stackoverflow.com/questions/71580652/how-to-solve-error-while-installing-pyqtwebengine
@ccordoba12 - Thanks again for your help.


Migrate to TypeScript

We plan to gradually migrate brain.js to TypeScript, code base is pretty large, so we would love your help! 💪
How to contribute?

Convert a file from .js to .ts
Add types, fix all type errors.
Submit a PR! 🎉

Here you can find a guide on how to contribute.
Want to convert something, let us know in the comment and go ahead! 😎
To avoid duplicate work please comment on which part you want to work on (as long as nobody else is working on it) so we can mark it as taken.
Reach out to us!
Feel free to reach if you have questions or need help getting started. You can leave comments here or you can tag me in your PR if you need any help or you're not sure about something!
You can also get in touch on our  & .
Happy Coding! 🤟

UPDATE:
Wohoooo!!! 🎉
All files inside src directory are migrated to typescript. (Except few ones which are already taken up and being worked on), though we are still looking on improvements to types in these files and removing any types from the source. You are welcome to contribute. 😊
__tests__ directory has still some files left that needs migration to typescript, so feel free to pick em up! 🍭

UPDATE
For anyone looking to contribute, here is the list of files that still needs typescript migration:
https://github.com/BrainJS/brain.js/search?l=javascript&p=1

I'd like to start working on this, I have some experience with Ts, can you tell me which module is up for grabs?

@yashshah1 You are welcome to contribute, please pick any module or portion for conversion and mention it here, so I can mark it in-progress (to avoid any duplication effort).

I'd like to work on this too

@nabeelvalley here is the short guide, super simple to start. Please pick any module or portion for conversion and mention it here, so I can mark it in-progress (to avoid any duplication effort).
Here you can find a guide on how to contribute.

@mubaidr I'm working from the utilities up, to avoid having untyped dependencies
The tests seem to be failing when I convert the file to JS because it looks like Jest is running against the TS files instead of the compiled JS - has the TS Compile been configured?

Yes, you can update test to expect Float32Array.
In the mean-time some tests might still fail, because they have not yet been updated recently. You can can continue working and make sure build process is successful and conversion does not cause increase in no. of failed tests.

Hi! I want to help. If I convert one file, do I need to convert related files too or something? Or just a single file?

You don't need to updated all the related files, just go file by file and make sure build is successful.

Utilities is taken, isnt it?

Activation functions seem like a good place to start, may I work on it?

Utilities is taken, isnt it?

Hi @HarshKhandeparkar, I haven't had a chance to work on this you're welcome to take utilities if you want, just note that needs to be updated above

Np @nabeelvalley. I think I'll work on activation functions :)

@nabeelvalley I think I managed to fix the jest error you were facing in #582. You may copy paste my changes to jest.config.json :)

estimator/ has a single file. Looks like a nice :dart:.

I think I'll snipe some of the utilities next, if you don't mind @nabeelvalley.

You would have probably started in alphabetical order, so I am going to start from the bottom.

Going to snipe utilities/values* tonight :)

Huzzah!

Build systems seems to be broken, I am looking into this issue. 🧯

Can I continue or should I wait?

You should continue your contributions! I will try to fix it asap.

Hi All, taking a look at doing some of these as well today, starting from the top of utilities seeing as @HarshKhandeparkar has started from the bottom

Note, I haven't done utilities/array-lookup-table as it looks like it's working with some dynamic types and a few different cases and it's a bit difficult to infer all of these (in my opinion) until recurrent/rnn-time-step is updated. So doing it after that seems to make sense

I am going to stop targeting utilitites for a while now. Apparently some of them refer to layer etc. So I am starting with layer/ today. I am going to drop a nuke on it :grin:

Love it!

I'll actually work on utilities/kernel first because it is referenced in layer/base. I'll do base after that.

NOTE: layer/base is a HOT MESS.

I also wanted to underline that but apparently underline isn't supported on github.

I'm taking over the layer folder from @HarshKhandeparkar (we coordinated already), and will need utilities either done soon, or will need to go ahead and handle:

utilities/kernel.js
utilities/random.js
utilities/randos.js
utilities/randos-2d.js
utilities/randos-3d.js

I'd also eventually like to consolidate these into a single file, but for a later task.

Going to continue working on utilities as well, will update on her as I go:

utilities/to-assay
utilities/max

__tests__/utilities/mse.js
  test('should return an array if object is passed', () => {
    const collection = {
      name: 'Steve Jobs',
      alive: false,
    };
    const temp = toArray(collection);

    expect(temp.constructor).toBe(Float32Array);
    expect(temp.length).toBe(Object.keys(collection).length);
  });

@robertleeplummerjr I remember doing utilities/kernel.

I can take utilities/random and utilities/random-weight if no one else wants to take it.

@mubaidr The imports are broken and the tests are failing, am I supposed to ignore all of that and force commit?

@HarshKhandeparkar I have fixed the issue. You can create pull request now.

I have randos done already, but if younknocknitnout, it all good.
On Thu, Sep 10, 2020, 12:58 PM Harsh Khandeparkar<EMAIL_ADDRESS>wrote:

I'll convert the randos kernel tomorrow :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BrainJS/brain.js/issues/551#issuecomment-690507706,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFFZO63XW4THPXV4XGVTNLSFEATPANCNFSM4NDROIUQ
.

I have randos done already, but if younknocknitnout, it all good.

Oops, sorry but I opened a PR while you were dreaming ;)

Which file(s) can I work on?

@ofirgeller

we've been handling them sort of from utilities up, because a lot of files depend on those, just keep us updated on here for whichever file you're doing and that should be fine
i'd noticed some tests are failing at the moment too, think @mubaidr was looking into that

@ofirgeller You can select any files from the the repo, and mention it here so that no effort is duplicated. Yes, we do have some failing tests, and we are working on this issue, but as @nabeelvalley said, you can start from utilities and we do have lint and typescript scripts to help catch errors during conversion.

Converted "array-lookup-table.js" (short and sweet), moving on to DataFormatter which will take a bit longer.

@ofirgeller I recommend creating a pull request per file (and related changes i.e. imports etc) to make it easier for code review and merge.

@mubaidr If each atomic change is a a separate commit on the same branch, do you still want each commit to be its own PR?

No. multiple commits per PR is good.

@mubaidr and @HarshKhandeparkar, layers are here: https://github.com/BrainJS/brain.js/compare/ts-layers
They are not done, but I've laid the foundation for what they'll look like.  I just pushed it, and will merge test into it shortly and resolve conflicts.  This includes some pretty heavy upgrades from typescript typings from gpu.js.

Tons more layer conversion, including pool.ts and convolution.ts!

All layers have now been converted.  Opened PR here: https://github.com/BrainJS/brain.js/pull/603.

Merged the above.  Picking up src/lookup.js

Hi, I'd like to work on this as well. Can you tell me which file is not being worked on?

I'm working on src/feed-forward.js, src/utilities/lookup-table.js, and src/praxis/momentum-root-mean-squared-propagation.js, and their tests.

Can I work on utils/range.js and utils/mse.js?

Sure!

Hello, Can I work on utilities/array-lookup-table.js? If not, tell me a peding file to convert

Welcome! That one & reccurrent/matrix is already being worked on (https://github.com/BrainJS/brain.js/issues/551#issuecomment-702838155).
You can pick any other js files from the src or even __tests__

I'm working on converting utilities/traverse-layers-from.js, utilities/traverse-layers-excluding-from.js, and utilities/to-svg.js.

Sure, so can I work on src/likely.js and src/utilities/flatten-layers.js?

Sounds good!

Can I work on src/utilities/layer-from-json.js?

If you are ok with this, I will take all JS files into src/recurrent/matrix

src/recurrent/matrix is already taken up: https://github.com/BrainJS/brain.js/pull/607
You can pick any other js files from the src or even __tests__. Looking forward!

What about __tests__/activation ?

Sure, go ahead!

Hello! Can i convert some js files in src/recurrent ?

@mubaidr Is anyone else working on __test__ folder? In the case of no one: I will convert the whole folder

@mberger75 src/recurrent/matrix is already taken up: https://github.com/BrainJS/brain.js/pull/607 but you are free to choose some other files in recurrent directory or even pick some subdirectory from __tests__ directory.
@lomby92 I recommend picking up one sub directory from __tests__ one at a time. This will help you focus more, help us in the review process and give others some content to contribute too. 😋

Hello, can I get any module to convert it to Typescript ?

You can start with /__tests__/utilities, simple to start with.

Hello! Can I work on /__tests__/utilities/random.js?

Starting on src/recurrent/rnn.js.

Hey! I'd like to start working on (probably three different PRs):

src/recurrent/gru-time-step.js
src/recurrent/gru.js
src/recurrent/lstm-time-step.js
src/recurrent/lstm.js
src/recurrent/rnn-time-step.js
src/recurrent/rnn.js

or any of those files have been already taken for someone else?

Hi! src/recurrent/lstm.js & src/recurrent/lstm-time-step.js is being worked on, you can work on gru.js & gru-time-step.js
Good luck!

Sure @mubaidr , I'll work on those files, thanks.

Any particular files I should work on?

@DarinDev1000 please check __tests__/utilities

Can someone pickup src/recurrent/matrix/copy.js?

It's already ts ¯_(ツ)_/¯

Ah, I was looking at the export, not extension.  I'll take care of converting to named export.

Hello! I want to contribute, can I convert /tests/utilities/layer-size.js to typescript?

@LuanSilveiraSouza sure, go ahead!

I'd like to start working on this, can you tell me which module i can start with?

@rohya8 Please select any file(s) from __tests__ directory and just mention it here.

I'll take a stab at https://github.com/BrainJS/brain.js/blob/master/tests/recurrent/end-to-end.js

I'll contribute to tests/utilities/data-formatter.js!

@dhairyagada /__tests__/utilities/ is fully migrated to typescript as per this pull request: https://github.com/BrainJS/brain.js/pull/630/files
You can pick other directory please.

@mubaidr Can I know which one's are remaining?

All other js files inside __tests__ are available. You can start with __tests__/neural-network/options.js

@mubaidr Thanks! I'll start with it 👍

@mubaidr Has src\neural-network.js already been converted to Typescript already? Because that is required for __tests__/neural-network/options.js

I have a locally converted tests/utilities/data-formatter.js

Hello again! I will convert /__tests__/neural-network/test.js to ts

hey , I would like to  convert /tests/neural-network/to-function.js to ts

Hey, I would like to convert /tests/neural-network/to-function.js to ts

@dhairyagada No, its in progress. But I don't think you would need it.
@LuanSilveiraSouza @Satyam-kumar-yadav Sure, go ahead!

@mubaidr
Let me know if I can pick any of test files for converting to ts.

@geekyrajshri You can work on one of the following:

https://github.com/BrainJS/brain.js/blob/master/__tests__/test-utils.ts
https://github.com/BrainJS/brain.js/blob/master/__tests__/neural-network/to-function.js
https://github.com/BrainJS/brain.js/blob/master/__tests__/neural-network/trainopts.js
https://github.com/BrainJS/brain.js/blob/master/__tests__/neural-network/json.js
https://github.com/BrainJS/brain.js/blob/master/__tests__/train-stream.js
https://github.com/BrainJS/brain.js/blob/master/__tests__/cross-validate.js

@mubaidr I will convert __tests__/cross-validate.js to ts

@mubaidr I'll take

https://github.com/BrainJS/brain.js/blob/master/tests/neural-network/to-function.js
https://github.com/BrainJS/brain.js/blob/master/tests/neural-network/trainopts.js
if you don't mind

I will take https://github.com/BrainJS/brain.js/blob/master/tests/neural-network/json.js

@vikgmdev https://github.com/BrainJS/brain.js/blob/master/tests/neural-network/to-function.js is already in progress, you can continue with https://github.com/BrainJS/brain.js/blob/master/tests/neural-network/trainopts.js. 👍🏼
@JCMais Sure, please

@mubaidr is there any file on which I can work on??

@mubaidr I will take https://github.com/BrainJS/brain.js/blob/master/tests/praxis/arthur-deviation-biases/end-to-end.js and https://github.com/BrainJS/brain.js/blob/master/tests/praxis/arthur-deviation-weights/end-to-end.js ...

hey @mubaidr, can you tell me how are you guys handling this...
Type '{ weights: {}; }' is missing the following properties from type 'ILayer': width, height, depth, deltas, and 9 more.
some properties are missing from interface, should i use type any or is there any other fix??

@mubaidr
which files can I take?

We have following available:

https://github.com/BrainJS/brain.js/blob/master/__tests__/praxis/arthur-deviation-biases/end-to-end.js
https://github.com/BrainJS/brain.js/blob/master/__tests__/praxis/arthur-deviation-weights/end-to-end.js
https://github.com/BrainJS/brain.js/blob/master/__tests__/recurrent/end-to-end.js

Great, I'll fix https://github.com/BrainJS/brain.js/blob/master/tests/praxis/arthur-deviation-biases/end-to-end.js first

@mubaidr I will take https://github.com/BrainJS/brain.js/blob/master/tests/praxis/arthur-deviation-weights/end-to-end.js

@mubaidr  I'll be taking up https://github.com/BrainJS/brain.js/blob/master/__tests__/recurrent/end-to-end.js

Hey! Would like to pick up some work here. I can help with removing any type or converting the file to typescript(although I don't see any file pending). You can direct me to "what is still pending"

@shubhi23994 That would be great! You can run npm run lint to identify any usages. Looking forward!

For anyone looking to contribute, here is the list of files that still needs typescript migration:
https://github.com/BrainJS/brain.js/search?l=javascript&p=1

hey. I'm going to convert https://github.com/BrainJS/brain.js/blob/370502c92ce228148524bc3bff95eac304a559a5/examples/javascript/childrens-book.js if that's not being worked on atm?

Looks like childrens-book.js has already been converted. I'm going to convert https://github.com/BrainJS/brain.js/blob/370502c92ce228148524bc3bff95eac304a559a5/examples/javascript/predict-tense.js if that's not being worked on atm?

I'm going to take a look at https://github.com/BrainJS/brain.js/blob/3b464186dbca121bd3e92845aa4e49ae71df61b5/src/neural-network.js too

Hello again I'll give a shot at migrating https://github.com/BrainJS/brain.js/blob/master/src/recurrent/rnn.js

@luisgregson @dhairyagada Please wait for input from @robertleeplummerjr I think he was working on these, if otherwise, you can then work on these, please.

Yea, nearly done, but was away last week.  Look for by end of week something for review.  @dhairyagada I'd love to get your help on it, however.

@mubaidr @robertleeplummerjr
Sorry, i couldn't see neural-network.js mentioned here so i thought it was free. I'll stop working on it. But let me know if you want any help 🙂

Sorry, neural-network.js is free, I am working on rnn.js.

@luisgregson You can continue working on neural-network.js, please 👍🏼

Yea, nearly done, but was away last week. Look for by end of week something for review. @dhairyagada I'd love to get your help on it, however.

Sure @robertleeplummerjr , I would love to help! Please let me know how can I contribute to it

I have pushed up the RNN changes to https://github.com/BrainJS/brain.js/tree/rnn-ts @dhairyagada.  If you want to check it out. It is nearly complete in script conversion, but I've not tested it entirely yet.  The typescript fails around serializing and deserializing from JSON.

I have pushed up the RNN changes to https://github.com/BrainJS/brain.js/tree/rnn-ts @dhairyagada.  If you want to check it out. It is nearly complete in script conversion, but I've not tested it entirely yet.  The typescript fails around serializing and deserializing from JSON.

Sure @robertleeplummerjr I'll check it out

Wow, this has made a lot of progress! Is there anything left where I can help?

I've got rnn.ts nearly done, upgrades to the data-formatter.ts (because they are so heavily linked) and some minor fixes along with the to-svg.ts fixes.  I even found a (semi naughty) way of fixing some weirdness we were seeing in equation.ts.  I'll have them pushed tomorrow.

Pushed rnn.ts up, with the related changes.  Going to go ahead and merge, and pick up rnn-time-step.js.

Hi there @mubaidr and everyone! Wondering if I could help anywhere? Could I work on cross-validate.js conversion to cross-validate.ts? (along with fixing the test bugs in there right now caused by RNNTimeStep class), doesn't seem like anyone is working on that right now.

@StoneT2000 I am working on this right now.  In fact, yesterday got the RNNTimeStep mostly converted to typescipt, but tests failing.  Now working on those here: https://github.com/BrainJS/brain.js/tree/rnn-time-step-ts.
Here are a list of files that you can choose from:
src/cross-validate.js
src/neural-network.js
src/neural-network-gpu.js
src/recurrent.js
GUYS!  Four files (and their tests 😄)?!?! That is all we have left!

@robertleeplummerjr I'll tackle src/cross-validate.js on my fork of master
Thanks! Ya'lls work on this looks amazing 💯 and lets make that final push!

@robertleeplummerjr @mubaidr just made the cross-validation PR and if it looks good and is merged, I can tackle the neural-networks.js file next (and get rid of the million todos caused by the missing types :) )

Awesome work @StoneT2000!

Dropping a mega gauntlet: https://github.com/BrainJS/brain.js/pull/677

Starting on src/recurrent.js while https://github.com/BrainJS/brain.js/pull/677 is waiting.

I'll get started on src/neural-networks.js

@StoneT2000, how is it going?
src/recurrent.js is now done: https://github.com/BrainJS/brain.js/pull/679 and https://github.com/BrainJS/brain.js/pull/677 is merged.

@StoneT2000, how is it going?
src/recurrent.js is now done: https://github.com/BrainJS/brain.js/pull/679 and https://github.com/BrainJS/brain.js/pull/677 is merged.

@robertleeplummerjr Uni just started again so put a short pause on what I was attempting with neural-network.js
There's also a few confusing typings for data that blocked me last I worked on it

@robertleeplummerjr Uni just started again so put a short pause on what I was attempting with neural-network.js
There's also a few confusing typings for data that blocked me last I worked on it

@StoneT2000 if you need me to take over, just let me know.  Meanwhile, since we have just the two files remaining src/neural-network.js and src/neural-network-gpu.js I'm going to shift to fixing all the broken unit tests on branch: https://github.com/BrainJS/brain.js/tree/ts-tests.

@StoneT2000 if you need me to take over, just let me know.  Meanwhile, since we have just the two files remaining src/neural-network.js and src/neural-network-gpu.js I'm going to shift to fixing all the broken unit tests on branch: https://github.com/BrainJS/brain.js/tree/ts-tests.

Actually yeah, now that i think of it I don't think I'll have much time in the next few weeks to make this a good conversion, feel free to take over @robertleeplummerjr
You can try and use some of the conversion i've made although tbf I think i might've interpreted some functions incorrectly.
I'll be around anyhow and good luck!

Actually yeah, now that i think of it I don't think I'll have much time in the next few weeks to make this a good conversion, feel free to take over @robertleeplummerjr
You can try and use some of the conversion i've made although tbf I think i might've interpreted some functions incorrectly.
I'll be around anyhow and good luck!

Started on src/neural-network.js last night, maybe I'll have something next week.

Started on src/neural-network.js last night, maybe I'll have something next week.

With https://github.com/BrainJS/brain.js/commit/6c86dce270ae0a0414a85cced0ff3c24ac8f30cd I believe we are now a full on typescript library.  Ty everyone for your hard work!

With https://github.com/BrainJS/brain.js/commit/6c86dce270ae0a0414a85cced0ff3c24ac8f30cd I believe we are now a full on typescript library.  Ty everyone for your hard work!


Web app launched from VSCode takes 5x memory usage over published app

I can hardly provide full steps, but basically the app takes tremendously more ram when launched from VSCode than fetched from published website.

You can see in red the memory usage from the single VSCode launched instance (at the same app state, in Release mode !) reaches 1.3Gb.
In blue are two instances of the published app in another chrome window.
So I'm wondering, how am I supposed to profile my app if even the release mode launched from vscode takes an astronomical amount of ram compared to the identical published version ?

Logs

No issues found! (ran in 5.2s)

[√] Flutter (Channel stable, 2.5.3, on Microsoft Windows [version 10.0.18363.1916], locale fr-CA)
    • Flutter version 2.5.3 at C:\Flutter\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (7 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[√] VS Code (version 1.62.3)
    • VS Code at C:\Users\matpa\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.28.0

[√] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 96.0.4664.45
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 96.0.1054.29

! Doctor found issues in 2 categories.

@mat100payette
Since you said launched from VSCode shows this behavior, does the same occur if you directly launch through command line ?
Can you also try the same using default counter app from VScode and see what you get ? Also, it will be good to get some code sample that makes such a huge memory usage. Do you use lot of images or other elements that takes a bit time to render ?
Such details would be useful to analyze this behavior better.

@mat100payette Since you said launched from VSCode shows this behavior, does the same occur if you directly launch through command line ? Can you also try the same using default counter app from VScode and see what you get ? Also, it will be good to get some code sample that makes such a huge memory usage. Do you use lot of images or other elements that takes a bit time to render ? Such details would be useful to analyze this behavior better.

I tried serving the release build from python http server locally, and that also reaches 1.4Gb ram.
I don't think counter app will show any of these symptoms given how small it is :/
My app fetches a large list (2-3k) of image urls to display using Image.network, arranged in a non-paginated (on purpose) GridView.

My app fetches a large list (2-3k) of image urls to display using Image.network, arranged in a non-paginated (on purpose) GridView.

Thanks for the update. Would be good to provide a minimal code sample involving above so that we can take a look at it better.

@darshankawar I don't have minimal code for this. If you've never seen this or have no clue how to approach it then I guess I'm out of luck...

@mat100payette
There are couple of related / similar issues, see if they resemble your case:
https://github.com/flutter/flutter/issues/84565
https://github.com/flutter/flutter/issues/81790


Yii::$app->getRequest()->resolve() after Yii::$app->runAction($route)

I have 2 controllers. I have done Yii::$app->runAction($route) from the action of the first controller to the action of the second controller. Then if I do Yii::$app->getRequest()->resolve() in the action of the second controller I receive the route of the action of the first controller.

Not a bug IMO. You didn't change request, so why ->resolve() should return different result? The same request will be always resolved to the same controller.


No help topic for 'encrypt'

I just signed up for keybase and installed the client on osx.
I create a profile and keys https://keybase.io/kevinsimper
But when I try to encrypt something it doesn't work
kevinsimper@blackbeard:[~]:
$ keybase id max
▶ INFO Identifying max
✔ public key fingerprint: 8EFB E2E4 DD56 B352 7363 4E8F 6052 B2AD 31A6 631C
฿ bitcoin 1BYzrCvfbn81dfiksmD1Bdgt8pgLi1SD7Z
✔ admin of DNS zone maxk.org: found TXT entry keybase-site-verification=uqzZedLr4yVILSkh6nLfctv5oxrqpp8rFLHsz-0xxy4
✔ admin of DNS zone oneshallpass.com: found TXT entry keybase-site-verification=Dld6FHUIWgetEGY0AN4c18Mh0jSc8kRt4RLi8vUakos
✔ admin of DNS zone nutflex.com: found TXT entry keybase-site-verification=S0xnyDtJlarl1KXd6Rb7NXYlUeZ-NdTU8b6Z4jCSOGA
✔ admin of keybase.io via HTTPS: https://keybase.io/.well-known/keybase.txt
✔ "maxtaco" on hackernews: https://news.ycombinator.com/user?id=maxtaco
✔ "maxtaco" on twitter: https://twitter.com/maxtaco/status/433688676975927296
✔ admin of oneshallpass.com via HTTPS: https://oneshallpass.com/.well-known/keybase.txt
✖ "maxtaco" on reddit failed: 429 Too Many Requests (code=129)
✔ "maxtaco" on coinbase: https://coinbase.com/maxtaco/public-key
✔ "maxtaco" on github: https://gist.github.com/8847250
kevinsimper@blackbeard:[~]:
$ keybase encrypt max -m "test"
No help topic for 'encrypt'

That's quite weird.  Can you paste the results of keybase encrypt max -m "test"? Thanks

$ keybase -d encrypt max -m "test"
No help topic for 'encrypt'

Hmm, can you try keybase version?  My guess is that you're running an old version...

$ keybase version
Client:  1.0.8-0
Service: 1.0.8-0

and when i run
$ keybase update check
▶ ERROR No updater available

Yeah, this is a super old version.  Can you upgrade via brew?

It solved it running brew update && brew upgrade keybase, but I thought I had the newest :smile:


Add JavaScript client

This PR fixes https://github.com/casper-network/casper-contracts-js-clients/issues/13
TODO:

[x] Add JavaScript client
[x] Add GitHub CI script

@gyroflaw as we discussed we want to follow the same patterns as in cep78-js-client so users will feel familiar when switching between clients.


Add support for JavaScript Tracker v3

There are some breaking changes from v2 to v3.
https://docs.snowplowanalytics.com/docs/collecting-data/collecting-from-own-applications/javascript-trackers/javascript-tracker/v2-to-v3-migration-guide/

At the moment, there is not much difference between v2 and v3 in regards to how the GTM Template works.
This is certainly on our roadmap to do but it'll likely not happen until Q3 once more of the Snowplow User Base has started migrating to v3 and there are additional benefits to the V3 upgrade within the Template.


Excessive Redistribution Events During Cluster Size Changes

Description
Currently, Refinery’s trace redistribution logic can trigger multiple redistribution events when cluster size changes occur back-to-back in a short time frame. Each cluster size change immediately triggers a redistribution, leading to increased traffic on nodes, which can worsen their stability, especially in environments with frequent scaling events.
Proposed Solution:
Introduce an initial delay using a timer before starting the redistribution process. The behavior will be as follows:

A timer will start after a cluster size change is detected.
Redistribution will only begin once the timer expires.
If another cluster size change occurs before the timer finishes, the timer will reset.
This ensures that redistribution only happens once the cluster has stabilized, reducing unnecessary redistribution events.

Fixed by #1403


paperspace command not found on npm install

Hi - I'm leading a course on machine learning and have successfully installed paperspace on my own machine, but I need to test student access on the school's laptops, and cannot get the cli working. Due to security restraints, I can only do an npm install paperspace-node (no global, no pip, and the binary just downloads as a text document that I cannot execute).
But after successful npm init, npm install, if I attempt to run paperspace ... anything, I get command not found paperspace. Would this be a security constraint? Or a difference between a global install and a local that means I need to do something different to invoke the CLI?

Hi @heaversm when you do an npm install -g paperspace-node it adds it to your path. This means you can use the paperspace command everywhere. You can also install with a regular npm install paperspace-node but then you need to make sure you add the install location to your PATH.
There are lots of way to add to your PATH, but usually a export PATH=$PATH:~/location/to/the/npm/binary will work. Here is a link http://osxdaily.com/2014/08/14/add-new-path-to-path-command-line/


KAFKA-3655; awaitFlushCompletion() in RecordAccumulator should always decrement flushesInProgress count

@ijuma Do you have time to take a look? Thanks!

Thanks for the PR. The change looks good. The usual question follows: can we include a test please?

@ijuma It is possible to write a test case for this. But I don't feel its value is worth the extra code here in this particular case. Adding unit tests for bug fixes is useful to make sure that we will not have the same bug again in the future. And it is generally for detecting bugs that is not evident by simply looking at the code.
However, the bug fixed in this patch is really straightforward and shouldn't happen again if anyone change this method in the future. A unit test for this bug should validate that flushesInProgress is decremented in the event of InterruptedException, which is essentially testing the usage of finally than testing any kafka-specific logic. If we should add tests for something as straightforward as this, we should probably add test for a bunch of other methods that use finally to enforce something, which doesn't seem feasible. What do you think?

@zhuchen1018, The fact that the bug exists in the first place implies that a test is worth it. Why do you think we can't regress on this? The code could be refactored and anything can happen at that point. Unless the test is slow or particularly complex, I don't see the harm in adding it.

btw the corresponding ticket is KAFKA-3655, not KAFKA-3651 as the PR title suggests.

@ijuma Thanks for the explanation. I agree a test can be useful here. I am going to write test for this particular patch. But I would like to discuss it a bit more to understand the best practice for adding tests in kafka , which hopefully can encourage more contributions in the open source community.
I think Kafka open source community should probably have a uniform standard for what should be tested. The standard should be enforced consistently for at least new patches. Bug is good for detecting something that were missed by originally developers and reviewers; once a bug is detected, we should start to require higher standard for tests by original developers so that similar issues can be covered in the future. Does this sound reasonable?
For this particular patch, I suppose that we can add a test like this: call beginFlush, call awaitFlushCompletion, interrupt the thread, and verify that flushInProgress() == false. Say it is useful to test it, should we similarly test that appendsInProgress does't change before and after RecordAccumulator.append()? Should all future developers who write finally to enforce that the statement in the finally is always executed for every exception that may be thrown?

@zhuchen1018 I agree that it would be good to be consistent around this. Your description regarding tests for bugs sounds reasonable to me. We should also do better on requiring good test coverage for new work (it's a process and it will take some time before we have a clear process for this).
For this PR, I'm happy if you add the first test you described.
As you said in another comment, we could write tests for a lot of tests if we want to ensure that our finally works correctly. It's also a bit subjective what is worth testing and what isn't. However, a simple rule is that if we have a bug for something, we should include a test with the fix unless there is a good reason not to (test is brittle, takes too long, too complex, etc.). I think this is a fair rule and it doesn't overburden contributors.

@ijuma Sure, I certainly agree that it is a good principle and common to "add test for a bug". Because the fact that there is a bug indicates this is probably hard to detect. But I feel that, if we couldn't require developers who originally write these code to add test for finally, it probably indicates that the benefits of these tests are not worth the developer's time; if this is not worth developer's time who write the feature, then we shouldn't require contributors who fix the bug to write the test. It would be a fair rule to hold developers of the feature to no lower standard than those who help fix their code, right?
It would be good to take cost of time into account and be consistent about a standard for test since there will be growing number of these bug fixes moving forward. The argument I make here is for long term development rather than trying to avoid adding test for a specific bug :)

@zhuchen1018 I think we're going in circles a bit. I'll leave some thoughts and if you want to discuss the topic more generally, then please start a mailing list thread. So here it goes:

I think focusing on a language construct (finally) is perhaps the wrong way to think about it. Let's describe the issue as an interruptible method. It is a good idea to have a test that causes the method to be interrupted because this is a common source of bugs (examples are both the issue in this PR and in ByteBuffer.allocate where there were actually multiple bugs and PRs that you were involved with).

When someone contributes a new feature, they contribute a bunch of tests and the reviewer tries to ensure that the test coverage is good. This is a bit of a subjective thing at the moment as we don't yet use coverage tools (and they have limits as well). Things will be missed, that's just the nature of it.

When someone contributes a bug fix, there is evidence that a test would have been useful so we take that into account.

When someone files a PR, it's worth being aware that reviewer time is also limited. Your PR is more likely to be merged faster if it fits the project guidelines.

I hope that instead of a growing number of bug fixes, we write code with less bugs. We will see!

OK, that's all I have to say for now. :)

@ijuma Thanks again for the review and explanation. I have added unit test. Can you take a look?

Thanks @zhuchen1018, a couple of minor comments. Looks good otherwise. Will merge as soon as they are addressed.

LGTM


Does not work with external monitor unless useMouseMoveEvent variable is set to true on like 719

For some reason, the useMouseMoveEvent variable must be true so the "move a pix move a mickey" logic runs. Is there a drawback to running this logic every time the moveMouse method is called?

It lacks precision, the cursor won't move as smooth, but as far as I know, that's it. The option is there to be used if it does not work else. So use it if you need it.

I have been using it all day with no issues. Thanks for putting this together. It is very helpful. Works much better than stock Windows mouse keys. Thanks!


Add PPP

#207

Codecov Report

Merging #220 (4303561) into master (7e18635) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #220   +/-   ##
=========================================
  Coverage     38.51%   38.51%
  Complexity      528      528
=========================================
  Files           125      125
  Lines          3191     3191
  Branches        409      409
=========================================
  Hits           1229     1229
  Misses         1781     1781
  Partials        181      181

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Navigation gets clipped

There should be a total of six files within my last subdirectory, but generally only the first five show up.
I am using the latest stable version of Chrome on Windows 7.

Can you link your project's repository?
Related to #1184 #1183

https://github.com/excessive/DOMinatrix/tree/documentation
http://dominatrix.readthedocs.org/en/latest/
I'm not really sure what Sphinx is?

I'm sorry for confusing you.
This is http://sphinx-doc.org/ what I meant. It's the docs tool used in the Python ecosystem (and sometimes in others).
Mkdocs is fine too. I'm not too familiar with, that's why give me a weekend time to research your problem :)

Cool, thanks!

Is this happening locally with your mkdocs build? This looks like it could be an issue with the mkdocs theme.

I don't use mkdocs locally, I let rtd handle all of that. I am not home
right now but you could try checking out my repo and building it locally to
test.
On Mar 14, 2015 7:49 PM, "Anthony"<EMAIL_ADDRESS>wrote:

Is this happening locally with your mkdocs build? This looks like it could
be an issue with the mkdocs theme.
—
Reply to this email directly or view it on GitHub
https://github.com/rtfd/readthedocs.org/issues/1185#issuecomment-80750192
.

While the wonky scroll bar is still an issue, the actual scrolling of the navigation seems to be working now.

Does the wonky scroll bar happen on your local build? I have a feeling this is a theme issue on mkdocs, though could be wrong.

Closing this due to inactivity. If this is still an issue with the latest mkdocs release, feel free to reopen.

This was fixed in MkDocs 0.12 or 0.13 I believe. The screenshots are not loading so I can't be sure tho'


docs: add german translation

PR Checklist

This will add german translation for the website and also for some first doc pages.
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
[optional] What gif best describes this PR or how it makes you feel?

Currently I translated only all website strings and a few docs pages. I'm not sure if I should already translate the full docs in one PR or if it should be multiple smaller PR since that will be easier to review for you guys.

Smaller PRs for other sections would be good 👍

Thanks! @allcontributors add @DerHerrGammler for doc,translation


React nhost client connected to wrong local graphql service

Describe the bug
My react app cannot connect to the locally running nhost stack. There's been some nhost upgrades, and it looks like it's using the wrong URL, but setting it manually doesn't change.
To Reproduce
Steps to reproduce the behavior:

Start the nhost stack
Start the react app
The react NhostClient subdomain is set to localhost
See error

Expected behavior
It should be connecting to https://local.graphql.nhost.run
** Versions:**
"@nhost/nhost-js": "^2.2.13",
"@nhost/react": "^2.0.28",
"@nhost/react-apollo": "^5.0.32",

I tried setting the graphql path manually, to no effect:
export const nhost = new NhostClient({
  subdomain: "localhost",
  graphqlUrl: "https://local.graphql.nhost.run/v1",
});

It's local not localhost


_getRootDomNode runs before  template inserted into document

I've just upgraded to the latest version of flow-router (1.16.1) and now I'm having a very basic problem. I have a template that sets my overall site layout inside a  tag, then I use FlowLayout.setRoot('main') to target the element I want to render my layouts inside of. This was working fine with flow-router (1.9.0), but now it seems that it FlowRouter._getRootDomNode is called before the body template is inserted, or something is causing that initial template to be cleared out.

  LOGO

<template name="appLayout">
  {{> Template.dynamic template=main }}
</template>

// /client/layouts/app-layout.js
FlowRouter.setRoot('main');

// /lib/router.js
FlowRouter.route('/', {
  name: 'root',

  action: function(params, queryParams) {
    console.log('Showing home page');
    FlowLayout.render('appLayout', { main: 'feed' });
  }
});

Result
Error: Root element does not exist
    at Object._getRootDomNode (layout.js:92)
    at layout.js:12
    at Object.Meteor.startup (startup_client.js:63)
    at Object.render (layout.js:11)
    at FlowRouter.route.action [as _action] (router.js?208806fd262b2b20e3e6a2bef0b15ca6bb5ad678:25)
    at utils.js:191
    at runMiddleware (utils.js:191)
    at Route._processMiddlewares (utils.js:191)
    at Route.callAction (utils.js:191)
    at utils.js:191

I've tried to debug this on my own but haven't sorted it out yet, so I figured I'd ask, just in case I'm doing something totally wrong.

After your first mention of FlowLayout.setRoot your refer to using FlowRouter.setRoot twice, even in your code example. I know you mean the former, just pointing it out for anyone who doesn't notice it right away.
I checked and I get this error in versions 1.16.x, but not in 1.14.0 and 1.15.0. The $(root) part doesn't return a real DOM element so its get(0) call returns undefined. I can then run FlowLayout.render() from the browser console and it works fine.

Thanks for testing with the other versions. I tried to follow through the logic flow to figure out where the problem occurs, but I ran out of time. Will revisit when I've got a moment.
My current hunch is that the FlowRouter.render is somehow ending up on the Meteor.startup queue before the Template.body.renderToDocument is getting on there.

Hmm. Good Idea. We may need to it at the end of the Meteor.startup. That'll fix this. I'll give this a try to fix it today.

Thanks for looking into this @arunoda. I made a very simple app that reproduces the bug that could possibly be of some help to anyone investigating this further.
https://github.com/fightingtheboss/flow-router-body-template-test-case

So this problem basically prevents you from using .setRoot. I dove deeper but none of my solutions yielded a satisfactory result. I thought about binding the FlowLayout.render to Template.body.onRendered, but that broke all the tests and is limiting in the general case where you don't have DOM nodes you want to render to on the body template.
The best solution would seem to be somehow delaying the FlowRouter.initialize (moving it further down in the Meteor.startup queue) or the FlowRouter.render until after Template.body.onRendered is called, but I don't know how to control that load order specifically in the general case. I guess if the app was built with the "everything is a package" paradigm, you could define the load order.
The workaround for now is to avoid .setRoot, don't put DOM nodes in the body template and have a "parent" layout for the shared chrome for your app.
Thoughts?

@fightingtheboss I think we can fix with without user involvement. @pahans  Just did a fix(see above) on the and I merged.
Can you check this with the master branch of flow-layout.
If it solves your issue. I can do a release.

Looks good to me! I forgot that Meteor.defer is essentially a setTimeout of 0, so it can be used on the client to move calls to the end of the event loop. Thought of it mostly as a backend trick. A valuable lesson, thank you!
Thanks for the fix, @pahans

Great. I will do a release.
On 2015 ජූනි 29, සඳුදා at ප.ව. 6.46 Mina Mikhail<EMAIL_ADDRESS>wrote:

Looks good to me! I forgot that Meteor.defer is essentially a setTimeout
of 0, so it can be used on the client to move calls to the end of the event
loop. Thought of it mostly as a backend trick. A valuable lesson, thank you!
Thanks for the fix, @pahans https://github.com/pahans
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/flow-layout/issues/25#issuecomment-116655701
.

Released a  new version 1.4.1

hi. i'm using the version<EMAIL_ADDRESS>and<EMAIL_ADDRESS>While trying the app, i faced the same error. In this version, something changed ?
BlazeLayout.setRoot("main");

FlowRouter.route('/', {
  action: function(params, queryParams) {
    BlazeLayout.render("dashboard", { main: "dashboard_main_content" });
  }
});

Thank you in advance.
Best regards.

@victorkurauchi Looks like you need a selector tag ('.' for class or "#" for id attributes).  Did you try doing
BlazeLayout.setRoot("#main");

If that doesn't solve it, did you remove the "meteor-platform" package? I had the same issue, so I followed the trace back to a jQuery function call; I had removed the meteor-platform package, removing jQuery along with it.


Tab location

Description of the new feature/enhancement
Allow tabs to be placed left, right, top or bottom.
With default "top".
Proposed technical implementation details (optional)

This probably classifies as a duplicate of #835.

This one specifies any of the sides to be configurable.
The #835 is specific only for the bottom side.
So, #835 is subset of this one. :)

Although I'm really inclined to close this one as a dupe, I'm thinking that the work for "tabs on the side" is different enough from "tabs on the bottom" that we can leave this open.

I'm inclined to merge them. :smile:

/dup #835


Power BI desktop not updating with the latest version

Describe the bug
Even if I reinstall Report Analyzer the External Tool keeps using version 1.6.0 instead of 1.6.2

This shouldn't happen. Please uninstall & reinstall report analyzer.

Hi,
Yes, tried this but didn't fix the problem.
Thanks
On Mon, 14 Nov 2022, 11:17 m-kovalsky, @.***> wrote:

This shouldn't happen. Please uninstall & reinstall report analyzer.
—
Reply to this email directly, view it on GitHub
https://github.com/m-kovalsky/ReportAnalyzer/issues/16#issuecomment-1313430497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANG6J3KRWHI6BB2RYF6TA53WIIGUTANCNFSM6AAAAAAR7UR6HU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Deleting Business Opps external tool fixed this.


Implementation of Conversion Infix  to Postfix using Stack

C program to implement infix to postfix conversion using stack

Source code for infix to postfix conversion using stack in C


Is this paper accepted by NIPS Workshop or NIPS?

In your arXiv version, it shows that this paper is accepted by NIPS Workshop 2017, while the current README of this project shows that this paper is accepted by NIPS.
NIPS and NIPS Workshop are quite different. The citation section in the current README is misleading and should be corrected?

Hi @Chen-Qiu ,
Sorry for the misleading. This paper was accepted by NIPS but was withdrawn by some mistakes made by us. So we re-submitted it to the workshop session. README has been revised. Many thanks for reminding us.

Very thanks for your kind reply :)


can-serve is not correctly proxying (1)

Issue by matthewp from Thursday Aug 20, 2015 at 22:34 GMT
Originally opened as https://github.com/donejs/donejs/issues/55

We had to do:
io("http://localhost:7070")

The proxying is not working

Comment by daffl from Thursday Aug 27, 2015 at 21:03 GMT

After some rather disappointing trial & error I think proxying websockets with our setup is not possible. We might be able to add it when we have the ability for can.route to define 404s which then gets passed to the proxy but that means that alls non-matching requests will be proxied.

This is fixed in latest can-ssr


Shards Thing

Isn't possible do a script to check shards and restart the bot if the counts isn't appropriated?
Like auto do the .shards thing if the result is less than a defined percentage it restart the bot? Maybe something to force a defined uptime ?!
Actually can go deeper and make check's to guilds, voice connects (when using the music module - it actually exist I think?!).

No, it's normal for shards to reset connection,, if the check triggers when they're down, it will needlessly restart.  Anyway shards issue will be hopefully gone when i upgrade public bot to 1.4x


multithread support

Dear team,
I saw that the openmp library was used/linked for multithread support, but I cannot see an option to use multiple threads. N =0 or 1, I see no difference.
Thanks,
Jianshu

Hi Jianshu,
Thanks for reaching out! Just as you mentioned, running the code with N=0 or with N=1 provides no difference, there is a note about this in the readme.
If you want to use multiple threads you need to choose N>1, e.g. N=2 will run using 2 threads. A good idea would be to se N equal to the number of cores available in your machine o cluster.
Regards,
Felipe

Thanks!this is clear. Multithreaded run will be much faster right.


feat: support React Native 0.76

Summary

[x] - align CMakeLists to account for merged RN SO libs
[x] - updated both tester apps to 0.76-rc.6
[x] - updated codegen rules to use hermes parser
[x] - updated flow-remove-types to include necessary fixes for RN 76
[x] - add missing fusebox related method to DevClient

Test plan

[x] - tester app works with old/new arch
[x] - tester federation works with old/new arch

flow-types still being removed incorrectly, this PR to flow-remove-types should fix it:
https://github.com/facebook/flow/pull/9213

[ ] - one more flow issue that needs to be tackled: https://github.com/facebook/flow/pull/9216
RNScreens on Android still seems to be failing build, works on iOS though

patched RNScreens using changes from this PR: https://github.com/software-mansion/react-native-screens/pull/2415, after that it builds correctly for Android 👍
all thats left is the flow issue and we should be g2g on this

Thanks for your work on this @artus9033 🎉

@jbroma I might be mistaken, these changes have not been back ported to 4.x, or?


How run this package

Hello
Thanks for this work.
I would like to try it on my PC, but there is no readme.
Could you provide something to help us?

I wrote the README. I'm really sorry for the delay.


downgrade WARN messages to INFO level

Scope & Purpose
Downgrade WARN messages to INFO level
because the failed operations will be retried and likely succeed later on.
Additionally, fix a compile warning for a potentially uninitialized
value in FunctionUtils.h. Finally, don't abort the server process if
the batch ping timer cannot be requeued in initial replication, as this
is not a critical error. Replication can still be retried, and even has
a chance to succeed without the batch ping timer being requeued.

[x] Bug-Fix for devel-branch (i.e. no need for backports?)

Testing & Verification
This change is a trivial rework / code cleanup without any test coverage.
https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/5783/

https://jenkins01.arangodb.biz/job/arangodb-matrix-pr-linux/7449/

https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/5798/

tests blue


Lane_extractConfig.h is not found

Deer engineer:
Could you please tell me more about how to generate files automatically？

Duplicate of #1


CHANGELOG.md Links Broken

Describe the bug
CHANGELOG.md: See Changelog/See Full Changelog links result in Page Not Found for all versions >= 2.19.3
To Reproduce
Steps to reproduce the behavior:

Go to Changelog v2.19.4.

Expected behavior
Should show the changelog of a version.

Looks like full changelogs are published under a maintainer's own github account (@marwahaha), and their account has (hopefully temporarily) been disabled.

Will you then be creating the new links for it or wait for @marwahaha's account to be enabled again?

@ichernev / @timrwood How is a 'full changelog' created?  Is this information lost forever if a maintainer's account gets shut down?

I think I've been re-enabled, sorry for the confusion!
@ashsearle the Moment release process is here:
https://gist.github.com/marwahaha/44deb629e81fbf02c599074ffc8a9c46
You can run this locally to get a changelog between two moment versions:
https://gist.githubusercontent.com/marwahaha/deb752493230f4785d32562d7aed8493/raw/


Performance issues when many rows

A few hundred lines of tab aligned columns makes Atom really really slow. It would be ideal if this was not the case. :)

Currently we recalculate the screen rows for each paint, it's normally only include visible lines in the screen (about 40 to 60 lines in typical screen monitor). So the cost would not relate to the line count of the file.
But the recalculation is based on DOM structure and measurement which is slow. This is the limitation of current implementation.

Cool. I guess this has to do with me working in 4k and also love having multi column JSONs formatted on screen. :)


How to execute npm install in a subdirectory?

I'm trying to execute npm install in a subdirectory before the release is symlinked, cause composer installs a package which requires it.
However, when I try
within release_path + '/foo/bar' do
  execute :npm, 'install'

It can't install, cause it only sees /foo/bar as the path... am I missing something?

Please see the mailing list at https://groups.google.com/forum/#!forum/capistrano for support.

Apologies, will look there.

No problem it is in the CONTRIB guidelines that people are supposed to see before opening an issue that we keep Github just for bugs and support happens on the list.


How to integrate with pkg

Hi, thank you very much for this package.
Could you provide an example of how to integrate with pkg?

Hi,
You can copy this .node node-hide-console-window.node in the same folder as your pkg binary.
This file is located in node_modules\node-hide-console-window\build\Release

Marked closed for lack of activity.


pvlib.pvsystem.retrieve_sam example has no attribute

I tried running example code from https://pvlib-python.readthedocs.io/en/stable/reference/generated/pvlib.pvsystem.retrieve_sam.html.  And it throws this error. AttributeError: 'DataFrame' object has no attribute 'AE_Solar_Energy__AE6_0__277V__277V__CEC_2012_'
Although I was just trying to get the example code working, what I really want is to get the solar modules available to choose from.

Hi @nativesci, thanks for pointing out this error in our documentation.  The CECmod and CECInverter tables (for PV modules and PV inverters, respectively) change over time as the CEC adds, removes, and renames entries.  It seems that the inverter name in that example code doesn't exist anymore in the CECInverter table currently included in pvlib.  We should change that example code to use an inverter name that is in the table.  Let's leave this issue open to track that documentation update.

what I really want is to get the solar modules available to choose from.

If you want PV modules (rather than inverters), you'll need to use pvsystem.retrieve_sam('CECmod').  This returns a table of parameters for the CEC single-diode PV module model, where each module is a column in the table.  That table is stored in a CSV file that you can also inspect with Excel if you like.  It can be downloaded here: https://github.com/pvlib/pvlib-python/blob/main/pvlib/data/sam-library-cec-modules-2019-03-05.csv

Thank you. I also want say that I would be happy to contribute to the documentation. Sent from my iPhoneOn May 18, 2023, at 7:01 AM, Kevin Anderson @.***> wrote:
Hi @nativesci, thanks for pointing out this error in our documentation.  The CECmod and CECInverter tables (for PV modules and PV inverters, respectively) change over time as the CEC adds, removes, and renames entries.  It seems that the inverter name in that example code doesn't exist anymore in the CECInverter table currently included in pvlib.  We should change that example code to use an inverter name that is in the table.  Let's leave this issue open to track that documentation update.
what I really want is to get the solar modules available to choose from.
If you want PV modules (rather than inverters), you'll need to use pvsystem.retrieve_sam('CECmod').  This returns a table of parameters for the CEC single-diode PV module model, where each module is a column in the table.  That table is stored in a CSV file that you can also inspect with Excel if you like.  It can be downloaded here: https://github.com/pvlib/pvlib-python/blob/main/pvlib/data/sam-library-cec-modules-2019-03-05.csv
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


clojure.repl/dir

Paren-soup is awesome! Is it possible to use "dir" function?

You can only run clojurescript functions in the console mode.


Make adding path() in urls.py optional

Hello again!
I discovered that I need to include the path to the API in the urls.py, even though I do not plan to use the API feature.
path("content-settings/", include("content_settings.urls")),

When I view a single content setting in the admin, eg: admin path ending in content_settings/contentsetting/1/change/, there is a handy API path hint Fetch one setting.
Currently with v0.10, if I omit the urls.py line, the display of the setting is broken, presumably due to being unable to determine the api path shown in Fetch one setting.
If the API urls.py is not referenced, perhaps you could omit the Fetch one setting display which would make specifying the path to the API truly optional?

hello again
thank you, that makes a lot of sense.
need to think how to write tests for that case.
one more testing project, probably

will be released in 0.10.1


MM-14305: Autofocus text input if external keyboard is attached

Summary

Adds a call to the native Obj-C method GCKeyboard *coalescedKeyboard to query whether an external keyboard has been connected. If so, the post input is automatically focused every time a new channel or thread is opened, similar to the behaviour on the web app.
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/11811

Device Information
This PR was tested on:
iPad Mini, iOS 15.5
Release Note

Auto-focusses text input if an external keyboard is connected to iPad.

Hello @glocore,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!