Breakpoints on justMyCode: false not working properply in 1.90.0

Type: Bug

Behaviour
Breakpoints on justMyCode: false not working properply in 1.90.0
Also submitted on vscode main github issues since I don't know where the error occurs. Someone else also posted this issue on main vscode GH, but with no details (link on the page).
Steps to reproduce:

VSCode version 1.90.0
"justMyCode": false,
set breakpoint anywhere in outside of your code.
hit run - the breakpoint will highlight somewhere in your code, the breakpoint outside of your code will be in green and steps will not move the selection forward (the app will proceed however). With each step, vscode will come back to some line or breakpoint in your code.

Works in version 1.89.1

Diagnostic data

launch.json configuration

{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "chat.py",
            "type": "debugpy",
            "request": "launch",
            "program": "${workspaceFolder}/chat.py",
            "console": "integratedTerminal",
            "justMyCode": false,
        },
        {
            "name": "uvicorn local",
            "type": "debugpy",
            "request": "launch",
            "env": {
                "OBJC_DISABLE_INITIALIZE_FORK_SAFETY": "YES"
            },
            "module": "uvicorn",
            "args": [
                "app:app",
                "--reload",
            ],
            "justMyCode": false,
        },
    ]
}

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

nothing

Output for Python Debugger in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python Debugger)

2024-06-06 14:59:31.112 [info] DAP Server launched with command: .../.venv/bin/python .../.vscode/extensions/ms-python.debugpy-2024.6.0-darwin-arm64/bundled/libs/debugpy/adapter
2024-06-06 14:59:32.141 [info] Connecting to DAP Server at:  <IP_ADDRESS>:59701
2024-06-06 14:59:32.167 [info] Connecting to DAP Server at:  <IP_ADDRESS>:59701

Extension version: 2024.6.0
VS Code version: Code 1.90.0 (89de5a8d4d6205e5b11647eb6a74844ca23d2573, 2024-06-04T19:34:48.028Z)
OS version: Darwin arm64 23.5.0
Modes:

Python version (& distribution if applicable, e.g. Anaconda): 3.10.13
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): VirtualEnvironment

System Info

Item
Value

CPUs
Apple M1 (8 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
1, 2, 2

Memory (System)
16.00GB (0.03GB free)

Process Argv
--crash-reporter-id cf0a53ce-7464-4c78-875e-81033be10546

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:31000450
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
jchc7451:31067544
chatpanelt:31048053
dsvsc021:30996838
945dj816:31013170
pythoncenvpt:31062603
a69g1124:31058053
dvdeprecation:31068756
pythonprt:31056678
dwnewjupytercf:31046870
26j00206:31048877

Apparently it's caused by the Python extension
https://github.com/microsoft/vscode/issues/214433#issuecomment-2152262171

Same issue

+1

Rolling back to 1.89.1 resolves the issue.

+1

Rolling back to 1.89.1 resolves the issue.

How to roll back to 1.89.1? I do not see this option in available versions

For those who are still wondering (as I was just a few minutes ago), you can download v1.89 from here

This should be a problem in VS code. See this issue:
https://github.com/microsoft/vscode/issues/214433


better performance  (Delay rendering of non-focused tabs to next tick)

Motivation
Delay rendering of not focused tabs to the next tick, for better performance.
In order to keep it opt-in and fully backward compatible, I decided to add a new prop: delayRenderOfNonFocusedTabs to TabView.
If you set this prop to true, your current tab will be rendered, and only on the next tick the other tabs will be rendered.
edge case: If you turn this option on, and the render time of the current tab is long, and the user is expected to switch to other tab very quickly, then he will see a blank tab for a split second. that's why this option is currently opt-in.
Results:
When clicking on a group called "blamos2", you can see how much time it takes to load the group items in a page with many tabs, before and after the change

Before
After

@satya164, as you suggested, I added this change to the tabView. I also edited the description of this PR, please read it:)

I'll check soon. Thank you!

@satya164 I added the props to the docs also.

Thank you! Merged in 5b0a661b24ea587a0e4c282c6cbbd553d402877e
I have removed the prop for now and enabled it by default. I'll add the prop if people want it.


Gradle 7.4 regression: setTaskNames no longer works if configure-on-demand is used

Expected Behavior
If I use gradle.startParameter.setTaskNames(...) during configuration, before the task graph is calculated, the modified list of task names should be used to determine which tasks are run.
Current Behavior
If configuration-on-demand is enabled, setTaskNames has no effect. Without configuration-on-demand, it works as expected.
Context
This worked in 7.3.3, and we've been relying on this behavior for some time.
Our company uses a variety of plugins that automatically run their own lock-writing tasks if --write-locks is used. This breaks that mechanism for the handful of large monorepos we have that use configure-on-demand for performance reasons.
We can work around this for the time being by manually adding those lock-writing tasks where appropriate, but this is error-prone, and the difference in behaviors will be annoying and confusing for users of the repos.
Steps to Reproduce
Repro at: https://github.com/AlexLandau/set-task-names-gradle-7.4
Your Environment
Gradle 7.4
Java 11
MacOS Catalina 10.15.7

Clarifying note: There are some cases where I'd expect a different behavior under configure-on-demand, if e.g. I run ./gradlew :foo:build and the setTaskNames invocation is in the unrelated bar project that doesn't get configured. But I would expect that if a project does get configured while searching for tasks, then any setTaskNames call that happens during that time will take effect.

Sorry for the late reply.
Thank you for providing a valid report.
The issue is in the backlog of the relevant team, but this area of Gradle is currently not a focus one, so it might take a while before a fix is made.

This is an issue with 8.3-rc-3.


docs: Update docs for 5.x changes.

These have been useful for me! Are they ready to merge into the 5.x branch?

@treshugart The result of merging this branch is not working because the branch 5.x-docs was still using propString but the 5.x was already on props.string.
So now in the 5.x branch you have type definitions for the wrong way.
If you're OK, I'll make a PR this week-end to fix the definitions temporarily while waiting for the full fix from @Hotell

@freeman yeah, that'd be great if it doesn't take too much time so that we can have a working 5.x branch. I have plans to autogenerate TS defs from Flow sources, which means quite possibly we'll move to using Flow and not have to maintain the TS defs. I want to wait a couple weeks to see where the generator is at because it would be nice to be able to make this change as part of 5.x.

Ok, thanks, expect my PR soon.
I have seen your branch about Flow.
Have you considered using typescript instead of Flow ? (Sorry if it has already been debated to death...)
I think that it enables easier contribution (flow is still kind of a pain with windows) and with the 2.3 release, we could migrate very progressively with the checkJs flag ( see https://blogs.msdn.microsoft.com/typescript/2017/04/27/announcing-typescript-2-3/ )

Yeah, I've seen that. As it stands, I'm going to be a little opinionated
about it because there is going to be a tool made available soon to output
both TS and Flow definitions from Flow source. Consumers > contributors,
IMO.
Once Babylon gets TS support via a plugin, using either will be much more
viable. But until then, I don't want to be locked into the TS featureset,
compiler and only able to export TS types.
I don't want this to spark any further discussion, so I'm just going to say
that, for now, Flow will probably be the direction and I'm happy to be
blamed for that as to curb any possible 🔥 wars that come from it :p
(Thanks for asking, though!)
On Sat., 29 Apr. 2017, 08:17 Michel Rasschaert<EMAIL_ADDRESS>wrote:

Ok, thanks, expect my PR soon.
I have seen your branch about Flow.
Have you considered using typescript instead of Flow ? (Sorry if it has
already been debated to death...)
I think that it enables easier contribution (flow is still kind of a pain
with windows) and with the 2.3 release, we could migrate very progressively
with the checkJs flag ( see
https://blogs.msdn.microsoft.com/typescript/2017/04/27/announcing-typescript-2-3/
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/skatejs/skatejs/pull/1101#issuecomment-298175084, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIVbEAIC4lcBaz_rV9zHZR8gU3Um2iCks5r01SDgaJpZM4Me1rN
.


Fix for RsaCtfTool

from lib import RSAAttack

ImportError: cannot import name 'RSAAttack' from 'lib' (/home/kali/Desktop/RsaCtfTool/lib/init.py)
just change the name in lib file jsut in case thats not a problem but i came a cross
but here is the problem i had
ISSUE:
eroor from:
from lib import RSAAttack
TypeError: 'module' object is not callable
FROM::
attackobj = RSAAttack(args)

just change the line 404
TO :::
attackobj = RSAAttack.RSAAttack(args)

Hi,
The line from lib import RSAAttack doesnt exist in the current code, it seems you are using either an old version of the code or a modified version.
Please try with the latest version.


Create version support and add redhat-4

Added support to allow version support in Launch and add a productized version to be supported.  Removed the redhat profile as support is being done with a separate "productized" repository.

Changing version labels and resubmitting PR


ArrayData recursion

It wouldn't exactly be non-breakable but is there any reason why the arrays below are not recursively converted to ArrayData? ThreeFour remains untouched in the below example.
$myArrayData = ArrayData::create([
   'One' => '1',
   'Two' => '2',
   'ThreeFour' => [
        'Three' => 3,
        'Four' => 4
    ]
]);

If you were to add ThreeFour as
'ThreeFour' => ArrayData::create([
    'Three' => 3,
    'Four' => 4
])

then everything is dandy, but then when you need to $myArrayData->toMap() the same lack of recursion is an issue and ThreeFour remains as an ArrayData object

Maybe a static method to convert and map recursively could be considered
https://github.com/vulcandigital/silverstripe-sendgrid/blob/master/src/SendGrid.php#L542-L557


Possibly Fixes #58

I haven't tested this, but it's a simple filter for the playtime which formats it to days, hours, minutes and seconds instead of displaying the seconds as minutes incorrectly.

Hey! I mean, I don't mind this, but isn't there some way to achieve this with momentjs?

I'll take a look if i can do it with momentjs, I agree that'd be a lot nicer


Minor fixes for #338

Minor fixes alongside the documentation PR #338 :
Includes one possibly controversial change: Make "App modules" not return the app classes so you can do:
  App = require("path").App

I did this because most apps did it this way and I wanted all apps to behave the same. It's arguable if this is right.

@lukego This should be merged alongside the documentation PR (already merged).

Looks like a valuable activity to me. I'm really happy to see some editing to gradually bring our varied and experimental coding styles closer together.
@alexandergall what do you think? you wrote most of the affected code.

It's certainly good to make the usage consistent across all apps. My only objection would be that a lot of the require code looks a bit awkward because most modules provide a single app that tends to have the same name as the module, e.g.
local baz = require("foo.bar.baz").baz
A nice one is
local RateLimiter = require("apps.rate_limiter.rate_limiter").RateLimiter
But I can live with that :)

Yeah. It would sure be nice to do away with those require lines entirely. Perhaps we could cook up a simple convention for giving the app name as a string and for config.lua to infer an app name.
One step at a time.. :-)

Just so it shows up on the record: The precedent is set by basic_apps, intel10g and pcap which are modules each providing multiple apps. So this is not only consistency but also the most common denominator.


Creating a class whose instances will be instances of Error

Hi,
I'm creating some custom error classes, but ran into an issue when using jasmine's expect(fn).toThrowError(MyCustomError);
The problem is that jasmine's toThrowError matcher checks that the exception thrown by fn is an instance of Error. This is clumsy to satisfy in TypeScript, as in lib.d.ts Error is an interface.
Thus, if one creates MyCustomError in the most straightforward and intuitive way supported:
export class MyCustomError implements Error {
  public name: string = 'MyCustomError';
  public message: string;
}

and throws an instance of this class, code such as that in jasmine that checks for an instance of Error will fail. And yet, their assumption seems quite legitimate.
Here's the code I ended up with:
export class MyCustomError implements Error {
  public name: string = 'MyCustomError';
  public message: string;
}

MyCustomError.prototype = Object.create(Error.prototype);

This will satisfy the instanceof Error constraint, but it's fairly unintuitive for what seems like a common task (creating a runtime subtype of Error) -- one that shouldn't require setting the prototype property manually.
Any thoughts?

Upgrade to TypeScript nightly build (npm install typescript@next), or wait until TypeScript 1.6, then write
class MyCustomError extends Error {
}

Oh cool. Out of interest, why was Error an interface in the first place and what motivated the change?

Prior to some other changes, it wasn't possible to augment an existing class the same way you can an interface (multiple class declarations were an error, rather than merging like interfaces do).
The change in 1.6 is that we allow arbitrary expressions in extends clauses instead of requiring the symbol to point to a class.

Looks like the original issue has been answered. please reactivate if you are running into other issues.


added in ability to enable/disable

I would like to have the ability to disable the unfriendly urls when testing.

Coverage decreased (-0.56%) to 99.44% when pulling dc8fdd29f9c488798ca95771364581217c2b622f on brechmos:develop into 63a43de633b1db37023bf4d6f59d472624936837 on tomatohater:develop.

Coverage decreased (-0.56%) to 99.44% when pulling dc8fdd29f9c488798ca95771364581217c2b622f on brechmos:develop into 63a43de633b1db37023bf4d6f59d472624936837 on tomatohater:develop.


Update FuelPHP.gitignore

This is a part of copy of the official gitignore of fuelphp ( https://github.com/fuel/fuel/blob/1.7/master/.gitignore ), but I commented out the core packages.
According to https://github.com/kechol/gitignore/blob/master/Composer.gitignore , composer.lock file is also  commented out.

@kechol apologies for the delay on this - thanks for the extra information!


Usgs greatcircle

I used great circle computation to remove the shapely and pyproj dependencies.
That was a breaker in a clients working environment

@candrsn thanks, it's nice to drop these dependencies.


fix eslint errors in storage-internal-avro pkg test files

addresses #10782 , specifically it fixes eslint errors in the storage-internal-avro subfolder test files

May consider adding the lint command to the pacakge.json
    "lint:fix": "eslint package.json api-extractor.json src test --ext .ts --fix",
    "lint": "eslint package.json api-extractor.json src test --ext .ts",

Hi @ljian3377 we already have those lint commands in the package.json file for this storage subpackage.
d

I see. Then maybe need to change the lint command as instructed in the originated issue:

Once all known issues are resolved, below change is required in package.json file in package root /sdk/storage/ to treat any new lint regression as hard failure in CI.
Remove following snippet from lint command in package.json
-f html -o -lintReport.html || exit 0

@ljian3377 We can follow up with updating the lint script in a separate PR.
Merging this now


Automatically Trim Backtraces

It would be nice if there was a way to automatically trim backtraces.  I generally have about 12 useless frames above what I actually care about (where the error originates).  It would be nice if failure could provide helpers to trim them down by chopping off stuff above known points (for instance when created via result_ext).

I was just about to make an issue for this as well.
Instead of trimming the backtrace itself, what about just adjusting the way it's printed? A good idea might be to emulate rustc, so when you set RUST_BACKTRACE = "1" it'll print a trimmed backtrace (removing the entry-point shim and the frames where we're starting to create a backtrace), but if you set RUST_BACKTRACE = "full" it'll give you the entire backtrace.
Considering the failure::Backtrace struct is an opaque type which is only useful for printing to the screen when the RUST_BACKTRACE variable is set,  it's probably not necessary to make changes to the underlying backtrace from libbacktrace (i.e. add a trim() method).


Implement efficient graph state simulation

Using https://journals.aps.org/pra/pdf/10.1103/PhysRevA.73.022334 (check ref 17-19 as well)

also useful https://arxiv.org/pdf/quant-ph/0504097.pdf

also https://journals.aps.org/pra/pdf/10.1103/PhysRevA.69.022316

In this recent paper on Fast simulation of planar Clifford circuits, for the software implementation, the authors utilized a GridCHP++. Given √n x √n cluster state, GridCHP++ simulates X/Y/Z measurements on all qubits.
P.S. The paper links [1, 3] actually seems to only work for those who have authorization to APS, meaning, these links don't tell the title of the paper. It just lands into APS website. I had to copy the links into search engine to check out these papers. So, providing the links of these papers again.
[1] Fast simulation of stabilizer circuits using a graph state representation
[3] Graphical description of the action of local Clifford transformations on graph states


Support multiple versions of the same package

obs-service-node_modules does not support multiple versions of the same package, it appears multiple versions overwrite each other and as often javascript projects have multiple versions of the same package through peer deps. This causes build issues and in the worst case could cause silent miscompliation issues

Can you provide an example?
What is assumed is that upstream appends version to the tarball which generally seems to happen. For example, some of the archives from cockpit package,
@babel-supports-color-5.5.0.tgz
balanced-match-1.0.2.tgz
binary-extensions-2.2.0.tgz
brace-expansion-1.1.11.tgz
braces-3.0.2.tgz
builtins-5.0.1.tgz
builtins-semver-7.5.4.tgz
call-bind-1.0.2.tgz
callsites-3.1.0.tgz
camelcase-5.3.1.tgz
camelcase-keys-6.2.2.tgz
chalk-4.1.2.tgz
chokidar-3.5.3.tgz
chrome-remote-interface-0.32.2.tgz
color-convert-2.0.1.tgz
colord-2.9.3.tgz
color-name-1.1.4.tgz
commander-2.11.0.tgz
concat-map-0.0.1.tgz
core-util-is-1.0.3.tgz
cosmiconfig-7.1.0.tgz
cross-spawn-7.0.3.tgz
cssesc-3.0.0.tgz
css-functions-list-3.2.0.tgz
cssnano-preset-lite-2.1.3.tgz

Hi!
I created a package on OBS to demonstrate the issue: https://build.opensuse.org/package/show/home:malikirri/cockpit-wicked-playground
If you run osc service manualrun you get this error:
ERROR:@testing-library/react/node_modules/aria-query: mismatch<EMAIL_ADDRESS><> https://registry.npmjs.org/aria-query/-/aria-query-4.2.2.tgz, sha512:1e7629004d58ea447228cfd7904ba24508531ef933301bab4c79e914b60b0463c8ca564d3ecf632
5cb8e3985b13cb242484b66653d18f2b1c3a2428deeb4538a <> sha512:a3f1de97086e2a94e3fdfaec3ac6cd2cd82734654816c54ffd25b6052175e20565ee401f30e27affcc14014bc6d51d4d1caadf1bedac1d94eed326ae4f5a81ac

So it's trying to match SHA sums of two different packages that share the .tgz filename.
And if you try to build it (like you can see from the OBS logs), it fails to find the correct aria-query module:
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>
Even though it's defined on the line 3706 in package-lock.json

I'll look into this.

Thanks!

I think related to old assumption that @scope was sufficient and there was no additional nesting.

Node packages are full of surprises :smile:

This problem should be now fixed. I'm now generating a more unique filename by incorporating parts of the URL path to form the filename stored in the cpio archive. Would be awesome if someone can verify there are no more issues here.

Looks like it's still not fully working. I updated the node module archive using node_modules.py from https://github.com/openSUSE/obs-service-node_modules/commit/1526eaa237032a4eefb7c540c02a9e40e99937aa and it's still not able to build correctly. It still gives the following error:
[   17s] npm ERR! code ETARGET
[   17s] npm ERR! notarget No matching version found for<EMAIL_ADDRESS>[   17s] npm ERR! notarget In most cases you or one of your dependencies are requesting
[   17s] npm ERR! notarget a package version that doesn't exist.

You can see the full log here: https://build.opensuse.org/package/live_build_log/home:malikirri/cockpit-wicked-playground/openSUSE_Tumbleweed/x86_64
The aria-query library is saved to the archive but looks like it's not able to find it there for some reason
$ cpio -t < node_modules.obscpio | grep aria-query
<EMAIL_ADDRESS>@testing-library-aria-query-4.2.2.tgz
<EMAIL_ADDRESS>aria-query-5.1.3.tgz
279239 blocks

The error is from npm.. Maybe there is some issue with the https://github.com/openSUSE/npm-localhost-proxy

looks like somehow @testing-library-aria-query-4.2.2.tgz is same as<EMAIL_ADDRESS>... re-opening

What happened was the old version of the code wrote the testing-library-aria-query-4.2.2.tgz from the @types/aria-query-4.2.2.tgz archive. The new version saw this file as originating from aria-query-4.2.2.tgz and never bother to replace it or check further information, like size, etc.. (algorithm only checks if the file exists in archive!). This is the offending section is here:
https://github.com/openSUSE/obs-service-node_modules/blob/master/node_modules.py#L454
I think this may merit a new issue for improvement.
Mystery solved!

I was able to build the test project successfully too. Thank you for figuring it out!


Removed PartialOrd and Ord of all enums in datatypes

Removed PartialOrd nor Ord, since there is no natural order on them.
Made IntervalUnit and TimeUnit Copy, since they are simple enums.

Hi @jorgecarleitao , arrow-datafusion added PartialOrd on Expr, which relays on DataType to be also PartialOrd:
#[derive(Clone, PartialEq, PartialOrd)]
pub enum Expr {
....
 Cast {
        /// The expression being cast
        expr: Box<Expr>,
        /// The `DataType` the expression will yield
        data_type: DataType,    // <- need DataType PartialOrd as well.
    },
}

Introduced in https://github.com/apache/arrow-datafusion/issues/1014 to provide the capability of sorting expressions, for common-subexpression-elimination and further expression transformations. any thoughts on this?

Ufff 😢
What is the semantics that the partial ord is trying to achieve in DataFusion?
I am a bit surprised that it is relying on the derived PartialOrd - semantically it seems to me that the order of the enum is meaningless. Is it to have a stable something?

Yes, I think so, for stabilizing tests (https://github.com/influxdata/influxdb_iox/pull/2564#discussion_r710289316)  and expression algebraic transformations, as the issue brought this in explained https://github.com/apache/arrow-datafusion/issues/1014

yeah, i don't think the ordering semantic matters here, just need a consistent ordering defined.

Pushed a workaround to implement partial ord using hash: https://github.com/houqp/arrow-datafusion/commit/99ee159fc8069c23829570126eadd63df0362678.

wow, quite neat idea! I had already given up and was dragging my feet to revert part of this PR and re-introduce partialOrd on

IntervalUnit
TimeUnit
Field
DataType

I am so glad that you found a way there!


Crashed on Heroku

After tuning on bot, I get this on logs
2021-10-15T16:00:03.377011+00:00 heroku[worker.1]: Starting process with command python3 -m udemy 2021-10-15T16:00:04.206658+00:00 heroku[worker.1]: State changed from starting to up 2021-10-15T16:00:04.928956+00:00 app[worker.1]: Traceback (most recent call last): 2021-10-15T16:00:04.928973+00:00 app[worker.1]:   File "/app/.heroku/python/lib/python3.9/runpy.py", line 197, in _run_module_as_main 2021-10-15T16:00:04.929162+00:00 app[worker.1]:     return _run_code(code, main_globals, None, 2021-10-15T16:00:04.929177+00:00 app[worker.1]:   File "/app/.heroku/python/lib/python3.9/runpy.py", line 87, in _run_code 2021-10-15T16:00:04.929312+00:00 app[worker.1]:     exec(code, run_globals) 2021-10-15T16:00:04.929325+00:00 app[worker.1]:   File "/app/udemy/__main__.py", line 11, in <module> 2021-10-15T16:00:04.929426+00:00 app[worker.1]:     bot = UdemyBOT() 2021-10-15T16:00:04.929439+00:00 app[worker.1]:   File "/app/udemy/bot/bot.py", line 13, in __init__ 2021-10-15T16:00:04.929533+00:00 app[worker.1]:     super().__init__("udemybot", api_id=api_id, api_hash=api_hash) 2021-10-15T16:00:04.929546+00:00 app[worker.1]:   File "/app/.heroku/python/lib/python3.9/site-packages/telethon/client/telegrambaseclient.py", line 299, in __init__ 2021-10-15T16:00:04.929726+00:00 app[worker.1]:     self.api_id = int(api_id) 2021-10-15T16:00:04.929775+00:00 app[worker.1]: ValueError: invalid literal for int() with base 10: 'c51ee30b8d9190bf1cdb0bea99239d47' 2021-10-15T16:00:05.153070+00:00 heroku[worker.1]: Process exited with status 1 2021-10-15T16:00:05.377980+00:00 heroku[worker.1]: State changed from up to crashed

Because you have swapped the values of id and hash. 🙂


Review group-07

the group which have be assigned to be reviewed by our group is group-07.

Done.

Done!


[Monaco] Describe grammar for syntax highlighting

Implement syntax highlighting for keywords, brackets and symbols.
Monaco uses Monarch format for grammar definition.
https://microsoft.github.io/monaco-editor/monarch.html
might want to refer to the native tokenizer for flux: https://github.com/influxdata/flux/blob/master/internal/scanner/scanner.rl
VSCode uses textmate format. monarch vs textmate grammar sadness:
https://news.ycombinator.com/item?id=14259463
https://stackoverflow.com/questions/37936919/writing-visual-studio-code-syntaxes-in-monarch

I came across https://github.com/NeekSandhu/monaco-textmate, not sure if this helps us at all.

Tools team is maintaining a textmate compatible syntax highlighter and we should use that.

cc @kelwang ^^

@desa @kelwang Is the syntax highlighter specific to Flux? Is it public?

@sanderson Monaco has support for multiple formats that are prebuilt into the library. The tools team has been working on a syntax highligher for vscode using the textmate format (which has become the standard format for text editors.

https://github.com/influxdata/vsflux/blob/master/syntaxes/flux.tmLanguage.json


Email body is empty in Outlook App on Android

When using this xam.messaging plugin to send email in Android and choosing Outlook app from the App Chooser menu, the email body is empty, while the subject and to fields get set correctly as expected. Please note that this only happened with Outlook App and the plugin works as expected with Yahoo and Google Inbox apps. This is an issue while using BodyAsHtml on EmailMessageBuilder.

Unfortunately there is no way for the plugin to support app specific behaviour for different e-mail clients.  This seems to be an issue with the Outlook App's support for sending HTML e-mail and not something that the plugin can control.


Implicit returns of (last) declaration

In the spirit of "everything is an expression", and e.g. if x := 5
BREAKING CHANGE: x := 5 now implicitly returns x
More complex example:
->
  let x = 5, y = x
↓↓↓
function() {
  let x = 5, y = x;return y
}

Oren motivates it as giving a name to the return value, which helps define a semantic meaning.  It can't really have much other purpose.
I'm pretty sure I remember @STRd6 saying he preferred this behavior. Could be worth a minor version bump though given that it would break some code that uses declarations to prevent return (but this seems like bad practice).
Also fix a bug in names computation for var declarations. If we don't merge this PR, we should at least fix that.

One potential issue: [x, y] := obj causes the return of y. Should it instead return [x, y]?

I spent several (painful) hours getting this to work. Along the way, refactored the AST a lot in hopefully useful ways. BindingProperty in particular now gains a delim property for the delimiter, which makes some manipulation easier.
Took me a while to realize that BindingElement and BindingElementList had very specific tuple formats, but this is now documented. I think it's also a decent approach, because RegularExpressionLiteral needs a wrapper of some sort for the delimiter. (Previously I tried to just put the delim property directly on the BindingElement, similar to BindingProperty, but this made delimiter removal harder and finally broke with RegularExpressionLiteral.)


To avoid /articles/:doi/meta surfacing 404s, redirect to article pages

We verified 404s were turned into 307s.


Makes resolved field optional

Overview
Per yarn's implementation "resolved" field of yarn.lock file is optional and can be missing.
This PR, addresses exception thrown when resolved field is missing, and package is neither local or symbolic directory.
This is prerequisite to fix: https://github.com/fossas/team-analysis/issues/845
Testing

Create yarn.lock file for simple node project
Remove resolved field from one of the entry
Replace following method in src/Strategy/Node/YarnV1/YarnLock.hs:

getLocations :: Maybe YL.Remote -> [Text]
getLocations = \case
  Just (YL.FileRemote url _) -> [url]
  Just (YL.FileRemoteNoIntegrity url) -> [url]
  Just (YL.GitRemote url rev) -> [url <> "@" <> rev]
  Just (YL.DirectoryLocal dirpath) -> [dirpath]
  Just (YL.DirectoryLocalSymLinked dirpath) -> [dirpath]
  _ -> []

Update yarnlock source package:

-- Current implementation of yarn-lock throws runtime error, as described by patches:
-- (a) https://github.com/Profpatsch/yarn2nix/pull/73
-- (b) https://github.com/Profpatsch/yarn2nix/pull/72
-- TODO: Remove once, both PRs are merged.
source-repository-package
  type: git
  location: https://github.com/fossas/yarn2nix
  tag: 276fb494a22bbc50634a8ff8911d6b53782ab1ed -- <- commit of this PR
  subdir: yarn-lock

Rebuild fossa-cli by: make install-dev
Analyze this project using fossa-dev analyze -o | jq (it should complete without any errors)

@spatten GH bot chose you, but feel free to assign this to Connor, Wes, or Zach. If you are not comfortable with haskell bits on this PR.


node-gyp rebuild fails

Error while installing robotjs
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1074:12)
gyp ERR! System Linux 3.2.0-86-generic
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/itsashis4u/projects/robot/node_modules/robotjs
gyp ERR! node -v v0.12.7
gyp ERR! node-gyp -v v2.0.1
gyp ERR! not ok
npm ERR! Linux 3.2.0-86-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "robotjs@latest" "--save"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the robotjs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls robotjs
npm ERR! There is likely additional logging output above.

Another rebuild error:
> $ npm install robotjs
-
><EMAIL_ADDRESS>install /home/mod/work/nodejs/robotjs/node_modules/robotjs
> node-gyp rebuild

child_process: customFds option is deprecated, use stdio instead.
make: Entering directory `/home/mod/work/nodejs/robotjs/node_modules/robotjs/build'
  CXX(target) Release/obj.target/robotjs/src/robotjs.o
cc1plus: warning: command line option ‘-Wbad-function-cast’ is valid for C/ObjC but not for C++ [enabled by default]
In file included from ../src/keypress.h:6:0,
                 from ../src/robotjs.cc:7:
../src/keycode.h:55:23: fatal error: X11/Xutil.h: No such file or directory
 #include <X11/Xutil.h>
                       ^
compilation terminated.
make: *** [Release/obj.target/robotjs/src/robotjs.o] Error 1
make: Leaving directory `/home/mod/work/nodejs/robotjs/node_modules/robotjs/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:267:23)
gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1067:12)
gyp ERR! System Linux 3.16.0-41-generic
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/mod/work/nodejs/robotjs/node_modules/robotjs
gyp ERR! node -v v0.12.0
gyp ERR! node-gyp -v v1.0.2
gyp ERR! not ok
npm ERR! Linux 3.16.0-41-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "robotjs"
npm ERR! node v0.12.0
npm ERR! npm  v2.5.1
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the robotjs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls robotjs
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/mod/work/nodejs/robotjs/npm-debug.log

drobati@ATL-N-DEREKR ~
$ npm install robotjs

<EMAIL_ADDRESS>install C:\Users\drobati\node_modules\robotjs
node-gyp rebuild

C:\Users\drobati\node_modules\robotjs>if not defined npm_config_node_gyp (node "c:\Program Files (x86)\nodejs\node_modules\npm\bin\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node  rebuild )
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack Error: connect ETIMEDOUT
gyp ERR! stack     at exports._errnoException (util.js:746:11)
gyp ERR! stack     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1010:19)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "node" "c:\Program Files (x86)\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\drobati\node_modules\robotjs
gyp ERR! node -v v0.12.7
gyp ERR! node-gyp -v v2.0.1
gyp ERR! not ok
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "c:\Program Files (x86)\nodejs\node.exe" "c:\Program Files (x86)\nodejs\node_modules\npm\bin\npm-cli.js" "install" "robotjs"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the robotjs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls robotjs
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\drobati\npm-debug.log

Sorry guys! For the first two Linux needs a few packages:
sudo apt-get install libxtst-dev libpng-dev
I need to get the README updated.
For the Windows error I'm not completely sure.

@octalmage I ran the command again. (The first time looks like some connection timeout.)
drobati@ATL-N-DEREKR ~
$ npm install robotjs
|
><EMAIL_ADDRESS>install C:\Users\drobati\node_modules\robotjs
> node-gyp rebuild

C:\Users\drobati\node_modules\robotjs>if not defined npm_config_node_gyp (node "
c:\Program Files (x86)\nodejs\node_modules\npm\bin\node-gyp-bin\\..\..\node_modu
les\node-gyp\bin\node-gyp.js" rebuild )  else (node  rebuild )
Building the projects in this solution one at a time. To enable parallel build,
please add the "/m" switch.
MSBUILD : error MSB3428: Could not load the Visual C++ component "VCBuild.exe".
 To fix this, 1) install the .NET Framework 2.0 SDK, 2) install Microsoft Visua
l Studio 2005 or 3) add the location of the component to the system path if it
is installed elsewhere.  [C:\Users\drobati\node_modules\robotjs\build\binding.s
ln]
gyp ERR! build error
gyp ERR! stack Error: `C:\Windows\Microsoft.NET\Framework\v4.0.30319\msbuild.exe
` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (c:\Program Files (x86)\nodejs\node_mo
dules\npm\node_modules\node-gyp\lib\build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1074
:12)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "node" "c:\\Program Files (x86)\\nodejs\\node_modules\\npm\\nod
e_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\drobati\node_modules\robotjs
gyp ERR! node -v v0.12.7
gyp ERR! node-gyp -v v2.0.1
gyp ERR! not ok
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "c:\\Program Files (x86)\\nodejs\\node.exe" "c:\\Program Files (x8
6)\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "install" "robotjs"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the robotjs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls robotjs
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\drobati\npm-debug.log

I got this. Looks like windows has some dependencies?

Looks like on Windows you need:

Python (v2.7.3 recommended, v3.x.x is not supported.
Visual Studio 2013 (Express works fine!).

@octalmage Thanks. It was resolved on linux by installing the mentioned packages. :+1:

@octalmage I'll download VS13 Express. Already have Python 2.7.3.

I'm going to close this, if building fails after installing the requirements please let me know! I've updated the readme to link to the node-gyp readme:
https://github.com/TooTallNate/node-gyp/#installation

Sorry to open this back up, but here's my rebuild error. I'm assuming this has to do with the newer version of XCode? I'm on version 7.1 currently and have Command Line Tools installed. I've tried a couple different node versions. v0.12.7 works fine, but I have other dependencies that need v4.x.x. The error below is running with v4.0.0.
$ npm install robotjs
-
><EMAIL_ADDRESS>install /Users/gregholguin/Documents/Projects/Personal/twitch-plays-undertale/node_modules/robotjs
> node-gyp rebuild

  CXX(target) Release/obj.target/robotjs/src/robotjs.o
../src/robotjs.cc:76:16: error: cannot initialize a variable of type
      'MMMouseButton' (aka 'CGMouseButton') with an rvalue of type
      '_MMMouseButton'
        MMMouseButton button = LEFT_BUTTON;
                      ^        ~~~~~~~~~~~
../src/robotjs.cc:129:16: error: cannot initialize a variable of type
      'MMMouseButton' (aka 'CGMouseButton') with an rvalue of type
      '_MMMouseButton'
        MMMouseButton button = LEFT_BUTTON;
                      ^        ~~~~~~~~~~~
../src/robotjs.cc:397:13: error: assigning to 'MMKeyFlags' (aka 'CGEventFlags')
      from incompatible type '_MMKeyFlags'
        *flags = MOD_ALT;
               ^ ~~~~~~~
../src/robotjs.cc:401:13: error: assigning to 'MMKeyFlags' (aka 'CGEventFlags')
      from incompatible type '_MMKeyFlags'
        *flags = MOD_META;
               ^ ~~~~~~~~
../src/robotjs.cc:405:13: error: assigning to 'MMKeyFlags' (aka 'CGEventFlags')
      from incompatible type '_MMKeyFlags'
        *flags = MOD_CONTROL;
               ^ ~~~~~~~~~~~
../src/robotjs.cc:409:13: error: assigning to 'MMKeyFlags' (aka 'CGEventFlags')
      from incompatible type '_MMKeyFlags'
        *flags = MOD_SHIFT;
               ^ ~~~~~~~~~
../src/robotjs.cc:413:13: error: assigning to 'MMKeyFlags' (aka 'CGEventFlags')
      from incompatible type '_MMKeyFlags'
        *flags = MOD_NONE;
               ^ ~~~~~~~~
../src/robotjs.cc:425:13: error: cannot initialize a variable of type
      'MMKeyFlags' (aka 'CGEventFlags') with an rvalue of type '_MMKeyFlags'
        MMKeyFlags flags = MOD_NONE;
                   ^       ~~~~~~~~
../src/robotjs.cc:479:13: error: cannot initialize a variable of type
      'MMKeyFlags' (aka 'CGEventFlags') with an rvalue of type '_MMKeyFlags'
        MMKeyFlags flags = MOD_NONE;
                   ^       ~~~~~~~~
9 errors generated.
make: *** [Release/obj.target/robotjs/src/robotjs.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/gregholguin/.nvm/versions/node/v4.0.0/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:270:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Darwin 15.0.0
gyp ERR! command "/Users/gregholguin/.nvm/versions/node/v4.0.0/bin/node" "/Users/gregholguin/.nvm/versions/node/v4.0.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/gregholguin/Documents/Projects/Personal/twitch-plays-undertale/node_modules/robotjs
gyp ERR! node -v v4.0.0
gyp ERR! node-gyp -v v3.0.1
gyp ERR! not ok
npm ERR! Darwin 15.0.0
npm ERR! argv "/Users/gregholguin/.nvm/versions/node/v4.0.0/bin/node" "/Users/gregholguin/.nvm/versions/node/v4.0.0/bin/npm" "install" "robotjs"
npm ERR! node v4.0.0
npm ERR! npm  v2.14.2
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the robotjs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls robotjs
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/gregholguin/Documents/Projects/Personal/twitch-plays-undertale/npm-debug.log

See #116, this is a new issue I'm looking into!


⚠ Bofrost.at has degraded performance

In 0ba0831, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 835 ms

Resolved: Bofrost.at performance has improved in 6463dc1 after 22 minutes.