add more documentation

[x] what needs changing after creating a new plugin from https://github.com/JetBrains/intellij-platform-plugin-template
[x] how are releasenotes being created i.e. cake.intellij.recipe ⇋ GRM ⇋ gradle (rather: patchPlugin-task in build.gradle.kts)
[x] how description is being created (or not)
[x] how the channel is determined on push
[x] extra steps in github-actions (gradle/wrapper-validation-action, actions/setup-java, more caching)
[x] document appVeyor builds

fixed in https://github.com/nils-a/Cake.IntelliJ.Recipe/commit/b64a8077ef311f3a2bb31350aca0e19d63ccbf14

fixed in https://github.com/nils-a/Cake.IntelliJ.Recipe/commit/b64a8077ef311f3a2bb31350aca0e19d63ccbf14


Support for none standard Dalamud UI Scale

On higher resolutions some users might increase the UI scale to make text readable, which could cause the card collection overlay to look like this: (1.3 scale)

Would be great if actual size could adjust based on dalamud ui scale, thanks!

Didn't know it's a thing.
Fixed in <IP_ADDRESS>


Internationalization

Pushed some commits (available in v0.8.2) for Datetime internationalization.

Wonder what your thoughts are on this in terms of copy/text.
The key goals I see for a solution:

keep components modular and able.to only depend on what they require
avoid having to send to the client all languages on initial rendering (perhaps set a default and then Ajax to load other language strings)

It would be ideal to be able to keep the strings with each component and on build create the various files per lang with all the strings from each component and inject a default language into the initial code pack.

@mikeburgh I'm using kazupon's dynamic locales in my App.vue (v0.8) created hook.
@rstoenescu there is a nice set of i18n features that was developed for meteor-vue.

@laurentpayot yes, kazupon is the right way to go for i18n. However we're not talking about this. Quasar requires i18n translations for a few components, otherwise, for example, you're going to get stuck with "OK" and "Cancel" buttons no matter the language :) This will soon get fixed.

On the loading translations bit. I would say, only the currently selected language of the user (or default for guests, if you have them), should be loaded initially. I'd also venture to say, only the language phrases needed for the currently rendered components should also be loaded. Not sure if that matters or is feasible, but it is how I think i18n and L10n systems should work to be effecient as possible, especially for mobile.
Scott

@smolinari We're on same page here. Thank you for your thoughts!

Is 24hr format part of it or should be referenced in DateTime issues?
Do we have 24hr format written down somewhere? It is important.

Just added a ticket for it #271
It's high on my priority list at the moment.

Just a few components use some labels that need translated, but managed to add props for them, so developers can use whatever labels they want to replace default ones.
It seems overkill for dev to handle Quasar i18n files just for a few labels. If in the future more components will require this, then an i18n system will be put in place. Until then, using props will suffice.


[BOT PR] to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

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

Needs approval from an approver in each of these files:

OWNERS

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

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0303c46e28fa9e610aed678920aeac16ddf9354e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0303c46e28fa9e610aed678920aeac16ddf9354e
link
/test istio-unit-tests

prow/e2e-mixer-no_auth.sh
0303c46e28fa9e610aed678920aeac16ddf9354e
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add "No Visual Basic templates" to the "Preview 1 limitations" section

The vsix download page said "C# and C++/WinRT are the only languages supported at this time". But this limitation was not listed in this page.

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

ID: 49f58672-73e9-87bb-d7f0-8d67b6878e20
Version Independent ID: 94fc53da-622a-93c8-a4c8-4faaf7e442b8
Content: WinUI 3.0 Preview 1 (May 2020)
Content Source: hub/apps/winui/winui3/index.md
Product: uwp
Technology: apps
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Thanks @Nukepayload2. The topic has been updated with the suggested detail.


Testing-edit-update-clear

In this pull request the following was done:

The edit, status update and clear functions were simplified so that the logic was seperated from the dom manipulation.
Unit test were written focusing solely on the inner logic of the edit, status update and clear functions.
After writing unit tests validations was added to functions, in order to get the unit tests to pass.

Invalid Code Review Request ❌
Hi @dvcorreia1984 ,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
You have submitted a project, that doesn't have any commits from your partner. This is a pair-programming activity..
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


fix(converter): interpolate encoded query string params

closes #11

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support arrays to be passed to withQuery

Describe the feature
Currently, withQuery only allows objects to be passed, which means that you cannot do things like: example.com/path?foo=1&foo=2&foo=3 because objects cannot have multiple keys with the same name. It would be good to allow arrays to be passed to withQuery.
Additional information

[X] Would you be willing to help implement this feature?

I saw that you are able to pass the object value an array which will solve my use case. i will close this issue


Guidance on passing and parsing arguments to script

@cauldnz Thank you for your contribution. (Did you used to do some Workflow Foundation? If so I think I brought some protein bars for you one time from here in the US)
@DavidStaheli Can you review this PR and then let me know and I will merge it.

Finally fixing this, I added it to https://github.com/MicrosoftDocs/azure-devops-docs-pr/pull/195

heh... @steved0x.... Sorry. Had missed the chatter above. Yes... did WF stuff a way back when... Probably back in the days when Paul Andrew was on the team. Been around the block a few times since and now blue-badger based in sunny Singapore!.


Current State

Target Languages:

[x] Unmanaged C#
[x] C
[x] Swift

Types:

[x] Classes
[x] Structs
[x] Enums
[x] Primitives
[x] Booleans
[x] Interfaces
[x] Delegates
[x] Events
[x] "Typed" Arrays
[x] out method parameters
[x] ref arguments
[ ] ref arguments in delegates
[x] ref return values
[ ] ref return values in delegates
[x] in arguments
[ ] in arguments in delegates
[x] Generic Methods (Only experimental support)
[x] Generic Types (Only experimental support)
[x] Tuples (Comes free with generic types support)
[ ] Managed Pointers (ie. char*)

Language Constructs:

[x] Constructors
[x] Destructors
[x] Methods
[x] Extension Methods
[x] Properties (Getters/Setters)
[x] Fields (Getters/Setters)
[x] Exception handling
[x] Overloads
[x] Type Conversion (Casting/as)
[x] Type checks (is)
[x] Boxing/Unboxing of primitives (Int32 <-> System.Object)
[x] Indexers
[ ] Nullability
[ ] Async stuff

Misc:

[ ] Stable ABI (We're far from the point where we can ensure that the generated code will be binary compatible from one version of the generator to the next)

@damianmehers have you seen this repo?


Fix non-concurrent index detection

Fixes https://github.com/Artur-Sulej/excellent_migrations/issues/33
The previous code expected index/3 to be wrapped by create, drop, etc letting you bypass the check by defining the index in a pipeline. The change here will match all calls of index/unique_index independently of the structure

NICE 🎉

My change introduced an issue where this ast would fail
> Code.string_to_quoted!("index")
{:index, [line: 1], nil}

I think is_nil(args) should be enough to prevent it though if you do index() it'll trigger a warning. This could be helped with something like https://github.com/Artur-Sulej/excellent_migrations/pull/29 so this check doesn't target backfill code

My change introduced an issue where this ast would fail
> Code.string_to_quoted!("index")
{:index, [line: 1], nil}

I think is_nil(args) should be enough to prevent it though if you do index() it'll trigger a warning. This could be helped with something like Artur-Sulej#29 so this check doesn't target backfill code

I think we could get around this issue by adding a catch-all clause for detect_index_not_concurrently that returns an empty list, no?
With this the AST wouldn't fail, leaving only the warning

@cgerling sorry I wasn't clear, the fix for the crash is here: https://github.com/scorebet/excellent_migrations/pull/1/commits/89b19b5b1c0876227468d391aca5e85067a01451
now what remains is the possibility someone, outside migration context, defines and calls an index function (regardless of arity). This would trigger a false positive as the script looks for concurrently: true, in other words
def index do
  ...
end

# this would trigger a warning
index()


Resize according to container when image is loaded already?

I'm having a really hard time figuring out how to make this plugin fit its container element when I'm resizing the browser. I have it setting the viewport and boundary on load, but that's pretty much it.
$('.croppie__tool').croppie({
    viewport: {
        width: cropWidth,
        height: cropHeight,
        type: 'square'
    },
    boundary: {
        width: columnWidth,
        height: 230
    },
    exif: true,
    showZoomer: false
});

Any assistance would be greatly appreciated. Thanks!

I think we're missing a feature that you need.  A while ago, we decided it wasn't worth the effort to make options configurable after initialization.  For example, with typical jquery plugins, you would say $('.my-el').croppie('option', 'viewport.width', 300) or something along those lines.
Instead, we decided to just destroy croppie instances, and reinitialize them.  So you should be able to call $('.croppie__tool').croppie('destroy') and then initialize again with your new dimensions (and bind data).
I'm guessing this isn't going to be very efficient for you, but it will at least give you something to get started.

I am able to crop images from 50K to 10 mb with no problem. When I transfer the trigram to the VPS server at Hostgater, The cropping process never finishes. The “ bubbles” never stop.
I configer the following setting on my VPS Server
max_execution_time => 300,
memory_limit => 1000,
post_max_size => 1000,
max_input_time=>60
any thing else i missed please help.
check the problem on following link https://www.boats4sale.com/b4sell/individual/dashbaord/editListing/684
username => Ajay@1234567890 ,password => Ajay@1234567890 what kind of setting require for big images crop on server please help


[Bug] isStandalone helper does not work with Angulars new default value

Is this a regression?
Yes
Description
Angular changed the default value of the standalone component (/pipe/directive) property.
It's now true instead of false and the isStandalone helper does not support this:
https://github.com/ng-doc/ng-doc/blob/main/libs/builder/helpers/angular/is-standalone.ts#L17
If the property is not available, it should return true instead of false.
Steps to reproduce

Have a playground with a demo component
Remove the standalone: true property from the component decorator
😢

Screenshots

Please provide the exception or error you saw
✘ [ERROR] NG8002: Can't bind to 'disabled' since it isn't a known property of '<component-selector>'.
1. If '<component-selector>' is an Angular component and it has 'disabled' input, then verify that it is included in the<EMAIL_ADDRESS>of this component.
2. If '<component-selector>' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.
3. To allow any property add 'NO_ERRORS_SCHEMA' to the<EMAIL_ADDRESS>of this component. [plugin angular-compiler]

    ng-doc/docs/guides/<component-route>/docs/playgrounds.ts:11:203:
      11 │ ...bounceTime']" [disabled]="properties['disabled']" [smallBorderR...

OS
Windows
Browser
Chrome
Node version
20
Anything else?
No response

Thanks!  Yea it makes sense, I missed that, I'll release a fix in a few minutes!

Thank you 🙏🏼😊

:tada: This issue has been resolved in version 19.0.1 :tada:
The release is available on:

v19.0.1
GitHub release

Your semantic-release bot :package::rocket:


nz-select in nz-modal

Hello,
I have an issue with the size of nz-select inside an nz-modal.
You can reproduce it with the nz-modal example and adding an nz-select component inside.
Thank you

I have encountered the same problem. Have you solved it?

Yes I add this style="width:-webkit-fill-available;"

and it works for me.

Should be fixed in the newer version


feat: Initial commit of a mutual exclusivity node.

This is the first commit of using the Highs solver to model binary variables, and using those variables to apply a mutual exclusivity constraint. I.e. flow is allow through up to N nodes at a time.
TODO:

[ ] Add support in pywr-schema.
[ ] Implement in CLP/CBC solver.
[ ] Add additional tests.

This starts to resolve issue #187.

Re CLP/CBC.
CLP is linear programme only. CBC is the MILP solver that uses CLP internally. We have two choices:

Retain the CLP solver for LP only models, and add a CBC solver for LP & MILP models.
Replace the CLP with a CBC solver.

I suggest (2) is preferrable as long as CBC isn't significantly slower for the LP only case.
It is probably worthwhile doing this switch in another issue/PR before completing this one.

Also, the tests will fail on the lack of support in CLP first.

I've created a project to try to organise how to move this forward: https://github.com/orgs/pywr/projects/1


Fix another bug introduced in #671

Description
This PR fixes a bug introduced in #671, which caused the keycloak client to be never initialized.

/deploy-staging

/merge


[Snyk] Security upgrade mlflow from 1.30.1 to 2.15.0

Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

examples/transformer/feast-enricher-transformer/requirements.txt

⚠ Warning
```
merlin-sdk 0.12.2 requires mlflow, which is not installed.
botocore 1.33.13 has requirement urllib3<1.27,>=1.25.4; python_version < "3.10", but you have urllib3 2.0.7.
</details>

---

> [!IMPORTANT]
>
> - Check the changes in this PR to ensure they won't cause issues with your project.
> - Max score is 1000. Note that the real score may have changed since the PR was raised.
> - This PR was automatically created by Snyk using the credentials of a real user.
> - Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

---

**Note:** _You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs._

For more information: <img src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiIyNjljOTA5Ny0wOGZjLTRlYzQtYmU5YS1lZWEwNGQ2ZWYzZDYiLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6IjI2OWM5MDk3LTA4ZmMtNGVjNC1iZTlhLWVlYTA0ZDZlZjNkNiJ9fQ==" width="0" height="0"/>
🧐 [View latest project report](https://app.snyk.io/org/1.-demo-env/project/7e478862-9503-4250-a3a8-093a4f7a3ec6?utm_source=github-enterprise&utm_medium=referral&page=fix-pr)
📜 [Customise PR templates](https://docs.snyk.io/scan-using-snyk/pull-requests/snyk-fix-pull-or-merge-requests/customize-pr-templates)
🛠 [Adjust project settings](https://app.snyk.io/org/1.-demo-env/project/7e478862-9503-4250-a3a8-093a4f7a3ec6?utm_source=github-enterprise&utm_medium=referral&page=fix-pr/settings)
📚 [Read about Snyk's upgrade logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities)

---

**Learn how to fix vulnerabilities with free interactive lessons:**

🦉 [Arbitrary Code Injection](https://learn.snyk.io/lesson/malicious-code-injection/?loc=fix-pr)

[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"mlflow","from":"1.30.1","to":"2.15.0"}],"env":"prod","issuesToFix":[{"exploit_maturity":"No Known Exploit","id":"SNYK-PYTHON-MLFLOW-7577152","priority_score":164,"priority_score_factors":[{"name":"confidentiality","value":"low"},{"name":"integrity","value":"low"},{"name":"availability","value":"high"},{"name":"scope","value":"unchanged"},{"name":"exploitCodeMaturity"},{"name":"userInteraction","value":"none"},{"name":"privilegesRequired","value":"none"},{"name":"attackComplexity","value":"low"},{"name":"attackVector","value":"network"},{"name":"epss","value":0.01055},{"name":"isTrending","value":false},{"name":"publicationDate","value":"Thu Aug 01 2024 14:24:52 GMT+0000 (Coordinated Universal Time)"},{"name":"isReachable","value":false},{"name":"isTransitive","value":true},{"name":"isMalicious","value":false},{"name":"businessCriticality","value":"high"},{"name":"relativeImportance","value":"high"},{"name":"relativePopularityRank","value":98},{"name":"impact","value":7.84},{"name":"likelihood","value":2.08},{"name":"scoreVersion","value":"V5"}],"severity":"high","title":"Arbitrary Code Injection"}],"prId":"269c9097-08fc-4ec4-be9a-eea04d6ef3d6","prPublicId":"269c9097-08fc-4ec4-be9a-eea04d6ef3d6","packageManager":"pip","priorityScoreList":[164],"projectPublicId":"7e478862-9503-4250-a3a8-093a4f7a3ec6","projectUrl":"https://app.snyk.io/org/1.-demo-env/project/7e478862-9503-4250-a3a8-093a4f7a3ec6?utm_source=github-enterprise&utm_medium=referral&page=fix-pr","prType":"fix","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["updated-fix-title","pr-warning-shown","priorityScore"],"type":"auto","upgrade":[],"vulns":["SNYK-PYTHON-MLFLOW-7577152"],"patch":[],"isBreakingChange":false,"remediationStrategy":"vuln"}'

🎉 Snyk hasn't found any issues so far.
✅ code/snyk check is completed. No issues were found. (View Details)


Disable go-critic hook

Broken: https://github.com/go-critic/go-critic/issues/1388

what are you replacing it for?

what are you replacing it for?

I'm not replacing it with anythin, just temporarily disabling it, since it's broken.
Fallback to older binary version doesn't help, so I assume this is transitive dependency issue.
Disabling it to prevent PR-validation failures, that can (or not) start happening after next builder image roll out.


Add support for clang-tools v16

I tried using this action from the main branch with version: 16 since clang-tools v0.8.0 added support for clang tools v16 but I got no output (similar to the problem with v15 in #139). Please see this workflow run in mirenradia/cpp-linter-action-test. The output I expect can be seen from the preceding workflow run.

Are you using ubuntu-latest? If so, there's a known seg fault problem for that platform when using the static binaries downloaded with clang-tools-pip. See compatibility chart in that repo's README.
Alternatively, you can ensure that the dynamically linked binaries are installed before the cpp-linter step.

Are you using ubuntu-latest? If so, there's a known seg fault problem for that platform when using the static binaries downloaded with clang-tools-pip. See compatibility chart in that repo's README.

I am using ubuntu-22.04 (which I think is the same as ubuntu-latest). I did see that note but I don't get a segfault with v14 or v15 which the table seems to suggest.

Alternatively, you can ensure that the dynamically linked binaries are installed before the cpp-linter step.

Yes, this was the next thing I tried and it works as expected (see [this workflow run](https://github.com/mirenradia/cpp-linter-action-test/actions/runs/4936314326.

I think I don't get a segfault because clang-tools tries to install the normal clang-tidy-xx packages but these aren't available for v16.

I think I don't get a segfault with v14 and 15 because clang-tools installs the normal clang-tidy-xx and clang-format-xx packages but these aren't available for v16.

I assume "normal packages" means those acquired via apt-get. Hmm, I see that apt-get update is done beforehand. I'm not sure this is a problem we can solve. Usually, the PPA is updated by LLVM for new releases.

clang tools v16 but I got no output (similar to the problem with v15 in https://github.com/cpp-linter/cpp-linter-action/issues/139).

IIRC, that issue was suffering from a seg fault as well. The typical seg fault message may not be showing up in the log, but I suspect it is still seg faulting for v16.

I assume "normal packages" means those acquired via apt-get. Hmm, I see that apt-get update is done beforehand. I'm not sure this is a problem we can solve. Usually, the PPA is updated by LLVM for new releases.

The official Ubuntu repositories don't always have the latest LLVM releases so one solution might be to add one of the LLVM PPAs (described here). Of course, this could be done manually as I have done but it would be nice if this was done automatically somewhere in these lines.

A PR would be welcome. Looking at that commit, the only changes we would require is saving the install script to the $GITHUB_ACTION_PATH (which is different from the runner's workspace). So, something like
wget https://apt.llvm.org/llvm.sh $GITHUB_ACTION_PATH/llvm_install.sh
chmod +x $GITHUB_ACTION_PATH/llvm_install.sh
sudo $GITHUB_ACTION_PATH/llvm_install.sh {{ version }}


document historical reindexing process for abci events

We need figure out the process for creating an ABCI event database from scratch. The process should look something like:

Download historical archive of chain state
Set up new node to run in archive mode
Sync node up to upgrade boundary
Update versions, rinse, repeat, until current height

Let's do it manually for now, working from private backups. When I last tried this, getting cometbft to "replay" blocks to pd over ABCI was fairly simple: unfortunately, that replay process did not result in ABCI events being emitted to a newly created sidecar indexing db. Why is that? Is there some other process we should use to handle this use case?
Closely related to #4525 & #4526.

Syncing a new cometbft instance will solve the event emission problem.

Progress!  The process so far is a bit clunky, but it's definitely tractable.
We want to reindex events on the penumbra-testnet-deimos-8 chain. There have been several upgrades already, so we'll need to rewind several versions to do that. Let's go back to 0.75.1. That version won't run out of the box with archived node state, because the halt-height will have been reached, causing pd to exit. So, branch from the v0.75.1 tag, increment the (now-defunct) const TOTAL_HALT_COUNT by one, and build that version of pd, saving it as pd-0.75.1-plus-halt. Use that to set up the initial version of the nodes.

Restore node state from a private backup. This backup was created by tarring up the entire ~/.penumbra/testnet_data/node0 directory on a fullnode. (Related we should sanitize this archive and host it, see #4525).
Install pd-0.75.1-plus-halt to the system. Run it.
Configure CometBFT v0.37.5 with extra options: --p2p.pex=false --p2p.seeds='' --moniker archive-node-1. Make sure its CometBFT RPC is accessible to a second node, soon to be created.
Set up another node with pd-0.75.1-plus-halt and cometbft v0.37.5. Join this node to the first node via pd-0.75.1-plus-halt testnet join <url>. Run its cometbft with extra options:: --p2p.pex=false --p2p.seeds='' --moniker='reindexer-1' --p2p.persistent_peers='tcp://<node_1_id>@<node_1_ip>:<node_1_port>'
Edit node2's cometbft config [tx_index] block and set indexer="psql" & psql_conn="<POSTGRES_DB_URL>". Run it.

Now you should see blocks syncing from node1 -> node2, and more importantly, node2 -> postgres. Once node2 catches up the node1's halt height, the ABCI event db will have blocks 1 -> halt height. Then it's time to shift versions, redeploy backups, and sync again. That piecemeal process, unscripted, should get us full recovery of historical events. Once we have a db, we can shove that behind the relevant frontends.

Update on progress: I've successfully reindexed blocks 0 through 219220 on penumbra-testnet-deimos-8, using pd v0.75.1, using the process described above. Next up, we need to reindex events from height 220220 to 326700, which matches pd versions 0.76.x to 0.77.x. To do so, we must perform some version gymnastics, related to the overhaul of halting logic in #4373: specifically, we want to run v0.76.0, but restoring from a backup on that state will not let us resume the chain. So we'll briefly use v0.77.0 to edit the halt-bit, then fall back to v0.76.0 to sync.
So the steps for reindexing 220220 to 326700 on penumbra-testnet-deimos-8 are:

Restore node0 state from a private backup of pre-77 state. This backup was created by tarring up the entire ~/.penumbra/testnet_data/node0 directory on a fullnode.
Install both v0.76.0 and v0.77.2 pd to the system.
Run pd-v0.77.2 migrate --ready-to-start to disable the halt bit.
Configure CometBFT v0.37.5 with extra options: --p2p.pex=false --p2p.seeds='' --moniker archive-node-1. Make sure its CometBFT RPC is accessible to a second node, soon to be created.
Set up second node with pd-v0.76.0 and cometbft v0.37.5. Join this node to the first node via pd-v0.76.0 testnet join <url> --archive-url https://snapshots.penumbra.zone/testnet/pd-migrated-state-75-76.tar.gz. The archive URL is mandatory. Run its cometbft with extra options:: --p2p.pex=false --p2p.seeds='' --moniker='reindexer-1' --p2p.persistent_peers='tcp://<node_1_id>@<node_1_ip>:<node_1_port>'
Edit node2's cometbft config [tx_index] block and set indexer="psql" & psql_conn="<POSTGRES_DB_URL>". Run it.

You should now see blocks streaming from node1 to node2, and ABCI events for each block added to the postgresql db. Wait for the next chain halt, at height 326700, and then it's time to update versions once again.

Nice!


feat: added typescript support

tsup for bundling
vitest for testing
added jsDoc for explanation and usage example
updated README.md
updated package.json for build and test script
updated package.json for main, typings, and module
support export:

import fakerKh from 'faker-kh'
import { fakerKh } from 'faker-kh'
import { address, noun, month } from 'faker-kh'

Issue: #1

Cool! Thank you for your contribution.


FLINK-16024 [jdbc-connector] JDBC Filter Pushdown

Implement Filter Pushdown for JDBC connector source, used pre-existing visitor to parse Flink SQL to JDBC SQL.
Expose the visitor via JDBCDialect to allow customization by user
JIRA: https://issues.apache.org/jira/projects/FLINK/issues/FLINK-16024?filter=myopenissues

What is the purpose of the change
Implement Filter Pushdown in JDBC Connector Source, this is an optimization that will avoid scanning the whole table into Flink when we only need a subset of the table.
Brief change log

Implement a SQL Visitor that converts Flink SQL AST to JDBC SQL String
Implement FilterPushdown in JDBCDynamicSource using the Visitor
Add tests for both Visitor and JDBCDynamicSource

Verifying this change
This change added tests and can be verified as follows:
(example:)

Added Integrated Test in JdbcDynamicTableSourceITCase to make sure there is no regression on SQL Filtering
Added JdbcFilterPushdownVisitorTest to verify we are generating SQL correctly

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

@flinkbot run azure


Adding a new small section in the right bottom for the stocks rate/ news headlines

We will be calling a free stock api to fetch the data and further dispaly it
API: https://www.alphavantage.co/documentation/

How is stock data relevant in a social media application made for wildlife enthusiasts?
Explain.

How is stock data relevant in a social media application made for wildlife enthusiasts? Explain.

@SwanandD121, yes you are correct, we can get latest wildlife news over there, which would be more insightful for our users.


CSS in JS syntax highlight is broken on Prism

Describe the bug
I try to highlight styled-components code with function, code got unexpected.
To Reproduce
Steps to reproduce the behavior:

append like those code

const Item = styled.div`
  grid-row: ${({ row }) => row};
  grid-columns: ${ ({ col }) => col};
`

wrap with Prism syntax highligter

    <SyntaxHighlighter language={"js"} style={prismStyle}>
      {code}
    </SyntaxHighlighter>

got unexpected code

const Item = styled.div`
grid-row: ___CSS_0___;
grid-columns: ___CSS_1___;
`

Expected behavior
I want to get like higlight.js version result.
const Item = styled.div`
  grid-row: ${({ row }) => row};
  grid-columns: ${ ({ col }) => col};
`

Screenshots
https://stackblitz.com/edit/react-tjouhm

Desktop (please complete the following information):

OS: Mac
Browser chrome,
Version: 77.0.3865.120

Smartphone (please complete the following information):

Device: iPhone6 SE
OS: iOS 13.1.3
Browser chrome
Version 78

Additional context
Add any other context about the problem here.

added smartphone part

i'm experiencing this, too.
const code = `
const Photo = styled.img\`
  width: \${props => props.scale * 368}px;
  height: \${props => props.scale * 200}px;
  border-radius: 4px;
  box-shadow: 0 7px 20px 0 rgba(0, 0, 0, 0.12);
  object-fit: cover;
  object-position: top;

  margin-left: \${props => (props.offset === "true" ? props.scale * 7 : props.scale * 87)}px;
  margin-right: \${props => (props.offset === "true" ? props.scale * 80 : 0)}px;
\`;
`;

will result in
const Photo = styled.img`
  width: ___CSS_0___px;
  height: ___CSS_1___px;
  border-radius: 4px;
  box-shadow: 0 7px 20px 0 rgba(0, 0, 0, 0.12);
  object-fit: cover;
  object-position: top;

  margin-left: ___CSS_2___px;
  margin-right: ___CSS_3___px;
`;

Reproduces for me too, with
const code = `const ColoredText = styled(Typography)\`
  color: \$\{ myColor \};
\`;
`

any word on this? I love that prism is highlighting my css in js rather than just leaving it as a string. it's not blocking for me because i'm happy to just tweak my examples, but it's definitely limiting and could become annoying down the road.

Hey @jack-bliss, I'll try to take a look at this soon! Thanks for your patience.


Unhelpful warning for short BG sounds

Raised by @turpaultn
Currently using a BG sound shorter than the soundscape (which should get tiled) raises the following warning which isn't very helpful:
/anaconda3/envs/pytorch/lib/python3.6/site-packages/scaper/core.py:1433: ScaperWarning: freesound source time tuple (const, 0) could not be satisfied given source duration (2.91) and event duration (10.00), source time tuple changed to (const, 0) but was still not satisfiable, likely due to using 'normal' distribution with bounds too close to the start or end of the audio file
  ScaperWarning)

This makes sense for FG events but not BG events. Should be an easy fix to first determine if it's FG or BG and issue the relevant warning accordingly.

Might be unrelated or a diff PR, but here's another unhelpful warning that someone got:
scaper/core.py", line 1865, in generate
    jam.save(jams_path)
  File "/home/abugler/.conda/envs/nussl/lib/python3.7/site-packages/jams/core.py", line 1776, in save
    self.validate(strict=strict)
  File "/home/abugler/.conda/envs/nussl/lib/python3.7/site-packages/jams/core.py", line 1812, in validate
    valid &= ann.validate(strict=strict)
  File "/home/abugler/.conda/envs/nussl/lib/python3.7/site-packages/jams/core.py", line 778, in validate
    raise SchemaError(str(invalid))
jams.exceptions.SchemaError: -3 is less than the minimum of 0
Failed validating 'minimum' in schema['items']['properties']['value']['properties']['event_time']:
    {'minimum': 0, 'type': 'number'}
On instance[0]['value']['event_time']:
    -3

What happened here was an event_duration of 4 was requested when sc.duration = 1, thus -3. Took a while to track down so we can probably make this better. Thanks jams for validation, though! Otherwise this would have been a terrible bug.
So we can probably improve a couple warnings or catch a few more cases like above.

@pseeth I think the bigger issue here is that somewhere we're setting the event_time to -3 when we should have checks in place to ensure it never gets set to something less than 0. Can you please open a separate issue for with along with a MWE?
In the meanwhile I'll implement a fix for the unhelpful warning in a PR to address just that.


musicbot module is not importable

Please tick all applicable boxes.

[x] I am using Python 3.5.3 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x] The latest master version (release-260419)
[ ] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
When the bot starts on Windows, I get the error that these 2 guys were getting (musicbot module is not importable):

https://github.com/Just-Some-Bots/MusicBot/issues/1864
https://github.com/Just-Some-Bots/MusicBot/issues/1553

The way I solved this error was to rename the "MusicBot" folder inside the MusicBot installpath to "musicbot" all-lowercase.
As soon as I did that, the problem was fixed. I got the idea by opening "run.py" and seeing that it checked for "musicbot" subfolder, not "MusicBot".
Steps to reproduce

On a Windows 10 system, follow the official guides to install the bot for your system
After you get the error that version of discord.py[voice] in requirements.txt depends on a different pynacl version than the pynacl version mentioned on requirements.txt and pip cannot resolve the issue, delete the version requirement of pynacl and only leave the name. now retry installing requirements.txt via update.py which will work now.
Rename the subfolder "MusicBot" to "musicbot" so that run.py doesn't give you musicbot module is not importable
find and open file "voice_client.py" (use the programme "Everything" if you can't find the file) and now seach for ".replace(':80', '')" and add ".replace(':443', '')" just after it without any spaces. This will fix error gaierror(11001, 'getaddrinfo failed') where even though the bot is up and has joined your channel, it won't be able to play any music (at least from youtube which is how i used it).
Enjoy!

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.

I can't find "MusicBox" or "musicbot" in run.py?

Open run.py with an advanced notepad like notepad++
Ctrl+F to open search, and search for "musicbox"
There it is! (currently line 305) under def req_ensure_env():
However, I'm not saying to change anything inside the run.py script! I merely say, rename the actual folder.
I'm pretty sure that renaming the "MusicBox" subfolder to "musicbox" is not too complicated for most users.
But that's just my opinion.

However, I'm not saying to change anything inside the run.py script
well, I was talking about the changes in voice_client.py

and, I cant find "musicbox" in run.py?

Oh, I see where the confusion lies. I apologise. "voice_client.py" is a file in its own right.
I don't expect people know where to find it, for instance, for me it's located in: C:\Users\GiannisM\AppData\Roaming\Python\Python37\site-packages\discord\voice_client.py
That's why I suggest that people download the programme "Everything" which is a powerful search programme and they just type "voice_client.py". "Everything" will instantly locate it.
Remember however, that this fix is not for "musicbot module is not importable", it's for:
Task exception was never retrieved
future: <Task finished coro=<VoiceClient._create_socket() done, defined at C:\Users\GiannisM\AppData\Roaming\Python\Python37\site-packages\discord\voice_client.py:171> exception=gaierror(11001, 'getaddrinfo failed')>
Traceback (most recent call last):
  File "C:\Users\GiannisM\AppData\Roaming\Python\Python37\site-packages\discord\voice_client.py", line 190, in _create_socket
    self.endpoint_ip = socket.gethostbyname(self.endpoint)
socket.gaierror: [Errno 11001] getaddrinfo failed

py -3 -m pip install -U discord.py discord.py[voice] is an more easy fix for the getaddrinfo error

Yeah, that seems a straightforward solution for it.
However, for me personally, it introduces another problem: WARNING: Owner could not be found on any guild (id: {MyIDWhichIDeleted})
I'll stick to my solution for now but for the rest of the people, if this problem does not arise, your solution is definitely much better.

You need to enable this 2 options

Unfortunately I still get WARNING: Owner could not be found on any guild (id: {DeletedIDHere})
On the "Bot" tab, my permissions are exactly like yours on the picture. Pushed save and refreshed the page to be sure. They're saved.

Updated in review, make sure to go to the Discord Developer Portal and enable intents on your bot application settings

Updated in review, make sure to go to the Discord Developer Portal and enable intents on your bot application settings


Email change after association

Hi omab..
should be exist a define like CHANGE_EMAIL_AFTER_ASSOCIATION that permit this:
If I'm a normal user with email<EMAIL_ADDRESS>and my email facebook is<EMAIL_ADDRESS>In my profile - after associaiton with FB- should remain<EMAIL_ADDRESS>and not<EMAIL_ADDRESS>Thanks in advance.
Regards!

Removing social_auth.backends.pipeline.user.update_user_details from your pipeline will stop your Django User from being updated on FB (or other provider) login.
If you want the other User fields updated and just not the email field, you could replace update_user_details with a function wrapping it that removes (and restores after) 'email' from details dict before it calls the django-social-auth updated_user_details function.


Cmake windows fix

These fixes allow Couchbase-Lite-C to compile with CMake on Windows.
It consists of adding a missing include to src/CBLReplicator_Internal.hh and the removal of the unresolved external symbols (commented out from CBL.def).
Other fixes (in other repositories) are also required since certain submodules used by this project also do not compile with CMake on Windows.

CBL.def appears to be very out of date; the symbols in it are an old version of the API. Probably best to just copy CBL.exp and change it to Windows syntax (removing leading underscores.)
@borrrden has a branch fix/ci_windows_etc where he's fixing Windows/Linux issues. It will probably be merged into dev soon. You might want to check out that branch and try it.

Is this a part of your newer PR?

No, this is deprecated by the new one. Closing.


Quartiles reported wrong when out of bounds

E.g. this question has its first quartile below the lower bound, but the table reports it at "500" (the lower bound), which is incorrect.

It should instead say "< 500".
Similarly the third quartile says "1M" but should say ">1M", because it's above the upper bound.
This also applies to the median, or if the first quartile is above the upper bound, etc. etc.

Ready for testing on prod

lgtm!


fix(components-angular): add components package as peer dependency in components-angular package

Installing @swisspost/design-system-components as a direct dependency in all the consumer projects, should not be necessary anymore.

@oliverschuerch , it's added as a peer dependency, not a direct dependency. Peer dependencies will still have to be installed explicitly. Would be good if the documentation would reflect that: https://design-system.post.ch/?path=/docs/833ef689-a573-40f5-a6a6-30a999b94733--docs


CI: Force docs warnings to be raised as errors (+ fix all)

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

What does this PR do?
Fixes #1183

This PR raises all docs build warnings as errors. They are now visible in CI.
Fixed 100+ docs warnings fixed
In the future, docs will break less often because CI will throw the errors.

@Borda here

Hello @awaelchli! Thanks for updating this PR.

In the file pytorch_lightning/loggers/trains.py:

Line 221:101: E501 line too long (105 > 100 characters)

@Borda I have reduced 100+ errors and warnings to now only 4 that I am stuck with. Maybe you see a good fix for some of them?

~@Borda I have reduced 100+ errors and warnings to now only 3 that I am stuck with. Maybe you see a good fix for some of them?~

That's SUPER COOL!

not true, as the name states it is required while installing the package...

not true, as the name states it is required while installing the package...

yes you are right, otherwise install breaks.


fix typos in cancel queries

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/34d629f8acd5f5e959f27b787443187ea08b89f8/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/34d629f8acd5f5e959f27b787443187ea08b89f8/


Update members.json

widget added

OK, looking good!

amazing!
On Wed, Apr 14, 2021 at 7:30 PM Googol @.***> wrote:

Merged #53 https://github.com/hackclub/webring/pull/53 into master.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hackclub/webring/pull/53#event-4593071770, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJP3VRDGXMCE4ILBENPXJDTTIV4HHANCNFSM4236WEJQ
.

--
This email and any attachments are confidential and may also be privileged.
If you have received this message in error, please (a) notify the sender
immediately, (b) destroy this email and any attachments, and (c) do not
use, copy, store and/or disclose to any person this email and any
attachments. The sender gives no representation or guarantee with respect
to the integrity of any emails or attached files and the recipient should
check the integrity of and scan this email and any attached files for
viruses prior to opening.


Document how to best type generic functions that return a subtype of the input type

As per flow's documentation, generics in flow tracks values around.
That means that this will raise an error:
function identity<T>(value: T): T {
  if (typeof value === 'string') {
    // $ExpectError
    return '';
  }

  return value;
}

As far as I can tell, the documentation does not mention how to best type a function like this. Given that flow has first class support for generics, I'm assuming there is a way to specify that the function returns a sub-type of the input type. Specifically a way to avoid less safe typings like these:
function identity<T>(value: T): (T | string) {
  ...
}

or
function identity(value: mixed): mixed {
  ...
}

If for some reasons there is no way to accomplish this with flow, I think it would also be valuable to put that in the documentation. I've spent a multiple hours try to figure this one out.
Thanks!

Why do you consider
function identity<T>(value: T): (T | string) {
  ...
}

to be unsafe?

I wouldn't call it unsafe, but at the very least it's definitely wrong. I too am curious to know how to correctly type this because I've encountered this in the past as well (tangentially related: https://github.com/facebook/flow/issues/7197#issue-382394110).

I'm assuming there is a way to specify that the function returns a
sub-type of the input type

But your function does not do that:
type Color = "red" | "blue";
const x: Color = "red";
const y: Color = identity(x);  // hmm...

Your function (called identity, but not the identity function) may
return the empty string, which is not a member of arbitrary subtypes of
string.
There is no (intended) way to convince Flow that your “identity”
function has the type <T>(T) => T, because that would simply be
unsound.
If you really do have a value whose type is a subtype of T, then you
could just return it. If S is a subtype of T, it is always valid to
provide a value of type S where a value of type T is wanted. This is
what it means to be a subtype.

I realize that the name identity is confusing. I didn't mean to name it that way - I tweaked an example and I forgot to rename it. Sorry about that.
@dsainati1 I would call it less safe because the return type is not as fine as it could be and forces callers to add un-needed type refinements at run time to be safe. For example2 is safer than example1.
// works but number is not needed
function example1(str : string) : string {
  return str;
}
// better
function example2(str : string) : string | number {
  return str;
}

@wchargin

Your function may return the empty string, which is not a member of arbitrary subtypes of string.

I'm not sure I follow why an empty string is not an "arbitrary subtype" of string since ('' : string) is valid. Can you give me an example of what would be an "arbitrary subtype" of string?
Just to confirm, are you saying that the best way to type a function is this?
function func<T>(value: T) : (T | string) {
  if (typeof value === 'string') {
    return '';
  }

  return value;
}

If that's case, it seems not ideal that the return type of func<number> is (string | number) when clearly string cannot be returned. Is there a way with flow to type a function to say that function will return a value that is the subtype of its input value when using generics?
Let me know if you'd like me to post a less contrived example where I feel I cannot express this naturally occurring type constrain.

@jnak The thing to notice is that passing the typeof value === 'string' check doesn't actually mean that the type of value is string; there is a difference between Flow's static types and the types that exist at runtime for the purposes of checks like these. Consider what might happen if this code were to pass Flow:
type Enum = "A" | "B";
function identity<T>(value: T): T {
  if (typeof value === 'string') {
    return '';
  }
  return value;
}
let x : Enum = "A";
let y : Enum = identity<Enum>(x);

In this case, x would pass the conditional check and thus we would return '' from the function, but '' isn't a subtype of Enum, and so y would have an unsound type .

What @dsainiti1 said.

I'm not sure I follow why an empty string is not an "arbitrary
subtype" of string since ('' : string) is valid. Can you give me
an example of what would be an "arbitrary subtype" of string?

Sure; let me be more explicit. It is not the case that, for every type
T that is a subtype of string, "" is a value of type T. For
instance, "red" | "blue" is one subtype of string, but "" is not a
value of type "red" | "blue".
Your generic function needs to work for all T. Like @dsainati1 said,
just because typeof x === "string" and x: T does not mean that T
is the type string; it merely means that T is a subtype of string.

@jnak you can use function overloading
declare function func(value: string): string
declare function func<T>(value: T): T
function func(value: mixed) {
  if (typeof value === 'string') {
    return '';
  }

  return value;
}

@goodmind: Please don’t use declare in that way. What you have written
gives func the unsound type of
((value: string) => string) & (<T>(value: T) => T)

whence an unsuspecting caller may freely write
function bad<T>(value: T): T {
  return func(value);
}
const wat: "wat" = bad("wat");  // unsound!

and we’re back where we started.

@wchargin @dsainati1 Thanks for answering. It looks like I tried to simplify my actual issue too much and that ended up being more confusing than helpful. My actual issue was a "typed" generic:
function func<T : string | number>(value: T): T {
  if (typeof value === 'string') {
    return '';
  }
  return value;
}

Here I was hoping to avoid typing it like func<T : string | number>(value: T): T | string because func<string> can only return string and func<number> can only return number. Can you confirm to me that there is no better workaround for this type of situations?

function func<T : string | number>(value: T): T {
  if (typeof value === 'string') {
    return '';
  }
  return value;
}

Has the exact same issue as before.
let y : "A" = func<"A">("A")

is unsound.
If you want to have two different functions for number and string, this would be a case for overloading:
declare function func(v : number) : number;
declare function func(v : string) : string;

I know this is contrived, but of curiosity, why can't I do this?
function func<T>(value: T): T {
  if (typeof value === 'string') {
    return String(value);
  }
  return value;
}

Has the exact same issue as before.

Yes, and to be a bit more explicit: the bound <T: string | number>
does not mean that T must be either the type string or the type
number; it means that T must be a subtype of string | number, and
that it may be any such subtype, including "red" | "blue".

I know this is contrived, but of curiosity, why can't I do this?
function func<T>(value: T): T {
  if (typeof value === 'string') {
    return String(value);
  }
  return value;
}

That is sound (I believe), but the type of String(…) as declared in
the standard library is not precise enough to show it:
https://github.com/facebook/flow/blob/2c77038221739f9eab7b52b4383b7de0d815f9af/lib/core.js#L331
For your example to typecheck, String would need to have a type more
like “<T>(T) => (T <: string ? T : string)”, which isn’t really
expressible in Flow. You might be able to hack something up with the
SFINAE-like semantics that I demonstrate in #6633, but as I mention in
that issue I wouldn’t be comfortable putting that in the standard
library without a few people more knowledgeable than me taking a good
hard look at it.

Okay sure, that does appear to be a problem with core.js. But my concern here is that the problem goes a little deeper than that.
Take this example instead:
function identity<T>(value: T): T {
  if (value === null) {
    return null;
  }
  return value;
}

Or this one:
function identity<T>(value: T): T {
  if (typeof value === 'undefined') {
    return undefined;
  }
  return value;
}

My point is that there's something else going on here that doesn't feel quite right.

Wait, something else is going on here...
function identity<T>(value: T): T {
  if (typeof value === 'string') {
    return value + 0;
  }

  return value;
}

Take this example instead:

Ah, I see. Yes. What’s happening there is that inside the if-statement
Flow refines value to be of type null but does not refine the type
parameter T to indicate that it must include the null value. This
kind of bidirectional type refinement is something that would be a
convenient addition. I don’t know how feasible it is in terms of
implementation.

Wait, something else is going on here...

This is https://github.com/facebook/flow/issues/7070.

OK yeah, it's the lack of bidirectionality in the type refinement that's messing with me. Gonna leave another example of this weird behavior just in case it helps anyone else see how weird that can get:
function identity<T>(value: T): T {
  if (typeof value === 'number' && value === 0) {
    return value;
  }

  return value;
}

2:   if (typeof value === 'number' && value === 0) {
                                                ^ number literal `0` [1] is incompatible with `T` [2].
References:
2:   if (typeof value === 'number' && value === 0) {
                                                ^ [1]
1: function identity<T>(value: T): T {
                               ^ [2]


logger-typescript dereferences undefined

Short description of the problem:
Transpiling breaks with
> ionic-app-scripts serve

[10:16:43]  ionic-app-scripts 0.0.46
[10:16:43]  watch started ...
[10:16:43]  build dev started ...
[10:16:43]  clean started ...
[10:16:43]  clean finished in 8 ms
[10:16:43]  copy started ...
[10:16:43]  transpile started ...
[10:16:50]  build dev failed: Cannot read property 'indexOf' of undefined

What behavior are you expecting?
transpiling to give a hint whats broken
Steps to reproduce:
I can't really reproduce it... but I found the faulting line:
       // my debug stuff
        if(!errorLine.html)
            console.log("Gotcha!3", errorLine);
        if (errorLine.html.indexOf('class="hljs') === -1) {
            try {
                errorLine.html = highlight_1.highlight(d.language, errorLine.text, true).value;
            }
            catch (e) { }
        }

which than results in something like this:
Gotcha!3 { lineIndex: 905,
  lineNumber: 906,
  text: undefined,
  html: undefined,
  errorCharStart: 54,
  errorLength: 1 }

When i move around the faulting line transpiling shows me the error in HTML but I see it points to an old HTML reprentation of my code. hence i suspect that the process of generating the html view of my code is somehow broken, if the offending line is than at the end of the document which got bigger, it will point to a line not existing in the html.
Which @ionic/app-scripts version are you using?
Your system information:

 ordova CLI: 6.3.1
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.13
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.46
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v6.9.1
Xcode version: Not installed

@ataraxus,
This is very helpful. Thanks! What are you doing to cause the issue?
Thanks,
Dan

Hey Dan, sorry that I can't give a guide for reproducing this failure. I think it would be still unnoticed, if my syntax error didn't happen on the last line....
I'll try later to reproduce it, but since I tried hard the first time, so I don't think I'll manage it this time. Feel free to close this issue. :(

Hi all,
This is resolved in 0.0.48 which was just published.
Please let me know if you have any issues.
Thanks,
Dan

Great! So you could actually reproduce and fix the issue? Kudos


Clean/dataframe

What is this PR for?
Removing the download button part and fixing some bugs, reworked the jquery selectors for the table
Scrollbar is not hidding the ordering button

Resize handle is not hidden by scrollbar

Horizontal scroll

Minimum size of the scrollbar handle

What is the Jira issue?
Related to PR#761
There will only be to change the test after merging this

@corneadoug , I am not able to understand how we will be able to provide feature to download data in csv or tsv?

I said it in your PR, this will be handle by a different PR


Clicking on empty space on tab bar activates window, but does not activate current terminal

Windows Terminal version
1.12.10982.0
Windows build number
10.0.19044.1645
Other Software
No response
Steps to reproduce

Open terminal.
activate some other window
click in terminal window at the empty space on tab bar
type with keyboard

Expected Behavior
Typed characters show up in currently active terminal on terminal window.
Actual Behavior
Nothing happens. Characters to not appear on active terminal.

Huh. Sure does. Wonder when that regressed... I sure doubt that ControlsV2 would have caused that, but that sure is annoying. Thanks!


Fix edge case with subfield accesses inside of contract calls

super weird edge case, draft PR for now as I'm still figuring out what exactly is going on

Closing, related to #577


Suggestion: rt & like from user

Can someone maybe send some code that i can put in the twitterbot.py?
Make it possible for me to rt and like all posts from users i choose. For example @FortniteXL. A Twitter that posts Fortnite giveaways.
Or maybe someone else who is doing this trick.
Retweet and like all of theirs!

add "from:fortnitexl" to the keywords list,
line 18 should read keywords = ["rt to", "rt and win", "retweet and win", "rt for", "rt 4", "retweet to", "from:fortnitexl"]


Installation problem on MAC

Hello guys,
I am trying to install CPU version of whisper_jax on my MAC.
I used following commands for the installation
pip install --upgrade pip
pip install --upgrade "jax[cpu]"
It seems that everything was installed properly.
But after I run python I have following error

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

Note that on Linux above installation is working properly.
Could you help please,
Thanks,
AlexG.

Did you run: pip install git+https://github.com/sanchit-gandhi/whisper-jax.git?

Yes, this solved the problem. Thank you.


added nextjs youtube tutoiral link

What does this PR do?
Add resource(s)
For resources
Description
Added Malayalam Tutorials from Web diary For Next js 13.
Why is this valuable (or not)?
This PR Adds additional resource on Nextjs tutorials , Which Expands the Malayalam Courses section.
How do we know it's really free?
They are Free YouTube playlists listed under the Public channel Web Diary. And anyone with the link will be able to watch the course freely.
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[X] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

added in #10988


Index DataFrame column using ['column_name'] syntax will allow for numeric column names.

As demonstrated in the example from this stack overflow response, when converting a NumPy array into a Dataframe, the column labels are initially numeric.  When trying to index by these column labels, I realized the backend DataFrame indexing is using the attribute method, df.column_label, and not the df[column_label] method.  The current method supports less variation in column label in that it only supports valid python identifiers, e.g., column labels cannot be purely numeric.  This Pandas' doc page has more details.  If possible, changing to the [ ] method (or __getitem__ method) would allow for a wider range of inputs, thereby simplifying the user experience.

We should be able to fix this; doesn't seem too difficult.  Thanks for the suggestion!

[A PR would hurry this process along, by the way! :-)]

I'll see what I can do.  I did not find any "contributing guide".  Is there more than just cloning the repo, branching, then doing a pull request?  Is the setup.py designed to accept the develop option?

Right; that's all there is to it.  If you have already done conda install datashader, it's probably easiest to do conda remove datashader (to remove datashader while leaving all the dependencies satisfied), and then do python setup.py develop.  That's what I do, anyway!

Yeah, I got a minute to create a new environment and get it setup; a quick test head positive results. I should have time to tinker on this tomorrow.

I tested the code as it is (without the change made by pr #285), and it did take column names that would not work with the df.col_name syntax, e.g., df['x values'].  It seems the real issue was it did not take numeric column names.


Fix for endless recursion for getLayoutExplorerNode on a Tooltip

Fixes https://github.com/flutter/devtools/issues/5946
While preparing DevTools for the Multi View changes, I noticed that inspecting a Tooltip causes an stack overflow.
This PR addresses that issue by protecting additional properties recursion using subTree depth.

(triage) Is this PR ready to land? :)


Remove unused imports and unnecessary type hint

This PR doesn't exclude the unused functions distributed across the code-base. Although, they probably shouldn't be present in the first place as the usage of any revision control software makes them unnecessary and noisy.

This PR doesn't exclude the unused functions distributed across the code-base. Although, they probably shouldn't be present in the first place as the usage of any revision control software makes them unnecessary and noisy.

Yes, during M4 we need to clean things up


[Bug Report] SyncVectorEnv has a deprecated but unremoved method seed(), causing AttributeError

Describe the bug
The SyncVectorEnv has a method seed(), in which super().seed(seed=seed) is called. However, the method seed() has already been deprecated in Env.
Code example
import gymnasium as gym

sync_env = SyncVectorEnv([lambda: gym.make("CartPole-v1") for _ in range(3)])
sync_env.seed(0)

This yields:

AttributeError: 'super' object has no attribute 'seed'

### System info

`gymnasium==0.28.1`

### Additional context

_No response_

### Checklist

- [X] I have checked that there is no similar [issue](https://github.com/Farama-Foundation/Gymnasium/issues) in the repo

note: reset_wait does have a seed argument
@pseudo-rnd-thoughts

You should do envs.reset(seed=123), this will be updated soon


chore: Remove the unnecessary field  from ApiEntity

picture_url is still in the ui

According to me, ApiModelEntity is not about the model displayed in the UI...


[NETSDKE2E] With NET 8.0 preview2, Debugging is failed with "Unable to attach to CoreCLR. Unknown Error: 0x80131c3c" on Ubuntu20.04

Environment data
dotnet --info output:
.NET SDK:
Version:   8.0.100-preview.2.23121.17
Commit:    1b05e59daa
Runtime Environment:
OS Name:     ubuntu
OS Version:  20.04
OS Platform: Linux
RID:         ubuntu.20.04-x64
Base Path:   /usr/share/dotnet/sdk/8.0.100-preview.2.23121.17/
Host:
Version:      8.0.0-preview.2.23116.1
Architecture: x64
Commit:       f2f734a7e4
.NET SDKs installed:
8.0.100-preview.2.23121.17 [/usr/share/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 8.0.0-preview.2.23116.9 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 8.0.0-preview.2.23116.1 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
Other architectures found:
None
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download
VS Code version:
Version: 1.75.1
Commit: 441438abd1ac652551dbe4d408dfcec8a499b8bf
Date: 2023-02-08T21:35:30.018Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.15.0-1033-azure
Sandboxed: No
C# Extension version:
v1.25.4
OmniSharp log
Steps to reproduce

Install .NET 8.0 Preview2 SDK on Ubuntu20.04
Create a new console app
cd into new created application folder and launch code
Run debugger pressing F5 or from Run menu

Expected  behavior
Can debug c# code
Actual behavior
Get error "Unable to attach to CoreCLR. Unknown Error: 0x80131c3c"

Additional context

This issue is not repro on Mac OS
This issue is not repro with .NET 7.0 or .NET 8.0 Preview1

Not repro on .NET SDK 8.0.100-preview.2.23127.13, we will keep tracking.

Not repro on .NET SDK 8.0.100-preview.2.23151.8 (runtime-8.0.100-preview.2.23128.3)，so closed it.


Create Name-Value-Units, Min-Constraint and Max-Constraint Input Groups

Make Input group for Name-Value-Units. Add Vmin to Min-Constraints Input Group. Add Vmax to Max-Constraints Input Group. Also see if you can add column headings with clearly delineated borders. Optionally Seek badge on all constraint - both visible (checked) and invisible (unchecked ones).
See 36 "As screen narrows break at constraint min then constraint max"

Tried doing this and the results looked worse. For example, the Name-Value-Units input group did not have a fixed width for all names instead each name was a different length which caused each value to also have a different length. The result looked messier. Will not be done, closing.


Is this dead?

One meeting date has already been missed (Week of 2014-12-15). Extrapolating, there should have been another meeting this week.

Hi @ben-page we did have that meeting, notes are here: https://github.com/joyent/nodejs-advisory-board/blob/master/meetings/2014-12-19/minutes.md
Next meeting information:
Please join our next meeting, Wednesday Jan 14, 2015 at 4:00 PM EST / 1:00 AM PST at
https://global.gotomeeting.com/join/524998381 or please dial-in in using your telephone.
Please join my meeting from your computer, tablet or smartphone.
https://global.gotomeeting.com/join/524998381

You can also dial in using your phone.
United States (Long distance): +1<PHONE_NUMBER>
Access Code: 524-998-381
More phone numbers: https://global.gotomeeting.com/524998381/numbersdisplay.html

Thank you for your response. May I suggest that you update the README.md and http://nodejs.org/advisory-board/? If a person doesn't dig into the repo, it appears that you all stopped meeting.


[Bug]: Incorrect error when editing default roles for RBAC

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When a user tries to edit default roles, we see an error for not having permission to edit role, nothing about the restrictions on modifying default roles.
Steps To Reproduce

Try to modify a default role from admin settings page and note the error displayed

Public Sample App
No response
Issue video log
No response
Version
Self-hosted v1.9 release version

@ramsaptami can we close this issue since this was happening on old fat container?

Yep, closing it


Error in Role Delete

While deleting the user Role, doesn't delete the records, rather it persist.
Grails Version - 2.5.0
runtime ":hibernate4:<IP_ADDRESS>" // or ":hibernate:<IP_ADDRESS>"
compile "org.grails.plugins:spring-security-core:2.0.0"
compile "org.grails.plugins:spring-security-ui:1.0-RC3"

@sakthisgit thank you for submitting this issue!
Are you able to provide more details or attach an example app which demonstrates the issue?
I have been unable to reproduce it.

I'm opting to close this Issue because it is not reproducible.
@sakthisgit please contact me if this is is still an issue for you and you would like it reopened.


pixi should manage its own .pixi/.gitignore

Problem description
Currently the user (or pixi init) must add .pixi to their .gitignore to prevent Git from tracking .pixi/. There are two issues associated with this:

It is a seemingly unnecessary additional installation step in light of the alternative below.
Whenever the user checks out a branch from before .pixi was ignored, Git will stop ignoring .pixi.

Suggested alternative:
Have pixi create a .pixi/.gitignore file to ignore all files under .pixi/. This way the .pixi/ directory becomes self-contained and unobtrusive from the Git perspective. And this makes initialization of new pixi projects simpler. (IMO you could also get by without .gitattributes so then you're down to just pixi.toml!)
Bonus: repeat with .dockerignore

From #questions-chat

Another situation where the current state of things is annoying:
I'm working on some open-source project that doesn't use pixi, so I add my own pixi.toml. Ideally I could just leave that and the rest of the pixi files untracked by Git. However, I need to modify .gitignore and be careful not to check in the changes. I rebase quite frequently, so whenever I rebase I have to stash my updated .gitignore and that causes a big mess.

We might not yet be able to entirely get rid of .gitignore in the project root due to .egg-info, see https://github.com/prefix-dev/pixi/pull/1693#issuecomment-2257883284.

If we get rid of .gitignore in the project root, then we will trigger https://github.com/pypa/hatch/issues/304. Probably the safest approach at first is to keep both .gitignores. We could also try and add .pixi to the built-in ignores as per https://github.com/pypa/hatch/pull/493.

I'm working on some open-source project that doesn't use pixi, so I add my own pixi.toml. Ideally I could just leave that and the rest of the pixi files untracked by Git. However, I need to modify .gitignore and be careful not to check in the changes.

I also work on an open source project where I've switched to using pixi to manage multiple environments (and finding it works great!) but the project does not use it. My workaround for this is to use the .git/info/exclude rather than the .gitignore to avoid accidentally checking stuff in.

I also work on an open source project where I've switched to using pixi to manage multiple environments (and finding it works great!) but the project does not use it. My workaround for this is to use the .git/info/exclude rather than the .gitignore to avoid accidentally checking stuff in.

I've had this problem as well, and settled on keeping pixi.toml one level up in the directory tree from the root of the repo. pixi walks up, so this just works when invoking pixi from within the repo (you just have to set cwd for tasks). It has multiple benefits: no need to worry about accidentally deleting or checking in pixi.toml, and you can keep pixi.toml and pixi.lock under version control (see https://github.com/rgommers/pixi-dev-scipystack/ for an example).

I have a PR up for review in #2361.
I also submitted https://github.com/pypa/hatch/pull/1762 to fix https://github.com/prefix-dev/pixi/issues/2115#issuecomment-2379136870 that should AFAIK allow for us to remove .pixi from the .gitignore in the project root.

I believe the .pixi in .gitignore is now redundant now that https://github.com/prefix-dev/pixi/pull/1693 has been merged.
Is it well-documented for what the .egg-info exclude is necessary? In case it's no longer necessary, then pixi no longer needs to touch .gitignore.


No completion in just opened terminal

Sometimes (around once in twenty cases) when the terminal is opened, there are no autosuggestion until you press enter at least once.  On attached git I'm typing different commands that should trigger autosuggestion:

I don't see how Hydro could be interfering with Fish completions and I haven't seen anything like this since I started using it.

It doesn't happen when using other prompts. I'll try to grab a trace and get back to you.

Thanks. Make sure you are also running the latest Fish.


Fix indentation error

This makes your image on the hub unusable. Rebuild and re-push. :)
Thanks for the code!

@madeddie Ta very much!


Provide messages for when a test case fails

@Newby555

Are all of the exercises ready to be merged once the config errors are resolved?

Yes everything is ready to merge (@Newby555 )


1.10.0 crashes sub.duplicate is not a function

Pulled down latest during build and app started crashing
+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| | `--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>
/usr/src/app/node_modules/socket.io-redis/index.js:57
  var subJson = sub.duplicate({ return_buffers: false });
                    ^

TypeError: sub.duplicate is not a function
    at adapter (/usr/src/app/node_modules/socket.io-redis/index.js:57:21)

Hi! Which redis client are you using? Which version? The default one?

redis-sentinel, looks like io-redis has same problem.

It seems ioredis does define a duplicate method: https://github.com/luin/ioredis/blob/c87dae642ab0d3905323e33f80939a33bbecb2d1/lib/redis.js#L355
So it should not affect this implementation :angel:

Confirmed switching from redis-sentinel to io-redis solves problem with 1.1.0. Seeing that redis-sentinel is a bit old, probably a good idea to do anyway. Was receiving the same error message when I tried quick switch to io-redis, but looks like that was actually being caused by passing wrong constructor params.

Yeah our production servers also started crashing after this update was pulled with a ^1.0.0 config of socket.io-redis. Not good.

@bf to be fair, the Readme clearly states:

If you decide to supply pubClient and subClient, make sure you use node_redis as a client or one with an equivalent API.


Add the Channel variables

Add the below channel variables.

Found Context
Found filename
Found hash

TIRCONTEXT: This is the context name of the found voice recognition. This sets only when the TIRSTATUS is FOUND.
TIRFILENAME: This is the file name of the found voice recognition. This sets only when the TIRSTATUS is FOUND.
TIRFILEHASH: This is the file hash of the found voice recognition. This sets only when the TIRSTATUS is FOUND.


remove Object from NSData

How can i remove an object i saved :
 NSString *path = [[[self class] documentsDirectory]<EMAIL_ADDRESS> NSData *data = [FastCoder dataWithRootObject:self];
 [data writeToFile:path atomically:YES];

[[[NSFileManager] defaultManager] removeItemAtPath:path error:NULL];

writeToFile:atomically: will write/overwrite “Establishment.reservation” file with archived data.
so just remove your file with removeItemAtPath:error: to remove this object
Serge.

10 марта 2015, в 15:00, gitabdelkarim<EMAIL_ADDRESS>написал(а):
How can i remove an object i saved :
NSString *path = [[[self class] documentsDirectory]<EMAIL_ADDRESS>NSData *data = [FastCoder dataWithRootObject:self];
[data writeToFile:path atomically:YES];
—
Reply to this email directly or view it on GitHub https://github.com/nicklockwood/FastCoding/issues/17.

Thanks, i wanted to completely remove the object so i used :
[[NSFileManager defaultManager] removeItemAtPath:path error:NULL];


Make IL2CPP games use a dotnet/runtime setup

This PR represents mine and Rin's painful attempts to replace the mono domain with a modern dotnet runtime when running on an IL2CPP game.
It's mostly backwards-compatible - of a random sample of 51 mods for VRChat I've tested, 49 seem to function as intended, with one not working due to poor code, and one not working due to some confusion around cctors.
Muse dash custom album mod works, but fails to load charts from zipped files - they have to be unzipped. This is due to the fact that it bundles a netframework-targeting build of Ionic.Zip, which attempts to load the IBM437 encoding, which isn't supported on modern .NET. A simple recompilation of the mod, targeting net6, should fix this.
The Long Dark mods are loading and working fine in my tests.
A lot of BTD6 mods are not working due to exceptions in the mod helper's harmony patches, but that mod uses old-style csproj files and can't be compiled against this branch and I'm not willing to spend the time required to fix compilation and then diagnose the issue. However mods that don't use the helper seem to work (as much as can be expected).
So far from this limited sample set, compatibility seems to be > 96% of all mods.
However, due to the removal of the mono reference remapper, very old mods, such as those which require MelonLoader.ModHandler.dll, which was removed in 0.3.0, will no longer work, but the likelyhood of most of them working regardless, considering how much your average game's codebase changes in 15 months, is slim. I've implemented specifically ModHandler backwards-compat.
Minor things left to do

[x] Move ML welcome message to managed
[x] Add a PrintEnvironment method to MelonEnvironment, replacing the game path printing in the unmanaged log
[x] (Potentially?) move analytics blocker to unmanaged
[x] Hook AssemblyVerifier up to AssemblyLoadContext
[x] Rename bootstrap's log to Latest_Bootstrap.log
[x] Rename managed log to Latest.log
[x] Potentially move hash code calc to managed
[x] Switch managed log to ANSI color code-style system, allowing full use of 32-bit colour in log files.
[ ] Investigate loading each melon into its own load context, and pulling dependent assemblies across the boundary between them, thus making them shared, and allowing unloading of entire contexts and thus melons.
[ ] Update InternalCalls to reflect re/moved methods, and move remaining ones to BootstrapInterop class
[ ] Test mono games still work
[ ] Test x86 il2cpp games.
[ ] Investigate potential user experience improvements when no .net runtime installed, when incorrect version/arch, when corrupt, etc.

Implementation Notes

The assembly generator is currently using a build from my forked branch of Il2CppAssemblyUnhollower, which contains changes required to get the assemblies to load in the new runtime (which is must stricter than mono).
The new bootstrap DotnetRuntime manager creates and spins up a coreclr 6.0 runtime, grabs a reference to the new MelonLoader.NativeHost assembly, and calls an entry point in there. This assembly has to be separate because the method has to be marked as [UnmanagedCallersOnly], which was added in .NET 5.0
Compiling currently requires you to have specifically .NET 6.0.3 SDK (SDK 6.0.103) installed to the default location, as it will look for that path for the header and lib files.
Launching ML also currently requires an installed .NET 6.0.3 runtime.
The change of runtime allows mods for IL2CPP games, and parts of ML itself, to target net6, and get all the performance optimizations made by the .NET foundation in the last 6 years.
Debugging works out of the box, and I've added a new command-line argument, --melonloader.launchdebugger, which calls Debugger.Launch() very early in the load process, allowing Visual Studio to attach to the process.

Breakpoints seem to work entirely as expected.
Hot reload needs more investigation - the button appears but didn't function the one time I tried it.
Other debuggers such as the one built in to JetBrains Rider didn't seem to work when I tried them - investigating why would be nice.

A non-trivial number of mods (about 5 of my sample size of 51, for VRChat, and the CustomAlbums mod for Muse Dash) call MelonUtils.NativeHookAttach with the detour being a pointer to a managed method. This worked (for god knows what reason) under mono but crashes the entire runtime when the method is called under CoreCLR.

A system is implemented to attempt to wrap these bad hooks in a [UnmanagedFunctionPointer]-annotated delegate type, and call Marshal.GetFunctionPointerForDelegate (which is what these mods should be doing) before attaching the hook, as well as logging a warning (or error and skip the hook attach, if the hook can't be wrapped) but having it enabled slows down the hook process considerably (to about 500ms per hook on my PC, which is no slouch) as each call must have its method pointer evaluated to see if it can be mapped to a managed method, which requires taking a snapshot of the runtime status.
I'm debating DISABLING this wrapping functionality by default, and requiring communities which refuse to fix their mods to tell their users to specify a new launch option (e.g. --melonloader.hookfix) to enable it, at the cost of (startup) performance.

Updating the runtime to future .NET versions (e.g. 7.0) should be trivial, and it may even be automatic as long as the user has a newer one installed.

Other Considerations

Some general code cleanup of both managed and unmanaged sides would probably not go amiss - moving things to c# where possible, splitting out il2cpp-specific code into a separate (non-net3.5) assembly, etc.
I've been investigating a way to obtain a full stacktrace, including native frames, where desired (e.g. find what's calling a harmony patch) but this seems to be very difficult to do.

Superseeded by #299


Refactor AddressBook to track Modules instead of contacts

Refactoring completed for the following Commands:

AddCommand
DeleteCommand
EditCommand
ListCommand
FindCommand
other commands did not need to be refactored for modules instead of contacts.

Parser and ParserManager  also refactored with "n/" for module name and "l/" for zoom link
Main program functions but test classes yet to be updated

Poor Pull Request with too many conflicts to merge easily. Will resolve conflicts and make a separate pull request


Fixing tests (PHPUnit) locally

requiring PHPUnit 4.0
tests autoloading
enhancing tests code

Wow you're quick ;-)
We will review your pull request and get back to you! Don't expect us to be as quick as you though... :-D

No problem, take your time! :-)

API keys can be configured with dedicated env tags in phpunit.xml config file:
<env name="API_KEY" value="API_KEY"/>
<env name="API_SECRET_KEY" value="API_SECRET_KEY"/>

Hey @Narno :airplane:
We just released a new PHP API wrapper.
It is now more PHP-ish (including namespace, PSR-0 compliance, and globally a better architecture).
All it requires is PHP 5.4
In addition to our API Guides, we would be happy to help you get started with it.
Guillaume, from Mailjet!


Add array_duplicates presto function

Summary

Adding the array_duplicates Presto function to Velox

array_duplicates(array(T))

Returns a set of elements that occur more than once in array.
T must be coercible to bigint or varchar.

For example:
select ARRAY_DUPLICATES(ARRAY [5, 2, 5, 1, 1, 5, null, null])); -- [null, 1, 5]
select ARRAY_DUPLICATES(ARRAY [true, true])); -- (SYNTAX_ERROR) line 1:8: Unexpected parameters (array(boolean)) for function array_dupes. Expected: array_duplicates(array(varchar)) , array_duplicates(array(bigint))

Diff: D31718890

Update:

Implement sort
[optimization] Use uniqueMap.find() to minimize visiting the map
Refine test cases

@darrenfu GitHub is having some trouble displaying PRs will many commits. Any chance you could squash all commits into one and update the PR?

@darrenfu There are build failures. Would you take a look?


WIP: bump apko dep, handle breaking changes

This changes deps on apko's pkg/fs to use pkg/vfs instead.
apko no longer seems to have UseProot -- do we no longer use proot?
cc @deitch

apko no longer seems to have UseProot -- do we no longer use proot?

Nope. Since apko does everything natively, no executing out, it doesn't need it.

use pkg/vfs instead

I am not sure that actually is used inside apko either. Have a look in pkg/apk/impl/fs. There is both an interface definition and some implementations.
I had thought about putting them in pkg/vfs/, but we have been talking about upstreaming everything in apk/impl/, or, at the very least, moving it into its own repository, so I wanted it to have no external dependencies on anything in chainguard-dev/apko. I might have missed a thing or two, but overall it is standalone.

Do you happen to know why the sshfs.yaml example would fail using vfs?
Error: failed to build package: unable to emit package: unable to write data tarball: writing TAR archive failed: readlink not supported by this fs: path (usr/sbin/mount.fuse.sshfs)

https://github.com/chainguard-dev/melange/actions/runs/4344653218/jobs/7588266733#step:5:493

Usually that happens because whatever fs.FS implementation was used, it does not implement ReadlinkFS. I am not quite sure what happened here, except maybe that it uses pkg/vfs and that isn't fully compatible with the new requirements? Those usually are implemented via apko/impl/fs/ (one of the filesystems in there).
FWIW,I have an implementation of melange using the latest apko (and supports limavm and docker, etc.). It almost works. I ran into the issue with the mounted directory, where it created problems.
The remaining step is to be able to treat all build runners as remote: docker, lima, bwrap, even Kubernetes. I was almost there and then ended up digging much more deeply into apko compatibility, so it has been on hold.
If you need this urgently, I can go back into that and get it in place, see if it solves these issues. If this can wait a bit, I can finish the apko details, this is next on my list (and mostly done).

If you need this urgently, I can go back into that and get it in place, see if it solves these issues. If this can wait a bit, I can finish the apko details, this is next on my list (and mostly done).

I don't need this urgently at all, I was mainly hitting some version compatibility stuff in wolfictl, which depends on both apko and melange, and noticed that melange wasn't depending on the latest-and-greated apko yet. I assumed the bump would be easy, and here we are 🙃
I'm fine waiting for you if you already have a good idea how this should work. Thanks for your great answers so far, and for understanding all of this better than I do 😆