psycopg2.errors.UndefinedObject: unrecognized configuration parameter "version"

Hi there,
Getting psycopg2.errors.UndefinedObject: unrecognized configuration parameter "version" when running agent check
agent version
Agent 7.19.0 - Commit: 914b764 - Serialization version: 4.30.0 - Go version: go1.13.8

agent check pgbouncer
2021-11-02 18:54:16 UTC | CORE | ERROR | (pkg/collector/python/datadog_agent.go:116 in LogMessage) | pgbouncer:92f76e2412c29bd1 | (pgbouncer.py:74) | Not all metrics may be available
Traceback (most recent call last):
  File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/datadog_checks/pgbouncer/pgbouncer.py", line 69, in _collect_stats
    cursor.execute(query)
  File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/psycopg2/extras.py", line 142, in execute
    return super(DictCursor, self).execute(query, vars)
psycopg2.errors.UndefinedObject: unrecognized configuration parameter "stats"
2021-11-02 18:54:16 UTC | CORE | ERROR | (pkg/collector/python/datadog_agent.go:116 in LogMessage) | pgbouncer:92f76e2412c29bd1 | (pgbouncer.py:74) | Not all metrics may be available
Traceback (most recent call last):
  File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/datadog_checks/pgbouncer/pgbouncer.py", line 69, in _collect_stats
    cursor.execute(query)
  File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/psycopg2/extras.py", line 142, in execute
    return super(DictCursor, self).execute(query, vars)
psycopg2.errors.UndefinedObject: unrecognized configuration parameter "pools"
2021-11-02 18:54:16 UTC | CORE | ERROR | (pkg/collector/python/datadog_agent.go:116 in LogMessage) | pgbouncer:92f76e2412c29bd1 | (pgbouncer.py:74) | Not all metrics may be available
Traceback (most recent call last):
  File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/datadog_checks/pgbouncer/pgbouncer.py", line 69, in _collect_stats
    cursor.execute(query)
  File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/psycopg2/extras.py", line 142, in execute
    return super(DictCursor, self).execute(query, vars)
psycopg2.errors.UndefinedObject: unrecognized configuration parameter "databases"
=== Service Checks ===
[
  {
    "check": "pgbouncer.can_connect",
    "host_name": "i-066505a42f068d57b",
    "timestamp":<PHONE_NUMBER>,
    "status": 0,
    "message": "Established connection to postgresql://***:******@<IP_ADDRESS>:5432/******",
    "tags": [
      "db:pgbouncer",
      "docker_image:bitnami/pgbouncer:1.16.0",
      "host:<IP_ADDRESS>",
      "image_name:bitnami/pgbouncer",
      "image_name:docker.io/bitnami/pgbouncer",
      "image_tag:1.16.0",
      "kube_container_name:pgbouncer",
      "kube_deployment:pooling-pgbouncer",
      "kube_namespace:pooling",
      "kube_service:pooling-pgbouncer",
      "pod_phase:running",
      "port:5432",
      "short_image:pgbouncer"
    ]
  }
]
=========
Collector
=========

  Running Checks
  ==============

    pgbouncer (1.9.0)
    -----------------
      Instance ID: pgbouncer:92f76e2412c29bd1 [ERROR]
      Configuration Source: kubelet:docker://bd88a317428d85781ca6ab5d0e9949334f5166770fe2fe20359e76d101f58a95
      Total Runs: 1
      Metric Samples: Last Run: 0, Total: 0
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 1, Total: 1
      Average Execution Time : 7ms
      Last Execution Date : 2021-11-02 18:54:16.000000 UTC
      Last Successful Execution Date : Never
      Error: unrecognized configuration parameter "version"

      Traceback (most recent call last):
        File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/datadog_checks/base/checks/base.py", line 820, in run
          self.check(instance)
        File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/datadog_checks/pgbouncer/pgbouncer.py", line 165, in check
          self._set_metadata()
        File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/datadog_checks/pgbouncer/pgbouncer.py", line 169, in _set_metadata
          pgbouncer_version = self.get_version()
        File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/datadog_checks/pgbouncer/pgbouncer.py", line 176, in get_version
          cursor.execute('SHOW VERSION;')
        File "/opt/datadog-agent/embedded/lib/python3.8/site-packages/psycopg2/extras.py", line 142, in execute
          return super(DictCursor, self).execute(query, vars)
      psycopg2.errors.UndefinedObject: unrecognized configuration parameter "version"

Check has run only once, if some metrics are missing you can try again with --check-rate to see any other metric if available.

As I understand it tries to get postgres version. In this case it should do show server_version;, instead of show version;

Fixed by pointing to pgbouncer database

Fixed by pointing to pgbouncer database

What does this comment mean exactly? Do I need to create a pgbouncer database on the server? I don't understand what this fixed exactly?


Production mode doesn't transform RegEx correctly

Which @angular/* package(s) are the source of the bug?
Don't known / other
Is this a regression?
No
Description
i have declared a RegEx like this:
  private readonly _DISALLOWED_CHARS_REGEX_GENERAL = /^[^\\/\?\!\&\:\*\<\>\|\~\%\§\@\{\}\[\]\(\)\=\$\€\"]+$/;
  /**
  * Generally Forbidden chars: '\' '/' '?' '!' '&' ':' '*' '<' '>' '|' '~' '%' '§' '@' '{' '}' '[' ']' '(' ')' '=' '$' '€' and '"'
  */
  public get DISALLOWED_CHARACTER_REGEX_GENERAL(): RegExp {
    return this._DISALLOWED_CHARS_REGEX_GENERAL;
  }

In development builds, the RegEx is properly evaluated, however, in prod mode systems, it's not.
The problem is pretty similar to #https://github.com/angular/angular/issues/53789 in my case it wrongly flags 0, 2, 7, a, c, u, x and sometimes also z.
After getting some initial help over at Stack Overflow  #https://stackoverflow.com/questions/78200826/ I was able to find the problem.
Could it be the same "issue" as it was here? #https://github.com/angular/angular-cli/issues/22359
Would I really need to replace some of the special characters by their ASCII representation?
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
After production mode, the RegEx transforms to /^[^\\/\?\!\&\:\*\<\>\|\~\%\\xa7\@\{\}\[\]\(\)\=\$\\u20ac\"]+$/

Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 15.2.10
Node: 20.11.1 (Unsupported)
Package Manager: npm 10.2.4
OS: win32 x64

Angular: 15.2.10
... animations, cli, common, compiler, compiler-cli, core
... elements, forms, language-service, platform-browser
... platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1502.10
@angular-devkit/build-angular   15.2.10
@angular-devkit/core            15.2.10
@angular-devkit/schematics      15.2.10
@angular/cdk                    15.2.9
@schematics/angular             15.2.10
rxjs                            7.8.1
typescript                      4.9.5

Anything else?
No response

The issue arises from redundant escaping in the regular expression, causing it to become invalid after processing by terser. Specifically, the redundant escape of € results in \\u20ac, and § in \\xa7.
To rectify this, I recommend removing unnecessary escapes from the original regular expression:
Input
- /^[^\\/\?\!\&\:\*\<\>\|\~\%\§\@\{\}\[\]\(\)\=\$\€\"]+$/
+ /^[^\\/?!&:*<>|~%§@{}[\]()=$€"]+$/

Output
- /^[^\\/\?\!\&\:\*\<\>\|\~\%\\xa7\@\{\}\[\]\(\)\=\$\\u20ac\"]+$/
+ /^[^\\/?!&:*<>|~%\xa7@{}[\]()=$\u20ac"]+$/

Additionally, it's suggest to configure your linter to issue warnings for unnecessary escaping. You can find an example of this configuration here: https://eslint.org/docs/latest/rules/no-useless-escape

Thank you very much! I wasn't aware that most of the special characters don't need escaping in Typescript.

Thank you very much! I wasn't aware that most of the special characters don't need escaping in Typescript.

It's actually JavaScript


#12 The endpoint-spec keeps getting updated.

Codecov Report

Merging #15 (4a301b0) into main (1417674) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          788       788
  Branches       122       122
=========================================
  Hits           788       788

Flag
Coverage Δ

data
100.00% <ø> (ø)

data-backend
100.00% <ø> (ø)

data-frontend
100.00% <ø> (ø)

metadata-jsonschema
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


[BUG] there is no parent span that tracks whole timer callback

bp-ihub-asb-asb-connector.zip

Created https://issues.apache.org/jira/browse/CAMEL-18661 to track it.

Closing - Camel issue is fixed in version 3.20.0.


Buggy 2nd Window Sometimes

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
When i make Chrome + Chatterino in Dock Mode (Split Screen) its opening an WIndow i cant Close and when i hover it dosent say anything
Screenshots

OS and Chatterino Version
Chatterino 2.5.1 (commit eafcb941) built with Qt 6.5.0, MSVC 193833135 Running on Windows 11 Version 23H2, kernel: 10.0.22631

I suppose this is only an issue on Windows 11, since Windows 10 doesn't have the snap feature. Do you happen to know what the window title is or - even better - what window class it is (maybe with Spy++; included in a Visual Studio installation in Common7\Tools\spyxx_amd64.exe)?


cmake: Require Python >= 3.10

The minimum Python version was bumped for Zephyr LTSv3, but only in the documentation and CI. The build system would still accept Python 3.8, yet some scripts in tree have already broken support with that version. Incompatibility errors should be prevented early.

If we require Pytyon 3.10 with Zephyr v3.7 LTS, we must backport this to v3.7-branch.


Highlight API: Add an explanation of style order for ranges within a HighlightRangeGroup

Adding a sentence about the order in which styles are applied to ranges within the same highlight range group.

Should we also add a line of having priorities for multiple highlighrangegroups?

Should we also add a line of having priorities for multiple highlighrangegroups?

I believe that is covered on line 67.


docs: multiple markdown docs fixed

Please briefly answer these questions:

what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)

dead link or missing link exists

what changes does this pull request make?

fix dead link and missing link
make options more readable by adding back quotes (`)
reformat some markdown file using Prettier

are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

no

rebased


helm create: make generated YAML indentation more consistent

What this PR does / why we need it:
It just makes the indentation of generated output of helm create indented more consistently.
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

As a note, commits do not need to be cherry-picked into a minor release. As per the release checklist, the next minor release is cut from the HEAD of the master branch. Cherry picks are only necessary for patch releases.
https://helm.sh/docs/community/release_checklist/#1-create-the-release-branch
Removing the needs-pick label accordingly. :)


PouchDB is not a constructor

Thank you for !
If I run
const PouchDB = require('pouchdb-react-native')
const db = new PouchDB('my_db')

I get the error message in ios PouchDB is not a constructor

I've reinstalled everything, and redone the project from scratch. It appears to be working constructor now.


Harmonized some discrepant text

It seems strange to progress from "Normal User" to "Developer" to "Power User" ... and also it is unclear whether "Power User" includes all "Developer" views -- which would also be strange.
I would generally expect (as is the case with all other software with which I'm familiar) to progress from "Normal User" to "Power User" to "Developer", and for "Power User" views to include all "Normal User" views and "Developer" views to include all "Power User" (and thus, all "Normal User") views.
This should likely feed a new issue, as it seems likely to require changes in many areas.

I agree that the way the Roles are handled at the moment is not optimal with it not being a progression Normal -> Power User -> Developer. The use of Roles is currently being reviewed to I will include this comment.


Change default encoding to UTF-8 (in configMap.yaml)

Proposed changes
Change default ASCII-8BIT encoding to UTF-8
splunk-connect-for-kubernetes/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-logging/templates/configMap.yaml:
add after line 75
(https://github.com/splunk/splunk-connect-for-kubernetes/blob/79d17645af594762c9421cc184d59201a7ccb0a6/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-logging/templates/configMap.yaml#L75):
encoding UTF-8
from_encoding UTF-8
Types of changes
What types of changes does your code introduce?
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)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I have read the CLA
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

What issues do you have without this ? Can you share logs of errors?

#568
See the issue:
https://github.com/splunk/splunk-connect-for-kubernetes/issues/568

To ensure backward compatibility you could add the below in the configMap.yaml file
      {{- if .Values.fluentd.encoding.containers.encoding }}
      encoding {{ .Values.fluentd.encoding.containers.encoding }}
      {{- end }}
      {{- if .Values.fluentd.encoding.containers.fromEncoding }}
      from_encoding {{ .Values.fluentd.encoding.containers.fromEncoding }}
      {{- end }}

together with the values.yaml
splunk-kubernetes-logging:
  fluentd:
    encoding:
      containers:
        encoding:
        fromEncoding:

The default values mean that no configuration for envoding or from_encoding would be set, however, if one wants
it we could override it, for example:
splunk-kubernetes-logging:
  fluentd:
    encoding:
      containers:
        encoding: UTF-8
        fromEncoding: UTF-8

(note: not sure about the best structure for the encoding configuration since it can probably can be more places where encoding  can be set. It would be more simpler if there were just one encoding value but I'm unsure about how it's used etc etc)

+1 for @LennartHaggkvistHM idea of making it optional to not break anybodys logging.
But I think the change should be more like https://github.com/openshift/origin-aggregated-logging/pull/622 where the uft-8 conversion is done independently of the source type. actually they introduced it because not every source type supports setting the encoding.

Fixed in #570, closing as NTD


package-level extension

In the WinformAppService example, the README.MD mentioned that:

"This component gets activated out-of-proc to the main application process, so we also need to declare a package-level extension to explain to the system how the class will get activated."

But I can't find more detail about the package level extension.
Also notice that the vcxproj file missing the Package.StoreAssociation.xml, is that related?

Hi Hongtao,
The package-level extension gets written automatically by VS during the build process, so you typically don’t need to worry about. If you look for the appxmanifest.xml in the build output you should be able to view the extension added for you automatically.
Let me know if this doesn’t answer your question.
Thanks,
Stefan
From: Hongtao Chen<EMAIL_ADDRESS>Sent: Monday, February 12, 2018 5:52 PM
To: Microsoft/DesktopBridgeToUWP-Samples<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [Microsoft/DesktopBridgeToUWP-Samples] package-level extension (#59)
In the WinformAppService example, the README.MD mentioned that:
"This component gets activated out-of-proc to the main application process, so we also need to declare a package-level extension to explain to the system how the class will get activated."
But I can find more detail about the package level extension.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FDesktopBridgeToUWP-Samples%2Fissues%2F59&data=04|01|swick%40microsoft.com|294fba80e2c34c8418c208d572845fae|72f988bf86f141af91ab2d7cd011db47|1|0|636540835224681415|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=0GJo5MtA%2FCTQod8Z29z0Uwj5NKu4s75dAqFA2WSXWT0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASRavBLYuWekO9XPstEXM2SUGK7Xt3vfks5tUOrAgaJpZM4SDGcg&data=04|01|swick%40microsoft.com|294fba80e2c34c8418c208d572845fae|72f988bf86f141af91ab2d7cd011db47|1|0|636540835224681415|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=U7r068AvAvUiKmww4wuJSGNcKXIIupdo%2FkD72zVSLd8%3D&reserved=0.

It's not there. I tried to follow the Win Form App Service, but no package level extension created for my runtime component.

I just tried it and it works for me. Here is how I tested:

Build the solution
Right-click on the “TestClient” project and select deploy
Go to \WinformsAppService\x64\Debug\TestClient\AppX folder
Inspect the created appxmanifest file

There you will find the package-level extension created as:

AppServiceComponent.dll

From: Hongtao Chen<EMAIL_ADDRESS>Sent: Tuesday, February 13, 2018 9:54 PM
To: Microsoft/DesktopBridgeToUWP-Samples<EMAIL_ADDRESS>Cc: Stefan Wick<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Microsoft/DesktopBridgeToUWP-Samples] package-level extension (#59)
It's not there. I tried to follow the Win Form App Service, but no package level extension created for my runtime component.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FDesktopBridgeToUWP-Samples%2Fissues%2F59%23issuecomment-365504053&data=04|01|swick%40microsoft.com|52df67612f3d424dddd908d5736f6015|72f988bf86f141af91ab2d7cd011db47|1|0|636541844629049065|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=gjYrxskSPlLCeD%2BeRcPX78%2BrmV7DgNZRs6nVsOmxla0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASRavCTDsfzj3lfB_Oqz7JAcIYFKl6l-ks5tUnUEgaJpZM4SDGcg&data=04|01|swick%40microsoft.com|52df67612f3d424dddd908d5736f6015|72f988bf86f141af91ab2d7cd011db47|1|0|636541844629049065|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=PSMq47uJf518OslqvbHIwvyIII24uuzJvWlnwNL2NsY%3D&reserved=0.

Thanks a lot, I put my POC in the one drive. https://1drv.ms/u/s!Ar4lO2gIHPQZsz3j2HTXbioHFFBq
I can build and deploy it. But no package level extension created in manifest xml, and when the win form app tries to connect to the app service. it fails with app unavailable.

The problem in your solution is that the “Component1” project is of output type “Class Library”. You need to change it to type “Windows Runtime Component”.

Stefan

From: Hongtao Chen<EMAIL_ADDRESS>Sent: Wednesday, February 14, 2018 7:53 PM
To: Microsoft/DesktopBridgeToUWP-Samples<EMAIL_ADDRESS>Cc: Stefan Wick<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [Microsoft/DesktopBridgeToUWP-Samples] package-level extension (#59)
Thanks a lot, I put my POC in the one drive. https://1drv.ms/u/s!Ar4lO2gIHPQZsz3j2HTXbioHFFBqhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2F1drv.ms%2Fu%2Fs!Ar4lO2gIHPQZsz3j2HTXbioHFFBq&data=04|01|swick%40microsoft.com|5262ac5e7bc943afe21b08d5742790f7|72f988bf86f141af91ab2d7cd011db47|1|0|636542635655394242|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=kvrXCim6cnIuwSDbl8vRjmj9czuwu30YMqjXLTEG7W8%3D&reserved=0
I can build and deploy it. But no package level extension created in manifest xml, and when the win form app tries to connect to the app service. it fails with app unavailable.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FDesktopBridgeToUWP-Samples%2Fissues%2F59%23issuecomment-365818635&data=04|01|swick%40microsoft.com|5262ac5e7bc943afe21b08d5742790f7|72f988bf86f141af91ab2d7cd011db47|1|0|636542635655394242|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=FxyGM9AoW7O4M%2FW9k2rudnn1dDm1AvWY2BiW4yqiC2A%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASRavOiAj0MPVWzdxDCC1SHloaHMfbyYks5tU6oKgaJpZM4SDGcg&data=04|01|swick%40microsoft.com|5262ac5e7bc943afe21b08d5742790f7|72f988bf86f141af91ab2d7cd011db47|1|0|636542635655394242|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwifQ%3D%3D|-1&sdata=vrk8zWZoElaFEcLr71PPlIAwTaZhjdb90AHHS2bOmu4%3D&reserved=0.

Thanks a lot, that helps. Now I see the extension session in the generated manifest xml file. But when the winform UI try to connect to it, I still get App unavailable error. Is there something I still missed?

The UI is the winform app still, I try to use it to connect to some features that converted to UWP runtime already. The Cpp project there is just a package project for now, business logic will be in the C# library. Because the project designed in the way that the winform app and UWP communicate with app service only, I cannot call the library directly from the winform app.

Still blocked by the application is not available problem, any idea?

Here is the updated example: https://1drv.ms/u/s!Ar4lO2gIHPQZs0474Vn6MRKoSYy-
When I click the button, I still get the service unavailable error.
Thanks,


Use Windows-1252 encoding for the copyright notice on Windows

Uses an escaped character code for the copyright symbol, keeping the source code clear of non-ASCII characters (so that UTF-8–based source-processing tools like tolf work), while still producing the intended Windows-1252 string in Windows DLL version-info blocks.
( This is a refactoring of PR #5508 .)

@WalterBright Happy now? : )

Auto-merge toggled on

You can't just do it inline in the sting literal?

@yebblies you're right

I could have, as just a quick fix to the problem. But I went out of my way not to, because I feel it's much healthier to avoid unannotated magic numbers where reasonable.
With symbol usage, one know what's being done immediately without pause; and when navigating huge swathes of new code or maintaining old code, the difference between having to pause for just half a second, every 5 or so lines of code and not having to pause at all and just being able to read the code as a story that doesn't need decoding, I find to be immense.

That has to be balanced against over-complicating the code.  I feel like the comment and the context make it fairly clear here.
Then again - @WalterBright is there anything stopping us from just using "(C)" here and avoiding the issue completely?

Note that it's also one of many things I'm loving about D so far: the first glimpses I got at its code base was written so beautifully readable.
It's not something I expected from a compiler's source code; and it made me fairly certain that it would be something that could see great growth through contribution from the community.

@yebblies Naturally, I agree. It's not exactly necessary here, given the context of repeated variations of the same string. :p
It's just the principled way that I like to code for the future me that throws those things into my code.


Adding functionality to check the balance of a given wallet.

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

          1 Message

:book:
See http://keepachangelog.com/en/0.3.0/ for details on good changelog guidelines

  Generated by :no_entry_sign: Danger

Awesome!

LGTM


Usage of different detectors

Hi, Looks to me this is a very nice tool for camera cailbration!
Wondering if there is any samples on how to use detector other than mrgingham. I had a look into doc, looks like there is some process to create .vnl file but still not clear how to create mrcal compatable .vnl file by running other detectors for example opencv findchessboardcorners function.
Thanks!

Are the docs unclear? The page is here: https://mrcal.secretsauce.net/recipes.html#non-mrgingham-detector It notes what the assumptions and expectations are, and includes an example data file. What else would you like?

Thanks again for quick reply! i was wondering if there is any possible flow as below
-> Run opencv detector to get corners
-> Run vnlog to create .vnl file
So wondering if there is any api from vnlog to create .vnl files.

I just creaetd a txt file with required format and changed extention. It works, thanks!


DOP-2319 - Add new quiz directives to snooty-parser

Stories/Links:
DOP-2319
Notes:
Added new directives to snooty-parser to support the new Quiz Widget

@carooobregon could you run make format and commit? CI is failing


CC-834 changed -E to -P

the purpose of this change is to keep consistency.
ripgrep requires the '-P' flag to enable support for backreferences.
more details here

We decided to not do this and instead mangle the your_grep.sh script used in the tester's tests


some comment do not need

some comment do not need
eg. open and close comment

Ok, I just remove the exit button except open button, I think the open button is still has its value.

This program doesn't have a command such as "secc a.cc", so it need a button to do this all.


Refine SegmentReadTask for disaggregated-mode and op-mode.

What problem does this PR solve?
Issue Number: ref #6834
Problem Summary:
What is changed and how it works?

Merging RNReadSegmentTask into SegmentReadTask and moving code of SegmentReadTask into new files SegmentReadTask.h and SegmentReadTask.cpp.
Replacing RNReadSegmentTask with SegmentReadTask.

Check List
Tests

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

Side effects

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

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-unit-test

/run-all-tests

I've pushed a commit to add some comments.

/run-all-tests

@JinheLin BTW, is "Data sharing" automatically enabled for disagg arch after this PR merged? Or there are still something need to be done in later PRs?

OK @JinheLin


RN 0.40

[ ] RN 0.40 has a breaking change, project can't be compiled after linking. Let's wait.
[ ] try https://github.com/este/este/issues/1235 and https://github.com/este/este/issues/1157
[ ] recheck native-fix
[ ] https://github.com/ncuillery/rn-diff

https://github.com/oblador/react-native-vector-icons alread fixed

https://twitter.com/grabbou/status/819279756021628929

I actually got it working on both ios & android after upgrading to react-native 0.40.0.
Following is the quickfix method I followed.

In node_module/fela-native/package.json
Line 85, change react-native dependency to 0.40.0

Upgrade React-Native by running following commands
a. npm install -g react-native-git-upgrade
b. react-native-git-upgrade
c. react-native upgrade

In este/package.json upgrade "react-native-vector-icons" to  4.0.0
a. npm install --save<EMAIL_ADDRESS>

And finally restart package manager and then gulp ios & gulp android worked just fine.

Thank you @arjunpola!


issue-408 - server timeout setting configurable through envs

Trying to implement issue-408.
Not exactly sure if it is a correct way to achieve it, so would appreciate any feedback!

Thank you for this contribution!


Prompt to update

Added feature that will show prompt to update message when a new version of Serverpod CLI is available
#611
Pre-launch Checklist

[x] I read the Contribute page and followed the process outlined there for submitting PRs.
[x] This update contains only one single feature or bug fix and nothing else. (If you are submitting multiple fixes, please make multiple PRs.)
[x] I read and followed the Dart Style Guide and formatted the code with dart format.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///), and made sure that the documentation follows the same style as other Serverpod documentation. I checked spelling and grammar.
[x] I added new tests to check the change I am making.
[x] All existing and new tests are passing.
[ ] Any breaking changes are documented below.

Message:
┌───────────────────────────────────────────────────────────────────────────────────────┐
│ A new version of Serverpod is available!                                              │
│                                                                                │
│ To update to the latest version, run "dart pub global activate serverpod_cli".        │
│ Also, do not forget to update packages in your server, client, and flutter projects.  │
└───────────────────────────────────────────────────────────────────────────────────────┘

I have read the CLA Document and I hereby sign the CLA

@yahu1031  @vlidholt
Added ability to call serverpod upgrade to upgrade CLI to the last available version.
Also updated prompt message:
┌───────────────────────────────────────────────────────────────────────────────────────┐
│ A new version 1.0.0 of Serverpod is available!                                        │
│                                                                                │
│ To update to the latest version, run "serverpod upgrade".                             │
│ Also, do not forget to update packages in your server, client, and flutter projects.  │
└───────────────────────────────────────────────────────────────────────────────────────┘

@vlidholt Hello. What about this one?

Please check the dart analyze issue @damienissa and this is good to go.

@yahu1031 Fixed.

The answer for Q1 is that this function is only being triggered from the upgrade command, So If the user is offline, We going to get values null at this line and it returns null and the function ends with nothing. I can suggest(@damienissa) have an already up-to-date prompt there.

This is merged in another PR. :)


How do i add torrent trackers in colab?

I want to add torrent trackers in colab because my downloading speed in colab is 0.00kbps and when i add trackers in uTorrent for the same torrent speed increased but how can i add those trackers in colab as well so that my download catch some speed.

@Psp29onetwo Try this https://stackoverflow.com/questions/51137770/libtorrent-add-multiple-tracker-in-magnet-link-uri
And let me know if it worked.


Rename gcs.endpoint to gcs.service.host

To make it in line with Java:
https://github.com/apache/iceberg/blob/6ee6d1327d3811dbd5795c4e87efdc41b7a58eaa/gcp/src/main/java/org/apache/iceberg/gcp/GCPProperties.java#L32

@Fokko, I can solve the missing import. Do you mind?

Thanks @kevinjqliu, using get_first_property_value is much cleaner


PIP dependency conflict?

When I run all cells on the Google Colab project, I see this error:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
datascience 0.10.6 requires folium==0.2.1, but you have folium 0.8.3 which is incompatible.

Any way to fix it?  I'm running on the free version of Google Colab, perhaps the demo no longer works on the free version?

Duplicate of https://github.com/saharmor/dalle-playground/issues/54
This shouldn't prevent you from generating images.


replace ncbitaxon altIDs by main taxon ID

Not sure if filling only the providedTaxonId column is enough. Should I fill in also the providedTaxonName?

I also noticed that https://github.com/marieALaporte/samara/blob/master/src/main/resources/org/planteome/samara/apsnet/taxonMap.tsv#L3
uses NCBI:9606 instead as NCBITaxon:9606 for the ID

since the NCBI id not provided by APS, the verbatim/mapped names are needed to make the desired ncbi mapping explicit. So, for example:

providedId
providedName
resolvedId
resolvedName

[empty]
Homo sapiens
NCBITaxon:9606
Homo sapiens

Note that both NCBITaxon: and NCBI: prefixes are supported.

for the providedTaxonName I took the source taxon name column from the aps scrape file. I hope this is the column I should have used

@marieALaporte thanks!


Add support for P<>K bridge

Goal:
Allow simulating transactions between Polkadot<>Kusama

related https://github.com/AcalaNetwork/chopsticks/pull/695 https://github.com/AcalaNetwork/chopsticks/issues/681

Here is an example tx from mainnet that could be used for testing purpose:
Source @ Asset Hub Kusama: https://assethub-kusama.subscan.io/block/7144107
Relayed @ Bridge Hub Kusama: https://bridgehub-kusama.subscan.io/block/3481382?tab=event
Received @ Bridge Hub Polkadot: https://bridgehub-polkadot.subscan.io/block/2764368?tab=extrinsic
Dest @ Asset Hub Polkadot: https://assethub-polkadot.subscan.io/block/6445172?tab=event&event=6445172-1
The bridge message pallet requires proof for relayed messages and I don't think we can easily spoof it without modify the upstream code. However, what it does eventually is to just dispatch some XCM, which we maybe able to do without using the bridge message pallet.
So the work that needs to be done are:

Detect bridge events BridgeKusamaMessages.MessagesAccepted / BridgePolkadotMessages.MessagesAccepted
Take the XCM out somehow
Identify the dest parachain
Dispatch the XCM somehow on bridge hub???
Profit

* Detect bridge events `BridgeKusamaMessages.MessagesAccepted` / `BridgePolkadotMessages.MessagesAccepted`

* Take the XCM out somehow

* Identify the dest parachain

* Dispatch the XCM somehow on dest bridge hub???

The message should be stored in the bridge message queue as encoded bytes: https://github.com/paritytech/polkadot-sdk/blob/master/polkadot/xcm/xcm-builder/src/universal_exports.rs#L381-L387.
But I think if we by-pass the real relay (message proofs) and "directly" trigger XCM dispatch on the dest BH, we won't test the real transport, which is also important here.

for by-passing, it should be possible to read XCM on the source BH: OutboundLanes and OutboundMessages
but not sure how to by-pass everything, maybe trigger XCM dispatch on the target BH with DMP/HRMP either enqueue_inbound_downward_messages or 'enqueue_inbound_horizontal_messages' - prepare some "valid" storage data with set_storage or set_validation_data or whatever

we can inject DMP/HRMP to the target BH but unsure if we can mimic the exact same XCM behaviour compare to dispatched by the message pallet

blocked by https://github.com/paritytech/polkadot-sdk/issues/4793

hello, any ETA on this? we would like to E2E test Kusama root track bridging to Hydration on Polkadot


Incomplete documentation - very confusing

The document "Quickstart" under the section "Deploy a contract using the CLI" (https://docs.aztec.network/dev_docs/getting_started/quickstart#deploy-a-contract-using-the-cli)
I don't know what on earth it's telling me to do. There is no method in the aztec-cli to "declare-accounts". I've spent way to long trying to figure this out.
https://github.com/AztecProtocol/aztec-packages/blame/f6573fa906a8f17af378176408d7150a8216ac95/yarn-project/end-to-end/src/guides/up_quick_start.sh#L6-L10

@rahul-kothari we talked about this in discord. it's related to the #include_code macro - the description on the code block is called declare-accounts so it caused confusion here. but we've had no other reports so i'm closing


PowerRename and Image Resizer context menu entries back in the 'Show more options' menu (Win11)

Description of the new feature / enhancement
There should be an option to make PowerRename and Image Resizer context menu entries appear in the 'Show more options' menu in Windows 11.
Scenario when this would be used?
Many users of Windows 11 use a registry tweak to force Explorer to only show the 'Show more options' menu by default and with the change in the PowerToys v0.60.0 update those context menu entries only appear in the original Windows 11 context menu and thus those entries are inaccessible to the users using this registry tweak.
Supporting information
No response

#19233?


x/playground: Inform users of run / format shortcuts

https://tour.golang.org/welcome/1 includes the helpful tips that shift+Enter runs the program, and ctrl+Enter runs the formatter.
These shortcuts exist also at play.golang.org, but are not discoverable.
What version of Go are you using (go version)?

play.golang.org

Does this issue reproduce with the latest release?
not applicable
What operating system and processor architecture are you using (go env)?
not applicable

/cc @toothrot

Thank you @NiloCK , I didn't know about Shift+Enter in the Playground to run the code, so I was left puzzled with Ctrl+Enter that would only format it, and I was about to create a bug specifically to ask for a keyboard shortcut for "Run"...
Now I realize that "shift-enter" is in the tooltip of the "Run" button at https://go.dev/play/  (I don't know if it was there all along, or a recent addition).
Yes definitely using the mouse for this, like I did, is suboptimal, and it would be very useful imo to make this information more prominent.


[DOCS] Add Badges

Issue Description
I would like to add interactive badges in a table format, int the header-section of the README documentation. This will enhance the Engagement & interface of the README.
I would like to work on this issue. can you assign this issue to me. @RymMichaut
Screenshots or Examples (if applicable)

Proposed Solution (optional)
Use badges to add dynamic & real time updating that will show simple stats.
Code of Conduct

[X] I have checked the existing issues.
[X] I am willing to work on this issue (optional)

hey @yashksaini-coder that would be good.
Can you limit the badges to the following, as in the attached picture?

Product version (could this be automated with the version in the doc
Licence type (Apache 2.0)
Python version needed 3.8

And three other cards to illustrate the current links:

📚 Explore the docs
🫱🏼🫲🏼 Discord support
👀 Demos & Examples

See image below:

@RymMichaut sure I can do that

@RymMichaut sure I can do that, can you add more for hacktoberfest and assign this issue to me

Sure.
Ensure you respect the charter by replacing the purple with the orange #ff371a

Yes no problem at all. @RymMichaut will follow your guidelines & make sure to meet your requested changes

hey @jrobinAV  i see the pr is still open , would like to give my contribution to this issue, please assign this to me

i would love to work on this @RymMichaut

@snehaamujri there's already someone on it @yashksaini-coder
But let's see how both of you perform :)

Hi @RymMichaut, can I work on this issue?

Hi @RymMichaut, can I work on this issue?

@RymMichaut Could you please review my code? If there are any issues or necessary changes, I’d be happy to resolve them. Otherwise, kindly proceed with merging my pull request.


mime: ParseMediaType reports an error on value that is not quoted but contains a space

What version of Go are you using (go version)?

$ go version 1.15

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/candeemis/Library/Caches/go-build"
GOENV="/Users/candeemis/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/candeemis/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/candeemis/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/candeemis/play-ground/go-mime-parser/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/z9/3q3874k11pd_h45kz05lngr40000gn/T/go-build706623001=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
After spending a few hours on debugging, I found that ParseMediaType function in mime/mediatype.go module throws the following error while parsing media type from the content type as application/pdf; x-unix-mode=0644; name=RG Mellowmessage 4.12.20.pdf
error:

Failed to ReadParts: mime: invalid media parameter

ParseMediaType function internally calls another function consumeMediaParam to parse the individual media types for example from the above-mentioned content type x-unix-mode and name (which is the name of the attached file) are two separate types. consumeMediaParam incorrectly parses the media type values if they contain any white space. For example, name contains the white spaces in the above-mentioned content type. So it assigns RG only as of the value to name key and keeps trying to parse the rest. Whereas, in the rest, there isn’t any key separated by = sign. That's why it throws the above-mentioned error.
What did you expect to see?
It should parse RG Mellowmessage 4.12.20.pdf as the value of name key. Which means it should ignore the white space in the value.
What did you see instead?
The following error:

Failed to ReadParts: mime: invalid media parameter

based on RFC 2045 Section 5.1

 parameter := attribute "=" value

 attribute := token
              ; Matching of attributes
              ; is ALWAYS case-insensitive.

 value := token / quoted-string

 token := 1*<any (US-ASCII) CHAR except SPACE, CTLs,
             or tspecials>

That looks like a malformed header

based on RFC 2045 Section 5.1

 parameter := attribute "=" value

 attribute := token
              ; Matching of attributes
              ; is ALWAYS case-insensitive.

 value := token / quoted-string

 token := 1*<any (US-ASCII) CHAR except SPACE, CTLs,
             or tspecials>

That looks like a malformed header

@seankhliao But the following doesn't rule out spaces in the value, no? 🤔
value := token / quoted-string

     token := 1*<any (US-ASCII) CHAR except SPACE, CTLs,
                 or tspecials>

     tspecials :=  "(" / ")" / "<" / ">" / "@" /
                   "," / ";" / ":" / "\" / <">
                   "/" / "[" / "]" / "?" / "="
                   ; Must be in quoted-string,
                   ; to use within parameter values

@seankhliao But the following doesn't rule out spaces in the value, no? 🤔
value := token / quoted-string

     token := 1*<any (US-ASCII) CHAR except SPACE, CTLs,
                 or tspecials>

     tspecials :=  "(" / ")" / "<" / ">" / "@" /
                   "," / ";" / ":" / "\" / <">
                   "/" / "[" / "]" / "?" / "="
                   ; Must be in quoted-string,
                   ; to use within parameter values

it could contain spaces, but in a quoted string, not the bare token you demonstrated

it could contain spaces, but in a quoted string, not the bare token you demonstrated

CC @bradfitz via owners.

CC @bradfitz via owners.

@candeemis Do you agree that mime.ParseMediaType behavior of reporting an error on that input is correct, in that it matches the specification?

@candeemis Do you agree that mime.ParseMediaType behavior of reporting an error on that input is correct, in that it matches the specification?

@dmitshur Well, I can say that the current implementation satisfies the RFC specification. Meanwhile, I still see room for improvement to go one step further a bit different implementation. That would not only satisfy the specification but also ignore minor problems like the one mentioned in the issue. I am working on that concept, but due to time constraint, I may take a bit of time.

@dmitshur Well, I can say that the current implementation satisfies the RFC specification. Meanwhile, I still see room for improvement to go one step further a bit different implementation. That would not only satisfy the specification but also ignore minor problems like the one mentioned in the issue. I am working on that concept, but due to time constraint, I may take a bit of time.

@candeemis Do you mean like observing Postel's Law?

@candeemis Do you mean like observing Postel's Law?

Sorry for the very late reply. But yes, as kode4food mentioned above, we should be a bit easy on the implementation as suggested by the Postal's Law/Robustness Principle


Set up was not assigning my public IP. Solved.

Hi everyone!
I have PiVPN running on a Pi Zero and for whatever reason, during setup, the script was not assigning my public IP to the .ovpn user config file. It may have been because my ISP recently gave me an IPv6 address instead of my original static IPv4 addr. I have since changed my router settings to force an IPv4 public IP.
On my iOS OpenVPN app I was getting a connection error of:

UDP send error: send: No route to host

After hours of scouring Google, GitHub, and OpenVPN forums, I was at my wits end. In desperation I looked at my old .ovpn and noticed that my public IP was in the line:

remote My IP 1134

For some reason the IP was not being included in the newly generated .ovpn or showing during installation using the pivpn script for Stretch.
The Solution
I simply edited the .ovpn file at:

/home/pi/ovpns/

to include my public IP at

remote 1134  <- What it looked like

remote My new IPv4 IP 1134 <- What I changed it to

and VIOLA it worked!
This may have been an obvious fix to the veteran users around here. Just wanted to share this in case anyone else gets stumped by this issue!
I hope I formatted this correctly.

ty for your input!! =)
Labeled this as information for further reference!
closing issue! =)
feel free to reopen if you need anything else.


Add some news posts

how easy it is to make a video for @bgruening
we left twitter, follow us on mastodon (outdated)
updated quoting features of the GTN to support citing URLs + author names, it's a standards compliant use of blockquote that basically no one uses, and no client renders, so we patch it in in JS. (sub-optimal for non-web clients, but I don't really feel like doing it in a plugin today.)

Feel free to merge it @bgruening

Thanks! Super cool


Consider to move optional argument to the end

testing #31475
The signature of startDebugging marks the first parameter as optional'ish which looks a little weird to me. I would have written it the other way around. Unsure why it done this way?
export function startDebugging(nameOrConfiguration: string | DebugConfiguration, folder?: WorkspaceFolder): Thenable<boolean>;

The only alternative would be something like this:
function startDebugging(folder: WorkspaceFolder, nameOrConfiguration: string | DebugConfiguration)
function startDebugging(configuration: DebugConfiguration)

But again, the existence of the second conveys the wrong message: the simple API is probably not what you need.

I understand, if DebugConfiguration makes use of variables like ${workspaceRoot} it needs a folder. I believe I just not a friend of the foo | undefined declaration but I can live with it

No change planned.


docs: update URL to in-toto bundle spec

Summary
Update URL to the in-toto bundle spec. The current link is broken.
Release Note
NONE

Documentation
Doc update included in PR

@joemiller can you please rebase your PR then we can merge it, thanks

@cpanato rebased 👍


Get package import broke my app

If i import this package not problem but if i only use it in one place for example only just 'GetMaterialApp' i didnt debug and i receive an error
or
if i import this package when project is debugging and use it anywhere  receive an error
and if i delete import, project can debug

i tried flutter clean and pub get and everything is okey in flutter doctor

My Bad


Refresh Token

Has the refresh token functionality gone away somewhere between recent versions of the old package, and this one?

There has been no code changes that have affected this. Do you have any more information?

ember-simple-auth-token had the following configuration options:
  ENV['simple-auth-token'] = {
    serverTokenRefreshEndpoint: ENV.apiHost+'/api/v1/sessions/refresh',
    refreshAccessTokens: true,
    timeFactor: 1000,
    refreshLeeway: 60 // Refresh the token 1 minute before it expires.
  };

I just didn't see any discussion of this stuff in the README here, and wanted to make sure it would still work  :)

This issue appear to be because I'm using an old version of node. I get warnings on ember-cli when using newer versions so figured I should leave it behind.


rouge-su

Hi,
I want to use rouge-su metric to evaluate my results , can this script compute rouge-su?
Thanks

Sorry for this --very-- late reply.
ROUGE is initially a Perl script. This python package is a wrapper around the original script. Using this package, you can set your own set of arguments that will be used during ROUGE call.
This means you should refer at ROUGE docs i.e. ROUGE/README.txt and in particuliar  the set of options and line 174 that mention -U


Bug 2353: Date Last Reconciled Update

GitHub Issue (if applicable): #2353 #2354
Trello Link (if applicable):
Explanation of Bugfix/Feature/Modification:
A clear and concise description of what changes you've made and why.
There are a couple of changed nodes we should be listening to:

Upon reconciliation through standard flow
Upon date change of a transaction
Upon removal of a reconciled transactions
Modifying the changed nodes to check the account value, the reconcile modal and any changes to the reconcile icon.

Check for single day since 1 "days" should be "1 day"
Some other changes:

Consolidating the container logic into one and using the ID to indicate which one to use during which feature
Removing duplicate styles

FYI @gpeden, please feel free to review.

@tonyt-vo  Nice! Tested and works for me.  I might use one line of code to set the days since text, but that's just a personal preference


Make retrieval of server variables pluggable

We're currently using bugsnag on a google app engine application using Laravel.
Unfortunately, the following line is causing us some issues: https://github.com/bugsnag/bugsnag-php/blob/master/src/Bugsnag/Request.php#L62
Now, given that this isn't bugsnag-laravel, I understand that we can't leverage any of the configuration or request/response libraries.  Although it would be ideal if the mechanism used to obtain values like these was pluggable.  Then, bugsnag-laravel could ship an implementation that retrieves the variables from its own abstraction.  Then, bugsnag-php can ship defaults that use the superglobals.

Thanks. I definitely plan to get this implemented in v3.

Going to implement this using a request resolver to resolve a request object.

If there's no request, we can have a nulled object.

Will land in 3.x shortly.


refac: Adjust lido-exporter Grafana dashboard

Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 44.70%. Comparing base (7948325) to head (0a675f8).

Files with missing lines
Patch %
Lines

cmd/lido-exporter/metrics/metrics.go
0.00%
3 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #424      +/-   ##
===========================================
- Coverage    44.71%   44.70%   -0.01%
===========================================
  Files          115      115
  Lines         9992     9994       +2
===========================================
  Hits          4468     4468
- Misses        5031     5033       +2
  Partials       493      493

Flag
Coverage Δ

unittests
44.70% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


I have changed my git data but nothing changes

I changed my git data (.name and .user) but in the moment that I'm creating a template, PkgTemplates is not recognizing this change. When I do ?Templates I see that
user::AbstractString="": GitHub (or other code hosting service)
username. The default value comes from the global Git config
(github.user). If no value is obtained, many plugins that use
this value will not work.
•  authors::Union{AbstractString, Vector{<:AbstractString}}="Veenty
<EMAIL_ADDRESS>and contributors": Package authors.
   Like user, it takes its default value from the global Git config
   (user.name and user.email).

How can I make the user thing recognize my user?

Is this the same issue as https://github.com/JuliaCI/PkgTemplates.jl/issues/327

Closing this now because I assume it has been resolved, without comment from here.
If not, please repopen and  post the full command you are running (or better yet a MWE), and the outputs of running it.
Sorry this got missed, for a while the package was without active maintainers.
Its still pretty light on them.


Provide sorting options

Hello there! Thank you for a fantastic addition to Github!
It is making life so much easier than doing it with shell magic and parsing the DOM ...
With that being said:
I am mostly interested on the projects that have the most modifications. And I can imagine different people might want to different things based on different criteria.
Exposing more of the sorting options would be a nice addition to this nifty tool!

I am mostly interested on the projects that have the most modifications. And I can imagine different people might want to different things based on different criteria.

Fair point, and I have considered it myself as well. However, this would quickly grow rather complex, not to mention all the UI work (which is my weakness) that would need to be done to enable the additional logic.
Maybe a list of the settings you had in mind could help narrow down on the specific needs of the community ?
Moreover, if you have the time and motivation for it, feel absolutely free to contribute a PoC : I will gladly review your PR. :)

I have promised (the few time that I spare) to other open sources already; but be sure I'll round-robin to this one.
I have anyway cloned the project locally 😀

I'm thinking useful-forks could at least provide customization for deciding how many commits should be used to filter out based on ahead_by and behind_by.
There are too many instances where some forks are displayed just because someone added a merge commit on the default branch (master or main). Same goes for forks which are thousands of commits behind (although that doesn't necessarily mean the fork doesn't contain some interesting additional content).


code-server --cert fails and code-server --cert "" starts the server without SSL

When I run code-server --cert it fails and code-server --cert "" starts the server without SSL?
How should we properly enable SSL using self signed cert?

v1 handles the cert flag differently than v2.
In v1 if you leave off the cert flag it'll generate a self-signed certificate.
In v2 it won't generate a certificate unless you explicitly tell it to by passing the cert flag.
I recommend using v2 if you can.

Thanks @code-asher i tiried this tho code-server --cert but i receive "argument missing" error when i try this?

Would you be able to try v2 instead? https://github.com/cdr/code-server/releases/tag/2.1638-vsc1.39.2
If you need to stick with v1 then drop the --cert flag and it should work.

Maybe I'm doing something wrong but I'm running the latest 2.1638-vsc1.39.2 like this:
code-server --auth password --cert --port 8443

But that's the output:
info  Server listening on http://localhost:8443
info    - Using custom password for authentication
info    - Not serving HTTPS

I also wiped my my ~/local/share/code-server and I can see that the certificate doesn't get generated.
The empty --cert parameter was generating the certificate in the previous version i was using (2.1523-vsc1.38.1)

@code-asher

Ah you're right something happened to the latest version of v2.

It seems VS Code's parser changed to filter out empty string arguments. Thanks for the bug report! :heart: We'll have a new version released by end of day.


Consider consistency for the exercises

Is there a style guide for prolog?
Are these styles encouraged or enforced?
Are there any conventions that we should adopt on this track for the sake of consistency?
Can we enforce these?
Is there a linter? Are there many? Should we use one?
Is there a common convention for filenames? If not, what should our convention be?

Note that this is about the exercises (the test suites and code examples), not people's solutions.

As far as I can tell, Prolog isn't used very often so I can't find any style guides for it. I did, however, find this linter, which should hopefully help minimize bugs.

Nice, yeah I think it's worth adding a linter to this repo, even if we don't end up recommending one to people who do the exercises.

There is this though a bit dense https://arxiv.org/pdf/0911.2899.pdf

Just something I noticed when writing the test documentation-- for simplicity, all tests should have the filename tests.plt or similar.

for simplicity, all tests should have the filename tests.plt or similar.

That sounds great. In that case, we can change the config.json to have that value as a string (no need for a regex).

Cool. I took test_pattern out in the hello-world PR assuming we would have "test" in then name. I will have to update the test file name though.

Oh, right. Yeah, we don't actually need the test pattern.

Yeah, I saw that in someone's VERY comprehensive launch checklist! ;-)

I wrote it and I still forgot. This is why we have checklists!


Not all ometiff have StructuredAnnotations...

Was getting this error in Vitessce because StructuredAnnotations is not defined in the OMEXML for the CODEX data.  Tested out locally in Viv and confirmed the fix with each OME-TIFF demo file.

Confirmed that the fix works in Vitessce.


Take setup-phraseapp GitHub Action under phrase org

Hey there!
I want to propose you take under your wing winify-ag/setup-phraseapp GitHub Action or, maybe create a new one and we can redirect everyone there.
I don't use the action anymore and don't have too much interest in maintaining it now. Although I see people have interest in it.
A few jobs ago when I was at Winify, I created this action, that downloads Phrase CLI.
I don't work there for a few years now, and the company also does not exist anymore.
Thanks for your understanding!

Hi @iamandrewluca, thank you for sharing. We also had some ideas to provide a similar action ourselves. So this works perfectly if you already built one! :tada:  Hope in the next few weeks I'll have some time to to take a closer look and take over the maintenance to make it an officially supported action

Instead of transferring the repo. I would recommend starting from scratch using actions/javascript-action which has better practices and is up to date, and just take the code from the original repo, which is under MIT Licence

@theSoenke Once this has been decided please make a post on winify-ag/setup-phraseapp to let people know what is decided what is available for us to use.

@skeeping when phrase makes an official action, I will add that info to the setup-phraseapp repo

@iamandrewluca We created https://github.com/phrase/setup-cli

@jablan Cool, Cool, Cool!!!
I will add a message to our repo, and redirect everyone to the new GitHub Action!


hud10837/map-rotation-kotlin

This was a very simple sample. Happy to edit it more if needed but I left in the steps to make it as clear as possible. @TADraeseke could you take a first look? Thanks

Thanks @TADraeseke @Rachael-E could you take a quick look again?

Thanks Trevor, merging now


Underline color of input doesn't change

I have gone through these following points

[x]   Check latest documentation: https://docs.nativebase.io/
[x]   Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[x]   Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[x]   Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[x]   For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[x]   For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description
I'm trying to change the success/error colors of the line that appears below an input. By default it is green/red. I have set the success and error props on <Form/> but don't see any change to the color.
node, npm, react-native, react and native-base version, expo version if used, xcode version
node: 8.10.0
yarn 1.13.0
react native: 0.57.8
native-base: ^2.12.1
xcode: 10.1
Expected behaviour
Change the <Form> success and error props should change the input color.
Actual behaviour
Input error and success colors do not respect the settings.
Steps to reproduce

Is the bug present in both iOS and Android or in any one of them?
I can only test this project on iOS.
Any other additional info which would help us debug the issue quicker.
<Form
        success="#4998F7" // light blue
        error="#a335ee" // red

      >
        <Item
          success={isSubmitted && isEmailValid === true}
          error={isSubmitted && isEmailValid === false}
        >
          Email
          <Input
            ref={this.handleEmailRef}
            onChangeText={this.handleEmailChange}
            onSubmitEditing={this.handleSubmit}
          />
          {isSubmitted && isEmailValid === true && (
            <Icon name="checkmark-circle" style={styles.successIcon} />
          )}
          {isSubmitted && isEmailValid === false && (
            <Icon name="close-circle" style={styles.errorIcon} />
          )}
        </Item>
      </Form>

In this case I would expect the line to render in blue, not green.

Works fine with boolean conditions

My question is how to change the color of the underline to something other than red/green.

Your issue description says

I'm trying to change the success/error colors of the line that appears below an input. By default it is green/red. I have set the success and error props on  but don't see any change to the color.

Accordingly i have answered with the snippet and output.

In this case I would expect the line to render in blue, not green.

To do so, you can customize the color of success / error input by ejecting theme

Ah, themes. That seems to be what I'm looking for. Thanks!


extendedAE incompatibility.

Crash when trying to open the game with extendedAE.
NeoForge version: 21.0.95
AE2WTLib version: 19.0.10-beta (I tried with all available versions.)
ExtendedAE version: 1.21-1.1.0-neoforge (I tried with all available versions.)
crash-2024-07-15_22.51.53-fml.txt

fixed in ExtendedAE (https://github.com/GlodBlock/ExtendedAE/issues/209)


netty 4.0.37 causes hangs in Apache Cassandra

Ever since we updated Apache Cassandra to using netty 4.0.37 from netty 4.0.36 we have been seeing random hangs on our OSS test suite where a response from the server to the client seemingly is getting lost or something (the client just hangs) until the test times out.  With the OSS test suite it only happens sometimes, it isn't super easy to reproduce (right now I can only reproduce it using a 1.5 hour test suite we have for DataStax Enterprise) but I bisected it to f984870ccca133d6056e8b0df0b2352f8f90b0fe.  I then reverted f984870ccca133d6056e8b0df0b2352f8f90b0fe from 4.0.37 and I can run my tests with 4.0.37+f984870ccca133d6056e8b0df0b2352f8f90b0fe reverted and everything seems good.

Hmm ... Can you also try 4.0.38.Final?
Also is it using nio or epoll? Can I run the testsuite locally somehow?

Am 01.07.2016 um 21:12 schrieb Jeremiah Jordan<EMAIL_ADDRESS>Every since we updated Apache Cassandra to using netty 4.0.37 from netty 4.0.36 we have been seeing random hangs on our OSS test suite where a response from the server to the client seemingly is getting lost or something (the client just hangs) until the test times out. With the OSS test suite it only happens sometimes, it isn't super easy to reproduce (right now I can only reproduce it using a 1.5 hour test suite we have for DataStax Enterprise) but I bisected it to f984870. I then reverted f984870 from 4.0.37 and I can run my tests with 4.0.37+f984870ccca133d6056e8b0df0b2352f8f90b0fe reverted and everything seems good.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I tried 4.0.38.Final and it still has the problem.  I tried both nio and epoll show the problem.

Sadly you can't run the "reliably fails" test suit locally.  The straight cassandra-testall tests show it sometimes.  These runs of our OSS all hung:
http://cassci.datastax.com/job/trunk_testall/975/
http://cassci.datastax.com/job/trunk_testall/982/
http://cassci.datastax.com/job/trunk_testall/985/
@jkni may have something that you can use to reproduce it, he is checking right now if the hang he is seeing is fixed by going back to 4.0.36 as I type.

Looking at that patch....  Is the issue handlerState isn't volatile?

Good idea @tjake I'll try that and see if it does the trick.

@tjake @JeremiahDJordan as far as I remember this is only ever accessed / modified from the same thread. Do I miss something ?

@JeremiahDJordan also do you modify the pipeline from outside the EventLoop at all ?

Not the pipeline but the ChannelHandlerContext can be called from outside the event loop I think.

@tjake ok... can you tell me why you think the handlerState should be volatile ?

I'm not sure tbh. Just looked like a racy condition.

4.0.38 with that changed to volatile still has the problem, so that isn't it.

@normanmaurer
If the pipeline isn't called from inside the eventloop you hit this
https://github.com/netty/netty/commit/f984870ccca133d6056e8b0df0b2352f8f90b0fe#diff-6b8633f415085302023f47004357891aR192
That calls callHandlerAdded0() from a random thread which calls ctx.setAdded()

Also the fact that that section used to waitOnFuture and now it doesn't is different.

Also, callHandlerAdded0 is no longer being called under a synchronized block in that scenario whereas before it was calling synchronized (DefaultChannelPipeline.this) in the Runnable

At the moment, we think we're seeing this fairly regularly on cassandra-dtest runs against trunk, which is running 4.0.37. The same dtests do not reproduce against 4.0.36 (yet). It's a little bit tedious, but it might give you an option to reproduce.
Any test from this run with a "Timed out creating connection" error is an option.
On my local Linux box, the best reproduction I have is
while nosetests -vvv -s json_test.py:FromJsonSelectTests.select_using_secondary_index_test; do :; done

On my machine, this takes 15 seconds to run and I typically have to loop for 15-20 minutes to reproduce. On 4.0.36, I've run the test for over an hour with no failures.
If you'd rather not get cassandra-dtests running or cannot reproduce on your machine, I'm available to help debug. jkni on cassandra-dev IRC.

Thanks!
I will investigate and ping you guys...

Am 01.07.2016 um 22:56 schrieb Joel Knighton<EMAIL_ADDRESS>At the moment, we think we're seeing this fairly regularly on cassandra-dtest runs against trunk, which is running 4.0.37. The same dtests do not reproduce against 4.0.36 (yet). It's a little bit tedious, but it might give you an option to reproduce.
Any test from this run with a "Timed out creating connection" error is an option.
On my local Linux box, the best reproduction I have is
while nosetests -vvv -s json_test.py:FromJsonSelectTests.select_using_secondary_index_test; do :; done
On my machine, this takes 15 seconds to run and I typically have to loop for 15-20 minutes to reproduce. On 4.0.36, I've run the test for over an hour with no failures.
If you'd rather not get cassandra-dtests running or cannot reproduce on your machine, I'm available to help debug. jkni on cassandra-dev IRC.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@tjake it will not call callHandlerAdded0(...) from a random thread as it schedule it on the EventLoop and then return.
See https://github.com/netty/netty/commit/f984870ccca133d6056e8b0df0b2352f8f90b0fe#diff-6b8633f415085302023f47004357891aR192 .
Or is there anything Im missing ? All the callHandlerAdded0(...) /  callHandlerRemoved0(...) should always happen in the EventLoop. That was one of the points of the PR ;)

@JeremiahDJordan @tjake @jkni one thing I would be interested in is if you ever see a condition in your AbstractChannelHandlerContext methods where isAdded() returns false. Could you test this and if so print the stacktrace ? Basically for all methods in there, like for example:
https://github.com/netty/netty/commit/f984870ccca133d6056e8b0df0b2352f8f90b0fe#diff-5d5eb2c10e26184226a4f493481bcf87R189

@normanmaurer, based on @tjake and @JeremiahDJordan findings, I think I found a way to reproduce this in a utest - pushed to gh in this branch. Hope this helps.
Wondering whether it makes a difference that the old version used waitOnFuture but the new one doesn't:

@snazy did you manage to get a thread dump?

So just running the unit test ?

Am 02.07.2016 um 10:14 schrieb Robert Stupp<EMAIL_ADDRESS>@normanmaurer, based on @tjake and @JeremiahDJordan findings, I think I found a way to reproduce this in a utest - pushed to gh in this branch. Hope this helps.
Wondering whether it makes a difference that the old version used waitOnFuture but the new one doesn't:
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Yup, seems like just the new utest helps to reproduce it.

Am 02.07.2016 um 11:33 schrieb Norman Maurer<EMAIL_ADDRESS>So just running the unit test ?

Am 02.07.2016 um 10:14 schrieb Robert Stupp<EMAIL_ADDRESS>@normanmaurer, based on @tjake and @JeremiahDJordan findings, I think I found a way to reproduce this in a utest - pushed to gh in this branch. Hope this helps.
Wondering whether it makes a difference that the old version used waitOnFuture but the new one doesn't:
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@snazy do I need to somehow install / configure cassandra first or is it self-containing ?

self-containing - just execute the utest with ant test -Dtest.name=NativeTransportStressTest or in an IDE. It complains with com.datastax.driver.core.exceptions.NoHostAvailableException: All host(s) tried for query failed (tried: localhost/<IP_ADDRESS>:53043 (com.datastax.driver.core.exceptions.OperationTimedOutException: [localhost/<IP_ADDRESS>] Operation timed out)) with 4.0.37

@snazy awesome! Will look into it later today.

Or use generate-idea-files and run the utest in IDEA - similar for Eclipse.

@snazy @JeremiahDJordan @tjake ok I took the time to debug the issue and thanks to @snazy it was quite easy to understand whats going on :) I would have taken me a lot longer without the unit test.
To make it short, its a cassandra bug IMHO and you was just "lucky" that it never broke before.
So here is the problem.... In Cassandra you are using your own EventExecutor implementation which break the EventExecutor contract.  The contract is that execute(...) calls must be executed in a serial fashion which is not the case here as you just return false from the inEventLoop() method and then delegate the execute call to an underlying Executor, which gives no such guarantees. Because of this a race can happen where an "event" may be handed over to an AbstractChannelHandlerContext before setAdded() is called, which will then have the effect that the handler is skipped.
This was previous no problem as we synchronized which had other problems, thus we removed it.
So to make it short your EventExecutor implementation must guarantee ordering. That said you can have an EventExecutorGroup that holds multiple EventExecutor implementations and each of them only needs to guarantee ordering by itself, not for the whole group.
Hope this makes sense.... Let me know if you have more questions.
@snazy when doing this patch it never fails for me with your branch:
iff --git a/src/java/org/apache/cassandra/service/NativeTransportService.java b/src/java/org/apache/cassandra/service/NativeTransportService.java
index 48839f1..e46b895 100644
--- a/src/java/org/apache/cassandra/service/NativeTransportService.java
+++ b/src/java/org/apache/cassandra/service/NativeTransportService.java
@@ -81,7 +81,7 @@ public class NativeTransportService

         org.apache.cassandra.transport.Server.Builder builder = new org.apache.cassandra.transport.Server.Builder()
                                                                 .withEventExecutor(eventExecutorGroup)
-                                                                .withEventLoopGroup(workerGroup)
+                                                                //.withEventLoopGroup(workerGroup)
                                                                 .withHost(nativeAddr);

         if (!DatabaseDescriptor.getClientEncryptionOptions().enabled)
diff --git a/test/unit/org/apache/cassandra/cql3/CQLTester.java b/test/unit/org/apache/cassandra/cql3/CQLTester.java
index 30ddf2c..f1fedce 100644
--- a/test/unit/org/apache/cassandra/cql3/CQLTester.java
+++ b/test/unit/org/apache/cassandra/cql3/CQLTester.java
@@ -344,7 +344,7 @@ public abstract class CQLTester
         server = new Server.Builder().
                             withHost(nativeAddr).
                             withPort(nativePort).
-                            withEventExecutor(new RequestThreadPoolExecutor()).
+                            //withEventExecutor(new RequestThreadPoolExecutor()).
                             build();
         server.start();

If you guys like I can fix it in cassandra... I think I have an idea how to fix it there while still give you the same behaviour as before. Just let me know

If you have an idea of how to fix it that would be great.

Alright stay tuned :)

Am 02.07.2016 um 17:18 schrieb Jeremiah Jordan<EMAIL_ADDRESS>If you have an idea of how to fix it that would be great.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@JeremiahDJordan @tjake @snazy so @belliottsmith made a good point on twitter that if the requirement of serial execution is not documented in the EventExecutor interface the change I made could be considered as a breaking API change in a bug-fix release. So I checked our docs and it seems like we never documented this 😭 .
I am currently looking into a way to change netty to work again with your code. I also have a patch with needed cassandra changes here to let it work with any netty version, but I wonder if I should not just fix it in netty after @belliottsmith made the point here... @JeremiahDJordan @tjake @snazy WDYT ?

http://netty.io/4.1/api/io/netty/util/concurrent/DefaultEventExecutor.html "execute all submitted task in a serial fashion"

@normanmaurer would you mind sharing the Cassandra patch anyway?

This is on the implementation but not on the interface...

Am 03.07.2016 um 09:43 schrieb Johno Crawford<EMAIL_ADDRESS>http://netty.io/4.1/api/io/netty/util/concurrent/DefaultEventExecutor.html "execute all submitted task in a serial fashion"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Once its complete yes

Am 03.07.2016 um 09:46 schrieb Johno Crawford<EMAIL_ADDRESS>@normanmaurer would you mind sharing the Cassandra patch anyway?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I would suggest revisiting the decision to impose this constraint.
In a system where requests have differing costs (and time to serve), forbidding out-of-order execution is debilitating, as it forbids QoS (long tasks block all short tasks to all clients being served by the same thread).  It also ensures a higher memory watermark that is harder to manage.
In Cassandra this would also pretty much bring down the entire cluster (to clients' perspective), at least temporarily, because of one node failure, as requests waiting for a failed node will prevent all healthy responses from responding to clients.
Further, limiting to sequential execution only at best permits CPU on the box to be utilised (assuming an even distribution of client connections, which is far from guaranteed - C* benchmarks generally do not saturate this); it prevents saturating the system when any other resource is the bottleneck (network, disk, other nodes).  You could of course pretend to execute sequentially by delaying the final step if you somehow can separate the completion task from the execution step, but that only fixes one of your problems.

@belliotsmith thanks for chiming in :) Like I said I think you have a point here and after investigate a bit I think I can remove this constraint easily while still preserve the fix that the original patch did. So let me do this:
Write a netty patch and a cassandra patch and then let us check both of them to see which one is preferable. Cassandra will still work the same with my cassandra patch btw.
That said I'm currently leaning to adjust netty and not cassandra. As this also makes netty more flexible in general

Sure - just giving my 2c (that, as you say, a more flexible netty is a good thing)

I think the netty patch is preferable if possible. There may be other projects out there which will have mysterious bugs after updating netty otherwise.
For Cassandra a netty patch would mean that users of older versions of Cassandra could update netty if they hit some problem without having to update to a newer version of Cassandra itself.

@JeremiahDJordan @tjake @jkni @snazy @belliottsmith PTAL https://github.com/netty/netty/pull/5506 . This fixes the issue for me without any cassandra changes.

Fixed by https://github.com/netty/netty/pull/5506


Why keras text classification examples are using Conv1D instead of Conv2D?

I was looking at examples of text classification using Convnets (e.g. imdb_cnn and pretrained_word_embeddings.py). I saw that 1D convolution is used. However, this does not make intuitive sense because we are not considering any local neighboring words in this way. For example if dimension of word embedding is 100 and convolution length is 5, we are only convolving over the emedding dimensions (e.g. [1,...,5], [2,...,6], ...) and not the adjacent words. By contrast, Conv2D can also consider local patterns of neighboring words. So my question is why Conv1D is used in the examples?

Conv1D takes care of neighboring words. A filter length of 5 would imply a context window of 5 words, i.e, the word embeddings of 5 words, not 5 elements within a single embedding. Images have height and width, so we use conv2d, sentences are linear lists of words, so conv1d. The "2d" or "3d" specifies how we loop through the matrix; its not the rank of the convolution kernel itself.

I see. Thanks for the clarification.

In the example imdb_cnn, when they use conv1d layer after embedding layer of dimension (None, 400,50), the resulting output dimension of conv1d is  (None,398,250). It seems that the conv1d layer is only applied across the length of the particular word, not along the number of words.
So how can you say that conv1d takes care of neighbouring words? In their example a filter length of 5 is only operating along the length of the word not along the context window of 5 words.

I think you are wrong @alihassanmirza because if operating only over singular words the output would be  of 400.

The embedding layer output is (batch_size, sequence_length, output_dim) while the conv1d input is (batch, steps, channels).  So in this case, the dimension of the vector space generated by the embedding layer represents the number of channels (like RGB in an image), while the context window will move on the other dimension (steps) which is the number of words in the sentence. A filter length of 5 would imply a context windows of 5 words in every channel, as @farizrahman4u said.


Bringing mumps back

By popular demand!
cc @sschnug

Current coverage is 91.47% (diff: 91.17%)

Merging #16 into master will increase coverage by 0.26%

@@             master        #16   diff @@
==========================================
  Files             5          6     +1
  Lines           273        340    +67
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            249        311    +62
- Misses           24         29     +5
  Partials          0          0

Powered by Codecov. Last update 34cd1f2...dbf1699


New version: vim.vim.nightly version 9.1.0354

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @chrisbra,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 877.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-04-18 17:27:52.576 [FAIL] Installer failed security check. Url: https://github.com/vim/vim-win32-installer/releases/download/v9.1.0354/gvim_9.1.0354_x64.exe Result: 0x80004005

(Automated response - build 877.)

@wingetbot run


Allow assingning multiple rules to a single item

The idea is to allow assigning multiple rules to a single item. That would allow to reuse of such rules more efficiently and have a more descriptive hierarchy which is useful for RBAC UI.
editPost
 |-(rule) editOwn
 |-(rule) editOwnGroup
 |-(rule) editCustomGroup(context=newbie)

The API might be:
public function getRuleNames(): array;

An item is allowed either if no rules are defined or if at least one rule allowed execution.
createPost
   v All
editPost
   o All
   v Own
   v OwnGroup
   o CustomGroup_____

TODO:

[ ] https://github.com/yiisoft/rbac/pull/79
[ ] https://github.com/yiisoft/rbac-php/pull/27
[ ] DB (https://github.com/yiisoft/rbac-db)
[ ] Spiral DB (https://github.com/yiisoft/rbac-spiral-db)


Question, Button.Image and SVG PCL [XF]

It's possible set a svg (PCL) icon in button image property?
I try this code:
<Button BackgroundColor="Transparent" VerticalOptions="Start" HorizontalOptions="Start" Image="ProjectName.Core.Resources.Images.arrow_back.svg, Converter={StaticResource SvgImageSourceConverter}}"/>
But the icon not displayed.

No, it's not possible, we'll need to make a new implementation, eg. CachedButton. The only way is to use our ImageService API:
var source = ImageSource.FromStream(async () =>
{
    return await ImageService.Instance
                                .LoadEmbeddedResource("ProjectName.Core.Resources.Images.arrow_back.svg")
                                .WithCustomDataResolver(new SvgDataResolver())
                                .AsPNGStreamAsync();
});

I try using ImageService API but have a follow error:
Method 'FFImageLoading.Svg.Platform.SvgDataResolver..ctor' not found.
System.IO.Stream stream = await ImageService.Instance                                 .LoadEmbeddedResource("Garzon.Core.Resources.Images.ic_add.svg")                 .WithCustomDataResolver(new SvgDataResolver())                                 .AsPNGStreamAsync();

You must do it in a platform specific code, I didn't add PCL implementation yet (but it should be easy to add this)


Multi-table support

When each file is read from rebuild-iptables, the first line is checked for a  table name (such as *nat).   If a name is found the rules in the file are added to that table.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

Since the CLA hasn't been signed in over 6 month for this PR I'm going to close it out at this point.  If the author would like to sign the CLA feel free to open up the PR again.


start outline

Includes potential implementations for arbitrage

Still rough, might make sense to adjust for wording in https://docs.gho.xyz/concepts/overview

This includes the a starting point using scaffold-eth-2


[Bug] The gem downloading is not fetches recent changes

When I download the current gem into my project, I get the gem version tagged as 1.0.0, which differs greatly from the main branch.
Please, bump a new tag pointing to the current main branch.
Thank you.

Hi Fattaf,
We want to apologize to you and the community because we haven't been paying enough attention to this gem. We are reorganizing our team to be able to maintain it properly. Here you can find the new version with the last changes,
Sorry again and regards.


Alfworld GPT-3 Results

Hi,
I wondered if you had more details or numbers from your GPT-3 results on Alfworld? For instance, do you have the splits of accuracy across the different subtasks (as in Table 3 in the paper)?
I would try to reproduce it, but I reckon the total cost would be > $100 and would like to avoid it if possible.

Hi, at the end of 134 instances, the six category
prefixes = {
    'pick_and_place': 'put',
    'pick_clean_then_place': 'clean',
    'pick_heat_then_place': 'heat',
    'pick_cool_then_place': 'cool',
    'look_at_obj': 'examine',
    'pick_two_obj': 'puttwo'
}

has the final result
134 r 0 rs [19, 19, 7, 17, 16, 8] cnts [24, 31, 23, 21, 18, 17] sum(rs)/sum(cnts) 0.6417910447761194

e.g. put tasks are 19/24 correct.

A more complete trajectory is at https://gist.github.com/ysymyth/01045e5b65651eccd63a5a46964b8216

Thank you!


Tooltip triggerOn "mousemove" not working on releases >= 3.6.1

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Steps to reproduce
Change the @antv/f2 version to 3.4.4 and the tooltip for "mousemove" events are working (https://codesandbox.io/s/funny-minsky-s96iq).

Environment
Info

f2
3.8.1

System
Ubuntu

Browser
Chrome Version 88.0.4324.182 (Official Build) (64-bit)

I played around with the F2 library and used https://antv.gitbook.io/f2/ as reference. An older version from a CDN was in one of the examples and the mousemove event worked fine. When I wanted to implement F2 into a project I imported the latest version and mousemove events stopped working. I understand that F2 is meant for mobile, would still be great to have mousemove events for webapps that server desktop and mobile users.

try this
  const canvas = document.querySelector('canvas')
  canvas.onmousemove = function(ev) {
    chart.showTooltip({
        x: ev.offsetX,
        y: ev.offsetY
      })
  }


How to reduce number of deprecation warnings?

We have 262 Capybara / RSpec examples. Using SitePrism 2.17 we get the warning
This config option is being removed in SitePrism v3. See UPGRADING.md on the repo for more details.
Going forwards the configuration / configurability will more closely mimic Capybara

4837 times (i.e. more than 18 times per example) in our build log, which is way more than the Travis CI website is capable to display.
I think there is at least one warning every time we expect existence of an element, i.e. we pass this line https://github.com/natritmeyer/site_prism/blob/v2.17/lib/site_prism/element_container.rb#L181
Is there any way to reduce the number of warnings?

Hi There Leo,
So ordinarily I'd agree with you (And I do agree that 4000+ is insane). Few points to note though.

3.0.beta when released removes all deprecated code (This will be released in the next week)
The config warnings you've mentioned there are only triggered when you either read or write to a config setting. So that means (If this is the only warning you get), that you read/write to a config setting on average 18 times per individual unit test? That sounds .... implausible?
However notwithstanding the above, yes, we do read the setting for implicit waits when performing the has_? wait check.

Considering the above I am happy for any of the following...

a PR which introduces a new setting to remove all warning messages, it itself would fire one warning message on initialisation (Don't like this personally*)
a PR which reduces the number of warnings / which messages fire warnings by setting some class instance variables so they only fire a maximum of say 3 times? (Better)

*I am planning on releasing a SitePrism logger where a lot of warning / error messages will bubble up and not pollute the stdout side of things, but that's likely to take a while longer to implement.
Considering all of the above, if you're happy raising a PR then I'd happily accept it. Also it's worth considering all of the warnings will disappear in 3.0, with a beta cut of the gem coming in the next few days - It might be worth trying to update?? There's an UPGRADING.md document here that briefly outlines the stuff you'd need to do / change.
I hope that explains / helps. Let me know if I can do anything else.

@luke-hill Thanks for providing some insight and outlining the roadmap.
Yes, that part of our code base is not the best, featuring way more than one expectation per example, lots have_? matchers there, hence frequently passing through
https://github.com/natritmeyer/site_prism/blob/eb68bd0cef120b281a12ba76d8117727b56fa84f/lib/site_prism/element_container.rb#L15-L21
I'd agree with you that waiting for 3.0 (or betas thereof) is probably best. Maybe we'll just keep this issue open nevertheless for others to find your explanations until 3.0 ships.

I too get the error, however, it occurs when calling set on an element. Example:
require_relative '../common/base_page'

module TestApp
  class Login < Common::BasePage

    set_url $config['ui']['apps']['test_app']['base_url']

    element :username, 'input#session_username'
    element :password, 'input#session_password'
    element :submit, '#create_session_link'

    set_url_matcher(/login/)

    def initialize
      load
    end

    def login(user_name, pass)
      username.set user_name
      password.set pass
      submit.click
    end
  end

For the time being, I now added this monkey patch to our project
# spec/spec_helper.rb
require 'site_prism/version'

fail "The monkey patch in #{__FILE__} is now obsolete. Please delete it." if Gem::Version.new(SitePrism::VERSION) >= Gem::Version.new('3')

module SitePrism
  class << self
    def show_removed_config_warning_messages; end
  end
end

Hi @leoarnold
I've cut both 3.0.beta and 2.17.1 this morning and both are released to rubygems.
For 2.17.1 I've put in a couple of bugfixes / optimisations, mainly I've reduced the warnings from appearing on all config settings, to only appearing on either block / writer assigns. Any reader methods won't trigger them now. You'll still get warnings, but I hope that you've gone from 4000+ to maybe 20-30 now across your suite. Can you try 2.17.1 and let me know if that's made it better (Without your monkeypatch)
If you feel inclined to do so you can try 3.0.beta. It is reasonably stable and has about 70% of the work I wanted to implement for the 3.0 cut. Naturally as a major release there are a few changes (Namely all config settings are now removed). But I'll let you decide what's best at the moment for your use case.
@mnohai-mdsol - The reason for so many warnings was that any call to merge_args (Which any DSL method or DSL meta-programmed method will do), triggered the check to the use_implicit_waits reader config, and this in turned triggered a warning - Which probably was an oversight on my behalf. Feel free to chip in and review more if / when you have time.
:point_up: same goes for yourself Leo if you're interested in picking up any of the TODO's.
I'm going to close this as fixed in both 2.17.1 AND 3.0.beta


Change the data structures for custom classes

Use arrays since they are faster than lists (My general rule of thumb is if size never changes, use array)
Use dictionaries for common Contains checks/index grabs.
Added a todo to CompAbilityTMBase that says to remove customIndex (this should probably be moved to TM_CustomClass so the usability is clearer (if cc is not null, it will have a customIndex because it gets loaded in at bootup)

See Pawn generator PR instead for an improved (though similar) implementation


Update cscript.md

'Option' is the term used for the command line switches used for the windows script host engine (at least in the help message) ,but not a parameter. Simple examples added.

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


Update apollo-ios support to 0.13.0

Just what I was looking for, thanks!
Should be merged in 👍

Closing this with the release of ApolloAlamofire 0.6.0, which depends on Apollo 0.19.0.


Taal: More elixirs

convergence_test replaces convtest
linear_structure replaces compute_linear_structure

I think I've addressed all of your comments in 99539f9, @sloede.