Which issue?

the sound issue, whys choppy like that.. i use your scrcpy and aux for sound but this morning got all messed up and got me sound like in my tiktok video above

Scrcpy does not forward sound, so the problem is not in scrcpy :smile: I don't know why you have this sound issue.

Scrcpy does not forward sound, so the problem is not in scrcpy 😄 I don't know why you have this sound issue.

that be sick if scrcpy could do sound too, i know it dont..  thought you had some knowledge on my problem but thanks 😄


fix image check bug

Image checker will try to check image with best effort. If registry return unexpected code such as 5xx/429, image checker will abort. We only failed the image checker when we make sure the image is not exist or authentication failed.
Fix: #17

Test step:
run a script in a worker node and call docker hub registry API, until it return 429
run a pai job and schedule the pod to the worker node.
make sure the job will not failed, even though registry api return 429


Update fast-refresh.md

Hi there!
As a library author I found it puzzling how fast-refresh affects the hooks. It is nice to document how useState and useRef differs from the others. Also giving an example of how useEffect runs related to a fast-refresh. Because it runs as if the component was removed and added again, which is just nice to know 😄
Awesome work on this!

I rewrote this a bit, as I don't think the original wording was clear enough. Hope it's ok.

For sure, iterations are always good, thanks! 👍 😄


exclude TERM env var by default

Fixes https://github.com/rust-lang/miri/issues/1702
r? @oli-obk

@bors r+

:pushpin: Commit 3c5cb89f638af28b8f645fb73a3210fb66bc725f has been approved by oli-obk

:hourglass: Testing commit 3c5cb89f638af28b8f645fb73a3210fb66bc725f with merge a12a48bf723e0e13f043979a7f79861d975e7187...

:sunny: Test successful - checks-actions
Approved by: oli-obk
Pushing a12a48bf723e0e13f043979a7f79861d975e7187 to master...


Headings.yml uses rule for capitalization based on IBM style guide rather than Red Hat's style guide

match: $sentence
message: "Use sentence-style capitalization in '%s'."
scope: heading
source: "IBM - Capitalization in headings and titles, p.16"

However, Section 3.2 of Red Hat's Style Conventions for Writers and Editors, Edition 5.1, says to use "title case" for the Capitalization of headings:

3.2. Heading Styles
[...]
Capitalization
The standard for all Red Hat technical documentation is title case for all headings and titles. Table headings, procedure headings, and formal paragraph titles fall under this heading, and consequently, standard title case capitalization rules apply.

See: https://redhat-documentation.github.io/vale-at-red-hat/docs/user-guide/redhat-style-for-vale/
I believe the current behavior is expected. The style in this repository is not based on Red Hat's Style Conventions for Writers and Editors, Edition 5.1, but on the Supplementary Style Guide https://redhat-documentation.github.io/supplementary-style-guide/ and IBM style guide.

Thanks for the clarifying comment. This rule now makes sense to me.


Add field validation on structs

Closes #43.
This fix takes advantage of the existing validation logic by recursively calling _find_errors whenever pl.Struct or pl.List(pl.Struct) columns are detected.
This fix exposes a (possible) bug with one of the pre-existing test cases test_model.test_missing_date_struct. I've currently added a @pytest.mark.skip to the failing test. Is this actually a bug, and if so, can I continue to skip this test case and raise a new issue to fix it? Do you have any insights into why Test.model_fields['c'].annotation.columns does not exist?

I think this is a seriously elegant solution, and I really hope we can make it work for the general case like this.
I think the skipped test error is related to the following example:
Consider a nullable struct column:
class XY(pt.Model):
    x: int
    y: int

class Coord(pt.Model):
    id: int
    xy: Optional[XY]

What we are saying here is that we allow the following values for xy:

{"x":1, y:1}
null
But not
{"x":1, y:null}

Now let's take a look at how polars materializes the above three column values:
schema = {"id": pl.Int64, "xy": pl.Struct({"x": pl.Int64, "y": pl.Int64})}
# note the difference between the two last elements in the struct:
data = {"id": [0, 1, 2], "xy": [{"x": 1, "y": 1}, None, {"x": None, "y": None}]}

df = pl.DataFrame(data, schema=schema)
print(df)
---
shape: (3, 2)
┌─────┬─────────────┐
│ id  ┆ xy          │
│ --- ┆ ---         │
│ i64 ┆ struct[2]   │
╞═════╪═════════════╡
│ 0   ┆ {1,1}       │
│ 1   ┆ {null,null} │
│ 2   ┆ {null,null} │
└─────┴─────────────┘

I think I was expecting null for the second row, that is, looking only at the last two rows:
shape: (3, 2)
┌─────┬─────────────┐
│ id  ┆ xy          │
│ --- ┆ ---         │
│ i64 ┆ struct[2]   │
╞═════╪═════════════╡
│ 1   ┆ null        │
│ 2   ┆ {null,null} │
└─────┴─────────────┘

I'm going to have to have a little think about how we do this one 😊

@thomasaarholt I've updated the PR to correctly select Type[Model] when dealing with Optional and have applied filters on all-null subrows to encode the nullable requirement, as discussed above.
Since polars treats None == {"null", "null", ...} when dealing with structs, I think this fix is valid. The only way I can see a more "legitimate" way of implementing this would be if polars itself either:

Provided  a pl.Optional datatype for marking structs as optional.
Changed its default coercion logic for handling None in the context of structs. Like @thomasaarholt, I was surprised to discover how polars currently coerces None for structs, as every other datatype I have come across is simply encoded as null... there must be some good reason for doing this that is unlikely to change.

I'll likely update this PR to include a few more test cases for testing optional lists of structs and optional deeply nested structs, but I don't think the current fix will need to change. What do you think?

Super! I overall agree with you - I've asked here on the discord just to make sure the implementation is correct. This is kinda an edge case, but someone might have an opinion.

Hi any news on this PR?

Let's go ahead and merge this, as well make a new release. @dsgibbons, if you want to make a few more test cases, especially demonstrating null behaviour, in another PR, then I'd appreciate it.

Yes, I'm still looking at adding additional test cases @thomasaarholt. I've been a bit busy lately but hopefully I'll be able to add more test cases over the next few weeks. Thanks for accepting the PR!


Add number of driftignore lines to telemetry

Q
A

🐛 Bug fix?
no

🚀 New feature?
yes

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#443

❓ Documentation
yes

Description
This PR adds number of driftignore lines to telemetry.

Can you put this one in draft for now since the #647 will probably change your RulesCount() method.

Can you put this one in draft for now since the #647 will probably change your RulesCount() method?

Tested #647 but it does not seem to be ready, may be I can merge this one today and @moadibfr will just have few conflicts to solve

Can you put this one in draft for now since the #647 will probably change your RulesCount() method?

@wbeuil Tested #647 but it does not seem to be ready, may be I can merge this one today and @moadibfr will just have few conflicts to solve

We're gonna talk this one on our next sprint meeting and see what should be the next step.

blocked by #705

Closing https://github.com/cloudskiff/driftctl/issues/443#issuecomment-880655915


Add supervisor error handling

The supervisor previously paniced when e.g. a Prometheus query was unsuccessful. This panic did not get recovered, but is now. Errors and log messages are logged via the standard package 'log' for now.

Addressed spelling nit


Error when using group_add directive

docker-compose group_add directive is supported by libcompose but when I run
it with my docker-file containing group_add I get this
$ libcompose up
WARN[0000] Note: This is an experimental alternate implementation of the Compose CLI (https://github.com/docker/compose)
ERRO[0000] Could not parse config for project komposefiles : Unsupported config option for web service: 'group_add'
FATA[0000] Failed to read project: Unsupported config option for web service: 'group_add'

docker-compose file
version: "2"
services:
  web:
    image: tuna/docker-counter23
    group_add:
      - sample
    ports:
      - "5000:5000"
  redis:
    image: redis:3.0
    ports:
      - "6379"

It works fine with docker-compose up.
$ docker-compose up
Starting komposefiles_web_1
komposefiles_redis_1 is up-to-date
Attaching to komposefiles_redis_1, komposefiles_web_1
redis_1  | 1:C 02 Mar 10:52:37.115 # Warning: no config file specified, using the default config. In order to specify a config file use redis-server /path/to/redis.conf
redis_1  |                 _._
redis_1  |            _.-``__ ''-._
redis_1  |       _.-``    `.  `_.  ''-._           Redis 3.0.7 (00000000/0) 64 bit
redis_1  |   .-`` .-```.  ```\/    _.,_ ''-._
redis_1  |  (    '      ,       .-`  | `,    )     Running in standalone mode
redis_1  |  |`-._`-...-` __...-.``-._|'` _.-'|     Port: 6379
redis_1  |  |    `-._   `._    /     _.-'    |     PID: 1
redis_1  |   `-._    `-._  `-./  _.-'    _.-'
redis_1  |  |`-._`-._    `-.__.-'    _.-'_.-'|
redis_1  |  |    `-._`-._        _.-'_.-'    |           http://redis.io
redis_1  |   `-._    `-._`-.__.-'_.-'    _.-'
redis_1  |  |`-._`-._    `-.__.-'    _.-'_.-'|
redis_1  |  |    `-._`-._        _.-'_.-'    |
redis_1  |   `-._    `-._`-.__.-'_.-'    _.-'
redis_1  |       `-._    `-.__.-'    _.-'
redis_1  |           `-._        _.-'
redis_1  |               `-.__.-'
redis_1  |
redis_1  | 1:M 02 Mar 10:52:37.116 # WARNING: The TCP backlog setting of 511 cannot be enforced because /proc/sys/net/core/somaxconn is set to the lower value of 128.
redis_1  | 1:M 02 Mar 10:52:37.116 # Server started, Redis version 3.0.7
redis_1  | 1:M 02 Mar 10:52:37.116 # WARNING overcommit_memory is set to 0! Background save may fail under low memory condition. To fix this issue add 'vm.overcommit_memory = 1' to /etc/sysctl.conf and then reboot or run the command 'sysctl vm.overcommit_memory=1' for this to take effect.
redis_1  | 1:M 02 Mar 10:52:37.116 # WARNING you have Transparent Huge Pages (THP) support enabled in your kernel. This will create latency and memory usage issues with Redis. To fix this issue run the command 'echo never > /sys/kernel/mm/transparent_hugepage/enabled' as root, and add it to your /etc/rc.local in order to retain the setting after a reboot. Redis must be restarted after THP is disabled.
redis_1  | 1:M 02 Mar 10:52:37.116 * The server is now ready to accept connections on port 6379
web_1    |  * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)

There are some attributes I've implemented in our libcompose fork (including group_add). If anyone wants to beat me to this feel free to move these commits upstream. Might also be a good reference for adding new fields.


Change module to use a Chrome specific one

Could we swap this to use its own endpoint so it doesn't accidentally get impacted by changes to other endpoint? There isn't a web_after_article module right now, so this isn't a huge deal, but better if we can get it right now.
@mmiermans can you confirm the backend is fine with passing arbitrary modules to RecIt?

https://github.com/Pocket/recit-ralph/pull/80 is the corresponding change in RecIt.


sign up steps failed

docker run -p 3000:3000 --volume ~/docker-volumes/sqlpad-oracle:/var/lib/sqlpad --detach sqlpad/sqlpad:latest
and next show me forbiden

{"level":30,"time":"2022-06-10T22:17:55.237Z","pid":1,"hostname":"5cab12225d08","name":"sqlpad-web","req":{"id":1,"method":"GET","url":"/","query":{},"params":{}},"res":{"statusCode":200},"responseTime":24,"msg":"request completed"}
{"level":30,"time":"2022-06-10T22:17:55.261Z","pid":1,"hostname":"5cab12225d08","name":"sqlpad-web","req":{"id":2,"method":"GET","url":"/javascripts/vendor/tauCharts/tauCharts.min.css","query":{},"params":{}},"res":{"statusCode":304},"responseTime":4,"msg":"request completed"}
{"level":30,"time":"2022-06-10T22:17:55.264Z","pid":1,"hostname":"5cab12225d08","name":"sqlpad-web","req":{"id":3,"method":"GET","url":"/assets/index.2a0a97ac.js","query":{},"params":{}},"res":{"statusCode":304},"responseTime":2,"msg":"request completed"}
{"level":30,"time":"2022-06-10T22:17:55.267Z","pid":1,"hostname":"5cab12225d08","name":"sqlpad-web","req":{"id":4,"method":"GET","url":"/assets/index.96304ac6.css","query":{},"params":{}},"res":{"statusCode":304},"responseTime":2,"msg":"request completed"}
{"level":30,"time":"2022-06-10T22:17:55.471Z","pid":1,"hostname":"5cab12225d08","name":"sqlpad-web","req":{"id":5,"method":"GET","url":"/api/app","query":{},"params":{}},"res":{"statusCode":200},"responseTime":10,"msg":"request completed"}
{"level":30,"time":"2022-06-10T22:18:02.474Z","pid":1,"hostname":"5cab12225d08","name":"sqlpad-web","req":{"id":6,"method":"POST","url":"/api/signup","query":{},"params":{}},"res":{"statusCode":403},"responseTime":41,"msg":"request completed"}

Hi @johnfelipe. Sign up functionality is only allowed for email addresses that have been "invited" to SQLPad.
For first account, you'll need to provide the email address and password using environment variables: https://getsqlpad.com/#/authentication?id=local-authentication

Is in an env file or how set admin user and admin pwd?
El sáb, 11 de jun. de 2022, 9:14 p. m., Rick Bergfalk <
@.***> escribió:

Hi @johnfelipe https://github.com/johnfelipe. Sign up functionality is
only allowed for email addresses that have been "invited" to SQLPad.
For first account, you'll need to provide the email address and password
using environment variables:
https://getsqlpad.com/#/authentication?id=local-authentication
—
Reply to this email directly, view it on GitHub
https://github.com/sqlpad/sqlpad/issues/1131#issuecomment-1153047771,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADIWFHLDGF74WT6ZWACJALVOVBXHANCNFSM5YO7WFUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yes there are environment variables SQLPAD_ADMIN and SQLPAD_ADMIN_PASSWORD.
You can set them any way you prefer to set them for docker if using that. Otherwise there is an env file you can provide https://getsqlpad.com/#/configuration

pls help me, i cant login


feat(all): support multiple versions od feature service

@stemey Is there any possibility that this is released soon? Currently i'm facing this issue in a project and it would help a lot. Tell me how i could help if help is needed. Thanks in advance.


No value for key [org.springframework.jdbc.datasource.LazyConnectionDataSourceProxy] bound to thread

I tried to integrate the grails-melody-plugin-1.73.2 into a Grails-3.3.8 application but with some GORM interactions I got the following stacktrace:
java.lang.IllegalStateException: No value for key [org.springframework.jdbc.datasource.LazyConnectionDataSourceProxy@64e3d4ab] bound to thread [pool-2-thread-1]
	at org.springframework.transaction.support.TransactionSynchronizationManager.unbindResource(TransactionSynchronizationManager.java:210)
	at org.springframework.orm.hibernate5.HibernateTransactionManager.doCleanupAfterCompletion(HibernateTransactionManager.java:654)
	at org.springframework.transaction.support.AbstractPlatformTransactionManager.cleanupAfterCompletion(AbstractPlatformTransactionManager.java:1021)
	at org.springframework.transaction.support.AbstractPlatformTransactionManager.processCommit(AbstractPlatformTransactionManager.java:815)
	at org.springframework.transaction.support.AbstractPlatformTransactionManager.commit(AbstractPlatformTransactionManager.java:734)
	at org.springframework.transaction.support.TransactionTemplate.execute(TransactionTemplate.java:150)
	at grails.gorm.transactions.GrailsTransactionTemplate.execute(GrailsTransactionTemplate.groovy:91)
...
	at org.springsource.loaded.ri.ReflectiveInterceptor.jlrMethodInvoke(ReflectiveInterceptor.java:1427)
	at grails.melody.plugin.MelodyInterceptorEnhancer$_enhance_closure1$_closure2.doCall(MelodyInterceptorEnhancer.groovy:77)
	at sun.reflect.GeneratedMethodAccessor349.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.springsource.loaded.ri.ReflectiveInterceptor.jlrMethodInvoke(ReflectiveInterceptor.java:1427)
...

It seems that Graeme Rocher investigated on that issue a while ago: https://github.com/grails/grails-core/issues/10675#issuecomment-315321643

This is an issue with the melody plugin. It is replacing the data source with another proxy which interferes with the references GORM uses to bind and unbind the session.

@brischniz Could you provide a sample application with the issue? Probably it's related to #61.

I confirm this is happening.
I'll try to investigate.

Were you by any chance successful in creating a sample application? It might be easier to lobby for some help if we can reproduce it reliably.

Added sample here: https://github.com/joemccall86/melody-sample

Thanks @joemccall86, I'll take a look at sample code.

I've created branch 65_no_value_for_key, I see issue even without ExpandoMetaClass.enableGlobally(). In this branch I'm creating a new integration test app so we can improve our regression suite and avoid this issues in future.

Thanks Sergio! I just had a new baby so I don’t have much time right now ;-)

@bassmartin Congratulations on the birth of your baby! Enjoy this time :-)

When I use the latest release, I don't have any SQL queries monitored.
sql Statistics sql - 1 day
None
Do you all still have SQL queries monitored?

I installed a custom version with this code modified (removed the if if (dataSource instanceof TransactionAwareDataSourceProxy) { and it works for me on Grails 3.3.9 and GORM 6.1.11...
Why is that if required?
 // Attempt DataSources for Grails 3.3 and newer if no lazy proxy created
        if (!lazyProxied) {
            String hibernateBeanName = 'hibernateDatastore'
            if (applicationContext.containsBean(hibernateBeanName)) {
                def bean = applicationContext.getBean(hibernateBeanName)
                def transactionManager = bean.transactionManager
                def dataSource = transactionManager.dataSource
                LOG.debug "Wrapping Hibernate DataStore DataSource"
                dataSource.targetDataSource = JdbcWrapper.SINGLETON.createDataSourceProxy(dataSource.targetDataSource)
            }
        }

Which class is this for you? I think my initial idea was to make sure that we're dealing with two levels of encapsulation to reach the datasource (store - targetds - targetds) but I guess we can check if property with this name exists instead of using instance of.

This might have to do with my configuration.
Here are some extracts :
grails:
    spring:
        transactionManagement:
            proxies: false

hibernate:
    cache:
        queries: false
        use_second_level_cache: true
        use_query_cache: true
        region.factory_class: org.hibernate.cache.ehcache.SingletonEhCacheRegionFactory
    flush:
        mode: AUTO
    jdbc:
        batch_versioned_data: false
        batch_size: 50
    connection:
        release_mode: ON_CLOSE
    hibernateDirtyChecking: true

Since the code in org.grails.datastore.gorm.jdbc.connections.DataSourceConnectionSourceFactory#proxy is
    protected DataSource proxy(DataSource dataSource, DataSourceSettings settings) {
        if(settings.isLazy()) {
            dataSource = new LazyConnectionDataSourceProxy(dataSource);
        }
        if(settings.isTransactionAware()) {
            dataSource = new TransactionAwareDataSourceProxy(dataSource);
        }
        return dataSource;
    }

I think we could replace if (dataSource instanceof TransactionAwareDataSourceProxy) by if (dataSource instanceof DelegatingDataSource) to support both types of proxy (since it's the superclass of both).
What do you think about that @sergiomichels ?

Yep, looks good. I think that probably there's an way to configure this for one specific integration test (instead of creating new project), can you take a look at Spring Security plug-in and see if it's using it? https://github.com/grails-plugins/grails-spring-security-core/tree/master/integration-test-app


Refresh token on server

Quick question -- I'm making api calls on the server which require the supabase access token. My question is: is the token automatically refreshed on the server? Or just the client? I had to add this in my middleware/auth to get it to work (note, just using useSupabaseSession()) wasn't actually refreshing the token.
export default defineNuxtRouteMiddleware(async (to, _from) => {
    const client = useSupabaseClient()

    const session = await client.auth.getSession()

    if (!session.data.session) {
        return navigateTo(
            Urls.SignIn({ redirectAfter: encodeURIComponent(to.fullPath) })
        )
    }
})

Could we potentially add an argument to useSupabaseSession which awaits on refresh token (if necessary)


IMPRO-928  Rewrite regrid CLI to be more of a standardisation CLI

Addresses #682
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

CLA

[ ] If a new developer, signed up to CLA

Rewrite the regrid CLI to be more of a standardisation CLI due to its additional functionality beyond just re-gridding. Re-gridding can be performed with (optionally) fixing float64 data, changing metadata and applying Iris nearest and extrapolation re-gridding features. The CLI can also be used to undertake standalone functions of checking/fixing float64 data and updating metadata. The name of the CLI has been changed from regrid to standardise to reflect its more general utility function.

Ref the comment (quite possible I'm missing something here though!) -

There is no path through the CLI to update meta-data and fix float-64 without applying the regrid step.

You can just provide the arguments source output --json_file fix_float64
The source will be fixed for float64 at the start of the CLI (as you noted) now lines 116-117:
if args.fix_float64:
check_cube_not_float64(source_data, fix=True)
then the metadata will be updated at line now beginning 169:
elif args.output_filepath and args.json_file:
Regriding only takes place if --target_grid_filepath has been specified (now line 132).
Also I've tried the changes you suggested with the if args.json_file block and having a final save_netcdf at the end of the CLI but it doesn't work for me, the regridding bats tests fail. Presumably they passed for you though; I can see the logic of the changes but I must be applying them incorrectly somehow.

Ref the comment (quite possible I'm missing something here though!) -

There is no path through the CLI to update meta-data and fix float-64 without applying the regrid step.
You can just provide the arguments source --output_filepath --json_file --fix_float64

The source will be fixed for float64 at the start of the CLI (as you noted) now lines 116-117:
if args.fix_float64:
check_cube_not_float64(source_data, fix=True)
then the metadata will be updated at line now beginning 169:
elif args.output_filepath and args.json_file:

This is my point. I don't think it will. The use of "elif" here will prevent it. "elif" adds the extra condition that all previous "if" and "elif" tests have been False. I don't think you want this in this case.

Also I've tried the changes you suggested with the if args.json_file block and having a final save_netcdf at the end of the CLI but it doesn't work for me, the regridding bats tests fail. Presumably they passed for you though; I can see the logic of the changes but I must be applying them incorrectly somehow and will have to look into it a further.
I didn't actually try them. I have just done so and found that there is one more thing that needs changing: There are two variable names in use for the output data. I have standardised these to "output_data" and the tests all pass again. I will push up the commit that does this so you can see what I have done.


[Help Wanted]: looking for a better way to export assets when building libraries

Steps to reproduce
For an example, you can check this project.
packages/webgl/src/components/snow/index.ts
import snowflakeUrl from<EMAIL_ADDRESS>
snowflakeUrl may be transformed to '/snowflake.hash.png', which is definitely wrong when use this library in another project.
Thus, I have to import assets inline, but it slows down the prebuild for the project using this library.
@laynezh/vite-plugin-lib-assets is an awesome solution for vite, but does not seem to work with farm.
Reproduce link
No response
What is actually happening?
x
System Info
System:
    OS: Linux 6.9 Arch Linux
    CPU: (16) x64 AMD Ryzen 7 5800H with Radeon Graphics
    Memory: 9.98 GB / 15.58 GB
    Container: Yes
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 22.2.0 - /run/user/1000/fnm_multishells/4278_1718689209819/bin/node
    Yarn: 1.22.22 - /usr/bin/yarn
    npm: 10.7.0 - /run/user/1000/fnm_multishells/4278_1718689209819/bin/npm
    pnpm: 9.4.0 - /run/user/1000/fnm_multishells/4278_1718689209819/bin/pnpm
  npmPackages:
    @farmfe/core: ^1.1.15 => 1.1.15

I'll take a look


feat: Fragment support to have multiple children in a component

Similar to React, where children in a Fragment will be 'extended' to a parent list (like children) instead of 'appended'
Example
import reacton
import reacton.ipywidgets as w

@reacton.component
def Children():
    with reacton.Fragment():  # option for reacton, mandatory for solara
        w.Button(description="1")
        w.Button(description="2")

@reacton.component
def Test():
    with w.VBox():
        Children()
Test()

Since solara overrides the '_default_container', this is mandatory to use with solara (even when just imported). For solara v2 we will probably also start using fragments.

To make this work in solara, use:
import solara # this changes reacton.core. _default_container
import reacton.core
# now restore it
reacton.core. _default_container = reacton.Fragment

Note, Fragment does not have to be used explicitly.
import reacton
import reacton.ipywidgets as w

@reacton.component
def Children():
    # will use fragment implicitly, because we have 2 elements
    w.Button(description="1")
    w.Button(description="2")

@reacton.component
def Test():
    with w.VBox():
        Children()
Test()


Add an option to loginWithRedirect that redirect to the signup page instead of login page.

Describe the problem you'd like to have solved
In my app, I know that some users would like to sign up, so I would like the ability to show the https://<domain>.auth0.com/u/signup page instead of https://<domain>.auth0.com/u/login.
Describe the ideal solution
Maybe something like :
loginWithRedirect({signup: true})
Alternatives and current work-arounds
I haven't found one yet.
Additional context

Hey @totobest ,
This is already possible, see https://github.com/auth0/auth0-react/issues/59#issuecomment-654333541.

Hey @totobest ,
This is already possible, see https://github.com/auth0/auth0-react/issues/59#issuecomment-654333541.


Can't move error dialog --> fixed

The error dialog cannot be dragged.

Root cause is the removal of the dialog title bar area. (WindowStyle=none).  Will either need to

Make new title area draggable
Figure out how to override child component and style the header

Validated


Add auto-author and auto-label to github workflow

Closes #17

Proposed Changes

The same as this PR for phys2bids but implemented for this repo
All I did was change the directory locations in the .github/labeler.yml file, otherwise the files are the same as the phys2bids files

Change Type

[ ] bugfix (+0.0.1)
[ ] minor (+0.1.0)
[ ] major  (+1.0.0)
[ ] refactoring (no version update)
[ ] test (no version update)
[x] infrastructure (no version update)
[ ] documentation (no version update)
[ ] other

Checklist before review

[x] I added everything I wanted to add to this PR.
[ ] [Code or tests only] I wrote/updated the necessary docstrings.
[ ] [Code or tests only] I ran and passed tests locally.
[ ] [Documentation only] I built the docs locally.
[x] My contribution is harmonious with the rest of the code: I'm not introducing repetitions.
[ ] My code respects the adopted style, especially linting conventions.
[x] The title of this PR is explanatory on its own, enough to be understood as part of a changelog.
[x] I added or indicated the right labels.

[ ] I added information regarding the timeline of completion for this PR.

[x] Please, comment on my PR while it's a draft and give me feedback on the development!

@smoia all checks passed - happy to review and merge?

LGTM! @smoia could you merge or add me as a member to this project?

@rgbayrak I added you as maintainer for this repo

@all-contributors add @RayStick for infrastructure


RFC: Replace Logic course in Advanced Math

Problem:
Introduction to Logic course on Coursera is broken and unmaintained.
Duration:
September 10, 2022 - September 24, 2022
Background:
The Stanford Introduction to Logic course on Coursera is broken.
Almost all of the external links of the course content and exercises

end up dead:

It's been like this for about a year. There were other issues with the course several years before that as well, such as broken proof tools, incorrect assignments that let you pass without submitting anything, etc. Many learners commented in Discussion forums but the issues don't get fixed.
A previous discussion / RFC about Logic courses was held on the OSSU Math repository. The ForallX project was chosen as the most suitable.
Here is the relevant pull request: #1075
Proposal:
Replace with the ForallX project
Alternatives:
The other two options that were also discussed in OSSU Math:

An Introduction to Formal Logic by Peter Smith
Logic in Action

I think this is a good change for OSSU CS.

I have a question: Why is logic in the syllabus? (sorry if it's an obvious question)
The basics are covered in Math for CS. I don't know why logic is related to computer science. I'm over halfway thorough the course and I haven't made the connection.

@riceeatingmachine Excluding basic logic covered in Math for CS, Logic is in the curricular guidelines:

IS/Advanced Representation and Reasoning (Elective) 124
IS/Natural Language Processing (Elective) 126
IS/Advanced Machine Learning (Elective) 127
PL/Logic Programming (Elective) 166
SE/Formal Methods (Elective) 185

(These are electives so it's in Advanced Math.) It's used a lot in software verification, compilers, automated reasoning, AI/ML and so on.

I'm over halfway thorough the course and I haven't made the connection.

Did you know that all computer programs are actually mathematical proofs ?

Closing the issue after the time period, proceeding with the Proposal.


Wrap calls to network in Jenkinsfile in retry() blocks

Describe the desired behavior
Our pipeline sometimes fails on networks calls, for example this git clone fail and this pip install fail.
So, we should wrap git clones and pip installs in retry() blocks. As always, aping the GO pipeline is considered helpful.

Closing this, since we haven't seen this problem in a long time


Roll Engine from c7f9725521c0 to bcf0668ac355 (3 revisions)

https://github.com/flutter/engine/compare/c7f9725521c0...bcf0668ac355
2020-06-08<EMAIL_ADDRESS>Roll Skia from 81aa5ddb9404 to f9ffdd6942f9 (4 revisions) (flutter/engine#18893)
2020-06-08<EMAIL_ADDRESS>Add a deprecation note to FlutterFragmentActivity (flutter/engine#18865)
2020-06-08<EMAIL_ADDRESS>Roll Dart SDK from 87c5e3612b70 to 66e2b22d14af (1 revision) (flutter/engine#18892)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Add progress for AudioKit.renderToFile and AVAudioEngine.renderToFile

When export audio as a file, This PR provider a way to know the export progress.

Thanks!


Fix Godot 4.2 import error (missing files)

As mentioned in here: https://github.com/miskatonicstudio/goat/tree/master/installation : Godot complains about 2 missing files, which are added to the repo. Restarting the project fixes this issue.

This might already be fixed in Godot 4.3.


Create a new gradient Button variant

Just like Tabnine has it: https://www.tabnine.com/pricing

Done!


Lots of depreciation errors with latest update

Bug Description
Just ran the latest update and noticed a lot of depreciation errors popping up:

Embedded Assets version
2.0.0
Craft CMS version
Craft Solo 4.8.6

Could you please post one of the stack traces?

36	Deprecation error: The images embedded asset property has been deprecated, due to being removed in Embed 4. Use image instead.
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/spicyweb/craft-embedded-assets/src/models/EmbeddedAsset.php:209
35	spicyweb\embeddedassets\models\EmbeddedAsset::__construct(["title" => "The Alps 4K - Scenic Relaxation Film With Calming Music", "description" => "The Alps are possibly the most magical region in the world! Enjo...", "url" => "https://www.youtube.com/watch?v=BTMjD7_evjE", "type" => "video", ...])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/spicyweb/craft-embedded-assets/src/Service.php:535
34	spicyweb\embeddedassets\Service::_getDimensions(["title" => "The Alps 4K - Scenic Relaxation Film With Calming Music", "description" => "The Alps are possibly the most magical region in the world! Enjo...", "url" => "https://www.youtube.com/watch?v=BTMjD7_evjE", "type" => "video", ...])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/spicyweb/craft-embedded-assets/src/Service.php:303
33	spicyweb\embeddedassets\Service::createEmbeddedAsset(["title" => "The Alps 4K - Scenic Relaxation Film With Calming Music", "description" => "The Alps are possibly the most magical region in the world! Enjo...", "url" => "https://www.youtube.com/watch?v=BTMjD7_evjE", "type" => "video", ...])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/spicyweb/craft-embedded-assets/src/Service.php:254
32	spicyweb\embeddedassets\Service::getEmbeddedAsset(craft\elements\Asset)
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/spicyweb/craft-embedded-assets/src/Variable.php:29
31	spicyweb\embeddedassets\Variable::get(craft\elements\Asset)
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Extension/CoreExtension.php:1635
30	twig_get_attribute(craft\web\twig\Environment, Twig\Source, spicyweb\embeddedassets\Variable, "get", ...)
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/craftcms/cms/src/helpers/Template.php:129
29	craft\helpers\Template::attribute(craft\web\twig\Environment, Twig\Source, spicyweb\embeddedassets\Variable, "get", ...)
Called from /Users/Rob/Sites/craft-base/build/craft/templates/_includes/cp.media-text.twig:28
28	__TwigTemplate_b3da91afbdb8e18fd4470e3fec93a5d7::doDisplay(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], [])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:394
27	__TwigTemplate_b3da91afbdb8e18fd4470e3fec93a5d7::displayWithErrorHandling(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], [])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:367
26	__TwigTemplate_b3da91afbdb8e18fd4470e3fec93a5d7::display(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...])
Called from /Users/Rob/Sites/craft-base/build/craft/templates/_includes/cp.page-content.twig:25
25	__TwigTemplate_80a04f400da56967898ad5af7ee0ec64::doDisplay(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], [])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:394
24	__TwigTemplate_80a04f400da56967898ad5af7ee0ec64::displayWithErrorHandling(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], [])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:367
23	__TwigTemplate_80a04f400da56967898ad5af7ee0ec64::display(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...])
Called from /Users/Rob/Sites/craft-base/build/craft/templates/_pages/default.twig:28
22	__TwigTemplate_7f8db50f9be079ff9334866c9b605e65::block_mainContent(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], ["mainContent" => [__TwigTemplate_7f8db50f9be079ff9334866c9b605e65, "block_mainContent"]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:171
21	__TwigTemplate_72216030be2c8181d19edbd35927df5c::displayBlock("mainContent", ["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], ["mainContent" => [__TwigTemplate_7f8db50f9be079ff9334866c9b605e65, "block_mainContent"]])
Called from /Users/Rob/Sites/craft-base/build/craft/templates/_layouts/main.twig:71
20	__TwigTemplate_72216030be2c8181d19edbd35927df5c::doDisplay(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], ["mainContent" => [__TwigTemplate_7f8db50f9be079ff9334866c9b605e65, "block_mainContent"]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:394
19	__TwigTemplate_72216030be2c8181d19edbd35927df5c::displayWithErrorHandling(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], ["mainContent" => [__TwigTemplate_7f8db50f9be079ff9334866c9b605e65, "block_mainContent"]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:367
18	__TwigTemplate_72216030be2c8181d19edbd35927df5c::display(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], ["mainContent" => [__TwigTemplate_7f8db50f9be079ff9334866c9b605e65, "block_mainContent"]])
Called from /Users/Rob/Sites/craft-base/build/craft/templates/_pages/default.twig:1
17	__TwigTemplate_7f8db50f9be079ff9334866c9b605e65::doDisplay(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], ["mainContent" => [__TwigTemplate_7f8db50f9be079ff9334866c9b605e65, "block_mainContent"]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:394
16	__TwigTemplate_7f8db50f9be079ff9334866c9b605e65::displayWithErrorHandling(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], ["mainContent" => [__TwigTemplate_7f8db50f9be079ff9334866c9b605e65, "block_mainContent"]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:367
15	__TwigTemplate_7f8db50f9be079ff9334866c9b605e65::display(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], [])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/TemplateWrapper.php:45
14	Twig\TemplateWrapper::display(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...])
Called from /Users/Rob/Sites/craft-base/build/craft/templates/_pages/entry.twig:2
13	__TwigTemplate_60828a1a86d759390196e131207ce858::doDisplay(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], [])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:394
12	__TwigTemplate_60828a1a86d759390196e131207ce858::displayWithErrorHandling(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry], "craft" => craft\web\twig\variables\CraftVariable, "currentSite" => craft\models\Site, ...], [])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:367
11	__TwigTemplate_60828a1a86d759390196e131207ce858::display(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Template.php:379
10	__TwigTemplate_60828a1a86d759390196e131207ce858::render(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/TemplateWrapper.php:38
9	Twig\TemplateWrapper::render(["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/twig/twig/src/Environment.php:280
8	craft\web\twig\Environment::render("_pages/entry.twig", ["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/craftcms/cms/src/web/View.php:482
7	craft\web\View::renderTemplate("_pages/entry.twig", ["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry]])
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/craftcms/cms/src/web/View.php:535
6	craft\web\View::renderPageTemplate("_pages/entry.twig", ["entry" => craft\elements\Entry, "variables" => ["entry" => craft\elements\Entry]], "site")
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/craftcms/cms/src/web/TemplateResponseFormatter.php:57
5	craft\web\TemplateResponseFormatter::format(craft\web\Response)
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/yiisoft/yii2/web/Response.php:1100
4	craft\web\Response::prepare()
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/craftcms/cms/src/web/Response.php:337
3	craft\web\Response::prepare()
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/yiisoft/yii2/web/Response.php:340
2	craft\web\Response::send()
Called from /Users/Rob/Sites/craft-base/build/craft/vendor/yiisoft/yii2/base/Application.php:390
1	craft\web\Application::run()
Called from /Users/Rob/Sites/craft-base/build/public_html/index.php:12

Thanks for posting that @splendidrob. Those deprecation warnings are intentionally being logged for embedded assets with those properties, so this isn't a bug. However, I do understand that the deprecation warnings might not be relevant for properties that aren't actually used in the Craft project, and it would be worth improving the deprecation logging to only happen when a property is actually accessed.

Just released 4.0.1 and 5.0.1 which limit the deprecation logging to front-end requests.

@ttempleton All makes sense! Thanks very much for the update...much appreciated!


key-bindings.zsh clobbers l key in vi-mode

key-bindings.zsh contains bindkey -s '\el' "ls\n" which is incompatible with vi mode.

Can this be closed? I haven't experienced the ls behavior with vi-mode activated...

@mcornella is it still in key-bindings?
to replicate, type "who" [esc] "l" for example:
% bindkey -s '\el' "ls\n"
% whols
zsh: command not found: whols

It is still in key-bindings.zsh but with vi-mode active if I do ESC+L it doesn't happen. Maybe I'm using vi-mode wrong?

essentially any bindkey -s '\e…' statements can cause an awkward interjection when switching from vi-insert to vi-normal mode. this just happens to be extra noticable since l along with h are common movement keys.
if it's not occurring for you, it's likely related to the key typing rate, there's a 0.4s default window after typing <ESC> that l would need to be pressed to see the issue. (see KEYTIMEOUT in http://zsh.sourceforge.net/Doc/Release/Parameters.html#Parameters-Used-By-The-Shell-1 )
% bash
$ which zsh
$ /usr/local/bin/zsh
$ zsh --version
zsh 5.0.7 (x86_64-apple-darwin14.0.0)
$ exec env -i TERM=xterm-256color zsh --no-globalrcs --no-rcs
% bindkey -v
% bindkey -s '\el' "ls\n"
;; typed: vim<ESC>l
% vimls
zsh: command not found: vimls
% echo $KEYTIMEOUT
40

% bindkey -v
Here's where I think the misunderstanding is, if you look at [line 18 of key-bindings.zsh](https://github.com/robbyrussell/oh-my-zsh/blob/master/lib/key-bindings.zsh#L18) you'll see we're using emacs keymap.
Once the `vi-mode` plugin switches to viins keymap, the defined '\el' key binding does not apply anymore. Please confirm by actually using oh-my-zsh directly instead of running `bindkey -s '\el' "ls\n"` on the command line, which you do _after_ switching to viins mode (therein the incompatibility).

If I run your same commands, it also happens to me, but not because of oh-my-zsh.

sounds good, we can close in that case. i'll assme the the issue is stale at this point.

Thanks!

And sorry for the long time to have someone comment (nearly 2 years!) :grin:

:) no prob, and thanks for the housekeeping! :+1:


AVS do not support multiple logout/login

Briefly summarize your issue:
AVS do not support well multiple logout/login, threads are leaking and oom crash occurs
What is the expected behavior?
multiple logout/login support without leaks
What behavior are you observing?
Some threads are leaking, RSS memory is increasing (more than 2Mbytes per logout/login)
Provide the steps to reproduce the issue, if applicable:
using avs in smart screen patching the version to call resetDevice when pressing navigation button
or in our smart screen set top box integration
Tell us about your environment:
What version of the AVS Device SDK are you using?
avs 1.24.0
smart screen 2.7.0
Tell us what hardware you're using:

linux ubuntu PC
or
bcm7268 set top box

Tell us about your OS (Type & version):

[x ] Linux

I have seen a first fix in smart screen 2.7.0 solving the serverThread leak issue
But in avs code, I see at least that other threads leaking:

mediaplayer workerloop threads: (8 threads added per logout/login)
DirectiveSequencer::receivingLoop (3 threads added per logout/login)
TimerDelegate::timerLoop (2 threads added per logout/login)
CertifiedSender::mainloop() (1 thread added per logout/login)

Hi,
Can you clarify if you are using the SampleApp from the AVS SDK, the SampleApp from the Smart Screen SDK or your own application?
Are you tearing down DefaultClient completely between restarts?

Hello Keith,
In the steps to reproduce I wrote
using avs in smart screen patching the version to call resetDevice when pressing navigation button
or in our smart screen set top box integration
So to be more precise, I use smartscreen vanilla ubuntu sample app + a little patch to call the resetDevice function when pressing navigation button
(I have also the same leak issue using our smartscreen sample app integration in our STB device)
here is the patch:
void GUIManager::handleNavigationEvent(smartScreenSDKInterfaces::NavigationEvent event) {
-    m_executor.submit([this, event]() {
         ACSDK_DEBUG3(LX(__func__).d(
             "processNavigationEvent in executor", smartScreenSDKInterfaces::navigationEventToString(event)));

         // TODO : Implement more robust visual presentation and granular channel orchestration for local navigation
         switch (event) {
             case smartScreenSDKInterfaces::NavigationEvent::BACK:
-                executeBackNavigation();
+                // executeBackNavigation();
+                resetDevice();
                 break;
             case smartScreenSDKInterfaces::NavigationEvent::EXIT:
-                executeExitNavigation();
+                //executeExitNavigation();
+                ACSDK_DEBUG3(LX(__func__).d(
+                    "RVDB", smartScreenSDKInterfaces::navigationEventToString(event)));
+                resetDevice();
                 break;
             default:
                 // Not possible as returned above.
                 break;
         }
-    });
 }

So I am only calling resetDevice.
Can you tell me how to tear down DefaultClient properly using smartscreen sample app ?

Unfortunately that is outside the scope this forum. I suggest posting this issue to https://github.com/alexa/alexa-smart-screen-sdk.
However, if you can determine that DefaultClient has been shutdown, yet there is still a memory leak, then please reach back to us.
Regarding the MediaPlayers, can you also ensure that those objects are not being kept alive even after they've been passed into DefaultClient.

hello @kjkh
I am a little bit lost, since @sngoc-amazon answered me to open an issue here in a comment of https://github.com/alexa/alexa-smart-screen-sdk/issues/97
Ok, I will try to understand how DefaultClient is initialized (and maybe not closed ?) in the vanilla smartscreen sampleapp context, but as far as I understand, it concerns a vanilla sample app, I was thinking you were interested

This issue is being tracked in the Alexa Smart Screen SDK forum


Improving appendix on Object-Oriented Programming

Thanks again, @sttaft,  for reviewing! I've just made that small grammar correction.


Changed .editorconfig to have 4 space indents for C files as well

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


[HUDI-8467][DNM] Auto commit enable deprecate (dis allowing auto commit in data table too)

Change Logs
Describe context and summary for this change. Highlight if any code was copied.
Impact
Describe any public API or user-facing feature change or any performance impact.
Risk level (write none, low medium or high below)
If medium or high, explain what verification was done to mitigate the risks.
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

I do see some usages for Flink auto commit:

for delete operation in FlinkDeleteHelper , the logic does not work at all before and we can just replace it with commit;
for drop partition operation in FlinkDeletePartitionCommitActionExecutor , Flink explicitly enabled the auto-commit in HoodieCatalogUtil.createWriteClient.


Imports Validation Schemas into the Examples README during CI

Many thanks for submitting your Pull Request :heart:!
Please specify parts of this PR update:

[ ] Specification
[ ] Schema
[x] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[x] Other (CI workflows)

Discussion or Issue link:
#801
What this PR does / why we need it:
At the moment, examples are located both in the README and in files for the verification pipeline of the CI.
This means one change needs to be applied to 3 locations: the README json sample, the README yaml sample and the example json file.
This PR adds the possibility to reference external files thanks to an include tag.
When a pull request is merged and at least an example file has been modified, a script will render README_TEMPLATE.md and replace README.md with the output (in the example folder).
This means modifications should not be applied manually to README.md anymore but to the TEMPLATE instead. To try and prevent users from modifying the README directly, a disclaimer header is added to the file and a specific CI check is run when a pull request is opened and the README has been modified.
For the template, the include tag have two attributes:

file (string) the file to render in place of the tag
format (string: json | yaml) if the file should be rendered as JSON or YAML

e.g.: <include file="event-based-greeting.json" format="json" />
When the CI workflow is run, a bot will push a signed commit to a branch and open a PR. To do so, the owners of this repository have to:

Create a (bot) account
Add the bot to the repository (and organization ?) collaborators
Validate the invite with the bot account
Create a signing key for the bot and add the public key in the bot account settings
Generate a Personal Access Token for the bot
Add some secrets in the repository:

BOT_USERNAME - The GH username of the bot
BOT_EMAIL - The email associated to the GH account of the bot
BOT_PAT - The personal access token of the bot
BOT_GPG_PRIVATE_KEY - The private key used to sign commits

Special notes for reviewers:
Additional information:

I forgot to signoff my commits... If necessary, I created an other branch with a signed off commit. I can close this PR and open a new one if it's a big no go.
@cdavernas @ricardozanini  @tsurdilo @antmendoza WDYT ?

@JBBianchi I think you can amend your commit and add the -s :D

@JBBianchi let's discuss tomorrow about the planning and the steps to install the bot! We might do it tomorrow during the meeting if necessary: #804

@JBBianchi I think you can amend your commit and add the -s :D

I just did but I think I messed up ^^'
I'll just close and open a new PR, it feels safer.


Redirect to Root if Subdomain Doesn't Exist

This will prevent accidental/ghost signups if the go to a subdomain that does not exist.

http://stackoverflow.com/questions/6664822/rails-how-to-redirect-if-record-is-not-found

Closed thanks for shirjeel-alam contribution.


chore: fix tests in ee

Updated imports.
Fixes WindowMessageListener_spec.ts.

/ok-to-test

@eco-monk  I will update this branch with a test that mocks the behaviour in ee. Don't trigger CI yet

@ohansFavour could we create a separate PR for that?
We can merge this, since it breaking EE tests now.

/ok-to-test

/ok-to-test sha=8e1fe54


Kuma reports unhealthy docker containers as PENDING, not DOWN

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
As the title says, is there any way to make Kuma treat unhealthy containers as downtime?

📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.11
💻 Operating System and Arch
Docker Compose
🌐 Browser
unrelated
🐋 Docker Version
Docker 24.0.7, build afdd53b
🟩 NodeJS Version
No response

This might be a bug. I assume you have not configured a large number of retries?

Yes.
I also tried it with the default retries = 0, resend notification if down = 0

Reproduced the error on a clean Kuma install. If I hover over the yellow line it says "unhealthy"

I saw this today where one of my containers was unhealthy but it never generated an event or notification for it. I have retries set at zero, so this should have alerted.


ActiLife csv

Hi Vincent,
I am trying to use ActiLife to export data to use in GGIR. I've exported it without the timestamp but I'm still having difficulties getting GGIR to run. I am getting the error:
Error in data.frame(value = H[, 2], row.names = H[, 1]) :
duplicate row.names: 0
Would you be able to provide a sample csv that works with GGIR? Or inform me what measures can be analyzed with GGIR. I've provided a screenshot of the possible measures that can be included.
Thank you,
Anissa

Hi Anissa,
It looks as if you are converting the ActiGraph epoch agd files to csv files. To use the files in GGIR you need the raw csv files.
You can get these by selecting 'Raw to Raw' from the 'Import/Export/Convert' menu in Actilife and converting the raw gt3x files to raw csv files. See attached screenshot.
Best,
Alex


fixed analog signal bug, added saccade parser

Sweet, I ended up adding these in other commits: 73b59cd52aaf56f0345d7438241585bb754d48bc and af2631e2b62e075cf42613ff188dbb26f2b50cdb


Python script - Pressing s to save the preview not working

OS - Ubuntu - 64 bit
Python version - 2.7.10
Steps to reproduce -

Run the python script script.py
Use printout of the scantemplate.pdf and hold against camera and adjust orientation till preview window appears.
Try saving the preview generated by pressing the 's' key.
Preview image is not saved as desired.

Resolved in your pull request.


Remove email field from Company structured data

This was originally added via: https://github.com/parameter1/base-cms/pull/113
NOTE: If we would like to add an aliased version of the publicField called email (likely to prevent any backwards breaking behavior that could come with removing email) then that can certainly be done, initially just wanted to remove the actual trouble spot before making any additional alterations.

 Remove private email from structured data


Missing letter

It was actually correct with separate, as separated would imply that they were combined at some point. For clarification, I've updated your PR to change to the word to "different", which has the same meaning here, but is a more common word in English. 🙂


Tracking does not last forever.

After many tests and revisions, we have found that, given a smartphone carried inside a building over multiple hours, set to track the region of all the beacons in that building, not running in the foreground, eventually location updates will cease to come until the app is woken back up. Typically this takes something in the realm of 30 minutes.
I've eventually found that Apple states that the location updates background mode doesn't actually allow ranging to run indefinitely - "Enabling this mode does not prevent the system from suspending the app, but it does tell the system that it should wake up the app whenever there is new location data to deliver." So even if tracking is left on, even if ranging is left on, eventually the process is killed until another kind of event triggers the location services.
Is this correct, or does this conflict with your findings? It would seem an additional source of triggers (such as the pedometer) will be needed to keep ranging live through a day.

We've not tested it for long periods of time yet. This is something that we will have to look at.

In our latest testing with the new firmware on the Crownstones, the tracking does last essentially for ever. The issue was the following of the ibeacon spec in the beacons.
Cheers


Multiple Hostname (virtual host?)

Hi all,
this is my docker compose file:
version: '3'
services:
  mailserver:
    image: zixia/simple-mail-forwarder:1.1
    container_name: mailserver
    hostname: mailserver1.org
    environment:
      - SMF_CONFIG=@mailserver1.org:mypersonalemail@gmail.com<EMAIL_ADDRESS>    ports:
      - "25:25"
    restart: always

The ip and MX records are configured correctly, however emails are only forwarded when received by mailserver2
How can I solve the problem ?

I found a possible solution to the problem
this is the modified docker compose file:
version: '3'
services:
  mailserver:
    image: zixia/simple-mail-forwarder:1.1
    container_name: mailserver
    hostname: mailserver1.org
    environment:
      - SMF_CONFIG=@mailserver1.org:mypersonalemail@gmail.com<EMAIL_ADDRESS>    ports:
      - "25:25"
    volumes:
      - "./hostname:/etc/hostname:ro"
    restart: always

And this is the hostname file:
mailserver1.org
mailserver2.org

Seems to work now

I found a possible solution to the problem
this is the modified docker compose file:
version: '3'
services:
  mailserver:
    image: zixia/simple-mail-forwarder:1.1
    container_name: mailserver
    environment:
      - SMF_CONFIG=@mailserver1.org:mypersonalemail@gmail.com<EMAIL_ADDRESS>    ports:
      - "25:25"
    volumes:
      - "./hostname:/etc/hostname:ro"
    network_mode: host
    restart: always

And this is the hostname file:
mailserver1.org
mailserver2.org

It seems that some emails have arrived, I will do more tests in the next days


Chapter1/homework/4

I asked for help on the discord but no1 responded so I'm creating a pull request in the hope that a mentor can help me fix this problem.  I've been able to pass all of the tests except for one of the prompt name tests.  Here is the error:

I'm not sure how to fix this, or approach it, any help would be appreciated!

I asked for help on the discord but no1 responded so I'm creating a pull request in the hope that a mentor can help me fix this problem. I've been able to pass all of the tests except for one of the prompt name tests. Here is the error:

I'm not sure how to fix this, or approach it, any help would be appreciated!

Quite simple 😄


PlatformException(VideoError, Video player had error androidx.media3.exoplayer.ExoPlaybackException: MediaCodecVideoRenderer error, index=0, format=Format(1, null, null, video/avc, avc1.64003C, -1, null, [8000, 1500, 25.0, ColorInfo(Unset color space, Unset color range, Unset color transfer, false, 8bit Luma, 8bit Chroma)], [-1, -1]), format_supported=NO_EXCEEDS_CAPABILITIES, null, null)

import 'package:flutter/material.dart';
import 'package:chewie/chewie.dart';
import 'package:video_player/video_player.dart';
class VideoView extends StatefulWidget {
final String url;
final bool autoPlay;
final bool looping;
final double aspectRatio;
const VideoView({super.key, required this.url, this.autoPlay=false, this.looping=false, this.aspectRatio=16/9,});
@override
State createState() => _VideoViewState();
}
class _VideoViewState extends State {
late VideoPlayerController _videoPlayerController;
late ChewieController _chewieController;
@override
initState() {
_videoPlayerController = VideoPlayerController.asset('assets/1.mp4');
initVideo();
_chewieController = ChewieController(
videoPlayerController: _videoPlayerController,
autoPlay: widget.autoPlay,
looping: widget.looping,
aspectRatio: widget.aspectRatio
);
super.initState();
}
Future initVideo() async {
await _videoPlayerController.initialize();
}
@override
void dispose() {
_videoPlayerController.dispose();
_chewieController.dispose();
super.dispose();
}
@override
Widget build(BuildContext context) {
double screenW = MediaQuery.of(context).size.width;
double screenH = MediaQuery.of(context).size.height;
return Container(
width: screenW,
height: screenH/2,
color: Colors.grey,
child: Chewie(
controller: _chewieController,
),
);
}
}

Does anyone have this problem? Please help me solve it. thank you.

same problem

same problem

Version 3.24.3 is problematic, downgrade to version 3.22.1 is okay

I have same issue with flutter version 3.24.3 .  But when i downgrade to 3.22.2 then it's Ok . I hope resolve this issues on new version flutter soon.


Missing parent GO:0101005   deubiquitinase activity

GO:0101005   deubiquitinase activity
Molecular Function
is defined
An isopeptidase activity that cleaves ubiquitin from a target protein to which it is conjugated.
but it is not currently a descendant of
GO:0070122 | isopeptidase activity
source: iEA mapping to GO:0070122 | isopeptidase activity here:
https://www.pombase.org/gene/SPAC31G5.13
is not suppressed as redundant.

Added 'ubiquitin-like protein peptidase activity' is_a isopeptidase activity.
This should be inherited by GO:0101005 deubiquitinase activity
@sjm41 I forget why we didn't do that. I think we were debating whether 'isopeptidase' was a useful term, s as it is not orthogonal to other terms in that branch.
Thanks, Pascale

I would be happy for 'isopeptidase' to go, there are too many axes of classification for peptidases!

Actually I found it: One of the children of GO:0101005 deubiquitinase activity is 'GO:0140492 metal-dependent deubiquitinase activity'
"Catalysis of the metal-dependent hydrolysis of an ester, thioester, amide, peptide or isopeptide bond formed by the C-terminal glycine of ubiquitin.
See discussion in #17398
Something is wrong because 'deubiquitinase activity' should be named 'deubiquitinase isopeptide activity' and the metallo-dependent version should not be a child.

Actually we could create 'deubiquitinase isopeptide activity' and give it the correct parent, and keep GO:0101005 deubiquitinase activity  as the grouping term. Does that work?

Hi both

My memory from our final #17398 discussions was that we were planning to obsolete the 'isopeptidase activity' GO term - there are only 6 EXP annotations, and 5 of those are to the (current) single child term "leukotriene-C(4) hydrolase" (and I'm not sure that is correctly classified anyway.... I'll make a separate ticket for that.)

I'm not sure the definition on 'GO:0140492 metal-dependent deubiquitinase activity' is correct - I don't easily see any statement about metal-dependent deubiquitinases working on non-isopeptide bonds in the given ref (PMID:19489724). I think the def just needs to model the def for GO:0101005 deubiquitinase activity, with 'metal-dependent' added.

FYI, we added this comment to GO:0019783 ubiquitin-like protein peptidase activity "While ubiquitin-like proteins can be rarely linked to substrates via bonds other than isopeptide bonds, all known ubiquitin-like peptidases cleave the isopeptide bond."

So, if we proceed with obsoleting 'isopeptidase activity' (as an unnecessary/complicating additional classification axis) then Val's reported issue goes away, and we don't need to make any new grouping terms here.

sounds good!


Support for array in hql

Within HQL the following is a valid create table statement:
CREATE TABLE a.b (
    `c` array<string>
)

But parsing this with DDLParser emits:
simple_ddl_parser.ddl_parser.DDLParserError: Unknown statement at LexToken(RT,'array<string>',1,23)

Poking through the existing tests and closed issues I see a specific form of array<struct<cola:string,colb:string>> is supported,  but it doesn't seem like the generic handling of arrays is working. The hive docs aren't particularly clear, but they show that all data types are valid within the <>, and elsewhere on the page gives a specific example using array<string>.

I have the same problem.How to solve this problem?


issue #2510 - make FHIRPathUtil threadsafe

FHIRPathUtil was using a shared static FHIRPathEvaluator but this is not
thread-safe. Now we construct a new evaluator for each request.
Signed-off-by: Lee Surprenant<EMAIL_ADDRESS>

Is this a nominal change? e.g. does not impact the benchmark? or is this additive? I'm expecting no impact, but want to confirm in discussion.


NoSuchElementException: Pool exhausted when starting app

Bug Report

Current Behavior

Stack trace
Caused by: java.util.NoSuchElementException: Pool exhausted
  at io.lettuce.core.support.BoundedAsyncPool.acquire()(Unknown Source)

Input Code

Input Code
 client = RedisClient.create();
        client.setOptions(
            ClientOptions.builder()
                .autoReconnect(true)
                .cancelCommandsOnReconnectFailure(true)
                .requestQueueSize(1024 * Runtime.getRuntime().availableProcessors())
                .build());
        final RedisURI redisURI = RedisURI.builder()
            .withHost(redisAddr)
            .withPort(6379) //默认值
            .withPassword(redisPassword)
            .build();

        pool = AsyncConnectionPoolSupport.createBoundedObjectPool(
            () -> client.connectAsync(StringCodec.UTF8, redisURI),
            //默认8个连接
            //refs:https://github.com/lettuce-io/lettuce-core/issues/1363
            BoundedPoolConfig.builder()
                .maxTotal(Runtime.getRuntime().availableProcessors() * 2)
                .minIdle(Runtime.getRuntime().availableProcessors())
                .build());
        final String pong = pool.acquire().get().sync().ping();

Expected behavior/code

Environment
    <lettuce.version>5.3.4.RELEASE</lettuce.version>

Possible Solution

Additional context

Isn't this fixed in refs:https://github.com/lettuce-io/lettuce-core/issues/1363
?

I also think that's a duplicate of #1363. By awaiting initialization via BoundedAsyncPool.create(…), the pool should no longer throw exceptions. I also think we should investigate whether we can detect in-flight creations and attach the acquisition to the asynchronous pool warming. Can you upgrade to 5.3.4 and retest whether the issue persists?

@mp911de Yes, I was using the 5.3.4, so I was wondering...

I am currently using lettuce to connect to aliyun's redis. I will try to dig this out.

#1363 introduced two new methods BoundedAsyncPool.create(…) and AsyncConnectionPoolSupport.createBoundedObjectPoolAsync(…) that return CompletionStage<Pool>. We didn't want to introduce blocking behavior so adding new methods was the only option to allow for synchronization.

@mp911de I will read through the PR and feedback.

@mp911de It's still there after I updated the code

Thanks for report. We need to check why this is.

@mp911de After I set the maxTotal to match the concurrent acquire threads number, it disappears.

Providing a minimal sample that reproduces the problem would be helpful. Since I don't have any insight into what the application is doing it's not possible to further diagnose the issue.

@mp911de I will close this one, cause it's turn out to be an underside issue. thanks.

@He-Pin , I encountered same issue, can you please share the code snippet to mitigate the issue ?
Timeout occurred while populating hgetall : java.util.concurrent.CompletionException: java.util.NoSuchElementException: Pool exhausted

Hi @vsharathis, I do not know if this is the correct way of doing stuff (@mp911de could validate) but to avoid that error, it seems you need to prepare your async pool like that:
public AsyncPool<StatefulRedisConnection<String, String>> createAsyncConnectionPool(RedisURI redisURI, int connectionPoolSize) {
        var boundedPoolConfig = BoundedPoolConfig.builder()
                .minIdle(connectionPoolSize)
                .maxTotal(connectionPoolSize)
                .build();
        return AsyncConnectionPoolSupport.createBoundedObjectPool(() -> redisClient.connectAsync(StringCodec.UTF8, redisURI), boundedPoolConfig);
    }

By forcing the pool to initialize all connections in IDLE mode, then no "pool exhauted" error :)
Cheers !

@Brico87 , Thanks for the comment, what is the best "connectionPoolSize" to manage, is it based the number of transactions per second or do need to follow some formula to define it. Please do clarify , thanks

I think I let that to @mp911de answer that but I put a random 10 :)

After setting pool size to 1000 and executor thread to 1000 still we encountering pool size exhaust exception

@Brico87 , yes ASYC is must

@vsharathis , can you elaborate the need of async here?

IIRC, I was increase the max pooled number, I have no access to the codes now. @vsharathis ，sorry。

Why would it even throw an exception like this instead of returning the connection later when it's available?


my profile

Fixes #122 add my profile

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This contains potentially hazardous tags that need manual review.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

811e4ccf86fada45efba5ad40d7aa106df531add


Auto Format corrupting paths in my scripts

Issue Type: Bug

Call a file in your script with "." or "&" with a relative path like & ..\Desktop\Temp\Test.ps1 (The file need to exist).
Trigger the Formatter by hitting Enter or Alt+Shift+F or any other way.
The path to the file is deleted.

This is a major issue as opening a big script, changing a line (That triggers the auto format) will result in all paths in the document to corrupt braking the script! And if you saved and don't have a backup you are in trouble.
Extension version: 1.12.0
VS Code version: Code 1.32.3 (a3db5be9b5c6ba46bb7555ec5d60178ecc2eaae4, 2019-03-14T23:43:35.476Z)
OS version: Windows_NT x64 10.0.17763
EditorServices.log
vscode-powershell.log

System Info

Item
Value

CPUs
AMD Ryzen 7 1800X Eight-Core Processor          (16 x 3593)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.95GB (9.09GB free)

Process Argv

Screen Reader
no

VM
50%

FYI Same problem reported with UNC paths:
https://github.com/PowerShell/vscode-powershell/issues/1830#issuecomment-479702167

It's a bummer to have to turn off and not use the PS Formatter b/c of this. The truncation of paths occurs with any formatting operation including formatting selected text.

I just downgraded to VSCode PowerShell 1.11.0
The state of this software "Stable" releases is ridiculous.
Every developer group I know who would make a release and get a bug of this magnitude discovered would rollback the update or release a patch version immediately. but here they just let more users data get damaged every day without taking action.
If this was a single developer volunteering it's time I can understand that. but this is a team with the backing of Microsoft.

I don't see it mentioned anywhere, but this is caused by the setting for 'Use Correct Casing for Cmdlets'.  While correcting the casing, it changes the extension, removes the path, etc...

So, what is the status of this bug? Is there any expectation for the fix?

Is there someone from the development team monitoring the Issues here or is this module is kind of Abandonware?

I have not had the time to repro yet but I think PR #1216 and/or PR #1210 that are already merged into the development branch. It would be great if you could confirm that it has been fixed @ili101 You can take the binaries from this appveyor build here (just replace the module that is used with the binaries from the build, choose either the Windows PowerShell or PowerShell Core build depending on what you use)

@bergmeister I updated to the last VSCode PowerShell v2019.5.0
Tested with default setting - No formatting at all. (As it was turned off until this is fixed)
Added "powershell.codeFormatting.useCorrectCasing": true to settings and tested again  - formatting worked and paths get corrupted.
Replaced the module in C:\Users\illym\.vscode\extensions\ms-vscode.powershell-2019.5.0\modules\PSScriptAnalyzer with AppVeyor version - problem not fixed paths still corrupted.

Sorry, I cannot repro this, can you please add more details about which formatting settings you use @ili101 or @rkeithhill ?Does the current working directory play a role, etc?

I'm still able to repro:
C:\Users\Keith
06-03 19:54:43 21> $def = Get-Content .\PSSA-Formatting.ps1 -Raw
C:\Users\Keith
06-03 19:55:18 22> $def
. .\Desktop\Temp\Test.ps1
. '.\Desktop\Temp\Test.ps1'
. .\Desktop\Temp\Test.ps1

get-process -name notepad

C:\Users\Keith
06-03 19:55:23 23> Invoke-Formatter -ScriptDefinition $def -Settings .\PSSA-Formatting-Settings.psd1
. Test.ps1
. Test.ps1
. Test.ps1

Get-Process -name notepad

C:\Users\Keith
06-03 19:55:35 24> gmo

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     1.1.0      DirColors                           {ConvertFrom-LSColors, ConvertTo-LSColors, Format-Colorize… Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Management     {Add-Content, Clear-Content, Clear-Item, Clear-ItemPropert… Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Object…}
Script     <IP_ADDRESS>    posh-git                            {Add-PoshGitToProfile, Expand-GitCommand, Format-GitBranch… Script     2.0.0      PSReadLine                          {Get-PSReadLineKeyHandler, Get-PSReadLineOption, Remove-PS…
Script     1.18.0     PSScriptAnalyzer                    {Get-ScriptAnalyzerRule, Invoke-Formatter, Invoke-ScriptAn…

Here is my setting file:

I cannot seem to reproduce this. but I do get the annoying change of cmd to cmd.exe.
I am pretty sure this is coming from the setting 'usecorrectcasing', but this is changing more than casing.

@bergmeister @msftrncs Regarding the reproduce problem, I did some more testing:

It looks like that there is some caching involved, so for example if the file is not in the path and you test it, then create the file then test again it will not change it. you will need to start a new PS session to "clear the cache".
In VSCode the relative path need to be valid in relative to the path the console was opened in, So if VSCode has a folder open it need to be relative to it and if not it need to be relative to C:\Users\MyUser. changing the folder after the console was opened has no effect.
It is easier to reproduce this with UNC paths like \\server\Temp\T1.ps1 as it doesn't matter what folder you are in.
I was able to reproduce it outside of VSCode with a UNC path but not with relative path, maybe VSCode send the base path to PSScriptAnalyzer Settings?

OK, I can repro now but only if I open PowerShell using the explorer context menu (which uses the -RemoveWorkingDirectoryTrailingCharacter -WorkingDirectory on pwsh.exe), I cannot repro in Windows PowerShell though. I can therefore confirm that this specific issue is still not fixed in the development branch. But that's enough as a start for me to look into it. I suspect it is a wrong CommandInfo cache entry, PSSA performance improvements in 1.18.0 were mainly due to the usage of caching therefore now we have since found some cases where entries are wrong, which we have to fix now. PSSA basically issues get-command calls to Powershell to get info about commands and sometimes it is even PowerShell's behaviour that gets into the way...
Just to repeat again: We have disabled the newly added vs code setting powershell.codeFormatting.useCorrectCasing to prevent this bug from affecting more users and if you find it annoying then until PSSA 1.18.1 is released (planned for next week), then I suggest to not use this new feature.
@msftrncs Your mentioned issue is already fixed in #1210


Front end not working on Windows 11

The front end displays a localhost:5000 not found error. No other error message can be found.

I just tried it on my laptop with Windows 11. If you clear your cache and launch from the command prompt, then it should work. This will probably happen every time the manager gets updated (because of new variables, models, etc.).

Good to know it's not a Windows 11 issue. Clearing my cache didn't resolve it. I imagine that a manager update would break further down the line (showing incorrect titles), not at the first load screen.
I can imagine it has something to do with the absence of admin privileges on this laptop, or possibly the security issues that GitHub flags. I'll open a ticket with the university IT, even though that'll likely take a few months before it's resolved..


Json control character escaping

Escaping the '' and '"' characters.

@zoranmax I know this is a pretty inactive repo, but I still use the nuget package, and ran into an issue recently in prod. :-)
I've added pretty thorough unit testing to try to make the PR pretty easy. Let me know if you have any questions.

Hi,
I am fine to integrate this one, but looking at the code it looks like that this could be potentially a breaking change, as some people might have implemented this quoting logic already somewhere... I am honestly just worried about this one. Perhaps a flag could be placed somewhere to disable it by default? What do you think?

That's a good suggestion. I'll include that.

Updated. I didn't add testing for the negative side (non-escaping) since the code was pretty straightforward, but let me know if that makes you uncomfortable.
Added a custom escaping callback also, since the escaping I included is pretty limited.

@zoranmax - could I trouble you to review this soon? Sorry to bother you.

Hi,
I just published the nuget package
https://www.nuget.org/packages/Google.DataTable.Net.Wrapper/4.1.
I'm really sorry that it took me some time :-)  Thank you very much for your support+

No problem! Been pretty busy myself, so I know how it goes.


Support for Observables

I want to create a ionic-native plugin for this cordova plugin. But the support for observable/promise does not seem to be considered. For example returning 2 values on start is not supported by observables/promises. And using callbacks does not seem normal for ionic/angular.
So I would like to change the code to better support observables and move the callback functions to actual functions. But that would be a huge change of the code. I do not know if it would be acceptable for such a change. If not, I would just use my fork for my project.

Feel free to make a Pull Request then we'll see.
You can return multiple values in Promise, for instance:
Promise.resolve([1,2]).then(([arg1,arg2]) => console.log('result', arg1, arg2))
Does this help?

Yes it did seem to return 2 values, but I will still try to change it. Observables are better IMO for Ionic/Angular. The only problem is, I have no idea how to program or test iOS. I neither have an iPhone to test it with nor an Mac to build it. We'll see how it works


await async not working on android

hey，bro，Thank you very much for your project，But I'm having some problems with it，project working on pc，but not on android，i find await async is not working on my android phone（android 13），did you work success on android？or any suggestion？

@busiyg thanks bro.
What package version did you try?

I've had time to test the upcoming version 5 on Android today....no problems whatsoever.
If you happen to provide reproduction steps (or a project), re-open or create a new issue.
Thanks!


Remove unused npm dependencies

we where requiring some things that we never use so I removed them
I also removed lodash.escaperegexp because we already use lodash

Waiting on #909


Get rid of plumber in error catching

I refactored error catching, using this example: http://stackoverflow.com/questions/23971388/prevent-errors-from-breaking-crashing-gulp-watch
Would be good to add it to toolkit

We need to review overall structure of gulp tasks.


Some storybook pages are broken

It seems in particular those that are "connected".

https://docs.voltocms.com/storybook/?path=/story/internal-components-context-navigation--connected&globals=measureEnabled:false
https://docs.voltocms.com/storybook/?path=/story/object-browser--connected&globals=measureEnabled:false (click on browse icon)

Example error response


Re-configure WiFi hotspot

Is there a way to re-configure the Wi-Fi instead of re-flash the firmware to get it reset to factory settings.

The base library HomeSpan has a "Device Configuration Mode" accessible through a button but there is not GPIO pin set by default and unless you did set one up on the web interface, you'll have to either configure wifi trough the serial terminal or just erase and re-flash.

Thanks for you reply!


Unable To Load IFrames With Cross-Origin-Opener-Policy

I have been using puppeteer to screen-scrape home.nest.com. It's been working fine, but it seems that they recently changed how they manage oauth tokens. home.nest.com now calls https://accounts.google.com/o/oauth2/iframe which has the following headers set:
cross-origin-embedder-policy: require-corp
cross-origin-opener-policy: same-origin
cross-origin-resource-policy: cross-origin

While desktop Chrome doesn't have any problems navigating the site and loading that iframe URL, puppeteer (even non-headless) blocks that URL with the following error:
(blocked:CoopSandboxedIframeCannotNavigateToCoopPage)
This document was blocked from loading in an iframe with a sandbox attribute because this document specified a cross-origin opener policy.
Learn more (https://web.dev/coop-coep/?utm_source=devtools)

I have tried passing all fo the following flags to getBrowser, with the same results:
--disable-web-security
--disable-features=IsolateOrigins
--disable-site-isolation-trials
--disable-features=site-per-process
--no-sandbox
--user-data-dir=/tmp/chrome

Has anyone else run into this yet? Is there any workaround at the moment?

Could you try running the desktop browser with the --remote-debugging-port parameter and connect to that instance using borwser.connect(). If the issue does not reproduce, it should be due to the flags Puppeteer starts the browser with. Also, could you provide the Puppeteer/browser version you are testing with and a basic script to reproduce the issue?

I upgraded puppeteer/chrome and the issue no longer appears. I'll reopen if the error pops up again. Thank you for the response!


add dependency to Pygmentize

I guess Pygmentize is used to highlight the Turtle sources ("implementation" of each term).
2) Html: multi-page
Loading graph...
Building visualization...
Error: Pygmentize Failed (30 times)

Add such dependency, or at least describe in the README how to install it.

Hello all!
I am having the same problem :(
I have installed pygments. I have version 2.2.0
I also have php version 7.2.9
My error also says
ERROR: No matching distribution found for pygmentize
Please help!!


Bit.Tooling.SourceGenerator lib file causing dependency to Microsoft.CodeAnalysis (Roslyn)

Bit.Tooling.SourceGenerator lib file causes dependency to Microsoft.CodeAnalysis (Roslyn) which in turn makes some issues in Android AOT compiler in release mode

        <IsRoslynComponent>true</IsRoslynComponent>
        <EmitCompilerGeneratedFiles>true</EmitCompilerGeneratedFiles>
        <CompilerGeneratedFilesOutputPath>$(BaseIntermediateOutputPath)Generated</CompilerGeneratedFilesOutputPath>
        <IncludeBuildOutput>false</IncludeBuildOutput>
        <DevelopmentDependency>true</DevelopmentDependency>

I was able to make it work by adding these values to source generator project


Cannot resolve symbol

I've just update the solution given by Nazurah.
Solution: https://github.com/CertifaiAI/CertifAI-Knowledge-Base/wiki/Update-maven-pom.xml


Transmit exemption data for an operation to FIN

Describe the task
We need to create a data pipeline so that operations that have been approved for carbon tax exemption have their data automatically submitted to Ministry of Finance so that they can issue an exemption certificate.
At this time, we don't know what data about an operation FIN needs in order to issue an exemption certificate, nor how they want us to send them the data. Business area/PO to reach out to FIN.
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

I'm not sure that we will need this actually. So far FIN doesn't seem to be indicating what, if anything, they would like us to provide. It could end up just a simple data extract or even spreadsheet. TBD.

This will probably be replaced by the work captured in #273


docs of opendal's mailing list moderation

OpenDAL's PMC members can request to join in mailing list moderation by sending mails in<EMAIL_ADDRESS>And there is a web ui for this https://webmod.apache.org/.
We should add docs in our pmc pages.

No PMC members interested, closing.


Fix admin logging

Previously logging for admin requests was validating that the upstream request uri header was set, which was incorrect.

/azp run tools - test-proxy - tests


TabController children count >= 4

when the children > 4
══╡ EXCEPTION CAUGHT BY GESTURE ╞═══════════════════════════════════════════════════════════════════
I/flutter (22320): The following assertion was thrown while handling a gesture:
I/flutter (22320): 'package:flutter/src/material/tab_controller.dart': Failed assertion: line 110 pos 12: 'value >= 0
I/flutter (22320): && (value < length || length == 0)': is not true.
I/flutter (22320): Either the assertion indicates an error in the framework itself, or we should provide substantially
I/flutter (22320): more information in this error message to help you determine and fix the underlying cause.
I/flutter (22320): In either case, please report this assertion by filing a bug on GitHub:
I/flutter (22320):   https://github.com/flutter/flutter/issues/new

please add the output of flutter doctor and
if possible the code that allows to reproduce

TabController's length needs to match the number of tabs. Perhaps you're not keep it in sync with the value of TabBar.tabs?

en, it is my mistake

@skeyboy Glad to hear you could figure it out.
I guess this issue can be closed.


supports browsing through a presentation on the phone

What was changed and why

[x] adds entries to .gitignore

When setting up the project as the readme states, one will eventually end up with node_modules, .lsp and .clj-kondo directory.

[x] adds style to support tap event helper carets

I styled the carets as boxes, to make them stand out and provide a bigger area to interact with.

[x] implements the caret tap component

Where the magic happens. Carets disappear after an interaction, be it a keydown or tap.

Please feel free to change to your desire and thank you for the opportunity to chip in 🤗
Screenshot

This is great, thanks for contributing. There is a style issue where if I make the window smaller the slides no longer fit. I will take a look at that and simplify this a bit before merging. Thanks!

@chr15m Thank you very much for this great little project ❤
Sounds like good ol' https://developer.mozilla.org/en-US/docs/Web/API/Window/resize_event, this always slips my mind  😬

Merged with some tweaks. Let me know if this UI works ok for you on iPhone.

UX is great!
Had the on-click in one iteration, too, but couldn't get it to work 😬
Your changes make it super neat and clean - wow 😮👏😍


EmrEtlRunner: improve error message for contract-failing configuration files

The current message doesn't help find what's wrong in the file.
Possibly we could do this by monkey patching https://github.com/egonSchiele/contracts.ruby/blob/master/lib/contracts.rb#L39-L50 or by defining our own version of the hash contract.

Is this a regression in Contracts (or somehow JRuby related)? It used to at least print out the failing hash and associated contract...

It still prints out the failing hash and contract - it's just that that information isn't very helpful for pinpointing the error since they are both quite large.

Agree - shall we do a PR into Contracts?

I think we should first explore creating a contract with a custom error message without changing the Contracts source, and make a PR if that approach is unworkable.

Sounds good!


HTML Compression after evaluate in pebble-spring5

Hi there,
I am currently compressing content generated by Pebble (Spring 2.3.3) using a simple javax.servlet.Filter, however, for some reason I don't fully understand, in an event of handling error, Filter chain is being called with an empty content in the ServletResponse.
For that reason, I would like to compress content somehow during Pebble parsing and not using another top-level mechanism like Filters.
Is it currently possible to intercept, in pebble-spring5, the content generated by the engine and run something over that?
Or I should write a custom https://github.com/PebbleTemplates/pebble/blob/master/pebble-spring/pebble-spring5/src/main/java/com/mitchellbosecke/pebble/spring/servlet/PebbleView.java and perform that operation after the template.evaluate(writer, model, locale); step?
Thanks

I finally wrote a new PebbleView and Resolver therefore I close the issue.


raftstore: add more duplicate entry check before proposing write command and batching commands

What is changed and how it works?

Issue Number: Ref #16818

What's Changed:
For the issue https://github.com/tikv/tikv/issues/16818.
Possible reasons for the occurrence of duplicate PUT requests to the lock CF for the same key in a single RaftCmdRequest are as follows:

Duplicate Entries in the Proposed RaftCmdRequest:
The RaftCmdRequest being proposed contains duplicate entries. For example, the transaction write command initiated during async write may have duplicate key entries. This is typically unexpected. In TiKV's transaction model, every key being written must acquire a latch before writing. Concurrent writes to the same key are sequenced through the latch and should not end up in the same RaftCmdRequest.

Batch Builder Aggregating Writes to the Same Key:
During the batch building process for proposing raft commands, writes to the same key are grouped into a single batch. However, this possibility is quite low because, as mentioned above, writes to the same key should only proceed after the latch is released during the apply phase. For writes to occur, apply must have already completed, making it unlikely that these commands are processed together in a single raft store loop.

Issues with Latch Release Timing:
The latch is expected to be released only after the apply phase is completed. If there's a problem with the latch release mechanism, it could allow duplicate writes to proceed. There is no any recent modification to this part of the code from my observation.

Batch Processing In The Apply Worker
It seems nearly impossible based on the code. While there is batch processing during the Apply phase, the panic check for duplicate entries in the change logs observed by CDC is still performed at the RaftCmdRequest level. In other words, batching during the Apply phase does not modify the contents of the original RaftCmdRequest. Therefore, it is unlikely that the problem originates from this step.

This PR attempts to add checks in the aforementioned areas. If a duplicate entry is detected, TiKV will panic, helping to narrow down the scope of the investigation. Once the issue is identified and resolved, the related checking code can be removed.
From the benchbot regression, there is no significant differences
Patched version:

Nightly version:

Related changes

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

Check List
Tests

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

Side effects

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

Release note

None

One suggestion: Given the additional check on the progress of async-write, it is recommended to provide performance comparisons between the master branch and this PR. This will help demonstrate that this PR does not introduce any significant performance regressions.

One suggestion: Given the additional check on the progress of async-write, it is recommended to provide performance comparisons between the master branch and this PR. This will help demonstrate that this PR does not introduce any significant performance regressions.

@LykxSassinator The sysbench write only bench test result is added to the descriptions

/retest

Wait for https://github.com/tikv/tikv/pull/17942 merged.