Very slow parsing

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I'm using eslint-plugin-svelte.
(*.svelte file linting does not work with the parser alone. You should also use eslint-plugin-svelte with it.)

[X] I'm sure the problem is a parser problem. (If you are not sure, search for the issue in eslint-plugin-svelte repo and open the issue in eslint-plugin-svelte repo if there is no solution.

[X] I have tried restarting my IDE and the issue persists.

[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
8.19.0
What version of eslint-plugin-svelte and  svelte-eslint-parser are you using?
latest
What did you do?
I'm using a very minimal configuration, only extending plugin:svelte/base which only contains 2 rules: svelte/comment-directive and svelte/system
I've run a benchmark on a substantially large project with hundreds of svelte files.
% time TIMING=1 eslint --ext .svelte,.js,.ts,.cjs 'src/**/*.svelte'
Rule                     | Time (ms) | Relative
:------------------------|----------:|--------:
svelte/comment-directive |     4.085 |    66.7%
svelte/system            |     2.041 |    33.3%

TIMING=1 eslint --ext .svelte,.js,.ts,.cjs 'src/**/*.svelte'  141.41s user 1.09s system 104% cpu 2:16.41 total

As you can see, those rules only added 6ms, but it took over 140 seconds to parse these files.
What did you expect to happen?
eslint should have finished within 2 seconds
What actually happened?
eslint finished over 140 seconds
Link to Minimal Reproducible Example
Because the project is proprietary, I cannot share it in a reproduction repo. But I'm confident it can be replicable on any large project.
Additional comments
No response

Actually, I've narrowed it down to this specific filesize, here is a sample svelte file that will take over 111 seconds!! Yes, for a single file: https://paste.ofcode.org/CZPqC28RdBCrkiipQwBuiy
% time TIMING=1 eslint --ext .svelte,.js,.ts,.cjs 'src/ssr/Head.svelte'
Rule                     | Time (ms) | Relative
:------------------------|----------:|--------:
svelte/system            |     0.177 |    52.1%
svelte/comment-directive |     0.163 |    47.9%
TIMING=1 eslint --ext .svelte,.js,.ts,.cjs 'src/ssr/Head.svelte'  111.59s user 0.46s system 100% cpu 1:51.91 total

Thank you for this issue.
I have debugged it, but if it replace the style content with whitespace , 'svelte/compiler' will be very slow.
import fs from "fs";
import { parse } from "svelte/compiler";

const fixture = fs
  .readFileSync("src/ssr/Head.svelte", "utf8")
  .replace(/(<style.*?>)([\s\S]*?)(<\/style>)/u, (_, p, c, s) => {
    return p + c.replace(/\S/gu, " ") + s;
  });

parse(fixture);

This parser causes this problem because it parses CSS content by replacing it with whitespace.
The reason for doing it is that svelte itself cannot parse SCSS etc.
I will consider a workaround when I have time, but it could be a problem with svelte itself.

I found a problematic regex in svelte. This regex consumes most of the time it takes.
https://github.com/sveltejs/svelte/blob/ccbf10c16326cffb1b38e721e2f74aad4b85dbf2/src/compiler/parse/index.ts#L37

I opened a issue in the Svelte repository.
https://github.com/sveltejs/svelte/issues/7675

I found a problematic regex in svelte. This regex consumes most of the time it takes.

Nice one! 👍
However, one thing I don't understand is that if I run this code:
import fs from 'fs';
import {parse} from 'svelte/compiler';

parse(fs.readFileSync('src/ssr/Head.svelte', 'utf8'));

directly using svelte parse(), it doesn't have any slowness?

import fs from "fs";
import { parse } from "svelte/compiler";

const fixture = fs
  .readFileSync("src/ssr/Head.svelte", "utf8")
  .replace(/(<style.*?>)([\s\S]*?)(<\/style>)/u, (_, p, c, s) => {
    return p + c.replace(/\S/gu, " ") + s;
  });

parse(fixture);

Is regex part of svelte-eslint-parser? if so can we do something like this? Removing the contents, rather than replacing it with whitespace?
import fs from "fs";
import { parse } from "svelte/compiler";

const fixture = fs
  .readFileSync("src/ssr/Head.svelte", "utf8")
  .replace(/(<style.*?>)([\s\S]*?)(<\/style>)/u, '$1$3');

parse(fixture);

When we remove the content, we need to recalculate the correct location of the node and token. If we use whitespace, we don't need it.

@ota-meshi Okay I see, fully understand the problem now.

Seems that we need an efficient way to disable styles, svelte3 plugin seems to have a similar feature https://github.com/sveltejs/eslint-plugin-svelte3/blob/00e24a4474dae29dd9d06c467cbe5f2c61c60b82/src/preprocess.js#L39 looks like it is more selective on the attributes. I guess if we replace it with any characters other than whitespace it might work? For example, underscores?
The root of the issue is from https://github.com/sveltejs/svelte/blob/ccbf10c16326cffb1b38e721e2f74aad4b85dbf2/src/compiler/parse/index.ts#L37 as you've mentioned, and according to the git history it's been there for half a decade, it might be unlikely they are not going to fix it. If it comes down to it, I might have to use pnpm/yarn patching features https://pnpm.io/next/package_json#pnpmpatcheddependencies

I opened PR in the svelte repository.
https://github.com/sveltejs/svelte/pull/7706

@ota-meshi it seems won't be merged any time soon, can I suggest we replace the style content with null (\0) instead of space? I think that will resolve the issue, what do you think?

No workaround needed once the PR is merged. I will not spend more time on this issue.

No workaround needed once the PR is merged. I will not spend more time on this issue.

Or maybe @shirotech can just raise a PR to fix that issue instead on our side?

Now that the latest version of Svelte has been released, So I close this issue.


Update Readme.md and Changelog to reflect code changes

See https://github.com/tighten/solana-php-sdk/issues/11 for details

Let me know if anything needs to be updated or if I missed something.


4.0 version plan

[x] Update Dart SDK to 3.0.0
[ ] #211
[x] #209
[ ] #212
[ ] #166
[ ] #154
[ ] #68

[ ] #223


update gitalk

update gitalk

This PR fixs the following problem.
Hello there!
On February 7th, 2020 at 10:03 (UTC) your application (gitalk) used its client_id and client_secret (with the User-Agent Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.106 Safari/537.36) as part of a set of query parameters to access an endpoint through the GitHub API:
https://api.github.com/repositories/198127550/issues
Please use Basic Authentication instead as using OAuth credentials in query parameters has been deprecated.
Depending on your API usage, we'll be sending you this email reminder at most once every 3 days.
Just one URL that was accessed with a User-Agent combination will be listed in the email reminder, not all.
Visit https://developer.github.com/changes/2019-11-05-deprecated-passwords-and-authorizations-api/#authenticating-using-query-parameters for more information.
Thanks,
The GitHub Team


Foreign key relationship

Hi guys,
Really like this pod so far, nice work! I was wondering if foreign key relationship is supported. Say you have this json:
{ "project":
    {   "id": 12345,
        "name": "My Project",
        "category": 1,
        "teamMemberIDs": [1, 2, 3, 4]
    }
 }

where the category would be a relation to a category and the teamMemberIDs a relation to multiple users.

Hi @risp!
Yes we have support for that but your JSON would have to look a bit different.
{
  "id":12345,
  "name":"My Project",
  "category":{
    "id":231,
    "title":"Category name"
  },
  "members":[
    {
      "id":1,
      "name":"Member 1"
    },
    {
      "id":2,
      "name":"Member 2"
    }
  ]
}

Then in your Core Data Model you'll have
Project ----- (to one)----> Category
Project ----- (to many)----> Member

Is that what you meant?
Take a look at the Tests project, it has various Core Data entities supporting all sorts of combinations.
We also have all sorts of JSON files that could help you as examples
https://github.com/hyperoslo/Sync/tree/master/Tests/Tests

Thanks, I found the Tests project already and it helped a lot. But in this case we designed our API to use as less bandwidth as possible so therefore we are only sending the id instead of the object. Of course in this example the difference in bandwidth is small but when there are a lot of objects in the response it increases fast.  It would be really nice if this could be supported as well because there are also external API's that use this kind of foreign keys in their JSON.

Oh, I totally understand why this is build.
@risp I see, what do you use the ids for? You still have to ask for their data right?
I agree, this is a common use case. Lets see if we can find a way to support it in Sync :)

Let me know how we can help :)

@risp I see, what do you use the ids for? You still have to ask for their data right?

Yes, that's right. In my project, objects have a category, project, paymentMethod. When the user logs in I preload these all so that from then on IDs can be used instead of the object.

I think you can still use Sync and in your main JSON just skip this IDs, and then when you get to the object detail you'll retrieve the JSON with the full details.
I might be wrong but let me know if we can improve Sync in a way the is more helpful for you. :)

I've been working on this in #140
Should be ready at the end of next week.


Add payload size limit

Description

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Code quality improvements

How to test

Checklist

[ ] I have tested this code
[ ] I have updated the documentation

Corresponding BE PR: https://github.com/source-academy/backend/pull/1150

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

1 of 7 (14.29%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 31.727%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/pages/sicp/subcomponents/chatbot/ChatBox.tsx
1
7
14.29%

Files with Coverage Reduction
New Missed Lines
%

src/pages/sicp/subcomponents/chatbot/ChatBox.tsx
2
7.89%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.009%

Covered Lines:
4813

Relevant Lines:
14342

💛  - Coveralls


RFC: Provide a middleware factory method for event_handler components

Is this related to an existing feature request or issue?
https://github.com/awslabs/aws-lambda-powertools-python/issues/1547
Which AWS Lambda Powertools utility does this relate to?
Event Handler - REST API
Summary
AWS Lambda Powertools provides the Middleware factory which is great for creating middleware for Lambda Event handlers which has been the focus of middleware since inception of the python powertools library.  With the API style Event Handlers based from BaseRouter we have provided a way for customers to create larger, more complex monolithic style APIs.  This can  cause code to become more complex and harder to maintain due to cross-cutting concerns that have an effect on multiple routes.
Use case
Enable middleware to run before/after a route to enable custom data to be injected to the route in a named parameter.  being wrapped up as middleware will enable re-use simply and easily for other routes.
Today this is not possible "cleanly" due to the Lambda Event and COntext being locked away within the Router class and not exposed to the route functions at all (which makes complete sense).
To add-in Middleware would require something like the following today to enable passing of custom "user_data"  into the route function handler.   In the below code the middleware cannot be self contained in nature and would need to access the global "app" instance to access the original Lambda Event or Context which is not ideal for middleware implementations
from aws_lambda_powertools.event_handler import APIGatewayRestResolver
from aws_lambda_powertools import Logger
import functools

app = APIGatewayRestResolver()
logger = Logger()

def route_decorator(route_func):
    """Define route decorator middleware function to add my user data"""
    @functools.wraps(route_func)
    def wrapper(*args, **kwargs):
        kwargs["user_data"] = {"param1":"value1"}
        logger.info("calling route_decorator")
        return route_func(*args, **kwargs)

    return wrapper

@app.get("/hello")
@route_decorator
def hello_world(user_data=None):
    return {"message": "Hello World", "user_data": user_data}

@logger.inject_lambda_context(log_event=True)
def lambda_handler(event, context):
    return app.resolve(event, context)

Proposal
Provide an event handler middleware factory implementation to simplify setup of route middleware and provide access to key data within the middleware parameters: event, context, router to reduce boilerplate and provide a simpler mechanism for code re-use for cross-cutting concerns.
from aws_lambda_powertools.event_handler import APIGatewayRestResolver
from aws_lambda_powertools.middleware_factory import router_lambda_decorator
from aws_lambda_powertools import Logger

app = APIGatewayRestResolver()
logger = Logger()

@router_lambda_decorator
def route_decorator(handler, router):
    """Define route decorator middleware function to add my user data"""
    user_data = call_some_func(router.current_event)
    response = handler(user_data)
    return call_some_response_wrapper(response)

@app.get("/hello")
@route_decorator
def hello_world(user_data=None):
    return {"message": "Hello World", "user_data": user_data}

@app.get("/foo")
@route_decorator
def get_foo(user_data=None):
    return {"message": "More Foo, Less Bar", "user_data": user_data}

@logger.inject_lambda_context(log_event=True)
def lambda_handler(event, context):
    return app.resolve(event, context)

Out of scope
...
Potential challenges
The main challenge with Middleware here is the fact that the routing middleware is interacting with the actual Resolver class and adding data to it for routing purposes which is a little different than the lambda_handler_decorator factory method so care is required in putting this solution together so that the call chaining makes sense and middleware function itself can be constructed in the same way all other python middleware functions are built.
Now that I've written this I feel it is less of a concern given the "app.resolve()" simply ends up calling the function and likely that is where the change for this feature actually is.
Dependencies and Integrations
No response
Alternative solutions
All other alternative solutions end up being a little "hacky" in nature.  I thought the same effect could be achieved using lambda_handler_decorator instead but the trouble is that fact the "app.resolve()" router function does not enable passing of data into it causing any central lambda handler mechanism requiring storing of globals or manipulating the actual event/context objects themselves which is less than ideal.
Middleware with correctly injected data dependencies is always nicer allowing self contained processing for the route handlers so this feature will no longer require global variables to be setup to pollute the lambda execution environment and potentially cause side effects between function invocations which is possible.
Acknowledgment

[X] This feature request meets Lambda Powertools Tenets
[ ] Should this be considered in other Lambda Powertools languages? i.e. Java, TypeScript

Oh - can probably close this in favour of #953 😁, did not realise it was already in dev.


NdrBaseArray.FormatType should print out array element count, not siz

Currently the string format for array types return the array size
between brackets. It should return the array element count instead.
Before :
HRESULT Proc0(
    /* Stack Offset: 0 */ handle_t p0,
    /* Stack Offset: 8 */ [In, Out] wchar_t[520] p1
);

After :
HRESULT Proc0(
    /* Stack Offset: 0 */ handle_t p0,
    /* Stack Offset: 8 */ [In, Out] wchar_t[260] p1
);

I've done it. I don't know enough about NDR format, so I don't know if NdrBogusArrayTypeReference.GetArraySize() still returns a correct value for constant conformant arrays.


Difference between quoted and unquoted mpd link

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.10.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[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. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
I have a link to the mpd manifest. Link contains no characters to be escaped but it contains second FQDN as its part.
When I pass it quoted onto brackets everything works fine, but when I pass it unquoted it gets a 404.
Investigated into this with fiddler proxy: the only difference is that quoted call makes yt-dlp use http user-agent with chrome version 91.0.4472.114, and unquoted call makes it to use user-agent with chrome 96.0.4664.93.
Why does webserver give 404 with one but not with another - dunno.
See link below at the debug output section.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
C:\Users\username>"C:\Programs\yt-dlp.exe" "https://htv-mag2-moscow2.mts.ru/htv-rrs.mts.ru/88888888/16/20210812/268628773/268628773.mpd" -vU
[debug] Command-line config: ['https://htv-mag2-moscow2.mts.ru/htv-rrs.mts.ru/88888888/16/20210812/268628773/268628773.mpd', '-vU']
[debug] Encodings: locale cp1251, fs utf-8, pref cp1251, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.10.04 [4e0511f] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 2020-09-30-git-9d8f9b2e40-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1690 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.10.04, Current version: 2022.10.04
yt-dlp is up to date (2022.10.04)
[debug] [generic] Extracting URL: https://htv-mag2-moscow2.mts.ru/htv-rrs.mts.ru/88888888/16/20210812/268628773/268628773.mpd
[generic] 268628773: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] 268628773: Extracting information
[debug] Identified a DASH manifest
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
ERROR: [generic] 268628773: This video is DRM protected
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1477, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1574, in __extract_info
  File "yt_dlp\YoutubeDL.py", line 1632, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 2569, in process_video_result
  File "yt_dlp\YoutubeDL.py", line 1027, in raise_no_formats
yt_dlp.utils.ExtractorError: [generic] 268628773: This video is DRM protected

C:\Users\username>"C:\Programs\yt-dlp.exe" https://htv-mag2-moscow2.mts.ru/htv-rrs.mts.ru/88888888/16/20210812/268628773/268628773.mpd -vU
[debug] Command-line config: ['https://htv-mag2-moscow2.mts.ru/htv-rrs.mts.ru/88888888/16/20210812/268628773/268628773.mpd', '-vU']
[debug] Encodings: locale cp1251, fs utf-8, pref cp1251, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.10.04 [4e0511f] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19044-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 2020-09-30-git-9d8f9b2e40-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1690 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.10.04, Current version: 2022.10.04
yt-dlp is up to date (2022.10.04)
[debug] [generic] Extracting URL: https://htv-mag2-moscow2.mts.ru/htv-rrs.mts.ru/88888888/16/20210812/268628773/268628773.mpd
[generic] 268628773: Downloading webpage
ERROR: [generic] Unable to download webpage: HTTP Error 404: Not Found (caused by <HTTPError 404: 'Not Found'>); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp\extractor\common.py", line 672, in extract
  File "yt_dlp\extractor\generic.py", line 2614, in _real_extract
  File "yt_dlp\extractor\common.py", line 823, in _request_webpage

  File "yt_dlp\extractor\common.py", line 805, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 3682, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default
urllib.error.HTTPError: HTTP Error 404: Not Found

C:\Users\username>

yt-dlp randomly sets a user-agent for each call. If that UA is rejected by server for generic links, there is nothing we can do about it. You can force a specific UA using the --add-headers option

Just noticed example is DRM!

DRM is next step, so pls dont bother about it. I`ll force --allow-unplayable to download it.

yt-dlp randomly sets a user-agent for each call. If that UA is rejected by server for generic links, there is nothing we can do about it. You can force a specific UA using the --add-headers option

Okay, thak you. Could you please say variance limits of this randomness, and close the issue.

It may change from version to version. Currently it's https://github.com/yt-dlp/yt-dlp/blob/98d4ec1ef287cc5655ce6afd7b17755c57a245cb/yt_dlp/utils.py#L76-L118


Wallet

Hello,
This is our first concept for the wallet module. We are building the TransctionManager too, but we are still discussing some main concepts.

Did this get migrated into the branch for working?  Please close this pull request if you are working in the branch...

@lock9 could you rebase and do a new clean PR ?


npm run pack error for ubuntu 16.04

➜  upterm git:(master) ✗ npm install

><EMAIL_ADDRESS>preinstall /home/jim/文档/upterm
> npm prune

npm WARN invalid config loglevel="notice"

><EMAIL_ADDRESS>postinstall /home/jim/文档/upterm
> electron-rebuild

✔ Rebuild Complete
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 0.14.x || ^15.0.0-0 || 15.x but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

updated 17 packages in 207.223s
➜  upterm git:(master) ✗ npm run pack

><EMAIL_ADDRESS>pack /home/jim/文档/upterm
> npm run compile && build

npm WARN invalid config loglevel="notice"

><EMAIL_ADDRESS>compile /home/jim/文档/upterm
> npm run cleanup && npm run tsc && npm run copy-html

><EMAIL_ADDRESS>cleanup /home/jim/文档/upterm
> rimraf compiled/src

><EMAIL_ADDRESS>tsc /home/jim/文档/upterm
> tsc

src/Output.ts(5,20): error TS2307: Cannot find module 'immutable'.
src/Output.ts(708,33): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/Output.ts(710,32): error TS7006: Parameter 'char' implicitly has an 'any' type.
src/Output.ts(815,17): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/PTY.ts(4,22): error TS2307: Cannot find module 'node-pty'.
src/references.d.ts(9,1): error TS6053: File '/home/jim/文档/upterm/node_modules/immutable/dist/immutable.d.ts' not found.
src/views/OutputComponent.tsx(6,20): error TS2307: Cannot find module 'immutable'.
src/views/OutputComponent.tsx(103,38): error TS7006: Parameter 'possiblyEmptyRow' implicitly has an 'any' type.

npm ERR! Linux 4.4.0-83-generic
npm ERR! argv "/home/jim/.nvm/versions/node/v8.1.2/bin/node" "/home/jim/文档/upterm/node_modules/.bin/npm" "run" "tsc"
npm ERR! node v8.1.2
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>tsc: `tsc`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>tsc script 'tsc'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the upterm package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     tsc
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs upterm
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls upterm
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/jim/文档/upterm/npm-debug.log

npm ERR! Linux 4.4.0-83-generic
npm ERR! argv "/home/jim/.nvm/versions/node/v8.1.2/bin/node" "/home/jim/文档/upterm/node_modules/.bin/npm" "run" "compile"
npm ERR! node v8.1.2
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>compile: `npm run cleanup && npm run tsc && npm run copy-html`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>compile script 'npm run cleanup && npm run tsc && npm run copy-html'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the upterm package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run cleanup && npm run tsc && npm run copy-html
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs upterm
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls upterm
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/jim/文档/upterm/npm-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>pack: `npm run compile && build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>pack script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/jim/.npm/_logs/2017-07-09T11_41_20_091Z-debug.log
➜  upterm git:(master) ✗

Also Ubuntu 17.10

As the log says, "This is probably not a problem with npm". Maybe the repository was unavailable or something. I can confirm that dependencies installation works now. Also, please make sure you use the latest node.

Similar problem:
node v8.1.4
npm  v3.10.10
~/upterm on  master ⌚ 12:25:40
$ npm install

><EMAIL_ADDRESS>preinstall /home/timofey/upterm
> npm prune

npm notice created a lockfile as package-lock.json. You should commit this file.
up to date in 0.095s
npm WARN deprecated<EMAIL_ADDRESS>Use mz or fs-extra^3.0 with Promise Support
npm WARN deprecated<EMAIL_ADDRESS>Use uuid module instead
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g

><EMAIL_ADDRESS>install /home/timofey/upterm/node_modules/electron-chromedriver
> node ./download-chromedriver.js

><EMAIL_ADDRESS>install /home/timofey/upterm/node_modules/fsevents
> node install

><EMAIL_ADDRESS>install /home/timofey/upterm/node_modules/fcopy-pre-bundled
> echo ''

><EMAIL_ADDRESS>postinstall /home/timofey/upterm/node_modules/electron
> node install.js

><EMAIL_ADDRESS>postinstall /home/timofey/upterm
> electron-rebuild

✔ Rebuild Complete
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 0.14.x || ^15.0.0-0 || 15.x but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

added 1219 packages in 127.038s

~/upterm on  master! ⌚ 12:28:00
$ npm run pack

><EMAIL_ADDRESS>pack /home/timofey/upterm
> npm run compile && build

npm WARN invalid config loglevel="notice"

><EMAIL_ADDRESS>compile /home/timofey/upterm
> npm run cleanup && npm run tsc && npm run copy-html

><EMAIL_ADDRESS>cleanup /home/timofey/upterm
> rimraf compiled/src

><EMAIL_ADDRESS>tsc /home/timofey/upterm
> tsc

src/Output.ts(5,20): error TS2307: Cannot find module 'immutable'.
src/Output.ts(736,33): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/Output.ts(740,27): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/Output.ts(829,40): error TS7006: Parameter 'char' implicitly has an 'any' type.
src/Output.ts(872,13): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/Output.ts(879,13): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/Output.ts(886,13): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/Output.ts(909,13): error TS7006: Parameter 'row' implicitly has an 'any' type.
src/PTY.ts(4,22): error TS2307: Cannot find module 'node-pty'.
src/references.d.ts(9,22): error TS6053: File '/home/timofey/upterm/node_modules/immutable/dist/immutable.d.ts' not found.
src/views/OutputComponent.tsx(4,20): error TS2307: Cannot find module 'immutable'.
src/views/OutputComponent.tsx(40,50): error TS2339: Property 'attributes' does not exist on type '{}'.
src/views/OutputComponent.tsx(40,67): error TS2339: Property 'attributes' does not exist on type '{}'.

npm ERR! Linux 4.10.0-28-generic
npm ERR! argv "/home/timofey/.nvm/versions/node/v8.1.4/bin/node" "/home/timofey/upterm/node_modules/.bin/npm" "run" "tsc"
npm ERR! node v8.1.4
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>tsc: `tsc`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>tsc script 'tsc'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the upterm package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     tsc
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs upterm
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls upterm
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/timofey/upterm/npm-debug.log

npm ERR! Linux 4.10.0-28-generic
npm ERR! argv "/home/timofey/.nvm/versions/node/v8.1.4/bin/node" "/home/timofey/upterm/node_modules/.bin/npm" "run" "compile"
npm ERR! node v8.1.4
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>compile: `npm run cleanup && npm run tsc && npm run copy-html`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>compile script 'npm run cleanup && npm run tsc && npm run copy-html'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the upterm package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run cleanup && npm run tsc && npm run copy-html
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs upterm
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls upterm
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/timofey/upterm/npm-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>pack: `npm run compile && build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>pack script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/timofey/.npm/_logs/2017-07-18T09_32_56_655Z-debug.log

Now we have official linux images, so you don't need to build it yourself anymore. https://github.com/railsware/upterm/releases

nice 👍


How to Change Default Connection Factory

When I create a console application and connect it to a database that was created in SQL Server Management Studio with Entity Framework 6.1.3, then the following code was created in the app.config. Which assume there is no explicit connection string, so it uses LocalDb instance for a context instead.
  <entityFramework>
    <defaultConnectionFactory type="System.Data.Entity.Infrastructure.LocalDbConnectionFactory, EntityFramework">
      <parameters>
        <parameter value="mssqllocaldb" />
      </parameters>
    </defaultConnectionFactory>
    <providers>
      <provider invariantName="System.Data.SqlClient" type="System.Data.Entity.SqlServer.SqlProviderServices, EntityFramework.SqlServer" />
    </providers>
  </entityFramework>
</configuration>

My question is what do I need to do or what do I need to set up either on SQL Server Management Studio or Visual Studio in order to have app.config to have something like following in the first place?
<entityFramework>
    <defaultConnectionFactory type="System.Data.Entity.Infrastructure.SqlConnectionFactory, EntityFramework">
      <parameters>
        <parameter value="Data Source=localhost; Integrated Security=True; MultipleActiveResultSets=True" />
      </parameters>
    </defaultConnectionFactory>
    <providers>
      <provider invariantName="System.Data.SqlClient" type="System.Data.Entity.SqlServer.SqlProviderServices, EntityFramework.SqlServer" />
    </providers>
  </entityFramework>
</configuration>
``

@BIngZL1983 This is the github repo for EF Core. EF6 can be found at https://github.com/aspnet/EntityFramework6
That being said, EF will create a default connection factory to localdb if it is installed, or to SQL Express if localdb is not found, but never to localhost. The connection factory is only used if your application does not specify a connection string. Docs can be found here: https://msdn.microsoft.com/en-us/library/jj592674(v=vs.113).aspx


Make logging optional and replaceable

Provide a default logger but make it possible to be replaced.
logger interface
#initialize context, options = {}
#level
#level=
#visible? level
#log level, *args
log levels:
error - 50
warn - 40
info - 30
debug - 20
trace - 10

can't we use a std ruby logger and its std interface? it is common to use this approach with other loggers so aspector would integrate automatically with any logger there is (like rails logger ,etc)

I've not looked at the standard logger before implementing this feature. What I would like to do is to be able to change log level externally. If the standard logger supports that, we can switch to use it.
I'll close this as it's done. Feel free to open a new issue to switch logger implementation


vm extension list on vm with no extensions throws error

expected: empty return object
actual:
'NoneType' object is not iterable
Traceback (most recent call last):
File "C:\azure-cli\src\azure\cli\main.py", line 55, in main
cmd_result = APPLICATION.execute(args)
File "C:\azure-cli\src\azure\cli\application.py", line 117, in execute
result = expanded_arg.func(params)
File "C:\azure-cli\src\azure\cli\commands__init__.py", line 212, in _execute_command
result = operation(client, **kwargs) if client else operation(**kwargs)
File "c:\azure-cli\src\command_modules\azure-cli-vm\azure\cli\command_modules\vm\custom.py", line 491, in list_extensions
result = [r for r in vm.resources if r.type == extension_type]
TypeError: 'NoneType' object is not iterable

I have verified it does not return an error now. here is an output
(env) Mayuris-MacBook-Pro:azure-cli mayuridiwan$ az vm extension list -g  --vm-name
[]


Files are not valid deposit resources

Instead they are deposited via a POST to /v1/file
Per discussion after standup 4/6/18

Let’s be extra clear here given recent confusion and a misleading title:
Files should not be deposited via a POST to the ‘/resource’ route. They should be deposited via a POST to ‘/file’ route. This is from sprint 0 discussions.
Updates to files (currently metadata only) are like any resource a PATCH to ‘/resource/{Id}’ route. We started work on update file (binary) but that was held back given team misunderstanding of current work.
On the ‘/file’ POST, a File is a valid deposit resource. But it builds and validated File metadata differently. Currently that build is invalid in master (it does not follow the MAP spec).
This is a change for the ‘/resource’ POST route and how it handles validations, which doesn’t include Files.

@cmh2166 yes, that is correct.

	Cool. Wasn’t asking, but clarifying for you and others - again, given confusion and unclear PR title.

	Get Outlook for iOS

On Mon, Apr 9, 2018 at 7:47 AM -0700, "Justin Coyne"<EMAIL_ADDRESS>wrote:
@cmh2166 yes, that is correct.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Use BigQuery for User Feedback analytics

Instead of fetching data from UA we now use the BigQuery API which
allows us to query the table for the specific metrics we need. It
already returns the information we need in the correct format and
reduces the amount of work required post query.
https://trello.com/c/6OkkMnm0/3411-5-migrate-ua-metrics-for-user-feedback-to-bigquery-in-content-data-api

Merging as @MuriloDalRi is on leave for 2 weeks.


npm run e2e fails in fresh install (Windows 7)

I'm running Windows 7.
I updated vue-cli to the latest version then did:

vue init webpack e2e
npm install
npm run e2e

And I got this error:
16:53:19.874 INFO - Launching a standalone Selenium Server
Setting system property webdriver.chrome.driver to E:\alex\e2e\node_modules\chro                              medriver\lib\chromedriver\chromedriver.exe
16:53:19.921 INFO - Java: Oracle Corporation 25.66-b18
16:53:19.921 INFO - OS: Windows 7 6.1 x86
16:53:19.921 INFO - v2.53.0, with Core v2.53.0. Built from revision 35ae25b
16:53:19.952 INFO - Driver class not found: com.opera.core.systems.OperaDriver
16:53:19.952 INFO - Driver provider com.opera.core.systems.OperaDriver is not re                              gistered
16:53:19.952 INFO - Driver provider org.openqa.selenium.safari.SafariDriver regi                              stration is skipped:
registration capabilities Capabilities [{browserName=safari, version=, platform=                              MAC}] does not match the current platform VISTA
16:53:19.952 INFO - Driver class not found: org.openqa.selenium.htmlunit.HtmlUni                              tDriver
16:53:19.952 INFO - Driver provider org.openqa.selenium.htmlunit.HtmlUnitDriver                               is not registered
16:53:20.014 WARN - Failed to start: SocketListener0@<IP_ADDRESS>:4444
java.net.BindException: Selenium is already running on port 4444. Or some other                               service is.
        at org.openqa.selenium.server.SeleniumServer.start(SeleniumServer.java:5                              09)
        at org.openqa.selenium.server.SeleniumServer.boot(SeleniumServer.java:32                              1)
        at org.openqa.selenium.server.SeleniumServer.main(SeleniumServer.java:24                              0)
        at org.openqa.grid.selenium.GridLauncher$1.launch(GridLauncher.java:58)
        at org.openqa.grid.selenium.GridLauncher.main(GridLauncher.java:127)

What could be the reason?

It looks like something is already running on port 4444. You'll have to end whatever that process is first.


Cassandra schema updates?

Hi. We just updated to 1.2 of this project... which was super easy btw :D  As we updated things we also updated the job to the 1.2 version.  The job ran again and got this output:
Warning: Cannot create directory at `/.cassandra`. Command history will not be saved.
system_schema  system              system_traces
system_auth    system_distributed  jaeger_v1_dc1
Generating the schema for the keyspace jaeger_v1_dc1 and datacenter dc1
Using template file /cassandra-schema/v001.cql.tmpl with parameters:
    mode = prod
    datacenter = dc1
    keyspace = jaeger_v1_dc1
    replication = {'class': 'NetworkTopologyStrategy', 'dc1': '2' }
    trace_ttl = 172800
    dependencies_ttl = 0
Warning: Cannot create directory at `/.cassandra`. Command history will not be saved.
<stdin>:179:InvalidRequest: Error from server: code=2200 [Invalid query] message="Index dependencies_ts_index_idx_1 is a duplicate of existing index dependencies_ts_index_idx"

Generally things seem to be fine so I dont see this as currently beign a problem. But we figured we'd bring it up since schema changing might happen in the future.
Steps we followed to update:
oc project jaeger-test
oc patch deploy jaeger-query -p '{"spec":{"template":{"spec":{"containers":[{"name": "jaeger-query", "image":"jaegertracing/jaeger-query:1.2"}]}}}}'
oc patch deploy jaeger-collector -p '{"spec":{"template":{"spec":{"containers":[{"name": "jaeger-collector", "image":"jaegertracing/jaeger-collector:1.2"}]}}}}'
oc delete jobs jaeger-cassandra-schema-job
oc create -f https://raw.githubusercontent.com/jaegertracing/jaeger-openshift/48478dc068ce18f252404bc62e17b3c53d695786/production/cassandra.yml

Looks like it's only a matter of changing the last CREATE CUSTOM INDEX to CREATE CUSTOM INDEX IF NOT EXISTS
Schema updates should be supported, it just so happens that we had none yet, so, there's only the v001.sql file.

By the way, would you be willing to contribute with a PR?

There ya go


[INTEL MKL] Fix to non-native gcc build.

This PR fixes following build error when building with non-native gcc version.
bazel-out/host/bin/external/swig/swig: /lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by bazel-out/host/bin/external/swig/swig)
Target //tensorflow/tools/pip_package:build_pip_package failed to build```

This PR fixes following build error when building with non-native gcc version.
bazel-out/host/bin/external/swig/swig: /lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by bazel-out/host/bin/external/swig/swig)
Target //tensorflow/tools/pip_package:build_pip_package failed to build```

What part of ENV is wrong without your change? Usually the right way to fix env problems is by adding a --action_env FOO=BAR to the bazel command line.

@mdfaijul Could you please check reviewer comments and keep us posted. Thanks!

@r4nt @gbaned I am using CentOS Linux release 7.5.1804, and building with GCC 6.3.
I tried with --action_env  to the bazel build command, but it did not work for me. Seeing same error:
ERROR: tensorflow/lite/python/optimize/BUILD:28:1: SWIGing tensorflow/lite/python/optimize/calibration_wrapper.i failed (Exit 1)
bazel-out/host/bin/external/swig/swig: /lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by bazel-out/host/bin/external/swig/swig)
Here is the build command I used:
bazel --output_base=<some_dir> --action_env LD_LIBRARY_PATH=<gcc_6.3_dir>/lib64 --config=mkl --copt="-mfma" --copt="-mavx2" -c opt //tensorflow/tools/pip_package:build_pip_package

Can you show the output of
$ ldd bazel-out/host/bin/external/swig/swig

@r4nt Here is the output of $ ldd bazel-out/host/bin/external/swig
linux-vdso.so.1 =>  (0x00007ffed8b62000)
	libstdc++.so.6 => /opt/tensorflow/gcc6.3/lib64/libstdc++.so.6 (0x00007fa63c357000)
	libm.so.6 => /lib64/libm.so.6 (0x00007fa63c055000)
	libgcc_s.so.1 => /opt/tensorflow/gcc6.3/lib64/libgcc_s.so.1 (0x00007fa63be3e000)
	libc.so.6 => /lib64/libc.so.6 (0x00007fa63ba71000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fa63c6d8000)

So ldd output is pointing to correct LD_LIBRARY_PATH setting.

Sorry for the delays. Can you run bazel with -s and paste the full command line (including where it sets all the env variables) for the failing action?

@mdfaijul Could you please check reviewer comments and keep us posted. Thanks!

@r4nt @gbaned Here is full command with -s option. I have more logs as I'm not sure what info will be useful to you.
SUBCOMMAND: # //tensorflow/python:_pywrap_py_func_version_script [action 'Executing genrule //tensorflow/python:_pywrap_py_func_version_script [for host]'] (cd /localdisk/amin/deps-tf/execroot/org_tensorflow && \   exec env - \     LD_LIBRARY_PATH=/opt/tensorflow/gcc6.3/lib64 \     PATH=/usr/local/cuda/bin:/nfs/site/home/mdfaijul/.local/bin:/opt/tensorflow/gcc6.3/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin \   /bin/bash -c 'source external/bazel_tools/tools/genrule/genrule-setup.sh; echo '\''{global:  init_pywrap_py_func;  init__pywrap_py_func;  PyInit__pywrap_py_func;  local: *;};'\'' >bazel-out/host/genfiles/tensorflow/python/_pywrap_py_func-version-script.lds') SUBCOMMAND: # //tensorflow/lite/python/interpreter_wrapper:tensorflow_wrap_interpreter_wrapper_py_wrap [action 'SWIGing tensorflow/lite/python/interpreter_wrapper/interpreter_wrapper.i [for host]'] (cd /localdisk/amin/deps-tf/execroot/org_tensorflow && \   exec env - \   bazel-out/host/bin/external/swig/swig -c++ -python -module tensorflow_wrap_interpreter_wrapper -o bazel-out/host/bin/tensorflow/lite/python/interpreter_wrapper/tensorflow_wrap_interpreter_wrapper.cc -outdir bazel-out/host/bin/tensorflow/lite/python/interpreter_wrapper -Iexternal/eigen_archive -Iexternal/swig -Ibazel-out/host/genfiles -Ibazel-out/host/genfiles/external/local_config_python -Iexternal/com_google_absl -Iexternal/gemmlowp -Iexternal/flatbuffers -Iexternal/arm_neon_2_x86_sse -Iexternal/farmhash_archive -Iexternal/swig/Lib -Iexternal/swig/Lib/cffi -Iexternal/swig/Lib/python -Iexternal/swig/Lib/std -Iexternal/swig/Lib/typemaps tensorflow/lite/python/interpreter_wrapper/interpreter_wrapper.i) SUBCOMMAND: # //tensorflow/lite/experimental/ruy:pack_avx2 [action 'Compiling tensorflow/lite/experimental/ruy/pack_avx2.cc [for host]'] (cd /localdisk/amin/deps-tf/execroot/org_tensorflow && \   exec env - \     LD_LIBRARY_PATH=/opt/tensorflow/gcc6.3/lib64 \     PATH=/usr/local/cuda/bin:/nfs/site/home/mdfaijul/.local/bin:/opt/tensorflow/gcc6.3/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin \     PWD=/proc/self/cwd \   /opt/tensorflow/gcc6.3/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections -fdata-sections '-std=c++0x' -MD -MF bazel-out/host/bin/tensorflow/lite/experimental/ruy/_objs/pack_avx2/pack_avx2.pic.d '-frandom-seed=bazel-out/host/bin/tensorflow/lite/experimental/ruy/_objs/pack_avx2/pack_avx2.pic.o' -fPIC -iquote . -iquote bazel-out/host/genfiles -iquote bazel-out/host/bin -iquote external/gemmlowp -iquote bazel-out/host/genfiles/external/gemmlowp -iquote bazel-out/host/bin/external/gemmlowp -g0 -g0 '-std=c++14' -fno-canonical-system-headers -Wno-builtin-macro-redefined '-D__DATE__="redacted"' '-D__TIMESTAMP__="redacted"' '-D__TIME__="redacted"' -c tensorflow/lite/experimental/ruy/pack_avx2.cc -o bazel-out/host/bin/tensorflow/lite/experimental/ruy/_objs/pack_avx2/pack_avx2.pic.o) ERROR: /ec/pdx/disks/aipg_lab_home_pool_01/mdfaijul/public-tf/tensorflow/lite/python/interpreter_wrapper/BUILD:60:1: SWIGing tensorflow/lite/python/interpreter_wrapper/interpreter_wrapper.i failed (Exit 1) bazel-out/host/bin/external/swig/swig: /lib64/libstdc++.so.6: version CXXABI_1.3.9' not found (required by bazel-out/host/bin/external/swig/swig)
Target //tensorflow/tools/pip_package:build_pip_package failed to build`

@r4nt @gbaned I see this issue on CentOS, but not on Ubuntu, BTW.

@r4nt @gbaned any update?

@r4nt @gbaned any update?

Just to confirm:

when you run bazel-out/host/bin/external/swig/swig on the command line you don't get the error?
can you potentially edit bazel-out/host/bin/external/swig/swig and replace it with a script that just prints LD_LIBRARY_PATH?

@mdfaijul Could you please check reviewer comments and keep us posted. Thanks!

@gbaned @r4nt @agramesh1 I am not able to reproduce the issue with current master. Hence closing this PR.

It happens to me on tensorflow 2.0.1 on centos using gcc-6.3.0
The change to the file in this PR fixes it. Thanks!
if there's a way to achieve the same results without the change, I would be interested to know.

SWIG has been a problematic when building on CentOS 6/7. The usual fix worked here:
diff --git a/tensorflow/tensorflow.bzl b/tensorflow/tensorflow.bzl
index 796143a..4624e72 100644
--- a/tensorflow/tensorflow.bzl
+++ b/tensorflow/tensorflow.bzl
@@ -1658,6 +1658,7 @@ def _py_wrap_cc_impl(ctx):
         outputs = outputs,
         mnemonic = "PythonSwig",
         progress_message = "SWIGing " + src.path,
+        use_default_shell_env = True,
     )
     return struct(files = depset(outputs))


workspace: leverage go.work to manage submodules

Locally, I use go workspace to please gopls so it's easier to work in submodules. Here's the approach,
Suppose you have a similar dir structure as the following,
.
├── btcsuite
├── lnd
...

Now create the workspace using,
go work init
go work use -r ./lnd
go work use -r ./btcsuite

Which will go through all the modules under ./lnd and ./btcsuite, add them to the go.work file.
IMPORTANT: Remember to remove ./lnd/tools from the go.work file.
You can also run go work sync to update the go.mod for every module.
And your dir would look like,
.
├── btcsuite
├── go.work
├── go.work.sum
├── lnd
...

This approach has a few pros and cons.
Pros,

modules are imported locally in lnd. I can easily edit the submodules and see the effect. For instance, running make unit tags="kvdb_sqlite" pkg=kvdb would now give me an error.
by running go work sync I can easily update the dependant module versions across all submodules.
gopls is now happy when working in submodules.

Cons,

as long as you are in the same or child dir of go.work, you can run go work sync to update go.mod. This can also be a pro, but it will create diffs in btcd and btcwallet in their current branch and this can be annoying.
our CI builds lnd differently by using older submodules, and we may still forget to tag their versions.

Solutions
There's ongoing debate about whether go.work should be checked into VCS. Because we have multiple projects, taproot-assets, loop, pool, etc, adding a go.work under lnd won't suit our case.

[ ] update development docs about creating and using go.work
[ ] add a new make command, check-gomod, that checks all the go.mod files are up-to-date - we could create a temporary go.work file and run go work sync and go mod tidy, fail the check if there are uncommitted changes. This should be built into our github actions.
[ ] consider creating repo for submodules. This may seem cumbersome at first, but with go.work, working on them locally should be seamless and, it makes us less likely to forget to update submodule versions in lnd.

IMPORTANT: Remember to remove ./lnd/tools from the go.work file.

Why's this step required?

Why's this step required?

I think it's because the tools package is used to pin versions for the tools we use in builds, not related to the actual code of lnd. The last time I tried to include it, I got quite some errors when running github actions. There may be a catch I'm missing here.


Belsas Slums NPC dialog

10928 - Belsas Slums NPC dialog
10805 - Typo fix

Hey :) just wondering who you are on the discord server


Flag tasks as outdated when last_edit_changeset is different.

When importing a geojson file that includes issues to create tasks from in a challenge I thought it would be very interesting and worth while for maproulette to detect when an feature includes a last_edit_changeset value that is different from the currently available OSM object.
I just created a challenge using a tool that generates the geojson and had a few features that had already been fixed on the OSM map. When I looked at the task page that was generated from the feature in the geojson it did not alert me that the object that I am looking at is different from the one that the feature in the geojson file was created from. I just thought it would be really nice to have a flag somewhere that showed the "outdated" task, and maybe even a way to see a count of "outdated" tasks when looking at a challenge.
Just a thought. Let me know if this is not interesting to others.

Hi @atiannicelli thanks for making this suggestion! One thing that occurs to me is that OSM elements can be modified for a number of reasons, and it's difficult to know if a modification actually impacts the changes requested by a given challenge unless a mapper manually inspects it and makes a determination (which is basically what they'd do today in their editor).
MapRoulette does currently fetch the latest task data from OSM for tag-fix cooperative challenges each time a task is loaded, as in that case MR understands exactly which changes are being proposed and can therefore intelligently determine whether updates to the OSM element have a bearing on the task.
But for standard challenges, the closest we currently offer is the "OSM Data" map layer, which loads the latest OSM data for elements shown on the task map. It'd certainly be possible to offer something similar but for just the OSM element in question. Do you think that'd be useful?

I think that what is in place probably is enough for now. I'm going to close this as I didn't know about the cooperative challenges at the time I wrote this.

After talking to others I am going to open this up. The issue happen when a challenge is created to flag an issue on a feature with last_edit_changeset of 5, and then while the task is sitting, waiting for someone to address it, the OSM mapping data is changed to update that feature to last_edit_changeset of 6. When a mapper open the challenge task that was created at changeset 5 then the task may be flagging an issue that was fixed in 6. Or the feature may have changed enough from 5 to 6 so that the task no longer applies. What we are asking for here is a visible flag on the task to indicate that the feature that the task was created to address is older than that latest version of that feature.  This may or may not indicate that the task is invalid, but it at least gives the mapper the heads up that they are looking at a task that was created using an older version of the map than they are now editing. It really is a "Heads Up" to the mapper.
The reason we are dealing with this is because some of the maps that we use to generate the challenges are a week to two weeks (or more) old since it may take that amount of time to perform a full test run and so the window for change between the OSM map version that was used to perform the scan and when this task gets worked on is bigger than normal. We have seen our mappers get confused when working on a task that asks them to make a change to a feature because when they open up the map to edit it the feature is different than was described in the task.

PR #1432 introduces a new, optional OSM History widget that can be used during task completion to view a history of the OSM element(s) in the task. If an element has been changed since the challenge data was sourced -- or has a version newer than that identified on the task via osmVersion or last_edit_changeset property -- then the history entry will show a warning icon indicating that the change was made after the task was created.


Create dronacharya

@iamdhruv Please rename lib/domains/info/dronacharya to lib/domains/info/dronacharya.txt to reflect this repository rules. Thank you!

Done.. Thanks:)

@iamdhruv Pull request merged. Thank you!

@iamdhruv Pull request merged. Thank you!


Abandoned repository

Hi @grishka
If you can mantain the repo can you open to other devs to can help in the repository? The repository of are multiple commits ahead of you https://github.com/omarss/Houseclub. I push PR to this reposirty because this are not updated.
If i can help you please contact me.
Regards!

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I saw it have invite feature, it work?

What exactly do you mean by "maintaining", anyway? I certainly could improve the bot that auto-closes issues because people just never ever fill the template.
Just to reiterate, for everyone reading this: if you aren't a developer, you aren't supposed to have a GitHub account.

Sorry i can't understand you :( I don't said any about the bot in issues
I only try to offer my hands to collaborate to have the repo updated

To have it updated with what? It's fine as it is.

The whole clubhouse hype has subsided anyway. And an official app is on the way, so there's not much sense to continue with this project.

In your repo has 20 PR with new features and bug fixes, that can make better the applicattion.
I understand you, but i think that the official app need almost 3 months, so if you like please close PR and issues and redirect users to my repo for example, because i try to learn more about the application and try to upgrade this.
My repo are https://github.com/vgpastor/ClubHouseUnOfficial and try to upload to playstore, mention you and your repo of course.

I reviewed all those PRs and there were many things I didn't like about them, which is why I didn't merge them.

But the open source consist in speak with other devs and accept modifications and collaborations, if don't like the code of others devs speak with them and propose an alternative, so you can the repo better.

There's such a thing as feature completeness. It's when you say your product is done. When there's nothing to be added to it. When you only fix bugs but don't add any new features. It's a misconception that if something isn't being constantly updated, it's dead.

But the bugs make the users can't test better the applicattion and send new feedback. In the actually repos that are not updated when users add code it's the same that don't have anything. If i refactor anything for exampe and you don't merge in a few days it's probably that others devs write anothers lines that in the future when you decide merge them exists conflict and make unsastainable.
I think that it's better add PR every time that you can, for this reason i offer you collaborate to make updated the repo

There is lots of spaghetti code and messy structures in the PRS my man. If he merges those prs it will turn into a mess , I have forked this repo and I am doing updates from time to time just for fun following the same code structure he built at first. But please don't take this too serious as it is not an official app , it is not supposed to be acting as an official app at all. Just the basic stuff for fun and for educational purposes as well.
If you REALLY want new updates try this release. Regardless that the official app is coming soon please don't do any pull requests or report issues to it or request new features because I will only work on it if I have free time. Thank you.

@itszakjo Your repository its only two commits ahead @grishka in https://github.com/vgpastor/ClubHouseUnOfficial you can see a version that i send to google play and are waiting approval, while you can download https://github.com/vgpastor/ClubHouseUnOfficial/releases/tag/2.0.1
I'm open to make you a developer in the repository if you like and centralize the updates

Lol this repo u sent was actually copied from - houseclub-updated - my repo in the first place

a version that i send to google play and are waiting approval

Publishing unofficial apps based on a reverse engineered API on Google Play is a very terrible idea. I hope you aren't doing that from an account that is valuable to you.

Someone uploaded the app already on google play and Google play took it down already.

Yeah, told you so

@itszakjo i see the forks and continue , the most updated repo was https://github.com/omarss/Houseclub.
I'm documented the base of codes

I don't think it's worth further. Here I found a complete application based on this project.
https://play.google.com/store/apps/details?id=app.clubhouse.br

@houska85 well, looking at the screenshots, it lacks many features included by forks, doesn't include the warning about possibility of being banned and has tons of ads.

I decompiled the application to be sure. Currently, almost everything is there. Ads are being implemented but are still inactive.


Remove couchers-sesh on bad token

If the token is no longer valid, it is impossible to see the logged-out homepage, as the middleware sees the cookie and tries to show the dashboard, which goes to the login page.
It seems the cookie isn't even removed on log-out?

I think we should figure out a better system here. Currently there are 3 states that aren't properly synchronized:

The HTTP cookie state (no cookie = logged out, cookie = logged in)
The backend cookie state (no cookie/wrong cookie = logged out, right cookie = logged in)
The web local storage state (keeps track of authenticated and user id, etc)

Problem is that the web (JS) can't read the HTTP cookie for security reasons. To query backend state requires making an API call.
I wonder if a good solution would be to have two cookies: couchers-sesh and couchers-user-id that are synchronized. The latter doesn't have to be protected from JS, and the backend would complain and return a 401 unauthenticated if they don't match up.


Why UC browser changed from 9.9 to 11?

https://github.com/Fyrd/caniuse/blob/master/features-json/apng.json#L272
    "and_uc":{
      "11":"n"
    },

See #2857.

I updated it to the latest version, that's all.

@Fyrd In autoprefixer if the configuration of the "uc": "9.9" will lead to the error, because this version does not exist in caniuse.

Multiple mobile browsers only have the latest version listed on caniuse,
always been this way so I'm not sure why autoprefixer shouldn't be able to
deal with it.
On Sun, Oct 16, 2016, 7:17 PM 一丝<EMAIL_ADDRESS>wrote:

@Fyrd https://github.com/Fyrd In autoprefixer if the configuration of
the "uc": "9.9" will lead to the error, because this version does not
exist in caniuse.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fyrd/caniuse/issues/2888#issuecomment-254097297, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAaY0HK8tz5Nt1FFczCPXlzV1VR5qTcrks5q0tqzgaJpZM4KTb3f
.

@yisibl  last uc version should work in browserslist


[Feature Request] The Tree component adds icon function

Existing Component
是
Component Name
tree
Description
tree组件建议增加图标功能，具体可以参考下 Ant Design of Vue中图标

Sorry, we have no plan to support this.


Rock aborts with interfaces

I'm trying to port some code to ooc.
I get this error - it doesn't give me any line numbers:
[31;1m[Exception in CastWriter]: Couldn't find implementor for InterfaceDecl IComponent  in CoverDecl Pointer
(SIGABRT) abort program
IComponent is an interface. If I change it to abstract class, then it just crashes on the next interface.

I think I get it. So, I'm coding a library, and it exposes interfaces to be used by the client. I am storing arrays and lists of some of those interfaces. Those are the ones that have the problem. In those case, and abstract works. It's predictable, and I can code for it, so I guess it's just an 'undocumented feature' :)


Remove redundant this

var debug = this.debug was already in scope, so use debug instead of
this.debug, since the value of this.debug does not change.

Thanks @mark-rushakoff !


Where should I write "RSA public key"?

Version of react-native-iap
Install the latest version yesterday
Version of react-native
0.57.1
Platforms you faced the error (IOS or Android or both?)
android
Expected behavior
I did not write it but it run
Actual behavior
Tested environment (Emulator? Real Device?)
Emulator. Signed App distribute "Alpha Version"
Steps to reproduce the behavior
const itemSkus = Platform.select({
ios: [
'test1',
'test2',
'test3'
],
android: [
'test1',
'test2',
'test3',
],
});
async componentDidMount() {
try {
console.log("itemSkus",itemSkus)
const products = await RNIap.getProducts(itemSkus);
this.setState({ products : products });
console.log("point componentDidMount",products)
} catch(err) {
console.log("point componentDidMount Error ",err); // standardized err.code and err.message available
}
}

You don't need to write it down in recent iap module.


Is there a ransac operation in pycolmap.match_exhaustive or pycolmap.IncrementalMapperOptions() ?

Hi, I am reading the demo pipeline you provided in Readme.
output_path: pathlib.Path
image_dir: pathlib.Path

output_path.mkdir()
mvs_path = output_path / "mvs"
database_path = output_path / "database.db"

pycolmap.extract_features(database_path, image_dir)
pycolmap.match_exhaustive(database_path)
maps = pycolmap.incremental_mapping(database_path, image_dir, output_path)
maps[0].write(output_path)
# dense reconstruction
pycolmap.undistort_images(mvs_path, output_path, image_dir)
pycolmap.patch_match_stereo(mvs_path)  # requires compilation with CUDA
pycolmap.stereo_fusion(mvs_path / "dense.ply", mvs_path)

Do you apply for a ransac operation here? If so, where is it? How can I adjust the parameters of ransac? Thank you so much.
Best regards

Hi @ForcewithMe66, the demo pipeline uses all default parameters. However, ransac is used extensively in the mapping pipeline. I suggest having a look into the IncrementalMapperOptions for an extensive list of available options and default parameters. From python, you can also run help(pycolmap.IncrementalMapperOptions) for an overview.

We indeed don't have bindings for IncrementalMapperOptions.mapper and IncrementalMapperOptions.triangulation. We'd need to bind IncrementalMapper::Options and IncrementalTriangulator::Options and create setters+getters in IncrementalMapperOptions. Feel free to give it a shot if you have time.

These options are now exposed, since https://github.com/colmap/pycolmap/commit/89ce832326e91a63a1188bdeacc2888f15477cea.


Help examples can specify the profiles they belong to

Specify min_profile and/or max_profile on any help examples you want to only appear for given profiles.
Currently implemented for storage account create -h

Other changes:

Change sample profiles to be dates instead of just YYYY-sample.
API version comparison function supports 'latest'
Default to 'latest' if a given profile version is not valid.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[n/a] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[n/a] Each command and parameter has a meaningful description.
[n/a] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #2919 into master will increase coverage by 0.01%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master    #2919      +/-   ##
==========================================
+ Coverage   63.08%   63.09%   +0.01%
==========================================
  Files         466      466
  Lines       26227    26252      +25
  Branches     4020     4027       +7
==========================================
+ Hits        16544    16565      +21
+ Misses       8636     8598      -38
- Partials     1047     1089      +42

Impacted Files
Coverage Δ

...azure-cli-core/azure/cli/core/profiles/__init__.py
100% <ø> (ø)
:arrow_up:

...storage/azure/cli/command_modules/storage/_help.py
100% <ø> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/_help.py
55.4% <100%> (+1.94%)
:arrow_up:

src/azure-cli-core/azure/cli/core/cloud.py
70.09% <100%> (+0.56%)
:arrow_up:

.../azure-cli-core/azure/cli/core/profiles/_shared.py
71.55% <77.27%> (-1.17%)
:arrow_down:

...-testsdk/azure/cli/testsdk/recording_processors.py
50.6% <0%> (ø)
:arrow_up:

...e-cli-iot/azure/cli/command_modules/iot/_params.py
95.74% <0%> (ø)
:arrow_up:

...li-role/azure/cli/command_modules/role/commands.py
84.44% <0%> (ø)
:arrow_up:

...ure-cli-vm/azure/cli/command_modules/vm/_params.py
96.19% <0%> (ø)
:arrow_up:

...i-redis/azure/cli/command_modules/redis/_params.py
77.77% <0%> (ø)
:arrow_up:

... and 18 more

Continue to review full report at Codecov.

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


code after the yield in a bound callback does not always execute on key release

🐛 Bug
In some situations, code after the yield in a bound callback does not execute on key release. e.g. just after clicking a layer in the layer list
EDIT should've read the docs - happy for this to be closed if it's an issue you're aware of
To Reproduce
import napari
from skimage import data

viewer = napari.Viewer()
layer = viewer.add_image(data.astronaut())

@viewer.bind_key('h')
def hide_image_layer(event=None):
    print('hiding')
    layer.opacity = 0
    yield
    print('showing')
    layer.opacity = 1

https://user-images.githubusercontent.com/7307488/122301440-97449e80-cef8-11eb-94ea-77c064b1e6ca.mp4
In this video, if I have clicked in the canvas then the callback works as expected: hide on keypress, show on key release
If I click on a layer in the layerlist then hit my key only the first part of the callback is activated, the code after the yield never executes.
The behaviour is the same with any keybinding I've tried. I'm happy to look into this but would need some pointers on how to debug 🙂
Environment
napari: 0.4.9
Platform: macOS-11.2.1-x86_64-i386-64bit
System: MacOS 11.2.1
Python: 3.9.4 | packaged by conda-forge | (default, May 10 2021, 22:13:15)  [Clang 11.1.0 ]
Qt: 5.15.2
PyQt5: 5.15.4
NumPy: 1.20.3
SciPy: 1.6.3
Dask: 2021.05.0
VisPy: 0.6.6

OpenGL:
  - GL version:  2.1 Metal - 71.0.7
  - MAX_TEXTURE_SIZE: 16384

Screens:
  - screen 1: resolution 1440x900, scale 2.0

Plugins:
  - console: 0.0.3
  - scikit-image: 0.4.9
  - svg: 0.1.5

@alisterburt I don't understand your edit... is this working as expected or not?

@jni the docs have a note saying

Currently the keybindings only work when the main canvas is in focus, we are working to ensure they always work.

So I guess the behaviour comes under that umbrella and is expected if you have read the docs properly, otherwise it still feels pretty unexpected

I believe this is the same issue as #2714

I can't reproduce this.
I used the script from the OP and then clicked in various places and used the keybind:
https://github.com/napari/napari/assets/76622105/4c5934e2-ebb4-4001-a268-9c53f190c052
Keybinds have changed a lot so I think this is fixed.
This means the docs should be updated. I'll make an issue in the docs repo.


Fix bug where Dash changes argument to property

Hello,
When calling Hashie::Dash.property, where the property_name argument is a string, the method modifies the original value of the argument using the String#concat method, appending '=' to the string.
klass = Class.new(Hashie::Dash)

my_property_name = 'foo'
# => 'foo'

klass.property(my_property_name)

my_property_name
# => 'foo='

I have changed the Hashie::Dash code to create a new String object instead of modifying the original.
Thanks!

Good catch, see my comment above.

I've updated the commit to use your suggested fix.

Merged, thanks.


Extremely long delay to display the control

The control is working. I have a reasonable amount of options, i.e. around 5 options in the control, not a million.
When I open a PBI using this control, everything displays good as usual. However, this multivalue control takes a weirdly long amount of time to display... i.e. 20 seconds.
This really ruinds the UX of this control.
This happens most of the time, i.e. 90% of the time I open a PBI. Weird enough, around 10% of the PBIs I open, have this control inmediately available. I did not find any consistent pattern on which PBIs have it available and which takes weirdly long to load.
Any ideas?

We're seeing this behavior as well.

@mohitbagra  @ostreifel  Can you take a look?

@jdshkolnik  @Jose-CF  Are you still seeing this issue? Has the behavior changed?

Still happens in TFS.

Can you take a fiddler trace of the page when this happens and share with us? We can provide a private email to send it to. @jdshkolnik

Same today in TFS 2017 Update 1 on a page with 3 multivalue controls. Sometimes slow, sometimes showing a message that it took too long

What version of the control are you on?
On Wed, Mar 21, 2018, 8:48 AM Bernard Vander Beken<EMAIL_ADDRESS>wrote:

Same today in TFS 2017 Update 1 on a page with 3 multivalue controls.
Sometimes slow, sometimes showing a message that it took too long
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vsts-extension-multivalue-control/issues/39#issuecomment-374987600,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASlGsbh0aEgYiedAl_R03cib8kPm3K3ks5tgnZTgaJpZM4R43J4
.

@mmanela Can you say whether the likely bottleneck for this is

our environment (TFS application server, SQL Server)
calls between TFS server and Microsoft services.
something else?

Hello,
We are using VSTS.
Over here seems the issue is fixed. i.e. the box with the options appears now quickly all the time (seems).

@jawn  There is a newer version of the multivalue control that you should update to.

Updated to 1.0.18.
Still see a slight delay from < 1 second to around a second, is this to be expected in a LAN environment between PC and TFS server? So should you ever see the 'loading' or should this be instantaneous?

Note that we have 3 of these on 1 form.

A slight delay is expected but the one you saw before wasn't. These extensions are sandboxed and loaded in frame after the form loads so they will not appear instantly. Ideally, once the JS is cached though, they should load quickly.

Closing this due to inactivity.


pytest-cov: does not cover install.py

Because it is not under the __init__.py directory?
Check py-test-cov source code, and etc.

Maybe this issue is the reason.: https://github.com/pytest-dev/pytest-cov/issues/88


Getters Chinese document translation defect

Getters Chinese document translation defect. It should be consistent with English documents. It is ‘getters’ instead of ‘getter’.

See also https://github.com/awxiaoxian2020/pinia/pull/1#discussion_r1002812260


Create and maintain release notes/change log

It would be useful to have a way to generate change log and release notes, especially for breaking API changes.

This is done! https://jax.readthedocs.io/en/latest/CHANGELOG.html


Private  repository, public issue, need an entry link

Private  repository, public issue. Can only be accessed via URL
The dashboard needs an entry link for easy access
...

Please remove the Button "new Site" in the Wiki for Anonymous. They dont need this Button.

@ModdyLP "new Site"?

Here you see it, i am anonymous and see a link to create a new wiki site.

@ModdyLP Can you reproduce on demo site? I can't reproduce..

No, there is no button. I am using the latest release, but not the develop version. May it was removed after the release in the develop version?

@ModdyLP define "latest release"

Release 0.11, I see you release a day ago another release. I haven't seen it because normally the release are on Tuesday. :D I will update an look if there is a change.

@ModdyLP yeah... found a critical bug :trollface:

Nice stuff... just missing access link for a private repo with public wiki/issues XD
But very happy to have this new feature on Gogs. Thanks guys.

I have make the Update an voila the bug is already there. I have after the Update also the button for creating a new wiki site as anonymous.


GitHub actions bump

Bumping GitHub Action versions to those supporting Node.js v20 - since v16 is now deprecated.

Release notes for these versions don't suggest any breaking changes from last major.

That's correct - it's basically just the bump from node16 -> node20 in most cases.


How to set the loading state to true?

I'm using #446 method to custom a search from remote server. However, the XHR request will take 2-4 seconds to return search result. I couldn't find a method to set Choices to loading state in the document.

+1


REGEX REPLACE in CMakeLists.txt fails for complex directory paths

Bug description
Pinocchio build fails when using complex (symbolic) directory paths, e.g., /test_pinnochio_build/1.0+gitAUTOINC+8ajb23j4b/pinocchio/build, as  REGEX REPLACE fails to correctly amend strings before attempting to create symbolic links (or copying).
This issue was found while using pinocchio with the Yocto SDK.
Expected behavior
The build should not fail for any directory paths given.
Reproduction steps
Steps to reproduce the behavior:

Create a test folder:

mkdir -p test_pinnochio_build/1.0+gitAUTOINC+8ajb23j4b

Clone Pinocchio

git clone git@github.com:stack-of-tasks/pinocchio.git

Initialize submodules

cd  test_pinnochio_build/1.0+gitAUTOINC+8ajb23j4b/pinocchio
git submodule update --init

Build

mkdir build && cd build
cmake ..

See error

Solution
The problem can be solved by replacing REGEX REPLACE with REPLACE, which given the fact that the desired string operation doesnt require the flexibility of regular expressions may make sense to move away from anyways (unless I am missing something here which might be the case as I am pretty new with Pinocchio). The swap must be done in three files:
1. Toplevel CMakeLists.txt
2. Python bindings CMakeLists.txt
3. Unittest CMakelists.txt
System

OS: Ubuntu 22.04
Pinocchio version: 2.6.20

Thanks @Pathorse for the nice report. @nim65s @jorisv Could you have a look to this issue in more depth?

I think we should simply switch to a more standard CMake use, and:

stop simlinking stuff around
list all header / sources in CMakeLists.txt

If agreed, I probably can work on it this week, but probably it would be better to do this on the private branch for v3

@Pathorse This issue should be fixed by #2070

Thanks!

Thanks a lot!

Solve via #2070


Revert "kubelet: Remove GetRecentDockerContainersWithNameAndUUID."

Reverts GoogleCloudPlatform/kubernetes#6470 , which breaks e2e test as per issue #6537

LGTM

LGTM


[question] Redis sentinal connect

Hi guys,
I tried to connect Sentinel with the following code:(I added password argument)
var redis = new Redis({
sentinels: [{ host: 'localhost', port: 26379 }, { host: 'localhost', port: 26380 }],
name: 'mymaster',
{password: ****}
});
it didn't work , I validated all the arguments a few times. I also checked the sentinel configurations,
I checked that the the name is of the master node. (our IT admin configured it.)
my question is how can I debug it? is it the correct syntax?  maybe there are known pitfalls regarding redis config that prevent the connection.
Thanks,
Shira

It looks correct. You can enable the debug mode to see what happened (DEBUG=ioredis:* yourapp.js).

Thank u very much!


Multiple superclasses exception with Apache Kafka

Hi, I am trying to search files by extension:
    new FastClasspathScanner().matchFilenameExtension("avsc", new FileMatchContentsProcessor() {
            @Override
            public void processMatch(String relativePath, byte[] fileContents) throws IOException {
                System.out.println(relativePath);
            }
        }).scan();

I have Apache Kafka on the classpath, it is compiled in Scala, which has a kind of multiple inheritance, so FastClasspathScanner throws an exception complaining about multiple superclasses.
I do not need it to parse the Scala class model, just not to throw the exception or to provide a way to suppress it.
I have always wished for the capabilities you provide, please help make it compatible with our stack.
Exception in thread "main" java.lang.RuntimeException: java.lang.RuntimeException: Multiple superclasses of class kafka.consumer.Blacklist : [kafka.consumer.TopicFilter, scala.runtime.AbstractFunction1]
	at io.github.lukehutch.fastclasspathscanner.FastClasspathScanner.scan(FastClasspathScanner.java:939)
	at io.github.lukehutch.fastclasspathscanner.FastClasspathScanner.scan(FastClasspathScanner.java:963)
	at io.github.lukehutch.fastclasspathscanner.FastClasspathScanner.scan(FastClasspathScanner.java:982)
	at com.mycompany.Main.<init>(Bifrost.java:43)
	at com.mycompany.Main.main(Bifrost.java:90)
Caused by: java.lang.RuntimeException: Multiple superclasses of class kafka.consumer.Blacklist : [kafka.consumer.TopicFilter, scala.runtime.AbstractFunction1]
	at io.github.lukehutch.fastclasspathscanner.scanner.ClassInfo.addSuperclass(ClassInfo.java:361)
	at io.github.lukehutch.fastclasspathscanner.scanner.ClassInfoUnlinked.link(ClassInfoUnlinked.java:119)
	at io.github.lukehutch.fastclasspathscanner.scanner.Scanner.call(Scanner.java:328)
	at io.github.lukehutch.fastclasspathscanner.scanner.Scanner.call(Scanner.java:55)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)

Interesting.. I quickly hacked together basic Scala support previously, which fixed a problem for a previous bug reporter, but I always suspected there would be more issues lurking, because Scala does lots of weird stiff with "companion classes".
Can you please put together a minimal standalone test case illustrating the problem, ideally with quick  instructions on how to set it up and run it in Eclipse (since  I know very little about Scala)? I do want to make sure Scala is properly supported. Thanks!

I do not have any Scala code in my project, it is just the dependency to Kafka (using Gradle):
    // Kafka
    compile 'io.confluent:kafka-avro-serializer:1.0.1'
    compile ("org.apache.kafka:kafka_2.10:<IP_ADDRESS>") {
        exclude group: 'com.sun.jdmk', module: 'jmxtools'
        exclude group: 'com.sun.jmx', module: 'jmxri'
        exclude group: 'org.slf4j', module: 'slf4j-log4j12'
        exclude group: 'log4j', module: 'log4j'
    }

It is Kafka that comes compiled in Scala.
You should be able to reproduce it just having the dependency to it.

Thanks, I can duplicate it.
This is complicated... scala.util.parsing.combinator.Parsers$Error$ has superclass scala.runtime.AbstractFunction2, but scala.util.parsing.combinator.Parsers$Error has superclass scala.util.parsing.combinator.Parsers$NoSuccess. The extra $ on the end of Parsers$Error$ is a companion class marker (there is also a suffix ...$class that is used for other companion classes). I am merging companion classes together right now, so that you don't have to deal with more than one class name when using Scala classes, but maybe this indicates that I can't do that in general. I'm going to have to think about the right way to handle this, but unfortunately I don't know much about Scala, so I don't know what the expectation should be. Do you have good knowledge of Scala?

Hi, this exception is removed in  2.0.2, which should fix your problem. I added some additional methods in this release that allow this to be dealt with on a case-by-case basis. Upgrading to 2.0.2 should fix your problem. I'll close this bug, because the exception doesn't even exist anymore, so it won't be thrown, but please comment here to confirm that it's working for you now.

It worked, thank you!
I have studied Scala a bit, but I am not using it.
If you want to support it, I think you need to consider its type system (http://docs.scala-lang.org/tutorials/tour/unified-types.html) and also its multiple-inheritance mechanisms like Traits and Mixins (http://docs.scala-lang.org/tutorials/tour/mixin-class-composition).
The hard part is that you will not see these structures directly, but only the crazy bytecode it generates.

@gustavobilert Thanks for the links. It is exactly mixins that caused this problem. Since Java has only a single inheritance model, Scala compiles two classes (or sometimes three), with slightly different name variations (it adds the suffix "$" or "$class" to the name). FastClasspathScanner merges these into the same ClassInfo object. I just removed the restriction of having just one superclass, but added methods that expect a single superclass for cases where that is the most likely scenario (i.e. for regular Java).
It is highly likely Scala does other weird things that I'm not handling properly. If you or anyone else comes across these, please file additional bug reports. Thanks!


Add script to create BreweryPi databases and pi user

Don't think this is necessary anymore. We'll eventually release images with a configuration.


withScope is marked as optional, but is required

In the README, it looks like withScope is marked as not required, but an error is thrown if you don't provide it. Looking at the code, it definitely seems to be required: https://github.com/BrunoScheufler/graphql-middleware-sentry/blob/master/src/index.ts#L75
Any reason it couldn't be defaulted to the identity function? If so I'd be happy to make a PR.

Thanks for adding the default 🙌 I'll make sure to release this as soon as possible 👍

:tada: This issue has been resolved in version 3.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


please release the master

the latest release is from 2017
please update it
I tried to build it but I get alot of errors
I don't want to look into golang atm ;)
thanks

I got your reply to the PR thread. I also received the 404 error when I built plexdrive from source with Go 1.13. In the below thread, it was found that it works when built from 1.12.
https://github.com/dweidenfeld/plexdrive/issues/327#issuecomment-510694073

ok, I got it
git clone was just not enough
I had to run go get first

install go
mkdir $GOPATH/src
cd $GOPATH/src
go get github.com/dweidenfeld/plexdrive
cd $GOPATH/src/github.com
git clone https://github.com/dweidenfeld/plexdrive.git
cd $GOPATH/src/github.com/plexdrive
go build -o /tmp/plexdrive
use the new file /tmp/plexdrive

now it works

Just running go get -u github.com/dweidenfeld/plexdrive should be enough. It should download all the dependencies, compile plexdrive and install it to $GOPATH/bin/.


Web API client login throwing 400 Bad Request

Before submitting an issue, make sure you have:

[x] Updated to the lastest version v1.6.0
[x] Read the README and docs
[x] Searched the bugtracker for similar issues including closed ones
[x] Reviewed the sample code in tests and examples

Which client are you using?

[ ] app (instagram_private_api/)
[x] web (instagram_web_api/)

Describe the Bug/Error:
When attempting to login to an account using the web api, the library client.py file is throwing a 400 status HTTPError "Bad Request" while trying to open https://www.instagram.com/accounts/login/ajax/.
I just used this exact same code in October 2021 without issue. It feels like Instagram changed something on their end, but I'm not certain what.

Paste the output of python -V here: Python 2.7.9
Code:
# Example code that will produce the error reported
import hashlib
import string
import random
from instagram_web_api import Client, ClientCompatPatch, ClientError, ClientLoginError
import  time

class MyClient(Client):

    @staticmethod
    def _extract_rhx_gis(html):
        options = string.ascii_lowercase + string.digits
        text = ''.join([random.choice(options) for _ in range(8)])
        return hashlib.md5(text.encode()).hexdigest()

    def login(self):
        self.username =<EMAIL_ADDRESS>        self.password = 'examplepassword'
        #self.on_login = True

        """Login to the web site."""
        if not self.username or not self.password:
            raise ClientError('username/password is blank')

        #time = str(int(datetime.datetime.now().timestamp()))
        #enc_password = f"#PWD_INSTAGRAM_BROWSER:0:{time}:{self.password}"
        enc_password = ("#PWD_INSTAGRAM_BROWSER:0:{}:{}".format(int(time.time()), self.password)) #python2.7

        params = {'username': self.username, 'enc_password': enc_password, 'queryParams': '{}', 'optIntoOneTap': False}
        self._init_rollout_hash()
        login_res = self._make_request('https://www.instagram.com/accounts/login/ajax/', params=params)
        if not login_res.get('status', '') == 'ok' or not login_res.get ('authenticated'):
            raise ClientLoginError('Unable to login')

        if self.on_login:
            cache_settings = login_res.settings
            with open("_instagram_settings.json", 'w') as outfile:
                json.dump(cache_settings, outfile, default=to_json)

            #on_login_callback = self.on_login
            #on_login_callback(self)
        return login_res

Error/Debug Log:
Traceback (most recent call last):
  File "test_instagram.py", line 14, in <module>
    web_api.login()
  File "C:\pythonapps\myapp\MyClient.py", line 33, in login
    login_res = self._make_request('https://www.instagram.com/accounts/login/ajax/', params=params)
  File "C:\Python27\lib\site-packages\instagram_web_api\client.py", line 289, in _make_request
    raise ClientBadRequestError(msg, e.code)
instagram_web_api.errors.ClientBadRequestError: HTTPError "Bad Request" while opening https://www.instagram.com/accounts/login/ajax/```

After further investigation, this seems to be authentication/security issues from Instagram. I got the original code to work using a different account. Closing this.


SQL logs with Accelerate have a trailing space at the end

SQL logs with Data Proxy have an extra space at the end compared to logs with regular client.
This is not a big issue but it may be annoying when doing things like snapshot testing and switching between direct connection to the database and the Data Proxy.
Steps to reproduce:

Use Data Proxy client
Enable query logs: const prisma = new PrismaClient({ log: [{ emit: 'event', level: 'query' }] })
Add event handler: prisma.$on('query', (e) => console.log(JSON.stringify(e.query)))
Do some queries so that events are emitted

             without Data Proxy ↓
-"SELECT ... WHERE 1=1 OFFSET $1"
+"SELECT ... WHERE 1=1 OFFSET $1 "
                 with Data Proxy ↑

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


Easier OpenGL shared contexts

So we added support for custom OpenGL toolkits, so that the application can take over creation of the window and OpenGL context, and RenderManager will tell the application where to put the window, what size to make it, etc...
That's good, but what if the user just wants to create their own context for the preview window, and let RenderManager create the window and context for the HMD window, and setup resource sharing between contexts automatically?
The feedback I'm getting from users is that setting up shared OpenGL contexts is too difficult. They can write their own toolkit, but all they really wanted was for RenderManager to create its own window and context and setup context sharing automatically. This is how the OpenGLSharedContext example currently works, for example. But that sample only appears to work when the application is also using SDL. Can we make it work for other toolkits using low-level OS-specific context calls like wglGetCurrentContext, and the Mac/Linux equivalents?
Above just making it easier for developers, this would make it possible to take advantage of RenderManager's ability to open up a direct mode context. We wouldn't necessarily expect applications to be able to create their own direct mode windows and contexts (through Direct3D/OpenGL interop or through extensions, if available).

OpenVR handles this very nicely. Their Steam VR Compositor operates as a separate process, so there is no context issues to work around. It uses DirectX & handles interop for you, if your application is in OpenGL. It also takes care of Direct Mode. I went in expecting similar functionality from the OSVR Render Manager & have been struggling since. My advice would be to take OpenVR's lead & not try and integrate the rendering manager within the same application, which causes the whole context issue. I want my own "mirror" rendering window (for spectators) & the rendering manager should be a separate tool to manage displaying to the headset.
Another huge benefit to making the rendering manager a separate process: if the host application locks up or takes too long to submit frames, the rendering manager can continue to run & display an empty scene with tracking. This prevents motion sickness during loading times, for example.

We already have plans to implement a compositor similar to OpenVR, as a separate process. That is sort of a longer-term project. This issue is somewhat orthogonal in that, compositor or not, we should be able to make it easier to share OpenGL contexts and/or handle direct3D interop for you.
Our previous solution, the custom toolkit, enables this, but it's a big hammer where the application becomes responsible for creating the window on the HMD and creating the context and setting up resource sharing with any other context in the application (like a preview window).
For this issue I'd like to see the following two scenarios made easier:

Allow RenderManager to create an OpenGL context (using SDL2 currently) and window, and have it automatically setup resource sharing with the existing context, if any. This would be extended mode only for now.
Allow RenderManager to create a Direct Mode Direct3D context, configure OpenGL interop with an existing OpenGL context, and have everything just work.

I commited my jMonkeyVR OSVR "work in progress", the meat of which can be seen here:
https://github.com/phr00t/jMonkeyVR/blob/master/src/jmevr/input/OSVR.java
This could be used as a framework for improving Render Manager's behavior. #2 is where we want to be, but if that is too far out, I could work with #1 in the meantime.
Right now, jMonkeyVR has the Render Manager create a context first, since it seems to play the "most nice" with my LWJGL-created context later. I can see output to my primary window, but the Render Manager displays white & gives me GL_INVALID_OPERATION errors, as seen in this issue:
https://github.com/sensics/OSVR-RenderManager/issues/223
If I create the context first with LWJGL, then try to open a Render Manager window, my primary display is black.

Here is a built JAR of the OSVR "Work in progress":
https://drive.google.com/open?id=0Bza9ecEdICHGT0lRZy1DR0NWR1k
github wouldn't take it due to the 100MB limit. It was originally around ~21MB, adding OSVR brings it to 103MB ;)