[Bug] [kafkaSink] The producer attempted a transactional operation in an invalid state

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When "semantics=EXACTLY_ONCE" is configured in kafkaSink, the data cannot be written successfully
The exception is as follows.
[org.apache.kafka.common.errors.InvalidTxnStateException: The producer attempted a transactional operation in an invalid state.]
SeaTunnel Version
2.3.2
SeaTunnel Config
kafka {
      topic = "mysql_user"
      bootstrap.servers = "node2:9092,node3:9092,node4:9092"
      format = json
      kafka.request.timeout.ms = 60000
      semantics = EXACTLY_ONCE
      kafka.config = {
        acks = "all"
        request.timeout.ms = 60000
        buffer.memory = 33554432
      }
  }

Running Command
./bin/seatunnel.sh -c /data/config/zeta-mysql-to-kafka-batch.conf -m local

Error Exception
2023-06-19 15:53:01,640 INFO  org.apache.kafka.clients.producer.internals.TransactionManager - [Producer clientId=producer-SeaTunnel6884-1, transactionalId=SeaTunnel6884-1] Transiting to fatal error state due to org.apache.kafka.common.errors.InvalidTxnStateException: The producer attempted a transactional operation in an invalid state.
	2023-06-19 15:53:01,641 ERROR org.apache.seatunnel.engine.server.checkpoint.operation.CheckpointFinishedOperation - [localhost]:5802 [seatunnel-686191] [5.1] org.apache.kafka.common.errors.InvalidTxnStateException: The producer attempted a transactional operation in an invalid state.

	org.apache.seatunnel.engine.common.exception.SeaTunnelEngineException: org.apache.kafka.common.errors.InvalidTxnStateException: The producer attempted a transactional operation in an invalid state.

		at org.apache.seatunnel.engine.server.checkpoint.operation.CheckpointFinishedOperation.lambda$run$0(CheckpointFinishedOperation.java:97) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at org.apache.seatunnel.common.utils.RetryUtils.retryWithException(RetryUtils.java:48) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at org.apache.seatunnel.engine.server.checkpoint.operation.CheckpointFinishedOperation.run(CheckpointFinishedOperation.java:81) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.Operation.call(Operation.java:189) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.call(OperationRunnerImpl.java:273) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:248) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:213) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:175) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:139) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.executeRun(OperationThread.java:123) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.internal.util.executor.HazelcastManagedThread.run(HazelcastManagedThread.java:102) [seatunnel-starter.jar:2.3.3-SNAPSHOT]
	2023-06-19 15:53:01,664 ERROR org.apache.seatunnel.engine.server.checkpoint.CheckpointCoordinator - complete checkpoint failed
	java.util.concurrent.CompletionException: org.apache.seatunnel.engine.common.exception.SeaTunnelEngineException: org.apache.kafka.common.errors.InvalidTxnStateException: The producer attempted a transactional operation in an invalid state.

		at java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:292) ~[?:1.8.0_361]
		at java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:308) ~[?:1.8.0_361]
		at java.util.concurrent.CompletableFuture.biRelay(CompletableFuture.java:1300) ~[?:1.8.0_361]
		at java.util.concurrent.CompletableFuture$BiRelay.tryFire(CompletableFuture.java:1284) ~[?:1.8.0_361]
		at java.util.concurrent.CompletableFuture$CoCompletion.tryFire(CompletableFuture.java:1034) ~[?:1.8.0_361]
		at java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:488) ~[?:1.8.0_361]
		at java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:1990) ~[?:1.8.0_361]
		at com.hazelcast.spi.impl.AbstractInvocationFuture.onComplete(AbstractInvocationFuture.java:1243) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.AbstractInvocationFuture.complete0(AbstractInvocationFuture.java:1234) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.AbstractInvocationFuture.completeExceptionallyInternal(AbstractInvocationFuture.java:1223) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.Invocation.completeExceptionally(Invocation.java:680) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.Invocation.notifyThrowable(Invocation.java:386) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.Invocation.notifyError(Invocation.java:330) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.Invocation.sendResponse(Invocation.java:230) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.Operation.sendResponse(Operation.java:483) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.sendResponseAfterOperationError(OperationRunnerImpl.java:426) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.handleOperationError(OperationRunnerImpl.java:420) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:253) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:213) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:175) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.process(OperationThread.java:139) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationexecutor.impl.OperationThread.executeRun(OperationThread.java:123) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.internal.util.executor.HazelcastManagedThread.run(HazelcastManagedThread.java:102) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
	Caused by: org.apache.seatunnel.engine.common.exception.SeaTunnelEngineException: org.apache.kafka.common.errors.InvalidTxnStateException: The producer attempted a transactional operation in an invalid state.

		at org.apache.seatunnel.engine.server.checkpoint.operation.CheckpointFinishedOperation.lambda$run$0(CheckpointFinishedOperation.java:97) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at org.apache.seatunnel.common.utils.RetryUtils.retryWithException(RetryUtils.java:48) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at org.apache.seatunnel.engine.server.checkpoint.operation.CheckpointFinishedOperation.run(CheckpointFinishedOperation.java:81) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.Operation.call(Operation.java:189) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.call(OperationRunnerImpl.java:273) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		at com.hazelcast.spi.impl.operationservice.impl.OperationRunnerImpl.run(OperationRunnerImpl.java:248) ~[seatunnel-starter.jar:2.3.3-SNAPSHOT]
		... 5 more

Flink or Spark Version
No response
Java or Scala Version
java version "1.8.0_361"
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Provide the topic information of Kafka, including partitions, etc.

Provide the topic information of Kafka, including partitions, etc.
The topic does not exist before the command is executed.

kafka config：
broker.id=1
num.network.threads=3
num.io.threads=8
socket.send.buffer.bytes=102400
socket.receive.buffer.bytes=102400
socket.request.max.bytes=104857600
log.dirs=/data/kafka
num.partitions=1
num.recovery.threads.per.data.dir=1
offsets.topic.replication.factor=3
transaction.state.log.replication.factor=3
transaction.state.log.min.isr=1
log.retention.hours=168
log.segment.bytes=1073741824
log.retention.check.interval.ms=300000
zookeeper.connection.timeout.ms=18000
group.initial.rebalance.delay.ms=0

```shell
./bin/seatunnel.sh -c /data/config/zeta-mysql-to-kafka-batch.conf -m local

Great! Please put the complete configuration here.

```shell
./bin/seatunnel.sh -c /data/config/zeta-mysql-to-kafka-batch.conf -m local

Great! Please put the complete configuration here.

env {
execution.parallelism = 1
job.mode = "BATCH"
}
source {
Jdbc {
driver = "com.mysql.cj.jdbc.Driver"
url = "jdbc:mysql://node5:3306/seatunnel?useUnicode=true&characterEncoding=UTF-8&zeroDateTimeBehavior=convertToNull&allowMultiQueries=true&serverTimezone=GMT%2B8&useSSL=false"
user = ""
password = ""
query = "select * from mysql_user"
}
}
transform {
}
sink {
kafka {
topic = "mysql_user"
bootstrap.servers = "node2:9092,node3:9092,node4:9092"
format = json
kafka.request.timeout.ms = 60000
semantics = EXACTLY_ONCE
kafka.config = {
acks = "all"
request.timeout.ms = 60000
buffer.memory = 33554432
}
}
}

@liugddx is it bug?

#4469 This pr has been discussed and resolved, waiting for merge.

THX


add updated usage for use and new clear commands in cli

Adds documentation for the following PR changes:
https://github.com/influxdata/influxdb/pull/7708
https://github.com/influxdata/influxdb/pull/7709

Fixes: https://github.com/influxdata/docs.influxdata.com/issues/949, https://github.com/influxdata/docs.influxdata.com/issues/950

Ported over your changes to the 1.2 docs in #945. Thank you, @corylanou!!!!!


HDDS-2283. Container Creation on datanodes take around 300ms due to rocksdb creation.

Container Creation on datanodes take around 300ms due to rocksdb creation. Rocksdb creation is taking a considerable time and this needs to be optimized.
2019-10-15 13:20:10,714 INFO  utils.MetadataStoreBuilder (MetadataStoreBuilder.java:build(124)) - Time before create, load options: 81 2019-10-15 13:20:10,715 INFO  utils.RocksDBStore (RocksDBStore.java:<init>(68)) - Time to load library: 0 2019-10-15 13:20:10,723 INFO  utils.RocksDBStore (RocksDBStore.java:<init>(75)) - Time to open: 8 2019-10-15 13:20:10,723 INFO  helpers.KeyValueContainerUtil (KeyValueContainerUtil.java:createContainerMetaData(85)) - Total time to create: {}95

A good find. LGTM +1

Can we please use the Ambari inspired template?
With info like -- a link to the HDDS JIRA, a clear description of the problem and information on how it was tested and how you expect the code reviewer to test it ?
Thanks


QotN refator

👷 Deploy Preview for soshrine processing.

Name
Link

🔨 Latest commit
e807c6333fe651f1c042574c28dcee6ffd0b658f

🔍 Latest deploy log
https://app.netlify.com/sites/soshrine/deploys/637f7100a9b7d900080e9cba


Add OpenZeppeling Safe App

Test Case OpenZeppelin
To test the app you will need two things
a) An upgradeable contract (this is a proxy pointing to an implementation) owned by the Safe
b) A non-upgradeable contract (i.e. a new implementation for your contract)

First you need to write a contract in an upgradeable manner (without a constructor or with an initializer), a simple contract will do
https://docs.openzeppelin.com/upgrades/2.8/writing-upgradeable#initializers

Then you need to deploy it using the OZ CLI to get the proxy address
https://docs.openzeppelin.com/cli/2.8/getting-started#deploying-to-a-development-network

Once you have your proxy, transfer ownership to the Safe multisig with set-admin
https://docs.openzeppelin.com/cli/2.8/commands#set-admin

Deploy a new version of the initial contract, it can be the same contract in a new address (it will be easier this way so you don’t need to worry about storage layout issues), deploy it as a regular, non-upgradeable contract otherwise you’ll get a Proxy address

Now you have (a) a safe-owned proxy and (b) a new version of the contract behind the proxy ready to be upgraded through the Upgrades Safe App, just use those two addresses in the app form

if more than one person has to do this, I’d recommend writing a small script doing all of that and printing the two addresses you need
#IPFS URL
https://ipfs.io/ipfs/QmQovvfYYMUXjZfNbysQDUEXR8nr55iJRwcYgJQGJR7KEA/

With Fernando Greco we were able to try to test it in Rinkeby.
https://github.com/fernandomg/oz-sapp
https://safe-team.dev.gnosisdev.com/app/#/safes/0x9913B9180C20C6b0F21B6480c84422F6ebc4B808/transactions

the app was added in v2.10.0 release


Ruff target-version can be automatically inferred if requires-python is set

According to https://beta.ruff.rs/docs/settings/#target-version, the target-version property used for ruff can be inferred from the project.requires-python field, if it is present.
Repo-review currently requires the target-version to be set in
https://github.com/scientific-python/cookie/blob/884ef327f322ce1dcd23f803ad622b73cbb8fe34/src/sp_repo_review/checks/ruff.py#L35-L50
Based on the above, this check could be relaxed a little. Maybe to something like:
class RF002(Ruff):
    "Target version must be set"
    requires = {"RF001"}

    @staticmethod
    def check(pyproject: dict[str, Any]) -> bool:
        """
        Must select a minimum version to target. Affects pyupgrade,
        isort, and others. Can be inferred from project.requires-python.
        """

        match pyproject:
            case {"tool": {"ruff": {"target-version": str()}}}:
                return True
            case {"project": {"python-requires": str()}}:
                return True
            case _:
                return False

Or even something where it is suggested to not specify tool.ruff.target-version whenever project.python-requires is specified.

This sounds great. I’d go with allowing either for now, as you have above, and later we could make having both disallowed.

(I suspect this might be somewhat new. Also wondering how it handles more complex version boudaries)

This sounds great. I’d go with allowing either for now, as you have above, and later we could make having both disallowed.

Do you want me to submit a corresponding PR?

(I suspect this might be somewhat new. Also wondering how it handles more complex version boudaries)

The feature was requested in https://github.com/astral-sh/ruff/issues/2039 and implemented in https://github.com/astral-sh/ruff/pull/3470 on March 13, which went into ruff v0.0.255 (https://github.com/astral-sh/ruff/releases/tag/v0.0.255). astral-sh/ruff#2039 has some details on the handling of version ranges (mostly refers to how black handles this as of version 23.1.0).

Sure, please do.

(FYI, this check was added in repo-review 0.5, https://github.com/scientific-python/repo-review/releases/tag/v0.5.0 - in February, which explains why it didn't have this :) )


kctrl: Flag to create namespace when adding new repo

What this PR does / why we need it:
When adding a new package repository to a cluster, it's now possible to create the installation namespace automatically by specifying the "--create-namespace" flag.
Which issue(s) this PR fixes:
Fixes gh-1001
Does this PR introduce a user-facing change?
Added a new flag `create-namespace` for creating the installation namespace when adding a new package repository to a cluster.

Additional Notes for your reviewer:
After a first review of this pull request to validate the behaviour of the new flag, I will submit a PR to update the documentation on the project website.
Review Checklist:

[X] Follows the developer guidelines
[X] Relevant tests are added or updated
[] Relevant docs in this repo added or updated
[] Relevant carvel.dev docs added or updated in a separate PR and there's
a link to that PR
[X] Code is at least as readable and maintainable as it was before this
change

Additional documentation e.g., Proposal, usage docs, etc.:

@100mik @praveenrewar thanks a lot for your guidance and review


Configuration consolidation

What does this PR do?
This restructures the configuration slightly into more homogeneous chunks and bases the loading of config on CLI args.
The CLI args are also chunked up into logical pieces only composed together in OperationCliArgs which contains the whole set of configuration options of the operator (represented by the OperatorConfiguration struct). Similarly, in the sibling PR in the oauth service, there is OAuthServiceCliArgs and OAuthServiceConfiguration which compose the common options with the unique oauth service options.
Notably, baseUrl is no longer a configuration option in the configuration file and is instead loaded from the CLI arguments (or environment variable BASEURL).
The deployment files were updated to define the config options in the shared-config ConfigMap which is injected as environment variables into both operator and oauth service deployments.
What issues does this PR fix or reference?
https://issues.redhat.com/browse/SVPI-175
How to test this PR?
make deploy_minikube should work exactly the same as before.
Additional information
There is an infra-deployments PR that should be merged after SPI with these changes is released: https://github.com/redhat-appstudio/infra-deployments/pull/638

So there is one big configmap for all the values (shared, operator, oauth)? Isn't it better to separate them, as we discussed before?

So there is one big configmap for all the values (shared, operator, oauth)? Isn't it better to separate them, as we discussed before?

I have added the oauth-service and operator-specific config maps as well as the shared configmap, which are all bound to the environments of the respective deployments.
Only the shared config contains the non-default values, the component-specific configmaps are empty.
I still am not sure this is a good thing to do from the UX perspective, but I seem to be alone in that POV.


cli: sort the list of user-selected components

Closes #639

Codecov Report

Merging #640 (77e2e58) into master (7bddf3f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #640   +/-   ##
=======================================
  Coverage   18.88%   18.88%
=======================================
  Files          26       26
  Lines        2134     2134
=======================================
  Hits          403      403
  Misses       1731     1731

Impacted Files
Coverage Δ

reana/reana_dev/utils.py
41.44% <100.00%> (ø)


How to set domestic hot water temperature

Hi,
as a follow up to the comments on you blog: I've set up the feature heating.dhw.temperature.main, but Homebridge does actually only show the current temperature, but there is no way to change the value.
My config:
…
                {
                    "name": "Warmwasser",
                    "feature": "heating.dhw.temperature.main",
                    "deviceId": "0"
                },
…

The output:

Am I missing something or is it not possible to setup the temperature via homebridge/homekit?
Thanks and Cheers,
Sebastian

I have the same problem. I can see all the temperatures but can't change anything. They act like sensors rather than actual controls. Also, I have to click on the authorization link manually every time I restart HomeBridge, otherwise it just hangs there doing nothing.

Same with me. I can only read temperatures, but not change them.


Unused branches?

@johndyer
https://github.com/johndyer/mediaelement/tree/dispatch-event-with-this
https://github.com/johndyer/mediaelement/tree/338-double-progress-from-flash-after-setsrc
https://github.com/johndyer/mediaelement/tree/development
Are unused branches?

@johndyer I see that https://github.com/johndyer/mediaelement/tree/dispatch-event-with-this and https://github.com/johndyer/mediaelement/tree/338-double-progress-from-flash-after-setsrc were updated 2 years ago. Is there anything there you consider relevant to be integrated at least on development? Or can they be removed?

@isantolin Closing issue since the branches mentioned above have been removed


Launch as administrator from limited account fails

Windows Terminal version (or Windows build number)
10.0.19043.0
Other Software
No response
Steps to reproduce
From a limited account :

Right click windows terminal
Select run as Administrator

Expected Behavior
I expect to type my admin password and have an elevated command prompt
Actual Behavior
Password is requested twice
Error then appears

Have you installed the Windows Terminal app on your administrator account?

I installed it on the account I'm using. Although I did so before I switched the account from admin to limited.

I confirm I have the same issue with the 1.8.1521.0 version, with the same behaviour. I hadn't it with 1.7, however.
Installed on both 'admin' and 'standard' accounts, running fine in both icons, and elevated in the 'admin' account. But not as elevated in the 'standard' account.
And it has a "funny" side-effect: 'Run as Administrator' breaks the icon in the menu tiles (not on the taskbar), which becomes dark grey with a bold white line below; the only way to restore it is to remove the icon and re-add it.

I installed it on the account I'm using. Although I did so before I switched the account from admin to limited.

Okay, so that sounds like you don't have the Terminal installed on your Admin account? That's something that simply doesn't work today. See #7806, /dup #4217


Keep Dtab obervation activities open

For better performance: resolve endpoint goes from 300 qps to 14k qps.
Many thanks to @adleong !

Updated PR according to comments.

Tests failed but the failure looks unrelated. :/

C'mon, CircleCI, tests are 💯 . :)

This is awesome, thanks for taking this on, @Ashald.  Do you think the Addr Cache should get the same treatment as the Bind Cache?

Hm, in theory - yes. In practice - let me try to implement it and see the performance impact.
Do you want that as addition to this PR or as a new one?

If it makes sense to do then we might as well do it at the same time.

Yeah, this gives 20K QPS almost constantly. I'll update PR over weekend - hope you won't do release today. :)
The only question - how to name those case classes? :) NameActClsoe and AddrActClos? :)

Actually was able to do that while riding in metro. :) How does it look to you?

:star: Looks great!

Yaaay! :)

lgtm! though, i'd prefer that we use the least permissive visibility for intermediate types.

@olix0r does it look better now? :)

thanks, @Ashald


Cherry-pick bugfix to Branch 2.1

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

run starrocks_be_unittest

run starrocks_be_unittest

run starrocks_be_unittest


Do not flush eac3(joc) decoder on reuse

eac3(joc) decoders do not need to be flushed to be reused for the next compatible track. This change allows for gapless playback on devices with Dolby decoders that require internal re-initialization on flush.

Please abandon this PR as it was pushed via a non CLA account. Thanks


[BUG] puppeteer 2.0 page.pdf not work

Steps to reproduce
Tell us about your environment:

Puppeteer version: 2.0
Platform / OS version: debian
URLs (if applicable):
Node.js version: 11 or 12
Chrome: 79.0.3945.16

What steps will reproduce the problem?
Please include code that reproduces the issue.
const  puppeteer = require('puppeteer-core')
const browser = await puppeteer.launch({
  executablePath: 'google-chrome',
  args: ['--no-sandbox', '--disable-setuid-sandbox']
})
const page = await browser.newPage()
await page.goto('https://www.apple.com/')

// here
await page.pdf({})

What is the expected result?
page.pdf should works as expected
What happens instead?
Throw error
Error: Protocol error (IO.read): Invalid parameters handle: string value expected
    at /app/node_modules/puppeteer-core/lib/Connection.js:183:56
    at new Promise (<anonymous>)
    at CDPSession.send (/app/node_modules/puppeteer-core/lib/Connection.js:182:12)
    at Function.readProtocolStream (/app/node_modules/puppeteer-core/lib/helper.js:254:37)
    at Page.pdf (/app/node_modules/puppeteer-core/lib/Page.js:1024:25)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Object.Pdf (/app/services/pdf/index.js:39:18)
  -- ASYNC --
    at Page.<anonymous> (/app/node_modules/puppeteer-core/lib/helper.js:111:15)
    at Object.Pdf (/app/services/pdf/index.js:39:29)
    at processTicksAndRejections (internal/process/task_queues.js:93:5) {
  message: 'Protocol error (IO.read): Invalid parameters handle: string value expected'
}

Update: it works with chrome 80.0.3983.2


Default Worker Adapter not working

Hi! I'm having a hard time setting up. I want it to use my already working delayed jobs and Redis.
So I added Redis to my gemfile like this
gem 'counter-cache', '~> 0.3.0'
gem 'redis', '~> 3.2', '>= 3.2.2'

In my config file
class TestWorkerAdapter
  def enqueue(delay, base_class, options)
    options[:source_object_class_name] = base_class.constantize
    counter_class = options[:counter].constantize # options[:counter] is the class name of the counter that called the adapter.
    counter = counter_class.new(nil, options)
    counter.save!
  end
end

Counter::Cache.configure do |c|
  c.default_worker_adapter = TestWorkerAdapter.new
  c.recalculation_delay    = 5.hours
  c.redis_pool             = Redis.new
  c.counting_data_store    = Counter::Cache::Redis
end

If I run it like that it says
undefined method get' for Counter::Cache::Redis:Class`
If I don't put the line c.default_worker_adapter when executing it says
undefined method enqueue' for nil:NilClass`
If I don't put the  c.counting_data_store line  it says
undefined method enqueue' for nil:NilClass`
Any idea on what's going on? What should I put in the Worker Adapter?
Thank you for your time

Hi @letiesperon,
I'm not totally sure, but looking at this documentation you may want to change TestWorkerAdapter.new to TestWorkerAdapter.

Yes, sorry but that doesn't work either. I need an adapter for delayed jobs, do you know where I can find it? Thank you!!!

I creates this worker adapter
class CounterWorker
  CounterJob = Struct.new(:base_class, :options) do
    def perform
      p 'Updating counters...'
      options[:source_object_class_name] = base_class.constantize
      counter_class = options[:counter].constantize
      counter = counter_class.new(nil, options)
      counter.save!
      p 'Updated counters'
    end
  end

  def enqueue(delay_time, base_class, options)
    Delayed::Job.enqueue(CounterJob.new(base_class, options), run_at: delay_time.from_now)
  end
end

But it updates the database value every time. How can I check that it is not enqueued if it was already? I thought this was already implemented but maybe I have to add this logic in the adapter too?

@letiesperon - The unique job constraint should be a function of the background worker framework you are using. I am not sure if Delayed::Job supports uniquing or not. We have used sidekiq which supports uniq jobs.

Thank you for your reply. Delayed jobs does not support it without any additional plugin so I added the logic in the adapter using redis keys. I will close this issue.


vite tree shaking invalid

Describe the bug
In utils.ts export function never be imported.
But the result of build contains methods that are not introduced.
Reproduction
https://github.com/GreenMashimaro/vite-tree-shaking-bug-reproduce
System Info
System:
    OS: macOS 12.2.1
    CPU: (8) x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
    Memory: 200.75 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.10.0 - ~/.nvm/versions/node/v16.10.0/bin/node
    Yarn: 1.22.18 - ~/.nvm/versions/node/v16.10.0/bin/yarn
    npm: 7.24.0 - ~/.nvm/versions/node/v16.10.0/bin/npm
  Browsers:
    Chrome: 103.0.5060.134
    Safari: 15.3
  npmPackages:
    vite: ^3.0.3 => 3.0.3

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This behavior is correct.
First, when ./packages/utils/src/index.ts is tree-shaked it will be:
import serialize from 'serialize-javascript' // (!) cannot remove import because import has a side-effect

export function getName() {
  return 'hello_coco'
}

Then, serialize-javascript will be tree-shaked.
This module.exports part can be removed because it is not used.
But this part cannot be removed easily.
rollup needs to check that each statements has no side-effects.
For example:

rollup will need to check randombytes package to remove require('randombytes')
for the code below, rollup needs to check if generateUID is side-effectless or not. In this case, rollup detected UID is not used and generateUID is side-effectful. So in the generated output, the code like below exists.

var UID                 = generateUID();
// omit
function generateUID() {
    var bytes = randomBytes(UID_LENGTH);
    var result = '';
    for(var i=0; i<UID_LENGTH; ++i) {
        result += bytes[i].toString(16);
    }
    return result;
}

generateUID();
function generateUID() {
  var bytes = randomBytes(UID_LENGTH);
  var result = "";
  for (var i = 0; i < UID_LENGTH; ++i) {
    result += bytes[i].toString(16);
  }
  return result;
}

@sapphi-red Thank you very much for your detailed explanation.
However, the results did not meet expectations.
Because the testTreeShaking function has not been used, so we expect serialize-javascript to be removed by tree shaking.
Is there a way to implement tree shaking without importing serialize-javascript?

AFAIK the easiest way is to use patch-package or pnpm patch to annotate generateUID and other functions that has no side-effect as pure.
var UID                 = /* #__PURE__ */ generateUID();

https://github.com/yahoo/serialize-javascript/blob/c68e16a34e8074b27a0d67e2733c8c7c98ea53eb/index.js#L13

@sapphi-red Thank you so much for your answer, it can solve the problem and also open the door to a new world.


Favouriting events: Saving/removing an event or scrolling event listings page causes heart icons to flicker (iOS only)

Description of issue
When on the Events listing page, tapping to add or remove an event from the saved events will cause the heart icons to flicker
Expected behaviour
Tapping to add/remove an event doesn't affect the icons for other saved events.
Steps to reproduce

Open the "Events" page
Favourite the top 2 events
Repeatedly tap the heart icon for the third event

The heart icons for the top 2 events flicker as the icon for the third event is tapped.

OS / Browser / Device model
iPhone 6 (10.3.3)
iPhone 7+ (10.3.1)
Screenshots

I also noticed that scrolling with a bunch of favourited events visible in the events listing page will also result in the icons flickering:

Again, it only affects iOS so I assume it is related to the above issue - if not I can log it as a separate issue.
It is quite easy to reproduce on iPhone 6 - I have reproduced on iPhone 7+ once or twice but it's not reliably reproducible.

@declanslevin I could not reproduce this on master. There could be a chance someone has made a change to fix. Would you mind showing me this when you are in?

Adding Accessibility label as the flash icons may have accessibility concerns.

Fixed in #282


Fixed broken image and anchor.

should this be ../_static/ rather than /_static/ ???  or is the /_static translated by sphinx to a relative URL??

This source:
./doc/source/api/variants.rst:.. image:: /_static/variant_objects.svg

Turns into:
./target/generated-docs/merged/html/api/variants.html:<img src="../_images/variant_objects.svg" />

I'm not clear where in the build the path transformation is occurring.
If "../_static" works, then this becomes really a question of convention. The downside to relative paths is that if the .rst file is moved (as it was in this case), then the image include may break. No strong opinion, here.


Allow config to be partially updated

Description
Allow configuration to be partially updated.
Motivation and Context
This allows only some configuration variables to be changed, whilst leaving the defaults in-tact.
Types of changes
Put an x in all boxes that apply

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

Checklist:
Put an x in all boxes that apply
If you're unsure about any of these, don't hesitate to ask. We're here to help!

[x] My code follows the code style of this project.
[x] I have added/updated docstrings to the code I have touched.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[ ] I have updated the Changelog to reflect my changes
[ ] Has the changes been tested against a real Slack instance?

Test failures don't look related to this PR


Update quay.io/opdev/preflight:stable docker digest to add156…

Updates preflight image to the latest release and adjusts the path in which results are stored due to multi-arch support.

/ok-to-test

/retest

/retest

/retest

/retest

/retest

/retest

I think that this is the root cause, it looks like the intermittent error where secret is prematurely removed (or wrong secret is used?)

/retest


How to we can reduce the size of cookie?

What is in the cookie generated by IdentityServer that size of it is 1054?
Is the any ways to reduce size of it?
Thanks for your attention.

Depends on what you put in there.
The cookie is set my the MS middleware and there is a certain overhead for the protection added to it.


Fixing default value for vtcombo's sharding column type to be consist…

…ent.
@guokeno0 this is needed for import to work. Will work on more test cases.

LGTM

vtgatev2 flakiness is unrelated, merging.


Inputs outputs logs

Most notably, this PR creates a harvest_dag.py and moves all dag tasks definitions there. Keeping code local to the general case - running the full harvesting pipeline - was helpful in thinking about structuring the inputs and outputs as ideal to this general case. @bibliotechy I can move these back either temporarily (to make reviewing the changes made by this PR within each task a bit easier), or permanently if you think it makes more sense to functionally encapsulate these task definitions.
This PR also

standardizes on using params method to get dag run parameters (as opposed to dag_run.conf or get_current_context())
standardizes dags as taking a collection id while dag tasks should take a collection metadata resource from the registry API.
updates the fetcher task to print the fetcher task status, and return a list of successfully fetched page filenames, eliminating the need to then go get the vernacular pages as the first step of the mapping task in the full harvest pipeline (though this is still needed when running just the mapper_dag, and this may require some finessing when using an s3 data source)
standardizing on usage of the words "status" to refer to conceptual task status (number of pages successfully fetched, mapping enrichment errors encountered, etc), rather than "summary" and "report"

@bibliotechy I'm confused about the Dags Check failure. I'm able to run this locally without any dag import errors in the airflow UI or the airflow local-runner container.
This section on the Taskflow API suggests that you can import and reuse tasks definitions: https://airflow.apache.org/docs/apache-airflow/stable/tutorial/taskflow.html#reusing-a-decorated-task, but maybe I need to specify a specific-to-the-dag task id? I would have assumed tasks get stored as a dag-task, such that I could reuse them without having to differentiate them in a global namespace.

@amywieliczka I'm also confused by that. It looks like it is at the dag ID level, and not the task ID level that it is complaining. Maybe you can set a specific dag_id for each dag we define? But not sure why you would need to, as it sounds like it is supposed to pick up the id from the name of the function it is decorating, and they are different.


feat(header): Support arbitary status codes

As discussed in #177 hyper must support status code outside the standard range for compatibility reasons.
BREAKING CHANGE: This removes unregistered status codes from the enum. Use FromPrimitive methods to create them now.
StatusCode and StatusClass can no longer be casted to u16, use ToPrimitive methods now.
For example status.to_u16().unwrap() to get the status code number.

There were the following issues with your Pull Request

Commit: c14bffb64ba9f27b5a09e001c22b3aca4465777d
Your commit message body contains a line that is longer than 100 characters

Guidelines are available at https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com

Test failure was in a doctest, that StatusCode wasn't imported.

@pyfisch any news here? We'd like to land this to unblock servo/servo#117.

You mean https://github.com/servo/servo/issues/4246 :)

Wow I have no idea what I was looking at.

If anything is left to do tell me.

I fixed up a couple tiny nits, and pushed this as #311


expose more information to ConnectionValidator

not sure my request makes sense  😄
currently the MqttConnectionValidatorContext includes ClientId,Username, Password, WillMessage.

If I want to implement a broker that can support IP whitelisting, the remote endpoint maybe good to be added to the MqttConnectionValidatorContext.  IP whitelisting might be tied with ClientId or Username

also if the client connects to broker through websocket,  the reqeust url might be good to be added to MqttConnectionValidatorContext (if that is possible) .  The url might contain some information as query string that will participate the authentication

Just a thought.

you should add client certificates if you just inspect message content that is not save

In general this should be possible (no matter if it makes sense or not). Since version 2.8.0 the adapters have a Endoint as string defined which contains the remote IP and port.
I will check how to add this Endpoint to the validator.
Adding more information from the WebSocket channel is more complicated because it has features which are not avaialble for TCP so me must add interfaces etc.

I update to version V2.8.2.  Now I can access the MqttConnectionValidatorContext.Endpoint.
However, I did a test, the context.Endpoint is string "0HLGFACF8503R"
I expect it would be in the format of "ip:port"
I did a simple snippet,
var a = new IPEndPoint(new IPAddress(new byte[] { 192, 168, 0, 1 }), 8000);
var s = a.ToString();

s would be "<IP_ADDRESS>:8000"
Am I missing something?
Thanks,

After I looked at the source code, I think the reason the Endpoint is something like  "0HLGFACF8503R" rather "ip:port" is because I used the ASP.NET Core 2.1  new TCP transport.
In the file MQTTnet\Source\MQTTnet.AspnetCore\MqttConnectionContext.cs
 public string Endpoint => Connection.ConnectionId;

It uses ConnectionId
Should it be something like this instead?
 public string Endpoint => (Connection as TcpConnection)._endPoint.ToString();

Of course, the _endPoint has to be changed to be public first.

TcpConnection is client side implementation of Connection.
on the server it is  Microsoft.AspNetCore.Http.Connections.Internal.HttpConnectionContext.
so please create an issue at https://github.com/aspnet/KestrelHttpServer/
currently I dont see any features that expose information about the remote endpoint (https://github.com/aspnet/KestrelHttpServer/tree/release/2.2/src/Connections.Abstractions/Features)

HttpConnectionContext add feature to expose RemoteEndPoint #2875
Will see what happen. :smiley:
Thanks,

@halter73 suggested a solution in the issue
https://github.com/aspnet/KestrelHttpServer/issues/2875#issuecomment-418473684
IHttpConnectionFeature has the remote end point information.
 the same way the HttpConnectionMiddleware does
would this work?

will try it out

@JanEggers Do you have a status update for this?

nope

At this point, I'd probably just wait for ASP.NET 3.0 to do anything on this, since starting in 3.0, ConnectionContext will directly expose the RemoteEndpoint similarly to HttpContext.Connection.

thx for letting us know

This should be fixed with https://github.com/chkr1011/MQTTnet/pull/882 (For NetCore 3.1+).


Add support for very small widget size (1x1 or 1x2 cells)

Maybe show only the number of available events today and/or tomorrow.

As we have much more flexible layouts today, maybe this is mostly done.
BTW, nobody asked for a layout for wearable Android watches yet :-)


Installation under windows

Hello @NikolovI
We would like to update our docs in order to make them suitable for folks who develop on Windows.
Could you share couple screenshots and steps you've went through on your Windows system?
Thanks in advance!

Hello,
I'm adding instructions draft, please let me know if it will do the job:

Install Cygwin

When selecting Cygwin packages make sure you have selected "make" and "git" packages (latest version should be fine)

Install Docker Desktop for Windows and make sure it is running. You should be able to see the empty docker dashboard

Open Cygwin terminal and navigate to your project folder

Run the "make init" command, follow the instructions for user creation and, if there are no errors in the terminal, you should be able to see the docker container running.

Open http://localhost:8086/ or click on the "Open in browser" button in the basicrum_bo_nginx container to view the back-office

If you like to have the demo data installed, in the Cygwin terminal, run the "make demo" command

After the initial install process is completed you can use the Cygwin terminal "make up" command to start your docker instance again when needed (e.g. after restart)

Thank you @NikolovI
We will wrap your notes in a how to article soon.
Small questions about:
1 ) Install Cygwin
Do you download it from here: https://www.cygwin.com/
2 ) When selecting Cygwin packages make sure you have selected "make" and "git" packages (latest version should be fine).
Did you get a popup about the packages option when you started an installation process? If possible could you add a screenshot that shows a bit more about the context/where are you seeing the things on the UI?

Thank you @NikolovI
We will wrap your notes in a how to article soon.
Small questions about:
1 ) Install Cygwin
Do you download it from here: https://www.cygwin.com/

Yes, this is the required software

2 ) When selecting Cygwin packages make sure you have selected "make" and "git" packages (latest version should be fine).
Did you get a popup about the packages option when you started an installation process? If possible could you add a screenshot that shows a bit more about the context/where are you seeing the things on the UI?

This is part of the setup process. Basically you're installing packages that you need, because Cygwin contains tons of stuff and you can't install everything. On my screenshots above you see only the part of the setup wizard with the needed packages. The entire, unfiltered screen looks like this:

Main things are the Current/New columns of the table. Current will show the installed version (if installed). New will allow you to install/update to specific version. If the drop-down menu has Skip selected - the package will not be installed.
Hope this makes more sense :)

Thanks @NikolovI
I ported your notes here: https://github.com/basicrum/backoffice/blob/master/doc/installation/WINDOWS.md


How to get Product Availability Info

Hi,
Is there a way to find availability of the product. Product availability in the response xml is under 'Offers.Offer.OfferListing.AvailabilityAttributes.AvailabilityType' element.
Thanks

There seems to be no "easy" (high level) method provided by this library, but you can access any XML node by using:
product._safe_get_element_text('Offers.Offer.OfferListing.AvailabilityAttributes.AvailabilityType')

I implemented a property to easily access the information you need via:
availability_type:
product.availability_type

Other availability related properties are also available now:
availability (Offers.Offer.OfferListing.Availability )
availability_min_hours (Offers.Offer.OfferListing.AvailabilityAttributes.MinimumHours )
availability_max_hours (Offers.Offer.OfferListing.AvailabilityAttributes.MaximumHours )

Thanks Norman!!
On Feb 21, 2017 3:44 PM, "Norman"<EMAIL_ADDRESS>wrote:
I implemented a property to easily access the information you need via:
availability_type
https://github.com/yoavaviram/python-amazon-simple-product-api/blob/master/amazon/api.py#L1338-L1344
:
product.availability_type
Other availability related properties are also available now:
availability (Offers.Offer.OfferListing.Availability)
availability_min_hours (Offers.Offer.OfferListing.AvailabilityAttributes.
MinimumHours)
availability_max_hours (Offers.Offer.OfferListing.AvailabilityAttributes.
MaximumHours)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yoavaviram/python-amazon-simple-product-api/issues/97#issuecomment-281301059,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXOhGHm9NQIzrljfOHGrMKLyUySR4fUjks5rerkQgaJpZM4LDSvU
.


jetson-tx2-devkit: kernel internal wireless regdb is disable by default

Signed-off-by: Ilies CHERGUI<EMAIL_ADDRESS>

The driver issue has been fixed.


Feature Request:  dynamic_cast i.e. optional runtime structural type checks

Motivation
Verify inputs at application boundary.
When a Typescript application interacts with an external system (e.g. receives a Json object from a http request, or receives an object from an external library) a typical approach is to use the cast operator for the input and to assume/hope that the application receives the required type.
If/when the external system changes the assumption can be broken resulting in sometimes hard to find bugs and unhelpful runtime errors like "undefined is not a function".
These bugs are also quite hard to catch in tests since unit tests isolate the external system, only integration tests would be able to catch these scenarios.
Proposal
A new cast operator could be introduced that does structural type checking at runtime.
Example:
$http.get('https://mySite.com/products', ( result: { data: IProduct[] } ) =>{
    var data:IProduct[] = dynamic_cast<IProduct[]>result.data;
});

The Typescript compiler could operate in 2 modes ( debug / release or runtime type checks on/off)
In release the compiler would simply remove the dynamic_cast operator.
In debug mode the compiler would wrap the expression in a typeCheck call. It would generate something like this:
$http.get('https://mySite.com/products', function( result ) {
    var data:IProduct[] = IProduct.CheckType(result.data);
});

The tricky bit comes in implementing the CheckType() function.
Interfaces that are dynamically checked would suddenly acquire a runtime footprint.
Also any interface and Type that appear as the type of members of any interface and type that are type checked would also need to receive a ChekType static member function.
It would be still nicer if it would be possible to support a syntax like:
$http.get('https://mySite.com/products', ( result: dynamic_cast<{ data: IProduct[] }> ) =>{
    var data:IProduct[] = result.data;
});

Although it is not very clear (to me) where would the cast function be scoped and how it would be called, probably the lambda would be wrapped in something ugly like:
$http.get('https://mySite.com/products',
    function (result){
        return (function( result ) {
            var data:IProduct[] = IProduct.CheckType(result.data);
        })( ???.CheckType(result));
});

looks like a duplicate of  #1549


[paper-64+] Chat no longer works and throws exception in 4.0.0-snapshot

So, whenever someone tries to use the regular chat in the game, this exception is thrown and spammed in console, while not sending the chat messages.
https://pastebin.com/zPZZCCa6
Please note this only happens in paper versions 64 and newer, and it happens with both 3.0.0 and 4.0.0. This async event event happens in older versions of paper as well, but the chat still works.

Hm, I don't have this with the latest version and I'm currently using the very latest version of paper right now. I suppose plugin conflict is not possible in this case? Can you try?

Yep.
So from what I got so far, the only plugins I could think that would be causing these issues could be viaversion, essentials, quickshop, cmi and luckperms (which is my permissions plugin). I've disabled all of them and it seems the chat is still not working, throwing the same exception.
Version of paper I am using is paper-68 to test these with, and I've redownloaded it just to make sure, but it's the same issue.
Here is the console dump from the command.
https://puu.sh/DD701/7a718c7425.png

It seems likely to be a config issue? Seeing as how I don't have the error. Maybe Pastebin your config as well.

Okay, here it is.
https://pastebin.com/1tzpChXb
On Sat, Jun 8, 2019 at 7:52 PM Jad<EMAIL_ADDRESS>wrote:

It seems likely to be a config issue? Seeing as how I don't have the
error. Maybe Pastebin your config as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Multiverse/Multiverse-Core/issues/2011?email_source=notifications&email_token=AB4T2YUTZK56EVFKQJB5CW3PZN6MRA5CNFSM4HVPW3Y2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXHRRTY#issuecomment-500111567,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB4T2YSVS2EE6UAXG47VZVTPZN6MRANCNFSM4HVPW3YQ
.

prefixchatformat: '%chat' should be prefixchatformat: '%chat%'
Let me know if it fixes your issue.

Alright, that actually fixed the issue. I'm not sure how a simple thing managed to completely wipe out chat, but, it works now. Thank you!


(v4.0.2) Labels don't display when label show is programmatically set to true

One-line summary [问题简述]
Labels don't display when label show is dynamically set to true. Rn labels only display after show is set to true and we hover the items.
Version & Environment [版本及环境]

ECharts version [ECharts 版本]: 4.0.2
Browser version [浏览器类型和版本]: latest Chrome
OS Version [操作系统类型和版本]: latest Mac OS

Expected behaviour [期望结果]
Labels should display when label show is dynamically set to true.
ECharts option [ECharts配置项]

var option = {
    xAxis: {
        type: 'category',
        data: ['Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun']
    },
    yAxis: {
        type: 'value'
    },
    series: [{
        data: [120, 200, 150, 80, 70, 110, 130],
        type: 'bar',
        itemStyle: {
            color: 'blue'
        },
        emphasis: {
            itemStyle: {
                color: 'blue'
            }
        },
        label: {
            // Initially, it should be false
            show: true
        }
    }]
};

var configs = {
    renderer: 'svg'
}

Other comments [其他信息]

http://echarts.baidu.com/examples/editor.html?c=bar-simple The bug persists even in the sandbox. Take the simple bar as the example. If you copy the above option to the editor (remember to set show to be false).  Hit render and then change show to true. (The sandbox will auto update the chart even without hitting render again. So I didn't hit render here)  Labels don't show up until we hover items one by one.

Do you mean that a label with initial show: false is still not visible when set to be show: true later? I cannot reproduce that.

@Ovilia Hi, thanks for your reply. We find a workaround on our end even though it's hacky.
We notice that it is not reproduced on our end either. But We swear we witnessed it the other day which is the reason we created this issue.
More background info:

We are using Ember as the front-end framework.
We have a config panel that allows users who wanna customize the chart. There is a checkbox option that controls the display of labels for data and defaults to be not showing the labels. (show: false)
Under the hook, when the checkbox is selected, it calls some internal render method to re-render(or re-create) the chart. Let's call this method renderChart.
Rn I know this renderChart is called, options is updated and show: true is present in the options. But as what aforementioned, labels don't show up until we hover items one by one.
Before we upgraded Echarts to v4.0.2, we were using v3.6.1 and it worked fine. (no code changed).
The workaround is to force the chart to destroy and re-create when the checkbox selected/unselected which is pretty expensive.

So it cannot be reproduced with v4.0.2? Anything you need me to do?

@Ovilia Sorry, I didn't make it clear. http://echarts.baidu.com/examples/editor.html?c=bar-simple The bug persists even in the sandbox.  Cannot reproduce anymore.
Any information on the mechanism of rendering labels would be appreciated.

I changed the code of http://echarts.baidu.com/examples/editor.html?c=bar-simple to be that in your first post, and set show: false, then render it, then set show: true. The label will show at last. I can't understand what's the problem here?

I think I need to make a video to help me explain.

var data = [{
    name: 'Grandpa',
    children: [{
        name: 'Uncle Leo',
        value: 15,
        children: [{
            name: 'Cousin Jack',
            value: 2
        }, {
            name: 'Cousin Mary',
            value: 5,
            children: [{
                name: 'Jackson',
                value: 2
            }]
        }, {
            name: 'Cousin Ben',
            value: 4
        }]
    }, {
        name: 'Aunt Jane',
        children: [{
            name: 'Cousin Kate',
            value: 4
        }]
    }, {
        name: 'Father',
        value: 10,
        children: [{
            name: 'Me',
            value: 5,
            itemStyle: {
                color: 'red'
            }
        }, {
            name: 'Brother Peter',
            value: 1
        }]
    }]
}, {
    name: 'Mike',
    children: [{
        name: 'Uncle Dan',
        children: [{
            name: 'Cousin Lucy',
            value: 3
        }, {
            name: 'Cousin Luck',
            value: 4,
            children: [{
                name: 'Nephew',
                value: 2
            }]
        }]
    }]
}, {
    name: 'Nancy',
    children: [{
        name: 'Uncle Nike',
        children: [{
            name: 'Cousin Betty',
            value: 1
        }, {
            name: 'Cousin Jenny',
            value: 2
        }]
    }]
}];

option = {
    visualMap: {
        type: 'continuous',
        min: 0,
        max: 10,
        inRange: {
            color: ['#2D5F73', '#538EA6', '#F2D1B3', '#F2B8A2', '#F28C8C']
        }
    },
    series: {
        type: 'sunburst',
        data: data,
        radius: [0, '90%'],
        emphasis: {
            label: {
                show: true,
                color: 'red'
            }
        },
        label: {
            show: false
        }
    }
};

@Ovilia
same issue emphasis not working

Same issue here. Force setting the label.show = true, it works fine if the width value of the chart bigger:

When make the width smaller, some labels disapeared:

https://echarts.apache.org/examples/zh/editor.html?c=bar-simple
option = {
    xAxis: {
        type: 'category',
        data: ['Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu', 'Fri',
        'Sat', 'Sun', 'Tue', 'Wed', 'Thu',
        'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu',
        'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu',
        'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu',
        'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu',
        'Fri', 'Sat', 'Sun', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat', 'Sun']
    },
    yAxis: {
        type: 'value'
    },
    series: [{
        data: [120343734534534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>5340,<PHONE_NUMBER>53410,<PHONE_NUMBER>53410,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>5340,<PHONE_NUMBER>53410,<PHONE_NUMBER>53410,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>5340,<PHONE_NUMBER>53410,<PHONE_NUMBER>53410,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534,
<PHONE_NUMBER>34534,<PHONE_NUMBER>45340,<PHONE_NUMBER>34534,<PHONE_NUMBER>4534],
        type: 'line',
        label: {show:true}
    }]
};


Is there use openfire server (XMPP) in this project?

Bro, I need to know which database you are using here. If you tell. It'll very useful to me.

It's working with Openfire server, you have to change the domain name to yours.
But I think they are using their own XMPP implementation. https://github.com/esl/MongooseIM


auto-merge and auto-approve fixes, and other improvements

additional context https://github.com/std-uritemplate/std-uritemplate-csharp/actions/runs/9803350207/job/27079082638?pr=8

depends on #9 to be merged first


Fix container name defaults

Adjust the multi-container default names to ensure they are in line with the API requirements. It also aligns the default spec with the generated ones.

😑 I forgot to adjust some of the reducers...

Adjust the rest of the reducers...

💯 🎭 👍  LGTM! 🎆


Add constraint

CODE REVIEW
This implements backtracking logic as described here: http://www.iccm-central.org/Proceedings/ICCM16proceedings/contents/pdf/MonK/MoKA1-04ge_ghiasimh224461p.pdf
The object passed into the Nelder-Mead function now needs to return a bool indicating if any constraint has been hit, that is either the guessed vertex or some dependent value has been hit.  If it has, the vertex will attempt to backtrack using some constant scalar.  If the backtracks cannot complete successfully, that is, this could not backtrack to a successful value, we return the best value we had guessed at the time of the backtrack.  If the initial guess the user provides does not conform to constraints, we return that guess as the best guess.
@beaulyddon-wf @rosshendrickson-wf @alexandercampbell-wf @tannermiller-wf @ericolson-wf @stevenosborne-wf @tylertreat-wf

+1

+1

+1


Allow for clean exit if branch is already deleted

Fixes #18.
Also reorganize for happy path.
(ref - https://medium.com/@matryer/line-of-sight-in-code-186dd7cdea88)

ah so sorry can you rebase since I merged the other PR first :)

onnit

I seem to have had a git-stroke. I thought I destroyed this branch when I rebased. This PR was supposed to just check for 422 and exit nicely, but I apparently merged this into #21 before I opened any PRs.
Nothing to see here, except my facepalm.


Github service: cannot unlink after deleting account

I have changed my github username and since it is not possible with readthedoc, I have deleted my account and created a new one.
When I try to connect readthedoc to github, it says The social account is already connected to a different account.. I have tried to revoke the connection to readthedoc on github but the message keeps appearing.
When I open https://readthedocs.org/accounts/github/login/?process=login, it redirects me to https://readthedocs.org/accounts/inactive/

Project URL: https://github.com/nlgranger/LazyProc
Build URL (if applicable): https://travis-ci.org/nlgranger/LazyProc

Thanks for your report.
This is a known problem and it will be solved once #3214 will deployed, I think.

Ok thanks, I will keep this issue open until then.

@agjohnson Can you manually delete the account?

@nlgranger What is the name of the account that is currently inactive? I'll reactivate so you can delete this account.

The account was pixelou

Activated! There may be a problem with removing your user, so you'll be our test subject. Report back if you hit any errors removing your user.

Should I keep the github link active on the old account while I delete it again or the cleanup functionnality is not active yet? Should I delete the projects from the account as well?


Lenses can't be created for classes with private constructors for Scala 3

Hello,
I've noticed that the code that compiles and works fine in scala 2 doesn't work for scala 3.
Scastie:
Scala 2 -> https://scastie.scala-lang.org/FmuLKaM0TpmNvQ3xHdpUSg
Scala 3 ->  https://scastie.scala-lang.org/7FBOxnIPQbemh5TomJrUig
Code:
import monocle.syntax.all._

case class A private (field1: String, field2: String)
object A {
  def apply(field1: String, field2: String): A = new A(field1, field2)

  val copiedObject = A("1", "2").copy(field1 = "3")
  val modifiedObject = A("1", "2").focus(_.field1).replace("3")

  println(copiedObject)
  println(modifiedObject)
}

val copiedObject = A("1", "2").copy(field1 = "3") //compile error
val modifiedObject = A("1", "2").focus(_.field1).replace("3") //compile error
println(copiedObject)
println(modifiedObject)

Error:
missing argument for parameter field2 of method copy in class A: (field1: String, field2: String): Playground.A
method copy cannot be accessed as a member of (from : Playground.A) from module class Playground$.

Could you please clarify whether it is expected behavior?

I believe it is expected when you use private in your case class definition.

Thank you for the response. Should the library behavior be the same for Scala 2 and 3? I expect that people migrating their codebase from Scala 2 to Scala 3 may face this issue.

Ideally, yes it should be the same but I believe this is due to a change of semantic between Scala 2 and 3 for private case class constructors. So Monocle is just following the Scala language.


Victoria 3 filepath mod vs. game assets

Hi,
sorry for being annoying about issues, but I found another minor one.
when referencing texture assets in Victoria 3 they are referenced gfx/... but are placed in game/gfx/...
In CWT file it just references filepath. This works for mod files but not for base game files.
Example of a modded script and base game texture is:
pmg_street_lighting = {
	texture = "gfx/interface/icons/generic_icons/mixed_icon_base.dds"
	production_methods = {
		pm_no_street_lighting
		pm_gas_streetlights
		pm_electric_streetlights
	}
}

Texture exists in: <steamapps>\Victoria 3\game\gfx\interface\icons\generic_icons\mixed_icon_base.dds
Here an example of a modded script and texture:
mana = {
	texture = "gfx/interface/icons/goods_icons/mana_crystals.dds"
	cost = 30
	category = staple
	tradeable = no

	prestige_factor = 8

	consumption_tax_cost = 200
}

Texture exists in: <project>\mod\gfx\interface\icons\goods_icons\mana_crystals.dds
IDE Setup:

Regards,
Chris

If dlcPath="game" in launcher_settings.json (this json file should under <gameRootPath> or <gameRootPath>/launcher), plugin should works fine.
It seems dlcPath!="game" for VIC3.
So paste launcher_settings.json here plz?

This issue is finally caused by the game/ path prefix in plugin builtin configs which are forked from cwtools-vic3-config repo, with no more compatible processing.
Plugin should work fine if you remove these game/ path prefixes in your local config files.

This issue is finally caused by the game/ path prefix in plugin builtin configs which are forked from cwtools-vic3-config repo, with no more compatible processing.
Plugin should work fine if you remove these game/ path prefixes in your local config files.

Sorry I'm not sure this is right, but plugin should do more compatible work for cwt configs indeed.
Like upper case identifiers (ROOT = COUNTRY, and the possible game/ path prefix ...)

Sorry but I modified the CWT for goods. For me it is only filepath. Missed adding it to my post.
I tested modifying the CWT files and when I use texture = filepath[game] base game images are found but not modded ones. And if I just use texture = filepath in the mod images are found but not base game images.
Also because you mentioned it in a deleted post. Here are my launcher settings:
{
	"formatVersion": 1.1,
	"gameId": "victoria3",
	"displayName": "Victoria 3",
	"version": "1.7.5 (Kahwah)",
	"rawVersion": "1.7.5",
	"distPlatform": "steam",
	"ingameSettingsPath": "pdx_settings.json",
	"gameDataPath": "%USER_DOCUMENTS%/Paradox Interactive/Victoria 3",
	"modPath": "mod",
	"modExtension": "zip",
	"modStubberPath": "../binaries/victoria3.exe",
	"modStubberData": {
		"args": [ "--mod_stubber" ],
		"type": "MOD_NAME_PROMPT"
	},
	"isLooseFilesModSupported": true,
	"distPlatformModsEnabled": true,
	"dlcPath": "../game",
	"ingameSettingsLayoutPath": "settings-layout.json",
	"themeFile": "./assets/theme-settings.json",
	"browserDlcUrl": "",
	"browserModUrl": "",
	"exePath": "../binaries/victoria3.exe",
	"exeArgs": [ "-gdpr-compliant" ],
	"alternativeExecutables": [
			{
		"label": {
			"de": "Spiel im Debug-Modus öffnen",
			"en": "Open game in Debug Mode",
			"es": "Abrir juego en modo de depuración",
			"fr": "Ouvrir le jeu en mode débogage",
			"ja": "デバッグモードでゲームを開",
			"ko": "디버그 모드로 게임 열기",
			"pl": "Uruchom grę w trybie debugowania",
			"pt": "Abrir jogo em modo debug",
			"ru": "Запустить игру в режиме разработчика",
			"zh-hans": "以调试模式打开游戏"
		},
		"exePath": "../binaries/victoria3.exe",
		"exeArgs": [ "-gdpr-compliant","-debug_mode" ]
		}
	]
}

Paradox-Language-Support-1.3.19.zip
Hi, I just uploaded a pre-release of 1.3.19, please check this. (IDE should reindex after plugin updated)

Hi it seems to work and it does not at the same time:

The path is now correctly found and clickable
But the group still says that the image does not exist

Path working and clickable:

Warning that the file does not exist:

What I tried:

Restarting IDE
Modifying file and texture path so it is reindexed
Inavlidated IDE caches and forced full reindex

The code logic about paradox path aka path shown in quick documentation may need to be  refactored,
for, some how, there are some entry paths (so named in plugin code) in different games.
e.g, for Stellaris, pdx_online_assets is an entry path.

Think of the file <gameRootPath>/pdx_online_assets/gfx/interface/online_feed_background.dds,
Can it be referenced by pdx_online_assets/gfx/interface/online_feed_background.dds or gfx/interface/online_feed_background.dds, or by pdx_online_assets/gfx/interface/online_feed_background.dds everywhere and by gfx/interface/online_feed_background.dds under pdx_online_assets.
I may need ask tboby or pdx for this question.

Paradox-Language-Support-1.3.19.zip

Worked like a charm.
Thank you as always!

Will be fixed in 1.3.19, but code logic about file path may need to be refactored in the future


Remove version python-3.7 from pyproject.toml, update lock

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Proposed Changes

Remove version python-3.7 from pyproject.toml
Update poetry.lock
Add poetry.toml into gitignore

Closing issues

Codecov Report

Merging #1183 (adbc280) into master (4041a08) will decrease coverage by 79.76%.
The diff coverage is n/a.

:exclamation: Current head adbc280 differs from pull request most recent head 7217527. Consider uploading reports for the commit 7217527 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master   #1183       +/-   ##
==========================================
- Coverage   87.62%   7.86%   -79.76%
==========================================
  Files         177     177
  Lines       10326   10322        -4
==========================================
- Hits         9048     812     -8236
- Misses       1278    9510     +8232

see 161 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


error when inserting osu! stream version into db

reported by -lokillo-#9990 in discord
lokillo@DESKTOP-KKAONBB:~/bancho.py$ ./main.py
[06:24:42AM] Fetching channels from sql.
[06:24:42AM] Fetching clans from sql.
[06:24:42AM] Fetching mappools from sql.
[06:24:42AM] Initializing housekeeping tasks.
[06:24:42AM] Startup process complete.
[06:24:42AM] Listening @ /tmp/bancho.sock
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/home/lokillo/.local/lib/python3.9/site-packages/anyio/streams/memory.py", line 81, in receive
    return self.receive_nowait()
  File "/home/lokillo/.local/lib/python3.9/site-packages/anyio/streams/memory.py", line 76, in receive_nowait
    raise WouldBlock
anyio.WouldBlock

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/base.py", line 41, in call_next
    message = await recv_stream.receive()
  File "/home/lokillo/.local/lib/python3.9/site-packages/anyio/streams/memory.py", line 101, in receive
    raise EndOfStream
anyio.EndOfStream

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/lokillo/.local/lib/python3.9/site-packages/uvicorn/protocols/http/h11_impl.py", line 373, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in __call__
    return await self.app(scope, receive, send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/fastapi/applications.py", line 208, in __call__
    await super().__call__(scope, receive, send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/applications.py", line 112, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/errors.py", line 181, in __call__
    raise exc
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/errors.py", line 159, in __call__
    await self.app(scope, receive, _send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/base.py", line 63, in __call__
    response = await self.dispatch_func(request, call_next)
  File "/home/lokillo/bancho.py/app/api/init_api.py", line 61, in http_middleware
    return await call_next(request)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/base.py", line 44, in call_next
    raise app_exc
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/base.py", line 34, in coro
    await self.app(scope, request.receive, send_stream.send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/base.py", line 63, in __call__
    response = await self.dispatch_func(request, call_next)
  File "/home/lokillo/bancho.py/app/api/middlewares.py", line 22, in dispatch
    response = await call_next(request)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/base.py", line 44, in call_next
    raise app_exc
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/middleware/base.py", line 34, in coro
    await self.app(scope, request.receive, send_stream.send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/exceptions.py", line 82, in __call__
    raise exc
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/exceptions.py", line 71, in __call__
    await self.app(scope, receive, sender)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/routing.py", line 656, in __call__
    await route.handle(scope, receive, send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/routing.py", line 472, in handle
    await self.app(scope, receive, send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/routing.py", line 656, in __call__
    await route.handle(scope, receive, send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/routing.py", line 259, in handle
    await self.app(scope, receive, send)
  File "/home/lokillo/.local/lib/python3.9/site-packages/starlette/routing.py", line 61, in app
    response = await func(request)
  File "/home/lokillo/.local/lib/python3.9/site-packages/fastapi/routing.py", line 226, in app
    raw_response = await run_endpoint_function(
  File "/home/lokillo/.local/lib/python3.9/site-packages/fastapi/routing.py", line 159, in run_endpoint_function
    return await dependant.call(**values)
  File "/home/lokillo/bancho.py/app/api/domains/cho.py", line 115, in bancho_handler
    login_data = await login(await request.body(), ip, db_conn)
  File "/home/lokillo/bancho.py/app/api/domains/cho.py", line 606, in login
    await db_conn.execute(
  File "/home/lokillo/.local/lib/python3.9/site-packages/databases/core.py", line 295, in execute
    return await self._connection.execute(built_query)
  File "/home/lokillo/.local/lib/python3.9/site-packages/databases/backends/mysql.py", line 149, in execute
    await cursor.execute(query_str, args)
  File "/home/lokillo/.local/lib/python3.9/site-packages/aiomysql/cursors.py", line 239, in execute
    await self._query(query)
  File "/home/lokillo/.local/lib/python3.9/site-packages/aiomysql/cursors.py", line 457, in _query
    await conn.query(q)
  File "/home/lokillo/.local/lib/python3.9/site-packages/aiomysql/connection.py", line 428, in query
    await self._read_query_result(unbuffered=unbuffered)
  File "/home/lokillo/.local/lib/python3.9/site-packages/aiomysql/connection.py", line 620, in _read_query_result
    await result.read()
  File "/home/lokillo/.local/lib/python3.9/site-packages/aiomysql/connection.py", line 1103, in read
    first_packet = await self.connection._read_packet()
  File "/home/lokillo/.local/lib/python3.9/site-packages/aiomysql/connection.py", line 591, in _read_packet
    packet.check_error()
  File "/home/lokillo/.local/lib/python3.9/site-packages/pymysql/protocol.py", line 220, in check_error
    err.raise_mysql_exception(self._data)
  File "/home/lokillo/.local/lib/python3.9/site-packages/pymysql/err.py", line 109, in raise_mysql_exception
    raise errorclass(errno, errval)
pymysql.err.IntegrityError: (1048, "Column 'osu_stream' cannot be null")
[06:24:55AM] [GET] 401 osu.ppy.sh/web/bancho_connect.php | Request took: 730.50 μsec```

closed with 2f972bbbe11ea9076d2eed9b527f300aec833def


Permission Denied to Laravel Log when installing with Docker

This is  a bug.

Prerequisites

[ x] Are you running the latest version?
[ x] Are you reporting to the correct repository?
(This project is made of many specialized packages: https://github.com/modularsoftware)
[ X] Did you check the documentation?
[ X] Did you perform a cursory search?

Description

When using the instructions from Docker.md instead of being able to view the web app you're met with a permissions error
Steps to Reproduce
Run the "Installation Using Docker" instructions in the Docker.md file
Expected behavior
Should have been met with the Genealogy login screen
Actual behavior
Error screen:
UnexpectedValueException The stream or file "/application/storage/logs/laravel.log" could not be opened: failed to open stream: Permission denied
Remedied by running the following commands:
docker-compose run --rm --no-deps genealogy-server php artisan cache:clear
docker-compose run --rm --no-deps genealogy-server chmod -R 777 storage/
docker-compose run --rm --no-deps genealogy-server composer dump-autoload

Added to documentation for the new branch and removed references to unused Neo4J.


Do we really need to only show docker logs with workflow_dispatch?

Right now, our action only takes into account the SHOW_DOCKER_LOGS input if the action is triggered by a workflow. What was the reason for that?

My mistake. This is taking in the action input. We're not handling workflow_dispatch inputs at all (github.events.inputs.foo).


Allow override of SDK and toolchain locations

Currently they are hard coded, this issue represents designing and implementing some way of overriding those locations that works on Mac and Windows.

Particularly solved in #6.


Fixed the return value of PLATFORM.architecture when MacOS is Ventura ( #31782 ).

i apologize if my PR is wrong. I'm not used to sending PRs. 🙇
i fixed the bug(#31782).
please read this issue for more information.

@apolcyn
PR review please 🙇


DRILL-5733: Unable to SELECT from parquet file with Hadoop 2.7.4

DRILL-5733: Unable to SELECT from parquet file with Hadoop 2.7.4
Description
For the case when a single parquet file was selected, Drill built pass for metadata cache files like this is a directory. It caused errors for Hadoop. Added check to verify that the parent path is a directory before constructing a path for metadata cache files.
Documentation
NA
Testing
Checked manually on Apache Hadoop cluster, checked that fix doesn't break existing unit tests. No test is provided since the error is not reproduced even with MiniDFSCluster.

+1, LGTM.

Sorry, missed a question about Hadoop 3. I have checked also with Hadoop 3.2.0 (previously was checked with Hadoop 2.8), and it works fine, even without this fix.


[Announcement] GitHub Issues Will Be Closed by 30 Nov

Hi students of CS2030,
The Github issues page and wiki will be closed from 30 Nov onwards. Please make your final contributions to the issues and the wiki before that date.

Can we create new pages in the wiki?

Nope, by then I will close the wiki as well. If you want to do it. Today is the last day.


Installing gekko in Virtual Server with ubuntu 16.04.02 npm installation issues

Hey hey,
i decided its a good idea to move gekko from my home server to a virtual server.
Virtual server has everything running so far, its running 16.04.02 Ubuntu lts.
However when i try to install the version of gekko that i have been running on my home server i runn into an sqlite issue with npm.
when i start to run gekko with "node gekko" it tells me i need to install sqlite3 database

The plugin sqlite adapter expects the module sqlite3 to be installed. However it is not, install it by running:

npm install<EMAIL_ADDRESS>
i sudo this command, however i recieve this error message then.

npm ERR! path /root/gekko-eth/node_modules/sqlite/node_modules/sqlite3/node_modules/node-pre-gyp/node_modules/npmlog/node_modules/are-we-there-yet/node_modules/delegates
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall rename
npm ERR! enoent ENOENT: no such file or directory, rename '/root/gekko-eth/node_modules/sqlite/node_modules/sqlite3/node_modules/node-pre-gyp/node_modules/npmlog/node_modules/are-we-there-yet/node_modules/delegates' -> '/root/gekko-eth/node_modules/sqlite/node_modules/sqlite3/node_modules/node-pre-gyp/node_modules/npmlog/node_modules/are-we-there-yet/node_modules/.delegates.DELETE'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2017-08-19T09_02_53_172Z-debug.log
so far i was not able to resolve this issue. maybe somebody has more knowledge about the process and whats actually going on there.
i also found 4 prommising answers from google research, but theese also did not solve the issue.

What version of node are you running?

Also, are you using Ubuntu's older Node install, or did you install a newer version with something like nvm?

im running node version  8.4.0

Can you try the LTS?
On 20 Aug 2017 08:51, "decoye"<EMAIL_ADDRESS>wrote:

im running node version 8.4.0
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1005#issuecomment-323569798, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MD6Zq6wzrnH7Oq8ssKvw8m_v3t4v3ks5sZ-VmgaJpZM4O8RvT
.

im affraid im not exactly sure what you mean by that...
i just noticed that everything fails that i try to reinstall by npm.

Here you can download the LTS: https://nodejs.org/en/download/
On 20 Aug 2017 13:37, "decoye"<EMAIL_ADDRESS>wrote:

im affraid im not exactly sure what you mean by that...
i just noticed that everything fails that i try to reinstall by npm.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1005#issuecomment-323582437, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MD4ENHd67t6fEbFGpNghtziYUE3zoks5saCifgaJpZM4O8RvT
.

i can install it but i cannot make the server use it. node --version tells me it still is 8.4.0

What if you remove all versions of node and than install the LTS?
On 20 Aug 2017 13:54, "decoye"<EMAIL_ADDRESS>wrote:

i can install it but i cannot make the server use it. node --version tells
me it still is 8.4.0
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1005#issuecomment-323583342, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MD_zxgQhkeGy4Cz0jJXgQxt-C4XEkks5saCyPgaJpZM4O8RvT
.

Will try this next
Keep you posted.

sent from mobile -
Mit freundlichen grüssen With kind regards

Andreas Schindler
<EMAIL_ADDRESS>-------- Ursprüngliche Nachricht --------Von: Mike van Rossum<EMAIL_ADDRESS>Datum: 20.08.17  14:57  (GMT+01:00) An: askmike/gekko<EMAIL_ADDRESS>Cc: decoye<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [askmike/gekko] Installing gekko in Virtual Server with ubuntu 16.04.02 npm installation issues (#1005)
What if you remove all versions of node and than install the LTS?
On 20 Aug 2017 13:54, "decoye"<EMAIL_ADDRESS>wrote:

i can install it but i cannot make the server use it. node --version tells

me it still is 8.4.0

—

You are receiving this because you commented.

Reply to this email directly, view it on GitHub

https://github.com/askmike/gekko/issues/1005#issuecomment-323583342, or mute

the thread

https://github.com/notifications/unsubscribe-auth/AA7MD_zxgQhkeGy4Cz0jJXgQxt-C4XEkks5saCyPgaJpZM4O8RvT

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/askmike/gekko","title":"askmike/gekko","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/askmike/gekko"}},"updates":{"snippets":[{"icon":"PERSON","message":"@askmike in #1005: What if you remove all versions of node and than install the LTS?\n\nOn 20 Aug 2017 13:54, "decoye"<EMAIL_ADDRESS>wrote:\n\n\u003e i can install it but i cannot make the server use it. node --version tells\n\u003e me it still is 8.4.0\n\u003e\n\u003e —\n\u003e You are receiving this because you commented.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/askmike/gekko/issues/1005#issuecomment-323583342\u003e, or mute\n\u003e the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/AA7MD_zxgQhkeGy4Cz0jJXgQxt-C4XEkks5saCyPgaJpZM4O8RvT\u003e\n\u003e .\n\u003e\n"}],"action":{"name":"View Issue","url":"https://github.com/askmike/gekko/issues/1005#issuecomment-323583500"}}}

I cannot reproduce on ubuntu 16.04 with the same node version as you are using:
root@test:~/gekko# npm cache clean --force
npm WARN using --force I sure hope you know what you are doing.
root@test:~/gekko#
root@test:~/gekko#
root@test:~/gekko#
root@test:~/gekko#
root@test:~/gekko#
root@test:~/gekko# lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:  Ubuntu 16.04 LTS
Release:  16.04
Codename: xenial
root@test:~/gekko# node -v
v8.4.0
root@test:~/gekko# npm install

><EMAIL_ADDRESS>install /root/gekko/node_modules/websocket
> (node-gyp rebuild 2> builderror.log) || (exit 0)

><EMAIL_ADDRESS>install /root/gekko/node_modules/sqlite3
> node-pre-gyp install --fallback-to-build

[sqlite3] Success: "/root/gekko/node_modules/sqlite3/lib/binding/node-v57-linux-x64/node_sqlite3.node" is installed via remote

><EMAIL_ADDRESS>postinstall /root/gekko/node_modules/relieve
> node links.js

added 576 packages in 27.465s

Ive now deleted all the nodejs files, and it is working now.
But it really took me a while to remove really everything there, even after the whole process node --version gave me an answer.
it was not smooth, and as you say, it was probably something with the installation of ubuntu. but still on a new machine from my provider i get this error.

Happy to hear your got it working. Installing a version of node on ubuntu is outside of the support I can give..

Yes yes absolutely. That has never been asked. I was jusz wondering why it workes on my server at home that has the exact same ubuntu version but on my real server it does not.

sent from mobile -
Mit freundlichen grüssen With kind regards

Andreas Schindler
<EMAIL_ADDRESS>-------- Ursprüngliche Nachricht --------Von: Mike van Rossum<EMAIL_ADDRESS>Datum: 21.08.17  10:19  (GMT+01:00) An: askmike/gekko<EMAIL_ADDRESS>Cc: decoye<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [askmike/gekko] Installing gekko in Virtual Server with ubuntu 16.04.02 npm installation issues (#1005)
Happy to hear your got it working. Installing a version of node on ubuntu is outside of the support I can give..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/askmike/gekko","title":"askmike/gekko","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/askmike/gekko"}},"updates":{"snippets":[{"icon":"PERSON","message":"@askmike in #1005: Happy to hear your got it working. Installing a version of node on ubuntu is outside of the support I can give.."}],"action":{"name":"View Issue","url":"https://github.com/askmike/gekko/issues/1005#issuecomment-323678046"}}}


Outlining expansion status gets lost

This issue has been moved from a ticket on Developer Community.

[severity:It bothers me. A fix would be nice]
Since the last few updates (my current version is 16.11.7) Visual Stduio keeps losing the outlining expansion state. I always liked to collapse things in code view and I especially liked that VS remembers the state of those. But since a few versions it happens more and more often, that I open a code file (C#) and everything is expanded.
This happens in a Net Core Web App project when publishing the project from Visual Studio to an Azure App Service and only to files which are open in Visual Studio at the time of publishing. It does not happen to all open files and it does not happen to always the same files. I cannot tell which version exactly introduced this “feature”

Original Comments
(no comments)

Original Solutions
(no solutions)

Hello, would you mind trying your scenario out in Visual Studio 2022? We believe we've solved this problem there.


RIP G+ icon

A suggestion to replace the G+ icon at the top of this page with Google icon instead because Google plus is officially dead as from April 1st, 2019.
The image I'm talking about: https://docs.microsoft.com/en-us/aspnet/core/security/authentication/social/index/_static/social.png?view=aspnetcore-2.2

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

ID: d1612f19-f267-493a-c230-2ca18e4eb50b
Version Independent ID: d312db0f-6cf1-87ca-09b0-604a66e18a07
Content: Facebook, Google, and external provider authentication in ASP.NET Core
Content Source: aspnetcore/security/authentication/social/index.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Fixed with #15245 .   I removed the outdated and misleading G+ icon rather than replace it since the graphic was not essential.


Weather App

Develop a Javascript Weather  app.
Description 📜
We will get the current weather data from Fess API fey from OpenWeatherMap and display the weather information like Temperature,  Weather condition, Humidity and Wind Speed according to the city on our website or app.
We will display the weather info on website from Free API using JavaScript.
In this weather app we will add one search box where user can enter the city name and get the weather information of the particular city.
Requirements 🛠

HTML/CSS: To structure and style your app.
*JavaScript: For handling user interactions, fetching data from the API, and updating the DOM.
*APIs: Understanding how to make HTTP requests to APIs and handle JSON responses.

Any number of people can work on a single issue 👨💻👨💻
This issue is open to all.🌍

Project location 👉🏻 {AppName}/Username/
👨🏻💻 Happy Coding 👩🏻💻

/attempt #1148

I may give it a try. I haven't contributed to anything yet, and I know some HTML/CSS, javascript and some React, but I am a bit scared to try...

Hi! can I work on this project

@ninja-hatori-dev  Yes

Hi! @mayurr7, Can I work on this project as a contributor please!

I would like to work on this project

Hey would like to help if you could assign me.

/attempt #1148


Anyone can create example for both android and ios?

PLEASE HELP.
on both android and ios need to be:

app open (already made it work)
app minimal (already made it work)
app killed/terminate by user swipe up or clear
phone lock screen

+2


Cannot deploy second contract through CLI

Describe the bug
When the directory has one smart contract, I can create a config with zk config and deploy through zk deploy.
Then if you add new contract and export it in index.ts -just like the first one-, CLI cannot deploy it.
Even if I create new config named for example berkeley2, select the smart contract 2, zk deploy berkeley2 cannot find my contract.
Also I tried zk deploy, select network, then it shows my contract to select. I select it but it still gives the same error, interestingly.
Steps to Reproduce
Copy and paste Add.ts -> Add2.ts. Change the contract name in the file. Export it in index.ts. Create new deploy config to deploy Add2.  Run zk deploy deploy2. It will not work.
System Info
System:
    OS: macOS 12.6
    CPU: (8) arm64 Apple M1
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
  npmPackages:
    snarkyjs: Not Found (not in a project)
  npmGlobalPackages:
    zkapp-cli: 0.5.3

Severity
annoyance
Additional Information
No response

Found the error, it was an import error two contracts, not related to CLI. However, we need to have better error messages in try catch blocks I think. Closed!


Add White-listed domain

Purpose
The administrator should be able to add white-listed domains to be validated during the login.
Related ticket: TolaDataV2?#657

Coverage increased (+0.2%) to 55.843% when pulling 641ef4653ec9fd2d40994dc75bc951bd032224e3 on jefmoura:add-whitelist-domain into 75dd35eb3ff00464ad81501dd8754cc6a5b283f7 on toladata:dev-v2.

Coverage increased (+0.2%) to 55.848% when pulling 6ef9384397505dd0a9256d4cde8e156d5d040817 on jefmoura:add-whitelist-domain into eb7d2d64891759db663beb26fb6372b9509b2a41 on toladata:dev-v2.


Support .pairs in the user's HOME directory, as advertised.

The README file states that a .pairs file may be "in project root or your home folder." This isn't true unless the working directory is under the user's home directory.
This PR contains two commits: one to slightly clean up the directory-search logic; and then a second to implement the addition of the user's home directory to the end of the search path.

In 1.4.0.


Generate changelog and add PR template

Part of purescript/purescript#3986

This PR's changelog didn't get generated. I received a 403 Forbidden HTTP error.

This PR's changelog didn't get generated. I received a 403 Forbidden HTTP error.

I've added the changelog to this PR.

I've added the changelog to this PR.


Implements Client-Side RowKey translation on import

Currently works for RowKeyColumnID type data
It works on roaring import.
It works on CSV import. Temporarily disabled that though, since it requires ignoreKeyCheck flag; unconditionally enabling it breaks some tests, so need to look into that.
Although not enabled currently, it should be easy to implement it for RowIDColumnKey and RowKeyColumnKey type data.
Value fields are not supported yet, but should be trivial to implement.

Coverage decreased (-1.1%) to 98.621% when pulling 7e16d5dbe66dc20e385093ee93a75d3c70a1c0d6 on yuce:row-keys-to-id-mapper into 4f922fff245e2e04b7336e0e9f61e879cd7907c7 on pilosa:master.

Coverage decreased (-1.1%) to 98.621% when pulling 7e16d5dbe66dc20e385093ee93a75d3c70a1c0d6 on yuce:row-keys-to-id-mapper into 4f922fff245e2e04b7336e0e9f61e879cd7907c7 on pilosa:master.


FIX resource 404's not returning styled error page

When viewing a resources URL which 404's (e.g https://www.silverstripe.com/_resources/themes/app/dist/images/404.jpg) the Apache default error page is shown rather than the SilverStripe provided 404 template.

@michalkleiner updated now

@emteknetnz I'm happy with this, we use it in PROD on CWP and it works ok. Do you see any issues with this change?


EDUPUB - complete nav required

The spec requires that all headings be represented in the toc nav.

All heading elements in all content documents?

To quote "a toc nav [ContentDocs301] with links to the full document heading hierarchy MUST be included in every EPUB Publication conformant to this profile."
http://www.idpf.org/epub/profiles/edu/spec/#h.lmfoatmvs6h7

I implemented this check in 2d34738bcf91d0c124d6be3416214468433a4baf.
@mattgarrish however, there's still an ambiguity:

section 6.1 Table of Contents says a toc nav MUST include links to the full document heading hierarchy.
section 4.2 Sectioning says sections SHOULD have a heading, i.e it's not required.

The spec may need to clarify that the toc nav MUST include links to the full section hierarchy instead?
What EpubCheck now does is count the section or body elements and check that the result is equal to the count of nav doc links. Hence some limitations:

toc nav links are not resolved. It w/b pretty difficult to check that a link resolves to a section, as the element being pointed to can vary (section,  first heading, first element, second nested div, whatever really), not even thinking about CFI.
heading-less sections are taken into account, thus varying a bit from the strict interpretation of 6.1.

I'm closing this issue for now, but feel free to reopen if the solution is not satisfactory.

On the plus side, that prose is out-of-date now, as I also opened a bunch of issues in the last edupub iteration to fix some of those inconsistencies. The coming edupub draft attempts to better clarify where headings are needed (what we mean by structural hierarchy), including noting that non-linear spine items are not subject to the parsing. Another late addition was that article can be used in addition to section, so we kept the sutructural hierarchy name.
Anyway, I also agree that this requirement is onerous and problematic, both for producers and validation, so only commenting to note for the record that the issue is open for the next edupub draft:
https://github.com/IDPF/epub-revision/issues/564

I'm re-opening the issue to make sure I do not forget to update it when IDPF/epub-revision#564 is solved.

What's the status on this one? @mattgarrish  Can I remove from the 4.0 milestone until the EDUPUB spec issue is solved?

It's still open as the edupub work had disbanded for the summer before we filed the bug.
I'm in favour of removing any checking from 4.0. Without clarification about how the check has to work given the potential pitfalls, I don't see the value in trying to enforce yet.

I've lowered the check to USAGE while waiting for the spec clarification.

@mattgarrish can you clarify the state of this issue? do we need to raise the severity level again now that the spec matured?

We ditched that requirement at some point. It's just a general recommendation to include links to all sections now. Probably not even worth a usage note.

Thanks Matt! Leaving the USAGE for now so that no one needs to take action here. Closing the issue.


[WFLY-4625] Reuse authenticated subject from incoming context when se…

…curity domains match

Linux Build 6471 is now running using a merge of 57fdcd80201a95b621994083c15c314f16643778

Linux with security manager Build 1377 is now running using a merge of 57fdcd80201a95b621994083c15c314f16643778

Windows Build 1549 is now running using a merge of 57fdcd80201a95b621994083c15c314f16643778

Linux with security manager Build 1377 outcome was SUCCESS using a merge of 57fdcd80201a95b621994083c15c314f16643778
Summary: Tests passed: 803, ignored: 320 Build time: 0:06:11

Linux Build 6471 outcome was SUCCESS using a merge of 57fdcd80201a95b621994083c15c314f16643778
Summary: Tests passed: 2845, ignored: 375 Build time: 0:46:48

Windows Build 1549 outcome was SUCCESS using a merge of 57fdcd80201a95b621994083c15c314f16643778
Summary: Tests passed: 2845, ignored: 375 Build time: 0:49:47