Enable multiconfig builds for testing / validation

Problem
Currently, a Thrust build must be reconfigured and rebuilt to test and validate various host/device system combinations. This quickly gets tedious when testing whether a change will break any of the backends.
Being an NVIDIA product, the core Thrust developers tend to focus on the HOST=CPP DEVICE=CUDA configuration, and bugs sometimes slip into the other backends that go unnoticed until a user reports them.
By simplifying the process of testing other configurations, we can provide a more stable and reliable product to users that rely on the OpenMP and TBB backends. This will also be useful as our CI improves, as we can have a single CI build generate multiple configurations.
Overview
I propose adding a THRUST_ENABLE_MULTICONFIG cmake option. By default, this will be off and everything will work as it does now. But when enabled, this will:

Hide and ignore the THRUST_HOST_SYSTEM / THRUST_DEVICE_SYSTEM cache options.
Create new THRUST_MULTICONFIG_ENABLE_${system} boolean options (Default ON).
Generate tests, examples, and header-tests for a variety of host/device configuration.

Configurations
The THRUST_MULTICONFIG_ENABLE_${system} options are used to specify the expected systems that a particular build should use. Dependencies of disabled systems will not be searched for, and any configurations that use a disabled system will not be tested. If a system is requested but an associated dependency is not found, a configuration error will occur.
The full list of configurations that will be considered are:

Config
Host
Device

1
CPP
CUDA

2
TBB
CUDA

3
OMP
CUDA

4
CPP
TBB

5
CPP
OMP

Target Naming
The test/example names will need to be modified to reflect which configuration the executable belongs to. For instance:
thrust.xxx.yyy --> thrust.xxx.yyy.HOST_CPP.DEVICE_CUDA

For non-multiconfig builds, the names will stay the same as they currently are.
Meta-targets
Custom CMake targets will be added to make it easier to just build a single configuration. For example:
add_custom_target(thrust.meta.test.HOST_CPP.DEVICE_CUDA)
add_dependencies(thrust.meta.test.HOST_CPP.DEVICE_CUDA [...])

This will allow all tests in the CPP/CUDA config to be built with ninja thrust.meta.test.HOST_CPP.DEVICE_CUDA.
I'm considering this initial set of meta targets:

Target
Description

thrust.meta.HOST_XXX.DEVICE_YYY
All targets associated with the configuration

thrust.meta.tests.HOST_XXX.DEVICE_YYY
All tests associated with the configuration

thrust.meta.examples.HOST_XXX.DEVICE_YYY
All examples associated with the configuration

Testing
CTest offers LABELS properties for tests that could be used to run tests for a single configuration, but these are redundant with the proposed naming scheme. Tests can be filtered using -R regexes when invoking CTest, e.g.:
ctest -R HOST_CPP.DEVICE_TBB

The cross product I used to test with Jenkins was
{cpp, omp, tbb} X {cuda, omp, tbb}
I don't know how useful it is to test e.g. host=tbb, device=tbb, but it's a possible configuration, so there must be some users out there doing that.

Ok, I'll start with that full cross product and see how expensive it is to build. If it's too much, we can split the configs up into a full set (for CI) and a "common" set for quick (well, for some definition of 'quick') local testing.

SGTM
I'm not familiar with the current state of Thrust's build, but I'd say the most valuable things we could do would be something like this:

Make the common build painless enough that it can be run frequently, and
Make any build combination convenient to initiate

I've updated the "Configurations" section of the issue with something that I think will work for most cases.
re: the current state of things, by default we just build and test CPP/CUDA. We can change the host and/or device for the entire build by reconfiguring with CMake.
After this change, that will still be the default behavior. This is most useful for feature development and iterative changes targeted at a single backend.
Turning on THRUST_ENABLE_MULTICONFIG will build and test multiple configs at a time. A THRUST_MULTICONFIG_WORKLOAD option will allow the number of configs to be tuned for hardware / usecase while optimizing coverage.

If we do this, we should probably do this for all the other variants, which I believe is:

C++ dialect (at least for this)
SM architecture (for CUDA backend)
RDC (for CUDA backend)

e.g. thrust.xxx.yyy.HOST_CPP.DEVICE_CUDA.CPP_DIALECT
Maybe we should do at least C++ dialect in this PR.

Updated proposal with C++ dialect options for C++11, 14, and 17. It defaults to just C++17 since I figure that'll give the most coverage.

Updated names to be less harsh and more useful for categorizing targets:
thrust.test.reduce.HOST_CPP.DEVICE_CUDA.CPP14 is now thrust.cpp.cuda.cpp14.test.reduce.

Here are some of my findings from testing out multiconfig builds this weekend. Hardware specs and raw data are in this gist. This is on my NVIDIA provided workstation, so these numbers should be similar (or better) for the other members of the NVIDIA developer team.
There is currently an issue preventing the tbb.cuda configurations from building. These build times are estimated to be similar to the cpp.cuda and omp.cuda measurements in the discussion below.
MultiConfig Cost Summary
The key takeaways are:

CPU backends are practically free. For a FULL build with all dialects enabled, the non-CUDA backends take up ~12% of the total build time -- a little over an hour for all 27 CPU-only host/device/dialect combinations.
Most of the expense comes from the CUDA backends, which account for the remaining ~88% of the total build time. The 9 host/cuda/dialect configurations take around an hour each.

The workload grouping has been updated to reflect this:

SMALL Minimal coverage: cpp.cuda, cpp.omp, cpp.tbb.
MEDIUM Cheap extended coverage: Adds cpp.cpp, omp.omp, tbb.tbb.
LARGE Expensive extended coverage: Adds tbb.cuda, omp.cuda.
FULL Complete coverage: Adds omp.cpp, omp.tbb, tbb.cpp, tbb.omp.

The MEDIUM workload seems to be the best value, as it adds all useful CPU-only systems to a cpp.cuda build for only ~12 minutes/dialect (a 20% increase over ~60 minutes/dialect for just cpp.cuda).
LARGE workloads bring in the additional heavy [omp|tbb].cuda configurations for a cost of ~2 hours/dialect. This provides complete coverage of all useful configurations.
Cost of increasing workload
To determine which workload to build, it's useful to know the cost of increasing to the next largest workload. This section tabulates the time needed to build only the new configurations in each workload.

Workload
cpp11
cpp14
cpp17
All Dialects

cpp.cuda
0:56:52
0:54:59
0:55:27
2:47:18

SMALL
0:04:21
0:04:21
0:04:28
0:13:10

MEDIUM
0:07:41
0:07:33
0:08:00
0:23:14

LARGE
2:05:31
0:02:36
2:06:39
6:14:46

FULL
0:11:03
0:11:24
0:11:43
0:34:10

Total build time per workload
To estimate the amount of time needed for a given workload / dialect combination, this section compares the full build times.

Workload
cpp11
cpp14
cpp17
All Dialects

cpp.cuda
0:56:52
0:54:59
0:55:27
2:47:18

SMALL
1:01:13
0:59:20
0:59:55
3:00:28

MEDIUM
1:08:54
1:06:53
1:07:55
3:23:42

LARGE
3:14:25
3:09:29
3:14:34
9:38:28

FULL
3:25:28
3:20:53
3:26:17
10:12:38

Details
CPUs backends are super cheap
Each build that doesn't require CUDA takes 1-4 minutes. Using the host==device configs to compare their build times, the trend here is about what you'd expect:

Config
Build Time

cpp.cpp
~1:35

omp.omp
~2:20

tbb.tbb
~3:45

The full cross product of the CPU backends -- including the unlikely/silly combos -- only takes about 25 minutes to compile for a single cpp dialect.
Most of the time is spent building CUDA configs
All configurations that use the CUDA backend take a full hour to build when all SM versions are enabled (except 5.3, 6.2, and 7.2, which don't build with RDC). Most of this is due to toolchain overhead in nvcc, but there's also more CUDA code to compile. A non-CUDA build has ~515 targets, but CUDA has ~895.
Some CUDA targets are very expensive
There are a few monster translation units that take a very long time on CUDA.
The most expensive targets in the cpp.cuda.cpp14 configuration are:

Translation Unit
Build Time

testing/scan.cu
~25:00

testing/scan_by_key.cu
~20:00

testing/minmax_element.cu
~17:00

testing/min_element.cu
~16:00

testing/max_element.cu
~16:00

testing/async_sort.cu
~13:00

and quite a few others in the 6-10 minute range. A timeline of the cpp.cuda.cpp14 build on 12 cores is shown below.
Download an interactive html version here.

Did some additional tests varying the number of target SMs on cpp.cuda.cpp14:

#SMs
compute_75
Walltime (-j13)

0
Yes
10 minutes

1
No
11 minutes

1 (SM75)
Yes
11 minutes

2
No
17 minutes

3
No
23 minutes

4
No
29 minutes

5
No
34 minutes

6
No
40 minutes

9
Yes
60 minutes

I've switched to just building the 2 SMs I can test locally, and this changes the total build time to for MEDIUM/cpp14 to around 30 minutes. This seems like a reasonable amount of coverage/expense for pre-commit testing.


Very creative and a good idea!

Good idea.

Yep! My wife had this funny idea to engrave a short link into the rings in case a ring gets lost. :smile:
So here's our way to recover a ring if lost: https://L❤H.ws


revise the causal explanation notebook, added test

Updating the notebook in light of the refactoring.

Added the following explanation:
In principle, we want to be able to condition on the observed consequents: as whatever happened to the consequents is what we want to explain. On the other hand, some of the conditions in the original definition involve queries in the unconditioned model (e.g., are the consequents different always if we intervene on a given candidate cause?) In a more general setting to mimic the definition from the book we would need to have two separate model runs, with and without conditioning and run inference to propagate the results of the conditioning. In this context, when we're dealing with not too many discrete variables, we can proceed in a simplified manner and get away with running forward with an unconditioned model. The conditioning on the consequent is then delegated to trace handling for those parts of the definition that require conditioning. We will emphasize this element of trace handling conditioning in a comment when it happens.

Let's land #441 first. I'm not entirely convinced by that reasoning about conditioning on consequents, but this notebook doesn't need to block #441.

@rfl-urbaniak can you resolve these new merge conflicts?

@rfl-urbaniak and @eb8680 , what is the current status of this PR? I have some spare cycles if you'd like my help reviewing.


CCR-1186: Tile for unsupported file types + watermark/PFP displays as a tile for incorrect file

…le type
Proposed Changes
Please use the Jira Key followed by the changes

JIRAKEY-1234

Some change
Another change

Checklist

[ ] I have added or updated unit tests
[ ] I have added or updated integration tests (if appropriate)
[ ] I have added or updated documentation / readme (if appropriate)
[ ] I have verified that my changes have not introduced new lint errors
[x] I have added a GitHub labels to the PR for either patch, minor, or major

Testing Instructions

@ntrevino-virtru can you merge this PR?


Upgrade to<EMAIL_ADDRESS>

Successor to #41.

@twokul please pay attention to it, as it is green with updated dependencies and working with new ember in viewport :)


[QUESTION] Use table location to access a table defined in unity catalog

Bug
Describe the problem
DeltaTable.forPath(spark, path) inconsistency behaviour with S3 and unity catalog
I'm working with unity catalog for the last week.
I'm refering to delta table by path, as follwing:
path='s3://<my_bucket_name>/silver/data/<table_name>
DeltaTable.forPath(spark, path)
I get an exception that "is not a Delta table"
using the table name using: DeltaTable.forName(spark, <table_name>)
everything works fine. both the attributes are exactly as apear on uc catalog (and data apears in S3.
Steps to reproduce
silver_table_name=<table_name>
from delta.tables import *
delta_from_name=DeltaTable.forName(spark,silver_table_name)
table_location=delta_from_name.detail().select('location').collect()[0].location
delta_from_path=DeltaTable.forPath(spark,table_location)
print(delta_from_path.detail())
on command:
DeltaTable.forPath(spark,table_location)
see images for two different versions (for version 10.4, also by name failed)
databricks version 11.2

databricks version  10.4

/192116659-e398dcd2-e017-4d12-a2b8-6175093ecca8.png)

Observed results
DeltaTable.forPath(spark,silver_table_uri)
/databricks/spark/python/delta/tables.py in forPath(cls, sparkSession, path, hadoopConf)
385 jsparkSession: "JavaObject" = sparkSession._jsparkSession # type: ignore[attr-defined]
386
--> 387 jdt = jvm.io.delta.tables.DeltaTable.forPath(jsparkSession, path, hadoopConf)
388 return DeltaTable(sparkSession, jdt)
389
/databricks/spark/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/java_gateway.py in call(self, *args)
1319
1320 answer = self.gateway_client.send_command(command)
-> 1321 return_value = get_return_value(
1322 answer, self.gateway_client, self.target_id, self.name)
1323
/databricks/spark/python/pyspark/sql/utils.py in deco(*a, **kw)
200 # Hide where the exception came from that shows a non-Pythonic
201 # JVM exception message.
--> 202 raise converted from None
203 else:
204 raise
AnalysisException: s3://..... is not a Delta table.

Expected results

delta table to be created using the valid path
Further details
The behaviour is incosistent and differ between cluster versions (as seen in the snapshots
As can be seen, same command pass on 1st command, but fail on next command.

Environment information

Delta Lake version:

Spark version: 11.2 (includes Apache Spark 3.3.0, Scala 2.12)

Scala version: 2.12

Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[ ] Yes. I can contribute a fix for this bug independently.
[ X] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[ ] No. I cannot contribute a bug fix at this time.

Closing this as this is not a Delta Lake issue.


Upgrading to Hibernate 5.1.0.  I'm also implementing the PersistenceU…

Upgrading to Hibernate 5.1.0.  I'm also implementing the PersistenceUnitInfoImp#getNewTempClassLoader method, which Hibernate 5.x now uses when bootstrapping the EntityManagerFactory.
Resolves #18

Coverage decreased (-0.2%) to 94.393% when pulling 54d8f5ad3cd67bd3bfe0ab9824e5019c9877a72f on issue/18/upgrade-to-hibernate-5.1 into 91e9c86f7d1e98339bace209ce4988b98b4dd68f on master.

Coverage decreased (-0.2%) to 94.393% when pulling dc56da5c0472aaaf3b9bb0b3f61a3babf6b31a54 on issue/18/upgrade-to-hibernate-5.1 into 91e9c86f7d1e98339bace209ce4988b98b4dd68f on master.

Coverage decreased (-0.3%) to 94.299% when pulling 29d49b728985ddeff07a952066250fc82dec3d8d on issue/18/upgrade-to-hibernate-5.1 into 91e9c86f7d1e98339bace209ce4988b98b4dd68f on master.

Coverage decreased (-0.3%) to 94.299% when pulling d59d9396f2b252a647fa800f6a34d747a65efe6b on issue/18/upgrade-to-hibernate-5.1 into 91e9c86f7d1e98339bace209ce4988b98b4dd68f on master.


New sitemap

Sitemap with structure of the different pages needed for the website. We can take the existing ones from the current site and add gaps that exist today
please add @denise to this task.  Keywords and sitemap are tied and I want to make sure we are optimizing.  I can take a lead role on this if you would like

atsign 2.0 Proposed Sitemap.pdf

@aannttoo it would be cool to have this in Mermaid so that it's more dynamic. We can generate SVGs from the Mermaid markup (like on @cconstab GitHub Profile)

Thanks @cpswan  - Will look into it today.

@aannttoo what are next steps?

@ksanty I'm working on a design and will share a draft this afternoon. I'm using the sitemap and staging site as a reference.

@ksanty I'm working on a design and will share a draft this afternoon. I'm using the sitemap and staging site as a reference.

TY!

We will roll with the version we have now. Future iterations can be made as needed.


ActiveRecord::JDBCError FATAL: connection requires a valid client certificate

Our jruby rails app is receiving "ActiveRecord::JDBCError: FATAL: connection requires a valid client certificate" when trying to connect to postgres over ssl.  I know there is some configuration to be done on the java side to allow a truststore/keystore.
I have tried to implement this and I am sure I am missing some configuration. I found this note which states:
# JRuby/JVM needs to be started with :
      #  -Djavax.net.ssl.trustStore=mystore -Djavax.net.ssl.trustStorePassword=...

Does anyone know how to implement this?
This is what I have done so far:
jruby version: jruby <IP_ADDRESS> (2.5.3) 2019-04-09 8a269e3 OpenJDK 64-Bit Server VM 25.252-b09 on 1.8.0_252-b09 +jit [linux-x86_64]
Rails version: Rails <IP_ADDRESS>
active record jdbc adapater: activerecord-jdbc-adapter (50.7-java)
Setting up keystore:
# adopted from https://stackoverflow.com/questions/31055623/google-cloud-sql-with-ssl-from-compute-engine-and-or-external-network

1)Convert server ca to der format
  openssl x509 -outform der -in server-ca.pem -out server-ca.der

2)Create your own keystore
  keytool -keystore mykeystore -alias postgresssl -import -file server-ca.der

3)Convert x509 certificate and private key to a pkcs12 file:
  openssl pkcs12 -export -in client-cert.pem -inkey client-key.pem -out client.p12

4)Convert the pkcs12 file to a java keystore:
  keytool -importkeystore -srckeystore client.p12 -srcstoretype PKCS12 -destkeystore keystore.jks -deststoretype JKS

On the rails side:
# database.yml
default: &default
  adapter: postgresql
  encoding: unicode
  username: <%= ENV["username"] %>
  password: <%= ENV["password"] %>
  host: <%= ENV["database_host"] %>
  database: <%= ENV["database_name"] %>

# Am I missing configs here? This is our config in a normal rails environment
production:
  <<: *default
  sslmode: allow
  sslrootcert: <path to ssl root cert>
  sslkey: <path to ssl key>
  sslcert: <path to ssl cert>

Start rails with java options to point to keystore:
#Pass java options with paths to truststore and keystore which were created as described above

bundle exec jruby -J-Djavax.net.ssl.trustStore=truststorepathhere -J-Djavax.net.ssl.trustStorePassword=passwordhere -J-Djavax.net.ssl.keyStore=keystorepathhere -J-Djavax.net.ssl.keyStorePassword=keystorepasswordhere -S rails s

Any suggestion would be greatly appreciated. I feel like I am close but I have been banging my head on this for a few days now.

The PG JDBC driver pgjdbc has its own properties, see: https://jdbc.postgresql.org/documentation/head/connect.html#connection-parameters. From a cursory look, this seems to match what you mention above.
For activerecord-jdbc-adapter, passing these options is a little different, they need to be in a properties block:
production:
  <<: *default
  properties:
    sslmode: allow
    sslrootcert: <path to ssl root cert>
    sslkey: <path to ssl key>
    sslcert: <path to ssl cert>

(we do understand the sslmode outside of properties, but nothing else). That being said, I never tried a connection with a client certificate.

The PG JDBC driver pgjdbc has its own properties, see: https://jdbc.postgresql.org/documentation/head/connect.html#connection-parameters. From a cursory look, this seems to match what you mention above.
For activerecord-jdbc-adapter, passing these options is a little different, they need to be in a properties block:
production:
  <<: *default
  properties:
    sslmode: allow
    sslrootcert: <path to ssl root cert>
    sslkey: <path to ssl key>
    sslcert: <path to ssl cert>

(we do understand the sslmode outside of properties, but nothing else). That being said, I never tried a connection with a client certificate.


Update the  CASE REVIEW #1761298

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

This PR appears to be a duplicate of the update in https://github.com/MicrosoftDocs/azure-devops-docs/pull/10134, except for the case number referenced in the PR title.

@v-abiss  Can you remove the javascript update since it is in #10134?

@v-abiss can you please change the branch to main instead of master?

Some of this PR seems to be a duplicate of https://github.com/MicrosoftDocs/azure-devops-docs/pull/10135, I'll see if I can fix it up

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

#sign-off

@steved0x: I'm sorry - only the author of this article, @juliakm, @ramiMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

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


Incompatibility with new version of Controlify mod (2.0.0 beta)

Mod Name
Controlify
Mod Homepage
https://modrinth.com/mod/controlify
Minecraft version
1.20.1
Describe the bug
Mod initializes, but gamepad controls and config screen don't work
Steps to reproduce
Usual installation

Install connector, extras, ffapi, controlify
Install yacl (i used 3.4.2 forge version, 3.4.4 always crash when I try to edit fabric mods)
Enter game and try to control it with gamepad or open it's config screen (options - controls - controller). It will not work

How to make mod work
4. Instead of beta versions install controlify 1.6.0
5. Open the game with it and download libraries, calibrate controller
6. Close the game
7. Replace controlify 1.6.0 with newer versions
8. Open minecraft again and it will work
Logs
https://gist.github.com/Axi0L/3fc9dd6e370575f03f089e892ddbcda5
Additional context
After I replaced version back and forward gamepad worked. Then I deleted controlify.json and open minecraft again, I got the same issue I had at the beginning.

Duplicate of https://github.com/Sinytra/Connector/issues/971


Disabled TLS certificate check

The field InsecureSkipVerify controls whether a TLS client verifies the server's certificate chain and host name. If set to true, the client will accept any certificate and any host name in that certificate, making it susceptible to man-in-the-middle attacks.
Impact: man-in-the-middle attacks

I will work on this issue

Our Internal PR has been opened for review , the code change will updated soon

@Nanmozhi22 thanks for sharing your progress and feel free to tag me on your internal PR I'll be happy to review it

@Nanmozhi22 I think that @gtully is working on an alternative solution. Could you share the progress on your internal PR?

Incorporated by #1038


ONT-21 - Import via Excel workbook or CSV

Changes

Added Traits sidebar menu items with placeholder view pages
Added file selector component
Added progress bar component
Added trait upload service

PUT / GET

Added imported traits datatable
Added Import traits page

Running

New dependencies were added for xstate & bulma 0.9 so you'll need to do npm install prior to running.
You'll need the latest develop branch of bi-api

Design

Import traits page most state management done with with XState

State chart diagram
XState visualizer only supports javascript so just converted on the typescript playground
More verbose but formalizing helps me manage more complex state

Left modal management out

Tried to break out file importing components so they can be reused as-is in other file import pages
Table loading state for when there is a lot of data (mostly an issue due to pagination mocking currently for large files)
Bulma has some new spacing helpers which are very useful

Testing

I created a large trait file with 50,0001 traits for testing the progress bar and table loading

File available here
Other test files available in the bi-api repo

Acceptance Criteria

See ONT-21 trello card for the acceptance criteria

New cards were created due to the narrowed scope of ONT-21

 ONT-21 - Import via Excel workbook or CSV

Overall the import function performs well and as expected, using Firefox/MacOS.
Known issue: on MacOS Mojave 10.14.6 using Chrome v.84.0.4147.89, any .xls files return an 'Unsupported mime type' error. This is because Chrome/Mac returns type application/octet-stream rather than the expected application/vnd.ms-excel type. However, .csv and .xlsx files can be imported (and throw proper errors for missing columns, etc) as expected.
Nick and I talked, and the plan is to write a new card for this browser-specific and type-specific bug (because the fix is backend rather than front-end).

Added card for this: https://trello.com/c/4KAHzhuG

You will need to assign your user to the program for this to work, to assign your user, run the following query in the DB,
insert into program_user_role (user_id, program_id, role_id, created_by, updated_by)
select bi_user.id, program.id, role.id, bi_user.id, bi_user.id from bi_user
join program on program.name = 'New Program 2'
join role on role."domain" = 'breeder'
where bi_user."name" = 'BI-DEV Admin'

Nice work! it meets all the acceptance criteria when I run it. One minor quip: I missed in my initial once-over the asterisks showing required fields in the downloaded template, so maybe highlight those columns to visually call attention to them.
We talked already about eventually using magic numbers to validate the excel file format. That could be done eventually as part of the file picker or just on the backend or both. For reference, the card I created for that is INF-186.

Thanks, I added a card to highlight required field in the file template: https://trello.com/c/dEqYwUpo. I linked your card and my card for the file validation since they overlap.


Delete some extra space

Delete some extra space

/approve
@kbhawkey, I'll wait for you and @xichengliudui to resolve the open questions.

/lgtm


How to implement phi-3-vision-128k-instruct (onnx_runtime_genai) with simpleAI ?

Bravo for your work, there is potential in this project!
I would like to use this project to create a compatible openai api for the model: phi-3-vision-128k-instruct (onnx version in the onnx_runtime_genai project).
First link is the tutorial for the onnx model, second one is the github project.
Only, I don't really understand how to go about it. Could you help me ?

I think i need the image support modification

Hi @Youho99, thanks for the kind words! I've been busy elsewhere these last months and haven't worked on this for a while, but happy to answer your question.
We would indeed have to implement the /images/ endpoints to make it work with vision models. That shouldn't be too hard to do (I might even find a bit of time for this) and would be a nice addition to the project.

Hi @lhenault !
I can help implement this feature.
This will allow me to understand in more depth how the whole system evolves.

We would indeed have to implement the /images/ endpoints to make it work with vision models. That shouldn't be too hard to do (I might even find a bit of time for this) and would be a nice addition to the project.

In my use, I do not want to "generate" an image however!
I want to use a VLM, so call the chat/completions endpoint, giving the image and the prompt as input

Oh ok, thanks for clarifying. I've done a bit of research about that, and I've found this thread explaining how to structure a message for this purpose.
Looks like it's still in beta, and expects the user to first upload the image to provide its URL.
If you want to implement this, I think you might want to skip the uploading part in a first time, and use an existing image online. On the model side of things, you can do something like this and then simply pass the input to the VLM.

In my case, I will use base64 images
I already have my model which works correctly in an API that I created (which is not Openai compatible)
So the backend of the model, I know it works, my question marks are not there.
Rather, it's the whole system. I started to look in depth, and I already saw that the code could not handle "content" correctly for an image for example.
I'm currently working on changing this, I'm going from bugs to crashes :laughing:


MySQL module not listed in Rust docs

mysql module is not listed in the docs.

@mjovanc, shouldn't running cargo doc --all should capture the other packages

@chaseWillden We are using cargo publish via a plugin in the release.yml workflow: https://github.com/katyo/publish-crates/tree/main
The cargo publish command should automatically run this.


packaging takes very long time

I am not sure if I am the only one who is facing this problem but
Packaging takes hell lot of time. May be I am missing some settings or something.
Last these three steps are the slowest, build stucks here for long long time.
[info] Merging 'org/apache/http/annotation/Immutable.class' with strategy 'deduplicate'
[info] SHA-1: WrappedArray(26, -125, -113, 64, 37, 94, -6, -79, -79, 41, 34, -92, 42, 7, -72, 31, -97, -57, 55, -2)
[info] Packaging ~/workspace/myproj/xyz/target/xyz-assembly-1.0.jar ..
[info] Done packaging.
[success] Total time: 724 s, completed 31 Jan, 2013 1:51:10 PM

Please let me know if anyone need more info ?

@joeroot I have the same issue of sbt assembly taking much longer in a VirtualBox guest (even with access to all cores) than the host. Did you ever find a way around this?

That's probably because disk i/o is more expensive inside a VM (those files need to be read in order to determine their SHA1 hashes)

@fblundun: I never found a way around this, instead I run set assembly from within Mac, which is slightly frustrating.
@iangelov: It could be i/o, but that seems quite extreme. My projects don't have that many files, and I tend to end up crashing, rather than completing very slowly.

One idea that was suggested by @jsuereth was that sbt-assembly try reading into the JAR directly without unzipping first. ymmv/pr welcome on this front.

If you're using vboxfs (the Vagrant default) there's no way around this. It's slow. You should use nfs instead.

@eed3si9n I have the exact same problem; my project is barebones, spark, akka, scala 2.11.8 - was there an option to disable calculation of SHA1 added?  Assembly is taking 774+ seconds
import Dependencies._
resolvers ++= Seq(
  Resolver.sonatypeRepo("releases"),
  Resolver.sonatypeRepo("snapshots")
)
lazy val root = (project in file(".")).
  settings(
    inThisBuild(List(
      organization := "me.free",
      scalaVersion := "2.11.8",
      version := "0.1.0-SNAPSHOT"
    )),
    name := "BondAgent",
    libraryDependencies ++= Seq(akkDeps ++ Seq(scalaCheck)
      ++ Seq(shapelessDep)
      ++ sparkDeps
      ++ Seq(redisScala)
      ++ argonautDeps ++ sparkConflictResolverDeps).flatten
  )
assemblyMergeStrategy in assembly := {
  case PathList("org", "aopalliance", xs@_*) => MergeStrategy.first
  case PathList("javax", "inject", xs@_*) => MergeStrategy.first
  case PathList("javax", "servlet", xs@_*) => MergeStrategy.first
  case PathList("javax", "activation", xs@_*) => MergeStrategy.first
  case PathList("org", "xml-apis", x@_*) => MergeStrategy.first
  case PathList("javax", "stax-api", x@_*) => MergeStrategy.first
  case PathList("org", "commons-collections", x@_*) => MergeStrategy.first
  case PathList("org", "jcl-over-slf4j", x@_*) => MergeStrategy.first
  case PathList("org", "apache", xs@_*) => MergeStrategy.first
  case PathList("com", "google", xs@_*) => MergeStrategy.first
  case PathList("com", "esotericsoftware", xs@_*) => MergeStrategy.first
  case PathList("com", "codahale", xs@_*) => MergeStrategy.first
  case PathList("com", "yammer", xs@_*) => MergeStrategy.first
  case "about.html" => MergeStrategy.rename
  case PathList("META-INF", xs@_*) => MergeStrategy.discard
  case "META-INF/ECLIPSEF.RSA" => MergeStrategy.first
  case "META-INF/mailcap" => MergeStrategy.first
  case "META-INF/mimetypes.default" => MergeStrategy.first
  case "plugin.properties" => MergeStrategy.first
  case "log4j.properties" => MergeStrategy.first
  case "overview.html" => MergeStrategy.first
  case x =>
    val oldStrategy = (assemblyMergeStrategy in assembly).value
    oldStrategy(x)
}
assemblyOption in assembly := (assemblyOption in assembly).value//.copy(includeScala = false)
assemblyJarName in assembly := "BondAgent.jar"

This is my assembly sbt -
import Versions._
import sbt._

object Dependencies {
  lazy val scalaCheck: ModuleID = "org.scalacheck" %% "scalacheck" % "1.12.2" % "test"
  lazy val akkaStream: ModuleID = "com.typesafe.akka" %% "akka-stream" % akkaStreamV
  lazy val akkaActor: ModuleID = "com.typesafe.akka" %% "akka-actor" % akkaV
  lazy val akkaSlf4j: ModuleID = "com.typesafe.akka" %% "akka-slf4j" % akkaV
  lazy val akkDeps = Seq(akkaStream, akkaActor, akkaSlf4j)
  lazy val sparkCore: ModuleID = "org.apache.spark" %% "spark-core" % sparkCoreV
  lazy val sparkSql: ModuleID = "org.apache.spark" %% "spark-sql" % sparkCoreV
  lazy val sparkDeps = Seq(sparkCore.exclude("org.eclipse.jetty.orbit", "javax.servlet").
    exclude("org.eclipse.jetty.orbit", "javax.transaction").
    exclude("org.eclipse.jetty.orbit", "javax.mail").
    exclude("org.eclipse.jetty.orbit", "javax.activation").
    exclude("commons-beanutils", "commons-beanutils-core").
    exclude("commons-collections", "commons-collections").
    exclude("commons-collections", "commons-collections").
    exclude("xml-apis", "xml-apis").
    exclude("javax.xml.stream", "stax-api")
    exclude("com.esotericsoftware.minlog", "minlog"), sparkSql.exclude("org.eclipse.jetty.orbit", "javax.servlet").
    exclude("org.eclipse.jetty.orbit", "javax.transaction").
    exclude("org.eclipse.jetty.orbit", "javax.mail").
    exclude("org.eclipse.jetty.orbit", "javax.activation").
    exclude("commons-beanutils", "commons-beanutils-core").
    exclude("commons-collections", "commons-collections").
    exclude("commons-collections", "commons-collections").
    exclude("xml-apis", "xml-apis").
    exclude("javax.xml.stream", "stax-api")
    exclude("com.esotericsoftware.minlog", "minlog"))
  lazy val funcDeps: ModuleID = "org.typelevel" %% "cats-core" % catsCoreV
  lazy val shapelessDep: ModuleID = "com.chuusai" %% "shapeless" % shapelessV
  lazy val hadoopClientDep: ModuleID = "org.apache.hadoop" % "hadoop-client" % hadoopCV
  lazy val parquetClient: ModuleID = "org.apache.parquet" % "parquet" % parquetV
  lazy val redisScala: ModuleID = "com.github.etaty" %% "rediscala" % redisScalaV
  lazy val argonautDeps: Seq[ModuleID] = Seq("io.argonaut" %% "argonaut").map(_ % argonautV)
  lazy val sparkConflictResolverDeps: Seq[ModuleID] = Seq("com.fasterxml.jackson.core" % "jackson-core" % "2.8.7",
  // test deps
    "com.fasterxml.jackson.core" % "jackson-databind" % "2.8.7",
    "com.fasterxml.jackson.core" % "jackson-annotations" % "2.8.7",
    "com.fasterxml.jackson.module" %% "jackson-module-scala" % "2.8.7",
    // With spark, to resolve java.lang.ClassNotFoundException: org.w3c.dom.ElementTraversal (error)
    //"xml-apis" % "xml-apis" % "1.4.01",
    // With spark, to resolve large stack trace on java.lang.ClassNotFoundException: de.unkrig.jdisasm.Disassembler (warning)
    "org.codehaus.janino" % "janino" % "3.0.7")
}

Not sure how to make it fast but it is affecting my development timelines - any help, suggestions, rants are welcome...

@AnirudhVyas are you building in a VM?

no - build on macbook os x

@AnirudhVyas Do you have SSD drive on your MacBook?
See https://github.com/sbt/sbt-assembly#other-things, and try turning off any of the optional features and see if it helps your situation.

I have an ssd, let me see thanks. Will report back...

not much difference - well I gained about a 100 second if thats a consolation ... do you know where the bottleneck is? it seems to be taking a while in packaging... Honestly I have been using this awesome plugin for a while, but I never used it with spark, I used to tar up, thought I'd give it a try while putting it along with sbt on a fresh project - something is missing ...

ok for now I made my app a spark submit - i was using a single package for a reason, with provided spark dependencies, less things to deal with, it completes in 180s ... very doable i think. But it would be great if know where the bottleneck was/is so that perhaps it could be improved? I am not implying that you haven't tried already - my apologies if I come across as blunt...

I personally haven't run into performance issues so I haven't really looked at current bottlenecks. If there's some specific issue around, feel free to create a sample project that someone could look into.

hello from 2019. I had this exact problem (yes,old project old version of sbt, but still) and doing sbt clean followed by assembly solved it.
YMMV

I'm on sbt 0.12.01 and macOS.  Every once in a while, the assembly goal simply hangs on the packaging jar step. Will upgrading my sbt help?

any updates on this thread it's hanging for long time in the ubuntu 18.04 LTS @karthicks @noahlz @devmanhinton ?

Wowwow found my way back to this issue via google. I have no insight except trying what past me did (clean assembly)


Redis Lego: Find large keys in redis

Write a lego that locate large keys in Redis (size is a given parameter)

I see this in production - I think we can close


There are redundant asset files after packaging

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
This is my project folder

After packaging, these resource files in the build folder are redundant

Reproducible demo
No response
Steps to reproduce
This is my project folder

After packaging, these resource files in the build folder are redundant

Expected behavior
These three folders should not exist
Actual behavior
This is my project folder

After packaging, these resource files in the build folder are redundant

Your environment

Public source code:
Public site URL:
Docusaurus version used:
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):
init env

Self-service

[X] I'd be willing to fix this bug myself.

Hi! The two invariants are: (1) static files are always copied to the output as-is. (2) imported files are always passed through loaders and emitted as separate assets.
If you import something from static folder, you will end up with two copies. One will be optimized/hashed/bundled, while the other is verbatim copied. They may not be equivalent. Generally, avoid putting things you will import in the static folder.
See #5362


Add the new 2023 ESC menu

Add the new 2023 roblox ESC menu to the bloxstrap menu options list as a v4 option, fast flags>in-game menu version>"version 4 (2023)"

I already added this for the next version. Just put it in the checklist so it's clearer that it's been added.


Adding and improving DEA Maps links on product pages

This PR originated from https://github.com/GeoscienceAustralia/dea-knowledge-hub/pull/274 but is more focussed and single-purpose.
Adding DEA Maps links on product pages

These links appear on the product pages as "See it on a map"

Product contact approvals:
The RED (Cedric) related changes are commented out so that they don't overlap with the RED project.

Product(s)
Contact
Status

Wetlands Insight Tool, WO, WO Stats, WO Sentinel-2
Bex
Approved

DEA Mangroves, High and Low Tide Imagery, Intertidal Extents
Stephen
Approved

FC, FC Percentiles, WO, WO Stats, Tasseled Cap
Cedric
Pending

Surface Reflectance products
Imam
Approved

Bex, Stephen, and Imam have reviewed these changes.


midi processor dispatch() not catching notes on channels other than 0

When running the midi_monitor example, I noticed it only registers midi events as note on/off  if they are coming in on the first midi channel.

I'm sorry I missed this, and it took so long. This is fixed now in develop and master. Silly bug! So simple:
https://github.com/cycfi/q/commit/86685e26b2a9d5f661b68fb2fed0401070ff29d9

I'm sorry I missed this, and it took so long. This is fixed now in develop and master. Silly bug! So simple:
https://github.com/cycfi/q/commit/86685e26b2a9d5f661b68fb2fed0401070ff29d9


Bagel Update 8 again

Changelog
-added circuit imprinters to science
-added zebra scarf to mime room
-added a 2nd pinpointer to the map in the vault
-added other HoS parade uniform so now there's pants and skirt version
-added a few more sources of gold to the map but most of it is still in the vault
-added several maint loot spawners to the map (thanks peptide)
-added a public bathroom across from youtool room so the map could have a mirror
-added sinks to bar so bartenders can give people nice refreshing glasses of woter
-added emergency lights around the map
-made disposals disposable
-skub

actually hang on i placed some lights wrong


Add log to understand why some AgentTablesQueryConfigurationTable disappear

Description
Task: https://github.com/dust-tt/tasks/issues/1295
Turns out this https://github.com/dust-tt/dust/pull/7256 was not enough and some users still report a lost configuration.
This is super weird since we removed the destroy, so to confirm data is deleted I'm adding some logs that are meant to stay here temporarily.
Risk
Can be rolled back.
Deploy Plan
Deploy front.

testing locally before shipping (at least the hook)

🙏


Upgrade golang version

Description
Upgrade golang version to 1.22
How can this be tested?
unit-tests
Checklist

[x] Unit tests have been updated/added
[x] PR is labeled accordingly with a single label
[x] I have read and understood the contribution guidelines

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (728b28f) 67.72% compared to head (ee52e10) 58.93%.
 Consider uploading reports for the commit 9694173 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2701      +/-   ##
==========================================
- Coverage   67.72%   58.93%   -8.79%
==========================================
  Files         250      300      +50
  Lines       13637    16935    +3298
==========================================
+ Hits         9235     9981     +746
- Misses       3593     5824    +2231
- Partials      809     1130     +321

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[BUG] Error: Got unexpected undefined, with-content-script-ui

What happened?
import cssText from "data-text:~/content.css"
import type { PlasmoContentScript } from "plasmo"

export const config: PlasmoContentScript = {
  matches: ["https://www.plasmo.com/*"]
}
export const getStyle = () => {
  const style = document.createElement("style")
  style.textContent = `${cssText}`
  return style
}

window.addEventListener("load", () => {
  console.log("content script loaded")

  document.body.style.background = "pink"
})

import someCoolImage from "data-base64:~assets/grey.png"

function IndexPopup() {
  return (

  )
}

export default IndexPopup

gitlab ci/cd
$ PARCEL_WORKERS=process pnpm build
><EMAIL_ADDRESS>build /data/gitlab-runner/builds/KySvgCD_/0/xxx
> plasmo build
🟣 Plasmo v0.44.0
🟠 The browser extension development framework.
🔵 INFO   | Prepare to bundle the extension...
🔴 ERROR  | Got unexpected undefined
          | Error: Got unexpected undefined
    at nullthrows (/data/gitlab-runner/builds/KySvgCD_/0/xxx/node_modules/.pnpm/nullthrows@1.1.1/node_modules/nullthrows/nullthrows.js:7:15)
    at Object.run (/data/gitlab-runner/builds/KySvgCD_/0/xxx/node_modules/.pnpm/@parcel+core@2.6.2/node_modules/@parcel/core/lib/requests/DevDepRequest.js:133:54)

Version
Latest
What OS are you seeing the problem on?
Linux
What browsers are you seeing the problem on?
No response
Relevant log output
No response
(OPTIONAL) Contact Details
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

content.tsx and popup.tsx use data-xxx:~

@crazyair I couldn't reproduce this. I have both my content and popup using a data-scheme, with a similar import path as yours

One thing that was diff for me is:
  style.textContent = cssText

I assign the variable instead of templating it.

in mac ok，but in gitlab ci error

If you set PARCEL_WORKERS=0 does it work?

This make me to wonder what the heck is gitlab putting in their CI xD....

yes, use PARCEL_WORKERS=0

This make me to wonder what the heck is gitlab putting in their CI xD....

run PARCEL_WORKERS=process pnpm build

import someCoolImage from "data-base64:~assets/grey.png"
changed
import someCoolImage from "url:~assets/grey.png"
is ok

@crazyair how's pnpm installed in this gitlab CI?

@crazyair how's pnpm installed in this gitlab CI?

npm i -g pnpm;
pnpm version 6.29.2

hmm, pnpm should be >7.x.x

hmm, pnpm should be >7.x.x, I think that's the root cause maybe :d.....

I can't use 7. Many projects use 6

Deployment should be pnpm independent

Can you try just npm instead (no pnpm)?

Can you try just npm instead (no pnpm)?

$ PARCEL_WORKERS=process npm run build
><EMAIL_ADDRESS>build
> plasmo build
🟣 Plasmo v0.44.0
🟠 The browser extension development framework.
🔵 INFO   | Prepare to bundle the extension...
🔴 ERROR  | Got unexpected undefined
          | Error: Got unexpected undefined

Can you push gitlab try it?

default config

@crazyair can you run pnpm dev -- --verbose and paste the log. I'll get down on this bug later after sleep xd

@crazyair BTW can you share the gitlab config you used to setup your project.

  script:
    - "pnpm i"
    - "PARCEL_WORKERS=process npm run build -- --zip"

Try this:
image: node:latest

test_async:
  script:
    - npm install -g pnpm
    - pnpm i
    - pnpm build -- --verbose

It works for me.

pnpm no v7

pnpm no v7

How do you setup your pnpm? And why stick to older version?

V7 will lead to different pnpm lock contents, but many projects are using V6

Try this:
image: node:latest

test_async:
  script:
    - npm install -g pnpm
    - pnpm i
    - pnpm build -- --verbose

It works for me.

try v6 is ok ？ i think ok！

Try this:
image: node:latest

test_async:
  script:
    - npm install -g pnpm
    - pnpm i
    - pnpm build -- --verbose

It works for me.

use v6 is ok ?
use npm is ok ?


Remove WeakRegistry references (class split and removed in Pharo 12)

with @AlessHosry
In the Pharo 12/Moose 12, this package cannot load anymore, due to the use of WeakRegistry.
The class was refactored into FinalizationRegistry.
We replaced the references in the following methods :
ODBCConnection>>#initializeStatements
ODBCConnection class>>#newConnections
Unfortunately, the new file format has created a lot of noise in the diff.

We close this in favour of #13 which is merged


Update footer styles per mock

Addresses #209
Also moving some images into subfolders

Completed in #468 -- thank you for filling in on this while it was still WIP :)


JS translations don't seem to get exposed.

Hello,
I am unable to use the TOTP module using dutch translations (nl).
I have the idea the JS lang files aren't exposed / are not detected correctly by the bundle file.
When I add the translations to the MFA module, the translations are detected and visible.
I wonder if the path of the lang files is correct (they say that Translation Tables should be automatically detected).
Maybe something from https://docs.silverstripe.org/en/4/developer_guides/i18n/ might clear things up.
Could you please look into this?
The SS installation is version 4.9.
Acceptance criteria

Login screen and all MFA screen can be translated and the translation show.
MFA registration screen in CMS display translation when available

PRs

https://github.com/silverstripe/silverstripe-admin/pull/1523
https://github.com/silverstripe/silverstripe-totp-authenticator/pull/123
https://github.com/silverstripe/silverstripe-mfa/pull/497
https://github.com/silverstripe/silverstripe-webauthn-authenticator/pull/141

I had a look at the underlying Dutch translations. The vast majority of the strings in these screenshots have bee translated and should be showing in Dutch ... but they are not.

There are a couple of problems at play here.
The first and biggest is that nl is not a valid locale setting to use. i18n is supposed to take 2-character locales and find the appropriate correct-formatted locale for it, but it seems to be failing to do so. The workaround for this is to use nl_NL as your locale, but I'll see what I can do about fixing i18n. Worst case I'll do some documentation to make this clearer.
The second is that some (not many) of the translation strings expected by the js aren't available in the client/lang dir and therefore are never translated. That's a much more minor issue, ultimately, but one that is easy to resolve so I'll resolve it.

@GuySartorelli You've used a POC branch to extract more of the js translations - could you create a new issue for that one explaining the issue and put the POC PR there?  As well as the unmerged "MNT Add missing translation strings to js src lang". I'm a bit confused as to why some but not all of the JS translations weren't picked up, I'm not sure if it's restricted to mfa modules or if it's more widespread

New issue created: https://github.com/silverstripe/silverstripe-framework/issues/10814

All PRs have been merged and there's no further work to be done. Closing.


less duplication in getAssets methods in @utils/aquarius

With ... we have:

getAssetsFromDidList()
getAssetsFromDtList()
retrieveDDOListByDIDs()

Where all of them have exact same fetch code with a slightly different filter being sent, and retrieveDDOListByDIDs() just returns results in different order.
This could be one method getAssets() or getAssetsFromList() or getAssetsFromIds():
export async function getAssets(
  list: string[], // list of DIDs or Datatoken addresses
  filter: string, //
  chainIds: number[],
  cancelToken: CancelToken
): Promise<Asset[]> {...}

But as utility methods they could be handy to keep separate, in that case we should at least have them all use a shared fetching method, and then only modify input/output differently in separate methods

They make sense as util functions because they all create the filter and then use queryMetadata , except  retrieveDDOListByDIDs that has some weird filters after fetching the data


training on single channel pictures

The loss function requires a vgg network, but there is no pre-trained vgg network for single-channel images. How to deal with this problem?

@njuhang solve by yourself


Interface with display

The display uses SPI, I think.
Check out orientation - is it going to be easy to rotate the display?

Rotation is easily changed via a flag. 2 SPI pins need to be routed, any pin can be chosen for SS.

There may be issues with long-term durability with this display - the docs say the min/max temperature is -10 to 60C. Storage in direct sunlight is bad as is humidity.


Completely Disable Awake option

Description of the new feature / enhancement
I see an old ticket that was not addressed and closed. I also have the same issue, and I won't be able to use the latest version of PowerToys until we have the ability to disable Awake option. My company doesn't allow Awake feature to be used in our machines. Could you please fix it?
Scenario when this would be used?
When a company has a policy not to disable screen lock, it will help them to be still able to use PowerToys.
Supporting information
No response

GPO is how we'd suggest doing this at a company level.  https://learn.microsoft.com/en-us/windows/powertoys/grouppolicy


health check issue in rancher

Hi,
I recently started using rancher. I ran EKS in which was successful, later when i tried to launch k8s in direct digital ocean or EC2 it showing this issue.
"[controlPlane] Failed to bring up Control Plane: Failed to verify healthcheck: Failed to check https://localhost:6443/healthz for service [kube-apiserver] on host [<IP_ADDRESS>]: Get https://localhost:6443/healthz: can not build dialer to c-pgpsr:m-mgjfc, log: I0730 10:08:17.005398 1 storage_rbac.go:279] created rolebinding.rbac.authorization.k8s.io/system:controller:bootstrap-signer in kube-public"

please help me to resolve this issue
i used "sudo docker run -d --restart=unless-stopped -p 8080:80 -p 4443:443 rancher/rancher
" to install rancher in my local machine.

Useful
Info

Versions
Rancher v2.0.6 UI: v2.0.61

Access
local admin

Route
global-admin.clusters.index

Please see https://github.com/rancher/rancher/issues/12657#issuecomment-391362566 and see what caues the issue

still showing same issue
i reinstalled rancher "sudo docker run -d --restart=unless-stopped -p 80:80 -p 443:443 rancher/rancher"

The comment shows 3 steps you can check to determine where it's failing. It does not say to reinstall Rancher.
If you check the 3 steps (1. Check connectivity from added host to Rancher, 2. check rancher/rancher-agent logging, 3. check LB/proxy configuration), you can determine where it is failing.

Got the same issue - isn't it a problem with rancher trying to access the health endpoint of the kube-apiserver which has a self signed cert?

@dgabrysch make sure you are not hitting this bug with your self-signed cert: https://github.com/rancher/rancher/issues/13831

@janeczku checked that :)

Actually it doesn't even work when I do not provide own certs and simply start the rancher server without any cert options.

Hi @bibin-skaria,
did you manage to resolve this issue or find a workaround for it? I am having same problem here :(
/ Thomas

If it doesn't work with the default certificates, there is either a network connectivity issue (Agent cannot reach configured server-url using TCP/443) or a proxy configuration blocking the communication. The steps to check are also listed above and it helps diagnosing issues if they are followed and if the output if provided.


Fix/43846

Details

Fixed Issues

$ #43846
PROPOSAL: https://github.com/Expensify/App/issues/43846#issuecomment-2175692058
Tests

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

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

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

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

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

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

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

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

[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/22982173/85542ee9-3882-46b5-aa29-b569ad65acf9

MacOS: Desktop

https://github.com/Expensify/App/assets/22982173/20022533-f63a-4730-bdf0-06627776c697

Hey @rushatgabhane, I changed the initial proposal noting that this fix will do the job.
I believe the issue is that the openReport is not being called when the teacher's unit report is opened.
Therefore, I made different changes to the referTeachersUniteVolunteer function than what was stated in the proposal.
Could you review the PR and tell me your opinion?

@Tony-MK that sounds like a hack. but i won't judge because i haven't seen the updated proposal.
Can you please update your proposal? I'm seeing the old proposal only on the issue

P.S. fix lint errors and fill the checklist. ty!

@rushatgabhane, I have updated my proposal to explain the alterations.
Thanks.

latest update here: https://github.com/Expensify/App/issues/43846#issuecomment-2226715011


Rendering multiple routes using the same component?

Happy Thanksgiving! I'm sitting here in a turkey coma trying to figure out how to get a component/page to update and re-render based on a location change / saga.  Basically, I have a page for viewing different sports (football, basketball, hockey, etc) called League. It loads the appropriate data by triggering an action in componentWillMount.  I don't have a problem when I'm routing to the League component from another page or by direct url, but if I am already looking at a sport (if I'm already viewing the League page), clicking one of the navigation links to switch to another sport changes the location in the url and triggers the router saga 'LOCATION_CHANGE' but it does not trigger the action in componentWillMount -- which I understand, because its already mounted.
I've tried a few things but I feel like I'm over-complicating something that should be pretty simple.  Like from your examples, would it be possible to view one blog post and then click a link within that post to have the page reload another post? Any suggestions would be greatly appreciated!

Hey, hope you out of turkey coma :)
Of course it is possible, depending on you project structure you can either use  or componentWillReceiveProps (or any other lifecycle methods) or you can subscribe in saga for location change and update data (make api request) based on that. There is no really one best way, it should depend on specific case.
In my case, to gain full control over urls and how it's updated I have a saga watching url changes and reflecting it in the store. The rest of components that need to be aware of url change are connected to that store, so every time URL changes it updates the store and the store change updates UI

Closing for inactivity. Here is simple example.


Perf-tests golint checks fix

Fixed golint checks and removed -min_confidence flag from verify-golint.sh as mentioned in #1163
File  clusterloader2/pkg/state/state.go had the following warnings:

exported method GetNamespacesState returns unexported type *state.namespacesState, which can be annoying to use
exported method GetResourcesVersionState returns unexported type *state.resourcesVersionsState, which can be annoying to use

So I exposed both NamespacesState and ResourcesVersionsState types;
I already ran the verify scripts locally without issues, this is my first time contributing in this project so apologies if I made a mistake

/ok-to-test

/approve
@qckzr - please rebase your PR and I will lgtm.

/lgtm
/approve


Allow SDM_ADMIN_CHANNEL to be a slack-connect channel shared with a 3rd party.

Is your feature request related to a problem? Please describe.
Currently you can point accessbot to a channel for approvals to happen. This is set by the environment variable SDM_ADMINS_CHANNEL
When setting this to a local channel to the workspace there are no errors. We have a client who has some requirements to have a 3rd party manage approvals for some infrastucture. This could be done by adding them to the admins channel and sharing that channel via slack.
Currently if you do this the accessbot cannot find the channel.
example error logs
ile "/opt/conda/lib/python3.9/site-packages/errbot/core.py", line 546, in _execute_and_send
for reply in replies:
File "plugins/sdm/accessbot.py", line 102, in access_resource
yield from self.get_resource_grant_helper().request_access(message, resource_name, flags=flags)
File "/errbot/plugins/sdm/lib/helper/base_grant_helper.py", line 27, in request_access
yield from self.__grant_access(message, sdm_resource, sdm_account, execution_id, request_id, flags)
File "/errbot/plugins/sdm/lib/helper/base_grant_helper.py", line 72, in __grant_access
yield from self.__request_manual_approval(message, sdm_object, sdm_account, execution_id, request_id, sender_nick, flags)
File "/errbot/plugins/sdm/lib/helper/base_grant_helper.py", line 96, in __request_manual_approval
self.__check_administration_availability()
File "/errbot/plugins/sdm/lib/helper/base_grant_helper.py", line 156, in __check_administration_availability
raise Exception("An Admin Channel was defined but it's unreachable.")
Exception: An Admin Channel was defined but it's unreachable.
"
Describe the solution you'd like
Allow the sdm_admins_channel have that check and post use the conversations api. https://api.slack.com/apis/connect
Describe alternatives you've considered
inviting the 3rd party to main slack workspace rather than use slack connect.

Hi @tonylokko!
We couldn't reproduce this error. Please update your AccessBot version and try again. Recently we fixed a bug related to configuring the admins channel, and maybe that was the problem.


update a few links in README.md

Thanks for maintaining this repo! I noticed a couple of broken/outdated links in README.md that I have fixed in this PR.

Thanks!


Dynamic Programming. 0-1 Knapsack Problem.

Hi, there. I finished my implementation of Knapsack Problem in Swift. I close old request because I accidentally deleted all my files and didnt save them. I rewrote the algo. Now It would be good if review my solution and give some feedback, or if it is okay can you accept my request. Thanks!

Closing due to inactivity. Feel free to open it up again if you still wish to add this in.


Add e2e test to destroy cluster deployment

Adds e2e test to destroy clusterdeployment and ensure that no DNSZones remain after deletion.

Looks great but something appears to have gone wrong, no uninstall job found.

Looks great but something appears to have gone wrong, no uninstall job found.

Yup, the uninstall job completed and then everything was cleaned up, so there's no uninstall job to output the log for... reworking that part.

Addressed comments, added code to write out the uninstall log. Should hopefully pass e2e now.

Addressed latest comments

/test e2e

/test e2e

@staebler this should be ready, it even passed the e2e :)

well, still need to squash if evereything looks good

squashed, if there are no more comments, this should be ready to merge

/test e2e

/test e2e

/lgtm


Add EMF Only Flag

Description:
Add emf only flag to aws cloud watch logs exporter
Link to tracking Issue:
N/A
Testing:
Unit tests
Documentation:
N/A

vul checked should be fixed with sync from upstream not related to this pr


Detect a long press on the calendar

To create an event where you press for example.

Are you waiting for another edit to merge this pull request?


Hot water boost on demand?

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
Apologies if this is already a standard feature but I cannot work out how to do it and couldn't find easy contact details to ask you.
I currently just have to accept when my heat pump wants to do the hot water run.
Describe the solution you'd like
I'd like to be able to manually flick a switch to tell it to do a hot water run.  And following on from this ability, be able to use HA to automate the hot water run (based on time or solar production) etc.
Describe alternatives you've considered
I can't think of a way I do this with the current features, but I may well have just missed this.
Additional context
.

I've been trying to set up an automation in Home Assistant to change the DHE required temp to a higher value.
However in the Actions> Add action > Device, after I have selected Mastertherm Heatpump, this is where I become unstuck.
In the selectable list of actions I can set values for a plethora of things including heat curves, winter/summer temps etc or I can change modes like the HVAC mode on DHW Control
Unfortunately there is no selectable option for "Set value for DHW Required Temperature" which to my mind is what I'm needing.?!?

I think I have eventually figured it out and have created an automation that sets the DHW required temperature to 50, waits an hour and then sets it to 45.
I've not included a trigger so that I can then either call it with a button (virtual or physical), set it into a schedule or call it from other automations

Hi,
Sorry for slow reply busy day, I don't really have anywhere else at the moment was going to start a thread in the forum but don't have the time much to monitor.
You are about right, you can't really control when the heat pump does hot water as a priority vs not but this is what I have mine is in the UI as an automation and I have a button on the dashboard to override it works fine.  For the switch is HC2 as I have mine setup with two separate water systems so they are not on the main circuit.
I turn it on at 1.30 am and off at 7pm just using the switch.  You can also use it to increase/ decrease requested temperature.
This is my on automation which I trigger twice just in-case it fails the first time for any reason.

I also have an automation to heat the water as high as possible on one night a week for sanity (before the thermostat takes over and heats the rest of the way to 60)


Should easycart enforce integer type on quantities?

I am using intercooler.js to make post request to easycart and looks like not always quantities are stored as integers so when computing totals easycart fails saying Decimal cannot be multiplied by sequence (unicode). I fixed it by converting to integer.
Are you interested in a pull request?
Thanks for your project.
Cheers,
Néstor

Hi. Thanks for the feedback. I'm a bit busy right now. I'll take a look at the issue and your pull request at the beginning of the next week.

Hmmm... That's strange. Both BaseItem.init() and BaseCart.add() call _clean_quantity() to convert the quantities to integer values. Can you provide more information on how to reproduce the issue?
You're welcome to open the pull request.

Hi nevimov, thanks for following up this issue.
I think cart should not allow negative values by just ignoring them, but not rising an exception. Also, setting 0 quantity should mean delete. Would you like me to open another issue to discuss this?

Hi. Yes, open another issue.


Something wrong on editing this extension

Hi,
I've downloaded this extension, but for some reason when I edit the code, nothing happen. For example, on updateMapPosition function I've change default position but is still focusing on Portugal. I know that Qlikview is taking this code because when I delete primary file the map doesn't appear.
What could be happening?

Hello,
This is the Qlik Sense version of the extension. Not QlikView.
In this version the default is pointing to Lund in Sweden where are the Qlik Offices. That value can be found indeed in the updateMapPosition function on line 216 and it is (55.716258,13.221009).
From your comment, I think you are referring to QlikView version of the extension that is in another repository:
https://github.com/rvspt/GoogleMapsAPIv3Toolkit-QV
You can update the map's default value on line 132, changing

accordingly.

Closing as this is related to another extension


fix: add old partitions constructor for backwards compatibility

Issue #
Possible fix for : https://github.com/awslabs/aws-sdk-kotlin/issues/1411
Description of changes
This is re-adding a constructor that was removed from Kotlin to fix backwards incompatibility between versions < 1.2.8 to 1.3.39.
Relevant PR: https://github.com/awslabs/aws-sdk-kotlin/pull/1335
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Could you provide an estimated timeline for when this fix might be available in the next version?
FYI, we are experiencing significant user crashes in production (https://github.com/awslabs/aws-sdk-kotlin/issues/1425). With this fix, we'd like to try the latest version to determine if the issue is resolved.

This has been 2 weeks already and we are facing some crashes due to the old version in our production app. 😞

Any updates ?

I'm working on updating this PR today. Hopefully we'll get it approved and released in the next day or two.

@drayan85 this change has been merged and should be present in today's SDK release v1.3.60.

Thanks


Darkstar RSO  Grand Unified factorio theory Ore generation Bug

Hey having a issue where the lithium from 248k removes the ore patch by Darkstar which makes it impossible to continue darkstar in a
RSO bob's angels 248k & Grand Unified factorio theory modpack.

https://mods.factorio.com/mod/rso-mod/discussion/64a577c172588e8c5a65c98b Related post on RSO

Ore bug.zip


How to write an Ingress controller docs

Does what each of the commits say, main goal is to describe how one writes an Ingress controller with a simple example.

@brendandburns can you reivew if you're not too busy with the release? since Tim is out for a few weeks. There's already a link leading here from user docs, but it's currently pretty bare.

Pretty sure Brendan's busy with the release. Mike danese to the rescue.

Very minor comments. LGTM otherwise.

Done, PTAL

LGTM feel free to merge when image is pushed

Thanks!


Custom note type doesn't appear in Chart creator

Describe your problem here.
The custom note type in my mod won't appear in my mod, can someone tell me what's going on? Here's the link for my mod. https://drive.google.com/file/d/1ELcgBGh6ZBgpTNro_F29rud_GQ37g-AY/view?usp=sharing
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Yes, I removed all weeks except tutorial.

Check to see how if the mod pack is on

I can't access the folder

How?


Fix issue #19

Removed redundant keyword info as in comments in #19

Closed because of #22

This seems to also fix #23


History navigation

When in the History tab, clicking next or previous cannot go to the next month/year.
Ex:

Dec 2017 can't go forward to Jan 2018
Apr 1st can't go back to Mar 31st

Issue fixed with #2


Updated SFML to use github

So you can install with --head.

@JonnyPtn,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks!


Implement a loggregatorreceiver locally that could receive all input from Forwarder Agent in loggregator format

Rather than converting loggregator v2 to OTLP in Forwarder Agent, we we should convert loggregator v2 to OTLP in the OTel collector, this release.
This has the following benefits:

Removes "special" code from the Forwarder Agent that is only used by this release.
Adds functionality to this release that could be extended to eventually allow it to replace Forwarder Agent.

I made this WIP branch with a spike that seemed to work, though I didn't test it thoroughly: https://github.com/cloudfoundry/otel-collector-release/tree/wip/loggregatorreceiver.


Expected ";" when you have a "use Object;" statement.

From @SilbinaryWolf on November 3, 2017 0:42
Silverstripe 3.6 uses the class name "Object" and runs fine in PHP 7.1.
I'm aware Object is a soft keyword and could stop working in future PHP versions, but until then, can this please be fixed.
Example code that gets an error:
https://github.com/SilbinaryWolf/silverstripe-phpstan/blob/master/src/MethodClassReflectionExtension.php#L19
Test repo, should be able to just composer install and test with the language server:
https://github.com/SilbinaryWolf/silverstripe-phpstan-testproject
Copied from original issue: felixfbecker/php-language-server#516

Also to be clear this is not allowed in PHP 7.2 and further according to https://wiki.php.net/rfc/object-typehint.

If it's still possible and fairly simple to support this kind of statement for PHP 7.1 and lower, that'd be fantastic.


How to use functions Moran_I and Geary_C ？

I see functions Moran_I and Geary_C in calculate_moran_I.py. But I don't see how to use them in tutorial. I also don't see any comments

These two functions are used for internal evaluation, so not mentioned in the tutorial. They are implemented based on their definition from Wiki.


Nested donut chart or pie chart

Is there a way to have nested pie or donut chart?. Can not use sunburst because is not size base (https://github.com/krispo/angular-nvd3/issues/471).

Best Regards.

I wanted to achieve something like this too


[Feature] Version 1

Version 1 will be a breaking change from the current versions and include the following:

[x] #29
[x] #22
[x] #21
[x] #20
[x] #26

Due to all of these being essentially breaking changes, they will be part of the v1 branch.

v1 has been merged to main, closing. Will release a v1.0.0 build shortly.


Call evothings.scriptsLoaded() to ensure all libraries are loaded

Examples that use libraries that include other libraries using evothings.loadScript() should call evothings.scriptsLoaded() to ensure all libraries are loaded.
These libraries are for example easyble.js and arduinoble.js.
Example:
document.addEventListener(
    'deviceready',
    function() { evothings.scriptsLoaded(initialize) },
    false);

function initialize()
{
    // Now it is safe to go head and initialise the app,
    // calling plugins and library functions.
}

This is implemented for several examples, but there may be examples that not use this.


Enhancement: Poll based on flow run and task run state names

Description
Updates the the interval for fetching flow and task run related assets to be based on the state so polling only happens when the flow run or task run is "Pending" or "Running"

@zhen0 good idea. We're using isRunningStateType in a few places. But including pending would be a nice touch. Will update 👍

@znicholasbrown the intent is that we should be polling consistently based on globally set rules for which states should poll. That way at implementation we're not making that decision case by case. Helps keep things consistent as well as reduces mental load.

Hm ok I buy it, in which case the name is a little off - what we're saying is "poll on state" but what we should be saying is "stop polling if you're passed a terminal state" or alternatively "only poll in a non-terminal, non-scheduled state". Maybe useStatePolling => useNonTerminalStatePolling? Not sure I like that much but it's at least clearer


Disable source maps or even use microbundle

I've noticed that the UMD build contains a source map, so it could be smaller. It would be cool because you recommend to add it to Git, instead of installing from npm.
I'll be happy to send a PR that either:

Disables source maps in the current webpack config;
Uses microbundle instead of webpack (it's based on Rollup and supposed to make smaller files).

What do you think?

Sounds great!

Looks like it's not that easy ;-(

Even when I disable source maps, webpack still add comments like //# sourceMappingURL=data:application/json;charset=utf-8;base64… in the file itself, and I don't see an easy way to remove them.

I couldn't make Microbundle inline dependencies, though I think there should be a way, maybe I'll try again when they make a new version (there are some PRs that may help).


Combine prompt elements "dir" and "dir_writable"

Would it make sense to combine the two prompt elements "dir" and "dir_writable", so that a read-only directory would change the color of the path (to red)?

Damn, I like that idea!

I agree - that is a really cool idea. Thanks so much for sharing your suggestion, @eumiro!
I want to maintain backwards compatibility, so I think the thing to do, here, is to add the dir_writable functionality to dir, so that users can choose one or the other in terms of functionality. Since the code already exists in dir_writable, this is actually a fairly "easy" PR that someone new to P9k could take on.
I'll let this one hang out for a couple of weeks to see if anyone is interested in submitting this as a contribution =)

This work was completed by @tippl in #592, and is now part of the v0.6.4 release!


[go_router] Pop is not functional when using refreshListenable

Steps to Reproduce

Clone the repo https://github.com/sarbagyastha/router
Checkout the refresh branch
Run the app and tap on "Go to Second page"
Tap on the back button in App Bar.

Expected results: I should be able to pop to the first page.
Actual results: I see the following error.
======== Exception caught by gesture ===============================================================
The following assertion was thrown while handling a gesture:
have popped the last page off of the stack; there are no pages left to show
'package:go_router/src/go_router_delegate.dart':
Failed assertion: line 71 pos 12: '_matches.isNotEmpty'

When the exception was thrown, this was the stack:
#2      GoRouterDelegate.pop (package:go_router/src/go_router_delegate.dart:71:12)
#3      GoRouter.pop (package:go_router/src/go_router.dart:167:32)
#4      GoRouterHelper.pop (package:go_router/go_router.dart:75:35)
#5      GoRouterHelper.pop.<anonymous closure> (package:go_router/go_router.dart:75:8)
#6      BackButton.build.<anonymous closure> (package:flutter/src/material/back_button.dart:104:21)

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.2, on macOS 12.4 21F79 darwin-arm, locale en-NP)
    • Flutter version 3.0.2 at /Users/sarbagya/Development/flutter
    • Upstream repository<EMAIL_ADDRESS>    • FLUTTER_GIT_URL =<EMAIL_ADDRESS>    • Framework revision cd41fdd495 (9 days ago), 2022-06-08 09:52:13 -0700
    • Engine revision f15f824b57
    • Dart version 2.17.3
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/sarbagya/Library/Android/sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_HOME = /Users/sarbagya/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.68.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.42.0

[✓] Connected device (3 available)
    • iPhone 13 Pro Max (mobile) • 52A89D3C-1D4B-42EE-817A-C77345806588 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-5 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 102.0.5005.115

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @sarbagyastha thanks for filing the issue.
This issue is reproducible on the latest stable and master channels.
It seems the issue is caused by using with Stream.periodic.
I also tried with router_stream_refresh that using normal stream and see it works well without exception.

Demo
https://user-images.githubusercontent.com/104349824/174287250-54413985-4ac6-4395-91ea-fdc25981f53a.mp4

Logs
======== Exception caught by gesture ===============================================================
The following assertion was thrown while handling a gesture:
Assertion failed: file:///Users/huynq/.pub-cache/hosted/pub.dartlang.org/go_router-4.0.0/lib/src/go_router_delegate.dart:71:12
_matches.isNotEmpty
"have popped the last page off of the stack; there are no pages left to show"

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49  throw_
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 29:3    assertFailed
packages/go_router/src/go_router_delegate.dart 71:21                          pop
packages/go_router/src/go_router.dart 167:32                                  pop
packages/go_router/go_router.dart 75:35                                       GoRouterHelper.pop
packages/go_router/go_router.dart 75:8                                        <fn>
packages/flutter/src/material/back_button.dart 104:11                         <fn>
packages/flutter/src/material/ink_well.dart 1005:21                           [_handleTap]
packages/flutter/src/gestures/recognizer.dart 198:24                          invokeCallback
packages/flutter/src/gestures/tap.dart 613:48                                 handleTapUp
packages/flutter/src/gestures/tap.dart 298:5                                  [_checkUp]
packages/flutter/src/gestures/tap.dart 269:7                                  acceptGesture
packages/flutter/src/gestures/arena.dart 157:12                               sweep
packages/flutter/src/gestures/binding.dart 449:20                             handleEvent
packages/flutter/src/gestures/binding.dart 425:14                             dispatchEvent
packages/flutter/src/rendering/binding.dart 329:11                            dispatchEvent
packages/flutter/src/gestures/binding.dart 380:7                              [_handlePointerEventImmediately]
packages/flutter/src/gestures/binding.dart 344:5                              handlePointerEvent
packages/flutter/src/gestures/binding.dart 302:7                              [_flushPointerEventQueue]
packages/flutter/src/gestures/binding.dart 285:32                             [_handlePointerDataPacket]
lib/_engine/engine/platform_dispatcher.dart 1105:13                           invoke1
lib/_engine/engine/platform_dispatcher.dart 185:5                             invokeOnPointerDataPacket
lib/_engine/engine/pointer_binding.dart 130:39                                [_onPointerData]
lib/_engine/engine/pointer_binding.dart 543:18                                <fn>
lib/_engine/engine/pointer_binding.dart 496:21                                <fn>
lib/_engine/engine/pointer_binding.dart 210:16                                loggedHandler
Handler: "onTap"
Recognizer: TapGestureRecognizer#3116f
  debugOwner: GestureDetector
  state: ready
  won arena
  finalPosition: Offset(21.0, 20.3)
  button: 1
  sent tap down
====================================================================================================

Sample code (only main classes from repo)

router.dart

import 'package:go_router/go_router.dart';
import 'package:router/first_page.dart';
import 'package:router/second_page.dart';

int count = 1;

final stream = Stream.periodic(
  const Duration(seconds: 1),
  (computationCount) => count = computationCount,
);

final router = GoRouter(
  initialLocation: '/first',
  refreshListenable: GoRouterRefreshStream(stream),
  routes: [
    GoRoute(
      path: '/first',
      builder: (_, __) => const FirstPage(),
    ),
    GoRoute(
      path: '/second',
      name: 'second',
      builder: (_, state) => SecondPage(
        queryParams: state.queryParams,
        extra: state.extra,
      ),
    ),
  ],
  redirect: (state) {
    print(state.location);
    print('Called $count times.');

    return null;
  },
);

first_page.dart

import 'package:flutter/material.dart';
import 'package:router/router.dart';

class FirstPage extends StatelessWidget {
  const FirstPage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('First Page')),
      body: Center(
        child: ElevatedButton(
          onPressed: () {
            router.pushNamed(
              'second',
              queryParams: {'a': 'b'},
              extra: 'test-extra',
            );
          },
          child: const Text('Go to Second Page'),
        ),
      ),
    );
  }
}

second_page.dart

import 'package:flutter/material.dart';
import 'package:go_router/go_router.dart';

class SecondPage extends StatelessWidget {
  const SecondPage({
    super.key,
    required this.queryParams,
    required this.extra,
  });

  final Map<String, String> queryParams;
  final Object? extra;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        leading: BackButton(
          onPressed: context.pop,
        ),
        title: const Text('Second Page'),
      ),
      body: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        children: [
          Text('Query Params: $queryParams'),
          const SizedBox(height: 8),
          Text('Extra: $extra'),
        ],
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.2, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.0.2 at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision cd41fdd495 (4 days ago), 2022-06-08 09:52:13 -0700
    • Engine revision f15f824b57
    • Dart version 2.17.3
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.7042882/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] Android Studio
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/203.7185775/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.67.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.42.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 102.0.5005.115

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.1.0-0.0.pre.1284, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.1.0-0.0.pre.1284 at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fae31eecce (30 minutes ago), 2022-06-16 19:58:05 -0700
    • Engine revision f8c0dc87bc
    • Dart version 2.18.0 (build 2.18.0-189.0.dev)
    • DevTools version 2.14.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13E113
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.7042882/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[!] Android Studio
    • Android Studio at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/203.7185775/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.68.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.42.0

[✓] Connected device (3 available)
    • Pixel 3a (mobile) • 964AY0WL20 • android-arm64  • Android 12 (API 32)
    • macOS (desktop)   • macos      • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)      • chrome     • web-javascript • Google Chrome 102.0.5005.115

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I have the exact same exception, any updates on this issue ?

I also get this exception after upgrading to gorouter from 4.1.0 to 5.2.4.

Any idea about this problem, guys ?

Any updates?


Migrate GOCi Software, data and dependencies to codon

Blocked at the moment by 2 requests waiting for TSC's response:
1.) Accssing /nfs/production/parkinso/spot from new VMs:
Detail 1:
In Hinxton all our VM's (Snoopy, Gromit, Garfield) & Noah Login had access to the share path /nfs/spot/
The corresponding path in Harlow & Codon cluster is /nfs/production/parkinso/spot
Though this path is accessible from codon, its not accessible from the new VMs
Our process requires jobs processed in our Data Release using Codon in /nfs/production/parkinso/spot to be accessible by the Harlow VMs
Request 1:
We need the new Harlow VMs (gwas-gromit, gwas-garfield, and gwas-snoopy) to be granted access to the path /nfs/production/parkinso/spot
2.) Accssing /nfs/gwas/data from Codon Cluster:
Detail 2:
The new gwas nfs storage path /nfs/gwas/data you created for us is accessible from the new VMs
But not accessible from Codon cluster which is a blocker for us.
Request 2:
We need the shared path [/nfs/gwas/data] to be accessible from Codon cluster as well
Ticket found here

The changes have been deployed & running successfully from Codon & can be closed