Client certificate

I am using axios on node.js 4.2.x. how do i mention SSL CA certificate for connecting to one of the secured services ?
Can anybody point me to an example working code ?

You should be able to use a custom Agent as follows:

var agent = new https.Agent({
  ca: fs.readFileSync('ca.pem')
});

axios.get(url, { agent: agent });

// or

var instance = axios.create({ agent: agent });
instance.get(url);

Hope it helps!

In my case, its a .crt file which has bunch of certificates in it from Godaddy.
when  i try to do follow the example code @nickuraltsev  mentioned, i get following error.
{ [Error: unable to get local issuer certificate] code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY' }

axios uses https module under the hood. All certificate related config parameters are passed to https. You can even configure https directly:
https.globalAgent.options.ca = fs.readFileSync('ca.pem'); // Note: this is a global setting

If you need help with using an SSL certificate with https module, please ask a question on StackOverflow.

I am trying to run this project in browser. I cannot find https module available. Can you tell me how this was made possible?

@voraparth91 You can configure axios to use specific certificates in Node.js, but not in browsers.

@nickuraltsev Is there any other way you can suggest me where I can configure specific certificates in browser for XHR requests?

Careful, the correct way to assign agent is by using the field httpsAgent.
'''
const instance = axios.create({ httpsAgent: agent });
'''

Try below code  worked ,thanks team.
const Agent = new https.Agent({
  rejectUnauthorized: false
})
const instance = axios.create()
instance.defaults.baseURL = `http://${process.env.HOST || 'localhost'}:${process.env.PORT || 3000}`
instance.defaults.httpsAgent = Agent
instance.defaults.headers.common['Authorization'] = 'Basic YWx0ZXJodTIwMjBAZ21haWwuY29tOmd1Y2hhbjEwMjY='
instance.defaults.timeout = 14500

Not advisable to setrejectUnauthorized: false. It puts https connections under threat.

In my case, its a .crt file which has bunch of certificates in it from Godaddy.
when i try to do follow the example code @nickuraltsev mentioned, i get following error.
{ [Error: unable to get local issuer certificate] code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY' }

In my case, its a .crt file which has bunch of certificates in it from Godaddy.
when i try to do follow the example code @nickuraltsev mentioned, i get following error.
{ [Error: unable to get local issuer certificate] code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY' }

I also encountered this problem, can you tell me how you solved it?


Junk characters while consuming messages using Kafka node.

Bug Report
I have a hosted cloud based Kafka server. The messages can be received perfectly using Kafka Tool.
However, when consuming the same messages using kafka-node, I am getting the following message.value -
""M`@������}�����!p^��L�$492e9-8be8-6612-bf83-c1cbf3187a1f��t,"strpoi��4353b846-727d-4d06-9bb3-932e1cfc66b2T��sNewCustom��false,"c�����"",����P72960��,"latitude":30.35516279286738,"long��-91.092605888928��}	��;�-35b1-gt541-5b0c-t531-001517f43a86","n�qOfTotesuq� �P�L��40.0������0zonF�US/Central","dtd":0}}����"
Environment

Node version: 6.14.2
Kafka-node version: 3.0.1
Kafka version: 0.11

Code
// consumer
const { load } = require('dotenv')
load()
const { KafkaClient, Consumer } = require('kafka-node')
const kafkaHost = ${process.env.KAFKA_HOST}:${process.env.KAFKA_PORT}
const topic = process.env.KAFKA_TOPIC
const offset = process.env.KAFKA_OFFSET || 0
const client = new KafkaClient({ kafkaHost })
const consumer = new Consumer(
client,
[{ topic, offset }],
{ autoCommit: false, fromOffset: true }
)
consumer.on('message', (message) => {
console.log(message.offset, message.value)
})

Need your expert advise on this, as I am unable to find a solution. Appreciate any help.

What is you try the newer kafka-node version? How is the message serialized? Depending on . the kafka tool your using it might be avro fromatted.

Thanks Gerard. Appreciate your thoughts on it.
We are using the kafka-node version - 4.1.3 & utf-8 encoding, but still getting the junk characters.

So it should be just a string, or a json as string? And witch Kafka tool?

It should be a JSON string. Am consuming messages inside the node app which uses the kafka-node npm module.

Might be the same problem I had, which is on the brokers LZ4 compression is 'enforced'. Kafka-node can't handle this, but also don't give any exception. We were lucky to use protobuf so did get an error. In our case the key was also empty. We solved it by moving to KafkaJs, that have a KafkaJs-LZ4 plugin that works. You could also try node-rdkafka. Our main reason for choosing KafkaJs is that it supports SSL the same way as this library, while node-rdkafka needs files.

Hi Gerard,
Sorry for the delay in replying. You were right. The LZ4 compression is enforced resulting in the junk characters. We are also planning to move to KafkaJs.
Highly Appreciate your advice, since it pointed to the exact cause.
Can't thank you enough.

No problem @vikcygit was very desperate needing to solve the bug with little Node experience. Once I got some help, and fixing the logging on the test environment, we got some further. Then during lunch it 'clicked' and ten minutes later I could locally reproduce the issue. Can you close the issue and/or mark it as duplicate? Just did a search and there seem 6 related issues https://github.com/SOHU-Co/kafka-node/issues?utf8=✓&q=lz4 all still open. Maybe @hyperlink can make one issue and close the others?

Was there any resolution to this problem?

Other than moving to KafkaJs? You could search for lz4 for Kafka-Node.


Update fingerprints.yml

changed for zemismart curtain driver

Those changes won't work

Those changes won't work


replacing rounded faraday constant with exact value in bpx.py

Description
The faraday constant has an exact value, calculated by the exact values of the avogadro constant and the elementray charge [see here]. Using a rounded value may introduce systemic errors further down the line that can be easily avoided with this new value.
Fixes # (issue)
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[x] No style issues: $ pre-commit run (or $ nox -s pre-commit) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[x] All tests pass: $ python run-tests.py --all (or $ nox -s tests)
[x] The documentation builds: $ python run-tests.py --doctest (or $ nox -s doctests)

You can run integration tests, unit tests, and doctests together at once, using $ python run-tests.py --quick (or $ nox -s quick).
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[not needed] Tests added that prove fix is effective or that feature works

FYI you can do pybamm.constants.F or pybamm.constants.F.value if you need a number instead of a symbol.

FYI you can do pybamm.constants.F or pybamm.constants.F.value if you need a number instead of a symbol.

I agree, using a constant from a library is better than updating this file.
One other thing to check would be if the BPX standard specifies a value for Faraday constant. Sometimes standards specify the precision to ensure that things are constant between implementations

Re-requested review/approval from @kratman since he previously requested changes


Sequence.map and where should use new this.type instead of []

See https://github.com/munificent/wren/commit/f8a9d7f3213205283fd84e3124a299e93ebef38b#commitcomment-5391488

Old bug is old. :) .map() and .where() return custom lazy sequence types now.


STAC REST API using docker-compose up

Hello,
Describe
Failed to up the stac server app using docker compose with respect to the tutorial
git clone https://github.com/CS-SI/eodag.git
cd eodag
docker-compose up

Additionally, I have tried including the build step, but I have the same result.
git clone https://github.com/CS-SI/eodag.git
cd eodag
docker-compose build
docker-compose up

To Reproduce
CLI commands executed by windows 10 to reproduce the bug:
Follow steps by default tutorial exposed in readme.md
Output
At the commande line tool displayed:
stac_server | standard_init_linux.go:228: exec user process caused: no such file or directory
stac_server exited with code 1

Environment:

Python version: 3.8.5
EODAG version: 2.3.1

Thanks for all!

Hello @pedringcoding and thanks for reporting this issue.
Can you please update the code and checkout the docker_windows branch, to see if it works better:
cd eodag
git pull
git checkout docker_windows

Run docker-compose using recently added verbose mode for stac-server:
docker-compose build
EODAG_LOGGING=3 docker-compose up

And post the output here.
Thanks

Hello @sbrunato, thank your team for all this work!
When I following indicated steps, get the next logs by command line:
For docker-compose build
WARNING: The EODAG_LOGGING variable is not set. Defaulting to a blank string.
Building browser
[+] Building 6.0s (15/15) FINISHED
=> [internal] load build definition from stac-browser.dockerfile                                                                                0.1s
=> => transferring dockerfile: 653B                                                                                0.0s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
=> [internal] load metadata for docker.io/library/node:lts-buster-slim                                                                                5.5s
=> [internal] load metadata for docker.io/library/nginx:stable-alpine                                                                                5.7s
=> [auth] library/node:pull token for registry-1.docker.io                                                                                0.0s
=> [auth] library/nginx:pull token for registry-1.docker.io                                                                                0.0s
=> [build-stage 1/6] FROM docker.io/library/node:lts-buster-slim@sha256:12e2d4e68982ea48a4e5a865987f8aacbb1a5767a346f1279f676ef0f72c8cd2                  0.0s
=> [production-stage 1/2] FROM docker.io/library/nginx:stable-alpine@sha256:7b801536219e8b3a5dcaf0c5a34c4d4d0514e648905c83105d98c56740eb4f07              0.0s
=> CACHED [build-stage 2/6] RUN apt-get update     && DEBIAN_FRONTEND=noninteractive apt-get install -y git                                               0.0s
=> CACHED [build-stage 3/6] RUN git clone https://github.com/radiantearth/stac-browser.git                                                                0.0s
[+] Building 64.9s (19/19) FINISHED
=> [internal] load build definition from stac-server.dockerfile                                                                                0.1s
=> => transferring dockerfile: 1.99kB                                                                                0.0s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
=> [internal] load metadata for docker.io/library/python:3.9-slim                                                                                3.2s
=> [auth] library/python:pull token for registry-1.docker.io                                                                                0.0s
=> [ 1/14] FROM docker.io/library/python:3.9-slim@sha256:2c018e29a8eada75e855d78641adda978a2c0a035fd928e281e1240144e8a337                                 0.0s
=> [internal] load build context                                                                                0.1s
=> => transferring context: 26.43kB                                                                                0.0s
=> CACHED [ 2/14] RUN apt-get update     && apt-get upgrade -y                                                                                0.0s
=> CACHED [ 3/14] RUN echo UTC > /etc/timezone &&     apt-get install -y tzdata &&     rm /etc/localtime &&     ln -snf /usr/share/zoneinfo/UTC /etc/loc  0.0s
=> CACHED [ 4/14] RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y locales                                                         0.0s
=> CACHED [ 5/14] RUN sed -i -e 's/# en_US.UTF-8 UTF-8/en_US.UTF-8 UTF-8/' /etc/locale.gen &&     dpkg-reconfigure --frontend=noninteractive locales &&   0.0s
=> [ 6/14] COPY setup.py setup.py                                                                                0.0s
=> [ 7/14] COPY setup.cfg setup.cfg                                                                                0.0s
=> [ 8/14] COPY README.rst README.rst                                                                                0.1s
=> [ 9/14] COPY ./eodag /eodag                                                                                0.1s
=> [10/14] RUN python setup.py install                                                                                58.4s
=> [11/14] COPY ./docker/run-stac-server.sh /eodag/run-stac-server.sh                                                                                0.1s
=> [12/14] RUN chmod +x /eodag/run-stac-server.sh                                                                                0.4s
=> [13/14] RUN addgroup --system user     && adduser --system --group user                                                                                0.6s
=> exporting to image                                                                                1.6s
=> => exporting layers                                                                                1.5s
=> => writing image sha256:6d5887bcad054241e92b23f03837013c7b4ba9ce4fe488f9e48fbdb65d832e0a                                                               0.0s
=> => naming to docker.io/library/eodag_stac                                                                                0.0s
I needed include .env file to add argument EODAG_LOGGING=3
For docker-compose config
networks:
eodag-stac-network:
driver: bridge
services:
browser:
build:
context: D:\pfernandezc\Projects\OnEO\TPI\workspace\pet-project\eodag
dockerfile: docker/stac-browser.dockerfile
container_name: stac_browser
networks:
eodag-stac-network: null
ports:
- published: 5001
target: 80
restart: unless-stopped
stac:
build:
context: D:\pfernandezc\Projects\OnEO\TPI\workspace\pet-project\eodag
dockerfile: docker/stac-server.dockerfile
container_name: stac_server
environment:
EODAG_LOGGING: '3'
networks:
eodag-stac-network: null
ports:
- published: 5000
target: 5000
restart: unless-stopped
version: '3.7'
For docker-compose up
Creating stac_browser ... done
Creating stac_server  ... done
Attaching to stac_server, stac_browser
stac_browser | /docker-entrypoint.sh: /docker-entrypoint.d/ is not empty, will attempt to perform configuration
stac_browser | /docker-entrypoint.sh: Looking for shell scripts in /docker-entrypoint.d/
stac_browser | /docker-entrypoint.sh: Launching /docker-entrypoint.d/10-listen-on-ipv6-by-default.sh
stac_server | /eodag/run-stac-server.sh: line 2: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 4: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 15: syntax error: unexpected end of file
stac_browser | 10-listen-on-ipv6-by-default.sh: info: Getting the checksum of /etc/nginx/conf.d/default.conf
stac_browser | 10-listen-on-ipv6-by-default.sh: info: Enabled listen on IPv6 in /etc/nginx/conf.d/default.conf
stac_browser | /docker-entrypoint.sh: Launching /docker-entrypoint.d/20-envsubst-on-templates.sh
stac_browser | /docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
stac_browser | /docker-entrypoint.sh: Configuration complete; ready for start up
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: using the "epoll" event method
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: nginx/1.20.1
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: built by gcc 10.2.1 20201203 (Alpine 10.2.1_pre1)
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: OS: Linux 5.10.25-linuxkit
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: getrlimit(RLIMIT_NOFILE): 1048576:1048576
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: start worker processes
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: start worker process 32
stac_browser | 2021/07/21 12:13:41 [notice] 1#1: start worker process 33
stac_server exited with code 0
stac_server | /eodag/run-stac-server.sh: line 2: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 4: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 15: syntax error: unexpected end of file
stac_server | /eodag/run-stac-server.sh: line 2: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 4: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 15: syntax error: unexpected end of file
stac_server | /eodag/run-stac-server.sh: line 2: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 4: $'\r': command not found
stac_server | /eodag/run-stac-server.sh: line 15: syntax error: unexpected end of file
Exception in thread Thread-7:
Traceback (most recent call last):
File "docker\api\client.py", line 268, in _raise_for_status
File "requests\models.py", line 941, in raise_for_status
requests.exceptions.HTTPError: 409 Client Error: Conflict for url: http+docker://localnpipe/v1.41/containers/cbb8fa39835d98dcda7252338e98d1dbd640636209f347e88abccafc776bfe89/attach?logs=0&stdout=1&stderr=1&stream=1
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "threading.py", line 950, in _bootstrap_inner
File "threading.py", line 888, in run
File "compose\cli\log_printer.py", line 224, in watch_events
File "compose\container.py", line 202, in attach_log_stream
File "compose\container.py", line 294, in attach
File "docker\utils\decorators.py", line 19, in wrapped
File "docker\api\container.py", line 62, in attach
File "docker\api\client.py", line 405, in _read_from_socket
File "docker\api\client.py", line 318, in _get_raw_response_socket
File "docker\api\client.py", line 270, in _raise_for_status
File "docker\errors.py", line 19, in create_api_error_from_http_exception
File "requests\models.py", line 881, in json
File "requests\models.py", line 829, in content
File "requests\models.py", line 751, in generate
File "urllib3\response.py", line 575, in stream
File "urllib3\response.py", line 518, in read
File "http\client.py", line 458, in read
File "http\client.py", line 502, in readinto
File "docker\transport\npipesocket.py", line 211, in readinto
File "docker\transport\npipesocket.py", line 20, in wrapped
RuntimeError: Can not reuse socket after connection was closed.
stac_server exited with code 2
If you need more information or execute other steps, pleace, tell me.
One more time, thanks!

Can you try again after having updated docker_windows branch ?

Also, I see in your output

stac_server | /eodag/run-stac-server.sh: line 2: $'\r': command not found

which is strange as there should not be any \r character in this file. Could you try to open docker/run-stac-server.sh and replace CRLF EOL characters with LF. This can be done using Notepad++ (From the "Edit" menu, select "EOL Conversion" -> "UNIX/OSX Format"), or by running:
sed -i 's/\r//g' run-stac-server.sh

Thanks

Another update, I added a .gitattributes file containing * -text to prevent git from normalizing EOL when running on windows

Great!
I using the last changes included in docker_window feature, and its correctly running.
I can check the API exposed from stac_server.
Thated really nice of you @sbrunato
Thanks you very much!


Configurable ingress.class semantics for KongConsumer resource

Problem
KongConsumers behave similar to Ingress v1beta1 currently, meaning, if you don’t configure ingress class in the controller, then you don’t need the ingress.class annotation on KongConsumer resource. If you configure ingress class as kong, the same thing applies, but if you configure the class as anything different, then you need to configure all KongConsumer with the same ingress class.
This is a problem and a feature:

Problem because majority of users use an ingress class to differentiate kong from other ingress controllers, in this case, applying the annotation on KongConsumer resources is a pain, especially because of higher cardinality.
When multiple deployments of the controller are deployed and the user needs control over KongConsumers mapped to specific deployment. This is where ingress class annotation is useful

Solution

By default ingress class annotation makes no difference on KongConsumers
This can be changed using a boolean configuration parameter ENABLE_INGRESS_CLASS_FILTER_KONG_CONSUMER (by default false). If this is enabled, the class on KongConsumer resources must match the class defined in the controller. This should ease the easy path i.e. cases where a user doesn't install multiple kong ingress controllers in the same cluster.

This is a breaking change but:

fewer users would be affected by this change
the change itself is configurable, hence the impact (in theory) should not be huge

For more background, please see the doc in #690.

Handled in #767.


Update dependency on spring-cloud-context to at least 2.1.1.RELEASE

We are using spring-cloud-config-server:2.1.2.RELEASE. It appears to pull in spring-cloud-context:2.1.0.RELEASE in its release builds. This version of that jar has an older definition for a KeyProperties class used by org.springframework.cloud.config.server.config.EncryptionAutoConfiguration. When our config server boots, it crashes with a NoSuchMethodException (specifically, "key.getKeyStore().getType()").
We get around it with an explicit dependency on spring-cloud-context:2.1.1.RELEASE, which contains a class with the right definitions. Could someone update the deps?
Thank you!

Are you using the spring cloud bom?
Here is what it is using. Maybe you have something else bringing in an old version?
https://github.com/spring-cloud/spring-cloud-config/blob/v2.1.2.RELEASE/pom.xml#L30

We are also facing similar issue. Below are the declared dependencies
dependencyManagement {
imports {
mavenBom "org.springframework.cloud:spring-cloud-dependencies:Greenwich.RELEASE"
}
}
dependencies {
compile "org.springframework.cloud:spring-cloud-config-server:2.1.2.RELEASE"
compile 'org.springframework.boot:spring-boot-starter-security'
compile("org.springframework.security.oauth.boot:spring-security-oauth2-autoconfigure:$springBootVersion")
Please note : default version on spring cloud config comes 2.1.0 with Spring boot 2.1.2.RELEASE however it has to upgrade because of https://pivotal.io/security/cve-2019-3799 this issue.

Does using Greenwich.SR1 make a difference? It is the latest.

Working for me.

Our only relevant dependency is this, in Gradle:
compile "org.springframework.cloud:spring-cloud-config-server:2.1.2.RELEASE"
It looks like spring-cloud-config-server brings in spring-cloud-config-client, which brings in spring-cloud-context:2.1.0. There's no explicit dependency in Gradle's dependencies graph from the config-server artifact to spring-cloud-context, though such a dependency exists in the code.
+--- org.springframework.cloud:spring-cloud-config-server:2.1.2.RELEASE
|    +--- org.springframework.cloud:spring-cloud-config-client:2.1.2.RELEASE -> 2.1.0.RELEASE
... (elided)
|    |    +--- org.springframework.cloud:spring-cloud-context:2.1.0.RELEASE

@jeblackburn @Datta4400 there is not need to specify the version in your gradle or pom files the spring cloud bom manages that.

For the CVE you need to, but there's no fix except to manually adjust versions until the next release

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Momentjs does not parse date with Chinese format

This is regarding an issue we are facing with Chinese date format.
I used a format - yyyy'年'mm'月'dd'日'
Date - any date
It gives me - 2015'年'12'月'24'日'
but it should give me 2015年12月24日 - notice there are no single quotes in the date. I formatted using this format with java or other jqx formatting - it gives me correct result.
Can you please see, what is to be done. Does moment work well with Chinese timezone.

I don't understand. You put '' in format and it ends up in the string. ' are preserved. Just remove them from your format string if you don't like them.

I completely agree with you. But this seems to be a format specific to Chineese or simplified chineese. As mentioned in the orignal issue, if we give same format using jqx formatting or java formatting, it simply works.
This is strange to me also. If you try to format using java(I used http://www.fileformat.info/tip/java/simpledateformat.htm for testing), here is the output it gives:

Please share your thoughts.

Moment's formatter specifiers are not 100% aligned with those from other languages, be it Java, .NET, PHP, C, Ruby, or others.  Please read the documentation to carefully choose the correct format specifiers.
Specifically, moment's escape sequence characters are square brackets [ and ], not single quote marks.

Also note that they are case sensitive.  We use upper-case YYYY, MM and DD for the year, month, and day components.


InPlace = true in a sequence

Are there any examples on how to replicate inplace=True in pytorch?
I do see functions with a underscore suffix such as relu_(), but I can't figure out how to use it in a sequence.
Thanks!

Not sure what you mean by using this in a sequence, you should be able to do something like xs.relu_().sigmoid_() and this would indeed modify the xs tensor in place but maybe you have something else in mind?

My apologies, let me clarify.
Yes, if xs is defined as mut, there is no problem using the in-place version of the functions. However, in a Sequence, the add_fn passes a closure that only has an immutable reference to xs, so we can only use the non-in-place version like add_fn(|xs| xs.relu()). Is there something we can do to use the in-place version in a Sequence?

Ah indeed, I agree that it's far from ideal, it's a bit baked in the Module trait and would be messy to duplicate or change especially as there is already some duplication because of ModuleT (the variant which a "training" flag).
I'm not entirely sure what would be the best here so it would be good to think about this a bit more in depth.
Meanwhile, you can probalby use some "interior mutability" like trick at the moment via shallow_clone, I've added a small test to illustrate this here.

that works very well for now. thanks!


[EN] Feedback for: /tools/iterate/test-layout/test-layouts

404 error not found!!

Where'd you find the link to this page?


feat(fromRenderProps): add support for multiple arguments (Apollo usecase) #693

Add support for multiple arguments in fromRenderProps HOC.
for issue #693

Thank you! Good catch!


Mute the k3d registry create help

Description
Changes proposed in this pull request:

add --no-help flag

/retest

/joke

/lgtm


Don't allow some commands to be run in certain channels

Some commands like []timeleft should only be allowed in the bot channel and for ones like timeleft the relevant channel. In that case being jam channels. How to go about informing that to the user is up to you, but at minimum this should stop people repeatedly using commands in channels like general in the middle of talks. It's a small change but at things like events or bot changes it'll help.

@CodeMyst Is this still required? If so, how should it be implemented on the user side?

Honestly don't see much of a need in it. But it could still be implemented and be useful in other ways. Maybe you could make a system where every channel has a list of ignored modules. So we can set the auto paste thingy to ignore the collab channels.

The auto paste thingy was fixed in #205 by adding a blacklist for that service.
The codeblock pasting service is a service, not a module, and is theoretically not tied to commands whatsoever.
So you would suggest white-/blacklisting entire modules instead of specific commands?

Nah that's fine. Just used the wrong terminology.
This can be closed then I guess, I don't see a reason to not allow some commands to be run in #general for example, since the way we tell people they can't do that and confusion will result in the same amount of clutter anyway, imo.

My thoughts exactly.
Well, this is closed then, we can re-open if it comes up again in the future!


undefined is not a function (evaluating 'this.element.querySelectorAll(selector)')

Hi, I encounter such error as the title says and my test fails.
avoriaz version: 4.2.0
My test is very simple:
describe('Welcome Page', () => {
  it('Should render welcome text', () => {
    const wrapper = mount(Welcome)

    expect(wrapper.contains('.welcome__header')).to.equal(true)
  })
})

And the component is simple as well:
<template>

        {{ $t('welcome.header') }}

        {{ $t('welcome.text') }}

      <el-button
        type="primary"
        class="login__button welcome__btn"
        native-type="button"
        @click="handleSample">
        {{ $t('welcome.loadSample') }}
      </el-button>
      <el-button
        type="primary"
        class="login__button welcome__btn"
        native-type="button"
        @click="$router.push({ name: 'Login' })">
        {{ $t('welcome.auth') }}
      </el-button>

</template>

What am I doing wrong?
P.S. Also how can I use $router, $t & $store inside of my component? (As I have warns & errors because of that) Thanks.

Are you using JSDOM to add a document in your tests?
To add $router, $t and $store, you can use options.globals:
const wrapper = mount(Component, {
  globals: {
    $router: {}, $t: {}, $store: {}
  }
})

@eddyerburgh
Thanks for your fast reply! No, I am not using JSDOM.

Ok, that's probably the problem. You need to run the tests in a browser, or you need to run JSDOM to create a virtual document. You can do that with a setup file that runs browser-env (which runs JSDOM under the hood) before your tests, like this - https://github.com/eddyerburgh/avoriaz-tape-example/blob/master/test/helpers/.setup.js#L6

Is it the same stuff for Mocha/Chai?

Now I get something like this - TypeError: undefined is not an object (evaluating 'store.registerModule')

Sounds like you're installing Vuex before your tests (using Vue.use(Vuex)).
You'll need to debug your code and find where the error is. You can add globals using the globals option. Anything that's undefined but used in the component will need to be added with this option:
const wrapper = mount(Component, {
  globals: {
    $router: {}, $t: {}, $store: {}
  }
})

I am adding router instance as $router property as well as others, however I still get the same output that these are undefined :(

Ah - you might be getting stung by this - https://github.com/eddyerburgh/avoriaz/issues/80
Make sure you don't accidentally Vue.use(Router) before you run your tests.

Still no luck :(
Unit testing seems overcomplicated to configurate

Yeah it can be complicated
The main things to note:

You need to run the tests in a Document environment. (use JSDOM to run in node)
You need to stub properties on the vue instance in components if they expect them. Using shallow instead of mount means you only stub dependencies in one component.
You shouldn't run any code that installs plugins on Vue in your unit tests. It's easy to do this without realizing. When you require('./some-file'), node executes the code in the file. This might lead to unexpected installs.

Try using one of the example projects and building on top of that.


Generating POJOs from JSON source type using maven plugin

Hi,
Is there a way to generate POJOs from a JSON example file instead of JSON schema file using maven plugin? Looking at your maven plugin source code, sourceType seems to be hardcoded to jsonschema and there is no setter method for this private field.
https://github.com/joelittlejohn/jsonschema2pojo/blob/master/jsonschema2pojo-maven-plugin/src/main/java/org/jsonschema2pojo/maven/Jsonschema2PojoMojo.java
Many Thanks,
Iris

Hi, yes this is possible. This is a private field with no setter only because Maven mojo config args work in this way.
These docs will be useful:
http://joelittlejohn.github.io/jsonschema2pojo/site/0.4.8/generate-mojo.html


Virtual Reality SDK Windows Mixed Reality failed to initialize.

I'm trying to build a basic application for hololens. I followed this tutorial by Microsoft: https://docs.microsoft.com/en-us/windows/mixed-reality/holograms-100
In chapter 5, after playing the scene I saw a warning in the console (Virtual Reality SDK Windows Mixed Reality failed to initialize. Will attempt to enable None instead). I saw this instead of a cube in Hololens. What could be the reason for getting this error? Any help would be appreciated.

Some problem here

just solved the problem by activating "WSA Holographic Remoting" in Player-> XR Settings


The "ja-jp" edition of App Service page is translated by Chinese.

The "ja-jp" edition of this page(*)  is translated by Chinese.
(*) 为 Azure 应用服务配置 ASP.NET Core 应用
https://docs.microsoft.com/ja-jp/azure/app-service/configure-language-dotnetcore

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

ID: 568d6695-8344-3d44-aaaf-c00786f9bee1
Version Independent ID: 60034daf-9dff-5b16-9ff8-f675bafac61d
Content: Configure ASP.NET Core apps - Azure App Service
Content Source: articles/app-service/configure-language-dotnetcore.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@akinaritsugo Thanks for the feedback! We are currently investigating and will update you shortly.

Same issue is occurring on multiple pages and in multiple languages.
//chinese docs is translated by Russian
https://docs.microsoft.com/zh-tw/azure/active-directory/develop/v2-permissions-and-consent

@akinaritsugo Thanks for bringing this to our attention. This is resolved now.  We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


SOLR-17381: Make CLUSTERSTATUS configurable

https://issues.apache.org/jira/browse/SOLR-17381

Description
Please provide a short description of the changes you're making with this pull request.
Modify the requests to retrieve only the necessary information, such as the cluster status for a specific collection, live nodes, or cluster properties. Ensure these changes maintain backward compatibility. Additionally, update the HTTP CSP to reflect these optimizations.
Solution
Please provide a short description of the approach taken to implement your solution.
Introduce additional params to fetch CLUSTERSTATUS. Also added a INCLUDE_ALL param which would return all info by default to maintain backwards compatibility.
Additionally, made changes in BaseHttpClusterStateProvider to request minimal info for every use case.
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
No new tests added. Ensured all existing tests pass
Checklist
Please review the following and check all that apply:

[X] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[X] I have created a Jira issue and added the issue ID to my pull request title.
[X] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[X] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

Can you please update the ref guide, cluster-node-management.adoc, as well?

Done, let me know if it needs more changes

@gerlowskija in V2, I'm thinking these are separate endpoints

Nice! Nearly ready to commit IMO; needs a CHANGES.txt (which I could add if needed).

Done.

+1 Excellent; I'll merge soon

git bisect points to this commit!
ERROR: The following test(s) have failed:
  - org.apache.solr.client.solrj.impl.ClusterStateProviderTest.testGetClusterState (:solr:solrj)
    Test history: https://ge.apache.org/scans/tests?search.rootProjectNames=solr-root&tests.container=org.apache.solr.client.solrj.impl.ClusterStateProviderTest&tests.test=testGetClusterState http://fucit.org/solr-jenkins-reports/history-trend-of-recent-failures.html#series/org.apache.solr.client.solrj.impl.ClusterStateProviderTest.testGetClusterState
    Test output: /tmp/src/solr/solr/solrj/build/test-results/test/outputs/OUTPUT-org.apache.solr.client.solrj.impl.ClusterStateProviderTest.txt
    Reproduce with: ./gradlew :solr:solrj:test --tests "org.apache.solr.client.solrj.impl.ClusterStateProviderTest.testGetClusterState" -Ptests.jvms=96 "-Ptests.jvmargs=-XX:TieredStopAtLevel=1 -XX:+UseParallelGC -XX:ActiveProcessorCount=1 -XX:ReservedCodeCacheSize=120m" -Ptests.seed=33F6364CB7D4507A -Ptests.timeoutSuite=600000! -Ptests.file.encoding=ISO-8859-1


Allow configuration of SSL/TLS connection options

Hello,
based on my old issue #1021, it's quite possible that people are actively lowering their security by allowing older cipher suites, so they just can connect to SQLServers with old certificates. Instead it should be possible to define the ciphers used for the TSL connection to the SQLServer.
PDO_MYSQL already has support for this:

PDO::MYSQL_ATTR_SSL_CA (int)
The file path to the SSL certificate authority.
PDO::MYSQL_ATTR_SSL_CAPATH (int)
The file path to the directory that contains the trusted SSL CA certificates, which are stored in PEM format.
PDO::MYSQL_ATTR_SSL_CERT (int)
The file path to the SSL certificate.
PDO::MYSQL_ATTR_SSL_CIPHER (int)
A list of one or more permissible ciphers to use for SSL encryption, in a format understood by OpenSSL. For example: DHE-RSA-AES256-SHA:AES128-SHA
PDO::MYSQL_ATTR_SSL_KEY (int)
The file path to the SSL key.
PDO::MYSQL_ATTR_SSL_VERIFY_SERVER_CERT (int)
Provides a way to disable verification of the server SSL certificate.
This exists as of PHP 7.0.18 and PHP 7.1.4.

PDO::*_ATTR_SSL_VERIFY_SERVER_CERT is already possible through connection parameter TrustServerCertificate.
So please implement those attributes or as connection parameters.

Can these be specified in the connection string for MySQL server?

Can these be specified in the connection string for MySQL server?

No, only per config and attributes. Idk if mysqlnd has support for the attributes/config.

It's unlikely we'd be able to add these settings, since most of them are not part of the driver.
In addition to TrustServerCertificate the is also ServerCertificate that allows one to specify a .pem, but the other attributes seem like the OpenSSL settings that the driver doesn't control. OpenSSL has its own config files, which may allow specifying these values.

So I guess I can close this, since is required to implement this upstream into msodbc?

I note that if ServerCertificate => '/path/to/.pem' exists on pdo_sqlsrv, it's not documented here:
https://learn.microsoft.com/en-us/sql/connect/php/connection-options?view=sql-server-ver16

I'll take a look at that doc. I believe that anything in the connection string gets passed by PHP driver to the underlying ODBC Driver, so anything supported by that should work, but I'll double check to make sure.


Theoretical question about test_timestamp

Hello!
Thank you for building this cool tool :)
In tutorials/custom_dataset.ipynb it is mentioned that
only database rows upto test_timestamp should be used by the model to predict over the test set

My understanding is that typically seed_times for rows in the test_table will be greater than dataset.test_timestamp.
(That belief comes from 1) it seems restrictive to assume that all seed times in the test set are identical and 2) it seems risky to have test examples occuring before the global test_timestamp. Thus it follows some seed times will exceed the test_timestamp.).
If this understanding is correct, masking the entire entity graph upto test_timestamp makes some test examples very difficult, or impossible to predict. For example, if the test_timestamp is 2010-01-01 and a test example occurs 2020 with less than 10 years of historical data attached to it, pruning the graph with a single time will make this test case have an empty history.
Moreover, I dont see why pruning the entity graph globally is necessary to prevent test leakage;
Doesnt the torch geometric's NeighborLoader prune away future data in the computation graph associated with a test example, thus preventing time leakage entirely? It's docs mention the follwing:
 time_attr (str, optional): The name of the attribute that denotes
            timestamps for either the nodes or edges in the graph.
            If set, temporal sampling will be used such that neighbors are
            guaranteed to fulfill temporal constraints, *i.e.* neighbors have
            an earlier or equal timestamp than the center node.
            (default: :obj:`None`)

TLDR; Can pruning the entire graph at test_timestamp hurt test performance, and is it really necessary to prevent leakage?
Best regards, Jacob

Thanks for your questions.

We built the benchmark such that the seed_times in the test table are always the test_timestamp (rel-f1 is the only exception because there would be very few test samples otherwise). So, your hypothetical scenario where pruning the graph at test_timestamp hurts test performance does not typically arise. One way to think of this is that we train the model with info upto test_timestamp, and test it to make future predictions just beyond test_timestamp. Since the predictions can be made for many entities, there are still many samples to test against.

Yes, under our current implementation this pruning is not necessary to prevent leakage. But the benchmark is more general and intended to be used by people to develop their own methods. We make the conscious choice to avoid showing the test rows to the model to prevent any bugs. Under the current setup, bugs would lead to worse test set performance and be detectable. Without pruning, temporal leakage bugs would lead to better test performance and would hinder fair comparison on the leaderboard for example.

Hope this clarifies your doubts.

Thank you Rishabh!
Your answer does clarify all of my questions and it is understandable that you want the public leaderbord be protected from data snooping :)


Update Travis Config

Fixes #450
Opening this early, just throwing everything in the mix, to see how the build goes.

Removed Django 1.3, since it's end of life.
Added Django 1.6, 1.7, 1.8 and master.

OK. So that's good news.


Switch to AES-GCM encryption

Hello.  Is there any update on this issue?  Is this project still being maintained?


[QUESTION] Initialize VarBuilder from binary data from multiple safetensor files

Hi, I am playing around with the llama example using the Llama-2-7b-hf model.
In the candle source code, it initializes VarBuilder from files by let vb = unsafe { VarBuilder::from_mmaped_safetensors(&filenames, dtype, &device)? };
I have all the necessary files read and stored as bytes, so I'm looking into using the function from_buffered_safetensors(data: Vec<u8>, dtype: DType, dev: &Device) in var_builder.rs. However, I'm not sure how to pass in the bytes of more than one file into this. I tried to combine the bytes of two safetensors file into one by simply extending the two Vec<u8> but it is not working.
Any help would be appreciated!

Hi @helenzhangyc, I recently worked on this, probably for the same purpose. Implementing BufferedSafetensors::multi in a similar fashion to MmappedSafetensors::multi would not produce the desired effects due to the implementation of storage function in the device module.

@edesalve Thanks for your quick reply! So does this mean it is not possible to initialize a VarBuilder from multiple safetensors data in bytes?
If that is the case, do you have any idea how to run an inference from bytes? I don't want to serialize them into files.

@helenzhangyc at the moment the only way I found is to implement BufferedSafetensors::multi or to dig much much deeper in the crate. If there were options to do something like this without actually forking the crate, that would be great.

I think you should be able to load multiple safetensor data that are in memory by:

Use candle::safetensors::load_buffer(data, device) to load the content of the different safetensor buffers.
Merge the resulting hashmaps.
Call VarBuilder::from_tensors on the merged hashmap.

@LaurentMazare It works! Thanks!

@LaurentMazare This works fine on CPU. However, I'm getting a cuda out of memory error when running it on GPU.
I checked the CPU memory usage and it requires 39 GB of memory, which exceeds the amount of memory I have on GPU (I have 23 GB of GPU memory).
I'm confused because the Llama-2-7b-hf model has two safetensors files, one 9.98 GB and one 3.5 GB, so it should only requires ~13.5GB of memory. Moreover, it runs fine with cargo run --example llama --release --features cuda and running it like this only costs 13.5GB of GPU memory.
I wonder why getting the data from bytes and running inference costs 3 times the memory as running it from mmaped files.
FYI, my code looks like this:
let mut hashmap1 = candle_core::safetensors::load_buffer(&st1, &Device::new_cuda(0)?)?;
let mut hashmap2 = candle_core::safetensors::load_buffer(&st2, &Device::new_cuda(0)?)?;
hashmap1.extend(hashmap2);
let vb = VarBuilder::from_tensors(hashmap1, dtype, &Device::new_cuda(0)?);
(Llama::load(vb, &cache, &config)?, tokenizer_filename, cache)

where st1 and st2 are buffered data in Vec<u8>.
Any help would be appreciated!


New version: yt-dlp.yt-dlp.nightly version 2<IP_ADDRESS>710

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Automatic Validation ended with:

Installation failed with exit code -1978334972

(Automated response - build 570.)


Petitions Creation

Adds the ability to create real petitions to the database
Adds in loading spinner and check to make sure they do not double create

I'm going to change the petition creation form to match the new database setup in a future pull request. Seems like i'd just need to remove the 'lock' feature on each form field except the summary. Does that seem right? Also do we need another data table and function for saved draft petitions?

I can add it in with that future pull request, but I think it might also be good to always show the loading animation briefly just so the user can tell its actually doing something.

I'm going to change the petition creation form to match the new database setup in a future pull request. Seems like i'd just need to remove the 'lock' feature on each form field except the summary. Does that seem right? Also do we need another data table and function for saved draft petitions?

Yeah at least for now that should work. For the saved draft I would have to make a new table for that but that's a later problem.


Use UITextAlignment instead of a custom enum.

The enum currently used to indicate alignment did not have any namespace protection and didn't provide any obvious functionality. Replaced with UITextAlignment to match other cocos2d labels.

Great class btw. Using it in my app.

Thanks! Wasn't aware of the existence of UITextAlignment


Update input-method-editor-error.md

Updating this documentation as the issue has now been addressed and must be applied to both server and client

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

@tyadon I've approved the changes, but I need to make sure if it's okay for your local main branch to merge into the remote main branch, as you seem to have made this edit in your local main instead of a branch off your main.


Translation progress

Hey! Here is a Russian version :)
You can follow the progress in Projects
If someone wants to help (wich is very appreciated 👍) take please tasks one by one

Very nice the way you structured the project https://github.com/design-patterns-for-humans/Russian/projects/1

@UsulPro I wanna help. And I will.
@ivandrofly maybe create free slack chat ?

@UsulPro I will help you, but only after 8 pm (Minsk | Moscow time), because i am working :)
So, I hope you will understand me. And I am really want to help you with Russian translation. Okay ?

Okey @Egorvoz!
I don‘t think that could be any problems with it.
Just write here what are you going to translate

maybe create free slack chat ?
https://designpatterns4humans.slack.com

@Egorvoz email please so I can add you.
so you can add the others too

@ivandrofly<EMAIL_ADDRESS>

Sent! @Egorvoz

@UsulPro pls write your email, so i or @ivandrofly can invite you in slack.

hey,

my email is here))

<EMAIL_ADDRESS>

@UsulPro as soon as i can i'll send you an invitation.

Hey, Guys!
First 3 patterns are done!
It'd be great if somebody could review them!

@UsulPro invite was sent. And give me some sections for translations.
In this weekend i'll translate them all :)
Thanks.

@Egorvoz just pick some from projects and write down them here

@UsulPro Decarator, State, Template Method. Sounds Good ?

Great! these 3 tasks for you!

@UsulPro hello. Sry for my disappearing. I am ready to help. Just write me in commet down below what i need to translate and how i can help you :)
P.S. You did really awesome work dude. Respect.


Fix VC-JWT v1.1 having empty mapped properties

fixes #401

Codecov Report

Merging #402 (ad1eb7e) into main (07707db) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #402   +/-   ##
=======================================
  Coverage   57.28%   57.28%
=======================================
  Files          67       67
  Lines        7297     7297
=======================================
  Hits         4180     4180
  Misses       2380     2380
  Partials      737      737

Impacted Files
Coverage Δ

credential/model.go
64.29% <ø> (ø)

credential/jwt.go
40.43% <100.00%> (ø)


Run CI against Xcode 12 beta

Originally tried this in #909 / #917 but reverted because Spectre wouldn't build with Xcode 12b3. It does build with b4, but GitHub Actions doesn't support it yet (incoming with https://github.com/actions/virtual-environments/pull/1409)
This PR will probably initially fail but once 12b4 (12A8179i) is deployed on GH, it should pass on re-run 🤞
@yonaskolb (cc @brentleyjones)

I saw that spectre has a new release for Xcode 12. We should probably update to that

@yonaskolb Unfortunately, it seems that Spectre 0.9.1 doesn't compile on beta 4 :(
Since GitHub is upgrading soon, it would work in the short term but then start failing once they roll up to beta 4.
/path/to/XcodeGen/.build/checkouts/Spectre/Sources/Spectre/XCTest@5.swift:11:21: error: 'XCTIssueReference' is not implicitly convertible to 'XCTIssue'; did you mean to use 'as' to explicitly convert?
    testCase.record(issue)
                    ^
                          as XCTIssue

@yonaskolb Re-triggered. Looks like CI has 12A8179i installed now so xcodegen built fine, but now Carthage is failing when setting up the project fixtures -_-

@yonaskolb Finally green after adding a workaround for Carthage :+1:
The fixtures including Xcode-12+-only targets are now successfully building in CI.

I heard that GitHub actions sometimes runs beta 4, but also sometimes runs beta 3 😕.

Oof. Then maybe we hold off until it's confirmed in the GH actions docs.

@yonaskolb @brentleyjones They merged https://github.com/actions/virtual-environments/pull/1409 today, so I would assume the rollout is 100% complete for beta4 on GitHub Actions.


Create a recognizable logo

The current logo on Twitter and Meetup isn't very "Java" recognizable.
We should change this.

@TomCools I enjoy drawing in my free time, so after seeing this, I gave it a try.  Tell me if it's near anything that can match what you're looking for.  Let me know your thoughts and feedback.

@lmagnette I love it! Mind if we throw it on the Twitter/Mastodon to see what the rest of the community thinks? 😊

@TomCools I'm happy to hear that you love it. No problem to me if you to share it and get some more feedback

Love it too! One small remark, maybe a small line or shadow is needed for the Duke's arms to make them stand out from the black background?

@lmagnette, I heard a lot of good noises! People like it! Any idea when you'll have the time to look into @FDelporte's comment? :-)

@TomCools here's an updated version based on @FDelporte remark.

That was quick! :) @FDelporte what do you think?

It's certainly clearer, mayhaps also give the french fries the white outline?

@SimonVerhoeven you're right I forgot the white outline on the french fries. Here's the updated version.

Love it :-)

I like it.

🚢  to production then! 😊 🚀


Doesnt take into account width of all text and allign the right comment tags

right side comment tags dont get aligned to the max width

Hey @dturco-kbx thanks for letting me know, I'll look into it. Could you please copy paste the settings you're using? Thanks!

"commentBox.styles": {
"defaultStyle": {},
"yaml":{
"capitalize": false,
"ignoreInnerIndentation": false,
"ignoreOuterIndentation": false,
"removeEmptyLines": false,
"textAlignment": "left",
"commentStartToken": "##",
"commentEndToken": "##",
"leftEdgeToken": "# ",
"rightEdgeToken": " #",
"topEdgeToken": "#",
"bottomEdgeToken": "#",
"topRightToken": "##",
"bottomLeftToken": "##",
}
},
From: slysherz @.>
Sent: Monday, November 14, 2022 9:43 AM
To: slysherz/vscode-comment-box @.>
Cc: Turco, Dan @.>; Mention @.>
Subject: Re: [slysherz/vscode-comment-box] Doesnt take into account width of all text and allign the right comment tags (Issue #21)
Sent by an external sender

Hey @dturco-kbxhttps://urldefense.com/v3/__https:/github.com/dturco-kbx__;!!HKYIif90!xyLJWZFzT0nxDuS42LiQ6g0uS6Xmn-e8IErNv47ZUG0tzWZU0Pa9fzu8ea7k8Gl8z4xozNXJevgUlKIatFIWD2-liUC4_A$ thanks for letting me know, I'll look into it. Could you please copy paste the settings you're using? Thanks!
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/slysherz/vscode-comment-box/issues/21*issuecomment-1313957279__;Iw!!HKYIif90!xyLJWZFzT0nxDuS42LiQ6g0uS6Xmn-e8IErNv47ZUG0tzWZU0Pa9fzu8ea7k8Gl8z4xozNXJevgUlKIatFIWD2-MU4XEaw$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A3WR7E4D5G3OXZUWQBSGCF3WIJMZ3ANCNFSM6AAAAAAR76T5LY__;!!HKYIif90!xyLJWZFzT0nxDuS42LiQ6g0uS6Xmn-e8IErNv47ZUG0tzWZU0Pa9fzu8ea7k8Gl8z4xozNXJevgUlKIatFIWD284fFf-Xw$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Ah, I know what this is, maxEndColumn isn't working as it should. You can work around this by increasing this setting while I work on a fix:
"maxEndColumn": 999

Awesome ! Thanks
From: slysherz @.>
Sent: Monday, November 14, 2022 5:35 PM
To: slysherz/vscode-comment-box @.>
Cc: Turco, Dan @.>; Mention @.>
Subject: Re: [slysherz/vscode-comment-box] Doesnt take into account width of all text and allign the right comment tags (Issue #21)
Sent by an external sender

Ah, I know what this is, maxEndColumn isn't working as it should. You can work around this by increasing this setting while I work on a fix:
"maxEndColumn": 999
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/slysherz/vscode-comment-box/issues/21*issuecomment-1314548304__;Iw!!HKYIif90!wc3yUTdicH54SOGA_VWL43NXepCIVxrYyzzpdbEz27moQWzEmcbqQO9A3-b2P_dBe_3ZL6YFFAiBjA8sfoQrL0pE0hnbEw$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A3WR7EY4MRWVTWTO5VR43G3WILEA3ANCNFSM6AAAAAAR76T5LY__;!!HKYIif90!wc3yUTdicH54SOGA_VWL43NXepCIVxrYyzzpdbEz27moQWzEmcbqQO9A3-b2P_dBe_3ZL6YFFAiBjA8sfoQrL0rwFF8j0g$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Add a workflow based on custom runner

@pizzamig Is it possible that this has been made obsolete by #236?

kinda.
This solution is based on a pot-based github action custom runner.
Running pot in pot is currently not supported.
I could change this PR in draft mode

Running pot in a jail would be extremely useful to me (pot in pot could also be useful for side-cars etc.).


added Google Messages recipe (based on the Allo recipe)

for Google's newly added web support for Messages

I have verified that the code for @ttal's recipe does not correctly pull the message count. I favor #169 over this PR

Closing this in favor of #169


Changes required to work with latest htsjdk

This is in preparation for the next htsjdk release (and can't be merged until then) which has several changes to the CRAM API. It passes all tests locally when run against htsjdk master.
cc @lbergelson @cmnbroad

@heuermh Could you take a look at the final form of this?  I'd like to do a release with these changes in the near future if there aren't any objections.

@lbergelson Sure, I'll run some testing against this branch this afternoon.  I ran into #112 and #116 running benchmarks in preparing for merging Disq into ADAM, I'll also try to see if there is anything actionable on those.

Bumping htsjdk to 2.20.0 and disq to 0.3.3-SNAPSHOT causes this ADAM unit test to fail
- convert alignments to reads *** FAILED ***
  "[6472783]:false" did not equal "[simread:1:101556378]:false"

AlignmentRecordDatasetSuite.scala:1871
Perhaps the way the sequence name is parsed has changed since 2.19.0?

Strange... I don't remember any deliberate changes to how we parse sequence names.

Other than that, I haven't found any runtime issues in my benchmarking scripts, so I'm a tentative +1

@heuermh I'm going to merge this and do release then so we can do a downstream gatk release on Tuesday.  Let me know if you have any progress on that weird read name issue.  Something seems very odd there.

Sounds good to me

@lbergelson I've created an ADAM pull request at https://github.com/bigdatagenomics/adam/pull/2195 that demonstrates the read name issue with htsjdk 2.20.0.  Very strange that only the first read name is truncated.


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2023-11-21T02:47:35+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote_istio_release-1.20
7010f2295e9afece1500ce4a506a86bf5302a0f0
link
unknown
/test integ-pilot-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote_istio_release-1.20
7010f2295e9afece1500ce4a506a86bf5302a0f0
link
unknown
/test integ-pilot-istiodremote

integ-security-istiodremote_istio_release-1.20
7010f2295e9afece1500ce4a506a86bf5302a0f0
link
unknown
/test integ-security-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote_istio_release-1.20
7010f2295e9afece1500ce4a506a86bf5302a0f0
link
unknown
/test integ-pilot-istiodremote

integ-security-istiodremote_istio_release-1.20
7010f2295e9afece1500ce4a506a86bf5302a0f0
link
unknown
/test integ-security-istiodremote

integ-security-multicluster_istio_release-1.20
7010f2295e9afece1500ce4a506a86bf5302a0f0
link
unknown
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

wait https://github.com/istio/istio/pull/47954

/retest


CET  acronym used before being defined

The acronym CET is used without explaining what the acronym stands for.

Related #16


Minified Version of SDK No Where to be found

Clicking on the link https://steemit.github.io/sc2-angular/sc2.min.js brings up an error page.
Is there somewhere else I can find the minified SDK for steemconnect?

You can get it here: https://github.com/steemit/example-steemconnect-angular/blob/master/sc2.min.js or build it from this repo.

You're great - thanks so much for the prompt response.


Fix presto doc

-Xbootclasspath/p is no longer a supported option.
https://prestosql.io/docs/current/connector/hive.html#alluxio-client-side-configuration
https://prestodb.io/docs/current/connector/hive.html#alluxio-client-side-configuration

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10245/
Test PASSed.

@alluxio-bot check

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.3 please


Can we add a feedback button to tutorials?

I'd love to see what's happening with Winter…but it would also be good to compare tutorial feedback with skillmap feedback.

Closing in favor of https://github.com/microsoft/pxt-arcade/issues/5911


version_bump_podspec does not use all given parameters (if multiple are given)

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
When using the action version_bump_podspec  with the parameter bump_type and version_appendix, the version number is not bumped. I am not sure if this is the expected behaviour, but I would expect it to bump the version and change the appendix. If the version number is for example <IP_ADDRESS> and I run
version_bump_podspec(
    path: "example_path_to.podspec",
    bump_type: 'patch',
    version_appendix: "0"
)

I would expect the new version number to be: <IP_ADDRESS> but currently the version number is set to <IP_ADDRESS>. Apparently the bump in the patch component is not executed. Even if this is the wanted behaviour, in my opinion it should be pointed out somewhere, that only one (and which one) of the actions is carried out when multiple parameters are provided.
Command executed
version_bump_podspec(
    path: "example_path_to.podspec",
    bump_type: 'patch',
    version_appendix: "0"
)

Complete output when running fastlane, including the stack trace and command used

Note: the version before the execution was: <IP_ADDRESS>

 fastlane run version_bump_podspec path:"My-Framework.podspec" bump_type:"patch" version_appendix:"0" --capture_output
[✔] 🚀
[12:48:05]: Get started using a Gemfile for fastlane https://docs.fastlane.tools/getting-started/ios/setup/#use-a-gemfile
DEBUG [2019-12-04 12:48:05.84]: Checking if there are any plugins that should be loaded...
DEBUG [2019-12-04 12:48:05.95]: Using path: My-Framework.podspec
DEBUG [2019-12-04 12:48:05.96]: Using bump_type: patch
DEBUG [2019-12-04 12:48:05.96]: Using version_appendix: 0
INFO [2019-12-04 12:48:05.96]: ----------------------------------
INFO [2019-12-04 12:48:05.96]: --- Step: version_bump_podspec ---
INFO [2019-12-04 12:48:05.96]: ----------------------------------
INFO [2019-12-04 12:48:05.96]: Result: <IP_ADDRESS>

Environment

 🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.14.6

Ruby
2.6.0

Bundler?
false

Git
git version 2.21.0 (Apple Git-122.2)

Installation Source
~/.rvm/gems/ruby-2.6.0/bin/fastlane

Host
Mac OS X 10.14.6 (18G1012)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.6.0/lib

OpenSSL Version
OpenSSL 1.1.1b  26 Feb 2019

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode11.2.1.app/Contents/Developer/

Xcode Version
11.2.1

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`

@podspec_path = "My-Framework.podspec"

lane :test do |options|
  version = version_get_podspec(path: @podspec_path)
  UI.message("Current verison is: #{version}")
  version_bump_podspec(
          path: @podspec_path,
          bump_type: 'patch',
          version_appendix: "0"
      )

  version = version_get_podspec(path: @podspec_path)
  UI.message("Bumped version is: #{version}")
end

No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.137.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

executable-hooks
1.6.0

bundler-unload
1.0.2

rubygems-bundler
1.4.5

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

terminal-table
1.8.0

plist
3.5.0

multipart-post
2.0.0

word_wrap
1.0.0

public_suffix
2.0.5

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.13.1

gh_inspector
1.1.3

mini_magick
4.9.5

multi_xml
0.6.0

rubyzip
1.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

jwt
2.1.0

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

memoist
0.16.0

googleauth
0.6.7

httpclient
2.8.3

google-api-client
0.23.9

digest-crc
0.4.1

google-cloud-storage
1.16.0

emoji_regex
1.0.1

nanaimo
0.2.6

colored2
3.1.2

claide
1.0.3

CFPropertyList
3.0.1

atomos
0.1.3

xcodeproj
1.13.0

unicode-display_width
1.6.0

addressable
2.7.0

tty-screen
0.7.0

tty-cursor
0.7.0

tty-spinner
0.9.1

babosa
1.0.3

excon
0.68.0

faraday
0.17.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

fastimage
2.1.7

json
2.2.0

dotenv
2.7.5

bundler
2.0.1

naturally
2.2.0

simctl
1.6.6

mime-types-data
3.2019.1009

mime-types
3.3

multi_json
1.14.1

signet
0.12.0

os
1.0.1

google-cloud-env
1.3.0

google-cloud-core
1.4.1

forwardable
1.2.0

logger
1.3.0

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

date
1.0.0

fileutils
1.1.0

rexml
3.1.9

psych
3.1.0

mutex_m
0.1.0

generated on: 2019-12-04

Latest version hasn't fixed the issue.


Extension issue

Issue Type: Bug
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT ia32 6.3.9600
VSCode version: 1.41.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:
{
	"messages": [],
	"activationTimes": {
		"codeLoadingTime": 27185,
		"activateCallTime": 78,
		"activateResolvedTime": 0,
		"activationReason": {
			"startup": false,
			"extensionId": {
				"value": "ritwickdey.LiveServer",
				"_lower": "ritwickdey.liveserver"
			},
			"activationEvent": "*"
		}
	},
	"runtimeErrors": []
}

it refuses to work.


Distribution: schema download link and corresponding documentation

For a Distribution, I see the property mobilitydcatap:dataModelSchema. Its description is: “This property describes the schema of the delivered content, as applied for the data model, see property mobilitydcatap:dataModel). The schema can be individually determined by the data provider (e.g., a stakeholder-based DATEX II profile) or a prescribed by other institutions (e.g., a DATEX II recommended reference profile). […]”
I do not quite understand how I have to read this; it reads like “a description of the schema/XSD has to be given in this property”. But XSD in itself is a description.

If we take DATEX II as an example, would it not be more, or also, suitable to provide the option to add to a distribution a download link to the schema (XSD)? (Say, maybe dataModelSchemaDownloadURL?)

Lots of elaborate documentation about DATEX II profiles (schemas) is given, for example on websites (datex2.eu). If mobilityDCAT-AP would allow to give a link to a schema for download, a property (say, maybe DataModelDocumentationURL) to point to the corresponding documentation for this schema would, in my opinion, be very good to have.

Regarding "I do no understand...":
We will refine the usage note as follows:
"This property refers to a data schema associated with the applied data model, see property mobilitydcatap:dataModel."
Regarding 1:
You propose to change the range of this property from "rdfs:Resource" to "dcat:Distribution".
If we do so, the schema information would need to include any mandatory properties from the class "dcat:Distribution", see:
https://mobilitydcat-ap.github.io/mobilityDCAT-AP/drafts/latest/#properties-for-distribution
This would unnecessarily explode the payload: For example, the property "mobilitydcatap:dataModel" would need to be noted again, which is equal to "mobilitydcatap:dataModel" of the original distribution.
I propose to keep the generic range "rdfs:Resource".
Regarding 2:
If we keep the range "rdfs:Resource" (see above), the NAP system can decide how to reference the schema. It might be a URL to a NAP-internal catalogue of schemas, a URL to an individual schema (that is provided by the metadata publisher), or a URL to an external catalogue of schemas (like on the datex2.eu page). I will add this as a usage note.
I propose NOT to not include any additional properties for mobilityDCAT-AP1.0. We have only a few days until the publication, and property additions would involve another review round. However, let's think about refinements of our format-related properties (which may be expressed as formal sub-properties) for mobilityDCAT-AP2.0.

Thanks for the proposals.
I was not aware that there are (only) a few days left until the release of version 1.0. So, it's nice that the usage note of 1 and 2 can still be adjusted.
To see later whether it is desirable to further develop format-related properties, looks like a sensible route to me.

I resolved this while resolving this issue:
https://github.com/mobilityDCAT-AP/mobilityDCAT-AP/issues/12
See point 2.
-> There is a dedicated new class "Mobility Data Standard", with a property "Schema".
-> The range of "Schema" is kept with "rdfs:Resource", for the reasons above.
-> The usage of "Schema" has been concretised, with the options how to describe a schema described above.


about cvt?

check that, the dim_head of stage 3 is 4 not 6 ?
https://github.com/lucidrains/vit-pytorch/blob/bad4b94e7b4baa544ca36149431f7912eccd4b49/vit_pytorch/cvt.py#L107-L137
Look like the default set is cvt13

Moreover, what's the meaning of proj_kernel?
Look like we should replace the 3 in DepthWiseConv2d to proj_kernel? Is it right?
https://github.com/lucidrains/vit-pytorch/blob/bad4b94e7b4baa544ca36149431f7912eccd4b49/vit_pytorch/cvt.py#L60-L78

@huixiancheng yes, you are right on both accounts https://github.com/lucidrains/vit-pytorch/commit/54af2209306d3d1c9fa1c9e2974b4dd93bac684f :pray:

Thank you for your code. :bow:

Hi Dear @lucidrains
Do your think we need LN between Patch embedding and Transfomer blocks?
https://github.com/lucidrains/vit-pytorch/blob/bad4b94e7b4baa544ca36149431f7912eccd4b49/vit_pytorch/cvt.py#L147-L150
The orginal show they: rearrange BCHW to  B(HW)C then with a LN.

I think flattened is not necessary like your code show. Since we will rearrange it back again in DepthWiseConv2d.
But look like LN is necessary at least for 1st stage. Maybe it's not need for stage 2 or 3.
However, I have no idea of the important of LN to Transformer

@huixiancheng i missed this detail! do you want to look at https://github.com/lucidrains/vit-pytorch/commit/3df6c31c612ab0261fe3a5f8dd39b7957f5b2a56 and see if that fits your needs? I also brought in a layernorm that works on the first dimension, so to avoid all the reshapes and flattening

thx!   Nice work!


Clean up Elaborator hacks

Problem
The elaborator and the old resolution/type checking passes re-use certain types and the same CI flow. This can lead to some hacks in the codebase to get certain types working in both the old pass as well as the new elaborator. We should remove all these hacks once we fully move over to the elaborator.
E.g. in https://github.com/noir-lang/noir/pull/5155 we added a test for explicit numeric generic that only works with the elaborator.
Happy Case
Link this issue anywhere there is elaborator tech debt and clean up the respective tech debt.
Project Impact
Nice-to-have
Impact Context
No response
Workaround
None
Workaround Description
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

@vezenovm Was this all suitably cleared up once we removed the old flow?

Yes I believe so, I cannot find the issue linked anywhere in the code base either so any new hacks can be linked in a new issue.


Fix/issue 9

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #9
What is the new behavior?

Fixes grouping
Introduces new builder pattern to achieve pre-groupingconst parentBuilder = new HotToastBuilder(ngTemplateGroup, toast)
.setOptions({
  position: 'top-right',
  autoClose: false,
  className: 'hot-toast-custom-class',
  group: {
    className: 'hot-toast-custom-class'
  }
});

// Create child toasts
const children = childNotifications(ngTemplateGroupItem).map((child) => {
  return new HotToastBuilder(child.options.message, toast)
    .setOptions(child.options);
});

// Add children to parent
children.forEach((child) => parentBuilder.addChild(child));

// Create the toast with all children (but don't show yet)
const parentRef = parentBuilder.create();

// Once all refs are attached, show the parent toast
parentRef.afterGroupRefsAttached.subscribe(() => {
  parentRef.show();
});

Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

:tada: This PR is included in version 3.1.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The TypeCommand type brings up a full notepad-esque interface.

http://i.imgur.com/fB44GBl.png

https://gist.github.com/Azaezel/c246aee4dbad00f4f019 quickiedump of additional spots to eyeball if we want to add that functionality to more places

Great idea.


FRONT-END All Projects.

This feature is used to create the interface for all the projects.

Done, except for community based

Done, except for community based by @martinwachira


Tutorial request

Hi
Just wondering, if there is any example of model (or some tutorial) that works with spherical healpy data. How does flax deal with that? convolution layers or pooling layers etc. for classification or denoising models.
If there is any tutorial that would be very helpful.
Many Thanks
Wasim

We don't have any tutorials that work with spherical healpy data. Is there an accepted standard for processing that type of data in the ML domain? I've never worked with this type of data before

I can understand. This is a domain specific issue. Specially in astronomy and astrophysics. Various kinds of cosmological data that heavily requires to preserve sky orientations, positions etc along with their features, foreground and background informations in the correct/exact sequence etc, suffer mostly from data distortion while flatting them. Very limited options are there in ML domain that deals with these data shape as it is not trivial. Thanks for your prompt response.


The feature in WebGLLayer cannot be obtained using map.forEachFeatureAtPixel

Declare a class
class WebGLLayer extends Layer {
        createRenderer() {
          return new WebGLVectorLayerRenderer(this, {
            stroke: {
              attributes: {
                color: function (feature) {
                  const color = [...asArray(feature.get("COLOR") || "#eee")];
                  color.forEach(
                    (_, i) => (color[i] = Math.round(color[i] * 0.75))
                  ); // darken slightly
                  return packColor(color);
                },
                width: function () {
                  return 1.5;
                },
                opacity: function () {
                  return 1;
                },
              },
            },
          });
        }
      }

Instantiate a class
let source = new VectorSource({
        url: "https://openlayers.org/data/vector/ecoregions.json",
        format: new GeoJSON(),
      });
let webGLLayer = new WebGLLayer({
        // WebGLLayer VectorLayer
        source: source,
      });

The feature cannot be obtained by the following method.
this.map.on("pointermove", function (evt) {
        const feature = that.map.forEachFeatureAtPixel(
          evt.pixel,
          (feature, layer) => {
            if (feature) {
              console.log(feature);
            }
            return feature;
          }
        );
      });

My ol version is 7.3.0

@jahow Has Hit detection implemented for this experimental renderer?

Not yet, unfortunately. Contributions very welcome!

@jia-mx I encountered the same problem, did you solve it?

@jia-mx I encountered the same problem, did you solve it?

@zj91zj Version 9.0.0 already supports obtaining features in the webgl layer through map.forEachFeatureAtPixel. But there are some problems with browser memory when the amount of data loaded is too large. The following is an example https://openlayers.org/en/latest/examples/webgl-vector-layer.html


Bad interaction between IPRO and ModifyCachingHeaders

If you have IPRO (the default) and turn off ModifyCachingHeaders then we'll remove ETag and Last-Modified:
    server {
      listen 80;
      server_name experiment;
      pagespeed on;
      root /var/www/;

      pagespeed ModifyCachingHeaders off;
    }

See:
$ http_proxy=www.jefftk.com:80 headers experiment/b.css
HTTP/1.1 200 OK
Server: nginx/1.8.1
Date: Mon, 21 Mar 2016 20:56:42 GMT
Content-Type: text/css
Content-Length: 34
Last-Modified: Wed, 25 Jun 2014 16:59:27 GMT
Connection: keep-alive
Vary: Accept-Encoding
ETag: "53aaffef-22"
Accept-Ranges: bytes

On the first load everything is fine, because IPRO hasn't made changes yet.  Then on the second load:
$ http_proxy=www.jefftk.com:80 headers experiment/b.css
HTTP/1.1 200 OK
Server: nginx/1.8.1
Date: Mon, 21 Mar 2016 20:56:43 GMT
Content-Type: text/css
Connection: keep-alive
Accept-Ranges: bytes
X-Original-Content-Length: 34
Vary: Accept-Encoding
Vary: Accept-Encoding
Content-Length: 34

If someone turns off modifying caching headers, we should just not modify them.
(The duplicate vary headers are because jefftk.com is running the latest beta, which doesn't have #1064 yet.)

(See #1271)


[release/1.11] Don't claim to have json payload when it is blank

Cherrypick of #186

LGTM

LGTM

LGTM.


Unable to run under Linux

Hello, I was unable to run Rotativia under Linux machine.  I've got below error
fail: Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware[0]
      An unhandled exception has occurred: Broken pipe
System.IO.IOException: Broken pipe
   at System.IO.FileStream.WriteNative(Byte[] array, Int32 offset, Int32 count)
   at System.IO.FileStream.FlushWriteBuffer()
   at System.IO.FileStream.Dispose(Boolean disposing)
   at System.IO.Stream.Close()
   at System.IO.StreamWriter.Dispose(Boolean disposing)
   at System.IO.TextWriter.Dispose()
   at Rotativa.AspNetCore.WkhtmlDriver.Convert(String wkhtmlPath, String switches, String html, String wkhtmlExe)
   at Rotativa.AspNetCore.WkhtmltopdfDriver.ConvertHtml(String wkhtmltopdfPath, String switches, String html)
   at Rotativa.AspNetCore.ViewAsPdf.<CallTheDriver>d__19.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Rotativa.AspNetCore.AsResultBase.<BuildFile>d__65.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Rotativa.AspNetCore.AsResultBase.<ExecuteResultAsync>d__66.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeResultAsync>d__19.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeNextResultFilterAsync>d__24.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow(ResultExecutedContext context)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeNextResourceFilter>d__22.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow(ResourceExecutedContext context)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeFilterPipelineAsync>d__17.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.<InvokeAsync>d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.<Invoke>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at DevExpress.AspNetCore.Internal.BinaryStorageMiddleware.<Invoke>d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at DevExpress.AspNetCore.Internal.ResourceManagerMiddleware.<Invoke>d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Session.SessionMiddleware.<Invoke>d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Session.SessionMiddleware.<Invoke>d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Localization.RequestLocalizationMiddleware.<Invoke>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()

.Net version:
.NET Core SDK (reflecting any global.json):
 Version:   2.1.302
 Commit:<PHONE_NUMBER>

Runtime Environment:
 OS Name:     centos
 OS Version:  7
 OS Platform: Linux
 RID:         centos.7-x64
 Base Path:   /usr/share/dotnet/sdk/2.1.302/

Host (useful for support):
  Version: 2.1.2
  Commit:  811c3ce6c0

.NET Core SDKs installed:
  2.0.0 [/usr/share/dotnet/sdk]
  2.1.302 [/usr/share/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.2 [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.2 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.0 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.2 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

Any advice how to run it?

this is not linux ELF images available yet, so it is runnable under only windows docker images and such

is it still not working on the MAC？

is it still not working on the MAC？

It does work on macOS, just rename the binary wkhtmltopdf to wkhtmltopdf.exe.

For me to correct this error I had to put these files that are available in the link below:
https://github.com/MatheusNCarvalho/wkhtmltox-0.12.4
and my Dockerfile looks like this:
### Stage 1 - Obtain the source and generate the Build ###
FROM microsoft/dotnet:2.2-sdk AS build-env
WORKDIR /app

### Copy and restore dependencies ###
COPY . /app
RUN dotnet restore Teste.csproj
RUN dotnet publish Teste.csproj -c Release -o /app/publish

### Stage 2 - Upload the application through the binaries ###
## Build da imagem ###

FROM microsoft/dotnet:2.2-aspnetcore-runtime

RUN ["apt-get", "update"]
RUN ["apt-get", "-y", "install", "libgdiplus"]

WORKDIR /app
COPY --from=build-env /app/publish .

RUN chmod 755 /app/wwwroot/Rotativa/wkhtmltopdf
RUN chmod 755 /app/wwwroot/Rotativa/wkhtmltoimage

ENV ASPNETCORE_ENVIRONMENT Staging
ENTRYPOINT ["dotnet", "Teste.dll"]


Availability group dashboard stuck on healty state

hello, i set up an AG with 2 replica and a configuration only instance (ubuntu 16.4, .mssql 2017)  i understand that this is an external managed cluster, but i expect the dashboard to work: now my 3 servers are switched off, and the ag state is healty, the synch state synchronized,  there is only a message in the title bar "an error occurred on the always on dashboard"  (failed to connect to server) , why this state is not reflected to the replica synch status / health? anfter some minutes ssms crashed (Application: Ssms.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.)

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

ID: e76df1cf-a234-3768-3c94-b7270d249829
Version Independent ID: 1235d9a1-8c44-2a11-a213-a1ced144616a
Content: Configure SQL Server Always On Availability Group for high availability on Linux - SQL Server
Content Source: docs/linux/sql-server-linux-availability-group-configure-ha.md
Product: sql
Technology: linux
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@baleng  Thank you for the question.
A similar issue was posted on the StackOverflow.
Please try upgrading your SSMS client to see if the issue is resolved.

@mseblot thank you for your input.
i have ssms 17.7 and my issue is bit different, my dashboard opens perfectly all is green but it seems to be a jpg image frozen on a "all ok" state after the servers are switched off.
by the way i'll try with the latest ssms version 18.2 and keep you posted.
thank you

@baleng  Sounds good, no problem.
If this is a new issue, we should move this to the SQL Server forums.

Hi @msebolt  , i installed ssms 18.2 , i have the same behaviour.
i have 3 replica (one two and three) , one is primary, two secondary , three configuration only.
if i make three unavailable (witness node), the dashboard reflects the critical situation.
if i make two unavailable the dashboard works as expected
if a make one (primary) unavailable the dashboard says

i would expect the green round on the left of one to be red and not synchonizing, and the availability group state to be NOT healty (with the red button)
and two times ssms crashed due unhandled exception
Application: Ssms.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.ArgumentException
at System.Drawing.Region.GetHrgn(System.Drawing.Graphics)
at System.Windows.Forms.Control.GetHRgn(System.Drawing.Region)
at System.Windows.Forms.Control.Invalidate(System.Drawing.Region, Boolean)
at System.Windows.Forms.LinkLabel.InvalidateLink(Link)
at System.Windows.Forms.LinkLabel.OnGotFocus(System.EventArgs)
at System.Windows.Forms.Control.WmSetFocus(System.Windows.Forms.Message ByRef)
at System.Windows.Forms.Control.WndProc(System.Windows.Forms.Message ByRef)
at System.Windows.Forms.Label.WndProc(System.Windows.Forms.Message ByRef)
at System.Windows.Forms.LinkLabel.WndProc(System.Windows.Forms.Message ByRef)
at System.Windows.Forms.Control+ControlNativeWindow.OnMessage(System.Windows.Forms.Message ByRef)
at System.Windows.Forms.Control+ControlNativeWindow.WndProc(System.Windows.Forms.Message ByRef)
at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr, Int32, IntPtr, IntPtr)
thanks

@baleng  Thank you for the details.  I've submitted this as a bug at https://feedback.azure.com/forums/908035-sql-server/suggestions/38432464-availability-group-dashboard-stuck-on-healty-state.
Also assigning the author.

Thank you for your interest
best regards!

@baleng At this time, I don't think that there is anything we can do from a documentation standpoint. And it looks like @msebolt added this as a bug/feedback-item. I am going to close the issue at this time. I apologize you ran into this experience, and please let me know if you have any other questions. Thank you!


Save.browser.export() should include the game release version in the default file name/output file

Is your feature request related to a problem?
According to the documentation of Save.browser.export():

filename: (string) The base filename of the browser save export, which gets slugified to remove most symbols. Appended to this is a datestamp (format: YYYMMDD-hhmmss) and the file extension .savesexport—e.g., "The Scooby Chronicles" would result in the full filename: the-scooby-chronicles-{datestamp}.savesexport.

Here's a scenario, though it may be far-fetched: imagine a player is unable to load their save because of a bug that you thought you already fixed. They send you their export file so you can troubleshoot. Without knowing what version of the game they exported from, how can you answer these questions:

Are they playing a version of the game before you fixed the bug?
Are they playing a version >= where you thought you fixed the bug?

Knowing the answers to these questions provides valuable information.
Describe the solution you'd like.
Include Config.saves.version in the output file or filename.
Describe alternatives you've considered.
I'm assuming the default save dialog will use this new function. Unless I'm misunderstanding, modifying the default filename produced by clicking this button would require relatively advanced JavaScript trickery: e.g., monkey patching the export button's ariaClick()
Additional context.
This relates to https://github.com/tmedwards/sugarcube-2/issues/190
You can find documentation for this here https://github.com/tmedwards/sugarcube-2/issues/248#issuecomment-1613881828
The conversation starts here: https://discord.com/channels/389867840406159362/389868418855075840/1124107029452181646

After thinking about it a bit.  I'm unsure how useful adding a game version to the suggested filename of a browser saves export would be.  The issues being that a) the export could potentially contain saves from not only different game and save versions, but b) potentially not even the game version that's on the filename.  In this scenario, adding the current game version to the suggested filename probably won't tell you much useful.
Adding a game version to the suggested filename of a disk save would be somewhat better, as you're guaranteed that the game version the save was created under matches what's in the filename.
All in all, I think it's probably better to add extra details like this to the metadata of each save, so you have access to it no matter when or where a save is created, or in what fashion you receive it.
To that end, while you can add metadata by manually calling the APIs, there's no convenient way to do so when using the Saves dialog.  Thus, I'm thinking what would be best is to add a new Config.saves API to allow default save metadata to be provided.  For example, something like simply using its value:
Config.saves.metadata = {
	gameVersion : 'Release 12: "Horny Toads Need Love Too" edition'
};

Or perhaps make it a callback that returns the value, so dynamic metadata would be possible:
Config.saves.metadata = function () {
	var metadata = {
		gameVersion : 'Release 12: "Horny Toads Need Love Too" edition'
	};

	if (/* some logic */) {
		metadata.something = 'a value';
	}

	return metadata;
};

All of the various save "load" APIs give you access to the save info chunk, which includes the metadata.  Thus, you'd easily be able to check the game version under which any save was created—either manually from the console or by using a Save.onLoad handler to report it in some way.
Cc: @ChapelR @greyelf

My understanding is the issue is that the save file doesn't work, so no one can (easily) load it to see the data, and therefore what the version of the save is? I'm not sure I quite get it. I guess the idea is that the user could hand over the malfunctioning save, and you can just see right from the filename that it's an old version or not. But people can change file names anyway, and if it's the only copy of their saved game and they don't remember when they made it, you're screwed anyway. It could be even worse if the player starts guessing.
I think what would make more sense (if this is a concern warranting a fix) would be like a "header" or something in the save file that isn't base64'd, just plain text. Maybe game title, save id, save version, date. Only for save to disk saves, obviously. I'm not sure it's that big of a concern, and also, you can like open a SugarCube save on Save Edit Online even if you aren't a power user and check the game version in like a second, so I don't think a header would really bring a ton of value. But if I were inclined to somehow have the game version in "plain text" in the save file, I'd do something like a header, not anything with the file name.

To that end, while you can add metadata by manually calling the APIs, there's no convenient way to do so when using the Saves dialog. Thus, I'm thinking what would be best is to add a new Config.saves API to allow default save metadata to be provided.

I don't see any reason not to do this. I think it's a good idea, esp the callback idea. Yeah, users can do it themselves, but making it super easy to set up as a config might encourage people to do this sort of versioning through the proper channels (meaning not through a story var instead).

Hmm.  I must have glossed over the bit about it being completely broken.  I thought the save that started this was broken in the sense that it simply fails to load.  Mea culpa.

One thing is maybe users should have more control over the filename of save files? That's one angle I think maybe hasn't been discussed. Like a config we can provide a string or a callback to.

The Save API adds a datestamp and file extension to the provided filename suggestion in addition to doing some sanitation.  The basic filenames are provided by the API consumers, which are:

End users using the API manually.  They have complete control over the filename they provide themselves.
The Saves dialog.  The filenames it provides are based on the story's own name.

It wouldn't be a huge thing to add a setting to override the use of the story's name by the dialog, I'm just not sure it really adds much.  /shrug

I'm not sure it adds much either. I think needing control over file name is a very niche desire, but I also don't see any rationale for not letting users have an easy config for it. I think this would technically give @BawdyInkSlinger exactly what he wants, and its not a crazy thing to do. But I also think it's of questionable overall value to most users.


-t option used on an unknown chart

When the -t option is used to target a chart that indeed does not exist, no error is raised and the execution of Spray silently continues/ends.
It would be better to at least have a warning printed in the logs, or even fail in error

Closed by https://github.com/gemalto/helm-spray/pull/42