connection refused in download_ncbi_associations

Hi!
thank you so much for developing this tool, it seems to be exactly what I'm looking for. However, I'm trying to run the following notebook :"Run a Gene Ontology Enrichment Analysis (GOEA)", and it seems that "download_ncbi_associations" is broken. Running the cell below spits an error message:
# Get ftp://ftp.ncbi.nlm.nih.gov/gene/DATA/gene2go.gz
from goatools.base import download_ncbi_associations
fin_gene2go = download_ncbi_associations()

FTP RETR ftp.ncbi.nlm.nih.gov gene/DATA gene2go.gz -> gene2go.gz
Traceback (most recent call last):
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py", line 219, in dnld_file
    rsp = http_get(src_ftp, dst_wget) if src_ftp[:4] == 'http' else ftp_get(src_ftp, dst_wget)
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py", line 202, in ftp_get
    ftp.retrbinary(cmd, open(fout, 'wb').write)  #           /usr/home/gene2go.gz
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py", line 425, in retrbinary
    with self.transfercmd(cmd, rest) as conn:
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py", line 382, in transfercmd
    return self.ntransfercmd(cmd, rest)[0]
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py", line 343, in ntransfercmd
    conn = socket.create_connection((host, port), self.timeout,
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/socket.py", line 808, in create_connection
    raise err
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/socket.py", line 796, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 61] Connection refused
**FATAL cmd: wget(ftp://ftp.ncbi.nlm.nih.gov/gene/DATA/gene2go.gz out=gene2go.gz)
**FATAL msg: [Errno 61] Connection refusedERROR:root:Internal Python error in the inspect module.
Below is the traceback from this internal error.

Traceback (most recent call last):
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py", line 219, in dnld_file
    rsp = http_get(src_ftp, dst_wget) if src_ftp[:4] == 'http' else ftp_get(src_ftp, dst_wget)
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py", line 202, in ftp_get
    ftp.retrbinary(cmd, open(fout, 'wb').write)  #           /usr/home/gene2go.gz
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py", line 425, in retrbinary
    with self.transfercmd(cmd, rest) as conn:
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py", line 382, in transfercmd
    return self.ntransfercmd(cmd, rest)[0]
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py", line 343, in ntransfercmd
    conn = socket.create_connection((host, port), self.timeout,
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/socket.py", line 808, in create_connection
    raise err
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/socket.py", line 796, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 61] Connection refused

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/interactiveshell.py", line 3418, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-20-fa8d3bdfc016>", line 3, in <module>
    fin_gene2go = download_ncbi_associations()
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py", line 133, in download_ncbi_associations
    dnld_file(file_remote, gene2go, prt, loading_bar)
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py", line 229, in dnld_file
    sys.exit(1)
SystemExit: 1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py", line 1170, in get_records
    return _fixed_getinnerframes(etb, number_of_lines_of_context, tb_offset)
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py", line 316, in wrapped
    return f(*args, **kwargs)
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py", line 350, in _fixed_getinnerframes
    records = fix_frame_records_filenames(inspect.getinnerframes(etb, context))
  File "/Users/ernestmordret/opt/anaconda3/envs/py3_goa/lib/python3.8/inspect.py", line 1503, in getinnerframes
    frameinfo = (tb.tb_frame,) + getframeinfo(tb, context)
AttributeError: 'tuple' object has no attribute 'tb_frame'
---------------------------------------------------------------------------
ConnectionRefusedError                    Traceback (most recent call last)
~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py in dnld_file(src_ftp, dst_file, prt, loading_bar)
    218         #### wget.download(src_ftp, out=dst_wget, bar=loading_bar)
--> 219         rsp = http_get(src_ftp, dst_wget) if src_ftp[:4] == 'http' else ftp_get(src_ftp, dst_wget)
    220         if do_gunzip:

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py in ftp_get(fin_src, fout)
    201     cmd = 'RETR {F}'.format(F=fin_ftp)   #                   gene2go.gz
--> 202     ftp.retrbinary(cmd, open(fout, 'wb').write)  #           /usr/home/gene2go.gz
    203     ftp.quit()

~/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py in retrbinary(self, cmd, callback, blocksize, rest)
    424         self.voidcmd('TYPE I')
--> 425         with self.transfercmd(cmd, rest) as conn:
    426             while 1:

~/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py in transfercmd(self, cmd, rest)
    381         """Like ntransfercmd() but returns only the socket."""
--> 382         return self.ntransfercmd(cmd, rest)[0]
    383

~/opt/anaconda3/envs/py3_goa/lib/python3.8/ftplib.py in ntransfercmd(self, cmd, rest)
    342             host, port = self.makepasv()
--> 343             conn = socket.create_connection((host, port), self.timeout,
    344                                             source_address=self.source_address)

~/opt/anaconda3/envs/py3_goa/lib/python3.8/socket.py in create_connection(address, timeout, source_address)
    807         try:
--> 808             raise err
    809         finally:

~/opt/anaconda3/envs/py3_goa/lib/python3.8/socket.py in create_connection(address, timeout, source_address)
    795                 sock.bind(source_address)
--> 796             sock.connect(sa)
    797             # Break explicitly a reference cycle

ConnectionRefusedError: [Errno 61] Connection refused

During handling of the above exception, another exception occurred:

SystemExit                                Traceback (most recent call last)
    [... skipping hidden 1 frame]

<ipython-input-20-fa8d3bdfc016> in <module>
      2 from goatools.base import download_ncbi_associations
----> 3 fin_gene2go = download_ncbi_associations()

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py in download_ncbi_associations(gene2go, prt, loading_bar)
    132             GZ=os.path.basename(gzip_file))
--> 133         dnld_file(file_remote, gene2go, prt, loading_bar)
    134     else:

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/goatools/base.py in dnld_file(src_ftp, dst_file, prt, loading_bar)
    228         sys.stderr.write("**FATAL msg: {ERR}".format(ERR=str(errmsg)))
--> 229         sys.exit(1)
    230

SystemExit: 1

During handling of the above exception, another exception occurred:

TypeError                                 Traceback (most recent call last)
    [... skipping hidden 1 frame]

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/interactiveshell.py in showtraceback(self, exc_tuple, filename, tb_offset, exception_only, running_compiled_code)
   2036                     stb = ['An exception has occurred, use %tb to see '
   2037                            'the full traceback.\n']
-> 2038                     stb.extend(self.InteractiveTB.get_exception_only(etype,
   2039                                                                      value))
   2040                 else:

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py in get_exception_only(self, etype, value)
    821         value : exception value
    822         """
--> 823         return ListTB.structured_traceback(self, etype, value)
    824
    825     def show_exception_only(self, etype, evalue):

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py in structured_traceback(self, etype, evalue, etb, tb_offset, context)
    696             chained_exceptions_tb_offset = 0
    697             out_list = (
--> 698                 self.structured_traceback(
    699                     etype, evalue, (etb, chained_exc_ids),
    700                     chained_exceptions_tb_offset, context)

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py in structured_traceback(self, etype, value, tb, tb_offset, number_of_lines_of_context)
   1434         else:
   1435             self.tb = tb
-> 1436         return FormattedTB.structured_traceback(
   1437             self, etype, value, tb, tb_offset, number_of_lines_of_context)
   1438

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py in structured_traceback(self, etype, value, tb, tb_offset, number_of_lines_of_context)
   1334         if mode in self.verbose_modes:
   1335             # Verbose modes need a full traceback
-> 1336             return VerboseTB.structured_traceback(
   1337                 self, etype, value, tb, tb_offset, number_of_lines_of_context
   1338             )

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py in structured_traceback(self, etype, evalue, etb, tb_offset, number_of_lines_of_context)
   1191         """Return a nice text document describing the traceback."""
   1192
-> 1193         formatted_exception = self.format_exception_as_a_whole(etype, evalue, etb, number_of_lines_of_context,
   1194                                                                tb_offset)
   1195

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py in format_exception_as_a_whole(self, etype, evalue, etb, number_of_lines_of_context, tb_offset)
   1149
   1150
-> 1151         last_unique, recursion_repeat = find_recursion(orig_etype, evalue, records)
   1152
   1153         frames = self.format_records(records, last_unique, recursion_repeat)

~/opt/anaconda3/envs/py3_goa/lib/python3.8/site-packages/IPython/core/ultratb.py in find_recursion(etype, value, records)
    449     # first frame (from in to out) that looks different.
    450     if not is_recursion_error(etype, value, records):
--> 451         return len(records), 0
    452
    453     # Select filename, lineno, func_name to track frames with

TypeError: object of type 'NoneType' has no len()

Do you have any clue of what might be causing it?
Best,
Ernest

@ernestmordret
This is probably too late to be useful - but I just tried the same and it worked (although the downloading is a bit slow due to the large file size). Is this still an issue for you?


⚠ Bofrost.com has degraded performance

In 4296647, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 1669 ms

Resolved: Bofrost.com performance has improved in 15c16fe after 6 minutes.


Integrate the new yb-client APIs for calling GetTabletListToPollForCDC

The api YBClient#getTabletListToPollForCDC has changed and instead of simply returning <TabletId, OpId>, it now returns <TabletLocationsPB,OpId>.
Make relevant changes in the connector as well.

Closing as the relevant changes have been merged in PR (#85)


[Suggestion] More Restrictive Lint Levels

Clippy shall ensure a good code quality.  However, the default configuration does not catch all mistakes such that a more restrictive setup would be useful.  I would like to suggest to configure Clippy to be more restrictive.  Therefore, one just needs to add the desired lint levels to a crate's root, i.e. src/main.rs, here.
My preferred setup is the following:
#![deny( // Basic setup with the most important lint levels.
  clippy::all,
  clippy::cargo,
  clippy::complexity,
  clippy::correctness,
  clippy::nursery,
  clippy::pedantic,
  clippy::perf,
  clippy::suspicious,
  clippy::style,
  deprecated,
  missing_docs,
  rustdoc::broken_intra_doc_links
)]
#![allow(clippy::multiple_crate_versions)] // Do not reject Clap's dependencies as implied by `clippy::cargo`.

@tobealive, which lint levels would you prefer to add?

I like that. I was about to add more strictness to the action runs. But I thought if Clippy yelled that much, it would be nasty, especially for new contributors. Having those right away at compile time would be great.
This level of strictness will probably require quite a bit of change. And we would need to remove things like missing_docs. If you want to add this setup and the changes suggested by clippy, go for it :)!

Okay, I will prepare a PR with the initial setup.  I will add the suggestion from the initial comment on this issue as it is despite commenting out the missing_docs.  I will also take care for the necessary changes due to the more restrictive lints.

When adding the very restrictive Clippy lints, there were about 200 errors.  I already resolved the half of them.  It might take a while to also resolve the others.  I will submit the resolved code by the end of the next week.

I will prepare a PR based on the changes in #106 as soon as it is merged.

Clippy shall ensure a good code quality. However, the default configuration does not catch all mistakes such that a more restrictive setup would be useful. I would like to suggest to configure Clippy to be more restrictive. Therefore, one just needs to add the desired lint levels to a crate's root, i.e. src/main.rs, here.
My preferred setup is the following:
#![deny( // Basic setup with the most important lint levels.
  clippy::all,
  clippy::cargo,
  clippy::complexity,
  clippy::correctness,
  clippy::nursery,
  clippy::pedantic,
  clippy::perf,
  clippy::suspicious,
  clippy::style,
  deprecated,
  missing_docs,
  rustdoc::broken_intra_doc_links
)]
#![allow(clippy::multiple_crate_versions)] // Do not reject Clap's dependencies as implied by `clippy::cargo`.

@tobealive, which lint levels would you prefer to add?

some of these are redundant, no?
clippy::all is an alias for correctness, suspicious, style, complexity, and perf.

The other piece of advice I would give is if you're using such restrictive lints, then I would strongly consider pinning the clippy version used in CI to prevent PRs from having errors which are from new lints and not related to the changed code

Thank you for your feedback, @danieleades!  I will take your comments into account when preparing the PR.

@danieleades introduced a very good basic setup regarding more restrictive lint levels in #106.  As there is nothing more to add to that basic configuration, I will close this issue as completed.


Use correct capitalization for PowerShell

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
All articles that use the word "Powershell" or "powershell".
What part(s) of the article would you like to see updated?
As pointed out in this issue, the correct capitalization is "PowerShell" (P and S both capitalized).
Additional information
No response

Code

Money


test: add an integration test for overflow rollback

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#635  👈 (View in Graphite)
#634
#633
#632
#631
#630
#626
#625
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 23, 8:13 AM EST: A user started a stack merge that includes this pull request via Graphite.


Could org.apache.skywalking:hbase-scenario:5.0.0 drop off redundant dependencies to loose weight?

Hi, I deployed project org.apache.skywalking:hbase-scenario:5.0.0 and found that its pom file introduced 98 dependencies. However, among them, 4 libraries (4%) have not been used by your project (the redundant dependencies are listed below). Reduce these useless dependencies can help prevent conflicts between library versions. Meanwhile, it can minimize the total added size to projects. It can also help enable advanced scenarios for this project.
This PR helps org.apache.skywalking:hbase-scenario:5.0.0 lose weight :) I have tested the revised configuration in my local environment. It is safe to remove the unused libraries.

Best regards

That is a test project, I don't know why you deploy that.


Set C/C++ defaults for FreeBSD

cc for C, c++ for C++ by default. gcc and g++ are also available on the image.

Since FreeBSD on staging envs are a little broken at the moment, we cannot completely test this PR there. However, we can test the Linux jobs (https://staging.travis-ci.org/github/BanzaiMan/travis_staging_test/builds/760515 https://staging.travis-ci.org/github/BanzaiMan/travis_staging_test/builds/760524).
Since their behavior is preserved, we will deploy this to production for an extra round of testing (with FreeBSD) before merging.

Stealthily deployed to production, and here's the result of a default C++ setup:
https://travis-ci.org/github/BanzaiMan/travis_production_test/builds/695057012#L32-L41
$ export TRAVIS_COMPILER=c++
$ export CXX=${CXX:-c++}
$ export CXX_FOR_BUILD=${CXX_FOR_BUILD:-c++}
$ export CC=${CC:-cc}
$ export CC_FOR_BUILD=${CC_FOR_BUILD:-cc}
$ c++ --version
FreeBSD clang version 8.0.1 (tags/RELEASE_801/final 366581) (based on LLVM 8.0.1)
Target: x86_64-unknown-freebsd12.1
Thread model: posix
InstalledDir: /usr/bin


[FEATURE]: download wallpaper repo's

Description
Allow user to choose a repo containing wallpaper and download a repo containing wallpapers

Handled using GitHub APIs without cloning the repo itself here: https://github.com/dianaw353/dotfiles/blob/40bbadbb98bf66b2c7dd19a597302720ccea8e8e/roles/ags/tasks/main.yml#L164-L173


Add Drivly SVG to Documentation Footer

Add Drivly SVG to Documentation Footer
This PR adds the Powered by Drivly SVG logo to the footer of the documentation pages.
Changes

Added PoweredByDrivly.svg to public/images directory
Implemented custom footer in app/layout.jsx that includes both the Drivly SVG and original Nextra footer
Updated next.config.mjs to properly configure Nextra theme

Testing
✅ Tested locally:

Verified SVG appears in footer
Confirmed link to driv.ly works
Checked SVG styling (brightness/invert filter) for visibility
Tested across different pages to ensure consistent footer display

Link to Devin run: https://app.devin.ai/sessions/7667d45d72d348488124ff92e52aa0aa

📝 Changed routes:

/_meta
/

(view full size)
(view full size)

2 deleted routes:

/[[...mdxPath]]
/

Commit 491d915a134c508e1c2f14f5c862376da2154871 (https://docs-template-pl1t2csr0.dev.driv.ly).


Why use mlab in "Implement the Serialization of a Passport User"?

Describe your problem and how to reproduce it:
Currently, the challenge named in the title says we can set up a free database on mLab.
That seems a bit odd, considering that the previous coding challenges had us create Mongo Atlas accounts.
Note that I am not advocating one service over the other. I just think it's strange and confusing to switch services mid-curriculum without any explanation.
Add a Link to the page with the problem:
https://learn.freecodecamp.org/information-security-and-quality-assurance/advanced-node-and-express/implement-the-serialization-of-a-passport-user

@raisedadead, Is there an advantage to choosing to adopt one over the other across all mentions or does it really matter?

All these will have to be updated in the next release of curriculum. In general, you can use any platform of your choice.

I'm going to close this due to the above reasons @kiwisquash. It doesn't really matter if we encourage different services at different points, it's not a rule; just a suggestion.
We will be changing this soon with our refactored curriculum, you can learn more about that here:
https://www.freecodecamp.org/forum/t/help-us-build-version-7-0-of-the-freecodecamp-curriculum/263546


BCDB Cleanup

Code clean up (pep8)
Documentation

Operation
API Docs (how to use)

Testing

Unit tests

moved to jsx


Do not propagate host RUSTFLAGS when checking for WASM toolchain

I usually export RUSTFLAGS with an alternative linker (mold) to speed up the compilation for development builds; unfortunately the check for the WASM toolchain fails when you do this as it then tries to use this linker for wasm32-unknown-unknown which obviously doesn't work. This PR fixes this issue by simply clearing RUSTFLAGS beforehand.

bot merge

bot merge

bot merge

I stumble upon a similar issue, but it seems like in my case RUSTFLAGS are still picked up from .cargo/config.toml.


Make a 2D cross product system (if it's intuitive).

Probably should have removed the "is this possible?" tag a while ago as I now know it definitely is possible. I'll be mostly just referencing the Float3.Cross(Float3, Float3) method and only including the Z-value. The method header will look like this:
public static float Cross(Float2 a, Float2 b)

This has been completed in the latest commit.


Add release-2024b to the pull bot for sync

Related to: https://issues.redhat.com/browse/RHOAIENG-12617

/override ci/prow/images


Report additional metrics to prometheus

We should consider adding at least the following consensus-layer metrics:

[x] Current epoch number
[x] IsRegistered
[ ] Blocks voted for by your node or something similar (just a way to determine if you're voting) -- for this we should check if there is anything existing from the Tendermint side.

kostko suggested two metrics for votes counting:

how many times the node voted and the vote got included
how many times the node proposed a block (was a proposer)


distributed mimir deployment scenario

The deployment scenario should follow one of the following options.
Option 3: gagent as gateway towards cos-lite
graph LR

subgraph mimir
worker1 ---|mimir-worker| coordinator
worker2 ---|mimir-worker| coordinator
worker3 ---|mimir-worker| coordinator
coordinator ---|logging| gagent
coordinator ---|scrape| gagent
coordinator ---|remote-write| gagent
coordinator ---|dashboards| gagent
coordinator ---|trace| gagent

end

subgraph cos-lite
gagent ---|logging| loki
gagent ---|remote-write| prometheus
gagent ---|dashboards| grafana
gagent ---|trace| tempo

end

Dashboards are owned by the coordinator, as they are "per-app".
We need the grafana-agent to scrape the Mimir workers, because they are not able to remote write themselves.
Option 4: gagent inside the worker charm
Another option is to have grafana-agent deployed in each worker as a sidecar.
graph LR

subgraph mimir
worker1+gagent ---|mimir-worker| coordinator
worker2+gagent ---|mimir-worker| coordinator
worker3+gagent ---|mimir-worker| coordinator

end

subgraph cos-lite
coordinator ---|logging| loki
coordinator ---|remote-write| prometheus
coordinator ---|dashboards| grafana
coordinator ---|trace| tempo

end

In this scenario, the coordinator owns both dashboards and alert rules. It will need to take care of duplicating the alert rules per each worker by adding their topology information. Self-monitoring happens:

internally, with nginx providing the remote_write endpoint of the appropriate Mimir worker;
externally, with the coordinator providing the Loki/Prometheus URLs so that the workers can remote write directly to them.

During an upgrade of the workers, there might be a temporary inconsistency between alert rules and metric names (i.e., if the metric names change). However, it might be something we don't mind as it simplifies the architecture considerably.

We decided to pivot to something similar to the Option 5 presented in the comment above.
However, the worker-coordinator organization we agreed upon is quite different; the new one is presented below.
Option 6: gagent in coordinator and component-specific relations
graph LR

subgraph mimir
worker:::charm --- compactor:::relation --- coordinator+gagent:::charm
worker --- distributor:::relation --- coordinator+gagent
worker --- ingester:::relation --- coordinator+gagent
worker --- querier:::relation --- coordinator+gagent
worker --- query-frontend:::relation --- coordinator+gagent
worker --- store-gateway:::relation --- coordinator+gagent
worker --- alertamanger:::relation --- coordinator+gagent
worker --- ruler:::relation --- coordinator+gagent

worker --- temp:::relation --- coordinator+gagent

classDef charm fill:#f70,stroke:#000,stroke-width:3px,color:#fff
classDef relation fill:#fbb,stroke:#000,stroke-width:2px,color:#000,stroke-dasharray: 5 5

end

subgraph cos-lite
coordinator+gagent --- logging:::relation --- loki:::charm
coordinator+gagent --- remote-write:::relation --- prometheus:::charm
coordinator+gagent --- dashboards:::relation --- grafana:::charm
coordinator+gagent --- trace:::relation --- tempo:::charm

end

The Mimir coordinator will run Grafana agent as a sidecar, to be able to push self-monitoring and worker metrics to cos-lite. On the same mimir_worker interface, the coordinator will accept one relation to a worker per each Mimir component. That relation determines the role the worker will assume.
Each Mimir worker can take on multiple roles; however, there can be no role replication among workers related to the same coordinator (this can be enforced with limit: 1 on the coordinator side of the relations). The role is determined by the type of relation between the worker and the coordinator: for a fresh deployment, something like juju relate worker:mimir-compactor coordinator will make the worker run the compactor role.
The worker metrics will be scraped by the Grafana agent on the coordinator and then remote written to Prometheus.
A minimal deployment consists of one worker related to the coordinator through all the interfaces of the required components.

Cool idea that was brought up for the Tempo integration for tracing:
The relation data between worker and coordinator could also include the tracing ID so that when we, for example, scale up a worker and pass that ID to the coordinator, so it can continue the trace; this would yield a single span for the whole action.


query parser should throw error on improper "now"

I think this query should throw a parser error instead of returning the results for all time:
> select count(value) from cpu_idle where time > now - 30d group by time(1d)
name: cpu_idle
--------------
time			count
1970-01-01T00:00:00Z	920900

The GROUP BY clause and the WHERE time clause were both ignored just because of the difference between now and now(). I would expect a parser error that now is not a valid comparator for time.

@beckettsean This seems to be a dupe of what you mentioned in #4044 .

dupe of #4044


Rubocop ignore rules

@pustovalov I'm pulling this out of email so the Hound team can see it:

Hey guys,

I installed rubocop to local environment and indicated this rule:

Metrics/LineLength: false

https://github.com/pustovalov/app/blob/master/.ruby-style.yml#L543

Rubocop show this:

false: 52 files inspected, 86 offenses detected
true: 52 files inspected, 114 offenses detected

But hound still send me a messages about long line:

https://github.com/pustovalov/app/pull/13#discussion_r43354062

My config file for Hound:
https://github.com/pustovalov/app/blob/master/.hound.yml

@croaky thx

The comment is made by the HAML linter. Try setting https://github.com/brigade/haml-lint/blob/master/lib/haml_lint/linter/README.md#linelength.

@gylaz I made this:
https://github.com/pustovalov/app/commit/71c43bf471366da98290d631ef131c6f4d0c9b28
And still can see error from hound:

Joining this thread, looks like hound is failing when rubocop is enabled for .haml files.
My hound config is here:

And I intentionally made a change to get warning:

However, I'm getting an error message about hound completely failing reviewing this PR

@umerkulovb Could you contact us at<EMAIL_ADDRESS>with the contents of your .hound.yml, along with .haml-style.yml? And we'll take it from there

@pustovalov If you also could contact us via our support email, we'll see what we can do about your issue to


Python bindings: "ImportError: arg(): could not convert default argument into a Python object [...]"

Hi,
I have the following problem with using eos-py.
After successfully installing eos-py with:
pip install eos-py
I get the following error while trying to import the eos library (inside python):

import eos
Traceback (most recent call last):
File "", line 1, in
ImportError: arg(): could not convert default argument into a Python object (type not registered yet?). Compile in debug mode for more information.

I would appreciate your help

Hi,
Hmm, I remember getting this exact error a few weeks ago with a different module, actually. I don't remember what the solution was though, but I do remember that I think it had nothing to do with the code itself. It was perhaps an issue with caching, the compiler, Python versions, something like that. I think after I cleared all build caches and did a clean Release/RelWithDebInfo build, it worked fine again.
I don't think this has anything to do with eos specifically. I think you'll just need to play around a bit with the compilation process, start from a clean build directory, perhaps try a different (64-bit) Python version, perhaps clone eos from GitHub and try installing from there. Perhaps even follow the suggestion in the error message, compile in debug mode, and see what happens.
Sorry that I don't remember more or can't be of more direct help. If you find out more, please let us know here.

Thanks for the quick response Patrik,
I will try playing some more with the installation process as you suggested.
Thanks again for sharing this awesome package

I get the error too. I compiled in debug mode and tried to import the eos library (inside python):
import eos
Traceback (most recent call last):
File "", line 1, in
ImportError: arg(): could not convert default argument 'vertex_definitions: akrzemi1::nullopt_t' in method '<class 'eos.morphablemodel.MorphableModel'>.init' into a Python object (type not registered yet?)
I need your kind help. Thank you so much.

I change my default g++/gcc version to g++-7/gcc-7 and rebuild it.
The import works fine now.

@huntkao Interesting! What was the previous gcc version that you used that didn't work?
Another idea I had was to completely remove the build directory, and then rebuild with CMAKE_CXX_STANDARD=17. This is/should be the default actually, but since in your error it mentions akrzemi1::nullopt_t, it looks like something has gone wrong with the build and it's using C++14 fallback types.
I remember now that I think I got this error in a similar scenario, and it also went away once I used a clean build directory, made sure CMAKE_CXX_STANDARD is on 17, and used a recent compiler.
@barhilleli, this should also help you.

My previous gcc version was gcc-6 (6.5.0 actually).
When I changed to gcc-7 and rebuilt, I left *CXX_FLAGS as its default values.
Thanks for your quick response.

@huntkao Thanks! It is indeed still a bit of a mystery. But I think if one makes sure to have a clean build directory and to build with CMAKE_CXX_STANDARD=17 (which is/should be the default), then it works (and should also work with gcc-6, unless there's some regression on that compiler).

Hi,
Does anybody have more update on this ?
I have the exact same issue. I tried all the solutions listed, no success..
Thank you

Anyone encountering this, please specify more information, otherwise it's not helpful. At the very least:

OS (Win/Linux/macOS)
Python version, including 32/64 bit
What eos version, and how was it installed? Via pip?
If you've installed from GitHub sources, give details on how and what you did.

Hi,

MacOS.
Python 3.7.2, 64bit
eos 1.1.1
I tried both pip, and manually building it with cmake :
After cloning the repo, I set "EOS_GENERATE_PYTHON_BINDINGS" to "ON" in the CMakeLists.txt.
I run cmake, then "make python-bindings".
I get the shared library eos.cpython-37m-darwin.so. I then copy it in the site-packages of my Python project.
I am compiling through PyCharm.
I get the same error :
"ImportError: arg(): could not convert default argument 'vertex_definitions: akrzemi1::nullopt_t' in method '<class 'eos.morphablemodel.MorphableModel'>.init' into a Python object (type not registered yet?)"

Many thanks

Hi @BenjBarral,
Okay, I think on macOS/XCode, you will need to make sure that the #if here (and the same in eos/cpp17/variant.hpp) is adjusted, so XCode/AppleClang chooses the C++17 branch. Is __cplusplus not >= 201703L in your AppleClang version? Which XCode version are you using?
If you self-compile, also make sure to do a Release build (I think CMake defaults to Debug but it depends on the generator). You can also do pip install .\eos to install from your local/potentially modified sources.
In any case just make sure it uses the C++17 branch in those files and it should be okay.

Hi Patrik,
Thanks a lot, this was indeed the problem.
Xcode 9 (and AppleClang9) do not fully support C++17.
Updating to Xcode 10 solved the problem.
Thanks again !
Benjamin

Hi @patrikhuber i am able to pip install but not able to import it.
My system info:
**- Ubuntu 16.4, 64 bit

Python 3.5.2
eos 1.1.1
GNU 7.4.0**
I cloned the repo and ran the setup.py to build (does this count as proper way to build?), I did not get any errors, below output of it.

the error persists, could you please tell me what other things should be checked.
Thanks in advance.

Hey @BenjBarral, aah great! That's fantastic, and I'm sure it'll help other people as well! Thanks.
@vineethbabu, if you get any error with akrzemi1::optional or mpark::variant when doing import eos in Python, then you haven't compiled with C++17/CMAKE_CXX_STANDARD 17.

Thank you @patrikhuber for your super quick reply. I changed my default gcc version to gcc 7.4 and installed a newer version of python, now the error is gone.
could you please tell me if there is a way to obtain .obj, mesh and texture from the python code?.
Thanks in advance.

@vineethbabu Great!
For your question, this is very off-topic here. Please have a look at demo.py, the documentation, play around with it and learn, and you'll find out.


Fit Option: Vertically AND Horizontally

Type of reader (if reader related)
image
Description of the new feature / enhancement
I am proposing a new fit option: Vertically and horizontally.
Currently, one can choose between either of these two, but not both at the same time.
Context:
I want to read manga on a tablet. Manga often have double spread pages from time to time. Regular pages are much taller than they are wide in terms of image dimensions, and double spreads are much wider than they are tall.
With a tablet, it makes sense to usually read in portrait mode and then rotate to landscape mode for a double spread (and then back to portrait afterwards).
With all of the four existing fit options, there is no way to have this work without pages either overflowing the screen (meaning that scrolling is necessary) in some dimension(s), or being unnaturally stretched/squished in one dimension (aspect ratio is not maintained).
(All of the following are with Clamp size to window width enabled.)

Free: Works fine in portrait mode (nothing overflows). In landscape mode, both single pages and double spreads overflow vertically.
1:1: Same as Free.
Fit Vertically: Works perfectly in landscape mode with both single pages and double spreads, unnaturally stretches/squishes both when in portrait mode.
Fit Horizontally: Works perfectly in portrait mode with both single pages and double spreads, overflows both when in landscape mode.

To make this "workflow" possible in both orientations without any overflowing or breaking aspect ratio, one more fit option would be needed that preserves aspect ratio and does not overflow the image either.
Or let me phrase this whole thing another way:
It would be great to have a fit option that always fits the image to the screen, without overflowing the screen in either dimension, while also preserving the aspect ratio. This is no matter what dimensions or aspect ratio the screen itself and the image in question have.
An example of a reader that has this fit option: Cubari
Scenario when this would be used?
No response
Supporting information
No response

Sorry, I couldn't understand. Can you show some recordings?


As doctor, I feel there should be link to go to consultation room from page appt_view.html

click on "See Patient Details" takes me to  appt_view.html on doctor dash board

appt_view.html has a link to go to consultation room.


[MATLAB] Create arrow.recordbatch convenience constructor function

Describe the enhancement requested
For parity with how we construct array objects, we should create a construction function called arrow.recordbatch() for arrow.tabular.RecordBatch:
>> t = table(["A"; "B"; "C"], [1; 2; 3])

t =

  3×2 table

    Var1    Var2
    ____    ____

    "A"      1
    "B"      2
    "C"      3

>> rb = arrow.recordbatch(t)

rb =

Var1:   [
    "A",
    "B",
    "C"
  ]
Var2:   [
    1,
    2,
    3
  ]

>> class(rb)

ans =

    'arrow.tabular.RecordBatch'

The arrow.tabular.RecordBatch constructor will accept a scalar libmexclass.proxy.Proxy object instead of a MATLAB table.
Component(s)
MATLAB

take


Battery Usage

I profiled my app (which uses ZXingObjC pod) for energy usage.
While ZXCapture is running (i.e. I call start on a ZXCapture * object like this: [self.capture start]), the total CPU activity is at about 60% (I tested with and without video input, and video input only accounts for a negligible 1-2% of the overall CPU activity...so it's not an issue of high GPU causing high overall CPU activity).
When ZXCapture is not running, the overall CPU activity is at about 10-15%.
I just wanted to bring this up in case there's something that can be done.
I know that there must be a lot of hardcore calculations to do real-time object recognition, but maybe there's some way to minimize / decrease overall CPU activity when ZXCapture is running, which I'm hoping will decrease battery usage as well.

No explanation as to why this is closed - ? Maybe something like, "We've looked into this and there's nothing we can do to reduce battery drain because of how taxing object recognition is on the CPU." But no explanation just leaves me wondering...... :(


Installing for Intel ARC

Hello,
Is there any chance for Intel ARC support being added to the roadmap? Thank you

Hi @ElliottDyson, unfortunately it's not on our roadmap at the moment because there are only few users. Instead, maybe we will consider adding support for Intel CPUs and Gaudi 2 in the future.

Hi @ElliottDyson, unfortunately it's not on our roadmap at the moment because there are only few users. Instead, maybe we will consider adding support for Intel CPUs and Gaudi 2 in the future.

Well if you do end up supporting intel CPUs it should also then inherently support intel ARC GPUs because they both rely on the same implementation, oneAPI. So that would be great. As for the small userbase, I totally understand where you are coming from

+1 for Intel GPU support

@ElliottDyson @itlackey
Hi, just in case you are still looking for, here's the initial patch to support for Intel GPU(ARC & PVC)
https://github.com/vllm-project/vllm/pull/2378
thanks, -yuan


How to use md-file to get a text file?

I have a similar question as  #379.
My use case has an md-file component, and then a "Submit" button that I want to use to send the file to the server in an AJAX call, usually with other props attached to the request body object.
How can I use the md-file component and an event method to get the file reference, but only read the file when I send it by clicking Submit? It seems this is getting out of the md-vue library, but any help would be greatly appreciated.
After reading more about the File / FileReader APIs, I think it should go like so: user clicks md-file, selects a file. The file name is displayed, and user enters stuff in other input fields, then clicks submit, and the method called when the button is clicked, does the FileReader.readAsText() stuff.
The documentation for the required v-model prop of the md-file could be improved, because I believe that just models the name of the file, which isn't the same as the html5 File api. Is that correct?

I believe that you should do these
Template
<md-input-container>
  <label>Single</label>
  <md-file v-model="file.nameFile" @selected="onFileUpload($event)" ></md-file>
</md-input-container>
<md-button class="md-raised" @click.native="submit()">Send</md-button>

JavaScript
:
data () {
 return {
  file: null,
 }
},
method: {
  onFileUpload (evt) {
    this.file = evet[0]
  },
  submit () {
    functionSend({file: this.file, otherParams})
  }
...

thanks that helped

For new release change this
<md-file v-model="file.name" @selected="onFileUpload($event)" ></md-file>
to this
<md-file v-model="file.name" **@md-change="onFileUpload($event)"** ></md-file>


sonobuoy fails to run but report failed tests: 0

What steps did you take and what happened:
sonobuoy fails to run
time="2019-03-05T17:25:52Z" level=info msg="Starting annotation update routine"
time="2019-03-05T17:25:52Z" level=info msg="Starting aggregation server" address=<IP_ADDRESS> port=8080
time="2019-03-05T17:25:52Z" level=info msg="Running plugin" plugin=e2e
time="2019-03-05T17:25:52Z" level=info msg="Running plugin" plugin=systemd-logs
time="2019-03-05T17:29:56Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-worker1 plugin_name=systemd_logs
time="2019-03-05T17:30:17Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-control-plane plugin_name=systemd_logs
time="2019-03-05T17:30:18Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-worker2 plugin_name=systemd_logs
time="2019-03-05T18:30:02Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-worker1 plugin_name=systemd_logs
time="2019-03-05T18:30:02Z" level=warning msg="Got a duplicate result systemd_logs/ipv4-worker1"
time="2019-03-05T18:30:23Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-control-plane plugin_name=systemd_logs
time="2019-03-05T18:30:23Z" level=warning msg="Got a duplicate result systemd_logs/ipv4-control-plane"
time="2019-03-05T18:30:25Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-worker2 plugin_name=systemd_logs
time="2019-03-05T18:30:25Z" level=warning msg="Got a duplicate result systemd_logs/ipv4-worker2"
time="2019-03-05T19:30:09Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-worker1 plugin_name=systemd_logs
time="2019-03-05T19:30:09Z" level=warning msg="Got a duplicate result systemd_logs/ipv4-worker1"
time="2019-03-05T19:30:34Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-control-plane plugin_name=systemd_logs
time="2019-03-05T19:30:34Z" level=warning msg="Got a duplicate result systemd_logs/ipv4-control-plane"
time="2019-03-05T19:30:36Z" level=info msg="received aggregator request" client_cert=systemd-logs node=ipv4-worker2 plugin_name=systemd_logs
time="2019-03-05T19:30:36Z" level=warning msg="Got a duplicate result systemd_logs/ipv4-worker2"
time="2019-03-05T19:30:40Z" level=info msg="received aggregator request" client_cert=e2e plugin_name=e2e
time="2019-03-05T19:30:40Z" level=info msg="Last update to annotations on exit"
time="2019-03-05T19:30:40Z" level=info msg="Running non-ns query"
time="2019-03-05T19:30:41Z" level=error msg="error querying CustomResourceDefinitions: the server could not find the requested resource"
time="2019-03-05T19:30:41Z" level=info msg="Collecting Node Configuration and Health..."
time="2019-03-05T19:30:41Z" level=info msg="Creating host results for ipv4-control-plane under /tmp/sonobuoy/f3ef6572-6a97-4978-9025-98f7a84b02b2/hosts/ipv4-control-plane\n"
time="2019-03-05T19:30:41Z" level=info msg="Creating host results for ipv4-worker1 under /tmp/sonobuoy/f3ef6572-6a97-4978-9025-98f7a84b02b2/hosts/ipv4-worker1\n"
time="2019-03-05T19:30:41Z" level=info msg="Creating host results for ipv4-worker2 under /tmp/sonobuoy/f3ef6572-6a97-4978-9025-98f7a84b02b2/hosts/ipv4-worker2\n"
time="2019-03-05T19:30:41Z" level=info msg="Running ns query (default)"
time="2019-03-05T19:30:43Z" level=info msg="Running ns query (heptio-sonobuoy)"
time="2019-03-05T19:30:45Z" level=info msg="Collecting Pod Logs..."
time="2019-03-05T19:30:48Z" level=info msg="Running ns query (kube-public)"
time="2019-03-05T19:30:50Z" level=info msg="Running ns query (kube-system)"
time="2019-03-05T19:30:52Z" level=info msg="Results available at /tmp/sonobuoy/201903051725_sonobuoy_f3ef6572-6a97-4978-9025-98f7a84b02b2.tar.gz"
time="2019-03-05T19:30:52Z" level=info msg="no-exit was specified, sonobuoy is now blocking"

However it reports that has 0 failed tests
# sonobuoy retrieve
201903051725_sonobuoy_f3ef6572-6a97-4978-9025-98f7a84b02b2.tar.gz
# sonobuoy e2e<PHONE_NUMBER>25_sonobuoy_f3ef6572-6a97-4978-9025-98f7a84b02b2.tar.gz
failed tests: 0

but no test ran
sonobuoy e2e --show=passed<PHONE_NUMBER>25_sonobuoy_f3ef6572-6a97-4978-9025-98f7a84b02b2.tar.gz

passed tests: 0

What did you expect to happen:
sonobuoy should report an error or indicate that didn't run any test at all. Current message is misleading and gives the false impression that everything went right.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Sonobuoy version: v0.13.0
Kubernetes version: (use kubectl version): v1.11.0
Kubernetes installer & version: kind 0.2.0-alpha & v1.13.4
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Sonobuoy tarball (which contains * below)

That is very misleading; and in an automated env you wouldn't have seen that very quickly.
Could you untar that and provide the plugins/e2e/results files? I'm curious both what the actual logs from ginkgo show (the e2e.log file) and also what junit it produced (junit_01.xml)

actually, that folder doesn't exist
# ls
hosts  meta  podlogs  resources  servergroups.json  serverversion.json

Thanks @aojea !
That first case (missing results entirely) should clearly error and is an easy case to handle.
The latter is fine IMO; thats just how ginkgo and any other test framework I've seen works. If nothing fails, it passes.

@johnSchnake should this fail because the file doesn't exist?

Yes, I'd say that should fail if it can't find the file. It seems to just provide a predicate which if the filename matches the given file it extracts it. If it never matches anything it still passes without a problem though.


Ecosystem model diagram refers to relationship which is not implemented

Expected Behavior

High-level ecosystem model diagram represents available relationships.
Current Behavior

In developing the system diagram map in #4502 , I realized that the dependsOn example shown in the sample model is not fully implemented in Backstage's system model. There are partial aspects of it but it seems incomplete?

Possible Solution

Either:

Update the diagram to remove dependsOn as a relationship type until a future time in which this relationship is fully provided by the model.
Update the model to fully support it.

Steps to Reproduce

View the docs
Check the model types in the system: https://github.com/backstage/backstage/search?q=dependsOn and https://github.com/backstage/backstage/search?q=RELATION_DEPENDS_ON

Context

I may be wrong in terms of how it is implemented, but it does not seem to me that it would fully work, even if it's roughed out in some constants?
Your Environment

NodeJS Version (v12):
Operating System and Version (e.g. Ubuntu 14.04):
Browser Information:

Right! I think the relation is sound, to the best of my knowledge it's the one we decided to use for this purpose. So your solution number 2 would be appropriate. This is now a feature request ticket! :)
We'd have to add a spec field to be able to either

point out resources that your component depends on, here and here, or
from a resource point out the components that depend on you, here and here

Or actually both could be supported! To fit different mental models or availability of source data for the field.
And then to update the relevant relation-generating processor.

I'd be quite interested in this, as it could help us model some of the resources created via our infrastructure as code projects and the dependencies on them.  I might take a shot at this if I have some free time next week.

Consider it yours!  Thanks, @jmgrimes. Feel free to reach out and ask for help from the community. I'll note the System Diagram, which was now merged in, would also need manual updating to support the new relationship type as well. (and potentially a better way to reflect multiple types, period!)

I was able to get the catalog model updated for both Component and Resource, and updated the entity processor to create the proper relationships.  Adding the relationship support to the graph appears to have worked, but there's a layout issue I'm encountering with the DependencyGraph component that I'm not able to sort out on my own.  Because of the specific model being used, I'm encountering some clipping on one of the edges.

@jmgrimes Agreed; there are definitely some formatting issues with SystemDiagram, some of which may be inherent to the library used for the DependencyGraph component itself. I wasn't really able to get it to better dynamically fit. Hoping other iterations and bug fixes can continue to improve it for sure!  But your adding support for your new modelling type is wonderful, so thx for that!

Yeah, I found when trying to dig into the right set of options to change to fix the clipping, I was in a bit over my head haha.  I'll poke around and see if there's anything else that I might be able to help with.  Thanks!


Multi-line q string with blank lines doesn't translate properly

Multi-line q string with blank lines doesn't translate properly.  For example:
my $prefix = q~
line 1

line 3
~;

Fixed in v0.012


Why 'override' responses for some of the status codes only?

Hi Guys,
We were building a custom plugin to do some checks and whenever I tried to respond with error code "500", I was not able to pass my custom message. Digging deeper, I stumbled upon this block here -
https://github.com/Kong/kong/blob/28e4baf323a3b01ede1be4301fd490408a1e5d6c/kong/tools/utils.lua#L917
Just wondering why is kong overriding response messages for only three status codes i.e. 405, 500 and 502 ?

Hello, this has been somewhat difficult to dig out because the code responsible for generating/providing response bodies has moved several times in the git history.
I identified the first commit which "overrides" the response body, here:
https://github.com/Kong/kong/commit/d2c50f3f2c159b005aadd93bc1eadc7a9dee8885
Other subsequent commits add similar "overrides" for the methods you have mentioned.
Unfortunately neither the commit message nor the Changelog for Kong 0.3.0, where this change was shipped, add an explanation for this. But I think I can venture an explanation.
The 500 status in particular is easy to understand. It is a "default error code".
The use you are trying to give to the 500 status code is the expected usecase. Your code is actively detecting a condition and returning the status code.
However, there are situations that the Kong source code might not be prepared to expect. Exceptions, if you will. Consider the following: For some combination of circumstances (maybe a bug) Kong is expecting to read and use a row from the database via a SELECT, but instead of getting it it gets an error message. Kong can't continue so it must abort the current request and return an error code. 500 is the "default" http status code for such responses, when no other status code has been specified.
Now consider what the response body should be. A naive approach would be returning "whatever error message has caused the problem". In the previous case, that would be the database's error message. You can probably see what the problem is now: said message could be The query 'SELECT id from Users where id=123456 and secret_code_that_shouldnt_be_public=55667' returned returned no rows. This kind of "unmanaged" response is thus very dangerous, since it can be used to reveal private information.
So that's what the override is there for: it is the last-line of defense against revealing private information by mistake. The cost of that defense is that a 500 message always will return Unknown server error in the response body. You can still print as much valuable information as you can on the logs, for the admins.
I'm pretty sure that's what's happening here because I've seen it done before. Rails, for example, handles 500 errors like this when deployed to a production environment.
I am slightly less sure about why 405 and 502 are also part of the overrides. I suspect they are used as default on certain occasions, and as a result they might "leak".  So the way to return information on those is by printing it in the log. This is my suspicion, I am afraid I don't have the time to confirm this.
In any case, I hope this explanation satisfies your curiosity.
I am closing this issue for now, if you need to discuss it further, please reopen.


powershell command update needed

[Enter feedback here]
In the Create the resource section, there are command examples for Get-AzExpressRoutePortsLocation in items 3 and 4. In order to get the view in the Example output, the command needs to be updated to pipe the command to format-list (fl). So the command for section 3 should be:
Get-AzExpressRoutePortsLocation | format-list

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

ID: 53b762f9-8c20-3648-f229-fe9580112fa7
Version Independent ID: 2066f90a-8539-5f6d-bebb-18bbd6c81d7f
Content: Azure ExpressRoute: Configure ExpressRoute Direct
Content Source: articles/expressroute/expressroute-howto-erdirect.md
Service: expressroute
GitHub Login: @duongau
Microsoft Alias: duau

@cchapin2020
Thank you for your feedback! We will review and update as appropriate.

@cchapin2020
Thank you for the feedback. I have made the suggested change and it will go live soon.


Layout does not minimize the width of the diagram

homotopy_io_export (6).zip

This is what this looks like now, so I think it's fixed.


add saga420/temporal-encryption-converter

URL: saga420/temporal-encryption-converter

The Temporal Encryption Converter is a comprehensive Go package for Temporal workflow engine, offering strong encryption/decryption for payloads and context propagation across workflows. It uses asymmetric cryptography for secure client-worker communication, supports key generation/derivation, employs AES256_GCM_PBKDF2_Curve25519 and XChaCha20_Poly1305_PBKDF2_Curve25519 encryption algorithms, and utilizes ZLib pre-encryption compression for payload optimization. Available for contributions under the MIT License.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.saga420 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Unexpected go to definition on pressing Ctrl button

Hi.
I setup the monaco editor for my custom language with definition provider. Whenever I press Ctrl while hovering a symbol, it jumps to the symbol's definition. How to disable this behavior?

Thank you so much.
monaco-editor version: 0.21.2
Browser: Chrome
OS: Ubuntu

Thanks for reporting this bug! Can you provide an example for the monaco playground so I can try to reproduce your issue?

Hi. Thank you for replying. I cannot find the playground link. I use this API https://microsoft.github.io/monaco-editor/api/interfaces/monaco.languages.definitionprovider.html#providedefinition

This is the playground: https://microsoft.github.io/monaco-editor/playground.html

Hi. I am so sorry. I figured out that was my bug. The monaco-editor is working fine. Thank you for replying.


Merge all misc commands into single CLI

any backwards compatibility concerns?

i don't think any one other than me is using these tools to debug


Incompatible with slick 1.5.9 in Safari?

The slick lightbox seems to be buggy when using with current slick.js (1.5.9) in Safari/iOS.
In all our test cases we couldn't swipe within the lightbox until the first element/image has been changed via prev/next-buttons.
The DOM changes on swipe, but still the first element is visible.

I also have an issue with Safari - when using "fade: true" the lightbox becomes absolutely unusable (= lagging). When not using the fade animation everything is alright.

I have problems as well - slick-lightbox is sadly not usable at all with slick versions newer than 1.5.5 and lots of slides in Safari.

As of now, it's not compatible with 1.5.9 in Safari.
Slick introduced the useTransform option, which is buggy. See related issue.


rust-s3 returns Ok result when garage responds with error

rustc 1.79.0-nightly
rust-s3 master branch
garage 1.0.0
let result = self.bucket
  .put_object_with_content_type(&key, &buffer, "image/avif")
  .await;
dbg!(result);

result = Ok(
    ResponseData {
        bytes: b"",
        status_code: 400,
        headers: {
            "content-length": "268",
            "content-type": "application/xml",
            "date": "Tue, 23 Apr 2024 06:33:58 GMT",
        },
    },
)

OOps, I just realised there is special feature fail-on-error!. Sori.


DiscordAPIError[10062]: "Unknown interaction" in command "undefined"

Reported by mephisto5558<PHONE_NUMBER>25055775) with bot<PHONE_NUMBER>02710047
DiscordAPIError[10062]: Unknown interaction
at handleErrors ([cwd]/node_modules/@discordjs/rest/dist/index.js:727:13)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async BurstHandler.runRequest ([cwd]/node_modules/@discordjs/rest/dist/index.js:831:23)
at async _REST.request ([cwd]/node_modules/@discordjs/rest/dist/index.js:1272:22)
at async ButtonInteraction.update ([cwd]/node_modules/discord.js/src/structures/interfaces/InteractionResponses.js:237:5)
at async ButtonInteraction.messageComponentHandler ([cwd]/Utils/componentHandler.js:29:39)
at async Client. ([cwd]/Handlers/slash_command_handler.js:91:11)

button click spam on clickcounter


OPNSense hangs after a while

After some hours of operation, i can't login into the administration panel anymore, the web address of the panel is simply not responding. A console login also fails, and it shows some strange errors:

In most cases, data transfer through the firewall is still possible.
After a hard reboot, the OS is repairing the filesystem and everything is working again...for a few hours.
OPNSense is version 19.1 on PC Engines APU 4c4 with AMD Embedded G series GX-412TC processor.

broken sd card?

We hear APU4 needs BIOS updates, SATA operation is flaky... cc @lattera

No, I'm using a ssd disk.
The former version of OPNSense worked without errors, if that helps.

quite often these kind of issues are hardware related, if connected via a sata cable, it might be worth a try replacing the cable.

I reached out to PC Engines directly. They verified it's a BIOS issue. They said downgrading to the latest mainline BIOS should resolve the issue.

(BTW, I'm in the same boat with multiple APU4 devices. I haven't been able to successfully downgrade the BIOS, yet.)

Sorry for being a little off-topic here, but I'd like to inform you guys that PCEngines has release a new coreboot (v4.0.23) for the APU2 boards yesterday:
https://pcengines.github.io/
It fixes a pfSense issue and introduces ECC support for the APU's memory.
I have upgraded my BIOS to v4.0.23 and OPNSense 19.1 has been running fine so far with it. Also make sure to use the "Legacy BIOS" for FreeBSD-based OSes like OPNsense.

@klada highly appreciated, thanks for the heads-up! :)

I've now verified that flashing v4.0.23 resolves the mSATA issues.
Here's the procedure I followed:

Download HardenedBSD installation media
Write HardenedBSD installation media to USB memstick with dd
Boot the installation media on the APU4c4
Partition the mSATA with a single 2gb partition, type freebsd-swap
swapon /dev/ada0p1
Set up a memory-backed filesystem (3gb in size): mdconfig -a -s 2g && newfs md0 && mount /dev/md0 /mnt && mkdir /mnt/update && mkdir /mnt/root
Populated the filesystem by running hbsd-update (as if I were creating a new jail from scratch): hbsd-update -V -T -t /mnt/update -d -n -r /mnt/root -U
Mount devfs in the new filesystem: mount -t devfs devfs /mnt/root/dev
chroot into the new filesystem: chroot /mnt/root
pkg install flashrom
fetch --no-verify-peer http://pcengines.ch/file/apu4_v4.0.23.rom.tar.gz
tar -xf apu4_v4.0.23.rom.tar.gz
flashrom -p internal:boardmismatch=force apu4_v4.0.23.rom
shutdown -p now

Once the system powers off, you can now boot the OPNsense 19.1 installation media and proceed with installation. :)
I now have to follow these steps again on two other APU4 devices I'm deploying. :)

You may also do this from an running instance of opnsense (starting from step 10, which worked in my case), as described here:
https://www.bsdforen.de/threads/pc-engines-apu2-bios-update.33587/

Yup. In my case, I didn't have a running install of OPNsense.
So, instead of the large amount of steps above, you could install OPNsense 18.7, flash the v4.0.23 BIOS, then move forward with the install of or upgrade to 19.1. :)

@tadrien is this still an issue for you? Should we keep the ticket open?

The argument -w is missing in step 13:
flashrom -p internal:boardmismatch=force -w apu4_v4.0.23.rom
Thanks to lattera for sharing the steps.

Whoops! Good catch! I'll edit the comment to add that. Thanks!

@lattera Currently not. I'm currently running my APU on the latest bios (<IP_ADDRESS>) and everything seems to be working fine, though i wasn't able to have my OPNSense running for a longer period of time since the bios upgrade.
The newest bios also gave me a nice boost in performance.
Thanks a lot for your help so far!


Add English as fallback for TMDB text

for https://github.com/elgatito/plugin.video.elementum/issues/1009
see the difference:

I think we will need a generic approach to get translations for []languages. But should be fine for now.
Thanks


File Upload summary (File name/link, who uploaded it, when etc...) (Ref - 38346)

@AshokPeddakotla-MSFT this is regarding the discussion on issue -  38346.
Is there any sample that show catching file upload notification on Azure function?
There is sample given here however it using .net console app which is running for test purpose however in real world we cannot keep azure function running just to poll the iot hub to receive file upload notification.
What is the recommended approach in this case? We need to gather File name, link to download it, device id and date time of upload.
Thanks,
Bhupal

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

ID: dbaf3777-34b8-b195-691f-2853c9cbe04a
Version Independent ID: 3b0a4384-e495-16db-1c36-dca0a9f95179
Content: Understand Azure IoT Hub file upload
Content Source: articles/iot-hub/iot-hub-devguide-file-upload.md
Service: iot-hub
GitHub Login: @robinsh
Microsoft Alias: robinsh

@BhupalPatil Thanks for your feedback! We will investigate and update as appropriate.

@BhupalPatil I haven't found an example for your scenario, However, there is FileNotification Class which you include in your code to get the details. It has the following properties.
BlobName
BlobSizeInBytes
BlobUri
DeviceId
EnqueuedTimeUtc
LastUpdatedTime
Hope this helps.

@AshokPeddakotla-MSFT , details are available however the blockers is how the notification will be caught at azure function side. very obvious design aspect, not sure why there is not solution provided for this scenario.

@BhupalPatil You can provide your feedback on the UserVoice for this feature.
As this is out of the scope of this document let me kindly ask that you open a new issue on MSDN or Stack Overflow so the community can help answer it.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Zod errors when executing procedures.

Describe the bug
When trying to execute any procedure, Zod takes it as if none of the parameters had been sent and throws 404.
To Reproduce
Steps to reproduce the behavior:

having a server with fastify v4
register a route get

app.get('/panel', async (req, res) => {
      res
        .status(200)
        .type('text/html')
        .send(
          renderTrpcPanel(appRouter, {
            url: 'http://localhost:6902/api',
          }),
        );
    })

to render the panel
3. try to execute any procedure with inputs
Expected behavior
that the executed procedure correctly receives the parameters
Screenshots

Additional context
I think the problem could be due to the way the payload is loaded using first the “json” key before the values.

Hey @agulencina96 I think you are right that is probably has something to do with the "json" key, as #26 is also having problems with it. However, I was able to get the panel running using fastify in this repo which is based on the trpc fastify example.
const server = fastify({ logger: dev });

server.get("/panel", async (_, reply) => {
		const html = renderTrpcPanel(appRouter, {
			url: "http://localhost:2022/trpc",
			transformer: "superjson",
		});
		reply.type("text/html").send(html);
	});

I think it is possible the issue could have something to do with superjson, but I'm honestly not sure since I'm not super familiar with fastify.
Could you create a repo that reproduces the issue, or at the very least send the file in which you initialized trpc and the fastify server? I doubt I'll be able to debug and fix the issue if I can't reproduce it.

I'm using express and I got the same error

Hey @lcjnil, first can you try updating to the the 1.0.5 release and see if your issue persists. It is possible that the changes in #28 fixed your issue.
If not, it would be super helpful if you could make a minimal reproduction of your issue in a public repo or on something like stackblitz? It would help a ton in solving the issue!

Just to mention I am also having similar issues but only when I am not using superjson.
I can confirm using 1.0.6 release works fine when I configure tRPC with superjson.
I think the problem lies in that, when superjson is enabled, the input needs to be in this format nested under a json key:
{"0":{"json":{"id":"abc"}}}

Whereas without superjson, the json key needs to be omitted:
{"0":{"id":"abc"}}


Replace update tab in Bottom bar

Describe your suggested feature
Please make an option in setting by which bottom bar update can be replaced by history tab.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to the latest version.
[X] I will fill out all of the requested information in this form.

Or add both and add an option to display only their icons without labels. :D

Add history In start scr3en view will be helpful

Any update on this?

I use this feature a lot back when I was using tachiyomi. Now, I need to click two times to get there. Having the option to change in between would be perfect.

I see it's one of the common requests, I hope the developers will  look into this issue

This has been fixed/added in the preview
https://github.com/aniyomiorg/aniyomi-preview
Steps to shift to this build:-

Create a backup from "Backup and Restore"
Download the preview apk and install it
Restore the backup you created before in the same tab "Backup and Restore"


Update the number of parameters

As calculated by:
from transformers import AutoModel
model = AutoModel.from_pretrained("bert-base-multilingual-cased")
print(f"{sum(p.numel() for p in model.parameters()):,}")

@googlebot I signed it!

The same change should be done here

Bro, I got a problem from a fake google developer in Indonesia, check who is entering my server, if there is an abuse, just kick it or report it to central google immediately handle it
Dikirim dari Redmi 4A saya
Pada googlebot @.***>, 21 Sep 2020 8.33 PM menulis:
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.

What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA datahttps://cla.developers.google.com/clas and verify that your email is set on your git commitshttps://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoothttp://go/cla#troubleshoot (Public versionhttps://opensource.google/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA datahttps://cla.developers.google.com/clas and verify that your email is set on your git commitshttps://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must also be attached to your GitHub accounthttps://github.com/settings/emails.

ℹ Googlers: Go herehttps://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fgoogle-research%2Fbert%2Fpull%2F1150 for more info.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/google-research/bert/pull/1150#issuecomment-696117098, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFQHH7B56IYG3FTCKMKZE4TSG5I35ANCNFSM4RURIIXA.


German phone numbers are not formatted correctly

The library formats the following number +491726123456 incorrectly:
Expected: +49<PHONE_NUMBER>
Actual:      +49 1726 123456
I tried to use Google's libphonenumber library and it formats the phone number as expected

I checked the issue on the source code. The problem is in this line:
https://github.com/ibrahimsn98/PhoneNumberKit/blob/b607fb18230a618805558fe13077f48a7a39b7f5/lib/src/main/java/me/ibrahimsn/lib/core/Core.kt#L45
It returns a sample phone number with the given country code, but Germany has different formats based on the national phone number.


add props to target native Input autocomplete attribute

Feature Request

Please add autocomplete on/off through a prop, this will be helpfull !

@warlock1996 try inputOptions.autocomplete: https://iamstevendao.github.io/vue-tel-input/documentation/props.html

@iamstevendao sorry , i noticed later , thanks


Fix get-user.decorator.ts to prevent error in getPlayers()

This update fixes the @getUser() decorator to work with NestJS 7.x. Previously hitting the /api/player endpoint with a GET request would return an error that user.email was not defined.

I tested these changes and it appears to fix the issue.


Cannot find module '@kbn/utils'

while executing command
yarn kbn bootstrap - I am facing issue as Cannot find module '@kbn/utils'
Please guide how can I fix this issue,
--------- Here are the complete log details
root@is-OptiPlex-5050:/home/kibana/kibana# yarn kbn bootstrap
yarn run v1.22.5
$ node scripts/kbn bootstrap
succ [bazel_tools] all bazel tools are correctly installed
info [bazel] INFO: Invocation ID: 8242e00a-efe8-47b8-8b8e-32887b7c979b
info [bazel] INFO: Analyzed target @nodejs//:yarn (0 packages loaded, 0 targets configured).
info [bazel] INFO: Found 1 target...
info [bazel] INFO: Elapsed time: 0.166s, Critical Path: 0.09s
info [bazel] INFO: 1 process: 1 internal.
info [bazel] INFO: Running command line: /root/.cache/bazel/_bazel_root/59c8214d5247a4c46712ee45ad74ea44/external/nodejs_linux_amd64/bin/yarn
info [bazel] info No lockfile found.
info [bazel] $ node ./preinstall_check
info [bazel] [1/5] Validating package.json...
info [bazel] [2/5] Resolving packages...
info [bazel] warning Resolution field "<EMAIL_ADDRESS>is incompatible with requested version "<EMAIL_ADDRESS>info [bazel] [3/5] Fetching packages...
info [bazel] info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info [bazel] info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info [bazel] [4/5] Linking dependencies...
info [bazel] warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>info [bazel] warning "@elastic/github-checks-reporter > @octokit/rest ><EMAIL_ADDRESS>has unmet peer dependency "@octokit/core@>=3".
info [bazel] warning "@storybook/addon-storyshots ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>info [bazel] warning " ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>info [bazel] warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>info [bazel] [5/5] Building fresh packages...
info [bazel] success Saved lockfile.
info [bazel] INFO: Invocation ID: 27877a14-912f-4e80-aeca-393a85c1e05a
info [bazel] INFO: Analyzed target //packages:build (1 packages loaded, 1 target configured).
info [bazel] INFO: Found 1 target...
info [bazel] INFO: Elapsed time: 0.533s, Critical Path: 0.10s
info [bazel] INFO: 1 process: 1 internal.
info [bazel]
succ yarn.lock analysis completed without any issues
info [@kbn/apm-utils] running [kbn:bootstrap] script
info [@kbn/config-schema] running [kbn:bootstrap] script
info [@kbn/tinymath] running [kbn:bootstrap] script
info [@kbn/utility-types] running [kbn:bootstrap] script
succ [@kbn/tinymath] bootstrap complete
succ [@kbn/apm-utils] bootstrap complete
succ [@kbn/utility-types] bootstrap complete
succ [@kbn/config-schema] bootstrap complete
info [@kbn/babel-code-parser] running [kbn:bootstrap] script
info [@kbn/std] running [kbn:bootstrap] script
info [@kbn/utils] running [kbn:bootstrap] script
succ [@kbn/utils] bootstrap complete
succ [@kbn/std] bootstrap complete
succ [@kbn/babel-code-parser] bootstrap complete
info [@kbn/apm-config-loader] running [kbn:bootstrap] script
info [@kbn/dev-utils] running [kbn:bootstrap] script
info [@kbn/legacy-logging] running [kbn:bootstrap] script
info [@kbn/logging] running [kbn:bootstrap] script
succ [@kbn/logging] bootstrap complete
succ [@kbn/apm-config-loader] bootstrap complete
succ [@kbn/legacy-logging] bootstrap complete
succ [@kbn/dev-utils] bootstrap complete
info [@kbn/ace] running [kbn:bootstrap] script
info [@kbn/analytics] running [kbn:bootstrap] script
info [@kbn/config] running [kbn:bootstrap] script
info [@kbn/es] running [kbn:bootstrap] script
succ [@kbn/config] bootstrap complete
info [@kbn/i18n] running [kbn:bootstrap] script
succ [@kbn/es] bootstrap complete
info [@kbn/plugin-generator] running [kbn:bootstrap] script
succ [@kbn/analytics] bootstrap complete
info [@kbn/storybook] running [kbn:bootstrap] script
succ [@kbn/ace] bootstrap complete
info [@kbn/telemetry-tools] running [kbn:bootstrap] script
succ [@kbn/i18n] bootstrap complete
succ [@kbn/telemetry-tools] bootstrap complete
succ [@kbn/plugin-generator] bootstrap complete
succ [@kbn/storybook] bootstrap complete
info [@kbn/docs-utils] running [kbn:bootstrap] script
info [@kbn/interpreter] running [kbn:bootstrap] script
info [@kbn/monaco] running [kbn:bootstrap] script
succ [@kbn/interpreter] bootstrap complete
succ [@kbn/docs-utils] bootstrap complete
succ [@kbn/monaco] bootstrap complete
info [@kbn/ui-shared-deps] running [kbn:bootstrap] script
succ [@kbn/ui-shared-deps] bootstrap complete
info [@kbn/optimizer] running [kbn:bootstrap] script
succ [@kbn/optimizer] bootstrap complete
info [@kbn/plugin-helpers] running [kbn:bootstrap] script
info [@kbn/test] running [kbn:bootstrap] script
(node:13374) UnhandledPromiseRejectionWarning: Error: Cannot find module '@kbn/utils'
at webpackEmptyContext (/home/kibana/kibana/packages/kbn-pm/dist/index.js:63260:10)
at CiStatsReporter.getKibanaUuid (/home/kibana/kibana/packages/kbn-pm/dist/index.js:59733:55)
at CiStatsReporter.timings (/home/kibana/kibana/packages/kbn-pm/dist/index.js:59663:68)
at runCommand (/home/kibana/kibana/packages/kbn-pm/dist/index.js:59570:24)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async Module.run (/home/kibana/kibana/packages/kbn-pm/dist/index.js:242:3)
(Use node --trace-warnings ... to show where the warning was created)
(node:13374) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:13374) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Done in 146.64s.

Duplicate of https://github.com/elastic/kibana/issues/94779


Update Dropbox to Version 11.25.0

dropbox version 11.25.0

check the upstream
https://github.com/dropbox/dropbox-sdk-python/tree/v11.25.0

check the pinnings
https://github.com/dropbox/dropbox-sdk-python/blob/main/tox.ini
https://github.com/dropbox/dropbox-sdk-python/blob/main/setup.py
https://github.com/dropbox/dropbox-sdk-python/blob/main/setup.cfg
https://github.com/dropbox/dropbox-sdk-python/blob/main/ez_setup.py

check the changelogs
https://github.com/dropbox/dropbox-sdk-python/blob/v11.25.0/UPGRADING.md
There were no breaking changes mentioned in the documentation

additional research
https://github.com/conda-forge/dropbox-feedstock/issues
There were no open issues in conda forge

verify dev_url
https://github.com/dropbox/dropbox-sdk-python

verify doc_url
https://dropbox-sdk-python.readthedocs.io/en/latest/

pip in the test section

veriy the test section

additional tests

In order to test the dropbox package version 11.25.0 the folowing
command was used:
conda build dropbox-feedstock --test
the test result was the following:
All tests passed

The package will be kept as an architecture specific package for now.

Why?

The package will be kept as an architecture specific package for now.

Why?

While we want to keep our recipes consistent with the upstream, we also might want to keep the rest of the packages consistent with previous versions until the next mayor version bump. Once we have the next version bump we will be able to update the package to noarch.

... we also might want to keep the rest of the packages consistent with previous versions until the next mayor version bump. Once we have the next version bump we will be able to update the package to noarch.

When/how was this decision to hold off until the next major version made?
The upstream feedstock has been noarch: python since 8.4, so I'm not understanding the value of waiting until version 12.0 to switch.

... we also might want to keep the rest of the packages consistent with previous versions until the next mayor version bump. Once we have the next version bump we will be able to update the package to noarch.

When/how was this decision to hold off until the next major version made?
The upstream feedstock has been noarch: python since 8.4, so I'm not understanding the value of waiting until version 12.0 to switch.

You have made a very valid point. There might not be that much benefit on waiting until version 12.0. Taking that into account, we probably should change the package to noarch now since we are making a minor version change.


Map vs Object Literal

new Map() vs {} setting and retrieving properties

PR open https://github.com/haensl/js-performance/pull/42

Done in v1.10.0


Removed dependency from H2 and JPA

Since this new version of the giftcard-demo uses the AxonServer instead of H2, I removed the dependency to H2 and JPA by replacing the EntityManager with a HashMap. Any feedback is welcome!

First and foremost, sorry for the long radio silence here @ppedrozzi.
I'm just taking the time to check all the outstanding PR's for this sample project, and yours has been on here already for the last two years...
Note that I do see value in your pull request by the way.
So if you are up for updating your PR to match the current version of master, that would be great.
If you do not have time to work on this right now, that's also fine; just let us know what your availability is on the matter and we can discuss things further.

In absence of a response, I've taken the liberty to merge your changes after some local adjustments for the merge conflicts.
I am trusting that this was a fair course.
Feel free to keep commenting, @ppedrozzi, if you think other changes should be included.

Hi Steven, sure! Sorry for not replying
Happy new year!
Pietro
Il giorno ven 31 dic 2021 alle ore 11:38 Steven van Beelen <
@.***> ha scritto:

In absence of a response, I've taken the liberty to merge your changes
after some local adjustments for the merge conflicts.
I am trusting that this was a fair course.
Feel free to keep commenting, @ppedrozzi https://github.com/ppedrozzi,
if you think other changes should be included.
—
Reply to this email directly, view it on GitHub
https://github.com/AxonIQ/giftcard-demo/pull/12#issuecomment-1003340291,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7B5W5YD5TOLI3563T446TUTWB3XANCNFSM4HI6NU3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Update docs following merge of proxy & router

Related to #1642, and following #2936.
Note that I've also deleted trailing whitespaces for the files I have modified.
This is done as a separate commit for ease of review, but should be squashed before merging.

@abuehrle do you want to review?


Add Disasters

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[-] Your submissions are formatted according to the guidelines in the contributing guide
[-] Your additions are ordered alphabetically
[-] Your submission has a useful description
[-] The description does not end with punctuation
[-] Each table column should be padded with one space on either side
[-] You have searched the repository for any relevant issues or pull requests
[-] Any category you are creating has the minimum requirement of 3 items
[-] All changes have been squashed into a single commit

@jacklaurencegaray thanks for contribution! Can you resolve the conflict, please?


Document differences in using Python2 and Python3

Client(decode_responses=True)

I don't know whether to be happy that I've found this, after two hours banging my head against the screen, or to be upset that almost a year later such an essential bit is missing from the doc.

@toyg i'm sorry you wasted two hours. But it's documented, in fact it's right on the first example in the https://github.com/RedisJSON/redisjson-py/blob/master/README.md.
   from rejson import Client, Path

   rj = Client(host='localhost', port=6379, decode_responses=True)

   # Set the key `obj` to some object
   obj = {
       'answer': 42,
       'arr': [None, True, 3.14],
       'truth': {
           'coord': 'out there'
       }
   }

I think it should be mentioned that it’s a compulsory parameter when using Python3, rather than something advanced and optional that may or may not be required. I would dare say it should be True by default, now that py2 is dead, but I guess that’s an opinion and switching it now might break old code.

I've submitted #37 to fix (docs only) this - please let me know if acceptable.
After spending some cycles mulling on this, I think we can relax this requirement and have the client do the decoding of rejson's replies if it wasn't initialized with decode_responses=True. That would have probably been the right course to take all that time ago but changing now will break things :/
Perhaps it is something worth considering for the next major release @gkorland


[Enhancement] Add Albums

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

I found in this app the best compromise of features and UX, but in my opinion adding custom albums it would make it even better!
to happen.

Describe the solution you'd like
A clear and concise description of what you want to happen.

Adding albums the same way Google photos does:
Create an album and give it a name
Put inside of it any picture you want
Possibly it would be nice to have a way to lock the album via either a pin or the preferred system unlock option (e.g biometrics) and to export the albums in some way so whenever someone uninstalls and reinstalls the app or switches to a new phone, they can import back the albums.

Additional context
Add any other context or screenshots about the feature request here.

There's already the possibility to do something like that with favorites, but I'd like to group some of my pictures in different albums without having to move them in different directories.

I second that.
I think it would be best to rename the current "album" feature to "Folders" or something similar, and create a new feature allows for custom albums. This way those two features are seperate.

I second that.
I think it would be best to rename the current "album" feature to "Folders" or something similar, and create a new feature allows for custom albums. This way those two features are seperate.

This is a great idea to me! Though I'd probably put them in the same tab as folders with a button for albums or even just a scrollable horizontal list with all the albums, something like Google photo already does with folders in the "library" tab!

@IacobIonut01 I am willing to try work on this.
However, it seems that the "album"-nomenclature is pretty deep baked into the code. Is there any deeper meaning behind that choice? I have replaced album with folders in my initial test, but maybe that is not a good idea.

@IacobIonut01 I am willing to try work on this.
However, it seems that the "album"-nomenclature is pretty deep baked into the code. Is there any deeper meaning behind that choice? I have replaced album with folders in my initial test, but maybe that is not a good idea.

The app reflects the data that the MediaStore uses, so the 'albums' are indeed folders. Working on this feature would require way more time and effort than actually opening a File Manager or actually from the Gallery App (with move/copy actions) to create new 'albums' and categorise the pictures you like in them directly. From my honest opinion adding both 'Folders' and Custom Albums is useless unless they are dynamically made (location based albums etc - which are planned)

From what i can see, i imagine custom albums working like "Favorite". It is always created from the database, and that database only contains a album<->media-id pairing. On a first glance, that doesn't seem like too much effort to me. Though: I have not yet worked with compose, so that part is new to me and i might missjudge the actual effort.
Also, dynamic albums don't really solve this specific usecase.
For me: I want to group pictures together that dont have a connection besides what it is in them. Also no exif-locations ;) The only way i can achieve that is by manually adding to an album.
If you generally dont oppose to this feature, i am going to give it a try. Though if you really don't want it, i'd like to know so that i dont spend time on something that is not helpful to you.

@IacobIonut01 I am willing to try work on this.
However, it seems that the "album"-nomenclature is pretty deep baked into the code. Is there any deeper meaning behind that choice? I have replaced album with folders in my initial test, but maybe that is not a good idea.

The app reflects the data that the MediaStore uses, so the 'albums' are indeed folders. Working on this feature would require way more time and effort than actually opening a File Manager or actually from the Gallery App (with move/copy actions) to create new 'albums' and categorise the pictures you like in them directly. From my honest opinion adding both 'Folders' and Custom Albums is useless unless they are dynamically made (location based albums etc - which are planned)

In my opinion custom albums wouldn't be useless considering that, first of all, is a feature that galleries like Google Photo or Photos from Apple allow. Surely dynamic ones based on common Metadata / topic are a good feature to add, but also letting the user choose how to manage the pictures without having to moving them in another directory or having duplicates.

Great!
Then i'll try my best. Also i am going to keep the "album"-naming, and the new custom-album one is going to be that, "custom album" internally, but user facing, i like "your albums"

I don't oppose the feature itself, so you can do it in the end if you would like to, also it wouldn't require any UI changes other than the one required to add/remove/create albums and inserting media to them, most of the work is in the local database. If you fail to do the feature, I will get back to it after the top-priority stuff is done (Photo&Video editor, media-grid rework)

to avoid confusion with "classic" albums (the ones that are folders), maybe calling them collections could be an alternative


Difficulty Reproducing Downstream Task Finetune Results with HF Trainer

Due to my unfamiliarity with using PyTorch Lightning (PL), I attempted to use the Hugging Face (HF) Trainer for pretraining and fine-tuning downstream tasks.

I was able to largely reproduce the results from your paper on pretraining. For example, the following figure shows the training curve for reproducing caduceus-ph. (Although using the original learning rate of 8e-3 was more unstable, I opted to use a learning rate of 1e-4).

However, I encountered significant difficulty when trying to fine-tune downstream tasks using HF. Fine-tuning NT benchmarks with HF is theoretically straightforward, and although my actual implementation differs, you can refer to my self-contained Colab: Colab Link. This Colab is mainly based on the fine-tuning tutorial provided by NT. When fine-tuning downstream tasks, the numerical results are highly unstable, often resulting in NaN losses, and I am completely unable to reproduce the results from the paper. The following figure shows my reproduction of the H4ac results. As you can see, with warmup_steps=500 and a learning rate of 1e-3, I can mostly reproduce the results from the paper (~0.6).

However, in most cases, the results are very unstable. You can observe in the training loss below that the curves representing a loss of 0 indicate occurrences of NaN.

I used AutoModelForSequenceClassification to load the initial weights you provided without adding any conjoin-related parameters. I am wondering if this is the main reason affecting performance, and I am also curious about the impact of conjoin. I did not seem to find related experimental explanations in the paper.

When running your PL-based code, I can mostly reproduce the results, but since the HF code would be much simpler, I would like to understand why my implementation fails to reproduce the results completely. I hope to get your help!

Thank you for your assistance!

In conclusion, I have two main questions:

Can the HF Trainer theoretically be used for pretraining and fine-tuning your model? Using the HF Trainer would significantly simplify the code.
Does directly loading the basic model (without any conjoin-related parameters and reverse complement augmentations) have a significant impact on downstream tasks?

We can observe a very strange phenomenon. The figure shows different learning rates: 1e-3, 5e-4, 1e-4, and 5e-5. Surprisingly, it is the intermediate value of 5e-5 that shows instability.

Sorry, I may need to confirm the issue of numerical instability again. I'm not sure if some random issues in the environment are causing occasional occurrences of {'loss': 0.0, 'grad_norm': nan, 'learning_rate': 0.0, 'epoch': 0.11} situations, which do not seem to be related to the specific learning rate set. Therefore, I just need you to help me understand whether RC augmentation in downstream tasks is critical.

This is a simple experiment to illustrate this issue. Each time I initialize an AutoModelForSequenceClassification and a Linear layer, I notice that the values in the linear layers of AutoModelForSequenceClassification are very unusual, whereas the values in the Linear layer are normal.

Here I provide my experiment code. I tried changing the hardware device but still observed the same phenomenon.
import torch
import torch.nn as nn

from transformers import AutoModelForSequenceClassification

MAX_TRAILS = 10

for i in range(MAX_TRAILS):
    model = AutoModelForSequenceClassification.from_pretrained(
        "kuleshov-group/caduceus-ph_seqlen-1k_d_model-256_n_layer-4_lr-8e-3", trust_remote_code=True)

    # print model.score's max and min value
    score_max_value = model.score.weight.max().item()
    score_min_value = model.score.weight.min().item()
    print(f"Trail {i}: Max value: {score_max_value}, Min value: {score_min_value}")

    # randomly initialize a linear with the same shape as model.score
    linear = nn.Linear(model.score.in_features, model.score.out_features, bias=False)
    # print linear's max and min value
    linear_max_value = linear.weight.max().item()
    linear_min_value = linear.weight.min().item()
    print(f"Trail {i}: Linear Max value: {linear_max_value}, Min value: {linear_min_value}")

Perhaps there is an issue in initializing the linear layers of CaduceusForSequenceClassification. Did you try manually initializing these, or not using this class and simply adding nn.Linear layers as you have above?

Yes. I tried to re-initialize the weight and it worked.

I am still unsure about the exact cause of the bug. Given that the hyenadna codebase does not appear to have such issues, could you provide some suggestions on how to effectively identify and fix initialization bugs?

I am also not sure why HF is not initializing the params of the linear prediction head in a standard way. In any case, I pushed a fix for this where I explicitly init these weights. See here, for example. This should hopefully resolve the issue.


Unexpected value from nativeGetEnabledTags: 0

Hi,
I tried  emulator and physical device(android) to run my app but results are same, it is "Unexpected value from nativeGetEnabledTags: 0". I changed body: in the scaffold as a Text widget then it worked.
I decided to change my "ProductList" codes with my course codes. After that, same results were happened again.
Thank you.

Hi @Saroglu
can you please provide your flutter doctor -v
And a minimal complete code sample to reproduce the problem

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, minimal reproducible sample code (if related), logs and the output of 'flutter doctor -v'  please.


requests_params: Optional[Dict[str, str]] = None should be requests_params: Optional[Dict[str, Any]] = None,

https://github.com/sammchardy/python-binance/blob/1b9dd4853cafccf6cdacc13bb64a18632a79a6f1/binance/client.py#L140

Thanks @son-hoangtien it will be included in the next update


Increase node's z-index on hover

Please describe the feature that you want to propose
My application shows a popup palette when a node is hovered:

The palette is nested into a node so it can be easily positioned relative to the node. But the problem is that it's overlapped by a selected node. Is it possible to increase z-index of a hovered node and all its nested nodes?
The following works partly:
.react-flow__node:hover {
  z-index: 2000 !important;
}

The palette appears fine, but all nested nodes disappears (the attribute in this example):

Another option is to use anchor positioning and don't nest the palette into a node. But it doesn't work with CSS translate
Could you suggest a solution?

A way to increase z-index of a hovered node with all it's contents
A way to replace CSS translate for node positioning by something else
Or the only way is to calculate a palette position based on node position and size?

Simplest solution to this is to utilize the NodeToolbar component.
Here's an example.
Just put your palette into the Toolbar component and you should be good to go.

Works perfect, thanks a lot!