Could you update to correct semver on releases

I've found that tools like https://github.com/constabulary/gb don't work nicely with your releases because of a semver issue
Basically the problem is http://semver.org/#spec-item-9
A pre-release version MAY be denoted by appending a hyphen and a series of dot separated identifiers immediately following the patch version
Point being it needs a hyphen to comply with the spec
v2.0.0rc1 is not valid, but v2.0.0-rc1 is.
Would it be too much trouble for you to please push another release with a valid tag?  The old one can still exist to as not to break things for other people.

Will do! Thanks for letting me know

On Jan 5, 2017, at 5:58 PM, Brad Murray<EMAIL_ADDRESS>wrote:
I've found that tools like https://github.com/constabulary/gb don't work nicely with your releases because of a semver issue
Basically the problem is http://semver.org/#spec-item-9
A pre-release version MAY be denoted by appending a hyphen and a series of dot separated identifiers immediately following the patch version
Point being it needs a hyphen to comply with the spec
v2.0.0rc1 is not valid, but v2.0.0-rc1 is.
Would it be too much trouble for you to please push another release with a valid tag? The old one can still exist to as not to break things for other people.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

No problem, the rapid response is encouraging for considering using your library :-)

@pkieltyka I think we can release v2.0.0 after HTTP2 fixes.

@VojtechVitek yea, I agree. It's been a long time since the rc

v2 is here! https://github.com/pressly/chi/releases/tag/v2.0.0


trans -U via cron

Hello
Everytime I run trans -U via cron I gt the following error.
tput: No value for $TERM and no -T specified
Thanks,

Can you post full command in your crontab?

5 0 * * *       root    trans -U -no-ansi

Crontab doesn't seem to pass around the env variable $TERM, which must be set for determining the correct terminal type. Since it's running as a cron job I guess we can safely ignore that.


Call-out expanded: title and fold icon

I think the screenshots show the problem adequately.

Hey, thanks for the report. I'll look into that, expect a fix by this evening :)

I'm sorry, but I wasn't able to reproduce this bug. Can you turn on restricted mode and deactivate all your snippets, if you have any?
My guess would be that it is being caused by your font. Could you change it, and then make another attempt at reproducing this bug?
Which OS are you on?

Found the culprit: a snippet, specifically this one:
.callout-title-inner {
  font-size: 15px !important;
}

Nevertheless, I do find the font size of the call-out title too big. Even when I use Obsidian's default font, i.e. Inter, I still get it:

So, how can I reduce the call-out title's font size without getting it crossed through by the dotted line?
My OS: Linux Mint Debian Edition (LMDE).

Ah, this explains the issue. Thank you anyways for the feedback - although changing the callout font size would be to personalized (and special) to include in style settings - because I also thought about adding it there. But seems like you found your solution :)
Thing about sizes: I'd suggest you set sizes (in your snippets for example) not in px, but in em (where 1em = your current font size -> in my theme, callout title font size is 1.4em).
Personally, I change my font size a lot, because I use it as a "zoom" feature, and set it depending on the distance between me and my screen...

I set it to 1em but that reduces the font size and brings back the issue, so 1em is not equal to my current 23px.
However, 1 rem is always (if I am not mistaken) equal to 16px, so my 23px = 1.4375rem. If I use that value in my snippet the font size does not change.
Someone once told me to start using rem (or even em) because px will eventually be dropped. However, I noticed that few people seem to be using those units, the overwhelming majority still uses px, so I reverted to px because I am used to it so the units mean more to me than em/rem. Perhaps I should revert back to using rem; I don't like em because it is not a fixed value, it depends on the "parent", whatever that is.
Another thing is you say that "if you lower your global font size, your modified callout font size will stay the same" — you mean it will stay the same if I use em as units. But if I use px it also stays the same if the global font size is changed, because, as I have always understood it, snippets override everything else — in any case that has been my experience.


feat: Allow replacing FlatList component via prop

Adding support for a drop in replacement of the FlatList component using the FlatList prop.
Fixes #24
(Draft at the moment as I haven't had a chance to test it all out and add any required docs, if someone already has an implementation they can try that'd be great ;))

I screwed the pooch on this one. Implemented as #62 . Thanks for the contribution, @kav!


Move testing from Travis CI to GitHub Actions

Travis CI has a new pricing model which places limits on open source.

https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing
https://www.jeffgeerling.com/blog/2020/travis-cis-new-pricing-plan-threw-wrench-my-open-source-works

Many projects are moving to GitHub Actions instead, let's do the same.
~We already have tests on GHA, just missing sending coverage to Codecov. Earlier it was tricky to do, but Codecov now have an action for this; added here.~ Done in #480.

We'll need to move deployment from Travis to GHA. We already got it working for another Jazzband project, but I'll do it in a followup PR.
So this one reduces the Travis testing to a single job.

~This also fixes a pytest warning:~ Done in #480.

I'm not so thrilled to add even more dependencies on Github. I don't see Travis limitation a problem for our usage, do you?

Unfortunately yes, others and I are seeing big problems with Travis CI.

https://travis-ci.community/t/contradictory-information-in-the-ui-on-whether-credits-are-replenished-on-the-free-plan/10518
https://travis-ci.community/t/extremely-poor-official-communication-of-the-org-shutdown/10568
https://travis-ci.community/t/org-com-migration-unexpectedly-comes-with-a-plan-change-for-oss-what-exactly-is-the-new-deal/10567
https://travis-ci.community/t/how-to-migrate-from-org-to-com-in-the-light-of-org-shutting-down/10566
https://travis-ci.community/t/builds-hang-in-queued-state/10250
https://travis-ci.community/t/builds-taking-1hr-to-start/10560
https://travis-ci.community/t/my-build-is-stuck-in-waiting-to-be-queued-phase/10533
https://travis-ci.community/t/build-delays-for-open-source-project/10272
https://travis-ci.community/t/exceeded-user-count-unlimited/10484

First, travis-ci.org has been really slow (maybe we were lucky just now, on a Saturday) with long delays of sometimes up to an hour before jobs begin.
That's because they've been moving resources from .org to travis-ci.com, and everything will need to be moved to travis-ci.com by the end of the year, when .org will be closed down.

https://docs.travis-ci.com/user/migrate/open-source-repository-migration

Second, there's a new credit-based pricing model on .com.  When that expires they encourage you to buy a plan or ask to be on some new OSS plan, where if the project meets the requirements, they will evaluate how many credits the project can have. It's not clear if this renews monthly or if the process must be repeated when the credits expire.

https://blog.travis-ci.com/2020-11-02-travis-ci-new-billing

I've applied for credits for another project but not got very far yet.

https://twitter.com/hugovk/status/1326935425903185920

Some more info here:

https://github.com/zopefoundation/meta/issues/38

Thanks so much for this detailed report. I would not rush however in changing things, firstly because tablib does not see so much activity (we can affort waiting 1-2 hours for tests) and Travis may react to the Web feedback at some point.

Okay, let's wait a bit.
I'll make a new PR with the first commit (GHA coverage) and third (fix pytest warning), they're still useful now.

(Rebased after merging subset #480.)

All Jazzband repos need porting over to GHA:

https://github.com/jazzband-roadies/help/issues/206

The alternatives of course are Azure, GitLab, Circle CI and a few others. Given the fact that GitHub Actions are designed to allow self-hosted test runners I'd say the risk of lock-in is managemable compared to the added benefits of direct integration in the GitHub development workflow.
Luckily there is https://github.com/ymyzk/tox-gh-actions so the migration of all projects should be relatively straight forward.


Error loading robot system

Hi, I'm getting this error when trying to load robot system:

Solution exception:The 'Base' start tag on line 17 position 9 does not match the end tag of 'Positioner'. Line 24, position 9.

I believe I followed the installation instructions correctly. For installing the robot libraries, I tried downloading the repo and copying them. After reading issues 4 and 15, I also tried downloading the xml from raw view. Any ideas?
Thanks, Brendan

Hi I'm having the same error but I did actually unblock bartlett.3dm. I've also downloaded grymsdyke to see if the bartlett file was the issue, and I can see the grymsdyke robots in the dropdown menu but i still get that error with the bartlett ones, the grymsdyke ones, and the default robots.

I have also downloaded the xml files from raw and believe I've followed the installation instructions

Ignore this. The issue ended up being that grasshopper was looking in my user/onedrive/documents rather than user/documents for the Robots folder


Input component page needs to be updated to reference forum issue

Resolves #164

 Issue #164: Input component page needs to be updated to reference forum issue


Update register-a-service-principal-name-for-kerberos-connections.md

What if the computer is joined to Azure AD?

Hi Rick, since you're staff, our contributor guide requires that you submit changes using the internal sql-docs-pr repository.
In the meantime, I'll need to verify this change with a PM.


Upgrade to Spring Boot 2.3.2 and Spring Cloud Hoxton SR7

Coverage decreased (-0.06%) to 93.182% when pulling f614368cdc08ee4479a2635ef100e29c2b97fb8f on mprimi:GENIE-645-boot23 into fb8a2d85eadc9419d1b2c353a673484a7673d8a5 on Netflix:master.


Fix spelling mistake.

This change is

Overall the docs have been incredibly helpful. Thank you to the maintainers!

I have signed the CNCF CLA.

/lgtm
Thanks for contribution!

@ianfisk Please rebase your branch so that it picks up 99fbc2b. That will fix the deploy/netlify failure. Thanks.

merging anyway -- tiny change for typo so seems safe.


Doc// typo mistake in contribution.md

In the contribution.md file,the first line has a typo mistake,it says First off,  instead it  should be First off all.

self assigned

#2050
Typo fixed in contribution.md

Actually, first off is correct English and the change you made must be "first of all" instead of "first off all" incase you really want to change the statement

There is no issue in this right ?


[Question]: when i query, whey always the same contents being sent to LLM

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
when i query, whey always the same contents being sent to LLM

now sure what i need to do

@axz91 can you share a more reproducible example? which models, embeddings, vector db


Network Subgraphs QA

Boba Network QA

Description
Value

Subgraph Reviewed
https://thegraph.com/hosted-service/subgraph/messari/network-boba

Date Reviewed
November 29 2022

Schema Version
1.1.1

Subgraph Version
1.1.3

Methodology Version
1.0.0

Evidence Sheet
https://docs.google.com/spreadsheets/d/14ScSLH27gfsbgkz54ShC7FoTQUMuYqJtYksk781rPz4/edit#gid=1278898122

Block Metrics

Block ID
Metric
Comments

897310
hash
Hash for the transaction looks incorrect subgraph is outputting 0x995825f51ad1f4fa9e0917c5bd9e9be1e141f644a05ad7412be3c19361129cb1 and bobascan  is outputting 0x56080c9e72c20e28c01170d7eeabc4f14a7bbaaa9170547163c41a9887d6fe4d

897310
gasLimit
Subgraph is reporting 11000000 and bobascan is reporting 412,706

897310
blockUtilization
This is off because gasLimit is not being recorded correct. Subgraph is reporting 182% and bobascan is reporting 48.69%

Cosmos QA

Description
Value

Subgraph Reviewed
https://thegraph.com/hosted-service/subgraph/messari/network-cosmos

Date Reviewed
November 29 2022

Schema Version
1.1.1

Subgraph Version
1.1.3

Methodology Version
1.0.0

Evidence Sheet
https://docs.google.com/spreadsheets/d/1zXe2TT-MzeBUU_vLhZcIdoLNetexTepFeY4ZBgAYh50/edit#gid=0

Block Metrics

Block ID
Metric
Comments

13020615
author
authorID being output by the subgraph is0x0b9b84f822a54e18fdf51b95da8245794c17a08f4e72efc52bd4ff41484ec9b2 when mintscan is outputting cosmosvaloper18ruzecmqj9pv8ac0gvkgryuc7u004te9rh7w5s

Author Metrics

Author
Metric
Comment

0x0b9b84f822a54e18fdf51b95da8245794c17a08f4e72efc52bd4ff41484ec9b2
id
This id does not exist but is reported for the block above. Check mintscan

0x0b9b84f822a54e18fdf51b95da8245794c17a08f4e72efc52bd4ff41484ec9b2
cumulativeBlocksCreated
Id does not exist so this is not being recorded properly either

Arweave QA

Description
Value

Subgraph Reviewed
https://thegraph.com/hosted-service/subgraph/messari/network-arweave-mainnet

Date Reviewed
November 29 2022

Schema Version
1.1.1

Subgraph Version
1.1.3

Methodology Version
1.0.0

Evidence Sheet
https://docs.google.com/spreadsheets/d/1RtBfol1ziBHLG59PjcTHC20Z_2e2jwmKHozNrg4jJ30/edit?usp=sharing

Network Metrics

Section
Value
Comments

networkDailySnapshots
cumulativeUniqueAuthors
This could be correct, but because daily is off going to mark this is as also off

networkDailySnapshots
dailyUniqueAuthors
Looks like this is comparing nodes and subgraph is recording 33 daily authors and arweavescan is reporting 66 nodes

networkDailySnapshots
cumulativeDifficulty
Subgraph is not decoding this value correct. Block 1058151 is recorded with a difficulty of<PHONE_NUMBER>09702060000000000000000000000000000000000000000000000000000000000000 on subgraph and is 32 on arweavescan

networkDailySnapshots
dailyDifficulty
Realizing cumulative difficulty doesn’t really apply to arweave so that is wrong it is just the current difficulty, and that is some function of how difficult it was to “mine” the most recent block

Block Metrics

Block ID
Value
Comments

1058151
author
authorID for this block is being recorded by subgraph as 0xfe7c7e404282730ef7aa927e4e6ab2aba0d33be8081693760b564db2492cc1b7 and arweavescan is reporting nx-QEKCcw73qpJ-Tmqyq6DTO-gIFpN2C1ZNskkswbc

1058151
difficulty
See dailyDifficultycomments

Author Metrics

ID
Value
Comments

0xfe7c7e404282730ef7aa927e4e6ab2aba0d33be8081693760b564db2492cc1b7
id
This id does not exist on arweavescan, so all metrics being reported are incorrect here

0xfe7c7e404282730ef7aa927e4e6ab2aba0d33be8081693760b564db2492cc1b7
cumulativeBlocksCreated
See above

0xfe7c7e404282730ef7aa927e4e6ab2aba0d33be8081693760b564db2492cc1b7
cumulativeDifficulty
See above and dailyDifficulty comments

@dmelotik Ethereum, Moonbeam, and Moonriver look good

Cosmos author should be using header.proposerHash


Remove details tab for charts, point to bitnami differently

Remove details tab for charts and make bitnami chart repository point to instruction page rather than having an error page opened from the link

@mt-gh-test  like the URL change.
What is the reason not to have it in a "details" tab? Is it because all need to know this?

It feels a bit strange to mention in the hidden info the repository which is going to be used in the excercises, rather than having it shown in the main page.
"One of the largest Chart Repositories is the BitNami Chart Repository, which we will be using in these exercises."


Align validation error and warning kind structs with 745

Description of changes
Follow on to #903 (needs to be merged after that PR), aligning the error and warning kind structs with #754.
I plan to follow this PR with further changes to eliminate the *Kind enums.
Issue #, if available
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):

[x] A breaking change requiring a major version bump to cedar-policy (e.g., changes to the signature of an existing API).

I confirm that this PR (choose one, and delete the other options):

[x] Updates the "Unreleased" section of the CHANGELOG with a description of my change (required for major/minor version bumps).
I confirm that cedar-spec (choose one, and delete the other options):

[x] Requires updates, and I have made / will make these updates myself. (Please include in your description a timeline or link to the relevant PR in cedar-spec, and how you have tested that your updates are correct.)

CI is not triggered. Maybe do a merge from main to restart it?


Very minor issue - can we please have a white/light theme for the docs?

Every other documentation page I use is white, and it's always jarring / hard to adjust to read http://passportjs.org/docs when switching between different doc pages. Any chance we can get a light theme?
Sorry for the minor question - and thanks!

Please file this issue here: https://github.com/passport/www.passportjs.org


Remove Xunit-Performance Runner from the master branch

We need to remove the xunit-performance runner from the master branch because it is considered deprecated. This change is in favor to the xunit-performance-api which is actively being developed to support benchmarking .NET core and full framework.
The old runner has been moved to the branch origin/1.0.0-alpha, where bug fixes and servicing will be done.
In addition, we need to change the versioning of the API from alpha-build00xx to beta-build000x to avoid conflicts with releases of the two branches.

Sounds good.  Thanks @jorive.

This was done a while back (old runner moved to a different branch to allow bug fixes to old users).


onSpinnerItemSelectedListener not properly listening on events

Please complete the following information:

Library Version 1.2.4
Affected Device(s) Google Pixel 6 API 28

Describe the Bug:
Selecting a spinner item in the UI does not trigger the expected onItemSelectedListener callback.  However, the correct item shows up in the UI and the dropdown closes
Expected Behavior:
the call back is just supposed to log the lambda parameters as a test, but the log is empty.
the listener declaration is this:
binding!!.spinner.setOnSpinnerItemSelectedListener { one: Int, two: String?, three: Int, four: String? -> println("$one, $two, $three, $four") }
Where binding is the fragment binding, spinner is the PowerSpinnerView object, and the block is the callback.

Hi, I'm facing the same case here..
any solutions?
thanks in advanced


✨ Install default-catalogs from Catalogd release

Description
Addresses #1026 by adding OperatorHub to the default installation script.

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

@trgeiger - requires rebase, and one of the e2e's is failing

@tmshort e2e passes now, it was mad about the presence of operatorhub.io so I just edited the setup script to remove it for the test. Codecov is mad but everything else is passing


NoUnused.CustomTypeConstructors: phantom type constructor name should not have to match

As types to use as a phantom type argument, people can for example use
type APhantomTag
    = APhantomTag Never -- not constructable

a : T APhantomTag

NoUnused.CustomTypeConstructors does not flag this constructor.
However, if the constructor name doesn't match the type name, the constructor is reported.
type APhantomTag
    = APhantomTagForT Never

→ Don't check for equal names.

Hey! Thanks for the issue :wave:
I have to admit I'm surprised the rule doesn't report an issue for the first case :thinking:
The documentation of the rule has this explanation for handling of phantom types:
This does not prevent you from using phantom types: A constructor won't be reported if

- It is the only constructor of a type that has no type variable
- It has no parameters
- It is used as an argument of a custom type, in the stead of a type variable that is not used in the definition in any of the type's constructors

It seems like it's a bit out of date or at least unprecise, because we do check for Never, contrary to the explanation.
Would it be a good solution for you if the explanation was done better? Or maybe if there was an automatic fix to add Never and/or rename the constructor?
Or do you have a problem with having this convention for phantom types? If so, could you please explain what bothers you about it?

Or maybe if there was an automatic fix to add Never

That would be 👍

a problem with having this convention for phantom types?

The constructor is Never used, it's name doesn't matter → don't check it
It could (and has) lead to name clashes.

type AOrB phantom
    = A
    | B

type A = A Never
--       ^ name clash

a : AOrB A

Nothing should hold you back from having a phantom type that has the same name as an existing variant.


Add option to interrogate redis less frequently

There could be a configuration to limit interrogation to redis to e.g. every nth transaction instead of every transaction.

This type of interaction would not scale if the application is deployed horizontally.
The point of Redis for this purpose, is to provide a single source of truth, and if we talk to it less frequently, we are no longer trusting Redis with that. If the requests are routed to the same node each time, sure, we don't "really" need to hit Redis each time, but because that isn't guaranteed, I'd recommend not changing the behavior.
You can use a memory based store if you want to reduce database interaction if that's the case.


Wrong image in example

This is a Bug Report

Problem:
Wrong image for this example
mysql:5.7 now uses oraclelinux:7-slim
This OS doesn't have a "hostname" command
image
Proposed Solution:
Using image mysql:5.7-debian
Page to Update:
https://kubernetes.io/docs/tasks/run-application/run-replicated-stateful-application/#statefulset

This issue is a dup of #35432.
/triage duplicate
There is a relevant PR: #35857.


Fix for null PyObject Pointers at test python wrapper factory

Fixes the following incorrect reference decrement crash seen at some builds:
Program terminated with signal 11, Segmentation fault.
#0  0x00000000011495ad in hazelcast::client::test::HazelcastServerFactory::shutdownServer (this=0x2cc88f8, member=...) at <https://hazelcast-l337.ci.cloudbees.com/job/cpp-linux-nightly-64-SHARED-Debug/ws/hazelcast/test/src/HazelcastServerFactory.cpp>:192
192                         Py_DECREF(resultObj);

Also, better handling of null Python object references.
Info: https://docs.python.org/2.0/api/refcountDetails.html

Test PASSed.

Test PASSed.

Test PASSed.

Test PASSed.

Test PASSed.

Test PASSed.

Test FAILed.

Test FAILed.

verify-linux

verify-windows

Test PASSed.


Fix ANSI coloring on queries

The SQL adapter has several sql_color functions to apply color coding to different query types. However, because it's being passed a %Postgrex.Query{} (or %Mariaex.Query{}, etc.) instead of a string, it never matches and always falls back to nil. Passing the String.Chars.to_string(query) result instead matches properly and returns the correct colors.
I didn't see anything in the existing tests related to testing the logger, but if there's somewhere I should/can add a test for this, I'll gladly update the pull request.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Proposal: SVC BR

This proposal would allow neighboring BRs to communicate with each other without specific address knowledge.
The SVC_BR is a special SVC address, where multicast is not allowed and anycast behaves like unicast, targeting a specific BR Control instance, to allow multiple instances registered with the same dispatcher.
To support multiple BR control plane instances registered with the same dispatcher, the BR data plane would replace the port, which for any other SVC address would be 0, with its control plane port (UDP checksum re-calculation required), so the dispatcher can use the port information to find the correct BR control instance to deliver the packet to.
These packets would use a OneHopPath extension, which in itself provides some degree of source authentication when BRs are directly connected to each other, but it is not enough in switched networks. For switched networks, authentication is desired, for example using DRkey, to verify the real source of the control packets. Connection oriented transport protocols do not fit given that we need to support multicast.
The BR control plane would have to interact with the certificate server to generate the neighbor AS DRkey to authenticate packets with OneHopPath extensions.
EXTRA NOTES
As an alternative to introducing SVC BR, we could leave the current behavior of BR control not able to directly communicate with neighboring AS BRs and instead use the Beacon Server (BS) as a sort of proxy that could verify packets with OneHopPath extensions, given that the BS already has DRkey support.

So after arriving to this issue regarding 'No ACL support' comment in the repo, this issue is related because the ACL feature needs to push rules to neighboring BRs.

Closing this, the router no longer runs a control-plane and we're going in the direction of removing SCION Service (SVC) addresses rather than adding more.

Closing this, the router no longer runs a control-plane and we're going in the direction of removing SCION Service (SVC) addresses rather than adding more.


Warden?

How do you integrate warden with your project ? i have some problems with session managment when i use my authenticate  method in a separate mount point
foo.rb
class Foo < Grape::API
 resource :foo do
    resource :login do
      post do
        warden.authenticate :api_v2
        # redirect "/api/v2/foo/#{current_user.seller_profile.id}", :permanent => true
      end
    end
end

v2.rb
class V2 < Grape::API

  Warden::Strategies.add(:api_v2) do
    def valid?
      params[:email] || params[:password]
    end

    def authenticate!
      user = User.find_by_email(params[:email])
      if user && user.valid_password?(params[:password])
        success!(user)
      else
        error!("You're unauthorized to access the API!", 401)
      end
    end
  end

  helpers do
    def warden
      env['warden']
    end

    def current_user
      warden.user
    end

    def authenticated?
      if warden.authenticated?
        true
      else
        error!("You're unauthorized to access the API!", 401)
      end
    end

    def logout!
      warden.logout
    end
  end

  resource :test do
    before do
      authenticated?
    end
    get 'ping' do
      {:ping => 'pong'}
    end
 end

What am i missing here ?

the sample is not online anymore... any other links / suggestions on this?

https://github.com/geekazoid/grape-warden
also http://code.dblock.org/2011/05/04/grape-api-authentication-w-devise.html


ERROR ref_block_num: number is out of range, wrong type on abi json field ?

Hi guys,
I've been testing this lib for a while.
I'm quite sure there is an issue with the file transaction.abi.json.
When the blockchain has more then 65535 blocks the abieos_json_to_bin_reorderable function fails with a number is out of range error (Iuse the same checks then the test file).
The problem is when the blockchain has more then 65535 blocks the json->bin is wrong. I've also made a script to compare the result with cleos.
Also if changing in  transaction.abi.json:
                {
                    "name": "ref_block_num",
                    "type": "uint16"
                },

to :
                {
                    "name": "ref_block_num",
                    "type": "uint32"
                },

it will remove the error but the result will be wrong and the bin->json is corrupted.
If you have a solution to correct the conversion corruption please let me know :) !
Thanks in advance.

Update: As an example of this issue, you can test is with my program here.

ref_block_num expects the 16 LSBs of the block number

Thanks a lot, it fixed my issue.


Use the larger runner for Windows clippy runs

Running clippy can apparently be longer than the tests in https://github.com/astral-sh/uv/pull/3040

Another option is to cross-run clippy on a linux machine:
cargo xwin clippy --target x86_64-pc-windows-msvc


qemu: 4.0.0 -> 4.1.0

Motivation for this change
Things done

[x] 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 nix-review --run "nix-review wip"
[x] 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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

ah, already here https://github.com/NixOS/nixpkgs/pull/66775


Add limetorrent logo

Limetorrent does not exist in the icon folder and should be added.

Merged!, thank you so much for this!

No problem, Am I safe to delete the branch? @gubatron


Release kibana 2.0.3-5.6.2 (automated commit)

Release kibana 2.0.3-5.6.2 (automated commit)
Description:
Source URL: https://infinity-artifacts-ci.s3.amazonaws.com/autodelete7d/kibana/20171110-003348-QgqvQwLoNDUNQHX3/stub-universe-kibana.json
Changes since revision 3:
0 files added: []
0 files removed: []
1 files changed:
--- 3/package.json
+++ 4/package.json
@@ -1,14 +1,14 @@
 {
     "packagingVersion": "4.0",
     "upgradesFrom": [
-        "2.0.1-5.5.1"
+        "2.0.2-5.6.2"
     ],
     "downgradesTo": [
-        "2.0.1-5.5.1"
+        "2.0.2-5.6.2"
     ],
     "minDcosReleaseVersion": "1.9",
     "name": "kibana",
-    "version": "2.0.2-5.6.2",
+    "version": "2.0.3-5.6.2",
     "maintainer"<EMAIL_ADDRESS>     "description": "Kibana 5, and optionally X-Pack",
     "selected": true,

Closing to fix version


TypeError: get_custom_vcr() got multiple values for argument 'base_uri'

Hello,
it seems that running this example in the docs throws a TypeError: get_custom_vcr() got multiple values for argument 'base_uri
The problematic code is directly from the doc:
#!/usr/bin/env python

import click
import json
from vcr.util import read_body
from cornell.cornell_server import CornellCmdOptions, start_cornell
from cornell.cornell_helpers import json_in_headers
from cornell.custom_matchers import requests_match_conditions

# Custom Matcher
@requests_match_conditions(json_in_headers, lambda request: request.body)
def vcr_json_custom_body_matcher(received_request, cassette_request):
    received_request_dict = json.loads(read_body(received_request))
    cassette_request_dict = json.loads(read_body(cassette_request))
    if received_request_dict == cassette_request_dict or "special_params" not in received_request_dict:
        return True
    return is_specially_matched(received_request_dict, cassette_request_dict)

@click.command(cls=CornellCmdOptions)
def start_mock_service(**kwargs):
    start_cornell(additional_vcr_matchers=[vcr_json_custom_body_matcher], **kwargs)

if __name__ == "__main__":
    start_mock_service()

Dockerfile to reproduce:
FROM python:3.9-alpine

RUN pip install cornell
COPY cornell/main.py .
EXPOSE 9000

ENTRYPOINT ["python", "main.py"]

It seems that changing the line 189 in cornell_server.py to     app.config.vcr = get_custom_vcr(app.config.base_uri, DUMMY_URL, record_errors,                                     *additional_vcr_matchers) fixes this, but it seems hacky. Overall I haven't been able to make the custom matchers work, anybody has a working example ? I'm looking to have a sidecar container that will record every request with same data parameters to an API, this seems like something cornell should do

Hello @AvnerCohen, yes as stated in my previous message, this does work

@Wats0ns , @AvnerCohen : I tend to agree that just removing the keywords is bit "hacky", please consider this fix

@Wats0ns Thanks for the report, this is now fixed in https://pypi.org/project/cornell/1.0.8/
@yaelmi3 👑


Help with custom input on arrays.

Hi, I've been struggling to make this work, I searched intensively on the issue but I can't make this work. I have this schema.
  images: {
    type: [String],
    defaultValue: []
  },
  'images.$': {
    type: String,
    autoform: {
      type: 'thumbnail'
    }
  },

It's an array of ids. But instead of showing an input field I want to show the image that belongs to that id. I did the following.
Template.afThumbnail.helpers({
  image: function() {
    return Images.findOne(this.value);
  }
});

AutoForm.addInputType('thumbnail', {
  template: 'afThumbnail'
});

<template name="afThumbnail">

</template>

It shows the image correctly, no issue here. But when I click on the button to remove one of them from the array, it removes the whole array instead. And it was working before with the default text input. Thank you for your time.

Probably because you are not providing any way to retrieve the input value from the "input", and you are not adding the provided attributes to it.
The simplest solution might be
<template name="afThumbnail">
  <input type="hidden" value="{{this.value}}" {{this.atts}}/>

</template>

The only drawback is that any extra attributes you provide on the afFieldInput or in the schema will end up on the hidden input rather than on the visible img. If that's a problem, you could make some helpers that divide up this.atts and put some on the hidden input and others on the img. It's important that the data-schema-key attribute ends up on the input.

Thank you, it works. I was trying to write a valueOut function and everything, and it was this simple. I'll close the issue, thank you for your time.

The trick is that if there is no valueOut function, it does .val() on whatever element has the data-schema-key attribute. Obviously .val() wouldn't work on an image element, but adding a hidden element makes the default logic work. You could have also stored the value in a data- attribute on the img and then added a valueOut that returns that.


Async support for PyMySQL

Hi Naoki,
I've decided to open a ticket here, because I did not find any other reliable way to contact you.
Anyways,
I'm co-maintainer of python-momoko - its a psycopg2 wrapper for Tornado. It utilizes psycopg2 excellent async interface and contains very little amount of psycopg2-specific code.
I was thinking about generalizing momoko by adding MySQL support as well, but I did not find any  async support in neither PyMySQL nor mysqlclient-python (both of them you are happen to maintain).
So I've thought I'll hack such support myself into PyMySQL and then plug it to momoko (or to any other event-based library for this matter). So far have have a proof of concept code that can only connect, and does not support SSL. (I've roughly merged it against your latest changes, so its most probably broken. I've pushed just to demonstrate).
Now I've run into your new Tornado-MySQL project and looks like you are forking PyMySQL all together for async support Tornado-specific way...
The thing is that Tornado *SQL support is rather sparse; everyone seems to do it their own way. Its not just for Tornado but for async + sql in general (psycopg2 being a notable exception).
I think there is an opportunity here to standardize it a bit.

sql libraries can expose a standard async API (can be based on psycopg2 definition) which is independent on particular event loop implementation
then Tornado or any other event library can easily utilize them

So, my question is whether you can help with adding psycopg2-like async support to PyMySQL, so I'll can concentrate my efforts on momoko. I think it will provide more mature MySQL support in tornado much sooner.
P.S. My idea regarding async+PyMySQL is as follows:

Convert all current functions that access socket to be async, with yields, etc.
Then create sync versions of those functions that will call self.poll() by themselves.
This way we'll have minimum amount of code duplication

P.P.S. I'm on vacation till October, and will not be able to contribute any code till then.

Awwww.. sad thread.  Did this get any traction?

No unfortunately.
On Oct 7, 2015 00:47, "Shane R. Spencer"<EMAIL_ADDRESS>wrote:

Awwww.. sad thread. Did this get any traction?
—
Reply to this email directly or view it on GitHub
https://github.com/PyMySQL/Tornado-MySQL/issues/1#issuecomment-146011088
.

Sorry. Current my thought is here


fix: sync network publicScope

这个 PR 实现什么功能/修复什么问题:
sync network publicscope
是否需要 backport 到之前的 release 分支:

release/2.10.0

Codecov Report

Merging #1194 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1194      +/-   ##
=========================================
- Coverage     7.9%   7.88%   -0.03%
=========================================
  Files         368     368
  Lines       57853   57859       +6
=========================================
- Hits         4575    4562      -13
- Misses      52880   52897      +17
- Partials      398     400       +2

Flag
Coverage Δ

#aFlag
7.88% <0%> (-0.03%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/networks.go
0.97% <0%> (-0.01%)
:arrow_down:

pkg/util/ucloud/network.go
0% <0%> (ø)
:arrow_up:

pkg/cloudcommon/db/sharablevirtual.go
0% <0%> (ø)
:arrow_up:

pkg/util/aws/network.go
0% <0%> (ø)
:arrow_up:

pkg/util/seclib2/seclib.go
91.48% <0%> (-4.26%)
:arrow_down:

pkg/util/rbacutils/rbac.go
71.29% <0%> (-3.48%)
:arrow_down:

Continue to review full report at Codecov.

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

/lgtm
/approve


[networks] Change util.Address implementation

What does this PR do?

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

I think you can update this code to not need a net.IP anymore: https://github.com/DataDog/datadog-agent/blob/main/pkg/network/tracer/gateway_lookup.go#L107

Examining anywhere util.NetIPFromAddress is used and seeing if we can get rid of net.IP would be good.


adding codeowners file

Added CODEOWNERS file
PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I think the nf-core lint fail will be fixed by the merge of #38

ping @Lucpen


topContentAdditionalInset updates after some delay.

[ Done ] I have searched existing issues and this is not a duplicate.

General information

iOS version(s): 10.0
Devices/Simulators affected: Both
Reproducible in the demo project? (Yes/No):  No

Bug report
I have added a custom view on JSQMessagsesViewController subclass and using topContentAdditionalInset to provide space to that view on top. JSQMessagsesViewController is embeded in navigation controller. For the first time when user enters JSQMessagsesViewController the custom view is present but after one pops out and again goes to JSQMessagsesViewController the custom view is not present but appears after some delay. I'm setting the custom view and topContentAdditionalInset in viewDidLoad().
So the problem is the custom view is already there but topContentAdditionalInset updates after some time once the JSQMessagsesViewController is visible.
The above problem is only with iOS 10.
It works properly in iOS 9 and 8.

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


fix argument error problem update from 1.6.0 to newer

Fixed an error related to date_as_argument option that occurred when temporarily deploying multiple versions of sidekiq-cron.
This problem affects when you have a deployment where multiple versions of sidekiq-cron temporarily exist (such as rolling update)
In this case, an option created in a newer version was unintentionally processed in the older version (1.6.0) and did not work correctly.
For more information, I have attempted to explain this at https://github.com/sidekiq-cron/sidekiq-cron/issues/350#issuecomment-1409798837
This PR avoids this problem by changing the hash of options created in the new version to work in the 1.6.0 version as well.
close: https://github.com/sidekiq-cron/sidekiq-cron/issues/350

@vsuhachev could you please review or leave your thoughts this one 🙏🏼?


We don't need call info method every time we want redis resource

@alireza-m Thank you for finding this!
But there is one possibility the version information would not get detected.
-> In case you add an existing and already connected redis resource to the resource manager - see https://github.com/alireza-m/zend-cache/blob/ede35a3dc6215decf88f23feaefac7438195e3cf/src/Storage/Adapter/RedisResourceManager.php#L337
The logic of this initialized flag is really strange on here and should be fixed.

But  the $resource['version'] can be set only once.
See  https://github.com/zendframework/zend-cache/issues/119


SignTools.exe  Where is it??

I am trying to build a docker container with a trimmed down msbuild tools in it.
I am currently using an offline installer that I created with this command:
.\msbuild2019.exe --layout C:\dev\jenkins-slave-msbuild\OfflineVS2019 --add Microsoft.VisualStudio.Workload.MSBuildTools --add Microsoft.VisualStudio.Component.VC.ATL --add Microsoft.VisualStudio.Component.VC.Tools.x86.x64 --add Microsoft.VisualStudio.Component.Windows10SDK.17763 --add Microsoft.VisualStudio.Component.Windows10SDK --add Microsoft.Component.ClickOnce.MSBuild --lang en-US
It builds okay but the container doesn't have SignTool.exe in it.
Is there a component that I should add? and if so can you reference the component on this page?  Just saying it gets installed with Visual Studio installation is pretty useless because there are a milion optional components in a visual studio installation.

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

ID: 72a4dbac-a145-e79f-121d-3d8df67552b1
Version Independent ID: 9ea61916-a9db-f23e-4c43-edc06650483d
Content: SignTool.exe (Sign Tool)
Content Source: docs/framework/tools/signtool-exe.md
Product: dotnet-framework
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@ornellaalt This is a question about Visual Studio installation - do you know who might be able to answer it?

On my machine, I found it exists in two places:
C:\Program Files (x86)\Windows Kits\10\bin\10.0.18362.0\x86\signtool.exe
C:\Program Files (x86)\Windows Kits\10\bin\x86\signtool.exe

However, I don't think you really need to know where they are. If you used "Developer Command Prompt for Visual Studio", you should be able to call it without really knowing where the exe exists.

But the issue is that if you are doing a minimal build of Visual Studio -- you need to know which component to add to make sure that you get the tools.  from what you posted it kind of looks like they are included in windows SDK10.

I created an offline installer using this command:
.\msbuild2019.exe --layout C:\dev\jenkins-slave-msbuild\OfflineVS2019 --add Microsoft.VisualStudio.Workload.MSBuildTools --add Microsoft.VisualStudio.Component.VC.ATL --add Microsoft.VisualStudio.Component.VC.Tools.x86.x64 --add Microsoft.VisualStudio.Component.Windows10SDK.17763 --add Microsoft.VisualStudio.Component.Windows10SDK --add Microsoft.VisualStudio.Component.VC.ATLMFC --lang en-US
and after installing I didn't have signtools.exe

But the issue is that if you are doing a minimal build of Visual Studio -- you need to know which component to add to make sure that you get the tools. from what you posted it kind of looks like they are included in windows SDK10.

My bad, I read only the title. I don't know in your case what you should do to be able to find/install it.

https://stackoverflow.com/questions/31869552/how-to-install-signtool-exe-for-windows-10
There is this link showing how to hack signtools out of a Visual Studio Installation and I am tempted to do it this way.


install directions for "from source"

on CentOS I needed to give an additional flag to wget
wget --output-document=pick-1.1.1.tar.gz.asc https://github.com/thoughtbot/pick/releases/download/v1.1.1/pick-1.1.1.tar.gz.asc
wget --output-document=pick-1.1.1.tar.gz https://github.com/thoughtbot/pick/releases/download/v1.1.1/pick-1.1.1.tar.gz
for those that want to just copy pasta what I did for the CentOS install, continue from the above:
gpg --verify pick-1.1.1.tar.gz.asc
tar -xzf pick-1.1.1.tar.gz
cd pick-1.1.1
./configure
make
sudo make install

Hey @mooreniemi,
What happened if you did not supply the --output-document flag? What version of wget are you running?

I'll close this issue due to inactivity.


change the way we test for pods and pod status; increase test duration by 50%

change the way we test for pods and pod status
increase test duration by 50%

:+1:


Handle pull requests with git-mirrors

This fixes a regression that was introduced in https://github.com/buildkite/agent/pull/1112 which means git-mirrors doesn't work with PRs from forks

@sj26 Added some comment output to the GitHub PR pathway! Feel free to merge if this is good

Perfect!
Do the tests pass? Is there a way to add a test for this easily? (Doesn't look they were run here, perhaps turned off for third party builds?)

go test is weird and not trustworthy for me locally so I’ve forced the issue by duplicating this in #1318

The existing tests are passing, nice.
We either need to manually try this out and verify it works, or add an automated test.

Took some doing, but I’ve added a spec in bc418f792194bacf3eb18d4f21691afdbfb9b2ef, which I hope covers enough of it to be useful. The rest of the GitHub-specific code is missing coverage, also.


Wrong objective formula in case of "grid cost" cost function

While looking at the code in order to understand the functioning of EMHASS, I noticed an inconsistency in the objective formulas.
The profit cost function shows this:
https://github.com/davidusb-geek/emhass/blob/bec6f76f1d7795a1ab8730fb0a61f883e75caf0c/src/emhass/optimization.py#L196-L204
which looks correct, and is consistent with the EMHASS documentation:

In the case of an energy contract where the totality of the PV produced energy is injected into the grid, the grid offtake volume equals = P_load + P_deferrable_sum
Else (only the excess of PV produced energy is injected), the grid offtake equals = P_grid_pos

However, the "grid cost" cost function seems to have the formulas inverted, which is unexpected:
https://github.com/davidusb-geek/emhass/blob/bec6f76f1d7795a1ab8730fb0a61f883e75caf0c/src/emhass/optimization.py#L205-L211
Therefore, I wonder if the objective formulas in "Grid cost" cost function shouldn't be inverted.
I'll propose a pull request for this.

Made a pull request immediately, therefore closing this issue


Add Note 5 user agent variant to Dpdb cache

We found that for one of our devices (a Samsung Note 5) the distortions were not being applied correctly, the reason being that the model number is "SM-N920W8" for Canadian devices.

I signed it!!

I'll accept this change for now, but keep in mind that DPDB is autogenerated from an internal database so at some point in the future, your change may be reverted.


Invalid keywords section in central docs home page

The keywords section in [1] does not contain all the keywords and also the existing keywords [2] gives 500 error.

[1] https://docs.central.ballerina.io/
[2] https://docs.central.ballerina.io/keywords/slalpha3/readonly

This issue is fixed with this PR: https://github.com/ballerina-platform/ballerina-dev-tools/pull/90
Hence closing the issue.


Fixed a typo in mixed_sp_wrr_traffic_test.go

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1232

Relevant Lines:
2159

💛  - Coveralls


Authentication isn't working

generated header
 {'Authorization': 'Basic REDACTED HASHED API KEY', 'Content-Type': 'application/json', 'Accept': '*/*', 'User-Agent': 'togglore'}

generated url
https://toggl.com/reports/api/v2/details?workspace_id=1&since=2019-11-25&until=2019-11-25&user_agent=togglore&page=1

Gives error:
 urllib.error.HTTPError: HTTP Error 403: Forbidden

To test my API key, this curl command works
curl -u REDACTED_API_TOKEN:api_token -X GET "https://www.toggl.com/api/v8/time_entries"

In addition to setting the variable API_KEY in the config file, ~/.togglore, the user_id and workspace_id need to be set.
[Authentication]
API_KEY = 90788685b93e0f6d91324c382146c416
[Work Hours]
hours_per_day = 8.2
excluded_days = 2019.12.24
[User Info]
id = SET_ME
workspace = SET_ME
I was able to retrieve my user_id and workspace_id with this curl command:
 curl -u API_KEY_REDACTED:api_token -X GET https://www.toggl.com/api/v8/me


Add endpoints for Ethereum RPC info

Endpoints

/about/ethereum-rpc/
/about/ethereum-tracing-rpc/

Result
{
    "version": "EthereumJS TestRPC/v2.13.2/ethereum-js",
    "block_number": 70,
    "chain_id": 1337,
    "chain": "GANACHE",
    "syncing": false
}

Coverage increased (+0.02%) to 88.589% when pulling 09305db84515c8b96b4dd8da55c6a1deb72b8b3f on add-rpc-endpoint into b3659c51e3e9d82fb5d4063ba63fee71045a9655 on master.

Because the transaction service has 2 nodes configured, one for tracing and other normal. You can use the tracing node for both, though


added static  class for attachmentpoints

Neat 👍 Now we just need #180 and a few others 😄

Oh, for SpecialAttachmentPoints the "," needs to be changed to " "


Excellent work!

Hi,
What tool was used to draw Figure 1 in paper(ie. images/result.png ) ?
Was it drawn by Excel ?

we use matplotlib


Sarah's edits links aren't being generated properly

Links for that section are being generated under /sarahedits/ but the pages actually exist under /sarah/  (I can navigate there manually to each of the pages)
I know eventually we won't need this set at all because Sarah's edits will become THE PAGES (and then be edited to your taste), but it at least helps to have them linked and accessible for previewing. Please and thank you!

Awesome, Ill take a look and get that fixed asap,

That should be the proper route inplace, pushed and merged the fix buddy, 😊

So I did manage to fix this, however forgot to close this issue down, 🙈
Just doing a bit of house keeping, dont mind me =)


View Position math + undo support

recently wrote a tool to do similar myself before being pointed to your tool only to find it was already done, very glad to see there's a gui option (since mine uses a predefined csv)
here are a few fixes from my experience writing my own for your tool:

noticed that the view position scaling here would not work for avatars that do not come in scale 1 1 1 (for example the izanagiroo collie at scale 1 1 1 is 81m tall)
added registering creation of each clone to the undo history so a quick ctrl+z can be used

Thanks for the PR. I will deploy it at some point however now that I use the "scale goes brr" mod I no longer have need for this plugin

Thanks for the PR!


[SPARK-42967][CORE][3.2][3.3][3.4] Fix SparkListenerTaskStart.stageAttemptId when a task is started after the stage is cancelled

What changes were proposed in this pull request?
The PR fixes a bug that SparkListenerTaskStart can have stageAttemptId = -1 when a task is launched after the stage is cancelled. Actually, we should use the information within Task to update the stageAttemptId field.
Why are the changes needed?
-1 is not a legal stageAttemptId value, thus it can lead to unexpected problem if a subscriber try to parse the stage information from the SparkListenerTaskStart event.
Does this PR introduce any user-facing change?
No, it's a bugfix.
How was this patch tested?
Manually verified.

Merging to master/3.4/3.3/3.2


Bus message listeners

This PR adds an ability to add a Consumer of envelopes of messages posted to a Bus. This is needed to avoid having a BusFilter which isn't filtering but simply consuming a posted message.
Other notable changes:

Improved method naming avoiding get prefixes.
Improved obtaining of generated events in BlackBoxBoundedContext. Previously, they were loaded from EventStore, now they are collected using Bus message listeners.

@armiol, PTAL.


Implement GitHub PR Issue Checker Workflow

Fixes: #900
Usage:

When a pull request is created or edited, the workflow automatically triggers.

The workflow checks for:

A non-empty PR description.
A valid issue reference format (Fixes #<issue-number> or Fixes #NEW).

If either check fails, the PR will be marked as failing, and the author will receive clear error messages in the PR comments.
If all checks pass, a confirmation message will indicate that the PR is valid.

Note: Kindly add all the labels added to the issue, to this PR!

i have updated the changes! merge it! @ajay-dhangar

hey @ajay-dhangar in the issue #900 .You added level2 label.
But now in the PR you added level1! please don't do that

Add hacktoberfest-accepted label! @ajay-dhangar


Correct docker network issue in docker-compose.yaml

Updated Docker Compose file to fix an error which was preventing the docker container from seeing other containers and local llm's through ollama and LMstudio on linux.
With the old variation docker was creating a network specifically for this container which did not have access to localhost or other containers. This change forces the container to use the 'host' docker network allowing access to the bare metal llms.
If this conflicts with the way docker works on Windows or Mac then maybe comment it out and add a comment to uncomment if having the above noted issues???
Please note, this is my first ever pull request. If i've gotten something wrong please let me know so i can fix it for next time.

this is one way to fix it.. but yes you cannot change the port now.. and if the host system is using that port then this will not work.
try to modify the command to accept arguments command: pnpm run dockerstart
you might need to check the package.json file to see the full command of dockerstart
if this is resolved I will merge it happily

I would agree that switching to host based networking is definitely not the best practice here, as it would introduce too many new deployment limitations.


GTM Support

We recently added a GTM integration, we should make it easy for someone to use GTM or GA

Closing, not going to get into analytics in this project. Going to submit a PR that removes what we do have already in.


Fixing broken badge

Hey @bnb, just fixing a badge.

Good catch. Thanks!

No worries, happy to help out!


Create logical files for text files without scanning source files

There are scenarios when text files need to be built i.e. copied from the local git repository to datasets during a build.  However the DBB Dependency Scanner should not scan these files as it often does not understand the syntax and either is a waste of process time or in some cases causes exceptions to be thrown or poor performance.
Instead zAppBuild should just create a default logical file to be used during impact analysis.

One solution would be:

Create a new build property.  It can go in the text.properties file:

#
# list of text file types to create logical files for without scanning source
textFileList=**/*.pkg,**/*.ctl, **/*.pln

In ImpactUtilities.groovy at https://github.com/IBM/dbb-zappbuild/blob/main/utilities/ImpactUtilities.groovy#L805-L807

replace
            def scanner = buildUtils.getScanner(file)
            try {
               def logicalFile = scanner.scan(file, props.workspace)

with
            try {
                LogicalFile logicalFile = null
                // create a default logical file for text source file
                List<PathMatcher> textFileMatchers = createPathMatcherPattern(props.textFileList)
                if (matches(file, textFileMatchers)) {
                    if (props.verbose) println "*** Creating default TEXT logical file"
                    String lname = CopyToPDS.createMemberName(file)
                    logicalFile = new LogicalFile(lname, file, "TEXT", false, false, false)
                }
                // else scan the source file
                else {
                    def scanner = buildUtils.getScanner(file)
                    logicalFile = scanner.scan(file, props.workspace)
                }

Delivered in zAppBuild Release 3.3.0


neonDESKTOP 1058 port conflict?

@granth7: Is seeing neonDESKTOP fail on start up due to a permissions problem with port 1058.  The desktop is trying to listen here to expose the API neon-cli uses to communicate with the desktop.
This has been working fine on his machine until today and is still working OK on my machine,  Grant is on a Windows Insider release though, so perhaps MSFT did some kind of sneaky update.
We'll have to investigate this.  Perhaps we'll need to choose different ports.

To add to this, trying to bind to port 1058 in a standalone ConsoleApp gives:
System.Net.Sockets.SocketException: 'An attempt was made to access a socket in a way forbidden by its access permissions.'
I tried changing the DesktopServicePort to a random port, which allowed neonDesktop to start but when trying to connect to a cluster I get:
Neon.Common.ExecuteException: 'Unable to connect to the server: dial tcp <IP_ADDRESS>:6443: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.

Grant was able to fix this by reinstalling Windows.

I'm beginning to wonder if this is problem with the .NET WebListener class which wraps the Windows HTTP.SYS API.  Here's the thought:

HTTP.SYS allows multiple processes to listen for web requests on the same port (as long as the path prefixes are different).
What if the neonDESKTOP crashed or quit in a way that Windows didn't stop listening on the path.
Then, the next time neonDESKTOP starts, it would essentially conflict with itself.
Perhaps this problem lives across system reboots because Windows is actually hibernating and not actually rebooting.

This seems unlikely though.  The way to verify this the next time this happens would be to hold the SHIFT key down while restarting Windows the next time to force a full reboot.
That being said, I kind of doubt this is the case.  Windows really must ensure that resources like this are released when a process terminates.  neonDESKTOP is known to get into a bad state when it encounters problems.  So it's likely this will be addressed after we do a cleanup pass.

I believe this is probably happening due to a neonDESKTOP bug.  I stopped working on the desktop Feb 2019 when it was kind of working or working enough to be somewhat usable.  There were some race conditions at the time which were generally pretty rare when cluster connections were healthy, but caused failures when the cluster was disconnected (e.g. due to a VPN being down).
I believe we're seeing a race condition here.  All of this needs to be cleaned up.  We're going to track that in the original neonDESKTOP issue: #433 and close this.


[TB] Typography Font Settings - WCAG 4.1.2: Ensures every ARIA input field has an accessible name (#primaryFontSelect)

Problem/Concern
When I run Accessibility Insights for Web on the Typography Font Settings page, I get this issue for each select element:
Issue
Ensures every ARIA input field has an accessible name (aria-input-field-name)
Target application
A11y ThemeBuilder
Element path
#primaryFontSelect
Snippet
<div tabindex="0" role="combobox" aria-controls=":r4:" aria-expanded="false" aria-haspopup="listbox" aria-labelledby="primaryFontSelect" id="primaryFontSelect" class="MuiSelect-select MuiSelect-outlined MuiInputBase-input MuiOutlinedInput-input css-1xd5...
How to fix
Fix any of the following:
  aria-label attribute does not exist or is empty
  aria-labelledby attribute does not exist, references elements that do not exist or references elements that are empty
  Element has no title attribute

Environment
Microsoft Edge version <IP_ADDRESS>
This accessibility issue was found using Accessibility Insights for Web 2.42.0 (axe-core 4.8.4), a tool that helps find and fix accessibility issues. Get more information & download this tool at http://aka.ms/AccessibilityInsights.
Screenshot

Proposed Solution
Ensure all select fields are properly labeled.
Reference: Material UI React - Select - Accessibility

I am now covered under my company's CCLA, so I can take this on if needed - this seems like a very small fix.

I am now covered under my company's CCLA, so I can take this on if needed - this seems like a very small fix.

If you can fix it, that would be great!  In this case, the label and the field are separated by text.  What is the best way to do this with Aria attributes?  I'm guessing if we label it incorrectly that a screen reader might read stuff out of order?

Probably using either Material UI's labelId property or aria-labelledby - as is, a screen reader will read the field as unlabeled, which causes confusion.

I was able to fix this issue locally by adding id="primaryFontLabel" to the
 element, then adding labelId="primaryFontLabel" to the primary select.
This ensures that the text label is properly associated with the select field.

Just made a commit on my fork - can you please assign me this issue so I can mark it as in progress and raise a PR?

PR merged - closing


GetInteriorVehicleData(subscribe = false)

@yang1070, please, advise on expected behavior for the following case
(in light of the proposal 0178-GetInteriorVehicleData)
Pre-conditions:
a. RC app is NOT subscribed for module_1
Steps:

RC app sends GetInteriorVehicleData (module_1, subscribe=false) request to SDL

Expected:
2.  ?

@dboltovskyi, @GetmanetsIrina please be aware of the issue

@KhrystynaDubovyk @dboltovskyi, @GetmanetsIrina
Expected:

SDL knows the app is not subscribed to module_1 before.
In general, SDL shall put parameter isSubscribed=status known to SDL in the response to apps if the request has parameter subscribe regardless of the value.
in case the cached data of module_1 is available ( other apps has subscribed to module_1), SDL gives a success response to the app with isSubscribed=false and cached module data and may possibly  have the info describes the situation, something like the app has not subscribed to the module_1 before.
in case the cached data of module_1 is not available, cut off parameter subscribe=false, and forward the GetInteriorVehicleData (module_1) to the vehicle HMI. When the response comes back from HMI, SDL add paramrter isSubscribed=false in the response to the mobile app. (info might get similar updated if it is empty from the HMI)

related to this question, if the RC app is subscribed for module_1 before, and sends a GetInteriorVehicleData (module_1, subscribe=true) again, the response shall have isSubscribed=true


Issue with TopicMessageQuery

Description
When creating multiple defined TopicMessageQuery, TopicMessageQuery doesn't return a response.
Steps to reproduce
const client = Client.forTestnet();
client.setOperator(
    "0.0.1548173",
    "302e020100300506032b657004220420e749aa65835ce90cab1cfb7f0fa11038e867e74946abca993f543cf9509c8edc"
);

var topics = [
    "0.0.2276269",
    "0.0.2884610",
    "0.0.2884609",
    "0.0.2895511",
    "0.0.2895510"
];
for (let index = 0; index < topics.length; index++) {
    const topicId = topics[index];
    const handle = new TopicMessageQuery()
        .setTopicId(topicId)
        .setStartTime(0)
        .subscribe(client, (e) => {
            console.log("Topic " + topicId, e);
        }, (message) => {
            console.log("Topic " + topicId);
        });
    await wait(1); // 1sec
    console.log("new TopicMessageQuery " + topicId);
}

const topicId = "0.0.2895551";

await wait(1);
new TopicMessageQuery()
    .setTopicId(topicId)
    .setStartTime(0)
    .subscribe(client, (e) => {
        console.log("Topic 0.0.2895551", e);
    }, (message) => {
        console.log("Topic 0.0.2895551", Buffer.from(message.contents).toString());
    });
console.log("new TopicMessageQuery " + topicId);

await wait(1);
await new TopicMessageSubmitTransaction({
    topicId: topicId,
    message: "message 1",
}).execute(client);

await wait(10);
await new TopicMessageSubmitTransaction({
    topicId: topicId,
    message: "message 2",
}).execute(client);

await wait(60)

Additional context
No response
Hedera network
testnet
Version
v2.1.1
Operating system
Windows

The mirror node limits you to 5 maxConcurrentCallsPerConnection. And if you're using our mainnet-public mirror node it's limited to 5 connections per IP globally.

Closing since this does not look like an SDK issue.


Guide for using laravel-mix as a standalone

Are you planning on adding guides for using laravel-mix as a standalone replacement for webpack, without requiring you to download all of laravel?
I'm asking because I'm not a PHP dev but I'd really like to use laravel-mix in my front-end stack, and I'm struggling to find documentation for how to efficiently achieve that.

You can check this repo: https://github.com/laravelista/laravel-mix-without-laravel

There's a quick guide here: https://github.com/JeffreyWay/laravel-mix/blob/master/docs/installation.md

@JeffreyWay  I tried following the instructions, but i get an error upon running npm run dev.

No configuration file found and no output filename configured via CLI option.
A configuration file could be named 'webpack.config.js' in the current directory.
Use --help to display the CLI options.

I'm on windows 7 (due to work, I'm used to ubuntu, so this might just require a trivial solution)

@Olian04 I had the same error when upgrading from v0.12.1. There is no more webpack.config.js file produced when following the stand-alone installation instructions since it's not meant to be editable, so if you try to run the previous NPM scripts it errors over a config file not existing. This worked for me: update your scripts in package.json to the following:
"scripts": {
    "dev": "NODE_ENV=development webpack --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "watch": "NODE_ENV=development webpack --watch --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "hot": "NODE_ENV=development webpack-dev-server --inline --hot --config=node_modules/laravel-mix/setup/webpack.config.js",
    "production": "NODE_ENV=production webpack --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js"
  }

When the script runs it will refer to the webpack.config.js file in node_modules/laravel-mix/setup.

It's really the same no matter the backend. I use it in several projects that don't use laravel. You just have to make sure your paths are set correctly. Actually I usually add a resources folder and mimic the asset structure of Laravel.

Hi ! @JeffreyWay
Maybe there is a slight problem in the documentation: https://github.com/JeffreyWay/laravel-mix/blob/master/docs/installation.md

When I run node_modules/.bin/webpack, I got this:

So, I can only add the following NPM scripts to my package.json file, and run npm run dev.
Thanks a lot !


Fix unwanted CLI caching

Description
Fixes #176
Fixes the unwanted CLI caching of some assets like project and example templates. Now the CLI will pull the most current assets for the user.
This was accomplished by utilizing the force option on the gittar fetch method https://www.npmjs.com/package/gittar#optionsforce. This option forces gittar to always first try to pull the assets over https ensuring the most current version of them. If there is no network connection or an error, a cache lookup will follow. This also still allows for offline use.
Tested
A test branch was created and linked to the cache of this version of the CLI. The test branch was used to update example and project templates for testing. Used this version of the CLI to successfully create new template and example projects that reflect the most current updated assets.

Looks nice! Can you squash this one when merging please?


SQL Notebook: keep intermediate results

Is your feature request related to a problem? Please describe.
With SQL notebooks, in contrast with e.g. Python notebooks, I must write my intermediate results to storage (temp table, CSV) when I want to act on them in subsequent cells
Describe the solution or feature you'd like
I'd like a feature that allows me to store intermediate results in some analogue of variables as I can with Python notebooks

I would love to see this feature as well. It would be amazing to help experiment with data letting variables / temp tables be shared across cells much like one could do with Python + Pandas. Having markdown in-between sql statements is also helpful to explain complex sql statements.


GD-118: False-negative for tests with float

The used Godot version: v3.4.beta2.official
OS including version: Linux 5.12.19-1-MANJARO x86_64 GNU/Linux
Describe the bug

Steps to Reproduce

Install gdUnit3 addon
Create simple method with float value manipulation
Create test for it with assert_float().is_equal()
Run test

Minimal reproduction project: project.zip

Thanks for reporting.
v3.4.beta2.official is not officially supported yet, but I will have a look at it.

This is not a real bug this is the nature of floats are stored/calculated.
	prints ("%.16f" % 2.2)
	prints ("%.16f" % (6.6/3.0))

Results in:
2.2000000000000002
2.1999999999999997

Godot provides a function to compare float by a approx is_equal_approx
I will add a better is_equal on AssertFloat to handle approximately equal to each other.

Added missing documentation
https://github.com/MikeSchulze/gdUnit3/wiki/GdUnitFloatAssert#is_equal_approx


build error: webpack missing

When building on ubuntu 20.04 it breaks with an error that webpack is missing:
npm run build

><EMAIL_ADDRESS>build /media/Daten/Forks/seabass2
> webpack

sh: 1: webpack: not found
npm ERR! code ELIFECYCLE
npm ERR! syscall spawn
npm ERR! file sh
npm ERR! errno ENOENT
npm ERR<EMAIL_ADDRESS>build: `webpack`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ubuntu/.npm/_logs/2020-05-22T18_52_34_405Z-debug.log

2020-05-22T18_52_34_405Z-debug.log
After installing webpack, it throws an error copy-webpack-pluginis missing.
pm run build

><EMAIL_ADDRESS>build /media/Daten/Forks/seabass2
> webpack

/usr/share/nodejs/webpack/node_modules/webpack-cli/bin/cli.js:93
				throw err;
				^

Error: Cannot find module 'copy-webpack-plugin'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:636:15)
    at Function.Module._load (internal/modules/cjs/loader.js:562:25)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (/usr/share/nodejs/webpack/node_modules/v8-compile-cache/v8-compile-cache.js:161:20)
    at Object.<anonymous> (/media/Daten/Forks/seabass2/webpack.config.js:2:20)
    at Module._compile (/usr/share/nodejs/webpack/node_modules/v8-compile-cache/v8-compile-cache.js:194:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (/usr/share/nodejs/webpack/node_modules/v8-compile-cache/v8-compile-cache.js:161:20)
    at WEBPACK_OPTIONS (/usr/share/nodejs/webpack/node_modules/webpack-cli/bin/utils/convert-argv.js:114:13)
    at requireConfig (/usr/share/nodejs/webpack/node_modules/webpack-cli/bin/utils/convert-argv.js:116:6)
    at /usr/share/nodejs/webpack/node_modules/webpack-cli/bin/utils/convert-argv.js:123:17
    at Array.forEach (<anonymous>)
    at module.exports (/usr/share/nodejs/webpack/node_modules/webpack-cli/bin/utils/convert-argv.js:121:15)
    at yargs.parse (/usr/share/nodejs/webpack/node_modules/webpack-cli/bin/cli.js:71:45)
    at Object.parse (/usr/share/nodejs/yargs/yargs.js:620:18)
    at /usr/share/nodejs/webpack/node_modules/webpack-cli/bin/cli.js:49:8
    at Object.<anonymous> (/usr/share/nodejs/webpack/node_modules/webpack-cli/bin/cli.js:366:3)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Module.require (internal/modules/cjs/loader.js:692:17)
    at require (internal/modules/cjs/helpers.js:25:18)
    at Object.<anonymous> (/usr/share/nodejs/webpack/bin/webpack.js:156:2)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `webpack`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ubuntu/.npm/_logs/2020-05-22T18_59_09_144Z-debug.log

2020-05-22T18_59_09_144Z-debug.log
So I am not able to properly build the app (trying for UT).
Well, clickable builds the app, but it does not work (= show files).

Just to be sure: you've run npm install command (before executing npm run build) in the seabass2 directory?
Build steps:

npm install (in the seabass2 directory). At this step a new directory called seabass2/node_modules will be created. All the build dependencies are installed there.
npm run build
clickable

Both webpack and copy-webpack-plugin packages should be installed for you automatically at the first step

I did npm install but there seems to be no node_submodules. So maybe something went wrong at that step already.

Seems like something went wrong with npm install. I would suggest to try again and check if it creates node_modules directory.

Tried it again. Now the node_submodules is there. I guess that was because webpack was not installed before?
Also npm install brings up some warnings. But i guess that does not harm.
pm install

><EMAIL_ADDRESS>postinstall /media/Daten/Forks/seabass2/node_modules/babel-polyfill/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /media/Daten/Forks/seabass2/node_modules/babel-runtime/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

><EMAIL_ADDRESS>postinstall /media/Daten/Forks/seabass2/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 1172 packages from 499 contributors and audited 1174 packages in 36.307s

42 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

OK. Now build works too. And clickable provides a working app. :+1:

Looks fine now! The warnings are OK.
Shouldn't be related to webpack, as it is just an NPM package itself.
It just happens to the first package that is required by npm run build script, so Node throws a error about webpack.

OK. Now build works too. And clickable provides a working app.

Great! I'm closing the issue then!

Yes. Thanks. Maybe you can find somehing in the logs that might help to improve the script to avoid my issue for other.


API Method Inputs with ":" in them

First check

[x ] I added a very descriptive title to this issue.
[ x] I used the GitHub search to find a similar issue and didn't find it.
[ x] I searched the FastAPI documentation, with the integrated search.
[ x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[ x] I already read and followed all the tutorial in the docs and didn't find an answer.
[ x] I already checked if it is not related to FastAPI but to Pydantic.
[ x] I already checked if it is not related to FastAPI but to Swagger UI.
[ x] I already checked if it is not related to FastAPI but to ReDoc.
[ ] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

I am trying to mimic an API's framework, and it requires some parameters to have a : in the name of the parameter:
Example:
env:sr
I can't name a parameter in the function with a : below, so I was wondering if there a way to specific the parameter name on the @app.post, get... etc...
Thank you
Example code below:
from fastapi import FastAPI

app = FastAPI()

@app.post("/", map_name={'env_sr' : 'env:sr'})
def read_root(env_sr:str):
    return {"Hello": env_sr}

I'm not 100% sure what you are trying to do. So you want to pass an argument with the name env:sr or is it the value?
The dead_root will return whatever param you send to it and it's expecting it to be a string so you can use ":" in the value.

I need to call my parameter env:sr , and I want the parameter name to show in the swagger doc as env:sr, so the user knows that this is the parameter name to pass.

@achapkowski you can try alias parameter of Query

@achapkowski yeah, tested it:
from fastapi import FastAPI, Query

app = FastAPI()

@app.get("/")
async def get_main(env_sr: str = Query(..., alias="env:sr")):
    return {"Hello": env_sr}

Works as expected: you can call the parameter like /?env:sr=abc and in docs it is shown as env:sr

it does work, thank you all!

Thanks for the help here @insomnes and @valentin994 ! 👏  🙇
Thanks for reporting back and closing the issue @achapkowski  👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


add datadog to list of docker monitoring tools

add datadog to list of docker monitoring tools

Thank you! Nice one.


Still can not modify and save an old or make a new macro

I still have the same issues.  I can not make a new macro and save it because it does not save and if I try toi edit an existing one and save it just will not work

Hi Despised,
Can you please try this.

exit WoW
go to the WTF/Account/YourAccountName/SavedVariables folder within WoW in Explorer or Finder
Rename the GSE.lua file to GSse.lua.old
restart WoW

Can you then see if you can import and edit.  The reason for the rename is that it will back up your existing macros without loosing them as when you start WoW it will create this file from scratch.
If you are able to import and save it means you had a corrupt macro within your old set and this has prevented the actions you are trying to complete.
(You could also before you do this export the macros you wish to keep to notepad first and then reimport them).
Of note is that old macros created before 8.0 will in most cases be incompatible and need significant changes and modifications to run.

What have you tried to resolve this?  The amount of information you have given me is “I have a broken lawnmower.  What is wrong with it?”
The challenge for me is that every other lawnmower appears to be working except yours so I just can’t go look at another one for what is wrong.
TimothyLuke

On 28 Jul 2018, at 10:54 pm, despised7<EMAIL_ADDRESS>wrote:
Still wont work
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Static files in sub-directories return 404

I'm trying to move my entire website into my Java project (as static files) but I can only get files inside /public and /public/dir. Whenever I try to access a file deeper in the filesystem, it returns 404.
i.e:

localhost:port/index.html - OK
localhost:port/js/main.js - OK
localhost:port/js/jquery/jquery.js - 404

I'm using the latest version of Spark (via Maven), so what am I doing wrong? (I know that I can use an external file source but since the users have access to the server, I'd prefer that everything will be inside the JAR file so they won't mess things up).

Can you create a minimal broken example project?
I have deeper structures in most my projects without any issues.

I will try to create something later today, but do you have a repository with a deep structure that you can show me so I can maybe find what's wrong with my file system?

Update: If I open a file that wasn't served (404), then add a new line and saves the file - it can later be found when I run the program! Any ideas for why's that? I don't want to do this for hundreds of JS files I got in there...

https://github.com/tipsy/OmriSoudry-example

I have the same issue, after adding a new line it's possible to access them. Like @OmriSoudry I can't add a new line to all of that css and js manually every time a compile a web project. Any workaround for this?
Maybe the css and js minification has something to do with it?

@quezadaesteban can you create a minimal broken example project?

Sure, @tipsy here it is SpaWithSpark.

Ok It already worked. Building the project is required for it to work. Running it updates java code but not static files.
Thanks for the quick reply.
Hope this helps.


TDS starting time

Currently the beggining of TDS is hard coded. This has to be changed rapidily

Moreover, the file for TDS should be in parameters as well. Not as input()

Done in #79


Produce full vtables in CPAOT

I believe it's inappropriate to try to "optimize" the VTable
as it's not us, it's the CoreCLR runtime that actually emits the
vtable based on type metadata. In contrast to full AOT, we're not
required to produce the complete code graph and so we currently
don't need the detailed VirtualMethodUse tracking that is required
for lazy vtable construction to work properly.
This change fixes CPAOT build of System.Private.CoreLib, hopefully
the last remaining issue in CoreCLR framework and ASP.NET assembly
builds.
Thanks
Tomas

This change fixes CPAOT build of System.Private.CoreLib,

What was the problem that this was hitting before? Isn't this change just masking some other bug?

Hmm, I can take a closer look what exactly is happening there. The primary problem in S.P.Corelib is that, during compilation of

[S.P.CoreLib]System.Runtime.CompilerServices.YieldAwaitable+YieldAwaiter.System.Runtime.CompilerServices.IStateMachineBoxAwareAwaiter.AwaitUnsafeOnCompleted(IAsyncStateMachineBox)

JIT calls getMethodVTableOffset for the method

[S.P.CoreLib]System.Threading.SynchronizationContext.Post(SendOrPostCallback,object)

and that fails with the lazy VTable as the proper VirtualMethodUse hasn't been called before. Not calling VirtualMethodUse may be a current CPAOT deficiency. Right now it's unclear to me whether we need this tracking as, unlike full AOT, we don't necessarily need to JIT everything, and whether it's something that should work all by itself or whether, by bypassing some of the RyuJit code paths in NodeFactory, we may have opted out of default CoreRT logic dealing with this reporting and we must put together an equivalent logic specific to CPAOT.

The implementation of getMethodVTableOffset looks CoreRT specific - it calls EETypeNode.GetVTableOffset.


Add ganache npm run script, plus documentation on how to use it.

🚀 Description

Added ganache npm run script
Added documentation about how to utilize Ganache

may not be necessary once https://github.com/OpenZeppelin/zeppelin-solidity/pull/800 is merged

Closing due to inactivity, also not sure if we should bundle this, since it's more of a truffle console thing.


Default appstore lane name conflicts with appstore action alias

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Fastlane warns that lane appstore conflicts with the appstore alias to deliver. Given this is the default lane name on fastlane init it makes sense to change one of these things.
Complete output when running fastlane, including the stack trace and command used
Name of the lane 'testflight' is already taken by the action named 'testflight'
Name of the lane 'appstore' is already taken by the action named 'appstore'

Environment
fastlane version (run fastlane -v):
1.101.0
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)?
No
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)?
rbenv

@dombarnes Thanks for reaching out about this. So when do you get this error? When you run the lane or on fastlane init? Can you run sudo gem update fastlane? Thanks!

I was getting the error when running my lanes I think. It was more a warning than an error.
Ran fastlane init on another project not using it yet, I get the default lanes as :beta and :appstore
Ran gem update fastlane and everything is up to date.
11:39:41]: -------------------------------------------------
[11:39:41]: --- Step: Verifying required fastlane version ---
[11:39:41]: -------------------------------------------------
[11:39:41]: fastlane version valid
[11:39:41]: ------------------------------
[11:39:41]: --- Step: default_platform ---
[11:39:41]: ------------------------------
[11:39:41]: Name of the lane 'appstore' is already taken by the action named 'appstore'
[11:39:41]: Driving the lane 'ios dummy' 🚀

The lane finishes without errors.

I have the same issue. Ran fastlane init and fastlane lanes and got that error.

Great catch, thanks, I'll look into this 👍

I just pushed https://github.com/fastlane/fastlane/pull/5895, which will be in the next release, thanks for letting us know. Please rename the lane to something else for now 👍

More exactly, rename it to release if you want to match the new fastlane's default name.


Licenses don't match

Your LICENSE file is MIT, but you don't use that in package.json: https://github.com/menghif/dodo-SSG/blob/main/package.json#L13.  This should also be MIT

Fixed! 3a4e0fc5d395ecec1280855c6f90ed0b6250997b


Add recon and record variables to Clojure handling

Shown in https://github.com/OpenRefine/OpenRefine/blob/5639f1b2f17303b03026629d763dcb6fef98550b/main/src/com/google/refine/expr/MetaParser.java#L89
We could additionally add recon and record to make it easier to access these GREL variables directly to evaluate with Clojure directly.

This sounds similar to #2523. We should probably be consistent about what variables we make available across languages.
The current list of variables for GREL is here: https://github.com/OpenRefine/OpenRefine/issues/2523#issuecomment-713835727


jpegxl.json: macOS/iOS Safari does not support animated JXL yet

You can check this with the following image using the Safari browser: Link of animated JXL sample
It should be animated like this  but it won't.
References

https://jpegxl.info/test-page/

Confirmed, thanks!


Closes #196

Thanks for the delay in merging this. finally got more time for the puppet modules.
Took an other look an it looks great. will merge as is and add some tests to it later.
Thanks!


Move react/socket to dev dependency

To reduce cyclic dependency between the two, as it is only used in tests

I wasn't aware of #19. No point in merging this if it's just going to be undone soonish.