Probable deadlock when requesting self

Expected behavior
I develop a distributed backend application which may at some point send TCP request to self (using BlueSocket library from IBM-Kitura). I expect all requests to succeed, including loopback ones.
Actual behavior
When application requests self using BlueSocket, it stops responding at all. It accepts new TCP connections, but does not react in any way. BlueSocket is stuck at reading from socket. However, it's not completely frozen. If there are internal timers or smth, it will go off as usual.
Steps to reproduce

Clone repository https://bitbucket.org/kirilltitov/testdrive
Execute swift run
Execute echo "normal" | nc <IP_ADDRESS> 1711 and notice correct response
Execute echo "deadlock" | nc <IP_ADDRESS> 1711, server is now hung
All further requests are also stuck.

SwiftNIO version/commit hash
ba077d3be8a833faeb5924b9ecf9a942bfafa690
Swift & OS version (output of swift --version && uname -a)
OS: macOS High Sierra, Mojave, Ubuntu 16.04 (docker)
Swift: 4.2 (swiftlang-1<IP_ADDRESS> clang-1<IP_ADDRESS>), also 4.1, same result
Most probably I screwed up with ServerHandler class, but the problem is so hilariously confusing that I don't even have a tiny clue on how to debug it. I tried literally everything.

Rule number one of SwiftNIO is that you must not block the event loop, but unfortunately your code does exactly that. The error is in the send function, which you call from inside a ChannelHandler (and so from on an event loop thread).
Allow me to reproduce your send function below, but with my own comment annotations.
func send(payload: String, to host: String, port: Int) -> String {
    let connection: Socket = try!
}

Each of the lines labelled Bad is a blocking operation that must not be invoked from on the event loop thread. This is because these functions will not return until the operation they have requested succeeds. In the case of connect(), that does not cause you a deadlock because the kernel will accept some number of connections without the input of SwiftNIO, so that connect call will succeed in this case. Similarly, write does not cause a deadlock because you have room in the socket send buffer, so the kernel can accept the data basically immediately. The read is where you encounter problems, because that read is blocking the event loop waiting for data. However, the event loop is the entity that is responsible for sending the data. In practice, the event loop never even gets to read the data you sent, because your blocking call prevents us from doing so.
The simplest code change you can make to get this to work is to change your send function to this:
func send(payload: String, to host: String, port: Int, promise: EventLoopPromise<String>) {
    DispatchQueue("sendQueue").async {
        let connection: Socket = try!
    }
}

Your calling code then needs a refactor too:
case "deadlock":
    // request self
    let p: EventLoopPromise<String> = ctx.eventLoop.newPromise()
    TestDrive.send(payload: internalURI, to: host, port: port, promise: p)
    p.futureResult.whenSuccess {
        response = "internal response: \($0)"
        print("Response is '\(response)'")
        self.send(response, to: ctx)
    }

With this change your code is now blocking a background thread, rather than the event loop. That frees the event loop up to process other work.
Of course, this is still not a recommended pattern, because you don't need that extra thread at all. If you want to use SwiftNIO to handle the incoming traffic, you would probably benefit from having SwiftNIO handle the outgoing traffic as well.
If you want to mix things that use BlueSocket with things that use SwiftNIO, you must always use background threads to handle the BlueSocket work, or you will at best harm the performance of your application, and at worse cause deadlock or livelock issues.

Thank you so much for such a detailed answer! I was considering migrating from BlueSocket to NIO (minus one dependency, you know). Now I see that this is a move I should've done in the first place.
Is it ok to use same MultiThreadedEventLoopGroup for both server and client in the same runtime?

It's not just ok, it's often encouraged. Unless you have a particularly deep insight into your application workload, it's generally a good idea to use relatively few event loop groups.

Yes that’s ok

Am 16.07.2018 um 20:30 schrieb Kirill Titov<EMAIL_ADDRESS>Thank you so much for such a detailed answer! I was considering migrating from BlueSocket to NIO (minus one dependency, you know). Now I see that this is a move I should've done in the first place.
Is it ok to use same MultiThreadedEventLoopGroup for both server and client in the same runtime?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

There is, but you need to be careful.
EventLoopFuture provides a wait() method you can call. That will block the calling thread until the future is resolved, and then provide the result. This is synchronous.
However, as with BlueSocket, you must never call wait from on the event loop. This is because synchronous waiting always blocks the calling thread, and if that's an event loop thread then you'll once again deadlock.

Oh, I was asking in context of NIO socket client :)

If you are in a NIO event loop, then anything that does I/O must not be a synchronous call.

Got it working, thank you all for help! There is one thing that bothers me in code snippet, that you provided, @Lukasa. Basically, in whenSuccess callback we capture self which is ServerHandler instance, and it won't be deallocated until promise resolved. I don't quite get the mechanics. whenSuccess returns immediately, as well as whole method execution. Say, promise takes some time to resolve, but what happens during that time? Why doesn't it block the next request? I mean, the ServerHandler instance isn't released yet.


[styles] Dynamic styling causes retaining obsolete resources in the heap

I use withStyles function to change background color of some components dynamically and I use the components to write an application that runs continuously.
When I executed the application, I noticed that the memory usage was increased over time while it was running. Then I checked the profiler and found many style closures left in the heap.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Changing style dynamically may make unreleased resources.
Expected Behavior 🤔

Obsolete resources are freed.
Steps to Reproduce 🕹

I wrote subset of the code that duplicates the issue on CodeSandbox.
(intentionally, duplicating the component objects to get leak much)
https://codesandbox.io/s/material-demo-ogyi7?file=/demo.js
I used Netlify to deploy the temporary code and ran it by Chrome to profile memory heap.
It looks so many style closures (at demo.js:7) are retained in the heap.
I also tried to use Styled-Component and it looks it also happens.
https://codesandbox.io/s/styled-test-73io1?file=/demo.js
Your Environment 🌎

Tech
Version

Material-UI
v4.9.14

React
16.13.1

Browser
Chrome  81.0.4044.138

I used Netlify to deploy the temporary code and ran it by Chrome to profile memory heap.

Could you add a link to that as well? Just so that we can verify it leaks in production. I'd be less concerned if this were dev-only.

Thanks.
Of course, here is the link.
https://csb-ogyi7.netlify.app/
This is Netlify deploy logs from the CodeSandbox.
running build command
><EMAIL_ADDRESS>build /codebuild/output/src078884104/src/package
> react-scripts build
Creating an optimized production build...
Compiled successfully.
File sizes after gzip:
62.88 KB build/static/js/2.748df6b0.chunk.js
777 B build/static/js/runtime-main.983a2216.js
525 B build/static/js/main.d191d666.chunk.js
The project was built assuming it is hosted at /.

I don't know if this is problematic or not. I recorded it for 20s and observed a very small increase. Is this really that bad?

It  looks like it can almost free everything it allocated. Might be interesting to compare this to the same code but using inline styles.
Either way this issue needs a bit more profiling. It's not clear from the issue description where these retained resources are.

I recorded for 2min and it looks increasing about 6MB heap size (about 180MB/hour)

And please check this.

Take a heap snapshot
Open (closure) class
Push "show all" button at the bottom of the closure list to show all closures
Sort them by Distance
See Distance 17

An update, this issue is being resolved in v5 thanks to #22342 and the new @material-ui/styled-engine package.

An update, we have now made enough progress with the new @material-ui/styled-engine package in v5 to move toward a progressive removal of the @material-ui/styles package (based on JSS). The current plan:

In v5.0.beta.0, this package will come standalone, in complete isolation from the rest.
In v5.0.0, this package will be soft deprecated, not promoted in the docs, nor very actively supported.
During v5, work on making the migration easier to the new style API (sx prop + styled() API). We might for instance, invest in the documentation for using react-jss that has more or less the same API. We could also invest in an adapter to restore the previous API but with emotion, not JSS.
In v6.0.0, this package will be discontinued (withStyles/makeStyles API).

This was made possible by the awesome work of @mnajdova.


create new buffer before releasing only when using marshal syntax

Codecov Report

Merging #93 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #93      +/-   ##
=========================================
+ Coverage    98.7%   98.7%   +<.01%
=========================================
  Files          36      36
  Lines        5166    5175       +9
=========================================
+ Hits         5099    5108       +9
  Misses         52      52
  Partials       15      15

Impacted Files
Coverage Δ

encode_pool.go
100% <100%> (ø)
:arrow_up:

encode_object.go
100% <100%> (ø)
:arrow_up:

encode.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Client fixes

Fixes #1434
Fixes #1437
Fixes #1440
Fixes #1441
Fixes #1452
Fixes #1454

Just fix conflicts


Add view parameter to userDidTapOnFailIcon()

Pass failed button view into it.

[Current coverage][cc-pull] is 72.98%

Merging [#167][cc-pull] into [dev][cc-base-branch] will increase coverage by 7.54%

@@                dev       #167   diff @@
==========================================
  Files            61         17     -44
  Lines          3310        929   -2381
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           2166        678   -1488
+ Misses         1144        251    -893
  Partials          0          0

Powered by Codecov. Last updated by [37dd3bd...03d549b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/badoo/Chatto/branch/dev?src=pr
[cc-compare]: https://codecov.io/gh/badoo/Chatto/compare/37dd3bd562eda871bf317092e2f579360b582b0b...03d549b8a01507e50f89ae448d1d91326305784b
[cc-pull]: https://codecov.io/gh/badoo/Chatto/pull/167?src=pr


Why not continue girl-sagit

↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑

It's already completed.It boots ARM64 bare metal binaries just fine.


Crop image

This would simplify significantly:
https://pillow.readthedocs.io/en/stable/reference/Image.html#PIL.Image.Image.crop

Yes this also improves detection


Additional build steps

Thanks for the quickstart. Not really a Java guy so I don't know if this is just related to my machine specific config but--

had to upgrade netty - I edited (path).m2\repository\com\microsoft\rest\v2\client-runtime\2.0.0\client-runtime-2.0.0.pom to add       <version>4.1.33.Final</version>  to all io.netty dependencies and rebuilt. Not sure if there is a better way.
was then able to Put and List, but Get gave me an error. After I set JAVA_HOME from jdk11 to jdk 1.8 , Get worked.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 804e65fe-aca5-fc8b-9a59-14860f3fa303
Version Independent ID: fa1f55ac-ea3c-d5d1-da27-33b501b1f116
Content: Azure quickstart: Create a blob in object storage by using Java Storage SDK V10
Content Source: articles/storage/blobs/storage-quickstart-blobs-java-v10.md
Service: storage
GitHub Login: @roygara
Microsoft Alias: rogarana

@amchargue Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@amchargue thanks for bringing this to our attention, I just investigated with a clean environment:
The only issue I found was that the pom.xml had an out of date version number for the , with the version 10.1.0 it did encounter the Get error you were receiving, when I changed it to version 10.5.0 (latest) the sample worked fine without any other alterations.
Didn't need to upgrade netty, so can't speak to specific environment issues.
I updated the sample code to have 10.5.0 as its version, it should be fine now on jdk11.0.2 (tested using this). Here's a direct link if you don't want to redownload the whole sample: https://github.com/Azure-Samples/storage-blobs-java-v10-quickstart/blob/master/pom.xml
Thanks again.
#please-close


Corrected "Indentation Amout" label typo, and adjusted dimensions to fit.

Corrected "Indentation Amout" label typo in Preferences pane, and adjusted the dimensions of the pane and sidebar so that the label fits.

Thanks Jason!!


LiveReload requires unsafe-inline script_src for CORS

When configuring CORS, Phoenix.LiveReload requires a script_src of 'unsafe-inline' to allow JavaScript to be injected.
This is less than ideal, because it introduces a discrepancy between the development environment and what one may reasonable want to do in production. E.g. If someone adds other inline JavaScript into the codebase, it will work locally in development, but cause a CORS violation when deployed.
Is there any way that Phoenix.LiveReload can be reworked to not require unsafe-inline?
Barring that, what would be the appropriate way to configure code reloading without the live reloading portion? I've done some experimentation, but clear guidance would be appreciated.

Is there any way that Phoenix.LiveReload can be reworked to not require unsafe-inline?

I don't know from the top of my head, feel free to give it a try.

Barring that, what would be the appropriate way to configure code reloading without the live reloading portion? I'

You can just remove LiveReloader from lib/my_app/endpoint.ex :)

Removing plug Phoenix.LiveReloader seemed to do the trick.
Do we need to keep socket "/phoenix/live_reload/socket", Phoenix.LiveReloader.Socket and phoenix_live_reload?
Some of my updates to endpoint.ex resulted in performance issues. I haven't yet narrowed it down.
It was either the removal of the Phoenix.LiveReloader.Socket or the addition of CheckRepoStatus:
plug Phoenix.Ecto.CheckRepoStatus,
      otp_app: :myapp,
      migration_lock: :pg_advisory_lock,
      migration_paths: &MyApp.Release.migration_paths/1

I'll try to narrow it down and report back.

You can remove the socket and phoenix_live_reload.
You still want to keep both CodeReloader and CheckRepoStatus inside the if code_loading? do block. They do affect performance but should only run in dev.

Closing this, I beleive it is address!


Career Counselor

https://chat.openai.com/g/g-yD0ZMqZLT-career-counselor
Empathetic career counselor offering guidance and market insights

Merged


Add support for 2D convolution

Is there any plan to add support for 2d convolutions?

We do not have plans for supporting 2D sparse convolutions in the near future. A quick hack for this might be always setting z to 0 (which might be not optimal in terms of acceleration). I'm wondering what the use case is for using 2D sparse convolutions.

@zhijian-liu In cases where the image is very sparse and you want to optimize the network, for example the MNIST digits is sparse.


VMStaticScrape metrics name

I have created a VMStaticScrape resource but I don't know if it is even doing anything.
Could you let me know what's the name of the time-series that I should be looking at to see the results?

Hey, @gtirloni.
This issue is still relevant to you?


Only compile Agda instances to a Haskell instance if they target a Haskell class

This is a little gift for @jakobn-ai , who asked in https://github.com/agda/agda2hs/pull/270#issuecomment-1948366775 if we could have instances on the Agda side that compile to regular functions in Haskell. I thought this should be easy since we can just check whether a given instance actually targets a type that is compiled to a Haskell class, and it turns out I was right :)

The second commit implements the same logic but for instance arguments: if the type of an argument is not compiled to a class in Haskell, then we compile it to a regular argument instead.


Adding observer affects original component

What is the current behavior?
Adding observer wrapper affects original render method. There's some regression after upgrading mobx-react  to version 6 caused by affecting the original (unwrapped) component.
class Foo extends React.Component {
  render() {
    //...
  }
}

const observableFoo = observer(Foo);

observableFoo.render as well as original Foo.render is overridden by adding
<Observer>
    <Component />
</Observer>

Since observer isn't pure, there's no possibility to reuse a component without mobx.
Here's a repository with repro steps: https://github.com/sergei-startsev/mobx-react-observer
What is the expected behavior?
There's an expectation that adding observer wrapper should not affect the original component. E.g.  adding observer wrapper in the default export below should not affect named Foo export:
export class Foo extends React.Component {
  // ...
  render() {
  }
}
export default inject('store')(observer(Foo));

// Foo shouldn't be wrapped
import { Foo } from './Foo.js';

I don't follow what's the issue. If you use exported Foo, it's unaffected by observer and as such can be reused without knowing anything about MobX. If you use default export in tests, it will have Observer in there, otherwise, it won't be there. I wouldn't consider it impure. Using the same input gives the same output.
Can you elaborate more please?

Seems to be done deliberately, check this PR
https://github.com/mobxjs/mobx-react/commit/d3820da16e45bf2b97695bb465605cb2d2faea23#diff-3035dd5a37335cc50db60f91ad838797
Maybe there is some reasoning in the original PR:
https://github.com/mobxjs/mobx-react/pull/523
And it's also mentioned in this comment (last sentence):
https://github.com/mobxjs/mobx-react/issues/588#issuecomment-436228990

@FredyC

If you use exported Foo, it's unaffected by observer and as such can be reused without knowing anything about MobX.

It is affected, take a look at Foo > shallow test from repro steps. It uses named unwrapped Foo export, however its snapshoot contains Observer component:
- <div>
-   Test
- </div>
+ <Observer>
+   <Component />
+ </Observer>

Is this a viable workaround?
export default inject('store')(observer(class ObserverFoo extends Foo {}));

@sergei-startsev I am looking at a snapshot file and I don't see any Observer there. That's why I am confused.
https://github.com/sergei-startsev/mobx-react-observer/blob/master/__snapshots__/Foo.test.jsx.snap#L9

@urugator
Yes, it works, but it doesn't look like a workaround though. I suppose the original render method shouldn't be overriden:
https://github.com/mobxjs/mobx-react/blob/4904b624f46be2e14d3a3d1017c1fa9bcb334f02/src/observer.js#L140
It seems to me that a new inherited class has to be created instead of changing prototype of the original:
https://github.com/mobxjs/mobx-react/blob/4904b624f46be2e14d3a3d1017c1fa9bcb334f02/src/observer.js#L126

@FredyC
Take a look at mobx-react 6 step: https://github.com/sergei-startsev/mobx-react-observer#mobx-react-6

I see your step, but writing something in readme does not make it true 😆 Why test snapshot is showing something else?

It seems to me that a new inherited class has to be created instead of changing prototype of the original:

Have you checked my first comment?
There was a PR doing what you're suggesting, but the PR was quickly reverted after a month or two.

@FredyC I've added both examples with mobx-react 5 and 6 to demo different results. See the issue description:

There's some regression after upgrading mobx-react to version 6 caused by affecting the original (unwrapped) component.

Just run steps from mobx-react 6 to see the diff.

It would be much more helpful if the snapshot would be showing an actual problem.
Either way, check the discussion on why the change you are suggesting was reverted. If you have idea how to make that work, PR is most welcome.
https://github.com/mobxjs/mobx-react/pull/523#issuecomment-423180044

See also #703, this behavior might need to be reverted anyway (not that it really does make a fundamental difference, v5 is also patching classes, but at least the behavior would be the same)
That being said, this problem is unavoidable from an implementation level, as we have to reach into some of the React internals. If you want a pure declaration: don't use observer, but use Observer instead. That is the "proper" composition abstraction React, but not without it's own downsides, such as making it hard to coordinate life cycle events between the "outer" component and reactive rendering in the "inner" component.

Fixed in 6.1.0


JSON-VBA works on Windows 10/Access 2016 32bit but not Windows 11/Access 2021 64bit

I was previously using VBA-JSON successfully on Windows 10 (32 bit) ...I'm now using the exact same VBA code on Windows 11, but VBA-JSON is throwing up an error....

Run time error '458'
Variable uses an Automation tpe not supported in Visual Basic

I searched & found  https://github.com/VBA-tools/VBA-JSON/issues/86 (issue 86).... which was closed with "Fixed in v2.2.4"
...I'm now using VBA-JSON v2.3.1 (I'm on Windows 11 Pro 23H2) ...but the problem persists... could do with a little coop by someone who has a handle on VBA-JSON

Did you step through the code and looked at the value in the locals window like they did in the issue you referenced? This would help identifying the exact source of the problem.

Hi,
Ok, so it looks like this might be a case of me being rubbish at coding. My code (using VBA-JSON) worked ok on Windows 10 32 bit (Access 2016), but same code barfed on Windows 11 64 Bit (office 2021)...therefore I made a (wrong) assumption it was the change in operating system & Office version.
I've revisited my code today & made a few changes (so unfortunately I can't be precise about what the actual remedy was, my feeling is my code was iterating past the the point where it should have, so for example instead of doing 10 loops/iterations ...it was doing 11...which threw up an error. I can't account for why this wasn't happening on my pre updage PC though - same code.).
So hands up...this one was down to me, nothing to see here. Could some kind soul close the issue down?
Thank you.

AFAIK only @timhall has maintenance rights. Although he is still active on Github, he hasn't done much here for years.
But you can close issues that you have opened.
There should be a 'close issue' option on your issue's page. (See doc)

(thanks ...just closed it myself :-) )


⚠ Bofrost.de has degraded performance

In 368a1bc, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1729 ms

Resolved: Bofrost.de performance has improved in 69239c5 after 1 hour, 23 minutes.


⚒[FEATURE] LatexTextSplitter Integration

Describe the feature you'd like
LatexTextSplitter Integration
Docs: https://js.langchain.com/docs/modules/indexes/text_splitters/examples/latex
✅Tasks

[ ] Add LatexTextSplitter node

🎯Acceptance Criteria

[ ] LatexTextSplitter node should be able to load documents in Latex format, and split content into documents based on the Latex syntax

👨💻How to contribute
Follow this guide for step by step on how to fork, commit, and submit PR.
👨💻Reference code

Recursive Text Splitter

More info
Like to work on this issue? Let us know, and we will assign it to you 🚀
If you need any help or just want to hangout, reach out to us on our Discord server👾

@HenryHengZJ Can you assign me the issue please.


Move main repo to the heroku/12factor

Maybe it makes sense to move the main repository to the https://github.com/heroku/12factor ?

ping @adamwiggins @jonmountjoy

That is the main repo, as noted in the description for this one. What other action do you suggest?

I mean transferring repository: https://help.github.com/articles/transferring-a-repository/
It will make main repository, not a fork.

Transferring repository should move this issue tracker to the main repository.

@jonmountjoy any ideas?

I think it's just fine as it is.  If anything, I think @adamwiggins can put the warning about the main repository elsewhere inside the README, and perhaps close down these issues and the issue functionality in this repo.   But I'm receiving plenty of issues and pull requests over at the other repo, and I like the fact that it's forked from here, because, well, it reflects the real history.

As easy as archiving this repo. At least in that way no changes or issues would exist and the "history" concern would still be kept


⚠ IMAP has degraded performance

In a37f81c, IMAP (https://news.ycombinator.com) experienced degraded performance:

HTTP code: 200
Response time: 4755 ms

Resolved: IMAP performance has improved in 9685750 after 5 minutes.


[Bug] Codeunit 90 "Purch.-Post" - OnFinalizePostingOnBeforeUpdateWhseDocuments

In codeunit 90 "Purch.-Post" the signature of the event OnFinalizePostingOnBeforeUpdateWhseDocuments is wrong because IsHandled is not var.
actual:
    local procedure OnFinalizePostingOnBeforeUpdateWhseDocuments(var PurchaseHeader: Record "Purchase Header"; WarehouseReceiptHeader: Record "Warehouse Receipt Header"; TempWarehouseReceiptHeader: Record "Warehouse Receipt Header" temporary;
        WarehouseShipmentHeader: Record "Warehouse Shipment Header"; TempWarehouseShipmentHeader: Record "Warehouse Shipment Header" temporary; WarehouseReceive: Boolean; WarehouseShip: Boolean; IsHandled: Boolean)

correct:
    local procedure OnFinalizePostingOnBeforeUpdateWhseDocuments(var PurchaseHeader: Record "Purchase Header"; WarehouseReceiptHeader: Record "Warehouse Receipt Header"; TempWarehouseReceiptHeader: Record "Warehouse Receipt Header" temporary;
        WarehouseShipmentHeader: Record "Warehouse Shipment Header"; TempWarehouseShipmentHeader: Record "Warehouse Shipment Header" temporary; WarehouseReceive: Boolean; WarehouseShip: Boolean; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Re-add boto2 dep to fix broken hooks imports

This should patch up #187 to unblock #183. boto2 and boto3 will happily co-exist until each of the hooks is migrated to the boto3 equivalent. Not sure how I missed this before...

Thanks!


Other resources

Congratulations to all for getting this process underway.
When preparing the first edition in 2008 of the Guide to Best Practices for Generalizing Sensitive Species-Occurrence Data, I published two other reports for GBIF. These may be of value in the discussions. These include:
Chapman AD (2006). Questionnaire on Dealing with Sensitive Primary Species Occurrence Data − Summary of responses. 56 pp. Copenhagen: GBIF.  https://doi.org/10.35035/vs84-0p13.
Chapman AD (2007). Dealing with Sensitive Primary Species Occurrence Data. Report. Report to the Global Biodiversity Information Facility 60pp. https://doi.org/10.35035/rajc-t668

Thanks @ArthurChapman
I've added these two references to the Readme doc


Provide a way to construct ada::url from a struct

For example Node uses the following struct. We need to construct a ada::url from a struct. If we use ada::url value; and then use value.set_port() it might not work, since there are limitations on the setters side. (For example, you can't set port if host does not have a value etc. This needs to also properly set the private variables inside our ada::url struct.
struct url {
  int32_t flags = URL_FLAGS_NONE;
  int port = -1;
  std::string scheme;
  std::string username;
  std::string password;
  std::string host;
  std::string query;
  std::string fragment;
  std::vector<std::string> path;
  std::string href;
};

with the following flags:
#define FLAGS(XX)                                                             \
  XX(URL_FLAGS_NONE, 0)                                                       \
  XX(URL_FLAGS_FAILED, 0x01)                                                  \
  XX(URL_FLAGS_CANNOT_BE_BASE, 0x02)                                          \
  XX(URL_FLAGS_INVALID_PARSE_STATE, 0x04)                                     \
  XX(URL_FLAGS_TERMINATED, 0x08)                                              \
  XX(URL_FLAGS_SPECIAL, 0x10)                                                 \
  XX(URL_FLAGS_HAS_USERNAME, 0x20)                                            \
  XX(URL_FLAGS_HAS_PASSWORD, 0x40)                                            \
  XX(URL_FLAGS_HAS_HOST, 0x80)                                                \
  XX(URL_FLAGS_HAS_PATH, 0x100)                                               \
  XX(URL_FLAGS_HAS_QUERY, 0x200)                                              \
  XX(URL_FLAGS_HAS_FRAGMENT, 0x400)                                           \
  XX(URL_FLAGS_IS_DEFAULT_SCHEME_PORT, 0x800)                                 \

The std::vector<std::string> path; attribute is related to https://github.com/ada-url/ada/issues/122 That is, a non-opaque path is supposed to be a list (here std::vector<std::string> path;), but std::vector<std::string> path;  is possibly a non-ideal choice: we can simply enumerate from a single string using std::string_view instances.
The flags parameter is a nice way to avoid things like std::optional but it is not strictly better. It leads to a struct that is slightly smaller, by say 8 bytes... than the ada::url struct... but these structs are ~ 256 bytes to begin with... so it is not significant.

std::vector<std::string> path is still unnecessary, and will be replaced with std::string since node.js already does this on the javascript side (it never exposes vector value to the userland.)
Node.js functions return struct url_data to the javascript side, and then depending on the action, updates the state on the javascript side. I can return the object but is_special kind of getter variables, are not stored within our architecture. I might introduce a layer on top of Ada to provide those fields before passing it into the JS layer, but I'm still investigating...

I think the correct implementation is to have a constructor of ada::url which allows us to directly set the values for username, password, host etc. without going through the parser.

Closing this. It's not needed.


Buggy under iCloud Drive

Describe the bug
Incorrect events being reported when a watched directory is in iCloud Drive.
Versions:

Chokidar version 3.0.2
Node version 12.8.1
OS version: macOS 10.14.6

To Reproduce

Create a directory under iCloud Drive (mkdir ~/Desktop/folder && cd ~/Desktop/folder)
Create watcher.js with the following contents:

const chokidar = require('chokidar');
const watcher = chokidar.watch(process.cwd());
watcher.on('ready', function () {
  watcher.on('all', function (event, path) {
    console.error('event', event, 'at', path);
  });
});

Run node watcher.js
Modify a file (touch file)
Observe the output

Expected behavior
Output:
event add at /Users/user/Desktop/folder/file

Actual behavior
Output:
event add at /Users/user/Desktop/folder/file
event addDir at /Users/user/Desktop/folder
event change at /Users/user/Desktop/folder/file
event addDir at /Users/user/Desktop/folder
event addDir at /Users/user/Desktop/folder
event addDir at /Users/user/Desktop/folder

You should be able to work around that with useFsEvents: false for now.

I've been using a modified version of Chokidar which ignores unknown events (by removing the info.event === 'unknown' check) and it's working well. The tests also pass without this bit. I've traced its addition to commit bb2a77257da129beb105d88a1d9a8632c4ee0fa4 in response to issue #165. The scenario described there seems to work without this check.

@mohd-akram please try my new branch of fsevents with chokidar's current unknown check in place. Cloning would now emit clone event, which is not unknown.

The issue remains. I don't believe all those extraneous events have a cloned flag, not sure what's their use.

Actually it resolves the issue for me. I don't see any additional events. Are you sure you've compiled fsevents properly?

Yeah, I've placed a bunch of print statements just to make sure. I can see it isn't sending unknown events any more, but they're still passing through chokidar as modified.
Before:
FSEVENT unknown
event addDir at /Users/user/Desktop/folder
FSEVENT deleted
event add at /Users/user/Desktop/folder/file
FSEVENT unknown
event change at /Users/user/Desktop/folder/file
FSEVENT unknown
event addDir at /Users/user/Desktop/folder

After:
FSEVENT created
event add at /Users/user/Desktop/folder/file
FSEVENT modified
event addDir at /Users/user/Desktop/folder
FSEVENT modified
event change at /Users/user/Desktop/folder/file
FSEVENT modified
event addDir at /Users/user/Desktop/folder
FSEVENT modified
event addDir at /Users/user/Desktop/folder

Hello, I am having issues with this too, and I am wondering if there is a way to detect if a folder is inside iCloud drive, and if so, then set useFsEvents to false.

Let me know how it feels on 3.3.


Cursor should not always be pointer when modal is open

Seems to be affecting electron only, and not the preferences modal

picking it up.

Picking it up.

This issue is fixed by https://github.com/vmware-tanzu/octant/pull/2328.


popcorntime: removed package as discussed in #15120

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

The Popcorntime project has been split into 3 forks. Two of those forks are questionable and it is unsure which one to use reliably. The last fork (project Butter) doesn't have a stable release yet.
That is why this package can be removed.
cc @joachifm

Thank you


chore: increase codeowners signed

superseeds #127

@GangGreenTemperTatum hmm still seems not be signed. i just created a PR on your behalf https://github.com/cohere-ai/cohere-toolkit/pull/130, could you ptal?


fix gap multiplier

we are using 3 as the default gap multiplier, however users should be able to modify it by intervals of size unit (1) rather than the default multiplier

Need a changeset for what's the fix (so user can track what was wrong)


Speed of Reconcile with Wikidata

Hi there,
Is there a way to know what is costing time when reconciling data with Wikidata? It feels very very slow when I try to reconcile a list of technologies names with the built-in wikidata reconciliation function (~10 minutes per entry). But when I go to wikidata to search these terms the website seems pretty fast.
I want to know what is taking time. When checking logs of OpenRefine, the only log that seems related is the following line:
17:32:08.047 [                   refine] POST /command/core/reconcile (29493ms)

Is there a way to inspect more about the communication between OpenRefine and the reconciliation service? Thanks.

10 minutes per entry is definitely way too much! Even for complex queries it should not take more than a few seconds. Can you share the details of your reconciliation? Which type do you reconcile to, which properties are you using to refine the candidates?
To inspect the communication between OpenRefine and the reconciliation service, I would use a network capture tool such as Wireshark.

Thank you. I just found I misunderstood the "Maximum number candidates to return" option for limiting the number of entries to reconcile. After reducing the numbers of entries to reconcile, the percentage of progress went much faster. It was a couple of seconds for each entry indeed.
Both the web UI and the logging didn't show actual activities of reconciliation. Is there some way for me to tweak the logging to show which entry is being reconciled?
Thanks for the suggestion of using Wireshark. Is there a way to configure OpenRefine to use an http proxy? https://www.charlesproxy.com is a tool that is easier to use for http API debugging.

I don't know about the proxy. I have migrated the issue to the relevant repository, since it is a server-side issue.


json exception

How are you， The result is not a valid json data, it for more than two   " "
if (listener instanceof ResultListener) {
mListeners.remove(id);
final String result;
if (data.has(Protocol.Field.RESULT)) {
result = data.get(Protocol.Field.RESULT).toString();
}
}

Thanks! Could you please provide some more details on this problem?
Where is this code from? Is it from the library or from your custom code?
If there's an exception, can you post the stack trace?

From meteor. Java file

I want to query databse user _id = 10 user query data? How to implement? can you help me？

Finding an entry from users where _id = 10 would be something like this in MongoDB:
db.users.find({ _id: 10 })
With this library, the equivalent would be similar to the following:
Map<String, Object> query = new HashMap<String, Object>();
query.put("_id", 10);
mMeteor.call("/users/find", query);

Thank you, I have solve this problem

@guoyoujin could you tell me how you solve your problem,
the slash at the beginning is only needed when accessing collections, so how do you handle the method in server because it's started with slash and need a return data, and how do you handle the data,,
can you give me some example code, please,

@isdzulqor Sorry, the example was probably wrong. As we found out in the other issues already, you can't call the find method. You really have to use subscriptions and onDataAdded(...).
Nevertheless, it would be interesting to know how you solved your problem, @guoyoujin -- that's right :)

@mwaclawek  yeah,, I have found my solution and it's solved

@isdzulqor Sorry, what is more, these days I was not pay attention to the mail, you can look at my code, though he is written using Chinese
https://github.com/guoyoujin/learning-android-ddp


WIP: Kernel Deployment

Kernel Instance development progress for #43 and #63

[ ] Seperate Kernel.sol and Instance.sol
[ ] Update Tests
[ ] Pass all tests

Closing since this is already setup in tests


Issue with scaling while in Gpose

For a while now, I've had an issue with friends who scale their bodies being reverted while in gpose. Everything was working fine until it wasn't, and I'm not entirely sure how to resolve this.

Known, fixed in testing but will take a bit to ensure stability


(HTTP code 404) unexpected - {"message":"manifest for cloudbourne/tomcat8-helloworld:latest not found"}

Expected behavior
Actual behavior
Information about the Issue
(HTTP code 404) unexpected - {"message":"manifest for cloudbourne/tomcat8-helloworld:latest not found"}
Steps to reproduce the behavior

...
...

Go to https://hub.docker.com/r/cloudbourne/tomcat8-helloworld/tags/
Observe no latest tags were created for that image
Select a different tag in Kitematic (if one exists)
🎉  enjoy container running.

Ideally you open an issue with the repo owner to let them know how frustrating it is that they don't have a latest image tagged


chore(main): release 2.0.0

:robot: I have created a release beep boop
2.0.0 (2024-09-18)
⚠ BREAKING CHANGES

cfg: semantic-release by conventional commits

Features

cfg: semantic-release by conventional commits (de79d44)

Bug Fixes

cfg: remove uneccesary configuration (eb7b35c)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.0.0
:sunflower:


:seedling: update skopeo openshift test

Description of the change:
This update does not affect OLM. This updates an e2e test that is used downstream by Red Hat. This PR updates the version of skopeo used in the test as well as the way the skopeo pod is built for it. Now, we mount the secret as a volume and convert its contents to an auth.json file that skopeo can use and also mount an emptydir as a local cache for skopeo (something the logs showed skopeo was complaining about). I've reproduced this issue and tested this fix using an Openshift cluster.
Motivation for the change:
The test expected Openshift to add credential annotations in a secret that contains the auth configuration for the internal registry.
Architectural changes:

Testing remarks:

Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Bug fixes are accompanied by regression test(s)
[ ] e2e tests and flake fixes are accompanied evidence of flake testing, e.g. executing the test 100(0) times
[ ] tech debt/todo is accompanied by issue link(s) in comments in the surrounding code
[ ] Tests are comprehensible, e.g. Ginkgo DSL is being used appropriately
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive
[ ] Tests marked as [FLAKE] are truly flaky and have an issue
[ ] Code is properly formatted

If this is the fastest way to handle this, I'm not opposed to the change, but is there a reason we can't shift openshift specific stuff out of this repository entirely?

If this is the fastest way to handle this, I'm not opposed to the change, but is there a reason we can't shift openshift specific stuff out of this repository entirely?

I have that on my list of things. Let's deal with getting downstream green first though and yanks this out as it's own task.


nanobox run fails to handle mysql

Added a log at https://gist.github.com/wdiechmann/323dd78e83e34e87660c
Got there doing:

rails new
cd rails
nanobox run
( would have expected nanobox to build me a vm running rails and another running mysql)
nanobox fails
gem install mysql (as suggested)
nanobox run
nanobox fails

and here I am

@wdiechmann thanks for reporting this. The engine that was detected was '[√] ENGINE   : ruby (generic)' I bet that is functionality that will be in the rails engine. @glinton would know more

great - so should I address @glinton directly - or sit quietly - or try my luck with editing the Boxfile - adding
build:
  engine: "rails"

he should have gotten an email when i referenced him in my comment. He should get back to us shortly

Currently we do not have a rails engine, but it is planned. Currently you will have to specify the services your app requires in your Boxfile here is an example the ruby engine should install the mysql client if you specify mysql1 in place of that postgresql1.

perfect - I'll report back how it fares - thx a mill for your prompt attention @lyondhill and @glinton
FYI I plan to use nanobox to build container-packs to distribute to subcontractors building parts of a heterogeneous environment comprising an M$ Dynamics AX, M$ SQL, IIS, NGINX, Rails, Redis, PubNub service and Android 4.1 app collecting salestruck data

I'm afraid that did not cut the mustard :(
updated the gist  https://gist.github.com/wdiechmann/323dd78e83e34e87660c
and now puma is squeezing itself into the mix - and not with any kind words ;)
I added a Boxfile in the project root - with this:
build:
  webserver: puma

web1:
  name: web

worker1:
  name: workerz

redis1:
  name: queue

mysql1:
  name: db

It looks like there was a copy/paste error in the engine service detection logic. The engine has been published with the fix. The docker build image should be updated within the hour. You'll need to update the docker images and then force a rebuild:
$ nanobox update-images
$ nanobox dev --rebuild

Thanks!

https://github.com/nanobox-io/nanobox-engine-ruby/commit/ac649a6a421fdf43b842a38cdfc79189951fd7bb

@tylerflint - oh no; It's me thanking you guys!


SysTick time driver

Arm Cortex processor cores commonly include a SysTick peripheral.
SysTick offers some significant advantages. It is a 24-bit downcounter, which means it wraps 256 times less frequently than e.g. an STM32 16-bit timer peripheral, which is desirable at high tick rates. Reading the SYST_CSR returns and implicitly clears COUNTFLAG, so absolute counter values can be reliably accumulated without needing two interrupts per cycle. Because SysTick is an Arm peripheral, code which uses SysTick is readily shared between devices from different vendors.
SysTick poses some significant challenges. SysTick supports two clock sources: either the processor clock or a vendor-defined "reference clock". Neither of these speeds are known at compile time, nor is there a vendor-agnostic way to determine them, so there will need to be a vendor-specific runtime translation from SysTick ticks to Embassy ticks. Additionally, there is no way to trigger an interrupt from any value except 0. Setting an alarm for a specific point in the current cycle therefore requires modifying the counter register while it's counting – a read-modify-write cycle which perturbs the overall tick count – or using SysTick as a free-running timebase plus a separate hardware timer for near-term events. SysTick is a core peripheral, so dual-core devices like the RP2040 have two independent SysTick timers and no way for one core to access its opposite's SysTick.
I intend to take a stab at implementing an SysTick-based embassy::time::driver::Driver for STM32. My motivating use case a) requires a high tick rate and b) targets a device with zero 32-bit timers (#784), so a 24-bit timer at 64 MHz (SYSCLK) or especially 8 MHz ("reference clock", here HCLK/8) downsampled to 1 MHz ticks would be a win over a 16-bit timer counting 1 MHz directly. The STM32 time driver using a 16-bit timer peripheral triggers at least two interrupts per cycle – every 32.768ms – vs a hypothetical SysTick driver every 262.144ms or 2097.152ms.

My initial impressions:

A SysTick-based central timer is so not worth it for multi-core devices. It would need to be gated for single-core devices. This shouldn't really be a loss because I don't know of any multi-core devices where SysTick is one of the better timers, but it does suggest a loss of generality.
Setting alarms is not fun. You set the reload register, then write to the counter to trigger a reload. But you also have to read the counter first to see how many ticks are remaining in this cycle, and read how long this cycle was originally to figure out how many ticks have already elapsed, and you have to accumulate that in a 64-bit counter. All this takes a nonzero amount of time, and if you don't want to lose those ticks, you have to have a calibration step where you measure how long this all takes. And of course you're racing the counter too, so if you're close to a reload event you can do something complicated, or you can just spin, wait for an automatic reload, and try to set the alarm again afterwards. All this adds up to program size.
Mapping SysTick ticks to Embassy ticks is a performance sink. armv6m does not have an integer division instruction, it has a library function. I couldn't make a divisor-decided-at-runtime version of now() anything resembling fast.

Some ideas for improvements which I may or may not pursue:

It would be advantageous to know at compile time that SysTick will run at (say) 8 MHz while Embassy ticks will run at 1 MHz. With that knowledge, now() could be specialized at compile time to use a bit shift. This would be a lot faster than dividing by an arbitrary quotient determined at runtime.
SysTick is part of the core so its tick rate will always be closely related to the speed with which instructions are processed. If the code which alters it takes constant time (or constant enough time), SysTick could be stopped, read, altered, and restarted without functionally losing count. This would be a lot simpler and a lot smaller than leaving it running.

:+1: to having a "portable" time driver that only uses ARM stuff, but ARM isn't making it easy, yeah... :sweat_smile:
"Calibrating" how much time stopping+restarting systick takes doesn't seem like a solid solution to me. There's lots of factors that could influence it, like flash cache state, being preempted by interrupts...
There's a few alternative options. Taking inspiration on impls for RTIC's Monotonic impls (roughly equivalent to embassy's time driver):

Make systick overflow at a fixed rate. This means if you want 1kHz precision you need the irq to fire at 1kHz which is not great. https://github.com/rtic-rs/systick-monotonic/blob/master/src/lib.rs
Using systick for alarms and DWT CYCCNT for free-running tick count. Using something intended for debug seems a bit ugly, but it looks very promising otherwise: https://github.com/rtic-rs/dwt-systick-monotonic/blob/master/src/lib.rs

Using SYSTICK and CYCCNT would be fine for many CPUs, just gate it's availability.
One of the issues I have is that I need ALL the timers on an H7, yes, all of them.  So I can't spare one for embassy's time driver...

Not having the systick driver is actually a serious gap. There is a whole class of devices where time precision is less important than power consumption. Even a very naive systick (interrupt - stop - get next planned task time - restart) is enough for those, because the biggest contributor to time jitter is the wakeup time from deep sleep. Typically based on RTC or LPTIM (which is not supported by embassy either atm).


Default custom image & document model

Addresses https://github.com/wagtail/rfcs/pull/44
This adds a default custom image, document, rendition model in the project template. To avoid reverse accessor conflicts, the abstract base classes have their related_names set to '+', and set explicitly on their children.
After discussion with @gasman about only being used as a fallback, default_alt_text is added to the new image model.
All tests/linting passing.

I am all in favor of this, however have recently experienced a big "gotcha" regarding custom images and 3rd party packages during the process of converting coderedcms from hard-coding Image to using the custom image correctly.
For a 3rd party package to properly support custom images, it must do 3 things:

In its models.py it must reference image foreign keys using get_image_model_string()
In its migration files, it must also use to=get_image_model_string() in its foreign keys (rather than the default that makemigrations will create which is wagtailimages.Image)
In its migration files, it must also specify a dynamic dependency on whatever future app will be implementing the custom image model, as so:

dependencies = [
    migrations.swappable_dependency(get_image_model_string()),
]

The problem with this dynamic approach, is if the first-party app (I'll call it home) depends on the 3rd party app (coderedcms in my case), then creating the custom image model in the home app will create a circular migration dependency error at runtime. This is because coderedcms is now dynamically depending on home for the custom image, and home is depending on coderedcms for other reasons.
Therefore, to "play nice" with any 3rd party packages you might depend on, you will need to define your custom image model in its own isolated app, to break the circular dependency chain.
I see that this pull request already implements the custom media in its own separate app which is good. However it might be worth noting that NOT implementing it in a separate app will potentially break 3rd party packages.

I’m going to close this now, on the basis that we have an RFC that needs approval before any further work can happen in Wagtail itself: RFC 44: Make "Custom models" the default. There’s renewed interest in default custom models in the context of RFC 97: Alt Text Capabilities.


slice function incorrectly cross-linking in docs

Description
Sphinx is confused with the multiple entries for slice.
Cross-linking might be going to the wrong place. For example, https://mxnet.incubator.apache.org/api/python/ndarray/sparse.html#mxnet.ndarray.sparse.CSRNDArray.setitem has a slice() reference that points to symbol but it would seem this should be going to ndarray.
Similar to the issue with NDArray and optimizer: https://github.com/apache/incubator-mxnet/issues/12829
Can this be fixed by something like https://github.com/apache/incubator-mxnet/pull/12886 ?
Here's the error:
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/ndarray.py:docstring of mxnet.ndarray.NDArray.__setitem__:None: WARNING: more than one target found for cross-reference u'slice': mxnet.symbol.Symbol.slice, mxnet.ndarray.sparse.CSRNDArray.slice, mxnet.symbol.sparse.slice, mxnet.ndarray.slice, mxnet.ndarray.NDArray.slice, mxnet.ndarray.sparse.slice, mxnet.symbol.slice
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/ndarray.py:docstring of mxnet.ndarray.NDArray.__getitem__:None: WARNING: more than one target found for cross-reference u'slice': mxnet.symbol.Symbol.slice, mxnet.ndarray.sparse.CSRNDArray.slice, mxnet.symbol.sparse.slice, mxnet.ndarray.slice, mxnet.ndarray.NDArray.slice, mxnet.ndarray.sparse.slice, mxnet.symbol.slice

Steps to reproduce
make html (from the docs folder)

@eric-haibin-lin - can you take a look... it seems part of this may have to do with the introduction of the sparse api.

Related error with sparse:
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.add:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.add:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.divide:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.divide:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.equal:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.equal:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.greater:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
/home/ubuntu/incubator-mxnet/python/mxnet/ndarray/__init__.py:docstring of mxnet.ndarray.greater:None: WARNING: more than one target found for cross-reference u'array': mxnet.ndarray.array, mxnet.ndarray.sparse.array
{and a lot more of this...}

And this one too:
/home/ubuntu/incubator-mxnet/python/mxnet/test_utils.py:docstring of mxnet.test_utils.rand_sparse_ndarray:None: WARNING: more than one target found for cross-reference u'dtype': mxnet.ndarray.sparse.CSRNDArray.dtype, mxnet.ndarray.sparse.RowSparseNDArray.dtype, mxnet.ndarray.NDArray.dtype

I can break this out as separate issues, but I get the feeling they're somewhat related around sparse.


BUG: brief description of the bug

Sealos Version
3.3.9-rc.11
How to reproduce the bug?
1.麒麟v10操作系统
2.安装kube1.18.14
What is the expected behavior?
安装成功
What do you see instead?

Operating environment
- Sealos version:
- Docker version:
- Kubernetes version:
- Operating system:
- Runtime environment:
- Cluster size:
- Additional information:

Additional information
No response

安装失败，大佬们怎么解决啊

3.x不做维护，请在4.x上测试后提交issue


Add a hide_outputs option and a hide_markdown option

Currently, we can remove outputs and markdown, but we cannot hide them so there's a button to reveal. This seems like something that should be doable and useful to people.
I believe the fix for this would need to be here:
https://github.com/jupyter/jupyter-book/blob/master/jupyter_book/book_template/assets/js/page/hide-cell.js

closed by #420


add test_all_types() test

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

Project coverage is 61.50%. Comparing base (34a6448) to head (94be889).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
+ Coverage   60.65%   61.50%   +0.85%
==========================================
  Files          36       36
  Lines        2145     2146       +1
==========================================
+ Hits         1301     1320      +19
+ Misses        844      826      -18

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

Thanks!


How does this handle project.json references?

Does this work for all types of projects or only a select list?
Which project types does this work for?
Project.Json does not include references in .csproj files, how does this handle those references?

There is no plan on supporting project.json. This project is assuming a regular 2015 project without all the steps Microsoft took in between.
So it should work for a pre 2017 csproj file, and it only tries to optimize it to the new csproj format. I've tested with class libraries/web projects/console apps etc.


PHP chmod(): Operation not permitted

Hello,
I am using ubuntu to work on projects that are written in PHP, I have a basic LAMP stack and everything worked well ever since I started using WSL until my last Windows update where PHP keeps getting hit by permission restrictions for things like chmod, creating cache folders and files etc.
My last feature update is version 1803 and my last quality update is KB4011253.

If your permission issues are on Windows directories, did you enable the Linux metdata feature in wsl.conf?
I suggest looking at https://blogs.msdn.microsoft.com/commandline/2018/02/07/automatically-configuring-wsl/ and https://blogs.msdn.microsoft.com/commandline/2018/01/12/chmod-chown-wsl-improvements/ and see if these are useful to you. Also something you might want to consider: https://blogs.msdn.microsoft.com/commandline/2018/02/28/per-directory-case-sensitivity-and-wsl/

@DarthSpock thank you for the links! I will try them out and see if the issues persist.

Worked for me, thanks :-)
sudo umount /mnt/c
sudo mount -t drvfs C: /mnt/c -o metadata

https://blogs.msdn.microsoft.com/commandline/2018/01/12/chmod-chown-wsl-improvements/

Maybe related to this issue: https://drupal.stackexchange.com/questions/273287/phpunit-loadtest-fail-due-to-chmod-operation-not-permitted

Still does not work, what we did so far:

install ubuntu
install a lamp stack
creating a virtualserver pointing to a windows-directory, wich resides on drive f (no, var/www or other wsl-directories are NOT an option, since we need to modify / create ecc files directly from windows)
installed Joomla
tried to install addons or other stuff
get a delete not permitted error, found out that it is due to a chmod operation performed before unlinking files within the joomla-core
mounted the f-drive wia drvfs with metadata enabled (mounted correctly, i've checked it)
tried to install joomla inside the directory directly from wsl (downloaded and unzipped with wget / unzip from the command line)
still does not work...
tried to install it to var/www/subdir, from there it works, but we cannnot create files directly from windows, since they are invisible to wsl / ubuntu

but we cannnot create files directly from windows

Addressed in Insiders with a Plan9 server embedded into /init. So next release of Windows will finally have the fix for that. See https://devblogs.microsoft.com/commandline/whats-new-for-wsl-in-windows-10-version-1903/
Otherwise, did you set metadata manually or via wsl.conf? The metadata won't persist without wsl.conf when you exit your wsl session.

So, if everything succeeds, already this April? That would be great, since we tried:

setting / mounting it manually with metadata
writing it to wsl.conf
on two separate machines, neither did work...

Best regards

Exact release date isn't out yet that I'm aware of. Only suggestion at this point would be to try out PengWin and see if that fares any better than Ubuntu. Be sure to run through the setup script.

Coming back to this thread, i've updated to 1903 and now it seems to work without issues!!
Thats great news for all webdevs on win10!

sudo umount /mnt/c

it says "umount: /mnt/c: target is busy"

@Faisal-nfl you must ensure that no shell points to a location under /mnt/c, eg. close all WSL windows except current terminal, and then:
cd /
umount /mnt/c
mount -t drvfs -o metadata C: /mnt/c

if the above still fails for you, try
umount -l -f /mnt/c

just beware, this will make things a bit less pleasant (likely shell will start failing with No such file or directory upon every next action)

Avoid umount -f. It doesn't do what most people assume it does. If the target is busy even after closing all shells but one, you can find the process holing a handle with http://manpages.ubuntu.com/manpages/xenial/man8/lsof.8.html:
$ lsof +f -- /mnt/c

Exit (or kill with fire) whatever is open. Or, wsl.exe --shutdown (from powershell or cmd.exe natch) is a good big hammer go-to.


Variable Speed Pumps only have On/Off

My pumps are variable speed, but in HASS they present as on/off devices.  Generally this is fine, I can always go to the Hayward app to change speeds, but it would be ideal if they could be adjusted from within HASS.
And thank you for all the work so far!

Then you should be able to use the omnilogic.set_pump_speed service to set the speed of variable speed pumps.
I have set up scripts/scenes with preset speeds and then trigger those with buttons in the UI and on my watch to make it easy. Also have a slider helper to allow manual setting with the service.
Does that work?

I can fiddle with that, I'm relatively new to HASS but got a fire HD to make into a yard dashboard. I was thinking maybe it could be presented as a variable speed fan so that it could be a matter of just setting speed. Would make it work with Google Home for voice control easier.

With that mix of C/F you must be Canadian. ;)
Can you let me know what the helper looks like for the slider? Also what widgets your temp history displays are?

No problem - here's the helper:

Then I have an automation based on change in input_number.pool_pump_speed which actions:
data_template:   entity_id: switch.knight_home_pool_filter_pump   speed: '{{ states.input_number.pool_pump_speed.state | int }}' service: omnilogic.set_pump_speed
(I also have another automation to update the slider value when the pool pump speed changes to make sure that other changes are reflected in the slider)
The pool temperatures are in a custom:mini-graph-card (from HACS).
Let me know if the service option works for you and this issue can be closed.

I'll learn from here, close away.


A bug will lead to a failed greedy policy 训练代码有一个bug会使得greedy策略失效

我们在使用该仓库代码rl_trainer/main.py进行我们模型的训练时，发现了一个代码中的bug
We have found a bug when we are using your code rl_trainer/main.py to train our own model
	    # …… 58-66
	    state_to_training = state[0] # you have defined state_to_training here
	    # …… 68-78
            while True:
            	# …… 80-86
                actions = logits_greedy(state_to_training, logits, height, width) # Here you use state_to_train to generate greedy policy
				# …… 87-90
				next_state, reward, done, _, info = env.step(env.encode(actions))
				next_state_to_training = next_state[0] # create new varible next_state_to_training
				next_obs = get_observations(next_state_to_training, ctrl_agent_index, obs_dim, height, width)
				# …… 90-116
				model.replay_buffer.push(obs, logits, step_reward, next_obs, done)

				model.update()

				obs = next_obs
				step += 1
				# …… 123-146

代码里面定义了state_to_training，greedy策略也是使用state_to_training作为观测，但是后续代码并未将更新后的状态next_state_to_training赋给state_to_training，使得greedy策略一直观测的是开始时的状态。当然，对于我们自己模型的训练并没有影响，因为get_observations用的是next_state_to_training。但这个bug会使得greedy策略失效，有可能比random还差
You have define state_to_training at the beginning of the code, which is above the loop of a training episode. During the one episode training, you have usedstate_to_training as an observation for greedy policy. But, you haven't updated state_to_training using the updated state next_state_to_training, which would make the greedy policy continuously observing the state at the very beginning. Of course, it doesn't matter the training of our own model, because the argument passing to get_observations is next_state_to_training. We suppose that such a bug will make the greedy policy failed, maybe worse than random policy.
所以应该在更新obs的时候也更新state_to_training
The supposed code to fix the bug is as following:
	    # …… 58-66
	    state_to_training = state[0] # you have defined state_to_training here
	    # …… 68-78
            while True:
            	# …… 80-86
                actions = logits_greedy(state_to_training, logits, height, width) # Here you use state_to_train to generate greedy policy
				# …… 87-90
				next_state, reward, done, _, info = env.step(env.encode(actions))
				next_state_to_training = next_state[0] # create new varible next_state_to_training
				next_obs = get_observations(next_state_to_training, ctrl_agent_index, obs_dim, height, width)
				# …… 90-116
				model.replay_buffer.push(obs, logits, step_reward, next_obs, done)

				model.update()

				obs = next_obs
				state_to_training = next_state_to_training
				step += 1
				# …… 123-146

Hello there,
感谢提问。你提出的方案确实是正确，可以直接提一个pull request^^
Thank you for asking. Your proposed correction has been checked and you could directly submit a request. ^^
Thanks again.
Yutong


⚠ Bofrost.fr has degraded performance

In 578ca10, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1445 ms

Resolved: Bofrost.fr performance has improved in 7b5c9f8 after 20 minutes.


Missing step to output token with Roles

This PS:
This script acquires the App Context Token and stores it in the variable $token for later use in the script.
Paste your Tenant ID, App ID, and App Secret (App key) into the indicated quotes below.
$tenantId = '' ### Paste your tenant ID here
$appId = '' ### Paste your Application ID here
$appSecret = '' ### Paste your Application key here
$resourceAppIdUri = '05a65629-4c1b-48c1-a78b-804c4abdd4af'
$oAuthUri = "https://login.microsoftonline.com/$TenantId/oauth2/token"
$authBody = [Ordered] @{
resource = "$resourceAppIdUri"
client_id = "$appId"
client_secret = "$appSecret"
grant_type = 'client_credentials'
}
$authResponse = Invoke-RestMethod -Method Post -Uri $oAuthUri -Body $authBody -ErrorAction Stop
$token = $authResponse.access_token
Does not output roles in the https://jwt.ms/. It is missing the roles, so Splunk is having an issue connecting.

Thanks for your feedback. We'll investigate and get back to you.

@kinstlerm - Thanks for your patience. After checking with the product team, it has been determined that there is not a problem with the code sample in the documentation. We recommend you open a support case if you are still having an issue connecting.


TODO: Support New Architecture

@wn-na can I work on this or are you already doing it?

@Ahmedhamed77 nope, unfortunately, i cant work for it...

hi @Ahmedhamed77
I've checked into that, and there doesn't seem to be any issues with the new architecture as it stands.
For that reason, i'm not working on it right now, and it will be fixed when i work on migrating to swift and kotlin


A11y: Sort dropdown ARIA label

Adding possibility for accessible name to the search results sort dropdown

Commits have been squashed and message amended.


URGENT, response.headers.Location error

Just on Chrome, after the upload, the response.headers object comes with the Location attribute lowercased.
I dont know if its a ember-plupload bug, and started happening after the last Chrome update.

Hi there, I don't have time to look at this, and don't really maintain this too much anymore. If you provide more details, community members may be able to help you.
As a courtesy, describing your ticket as "URGENT" is less helpful than saying "Location header missing from response in Chrome X.X.X"

@trampos did you find a solution to this issue ?

@MZein1292, iv fixed my app changing the way i get the location
from:
let docurl = response.headers.Location
to:
let docurl = response.headers.Location || response.headers.location

I ended up doing the same :+1:


Update eng-US.js

Description of Changes for Review:
Added  a few new meow facts

good work, thank you for making a pull request with additions.


Fix benchmark sequence order for single-threaded runs.

Fix single-threaded runs so that the first executed benchmark is given
sequence number 1, and the last is given sequence number N.

test this please

LGTM.

LGTM

test this please


Outsource saving of bounds check errors

During subcell IDP limiting, the given subcell bounds can be checked. This happens in the stage callback BoundsCheckCallback. It contains, on the one hand the calculation of the maximum deviation for each variable, and on the other hand, a possible saving of these errors to a txt file.
This PR outsources the saving of the bounds check errors into its own function.
First, this is a much cleaner way to of structuring the bounds check routine.
But mainly, I noticed some very weird behaviors of this bounds check regarding allocations a couple of times in the last months. In some cases, there are many allocs.
And a finally found the reason for that in the error saving lines of code. Sometimes this phenomenon didn't really make sense since save_errors was set to false, but probably there were problems with this sort of IO inside the same routine. Nevertheless, with this variant here, that is solved.
Additionally, this PR increases the coverage by activating saving the bounds check errors for a local limiting and adapts the first time step the deviations should be saved (in 3b513ed).

To complete testing this "Save deviations"-part, I added another test in 8111037.


Added most important question...

Just a silly test of the pull request process.

This is going to be good.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3607

Questions ?
Please refer to the Backport tool documentation


cannot open dcm file with a alert "Assertion Failed: argument is undefined or null"

environment
vscode version : v1.91.0
platform : windows x64
extension version: v1.0.5
etc
btw, when i trying v1.0.4 extension, no alert but custom editor take empty black background, and ohif-viewer's cross origin exception popup showed at the bottomright.

Didn't have chance to check it on Windows. So I guess this could be the issue as the v1.0.5 on vscode 1.91.0 works fine on Ubuntu.


[Scope change] table 8620 "Config. Tmpl. Selection Rules" - FindTemplateBasedOnRecordFields

Describe the request
Can you please remove the 'OnPrem' scope of function FindTemplateBasedOnRecordFields in table 8620 "Config. Tmpl. Selection Rules"?
Additional context
We'd like to use the 'API Setup' framework for custom development
Internal work item: AB#537583

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 24627.


Implement 404 view that matches the one from vaadin.com

See https://vaadin.com/foobar

This is now:

Missing add-ons are presented with custom error message
Other missing URLs redirect back to search page


Any modification on original code of cuda gaussian renderer

DId you have any modification on the original code of cuda code from original repo?

Add depth forward pass to render depth map
Use abs gradient instead of vanilla gradient to guide densification.


Incorrect mapping of ZeroToSet

It looks like the fix in #673 has a bug. At https://github.com/rust-embedded/svd2rust/blame/6de189fc83268cfd5f66377c65a5aa2430de9e74/src/generate/register.rs#L858, ZeroToSet maps to "BitWriter0C". Should this not be "BitWriter0S"?

Yes. It's a typo.
It is strange nobody's found this yet.
Open PR, please.


docs: generate cli reference

This adds CLI reference generation for the main docs. Previously we only had it for zenith docs:

#6509

Since there’s now two different front matters, I thought it’s better to decouple from being hardcoded anyway (--frontmatter), which also reduced the complexity over parsing the file name to get the right slug for it. This also allowed printing the result to stdout because the file name no longer matters (--output flag).
Fixed an issue where dagger completion was being excluded from usage, but not from “See also” links. Workaround for this is to set cmd.Hidden based on a condition, before generating the docs.
Added an  --include-experimental flag, which defaults to false.
./hack/make docs:generate now updates both the main docs and zenith version, which holds the hardcoded paths to the .mdx files, and their corresponding front matters.

./hack/make docs:generate now updates both the main docs and zenith version, which holds the hardcoded paths to the .mdx files, and their corresponding front matters.

I love it! Thank you @helderco this is brilliant!


Power State is showing off in Home app

Xbox Series X shows as powered off in the Home app when it's actually powered on. When I tap the tile in Home app to power it on I see a message on my Xbox's display that "Your Xbox is already on."
I have the Web API all connected properly, just not sure why the power is not being shown properly in Home app.

When I power off the Xbox manually and then try to power on from Home app:
[4/18/2021, 9:39:22 AM] [XboxTv] Device: <IP_ADDRESS> Xbox Series X, web api set power ON, error: {   status: 404,   body: '{\r\n' +     '  "status": {\r\n' +     '    "errorCode": "ErrorWNSTempDisconnected",\r\n' +     '    "errorMessage": "Xbox temporarily disconnected from WNS"\r\n' +     '  },\r\n' +     '  "result": null,\r\n' +     '  "uiText": null,\r\n' +     '  "destination": null,\r\n' +     '  "userInfo": null,\r\n' +     '  "opId": "7f88326e-e3c9-4df5-b61a-611ad079f4b0"\r\n' +     '}' }

Are U enabled instantPower ON on Console?

Yes

Have you tried restarting homebridge?

As U see here:
"errorMessage": "Xbox temporarily disconnected from WNS"\r\n'
Your Xbox is disconnected from Windows Network System, try your firewall, nat, etc.

Currently having the same issue within the home app. The tike says it is powered off when it is on.

Because Your console is not connected to Net.
Check readme and configure Your console correct.

Not true - my console is definitely connected to the net.

Please enable debug mode and get the complete log after restart homebridge

How do I get the complete log?

Are You sure You have set correct xboxliveid (consoleID)? In config settings?

I’m sure - the tile in the home app recognises when I change game or go back to dashboard. The home app just won’t turn off / on the console

Are You enabled Instant Power ON on console?
Can You enable console from Xbox.app or Xbox GamePass.app ?

Instant On is enabled.
I can turn on using the Xbox app on my phone

We error - same as the original poster.
Only really having trouble since webAPI has been implemented

I have the same issue, do you fix it ?

Same issue here. Also was getting the below error while trying to use the TV app to control xbox.
[7/12/2021, 11:08:21 AM] [homebridge-xbox-tv] This plugin threw an error from the characteristic 'Remote Key': Unhandled error thrown inside write handler for characteristic: this.xbox.getManager(...).sendCommand is not a function. See https://git.io/JtMGR for more info.
[7/12/2021, 11:08:22 AM] [homebridge-xbox-tv] This plugin threw an error from the characteristic 'Remote Key': Unhandled error thrown inside write handler for characteristic: this.xbox.getManager(...).sendCommand is not a function. See https://git.io/JtMGR for more info.
[7/12/2021, 11:08:22 AM] [homebridge-xbox-tv] This plugin threw an error from the characteristic 'Remote Key': Unhandled error thrown inside write handler for characteristic: this.xbox.getManager(...).sendCommand is not a function. See https://git.io/JtMGR for more info.
[7/12/2021, 11:08:24 AM] [homebridge-xbox-tv] This plugin threw an error from the characteristic 'Remote Key': Unhandled error thrown inside write handler for characteristic: this.xbox.getManager(...).sendCommand is not a function. See https://git.io/JtMGR for more info.
[7/12/2021, 11:08:25 AM] [homebridge-xbox-tv] This plugin threw an error from the characteristic 'Remote Key': Unhandled error thrown inside write handler for characteristic: this.xbox.getManager(...).sendCommand is not a function. See https://git.io/JtMGR for more info.
[7/12/2021, 11:08:25 AM] [homebridge-xbox-tv] This plugin threw an error from the characteristic 'Remote Key': Unhandled error thrown inside write handler for characteristic: this.xbox.getManager(...).sendCommand is not a function. See https://git.io/JtMGR for more info.
[7/12/2021, 11:08:26 AM] [homebridge-xbox-tv] This plugin threw an error from the characteristic 'Remote Key': Unhandled error thrown inside write handler for characteristic: this.xbox.getManager(...).sendCommand is not a function. See https://git.io/JtMGR for more info.

Same error for me. Any fix?

Will check this today


Unhandled Promise rejection: [$injector:modulerr] Failed to instantiate module $$UpgradeModule due to: Error: [$injector:modulerr] Failed to instantiate module {"_invokeQueue":[],"_configBlocks":[["$injector","invoke",{}]],"_runBlocks":[],"requires":["ngAnimate","ngMessages","ngResource","ngCookies","ui.router","base64","smn-ui","core"],"name":"nexus"} due to: Error: [ng:areq] Argument 'module' is not a function, got Object http://errors.angularjs.org/1.6.9/ng/areq?p0=module&p1=not a function%2C got Object

When trying to make an application hybrid with angularJs and angular 4 this error, I use this example https://vsavkin.com/migrating-angular-1-applications-to-angular-2-in-5-simple-steps-40621800a25b.

Can anybody sugest a valid example for angular ugrade?

@pcbsytem , please post your question in stackoverflow, this is not a zone.js issue.

[$injector:modulerr] Failed to instantiate module $$UpgradeModule due to:
Same error throwing myself also. when i try to reload the page with angular and angular 5

@sathishkumar94  Me too facing the same issue. Were you able to fix it?

will close this for now because of lack of information.


[x] Use ngrok to build slack locally.
[x] Create a personal Slack bot and run Ask-Astro.
[x] Please outline the changes required to the slack bot for the Ask Astro channel on OSS airflow and create corresponding smaller issues if needed.

As we already have an ask-astro slack bot ready, the only thing required is asking one who has the permission to add the app


[FEATURE] Quick search list in taxonomy pages

For some reason, I did some rabbit hole browsing until I got lost and saw the individual tag pages of this website.
If you look at one of the tags page, you would see a quick list of search queries with the particular tag. It's pretty neat. The feature has to be data-driven thus customizable (using the data folder).
Here's a quick draft of how to customize it:
The data would be called as query and contains a list of objects with the following keys:

id - The identifier of the item. It's mandatory.
url - The search URL of a specific site. This has a special where it should contain a specific pattern where the name should appear. For now, this specific pattern will be ${{__DATA__}}.  Also mandatory.
name - The name of the search site. This is optional as the theme will use the id as the name instead, if absent.

This is now implemented in the development branch. Just waiting for the official release.


ACS - Video Call Switch camera not working

const cameras = await callClient.getDeviceManager().getCameras(); const camera = cameras[1]; localVideoStream.switchSource(camera);
When I follow this instruction, the camera only switches on locally. RemoteStream still uses the old source, which means:
I have camera _1 and camera_2,
When I start the call with localVideoStream and use the camera_1 => remote showing stream from camera_1
Then I call the switchSource() method to use the camera_2 => only the local video stream has been affected. Remote video stream still using the old one. Can anyone have any idea to help with this? Thank you.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: b2b51957-760c-c24c-cc33-b306771f5cf3
Version Independent ID: 158ae051-b329-1409-550d-3fb87a08cde3
Content: Manage video during calls - An Azure Communication Services how-to guide
Content Source: articles/communication-services/how-tos/calling-sdk/manage-video.md
Service: azure-communication-services
Sub-service: calling
GitHub Login: @tophpalmer
Microsoft Alias: chpalm

@dominica-pudding
Thanks for your feedback! We will investigate and update as appropriate.

@dominica-pudding Could you please reach out to us by sending an email with subject line “Attn:Sneha |thread title” to AzCommunity[at]microsoft[dot]com referencing this thread.
Since working offline We will close this out here and would share update accordingly.

@SnehaAgrawal-MSFT @dominica-pudding Any update on this?


Add more information in application review page

Description
Add more information to the application review page for a single applicant so landing directly on this page gives enough context to the program admin. Specifically, these are

program name
applicant name with id

Checklist

[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

(nit) can you add a screenshot?

(nit) can you add a screenshot?

Added.


Fixes cleanup issues

Fixes
#3707 test-jar in google-cloud-bom.
#3728 Don't bundle tests in jars.

@chingor13, do you have any issues with this?  @elharo, does this look right to you?


Being able to access the login page even if I'm already logged in

I can access the login page and login again, even though I'm already logged in. If I'm already logged in, I shouldn't be able to see the login page again, unless we implement some kind of security feature that forces users to login again given some set of circumstances. There should be an internal redirection to some other page if a user is logged in, and tries to access the login page

Should be fixed after integrating next-auth. We could've something like this in the login page:
getServerSideProps(({ req, res }) => {
  const session = await getSession({ req })

  if (!session) return res.end()

  return return {
      redirect: {
        permanent: false,
        destination: "/curriculum"
      }
    }
})

Closed in https://github.com/garageScript/c0d3-app/pull/1953


Mac module_test_ios is flaky (2.41%) due to FlutterUITests failures

2 most recent failures:
https://ci.chromium.org/ui/p/flutter/builders/prod/Mac module_test_ios/3264/overview
Error message:
FlutterUITests/FlutterUITests.m:91: error: -[FlutterUITests testDualCold] : (([app.staticTexts matchingType:XCUIElementTypeStaticText identifier:@"This is Marquee"].count) equal to (2)) failed: ("1") is not equal to ("2")

https://ci.chromium.org/ui/p/flutter/builders/prod/Mac module_test_ios/3254/overview
Error message:
FlutterUITests/FlutterUITests.m:52: error: -[FlutterUITests testFlutterViewWarm] : (([app.staticTexts[@"Button tapped 0 times."] waitForExistenceWithTimeout:60.0]) is true) failed

At least for the first case, it looks like the test found the native "This is Marquee" but didn't wait enough time for the Flutter view to show it.
  Assertions: Assertion Failure at FlutterUITests.m:91: (([app.staticTexts matchingType:XCUIElementTypeStaticText identifier:@"This is Marquee"].count) equal to (2)) failed: ("1") is not equal to ("2")
  File: FlutterUITests.m:91

I think in both cases elements are being verified before being awaited.  Assigning to me.

Ran into a similar timeout for testFlutterViewWarm on the engine roll recently: https://ci.chromium.org/ui/p/flutter/builders/try/Mac module_test_ios/8289/overview

@bdero That one looks similar to the second link, probably because the buttons existence isn't being awaited before a tap is attempted.  Hopefully https://github.com/flutter/flutter/pull/85650 reduces the flakiness. If you see it again can you file a new issue and @ me?

@jmagman will do!


Enable tests after storage server fixes the issue where sas access policy isn't effective after 30 seconds

both in datalake sas test and blob sas test

close in favor of https://github.com/Azure/azure-sdk-for-cpp/issues/3592


apiserver 403

FAILED - RETRYING: Master | wait for the apiserver to be running (1 retries left).
fatal: [node8]: FAILED! => {"attempts": 20, "changed": false, "connection": "close", "content": "{\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"forbidden: User \\\"kube-apiserver\\\" cannot get path \\\"/healthz\\\"\",\"reason\":\"Forbidden\",\"details\":{},\"code\":403}\n", "content_length": "190", "content_type": "application/json", "date": "Thu, 26 Apr 2018 15:08:22 GMT", "json": {"apiVersion": "v1", "code": 403, "details": {}, "kind": "Status", "message": "forbidden: User \"kube-apiserver\" cannot get path \"/healthz\"", "metadata": {}, "reason": "Forbidden", "status": "Failure"}, "msg": "Status code was 403 and not [200]: HTTP Error 403: Forbidden", "redirected": false, "status": 403, "url": "https://<IP_ADDRESS>:6443/healthz", "x_content_type_options": "nosniff"}
fatal: [node7]: FAILED! => {"attempts": 20, "changed": false, "content": "", "msg": "Status code was -1 and not [200]: Request failed: <urlopen error [Errno 111] Connection refused>", "redirected": false, "status": -1, "url": "https://<IP_ADDRESS>:6443/healthz"}
fatal: [node6]: FAILED! => {"attempts": 20, "changed": false, "content": "", "msg": "Status code was -1 and not [200]: Request failed: <urlopen error [Errno 111] Connection refused>", "redirected": false, "status": -1, "url": "https://<IP_ADDRESS>:6443/healthz"}

v2.4.0

incomplete issue report


Initial implementation

Sets up a basic rest service with dropwizard, liquibase migrations, mysql db support, guice, rosetta, jdbi, immutables and jackson
This is only fit to run locally since the db user and pass are in the config file i'm checking in here.  We'll need to supply those via command line args or something else
@nirrattner

Looks great!


My COVID Alerts in Settings: If Bluetooth or Exposure notifications are disabled the Turn On button appears but doesn't do anything

App version: <IP_ADDRESS>
OS: Android 11
Device:  Pixel 3 XL
Reproducible: Always
Summary: My COVID Alerts in Settings: If Bluetooth or Exposure notifications are disabled the Turn On button appears but doesn't do anything
Steps:

On Android disable Bluetooth OR Go to device settings and turn off Exposure notifications
Open app and go to Settings > My COVID Alerts - for ex notifications off you can also go to the COVID alert tab and try the button
Press the button "Turn On"
Results: Nothing happens
If BT is enabled the button changes to "Update Settings" and that button works fine

Note: Go to Settings button works from My COVID Alerts tab

Testing in <IP_ADDRESS>
Android -

Bluetooth off =

Update settings button seen in Settings > My COVID Alerts with message that Closeness Sensing is currently inactive Please turn on Bluetooth...
Button opens up device settings for Exposure Notifications and I can turn on Bluetooth there.
Once on and I go back to app the message shows active.
Go to Settings button seen in My COVID Alerts tab with message that Closeness Sensing is not active Please turn on Bluetooth...
Button opens up device settings. Search for Bluetooth there and turn on.
Once on and I go back to app the message shows active.

Exposure Notifications =

iOS double checked this is ok and it is

Testing in <IP_ADDRESS>
Android -

Bluetooth off =

Update settings button seen in Settings > My COVID Alerts with message that Closeness Sensing is currently inactive Please turn on Bluetooth...
Button opens up device settings for Exposure Notifications and I can turn on Bluetooth there.
Once on and I go back to app the message shows active.
Go to Settings button seen in My COVID Alerts tab with message that Closeness Sensing is not active Please turn on Bluetooth...
Button opens up device settings. Search for Bluetooth there and turn on.
Once on and I go back to app the message shows active.

Exposure Notifications =

iOS double checked this is ok and it is


Unpublished Horizen VPN

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/6efd26bca318f28/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/6efd26bca318f28/output.txt
Total script time: 1.98 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global


Moved history file to ~/.cache

A tiny edit for just not cluttering the home directory with a hidden file.

Now I realize it needs to first try to detect if one already exists at the previous location and move it if true


Remove trailing whitespace from Json

Using Entity Framework Core I created a POCO with a MaxLength(400) data annotation.
When the object is returned by a WebApi action method the field always gets serialized with a length of 400.
Although the relevant string is much shorter trailing whitespaces are appended.
Is there a way to provide EF with the necessary information for creating the field in the database and not have the json serializer append whitespaces ?

Please open this issue on the https://github.com/aspnet/EntityFramework repository and they may be able to help you. I'd suggest also including your model code (or a simplified version of it).

@rowanmiller @divega

Never mind. As it turns out that was neither the fault of serialization nor EF. Data was corrupted in database to start out with.
Sorry guys!


Rename? (Lace -> Fildesh)

Even though my https://github.com/grencez/lace repo has been around for a long time, its name isn't super unique. Due to the lack of stars, it should be safe to do a mass rename now.
Motivation: https://github.com/trolando/lace is a dependency of https://github.com/trolando/sylvan, which is a BDD library that I might want to use in another project.

Maybe "agleye" = "aglet" + "eyelet". That's 2 more characters than "lace". But it's very unique and fairly self-descriptive.
Can't really play around with that, or it gets to close to "agile" or "agley". Though, I suppose it would be fair to call a digraph of pipes either of those words.

Another idea is "lacet", which is a French word with basically the same meaning as "lace".

Another idea is "shlace", short for "lace in the shell" with an obvious shoelace pun. The ".sh" in a ".shlace" file extension might hint that the file is a script.
Con: Hard to pronounce.
Con: The first search result is an unpopular urban dictionary entry 😐😐😐.
Con: Doubling down on the shoe metaphor is kinda cringe.

Another idea is to not rename at all. Really just need a unique header. My symbol names are pretty long and probably wouldn't cause a conflict.

Another idea is "fildesh". It's somewhat common to abbreviate "file descriptor" as "fildes". The "sh" at the end would indicate a shell script.

It's effectively renamed.
Some internal functions in the builtin and compat libs still need renames though... Guess I shouldn't close this out until then.

Well that was a big time sink. Done now though!


check if 'custom_design_from' should be removed

I have seen that there is a category has an API field named custom_design_from.
Why is it still there but never used?
Should we remove it?
see: https://github.com/DivanteLtd/vue-storefront/search?q=custom_design_from

same for custom_design_to

I have seen that there is a feature in Magento to define a date range for a custom design. But is it then correct to have this in the default configuration file, even if you don't use Magento?


Error performing request: Get https://api.local.pcfdev.io/v2/info: Service unavailable

Hi over the weekend my system got restarted and morning when i tried to log in to apps manager it was not up. Then i went to power shell window, did a cf dev destroy and then did cf dev start. After all the 56 services start it asked to login and when i tried : cf login -a https://api.local.pcfdev.io --skip-ssl-validation , it is giving this error:
Error performing request: Get https://api.local.pcfdev.io/v2/info: Service unavailable
where as i am able to log in to apps manager through browser. Any reason why it is saying service unavailable ??

Can you provide the full output of cf dev start?

you mean power shell window screen shot ? if so i am sorry i just closed the window.  Do you want me to try cf login command again ?

I opened new power shell window and ran cf login command and i was able to login to https://api.local.npcfdev.io.


Current version 1.1.1 doesn't work

Adb logcat shows many Not (yet) bound message when Apple Wifi backend is chosen. Could somebody fix this please? Thanks!

Have you tried the latest version?


Testing and linting

Added a substitute for score image generation until a better solution for importing lilypond can be found.

Added testing (unit tests and integration tests), along with the infrastructure to run this tests via the makefile.
Running "make coverage" runs the tests and generates coverage statistics.


Fixed doc: finalize changed to add_finalize

Sorry, I'm just nitpicking here but I thought I would fix it anyway.

Thanks!