Unable to get User Information

Unable to get user information
Code used to reproduce
import { LeetCode } from "leetcode-query";

const leetcode = new LeetCode();
const user = await leetcode.user("username");

Error
Uncaught (in promise) TypeError: Failed to execute 'fetch' on 'Window': Illegal invocation
    at _fetch (index.js:71:1)
    at get_csrf (index.js:87:1)
    at Credential.init (index.js:111:1)
    at getUser (Card.js:10:1)
    at Card (Card.js:15:1)
    at renderWithHooks (react-dom.development.js:15486:1)
    at mountIndeterminateComponent (react-dom.development.js:20103:1)
    at beginWork (react-dom.development.js:21626:1)
    at beginWork$1 (react-dom.development.js:27465:1)
    at performUnitOfWork (react-dom.development.js:26596:1)

Help me with this. @JacobLinCool

Hello. Which environment did you run it on? I tested it, and the code works on Node, Deno, and Bun without any problems.

facing a similar issue, this is on nextjs

Hello. Which environment did you run it on? I tested it, and the code works on Node, Deno, and Bun without any problems.

I tried to run the code in React, and I was facing error, the react boilerplate code was created using create-react-app.

Ok, please check the following:

It may not be possible to use it in the browser due to the CORS problem; please run it in the server environment instead.
If you still encounter this issue on the server side, please try using a custom fetcher:

import { LeetCode, fetcher } from "leetcode-query";

// use a custom fetcher
fetcher.set(fetch);

// use as normal
const lc = new LeetCode();
const daily = await lc.daily();
console.log(daily);


clean build fails on installed tools

$ devicescript build
using devs: v1.11.4, runtime: v6.5.0, node: v18.12.1 from /Users/peli/test/node_modules/@devicescript/cli/built
exception: (0 , import_compiler4.resolveBuildConfig) is not a function

fixed


Solving some typos in the spanish translation

PR Details

Description

Related Issue

Motivation and Context

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

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

I'll have to reject it because you change the whole files. Please make sure to not concert line endings or encoding.
Also consider contributing to the Weblate project directly instead. That will ensure that you modify the latest version and keep in sync with every other contributors.

@xen2 I don't see a link to Weblate in the readme. We should add one.

I can't edit the locale files in my favourite text editor? :(

For the next time what kind of text encoding must I have to use?

@meriaizen86 you can edit locally. But then we can run in this kind of situation were the files have not been properly modified.
Note that it could happen for any source files. Make sure that your git config is correct, especially core.autocrlf must be set to true.
See https://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration#_formatting_and_whitespace and https://help.github.com/en/articles/dealing-with-line-endings

Unfortunately as @Kryptos-FR mentioned, I can't merge it if it introduce CRLF changes (it would confuse tools such as weblate).
Please either fix the CRLF or do the changes directly in weblate.

Because the PR was created out of the master branch of @meriaizen86 it is now getting out of hand. It cannot be salvaged so I'm closing it.


Fix build when JSON_USE_EXCEPTION == 0

🦟 Bug fix
Summary
Newer versions of jsoncpp do not provide exception classes when JSON_USE_EXCEPTION is off.
Updating the #if directives in JSONParse.cc allows this file to compile against jsoncpp configured to not throw.
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@vslashg can you sign off your commits for the DCO checker?  Instructions are here: https://github.com/gazebosim/gz-fuel-tools/pull/309/checks?check_run_id=9551648415

@vslashg friendly ping

Thanks for the contribution @vslashg. I'm moving this work to #332.


Verify that "Budeni" is listed in the dropdown menu when searched

Description:
"Budeni" should appear in the dropdown menu with correct spelling and locality information.
Preconditions:
The website is up and running.

Step 1
Type "Budeni" in the search bar.
Expected results:
The dropdown is listing "Budeni" as the first location in the dropdown menu.

Step 2
Observe the dropdown menu.
Expected results:
"Budeni" is displayed and selectable from the dropdown menu.


fix links

Fixes #2059
@Rick-Anderson

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnetcore/client-side/yeoman.md
:white_check_mark:Succeeded

Details

aspnetcore/fundamentals/hosting.md
:white_check_mark:Succeeded

Details

aspnetcore/migration/mvc.md
:white_check_mark:Succeeded

Details

aspnetcore/mvc/views/tag-helpers/authoring.md
:white_check_mark:Succeeded

Details

aspnetcore/mvc/views/working-with-forms.md
:white_check_mark:Succeeded

Details

aspnetcore/publishing/iis.md
:white_check_mark:Succeeded

Details

aspnetcore/security/authentication/azure-active-directory/index.md
:white_check_mark:Succeeded

Details

aspnetcore/security/authentication/index.md
:white_check_mark:Succeeded

Details

aspnetcore/security/authentication/sociallogins.md
:white_check_mark:Succeeded

Details

aspnetcore/tutorials/first-mvc-app/adding-controller.md
:white_check_mark:Succeeded

Details

aspnetcore/tutorials/first-mvc-app/validation.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update requirements for top-level "included"

From the consortium meeting on 26.11.2019 we have decided the following:
If a related resource is of type references it SHOULD be included under the top-level included field.
For any other related resource types it MAY be included under the top-level included field.
There is as of yet no consensus on limitation on how many resource objects may be returned under included.
There is also no consensus on whether we want to use the include query parameter.

Tagging main contributors to the discussion if there are any additions: @fawzi, @rartino, @ml-evs, @sauliusg, @merkys

There is issue #183 opened to discuss the support of JSON API's include query parameter.

My opinion:

I'm OK with references having a special role. As discussed in the Web meeting, a user most likely will want to get them anyway.

Concerning the limitation on included, JSON API seems to forbid limits on inclued.

Concerning the include and included I have the following proposal: let's omit the included if include is not present, and include only those entry types that are explicitly requested by include. This would reduce the load of the most usual queries, but still allow inclusion upon requests. And this proposal seems to agree with JSON API specification.

What do you think?

@merkys, so to sum up, from the POV of the OPTiMaDe spec, what I understand by your points (just to ensure we're on the same page):
The top-level included field MUST/SHOULD be omitted unless the include parameter has been specified with a valid value.
And then either:

The default value for include, if it is not supplied explicitly, is include=references; or
The first rule should be foregone if any primary data return a relationships of the type references. Relationship data of type references MUST/SHOULD always be passed under the top-level included field, if present.

@CasperWA, now I see the conflicting statements I've made :)
So when include is provided, we should follow the JSON API and show only what is requested.
And there are two alternative ways to deal with the implicit case, i.e., when include is not provided:

Omit included altogether;
Act as if include=references was supplied.

I have a feeling (can't prove it, though) that 2. might be against JSON API spec., as presence of included most probably makes a response a compound document, and then all referenced objects MUST be in included. But I may be wrong. Then option 2. would be OK.

So when include is provided, we should follow the JSON API and show only what is requested.
(...)
I have a feeling (can't prove it, though) that 2. might be against JSON API spec., as presence of included most probably makes a response a compound document, and then all referenced objects MUST be in included. But I may be wrong. Then option 2. would be OK.

Considering these two things, it seems to me that when include is provided, only relationships of the type included in the value for include should be returned under included (convoluted sentence, sorry).
I.e., if we have include=references (option 2.) being the default, then no matter what kind of relationship-types there are in the primary data, only those of type references should be returned under included, which is compliant with the JSON API defintion of the include parameter. Right? :)

Considering these two things, it seems to me that when include is provided, only relationships of the type included in the value for include should be returned under included (convoluted sentence, sorry).

My interpretation is (and would sound) the same :)

I.e., if we have include=references (option 2.) being the default, then no matter what kind of relationship-types there are in the primary data, only those of type references should be returned under included, which is compliant with the JSON API defintion of the include parameter. Right? :)

I meant option 2. being the default when include is not provided: the API lists references in included if include is not given.

Great! I think we can try putting this into the spec. now :+1:


Add my first file

I have created a branch, created a file, and made a commit!

Add my first file


Open pipeline editor in edit mode if used environments by the pipeline are build not all environments of a project

Describe the problem this improvement solves
When opening a pipeline of a project, it will first check whether all environments have been build. This means that if an environment takes a very long time to build it will prevent the entire pipeline from being opened in edit mode.
I think this is acceptable in most cases, since you won't be able to run your pipeline without its environments being build.
Describe the solution you'd like
Instead of checking whether all environment have been build, we should check whether all environments that the pipeline uses have been build. In a project it can very well be that a pipeline doesn't use all environments.
Cons:

The kernels that are shown in jupyterlab will now also show kernels that won't actually start.

Screenshots

Final notes
I am not completely sure about this improvement due to the listed cons. We might make it faster in certain cases but possibly also introduce bugs. I think I would rather stay on the safe side and not optimize startup. Keen to hear other people's thought.

This improvement only makes sense if users tend to have multiple pipelines in one project and use multiple environments instead of one huge environment with everything included.

Another case where the dialog pops up is if you add a new environment to a project (even if the session is already active). This feels pretty intrusive imo.


Interpolation along dimension with a single element

Let me consider a DataArray with a dimension containing a single element:
In [1]: import numpy as np

In [2]: import xarray as xr

In [3]: da = xr.DataArray(np.linspace(0.0, 100.0, 101).reshape(1, 101),
                                      coords=[('x', [1.0]), ('y', np.linspace(0.0, 1.0, 101))])

Asking x=1.0 in interp fails.
More precisely, the 2D interpolation returns a NaN
In [5]: da.interp(x=1.0, y=0.4)
/opt/anaconda3/lib/python3.7/site-packages/scipy/interpolate/interpolate.py:2533: RuntimeWarning: invalid value encountered in true_divide
  (grid[i + 1] - grid[i]))
Out[5]:
<xarray.DataArray ()>
array(nan)
Coordinates:
    x        float64 1.0
    y        float64 0.4

and the 1D interpolation returns a ValueError:
In [6]: da.interp(x=1.0)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-6-efd4490bc97e> in <module>
----> 1 da.interp(x=1.0)
...
ValueError: x and y arrays must have at least 2 entries

I understand why the interpolation is impossible in an array with a single element.
However, this behavior breaks my assumption that interp is a superset of sel, meaning that it gives the same result as sel for all inputs that are valid inputs for sel (except for a possible int to float type conversion).
Here, I would expect da.interp(x=1.0) to give me the same result as da.sel(x=1.0).
What do you think of this corner case?

Sorry this never got a response @mancellin .
Do people think this is an issue? I'm not a regular user of interp. Does sel & nearest cover the case?

No worry, I had actually forgotten about this.
In a more recent version (2022.3.0), both 1D and 2D case return the same ValueError with a relatively clear message x and y arrays must have at least 2 entries.
I'm still proposing that any valid code with sel should behave the same with interp, and thus that interp should work when a dimension has a single entry. But this is a low priority issue with easy workarounds.


[RFC][useButton][base] tabIndex should be returned by getRootProps if explicitly provided

What's the problem? 🤔
The problem lies in this section of useButton's implementation:
There are some conditions under which a tabIndex value will come in getRootProps, and some conditions under which it won't, for example:

It won't come when the host element is a BUTTON
It will come when using it for a link and the link is disabled

This makes for a confusing API when the user provides a tabIndex:
const { getRootProps } = useButton({ tabIndex: 1 });

// button won't get tabIndex=1 even though explicitly provided
<button {...getRootProps()}>Button</button>

What are the requirements? ❓
getRootProps return value should contain the tabIndex if explicitly provided
What are our options? 💡

Not receiving a tabIndex, if the user wants to provide a specific value it would do so directly to the button element. The user can also decide to do so depending on if getRootProps comes with a value for tabIndex or not.
Keep implementation as is and document this behavior

Proposed solution 🟢

If no tabIndex value is provided, maintain the current behavior
If a tabIndex value is provided, that will come in getRootProps, except if focusableWhenDisabled is true here, in which case it will be -1

I think this implementation should achieve it:
  const buttonProps: AdditionalButtonProps = {};
+ buttonProps.tabIndex = tabIndex;

  if (hostElementName === 'BUTTON') {
    buttonProps.type = type ?? 'button';
    if (focusableWhenDisabled) {
      buttonProps['aria-disabled'] = disabled;
    } else {
      buttonProps.disabled = disabled;
    }
  } else if (hostElementName !== '') {
    if (!href && !to) {
      buttonProps.role = 'button';
      buttonProps.tabIndex = tabIndex ?? 0;
    }
    if (disabled) {
      buttonProps['aria-disabled'] = disabled as boolean;
      buttonProps.tabIndex = focusableWhenDisabled ? tabIndex ?? 0 : -1;
    }
  }

Resources and benchmarks 🔗

React Aria's useButton works as the proposed solution: https://github.com/adobe/react-spectrum/blob/main/packages/%40react-aria/button/src/useButton.ts (It would merge the provided tabIndex on line 101)

Added PR for proposed solution


In InviteMembers page when we search 998243aaaa we can invite  +529982432222

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to Invite Members page in workspace
Enter 998243aaaa

Expected Result:
This should show No results found
Actual Result:
In the list we can invite +529982432222
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @ShogunFire
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688428528032189
View all open jobs on GitHub

This exists in regular search too from what I can see

Also reproduced on Invite new members:

Proposal
Please re-state the problem that we are trying to solve in this issue.
When we search a string beginning with numbers but also conatining letters sometimes they are recognized as phone numbers
What is the root cause of that problem?
It happens because of the way the library awesome-phonenumber works.
What changes do you think we should make in order to solve the problem?

I propose that we make a new function that has the same goal as parsePhoneNumber and return the same thing but add a condition that the search value must be composed of numbers and + , we can then call this function everywhere where we are currently calling parsePhoneNumber
What alternative solutions did you explore? (Optional)

Will review this in my morning

Not able to reproduce

Hey sorry the steps have changed a bit

@ShogunFire can you share the steps?

It depends of what country you are in and what is a valid number there, I think for the us the first number should be 1. Otherwise I think that you can put the international code and still reproduce. For example try with +52998243aaaa

@ShogunFire thank you i am able to reproduce it now.

Proposal
Please re-state the problem that we are trying to solve in this issue.
phone number with letters will show a phone number
What is the root cause of that problem?
letters in phone numbers are considered as possible from awesome-phonenumber library then we're converting it to numbers here if it is possible:
https://github.com/Expensify/App/blob/60e73fa3d5b5ddeb042d2d532fbc110ec40a2f22/src/libs/OptionsListUtils.js#L637 since it's recognized as possible it will convert it to numbers format. but previously it was decided not to be a valid phone number here we have fixed similar issue https://github.com/Expensify/App/issues/21247

Maybe, but we don't translate +1-212-456ww7890 into +1-212-456997890. Either way, that's incorrect!
So there are two potential solutions: reject non-numeric phone numbers (ie: show an error when you enter +1-212-456ww7890) or translate the letters into numbers (ie: translate +1-212-456ww7890 into +1-212-456997890`.

What changes do you think we should make in order to solve the problem?

We should not convert the original value to make the results consistent across all forms, then we should validate the phone number using Str.isPhoneValid by removing + from the value.
we should add this condition to validate the phone number further
(parsedPhoneNumber.possible && Str.isValidPhone(searchValue.slice(1))

here:
https://github.com/Expensify/App/blob/60e73fa3d5b5ddeb042d2d532fbc110ec40a2f22/src/libs/OptionsListUtils.js#L789
Result:

What alternative solutions did you explore? (Optional)

How does the above proposal look to you @abdulrahuman5196 ?

@abdulrahuman5196 There are actually 3 proposals to review

haha yes, apologies there are multiple. Thanks for the correction @ShogunFire

There are some proposals here for you to review @abdulrahuman5196 👀

Reviewing this now

Its seems all the proposals here have same root cause and relatively same solution. So I am going with the first proposal.
@GItGudRatio 's proposal here https://github.com/Expensify/App/issues/22521#issuecomment-1627662849 looks good and works well. Instead of that regex mentioned in the proposal we can use Str.isValidPhone in the PR.
🎀 👀 🎀
C+ Reviewed

@amyevans C+ approved comment is here - https://github.com/Expensify/App/issues/22521#issuecomment-1644186659

@abdulrahuman5196 i exactly suggested to use Str.isValidPhone() while the selected proposal mentions new regex.
https://github.com/Expensify/App/issues/22521#issuecomment-1636289339

@abdulrahuman5196 i exactly suggested to use Str.isValidPhone() while the selected proposal mentions new regex. and i see you suggested to go with my approach in the PR it's not fair.
#22521 (comment)

I think of it as more of a PR comment, Even if we just had the accepted proposal. We could asked to use some common functions or create a common functions. So I am unable to consider yours as a different proposal. But If you still have concerns, internal engineer also will review this and they can re-assign if they think otherwise.

I think of it as more of a PR comment, Even if we just had the accepted proposal. We could asked to use some common functions or create a common functions. So I am unable to consider yours as a different proposal. But If you still have concerns, internal engineer also will review this and they can re-assign if they think otherwise.

even the regexes are different, the comment suggested to change from the const regex to existing function. if that wasn't considerable the change won't be necessary in the first place.

These are my humble opinion on your concern @getusha

selected proposal mentions new regex

I think the intention was to test phone number validity. Even If your proposal wasn't provided, reviewers might have pointed out to use a common existing function to achieve the same purpose.

even the regexes are different, the comment suggested to change from the const regex to existing function. if that wasn't considerable the change won't be necessary in the first place.

If we have to consider proposals at that minor level, even the exact proposal you had provided is also not perfect and not working properly on all cases.
For example,

(parsedPhoneNumber.possible && Str.isValidPhone(searchValue.slice(1))

We shouldn't slice here since the searchValue already doesn't have a '+'. And moreover the regex in the proposal actually accepts '+' start as well, so not sure why we have to slice https://github.com/Expensify/expensify-common/blob/fea1a755affabdbb8b6d7c5b754241849124f488/lib/CONST.jsx#L545. And even if i just copy paste your proposed change it was not working for the input in the OP.
But in a scenario if your's where the first proposal and the next contributor's proposal made these PR comment level modifications and requested to consider it as a different proposal. I would still have considered your proposal as the valid one.
I have just provided the above scenario as example.
Note: All these are just my opinion, If others feels different, I am fine with that decision as well.

Thanks all for your proposals and input here. And sorry, I know that sometimes the outcome can be frustrating. But I agree with @abdulrahuman5196's assessment, this is a straightforward issue where everyone has correctly identified the root cause, so out of fairness and consistency it's best to go with the first proposal. We discourage getting too much into the weeds of code diffs in the proposal stage - that is best saved for the PR implementation stage.
Assigning @GItGudRatio

@GItGudRatio Kindly let me know once the PR is ready.

Really sorry for the delay, had a family issue

No worries. Hope everything is fine. @GItGudRatio

Reminder added to calendar for eventual payment. Let's square off the BZ checklist above in the meantime @abdulrahuman5196 !  🚀

I think @MitchExpensify  is OOO, but bump for Monday 😄

https://www.upwork.com/freelancers/pierremichel5

Great! Thank you both
@GItGudRatio paid and contract ended
@ShogunFire offer sent!

All paid! Thanks eveyone


webviewx black screen in mobile view

I am trying load local asset html from assets folder in flutter and while doing so it is working fine in web, thanks webviewx team but while viewing in mobile I am seeing black spot on top and html page is not rendering at all due to import issue
`
import 'package:flutter/material.dart';
import 'package:webviewx/webviewx.dart';
class FormDetailPage extends StatefulWidget {
@override
_FormDetailPageState createState() => _FormDetailPageState();
}
class _FormDetailPageState extends State {
WebViewXController webviewController;
String title = 'Form Page';
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
title: Text(title),
),
body: WebViewX(
javascriptMode: JavascriptMode.unrestricted,
initialContent:
' Loading  Loading ',
initialSourceType: SourceType.HTML,
onWebViewCreated: (controller) {
webviewController = controller;
//_loadHtmlFromAssets();
},
),
);
}
}`

Hi, are you sure this is a problem coming from webviewx? I didn't encounter this behaviour so far.
Recently I have updated dependencies, including webview_flutter's version. Maybe give it a try again now to see if that was the issue.
If that does not fix the problem, please post the full code (including the html page you're using) for further investigation.
Thanks

Yes I have check again and still I am getting back spot but only in case of mobile device and I am using emulator to test.
I am using latest versin as well. webviewx: ^0.0.4. For now I am using two plugin for worekaround. But if you fix this issue it would be a great help.
Here is index.html from assets (templete from w3school)

Sorry, I should've been more explicit. By "give it a try now" I actually meant you should try to use the webviewx package from the main branch here, on github, instead of using it from pub.dev.
This is how it should be done:
dependencies:
  webviewx:
    git: https://github.com/adrianflutur/webviewx.git

I didn't publish a new version yet because there are a few things that need to be addressed before I can increment the version to 0.0.5 and upload to pub.dev.

I tried and ot this error,
`/C:/src/flutter/.pub-cache/git/webviewx-5f95a924df48f6691e9acd3d61d0017780b3abc6/lib/src/view/mobile.dart:157:39: Error: A value of type 'WebViewController?' can't be assigned to a variable of type 'WebViewController' because 'WebViewController?' is nullable and 'WebViewController' isn't.

'WebViewController' is from 'package:webview_flutter/webview_flutter.dart' ('/C:/src/flutter/.pub-cache/hosted/pub.dartlang.org/webview_flutter-2.0.8/lib/webview_flutter.dart').
webViewXController!.connector = originalWebViewController;`
I believe something to do with null safety

Yes, it's related to null safety. That's why I can't publish it as 0.0.5 yet 😁
Meanwhile, please add an if-not-null check inside onWebViewCreated and set your controller inside it.

@CodingWithTashi Hi, you might want to try again now.
https://github.com/adrianflutur/webviewx/pull/14

@adrianflutur seems working fine in real devices, but for emulator still I am getting black screen. anyway we will be using real device end of the day. it would be fine I think.

@CodingWithTashi Thats good,
Yes, adding desktop support is on the list, but it's somewhat a long-term thing because platform-views are not yet implemented for theese platforms.
In fact, Linux platform-views are currently waiting to be merged here https://github.com/flutter/engine/pull/26288 , so maybe it's not going to take so long after all :)
Also closing this now, so please open a new issue if you have any other problems. Thanks!


Build....

How can I publish it as an exe?

You can't, it's an UWP application; the closest to an exe you'll get is an appx.


Feat/predicate cost

Implements #320.

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


Bug

When opening discord it crash
VD-Manager-1712555570575.log

why are you trying to install vendetta 😭

Try BunnyManager


Support Windows 10 builds

Windows 10 includes an option to install a beta-version Windows Subsystem for Linux, which  is a full Ubuntu Linux image, obviating the need for virtualization or emulation.  I'm submitting this issue to track progress toward building OpenCoarrays inside the Windows 10 Subsystem for Linux.
Based on a fresh installation of Windows 10, I have verified that Microsoft's instructions for installing Ubuntu work.  After installing Ubuntu on Windows 10, the following script progresses to the point of downloading and uncompressing the GCC 6.1.0 tar ball:
#/bin/bash

sudo apt-get install g++  # As of October 2016, this installs gcc 4.8

# Install GCC prerequisites:
sudo apt-get install automake   # As of October 2016, this installs m4 1.4.17 and aclocal 1.14.1
sudo apt-get install flex       # As of October 2016, this installs flex 2.5.35
sudo apt-get install bison      # As of October 2016, this installs bison 3.0.2
sudo apt-get install texinfo    # This installs makeinfo as required for building GCC documentation

if [ ]; then
  echo "Oops... the conditional statement that blocks broken steps didn't work"
  exit 1
  sudo apt-get install mpich    # Windows Ubuntu subsystem claims this is already installed but I can't find it
  sudo apt-get install cmake    # As of October 2016, this installs cmake 2.8 (OpenCoarrays requires >3.4.0)
  sudo apt-get install gfortran # As of October 2016, this installs gfortran 4.8 (OpenCoarrays requires >6.1.0)
fi

./install.sh --yes-to-all

I'll post more as the build progresses.   If this approach succeeds, we can drop the minimum required  flex version and bison version to those that apt-get currently installs on Windows 10.   These are flex 2.5.35 and bison 3.0.2 as noted in the above script.

So far, so good.  The installer has progressed through the GCC build, which succeeds but is exruciatingly long.  I didn't run the build continuously in one sitting and had to put the laptop to sleep several times so it's hard to estimate how long it took, but I'm guessing it was on the order of 8 hours, give or take a couple of hours.   @zbeekman, this probably makes the strongest case for having a binary payload -- at least for Windows.   Any chance of reviving that idea for the 2.0 milestone?

Hmmm.... a binary payload is an interesting idea... In the past, though, I think we were mostly talking about:

A self extracting install script (you can append a binary compressed tarball to a script that will extract the tarball from itself, then run an install script within the tarball
Homebrew and Linuxbrew "bottles" for OpenCoarrays

I actually was working on setting up automatic building of MPICH against the latest stable GCC via Homebrew, then uploading to bintray to host the binaries using Travis-CI, nightli.es and bintray. (This way I don't have to manually rebuild MPICH bottles to use for our Travis-CI overtime a stable GCC or MPICH is released or the "formula" is revised/patched-it will be taken care of automatically. Without the custom MPICH bottles, MPICH bottles distributed with Homebrew use clang as the C compiler.)
It is possible that we could do something similar with OpenCoarrays (pretty easy, could use Travis-CI, once the Homebrew/Linuxbrew formula is finalized) but GCC proves more challenging. In order to automate it, we would need to find an alternate CI provider, that won't time out the GCC builds. We could build latest GCC HEAD via homebrew/linuxbrew, and their bottles, and upload to bintray.
Another alternative, is to investigate @jlinford's ptpkg freeze capability for packaging binaries. I have yet to try this, so I don't know if it will suit our needs.
I'd be curious to see if you could install linuxbrew on Win10/Ubuntu
ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Linuxbrew/install/master/install)"
PATH="$HOME/.linuxbrew/bin:$PATH"
echo 'export PATH="$HOME/.linuxbrew/bin:$PATH"' >>~/.bash_profile

And then try something like:
brew install --HEAD --build-bottle gcc

@zbeekman,
I thought the GCC build would be the hardest part, but the MPICH build is failing with the error message below.  I think the problem is that apt-get installs automake-1.14.1; whereas MPICH appears to want 1.15.  I tried building automake-1.15, but that build fails too.  I'm wondering if linuxbrew could build MPICH 3.1.4 for me.  I'm also considering going with an earlier version of MPICH in case it accepts automake-1.14.1. @afanfa, if you could let me know how far back I can go with MPICH versions safely, that might be helpful.
CDPATH="${ZSH_VERSION+.}:" && cd /home/rouson/Desktop-shortcut/opencoarrays/prerequisites/downloads/mpich-3.1.4 && /bin/bash /home/rouson/Desktop-shortcut/opencoarrays/prerequisites/downloads/mpich-3.1.4/confdb/missing aclocal-1.15 -I confdb
/home/rouson/Desktop-shortcut/opencoarrays/prerequisites/downloads/mpich-3.1.4/confdb/missing: line 81: aclocal-1.15: command not found
WARNING: 'aclocal-1.15' is missing on your system.
         You should only need it if you modified 'acinclude.m4' or
         'configure.ac' or m4 files included by 'configure.ac'.
         The 'aclocal' program is part of the GNU Automake package:
         <http://www.gnu.org/software/automake>
         It also requires GNU Autoconf, GNU m4 and Perl in order to run:
         <http://www.gnu.org/software/autoconf>
         <http://www.gnu.org/software/m4/>
         <http://www.perl.org/>
make: *** [/home/rouson/Desktop-shortcut/opencoarrays/prerequisites/downloads/mpich-3.1.4/aclocal.m4] Error 127

I have confirmed that the install script is able to build the default version of CMake (3.4.0) so the only remaining barrier to building OpenCoarrays on in the native Windows 10 Ubuntu Subsystem is the above problem with building MPICH.

Every MPICH 3.x should work correctly (excluding bugs).

All,
In my attempts to build OpenCoarrays on the Ubuntu bash shell provided by the Windows 10 Subsystem for Linux, the OpenCoarrays installation script correctly builds GCC 6.1.0 and CMake 3.4.0, which leaves the building of MPICH as the only known stumbling block.  I just sent the email below to the MPICH mailing list.    Any suggestions are welcome.
Damian

Begin forwarded message:
From: Damian Rouson<EMAIL_ADDRESS>Subject: [mpich-discuss] Errors building MPICH on Windows 10 Ubuntu subsystem
Date: October 10, 2016 at 9:33:51 AM PDT
To<EMAIL_ADDRESS>Reply-To<EMAIL_ADDRESS>All,
In attempting to build MPICH 3.1.4 or 3.2 in the Windows 10 Subsystem for Linux, which provides a full Ubuntu bash shell, I receive the following error message after configuring:
mpich-3.2/confdb/missing: line 81: aclocal-1.15: command not found
WARNING: 'aclocal-1.15' is missing on your system.
You should only need it if you modified 'acinclude.m4' or
'configure.ac http://configure.ac/' or m4 files included by 'configure.ac http://configure.ac/'.
The 'aclocal' program is part of the GNU Automake package:
<http://www.gnu.org/software/automake http://www.gnu.org/software/automake>
It also requires GNU Autoconf, GNU m4 and Perl in order to run:
<http://www.gnu.org/software/autoconf http://www.gnu.org/software/autoconf>
<http://www.gnu.org/software/m4/ http://www.gnu.org/software/m4/>
<http://www.perl.org/ http://www.perl.org/>
make: *** [/home/rouson/Desktop-shortcut/opencoarrays/prerequisites/downloads/mpich-3.2/aclocal.m4] Error 127
Prior to configuring, I installed aclocal installed via “sudo apt-get install automake,” which provided aclocal-1.14. I attempted unsuccessfully to build aclocal-1.15, but the discussion at the following URL suggests that the problem might relate to time stamps on file MPICH tar ball:
http://stackoverflow.com/questions/18769770/user-of-autotools-generated-tarball-gets-error-message-aclocal-1-13-command-no http://stackoverflow.com/questions/18769770/user-of-autotools-generated-tarball-gets-error-message-aclocal-1-13-command-no
I tried several of the steps suggested at the above link, but each leads to the following error message during the make step:
$ make
( cd ./src/binding/fortran/mpif_h && ./buildiface )
Replacing mpif.h.in http://mpif.h.in/
( cd ./src/binding/fortran/use_mpi && ./buildiface )
Replacing mpi_base.f90.in http://mpi_base.f90.in/
Replacing mpi_constants.f90.in http://mpi_constants.f90.in/
Replacing mpi_t1.f90
( cd . && rm -f .err unusederr.txt ; rm -rf .tmp )
( cd . &&
./maint/extracterrmsgs -careful=unusederr.txt
-skip=src/util/multichannel/mpi.c
cat maint/errmsgdirs > .tmp 2>.err )
( cd . && if test -s .err ; then rm -f .tmp ; cat .err ; exit 1 ; fi )
Warning: attempt to redefine **fail.  Duplicate ignored.
make: *** [src/mpi/errhan/defmsg.h] Error 1
Can anyone advise me about how to resolve this problem? Information on the Windows 10 Ubuntu distribution is here:
http://www.howtogeek.com/249966/how-to-install-and-use-the-linux-bash-shell-on-windows-10/ http://www.howtogeek.com/249966/how-to-install-and-use-the-linux-bash-shell-on-windows-10/
My goa is to support users of OpenCoarrays (https://github.com/sourceryinstitute/opencoarrays https://github.com/sourceryinstitute/opencoarrays), which uses MPICH to build and execute coarray Fortran codes with the GNU Fortran compiler (gfortran).
Best Regards,

Damian Rouson, Ph.D., P.E.
President, Sourcery Institute
http://www.sourceryinstitute.org http://www.sourceryinstitute.org/
+1-510-600-2992 (mobile)

discuss mailing list<EMAIL_ADDRESS>To manage subscription options or unsubscribe:
https://lists.mpich.org/mailman/listinfo/discuss

Damian Rouson, Ph.D., P.E.
President, Sourcery Institute
http://www.sourceryinstitute.org
+1-510-600-2992 (mobile)

Hi Damian,
I can't directly help myself, but did just some web searching, leading to
this:
http://stackoverflow.com/questions/33278928/how-to-overcome-aclocal-1-15-is-missing-on-your-system-warning-when-compilin
There, the last entry points to a script for installing the latest aclocal:
https://github.com/gp187/nginx-builder/blob/master/fix/aclocal.sh
Not sure if this may help.
cheers
Michael
Am Montag, 10. Oktober 2016 21:08:13 UTC+2 schrieb Damian Rouson:

All,
In my attempts to build OpenCoarrays on the Ubuntu bash shell provided by
the Windows 10 Subsystem for Linux, the OpenCoarrays installation script
correctly builds GCC 6.1.0 and CMake 3.4.0, which leaves the building of
MPICH as the only known stumbling block.  I just sent the email below to
the MPICH mailing list.    Any suggestions are welcome.
Damian
Begin forwarded message:
*From: *Damian Rouson<EMAIL_ADDRESS><javascript:>>
*Subject: *[mpich-discuss] Errors building MPICH on Windows 10 Ubuntu
subsystem
*Date: *October 10, 2016 at 9:33:51 AM PDT
*To<EMAIL_ADDRESS><javascript:>
*Reply-To<EMAIL_ADDRESS><javascript:>
All,
In attempting to build MPICH 3.1.4 or 3.2 in the Windows 10 Subsystem for
Linux, which provides a full Ubuntu bash shell, I receive the following
error message after configuring:
mpich-3.2/confdb/missing: line 81: aclocal-1.15: command not found
WARNING: 'aclocal-1.15' is missing on your system.
You should only need it if you modified 'acinclude.m4' or
'configure.ac' or m4 files included by 'configure.ac'.
The 'aclocal' program is part of the GNU Automake package:
http://www.gnu.org/software/automake
It also requires GNU Autoconf, GNU m4 and Perl in order to run:
http://www.gnu.org/software/autoconf
http://www.gnu.org/software/m4/
http://www.perl.org/
make: ***
[/home/rouson/Desktop-shortcut/opencoarrays/prerequisites/downloads/mpich-3.2/aclocal.m4]
Error 127
Prior to configuring, I installed aclocal installed via “sudo apt-get
install automake,” which provided aclocal-1.14. I attempted unsuccessfully
to build aclocal-1.15, but the discussion at the following URL suggests
that the problem might relate to time stamps on file MPICH tar ball:
http://stackoverflow.com/questions/18769770/user-of-autotools-generated-tarball-gets-error-message-aclocal-1-13-command-no
I tried several of the steps suggested at the above link, but each leads
to the following error message during the make step:
$ make
( cd ./src/binding/fortran/mpif_h && ./buildiface )
Replacing mpif.h.in
( cd ./src/binding/fortran/use_mpi && ./buildiface )
Replacing mpi_base.f90.in
Replacing mpi_constants.f90.in
Replacing mpi_t1.f90
( cd . && rm -f .err unusederr.txt ; rm -rf .tmp )
( cd . &&
./maint/extracterrmsgs -careful=unusederr.txt
-skip=src/util/multichannel/mpi.c
cat maint/errmsgdirs > .tmp 2>.err )
( cd . && if test -s .err ; then rm -f .tmp ; cat .err ; exit 1 ; fi )
Warning: attempt to redefine **fail.  Duplicate ignored.
make: *** [src/mpi/errhan/defmsg.h] Error 1
Can anyone advise me about how to resolve this problem? Information on the
Windows 10 Ubuntu distribution is here:
http://www.howtogeek.com/249966/how-to-install-and-use-the-linux-bash-shell-on-windows-10/
My goa is to support users of OpenCoarrays (
https://github.com/sourceryinstitute/opencoarrays), which uses MPICH to
build and execute coarray Fortran codes with the GNU Fortran compiler
(gfortran).
Best Regards,

Damian Rouson, Ph.D., P.E.
President, Sourcery Institute
http://www.sourceryinstitute.org
+1-510-600-2992 (mobile)

discuss mailing list<EMAIL_ADDRESS><javascript:>
To manage subscription options or unsubscribe:
https://lists.mpich.org/mailman/listinfo/discuss

Damian Rouson, Ph.D., P.E.
President, Sourcery Institute
http://www.sourceryinstitute.org
+1-510-600-2992 (mobile)

@jeffhammond out of curiosity, do you have any advice on this issue.  I'm asking because I see that you're active on the MPICH mailing list and building MPI is the one step that blocked me from building OpenCoarrays in the Windows Subsystem for (Ubuntu) Linux.  I got the same 'aclocal-15' error in building MPICH and OpenMPI.   I tried emailing the MPICH list, but didn't receive any solutions.  I haven't tried the OpenMPI list.

@rouson
Neither MPICH nor Open-MPI support Windows any more.  It's a huge effort for them and they derive zero professional benefit from it.
The MPICH response to all Windows-related queries since version 1.4 is to link to Why can't I build MPICH on Windows anymore?.
Just to give an idea of the effort level required, when MPICH supported Windows, they had a full-time developer for it and it alone.  Eventually that guy decided to move on to more interesting things.
Microsoft MPI is freely available, although I don't know how usable it is.  Intel MPI likely requires a commercial license but I expect it is usable, because I have very good experiences with it on Linux platforms.  In particular, the Intel MPI team has been very responsive to RMA-related issues that I have reported to them.
Both Microsoft MPI and Intel MPI are based on MPICH.
As you probably know, Windows 10 has something resembling Linux.  There has been a bit of interest in trying to get MPICH working there, but I have not seen anything happen yet.  I was interested in trying to get MPICH working there so that I could write MPI code on a Surface notebook but it remains below ZBB line.

As you probably know, Windows 10 has something resembling Linux.  There has been a bit of interest in trying to get MPICH working there, but I have not seen anything happen yet.  I was interested in trying to get MPICH working there so that I could write MPI code on a Surface notebook but it remains below ZBB line.

Yeah that's what we had been trying.
@rouson: I wonder if Sameer has a windows machine with MSMPI or IntelMPI we could try?
FYI I'm planning to creat a Docker image with OpenCoarrays that could be a "work around" solution FTTB

If no one objects, I propose closing this issue as a "won't fix", at least for the time being, since we can't figure out how to get an upstream dependency (i.e. MPICH) installed on windows. If no one objects, or I don't hear back by Tuesday's call, I'll close this. ⌛

I waste quite a bit of time over the past two weeks trying to get WSL working on a Windows 10 VM but could never get it to connect to the network (i.e. apt-get update -y timed out on every server).  This is likely because I'm behind a firewall, but the Ubuntu host running the VM works just fine and I can SSH from the Windows 10 VM to the host IP.  Given what I observe on https://github.com/Microsoft/BashOnWindows/issues, it's not worth my time trying to debug.  The networking stack appears to be woefully inadequate.

@jeffhammond :bow: many many thanks for your input!

In fact, I'm going to close this... if anyone objects they can reopen.

@zbeekman I'm still really curious about MPICH in WSL but sadly have no computer on which to test it and I refuse to buy something just to solve Microsoft's problems.

FYI, opencoarrays build and pass all tests on my WSL. This is true for both the master trunk (bb1b8ee) and the 1.8.2 release. I don't use install.sh, I install all dependencies via apt-get and do cmake .. && make && ctest.
I noticed Damian's issue here and a few folks on comp.lang.fortran reporting that they get gcc-4.x in WSL out of the box. I am wondering whether there are multiple branches of WSL because I had gcc-5.4.0 out of the box since I enabled WSL for the first time. It may be important that I am on their Windows Insider Preview update cycle (current build is 14986.rs_prerelease.161202-1928).
The versions of dependencies (all obtained via apt-get install) are:

automake-1.15
bison-3.0.4
flex-2.6.0
cmake-3.5.1
make-4.1
gcc-5.4.0
mpich-3.2

FYI, you only really need CMake, MPICH and GCC.... those other packages are build dependencies of one or more of CMake, GCC, MPICH.

@milancurcic , this is wonderful news!  Thank you for informing us.  You win the first-known WSL Build Prize.  🏆
If I recall correctly, I installed WSL by following these instructions.  I built GCC 6.1.0 from source, but sudo apt-get install mpich installed only limited components of MPICH, e.g.,mpiexecbut not mpif90, mpifort, or mpicc.  Did you use a command other than sudo apt-get install mpich to get the MPI compiler wrappers?  Also, are you booting into Windows or using virtualization.  For now, I have a macOS system set up for dual-boot with Windows as the other option.  In the long run, I'd like to have Windows in a virtual machine, but @jeffhammond reported difficulties with that approach and I can't invest too much time in it.  I'd just like to get something working.
I've never heard of Windows Insider Preview. I'll research how to install it.  If you have any links or other advice, please let us know.

@milancurcic I think I just signed up for Windows Insider Preview, but it's not clear how to update my Windows installation nor how to tell my current installation revision number.  sudo apt-get install still returns only gcc/g++/gfortran version 4.8.4.

@rouson:

I am using a regular (non-VM) install of Windows 10 Home edition 64-bit. I have no experience with Windows in VMs.

At the time when I was working to enable bash on Windows (October 2016), installation guides (I don't recall which one I used) said I needed at least build 14316. The only way I could bump my version above that number was that I enroll in Windows Insider Program. I remember there was a choice of few update tiers, I chose the fastest one. Currently, my system settings say I'm on the Insider Program level "Fast". Practically this means my Windows updates (whole OS) every few weeks ago. It is slower than before, and one of the updates broke my screen dimming function keys.

I think that once you enroll in the program, your OS needs to update to see any changes. After an update, your destop screen will have full version number in the bottom right corner. Unfortunately I am not very proficient with Windows and I wish I knew more but this is it.

I installed MPICH via sudo apt-get install mpich and I got working wrappers (mpicc, mpif90, mpiexec) available out of the box.

Check your Ubuntu version (lsb_release -a). This package/gcc availability may have to do something with the OS version. I remember that Win10 originally enabled Ubuntu 14.x, but after some update a developer build included Ubuntu 16.x. My lsb_release -a says 16.04.1.

@milancurcic This is very helpful.

I am using a regular (non-VM) install of Windows 10 Home edition 64-bit.

Same for me.

At the time when I was working to enable bash on Windows (October 2016), installation guides (I don't > recall which one I used) said I needed at least build 14316.

My OS Build is 14393.693

The only way I could bump my version above that number was that I enroll in Windows Insider Program. I remember there was a choice of few update tiers, I chose the fastest one. Currently, my system settings say I'm on the Insider Program level "Fast".

Apparently I was set on "Release Preview."  I just switched to "Fast."

I think that once you enroll in the program, your OS needs to update to see any changes. After an
update, your destop screen will have full version number in the bottom right corner.

Ok. I'll leave the machine powered on and connected to the Internet in hopes it will update itself.  I haven't discovered a way to update proactively.

Unfortunately I am not very proficient with Windows and I wish I knew more but this is it.

Ditto

Check your Ubuntu version (lsb_release -a). This package/gcc availability may have to do with the OS
version. I remember that Win10 originally enabled Ubuntu 14.x, but after some update a developer build > included Ubuntu 16.x. My lsb_release -a says 16.04.1.

This was extremely helpful information.  The above command shows that I'm on Ubuntu 14, which  I'm reasonably confident explains why you apt-get installs GCC 5.4.0 for you whereas it installs GCC 4.8.3 for me.  If I can figure out how to update to Ubuntu 16 and if sudo apt-get install mpich gives me the compiler wrappers, then it's highly likely I'll be able to build OpenCoarrays too.  At that point, I'll add a windows-install.sh script to OpenCoarrays.  Thanks tons!!
@zbeekman I'll reopen this in anticipation that this issue will be resolved reasonably soon via the steps above.

My OS Build is 14393.693

I think this may be the key. The article I linked above says Ubuntu 16.04 was introduced in build 14936.

Ok. I'll leave the machine powered on and connected to the Internet in hopes it will update itself. I haven't discovered a way to update proactively.

Try rebooting. It may trigger an update.

Control panel has a section with Updates in the title.  I trigger updates manually this way.

Control panel has a section with Updates in the title. I trigger updates manually this way.

Very interesting, my control panel does not have any section related to Updates, only Windows Settings does (called Update & Security). I see no options to manually update.

@rouson, OK, so you'll close this, once you confirm it's working? I'm not sure I see the utility of reopening, because this seems like the only thing for us to do is confirm that it works or it doesn't: the rest is out of our hands.

@milancurcic @zbeekman @jeffhammond @afanfa
I have now added a draft windows-install.sh script on a new windows-build branch.  The script completes the OpenCoarrays build process without error, but 33 of 34 tests fail.  The only test that passes is the one containing unit tests for our installation scripts, which is the only test that does not require MPI.
Even the simple MPI program below produces the same error as our CAF tests:
program main
  use mpi
  implicit none
  integer error
  call MPI_Init(ierror)
  call MPI_Finalize(ierror)
end program

The error message is the same as what is listed here except for the machine name.  Any advice?

All,
Never mind. The advice at this page worked, except that I had to substitute my host name where the page shows macbook-pro.local.  I'll edit the script to suggest this to the user and then submit a pull request that finally closes this issue.   Hooray!!!   Thanks, everyone, for all the input and especially thank you, @milancurcic, for providing the critical advice that got us over the hump.  Sometime soon, I'll update our documentation and update  OpenCoarrays

testing on Travis-CI will require a docker image most likely; Travis-CI provides 12.04 and 14.04 out of the box so we'll have to roll our own docker image to emulate WSL 16.04


ERROR : git clone git@github.com:uber/ludwig.git

Cloning into 'ludwig'...
The authenticity of host 'github.com (<IP_ADDRESS>)' can't be established.
RSA key fingerprint is SHA256:nThbg6kXUpJWGl7E1IGOCspRomTxdCARLviKw6E5SY8.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added 'github.com,<IP_ADDRESS>' (RSA) to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

You can clone using the url if you are having problems like this git clone https://github.com/uber/ludwig

@Denniso2 thanks a lot !!

I updated the README with the https url, which should solve the issue for people in the future. Closing this, thank you both!

aaa just spent 3 hours on this. why they dont just do that then


Documentation for how to use Zoon

Hello guys :smile:
Some of us needs to rely heavily on documentation while coding in order to understand how to use the good stuff. And it seems we need some of it now in order to make the project more accessible to new Rust/web Programmers.
For now, I do not understand how MZ works in the inside but I follow MZ since it's very first day when there was actually no code yet. So I would like to contribute to make it more accessible and maybe challenge all of us to enlighten its mechanics and user flow to anybody interested by the project.
Our lovely Rust language has incredible :grinning:  good documentation with many examples that helps us everyday to grow on it. I think it is a good example to start with.
The main objective of this PR is to document most commons Elements , including styling, methods and patterns a programmer needs to make a web page when using Zoon. I keep the internal documentation for later.
Here is the Todo list for documentation :

[ ] ~Elements such as Button, Raw and others.~
[x]  ~Css properties~ Styling such as Background and others.
[x] Use of Signal and local state-management.
[x] Have the examples compiling correctly.
[x] Have the phrasing checked together.

Please read the content and give me feedback on the phrasing and wording. My English for sure needs to be improved :wink: and maybe the explanations need to be worked out.
Even if the framework might change later, it seems to me to be healthy to start now with the documentation.
MZ has an insane potential which only need to be revealed :stuck_out_tongue:
Thank you :wink:
UPDATE: The PR is massive enough, documentation on Elements will be done in a later PR.

Hello, I will continue to read your answers and work out the PR, I just need some time. Thank you a lot for your answers :wink:

I will continue the PR. Maybe a good idea to do doc on styling before continuing on other elements.

Well, I would like to document Align . It seems Zoon is using some class there

    pub fn center_x(mut self) -> Self {
        self.static_css_classes.insert("center_x".into());
        self.static_css_classes.remove("align_left".into());
        self.static_css_classes.remove("align_right".into());
        self
    }

    ```
    I do not find the class definition itself, could you point me out where is it ?

I do not find the class definition itself, could you point me out where is it ?

In basic.css - see the css folder.
Note: You can disable basic styles (modern-normalize.min.css and basic.css) with Frontend::default_styles.

I am not sure how to document the implementation . Maybe there is no need.

I agree, there is no need. The implementation isn't too complicated but bit ugly/verbose/maybe confusing a bit, I would like to try to refactor it in the future.

Is it for allowing Align to be used inside the s() method ?

Exactly.

Hello :smile: .
Today I am documenting Borders and maybe I see something we could check.
In the code we have :

    pub fn x(self, border: impl Borrow<Border>) -> Self {
        let border = border.borrow();
        self.left(border).right(border)
    }

The method is called x , so I would expect to set borders horizontally, which are top and bottom borders but it does actually set left and right borders which are vertical borders.
Is it my point of view which is strange or does the code actually needs correction ?

Hello :smile:
I am working out an example for LayerIndex, is there a reason why we have u32 ? I have a hard time to position an element behind without using a negative value..
When I set i32 instead, then it become very easy to have a good example explaining the LayerIndex.
      Column::new()
        .s(Background::new().color(GREEN_7))
        .item(
            Paragraph::new()
                .s(LayerIndex::new(-1))
                .content("Behind and trying to hide"),
        )
        .item(Paragraph::new().content("Front and you can see me"))

With only positive value, like setting 0 for the element hiding and 1 for the others, I cannot get the same rendering.
What do you think ?

Today I started to document transitions. I did not find any example but i made one this way.
    Button::new()
        .update_raw_el(|raw_el| {
            raw_el.style_group(StyleGroup::new(":hover").style("margin-right", "40%"))
        })
        .s(Transitions::new([
            Transition::property("margin-right").duration(2000)
        ]))
        .label("Hover me")

It seems I had to use raw css to make it work.
Do you guys think there is an other way to make it ?

I get two weird behaviors with this :
    let (is_hovered, hover_signal) = Mutable::new_and_signal(false);
    Button::new()
        .s(Width::with_signal(hover_signal.map_bool(|| 50, || 100)))
        .on_hovered_change(move |hover| is_hovered.set(hover))
        .on_press(|| log("click"))
        .label("hover me")

On Firefox : when I hover the left part of the button the cursor pointer does not update so it looks like clicking will have no effect but on_press is firing  actually, but the hover signal is not triggered unless I move more to the left it seems.
On Brave ( chromium ) : the hover works fine, but after the button get smaller, the area where the button is no more, is still clickable and the on_press fires. If I have the debugger opens, the hover event re-fires on the area where the button is not supposed to be any more.
Can somebody try the code and checkout the same glitch ?
https://user-images.githubusercontent.com/30114057/159246116-0e503bb1-88ac-4cae-b4f7-c3b650c8eda5.mp4

I will re-read the examples and doc this afternoon and check the styling milestone when everything is consistent and corrected :smile:

I get two weird behaviors with this :

It looks like an infinite loop so I think the behavior depends on the browser implementation and luck a bit. What's the expected behavior from your point of view?

Thank you for your message :grin:
Well for Firefox I would expect :

The hovering should work correctly with then mouse cursor  that shows the button is clickable  with the hand icon.
For Brave - Chromium :
Once the UI is updated and the button is smaller because of the first signal, the mouse cursor should go  back to default and  the hover event not be triggered again.

For Brave, yes it seems the hover event is triggered multiple times. It looks like something is not fully updated in both cases somehow.
I test my code within the counter example where I replace the original code with this one. Would be nice if somebody else try.

For documenting styling. I think I am done for this first version.
Should I document what we have inside style.rs or is it supposed to be mainly used internally in the future or specific cases ?
We already have 126 unit test as documentation code :open_mouth:
test result: ok. 126 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 2.05s

Thank you so much for your time :heart: . I will look at it and apply the commits for most of them I think.  Damn it, I really have spacing issues sometimes :laughing:

I will check the issue with container vs parent_element , but I think we got most of the stuff done.
The page keeps hiding many of the reviews when the page is reloading, it was hard to keep track of everything. Next time. I will do things with smaller steps.

Let me know when I can rebase :wink:

For the space around # Example, I am not planning to do it now because it does not make visible changes on the generated documentation or in the IDE I am using. I guess somebody can do it later.
Thank you for your message, I'll the rebase tomorrow :)

The description of the PR has been updated  to inform that documentation on Elements will be done later.
Thank you for your help @MartinKavik , this PR is good to be merged :wink:


CI job update 2018-03-30 05:30

Update from CI

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Azure/azure-docs-sdk-dotnet, working directory: W:\34jp-s, target branch: master, target branch commit: 1fd906d7e0000a2d63e388a2cc68079cc7dfe767, source branch: 887dbcc04c6ceff76a50289d8f6f708d/master, source branch commit: .

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Unable to download webpage: HTTP Error 503: Service Unavailable

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.08.02
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', 'best[format_id^=hls]', 'https://www.dailymotion.com/video/x4uoq9y', '-o', 'download.mp4', '--hls-prefer-native']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.08.02
[debug] Python version 3.6.8 (CPython) - Linux-4.14.79+-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6
[debug] Proxy map: {}
[dailymotion] x4uoq9y: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 503: Service Unavailable (caused by <HTTPError 503: 'Service Unavailable'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/lib/python3.6/dist-packages/youtube_dl/extractor/common.py", line 627, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/lib/python3.6/dist-packages/youtube_dl/YoutubeDL.py", line 2227, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.6/urllib/request.py", line 532, in open
    response = meth(req, response)
  File "/usr/lib/python3.6/urllib/request.py", line 642, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python3.6/urllib/request.py", line 570, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.6/urllib/request.py", line 650, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)

Description

I've run this command since last week and it worked well. I download video from dailymotion daily.
But today I have got this error.
Help me solve this problem.
Thanks!

> py -2.7 .\youtube_dl\__main__.py https://www.dailymotion.com/video/x4uoq9y
[dailymotion] x4uoq9y: Downloading webpage
[dailymotion] x4uoq9y: Downloading metadata JSON
[dailymotion] x4uoq9y: Downloading m3u8 information
[dailymotion] x4uoq9y: Downloading m3u8 information
[dailymotion] x4uoq9y: Downloading m3u8 information
[dailymotion] x4uoq9y: Downloading m3u8 information
[dailymotion] x4uoq9y: Downloading m3u8 information
[download] Destination: Wizards of Waverly Place - S 1 E 3 - I Almost Drowned In a Chocolate Fountain!-x4uoq9y.mp4
[download]   0.0% of 130.12MiB at 55.90KiB/s ETA 39:42
ERROR: Interrupted by user


Add Bootstrap navigation

Step 1/3 for creating static pages

The pieces are there... just need some cleanup.
Can save that task for later


Register/unregister panel should use id for comparison

Replay of bug: https://app.replay.io/recording/should-display-toggleable-stack-for-traces--a7935dc5-c1f8-4a90-ba21-1cfa39d34e26

I don't remember why I opened this and the Replay has been deleted :(


Use Bundle.module.resourceURL to point to Hylo standard library

This change removes a hard-coded path to the standard library, which pointed to the pre-build source directory; i.e. directly to //repo-root/Library/. Instead, we use Bundle.module.resourceURL to point to the post-build location of the standard library resource files.

I don't know why I approved this.  It totally broke something I had already fixed.


Cannot train DIET classifier with Bert '

Rasa version: 1.9.1
Rasa SDK version  not used
Rasa X version not used
Python version: 3.6.9
Operating system ubuntu 18.04
Issue: I followed this instruction (link: https://rasa.com/docs/rasa/master/nlu/components/#hftransformersnlp) to fed rasa's DIET classifier with BERT features and i got  the following error.
Error (including full traceback):
Traceback (most recent call last):                                                                                File "/home/optio/rasaprojects/rasaenv/bin/rasa", line 8, in <module>                                                 sys.exit(main())                                                                                File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/__main__.py", line 91, in main                cmdline_arguments.func(cmdline_arguments)                                                                         File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/cli/train.py", line 76, in train              additional_arguments=extract_additional_arguments(args),                                                          File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/train.py", line 50, in train                  additional_arguments=additional_arguments,                                                                        File "uvloop/loop.pyx", line 1456, in uvloop.loop.Loop.run_until_complete                                           File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/train.py", line 101, in train_async           additional_arguments,                                                                                File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/train.py", line 188, in _train_async_internal                                                                                additional_arguments=additional_arguments,                                                                        File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/train.py", line 245, in _do_training          persist_nlu_training_data=persist_nlu_training_data,                                                              File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/train.py", line 474, in _train_nlu_with_validated_data                                                                                persist_nlu_training_data=persist_nlu_training_data,                                                              File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/train.py", line 86, in train              interpreter = trainer.train(training_data, **kwargs)                                                              File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/model.py", line 191, in train             updates = component.train(working_data, self.config, **context)                                                   File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/utils/hugging_face/hf_transformers.py", line 472, in train                                                                                batch_docs = self._get_docs_for_batch(batch_messages, attribute)                                                  File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/utils/hugging_face/hf_transformers.py", line 417, in _get_docs_for_batch                                                                                batch_examples, attribute                                                                                File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/utils/hugging_face/hf_transformers.py", line 249, in _get_token_ids_for_batch                                                                                example, attribute                                                                                File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/utils/hugging_face/hf_transformers.py", line 219, in _tokenize_example                                                                                split_token_ids, split_token_strings File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/utils/hugging_face/hf_transformers.py", line 154, in _lm_specific_token_cleanup                                                                               return model_tokens_cleaners[self.model_name](split_token_ids, token_strings)                                     File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/utils/hugging_face/transformers_pre_post_processors.py", line 234, in bert_tokens_cleaner                                                                     return cleanup_tokens(list(zip(token_ids, token_strings)), "##")                                                  File "/home/optio/rasaprojects/rasaenv/lib/python3.6/site-packages/rasa/nlu/utils/hugging_face/transformers_pre_post_processors.py", line 26, in cleanup_tokens  token_ids, token_strings = zip(*token_ids_string)                                                               ValueError: not enough values to unpack (expected 2, got 0)

Command or request that led to error:
rasa train

Content of configuration file (config.yml) (if relevant):
language: en
pipeline:
- name: HFTransformersNLP
  model_weights: "bert-base-uncased"
  model_name: "bert"
- name: LanguageModelTokenizer
- name: LanguageModelFeaturizer
- name: DIETClassifier
epochs: 50

policies:
  - name: MemoizationPolicy
  - name: TEDPolicy
    max_history: 5
    epochs: 100
  - name: MappingPolicy

Thanks for the issue, @tmbo will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

Any update on this? I'm also interested


Rename "bootstrap" tag to a more cifmw-centric value

As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

Useless change - bootstrap tasks are still running during "packages" stage (even the ones not related to packages).


Rename isInProjectView into showsProjects

To follow the convention of other methods like showsFullClassHierarchy and showsFlatClasses

rename is done in #304 as part of other browser API changes


Switch from "/dev/std*" to "/proc/$$/fd/*" (as is used later) and allow it to fail (esp. for SELinux)

Fixes #149

I'm still having this issue with Fedora 25
db_1   | chown: changing ownership of '/proc/1/fd/1': Permission denied
db_1   | chown: changing ownership of '/proc/1/fd/2': Permission denied


Capability List: Seperate List per Capability Type

Following #55 and #44, procedure#clist is a heterogeneous list of capability types, and requires parsing in order to access a capability by index. This makes syscalls more expensive and also doesn't take advantage of a clist that could contain multiple capabilities of the same type and size.
In order for a procedure to contain multiple capabilities of the same type, we need to seperate the procedure#clist into a list of separate lists of homogenous capability types.
While this would make indexing faster, it brings additional complexity which needs to be addressed. This should be done after #44 in a separate PR.

Closing on #118


[Task] Handle failed updates

[x] I'm sure this issue is not a duplicate.

It so happens that in some cases when the extension gets updated and the user does not restart (not reload) the editor, loading the extension again results to misbehavior.
We should add a handler for such cases that will clean up the mess and if needed prompt the user to restart the editor.

@vscode-icons/development I was thinking of creating an integrity check of the icons folder content using Nodejs crypto. Any other suggestions?

Here is an example how vscode is using the same idea.

Hopefully we will not use md5. :/

Why not? We don't need any special cryptography. I believe using sha is overkill.

I'm afraid of hash collisions :/

@robertohuertasm Are you OK with this concept?

Yes, no problem.

Good, cause I have done half the coding.

This is going to be covered by nsri in v9. The default hash algorithm is sha-1 for files and sha-512 for directories.


Improve get PRs details performance

Azure DevOps Extensions
Generate Release Notes (Node Cross Platform)
Is your feature request related to a problem? Please describe.
Generating release notes is really slow in our case because "Getting associated PRs" will fetch about 100 000 PRs.
Describe the solution you'd like
According to this comment from MSFT, the pull request list API will sort completed PRs by completion date.
With that in mind, in getPullRequest we could stop the PR iteration when we reached the oldest build/release date that we are comparing against.
Describe alternatives you've considered
No response
Additional context
No response

Interesting idea, note sure when I might have time to experiment with this technique, but it is on the backlog

The fix #1588 has just been released, this provides some new option that could help PR discovery performance


JAVA : Java Stdin and Stdout I

Write your solution to this JAVA problem of hackerrank:-
Problem link : Java Stdin and Stdout I

@Riddhi9570 Please assign this issue to me for Hacktoberfest 2022

I am willing to contribute to this solution. As a part of hacktober-fest.


Forgets keybinding

Ubuntu 16.04
1.6.0-beta7
Keybinding forgotten at startup (shift-left click)
visiting edit->preferences->packages
causes it to remember and the key binding kicks in and sublime-syle-column-selection then works.
till the next startup

I noticed this really strange behavior too.
Sublime-style-column-selection 1.5.1
Atom 1.7.3
windows 8.1
I'm using the default keybinding (alt + shift-left click). The keybinding is not recognized until I open the settings page.

Yes, I've seen this too
Sublime-style-column-selection 1.5.1
Atom 1.7.4
Mac OS X 10.11.5
Using the default keybinding; not working until I visit the Settings page.


Full page reload on search params change

Which project does this relate to?
Start
Describe the bug
Hey, I am unsure if this is a bug or if I am doing something wrong. Either way, it's not clear to me how to solve this.
I have a page with a loader that takes quite a lot of time to fetch. So I am prefetching the data using queryClient.prefetchQuery and useSuspense query with Suspense.
On the page, I have a dynamic layout that I want to save in the URL. I set the validateSearch with zodSearchValidator. I read the search params via Route.useSearch() and wrote the change on click of a Link in the suspended component. (setting from and search function).
If I navigate to that page fresh, with no cache, everything works as expected. (I set shouldReload: false for the page to not trigger the loader on the search change). However, if I refresh the page, the page data is loaded from the cache. When I quickly switch the search param via the Link it does a full page reload.
In the video you can see the full page reload indicated by the dark theme flicker.
(The example code shown below, is not accurate as it is not using Start, just tan stack router, but the implementation is something like that but with Start)
https://github.com/user-attachments/assets/8f420427-494a-4e97-a7ab-3cccfe1f8813
Your Example Website or App
https://stackblitz.com/edit/github-sjfxnb?file=src%2Froutes%2Findex.tsx
Steps to Reproduce the Bug or Issue

Go to home
Reload
Switch column or grid links

Expected behavior
I would expect the page to not reload. The way it works when I wait for what I assume is hydration.
Screenshots or Videos
No response
Platform

Linux
Firefox
Version: Latest

Additional context
As an extra note, the css file that I am attaching via links: () => [{ rel: 'stylesheet', href: appCss, type: 'text/css' }] in __root.tsx seems to be loaded after the rendered page. Resulting in the flickering. I have a simple tailwind setup with CSS variables. I am not using a dark theme switcher here. Just loading CSS variables with the tailwind. The CSS in general seems to load a bit later.

I cannot reproduce this using your example:
https://github.com/user-attachments/assets/d51859fa-582e-4822-be93-886b6d7fd4f8

I cannot reproduce this using your example:
repro.mov

Hi, I did not manage to reproduce it in Stack Blitz. There were some strange issues. I have created a repo with a sample that is as close as possible to what I have but with real data and real requests. But I haven't managed to reproduce the same behavior. In this project. the page reloads every time I change the search parameters...giving me cached results, but It does a page reload. In my real project, this happens only before hydration finishes, after that it changes only the client-side URL.
Here is the repo: https://github.com/allan-from-meiro/tsr_url_reload


What is the contrastive loss?

Hi,
What is the contrastive loss defined here. Is it used in decoding?

This loss is not used in training. They can be deleted.


Label for HTML block

If the label for an HTML block in Gravity Forms is set it will display on the frontend. The default label is "HTML Block" which is displayed above the content. If I remove this label from the Gravity Form the label is still displayed but rendered as null on the frontend.
As I understand based on the field label help text  "Enter the label for this HTML block. It will help you identify your HTML blocks in the form editor, but it will not be displayed on the form."  I don't believe displaying this text is the intended behavior.
I don't have a patch to submit but removing labelFor={labelFor} from the InputWrapper in /src/components/Html/index.js will resolve this.

Good Afternoon, thanks for reporting. It should be resolved now can you check?


Check TF 2.0 support status on Travis.

Refs #93.

Allow failures.

Codecov Report

Merging #97 into master will decrease coverage by 0.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
- Coverage   91.88%   91.56%   -0.32%
==========================================
  Files          30       30
  Lines        2574     2574
  Branches      437      437
==========================================
- Hits         2365     2357       -8
- Misses        142      148       +6
- Partials       67       69       +2

Impacted Files
Coverage Δ

adanet/core/report_accessor.py
91.3% <0%> (-4.35%)
:arrow_down:

adanet/core/estimator_distributed_test_runner.py
85.85% <0%> (-3.04%)
:arrow_down:

adanet/core/estimator.py
91.85% <0%> (-0.19%)
:arrow_down:

Continue to review full report at Codecov.

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


GSK-278/task Disparate impact test

The signature of the function is not correct to me. To me, the function should have 2 slices as argument and a dataset because:

As a user, the way we think about this test is comparing two slices inside the same dataset. There is no sense in giving the possibility to provide 2 different datasets as argument. It can even be misleading in the way we understand disparate impact
In AI Test v2, users will tend to create test suite so that slices_men + slice_women are "fixed" inputs that are shared between different fairness tests for gender and the dataset as "existing" input of the test suite. This logic of opening should be prepared from now. See image below.

In the way, we create AI Test v2, it's important that the test function has the same argument as the argument in the UI that enables users to instantiate the test function. In the disparate impact case, the inputs that people will have in mind are the 2 population (ex: men and women) and the dataset.
In terms of product strategy, It's really important to nudge users to create slices because our main product differentiator is creating business test. This is why it's crucial to have it as a test argument.

Thanks @jmsquare for these details. Ok I understand the logic and the motivation.
When we talk about passing slices as arguments, does that mean that instead of something like this:
    def test_disparate_impact(self,
                              gsk_dataset: GiskardDataset,
                              protected_slice: Callable[[pd.DataFrame], pd.DataFrame],
                              unprotected_slice: Callable[[pd.DataFrame], pd.DataFrame],
                              model: GiskardModel,
                              positive_outcome,
                              threshold=0.8,
                              classification_label=None,
                              ) -> SingleTestResult:

where we can have for instance:
protected_slice=lambda df: df[df.sex == "male"]
unprotected_slice=lambda df: df[df.sex == "female"]

Yes exactly. And dataset should also be an argument

related to https://github.com/Giskard-AI/giskard/pull/338


actual use

I am trying to use this package for my react app. And I am confused as how to actually build and use this package. for the constructor do I need to have a callback?? I just want to use this to monitor the inactivity and then logout if a certain amount of time of inactivity has gone by.
const timeout = idleTimeout(
() => {
return;
},
{
element: document,
timeout: 1000 * 60 * 5,
loop: false,
},
);
that is my constructor but when I try to call the internal functions it gives me errors. Do i need to declare the functions i am using in the callback section?

@ppolicherla Yeah, the first argument need to be the callback where you can add you logging logic.
Make sure to set loop: true in the options when you want to get a callback every time the idle timeout is reached.


random_quantum_circuit_generation potential speed improvement

As someone who's in the habit of creating a bunch of random quantum circuits, I noticed that it's pretty slow. I ran snakeviz on the creation of a lot of random circuits. The total cumulative time is 7.9s and 3.28s was spent in line 658. With some digging: it looks like the while loop to exclude consecutive single-qubit gates is big slow. In particular, it spends a decent amount of time canonicalizing PhasedXZ gates to make sure the current gate isn't the same as the previous one.
You can very easily imagine switching to sampling gates not by value equality, but by index into the set of possible single qubit gates. You could probably do a more sophisticated way of choosing "random number between 0 and n except for x" than repeatedly sampling and rejecting.

I don't think this is skill-level/advanced. only normal cs skills required

Line 658 of what?

Yeah sorry. The file/module name is in the issue title. I could have been much more clear!


Support LIKE expression

TPCH has some LIKE expressions and we don't support it

FSST couldn't push this down but we could implement for Dict, VarBin and VarBinView.
This could make for an interesting compression strategy that factors in the kinds of compute functions you'll want to run on the encoded data.

Fixed in #1525


call get ice servers when offer is received to avoid stale ice server…

… credential
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This fixes the issue of stale ice server credentials but RTCPeerConnection.setConfiguration() is not supported in Firefox.

So a tradeoff with this approach is that the startup time might be increased due to having to fetch a new ICEServerConfig before the browser can start gathering ICE candidates on the master side.

Am I perhaps missing something on the potential latency impact?
Are there any alternative ways to accomplish the same thing (having non-stale TURN credentials)? Perhaps the master could periodically refresh the TURN credentials instead? Or even keep a pool of non-stale credentials?


Wrong IP address reported in OpenVZ containers

I have never been able to get the correct IP address reported on OpenVZ containers.  I instead always get <IP_ADDRESS> as the IP.
Here's a screenshot from the management console:
http://take.ms/8q2bA # note that the last one is VMWare, not OpenVZ.
The partial ifconfig output from a container is below.
venet0    Link encap:UNSPEC  HWaddr 00-00-00-00-00-00-00-00-00-00-00-00-00-00-00-00
          inet addr:<IP_ADDRESS>  P-t-P:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
          UP BROADCAST POINTOPOINT RUNNING NOARP  MTU:1500  Metric:1
          RX packets:3282819 errors:0 dropped:0 overruns:0 frame:0
          TX packets:3515429 errors:0 dropped:0 overruns:0 carrier:0
          collisions:0 txqueuelen:0
          RX bytes:4235147025 (4.2 GB)  TX bytes:4100779262 (4.1 GB)

venet0:0  Link encap:UNSPEC  HWaddr 00-00-00-00-00-00-00-00-00-00-00-00-00-00-00-00
          inet addr:x.x.x.184  P-t-P:x.x.x.184  Bcast:x.x.x.184  Mask:<IP_ADDRESS>
          UP BROADCAST POINTOPOINT RUNNING NOARP  MTU:1500  Metric:1

I suggest that if multiple IPs are returned, Ohai/Chef keep the first one that doesn't start in "127".  It may be worth noting that hostname -i returns the correct IP.

This most likely needs to be refiled as an ohai bug in http://github.com/chef/ohai/issues/new


jsdoc has a hard dep on github

jsdoc has Github dependency on taffydb.. sometimes Github goes down (as it did last week) making jsdoc (and downstream tools like jsdoc2md) uninstallable, until Github sort out their problem.
Can the jsdoc project reference the taffydb npm module? It seems the fork it currently depends on is 4 years stale and only adds a readme and package.json (which are both present in taffy these days).
One other reason to change is performance, using github deps slows down the install process (as the code needs to be cloned/downloaded each time, whereas npm caches modules in ~/.npm)

This is an annoyance for our team too. Best to stick with NPM dependencies only for release builds.

Duplicate of #961.

👍


chore: update minimum Node version in package.json

This PR updates the minimum Node version for this package
from v10 to v12. Node.js 10 is no longer supported (see here).
This change also means the minimum TLS version has been set to TLS v1.2 by default.
tls.DEFAULT_MIN_VERSION - Default: 'TLSv1.2' (Added in: v11.4.0) Docs
Checklist

[] npm test passes (tip: npm run lint-fix can correct most style issues)
[ ] tests are included
[ ] documentation is changed or added

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

@ricellis Sorry, wrong PR :)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Contact state type not reporting ACW

https://github.com/amazon-connect/amazon-connect-streams/blob/932ad6a50f107b57b9e66a8b58c79dcd5eb0c22e/src/index.d.ts#L644C1-L645C1 -> I can never get my contact to return this type when we are in fact in ACW.
streams version: 2.5.0
flow: incoming call -> accept call -> end call (here we don't get ACW state type)
code repro:
// place call
    // ... ring ring

//answer call
contact.accept();

// end call
const agentConnection = contact.getAgentConnection();
connection.destroy();

// print type
const contactStateType = contact.getState().type;
console.log(contactStateType); // <- repeatedly returns "ended" but never changes to "acw".

There is no direct contact state 'ACW', I'll update documentation to avoid confusion.
A better way to detect ACW state would be subscribing to contact.onACW event
https://github.com/amazon-connect/amazon-connect-streams/blob/932ad6a50f107b57b9e66a8b58c79dcd5eb0c22e/Documentation.md#contactonacw

There is no direct contact state 'ACW', I'll update documentation to avoid confusion.
A better way to detect ACW state would be subscribing to contact.onACW event https://github.com/amazon-connect/amazon-connect-streams/blob/932ad6a50f107b57b9e66a8b58c79dcd5eb0c22e/Documentation.md#contactonacw

yes I am familiar with that event. I'm simply stating your model has a property that never seems to be used: https://github.com/amazon-connect/amazon-connect-streams/blob/master/src/index.d.ts#L644
so it's more than updating docs right?

Yes, we will update the model as well.


Move start and goal out of planner

start and goal states should not be part of a planner struct / trait

See https://samtal.atlassian.net/browse/MPF-27

Won't Do.


image upload remove os type

这个 PR 实现什么功能/修复什么问题:

镜像上传取消os type参数

是否需要 backport 到之前的 release 分支:

release/2.10.0

Codecov Report

Merging #1080 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #1080      +/-   ##
=========================================
- Coverage     7.9%   7.87%   -0.03%
=========================================
  Files         356     356
  Lines       56477   56477
=========================================
- Hits         4464    4448      -16
- Misses      51618   51631      +13
- Partials      395     398       +3

Flag
Coverage Δ

#aFlag
7.87% <ø> (-0.03%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/rbacutils/rbac.go
71.56% <0%> (-4.48%)
:arrow_down:

pkg/appsrv/response.go
66.15% <0%> (-3.08%)
:arrow_down:

Continue to review full report at Codecov.

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

/lgtm
/approve


FEATURE REQUEST: Allow re-authentication with face ID or fingerprint

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app
Login with device supporting Face ID or fingerprint
Select to enable Face ID or fingerprint login
Log out
Try to re-login with Face ID or fingerprint

Expected Result:
User should be able to login with Face ID or fingerprint
Actual Result:
This option is not available now
Workaround:
unknown
Platform:

Where is this issue occurring?

iOS
Android

Version Number: 1.1.26-0
Reproducible in staging?: n/a
Reproducible in production?: n/a
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @thienlnam
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1641513675063100
View all open jobs on GitHub

Triggered auto assignment to @sakluger (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

isn't it usual to clear face id when you logout? how else would you log in with another account or remove your login?
face id is used to re-auth if your session has expired or if you set an app to lock when it loses focus for x amount of time.

Yeah, I was thinking we could either

Tie faceID to an account (Not sure how this works) but then if they try to log in with the email again, we use faceID
Clear faceID when the user logs out and only use faceID / touchID to re-auth if the authToken expires (like you mentioned)

Triggered auto assignment to @Christinadobrzyn (Internal) because issue was reported by a contributor who needs to be compensated if this issue is fixed

This will likely require some internal changes first - I picked this up and will investigate what it will take to implement this

Agreed, I think you'd pass a public key to the back end when you setup biometrics and then sign a message using biometrics to do Authy things.
On logout delete from the keystore.
When authToken and refreshToken are expired, use biometrics to get a new authToken.
After first fail of biometrics let user re-try or logout or use username and password.

I think this needs to go to Engineering after the triage

Current assignee @thienlnam is eligible for the Engineering assigner, not assigning anyone new.

Hey @thienlnam if this looks good, can you add the External label so we can get this in Upwork as a job?

Sorry I think I'm confused, I posted my questions here. I didn't think we created a GH/Upwork job for Feature Requests - so just checking with the team on what to do next.

Looks like Alex answered your questions but yeah this will be a CME issue and will not need an upwork issue

Discussing authentication related aspects in an internal thread -> https://github.com/Expensify/Expensify/issues/195606

@sakluger, @thienlnam Huh... This is 4 days overdue. Who can take care of this?

@sakluger, @thienlnam Eep! 4 days overdue now. Issues have feelings too...

@sakluger, @thienlnam Eep! 4 days overdue now. Issues have feelings too...

@sakluger, @thienlnam Eep! 4 days overdue now. Issues have feelings too...

@sakluger, @thienlnam Huh... This is 4 days overdue. Who can take care of this?

Wow melvinbot spam, putting this on a weekly while we hash out the authentication details internally

Same as above, need to resolve internal issue first

From this internal thread, we're going to put on the side for now as it's more of a polish item and not directly newsletter related expensify.slack.com/archives/C03U7DCU4/p1650578288196109


Fix missing gradle metadata preview

Having his unfortunately doesn't fail the build, but generates a broken publication for
projects using experimental Kotlin multiplatform.

Whoops, wrong project. The right one doesn't have the problem too. Closing.


Enabling userroles for tracking causes Fatal error

When enabling Userroles under "Userroles tracking", it causes fatal error:
Fatal error: Uncaught TypeError: in_array(): Argument #2 ($haystack) must be of type array, string given in /var/www/wimh/wp-content/plugins/plausible-analytics-2.0.5B1/src/Includes/Actions.php:60 Stack trace: #0 /var/www/wimh/wp-content/plugins/plausible-analytics-2.0.5B1/src/Includes/Actions.php(60): in_array() #1 /var/www/wimh/wp-includes/class-wp-hook.php(324): Plausible\Analytics\WP\Includes\Actions->maybe_register_assets() #2 /var/www/wimh/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters() #3 /var/www/wimh/wp-includes/plugin.php(517): WP_Hook->do_action() #4 /var/www/wimh/wp-includes/script-loader.php(2262): do_action() #5 /var/www/wimh/wp-includes/class-wp-hook.php(324): wp_enqueue_scripts() #6 /var/www/wimh/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters() #7 /var/www/wimh/wp-includes/plugin.php(517): WP_Hook->do_action() #8 /var/www/wimh/wp-includes/general-template.php(3052): do_action() #9 /var/www/wimh/wp-content/themes/wpzoom-derive/header.php(106): wp_head() #10 /var/www/wimh/wp-includes/template.php(790): require_once('...') #11 /var/www/wimh/wp-includes/template.php(725): load_template() #12 /var/www/wimh/wp-includes/general-template.php(48): locate_template() #13 /var/www/wimh/wp-content/themes/wpzoom-derive/page-templates/homepage-builder-bb.php(6): get_header() #14 /var/www/wimh/wp-includes/template-loader.php(106): include('...') #15 /var/www/wimh/wp-blog-header.php(19): require_once('...') #16 /var/www/wimh/index.php(17): require('...') #17 {main} thrown in /var/www/wimh/wp-content/plugins/plausible-analytics-2.0.5B1/src/Includes/Actions.php on line 60

Version of plugin is 2.0.5 Beta 1

Upgraded to version 2.0.5 Beta 2 and no errors anymore on frontpage.

Great! Still curious why you were getting those errors, even though wp_parse_args() should assure that the enhanced measurements option was an array, but regardless, adding extra checks doesn't hurt anyone.


Load balancing

If we have multiple popular experiments running, will servers still handle traffic?

I do think that Django is the bottleneck here - most of the heavy duty is done in the backend.


why all  the next_sentence_labels equal 1 with run create_pretraining_data.py?

i understander wrong?

"1" actually means "is random" because that's the convention I used originally. The problem is that there are blank lines between your input sentences so the system thinks that each its its own document, and next sentence can't cross documents. So just remove the empty line between segments in the same document.

thanks a lot


Patches #124 Problems with slashes in autocomplete options

Fixes autocomplete issue when using slashes.
When a partial match was made after the user entered a command with a slash, the autocomplete function
would ignore the matched prefix.
When a autocomplete was used with a slash the autcomplete would fail when post prefix was an empty string
Tests added

Any chance of this getting merged? @dthree

LGTM, thanks! :+1: