Unable to use nlu http interpreter with core model loaded in 1.0.2

Rasa version: 1.0.2
Python version: 3.6.3
Operating system: osx
Issue: I have a core model trained & running in the 1.0.1 rasa version. And I need it to use a NLU endpoint over http (RasaNLUHttpInterpreter). I have added the endpoint in my endpoints.yml file, but the core model still keeps defaulted to regexinterpreter. The issue seems to be because the new rasa model appends 'core' & 'nlu' to the unpacked rasa model path to get the respective model paths. And since there's no 'nlu' model at the expected path it defaults to using the RegexInterpreter. (even though there's an NLU 'endpoint' available)
Content of endpoints file (endpoints.yml):
nlu:
  url: http://localhost:8080/

Thanks for raising this issue, @tmbo  will get back to you about it soon.

What command are you using to start the core server and what command are you using to start the nlu server?

@tmbo  Here's the command I am using to start the core server (in 1.0.2):
rasa run --model ../path_to_model --endpoints ../path_to_endpoints_file --credentials ../path_to_credentials_file --enable-api
And I am running a separately hosted NLU server which is on an older version of rasa. And it is available over http at http://localhost:8080/

There are also 2 interesting things in the code at rasa/core/interpreter.py:

The Exception still suggests you to use the --nlu flag to specify the model to be used. But I believe this flag is no longer supported in 1.0 & above versions.

I also tried starting the core server with a rasa model that has both core & nlu models (in 1.0.2). But I also added the nlu endpoint in my endpoints.yml file. In this scenario, the code reaches the section that tries to create a RasaNLUHttpInterpreter. (This seems to be because of an incomplete/incorrect if condition). But since the path provided is the actual filepath to the local NLU model, even the RasaNLUHttpInterpreter is created incorrectly. The code I'm talking about is in the create method at  rasa/core/interpreter.py

I can confirm this bug - we stumbled over this as well.
There does not seem to be a way to use the RasaNLUHttpInterpreter. We tried creating a dummy nlu-%Y%m%d-%h%M%S.tar.gz to satisfy the obscure creation logic in NaturalLanguageInterpreter.create but this fails as well since the unpacked dummy model path will be /tmp/tmpABCD/nlu which has three parts after split which in turn always leads to an exception.
As a workaround we monkey-patched rasa.core.run.load_agent_on_start but this is obviously not a good way forward.
Tested with rasa 1.0.6.

we'll look into it :)


Add CI action to publish wave-cli to homebrew

This PR will add the ability to publish wave-cli package to homebrew

@marcodelapierre can you please test this?

The added formula file seems reasonable.
However, as far as I understand (I am new to JReleaser), we also need a jreleaser.yml in the root dir, with a packages: brew: section, as per the YAML example in  https://jreleaser.org/guide/latest/reference/packagers/homebrew.html .
In fact, I also see that tower-cli has such a file: https://github.com/seqeralabs/tower-cli/blob/master/jreleaser.yml
Munish can you investigate this aspect?
After this addition, I imagine that testing the github workflow on a test repo (personal fork?) should result in the Brew artifact being created.

Also, note this note in the jreleaser page for homebrew:
JReleaser does not generate a valid homebrew-core formula. The generated formulas are meant to be published as [homebrew taps](https://docs.brew.sh/Taps.html), and in their current form will not be accepted in any of the official homebrew repositories.

This seems to imply some additional steps.
Look at the this brew help output:
$ brew help tap
Usage: brew tap [options] [user/repo] [URL]

Tap a formula repository. If no arguments are provided, list all installed taps.

With URL unspecified, tap a formula repository from GitHub using HTTPS. Since
so many taps are hosted on GitHub, this command is a shortcut for brew tap
user/repo https://github.com/user/homebrew-repo.

I think we should be setting up a seqeralabs/homebrew-SOMETHING to host our brew formulas.
Then users will be able to do:
# only once
brew tap seqera/SOMETHING

# and then any time needed
brew install wave
brew install tower-agent
brew install tower-cli
......

@pditommaso if this makes sense, then @munishchouhan you could take care of this after the YAML step above is done.

Yes please! what other projects do usually use for the SOMETHING part?

I think I just love Hashicorp's take: hashicorp/tap
https://www.hashicorp.com/blog/announcing-hashicorp-homebrew-tap

@ewels  pointed out that having to resort the custom tap can be annoying to users.
This is currently a requirement due to JReleaser restriction.
So:

Phil, how bad of a constraint is this? Should we look into opening an issue with JReleaser to improve on this?

Paolo, would this be a deal breaker vs using JReleaser?

Phil: is it possible to have homebrew core formula collection to accept syncing of our custom tap??

For reference: JReleaser page on Homebrew: https://jreleaser.org/guide/latest/reference/packagers/homebrew.html

I think the custom tap is fine. I've seen many times. The tap naming is ok!

Phil, how bad of a constraint is this? Should we look into opening an issue with JReleaser to improve on this?

Not bad - as discussed in the meeting, main downside is that it requires people to read the Seqera docs and copy + paste commands rather than just searching Homebrew or doing brew search wave (ie. better discoverability in core formula).

Phil: is it possible to have homebrew core formula collection to accept syncing of our custom tap??

We looked into this a bit during the meeting and it's not just pushing to the core collection that's an issue, it's that JReleaser creates manifests that are not valid for the core collection. Sounds like there might be quite a bit of work involved (?) so probably not worth it.

@munishchouhan  when you are back, I think we are ready to go ahead and work on the updated jreleaser.yml, see comment further up :   https://github.com/seqeralabs/wave-cli/pull/37#issuecomment-1820202263

new repo has been created for home brew tap
https://github.com/seqeralabs/homebrew-tap

Thanks for the progress Munish. How far are we from being able to test?
E.g. we could have a condition on the brew formula workflow, that checks for a string in the commit name, such as [test brew]

FWIW core formulae are supposed to be built from source. At the moment JReleaser builds formulae from binaries, which is why taps are preferred.
Discoverability is hindered a bit when using taps as one must perform one extra step (linking the tap) before installing the target formula. However, it's a known practice up to a certain point.

Yes, you can invoke either the prepare or package commands in JReleaser. this will generate the formula inside the out/jreleaser/ directory, which you may install from source (using brew) . Be advised that you may need to update the url: to point to a local path in your filesystem as the formula would expect the published url to be available by default.

how is the testing going?

Currently, I am getting this warning, so I will create a zip distribution for homebrew
[WARN]  distribution.wave-cli.brew was disabled because there are no matching artifacts [.zip]

Be advised on the different types of distributions supported by JReleaser. Given your current setup you need either of these 2
https://jreleaser.org/guide/latest/concepts/distributions/binary.html
https://jreleaser.org/guide/latest/concepts/distributions/flat-binary.html

@marcodelapierre @pditommaso I got this working now
you can check
https://github.com/munishchouhan/wave-cli/actions/runs/7102346973
https://github.com/munishchouhan/homebrew-tap

But still need to figure out the platform part
because we have to publish it for AMR and x86

A new GitHub token is required, which have access to write on https://github.com/seqeralabs/homebrew-tap
@jordigg, can you please generate a Github token with the necessary permissions and put it in the environment of this repository?

A new GitHub token is required, which have access to write on https://github.com/seqeralabs/homebrew-tap
@jordigg, can you please generate a Github token with the necessary permissions and put it in the environment of this repository?

@munishchouhan  isn't this something we can do ourselves?
If not, and if Jordi has not actioned this yet, I suggest you raise an internal help-it request to bring it to his attention.

@marcodelapierre @munishchouhan I reached out via Slack to Munish about this.
As far as I know, there are two types of credentials you can use. Organization credentials or a "service account", and I'm not sure which one you need.
For other CI work, the engineering team uses the seqera-sa-eng account which is stored in 1Password under the Engineering Vault. You can login and set up anything you need in there. I'm trying not to be a blocker, so I prefer for people to self-serve and if something depends on me we change it.
I have added the seqera-sa-eng to a ci-automation group and gave it write permissions to this repo.
I don't think the Repo has any admin, not sure which team should have this.
Let me know if you need me to do anything.

@jordigg please give this user seqera-sa-eng write permission on homebrew-repo also

@munishchouhan done.
I have added [ci-automation](https://github.com/orgs/seqeralabs/teams/ci-automation) to the repo.

@pditommaso @marcodelapierre @jordeu This PR is ready for review
you can test this by following the instructions mentioned in the below repository
https://github.com/munishchouhan/homebrew-tap

Finally, I propose to expand the readme in the Tap repository: https://github.com/seqeralabs/homebrew-tap/pull/1

Thanks Munish!

@pditommaso Please review this PR

@pditommaso Homebrew support for Wave CLI merged


mount a volume into postgre container to support run postgre db by a non-root user

On OpenShift clusters, containers will be assigned to an arbitrary user ID within the range of [1000670000,<PHONE_NUMBER>], and they don't have permission to write  /var/run/postgresql which is the default data directory of the database image.
This PR is to fix the issue above by mounting a volume into the container and let the db put data into a subdirectory, so that it will be writable.
chmod: /var/run/postgresql: Operation not permitted

<Explain your change!>
Did you sign your commit? Instructions

Yes

Have you read Contributing guidelines?

Yes

This application is referenced in this doc:
https://docs.konghq.com/mesh/latest/features/openshift-quickstart/


M07-LAB07: quick fix typo

quick fix a typo
Replace issue name M00-LAB00:QUICK_DESCRIPTION, for example "M01-LAB01: My new issue" (or same name as linked Issue)
Related Issue
Link related Github Issue 🢂 Fixes # . (Include issue number after #)
Checklist
Mark completed with "x" between brackets, "[x]", or checking the box once the PR is created:

[ ] Has related GitHub Issue 💥 Create Issue 📝
[ ] Tested it
[ ] Read the PR collaboration guide 👓 Collaboration Guide 📝

Changes proposed in this pull request:

thanks :)


[BD-46] fix: consider 0 a truthy value in input components

Description
Fix useHasValue (hook that checks whether input component has a value) to consider 0 as a truthy value.
Deploy Preview
https://deploy-preview-1848--paragon-openedx.netlify.app/components/form/form-control/
Merge Checklist

[x] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[x] Does your change adhere to the documented style conventions?
[x] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[x] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[x] Were your changes tested in the example app?
[x] Is there adequate test coverage for your changes?
[x] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

@viktorrusakov 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

:tada: This PR is included in version 20.21.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 21.0.0-alpha.12 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Azure Media Services module is outdated

Azure Media Encoder is deprecated since december 2015 and replaced by "Media Encoder Standard"
Also need to update built-in preset list for Media Encoder Standard
"Windows Azure Media Packager", "Windows Azure Media Encryptor", "Storage Decryption" are also deprecated
 (They are referenced in the module but don't seem to be used)

There is a quite old PR #6132 to update the player, but we might rather use Azure Media Player. It's the officially supported player, based on a customized version of Video.js. Style and script are CDN hosted (no source available...)
Are there people using this module ?

This is the PR you should look at instead: https://github.com/OrchardCMS/Orchard/pull/7618

@sebastienros : how can I merge the PR with latest 1.10.x code to test it?


Turrets able to shoot players through walls as well as activating and deactivating on their own

Both in single player and multiplayer modes, turrets have the ability to somehow shoot through solid surfaces including walls and closed doors enabling for unfair deaths to which the player should of been safe behind cover.
I also want to report issues with turrets activating on their own on the map whenever another player in a multiplayer setting (or a single player setting) consumes either a heavy or light potion, almost as if they know exactly where that player is on the map.
(This may also apply to some of the enemies pathfinding AI as well but that is uncertain which is entirely based on the assumption that they look down at the ground and up at the ceiling either repeating the same pathfinding patterns or standing still completely.)
Hope this info was enough to convey the issue as this mod is hella fun.
Thanks.

I'll look into this in the evening. Likely an oversight from the recent change to make turrets able to target smaller players.
Sorry for the inconvenience and thank you for reporting the issue this detailed

As of right now the issue seems to be fixed. Turrets are not longer activating and deactivating and can no longer kill players behind walls and doors based on testing. I will notify you in this thread to mention anything else that may come up, maybe in a heavily multiplayer environment. Thanks!

Not sure if this is an issue with tiny company, but now when turrets detect you in a line of sight, they aim directly upwards. and shoot upwards.

I had to adjust their aiming and lowered it from our side.
Not sure if tiny company patches this aswell. I'll ask the mod dev.
You can use the new config option "DefaultPlayerSize" as an alternative way to change your default scale.
I've tested the turret aiming for sizes between 0.2 and 1.7 and it worked decently so far. Only shoots too high if you stand very close to it.
Once I have more time again I'll revisit this and see if I can adjust it better than now

Aiming is decent as of now. Might get a rework later on


Fix graphql-gateway build

Fixed building of graphql-gateway

Removed auto-generated files from the repo
Referenced go-types module
Added a Taskfile with lint and build steps

Added a workflow for building the graphql-gateway
Added a short README
Added Dockerfile
Split workflows for back-ends to 1 per module

New dependencies:

go-types : Podkrepi.bg types

After this PR is approved I will update the contact module as well. Such that it has a similar structure. At that point I think we can delete the Taskfile.yml from the backend folder and all the proto files

Rebased on master


Introduce Scalar as a placeholder for (number|string|boolean|number)

You currently refer to scalar types as for being 'Simple' types. Would it be possible to replace that by just 'Scalar'? The rationale here would be that we are operating on the programming language level and not on the (xml) schema level.
And, instead of repeatedly writing down
@param {(number|string|boolean|number)} param the param

I'd like to instead
@param {Scalar} param the param

With Scalar also supporting generic notation, e.g.
@param {Scalar<number,string>} param the param

to limit the number of available options in the interface.

Since mozilla does not provide a general documentation page for the existing scalar types, I reckon that upon
@param {Scalar} param the param

the following documentation should be generated
param : Scalar<number, string, boolean, ...>

with the appropriate links pointing towards the official mozilla documentation.

@silkentrance
You can use @typedef or @type to define your custom type and reuse it later.
As long as JS does not support generics, you cannot use them easily. The only way I see is adopting Google Closure Compiler's @template tag to ES6 classes (it will look very similar to typescript generics. It is also [supported by jsdoc3](https://github.com/jsdoc3/jsdoc/issues/

@silkentrance You can use @typedef. I do not plan to support Scalar. Because it is far from JavaScript.
@raveclassic Thanks for comment!


Disabling cursorToEnd?

Version
1.1.1
Test Environment
Firefox, Win10
Current Behavior
When I have markdown enabled as default in the editor then the window will automatically scroll to show the editor (if it needs to be scrolled to, to be seen). If I have WYSIWYG enabled as default then the window will not scroll to the editor.
My guess would be that cursorToEnd is true by default. How can I disable this?
Expected Behavior
I'd like the window not to scroll to the editor's position unless I ask it to.

I did not really understand your issue. Does scrolling automatically scroll to where the editor is? Or does the editor have an initial content and scroll to the end of the content?

Does scrolling automatically scroll to where the editor is?

Yes, it seems that the page automatically shows the editor when you have markdown enabled. It does not focus on the editor, just shows it.

Or does the editor have an initial content and scroll to the end of the content?

Yes here as well. The editor does have initial content and when shown in markdown-mode problem occurs. The problem does not occur in WYSIWYG-mode.

We do have  the same issue when using the editor with a v-model. When the value of the editor is changed from the outside (e.g. when the database query is finished) the watcher for value triggers setValue() of the JS component with cursorToEnd = true. This causes the JS component to set the cursor which also scrolls down the page if the editor is not within the viewport.
Would it make sense to just pass cursorToEnd = false as default? That fixes it for me locally.


Automated cherry pick of #4081: fix: Fix the unexpected json field coverage after GetCustomizeColumns

Cherry pick of #4081 on release/2.13.
#4081: fix: Fix the unexpected json field coverage after GetCustomizeColumns

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@bba9af2). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #4082   +/-   ##
==============================================
  Coverage                ?   6.89%
==============================================
  Files                   ?     508
  Lines                   ?   85845
  Branches                ?       0
==============================================
  Hits                    ?    5916
  Misses                  ?   79429
  Partials                ?     500

Flag
Coverage Δ

#aFlag
6.89% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/groups.go
5.17% <0%> (ø)

pkg/cloudcommon/db/db_dispatcher.go
0% <0%> (ø)

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 bba9af2...40de4cb. Read the comment docs.


Added Image slider Gallery

Don't forget to add the Hactoberfest label
~Thanks

@priyanshu-bhatt This PR cannot be merged as it is not following the rules.
To contribute to this repo, the rule is :

you need to first find an issue and assign it to you in your name Or create your own issue and assign it to you in your name
after that you will then create a pull request of your project indicating the issue no.
Then your PR will be accepted and merged.

I'll Open an issue for this then.


Segment Tree implementation in different languages

Proposal
Implement segment tree methods in different languages.
Describe the solution you'd like
Implement the initialisation, update, and query methods for segment tree, and implementing the concepts of lazy propagation and persistence after the previous 3 methods have been implemented
 Have you read the Contributing Guidelines on Pull Requests?
Yes
Please answer the following questions for yourself before submitting an issue.

[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed

@ag278 @Rj-coder-iitian i would like to solve this in C++ if this issue is accepted

I would like to implement in python.

I would like to implement in Java.

Assigned the issue to @vidit21srivastava in java

Assigned the issue to @vishnu921 in python

Assigned the issue to @PranayB003 in C++

@ag278  kindly review my PR


Remote image access performance improvement, by peer-to-peer distribution system

OverlayBD provides block device with rootfs using remote image, the performance is highly associated with registry throughput and latency.
Large scale container launch may cause hot spot on registry, and P2P distribution system should help.

how does overlaybd connect to p2p service?
as a party of overlaybd backstore, or as an http proxy that worked transparently to overlaybd, or connect overlaybd to a p2p agent?

how does overlaybd connect to p2p service?
as a party of overlaybd backstore, or as an http proxy that worked transparently to overlaybd, or connect overlaybd to a p2p agent?

I think http transparent proxy-like application supposed to be a good choice, so that overlaybd will have no needs to make change for p2p distribution. this suppose to intercept get blob requests

@Coldwings  come on, boy. commit your codes!

hello, can the current p2p code work?

hello, can the current p2p code work?

P2P code in this repo is basicly able to work, but still having a lot work to be done.
The P2P image transportation now have a standalone repo. However, it will be re-designe in nearly future.

hello, can the current p2p code work?

P2P code in this repo is basicly able to work, but still having a lot work to be done.
The P2P image transportation now have a standalone repo. However, it will be re-designe in nearly future.

Is there deployment documentation? When the container engine pulls the image, the uri is generally like this: /v2/ImageName/blobs/sha256:xxxxx ， but the code is like this:
dadi-p2proxy/pkg/p2p/server/proxy_server.go
Line 41 in cc3f176
const matchPattern = "./v2/blobs/sha256/./data"


Cancel submission

When you cancel a running submission, is it really cancelled in the worker?

This issue is a duplicate of:

#872

We can pursue the discussion there.


Missing Active Authority error while creating an account on https://testnet.steemitdev.com

I can not create an account on https://testnet.steemitdev.com. Neither calling steem.broadcast.accountCreate(..) nor preparing, signing and broadcasting transaction manually doesn't work for me giving the same error: "RPCError: missing required active authority:Missing Active Authority ...". Neither active nor owner private key doesn’t work.
Expected behavior
Account is created successfully.
Actual behavior
RPCError: missing required active authority:Missing Active Authority ...
How to reproduce
const steem = require("steem");
steem.api.setOptions({
    url: "https://testnet.steemitdev.com",
    useAppbaseApi: true
});
steem.config.set("address_prefix", "TST");
steem.config.set("chain_id", "18dcf0a285365fc58b71f18b3d3fec954aa0c141c44e4e5cb4cf777b9eab274e");
const keys = steem.auth.getPrivateKeys("newAccount", "newAccountPassword", ['owner', 'active', 'posting', 'memo']);
steem.broadcast.accountCreate("creatorActivePrivateKey", "0.000 TESTS", "creatorAccount", "newAccount",
    { weight_threshold: 1, account_auths: [], key_auths: [[keys.ownerPubkey, 1]] },
    { weight_threshold: 1, account_auths: [], key_auths: [[keys.activePubkey, 1]] },
    { weight_threshold: 1, account_auths: [], key_auths: [[keys.postingPubkey, 1]] },
    keys.memoPubkey,
    "",
    function (err, res) {
        console.log(err || res);
    });

Use your actual values for newAccount, newAccountPassword, creatorAccount and creatorActivePrivateKey.
Environment information
MacOS 10.14, NodeJS 8.11.3

The reason was wrong chain_id. That means condenser_api.get_config returns wrong chain_id for https://testnet.steemitdev.com at the moment.


Do all jobs for branch starts with all

In these case, we want to do all jobs.

bump vyper version
bump k & kevm version

Nice!
Then, kprove_and_test_for_all becomes redundant, I think.
Would you remove kprove_and_test_for_all ?


Eliminate some warnings

Eliminate some warnings
Description

Eliminate some printf() related warning messages by changing specifiers and applying appropriate casts.
Test Steps
Build the source code with GCC Xilinx ARM compiler version 11.2 to generate said warnings.
Related Issue
N/A
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Let me see what I can do Pete. The errors aren't always obvious.

/bot run uncrustify

Looks like the formatting check was broken. The other check failure was just transient. I've pinged @aggarg once again to approve.

Hi @kstribrnAmzn ,
I see, guessing by your commit it was a formatting problem...  I work in Windows and don't have any experience of Uncrustify, I have now added it  to my mingw folders and will see if I can get it working so I can run it before I submit any future pull requests.
Thanks for for looking into it.
Kind Regards,
Pete


[plot gl] Overcome float32 precision

This PR overcomes the float32 precision issue occurring for data with difference only visible with double precision. This is due to sending float32 to OpenGL and using data coordinates as coordinates for the display.
It is not complete and not tested (TODOs: error bars, images, shapes, log scale,...) but it works for the case of #1772.
closes #1772

Thanks for doing this. I briefly tested it as it is now and I can confirm that this resolves my issue #1772.
It did crash, however, when I selected a cross hair cursor and moved the cursor around for a while. It then gave the following error.
Traceback (most recent call last):
  File "/home/pepijnk/src/extern/silx_t20100/build/lib.linux-x86_64-3.5/silx/gui/_glutils/OpenGLWidget.py", line 234, in paintGL
    parent.paintGL()
  File "/home/pepijnk/src/extern/silx_t20100/build/lib.linux-x86_64-3.5/silx/gui/plot/backends/BackendOpenGL.py", line 546, in paintGL
    self._paintFBOGL()
  File "/home/pepijnk/src/extern/silx_t20100/build/lib.linux-x86_64-3.5/silx/gui/plot/backends/BackendOpenGL.py", line 491, in _paintFBOGL
    self._renderPlotAreaGL()
  File "/home/pepijnk/src/extern/silx_t20100/build/lib.linux-x86_64-3.5/silx/gui/plot/backends/BackendOpenGL.py", line 882, in _renderPlotAreaGL
    isXLog, isYLog)
  File "/home/pepijnk/src/extern/silx_t20100/build/lib.linux-x86_64-3.5/silx/gui/plot/backends/glutils/GLPlotCurve.py", line 1262, in render
    self.lines.render(matrix, isXLog, isYLog)
  File "/home/pepijnk/src/extern/silx_t20100/build/lib.linux-x86_64-3.5/silx/gui/plot/backends/glutils/GLPlotCurve.py", line 442, in _renderSolid
    self.yVboData.setVertexAttrib(yPosAttrib)
  File "/home/pepijnk/src/extern/silx_t20100/build/lib.linux-x86_64-3.5/silx/gui/_glutils/VertexBuffer.py", line 210, in setVertexAttrib
    c_void_p(self.offset))
  File "/usr/lib/python3/dist-packages/OpenGL/latebind.py", line 61, in __call__
    return self.wrapperFunction( self.baseFunction, *args, **named )
  File "/usr/lib/python3/dist-packages/OpenGL/GL/VERSION/GL_2_0.py", line 426, in glVertexAttribPointer
    contextdata.setValue( key, array )
  File "/usr/lib/python3/dist-packages/OpenGL/contextdata.py", line 58, in setValue
    context = getContext( context )
  File "/usr/lib/python3/dist-packages/OpenGL/contextdata.py", line 41, in getContext
    """Attempt to retrieve context when no valid context"""
OpenGL.error.Error: Attempt to retrieve context when no valid context

As you say above, you are still working on this so some errors are still to be expected. Just wanted to let you know.

Thanks for the testing and the feedback.
As it is now it clearly needs more work and validation.

At last, reworked plot OpenGL backend to extend precision support by:

subtracting an offset from the curves data before sending them to the GPU (which implied applying axes log scale to data before sending them)
doing CPU matrix operations in float64
displaying shapes, markers, crosshair in pixel coordinates (doing data to pixel conversion on the CPU)

This also includes quite a few bug fixes (in particular with log scaled axes) and code clean-up (remove axes scales shader code).
@titusjan I could not reproduce your issue, under which conditions did it occur?
It still needs a bit of testing.
BTW, this PR does improve precision support for images.

@t20100 Thanks for putting in the effort. I really appreciate it.
Do you mean the issue with the cross hair? I'm afraid I can't remember anything next to what I typed earlier. Next Tuesday I will be at work again and I can try to reproduce it again.

Yes I was wondering about the crosshair issue.

I propose to review it as is and do extra fixes in other PRs.

I will merge it this afternoon after checking it is working with https://github.com/silx-kit/silx/issues/680. This dataset is known to be unaccurate in float32.

LGTM


fastfetch: 2.13.2 -> 2.14.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for fastfetch is: Like neofetch, but much faster because written in C
meta.homepage for fastfetch is: https://github.com/fastfetch-cli/fastfetch
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.14.0 with grep in /nix/store/4ciyf5dykdigl3avlyi6pr8s0r6nwxm5-fastfetch-2.14.0
found 2.14.0 in filename of file in /nix/store/4ciyf5dykdigl3avlyi6pr8s0r6nwxm5-fastfetch-2.14.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

fastfetch

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4ciyf5dykdigl3avlyi6pr8s0r6nwxm5-fastfetch-2.14.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fastfetch https://github.com/r-ryantm/nixpkgs/archive/d313919498cdebe8c1c3d80509c4b6265f9b329a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d313919498cdebe8c1c3d80509c4b6265f9b329a#fastfetch

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4ciyf5dykdigl3avlyi6pr8s0r6nwxm5-fastfetch-2.14.0
ls -la /nix/store/4ciyf5dykdigl3avlyi6pr8s0r6nwxm5-fastfetch-2.14.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

fastfetch
fastfetch.man

Maintainer pings
cc @khaneliman for testing.

Result of nixpkgs-review pr 316083 run on x86_64-linux 1

2 packages built:

fastfetch
fastfetch.man

Result of nixpkgs-review pr 316083 run on aarch64-darwin 1

2 packages built:

fastfetch
fastfetch.man

@NixOS/nixpkgs-merge-bot  merge

Result of nixpkgs-review pr 316083 run on x86_64-darwin 1

2 packages built:

fastfetch
fastfetch.man


Add request and response log

fix #29

Coverage decreased (-0.6%) to 97.113% when pulling 1922c6b22825a6eaaf0fae3783308b69929802f8 on push/log into 9a8e5d41cab205a77b0948b7785bf17a33ef2dc4 on master.

Coverage decreased (-0.6%) to 97.113% when pulling 88dd19026e0e24426133997c1db7783dee0f747a on push/log into 9a8e5d41cab205a77b0948b7785bf17a33ef2dc4 on master.

Coverage decreased (-1.8%) to 95.881% when pulling 18e17f9aa93834f90a0fa3b1630e930780adaf82 on push/log into 9a8e5d41cab205a77b0948b7785bf17a33ef2dc4 on master.

Coverage decreased (-0.9%) to 96.796% when pulling<PHONE_NUMBER>18f66f96e1e4fca018f7e882d63535 on push/log into 9a8e5d41cab205a77b0948b7785bf17a33ef2dc4 on master.

Coverage decreased (-0.8%) to 96.916% when pulling fe6633ea20a78852d71ef424c403bd2814889376 on push/log into 9a8e5d41cab205a77b0948b7785bf17a33ef2dc4 on master.

Coverage decreased (-0.1%) to 97.577% when pulling 5a73ea2a1828c22ee5a0da66e78dcc49d45b8e1c on push/log into 9a8e5d41cab205a77b0948b7785bf17a33ef2dc4 on master.


Put "Required configuration" in the README

Could you please put a "Required configuration" section in the README?  I spent a lot of time wondering why I wasn't collecting any statistics.  I didn't look at the System.Remote.Monitoring Haddock page because the README reads as though contains complete instructions.

(I'm talking specifically about the requirement to use +RTS -T.)

Sounds like a good idea. Mind putting together a pull request?

Thanks, I'll do that.

Thanks.  Merged here: https://github.com/tibbe/ekg/pull/56


Problems with drag events on finite sliders with variable width items

Finite sliders with variable width items can be dragged passed the right hand edge of the slide container.
Expected result:
Assuming a left-to-right slider, the right-hand edge of the last item should never exceed the right-hand edge of the container to avoid undesirable white-space.
Related issue:
Whilst dragging the last item, if the cursor goes beyond the left-hand side of the container, the slider scrolls back to the first item. At this point, the slider can no longer be dragged.
Example:
http://jsfiddle.net/j97s50k7/13/

The related issue is fixed on dev, the expected result won't be.

Did this make it into 1.4.0?
I've just checked the jsfiddle (which is now pointing to 1.4.1) and can still drag the last item all the way to the left.

The second part is. The original bug wont be fixed in 1.*
The way this should work goes against the way the positions get calculated. Look for it in 2.0

Cool, thanks for the update

Is there a quick work around for this until 2.0? Having similar issues http://jsfiddle.net/picard102/mfxep5m0/7/
Is there a way to force new items to clone in?


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1807 ms

Resolved: Bofrost.de performance has improved in 37de0a4 after 29 minutes.


[EPIC] BC Wallet Mediator

Tasks
Acceptance Criteria

[ ] Required - Mediator Service deployment in HA-fashion and running multiple replicas (3+)

This includes the Agent (auto-scaled), wallet (HA instance of Postgres), Proxy (auto-scaled), External Queue (HA instance of Redis or Kafka), Message Workers (auto-scaled).
Note regarding the Proxy, it would be nice to eliminate this layer, though it is useful for traffic control and rate limiting.  However relying on it to route messages to the separate http and ws ports of the aca-py agents is undesirable.
With the use of web sockets auto-scaling is best performed on the basis of the web socket connections themselves.  This helps to ensure that pods with active web socket connections are not terminated by the system scaling them down prematurely.  There are several ways to accomplish this.

[ ] Highly Desired - No session/cookie affinity: A agent may seamlessly connect and be served by any of the replicas

The use of web sockets defines the current need for session affinity.  The socket is opened between a client and an agent/worker instance.  Once established all traffic must be routed between the same client and agent/worker instance.  This is a challenge in K8S/OCP, especially when to comes to HPAS.
Are there any alternatives to using web sockets?

[ ] Required - Uptime/performance monitoring at Aries protocol level (not just http/s)

For example, the k8s compatible status endpoints on the agents do not provide sufficient information regarding the state and health of the websocket connections, nor do they provide any metrics on the durability and longevity of the websocket connections.

Blocked By:

[ ] hyperledger/aries-cloudagent-python#2116

This PR addresses vertical scalability on a single instance mediator, it does not address any of issues encountered in an HA or horizontally scalable environment.

I did a bit of catching-up and while there are still some items that will require review and planning, I think we have a couple of options to focus on for the short/medium term.

The PR linked in the issue description will allow the mediator agent to scale vertically and manage throughput of 2400+ connections: this should be enough to handle the user volumes we expect in the immediate future. This does NOT help with scenarios involving pod rollout, as the in-progress queue would be lost.
This PR (https://github.com/hyperledger/aries-cloudagent-python/pull/2116) includes changes that, in theory, should help with preventing websockets from being dropped due to scaling, by using sticky sessions and affinity. The changes are already deployed in our dev environment, however testing appears to have been interrupted before it could confirm whether the change was helpful or not sue to a shift in priorities. It would be a good idea to wrap-up the testing and confirm whether this approach resolves, or at least mitigates, the horizontal scaling issues.

Both the above approaches should be accompanied by running a persistent queue to handle messages, so that they will not be lost in case of rollouts/re-deployments/failures.
I would suggest we focus on these three items in the short term, and in the meantime completethe investigation of potential next steps/long term strategies to manage mediation.


generation hangs forever

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[x] I carefully followed the README.md.
[x] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[x] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Expected Behavior
The following code should print the output
from llama_cpp import Llama
llm = Llama(model_path="/scratch/yerong/.cache/pyllama/Llama-2-7b/ggml-model-q4_0.gguf", n_gpu_layers= 100, n_ctx=100)
output = llm("Q: Name the planets in the solar system? A: ", max_tokens=32, stop=["Q:", "\n"], echo=True)
print(output)

Current Behavior
Currently it hangs forever
llm_load_print_meta: model ftype    = mostly Q4_0
llm_load_print_meta: model size     = 6.74 B
llm_load_print_meta: general.name   = LLaMA v2
llm_load_print_meta: BOS token = 1 '<s>'
llm_load_print_meta: EOS token = 2 '</s>'
llm_load_print_meta: UNK token = 0 '<unk>'
llm_load_print_meta: LF token  = 13 '<0x0A>'
llm_load_tensors: ggml ctx size =    0.09 MB
llm_load_tensors: using CUDA for GPU acceleration
llm_load_tensors: mem required  =   70.41 MB (+   50.00 MB per state)
llm_load_tensors: offloading 32 repeating layers to GPU
llm_load_tensors: offloading non-repeating layers to GPU
llm_load_tensors: offloading v cache to GPU
llm_load_tensors: offloading k cache to GPU
llm_load_tensors: offloaded 35/35 layers to GPU
llm_load_tensors: VRAM used: 3628 MB
..................................................................................................
llama_new_context_with_model: kv self size  =   50.00 MB
llama_new_context_with_model: compute buffer total size =   15.24 MB
llama_new_context_with_model: VRAM scratch buffer: 13.77 MB
AVX = 1 | AVX2 = 1 | AVX512 = 1 | AVX512_VBMI = 1 | AVX512_VNNI = 1 | FMA = 1 | NEON = 0 | ARM_FMA = 0 | F16C = 1 | FP16_VA = 0 | WASM_SIMD = 0 | BLAS = 1 | S
SE3 = 1 | SSSE3 = 1 | VSX = 0 |

Environment and Context
Please provide detailed information about your computer setup. This is important in case the issue is not reproducible except for under certain specific conditions.

SDK version, e.g. for Linux:

(lla) [yerong2@ccc0351 self-instruct]$ python3 --version
Python 3.11.4
(lla) [yerong2@ccc0351 self-instruct]$ make --version
GNU Make 3.82
Built for x86_64-redhat-linux-gnu
Copyright (C) 2010  Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
(lla) [yerong2@ccc0351 self-instruct]$ g++ --version
g++ (GCC) 11.2.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Failure Information (for bugs)
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
Please provide detailed steps for reproducing the issue. We are not sitting in front of your screen, so the more detail the better.

make clean;make LLAMA_CUBLAS=1 -j libllama.so
cp libllama.so /scratch/yerong/.conda/envs/lla/lib/python3.11/site-packages/llama_cpp

from llama_cpp import Llama
llm = Llama(model_path="/scratch/yerong/.cache/pyllama/Llama-2-7b/ggml-model-q4_0.gguf", n_gpu_layers= 100, n_ctx=100)
output = llm("Q: Name the planets in the solar system? A: ", max_tokens=32, stop=["Q:", "\n"], echo=True)
print(output)

Note: Many issues seem to be regarding functional or performance issues / differences with llama.cpp. In these cases we need to confirm that you're comparing against the version of llama.cpp that was built with your python package, and which parameters you're passing to the context.
Try the following:

git clone https://github.com/abetlen/llama-cpp-python
cd llama-cpp-python
rm -rf _skbuild/ # delete any old builds
python setup.py develop
cd ./vendor/llama.cpp
Follow llama.cpp's instructions to cmake llama.cpp
Run llama.cpp's ./main with the same arguments you previously passed to llama-cpp-python and see if you can reproduce the issue. If you can, log an issue with llama.cpp

Failure Logs
Please include any relevant log snippets or files. If it works under one configuration but not under another, please provide logs for both configurations and their corresponding outputs so it is easy to see where behavior changes.
Also, please try to avoid using screenshots if at all possible. Instead, copy/paste the console output and use Github's markdown to cleanly format your logs for easy readability.

+1, It is not working, i tried using two different types of model, but both didn't work. It just get stuck when you try to load it.

I tested the latest version, still the python binding does not work while the original ./main works.

me too

In your steps you enter the llama-cpp-python folder and there call "python setup.py", but the file setup.py does not exist in this code repo, where did you get it from?


[github-actions] pty-macos fails intermittently

logs_78735.zip

It looks like ot-daemon got stuck. Need more time to debug.


Adding pagination to SelectionList component

Details

This PR adds pagination feature to the SelectionList component the same way as is in currently deprecated OptionsSelector.
Fixed Issues

$ https://github.com/Expensify/App/issues/36121
PROPOSAL: no-proposal
Tests

same as QA
[ ] Verify that no errors appear in the JS console

Offline tests

same as QA

QA Steps

[ ] Open any list that uses SectionList (like SearchPage) on a high traffic account
[ ] scroll to the very bottom and tap "ShowMoreBottom"
[ ] look for any odd behaviour
[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/134388952/da8cf7d6-0036-45e7-8f03-4875347c295d

Android: mWeb Chrome
https://github.com/Expensify/App/assets/134388952/fcb2cc13-935a-462b-9235-49449e9d8b29

iOS: Native
https://github.com/Expensify/App/assets/134388952/13ebbbce-b866-47da-adde-b16b8bde0e4c

iOS: mWeb Safari
https://github.com/Expensify/App/assets/134388952/073e3df6-989c-403b-9b93-4d4a306d1e7b

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/134388952/2dc83dc5-0c58-4d11-b06a-0c38b975f34d

MacOS: Desktop
https://github.com/Expensify/App/assets/134388952/4ada5d2c-6fc7-4fd3-81a6-c9303dfb121e

All checks were passed when I merged 🤔

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/cristipaval in version: 1.4.51-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.4.51-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Troubles using external libs

Hey Isaac (@ityonemo) great work with Zigler. I am looking forward to using it in my projects.
I am having some troubles getting an external library to work and was wondering if you could help.
(Link to repo with reproducible code: https://github.com/munjalpatel/exzig)
Here is what I am trying to do:
Go code --> shared lib (.h and .so) --> Elixir (using Zigler)

Here is the error I am getting when try to run iex -S mix:
Erlang/OTP 25 [erts-13.1.3] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1] [jit:ns] [dtrace]

Compiling 2 files (.ex)

15:04:24.143 [debug] running command: /Users/munjalapatel/workspace/playground/elixir-learning/exzig/deps/zigler/zig/zig-macos-x86_64-0.10.1/zig fmt /var/folders/zv/ypxkdwcx46d9vrf40tckqq880000gn/T/Elixir.Greeter/build.zig

15:04:24.424 [debug] wrote build.zig to /var/folders/zv/ypxkdwcx46d9vrf40tckqq880000gn/T/Elixir.Greeter/build.zig

15:04:24.427 [debug] running command: /Users/munjalapatel/workspace/playground/elixir-learning/exzig/deps/zigler/zig/zig-macos-x86_64-0.10.1/zig run /Users/munjalapatel/workspace/playground/elixir-learning/exzig/_build/dev/lib/zigler/priv/beam/sema.zig --pkg-begin analyte /Users/munjalapatel/workspace/playground/elixir-learning/exzig/lib/.Elixir.Greeter.zig --pkg-begin beam /Users/munjalapatel/workspace/playground/elixir-learning/exzig/_build/dev/lib/zigler/priv/beam/beam.zig --pkg-begin sema /Users/munjalapatel/workspace/playground/elixir-learning/exzig/_build/dev/lib/zigler/priv/beam/sema/on.zig --pkg-end --pkg-begin erl_nif /Users/munjalapatel/workspace/playground/elixir-learning/exzig/_build/dev/lib/zigler/priv/beam/erl_nif.zig --pkg-end --pkg-end --pkg-begin erl_nif /Users/munjalapatel/workspace/playground/elixir-learning/exzig/_build/dev/lib/zigler/priv/beam/erl_nif.zig --pkg-end --pkg-begin sema /Users/munjalapatel/workspace/playground/elixir-learning/exzig/_build/dev/lib/zigler/priv/beam/sema/on.zig --pkg-end --pkg-end --pkg-begin sema /Users/munjalapatel/workspace/playground/elixir-learning/exzig/_build/dev/lib/zigler/priv/beam/sema/on.zig --pkg-end -lc

== Compilation error in file lib/greeter.ex ==
** (MatchError) no match of right hand side value: ["error(link): undefined reference to symbol '_Greet'\nerror(link):   first referenced in '/Users/munjalapatel/.cache/zig/o/0e040908f395283c64be94201a6a73b1/sema.o'"]
    (zigler 0.10.3) lib/zig/compile_error.ex:11: Zig.CompileError.to_error/2
    (zigler 0.10.3) lib/zig/sema.ex:88: Zig.Sema.run_sema!/3
    (zigler 0.10.3) lib/zig/compiler.ex:84: Zig.Compiler.compile/4
    (zigler 0.10.3) expanding macro: Zig.Compiler.__before_compile__/1
    lib/greeter.ex:1: Greeter (module)

I verified that the compiled .so has the Greet in the dynamic symbol table.
Here is the output of nm greeter.so | grep "Greet"

0000000000093f40 T _Greet
0000000000093ec0 t __cgoexp_d7fc1fbdb657_Greet
0000000000093e20 t _main.Greet

I also created a small tester in C to ensure the compiled code works as expected.
#include "greeter.h"
#include <stdio.h>

// Build using: gcc -o test_greeter test_greeter.c ./greeter.so
// Run test: ./test_greeter

int main()
{
  char *greeting = Greet("Munjal");
  printf("%s", greeting);

  return 0;
}

The output of gcc -o test_greeter test_greeter.c ./greeter.so && ./test_greeter
Hello, Munjal!

Here is my setup:
lib/greeter/src/main.go
package main

// Build using: go build -buildmode=c-shared -o greeter.so ./src/...

// #include <stdio.h>
// #include <stdlib.h>
import "C"
import "fmt"

//export Greet
func Greet(name *C.char) *C.char {
    return C.CString(fmt.Sprintf("Hello, %s!\n", C.GoString(name)))
}

func main() {}

lib/greeter.ex
defmodule Greeter do
  use Zig,
    otp_app: :exzig,
    link_lib:
      "/Users/munjalapatel/workspace/playground/elixir-learning/exzig/lib/greeter/greeter.so"

  ~Z"""
  const beam = @import("beam");

  const greeter = @cImport({
    @cInclude("/Users/munjalapatel/workspace/playground/elixir-learning/exzig/lib/greeter/greeter.h");
  });

  pub fn greet(env: beam.env, name: []u8) beam.term {
    return beam.make(env, .{.ok, greeter.Greet(@as([*c]u8, &name[0]),)}, .{});
  }
  """
end

lib/exzig.ex
defmodule Exzig do
  def greet(name) do
    Greeter.greet(name)
  end
end

error says "undefined reference to _Greet" I would have expected it to say "undefined reference to Greet"  Is there something going on with the underscore?


DASHBUILDE-88: Dataset columns are case sensitive (0.4.x)

Fix + tests.
Verified against all the supported DBs

+1


wasmtime: In-process sampling profiler

This isn't ready for detailed review but the basic functionality seems to work so I'm putting it up in case anyone wants to play with it.
Unlike the existing profiling options, this works on all platforms and does not rely on any external profiling tools like perf or VTune. On the other hand, it can only profile time spent in the WebAssembly guest, not in Wasmtime itself or other host code. Also it can't measure time as precisely as platform-native tools can.

This PR now produces output that works as expected with https://profiler.firefox.com/, and I don't think it's fundamentally missing anything.
Before merging, I'd like to move the GuestProfiler struct somewhere in the wasmtime crate, instead of leaving it in wasmtime-cli where only wasmtime run can use it.
I don't think any of the implementation will change though, just move elsewhere, so I'm marking this as ready for review.

I got rid of the need for Store in sample(). Then I discovered that I can't put a GuestProfiler directly in the Host structure because some of the WASI extensions require that Host must be cloneable.
As far as I can tell I end up needing an Arc and Mutex anyway. I would love for somebody to tell me how to avoid that.
It was still worth doing this because now any particular PC is only looked up once per profile and cached by fxprof_processed_profile. Different PCs within the same function need to be looked up separately but since we only collect stack traces at epoch interruption points there aren't so many distinct PCs. This should significantly reduce profiling overhead but I haven't measured how much impact it actually has.
I struggled with this a bunch because I was trying to make it automatically use all modules which have been registered within the current Store's ModuleRegistry. But for WASI commands the module isn't registered in the store until you actually try to invoke it, and I tried all sorts of ways to hook into that registration before realizing that the caller knows which modules they care about and I can just make it their problem to tell me.
Then I decided that it's a feature to be able to exclude modules from profiles: if you have some proprietary module that you're linking with your customer's modules, you may not want to reveal anything about the implementation of your module when providing them with a profile.

As far as I can tell I end up needing an Arc and Mutex anyway.

Ah this is due to the integration of wasi-threads where that's modeled by cloning the store's context to duplicate it on another thread. Otherwise though the cloning is not required. You can take a similar route as the WasiNnCtx integration, for example, which additionally does not implement Clone so what it does is it's bundled into an Arc where whenever it's required it calls Arc::get_mut. That fails if there's more than one clone, or more than one thread, and the error message says "this doesn't support threads yet". I think that profiling will fall into the same bucket?
In the long-term future I could imagine that there's a standalone profiler struct where you acquire thread-specific profilers from that structure. The Host could store the standalone profiler (which internally probably has an arc and a mutex) and additionally thread-specific profilers. The Clone for Host impl would then use the standalone profiler to get a new thread-specific profiler, allowing each thread to have mutable access to its profiler but no others. Something like that. No need to do here of course, mostly just modeling how this may look in the future with support for threads.

I can just make it their problem to tell me.

This seems like a neat idea to me! I like the idea of precomputing symbol tables. I do think this has the downside of that it has more setup required, but that's also perhaps not a bad thing and it's not exactly onerous at this time, so it seems fine by me.

I got rid of the need for Store in sample().

Personally I'm a bit wary of this because to me I wouldn't want to commit long-term to being able to capture a stack trace without any input. I'm not sure how we'll ever move away from those thread locals but I sure would love to one day if someone gets a good idea. In such a situation there's no way to start the backtrace without a contextual store being passed  in. All that being said though it's not like we need to get the API 100% prefect to begin with, that's why we have major version bumps after all. I mostly wanted to put this out there, but I additionally think it's fine to land as-is with your current strategy. If and when we remove TLS (fingers crossed) we can figure out how to change this API at this point. I realize that passing the store into something stored within the store is not easy, so let's stick with what you've got.

Thanks for the tips, Alex: that worked!
Okay, I've addressed most of the issues that you all have raised. What I haven't done is added documentation to the Wasmtime book, or API usage examples to the doc-comments. I'd kind of like to merge the implementation and then come back to that, if that's okay. I hope the doc-comments I've written so far will be a good start for the book, and the usage in run.rs will be a good start for the usage examples…
There are also still several to-do items, like measuring CPU time and noting wasm entry/leave events, which I also want to defer to future work.


How to support multiple languages on 1 page?

Would like to print multiple languages on a page like below, how to support this?
Thanks.

Is this a puppeteer question?

Kind of, as I am using Puppeteer Sharp which is deployed to Azure Linux Functions. However, some characters could not be printed successfully there.

Ah got it. I think this should be reported on AZ. This is about the VM running the function.

I reported to Azure support and they have no ideas yet. Thus, they asked me to ask here to see whether other people faced similar issues before with solutions.
Please let me know if others know the solutions.
Thanks.


Could we have Notero generate zotero://select URIs instead of http ones?

I would prefer the Standalone app to open URIs rather than the browser. Could we toggle these in a setting?

Hi @7homR, thanks for the question!
I too would prefer to open items in the Zotero desktop app. This has been discussed previously in https://github.com/dvanoni/notero/issues/9#issuecomment-962534885, and the biggest hurdle is that Notion currently doesn't support opening URLs that don't start with http or https. I've submitted a feature request to Notion, but I haven't heard anything about it yet.
That being said, we could definitely save the zotero://select URL into Notion, but you'd either have to manually copy it and paste it into your browser, or you may be able to use some sort of utility to streamline that a bit. Would that be useful for you?

Thank you for the reply, and sorry to have missed the earlier thread. I'll contact Notion as well with this request, to add some slight further pressure, and explore the File Path property in the meantime.

Just as a follow up, I was happy to discover, by consulting this post, that we can create a TinyURL link which redirects to a zotero://select link that will open Zotero, and just use the TinyURL URL for a clickable link in Notion. Takes some frontend work to register as a TinyURL, but then works great.

My current workaround with Notion, is to use this Chrome Extension Redirect URLs to make a rule that replaces the zotero with zotero://select.

When I click the link in Notion, it opens a tab in chrome, I have to click open in Zotero app, and then it goes to the app. I'd like to try and figure out how to remove the confirmation popup, but it's working pretty well.

I think it would also be a great feature to have the link to the PDF on Zotero so that you could use zotero://open-pdf with it.

Note redirection rules are also possible on Firefox.

have you checked the URI to zotero. Just in Chrome navigate to zotero://select/library/items/xxxxxxx

Hi @7homR, thanks for the question!
I too would prefer to open items in the Zotero desktop app. This has been discussed previously in #9 (comment), and the biggest hurdle is that Notion currently doesn't support opening URLs that don't start with http or https. I've submitted a feature request to Notion, but I haven't heard anything about it yet.
That being said, we could definitely save the zotero://select URL into Notion, but you'd either have to manually copy it and paste it into your browser, or you may be able to use some sort of utility to streamline that a bit. Would that be useful for you?

Hello! Just went down this beautiful rabbit hole and nearly got everything working the way I want it to. The only thing I cant seem to do is get Notion to populate a field with "zotero://select/..." so that I can Tiny URL it after. I managed to get "File Path" but dont see a category for the above. Please let me know if Im missing something!

@dvanoni thought Id tag you too! Thanks again

Hi, based on the comments here I installed this browser plugin  https://github.com/einaregilsson/Redirector
With the regex source url ^https?://(?:www\.)?zotero\.org/[^/]+/items/([^/]+)$
into destination url zotero://select/items/1_$1, which works with Zotero URL uri fine on my mac


test: don't compress test registry crates

They are still nominally gzipped, but using Compression::none() makes
them consistent even across zlib and zlib-ng, and this fixes checksum
differences in the testsuite. There is a one-time update of all those
checksums to catch up with this change though.
r? @weihanglo

Tried turning zlib-ng feature on, it works flawlessly on Linux for me.
@bors r+

:pushpin: Commit a70f23c50b61c1a3335f2943375a04ae7abf2fa4 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit a70f23c50b61c1a3335f2943375a04ae7abf2fa4 with merge 7e31f62a80260c70b0e7604c615ed06fcf98928c...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 7e31f62a80260c70b0e7604c615ed06fcf98928c to master...


cannot resolve 'fs' in 'node_modules/phantomjs_prebuilt/lib'

WARNING in ./~/natives/index.js
44:17-28 Critical dependency: the request of a dependency is an expression
ERROR in ./~/tmp/lib/tmp.js
Module not found: Error: Can't resolve 'fs' in 'E:\ITB with DOmain Tracker\irr\node_modules\tmp\lib'
@ ./~/tmp/lib/tmp.js 13:11-24
@ ./~/webshot/lib/webshot.js
@ ./src/store/reducers.js
@ ./src/store/index.js
@ ./src/index.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 ./src/index.js
ERROR in ./~/webshot/~/cross-spawn/index.js
Module not found: Error: Can't resolve 'fs' in 'E:\ITB with DOmain Tracker\irr\node_modules\webshot\node_modules\cross-spawn'
@ ./~/webshot/~/cross-spawn/index.js 1:9-22
@ ./~/webshot/lib/webshot.js
@ ./src/store/reducers.js
@ ./src/store/index.js
@ ./src/index.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 ./src/index.js
ERROR in ./~/webshot/~/cross-spawn/index.js
Module not found: Error: Can't resolve 'child_process' in 'E:\ITB with DOmain Tracker\irr\node_modules\webshot\node_modules\cross-spawn'
@ ./~/webshot/~/cross-spawn/index.js 3:9-33
@ ./~/webshot/lib/webshot.js
@ ./src/store/reducers.js
@ ./src/store/index.js
@ ./src/index.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 ./src/index.js
ERROR in ./~/phantomjs-prebuilt/lib/phantomjs.js
Module not found: Error: Can't resolve 'fs' in 'E:\ITB with DOmain Tracker\irr\node_modules\phantomjs-prebuilt\lib'
@ ./~/phantomjs-prebuilt/lib/phantomjs.js 8:9-22
@ ./~/webshot/lib/webshot.js
@ ./src/store/reducers.js
@ ./src/store/index.js
@ ./src/index.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 ./src/index.js
ERROR in ./~/phantomjs-prebuilt/lib/phantomjs.js
Module not found: Error: Can't resolve 'child_process' in 'E:\ITB with DOmain Tracker\irr\node_modules\phantomjs-prebuilt\lib'
@ ./~/phantomjs-prebuilt/lib/phantomjs.js 10:12-36
@ ./~/webshot/lib/webshot.js
@ ./src/store/reducers.js
@ ./src/store/index.js
@ ./src/index.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 ./src/index.js
ERROR in ./~/natives/index.js
Module not found: Error: Can't resolve 'module' in 'E:\ITB with DOmain Tracker\irr\node_modules\natives'
@ ./~/natives/index.js 2:13-30
@ ./~/webshot/~/graceful-fs/fs.js
@ ./~/webshot/~/graceful-fs/graceful-fs.js
@ ./~/webshot/lib/webshot.js
@ ./src/store/reducers.js
@ ./src/store/index.js
@ ./src/index.js
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080 ./src/index.js

Use target: 'node' for your nodejs app.
Learn more:
https://webpack.js.org/concepts/targets/
https://webpack.js.org/configuration/target/

Use target: 'node' option in your webpack configuration file for your nodejs app.
Learn more:
https://webpack.js.org/concepts/targets/
https://webpack.js.org/configuration/target/

Use option: target: 'node' in your webpack configuration file for your nodejs app.
Learn more:
https://webpack.js.org/concepts/targets/
https://webpack.js.org/configuration/target/

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Fix CSS Not found and Images download

Hi, I tried to fix CSS not found issue at #196, #198
thanks to @jp-13

Hi @noxymon,
thank you. Could you test the last push with this different approach to the css?
Thanks!!


force chunk in concat_rows

The problem is described here, and this is to follow through with the suggestion here

Hi @josevalim, thanks for the feedback!
I've separated the test for this. There are existing tests for concat_rows, but I've added a new section just because it feels slightly out of place with the mutate and explanation that it may panic. Please let me know if you'd like it done differently.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Why "spaces indentation is not supported"?

Sounds strange to me.
can you tell me how difficult it will be to support spaces as indentation?

It wouldn't be too hard, relative to other implementation tasks.
However, at this point I'm leaning towards not implementing it at all. I've updated the README in 2e0c448 to not waffle about it so much. The reason for this is that I think it's clear that Python's mixed mode, where both tabs and spaces are allowed, makes things harder. In the choice of tabs versus spaces, tabs seem to be a better fit for indentation, simply because that's what they were designed for. I also like the fact that the use of tabs allows people to use different tab sizes as they see fit.
Go is an example of a language mandating tabs for indentation (through go fmt), so I don't think this a bad choice, although it's perhaps slightly incongruous with the desire to be easy for Python developers to pick up. However, I think that would mostly be resolved by setting up editors correctly.
Thanks for the feedback!


How to customize the rule of cluster autoscaler?

Like when to trigger a scaling task?

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

ID: 077f1bb5-2ba9-490c-2c6e-0de292266539
Version Independent ID: 0fd87b4f-88a4-c2eb-3ebc-f2b113ac203c
Content: Use the cluster autoscaler in Azure Kubernetes Service (AKS)
Content Source: articles/aks/cluster-autoscaler.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! We are currently investigating and will update you shortly.

You can find more information here: https://github.com/kubernetes/autoscaler/blob/master/cluster-autoscaler/FAQ.md#what-are-the-parameters-to-ca
Besides that, I see you are an FTE, if you have further questions feel free to ping me offline and I can provide you with some resources and DLs to reach out to.


spotify_player 0.9.4 fails to play any track

@aome510
The player jumps through songs and when reaching the end of the list it starts over again. Killing the app is the only way out if this loop.
Attached the log file, although it generates several of these one after the other.
Backtraces are empty :(
ClientID has been hidden on the log file.
If you need to check my configurations see, https://codeberg.org/pin/leftwm-netbsd/src/branch/main/.config/spotify-player
spotify-player-22-08-04-09:32.log
EDIT: Downgraded to 0.9.3 and the bug is already present there. I've been on holiday and just pushed the updates.
Downgraded to 0.9.2 and it's the same, then downgraded to 0.9.0 and the same happens.
I'm lost, I start to think that something else is causing this. I've also built 0.8.0 just to be sure, as I know this one worked before and it does the same :(
I've no clue how to fix this or what is causing it.

@aome510 just got confirmed that ncspot is behaving the same for a fellow NetBSD developer, that claims it was working yesterday but, not today. So, not an issue with spotify-player.
Are you aware of any changes at the Spotify end? Are you having any issues yourself?
I've merged the latest librespot, should that dependency be updated?
A lot of questions, I know.

Are you aware of any changes at the Spotify end? Are you having any issues yourself?

I'm not. Just quickly tested with 0.9.3 on my local and still works. It also worked yesterday for me.

I've merged the latest librespot, should that dependency be updated?

I don't know there is a new update for librespot. I can bump a new version.
My best guess is this is because of some Spotify client bug, so let's wait for a few days to see if the issue still persists.

@aome510 Thanks for checking at your end, really appreciated.
I've tried today and it's still the same, I'll give it a few days.
The reason I was mentioning librespot is the ChangeLog provided with the release of v0.4.2, quoting

this point release is mainly to blacklist the ap-gew4 and ap-gue1 access points that caused librespot to fail to playback anything.

So, I thought this could be related. Anyway, librespot is preparing some major changes, I'd recommend to check it out, again quoting,

Development will now shift to the new HTTP-based API, targeted for a future v0.5.0 release. The new-api branch will therefore be promoted to dev. This is a major departure from the old API and although it brings many exciting new things, it is also likely to introduce new bugs and some regressions.

Shall I close this issue? I'm convinced this is not an issue with spotify-player.

The reason I was mentioning librespot is the ChangeLog provided with the release of v0.4.2, quoting

this point release is mainly to blacklist the ap-gew4 and ap-gue1 access points that caused librespot to fail to playback anything.

So, I thought this could be related. Anyway, librespot is preparing some major changes, I'd recommend to check it out, again quoting,

Development will now shift to the new HTTP-based API, targeted for a future v0.5.0 release. The new-api branch will therefore be promoted to dev. This is a major departure from the old API and although it brings many exciting new things, it is also likely to introduce new bugs and some regressions.

Ah, I see. Thanks for the information.

Shall I close this issue? I'm convinced this is not an issue with spotify-player.

Let's close if for now. Will open again if I find any issues related to spotify-player

@aome510 Found the issue.  From ncspotrelease notes

Update to librespot 0.4.2
Should fix ChannelError issues (ncspot skipping through all tracks) which were previously fixed with a workaround in /etc/hosts (https://github.com/hrkfdn/ncspot/issues/864

See the referenced issue and https://github.com/librespot-org/librespot/issues/972 on librespot.
Locally adding <IP_ADDRESS>  ap-gew4.spotify.com to /etc/hosts fixes the issue.
Anyway, updating to librespot-0.4.2 is the right solution.
Regards,
pin

@aome510 Found the issue.  From ncspotrelease notes

Update to librespot 0.4.2
Should fix ChannelError issues (ncspot skipping through all tracks) which were previously fixed with a workaround in /etc/hosts (hrkfdn/ncspot#864

See the referenced issue and librespot-org/librespot#972 on librespot.
Locally adding <IP_ADDRESS> ap-gew4.spotify.com to /etc/hosts fixes the issue. Anyway, updating to librespot-0.4.2 is the right solution.
Regards, pin

@0323pin Thanks for the context. Great to know that it's now fixed. I'll release a new version soon with the deps upgrade

Thanks!
I've posted a brief note for NetBSD users, https://mail-index.netbsd.org/pkgsrc-users/2022/08/10/msg036056.html
I want to avoid complains, as I'm the maintainer of all the Spotify clients 😄

@0323pin release new version 0.9.5 with the librespot upgrade. I also specify the latest build of souvlaki in Cargo.toml, so you can build the app with media-control feature. Please lmk if something doesn't work.

@aome510 Thanks! Unfortunately, pulling souvlaki from git is not such a good idea. Within pkgsrc distfiles should not be pulled from the internet at compile time and builds should be done in offline mode, so I hit the following error,
===> Building for spotify-player-0.9.5
error: failed to get `souvlaki` as a dependency of package `spotify_player v0.9.5 (/usr/pkgsrc/wip/spotify-player/work/spotify-player-0.9.5/spotify_player)`

Caused by:
  failed to load source for dependency `souvlaki`

Caused by:
  Unable to update https://github.com/Sinono3/souvlaki#b7020dfa

Caused by:
  can't checkout from 'https://github.com/Sinono3/souvlaki': you are in the offline mode (--offline)
*** Error code 101

Stop.

I can patch this but, have no time for it today.

@aome510 ok, just a few hours after and not tomorrow 😄
I couldn't resist.
All good, no hacks in /etc/hosts needed anymore and I can listen to music again.
Thank you!
I'll merge this on the main repo before  sleeping tonight.

I know this maybe unrelated but, I didn't want to create a new issue.
Yesterday, I streamed music without an itch for over an hour.
Today, I mostly get No playback found. Please make sure there is a running Spotify client and try to connect to it using the 'SwitchDevice' command.
Sometimes it works but mostly it doesn't and I get that error.
spotify-player-22-08-11-06:57.log

Today, I mostly get No playback found. Please make sure there is a running Spotify client and try to connect to it using the 'SwitchDevice' command.

Did you get that during startup or after using the app for a while?
I just look into the log,

2022-08-11T04:57:48.168309Z  INFO spotify_player::streaming: Initializing a new integrated player with device_id=c5e6a5a2-69ac-40aa-9d52-d39810a5bab5
...
2022-08-11T04:57:48.328265Z  INFO spotify_player: No playing device found on startup, trying to connect to an available device
2022-08-11T04:57:48.389242Z  INFO spotify_player::client: Available devices: []
2022-08-11T04:57:48.389275Z  INFO spotify_player::client: No available device found, use the integrated device (id=c5e6a5a2-69ac-40aa-9d52-d39810a5bab5)
2022-08-11T04:57:48.455336Z ERROR Client_request{request=Player(TransferPlayback("c5e6a5a2-69ac-40aa-9d52-d39810a5bab5", false))}: spotify_player::client::handlers: Failed to handle client request: http error: status code 404 Not Found

it seems it failed to find a device to connect to on startup, which can be because of

initializing the integrated device failed
initializing the integrated device succeeded but the device wasn't shown up in the device list when the get_device API is called

To confirm 1, did you get any backtrace errors?
To confirm 2, does it show any devices when you press D (SwitchDevice command)?

@aome510 Thanks! This happens at startup, as you already figured out.
I only use the integrated device, I've never used the app to control another running device. Yes, I do get backtraces but, they are empty.
Pressing D shows an empty list, as I only use the integrated device.
Could it be librespot failing to start?

Could it be librespot failing to start?

I guess it could be the case. But, it's pretty weird there is no error backtrace.

Pressing D shows an empty list, as I only use the integrated device.

Even if you only use the integrated device, press D should also show that device unless you don't specify your own client_id in the app.toml. In that case, get_device always return empty.
Does this issue happen every time you open the app? Also, does press R (RestartIntegratedClient command) work?

I've logged in on my working Windows machine on chrome, then launched the app on NetBSD and I can control the device.

Does this issue happen every time you open the app?

No, sometimes it works as expected.
Logged off in Windows and re-launched the player. Got the same error but, got the device in the list when pressing D.
Killed and restarted the app 6 times and twice it worked as expected.
This is though to debug.

When failing it always generates a backtrace,

But these are all empty.

When failing it always generates a backtrace,

The app generates a backtrace file and a log file whenever it starts. Empty backtrace probably means no error.
Based on my experience, as I experienced a similar issue before [1]. This was more because,

initializing the integrated device succeeded but the device wasn't shown up in the device list when the get_device API is called

The app starts by registering an integrated device then connecting to an available device. The connect process can fail if the newly registered device hasn't been notified to the Spotify server.
This can be fixed by either calling SwitchDevice (D) command to manually connect to one or RestartIntegratedClient (R) to restart the integrated device then connect to it.
[1] This issue used to happen more frequently for me before I made some changes to improve the register-connect progress. I'll take another look to see if I can fix this.

Thank you for the explanation.
It's running now, playing music while I work on some documents.
I'll try D and R when this happens again or, I'll just restarts until it works.

[1] This issue used to happen more frequently for me before I made some changes to improve the register-connect progress. I'll take another look to see if I can fix this.

Awesome but, no rush, it does work.

wrong place for this, I know ... just to let you know souvlaki has provided a new release, https://github.com/Sinono3/souvlaki/issues/25

Thanks for letting me know. Will update the app shortly

@aome510 Take your time. This week I'm rather busy with real life work. I'm still doing package updates but, don't have that much time to play with package build options, even though it's not that difficult :)
Better focus on it when doing, so I don't miss anything.


Insufficient permissions for maniaexchange

As I said in the tittle, I get the error "Insufficient permissions for "maniaexchange"", I have no idea why or how this is happening, but it is the only error that says something about this plugin, I have tried reinstalling it and installing optional plugins (better chat, itemexchange), and it did not help, it's my first time using openplanet and maniaexchange.

You need to run Standard edition of the game since it allows to play custom maps

You need to run at least Standard edition of the game since it allows to play custom maps

ohhhh, okay thanks!


remove subdirectory for gantt-chart in 1.3.15

Description
Gantt chart build is failing due to https://github.com/opensearch-project/opensearch-build/pull/4375. This was called out in the original PR

In all future release versions (>=3.0.0, >=2.12.0, >=1.3.15), the manifest file should not include working_directory: gantt-chart

as well as confirmed in this issue https://github.com/opensearch-project/dashboards-visualizations/issues/327#issuecomment-1928037969, but somehow still missed in the new 1.3.15 manifest
Issues Resolved
https://github.com/opensearch-project/dashboards-visualizations/issues/349
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

part of [RELEASE] Release version 1.3.15


Clean up doc/Makefile a bit

This series cleans up doc/Makefile a bit by removing some unused/probably unused stuff and some redundant targets.
Please check if the latex_paper_size stuff is used. I just removed it. It was removed in Sphinx 1.6.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8288   +/-   ##
=======================================
  Coverage   64.53%   64.53%
=======================================
  Files         420      420
  Lines       40122    40122
  Branches     6763     6763
=======================================
  Hits        25892    25892
  Misses      11110    11110
  Partials     3120     3120

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 7cfd5a4...22064f9. Read the comment docs.

@dbkinder
I'm all for removing stuff that's not used as well. Didn't know only the html stuff was used.


Increase hitSlop and pressRetentionOffset for navbar buttons

The nav bar buttons are more difficult to touch compared to iOS's, and the "press retention offset" is also less generous (while continuing to touch the screen, move your finger away and back). We should identify what these values roughly are on iOS and set them here too.

Will include that in my header refactoring unless someone else picks this up today or tomorrow.

Thanks for filing this @ide, this is definitely a recent regression that I've noticed as well. @grabbou, thanks for taking care of this in your header fixup!

Pinging OP @ide since this issue is quite old and related to an old version of the lib. If no answer I'll close in 7 days.

Last time I used React Navigation a couple months ago this wasn't fixed and I made the changes to the header buttons myself. It's the kind of thing that isn't visual but improves the experience.

Got it.
Then we can keep this open, thanks for letting me know. :)

Has this been resolved?

I can see the buttons in navbar areas are pretty well occupying the empty spaces.

I haven't had time to check on this yet, will soon

Let's see :) Added on my list, might attempt today or tomorrow!

1+ for a way to change hitslop for BottomTabNavigator navbar buttons.


IE9 - some grid are not loading on the documentation site

http://griddlegriddle.github.io/Griddle/customization.html

Thank you for reporting this -- I realize some time has gone on since this issue was filed but we're going to close for now. We recently released Griddle 1.0 and are still trying to determine which version of IE will be the lowest one we support. Thanks! 👍


Counter Module

Counter module counts how many visitors on your website. This module provides real time counting. All data saved to database.
https://www.drupal.org/project/counter

I don't think nowadays something like this is too useful. You want to gather more info about users than simply counting page loads, and the various analytics services can do that. If you want to go full server-side (as opposed to e.g. Google Analytics requiring JS and the consent of users) then something like Azure Application Insights provides ample data.

You're right, but as you said some needs some scripts or you have access to Cloud-based tools


gitlab: support "environment_scope" tag for GetAllSecrets

Support "environment_scope" as tag for GetAllSecrets.
Signed-off-by: Dominik Zeiger<EMAIL_ADDRESS>

Thanks for the swift reviews and approvals.


Add documentation for init.d starting nginx.

Write developer documentation for in developer docs for init.d and Linux in https://github.com/zulip/zulip/blob/master/docs/ssl-certificates.md
The main purpose of this is just to make clean that Zulip uses the system nginx, not something run via supervisord, so when one changes a certificate, one needs to restart the system nginx.
For reference: Zulip chat

@feorlen would you be for helping with this?  Seems like probably a pretty quick one.

@zulipbot claim

@zulipbot claim


apply for voucher page is misleading

i click on apply for voucher, i get redirected to account settings to apply a voucher not request a new voucher

I suggest to open the modal once we redirect the user


Quantization refactor

Changes in quantization:

Created flag to remove Q/DQ from output activations for conv layers. Set to True by default.
Changed default to True for flag to remove Q/DQ from output activations for linear layers.
Created flag to change number of bits for weight activation (similar to the already existing flag for activation bits).
Created flag to avoid quantization of BatchNorm layers when fusing is not used. Set to True by default.
Created flag to enable quantization compatible with tensorrt. Set to False by default.

Sets activation quantization to symmetric and UINT8.
Sets default fusing function to 'no_fuse'.

Created a wrapper module (BNWrapper) that allows one to freeze BatchNorm statistics when BN layers are not fused. PyTorch default only works when BN layers are fused.
Crated a wrapper module (_AddReLU) for the FloatFunctional class that performs the Add + ReLU operations in ResNet. This wrapper enables the module to be wrapped by a QATWrapper when quantized, and is set to quantize only the first input.

looks like this also breaks some tests as well since we change the default behavior for quantizing major ops. will take a look

one other thing to note is that I am very wary / leaning against defaulting to using the new quantization scheme for Convs as this will break existing workflows. I do think that we should eventually default, let's sync on a strategy for this

merged in export flow (#644)

merged in fixes for testing and additional tests from #685


Cannot enter hostnames with punctutations during install

During install (choosing option 4) to change hostname I get
Invalid Hostname. Please try again
For instance I am trying tredly01.prd.nv.us.domain.internalzone

hi dinapappor
Thanks for the bug report - I've implemented better hostname checking as per RFC 1123. Fixes are in the PR, just awaiting a merge (wont be far off)


onPageChange is firing twice

I am handling the current page with backend pagination, but the onPageChange is firing twice; the state is changing two time and it's breaking everything.
onPageChange: (page, sizePerPage) => {
console.log(page, sizePerPage);
},

Note that onSizePerPageChange is firing once and everything's working just fine.
react-bootstrap-table2-paginator version: 2.1.0
react-bootstrap-table-next: 3.3.5

@AllenFang I appreciate your time If you can help me with this.

you have to add the options
remote
onTableChange={() => {}}
to the table properties.


fix: do not add 'helm.sh' annotations on deployment

What this PR does / why we need it:
Add your description
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @hicaistar

Release note:

NONE

/hold
This example PR is not going to merge.

保持原样？

/hold cancel
PR is ready

/lgtm


CLI support to open a new Zed instance

VS Code uses code -n to open a new instance

And, obviously --new-window for longer version

@JosephTLyons - not sure if it's a bug or if it would be covered by this enhancement request but currently in 0.111.5 from the built in terminal you can't use the zed CLI to open a new window for a subdirectory of the current project.
For example if I have a project open for ~/dev/foo and from the terminal within that project window run zed ~/dev/foo/bar nothing happens, although you might expect a new project window to open.
Would the -n flag cover this case as well, or is this a bug?


fix(SigTable): RHINENG-2325 - Add padding to footer via TableToolbar

The padding via the TableToolbar is required to avoid colliding with the "lightbulb"
Before:

After:

We're currently processing your upload.  This comment will be updated when the results are available.

🙈 I missed those! Thanks for catching that! They have a TableToolbar now to. The InventoryTable will also have more padding when the Inventory is updated.


Voxel correspondance between Non-Linearly aligned Volumes

Greetings!
I am working on my PhD using 3D Brain MRI Data. I'm using dipy to perform linear and non-linear registration between my volumes and a template brain (colin_27).
For each of my volumes I first perform affine registration.

As a result, I have a group of linearly aligned volumes, and another group of non-linearly aligned volumes, all referred to the same 3D brain MRI Template.
To perform a voxel-wise comparison between my volumes, I would like to, for each voxel on the template, obtain the corresponding voxel of each of my linearly-aligned volumes. I believe this correspondence can be somehow obtained by using the deformation map obtained for each of the transformations?
Thus, my objective is for each voxel (x, y, z), obtain the corresponding voxel (x + dx, y + dy, z + dz ) for each of the linearly aligned volumes.
Thanks in advance for any solutions.

Hi @jrdimale
What you need is essentially the displacement field. The majority of registration algorithms utilize a trilinear sampling scheme, meaning the registered image is obtained by sampling the moving image based on the deformed image grid. After registration, the pixel value M(x, y, z) transitions to M(x+dx, y+dy, z+dz), aligning with the pixel value of the fixed image at (x, y, z) i.e., F(x, y, z). Here, the mapping (dx, dy, dz) is the displacement field.
-Junyu


Working as expected with Router.push(url, url).

Steps —
Page /test
Router.push("/?test")

Page /
if (queryAsString(this.props.url.query) === "test") {
  Router.replace("/", "/", {shallow: true})
}

Outcome —
<style jsx global>s persist across client-side routing.

I think this has been fixed since then.


[Backport] Get proper Kinesis index task AWS credentials (#9082)

Backport of https://github.com/apache/druid/pull/9058 to 0.17.0-iap.

Clean backport


The order list with index error

Do you want to request a feature or report a bug?
I think is a bug
What is the current behavior?
the order list with error-index when inserting some text into 2 order list.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/gmertk/e61z7nfa/.
the demo link: http://www.roserobertson.me/markdown-draft-js/
To Test:

Enter some text between the two OrderLists

there are some videos and screenshots for regression.  The following:

What is the expected behavior?
With correct index
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
Chrome Version 79.0.3945.88 (Official B
uild) (32-bit)

Thanks very much for this report! I'll take a look when I have a chance 😊

Also running into this on a project called Flow, it appears to have been solved here, though no coverage has been added for it.  I'll look at adding that.

Merged into main, still need to publish a new release with the fix


Ensure AngleTool and ArrowAnnotateTool fire MEASUREMENT_COMPLETED event

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
ArrowAnnotate and Angle to fire MEASUREMENT_COMPLETED event

What is the current behavior? (You can also link to an open issue here)
ArrowAnnotate and Angle not firing MEASUREMENT_COMPLETED event

What is the new behavior (if this is a feature change)?
ArrowAnnotate and Angle do fire MEASUREMENT_COMPLETED event

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no breaking change

Other information:

@ladeirarodolfo, you can @mention me or assign me as the reviewer to expedite.

:tada: This PR is included in version 4.14.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: