parquet metadata only queries perform full scans

Describe the bug
queries such as read_parquet().count_rows() should not do a full scan, and instead be should able to be fulfilled by the metadata only.
The need for the full scan should be optimized away during physical planning.
To Reproduce
Steps to reproduce the behavior:
daft.read_parquet().count_rows() **Expected behavior** Metadata only operations such as count(*)orcount_rows()` usually can be fulfilled without needing to perform a full scan.
Additional context
dump from daft.read_parquet('lineitem.parquet').explain(show_all=True)
== Unoptimized Logical Plan ==

* GlobScanOperator
|   Glob paths = [../Daft/lineitem.parquet]
|   Coerce int96 timestamp unit = Nanoseconds
|   IO config = S3 config = { Max connections = 8, Retry initial backoff ms = 1000, Connect timeout ms = 30000, Read timeout ms = 30000, Max retries = 25, Retry mode = adaptive, Anonymous = false, Use SSL = true, Verify SSL = true, Check hostname SSL = true, Requester pays = false, Force Virtual Addressing = false }, Azure config = { Anoynmous = false, Use SSL = true }, GCS config = { Anoynmous = false }
|   Use multithreading = true
|   File schema = l_orderkey#Int64, l_partkey#Int64, l_suppkey#Int64, l_linenumber#Int64, l_quantity#Int64, l_extendedprice#Float64, l_discount#Float64, l_tax#Float64, l_returnflag#Utf8, l_linestatus#Utf8, l_shipdate#Date, l_commitdate#Date, l_receiptdate#Date, l_shipinstruct#Utf8, l_shipmode#Utf8, comments#Utf8
|   Partitioning keys = []
|   Output schema = l_orderkey#Int64, l_partkey#Int64, l_suppkey#Int64, l_linenumber#Int64, l_quantity#Int64, l_extendedprice#Float64, l_discount#Float64, l_tax#Float64, l_returnflag#Utf8, l_linestatus#Utf8, l_shipdate#Date, l_commitdate#Date, l_receiptdate#Date, l_shipinstruct#Utf8, l_shipmode#Utf8, comments#Utf8

== Optimized Logical Plan ==

* GlobScanOperator
|   Glob paths = [../Daft/lineitem.parquet]
|   Coerce int96 timestamp unit = Nanoseconds
|   IO config = S3 config = { Max connections = 8, Retry initial backoff ms = 1000, Connect timeout ms = 30000, Read timeout ms = 30000, Max retries = 25, Retry mode = adaptive, Anonymous = false, Use SSL = true, Verify SSL = true, Check hostname SSL = true, Requester pays = false, Force Virtual Addressing = false }, Azure config = { Anoynmous = false, Use SSL = true }, GCS config = { Anoynmous = false }
|   Use multithreading = true
|   File schema = l_orderkey#Int64, l_partkey#Int64, l_suppkey#Int64, l_linenumber#Int64, l_quantity#Int64, l_extendedprice#Float64, l_discount#Float64, l_tax#Float64, l_returnflag#Utf8, l_linestatus#Utf8, l_shipdate#Date, l_commitdate#Date, l_receiptdate#Date, l_shipinstruct#Utf8, l_shipmode#Utf8, comments#Utf8
|   Partitioning keys = []
|   Output schema = l_orderkey#Int64, l_partkey#Int64, l_suppkey#Int64, l_linenumber#Int64, l_quantity#Int64, l_extendedprice#Float64, l_discount#Float64, l_tax#Float64, l_returnflag#Utf8, l_linestatus#Utf8, l_shipdate#Date, l_commitdate#Date, l_receiptdate#Date, l_shipinstruct#Utf8, l_shipmode#Utf8, comments#Utf8

== Physical Plan ==

* TabularScan:
|   Num Scan Tasks = 21
|   Estimated Scan Bytes =<PHONE_NUMBER>
|   Clustering spec = { Num partitions = 21 }

Indeed. We need a mechanism for Daft to represent tables with "no columns", but still have valid metadata.
Currently the scan will naively read the first column I believe.


promptToUpdatePowerShell not work

VS Code Version: 1.56.1

OS Version: 19042.928

Does this issue occur when all extensions are disabled?: Yes/No

Hi @HppZ! Couple of things...
First off, that setting and feature is provided by the PowerShell extension, which can be found here:
https://github.com/PowerShell/vscode-powershell
Additionally, that setting only controls notification that pops up in vscode. The notification your seeing in the console is from PowerShell itself.
That can be disabled by putting this in your PowerShell profile:
$Env:POWERSHELL_UPDATECHECK = 'Off'
If you want the extension to do more here, please open an issue on vscode-powershell.
Cheers!

this NoLogo config disables that update tip in Windows Terminal, but not in VSCode.

@HppZ does your terminal tab/dropdown say "pwsh" or "PowerShell Integrated Console"?

Sorry I meant in vscode @HppZ

did you mean that above?

Yeah ok that's what I wanted to see. @meganrogge args might not be being honored.

Works for me on Insider's

You're probably seeing https://github.com/microsoft/vscode/issues/123188

Works for me on Insider's

work for you but not for me.

@HppZ did you try it using the insiders build? https://code.visualstudio.com/insiders

no insider build installed.

why closed?


bionic builder description should include .NET

The order for the cloudfoundry/cnb:bionic builder includes dotnet-core buildpacks, but the list of languages in the description does not include .NET.
pack inspect-builder cloudfoundry/cnb:bionic
Inspecting builder: cloudfoundry/cnb:bionic

REMOTE:

Description: Ubuntu bionic base image with buildpacks for Java, NodeJS and Golang

Full: https://github.com/paketo-buildpacks/full-builder/pull/88
Base: https://github.com/paketo-buildpacks/base-builder/pull/44
Tiny: https://github.com/paketo-buildpacks/tiny-builder/pull/29

Closing since each of the above linked pull requests have been merged.


bookmarksProperties2.xul isn't styled

The popup window that opens if you have multiple tabs opend and use "Bookmark all Tabs..." from right-click on a tab is currently not styled.

System:
Windows 7 Pro N 64bit
Firefox Portbale 59.0.2

Thanks for letting me know!
I've added a quick fix to make it look more similar to the bookmarks popup (from address bar) and not quite so ugly.
I want to keep this open for now though because I don't love the look and the bookmarks code could use some work.


Unable to select latest 0.16.8 SDK

The 'install SDK' button allows me to choose from versions 0.15.1, 0.16.4, 0.16.5-1, and 0.16.6, however the latest version 0.16.8 is not present in the list and typing it in does not work.

Thanks for the issue. I will try to issue an update this week

Hi could you try the latest prerelease version of the app and see if the issue is resolved. >V1.0.37. There are other gui related changes on the prerelease version as well.


Load article share component in article component

And also load share component for new articles loaded with AJAX
Create action_sheet.js and update index.js

Few comments, otherwise :+1:


Return feature importances as visible

Feature importances are returned as invisible. I don't get why would anyone want them invisible. This PR fixes it.

@david-cortes please correct me if I've misunderstood the effect or intention of this PR.

Thing is, even you assign the result to an object, it will not print it the first time you try to do so, because it is invisible. I'm attaching a video.

BTW, the overhead is measured in microseconds:
library(data.table)
library(microbenchmark)
set.seed(1)
X <- as.data.table(rnorm(1e4*1e3), nrow=1e4)
make_inv <- function(x) invisible(x)
make_vis <- function(x) withVisible(x)$value
Xinv <- make_inv(X)
microbenchmark({Xvis <- make_vis(Xinv)})

Unit: microseconds
                           expr   min    lq     mean median     uq      max neval
 {     Xvis <- make_vis(Xinv) } 1.473 1.503 16.09723 1.5075 1.5325 1444.489   100

Thanks for that investigation!
I just tested and I don't think this PR will actually provide the behavior you're looking for. When you use := expressions in a data.table, it will not be printed the first time it's returned.
Try running the code below:
f <- function(){
    someDT <- data.table::data.table(
        x = rnorm(1e4)
        , y = rnorm(1e4)
    )
    someDT[, z := rnorm(.N)]
    return(someDT)
}

f2 <- function(){
    someDT <- data.table::data.table(
        x = rnorm(1e4)
        , y = rnorm(1e4)
    )
    someDT[, z := rnorm(.N)]
    return(withVisible(someDT)$value)
}

f3 <- function(){
    someDT <- data.table::data.table(
        x = rnorm(1e4)
        , y = rnorm(1e4)
    )
    someDT[, z := rnorm(.N)]
    return(someDT[])
}

# does not print
f()

# does not print
f2()

# prints
f3()

I'm ok with adding a workaround to ensure printing of the result for this function. Since that result is a data.table we can trust that it will only print a handful of rows no matter how big it is, and the number of columns will always be the same 4 (Gain, Cover, Frequency, Feature).
I'll leave a suggestion on how to achieve the behavior you're looking for.


Cannot mass-copy messages from the Problems window

Issue Type: Bug
In the Problems pane, right-clicking provides options to Copy or Copy Message, but Select All is not available.
Pressing CTRL+A appears to work - everything gets highlighted - but pressing CTRL+C only copies a single entry.
It would be nice if, as with search results, one could treat the contents of the Problems Window as data, so one could, for example, generate a report for their team of all the warnings they have to fix, etc.
VS Code version: Code 1.50.1 (d2e414d9e4239a252d1ab117bd7067f125afd80a, 2020-10-13T15:06:15.712Z)
OS version: Windows_NT x64 10.0.19041

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (4 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.68GB (7.23GB free)

Process Argv
--crash-reporter-id 5021c1ca-a526-46e0-ac29-364923fa1827

Screen Reader
no

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

vscode-markdownlint
Dav
0.37.1

xml
Dot
2.5.1

vscode-html-css
ecm
0.2.3

scad
efb
1.0.1

vscode-edit-csv
jan
0.4.2

jenkins-pipeline-linter-connector
jan
1.2.0

zenuml
MrC
0.5.4

vscode-docker
ms-
1.7.0

csharp
ms-
1.23.4

python
ms-
2020.9.114305

remote-containers
ms-
0.145.1

remote-ssh
ms-
0.55.0

remote-ssh-edit
ms-
0.55.0

remote-wsl
ms-
0.50.1

cpptools
ms-
1.0.1

powershell
ms-
2020.6.0

vscode-groovy-lint
Nic
1.2.9

rewrap
stk
1.13.0

markdown-pdf
yza
1.4.4

markdown-all-in-one
yzh
3.3.0

/duplicate
https://github.com/microsoft/vscode/issues/105657


Verify that specs are adaquate for iiif_print SS QA column

Summary
Verify that there are specs before closing out the iiif_print tickets for SoftServe  QA.  These tickets are not quite QA-able and their behaviors will be covered by a test plan.
Acceptance Criteria
Does each PR have a good spec? ref: https://github.com/orgs/scientist-softserv/projects/43/views/1?filterQuery=label%3Aiiif_print+status%3A"SoftServ+QA"
If so

[ ] move ticket to done

if not

[ ] add specs and leave a comment noting the ticket number that got updated.

Screenshots or Video
Testing Instructions
Notes
Confused how to fill this out? Check out the playbook

Kirk completed this ticket and closed the relevant tickets.


adding psql to client vm

scenario: https://azurearcjumpstart.io/azure_arc_jumpstart/azure_arc_data/aks/aks_postgresql_hyperscale_arm_template/
would it be possible to add psql client on client vm? this is a very basic tool commonly used for postgres.
it is nice to have azure data studio but psql would be really appreciated.
the installer from EDB has got the quiet mode to be used in the automated installations

Yes. We will include this in our May Jumpstart release. /cc @dkirby-ms


Added basic iOS and wOS example.

Added a view controller where user can set/retrieve a user account and password.
Added a watch interface controller where the user can retrieve a password for a hard coded account name, or set a hard coded password for that account.
TODO: Add watch connectivity to sync keychain between watch/iphone examples.

Thanks very much! :)


Description and Topics

@callum-ramage
There's a typo in the description. You should probably also add topics.
You could just paste this line in:

An Atom package designed to make editing PoE item filters easier.

With the following topic tags:

path-of-exile
atom-package
atom-editor
atom-plugin

Thanks, fixed


SPM error: NoSources

I tried to build your Package with the newest version of the Swift Package Manager. I failed with: NoSources(".../Packages/PathKit-0.6.1/Tests/Fixtures").

Given the documentation for SPM says the following:

Directories named Tests are ignored
Sub directories of a directory named Sources, Source, srcs or src become modules

I'd say this is an SPM bug and not a problem with PathKit.

The newest Version of SPM has support for testing, so the folder isn't ignored. The documentation is out dated.

I've got same issue. Testing feature comes with latest DEVELOPMENT-SNAPSHOT-2016-02-25-a release, so would be great to get released fix soon.

Running into the same problem with the 2-25 snapshot. Temporary workaround seems to be deleting the Tests directory entirely.

Yeah, can't solve this at the moment. There is a bug in the swift package manager's exclude [https://twitter.com/mxcl/status/703073165254070275].

I think https://github.com/apple/swift-package-manager/commit/02cf82d947f4d6ff506da61fcbaecc01d06fc4a2 will ultimately solve this problem. Tests shouldn't even be built when using PathKit as a dependency.

This can stay open, will need to add the Fixtures directory to excludes once https://github.com/apple/swift-package-manager/pull/152 is merged and released.


[BUG] Wrong duration rounding

I think there is 5 years between 2015-01-01 and 2020-01-01. But momentjs don't think so.
start = moment('2015-01-01');
end = moment('2020-01-01');
duration = moment.duration(end.diff(start));
duration.years(); // 4
duration.months(); // 11

BUT
start = moment('2015-01-01');
end = moment('2020-01-01').add(1, 'day');
duration = moment.duration(end.diff(start));
duration.years(); // 5
duration.months(); // 0

AND
start = moment('2015-01-01');
end = moment('2020-01-01').add(23, 'hours');
duration = moment.duration(end.diff(start));
duration.years(); // 4
duration.months(); // 11

start = moment('2015-01-01');
end = moment('2020-03-01').date(2)
duration = moment.duration(end.diff(start));
duration.months(); // 1

all snippets could be reproduced from browser console on http://momentjs.com/docs/#/get-set/
Environment:
node v8.4.0 mac os
Other information that may be helpful:

GMT+6
Thu Sep 07 2017 21:59:49 GMT+0600 (+06)
plain momentjs library

Diffing to MS and then converting to a duration is lossy because it loses track which years they are and years aren't all the same length. It's therefore not at all surprising than when you rolled those milliseconds back up to years, you didn't get the same thing you'd get if you just diffed to years in the first place. However, through some luck you had just the right number of leap days that the diff ended up with the same number MS as the Moment's year->ms conversion would give you anyway.
So there's still an issue. Put more simply:
ms = moment.duration(5, 'years').asMilliseconds()
moment.duration(ms).asYears() //=> 4.999418194761015

Similar to #3209. I've never really understood these asymmetries, but @ichernev has a better grasp of this stuff than I do.

You simply can't mix year/month arithmetic with day/hour/minute/second arithmetic. Under the hood, Moment treats them separately for good reason.
Every year has 12 months. 12 months always make 1 year. But every month could have 28, 29, 30, or 31 days. And every year could have 365 or 366 days. And that bleeds on down into hours and the lesser units.
When Moment has to convert between months and days, it uses these functions:
function daysToMonths (days) {
    // 400 years have 146097 days (taking into account leap year rules)
    // 400 years have 12 months === 4800
    return days * 4800 / 146097;
}

function monthsToDays (months) {
    // the reverse of daysToMonths
    return months * 146097 / 4800;
}

When converting down from months to days, the output of that function is always forced to an integer, because however many days a month has, it  always has a whole number of days. (see https://github.com/moment/moment/pull/1925)
When converting up from days to months, the average is used. Over 400 years, each year averages 365.2425 days. I think the problem stems from the fact that no single year is actually ever 365.2425 days long. However many days a year has, it always has a whole number of days.
I wonder if we could use a modified floored or rounded average here. In this particular case:
1826 / 365.2425 = 4.999418194761015

but
1826 / (Math.floor(365.2425 * 5) / 5) = 5

I'll try to come up with a list of test cases that give unexpected results with the current conversion, and see if modifying the daysToMonths conversion in this way will fix them.


Stricter type definition of Subject

Description:
Make type definition of Subject more strict to avoid common bugs when using Subject.
Related issue (if exists):
This issue been discussed extensively in #2852 and #5066 .
Sensitive points
As discussed in the issues linked above, it is possible to omit the argument to next when the Subject is of type Subject<void>.
When the template type is anything other than void, then an argument for next is required.
The primary case where this can be a problem is when people instantiate Subject<any> and then use subject.next() with no arguments. This will break.
The present change makes the default template type of Subject void instead of any. This means that when you write new Subject(), you get a Subject<void> (instead of Subject<any>) and are therefore able to write subject.next() or subject.next(value) if value is either any or undefined.
Other good candidates for default template type are unknown and any. We can discuss whether void is the best choice here, I am not completely sure yet.

@cartant I added tests and some documentation to explain the breaking change and the motivation for choosing void as a default type.

The only issue I see with this is that it will break usages like subject.next(), users will be forced to pass in undefined, which isn't very ergonomic. I'm not sure I want to merge this one. We should ping the TS team on what the best solution is here.


Type.GetConstructor does not work if the constructor is set to internal

internal class Test
{
    internal Test()
    {
    }
}

// This fails
typeof(Test).GetConstructor(new Type[0]);

And yes, I know, setting constructor as public in this case makes no difference to its access level, I just wanted to report an error I found.

It is not bug, .NET gives the same result
https://dotnetfiddle.net/z2TP0m

It is not bug, .NET gives the same result
https://dotnetfiddle.net/z2TP0m

Interesting, did not realize that, change my request to allowing to pass BindingFlags when fetching the constructor than: https://msdn.microsoft.com/en-us/library/0h6w8akb(v=vs.110).aspx

I believe the issue here is clarified, right? Let us know if otherwise.

As stated above, change my request to allow to pass BindingFlags when fetching the constructor than: https://msdn.microsoft.com/en-us/library/0h6w8akb(v=vs.110).aspx


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for docbracing processing.

Name
Link

🔨 Latest commit
2bcc3d0df4afb5771b220797f47536773d17dd5c

🔍 Latest deploy log
https://app.netlify.com/sites/docbracing/deploys/650ec7a522759f0008438c56


Can't setState inside hover

hover (props, monitor, component) {
  const item = monitor.getItem();
  const [left, top] = getAndCalculateDropPosition(monitor);

  component.setState({
    hoverOn: {
      left: left,
      top: top
    }
  });
}

I want to use setState within hover but that leads to unexpected behavior, what can I do?

The sortable example is using setState inside hover, http://gaearon.github.io/react-dnd/examples-sortable-simple.html
Reproduce:
Drag an item out of drop target, see the can't drop icon.
Drag item to another spot to sort, drag item out of drop target, can't drop icon is not shown. it is confused.

That example doesn't call setState inside hover, it calls a method on props. Also there is no cancel functionality in that example which is why the can't drop icon is not shown.
I had to do something similar to what you want to do, i ended up passing in an event emitter via props. I just used a jquery object $({}). Then i trigger events on it in the hover method, and hook up a listener in the constructor of my component, which sets the state based on the info it receives.

that leads to unexpected behavior, what can I do?

Can you describe it in more detail, or better, have a reproducible example?

@mvestergaard the method on props calls setState that further causes a rerender on droptarget so it counts.
@gaearon I can reproduce it in the example: http://gaearon.github.io/react-dnd/examples-sortable-simple.html.

Drag an item to the right, see can't drop icon, that's normal behaviour.
Drag an item over the list, see it sorting, drag item to the right, the can't drop icon is not visible. that's unexpected.

The problem with the icon is real because it confuses if I can drop the item or not. I get unexpected drag drop experience because of this issue.
Note that I don't have this issue if I remove setState from within hover.
I noticed this problem while writing canDrop method. It was always returning true but still I was getting can't drop icon.
And it's not just the icon I am getting unexpected behaviour whether the item is dropped or not.

@eguneys true, but it sets the state of a parent component, not the drop target itself, which could probably make a difference.
I'm working on adding reordering with drag/drop to a treeview in a project i'm working on. I wanted it to add a line before or after the target component depending on whether the mouse is in the top or bottom half of the target component. I tried using setState on the component, but it did nothing. The only solution I found that works was the one I described above, using an event emitter. Not sure if that's useful for this issue :)

I've found that running component.setState inside hover actually changes the props, not the state in the dragged component. Anyone care to explain why this happens?

If you have several wrappers, you are operating on the next target. Don't call setState() in hover, it's generally an anti-pattern. Instead hoist the state up to the parent component, and trigger its changes via callback props.


Options to Improve Initial Startup Time and Window Rendering

Hello, I've recently built my own Desktop Application Launcher called rmenu using Dioxus which was a fantastic experience, but I'm having minor difficulties with application startup time in comparison to other existing application-launchers.
I've noticed, especially on slower hardware, that rmenu can be very slow to pop up on the screen, sometimes up to 10 seconds even with a relatively simple html layout, when compared to similar launchers like wofi which also uses gtk. I've done some profiling of my code and it seems like most of the time spent under execution, including startup is related to libwebkitgtk.
I understand webview might be more complicated to startup, but are there any options available to configure gtk and webkit that might help improve the performance on startup for a faster first time to render?

@ealmloff thanks for the quick response. That's good to know. I'll see if I can put something together with wry :)


Command discussion

/help | /? | ? - Display help
/ships - List all controllable ships
/ships ship_name [orbit dock mine edit goto] [destination planet | plot]
Ships - What options do we want to do? When do switch to UI commands? Will it be easier to write UI then cli?


Tabs: fix activated tab is out of visual range bug

Please make sure these boxes are checked before submitting your PR, thank you!
Close #16563
Tabs scrollToActiveTab 之前没有考虑竖向滚动的情况

[ ] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

Also close #16532


StrawmanTest.mainTest failing in Scala 2.13 community build

[collection-strawman] [error] Test strawman.collection.test.StrawmanTest.mainTest failed: java.lang.AssertionError: assertion failed, took 0.472 sec
[collection-strawman] [error]     at scala.Predef$.assert(Predef.scala:204)
[collection-strawman] [error]     at strawman.collection.test.StrawmanTest.lazyListOps(Test.scala:466)
[collection-strawman] [error]     at strawman.collection.test.StrawmanTest.mainTest(Test.scala:576)
[collection-strawman] [error]     ...

details at https://scala-ci.typesafe.com/view/scala-2.13.x/job/scala-2.13.x-integrate-community-build/856/consoleFull
doesn't seem to be intermittent (failed in both runs last night)

or, hmm, maybe it is intermittent, there is a run currently in progress where collection-strawman already went green: https://scala-ci.typesafe.com/view/scala-2.13.x/job/scala-2.13.x-integrate-community-build/857/consoleFull
we'll see how the next few runs go

Did you change something about the evaluation of by-name right-associative operators?

Yeah now I remember that one: https://github.com/scala/scala/pull/5969
But the test should always fail with the new evaluation model.

I failed to notice that run 857 was manually triggered by Miles against a different 2.13 build.
So yes, always failing.

Hey @SethTisue! What should we do about that? Is this blocking you?

it's not blocking me.
I don't understand the issue, I don't know what you should do. but in general, this repo ought to compile, and tests pass, on whatever the latest 2.13 nightly is, since before long some descendant of that nightly will become 2.13.0-M3. (@szeiger?)

The issue is that we have a test that checks the semantics of an operation (#:: on ~Stream~ LazyList) and that this semantics has changed because the language has changed. I would be in favor of changing the test so that it takes the new language semantics into account. However, the fact that the semantics of #:: will have changed between 2.12 and 2.13 might be a problem for existing users, though I doubt something relies on the fact that this operator is not really lazy.

I would be in favor of changing the test so that it takes the new language semantics into account

👍

Fixed by https://github.com/scala/scala/pull/6509


Kites run as Windows Service

The program can be packaged with @nexe and service wrapped by @winsw. This will resolve startup time of nodejs server at level discover.

Out of scope!


no usable macos build available

I tried to get DockerStation on MacOS using two methods, both failed.
brew
brew cask install dockerstation

This installed 1.5.1 version but when trying to run it MacOS refused to start it because is not signed. For obvious reasons I am not willing to unsecure my machine just for testing it.
github releases
Downloaded the archive and tried to run the app, this time an even more awkward error appeared The application “DockStation.app” can’t be opened.
error	19:45:24.843668+0100	Finder	spawn_via_launchd() failed, errno=111 label=<private> path=<private> flags=1
error	19:45:25.084384+0100	swcd	Error getting enterprise-managed associated domains data. If this device is not enterprise-managed, this is normal: Error Domain=SWCErrorDomain Code=1701 UserInfo={Line=191, Function=<private>}

Note: I used latest release of MacOS 10.15

Hi @ssbarnea Thanks for the issue!
I'm still on 10.14 I'll update in the next few days and will check.
The app was signed, but maybe it relates with some new Apple Security Policy changes.
Can you open the app via the context menu?

$ codesign -dv --verbose=4 /Applications/DockStation.app/
Executable=/Applications/DockStation.app/Contents/MacOS/DockStation
Identifier=com.dockstationio.dockstation
Format=app bundle with Mach-O thin (x86_64)
CodeDirectory v=20200 size=349 flags=0x0(none) hashes=3+5 location=embedded
VersionPlatform=1
VersionMin=657664
VersionSDK=658432
Hash type=sha256 size=32
CandidateCDHash sha1=de30e346760a7fbab556bc73737a219ab955bcd1
CandidateCDHash sha256=c0d8ff88a814400a2ff87c5f3998277fefa92b43
Hash choices=sha1,sha256
Page size=4096
CDHash=c0d8ff88a814400a2ff87c5f3998277fefa92b43
Signature size=9009
Authority=Developer ID Application: Igor Kozlovsky (SKSVUFLAH3)
Authority=Developer ID Certification Authority
Authority=Apple Root CA
Timestamp=Apr 11, 2019 at 2:29:32 AM
Info.plist entries=24
TeamIdentifier=SKSVUFLAH3
Sealed Resources version=2 rules=13 files=65
Internal requirements count=1 size=192

@igor-lemon I get almost the same output but I cannot open the app, even with side-menu.
https://sbarnea.com/ss/Screen-Shot-2019-10-15-21-46-21.17.png
I can browse the package content.

@ssbarnea Ok. I'll update macOS and write some additional information. I think we need only to rebuild the app.

Here is something interesting: The error described above happened while trying the zip download for mac. I tried the DMG one and that one gives

“DockStation.app” cannot be opened because the developer cannot be verified. macOS cannot verify that this app is free from malware.

I think that the zip is corrupt and should not be used, but also the DMG version is missing the signing.
https://sbarnea.com/ss/Screen-Shot-2019-10-15-21-51-23.78.png

I updated macOS, confirm the problem.


Get a common value from array

I' ve found that there is no function that can retrieve a common value from an array. Maybe you can add something like this?
function commonVal(arr) {
        const sorted = arr.sort();

        let obj = {};

        let tempArray = [];

        sorted.forEach((val, i, arr) => {
            if (arr[i] != arr[i + 1]) {
                tempArray.push(arr[i]);
                obj[tempArray.length] = tempArray;
                tempArray = [];
            } else {
                tempArray.push(arr[i]);
            }
        });

        const numsArr = Object.keys(obj).map(el => +el);

        const getCommon = obj[Math.max(...numsArr)][0];

        return getCommon;
}

I may be interpreting this incorrectly, but are you looking for the value with the most occurrences in a list (ie: mathematical "mode")?
Ramda does have a few math functions but only very simple ones. Maybe this is better off in the cookbook?
I got an implementation of commonVal using ramda functions (more akin to whats currently in the cookbook) started.

The :woman_cook: :book: is the place for this. :)

@alavkx , that's exactly what i meant. I'm just new to Ramda, that's why my impelementation is on pure JS

https://github.com/ramda/ramda/wiki/Cookbook#mode-return-most-common-item-from-a-list
:)


Unpack archive quietly

Description of Changes
Please check that the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Average (%)
Memory Average (MB)

1
1000
3.4655704
30.63416

1
5000
8.724368
40.576645

1
10000
15.034778
54.13389

1
50000
66.22595
208.73209

1
100000
128.31378
309.73035

10
100
5.7932167
28.343481

10
500
10.448537
40.137527

10
1000
19.13836
51.99717

10
5000
70.22554
194.95299

10
10000
131.05191
331.80295


Microsoft Word and PowerPoint support?

Documentation says this supports MS Word and PowerPoint. Tried uploading .doc file and search index built has 0 data.
Please validate?
./scripts/datautils.py looks like only the ff filetypes is supported.
FILE_FORMAT_DICT = {
        "md": "markdown",
        "txt": "text",
        "html": "html",
        "shtml": "html",
        "htm": "html",
        "py": "python",
        "pdf": "pdf"
    }

Also logged this issue

Hi @raffertyuy, thanks for your feedback. MS word and PowerPoint are supported on AOAI studio https://oai.azure.com/portal, not through scripts now.


Input stream skip props

Skipping the initial getProperties when opening a blobInputStream in favor of a download call for better performance
Resolves #23326

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

Is this PR still progressing?

API changes have been detected in com.azure:azure-storage-common. You can review API changes here
API changes
+         protected StorageInputStream(long rangeOffset, Long rangeLength, int chunkSize, long contentLength, ByteBuffer initialBuffer)

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - ci


Improve the network profiler UI

Add icons to the up/down bandwidth LineEdits for better visual grepping.
Make bandwidth LineEdit texts more prominent when data is being received/emitted.
Add more spacing between the up and down bandwidth LineEdits.
Initialize the bandwidth texts using set_bandwidth() to ensure consistency with the actual bandwidth texts once the profiler is started.
Fix icon colors when switching from a dark theme to a light theme and vice versa.
Add missing EDSCALE constants for hiDPI scaling.

Preview

Thanks!


Wakefield College Email Request

File generated from jetbrains site
Proof that this email is for wakefield college:

Here's an IT course:
https://courses.wakefield.ac.uk/PSWebEnrolment/webenrolment.aspx?page=~/webcontrols/coursedisplay.ascx&CourseInformationID=2951
And lastly the main site page is:
https://www.wakefield.ac.uk

@Officialy Pull request merged. Thank you!
For the record: actual domain ownership proof is here: https://moodle.wakefield.ac.uk/mod/forum/discuss.php?d=6318


foo:={"bar":"baz"} doesn't work as expected?

The following example results in a string as the value, instead of raw json:
> bat POST :8000 name:='{0:0}'
POST / HTTP/1.1
Host: localhost:8000
Accept: application/json
Accept-Encoding: gzip, deflate
Content-Type: application/json
User-Agent: bat/0.1.0

{"name":"{0:0}"}

The same method with an array instead of an object results in an array, as expected:
> bat POST :8000 name:='[0,0]'
POST / HTTP/1.1
Host: localhost:8000
Accept: application/json
Accept-Encoding: gzip, deflate
Content-Type: application/json
User-Agent: bat/0.1.0

{"name":[0,0]}

pls be aware that {0:0} is not valid JSON. The key must be a string

pls be aware that {0:0} is not valid JSON. The key must be a string


Fix account_index

This fixes #735.
The X-Goog-PageId header determines the active account, not the X-Goog-Authuser header. Data sync IDs come from the /getDatasyncIdsEndpoint endpoint and are saved in local storage.
The prettyPrint and alt parameters need to be removed to prevent this response to /playlist requests:
)]}'
{"reload":"now"}

Here are examples of data sync IDs for the number of accounts signed in.
Three accounts:
id_1||id_2::yt-player::yt-player-lv
id_0||::yt-player::yt-player-lv
Two accounts:
id_1||id_0::yt-player::yt-player-lv
id_0||::yt-player::yt-player-lv
One account:
id_0||::yt-player::yt-player-lv

This pull request seems to assume that every user of the library is only using it directly on YouTube's own website and this will blow up for everyone that doesn't (which is probably most users).
A better approach would be to add an extra parameter so the session creation so you can pass the relevant parameters into the library, the same way that visitor data and poToken's are handled.

The changes only apply if the cookie is set, and this only works in the browser:
https://github.com/LuanRT/YouTube.js/blob/4942992/src/utils/HTTPClient.ts#L140
Are you suggesting limiting the changes to accepting a data sync ID and setting the X-Goog-PageId header? In that case, the account_index option would need to be removed entirely.

I added a client_name check to make the intent clearer.

Are you suggesting limiting the changes to accepting a data sync ID and setting the X-Goog-PageId header?

Better that it breaking for people passing cookies to YouTube.js on Node, Deno or ReactNative or people using it in web browsers but not on the YouTube website.

It should be more robust now, the only risks I see are:

Only the web client will get the X-Goog-AuthUser header, which didn't appear to work previously.
Other websites will not have additional URL search parameters.

Works with the web client and in Node.js.


Updated README.md to resolve issue #13

#13

thx @eric-horvat but the MR title is misleading, you actually fixed the code, not the README :)


remove prerelease.txt

This was a marker file for docs generation, inherited from https://github.com/Particular/docs.particular.net/tree/89cb47f058a999db257ee5e24465f8d7dd4c8a8f/samples/step-by-step/Core_6

@Particular/hashbus-maintainers please merge


feat: Implement python binding

Maybe like this: https://github.com/pgvector/pgvector-python

hi master @gaocegege , I'm interested in this, but can you provide more details? I have some thoughts but am still a bit confused.

/cc @VoVAllen

@VoVAllen  hi master, sorry for the late reply 😅. A bit busy with new semester enrolment. I'm alr in the discord channel, but I didn't found your name to at. Could you at me later when you are free?

I'm currently working on this and the following is considered to be implemented. The list may update frequently.
Low-level python binding is to add support of vector-related Python structures for the Python postgres libraries.

From the view of libraries:

[x] Support SQLAlchemy (ref: https://github.com/pgvector/pgvector-python#sqlalchemy)
[ ] Support Psycopg 3 (ref: https://github.com/pgvector/pgvector-python#psycopg-3)
[ ] Support Psycopg 2 (ref: https://github.com/pgvector/pgvector-python#psycopg-2)
[ ] Support SQLModel (ref: https://github.com/pgvector/pgvector-python#sqlmodel)
[ ] Support asyncpg (ref: https://github.com/pgvector/pgvector-python#asyncpg)
[ ] Support Peewee (ref: https://github.com/pgvector/pgvector-python#peewee)

From the view of Python structures:

[ ] Read/Write 1-d np.array

[ ] Considering #86 , elements of vector may be u8

[ ] Index options by Python dict/list structure, instead of TOML string


streamline bibtex import

I am using hugo academic to build a website for a large team that spans several organisations. The publications of this team are collected in a central bibtex file, which is large. An import with the present version of the academic admin tool still requires considerable manual work per publication:

copying the generated *.bib file into cite.bib (required to activate the citation button)
when the bibtex file contains an abstract entry this is included in the generated bib-file, which is generally not what users want that click the cite button.
adding the projects identifier in the markdown file of the publication to link this publication to specific projects

With this pull request the manual work per publication is reduced to a minimum, regarding the bibtex-file as the single source of information.

per publication a file cite.bib is generated
the projects identifiers are auto-generated from corresponding entries in the central bibtex file. These projects entries can include several project identifier strings (without the dashes) separated by white space.
The entries abstract and projects are removed from the file cite.bib that is generated per publication.

The only remaining work that needs to be done is to copy the PDF of the publications into the generated folders.

I am providing the input file, bibliography.bib, and the generated files, index.md and cite.bib.
files.zip

Thanks for the suggestion.
The main direction for future work on this repo is to support using the main reference managers (e.g. Zotero and Mendeley)  to manage almost all options for generating Academic publication pages. So your suggestion is very much aligned with this vision :)
The BibTeX spec does not have an entry named projects, and more importantly, such an entry wouldn't be compatible with the main reference managers like Zotero.
As discussed in other issues, we should use the extra field that is available in Zotero etc. to store key/value mapping for Academic options such as projects. The url entry can be used to store the paper URL.

Thanks for the quick reply. A simple fix would be to just swap the projects with the extra field, passing only the values and let admin import interpret these as projects values for the generated markdown files. But this would work only when there is no other use for the extra field of the bibtex file?

Closing in favour of #48


Color-code the "start of ...", "tag...", "severe bug..." balloons in the branching model figure

Not relevant anymore.


fix block finding

Fixes both multiline backtick strings and modules:
```
kjgkg
```
module Foo
1+1
2+2
3+3
end

Regression tests would probably help a lot here, but  ¯\_(ツ)_/¯

sp/fixblockfindingagainandagainandagain omg 😅

Regression tests would probably help a lot here, but ¯_(ツ)_/¯

I thought about the exact same thing yesterday actually.
I would like to open an issue about this on Juno.jl or this repository as a reminder, and thinking about possible contribution from others.
Also what do you think about using the Julia runtime (especially CSTParser I mean) here as well ?
I think about this only naively, but since most features using blocks.js rely on running Julia anyway, I guess it doesn't limit things but we can use our fav language for this as well (and then we can write tests easily as well).
We need to make cells.js not rely on getLines in blocks.js then though.


MIUI13 not support

Describe the bug
1|cupid:/data/local/tmp/ecapture-v0.1.10-android_gki-aarch64 # ./ecapture
2022/06/25 04:54:19 BTF not support, please check it. shell: cat /boot/config-`uname -r` | grep CONFIG_DEBUG_INFO_BTF
 You can compile a no BTF version by youeself with command `make nocore`,Please read Makefile for more info.

Screenshots
If applicable, add screenshots to help explain your problem.
Linux Server/Android (please complete the following information):

OS: [e.g. Ubuntu 21.04]
Linux localhost 5.10.43-android12-9-gb8e472053352-ab8059253 #1 SMP PREEMPT Mon Jan 10 06:58:42 UTC 2022 aarch64
5.10.43-android12-9-gb8e472053352-ab8059253
Arch: [e.g. arm_aarch64]
Kernel Version: [e.g. 5.10]
Version: [e.g. 0.1.3-20220313-69c1e0]

Additional context
Add any other context about the problem here.

Master branch is already supported.  I'll release version in the near future.
or you can compile it by yourself.
If you can read Chinese,Join QQ Group, I'll send a new version.(群好吧在微信公众号里)


ci: 创建PR时，如果文档有更新，生成预览文档地址

Codecov Report
Base: 64.63% // Head: 64.63% // No change to project coverage :thumbsup:

Coverage data is based on head (f6aa5b5) compared to base (2f05445).
Patch has no changes to coverable lines.

:exclamation: Current head f6aa5b5 differs from pull request most recent head c4bd1ab. Consider uploading reports for the commit c4bd1ab to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #350   +/-   ##
=======================================
  Coverage   64.63%   64.63%
=======================================
  Files         102      102
  Lines        2842     2842
  Branches      353      353
=======================================
  Hits         1837     1837
  Misses        960      960
  Partials       45       45

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


mapping between hero name and ability name?

Great dataset, thanks for putting this together and keeping it updated.
I dont see a mapping to associate hero names with their abilities (including talents). Am I missing it? Alternatively, this is a request to add it. I'm happy to lend my help to make it happen.

yeah, this doesn't exist right now since I haven't had a need to add it.  If you can find a source for the data though, just edit updateconstants.js to pull it into a new file.  You could also try to keep it manually updated but that may be a lot of work

I know you can find this mapping in the game files (dota 2 beta\game\dota\scripts\npc\npc_heroes.txt), and have read elsewhere that a lot of such info is fetched from the web by the client anyway, that the game files are there for fetch failures. Does that ring any bells? I will look around for a web version.

How would you feel about adding some processing to derive the associations, e.g. (apologies for Python syntax)
NPC_PREFIX = 'npc_dota_hero'
SEP = '_'
heroKeys = ['npc_dota_hero_antimage', 'npc_dota_hero_weaver'] #, ...]
abilityKeys = ['antimage_mana_break', 'weaver_shukuchi'] # ,...]

heroes = [h.split(NPC_PREFIX+SEP)[1] for h in heroKeys] # = ['antimage', 'weaver']
hero2ability = {}
for abilityKey in abilityKeys:
   for hero in heroes:
      if abilityKey.startswith(hero):
          if hero not in hero2ability: hero2ability[hero] = []
          hero2ability[hero].append(abilityKey)
          break

I'm sure the double-loop could be optimized, but maybe you get the idea.

Sure, sounds fine to me.  Even if it doesn't work there's no risk of regression since no one currently uses the new file.

I've found 2 scenarios where this would be useful for me.
last update was January, @austinmilt started anything on this, or should I try and do it?
However I would like the data to contain info on:

which abilities are ultimates

which level talents are (ie 1, 2, 3, or 4)

We should be able to compute this by parsing this file:
https://github.com/dotabuff/d2vpkr/blob/master/dota/scripts/npc/npc_heroes.json
It contains both abilities and talents.

@ThePianoDentist, I ended up writing a custom script to parse a bunch of stuff out from the game files themselves rather than a secondary source. I'm not confident my script is without bugs, and being that it's integral to a larger project I'm working on, I'd rather not share it here. Hopefully I'll be posting it at some point in the future. However, the specific files I use are summarized in the Python dict below:
DEF_RESOURCE_FILES = {
'abilities': {
'local': os.path.join(DEF_DOTA, 'scripts', 'npc', 'npc_abilities.txt'),
'url': r'https://raw.githubusercontent.com/dotabuff/d2vpkr/master/dota/scripts/npc/npc_abilities.txt'
},
'heroes': {
'local': os.path.join(DEF_DOTA, 'scripts', 'npc', 'npc_heroes.txt'),
'url': r'https://raw.githubusercontent.com/dotabuff/d2vpkr/master/dota/scripts/npc/npc_heroes.txt'
},
'units': {
'local': os.path.join(DEF_DOTA, 'scripts', 'npc', 'npc_units.txt'),
'url': r'https://raw.githubusercontent.com/dotabuff/d2vpkr/master/dota/scripts/npc/npc_units.txt'
},
'items': {
'local': os.path.join(HERE, 'scripts', 'npc', 'items.txt'), ## has to be extracted from vpk first
'url': r'https://raw.githubusercontent.com/dotabuff/d2vpkr/master/dota/scripts/npc/items.txt'
},
'resource': {
'local': os.path.join(DEF_DOTA, 'resource', 'dota_english.txt'),
'url': r'https://raw.githubusercontent.com/dotabuff/d2vpkr/master/dota/resource/dota_english.txt'
}
}

about to submit pull request for this (maybe should make new issue)
(I havent added anywhere that indicates/tracks which abilities are ultimates, because things like kotl make it difficult. will make a separate issue for that if I can make it work.)
however after pulling latest constants, Im getting
/home/jdog/dotaconstants/tasks/updateconstants.js:152
item.target_type = formatBehavior(scripts[key].AbilityUnitTargetTeam) || undefined;
^
ReferenceError: item is not defined
and
cant find attribute %cleave_radius%
both sporadically (although at least one always occurs). Pretty sure unrelated to my changes.


Fix Mobile Navbar (Burger Menu)

Fix needed:

Show Burger button again
correctly show all navbar entries

possible fix: transfer to JS solution

All navbar entries are aligned nicely and responsive.

Verified with google chrome latest on Windows 10, commit  00695fb.


Add a preset for avalanche transceiver

Description
Hi,
A new tag has been approved for avalanche transceiver training zone and checkpoint. Would it be possible to have a preset in the iD editor for this two objects ?
Thanks a lot !
Screenshots
No response

This should be in https://github.com/openstreetmap/id-tagging-schema


SW: need Tensorflow/Python integration with TIDL

Tensorflow Lite support is on the roadmap for 2020Q1.

With the release of TIDL 3.15.1, TensorFlow and TensorFlow Lite has been added to the supported frameworks among Caffe, TensorFlow Lite and ONNX although, not yet a verified model like the Jacinto11, MobileNetV2 ,etc.
The model import feature of TIDL there is provides constraint on the supported operators and layers. Irrespective of the framework, the supported layers are
Pooling Layer (Average and Max Pooling)
ReLU Layer
Element Wise Layer (Add, Max, Product)
Inner Product Layer (Fully Connected Layer)
Soft Max Layer
Bias Layer
Deconvolution Layer
Concatenate layer
ArgMax Layer
Scale Layer
PReLU Layer
Batch Normalization layer
ReLU6 Layer
Crop layer
Slice layer
Flatten layer
Split Layer
Detection Output Layer

But for the supported operators, during the import process, some operators/layers are combined or converted into TIDL layers. For TensorFlow specifically, the supported operators are:

TensorFlow Operator
TIDL Layer

Placeholder
TIDL_DataLayer

Conv2D
TIDL_ConvolutionLayer

DepthwiseConv2dNative
TIDL_ConvolutionLayer

BiasAdd
TIDL_BiasLayer

Add
TIDL_EltWiseLayer

Mul
TIDL_ScaleLayer

FusedBatchNorm
TIDL_BatchNormLayer

Relu
TIDL_ReLULayer

Relu6
TIDL_ReLULayer

MaxPool
TIDL_PoolingLayer

AvgPool
TIDL_PoolingLayer

ConcatV2
TIDL_ConcatLayer

Slice
TIDL_SliceLayer

Squeeze
See note below

Reshape
See note below

Softmax
TIDL_SoftMaxLayer

Pad
TIDL_PadLayer

Mean
TIDL_PoolingLayer

Whereas for TensorFlow Lite,

TensorFlow Operator
TIDL Layer

Placeholder
TIDL_DataLayer

CONV_2D
TIDL_ConvolutionLayer

TRANSPOSE_CONV
TIDL_Deconv2DLayer

DEPTHWISE_CONV_2D
TIDL_ConvolutionLayer

ADD
TIDL_EltWiseLayer

MUL
TIDL_ScaleLayer

RELU
TIDL_ReLULayer

RELU6
TIDL_ReLULayer

MAX_POOL_2D
TIDL_PoolingLayer

AVERAGE_POOL_2D
TIDL_PoolingLayer

CONCATENATION
TIDL_ConcatLayer

RESHAPE
See note below

SOFTMAX
TIDL_SoftMaxLayer

ARG_MAX
TIDL_ArgMaxLayer

PAD
TIDL_PadLayer

MEAN
TIDL_PoolingLayer

FULLY_CONNECTED
TIDL_InnerProductLayer

Note:

“Reshape” and “Squeeze” are supported by being coalesced into other layers:
If “Reshape” immediately follows “Squeeze”, they both are coalesced into TIDL_FlattenLayer.
If “Reshape” immediately follows “AvgPool”, “Reshape” is coalesced into TIDL_PoolingLayer.
If “Reshape” immediately follows TIDL_InnerProductLayer, it is coalesced into TIDL_InnerProductLayer.

Source: Official docs for TIDL: Foundational Components

Any updates on whether TF Lite support is imminent?


Dealing with bcbio-gff update to 0.7.1, using strand as a part of the location obj

tested on ensembl-production-metazoa/meta/109_for_110/dmel


Is the manifest+json MIME type actually required?

Do browsers actually check the MIME type if valid JSON is served at the target of the manifest link? Will they start doing so if they don't now?

So far the only implementation out in the wild is chrome, and it does absolutely check the mime type.    However, for manup.js we don't actually ckeck it.

Chrome also does not check for the correct MIME type, but is required. You can look it up here: https://w3c.github.io/manifest/#media-type-registration

cool thanks for the info.  I'll update the readme.


merge GO:0019539    siderophore biosynthetic process from hydroxamic acid

can we merge
GO:0019539    siderophore biosynthetic process from hydroxamic acid
The chemical reactions and pathways resulting in the formation of a siderophore from other compounds, including hydroxamic acid.
into the parent,
since it only has one child,
GO:0031169    ferrichrome biosynthetic process
which would be easier to find without this intervening term.
and the definition  is too vague ( from other compounds, including hydroxamic acid) does not offer any specificity over the parent.

see the number of siderophores here: https://pdfs.semanticscholar.org/7946/e6339f9fbb60b054747ee4d706e53958b980.pdf?_ga=2.67985041.1432405903.1614609213-563084161.1614609213

Changed labels:

from: GO:0019540 ! siderophore biosynthetic process from catechol
to: GO:0019540 ! catechol-containing siderophore biosynthetic process

and

from: GO:0019539 ! siderophore biosynthetic process from hydroxamic acid

to: GO:0019539 ! hydroxymate-containing siderophore biosynthetic process
From the review PMID:20376388, those siderophores are not synthesized from hydroxamic acid or catechol directly, but those are the functional groups that bind iron.

Added  PMID:20376388 as a reference

Removed 'has_primary_input CHEBI:24650 ! hydroxamic acid' and has_primary_input CHEBI:18135 ! catechol as explained above.

Thanks, Pascale


[ACM-10560] Added discovered cluster controller for automatic rosa import

Description
Implemented a DiscoveredCluster controller that will automatically import ROSA cluster into the ACM hub cluster. This will allow users to add an annotation to the cluster to trigger the automatic import
Related Issue
https://issues.redhat.com/browse/ACM-10560
Changes Made
Added new controller to import ROSA clusters automatically into ACM hub cluster.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[x] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[x] I have ensured that my code follows the project's coding standards.
[x] I have checked for any potential security issues and addressed them.
[x] I have added necessary comments to the code, especially in complex or unclear sections.
[x] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/retest

/cc @cameronmwall @ngraham20 @joeg-pro

/unhold

/override "KinD tests (1.21, latest)"


docs: fix syntax for configuration example

Quick fix I noticed while using the project. Thanks for the project.

Hello @jasonraimondi,
Good spotting, thank you!
Cheers,
Peter


Refactor walker

PR checklist

[x] Addresses an existing issue: #1900
[x] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

What changes did you make?
See tracking issue #1900
Is there anything you'd like reviewers to focus on?
I made a separate commit for every change. If you want I can submit them as individual PRs.
There will be a followup PR with the breaking changes for cleanup. That includes removing the is*Boundary methods from their classes and only use utility function instead.

@ajafff thanks!

@ajafff thanks!


Replace Vim with Vis

https://github.com/martanne/vis/wiki/FAQ#why-dont-you-just-use-vim
Vis is a much better and less bloated alternative to Vim.

Thanks for your interest in glaucus.
By replace vim with vis, you mean in the bootstrap process, because as a user you can choose whatever text editor you want.
That being said, I already have vim working just fine with libedit and netbsd-curses, and I have attempted to replace it with vis in the past, and it required adding more dependencies to the bootstrap process (e.g. libtermkey, lpeg and tre).

Okay, thanks.

No worries, thanks for your interest in glaucus!


Any chance to scrcpy access remote Android smartphone via Wifi rather than USB cable?

If I connect my smartphone via USB cable with my Windows Desktop computer scrcpy works fine.
However fiddling out the USB cable each time and plug it in is somehow unhandy on the long run.
Any chance to access the smartphone via WiFi?
For security reasons scrcpy could generate a permission file on Windows, then I copy it manually to Smartphone
and Android recognize that a remote program/device is allow to access the local display/smartphone.
Something like that
...just an idea.

Wifi connection is supported: https://github.com/Genymobile/scrcpy/blob/master/doc/connection.md#tcpip-wireless

Ok, thank you.
As I read I need the IP and the listening port for adb of my smartphone.
My Smartphone gets ts IP dynamically from a DHCP server and may change from time to time.
Any chance to use a device name instead of IP?
How do I find out the port number on which adb process on my Android 14 smartphone is listening for incoming connections?
Yes, I know ,the default is 5555.
But how do I get this confirmed?

Are you referring to a home LAN environment? If so, you can use the static IP function in the router


Dataset download link

Hi,
I downloaded thru Linux terminal as described in Files section:
wget -r -N -c -np --user p***** --ask-password https://physionet.org/files/eicu-crd/2.0/
And filenames are completely different from file table https://physionet.org/content/eicu-crd/2.0/
I was hoping to populate sql tables according to https://eicu-crd.mit.edu/tutorials/install_eicu_locally/
Am i supposed to choose zip file?
ADMISSIONS.csv
ADMISSIONS.csv.gz
CALLOUT.csv
CALLOUT.csv.gz
CAREGIVERS.csv
CAREGIVERS.csv.gz
CHARTEVENTS.csv
CHARTEVENTS.csv.gz
CPTEVENTS.csv
CPTEVENTS.csv.gz
DATETIMEEVENTS.csv
DATETIMEEVENTS.csv.gz
D_CPT.csv
D_CPT.csv.gz
DIAGNOSES_ICD.csv
DIAGNOSES_ICD.csv.gz
D_ICD_DIAGNOSES.csv
D_ICD_DIAGNOSES.csv.gz
D_ICD_PROCEDURES.csv
D_ICD_PROCEDURES.csv.gz
D_ITEMS.csv
D_ITEMS.csv.gz
D_LABITEMS.csv
D_LABITEMS.csv.gz
DRGCODES.csv
DRGCODES.csv.gz
ICUSTAYS.csv
ICUSTAYS.csv.gz
index.html
INPUTEVENTS_CV.csv
INPUTEVENTS_CV.csv.gz
INPUTEVENTS_MV.csv
INPUTEVENTS_MV.csv.gz
LABEVENTS.csv
LABEVENTS.csv.gz
LICENSE.txt
MICROBIOLOGYEVENTS.csv
MICROBIOLOGYEVENTS.csv.gz
NOTEEVENTS.csv
NOTEEVENTS.csv.gz
OUTPUTEVENTS.csv
OUTPUTEVENTS.csv.gz
PATIENTS.csv
PATIENTS.csv.gz
PRESCRIPTIONS.csv
PRESCRIPTIONS.csv.gz
PROCEDUREEVENTS_MV.csv
PROCEDUREEVENTS_MV.csv.gz
PROCEDURES_ICD.csv
PROCEDURES_ICD.csv.gz
README.md
SERVICES.csv
SERVICES.csv.gz
SHA256SUMS.txt
TRANSFERS.csv
TRANSFERS.csv.gz
Best,
Philip

@phamidko it looks like you have downloaded MIMIC-III, so I assume that wget was run on https://physionet.org/content/mimiciii/1.4/ instead of https://physionet.org/content/eicu-crd/2.0/.
Please try again, using the original command in your post (wget -r -N -c -np --user p***** --ask-password https://physionet.org/files/eicu-crd/2.0/).

Thank you. Got it


Incorrect "run" instructions for arguments that require escaping/quoting

brew doctor output
Your system is ready to brew.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.1.5
ORIGIN: https://github.com/Homebrew/brew
HEAD: b8b46501e4caa1cad75b30627c123127ab09a6a7
Last commit: 2 days ago
Core tap JSON: 16 Aug 00:03 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 14.0.3 build 1403
Git: 2.39.2 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.88.1 => /usr/bin/curl
macOS: 13.4.1-arm64
CLT: <IP_ADDRESS>.1.1683849156
Xcode: 14.3.1
Rosetta 2: false

What were you trying to do (and why)?
BEFORE CLOSING THIS OUT: I know this "reference[s] a single formula", but I am 99% sure this issue is with brew and not this specific formula, I just don't know of another example with argument issues like this.
I was trying to install nginx (so I could use it as a web server)
What happened (include all command output)?
% brew install nginx
==> Fetching nginx
==> Downloading https://ghcr.io/v2/homebrew/core/nginx/manifests/1.25.1_1
Already downloaded: /Users/pzich/Library/Caches/Homebrew/downloads/0ac25af4c890976dae08285baf4c9424fbd5055bd73c5b6ddbe9e5a9723d6fac--nginx-1.25.1_1.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/nginx/blobs/sha256:29aa5568ae9d
Already downloaded: /Users/pzich/Library/Caches/Homebrew/downloads/d8804ba46fe52a9cd7ea7aa52d3ed3be9392f48d01527cc15992c766151b10ba--nginx--1.25.1_1.arm64_ventura.bottle.tar.gz
==> Pouring nginx--1.25.1_1.arm64_ventura.bottle.tar.gz
==> Caveats
Docroot is: /opt/homebrew/var/www

The default port has been set in /opt/homebrew/etc/nginx/nginx.conf to 8080 so that
nginx can run without sudo.

nginx will load all files in /opt/homebrew/etc/nginx/servers/.

To start nginx now and restart at login:
  brew services start nginx
Or, if you don't want/need a background service you can just run:
  /opt/homebrew/opt/nginx/bin/nginx -g daemon off;
==> Summary
🍺  /opt/homebrew/Cellar/nginx/1.25.1_1: 26 files, 2.4MB
==> Running `brew cleanup nginx`...
Disable this behaviour by setting HOMEBREW_NO_INSTALL_CLEANUP.
Hide these hints with HOMEBREW_NO_ENV_HINTS (see `man brew`).

This states "if you don't want/need a background service you can just run" and the command /opt/homebrew/opt/nginx/bin/nginx -g daemon off, however, this is incorrect. Running that verbatim results in
% /opt/homebrew/opt/nginx/bin/nginx -g daemon off;
nginx: invalid option: "off"

Looking at the JSON API this provides the following .service.run arguments:
% curl -s https://formulae.brew.sh/api/formula/nginx.json | jq .service.run
[
  "$HOMEBREW_PREFIX/opt/nginx/bin/nginx",
  "-g",
  "daemon off;"
]

It seems the printed command is essentially joining these strings with spaces, but the last argument needs to be escaped or quoted, i.e. $HOMEBREW_PREFIX/opt/nginx/bin/nginx -g 'daemon off;' which has the desired behavior.
What did you expect to happen?
I would expect the .service.run arguments to be properly escaped and for the command that is printed to be runnable verbatim.
Step-by-step reproduction instructions (by running brew commands)
- `brew install nginx`
- Then read the output above
- See the error with `/opt/homebrew/opt/nginx/bin/nginx -g daemon off`

This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

It affects a single formula. Moving to homebrew-core.

Ignore previous comment, misread on first attempt (not had my caffeine yet, sorry!).


Restore removeAttribute backwards compatibility

Description
SW 5.3 includes a compatibility layer that maps the old \Shopware\Components\Model\ModelManager::*Attribute methods onto the newer \Shopware\Bundle\AttributeBundle\Service\CrudService. However, the delegate for \Shopware\Components\Model\ModelManager::removeAttribute behaves in a backwards-incompatible way: When deleting an attribute, it will also traverse the "dependingTables" and try to delete the attribute there. If any of the dependingTables does not have that attribute, the called CrudService method will throw. This is inconsistent with the original method's behaviour and causes existing plugins to break for no good reason. This PR makes removeAttribute() behave in a backwards-compatible way again.

Questions
Answers

Why?
See above

BC breaks?
BC is actually improved ;)

Tests exists & pass?
yes

Related tickets?

How to test?
Here is a minimal test plugin that shows the problem this PR fixes

Requirements met?
yeah

Props for finding this issue go to @mihaiplasoianu.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-18781.
Please use this issue to track the state of your pull request.


Add pandas to Python libraries

Added new category under Python libraries - "Data Analysis" and add pandas library to the list.
Link  - http://pandas.pydata.org/

Thanks bhai! :+1: :clap:


Eyetracking QoL and updating test icons

Overview
Follows up on the work of https://github.com/microsoft/MixedRealityToolkit-Unity/pull/8385. Unfortunately, using eyetracking on nested gameobjects won't work for our current interaction model, but we're ingesting many of the QoL improvements and the updates to the test icons put there.

/azp run

/azp run


Add a FPS limit applied when the project window is unfocused or minimized

This helps decrease power usage while the project window is unfocused or minimized.
The default FPS limit is 10[^1]. It can be disabled by setting the application/run/max_fps_when_unfocused project setting to 0.
The FPS limit is disabled by default when running the project from the editor to avoid interfering with profiling tools.
This closes https://github.com/godotengine/godot-proposals/issues/2001 and closes https://github.com/godotengine/godot/issues/19741.
[^1]: I've decided to go with 10 instead of 20, because 20 doesn't result in sufficient power savings on low-end machines that already struggle with rendering projects at 30-40 FPS.
This PR can be remade for the 3.x branch if we reach an agreement on it.
Testing project: test_fps_limit.zip

I would like this feature 👍
Has it been discussed? Is there a reason it was not merged (other than that there are thousands of PR 😅)

Has it been discussed off-github?

Not that I know of.

Is there a reason it was not merged (other than that there are thousands of PR sweat_smile)

The PR is lacking a review :slightly_smiling_face:
The proposal itself seems to have reached consensus (with a score of +18 as of writing and no comments opposing it).

The problem with this is that if you are profiling, the window it will break the profiler. This would need to be disabled by the editor or via debugger protocol.

The problem with this is that if you are profiling, the window it will break the profiler. This would need to be disabled by the editor or via debugger protocol.

Maybe I could add an .editor feature tag override for the unfocused FPS limit, setting it to 0 (no limit) there? This means the unfocused FPS limit would only be used in exported projects by default.

We discussed this in a PR review meeting and were not fully convinced yet by the approach. It feels like this adds something that can indeed be useful but this might require more thinking to make sure it works well and matches users' expectations (also to be assessed together with the existing low CPU mode option).

2 years have passed. Has something changed that would make this PR more robust with the profiler?

2 years have passed. Has something changed that would make this PR more robust with the profiler?

This PR no longer affects the running project if it's being run from an editor build, so the profiling issue should be resolved for the local profiling scenario now.
However, remote profiling deploys a debug export template which would have the FPS limit applied. Perhaps we should make it only apply to release export templates instead to avoid this.


How about a v0.8.4?

Greetings!
I managed to update the version string in the old compiled nib, and did a bit of other cleanup.
There are also some good fixes on keycastr/master which should be made more easily available to others.
After making a release on github and uploading a built app, we should also update the homebrew cask to point back to this repo.
Thanks!

One thing I forgot to mention is that a memory leak is fixed in these changes, which would be good to release to the community.
Any other feedback or comments on these changes?  Otherwise we should merge this in and cut a new release soon.


Read SAP Mobile Secure configuration data

in my trial account I don't have App & Device Management under Mobiles
I only have Development and Operations

Thank you for your feedback. We haven't heard from you in the past 30 days, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Topics should indicate what general feature(s) they happen to demonstrate

Although topics will be titled by domain area, need some way to show what features each demonstrates.
(From https://github.com/bokeh/datashader/issues/437)

Examples on examples.pyviz.org now do have tags to show what they are demonstrating (typically packages).


Question about step_T

Thanks for such an elegant work on watermark attack!
I wonder how many steps of random walk the method requires on average in your experiments? I found that the default value is set to 400.

Thanks for reaching out! We used the default hyper-parameters as suggested in the code. Definitely the amount of steps would depend on span length, total and generated text length in different applications.

Thanks for your response!


Correct facebook channelData for fb_attachment from CMS

Remove attachments from outgoing.channelData when generating fb_attachment especially on botkit CMS
No Harm code, just enable botkit CMS to correctly display fb_attachment carousel on emulator as well as messenger

the fb_attachment field that originates in the CMS should not be getting this far into the conversation system. This field should be removed during the loading process:
https://github.com/howdyai/botkit/blob/0ed3bc6b68adef015f9659c23a355329d635fd8e/packages/botkit-plugin-cms/src/cms.ts#L200
What are you trying to achieve here?

Updated a missing additional commit required on botkit-plugin-cms to complete and clarify the pull.
Idea was to allow creation of dynamic fb_attachment (like messenger generic template carousel) from local or remote botkit CMS.
These change were required to make it work.

Dear @benbrown
Did you look at this PR by any chance ? It mainly add support for botkit-cms fb_attachment as function
Below line works only if fb_attachement is not a function, if it is a function below just fails
https://github.com/howdyai/botkit/blob/0ed3bc6b68adef015f9659c23a355329d635fd8e/packages/botkit-plugin-cms/src/cms.ts#L200

1-So if it is a function it was not discarded , and similarly like other block type, additional check is required for the case fb_attachment is a function hence this change :
https://github.com/howdyai/botkit/blob/d089a66d353a4246f5ccc9683641e620f9a6b698/packages/botkit-plugin-cms/src/cms.ts#L176
suggested to introduce function exclusion just to avoid failure
        if (line.fb_attachment && typeof (line.fb_attachment) !== 'function') {

2-This change just certify fb_attachment is not included in slack or generic channelData.attachments
https://github.com/howdyai/botkit/blob/d089a66d353a4246f5ccc9683641e620f9a6b698/packages/botkit-plugin-cms/src/cms.ts#L170-L172
suggested to exclude line.fb_attachment when handling slack or generic
        // move slack attachments
        if (line.attachments && !line.fb_attachment) {
            line.channelData.attachments = line.attachments;

3-Accordinly when handling attachments function, now that fb_attachment function is supported
https://github.com/howdyai/botkit/blob/d089a66d353a4246f5ccc9683641e620f9a6b698/packages/botkit/src/conversation.ts#L823-L826
suggested to introduce required handling of fb_attachment function
        if (typeof (line.attachments) === 'function') {
            // set both locations for attachments
            outgoing.attachments = outgoing.channelData.attachments = await line.attachments(line, vars);
            if (typeof (line.fb_attachment) === 'function') {
                    outgoing.channelData.attachment = await line.fb_attachment(line, vars);
                    delete outgoing.channelData.attachments;
            }
        }

Not sure this PR is still valid, if required will make a new PR
Use case where fb_attachment as function is critical such as dynamic carousel, please double check, from my check it has no regression.
Users of botkit-cms are also welcome to test or comment.
Thank you


50x errors and open files limit

Hi,
I am using aquarium to scrape some data from websites. My configuration is:

8 CPU/40GB RAM GCP instance
8 splashes; 5000 MB maxrss limit; 5 slots

For the several list of sites I am experience some issues. Scrapy logs are showing following info:
<container_name>              | 2019-02-04 13:23:50 [scrapy.downloadermiddlewares.retry] DEBUG: Retrying <GET <site_name> via http://<IP_ADDRESS>:8050/render.json> (failed 1 times): 503 Service Unavailable
for first 20-30 urls; then scrapy successfully scrapes for about 3 or 5 urls and then again 20 to 30 503 errors. Also there are 502 and 504 errors, but in smaller amounts.
At the same time I see following logs from aquarium:
splash0_1  | 2019-02-04 13:23:50.346828 [-] Open files limit: 1048576
splash0_1  | 2019-02-04 13:23:50.346965 [-] Can't bump open files limit
Also, idk if it's important, user, that starts docker process has 1024 and 4096 soft and hard limit respectively.
At the end of scraping there are following results:
{'categories': {'gallery': 3426, 'story': 11157},
 'downloader/exception_count': 3749,
 'downloader/exception_type_count/twisted.internet.error.TimeoutError': 18,
 'downloader/exception_type_count/twisted.web._newclient.ResponseFailed': 125,
 'downloader/exception_type_count/twisted.web._newclient.ResponseNeverReceived': 3606,
 'downloader/request_bytes': 73857486,
 'downloader/request_count': 71893,
 'downloader/request_method_count/GET': 655,
 'downloader/request_method_count/POST': 71238,
 'downloader/response_bytes':<PHONE_NUMBER>1,
 'downloader/response_count': 68144,
 'downloader/response_status_count/200': 15258,
 'downloader/response_status_count/404': 3,
 'downloader/response_status_count/502': 8621,
 'downloader/response_status_count/503': 43785,
 'downloader/response_status_count/504': 477,
 'dupefilter/filtered': 666,
 'finish_reason': 'finished',
 'finish_time': datetime.datetime(2019, 2, 2, 14, 27, 19, 916564),
 'httperror/response_ignored_count': 16396,
 'httperror/response_ignored_status_count/404': 3,
 'httperror/response_ignored_status_count/502': 2399,
 'httperror/response_ignored_status_count/503': 13881,
 'httperror/response_ignored_status_count/504': 113,
 'item_scraped_count': 14583,
 'log_count/DEBUG': 102871,
 'log_count/ERROR': 685,
 'log_count/INFO': 17535,
 'log_count/WARNING': 148,
 'memusage/max': 192880640,
 'memusage/startup': 47513600,
 'request_depth_max': 2,
 'response_received_count': 31654,
 'retry/count': 39577,
 'retry/max_reached': 17055,
 'retry/reason_count/502 Bad Gateway': 6222,
 'retry/reason_count/503 Service Unavailable': 29904,
 'retry/reason_count/504 Gateway Time-out': 364,
 'retry/reason_count/twisted.internet.error.TimeoutError': 16,
 'retry/reason_count/twisted.web._newclient.ResponseFailed': 99,
 'retry/reason_count/twisted.web._newclient.ResponseNeverReceived': 2972,
 'scheduler/dequeued': 103554,
 'scheduler/dequeued/memory': 103554,
 'scheduler/enqueued': 103554,
 'scheduler/enqueued/memory': 103554,
 'spider_exceptions/Exception': 23,
 'splash/render.json/request_count': 31661,
 'splash/render.json/response_count/200': 14606,
 'splash/render.json/response_count/502': 8621,
 'splash/render.json/response_count/503': 43785,
 'splash/render.json/response_count/504': 477,
 'start_time': datetime.datetime(2019, 2, 1, 19, 34, 43, 765605)}

At the same time with same setup another sites have been scraped successfully.
Also, on successful scraping there are only around 100k files in the output folder, so even if scrapy does not close all of the opened files I don't see the reason why 1 million limit on open files should be bumped.
What could be the issue?

I think that Can't bump open files limit message from the logs is harmless and not related to 503s.
If 503s are coming from the site itself, not from splash (I presume this is the case?), then it could be due scraping too aggressively?

I have concurrent_requests set to 16. Could 8*16=128 concurrent requests count as "too aggressively", keeping in mind, that aquarium uses tor (I suppose for bypassing emitting requests from one ip").
Either way, I reduced 16 concurrent requests to 8 and increased splash_wait from 10 to 50. At the first sight it reduced number of 50x errors. But still have some "rows" of 20-30 50x requests, but less often. Will keep an eye on final results after finish.
Another question: If I see this in logs:
2019-02-04 20:41:10 [scrapy.downloadermiddlewares.retry] DEBUG: Gave up retrying <GET <url> via http://<IP_ADDRESS>:8050/render.json> (failed 3 times): 503 Service Unavailable
Does it mean, that resource has returned 503 or is it status code from splash? In aquarium logs I see, that containers are restarting pretty often. So could it be, that request is routed to container, that is in process of restarting?


Setting watchOptions.polling does nothing

I use a custom docker image:
FROM node:latest
RUN yarn global add react-static

With this added to static.config.js
  webpack: config => {
     config.watchOptions = {
       poll: true,
     }
     return config
   },

But changes in files wouldn't trigger rebuild after running development server.
I forked the repository and added:
    watch: true,
    watchOptions: {
      poll: true,
    },

to /src/static/webpack/webpack.config.dev.js, built it and used my fork as a local dependecy. Still nothing
This worked in other static site generators I used: Next.js, Nuxt,js, brunch.io.
Is polling broken?

Good question.  That sounds frustrating.
react-static uses webpack-dev-server and provides watching options to it directly.  I think that's overriding the configuration you've set in the webpack config.  You can see this happening here: https://github.com/nozzle/react-static/blob/master/src/static/webpack/index.js#L84-L87
We might be able this "just work" as expected by changing the way react-static configures webpack-dev-server, but in the mean time you can get the desired results by configuring the dev server directly.  Try something like this in your static.config.js:
module.exports = {
  devServer: {
    watchOptions: {
      ignored: /node_modules/,
      poll: true,
    },
  },
}

This would be an easy PR:
...config.devServer,
watchOptions: {
  ignored: /node_modules/,
  ...(config.devServer.watchOptions || {})
},

Thanks for making me understand it better! I created a PR https://github.com/nozzle/react-static/pull/493.


Vivaldi: menu item selection doesn’t work

Any menu item selected in the Vivaldi browser doesn't seem to do anything. However it seems like the problem is within Vivaldi itself as selecting menu items through the Mac’s help menu doesn't work either.

This seems to be a bug within Vivaldi itself. This has been reported to its developers (issue VB-95561).


Hole size variable scale factor depending on zoom level

Part of #295

Closing. Will be requested again if feature is still relevant.


ca: Fix log message typo

Change "shuting" to "shutting"

:) Nice, LGTM on green

Current coverage is 54.65% (diff: 100%)

Merging #1831 into master will increase coverage by 0.11%

@@             master      #1831   diff @@
==========================================
  Files           102        102
  Lines         17025      17025
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           9286       9305    +19
+ Misses         6599       6588    -11
+ Partials       1140       1132     -8

Powered by Codecov. Last update 3c82476...765bed3

LGTM


search bar config

MA-2238

@BenjiLee please review

👍

@@            master    #671   diff @@
======================================
  Files          459     459
  Stmts        19713   19718     +5
  Branches      2862    2863     +1
  Methods          0       0
======================================
+ Hit           3144    3159    +15
- Partial        373     376     +3
+ Missed       16196   16183    -13

Review entire Coverage Diff as of 9980a8d

Powered by Codecov. Updated on successful CI builds.

👍

Can we land this so we can put out an android release candidate?


NEW DEVICE: GENBODY COVID-19 ANTIGEN RAPID TEST

Please send a confirmation email when this device has been added to<EMAIL_ADDRESS>original message
Good morning,
I am inquiring about a testing device/kit that does NOT appear on the SimpleReport drop down list.  Per the SimpleReport manual, “If the device isn’t available for them to add, the administrator will need to contact SimpleReport to request this."
⚠ Request to add the GENBODY COVID-19 ANTIGEN RAPID TEST device to the drop down list of devices
We are transitioning to this device next week.  Therefore, I wanted to update my facility for the use of this …  but I did not see it listed.  Please let me know if there is anything I can do to help this process.
Thank you very much for your help.
Links:
https://www.fda.gov/media/150785/download
https://www.fda.gov/medical-devices/coronavirus-disease-2019-covid-19-emergency-use-authorizations-medical-devices/in-vitro-diagnostics-euas-antigen-diagnostic-tests-sars-cov-2

Device added


Error : Super expression must either be null or a function after building

Current Behavior
When I build my app with react-player I've got this error :
Uncaught TypeError: Super expression must either be null or a function
Just : import { ReactPlayer } from "react-player"; Lead to this error
Expected Behavior
Should not print this error (It's working with start with webpack dev server)
Steps to Reproduce

Add react-player into package json
Build using webpack
App cannot start because of error (Work properly if I remove react-player from my package.json)

Environment

Browser: Chrome 75.0
Operating system: Mac OS X Mojave 10.14.5

Other Information
Maybe related to different react version ?
Can be related to webpack configuration ?
Thanks ! :)

Seeing the same thing.  Changing webpack mode from production to development seems to fix the issue, but not a long-term solution...

Hopefully this thread helps: https://github.com/airbnb/react-dates/issues/1456
It looks like a problem with older versions of uglifyjs and the webpack plugin for it.


Use newer ipympl package for drawing matplotlib figures into a notebook

%matplotlib notebook doesn't work in Jupyter-Lab and since now also the plain notebook runs on the jupyterlab engine, it would be nice to support it. This means not using the internal matplotlib.backends.backend_nbagg but rather the ipympl package.

Thanks @wmvanvliet !


Address frontend memory leak

Goal of this PR is to ensure JS heap size does not keep increasing if octant is left idle in a browser.
currently:

0.7

Signed-off-by: GuessWhoSamFoo<EMAIL_ADDRESS>

@wwitzel3 The dev environment feels a lot better and less prone to crashing.
There is a post (https://stackoverflow.com/questions/14034107/does-javascript-setinterval-method-cause-memory-leak) suggesting setInterval is what causes heap usage to appear growing but it might actually by fine.


Add support for operation templates and operation signature reuse

This change fixes #459 by implementing support for both templated operations and the definition of operations that are based on the signature of another operation (no matter how it was defined).
Here's a contrived example (sample output here):
import "@cadl-lang/rest";

using Cadl.Http;

@error
model ErrorDetails {
  code: int32;
  message: string;
}

model CodeSignAccount {
  name: string;
}

model AccountProfile {
  value: int32;
}

@get
@doc("Reads an instance of the {name} resource.", TResource)
op ResourceReadBase<TResource, TError>(@path name: string): TResource | TError;
op ResourceRead<TResource>: ResourceReadBase<TResource, ErrorDetails>;

@post
@doc("Reads an instance of the {name} resource.", TResource)
op ResourceCreateBase<TResource, TError>(@body resource: TResource): TResource | TError;
op ResourceCreate<TResource>: ResourceCreateBase<TResource, ErrorDetails>;

@route("codeSignAccounts")
interface CodeSignAccounts {
  get: ResourceRead<CodeSignAccount>;
  create: ResourceCreate<CodeSignAccount>;
}

interface ResourceOperations<TResource> {
  get: ResourceRead<TResource>;
  create: ResourceCreate<TResource>;
}

@route("accountProfiles")
interface AccountProfiles extends ResourceOperations<AccountProfile> {
}

A couple of important things to note:

You can base an operation on the signature of an operation that was derived from yet another operation (to aid in building abstractions)
Decorators applied to operations that you reference are applied directly to the final operation, similar to model is semantics

If we make any distinction between namespace and interface operations for this syntax, it will kind of makes me regret allowing op previously in interfaces. I think it should remain the case that you can paste any namespace operation into an interface to keep that justified.
I agree changing return type now would be very disruptive, much as I like => for other reasons. And even though I like =>, I still don't like having => and : go in the same position to mean different things. I'd probably end up getting confused, especially since we had : for so long with the other meaning.

The other style strangeness I see is the casing. We previously said op names should be camelCased, but : camelCased<T> looks weird to me now. Would we recommend different casing for operations and operation templates?

Yeah, the pattern I've been following without really thinking about it is to have reusable signature names be pascal-cased. I didn't follow that in the unit tests I wrote, though.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Nick Guerrera @.>
Sent: Wednesday, May 25, 2022, 9:59 PM
To: microsoft/cadl @.>
Cc: David Wilson (AZURE SDK) @.>; Mention @.>
Subject: Re: [microsoft/cadl] Add support for operation templates and operation signature reuse (PR #552)
The other style strangeness I see is the casing. We previously said op names should be camelCased, but : camelCased looks weird to me now. Would we recommend different casing for operations and operation templates?
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fcadl%2Fpull%2F552%23issuecomment-1137730306&data=05|01|David.Wilson%40microsoft.com|a7c6d2ee1016442e703808da3e80b107|72f988bf86f141af91ab2d7cd011db47|1|0|637891019699508419|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=KsQZKNWxCdYbAuxHS%2FXpi0yGIBmremLP%2BRuA%2BsS0eWY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAATMLNA37KUBBVVEJTJYDTVLZ2A7ANCNFSM5W2HAZNA&data=05|01|David.Wilson%40microsoft.com|a7c6d2ee1016442e703808da3e80b107|72f988bf86f141af91ab2d7cd011db47|1|0|637891019699508419|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ieqf9n9pMn5ZWJCGCxU6xqhMKxhi9f%2FD%2BLqAWMl19h8%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

is is really that bad in interfaces? @nguerrera what do you think about is for this?

I don't have a strong feeling. I'm OK with is everywhere for this.

Here's what it would look like to use is inside of an interface:
interface Deployments {
  @doc("Creates a new deployment or updates an existing one.")
  createOrUpdate is ResourceCreateOrUpdate<Deployment>;

  @doc("Gets the details of a deployment.")
  get is ResourceRead<Deployment>;

  @doc("Deletes a project deployment.")
  delete is ResourceDelete<Deployment>;
}

It looks a little "off" to me aesthetically (and "get is ResourceRead ..." sounds weird), but if the consensus is that we should go with is everywhere, I'm happy to make the change.

Or in other words, please let me know if I should make the change and I'll get it done today :)

I went ahead and pushed a commit to this PR with the op is changes, take a look and let me know what you think @bterlson

I'm not sure, but I think this change might throw off the tmlanguage coloring, so that's worth double checking. Maybe add a test to packages/cadl-vscode/test.
I thought of this as I was thinking coloring to set-off the name from the is might help it look a little nicer. This syntax with is does feel slightly off to me (but not terrible). And then I think about how it really is directly analogous to model is and it seems like a lesser problem than overloading the meaning of : and I'm convinced this is the right way to go, at least vs. anything else that's been proposed so far.

Thanks @nguerrera for the hand-holding on the tmLanguage changes!

Hmmm...  Weird build failure on Windows that I can't account for:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=1616504&view=logs&jobId=3a997707-dc7c-502f-c0ea-54c37847f469&j=3a997707-dc7c-502f-c0ea-54c37847f469
Looks like none of the code projects failed to build, is it possible that all those npm warnings had something to do with it?
/cc @timotheeguerin

Seems like a new issue in node 18 https://github.com/npm/cli/issues/4980, though this is so annoying that rush assume that stderr is a failure

Can you try maybe pinning to node 18.2.0 which would not have this issue

That sounds like it :/  Thanks for checking into it!  Once @bterlson has a chance to review the final changes, I might go ahead and admin-merge since the rest of the build jobs passed (unless someone has a strong objection to it.)

PR here: https://github.com/microsoft/cadl/pull/576

Oh lord, what now...
Azure Artifacts Configuration Analysis found 1 vulnerable NuGet package manifest in the repository. Visit http://aka.ms/azure-artifacts-configuration-analysis for more details.

Since this is unrelated to this PR, I'll go ahead and merge and then chat with @markcowl and others about how we can resolve the security alert issue correctly (it's referring to NuGet packages).


closing the editor doesn't terminate processes running in the terminal

VSCode Version: 1.12.2
OS Version: Arch Linux 64bit

Steps to Reproduce:

Open the terminal (ctrl+`)
Run something in it that doesn't exit (go run some-server.go)
Exit the editor.
the process will stay in the background even after vscode is completely terminated.

I can't reproduce this (using php bin/console server:run). Does Kill Terminal work? What shell are you using?

I can't reproduce either, even when I kill the VS Code. @OneOfOne could you provide a minimal go file example?

It was a bug in the app itself, sorry for the noise.

It is not resolved yet, still VS Code terminal is leaking the process 👎

@provish Linux?

This is also happening for me - After closing and reopening VS Code, I have to manually end both the Node.JS and .NET Core processes:
react-scripts start

dotnet run

OS: Windows 10
VS Code Version: 1.16.1
Terminal: Git Bash

For the Windows version of this bug see #26807


Resource of type cloudflare_notification_policy with alert_type "http_alert_origin_error" can't be imported

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.3.6
on linux_amd64

provider registry.terraform.io/cloudflare/cloudflare v3.29.0
provider registry.terraform.io/hashicorp/aws v4.46.0

Affected resource(s)
cloudflare_notification_policy filter schema
Terraform configuration files
resource "cloudflare_notification_policy" "origin_error_rate_alert" {
  account_id  = var.cf_account_id
  name        = "Origin Error Rate Alert"
  description = ""
  enabled     = true
  alert_type  = "http_alert_origin_error"

  email_integration { id =<EMAIL_ADDRESS>}

 filters {
   slo = ["99.7"]
   zones = [
     "111111",
     "222222"
   ]
 }
}

Link to debug output
https://gist.github.com/nimda7/f4942fbb9ff47a7eb4d48aefca4f2748
Panic output
No response
Expected output
A resource should be imported.
Actual output
Error: failed to set filters: Invalid address to set: []string{"filters", "0", "alert_trigger_preferences"}
Import failed because of unknown optional field in ColudFlare API
Steps to reproduce
terraform import cloudflare_notification_policy.passive_origin_monitoring  AAAAA/BBBBBBB
Additional factoids
Actual Cloudflare schema for the alert http_alert_origin_error is that
    "Origin Monitoring": [
      {
        "display_name": "Passive Origin Monitoring",
        "type": "real_origin_monitoring",
        "description": "Cloudflare is unable to reach your origin",
        "filter_options": []
      },
      {
        "display_name": "Origin Error Rate Alert",
        "type": "http_alert_origin_error",
        "description": "High levels of 5xx HTTP errors at your origin",
        "filter_options": [
          {
            "Key": "zones",
            "ComparisonOperator": "==",
            "Optional": false
          },
          {
            "Key": "slo",
            "ComparisonOperator": "==",
            "Optional": false
          },
          {
            "Key": "alert_trigger_preferences",
            "ComparisonOperator": "==",
            "Optional": true
          }
        ]
      }
    ],

It has an optional field alert_trigger_preferences which is caused issue on import.
References
No response

Thank you for reporting this issue! For maintainers to dig into issues it is required that all issues include the entirety of TF_LOG=DEBUG output to be provided. The only parts that should be redacted are your user credentials in the X-Auth-Key, X-Auth-Email and Authorization HTTP headers. Details such as zone or account identifiers are not considered sensitive but can be redacted if you are very cautious. This log file provides additional context from Terraform, the provider and the Cloudflare API that helps in debugging issues. Without it, maintainers are very limited in what they can do and may hamper diagnosis efforts.\n\nThis issue has been marked with triage/needs-information and is unlikely to receive maintainer attention until the log file is provided making this a complete bug report.

http_alert_origin_error is not an allowed filter for the schema so it won't be able to import it.


Overzealous whitespace removal in paths

Sometime in the past year, svgo has started removing more whitespace from paths than it used to (it looks like it may be since commit 258fecf?), in a way that breaks the SVG for some viewers. For example, with this image:
<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="573.331" height="367.563" viewBox="0 0 429.998 275.672"><defs><symbol overflow="visible" id="c"><path d="M1.922-.828h3.437V0H.734v-.828c.375-.383.883-.906 1.532-1.563a52.138 52.138 0 0 0 1.218-1.265c.313-.352.532-.649.657-.89.125-.25.187-.493.187-.735 0-.383-.137-.696-.406-.938-.274-.25-.625-.375-1.063-.375-.312 0-.64.059-.984.172-.336.106-.7.266-1.094.484v-1a8.838 8.838 0 0 1 1.11-.359c.343-.082.66-.125.953-.125.75 0 1.347.188 1.797.563.457.375.687.882.687 1.515 0 .305-.058.59-.172.86-.117.261-.32.574-.61.937-.085.094-.343.371-.78.828-.438.45-1.055 1.078-1.844 1.89zm0 0"/></symbol></defs><g transform="translate(-11.094 -12.11)"><use height="100%" width="100%" y="208.938" x="16.004" xlink:href="#c"/></g></svg>

svgo v1.3.2 modifies the path in these places:
Before: a52.138 52.138 0 0 0 1.218-1.265
After:  a52.138 52.138 0 001.218-1.265
Before: a8.838 8.838 0 0 1 1.11-.359
After:  a8.838 8.838 0 011.11-.359

Some viewers (e.g. Firefox) seem to be ok with this, but others (e.g. rsvg-convert) are not:
Before:

After:

Oops, looks like this is a duplicate of #1137 and #1164


Don't output mongodb info in logs

Pull Request Test Coverage Report for Build 1668

2 of 5 (40.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 49.303%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repairnator/repairnator-core/src/main/java/fr/inria/spirals/repairnator/config/RepairnatorConfig.java
2
5
40.0%

Totals

Change from base Build 1667:
0.01%

Covered Lines:
2368

Relevant Lines:
4803

💛  - Coveralls


PeoplePicker titleText property should not be required

Category
[ x ] Enhancement
[ ] Bug
[ ] Question
Version
Please specify what version of the library you are using: 1.10.0
Expected / Desired Behavior / Question
The titleText property of the PeoplePicker (and other controls) should not be required, as is the case with many other third party react input controls. It should be optional. The user should be able to render the control without a title, or perhaps in a table cell where the title is in the header of the column.
Observed Behavior
The titleText property of the PeoplePicker is required, and if an empty string is given, it renders a default title.

This will be included in version 1.11.0.