Mock API

As described in #30. I opted to use an environment variable to control whether the mock API is used and for now I'm just returning zeros or current state deterministically everywhere. I also updated the Demo-Drive notebook to use the mock API when no key is specified.

Feel free to merge once @AlirezaMorsali  is okay with it

I can fix the conflicts if you are working on something else.

Go ahead - I'm working on the docs.


Evalutor only supports esp32s3? (AIV-671)

Using latest code from master branch and quantizing model for esp32, got the error below:

Generating the quantization table:
Converting coefficient to int8 per-tensor quantization for esp32
Exporting finish, the output files are: /content/drive/MyDrive/hand_recognition//trained_model//coefficient.cpp, /content/drive/MyDrive/hand_recognition//trained_model//coefficient.hpp
Quantized model info:
model input name: conv2d_input, exponent: -7
Reshape layer name: StatefulPartitionedCall/sequential/conv2d/BiasAdd__6, output_exponent: -7
Conv layer name: StatefulPartitionedCall/sequential/conv2d/BiasAdd, output_exponent: -8
MaxPool layer name: StatefulPartitionedCall/sequential/max_pooling2d/MaxPool, output_exponent: -8
Conv layer name: StatefulPartitionedCall/sequential/conv2d_1/BiasAdd, output_exponent: -8
MaxPool layer name: StatefulPartitionedCall/sequential/max_pooling2d_1/MaxPool, output_exponent: -8
Conv layer name: StatefulPartitionedCall/sequential/conv2d_2/BiasAdd, output_exponent: -8
MaxPool layer name: StatefulPartitionedCall/sequential/max_pooling2d_2/MaxPool, output_exponent: -8
Transpose layer name: StatefulPartitionedCall/sequential/max_pooling2d_2/MaxPool__28, output_exponent: -8
Reshape layer name: StatefulPartitionedCall/sequential/flatten/Reshape, output_exponent: -8
Gemm layer name: fused_gemm_0, output_exponent: -9
Gemm layer name: fused_gemm_1, output_exponent: -9
Softmax layer name: StatefulPartitionedCall/sequential/dense_1/Softmax, output_exponent: -9
Evaluating the performance on esp32:
Traceback (most recent call last):
File "//content/drive/MyDrive/hand_recognition//esp_dl_quant.py", line 153, in
'minmax' if args.calib_method is None else args.calib_method)
File "//content/drive/MyDrive/hand_recognition//esp_dl_quant.py", line 61, in quantize
eva = evaluator.Evaluator(bw, granularity, target)
File "evaluator.py", line 32, in evaluator.Evaluator.init
ValueError: currently only supports esp32s3, while received esp32

Evalutor only supports  esp32s3?

Yes, it's only supports esp32s3


[Bug] Creating an application with the name of a just deleted application fails

Description
When creating an application with the name of an application that was just deleted, it fails with an error An application with this name already exists. If we refresh the page, the error doesn't happen and we can create the application with that name again.
Steps to reproduce the behaviour:

Create a new application called one.
Delete the application one.
Immediately, without refreshing the page, create an application in the same organization called one.
It will fail.
Refresh the page and try doing the same thing.
It will succeed.

Important Details

Version: Cloud
OS: macOS
Browser Firefox

Earlier issue was closed recently #796

I think that issue is different. It's about errors showing up when deleting an application. This one's about not being able to create an application with the name of a deleted one.

@sharat87 Is this a duplicate of #827 ?

Yeah looks like. Sorry folks, I should definitely be looking before raising issues. Will keep in mind going forward. Closing as duplicate. Thanks Arpit.

Also unable to reproduce this issue. It seems to have been fixed. Please re-open it if you feel differently.

I noticed it on prod, but looks like it's already fixed on staging.


Performance javascript grammar

Hi,
I have been using a variant of the javascript grammar for a while, but I have found a performance issue with multiple calls in a single statement.
The example function is:

    function test(
         x,
         y,
         z,
         m,
         a,
         b,
         c
    )  {

            var monkey = call(
                call2(
                    object1.call3(
                        "xxxxx",
                        call4(),
                        call5(
                            object2.call6(
                                call7(
                                    "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
                                ),
                                x,
                                y,
                                z,
                                m++,
                                s
                            )
                        )
                    )
                ),
                a,
                b,
                c
            );
    }

as you can see this is due to many inner calls, up to call6 the performance is around 1 second the it might be more than 2 seconds..
Do you have any ideas on how to improve this?

Can you isolate this to a small grammar input that shows the same behavior?  These sorts of things are sometimes ReDoS-style issues.

Yes that might be the case although is rather complex to reduce the grammar we have CallExpressions https://github.com/peggyjs/peggy/blob/main/examples/javascript.pegjs#L631MethodExpressions etc. Maybe creating a MultiCallExpression might be able to help.

If you turn on tracing, you might see the problem, with the same piece of input being parsed a lot.

@hildjj Fixed it on my end, but if anyone encounters this adding caching (or the lack of) was one of the culprits, it might have been on the migration that from peg that the setting was removed. Thank you very much.

Did it work better with caching on, or with caching off?  We might not have quite enough thought given to how caching interacts with the newer features.


fixes #64: include toggle for issuers not in Mozilla's root program

Let me know what you think. In particular, I kept the Top 10 CAs / Worst CAs as only considering issuers in Mozilla's program. This does make the UI potentially confusing.

Looks good!


[bug]: Import models from huggingface or civitai fail error code 5 (Google Collab)

Is there an existing issue for this problem?

[X] I have searched the existing issues

Operating system
Windows
GPU vendor
None (CPU)
GPU model
Google Collab Nvidia T4
GPU VRAM
16 GB
Version number
4.0.4
Browser
Firefox Latest version
Python dependencies
No response
What happened
Hello it happens randomly when i import any file sometimes it refuses to copy and gives me this error inout output in the install queue tab, neither resetting the collab works, it does work if you are lucky
What you expected to happen
i cant replicate it , it does happen casually
How to reproduce the problem
No response
Additional context
No response
Discord username
No response

Please provide the terminal output showing the full error message.
Without seeing the full error, it sounds like a network error, which means the problem is between the computer on which invoke is running and the remote server hosting the model file. We can't do anything about that.
Also, we don't support Google Colab. In fact my understanding is that using Invoke on Colab is against the Colab terms of service.


Unknown icon is repo search results

I was asked to create this based on https://github.com/microsoft/vscode-remote-repositories-github/issues/320
Does this issue occur when all extensions are disabled?: Yes/No
Unknown.

VS Code Version:

PS C:\> code --version
1.77.1
b7886d7461186a5eac768481578c1d7ca80e2d21
x64

OS Version:

Caption        : Microsoft Windows 10 Pro
Version        : 10.0.19045
DisplayVersion : 22H2
BuildType      : Multiprocessor Free
OSArchitecture : 64-bit

Steps to Reproduce:

Search remote GitHub repository
Unknown icon occasionally appears in filename list. Looks like it might be related to communication with the remote repo, but unknown meaning.

Can you give reproducible steps?

Which remote repository do you open
what do you search for
when you open the file with the unknown icon, what language does it show in the status bar? Is the icon correct on the editor tab?

The repo is a GitHub Enterprise Cloud repo for a Fortune < 10 company. It seems unlikely that even if I could tell you what repo it is that you would be able to access it.
I provided a picture of the icon. When is that image used?

I need reproducible steps to be able to investigate. If your repor is private, can you maybe reproduce/reconstruct  it with a public repository?

what do you search for
when you open the file with the unknown icon, what language does it show in the status bar? Is the icon correct on the editor tab?


Hinge Joint Inspector - Change range for motor target velocity and params bias values

please refer to Issues #19203 and #19204 Hinge Joint Motor
Motor/Target Velocity: Previously it would not allow negative values. This is needed to set the rotation in either positive or negative rotation direction.  The range max of 4096 represented around 39000 rpm, which seems excessive. The new range is [-200,200] but that can be overridden by typing in a value out of range.
Params/bias: Previously it would set the min value to 0.01. A message would be issued at run time that bullet hinge constraint does not support bias.  The range has been modified to support a zero value for the bias.
Edit: spelling. Correction about bullet message.

 Naming issues is good in the description of the commit, but in the title, we mostly need a short explanation in English. See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#format-your-commit-logs-with-readability-in-mind for details on how Git handles commit logs.
For the bias, if it's not used at all in HingeJoint but still inherited from PhysicsJoint, the proper fix would be to silently discard the params/bias value without showing a warning. Then the documentation could state that this property has no impact on HingeJoint. Whether it's better to also have the range [0;0.99], [0;1] or [0.01;0.99], I leave it to @AndreaCatania to tell us :)

@akien-mga

I will ammend the commit log with a clearer description.

It's easy to discard the warning (just won't generate it). But here's the problem:  The Inspectors displays the bias slider for generic Hinge Joints. It's just that the Bullet Hinge Joint does not support it.  So, if the user sees the slider and sets to a non-zero value and we swallow the warning, then she/he might wonder why nothing happens.  So I would leave the message in and allow for zero bias setting.  A zero value bias setting generates no message.

Waiting for input from @AndreaCatania

Sorry if I didn't answered before but to be honest I didn't do that because I waited to have the time to rework all implementation of joints in order to make it works correctly.
The problem was that I didn't had the time and I don't want block anymore this PR so @akien-mga this one can be merged, also @xemjeff Thanks you for your PR!

Now that Godot 3.1 has been released, we don't plan to cherry-pick new features and enhancements to the 3.0 branch, unless there is very strong support in favor of it. Removing cherrypick label for 3.0.


Swagger Annotations Help, Using Springfox

These are the method headers for the sample application I'm trying to make. Unfortunately I don't see the @ApiOperation or @ApiResponse or @ApiResponses annotations taking effect.
I also have a sample class and an Application class(which contains swagger configuration info and the like)
@Api(value = "/v1/", description = "Test API", produces = "application/json")
@RestController
@RequestMapping("/v1/")
class SampleRestController {

     @ApiResponses(value = {
            @ApiResponse(code = 200, message = "Successfully added"),
            @ApiResponse(code = 416, message = "List is empty") })
   @RequestMapping(produces = MediaType.APPLICATION_JSON_VALUE, value = "samples/{sb}",
   	method = RequestMethod.GET)
   public Map<String, String> sbs(@PathVariable("sb") String sb) {

	}

   @ApiOperation(
      value = "Seeing where this shows up",
      notes = "we should see this in the implememtation notes",
      response = Sample.class,
      httpMethod = "GET"
    )
        @ApiResponses(value = {
            @ApiResponse(code = 200, message = "Successfully added"),
            @ApiResponse(code = 416, message = "List is empty") })

   @RequestMapping(produces = MediaType.APPLICATION_JSON_VALUE, value = "samples/{pb}",
    method = RequestMethod.GET)
   public Sample ps(@PathVariable String pb) {

   }

   @RequestMapping(produces = MediaType.APPLICATION_JSON_VALUE, value = "samples",
   	method = RequestMethod.GET)
   public Collection<Map<String, String>> cs(@RequestParam(value = "cid", required = true, defaultValue = "")
    String cid) {

   }

}

Here is the relevant portion of my swagger json
{"swagger":"2.0","info":{"description":"All about the Samples","title":"Samples API","contact":{},"license":{}},"host":"localhost:8080","basePath":"/","tags":[{"name":"sample-rest-controller","description":"Sample Rest Controller"}],

"paths":{

"/v1/samples":{"get":{"tags":["sample-rest-controller"],"summary":"cs","operationId":"csUsingGET","consumes":["application/json"],"produces":["application/json"],"parameters":[{"name":"cid","in":"query","description":"cid","required":true,"type":"string"}],"responses":{"200":{"description":"OK","schema":{"$ref":"#/definitions/Collection«Map«string,string»»"}},"401":{"description":"Unauthorized"},"403":{"description":"Forbidden"},"404":{"description":"Not Found"}}}},

"/v1/samples/{pb}":{"get":{"tags":["sample-rest-controller"],"summary":"ps","operationId":"psUsingGET","consumes":["application/json"],"produces":["application/json"],"parameters":[{"name":"pb","in":"path","description":"pb","required":true,"type":"string"}],"responses":{"200":{"description":"OK","schema":{"$ref":"#/definitions/Sample"}},"401":{"description":"Unauthorized"},"403":{"description":"Forbidden"},"404":{"description":"Not Found"}}}},

"/v1/samples/{sb}":{"get":{"tags":["sample-rest-controller"],"summary":"sbs","operationId":"sbsUsingGET","consumes":["application/json"],"produces":["application/json"],"parameters":[{"name":"sb","in":"path","description":"sb","required":true,"type":"string"}],"responses":{"200":{"description":"OK","schema":{"type":"object","additionalProperties":{"type":"string"}}},"401":{"description":"Unauthorized"},"403":{"description":"Forbidden"},"404":{"description":"Not Found"}}}}},

I'm using springfox-swagger2 version 2.1.1 and I'm viewing my UI through the default UI provided by swagger at their live demo app.
Can someone tell me what I'm doing wrong?
Thanks!

That's really a question for springfox and not here.


Cron Backfill example doesn't work

I want to test this example https://raw.githubusercontent.com/argoproj/argo-workflows/master/examples/cron-backfill.yaml, , but unfortunately it doesn't work.
When I try to submit the CronWorkflow, I get this error :
spec.arguments.date.value is required
Here is the complete file,
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: duwan-test-job
spec:
  entrypoint: main
  arguments:
    parameters:
      - name: date
  templates:
    - name: main
      inputs:
        parameters:
          - name: date
            value: "{{inputs.parameters.date}}"
            # use "yesterday" is a magic value to tell the script it needs to determine the day to run
            default: yesterday
      script:
        image: docker/whalesay:latest
        command:
          - sh
        source: |
          date="{{inputs.parameters.date}}"
          if [ $date = yesterday ]; then
            date=$(date -d yesterday +%Y-%m-%d)
          fi
          echo "run for $date"
---
apiVersion: argoproj.io/v1alpha1
kind: CronWorkflow
metadata:
  name: duwan-test-daily-job
spec:
  schedule: "0/5 * * * *"
  workflowSpec:
    workflowTemplateRef:
      name: duwan-test-job

I also read this issue: https://github.com/argoproj/argo-workflows/issues/3807 and add entrypoint:
workflowSpec:
    entrypoint: main
    workflowTemplateRef:
      name: duwan-test-job

But it does not work too.
My argo version:
➜   argo version
argo: v3.1.10
  BuildDate: 2021-09-10T18:47:08Z
  GitCommit: 2730a51a203d6b587db5fe43a0e3de018a35dbd8
  GitTreeState: clean
  GitTag: v3.1.10
  GoVersion: go1.15.7
  Compiler: gc
  Platform: darwin/amd64

Had a look, I think the example isn't exactly correct.
Can you try the following
spec:
  entrypoint: main
  arguments:
    parameters:
      - name: date
        # use "yesterday" is a magic value to tell the script it needs to determine the day to run
        value: yesterday
  templates:
    - name: main
      inputs:
        parameters:
          - name: date
            default: "{{inputs.parameters.date}}"
      script:
        image: docker/whalesay:latest
        command:
          - sh
        source: |
          date="{{inputs.parameters.date}}"
          if [ $date = yesterday ]; then
            date=$(date -d yesterday +%Y-%m-%d)
          fi
          echo "run for $date"

@tczhao Thank you! It works.
So this example is wrong: https://raw.githubusercontent.com/argoproj/argo-workflows/master/examples/cron-backfill.yaml


Scaling book reference cleanup

I missed references to the original home of the scaling book, https://github.com/bitcoinops/scaling-book, in reviewing https://github.com/bitcoinops/bitcoinops.github.io/pull/1830 (apologies!)
This commit removes the most obvious version of that in about.md but there are 3 more (sans localizations):

_posts/en/2018-12-28-2018-optech-annual_report.md
_posts/en/2019-02-11-rbf-in-the-wild.md
_posts/en/newsletters/2019-12-28-newsletter.md

If we plan to leave the scaling book repo, these references are probably fine to leave. Feedback welcome.

I think we should mark the repository as archived and remove the reference from en/about.  I think we should leave the blog and newsletter references, as that's clearly dated content, so it's ok if its out-of-date.

Marked bitcoinops/scaling-book as archived.


onyo get needs to differentiate between different none-value cases

onyo get needs to allow to query, and display differently:

key exists but without value
key does not exist at all

We currently don't allow empty values. So we don't need to be able to query for that.
Not existing keys can be queried via <unset>:
onyo get --match dne=<unset>
As of now, this is documented in get's help.


When will I be able to see an solution of example?

Hi. I am enjoying your mastering python book.
When will I be able to see an solution of example?

I've been working on it but it's taking quite a bit of time and I'm very limited in time right now. If you have any specific exercises that you want a solution for, please let me know. That should be much faster to arrange :)

Can I get the solution from Ch.13(asyncio) and Ch.14(multiprocessing) first? Thank you for answer.

I've added some example solutions for those chapters :)
You are welcome (and encourages) to fork and add your own version with improvements.


Add trim-left-pattern support to no-consecutive-blank-lines rule

Bug Report
Filing on behalf of @akoidan.

TSLint version: 5.15.0
TypeScript version: 3.4.0
Running TSLint via: CLI

~TypeScript~ Vue code being linted
<template>

asd

</template>

with tslint.json configuration:
{
    "rules": {
        "no-consecutive-blank-lines": true
    }
}

Actual behavior
Quoting #4643:
no-consecutive-blank-lines doesn't work in .vue files since to enable sourcemaps vue-loader will leave blank spaces in <template> section and this will never change.
Expected behavior
There should be a way to ignore these preceding blank lines in .vue files, since the rule is basically unusable at this point.
However, no-consecutive-blank-lines is no longer a priority for TSLint per #4534; the recommendation is typically to use Prettier.
Same discussion as #4646. What's the best way to handle this?

@akoidan does prettier work for vue files? is there anything blocking you from using that for code formatting?

Why would I use different formatters for vue files and ts files? I want tslint and tslint-microsoft-contrib rules, not prettier limited set of rules.
As for now I ended up with adding my customs rules to every project. I also don't know what's wrong with that PR that I created that fixes those things. It doesn't break existing functionality, it comes with tests, and it doesn't break any programming principles I'm aware of.
Am I the only one facing that issue, does none use vue with typescript? Or no one using tslint? I guess if this PR is rejected I will just ship a separate npm package with custom tslint rules for vue file. That would be ridiculous though.

Why would I use different formatters for vue files and ts files? I want tslint and tslint-microsoft-contrib rules, not prettier limited set of rules.

I'm not suggesting different formatters, but rather switching to prettier for all file types because TSLint's formatting rules are essentially deprecated... or, you could use tslint-plugin-prettier + tslint-config-prettier + whichever tslint formatting rules do not conflict with prettier (very few, I imagine).

Am I the only one facing that issue, does none use vue with typescript? Or no one using tslint?

Good question, have you checked out https://github.com/palantir/tslint/issues/2099?

Per #4379, a Vue-specific workaround isn't going to be able to land in TSLint core:

Given the lack of discussion or concrete proposal here and the upcoming deprecation of TSLint in favor of ESLint (#4534), I'm going to go ahead and close this issue and related Vue-specific reports for housekeeping purposes.
If this is still an issue for you in typescript-eslint, I'd recommend filing an issue there. Best of luck!


TaskRun results in 'Defaulted container "step-echo" out of: step-echo, place-scripts (init), place-tools (init)'

Expected Behavior
I am trying to deploy a RobotFramework task from tekton HUB:
https://hub.tekton.dev/tekton/task/robot-framework
Running the task as:
kubectl apply -f robot-task.yaml
kubectl apply -f robot-task-run.yaml

kubectl logs returns:
kubectl logs --selector=tekton.dev/taskRun=robot-task-run
Defaulted container "step-robot-tests" out of: step-robot-tests, place-scripts (init), working-dir-initializer (init), place-tools (init)

Deleted all the crud from yaml files and tried bare bones, no luck. I am guessing the issue is with the container, so I tried 'alpine' 'docker.io/python:3.10.5', 'python', I always get that 'Defaulted container' error. Tried both work OCP cluster as well as local minikube.
The pod status is 'Pending':
NAME                       READY   STATUS      RESTARTS   AGE
hello-task-run-pod-7xwmw   0/1     Completed   0          15m
robot-task-run-pod-snp7t   0/1     Pending     0          4m9s

This goes on for 1 hr and it Times Out.
Actual Behavior
Steps to Reproduce the Problem

Install the task using the above URL: kubectl apply -f
Run the task: kubectl apply -f <run-file.yaml>
Check the taskrun log and the pod status:  eg

kubectl logs --selector=tekton.dev/taskRun=robot-task-run
kubectl get pods

Additional Info

Kubernetes version:
Output of kubectl version:

Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.2", GitCommit:"f66044f4361b9f1f96f0053dd46cb7dce5e990a8", GitTreeState:"clean", BuildDate:"2022-06-15T14:14:10Z", GoVersion:"go1.18.3", Compiler:"gc", Platform:"darwin/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.3", GitCommit:"816c97ab8cff8a1c72eccca1026f7820e93e0d25", GitTreeState:"clean", BuildDate:"2022-01-25T21:19:12Z", GoVersion:"go1.17.6", Compiler:"gc", Platform:"linux/amd64"}

- Tekton Pipeline version:

**Output of `tkn version` or `kubectl get pods -n tekton-pipelines -l app=tekton-pipelines-controller -o=jsonpath='{.items[0].metadata.labels.version}'`**

Client version: 0.24.0
Pipeline version: v0.24.3
<!-- Any other additional information -->

hi @95rade thanks for filing! I think the message Defaulted container "step-robot-tests" out of: step-robot-tests, place-scripts (init), working-dir-initializer (init), place-tools (init) that you are seeing is not really an error-- this just means that you asked kubernetes for the logs in some pod but didn't specify which container in that pod you wanted logs from, so it chose one for you. (In this case, the container running a step instead of the init containers injected by tekton.)
It looks like your pod hasn't produced any logs because it is still running. Is this something you expect to take a long time before producing output? If so, you may want to just increase the timeout. If not, you might want to take a look at the taskrun's status to see what's going on. the Tekton CLI might make that a bit easier!

Hi @lbernick and thanks for the comment. The task in question has only one step so there should only be one container running in that pod. The pod is stuck in 'Pending' state and the timeout is set to 1hr which is enough. The expectation is that when this task runs it should scan rf-test directory for any .robot files and when it finds them execute them. Then stop. This is not happening. If you could please take a look at the task (link at the top) and help me figure out whats broken with it I would be grateful. I am a tekton newbie coming up to speed.  thanks a lot

The container step-robot-tests is the container for the step defined in your Task. The other 3 containers are part of the Tekton machinery and are automatically added, this is expected. As Lee stated above, the 'Defaulted container' message is not an error, it is normal behaviour when you request logs from a Pod with multiple containers without specifying the container name.
If you inspect the pending pod you should find more information to explain why it is pending, for example kubectl describe pod robot-task-run-pod-snp7t and check the events field. The Kubernetes documentation contains some useful information for debugging pods that are stuck in pending state, including links to additional resources that may be relevant: https://kubernetes.io/docs/tasks/debug/debug-application/debug-pods/#my-pod-stays-pending

As commented, this is not a bug, but just the way tekton reports its output.


期待大佬的开源

期待大佬的开源

Code is now up, feel free to browse and let me know if anything is faulty/missing!
Best,
Jim

Code should be available in about 2 weeks, thank you for your patience and your interest in this work!


handle warnings from temperature model tests

pvlib python pull request guidelines
Thank you for your contribution to pvlib python! You may delete all of these instructions except for the list below.
You may submit a pull request with your code at any stage of completion.
The following items must be addressed before the code can be merged. Please don't hesitate to ask for help if you're unsure of how to accomplish any of the items below:

[x] Closes  #795
[x] I am familiar with the contributing guidelines.
[ ] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests (usually) must pass on the TravisCI and Appveyor testing services.
[x] Updates entries to docs/sphinx/source/api.rst for API changes.
[x] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.
[ ] Code quality and style is sufficient. Passes LGTM and SticklerCI checks.
[x] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[x] Pull request is nearly complete and ready for detailed review.

Move one warning to avoid raising two warnings for the same cause

It all passed on py35_min, so I'll merge.


PHP: change the php ssl server credentials API so that a server can have multiple key/cert pairs

This fixs #1122.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Let's do that last commit add TSRMLS_CC param here in this PR. Please remove it.

OK

please see 8044

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.


Empty render in IE7

Hi there,
I am using hogan.js compiling on the fly from templates stored in a script tag
<script type="text/html" id="template-id">{{hulk}}</script>

This works fine everywhere except ie7 where it just renders all the templates blank. I can see hogan is getting the html from inside the script tag but once it comes to template.render(), nothing!
Thanks

Why not write a gist showing the problem. With this report, it's hard to tell whether Hogan has a bug.


When creating a new API key, default space should be Company Data

It is currently the first space by alphabetical order, for us Abboud's vault

(also mentions vault vs space but I presume will be tackled as part of the Vaults -> Space project)


Add json parser

WIP

Reverted the removal of the gemfile so that circle picks up the dependencies without me having to run gem install --dev


Fix vowelCount method

[x] Running yarn lint does not trigger any linter warnings
[x] The test of the method you have fixed is passing
[] You have written a new skeleton method for someone else to work on!

Hey @ripzery  ! Thanks for your contribution- we are stoked to have you on board. Unfortunately your pull request does not meet all of the criteria of our Contributing Guide. We will be closing out the pull request for the time being. Please follow the guidelines set out in CONTRIBUTING.md and reopen the pull request! Thanks!!


Pyautogui cannot run after screen lock

I wrote a program to automatically lock and unlock the screen for windows 10,
However, I found that when unlocking the input password, the mouse positioning failed, but the keyboard input "enter" was valid
help! !  !

Can you share your code?

I wrote a program to automatically lock and unlock the screen for windows 10,
However, I found that when unlocking the input password, the mouse positioning failed, but the keyboard input "enter" was valid
help! ! !

I have some kind of same situation. I am writing program to control my chrome. My PyAutoGUI-based program is running on a machine which is accessed remotely from another computer (my main computer) by RemoteDesktop, when I minimized the RemoteDesktop window on my main computer, the pyautogui-program jumps into error, saying "PyAutoGUI fail-safe triggered from mouse moving to a corner of the screen. To disable..."

i got the same problem like @NgJun did you figure smth out by now?


fetchgit: add tag argument

Motivation for this change
While it is possible to pass rev = tag to fetchgit, it may result in ambiguity when here is a branch with the same name as the tag. To be sure, we need to use rev = "refs/tags/${tag}".

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Well thats one upside of not having documentation I guess. Nothing to update.
Should we then do the same thing with fetchFrom...? Would be good for consistency and I feel like those are used more often than fetchgit (although a grep through the repository tells me otherwise).

Added the argument to fetchFromGit{Hub,Lab}.
Needed to escape the rev in fetchFromGitLab to avoid errors like:
building '/nix/store/cxfkh2lgfbhhpgcphsh8wkn2j25y50hn-source.drv'...
trying https://gitlab.gnome.org/api/v4/projects/gnumdk%2Flollypop-portal/repository/archive.tar.gz?sha=refs/tags/0.9.7
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 15537  100 15537    0     0  15011      0  0:00:01  0:00:01 --:--:-- 15011
unpacking source archive /build/0.9.7
do not know how to unpack source archive /build/0.9.7
builder for '/nix/store/cxfkh2lgfbhhpgcphsh8wkn2j25y50hn-source.drv' failed with exit code 1

Also I am not sure about what should the rev introduced with https://github.com/NixOS/nixpkgs/pull/9273 and https://github.com/NixOS/nixpkgs/pull/15139 be. I used refs/tags/${tag} for now.

While it is possible to pass rev = tag to fetchgit, it may result in ambiguity when here is a branch with the same name as the tag. To be sure, we need to use rev = "refs/tags/${tag}".

AFAIK tags have precedence over a branch refs https://git-scm.com/docs/git-rev-parse.html#_specifying_revisions, i.e. rev = "refs/tags/${tag}" should not be necessary

Then I do not understand why refs/tags/${version} is so frequent in nixpkgs.

Not really in favor of this because it makes fetchgit even more complex, and adds another inconsistency with builtins.fetchGit.


passing parameters

Is there a way to pass the voice-recognizer parameters? Basically I want bypass the mic and feed  it text instead of speech.  This might sound vague but I can be more specific if need. I just want the recognizer to take text as well as speech.

There's no built-in support for this, but you can modify the code to call [actor.handle](https://github.com/google/aiyprojects-raspbian/blob/487522740728d3a646c1487faf40efca58473da1/src/main.py#L390) with your own text.
Does that help?

yeah it does thanks for the suggestion

Glad to hear it!


Move model card info to readme and model-information.csv

model cards now in model-information and model directory readme
closes https://github.com/NVIDIA/Morpheus/issues/19

Can one of the admins verify this patch?


unknown property targets

Could not set unknown property 'targets' for Kotlin/Native test suite 'test' of type org.jetbrains.kotlin.gradle.plugin.experimental.internal.KotlinNativeTestSuite.

targets throw error, target don't.


Updated German translations

I, as a German native speaker, reviewed all the translations and added some new where some were missing. Some translations/terms were not consistent, so I smoothed these out. Hope this helps.

Thanks @heartcrazed! 🙂


add link to the tour guide application doc from the introduction page

Since the application has been separated from the tour guide and these pages have been moved to a separate GitHub repository this issue is no longer relevant.


help Installing t2t-trainer via pip says no matching distribution

Hi! I am trying to install the Tensor2Tensor framework, running pip install tensor2tensor --user installs it without issues, but then running pip install t2t-trainer --user says
" Could not find a version that satisfies the requirement t2t-trainer (from versions: )
No matching distribution found for t2t-trainer"
How to install t2t-trainer? Thank you!

The issue was caused by a lack of permissions and improper path settings on the machine I was using.


[Bug] [MasterServer] NPE in netty processor

Search before asking

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

What happened
[ERROR] 2021-12-10 16:36:58.596 org.apache.dolphinscheduler.remote.handler.NettyClientHandler:[156] - process command Command [type=TASK_EXECUTE_RESPONSE, opaque=13, bodyLen=130] exception
java.lang.NullPointerException: null
	at org.apache.dolphinscheduler.server.master.processor.TaskResponseProcessor.process(TaskResponseProcessor.java:72)
	at org.apache.dolphinscheduler.remote.handler.NettyClientHandler.lambda$processByCommandType$0(NettyClientHandler.java:154)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

What you expected to happen
no error
How to reproduce
just run a process instance.
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

This issue was fixed.


[ENH] add python2 support

I think keeping the code backward compatible will be a plus.

Thanks! Keeping py27 compatibility is absolutely crucial!

Thanks! Merging!


Is border effect works for API version below 21?

@swasti29
I'm sorry, but this library don't support  Api below 21.

Thanks for the quick response. In future any support for below API 21 ?

@swasti29
Sorry, for now I don't have any plans to support Api21.


AsyncParsableCommand exits before running run() async

When using an AsyncParsableCommand with @main, the command-line-tool exits without running the run() async function and prints the help log.
Note: run() without async runs the function.
ArgumentParser version: 1.2.0
Swift version: swift-driver version: 1.45.2 Apple Swift version 5.6 (swiftlang-<IP_ADDRESS>3.62 clang-13<IP_ADDRESS>)
MacOS 12.6
Checklist

[x] If possible, I've reproduced the issue using the main branch of this package
[x] I've searched for existing GitHub issues

Steps to Reproduce
Option A: Reproduce by running the Example in this repository
Option B:

Expected behavior
The run() async function should run
Actual behavior
The run() async function isn't run and the program exits with the help log.

As another example of what works and what doesn't. Running it from command line:
This works:

This doesn't work:

I'm using Swift 5.6, so I need to create a standalone type as your asynchronous @main entry point.

Reopening it as the documentation of AnycMainProtocol indicates to use @main directly as I was using it which causes the issue above.

I ran into this or at least a very similar issue as well. I did not use the main functions provided by ArgumentParser but invoked run() directly.
After looking a bit deeper, I noticed that the default implementation of ParsableCommand.run() is executed, instead of AsyncParsableCommand.run().
Reproducible with the following snippet in a main.swift file or playground:
import ArgumentParser

struct Foo: AsyncParsableCommand {
    mutating func run() async throws {
        print("Foo")
    }
}

// Like AsyncParsableCommand.main()
do {
    var cmd: ParsableCommand = try Foo.parseAsRoot()
    if var asyncCmd = cmd as? AsyncParsableCommand {
        try await asyncCmd.run()
    } else {
        try cmd.run()
    }
} catch {
    Foo.exit(withError: error)
}

The Foo.run() function is never executed. If you set a breakpoint at the default implementation in ParsableCommand.run() that will be hit instead.
Note that the compiler warns that the await expression is useless, since no async operations occur within.
The underlaying problem seems to be that Foo has two run() functions, see simplified:
struct Foo {
    func run()
    func run() async
}

This overload behaviour was introduced in SE-0296, see specifically the section Overloading and overload resolution, and also discussed in the Swift forum. The overloads are called based on their context. If you are in another async function, you get the async overload, if you are in a synchronous context, you get the non-async overload.
Seems like the main context in main.swift allows both, which can quickly be confirmed by adding a non-ambiguous async function:
extension AsyncParsableCommand {
    mutating func runAsync() async throws {
        try await self.run()
    }
}

do {
    var cmd: ParsableCommand = try Foo.parseAsRoot()
    if var asyncCmd = cmd as? AsyncParsableCommand {
        //try await asyncCmd.run()
        try await asyncCmd.runAsync()
    } else {
        try cmd.run()
    }
} catch {
    Foo.exit(withError: error)
}

Warpping everything into an async function works as well, which is basically what AsyncParsableCommand.main() does. For ParsableCommand there is no conflict and the synchronous function will be run.
There is a similar effect for the main function. Looking at the @available annotations in this project's code and the comments in this thread, it seems like macOS 10.15 fixed the @main wrapper, so that it properly calls the async main function. However, if you are using main.swift and invoke main directly, the synchronous main function is preferred, which also triggers failAsyncPlatform() (with a less helpful error message). So you would need to wrap it into an asynchronous function explicitly:
func main() async {
    await Foo.main()
}
await main()

Or disambiguate the overload:
extension AsyncParsableCommand {
    static func mainAsync() async throws {
        try await main()
    }
}
await Foo.mainAsync()


Issue with 5k List upload

This list is failing to upload to OS hub staging.
It's been checked for new line characters and any fields longer than 200 characters have been removed. Any idea why it's failing?[Uploading... WB_testList_sectors-removed.csv]

Two questions/requests:

It looks like you tried to attach a file to the issue description but the Github attachment process did not complete.
What does the failure look like?

Thanks.


Migrate away from Tox to UV with Just

Switching to UV and Just in favour of Tox due to speed and a long-term plan to improve CI. This PR is a derivative of:

https://github.com/canonical/mimir-coordinator-k8s-operator/pull/90

which uses Make instead of Just.
To test with this PR you can run the following commands:

tox -e lint -> just unit
tox -e static -> just static
tox -e fmt -> just fmt
tox -e unit -> just unit
tox -e scenario -> just scenario
tox -e integration -> just integration

Want to know which recipes you can run?

just
Want to run multiple recipes?
just lint static fmt
Want to run all fast recipes at once?
just all-fast
Want to use args?
just unit --help
just integration --keep-models

CI for charm-pull-request.yaml is being updated to conditionally test with just instead of tox depending on the existence of a tox.ini file in the repo root.
Tandem PR:

https://github.com/canonical/observability/pull/229

Currently missing:

[x] scenario tests
[x] colour formatting in GH CI output
[x] requirements.txt management
[ ] Can we consolidate the charmcraft.yaml charm-binary-python-packages since we have a pyproject.toml (single source of truth)
[ ] Clean positional-args in just recipes
[ ] Update charm-pull-request.yaml to support just in addition to tox and make

Closing in favour of the Make PR:

https://github.com/canonical/mimir-coordinator-k8s-operator/pull/90


Import Image modules form the module PIL for Pillow interoperability

Sorry for the 5 commits but I used github web interface.

Hi, sorry this wasn't merged in when you submitted it, but this library is being deprecated. We're now using https://github.com/adafruit/Adafruit_CircuitPython_RGB_Display. You can find more info in our learn guide:
https://learn.adafruit.com/adafruit-2-dot-8-color-tft-touchscreen-breakout-v2/python-wiring-and-setup


Angat branch 2

-ClearDB integrated
-Logout Fixed
-Search Hospital and Hospital Profile added
-Invalid Credentials Fixed

user req ClearDB not yet integrade


Added support for extracting images from Webpack assets stats

This PR adds support for extracting images processed by Webpack, related to #18

Added images array to resulting JSON which contains name and path for images processed by Webpack
Wrapping extracted chunks with static assets into chunks object to separate entry points related data from images array

Since generated JSON structure changes I suggest bumping major version if this PR gets merged into master.

@dimchez great stuff!

Perfect!

@sporto could you add me to npm package owners? My username there is kossnocorp.

@kossnocorp 	should be done now

@sporto thank you a lot!
@dimchez give me few days to test everything and I'll publish the new version.

Whats the progress here?

I tested the current master and I'm not quite sure, how this is supposed to work. From the current index.js:
output.assets = stats.assets.filter(function (asset) {
  return self.options.assetsRegex.test(asset.name)
}).map(function (asset) {
  return { name: asset.name, path: assetPath + asset.name }
});

What happens, if we use something like [hash]  inside our file loader name parameter? The asset name and path would be the same containing the hash except for the suffixed asset path.
This way it is very hard to parse the JSON manifest file on the other side without extra configuration for parsing the file name and excluding the hash for example.
Sample output from my setup:
{
  "entries": {
    "main": {
      "js": "main-[sha512:hash:base64:7].js",
      "css": "main-[sha512:hash:base64:7].css"
    }
  },
  "assets": [
    {
      "name": "/images/3qSDcel.jpg",
      "path": "/images/3qSDcel.jpg"
    }
  ]
}


Use Service Fabric Reliable Collection for Identity server

Hi Team,
We are using Service Fabric(.Net Framework) for our application, so we are in a plan to migrate Identity Server Web application to Service Fabric Stateful service with Reliable collections. Can you provide necessary steps to migrate the database tables to Reliable collections and necessary steps to migrate the web application.

No, sorry. We don't have any information on that.


[FEATURE] Support horizontal scrolling

Conditions:

All widths must be defined on all visible columns
Widths must be the same unit

@taras I would love your feedback on this one

How complete is this implementation? I guess it just uses regular scrollbar?

Its very naive. It works well but just uses the native scrollbar.

Hi, any idea when this PR going to be merged into master ?

I've been waiting on @alexander-alvarez to see if he can find a workable solution using ember-scrollable. Maybe I can release this as a temporary solution until then.

progress has been made... about to attempt to do the double scrolling - https://github.com/alphasights/ember-scrollable/pull/29

@alexander-alvarez thats great to hear! Looking at the PR, it seems like your solution will take a bit of work and some time until it gets reviewed / merged. I think Im gonna go ahead and merge this so that way there is no real pressure to get a final solution in.

Thanks, it works great and sufficient for our need right now. I look forward to check ember-scrollable solution though.

Actually what I eventually did is setup an event handler for scroll on the body scrollable-content div with jQuery in the didRender hook of my component.  I then simply modify the scrollLeft of the .lt-head-wrap to match the scrollLeft of the ".lt-body-wrap .scrollable-content" . It works pretty well, be you need to enable overflow-x:auto on the body content and set an id to the light-table component.

Does it work? I have header and footer fixed both and long header, but I can't scroll and they are overlaping each other.

@meliborn yes its works. See here

@meliborn Make sure all your columns specify a width and all width are the same unit.

Hi @offirgolan, the horizontal scroll works fine when the element containing the table is narrower than the table, but as soon as the container is wider than the table the table doesn't fill up the container. I guess it would be nice if the table width matches the container width if the container is wider than the table width.
Would it be possible ?

@offirgolan What should I do to enable horizontal scrolling? Can't find any specific code in provided example.

Had to add .table-container. But there is a problem with pagination - it scrolls as well, even with fixed=true.

@meliborn can you open a new issue with some screenshots and maybe a reproduction?


Add Github Actions CI badge and Codecov

As a developer
I need to add a Github Actions CI badge and Codecov badge to README.md
So that everyone will know if the build is broken
Details and Assumptions

Users can click the CI badge and see the build progress
Can click the codecov badge to see codecov report
Details on how to do this given in the lecture on CICD. Can copy the code from the example repo used in that lecture and just change the hyperlinks.

Acceptance Criteria
Given we have set up the CI badge and codecov badge on our GitHub repo
When our automatic CI tests fail with coverage 90%
Then the CI badge should read failing and codecov badge should read 90%
When our automatic CI tests pass with coverage 95%
Then the CI badge should read passing and codecov badge should read 95%

Added codecov badge in addition to GitHub CI Build badge


refactor: Fix api fuzzer and refactor tests

Description
This PR fixes the api fuzzer and refactor some tests.  It is ready to be reviewed however pls dont merge it until #247 is fixed or it will break the CI
Closes #238
Closes https://github.com/input-output-hk/catalyst-voices/issues/49
Description of Changes

Fix docker compose for the api fuzzer to make it wait for the containers to be healthy
Remove integration tests example as now we have some actual tests
Move the test-open-api-linter to the tests Earthfile to keep the test targets in one place
Pin earthly dnd image to alpine 3.19
Move catalyst-core-package to the main earthfile

Breaking Changes
Describe any breaking changes and the impact.
Screenshots
If applicable, add screenshots to help explain your changes.
Related Pull Requests
If applicable, list any related pull requests.
Please confirm the following checks

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream module

@kukkok3 Sorry, just noticed it right now 😊. What do you think to rename tests dir to integration_tests ? So explicitly emphasise that here we have just integration tests.

I dont see any advantage in to renaming the folder tests. The folder tests is a special folder in rust and the compiler knows there are only tests in there. It is already the standard folder where to put integration tests. Also if in the future we want to put some other kind of tests other then integration-tests in there we need to change the folder name back to tests


Add the ability to select NEURON branch for jenkins CI

This ticket is about selecting a specific NEURON branch in the Jenkins Pipeline CI of CoreNEURON based on the PR description.
Something similar is done in the SimulationStack CI here: https://bbpcode.epfl.ch/code/#/c/51178/
The file that configures Jenkins CI and needs changes is: https://github.com/BlueBrain/CoreNeuron/blob/master/tests/jenkins/Jenkinsfile

I think this was done by https://github.com/BlueBrain/CoreNeuron/pull/539/commits/c8809367e759a2ebc2eb1edff20ea7ef426bd2f7, but in any case the Jenkins CI should be decommissioned soon.


chore(main): release 0.12.0

:robot: I have created a release beep boop
0.12.0 (2024-11-18)
Features

add header / footer and pagination to pages (23fdc2a)
add support to postgrest (00be8a6)
allow isAPI global variable to handle 404 errors (b6ef99c)
cronjobs + fix database.json.sqlite.sampl (88011f6)
Dockerfile is working (6fcbaff)
improve controller performances by 50% (84ecc96)
improve controller performances by 50% (46fbfb7)
improve controller performances by 50% (bc68813)
LogError() (3e813ff)
PostGrest => remove upsert for now (f73b519)
WIP PDFGenerator (4f39d38)

Bug Fixes

add controllers requirable (f1f1d13)
add header / footer and pagination to pages (e5abe92)
add header / footer and pagination to pages (cf328b3)
allow POST on / in the CustomRoutes (3a004bf)
controller template (8561f98)
enhance tables (cefcbf1)
HandleSqliteFork (042ce81)
pdf => set metrics for Helvetica-Bold (08986f7)
PDF: add textCOlor to cells (4dc4667)
PDF: custom & default font fixes (a87ba35)
PDF: provide a pdf usage sample (43d213d)
PDF: provide a pdf usage sample (96368cd)
PDF: provide an easy way to update it for lpais (d1af8d6)
PDF: put cursor_y after image (1ed1702)
remove log file (7ad46be)
rename DefineRoutes method (54c714e)
restore controller specs (f94a4e5)
restore controller specs (cda66fb)
restore controller specs (8c54443)
test mode (cb67dc3)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/solisoft/luaonbeans/releases/tag/v0.12.0 :sunflower:


support different protocols specify different handler chains

support different protocols specify different handler chains

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 1909

3 of 11 (27.27%)  changed or added relevant lines in 2 files are covered.
20 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.07%) to 66.847%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

chassis.go
2
10
20.0%

Files with Coverage Reduction
New Missed Lines
%

eventlistener/loadbalance_event_listener.go
2
66.67%

core/options.go
2
85.96%

core/handler/tracing_handler.go
3
0.0%

core/lager/logrotate.go
13
38.53%

Totals

Change from base Build 1905:
-0.07%

Covered Lines:
5815

Relevant Lines:
8699

💛  - Coveralls

thanks for that， that is good feature
but 2 things：

write documents https://docs.go-chassis.com/user-guides/handler-chain.html#
UT can not drop down anymore

thanks for that， that is good feature
but 2 things：

write documents https://docs.go-chassis.com/user-guides/handler-chain.html#
UT can not drop down anymore

ok,i  will finish it tomorrow

awesome job, thanks


[es-dev-server]: API fallback middleware does not handle url encoded parameters

I have a parameter in the path part of the URL that is an URL encoded URI. The encoded value has a dot in it (in .dom) that is interpreted as an extension. This causes the dev server to request for the next middleware while this should be handled by the fallback API.
Example full URL:
http://localhost:8000/demo/project/http%3A%2F%2Fmulesoft.com%2Fmodeling%2Finstances%2Fd8ca6956-1a8d-40bc-af74-55bd4bd84f24

I'm not sure what we can do here. The idea of history API fallback, or SPA routing, is to return the index.html file on route requests and technically speaking this is a file request. Checking for the file extension is an important part of the consistency, there aren't many other checks we can do.

I've made this configuration that helps me with that. Maybe you can use it in the dev sever:
const fs = require('fs');
const path = require('path');

module.exports = {
  watch: true,
  nodeResolve: true,
  appIndex: 'demo/index.html',
  plugins: [],
  moduleDirs: ['node_modules'],
  middlewares: [
    function rewriteIndex(ctx, next) {
      // whether it starts with "cfg.appIndex"
      if (!ctx.url.startsWith('/demo/')) {
        return next();
      }
      // test if file exists
      const test = path.join('./', ctx.url);
      const exists = fs.existsSync(test);
      if (exists) {
        return next();
      }
      // use the fallback
      ctx.url = '/demo/index.html';
      return next();
    }
  ]
};

This will cause HTML to be served when a file doesn't exist, hiding the actual 404.

Isn't that the goal of having fallback for the history API?

I think you want this behavior for route requests like /demo/foo and /demo/bar but not for others. But maybe that's just my assumption. We could check how other tools are handling this and align the logic.

When using browser's history API (for example window.history.pushState({}, null, '/project/something')) the path is being rewritten (I have logic inside the app that handles the /demo/ case). This means that any URL that is not requesting an actual file has to go to the main file for the routing to ever work. Here I don't rely on server handling 404 as I have a 404 page inside the application that is rendered when a route doesn't match.
Take a look into lit-element-router module that uses a similar pattern to handle 404s on the client side.


Map Fixer is not applying a fix found in the moves database

The map has a datasource in /a/c/GNIS.gdb.  The moves database says that /a/b/c/gnis.gdb has moved to /1/2/3/gnis.gdb.  The map is not fixed.
The check for workspaces and data source names needs to be case insensitive.

Fixed with a112b8ca726bd93527813ef86fad3533583e67b1


🧩  Cannot find module '@demo/routes'

Discuss anything related to Knip
$ git clone https://github.com/taiga-family/taiga-ui.git
$ npm ci

$ npx -y knip
Analyzing workspace projects/demo-playwright...
ERROR: Error loading /Users/m.ivanov4/IdeaProjects/taiga-ui/projects/demo-playwright/playwright.config.ts
Reason: Cannot find module '@demo/routes'
Require stack:
- /Users/m.ivanov4/IdeaProjects/taiga-ui/projects/demo-playwright/tests/demo/get-demo-paths.ts

This sounds more like a candidate for a bug report?
https://github.com/webpro-nl/knip/issues/new/choose


Migration to DSharpPlus

https://github.com/DSharpPlus/DSharpPlus
This seems to be better-maintained than Discord.Net, so it might be worth jumping ships.

Neeeevermind, there's https://github.com/Discord-Net-Labs/Discord.Net-Labs/tree/Interactions


join us form enhanced

Issue that this pull request solves
Closes: #337 (issue number)
Proposed changes
Brief description of what is fixed or changed
Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface

Other information
Any other information that is important to this pull request
This PR is under DWoC.

Hey @vedikaag99 please make the checkbox list row-wise, similar to year of study

Is this okay? @Kajol-Kumari

Is this okay? @Kajol-Kumari

Looks good! Try putting the option poster Designing  just after Video editing so that it can fit in the row and look better.

done @Kajol-Kumari please review now.

Hey @vedikaag99 can you please fix the submit button size and year of Study section?

Hey @vedikaag99 can you please fix the submit button size and year of Study section?

What needs to be fixed for the year of Study section?

just my thinking : I think, There is a little bit more space between checkbox and text . Under the year of study

@Kajol-Kumari done

@vedikaag99 please fix the width of submit. Rest looks good to me

Do I make it even smaller ? @Kajol-Kumari

@vedikaag99 please fix the width of submit. Rest looks good to me

Do I make it even smaller ? @Kajol-Kumari

It's width is appearing too small. we need to keep the same width as it was before.

Do i increase it even further? @Kajol-Kumari

Do i increase it even further? @Kajol-Kumari

It's not appearing this way when I tested this PR. Can you please check if you have the latest pull?

@Kajol-Kumari Ah sorry! I wasn't working on the latest pull. Well now I have pulled the latest changes but I am having some trouble in pushing my code. Hence I am closing this PR and opening a new one. Sorry for the inconvenience caused!


In out count

How can i be able to get the vehicle in and vehicle out count?

You should work on object_counting_mode.
The current version of the TensorFlow Object Counting API does not provide in/out count but new version, that will be published soon, will provide!


housekeeping: added some instructions on how to manually build the website

Great call - without the token the source download will fail, probably confusing most people when the build fails.

Thanks!


Convert many VirtualFile APIs to async

Problem
VirtualFile does both reading and writing, and it would be nice if both could be converted to async, so that it doesn't have to support an async read path and a blocking write path (especially for the locks this is annoying as none of the lock implementations in std, tokio or parking_lot have support for both async and blocking access).
Summary of changes
This PR is some initial work on making the VirtualFile APIs async. It can be reviewed commit-by-commit.

Introduce the MaybeVirtualFile enum to be generic in a test that compares real files with virtual files.
Make various APIs of VirtualFile async, including write_all_at, read_at, read_exact_at.

Part of #4743 , successor of #5180.
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

This failure is legitimate (I can reproduce it locally):
thread 'virtual_file::tests::test_virtual_files' panicked at 'assertion failed: file_a.read_string().await.is_err()', pageserver/src/virtual_file.rs:680:9
stack backtrace:

I don't know where the problem is from, as the OpenOptions are being passed through, so the file should theoretically be open as write-only. Need to investigate this more starting Monday.

To expand on why I did the last commit with read_to_end, bringing the discussion out of DMs:

The test failure one would run into at the start would be that read_string on a write-only file would succeed because the cursor would be at the end of the file, and thus there'd be zero data to read. The code would do a read call with zero size, and there is an optimization somewhere that lets that read call return an empty buffer without actually issuing a read system call (you can see above that there is no read system call happening). I first "worked around" this issue by seeking to the start of the file before doing read_string, but this is just changing the test to accomodate for behaviour changes, so not a good idea: the regression would still be there. Instead, now we do what the file API is also doing: read a non-zero piece of data, which then can still return 0, if nothing is available, but more importantly, it does a system call and thus yields the expected error.
VirtualFile has a pos field. The code before the last commit recalculated pos via performing a seek operation, which is sub-optimal. Ideally we'd use the already existing position tracking.
More importantly, read_string on native files affects the position inside the file, while our read_string implementation wouldn't. This was because we used read_string_at which doesn't update pos. Therefore, the test would fail where it first expected the read to return "foobar" and then to return "" (with us returning foobar both times). The read_to_end implementation now correctly updates pos (with a fine granularity even, so in error conditions we also get similar update behaviour of our cursor).


Multi-project example

Your plugin was working great until I moved to a multi-project layout. It seemed to not aggregate the jacoco reports and send each individually, so only one module was shown. I fixed this by having the reports appended and stored at the root project's build directory. The html report from jacoco comes out great.
Coveralls unfortunately can't find the source, so nothing appears on the dashboard. The plugin gives the following output:
:coveralls
service name: travis-ci
service job id: 46289337
repo token: null
No source file found on the project: "caffeine"
With coverage file: /home/travis/build/ben-manes/caffeine/build/reports/jacoco/test/jacocoTestReport.xml

The closed multi-project issues lead to a non-existent repository or oblac/jodd's manual invocation. An example of how to setup this plugin for multi-project would be really useful.

I was able to work this out using an aggregated Jacoco report and setting the plugin's sourceDirs and jacocoReportPath accordingly. See build,gradle and code_coverage.gradle if interested.

Thanks!
The above info is very helpful for me. I will put this workaround to readme. Thanks!


How to use if/case in template file

Hi,
I found that rebar_templator.erl processes if and case instructions, could you please give me some examples?

See #138 and #195. We should document this in an example template or at least as part of the help string printed for rebar help create (and create-*). @jacktang, would you like to submit a patch for that?

OK, it is my pleasure.

Thanks.

I modified simplemode.template a little to make it as 'if' instruction example. And we should defined {withtest, true} in foo.vars file, the command line looks like rebar create template=simplemod template_vars=foo.vars. In this alternative, I don't know how to improve rebar help create info.
The another way, how about the command line rebar create template=simplemod withtest=true? Please review it.

Requiring withtest=true is a breaking change in the simplemod template, and there's no good reason to. We have no template syntax example in rebar help create, so printing a template that showcases all features would work there, but it may not be a perfect fit. Maybe we should take this opportunity to extend rebar help and implement rebar help -g templates, where -g GUIDE prints a guide. That way, we could embed user guides.
@ferd, what do you think?

Also, we should document both if and case.

Yeah, I'd not break anything we have. If you want it backwards compatible, you might want to use withouttest or something, and conditionally omit things.
Otherwise, having a template to showcase it is good. Let's not put stuff where it isn't needed just to show it off. Documenting them and having them as an ad-hoc demo are not required to both be there if we don't have a good reason to.

Also, we should document both if and case.

Wiki is the better place to document the template syntax?
I didn't find the topic in wiki pages so that I read the source code and wrote some examples.

I'd prefer to have more help built-in, but wiki is okay as a quick solution.

I add case instruction example here . If someone write the wiki page, you may refer to it (now I don't have right to edit the wiki pages).
And let me try to implement the rebar help -g templates idea.

If someone write the wiki page, you may refer to it (now I don't have right to edit the wiki pages).

Unfortunately, r/w access had to be limited to admins and org members due to SPAM and malicious edits. You can clone rebar.wiki.git and file a ticket to merge your commits.

Yeah, I'll have to manually merge the wiki edits. People kept vandalizing them.

Hi, this issue was closed in an attempt to do quick basic filtering. All tickets prior to March 2017 were closed and will be reopened on a per-request basis, please give rebar3 a try and feel free to reopen if it's a pressing issue that is still unresolved.


Suppress checkstyle warnings in runescape-client package

This makes Checkstyle's IDEA plugin ignore the clashing code style in the deobbed RS client.

When does this happen? The maven build config specifically has this disabled https://github.com/runelite/runelite/blob/master/runescape-client/pom.xml#L73


Drop support for Django versions < 1.9

Django=>1.9 only, recreate settings and test project from scratch and drop south migrations

Closed in
https://github.com/greenday2/django-cities-light/commit/d2aee99926739ee0e1605dfdcb88b4c0b29bc3b6
https://github.com/greenday2/django-cities-light/commit/16e47de2cf8c3711dbc38ec6026e4c012b4e60f1
https://github.com/greenday2/django-cities-light/commit/a7032c1a6dd036d66a6fa0c6199d9e11eaf87f3b
https://github.com/greenday2/django-cities-light/commit/698ceb50d3c32ca30e95555e57a1b69254d6fe4f
https://github.com/greenday2/django-cities-light/commit/47837033cb9301e5e248871394dc0ffc33db21d8
https://github.com/greenday2/django-cities-light/commit/7903dd202ee32d265f1932665a40a8a44cf94c0f
https://github.com/greenday2/django-cities-light/commit/3db5243edab4a2e42ea00c95217b2109cc10e383
https://github.com/greenday2/django-cities-light/commit/7b40098719832e41bc90da3804b17595ad0d7a70
https://github.com/greenday2/django-cities-light/commit/cbd2b0cad537216a8cae96ec21e9abdbfd74e808
https://github.com/greenday2/django-cities-light/commit/94394148b78b1fe5b94d197e5dc8ea21db17eef6
https://github.com/greenday2/django-cities-light/commit/1632113183b98f1122c23032300044deec271dd7
https://github.com/greenday2/django-cities-light/commit/c53eec572c5ef13dfb5298381703ea47aef2cf6e


fix: updated communication interface

Fixes #103
Updated the repository page and the filters coms to match new backend format.

Issue opened: https://github.com/GoogleCloudPlatform/flaky-service/issues/103


Number of working threads

Hi there !
I just wanted to confirm with you that as far as to now, Racon runs only with one thread.
Thus, it is unfit to be used on an HPC for a genome wide job.
Thank you,
Anne-Lise

Hello Anne-Lise,
could you please elaborate on the 1 thread thingy? I am asking because you can specify number of threads with the parameter '-t '.  Does HPC stand for high performance cluster? If so, Racon does not yet support execution on a cluster, but you can try and run Racon on different sets of contigs simultaneously on several cpus of your cluster.
Best regards,
Robert

Hello Robert,
Yes, I am trying to run Racon on a high performance cluster and I am new to all the terminology you mentioned but I did a little research this morning.
My script was flawed as I set up a variable to use 144 cores on 6 nodes while I didn't invoke -t in my command line. Thus, when I checked out the log, I realized that I was using just one node and the working thread was the same for each processed read (thread_ID=0).
Now, if in my script, I do something like ( I skip lines to focus on the -t option):
NUMCORE=144
/bin/racon -t $NUMCORE --erc blah.fq blah.paf blah.fq blah.fa
It will not induce 144 threads to work ?
I think it was a gross mistake of mine to space out on -t. However, I think it is a point that maybe would need to be mentioned on the code page of this repository. As a rookie, I wasn't really looking for it but I know that if I came upon something about it, I would have definitely not spaced out on it in my command line. Then it seems not possible for me to invoke in bash --options or -h. I finally realized my mistake when I looked at the exemple3-erc.sh where I saw for the first time -t. I think that proficient bioinformaticians wouldn't have done this mistake but for newbees like me, obvious mentions of simple things are still useful. :-)
Thanks again for taking the time to help me.
AL

Hello Anne-Lise,
as I mentioned before, Racon won't use all cores on a cluster automatically. I would advise you to run Racon with '-t 24' (I assume that 1 node has 24 cores). If you feel confortable with a bit file management you can run Racon on each core at the same time manually. With error correction mode, split your reads into 6 files, map the whole read set on each of the 6 disjoint subsets and run Racon on each of 6 nodes with '-t 24'. This way you are effectively running on 144 cores.
Most bioinformatics tools have a detailed list of parameters when you run them without any parameters or with '-h' (or '--help').
Best regards,
Robert

Thank you Robert, this helps a lot !
Cheers,
AL


enhance(core/event): filter js listeners on rust side only then emit filtered ids to be triggerd

fix regression introduced in https://github.com/tauri-apps/tauri/pull/8930 , and reported in https://github.com/tauri-apps/tauri/pull/8930#issuecomment-1986279046

removed the change file since an alternative fix has been released now


V1.3 help command

Implemented help command of a specific valid command.

Update DG with the implementation of help command.

Update the UI diagram of DG

Codecov Report

Merging #130 into master will decrease coverage by 0.17%.
The diff coverage is 63.15%.

@@             Coverage Diff              @@
##             master     #130      +/-   ##
============================================
- Coverage     68.39%   68.22%   -0.18%
- Complexity      603      612       +9
============================================
  Files           102      103       +1
  Lines          2082     2118      +36
  Branches        221      226       +5
============================================
+ Hits           1424     1445      +21
- Misses          572      580       +8
- Partials         86       93       +7

Impacted Files
Coverage Δ
Complexity Δ

.../seedu/address/logic/parser/HelpCommandParser.java
50.00% <50.00%> (ø)
4.00 <4.00> (?)

...java/seedu/address/logic/commands/HelpCommand.java
54.54% <57.14%> (-45.46%)
6.00 <5.00> (+4.00)
:arrow_down:

.../seedu/address/logic/parser/AddressBookParser.java
80.64% <100.00%> (ø)
17.00 <0.00> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
91.66% <100.00%> (+0.75%)
1.00 <0.00> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
76.81% <100.00%> (+1.81%)
26.00 <1.00> (+1.00)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8fcf7f4...379b584. Read the comment docs.


Destination folder defaults to 'SysWOW64'

I found the files were being downloaded to 'C:\Windows\SysWOW64'. Every other time I've run, files downloaded to the exe directory. Is it pulling a resource from that directory and then not moving on? Any fix?
W10 64 bit

Destination folder defaults to the current directory you execute youtube-dl from. Don't run it from C:\Windows\SysWOW64 and you won't get files downloaded to this directory.

I've run this from the user desktop in all cases. Just tested on a w7 64 bit and it DL'ed to C:/users/{user}
How is the program determining these output locations?


Remove unuse subset from authn policy.

This to follow API clean up in https://github.com/istio/api/pull/469

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: diemtvu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e488e5c2016eb18d8bebc9acc3b64ed509f47832
link
/test istio-unit-tests

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test e2e-bookInfo

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test e2e-bookInfo

prow/e2e-bookInfoTests-v1alpha3.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test e2e-bookInfo-envoyv2-v1alpha3

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test e2e-bookInfo

prow/e2e-bookInfoTests-v1alpha3.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test e2e-simple

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.

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

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
1b123ff2a6df2c35252b240c899d095a2bb08d52
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
b8daef9c2aa66d075554c1ddb97ddf672b6dd5a5
link
/test istio-pilot-e2e

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
b8daef9c2aa66d075554c1ddb97ddf672b6dd5a5
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
b8daef9c2aa66d075554c1ddb97ddf672b6dd5a5
link
/test e2e-bookInfo

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.


Class Properties causes syntax error

This is a:

[x] Bug Report
[ ] Feature Request
[ ] Question
[ ] Other

Which concerns:

[ ] flow-runtime
[x] babel-plugin-flow-runtime
[ ] flow-runtime-validators
[ ] flow-runtime-mobx
[ ] flow-config-parser
[ ] The documentation website

What is the current behaviour?
When I define a class property and its type, I got errors in transpiled files as below:

Unexpected token = error using transform-decorators-legacy.
Unexpected character '@' without using transform-decorators-legacy.

Is there a way to use class properties?
class Test {
  a: number;

  constructor() {
    this.a = 2;
  }
}

"plugins": [
    "add-module-exports",
    "transform-export-extensions",
    "transform-decorators-legacy", [
      "flow-runtime", {
        "assert": true,
        "annotate": true
      }
    ]
  ],

Which package versions are you using?
babel-plugin-flow-runtime 0.15.0

you need to enable the class properties transform (stage-2 i believe) and unfortunately we have a dependency on decorators when annotating classes. You should be able to avoid it if you turn annotate off, but then flow-runtime will be less accurate.


[WIP] Fix stopping criterion bug in beam search

Small fix for a large bug in beam search. In the context of generating beam visualisation similar to OpenNMT's (PR coming at a later stage) I noticed that the beam search was running along many more time steps after all the hypothesis were completed, often hitting the max length. Here is an example:

In the above example, the beam went on for 4 extra steps that are essentially wasted computation.
The problem was relying on best_word_indices to determine which hypothesis are finished. Since we run the whole beam forward, the hypothesis were generating beyond the EOS_TOKEN (ID 3), and this was throwing off the finished array.
For example, at this time step t, the second hypothesis has finished and all the values seem reasonable:

If we move the beam one step forward, finished only marks the fourth and fifth hypotheses as finished. The second one went on a loop generating token 30:

The fix is not to rely on best_word_indices to determine which hypothesis are complete, but get it directly from sequences through a max over axis 1.
I tested my change on a small model trained on IWSLT data and got a 1.5x speed up (GPU decoding on a p2.xlarge):

Without fix: [INFO:__main__] Processed 5431 lines in 5431 batches. Total time: 1773.8679, sec/sent: 0.3266, sent/sec: 3.0617
With fix: [INFO:__main__] Processed 5431 lines in 5431 batches. Total time: 1181.2254, sec/sent: 0.2175, sent/sec: 4.5978

The quality also improved (detokenised, IWSLT DE->EN test set):

Without fix: 26.82 BLEU
With fix: 32.76 BLEU

A quick diff on the tokenised data shows that a lot of the end repetitions are gone (fix on the right):

A look at a before and after on one of the more wasteful beam searches shows the improvement:
Without fix:

With fix:

Still to do

The above was tested on sockeye 1.12.0. I would have to confirm the changes on the latest master.
PR forthcoming on adding the beam visualization and dumping to a json format, to better diagnose these kinds of problems in the future.
It seems the problem is not fully fixed, as can be seen in one of the beam searches with the fix below, it seems that sockeye is still trying to generate hypotheses past an EOS. I haven't had time to do a deeper dive into this. Probably the beam search code should be rewritten to be cleaner and allow more flexibility for dropping/filtering hypotheses. For now the current change already brings improvement.

Pull Request Checklist

[ ] Changes are complete (if posting work-in-progress code, prefix your pull request title with '[WIP]'
until you can check this box.
[ ] Unit tests pass (pytest)
[ ] System tests pass (pytest test/system)
[ ] Passed code style checking (./pre-commit.sh or manual run of pylint & mypy)
[ ] You have considered writing a test
[ ] Updated major/minor version in sockeye/__init__.py. Major version bump if this is a backwards incompatible change.
[X] Updated CHANGELOG.md

I'm not sure I fully understand this, yet. Once a hypothesis is finished we make sure that the hypothesis can only generate PAD symbols from that point onward:
...
                scores = mx.nd.where(finished, pad_dist, scores)
...
in https://github.com/awslabs/sockeye/blob/master/sockeye/inference.py#L1052

What you're saying is that this wasn't working correctly. I'd like to get a better understanding for why it wasn't correctly. Do you have an idea why?
Btw, I'm looking forward to the beam search visualizations :)

As an update, I trained an RNN and a Transformer on sockeye 14.1 and the problem seems to be gone there. After producing an EOS token the rest of tokens are PAD:

The v12 model I had was a transformer, could be that there was a bug in the earlier transformer implementations. I'll have a dig and see. Either way, seems this is now fixed.

These visualizations look really cool, excited to see this coming :)
Agreeing with Tobi here. Theoretically it should not happen that a finished hypothesis is extended with a non-pad symbol, and in that case the updating of the finished hypothesis is correct.
As part of the fast-decoding transformers PR (#226) I did a similar change (https://github.com/awslabs/sockeye/pull/226/files#diff-955620b8cedc37b65c90b372a094fb17R1067), but it also should not be necessary.

As discussed, I'll close the PR for now.


code metrics not showing for functions containing assignment expressions

Sourcery doesn't show code metrics if the function or method contains an assignment expression.
Win10
VS Code 1.58.1
Sourcery 0.9.3
Python 3.9.2

Thanks for raising! This is a duplicate of https://github.com/sourcery-ai/sourcery/issues/130.
Will close this one and handle it from there - am actually working on the issue soon,


Closing segments & tasks

I realize this may be a silly question but I can't seem to find the answer anywhere on the documentation or the GUI itself. Is there any way to 'close' (ie: mark as done and remove from the list) task segments and task themselves?
I suppose I could just delete the task after dumping the annotations, but that is a bit inconvenient, particularly since it is not yet possible to import tasks (I am aware there is already another issue for this functionality). Is there no other way to achieve this as of now?

@4sfaloth,
At the moment it is not possible to hide some tasks. Let's keep the feature request. I believe it is worth to implement in the future.


[Storm Furies] New sheet for Storm Furies by John Harper

Changes / Comments
Created new sheets for Storm Furies based on existing Agon sheet (authors mentioned in instructions).
Storm Furies can be found here: https://johnharper.itch.io/storm-furies or here: https://www.evilhat.com/home/paragon/
Contains a sheet type for pilots, the squadron and the threat player, as well as roll templates for a contest, the pilots rolls and stormcraft damage.
Usage of images of the original game as well as the sheet itself have been approved by John Harper.
Followed the character sheets standards and the original Agon sheet implementation.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[X] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


[Task] Support relative time calculation expressions

Description
Support relative time calculation expressions
about discussion #1979
Task List
No response

Hello, please assign it to me, I can complete this feature. Thank you.


Loading pre-saved krakend.json files

Firstly, great job with the development of Krakend designed.
Secondly, apologies - this is more of a feature suggestion than an "issue", per se.
So, was wondering if there are any plans to reload a krakend.json file that's been created in the designer back into the designer so you can start editing again? Currently all edits are lost when you close the page so you have to start again from scratch.

Hello @ChristofBlanc ,
This feature is implemented from day one, wondering why you can't see it. When you open the KrakenDesigner, on its front page (the Dashboard), you should see a gray square with a bigger text saying "Drop configuration file to load it.". Just drag the krakend.json file in there and you'll continue where you left it. If you have problems using this, please reopen and show me some screenshots.

Hello @ChristofBlanc ,
This feature is implemented from day one, wondering why you can't see it. When you open the KrakenDesigner, on its front page (the Dashboard), you should see a gray square with a bigger text saying "Drop configuration file to load it.". Just drag the krakend.json file in there and you'll continue where you left it. If you have problems using this, please reopen and show me some screenshots.

You are right - apologies! 🤭
I had been using the "save" icon on the left hand panel to save my config file rather than the "Download this configuration" button on the landing page hence was probably expecting to see a "load" icon near it.
Anyhoo, glad the feature is there. Thanks.

You are right - apologies! 🤭
I had been using the "save" icon on the left hand panel to save my config file rather than the "Download this configuration" button on the landing page hence was probably expecting to see a "load" icon near it.
Anyhoo, glad the feature is there. Thanks.


chapter 7: countplot() got multiple values for argument 'data'

fig, ax = plt.subplots(1, 2, figsize = (7, 5))
titanic["Pclass"].value_counts().plot.bar(color = "skyblue", ax = ax[0])
ax[0].set_title("Number Of Passengers By Pclass")
ax[0].set_ylabel("Population")
sns.countplot("Pclass", hue = "Survived", data = titanic, ax = ax[1])
ax[1].set_title("Pclass: Survived vs Dead")
plt.show()

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[10], line 6
      3 ax[0].set_title("Number Of Passengers By Pclass")
      4 ax[0].set_ylabel("Population")
----> 6 sns.countplot("Pclass", hue = "Survived", data = titanic)
      7 ax[1].set_title("Pclass: Survived vs Dead")
      8 plt.show()

TypeError: countplot() got multiple values for argument 'data'

I have got the same error, any solutions?

@gokcenazakyol  let me have a look, probably you are using some latest version of library. I will push a fix with never version, today.

Feel free to create a pull request.
On Thu, 30 Nov 2023 at 09:43, Gökçenaz Akyol @.***>
wrote:

I think we solved the problem.
sns.countplot("Pclass", hue = "Survived", data = titanic, ax = ax[1])
In that line, we need to define x="Pclass", otherwise, (I think) we try to
reassign data value and that cause the error.
—
Reply to this email directly, view it on GitHub
https://github.com/PacktPublishing/Hands-on-Exploratory-Data-Analysis-with-Python/issues/7#issuecomment-1833327844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPN4B2UGAZNAUVBFZMXEXTYHBBKHAVCNFSM6AAAAAA2EHPT56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZTGMZDOOBUGQ
.
You are receiving this because you were assigned.Message ID:
<PacktPublishing/Hands-on-Exploratory-Data-Analysis-with-Python/issues/7/1833327844
@github.com>

I think we solved the problem.
sns.countplot("Pclass", hue = "Survived", data = titanic, ax = ax[1])
In that line, we need to define x="Pclass", otherwise, (I think) we try to reassign data value and that cause the error.

Thank you for this solution @gokcenazakyol .

try :  sns.countplot(x='Survived', data=titanic, ax=ax[1])


Exception occurred in Gauge plugin

Describe the bug
While opening IDE, Gradle libraries show exception in Event Logs while they are built.
To Reproduce
Steps (or project) to reproduce the behavior:

Create a gauge project with Gradle
Define the following dependencies on Gradle
dependencies {  implementation 'com.thoughtworks.gauge.gradle:gauge-gradle-plugin:1.7.3'  implementation 'com.thoughtworks.gauge:gauge-java:0.7.13'  testImplementation group: 'org.assertj', name: 'assertj-core', version: '3.12.2'  implementation 'org.seleniumhq.selenium:selenium-java:3.141.59'  implementation 'io.github.bonigarcia:webdrivermanager:4.3.1'  implementation 'org.slf4j:slf4j-log4j12:1.7.30' }
Reload all Gradle project
Invalidate Caches/Restart..
See Event Log

Logs
Exception occurred in Gauge plugin
More details...
Please report an issue in https://youtrack.jetbrains.com/issues/IDEA with following details:
#### Gauge process exited with code ```1```
Gauge daemon initialized and listening on port: 2790
* IDE version: 2020.3.1
* API version: IC-203.6682.168
* Plugin version: 203.5981.155
* Gauge version: 1.1.7

Expected behavior
There shouldn't have been an exception.

Versions:
gauge -v: 1.1.7
gradle -v: 6.8
java -version: 15.0.1

OS Windows 10
Runner info: Java jdk 15.0.1
IDE info: Intellij IDEA

Additional context
I don't know what is causing Exception right now. It is only seen when Ide first opened. Other than that, it is not observed again.

Can you please report this to https://youtrack.jetbrains.com/issues/IDEA

Can you please report this to https://youtrack.jetbrains.com/issues/IDEA

Hi,
A similar problem was reported on the page you specified, but they also wrote that this problem should be reported to you.
Report Id: IDEA-259583
Report Url: https://youtrack.jetbrains.com/issue/IDEA-259583
Thanks

Hi,
A similar problem was reported on the page you specified, but they also wrote that this problem should be reported to you.
Report Id: IDEA-259583
Report Url: https://youtrack.jetbrains.com/issue/IDEA-259583
Thanks

Can you please refer https://docs.gauge.org/faqs.html#where-does-gauge-non-project-specific-logs-like-plugin-installation-etc to see if there any exceptions in the log files?

Can you please refer https://docs.gauge.org/faqs.html#where-does-gauge-non-project-specific-logs-like-plugin-installation-etc to see if there any exceptions in the log files?

I know this is late, but is this still an issue? I was able to create a new gradle project (the template is updated to use gradle 6.8.2)  and add the dependencies you mentioned. Could not see any errors.

This seems old/stale, so closing. Can re-open if its still an issue.


README.md update proposals for Pulsar newbies

Hi,
I was testing Pulsar with your snowflake-connector. This was my first Pulsar exercise. So I am just summing up here some issues, that I was facing and which might be worth mentioning in README.

Pulsar version dependency - the connector is not working with latest stable Pulsar version (2.8.1). You need to build Pulsar from master branch.
For my first tests, I was using Pulsar CLI client to create testing messages with JSON content. The message content in Snowflake table was encoded using Base64 and I did not understand why. In the end I found out, that message schema matters (and value.converter class does not). The Pulsar CLI client creates messages with BINARY schema. Once I used Java client and set schema on Producer to JSON, everything start working. Still not sure here, but probably value.converter (and may be key.converter) kafkaConnectorConfigProperties has no effect, since Pulsar messages are converted to Kafka messages in org.apache.pulsar.io.kafka.connect.KafkaConnectSink.toSinkRecord() method. This is very confusing. If there are some required kafkaConnectorConfigProperties, which are effectively ignored - it should me mentioned in the doc.
The configs.topic configuration property seems to be ignored as well. Created this bug: https://github.com/apache/pulsar/issues/12880 This should solve the topic to table name mapping issue.

Anyway after checking some source code, I was able to make the connector running. So thanks for your project!

Thank you!
The connector needs Apache Pulsar 2.9+ or Datastax Luna Streaming 2.8 (relevant changes cherry-picked there).
I'll work on a better documentation; if you have a fix for https://github.com/apache/pulsar/issues/12880 I'll be happy to  review the PR (or I'll get to fixing it a little bit later).
regarding the schema, I used the following CLI commands:
Set retention for the namespace:

$ bin/pulsar-admin namespaces set-retention public/default -t -1 -s 100M

Set topic schema:

$ cat ~/schema_sf_js.json
{
    "type": "JSON",
    "schema": "{\"type\":\"record\",\"name\":\"SnowflakeTestSchema\",\"namespace\":\"com.foo\",\"fields\":[{\"name\":\"col1\",\"type\":[\"null\",\"string\"]},{\"name\":\"col2\",\"type\":[\"null\",\"string\"]}]}",
    "properties": {}
}

$ bin/pulsar-admin schemas upload --filename ~/schema_sf_js.json snowflake-test

and to send the messages:
$ cat ~/a.json
{"col1": "val1", "col2": "val2"}

$ bin/pulsar-client produce snowflake-test -f ~/a.json -n 200 -vs json:"{\"type\":\"record\",\"name\":\"SnowflakeTestSchema\",\"namespace\":\"com.foo\",\"fields\":[{\"name\":\"col1\",\"type\":[\"null\",\"string\"]},{\"name\":\"col2\",\"type\":[\"null\",\"string\"]}]}"

@dig99 Thanks, I was not aware about the -vs option in pulsar client...
I created the PR: https://github.com/apache/pulsar/pull/12893

Here is a short getting started guide: https://medium.com/building-the-open-data-stack/datastax-presents-snowflake-sink-connector-for-apache-pulsar-53629b196064
It is a bit late to help you but it I hope it will help someone else

Added link to the blog post into the readme.