match fails with error "[!] undefined method `com' for nil:NilClass (NoMethodError)"

New Issue Checklist

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

Issue Description
I followed match tutorials from here,
but It doesn't work as it is supposed.
Complete output when running fastlane, including the stack trace and command used

❯ fastlane match appstore

Looking for related GitHub issues on fastlane/fastlane...

➡  `fastlane deliver download_metadata` crash: NoMethodError: [!] undefined method `version' for nil:NilClass
    https://github.com/fastlane/fastlane/issues/11046 [open] 8 💬
    16 hours ago

➡  Type Checking error: got 'Fixnum' for attribute 'fileEncoding' and undefined method `each' for nil:NilClass
    https://github.com/fastlane/fastlane/issues/11016 [open] 1 💬
    a week ago

➡  `app_store_build_number` fails with `undefined method `current_build_number' for nil:NilClass (NoMethodError)`
    https://github.com/fastlane/fastlane/issues/10447 [open] 11 💬
    a week ago

and 250 more at: https://github.com/fastlane/fastlane/search?q=undefined%20method%20%60com%27%20for%20nil&type=Issues&utf8=✓

🔗  You can ⌘ + double-click on links to open them directly in your browser.
/Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/fastlane_core/lib/fastlane_core/configuration/configuration_file.rb:43:in `eval': [!] undefined method `com' for nil:NilClass (NoMethodError)
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/fastlane_core/lib/fastlane_core/configuration/configuration_file.rb:43:in `eval'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/fastlane_core/lib/fastlane_core/configuration/configuration_file.rb:43:in `initialize'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/fastlane_core/lib/fastlane_core/configuration/configuration.rb:175:in `new'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/fastlane_core/lib/fastlane_core/configuration/configuration.rb:175:in `load_configuration_file'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/match/lib/match/commands_generator.rb:53:in `block (3 levels) in run'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.5/lib/commander/command.rb:178:in `call'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.5/lib/commander/command.rb:178:in `call'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.5/lib/commander/command.rb:153:in `run'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.5/lib/commander/runner.rb:476:in `run_active_command'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:66:in `run!'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/commander-fastlane-4.4.5/lib/commander/delegates.rb:15:in `run!'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/match/lib/match/commands_generator.rb:134:in `run'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/match/lib/match/commands_generator.rb:13:in `start'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/fastlane/lib/fastlane/cli_tools_distributor.rb:67:in `take_off'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.68.0/bin/fastlane:20:in `<top (required)>'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/bin/fastlane:22:in `load'
	from /Users/HyeonsuGo/.fastlane/bin/bundle/bin/fastlane:22:in `<main>'

Environment
Captured Output
Command Used: env
Output/Log

Successfully loaded Appfile at path '/Users/HyeonsuGo/workspace/iOS/Washow/fastlane/Appfile'
- app_identifier: 'com.jocoos.washow.app'
- apple_id<EMAIL_ADDRESS>- team_id: '5CXMSK555F'
-------
WARN [2017-12-01 21:22:34.39]: Generating fastlane environment output, this might take a few seconds...
DEBUG [2017-12-01 21:22:37.63]: Checking if there are any plugins that should be loaded...
<details><summary>✅ fastlane environment ✅</summary>

### Stack

| Key                         | Value                                       |
| --------------------------- | ------------------------------------------- |
| OS                          | 10.12.6                                     |
| Ruby                        | 2.2.4                                       |
| Bundler?                    | false                                       |
| Git                         | git version 2.13.6 (Apple Git-96)           |
| Installation Source         | ~/.fastlane/bin/bundle/bin/fastlane         |
| Host                        | Mac OS X 10.12.6 (16G29)                    |
| Ruby Lib Dir                | ~/.fastlane/bin/bundle/lib                  |
| OpenSSL Version             | OpenSSL 1.0.2g  1 Mar 2016                  |
| Is contained                | true                                        |
| Is homebrew                 | false                                       |
| Is installed via Fabric.app | false                                       |
| Xcode Path                  | /Applications/Xcode.app/Contents/Developer/ |
| Xcode Version               | 9.1                                         |

### System Locale

| Variable | Value       |   |
| -------- | ----------- | - |
| LANG     | en_US.UTF-8 | ✅ |
| LC_ALL   | en_US.UTF-8 | ✅ |
| LANGUAGE | en_US.UTF-8 | ✅ |

### fastlane files:

<details><summary>`./fastlane/Fastfile`</summary>

```ruby
# Customize this file, documentation can be found here:
# https://docs.fastlane.tools/actions/
# All available actions: https://docs.fastlane.tools/actions
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "2.68.0"

generated_fastfile_id "6f67a1fd-042a-4660-8fe4-363928105358"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."
    cocoapods
    # carthage
  end

  desc "Runs all the tests"
  lane :test do
    run_tests
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
	get_certificates(           # invokes cert
			development: true)
    get_provisioning_profile   # invokes sigh
    build_app(
			workspace: "Washow.xcworkspace",
			scheme: "Washow") # more options available
#upload_to_testflight

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)

    # upload to Beta by Crashlytics
	crashlytics(
    # keys for organization: Jocoos_temp
    api_token: "17814dd81835807f712cbfb0c9cee57818fbba48",
    build_secret: "9fcd12a24f2c4705b0e0a60bcb7449254ed98f7aa56f629d10dde033c9e09e32"
    )
  end

  desc "Deploy a new version to the App Store"
  lane :release do
    # sync_code_signing(type: "appstore")
    capture_screenshots
    build_app(scheme: "washow") # Build your app - more options available
    upload_to_app_store(force: true)
    # frame_screenshots
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://docs.fastlane.tools/advanced/#control-configuration-by-lane-and-by-platform
# All available actions: https://docs.fastlane.tools/actions

# fastlane reports which actions are used. No personal data is recorded.
# Learn more at https://docs.fastlane.tools/#metrics

`./fastlane/Appfile`
app_identifier "com.jocoos.washow.app" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address

team_id "5CXMSK555F" # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.68.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

CFPropertyList
2.3.5

activesupport
4.2.10

addressable
2.5.1

babosa
1.0.2

bigdecimal
1.2.6

bundler
1.14.6

claide
1.0.2

cocoapods
1.3.1

cocoapods-core
1.3.1

cocoapods-deintegrate
1.0.1

cocoapods-downloader
1.1.3

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

cocoapods-trunk
1.3.0

cocoapods-try
1.1.0

colored
1.2

colored2
3.1.2

commander-fastlane
4.4.5

concurrent-ruby
1.0.5

declarative
0.0.10

declarative-option
0.1.0

domain_name
0.5.20170404

dotenv
2.2.0

escape
0.0.4

excon
0.55.0

faraday
0.12.1

faraday-cookie_jar
0.0.6

faraday_middleware
<IP_ADDRESS>

fastimage
2.1.0

fourflusher
2.0.1

fuzzy_match
2.0.4

gh_inspector
1.0.3

google-api-client
0.13.6

googleauth
0.5.1

highline
1.7.8

http-cookie
1.0.3

httpclient
2.8.3

hurley
0.2

i18n
0.9.1

io-console
0.4.3

json
1.8.1

jwt
1.5.6

libxml-ruby
3.0.0

little-plugger
1.1.4

logging
2.2.2

memoist
0.15.0

mime-types
3.1

mime-types-data
3.2016.0521

mini_magick
4.5.1

minitest
5.4.3

molinillo
0.5.7

multi_json
1.12.1

multi_xml
0.6.0

multipart-post
2.0.0

nanaimo
0.2.3

nap
1.1.0

netrc
0.11.0

nokogiri
1.5.6

os
0.9.6

plist
3.2.0

power_assert
0.2.2

psych
2.0.8

public_suffix
2.0.5

rake
10.4.2

rdoc
4.2.0

representable
3.0.4

retriable
2.1.0

rouge
1.11.1

ruby-macho
1.1.0

rubygems-update
2.5.2

rubyzip
1.2.1

security
0.1.3

signet
0.7.3

slack-notifier
1.5.1

sqlite3
1.3.13

terminal-notifier
1.7.1

terminal-table
1.7.3

test-unit
3.0.8

thread_safe
0.3.6

tty-screen
0.6.3

tzinfo
1.2.4

uber
0.0.15

unf
0.1.4

unf_ext
<IP_ADDRESS>

unicode-display_width
1.1.3

word_wrap
1.0.0

xcodeproj
1.5.3

xcpretty
0.2.6

xcpretty-travis-formatter
0.0.4

generated on: 2017-12-01

WARN [2017-12-01 21:22:37.67]: Take notice that this output may contain sensitive information, or simply information that you don't want to make public.
INFO [2017-12-01 21:22:37.67]: Open https://github.com/fastlane/fastlane/issues/new to submit a new issue ✅

</details>

<details><summary>✅ fastlane environment ✅</summary>

### Stack

| Key                         | Value                                       |
| --------------------------- | ------------------------------------------- |
| OS                          | 10.12.6                                     |
| Ruby                        | 2.2.4                                       |
| Bundler?                    | false                                       |
| Git                         | git version 2.13.6 (Apple Git-96)           |
| Installation Source         | ~/.fastlane/bin/bundle/bin/fastlane         |
| Host                        | Mac OS X 10.12.6 (16G29)                    |
| Ruby Lib Dir                | ~/.fastlane/bin/bundle/lib                  |
| OpenSSL Version             | OpenSSL 1.0.2g  1 Mar 2016                  |
| Is contained                | true                                        |
| Is homebrew                 | false                                       |
| Is installed via Fabric.app | false                                       |
| Xcode Path                  | /Applications/Xcode.app/Contents/Developer/ |
| Xcode Version               | 9.1                                         |

### System Locale

| Variable | Value       |   |
| -------- | ----------- | - |
| LANG     | en_US.UTF-8 | ✅ |
| LC_ALL   | en_US.UTF-8 | ✅ |
| LANGUAGE | en_US.UTF-8 | ✅ |

### fastlane files:

<details><summary>`./fastlane/Fastfile`</summary>

```ruby
# Customize this file, documentation can be found here:
# https://docs.fastlane.tools/actions/
# All available actions: https://docs.fastlane.tools/actions
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "2.68.0"

generated_fastfile_id "6f67a1fd-042a-4660-8fe4-363928105358"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."
    cocoapods
    # carthage
  end

  desc "Runs all the tests"
  lane :test do
    run_tests
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
	get_certificates(           # invokes cert
			development: true)
    get_provisioning_profile   # invokes sigh
    build_app(
			workspace: "Washow.xcworkspace",
			scheme: "Washow") # more options available
#upload_to_testflight

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)

    # upload to Beta by Crashlytics
	crashlytics(
    # keys for organization: Jocoos_temp
    api_token: "17814dd81835807f712cbfb0c9cee57818fbba48",
    build_secret: "9fcd12a24f2c4705b0e0a60bcb7449254ed98f7aa56f629d10dde033c9e09e32"
    )
  end

  desc "Deploy a new version to the App Store"
  lane :release do
    # sync_code_signing(type: "appstore")
    capture_screenshots
    build_app(scheme: "washow") # Build your app - more options available
    upload_to_app_store(force: true)
    # frame_screenshots
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://docs.fastlane.tools/advanced/#control-configuration-by-lane-and-by-platform
# All available actions: https://docs.fastlane.tools/actions

# fastlane reports which actions are used. No personal data is recorded.
# Learn more at https://docs.fastlane.tools/#metrics

`./fastlane/Appfile`
app_identifier "com.jocoos.washow.app" # The bundle identifier of your app
apple_id<EMAIL_ADDRESS># Your Apple email address

team_id "5CXMSK555F" # Developer Portal Team ID

# you can even provide different app identifiers, Apple IDs and team names per lane:
# More information: https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.68.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

CFPropertyList
2.3.5

activesupport
4.2.10

addressable
2.5.1

babosa
1.0.2

bigdecimal
1.2.6

bundler
1.14.6

claide
1.0.2

cocoapods
1.3.1

cocoapods-core
1.3.1

cocoapods-deintegrate
1.0.1

cocoapods-downloader
1.1.3

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

cocoapods-trunk
1.3.0

cocoapods-try
1.1.0

colored
1.2

colored2
3.1.2

commander-fastlane
4.4.5

concurrent-ruby
1.0.5

declarative
0.0.10

declarative-option
0.1.0

domain_name
0.5.20170404

dotenv
2.2.0

escape
0.0.4

excon
0.55.0

faraday
0.12.1

faraday-cookie_jar
0.0.6

faraday_middleware
<IP_ADDRESS>

fastimage
2.1.0

fourflusher
2.0.1

fuzzy_match
2.0.4

gh_inspector
1.0.3

google-api-client
0.13.6

googleauth
0.5.1

highline
1.7.8

http-cookie
1.0.3

httpclient
2.8.3

hurley
0.2

i18n
0.9.1

io-console
0.4.3

json
1.8.1

jwt
1.5.6

libxml-ruby
3.0.0

little-plugger
1.1.4

logging
2.2.2

memoist
0.15.0

mime-types
3.1

mime-types-data
3.2016.0521

mini_magick
4.5.1

minitest
5.4.3

molinillo
0.5.7

multi_json
1.12.1

multi_xml
0.6.0

multipart-post
2.0.0

nanaimo
0.2.3

nap
1.1.0

netrc
0.11.0

nokogiri
1.5.6

os
0.9.6

plist
3.2.0

power_assert
0.2.2

psych
2.0.8

public_suffix
2.0.5

rake
10.4.2

rdoc
4.2.0

representable
3.0.4

retriable
2.1.0

rouge
1.11.1

ruby-macho
1.1.0

rubygems-update
2.5.2

rubyzip
1.2.1

security
0.1.3

signet
0.7.3

slack-notifier
1.5.1

sqlite3
1.3.13

terminal-notifier
1.7.1

terminal-table
1.7.3

test-unit
3.0.8

thread_safe
0.3.6

tty-screen
0.6.3

tzinfo
1.2.4

uber
0.0.15

unf
0.1.4

unf_ext
<IP_ADDRESS>

unicode-display_width
1.1.3

word_wrap
1.0.0

xcodeproj
1.5.3

xcpretty
0.2.6

xcpretty-travis-formatter
0.0.4

generated on: 2017-12-01

I found that I wrote username without quotes("") in Matchfile.


[Update Test Case Configuration]: MaxSystemContextConversation_test.py and MaxSizeConversation_test.py

Outdated/Incorrect Configuration Area
MaxSystemContextConversation_test.py and MaxSizeConversation_test.py
Reason for Update
Missing test cases for BaseComponent settings, add_message, and add_messages
Proposed Configuration Update
Create a test case file for this class
Additional Context (optional)
No response

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


linux compatibility and more

A number of tiny fixes to enhance output and a fix to check git on linux

Nice, this should fix isses #5 and #7 !

@naderghanbari Are you able to give feedback on this?
I'm evaluating using the plugin at work but if it's not being actively maintained then I'll have to be willing to commit to maintaining an internal fork, and publish our own private artifacts. I'd rather know that I can contribute back to the upstream and have proposals actively reviewed and integrated.
On that note, does your license note section in the README mean that you license your work on this fork under Apache 2 like the original? If so, it'd be great if you could explicitly state in writing that this fork remains licensed as Apache 2, it's currently ambiguous.

@ches Sorry for the late reply. I will try to maintain this repo.
It will be Apache 2 like the original, I will add a license file.

@andreaTP Thanks for contribution and sorry for the late reply. I assume it's not fair to ask for improvements this late otherwise I would have asked you to submit separate PRs for each concern. The commits are clear and atomic though so I'm gonna merge it.


clarifying name arg determines vim command name

Motivation for this change
The name arg of vim_configurable.customize does not only determine
the package name, but also the name of the command/ executable to be
called.
In my opinion this is not documented properly and finding that out took
me several hours.
Things done

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

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

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

If accepted, this also needs to be backported to release-18.09

backport: dbc3d6fdd9a


Feature request: Enhance OpenAPIResponse with other fields from OpenAPI specification

Use case
I use an API Gateway event handler with validation enabled and I'd like to document various responses my endpoint can return. I can add responses parameter to my endpoint definition, however it's not ideal:

The OpenAPIResponse type doesn't support all the fields that I could use as per the OpenAPI specification for the response object, especially headers to document HTTP headers I could return to the consumer.
The content field of the OpenAPIResponse also doesn't support all the fields available for the media type object, especially example/examples to provide example(s) of the response payload.

As OpenAPIResponse is a TypedDict, I can actually provide a dict with additional fields, but in that case it shows type errors in the IDE (VS Code in my case) which I have to suppress with # type: ignore. Here's an example:
@app.get(
    "/foo",
    responses={
        200: {
            "description": "Successful Response",
            "content": {
                "application/json": {
                    "schema": {"$ref": "#/components/schemas/FooModel"},
                    "examples": {
                        "example1": {...}
                    }  # type: ignore
                }
            }
        }
    }
)
def foo() -> FooModel:
    ...

Also, note that when I want to add examples to the content value, I have to provide schema and rely on the fact that the response model would be added to the OpenAPI schema automatically due to the endpoint function return value annotation. I can't use model because in that case the examples wouldn't be present in the generated OpenAPI schema (see here).
The issue was previously discussed on Discord – https://discord.com/channels/1006478942305263677/1006527338621710376/1266007500336005120.
Solution/User Experience
The proposed user experience can be seen from the following example:
@app.get(
    "/foo",
    responses={
        200: {
            "description": "Successful Response",
            "headers": {...},
            "content": {
                "application/json": {
                    "model": FooModel,
                    "examples": {
                        "example1": {...}
                    }
                }
            }
        }
    }
)
def foo() -> FooModel:
    ...

To summarize:

I could use any field supported by the OpenAPI specification for the response object.
I could use a model field with my response's Pydantic model and it would generate the correct OpenAPI schema while retaining other fields supported by the OpenAPI specification for media type object.

Alternative solutions
No response
Acknowledgment

[X] This feature request meets Powertools for AWS Lambda (Python) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Java, TypeScript, and .NET

Hey @tlinhart! Thanks a lot for opening this issue! I'm adding this to our backlog and expect to work on this early next month.


Insert wait_for_indexing steps.

Can we infer where they need to be added?

Can we get by with fewer?


bearer should be case insensitive

Hi truqu!
We have been using your elm-oauth2 with great success in our project at work. However when upgrading to Elm 0.19 and elm-oauth2 4.0 we ran into some trouble. I have traced the problem to the function OAuth.tokenFromString which creates a token when the case ("Bearer", t) holds. However, our Keycloak IDP is sending AuthenticationSuccess using the smallcap "bearer" instead of initcap "Bearer". This made the token creation fail.
According to spec (https://tools.ietf.org/html/rfc6749#section-4.2.2) token type should be case insensitive. My suggestion for a new tokenFromString function:
tokenFromString : String -> Maybe Token
tokenFromString str =
    case ( String.toLower (String.left 6 str), String.dropLeft 7 str ) of
        ( "bearer", t ) ->
            Just (Bearer t)
        _ ->
            Nothing

Keep up the good work!
Best regards
Vidar Evenrud Seeberg

Ho. That's a mistake indeed, but not in the tokenFromString. This one is intended to be used for parsing an authorization header where the case is sensitive.
Though, the makeToken shouldn't use this and should here be case insensitive:
https://github.com/truqu/elm-oauth2/blob/master/src/OAuth.elm#L95
Happy to take PR, or will do later this week :+1:

Haha. As you prefer, yet, that's how you lear ;) You make a PR, you get constructive feedback, you feel empowered and make someone happy :)

Thank you, @KtorZ , I will still like to wait a bit and leave this to you. I will keep my eye on the fix, though, to (for now) learn the sub optimal way :). Maybe I'm in later...
Really appreciate your work (and all other volunteers producing excellent tools for us consumers)!

See https://package.elm-lang.org/packages/truqu/elm-oauth2/4.0.1
Thanks for reporting :)

Nice, thank you for quick fix :). I will try it out first thing tomorrow!

Works like a charm, @KtorZ :+1: ! Thanks a lot!


When loading skinned primitive from gltf the programm crashes

Bevy version
0.12.1
[Optional] Relevant system information
2023-12-26T11:16:49.945923Z  INFO bevy_winit::system: Creating new window "App" (0v0)
2023-12-26T11:16:50.591403Z  INFO bevy_render::renderer: AdapterInfo { name: "NVIDIA TITAN Xp", vendor: 4318, device: 6914, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "537.58", backend: Vulkan }
2023-12-26T11:16:51.724808Z  INFO bevy_diagnostic::system_information_diagnostics_plugin::internal: SystemInfo { os: "Windows 10 Pro", kernel: "19045", cpu: "AMD Ryzen 7 2700X Eight-Core Processor", core_count: "8", memory: "63.9 GiB" }
2023-12-26T11:16:54.858069Z  INFO bevy_window::system: No windows are open, exiting
2023-12-26T11:16:54.859490Z  INFO bevy_winit::system: Closing window 0v0

What you did
Program crashing when i trying to load primitive with joints and weights attributes.
Im trying to load the PUG gltf file from Kenney's (Can be downloaded from https://drive.google.com/drive/folders/1UNNT0MeVX0O04RGgu8aLkKe3fwB9_t-A )
I want to load Mesh0/Primitive0 with Material0. But on load program crash. If i remove joints and weight from primitive, then program works.
fn setup(
    mut commands: Commands,
    mut asset_server : Res<AssetServer>,) {
    commands.spawn(Camera3dBundle::default());

    let mesh_handle = asset_server.load("Pug.gltf#Mesh0/Primitive0");
    let material_handle = asset_server.load("Pug.gltf#Material0");

    commands.spawn((
        PbrBundle {
            mesh: mesh_handle,
            material: material_handle,
            ..default()
        }
    ));
}

What went wrong
2023-12-26T11:21:06.501637Z  INFO bevy_winit::system: Creating new window "App" (0v0)
2023-12-26T11:21:07.099166Z  INFO bevy_render::renderer: AdapterInfo { name: "NVIDIA TITAN Xp", vendor: 4318, device: 6914, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "537.58", backend: Vulkan }
2023-12-26T11:21:08.271808Z  INFO bevy_diagnostic::system_information_diagnostics_plugin::internal: SystemInfo { os: "Windows 10 Pro", kernel: "19045", cpu: "AMD Ryzen 7 2700X Eight-Core Processor", core_count: "8", memory: "63.9 GiB" }
2023-12-26T11:21:09.345420Z ERROR wgpu::backend::direct: Handling wgpu errors as fatal by default
thread '<unnamed>' panicked at 'wgpu error: Validation Error

Caused by:
    In a RenderPass
      note: encoder = `<CommandBuffer-(0, 1, Vulkan)>`
    In a draw command, indexed:true indirect:false
      note: render pipeline = `pbr_opaque_mesh_pipeline`
    The pipeline layout, associated with the current render pipeline, contains a bind group layout at index 2 which is incompatible with the bind group layout associated with the bind group at
2

', C:\Users\rewin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\wgpu-0.17.1\src\backend\direct.rs:3056:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Encountered a panic in exclusive system `bevy_render::renderer::render_system`!
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: RecvError', C:\Users\rewin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\bevy_render-0.12.1\src\pipelined_rendering.rs:145:45
error: process didn't exit successfully: `target\debug\examples\minamal_gltf_test.exe` (exit code: 101)

Additional information
I tried another random files from Kenney's models. All failed.
Primitive from gltf file
 "meshes" : [
        {
            "name" : "Cube.004",
            "primitives" : [
                {
                    "attributes" : {
                        "POSITION" : 0,
                        "NORMAL" : 1,
                        "COLOR_0" : 2,
                        "JOINTS_0" : 3, <-- if i remove this line, program will not crash
                        "WEIGHTS_0" : 4 <-- if i remove this line, program will not crash
                    },
                    "indices" : 5,
                    "material" : 0
                },

This looks like a dup of #10810

Ye, it's very similar. Just different in materials (one from gltf, one was generated)


Juice themes

idea for 2022 (or the 4.0 port): holiday styles for the editor juice plugin. Christmas-themed, Hanukkah, Halloween and so on.
Can be non-holiday themes as well. Maybe themed around other major Jotson™ projects. Just a nice little dropdown selection that can be extended with more items as time goes on. Maybe a somewhat accessible/documented format for people to submit their themes in on github.

That's a nice idea!


Add "column headers detection function" when column headers row is unknown

Is your feature request related to a problem? Please describe.
When column headers differ and their row differs per document (different "intro" in each document)
I have to parse once to find the header and then do it again to work from that line.
Also "from" vs "from_line" becomes confusing when the header is not the first row
Describe the solution you'd like
I think aside from static row number there should be option to add function that returns true or array of headers (column names) when it detects columns header row, and then we parse from that on with columns ( object fields ) named as headers
Describe alternatives you've considered
I've tried a lot of workarounds. using on_records with some global isHEader boolean and then columns to manually convert to columns. All felt like reinventing what library already does well (when the header is in the same place).
Second workaround was to parse once to locate header, and then start over parsing from that line - that seems to be working but code is much much complicated and I cannot just stream the file and do it in one pass.
Let me know how much of a problem would it be to make header detection and if you are accepting PRs what conditions / requirements for it to be accepted without much back and forth - maybe I could contribute. Whatever would work :-)
Aside from that - great job and awesome library guys! - It helped me a lot with a LOT of huge and nasty csv files. :-)

If I understood well, the column options can already be defined as an array, see those 3 tests. It is also documented. Does this answer your need or did I read your request to fast? Please provide a little wished sample to ease my understanding.

Nope, I know about that one and unfortunately, the columns array doesn't help.
The problem is when header position (its row number) is not known.
There is no way to "detect header" row and THEN return Array (or true for auto) like the option you suggested.

Each document I have has a different start/header row position.
Column labels also changes. Each time I have to find header first and then start parsing again.

Then please illustrate your requirements with a minimalist sample.

First of all, thank you for this awesome library!
We encountered the same issue on our side too.
As @ev45ive described, we need to skip from 1 to 8 rows before reaching the header position.
In addition, the columns order may differ from one file to another: it can be a,b,c, a,b,c,d or a,c,b
Currently the only solution we could find was the same as described above:

parse the file once to find the header position
and parse it again starting from the header position found previously.

Because we are managing large files, we must ensure that only a small section of the file is computed during the first operation (with from_line, to_line). And of course, if the header could not be found the whole process needs to be aborted.
Here's a small illustration of the files we may have to deal with for the same parser:
line to skip 1
a,b,c
1,2,3
5,6,7

line to skip 1
line to skip 2
a,c,b
1,3,2
5,7,6

line to skip 1
line to skip 2
a,b,c,d
1,2,3,4
5,6,7,8


Slow reload time of SpeckleProperties

Right now, SpeckleProperties component serializes a lot of data, using our custom serializer which was optimised for serializing lots of objects in bulk, not small single objects.
We should scope some alternatives that provide equivalent DX, but are more optimal for serialization and memory performance.

see https://github.com/specklesystems/speckle-sharp/pull/2967

Closing this ticket as we are moving to Jira to reduce noise. You can access it there.


[Question] Is there any Error Event for Pooling Connection?

Is there any error event to catch fatal errors for pooling connections like there is one for the connection object.
I tried using
var pool  = mysql.createPool({
  connectionLimit : 10,
     host            : mysqlHost,
     user            : mysqlUser,
     password        : mySqlPassword,
     database        : mysqlDb
});

pool.on('error', function(err) {
     console.log(err);
});
pool.query('USE name_of_db_that_does_not_exist');

but it doesn't work.

Hi @raghav196 this is how the pool is designed, which follows what happens when you make a query without a callback that is an error without the pool. If you want to actually get the error, you need to either (a) provide a callback to the query or attach an error event listener to the Query object returned by pool.query method.
I hope this helps!


Implement Hive QL Parsing

This is a highly experimental start to implementing Hive QL in the parser. Luckily, the parser's design lends itself well to adding in some of these things, but there are still a few oddities and things to implement to make something for Hive complete. I'd like to get people reviewing this early because in my opinion I do some janky things here I should clean up and I'm hoping early review will help me nip that in the bud.
Also, my tests are a bit on the light side right now, but I have verified a lot of this functionality with some internal HQL that I cannot share publicly. I am confident that this covers the majority of edge cases on how not to write HQL.

Pull Request Test Coverage Report for Build 181646624

286 of 418 (68.42%)  changed or added relevant lines in 10 files are covered.
6 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-1.9%) to 89.699%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tests/sqlparser_hive.rs
31
33
93.94%

src/tokenizer.rs
9
12
75.0%

src/parser.rs
150
212
70.75%

src/ast/mod.rs
73
138
52.9%

Files with Coverage Reduction
New Missed Lines
%

src/ast/data_type.rs
1
75.0%

src/ast/query.rs
1
89.88%

src/parser.rs
1
86.57%

src/ast/mod.rs
3
73.88%

Totals

Change from base Build 175941714:
-1.9%

Covered Lines:
4415

Relevant Lines:
4922

💛  - Coveralls

Looks great, would love to have this! I added a few comments

So @Dandandan I ran into a discussion point here. Hive allows you to have column aliases beginning with numbers such as SELECT a as 3b from table with no quotes or anything of the sort around it. This basically breaks most ident parsing, which led me to disable the hive dialect in the ansi tests for now. While I don't believe hive has ever conformed to anything close to ansi sql there might be a better way to do this or support numbers as beginning identifiers.

@hntd187 did you try adding numbers to
is_identifier_start for this dialect? I am not sure how that combines with other parts of the tokenizer, but if this causes any issue, I think the tokenizer could be adapted for this case.

Yes I did @Dandandan it basically breaks 30 tests in the ansi dialect comparison leading me to remove that change for now. I think Hive should parse the standard ANSI statements the exact same way and do it's own thing for the weird off standard syntax, but it seems like no dialect supports numbers first.

I think the tokenizer / dialect trait needs to change a bit for numbers in the start position.
If the start character is a number, there needs to be a alphabetical character as well (otherwise it would be a number).
E.g. changing this in the tokenizer (yes,code is a bit messy, but this can be cleaned up / made easier / redesign a bit) succeeds all the test after allowing to start on 0-9.
                ch if self.dialect.is_identifier_start(ch) => {
                    chars.next(); // consume the first char
                    let s = self.tokenize_word(ch, chars);

                    if s.chars().all(|x| x >= '0' && x <= '9' || x == '.') {
                        let mut s = peeking_take_while(&mut s.chars().peekable(), |ch| match ch {
                            '0'..='9' | '.' => true,
                            _ => false,
                        });
                        let s2 = peeking_take_while(chars, |ch| match ch {
                            '0'..='9' | '.' => true,
                            _ => false,
                        });
                        s += s2.as_str();
                        return Ok(Some(Token::Number(s)));
                    }
                    Ok(Some(Token::make_word(&s, None)))
                }```

no dialect supports numbers first.

MySQL does, which leads them to make recommendations like "avoid using 1e as an identifier, because an expression such as 1e+3 is ambiguous. Depending on context, it might be interpreted as the expression 1e + 3 or as the number 1e+3.", which makes me want to start with documenting the expected behavior before discussing the implementation. I would suggest separating this from the rest of Hive support.

I think that's reasonable, but idents starting with numerics is part of hive support, but since it's not hive specific should happen at the tokenizer level like Dan suggested, I suppose the MySQL Dialect would need something similar in it to what hive has.

@hntd187 @Dandandan Seems like we should go ahead and merge this now? I will skim through and merge if there are no objections.

@hntd187 @andygrove
To me it looks great as well, added some final comments, could also be done in a follow-up.
Thanks @hntd187, for the great addition and for your patience 😎


Colors js 1.4.1 has been compromised. Immediate fix is needed.

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
Latest, problem a lot of earlier versions too
Electron version
13.1.1
Operating system
All
Last known working Electron Forge version
Colors-js 1.4.0
Expected behavior
When you run an electron forge command the expected output should be produced.

Steps to reproduce
Run any electron forge command.
Additional information
A simple fix is needed, change colors.js to v1.4.0, I will work on a pr now. I know I didn't fill this out all correctly, but this is a total blocker for any new development.

A workaround is to pin colors in your package.json file.

NPM
Yarn

The correct change is to replace colors with chalk. I'm not particularly interested in a PR which will pin colors in Forge's package.json files.

Okay yes, I won't do that PR then.


Some perf improvements in Azure.Core and Storage

Headers are accessed in a faster way via HttpHeaders.NonValidated, which is available in net6.0+
HMACSHA256 instance is cached

Expected benefit should be about 15%

API change check
API changes are not detected in this pull request.


datasketches does not work with python 3.9

Datasketches module <IP_ADDRESS> seems to have been compiled with python 3.10 and does not work with python 3.9

root@907f428e7dc2:/temp# pip install datasketches --target .
Collecting datasketches
Downloading datasketches-<IP_ADDRESS>-cp39-cp39-manylinux_2_12_x86_64.manylinux2010_x86_64.whl (474 kB)
|████████████████████████████████| 474 kB 658 kB/s
Collecting numpy
Downloading numpy-1.22.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (16.8 MB)
|████████████████████████████████| 16.8 MB 7.3 MB/s
Installing collected packages: numpy, datasketches
Successfully installed datasketches-<IP_ADDRESS> numpy-1.22.2
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
WARNING: You are using pip version 21.2.4; however, version 22.0.3 is available.
You should consider upgrading via the '/usr/local/bin/python -m pip install --upgrade pip' command.

root@907f428e7dc2:/temp# ls -lrt
total 1056
drwxr-xr-x  2 root root    4096 Feb 12 06:16 numpy.libs
drwxr-xr-x 19 root root    4096 Feb 12 06:16 numpy
drwxr-xr-x  2 root root    4096 Feb 12 06:16 bin
drwxr-xr-x  2 root root    4096 Feb 12 06:16 numpy-1.22.2.dist-info
-rwxr-xr-x  1 root root 1049008 Feb 12 06:16 datasketches.cpython-310-x86_64-linux-gnu.so
drwxr-xr-x  3 root root    4096 Feb 12 06:16 tests
drwxr-xr-x  3 root root    4096 Feb 12 06:16 src
drwxr-xr-x  2 root root    4096 Feb 12 06:16 datasketches-<IP_ADDRESS>.dist-info

root@907f428e7dc2:/temp# python
Python 3.9.10 (main, Jan 26 2022, 20:56:53)
[GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.

import datasketches
Traceback (most recent call last):
File "", line 1, in
ModuleNotFoundError: No module named 'datasketches'

As per the below link 3.9 is supported
https://www.piwheels.org/project/datasketches/
But the compiled library is that of 3.10  : datasketches.cpython-310-x86_64-linux-gnu.so

root@907f428e7dc2:/temp# ln -s datasketches.cpython-310-x86_64-linux-gnu.so datasketches.so

root@907f428e7dc2:/temp# python
Python 3.9.10 (main, Jan 26 2022, 20:56:53)
[GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.

import datasketches
Traceback (most recent call last):
File "", line 1, in
ImportError: Python version mismatch: module was compiled for Python 3.10, but the interpreter version is incompatible: 3.9.10 (main, Jan 26 2022, 20:56:53)
[GCC 10.2.1 20210110].

Thanks @jmalkin. It works on my mac too. I hit this issue when I deployed my application using AWS lambda. It currently support only python 3.9. And this issue is seen even when I use python's 3.9 linux container. Have not tried this on windows.

@MaheshGPai Which specific python 3.9 container were you using for this? They seem to have lots of variants.
I haven't done much with python containers, so if you can share a minimal Dockerfile that'd be appreciated. I was clearly running into path issues when experimenting yesterday.

https://hub.docker.com/layers/python/library/python/3.9.10/images/sha256-6f468ed026185ad7f1a13d1dde8f307449fce3777695bfacf91da8f7987b5fd5?context=explore

I got this working by building datasketches using python 3.9.10.

Hello,
I run against this issue on another version, after looking into, its looks like the current package on pypi (<IP_ADDRESS>) is incorrect, the .so file in all wheel is build for python 3.10 (datasketches.cpython-310-x86_64-linux-gnu.so), so it doesn't load on other versions than 3.10.

Sorry, I haven't had a chance to get to this. Still quite backlogged.
The underlying issue is that cibuildwheel is only generating with 3.10, even when the naming suggests it should be otherwise. But I have no clue why this is happening. Presumably there's something incorrectly configured in the github action that generates the set of wheels.

Trying to cycle back to this finally. My best guess right now is that cmake is automatically finding a different version of python from what cibuildwheel expects. But that's pure speculation -- the only thing I can really confirm is that a newer version of cibuildwheel didn't fix it.

Ok, way too slow addressing this, but I think the artifact.zip at https://github.com/apache/datasketches-cpp/actions/runs/2261547153 (the link is at the bottom) should work. If anyone is still following this and able to try it, please let me know.

This should be resolved if you build yourself. For a pre-built artifact, we need to wait for the next release.

3.4.0 has been released on pypi which should finally close this out.


O(n) behavior for large allocations

This re-released version of tcmalloc is missing the fix that's available in gperftools: https://github.com/gperftools/gperftools/commit/06c9414ec423ffe442c047b2560555f9d5847b1d
The O(n) search over large spans becomes very expensive for any long-running application with >1MB allocations, since over time the large span list can accumulate thousands of entries due to fragmentation.

TCMalloc has mostly moved to an alternative page heap implementation (Huge Page Aware Allocator), which doesn't use the same span list design.


mention osx install step "brew install libidn"

2nd time I needed to google around for this ... how about some readme instructions on how to install libidn ? :)

thank you.

extconf failed, exit code 1

Gem files will remain installed in /Users/jpope/.rvm/gems/ruby-2.5.0/gems/idn-ruby-0.1.0 for inspection.
Results logged to /Users/jpope/.rvm/gems/ruby-2.5.0/extensions/x86_64-darwin-18/2.5.0/idn-ruby-0.1.0/gem_make.out

An error occurred while installing idn-ruby (0.1.0), and Bundler cannot continue.
Make sure that `gem install idn-ruby -v '0.1.0' --source 'https://rubygems.org/'` succeeds before bundling

 brew install libidn
Updating Homebrew...
==> Auto-updated Homebrew!
Updated 1 tap (homebrew/core).
==> New Formulae
azure-storage-cpp                                                mesa
==> Updated Formulae
buku                                       git-cinnabar                               jhipster

==> Downloading https://homebrew.bintray.com/bottles/libidn-1.35.mojave.bottle.tar.gz
######################################################################## 100.0%
==> Pouring libidn-1.35.mojave.bottle.tar.gz
==> Caveats
Emacs Lisp files have been installed to:
  /usr/local/share/emacs/site-lisp/libidn
==> Summary

 gem install idn-ruby -v '0.1.0'
Building native extensions. This could take a while...
Successfully installed idn-ruby-0.1.0
Parsing documentation for idn-ruby-0.1.0
Installing ri documentation for idn-ruby-0.1.0
Done installing documentation for idn-ruby after 0 seconds
1 gem installed

fixed


This happen

Can you give more information? Was autocomplete involved? Was it this issue? https://github.com/atom/autocomplete-plus/issues/511

I don't know, i was coding and then, pum! the whole thing was just like the pic, restart de atom fixed
Date: Thu, 9 Jul 2015 14:02:56 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [atom] This happen (#7820)
Can you give more information? Was autocomplete involved? Was it this issue? atom/autocomplete-plus#511
—
Reply to this email directly or view it on GitHub.

I'm closing as a duplicate of atom/autocomplete-plus#511 It should be fixed in the next atom release

ThanksDate: Thu, 9 Jul 2015 19:42:25 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [atom] Editor area is resized incorrectly relative to the window size (#7820)
Closed #7820.
—
Reply to this email directly or view it on GitHub.


Choice of encoding?

The documentation says that the lib opens files in 'w' mode (which is text). I don't see any mention of the encoding, though. Not only is there no explicit default, there isn't even a documented way to choose an encoding. After a long look I finally found a trail showing that one can, in fact, pass the encoding parameter: the passthrough atomic_write(**cls_kwargs) → AtomicWriter(**open_kwargs) (the latter never mentioned).
The problem with that? Well, most people will probably assume it's UTF-8, but instead it seems to be getdefaultencoding (not even sure, it's hard to find). It seems like that will help to perpetuate the problems described at https://www.python.org/dev/peps/pep-0597/#using-the-default-encoding-is-a-common-mistake
So there are 2 (3?) issues I see:

The default encoding is implicit.

It's not documented that it's implicit.

A way to choose an encoding is not shown.

I added documentation for **open_kwargs to fix issue 3 but the implicit encoding part is done consistently with the stdlib.
This library is concerned with writing files atomically, not with providing a universally better user interface over file writing. As such it chooses consistency with open() over better defaults.


RBAC section - include link to RBAC documentation

RBAC section - include link to RBAC documentation
role-based access control

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cloud-adoption-guide/subscription-governance.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/cloud-adoption-guide/subscription-governance.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


OpenAPI validator plugin doesn't support for tags and operationId filters

Description:
@openapi:ServiceInfo {
    contract: "openapiwetherMap.yaml",
    excludeTags: ["Weather Forecast", "Current01"]
    operations: ["getWeatherForecast"]
}

Annotation didn't support filtering operation and tags.
Steps to reproduce:
Affected Versions:
Ballerina 2201.0.0 (Swan Lake) Snapshot 1
OS, DB, other environment details and versions:

This issue occurs due to the annotation field value return with double-quotes through the api.


feat: field-level access control

Implementing field-level access control: for "read" and "update" only, with field-level attributes @allow and @deny.
For "read": when a field has policy rules, it'll be omitted from the read if any deny rule evaluates to "true" or none of the allow rule evaluates to "true".
For "update": when a field has policy rules, "update" will be rejected or filtered if any deny rule evaluates to "true" or none of the allow rule evaluates to "true".


Fix URL in email ("." should be "/").  Fixes #1096

Fix the URL in some emails ("." should be "/").
The problem is that we used the wrong helper; we used
"projects_url" (plural name) when we should have used
"project_url" (singular name).  We're referring to a
singular project, so we must use the singular helper.
Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

LGTM


Implement the UnwindSafe/RefUnwindSafe traits on channels

To my surprise, this trait isn't implemented on channels, even though channels are kind of a critical component to unwind safety.
See the code comment for an explanation of the reasoning.
I'm not 100% confident in what I'm doing, so feel free to summon unwind-safety experts, if there is any.

Thanks for the PR. I think we should make AtomicWaker UnwindSafe (#2211) first.


Analyzing and logging Moya Errors. Response & request correlation.

Originally Alamofire provides both response and request as a part of Response structure. It helps with debugging and logging of incidents. Current implementation of Moya.Response has only response NSURLResponse (instead of NSHTTPURLResponse in Alamofire) and no request information at all. With such approach when error happened there is no way to find for which request it happened. There is also no way to find header fields from NSURLResponse. With all being said what would be a recommended way of analyzing and logging Moya Errors?

To debug using Moya, I generally take advantage of the NetworkLoggerPlugin. See the Providers documentation.

Yeah, basically you pass the plugins in init of your provider, the same as closures, manager etc. For example something like:
MoyaProvider<GitHub>(plugins: [NetworkLoggerPlugin()])

NetworkLoggerPlugin  does not have correlation capabilities. It seems like logging strategy now "all or nothing" while I have a particular need in mind which I believe is common. When error happened I would like to log both response with an error and request that caused it. Does it look like a valid problem?

I've written a whitelist/blacklist logger plugin. It's not elegant (which is why it's not in Moya 😛 ) but it works: https://github.com/artsy/eidolon/blob/master/Kiosk/App/Networking/NetworkLogger.swift

Would it be a natural thing to return back correlation of request & response into Moya result? Why in Miya.swift on line 207 of Alamofire response, NSURLRequest just discarded, while the rest is propagated as result? It is common feature of networking libraries. It was in Alamofire, MKNetworkingKit etc as well as for libraries on other platforms. As I am not really familiar with details of some architecture decisions It would be interesting to understand what was a decision process for removing this feature from Moya.

I can think of a few ways to accomplish what you're after.

Hard: Fork Moya and add NSURLRequest to the completion.  Then you'd have to maintain a fork, which is no fun.
Reasonable: Fork Moya and add a plugin method to line 210 plugins.forEach { $0.didReceiveRequest(request, result: result, target: target) }.  Again a fork, but such a subtle difference that this would probably be easier to maintain.  And the debugging would be off on its own, in a plugin, which is nice.
Easy: Just re-create the request in your response handler!  There's nothing special about the NSURLRequest instance, and it's easy to create a copy.

provider.request(.Zen) { result in
  let request = Api.Zen.urlRequest
  ...

Better yet would be to wrap this in a networking layer, and then you can send the request back in a custom completion block:
class Network {
  static let provider = MoyaProvider<MyService>()

  static func request(endpoint: MyService, completion: (NSURLRequest, Result) -> Void) {
    provider.request(endpoint) { result in completion(endpoint.urlRequest, result) }
  }
}

Yeah that would be awesome to have request printed such as:
curl -X GET blablabla

You can now get curl commands logged using the included network activity plugin.

@colinta recreating URL requests is not the same as getting a reference to the one that were sent. There could be various random identifiers attached etc, so the question to trace an exact request sent. Somehow "hard way" looks most logical one, which is very unfortunate. Thanks.

@nikita-leonov what do you think, has this issue been addressed to your satisfaction? If not, what are the next steps?

@ashfurrow as for today we do not have a good solution. The only elegant solution is option 1 suggested by @colinta, but maintaining an own fork of Moya at the current stage of our project is killing. Having correlation logging in the plugin does not work as well since such option missing a context of the call and many other benefits of having request together in response in a place of request initialization.
Recreating a call when needed does not work, since as I mentioned it missing parameters uniquely generated for the response.
Do you think re-introducing request & response correlation, that was initially available in AFNetworking would be accepted as PR into Moya? I would be happy to work on it.

I'd be happy to review a PR, I'm not sure what request/response correlation would look like so I can't say for sure if it'd be accepted. Could you explain your implementation idea?

@ashfurrow I provided PR. Understand a point about "no guarantees on accepting it", no worries. I am happy to discuss and provide changes if you think something is not in line with grand-plan of Moya.

PR is merged! Are there any next steps on this issue?

Closing this for now, as it seems this has been resolved by merging the PR. Please reopen if there's anything else that needs discussing! :)

Actually merging PR is not enough to get request information when "request" method is failed.
Alamofire has custom struct that has request property, in this way we can grab request information in response closure like below
Problem
MoyaProvider<NetworkAPI>().request(.getList) { (result) in
            switch result {
                case .success(let response)
                break
                case .failure(let moyaError)
                // We can not get any request information here
                // Because Response that has request information object not available here
            }
        }

Alamofire Able To Do This

        Alamofire.request(url,
                          method: method,
                          parameters: parameters,
                          encoding: encoding,
                          headers: headers)
            .responseJSON { (dataResponse) in

                if let request = dataResponse.request {
                    ...
                }
}

Alamofire Data Response Struct
public struct DataResponse<Value> {
    /// The URL request sent to the server.
    public let request: URLRequest?
...
}

Moya Compilation
public typealias Completion = (_ result: Result<Moya.Response, MoyaError>) -> Void

My Offers
1- Add request property to compilation as a new parameter,
2- Create wrapper struct for Result<Moya.Response, MoyaError> like Alamofire did.

public struct MoyaResponse<Value, Error> {
    /// The URL request sent to the server.
    public let request: URLRequest?

    /// The result of response serialization.
    public let result: Result<Value, Error>

    /// Returns the associated value of the result if it is a success, `nil` otherwise.
    public var value: Value? { return result.value }

    /// Returns the associated error value if the result if it is a failure, `nil` otherwise.
    public var error: Error? { return result.error }

    public init(request: URLRequest?,
                result: Result<Value, MoyaError>) {
        self.request = request
        self.result = result
}


'envoy --version' should provide actual version, ex: 1.5.0

Title: 'envoy --version' should provide actual version, ex: 1.5.0
Description:
currently 'envoy --version' provides you with an SHA which is not much useful for end users. I would expect to see a version such as "1.5.0" there.
Something like:
envoy --version
version: 1.5.0
tag: a8507f67225cdd912712971bf72d41f219eb74ed/Modified/RELEASE

I'm working on this. I'm thinking just simply put VERSION file of which content is 1.6.0.dev, then read that in build phase. WDYT?
The first try is using bazel/get_workspace_status but it seems bazel only supoprt specific keys.
So next try will be using genrule. Please tell me any advices if you have.

Honestly I would probably just hard code it in the code and we can bump it in each release:
https://github.com/envoyproxy/envoy/blob/master/source/common/common/version.h
You could add a new member there called release_version or something like that, or potentially rename some of those variables to be more clear.


[Enhancement] nvm segment now shows version when 'nvm current' is 'system'

Description
This improves the nvm segment, and has it render the version of system node when nvm current outputs system. It also prevents truncating the first character off system

Questions
I haven't made a test for this enhancement, as I can't find a reliable way to fake the version of system.
I don't think I've put my patch on the right branch... which branch should have I branched from? I think I branched from next ... :/

Hi @Glutnix ,
Thanks for your PR. 👍
To answer your questions:
You are already on next, but you need to update your PR. I did some changes/merged some features since you created your fork.
To create a mock that depends on a given argument, have a look at the go tests.

Hi @dritter, thanks for the friendly welcome, and the very useful advice on the tests.
I've improved my patch on this PR on the next branch, along with adding a test, which is passing!

Thx @Glutnix and welcome to P9K!

🙌
Thanks for accepting my PR :)


[SWF] change dat extension to txt because of possible forbidden MIME type

I have several projects for big accounts that can't work with .dat file because of forbidden MIME type. But it works with .txt files.

Do you know if it works as *.bin instead? I understand that adding new MIME types to web servers in order to use OpenFL is not ideal, but I want to leave it open in the future for us to use a binary format. I wonder if this is supported widely?

Hello.
It seems that I have at least one customer that forbids .dat and allows .bin. For the others, I don't know, I'm waiting for their complete list of mime type. If we use .bin, it could be great to have a fallback to .txt because the txt format is widely supported I think.

i have no opinion about what the file extension is, as long as it doesn't change frequently. we have a lot of assets that would have to be recompiled with this type of change.
but i am curious about the details surrounding the experience you're reporting. what are we trying to work around--a very restrictive http proxy? i am imagining like a library system that is trying to protect its users from malicious formats. but then i wonder, what are they allowed to do with such a restrictive ruleset? and what about files without extensions? and have you considered overriding the http response header Content-Type:? i can't think of any browser that would hold the url's file extension as a higher precedent than that. or are we talking static targets--is it an anti-virus system that is suspicious of .dat format?
more detail please.

The issue is restricted MIME type on web server. We can't do anything against that (customer will not change anything for security reason or whatever) except trying to use different file format when problem occurs. So basically when problem occurs the server don't send the file (error 404 I think).
But I agree, maybe we should use .dat by default to not break old projects and use conditional compilation to have a different format when needed.


Consider removing unused constant

https://github.com/MLH-Fellowship/0.4.1-Execute.ly-frontend/blob/dd5ec0204fb51ec40dc72c2f2600abe21f1c26a1/src/utils/constants.js#L3
Since now the plan for the python model has dropped, I think we can remove this

We are using actually planning to this for automatic language detection as per our latest discussion.


support variable property names

Fixes #97. Allows the use of variable in declaration property names.
Example:
$name: custom-property;
$value: black;

.class {
    --$(name): $value;
    color: var(--$(name))
}

results in
.class {
    --custom-property: black;
    color: var(--custom-property)
}

Wow, this looks great. Thanks, @benmusson! Let’s cut a release soon. I wish my PRs automatically ran the tests. Maybe I can ask a robot to write the actions yml file for that. 😂

I wish my PRs automatically ran the tests

https://github.com/csstools/postcss-advanced-variables/pull/101


Update README.md

Adding link to the AI Speech Coach - BeyondX

Hey, I think it's not relate this repo, submit it to Awesome AI Tools repo


Add my portfolio and updated theme in User Guide

Coverage remained the same at 90.497% when pulling 2c216da3a497cf0974697b1a638919a6909d56eb on chuaweiwen:add_my_portfolio into 93245c873ca49b789cb8f9d90cc392a7c2b986b8 on CS2103AUG2017-W14-B1:master.


async context manager and task queue

uses an async context manager for context generation and implements the task queue for sending messages.  doesn't currently contain the features involving replying to messages, but provides an easier starting point to expand this way in future

Looking good :D


following your instructions for collecting my own object detection models doesn't work

2022-02-07 08:30:40 - Using CUDA...
2022-02-07 08:30:40 - Namespace(balance_data=False, base_net=None, base_net_lr=0.001, batch_size=4, checkpoint_folder='models/Cosmetics', dataset_type='voc', datasets=['data/Cosmetics'], debug_steps=10, extra_layers_lr=None, freeze_base_net=False, freeze_net=False, gamma=0.1, lr=0.01, mb2_width_mult=1.0, milestones='80,100', momentum=0.9, net='mb1-ssd', num_epochs=100, num_workers=2, pretrained_ssd='models/mobilenet-v1-ssd-mp-0_675.pth', resume=None, scheduler='cosine', t_max=100, use_cuda=True, validation_epochs=1, weight_decay=0.0005)
2022-02-07 08:30:40 - Prepare training datasets.
Traceback (most recent call last):
File "train_ssd.py", line 214, in
target_transform=target_transform)
File "/jetson-inference/python/training/detection/ssd/vision/datasets/voc_dataset.py", line 36, in init
self.ids = self._read_image_ids(image_sets_file)
File "/jetson-inference/python/training/detection/ssd/vision/datasets/voc_dataset.py", line 111, in _read_image_ids
if self._get_num_annotations(image_id) > 0:
File "/jetson-inference/python/training/detection/ssd/vision/datasets/voc_dataset.py", line 123, in _get_num_annotations
objects = ET.parse(annotation_file).findall("object")
File "/usr/lib/python3.6/xml/etree/ElementTree.py", line 1196, in parse
tree.parse(source, parser)
File "/usr/lib/python3.6/xml/etree/ElementTree.py", line 586, in parse
source = open(source, "rb")
ValueError: embedded null byte
Why am I getting these errors, I spent hours taking pictures for horrible results
The only difference I did was not to press the merge button I wanted to learn the regular way
Please help me ASAP I have spent countless hours trying to figure out what the problem is and there is no information on github or nvidia to help me
labels.txt

2022-02-07 08:30:40 - Using CUDA...
2022-02-07 08:30:40 - Namespace(balance_data=False, base_net=None, base_net_lr=0.001, batch_size=4, checkpoint_folder='models/Cosmetics', dataset_type='voc', datasets=['data/Cosmetics'], debug_steps=10, extra_layers_lr=None, freeze_base_net=False, freeze_net=False, gamma=0.1, lr=0.01, mb2_width_mult=1.0, milestones='80,100', momentum=0.9, net='mb1-ssd', num_epochs=100, num_workers=2, pretrained_ssd='models/mobilenet-v1-ssd-mp-0_675.pth', resume=None, scheduler='cosine', t_max=100, use_cuda=True, validation_epochs=1, weight_decay=0.0005)
2022-02-07 08:30:40 - Prepare training datasets.
Traceback (most recent call last):
File "train_ssd.py", line 214, in
target_transform=target_transform)
File "/jetson-inference/python/training/detection/ssd/vision/datasets/voc_dataset.py", line 36, in init
self.ids = self._read_image_ids(image_sets_file)
File "/jetson-inference/python/training/detection/ssd/vision/datasets/voc_dataset.py", line 111, in _read_image_ids
if self._get_num_annotations(image_id) > 0:
File "/jetson-inference/python/training/detection/ssd/vision/datasets/voc_dataset.py", line 123, in _get_num_annotations
objects = ET.parse(annotation_file).findall("object")
File "/usr/lib/python3.6/xml/etree/ElementTree.py", line 1196, in parse
tree.parse(source, parser)
File "/usr/lib/python3.6/xml/etree/ElementTree.py", line 586, in parse
source = open(source, "rb")
ValueError: embedded null byte
Why am I getting these errors, I spent hours taking pictures for horrible results
The only difference I did was not to press the merge button I wanted to learn the regular way
Please help me ASAP I have spent countless hours trying to figure out what the problem is and there is no information on github or nvidia to help me
labels.txt

I highly respect you and I know you will get me out of this with the help I need

You should go to https://github.com/qfgaohao/pytorch-ssd which is where the training code is branched from and see if someone had a similar error

thank you very much I hope dusty gets back to me soon please and thank you

thank you sir you are an admirable worker

thank you very much you truly are the best teacher I ever had

No problem @Gianni2045, happy to help :)
Hope you get it working, if not let me know.

please forgive my ignorance i try my best not to be but I have been trying to send you my information attaching my files here it wont accept that type of file, how do I please send you the data in google drive?

20220208-022651.zip

I think I figured it out how to share my data thank you so much for your care as a teacher

hope i will understand your feedback on this shared data

I sent you the data please with all respect lete know what information of problems you found in my data Dusty

Hi @Gianni2045 @GianniM2045, I'm having trouble downloading all the individual folders you sent me on Google Drive.  Can you compress your entire dataset into a single .zip or .tar.gz archive and send me that on google drive instead?  Thanks.

Yes I will try
On Mon, Feb 14, 2022, 12:05 PM Dustin Franklin @.***>
wrote:

Hi @Gianni2045 https://github.com/Gianni2045 @GianniM2045
https://github.com/GianniM2045, I'm having trouble downloading all the
individual folders you sent me on Google Drive. Can you compress your
entire dataset into a single .zip or .tar.gz archive and send me that on
google drive instead? Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/1353#issuecomment-1039390789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AORU47FX4JFWFG43JKGX5M3U3E75BANCNFSM5NWYQXUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

20220208-022651.zip

20220208-022651.zip
https://drive.google.com/file/d/1QaMNDQ-UAQogwbqAomApEzW1D6OlN2V_/view?usp=drive_web
Message ID: @.***>

https://drive.google.com/file/d/1QaMNDQ-UAQogwbqAomApEzW1D6OlN2V_/view?usp=drive_web

Hi @GianniM2045, is this the link?  I've requested access to download it.

suddes.zip
https://drive.google.com/file/d/1tgTWfsq-xUyL2iNOywVysq4vb2Et66o9/view?usp=drive_web
El lun, 14 feb 2022 a la(s) 15:07, Jorge Morales @.***)
escribió:

Yes I will try
On Mon, Feb 14, 2022, 12:05 PM Dustin Franklin @.***>
wrote:

Hi @Gianni2045 https://github.com/Gianni2045 @GianniM2045
https://github.com/GianniM2045, I'm having trouble downloading all the
individual folders you sent me on Google Drive. Can you compress your
entire dataset into a single .zip or .tar.gz archive and send me that on
google drive instead? Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/1353#issuecomment-1039390789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AORU47FX4JFWFG43JKGX5M3U3E75BANCNFSM5NWYQXUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

yes this is the link and this one also
El mar, 15 feb 2022 a la(s) 10:25, Jorge Morales @.***)
escribió:

suddes.zip
https://drive.google.com/file/d/1tgTWfsq-xUyL2iNOywVysq4vb2Et66o9/view?usp=drive_web
El lun, 14 feb 2022 a la(s) 15:07, Jorge Morales (
@.***) escribió:

Yes I will try
On Mon, Feb 14, 2022, 12:05 PM Dustin Franklin @.***>
wrote:

Hi @Gianni2045 https://github.com/Gianni2045 @GianniM2045
https://github.com/GianniM2045, I'm having trouble downloading all
the individual folders you sent me on Google Drive. Can you compress your
entire dataset into a single .zip or .tar.gz archive and send me that on
google drive instead? Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/1353#issuecomment-1039390789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AORU47FX4JFWFG43JKGX5M3U3E75BANCNFSM5NWYQXUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

thank you very much
El mar, 15 feb 2022 a la(s) 10:27, Jorge Morales @.***)
escribió:

yes this is the link and this one also
El mar, 15 feb 2022 a la(s) 10:25, Jorge Morales (
@.***) escribió:

suddes.zip
https://drive.google.com/file/d/1tgTWfsq-xUyL2iNOywVysq4vb2Et66o9/view?usp=drive_web
El lun, 14 feb 2022 a la(s) 15:07, Jorge Morales (
@.***) escribió:

Yes I will try
On Mon, Feb 14, 2022, 12:05 PM Dustin Franklin @.***>
wrote:

Hi @Gianni2045 https://github.com/Gianni2045 @GianniM2045
https://github.com/GianniM2045, I'm having trouble downloading all
the individual folders you sent me on Google Drive. Can you compress your
entire dataset into a single .zip or .tar.gz archive and send me that on
google drive instead? Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/1353#issuecomment-1039390789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AORU47FX4JFWFG43JKGX5M3U3E75BANCNFSM5NWYQXUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

I accepted your request thank you so much for your help
On Tue, Feb 15, 2022, 10:36 AM Dustin Franklin @.***>
wrote:

suddes.zip
https://drive.google.com/file/d/1tgTWfsq-xUyL2iNOywVysq4vb2Et66o9/view?usp=drive_web
OK, I requested access to this one also
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/1353#issuecomment-1040495355,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATVZXRLJGYEZ7NSLO4EHORLU3J6HJANCNFSM5NWYQXUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you so much for completing my submission
On Thu, Apr 27, 2023, 11:56 AM Dustin Franklin @.***>
wrote:

Closed #1353 https://github.com/dusty-nv/jetson-inference/issues/1353
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/1353#event-9116898688,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AORU47BWKD2LM5BCPJV6YUTXDKQKXANCNFSM5NWYQXUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hope you are doing well dusty, Jesus bless you always.
On Thu, Apr 27, 2023, 12:40 PM Jürgen Morales Rizzo Vagner <
@.***> wrote:

Thank you so much for completing my submission
On Thu, Apr 27, 2023, 11:56 AM Dustin Franklin @.***>
wrote:

Closed #1353 https://github.com/dusty-nv/jetson-inference/issues/1353
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/1353#event-9116898688,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AORU47BWKD2LM5BCPJV6YUTXDKQKXANCNFSM5NWYQXUA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Referencing width on SVGElement when expected type is actually SVGSVGElement

Good day!
It looks like in zoom.js the type check at the head of the function is for an SVGElement, but it ought to be the SVGSVGElement.
https://github.com/d3/d3-zoom/blob/main/src/zoom.js#L17-L28
Per the definition on the MDN this results in an opportunity for misuse to result in an element being passed in that is not the outermost SVG element.
https://developer.mozilla.org/en-US/docs/Web/API/SVGSVGElement
https://developer.mozilla.org/en-US/docs/Web/API/SVGElement
Also noticed that the repo has not had its open PRs merged in in some time.
If the there is a maintainer around and interest I can get a PR in for review later in the week. Looks like it should be a really quick changeset.

It seems that the code is correctly answering this on line 20 by switching to the ownerSVG, if any:
e = e.ownerSVGElement || e;


Accept white-spaces in plugins parameter

@Romanior very smart regex to avoid trim()!
Problem:
" a,b,c ".split(/[\s,]+/) leads to ["", "a", "b", "c", ""].
We need to consider spaces before and after names, or we just trim once before spliting.


[Tracking] MambaLM training script

This issue is for tracking my progress towards a MambaLM training script on any NLP dataset.

See train branch for development towards this goal.

Initial training script is complete on text8. Haven't rigorously verified everything though.


chore: pin minor version of aiohttp

Yesterday, 3.11 version of aiohttp was released with breaking changes  which breaks aioresponses library used for our mocks and as consequence some of our tests are failing.
Ref: https://github.com/pnuckowski/aioresponses/pull/262
Regardless of whether this is fixed in aioresponses, aiohttp can introduce breaking changes in minor versions, so I pin it to the latest tested version as this library is critical for the stability in some of our features.

We are going to ship a 3.11.1 to restore back-compat for pnuckowski/aioresponses#262
I want to note that aioresponses could break at any patch release of aiohttp since third party libs that monkey patch aiohttp internals are not something we generally plan for upstream.
If you can use aiohttp's pytest plugin you should be a lot safer off from breaking changes in test infrastructure.

Thanks for your feedback @bdraco
We will explore the plugin that you suggest.


Timestamps in cache

Creates a notion of a header cache entry to encapsulate the info section along with time, this can be used to age out the entry

Not sure if we need that if not going to merge corresponding Carbon change...


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 10:36:29 on 2023.04.25
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 1, errors out: 0
Original problem ID: 19951
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=19951

Problem has been resolved in 3m 0s at 10:39:29 on 2023.04.25
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 19951


Modify return value of get/post function

This PR resolves return request response of get/post method

add return the request response

get/post function


Made compatible with Lua 5.3 and removed a warning when compiling abo…

…ut unused variable len.

:+1:


Extract data to xml or nsdictionnary

Hello,
i'm seeing some function to get a NSDictionnary but it seem to be from xml so i try to find a solution in your work but ...
did i miss something ? a method to extract directly in NSDictionnary ? or a method to extract xlsx to XML ?
Cheers

NSDictionary are used to read / write XML files.
XLSX are not converted to NSDictionary

okay, so you're solution can't be used for that.
just to try, do you try to do this ? or know solution for that ?

I haven't try since XLSX files are not just some value in a table. You have to deal with shared strings, styles, merged cells and many other things. You could generate some NSDictionary for your needs, but there's few chances that your needs match with someone else needs


Always return the id attribute

The id attribute is defined with "a 'returned' characteristic of
always" [1].
[1] https://tools.ietf.org/html/rfc7643#section-3.1
Fixes #151

Are you adding tests too?

Only if I have to :)

If you tell me how, @wallner, I can do this. Unfortunately, there are no tests at all, that care about the attribute filtering when searching for resources. I really don't know how to do this with MockMVC and the current attribute filtering process. As you are the author of both, please give me some advice how to test the stuff in this PR.

Alright, I've added 2 simple tests.

Excellent. Thank you!


Translator case insensitivity on Windows

Laravel Version: 5.5.21
PHP Version: 7.0.10
Database Driver & Version: Irrelevant

Description:
The translator class looks for the translation file in a case insensitive matter on Windows. Causing it to look up a PHP translation file instead of looking for a string in the JSON file.

Expected result:* a string from the JSON translation file (e.g. resources/lang/en.json)
Actual result:* the content from a PHP translation file (e.g. resources/lang/en/faq.php)

__('faq') should return the contents of the file resources/lang/en/faq.php.
__('FAQ') (or any other non-lowercase variant) should return a string from resources/lang/en.json.
Steps To Reproduce:

Create a file named faq.php in resources/lang/en and make it return an array:

<?php
return [
  'key' => 'value',
];

Make sure resources/lang/en.json does not exist or does not contain the key FAQ.
Call __('FAQ') or app('translator')->getFromJson('FAQ') from a controller.

The quick fix for this issue to add the keys causing problems to resources/lang/en.json.

Yeah you need to watch out from this edge case if you're using both translation methods in your project.

@themsaid This is not very clear from reading the documentation page.
Also, I assume this is not an issue on Unix-like systems where file names are case-sensitive. I haven't tested it, yet.

Laravel will first check for a JSON translation string before trying to find a PHP file, so yes including a JSON translation line for that key would fix the issue.