[DHIS2-5499] Add database tables to support media

With this PR there is also support when creating an app to attach an icon that gets saved to db, and a getMedia handler that can send out the file with correct content-type/mime

Ok, thanks for the clarifications!
On Fri, Mar 22, 2019 at 3:37 PM Erik Arenhill<EMAIL_ADDRESS>wrote:

@erikarenhill commented on this pull request.
In packages/server/migrations/20190320113929_appversionmedia.js
https://github.com/dhis2/app-store/pull/17#discussion_r268196476:

@@ -0,0 +1,62 @@

+exports.up = async (knex) => {
+
+
its an es-rule from hapijs, but we should unify this with our other
styling. I can add that as a new separate Jira task.
Storing the mime type is because we currently save the file to AWS s3 and
wanted this to be a technology neutral solution, e.g. if we build support
for storing the files in a database (or something else) we could just push
the raw bytes through and add the previously stored mime to the http header
in the response without having to check the actual data. Thats my thought.
unless we're supporting self-hosted external appstores too
this new app store is supposed to be able to run self-hosted for anyone
that wants to brand it, or contain their own sets of apps for a specific
instance.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dhis2/app-store/pull/17#discussion_r268196476, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA52sHx8XAzfeVBt8k-TfTi64w_t5xatks5vZOqvgaJpZM4cDlGn
.


How to load a model from a given path?

System Info
tgi version:0.9.0
Information

[X] Docker
[ ] The CLI directly

Tasks

[ ] An officially supported command
[ ] My own modifications

Reproduction
I just want to use tgi to run llama-7b model to get the throughput on A100.  The model files are preloaded in a given path. I followed the readme and found the following error.
Is theres any option for load model from a path? Thanks~
me@ubuntu20-02:~/zy$ docker run --gpus all --shm-size 1g -p 8080:80 -v ~/w/data:/data ghcr.io/huggingface/text-generation-inference:0.9.2 --model-id /shared/models/huggingface/llama-7B-hf/
2023-07-23T14:17:02.797888Z  INFO text_generation_launcher: Args { model_id: "/shared/models/huggingface/LLM/llama-7B-hf/", revision: None, validation_workers: 2, sharded: None, num_shard: None, quantize: None, dtype: None, trust_remote_code: false, max_concurrent_requests: 128, max_best_of: 2, max_stop_sequences: 4, max_input_length: 1024, max_total_tokens: 2048, waiting_served_ratio: 1.2, max_batch_prefill_tokens: 4096, max_batch_total_tokens: 16000, max_waiting_tokens: 20, hostname: "1401cbf60306", port: 80, shard_uds_path: "/tmp/text-generation-server", master_addr: "localhost", master_port: 29500, huggingface_hub_cache: Some("/data"), weights_cache_override: None, disable_custom_kernels: false, json_output: false, otlp_endpoint: None, cors_allow_origin: [], watermark_gamma: None, watermark_delta: None, ngrok: false, ngrok_authtoken: None, ngrok_domain: None, ngrok_username: None, ngrok_password: None, env: false }
2023-07-23T14:17:02.798147Z  INFO text_generation_launcher: Starting download process.
2023-07-23T14:17:08.906356Z ERROR text_generation_launcher: Download encountered an error: Traceback (most recent call last):

  File "/opt/conda/bin/text-generation-server", line 8, in <module>
    sys.exit(app())

  File "/opt/conda/lib/python3.9/site-packages/text_generation_server/cli.py", line 109, in download_weights
    utils.weight_files(model_id, revision, extension)

  File "/opt/conda/lib/python3.9/site-packages/text_generation_server/utils/hub.py", line 96, in weight_files
    filenames = weight_hub_files(model_id, revision, extension)

  File "/opt/conda/lib/python3.9/site-packages/text_generation_server/utils/hub.py", line 25, in weight_hub_files
    info = api.model_info(model_id, revision=revision)

  File "/opt/conda/lib/python3.9/site-packages/huggingface_hub/utils/_validators.py", line 112, in _inner_fn
    validate_repo_id(arg_value)

  File "/opt/conda/lib/python3.9/site-packages/huggingface_hub/utils/_validators.py", line 160, in validate_repo_id
    raise HFValidationError(

huggingface_hub.utils._validators.HFValidationError: Repo id must be in the form 'repo_name' or 'namespace/repo_name': '/bigdata/shared/models/huggingface/LLM/llama-7B-hf/'. Use `repo_type` argument if needed.

Error: DownloadError

Expected behavior
output the running log.

make sure/sharedis accessible from inside docker:
docker run --gpus all --shm-size 1g -p 8080:80 -v /shared:/shared -v ~/w/data:/data ghcr.io/huggingface/text-generation-inference:0.9.2 --model-id /shared/models/huggingface/llama-7B-hf/

make sure/sharedis accessible from inside docker:
docker run --gpus all --shm-size 1g -p 8080:80 -v /shared:/shared -v ~/w/data:/data ghcr.io/huggingface/text-generation-inference:0.9.2 --model-id /shared/models/huggingface/llama-7B-hf/

Thanks. tgi demo could work after make /shared accessible from inside docker.


[Event Change Request] codeunit 226 "CustEntry-Apply Posted Entries" - OnBeforeUnApplyCustomer

Hello, can you please Add a IsHandled variable in Event OnBeforeUnApplyCustomer?
    local procedure UnApplyCustomer(DtldCustLedgEntry: Record "Detailed Cust. Ledg. Entry")
    var
        UnapplyCustEntries: Page "Unapply Customer Entries";
        IsHandled;
    begin
        IsHandled := false;
        OnBeforeUnApplyCustomer(DtldCustLedgEntry, IsHandled);
        If IsHandled then
             exit;

        DtldCustLedgEntry.TestField("Entry Type", DtldCustLedgEntry."Entry Type"::Application);
        DtldCustLedgEntry.TestField(Unapplied, false);
        UnapplyCustEntries.SetDtldCustLedgEntry(DtldCustLedgEntry."Entry No.");
        UnapplyCustEntries.LookupMode(true);
        UnapplyCustEntries.RunModal();

        OnAfterUnApplyCustomer(DtldCustLedgEntry);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeUnApplyCustomer(DtldCustLedgEntry: Record "Detailed Cust. Ledg. Entry"; var IsHandled: Boolean);
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 13192.


zkApp doesn't enforce the validator address

Currently, we only check the epoch and index and not the validator address in the zkApp. As there are only 8 slots, and if we want a payout address to send the validator fees to, that would mean removing the fee state variable:

Epoch
Index
Oracle key (takes 2)
Validator key (takes 2)
Validator Payout (takes 2)

We can set the fee to a constant in the zkApp - the only downside is we can't then view this information in the state.

Fixed by #27


fix destination.user match in rbac

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ x ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
ServiceRole:
apiVersion: "rbac.istio.io/v1alpha1"
kind: ServiceRole
metadata:
  name: http-viewer
  namespace: default
spec:
  rules:
  - services: ["asm-test-http.*"]
    methods: ["GET", "HEAD"]
    constraints:
    - key: destination.user
      values: ["default1"]

Pod ServiceAccount:
➜  istio-rbac kubectl get pod asm-test-http-79965fc4f9-jfjhz -o yaml|grep Account
  serviceAccount: default
  serviceAccountName: default

in my understanding, the above config ServiceRole http-viewer will not be applied to the service-side pod asm-test-http, because of the destination.user in ServiceRole constraints dose not match the serviceAccount of pod, and pod asm-test-http will have no rules in rbac filter.
but the Listener in asm-test-http still has rbac rules
{
    "name": "envoy.filters.http.rbac",
    "typed_config": {
        "@type": "type.googleapis.com/envoy.config.filter.http.rbac.v2.RBAC",
        "rules": {
            "policies": {
                "http-viewer": {
                    "permissions": [
                        {
                            "and_rules": {
                                "rules": [
                                    {
                                        "or_rules": {
                                            "rules": [
                                                {
                                                    "header": {
                                                        "name": ":method",
                                                        "exact_match": "GET"
                                                    }
                                                },
                                                {
                                                    "header": {
                                                        "name": ":method",
                                                        "exact_match": "HEAD"
                                                    }
                                                }
                                            ]
                                        }
                                    }
                                ]
                            }
                        }
                    ],

other client-side pod that are binding with the ServiceRolehttp-viewer ServiceRole are still access the service asm-test-http

@dzdx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
65f554056944301f050b3db45f5aed32bab289d7
link
/test lint

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.

/test lint

@dzdx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
65f554056944301f050b3db45f5aed32bab289d7
link
/test lint

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.

@dzdx: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
65f554056944301f050b3db45f5aed32bab289d7
link
/test lint

racetest_istio
65f554056944301f050b3db45f5aed32bab289d7
link
/test racetest

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.

/retest

@dzdx: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

racetest_istio
65f554056944301f050b3db45f5aed32bab289d7
link
/test racetest

lint_istio
65f554056944301f050b3db45f5aed32bab289d7
link
/test lint

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.

/retest

What is the protocol for backporting fixes like this to prior stable releases? Though a small bug, this makes it very hard to use istio in places that depend on destination.user rather than the service itself, i.e. when writing SMI config.

What is the protocol for backporting fixes like this to prior stable releases? Though a small bug, this makes it very hard to use istio in places that depend on destination.user rather than the service itself, i.e. when writing SMI config.

We could cherry-pick it to previous versions as it's a bug fix. But here I'm afraid the bigger problem is that Istio 1.4 introduces the new v1beta1 authorization policy that deprecates the v1alpha1 RBAC policy and the destination.user is no longer supported in the v1beta1 policy. That means if you're using the destination.user to decide where to apply the policy, you have to migrate it to the v1beta1 policy which only supports to use workload selector to decide where to apply the policy.

I see, thanks for clarifying. Migrating to the workload selector isn't straightforward for the SMI istio adapter because the SMI spec currently only uses identity to determine the destination for policy. We are discussing changes to the spec to address this here: https://github.com/deislabs/smi-spec/issues/82.


JDK version determined incorrectly by Gradle on JDK 10

https://github.com/policeman-tools/forbidden-apis/wiki/BundledSignatures
"Gradle automatically add the compile Java version"
forbiddenApis {
  bundledSignatures = ['jdk-internal']
}

Error:
 Parsing signatures failed: Invalid bundled signature reference (JDK version is invalid): jdk-internal-1.10
Workaround:
forbiddenApis {
  bundledSignatures = ['jdk-internal-10']
}

org.gradle.api.JavaVersion

    // Since Java 9, version should be X instead of 1.X
    // However, to keep backward compatibility, we change from 11

Possible fix:
def javaVersion = JavaVersion.current()

def fixed = javaVersion.java9Compatible ? javaVersion.majorVersion : javaVersion

This is not a bug. The version should anyways be defined in the project setup, checking against the current version is wrong anyways.

https://docs.gradle.org/current/userguide/java_plugin.html#other_convention_properties
Both sourceCompatibility and targetCompatibility are of type org.gradle.api.JavaVersion.
JavaVersion.VERSION_1_10 => jdk-internal-1.10 => does not work
JavaVersion.valueOf("1.10") => jdk-internal-1.10 => does not work
JavaVersion.valueOf("10") => No enum constant JavaVersion.10 => does not work
JavaVersion.toVersion("1.10") => jdk-internal-1.10 => does not work
JavaVersion.toVersion("10") => No enum constant JavaVersion.10 => does not work

Actually this is a serious bug in Gradle. The forbiddenapis backend solely uses official version numbers, as accepted by javac. Javac does not accept "1.[9|10|11]" as version for the new -release parameter, so i have no idea, how the compile task handles that internally. Forbiddenapis enforces full version numbers for Java versions >=9, for previous versions it accepts both variants ("1.8" and "8"), but normalizes them (to "1.8").
Nevertheless, the targetCompatibility property of the forbiddenapis task is of type String (for good reason), so you can simply specify it as a string in the task config, no need to rely on shitty Gradle enum constants.
I am not sure If I should add a hack for this because the forbiddenapis main task is about correctness, so I'd like to enforce correctness - also in Gradle. It works correctly in Maven and Ant, only Gradle is broken (as always).

Neither setting sourceCompatibility nor targetCompatibility is not recommended but allowed.

For build reproducibility it is required to be set. Otherwise the project build depends on the actually used Java version and produces artifacts solely working with this Java version. This may also cause compile errors, as the source code parsing uses an undefined Java version.

Sadly even setting source/target doesn't always help (for example, covariants on ByteBuffer.flip() can cause incompatibilities if compiled under java 9 with 1.8 as the target).

Actually this is a serious bug in Gradle

https://github.com/gradle/gradle/blob/master/subprojects/base-services/src/main/java/org/gradle/api/JavaVersion.java#L46

For build reproducibility it is required to be set.

I know but some people like pain 😳

This line should be modified:
https://github.com/policeman-tools/forbidden-apis/blob/master/src/main/resources/de/thetaphi/forbiddenapis/gradle/plugin-init.groovy#L63

@sdavids I can add a fix there, but I would do it only for the 2 enum entries (Java 9 and Java 10). Starting with Java 11, Gradle behaves correct. So it would apply the setting as-is, but for Java 9 and Java 10 it would set it to "9" and "10" manually.

My fix would be (just workaround the 2 bogus versions):
// Define our tasks (one for each SourceSet):
project.sourceSets.all{ sourceSet ->
  def getSourceSetClassesDirs = { sourceSet.output.hasProperty('classesDirs') ? sourceSet.output.classesDirs : project.files(sourceSet.output.classesDir) }
  project.tasks.create(sourceSet.getTaskName(FORBIDDEN_APIS_TASK_NAME, null), CheckForbiddenApis.class) { task ->
    description = "Runs forbidden-apis checks on '${sourceSet.name}' classes.";
    conventionMapping.with{
      extensionProps.each{ key ->
        map(key, { extension[key] });
      }
      classesDirs = { getSourceSetClassesDirs() }
      classpath = { sourceSet.compileClasspath }
      targetCompatibility = { EnumSet.of(JavaVersion.VERSION_1_9,JavaVersion.VERSION_1_10).contains(project.targetCompatibility) ? project.targetCompatibility.majorVersion : project.targetCompatibility?.toString() }
    }
    // add dependency to compile task after evaluation, if the classesDirs collection has overlaps with our SourceSet:
    project.afterEvaluate{
      def sourceSetDirs = getSourceSetClassesDirs().files;
      if (classesDirs.any{ sourceSetDirs.contains(it) }) {
        task.dependsOn(sourceSet.output);
      }
    }
    forbiddenTask.dependsOn(task);
  }
}

Actually the fix is much easier:
      targetCompatibility = { project.targetCompatibility?.majorVersion }

This works, because Forbiddenapis always accepts the pure Major version (6,7,8,9,10,...) (like Java does, too).  I thinks that fix is easiest.

The previous suggestion failed with older Gradle versions, as the enum constants were missing!
Any other suggestions?

I checked the old Gradle docs. Minimum Gradle version of forbiddenapis is 2.3. This one has (https://docs.gradle.org/2.3/javadoc/org/gradle/api/JavaVersion.html) the Major Version property, so we can use it. This is also tested on Jenkins. I changed the check now to use the major version for everything below Java 11. Starting from Java 11 (where gradle is correct), it uses toString(). The latter is done to be future-proof (if JDK decides to add minor versions again,...).

      targetCompatibility = { (project.targetCompatibility?.hasProperty('java11Compatible') && project.targetCompatibility?.java11Compatible) ?
        project.targetCompatibility.toString() : project.targetCompatibility?.majorVersion }

I will release a new version anyways for compatibility with Java 11, so this fix should be out soon.


perspective_gaussian_kernel

Excuse me , can you upload perspective_gaussian_kernel.py  ,please?

can you upload perspective_gaussian_kernel.py?

Please update perspective_gaussian_kernel.py


Add Google Colab Link

Add Google Colab Link

Feature Request Description

We were thinking of adding a link for Google Colab. This would link to a notebook that help users walk through installing and using Featuretools. Colab comes with an older version of Featuretools pre-installed, and this notebook would try to prevent some errors users run into.
Is this something that our users would be interested in?

If so, feel free to comment below or leave a  :thumbsup:

While this is something we're interested in, closing due to inactivity.


Documentation: ⚠ API Documentation is no longer working

Problem

The Wiki links to the API Documentation
This link no longer works

Next steps
@adamcooke: Are we able to bring this back online? https://github.com/postalhq/postal-api

Simple fix... the link just needs to be updated to: https://postalhq.github.io/postal-api/

Fixed


Fix TSig check on second DNS message (TCP) #180

The next chunk needs to use the previous MAC
Using this fix, I can successfully verify the signatures of not
only the first but the subsequent envelopes as well.
Patch was proposed by andrewtj in a comment. Kudos !

Awesome! :-)


Lexicographic rank of a string

Description
Given a string, find its rank among all its permutations sorted lexicographically.
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Docker service logs

Feature idea
Docker service is missing logs feature docker-cloud has.
Steps to reproduce the issue:
docker service logs ...

This would show logs, same way as docker-cloud does.

Duplicate of https://github.com/docker/docker/issues/24812


Move issues from one repository to another on the same organization

Sometimes we need split one repository into multiple, but the issues can not be split. If this feature has been implemented, it's very useful. This operation could ask admin permission of organization.

Has this been addressed, or is there a workaround? It is a pain to copy an issue and all its comments and uploads. In our case, it is sometimes because of a simple mistake (posting on another repo by mistake, then needing to move the issue to the right repo)...

@WAMH no people are working on this.

Yes, I would love this feature as well. In a big project, it is not uncommon for people to create the issue on the wrong repo and then need to move it later. Especially if the people creating the issue are not super familiar with the code base.

People creating issues in the wrong repo is a recurring issue in my organisation, this feature would be really great.

We have a triage project where we work on misc issues.
When issue grows to own project, it would be great to be able to move issues.

duplicated as #453


agent/informant: Update lastSuccessfulInformantComm for monitor comms

This should fix false positives for the "stuck VMs" alert on staging. Validated the old & new (fixed) behavior locally.

Going to merge without waiting fore CI because it previously passed and the newest commit is just a comment.


Fix for: BHV-5137

Task
Add a renderOnShow option to enyo.Control as a base API that allows individual controls (and their children) to be created but not rendered until explicitly set via the showing property.
Enyo-DCO-1.1-Signed-Off-By: Cole Davis<EMAIL_ADDRESS>

@clinuz  and @aarontam  - should this PR be closed out?

Hi @lishammel, yep I believe we can close this for now.


install failure on LinuxMint/Ubuntu

Hi Dan,
Many thanks for great tool - localturk.
Trying to install it on LinuxMint v18.2 (Sonya) with "sudo npm install localturk -g" I'm getting following error:
david@DP55L ~/OCR/localturk $ sudo npm install localturk -g
npm ERR! path /usr/local/lib/node_modules/localturk/dist/localturk.js
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod
npm ERR! enoent ENOENT: no such file or directory, chmod '/usr/local/lib/node_modules/localturk/dist/localturk.js'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! A complete log of this run can be found in:

And log shows:
...
1698 verbose stack Error: ENOENT: no such file or directory, chmod '/usr/local/lib/node_modules/localturk/dist/localturk.js'
1699 verbose cwd /home/user/
1700 verbose Linux 4.10.0-40-generic
1701 verbose argv "/usr/bin/node" "/usr/local/bin/npm" "install" "localturk" "-g"
1702 verbose node v9.2.1
1703 verbose npm  v5.6.0
1704 error path /usr/local/lib/node_modules/localturk/dist/localturk.js
1705 error code ENOENT
1706 error errno -2
1707 error syscall chmod
1708 error enoent ENOENT: no such file or directory, chmod '/usr/local/lib/node_modules/localturk/dist/localturk.js'
1709 error enoent This is related to npm not being able to find a file.
1710 verbose exit [ -2, true ]

I've tried creating localturk under node_modules folder without luck. Tried that but trust it should be automatically created.

ok, found issue.
When running npm install localturk -g one has to be in the folder within which "localturk" and below folders as synced from git are.

You should be able to install localturk from anywhere, so this is definitely a bug. Thanks for the heads up!

This should be fixed with the 2.0.3 release. If you can verify by installing from outside the localturk directory, that'd be much appreciated.
Thanks again for the heads up. (Also curious how you discovered localturk!)

How did I find localturk?
Working on small private project around OCR & training, came across your great article:
http://www.danvk.org/2015/01/11/training-an-ocropus-ocr-model.html
I find locaturk written in very smart way - like it very much.
In fact if I can, will submit a small RFE.
Trying to re-install localturk (workedaround by removing localturk and classify-images from /usr/local/bin/

/usr/local/bin/localturk -> /usr/local/lib/node_modules/localturk/dist/localturk.js
npm ERR! path /usr/local/bin/classify-images
npm ERR! code EEXIST
npm ERR! Refusing to delete /usr/local/bin/classify-images: ../lib/node_modules/localturk/classify-images.js symlink target is not controlled by npm /usr/local/bin
npm ERR! File exists: /usr/local/bin/classify-images
npm ERR! Move it away, and try again.

Other than that worked like a charm below is run from within /usr/local/bin
/usr/local/bin $ sudo npm install -g localturk
/usr/local/bin/localturk -> /usr/local/lib/node_modules/localturk/dist/localturk.js
/usr/local/bin/classify-images -> /usr/local/lib/node_modules/localturk/dist/classify-images.js

<EMAIL_ADDRESS>added 64 packages in 5.889s

Dear All,
Thanks for your help!
Hope you dont mind this quick question before I dive into this repo:
Have you try running localturk on a LAN?
I discovered it via Google.
I was looking for simple way to crowd sourcing training set collection.
The following are my google search keyword leading me here : (reversed chronological order)
&q=mechanical+turk+implementation+github&
&q=mechanical+turk+versus%2C+DIY+web+server&
&q=mechanical+turk+versus%2C+implement+web+server&
&q=mechanical+turk+like%2C+implement+web+server&
&q=mechanical+turk%2C+implement+web+server&
?q=mechanical+turk&
&q=crowdsourcing+implementation%2C+web+server+to+label+data%2C+stackoverflow&
&q=crowdsourcing+implementation+suggestion%2C+web+server+to+label+data%2C+stackoverflow&
&q=web+server+to+label+data%2C+stackoverflow&
?q=php+to+take+survey&
BR,
Jimmy

Dear All,
Thanks for your help!
After updating nodejs version(ps.1), I found it works on LAN without needing any modification, config, etc.
And then I encountered my first problem in localturk:
Error: Error: Number of columns is inconsistent on line 2
at /usr/local/lib/node_modules/localturk/dist/csv.js:117:31
at step (/usr/local/lib/node_modules/localturk/dist/csv.js:32:23)
at Object.next (/usr/local/lib/node_modules/localturk/dist/csv.js:13:53)
at resume (/usr/local/lib/node_modules/localturk/dist/csv.js:43:44)
at fulfill (/usr/local/lib/node_modules/localturk/dist/csv.js:45:31)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
BR,
Jimmy
ps.1,
/usr/local/lib/node_modules/localturk/node_modules/csv-parse/lib/index.js:114
Parser = function(options = {}) {
^
SyntaxError: Unexpected token =
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:374:25)
at Object.Module._extensions..js (module.js:417:10)
at Module.load (module.js:344:32)
at Function.Module._load (module.js:301:12)
at Module.require (module.js:354:17)
at require (internal/module.js:12:17)
at Object. (/usr/local/lib/node_modules/localturk/dist/csv.js:55:16)
at Module._compile (module.js:410:26)
at Object.Module._extensions..js (module.js:417:10)

@speculaas This issue arises in case your csv file contains commas and the parsing of tasks.csv fails.


UI improvements

replaced count-numbers in braces or brackets in Data and Result Browser by badges at the end.

the number of found results in Data Browser is now formatted

replaced the whitespace at the beginning of each line of the search results in Data Browser by a margin. This has the effect that selecting a file name by double click will no longer select the unnecessary whitespace at the beginning

Stretching the facet count inside the dropdown was not possible in a pretty way as line break can appear there and then it looks completely bad

I do not get very well the last point

replaced the whitespace at the beginning of each line of the search results in Data Browser by a margin. This has the effect that selecting a file name by double click will no longer select the unnecessary whitespace at the beginning

as I am double click selecting one file and it pastes without whitespace, but maybe I am wrong.
LGTM

Maybe your computer automatically trims the stuff in the clipboard, but it does not for me. Here ist the result of double clicking and pasting in the current xces. It's not a big deal, but I thought this might be annoying to some people

ah, I see. Thanks
sorry, I forgot that here I had to explictly accept the changes, done.


fix(unit-tests): Added a test case for ErrDatabaseError (AEROGEAR-8568)

Motivation
https://issues.jboss.org/browse/AEROGEAR-8568
What
Added test case to existing test table to cover models.ErrDatabaseError.
Why
Test coverage decreased because this new error model was added to a switch statement but the test table for the switch was no updated.
Verification Steps

Run go test ./pkg/httperrors -cover
Read the output. Is code coverage at 96.3%?

ok      github.com/aerogear/mobile-security-service/pkg/httperrors      (cached)        coverage: 96.3% of statements

Checklist:

[x] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Progress

[x] Finished task

Coverage increased (+0.5%) to 66.844% when pulling efb4d3c2afebad16181f3a862acce1ab7660d9a0 on craicoverflow:AEROGEAR-8568 into 4b4c20c21e393e34883772d2580d204c77acdba8 on aerogear:master.

Coverage increased (+0.5%) to 66.844% when pulling a3485eb2f27b91f353fce03c3edf1de3254e16d5 on craicoverflow:AEROGEAR-8568 into 4b4c20c21e393e34883772d2580d204c77acdba8 on aerogear:master.


fix text size for screen resolution

Not sure about this one. 1920 and GetSystemMetrics(SM_CXSCREEN) would more or less just output the exact same result no matter the actual game resolution size. It's just that one method is way less expensive to use over the other since this function will be called hundreds of thousands of times.
Using 1920 would still scale properly I believe. I just tested this in-game and they both do the same thing.

reason for the change was the text size seemed alot larger on my 2560p monitor once the that change was made it seemed to look appropriately correct, otherwise if was left at 1920 i would have to decrease the actual text size so current text size was 22 so i would have to make it 20 for it look okay

with all the changes i applied this is how it looks on my monitor

Is that what it looks like WITH your changes or WITHOUT your changes? Because the sizing for everything seems way off

thats with my changes

this is if i left 1920 at the end instead of GetSystemMetrics

this another screen commented out the SetResolution but left GetSystemMetrics at the end of the line

The first picture is better, but the size for the menu itself is way off in all pictures. This is what it's supposed to look like

yeah if i changed my resolution in game to 1920x1080 it does look exactly like that

so seems the menu shrinks by having higher resolution

Should be improved with the latest update.


eslint minified files

Does Eslint works on minified Java script files? If yes then does it need any extra functionality to be added. Currently my ESLINT is not scanning any minified files.

Does eslint executed from command line scan these files?
If yes, can you please submit a small project which replicates this?


Request: Option for custom input properties, e.g. ui-keypress

Hi I want to add my own input field properties, e.g.
<input ui-keypress="{enter: 'myCallBack()'}">

This way I can ensure the database gets hit, when a user hits enter and not on "selectedObject".

You can probably use 'input-changed' function.


feat(initial commit of plugin code structure): storing roject structure

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Failing to parse timestamp values without milliseconds

There is a bug in the parsing of timestamp values without milliseconds while using select
E.g. the value [2016-02-03T08:16:26.389] is successfully parsed, but the next value [2016-02-03T08:19] fails:
Failed with exception java.io.IOException:org.elasticsearch.hadoop.rest.EsHadoopParsingException: Cannot parse value [2016-02-03T08:19] for field [receivedTimestamp]
The error is consistent across different timestamp fields.
Using elasticsearch-hadoop-hive-2.2.0.jar

That's because there is no mapping defined in ES and thus, the first value is used to determine the time. Since the second value doesn't match exactly the first value, ES cannot properly map it and hence the exception.
To work-around this, one would define the mapping before hand as explained in the docs.

As there hasn't been any update, closing the issue.

I have same trouble and i don't understand why . There is something that i don't master at all about ES working

'es.mapping.date.rich'='false'

For a more concrete example.
#assuming you have a string field called customer and a date field called createdAt in elasticsearch

hive> DROP TABLE IF EXISTS my_table;

hive> CREATE EXTERNAL TABLE my_table (id string, customer string, createdat string)
STORED BY 'org.elasticsearch.hadoop.hive.EsStorageHandler'
TBLPROPERTIES (
    'es.resource' = 'my_table_es_index',
    'es.nodes' = 'es_url:9200',
    'es.read.metadata' = 'true',
    'es.mapping.names' = 'id:_metadata._id,customer:customer,createdat:createdAt',
    'es.mapping.date.rich' = 'false'
);

#print date as string
select id, createdat, customer from my_table limit 10;

#convert the string described above to long (this type of logic can better reside inside a UDF
select id, unix_timestamp(createdat, "yyyy-MM-dd'T'HH:mm:ss"), customer from my_table limit 10;


598 feed plugin image support

Description

Add image support to feed plugin.
Related Issues

Addresses #598.
Check List

[ ] Have you read the
CODE OF CONDUCT
[x] Have you read the document
CONTRIBUTING

[x] One pull request per feature. If you want to do more than one thing,
send multiple pull request.
[x] Write tests.
[x] Run deno fmt to fix the code format before commit.
[x] Document any change in the CHANGELOG.md.

Thanks. The tests are failing due the new image file in the assets folder. Can you run deno task test tests/feed.test.ts -- --update to update it?

Thanks. The tests are failing due the new image file in the assets folder. Can you run deno task test tests/feed.test.ts -- --update to update it?

It seems that my change to test site also impacted snapshot of other tests. I've updated snapshot of those test in a separate commit. I can't run test suite locally, favicon test seems to never end so some failing test may remain. Let me know if that's the case.

The favicon issue is a bug from Deno in its support for NPM packages (https://github.com/denoland/deno/issues/21686)
I'm using Deno 1.38.5 to run the tests because it's the most recent version without this bug. You can run deno upgrade --version 1.38.5 to update the tests. Then back to the latest Deno version with deno upgrade.

Should be ok now.

Thank you!


User with admin role should not delete himself.

User with admin role should not delete himself.

API updated by #657 .

Fix #660


Add opt lists

Quick addition to add support for displaying pick lists
Fixed up a broken unit test while i was at it

Awesome, thanks!

no; thanks for the repo!  Very useful as a "super cheap" UI :-)


WIP: Test github.com/mtrmac/image:mirror-reference

Just to run tests for https://github.com/containers/image/pull/912 .

The c/image PR was merged, so this is no longer necessary.


chore: promote test15 to version 0.0.17

chore: promote test15 to version 0.0.17
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign q1323829945
You can assign the PR to them by writing /assign @q1323829945 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@q1323829945: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
23fe5d2f267a467f039d019f13ebb9dee2ebf185
link
/test verify

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 jenkins-x/lighthouse repository. I understand the commands that are listed here.


Workspaces doesn't differentiate between Opera GX and Opera

Microsoft PowerToys version
0.85.1
Installation method
Microsoft Store
Running as admin
No
Area(s) with issue?
Workspaces
Steps to reproduce
While trying to capture a Workspace when both, regular Opera browser and Opera GX browser are running, Workspaces doesn't differentiate between them and captures both as Opera GX

✔ Expected Behavior
Workspaces capture Opera GX and Opera as separate programs.
❌ Actual Behavior
Workspaces capture Opera GX and Opera as one instance of Opera GX.
Other Software
No response

I have similar issues with Visual Studio 2019 & 2022
Used VS2022 to  CREATE the workspace, VS2019 opens when  USING the workspace

Yeah, Workspaces are particularly buggy for me, they have a new bug each release. Before that, I was using a simple bat file to run programs I wanted for a particular workflow, and now I am starting thinking that this simple solution might be better than Workspaces after all 😉

I have similar issues with Visual Studio 2019 & 2022 Used VS2022 to CREATE the workspace, VS2019 opens when USING the workspace

I am experiencing the same issue with VS 2022 and VS 2019, so I created an issue for it: https://github.com/microsoft/PowerToys/issues/35554


Create USAGE

It will output description of this generator, when you run $ yo bespoke --help like generator-generator's USAGE.

Hi @sotayamashita.
I like this idea :smile:. I used your idea in the next version I'm working on. You can see the roadmap here : #36. That's why I'm closing this issue.
Feel free to review and comment what I've done in branch v3 and especially on the USAGE commit.


Add proxy for cross-orgin AJAX calls

No dice:
XMLHttpRequest 'https://www.google.com/jsapi'
because it results in:
"XMLHttpRequest cannot load https://www.google.com/jsapi. Origin http://localhost:8000 is not allowed by Access-Control-Allow-Origin."
instead, add server-side support for:
'/proxy?url=https://www.google.com/jsapi'

On second thought, I don't like the security implications of this plan.  It would allow folks to point browsers to:
icontrib.org/proxy?url=http://AmIEvil.com/Yes/YesIAm.html
Instead, can you add?
/mirror/google/jsapi

agreed
On Jan 4, 2012, at 9:48 AM, Greg Fitzgerald wrote:

On second thought, I don't like the security implications of this plan.  It would allow folks to point browsers to:
icontrib.org/proxy?url=http://AmIEvil.com/Yes/YesIAm.html
Instead, can you add?
/mirror/google/jsapi

Reply to this email directly or view it on GitHub:
https://github.com/garious/icontrib/issues/16#issuecomment-3358061


EE Version Of Meltano

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/158
Originally created by @jschatz1 on 2018-11-08 13:27:01

Enterprise features will be needed. How should be separate the CE code from the EE code of Meltano? How do we avoid the current difficult merging situation that GitLab has with merging CE and EE together?
cc @mbergeron @iroussos

View 7 previous comments from the original issue on GitLab


Added AmiiboAPI per issue #1429

https://www.amiiboapi.com/
issue #1429

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side <<----- THIS DID NOT MATCH THE CONSISTENCY OF THE OTHER COLUMNS
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

See #1440


Error pod

When i added to podfile and run pod install, have error:
Unable to find a specification for SwiftyContacts

please update your podspex repo using gem pod update.


Add IBMPowerVSCluster to status subresource set

What this PR does / why we need it:
Add IBMPowerVSCluster to status sub resource set since IBMPowerVSCluster has also status sub resource.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/cc @mkumatag

/assign @alvaroaleman


Test classes fix

Fixed issues when specifying which test classes to use when running instrumentation.
Had to add PowerMock to suppress the part of the integration test where it tries to talk to a device.

Looks good at first glance. Will merge locally and play with it before I pull it in. Should be in a couple of days latest.

The fix for one of the issues in https://github.com/jayway/maven-android-plugin/pull/93 is more cleaner (I assumed that the collections were missing generics for a reason, so I never went that route).

I pulled in the other fix. I will close this one.. if there is anything else you want to contribute please create a new pull request. Thanks for you work.


Live SDK Deprecated

Live SDK Deprecated but the code sample under Client Flow login with Azure AD still uses it. should be updated to graph API
[Enter feedback here]

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

ID: 664f306a-a900-e59a-356c-a446bafe98f7
Version Independent ID: e8ecb2a6-8056-dfd9-be49-646bcbbc7cef
Content: How to Use Apache Cordova Plugin for Azure Mobile Apps
Content Source: articles/app-service-mobile/app-service-mobile-cordova-how-to-use-client-library.md
Service: app-service-mobile
GitHub Login: @conceptdev
Microsoft Alias: crdun

Thanks for your feedback @Ahmad-Al-Ali - the page will be updated to remove the references to deprecated SDKs.

@Ahmad-Al-Ali We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Upgrade access for OQS project non--core-team members

Give @geedo0 write access to https://github.com/open-quantum-safe/openssh. Gerardo is actively working on bringing the openssh project up to date.
Give @geedo0 triage access across OQS projects.
Give @jplomas read access to liboqs, and hence the ToB project board. JP is a regular attendee on liboqs status calls.

It would be better to add a team with read access to the repo, and then add jplomas

Tagging @baentsch @christianpaquin for review and merge as I don't have sufficient perms in this repo.

@baentsch In addition to the changes in the original comment, I've now added an "oqs-provider-codeowners" team which has write access on oqs-provider and contains yourself, @bhess, @feventura, @iyanmv, and @thb-sb as per https://github.com/open-quantum-safe/oqs-provider/pull/458.

Don't understand the difference between codeowners and committers but what's one more thing I don't understand..:-/

I opted to make a new team since https://github.com/open-quantum-safe/oqs-provider/pull/458 didn't add @feventura and @iyanmv to the list of committers in GOVERNANCE.md, and I didn't want to overstep that file. Happy to consolidate the two lists if that's preferable.

Ahh--now I understand. Thanks @SWilson4 . Then I'm indeed fine with this PR as-is (and hope the corresponding GH permissions it brings makes the CODEOWNERS file become valid) .

Hmm -- this didn't go to plan I'd guess, @SWilson4 @ryjones : 1) See error messages above and 2) this screenshot
 of https://github.com/open-quantum-safe/oqs-provider/pull/458

Perhaps you can give crockeea's seat to me instead?

I upgraded to 100 seats

@geedo0 you now have an invite, and everyone else is also added.


added support for a comma-separated list of crl

added support for a comma-separated list of crl

Could you resubmit please against the trunk branch rather than 2_0_x-fixes? I'll take care of backmerging to 2.1 + 2.0. Also please make sure there are no whitespace changes, your patch changed the indentation in a few places.

ok should be done. I have avoid any reformat code to keep indentation. Hope I didn't mess with the pull request...


Add Arc MIAA username configuration

Fixes https://github.com/microsoft/azuredatastudio/issues/12080
With validation to not allow sa username

Pull Request Test Coverage Report for Build 260273763

0 of 13 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 38.354%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/arc/src/ui/tree/controllerTreeNode.ts
0
2
0.0%

extensions/arc/src/models/miaaModel.ts
0
11
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/arc/src/models/miaaModel.ts
2
5.77%

Totals

Change from base Build 260263215:
-0.01%

Covered Lines:
19128

Relevant Lines:
45712

💛  - Coveralls


[Question] Is there a way to customize property threshold for new lines?

I saw in another issue that it was maybe possible to adjust the property threshold before lines get added. Just curious if there's a way to adjust that threshold.
Thanks!

Hi, currently there isn't a way to optionally set the threshold without forking the whole project. I'd like to keep the options at their reasonable defaults. Besides, as far as I know there isn't a clean way to make this since it's a JSON config and you can't really accept user input. It needs to be like below and I don't like the look of it.
{
  "extends": ["stylelint-config-clean-order", "stylelint-config-clean-order/minimal-threshold"]
}

There is a way using a JS stylelint config: stylelint.config.js. Also not ideal but it works.
const cleanOrderConfig = require('stylelint-config-clean-order');

// Overwrite threshold
cleanOrderConfig.rules['order/properties-order'][1]["emptyLineMinimumPropertyThreshold"] = 5;

module.exports = cleanOrderConfig;

I'm closing this issue as there are no current plans to implement the requested feature. Thanks!


All workflows

@colebemis commented: "I think we can remove all usages of this icon, so we don't need to design a v2 icon for it. @ashygee knows more about this"
Would we make the newer version of this icon or find a substitution?

This is not used in dotcom anymore


Return [Error?, TData?] instead of { error?: Error, data?: TData } ?

Should the Web3ApiClient invoke method return [Error?, TData?] instead of { error?: Error, data?: TData }?

Would it work on other clients implemented in different languages which aren't so lenient when it comes to type?

That's a great point @Niraj-Kamdar, no it would not. In my opinion I'm very "on the fence" about this change, and feel that the down-side you've brought to light is enough to make me think this is not a great idea.


Think about the cmp function

We've added a cmp function that can be used to compare the resolved variant version against a versionspec.
Example uses:
cmp(python, "3.7")
cmp(python, ">3")
cmp(python, "<=3.1")

This can then be used for example in an if statement:
if: cmp(python, ">3.7")
then:
- attrs

The nice thing about this is that it's a generalization of something that existed in conda (selectors like py2k, py>3, etc). and works with any variant (e.g. ruby, openssl, ...).
However, when had this in the CEP, people where not sure about the name & semantics, and we removed it from the initial CEP.
So now we're here and looking for more feedback.

Suggestions from @baszalmstra:

use matches(python, "3.7") instead of cmp because it's more clear that it's a matchspec / versionspec
instead of a implicit variable like python, use explicitly variant["python"], so that it becomes something like matches(variant["python"], "3.7").
Or we could make it similar to the env object that we recently introduced and make it read matches(variant.python, "3.7") or matches(variant.get("python"), "3.7")

I would prefer the last item ofc. I think it has some nice benefits:

We only have to teach the user about some top-level objects (variant, env, more?).
The variable python or libfoobar is not some magical value.
We can easily deal with package names that are not valid variable names (e.g. py-rattler is not a valid variable name, but variant.get("py-rattler") just works).
It's easy to provide contextual error messages (e.g. variant.bla, bla is not a valid variant, read more here and here and here).

For now, we have fixed these and match is already being used by a bunch of recipes in conda-forge! :)


flutter sdk release notes added

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: \ flutter sdk release notes added
Jira/GitHub Issue numbers (if any): \ FFM-11667
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[X] Successful preview build.
[X] Code owner review.
[X] No merge conflicts.
[X] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66700028e04b891bb13b8cb6--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6670003c41e4c31c86f812bc--harness-developer.netlify.app


algoPractice1

Pushed my local branch algoPractice1 to the remote repo.

added my local branch algoPractice1 to the remote repo. the changes in the algoPractice1 branch have already been merged into the main branch.


Add support for exporting to WebM

Fixes #86

@sindresorhus just did a quick review (3AM right now), and it looks awesome 😌
Will checkout tomorrow to test, but looks great to me 💯

@matheuss Should be ready now.

Thank you @sindresorhus ❤


Formatting pre and post cast comments enhancements

Enhancements for formatting pre and post cast comments: /* pre */ as /* post */.
The enhancements are based on discussions and issues examples in #4391.
Test cases were added in issueCastComments.rs.
Changes assume that pre/post cast comments should be trimmed, i.e. that comments' original leading and trailing whitespaces should be removed.
Changes seem to fix issue #3528 and issues of leaving trailing spaces which may be related to #2896.  Test cases for these issues were also included .

Thanks for the PR! Just as a heads up It'll probably be a few days before I have the bandwidth to go through this, need to deal with the broken toolstate issue on nightly first.

Inline suggestions below for moving the infix suffix/prefix spacing logic into this function as mentioned in the other inline comment in expr.rs. Written somewhat hastily inline in the GitHub Web UI (always dangerous 😆) so may not be perfect, but should be enough to express an idea how to implement the needed change

I have no comments to the suggested changes (look quite final to me ...).  I will update and test the PR accordingly.

Thanks for the updates, LGTM!

Heads up: filed a regression that bisects to this PR. Would love if one of you could have a look. #4534
  fn main() {
-     let z = (x as f64 / y as f64).floor() as usize;
+     let z = (x as f64 / y as f64).floor() as f64 / y as f64).floor()
+         as f64 / y as f64).floor() as usize;
  }


Translations to Simplified Chinese

A *.json file couldn't be uploaded here so I changed its extension to '.txt'.
zh_cn.txt
You may describe it like this: 'zh_cn - 简体中文' .

And try crwd.in! It's free for open source projects.

Great, thank you! I'll work on getting a crwd.in open as well.

I tested out crwd.in and while it looks nice and performs well it seems like a lot of the features that it comes with are premium features. After comparing a few different websites, I went with OneSky. Link for it can be found Here.


Height in meters?

I like the new layout. Since you are now displaying a hint in the lower right corner, would it be possible to display the altitude there in meters instead of hectopascals?

Yes, please!
Ideally a choice of height in km (for laymen), ft (for aeronauts) or hPa (for physicists).
Many thanks in advance!


Protect against running multipe Stream/Faye clients

Running multiple Faye clients leads to this bug:
https://github.com/GetStream/stream-js/issues/72
Which is very obscure and hard to debug. Wonder if we could provide a better error.

i believe this is related to #198:
faye client is expecting subscription  id,
also the dispatcher will bind to the subscription id as we can see here.

@tschellenbach correct me if I wrong but AFAIK we observed this issue back when we were still using the JS implementation of Faye server-side and concluded that the problem was client-side

Yeah this is broken with the standard Faye server. ID generation is global, so if you start 2 clients you get clashes.


fix: trim db urls and remove special characters during backup restore

fixes: #36176

[!WARNING]
Tests have not run on the HEAD 346f56728e4ec2c6a8df605b3a4fdb31b3a8e730 yet
Mon, 09 Sep 2024 13:20:11 UTC

Summary by CodeRabbit

Bug Fixes

Improved handling of database URLs by removing leading and trailing whitespace, enhancing robustness and preventing potential issues.

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true


Crashlytics reported that flutter app crashed on start on Android: couldn't find "libflutter.so"

I don't know why this issue #32756 was closed without any solution. And I couldn't find a similar issue, so I create a new one.

Flutter doesn't support 32 bit architecture that's libflutter.so is missing, you can analyze your apk in android studio and you will find that libflutter.so is missing in 32 bit folder.
Also GPay uses flutter so you can check that GPay isn't available for devices that support only 32 bit apps.

Hi @windows7lake
duplicate of https://github.com/flutter/flutter/issues/32756, please read  https://github.com/flutter/flutter/issues/32756#issuecomment-798948280


Integrate functionality of SandboxTest.ps1 script

Description of the new feature/enhancement
We already have the SandboxTest.ps1 script to test manifests in the Windows Sandbox, which is super useful but currently does not integrate well with wingetcreate. I suggest to include a command in wingetcreate to perform the same more conveniently via the tool. We might even want to consider adding a prompt to the manifest creation process, asking whether to test the installation in the sandbox before submitting a PR.
Proposed technical implementation details (optional)
In essence, wingetcreate could do exactly the same as the SandboxTest.ps1 script, just integrated into the tool.

What would be really nice is if we could test manifests in Windows containers, since the Sandbox is extremely inflexible for anything other than occasional tests (you can't customize the base image outside of the intial bootstrap PowerShell script that you have to run every time, for example). The biggest problem with it is that you can only run one at a time, so you have to wait for the installer to finish before running the next one (a gigantic bottleneck if you want to test multiple architectures or scopes or something). You also have to have Windows 10 Pro/Enterprise to run Sandbox, so you couldn't use it in CI environments that run on Windows Server.
I've been playing around with Docker for Windows and getting winget running under that environment, and it works for everything except msix packages which fail with an unknown error (I don't know if that's winget or something with appx provisioning though). The biggest problem is that there isn't a way to get RDP or a console up in the container even though you can run explorer.exe, but other than that they are a lot more flexible.


vcpkg-configuration.json: Allow for branch/tag names as "baseline" values

Is your feature request related to a problem? Please describe.
We're using vcpkg manifest mode in our project, and have a vcpkg-configuration.json file with registry entries for vcpkg itself as well as a set of overlay ports we maintain for convenience. Both entries have to use a git commit hash as their "baseline" value, which means that we need to update the overlay ports' hash every time we push something new to master. This is tedious and error-prone, and has been the source of various issues in the past.
Proposed solution
It would be good to be able to specify a branch/tag name instead of a commit hash. That way, we could have e.g. "2022.10.19" for the vcpkg baseline, and "master" for the overlay ports.

Duplicate of #25044, now #25087.


WebSocket requests with high response time trigger recycle too frequently

Hi there,
I have an ASP.net Core WebSocket Server hosted as Azure Web App.
In Azure Portal I have very high response times. It seems like the complete time a WebSocket connection is opened, counts as response time.
Due to these high request times, the Web App is recycled more frequently.
Today I got three events with ID 2299 (Worker Process requested recycle due to 'Percent Slow Requests' limit.) in "Application Events".
Is it possible to tell Azure, that it shouldn't count the response times of WebSocket requests to these limits?
Or is it possible to deactivate this limit completely?
Best regards.
Alex

@shirhatti , where should an app service specific issue like this go?

@davidebbo ?

I'm not familiar with this process recycle mechanism. @ndreisg can you start a thread in either the MSDN Web App forum or Stackoverflow to reach the right crowd? Thanks.

Thread in MSDN Web App Forum: https://social.msdn.microsoft.com/Forums/en-US/e3bcdb69-a28f-482e-a89a-49b1a7e00ecf/websocket-requests-are-leading-to-high-response-times-in-azure-monitoring

Closing because it appears there is an accepted answer at https://social.msdn.microsoft.com/Forums/en-US/e3bcdb69-a28f-482e-a89a-49b1a7e00ecf/websocket-requests-are-leading-to-high-response-times-in-azure-monitoring.


Garbage in TMPDIR resulting in Failed to create CoreCLR, HRESULT: 0x80004005

Steps to reproduce
See comments in https://github.com/dotnet/runtime/issues/46462
Hm, it seems it is difficult to reproduce from an interactive Shell.
The automated Script is started from a daemon process, several times a minute ...
...
ls -lrtd /tmp/????????-????-????-????-????????????

Expected behavior
Empty output
Actual behavior
Garbage as this ist left:
        $ find 88a84cf3-05b5-40c3-a29e-52d611e919e7
        88a84cf3-05b5-40c3-a29e-52d611e919e7
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.cache
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.cache/powershell
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.cache/powershell/StartupProfileData-NonInteractive
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.cache/powershell/telemetry.uuid
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.local
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.local/share
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.local/share/powershell
        88a84cf3-05b5-40c3-a29e-52d611e919e7/.local/share/powershell/Modules

Environment data
Same with the LTS 7.0.6 and non-LTS 7.1. CentOS 7 with MS Repo attached:
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.0.6
PSEdition                      Core
GitCommitId                    7.0.6
OS                             Linux 3.10.0-1127.19.1.el7.x86_64 #1 SMP Tue Aug 25 17:23:54 UTC 2020
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I believe this is related to https://github.com/PowerShell/PowerShell/pull/13239. PowerShell creates a temporary folder when the environment variable Home doesn't exist. Can you try setting the env variable Home and see if the issue is gone?

Yep, looks like that. This works as expected:
mkdir xxx
TMPDIR=./xxx pwsh

This is the content wheil pwsh was running. They dissappear:
prwx------ 1 alexei alexei 0 May  6 22:52 clr-debug-pipe-4432-226815218-in
prwx------ 1 alexei alexei 0 May  6 22:52 clr-debug-pipe-4432-226815218-out
srwxrwxr-x 1 alexei alexei 0 May  6 22:52 CoreFxPipe_PSHost.D742B9AE.4432.None.pwsh
srw------- 1 alexei alexei 0 May  6 22:52 dotnet-diagnostic-4432-226815218-socket

If I run it like this:
TMPDIR=./xxx HOME=/nowhere pwsh

Garbage remains in ./xxx:
$ find .
.
./b1409506-7aae-4211-b3c0-0d24ddbfbdf5
./b1409506-7aae-4211-b3c0-0d24ddbfbdf5/telemetry.uuid
./b1409506-7aae-4211-b3c0-0d24ddbfbdf5/StartupProfileData-Interactive
./b1409506-7aae-4211-b3c0-0d24ddbfbdf5/7.1.3
./b1409506-7aae-4211-b3c0-0d24ddbfbdf5/7.1.3/sentinel1-2021-5-6.done

I thought about this issue more, and believe we should fix it in our code to make PowerShell work better in a container environment without the env variable Home defined. After all, it's not uncommon for a container to not have Home env var. I think it can be fixed by adding a well-known prefix to the temporary folder name, and next time PS session starts, it can just look for an folder under tmp with that well-know prefix and use that as the pseudo Home directory.


bump electron-builder version

🚀
I compiled the app npm run electron:mac and it worked 👌
Glad to see bumping version doesn't break things (as expected, though there are times when it does 😅 )
Merging -- no need to review 👌

🚀


📝 Update includes for docs/tutorial/fastapi/teams.md

📝 Update includes for docs/tutorial/fastapi/teams.md

Thanks for the effort! This was covered in another PR, so I'll close this one now. :coffee: :cake:


flutter web empty response header

New Issue Checklist

[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
flutter web

Flutter Version
Channel beta, 1.18.0-11.1.pre, on Linux

Dio Version
^3.0.9

Android Studio / Xcode Version
Android Studio (version 4.0)

Repro rate
all the time (100%)

Repro with our demo prj

Demo project link

Issue Description and Steps
hi everyone
I have a problem with response headers in POST request in web version.
response header is empty
but in google chrome network inspect everything is fine
flutter doctor output
`[✓] Flutter (Channel beta, 1.18.0-11.1.pre, on Linux, locale en_US.UTF-8)
• Flutter version 1.18.0-11.1.pre at /home/mhalizadeh/Android/flutter.beta
• Framework revision 2738a1148b (2 weeks ago), 2020-05-13 15:24:36 -0700
• Engine revision ef9215ceb2
• Dart version 2.9.0 (build 2.9.0-8.2.beta)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /home/mhalizadeh/Android/Sdk
• Platform android-29, build-tools 29.0.2
• ANDROID_HOME = /home/mhalizadeh/Android/Sdk
• Java binary at: /opt/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build
1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• CHROME_EXECUTABLE = google-chrome-stable
[✓] Android Studio (version 4.0)
• Android Studio at /opt/android-studio
• Flutter plugin version 44.0.2
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build
1.8.0_242-release-1644-b3-6222593)
[✓] Connected device (3 available)
• Android SDK built for x86 • emulator-5554 • android-x86    • Android 9
(API 28) (emulator)
• Web Server                • web-server    • web-javascript • Flutter Tools
• Chrome                    • chrome        • web-javascript • Google Chrome
• No issues found!
`

Have the same problem, except in my case there is only 'content-type' in response headers. I need X-Pagination-.. info. Works fine with android device. dio 3.0.9

@temirfe did you manage to find the cause?


Allow bold and blinking text in Terminal

A bold font colour is being set in the theme, but the preference to allow bold text was set to false.

lgtm

Thanks!


Added styles v1

All the options are centred with the help of Flexbox. The final look

@PrathameshhW
You have done a great job. The design idea is really amazing. It would be great if some more changes were made.
Firstly, a fix is needed in yours, if we add more than 4 subjects, it ain't possible to go to the heading section. As we add more subjects, we can't reach the subject boxes at the top.

Secondly, I would like the heading GPA calculator to be fixed i.e. it should not change position when new subjects are deleted or existing ones deleted.
Thirdly, can we make the look more pleasing by adding colours to the subject box?
Fourthly, decrease the font size of the Credit and Grade
Thank you!

Sure. We can work on that. I'll send the PR after some time.

Sure. We can work on that. I'll send the PR after some time.

@PrathameshhW
No need to create a new PR. You can make the changes in the current branch of yours and push it.

And also, please mention the issue in the PR description.

I'm closing this as you submitted the other PR which also contains these changes.


Support eating food whilst staring at blocks

GD: 1.2.6
SF: 1.12.2-2838-7.2.0
LP: 5.0.116
Currently if I were to have interact-block-secondary false in claims then a user looking at a block and attempting to eat food would not be able to do so as shown in the two calls in this debug:
I would suggest a few methods in ensuring this works.

Allowing the #food context key to be used in the context[used_item=]

Checking either the item-use or interact-item-secondary flags if interact-block-secondary fails

My secondary concern with allowing this however would be allowing users to use chests/tile-entities if they are allowed to eat food, so if there's a way to ensure that doesn't happen if this is implemented that'd be fantastic!

I would also love to use this for things like splash and normal potions, bows, mod backpacks, etc!

This was resolved in 1.4.0+


useA11yEnabled cleanup callback is throwing an error on development

I encountered an error during development, and I'm unsure if this issue is occurring in production (although it's likely not).

I made a research and I found that this is the line of the error and this is not assigned because here we are using
return listener.remove;

and we lose this pointer
the best alternative for me is to use
return () => listener.remove();

but we could also use
return listener.remove.bind(listener);

Hello @leobastiani,
Thank you a lot for a help and creating issue.
This has been fixed with version 0.3.2.
I won't close this issue until I fix it in "0.3.0-android.2" which is used for supporting old  version of RN

Old React Native versions support approach has been changed with 0.4.2 version.
It means that this issue has been fixed for "all" versions.


Cannot find HanaConfig.cmake

Hello,
I tried to use your library with my project but I have this error with CMake :
  CMake Error at CMakeLists.txt:17 (find_package):
  By not providing "FindHana.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "Hana", but
  CMake did not find one.

  Could not find a package configuration file provided by "Hana" with any of
  the following names:

    HanaConfig.cmake
    hana-config.cmake

I installed Boost 1.65.1 with Homebrew on macOS High Sierra, but it doesn't seem to have this file configured while I can add other Boost librairies in my CMake file.

To make it work I used your script in the dependencies folder !

Ahhh. Actually, I think this is a bug with the Boost distribution. When you install Boost, it should install a lib/cmake/Hana/HanaConfig.cmake file with it, but it doesn't. Reopening to keep track of this.

For people experiencing the same problem and want to use a global installation of Boost, I've used:
find_package(Hana QUIET)
if (Hana_FOUND)
  target_link_libraries(dyno INTERFACE hana)
else()
  find_package(Boost 1.61)
  if (Boost_FOUND)
    target_link_libraries(dyno INTERFACE Boost::boost)
  else()
    message(SEND_ERROR "hana not found: either install stand-alone or use Boost>=1.61")
  endif()
endif()

Do the same for CallableTraits as well (but change the version to 1.64).
Also we need to change cmake/dyno-config.cmake file as well.
include(CMakeFindDependencyMacro)
find_package(Hana QUIET)
if (NOT Hana_FOUND)
  find_package(Boost 1.61 REQUIRED)
endif()

find_package(CallableTraits QUIET)
if (NOT CallableTraits_FOUND)
  find_package(Boost 1.64 REQUIRED)
endif()

# Below is the same
if(NOT TARGET Dyno::dyno)
  include("${CMAKE_CURRENT_LIST_DIR}/dyno-targets.cmake")
endif()

I am not sure whether this should be reported to Boost or packagers of my distribution but this is a workaround.


Feature/scheduler

Addin task on runtime will be possible with this PR
this should close #2 #5

Maybe @jphilipps has time for a review? 😁

Maybe @jphilipps has time for a review? 😁

Of cause, I’ll take a look as soon as I can.


Not built (no dist/ dir) using bower

Should the library build itself on install? Currently the 'build' grunt task dies not sppear to be run, and there is no bower.json
I'm sure it worked yesterday.

dupe of #99?

Yes. That's closed though, didn't think I could re-open it.
On 30/01/2016 05:12, Kevin Decker wrote:

dupe of #99 https://github.com/kpdecker/jsdiff/issues/99?
—
Reply to this email directly or view it on GitHub
https://github.com/kpdecker/jsdiff/issues/100#issuecomment-177062115.

I'm really not clear what you are asking between these two issues. Install via bower? Install via git? npm?

Note that the bower package lives here: https://github.com/components/jsdiff as this contains built components. If bower is not pointing to that, then that may be the source of the problem. Grunt should in no way be involved for bower users.

kpdecker -- many thanks, your last comment fixed my silliness.
It had been a long day, and I'd not read any of the sources, just assumed that the module built itself on bower install
Thanks for your help, and a very useful module.

Glad to hear!
On Mon, Feb 1, 2016 at 3:00 AM Lee Goddard<EMAIL_ADDRESS>wrote:

Closed #100 https://github.com/kpdecker/jsdiff/issues/100.
—
Reply to this email directly or view it on GitHub
https://github.com/kpdecker/jsdiff/issues/100#event-533862890.


tabs 风格与官方 code-group 不相符

tabs style does not match the official code-group.

tabs スタイルは公式 code-group と合わない.

It's because these plugin existed before the code-group feature was added.
PRs to change using the code-group feature for plugin-detype and plugin-npm-commands are welcome.

Duplicate of https://github.com/sapphi-red/vitepress-plugins/issues/12


Add husky and precommit hook

This hooks will run every time you commit a code. This should run npm run build and add all changes on dist folder to the commit.

@willianjusten Why not add /dist to .gitgnore to keep commits clean?

@willianjusten Good idea, thanks for the PR!


Work around broken generator + requested changes from older PR

@nicogeburek I can merge your PR after you have resolved the conflicts.


Generate svg from shapes layer

Description
As requested in #178 this PR supports generating an svg from the shapes layer by adding a to_svg method onto the shapes layer. It will generate an svg string that can then be written to a file. It will include all the support shape types and their correct colors, line widths, opacities, and z_order. In the future the to_svg method could be moved to the viewer so that an svg of all the layers could be generated. We would need to then add svg support to the other layer types such as images / markers / vectors etc.
Here is a screenshot of one of the generated svg files opened in another graphic editing app

Type of change

[x] New feature (non-breaking change which adds functionality)

How has this been tested?

[x] running examples/shapes_to_labels.py will now also generate an svg file. Maybe people don't like that this example now writes a file. If so I can remove that.

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

Hello @sofroniewn! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file napari/layers/_shapes_layer/shapes/path.py:

Line 88:39: E225 missing whitespace around operator

In the file napari/layers/_shapes_layer/shapes/polygon.py:

Line 87:39: E225 missing whitespace around operator


TeslaMate MQTT warning at every HA startup

After the release of the TeslaMate MQTT function, which I don't use, I'm getting this warning at every startup of Home Assistant:
WARNING (MainThread) [custom_components.tesla_custom.teslamate] Cannot unsub from TeslaMate as MQTT has not been configured.
Not a big deal but a bit annoying.

I am also experiencing this issue.


Adjust terms & conditions, etc

currently lorem ipsum
Possible to read from markdown files or something like this?
Will create documents based on eHive documents (sent by Micol) and StudyU documents

Handled on Sinai remote using static files


chore: a kokoro job to store the commit hash of each release

See go/cpp-sbom for rationale.

This change is

Ready for another look.


MM-48313: Use the term Cost for easier l18n

Summary
See discussion at https://community.mattermost.com/core/pl/qkhuqqxw7frpxro6nyiwhfmt1a, but essentially it boils down to a small text change from Dollars to Cost, simplifying the localization:

Ticket Link
Fixes: https://mattermost.atlassian.net/browse/MM-48313

/update-branch


fix: Typing of labelMap

SUMMARY
Fixes the typing of labelMap to use string instead of DataRecordValue as values.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@michael-s-molina the types here and here also needs to be changed.


[Bug]: pkg's binary don't run as linux service

🔎 Search Terms
serviced binary
The problem
0
I have a very simple node js application that uses the npm Winston package. The code is only a loop printing date using the Winston package The app runs just fine. I created a binary using pkg that also runs fine. The problem is that when I run it as a service using the file
[Unit] Description=Cryptween daemon service After=systend-user-sessions.service [Service] Type=simple WorkingDirectory=$CRYPTWEEN_BIN_DIR ExecStart=$CRYPTWEEN_BIN_DIR/cryptween-daemon-linux ExecReload=/bin/kill -HUP $MAINPID Restart=always StandardOutput=syslog StandardError=syslog SyslogIdentifier=CttWBot_Console [Install] WantedBy=multi-user.target
I get the error
     cryptween.service - Cryptween daemon service
     Loaded: loaded (/etc/systemd/system/cryptween.service; enabled; vendor preset: enabled)
     Active: failed (Result: exit-code) since Thu 2022-03-24 10:54:45 -03; 1s ago
    Process: 213784 ExecStart=/opt/cryptween/bin/cryptween-daemon-linux (code=exited, status=1/FAILURE)
    Main PID: 213784 (code=exited, status=1/FAILURE)
    mar 24 10:54:45 gintama CttWBot_Console[213784]:     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
   mar 24 10:54:45 gintama CttWBot_Console[213784]:     at Module.load (node:internal/modules/cjs/loader:981:32)
    mar 24 10:54:45 gintama CttWBot_Console[213784]:     at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    mar 24 10:54:45 gintama CttWBot_Console[213784]:     at Module.require (node:internal/modules/cjs/loader:1005:19)`

mar 24 10:54:45 gintama CttWBot_Console[213784]:     at Module.require (pkg/prelude/bootstrap.js:1719:31)
mar 24 10:54:45 gintama CttWBot_Console[213784]:     at require (node:internal/modules/cjs/helpers:102:18)
mar 24 10:54:45 gintama CttWBot_Console[213784]:     at Object.<anonymous> (/snapshot/test-error-daemon/src/config/winston-logger.js:2:16)

mar 24 10:54:45 gintama CttWBot_Console[213784]:     at Module._compile (pkg/prelude/bootstrap.js:1794:22)
mar 24 10:54:45 gintama systemd[1]: cryptween.service: Main process exited, code=exited, status=1/FAILURE
mar 24 10:54:45 gintama systemd[1]: cryptween.service: Failed with result 'exit-code'.`

the las error report line is pointing winston's require package
Thank you in advance
What version of Winston presents the issue?
v3.3.3
What version of Node are you using?
v16.9.1
If this worked in a previous version of Winston, which was it?
No response
Minimum Working Example
git repo
Additional information
No response

You might get better luck asking on StackOverflow, crosslinking between questions - we don't really have bandwidth to address a lot of time on usage questions (and I'm not personally familiar enough with that style of running things to be able to help much myself).

If you already did ask, you should have included a link to that question here, and vice versa.
Also, if you have an infinite loop, how do you expect a non-error exit code?  Maybe have something in your code that listens for a shutdown signal and does a graceful shutdown of all resources.  That's a bit out of scope for this forum focused on bugs in the Winston software, though.

If you already did ask, you should have included a link to that question here and vice versa. Also, if you have an infinite loop, how do you expect a non-error exit code? Maybe have something in your code that listens for a shutdown signal and does a graceful shutdown of all resources. That's a bit out of scope for this forum focused on bugs in the Winston software, though.

I thank you for the comment. I will link the stackoverflow question here. I did share the code with the link to a github repo. If you take a look at it, you will see that it is just a line that calls the Winston package. It is the simpler sample you can get.
      'use strict'
      process.env.APP_NAME = "test"
      const logger = require('./config/logger');

      function delay(ms) {
        return new Promise(resolve => setTimeout(resolve, ms))
      }

      (async function main() {
        while (true) {
          let date = Date.now()
          logger.info(date.toString())
          await delay(5000)
        }
      })()

where logger is winstion logger

well it was all about access problem. Sorry. Thanks @wbt

well it was all about access problem. Sorry. Thanks @wbt


Implement rename collection

Description
Closes #1517 .

Readiness checklist

[ ] I added/updated unit tests.
[ ] I added/updated integration/compatibility tests.
[ ] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[ ] I ensured that PR title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Assignee, Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

Closing this PR in favor for https://github.com/FerretDB/FerretDB/pull/2343


feat: per-channel backend support

Description
To support a hybrid example in the SDK, per-channel backend is
necessary. The openapi specification is updated to accept new fields:
backend and brokerHost in the channel component.
To check its working, a job example is created under top-level
examples folder, based on a hybrid example from the SDK.
Type of Change

[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[ ] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ ] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[ ] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

This is a revised version of #373.


add lower command

$ lower 33C8964B-0CB1-4496-80E6-EDDE7279D54D

https://stackoverflow.com/questions/2264428/how-to-convert-a-string-to-lower-case-in-bash


Bug with monthHeaderBinder

Reproduce:
 <com.kizitonwose.calendarview.CalendarView
                android:id="@+id/calendarView"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                app:cv_dayViewResource="@layout/item_calendar_day_layout"
                app:cv_orientation="horizontal"
                app:cv_scrollMode="paged"
                app:cv_hasBoundaries="false"
                app:cv_monthHeaderResource="@layout/item_month_header"
                app:cv_maxRowCount="6"/>

inner class MonthHeader : MonthHeaderFooterBinder<MonthViewContainer> {

        override fun bind(container: MonthViewContainer, month: CalendarMonth) {
            container.setName(month)
        }

        override fun create(view: View): MonthViewContainer = MonthViewContainer(view)
    }

CalendarMonth always the same.

You have hasBoundaries set to false so it's not that the header view is not updated but it's updated with the same value. Please set hasBoundaries to true unless you really want the boundaryless behavior.

Closing this as it works as intended.


update packages

:white_check_mark: Build TheIdServer 4.0.2-fix-dependencies1-0009 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/00572ea3da by @)


support defaults in the generated json schema

prerequsite: https://github.com/vega/ts-json-schema-generator/issues/249

not sure what this should be used for, closing


Certain heights/margins before editor container cause misplaced hover.

monaco-editor version: 0.20.0
Browser: Chrome v80, Firefox v74
OS: Windows 10
Playground code that reproduces the issue:
The example provided is the same as the hover provider example in the playground with the removal of the XMLHttpRequest. I have the min folder installed at lib/monaco locally.
<!DOCTYPE html>

Header

Any of the following html snippets placed before the container will cause this issue.

Header

Header

Header

Header

Setting both margin-top and margin-bottom fixes this, in any combination that is less that 40px (which causes the issue again).

Header

Header

Testing with a div element with margin: 20px led to determining that heights >= 36px cause the same problem.

Footnote: I've been working with Monaco Editor a lot in the past few days and have found a couple other things that are issues/odd behavior/feature requests. If you don't mine a flurry of new issues for things I'm happy to open something for them.

I cannot reproduce:

Closing due to inactivity.
If this bug still applies, please create a new issue. Make sure to select the form for bugs and include a minimal reproducible example. Thanks!


Question: Subnet Placement

Hello,
Is this EC2 instance meant to be placed in a Private or a Private Isolated Subnet?

Hey,
in a private subnet. I never tried to place it in an isolated subnet, so I'm not sure if you would still be able to connect to the instance. But I would be very much interested in the outcome 🙂


[Feature] Add "Contact Us" Page

Overview:
I would like to propose the addition of a "Contact Us" page to this project. This page should be easily accessible through the navbar or the footer for better visibility and user engagement.
Implementation Details:

File Location: ./src/components/ContactUs.jsx
Navbar and Footer Links: Ensure that the "Contact Us" link is included in both the navbar and the footer for easy accessibility.

Additional context
Include a link to the "Contact Us" page in the main navigation bar as well as in the Footer.

@Someshog work on it


"search" proprerty set true the "disableSelectAll"

Hi, when we set "search" proprerty to "false" the "Select all" button disappears, i try to set "disableSelectAll" => "false" manually but its not working. Can we just get the "Select all" button without "Search" ?

@ahsynv this issue got fixed and available in the release v1.0.6

I think this regressed, I need the search button to make http requests and fill the array of options, but I don't see the "select all" option.


Suggest to use email address as silence creator

Using email addresses provides a unique identifier with a defined
format. It will reduce the amount of different and ambigious identifiers
used (first, last, full, initials, different case used, etc.) while
still allowing others to use a different format.
I could also imagine making an email address mandatory.
@juliusv @matthiasr

:+1: approved

:+1:


[patch] Fix readthedocs config

By letting the pre-build job have exclusive control over the install

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2

Relevant Lines:
2

💛  - Coveralls


Fix traceback for unsupported git providers

Each committime exporter supports only one GIT provider.
Currently it's implemented in a way it tries to talk to all GIT URLs except well known other provires. This allows to collect data from self hosted GIT providers with custom fqdn.
Fix is to properly propagate error for unsupported by each individual class GIT providers.
resolves #577
Testing Instructions
Setup exporter as in the bug and check the logs.
With the current implementation we get:
08-02-2022 12:46:12 WARNING  Missing data for CommitTime metric from Build pelorus/failure-exporter-1 in app failure-exporter: Skipping non GitLab server, found https://github.com

/retest

/lgtm


You can examine entities hidden behind the viewport size cutoff

Description

Find an entity.
Make sure to not hide it behind FOV, maintain line of sight with it
Move so it's outside of the viewport and in the black regions on the side.
Right click it.

The airlocks at spawn work well for this.

dupe #4651


chore(parser): explain the reason for omitting "}" and ">" in jsx text lexer

closes #2094

Current dependencies on/for this PR:

main

PR #2097   👈

This stack of pull requests is managed by Graphite.


Handlebars static and dynamic helpers

Use same handlebars helpers for static templating and dynamic templating.
Currently the "hb:static" task uses the helpers from /lib/hbs-helpers.js.
For dynamic templating on runtime we use a different file /resources/js/handlebars.helper.js.
It would be better to inject the handlebars.helper.js into the hb:static task, in order to have only one place to define helpers.
Disscussion so far:
@timomayer Not sure if this is a good idea we should not mixup build framework and frontend Framework
@alxbenz I think it would be good to at least have this opportunity: for example if one project uses some complex hbs template with custom helpers for runtime templating but also needs to have a static preview.
@janrembold In my opinion the best way would be to extract our helpers into its own repository and include it anywhere we need it

@janrembold  @timomayer can't we just define an array of paths to helper files in the projectConfig file? This would keep build framework and frontend framework separated, but still make it possible to infuse helpers to the static handlebars build process.

@timomayer @janrembold I have opened a pull request https://github.com/biotope/build-framework/pull/38/ that allows adding custom helpers in projectConfig.js. I am opposed to using the same helper file for both frontend and build templating as this can have unnecessary performance penalty if we have many helpers that are needed by build, but not by run-time environment.

The changes to projectConfig.js were reverted. Helpers in resources/js/handlebars.helper.js are now available in both build and frontend frameworks.

@smohadjer i agree with what you said in your previouse comment about performance when we have bigger helper files. think we should use projectConfig so it is cobfigurable if it should be the same helpers or not.


django-offsite-storages raises NotImplementedError on image upload

An error is raised whenever a ProductImage's path property is called upon. I am not able to upload any media files due to this and not sure of a workaround. Everything works fine in local dev environment. I'm using Amazon S3 and my static files are working fine (different bucket, same credentials.
/lib/python3.4/site-packages/offsite_storage/storages/s3.py in path
def listdir(self, name):
        raise NotImplementedError()
    def modified_time(self, name):
        raise NotImplementedError()
    def path(self, name):
                    raise NotImplementedError()

Full traceback:
lib/python3.4/site-packages/django/core/handlers/base.py in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs) ...
lib/python3.4/site-packages/django/contrib/auth/decorators.py in _wrapped_view
return view_func(request, *args, **kwargs) ...
/media/storage/Programming/sale/store/saleor/dashboard/product/views.py in product_image_edit
product_image = form.save() ...
lib/python3.4/site-packages/django/forms/models.py in save
construct=False) ...
lib/python3.4/site-packages/django/forms/models.py in save_instance
instance.save() ...
/media/storage/Programming/sale/store/saleor/product/models/images.py in save
super(ProductImage, self).save(*args, **kwargs) ...
lib/python3.4/site-packages/django/db/models/base.py in save
force_update=force_update, update_fields=update_fields) ...
lib/python3.4/site-packages/django/db/models/base.py in save_base
updated = self._save_table(raw, cls, force_insert, force_update, using, update_fields) ...
lib/python3.4/site-packages/django/db/models/base.py in _save_table
result = self._do_insert(cls._base_manager, using, fields, update_pk, raw) ...
lib/python3.4/site-packages/django/db/models/base.py in _do_insert
using=using, raw=raw) ...
lib/python3.4/site-packages/django/db/models/manager.py in manager_method
return getattr(self.get_queryset(), name)(*args, **kwargs) ...
lib/python3.4/site-packages/django/db/models/query.py in _insert
return query.get_compiler(using=using).execute_sql(return_id) ...
lib/python3.4/site-packages/django/db/models/sql/compiler.py in execute_sql
for sql, params in self.as_sql(): ...
lib/python3.4/site-packages/django/db/models/sql/compiler.py in as_sql
for obj in self.query.objs ...
lib/python3.4/site-packages/django/db/models/sql/compiler.py in
for obj in self.query.objs ...
lib/python3.4/site-packages/django/db/models/sql/compiler.py in
) for f in fields ...
lib/python3.4/site-packages/versatileimagefield/fields.py in pre_save
file = super(VersatileImageField, self).pre_save(model_instance, add) ...
lib/python3.4/site-packages/django/db/models/fields/files.py in pre_save
print(file.path) ...
lib/python3.4/site-packages/django/db/models/fields/files.py in _get_path
return self.storage.path(self.name) ...
lib/python3.4/site-packages/offsite_storage/storages/s3.py in path
raise NotImplementedError() ...

@iansprice Fix is on the way. path method shouldn't be called on the external storage.

The new version of django-offsite-storage has been released.
pip install django-offsite-storage==0.0.7

Works like a charm, thanks! Didn't want to go down the Boto road with this :)


Importing glTF with multiple animations fails to load all but one animation

Description
Importing a glb (or gltf) with multiple animation sequences will only load the first animation. The importer used to load all animations correctly in Blender 2.80, and since Blender 2.81a this stopped working.
To Reproduce

Unzip the example box_rotate.zip of two identical box rotation animations saved as gltf/bin.
Import the gltf file in Blender 2.82 or 2.81a.
Scrubbing the playhead will only animate one of the meshes.

Expected behavior
Both given meshes should be animated (which is what happens in Blender 2.80).
.blend file/ .gltf
box_rotate.zip
Version

OS: Pop!_OS 19.10
Blender Version 2.82

I'm the one who changed this, see #671. I think this is working correctly, but we can revisit it. I think @fire also thought we should play the first action that can be played on an object.
First, all the animations do get loaded, it's just that only the first animation plays.
The animations are organized like this in the NLA editor:

Why I think playing only the first animation is the right thing to do:

This matches both the Three.js and Babylon.js viewer, which both play only the first animation by default.
Each glTF animation is self-contained. There's no way to know that two animations should "play together". Sometimes it will be true, but we should be conservative and not assume that. Things that should always play together should be part of the same animation.
If two animations target the same object and we tried to play both simultaneously, we'd have pick one animation that should "win" at that object, and we could wind up with part of one animation playing on one part of a scene and part of another animation playing somewhere else, creating a potentially nonsensical result animation that never existed in the glTF file.
"Always plays the first animation" is easy to explain and understand.
You can still set it up to do whatever you want by hand after import.

cc @julienduroure @donmccurdy @emackey

@elrnv if the intention for gltfgen output is to have a scene with a single global animation timeline, played consecutively, then it would be best to output only a single animation rather than one per object. Playback in glTF viewers will behave more as you expect with that approach.
@scurest I agree with your points above. This model seems unusual in having N distinct animations each affecting exactly one of N objects, where (purely from a Blender artist's perspective) it would be understandable to expect one active Action on each object after importing. That's an unusual case, and not one we encourage glTF exporters to use, so I'm not necessarily saying we should try to detect it and handle it differently than the (easier to explain) behavior you describe.

Ah that makes sense. I actually have not issue with this being the default behaviour. I think I just misunderstood how the animation system is supposed to work. I will adjust gltfgen to output a single animation. Thank you for the feedback!


Ram 1500: more generic EPS steer to zero check

@realfast what do you think about this? I'd like to make the check more generic to catch new cars without a specific revision of the FW. It looks like that there's a clear pattern with the FW versions. Older major revisions allow a lower min steering speed. Same with the Chrysler check above. We might be able to unify them in the future when we see more Ram FW.

I agree this is a great way to handle this.


Shaped Script updated to 13.8.0

Greethings,
Did you intend to delete all these files ?
Thanks,
Cassie

@clevett yes, it is just removing old versions. Compare https://github.com/Roll20/roll20-api-scripts/tree/master/5eShapedCompanion vs https://github.com/mlenser/roll20-api-scripts/tree/master/5eShapedScript to see that the version released to roll20 is now just the latest version


Add Prelude 0.6

Summary

Added prelude of v0.6.
Added migration guide to v0.6.

Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.028%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4510

Relevant Lines:
4848

💛  - Coveralls

Can update this release note: switch-to-qiskit-algorithms-b0af38fe547bf5b5.yaml. It has a bit which is also in the prelude but can we change Qiskit Terra to simply Qiskit

Qiskit Optimization 0.6 switches from qiskit.algorithms of Qiskit Terra to Qiskit Algorithms. Qiskit Optimization 0.6 drops supports of the former algorithms based on qiskit.algorithms, QuantumInstance, and Opflow of Qiskit Terra.

This is also explained in the blog but I guess its ok to have this short text there too which was done earlier.

Thanks. I updated it.


Test for Python syntax errors and undefined names

These tests will pass when #1 is resolved.
http://flake8.pycqa.org will flag the build if there are Python syntax errors or undefined names.
E901,E999,F821,F822,F823 are the "showstopper" flake8 issues that can halt the runtime with a SyntaxError, NameError, etc.  Most other flake8 issues are merely "style violations" -- useful for readability but they do not effect runtime safety.  This PR therefore recommends a flake8 run of these tests on the entire codebase.

F821: undefined name name
F822: undefined name name in __all__
F823: local variable name referenced before assignment
E901: SyntaxError or IndentationError
E999: SyntaxError -- failed to compile a file into an Abstract Syntax Tree

Thanks for the PR!
Since flake8 is linting .py files, the entire codebase is in this case only a single file which contains some configuration code that should be added to jupyter_notebook_config.py. Thus, I think this is a bit overkill for the repo. In addition, I explained in #1 why this may not be a good idea for the potential future .py files.
Anyway, I think that most Python repos should have some kind of automatic linting as part of their CI system and flake8 is usually the best option for that. It's just not a good fit for this project.