[tests-only][full-ci] Bump core commit id to the latest upto 2022-05-25

Signed-off-by: Kiran Parajuli<EMAIL_ADDRESS>
Description
Update core commit id to the latest up to date 2022-05-25
Related Issue

Part of https://github.com/owncloud/QA/issues/742

Motivation and Context

How Has This Been Tested?

:robot:

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

failed at cs3ApiTest-ocis https://drone.owncloud.com/owncloud/ocis/12087/35/5
Scenario: Change treesize of personal home and for only one subtree                                                                                # /var/lib/cs3api-validator/etag-propagation.feature:60
    Given user "admin" has created a folder "a-folder/a-sub-folder-B" in the home directory with the alias "a-sub-folder-B"                                # steps.go:29 -> *ResourcesFeatureContext
    When user "admin" has uploaded a file "a-folder/a-sub-folder/testfile.txt" with content "text" in the home directory with the alias "testfile.txt"     # steps.go:108 -> *ResourcesFeatureContext
    error statting new file
    Then for user "admin" the treesize of the resource with the alias "Admin Home" should be 4                                                             # steps.go:419 -> *ResourcesFeatureContext
    When user "admin" has uploaded a file "a-folder/a-sub-folder-B/testfile2.txt" with content "text" in the home directory with the alias "testfile2.txt" # steps.go:108 -> *ResourcesFeatureContext
    Then for user "admin" the treesize of the resource with the alias "Admin Home" should be 8                                                             # steps.go:419 -> *ResourcesFeatureContext
    And for user "admin" the treesize of the resource with the alias "a-folder" should be 8                                                                # steps.go:419 -> *ResourcesFeatureContext
    And for user "admin" the treesize of the resource with the alias "a-sub-folder" should be 4                                                            # steps.go:419 -> *ResourcesFeatureContext
    And for user "admin" the treesize of the resource with the alias "a-sub-folder-B" should be 4

same failure again.

https://drone.owncloud.com/owncloud/ocis/12104/55/9
strange, failed while creating users:
test log:
 And these users have been created with default attributes and without skeleton files in the server:
      │ username │
      │ Alice    │
      │ Brian    │
      │ Carol    │
    ✖ failed
      Error: Error: HTTP Request Failed: Failed while creating user, Status: 401 undefined
          at Object.checkStatus (/usr/src/app/src/helpers/httpHelper.js:44:11)
          at /usr/src/app/src/helpers/graphHelper.js:27:33
          at processTicksAndRejections (node:internal/process/task_queues:96:5)
          at async Promise.all (index 0)
          at async /usr/src/app/src/app.js:61:7
          at /srv/app/web/tests/acceptance/stepDefinitions/middlewareContext.js:38:15
          at processTicksAndRejections (internal/process/task_queues.js:95:5)ssssssssssssssssssssssssssssssssssssssssssssssssss

ocis log
{"level":"error","service":"proxy","error":"{\"id\":\"go.micro.client\",\"code\":500,\"detail\":\"service com.owncloud.api.settings: not found\",\"status\":\"Internal Server Error\"}","time":"2022-05-26T04:26:49Z","message":"Could not load roles"}

middleware logs
[error] - 2022-05-26T04:27:33.265Z : Error: HTTP Request Failed: Failed while creating user, Status: 401 undefined
    at Object.checkStatus (/usr/src/app/src/helpers/httpHelper.js:44:11)
    at /usr/src/app/src/helpers/graphHelper.js:27:33
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Promise.all (index 1)
    at async /usr/src/app/src/app.js:61:7

@phil-davis only two pipes fail in which every scenario fails on the user creating step as reported in the above comment. do you have any idea about this?

@phil-davis only two pipes fail in which every scenario fails on the user creating step as reported in the above comment. do you have any idea about this?

no idea - I guess it is some intermittent problem?
Rebase, force push and let it run again - not much else we can do right now.


Pull the dummypod image from Quay

This reverts part of
https://github.com/submariner-io/shipyard/pull/940 to allow CI tests to complete. Currently, CI ends up relying on nettest from the local registry, but downstream projects shouldn't have to know this; this reverts to the previous behaviour of always pulling the image from Quay.
We'll fix this properly later on.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

See if #997 fixes it, if not merge this one

Still seeing the failure
    Warning  Failed     4m11s                  kubelet            Failed to pull image "quay.io/submariner/nettest:devel": rpc error: code = Unknown desc = failed to pull and unpack image "quay.io/submariner/nettest:devel": failed to resolve reference "quay.io/submariner/nettest:devel": failed to do request: Head "https://quay.io/v2/submariner/nettest/manifests/devel": dial tcp <IP_ADDRESS>:443: connect: connection refused


Cluster design

Hi,
Is there any place I can read about the inside VerneMQ cluster design? How the messages are distributed among nodes? How does it scale?
I have searched the main website but could not find any.
Sorry in case the question is not suitable for posting here.
Thank you.

Hi @anphung, good question! :)
The basics are this:

Messages distribution goes over dedicated TCP connections between cluster nodes
Messages are only distributed to cluster nodes that actually need the message because they have local subscribers.
Subscriber information is held in a replicated subscriber store based on LevelDB.
The replication of the subscriber store is done over plumtree, an epidemic broadcast gossip protocol
Every cluster node builds a routing tree in RAM from the subscriber store, to know where to route messages to. This looks different on every node in contrast to the replicated LevelDB store.

Possible scalability bottlenecks are:

Constant high rates of incoming new subscribers
Constant high rates of triggered queue migration between nodes (triggered by the same clients logging in to different cluster nodes)

Only tests can show, if a specific cluster can run the numbers you have in mind.
Don't hesitate to ask for more information anytime!
Thanks.

Thank you for your detailed answer.

hi @ioolkos,
does replicated subscriber store mean that qos1 and qos2 messages won't be lost on cluster node leave?


feat: use Hail within the Genotypes class

Instead of using cyvcf2 inside of the Genotypes class, we could use Hail. Why? Well...

Hail supports PLINK2 and BGEN files automatically. So we wouldn't need to change our code to work with those files later on.
I did a quick back-of-the-envelope calculation and found that PLINK2 files can be read much more quickly: in 1% of the time that it takes to read VCF files composed of the same number of samples and variants. So we're probably gonna want to use PLINK2 later when we scale up to large datasets. The sooner we use Hail, the less code we'll have to rewrite.
Hail supports easy parallelization of reads and writes, so we won't have to worry about running out of memory on a node.

note: this issue is a WIP

seems like hail does indeed require spark, and we don't want to have to depend on spark, so I'm gonna close this for now


Fail fast if unable to connect to apiserver

We should do an availability check to the apiserver as soon as we create the kubernetesClient
https://github.com/apache-spark-on-k8s/spark/blob/38814043070bfdd522c2c9350b9411b943888cee/resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/kubernetes/submit/Client.scala#L90
This would allow us to give a better error message when unable to connect to the apiserver.
Otherwise it's not until later in the code that a failure appears, and it's usually less clear what's causing the failure.

@ash211 is this still applicable?

Could be, not sure.  I haven't had to debug an issue with connectivity to apiserver in a while

I believe the only time I've encountered something like this was with an unsuccessful installation of Weave. Failing fast could probably help people debug faulty Kubernetes or pod network configurations.

This seems like a good idea on principle.  Especially for new users, but failing fast is always preferable to deferring it.


Trade complete time unix bug

This fix bug related to setting trade completion status.
It closes #209
@tiero @altafan

With these changes, a trade could be retrieved by its txID if and only if the trader calls TradeComplete, because trade.Complete is the place where the TxID is set. Am I right?

We need to set the trade.TxID in trade.Accept
As the trade domain is designed, its fields are updated only by its methods, while info can be retrieved by both methods or public fields. I think this is a clear design, so I don't like so much updating the status in the closure as it's currently done within the bc listener.
I've got a proposal: what if we add a new status SETTLED that can come only after COMPLETE or ACCEPT-but-not-EXPIRED. This status would be set only by the bc listener once it detects the transaction included in the blockchain. We could create an ad hoc trade.Settle() method for this in the domain, that would be simply called by the listener.

I agree on having an additional SETTLED status.

We need to set the trade.TxID in trade.Accept

TxID is added in trade.Complete  and trade.Accept in here

I've got a proposal: what if we add a new status SETTLED

I think that adding additional status would not give us more info about the entity but only add complexity, issue here is that we have two scenarios:

Client calls TradeComplete so that daemon can broadcast tx
Client itself broadcast tx
In both ways, we need to wait for tx to be confirmed on blockchain, only then (in the listener)  we can update the completion time, we have option to update trade.Complete method to support both flows, I generally don't prefer overcomplicated methods.
Maybe it would be easies to just update trade.AddBlocktime, rename it to trade.Confirm and add complete status in it.

I think for now, let's just fix the bug we will think later about adding extra status.
let's the BC listener to setup the status COMPLETED and add txid etc..
If we pass from TradeComplete, we should only update the PSET, without touching the status and let the listener to put it that.
To differentiate this two paths we can have (only internally) two methods one Trade.Complete (that used by TradeComplete rpc to set the pset) and one Trade.Settle used by the listener


DISCUSS add resource groups to mini accumulo

Posting this draft PR solely for the purpose of discussion and feedback, it does not actually implement anything.  The elasticity branch is heavily using the concept of servers organized into resource groups.   MiniAccumulo's public api does not support resource groups and there are comments about this in the elasticity branch here. The changes in the PR were an attempt to solve the following problems.

Mini Accumulo public API does not support resource groups.
Accumulo uses Mini Accumulo to test itself and this internal code for testing does not consistently support resource groups or making starting and stopping servers in resource groups easy.

The changes in the PR stub out external and internal API changes that attempt to address the problems above.  However there is no implementation of these stubbed out APIs.  The following is guide to the changes so far.

The change in the external API were made in o.a.a.minicluster.*
An example of code that uses the new public API is in MiniAccumuloResourceGroupsTest
The changes to internal API used for testing were made in ClusterControl
Some examples of using the new internal API can be seen in a few ITs

One thing I do not like about these change is that  there is currently a runtime exception if server processes are placed outside of the default resource group.  This seems a bit tricky, not sure how to improve it.
One thing I do like about these changes is the internal API may make it easier to start and stop processes in resource groups in a consistent way in ITs.  So the way that process would be start for a tserver RG is the same as a compactor RG.  I also like the mode of specifying a new expected state and processes being started and/or stopped based on how the current state and the new state align.
One problem area for this PR may be that tserver resource groups do not exists in main, but only in elasticity.  This is a similar problem to #4220.

Hoping changes that address the problems mentioned will eventually support testing of multiple manager in a consistent way.  This is assuming that elasticity eventually supports multiple active managers.

Another goal of these changes is to deprecate Mini Accumulo APIs in 3.X so those could be remove in 4.0.0.

@keith-turner  - I assume you have seen ClusterServerConfiguration in the elasticity branch?  I didn't add it to the public API when I created it, but you could add an interface for it in elasticity, then backport it to main.

@keith-turner - I assume you have seen ClusterServerConfiguration in the elasticity branch? I didn't add it to the public API when I created it, but you could add an interface for it in elasticity, then backport it to main.

Yeah I am familiar with that and this conceptually builds on it, but moves from a mutable config model to a model of an immutable config.  I had not looked at that specific class recently, looking at it I changed some of the type and method names in this PR and that solved a problem I was having with being able to specify config that did not make sense.  Overall, still not sure about some of the names, initially my main focus was exploring this concept of immutable config so have not thought too much about the names and other aspects.
Hoping that the model of having an immutable sever config w/ getters and setters in ClusterControl will allow us to simplify how tests start and stop servers. Also hoping that we can remove most of the methods in ClusterControl related to starting and stoping servers (I added TODOs in this PR about removing them).  Currently in elasticity the code will get a reference to the mutable cluster server config, change it, and then call methods on ClusterControl to start/stop servers.  Hoping this streamlines that a bit for our ITs.

In 3.1.0 we could only update MiniAccumulo public API, but not change any internal test code.  Later in elasticity we could optionally update ClusterControl to use the new immutable server config public API type for internal testing purposes.


status options + docs + general update

This pull request adds the feature for the user to configure the status & readyState of the request, in case that they have to do checks on the AJAX process.
There is also now a docs folder for the GitHub Page(s) if a user wants to check it out on download; plus it just helps to keep all the together.
All contributors have been included.
Files, such as README.md, have been shortened and adjusted as needed.

docs still has to be adjusted accordingly, which is currently a work in progress.

@ForgivenNin will try this out ASAP and let you know.

@flouthoc any suggestions/thoughts?

@ForgivenNin Everything else look fine , just wanted some input about why can't we have a detailed readme ?

Also you can squash the commits.

It is detailed; but it's just the same amount of information is presented in a far less amount of space.
How would I squash the commits? Doesn't that have to be done by the merge itself?

@flouthoc
It appears that there are also two additional methods to GET & POST, which are PUT and DELETE. I haven't done anything with this yet, though, because I'm relatively new to AJAX (since it requires a server).

@flouthoc is there anything else that I should work on?

@flouthoc anything this could be applied to?
https://github.com/ForgivenNin/retrieve.js

@flouthoc Do you have any opinion about the pull request or anything else to be added?


directive support in vs code

What editor would you recommend for us to use for Solid js?
I am currently using vs code, but I am getting syntax error for directive:

And this is my tsconfig.json, which is generated by solid js template

I have the same problem. I don't know if it's the typescript itself.

Did you add your directive to the module "solid-js" / namespace JSX / interface Directives like described in the docs? https://www.solidjs.com/docs/latest/api#use%3A___

@atk  Thanks for that link. That solved the issue!


phigaro-setup - non-interactive setup possible?

Hello!
Is there a built-in method to skip the "please select appropriate "x" location" prompts during setup?
...
Please select appropriate Prodigal location
[1] /local/home/rpetit/miniconda3/envs/bt-phigaro/bin/prodigal
[2] Add another path manually
Choose your option (Enter for /local/home/rpetit/miniconda3/envs/bt-phigaro/bin/prodigal):
Please select appropriate HMMER location
[1] /local/home/rpetit/miniconda3/envs/bt-phigaro/bin/hmmsearch
[2] Add another path manually
Choose your option (Enter for /local/home/rpetit/miniconda3/envs/bt-phigaro/bin/hmmsearch):
Found Prodigal in: /local/home/rpetit/miniconda3/envs/bt-phigaro/bin/prodigal
Found HMMER in: /local/home/rpetit/miniconda3/envs/bt-phigaro/bin/hmmsearch
...

Thank you!

Not for now. Is it neccessary?

Probably not necessary, but would be useful.
I'm building a Nextflow workflow for Phigaro using the conda build. It would be useful to just run phigaro-setup without interacting since Prodigal and HMMER will always be available.
As an alternative, if a config file is provided does it use the provided paths or also ask to verify? (I can check shortly)

As far as I remember it does not ask to verify

Ok, so here's what I'm doing to allow "non-interactive" setup of databases.
rm -rf .phigaro/* && printf '\n\nN\n' | phigaro-setup --no-updatedb

This would allow the database to at least be setup in a programmatic way (e.g. in a workflow).
Going to close this issue.

+1 for non-interactive setup


app_links / firebase_auth hard crash on custom URL scheme from cold boot

flutter_test_app.zip
Open the project.
Run in simulator, kill the app in simulator,
Run xcrun simctl openurl booted m1nte://test in the command line
----CRASH----
Comment out app_links in pubspec.yaml
Run in simulator, kill the app in simulator,
Run xcrun simctl openurl booted m1nte://test in the command line
----NO CRASH----
This is seriously affecting my live App so I would appreciate an escalation if possible.
Thanks a lot

I have replicated this issue with uni_links so closing this issue as it seems to be firebase related


Support Ember Data beta.17

fixes #116.

Hey you have any idea why it's failing? I have other repos failing as well...

Er, I think it's because bower installs Ember Data beta.18 and Ember 1.12.0, both of which introduces deprecations that affects the adapter. I'll take a closer look tomorrow.

Okay, got tests to pass again.
There's a lot of deprecation warnings though. Will look at those next.

Removed all deprecation warnings except for the one about Ember.reduceComputed. Any idea one what it's about?

Nope. Never saw it before.

Can we bump version, and npm publish please?

Published!


Exported QIF file: we cannot import it in GnuCash desktop

Problem
I'm trying to import the following QIF files generated in GnuCash Android inside GnuCash desktop but the desktop application exits without messages or warnings and the import is NOT successful.
can-not-import-desktop.zip
Additional information

I've installed GCA 2.0.5 on my Nexus 5 running Android 6.0.1;
I've tested the files in GnuCash Desktop 2.6.10 for Windows and for Mac with the same behavior.

I think there is an issue with importing qif's into desktop version 2.6.10 regardless of origin so this may be your problem. Version 2.6.9 and the soon to be released 2.6.11 are good for qif imports apparently.
Cheers Dave H.

On 12 Jan 2016, at 5:19 AM, alceurneto<EMAIL_ADDRESS>wrote:
Problem
I'm trying to import the following QIF files generated in GnuCash Android inside GnuCash desktop but the desktop application exits without messages or warnings and the import is NOT successful
can-not-import-desktopzip
Additional information
I've installed GCA 205 on my Nexus 5 running Android 601;
I've tested the files in GnuCash Desktop 2610 for Windows and for Mac with the same behavior
—
Reply to this email directly or view it on GitHub.

Thanks for the update @halvey

After updated GnuCash Desktop to version 2.6.11 attached files are imported successfully.


Enable status page certificate expiry badge for all HTTP(s) monitors

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature, Other
🔖 Feature description
For the monitor type “HTTP(s)”, a badge showing SSL certificate expiry information can be shown (added in #3357). Although the monitor types “HTTP(s) – Keyword” and “HTTP(s) – JSON Query” offer all the other features related to Certificate Expiry (e.g. notification, showing info on dashboard), the status page badge cannot be enabled for them at the moment.
✔ Solution
Certificate Expiry badge can be added to status page for monitors of type “HTTP(s) – Keyword” and “HTTP(s) – JSON Query”.
❓ Alternatives
No response
📝 Additional Context
No response

https://github.com/louislam/uptime-kuma/pull/3649 was merged and is avaliable since the 1.23.3 release.
I think this issue was mistakenly left open
=> closing as resolved


JuttleMoment: new constructor option: rawDurationString

This new option is used for creating a JuttleMoment duration from the native string representation of a momentjs moment.duration.
Will be used by outrigger for instantiating JuttleMoment durations from momentjs moment.duration.
@demmer

lgtm


Playlist creation

Clear and concise description of the problem
I would suggest a way to create a playlist where we can add and order the podcast episodes
Suggested solution
Be able to listen our podcast in desired order
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Thanks for the issue/enhancement. Should the playlist thing be persistent, i.e., do you want to listen the playlist multiple times like a Spotify music playlist? This would mean listening to the same podcast multiple times or should playlist mean that you start your work at the morning, start the playlist and have them being played sequentially so you don't have to switch between the browsers to start the next podcast?

Hello,
It can be a playlist called "Audition queue" where we can add the episodes that we would like to play and order them accordingly with the sequence we want.
Also, when the episode finishes it automatically is cleared from there.
No need for a persistent playlist because repeat episodes are not so used in the case of podcasts.

You can test the playlist feature with the docker tag samuel19982/podfetch:playlists

What I'd like to know:
Does this solve your issue?
Did you find anything missing/wrong?


python310Packages.python-arango: 7.5.3 -> 7.5.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.python-arango is: "Python Driver for ArangoDB"
meta.homepage for python310Packages.python-arango is: "https://github.com/ArangoDB-Community/python-arango"
meta.changelog for python310Packages.python-arango is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 7.5.4 with grep in /nix/store/mqbqy30qgk2w7cqyi1rqs4c3mk941db8-python3.10-python-arango-7.5.4
found 7.5.4 in filename of file in /nix/store/mqbqy30qgk2w7cqyi1rqs4c3mk941db8-python3.10-python-arango-7.5.4

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.python-arango
python39Packages.python-arango

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mqbqy30qgk2w7cqyi1rqs4c3mk941db8-python3.10-python-arango-7.5.4 \
  --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 python310Packages.python-arango https://github.com/r-ryantm/nixpkgs/archive/0c4a91ee5ffed71e766b51be7e26f51a7b91dc4a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mqbqy30qgk2w7cqyi1rqs4c3mk941db8-python3.10-python-arango-7.5.4
ls -la /nix/store/mqbqy30qgk2w7cqyi1rqs4c3mk941db8-python3.10-python-arango-7.5.4/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 run on x86_64-linux 1

2 packages built:

python310Packages.python-arango
python39Packages.python-arango

Maintainer pings
cc @jsoo1 for testing.

Looks good to me. Tests pass (with doCheck = true).

@GrahamcOfBorg eval


Length and Detail of Text Condition

Would a more detailed a long text condition give the users a better clustering result or would it make the performance worse? I am wondering of a situation where users might want to cluster on conditions that is not explainable in a single word or a phrase.

Thank you for your question. I believe that the longer the text criteria, the more it reduces ambiguity and increases specificity, thereby improving performance. (In this perspective, the existing clustering algorithm is a case where the length of the text criteria is 0.) However, since we are using an off-the-shelf LLM, it is important that this text criteria be written in a way that the LLM can easily understand. Thank you for your interesting question.
Sincerely,
Sehyun

Makes sense. Great work!


Translation files for BM110ABD009_CD_DOC_Ship

I am out of the office until 02/07/2016.
Note: This is an automated response to your message  "[docs] Translation
files for BM110ABD009_CD_DOC_Ship (#63)" sent on 2/5/16 2:09:25 AM.
This is the only notification you will receive while this person is away.

I am out of the office until 02/07/2016.
Note: This is an automated response to your message  "Re: [docs]
Translation files for BM110ABD009_CD_DOC_Ship (#63)" sent on 2/5/16 2:09:30
AM.
This is the only notification you will receive while this person is away.


[Docs]- Fix Flickering in Links

Notes for Reviewers
This PR fixes #9196
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 94d333d83b9a03d2775a12e6b6f70843f617c363 at: https://65391b77e54a6c1c5a574561--meshery-docs-preview.netlify.app

Links that flickers in https://docs.meshery.io/ -

https://docs.meshery.io/installation/mesheryctl
https://docs.meshery.io/installation/compatibility-matrix
https://docs.meshery.io/project/contributing

// @vishalvivekm

@coder12git merge conflict


Multiple deploy

Issue solved: #67
Added option to deploy multiple applications by fileSet on deploy goal.
Files changed:

deploy.md: Added example to multiple deploy.
basic-it/pom.xml : Added deploy and undeploy executions to it.
deployAppMojo.java: Added fileSet parameter.
MvnMessages.properties

With the latest 3.0 release, the install-apps and deploy goals were combined. Also, the applications that are deployed/installed are either from the project, the dependencies or both. We no longer support specifying appArchive or appArtifact on the deploy goal. Closing this PR since it is no longer relevant to the current code base.


[FEATURE] Dockerize the web app

Description
Dockerizing this project will help developers contribute more easily. How? Let me explain. Before making any contributions, developers typically need to set up their development environment with all the project dependencies. By dockerizing this project, they can eliminate the overwhelming task of setting up the environment.
Screenshots
NA
Additional information
NA
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

won't they now need to install docker in their system as well?

@narayan954  yes they need to install docker but it is Very easy to install and only command to run the application. After they do the changes and run that command again.
Thank you for assigning the above task to me I will try to compete as sson as possible.


feat: add support for unknown scopes

@jcchavezs if your concerns are addressed, do you mind re-reviewing?


Circle CI build failed

The change to build .mo files seems to have broken the build:

https://circleci.com/gh/Volentix/venue-server/337?utm_campaign=build-failed&utm_content=failing-command&utm_medium=email#tests/containers/0/actions/110
Please take a look and resolve.
Also given that it can break the build, we should build the .mo files for every branch, not just on the master branch.

oh, I forgot that circleci can't work with volumes.

we should build the .mo files for every branch, not just on the master branch.

I don't see a way to build the files for each branch in the CircleCI. I mean that I'm about to add the files to the image that ci pushes to the repository and we do it only for the master. If you want to have the files into every branch, we have to keep them in the git instead and just re-compile them when you change translations.

We don't need to keep the files, just to build them -- that is, if it is possible for the build process to fail. If the build process can never fail, then you're right, they only need to be in master.
Basically, if it's possible for a part of the build to fail, it should be done in every branch.

Ok, I see your point. And yes it's possible that it will be failed. So I'll add it just after the tests.


Fix v3.0 and v3.1 swagger.json URLs were reversed

@rglidden : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks correct based on context; merging.


Can't make send transaction

I can't send my funds to external wallet.
On broadcasting transaction execution i getting error:
_The server returned an error when broadcasting the transaction.
Consider trying to connect to ad different server, or updating Electrum.
The transaction was rejected because it is too large (in bytes)_
Traceback:
Traceback (most recent call last):
File "electrum\gui\qt\invoice_list.py", line 175, in
File "electrum\gui\qt\main_window.py", line 2012, in show_onchain_invoice
File "electrum\gui\qt\main_window.py", line 970, in format_amount
File "electrum\simple_config.py", line 685, in format_amount
AttributeError: 'str' object has no attribute 'value'
Additional information:
Electrum version: 0.5.2
Python version: 3.9.13 (tags/v3.9.13:6de2ca5, May 17 2022, 16:24:45) [MSC v.1929 32 bit (Intel)]
Operating system: Windows-10-10.0.19044-SP0
Wallet type: standard
Locale: ru_RU

I can't send my funds to external wallet.
On broadcasting transaction execution i getting error: _The server returned an error when broadcasting the transaction. Consider trying to connect to ad different server, or updating Electrum.
The transaction was rejected because it is too large (in bytes)_
Traceback: Traceback (most recent call last): File "electrum\gui\qt\invoice_list.py", line 175, in File "electrum\gui\qt\main_window.py", line 2012, in show_onchain_invoice File "electrum\gui\qt\main_window.py", line 970, in format_amount File "electrum\simple_config.py", line 685, in format_amount AttributeError: 'str' object has no attribute 'value'
Additional information: Electrum version: 0.5.2 Python version: 3.9.13 (tags/v3.9.13:6de2ca5, May 17 2022, 16:24:45) [MSC v.1929 32 bit (Intel)] Operating system: Windows-10-10.0.19044-SP0 Wallet type: standard Locale: ru_RU

Hi there @AlexSuvorov2k
It seems you're encountering an error while trying to send funds to an external wallet in Electrum. Here's how to resolve it:

Try a Different Server: Go to Tools > Network and select a different server to connect to.
Reduce Transaction Size: The error suggests the transaction is too large. Try sending a smaller amount or use the "coin control" feature.
Update Electrum: Download the latest version from the official website and update your wallet.
Check for Attribute Error: If the issue persists after updating, report it on Electrum's GitHub.

For further assistance, you can also reach support via this link: Electrum Wallet dApps Support Ticket Requests. Click the live chat icon in the bottom corner for quick help.


update EMR to 5.14.0

Coverage remained the same at 71.073% when pulling ed82b713b01c0da3f0948a9da923b798de9440ac on akmorrow13:updateEMR into f0e779dd1492853e6c9966cc50eafd5268de48b9 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/665/
Test PASSed.


doc: update readthedocs

Being addressed since https://github.com/uccross/arrow/commit/ea2a2f72cf1f9a0e73e51fe6c011deded8086708

Looks like the C++ Datasets API is not exposed in the Documentation. It is exposed only through the Python bindings. So, this depends on #9

Duplicate od #48


How do I generate random numbers using Lua?

What is your question?
I wanted to try and recreate the storm mechanic in week 2 using lua but i don't know how to get random numbers in lua

math.random([MINIMUM], [MAXIMUM]);

thanks

actually use getRandomInt((min), (max), true) because math.random will no longer be random in the next update

thankss

try this
https://github.com/IhavenotComeToCompete/Pure-Random-Number-Generator-Love2D-LUA


Make it work with free account

Deployment fails because of this:
    Deploying services on project geogeo in wedeploy.com...
    ! Deployment failed in 2s.
    ! Upload failed db-geogeo.wedeploy.io.
    ! Upload failed ui-geogeo.wedeploy.io.
    ! Desired service spec cpu exceeded the max limit for provided plan standard. The max limit is 4.

Such example should work on free plan, as thats the point , right?

Closing since this should work with a free account.


Slow loading on Firefox 78.0.2

While the widget does eventually load, it seems to take almost exactly 30 seconds to load on Firefox when in Google or Chrome it loads instantly. Not sure if there's anything you can do to remedy that or if it's a Firefox issue, which I suspect it is.

Sounds like something might be timing out. Are you able to provide console logs while the page is trying to load?

Here's everything that the console spits out; note that the "Brand New Animal" and "12/12" lines appear exactly as the graphics load in, which again, is almost exactly 30 seconds after that red error block.

Hmm yeah I was able to reproduce this on FF as well. It seems to be hanging on the load of progressbar.min.js for some reason. I'm not sure why this is, so it may take some time for me to figure it out and push out a fix. I may be able to work around it by using a CDN to load that specific file over serving it locally but its something I'll have to explore.
Honestly this is going to be fairly low priority as I'm more interested in rewriting this whole project as its been long overdue.

I wonder if it would be as simple as moving the script declarations around so that progressbar.min.js comes before updater_functions.js in the HTML so that it's loaded before any updates occur? Or at least on the asenshi test site the declarations seem to follow the loading order show in the console here.

Hmmm that's a good idea and certainly worth trying.

@Adamantcheese I've made the changes and updated the version that's running on the website. It seems to load correctly on FF for me now - please take a look and let me know if the issue persists.

Working correctly now for me! Thanks for the update!


Race on file watcher

We appear to have a race where:

We FTP up the files.  After Function.json ,  but before the corresponding csx,
our File Watcher triggers
We scan the dir, and csx is missing. So we throw this error.

We throw here:
https://github.com/Azure/azure-webjobs-sdk-script/blob/d51608db9254bf1953ebf03ccfb2bfd2d6573a9c/src/WebJobs.Script/Host/ScriptHost.cs#L963
The customer works around this by uploading twice.

Customer hit this: https://icm.ad.msft.net/imp/v3/incidents/details/37331477/home


Converting C structs issue

If I have a struct such as this:
typedef struct unz_file_info_s
{
    int version;              /* version made by                 2 bytes */
    int version_needed;       /* version needed to extract       2 bytes */
    int flag;                 /* general purpose bit flag        2 bytes */
    int compression_method;   /* compression method              2 bytes */
    int dosDate;              /* last mod file date in Dos fmt   4 bytes */
    int crc;                  /* crc-32                          4 bytes */
    int compressed_size;      /* compressed size                 4 bytes */
    int uncompressed_size;    /* uncompressed size               4 bytes */
    int size_filename;        /* filename length                 2 bytes */
    int size_file_extra;      /* extra field length              2 bytes */
    int size_file_comment;    /* file comment length             2 bytes */

    int disk_num_start;       /* disk number start               2 bytes */
    int internal_fa;          /* internal file attributes        2 bytes */
    int external_fa;          /* external file attributes        4 bytes */

    int tmu_date;
} unz_file_info;

when converted, it will create this and cut off the last two fields?? Do you limit the total number of fields or what is going on with this? How can I fix it?
type
  Punz_file_info_s = ^unz_file_info_s;

  unz_file_info_s = record
    version: Integer;
    version_needed: Integer;
    flag: Integer;
    compression_method: Integer;
    dosDate: Integer;
    crc: Integer;
    compressed_size: Integer;
    uncompressed_size: Integer;
    size_filename: Integer;
    size_file_extra: Integer;
    size_file_comment: Integer;
    disk_num_start: Integer;
    internal_fa: Integer;
  end;

  unz_file_info = unz_file_info_s;

I created a simple .h file with just the struct you mentioned above. When I translate this using version 1.4.1 of Chet, I get the following output (as expected):
unit test;
{ This unit is automatically generated by Chet:
  https://github.com/neslib/Chet }

{$MINENUMSIZE 4}

interface

const
  {$IF Defined(WIN32)}
  LIB_TEST = 'Test_win32.dll';
  _PU = '';
  {$ELSE}
    {$MESSAGE Error 'Unsupported platform'}
  {$ENDIF}

type
  // Forward declarations
  Punz_file_info_s = ^unz_file_info_s;

  unz_file_info_s = record
    version: Integer;
    version_needed: Integer;
    flag: Integer;
    compression_method: Integer;
    dosDate: Integer;
    crc: Integer;
    compressed_size: Integer;
    uncompressed_size: Integer;
    size_filename: Integer;
    size_file_extra: Integer;
    size_file_comment: Integer;
    disk_num_start: Integer;
    internal_fa: Integer;
    external_fa: Integer;
    tmu_date: Integer;
  end;

  unz_file_info = unz_file_info_s;

implementation

end.

It correctly translated all the fields in the struct.
Can you create a reproducible scenario, or send me the complete project?

Hmm, I just tried it again and now it seems to work. It totally was not working in my main project where I'm pulling in and converting several headers combined (because I am unable to fig out how to set a system path so it pulls in all the extra headers). Check FB Msgr, I had sent you a msg about this.
Basically, if all I want to do is convert say unzip.h which will reference zlib.h I dont want all of zlib.h, just unzip.h. If zlib.h is put in the system include path that I get the desired results. But I can't put a bunch of files there. I need to find a way to tell chet that a folder is a system include path as well. Is this possible?

Ok, I've identified the source of the problem, until I can issue a PR, if you are using post processing, just replace:
RemoveAllLines('external', 2); with RemoveAllLines('external ', 2); in function TScriptStringList.CreateDynamicImport(aType: Integer): Boolean;


Optimize webpack by using thread-loader

Summary
Updated webpack to use the thread-loader and also updated webpack to 3.10 which includes performance optimizations from versions between our current (3.6). On my local machine this speeds up the initial webpack compilation from 40s to ~22s.

Did anyone pull this down and also see performance improvement?

Webpack output: Before 41395ms && Your Changes 2487ms


Add examples for user management

Hi,
could you add some examples for adding/editing/deleting users in tenable.io?
Thanks!

Hi @Brueggus,
I'll add a new examples file for user management and check to see whether our documentation could be made more clear.
Thanks

This has been added in https://github.com/tenable/Tenable.io-SDK-for-Python/blob/master/examples/users.py


obs annotation added as bound method

Hi,
I have multiple anndata objects (adata1, adata2, adata3...) that I want to concatenate. Rather than specify keys to track these adata objects at the point of concatenation, I'd like to add a string categorical label as an annotated observation to each adata object individually.
i.e.
adata1.obs['sample'] = 'sample1'
adata2.obs['sample'] = 'sample2'
adata3.obs['sample'] = 'sample3'

Then concatenate in a later step.
However, these annotations keep getting added to my adata objects as a bound method rather than as a dtype.
How do i ensure my string annotations are added as a dtype? (ultimately to be categorical in later concatenated adatas)
Thanks!

To make sure I'm understanding correctly, is something like this not working for you?
a.obs["label"] = "a"
b.obs["label"] = "b"
combined = anndata.concat([a, b])
combined.obs["label"]

I'm a little confused about what you mean by this:

However, these annotations keep getting added to my adata objects as a bound method rather than as a dtype.

Could you elaborate on this? An example would be great.

Apologies, but this is a big error on my part in terms of calling the annotation from obs.
As per your example...
a.obs["label"] = "a"
b.obs["label"] = "b"
combined = anndata.concat([a, b])
combined.obs["label"]

Returns the list
But I had been calling
combined.obs.label

which returned
<bound method NDFrame.sample of      cell_ids   area  sample


Bug in risk comp_posterior_state_dist() in unilateral

There is an issue in the comp_diagnose_encoding function in [unilateral.py](https://github.com/rmnldwg/lymph/blob/main/lymph/models/unilateral.py.
in line 876 we have:
    diagnose_given_state = diagnose_encoding @ self.observation_matrix

This does not work as the two matrices do not have matching dimensions. After going through all the underlying code I assume that the dimensions of the self.observation_matrix are simply transposed. The following fix makes the function functional:
        diagnose_given_state = diagnose_encoding @ self.observation_matrix.T

Good catch! I've fixed it in both the uni- and the bilateral model and added tests. The fixes live in the dev branch for now. Will be merged with the next release.


Improve nanoc!

I made a few useful changes.

I tested locally. It should work. (check me)

It overall looks like to much to consider at once, like for sure you should separate adding RVM project files and un-hiding autocompile.
Also whats up with removal of new line at the end of few files, that would be problematic for anyone using shell tools to work with the files (like me using grep to find something).

now, can you separate this into separate PR's, some of the changes might be rejected and it influences the rest, also open the new PR's against release-3.7.x instead of master

There is no need... Everything we can examine here! :wink:

This PR has major issues, and provides no value.
Non-useful changes:

Rearranging .gitignore is not useful.
Adding _.ruby-xyz` files is not useful.
Adding comments to .travis.yml is not useful.
Changing the header of the README is not useful.
Adding installation instructions to the README is not useful, because nanoc already has installation instructions.
Changing the nanoc command description is not useful.
Rearranging the gemspec is not useful.

Breaking changes:

Explicitly setting Encoding.default_internal and Encoding.default_external.
Removing Nanoc3.
Un-deprecating the autocompile command.
Un-deprecating the watch command.

Interesting changes:

Adding osx to .travis.yml: interesting idea, but I’m not convinced this is useful. nanoc has no OS-dependent parts. If anything, it should add Windows. I’m more on the side of not adding OS X as a tested platform.

Updating the release branch name. Although, since it’s used as an example only, there’s not all that much value to changing it.

Adding # encoding: utf-8: perhaps useful, but then Rubocop config should be adjusted to take tasks and YARD handlers into account.

for .ruby-version - well adding some kind of ruby version specification helps those that use multiple rubies and would avoid installing nanoc dependencies in all the rubies, it's like "recommended ruby version", I'm cursing lack of this file every time I switch to nanoc, I have to either find ruby I used last time or install all gems again, with .ruby-* files ruby switchers would automatically select proper ruby to work with ... or .gitignore all of them so users can define their own file.

just add the file to the directory yourself. by forcing a version, you screw everyone else who doesn't want to use that version. it's actually kind of rude ;)

Agree with @bobthecow. But having .ruby-version in .gitignore would be a good idea, in my opinion.

Yeah, but not the end of the world even if it's not added. Did you know you can ignore things in your own copy of a repo, even if the project owner doesn't add them to .gitignore?
echo .ruby-version > .git/info/exclude

:)

I’ve recently switched to using the Ruby provided by Homebrew, but I could also have been using the default one provided by the OS.
There was a time where you almost needed to have multiple Rubies installed, but that time’s pretty much past.

I still use rbenv because I've got enough production Ruby apps locked to various Ruby versions that I don't want to mess with keeping in sync :)


reinstall issues with CUDA

I had my WIS and WAS working, but change the API key on HA. I could not get to WAS to re-generate the willow images, and WIS was stopping and starting all by itself - so I give up.
I decided to re-install everything on my WIS/WAS server.
Well, I finally got WIS up and running after following the instructions (is this about 4months ago and I did sleep in the mean time) - But it does not pickup the CUDA (1080ti) device.
Instead it defaults to CPU with this message:
willow-inference-server-wis-1    | [2023-10-20 16:38:51 +0000] [60] [INFO] Starting gunicorn 20.1.0
willow-inference-server-wis-1    | [2023-10-20 16:38:51 +0000] [60] [DEBUG] Arbiter booted
willow-inference-server-wis-1    | [2023-10-20 16:38:51 +0000] [60] [INFO] Listening at: http://<IP_ADDRESS>:19000 (60)
willow-inference-server-wis-1    | [2023-10-20 16:38:51 +0000] [60] [INFO] Using worker: uvicorn.workers.UvicornWorker
willow-inference-server-wis-1    | [2023-10-20 16:38:51 +0000] [62] [INFO] Booting worker with pid: 62
willow-inference-server-wis-1    | [2023-10-20 16:38:51 +0000] [60] [DEBUG] 1 workers
willow-inference-server-wis-1    | [2023-10-20 16:38:58 +0000] [62] [INFO] Willow Inference Server is starting... Please wait.
**willow-inference-server-wis-1    | [2023-10-20 16:38:58 +0000] [62] [INFO] CUDA: Not found - using CPU with 4 cores**
willow-inference-server-wis-1    | [2023-10-20 16:38:58 +0000] [62] [INFO] Started server process [62]
willow-inference-server-wis-1    | [2023-10-20 16:38:58 +0000] [62] [INFO] Waiting for application startup.
willow-inference-server-wis-1    | [2023-10-20 16:38:58 +0000] [62] [INFO] CTRANSLATE: Supported compute types for device cpu are {'int8_float32', 'int8', 'float32'}- using configured int8
willow-inference-server-wis-1    | [2023-10-20 16:38:58 +0000] [62] [INFO] Loading Whisper models...
willow-inference-server-wis-1    | [2023-10-20 16:39:28 +0000] [62] [INFO] Loading TTS models...
**willow-inference-server-wis-1    | [2023-10-20 16:39:33 +0000] [62] [INFO] Skipping warm_models for CPU**
willow-inference-server-wis-1    | [2023-10-20 16:39:33 +0000] [62] [INFO] Willow Inference Server is ready for requests!
willow-inference-server-wis-1    | [2023-10-20 16:39:33 +0000] [62] [INFO] Application startup complete.

Can anyone please tell me the complete instuctions to install everything needed for the CUDA/NVidia drivers?
I followed these instructions: [https://docs.nvidia.com/datacenter/tesla/tesla-installation-notes/index.html#ubuntu-lts](url)
But doing the post-install steps, all failed as there is no "/usr/local/cuda" directory.

Please help - I am totally willing to redo the server from scratch again. I have taken down on the step/instructions I did so far, but the NVidia/Cuda part is a broken mistery. Everything that I can find on the web is either old, or does not work.

Does anyone have a list of instructions and the order that it needs to be installed, for this to work?

Thanks

Here is the solution - This works:
[https://gist.github.com/denguir/b21aa66ae7fb1089655dd9de8351a202]
I will redo the server from scratch later - but for now it picked up the NVidia Drivers.


Return error when creating gitlabdownloader failed

related with #12779

Codecov Report

Merging #12790 into master will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #12790      +/-   ##
==========================================
+ Coverage   43.23%   43.24%   +0.01%
==========================================
  Files         650      650
  Lines       72048    72041       -7
==========================================
+ Hits        31150    31155       +5
+ Misses      35852    35842      -10
+ Partials     5046     5044       -2

Impacted Files
Coverage Δ

modules/migrations/gitlab.go
1.06% <0.00%> (+0.01%)
:arrow_up:

modules/indexer/stats/queue.go
64.70% <0.00%> (-11.77%)
:arrow_down:

modules/indexer/stats/db.go
43.47% <0.00%> (-8.70%)
:arrow_down:

modules/charset/charset.go
68.53% <0.00%> (-4.50%)
:arrow_down:

modules/git/utils.go
73.77% <0.00%> (-3.28%)
:arrow_down:

modules/queue/workerpool.go
58.77% <0.00%> (-1.23%)
:arrow_down:

models/gpg_key.go
54.81% <0.00%> (-0.59%)
:arrow_down:

modules/git/repo.go
49.23% <0.00%> (-0.51%)
:arrow_down:

models/error.go
35.66% <0.00%> (+0.85%)
:arrow_up:

models/notification.go
67.04% <0.00%> (+0.91%)
:arrow_up:

... and 5 more

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 b333aa7...ff8a843. Read the comment docs.

thanks


Refer Directly to Docs

This is a SOLID project.
Would it be possible to leverage a specific prompt that would refer to a document(s) directly?
Say I need to pull A from Page X and pull B from Page Y, being able to directly point to where the information can be found would be helpful.

Currently when you add documents to a workspace you get relevant document citations at the bottom of the message - so you know where that information comes from.
That being said, I really want to add in-app document citations so you can go exactly the to doc + text it used in context. Langchain does not help here so we would have to roll our own.
Not a big ask at all, but half of what you are asking for is already live 🎉

Currently when you add documents to a workspace you get relevant document citations at the bottom of the message - so you know where that information comes from.
That being said, I really want to add in-app document citations so you can go exactly the to doc + text it used in context. Langchain does not help here so we would have to roll our own.
Not a big ask at all, but half of what you are asking for is already live 🎉

I appreciate the response - however, I don't think I was clear in what I meant.
What I am talking about is the ability to provide a specific prompt and direct OpenAI to look at the contents prior to returning a response.
Example: "I need you to determine whether or not Document X that is inside of my Vector database can be used for..."
Essentially, it is directing OpenAI to target those documents specifically as it relates to the conversation at large.
Apologies for the confusion.

Do you have any idea as to whether you will incorporate a direct reference?

Do you have any idea as to whether you will incorporate a direct reference?

Going to mark this as closed since we will be implementing a slash command functionality to be able to refer to a document directly in a chat.


[Linux] Initial support of WiFi provisioning

Problem
CHIP Linux Example app does not support network provisioning.

Summary of Changes
Add initial support of wifi provisioning to Linux platform, this is intended to unblock those who are testing this. Follow ups is required to make this better.

Fixes #3383

There are two consecutive PRs for this
https://github.com/project-chip/connectedhomeip/pull/3595
https://github.com/project-chip/connectedhomeip/pull/3596

You lost the race (just barely)
Can you rebase on #3595

dup #3595

....LOL, what a coincidence. @erjiaqing You can indicate in the issue and say you are working on it to avoid effort collision in future.


Having issues rendering images with WSL. Just get blank GUI windows.

Version
Microsoft Windows 11 [Version 10.0.22621.1105]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
Visual Studio Code 1.74.3 with Remote Development Extension 0.23.0 (WSL 0.72.0)
Repro Steps
I had to do a clean install of Ubuntu on WSL2 and have since had issues getting graphics to render with libraries I use, specifically vedo (https://github.com/marcomusy/vedo). I use Miniconda to install everything. It has worked fine in the past on the same computer (HP Spectre x360 Convertible 14t-ea100 with Intel Iris Xe graphics). My new Ubuntu didn't have libGL.so.1 installed so I did sudo apt-get install libgl1-mesa-glx libgl1-mesa-dri. I have the same issue with PyVista (https://github.com/pyvista/pyvista) not rendering, but Matplotlib works fine, so I'm guessing it has something to do with VTK. I have tried completely uninstalling WSL and starting from scratch, as well as tried with Ubuntu 20.04.5 LTS. While I didn't have libGL.so.1 issues with 20.04.5, I still had the rendering issues. I also tried turning Windows Hypervisor Platform on.
Expected Behavior
From https://vedo.embl.es/autodocs/content/vedo/index.html

From https://docs.pyvista.org/api/examples/_autosummary/pyvista.examples.downloads.download_bolt_nut.html

Actual Behavior
I just get a blank GUI window like this for the vedo example

And for the PyVista example I get this

Followed by this directly after I exit out of the previous window.

Diagnostic Logs
No response

I have the same/similar problem with the FortiClient GUI:

These are the logs when running it:

I was getting just a blank black window from glxgears. Updating the Intel graphics driver resolved it for me.
https://www.intel.com/content/www/us/en/support/intel-driver-support-assistant.html
Beware that Windows update will reinstall the older driver but that can be resolved by hiding the update.
https://social.technet.microsoft.com/wiki/contents/articles/53184.show-or-hide-updates-utility-fixing-automatic-installation-of-a-problematic-update-in-windows-10-version-1903.aspx

So updated my Intel graphics driver and now the GUI window immediately closes after opening. When I run glxgears it does the same thing, window closes immediately, and the output says "Segmentation fault"

I also have this "Segmentation fault" problem, in many GUI programs (Google Chrome or NoSQL Booster, for example)if I run then after I already run something in WSL (or even if i run them first, it is totally inconsistent).
Also see this: https://github.com/microsoft/WSL/issues/8696
(It appears that Microsoft team tries to fix this problem)


Failed parsing input: parser: bad import "unsafe"

This package is used in: https://github.com/gin-gonic/gin which supposedly can play nice with Google app engine.
But when try to run my GO app through the local app engine emulator, I'm getting this error during build:
go-app-builder: Failed parsing input: parser: bad import "unsafe" in github.com/json-iterator/go/feature_adapter.go from GOPATH
Any idea why this could be happening?

maybe GAE does not allow using unsafe.Pointer

gin added build tag -tags=jsoniter


PR: Improve and update wording of funding announcement, and put screenshot above it

As somewhat of a follow-on to #6909 except just for master, this PR:

Positions the screenshot above the funding announcement, with the rationale that it is more important to hook users' interest about the application itself first, showing them what it can do, before hitting them with a plea for funds for an application they don't yet have any compelling reason to support, and potentially scaring them into thinking it is no longer modern or actively developed.
Improves the text of the announcement itself, including:

Fixes grammar errors, awkward/unidiomatic prose, and unclear portions
Updates the blurb to better reflect the current situation
Tries to give it a more optimistic and productive tone to avoid scaring potential users into thinking the project is in decline, while still conveying the need for community support, by focusing on the benefits users' help can bring to the project rather than the bad consequences if we don't get it.
Cleans up formatting and adds some useful embedded links.

@CAM-Gerlach I made a few more comments, but in general I think it's great!

Thanks @bcolsen ; great feedback!

Looks good to me

@ccordoba12 The "important announcement" is currently only on the master readme for whatever reason, not 3.x (or I would have suggested the change with #6909 and based against it accordingly). Therefore, are you effectively asking me to backport the whole announcement to 3.x? I'll have to just start from scratch with clean 3.x, paste the blurb in, commit that in one go, and then you'll have to deal with whatever resulting mess merging back to master.

The "important announcement" is currently only on the master readme for whatever reason

Me and @goanpeca decided to be like that. I just forgot it, sorry.

you'll have to deal with whatever resulting mess merging back to master.

I don't have time for that. Let's merge this one as it is then, Thanks!


Is print function really required?

Just quickly, does 'running the optimizer' really require the printing of the 'Best Agent accuracy and parameters'? - Apologies as I am new to all of this and it maybe a silly question.

I worked it out.


Show artboard names in viewport

Currently it is possible to rename an artboard by double clicking in the layer tree. However this change is not reflected in the viewport, where it is still titled "Artboard".

See impl GraphicElementRendered for Artboard for the artboard rendering code. This change necessitates the passing of the name string into the graph execution context, whereas currently it is just visible as metadata on the document graph and so not sent to the actual node implementations at all. If the ConstructLayerNode got the layer name as input then it could apply the name to the passed in GraphicElement, which would involve storing the string in the Artboard struct and then retrieving that during rendering.

Hello, I would like to work on this issue. If it is OK, I can send a draft PR and start working.

This should be available, so please open a PR for it, thanks!

Closed by #1795


Drawer Input Obstructed by Keyboard in Mobile

Discussed in https://github.com/shadcn-ui/ui/discussions/2564

Originally posted by mrm2234 January 25, 2024
Hi All,
I am trying to use a drawer to capture a user's input using an  component in a Drawer. On mobile, the keyboard is obstructing the user as they are trying to input a number.
Has anyone also seen this behavior / have a fix for it?
TIA!

I'm also facing this issue, the responsive drawer demo on the official website also has this problem when opening in mobile. It does weird height adjustments, often obstructing inputs or adding a lot of empty space at the bottom.

Hello, i am also having the same issue.

A minimal repro https://github.com/WhyAsh5114/shadcn-drawer-mobile
https://github.com/shadcn-ui/ui/assets/71895020/42cc6363-e0bb-4e47-a1bd-e4f583614955

+1

+1

Hi! I had the same issue, but after scrolling vaul git find this solution
// This code is simplified for the sake of this article
React.useEffect(() => {
function onVisualViewportChange() {
const visualViewportHeight = window.visualViewport.height;
const keyboardHeight = window.innerHeight - visualViewportHeight;
// Difference between window height and height excluding the keyboard
const diffFromInitial = window.innerHeight - visualViewportHeight;

const drawerHeight = drawerRef.current.getBoundingClientRect().height || 0;

drawerRef.current.style.height = `${visualViewportHeight - OFFSET}px`;
drawerRef.current.style.bottom = `${Math.max(diffFromInitial, 0)}px`;

}
window.visualViewport?.addEventListener("resize", onVisualViewportChange);
return () =>
window.visualViewport?.removeEventListener(
"resize",
onVisualViewportChange
);
}, []);

This issue duplicates #2831.
I will try the solution shared by @NeDurov, thanks!

@rustyjux let me know if it works, I'm very curious.

This issue duplicates #2831. I will try the solution shared by @NeDurov, thanks!

Did u fix it?

This issue duplicates #2831. I will try the solution shared by @NeDurov, thanks!

Having the same issue, a white blank "thing" appears wenn i click on an input

Hi this problem is a issue in vaul. I made a comment in a discussion over there.
Hope this helps or maybe better to move the questions over there because those are the people directly working on this part.
I commented on the issue with a temporary fix so if you need it feel free to check it out

I have encountered the same problem using Chrome on Android - but it seems to work fine using Firefox for Android!

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.

Hello, I am still currently experiencing this issue. I tried to fix it but to no avail, so I temporarily resorted to using a Dialog instead of a Drawer. Thanks!!

Did anyone fix this issue? I'm still having a problem with this

I solved this issue like that
  const formContainerRef = useRef<HTMLDivElement | null>(null);

  useEffect(() => {
    const handleResize = () => {
      if (formContainerRef.current) {
        formContainerRef.current.style.setProperty('bottom', `env(safe-area-inset-bottom)`);
      }
    };

    if (window.visualViewport) {
      window.visualViewport.addEventListener("resize", handleResize);
      handleResize(); // Initial call in case the keyboard is already open
    }

    return () => {
      if (window.visualViewport) {
        window.visualViewport.removeEventListener("resize", handleResize);
      }
    };
  }, []);

<DrawerContent ref={formContainerRef} className="min-h-[70vh]">
...
</DrawerContent>

The problem i see here is IMO not the drawer itself but rather IOS zooms in to focus on input field with font size < 16px. I found 2 potential solutions:

add maximum-scale=1 attribute to your html's viewport meta more on the topic
make sure input you're focusing in has font size >= 16px more on the topic

for me, adding
<head>
    <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1" />
 </head>

to html resolved the issue


HOTKEY "HOME", "END".

im currently using 96 keys keyboard. is there any chance i can change the default "HOME", "END" key to open the GUI ?
EDIT. BECAUSE IT DOESNT HAVE "HOME", "END" KEY.

sorry my english bad but "shift + A" isnt working the GUI of renskin isnt opening.

im currently using 96 keys keyboard. is there any chance i can change the default "HOME", "END" key to open the GUI ?
EDIT. BECAUSE IT DOESNT HAVE "HOME", "END" KEY.

u can try using On-Screen Keyboard, just search in windows search bar and it works to me fine. I hope it helps u


Remove deadlock from AuthMap Endpoint GC (rebased)

https://github.com/cilium/cilium/pull/29082 rebased against the latest origin/main

/test

ci-runtime - https://github.com/cilium/cilium/issues/29645
ci-aks - failed to connect to <IP_ADDRESS>


Discussion on behaviors of orders that have sub-orders

As you might know, SimplCommerce has support multi-vendors feature from its core. This feature allow other sellers sales their product in the site. They can create and manage their own products, also their warehouses, their stock in the site.
Whenever a customer place an order which contains products from vendors, sub-orders are created for each vendors so that they can manage by themselves.
Currently there is no difference between orders and sub-orders. I think we should distinguish them. For example:

In the master orders show links to its sub-orders
Create shipment is not allowed on master orders
The master order status will automatically updated base on its sub-orders ?

I think it's good to to consider the site owner is also a vendor in multi-vendor setup.
Please let me know how do you think about this

@hamdiRuwiss how do you think?

Some ideas:

Parent order link to children orders
Children orders links to parent order
Children orders have a suffix, such as "-1", "-2", "-3", etc.
Parent order depends on statuses of children orders
Children order payments can be individually handled, e.g. to allow the child order to be captured when shipped
Checkout page shows a breakdown of which items belong to which vendors
Ability to cancel entire parent order with its children, or each child order individually

Some ideas as well:
I would call them order  groups, only grouping order line items when we have products from multiple vendors in place.
Each group of items would map into different proffered payment gateway by the vendors (not  a concern to the customers)
From the view of the customer orders are atomic ,I don't think it is a good idea to break down into sub orders, but form  the view of the vendors you are right to break the order into sub orders or groups so that every order  or group is handled by the seller.
Customers should track which order products group shipped
Changing the order status should be a matter of its groups one vendor one order group one status,
two vendors one order two order groups two statuses

The current implementation:

If the order contains products from vendors, beside the original order, create a sub-orders for each vendors.
Except the COD, the payment has already done before creating orders.
Customers do not see sub-orders
No special status handling for this scenarios have been implemented

You can checkout the code here https://github.com/simplcommerce/SimplCommerce/blob/master/src/Modules/SimplCommerce.Module.Orders/Services/OrderService.cs


Refactoring of Schema::normalizeDefaultValue() method

CURRENT_TIMESTAMP for default value was not Expression()
fix empty strings '' in default value
fix single quote in default value
fix DEFAULT NULL definition

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
-

👍


Add ruff to pre-commit

This is opt-in feature to run ruff as pre-commit.
Enable this with
poetry run pre-commit install

Then, every time you commit, ruff will run on the Python code.

Thank you for this, Jiri. I suppose that it works just fine in your setup where you have global pre-commit hook configured. There was some discussion how this will work in a setup where there is a pre-commit hook per repo? Has this been tested?
Note: not sure if I expressed myself correctly :smile:

Anyway, if this is something that should be enabled/disabled manually per user - we should put some note somewhere in our readme/doc. WDYT?

Also, what happens once the pre-commit dependency is updated in the pyproject.toml file? Is it necessary to enable it again so the updated dependency is used or is it grabbed automatically?

You said it exactly right. Actually, what I have is the unsupported configuration that requires workarounds to make it work. https://stackoverflow.com/questions/67793193/cowardly-refusing-to-install-hooks-with-core-hookspath-set/78243646#78243646 What everybody else has is the recommended setup and this should work seamlessly with it.

I wanted to first see how it works for others. I added a topic to tomorrow's meeting, proposing to merge this so people can experiment how it works for them.

The pre-commit tool is responsible for setting up a local git hook, which is a shell script. The way the shell script works is that it detects active venv and runs pre-commit from a venv. I'm willing to take it on faith that it will work fine. Pre-commit is what everybody uses.


Fixed eslint workflow syntax

#1039  👈
main

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

Join @CatChen and the rest of your teammates on  Graphite


Arrow down pictogram next to G icon in tray

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

📝 Description
What is this icon that won't go away?
If I had to guess - it's a new version available even though nothing would suggest that. On github I can see a newer version is available; though in the app settings there's a green mark suggesting latest version is installed (5.12.1; not 5.13.1 as in gh releases).
It's a bit annoying with absolutely no info.
🪜 Steps To Reproduce
look at the tray.
Gitify Version
5.12.1
Operating System
macOS
GitHub Account
GitHub Cloud
📸 Screenshots

Please update to 5.13.1 which fixes this bug


json rendering

need to be more consistent on how we render json
currently using both the json conn .... function and also the plain render function with a view template

closed via #81


Add instrumentation for knex.

https://www.npmjs.com/package/knex
weekly downloads 640k+
More context:
https://github.com/lightstep/otel-launcher-node/issues/2

This instrumentation has been added by @Rauno56
https://github.com/open-telemetry/opentelemetry-js-contrib/tree/main/plugins/node/opentelemetry-instrumentation-knex


emit code didn't follow .mts .cts extensions

Bug Report

🔎 Search Terms
mts

🕗 Version & Regression Information

This is a crash
This changed between versions ______ and _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link

Playground link with relevant code
💻 Code
temp.cts
export default 77

temp.mts
export default 88

🙁 Actual behavior

.cts output as esm when tsconfig is "module": "esnext"
.mts output as CommonJS when tsconfig is "module": "CommonJS"
🙂 Expected behavior

.cts should always output as CommonJS when tsconfig is "module": "esnext"
.mts should always output as esm when tsconfig is "module": "CommonJS"

AFAIK .cts and .mts are only supported with module set to node12 and nodenext
See https://www.typescriptlang.org/docs/handbook/esm-node.html for details about experimental (Nightly builds only) support for these types of files.

This is working as expected - these extensions function identically to .ts (in terms of emit) except in the node12 and nodenext module modes, where they adopt the emit implied by the extension.


feat: Infer simple TS types

This PR resolves #33.
Summary
This PR implements type inference based on right-hand side value. At the moment, only simple types are inferred.
Examples:

const n = 100 is supported.
const d = new Date() is supported.
const sum = add(1, 2) is not supported.

Output
sample.ts
export const hello = "hello, world";
export const i = 100;
export const b = true;
export const re = /hello/
export const bi = 100n;
export const d = new Date();
export const tpl = `foobar`;

Before this fix:
Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:3:0

const b

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:5:0

const bi

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:6:0

const d

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:1:0

const hello

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:2:0

const i

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:4:0

const re

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:7:0

const tpl

After this fix:
Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:3:0

const b: true

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:5:0

const bi: 100

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:6:0

const d: Date

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:1:0

const hello: "hello, world"

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:2:0

const i: 100

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:4:0

const re: RegExp

Defined in file:///home/uki00a/ghq/github.com/uki00a/deno_doc/sample.ts:7:0

const tpl: "foobar"

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[AutoDiff] Canonicalize SIL type for JVP/VJP methods.

Canonicalize JVP/VJP type calculation for methods so that JVP/VJP methods return
a linear map that take/return self's tangent/cotangent last, instead of first.
This matches the type calculation logic for top-level functions.
Changes:

Remove method-specific logic from
SILFunctionType::getAutoDiffAssociatedFunctionType.
Handle self like a normal differentiation parameter.
Thunk user-defined JVP/VJP methods, reordering the position of self's
tangent/cotangent in returned linear maps.

Relevant code:
SILGenModule::getOrCreateAutoDiffAssociatedFunctionReorderingThunk and
SILGenFunction::getOrCreateAutoDiffLinearMapReorderingThunk.

Change type computation for JVP/VJP method protocol witnesses.

Relevant code: getSILFunctionType in SILFunctionType.cpp and
SILGenFunction::emitProtocolWitness.

Move some functions to a common location:

AnyFunctionType::getAutoDiffOriginalFunctionType
autodiff::getAutoDiffFunctionLinkage

This is a step towards refactoring the differentiation transform (iterative autodiff_function worklist redesign), which will unblock control-flow support.

Triggering tests to see if anything fails. More cleanup/doc comments coming up later.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

Can you make the AST type of the associated functions have self last? i.e. make AnyFunctionType::getAutoDiffAssociatedFunctionType return types with self last?

Good idea! We discussed in person and I discussed with @rxwei - we agree that computing self-parameter-reordered JVP/VJP types on AST function types instead of SIL function types is safer and simpler.
Done in https://github.com/apple/swift/pull/24775/commits/063f4fd2b14fed4c9a380cee3ff89c8fad697dc7, removing 100+ lines from SILGen and adding <10 lines to AST/Sema.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow


Match on Req.Request to allow additional options (e.g. ipv6)

Besides providing a binary or %URI{}, I'd like to be able to provide an ipv6 address. It currently fails because Req/Mint expects it as option (inet6). So instead I would suggest providing an additional match on Req.Request{}:
req = Req.new(base_url: "http://example-app.flycast/api", connect_options: [transport_opts: [inet6: true]])
api = Ollama.API.new(req)
Ollama.API.completion(api, [model: "llama2", prompt: "Why is the sky blue?"])

Thanks - nice change
Will do a new release tomorrow 👍🏻


feat: check commit messages to see if a new version needs to be published

What does this PR do?
Checks commit messages to see if we need to publish a new version or not. For example

if only a chore commit is found, then it will exit early.
if both a chore and feat commit is found, then it will continue with the release

TODO:

[x] fix broken unit tests

What issues does this PR fix or reference?
@W-8775832@
@W-8820671@

Can we also add some information to a readme somewhere if it isn't already? Seems like it should be doc'ed

Can we also add some information to a readme somewhere if it isn't already? Seems like it should be doc'ed

Testing notes:
If I have no type or a random invalid type, it will still publish. You could argue that is a problem with the repo not validating commit messages but it could also be bypassed by the command line or UI. In that case, maybe it should be published. If the answer is on, Instead of skippableCommitTypes we should have commitTypes: ['feat', 'feat!', 'bug', 'bug!', 'perf', 'refactor'].  I'm fine keeping it skippable too. Either way, it is a list we need to maintain.
I published ~15 package versions with a variety of commit messages in between each release testing patch, minor, and major. Other than the issue above it works great.
I'll wait for @mdonnalley feedback on the issue before merging.

Testing notes:
If I have no type or a random invalid type, it will still publish. You could argue that is a problem with the repo not validating commit messages but it could also be bypassed by the command line or UI. In that case, maybe it should be published. If the answer is on, Instead of skippableCommitTypes we should have commitTypes: ['feat', 'feat!', 'bug', 'bug!', 'perf', 'refactor'].  I'm fine keeping it skippable too. Either way, it is a list we need to maintain.
I published ~15 package versions with a variety of commit messages in between each release testing patch, minor, and major. Other than the issue above it works great.
I'll wait for @mdonnalley feedback on the issue before merging.

@amphro I think it's fine to allow it to publish in those scenarios. If someone has gone through the trouble of bypassing the commit linting, then they are probably (hopefully) doing it purposely. That being said, we can always address that scenario later if it becomes an issue

@amphro I think it's fine to allow it to publish in those scenarios. If someone has gone through the trouble of bypassing the commit linting, then they are probably (hopefully) doing it purposely. That being said, we can always address that scenario later if it becomes an issue

Sounds good to me. I just wanted to make sure we had the conversation.

Sounds good to me. I just wanted to make sure we had the conversation.


fix(config): make blink lua config fields optional

Hello @Saghen
First of all, congrats on the plugin! My initial impression has been great.
This patch fixes a somewhat selfish issue of mine. I prefer to use the config function to run configure plugins over opts just for a bit more flexibility and the added benefit of completions for option names.
However, in the current blink.cmp.Config class, all fields are required. This results in a bit of an unsightly warning:

With this fix, the options aren't required anymore (which reflects reality) and the warning is no longer present when configuring via the config option in lazy.
Let me know if you have any questions!

Completely understandable!
I'm not aware of anything like this, but will look into a suitable compromise.

@scottmckendry See this from nvim-best-practices which talks about exactly how you would solve this specific issue: https://github.com/nvim-neorocks/nvim-best-practices?tab=readme-ov-file#wrench-configuration
@Saghen I would recommend looking at some of the other recommendations in this repo (it is currently being upstreamed to neovim documentation at the request of a main neovim contributor) as I believe this plugin would benefit from some of the other notes also. Amazing plugin by the way! I was thinking about writing one myself because of some of the pitfalls of cmp and this plugin came in at the perfect time.

Thanks for the steer @simifalaye. This is a really handy resource.
In this case, I'll leave the decision to @Saghen. The benefits of the multi-class approach are there, but my opinion is that they're not worth the effort of maintaining for Blink.
My reasoning for this:

There are very few references to blink's config types outside of their definitions in config.lua. I counted less than 10 (as of right now in main).
We're not really losing out on any type-safety by making the fields optional. You'll still get type warnings, and the defaults are set on every field where nil isn't valid.
I started adding the internal type annotations started to get pretty crazy looking. It might be overwhelming for other contributors in the future, as mentioned in the best-practices doc:

As this is fairly uncommon, first-time contributors will often overlook one of the configuration types.

I'm fine with whatever decision, just thought I would put my two cents in 🙂

Makes sense, thanks for the PR!


Merge pull request #1 from github/master

Incorporating new changes
Reasons for making this change:
TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

Created by mistake


Redesign filter sidebar

Why these changes are being introduced:
The filter sidebar can become unwieldy, as all available filters are shown at all times.
Relevant ticket(s):
https://mitlibraries.atlassian.net/browse/GDT-127
How this addresses that need:
This restyles the filter sidebar as dropdown menus. In so doing, it also changes some behavior in the filter logic:

Number of results is listed in the filter sidebar header.
Applied filters are highlighted (previously, the 'applied' filter was added, but we hadn't done anything with it).
Clicking an applied filter removes that filter (this replaces the previous removal mechanism, which removed all applied filters).

Side effects of this change:

Some tests have been updated to reflect the new logic.
The sidebar elements have an animation that's tied to height. I've set the height to 200px and added a scroll-behavior rule if the number of filters available exceeds that height. This is somewhat different from how the advanced search sidebar works, so I wanted to draw attention to it unless there are any concerns.
The JS for the dropdown menus is so minimal that I included it in the view as a script tag, but I'm happy to load it via Sprockets if that's preferable.

Developer

[x] All new ENV is documented in README
[x] All new ENV has been added to Heroku Pipeline, Staging and Prod
[x] ANDI or Wave has been run in accordance to
our guide and
all issues introduced by these changes have been resolved or opened as new
issues (link to those issues in the Pull Request details above)
[ ] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer

[ ] The commit message is clear and follows our guidelines
(not just this pull request message)
[ ] There are appropriate tests covering any new functionality
[ ] The documentation has been updated or is unnecessary
[ ] The changes have been verified
[ ] New dependencies are appropriate or there were no changes

Requires database migrations?
NO
Includes new or updated dependencies?
NO

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.008%) to 99.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
355

Relevant Lines:
357

💛  - Coveralls

The latest commit addresses some initial feedback on this, both from this review and from Darcy (I shared it with her in a meeting yesterday afternoon). There are also a couple of things that came up while I was doing some testing related to the code review/QA feedback. More details in the commit message.


WIP: Fix tests ocp version

Only a test

Ah... it was listed as failure b/c we changed the RTs for this: 4.11 was removed, and 4.12 was added

/close


list all addons and their state

would be nice to list all add-ons on aks and if they are enabled.

@smartpcr thanks for the feedback, we've backlogged this request in agreement.

Revisiting, given this is available in the enable-addons command help text I am going to close for now. But I have opened this related issue on AKS to provide a way to retrieve all available addons programatically. https://github.com/Azure/AKS/issues/1341
➜  ~ az aks enable-addons -h
The behavior of this command has been altered by the following extension: aks-preview
The extension is in preview

Command
    az aks enable-addons : Enable Kubernetes addons.
        These addons are available:
            http_application_routing - configure ingress with automatic public DNS name creation.
            monitoring - turn on Log Analytics monitoring. Requires "--workspace-resource-id".
            virtual-node - enable AKS Virtual Node (PREVIEW). Requires "--subnet-name".
            azure-policy - enable Azure policy (PREVIEW).

Hi
think the result of this cmd 'az aks enable-addons -h' is just what values of addons it can accept, not what kinds of addons is available in Azure environment.
Problem will occur, when I use it in AzureChinaCloud.
It will give the same result as Azure Global, however, features like http_application_routing is not available for AKS in AzureChinaCloud

got answered : https://github.com/Azure/azure-cli/issues/26937#issuecomment-1642874929


pg-replication-slot FIX: Calling super.doStart() after initializing the consumer to avoid poll() being called before the Consumer is initialized

Description
Calling super.doStart() before initializing the consumer might cause poll() being called before the consumer is ready. This can happen if connect() takes relatively long time.
Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

Thanks, I'm going to backport on 4.4.x and 4.0.x too


Dependent parent - child lists possible?

Hey, is there a way where I can set up a parent child dependency with 2 or even more lists so the child list shows only entries of the selected parent?
Would be great feature!

Hi,
Not at the moment, but indeed it's a very nice feature. I already found myself needing it. I'll implement it in the future.
Cheers!


fatal libpng error: Read Error: truncated data

PHP Fatal error: imagecreatefrompng(): fatal libpng error: Read Error: truncated data in phar:///media/pictures/gayfurries/e6brs/e6BRS.phar/app.php on line 743
This error appeared after picture 541/1671 of a Tumblr rip I did on my blog. The picture that it threw this error on is in fact truncated, hence the error message, but isn't there a fail-safe that can be implicated to work around this? Had I not been at my computer while this was running, I would've come back hours later to realize that the tool had thrown an error message hours before.

@Youboontoo There wasn't any fail-safe because I didn't know about this issue before, all errors from 'imagecreate' were warnings for me.
Will apply a fix soon.

The same error is occurring, even after updating the script:
[1/1131] Searching 'tumblr_oves74yED51tek5qpo1_raw.png':
Trying md5 sum... no matching posts found!
Trying reverse search...PHP Fatal error: imagecreatefrompng(): gd-png: fatal libpng error: Read Error: truncated data in phar:///media/pictures/gayfurries/e6brs/e621BRS.phar/app.php on line 744

However, I think the gd-png part is new... Lemme check if all my binaries are up to date.

Nope, everything is up to date.
uname -a:
Linux tux-server 4.10.0-38-generic #42~16.04.1-Ubuntu SMP Tues Oct 10 16:30:51 UTC 2017 i686 i686 i686 GNU/Linux
php -v:
PHP 7.0.22-0ubuntu<IP_ADDRESS> (cli) ( NTS )

@Youboontoo Can I have the problematic image so I can test it?

From what it looks like before the truncation, it is an NSFW picture so beware.
https://imgur.com/a/aovVC

Like I mentioned earlier, the picture is in fact truncated, so that's what is causing this issue.

@Youboontoo I Googled a bit and it seems to be some kind of bug in PHP on Linux, could you try with PHP 5.6?
This issue does not happen for me on Windows PHP 5.6 and 7.1.

As soon as I get back home, I'll roll back to php5.6 and see if the issue
is resolved. If all else fails, I'll try it on my Windows box.
On Nov 8, 2017 12:31 PM, "jacklul"<EMAIL_ADDRESS>wrote:
@Youboontoo https://github.com/youboontoo I Googled a bit and it seems to
be some kind of bug in PHP on Linux, could you try with PHP 5.6?
This issue does not happen for me on Windows PHP 5.6 and 7.1.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jacklul/e621-Batch-Reverse-Search/issues/7#issuecomment-342949905,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APjAN4YOGSYy-nt1PBzeGoQOR1jbn1PNks5s0g-9gaJpZM4QV9UE
.

Rolling back now, will post results shortly

php -v:
PHP 5.6.32-ubuntu16.04.1+deb.sury.org+1 (cli)
Error message is still same as above, will test on Windows next.

Windows does not reproduce the same error message.
[1051/1131] Searching 'tumblr_oves74yED51tek5qpo1_raw.png':
 Trying md5 sum... no matching posts found!
 Trying reverse search... error: cannot create JPEG - expected resource input

Applied a workaround and tested it on Windows Subsystem for Linux, seems to work fine.

I can confirm that this fixed the issue. Thanks for your patience.


lsrepo

--not-clean
--multi-branch
--multi-remote
--remote-info

--global (all repos in GOPATH, ~)
--list (show repo info in on line per repo)

--one ? (show repo info one by one)

the result is
~/xx/oo origin:haoxin/oo upstream:org/oo


Progress bar no longer launches [BUG]

Describe the bug
I'm using the progress bar to track a process in my script.
I can't set the bar like "for _ in stqdm(range(x)):" because I have several loops to control process and I only want one bar.
after several uses or when I do large calculations, the bar is no longer usable and the script no longer works. There is no error displayed and the bar is no longer displayed. However the st.spinner launches well before.
To Reproduce
import streamlit as st
from stqdm import stqdm

def exemple_run(test1, test2):
    total_iterations = len(test1) + len(test2)
    with stqdm(total=total_iterations, desc='Calculating...', mininterval=0.1) as pbar:
         for _ in test1:
            pbar.update(1)
         for _ in test2:
            pbar.update(1)

if st.button("Exemple"):
   with st.spinner("Processing..."):
      exemple_run(test1, test2)

Desktop (please complete the following information):

Browser: this is a Streamlit Cloud
Version of streamlit: latest
Version of stqdm: latest
Version of tqdm: latest

Hi @Jumitti ,
I tried to reproduce your example here : https://stqdm-bugs-with-concatenated-iterator.streamlit.app/
With the following code https://github.com/Wirg/stqdm/blob/0c2e476abdd1aa1ac5f0fb1b4b58eb96d1ddd551/examples/bugs_with_concatenated_iterator.py . I had to infer what test1 / test2 could be.
Does it fit what you're doing ?
Can you clarify what you're doing to enter the blocking situation ?
I really had to mash the buttons especially both "Exemple" and the "STOP" (from streamlit interface) to run into a blocking situation. Then it solves itself if I just reload the page.
It does not seem to be what you are running into tho.
Does it need to be longer ?

Hello @Wirg
Thanks to answer :)
Your code is very similar to mine, at least in the process.
I think after reflection and a lot of testing that it is due to the processing speed of Streamlit Cloud or the 1GB RAM.
I have this problem in 3 cases.
When I do too many pbar.update(1)
When my function takes too long
When I modify a widget which is necessary to launch the function.
In the first 2 cases I think it's the 1GB of RAM that breaks everything. And for the 3rd case I think it's the fact that Streamlit restarts the whole script as soon as you make a modification with a widget.
I tried STOP during the execution of the function but it still creates the bug. After refreshing the application page, all functions where stqdm is used no longer work until I completely reboot the application.
I think that's really the way Streamlit Cloud is made. Because when I do it locally it works fine.
For information, I do more than 75,000,000 calculations with one function. I ask a lot of streamlit :)
And it's probably not an .update problem. I decreased the number of .updates I do and I also had the problem.

Hi @Jumitti
You're probably asking a bit too much from their 1GB of RAM.
On the stqdm-side you might be able to reduce the load by relying on tqdm parameters. They've done a lot of work to handle this kind of things.
For example, you could set mininterval to 0.5s so that there are around 5x less updates to do.
There could be some locking issue with stqdm / streamlit in some cases.
I still need to investigate but it seems that :

on windows there is the locking issue reported here
in some case with numerous forced stops or disconnection, there is a sort of race condition that could not completely stop the process. I need to take the time to properly investigate this but it seems to be pretty niche.

Hi @Wirg
Thanks you again, and yes i think your are right 😅
Yes i try to set an higher mininterval. But bug append also. It's really a problem between how python works and how streamlit cloud works i think.
On my PC in localhost there is no problem. So good luck to find how to solve it with Streamlit Cloud 😅 If you need any help, let me know 😊


Auth token lookup failed

Hi, I'm from China. I tried to debug the tool on my cellphone, and got the following issue.
3928-5053/com.github.mobile:auth D/GHAccountAuthenticator﹕ Retrieving OAuth2 token
06-28 21:34:43.957    3876-5011/com.github.mobile E/GitHubAccount﹕ Auth token lookup failed
java.io.IOException: org.eclipse.egit.github.core.client.RequestException: Validation Failed (422): OauthAccess resource with 'description' field already exists

Hi, we're in the process of fixing oauth right now, so please bear with us
for a bit.
On Sun, Jun 28, 2015, 6:41 AM jimmy.m.gong<EMAIL_ADDRESS>wrote:

Hi, I'm from China. I tried to debug the tool on my cellphone, and got the
following issue.
3928-5053/com.github.mobile:auth D/GHAccountAuthenticator﹕ Retrieving
OAuth2 token
06-28 21:34:43.957 3876-5011/com.github.mobile E/GitHubAccount﹕ Auth token
lookup failed
java.io.IOException: org.eclipse.egit.github.core.client.RequestException:
Validation Failed (422): OauthAccess resource with 'description' field
already exists
—
Reply to this email directly or view it on GitHub
https://github.com/forkhubs/android/issues/832.


Use voluptuous for Zone

Description:
Migration of the configuration check to voluptuous.
Related issue (if applicable): fixes 127528299
Example entry for configuration.yaml (if applicable):
zone:
  name: School
  latitude: 32.8773367
  longitude: -117.2494053
  radius: 250
  icon: mdi:school

zone 2:
  name: Work
  latitude: 32.8753367
  longitude: -117.2474053

It would be nice if somebody could take a look at the changes and run a quick test. Thanks.

Something not right with this. Getting an "expected a dictionary for dictionary value" error.

Fixed it in #3413