Need a way to alias joins for multiple joins on the same relation

For example the following schema
class Person
	has_many :tags
end

#  id              :integer         primary key
#  name         :string
#  person_id   :integer
class Tag
	belongs_to :person
end

Let's find all people who have tag "happy"
Person.joins{tags}.where{tags.name == "happy"}

Let's find all people who have tag "happy" or "fun"
Person.joins{tags}.where{(tags.name == "happy") | (tags.name == "fun")}

Let's find all perople who have tag "happy" and "fun"
1st attempt
Person.joins{tags}.where{(tags.name == "happy") & (tags.name == "fun")}

The above will not work because a single tag object cannot be both "happy" and "fun". We are asking the wrong question.
2nd attempt
Person.joins{tags}.where{(tags.name =="happy")} & Person.joins{tags}.where{(tags.name == "fun")}

will give the correct result but this is actually implemented in ruby code where two seperate queries are run and then the two queries filtered by a set AND operation on the two arrays. This can be very inefficient if the two queries by themselves return large sets but the intersection of the two is small.
In theorey this can be achieved by aliasing the join of the first query and then aliasing the join of the second query and merging into a second query. Something like this.
select person.* from person
 join tags tags_1 on tags_1.person_id == person.id
 join tags tags_2 on tags_2.person_id = person.id
 where tags_1.name == "happy" and tags_2.name == "fun"

I can't figure out how to do this kind of composition in AR or SQUEEL without resorting to manual SQL.
Have I missed something here?
Regards
Brad

I don't believe JoinAssociations can have a manually-specified alias -- the JoinDependency takes it upon itself to track/set aliases as needed.
That being said, why not:
Person.joins{tags}.where{(tags.name == 'happy') & id.in(Person.joins{tags}.where{tags.name == 'fun'}.select{id})}

This should (i believe, this is untested) use a subselect and do the trick.


feat: PHPUnit 10.4 support

Fixes: https://github.com/php-mock/php-mock-phpunit/issues/61
Improves: https://github.com/php-mock/php-mock-phpunit/pull/62

Thanks @marcoscoelho!


java.lang.NoClassDefFoundError: org/apache/hc/core5/http/HttpRequest

When I run a query with client ,there is a warn log, the result can be showed correcttly
Expected behaviour
Code example
2023-11-15T13:48:08.235+08:00  WARN 1802624 --- [ckHouseWorker-1] c.c.c.h.ClickHouseHttpConnectionFactory  : Error when creating APACHE_HTTP_CLIENT, fall back to HTTP_URL_CONNECTION

java.lang.NoClassDefFoundError: org/apache/hc/core5/http/HttpRequest
	at com.clickhouse.client.http.ClickHouseHttpConnectionFactory.createConnection(ClickHouseHttpConnectionFactory.java:22) ~[clickhouse-http-client-0.5.0.jar:clickhouse-http-client 0.5.0 (revision: d384444)]
	at com.clickhouse.client.http.ClickHouseHttpClient.newConnection(ClickHouseHttpClient.java:56) ~[clickhouse-http-client-0.5.0.jar:clickhouse-http-client 0.5.0 (revision: d384444)]
	at com.clickhouse.client.http.ClickHouseHttpClient.newConnection(ClickHouseHttpClient.java:26) ~[clickhouse-http-client-0.5.0.jar:clickhouse-http-client 0.5.0 (revision: d384444)]
	at com.clickhouse.client.AbstractClient.getConnection(AbstractClient.java:198) ~[clickhouse-client-0.5.0.jar:clickhouse-client 0.5.0 (revision: d384444)]
	at com.clickhouse.client.http.ClickHouseHttpClient.send(ClickHouseHttpClient.java:90) ~[clickhouse-http-client-0.5.0.jar:clickhouse-http-client 0.5.0 (revision: d384444)]
	at com.clickhouse.client.AbstractClient.sendAsync(AbstractClient.java:161) ~[clickhouse-client-0.5.0.jar:clickhouse-client 0.5.0 (revision: d384444)]
	at com.clickhouse.client.AbstractClient.lambda$execute$0(AbstractClient.java:273) ~[clickhouse-client-0.5.0.jar:clickhouse-client 0.5.0 (revision: d384444)]
	at java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1768) ~[na:na]
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) ~[na:na]
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) ~[na:na]
	at java.base/java.lang.Thread.run(Thread.java:833) ~[na:na]
Caused by: java.lang.ClassNotFoundException: org.apache.hc.core5.http.HttpRequest
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641) ~[na:na]
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188) ~[na:na]
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520) ~[na:na]
	... 11 common frames omitted

Error log
WARN 1802624 --- [ckHouseWorker-1] c.c.c.h.ClickHouseHttpConnectionFactory  : Error when creating APACHE_HTTP_CLIENT, fall back to HTTP_URL_CONNECTION
Configuration
Environment

Client version: 0.5.0
Language version: jdk 17
OS: linux

I think it's because the constructor is protected

@sxh-lsc Did you add the Apache Client HTTP library as a dependency?

fixed by reduce the version

Pls are you reduce clickhouse version? Also, whether there is an alternative solution


[ICE: 48] Mode-Blast route is displayed

Describe the bug
Mode-Blast is looks like supported route with some warning.
It should be displayed here: "Route is not supported"
Where did the user initiate this transfer?
https://bridge.connext.network/EZETH-from-mode-to-blast?amount=12
Ticket: https://app.intercom.com/a/inbox/qge2vd8z/inbox/shared/all/conversation/1066?view=List
Screenshots

Ensure that the fields can be filled only for Ethereum - Blast ezETH, and nowhere else.

Ensure that the fields can be filled only for Ethereum - Blast ezETH, and nowhere else.

@ekbainova It appears that this is usually true, but you can still break the UI by:

Choose Ethereum as source
Choose Blast as destination
Change source to something else
Bug appears: User has Blast destination and non-Ethereum as source

I've pushed up a change here: https://github.com/connext/monorepo/issues/6223
My update will automatically change the source to Ethereum when Blast is chosen as the destination. If we want a different approach let me know! I've attached a screengrab of the change in action:
https://github.com/connext/monorepo/assets/5316399/27d5cb98-0d50-4df3-9d16-afb3412773f8


generate_package_config.py fail due to dart sdk version missmatch

python3 tools/generate_package_config.py currently fails with the following error:
src/third_party/dart/tools/generate_package_config.dart:1:1: Error: The specified language version is too high. The highest supported language version is 2.16.
// Copyright (c) 2020, the Dart project authors.  Please see the AUTHORS file
^
Error: Command 'python3 src/third_party/dart/tools/generate_package_config.py' returned non-zero exit status 254 in /Users/goderbauer/dev/engine
src/third_party/dart/tools/generate_package_config.dart:1:1: Error: The specified language version is too high. The highest supported language version is 2.16.
// Copyright (c) 2020, the Dart project authors.  Please see the AUTHORS file
^

The dart sdk used to run this script is defined in https://github.com/dart-lang/sdk/tree/main/tools/sdks, which is currently 2.16.2, but the script seems to need 2.17.
This is possibly related to the analyzer_server bumping its lower bound to 2.17 in https://github.com/dart-lang/sdk/commit/d422b637586b044f8c80d06bc8c4c4661558befb.

/cc @devoncarew @scheglov

We may not want to boost the checked-in SDK to 2.17 before we release stable 2.17.
I think the bound for analyzer_server should not have been bumped past the checked-in version of the SDK in https://github.com/dart-lang/sdk/commit/d422b637586b044f8c80d06bc8c4c4661558befb.
@athomas

I've created https://dart-review.googlesource.com/c/sdk/+/241516 to get us in 2.17 beta. We should have high enough confidence in our betas to use them in the checked-in SDKs. If something turns out to be broken in the beta, it gives us a chance to fix before the stable.

Assuming this was fixed by the checked-in SDK update.


Deleted buttons that were redundant on sign-in and login page

Fixes #348

Hey @agundy could you post here as to what changes you want to this PR

i'll just add a link on the bottom.

This should have a similar layout to github's login - The reset password button should be across from the password title, and there should be a "New to RCOS? Create an account." link below on the login page (as a bootstrap button styled as a link)

Updated. See screenshots.

Pulled in through #363


Robot os no longer working on insider build

Robot OS on WSL was working for me on the stable windows version.
Microsoft Windows [Version 10.0.15063]
Today I upgraded to windows insider and now the publish subscribe system is no longer working.
My current windows version:
Microsoft Windows [Version 10.0.16299.15]
roswtf errors with the message The following nodes should be connected but aren't.
The output was recorded while running one of the examples. In the following order:

Terminal was running roscore.
One terminal was running the command: "rosrun turtlesim turtlesim_node". This succesfully launches a window with a turtle in it.
The last terminal was running the command "rosrun turtlesim turtle_teleop_key". This launches the app succesfully but no matter what keys are pressed nothing happens.

An interesting thing to note is that running turtle_teleop_key and once that is started running turtlesim_node will work. I can then move the turtle using the keys.
Calling the clear service works in the later case also but I can't verify in the failing case.
Here is the roswtf output for the first case.
Loaded plugin tf.tfwtf
No package or stack in context
================================================================================
Static checks summary:

No errors or warnings
================================================================================
Beginning tests of your ROS graph. These may take awhile...
analyzing graph...
... done analyzing graph
running graph rules...
... done running graph rules

Online checks summary:

Found 1 warning(s).
Warnings are things that may be just fine, but are sometimes at fault

WARNING The following nodes are unexpectedly connected:
 * unknown (http://DESKTOP-5KR77SI:50230/)->/rosout (/rosout)

Found 1 error(s).

ERROR The following nodes should be connected but aren't:
 * /teleop_turtle->/turtlesim (/turtle1/cmd_vel)
 * /teleop_turtle->/rosout (/rosout)
 * /turtlesim->/rosout (/rosout)

and the second working case:
Loaded plugin tf.tfwtf
No package or stack in context
================================================================================
Static checks summary:

No errors or warnings
================================================================================
Beginning tests of your ROS graph. These may take awhile...
analyzing graph...
... done analyzing graph
running graph rules...
... done running graph rules

Online checks summary:

Found 1 warning(s).
Warnings are things that may be just fine, but are sometimes at fault

WARNING The following nodes are unexpectedly connected:
 * unknown (http://DESKTOP-5KR77SI:50230/)->/rosout (/rosout)

Found 1 error(s).

ERROR The following nodes should be connected but aren't:
 * /teleop_turtle->/rosout (/rosout)
 * /turtlesim->/rosout (/rosout)

I can attach strace files if necessary but since this there was allready specific work on ros as I can tell from this issue: https://github.com/Microsoft/BashOnWindows/issues/1450 I don't think it is necessary. They are rather large...
Roscore output:
... logging to /root/.ros/log/3533c16a-ae92-11e7-bd23-bc8385db5709/roslaunch-DESKTOP-5KR77SI-12375.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.

started roslaunch server http://DESKTOP-5KR77SI:50203/
ros_comm version 1.13.2

SUMMARY
========

PARAMETERS
 * /rosdistro: lunar
 * /rosversion: 1.13.2

NODES

auto-starting new master
process[master]: started with pid [12386]
ROS_MASTER_URI=http://DESKTOP-5KR77SI:11311/

setting /run_id to 3533c16a-ae92-11e7-bd23-bc8385db5709
process[rosout-1]: started with pid [12399]
started core service [/rosout]

I can confirm this issue. I also checked ROS's log file and there are no errors listed there. However, I had a different issue with ROS before the update that I was able to solve by changing the function
def close_half_closed_sockets():
    for proxy in _proxies.values():
        transport = proxy("transport")
        if transport._connection and transport._connection[1] is not None and transport._connection[1].sock is not None:
            state = transport._connection[1].sock.getsockopt(socket.SOL_TCP, socket.TCP_INFO)
            if state == 8:  # CLOSE_WAIT
                transport.close()

in /opt/ros/lunar/lib/python2.7/dist-packages/rosmaster/util.py to immediately return, and that solved this issue for me as well. I found this solution because the other error I had did show up in the log file, so I was able to track the cause. The specific line that causes the issue is ...getsockopt(socket.SOL_TCP, socket.TCP_INFO), which previously failed with an error. I'm guessing this option has been stubbed out (though I don't see it in the logs) so it doesn't error now, but it still doesn't work.

@Yay295 - What did you stub to get this work? Do you know the root cause? From your post above, it doesn't sound like TCP_INFO was causing the failure.

I think I may have been looking in the wrong place before, because I have found the error message now.
Using ROS Lunar version 1.13.2 and the steps in the original post I found this
[rosmaster.threadpool][ERROR] 2017-10-30 14:35:11,066: Traceback (most recent call last):
  File "/opt/ros/lunar/lib/python2.7/dist-packages/rosmaster/threadpool.py", line 218, in run
    result = cmd(*args)
  File "/opt/ros/lunar/lib/python2.7/dist-packages/rosmaster/master_api.py", line 210, in publisher_update_task
    ret = xmlrpcapi(api).publisherUpdate('/master', topic, pub_uris)
  File "/opt/ros/lunar/lib/python2.7/dist-packages/rosmaster/util.py", line 67, in xmlrpcapi
    close_half_closed_sockets()
  File "/opt/ros/lunar/lib/python2.7/dist-packages/rosmaster/util.py", line 76, in close_half_closed_sockets
    state = transport._connection[1].sock.getsockopt(socket.SOL_TCP, socket.TCP_INFO)
  File "/usr/lib/python2.7/socket.py", line 228, in meth
    return getattr(self._sock,name)(*args)
error: [Errno 92] Protocol not available

in /.ros/log/latest/master.log.

@Yay295 - Great, thanks. Are you able to stub out the call to getsockopt(TCP_INFO) and see if that resolves the issue? Not suggesting that for proper fix, but, just for information. If so, then we can definitely mark this dupe of #1982. Also, TCP_INFO is our backlog.

No, getsockopt is a read-only method on Python's built-in socket class. This does seem like a duplicate of that issue though.


Glossi non-responsive on select uwp app screen

On Windows 11 when trying to add a UWP app Glossi <IP_ADDRESS> opens the Select UWP App screen and looks like it is searching for UWP apps but instead it locks completely and becomes non-responsive and does not recover.
Event viewer reports the following:
Fault bucket , type 0
Event Name: AppHangB1
Response: Not available
Cab Id: 0
Problem signature:
P1: GlosSIConfig.exe
P2: <IP_ADDRESS>
P3: 61d8579a
P4: e4ff
P5: 134217728
P6:
P7:
P8:
P9:
P10:
I've attached the windows error report
Windows Error Report.zip
If glossi has any log files anywhere let me know and I can provide them.

Can you try the latest snapshot build?
https://ci.appveyor.com/project/Alia5/glossi/build/artifacts

Hi, I am experiencing the same issue on win10 with the snapshot you linked.
Event Viewer File:
glossicrash.zip

Windows event viewer won't help me much.
Try running the bundled GetAUMID.ps1 script and check the output

Okay. I will try that tonight or tomorrow and let you know how it goes.

Windows event viewer won't help me much. Try running the bundled GetAUMID.ps1 script and check the output

That file must be unlocked on Windows, otherwise GlosSI will hang

May you try the latest dev-snapshot again?
https://ci.appveyor.com/project/Alia5/glossi/build/artifacts
It should now display an Error instead of hanging.
If the program still hangs, it should take a maximum of 60s

I am also having this issue with the newest build.

Also having this issue with latest build. I'm hoping this is fixable and not a windows limitation. I was hoping to use my XBOX Elite Series 2 with steaminput for UWP games (so the paddles can have unique functions, etc.)

So an update, the dev snapshot you posted fixed the ump listing issue.  I have some other issues and feedback that I will post in a new thread.

I also would like to chime in that the snapshot is enumerating UWP apps for me as well. I had only tried the latest build from the releases page being it was more recent than that comment in this thread.


[Feature Request]: Move "About Us" Option from Footer to Header Navigation for Better Usability

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The "About Us" link is currently located in the footer, which may not provide optimal visibility for users. It's important for users to easily access key information about the organization without scrolling to the bottom of the page.
Kindly assign me this work under gssoc-ext, hactoberfest24 and level of your choice.
Use Case

Improved user experience.
Easier Navigation.
Increased visibility.

Benefits

Enhanced User Accessibility
Improves Site Navigation
Aligns with User Expectations

Add ScreenShots

Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@khushi1315 I think for this project, its suited there in the footer, please open another issue if you have any better idea
thank you for the suggestion


Add module to depmap database

Request type
New module
Request description
https://depmap.org/
Example command
No response
Example return value
No response

Cool! Looking forward to this. I would like to use it in https://github.com/GilbertLabUCSF/CanDI.

Note: Some DepMap data will be accesible through gget opentargets in gget version >= 0.29.0


Call query hook for count queries

Closed by https://github.com/uptrace/bun/commit/11f90654d5745815b1dd165967c58405cac285c1


Electrum indexer refactoring and bug fixing

Alternative to #61 which additionally fixes other bugs and issues with the code:

consensus deserialization which instead of binary bytes takes a byte representation of the hex string, resulting in invalid transaction data: https://github.com/BP-WG/bp-wallet/blob/dbfb8ba551240a788033f157feed5a6a51351219/src/indexers/electrum.rs#L174
panics due to expects in all data processings, which will result in global crashing and DDoS for all RGB wallets if a major electrum server used by them would be compromised or will have an API break

It also removes overall convolution of the logic which makes it hard to review and audit
The PR leaves further required and important improvements for the later: https://github.com/BP-WG/bp-wallet/issues/64

consensus deserialization which instead of binary bytes takes a byte representation of the hex string, resulting in invalid transaction data: https://github.com/BP-WG/bp-wallet/blob/dbfb8ba551240a788033f157feed5a6a51351219/src/indexers/electrum.rs#L174

I am sorry, this actually not true, I misread the previous code: the hex_bytes are binary coming out of Vec::from_hex
But still it is better to use simpler Tx::from_str form.

I have checked integration tests, they all do pass


LBFGS example, trait bounds

Trying to run the example of LBFGS but running into the issue of

the trait bound ArrayBase<OwnedRepr<f64>, Dim<[usize; 1]>>: argmin_math::ArgminSub<ArrayBase<OwnedRepr<f64>, Dim<[usize; 1]>>, ArrayBase<OwnedRepr<f64>, Dim<[usize; 1]>>> is not satisfied

all features are enabled so am at an impasse as to what to do. Also have a similar trait issue when Param is f64.

Was just my failure to check to import everything

Cargo.toml dependencies are:
'''
[dependencies]
Required
anyhow = "1.0"
argmin = {version = "0.8.1", features = ["slog-logger", "serde1", "slog", "slog-term"]}
instant = {version = "0.1" }
paste = "1"
num-traits = { version = "0.2" }
rand = { version = "0.8.5" }
rand_xoshiro = { version = "0.6.0" }
thiserror = "1.0"
argmin-math = { version = "0.3", default-features = false, features = ["primitives"] }
optional
bincode = { version = "1.3.3", optional = true }
ctrlc = { version = "3.2.4", optional = true }
getrandom = { version = "0.2", features = ["js"], optional = true }
gnuplot = { version = "0.0.37", optional = true }
rayon = { version = "1.6.0", optional = true }
serde = { version = "1.0", features = ["derive", "rc"], optional = true }
serde_json = { version = "1.0", optional = true }
slog = { version = "2.7", optional = true, features = ["dynamic-keys"] }
slog-term = { version = "2.9", optional = true }
slog-async = { version = "2.7", optional = true }
slog-json = { version = "2.6", optional = true }
finitediff = { version = "0.1.4", features = ["ndarray"] }
argmin_testfunctions = "0.1.1"
ndarray = { version = "0.15", features = ["serde-1"] }
'''

Issue identified as argmin maths missing ["ndarray_latest-serde"] as a dependency


Request to share the Dockerfile & update to superset latest master 2.0.0

Thanks to this quick start guide - https://aws-quickstart.github.io/quickstart-apache-superset/, we are able to launch Superset as ECS cluster. The guide uses a docker image - public.ecr.aws/p9r6s5p7/superset:v1.4.1.
I was able to find this docker hosted in AWS Public images gallery - https://gallery.ecr.aws/p9r6s5p7/superset, however this is more than 9 months old. I wish to extend this and base it on the latest version of superset master - release 2.0.0. However I cannot find the Dockerfile associated with this, can you please share the Dockerfile so I can extend this?
I am happy to raise a PR with latest version of superset if you share instructions

Container image hosted in https://gallery.ecr.aws/p9r6s5p7/superset was a boilerplate image without concern of customization, and upstream sync up from community will mainly depend on new features requirement and planned roadmap. May I know what feature you need in Superset 2.0.0?

support for horizontal bar chart and in general the experience on charts have improved in 2.0.0 and we want that. We also want to customise jina macros.

This would be great if the docker file and some info on how to update the cloudformation template and apply custom changes superset_configs could be made available - would really help in trying to use this template for some PoC work I am doing.

I am doing the upstream update for v2.0.0 and PR will be raised by end of Aug.
Before that, you may pull off the image from https://gallery.ecr.aws/p9r6s5p7/superset, add/override with your customizations (superset_configs e.g.), build up the new docker image and replace with image address in original CloudFormation template.

Ok great thanks

PR merged, you can check for the v2.0.0 now

@yike5460 Thanks


Uninitialized constant Aws::S3::Errors while deleting a deployment when using S3 Blobstore

I have S3 blobstore setup in bosh.
Everything OK, but when deleting deployment I got this error:
E, [2016-02-25 16:21:25 #9144] [] ERROR -- DirectorJobRunner: Worker thread raised exception: uninitialized constant Aws::S3::Errors - /var/vcap/packages/director/gem_home/ruby/2.1.0/gems/blobstore_client-1.3192.
0/lib/blobstore_client/s3_blobstore_client.rb:103:in `rescue in delete_object'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/blobstore_client-1.3192.0/lib/blobstore_client/s3_blobstore_client.rb:94:in `delete_object'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/blobstore_client-1.3192.0/lib/blobstore_client/base.rb:69:in `delete'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/rendered_job_templates_cleaner.rb:24:in `block in clean_all'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/rendered_job_templates_cleaner.rb:22:in `each'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/rendered_job_templates_cleaner.rb:22:in `clean_all'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/instance_deleter.rb:37:in `block (2 levels) in delete_instance_plan'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/error_ignorer.rb:9:in `with_force_check'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/instance_deleter.rb:36:in `block in delete_instance_plan'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/event_log.rb:97:in `call'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/event_log.rb:97:in `advance_and_track'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/instance_deleter.rb:19:in `delete_instance_plan'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.3192.0/lib/bosh/director/instance_deleter.rb:55:in `block (3 levels) in delete_instance_plans'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh_common-1.3192.0/lib/common/thread_pool.rb:77:in `call'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh_common-1.3192.0/lib/common/thread_pool.rb:77:in `block (2 levels) in create_thread'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh_common-1.3192.0/lib/common/thread_pool.rb:63:in `loop'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh_common-1.3192.0/lib/common/thread_pool.rb:63:in `block in create_thread'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/logging-1.8.2/lib/logging/diagnostic_context.rb:323:in `call'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/logging-1.8.2/lib/logging/diagnostic_context.rb:323:in `block in create_with_logging_context'

It seems like  a bug in the code, like it cannot find the Module.
Changing  /var/vcap/packages/director/gem_home/ruby/2.1.0/gems/blobstore_client-1.3192.0/lib/blobstore_client/s3_blobstore_client.rb with Exception instead of Aws::Errors and works.

Personally I do not understand why it fails.  The module is defined in aws-sdk-resources:
$ GEM_HOME=/var/vcap/packages/director/gem_home/ruby/2.1.0/ /var/vcap/packages/ruby/bin/ruby -raws-sdk-resources -e 'puts Aws::S3::Errors.class'
Module

I wonder if this is some kind of cross thread class loading issue. Is this something you are seeing consistently?

@cppforlife This doesn't appear to be consistent, some of the tasks will fail with this error, but on re running the delete deployment command they will be able to delete the instance.

To add to this, I'm seeing quite strange behaviour when deleting a deployment. I've seen 3 different errors on repeated runs as follows:
1st run: Failed deleting instances > example_z1/0 (3483c6de-e5a2-4372-bdf2-b6f97e8b385d): uninitialized constant Aws::S3::Errors
2nd run: Failed deleting instances > example_z1/0 (3483c6de-e5a2-4372-bdf2-b6f97e8b385d): wrong number of arguments (1 for 0)
3rd run: Failed deleting instances > example_z1/0 (3483c6de-e5a2-4372-bdf2-b6f97e8b385d): PG::Error: ERROR:  update or delete on table "instances" violates foreign key constraint "rendered_templates_archives_instance_id_fkey" on table "rendered_templates_archives" DETAIL:  Key (id)=(116) is still referenced from table "rendered_templates_archives".
Continuing to run it resulted in the number of errors gradually reducing until it succeeded.

In the AWS CPI we have a bit of code that seems like it was intended to fix a similar issue:
https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/blob/77f493fd1723e0403e57e55f7e41132ad2a65d3d/src/bosh_aws_cpi/lib/cloud/aws/cloud.rb#L61-L63
# AWS Ruby SDK is threadsafe but Ruby autoload isn't,
# so we need to trigger eager autoload while constructing CPI
AWS.eager_autoload!

Might be worth a try to drop that line into the blobstore client and see if the issue persists.

We have resolved this issue in recent bosh releases by switching Director to use s3cli and davcli binaries that are used by the Agent as well (to maintain nice consistency).


JFXButton css -fx-rippler-fill not work

JFXButton ripple is always painted using -fx-text-fill .

Note that in oder to user the css property -fx-rippler-fill, you need to specify the rippler css class.
for example:

.jfx-button .jfx-rippler{
      -fx-rippler-fill:RED;
}

@jfoenixadmin I used jfx-rippler-fill instead of fx and it is working for me.

for me too works withjfx-rippler-fill


feat(actions): bind gh to mousehover

It's quite frequent usecases to use mousehover to see compiler error messages / or other information provided like below:

This PR attempts to mimic vscodevim's keybinding, bringing gh to server as mouse hover in vscode.
Honestly not sure if this is desired changes (and as vim beginner unsure if there's predefined bindings), open to suggestions.
p.s: bumd q, is it possible user can configure this actions by own instead of upstream?

Thank you for the PR! gh is related to the select mode but i think we can sacrifice it for now (it doesn't work now anyway and i don't intent to do it unless someone will ask specially)
I'd also add gH binding to editor.action.triggerParameterHints

p.s: bumd q, is it possible user can configure this actions by own instead of upstream?

Yes, put the same into your init.vim

why not use K for this?

yes, good idea


Error creating XInput Controller in Windows 7

I am trying to create a SharpDX.XInput.Controller for UserIndex.One.  When my code calls the Controller constructor, a dialog pops up as follows

Once I click OK, my program proceeds as expected. The Controller object is created properly and seems to work fine. Also, subsequent attempts to create a Controller object for any UserIndex (even One again) work correctly without showing the dialog.

Do we need to check the OS version number before attempting to load XInput14? It seems that is a Windows v8+ thing.

Oops... Totally user error... Further testing has revealed this was due to me putting the Xinput1_4.dll into my system folder. It doesn't properly run on Windows7, and it was the LoadLibrary call that was causing this dialog to be displayed. Issue can be closed...


Local folder references aren't passed correctly to Claude 3.5 at Jetbrains plugin

Before submitting your bug report

[ ] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Windows 11
- Continue: 0.0.62
- IDE: Pycharm Pro 2024.1.3
- Model: Claude 3.5
- config.json:
  {
  "models": [
    {
      "title": "Claude 3.5 Sonnet",
      "provider": "anthropic",
      "model": "claude-3-5-sonnet-20240620",
      "apiKey": "INSERT_ANTROPIC_API_KEY_HERE"
    }
  ],
  "embeddingsProvider": {
    "provider": "openai",
    "model": "text-embedding-3-small",
    "apiKey": "INSERT_OPENAI_API_KEY_HERE"
  }
}

### Description

While asking a question about a code while providing a reference to an existing local folder (e.g., @the_local_folder) in the context in PyCharm plugin, the following error appear: "Error getting context items from folder. TypeError: db.search is not a function"
Claude 3.5 answer specifies that he don't have information about the_local_folder.

When I try to provide a local file reference, it's being accurately resolved and properly referenced in questions about it.

### To reproduce

1. Install Continue.Dev PyCharm Plugin (v0.0.62)
2.  Load the abovementioned configuration, replacing INSERT_ANTROPIC_API_KEY_HERE and  INSERT_OPENAI_API_KEY_HERE with valid and working api keys.
3. Open a local project of a local git repository
4. Ask a question relating the code, providing a sub-folder of the local repository

### Log output

```Shell
[2024-08-26T11:42:16] Error retrieving from FTS: Error: SQLITE_ERROR: no such table: fts
[2024-08-26T11:42:35] (node:18516) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
(Use `continue-binary --trace-warnings ...` to show where the warning was created)

This is also happening with v0.9.199 (pre-release) of the vscode extension on windows 11.

I also encountered the same error in WebStorm when I used the chat command @codebase, and the error "db.search is not a function" popped up

I was able to fix the error by downgrading to the stable release version (v0.8.46) in vscode

For Jetbrains plugin, the bug replicates also on the latest stable release (currently 0.0.62), downloaded from within PyCharm Pro.

I can replicate this as well. No files are provided with the @codebase context.

I can also replicate this.
This was working fine on Tuesday August 27th. So I think the release on August 27th perhaps broke this.

Here is some hopefully helpful information.
In version 0.8.46 of the VSCode plugin, the ~/.continue/index/index.sqlite database has the following tables:

fts
fts_config
fts_content
fts_data
fts_docsize
fts_idx
fts_metadata

In version 0.8.47 those tables do NOT exist.

the db.search problem should now be solved in the latest pre-release for VS Code. The update will make its way to main release and Jetbrains in the next 1-2 days


[PORT] Add Bot.Builder.Azure.Queues

See https://github.com/microsoft/botbuilder-js/issues/2686 for details.

Related: https://github.com/microsoft/botbuilder-js/issues/2933

Resolved via https://github.com/microsoft/botbuilder-js/pull/3083


Default metadata issues deploying to Production or sandbox org

While deploy to full org current repo has these issues in metadata that needs to be removed from package
Reports and Dashboard Folders are shared to specific user ID that only for scratch but will never exists on other target org as this examples
Dashboard Folder
<EMAIL_ADDRESS><folderShares>
    <accessLevel>Manage</accessLevel>
<EMAIL_ADDRESS>    <sharedToType>User</sharedToType>
</folderShares>

Report or Folder
<folderShares>
    <accessLevel>Manage</accessLevel>
<EMAIL_ADDRESS>    <sharedToType>User</sharedToType>
</folderShares>

Home page layout has unknown item  standard-Tags need to remove that
  <HomePageLayout xmlns="http://soap.sforce.com/2006/04/metadata">
    <narrowComponents>standard-CreateNew</narrowComponents>
    **<narrowComponents>standard-Tags</narrowComponents>**
    <narrowComponents>standard-RecentItems</narrowComponents>
    <wideComponents>standard-Calendar</wideComponents>
    <wideComponents>standard-Tasks</wideComponents>
    <wideComponents>standard-ProcessWorkitems</wideComponents>
    <wideComponents>standard-Dashboard</wideComponents>
</HomePageLayout>

App Menue - need to remove from repo it not required for any org.

Added to Wiki Known issues


SQL Server connection string for Mono

EF7 supports SQL Server on Linux using Mono (http://ef.readthedocs.org/en/latest/providers/index.html#entityframework-sqlserver). However, I have no idea what the connection string will be on Linux using Mono?
I presume we dont have to use localdb either and can use an actual server, but again I have no documentation or idea where to start.

EF passes the connection string through to the ADO.NET provider. Mono uses the same format that full .NET uses. Official docs: https://msdn.microsoft.com/en-us/library/ms254500(v=vs.110).aspx. More useful docs: https://www.connectionstrings.com/sqlconnection

I was after some info on how to actually use in on linux/mono. The links you provide are just MS links.
For example, installed mono on linux. Is that it?
Do I need to manually install something for SQL to be on linux/mono?
How do I see the database, start it, stop it, manage it?
What's the server name? (local), ., machine name, something else?
With my current build on linux with mono installed, running dnx ef database update throws:
Using context 'AppContext'.
Using database 'test-database' on server '.'.
System.Data.SqlClient.SqlException: Server does not exist or connection refused.
Connection string is:
optionsBuilder.UseSqlServer(@"Server=.;Database=test-database;Trusted_Connection=True;MultipleActiveResultSets=True");

Based on your questions, I suggest you read some tutorials on the fundamentals of using SQL Server. I would suggest this as a starting place: Database Engine Tutorials on MSDN.
Regarding your question about mono, your error message indicates to me you have mono installed correctly. Mono comes with a version of SqlClient out-of-box, but it is very outdated (2005) and will you will likely have errors (just search EF issues and you will find some.) We are working on .NET Core implementation of SqlClient that will work better.

I have 5 years SQL Server experience on Windows and run multiple 1 million plus user sites so I'm not sure why you feel I would need basic tutorials on how it works.
I've asked specific questions on how it works on Linux which I have zero experience in but again the questions were ignored.
I'll just look at the .Net source code I'm clearly getting nowhere here

@angelsix My apologies. Using SQL Server from Linux is the same as using it from another Windows instance except that (obviously) you cannot have a localhost instance of SQL Server. Add the server DNX name or IP address to your connection string. Furthermore, IntegratedSecurity does not work on Linux as that is a Windows-only feature, so you will need SQL Server authentication.

I have mis-read the supporting document. I was under the impression Mono had a version of SQL Server for Linux (which I was very surprised by) and thought it was a new .Net Core feature that SQL Server had been ported to run on linux.
Re-reading it, it looks like its just a provider to access an SQL server running on a windows machine.
So that puts me back to a new requirement. My project will run entirely on Linux so I need a database on Linux. What's the best option for supporting a large complex Entity Framework project?

Yup, you and many wish for SQL Server hosting on Linux. Fortunately, EF7 will work with other providers. There is pre-release support for postgres (http://ef.readthedocs.org/en/latest/providers/index.html#entityframework-npgsql) being done by another group (external to the ASP.NET team). It is still a work in progress. You can track the progress on https://github.com/npgsql/npgsql/issues/249.

Thanks. Looks like I'll have to switch to postgres. Is that the only EF database linux option? (in case I find issues with postgres)

At the moment, yes postgres is it. (Other than SQLite, which is really only meant for embedded/mobile devices.)

SQLite is also good for local dev work if you just want to get up and running without needing to spin up Postgres (you can think of it as somewhat akin to LocalDB). We are working with other provider writers to have EF7 providers for the other common database choices on Linux.

I dont mind putting the work in (currently have Postgres installed a user created and a database manually created via psql) so I'll work with PostgeSQL as its the only option.
I like to have alternative options though in case of issues. Don't suppose you guys plan on porting SQL Server to Linux natively? :)

No plans that I'm aware of.
However, we are hard at work getting SQL Client working well on Linux on .NET Core (not Mono). A recent change in CoreFX (the .NET Core implementation of System.Data.SqlClient) makes huge steps towards better Linux support. It should be reflected in our product fairly soon.
But of course, this means you still need a Windows box to run the SQL Server. So this may not make a big difference for you if you need the database to run on Linux too.

Ok good to know. It would be nice to have a good SQL client as my main machine is now a Linux box and SQL clients really suck for linux.
Looks like I'll work hard at PostgreSQL and hope it works with EF7!

All done and working (at least the initial EF migration) with PostgreSQL.
For anyone wanting to use it basic steps are:

Install PostgreSQL (http://www.postgresql.org/download/linux/)

Create a username and pass you would like to use in EF. Via terminal type:
sudo -u postgres psql
psql#> create user  createdb createuser password ‘’;
psql#> \q

In your project.json add reference to Npgsql
"frameworks": {
"dnx451": {
"dependencies": {
"EntityFramework7.Npgsql": "3.1.0-beta7-1"
}
}

In your Startup.cs in ConfigureServices, use Npgsql
services.AddEntityFramework()
.AddNpgsql()
.AddDbContext();

In your context (or in the AddNpgsql() call above) setup the connection string to match the username and password created in step 2. I've done it in the DbContext:

protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
{
// And PostgreSQL for Linux
optionsBuilder.UseNpgsql("Server=<IP_ADDRESS>;Port=5432;Database=desired_database_name;User Id=username_from_step2; Password = password_from_step2;");
}

Back to terminal, go to your project directory, do a dnu restore and dnu build to make sure your project builds ok.

From project directory, create your EF initial migration if you havent already.
dnx ef migrations add InitialMigration

Now get ef to create the database in your PostgreSQL server
dnx ef database update

Install DbVis or SquirrelSQL to connect to your database and visuall confirm all is well.

@angelsix am the main Npgsql dev, happy to see things work for you. Please note that the provider is still not production-ready although most thing should work fine. Please report any issues you run across to https://github.com/npgsql/npgsql

@angelsix

Don't suppose you guys plan on porting SQL Server to Linux natively? :)

https://blogs.microsoft.com/blog/2016/03/07/announcing-sql-server-on-linux/


Angular: unclear debug error message

In the angular test tour:
After running npm start try to debug.
Go Debug and press the play button.
You get an error message that does not make sense to the users. For example, it's unclear where the serve part comes from.

duplicate of #24198


Highlight single and double citations in text

Words or sentences inside single or double citation marks  ``like this'' will be highlighted as a string.
Pretty straight-forward. Captures everything except newlines.

I think you do not need to escape  ` and " but you must check that these characters are not escaped by a backslash before matching them.

I am fine with the feature but the implementation should go to syntaxes/data/LaTeX.tmLanguage.json, see https://github.com/jlelong/vscode-latex-basics/blob/8581ce5042d5387c485d5fbc95b149b7baa883f4/syntaxes/README.md.
Moreover, the regex still has issues: [^\\] requires a character different from backslash. You need to use a negative lookbehind (?<!\\\\). See https://javascript.info/regexp-lookahead-lookbehind


Add the function of generating correlated graph pairs and test.

Add a document to generate the correlated graph pairs  and write tests for it.

Codecov Report

Merging #284 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #284      +/-   ##
==========================================
+ Coverage   87.51%   87.56%   +0.05%
==========================================
  Files          33       34       +1
  Lines        1954     1962       +8
==========================================
+ Hits         1710     1718       +8
  Misses        244      244

Impacted Files
Coverage Δ

graspy/simulations/simulations.py
96.37% <ø> (ø)
:arrow_up:

graspy/simulations/simulations_corr.py
100% <100%> (ø)

Continue to review full report at Codecov.

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

@bdpedigo Could you please review this PR and leave some comments for me?

Here is a notebook showing the properties of sample_edges_corr, er_corr and sbm_corr functions.
https://nbviewer.jupyter.org/github/NeuroDataDesign/team-networkclass/blob/master/Shan/notebook_corr.ipynb

Hi @SHAAAAN, nice progress. I still need to take a closer look but a few things at first glance

all of the most recent commits to graspy are shown as new commits here, how did you go about keeping things up to date with master? Did you do a rebase or merge? We don't want those changes reflected as commits that you made as part of this PR (since they are already in graspy) so we need to revert those changes in your branch, ideally
hard for me to tell if the SBM is working, graph 1 and graph 2 look a bit off. In terms of probability and maybe rho. have you tested against Bear's code at all? Do you agree that they look slightly shifted on your plots? How long do these sims take, perhaps just do more simulations if these don't take very long?
jovo said he wanted to see that the prob/rho values still looked right for a few other different values of rho and p.
have you tried any edge cases? Like what happens when rho is 1 or 0, or p is 1 or 0?

Hi Pedigo, I found a dimension mistake in the previous notebook and I revise it earlier today. Now I can see that the function works well. I also test the properties of the function when p ranging from 0.001 to 0.999 and rho ranging from 0.00 to 1.00. It seems still working well. (This jupyter notebook is the same link above.)
As for the Bear's codes, I have tested it for several pairs. But like we have discussed before, we have to define edge probability for both graphs. As the result, I haven't use his code so far.

Hi @SHAAAAN

Hi @bdpedigo , I found a dimension mistake in the previous notebook and I revised it earlier today. Now we can see that the function works well. I also test the properties of the function when p ranging from 0.001 to 0.999 and rho ranging from 0.00 to 1.00. It seems still working well. (This jupyter notebook is the same link above.) In addition, I restore the recent four commits back to the previous version.

Thanks for updating the notebook. A few things:

Somehow you are not calculating correlation of the sampled graphs correctly. I know this from your plots because correlation should be defined from [-1, 1] and your correlation looks like it 1) cannot be negative and 2) can be bigger than 0. I also don't see why you think the experiment with rho=0 worked, to me it look like the mean and expected are very different?

As for the Bear's codes, I have tested it for several pairs. But like we have discussed before, we have to define edge probability for both graphs. As the result, I haven't use his code so far.

I don't know what you mean about "define edge probability for both graphs." I can't quite remember the discussion that we had before. Does this mean that the two graphs can have different P matrices? If so, why can we not just set them the same and test against that?

Hi @bdpedigo , I have updated the above notebook and add the limitation of the relation between p and rho for er_corr and sbm_corr functions. And Bear's codes are also tested at the end of the above notebook.
So far I have two "sklearn" errors, which I think is similar to Ali's. But other things seem to be correct.

Hi Shan:
corr.ipynb

I think the paragraph at the beginning needs to be changed. I wouldn't talk about the implementation at all. I would basically just say that the correlated model means that we sample two graphs, but we can modify the probability of observing the same edge in both graphs by adjusting the correlation value.
for the titles of the first two heatmaps, just say Correlated ER Graph 1/2
In the demo, I would only use the er_corr and sbm functions
Can you remove the changes you made to erdos_reyni.ipynb and sbm.ipynb
Have you tried to see if you can just do an import as from graspy.simulations import er_corr? That is what we want.
Main code
You have a lot of redundant code where you check inputs to er_corr and sbm_corr. Any common inputs (or parts of inputs that they share) you are welcome to make private functions to deal with them in both
Anywhere you have math or notationin the docstrings, you should get to render as math in sphinx. See examples in the rdpg code or RDPG estimator, it looks like :math: before some piece of text
where is the most up to date proof notebook? Can you edit the PR description at the very top to include that, I would like to take another look and also have it linked to the PR itself.

The code has come a really long way, nice work Shan!

@SHAAAAN this looks pretty good. I want to play with the code a bit before merging but that's on me, not you. So for the purposes of class, no need to worry, I will try to merge it tomorrow but if I don't that's not your fault.
Nice work!


Create an Issue Template file.

Create an Issue Template file.
Create the file in the administrative/ directory.
Fill it out. (This requires first creating a template in the https://github.com/JamieBort/LearningDirectory/tree/master/Git/issues/issue_templates ~~https://github.com/JamieBort/LearningDirectory/tree/master/Git/issue_templates/~~ directory.)
Do this second
For each of these

Personal Dashboard
Portfolio Website
Learning Directory

repos,

add the LearningDirectory/Git/issue_templates/issue_template.md file to the repo,
adapt the template to that repo, and
add a reference to it in the root directory README.md file (if relevant).
add a reference to it in the root directory CONTRIBUTE.md file (if relevant).
If necessary, tweak the issue_template.md file in the https://github.com/JamieBort/LearningDirectory/tree/master/Git/issues/issue_templates ~~https://github.com/JamieBort/LearningDirectory/tree/master/Git/issue_templates/~~ directory so that it is better suited to my repos moving forward.

Todo

[ ] Do this first. Create an issue_template.md file in the LearningDirectory/Git/issue_templates/ directory.
[ ] Do this second. Address the list of things to do in the Do this second section above.

Status
Created the Issue Template.
See this https://github.com/JamieBort/Personal-Dashboard/issues/41#issuecomment-1984861544 comment below.

Created this https://github.com/JamieBort/Personal-Dashboard/pull/45 Pull Request by following this Configuring issue templates for your repository documentation.
However I then added the same files manually.
See this commit fff8d3228e1ff69a544eebe47a460d65a200abd9

First task complete. See commit fe743d7d7be5f07a0e7368ed842a6066d86a35f8

All tasks above are complete.


Add documentation for Bucket Sort

Bucket sort is an algorithm that works by distributing array elements in what are known as buckets!
The docs should be in /Sorting/Bucket Sort/README.md
Happy Open-Sourcing!
For your reference: (Wikipedia)[https://en.wikipedia.org/wiki/Bucket_sort]

Hi @ekdnam
I am new to open source and I would like to work on this. What I wanted to ask was whether I need to give the code as well as the documentation update.
Thank you

Hey @Dutta-SD !
If you want to add both, go right ahead! While making the PR, mention both the issues. That would be it!

Hey
I have made the pull request. Do check it out

Changed the code to accept user input @ekdnam


Support Log4j 2

It seems that vertx still only support Log4j 1.x, please update

Already done: https://github.com/eclipse/vert.x/pull/1303.
It will be released with 3.3.0


ci+scripts: Add version check scripts + GHA workflow.

What changed?
This commit adds new scripts for extracting version info from the .NET project files, and well as the Speakeasy configs. In the new GHA workflow job version-check, we pull both versions, compare them, and conditionally fail the job if there's a mismatch.
This extra PR check should catch cases where speakeasy generate ... was run locally, but the .NET project wasn't updated to match.
TODOs

[ ] Add a sed script to update the .NET project file version whenever Speakeasy generate GHA workflows run.

:information_source: Perfect! The check step fails as intended: https://github.com/StyraInc/opa-csharp/actions/runs/8787784050/job/24113759111?pr=19

:information_source: Now the error case includes a pre-built sed command for adjusting the version number.

:information_source: Currently experimenting with having the GH action conditionally append a "version bump" commit if there's a version mismatch between project file and Speakeasy.


feat: Collect license used to install cluster in support bundles

What this PR does / why we need it:

Store license file used to install cluster in <data-dir>
Add host collector spec to collect the license in support bundle

Demo: https://asciinema.org/a/uhpIRKKc5C9k7MtP1ovI69Kte
I missed showing the file permissions of license.yaml in the demo
[evans] $ make ssh-node0
root@node0:/replicatedhq/embedded-cluster# ls -al /root/ec-data-dir/license.yaml
-r-------- 1 root root 5514 Dec 23 16:24 /root/ec-data-dir/license.yaml
root@node0:/replicatedhq/embedded-cluster#

Which issue(s) this PR fixes:
sc-116758
Does this PR require a test?
NONE
Does this PR require a release note?
Store installer license file in data-dir and add collect when generating a support bundle

Does this PR require documentation?
NONE

well, that solution.... did not work
(two reasons: one is that the new location for the check also encountered the 'previous-stable' tests, which don't have this code yet, and another is that it also ran after restores which don't have the file either)


Moved golang.org/x/net/context to context

Corresponding to https://github.com/google/subcommands/pull/5.

https://tip.golang.org/doc/go1.7#context

Currently, go get github.com/kotakanbe/go-cve-dictionary is failed. It's ok according to a latest README, but an old README is still referred, so I fixed it.

https://github.com/kotakanbe/go-cve-dictionary/commit/683256b65f7f968df15a99f08054c08e4b597490#diff-04c6e90faac2675aa89e2176d2eec7d8L52
http://dev.classmethod.jp/etc/vuls-security-install/
https://blog.animereview.jp/vuls/
http://qiita.com/sadayuki-matsuno/items/0bb8bb1689425bb9a21c

Many Thanks!
LGTM :)


System status - Subscribe with Slack

One way to solve it without waiting for BetterUptime to implement support for Slack or for us to create a custom Slack App is to forward emails to Slack.
https://slack.com/blog/productivity/email-meet-slack-slack-email
It works great and is simple


After running the workflow normally, I find that I don’t know what to do next.

For example, in this processed image, I don’t know what operations should be done on the face. Do you have any good suggestions?

for example, you can use the mask you get to MaskDetailer for inpainting. It can be used to face detailer, hands detailer and so on.


Accessing Individual Shipping Options in admin stalls browser

Accessing individual shipping options in admin crashes browser. User reported this issue and we reproduced in Firefox, Chrome, and Safari. To reproduce, log into admin and select Settings->Shipping.  Selecting Custom Shipping Code seems to reproduce it always for me, but if I select a few different options one after the other, it may freeze up as well.

This turned out to be an admin issue. Partially fixed (the page no longer freezes) but some weirdness may remain until the hAPI fix for the missing slash in shipping hrefs is deployed.


issue #127 - repeated whitespace()

My attempt at this issue. I tried to keep the custom parser, but I wasn't sure how that could be implemented as a Repeated, so I used filter instead. This might be breaking because the return is now a Vec<()> instead of just (), but it can be used with an ignored() at the end to be like the original.
pub fn whitespace<'a, C: Character + 'a, E: Error<C> + 'a>(
) -> Repeated<impl Parser<C, (), Error=E>+Copy+Clone + 'a>{
    filter(|c: &C| c.is_whitespace()).ignored().repeated()
}

This looks about right, thanks!


In custom commands which take a "--staged" argument, allow "--cached" as a synonym

Allow --cached as a synonym of --staged in custom commands, as Git core does the same thing.
See also #11.

It looks like I'll get this done in the process of converting everything to Python.


Move build.sh into the sdk binary

The build.sh script is mostly unnecessary and does not provide any benefits over handing the building from the operator-sdk's binary. In addition, any updates that the sdk gets will not be reflected in existing project as build.sh is generated on new. Currently, this will cause issues for people created a project before the --enable-tests flag was added to the sdk and then try to use --enable-tests.
In addition, we should look into the way that dockerfiles are managed in cases like this. Currently, the above situation (trying to build an image with tests on a project that was generated before that was supported) will not work even with the build script merge because it will be missing the test-framework dockerfile. For now, we can just check if it exists and generate it if it doesn't, but in the long-term, we should find a better solution for managing these files.

Elaborating on the current issue.
Before #469  operator-sdk build would run the build script tmp/build/build.sh
#!/usr/bin/env bash

set -o errexit
set -o nounset
set -o pipefail

if ! which go > /dev/null; then
	echo "golang needs to be installed"
	exit 1
fi

BIN_DIR="$(pwd)/tmp/_output/bin"
mkdir -p ${BIN_DIR}
PROJECT_NAME="memcached-operator"
REPO_PATH="github.com/operator-framework/operator-sdk-samples/memcached-operator"
BUILD_PATH="${REPO_PATH}/cmd/${PROJECT_NAME}"
echo "building "${PROJECT_NAME}"..."
GOOS=linux GOARCH=amd64 CGO_ENABLED=0 go build -o ${BIN_DIR}/${PROJECT_NAME} $BUILD_PATH

But now with operator-sdk build --enable-tests it expects the script to build the test binary as well:
#!/usr/bin/env bash

set -o errexit
set -o nounset
set -o pipefail

if ! which go > /dev/null; then
	echo "golang needs to be installed"
	exit 1
fi

BIN_DIR="$(pwd)/tmp/_output/bin"
mkdir -p ${BIN_DIR}
PROJECT_NAME="memcached-operator"
REPO_PATH="github.com/operator-framework/operator-sdk-samples/memcached-operator"
TEST_PATH="${REPO_PATH}/${TEST_LOCATION}"
BUILD_PATH="${REPO_PATH}/cmd/${PROJECT_NAME}"
echo "building "${PROJECT_NAME}"..."
GOOS=linux GOARCH=amd64 CGO_ENABLED=0 go build -o ${BIN_DIR}/${PROJECT_NAME} $BUILD_PATH
if $ENABLE_TESTS ; then
	echo "building "${PROJECT_NAME}-test"..."
	GOOS=linux GOARCH=amd64 CGO_ENABLED=0 go test -c -o ${BIN_DIR}/${PROJECT_NAME}-test $TEST_PATH
fi

So for projects with the older script the operator-sdk build --enable-test command will fail. So we should stop generating helper scripts and just implement them as Go code in the binary.
I've already done the same for the codegen script tmp/codegen/update-generated.sh in the controller-runtime refactor branch.
https://github.com/operator-framework/operator-sdk/commit/5eb5814c2f5a34d8e653f33dd921eb4956b72336


Unable to consume AWS STS credentials generated by Lambda for connecting to IoT Core endpoint.

I am unable to connect to AWS IoT endpoint, using STS credentials (AssumeRoleUsingWebIdentity) generated by AWS Lambda.
It is throwing me AMQJS0008I Socket closed error
I tried connecting to the same endpoint with the STS credentials (AssumeRoleUsingWebIdentity) and roles generated via AWS CLI & it worked.
But, not sure why this is not working with Lambda.
My Architectural Diagram is

My Code is
const params = {
DurationSeconds: 43000,
RoleArn: 'arn:aws:iam::123456789123:role/TestRole',
RoleSessionName: "IoT-User",
WebIdentityToken: "",
Policy: "{"Version":"2012-10-17","Statement":[{"Sid":"Stmt1","Effect":"Allow","Action":"s3:ListAllMyBuckets","Resource":"*"}]}"
};
const data = await sts.assumeRoleWithWebIdentity(params).promise();

The policy above only includes s3:ListAllMyBuckets. From the docs on AssumeRoleUsingWebIdentity:

"The resulting session's permissions are the intersection of the role's identity-based policy and the session policies."

Therefore, even if the assumed role has IoT permissions, the final credentials will not due to the specified inline policy.

Thanks  justinboswell


Design issues of this sample

This sample is somehow incomplete:

it creates new symetric key at each start, which actually prohibits decryption of SECRET_MESSAGE anytime later
what's use of the SECRET_MESSAGE in this sample, when it's just encrypted (with problem above), and result is not saved, not used next time, it's just displayed on screen

I think that basic principles of AndroidKeyStore and FingerprintManager are somehow mixed to show that finger scanning works, but result is strange. All I get is "password entered or your finger recognized".
I'd expect that symetric key is created once if it's already not in keystore.
Then possibly encrypt user-entered password with the key, and save it to a file. Next time when fingerprint is used, decrypt saved encrypted password, and use it same way as if user entered it manually.
So user has choice to use fast path with finger, or slower path by typing password.

I think the workflow should be like this:

Asking plain password on devices without possibility to use fingerprint sensor:

If fingerprint sensor is usable, and there is no saved encrypted password, let user to enter password and optionally encrypt it afterwards:

After you have encrypt password, save the encrypted form for future use.

If you have saved encrypted password, and fingerprint sensor is usable, let user either type password or use fingerprint to decrypt the saved password.

In all cases, the dialog would be there to obtain the password and return it to app, either directly typed or decrypted from saved copy. And you can't create new keys in keystore each time, otherwise your saved encrypted password would be invalidated.

I am closing this issue/PR, as it has been migrated to the new repo linked above in the comments. Thank you!


Hotfix/oneid 558/update2factortut

Delaying merging PR because the oneID-connect-sample-apps repo is private. Kate, Ryan and Jeff are on an email to figure out if the repo needs to go through compliance. Pending that decision, we will delay this PR.


[flash_ctrl] Width mismatch fixes

I'm not sure that the second patch should really be required, because the errors all come about when using a literal value. I've filed an issue on the Verilator bug tracker (https://github.com/verilator/verilator/issues/2865) to discuss it. Of course, even if that behaviour changes, it'll be a while before we can assume everyone has the new version, so it probably makes sense to do the obvious fix here.

@rswarbrick looks like for this one, we need to make the ArbCnt parameter unsigned, otherwise we get another signed select error.
Do you want to merge this in as is and I can fix those separately? There are a few others that slipped through, so i'd need to fix them anyways.

Gah, it's annoying not being able to run one of our linters! I'll fix the ArbCnt issue and then merge. Sorry about the others that slipped through.


Merge IPAM summary sections

The following section shows up when I search for "IPAM" in the docs:
https://docs.cilium.io/en/stable/gettingstarted/ipam/#gsg-ipam
This implies that there's only two IPAM options. However, there are many others. There is a better list of IPAM options here:
https://docs.cilium.io/en/stable/concepts/networking/ipam/
The first page should maybe be removed or replaced with the second page.

@joestringer may I handle this issue? :)

/assign

@joestringer the link are not working

@swastik959 should be fixed now.


Can I use this on Pixel 3(blueline)?

Prerequisites

[x] Have you read the wiki?
[ ] Is your device in the supported list?
[x] Does the device have a Snapdragon SOC which is supported in this repo?

Description
If I chose a version which support Qualcomm Snapdragon 845, will it work fine on Google Pixel 3?

From my knowledge, no. Most devices have a very different partition table and therefore if it does boot at all ,expect massive issues. Not to mention screen resolution differences, and Windows could never boot due to the ACPI table being different. A few exceptions do occur from my testing, for example the OnePlus 7T image does work on a 7, same as the OP6 and 6T, with minor issues. Fair warning, the guide specifically says pixel devices do have an issue with wiping the entire flash chip bricking the phone, which is very very bad and as far as I know is considered unrepairable. I realize this issue was opened months ago, and hopefully this answers your question and you will close this issue


"Tracker is lost" on KITTI rosbag

Alright, i was able to launch example code using docker. It works. But I'm still getting nowhere trying to launch one of the kitti rosbags using ros2 bag play ../datasets/kitti/rosbags/ros2/kitti_2011_09_29_drive_0071_synced/ --remap /kitti/camera_gray_right/image_raw:=/stereo_camera/right/image /kitti/camera_gray_left/image_raw:=/stereo_camera/left/image /kitti/camera_gray_left/camera_info:=/camera_info_left /kitti/camera_gray_right/camera_info:=/camera_info_right command. Still "Tracker is lost" issue.
Originally posted by @DaddyWesker in https://github.com/NVIDIA-ISAAC-ROS/isaac_ros_visual_slam/issues/37#issuecomment-1188968363

@DaddyWesker Could you tell us where you obtained this KITTI rosbag from? There may be missing frames in the recording that can lead to "tracker lost" issues. Here are the official results for Isaac Visual SLAM on KITTI.

Well, I've downloaded a sequence from kitti website, used kitti2bag to make a rosbag (ros1 bag) which I've used with other SLAM systems and it worked fine.
Then, since your isaac bis based on ROS2 i needed to convert ros1 bag to ros2 bag. I've used these two repos for that purpose.
https://github.com/leo-drive/ros1_bridge
https://github.com/leo-drive/proto_to_rosbag2

Are you seeing "Tracker is lost" frequently or just at the beginning? Also, could you verify if the output of your rosbag and output poses from Isaac ROS Visual SLAM are correlated? Do you see any sudden jumps in the image sequence (from your rosbag)? This is just to verify if there are any skipped frames or timestamp issues that could lead to a "Tracker is lost" scenario.

I prepared the KITTI dataset in the same way as @DaddyWesker  and ran the sample code. However, I frequently get the message "Tracker is lost". rviz confirmed that there are no sudden jumps in the image sequence.


Cloud shell storage support secondary region

[Enter feedback here]

Hi I'm Microsoft FTE. I would like to suggest improve this docuemtn for better customer understanding in Cloud Shell.

Cloud Shell storage support secondary region, but this document sometimes describes that the cloud shell storage must be located in the same region as Cloud Shell machine.

Need to fix the followings
a.  https://learn.microsoft.com/en-us/azure/cloud-shell/persisting-shell-storage#use-existing-resources
'Using the advanced option, you can associate existing resources. When selecting a Cloud Shell region, you must select a backing storage account co-located in the same region. For example, if your assigned region is West US then you must associate a fileshare that resides within West US as well.'

b. https://learn.microsoft.com/en-us/azure/cloud-shell/persisting-shell-storage#prerequisites-for-manual-mounting
'If you mount an existing fileshare, the storage accounts must be located in your select Cloud Shell region. Retrieve the location by running env and checking the ACC_LOCATION.'
Please note that for the 'b', 'clouddrive mount' command gives error if the storage account region is different with region of cloud shell machine for now. However, I confirmed that it is a bug and it will be fixed soon (ETA: 22 Sep. 2023).

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

ID: ecd5d368-8d1c-f984-4922-ab4c7ae14db5
Version Independent ID: dbb6bc4c-87f4-18a0-f6e9-f9f48be1f289
Content: Persist files in Azure Cloud Shell
Content Source: articles/cloud-shell/persisting-shell-storage.md
Service: cloud-shell
GitHub Login: @sdwheeler
Microsoft Alias: sewhee

@eyoh5
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

We will update the documentation when the bug is resolved.

Hi @sdwheeler , the following sentence was not modified yet. Could you please check this below again?
b. https://learn.microsoft.com/en-us/azure/cloud-shell/persisting-shell-storage#prerequisites-for-manual-mounting
'If you mount an existing fileshare, the storage accounts must be located in your select Cloud Shell region. Retrieve the location by running env and checking the ACC_LOCATION.'


[Feature Request] Don't update manga if not in your chosen language

Hi there
Would it be possible to only show manga updates if they are in your chosen language?  I only read in English, but whenever I see a title has been updated and nothing appears, then it means only the non-English version has been uploaded.
Thanks in advance.

If you are talking about in the browse screen. Can you try going to your mangadex account and hiding other languages. Then see if they works?  If not there is nothing I can really do currently. The search in mangadex doesn't show chapters that were updated so I can't filter out manga without making extra network calls

Hi there
Setting the language to English in Browse works great, would it be possible then to set the Browse language to always default to your chosen language, rather than having to manually set it all the time?

thats a site setting on mangadex isnt it?  I figured it would persist through out login


Django default signup commented

The goal is only allow facebook signup for user verification but the signup is not well commented, use
{% comment "Optional note" %}

Commented out text with {{ create_date|date:"c" }}

{% endcomment %}

instead for comments

@juliancantillo can u share me your email?

@johnfelipe I sent it to your github profile email

It's not an issue anymore.


Fix CONTRIBUTING.md links and add Melrose Schools

Fixed CONTRIBUTING linking to leerilly's swot instead of JetBrain's. Also added Melrose Public Schools

The format for the emails are<EMAIL_ADDRESS>for students.


Define and populate this repo with sample tasks

The intended purpose of this repository is to hold sample KeptnTaskDefinitions (aka KeptnTasks) so that new users to Keptn Lifecycle Toolkit have easy access to working tasks.
This will get new users up and running more quickly.
This issue should be used to define what tasks we should create, why and then sub-issues can be defined and linked for the implementation of each KeptnTask.

Hey! can I start look into this @agardnerIT

hey!! folks can someone please guide me the way to contribute into this matter?


[Request for External] Table 37 CalcInvDiscToInvoice

Please make procedure CalcInvDiscToInvoice in table 37 global and external. This has already been done in table 39.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Method not found in PhpStorm

Trying to make the ide-helper work for all the cases. I enabled helper support in the config, cleared the cache, built the _ide_helper.php and the .phpstorm.meta.php, restarted PhpStorm as described in the documentation. Code completion works for many cases, but not for all. For instance:
public/index.php
$kernel = $app->make(Illuminate\Contracts\Http\Kernel::class); // $kernel unresolved
$response = $kernel->handle( // Method 'handle' not found
    $request = Illuminate\Http\Request::capture()
);
$response->send(); // Method 'send' not found
$kernel->terminate($request, $response); // Method 'terminate' not found

database/factories/ModelFactory.php
// Method 'define' not found in TheSeer\Autoload\Factory
$factory->define(App\User::class, function (Faker\Generator $faker) {

database/migrations/2014_10_12_000000_create_users_table.php
            // Method 'unique' not found in Illuminate\Support\Fluent
            $table->string('email')->unique();

app/Http/Middleware/Authenticate.php
        // Method 'guard' not found in \Illuminate\Support\Facades\Auth
        if (Auth::guard($guard)->guest()) {
            if ($request->ajax() || $request->wantsJson()) {
                return response('Unauthorized.', 401);
            } else {
                return redirect()->guest('login'); // Method 'guest' not found in
            }
        }

In such cases just adding /** @var $kernel
\Illuminate\Contracts\Http\Kernel */ will do the trick. Unfortunately
this package can't solve all situations of
auto-completion/object-recognition. Also, everything \Fluent related is
not supported yet, because that class is generic thing used for many
things: from collections to Db.
On 08-05-2016 13:44, danilcha wrote:

Trying to make the ide-helper work for all the cases. I enabled helper
support in the config, cleared the cache, built the _ide_helper.php
and the .phpstorm.meta.php, restarted PhpStorm as described in the
documentation. Code completion works for many cases, but not for all.
For instance:
public/index.php
$kernel = $app->make(Illuminate\Contracts\Http\Kernel::class); //
$kernel unresolved
$response = $kernel->handle( // Method 'handle' not found
$request = Illuminate\Http\Request::capture()
);
$response->send(); // Method 'send' not found
$kernel->terminate($request, $response); // Method 'terminate' not found
database/factories/ModelFactory.php
// Method 'define' not found in TheSeer\Autoload\Factory
$factory->define(App\User::class, function (Faker\Generator $faker) {
database/migrations/2014_10_12_000000_create_users_table.php
// Method 'unique' not found in Illuminate\Support\Fluent
$table->string('email')->unique();
app/Http/Middleware/Authenticate.php
// Method 'guard' not found in \Illuminate\Support\Facades\Auth
if (Auth::guard($guard)->guest()) {
if ($request->ajax() || $request->wantsJson()) {
return response('Unauthorized.', 401);
} else {
return redirect()->guest('login'); // Method 'guest' not found in
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/barryvdh/laravel-ide-helper/issues/348

Additionally, instead of Facades, I recommend using
app('facade-shortcut-here'), in your case app('auth.driver')
On 08-05-2016 13:44, danilcha wrote:

Trying to make the ide-helper work for all the cases. I enabled helper
support in the config, cleared the cache, built the _ide_helper.php
and the .phpstorm.meta.php, restarted PhpStorm as described in the
documentation. Code completion works for many cases, but not for all.
For instance:
public/index.php
$kernel = $app->make(Illuminate\Contracts\Http\Kernel::class); //
$kernel unresolved
$response = $kernel->handle( // Method 'handle' not found
$request = Illuminate\Http\Request::capture()
);
$response->send(); // Method 'send' not found
$kernel->terminate($request, $response); // Method 'terminate' not found
database/factories/ModelFactory.php
// Method 'define' not found in TheSeer\Autoload\Factory
$factory->define(App\User::class, function (Faker\Generator $faker) {
database/migrations/2014_10_12_000000_create_users_table.php
// Method 'unique' not found in Illuminate\Support\Fluent
$table->string('email')->unique();
app/Http/Middleware/Authenticate.php
// Method 'guard' not found in \Illuminate\Support\Facades\Auth
if (Auth::guard($guard)->guest()) {
if ($request->ajax() || $request->wantsJson()) {
return response('Unauthorized.', 401);
} else {
return redirect()->guest('login'); // Method 'guest' not found in
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/barryvdh/laravel-ide-helper/issues/348

Thank you. I'm starting to understand why it does not work in all cases. The facades defined in _ide_helper.php are in global namespace and work only if a php script does not explicitly import the original version, right? So it works with Route, for instance, but does not work with Auth.
The whole Laravel looks like it is written by someone with a background in interpretable languages and Notepad. All those Fluent (why not define a custom class for column definitions?), magic methods, Facades, $app = require_once __DIR__.'/../bootstrap/app.php'; (wtf?!). The very fact that some facades I have to import and some I don't...
There is a PhpStorm ticket about implementing external PHPDoc annotations, hope it will improve the situation with Laravel.

You must have gotten all this upside down :) Because you technically make no sense:
Whatever Facade you want to use in a namespaced script, you need to import it:
use \Route,
    \Auth;

This is not only about Facades, this is how PHP works: e.g. use \Exception;
Laravel has quite advanced abstraction and design pattern implementation, mostly ported from Ruby.
@barryvdh , I recommend closing this issue, it's rapidly becoming "PHP 101".

Getting the same problem in RedirectIfAuthenticated Middleware on a fresh Laravel install. The problem is that by default, Auth is included with use Illuminate\Support\Facades\Auth;, so I have to remove that line and use \Auth instead, so that PhpStorm knows to look into _ide_helper.php

Getting the same problem in RedirectIfAuthenticated Middleware on a fresh Laravel install. The problem is that by default, Auth is included with use Illuminate\Support\Facades\Auth;, so I have to remove that line and use \Auth instead, so that PhpStorm knows to look into _ide_helper.php

Seeing the same issue as @clytemnestra, when we include the full class path for Facades.  We do this for clarity on which specific class is being loaded.
@barryvdh, is there any chance the IDE Helper could add support for full class paths when importing Facades?  e.g. use Illuminate\Support\Facades\Request;

Just another voice asking for this too. Thank you Barry.

Nope, only by modifying the vendor files

Ok thanks for the information Barry.
I take it by your response that Taylor and others have already said no to any changing or updating of Laravel docblocks?
I know Graham Campbell is always very "we program for humans not IDE's" orientated - which is fairly frustrating. When you're immersed in the code it's easy to be dismissive of how easy it all is, but for others like me, a little hand holding by the IDE is very welcome and saves me hours of mistakes.
BTW Barry, If I could star this repo more than once I would. It's just SO crazy useful. Thanks for creating it!

Well I think Taylor actually was open to adding some phpdocs and accepted some pull requests lately.. I just meant this package can't do that.

I'm not sure if there has been a breaking change made in PHPStorm lately, but I can't get this working for the life of me:
http://itsolutionstuff.com/post/how-to-create-custom-facade-in-laravel-52example.html
I've tried both the pre-generated, and generated my own, _ide_helper, generated .phpstorm.meta.php, made sure I have the Laravel Plugin installed, but no matter what I do, the "most working" I can get it when I Ctrl-Click on the Facade invocation, it navigates to the class definition in _ide_helper, instead of the concrete class being facaded.

Is this package broken for the latest versions of PHPStorm?
Thanks!

still get same issue, is there have solution now?

Here you can find good solution https://github.com/barryvdh/laravel-ide-helper/issues/513#issuecomment-318315583

This is a problem issue; there are so many different things pointed out, some may already been fixed, some are still present (namespaced returned values in Facades don't work correctly, there are already multiple issues about this)
I encourage everyone still having issues to a) search the existing issues and vote/comment there or b) create new individual issues per encountered problem.
@barryvdh I think https://github.com/barryvdh/laravel-ide-helper/issues/348#issuecomment-218684683 made already the right call and therefore I too suggest to close this issue :}


ignoreAnnotated not working with complexity>TooManyFunctions

Expected Behavior
The annotations mentioned in ignoreAnnotated should not be counted towards TooManyFunctions.
Observed Behavior
The methods that have been annotated with the annotation to ignore is also getting added.
Steps to Reproduce

Create an annotation.
Added the annotation to the ignoreAnnotated under TooManyFunctions
Add more functions than the limit in the class.

Context
We want to limit the business logic and the number of functions being created in the ViewModel. But we also have a few analytic events in the ViewModel. We don't want to count these analytic events towards the functions that are actually containing any business logic.
Your Environment

Version of detekt used: 1.22.0
Version of Gradle used (if applicable): 7.4.0
Gradle scan link (add --scan option when running the gradle task):
Operating System and version: Mac OS 13
Link to your project (if it's a public repository): Not public but sharing a zip of the sample codebase where the issue can be reproduced. FYI using Android Studio.
TestApp.zip

That's not what ignoreAnnotated does. To make it work you should annotate the class and it will just ignore the issue.
We could create a new configuration on TooManyFunctions so it doesn't count certain functions.
But I'm not 100% sure if we want to add something like that. It seems like a edge case to me. Something that could be done by creating a custom rule instead of adding it to the default rule set. I would like to know what other people think about this.

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

This issue was closed because it has been stalled for 7 days with no activity.


Incorrect DYNAMIC_INVOKE hint with immediate call of closure

The following code produces the strong mode hint DYNAMIC_INVOKE (((x) => 1.0)(() {return 1;}) requires a dynamic invoke):
    var v = ((x) => 1.0)(() { return 1; });

According to @leafpetersen, in general, applying a function with arguments of type dynamic is a dynamic invoke in DDC because of its "fuzzy arrows" feature, however this case should be recognized as a case where no dynamic invoke is actually required.

hmmm, tricky one.
I wonder what we're doing for FunctionExpression's type, if it's a strict or fuzzy arrow. My guess is we use fuzzy arrow right now, because we'd want a fuzzy arrow if we use that type in upwards inference, for example var x = (x) => 1.0).
Assuming the problem is the invocation of (x) => 1.0 (which is my guess), we could handle this in FunctionExpressionInvocation.

https://codereview.chromium.org/2211053002


Invalid JSON (Invalid UTF-8 character at position 0 in state STRING1)

We're indirectly using jsonparse via JSONStream to stream in JSON data stored in Google Cloud Storage and we're intermittently seeing the following error:
Invalid JSON (Invalid UTF-8 character at position 0 in state STRING1)

99% of the time the data is parsed successfully so I'm guessing it's related to where the chunks of data are split over http -- I believe it could be related to emoji characters or Japanese chars as both exist in our json but I'm struggling to pin point exactly where it's failing.
Is there perhaps a way to log more information re: the string value it failed on?

Okay, I've found the problematic character, it's a right double quote and it only happens when at position 0:

Any idea why this character would be misinterpreted, is it an issue on our side or a bug in here?

It looks like this library (and JSONStream) is no longer actively maintained, so for anybody else who is unfortunate enough to run into this issue, I ended up using stream-json which hasn't presented the same problem e.g.
Before:
import _ from 'highland';
import JSONStream from 'JSONStream';

// { data: [{}, {}, {}] }
_(readableStream)
   .through(JSONStream.parse('data.*'))
   .toArray((result) => console.log('DONE', result))

After:
import _ from 'highland';
import { parser } from 'stream-json';
import { pick } from 'stream-json/filters/Pick';
import { streamArray } from 'stream-json/streamers/StreamArray';

// { data: [{}, {}, {}] }
_(readableStream)
    .through(parser())
    .through(pick({ filter: 'data' }))
    .through(streamArray())
    .map(({ value }) => value)
    .toArray((result) => console.log('DONE', result))

For others who stumble across this but still want to use this library, I think changing https://github.com/creationix/jsonparse/blob/b2d8bc6db4f6be3f276752b3b9f882b1945afede/jsonparse.js#L166-L171 can fix this.
Only emit the new character if the buffer contains at least as many bytes as are remaining in the sequence:
        var toConsume = Math.min(this.bytes_remaining, buffer.length);
        for (var j = 0; j < toConsume; j++) {
          this.temp_buffs[this.bytes_in_sequence][this.bytes_in_sequence - this.bytes_remaining + j] = buffer[j];
        }
        this.bytes_remaining -= toConsume;

        if (this.bytes_remaining === 0) {
          this.appendStringBuf(this.temp_buffs[this.bytes_in_sequence]);
          this.bytes_in_sequence = 0;
        }

My fork is pretty far removed from this one


Make gqlgen StructFieldsAlwaysPointers configurable

When updating to a more recent version of gqlgenc, we encountered issues with the generation of models for non-null fields. Non-null structs fields are always pointers instead of being defined as values. We were impacted by the following two changes:

Match gqlgen default settings: #202
struct type field for fragment is pointer: https://github.com/Yamashou/gqlgenc/pull/104/commits/2bb5b94f1784f7dc1cbd8aee7228ff226192457e. This one is quite old, but that change was in only done in client v2, and we were still using client v1.

This PR is to make the generation of non-null structs fields as pointers configurable.
I've used the same name used by gqlgen for the similar config: struct_fields_always_pointers. Unfortunately, that name doesn't fit well with other settings like clientInterfaceName and omitEmptyTypes. Do you prefer a different name? Perhaps structFieldsAlwaysPointers?

@mlaflamm Thanks! Let's name it structFieldsAlwaysPointers!


trivia games joinable midgame + show who submits qs

Made it so you can join trivia games midgame. I also made it so they can't join if unable to talk. The last change was showing who submitted the question in QW.

@ascriptmaster I squashed this if you want to check.

@ascriptmaster adding canTalk to addParticipant wasn't the best idea, so I moved it to the join command.


Doc corrections

Just a couple minor doc corrections.

I've also noticed that there a few other options that aren't documented. Should I add them? Or are they meant to undocumented?

I've also noticed that there a few other options that aren't documented. Should I add them? Or are they meant to undocumented?

Pretty please.
I've been very busy and documentation has been a little neglected.
There are some formatting symbols missing, along with a couple of options.
Help is very much appreciated as I spend more time perfecting the calendar


DRAFT: Add examples to environment variables.

I was struggling to figure out exactly what value each ENV var would produce, so I created a PR that dumped them all out as examples.  With those, I thought it might be helpful to add those examples to the docs.  I chose a table layout for readability.
I've only done the first section so far as a POC.  Thoughts?

@KeyboardCowboy I like this a lot! What else do you need from us to move this forward? You might try running the yarn lint-fix command locally to fix any linting errors.

Thanks, @q0rban!  This is just one section, so I'd like to apply the table to the variables in the other sections as well before we merge it.  I'll see if I can get back on it next week.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DockerBuild logs are not contextual

We encode the cue path in LLB operations. Because call an external frontend for DockerBuild, and therefore we are not the ones generating the LLB, the contextual information is missing.
This needs investigating, a possible solution would be to use dockerfile2llb directly and alter the LLB, however there are drawbacks (e.g. dockerfile.v0 does more than dockerfile2llb, we'd have to reimplement that or we lose functionality. Also, the problem would persist for alternative Dockerfile syntaxes)

Is this issue still relevant even with Cloak ?

I cannot visualise the contextual information that we are missing. I don't even know how to get this information. I am assuming that these changes would need to happen in client/client.go, but I'm not sure.

test pour camille

Superseeded by #3587


Undocumented -packagesroot parameter

Where a custom model references an ISV model, if the -PackagesRoot parameter is not included, the following error occurs:
The source for referenced module '' is missing from the model store. Please specify the -packagesRoot parameter to instead use binary metadata for referenced modules.
The solution was to add the following to the command line: -packagesroot=K:\AosService\PackagesLocalDirectory
However this was not immediately obvious from the available documentation. I think this page should mention it.

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

ID: 2cbcd513-d8ab-ddc6-ce0d-43e03d4692c2
Version Independent ID: d1119159-09e4-1aa1-d90e-c8f23552adbc
Content: Customization Analysis Report (CAR) - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/customization-analysis-report.md
Service: dynamics-365-cross-app
GitHub Login: @RobinARH
Microsoft Alias: jorisde

@glenturnbullazure  , thank you for contacting us regarding this help topic. I have passed your comment on for consideration.

BUG 606554 opened to track this issue

I updated the topic and the new content should go live this evening.


feat(ast): allow conversion between ExpressionStatement and Statement with FromIn trait.

Our down stream rolldown is using this, If we want then to adopt our traits we should provide this implementation for them

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

#4124  👈
#4122
#4117
#4116
#3890
#4098
#4090
#4088
main

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

Join @rzvxa and the rest of your teammates on  Graphite


Remove assert that doesn't always hold

As per https://github.com/whatwg/html/issues/3295 it seems that referencingScriptOrModule can be NULL in the context of import() in an event handler, which is good motivation to drop this Assert since it effectively "crashes" the spec as currently written.

More generally, there are circumstances in any host where there is no active script or module, and import() might be called. The HTML example gives one way this could happen in one specific host. It was just my lack of imagination in thinking this would never happen, when originally writing the import() spec.
Probably I should update the spec text to note this, and ensure hosts are able to handle it.

Makes sense, it seemed that HTML was the motivating consumer for this, but yeah I imagine this is more universal. Thanks!


Add acl to near-token-contract

Adds access control based on near-plugins to near-token-contract.
It's the first time Acl from near-plugins is used externally and it uncovered some flaws. I'll try to fix them in near-plugins and then usage of acl here should become more ergonomic.
Running the following test verifies that access control is set up correctly:
cargo test test_near_token_contract_acl

If other developers using the acl plugin are expected to write similar tests, we might consider exporting test helpers from near-plugins. Something similar to this.

The conflicts are resolved and the dependencies (near-plugins, workspaces) updated.

What's new: Made the changes proposed in #24 and cleaned up several things (see commit messages for details).


Creep Id panic

When I try to call creep.id() and print it's results, it panics with
Error:unwrap_throwfailed     at imports.wbg.__wbindgen_throw (file:///user/screeps-arena-starter-rust:424:15)     at <anonymous>:wasm-function[414]:0x28dce     at <anonymous>:wasm-function[293]:0x258fe     at <anonymous>:wasm-function[382]:0x284d1     at <anonymous>:wasm-function[92]:0x14f04     at <anonymous>:wasm-function[250]:0x247a2     at <anonymous>:wasm-function[132]:0x1a1cf     at <anonymous>:wasm-function[378]:0x2847b     at <anonymous>:wasm-function[311]:0x2604e     at <anonymous>:wasm-function[92]:0x14f04
I only get this issue with creeps, not structures.
From debugging, it seems to work if I instead call js_sys::Reflect::get(creep, JsValue::from("id") )
But it actually returns a JsValue number instead of string. I think the cause is that the JS API is inconsistient with storing the ID attribute as strings or numbers.
Also another issue is that it panics when you call creep.id() before it spawns. In which case js_sys::Reflect returns undefined attribute instead.

Sorry this is probably more suited for the game api repo


Whither extptr and extref?

Have <extptr> and <extref> been dispensed with?  What type of linking
mechanisms is available if we're not linking to digital representations from
the collection?
Submitted on Monday, April 29, 2013 - 10:54am
Submitted values are:
Name: Mark Carlson
Affiliation: University of Washington
Does this represent an official comment from your affiliated group? No

 and  to be used for both internal and external links. TL includes this in description for these elements.


MultiplePickerField does not show validation errors

MultiplePickerField does not have any error text or icon when validations are applied.
Consider the following layout:
<input:FormView>

	<mtrl:PickerField
		Title="My Single Picker"
		ItemsSource="{Binding Items}">
		<mtrl:PickerField.Validations>
			<v:RequiredValidation/>
		</mtrl:PickerField.Validations>
	</mtrl:PickerField>

	<mtrl:MultiplePickerField
		Title="My Multi Picker"
		ItemsSource="{Binding Items}">
		<mtrl:MultiplePickerField.Validations>
			<v:RequiredValidation/>
		</mtrl:MultiplePickerField.Validations>
	</mtrl:MultiplePickerField>

	<Button
		Text="Submit"
		input:FormView.IsSubmitButton="True"/>

</input:FormView>

When the submit button is pressed without filling out either field it only shows the error for the single picker field.

Probably the items colletion is not null, it's an empty collection. You probably need to create a validation something like that:
public class CollectionNotEmptyValidation : IValidation
{
    public string Message { get; }

    public bool Validate(object value)
    {
        if (value is ICollection<object> collection)
        {
            return collection.Count > 0;
        }
        return false;
    }
}

And use it like this:
<material:MultiplePickerField Title="Multiple Picker">
    <material:MultiplePickerField.Validations>
        <root:CollectionNotEmptyValidation Message="At least one option must be selected." />
    </material:MultiplePickerField.Validations>
</material:MultiplePickerField>

I'll consider handling collections in RequiredValidation in the library in the next version

I see. Having RequiredValidation check for empty collections would be useful, however that is only part of the problem.
I dug a little deeper and found the real issue. MultiplePickerField does not override the GetValueForValidator() virtual method defined on InputField. Because of this it uses the implementation in InputField which just returns new object() (see below).
https://github.com/enisn/UraniumUI/blob/45858514387014504eb82f1ed90ceae5846a9a9e/src/UraniumUI.Material/Controls/InputField.Validation.cs#L97
This could be resolved with the following code added to MultiplePickerField.
protected override object GetValueForValidator()
{
    return SelectedItems;
}


fix(vue-app): properly catch component loading error (#5687)

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Component prefetching from nuxt-link's was using try/catch to catch loading error but as the call to load the component returns a promise, in practice actual loading error wasn't caught.
That triggered majority of errors that are reported to Sentry in my production site.
Such errors can trigger fairly easily by canceling loading of a page or navigating to different URL while loading is in progress. We shouldn't care about those failing and shouldn't report them to console/sentry.
Resolves: #5687
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
Honestly I didn't have time to look into it and also I didn't find existing tests for prefetch failure. So investigating how to create new test would require me to spend a lot more time.
[ ] All new and existing tests are passing.
Yes apart from Address /Users/xxx/workspace/github/nuxt.js/test/fixtures/sockets/nuxt.socket is already in use which seems unrelated.

Codecov Report

Merging #5688 into dev will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #5688      +/-   ##
==========================================
- Coverage   95.61%   95.57%   -0.04%
==========================================
  Files          81       81
  Lines        2645     2645
  Branches      673      673
==========================================
- Hits         2529     2528       -1
- Misses         97       98       +1
  Partials       19       19

Impacted Files
Coverage Δ

packages/vue-renderer/src/renderer.js
93.49% <0%> (-0.82%)
:arrow_down:

Continue to review full report at Codecov.

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

Do note that previously Component.__prefetched = true was set incorrectly even before component has loaded so that was safe-guarding from requesting load multiple times while after this change, in theory we might accidently try to prefetch same component multiple times.

I'm look at this PR and I'm not seeing it merged into nuxt.js v2.7.0 pr v2.7.1. Did it somehow get skipped?

2.7.0 has it but in this form: https://github.com/nuxt/nuxt.js/commit/8588ca9a7e7eb69593ab286a72f81c58b9ffbc17

Curious. I'm still getting the same error.
app.js:1905 Uncaught TypeError: Component(...).catch is not a function
    at VueComponent.prefetch (app.js:1905)
    at app.js:1834
    at Array.forEach (<anonymous>)
    at IntersectionObserver.push../.nuxt/components/nuxt-link.client.js.observer (app.js:1826)

That's this bug: https://github.com/nuxt/nuxt.js/issues/5751 (side effect of this fix).

my apologies - i didn't see #5751. Thanks! I'll disable prefetch until this is released.