Memory leak in Abourget fork

Hi everyone,
Not sure if this is the right place to post this but github does not allow me to submit issues to a fork.
There appears to be a memory leak in the abourget version, specifically relating to the following code at ctx.go (https://github.com/abourget/goproxy/blob/master/ctx.go):

  clientTlsReader := bufio.NewReader(rawClientTls)
  for !isEof(clientTlsReader) {
  	// This reads a normal "GET / HTTP/1.1" request from the tunnel, as it thinks its
  	// talking directly to the server now, not to a proxy.
  	subReq, err := http.ReadRequest(clientTlsReader)

What happens here is that various sites do not properly close the TLS handshake and so the for loop continues on indefinitely. I've placed a timeout of 10 seconds on the loop, which helps a great deal. However, some connections cause the ReadRequest() to hang indefinitely. There is actually a comment above this section of code which describes the same problem.
I'm not sure why this section of code is in a for loop. Seems that a simple if...then statement would be a better way of handling this. And if anyone has any ideas of how to properly fix this, please let me know. Otherwise, I'll probably just figure out a way to timeout the connection and kill it.

Looks like this has already been fixed in the elazarl version:
https://github.com/elazarl/goproxy/issues/209
Setting the header in ctx.go:forwardMITMResponse() seems to do the trick!
Guess this begs the question... are there still any plans for you guys to merge your versions?


Support Atlantis for URLSessionUploadTask

Acceptance Criteria

Able to capture all Upload Requests with body data
Work fine with URLSessionUploadTask
Work with Alamofire upload data or multiple-part

PR #49

PR #49


Handle parsing errors/warnings

Currently, exceptions in code formatting plugins are silently ignored.
I think this is a bit "dangerous" in that, if a plugin is faulty, people won't realise that their files are not being formatted as intended.
At a minimum, I think the errors should be logged, as I have added in #36.
But perhaps also it should be the default or an option (using #35) to fail on such exceptions.
Another place where reporting/failing should be considered, is when duplicate_refs are found in the env (after parsing)

I was also unsure (and still am) what the best default here is. I made mdformat swallow code formatter exceptions for now, since I feel like it might be quite common to write broken code in Markdown code blocks, e.g.
here's broken json in a markdown code block
```json
{
  "item1": 1,
  "item2": 2,
  ...
}
```

I'm not sure if this assumption is valid though, and for my own use, I'd be happy with mdformat returning a non-zero error code if a code formatter raises. If there's two schools of thought here I'd happily merge a CLI arg to change this behavior.

At a minimum, I think the errors should be logged

Agreed.

Another place where reporting/failing should be considered, is when duplicate_refs are found in the env (after parsing)

What do you mean by duplicate_refs? Is this something that happens (and should error) in markdown-it-py perhaps?

yeh good point about writing broken code, I think then failure should be optional via a CLI arg 👍

What do you mean by duplicate_refs?

See https://github.com/executablebooks/markdown-it-py/blob/c3247296415c69cb3083376a78c56e4da8249815/markdown_it/rules_block/reference.py#L197, if you write:
[a]: href1
[a]: href2

The 2nd one will be stored in duplicate_refs.
It is the renderers job to decide if it/how they want to act on these duplicates (for example in myst-parser a warning is sent to the Sphinx error reporter)

Ah gotcha.
I'm now thinking since mdformat is a formatter, not a validator, if there is a problem that can't be fixed by formatting, then the exit code should probably by default always be success (this goes for "invalid" code block contents). We could consider printing a warning to stderr though.
When it comes to duplicate_refs, if i understand correctly, that is something that mdformat can fix, by simply only rendering the first ref (with the assumption that rendering ref links is supported, currently isn't). So I feel like no warnings or anything should be needed.

Yeh I agree these things should probably default to success, but it would be nice to (optionally) get some kind of warning.
maybe make this a bit less "hardcoded" than directly printing to stderr, by doing something like:
        try:
            code_block = fmt_func(code_block, info_str)
        except Exception:
            # Swallow exceptions so that formatter errors (e.g. due to
            # invalid code) do not crash mdformat.
            options["mdformat_logger"].warn("some message")

this is probably not the ideal solution, but you get the idea


update readme file

say something

go on

i want  finish my test

I  did not find the merge button.....


Handle lost webgl contexts more gracefully

When a webgl context is lost, an error is thrown that is not really relevant.

Brought up on the forum here: https://groups.google.com/forum/#!topic/cesium-dev/406L37WfjWs
It looks like the lost context can be specifically checked: https://www.khronos.org/webgl/wiki/HandlingContextLost

Cesium is unlikely to fully restore the state as I suspect potentially doing it in the application is the right level of abstraction, e.g., an app can easily reload a CZML file compared to Cesium tracking and recreating all the renderer/primitive/entity state.
However, we may expose this event if it is widely requested.
Most engines do not implement this AFAIK.

SketchFab shows a friendly dialog box saying something along the lines of "Sorry, but it appears your browser has had a problem running WebGL.  Please reload this page to restore functionality." and there's a RELOAD button to click.  I realize this is an app-level concern but maybe we could expose some kind of helper for this.

I recognize this isn't absolutely universal but I suspect most consuming "apps" use a Viewer (or a directly-managed CesiumWidget) as a cornerstone of the page being viewed.  Since the app is likely to be more or less useless with a crashed WebGL, you could show the default error handler with a message along the lines of Ed's suggestions ("Something has gone wrong with your browser's 3D graphics; reloading this page will likely fix the problem.", etc).
That said, if it's a  Viewer, I would naively expect it to be possible to destroy the underlying Widget and re-populate it by interrogating viewer.dataSources.  That would be far from guaranteed to get back to previous state, but it might be a better user experience than "please click the refresh button".

Hi! We're running into this problem as well. I'd like to catch the webgl context lost event as well. I don't need Cesium to gracefully recover of anything, just something that we get control back and I can reboot cesium would already help a lot.
Also, what kind of thing causes webgl contexts to be lost? It almost never happens to me on my MacBook, nor on my colleagues laptop, but our business development guy that does product demo's frequently has them (as you can imagine this is awkward for us) on his surface book. I tried reducing the GPU memory limit, to 256, and that didn't work. We're working with complex models with high resolution textures.

I'm developing my (Cesium-based) application on a Surface Book 2 as well.  It has a kind of unique 3D graphics situation where the dedicated GPU is in the base and the onboard GPU is part of the tablet/display, so it has to be able to handle seamlessly going between the two.  I could definitely imagine a scenario where a glitch in the handoff causes the browser to try and fail to transfer the WebGL context from one to the other, if that makes sense.
I was going to bring up a related issue, about how much detail Cesium is getting from the browser, but when I went back to check the one I was thinking of is actually about initializing the context, not losing it.  Still, might be relevant?

Oh jeez you gotta be kidding me. What are the odds that our demo guy is using specifically the unique kind of laptop that might have an issue with creating WebGL contexts.
Maybe I can workaround by when our app is loaded allocate a WebGL context myself, so there's always at least one running and we don't care if that one crashes..

A given canvas can only have one (3D) context, and I don't think we have any control over how Cesium manages it.  And if there is some kind of rare driver glitch that's causing the context to fail, there's no reason to think it wouldn't cause all active WebGL contexts to fail at once.  I think the best thing for it is to set up reasonable error handlers and make the restart process as smooth as possible.  If it's happening to you a lot, maybe try a different browser, or capture some debugging information and see if you can report the issue to the browser team.  I've had this happen before, but it's certainly not common for me.

Ran into this pretty hard today.  Some folks here planned a demo that involves about 20 different Cesium visualizations to be pre-loaded onto various tabs, with cameras placed here and there.  Similar demos were frequently given in 2017 and 2018 without issue, but modern versions of Chrome and Firefox are much more aggressive about proactively losing context when it looks like many tabs are hogging up GPU resources.  Typically, all Cesium tabs will crash in unison after some idle time.
The problem with simply reloading the page is that it loses one's place.  Camera view angle is lost, tracked entity is lost, data source must be re-requested from server, etc.
The workaround for this week's demo is to split up the visualization load across multiple browsers, and possibly across multiple PCs.  But longer term, it would be fantastic for Cesium to be able to recover from context loss without losing things like the entity collection and camera position.

Also requested in https://github.com/CesiumGS/cesium/issues/11533


Source IP Adress is always <IP_ADDRESS>

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

A clear and concise description of what the bug is, including last known working version (if any).
All packets sent by the OPNsense itself through the LAN interface are sent with the source IP <IP_ADDRESS> instead of the LAN interface IP. The error only affects packets sent by the OPNsense itself everything else works.
We suspect that the error occurred when upgrading from version 22.4.3 to version 22.10. But we are not 100% sure, because we cannot revert from 22.10 to 22.4. (If this should be possible after all please an info about it).
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:

Go to Firewall->Log Files->Live View
Set "Source is <IP_ADDRESS>".
Open a new tab
Go to "Interfaces->Diagnostics->Ping
Ping a device in LAN
the ping is sent with the source IP <IP_ADDRESS> (Screenshot)

You can also test any other request sent directly from the OPNsense. The source IP is always <IP_ADDRESS>
If an address outside the LAN is pinged, for example <IP_ADDRESS>, the WAN address configured on the OPNsense is used as the source IP.
Expected behavior
All packets sent directly from the OPNsense to the LAN get the IP of the LAN interface as source.
Alternatives
We have not found any alternatives or workarounds yet.
If applicable, information from log files supporting your claim.
Additional context
The reason why this error is a problem at all is that the OPNsense cannot connect to the server via LDAP to authenticate users for the OpenVPN.
Environment
OPNsense DEC750EU 22.10-amd64
FreeBSD 13.1-RELEASE-p5
OpenSSL 1.1.1q 5 Jul 2022
AMD Ryzen Embedded V1500B (4 cores, 8 threads)

duplicate https://github.com/opnsense/core/issues/6036 let's keep this in the same ticket, closing this one. See my last comment about the routing table (netstat -nr) in https://github.com/opnsense/core/issues/6036#issuecomment-1349476393

I apologize, but the two problems are not identical. The problem in the linked ticket is about the WAN and not about the LAN as described in our case. We have already looked at the linked ticket before creating our own. We had already tested the suggested solutions as far as we could and unfortunately it didn't work.

They sound similar, but I don't mind keeping this open as well. Same question as in the other ticket maybe relevant here as well (routing table).

I took a closer look at the routes and noticed a similar phenomenon. Unfortunately, I don't have a before and after comparison but the following route is very similar to the route added to AdSchellvi's routing table:
Internet:
Destination        Gateway            Flags     Netif Expire
xxx.xxx.xxx.0/24    link#1             U          igb0
xxx.xxx.xxx.1       link#1             UHS         lo0
xxx.xxx.xxx.10      link#1             UHS        igb0  <--
My question would be whether I can simply delete this route, since it overlaps with the other route.

@nuv-it if possible, I would try to remove the entry and check if it makes a difference. Is this interface configured via dhcp by the way?

I had a similar case with @mimugmail yesterday, was able to reproduce and seemed to be caused by default gateway switching. @fichtner added a patch in https://github.com/opnsense/core/commit/0e286b3a34cf366efa88a2616e1e1a0c22b8180c, if the issue returns after 23.1.1 we can always reopen this ticket.


Drum notations are not supported

Most drum notation software use MusicXML as output format, it would be nice if vexflow could handle drum glyphs.

If drum glyphs are added to the upstream VexFlow, I could add MusicXML parsing support for them. As far as I know, they are not currently available. If you don't mind, please open an issue at https://github.com/0xfe/vexflow.

I've just added percussion clef support to the MusicXML parser. I think the remaining steps are to add the "x" note glyph and any other custom glyphs, and then I will also need to add support for those in MusicXML.

Actually, I took a look at Vexflow source code, and percussions are partially handled. You can find some example in the /test/test_percussions.js file. Yet, they don't seam to be implemented in VexTab, hence my confusion. Moreover, the chosen symbols are not standard.


Consider switching from unittestpp to Catch?

See here: https://github.com/philsquared/Catch
It's header-only so probably doesn't require all the complicated build system support that UnitTest++ does...

Recommended as alternative to Catch: https://github.com/onqtam/doctest

...whoa. Thanks for linking doctest! I've been using Catch because it was the quickest to integrate, but the compile time cost was atrocious, this is a huge help!

Btw, for my emulator stuff and the sokol headers I'm now using https://github.com/sheredom/utest.h, it's also super-easy to integrate since it's just a single C header (not C++).

Even better, thanks! :)


Add MQTT Broadcasting

Hey Alex,
first of all thank you very much for the MaraX timer. I always wanted to tinker with the ESP8266 platform but never found a really useful project. By accident I discovered that the MaraX is a great hackable machine and was very surprised to see a project like this out in the wilds :)
I refactored the code to make it more extensible and added an MQTT broker which is totally optional but plays well with gathering data. Also I've changed the display power on/off logic to sync with the machine state.
If you like the contribution but see some issues in the code base (this my my very first project with the Arduino IDE) feel free to request changes, otherwise simply close this PR :)

@alexander-heimbuch I think will leave the project as is for now, but will consider adding this in the future. Thanks.


[Bug Report] When there is a fixed column in the table and the table in the expanded row sets max-height, it cannot slide properly.

Element UI version
0.1.0
OS/Browsers version
68.0.3440.106
Vue version
2.6.6
Reproduction Link
https://jsfiddle.net/eb4o0h7w/
Steps to reproduce
展开第一行数据，点击展开行下面的表格右侧的下拉条
What is Expected?
展开行中table的所有数据随下拉条滑动而滑动
What is actually happening?
展开行中table的商品ID一栏并没有随下拉条进行滑动

Sorry, we have no plan to support this.


v0.6.0 release breaks Tour of Heroes tutorial

I apologize if I missed something along the ways as this is my first venture into JavaScript let alone AngularJS.
I am following along with the Angular Tour of Heroes tutorial, and installed this package as per the instructions:
npm install angular-in-memory-web-api --save

All my calls to getHeroes() returned no data and the error text was undefined.
When I changed my package.json to reference ^0.5.4 instead of ^0.6.0, everything worked as expected.  I looked at the release for v0.6.0 and it appears to be for Angular 6 while I am using Angular 5.  I suspect that because the release was only three days ago that most folks will not have hit this issue.
While this release appears to have broken the tutorial, I do not believe it is this package's fault.  That being said, someone needs to fix something. :-)  If this issue belongs somewhere else, please let me know and I will file it there.  Thanks!

I've just run into this too. Seems like the tutorial should state which version of angular-in-memory-web-api should be installed based on the installed version of Angular / rxjs.
You could try raising the issue over on the Angular repo to get this part of the documentation updated.

Same here...

see #190

i had same problem in hero tutorial,  but i have installed version 0.5.0 and it works
npm install<EMAIL_ADDRESS>

Yes, that works but doesn't address the problem.
On Fri, Mar 30, 2018 at 5:12 PM, Luis Nido<EMAIL_ADDRESS>wrote:

i had same problem in hero tutorial, but i have installed version 0.5.0
and it works
npm install<EMAIL_ADDRESS>—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/in-memory-web-api/issues/189#issuecomment-377650264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADo_RYcE_oDKVwufoyLITCUATiKjnrmks5tjsoHgaJpZM4S6Qg9
.

The 0.5.0 version doesn't works with the Angular CLI 1.7.3 and Compiler CLI 5.2.0 - but the 0.5.4 version does ...
npm install<EMAIL_ADDRESS>--save

I should have checked this sooner!
With the v0.6.0 I get a 500 error, "object (...) is not a function".
modify package.json to
"angular-in-memory-web-api": "~0.5.0",
and npm install to fix for now
Does anyone know how we can temporarily fix the heroes tutorial?
https://angular.io/tutorial/toh-pt6#enable-http-services
NOTE: the above command results in 0.5.4 being installed, which works!

it works fine with 0.5.3

We know it works with 0.5.4, does anyone know why it doesn't work with 0.6.0?

@healyje I believe it does not work because there were significant changes between Angular v5 and Angular v6, and that some of those changes impacted this API, and that the API got updated before the parts of Angular used in the tutorial.

I am new to angular and started with Tour of Heroes tutorial. Now I am facing same issue with angular 6 version. I am using all the packages with updated version and still "angular-in-memory-web-api" is not working for 0.6.0 , I can't use 0.5.4 since my CLI and Angular are v6.
Any work around to continue with tutorial?

This is really bad form guys, this is the first impression of Angular, you´ve known about it for months and haven´t fixed the problem. First time users of Angular (me) hit this straight away.

I ran into this issue trying to learn Angular with the TOH tutorial. I couldn't get npm install angular-in-memory-web-api --save to work. Should I try to update the dependency packages with patch 6.0.5?

I'm running into the same issue with Angular version 5.2.11 on CLI 1.7.4


Make it possible to provide extra attributes to Measurements payload

Description
The measurement API is missing the option to add attributes/context data like the error, logorevent` APIs are doing.
User often need a possibility to transport extra information within the respective Meta.
Proposed solution
Add an option to pushMeasurement API to extra data.
Context
Community Slack
### Tasks
- [x] Add functionallity to Web-SDK
- [x] Add functionallity to the Reciever
- [ ] Add functionallity to the Agent
- [x] Update Faro docs

Web-SDK PR
Faro Receiver PR
Grafana Website PR (docs)
Grafana Agent: TODO


chore: rename Module, Submodule

Since Module has a somewhat special meaning in Go due to the go mod dependency management system, the Module class is being renamed to BasePackage, and Submodule to InternalPackage. This reserves the Module name for code related to dependency management.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: dc3d70fc6b52ff635f50e6f56778bc27d978321c
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Fixed #24424 - sqlite schema editor causes SQL syntax error

Replaces pull request #4223

Closing per comment on the ticket: https://code.djangoproject.com/ticket/24424#comment:11

@MarkusH in your comment to the ticket you mentioned that "the fundamental problem is not just the SQLite backend, but also the autodetector that generates models without any fields".
I think this is not quite right, because the the case where I discovered this bug only happened with the SQLite schema editor. Of course, under normal circumstances it would be absurd to have a model with zero fields (not even an id field), but there is an edge case where this can happen: when a model "A" extends an abstract model "B" and is modified to extend abstract model "C". There may be other cases, the bug seems to have been first reported 5 years ago. This change only seems to break with the SQLite schema editor because SQLite has limited "ALTER TABLE" support and works around this by copying data between temporary tables. The empty SQL table is not the final destination, rather a midway step in a supposedly transactional migration. This midway step would not be manifested with MySQL, Postgres etc.
Essentially I believe the bug is in the function django.db.backends.sqlite3.schema._remake_table() because:

The list model._meta.local_fields may be empty for various reasons (L51)
Therefore mapping may be empty (L54)
Therefore field_maps may be empty (L139)
Therefore invalid SQL may be executed.

Therefore I would appreciate you reopen this PR and merge.
Thanks and regards,
Adam
(CC @timgraham)

Hey @adnam , the error doesn't only manifest on SQLite. When you run the test on MySQL it fails with
Traceback (most recent call last):
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/django/db/backends/mysql/base.py", line 124, in execute
    return self.cursor.execute(query, args)
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/tests/.env/lib/python3.3/site-packages/MySQLdb/cursors.py", line 219, in execute
    self.errorhandler(self, exc, value)
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/tests/.env/lib/python3.3/site-packages/MySQLdb/connections.py", line 38, in defaulterrorhandler
    raise errorvalue
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/tests/.env/lib/python3.3/site-packages/MySQLdb/cursors.py", line 205, in execute
    r = self._query(query)
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/tests/.env/lib/python3.3/site-packages/MySQLdb/cursors.py", line 372, in _query
    rowcount = self._do_query(q)
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/tests/.env/lib/python3.3/site-packages/MySQLdb/cursors.py", line 336, in _do_query
    db.query(q)
  File "/home/jenkins/workspace/pull-requests-trusty/database/mysql/label/trusty-pr/python/python3.3/tests/.env/lib/python3.3/site-packages/MySQLdb/connections.py", line 282, in query
    _mysql.connection.query(self, query)
_mysql_exceptions.OperationalError: (1090, "You can't delete all columns with ALTER TABLE; use DROP TABLE instead")

Hi @MarkusH, which test was that? The test I wrote has a decorator to ensure it only runs against sqlite.
The point I'm trying to make is that the empty-model problem does not manifest itself with MySQL and Postgres migrations, because these databases have greater ALTER TABLE support than sqlite; hence the test only concerns the sqlite db backend.

I'm referring to 235a8394734b63a83e9b709bc6696a66e39eb2b9 in #4223: https://github.com/adnam/django/blob/235a8394734b63a83e9b709bc6696a66e39eb2b9/tests/schema/tests.py#L1375
The test shouldn't behave differently on any backend thus I'm -1 on the @skipUnless decorator. Side note: PG behaves as expected and ends up with a table without columns.

Yes, strangely postgres can have tables without columns ... it must be a new database paradigm called "no column store" \o/ ^_^
I can only emphasise that the "empty table" state is a midway point of an sqlite migration, not the intended final result. Perhaps I could add a comment explaining this to the unit test, or find somewhere else for it to live? I honestly believe the real bug is in _remake_table() as I mentioned above.


Support slash in GitHub branch name

This would previously error
Also updated bootstrapper to version including my fix for group policy restrictions

The only thing is that a test for this function might make sense (we can make it internal and use InternalsVisibleTo if needed)

Agreed, is there an already existing test fixture?
Also looks like I broke something as both CI gone red 😞

@matthid turns out there is tests (just not for this case) and i broke them :(
Adding my new scenario into that test case and will ensure all tests pass!

@matthid updates made, should be ready to go now...

thx


Refactored calculate_constraint_residual_gradient.hpp to pass Clang-Tidy

This code refactors the CalculateConstraintResidualGradient kernel to pass Clang-Tidy checks.  It does this, in part, by refactoring it into one function for each type of constraint.  Each function is free from conditionals, so it should be much easier to understand what's going on for any given constraint, and it should be a little more performant on GPUs.  It also doesn't allocate stack memory for auxiliary variables that aren't used for that constraint.

Looks good -- just curious about the scope of the clang-tidy checks, Is it not acting on the whole codebase?

Looks good -- just curious about the scope of the clang-tidy checks, Is it not acting on the whole codebase?

@faisal-bhuiyan
Before this commit, it was only acting on .cpp files.  This commit lets it look at .hpp files, too (That's the change in the .clang-tidy file).  It was an oversight of mine when I was first setting up these checks however many months ago, but it also made a good way to divide up the work of fixing the warnings.


should we maintain an object "type" ?

See indexd ...
https://github.com/LabAdvComp/indexd/blob/master/indexd/index/schema.py#L13-L18
    "form": {
      "enum": [
        "object",
        "container",
        "multipart"
      ]
    }

Since we have the bundle concept we don't need container. Multipart can probably be a metadata tag, so perhaps not.


Confusion about how the KEGG edges were filtered

Hi!
In the paper, you mentioned:

KEGG contains 290 pathways, and Reactome contains 1,581
pathways. For both, we only select directed edges with either activation or
inhibition edge types and filter out cyclic gene pairs where genes regulate
each other mutually (to allow for a unique label for each pair). In total, we
have 3,057 proteins with outgoing directed edges in KEGG, and the total
number of directed edges is 33,127. For Reactome, the corresponding
numbers are 2,519 and 33,641.

What I am not clear is if you removed all the cyclic gene pairs (A->B, and B->A), or just kept one direction. So for example, if you have A->B and B->A,  you kept A->B but not B->A, or you removed both of them directly (?)
Also, if instead of predicting regulatory pairs, I only want to predict the pairs in the same pathway no matter their causality, then can I keep other KEGG edge types that are not activation or inhibition?
Thanks in advance!!

Hi，
You are welcome.
The training and test data generation is very flexible, which depends on what you want the model to learn.

In the original paper, we removed all cyclic gene pairs.
I think you can try it, and just keep in mind that the training and test should follow the same standard.


Include StockKit log file in exceptions

I've added a custom Exception class, StochKitSimulatorException.
StochKit often has STDOUT messages that include something like:
Check log file "compile-log.txt" for error messages

The new Exception class parses the log message with a regex,
and attempts to read the log file for inclusion in the error
message. This is helpful since the log file and other temp
files are cleanup up by default.

Coverage decreased (-1.0%) to 77.816% when pulling 923aeb8f7e960752ba39873ab528ab31570db552 on alubbock:stochkit_logfile into b6ba27d45eb17b8fbb239f9b592d529e107e4815 on pysb:master.


How to use android afreerdp  connect to windows without password?

Dear ALL,
I want to use android afreerdp connect to windows ,but this computer just have username,no password.
If I connect it,afreerdp alert a dialog "Please enter your credentials" .How to ignore this dialog,and connect my computer.
My English is very pool,but I need your help!
Thanks very much!

I build aFreeRDP version is stable-1.1 ,Thanks!

I found the reason in myself.when you add connection,shuld set Advanced--->Security--->RDP


Documentation and Bugfix

Please also review the wiki page!
You can edit the wiki page on github directly, so feel free to change anything that you see, or reply here if you have questions or concerns.

[Tutorial] Post‐Login is verified and updated after testing.


Add Prime95 v28.9

Checklist

[x] The commit message includes the cask’s name and version.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Additionally, when adding a new cask:

[x] Checked there are no open pull requests for the same cask.
[x] Checked there are no closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

@vitorgalvao I think I've corrected all the issues, sorry about that!

You did. Thank you.


Add promql/regexp check

Codecov Report

Merging #147 (caac98e) into main (dcf8585) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
+ Coverage   92.18%   92.27%   +0.09%
==========================================
  Files          62       63       +1
  Lines        3965     4012      +47
==========================================
+ Hits         3655     3702      +47
  Misses        239      239
  Partials       71       71

Impacted Files
Coverage Δ

internal/checks/base.go
100.00% <ø> (ø)

internal/checks/promql_regexp.go
100.00% <100.00%> (ø)

internal/config/config.go
100.00% <100.00%> (ø)

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 dcf8585...caac98e. Read the comment docs.


Bdb

Break Loop Cycles into something that can be easily manipulated using json from a script perspective:
freeapsx_stats=# select statistics_json->'Build_Version' AS "Build Version",        statistics_json->'iPhone' AS "iPhone",        statistics_json->'iOS' AS "iOS",        statistics_json->'LoopStats'->0->'no_of_loops' AS "# of Loops",        statistics_json->'LoopStats'->0->'success_rate' AS "Success Rate"   from statistics.statistics;  Build Version |    iPhone    |   iOS    | # of Loops | Success Rate ---------------+--------------+----------+------------+--------------  "0.3.3"       |              |          |            |  "0.3.7"       | "iPhone12,1" | "16.1.1" |            |  "0.3.8"       | "iPhone15,2" | "16.1.1" | 252        | 98  "0.3.4"       |              |          |            |  "0.3.4"       |              |          |            |  "0.3.3"       |              |          |            |  "0.3.4"       |              |          |            |  "0.3.4"       |              |          | 175        | 96 (8 rows)
I didn't remove the 'text version' of Loop Cycles ...its the same data, but the sub-json makes it more machine friendly to read ...

Not sure how this works, but this incorporates the changes discussed in term of labels for the json

Add code to map iPhone15,2 -> iPhone 14 Pro ( and others that I've accumulated )

Add some more iPhone's to the list, and include processor type

This batch of changes includes extending loopstats.json to include start (createdAt ), end and duration of loop, as well as updating the code for DailyStats involved with pulling this in.
No changes to statistics.conf was made, just the values that are put out to it


Can we include Node JS binary files in our project?

If we include them, where do we put them to?
If not, what is our fallback method? (e.g. download via script)

@JayChl is working on this issue. He is implementing NodeJS Downloader via bash script with wget.

Ah, it is duplicated with #19. So, I'll close this issue.


How to open the App Groups function

Fastlane is very good tool.But I create appId without App Groups.Please see below:

Please tell me how to add this function.Thanks

Key
Value

OS
10.11.6

Ruby
2.4.0

Bundler?
false

Git
git version 2.10.1 (Apple Git-78)

Installation Source
~/.rvm/rubies/ruby-2.4.0/bin/fastlane

Host
Mac OS X 10.11.6 (15G31)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.4.0/lib

OpenSSL Version
OpenSSL 1.1.0e  16 Feb 2017

Is contained
false

Is homebrew
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.2.1

System Locale

Variable
Value

LANG
zh_CN.UTF-8
✅

LC_ALL

LANGUAGE

https://github.com/fastlane/fastlane/blob/master/produce/README.md
Enabling / Disabling Application Services
If you want to enable Application Services for an App ID (HomeKit and HealthKit in this example):
fastlane produce enable_services --homekit --healthkit

If you want to disable Application Services for an App ID (iCloud in this case):
fastlane produce disable_services --icloud

If you want to create a new App Group:
fastlane produce group -g group.krausefx -n "Example App Group"

If you want to associate an app with an App Group:
fastlane produce associate_group -a com.krausefx.app group.krausefx

Thanks for the help @VolkerSchiewe, that should help 👍

how can we use fastlane produce associate_group -a com.krausefx.app group.krausefx with fastlane lane?

No, you'll have to use sh "fastlane produce associate_group .." to shell out


Why DisplayObject or Container doesn't have anchor property?

Sometimes I need to have a container centered, while do not want to use specific width/height value.
But I find that PIXI#Container (or DisplayObject in fact) doesn't have anchor, so how can i make it centered without width/height ?
or, is it possible to write a .setPosition(w, h) method, whose w/h ranges between 0-1 ?

"anchor" is a concept for textures only; Containers and Display Objects do not have textures so do not have this property. Instead, you can use the property "pivot", which works on pixel values rather than percentages.
Example: https://pixijs.github.io/examples/#/basics/container-pivot.js

@themoonrat Thank you! I've solved my problem!

since many people ask about this question, look at the code below :
class MyContainer extends PIXI.Container {
    _anchorX = 0;
    _anchorY = 0;

    set anchorX(value) {
        this._anchorX = value;
        this.pivot.x = value * this.width / this.scale.x;
    }

    get anchorX() {
        return this._anchorX;
    }

    set anchorY(value) {
        this._anchorY = value;
        this.pivot.y = value * this.height / this.scale.y;
    }

    get anchorY(){
        return this._anchorY;
    }

}

Then the only thing you need to do is to is to update it the pivot everytime you change the affective ones (width, height, scale.x, scale.y) you can do so by
myContainer.anchorX = myContainer.anchorX;
or you can add something like that inside MyContainer class:
refershAnchors(){
   this.anchorX = this.anchorX;
   this.anchorY = this.anchorY;
}

I personally override the setters and getters for height, width, scale.x, scale.y inside MyContainer class

@msqaddura partial solution, we cant include it into library but of course I'm all hands for that kind of hacks if people want to use them.

@ivanpopelyshev  I can't but agree with you, the solution is partial and hacky (if we consider the given code)
the idea is just shed the light on how to calculate the the pivot on 0..1 basis, like an anchor. and that we have to do this.pivot.x = value * this.width / this.scale.x; i will refer to it as snippetX
By my side, I am using twin design pattern between my class and pixi classes (DisplayObjects) hence writing the code i use wont help us here. However, I will explain how to port the functionality to Pixi.
Let us first keep in mind these rules:

width does not change, instead it scales by the factor of difference between current width and new width, same applies to height, in code below (plus my modification) :

    set width(value) // eslint-disable-line require-jsdoc
    {
        const width = this.getLocalBounds().width;

        if (width !== 0)
        {
            this.scale.x = value / width;
        }
        else
        {
            this.scale.x = 1;
        }

        this._width = value;
        //we add this line here because we alread know that scale.x is changed by now
        // snippetX
        this.pivot.x = this.anchor.x * this.width / this.scale.x;
    }

scale also changes if developer wants to scale a DisplayObject implicitely. Since it is a Point (if am not wrong) , we can use  ObservablePoint instead and in the callback we call snippetX.

now as well, anchor itself might change so we need to apply the same snippetX to anchor.x (ObservablePoint?)

backward compatibility ;( if someone is using the pivot in his code already, and we introduce this change then pivot will conflict with anchor. unless then we put a flag to use pivot XOR anchor

That is why in the first place i did not want to explain it in details, i believe it is just better to shed light upon pivot=anchor*width/scale. In my case I completely disabled pivot and this is how my code looks like
    class Entity {
    //only code related to this question
   _anchorX = 0;
    constructor(){
        this.pixiTwin = new Pixi.Container();
    }
    get anchorX(){
        return this._anchorX;
    }

    set anchorX(value){
        this._anchorX = value;
        this.pixiTwin.pivot.x = this.pixiTwin.width/this.pixiTwin.scale.x * value;
    }
}

Illl just reference your post in pixi wiki under "Gotchas" :) I like those little snippets.

Done, look at https://github.com/pixijs/pixi.js/wiki/v4-Gotchas

Thanks! :)
I hope this will other developers get around this question


in PDF.js , window.getSelection().getRangeAt(0).getClientRects() is coming always 0 in IE

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version: IE 11
Operating system and its version: Windows
PDF.js version: 2.0.7XX
Is a browser extension:

What is the expected behavior? (add screenshot)
I need to get the highlighted/selected text from Angular
What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Duplicate of #5513.


: No such file or directory

Hi,
Why I can't call the binary outside of node_modules?
root@localhost:/home/API# ./node_modules/htpasswd/bin/htpasswd -c myfile myuser
: No such file or directory

root@localhost:/home/API/node_modules/htpasswd/bin# htpasswd -c myfile myuser
New password:
Re-type new password:
Adding password for user myuser

It is happening the same if I install it globally (-g).
I'm using CentOS 7.
Regards.

Just use htpasswd -c myfile myuser from both locations without ./node_modules/htpasswd/bin/ prefix.
Let me know if it works.

It only works if I change the file htpasswd from Windows format (CRLF) to Linux format (LF).
Also, I think the shebang "#!/usr/bin/env node" will not work in debian-like systems (it should be nodejs instead of node).

Thanks for letting me know, formatting was obviously wrong, I have this problem again and again :(
For shebang, I don't know proper solution that works on all platforms, but I know that is common practice for node developers to create symbolic link node and point it to nodejs.

Now it is working. It would be great if you change the shebang according to the operative system. Thanks.

Yeah, I just added tests also so hopefully it will never happen again, unfortunately I don't know any good way of changing shebang :(


Upgrade default PMD version to 7

Expected Behavior
Currently the PMD plugin uses the default PMD version 6.55.0 (from February 2023)
https://github.com/gradle/gradle/blob/2d97893cdd41ab870c6ec8a7753a90c5ee7d9abe/platforms/jvm/code-quality/src/main/java/org/gradle/api/plugins/quality/PmdPlugin.java#L65
This version is outdated and it e.g. fails with Java 21 code.
It should be updated to version 7.
Context
While upgrading an IntelliJ plugin to Java 21 I noticed that all PMD checks started to fail with "Unsupported class file major version 65" that's when I found out that PMD version 6 is used by default.
See also https://github.com/gradle/gradle/issues/24502

This feature request is in the backlog of the relevant team and is prioritized by them.

It's unlikely that we will update it in 8.x since it's technically a potential breaking change (because it's a major version switch). But we should do it in 9.0.


SlickJS not working within a Polymer element

I had it working with Polymer 0.5, but it has broken completely on the migration of my app to polymer 1.0.
It seems to replicate an instance of the generic non-computed element for some reason.
For example, if I have an element that is

{{ text }}
,
every element will literally be

instead of something like

Hello!
.

You may need to wait until polymer renders the tag before initializing slick

I tried to call slick in the attached callback or setting a timeout of 7s before initializing it.
Both didn't work.
Looking around, I read about other devs having problem in Polymer 1.0 that they didn't have with Polymer 0.5 with libraries that manipulate the DOM and it seems to be related, in those cases, to Shady DOM: a new Shadow DOM shim introduced in Polymer 1.0.
I am not sure that's the case here, because it seems to me SlickJS only touches the external tag, it doesn't care if there is a shadow root or simple html inside, but I thought it was worth mentioning.
What do you guys think?


ruby doesn't work on bun

1 | import { make_response, module, rubyvm, helpers }
2 |   from "@primate/binding/ruby";
3 | import { File } from "rcompat/fs";
4 |
5 | const { vm } = await rubyvm(module);
                                ^
ReferenceError: Cannot access uninitialized variable.
      at /home/blue/projects/primatejs/app/build/routes/rb/json.js:5:29

is this a bug with ruby itself? if so link to issue.

I think it's a bun issue, I still need to look further into it, this is mostly a placeholder for now.

https://github.com/oven-sh/bun/issues/12755


Dashboard / Charts not loading (first time after login, every login) in superset #supersetissues #superset

While opening charts/dashboard after login for the first time am getting the following error in superset. However after sometime in the same login when load it again, I am getting the dashboards / charts properly.

P.S.: This happens at the first instance of every login.
Kindly let me know a solution.

If you're referring to local env with docker-compose so you should wait for the frontend build to finish


Fix Refresh token AbstractController::DoubleRenderError

In app/controllers/api/devise/tokens_controller.rb#refresh was causing AbstractController::DoubleRenderError when access_token is revoked due to a missing return.
        if current_devise_api_refresh_token.revoked?
          error_response = Devise::Api::Responses::ErrorResponse.new(request, error: :revoked_token,
                                                                              resource_class: resource_class)

          return render json: error_response.body, status: error_response.status
        end

In spec/requests/token_spec.rb some refresh_token tests was passing access_token in headers
      before do
        post refresh_user_tokens_path, headers: authentication_headers_for(user, devise_api_token, :refresh_token), as: :json
      end

I was about to submit a PR to fix this same issue, but found this one already open. It'd be great if this could be merged.


Primm's algorithm and PrioirtyQueue test reference methods that don't exist

I am guessing there is a new version of PrioirtyQueue that maybe did not get pushed?  Both of the programs in the subject reference PrioirtyQueue.poll(), PrioirtyQueue.peek() and PrioirtyQueue.isEmpty(), but I do not see them in the data structure. Any help solving this would be much appreciated, your data structures are awesome!

@trekhleb I see what you are saying, my apologies. I guess my real problem is, the test for MinHeap all pass. However, when I run the Prioirty Queue tests, they all fail when using the peek and poll functions in Heap.
`  PriorityQueue
✓ should create default priority queue (4ms)
✕ should insert items to the queue and respect priorities (66ms)
✕ should poll from queue with respect to priorities (15ms)
✕ should be possible to change priority of internal nodes (10ms)
✕ should be possible to change priority of head node (9ms)
✕ should be possible to change priority along with node addition (8ms)
✓ should be possible to search in priority queue by value (4ms)
● PriorityQueue › should insert items to the queue and respect priorities
expect(received).toBe(expected) // Object.is equality

Expected: 10
Received: 5

  16 |
  17 |     priorityQueue.add(5, 2);
> 18 |     expect(priorityQueue.peek()).toBe(10);
     |                                  ^
  19 |
  20 |     priorityQueue.add(100, 0);
  21 |     expect(priorityQueue.peek()).toBe(100);

  at Object.toBe (src/dataStructures/test/PriorityQueue.test.js:18:34)

● PriorityQueue › should poll from queue with respect to priorities
expect(received).toBe(expected) // Object.is equality

Expected: 100
Received: 5

  30 |     priorityQueue.add(200, 0);
  31 |
> 32 |     expect(priorityQueue.poll()).toBe(100);
     |                                  ^
  33 |     expect(priorityQueue.poll()).toBe(200);
  34 |     expect(priorityQueue.poll()).toBe(10);
  35 |     expect(priorityQueue.poll()).toBe(5);

  at Object.toBe (src/dataStructures/test/PriorityQueue.test.js:32:34)

● PriorityQueue › should be possible to change priority of internal nodes
expect(received).toBe(expected) // Object.is equality

Expected: 200
Received: 5

  47 |     priorityQueue.changePriority(10, 20);
  48 |
> 49 |     expect(priorityQueue.poll()).toBe(200);
     |                                  ^
  50 |     expect(priorityQueue.poll()).toBe(5);
  51 |     expect(priorityQueue.poll()).toBe(100);
  52 |     expect(priorityQueue.poll()).toBe(10);

  at Object.toBe (src/dataStructures/test/PriorityQueue.test.js:49:34)

● PriorityQueue › should be possible to change priority of head node
expect(received).toBe(expected) // Object.is equality

Expected: 100
Received: 5

  64 |     priorityQueue.changePriority(10, 20);
  65 |
> 66 |     expect(priorityQueue.poll()).toBe(100);
     |                                  ^
  67 |     expect(priorityQueue.poll()).toBe(5);
  68 |     expect(priorityQueue.poll()).toBe(200);
  69 |     expect(priorityQueue.poll()).toBe(10);

  at Object.toBe (src/dataStructures/test/PriorityQueue.test.js:66:34)

● PriorityQueue › should be possible to change priority along with node addition
expect(received).toBe(expected) // Object.is equality

Expected: 100
Received: 5

  83 |     priorityQueue.add(15, 15);
  84 |
> 85 |     expect(priorityQueue.poll()).toBe(100);
     |                                  ^
  86 |     expect(priorityQueue.poll()).toBe(5);
  87 |     expect(priorityQueue.poll()).toBe(200);
  88 |     expect(priorityQueue.poll()).toBe(15);

  at Object.toBe (src/dataStructures/test/PriorityQueue.test.js:85:34)`

I believe this is happening because the heap itself does not have the priority defined by the prioirty queue. In the Heapify Up, it is looking at the values given which in this case:
    priorityQueue.add(10, 1);
    expect(priorityQueue.peek()).toBe(10);

    priorityQueue.add(5, 2);
    expect(priorityQueue.peek()).toBe(10);

Would be 10, then 5. It puts them in order and the 5 is at the zeroth index. That is incorrect because the 10 has a higher prioirty.
PrioirtyQueue.add:
  add(item, priority = 0) {
    this.priorities[item] = priority
    super.add(item)

    return this
  }

I don't see how Heap.peek() would ever get a value with a higher priority when used in conjunction with PrioirtyQueue.add()


End to end tests

E2E tests spin up a test Tolgee instance on port 22222 and uses it for all the tests. Tests then use helpers from e2e/utils to run the CLI or interact with the Tolgee server.
Configured GitHub actions to run E2E tests and unit tests separately.
Also included a HACKING.md document for future contributors, giving a light overview of the CLI structure

In the other pipelines we are using eslint with prettier plugin, but that's a nitpick.


Return larger shape logs in chunks

It's not a good idea to return huge shape logs in one request as:

many HTTP proxies won't cache large shapes (e.g. 20mb+)
clients can't process the shape log until they finish downloading it — e.g. if a client is inserting data into pglite, it'd be faster if they could parallelize downloading/parsing data with inserting rows into pglite. The same is true of any integration.

Ideal chunk size is around ~10mb.
The API was designed for this as clients continue requesting data until they the server reports they're at the end of the log so there'd be no breaking changes required for this.

@kevin-dp and I are workshopping this - will keep the thread updated


tag with space should be ignore

"how does it feel to be rejected? #loner tt ggg sjdsj dj  #link"
[loner linker]
"how does it feel to be rejected? # it is #loner tt ggg sjdsj dj  #linker"
Expect: [loner linker]
Got: [ it is #loner tt ggg sjdsj dj  #linker]

@kkdai thanks for reporting. I just made the fix, you can reopen this issue if it is not yet resolved for you.


Change on Documentation

Regarding the Schema creation using the StructType in Spark v.1.6 the schema can be created as follows:
    val customSchema = StructType(
      StructField("year", IntegerType, true) ::
      StructField("make", StringType, true) ::
      StructField("model", StringType, true) ::
      StructField("comment", StringType, true) ::
      StructField("blank", StringType, true) :: Nil)

based on the following doc: http://spark.apache.org/docs/latest/api/scala/index.html#org.apache.spark.sql.types.StructType

@raschild This is just a question. So, the schema creation in documentation is deprecated or does not work, and this way is recommended?

The provided example in the ReadMe.md for spark 1.4+ did not work for version spark 1.6 (most probably deprecated). Given the above documentation for initializing StructTypes you need to create a sequence of StructFields.

I think maybe https://github.com/databricks/spark-csv/pull/276, https://github.com/databricks/spark-csv/pull/277 have fixed this one?
(1 :: 2 :: Nil) == List(1,2)

As you know, both expressions above is identical and StructType takes arguments as Seq which includes List and Array.


Support (pythonic) way of installation

I personally don't like messing with my PYTHONPATH if I don't have to. Is it possible to provide a way to install this project in a conda environment with pip/setup.py?

Conda, what is it? :laughing:  But, you are absolutely right, and once I have the testing mechanism in place, proper installation would be the next one.


add code to generate a plot to show the scale-free fit for different …

Fei wrote some R code to generate a figure to show the scale-free fit for different powers used in WGCNA.

Fei, don't we need to use upto max power at least not limited to 20?

It is 20 right now.
https://github.com/kbase/coexpression/blob/develop/scripts/coex_cluster2.R
powers = c(c(1:20),seq(from=1, to=20,by=2))
On Tue, Feb 2, 2016 at 1:04 PM, sjyoo<EMAIL_ADDRESS>wrote:

Fei, don't we need to use upto max power at least not limited to 20?
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/coexpression/pull/17#issuecomment-178761344.

--
Fei He

What if if the max power is beyond 20?
Thanks,
-SJ
On Tue, Feb 2, 2016 at 2:07 PM, plane83<EMAIL_ADDRESS>wrote:

It is 20 right now.
https://github.com/kbase/coexpression/blob/develop/scripts/coex_cluster2.R
powers = c(c(1:20),seq(from=1, to=20,by=2))
On Tue, Feb 2, 2016 at 1:04 PM, sjyoo<EMAIL_ADDRESS>wrote:

Fei, don't we need to use upto max power at least not limited to 20?
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/coexpression/pull/17#issuecomment-178761344.

--
Fei He
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/coexpression/pull/17#issuecomment-178763035.

Use 20 is just to follow the default setup in WGCNA.
Showing the scale-free fitting does not mean the network has to be
scale-free.
On Tue, Feb 2, 2016 at 1:09 PM, sjyoo<EMAIL_ADDRESS>wrote:

What if if the max power is beyond 20?
Thanks,
-SJ
On Tue, Feb 2, 2016 at 2:07 PM, plane83<EMAIL_ADDRESS>wrote:

It is 20 right now.

https://github.com/kbase/coexpression/blob/develop/scripts/coex_cluster2.R

powers = c(c(1:20),seq(from=1, to=20,by=2))
On Tue, Feb 2, 2016 at 1:04 PM, sjyoo<EMAIL_ADDRESS>wrote:

Fei, don't we need to use upto max power at least not limited to 20?
—
Reply to this email directly or view it on GitHub
<https://github.com/kbase/coexpression/pull/17#issuecomment-178761344
.

--
Fei He
—
Reply to this email directly or view it on GitHub
https://github.com/kbase/coexpression/pull/17#issuecomment-178763035.

—
Reply to this email directly or view it on GitHub
https://github.com/kbase/coexpression/pull/17#issuecomment-178763840.

--
Fei He


zenith pg list command shows wrong LSN

I launched a postgres node at an older LSN:
$ ./target/debug/zenith pg list
NODE	ADDRESS	TIMELINE	BRANCH NAME	LSN		STATUS
main	<IP_ADDRESS>:55432	3351c4802b77a416df038a952749d674	main	0/180BDB8	running

$ ./target/debug/zenith pg start pitr --lsn=0/176B168
Starting new postgres pitr on timeline 3351c4802b77a416df038a952749d674 ...
Extracting base backup to create postgres instance: path=.zenith/pgdatadirs/tenants/7071f8f508f9f394da876799259ef014/pitr port=55433
Starting postgres node at 'host=<IP_ADDRESS> port=55433 user=zenith_admin dbname=postgres'
waiting for server to start.... done
server started

But zenith pg list shows the tip of the branch as the LSN, not the LSN the server is actually anchored at:
$ ./target/debug/zenith pg list
NODE	ADDRESS	TIMELINE	BRANCH NAME	LSN		STATUS
main	<IP_ADDRESS>:55432	3351c4802b77a416df038a952749d674	main	0/180BDF0	running
pitr	<IP_ADDRESS>:55433	3351c4802b77a416df038a952749d674	main	0/180BDF0	running

Connecting to the server and running queries confirms that it is seeing the older data, so the pg list command is at fault here.

The CLI code does use the last_record_lsn: https://github.com/zenithdb/zenith/blob/063f9ba81dfaa8f6c9b0b8797d41532715a40669/zenith/src/main.rs#L645-L648
and that seems to be filled correctly too: https://github.com/zenithdb/zenith/blob/edc7bebcb5a452ad84c5c3cfd46b727c6e6f1c48/pageserver/src/timelines.rs#L55
https://github.com/zenithdb/zenith/blob/28bc8e3f5c961532f4177fb3e803b73f6a2adb5a/pageserver/src/layered_repository.rs#L1010-L1012
so the issue seems to lie in the init code, that fails to replay onto --lsn=0/176B168 properly.

The CLI code does use the last_record_lsn: https://github.com/zenithdb/zenith/blob/063f9ba81dfaa8f6c9b0b8797d41532715a40669/zenith/src/main.rs#L645-L648
and that seems to be filled correctly too: https://github.com/zenithdb/zenith/blob/edc7bebcb5a452ad84c5c3cfd46b727c6e6f1c48/pageserver/src/timelines.rs#L55 https://github.com/zenithdb/zenith/blob/28bc8e3f5c961532f4177fb3e803b73f6a2adb5a/pageserver/src/layered_repository.rs#L1010-L1012
so the issue seems to lie in the timeline init code, that fails to replay onto --lsn=0/176B168 properly, when creating a timeline.

This test doesn't create a new timeline, both Postgres servers are running on the same timeline. That can been seen in the CLI output, too, both servers have the same "branch name" 3351c4802b77a416df038a952749d674. (The "branch name" is a misnomer now too, because it actually shows the timeline ID, not a human-readable name, but that's a separate issue)
So both Postgres servers are running on the same timeline, and the CLI shows the last record LSN for that timeline, for both servers. But that's misleading, as the 'pitr' Postgres server is anchored at an older LSN, not at the last record. The output should look like this:
$ ./target/debug/zenith pg list
NODE	ADDRESS	TIMELINE	BRANCH NAME	LSN		STATUS
main	<IP_ADDRESS>:55432	3351c4802b77a416df038a952749d674	main	0/180BDF0	running
pitr	<IP_ADDRESS>:55433	3351c4802b77a416df038a952749d674	main	0/176B168	running

(The "branch name" is a misnomer now too, because it actually shows the timeline ID, not a human-readable name, but that's a separate issue)

Created https://github.com/neondatabase/neon/issues/1628 for that


Failsafe from rollbar-gem

About 60 minutes ago I started seeing this show up continuously:
Failsafe from rollbar-gem. TypeError: "wrong argument type Nokogiri::XML::Element (expected Data)" in /app/vendor/bundle/ruby/2.2.0/gems/nokogiri-<IP_ADDRESS>/lib/nokogiri/xml/pp/node.rb:8:in `node_name': error in process_item

There is no stack trace.
Recent deploys don't have anything suspicious (just changing an AR scope and a regex in a string filter of sorts).
The only correlating thing seems to be a huge spike in a specific exception I'm seeing (postgres deadlocks).

Elaboration on why there might be a correlation to increased error rate (or perhaps it just maximises the chances of experiencing the bug):

I think this might be due to some data that Rollbar is trying to encode being a Nokogiri::XML::Element. This deadlock error is happening in a background job (which predates me so I don't fully understand it) that is passing XML nodes around.
Is there any way I can get more information on the root cause here?

I've dug into this enough to determine that this is not a problem with Rollbar.


cargo-semver-checks report not shown in release PR

Seen in https://github.com/MarcoIeni/rust-workspace-example/pull/214

The version is updated correctly, but the report wasn't shown. So you didn't know what the breaking change was


Final Corrections

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

80 of 80 (100.0%)  changed or added relevant lines in 20 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
559

Relevant Lines:
559

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

80 of 80 (100.0%)  changed or added relevant lines in 20 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
559

Relevant Lines:
559

💛  - Coveralls


Your requirements could not be resolved to an installable set of packages.

Detailed description
I have the exact same issue as described here: https://github.com/acacha/adminlte-laravel/issues/254
and here: https://github.com/acacha/adminlte-laravel/issues/273
*********** part of the error: ***************
Your requirements could not be resolved to an installable set of packages.
Problem 1
- acacha/adminlte-laravel-installer 3.1.6 requires acacha/admin-lte-template-laravel dev-master -> satisfiable by acacha/admin-lte-template-laravel[dev-master] but these conflict with your requirements or minimum-stability.
- acacha/adminlte-laravel-installer 3.1.5 requires acacha/admin-lte-template-laravel dev-master -> satisfiable by acacha/admin-lte-template-laravel[dev-master] but these conflict with your requirements or minimum-stability.
- acacha/adminlte-laravel-installer 3.1.4 requires acacha/admin-lte-template-laravel dev-master -> satisfiable by acacha/admin-lte-template-laravel[dev-master] but these conflict with your requirements or minimum-stability.
(...)

Context
I am using a Homestead environment, laravel works fine. I've already tried the solutions on both issues above (composer global update), still no luck :(
Possible implementation
Not obligatory, but suggest an idea for implementing addition or change.
Your environment
PHP 7.1.10-1+ubuntu16.04.1+deb.sury.org+1

As I say in the ohter issues I was unable to reproduce the problem. I don't see any dev-master references in the packages involved. Please could somebody provide acces to an environment on exactly steps to reproduce the problem?


Deployment Epic

We need a clear plan/vision for the future of our deployment tool. Ideally, one document that details the definitions, composition and workflow.
Prior discussions and assets:

https://gist.github.com/zyndiecate/7d28bd97633c22731c14

General concepts and possible workflow

https://github.com/giantswarm/architecture/pull/25

General discussion on the future of releaseit

https://github.com/giantswarm/giantswarm/issues/276

Original issue on improving releaseit

https://gist.github.com/zyndiecate/d05bc87dafb8cb2ee6ce

Example for possible format of configuration file

https://github.com/giantswarm/architecture/files/60670/configuration.-.deployment.overview.pdf

Overview PDF

https://github.com/giantswarm/architecture/pull/26

General deployment process thoughts

https://docs.google.com/presentation/d/1iaPmFxwXcdruu-NgZ16Duvi_O0MjnXbxkOcPQl8ZcEs/edit?usp=sharing

Overview slides

TODO:

[ ] Workflow designs
[ ] Tool feature scope
[ ] Configuration management designs
[ ] Rough roadmap

https://github.com/giantswarm/architecture/pull/27


Set date/time of a photo

Here is a naïve question: is it possible to set the date and time of a picture using this library?

Yes.  The following snippet changes the date/time of the first selected photo. You cannot change timezone. The date/time must be a timezone-naive datetime object in the local timezone.
>>> import photoscript
>>> import datetime
>>> photo = photoscript.PhotosLibrary().selection[0]
>>> photo.date
datetime.datetime(2022, 11, 22, 0, 0)
>>> photo.date = datetime.datetime(2022,11,23,12,34,56)
>>> photo.date
datetime.datetime(2022, 11, 23, 12, 34, 56)

For more advanced date/time manipulation, see the timewarp command of the osxphotos tool which does allow manipulation of timezone.  (I'm the author of osxphotos as well).

You can see all the properties of the Photo object here

Thanks for this information.  I'll try this out soon.


"Bridge response is not Valid" when enabling Hue Entertainment

[x] I confirm that this is an issue rather than a question.

Bug report
Using diyHue Hue Emulator, enabling Entertainment mode immediately disables the LED device. In logs I can see the following messages:
2020-12-12T00:25:51.645Z [hyperiond LEDDEVICE] (DEBUG) (ProviderRestApi.cpp:121:get()) GET: [http://<IP_ADDRESS>:80/api/c240d1f03bf811ebaa38dca632806f54/groups/7]
 2020-12-12T00:25:51.650Z [hyperiond LEDDEVICE] (DEBUG) (ProviderRestApi.cpp:179:getResponse()) Reply.httpStatusCode [200]
 2020-12-12T00:25:51.651Z [hyperiond LEDDEVICE] (DEBUG) (LedDevicePhilipsHue.cpp:1095:startStream()) Start entertainment stream
 2020-12-12T00:25:51.652Z [hyperiond LEDDEVICE] (DEBUG) (ProviderRestApi.cpp:150:put()) PUT: [http://<IP_ADDRESS>:80/api/c240d1f03bf811ebaa38dca632806f54/groups/7] [{"stream":{"active":true}}]
 2020-12-12T00:25:52.216Z [hyperiond LEDDEVICE] (DEBUG) (ProviderRestApi.cpp:179:getResponse()) Reply.httpStatusCode [200]
 2020-12-12T00:25:52.217Z [hyperiond LEDDEVICE] (ERROR) Device disabled, device 'philipshue' signals error: 'set stream to true: Bridge response is not Valid'
 2020-12-12T00:25:52.217Z [hyperiond COMPONENTREG] (DEBUG) (ComponentRegister.cpp:36:setNewComponentState()) LED device: disabled
 2020-12-12T00:25:52.218Z [hyperiond LEDDEVICE] (ERROR) Philips Hue Entertaiment API could not initialisized!

Most notably the message: set stream to false: Bridge response is not Valid
To me, it appears the request to either enable streaming is successful (200 code, body looks correct according to this) but I'm not sure what else could cause this. I've tried manual PUTs to enable/disable streaming on diyHue before enabling the LED device, but no go.
I found #769 relates to enabling support for diyHue and specifically this comment by @Lord-Grey indicating a fix to use the client key from the wizard that worked. It seems that fix is built in now? so I'm thinking this is a different issue. I've also tried skipping the wizard entirely and using manual entries for IP, group ID, username and client -- same issue.
 Seeding the random number generator... ok
  . Loading the CA root certificate ... ok (0 skipped)
  . Loading the server cert. and key... ok
  . Bind on udp://*:2100/ ... ok
  . Setting up the SSL/TLS structure... ok

In the Hue Sync app, a connection is made to Hyperion as the entertainment group has the message "Syncing with hyperion (xxxxxxxx)" and it has to be kicked off to start an entertainment sync session with another app.
If there's any better logging that I'm not providing, or any tips to debug further, please let me know!
Steps to reproduce
Install diyHue
Create entertainment group
Install hyperbian
Add new LED controller
Select philipshue, check entertainment
Use wizard to setup
What is expected?
Streaming to succeed, or at least a more helpful error message
What is actually happening?
LED controller is disabled
System
Hyperion Server:
- Build:           (HEAD detached at 83338dc3) (Paulchen Panther-ca50487e/83338dc3-1605733041)
- Build time:      Nov 18 2020 21:32:08
- Git Remote:      https://github.com/hyperion-project/hyperion.ng
- Version:         2.0.0-alpha.9
- UI Lang:         auto (BrowserLang: en-US)
- UI Access:       expert
- Avail Capt:      dispmanx,v4l2,framebuffer,qt
- Database:        read/write

Hyperion Server OS:
- Distribution:   HyperBian GNU/Linux 10 (buster)
- Architecture:   arm
- CPU Model:      ARMv7 Processor rev 4 (v7l)
- CPU Type:       Raspberry Pi 3 Model B Plus Rev 1.3
- CPU Revision:   a020d3
- CPU Hardware:   BCM2835
- Kernel:         linux (5.4.72-v7+ (WS: 32))
- Qt Version:     5.7.1
- Python Version: 3.5.3
- Browser:        Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Safari/537.36

update: compiled manually and the bug is not present - seems like HyperBian specific? hopefully this will help someone else!
closing


(WIP)(GH-66) Fix go to definition

Fixes #66, reported in https://github.com/lingua-pupuli/puppet-vscode/issues/331

@jpogran #70 will pin rubocop.

I'm happy to merge this without red CI.  Rubocop is tested last for a reason :-)


Add addPath option

This PR adds addPath option to transform config which allows adding a new file to outputDir with frontMatter using config like below,
  - addPath: _index.md
    frontMatter:
      template: |
        weight: 100000

It cannot be specified alongside path or glob.
Only Origin.LastMod and target.FileName are specifiable in addPath frontMatter template. Fixes #51.

Closing as discussed in 1:1. 🙂


Add an option which prints warning and error instead of the unicode character

Hi there. Thanks for this handy tool :)
I want to create a problem matcher for vscode in a custom task where i pipe the build output from xcodebuild through xcbeautify. In order to distinguish between and error and a warning i need to match the words as a string with a regex. VSCode can then present the errors and warnings in the problems tab and i can jump to them.
At the moment this is not working quite well since warning and error are printed by xcbeautify as \u{9888} and \u{26A0}. It would be nice if there were an option i can pass in and then warning and error is printed instead of the special unicode character.

There's already an option called --disable-colored-output. You can also disable colored output by setting NO_COLOR=1 in your environment variables.

Yes this would work. But i like the colored output since its much easier too read :) i'd like to have both. Colored without special unicode characters


Gap Duration Analysis:

Acceptable gaps should be SumGapDuration1 divided by 4.1 (if critical gap = 4.1) or SumGapDuration2 divided by 5.3 (if critical gap = 5.3); Line 60, LeftTurnGapOutAnalysis.cs
Line 48, LeftTurnGapOutAnalysis.cs needs to be Demand divided by Capacity. So it will be  - gapDurationResult.GapDurationPercent = gapDurationResult.Demand / gapDurationResult.Capacity;

nice job Morgan


Committment discounts show blank

I have lot of RI purchased but the commitments/RI  savings show blank in both the cost summary and rate optimization reports.

@saleemmy By any chance, are you using FOCUS 1.0?
There was a bug that was fixed recently in Cost Management exports where PricingCategory was set to "Standard" for commitment discount usage. FTK reports filter on PricingCategory == "Committed", so this is why the bug ends up showing no commitment discounts. The bug was fixed in late July, so any exports that ran at the end of the month should have fixed this however. If you're seeing this, rerun the exports and you should be good. We'll also add a local workaround for this in 0.5.

@flanakin
I just reinstalled finops hub toolkit from the site, and I am sure it is 0.4 version. The exports are Focus 1.0 (preview). I have manually exported all the previous months for both daily and monthly settings.
I am supposed to see more than 100K of RI and SP savings but see only 4K which is for the SP savings and all RI savings for both VM and OS show zero. Even storage RI shows zero savings.
The SLES OS RI show as Virtual Machines service instead of SUSE Linux just like in the Azure portal.

Can anyone help to resolve this ?

@saleemmy Can you share a screenshot of what you see on the Rate optimization > DQ page? The first set of numbers give a high-level summary of the data we see.


x11: Poll the window until it is really visible

Fixes #697
It seems that XSync doesn't really makes the window viewable.
This feels hacky, other option to do it could be using XIfEvent or
similar to listen to MapNotify events, but we'll have a loop still.
In practice, this lasts between two and thre iterations on my machine,
which is something not noticeable.

Well, I'm going to trust you on this one.