/api/* returns a 500 Internal Server Error

Description
Calling the API returns a 500
Steps to Reproduce

Calls to /api/* endpoints times out and returns a 500

# curl -I localhost:4000/api/pools
HTTP/1.1 500 Internal Server Error
Date: Fri, 20 Jul 2018 18:20:21 GMT
Server: Kestrel
Content-Length: 0

Expected behavior
I would expect a 200 response with corresponding data.
Actual behavior
Returns a 500
Reproduces how often
100% of the time it's reproducible
Version
git checkout from current master:
7207861cf09d8c2372a7a641883d0f35592f43d5
I've also tried using the latest docker image with the same results.
This is all on CentOS 7.5
Additional Information
Since I'm able to reproduce this issue using both master checkout as well as the docker image using the same config, I'm assuming this is an issue with my config.  However, I've looked over it several times and have been unable to find what's wrong/missing.  I've also tried remaking the config and end up at the same result.  When I have the logs in debug, I do see a "The operation has timed out" message in the logs when I make a call to the api endpoints.
Below is my config:
{
  "logging": {
    "level": "info",
    "enableConsoleLog": false,
    "enableConsoleColors": false,

    // Log file name (full log)
    "logFile": "miningcore.log",

    // Folder to store log file(s)
    "logBaseDirectory": "/var/log/",

    // If enabled, separate log file will be stored for each pool as <pool id>.log
    // in the above specific folder.
    "perPoolLogFile": true
  },

  "banning": {
    // "integrated" or "iptables" (linux only - not yet implemented)
    "manager": "integrated",
    "banOnJunkReceive": true,
    "banOnInvalidShares": false
  },

  "notifications": {
    "enabled": true,
    "email": {
      "host": "smtp.example.com",
      "port": 587,
      "user": "user",
      "password": "password",
      "fromAddress"<EMAIL_ADDRESS>      "fromName": "pool support"
    },

    "admin": {
      "enabled": true,
      "emailAddress"<EMAIL_ADDRESS>      "notifyBlockFound": false
    }
  },

  // Where to persist shares and blocks to
  "persistence": {
    // Persist to postgresql database
    "postgres": {
      "host": "<IP_ADDRESS>",
      "port": 5432,
      "user": "someuser",
      "password": "somepassword",
      "database": "mydb"
    }
  },

  // Generate payouts for recorded shares and blocks
  "paymentProcessing": {
    "enabled": true,

    // How often to process payouts, in milliseconds
    "interval": 600,

    // Path to a file used to backup shares under emergency conditions, such as
    // database outage
    "shareRecoveryFile": "recovered-shares.txt"
  },

  // API Settings
  "api": {
    "enabled": true,

    // Binding address (Default: <IP_ADDRESS>)
    "listenAddress": "<IP_ADDRESS>",

    // Binding port (Default: 4000)
    "port": 4000
  },

  "pools": [{

    // DON'T change the id after a production pool has begun collecting shares!
  "id": "RVN",
     "enabled": true,
     "coin": {
       "type": "RVN"
     },

    // Address to where block rewards are given (pool wallet)
    "address": "RQNT9JHRyTXsip9jN1uRauExSZoSjJj9yF",

    // Block rewards go to the configured pool wallet address to later be paid out
    // to miners, except for a percentage that can go to, for examples,
    // pool operator(s) as pool fees or or to donations address. Addresses or hashed
    // public keys can be used. Here is an example of rewards going to the main pool
    // "op"
    "rewardRecipients": [
      {
        // Pool wallet
        "address": "RQNT9JHRyTXsip9jN1uRauExSZoSjJj9yF",
        "percentage": 1.0
      }
    ],

    // How often to poll RPC daemons for new blocks, in milliseconds
    "blockRefreshInterval": 400,

    // Some miner apps will consider the pool dead/offline if it doesn't receive
    // anything new jobs for around a minute, so every time we broadcast jobs,
    // set a timeout to rebroadcast in this many seconds unless we find a new job.
    // Set to zero to disable. (Default: 0)
    "jobRebroadcastTimeout": 0,

    // Remove workers that haven't been in contact for this many seconds.
    // Some attackers will create thousands of workers that use up all available
    // socket connections, usually the workers are zombies and don't submit shares
    // after connecting. This features detects those and disconnects them.
    "clientConnectionTimeout": 300,

    // If a worker is submitting a high threshold of invalid shares, we can
    // temporarily ban their IP to reduce system/network load.
    "banning": {
      "enabled": true,

      // How many seconds to ban worker for
      "time": 600,

      // What percent of invalid shares triggers ban
      "invalidPercent": 50,

      // Check invalid percent when this many shares have been submitted
      "checkThreshold": 50
    },

    // Each pool can have as many ports for your miners to connect to as you wish.
    // Each port can be configured to use its own pool difficulty and variable
    // difficulty settings. 'varDiff' is optional and will only be used for the ports
    // you configure it for.
    "ports": {
      // Binding port for your miners to connect to
      "3667": {
        // Binding address (Default: <IP_ADDRESS>)
        "listenAddress": "<IP_ADDRESS>",

        // Pool difficulty
        "difficulty": 16,

        // Variable difficulty is a feature that will automatically adjust difficulty
        // for individual miners based on their hash rate in order to lower
        // networking overhead
        "varDiff": {

          // Minimum difficulty
          "minDiff": 0.01,

          // Maximum difficulty. Network difficulty will be used if it is lower than
          // this. Set to null to disable.
          "maxDiff": null,

          // Try to get 1 share per this many seconds
          "targetTime": 15,

          // Check to see if we should retarget every this many seconds
          "retargetTime": 90,

          // Allow time to very this % from target without retargeting
          "variancePercent": 30,

          // Do not alter difficulty by more than this during a single retarget in
          // either direction
          "maxDelta": 500
        }
      }
    },

    // Recommended to have at least two daemon instances running in case one drops
    // out-of-sync or offline. For redundancy, all instances will be polled for
    // block/transaction updates and be used for submitting blocks. Creating a backup
    // daemon involves spawning a daemon using the "-datadir=/backup" argument which
    // creates a new daemon instance with it's own RPC config. For more info on this,
    // visit: https:// en.bitcoin.it/wiki/Data_directory and
    // https:// en.bitcoin.it/wiki/Running_bitcoind
    "daemons": [{
        "host": "<IP_ADDRESS>",
        "port": 8666,
        "user": "someuser",
        "password": "somepassword",

        // Enable streaming Block Notifications via ZeroMQ messaging from Bitcoin
        // Family daemons. Using this is highly recommended. The value of this option
        // is a string that should match the value of the -zmqpubhashblock parameter
        // passed to the coin daemon. If you enable this, you should lower
        // 'blockRefreshInterval' to 1000 or 0 to disable polling entirely.
        "zmqBlockNotifySocket": "tcp://<IP_ADDRESS>:8667",

        // Enable streaming Block Notifications via WebSocket messaging from Ethereum
        // family Parity daemons. Using this is highly recommended. The value of this
        // option is a string that should  match the value of the --ws-port parameter
        // passed to the parity coin daemon. When using --ws-port, you should also
        // specify --ws-interface all and
        // --jsonrpc-apis "eth,net,web3,personal,parity,parity_pubsub,rpc"
        // If you enable this, you should lower 'blockRefreshInterval' to 1000 or 0
        // to disable polling entirely.
        "portWs": 18545,
      }
    ],

    // Generate payouts for recorded shares
    "paymentProcessing": {
      "enabled": true,

      // Minimum payment in pool-base-currency (ie. Bitcoin, NOT Satoshis)
      "minimumPayment": 0.01,
      "payoutScheme": "PPLNS",
      "payoutSchemeConfig": {
        "factor": 2.0
      }
    }
  }]
}

@calebcall You're calling the wrong endpoint. Check here.

Sorry, my bad. I saw Call /api endpoint timesout and returns a 500, and did not read quoted part.

@calebcall Can you provide miningcore log?

Thanks, I just clarified the description so as not to cause confusion.  I'll get a clean log (with error), current one hasn't rotated so it's several hundred MB.

Here's my miningcore.log
miningcore.log

Man, just figured it out. My stratum server couldn't connect to my postgres database.  I've checked this many times but just checked again and was getting blocked by the firewall.  Live and learn 👍


Add support for double slashes in silo directory paths in visit expressions

Here is a request from Matt O'Brien.
Hi Eric,
Visit does not seem to like silo paths with double (multiple) slashes in the directory path, such as
/path/in//silo///file/with//multiple/slashes
Is it possible to make visit support this, kind of similar to how the file system allows this?
Thanks
-Matt
Matthew O'Brien
Expert
LLNL

Do we happen to know for sure...is it VisIt or maybe the *Silo library that is causing the issue?

Mark believes this is an issue in VisIt. He believes Silo can handle duplicate slashes.


Support HTML language

Is it planned to support HTML in the language list ?
Would be very helpful

Hi @fvisticot , it supports HTML, use xml.
xml, html, xhtml, rss, atom, xjb, xsd, xsl, plist, svg are all under the xml language.


Update to all examples.

Update to all examples
Bumps all safe versions in every example/solution. Next on our todo is running an script weekly to automatize this process.
Upgrade Tailwind Config to v3
Tailwind introduces breaking changes in the v3. All examples have the following replacements done.

Removing darkMode -> darkMode.*$
Removing the variants configuration -> variants:((.|\n)*)(\}\,)
Removing and/or replacing purge and configuring content sources: purge:((.|\n)*?)(\]\,)
Updating colors -> just one.
Updating outline config (none)
Removing unneeded classNames because of Automatic transforms and filters

Should we close all the dependabot PRs for this one?

FYI, I closed all of the Dependabot PRs for updating Next.js to latest 12.0.9 - could we do that here? 🙏


Update CONTRIBUTING.md

This contribution is a detailed, beginner friendly  step by step guide on how to include checkboxes in contribution.md
Describe your change:

[X] Add an algorithm?
[X] Fix a bug or typo in an existing algorithm?
[X] Documentation change?

Checklist:

[X] I have read CONTRIBUTING.md.
[X] This pull request is all my own work -- I have not plagiarized.
[X] I know that pull requests will not be merged if they fail the automated tests.
[X] This PR only changes one algorithm file.  To ease review, please open separate PRs for separate algorithms.
[X] All new Python files are placed inside an existing directory.
[X] All filenames are in all lowercase characters with no spaces or dashes.
[X] All functions and variable names follow Python naming conventions.
[X] All function parameters and return values are annotated with Python type hints.
[X] All functions have doctests that pass the automated testing.
[X] All new algorithms include at least one URL that points to Wikipedia or another similar explanation.
[X] If this pull request resolves one or more open issues then the description above includes the issue number(s) with a closing keyword: "Fixes #ISSUE-NUMBER".

does this pull request have someone working on it?

No sir, pls review
On Sun, Oct 8, 2023, 17:15 Jitanshu-commits @.***>
wrote:

does this pull request have someone working on it?
—
Reply to this email directly, view it on GitHub
https://github.com/marameref/Python/pull/1#issuecomment-1752096362, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOQ3CP2XOWPV6RS6PWE5DITX6LGTHAVCNFSM6AAAAAA5WUQNZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJSGA4TMMZWGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Package.json main field is incorrectly configured.

So according to my experiences installing this package, as well as the interactive documentation (http://browsenpm.org/package.json) about the file, the main field should not be in an array, which causes any require('vue-html-editor') call to fail.

This is fixed in the pull request by Irodham.


[AIRFLOW-6109] [AIP-21] Rename GCP function operators and hook

Jira

[ ] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-6109
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.
In case you are proposing a fundamental code change, you need to create an Airflow Improvement Proposal (AIP).
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does
If you implement backwards incompatible changes, please leave a note in the Updating.md so we can assign it to a appropriate release

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0f7c456). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #558   +/-   ##
=========================================
  Coverage          ?   84.41%
=========================================
  Files             ?      679
  Lines             ?    38497
  Branches          ?        0
=========================================
  Hits              ?    32499
  Misses            ?     5998
  Partials          ?        0

Impacted Files
Coverage Δ

airflow/contrib/hooks/gcp_function_hook.py
100% <100%> (ø)

airflow/gcp/operators/functions.py
98.01% <100%> (ø)

airflow/gcp/example_dags/example_functions.py
79.41% <100%> (ø)

airflow/contrib/operators/gcp_function_operator.py
100% <100%> (ø)

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 0f7c456...bb061c4. Read the comment docs.


Feedback for: engine/getstarted-voting-app/node-setup.md

File: engine/getstarted-voting-app/node-setup.md
Thanks for the great work on the Graceful shutdown in this tutorial.
Another issue I ran into was that, as a Docker for Mac user, I had never installed VirtualBox, so when I got to the about using docker-machine to create local virtual machines, it failed. I had to dig into the docs to see if I could do this with the driver included with Docker for Mac. (There was some warning about if you were using Docker for Windows, but as I was not, I unwisely skipped it). I ended up installing virtualbox so I could continue to follow the tutorial, but a more explicit explanation of what to do if you are on a Mac would be useful.
Separately, it would be nice to have a tutorial on how to use a compose file with tools that are compatible with the engine that DOES come with Docker for Mac (with Docker compose?). I assume that would mean not having multiple nodes, but multiple containers on a single node could still be useful--I am working to set up a dev environment that has multiple interacting containers, for example. Jumping straight from the single container tutorial to the multi-node, swarm tutorial without an intermediate step was kind of rough.

@karptonite , @ManoMarks - working on the first part of this, which is great - illuminated some holes in the docs. We have a short-term solution, and a great long-term one. Look for next iteration of docs in the next hour or so. I'll address the Docker Compose issue after that. Thanks!

@karptonite , @ManoMarks see updated docs here on Set up Dockerized machines.
Per your second point, I'll see what I can do re: adding in an example of working with a single-node Swarm, per @shkrid's nice write-up. I'll leave this issue open with regard to that point.

@karptonite I've added another topic to the "graceful shutdown" procedures per the PR #2244
I'll merge this as soon as the build finishes running, so it should be published today.
I'm going to close this issue, as I experimented with a use case for running a pseudo multi-node swarm on Docker for Mac, and I don't think it's a useful path. The better solution will be for us to provide more samples e.g., using Docker Cloud with Swarm Mode, which we are starting to work on now.
With regard to ability run/test multiple local nodes on Docker for Mac and Docker for Windows (with something other than docker-machine), I'll pursue this feature request on another issue. It's come up before, although I'm not sure how likely it is, I do think it's a worthwhile request.

@londoncalling Thanks! As for run multiple local nodes, or pseudo multi-node swarm, I don't think that was what I was looking for anyway, although I may bet getting lost in the terminology of a system I don't understand yet.
What I was (and still am) looking to do was to be able to spin up on my local machine (or any of our devs' local machines) a set of networked instances to emulate our production environment as closely as possible. That mean separate instances for our web server, our several dbs, memcached, etc, etc--maybe 7-10 instances talking to each other. Is that an unusual use case? Since it is for development, multiple nodes shouldn't be important (if I understand properly what a node is)--I was just going step by step through the tutorial when I hit swarms in the "Define and Deploy your app" section.

@karptonite I believe a better solution to this issue is on it's way with regard to the capabilities of Docker for Mac and Windows. Stay tuned to upcoming releases, and feel free to reach out again on this thread.


Error installing organize-tool[textract] with pipx extract-msg (<=0.29.*) and workaround

With the current version 2.4.0 it is not possible to install the package via PyPi using pipx:
  File "/usr/lib/python3/dist-packages/pipx/venv_inspect.py", line 54, in get_package_dependencies
    for req in map(Requirement, dist.requires or []):
  File "/usr/lib/python3/dist-packages/packaging/requirements.py", line 37, in __init__
    raise InvalidRequirement(str(e)) from e
packaging.requirements.InvalidRequirement: Expected closing RIGHT_PARENTHESIS
    extract-msg (<=0.29.*)

This is an issue with a change in the wheel package. See for example: https://github.com/deanmalmgren/textract/issues/461
As a workaround one can first install organize-tool and then manually install a newer version of textract. This works for me.
pipx install organize-tool
pipx runpip organize-tool install textract==1.6.5

Happy and heavy user of the tool. :heart:

Hey thanks for the heart!
I can reproduce this, but I'm not yet sure how to fix this. Maybe pinning wheel to an older version?
Let me know if you have a good solution. I'll try some things in the next days.

It's strange that installing textract separately works. I am not sure how to solve this. I tried debugging this further but following pip installing a package is quiet hard. Others seem to have tried to pin wheel to an older version. Hopefully textract will release an update with a change to the extract-msg dependency.
Also installing with pip into a venv also works. So maybe it's a problem with pipx.

Not sure if this is related; but after upgrading to Python 3.11.3, I cannot install organize-tool[texttract] either using pip. Error seems very weird to me:
pip install organize-tool[textract]
zsh: no matches found: organize-tool[textract]

@lensaffair Could it be related to the project not listing py3.11 as supported on Pypi? https://pypi.org/project/organize-tool/

@tfeldmann @lensaffair I just submitted a PR that adds py3.11 to the pyproject file and the unit-tests.

Not sure if this is related; but after upgrading to Python 3.11.3, I cannot install organize-tool[texttract] either using pip. Error seems very weird to me:
pip install organize-tool[textract]
zsh: no matches found: organize-tool[textract]

The issue is that zsh considers [] as part of it's syntax instead of passing the arguments to pip. Try to quote it so it's consider a string:
pip install "organize-tool[texttract]"

That's it - works like a charm. Thanks a lot!

This should be fixed with v3.


[WIP] Don't panic if a device doesn't have any tabs open

Resolves #23.
WIP because I’d really like to get a test harness (#25) before merging, as this is the sort of edge case that would be pretty useful to test.

Also: needs a changelog.

Tests would be nice, but proving to be a bit more fiddly than I’d expected, so merging for now and I’ll come back to tests later.


Mosquitto migration

There are some troubles with using the new MQTT Add-on. The problem is, that they edit the confiuration yaml and that doesn't work for the new generation of integration components aka config flow.
More information
-> https://www.home-assistant.io/addons/mosquitto/
Migration

Remove the broker settings inside old configuration.yaml if you have one.
Remove integration (imported from configuration.yaml) on Configuration/Integration panel
Restart Home Assistant
Go to integrations and install the MQTT settings from Discovered section.

FAQ

Do I need to enable discovery? -> No
homeassistant and addons don't work inside local logins. -> That now internal users and needed for auto setups. You need to use a new name.
What kind of login data is now needed? -> You can use your normal user they were created inside Home Assistant. It's currently also supported to set login data with config, but that will remove in future after you can setups rights inside Home Assistant.
Can I use the old manual settings? -> Yes, but they will also be converted to an Integration. After this manual setting are imported into a new integration panel, you can't change things inside the config.

I like where this is going.
Not being able to use 'homeassistant' as username with mosquito while that is the only option with the embedded broker, makes switching between those two a bit painfully right now though.

In the video you don't have login/password configured in Mosquito MQTT.  I have tried the integration and selected discover twice now and I get the Success! prompt but when I click on MQTT Configuration it says "This integration has no devices". All of my MQTT devices are coming from SmartThings MQTT. Will that not work?

@dmoses1969 I have the same problem. No devices. I have a username and login set in the config for the addon since I have a lot of other connections to mqtt that I would have to change. I plan to do this in the future but for now I would like it to just work in the meantime..

@dmoses1969 @isabellaalstrom with the previous integration (before v3) I would get "no devices" in that window but the few mqtt entities I had would still work. I assumed seeing them in that panel was something coming in a future release.

@cogneato Not only were the devices not showing in the integration window. They did not exist in HA anymore. The only way to get them back was to add them back into my binary_sensor.yaml file. After doing that and rebooting my SmartThings decided are showing again.

I removed the credentials from Mosquito MQTT config. Removed all MQTT config from my configuration.yaml file, added all my devices back in to my binary_sensor.yaml and everything is good now. Apparently SmartThings devices are non discoverable.

Hi, what about the bridge with mqtt and cloudmqtt, with the latest update the battery sensors stop working


kexec fails to compile

Unfortunately, the build process is still fighting me. At the moment, kexec fails to compile apparently with legitimate code errors. Do I need to use a different npm version? Error: http://pastebin.ubuntu.com/17719186/

Was using the wrong node version :)

Kexec was giving me lately some ugly problems... :-/ Maybe we are giving
too much pressure on the build mechanism?
El 23/6/2016 12:02 AM, "Lite McFish"<EMAIL_ADDRESS>escribió:

Was using the wrong node version :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/254#issuecomment-227890655, or mute
the thread
https://github.com/notifications/unsubscribe/AAgfvopgx1Uc--SyuRNVlYtpI-2xHtaPks5qObDdgaJpZM4I8RGb
.

@piranna what do you mean by that?

Problems with node-gyp where memory rampaged.
El 23/6/2016 12:15 AM, "Lite McFish"<EMAIL_ADDRESS>escribió:

@piranna https://github.com/piranna what do you mean by that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/254#issuecomment-227894200, or mute
the thread
https://github.com/notifications/unsubscribe/AAgfvnbFs-0SVKbcOIsHvjrqgtOCyMz4ks5qObQLgaJpZM4I8RGb
.

@piranna oh! Yeah the memory usage of the build process is remarkable. Are there ways that we could shrink it? What exactly is it that's eating so much memory? Only node-gyp?

Memory is mostly wasted by 'download' module, since it cache the full file
in memory. Seems it now support streams, so if you want to update
'download-manager', go for it.
El 23/6/2016 12:20 AM, "Lite McFish"<EMAIL_ADDRESS>escribió:

@piranna https://github.com/piranna oh! Yeah the memory usage of the
build process is remarkable. Are there ways that we could shrink it? What
exactly is it that's eating so much memory? Only node-gyp?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/254#issuecomment-227895146, or mute
the thread
https://github.com/notifications/unsubscribe/AAgfvq4JgMQujKSc3JiSmrYI2xxFrhDgks5qObUxgaJpZM4I8RGb
.

@piranna will do. Could you tackle the new error I hit? I'm not finding the cause right off the bat, and since error's aren't reported with the line that called it... :P

You can find the line by searching the exit status. Anyway it should be
already fixed.
El 23/6/2016 1:08 AM, "Lite McFish"<EMAIL_ADDRESS>escribió:

@piranna https://github.com/piranna will do. Could you tackle the new
error I hit? I'm not finding the cause right off the bat, and since error's
aren't reported with the line that called it... :P
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/254#issuecomment-227904095, or mute
the thread
https://github.com/notifications/unsubscribe/AAgfviKojTjKZwa_TknDM0ivIPhjFC95ks5qOcBSgaJpZM4I8RGb
.

@piranna awesome I'll pull and try again.


Discussion: Refactor/Rewrite/Replace some parts of this library

This library has some deprecated libraries e.g. request. Node.js has also progressed since when the core of this library was written; It has support for native promises/Async await e.t.c.
Some of the proposals are:

Replace the bloated and deprecated request library with a more light weight alternative. The current proposals include: ethanent/phin, node-fetch/node-fetch, matthew-andrews/isomorphic-fetch, sindresorhus/got .The replacement library should support most of what request supported.

Potentially replace Event Emitter 3 with a better drop in replacement like mcollina/mqemitter or native Event Emitter. This needs more research.

Replace bluebird with native promises/async-await.

Support Node.js v12+ as per LTS schedule. Any feature/polyfill supporting < v12 should be removed.

Replace pump with pipeline.

Feel free to make your own proposals.

Move the samples/tutorials to a secondary repository
Possibility of making an interactive documentation using documentationjs

Would transition to TypeScript be considered too?

Hey I would like to work on replacing the request library with node-fetch/node-fetch library
But the problem is what about the options.request parameter ? It is compatible with the old request library
https://github.com/request/request#requestoptions-callback

Replace the bloated and deprecated request library with a more light weight alternative. The current proposals include: ethanent/phin, node-fetch/node-fetch, matthew-andrews/isomorphic-fetch, sindresorhus/got .The replacement library should support most of what request supported.

node-gyp goes with https://github.com/npm/make-fetch-happen

I suggest https://github.com/telegraf/client. It's typed, and should be close to supporting https://grammy.dev/advanced/transformers.html.

Agree with @wojpawlik, high time this library used a lite client like telegraf/client while keeping the API consistent with the current version so that there is no barrier to entry for existing devs.
Maybe even use middleware composition instead of event emitters.

@telegraf/client was archived and is now being upstreamed into telegraf. I suggest you to just copy and adjust it to your needs.
Related to #1007.

Do you have any news about migrating this amazing package to more lightwight?

I have very humble needs just to send a message to a channel - maybe there are simpler and lighter alternatives? I didn't find anything well-documented and maintained like node-telegram-bot-api
For now the package size is ~ 600k and startup time ~300ms on my local machine (fast and modern)


Selenoid VNC Requirements?

I have a Kubernetes cluster of a go application (15-20 instances/replicas) and a Selenoid droplet with the 8gb mem 160gb disk tier. My cluster's pods connect to my Selenoid droplet and everything works fine, but how is having the result of VNC enabled different spec-wise? Will having more instances pointed to the hub slow existing connections to the hub any? Does each VNC instance require 1GB memory & 1 GB ram? Will the connectivity performance of my pods be slower due to the low spec tier I selected via DigitalOcean?
I just need to know this so I can properly scale my cluster and droplet.

Nevermind, solved this by diving-deeper in the docs.


在vite中使用rollup-plugin-visualizer打包时error InputNumber DatePicker 无法解析

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info

Package Name And Version<EMAIL_ADDRESS>Framework version: react 18.2
Browser: chrome<IP_ADDRESS>
Reproduction Link: https://codesandbox.io/p/github/YOUAHAHAH/react-acro-admin/main?file=/src/components/SearchBox/src/searchBox.tsx:14,3-14,14&workspaceId=0d348359-9304-4cbc-a9d4-f64ac4fb8875

Steps to reproduce
npm run build 的时候
Rollup failed to resolve import "@arco-design/web-react/es/input-number" from "C:/Users/YOUAH/Desktop/react/Acro/react-acro-admin/src/components/SearchBox/src/searchBox.tsx".
This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
build.rollupOptions.external

上边的地址无法打开


Stopped parseInput from mutating props

This is a tiny change, but after being confused writing component tests for a while, I found this and I figured it was less work to change it than it was to write an issue.
Thanks for the component! It works really well.

Thanks @natenorberg :)


Update instructor material from 2020 course

This PR has two major changes:

I've updated the Sammy's documents on how to get started with running the bootcamp based on the experience from this year (2020)

I've added a new instructor document on how to organize lectures and hw for future bootcamps. @vreuter I tried to incorporate your suggestion into this document so please feel free to edit.

@sklasfeld going to merge this in for now. If you do find any issues, please let me know or report them and I'll be happy to make those changes.


Feat/deploy workflow

This pull request has been deployed to Vercel.

Latest commit:
8eee71e

✅ Preview:
https://yieldgate-lk3k6j11g-dennis-zoma.vercel.app

🔍 Inspect:
https://vercel.com/dennis-zoma/yieldgate/EF9vFVacv2HV4NkAa1qjmry9eMwo

View Workflow Logs

Little update from my side:

Added @yield-e to organization & project
Updated project secrets accordingly (Vercel-, and Github-tokens)
Created & configured new Vercel-Account with @yield-e and setup Deployment/Environment (got a manual deploy running through even, see https://ygyg.vercel.app/)

But: The deploy workflow doesn't run through now :( Immediately errors with an error-message I couldn't really decipher/debug. I created an issue here: https://github.com/BetaHuhn/deploy-to-vercel-action/issues/187
In the meantime, I'll further improve the script to also cover production deploys.
(Also FYI a little tip: after re-adding the pull_request: branches: [develop] to the script for debugging i found out that i can already trigger the script from my local terminal without having the trigger-button visible yet: gh workflow run deploy.yml --ref feat/deploy-workflow)

Added different deploy steps for different environments:
→ pr-{PR}.ygyg.vercel.app for manual workflow dispatches
→ staging.ygyg.vercel.app for pushes on develop
→ prod.ygyg.vercel.app for pushes on main (yieldgate.xyz will be another alias when domain is moved)

@BetaHuhn thanks for the quick reply! Yes the error is still coming up unfortunately.

Changed the domain alias scheme as follows (vercel does not seem to like subdomains):

Preview: ygyg-{SHA}.vercel.app
Staging/Develop: ygyg-dev.vercel.app
Prod: ygyg-main.vercel.app

simply changing the version to 1.9.0 doesn't work for the rest of us. is there something else you guys are doing to make this work ?

simply changing the version to 1.9.0 doesn't work for the rest of us. is there something else you guys are doing to make this work ?

Hey @airtonix, if I remember correctly that was what fixed the issue for us 🤷♂
You can have a look at our deploy workflow here: https://github.com/yieldgate/yieldgate/blob/develop/.github/workflows/deploy.yml
Sorry for not being more helpful..

this is my workflow that is struggling:
name: Release

on:
  push:
    branches:
      - master

concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true

jobs:
  Release:
    name: Release Packages and Changelogs

    runs-on: ubuntu-22.04

    permissions:
      contents: write
      pull-requests: write
      issues: write
      actions: write
      deployments: write
      packages: write

    steps:
      - uses: actions/checkout@v3
        with:
          # We need to fetch all branches and commits so that Nx affected has a base to compare against.
          fetch-depth: 0
          clean: false

      - name: Derive appropriate SHAs for base and head for `nx affected` commands
        uses: nrwl/nx-set-shas@87839d6fba2ae7c5c7fd235475c0a59c58099fb8
        with:
          main-branch-name: master

      - uses: ./pkg/actions/setup-tooling
      - uses: ./pkg/actions/setup-yarn

      - name: Build a list of affected packages
        uses: dkhunt27/action-nx-affected-list@66859c9e82c136ecbb996881ba213bbc03f5a5c3
        id: affected
        with:
          base: ${{ env.NX_BASE }}
          head: ${{ env.NX_HEAD }}

      - run: yarn nx affected --target=semantic-release --maxParallel=1
        env:
          GITHUB_TOKEN: ${{ secrets.BOT_TOKEN }}
          GIT_AUTHOR_EMAIL: ${{secrets.BOT_GIT_EMAIL}}
          GIT_AUTHOR_NAME: ${{secrets.BOT_GIT_NAME}}
          GIT_COMMITTER_EMAIL: ${{secrets.BOT_GIT_EMAIL}}
          GIT_COMMITTER_NAME: ${{secrets.BOT_GIT_NAME}}

      - name: Deploy App:Storybook to Vercel
        if: contains(steps.affected.outputs.affected, 'storybook')
        uses: BetaHuhn/deploy-to-vercel-action@b3d098918bfd16ccb8519fe5b2a43446e86987f4
        with:
          GITHUB_TOKEN: ${{ secrets.BOT_TOKEN }}
          VERCEL_TOKEN: ${{ secrets.VERCEL_TOKEN }}
          VERCEL_ORG_ID: ${{ secrets.VERCEL_ORG_ID }}
          VERCEL_PROJECT_ID: ${{ secrets.VERCEL_PROJECT_ID_STORYBOOK }}
          GITHUB_DEPLOYMENT_ENV: storybook_production

      - name: Deploy App:Dashboard to Vercel
        if: contains(steps.affected.outputs.affected, 'dashboard')
        uses: BetaHuhn/deploy-to-vercel-action@b3d098918bfd16ccb8519fe5b2a43446e86987f4
        with:
          GITHUB_TOKEN: ${{ secrets.BOT_TOKEN }}
          VERCEL_TOKEN: ${{ secrets.VERCEL_TOKEN }}
          VERCEL_ORG_ID: ${{ secrets.VERCEL_ORG_ID }}
          VERCEL_PROJECT_ID: ${{ secrets.VERCEL_PROJECT_ID_DASHBOARD }}
          GITHUB_DEPLOYMENT_ENV: dashboard_production

BetaHuhn/deploy-to-vercel-action@b3d098918bfd16ccb8519fe5b2a43446e86987f4 is the ref to<EMAIL_ADDRESS>
but on my PR workflow: no problems...
name: CI

on:
  pull_request:
    branches:
      - master

concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}
  cancel-in-progress: true

jobs:
  Review:
    name: Verify
    runs-on: ubuntu-22.04

    permissions:
      contents: read
      actions: write
      pull-requests: read
      checks: read

    steps:
      - uses: actions/checkout@v3
        with:
          # By default, PRs will be checked-out based on the Merge Commit, but we want the actual branch HEAD.
          ref: ${{ github.event.pull_request.head.sha }}
          # We need to fetch all branches and commits so that Nx affected has a base to compare against.
          fetch-depth: 0
          clean: false

      - name: Derive appropriate SHAs for base and head for `nx affected` commands
        uses: nrwl/nx-set-shas@v2
        with:
          main-branch-name: master

      - uses: ./pkg/actions/setup-tooling
      - uses: ./pkg/actions/setup-yarn

      - name: Build a list of affected packages
        uses: dkhunt27/action-nx-affected-list@66859c9e82c136ecbb996881ba213bbc03f5a5c3
        id: affected
        with:
          base: ${{ env.NX_BASE }}
          head: ${{ env.NX_HEAD }}

      - run: yarn nx workspace-lint
      - run: yarn nx affected --target=lint --parallel=3
      - run: yarn nx affected --target=test --parallel=3 --ci --code-coverage
      - run: yarn nx affected --target=build --parallel=3

      - name: Deploy App:Storybook to Vercel
        if: contains(steps.affected.outputs.affected, 'storybook')
        uses: BetaHuhn/deploy-to-vercel-action@b3d098918bfd16ccb8519fe5b2a43446e86987f4
        with:
          GITHUB_TOKEN: ${{ secrets.BOT_TOKEN }}
          VERCEL_TOKEN: ${{ secrets.VERCEL_TOKEN }}
          VERCEL_ORG_ID: ${{ secrets.VERCEL_ORG_ID }}
          VERCEL_PROJECT_ID: ${{ secrets.VERCEL_PROJECT_ID_STORYBOOK }}
          GITHUB_DEPLOYMENT: false

      - name: Deploy App:Dashboard to Vercel
        if: contains(steps.affected.outputs.affected, 'dashboard')
        uses: BetaHuhn/deploy-to-vercel-action@b3d098918bfd16ccb8519fe5b2a43446e86987f4
        with:
          GITHUB_TOKEN: ${{ secrets.BOT_TOKEN }}
          VERCEL_TOKEN: ${{ secrets.VERCEL_TOKEN }}
          VERCEL_ORG_ID: ${{ secrets.VERCEL_ORG_ID }}
          VERCEL_PROJECT_ID: ${{ secrets.VERCEL_PROJECT_ID_DASHBOARD }}
          GITHUB_DEPLOYMENT: false

Would be good to continue this conversation in an issue/PR of the affected repo. This has nothing to do with this PR here :)


requestLayout() improperly called

I am getting this warning , may be you want to look at it
W/View: requestLayout() improperly called by com.nshmura.recyclertablayout.RecyclerTabLayout{10a88d34 VFED.... ......ID 30,0-1050,144 #7f0e0093 app:id/recycler_tab_layout} during layout: running second layout pass
Android SDK : 24.0.1

@HashirLabs Thank you for your reporting! I will check this warning.

This problem was improved by https://github.com/nshmura/RecyclerTabLayout/pull/30 .


auto-install jupyter into virtual environment

When trying to start a cell on a python environment without jupyter installed, you get an error like:
Jupyter kernel cannot be started from 'Python 3.7.2 64-bit'. Using closest match Python 3.7.2 64-bit instead.
In situations like this, an option bar should pop up that lets you auto-install jupyter and any dependencies into your environment, instead of automatically using a different environment

Makes sense, given that it already does suggest to auto-install things like autopep8 if you try to auto-format a file and it's not installed.
(Btw: if it's a conda environment, I hope it would use conda to install it and not pip?)

@aldanor - seems reasonable!

We can offer to auto-install when conda is present.

We can offer to auto-install when conda is present.

Why not do it regardless? I.e. why rely on Conda?

Agreed. I don't see a reason conda should be necessary to auto install

In triage, the thinking was that with conda the installed Jupyter will be compatible with other packages in the env. But agree it's not required. We can offer a button that installs Jupyter for users, and prompt for whether conda or pip should be used.

Just to note, there's more than just the 'we used a different python' where this should apply. It would also apply if no jupyter at all is found.
When this comes up:

The feature will be available on the July release.


FeedAHelp: Design the login and registration screens MODAL

Related to #67

The Easy Login Modal Popup will be integrated into the existing application and provide users with a convenient way to authenticate themselves. The following functionalities should be supported:

Social Media Login: Users can log in using their social media accounts (e.g., Facebook, Google, Twitter).
Registered Email and Password Login: Users can log in using their registered email address and password.
Forgot Password: The modal should include a "Forgot Password" link that allows users to reset their passwords.
Close Modal: Users should have the option to close the modal without logging in.

The Easy Login Modal Popup should adhere to the following UI guidelines:

Visibility and Positioning: The modal popup should be visually appealing, centered on the screen, and clearly visible.
Responsive Design: The modal should be responsive and compatible with different screen sizes and devices, including desktops, tablets, and mobile phones.
Social Media Icons: Clearly display recognisable icons for social media login options (e.g., Facebook, Google, Twitter) along with their respective labels.
Email and Password Fields: Provide input fields for users to enter their registered email address and password.
Forgot Password Link: Include a visible "Forgot Password" link near the email and password fields.
Submit Button: A prominent and disabled "Submit" button should appear initially and become enabled once valid inputs are provided.
Close Button: Offer a clearly visible and easily accessible "Close" button to allow users to dismiss the modal.

Final Design archi

We have #100 #98 , we can close this


Change collection types in ArithmeticCircuit from Maps to Tries

Trie is a search tree for string-like datatypes. As variables are now 32-byte SHA2 digests, migrating to tries might give a performance boost.
(Or might not, this needs to be investigated.)
There is a bytestring-trie package on Hackage, but it suffers from bugs and its Semigroup and Monoid instances are a little cumbersome. Maybe I didn't find a better package; maybe we need to roll out our own trie.

I was thinking of MemoTrie but it doesn't have a HasTrie ByteString instance.

I was thinking of MemoTrie but it doesn't have a HasTrie ByteString instance.

Nice package! However it is more useful for memoization and it is not what would help in our case: MemoTries build a memoized total map, so they would not benefit from early returns which can be implemented in efficient tries. We could write down instance HasTrie ByteString as if ByteString was a [Word8], but the implementation for it is not as efficient in MemoTrie as it could be (once again because of totality). At this point, we can roll out our own ByteString :->: a, which would amount to implementing our own trie from scratch.

We need something akin to
data Trie a = TTrie
  { keys :: Set ByteString
  , root :: TrieNode a
  }

data TrieNode a = TNode
  { value :: Maybe a
  , children :: IntMap (TrieNode a) -- used like Map Word8, but is more efficient
  }

Can we use hashmaps or hashtables? Either should be more efficient than Ordered Maps.

Can we use hashmaps or hashtables? Either should be more efficient than Ordered Maps.

@echatav, I was discussing this on one of the previous meetings and @zlonast mentioned that actually Maps are often faster because of the big constant overhead hashmaps introduce.
With IntMaps, the running time is practically $O(1)$: https://hackage.haskell.org/package/containers-0.7/docs/Data-IntMap.html

I think IntMap (Patricia tree) is a good option.
IntMap uses machine sized big endian integers which should be big enough.

Another option is a (Radix tree)[https://hackage.haskell.org/package/radix-tree-<IP_ADDRESS>/docs/Data-RadixTree-Word8-Strict.html].

Oh, that's exactly a ByteString trie though
The only performance gain we might have over it is if:

we account for the constant length of keys
we change the "word size" to be equal to size of Int when using IntMaps

I vote for IntMap then

No new dependency
No new container type code
Very efficient and simple

Sadly, just IntMap wouldn't do because digests are 32 byte, so they wouldn't fit into an Int

critbit and Radix tree both would do, yeah
The question is how efficient are they in comparison with each other, with Map, hashmap and with the container we can roll out ourselves

Sadly, just IntMap wouldn't do because digests are 32 byte, so they wouldn't fit into an Int

We could truncate the hash but I am guessing there would be a small chance of a collision then.

One thing about all these trie types is that they exhibit performance based on key prefix ordering and degrade when inserts happen using random out-of-order keys. But SHA2 digests will be generated as random out-of-order values. We may want to look into existing merkle tree libraries if only for design inspiration if we can't find a suitable one.

On the contrary, I'd say that tries perform better when keys are random-order which means that there is a good chance that the first bit where they are different is in the beginning of the key

Although looking for existing solutions is never a bad idea, yeah


Use Test task to run clojure.test

By using Gradle's built in Test task, we can leverage its features for
test filtering, logging, and reporting.
Currently, this generates non-munged test names as it seems more natural
for the user, though we'll see if that makes it less friendly for
Gradle.
This fixes #17.

I think this is ready to review now.

You rock Andrew.  It looks pretty good to me.  I'm not sure why the build failed... it appears to run fine locally.

@jszakmeister Thanks, John. Not clear what's causing the failure in Travis, works locally for me too.


Shader defined effects using naga oil virtual functions

Describe the solution you'd like
As the title describes, naga oil has functionality to declare virtual functions. It'd be fantastic if this could be leveraged to be able to write custom age, init, update functions, etc instead of using the expression builder. This experience would be much like writing vertex/fragment shaders but on a per particle basis.
Describe why you want that solution. Is this related to a problem?
Hanabi's effect expression builder can be quite complicated and in some cases, a particle effect could be authored more efficiently using plain wgsl. It would still be possible to support both experiences by using virtual functions in the shader builder.
Describe alternatives you've considered
Using the hanabi expression api as it currently exists
Additional context
Add any other context or screenshots about the feature request here.
This is a great talk about an extremely flexible and powerful particle system that takes this approach
https://youtu.be/qbkb8ap7vts?si=KIMa32aVuG80uxPr

Thanks @ncthbrt for opening that issue, and for the reference of that GDC talk it's fantastic, I had missed it! This feels so energizing to see all the possible things Housemarque can build with particles.
On the actual feature request now, let me elaborate a bit on the vision for Hanabi and how this fits. If you check the Expression API implementation you'll note that it's under a graph module. However this part of the code (the one under that module but which is not expression related) has never really been used much by anyone, because it's not finished. The goal of the Expression API has never been to allow users to build effects from code; instead, it was to lay the foundation of a (visual) graph editor, where tech artists could build effects with a node graph, similar to all major game engines out there. The rationale is that building a VFX without real-time visual feedback is a dead end in my opinion; no one wants to do that, no one serious about development has time for that. Now, practically though unfortunately Hanabi is blocked by a couple of things, and notably:

lack of decently mature UI in Bevy or other to build an editor, and time to do so;
limitations in the asset serialization/deserialization system (due in part to the inability to handle trait objects).

As a consequence, for now most VFX are authored via code. I say "most" because I know about at least 1 user created editor that I tried, and it's amazing, but was never released publicly, but at least confirmed my instinct that this was the right path. But authoring by code is not ideal by any measure; this is only a stop gap. I believe once those limitations overcome, the visual editing is the path forward for 99% of the users.
That being said, I appreciate that, in part because Hanabi's modifiers and expressions are not user-extensible (and I'm not sure how much they could be, it's reasonably complex), having an escape hatch down to raw shader code is necessary. So your suggestions 100% fits in that vision I think. I'm not sure exactly what naga_oil's virtual functions are though; my intuition is that we can simply add a ShaderExpression which injects raw WGSL code directly into the generate one, provided you know what you're doing (great power great responsibility). I think that would be enough to unlock a lot of "exotic" effects. As Housemarque themselves note, not having a visual editor (or as  a stop gap, expression API) which non-developers can use was a major bottleneck during production, so I'm skeptical about the all-shader-code approach; the more authors write shader code by hand, the harder it is to build an editor for non-devs (and in fact Housemarque seem to have a meta-language with some constraints, which is the only sane way). Or is there anything more to naga_oil's virtual functions than what I described? (I'm not sure what they are nor why they're called "virtual")

Thanks for the detailed reply!
To expand on virtual functions, here is a snippet copied from the documentation:

virtual functions can be declared with the virtual keyword:

    virtual fn point_light(world_position: vec3<f32>) -> vec3<f32> { ... }

virtual functions defined in imported modules can then be overridden using the override keyword:

#import bevy_pbr::lighting as Lighting

override fn Lighting::point_light (world_position: vec3<f32>) -> vec3<f32> {
    let original = Lighting::point_light(world_position);
    let quantized = vec3<u32>(original * 3.0);
    return vec3<f32>(quantized) / 3.0;
}

overrides must either be declared in the top-level shader, or the module containing the override must be imported as an additional_import in a Composer::add_composable_module or Composer::make_naga_module call. using #import to import a module with overrides will not work due to tree-shaking.

Also we had a good discussion in rendering on the bevy discord about how it might be possible to introspect on wgsl structs to allow for reflection in rust land which could be good for tweaking effect settings without having to drop into rust code.


Crash after teleporting to Nardah from Iowerth Camp

Please check if your issue is not a duplicate by searching existing issues
Describe the bug
Full crash after teleporting to Nardah during Song of the Elves
To Reproduce
Steps to reproduce the behavior:
During the clue scroll task while doing song of the elves (south gate of prif), I teleported to Iowerth camp with only the master clue book in inventory. I did not gear up properly, so I tele'd to nardah using the master clue book. This caused an application crash.
Expected behavior
I expected the application to load into nardah, not crash. I successfully tele'd to nardah upon login.
Screenshots
N/A
Environment (please complete the following information):

OS: Windows 10
RuneLite version: 1.8.16
Launcher version: 2.1.3

Additional context
File attached to bug report
jvm_crash_pid_19044.log

This is a JVM crash caused by code run in the skybox plugin. Duplicate of #10549


AT UART SPEED

what is the maximum uart speed  when running at ethernet and wifi command under esp32?
Thanks.

I tried 2mb, and 5mb with pin tx1 and rx3, not working. break at the middle, when doing stress test.

@zf201911 The maximum uart baud rate can reach to 5Mbps.
I am not sure have you enable UART hardware flow control on esp-at and PC serial port tool.

Closing due to lack of response, please feel free to reopen if this is still an issue and you can supply more details. Thanks.


Round interpolation for RGB components

The CSS spec doesn't allow for decimal values inside of rgb, however the RN implementation does, so there was a disconnect here.
This tests to see if the output range is an rgb color, and if so, rounds the first 3 interpolated components (but not the 4th, since that would be opacity and allows for a decimal).
cc @vjeux

I also want to get rid of this for react-native but couldn't because of this problem. Would love if you added this to rn as well

👍  will do.

Just for my own edification is this the case for hsl and others as well?

@browniefed it looks like HSL allows decimal values for all 3 of it's entries, but the second and third components are percentage units. That said, with #28 these colors will all be in rgba format by this time in code....


Custom Resource Documentation

Docs for CustomResourceDefinition and the concept of custom resources in general.

This change is

@enisoc Is this ready for review now?

@chenopis Sorry, not yet. I'm planning to push a ready-for-review commit by the end of today.

@enisoc No worries, just wasn't sure. Ping me when it's ready. Thx

This is now ready for review.
/assign @chenopis
/assign @deads2k
cc @erictune @foxish

nice doc. lgtm.

@abiogenesis-now Can you take a look at this one as well? Thx

@enisoc FYI, all feedback must be addressed and LGTMs given by EOD Tue, June 27th so that this can be merged for the 1.7 release on June 28th.

@cody-clark @abiogenesis-now PTAL. Thx

responded with some rewording suggestions, but these are optional (e.g. use them if you think they're useful). otherwise LGTM

Added some more edits for the parts about finalizers and prereqs for migration.

@enisoc Sorry, due to some other recent PR merges, there are now conflicts w/ the _data/concepts.yml and _data/tasks.yml files.

@chenopis lgtm

Conflicts resolved.

Review status: 0 of 10 files reviewed at latest revision, 30 unresolved discussions.

Comments from Reviewable

Reviewed 1 of 11 files at r1, 10 of 10 files at r2.
Review status: all files reviewed at latest revision, 30 unresolved discussions.

Comments from Reviewable


Allow scan job templates

Coverage decreased (-0.1%) to 99.863% when pulling 0ee479a5f76310abac00aa487fc1bd4a527b012b on AlanCoding:scan_jobs into 288f9a87de12893308030d24c3b7d093d4c508d3 on ansible:master.


Missing Stripe production instructions in Wiki

Describe the bug
The README does a great job of walking a user through setting up their Stripe dev/test environment, but the same is not done on the production settings in the Wiki. Specifically, a link should be added to the production settings with a brief instruction on setting up the production webhook for Stripe.
If the webhook is not set up, subscriptions won't be created successfully in the app, and for new users of Stripe and new developers using this codebase, it may produce a ambiguous experience of trying to locate what went wrong in the code, when what was missing was the webhook creation within Stripe.

I added more documentation at the bottom of the production settings wiki page.


Fix error in test_pycc in single-process mode

Fix the following error in test_pycc:
[...]
  File "/home/antoine/34/lib/python3.4/tempfile.py", line 200, in _mkstemp_inner
    fd = _os.open(file, flags, 0o600)
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmps0e8jvnk/z_e0ztq4'

You're right, I've now removed it.


Added a Load button for the chat tab

This will speed up loading times and optimize memory usage for users who don't want to use that feature

@jpgallegoar is it ready for merge?
dont mind for the formatter, i'll just take care of it


Add offline update and beta release install instructions

Should this mention the contribution guide local build instructions as well for installing pre-release versions?

Should this mention the contribution guide local build instructions as well for installing pre-release versions?

Yeah we should I think


Make it globally installble

After consideration, this will not be done as it is not the intended use of the kit.


Validate the format of the answer

Plain Text

string_plain: Plain text, up to 512 characters (default).
string_plain_max_[num]: Plain text, up to [num] characters.
string_plain_min_[num]: Plain text, at least [num] characters.
string_plain_[n1]:[n2]: Plain text, at least [n1] characters and up to [n2] characters.

Number

num_int: Number, integers.
num_float: Number, can be decimal.
num_min_[n1]: Number, not less than [n1].
num_max_[n1]: Number, not greater than [n2].
num_[n1]:[n2]: Number, from [n1] to [n2].

Rating

rating_5: Rating from 1 to 5.
rating_10: Rating from 1 to 10.
rating_[n1]:[n2]: Rating from [n1] to [n2].

Email

string_email: Email address.

Phone Number*

string_phone_[area]: Phone number for area code [area].

Radio

radio_[q1],[q2],[q3];: Radio, text from [q1], [q2], and [q3].

Multiple

multi_[q1],[q2],[q3];: Multiple, text from [q1], [q2], and [q3].

Hidden

hidden: Hidden for Radio or Multiple.

Date

date

Time

time

"types": {
  "short_answer": "Short Answer",
  "paragraph": "Paragraph",
  "multiple": "Multiple Choice",
  "checkboxes": "Checkboxes",
  "dropdown": "Dropdown",
  "file": "File Upload",
  "linear": "Linear Scale",
  "date": "Date",
  "time": "Time"
}


Getting Error when creating location

Getting the following in cloud watch for CWBI test.  Don't think this is a schema issue since I was able to create a location in our local database without an issue using the same schema.  This was working a couple weeks ago in CWBI-test.
When running
curl -X 'POST'
'https://cwms-data-test.cwbi.us/cwms-data/locations'
-H 'accept: /'
-H 'Authorization: apikey XXX'
-H 'Content-Type: application/json'
-d '{
"office-id": "MVP",
"name": "TestLoc_evn3432",
"latitude": 0,
"longitude": 0,
"active": true,
"public-name": "TEST",
"long-name": "TEST",
"description": "TEST",
"timezone-name": "America/Los_Angeles",
"location-kind": "PROJECT",
"nation": "US",
"published-longitude": 0,
"horizontal-datum": "NAD83",
"published-latitude": 0
}'
| 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 3700
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "MDSYS.MD", line 1723
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "MDSYS.MDERR", line 17
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "MDSYS.PRVT_IDX", line 12
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 9928
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 2081
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 3658
  | 2024-08-16T21:39:18.306Z | ORA-20025: LOCATION_ID_NOT_FOUND: The Location: "TestLoc_evn3432" does not exist.
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 1453
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_ERR", line 80
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 245
  | 2024-08-16T21:39:18.306Z | ORA-01403: no data found
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 213
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 182
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 274
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 1054
  | 2024-08-16T21:39:18.306Z | ORA-06512: at "CWMS_20.CWMS_LOC", line 3628

Schema no, database, yes. there appears to be something wrong with the geospatial index, hence the MDSYS.* lines in that output. Well, or the schemas are slightly different, log/long 0/0 is not within the MVP office, the US for that matter, so maybe this db doesn't have the change for allowing the use to dictate the bounding office, or you may need to set it so it doesn't try looking it up.

you are right it was the Lat and Longs being 0.  I copied an existing location information and it worked.  It allowed me to do that before.  and I was able to do it in our test databases.  I tried some other values as well and it errored out so I don't what the boundary is that it is checking.

I think there might be more to it. That error message above is incomplete. I get that same message, but that's after the real error occurred.
27-Sep-2024 21:13:22.253 INFO [https-openssl-nio-8243-exec-44] cwms.cda.data.dao.AuthDao.prepareContextWithUser Validated Api Key for user=L2EDHDTO [CONTEXT ratelimit_period="5 SECONDS" ]
27-Sep-2024 21:13:22.396 INFO [https-openssl-nio-8243-exec-44] cwms.cda.ApiServlet.lambda$init$12<PHONE_NUMBER>750825719: Not Found.
        cwms.cda.api.errors.NotFoundException: ORA-06502: PL/SQL: numeric or value error: character string buffer too small
                at cwms.cda.data.dao.JooqDao.buildNotFound(JooqDao.java:338)
                at cwms.cda.data.dao.JooqDao.wrapException(JooqDao.java:230)
                at cwms.cda.data.dao.JooqDao.connection(JooqDao.java:564)
                at cwms.cda.data.dao.LocationsDaoImpl.storeLocation(LocationsDaoImpl.java:221)
                at cwms.cda.api.LocationController.create(LocationController.java:312)
                at io.javalin.apibuilder.CrudFunction$3.invoke$lambda-0(CrudHandler.kt:32)
                at io.javalin.apibuilder.CrudFunctionHandler.handle(CrudHandler.kt)
                at cwms.cda.security.KeyAccessManager.manage(KeyAccessManager.java:30)
                at cwms.cda.security.MultipleAccessManager.manage(MultipleAccessManager.java:41)
                at io.javalin.http.JavalinServlet.addHandler$lambda-0(JavalinServlet.kt:96)
                at io.javalin.http.JavalinServlet$lifecycle$2$1$1.invoke(JavalinServlet.kt:43)
                at io.javalin.http.JavalinServlet$lifecycle$2$1$1.invoke(JavalinServlet.kt:43)
                at io.javalin.http.JavalinServletHandler.executeNextTask(JavalinServletHandler.kt:99)
                at io.javalin.http.JavalinServletHandler.queueNextTaskOrFinish$lambda-1(JavalinServletHandler.kt:85)
                at java.util.concurrent.CompletableFuture.uniComposeStage(CompletableFuture.java:995)
                at java.util.concurrent.CompletableFuture.thenCompose(CompletableFuture.java:2137)
                at io.javalin.http.JavalinServletHandler.queueNextTaskOrFinish$javalin(JavalinServletHandler.kt:85)
                at io.javalin.http.JavalinServletHandler.executeNextTask$lambda-11$lambda-10(JavalinServletHandler.kt:119)
                at java.util.concurrent.CompletableFuture.uniApply(CompletableFuture.java:616)
                at java.util.concurrent.CompletableFuture.uniApplyStage(CompletableFuture.java:628)
                at java.util.concurrent.CompletableFuture.thenApply(CompletableFuture.java:1996)
                at io.javalin.http.JavalinServletHandler.executeNextTask(JavalinServletHandler.kt:119)
                at io.javalin.http.JavalinServletHandler.queueNextTaskOrFinish$lambda-1(JavalinServletHandler.kt:85)
                at java.util.concurrent.CompletableFuture.uniComposeStage(CompletableFuture.java:995)
                at java.util.concurrent.CompletableFuture.thenCompose(CompletableFuture.java:2137)
                at io.javalin.http.JavalinServletHandler.queueNextTaskOrFinish$javalin(JavalinServletHandler.kt:85)
                at io.javalin.http.JavalinServlet.service(JavalinServlet.kt:89)
                at javax.servlet.http.HttpServlet.service(HttpServlet.java:623)
                at cwms.cda.ApiServlet.service(ApiServlet.java:865)
                at javax.servlet.http.HttpServlet.service(HttpServlet.java:623)
                at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:210)
                at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:154)
                at org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:51)
                at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:179)
                at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:154)
                at org.apache.catalina.filters.HttpHeaderSecurityFilter.doFilter(HttpHeaderSecurityFilter.java:129)
                at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:179)
                at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:154)
                at org.apache.catalina.filters.CorsFilter.handleNonCORS(CorsFilter.java:331)
                at org.apache.catalina.filters.CorsFilter.doFilter(CorsFilter.java:158)
                at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:179)
                at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:154)
                at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:168)
                at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:90)
                at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:596)
                at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:130)
                at org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:670)
                at org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:670)
                at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:93)
                at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:93)
                at org.apache.catalina.authenticator.SingleSignOn.invoke(SingleSignOn.java:241)
                at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:74)
                at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:346)
                at org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:390)
                at org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:63)
                at org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:928)
                at org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1786)
                at org.apache.tomcat.util.net.SocketProcessorBase.run(SocketProcessorBase.java:52)
                at org.apache.tomcat.util.threads.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1191)
                at org.apache.tomcat.util.threads.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:659)
                at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:63)
                at java.lang.Thread.run(Thread.java:750)
        Caused by: java.sql.SQLException: ORA-06502: PL/SQL: numeric or value error: character string buffer too small
ORA-06512: at "CWMS_20.CWMS_LOC", line 3700
ORA-06512: at "CWMS_20.CWMS_LOC", line 1876
ORA-06512: at "CWMS_20.CWMS_LOC", line 3658
ORA-20025: LOCATION_ID_NOT_FOUND: The Location: "Animas River @ Farmington" does not exist.
ORA-06512: at "CWMS_20.CWMS_LOC", line 1453
ORA-06512: at "CWMS_20.CWMS_ERR", line 80
ORA-06512: at "CWMS_20.CWMS_LOC", line 245
ORA-01403: no data found
ORA-06512: at "CWMS_20.CWMS_LOC", line 213
ORA-06512: at "CWMS_20.CWMS_LOC", line 182
ORA-06512: at "CWMS_20.CWMS_LOC", line 274
ORA-06512: at "CWMS_20.CWMS_LOC", line 1054
ORA-06512: at "CWMS_20.CWMS_LOC", line 3628
ORA-06512: at line 1

No matter what I do, I can't create a location, and I have no idea why.
I've attached a script that fails when I run it. I get a server error, and the server error is the above trace.
locations_create.py.txt

Ok, to follow-up, I finally figured out the problem: the location name was too long.
I spent 5 hours on this issue. CDA needs better error messages, and it needs them now.
It's frustration, anger, and wasted time by people trying to use this (and I'm not the only one).

I don't disagree with you, but just magically coming up with good error messages has been difficult.

We're taking a vague first approach so we don't have to panic about any vulnerability reports with annoying deadlines.
Often we're fighting with vague messages from the database that are difficult to map. (That's assuming we even get an error from the db.)

Like here for example "character string buffer too small" is clearly the cause but where? We know it's the location id, but there's no context for CDA to pick it up.
Good new is this one should be fairly easy to address. However, not every error will be, please keep opening issues, and if at all possible consider helping with the actual fix.

I'm going to close this one, it was originally for a specific error. The new update has a bit more of a generic bent and an issue focused on the error reporting (same context (too long) different table/column) was opened a few weeks ago.
Additional identified need covered in #870


pip_api: type hints

WIP. I believe this covers the majority of the public API surface; will test these hints against my work in pip-audit.
Closes #96.

Turns out that this is a significant pain due to the vendored modules:
$ mypy pip_api
pip_api/_vendor/pyparsing.py:123: error: Module "itertools" has no attribute "ifilterfalse"; maybe "filterfalse"?
pip_api/_vendor/pyparsing.py:123: error: Name "filterfalse" already defined (possibly by an import)
pip_api/_vendor/pyparsing.py:126: error: Module "_thread" has no attribute "RLock"
pip_api/_vendor/pyparsing.py:143: error: Cannot find implementation or library stub for module named "ordereddict"
pip_api/_vendor/pyparsing.py:143: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
pip_api/_vendor/pyparsing.py:143: error: Name "_OrderedDict" already defined (possibly by an import)
pip_api/_vendor/pyparsing.py:145: error: Cannot assign to a type
pip_api/_vendor/pyparsing.py:145: error: Incompatible types in assignment (expression has type "None", variable has type "Type[OrderedDict[Any, Any]]")
pip_api/_vendor/pyparsing.py:150: error: Name "SimpleNamespace" already defined (possibly by an import)
pip_api/_vendor/pyparsing.py:233: error: Module has no attribute "maxint"
pip_api/_vendor/pyparsing.py:234: error: Name "xrange" is not defined
pip_api/_vendor/pyparsing.py:236: error: Name "_ustr" already defined on line 227
pip_api/_vendor/pyparsing.py:260: error: Cannot find implementation or library stub for module named "__builtin__"
pip_api/_vendor/pyparsing.py:1800: error: Name "_FifoCache" already defined on line 1771
pip_api/_vendor/pyparsing.py:1829: error: Need type annotation for "packrat_cache" (hint: "packrat_cache: Dict[<type>, <type>] = ...")
pip_api/_vendor/pyparsing.py:2902: error: "Type[ParserElement]" has no attribute "_literalStringClass"
pip_api/_vendor/pyparsing.py:5086: error: Not all arguments converted during string formatting
pip_api/_vendor/pyparsing.py:5944: error: "Callable[[VarArg(Any), KwArg(Any)], Any]" has no attribute "ANY_VALUE"
pip_api/_vendor/pyparsing.py:6730: error: Need type annotation for "_ranges" (hint: "_ranges: List[<type>] = ...")
pip_api/_vendor/pyparsing.py:6799: error: Need type annotation for "_ranges" (hint: "_ranges: List[<type>] = ...")
Found 19 errors in 1 file (checked 21 source files)

The CI will pass above, but the parent call to mypy in pip-audit will continue to fail because of these inner errors. I've tried to mark the imports for pyparsing with # type: ignore, but that doesn't seem to have done anything...

Urgh, this might actually be fixable with py.typed. Will dig in further tomorrow.

Okay: in local testing, adding py.typed to the distribution allows pip-audit to typecheck against its usage of pip-api, since said usage is limited to the public APIs that we've annotated.
For the package itself, mypy in the CI still won't work, for the vendoring reasons mentioned above. If that's acceptable, this should be good to go 🙂

For the package itself, mypy in the CI still won't work, for the vendoring reasons mentioned above. If that's acceptable, this should be good to go 🙂

I think that's acceptable for now.

Okay, this should be good to go once the CI goes through!
A minor bump once it's merged would also be appreciated 🙂

Released in https://pypi.org/project/pip-api/0.0.21/


Process All Files and submit once

Can all the files be processed before being appended to the form data...
so the form doesnt have to been submitted for each file upload..

You could do this by writing a custom upload function that requests all files from FilePond (and not have FilePond do the uploading).


720p monitor problem

Every time i open Leecher, it's looking like this:

At least make it, so it remembers its position and size.
p.s. You are welcome to correct my english. =)

Twitch Leecher is optimized for vertical resolutions >1024. I honestly did not think anyone streaming or watching on Twitch would show up with such a tiny vertical resolution. It has been a very long time since I saw the last 720p display ;)
Take a look from the other side of the medal: If you open up TL on a 1440p screen and the application pops up as a tiny square in the middle of the display I also looks kinda weird.
The solution would be a variable default size for the window related to the vertical resolution. Something like 80% perhaps. That would please all users.
Saving application size and position is on my list since the beginning. The biggest issue is my lack of freetime atm.

🎉


[CT-3196] [Bug] config block doesn't respect ref('')

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior

When I use ref() under model config function, it replace current relation instead of one specified in ref()

Expected Behavior

It should just respect ref() under the model config function

Steps To Reproduce

For simple explanation purpose only

i used ref under the pre_hook, it's not limited to pre_hook
also use lame way to create first_model

-- first_model.sql
{{
  config(
    materialized = 'view',
    pre_hook = 'CREATE OR REPLACE TABLE new_table AS SELECT * FROM ' ~ ref('second_model')
    )
}}

SELECT
    *
FROM
    new_table

When I hit dbt build then pre_hook query will generate as below

CREATE OR REPLACE TABLE new_table AS SELECT * FROM mydb.myschema.first_model

if we look closely ref('second_model') is replaced with mydb.myschema.first_model, but it suppose to replace mydb.myschema.second_model`

Relevant log output
Began running node model.dbt_materializations_integration_tests.first_model
1 of 1 START sql view model dbt.first_model .................................... [RUN]
Re-using an available connection from the pool (formerly list_awsguzzle_dbt, now model.dbt_materializations_integration_tests.first_model)
Began compiling node model.dbt_materializations_integration_tests.first_model
Writing injected SQL for node "model.dbt_materializations_integration_tests.first_model"
Timing info for model.dbt_materializations_integration_tests.first_model (compile): 16:43:52.871647 => 16:43:52.944055
Began executing node model.dbt_materializations_integration_tests.first_model
Using snowflake connection "model.dbt_materializations_integration_tests.first_model"
On model.dbt_materializations_integration_tests.first_model: /* {"app": "dbt", "dbt_version": "1.6.2", "profile_name": "dbt_red_snow", "target_name": "dev", "node_id": "model.dbt_materializations_integration_tests.first_model"} */
CREATE OR REPLACE TABLE new_table AS SELECT * FROM awsguzzle.dbt.first_model
Opening a new connection, currently in state closed
Snowflake adapter: Snowflake query id:
Snowflake adapter: Snowflake error: 002003 (42S02): SQL compilation error:
Object 'AWSGUZZLE.DBT.FIRST_MODEL' does not exist or not authorized.

Environment
- OS:Ubuntu 22
- Python:3.10.12
- dbt-core: 1.6.2
- dbt-snowflake: 1.6.2

Which database adapter are you using with dbt?
snowflake
Additional Context

If this behaviour is on purpose then it should throw error "Found a cycle: model.dbt_materializations_integration_tests.first_model" right ?

this might be a duplicate issue (sorry my bad)
using {{ ref('anything') }} in a set_sql_header call resolves to the current model and not 'anything' https://github.com/dbt-labs/dbt-core/issues/2793
so what was conclusion ? not to fix and just document it ?

No worries @sudo-pradip.
I think this comment has the info you are looking for: https://github.com/dbt-labs/dbt-core/issues/6837#issuecomment-1413924647
Could you give that a read, try it out, and let us know what you think?

nice workaround, now it's working, you are kind, thanks you :star2:
{{
  config(
    materialized = 'view',
    pre_hook = "{{ 'CREATE OR REPLACE TABLE new_table AS SELECT * FROM '  ~ ref('second_model') }}"
    )
}}

SELECT
    *
FROM
    new_table

Awesome @sudo-pradip 🎉 Thanks for sharing your nice example too -- I hadn't tried that variant before 🏆
I'm going to close this as "it's a (complicated) feature, not a bug" 😎


Quickjs, Okab (altair saver)

Hey all, thanks for working on this! Just to let you know, there is similar development effort going on at okab. It started as a way to provide a better altair-saver method I think. I've asked the author if he's interested in somehow joining forces, see this thread.
The thread started with my idea of using QuickJs javascript engine, which might also be relevant here. See also quickjs-rs. QuickJS does run the minified vega and vega-lite code.
What are your thoughts?

Hi @k-groenbroek, thanks for reaching out and for digging into quickjs/quickjs-rs.  I haven't played with it, but your thread over in https://github.com/daylinmorgan/okab/issues/3 is certainly interesting and helpful.
I'm honestly pretty happy with Deno. It integrates well with Rust and has a growing community (and company) backing it. Here are a few considerations that come to mind:

Support for data URLs is important. based on the that thread it looked like Quickjs may not support the required fetch function (which totally makes sense because this is no small task and requires things like SSL integration).
My intuition is that Deno's performance for rendering large figures is going to be better than Quickjs since Deno uses v8 which has advanced JIT compilation built-in.
Down the road, I'm interested in using the Vega WebGPU renderer to accelerate PNG export of charts with lots and lots of marks (e.g. scatter plots with a million points). Deno has initial support for WebGPU, which is something that even Node.js doesn't have yet.

Is your main motivation for investigating Quickjs to reduce the size of the Python wheel?  The current vl-convert-python wheels come it an around 20MB. This isn't small by any means, but it's also not unreasonable. That's about 2x the size of pandas and 1/10th the size of tensorflow.
It would be great to join forces! But I just want to be up front that, for my current and future priorities, I'm pretty set on sticking with the Deno in Rust architecture.

CC @lsh who works on the WebGPU renderer.

Awesome! My motivations were:

smaller package size
from python package perspective: add dependency on javascript engine instead of bundling it. To prevent ending up with multiple js engines inside one python environment.

But I agree with all above comments to prefer the current Deno setup. I'll close this issue. Thanks!


revisit CDN issue because GitHub Pages now enforces https

I realize that https CDN issues have been discussed in the past (https://github.com/Leaflet/Leaflet/issues/4080), but it may be time to revisit Leaflet documentation on this topic.
Last week, GitHub Pages began to enforce https. This means that anyone who follows the current Leaflet documentation on "Using a Hosted Version" will encounter errors when hosting on GitHub Pages, as I recently discovered. Not fun.
I saw @Mourner state his preference for NPMCDN, but at present it only supports Leaflet 0.7.7, and none of the newer versions: https://npmcdn.com/leaflet/
Whether the Leaflet team decides to update NPMCDN, or point us to CDNJS or somewhere else, I encourage you to revise the documentation.  Thanks.

@JackDougherty that's a valid concern, I think we should replace the links with NPMCDN ones. All recent versions like 1.0.0-rc.1 should be present there as well since they're released on NPM too.

According to NPMCDN docs, this link should show all Leaflet contents
https://npmcdn.com/leaflet/
But only 0.7.7 appears there.

@JackDougherty it's probably because 1.0.0-rc.1 doesn't have a "latest" tag in NPM. But files are accessible, e.g<EMAIL_ADDRESS>

Fixed with #4679


unsupport expo.....

Hi @tiyo. Not sure if you are asking about Expo Go support or not, if you are: https://github.com/bluesky-social/react-native-uitextview/issues/9#issuecomment-2118625123
Feel free to open something else if you just accidentally left this issue blank!


Fix mod

bors r+

seems like a nightly-stable regression

https://github.com/rust-lang/rust/issues/79487

bors r+


Binding Path=ActualHeight-20 Error

Help~
Plus is okay，But subtract is error！
What should I do ?

you can try {Binding ActualHeight-20, Elementname=xxx}

you can try {c:Binding ActualHeight-20, Elementname=xxx}


Stape Logging standart

It will be great if tag will support Stape Logging Standard.
Currently it supported by majority of sGTM tags.
https://github.com/stape-io/gtm-standards/blob/main/standards/GTMS-3.md
If it's good for you I can prepare PR.

Hi @Bukashk0zzz!
That would be a good addition. Feel free to draft/prepare a PR.

Thanks for the contribution @Bukashk0zzz :).


Why do I have to install angular-cli@webpack locally if I have already installed globally?

I did npm install -g angular-cli@webpack.
After I ng new a project, I found that it installs a local 1.0.0-beta.11-webpack.8 .
If I delete it, ng serve complains about the project is not a valid angular-cli project.

you have the option to link to the global path when you create a new project and it will symlink to it with ng new  --link-cli

would it be possible to have an automatic detection, if angular-cli is globally available?

It's ideal to not have a global cli or rely on any global packages.  If one can pull a project down and run npm i and have everything needed, that is better IMO.

The global install is what allows you to do stuff like ng new project, while the local one is used afterwards. This is especially relevant when using different versions of global and local, across different projects.


[WIP] return inserted record

Trying out build similar to #1383

[ ] Unit test all changes
[ ] Update README.md if applicable
[x] Add [WIP] to the pull request title if it's work in progress
[ ] Squash commits that aren't meaningful changes
[ ] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

Close in favor of #1489


Error response from daemon failed to create endpoint <> on network bridge: failed to set MTU on host interface vethxxxxx: invalid argument

Output of docker version:
Client:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   8acee1b
 Built:
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   8acee1b
 Built:
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 3
 Running: 0
 Paused: 0
 Stopped: 3
Images: 2
Server Version: 1.10.3
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.5.0-coreos-r1
Operating System: CoreOS 1010.1.0 (MoreOS)
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 997.2 MiB
Name: master
ID: QNQT:L2CW:ROUB:GLKX:V5UV:DKJI:EPAT:A5TG:B5KL:UOFQ:JTNC:MVNS

Additional environment details (AWS, VirtualBox, physical, etc.):
Vagrant script creating 1 master (running Kubernetes and a Docker registry) and 2 nodes (running docker images). All through VirtualBox.
Steps to reproduce the issue:

Do a 'vagrant up' at work.
Go home and do 'vagrant halt', then 'vagrant up'
in Master image, I run 'docker run -d -p 5000:5000 registry:2'

Describe the results you received:
I'm  getting a
a3918b8c23286f4a46565edd562fb716bc025d4b2d7b040c29eeae8669b2a21d
docker: Error response from daemon: failed to create endpoint berserk_newton on network bridge: failed to set MTU on host interface veth45f9316: invalid argument.

While I can understand the network as changed between the work and home, I would expect the guest machine to "reset" the network config update halt/up of the vagrant script.
Describe the results you expected:
Is there a way to reset the network in the guest OS to avoid the "vagrant destroy" and "vagrant up" which takes a long time as I need to pull some (big) images from my company's docker registry.

I don't think we can help you much here; you're using Vagrant, CoreOS and VirtualBox; none of these are something from this repository, so I suggest to ask this question in the issue tracker for one of those

Thanks. I agree it's a mix of different products and therefore hard to figure out...
One work around is to stop the Vagrant VMs, restart my latop and start them again. It seems the hibernation I was doing on my laptop causes this issue.

Yes, I think this is a VirtualBox issue, so not sure how it can be resolved in a different way

Output of docker version:
`Client:
Version:      1.11.1
API version:  1.23
Go version:   go1.6.2
Git commit:   5604cbe
Built:        Mon May  2 00:06:51 2016
OS/Arch:      linux/amd64
Server:
Version:      1.11.1
API version:  1.23
Go version:   go1.6.2
Git commit:   5604cbe
Built:        Mon May  2 00:06:51 2016
OS/Arch:      linux/amd64
Output ofdocker info:
Containers: 4
Running: 0
Paused: 0
Stopped: 4
Images: 5
Server Version: 1.11.1
Storage Driver: devicemapper
Pool Name: docker-8:6-3160898-pool
Pool Blocksize: 65.54 kB
Base Device Size: 107.4 GB
Backing Filesystem: ext4
Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 2.541 GB
Data Space Total: 107.4 GB
Data Space Available: 94.24 GB
Metadata Space Used: 2.322 MB
Metadata Space Total: 2.147 GB
Metadata Space Available: 2.145 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Deferred Deletion Enabled: false
Deferred Deleted Device Count: 0
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
WARNING: Usage of loopback devices is strongly discouraged for production use. Either use --storage-opt dm.thinpooldev or use --storage-opt dm.no_warn_on_loop_devices=true to suppress this warning.
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.122 (2016-04-09)
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: null host bridge
Kernel Version: 4.5.1-1-ARCH
Operating System: Antergos Linux
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 7.609 GiB
Name: archost
ID: NPBP:MZFF:ISB7:6SSN:PU6H:GBF6:NWSO:F3KL:UV5M:S4S2:VLST:WTU7
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): false
Registry: https://index.docker.io/v1/
Output ofdocker start b550b490a45f :
Error response from daemon: failed to create endpoint admiring_darwin on network bridge: failed to add the host (vethe955b0a) <=> sandbox (veth250aedd) pair interfaces: operation not supported
Error: failed to start containers: b550b490a45f
`

I am encountering same error like mentioned above ...heck
bhaskar@ArchLinux_12:58:31_Mon Jul 18:~>sudo docker run -d -p 5000:5000 --name registry registry:2 [sudo] password for bhaskar:  ccf9156579dc4673d689aa101f896275163921d917b1143aac74008dfcdf4067 docker: Error response from daemon: failed to create endpoint registry on network bridge: failed to add the host (veth3995dd3) <=> sandbox (vethe432b93) pair interfaces: operation not supported.

Well , it was damn "virtualbox" module not loaded properly causing that troube after kernel update.
Fix:
reboot the machine and voila!
bhaskar@ArchLinux_13:56:39_Mon Jul 18:~>docker run -d -p 5000:5000 --name registry registry:2 c091161e01ee751b13c6136b7f3dc84e563704dcea4b435bcd17b2d70a30cb46 bhaskar@ArchLinux_13:56:47_Mon Jul 18:~>docker ps -a CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS                    NAMES c091161e01ee        registry:2          "/bin/registry serve "   7 seconds ago       Up 4 seconds        <IP_ADDRESS>:5000->5000/tcp   registry

I use Docker on RaspberryPi. On build image I have error 'failed to create endpoint gracious_archimedes on network bridge: failed to add the host (vethc33f967) <=> sandbox (vethc453c00) pair interfaces: operation not supported
'


add refer to add gem for rails 5.1+

as in https://github.com/twbs/bootstrap-rubygem, refer needs too.

Already fixed in the next branch, closing


Uncaught ReferenceError: turnstile is not defined

There can be some instances when the code for the hook runs before Turnstile is loaded.
In those cases you'll see some error like:
Uncaught ReferenceError: turnstile is not defined
    at mw.mounted (app-5bd87c1dd10a5790e59652fef1818e7d.js?vsn=d:615:6089)
    at mw.__mounted (app-5bd87c1dd10a5790e59652fef1818e7d.js?vsn=d:30:26229)
    at k5.maybeAddNewHook (app-5bd87c1dd10a5790e59652fef1818e7d.js?vsn=d:30:38586)
    at app-5bd87c1dd10a5790e59652fef1818e7d.js?vsn=d:30:37656
    at Array.forEach (<anonymous>)

In the mean time I wrote some code that retries until it's defined:
https://github.com/lobo-tuerto/phoenix-turnstile/commit/06feb8e60848f6cdc44878e4aa5591cd7347d039
If this is acceptable I can make a PR.
Cheers!

A current workaround for this is to place Turnstile.script in above the script tag that loads app.js (in root.html.heex), that way the turnstile dependency is loaded before the hook is executed


fuse: failed to open /dev/fuse: Operation not permitted

debian.
According to this page the installation was successful，but the last step failed。
google-drive-ocamlfuse -label googledrive  /googledrive
fuse: failed to open /dev/fuse: Operation not permitted

Have you tried mounting with -o allow_other?

Are you mounting in an lxc container?

Are you mounting in an lxc container?

OpenVZ  Server.Debian GNU/Linux 7.9 (wheezy).
Can't mounting？
Thanks

I did manage to make to make google-drive-ocamlfuse work in an LXC container. But you need to configure AppArmor and OpenVZ to allow access to /dev/fuse.


chore: Remove redundant dependencies

Explanation
In various packages, we have a package listed as both a peer dependency and as a dependency. This was done by mistake, the dependency entries aren't actually used in practice, as we require the package to be installed as a "peer" package anyway. It's also incorrect to list other controllers as a dependency in this case because we don't know which version we need. In these cases we need it to match the version setup with the global messenger, which we don't know.
References
None
Changelog
Changelogs updated in diff
Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Ideally we'd have a constraint for this as well, but I'll wait for the Prolog => JavaScript conversion to be merged before attempting that, to avoid slowing that effort down (and because I don't want to re-learn Prolog again 😅 )


Farming station damage Flux-infused tool

EnderIO Version: <IP_ADDRESS>7
Redstone Arsenal Version: 1.1.0 RC1-24
When I use Flux-infused axe for farming station, it will damage the tool instead of using RF.

The farm was still looking for instanceof ItemAxe instead of using the new enum. Fixed now.


Add

🙋 Feature Request
I would love to see   added as a scrollable counterpart of . Scrollability could also be achieved by adding necessary props to , but having a separate component would be inline with SwiftUI, React Native, and others.
🤔 Expected Behavior
The  component should strictly extend  in functionality, everything available for  should be available here. It should have props for controlling scroll, for example for scroll-snap. Scroll direction should follow the "direction" prop, vertically scrolling horizontal flexes or the other way around shouldn't be possible.

That's an interesting idea. Would this be equivalent to overflow: auto on the container or something more? I think SwiftUI's Lazy stacks are virtualized. Is that what you're thinking?

I thought about virtualization. What we really need is not just virtualization, but solving the whole "representing a big amount of items" problem. That involves infinite scrolling in both directions, loading items async, managing loading state, cleaning data for unused items and a lot more. It seemed it would be out of scope of this project so I didn't mention.
So I was just thinking about an explicit component for scrollable views. I want scrollability to be clearly seen and not hidden under props, styles or classes. I also believe this will push developers to be more conscious about them. (Accidentally nesting them causes problems hard to catch for example.)

Yeah. We have a <Virtualizer> component that we use internally inside components like Picker and ListBox already, which supports different configurable layouts. It supports most of the features you mentioned there. I really liked SwiftUI's LazyHStack/LazyVStack components when I saw them, so I think some kind of LazyFlex component might make sense for React Spectrum. It could possibly use the Virtualizer internally.


Pending icon for show page

Added pending icon to importer show page for fileset entries and exporter show page for work entries.

@cla-bot recheck

@cla-bot check


Added conversion from ed25519 Public and Private Key

I added the option for conversion from ed25519-dalek. I implemented this to make procedures possible, where both Diffie-Hellmann and Signing is required. Please review this code carefully as I don't really understand the underlying math.

I forgot to update the version number in Cargo.toml. When merging the pull request please change this and update the crate on Crates.io.

Found major mistake. Closing.


Fail to reset tag queries which in the address bar when navigating to the Snapshots/Versions view previously

Storage Explorer Version: 1.28.0-dev
Build Number: 20221207.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.0.1 (Apple M1 Pro)
Architecture: ia32/x64
How Found: Ad-hoc testing
Regression From: Previous release (1.21.3)
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create one blob container -> Upload one blob.
Right click the blob -> Click 'Edit Tags...' -> Add one tag for the blob.
Open the view options panel -> Select the 'Tag Query' -> Input the blob's tag -> Click 'Apply'.
Click 'Manage History -> Manage Snapshots/Manage Versions' -> Exit the 'Snapshots/Versions' view.
Reset the tag query -> Check whether succeeds to reset tag queries in the address bar.

Expected Experience
Succeed to reset tag queries.
Actual Experience

Fail to reset tag queries.
Navigate to the Snapshots/Versions view.

Additional Context
Similar issue: 4756.

Verified this issue on the build main 20221211.1, it still reproduces.

Verified this issue on the build 20221222.8. Fixed. So, we are going to close it.