Thanos store extremely slow queries

Thanos, Prometheus and Golang version used:
Docker image: quay.io/thanos/thanos:v0.23.1

Object Storage Provider:
Minio S3 gateway to s3.wasabisys.com with disk caching
Removing Minio cache doesn't make any difference
What happened:
Queries are extremely slow without any obvious reason, making Thanos unusable.

I think it started to happen after testing configuration with vertical compaction. No clue in logs.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Running deployment with Prometheus + sidecar:
Prometheus opts:
          - --storage.tsdb.retention.time=7d
          - --storage.tsdb.min-block-duration=2h
          - --storage.tsdb.max-block-duration=2h
          - --config.file=/etc/prometheus/prometheus.yaml
          - --web.external-url=http://prometheus.domecek:9090
          - --web.route-prefix=/
          - --web.enable-lifecycle

Sidecar:
          - sidecar
          #- --shipper.upload-compacted
          #- --shipper.ignore-unequal-block-size
          - --prometheus.url=http://localhost:9090
          - --objstore.config-file=/etc/prometheus/thanos_bucket.yml
          - --tsdb.path=/prometheus/data
          - --http-address=<IP_ADDRESS>:19191
          - --grpc-address=<IP_ADDRESS>:19090

Store:
        - store
        - --log.level=info
        - --log.format=logfmt
        - --data-dir=/var/thanos/store
        - --grpc-address=<IP_ADDRESS>:10901
        - --http-address=<IP_ADDRESS>:10902
        - --objstore.config-file=/etc/prometheus/thanos_bucket.yml
        - --ignore-deletion-marks-delay=24h

Compactor:
        - compact
        - --wait
        - --log.level=info
        - --log.format=logfmt
        - --objstore.config-file=/etc/prometheus/thanos_bucket.yml
        - --data-dir=/var/thanos/compact
        - --debug.accept-malformed-index
        - --retention.resolution-raw=0d
        - --retention.resolution-5m=0d
        - --retention.resolution-1h=0d
        - --delete-delay=48h
        #- --downsampling.disable
        #- --compact.enable-vertical-compaction

Full logs to relevant components:

Anything else we need to know:

@yeya24 Yes, this is what I have now (see snippets above) - disabled vertical compaction in compactor, disabled compaction in prometheus with 7d local retention. But it didn't fix the issue.
Is there some way to perform cleanup if this is what happened?


Release v0.15.6

[Fix] Fix path to source in source maps.
[Fix] Disable number-literal-format in default Tslint rules.
[Internal] Enable integration with Greenkeeper.
[Internal] Enable integration with Codecov
[Internal] Enable code coverage

Codecov Report

Merging #58 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #58   +/-   ##
=======================================
  Coverage   66.19%   66.19%
=======================================
  Files          35       35
  Lines         840      840
  Branches       87       87
=======================================
  Hits          556      556
  Misses        223      223
  Partials       61       61

Continue to review full report at Codecov.

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


go: Fix AWS AEAD encryption with no additional data

When no additional data is provided (the default across languages),
the Go library is adding a
"encryptionContext": {
  "additionalData": null
}

to the request to KMS, making it incompatible with other languages.

Done. TEST_SRCDIR and TEST_WORKSPACE environment variables in that test makes it really hard to run btw.

Thanks for approving! I don't seem to have permissions to merge though.

Rebased. The change is no longer needed, but I think the test is still worth merging.

Thanks!


[Feature Request] Add in correct libs for drp

Related to a problem?
Just looked at EasyBot.py and the correct libs were not imported
Solution
from discord import Embed, Activity, ActivityType

Describe the Alternatives
Nope
Anything else?
nope

Is there an existing feature request for this?

[X] I have searched the existing feature requests

Just got that fixed


Visual Template update.

Visual Template update.

Hi @DenisKudelin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Denis Kudelin (Akvelon)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Closes #297

Reviewed, lgtm


multithreading problem

Zachary Zwiesler reported a possible issue with the multi-threading code:

I have been attempting to run fqtrim with multithreading. These are my command line arguments:
fqtrim -V -y 3 -p 8 -o $out $1,$2
The program executes as expected, but I haven't observed any increase in speed with more threads specified. % CPU usage appeared maxed out at up to four cores, but elapsed time remained similar, and assignment of greater than four cores showed  % CPU usage around 400 and no decrease in elapsed time. In fact when running ~500 MB files, I observed an increase in elapsed time.

Very likely this is a problem with my multi-threading code (inefficient producer-consumer implementation, with unnecessary locks & wait cycles etc.), similar to the issues that plagued the initial versions of StringTie's multi-threading code..
As an aside, fqtrim is largely I/O bound (i.e. mostly limited by disk/network streaming speed) so we shouldn't expect a significant decrease of wall time with multiple CPUs (definitely NOT linear), but same/increase of wall time is clearly a problem in the current implementation.

Very likely this is a problem with my multi-threading code (inefficient producer-consumer implementation, with unnecessary locks & wait cycles etc.), similar to the issues that plagued the initial versions of StringTie's multi-threading code..
As an aside, fqtrim is largely I/O bound (i.e. mostly limited by disk/network streaming speed) so we shouldn't expect a significant decrease of wall time with multiple CPUs (definitely NOT linear), but same/increase of wall time is clearly a problem in the current implementation.


Video doesn't fill container in Edge and FireFox

In version 1.0.20 has been changed getUserMedia video initialization from:
this.mediaDevices.getUserMedia({
"audio": false,
"video": this.params.constraints || {
mandatory: {
minWidth: this.params.dest_width,
minHeight: this.params.dest_height
}
}
})
to:
this.mediaDevices.getUserMedia({
"audio": false,
"video": this.params.constraints || {
width: this.params.dest_width,
height: this.params.dest_height
}
})
This change causes video to not fill container in Edge and FireFox. On IE 11 and Chrome works correctly.

I apologize, I didn't mean to create any issues with this change.  The old mandatory syntax is deprecated, and isn't even listed on the MDN documentation anymore.  I thought changing to the new format would be ideal for all browsers.  However, no matter what settings I try now seems to upset one browser or another.  There doesn't seem to be one single setting that satisfies them all.
I will revert the change, and go back to the old mandatory thing.


global: fix alembic dependency version

FIX Fixes upgrader and database creation by limiting alembic
version to <0.7.

Signed-off-by: Nicolas Harraudeau<EMAIL_ADDRESS>PS: I chose the same version limit as in invenio-upgrader.

This is connected with: inveniosoftware/invenio-upgrader#3

@nharraud Can you please amend the commit log message to something like:
installation: fix alembic dependency version

* FIX Fixes database creation and upgrading by limiting Alembic version
  to <0.7.

and repush?  (so that the news would appear in the "installation" section of the release notes, for clarity). Thanks.

@tiborsimko done


Pull requests becoming stale

Issue description
Pull requests seem to become stale at a time when they are ready to merge. I see this happening in a number of highly upvoted open pull requests. This issue isn't so much for these PRs specifically, but rather the problem in general: how can we avoid PRs from becoming stale.
Should something like probot-stale be added to Nixpkgs, so that we can get reminder-comments when a PR is becoming stale?

Just to illustrate why I created this issue: when I sort the PRs by number of 👍's I'm seeing a lot of PRs that seem to be ready for quite some time, but are not being merged. It is likely they still need some changes, but they aren't getting feedback:

https://github.com/NixOS/nixpkgs/pull/32422
https://github.com/NixOS/nixpkgs/pull/17494
https://github.com/NixOS/nixpkgs/pull/30890
https://github.com/NixOS/nixpkgs/pull/17757

There are also some that are so old and large that it would be quite a bit of effort to get it mergable again:

https://github.com/NixOS/nixpkgs/pull/18142
https://github.com/NixOS/nixpkgs/pull/10851

I wonder what the bottleneck is here:

if core maintainers no longer have sufficient time, could we authorize other contributors to help out on this?

who can merge PRs now?
what existing contributors would make good candidates?

if only a small core team has the Nix/NixOS know-how to handle PRs, would it be possible to train other contributors to this end?

what knowledge is essential here?

is this documented to give prospective maintainers a good starting point?

 People with a convincing track record (tens of accepted PRs, recent PRs being technically fine from the beginning, not completely inactive recently) get commit access if they ask — finding and informing such people that they could just ask might be a useful community service.
Unfortunately, there are PRs that get missed by mistake — more eyes would help here — and there are PRs that are missed because people are unsure if there is a «wrong approach» flamewar waiting to happen and do not want to be in the middle of it. Reading lightdm PR, for example, I am not completely sure if it is of the second kind, and I guess nobody is…

@7c6f434c:

People with a convincing track record [...] get commit access if they ask — finding and informing such people that they could just ask might be a useful community service.

If we notice someone has a great track record, might it help to mark them as contributors without waiting for them to ask? This might lower the barrier to entry, and hopefully get this group more excited about contributing further.

I think people who can give commit access would generally prefer not to monitor the statistics on their own (looking up track records of people who ask can be done with a search per person). And I think when I compiled a list of recommendations, there were no objections to giving access to everyone on the list who asks, but there were some reservations about giving out access without asking.

Just merged #32422. That one is pretty useful & harmless to those not using it.
All of the others are mostly questions of what defaults should be. These are harder to get consensus on even if they are probably good changes. More discussion is needed!

Reminder bot looks ok to me. I'd also like to have the mention bot back, I used to use it mentions as a criteria in filtering and it worked pretty well, now I have to use keywords instead. Why did it get nuked?
OfBorg could also just schedule PRs for automatic merging if they have some :+1:/LGTMs and no :-1:/Objections from trusted users.
E.g.:

OP> Here's a PR.
OfBorg> Blame information suggests @TrustedUser and @AnotherTrustedUser as reviewers.
TrustedUser> LGTM.
AnotherTrustedUser> LGTM.
OfBorg> All checks pass, got 2 positive reviews. Scheduled for merge on YYYY-MM-DD unless someone objects.
[YYYY-MM-DD 00:00]
OfBorg> Merging today at 23:59 unless someone objects.
[YYYY-MM-DD 23:59]
OfBorg> Merged.

OP> Here's a PR.
OfBorg> Blame information suggests @TrustedUser and @AnotherTrustedUser as reviewers.
TrustedUser> LGTM.
AnotherTrustedUser> LGTM.
OfBorg> All checks pass, got 2 positive reviews. Scheduled for merge on YYYY-MM-DD unless someone objects.
YetAnotherTrustedUser> Objection!
OfBorg> Merge unscheduled. This PR is stalled.
... some discussion ...
... some changes ...
[week after last update]
OfBorg> This PR now conflicts with PR #XXX recently merged to master, please rebase onto REVISION to continue.
... rebase ...
... some more discussion ...
[week after last update]
OfBorg> All checks pass, got 2 old positive reviews, 1 old negative review. This PR is stalled.
TrustedUser> Still LGTM.
AnotherTrustedUser> Still LGTM.
OfBorg> All checks pass, got 2 positive reviews, 1 old negative review. This PR is stalled.
YetAnotherTrustedUser> Well, fine, LGTM, I guess.
OfBorg> All checks pass, got 3 positive reviews. Scheduled for merge on YYYY-MM-DD unless someone objects.
[YYYY-MM-DD 00:00]
OfBorg> Merging today at 23:59 unless someone objects.
[YYYY-MM-DD 23:59]
OfBorg> Merged.
This will create pressure to review things.

@oxij those sound like really good ideas!
@7c6f434c:

people who can give commit access would generally prefer not to monitor the statistics on their own

That's fair, yeah, this does sound like extra burden.
My experience here is mostly from smaller personal projects, but for those when accepting a PR from someone I remembered to have made several good contributions before, I'd just mark them as contributor -- so here I wouldn't be actively monitoring fwiw. But I can't speak for bigger projects like this, admittedly.

And of course the people who do have access to giving out commit bits don't often have time to review PRs, either…

If we could give per folder/files commit rights, then it would be easier to give those rights. Like I wouldn't mind asking commit rights for neovim (and some other packages I use, ns-3, cmd2, protocol) but asking them for the whole nixos repositoy is intimidating.

Maybe we need to move back to SVN?

Since creating this issue yesterday already 2 of the mentioned PRs were merged! :tada: :smile: (thanks @matthewbauer)
I think the maintainership and rights is a related, but separate issue. Contributors can just as easily approve PRs and if there are enough approvals it's easier for member to merge those PRs.
However the problem is also partly that usually PRs are only approved/reviewed by one person (or at least one is giving their approval). This might be solved by automatically adding contributors as reviewers as well, so that reviews become part of their 'flow'.
I really like the proposal of @oxij. If there are enough people thinking this is a good idea, what would be needed to implement this? I'm quite unaware how OfBorg is currently working (is it a bot based on an existing one? Where can I find its code?)

I really like the proposal of @oxij. If there are enough people thinking this is a good idea, what would be needed to implement this?

There is a general idea that ofborg development should be careful and conservative.
There is a plan to add conditional-merge, delayed-merge and remind-me commands, but it is work and testing. Obviously these have to work well before even considering what amount of approvals without explicit merge command should mean what and who is trusted more or less.

I'm quite unaware how OfBorg is currently working (is it a bot based on an existing one? Where can I find its code?)

How to find: click the nickname, got to the profile, look at the homepage, it is a repo.
Where to find: this will lead you to https://github.com/NixOS/ofBorg/
There is a README.md, and development is indeed done via PRs and issues.

Ah, interesting. I was thinking that repo was used for the builder of OfBorg, but it's also the bot itself.
I think the following issues are very much related:

Ping meta.maintainers
Automatic Merging Implementation
Feature Request: Automatic merging

Should another issues be created to notfy that a PR is becoming stale? The exact details are yet to be defined, but I can imagine something that a reminder for people of a PR being inactive for some time could bump the activity again.

I think there is also an issue about periodic re-evaluation? It is probably also relevant.
I would say that unless you have a specific design that can be discussed, there are enough prerequisites well-described in issues that creating further issues can wait.

Earlier there was an experiment with nixbot which merges when asked by maintainers of expressions. With some small improvements, mainly getting rid of maintainer-paths.json, I think that would be a major step forward. We should also include at that point an additional meta attribute from blacklisting certain derivations to be merged that way though.

If we could give per folder/files commit rights, then it would be easier to give those rights.

Maybe we need to move back to SVN?

Git with gitolite supports per folder/file permissions. (When nixpkgs moved from svn to git I was kind of hoping we'd use gitolite.)

@bjornfor Nixpkgs uses now the codeowners feature. See https://github.com/NixOS/nixpkgs/blob/a61bd1e42d5512f0e5180edf7ea6090afcd9544b/.github/CODEOWNERS

LOL.

RFC51 added a stale bot, which I think addresses the main actionable issue in the original post I don't think it gets us much to leave this open. If people want to discuss general improvement ideas they can use Discourse, and if they want to discussion specific actionable issues, they can open a new issue.
I'm going to close this based on that, but please let me know if you object and we can reopen.


Style: OnTouches* API

Currently, the API in OnTouchesMoved and company are using a List<CCTouch> as a parameter. See here.
I personally prefer callback based things to not be modifiable. The API could probably return an IReadOnlyList<CCTouch> (preserving indexing, but not misleading towards a - you can modify this list), or an IEnumerable<CCTouch>?

Therzok
This makes sense.  We will look into it.  Not sure about the IReadOnlyList though as that was only introduced in 4.5 if I remember correctly.  I think the IEnumerable would be the safest.

Going down the IEnumerable<CCTouch> is an API breaking change. Unless we can use IReadOnlyList, I see no use to switching to IEnumerable. Most of the samples use touches[0] too.

@Therzok I have already looked at this and there would be internal changes that would need to be done in our EventDispatcher which would include some casting to a List and back.  That would not be good.  It is more involved than just changing to IReadOnlyList and needs some more analysis.  Sorry for not get back sooner.


Create barnsley-academy.txt

@StefanoChiodino  Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Amend the text of some of the Countryside Stewardship Finder filters

This will need a corresponding change to the doctype schema in Rummager: https://github.com/alphagov/rummager/blob/master/config/schema/default/doctypes/countryside_stewardship_grant.json


tls/store/basic: introduce KeyList and KeySet

Summary by CodeRabbit

New Features

Introduced a KeyList type for managing private keys with various manipulation methods.
Added a KeySet type for maintaining a unique collection of cryptographic keys with error handling.
Launched a new basic package for a programmable TLS store.

Bug Fixes

Implemented error handling in key management methods to ensure robustness.

Documentation

Enhanced documentation for new types and methods to improve usability and understanding.

superseeded by https://github.com/darvaza-proxy/x/pull/135


More powerful selection set filtering on many to many relations

It looks like filters on relations in selection sets can only filter by the by. This is somewhat useful, but vastly less than what is going to be needed for many use cases. Can we allow filters on relations in selection sets to have all of the capabilities of the filters on find queries? Limit, order by, paging, everything should be possible wherever filters are allowed

Or maybe I'm doing something wrong with setting up the relation that is causing it to not fully support filtering?

It looks like this is a problem with many to many relationships. Is it currently possible to do filtering in selection sets on a many to many relationship?

This might be a duplicate of this: https://github.com/aerogear/graphback/issues/1343
This issue highlights the problem in a selection set, but under the hood the filtering might all be the same, not sure

Can we allow filters on relations in selection sets to have all of the capabilities of the filters on find queries? Limit, order by, paging, everything should be possible wherever filters are allowed

This looks like a duplicate of https://github.com/aerogear/graphback/issues/1122. If so let's close this in favor of the other.
/cc @lastmjs @craicoverflow @wtrocki


Add delegate methods to notify that CropView became resettable

I'm building an embedded version of the CropViewController with my own overlay UI.
To my surprise, there was no easy way for me to activate/deactivate "reset" button.
While CropViewDelegate provides the necessary methods, accessing it is impossible (or I didn't figure it out).
The most simple solution, like subclassing CropViewController and overriding "didBecomeResettable" is impossible because
"extension CropViewController: CropViewDelegate" is not public.
The only solution I see right now is to fork the repo and manually add necessary methods to CropViewControllerDelegate.
Is there a simpler and more elegant way?

@xinatanil
Do you mean you want to provide two status active/deactivate for reset button?

Yes, something like that via KVO or delegate method.
For my custom UI I'd like to hide the reset button when it's not needed.

@xinatanil
So do you mean you want to set your own icon for reset button?

No, I want a way for me to know that reset button has been activated/deactivated, so that I could disable my own custom UI elements.
Currently, I see no easy way for that.

@xinatanil
For activated status, you can use delegate function cropViewControllerDidImageTransformed. For deactivated status, I can add a new delegate function for it.

That'd be great!
Something like func cropViewController(_ cropViewController: CropViewController, didBecomeResettable resettable: Bool) would be amazing

@xinatanil
Please check #317 to see if it works for you, then I can make a new release later.

Looks fantastic!
Thank you for a quick response!

@xinatanil
Just released Mantis 2.12.0 for this feature. Please have a try.


problem building dlib with cmake on ubuntu 16.04

Hello
I am trying to build dlib on my ubuntu machine, but am facing some issues when executing:
       cmake --build . --config Release

I tried adding SET ( CMAKE_CXX_FLAGS "-D_GLIBCXX_USE_CXX11_ABI=0" ) after PROJECT(examples) as suggested by other people on askubuntu.com, but the issue is not solved:
part of the modified CMakeLists.txt file: http://paste.ubuntu.com/25183001
this is the error I keep getting: http://paste.ubuntu.com/25182997
I seem to not be the only one with that issue:
https://askubuntu.com/questions/931225/dlib-19-4-will-not-build-with-cmake-on-ubuntu-16-04
https://askubuntu.com/questions/940237/cmake-build-issue-undefined-reference-to-std-cxx11basic-stringchar-std
How can I solve this issue?
Thank you

Those links are broken.  Paste the errors here.

@davisking my bad, I corrected the links. The output is a bit lengthy...

The error:
sqlite_ex.cpp:(.text._ZN4dlib5randmEll[_ZN4dlib5randmEll]+0xd3): undefined reference to `__cxa_throw_bad_array_new_length'

is not a dlib error.  Getting that error means there is something terribly wrong with your compiler as __cxa_throw_bad_array_new_length is a core part of the compiler toolchain.  You need to get a working c++ compiler.

@davisking the only time I get this type of error is when trying to use dlib. Could you maybe help to fix this, so I can try use your open source software?

Google __cxa_throw_bad_array_new_length.  Lots of people have this error and it has nothing to do with dlib.  It's because something is wrong with how you installed GCC.

@davisking  I didn't even really install gcc myself, I just use the one that comes when installing ubuntu on my machine....


Creating Elements in PhysicsJs.activity that're Bigger than canvas

Collisions are only detected for the top edge of view-port (i.e base of toolbar) for bodies/elements while creating.
So elements bigger than canvas are easily possible to create from other edges
e.g

Solution: consider each edge of viewport and stop increase in size once edge is touched.

Make sense.

Fixed in https://github.com/llaske/sugarizer/pull/1172


Confirm

Echo "$Confirm issue resolutions"

Not sure what you meant 😅

Not sure what you meant sweat_smile

Have the bugs been resolved yet?


How to solve this problem? It seems that the cookie is occupied and the content cannot be read.

How to solve this problem? It seems that the cookie is occupied and the content cannot be read.
C:\Users\User\Downloads\Extract-Chrome-Cookies-main>Extract_Cookies.py
IzinError: [Errno 13] Permission denied: 'C:\Users\User\AppData\Local\Google\Chrome\User Data\Default\Network\Cookies'

You need admin privileges when running the code


Bug: type error in D3D12_SUBOBJECT_TO_EXPORTS_ASSOCIATION::pExports

Which crate is this about?
No response
Crate version
No response
Summary
pub struct D3D12_SUBOBJECT_TO_EXPORTS_ASSOCIATION {
    pub pSubobjectToAssociate: *const D3D12_STATE_SUBOBJECT,
    pub NumExports: u32,
    pub pExports: *mut ::windows::core::PWSTR,
}

->
pub struct D3D12_SUBOBJECT_TO_EXPORTS_ASSOCIATION {
    pub pSubobjectToAssociate: *const D3D12_STATE_SUBOBJECT,
    pub NumExports: u32,
    pub pExports: *mut ::windows::core::PCWSTR,
}

typedef struct D3D12_SUBOBJECT_TO_EXPORTS_ASSOCIATION {
  const D3D12_STATE_SUBOBJECT *pSubobjectToAssociate;
  UINT                        NumExports;
  LPCWSTR                     *pExports;
} D3D12_SUBOBJECT_TO_EXPORTS_ASSOCIATION;

https://learn.microsoft.com/en-us/windows/win32/api/d3d12/ns-d3d12-d3d12_subobject_to_exports_association
Toolchain version/configuration
No response
Reproducible example
No response
Crate manifest
No response
Expected behavior
No response
Actual behavior
No response
Additional comments
No response

Looks like the Win32 metadata is missing a const attribute. I'll forward this issue over there.


Typo in Developer Guide

I was trying to set up Singa-auto in my local system, however I realised that there are several typos in the Developer Guide.
In the 'setup.rst' file

There are some instructions to do bash scripts/start.sh  but according to the files in this repository, it should be either bash scripts/kubernetes/start.sh or bash scripts/docker_swarm/start.sh

In the 'development.rst' file

There is an instruction to 'Before running any individual scripts, make sure to run the shell configuration script: source .env.sh'. However, when I ran it, it gave me an error because there is no such file found. I believe it should be  source ./scripts/docker_swarm/.env.sh instead.
In the section 'Managing Rafiki's DB' afterwards, there are some typos as well. All of the variables still have 'rafiki' as its name, and I believe it should be change to Singa-auto instead. I encountered some error when trying to do this part as well.

In the 'folder-structure.rst' file

As mentioned above, there is no .env.sh in the root directory anymore so I believe it should not be there in the folder structure file.
There are some discrepancies as well to the folder structure description, as the folders are no longer structured as described in this file.

https://github.com/nusdbsystem/singa-auto/pull/174
https://github.com/nusdbsystem/singa-auto/pull/169
https://github.com/nusdbsystem/singa-auto/pull/166
Hi agnesnatasya, documents are updated accordingly, pls see the newest merged PRs.
Moving onwards, Singa_Auto might be renamed in the near future, so pls temporarily bear with us about the current name convention. Once the new name is set, we would make its name consistent.

https://github.com/nusdbsystem/singa-auto/pull/174
https://github.com/nusdbsystem/singa-auto/pull/169
https://github.com/nusdbsystem/singa-auto/pull/166
Hi agnesnatasya, documents are updated accordingly, pls see the newest merged PRs.
Moving onwards, Singa_Auto might be renamed in the near future, so pls temporarily bear with us about the current name convention. Once the new name is set, we would make its name consistent.


Does not run with Linux OS

Hi Sergio,
Thanks for developing such a great command! It runs perfectly on my Desktop computer, which has a Windows 10 OS. However, when I try to run it on the NBER cluster (Redhat Linux OS), I get the following error message:
accuracy not found in class FixedEffects
(113 lines skipped)
(error occurred while loading ppmlhdfe.ado)
r(3000);
I am happy to provide you with any other info you might want.
Joe Staudt

Hi Joe,
The fixedeffects class is part of reghdfe, so the first thing I should check is that you have the latest version installed (of both packages, as well as of ftools)
Also, I work with ppmlhdfe on Linux so that shouldn't be an issue

Hi Sergio,
You are exactly right. I had the latest versions installed in my PLUS directory, which lead to my confusion. However, there is another directory for ado files on the NBER cluster, which is searched first by Stata, and did not have the latest versions. Once these were updated by the admin, everything worked perfectly.
Thanks again for such a useful command!
Joe


how to set value from php after load page

I want to set value on select2 after i open some page, i use codeigniter framework.
Thank's

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


[Tweet] just Testing

What do you want to tweet?
Just testing the workflow
Caution

[X] This tweet follows the 280 characters limit

Why you want to tweet this?
To test the run once again
Extra notes.
Do not merge the PR if created. This is just a test tweet

The PR was successfully made this time


User can pass along locals similar to render

Render syntax is pretty well defined and we're sort of mimic'ing it

Love it. Code LGTM. Should there be an update to the README?


fix: add-to-project action

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #5487   👈

This comment was auto-generated by Graphite.


Tiny refactoring

I've added a possibility to specify the root path for StartupFiles, as well made Punishment mapping less messier.

Thank you for the contribution!


Serum Support?

Can I get this to work with Serum (VST), which uses DirectDraw as it's  rendering backend?

Might have to see a debug log, here's how you can generate one:
https://github.com/CnCNet/cnc-ddraw/issues/9#issuecomment-488805261

Can't really find that specific version that's using DirectDraw nor do I have a log I could check so I'm closing this for now.


Fix useless error message of render

This is still WIP.

@jinzhu please review this.
When using render without a layout file. We can give helpful error message to the user.
Current error message when render users/profile.tmpl and missing layout application:

Got error when render template users/profile: template: "." is an incomplete or empty template; defined templates are: "profile.tmpl"

The new error message will be

Got error when finding layout: Cannot find layout: '/Users/raven1/go/src/github.com/qor/render/layouts/application.tmpl'

@jinzhu Can this be merged? I met the same problem again 😒


Ios 10 crash

hi with ios 10 it crash at start
2016-11-17 17:43:11.092619 FaceDetection[4651:1307406] [access] This app has crashed because it attempted to access privacy-sensitive data without a usage description.  The app's Info.plist must contain an NSCameraUsageDescription key with a string value explaining to the user how the app uses this data.
but NSCameraUsageDescription was added

@boardmain Thanks for raising this! I've added the key to the Info.plist and it now asks for permission on iOS 10. 📷 👍


fix: return correct brightness value for Litra Beam and Litra Beam LX devices

Litra Beam and Litra Beam LX devices can currently return an incorrect brightness value when brightness is set above Litra Glow's 250 Lm maximum brightness.
Fixes #36.

@insystic I have a feeling that this will fix your issue, #36, but I'm not easily able to test that because I have a Litra Glow with a maximum brightness of 250 Lm.
Would you be able to test this out? If you're not comfortable compiling yourself, I'd be happy to create a binary for you if you can tell me what OS you're running 😊

You can actually download litra binaries from GitHub Actions: https://github.com/timrogers/litra-rs/actions/runs/8280358937?pr=38

Here's a macOS binary:
litra_timrogers_fix-36_darwin-amd64.zip

@insystic confirmed that these changes fixed the issue ✅


Unable to update TLS version using Python management SDK on Service bus namespace

Package Name: azure-mgmt-servicebus
Package Version: 7.1.0 and 8.2.0
Operating System: Windows
Python Version: Python 3.6 and 3.7

Describe the bug
This morning I've been trying to change/update the TLS version of my Service Bus namespace using the Python SDK. However, every time I run the code, I can make changes to other components of the namespace, however the TLS version remains the same. For example, here's the code:

import os
from azure.identity import DefaultAzureCredential, AzureCliCredential
from azure.mgmt.servicebus import ServiceBusManagementClient
def main():
SUBSCRIPTION_ID = "SUBSCRIPTION_ID"
GROUP_NAME = "RESOURCEGROUPNAME"
NAMESPACE = "SERVICEBUSNAMESPACE"
Create client
servicebus_client = ServiceBusManagementClient(
credential=AzureCliCredential(),
subscription_id=SUBSCRIPTION_ID
)
Get namespace
namespace = servicebus_client.namespaces.get(
GROUP_NAME,
NAMESPACE
)
print("Get namespace:\n{}".format(namespace))
Update namespace
namespace = servicebus_client.namespaces.update(
GROUP_NAME,
NAMESPACE,
{
"properties": {
"minimumTlsVersion": "1.2"
}
}
)
print("Update namespace:\n{}".format(namespace))
if name == "main":
main()

My namespace currently is on 1.1 so I ran the code, but it did not change the version to 1.2, regardless of the Python version or azure-mgmt-servicebus I have tested both below:
Python 3.6, azure-mgmt-servicebus==7.1.0
Python 3.7, azure-mgmt-servicebus==8.2.0
Both got the same result.
Expected behavior
I do expect to be able to change the version of TLS.
Additional context
If I want to update tags, I can without a problem. Reference https://github.com/Azure/azure-sdk-for-python/issues/new?assignees=&labels=&projects=&template=bug_report.md&title=

Thank you for the feedback @testenvmicro . We will investigate and get back to you asap.

Hi, @testenvmicro
azure-mgmt-servicebus generated based on multi api versions, and I found that minimumTlsVersion is defined in version 2022-10-01-preview under method create_or_update. So you need to create a client using different version and change the method.
The main code is like:
    SUBSCRIPTION_ID = "xxx"
    GROUP_NAME = "xxx"
    NAMESPACE = "xxx"

    servicebus_client = ServiceBusManagementClient(
    credential=AzureCliCredential(),
    subscription_id=SUBSCRIPTION_ID,
    api_version="2022-10-01-preview"
    )

    namespace = servicebus_client.namespaces.begin_create_or_update(
    GROUP_NAME,
    NAMESPACE,
    {"location": "eastus","properties": {"minimumTlsVersion": "1.2"}})
    print("Update namespace:\n{}".format(namespace.result()))

Hi @ChenxiJiang333 by using above code, I am forced to set a location property and if I do that, I get the following:
Code: InvalidResourceLocation
Message: The resource 'premiumsbgonzo' already exists in location 'centralus' in resource group 'RG01'. A resource with the same name cannot be created in location 'eastus'. Please select a new resource name.
If I remove the location I get the following instead:
azure.core.exceptions.HttpResponseError: (LocationRequired) The location property is required for this definition.
Code: LocationRequired
Message: The location property is required for this definition.

Hi @ChenxiJiang333 by using above code, I am forced to set a location property and if I do that, I get the following:
Code: InvalidResourceLocation Message: The resource 'namespacename' already exists in location 'centralus' in resource group 'resourcegroupname'. A resource with the same name cannot be created in location 'eastus'. Please select a new resource name.
If I remove the location I get the following instead:
azure.core.exceptions.HttpResponseError: (LocationRequired) The location property is required for this definition. Code: LocationRequired Message: The location property is required for this definition.

Sorry I didn't mention it clearly, since your resource is in location 'centralus', you need to update the value of location to 'centralus' in the code as well.

Hi @ChenxiJiang333
I ran this code. i am able to update the tls version. but it requires to have location property to be set. why is location required? can't we update tls without passing location property.

Hi @ChenxiJiang333
I ran this code. i am able to update the tls version. but it requires to have location property to be set. why is it required? can't we update tls without passing location property.

Hi @ChenxiJiang333
I ran this code. i am able to update the tls version. but it requires to have location property to be set. why is location required? can't we update tls without passing location property.

First please know that sdk is generated by the swagger definition.
I found you use update method at first and it don't need the location property, but it's required when you try to create_or_update a resource, and minimumTlsVersion only defined here as parameter of create_or_update operation, yet for just update(definition can be found here), there's not.
That's why for now you have to use create_or_update and pass location property with it.
To solve it, you can request to add minimumTlsVersion property for update operation by opening an issue in https://github.com/Azure/azure-rest-api-specs/issues and mark it with label feature-request. Once swagger updated, a new version of python sdk will be released.


Make it possible to remove inputs

This will cause way too many merge conflicts right now, and I'm not sure we'll be able to do this properly


Add super.onCreate() call in generated onCreate method

Related to: https://github.com/NativeScript/nativescript-dev-webpack/issues/952
Currently, if the user hasn't provided an onCreate method implementation, the SBG generates the onCreate method for object initialization. This auto-generated method, however, doesn't have a call to the parent class' method and in some cases like extending an IntentService an exception occurs when Android creates an instance of the service. This commit adds a call to super.onCreate()

@vtrifonov linked to the origin issue in the webpack repo


Add acceptance test to run git cookbook

Sweet @jkeiser. :+1:
Possibly a dumb question but how does one select ec2 / vagrant / digitalocean while running the tests?

@sersut KITCHEN_DRIVER=ec2
But we haven't thoroughly tested them yet so I didn't add that.

Sweet! Thanky...

:+1 with minor comments


Adjust "all dreams & stats"

The page https://dreams.kiezburn.org/guideview needs some adjusting to our particular situation. Below the table as it should look like. The numbers are simply copied over, and should stay dynamic.

CURRENT
TOTAL
(%) OF TOTAL

Kiezburner participation (at least 1 heart given)
0
3210

Total grants given
0
35481

Total of minimum budgets (+ as % of total Dream budget)

0

As a current placeholder for the total Dream budget, you can enter 20000

noiced and changed


Custom extension location setting is not considered

VSCode Version: 1.15.0-insider 64-BIT
Build Date: 2017-07-19
Extension Version: 1.3.0
Issue Reproduction:

VSCode uses following launch parameters
 --user-data-dir "userdata" --extensions-dir "extensions"
The extension is supposed to store it's config details at "extensions" folder relative to VSCode executable.
But instead is still creates at the following location
C:\users\<user>\config\configstore\cspell.json

Thank you. I'll investigate.

I found the issue. This should be fixed soon.

Fixed with 1.3.1
Can you verify?

We use configstore to enable installing custom dictionaries from cspell dictionaries. If one is installed, it will write to ~/.config/configstore/cspell.json. This spell checker uses configstore to check for their existence.  The issue is that the current version of configstore creates the file on startup.
I have forked configstore to have it only create the file on write.

Hi Jason, should there be an updated extension of "Code Spellchecker"?, because I don't have any updates and the current version in my machine is 1.3.0.
Please let me know how to verify.

On Mon, 24 Jul 2017 at 06:02, Kannan Balasubramanian <
<EMAIL_ADDRESS>wrote:

Hi Jason, should there be an updated extension of "Code Spellchecker"?,
because I don't have any updates and the current version in my machine is
1.3.0.
Please let me know how to verify.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Jason-Rev/vscode-spell-checker/issues/110#issuecomment-317314097,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEoDjUtkFHbbwWmaqpZsi64F4wWuWCUlks5sRBdggaJpZM4Odkoa
.

If you type in 'spell' into the extension search box it should cause it to
update.
Please delete the config file that had been created.

Seems to be working.
I updated to the latest version of extension, deleted the 'configstore' folder and when launching the VSCode, I couldn't find the config file or folder created at C:\users\<user>\config\configstore\.
Thanks for fixing the issue.


text file download creates only half the model

When I click "Download text file" the output only has half the model in it, it is vertically split in the middle. Tested using Safari and Firefox. Attaching some sample output.
ASCII-1.txt

This is what I see when I open this text, does this look correct to you?
I had to resize the window to fit the entire image, maybe give that a shot?

🤦 That really is all there was to it, I'm sorry for not having noticed it! Duh! Thanks for getting back to me about it.

Marking as closed!


VerifyDiscordRequest should return an async function

In utils.js, the function returned by VerifyDiscordRequest() should be async, because verifyKey is async. verifyKey() is currently not awaited, which means isValidRequest is a Promise which is always thruthy.
However, this means that express.json({ verify: VerifyDiscordRequest(process.env.PUBLIC_KEY) }) will not work, because the verify function is expected to be synchronous. Therefore it would probably be better to refactor this and rather use the verifyKeyMiddleware from discord-interactions.

Thanks for surfacing this!  We got rid of that method all together, and just use verifyKeyMiddleware now.


fix(layer.js): remove unnecessary returns, to more-gracefully watch l…

…ayer property changes
Resolves #336

🤔 going to check this tomorrow on my codebase to verify nothing breaks. Actually I'm not quite sure why these return statements are in there. They have been there already when I took over maintenance of this project 😄. Thx for ur PR

@aagg1234 Could you maybe try to include these mods in our codebase to verify it doesn't break anything?

Sorry, accidentally brough additional commits (related to other fixes and openlayers version upgrades). I'm not sure how to roll those back here. Let me know how to best handle.

See #389


2023 updates

tables_2023.R :

copies habitat_confirmations.xls over from skeena 2023 repo
builds objects needed for fpr_table_fish_site(), fpr_table_fish_density() and fpr_plot_fish_box()
renames objects to _updated so they are distinguishable from objects built in tables.R
tables_2023.R  is run before tables.R so that objects are over ridden by tables.R
tables_2023.R  can be used in any repo by changing year
followed workflows described in https://github.com/NewGraphEnvironment/fish_passage_skeena_2022_reporting/issues/130, please let me know what you think before I move on to updating the other report.

Tributary to Skeena River - 198217:

added fish sampling section with calls to fpr_table_fish_site(), fpr_table_fish_density() and fpr_plot_fish_box()
We need to populate habitat_confirmations.xls with electrofishing and site area data before fpr_table_fish_density() and fpr_plot_fish_box() can be run. I've done a bunch of sleuthing and can't find where this data is added to step_2_fish_coll_data, but its present in past repos so it must be somewhere...
added some fish photos to data/photos and tagged them with _k_2023 so that fpr_photo_pull_by_str still works (doesn't work if I put them in their own separate 2023 folder). Tested build to gitbook and pdf and both format properly.
Background info needs work still #60

Dale Creek - 198215:

added fish sampling section with calls to fpr_table_fish_site(), fpr_table_fish_density() and fpr_plot_fish_box()
We need to populate habitat_confirmations.xls with electrofishing and site area data before fpr_table_fish_density() and fpr_plot_fish_box() can be run
added to fish sampling section, "Due to the crossings proximity to the Kispiox Rover and the significant outlet drop on the culvert with moderate flow, fish passage is likely limited to times of high flow." but please change if u don't think this is correct.
updated background info
added fish photos, fish Id concluded from habitat_confirmations.xls

This is awesome! Not discouraging at all, actually super helpful for me to advance my writing and improve my overall skills. Still pretty new to all the report writing so all this feedback is great. I find this format (PR comments) super good. Thank you!

The sik-e-dakh memo is perculating up the priority funnel since there is so much going on there and as i said it makes sense for me to finish. for that reason, please let me know when you want to work on this repo and try to get PRs in daily so that I can jump on it and finish it at some point when I am not trying to do a million other things - ha

I will get started on these edits on Tuesday and will make sure to PR daily. I will let you now what files I am planning on editing so we can avoid merge conflicts.
Thank you so much!


Docker Desktop Kubernetes documentation

There's no documentation for Docker Desktop Kubernetes here. I got KFP working pretty easily on DDK and would be happy to contribute documentation upstream for it if ya'll would welcome it.

/close

Thanks for closing. I switched to Colima anyway 🙃 . We were just talking about DeployKF in my work slack!


Expose String::camelcase_to_underscore

This function can be useful in plugins for converting filenames or in code for converting identifiers between different languages.

Needs a rebase, otherwise should be fine to merge.

@dalexeev Can you rebase this? :)

Can you rebase this?

Done. But I think we'd better implement this:

s.to_lower()
s.to_upper()
s.capitalize()
s.to_snakecase()
s.to_camelcase()
s.to_pascalcase()

Done. But I think we'd better implement this:

s.to_lower()
s.to_upper()
s.capitalize()
s.to_snakecase()
s.to_camelcase()
s.to_pascalcase()

I think that's a good idea. Should we then close this PR and have someone (you?) work on that proposal (which solves https://github.com/godotengine/godot-proposals/issues/719 I suppose)?

I think that's a good idea. Should we then close this PR and have someone (you?) work on that proposal (which solves godotengine/godot-proposals#719 I suppose)?

#63902


MD5 is not checked if it's empty

At
https://github.com/mgomes/api_auth/blob/master/lib/api_auth/headers.rb#L77
def md5_mismatch?
    if @request.content_md5.empty?
        false
    else
        @request.md5_mismatch?
    end
end

So the md5 is not checked if the header "Content-MD5" is empty. Which mean I can replace the payload, remove the Content-MD5 and get passed the check.

The content-md5 header is indeed optional. However, if a client adds one, it can't be replaced as you suggested. You could remove it from the canonical string, but the Authorization header has already been hashed with the private key and includes the md5 signature.

silly me! thanks for the clarification


Request to transfer opentracing-java-jdbc under opentracing-contrib

@bhs
Please create repository java-jdbc under opentracing-contrib.
I would like to transfer OpenTracing JDBC Instrumentation (https://github.com/malafeev/opentracing-java-jdbc).

@malafeev, what's your opinion about the existence of both JDBC and JDBI instrumentation? Are they both warranted? Should one replace the other? Should they be part of the same repo? If so, which?

@bhs jdbc is the most low level api.
jdbi works above jdbc.
Although jdbc instrumentation can replace jdbi instrumentation I suppose that in some cases
jdbi instrumentation is convenient to use.

@bhs I propose to create java-jdbc repo with jdbc instrumentation.
and deprecate jdbi (pointing to jdbc)

nit: I don't know how jdbc/jdbi instrumentations works, but sometimes it is possible to start spans at lower levels and add some meaningful tags/logs in higher levels. e.g. web filter and spring interceptor. But I don't know if this is the case.

@pavolloffay jdbc/jdbi instrumentation works almost the same way as instrumentation of http clients: start span before request and finish span when response is received.
Difference that request/response protocol is not http but jdbc.

https://github.com/opentracing-contrib/java-jdbc/invitations


democracy vote not supported

Good evening,
I've tried voting on democracy (Ledger Nano X, firmware version 2.0.2, Kusama Version 11.9180.1 ) and it fails due to "Unexpected call index" although it says in your documentation that it is supported. Is this a known issue?
Regards,
ever

Hi @everalesson! Could you provide more info? Some snapshot would be nice! I checked that democracy pallet is indeed supported but you are getting a message that means that it's not.
Besides, there was a change in tx version and the last app version supports both (tx version 10 and 11). However, democracy pallet was added to tx version 11. Could you tell me which version were you using? If you try now, it should work.

Hi @ftheirs! Just tried again today and same result, I'm not sure what are you asking about the version but the version of the Kusama app installed in my ledger is 11.9180.1.
Attached you can find a snapshot of the input and the error:

Let me know if you need anything else.

Hi! I was revising the repo and looks like the democracy pallet has been added in version 11.9180.2, exactly in this commit.
It looks like updating the app will solve it!
I'll close this, feel free to reopen if doesn't work!

Update is not available, do you know when that patch will be released?

Hmm, I just installed in my nanoS+ the 11.9180.3.
Maybe Ledger has not released yet the app in nanoX. We (zondax) don't control when the app is released in the official Ledger Live market, they have to check the apps and some sort of processes that may take some time. Sorry about that :(
Have a nice week!

Okay, thank you for your help!!


Autofix remove types from docstring

Currently types can be added in two places in a function as a type hint and in the documentation.
def myfunc(value: int):
   """
   Args:
      value (int): any value you want
   """
   pass

This can lead to potential conflict between the type hint and the docstring leading to multiple sources of truth. #5541 seeks to solve this by fixing the docstring, however this could still lead to the reverse issue; that the docstring is updated but not the type hint. It also avoid duplicating information. The reason to prioritise the type hint is meaningful as it can be checked by type hinters etc.
There might still be reasons to have types in the docstring such as documentation, but there already exist extension which remedy this need (e.g. https://pypi.org/project/sphinx-autodoc-typehints/). I probably wouldn't set it as a default though.
Let me know if there is something I have missed. I very much enjoy the package.
Edit: In a similar vein, it might also be worth discouraging "Defaults to {value}" in the docstring as it similar can be outdated and is already specified function signature.

For the same reasoning we also dropped type information from our docstrings. The redundancy is error prone and adds little information.
Another similar rule would be type docstrings for classes, where the type hint can be provided by instance variable annotations

Great idea! At work, we switched to sphinx-autodoc-typehints but now we have lots of code that is inconsistent, with old docstrings containing the type and new code omitting it.

Also PEP 727 maybe could change the way of how to document parameters. then, after it standardized, we need to remove junks from docstrings.

In order to implement an autofix, a new rule is needed first, right? Something like Parameter type already documented via type hint? This would probably fit into the D4XX category.

Also PEP 727 maybe could change the way of how to document parameters. then, after it standardized, we need to remove junks from docstrings.

I think that is orthogonal to this issue and should be treated separately, once PEP 727 has been accepted. The rule could be something like Parameter already documented in function signature.

This seems reasonable to me! Although we do not yet have a dedicated docstring parser and may want to wait until we write one.

See also https://github.com/astral-sh/ruff/issues/458


Changing item height after init

Hi there
Your virtualscroll looks awesome!
I'd like to know if it's possible to update an item height after init?
Like for having a "view more" button in some items in list.

@HugoHeneault, after the latest fix, changing item height after init should work fine, if you set childHeight input. Please reopen this if you are facing an issue.


Support for zones (rooms)

Would it be possible to add support for cleaning specific zones?
I know this isn’t supported (yet) by HA but maybe it could be used by send_command - https://www.home-assistant.io/integrations/vacuum/#service-vacuumsend_command

I don't have any Dyson vacuum that supports cleaning specific zones so the development and testing is going to be hard. I'm not going to work on that, at least for now. But any PRs are welcome if someone want to help with that.

That makes sense. I’ll do some testing and see what I can find, hopefully it isn’t too difficult.

Looks like the zone names and other info are stored on the Dyson servers. I was able to de-compile the Android app to get the API endpoints for these.
I also snooped the MQTT messages to figure out the correct params for cleaning specific zones. I'll keep working on this and hopefully be able to piece it all together.

Ah, I see that you mentioned zone cleaning here: https://github.com/shenxn/ha-dyson/issues/25. Custom services in HA would be a good way to make it work... maybe once I get the core functionality here I/we can work on that in HA.


chore: upgrade to K8s 1.26; remove PruneWhitelist

See: https://kubernetes.io/blog/2022/11/18/upcoming-changes-in-kubernetes-1-26/#deprecation-of-non-inclusive-kubectl-flag

@TylerGillson Thanks for the PR! I'll tag a new patch version for you. :)

@TylerGillson Thanks for the PR! I'll tag a new patch version for you. :)

Thanks @matskiv!


trip detail optimisation

Suggestion by mr X :
Maximum space should be reserved for items. For that, you should remove the line 'List for. . .' from the action bar. Users already know they're looking at the list for a particular trip. Besides, that line can seen completely only in landscape mode. And I'm pretty sure that most users set portrait mode as their phone's default orientation.
Shift 'edit trip' and 'add item' in the space vacated by 'List for. . .' and replace them with icons. Discard 'more' and also the field for entering names at the bottom.

duplicate. lack of interest, closing


g

g

This is the wrong project for your issue, please create your issue at https://reddit.com/r/ggggg instead


NTR: tRNA 5'-end processing / children of RNA repair? / 3′-5′ RNA polymerase activity

I have several papers about 4 Dicty 3′-5′ RNA polymerases that describe different functions for each. One is Dicty Thg1, which has the same tRNA guanylyltransferase activity (GO:0008193) as yeast THG1. The other 3, Tlp2-4 (Thg1- Like Proteins) also have 3′-5′ RNA polymerase activity (no such term in GO), but each acts differently (PMID:27484477, also PMID:26143376, PMID: 21307182)

Tlp2: catalyzes a mitochondria-specific tRNAHis maturation reaction, which is distinct from the tRNAHis maturation reaction typically catalyzed by Thg1 on cytosolic tRNA.

Tlp3: catalyzes tRNA repair during mitochondrial tRNA 5′-editing in vivo and in vitro showing 3′-5′ RNA polymerase activity of TLPs as a bona fide biological activity.

Tlp4: cytosolic, and catalyzes robust 3′-5′ polymerase activity on non-tRNA substrates (other ncRNAs).

A) New terms:

tRNA 5'-end processing / mitochondrial  tRNA 5'-end processing
. . <- GO:0090646 tRNA processing
. . . <- GO:1990180 mitochondrial tRNA 3'-end processing
. . . <- GO:New: mitochondrial tRNA 5'-end processing

Children of RNA repair (suggested, can be discussed if specific 5' terms are necessary in conjunction with other terms)
. . <- GO:0042245 RNA repair
. . . <- GO:New: tRNA repair
. . . . <- GO:New: mitochondrial tRNA repair

3′-5′ RNA polymerase activity  - all DNA/RNA polymerase/transcription activity by default is 3' → 5' , so not sure what parentage could be, certainly not regular transcription, RNA-templated - maybe even up to RNA metabolic process?
. . <- GO:New: 3′-5′ RNA polymerase activity

Thanks!
Petra

HI Petra,
For your #3, it looks like your are suggesting an MF term for "3′-5′ RNA polymerase activity", but the possible parent terms you are talking about, e.g. "transcription, RNA-templated" are BP terms.
Whether an MF term for  "3′-5′ RNA polymerase activity" can go under one of the existing MF terms (two relevant term defs included below) depends on whether it would fit within the definitions of the reactions. Here are two existing term definitions:

"RNA polymerase activity" (GO:0034062) -
Def: Catalysis of the reaction: nucleoside triphosphate + RNA(n) = diphosphate + RNA(n+1); the synthesis of RNA from ribonucleotide triphosphates in the presence of a nucleic acid template.
"DNA-directed RNA polymerase activity" (GO:0003899) -
Def: Catalysis of the reaction: nucleoside triphosphate + RNA(n) = diphosphate + RNA(n+1). Utilizes a DNA template, i.e. the catalysis of DNA-template-directed extension of the 3'-end of an RNA strand by one nucleotide at a time. Can initiate a chain 'de novo'.

The current definition of "RNA polymerase activity" does not appear to specify directionality of the reaction, but I don't know enough about the 3'-5' RNAP activity to be able to say if it fits into this definition. A brief skim of the Jackman et al review suggests to me that the reactions of the 5'-3' and 3'-5' RNA polymerases are largely similar, though it also mentioned that some of these 3'-5' RNA polymerases can use an "activated substrate" that has a triphosphate at the 5' end don't require an exogenous ATP. That might suggest a need to expand the definition of "RNA polymerase activity".
In contrast, the definition of "DNA-directed RNA polymerase activity" DOES specify extension of the 3'-end of an RNA strand. [Perhaps adding 5'-3' to the term name would be useful.]
If the current definition of "RNA polymerase activity" (GO:0034062) is broad enough for these 3'-5' RNAP activities, then it seems we could create your suggested new MF term under this parent.
However, also take a look at the existing term "tRNA nucleotidyltransferase activity" (GO:0009022) to see if this is already pretty much what you want. It might be that we need to do is adjust the definition and parentage of this term, rather than create a new term.
-Karen
Jackman JE, Gott JM, Gray MW. Doing it in reverse: 3'-to-5' polymerization by
the Thg1 superfamily. RNA. 2012 May;18(5):886-99. doi: 10.1261/rna.032300.112.
Epub 2012 Mar 28. Review. PubMed PMID: 22456265

Hi Karen,
Thanks, and sorry that I mixed MF and BP in 3) of the long request, at the end. The current def of GO:0008193: tRNA guanylyltransferase activity is somewhat specific, but then either not specific enough or too broad to use for all these enzymes.
Def: Catalysis of the posttranscriptional addition of a guanyl residue to the 5' end of a tRNA molecule; observed for His tRNAs.
Thg1, which is currently annotated to this term (child of term you proposed), adds a G at position -1 of tRNAHis (standard RNA numeration). PMID:27484477, also from the Jackman group, has a nice drawing of this in Fig. 5. The Jackman review you cite also talks specifically about the G-1 addition by Thg1. And this happens by Thg1 exclusively on cytosolic tRNAHis. So maybe this term needs tightening, will look more into specificity of the other enzymes.
I'm in Europe and running out of time as staying with friends. On Monday I'll fly back to US. Will add to 3) and come up with suggestions some time next week.
If the BP terms 1) and 2) are more straightforward, maybe those can be taken care off first.
Thanks,
Petra

Hi Paola,
I think number 1 can be added, as the sibling for 3' is there already. Let's solve these term requests one by one:

tRNA 5'-end processing / mitochondrial tRNA 5'-end processing
. . <- GO:0090646 tRNA processing
. . . <- GO:1990180 mitochondrial tRNA 3'-end processing
. . . <- GO:New: mitochondrial tRNA 5'-end processing

Thanks!

Hi @pfey03 ,
Wrt your request number 1) for GO:New: mitochondrial tRNA 5'-end processing, I will add as follows:
GO:New: mitochondrial tRNA 5'-end processing
is_a mitochondrial tRNA processing
is_a RNA 5'-end processing
The process in which the 5’ end of a pre-tRNA molecule is converted to that of a mature tRNA in the mitochondrion.
GOC:pf
Please confirm which PMID(s) I should attach to this term among the ones you indicated (i.e. PMID:27484477, PMID:26143376, PMID: 21307182).

Hi again @pfey03 ,
Wrt your requests number 2) for GO:New: tRNA repair and GO:New: mitochondrial tRNA repair, are those processes really different from their parent GO:0042245 RNA repair? Do they occur differently? It sounds as if it’d be better to annotate to ‘RNA repair’ and add annotation extensions for GO ’mitochondrion’ and SO ‘tRNA’. But if those processes should indeed be represented by separate terms, please suggest definitions for both to underline the differentia, and confirm which PMID(s) I should attach to each term among the ones you indicated (i.e. PMID:27484477, PMID:26143376, PMID: 21307182).
Thanks!

Finally, @pfey03 , for your request 3) GO:New: 3′-5′ RNA polymerase activity, I will wait for you to look into it further.

Hello again @pfey03 ...
Having second thoughts about your request 1) based on my comment about 2). Could you please explain how is GO:New: mitochondrial tRNA 5'-end processing different from an RNA 5'-end processing that occurs on a tRNA and in the mitochondrion? (Can't remember, do you not use AEs?) Or maybe it is truly different. Thanks.

Hello @paolaroncaglia
Request 1)
I thought this is easy to add because the equivalent 3' term exists already (GO:1990180 mitochondrial tRNA 3'-end processing).
YES, we do AE in P2GO though they are not yet imported into our current, old database. Working on new database and then it will also be exported with extensions, but will still be a little while. But therefore, I agree that 'occurs_in mitochondrion' could easily be added.
However, 3' processing and 5' processing should be differentiated as they are distinctly different processes. But the general tRNA processing terms are NOT available. So maybe we should do this:
GO:0008033 tRNA processing
-> GO:NEW tRNA 3'-end processing
-> GO:NEW tRNA 5'-end processing
OBSOLETE: GO:1990180 mitochondrial tRNA 3'-end processing, and annotate current entries to new parent 'tRNA 3'-end processing'
In sum, If you obsolete the the mitochondrial tRNA 3' term, I think the processes are essentially the same in the different compartemnts. Just the enzymes are typically very specific for the compartment they act on tRNAs.
Let me now if this seems a good plan for 1).

Hi @pfey03 ,
I was going to say that your plan for 1) looked good to me, but I’m now not so sure based on your note that “I think the processes are essentially the same in the different compartments. Just the enzymes are typically very specific for the compartment they act on tRNAs.”. So those enzymes would always be coannotated to ‘mitochondrion’ and to the processing term. Based on what we’ve said at previous GO meetings, that may be ground for allowing specific mitochondrial processing terms… (whether they’d be created automatically or not.)
So, I added your new term as follows:
[Term]
+id: GO:0097745
+name: mitochondrial tRNA 5'-end processing
+namespace: biological_process
+def: "The process in which the 5' end of a pre-tRNA molecule is converted to that of a mature tRNA in the mitochondrion." [GOC:pf, PMID:21307182, PMID:26143376, PMID:27484477]
+is_a: GO:0000964 ! mitochondrial RNA 5'-end processing
+is_a: GO:0090646 ! mitochondrial tRNA processing

@pfey03 ,
Now back to your requests number 2) for GO:New: tRNA repair and GO:New: mitochondrial tRNA repair. Are those processes really different from their parent GO:0042245 RNA repair? Do they occur differently? Or are they the same, but are there specific gene products that only act on tRNAs (i.e. not on other types of ncRNAs)? If so, are gene products involved in mitochondrial tRNA repair ever known to work outside of the mitochondrion? It might be better to annotate to ‘RNA repair’ and add annotation extensions for GO ’mitochondrion’ and SO ‘tRNA’. But if those processes should indeed be represented by separate terms, please suggest definitions for both to underline the differentia, and confirm which PMID(s) I should attach to each term among the ones you indicated (i.e. PMID:27484477, PMID:26143376, PMID: 21307182).
Thanks!

@pfey03 ,
Finally, for your request 3) GO:New: 3′-5′ RNA polymerase activity, when you’ve had a chance to look into it further, if you’d like to suggest any additions or changes, please open a new ticket. That would be easier and quicker, rather than disentangling conversations in tickets containing multiple threads. Thanks again.

Hi @paolaroncaglia,

Thanks for doing what I initially thought is the solution, adding the corresponding 5' term!

Will get back about 2) soon, and I'll open a new ticket for 3) once I'm up to date on it, reading some more, suggesting def and parentage etc.
Thanks for your patience!

@paolaroncaglia,

I read a recent review (PMID:27536007) about RNA effectors and repair. It seems a lot is still unknown, and probably we leave the unspecific 'GO:0042245 RNA repair' alone for now. As it is here for me, if there are other processes and/or functions better to describe an RNA repair protein, this term might be fine as is. However, at one point with more reading and data, 'RNA repair' might not be childless forever.

Will add new issue for 3) soon!

Thanks @pfey03 . Closing this one now.


Update module.json

Added Italian localization

Thank you for this contribution. Unfortunately, because you only translated the dndv3 branch, the module will be missing one entry in the Italian localization file for the main branch. Would you mind also translating the following so I can update both branches (and thus have Italian for all users of this module)?
English:

The parent item has no limited uses that can be consumed, hence this field is disabled.

This is a tooltip in the overrides dialog to tell the user that they can't set the spell to use charges of the parent item.

Also, it would've been better if both file changes would be part of the same pull request. You can do that by making the changes in the same branch of the repo on your account. GitHub will then update the pull request with your changes, so I can merge them in one go.

English:

The parent item has no limited uses that can be consumed, hence this field is disabled.
This is a tooltip in the overrides dialog to tell the user that they can't set the spell to use charges of the parent item.
EDIT For now I'll use the translation that Deepl gave me:
L' oggetto principale non ha usi limitati che possono essere consumati, quindi questo campo è disattivato.

L' oggetto principale non ha usi limitati che possono essere consumati, quindi questo campo è disattivato.
It's a very good translation


Document how to whitelist IPs

"We are having issues when creating an ingress with the number of loadBalancerSourceRanges and open ports exceeding 50. The aws loadbalancer will not be created as error authorizing security group ingress: RulesPerSecurityGroupLimitExceeded. Amazon say that the maximum number of rules is 50 and cannot be increased. Is there possibly a way around this. Say to try bundle rules together or open all tcp for the loadBalancerSourceRanges "
One option will be to allow more IPs from outside. Then restrict at the HAProxy level . https://raymii.org/s/snippets/haproxy_restrict_specific_urls_to_specific_ip_addresses.html

ingress.appscode.com/keep-source-ip: "true" must be used to ensure that HAproxy can see client IP.

@sadlil , in TCP mode, this needs to be added to frontend. :(
http://cbonte.github.io/haproxy-dconv/configuration-1.5.html#4-tcp-request connection
http://marcelog.github.io/articles/restrict_block_acl_ip_address_haproxy_tcp.html

This can be done using Frontend Rules
https://github.com/appscode/voyager/blob/master/docs/user-guide/ingress/frontend-rule.md


How can I port this into unity?

Hello thank you for this, I am very nube i .net framework and have no clue how to work with the .net CLI, so cant download the init package, I am very confused. I am linux (fedora) and need help. Could you provide a bit of a nube guide pls.

try options from here https://stackoverflow.com/questions/53447595/nuget-packages-in-unity
i am not familiar with unity project structure, but if it generates sln file, you can open solution via rider (with gui nuget package manager) or using terminal; also check that dotnet is installed in your system (dotnet --info)


Deb sign

What does this PR do?
Introduces deb signing during publishing
Motivation
Validate binary distribution

@Voronenko
can you resolve the merge conflict ?
and where is the .asc key that we can use for apt-key add ?

@TizianoPerrucci , correct, adjusted and superseded with  https://github.com/StackVista/stackstate-agent/pull/26


NavBar component

This is the element at the bottom of every screen with drawer, explore, add evidence, my obs, and notifications. In designs we've been calling it the Tab Bar, and in code we're calling it the Footer, but I want to unify around the term NavBar (nothing's perfect, but I think that's a little less-likely to be confused with other elements... maybe).

[ ] Rename Footer to NavBar
[ ] Replace My Obs button image with the current user's profile pic when signed in, use the same generic user icon when signed out
[ ] Make icon green when the user is viewing that part of the app (for My Obs while signed in the profile pic should have a green border
[ ] Add those drop shadows on the whole bar and the add evidence button
[ ] Adjust dimensions to match the spec

I don't think this needs to be in the UI library since it's only used in one, really obvious place.

I can make some incremental changes on this. I can have a PR up for renaming, user's profile pic, and making icons green.

This PR https://github.com/inaturalist/iNaturalistReactNative/pull/407 addresses the bullet points as mentioned above.


Promise support

To handle #24
There is the promise support WIP branch to make the cache manager handle promises.
I have some issues on the existing cache providers, and for tests. Right now only wrap (simple cache and multi cache) and get/set (multi cache) are implemented.
The issue with single store cache is that cache-manager is directly binded to the underlying methods, and to add promise support in get I need to change the method usage and add a breaking change, that I would like to avoid.
Some more tests to be added (100% on wrap, drop to 97% on get/set ATM)

Travis should be configured to use node >= 0.11.13 with --harmony tags to make the tests pass, or a recent stable version including native Promises. A shim can be used to keep retro-compatibility with older versions of node, such as Bluebird, but this makes the dependency list grow

:+1: !!!

Thanks for this.  I need to think a bit about how to go forward.  Maybe releasing a 2.0.0 version with promise support and tell anyone using older versions of node need to use 1.2.x.  I'm hesitant to do that though.
Also, is it possible for a user to just do promisifyAll() or equivalent to get this to work?  I.e, is it difficult for a user to get 1.2.2 to work with promises without changing the node-cache-manager code?

I think a user can use promises using promisifyAll but I'm not sure about that because I don't used much libs such as bluebird.
What libs are usually doing, is returning a promise only if the callback parameter is not provided. Is it something we can imagine, so lib would stay in 1.x?

Just did that, but I'm using deprecated defer() methods with promises, this should be refactored but need more work.
ATM it is compatible using callback on every node versions, and using promises on node > 0.12
However not compatible with polyfills and bluebird because of deprecated method usage

@BryanDonovan What do you think of this PR now ?
I think it's an awesome improvement and to be honest, I can't wait to use it this way !!

This is looking good.  I'm not sure I understand the implications of using the deprecated defer() method though.

Here we go: I removed the deprecated usage of defer() and used simple wrappers around functions when no callback are provided.
Now it is compatible with older versions of node, as long as users use the lib the same way as they did before: no breaking changes.
If they have node > 0.12, they have Promise support and can use the lib the same as before or with Promises.
With node < 0.12 you can use the lib with promises if you include a Polyfill, like:
global.Promise = require('es6-promise').Promise;

I think it is safe to tag it 1.3

:+1:

OK, this is now in 1.3.0.  Thanks guys!


Added a check for links in the Say command to shorten things up

Not sure if I pulled this off correctly (it's my first C# in 10 years) but figured I'd share and give you something to laugh at ;)

I worked on my branch that had the start of a test project.
It is in there now.  I have some stuff ignored and commented out.
Give it a look and see if you can get your stuff tested.

Killing this PR


[OpenAI] Bug:function_call no longer takes auto

Previously, function_call accepted the auto value but not now.
Recorded test JSON:
Azure:
https://github.com/Azure/azure-sdk-assets/blob/java/openai/azure-ai-openai_a191d5c4de/java/sdk/openai/azure-ai-openai/src/test/resources/session-records/OpenAIAsyncClientTest.testChatFunctionAutoPreset[1].json#L43
Non-Azure:
https://github.com/Azure/azure-sdk-assets/blob/java/openai/azure-ai-openai_a191d5c4de/java/sdk/openai/azure-ai-openai/src/test/resources/session-records/NonAzureOpenAIAsyncClientTest.testChatFunctionAutoPreset[1].json#L43
Exception in thread "main" com.azure.core.exception.HttpResponseException: Status code 400, "{
"error": {
"message": "Invalid value for 'function_call': no function named 'auto' was specified in the 'functions' parameter.",
"type": "invalid_request_error",
"param": "function_call",
"code": null
}
}

It doesn't work for both azure and non-azure endpoint.

After further more investigation, it turns out the issue is azure-search-documents's dependency azure-core-serializer-json-jackson is not compatible with azure-core serialization. Java OpenAI SDK works fine independently but not along with azure-search-documents. The work around solution is to exclude azure-core-serializer-json-jackson explicitly.
<dependency>
   <groupId>com.azure</groupId>
   <artifactId>azure-search-documents</artifactId>
   <version>11.6.0-beta.7</version>
   <exclusions>
       <exclusion>
           <groupId>com.azure</groupId>
           <artifactId>azure-core-serializer-json-jackson</artifactId>
       </exclusion>
   </exclusions>
</dependency>


Remove nsp

Relates to issue #9

Pull Request Test Coverage Report for Build 23

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.765%

Totals

Change from base Build 22:
0.0%

Covered Lines:
160

Relevant Lines:
162

💛  - Coveralls


Java project can't import classes from 3.0-26-gf176d02

On pom.xml I've added the following:
       <dependency>
            <groupId>org.jitsi</groupId>
            <artifactId>ice4j</artifactId>
            <version>3.0-24-g34c2ce5</version>
        </dependency>

The IDE is able to download the dependencies, I can check the existence of the JAR file at ~/.m2/repository/org/jitsi/ice4j I even extracted the content of the JAR file and can see that contains some .class files under org/jitsi but somehow the maven and java cant import or compile with this version.
Is this happening for anyone else?

Out of nowhere started to working. I will close.

Please use the community forum before opening new issues, thank you.

@damencho Sure, I was thinking could be build/release problem. Apologies.


WIP: Compile-test OpenBSD

I forgot why we weren't able to do this in the past, but I'd like to get this working for us now as we have too many people submitting PRs that just fix small things on OpenBSD, so let's try to get it to a Tier-2 platform.

Yep, I forgot what the issue was until I opened this. Good to check in on it every so often.


Register

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/2058
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.0 -m "<description of version>" 03e0404dd65dc71dc04a390d814ee8b364bd872e
git push origin v0.4.0

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/2153
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.1 -m "<description of version>" e33fc5fa9923485c765d3d97168ebc99e681e1e3
git push origin v0.4.1


Creating new recordings in the Python SDK doesn't result in new recordings in the viewer

Describe the bug
When I create two recordings with the same application id using the python SDK, only a single recording is shown in the viewer.
To Reproduce
import rerun as rr
rr.init("test", spawn=True)
rr.init("test", spawn=True)

Expected behavior
The viewer should show two recordings with the application id "test"
Screenshots
It only shows a single recording

Rerun version
0.9

I put this on the 0.9.1 milestone but it's ok to move if it's too hard to get in in time.

This works:
fn main() -> Result<(), Box<dyn std::error::Error>> {
    let rec =
        rerun::RecordingStreamBuilder::new("test").connect(rerun::default_server_addr(), None)?;
    let rec =
        rerun::RecordingStreamBuilder::new("test").connect(rerun::default_server_addr(), None)?;
    Ok(())
}

This works:
int main() {
    {
        auto rec = rr::RecordingStream("test");
        rec.connect("<IP_ADDRESS>:9876").throw_on_failure();
    }

    {
        auto rec = rr::RecordingStream("test");
        rec.connect("<IP_ADDRESS>:9876").throw_on_failure();
    }
}

This doesn't work:
rr.init("test")
rr.connect()
rr.init("test")
rr.connect()

This doesn't work:
rr.init("test", spawn=True)
rr.init("test", spawn=True)

This actually behaves as expected, it's just surprising that the recording ID defaults to the process' authkey: this should be opt-in.
Decision:

Make sure the current behavior is documented properly for 0.9.1 (TODO: open a ticket)
Make the authkey logic opt-in for 0.11 (this ticket)

I suggest we do this at the same time as we refactor init overall


Compiler warning when augementing BusinessEventsServiceBusAdapter with custom payloadContext

The section about adding a custom Payload Context https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/business-events/business-events-dev-doc#payload-context
is outdated IMHO.
The class BusinessEventsServiceBusAdapter is marked as internal and as of PU30 the compiler gives a warning when I try to augement its method in my own model.
Please advise on what to do in the case I want to add custom payload context field e.g in the service bus adapter case!

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

 There is a compatibility issue that was un-covered in PU30 which is being tracked as a bug. We are looking into fixing this issue. In the meantime, the troubleshoot wiki has been updated about this issue to create awareness. Thanks again.


complex eigenvalue decomposition bug

Original Report: https://mathnetnumerics.codeplex.com/workitem/5694
// Breakpoint before the decomposition shows the content of the first DenseMatrix in the array of dense matrices (index = [0] )
base {MathNet.Numerics.LinearAlgebra.Complex32.Matrix} = {(0.9275488, 0),(0, 0),(0, 0)(-0.9275488, 0),(0.9275488, 0),(0, 0)(0, 0),(-0.9275488, 0),(0.9275488, 0)}
UserEvd eigVR = new UserEvd(mArrayOld[0]); // halts the program

has this issue been resolved?


Add overloads of GetStream that take a long requestedSize

Resolves #231
This adds overloads to RecyclableMemoryStreamManager.GetStream that take long requestedSize instead of int. This creates consistency between long RecyclableMemoryStream.Length and GetStream and reduces the need to cast.

It's a shame to add yet more overloads to GetStream(). Might adding just GetStream(Guid id, string tag, long requiredSize, bool asContiguousBuffer) be better?

It's a shame to add yet more overloads to GetStream(). Might adding just GetStream(Guid id, string tag, long requiredSize, bool asContiguousBuffer) be better?

Or go the way of XmlReader/XmlWriter and add a settings class. Or in struct.

It does seem like a setting struct might be the way to go at this point. That would be a good idea for a new major version revision, and we can clean up the APIs. I'll go with just adding a single new overload for now.

A struct is a good idea because it will not require allocations.
However, this might grow over time and the copying might become more costly than a short lived class allocation.
It can be passed as in, but that's not available for every target. it could be replaced with ref for those targets, though.
In any way, this would be an added overload that doesn't require an overhaul of the API.

A struct is a good idea because it will not require allocations.
However, this might grow over time and the copying might become more costly than a short lived class allocation.
It can be passed as in, but that's not available for every target. it could be replaced with ref for those targets, though.
In any way, this would be an added overload that doesn't require a overwalk


There was an error processing your request: An error occurred.

Describe the bug
Hi there,
I have place my API key from OpenAI in my .env and on the local website but same error everytime so i cant use this chat
Link to the Bolt URL that caused the error
http://localhost:5173/
Steps to reproduce

Starting project in cmd with pnpm run dev
Go in http://localhost:5173/
Select OpenAI with GPT-4o
Writting prompt
Error occurred

Expected behavior
Let it work
Screen Recording / Screenshot

Platform

OS: Windows
Browser: Chrome Canary
Version: Version 133.0.6907.0 (Build officiel) canary (64 bits)

Provider Used
OpenAI
Model Used
GPT-4o
Additional context
No response

Hi, same error

same too :-)

same

same for me too

Same issue

same

same too...

same here...

SET in .env
OLLAMA_API_BASE_URL=http://<IP_ADDRESS>:11434

don't set

OLLAMA_API_BASE_URL=http://localhost:11434
it is wrong due to some IPV6 problem

and in UI that input field should be clear otherwise it will override the .env

same too...

The issue is ongoing ?

Hello bro,
Hav you set the correct API key in .env file in root folder
If yes give some screenshots of errors from terminal

Hello bro, Hav you set the correct API key in .env file in root folder
If yes give some screenshots of errors from terminal

Hello bro,
Yes i have set the correct API key in .env file in root folder
I give you a screenshot from terminal

has anyone found a solution?

If you are running ollama locally, You must have to set the OLLAMA BASE URL = http://<IP_ADDRESS>:11434 instead of localhost:11434

A solution please

A solution please

Can you share your setup
What model you are using
And anything you facing

A solution please

Can you share your setup What model you are using And anything you facing

There was an error processing your request: An error occurred.

Okey found it
Instead of using http://localhost:11434
Use http://<IP_ADDRESS>:11434  wherever you use it
Because IPV6 conversation will convert localhost to ::1:11434

Okey found it
Instead of using http://localhost:11434 Use http://<IP_ADDRESS>:11434 wherever you use it
Because IPV6 conversation will convert localhost to ::1:11434

It didn't work

Change .env.example -> .env
In .env set the base url of ollama as http://<IP_ADDRESS>:11434

Don't use extra space after it
Check if the ollama is running or not

And if you have set the OLLAMA_BASE_URL in the BOLT UI > Setting > Providers
Then change it to http://<IP_ADDRESS>:11434 also or keep it blank

Change .env.example -> .env In .env set the base url of ollama as http://<IP_ADDRESS>:11434

Don't use extra space after it
Check if the ollama is running or not

And if you have set the OLLAMA_BASE_URL in the BOLT UI > Setting > Providers Then change it to http://<IP_ADDRESS>:11434 also or keep it blank

The settings are already that way now, but the problem still persists.
Ollama logs;
Dec 24 14:30:36 Can ollama[259]: [GIN] 2024/12/24 - 14:30:36 | 200 |    1.289096ms |       <IP_ADDRESS> | GET      "/api/version"
Dec 24 14:33:52 Can ollama[259]: [GIN] 2024/12/24 - 14:33:52 | 200 |    9.732224ms |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:34:02 Can ollama[259]: [GIN] 2024/12/24 - 14:34:02 | 200 |      423.51µs |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:45:24 Can ollama[259]: [GIN] 2024/12/24 - 14:45:24 | 200 |      54.596µs |       <IP_ADDRESS> | GET      "/"
Dec 24 14:45:25 Can ollama[259]: [GIN] 2024/12/24 - 14:45:25 | 404 |       6.442µs |       <IP_ADDRESS> | GET      "/favicon.ico"
Dec 24 14:45:46 Can ollama[259]: [GIN] 2024/12/24 - 14:45:46 | 200 |   11.318114ms |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:46:05 Can ollama[259]: [GIN] 2024/12/24 - 14:46:05 | 200 |     514.286µs |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:46:09 Can ollama[259]: [GIN] 2024/12/24 - 14:46:09 | 200 |     608.689µs |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:54:03 Can ollama[259]: [GIN] 2024/12/24 - 14:54:03 | 200 |       71.99µs |       <IP_ADDRESS> | GET      "/"
Dec 24 14:55:14 Can ollama[259]: [GIN] 2024/12/24 - 14:55:14 | 200 |      28.265µs |       <IP_ADDRESS> | GET      "/"
Dec 24 14:55:20 Can ollama[259]: [GIN] 2024/12/24 - 14:55:20 | 200 |     665.629µs |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:55:23 Can ollama[259]: [GIN] 2024/12/24 - 14:55:23 | 200 |     580.287µs |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:55:34 Can ollama[259]: [GIN] 2024/12/24 - 14:55:34 | 200 |     442.908µs |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:55:34 Can ollama[259]: [GIN] 2024/12/24 - 14:55:34 | 404 |   10.754563ms |       <IP_ADDRESS> | POST     "/api/chat"
Dec 24 14:55:52 Can ollama[259]: [GIN] 2024/12/24 - 14:55:52 | 200 |     471.022µs |       <IP_ADDRESS> | GET      "/api/tags"
Dec 24 14:55:52 Can ollama[259]: [GIN] 2024/12/24 - 14:55:52 | 404 |      790.32µs |       <IP_ADDRESS> | POST     "/api/chat"

You will have to go to the providers from the settings tab and scroll down to ollama and paste ollama base url as - http://<IP_ADDRESS>:11434 (default port used to interact with ollama). Refresh and it shows up now.

You will have to go to the providers from the settings tab and scroll down to ollama and paste ollama base url as - http://<IP_ADDRESS>:11434 (default port used to interact with ollama). Refresh and it shows up now.

Hello again, ollama working on the default port and my settings and other configurations are as you said.

Hello bro, Hav you set the correct API key in .env file in root folder
If yes give some screenshots of errors from terminal

Hello bro, Yes i have set the correct API key in .env file in root folder
I give you a screenshot from terminal

Is this the problem you are facing...
This is how I fixed that issue by myself
If it is not working then try to brainstorm

Hello,
I'm using ollama and the models just don't show up. The settings page is blank, API key is blank, and I set the .env file to http://<IP_ADDRESS>:11434. ths is what the console show me:

Please Help :(

Hello,
I'm using ollama and the models just don't show up. The settings page is blank, API key is blank, and I set the .env file to http://<IP_ADDRESS>:11434. ths is what the console show me:

Please Help :(

Update: I reinstalled Ollama so now the models appear, but  when I try messaging my models, it still shows the error message both in the console and on the webpage.

Let me clarify some important steps for local ollama model

Download Ollama
Download your desired model according to your machine capability.
Open the project file
Rename .env.example to .env
Inside .env set the OLLAMA_API_BASE_URL=http://<IP_ADDRESS>:11434

DONT USE http://localhost:11434 due to IPV6 issues

If you have previously set the OLLAMA_API_BASE_URL inside the BOLT website's setting page > provider, then clear that (if you don't understand this line, let me know)
Restart the server
Connect to a good network (as the web container is being fetched from stackblitz server
Make sure to start the ollama server (you can check it here OLLAMA API BASE URL )

Hello,
I'm using ollama and the models just don't show up. The settings page is blank, API key is blank, and I set the .env file to http://<IP_ADDRESS>:11434. ths is what the console show me:

Please Help :(

You have some messages inside your console, but it is hidden as you haven't scroll down 🙄

Hello,

I'm using ollama and the models just don't show up. The settings page is blank, API key is blank, and I set the .env file to http://<IP_ADDRESS>:11434. ths is what the console show me:

Please Help :(

You have some messages inside your console, but it is hidden as you haven't scroll down 🙄

The rest is just more fetch failed errors

Let me clarify some important steps for local ollama model

Download Ollama

Download your desired model according to your machine capability.

Open the project file

Rename .env.example to .env

Inside .env set the OLLAMA_API_BASE_URL=http://<IP_ADDRESS>:11434

DONT USE http://localhost:11434 due to IPV6 issues

If you have previously set the OLLAMA_API_BASE_URL inside the BOLT website's setting page > provider, then clear that (if you don't understand this line, let me know)

Restart the server

Connect to a good network (as the web container is being fetched from stackblitz server

Make sure to start the ollama server (you can check it here OLLAMA API BASE URL )

I can confirm that I did all that.

Okey encontrou-o
Em vez de usar http://localhost:11434Usar http://<IP_ADDRESS>:11434 onde quer que o uses
Porque a conversa IPV6 converterá localhost para::1:11434

Mayor, you solved my problem, excellent tip.  Thank you.

After many tests and trying all the tips I found on the internet, only one solution worked.
Although it seems like a local problem, or in the platform, it is actually an API problem, or rather, it seems to me that somehow the APIs are not recognizing or validating the requests from the local Bolt, and I can't imagine why this is happening.
The only solution that worked for me was to disable all providers and base urls, except Mistral. I created an API to use with Mistral Large Las, and bingo! Everything worked perfectly.

I hope it works for you!

So what is someone what's to use local llm such as ollama

As in the original request I tried using OpenAI's GPT-4o Model. However, even though I configured the API key as explained by OpenAI and in the git-repo, i get the error "There was an error processing your request: An error occurred."
In the Terminal i get the error "WARN Constants Failed to get Ollama models:  fetch failed".
Why is that? As far as I understood we do not need to have Ollama running to use the OpenAI model?
I also disabled all providers except of OpenAI.
Maybe I do not understand that correctly but I guess this is an issue for many...
Setup:
Macbook Pro
MacOS Sequoia 15.3
Google Chrome
OpenAI GPT-4o Model

Same here. I tried with 4o and Gemini 1.5 Flash running on docker.

Same here, not trying to run ollama but gettnig ollama errors

Same error on mac.
ECONNREFUSED
Checked firewall rules ...
Log message :
Failed to get Ollama models: TypeError: fetch failed
app-dev-1  |     at node:internal/deps/undici/undici:13185:13
app-dev-1  |     at processTicksAndRejections (node:internal/process/task_queues:95:5)
app-dev-1  |     at OllamaProvider.getDynamicModels (/app/app/lib/modules/llm/providers/ollama.ts:41:24)
app-dev-1  |     at async Promise.all (index 0)
app-dev-1  |     at LLMManager.updateModelList (/app/app/lib/modules/llm/manager.ts:71:27)
app-dev-1  |     at Module.getModelList (/app/app/utils/constants.ts:43:10)
app-dev-1  |     at Module.streamText (/app/app/lib/.server/llm/stream-text.ts:99:22)
app-dev-1  |     at chatAction (/app/app/routes/api.chat.ts:101:20)
app-dev-1  |     at Object.callRouteAction (/app/node_modules/.pnpm/@remix-run+server-runtime@2.15.0_typescript@5.7.2/node_modules/@remix-run/server-runtime/dist/data.js:36:16)
app-dev-1  |     at /app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:4899:19
app-dev-1  |     at callLoaderOrAction (/app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:4963:16)
app-dev-1  |     at async Promise.all (index 0)
app-dev-1  |     at defaultDataStrategy (/app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:4772:17)
app-dev-1  |     at callDataStrategyImpl (/app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:4835:17)
app-dev-1  |     at callDataStrategy (/app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:3992:19)
app-dev-1  |     at submit (/app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:3755:21)
app-dev-1  |     at queryImpl (/app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:3684:22)
app-dev-1  |     at Object.queryRoute (/app/node_modules/.pnpm/@remix-run+router@1.21.0/node_modules/@remix-run/router/router.ts:3629:18)
app-dev-1  |     at handleResourceRequest (/app/node_modules/.pnpm/@remix-run+server-runtime@2.15.0_typescript@5.7.2/node_modules/@remix-run/server-runtime/dist/server.js:402:20)
app-dev-1  |     at requestHandler (/app/node_modules/.pnpm/@remix-run+server-runtime@2.15.0_typescript@5.7.2/node_modules/@remix-run/server-runtime/dist/server.js:156:18)
app-dev-1  |     at /app/node_modules/.pnpm/@remix-run+dev@2.15.0_@remix-run+react@2.15.0_react-dom@18.3.1_react@18.3.1__react@18.3.1_typ_3djlhh3t6jbfog2cydlrvgreoy/node_modules/@remix-run/dev/dist/vite/cloudflare-proxy-plugin.js:70:25 {
app-dev-1  |   [cause]: Error: connect ECONNREFUSED <IP_ADDRESS>:11434
app-dev-1  |       at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1607:16)
app-dev-1  |       at TCPConnectWrap.callbackTrampoline (node:internal/async_hooks:130:17) {
app-dev-1  |     errno: -111,
app-dev-1  |     code: 'ECONNREFUSED',
app-dev-1  |     syscall: 'connect',
app-dev-1  |     address: '<IP_ADDRESS>',
app-dev-1  |     port: 11434
app-dev-1  |   }
app-dev-1  | }


demo ko

Your demo is ko
Please fix the problem, I want to see how this plugin looks
The console log says:
leaflet.css:1 Failed to load resource: net::ERR_CERT_COMMON_NAME_INVALID
leaflet-src.js:1 Failed to load resource: net::ERR_CERT_COMMON_NAME_INVALID
AnimatedMarker.js:1 Uncaught ReferenceError: L is not defined
at AnimatedMarker.js:1
demo.js:7 Uncaught ReferenceError: L is not defined
at demo.js:7
at demo.js:63
/favicon.ico:1 Failed to load resource: the server responded with a status of 404 (Not Found)

Yup. If fixed it would be nice to link it here:
https://leafletjs.com/plugins.html


renamed @nimiq/core package

With the merge of https://github.com/nimiq/core-rs-albatross/issues/2449 a new package name has been created.
We merge this PR once the new @nimiq/core version is released

TODO: version in package.json needs to be updated

@sisou did we publish the lib with the new name?

Oh, I thought this was already merged, sorry :/