augeas: 1.10.1 -> 1.11.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from augeas.
meta.description for augeas is: '"Configuration editing tool"'.

Checks done (click to expand)

built on NixOS
/nix/store/7aaw58n56hjlqlal8515dwpkv32r28lk-augeas-1.11.0/bin/augtool passed the binary check.
/nix/store/7aaw58n56hjlqlal8515dwpkv32r28lk-augeas-1.11.0/bin/augparse passed the binary check.
/nix/store/7aaw58n56hjlqlal8515dwpkv32r28lk-augeas-1.11.0/bin/augmatch passed the binary check.
/nix/store/7aaw58n56hjlqlal8515dwpkv32r28lk-augeas-1.11.0/bin/fadot passed the binary check.
4 of 4 passed binary check by having a zero exit code.
0 of 4 passed binary check by having the new version present in output.
found 1.11.0 with grep in /nix/store/7aaw58n56hjlqlal8515dwpkv32r28lk-augeas-1.11.0
directory tree listing: https://gist.github.com/f80f83412d8b74e72cc9a364bc55d915
du listing: https://gist.github.com/76301f02564125fffabe175796ebfce2

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/07d33b002400649f9f3f42e574020ea719651e8f.tar.gz -p augeas
ls -la /nix/store/7aaw58n56hjlqlal8515dwpkv32r28lk-augeas-1.11.0

cc @offlinehacker @ndowens for testing.

Success on aarch64-linux (full log)
Attempted: augeas
Partial log (click to expand)
shrinking /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0/bin/augparse
shrinking /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0/bin/augtool
shrinking /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0/lib/libaugeas.so.0.24.1
shrinking /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0/lib/libfa.so.1.5.2
gzipping man pages under /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0/share/man/
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0/lib  /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0/bin
patching script interpreter paths in /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0
checking for references to /build in /nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0...
/nix/store/nmpjk5xnv81ni32v53lmgy4g3vs6997a-augeas-1.11.0

https://github.com/hercules-team/augeas/blob/release-1.11.0/NEWS


whitespace breaks sed command

Apologies, the whitespace in the sed command -i "" broke the pre-deploy.sh script.  The pull request fixes that issue.
JAR_DIR:/home/ec2-user/environment/sleeper/scripts/jars
VERSION:0.11.1-SNAPSHOT
Generating properties
sed: can't read : No such file or directory

Aside: Interestingly, MacOS bash does not like the empty string:
Generating properties
sed: -I or -i may not be used with stdin

However if I put an extension in there, it is happy.

Looks like we have the same issue in scripts/dev/updateVersionNumber.sh. Could you fix that at the same time, please?

Thanks, merged into main.


Optional Type guessing at import UI

Right now, the only way to avoid it is using the Import API, but this workaround is not easy for some users.
We've had multiple tickets about this, for example:
https://cartodb.supportbee.com/tickets/3727186
https://cartodb.supportbee.com/tickets/3615729
https://cartodb.supportbee.com/tickets/3570980
...
This provokes for example issues with zipcodes, as it removes the zeroes at the left.

Designed in the new dashboard

cc @xavijam


fix: handle blank (as opposed to nil) links

Closes: #1
Goal
Don't add link copy when links are blank.
Approach
Add .present? to the conditional.

Pull Request Test Coverage Report for Build 709993296

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.094%

Totals

Change from base Build 709910483:
0.0%

Covered Lines:
123

Relevant Lines:
128

💛  - Coveralls


Added 'submit feedback' button to the page that shows up when a person cannot start a cloud server

Also added new feedback option that reflected this error

This change is

This looks great! wrt UI would you be okay trying something a little more subtle?

Fixed the style!

Review status: 0 of 5 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Do you have a new screenshot?
Also will you make the related change for the website? https://github.com/uProxy/uproxy-website/blob/master/polymer/uproxy-feedback-summary-table.html
Awesome stuff

Reviewed 2 of 5 files at r1, 1 of 3 files at r2.
Review status: 3 of 5 files reviewed at latest revision, 1 unresolved discussion.

src/interfaces/uproxy_core_api.ts, line 25 [r2] (raw file):

  CLOUD_CONNECTIONS_DISCONNECTED = 2,
  CLOUD_SERVER_NO_CONNECT = 3,
  CLOUD_SERVER_NO_START = 4,

Oh! It's great to group the cloud ones together!
Just as a side note: It's ok here to shift all the numeric values by 1 because in ui.ts, we convert feedback type from numbers to strings before we send the feedback to the backend. However, if we sent the feedback type to the backend as a number, then old and new feedback types could get mixed up (e.g. no_friends used to be 5 but now trouble_signing_in is 5). Not a problem at all here, but just thought it was worth a mention!

Comments from Reviewable

Yup! Here's the pic

Review status: 3 of 5 files reviewed at latest revision, 1 unresolved discussion.

src/interfaces/uproxy_core_api.ts, line 25 [r2] (raw file):
Previously, lucyhe wrote…
> Oh! It's great to group the cloud ones together!
> Just as a side note: It's ok here to shift all the numeric values by 1 because in ui.ts, we convert feedback type from numbers to strings before we send the feedback to the backend. However, if we sent the feedback type to the backend as a number, then old and new feedback types could get mixed up (e.g. no_friends used to be 5 but now trouble_signing_in is 5). Not a problem at all here, but just thought it was worth a mention!

Good to know, thanks!

Comments from Reviewable

:+1:


Testing with remote mode is not stable

When testing locally with Ruby 3.1.0-dev, I sometimes get errors like this:
Error: test_break_with_namespaced_class_method_stops_at_correct_place(DEBUGGER__::BreakAtMethodsTest): Errno::EMFILE: Too many open files - fork failed
/Users/st0012/projects/debug/test/support/utils.rb:77:in `spawn'
/Users/st0012/projects/debug/test/support/utils.rb:77:in `block in new_child_process'

Or timeout error:
Failure: test_backtrace_prints_the_return_value(DEBUGGER__::BasicBacktraceTest):
  TIMEOUT ERROR (10 sec)
  [DEBUG SESSION LOG]
  > [1, 10] in /var/folders/yg/hnbymwxd5pn7v94_clc59y6r0000gn/T/debugger20210701-59574-vo4b86.rb
  > =>    1| class Foo
  >       2|   def first_call
  >       3|     second_call(20)
  >       4|   end
  >       5|
  >       6|   def second_call(num)
  >       7|     third_call_with_block do |ten|
  >       8|       num + ten
  >       9|     end
  >      10|   end
  > =>#0        <main> at /var/folders/yg/hnbymwxd5pn7v94_clc59y6r0000gn/T/debugger20210701-59574-vo4b86.rb:1
  > b 4
  >
  > (rdb) b 4
  .
  <false> is not true.

The master build also fails for timeout error. I can't rerun it but it looks like a random failure too.

@ono-max can you take a look? thanks

I think this is not issue.I saw kind of this error before merging remote test.
Since we started testing on remote, the number of tests increased by 3 times.That's why probability of random failure also increased.

Hmm... I can't reproduce the Errno::EMFILE:. When I look into it, it may be able to be solved this problem by configuring your local environment.

This issue happens a lot on Ruby 3.1.0-dev, have you tested against it?

Since we started testing on remote, the number of tests increased by 3 times.That's why probability of random failure also increased.

Even with remote tests, we only have < 500 assertions. It's not acceptable to just say "oh it fails sometime".
Random failures make our tests unreliable and increase the chance to let real issues slip in. It also increased the maintainers' work because now the CI needs to reran very often.

In the recent 8 builds, 4 of them failed, including master's. The 50% chance of failure is too high to accept.

This issue happens a lot on Ruby 3.1.0-dev, have you tested against it?

Sorry, I missed the version. I'll check it again.

I fixed Errno::EMFILE: Too many open files - fork failed, so I'll continue to investigate how to fix TimeoutError.

@ono-max that sounds great, thank you 👍

@st0012
I think we can close this issue because TimeoutError was fixed in https://github.com/ruby/debug/pull/131.
Do you agree?

sure, thanks for al the fixes 😄


[LeftNav] Only transition the transform property

Fix @mbrookes comment:

It looks like when the Nav opens responsively it first appears in front of the AppBar, then pops behind it. Can that be avoided?

and https://github.com/callemall/material-ui/issues/2077.

@oliviertassinari I think you can obliterate modernizr.custom now :D :D
from the webpack, package, file system and all that. This was the only place it was used

Also is-browser

@alitaheri Yeah, that's good :tada:

+4 -313 True obliteration :D :D
Thanks a lot @oliviertassinari :+1: :tada: :balloon: :tada:


Machine Learning Services(Formerly R Services) in Preview in Managed Instance

The page shows that ML services is not supported on Managed Instance but it's in supported in Azure SQL Database. This seems incorrect.
ML Services is now in preview on SQL MI https://docs.microsoft.com/en-us/azure/azure-sql/managed-instance/machine-learning-services-overview
ML Services is out of support from Azure SQL DB now.
[Enter feedback here]

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

ID: 3fb7dad4-ff47-5b16-e8ee-0a5e037ea20f
Version Independent ID: 98ec245b-bd8f-bd96-ec6d-0048599246fa
Content: Compare the database engine features of SQL Database and SQL Managed Instance - Azure SQL Database & SQL Managed Instance
Content Source: articles/azure-sql/database/features-comparison.md
Service: sql-db-mi
Sub-service: features
GitHub Login: @jovanpop-msft
Microsoft Alias: jovanpop

@sumitsar Thank you for the feedback. We are actively investigating and will get back at the earliest.

@sumitsar Thanks for you valuable feedback.
Machine learning Services for Azure SQL Managed Instance is still in preview phase and hence it is not updated as supported on all the articles yet. However, once it is Generally Available(GA), the same shall be updated across the documents.

@sumitsar Please let us know if you have any other query on the same.

@sumitsar Please let us know if you have any other query on the same.

Thanks you Anurag for a quick reply on this. I don't have further queries. Thanks again!


chore: switch default web font provider

Google Fonts tracks >20% of the web traffic. No need to add to that.

Source: https://whotracks.me/trackers/google_fonts.html
Signed-off-by: Patrick Heneise<EMAIL_ADDRESS>

Thanks but I would need to reject this change. As the providers already provide the capability of using bunny if you want to, I don't think it's safe to flip this behavior for every user.

I didn't know of bunny before. Would be nice to see this in a major change, so it doesn't break behavior.

I didn't know it either, so I can't make promise to count on it's availability. It would be used with users' explicit awareness


Handle null locations on getLastLocation and remove stale last locations

This seems that it can happen on rare occasions
https://developer.android.com/training/location/retrieve-current#java

[x] Handle null locations after calling getLastLocation on the FusedLocationClient
[ ] Reset lastLocation to null if the location services or location on the device has been turned off
[x] Improve last location and location updates accuracy and suitability by using the algorithm provided by google https://developer.android.com/guide/topics/location/strategies#BestEstimate

To be able to save battery, once an accurate location have been retrieved. Can the Geowidget support a way for the host app to increase the frequency of updates.
https://developer.android.com/guide/topics/location/strategies#set-the-location-providers-to-return-updates-less-frequently

You mean decrease the frequency of updates?

Yes

Should we do that out-of-the-box or should that be something that is configurable by a host application if it wants that? I am thinking of a standard way to do that. I do not know if this should be the standard out-of-the-box?
Can you also provide the numbers in terms of updateIntervals that you are thinking about?

This can be configurable to the host application. The host application can should be able to setup updateIntervals


Flush session before logout.

it looks like this fixes the problem but some tests need to be updated


[shardformer] support disabling tensor parallelism

Tensor parallelism is not suitable for cases, e.g. small model training. We can set it to be optional.

Closed by #4144


new loki realtime logging handler

Thanks Wen. But can you please add function documentation including type hints? Also, please run pylint on the code and make sure you get a score around 9. This will soon all be requirements.

I updated the codes. Now pylint score is 9.25.

Great. Can you add the type hints as well please?

Added type hints

Sorry Wen, one more change - the requests module is not a standard Python module, can you please put a try-statement around it (the import + the single usage)?

updated to catch exceptions during importing requests.

Thanks


feat(lib): added createable select dropdown

there is now a createable select component, where the user can add custom text fields
https://github.com/mediatool/northlight/assets/81630176/6f4edd73-68ac-4e7d-be96-1d8dc6194976
closes DEV-11051

@TheLudd


Error uploading from OneDrive folders

{
"errno": -4071,
"code": "EINVAL",
"syscall": "stat",
"path": "C:\Users\lc\OneDrive - International Communication Partners Ltd\VideoProjects\TC_Kayak_2.mp4"
}

Sorry didn’t realize this is beta
Please let me know next time
thanks
From: Craig<EMAIL_ADDRESS>Sent: 27 April 2018 18:41
To: Microsoft/AzureStorageExplorer<EMAIL_ADDRESS>Cc: Lars Christiansen<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Microsoft/AzureStorageExplorer] Error uploading from OneDrive folders (#102)
Closed #102https://github.com/Microsoft/AzureStorageExplorer/issues/102.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/AzureStorageExplorer/issues/102#event-1599097196, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGjF6s7wyllIGKIfRCy1GUGsZ5Yi0-Zjks5ts0oTgaJpZM4TqQbx.

Check out #96 for news on this issue. The latest version of Storage Explorer is currently 1.0.0. We've officially released; we're not in beta.

Craig
With a bug like this i suggest you stay at beta untill fixed
Lars
Sent from my iPhone
On 27 Apr 2018, at 22:29, Craig <notifications<EMAIL_ADDRESS>wrote:
Check out #96https://github.com/Microsoft/AzureStorageExplorer/issues/96 for news on this issue. The latest version of Storage Explorer is currently 1.0.0. We've officially released; we're not in beta.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/AzureStorageExplorer/issues/102#issuecomment-385085536, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGjF6osvP0JpcIXu75JQACAFNaRG9h0fks5ts3-agaJpZM4TqQbx.


PlaneRenderer - Move to Filament mesh builder

Would you mind doing the move from RenderableDefinittion to Filament direct mesh creation?

We just need to make sure that we aren't missing any important features that Sceneform provided for mesh creation. I actually only used the direct Filament approach.
Originally posted by @grassydragon in https://github.com/SceneView/sceneview-android/issues/39#issuecomment-1066126542

I'll continue working on it after we resolve issues with Filament picking.


Issue with Advanced Paste and Slack interaction

Microsoft PowerToys version
0.81.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Advanced Paste
Steps to reproduce
When Advanced Paste is enabled and I click on the chat bubble for Slack that appears for new messages in the lower right corner it automatically pastes whatever is in my clipboard to the chat window.  I am running a fully updated version of Windows 10 and current version of Slack.
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
Slack Production 4.38.127 64-bit

This is #32971 .
This was supposed to be fixed on 0.81.1, which you seem to be running according to the issue description.
Can you please send a /bugreport so we can check if something is wrong with your 0.81.1 installation?

This happens to me too.
Reproduce:

Enable, Advanced Paste
Copy some text to the clipboard
Minimize Slack to system tray, double-click the icon to restore it
The text from the clipboard is now pasted into chat.
Disable, Advance Paste
Repeat steps 2 and 3, it no longer auto-pastes clipboard contents

@DamianSuess , Can you please update to 0.81.1 ? According to your bug report it seems you still have 0.81.0.

Thanks, @jaimecbernardo. Sorry for wasting your time, at a quick glance, I missed the x.y.1


Nested functons, closures and/or lambdas

Initially, this needs to only be for Numba-internal use.  List comprehensions require lambda support.  We may also want returnable inner functions.

List comprehensions are now supported in ParallelAccelerator.  Closing unless we come up with more concrete use cases.


Move platform-specific openers to their own modules.

@jonparrott I noticed you made some test changes. Did you make any changes (other than de-dents) to the code moved out into the other modules?

I noticed you made some test changes.

Just fixing the mock patching.


Channeldb: Store HTLC Extra TLVs in Onion Blob Varbytes

Opening this in draft to illustrate the scope of a possible solution for #7297.
This PR demonstrates a workaround for persisting TLV data that is transmitted in update_add_hltc:

HTLCs are serialized inline (so we can't easily add on another field, because we wouldn't know when it is / isn't there easily).
The OnionBlob for HTLCs is serialized as var bytes, but we know this will always be 1366 bytes (as the spec demands).
We can use this variable encoding to squeeze in our TLVs as follows:

Write OnionBlob + TLVStream as a single var bytes blob where we previously had just OnionBlob
This will write a lengh prefix of 1366 + len(TLV stream)
To deserialize, read the full var bytes blob, copy the first 1366 into the OnionBlob and treat the rest as our TLV Stream

When we want to save extra data (as provided in our PaymentDescriptor, all we have to do is encode the TLVs and set channeldb.HTLC.ExtraData (see last commit for example of where we'd need to set this).

Happy to see you came around to this approach! May not be the ideal path, but it gets the job done and also lets us avoid migrations in this area as well.

Took me a minute to grok, ngl, but seems like the lesser evil (by far)!

Before proceeding we should still do a manual comb through and ensure there're no other areas where an exact length is used (not serialized as var bytes).

Will post separately! WDYT about this PR going in as-is vs combining in a larger preparatory route blinding PR? I lean towards to former, keeping DB workaround PRs more isolated but happy with either approach.

References to HTLC.OnionBlob
channeldb/channel.go
Serialized/Deserialized with updated functions - always stored as varbytes.

File
Line
Reference

channeldb/channel.go
2004
onionHash := sha256.Sum256(htlc.OnionBlob[:])

channeldb/channel.go
2012
onionHash := sha256.Sum256(htlc.OnionBlob[:])

channeldb/channel.go
2104
copy(onionAndExtraData, htlc.OnionBlob[:])

channeldb/channel.go
2162
htlcs[i].OnionBlob[:]

contractcourt
decodePayload reads the HTLC's fixed size blob - these HTLCs are serialized and deserialized with our updated logic as part of commitSet.
Legacy nodes used LogChainActions/FetchChainActions which also serializes and deserializes using var bytes.

File
Line
Reference

contractcourt/htlc_incoming_contest_resolver.go
490
onionReader := bytes.NewReader(h.htlc.OnionBlob[:])

lnwallet/channel.go
HTLCs are stored as var bytes in ChannelCommitment and re-loaded into memory.

File
Line
Reference

lnwallet/channel.go
748
copy(h.OnionBlob[:], htlc.OnionBlob)

lnwallet/channel.go
772
copy(h.OnionBlob[:], htlc.OnionBlob)

lnwallet/channel.go
860
OnionBlob:          htlc.OnionBlob[:]

Latest push just addresses nits and adds some comments, no functionality changes.

@roasbeef: review reminder


Requiring Java version 1.5 to be used for our APIs.

Demo is exempt from this requirement. animal-sniffer plugin used for this.
See https://github.com/googlei18n/libphonenumber/issues/181

I'm trying to fix the issue with the jars... that is a separate build issue with nanoproto that I am having.

In general looks good, but I want to patch it in and do some release dry runs once the build is resolved, on my machine - to make sure you didn't do local setup to get some minor bug to work - which I did once :)

Yep, sounds good.
Note I changed something in phonenumberutil to use isEmpty and it fails as
expected (that is not allowed in 1.5).
Also if I enable it on the demo it fails as expected (we aren't staying
v1.5 there).
2016-04-22 14:23 GMT+02:00 Keghani Kouzoujian<EMAIL_ADDRESS>
In general looks good, but I want to patch it in and do some release dry
runs once the build is resolved, on my machine - to make sure you didn't do
local setup to get some minor bug to work - which I did once :)
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/googlei18n/libphonenumber/pull/1088#issuecomment-213402339

I'll take a look next week given upcoming holidays and deadlines.

Actually why include the jar files? They're not generated with these poms.

I included the jar files because I ran mvn package (to test everything runs okay and tests pass) and then they had a diff so...
But we can revert. We can do a separate update of them at some point if we want to.

Yeah I got that diff too when doing the Maven hierarchy fix (https://github.com/googlei18n/libphonenumber/pull/1160), it's the new getExampleNumber API and isNumberGeographical. Let's do them separately (even separately from the Maven hierarchy fix) after this.

Also should we add this to the pending changes? It doesn't affect the code, it's just a test for us, right?

Yes, you're right, so I think we don't need to.
2016-06-21 11:43 GMT+02:00 Keghani Kouzoujian<EMAIL_ADDRESS>
Also should we add this to the pending changes? It doesn't affect the
code, it's just a test for us, right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/libphonenumber/pull/1088#issuecomment-227391463,
or mute the thread
https://github.com/notifications/unsubscribe/AIjis-ckrQ1raDfotT64QPn6v-o2vkWZks5qN7IxgaJpZM4INizF
.

I ran a test in another branch (https://github.com/googlei18n/libphonenumber/commit/69ada95b369ce601e2602359c26765b04d833cba). Running mvn package from java/ results in no changes to tracked files (and none to Phonemetadata.java).
Can you try mvn package in a clean setup from java/? We should not be changing Phonemetadata.java, I wonder if this will come up for you every time.

Tried it, it only happens when I do it from head, not from java/, but it
happens in a clean set-up too.
I'm using libprotoc 2.6.1 - what are you using? (protoc --version)
2016-06-21 15:10 GMT+02:00 Keghani Kouzoujian<EMAIL_ADDRESS>
I ran a test in another branch (69ada95
https://github.com/googlei18n/libphonenumber/commit/69ada95b369ce601e2602359c26765b04d833cba).
Running mvn package from java/ results in no changes to tracked files
(and none to Phonemetadata.java).
Can you try mvn package in a clean setup from java/? We should not be
changing Phonemetadata.java, I wonder if this will come up for you every
time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlei18n/libphonenumber/pull/1088#issuecomment-227434278,
or mute the thread
https://github.com/notifications/unsubscribe/AIjis7Ra64nl0oCtREgRHAECL7nE7Jo7ks5qN-KvgaJpZM4INizF
.

Running mvn package from root with these pom.xml file changes also gives me no changes in Phonemetadata.java (only in the checked-in jar files and some untracked files, as expected).
protoc --version gives libprotoc 2.6.1, but this may be different than your 2.6.1 (I think installed from sources).
which protoc gives /usr/bin/protoc
What do you have under ~/.m2/repository/com/google/protobuf/protobuf-java/? I have just one directory, 2.6.1.

Can we limit the scope here to the 4 pom.xml files? The changes there by themselves don't cause the changes in the jars or Phonemetadata.java. We can investigate that in https://github.com/googlei18n/libphonenumber/pull/1172 and https://github.com/googlei18n/libphonenumber/pull/1170.
The pom.xml files by themselves look good for merging.

Right. Reverted them :)

Thanks, LG :)


Samples with response_time None crashes stats.py

Certain async protocols/clients (like websockets, MQ etc) dont really have a response time. They should be able to report a None response time. Using zero is not good enough, because a test may contain a mix of synchronous requests (like HTTP) and async, and we dont want to "taint" the average response time with zeroes.
from locust import HttpLocust, task, TaskSet
from locust.events import request_success

class UserBehavior(TaskSet):
    @task
    def my_task(self):
        request_success.fire(request_type="foo", name="bar", response_time=None, response_length=0)

class MyHttpLocust(HttpLocust):
    task_set = UserBehavior

Result:
[2019-09-22 09:57:13,877] lafp-mac-JG5J.int.svenskaspel.se/ERROR/stderr: Traceback (most recent call last):
  File "/Users/lafp/git/locust/locust/core.py", line 361, in run
    self.execute_next_task()
  File "/Users/lafp/git/locust/locust/core.py", line 387, in execute_next_task
    self.execute_task(task["callable"], *task["args"], **task["kwargs"])
  File "/Users/lafp/git/locust/locust/core.py", line 399, in execute_task
    task(self, *args, **kwargs)
  File "/Users/lafp/git/locust-plugins/examples/foo.py", line 10, in my_task
    request_success.fire(request_type="foo", name="bar", response_time=None, response_length=0)
  File "/Users/lafp/git/locust/locust/events.py", line 34, in fire
    handler(**kwargs)
  File "/Users/lafp/git/locust/locust/stats.py", line 559, in on_request_success
    global_stats.log_request(request_type, name, response_time, response_length)
  File "/Users/lafp/git/locust/locust/stats.py", line 93, in log_request
    self.total.log(response_time, content_length)
  File "/Users/lafp/git/locust/locust/stats.py", line 239, in log
    self._log_response_time(response_time)
  File "/Users/lafp/git/locust/locust/stats.py", line 250, in _log_response_time
    self.total_response_time += response_time
TypeError: unsupported operand type(s) for +=: 'int' and 'NoneType'

This PR should have tests. I'm a little bit concerned that these changes could have unintended effects since the code up until now have always assumed that each request has a response time. That might be an unfounded concern though, but in any case, I think these changes should come with multiple tests where non-response-time requests are made both by themselves, as well as together with "normal" requests that have a response time.

I'll try to add add tests for None response mixed with integer response times

Cool! Hopefully that should be able to catch any unintended consequences if there are any (I've been looking a bit more at the code, and couldn't see any immediate problems with the changes).

When I added the tests I immediately found some issues :) (related to average and median calculation)
I am trying to fix them...

I've fixed the issues with calculating averages and median.


Update golangci-lint.yml

fixes https://github.com/golang-cafe/job-board/issues/69

@RANJITHROSAN17 thanks for the PR - but that doesnt seem to have solved the issue


Updated composer.json

Removed Intervention\Image ServiceProvider and Alias as that package already has Laravel auto package discovery enabled.

Thank you @jnbn


tldr for own custom scripts?

Hey again, how should one go about creating a custom .md file that just doesn't qualify for the public repo?
Say I have a custom shell script I wrote and use twice a month (and I never remember the parameters), tldr would be the perfect tool to just remind myself of the script's usage.
Can I add my own pages, currently? Won't they get deleted from the cache?
Edit: So it turns out adding my own .md under /common doesn't work, possibly because tldr looks up the online repo and decides the command doesn't exist.

Yeah, tldr is looking in the index.json file first for the page. I've just pushed an update to look for the page in $configdir/local/ first. The commit message contains details:  https://github.com/raylee/tldr/commit/92e0545048b9bb702aca62b7df30fe299defb5c9

If the new version doesn't work for you, please re-open this issue. Thanks!


Need BOM

I am really interested in this project and have just ordered a set of boards so that I can build 1 for myself. I would have preferred to order the boards populated but I can order the parts separately and solder the surface mount components. A BOM would help with ordering the boards. Also I was having a hard time finding the actual components on the JLCPCB site so I decided to just get the boards and then order the parts later if I didn't have the resistors and capacitors already.
I was also wanting to reach out and see what I can do to contribute to this project. I have recently gotten back into retro computing and have obtained both an actual C64 and C128 that I have been doing some assembly development on. I had the idea earlier to see about using a Raspberry Pi to interface to a C64 and started to do some research into what existing projects there were and ran across your use of the Pico. I have done some development with the Pico and it is a brilliant solution to the interface that I am thinking about.
Also, I saw the 2 Pico solution that would allow for more capabilities, and I am interested in that also. I also saw another project where a Raspberry Pi was used to generate HDMI directly from a cartridge port.

Would something like this work? picocart.csv
KiCad has a lot of BOM output generators, this one is from bom_csv_grouped_by_value.py. If that looks useful, I'll add it to the repo.
I'll be interested to see if it works on the C128. Are they PAL or NTSC machines? As far as I know, this has only been tested on NTSC (though I can't imagine PAL would cause a problem).

The BOM is a start I will see if I can get exact part numbers and specifications. I think I was trying to find the exact parts @ JLCPCB so they could pre-populate the board. I am new to ordering the boards prepopulated so wanted I make sure I got everything exactly right. If you are thinking of moving forward with the 2 Pico solution, I would be interested in helping with the development of that and I can help by paying for and ordering the boards and testing the boards. This solution would allow for some great solutions with the available add-ons to the Pico that are available: LCD display, HDMI output etc... I could see a lot of different ideas going out of this. I am also involved with the Foenix system and have a Mega65 on order so 6502, 4510 and the 65816 processors so I am improving my assembly skills.
I will let you know when I get the blank boards and start populating them...
The C128 is NTSC (rasterOverscan = $06) and the C64 is PAL (rasterOverscan is $37). I found some assembly that would determine the machine type through code by getting the rasterOverscan. I made this into a routine so I could the timing so that I could use the jiffyTimer to time executions:
FUNCTION GetRasterOverflow() : byte;
BEGIN
ASM
SEI
System_GetRasterOverflow_WaitBMI:
LDA $D011
BMI System_GetRasterOverflow_WaitBMI
System_GetRasterOverflow_WaitBPL:
LDA $D011
BPL System_GetRasterOverflow_WaitBPL
System_GetRasterOverflow_WaitNoBPL:
LDA $D012
BIT $D011
BPL System_GetRasterOverflow_WaitDone
STA System_rasterOverflow
BMI System_GetRasterOverflow_WaitNoBPL
System_GetRasterOverflow_WaitDone:
CLI
END;
RETURNVALUE(rasterOverflow);

END;

For the ICs, the exact model doesn't matter so much. I used TI SN74LV244APWRG3, SN74LV244APWRG4, SN74LV244ATPWR; as well as Nexperia 74LV244APWJ when I was prototyping these.
I do plan on spending some time on the 2 pico idea at some point soon. I have a new design I'll put in a new repo once I can do some testing.


python312Packages.taxii2-client: init at 2.3.0

TAXII 2 client library
https://github.com/oasis-open/cti-taxii-client/
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

20 packages built:

python311Packages.cybox
python311Packages.cybox.dist
python311Packages.maec
python311Packages.maec.dist
python311Packages.mixbox
python311Packages.mixbox.dist
python311Packages.stix2-validator
python311Packages.stix2-validator.dist
python311Packages.taxii2-client
python311Packages.taxii2-client.dist
python312Packages.cybox
python312Packages.cybox.dist
python312Packages.maec
python312Packages.maec.dist
python312Packages.mixbox
python312Packages.mixbox.dist
python312Packages.stix2-validator
python312Packages.stix2-validator.dist
python312Packages.taxii2-client
python312Packages.taxii2-client.dist


sorbet/files.yaml: set utils/tty.rb to true

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

This PR moves utils/tty.rb to true so that type errors on this file may be reported in the future.

Thanks @vidusheeamoli!


Add feature to exclude ip from flow

As a user of this agent, it can overwhelm the flow pipeline (for k8s enriching) on very large clusters. It would be optimal, if I could add ip filtering in the netobserv-ebpf-agent so that I can pass only interesting flow to downstream flow pipline.
As a very hacky example https://github.com/netobserv/netobserv-ebpf-agent/commit/c55b0acbb0238b2f21beba022381ea87b9014f31

Hi @hihellobolke thanks for reaching out we have plans to filter at the low level in ebpf per direction so only matching flows will be stored in our tables will that work for your usecase ?

Yes @msherif1234 that will work well!


Item Outputs Not Increased + Laggy Machine Processing

Latest 1.20.1 Forge.
So far it doesn't seem that outputs are higher. With some combs I still see minimum outputs when giving an output, & I never see values that seem to match up to the related centrifuge. On a single hive system with a Nuclear centrifuge for an example, I'm observing Experience Comb Blocks only outputting 400mb at a time, which is the default value.
Processing times are also not that fast, much less multiple times faster than the heated centrifuge. It almost looks like the system is laggy trying to get up to speed. Which isn't necessarily bad presently, since even Mekanism's ultimate pipes + transporters can't keep up with the output load & even struggle with the input a bit.
Still quite useful when you have dozens of hives producing enormous amounts of material, but the expense is wild when the main features aren't fully functional.
In later addendum now, Spark profiler also shows severe lag issues with this mod that seems to stem from checking for valid items.

1.20.1 is not supported anymore


After adding tsconfig.json, .ts doestn't compile anymore [VS 2015.3]

Hi,
I'm using latest version of TypeScript, VS 2015.3 & Windows 10 x64.
When I add tsconfig.json to my project (web application, using web-form), .ts file doesn't compile anymore.
I added tsconfig.json in root of my project, also didn't change any options (vs add some default config to it), I moved config file to the folder that contains .ts file, but no success.
p.s: When I using command-line with tsc --p tsconfig.json command, everything works just fine.
Thank you!
-mehdi

Please share your full build log with 'MSBuild /verbosity: diagnostic' or a project that reproduces the issue.

@mhegazy I figured out you added similar answer to almost all reported bugs, I think I added enough info in my post. below you can see how you can get my project :)

Open Visual Studio 2015.3
Hit Ctrl+Shift+N
Select C# - Web - ASP.NET Web Application (.Net Framework), then form ASP.NET 4.5 Templates, select Empty
Try to add new typescript file
Add some JS code and hit Ctrl+S, you can see, the compiled JS file will show up.
Add tsconfig.jon, make some change to JS file, hit Ctrl+S, you can see, the compiled JS file won't update.

If you can get compiled file when tsconfig.json file is exists, please share your project with me.

Thanks for the info. Your original report did not have any mention to Compile-on-Save. you only mentioned build. To enable compile-on-save, set "compileOnSave": true  in your tsconfig.json file. Please find documentation at http://www.typescriptlang.org/docs/handbook/tsconfig-json.html#compileonsave
Given the volume of issues we get everyday, and the finite nature of time, we would appreciate it if you give us enough details to reproduce issues locally.
Here is my project as requested:
WebApplication11.zip

Thank you for your answer, seems like I reported wrong issue. with "compileOnSave": true everything works, but I need to add outDir option too. right now my tsconfig.json has following content:
{
    "compilerOptions": {
        "noImplicitAny": false,
       "noEmitOnError": true,
       "removeComments": false,
       "sourceMap": true,
       "target": "es5",
       "outDir": "../js/modules/"
    },
    "compileOnSave": true,
    "exclude": [
        "node_modules",
        "wwwroot"
    ]
}

If I remove "outDir": "../js/modules/" everything works.
BTW as I said when I using command-line with tsc --p tsconfig.json including "outDir": "../js/modules/" command, everything works just fine.

not sure i follow the issue now. so with outDir there is no output generated? or is it generated in the wrong place?

@mhegazy there is no output generated, but (as I mentioned) using command-line files will be generated successfully.

does setting "rootDir": "./" helps?

Works like a charm ! Why I didn't that before !!
Thank you!


formtastic 3 and bundle install

lets test having formtastic-bootstrap < 3 (for bootstrap 2) and formtastic 3
PR from f86eaef93bea0a06879b3977d7554864964a623f (v2.1.3)
just relaxing formtastic dependency
there seems not to have breaking changes in formtastic API
https://github.com/formtastic/formtastic/blob/3.0-stable/CHANGELOG
lets hope

autre piste potentielle : https://github.com/r-spell/boomtastic


Ci/npm schema publish

Publish the GraphQL schema as an npm package on Github packages & npmjs under the @okp4 scope.
The publication is currently equivalent for both targeted registries:

Publish latest tag on new git tag version.
Publish next tag on main push with a next.$DATE version preid.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


update using core-store

will have a test together with an updated app

Thanks, @sevenEng this looks good, I will give it a test and merge it when we move to v0.3.0.


Uncaught Error: Cannot find module<EMAIL_ADDRESS>

[Enter steps to reproduce:]

Updated Hydrogen to 2.14.3. Received this error.
Uninstalled and reinstalled Hydrogen. Same issue.
Went to<EMAIL_ADDRESS>as guided from the error flag. There was know 'lib' folder, therefore no index.js file.

Atom: 1.48.0 x64
Electron: 5.0.13
OS: Mac OS X 10.15.5
Thrown From: Hydrogen package 2.14.3
Stack Trace
Uncaught Error: Cannot find module<EMAIL_ADDRESS>Please verify that the package.json has a valid "main" entry
At internal/modules/cjs/loader.js:248

Error: Cannot find module<EMAIL_ADDRESS>Please verify that the package.json has a valid "main" entry
    at tryPackage (internal/modules/cjs/loader.js:240:19)
    at Function.Module._findPath (internal/modules/cjs/loader.js:378:18)
    at Module._resolveFilename (internal/modules/cjs/loader.js:670:27)
    at Module._resolveFilename (/Applications/Atom.app/Contents/Resources/electron.asar/common/reset-search-paths.js:43:12)
    at Function.get_Module._resolveFilename (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:155487)
    at Module.require (/app.asar/static/index.js:61:43)
    at require (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:146819)
    at /packages/Hydrogen/node_modules/@nteract/outputs/lib/components/media/markdown.js:13:36)
    at /packages/Hydrogen/node_modules/@nteract/outputs/lib/components/media/markdown.js:27:3)
    at Module.get_Module._compile (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:147503)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:151051)
    at Module.load (internal/modules/cjs/loader.js:701:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:633:12)
    at Function.Module._load (internal/modules/cjs/loader.js:625:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:146819)
    at /packages/Hydrogen/node_modules/@nteract/outputs/lib/components/media/index.js:11:18)
    at /packages/Hydrogen/node_modules/@nteract/outputs/lib/components/media/index.js:20:3)
    at Module.get_Module._compile (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:147503)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:151051)
    at Module.load (internal/modules/cjs/loader.js:701:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:633:12)
    at Function.Module._load (internal/modules/cjs/loader.js:625:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:146819)
    at /packages/Hydrogen/node_modules/@nteract/outputs/lib/index.js:24:28)
    at /packages/Hydrogen/node_modules/@nteract/outputs/lib/index.js:27:3)
    at Module.get_Module._compile (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:147503)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:151051)
    at Module.load (internal/modules/cjs/loader.js:701:32)

Commands
     -4:15.3.0 hydrogen:run (input.hidden-input)

Non-Core Packages
atom-material-syntax 1.0.8
atom-material-ui 2.1.3
autocomplete-python 1.16.0
dark-python-syntax 2.3.0
file-icons 2.1.41
Hydrogen 2.14.3
minimap 4.29.9

I had the same problem just now. Updated Hydrogen. Uninstalled and reinstalled. Went to look at the indicated directory and there was no lib folder.

Installing previous versions of hydrogen doesn't help :((

Same here. The lib folder is missing. My setup is like rocketbarn but running Windows 10 x64.

try the latest

@aviatesk Thank you, it is working perfectly now.


add _.after

You're just reimplementing lib at this point which kind of strays from the point of this project.

what do you mean

You're simply re-implementing an after method with Array#forEach. It starts to drift from the message of not needing dependencies if you're re-implementing the methods in a slightly different way.

I don't get you. I just explain how to not using _.after

Lodash is just JavaScript so yes you can implement it's methods in JavaScript. The point of this project is to show built-in equivalents not implementations of Lodash methods written with a dash more ES5/ES6 thrown in.

I just show the native implementation rather than use lodash/underscore. _.after it's not native

hi @infacq,  thank you for your PR. Yes, @jdalton  is right about this list. We are not going to implementing functions at the moment. But I've created a polls to check if is good idea to have it. So let's just temporarily hold this PR and wait for others' voice. Cheers.

look forward the polls. I still believe to use native rather than load library file in a webpage

@infacq  not so many people vote  though.  ( :sweat: )

look forward the polls. I still believe to use native rather than load library file in a webpage


MAINT: Drop jQuery and use Bootstrap 5

Fix #953, Fix #764
for bootstrap I've followed the instructions from: https://getbootstrap.com/docs/5.0/migration/
which lead to the following changes:

include bootstrap 5.1 to packages
update to popperjs/core (popper v2)
change all occurences of left (res l) by start(res. s) and right (res. r) by end (res. e)
check all the brekpoints as they changed how they work
update all toggle like data-* by data-bs-*

and for jquery: https://tobiasahlin.com/blog/move-from-jquery-to-vanilla-javascript/#network-requests-with-get-or-ajax
which lead to the following changes:

create a function to mimic jquery .ready()
replace $.ajax, $.getJSONand $.get by the generic fetch
replace jquery selectors and manipulators ($().each etc...)
drop jquery everywhere

It's a sensitive change as I'm modifying pretty much every js of theme so I tagged most of the regular maintainers. Let me know what you think and your suggestions. Also note that the package-lock.json ended up being a lot modified, if you know a way to cherry-pick the changes I'm happy to change it.

do you know a way to edit package-lock.json but only the edited dependencies?

I think with #868 it worked that way because FontAwesome doesn't have any dependencies. AFAIK npm update package_name will always update that package's dependencies too; there is no equivalent to pip install --no-deps package_name (or if there is, I don't know about it)

I took a pass through the preview docs, and they seem to look good to me. Scrolling behavior works and things seem to be formatted properly. Anything else that @12rambau wants to do here or are you ready for final review + merge? My feel is we should:

cut a release before merging this one
include an announcement banner saying that we updated to Bootstrap 5 and removed JQuery, so downstream authors should be aware of those changes.
to do this quickly so it doesn't become out of date

cut a release before merging this one
include an announcement banner saying that we updated to Bootstrap 5 and removed JQuery, so downstream authors should be aware of those changes.
to do this quickly so it doesn't become out of date

SGTM!

what type of banner are you thinking about? Using the header banner or adding a warning admonition on the landing page ?

I was thinking header banner - what do you think? I'm happy with whatever path you think is best

I agree that loud banners would be quite useful! Though I won't have any time to finish it any time soon. I am happy to review PRs if you or others want to take it over and get it over the finish line.

Now that 0.12 is out should we commit this one to master ?

If you think that this is ready to go, then I suggest we merge it quickly and iterate from there. One suggestion: could you provide a short summary of the changes that you needed to make in this PR, as a "mini guide" to downstream authors who might need to make the same changes in their themes?

@choldgraf done (I'll may need some proofreading though).
I think this page should be advertised in the loud banner i.e. in a follow-up PR.


[Question]: I want to pull the new version, how to ensure that the current data is not lost.

Describe your problem
First of all, thank you very much. I love your project and have followed it since the beginning. I see that the new version has been updated a lot, I want to try, but I already have a lot of local data, can I directly pull the new project to retain the original data? Or it's already mounted in a local directory

Hello, here some recommendations:

First, make a backup copy of your current ragflow setup including your data
I recommend that you use the latest release v0.9.0 it is more stable that current dev version (i.e. in current dev version using Cohere break the Chat pages for all Knowledge Base that use Cohere LLMs)
Edit your /docker/.env file and change the line from RAGFLOW_VERSION=dev to RAGFLOW_VERSION=v0.9.0
Restart ragflow and  the new docker image for v0.9.0 will be pull, run and connect to your current Docker volumes
If roll back is needed then change back to your previous value RAGFLOW_VERSION and restart ragflow
If the update works then delete previos ragflow image to save almost 19GB
I've tested this procedure myself and it works.
Hope it helps.

Theoretically, you're not gona loss data by upgrade.
In term of data security, just backup the docker volumes.


Query parameters

First of all, thanks for your addon.
I'm using your addon for a few days, and I have a scenario where I pass some query parameters based on select options. So depending on what option is selected I'll pass a different parameter to the query string. That way I'll need to set many different items in the mockData to the same API url.
I believe that the addon should ignore the query string passed. Or at least have an option to not take the query string passed into account.

@carloschneider there are some cases where we need to consider the query parameter to distinguish between two separate calls. If we ignore the query string, it will lose the functionality of a specific URL.
In your case, one possible solution would be what you mentioned. You can create multiple mockData for multiple query parameters. It's also good to separate the concern.
Any other solution would be welcome  :)

@nutboltu can there be a solution similar to how cypress handles fixtures? where the url can accept glob patterns at the end, and/or regex to determine a match in the url?
just because some may want different mock data per query param doesn't mean all do.

An alternative is add a parameter like: queryString: false as the same way that you have the delay parameter for each request.

@jfarris587 I might need to look at how cypress handles such a pattern. But any PR is welcome 😃
@carloschneider queryString: false might be an alternative but if we can handle it using regex pattern in the URL that would be more standard way.

@nutboltu Sorry, I've take a look into the code, and you already check the parameters excluding values, just the keys. I'll close this issue for now. Thank you.


genewise: only show variants that meet the require number of optional…

… filters
reported by @jxchong

Ah ok. Seems like this might have reverted the previous fix (that wouldn't output variants that didn't contribute to passing the gene level --min-filters).  I now get variants at 14365 and 14582 even though both are marked as only passing the required filters.
chrom	start	end	ref	alt	gene	impact	vep_feature_type	filter	gts	variant_filters	n_gene_variants	gene_filters
chrMT	14317	14318	T	C	MT-ND6	missense_variant	Transcript	None	T/T,T/T,T/C,./.,T/T,T/T,./.,./.,T/C,T/T,T/T	required[1],required[2],required[3],1	4	1,2
chrMT	14364	14365	C	T	MT-ND6	synonymous_variant	Transcript	None	C/C,C/C,C/C,C/C,C/C,C/C,C/C,T/T,C/C,C/C,C/C	required[1],required[2],required[3]	4	1,2
chrMT	14568	14569	G	A	MT-ND6	synonymous_variant	Transcript	SnpCluster	G/G,A/A,A/A,G/G,A/A,G/G,G/G,G/G,A/A,A/A,G/G	required[1],required[2],required[3],2	4	1,2
chrMT	14581	14582	A	G	MT-ND6	missense_variant	Transcript	SnpCluster	A/A,A/A,A/A,A/A,A/A,A/A,A/A,G/G,A/A,A/A,A/A	required[1],required[2],required[3]	4	1,2

hmm.  you sure you have the latest with that? can you show the query you were running?

Yes I'm running commit 6fa548e
$ gemini gene_wise \
> --columns "chrom,start,end,ref,alt,gene,impact,vep_feature_type,filter,gts,gt_quals,gt_depths" \
> --where "chrom='chrMT'" \
> --gt-filter-required "(gt_depths).(phenotype==2).(>=6).(count >= 1) and (gt_quals).(phenotype==2).(>=20).(count >= 1)" \
> --gt-filter-required "(gt_quals).(=HET).(>=20).(any) or (gt_quals).(=HOM_ALT).(>=20).(any)" \
> --gt-filter-required "(gt_depths).(=HET).(>=6).(any) or (gt_quals).(=HOM_ALT).(>=6).(any)" \
> --gt-filter "gt_types.151854 == HET" \
> --gt-filter "gt_types.171139 == HOM_ALT" \
> --min-filters 2 \
> MT.VT.VEP.db
chrom	start	end	ref	alt	gene	impact	vep_feature_type	filter	gts	variant_filters	n_gene_variants	gene_filters
chrMT	14317	14318	T	C	MT-ND6	missense_variant	Transcript	None	T/T,T/T,T/C,./.,T/T,T/T,./.,./.,T/C,T/T,T/T	required[1],required[2],required[3],1	4	1,2
chrMT	14364	14365	C	T	MT-ND6	synonymous_variant	Transcript	None	C/C,C/C,C/C,C/C,C/C,C/C,C/C,T/T,C/C,C/C,C/C	required[1],required[2],required[3]	4	1,2
chrMT	14568	14569	G	A	MT-ND6	synonymous_variant	Transcript	SnpCluster	G/G,A/A,A/A,G/G,A/A,G/G,G/G,G/G,A/A,A/A,G/G	required[1],required[2],required[3],2	4	1,2
chrMT	14581	14582	A	G	MT-ND6	missense_variant	Transcript	SnpCluster	A/A,A/A,A/A,A/A,A/A,A/A,A/A,G/G,A/A,A/A,A/A	required[1],required[2],required[3]	4	1,2

can you share your db or vcf via email? the one you sent me only reports chrMT:14317 and chrMT:14568 for that query.

Sent a dropbox invite.

thank you. can you check the latest commit? sorry for the repeated bugs. I think this time it's really resolved.

Not quite right, I think. If I run --min-filters 1, then I get both variants I expect to get, but if I run --min-filters 2, I don't get any variants (but I should get both since each variant passes 1 filter, so that in total the gene MT-ND6 passes a minimum of 2 filters)
$ gemini gene_wise --columns "chrom,start,end,ref,alt,gene,impact,vep_feature_type,filter,gts,gt_quals,gt_depths" --where "chrom='chrMT'" --gt-filter-required "(gt_depths).(phenotype==2).(>=6).(count >= 1) and (gt_quals).(phenotype==2).(>=20).(count >= 1)" --gt-filter-required "(gt_quals).(=HET).(>=20).(any) or (gt_quals).(=HOM_ALT).(>=20).(any)" --gt-filter-required "(gt_depths).(=HET).(>=6).(any) or (gt_quals).(=HOM_ALT).(>=6).(any)" --gt-filter "gt_types.151854 == HET" --gt-filter "gt_types.171139 == HOM_ALT" --min-filters 1 MT.VT.VEP.db

chrom	start	end	ref	alt	gene	impact	vep_feature_type	filter	gts	variant_filters	n_gene_variants	gene_filters
chrMT	14317	14318	T	C	MT-ND6	missense_variant	Transcript	None	T/T,T/T,T/C,./.,T/T,T/T,./.,./.,T/C,T/T,T/T	required[1],required[2],required[3],1	2	1,2
chrMT	14568	14569	G	A	MT-ND6	synonymous_variant	Transcript	SnpCluster	G/G,A/A,A/A,G/G,A/A,G/G,G/G,G/G,A/A,A/A,G/G	required[1],required[2],required[3],2	2	1,2

$ gemini gene_wise --columns "chrom,start,end,ref,alt,gene,impact,vep_feature_type,filter,gts,gt_quals,gt_depths" --where "chrom='chrMT'" --gt-filter-required "(gt_depths).(phenotype==2).(>=6).(count >= 1) and (gt_quals).(phenotype==2).(>=20).(count >= 1)" --gt-filter-required "(gt_quals).(=HET).(>=20).(any) or (gt_quals).(=HOM_ALT).(>=20).(any)" --gt-filter-required "(gt_depths).(=HET).(>=6).(any) or (gt_quals).(=HOM_ALT).(>=6).(any)" --gt-filter "gt_types.151854 == HET" --gt-filter "gt_types.171139 == HOM_ALT" --min-filters 2 MT.VT.VEP.db
$ [no results]

could you have a look at the latest commit? If that doesn't fix it, then I'll have to look next week.

Ok I think that did it!
I now get:
chrom	start	end	ref	alt	gene	impact	vep_feature_type	filter	gts	variant_filters	n_gene_variants	gene_filters
chrMT	14317	14318	T	C	MT-ND6	missense_variant	Transcript	None	T/T,T/T,T/C,./.,T/T,T/T,./.,./.,T/C,T/T,T/T	required[1],required[2],required[3],1	2	1,2
chrMT	14568	14569	G	A	MT-ND6	synonymous_variant	Transcript	SnpCluster	G/G,A/A,A/A,G/G,A/A,G/G,G/G,G/G,A/A,A/A,G/G	required[1],required[2],required[3],2	2	1,2

I'll try to do some more testing.

So far so good.

The one thing I'm noticing is that I cannot access per-sample gts, gt_depths, or gt_quals. I can print out all of the samples in the database if I use gts,gt_depths,gt_quals, but I cannot access gt_depths.sample12345 or gts.sample12345, etc. sqlalchemy.exc.NoSuchColumnError: "Could not locate column in row for column 'gts.151854'"
I think this isn't a bug, just a potentially useful/missing feature.

added


PSS signatures: Could not find 'sha2' in rsa

When using the code below, the error "could not find 'sha2' in rsa" occurs.
use rsa::RsaPrivateKey;
use rsa::pss::{BlindedSigningKey, VerifyingKey};
use rsa::signature::{Keypair,RandomizedSigner, SignatureEncoding, Verifier};
use rsa::sha2::{Digest, Sha256};
let mut rng = rand::thread_rng();
let bits = 2048;
let private_key = RsaPrivateKey::new(&mut rng, bits).expect("failed to generate a key");
let signing_key = BlindedSigningKey::::new(private_key);
let verifying_key = signing_key.verifying_key();
// Sign
let data = b"hello world";
let signature = signing_key.sign_with_rng(&mut rng, data);
assert_ne!(signature.to_bytes().as_ref(), data);
// Verify
verifying_key.verify(data, &signature).expect("failed to verify");

As noted in the docs:
https://docs.rs/rsa/latest/rsa/#pss-signatures

Note: requires sha2 feature of rsa crate is enabled.

The next release will eliminate the need to enable a feature.


Support updating lifecycle scripts on existing containers

Because the container configuration is cached in the container labels after creation, any changes to the local config file are ignored for future operations on the container.
In the case of the lifecycle scripts (onCreateCommand, postCreateCommand, ...) it would be useful in cases of pooling container if changes to the config would be honored so that the container doesn't need to be recreated.
For example, if a container is created with an empty configuration for a pool, then we should be able to later change the configuration to and re-apply the post-creation lifecycle logic against the existing container.
{
    "postCreateCommand": "touch foo"
}

It would also be useful if the cli could detect the case of a configuration collision and fail with a reasonable message. For example, if the container is created with a postCreateCommand already and then the value is changed, then this re-application logic should fail.

For example, if a container is created with an empty configuration for a pool

Important to call out here that there's never a container created via an "empty configuration" (that's more a Codespace user-concept, but behind the scenes there's always a config)
+1 I think this is a nice optimization, either automatically or via a new CLI command (like devcontainers update)?)

To make the pooling work, we could add a command line option (maybe --reapply-config?) that would make the code at https://github.com/devcontainers/cli/blob/be1f2034e10395f42843ebf26600a42e11e66531/src/spec-node/imageMetadata.ts#L427 take the else case.
That would add the config on top of the existing metadata on the container (which is readonly). We also use this codepath when we locally attach to a container that was started by the user using the Docker CLI and the image comes with dev container metadata. The merging is the same codepath as when you regularly start a dev container.


Multiple roots

It stops working when I have two roots (two separate apps) running on the same page. The left sidebar might show the right information (multiple roots) but clicking on any of the roots or its children (if shown) will not update the data on the right sidebar.

We seem to have this issue, too.

@yyx990803 Any news on this? Can we support this, or is it just not an option in the current devtools setup?

Ping @simplesmiler ?

Not sure if this is the right issue for this, but I got a minor issue with multiple vue roots on my page.
I built a small modal system that attaches itself as a new vue instance at the end of the body. The devtools only seem to pick it up once I hit its "Refresh" button.
Just something minor, but would be great if there's a way to pick this up automagically.


chore: export core/client type

Description:
export core/client type
BREAKING CHANGE:
Related issue (if exists):

thanks!


Swap out white with a more grey approach

Swap out the white color, in order for the theme to be viable in dark enviourments?

Perhaps maybe just a light / darker version of blue?

Check :white_check_mark:


pull request for e2e test at 12/07/2016 00:56:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Permission section could not be loaded

Hi, I was pretty excited to learn about your extension. However, when I try to install it on our on-premises Azure DevOps 2019.Update 1 (17.153.29207.5) server, I get this error message; "We've encountered an error while downloading the extension. Please try again later." on the side panel I see: "Permission section could not be loaded", which I suspect is the root issue. (screenshot attached)
I've confirmed that I can install other extensions and I can pull down the VSIX directly (without going through the Azure DevOps extension install pages). Unfortunately, with 2019, I no longer have the option to upload an extension, so I'm not able to get past this. I'd appreciate any suggestions.

I'm guessing this is the issue: https://github.com/dependency-check/azuredevops/issues/18

That's embarrassing. I read that one, but thought it was unrelated. I completely missed that the symptoms were exactly the same. The registry fix did the trick.

Excellent! Glad that did the trick! :)


Bcgov issue

Description
Changes proposed in this pull request.
Test plan
Outline the test plan used to test this change before merging it.

Fixes #.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "3.14-bcgov",
    "component": "trireme-lib",
    "pr-id": "1033",
    "commit-sha": "9349f5551b3c8729e018e86b98503a66f72f7d2f",
    "pipeline": "release-3.14.0"
  },
  {
    "project": "3.14-bcgov",
    "component": "enforcerd",
    "pr-id": "1939",
    "commit-sha": "455773dd5b9f2ed802dcc6ac0e281ccde2a364af",
    "pipeline": "release-3.14.0"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "3.14-bcgov",
    "component": "trireme-lib",
    "pr-id": "1033",
    "commit-sha": "9349f5551b3c8729e018e86b98503a66f72f7d2f",
    "pipeline": "master"
  },
  {
    "project": "3.14-bcgov",
    "component": "enforcerd",
    "pr-id": "1939",
    "commit-sha": "455773dd5b9f2ed802dcc6ac0e281ccde2a364af",
    "pipeline": "master"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "3.14-bcgov",
    "component": "trireme-lib",
    "pr-id": "1033",
    "commit-sha": "9349f5551b3c8729e018e86b98503a66f72f7d2f",
    "pipeline": "master"
  },
  {
    "project": "3.14-bcgov",
    "component": "enforcerd",
    "pr-id": "1939",
    "commit-sha": "455773dd5b9f2ed802dcc6ac0e281ccde2a364af",
    "pipeline": "master"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "3.14-bcgov",
    "component": "trireme-lib",
    "pr-id": "1033",
    "commit-sha": "9349f5551b3c8729e018e86b98503a66f72f7d2f",
    "pipeline": "release-3.14.0"
  },
  {
    "project": "3.14-bcgov",
    "component": "enforcerd",
    "pr-id": "1939",
    "commit-sha": "455773dd5b9f2ed802dcc6ac0e281ccde2a364af",
    "pipeline": "release-3.14.0"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "3.14-bcgov",
    "component": "trireme-lib",
    "pr-id": "1033",
    "commit-sha": "9349f5551b3c8729e018e86b98503a66f72f7d2f",
    "pipeline": "release-3.14.0"
  },
  {
    "project": "3.14-bcgov",
    "component": "enforcerd",
    "pr-id": "1939",
    "commit-sha": "455773dd5b9f2ed802dcc6ac0e281ccde2a364af",
    "pipeline": "release-3.14.0"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "3.14-bcgov",
    "component": "trireme-lib",
    "pr-id": "1033",
    "commit-sha": "9349f5551b3c8729e018e86b98503a66f72f7d2f",
    "pipeline": "release-3.14.0"
  },
  {
    "project": "3.14-bcgov",
    "component": "enforcerd",
    "pr-id": "1939",
    "commit-sha": "455773dd5b9f2ed802dcc6ac0e281ccde2a364af",
    "pipeline": "release-3.14.0"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "3.14-bcgov",
    "component": "enforcerd",
    "pr-id": "1939",
    "commit-sha": "455773dd5b9f2ed802dcc6ac0e281ccde2a364af",
    "pipeline": "release-3.14.0"
  },
  {
    "project": "3.14-bcgov",
    "component": "trireme-lib",
    "pr-id": "1033",
    "commit-sha": "9349f5551b3c8729e018e86b98503a66f72f7d2f",
    "pipeline": "release-3.14.0"
  }
]


azure-ai-formrecognizer: ModuleNotFoundError: No module named 'azure.core.polling.base_polling'

Package Name: azure-ai-formrecognizer
Package Version: azure-ai-formrecognizer>=3.0.0, azure-core==1.26.4
Operating System: Linux
Python Version: 3.10

Describe the bug
Traceback (most recent call last):
  File "qa_pipeline.py", line 5, in <module>
    import azure.ai.formrecognizer
  File "$HOME/.local/lib/python3.8/site-packages/azure/ai/formrecognizer/__init__.py", line 8, in <module>
    from ._form_recognizer_client import FormRecognizerClient
  File "$HOME/.local/lib/python3.8/site-packages/azure/ai/formrecognizer/_form_recognizer_client.py", line 18, in <module>
    from azure.core.polling.base_polling import LROBasePolling
ModuleNotFoundError: No module named 'azure.core.polling.base_polling'

To Reproduce
Install azure-ai-formrecognizer
import azure.ai

Ok, so I tried uninstalling azure-core and azure-ai-formrecognizer. Now I get
ImportError: cannot import name 'AzureKeyCredential' from 'azure.core.credentials' (/usr/lib/python3/dist-packages/azure/core/credentials.py)

Thank you for the feedback @lambdaofgod. Ill be assigning this to my colleague who can better assist you.
In the meantime, I tried installing this on a WSL instance with py 3.10
python3.10 -m venv formrecog
source formrecog/bin/activate
pip install azure-ai-formrecognizer==3.3.0b1

which gives me the following on pip freeze
azure-ai-formrecognizer==3.3.0b1
azure-common==1.1.28
azure-core==1.26.4
certifi==2023.5.7
charset-normalizer==3.1.0
idna==3.4
isodate==0.6.1
msrest==0.7.1
oauthlib==3.2.2
requests==2.31.0
requests-oauthlib==1.3.1
six==1.16.0
typing_extensions==4.6.2
urllib3==2.0.2

I then made a simple hello_world.py, which ran successfully
import azure.ai
print("Hello world")

Would you be able to test in a new venv please ?

I tested this in a fresh 3.10 anaconda env and it indeed works.


Implement best practices for linkerd-viz app

Coming from here.
The goal of this issue is to have the App eventually adhere to standard cloud-native best-practices. The points below are to be considered non-exhaustive. Feel free to add some that make sense and remove some that don't. If this issue does not make sense (e.g. is a duplicate or covers a deprecated app) please close it.
Expected outcome:

[x] the app has properly set PDBs (as percentages)
[x] the app can scale horizontally based on Custom Metrics (if it makes sense)
[x] the app runs with more than 1 replica by default (if it needs HA)
[x] the app does not use emptyDir volumes if not strictly necessary

We'll contribute PDB to upstream: https://github.com/linkerd/linkerd2/issues/11248

the app can scale horizontally based on Custom Metrics (if it makes sense)

It doesn't make much sense in this case, as scaling is largely dependent on cluster size / tap usage. We might need to revisit scaling in the future, depending on customer use.

the app does not use emptyDir volumes if not strictly necessary

it does require it indirectly, as all linkerd.

1.2.0 released including PDB. Upstream PR is still open, hopefully it gets merged.


Sign Transactions Offline and Broadcast Online error

In Tools created offline unsigned tx
Using offline computer in Tools created signed tx data
Back to computer in step 1 in Tools -> Push transaction pasted sign tx data then selected push
Screen became white and received the below error

Confirmed in Windows and Linux
Last signed tx data used:<PHONE_NUMBER>db702ca545dee667d8d23028da15a5322bb754df3132da53792cf9f210a10bd1000000006a47304402205df15a6176012e45d4050cf082dad0da1e61fe67db6c1755dabab236f645de1702203d047bb6b6b12e28f60083ecd1abfd7035970d9c5d9eaf444828b8895be7f6ed0121038b7822b465403063212069255ea762b6ac49318526a37a00268994f9a4fd0ec2ffffffff02808d643d000000001976a91424bbb2c41cf3a2c9c3192fb7ee4e8ba8524a437488ac85273e00000000001976a914763d7dfd7b87a788a8cf7ecda6e5ee474cc4309f88ac9ea6115c

Retest pass, closing


UnboundLocalError: local variable 'conda_cuda_libs' referenced before assignment on bitsandbytes 0.40.1post1

Hi there, nice work as always!
Have started to get this issue after updating bistandbytes to 0.40.1post1 version.
The complete error is
Traceback (most recent call last):
  File "I:\merge model\mergemodel.py", line 5, in <module>
    from peft import PeftModel
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\peft\__init__.py", line 22, in <module>
    from .mapping import MODEL_TYPE_TO_PEFT_MODEL_MAPPING, PEFT_TYPE_TO_CONFIG_MAPPING, get_peft_config, get_peft_model
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\peft\mapping.py", line 16, in <module>
    from .peft_model import (
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\peft\peft_model.py", line 34, in <module>
    from .tuners import (
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\peft\tuners\__init__.py", line 21, in <module>
    from .lora import LoraConfig, LoraModel
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\peft\tuners\lora.py", line 41, in <module>
    import bitsandbytes as bnb
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\__init__.py", line 6, in <module>
    from . import cuda_setup, utils, research
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\research\__init__.py", line 1, in <module>
    from . import nn
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\research\nn\__init__.py", line 1, in <module>
    from .modules import LinearFP8Mixed, LinearFP8Global
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\research\nn\modules.py", line 8, in <module>
    from bitsandbytes.optim import GlobalOptimManager
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\optim\__init__.py", line 6, in <module>
    from bitsandbytes.cextension import COMPILED_WITH_CUDA
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\cextension.py", line 13, in <module>
    setup.run_cuda_setup()
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\cuda_setup\main.py", line 126, in run_cuda_setup
    binary_name, cudart_path, cc, cuda_version_string = evaluate_cuda_setup()
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\cuda_setup\main.py", line 440, in evaluate_cuda_setup
    cudart_path = determine_cuda_runtime_lib_path()
  File "F:\ChatIAs\oobabooga\venv\lib\site-packages\bitsandbytes\cuda_setup\main.py", line 315, in determine_cuda_runtime_lib_path
    cuda_runtime_libs.update(conda_cuda_libs)
UnboundLocalError: local variable 'conda_cuda_libs' referenced before assignment

The error goes away if using bitsandbytes 0.40.0post4

I'm going through the code now. 0.40.1 build will be fixed and uploaded soon, then I will update to 0.40.2.

I have uploaded a hopefully fixed build:
https://github.com/jllllll/bitsandbytes-windows-webui/releases/download/wheels/bitsandbytes-0.40.1.post1-py3-none-win_amd64.whl
Please let me know if it works. Just started the 0.40.2 build, which should complete in around 30 minutes.

I have uploaded a hopefully fixed build: https://github.com/jllllll/bitsandbytes-windows-webui/releases/download/wheels/bitsandbytes-0.40.1.post1-py3-none-win_amd64.whl
Please let me know if it works. ~Just started the 0.40.2 build, which should complete in around 30 minutes.~ 0.40.2 build is now uploaded as well.

Just tested and it worked without issues, thanks!


Propose limit.with API to streamline map() usage

Hello, I'd like to propose introducing a limit.with API to streamline the process of creating limited functions with Array.prototype.map()
Current API:
const limit = pLimit(5)

// With inline function
await Promise.all(
  items.map(item => limit(async () => {
    await process(item)
  }))
)

// With function declaration
async function processItem(item) {
  await process(item)
}

await Promise.all(
  items.map(item => limit(() => processItem(item)))
)

Proposed API:
const limit = pLimit(5)

// With inline function
await Promise.all(
  items.map(limit.with(async item => {
    await process(item)
  }))
)

// With function declaration
async function processItem(item) {
  await process(item)
}

await Promise.all(
  items.map(limit.with(processItem))
)

While the difference is subtle, the intention is to remove some boilerplate and improve readability. I put together a POC here and am happy to follow up with documentation, any API/naming/etc changes, and opening a PR

While it does not allow you to reuse the limiter across several places you could also find p-map interesting: https://github.com/sindresorhus/p-map


Remove the job status job ID message in comments

Fixes #350, also rewords "generate" for less confusion with generate job type.

Closes #350


uiCanExit and redirect to feature

I have a state {name : 'A', redirectTo: 'A.A'}.
I have another state {name: 'B'}. State 'B' implement uiCanExit method and returns a promise returned by modal confirmation popup.
If i navigate to state 'A' from 'B' then uiCanExit called 2 times. As i understand first time because transition to 'A' then it rejected because redirectTo then it navigates to 'A.A'
Not sure, but problem might be in redirectTo implementation, probaly redirectTo handled too late. It works fine if i use hook and moreover i can use abstract state with redirectTo
 $transitions.onBefore({
                to: function (targetState) {
                    return !_.isEmpty(targetState.redirectTo)
                }
            }, function (trans) {
                return trans.router.stateService.target(trans.to().redirectTo);
            });

Agreed, this should work intuitively.  Will add a fix for rc.2


Element not found bug when using PhantomJS?

Not quite sure why this is happening. If I have the code:
Session
    .NavigateTo<LogonPage>(Resources.BaseUrl)
    .Username.EnterText(Users.AdminUser.Username)
    .Password.EnterText(Users.AdminUser.Password)
    .LogonButton.Click<HomePage>()
    .MyDetailsLink.Click();

I get an 'Element(s) NOT Found: GAVE UP' timeout from the WebElementLocator on the MyDetailsLink element.
If I add a ScopeTo line like so:
Session
    .NavigateTo<LogonPage>(Resources.BaseUrl)
    .Username.EnterText(Users.AdminUser.Username)
    .Password.EnterText(Users.AdminUser.Password)
    .LogonButton.Click<HomePage>()
    .ScopeTo<HomePage>
    .MyDetailsLink.Click();

Then everything works as expected. Note that this only seems to happen using PhantomJS - Chrome is fine. If I replace the ScopeTo with a Pause, I still get the error described above.

Hi,
I can’t tell from your code how the HomePage is instantiated, but my guess is that the HomePage class constructor needs to add a check for the MyDetailsLink on construction.  The WebBlock base class that we provide has a Wait property that you can leverage in this case.  You could also put this wait (which returns an element) in your MyDetailsLink property.
Example:  (gist is here https://gist.github.com/toddmeinershagen/ad5f8e61624418eac537 )
public class HomePage : WebBlock
{
public HomePage(Session session)
: base(Session)

{
Wait.Until(driver => driver.GetElement(By.TagName("myDetailsLink")));

}
}
Hope this helps,
Todd
From: dnkmiller<EMAIL_ADDRESS>Sent: Friday, February 20, 2015 6:29 AM
To: patrickherrmann/Bumblebee
Subject: [Bumblebee] Element not found bug when using PhantomJS? (#33)
Not quite sure why this is happening. If I have the code:
Session
.NavigateTo(Resources.BaseUrl)
.Username.EnterText(Users.AdminUser.Username)
.Password.EnterText(Users.AdminUser.Password)
.LogonButton.Click()
.MyDetailsLink.Click();
I get an 'Element(s) NOT Found: GAVE UP' timeout from the WebElementLocator on the MyDetailsLink element.
If I add a ScopeTo line like so:
Session
.NavigateTo(Resources.BaseUrl)
.Username.EnterText(Users.AdminUser.Username)
.Password.EnterText(Users.AdminUser.Password)
.LogonButton.Click()
.ScopeTo
.MyDetailsLink.Click();
Then everything works as expected. Note that this only seems to happen using PhantomJS - Chrome is fine. If I replace the ScopeTo with a Pause, I still get the error described above.
—
Reply to this email directly or view it on GitHub https://github.com/patrickherrmann/Bumblebee/issues/33 .  https://github.com/notifications/beacon/AAK1ZOabhnv7cnp4SNRgkAL-KQIjiJQSks5ntx_9gaJpZM4DjQsB.gif

DId my suggestion to put logic in the constructor to wait for the details link resolve your issue with Phantom?

Yes. All resolved now - thanks for the help.


Are Kubernetes Ingress annotations still a work in progress?

I've been trying to achieve a kubernetes based setup where I'm using Traefik 2 as a router and cert-manager to manage requesting Let's Encrypt certificates for my ingresses. So far I've managed to get things working. I've set up Traefik with a HTTP and HTTPS entrypoint, cert-manager is able to create an ingress through Traefik's HTTP entrypoint and Let's Encrypt successfully issues certificates, which are being picked up by Traefik.
I now want to proceed configuring my ingress to redirect any traffic to the HTTPS endpoint. I've not managed to do so. I assumed I could get this done by adding annotations to the ingress. This worked for Traefik 1.7, so it should work for 2.x. After a long dig I failed to find any info on the 2.x annotations in the docs. After a log of digging I suddenly ended up on a page containing a list of possible annotations, but these annotations seemed to do nothing when I applied them to my ingresses. Then I noticed the "master" part in the URL of the documentation page. I started digging the repository and found  commit https://github.com/containous/traefik/commit/6b7be462b84132c561ba1d2ef5b8a1a16ff2cc62 in the master branch.
May I safely assume that Kubernetes Ingress annotations haven't made their way into the 2.x code yet? And if so, is there any ETA of  the aforementioned commit to make it into a future release? If annotations are still a work in progress, could you perhaps add it to the documentation of the current 2.x releases? It would save anyone looking for this functionality a lot of time.

Hey @leonboot ,
thanks for your interest.
correct. Ingress Support will be added with the release v2.2 later this year.
What documentation page do you mean? You should only be able to see it, if you selected the "experimental" version on the documentation. On the current stable v2.1, there should be no indication of the "new" annotations.

Hi @SantoDE,
Thanks for your swift reply! Thanks for confirming the annotations support is still being worked on. I found the page with the annotations by googling for one of the annotations that I somehow managed to find, I think I found it in the test fixtures in this repository. Google returned this page: https://docs.traefik.io/master/routing/providers/kubernetes-ingress/ After trying a few other annotations from that page I noticed the 'master' part in the URL and figured it was the expirimental version of the documentation, not the stable.
As Traefik 1.7 supports annotations for the Kubernetes Ingress provider, I assumed version 2 also had support for them. I can imagine more people who are upgrading from 1 to 2 have this assumption and might end up going down this rabbit hole. Therefore I suggested adding a note to the 2.0 and 2.1 docs stating that annotations for the Kubernetes Ingress provider is still under development.
Anyway, I'm glad to hear it's on the radar and due for release in version 2.2. I hope it gets releaseed soon!

@SantoDE Problem is that You can not force http->https redirection without annotations with ingress provider...

@lukasmrtvy with the merged support, you can. You can create a middleware and reference that from the Ingress :-)

@SantoDE yes, but there is no ETA for release yet..
Problem:

ingressCRD provider ( external-dns will not work with annotation https://github.com/containous/traefik/issues/4655, there is workaround with externalservice https://github.com/kubernetes-sigs/external-dns/issues/1257, but it is not clean..., cert-manager will not work with annotation  )
ingress provider ( external-dns will work with annotation, cert-manager will work with annotation  http->https redirection will not work )

Would be nice to have integration for IngressRoute to basic infrastructure tools like external-dns, and cert-manager.

I agree with @lukasmrtvy on this. One of the primary reasons my organization is not adopting Traefik 2.x and is instead staying on Traefik 1.x is because external dns doesn't work with the new Traefik IngressRoute.
With Traefik 1.x this just "worked". No need to worry about manually setting up CNAMES or alias records in cloud providers to point to the underlying LB. Sure, custom solutions could be created but one of the greatest allures to traefik in the first place was that everything "just works". Having that not be the case with Traefik 2.x makes it very difficult to migrate to and ultimately may force us to move away from traefik entirely once support for traefik 1.x ends if this is not addressed.

I'd recommend this issue get locked and/or split into other tasks, e.g. relating to external DNS. For the time being, native Kubernetes ingress is supported and we are using it successfully in production, including referencing CRD middlewares by annotation. The issues raised since this was closed seem to address more specific functionality.

@bradjones1 - can you expand on this:

including referencing CRD middlewares by annotation

Is there documentation on how this can be accomplished/what annotations are available? I don't see anything in the v2.1 docs on the traefik website about this.

@ryaneorth https://docs.traefik.io/middlewares/overview/#provider-namespace

Thanks @bradjones1 . Perhaps I am blind, but I don't see anywhere in that documentation how you can reference middlewares from an Ingress object (as opposed to IngressRoute) using annotations.

You are correct this is not entirely obvious from the documentation. An example annotation referencing a CRD middleware:
traefik.ingress.kubernetes.io/router.middlewares: other-namespace-name-of-middleware@kubernetescrd
See related: https://github.com/containous/traefik/issues/6318


raft_log_engine: update to latest version.

What is changed and how it works?

Issue Number: ref https://github.com/tikv/raft-engine/pull/346

What's Changed:
Update `raft_log_engine` lib to the latest version, to fix some issues, including:
- rewrite: optimize the interval of sync when rewriting memtables #347.
- Return error instead of panicking if rewriting fails #343.

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[x] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None.

Performance tests
Conclusion: no performance regression.
Using tpcc(3000 warehouses, 50 threads, 16 Cores)

QPS
avg_latency(ms)
P99_latency(ms)
P999_latency(ms)
tikv_cpu_usage(%)

Master (nightly version)
21554
1.74
14.2
26.8
1517

this pr(#16294)
21595
1.74
14.1
26.4
1517

/cc @overvenus @Connor1996 It's time to merge this pr.

/merge


426 enable to hold tiled embeddings in memory

used zarr.Group with in memory store to utilize same behaviour like zarr.open() (function returns zarr.Group with directory store).
Passed the zarr.Group(store=MemoryStore) to the _preprocessing functions that do the tiling etc.
Additionally, there was an unreachable code snippet: https://github.com/computational-cell-analytics/micro-sam/blob/426-enable-to-hold-tiled-embeddings-in-memory/micro_sam/util.py#L674
Made it reachable again

Once this is working we should address this FIXME: https://github.com/computational-cell-analytics/micro-sam/blob/dev/test/test_prompt_based_segmentation.py#L24


[3.x] Fix trim when importing WAV

Backport because the loop mode is in AudioStreamSample in 3.x
4.x version: #75261

Thank you!

Cherry-picked for 3.5.3.


Add Feature Rich User Friendly Action UI Part

Love how you have made the alarm functionality so user friendly and (almost) gathered everything under one UI.
I have some ideas on how to make it a more "complete" and very user friendly integration for setting up your alarm in HA, from start to last action.
What I felt was missing when I set it up myself was the action parts.
I set up the alarm states, sensors, users etc then ok now I want my alarm to actually do something when it triggers/disarms etc but had to switch from the nice UI to YAML to get my speaker to play different TTS or local files when arming or disarming the alarm. I want to flash all my lights, take a snapshot with my camera and start recording and upload them somewhere but then i was left to yaml and script when everything else is setup with a nice UI so everyone could do it.
So one idea is to for example add a menu item before "Actions" named "Entities" where you can add entities(from specific domains) similar to how sensors work and also group these entities like the new feature you added for sensors, Device Type.
The main things i thought of would be media_players so it tells me straight that my balcony door is open, close it before arming again. Tell me how many seconds I have left to enter the disarm code. Also when the alarm triggers you can play a siren sound on high volume without the need to buy a stand alone siren with a switch to turn on and off. Or a nice welcome message when you have entered the disarm code,
Under create Action you already have the UI to pick when this action should occur, we just have to check what domain/device type they picked and from there display different options.
Other examples that are very popular from my research on what people usually want to do is flashing lights for example, so the ability to add entities from the light domain so they can get access to set color, brightness etc.
Camera entities would also be a fantastic entity to easily be able to add as an action for snapshots, start/stop recordings.
There are some really nice things that could make this integration very easy to use for everyone and still have the flexibility to let users call their own scripts and service calls, all under one UI.
Maybe I'm way off course and missed something obvious but I would be willing to spend some time on this, many things I will do anyway but maybe not add all the UI stuff if it's not something you or other users think would benefit other users.
PS. Sorry if i have posted this in the wrong section or something, tried to find you on discord to first check if it sounds interesting at all but coulnd't find a way to contact you, personally i rather use this part to specific issues. If im way off could someone please point me in the right direction on where i should post this or how to send a PM or similar to @nielsfaber
Best Regards Simon

Hi Simon,
I'm Glad you took the initiative to open this discussion!
I completely agree with you that the actions/notification UI is very limited and deserves more attention.
Looking back, initially (v1.0.0) there was only the option to pick a switch entity as your siren device, and enable notifications, which were hard-coded texts.
The added GUI allows you with more control over both items, but still is very basic.
So it's a work-in-progress.
The whole reason I added this function, is because I'm not a big fan of automations in HA.
The GUI for creating automations in HA is very confusing to me, and with YAML I find that the files tend to grow very long and I lose overview.
Then there are the "device automations" in HA, which I never really gave a try:

I would like to avoid making a duplicate of the automations implementation in HA.
All functionality that you want to add can already be achieved through automations.
So it's important to keep asking if it's really worth the effort to keep expanding the Actions for all possible use-cases.
Adding functionality to the UI could make it look confusing at some point, ideally i would like to keep things simple, without making it feel restricted.
I do agree that I heard a lot of users talk about wanting to use smart speakers / tts for notifications, and also the flashing of lights, and the ability to run an action for X time and then abort it (optionally with a closing action).
Currently my priority is creating support for multiple areas/zones, which can each be independently armed/disarmed.
I consider this a core function, which is really restricting usage currently.
While working on this, I'm hoping everyone is able to find a way to combine Alarmo Actions with HA automations, to fit their use-cases.
Let's keep the discussion open for what would be good additions for the Actions/Notifications.
It would be helpful to me if we can place things in the right order (prioritize), and also think about how the GUI should look to be kept simple.
Yesterday I added a new graphical dropdown list to Alarmo, think it would be a great way to give the user a choice over various type of actions, and have the GUI adapt to it:

Anyway, I'd like to ask you (and others) for some patience.
I know Alarmo looks simple, but creating it is not (at least not for me).
I would like to have the community decide which features will be added next, so it's helpful to me that you take the effort of sharing your ideas.
Maybe we can open some sort of vote on which feature(s) are missed the most, to create a wishlist.

I think you are right in focusing on the basics of an alarm system (first), and leave parts that can be achieved elsewhere (more or less easily) aside for the time being. For my part, I only just installed Alarmo (as it sounded exactly what I was looking for - a basic, simple, easy to manage alarm panel), and one thing I miss out of the box: delayed triggering of alarm based on which zone triggered it. Triggering the alarm should only be delayed for regular access points to the house (front door open and motion, garage door open and motion) to leave time to turn off the alarm. All other sensors, when detecting motion, window opening or vibration etc., should trigger the alarm right away - no delay.
Maybe create a feature request list, where everyone can vote their priorities? And you to decide if you agree with them, of course :)

I think you are right in focusing on the basics of an alarm system (first), and leave parts that can be achieved elsewhere (more or less easily) aside for the time being. For my part, I only just installed Alarmo (as it sounded exactly what I was looking for - a basic, simple, easy to manage alarm panel), and one thing I miss out of the box: delayed triggering of alarm based on which zone triggered it. Triggering the alarm should only be delayed for regular access points to the house (front door open and motion, garage door open and motion) to leave time to turn off the alarm. All other sensors, when detecting motion, window opening or vibration etc., should trigger the alarm right away - no delay.
Maybe create a feature request list, where everyone can vote their priorities? And you to decide if you agree with them, of course :)

one thing I miss out of the box: delayed triggering of alarm based on which zone triggered it. Triggering the alarm should only be delayed for regular access points to the house (front door open and motion, garage door open and motion) to leave time to turn off the alarm. All other sensors, when detecting motion, window opening or vibration etc., should trigger the alarm right away - no delay.

This is already in place. Tell Alarmo which sensors are doors/windows/motion etc.

one thing I miss out of the box: delayed triggering of alarm based on which zone triggered it. Triggering the alarm should only be delayed for regular access points to the house (front door open and motion, garage door open and motion) to leave time to turn off the alarm. All other sensors, when detecting motion, window opening or vibration etc., should trigger the alarm right away - no delay.

This is already in place. Tell Alarmo which sensors are doors/windows/motion etc.

one thing I miss out of the box: delayed triggering of alarm based on which zone triggered it. Triggering the alarm should only be delayed for regular access points to the house (front door open and motion, garage door open and motion) to leave time to turn off the alarm. All other sensors, when detecting motion, window opening or vibration etc., should trigger the alarm right away - no delay.

This is already in place. Tell Alarmo which sensors are doors/windows/motion etc.

So sorry - just discovered this in "advanced settings" :) Thanks for a very quick reply; will do some more exploring before further comments!

one thing I miss out of the box: delayed triggering of alarm based on which zone triggered it. Triggering the alarm should only be delayed for regular access points to the house (front door open and motion, garage door open and motion) to leave time to turn off the alarm. All other sensors, when detecting motion, window opening or vibration etc., should trigger the alarm right away - no delay.

This is already in place. Tell Alarmo which sensors are doors/windows/motion etc.

So sorry - just discovered this in "advanced settings" :) Thanks for a very quick reply; will do some more exploring before further comments!

@InjunBob That’s alright. Documentation is quite minimal still, so there’s a bit of a learning curve.
I like the idea of voting for feature requests, I will use the forums for that.

@InjunBob That’s alright. Documentation is quite minimal still, so there’s a bit of a learning curve.
I like the idea of voting for feature requests, I will use the forums for that.

@InjunBob That’s alright. Documentation is quite minimal still, so there’s a bit of a learning curve.
I like the idea of voting for feature requests, I will use the forums for that.

Hi Niels,
Is the voting system already online? love to follow the future developments of this great tool!
Thanks!

@dannymertens You can vote by giving a thumbs up to someone else's feature request.
If everyone would do this, it would help me prioritize improvements


V2 APIs

Heyho,
I added the available v2 APIs to the gem.
Will do Auth tomorrow :)

Coverage remained the same at 100.0% when pulling d50d5d5bcb4e4559b547c6b4f0a0b12ad2ec32c4 on Downi:master into 7bb7c02c3f522672fb1264950494ec767bb47920 on parix:master.

+1 on this :)

Sadly I got no time for the Auth API at the moment..But I will do it, someday :P

i was only interested by the commerce endpoint anyway :)

Coverage remained the same at 100.0% when pulling ce040f7203af7c87fd29776d3f80467fb3285ced on Downi:master into 7bb7c02c3f522672fb1264950494ec767bb47920 on parix:master.

This looks good, some tests are broken but that's ok I'll fix them.

I'll be merging this tonight.

Yeah sorry about the tests I forgot about them the time I did the new tp listing stuff :D

I'll push the 'final'  login key stuff tomorrow btw, if you want to wait for it :)

Ah ok, I'll wait for it. =)

Coverage remained the same at 100.0% when pulling 9c26133538102c2b820e6ecb8f2728e23571de00 on Downi:master into 7bb7c02c3f522672fb1264950494ec767bb47920 on parix:master.

Thinking about experimenting with the API. Curious if there is any chance of this getting merged in?

Same - are there plans to move this forward? Anything I can do to help?


Implemented Unit Test DetailsReturnsHttpNotFoundResultWhenEventIsNull

Continues #684

@SteveKennedy I'll try to get a code review of this tonight

@SteveKennedy looks like the build is failing on implementation of IAllReadyDataAccess in the unit tests. Can you rebase on the latest? The other PRs were building okay.

@MisterJames I think so.  Still trying to learn my way around Git.  I've been on TFS forever, and I'm still not quite sure what I'm doing in Git.  So I need to ensure that my own branches have the latest, so that I'm working with the latest/greatest, right?  Also, did the build break as a result of something I inadvertently did?

@SteveKennedy, there is a really great blog post that @SteveGordon wrote about the rebase process for AlllReady: http://stevejgordon.co.uk/contributing-to-allready
I too came from TFS background and had to learn git, and the blog post helped me a lot.

@SteveKennedy, any progress on this? Reach out to me if you need help

@SteveKennedy nothing you did, error by omission only. The rebasing bits aren't too scary and I'd be willing to walk you through them. Have a look at Steve's post, ping @mgmccarthy or myself here or whatever you need and we'll try to help!

I'm so sorry guys. #LifeGotInTheWay.  Looking at this now.

Ok.  I think I did the steps to rebase....

@MisterJames @mgmccarthy Did I do something extremely wrong here. I just recognized there's 32 commits associated with this PR.  Should I back out / close this request and just make 1 new PR?

 So I closed this craziness and opened #844, with same changes.