Add Template workflows for GusHebblewhite/wh40k

To add GitHub Actions workflows to your repository:

create this issue
add a new comment with the content below
replace [repo name] with your repository name in the format owner/repo, e.g. BSData/test.

/template-workflows-pr GusHebblewhite/wh40k

/template-workflows-pr GusHebblewhite/wh40k


config auto cannot generate chain config without valid WebSocket endpoint

Summary of Bug
We auto generate hermes config using this command:
$ hermes config auto --output $HOME/.hermes/config.toml --chain cifer --chain
Then we get this output:
However, configurations for the following chains were not able to be generated: cifer
Version
hermes version 1.8.2, ibc-rs version 1.0.0
Steps to Reproduce
We suspect may we miss some parameters in our Chain config such slip44 and key_algo as you can see in this repo:
https://github.com/cosmos/chain-registry/tree/master/cifer
Acceptance Criteria
In this case, please kindly check if we miss any parameters in our config file that cause Hermes unable to generate Cifer chain config in config.toml

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

I looked into it and the issue was that config auto failed to find a valid WebSocket endpoint.
I have changed the logic so that it nows uses the pull event source by default, which does not rely on WebSocket.

Here is the config file it generates now
[global]
log_level = "info"

[mode.clients]
enabled = true
refresh = true
misbehaviour = true

[mode.connections]
enabled = false

[mode.channels]
enabled = false

[mode.packets]
enabled = true
clear_interval = 100
clear_on_start = true
tx_confirmation = false
auto_register_counterparty_payee = false

[mode.packets.ics20_max_memo_size]
enabled = true
size = 32768

[mode.packets.ics20_max_receiver_size]
enabled = true
size = 2048

[rest]
enabled = false
host = "<IP_ADDRESS>"
port = 3000

[telemetry]
enabled = false
host = "<IP_ADDRESS>"
port = 3001

[telemetry.buckets.latency_submitted]
start = 500
end = 20000
buckets = 10

[telemetry.buckets.latency_confirmed]
start = 1000
end = 30000
buckets = 10

[[chains]]
type = "CosmosSdk"
id = "cifer"
rpc_addr = "https://cif_node.cifer.ai/"
grpc_addr = "http://grpc.cifer.ai/"
rpc_timeout = "10s"
trusted_node = false
account_prefix = "cife"
key_name = ""
key_store_type = "Test"
store_prefix = "ibc"
default_gas = 100000
max_gas = 400000
gas_multiplier = 1.1
max_msg_num = 30
max_tx_size = 180000
max_grpc_decoding_size = 33554432
query_packets_chunk_size = 50
clock_drift = "5s"
max_block_time = "30s"
client_refresh_rate = "1/3"
ccv_consumer_chain = false
memo_prefix = ""
sequential_batch_tx = false

[chains.event_source]
mode = "pull"
interval = "500ms"
max_retries = 4

[chains.trust_threshold]
numerator = 2
denominator = 3

[chains.gas_price]
price = 5.0
denom = "ucif"

[chains.packet_filter]
policy = "allowall"

[chains.packet_filter.min_fees]

[chains.dynamic_gas_price]
enabled = false
multiplier = 1.1
max = 0.6

[chains.address_type]
derivation = "cosmos"

[chains.excluded_sequences]

[tracing_server]
enabled = false
port = 5555

Thank you, the new generated config works.

Thank you, the new generated config works now.


Refs #32800 -- Added _add_new_csrf_cookie() helper function.

This is the last preparatory PR before submitting a fix for ticket #32800.
It adds some more test improvements (e.g. adding a test and strengthening others), and it adds a helper function so that more of the logic for creating a new cookie can go through a single code path. In particular, the fix for the ticket will involve a change to this code path (not calling _mask_cipher_secret() when creating the cookie instead of calling it).

@apollo13 @shaib I'm hoping ticket #32800 can be resolved in time for 4.0, so it can go along with the other CSRF internal API changes (it will be nice to have them all together). Once this PR is reviewed and approved (hopefully with one or both of your help), I can move on to posting a PR for the actual change. 😁

Yeah naming wise I was confused a bit as well. But I do not mind either way as long as it is consistent.
On Fri, Aug 13, 2021, at 15:41, Carlton Gibson wrote:

@.**** commented on this pull request.
In django/middleware/csrf.py
https://github.com/django/django/pull/14761#discussion_r688522784:

@@ -229,7 +231,7 @@ def _get_token(self, request):
request.META['CSRF_COOKIE_NEEDS_UPDATE'] = True
return csrf_token

def _set_token(self, request, response):

def _set_csrf_cookie(self, request, response):
Is this the right name? If I'm using settings.CSRF_USE_SESSIONS then
no cookie gets set.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/14761#pullrequestreview-729628816, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAT5C4JTBE7ZVGXZR3XV6DT4UOIXANCNFSM5B5F5V4A.
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&utm_campaign=notification-email.

Hey @cjerdonek — I had a quick look at this this afternoon. I need a bit longer to think through the test changes entirely. (@apollo13 has said yes but I'd like to make sure I'm happy too.) I will take another look Tuesday. Thanks.

I need a bit longer to think through the test changes entirely. (@apollo13 has said yes but I'd like to make sure I'm happy too.) I will take another look Tuesday. Thanks.

@carltongibson Sounds good, take your time. Also, when reviewing the test changes, you may want to review them one commit at a time. I broke them up into separate changes to make them easier to review.

Thanks @cjerdonek. Don't worry too much about the name question. 😀 @shaib is right. We can adjust it separately if a better idea isn't clear now.
Have a good weekend!


feat: make the project configurable

This has been rebase and all the unrelated modification has been deleted

After those comments, you can consider you have my 👍


[PDI-18683] VFS connection is not created if edited while being created

:white_check_mark: Build finished in 3m 11s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/connections/ui

:exclamation: No tests found!
:information_source:  This is an automatic message


Bug 2087125: Check all batches for upgrade complete

Last batch processing now evaluates all batches before declaring upgrade complete.

/bugzilla refresh

/hold

/cherrypick release-4.11

/cherrypick release-4.10


how does one insert orgmode snippets that contain * ?

Hi again
i was wondering how using yankpad one would insert snippets that have org headers in them (ie ***). in yas its not a problem ie to have a snippet like this
# -*- mode: snippet; require-final-newline: nil -*-
# name: projfile
# key: projfile
# binding: direct-keybinding
# --
** P0XX.$1
*** Aims
*** related papers
*** manuscript
*** work
**** data
**** model
*** results

yet in yankspad due to the fact that the snippets them selves are org mode headers/content this above snippet messes up things.
anyway around this?
best
z

You can try indenting the whole snippet one space (mark the whole snippet and use M-x indent-rigidly). I believe this will work as long as you do not have lines with only one star, because then org-mode will convert it into a list. I should probably look into this...

I'm not quite sure on how to fix this. The snippet could be inserted as raw data, but then the snippet would have to be indented (and the indentation will be inserted too).

hmmm just a wild thought. perhaps the use of orgmode code blocks/ babel can help here?
for example for snippets with problematic characters such as * etc we could put that inside orgmode example blocks
#+BEGIN_EXAMPLE
* snip
#+END_EXAMPLE

would that help at all?
best
Z

It could work, but I think it would require the use of the flet function (or be really cumbersome to code), which is deprecated in Emacs lisp.

Okay, so I've submitted a solution: \* at the beginning of a line will be replaced by \*. So in your example:
\** P0XX.$1
\*** Aims
\*** related papers
\*** manuscript
\*** work
\**** data
\**** model
\*** results

just to be clear this will insert the \* not just * so one still needs to go and delete the \ right?
thx
Z

No, * at the beginning of a line (column 0) should insert * and nothing
else.
On Mon, May 23, 2016, 13:39 zeltak<EMAIL_ADDRESS>wrote:

just to be clear this will insert the * not just * so one still needs to
go and delete the \ right?
thx
Z
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/Kungsgeten/yankpad/issues/7#issuecomment-220957056


PHP SDK

Hi! Thanks for an awesome library — do you have plans for PHP SDK?

Hi, and thanks!
We don't use PHP so we don't have plans to make a client for it.  You can interface to the API directly (docs are https://checkr.github.io/flagr/api_docs) and if you write a client library we'll happily link it!


Add header option to Webget

I have been using a lot of API websites but some of them requires a header (for secret keys)
The example script could be:

~webget "http://google.com" header:MyRandomHeader save:google

If possible, I suggest the arguement is a list for multiple headers:

~webget "http://google.com" header:li@MyHeader|Accept save:google

By researching a bit further on headers and the way it works on Denizen. We seem to be using HttpURLConnection and its headers are called with .setRequestProperty which takes 2 arguements.
So I think the example should be:

~webget "http://google.com" header:li@Username,MyUsername|Status,Accept|Type,Latest save:google


Range Card Tools should include Visibility Analysis

Part of the workflow for developing a range card is identifying the "dead space," area that is not visible to the firing position.
Currently, the user can sketch out the dead space, but it would be much easier and accurate to provide a tool that does that automatically.

@joebayles - Take a look at @mfunk comments and provide feedback.

@mfunk, that's exactly what I was talking about. Thanks.
@jrweakland I'll add it to the Asana task for Range Card template conversion to Pro.


rdap.lacnic.net is still agressively rate limiting

While LACNIC RDAP says "Rate Limit is maxed at 10 queries per 1 minutes.", I am getting far fewer than 10 queries in. Tested from several networks. I am testing with pure wget at this point (no ipwhois code). My very first query from a dedicated VPS IP resulted in the rate limit error.
If anyone has the spare cycles, I could use help testing this and also figuring out how long the temp ban lasts.

I'm having this same issue. Whois has approach to work around this issue, where you consult ip addresses in batch. Is there a way to do the same with ipwhois?

The only RDAP batch lookups I know of are per user request e.g., LACNIC: http://www.lacnic.net/documents/10834/1535064/solicitud-rdap-beta-en.pdf
I can't do this at the library level obviously, each user/org would need to request it.
The real problem still lies on their rate limiting bug.

I reached out to lacnic, will report back.

No response yet, trying some additional contacts.

I got a response and am working with LACNIC to help troubleshoot the issue.

Lacnic responded that they fixed the problem. Everything appears to be working again. I will leave this open while I run tests for 0.13.0.


Delete lib/src/js/debug folder in release mode.

Moved from https://github.com/dart-lang/polymer-dart/issues/405. Doing this will cut output size in half.

Will that include polymer_elements also?

No, this is just the debug sources for polymer. It only affects the amount of code deployed to the server as well, not the amount delivered to the user.

Is there anyway to build and deploy only the files needed by the user. Noticed there are test files being deployed.
/build/web/packages/polymer_elements/src/prism/tests/languages/yaml/key_feature.test
/build/web/packages/polymer_elements/src/prism/tests/languages/yaml/null_feature.test
/build/web/packages/polymer_elements/src/prism/tests/languages/yaml/number_feature.test
/build/web/packages/polymer_elements/src/prism/tests/languages/yaml/scalar_feature.test
/build/web/packages/polymer_elements/src/prism/tests/languages/yaml/string_feature.test
/build/web/packages/polymer_elements/src/prism/tests/languages/yaml/tag_feature.test
...

Feel free to file a bug on polymer_elements, deletion patterns can be added to this file so the next time the elements are generated it won't include those files.


GitHub Workflows & Templates

Need to create different GitHub workflows and templates for this repo.

https://github.com/marketplace/actions/vuepress-deploy


api/configuration not found

Describe The Bug:
I think sonnen GmbH changed the software on the batteries in the last two weeks, because now the api/configuration web path is not available anymore. Additionally the port 8080 which worked since 2016 on my eco8 now redirects to a 404 page.
the url with /api/v1/status still do work.
I also contacted the support from sonnen, because my password also does not work anymore ;-/
To Reproduce:
default configuration with http://ip-to-my-sonnenbatterie, start homebridge
Expected behavior:
runs successfully
Logs:
FetchError: request to http://x.x.x.x/api/configuration failed, reason: connect EHOSTUNREACH x.x.x.x:80
    at ClientRequest.<anonymous> (/homebridge/node_modules/homebridge-sonnenbatterie/node_modules/node-fetch/lib/index.js:1491:11)
    at ClientRequest.emit (node:events:527:28)
    at Socket.socketErrorListener (node:_http_client:454:9)
    at Socket.emit (node:events:527:28)
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  type: 'system',
  errno: 'EHOSTUNREACH',
  code: 'EHOSTUNREACH'
}

Plugin Config:
{
    "bridge": {
        "name": "iHomebridge",
        "username": "xxxx",
        "port": xxxx,
        "pin": "xxxx"
    },
    "platforms": [
        {
            "name": "Config",
            "port": xxxx,
            "platform": "config"
        },
        {
            "name": "sonnenBatterie",
            "url": "http://x.x.x.x",
            "platform": "sonnenBatterie"
        }
    ]
}

Environment:

Plugin Version: 1.0.1
Homebridge Version: 1.5.1
Node.js Version: 16.16.0
NPM Version: 8.11.0
Operating System: Raspbian GNU/Linux 11 (bullseye)

Update: after a firmware update the new URL is http://<ip-sonnenbatterie>/api/v2/status.
Additional information

API documentation available via http://<your-local-sonnenbatterie-ip>/api/doc.html

Necessary API call is http://<your-local-sonnenbatterie-ip>/api/v2/status which doesn't need an API token to get the data.
Example data result:
{
  "Apparent_output": 226,
  "BackupBuffer": "0",
  "BatteryCharging": false,
  "BatteryDischarging": false,
  "Consumption_Avg": 2116,
  "Consumption_W": 2112,
  "Fac": 50.00199890136719,
  "FlowConsumptionBattery": false,
  "FlowConsumptionGrid": true,
  "FlowConsumptionProduction": true,
  "FlowGridBattery": false,
  "FlowProductionBattery": false,
  "FlowProductionGrid": false,
  "GridFeedIn_W": -2113,
  "IsSystemInstalled": 1,
  "OperatingMode": "2",
  "Pac_total_W": -5,
  "Production_W": 4,
  "RSOC": 4,
  "RemainingCapacity_Wh": 163,
  "Sac1": 75,
  "Sac2": 75,
  "Sac3": 75,
  "SystemStatus": "OnGrid",
  "Timestamp": "2022-11-03 20:15:29",
  "USOC": 0,
  "Uac": 231,
  "Ubat": 48,
  "dischargeNotAllowed": false,
  "generator_autostart": false
}

Do you think you can get this merged in?  The old interface v1 still seems to be working.  Not sure how long Sonnen will leave this available.

I updated my pull request to the new api.


Restricting kibana logout

Hello Team,
In one of our applications, we are opening up kibanaui (same domain as main app) in a new tab, authenticated using JWT. The kibana dashboard is loading properly.
However, once we do a kibana logout, it is observed that:

it clears all the localStorage, as mentioned in https://github.com/opendistro-for-elasticsearch/security-kibana-plugin/issues/118#issuecomment-614897218
and the main application (from where kibanaui is opened from) is also logged out !

Another case is - if we try to do a logout from main app, the kibanaui session is still maintained, and hence the kibana dashboard is still visible even after browser refresh.
Summarizing:

If we do a kibana app logout - only kibana should be logged out, not the main app.
If we do a main app logout - both (main+kibana) should be logged out.

I'm not sure as what is causing this issue OR how to fix this. It'd be great if you could resolve the issue in order to proceed with this.
Please do let me know, if you need any other information in this regard.
Thanks,

We are doing some "spring cleaning in the fall", and to make sure we focus our energies on the right issues and we get a better picture of the state of the repo, we are closing all issues that we are carrying over from the ODFE era (ODFE is no longer supported/maintained, see post here).
If you believe this issue should still be considered for current versions of OpenSearch, apologies! Please let us know by re-opening it.
Thanks!


Removed post.page from Content API v2

refs #10335
[ci skip]

@ErisDS This PR is not fully finished, but i went over the usages of post.page and this.page in the code base to eliminate it completely to be able to remove .page from the Content API v2.
Maybe if you have some minutes, could you pls double check if i have forgotten a place or/and if you see any problems with this approach? This solution was for me right now the only logical conclusion 😁If you have other ideas, pls share them
We should probably also remove .page it from the pages endpoint? Otherwise it's inconsistent. Would do in a separate PR.


AKS Support

I would like to work on AKS support.
As fare as I can tell there are no issues or pull-requests related to AKS at the moment, so I assume I won't be duplicating effort.
Would anyone object to me looking into this? :)
Edit:
I was referring to the sdk directory. I had overlooked the services directory, it seems to support interacting with AKS.
This issue doesn't really make sense.

Yes services/mgmt supports service control planes as long as the service swagger is present.

I think this might need to be re-opened. https://github.com/Azure/azure-rest-api-specs/tree/main/specification/containerservice/resource-manager/Microsoft.ContainerService/aks is present, but it's not in the Rust SDK


Would like to submit pull request but getting "Permission denied"

I'd like to submit a pull request for a feature enhancement that I wrote. However, when I try to run git push origin my-local-branch, I get an error remote: Permission to keolo/mixpanel_client.git denied to ChaosFreak.. I have submitted branches in the past with no problem. Is there something I'm doing wrong?

Sorry, I had forgotten to work on my fork. I've submitted the PR just now.


Map v3

Change Map to dungeon style

I moved the file in the source to the resources folder. I will close PR #5


[HOLD FOR PAYMENT 10/1][$250] [Guided Setup Stage 3] Update the Pay w/ Expensify selectors

Part of the Extend Onboarding to Invited Users project
Main issue: https://github.com/Expensify/Expensify/issues/392615
Doc section: https://docs.google.com/document/d/1EX1tKSkhffpqP4TXD7iKIPKWnU5U8ELv32j-QTtXH2g/edit#heading=h.pnxf7cla3pn
Project: #wave-collect
Feature Description

When a user is invited to NewDot from an IOU, we initially assume they are going to be splitting bills and assign them the newDotSplitChat choice and an inviteTye of iou on their nvp_introSelected, but we want to wait till they select a payment type before we send them their onboarding message. If they select:

"Pay with Business Bank Account" we want to

Send them the newDotManageTeam onboarding message
Optimistically set nvp_introSelected

choice to newDotManageTeam
isInviteOnboardingComplete to true

"Pay with Personal Bank Account" or "Pay with Debit Card" we want to

Send them the newDotSplitChat message
Optimistically set nvp_introSelected

isInviteOnboardingComplete to true

When a user is invited to NewDot from an invoice, we initially assume they are going to be managing expenses for a team and give them the newDotManageTeam choice and an inviteType of invoice on their nvp_introSelected, but we want to wait till they select a payment type before we send them their onboarding message. If they select:

"Pay as an Individual" we want to

Send them the newDotSplitChat onboarding message
Optimistically set nvp_introSelected

choice to newDotSplitChat
isInviteOnboardingComplete to true

"Pay as a Business" we want to

Send them the newDotManageTeam message
Optimistically set nvp_introSelected

isInviteOnboardingComplete to true

Manual Test Steps
Note: These won't be manually testable till the following backend GHs are done and deployed
https://github.com/Expensify/Expensify/issues/411150
https://github.com/Expensify/Expensify/issues/411149
https://github.com/Expensify/Expensify/issues/411148
https://github.com/Expensify/Expensify/issues/411146
See the Pays an IOU or invoice via VBBA and Pays an IOU or invoice via Elsewhere/PBA/Debit/Credit sections from the doc.
Automated Tests
If possible, add tests to assert that

Users with nvp_introSelected.inviteType: iou that click the "Pay With Business Bank Account" button end up with nvp_introSelected.choice: newDotManageTeam and nvp_introSelected.isInviteOnboardingComplete: true and are sent the newDotManageTeam onboarding message.
Users with nvp_introSelected.inviteType: invoice that click the "Pay as an Individual" button end up with nvp_introSelected.choice: newDotSplitChat and nvp_introSelected.isInviteOnboardingComplete: true and are sent the newDotSplitChat onboarding message.

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021841133888555964659
Upwork Job ID:<PHONE_NUMBER>555964659
Last Price Increase: 2024-10-01

Issue OwnerCurrent Issue Owner: @

Confirming who from SWM is taking this here: https://expensify.slack.com/archives/C036QM0SLJK/p1720631312670319?thread_ts=1720454141.387759&cid=C036QM0SLJK

Hi, I’m Tymoteusz Kałuzieński from Software Mansion, the expert agency, and I’ll be working on this issue.

Hi @cdOut, thanks for taking this on!

Waiting for the backends.

I'm working on the PR to send the message to non-IOU/Invoice customers and should have that wrapped tomorrow. Once it's doen this should be very easy to complete shortly thereafter. Hoping to have it done tomorrow afternoon.

@cdOut The PR to push nvp_introSelected to onyx when invited users sign in for the first time has been deployed to staging, so you should be able to at least work with that NVP. It got too late in the day for a Friday to deploy the back end changes to handle. I think you can test your changes with Report.completeOnboarding without any backend changes to CompleteGuidedSetup, the only piece missing is that we won't updated the NVP in the backend. I am going to ask our Aussie employees to deploy the back end changes on their Monday morning, which should still be Sunday night for you in CEST. I'll let know one way or another if it happens or not.

@deetergp I'll look over the changes to the docs and try out the nvp_introSelected, could you also update me if the rest of the backend changes have been pushed to staging also?

@cdOut The backend has been updated to handle updating the NVP changes when an IOU-invited user selects BBA and an Invoice-invited user selects anything other than BBA. It should also send the onboarding message for any IOU or Invoice invited user despite which payment type they choose. Please test away and let me know if you hit any speed bumps along the way. Thanks!

Thank you, I'll test and add the recent changes to the doc and see if all works well. Will ping you if I have any further questions.

I worked on it yesterday and will continue today, both issues should be ready for review sometime tomorrow.

Looking forward to it @cdOut, thanks!

Any word on progress @cdOut?

@deetergp back to the slack thread, is there anything I need to add / specify when running it locally for the invite email to open correctly? In the video I've sent it clearly does not work as if something is not set correctly and those tests were run on the latest main.

Something has definitely changed on the back end. When an invitee responds to an invitation we are now getting a ton of permissions errors on the back end when trying to setup the app for them. I'm going through them now and trying to figure out what they are and granting them to the chat token on the back end. I'll let you know once I get that sorted.

Thank you, keep me updated and I’ll get on it as soon as you’re done with that. In the mean time, should I also include automated tests in these PRs?

@Christinadobrzyn I am heading OoO so reassigning this to keep eyes on it. Thanks!

@cdOut The fix for the issue that was blocking you has been merged and deployed to the staging server, which should hopefully unblock you now.

Any updates @cdOut?

I am back from OoO — thanks @Christinadobrzyn for holding down the fort!

@CortneyOfstad no luck with further testing the PR, we've been blocked with this for a few days now. A very crucial part of this feature, which is sending invites to the user added for an expense, is not working on the backend side of our app. You can refer to the thread over here on slack for further details. I will notify when we'll be able to actually push this PR further.

@cdOut We identified the issue and deployed the fix. You should be seeing the IOU/Invoice invites again now.

Now that the bug preventing IOU invites from sending has been resolved we're hoping to get @cdOut's PR soon.

Of course, I'm working on it today and will update you when it's ready for a final review.

@CortneyOfstad Of course, here's the link to the PR.

Any updates @cdOut?

Answered the above in the slack thread for this issue.

Latest update is here.

The PR to unblock @cdOut went out in the auth deploy today. I'm OOO Friday & Monday, but will check in to make sure I'm not blocking anyone in the meantime.

@DylanDylann is reviewing this PR: https://github.com/Expensify/App/pull/45264

Review still pending

I am on it. Will complete in an hour

@deetergp Could you help to verify the below conflict? Is it expected?

Was a typo, my bad. I've updated it in the description.
Yes, we are not setting choice when a user invited by an invoice pays with a business bank account as it has already been set to newDotManageTeam when they were invited. We are also not setting choice for IOU invited users when they pay with a non business bank account, as newDotSplitChat was set when they were invited.

I approved the PR, but it's got a merge conflict. Should be quick and easy enough for Tymo to resolve it and we'll get it merged.

This ended up getting fixed and merged last Friday.

@deetergp to confirm, payment is needed here? If so, standard for @DylanDylann as @cdOut is with SM? TIA!

@deetergp bump on the above ^^^ thank you!

@CortneyOfstad Sorry! Yes, you are correct, standard for @DylanDylann. @cdOut is with SWM.

@CortneyOfstad Please help to handle payment here. Thanks

Sorry for missing this as I was OoO — getting the payments sorted now!

Sorry the ping ^^^ Had to apply the external label to get the job post to be created!

@DylanDylann I just sent you an offer via Upwork — please let me know once you accept and I will get that paid ASAP. Thanks!

@CortneyOfstad I accepted it, thank you

Payment Summary
@DylanDylann — paid $250 via Upwork
@cdOut — contracted position


Fix link to script on kubernetes-build-win-binaries.md

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

@bobbypage,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[SPARK-25417][SQL] ArrayContains function may return incorrect result when right expression is implicitly down casted

What changes were proposed in this pull request?
In ArrayContains, we currently cast the right hand side expression to match the element type of the left hand side Array. This may result in down casting and may return wrong result or questionable result.
Example :
spark-sql> select array_contains(array(1), 1.34);
true

spark-sql> select array_contains(array(1), 'foo');
null

We should safely coerce both left and right hand side expressions.
How was this patch tested?
Added tests in DataFrameFunctionsSuite

Test build #96009 has started for PR 22408 at commit 2c6e8dc.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3062/
Test PASSed.

Merged build finished. Test PASSed.

Test build #96009 has finished for PR 22408 at commit 2c6e8dc.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96009/
Test FAILed.

Test build #96017 has started for PR 22408 at commit 4fb4c1a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3070/
Test PASSed.

Test build #96017 has finished for PR 22408 at commit 4fb4c1a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96017/
Test PASSed.

Test build #96031 has started for PR 22408 at commit 63f6ce5.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3080/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3085/
Test PASSed.

Test build #96036 has started for PR 22408 at commit 04915fd.

Test build #96036 has finished for PR 22408 at commit 04915fd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96036/
Test FAILed.

Test build #96031 has finished for PR 22408 at commit 63f6ce5.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96031/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3090/
Test PASSed.

Test build #96044 has started for PR 22408 at commit 04915fd.

Test build #96044 has finished for PR 22408 at commit 04915fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96044/
Test PASSed.

plz update the migration guide.

How about this decimal case?
// v2.3.1
scala> spark.range(10).selectExpr("cast(id AS decimal(9, 0)) as value").selectExpr("array_contains(array(1, 2, 3), value)").show
+--------------------------------------------------+
|array_contains(array(1, 2, 3), CAST(value AS INT))|
+--------------------------------------------------+
|                                             false|
|                                              true|
|                                              true|
|                                              true|
|                                             false|
|                                             false|
|                                             false|
|                                             false|
|                                             false|
|                                             false|
+--------------------------------------------------+

// this patch
scala> spark.range(10).selectExpr("cast(id AS decimal(9, 0)) as value").selectExpr("array_contains(array(1, 2, 3), value)").show
org.apache.spark.sql.AnalysisException: cannot resolve 'array_contains(array(1, 2, 3), `value`)' due to data type mismatch: Input to function array_contains should have been array followed by a value with same element type, but it's [array<int>, decimal(9,0)].; line 1 pos 0;
'Project [unresolvedalias(array_contains(array(1, 2, 3), value#2), Some(<function1>))]
+- Project [cast(id#0L as decimal(9,0)) AS value#2]
   +- Range (0, 10, step=1, splits=Some(4))

@maropu This is the case we were discussing for which @ueshin suggested using findWiderTypeWithoutStringPromotion. Lets see what @cloud-fan and @gatorsmile think and we will do accordingly. We are picking a much restrictive cast (since we are injecting this implicitly).

What is the corresponding behavior in Presto?

My general idea is to avoid risky implicit type casting at the beginning. We can relax it in the future, if needed. After all, users can manually cast the types after seeing the reasonable error message. This should not be a big deal. However, returning a confusing result due to implicit type casting is not good in general.

@ushin @gatorsmile Here are the results from presto. Please let me know if you want me try any case in particular. One thing to note is that presto allows comparison between int and decimal. In our findTightestCommonType we don't do the promotion.
presto:default> select contains(array[1,2,3], '1');
Query 20180914_053612_00006_pru6h failed: line 1:8: Unexpected parameters (array(integer), varchar(1)) for function contains. Expected: contains(array(T), T) T:comparable
select contains(array[1,2,3], '1')

presto:default> select contains(array[1,2,3], 'foo');
Query 20180914_053729_00007_pru6h failed: line 1:8: Unexpected parameters (array(integer), varchar(3)) for function contains. Expected: contains(array(T), T) T:comparable
select contains(array[1,2,3], 'foo')

presto:default> select contains(array['1','2','3'], 1);
Query 20180914_053850_00008_pru6h failed: line 1:8: Unexpected parameters (array(varchar(1)), integer) for function contains. Expected: contains(array(T), T) T:comparable
select contains(array['1','2','3'], 1)

presto:default> select contains(array[1,2,3], cast(1.0 as decimal(10,2)));
 _col0
-------
 true
(1 row)

presto:default> select contains(array[1,2,3], cast(1.0 as double));
 _col0
-------
 true
(1 row)

what does presto return for select array_contains(array(1), 1.34);?

@cloud-fan It turns false.
presto:default> select contains(array[1], 1.34)
             -> ;
 _col0
-------
 false
(1 row)

LGTM. I think the last piece is the migration guide, to explain what changed from 2.3 to 2.4

@cloud-fan Thanks a lot. Let me take a stab at the migration guide changes.
cc @ueshin

@ueshin @cloud-fan @gatorsmile Can you please look at the doc changes when you get a chance.. Thanks.

Test build #96126 has started for PR 22408 at commit d2d8afe.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3149/
Test PASSed.

Test build #96126 has finished for PR 22408 at commit d2d8afe.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96126/
Test PASSed.

Since it's a little risky to backport https://github.com/apache/spark/pull/22448 , I'd like to merge this PR without https://github.com/apache/spark/pull/22448 . Can you fix the conflict? thanks!

Test build #96322 has started for PR 22408 at commit df5ea47.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3280/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3284/
Test PASSed.

Test build #96329 has started for PR 22408 at commit d79e9d4.

LGTM

Test build #96329 has finished for PR 22408 at commit d79e9d4.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test build #96322 has finished for PR 22408 at commit df5ea47.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96329/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96322/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3287/
Test PASSed.

Test build #96333 has started for PR 22408 at commit d79e9d4.

thanks, merging to master/2.4!


hurd: fix definition of utsname struct

drop the domainname field, as it is not actually used
bump the length of the other fields to 1024, which is the actual size of the C arrays on Hurd

@sthibaul this should fix crashes when using this struct (by calling uname()); because of this, I'm nominating this for a backport in libc-0.2, even if "breaking".
@rustbot label stable-nominated

For reference:

sys/utsname.h: https://sourceware.org/git/?p=glibc.git;a=blob;f=posix/sys/utsname.h;h=c6884255e04595996dbbc337ac667d0defd035f3;hb=ab564362d0470d10947c24155ec048c4e14a009d

the general definition of the utsname struct

bits/utsname.h: https://sourceware.org/git/?p=glibc.git;a=blob;f=bits/utsname.h;h=c5c57e840397d9efa6ad6a3b614569d094d26c21;hb=ab564362d0470d10947c24155ec048c4e14a009d

the actual bits used on Hurd, setting 1024 as fields size and not enabling domainname

agreed!

Thanks for the sources, agreed that we can backport this (thanks for the ack @sthibaul).
Could you add const _UTSNAME_LENGTH: usize = 1024; and use that to make it a bit more consistent and greppable?

Could you add const _UTSNAME_LENGTH: usize = 1024; and use that to make it a bit more consistent and greppable?

Makes sense, done so -- thanks for the suggestion!


[WIP] Un-pin matplotlib

This accounts for https://github.com/conda-forge/parcels-feedstock/issues/29 and https://github.com/conda-forge/parcels-feedstock/pull/30
I'm testing this here before merging the PR in conda-forge.

Only problem seems to be in ax.scatter(...) of the Argo example: Matplotlib has problems handling the masked arrays, takes a different road to still produce a plot and then fails at dealing with the singleton dims of x, y, and z.  66b3a89 should fix this.

I think parcels also pins ffmpeg. See my comment on the conda-forge feedstock about maybe removing the matplotlib and the ffmpeg dependency altogether. (But note that this should go together with an update of the installation instructions on the website that then would need to point out that the users need to install matplotlib and ffmpeg manually (like they do with cartopy already).)
Am 15. Oktober 2018 17:02:50 schrieb Philippe Delandmeter<EMAIL_ADDRESS>
delandmeterp approved this pull request.
Yeah sure, it was the only dependency with a fixed version, right?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/OceanParcels/parcels/pull/472#pullrequestreview-164749786


Verify libraries exist

Eg run ldd on clamscan or the library

Running clamscan is enough for a successful check for now


Write Permissions Cannot Create Client

Seeing this stack trace continually with any openapi spec.. seems to not have permission to write or something.
[main] INFO  o.o.codegen.TemplateManager - writing file /github/workspace/kotlin-client/src/main/kotlin/com/company/project/model/SomeType.kt
Exception in thread "main" java.lang.RuntimeException: Could not generate model 'AccountType'
	at org.openapitools.codegen.DefaultGenerator.generateModels(DefaultGenerator.java:569)
	at org.openapitools.codegen.DefaultGenerator.generate(DefaultGenerator.java:946)
	at org.openapitools.codegen.cmd.Generate.execute(Generate.java:487)
	at org.openapitools.codegen.cmd.OpenApiGeneratorCommand.run(OpenApiGeneratorCommand.java:32)
	at org.openapitools.codegen.OpenAPIGenerator.main(OpenAPIGenerator.java:66)
Caused by: java.nio.file.NoSuchFileException: /github/workspace/kotlin-client/src/main/kotlin/com/company/project/model/SomeType.kt
	at java.base/sun.nio.fs.UnixException.translateToIOException(Unknown Source)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(Unknown Source)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(Unknown Source)
	at java.base/sun.nio.fs.UnixFileSystemProvider.newByteChannel(Unknown Source)
	at java.base/java.nio.file.spi.FileSystemProvider.newOutputStream(Unknown Source)
	at java.base/java.nio.file.Files.newOutputStream(Unknown Source)
	at java.base/java.nio.file.Files.write(Unknown Source)
	at org.openapitools.codegen.TemplateManager.writeToFileRaw(TemplateManager.java:255)
	at org.openapitools.codegen.TemplateManager.writeToFile(TemplateManager.java:237)
	at org.openapitools.codegen.TemplateManager.writeToFile(TemplateManager.java:198)
	at org.openapitools.codegen.TemplateManager.write(TemplateManager.java:1[64](https://github.com/company/project-service/actions/runs/5203732454/jobs/9387007786#step:4:66))
	at org.openapitools.codegen.DefaultGenerator.processTemplateToFile(DefaultGenerator.java:1092)
	at org.openapitools.codegen.DefaultGenerator.processTemplateToFile(DefaultGenerator.java:10[79](https://github.com/company/project/actions/runs/5203732454/jobs/9387007786#step:4:81))
	at org.openapitools.codegen.DefaultGenerator.generateModel(DefaultGenerator.java:408)
	at org.openapitools.codegen.DefaultGenerator.generateModels(DefaultGenerator.java:560)

Someone smart helped me fix this. I changed the user from 1001 to 1000
runner:x:1000:1000:,,,:/home/runner:/bin/bash.
It's mounted as a user that doesn't exist in our runner
hard coding permissions is a poor practice, and this would be an intermittent failure caused by action configuration.
1:51
It's mounted as a user that doesn't exist in our runner
1:53
hard coding permissions is a poor practice, and I agree that this is a bug on their end
1:50
runner:x:1000:1000:,,,:/home/runner:/bin/bash
1:51
It's mounted as a user that doesn't exist in our runner
1:53
hard coding permissions is a poor practice, and I agree that this is a bug on their end

We set the UID for a reason, but I don't remember what. @emann do you remember why we needed that?

hitting the same issue. should the user be an input?

Yeah probably. To get it to work I had to fork it and use user 1000.
Feel free to use my fork.. it worked for me at least.

@alexef don't forget to update the docs for your PR

Should be fixed in 1.4.1, thanks!


Fixes guard-process collision

When using guard-rspec and guard-process Guard::Process.wait2 method is called, which doesn't exists, the expected behaviour is to call Process.wait2 from ruby's stdlib, this happens due to a conflict in resolving Process constant which is evaluated against Guard.

Awesome, thanks!

Released as v4.6.4


App permissions protobufs

Adding protos detailed in Permissions tech spec.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Katherine Pioro seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


cannot start syslog listener

cannot run syslog listener on port 514 because (nonexistent) user logstash runs as does not have privileges to start services on ports < 1024. i've tried calling the docker-compose up command as root, as well. this is probably because i don't know docker well at all :\
observe

I do not understand what you are trying to achieve here. Are you just not able to start the stack? Did you made any configuration changes? I'll need more details here in order to help you.

Sure sure. I'm trying to start logstash with the rest of the services (kibana and elasticsearch), but I have modified the logstash conf like so:
    input {
         syslog {}
    }

but the service fails to bind to port 514, since that is a privileged kernel-level port. I'm unsure of how to specify the user the service should run as, so it can bind to that port.

Actually, you could just change the port syslog is listening to. Here are my changes:
logstash.conf:
input {
	syslog {
		port => 5000
	}
}

docker-compose.yml:
logstash:
  image: logstash:latest
  command: logstash -f /etc/logstash/conf.d/logstash.conf
  volumes:
    - ./logstash/config:/etc/logstash/conf.d
  ports:
    - "5000:5000"
    - "5000:5000/udp"
  links:
    - elasticsearch

Then, you can try to send a message to logstash using netcat:
$ nc -w0 -u localhost 5000 <<< "message to logstash via syslog"

I can change the port, but I wanted to replace the listening regular syslog
with logstash on the usual syslog port.
On Fri, Mar 25, 2016 at 2:10 PM, Anthony Lapenna<EMAIL_ADDRESS>wrote:

Actually, you could just change the port syslog is listening to. Here are
my changes:
logstash.conf:
input {
syslog {
port => 5000
}
}
docker-compose.yml:
logstash:
image: logstash:latest
command: logstash -f /etc/logstash/conf.d/logstash.conf
volumes:
- ./logstash/config:/etc/logstash/conf.d
ports:
- "5000:5000"
- "5000:5000/udp"
links:
- elasticsearch
Then, you can try to send a message to logstash using netcat:
$ nc -w0 -u localhost 5000 <<< "message to logstash via syslog"
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/deviantony/docker-elk/issues/46#issuecomment-201504808

Ok, I found a solution, tell me if it suits your situation.
logstash.conf:
input {
	syslog { }
}

docker-compose.yml:
logstash:
  image: logstash:latest
  command: /opt/logstash/bin/logstash -f /etc/logstash/conf.d/logstash.conf
  volumes:
    - ./logstash/config:/etc/logstash/conf.d
  ports:
    - "514:514"
    - "514:514/udp"
  links:
    - elasticsearch

Send a message using netcat:
$ nc -w0 -u localhost 514 <<< "message to logstash via syslog input"

Basically, all you need to change is the command line of the logstash container in the docker-compose.yml file.
Explanation: If you have a look at the logstash image Dockerfile here https://github.com/docker-library/logstash/blob/4f3a81ff0f152003653eecd60d0c4e1f05ce99f0/2.2/Dockerfile you can see that it is using gosu.
gosu iis a tool to manage user/permissions and execute a command. You can find more information about it here: https://github.com/tianon/gosu
From the Dockerfile, you can see that an entry point is defined (https://docs.docker.com/engine/reference/builder/#entrypoint), which is the script docker-entrypoint.sh. From this script (https://github.com/docker-library/logstash/blob/4f3a81ff0f152003653eecd60d0c4e1f05ce99f0/2.2/docker-entrypoint.sh), you can see the following content:
# Run as user "logstash" if the command is "logstash"
if [ "$1" = 'logstash' ]; then
	set -- gosu logstash "$@"
fi

So if you run any logstash command in the container, it will be started with the logstash user. The trick here is to pass the full path to logstash binary to avoid the gosu setup, i.e: /opt/logstash/bin/logstash

@hlmtre Did you had time to check my solution?

Apologies, I have not. While I was playing around with various ELK
stacks/prebuilt things I destroyed the VM that I'd had your docker stack
on. I hope to try it again tonight.
On Thu, Mar 31, 2016 at 2:44 PM, Anthony Lapenna<EMAIL_ADDRESS>wrote:

@hlmtre https://github.com/hlmtre Did you had time to check my solution?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/deviantony/docker-elk/issues/46#issuecomment-204143213

I'll close this issue for now, feel free to re-open if you are still having problems.

I can confirm that the solution above works - however, at some point the location of logstash changed, so the command now needs to be:
command: /usr/share/logstash/bin/logstash -f /etc/logstash/conf.d/logstash.conf


Prevent overwriting the manifest in the first phase of the build

Since we store the optimization manifest in the build directory, and since it doesn't have a counterpart in the source directory the ManifestResource needs to behave a little differently than your average resource. Instead of reading a file from source and writing it to build we want to read from build if the file already exists, so that we preserve the contents of that file through the first stage of the build.
In one of my projects this PR takes a 22 minute build down to 2 minutes.
Fixes #60
Fixes #41, Closes #58  (Specifying the manifest path shouldn't be needed if we don't overwrite it, and storing it outside the build directory doesn't exactly make sense since it is an artifact of a particular build, or sequence of builds. To say it another way, if you used an existing manifest with an empty build directory you'd end up skipping optimization of images that would need to be optimized.)

Any news on merging this?


settings: Convert settings_overlay.html to handlebars template.

This PR converts the settings_overlay to handlerbars template.
Fixes #18796.

This PR is not completed yet as I am facing a weird a bug for which I have posted here https://chat.zulip.org/#narrow/stream/9-issues/topic/convert.20settings_overlay.20to.20handlerbars.20.2318796. Apart from this bug there are couple of other changes as well.

OK.  We may also get to remove some fields from the context for home that were only used in this menu.

@sahil839 do we have a path forward for how to unblock this PR?  It's one of the last few things left in our template migration effort.

Unfortunately, I am still not able to fix the bug of tables disappearing on reload.

I was able to diagnose the SimpleBar problem; it had already been filed as Grsmto/simplebar#552, but I found a reproduction recipe and added it to the issue.

Sent a fix as Grsmto/simplebar#580.

Awesome, hopefully upstream will review that soon enough that we don't feel tempted to run a fork to get that fix.

simplebar already did a release with the fix, so we can rebase this!
@aryanshridhar FYI -- you can check if rebasing and provisioning your PR with a simplebar issue also is happy with this new release.

Merged, thanks @sahil839!


Awake Mode Options (taskbar)

Microsoft PowerToys version
0.41.3
Running as admin

[x] Yes

Area(s) with issue?
General
Steps to reproduce
Right click on Awake in Taskbar > Mode (options)
✔ Expected Behavior
PowerToys Awake list Mode Options
❌ Actual Behavior
PowerToys Awake list Mode Options does not show
Other Software
No response

Same issue.
Also same as #12224 (I just happened to see that other one first)

Dupe of #12145.


V3: HV: parse seed from ABL

ABL pass seed_lists to HV through different interface/structures.
So, in this patch, add interface to retrieve seed from ABL and refactor
the seed parsing logic.
Signed-off-by: Qi Yadong<EMAIL_ADDRESS>

@bzhu5 @kwang13
updated, please review it if you got a chance.


Misreporting of platform default dependencies?

Using cargo-tree 0.26.0, I noticed something strange with a project depending on flate2 and its transitive dependencies.  Namely the dependency tree includes miniz_oxide_c_api even though the rust_backend feature is not enabled.
The flate2 1.0.9 Cargo.toml appears to make the otherwise non-default miniz_oxide_c_api mandatory on wasm32.  But I'm not targeting wasm.  See excerpt below.  Is that what's confusing cargo-tree?
[dependencies]
miniz-sys = { path = "miniz-sys", version = "0.1.11", optional = true }
miniz_oxide_c_api = { version = "0.2", optional = true, features = ["no_c_export"]}

[target.'cfg(all(target_arch = "wasm32", not(target_os = "emscripten")))'.dependencies]
miniz_oxide_c_api = { version = "0.2", features = ["no_c_export"] }

[features]
default = ["miniz-sys"]
zlib = ["libz-sys"]
rust_backend = ["miniz_oxide_c_api"]

Here is a minimal reproduction test project:
[package]
name = "cargo-tree-opt-dep-test"
version = "0.1.0"
edition = "2018"

[dependencies]
flate2 = "^1.0.9"
brotli = {version = "^3.3.0", optional = true}

[features]
default = []

With the following output from cargo tree:
cargo-tree-opt-dep-test v0.1.0 (/home/david/src/cargo-tree-opt-dep-test)
└── flate2 v1.0.9
    ├── crc32fast v1.2.0
    │   └── cfg-if v0.1.9
    ├── libc v0.2.60
    ├── miniz-sys v0.1.12
    │   └── libc v0.2.60 (*)
    │   [build-dependencies]
    │   └── cc v1.0.38
    └── miniz_oxide_c_api v0.2.3
        ├── crc32fast v1.2.0 (*)
        ├── libc v0.2.60 (*)
        └── miniz_oxide v0.3.0
            └── adler32 v1.0.3
        [build-dependencies]
        └── cc v1.0.38 (*)

Note that the optional brotli dependency isn't shown, so cargo tree does pay attention to features.  Here is cargo clean && cargo build output showing miniz_oxide_c_api isn't built:
  Compiling libc v0.2.60
   Compiling cc v1.0.38
   Compiling crc32fast v1.2.0
   Compiling cfg-if v0.1.9
   Compiling miniz-sys v0.1.12
   Compiling flate2 v1.0.9
   Compiling cargo-tree-opt-dep-test v0.1.0 (/home/david/src/cargo-tree-opt-dep-test)
    Finished dev [unoptimized + debuginfo] target(s) in 3.85s

Also I should mention: I get the exact same dependency tree, if I explicitly specifying my default target via cargo tree --target x86_64-unknown-linux-gnu, which is obviously not wasm32.

We're probably touching some bit of cargo's internals incorrectly. Luckily, it seems like we're pretty close to cargo metadata having all of the bits of information we need to stop using cargo-the-library entirely which should remove a whole set of issues like this.


Should include babel config in npm

CodeSandbox demo

N/A compile issue
Steps to reproduce the problem

packaging popper to other projects and using babel, it will compile node_modules recursive
because of package.json include the babel config file path if will try to read it
none of config file found, babel will throw an error when compile

just like this
ERROR in<EMAIL_ADDRESS>Module build failed: Error: [BABEL]<EMAIL_ADDRESS>Couldn't resolve extends clause of ./.config/babel.config in<EMAIL_ADDRESS>    at Logger.error (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\logger.js:41:11)
    at ConfigChainBuilder.mergeConfig (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:186:32)
    at ConfigChainBuilder.addConfig (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:157:10)
    at ConfigChainBuilder.findConfigs (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:102:30)
    at buildConfigChain (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:61:13)
    at OptionManager.init (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\options\option-manager.js:354:58)
    at File.initOptions (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\index.js:212:65)
    at new File (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\file\index.js:135:24)
    at Pipeline.transform (D:\workspace\iviewex\node_modules\babel-core\lib\transformation\pipeline.js:46:16)
    at transpile (D:\workspace\iviewex\node_modules\babel-loader\lib\index.js:50:20)
    at Object.module.exports (D:\workspace\iviewex\node_modules\babel-loader\lib\index.js:173:20)
 @ ./node_modules/babel-loader/lib!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./node_modules/iview-loader??ref--2-1!./package/datepicker/datepickerAdapter.vue 27:12-37
 @ ./package/datepicker/datepickerAdapter.vue
 @ ./package/datepicker/index.js
 @ ./package/index.webpack.js

What is the expected behavior?

It shouldn't include in package.json or include the config file in npm package.
What went wrong?

Any other comments?

Same thing with ESLint. eslint.config is missing from .config folder but package.json references to it.
[Error - 12:10:54 PM] ESLint stack trace: [Error - 12:10:54 PM] Error: Failed to load config "./.config/eslint.config" to extend from. Referenced from<EMAIL_ADDRESS>   at configMissingError (*********/node_modules/eslint/lib/cli-engine/config-array-factory.js:265:9)     at ConfigArrayFactory._loadExtendedShareableConfig (*********/node_modules/eslint/lib/cli-engine/config-array-factory.js:826:23)     at ConfigArrayFactory._loadExtends (*********/node_modules/eslint/lib/cli-engine/config-array-factory.js:731:25)     at ConfigArrayFactory._normalizeObjectConfigDataBody (*********/node_modules/eslint/lib/cli-engine/config-array-factory.js:660:25)
 "eslintConfig": {     "extends": "./.config/eslint.config"   },

Should we strip the rules from package.json or include the config?

Should we strip the rules from package.json or include the config?

Yes, using .eslintrc.js may be a better choice, eslint will auto read the file when it exist, it will be better in this case.

Is it possible to suppress the eslint warnings until proper fix is released? I have tried to add node_modules to .eslintignore but it still produces those warnings. We have like bazillion of them in the CI log making it hard to read ;(

Make sure CI see this eslintignore. By the way eslint v7 ignores node_modules by default.

Adding node_modules into .eslintignore does not help for webpack build chain. Only for yarn eslint.
But we have figured out hotfix - split webpack JS chain into 2 separate chains:
webpack.config.js:
rules: [
          {
            // node_modules - only babel, no eslint
            test: /\.(js|jsx)$/,
            include: [
              path.resolve(__dirname, 'node_modules')
            ],
            use: [
              {
                loader: 'babel-loader'
              }
            ]
          }, {
            // project files - babel + eslint
            test: /\.(js|jsx)$/,
            include: [
              path.resolve(__dirname, './your-project-root/')
            ],
            use: [
              {
                loader: 'babel-loader'
              }, {
                loader: 'eslint-loader'
              }
            ]
          }
        ]

This is kinda proper fix. There is nothing to do with this package. Eslint should ignore all node_modules. Btw exclude: /node_modules/ works too.

@TrySound what about babel?

I presume the exclude: /node_modules/ is defined per-rule but not per-loader, so the only way is the pipeline split as I have suggested?


Human posture sequence

how is the historical human posture sequence obtained ?

Hi,
Following previous methods, the sequences are extracted firstly with alphapose for the pose estimation and then with poseflow for tracking a person in the video.

Thank you

Hi!
We added the code that we used to annotate UBnormal. Please refer to this README for further details.


Badge undetected when placed before other badges

The following image illustrates the duplication of the badge when the existing badge was placed before other badges. This is most likely a RegExp error. I haven't checked the log for this case.

Another issue is that both states are being changed when it should only change one unless permitted.


custom extension for file attachment syntaxes ![][] and [][]

I want to create an extension for these file attachment syntaxes ![Alt Text][only image attachment id] and [Alt Text][normal file attachment id].
Basically the behavior is similar to link syntax but use [ instead of ( for link information.
For example:
![myimage][12345] will be converted to an img tag pointing to url for that image attachment file.
[my report][12345] will be converted to an a tag pointing to url for that normal attachment file.
Could you give some pointer which codes or extension should i learn from?
Thank you.

@rshah, the syntax you want to use is already implemented for inline parser as link and image refs.
The easiest way to get what you want is to create a custom node renderer for link and image refs which will not resolve using defined references but behave like link and image nodes.

LinkRef rendering: CoreNodeRenderer.java: Lines 942-1001

ImageRef rendering: CoreNodeRenderer.java: Lines 888-940

Link rendering: CoreNodeRenderer.java: Lines 846-866

Image rendering: CoreNodeRenderer.java: Lines 818-844

Custom node renderer sample: NodeRendererSample.java

Both LinkRef and ImageRef nodes are RefNode classes and the .getText() returns the alt or text part of the node while .getReference() the url part of the node.
One caveat is that the syntax for link and image refs allows [ref] and ![ref], in which case the ref is provided and the text is equal to the ref. In this case the RefNode.isReferenceTextCombined() is true and .getText().isNull() is also true, with .getText() returning an empty based sequence.

@vsch
Thank you for fast reply. I managed to implement basic renderer for my case.


:bomb: Bug: Fail to apply "quick start"

What Happened?

How to Reproduce?
config:
state: # state config, backend can be local, s3 or k8s
backend: local
options:
stateFile: devstream.state
vars:
RepoOwner<EMAIL_ADDRESS>ImageRepoUser: jiangyoujian
tools:

name: repo-scaffolding
instanceID: golang-github
options:
destinationRepo:
owner: [[ RepoOwner ]]
name: go-webapp-devstream-demo
branch: main
scmType: github
token: [[ env GITHUB_TOKEN ]]
sourceRepo:
org: devstream-io
name: dtm-repo-scaffolding-golang-gin
scmType: github
name: github-actions
instanceID: default
dependsOn: ["repo-scaffolding.golang-github"]
options:
scm:
owner: [[ RepoOwner ]]
name: go-webapp-devstream-demo
branch: main
scmType: github
token: [[ env GITHUB_TOKEN ]]
pipeline:
configLocation: https://raw.githubusercontent.com/devstream-io/dtm-pipeline-templates/main/github-actions/workflows/main.yml
language:
name: go
framework: gin
imageRepo:
user: [[ ImageRepoUser ]]
password: [[ env IMAGE_REPO_PASSWORD ]]

Anything else
dtm apply -f config.yaml -y
DevStream Version
latest

I guess it's because of a problem with the RepoOwner variable setting, which should use the GitHub ID instead of the email. Try changing it to tmacInChina. @tmacInChina
(Sorry, I replied to you yesterday, but maybe the network didn't work)


Update or replace edx-jsme

We use the package edx-jsme in edx/edx-platform. Last commit to this package's repo was made in 2017 and it hasn't yet added support for Django3.2. Please follow the guidance in https://openedx.atlassian.net/wiki/spaces/AC/pages/3036972032/Handling+Outdated+Dependencies to resolve the problem this poses for the Open edX Django 3.2 upgrade.

I'm not sure how to proceed here.
The package uses Django in this line and that is used just here.
from Django.utils.translation import ugettext as _ can still be used in Django 3.2, proof here, so we are safe there.
I also tried to run the package tests, but they depend on the Capa module.
Is what I mentioned enough to say that it supports Django 3.2 and it does not need any intervention?

@Jawayria Could you please help @mariajgrimaldi here?

of course @natabene! Thanks :)

Yes @mariajgrimaldi we don't need any intervention in this package. Thanks for having a look at it. @natabene we can close this issue.

I will follow up with natea about doing a release and whether they still maintain this or it needs another owner.

@natabene, thanks for the update. Would you mind letting me know if you need any help?

@mariajgrimaldi Thanks for offering, I am good.

Decided to deprecate, effort tracked in https://openedx.atlassian.net/browse/DEPR-173. Closing this ticket.


Cannot import tensorflow-gpu in jupyter notebook.

Hi,
I am using the following conda environment and jupyter notebook.
python==3.5
tensorflow-gpu==1.4.0
cuda==8.0
cudnn==6.0
While I can correctly import tensorflow in the cmd line:
(tensorflow) [dushu@ip-172-20-149-210 Mask_RCNN]$ python
Python 3.5.5 |Anaconda, Inc.| (default, May 13 2018, 21:12:35)
[GCC 7.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as t
/home/dushu/anaconda3/envs/tensorflow/lib/python3.5/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
>>> t.test
<module 'tensorflow.python.platform.test' from '/home/dushu/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/platform/test.py'>
>>> t.test.gpu_device_name()
2019-02-05 04:08:32.393567: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA
2019-02-05 04:08:35.257250: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:892] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2019-02-05 04:08:35.257629: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1030] Found device 0 with properties:
name: Tesla K80 major: 3 minor: 7 memoryClockRate(GHz): 0.8235
pciBusID: 0000:00:1e.0
totalMemory: 11.17GiB freeMemory: 11.10GiB
2019-02-05 04:08:35.257662: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1120] Creating TensorFlow device (/device:GPU:0) -> (device: 0, name: Tesla K80, pci bus id: 0000:00:1e.0, compute capability: 3.7)
'/device:GPU:0'

I can' t import it in the jupyter notebook:
ImportError: Traceback (most recent call last):
  File "/home/dushu/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "/home/dushu/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "/home/dushu/anaconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "/home/dushu/anaconda3/envs/tensorflow/lib/python3.5/imp.py", line 243, in load_module
    return load_dynamic(name, filename, file)
  File "/home/dushu/anaconda3/envs/tensorflow/lib/python3.5/imp.py", line 343, in load_dynamic
    return _load(spec)
ImportError: libcublas.so.8.0: cannot open shared object file: No such file or directory

Failed to load the native TensorFlow runtime.

See https://www.tensorflow.org/install/install_sources#common_installation_problems

for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

The jupyter seems cannot find cuda path. The other similar issues don' t work either.
Any help please ?

solved by specifying
sudo ldconfig /usr/local/cuda-8.0/lib64

any updated for windows 10 ? I am getting this same error


[Bug] Hell's Island Timer

I'm seriously thinking about having to go through each of the maps to see the timers hahaha

Sorry for the (huge) delay - I will get into this issue today.
I was enjoying Payday 3, and also looking into modding it.

haha no problem

Do you by any chance have knowledge what this timer does?

I would have to play the map again to tell you, tomorrow I will write to you to see if I can find the timer


Fix build dependency issue between wherehows-web & web.

Coverage remained the same at 0.0% when pulling 65ff63a852b1994409f7265dd4e9582895723252 on mars-lan:ember2 into 36e47cb62c80f3a056a996321c3f8f87e35dffd9 on linkedin:linkedin.


Question: About ThreadLocal memory leak

In this issue:
https://github.com/spring-cloud/spring-cloud-sleuth/issues/27?spm=ata.13261<IP_ADDRESS>bf4985CXnIg1 @rwinch  report a bug which may cause memory leak, but the reference to this is a weak reference which kept in thread, so call ThreadLocal.set(null) or ThreadLocal.remove() has no matter. What may cause memory leak is that the thread local is a customed  class (e.g NamedThreadLocal) and an outlasts thread instance has its reference, so cause WebAppClassLoader leak.

Sleuth is not storing any tracing information directly. We do it either via Brave or OpenTelemetry libraries. If you see memory leaks please file issues in the corresponding libraries.

https://github.com/spring-cloud/spring-cloud-sleuth/issues/27 is pasted in Sleuth.
Memory leaks may not caused by the bug mentioned on the paste, so I wanna to know why the code has changed.


Fix-登录时用户名不存在空指针，增加日志提醒

Fixes 272
Changes proposed in this pull request:

Check mailbox configuration when submitting. Contributor Guide

#272


[#7091] Fix batch auto configuration error

Fix UnsatisfiedDependencyException in Pinpoint web when using batch profile
close #7091

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #7092 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7092      +/-   ##
==========================================
- Coverage   38.93%   38.92%   -0.01%
==========================================
  Files        3289     3289
  Lines       95763    95763
  Branches    11964    11964
==========================================
- Hits        37283    37278       -5
- Misses      55513    55519       +6
+ Partials     2967     2966       -1

Impacted Files
Coverage Δ

...c/main/java/com/navercorp/pinpoint/web/WebApp.java
0.00% <ø> (ø)

...p/pinpoint/rpc/client/WriteFailFutureListener.java
56.25% <0.00%> (-18.75%)
:arrow_down:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
55.55% <0.00%> (-11.12%)
:arrow_down:

...p/pinpoint/rpc/stream/StreamChannelRepository.java
75.00% <0.00%> (-10.00%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75.00% <0.00%> (-3.13%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
72.50% <0.00%> (-1.67%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.22% <0.00%> (-1.06%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
72.40% <0.00%> (+0.32%)
:arrow_up:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
59.57% <0.00%> (+2.12%)
:arrow_up:

...vercorp/pinpoint/web/vo/ApplicationAgentsList.java
69.23% <0.00%> (+2.56%)
:arrow_up:

... and 1 more

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 08f1c35...011c1a9. Read the comment docs.


Travis integration

@nullbio This adds Travis integrations for the tests.
I expect some of these tests will initially fail on some versions of go - once i get it merged in and can see what might fail on what versions, I can move those tests into build tag protected files.
Let me know if you have further ideas around this

Appreciate the note! I'll get this pushed across tomorrow night
On Sep 25, 2016 11:42 PM, "Patrick O"<EMAIL_ADDRESS>wrote:

@nullbio approved this pull request.
This looks great, love your work mate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StabbyCutyou/generics/pull/8#pullrequestreview-1471861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdTKt17GAKRWztyYAgsOhcXcN8RZcVbks5qtz88gaJpZM4KGG71
.


Starting a transaction on MSSQL 2012 fails

Elixir version
1.14.0
Database and Version
SQL Server 2012
Ecto Versions
3.10.3
Database Adapter and Versions (postgrex, myxql, etc)
tds 2.3.2
Current behavior

Make a new Phoenix app with mix phx.new.
Configure a MSSQL 2012 database and corresponding Phoenix config.
Create an Ecto schema and try to migrate with mix ecto.migrate.

At this point, you may see that the database throws a syntax error. This originates here:
def lock_for_migrations(meta, opts, fun) do
    %{opts: adapter_opts, repo: repo} = meta

    if Keyword.fetch(adapter_opts, :pool_size) == {:ok, 1} do
      Ecto.Adapters.SQL.raise_migration_pool_size_error()
    end

    opts = Keyword.merge(opts, timeout: :infinity, telemetry_options: [schema_migration: true])

    {:ok, result} =
      transaction(meta, opts, fn ->
        query =
          "sp_getapplock @Resource = 'ecto_#{inspect(repo)}', @LockMode = 'Exclusive', @LockOwner = 'Transaction', @LockTimeout = -1"

        Ecto.Adapters.SQL.query!(meta, query, [], opts)
        fun.()
      end)

    result
  end

The workaround for me was to add an exec to the query, i.e. query = "exec sp_getapplock ...", but I just used the library as-is to connect to a SQL Server 2017 instance and it works fine. I'm quite green in Elixir so I can't say whether this is the root cause or not. But it seems to be.
Expected behavior
I can connect to MSSQL 2012 and run transactions.

Thanks for the report. Could you please copy the error and stack trace you received ?

I am fine with not supporting MSSQL 2012. It is no longer officially supported by Microsoft unless you are paying for extended security policies: https://learn.microsoft.com/en-us/lifecycle/products/microsoft-sql-server-2012.  If someone wants to send a PR that supports both versions and it is not tons of work, then it is fine. :)
Although we should note at least 2017 is required in the Ecto.Adapters.TDS docs.

Here's the trace:
10:41:47.295 [debug] Line 1 (Error 8180): Statement(s) could not be prepared.
** (Tds.Error) Line 1 (Error 102): Incorrect syntax near 'sp_getapplock'.
    (ecto_sql 3.10.1) lib/ecto/adapters/sql.ex:913: Ecto.Adapters.SQL.raise_sql_call_error/1
    (ecto_sql 3.10.1) lib/ecto/adapters/tds.ex:292: anonymous fn/4 in Ecto.Adapters.Tds.lock_for_migrations/3
    (ecto_sql 3.10.1) lib/ecto/adapters/sql.ex:1203: anonymous fn/3 in Ecto.Adapters.SQL.checkout_or_transaction/4
    (db_connection 2.5.0) lib/db_connection.ex:1630: DBConnection.run_transaction/4
    (ecto_sql 3.10.1) lib/ecto/adapters/tds.ex:290: Ecto.Adapters.Tds.lock_for_migrations/3
    (ecto_sql 3.10.1) lib/ecto/migrator.ex:574: Ecto.Migrator.lock_for_migrations/4
    (ecto_sql 3.10.1) lib/ecto/migrator.ex:428: Ecto.Migrator.run/4
    (ecto_sql 3.10.1) lib/ecto/migrator.ex:170: Ecto.Migrator.with_repo/3

I totally understand not wanting to support 2012...I don't want to work with MSSQL 2012 😅 alas...

I wonder if the exec variant works for all versions. But I never used sql server even once so not sure what kind of land mines are hidden

Well, let's try: https://github.com/elixir-ecto/ecto_sql/pull/548


Use current time twice  in a Plyr instance

Plyr seems great so far 👍🏼
I'm hoping to display the current time twice on my player—once on the left of the progress and once, inverted, on the right of the progress.
For example, this screenshot would be showing two current times for a 31 second video 👇🏼

I'm just editing the DOM for this example. If I try to use two 'Current times', only the first instance is recognized and the second remains 00:00.

Dear friends in the sampotts
Please check this out

Plyr seems great so far 👍🏼
I'm hoping to display the current time twice on my player—once on the left of the progress and once, inverted, on the right of the progress.
For example, this screenshot would be showing two current times for a 31 second video 👇🏼

I'm just editing the DOM for this example. If I try to use two 'Current times', only the first instance is recognized and the second remains 00:00.

You can do it through the custom controls markup,
That way :
const controls = `

00:00

		<input data-plyr="seek" type="range" min="0" max="100" step="0.01" value="0" aria-label="Seek">
		<progress class="plyr__progress__buffer" min="0" max="100" value="0">% buffered</progress>
		00:00

00:00

	<button type="button" class="plyr__control" aria-label="Play, {title}" data-plyr="play">
		<svg class="icon--pressed" role="presentation">
			<use xlink:href="#plyr-pause"></use>
		</svg>
		<svg class="icon--not-pressed" role="presentation">
			<use xlink:href="#plyr-play"></use>
		</svg>
		Pause
		Play
	</button>

`;
const doctors__voice_message = new Plyr('#doctors__voice_message', {controls});

It's not currently possible. You can display the current-time (which you can click to toggle as a countdown = e.g. -4:30) and duration (fixed, e.g. 4:30) as options but not a secondary current-time that is a countdown. This is something I can include though.

It's not currently possible. You can display the current-time (which you can click to toggle as a countdown = e.g. -4:30) and duration (fixed, e.g. 4:30) as options but not a secondary current-time that is a countdown. This is something I can include though.

Thanks for the reply. If it's not difficult to include or if other Plyr users would want it, I would love to see this feature added.


chore: github action for binary build

Run this script can generate binaries
PLATFORMS=darwin/amd64,darwin/arm64,windows/amd64,windows/arm64,linux/amd64,linux/arm64 bash build.sh

TLS expires
When creating the Zipper, Downstreams and Auth Options are not added. The development mode has no problem, but need to configure this parameter in the production environment
'yomorun/psig' and prscd both references 'yomo', it is easy to cause yomo version conflict, recommended to put it directly in the 'prscd' repository

@venjiang Yes, you're right. I will merge psig to this repo after merging this PR.


how to draw double line in makerjs?

Hi ,i am new to makerjs. we often use double line in autoCAD to means walls.I want to draw double line using makerjs and export cxf file. is there any easy way to draw double line in makerjs?
hope for you help   thx

Yes, you can use the Wireframe technique. First, make a single line drawing. Then, call makerjs.model.expandPaths.
See Expanding paths for more info.

it works well!   thank you so much


Better way to detect hip_platform

Hi,
I have a problem with detecting the correct hip_platform on Nvidia machines. The problem lies in the "hipconfig" script below. Since I have installed rocm-dev package for cross-compilation reasons, the command "/opt/rocm/hcc/bin/hcc --version" always runs gracefully and the HIP_PLATFORM is set to "hcc". This causes a problem when the gpu card on the machine is Nvidia. I was wondering if there is a better way to detect platform such as actually probing the system for gpu device.
sub can_run {
    my ($exe) = @_;
    `$exe --version 2>&1`;
    if ($? == 0) {
        return 1;
    } else {
        return 0;
    }
}

and
if (not defined $HIP_PLATFORM) {
    if (can_run("$HCC_HOME/bin/hcc") or can_run("hcc")) {
        $HIP_PLATFORM = "hcc";
    } ...

Note: I know "export HIP_PLATFORM=nvcc" solves this issues but ideally I don't want to require the end-user of my program to do this.
Thanks

@SarbojitAMD @mangupta @kzhuravl This issue can be finally be closed now with the internal merge of https://github.com/ROCm/HIPCC/pull/145 !!!


New Script Type Update_token

Hi Mustafa,
Today I installed a fresh jans VM. when I tried to get list of custom scripts I got the error of script Type Update_token. Please see attached screen shot.
I think its because of a new script type added of type Update_token.

swagger client update fixed this issue


Backport #6177 - fix #6174 #5570: merging new entities should also trigger prepersist lifecycle callbacks with the merged data

Backport of #6177

Travis looking good!


feat: add @remix-run/google-cloud-functions package

This is an adapter for google cloud functions. We believe that it works with gen1, gen2 and firebase http functions on google cloud.
The implementation is extremely similar to the express adapter. The main differences are that this uses types from @google-cloud/functions-framework and in createRemixRequest the rawBody parameter is used:
  if (req.method !== "GET" && req.method !== "HEAD") {
    init.body = req.rawBody;
  }

The latter is needed because GCF includes some pre-processing that already parses POST, PUT etc and sets req.body into an object - but Remix isn't expecting that to have happened and ends up blocking indefinitely inside busboy or something like that. Using rawBody provides a URL encoded buffer - which is one of the body types that Remix is expecting.

Hi everyone!
Thanks for all the feedback, this was left without any feedback for a long time so it's caught me a bit off-guard.
I will try and take a look at this and update it as soon as I can

I've opened https://github.com/eastlondoner/remix/pull/1 which adds the unit tests from the now-closed #3397

@MichaelDeBoey I think this is ready for review again when you have time :-)

Approved from me other than

https://github.com/remix-run/remix/pull/2266#discussion_r892156729
https://github.com/remix-run/remix/pull/2266#discussion_r892158040

It would be nice to have a unit test that covers an error being thrown as per my comment at

https://github.com/remix-run/remix/pull/2266#discussion_r890312851

...however off the top of my head I don't have a suggestion on how to create this test 😄

@MichaelDeBoey do you have the ability to re-run the failing Github Action ? It's failing with a timeout while installing packages which I am pretty sure is a build system problem not a problem this PR has introduced. But I cannot see a place where I can trigger the CI to re-run

Actually moving back to draft status - we did a manual test with real deployment google cloud functions and we're not sure that this is working - hold on while we check ourselves

Update from our testing (thanks @danny-yu):
Deploying with regular google cloud functions
If we deploy a remix app using this adapter to a google cloud function called 'remixApp' like this:
gcloud functions deploy remixApp --runtime nodejs16 --trigger-http --source . --allow-unauthenticated

then we get a function that we can visit at a url something like https://us-central1-<my project id>.cloudfunctions.net/remixApp
notice that google cloud functions adds a base path /remixApp.
The remix app doesn't know about the base path and all <Form> and similar actions in the remix app are immediately broken because they don't include it. A simple Form tries to post to https://us-central1-<my project id>.cloudfunctions.net/ which gets back a google cloud permissions error.
The google cloud function behaviour is documented here: https://cloud.google.com/functions/docs/calling/http
this will probably throw a lot of people who try to use this adapter off because this is the most straightforward way to check an app after deployment but what they will experience is something that looks OK but breaks unexpectedly on any form submission - not a good experience
I know that there are other ways to use this

using google cloud functions 'gen2' doesn't have this problem, the deployed function has a unique url without an extra path - we are currently checking that this adapter works in that situation.
deploying using firebase functions  - I don't know what happens if you use firebase functions, @penx might know ?  Otherwise @danny-yu could investigate.
I don't know if there are deployment options to combine gen 1 functions with other google cloud infrastructure like load balancers that might fix this - but it would affect users trying out the simplest use case.

I have a Remix application deployed to Firebase functions v1 and v2 using an adapter almost identical to this, which I published to npm at https://www.npmjs.com/package/remix-google-cloud-functions
I have this in a draft PR against the Remix Firebase example at https://github.com/remix-run/remix/pull/3398
The things that may be of help:

All incoming http requests go to Firebase hosting which has a rewrite rule in firebase.json to pass all requests other than /public/* to the function:

    "rewrites": [{
       "source": "**",
       "function": "remix"
     }]

from what I remember, when I initially deployed the function I had to set some permissions on it so that it will run for unauthenticated users, but after I'd done this once it didn't come up again. When I next get some time, I'll run through the process again and add add it to the readme for #3398.

When deploying via the Firebase CLI, it said that v2 functions need to be lowercase, I'm not sure if that's a Firebase specific thing though.

great update @penx
I am very convinced this works for google cloud functions provided the correct configuration / deployment - it's just slightly sad that the most basic of google cloud function deployments will see limited functionality.
Some thoughts:

is there a good place to document google cloud deployment configurations that do work and those that don't work?
what about inspecting the host header of incoming requests and logging a warning in the adapter if we see *.cloudfunctions.net?

Promoting this back to a PR though - I'm in favour or merging as-is

@MichaelDeBoey think this is ready for another review, in case you missed it 🙂

@MichaelDeBoey this is updated ready for review - I've removed the error handling.
There doesn't seem to be definitive documentation from google regarding error handling so we carried out quick tests and it seems that thrown errors and rejected promises are handled by google cloud functions gen 1 and gen 2 both returning a 500 error in good time when the result of createRequestHandler is awaited so my claims about hanging requests must have been based on cases where I wasn't correctly awaiting the response, sorry.
our tests if you are interested:
https://gist.github.com/danny-yu/65e52467123991e09cab03bf44ec5925
https://gist.github.com/danny-yu/de77a8508781137f544bb4b1de623055

@MichaelDeBoey I think we made the last changes that you requested, is there anything else ?

@eastlondoner Could you please rebase your branch onto latest dev & resolve conflicts?

@MichaelDeBoey rebased and green 😀

hi @MichaelDeBoey is there anything else needed on this PR ?

@eastlondoner The team will look into this PR whenever they have the time for it, but new adapters are not high on the priority list for the moment.

thanks for the update :-)

We aren't looking to add new adapter packages for the time being, though we encourage hosts to maintain and promote them! If anything changes on that front we will reopen this.

We aren't looking to add new adapter packages for the time being, though we encourage hosts to maintain and promote them! If anything changes on that front we will reopen this.

This seems to be a wise move. Can you please make this "official", for example by adding a note in the the contributing guidelines or in the documentation.

Could be documented in https://github.com/remix-run/remix/blob/main/docs/other-api/adapter.md as well 👌🏼

FWIW this doubles as a Firebase adapter, and there was a ticket for this, though now closed #125.
I have published the adapter I made in #3398 to npm, from my branch at https://github.com/penx/remix/tree/firebase-adapter:

https://www.npmjs.com/package/remix-google-cloud-functions
https://www.npmjs.com/package/remix-firebase

Though these haven't seen much activity as I was hoping this PR would go in.
I'll revisit these and release them as v1 with their own repository at some point.

@penx ping us on discord when it's done, we'll add links in the documentation


Typescript

Hi! Great library.
Can I use it with typescript?

Adding typings is on my todo list but didn't have the time yet. Altho, you should be able to use it without any issue by using your own (note that intro.js have some typings that should be reusable for anything directly related to intro.js but I didn't try them yet).


code script for macOS not working

VSCode Version:
OS Version:

Steps to Reproduce:

open terminal and type "code -enter" you'll get an error

it basically brings up a python error.
if you remove the python function and the CONTENTs formula like this:
codePath=$(which code)
codeDir=${codePath:0:${#codePath}-5}
CONTENTS="$codeDir/../../.."
it will work just about fine

Does this issue occur when all extensions are disabled?: Yes/No

not sure what you mean. when I enter code -enter, I get
Warning: 'e' is not in the list of known options, but still passed to Electron/Chromium.
Warning: 't' is not in the list of known options, but still passed to Electron/Chromium.

What error do you see wen you enter that and what do you expect to see isntead?

not sure what you mean. when I enter code -enter, I get
Warning: 'e' is not in the list of known options, but still passed to Electron/Chromium.
Warning: 't' is not in the list of known options, but still passed to Electron/Chromium.

What error do you see wen you enter that and what do you expect to see isntead?

well it didn't work on mine.. -enter was supposed to mimic the "enter" key...
anyway.. for me it only worked by calling Electron directly but not as everywhere proposed "code" (which points to Electron).. and with my fix it works for everyone

well it didn't work on mine.. -enter was supposed to mimic the "enter" key...
anyway.. for me it only worked by calling Electron directly but not as everywhere proposed "code" (which points to Electron).. and with my fix it works for everyone


Global options mysteriously not applied

Describe the bug
When {suspense: true} is enabled globally, my consuming component is not suspending unless I explicitly enable it again in the useQuery() options
Your minimal, reproducible example
https://github.com/joshribakoff-sm/rq-bug/tree/no-suspense
Steps to reproduce
Wrap app in query client:
const queryClient = new QueryClient({ suspense: true });

Try to consume it and don't change any options:
  const result = useQuery(
    "foo",
    async () => {
      console.log('query ran')
      await new Promise((res) => setTimeout(res, 10000));
      return () => "foo" + Math.random();
    },
    // no suspense?
  );

Expected behavior
Component should suspend
How often does this bug happen?
No response
Screenshots or Videos
No response
Platform
macOS, chrome
react-query version
3.34.16
TypeScript version
No response
Additional context
This is not happening in my real project as far as I know, it's something I only noticed while building the reproduction repo for my previous bug report #3432

Thanks, I am using Typescript in my main project and only used JS to produce the previous bug report, which explains:

This is not happening in my real project as far as I know, it's something I only noticed while building the reproduction repo for my previous bug report https://github.com/tannerlinsley/react-query/issues/3432

Sorry for the false report :)


nes: fix memory access parameter handling

Codecov Report

Merging #15 (6677920) into main (6bb4822) will increase coverage by 0.62%.
The diff coverage is 61.53%.

@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   39.64%   40.27%   +0.62%
==========================================
  Files          60       60
  Lines        5211     5207       -4
==========================================
+ Hits         2066     2097      +31
+ Misses       3086     3051      -35
  Partials       59       59


Mention the need for a dev toolchain in the package readme.

This PR modifies the package readme file to explicitly state that a development toolchain is required. Getting Started continues to describe the limited Swift 5.10 support offered by swift-testing.
I've also updated references to "development toolchain" in other files to explicitly say "trunk development toolchain".
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

No source changes, just docs. Merging without CI.


[consideration] Sniper Crosshair disabled when Zoom?

Hello Daniel,
May I ask you why you hide the Sniper Cross on isScoped?
Isn't it better to remove the check to have the Sniper Cross always visible even in Scoped situations?
https://github.com/danielkrupinski/Osiris/blob/fd2e13ac29575badd553ccede3e61c5592db21dd/Osiris/Hacks/Misc.cpp#L79
Wouldn't it be enough to have the SniperCrosshair always visible?
showSpread->setValue(config.misc.sniperCrosshair ? 3 : 0);

Why would you need a crosshair when you're scoped anyways? I believe it's meant for noscoping

Why would you need a crosshair when you're scoped anyways? I believe it's meant for noscoping

thats why he wants it gone when he scopes..

I find the Crosshair very useful because it doesn't blure when you move.