version 2.0.3 broke google auth

When I use the latest version of the package (2.0.3) i have the following problem:
Google\Cloud\Core\Exception\BadRequestException
<?xml version='1.0' encoding='UTF-8'?><Error>InvalidAuthentication<Message>The provided authentication header is invalid.</Message>Cannot use OAuth Authorization header with form POST.</Error>

When reverting to 2.0.2... all work fine.

@hotrush can you take a look at this?

@veneliniliev share please your config file (without secrets of course, just structure)

Tested:

with path to config file

        'gcs' => [
            'driver' => 'gcs',
            'key_file_path' => '/app/gcs.json',
            'key_file' => [],
            'projectId' => null,
            'bucket' => 'BUCKET',
            'pathPrefix' => '',
            'apiEndpoint' => null,
            'visibility' => 'private',
            'metadata' => ['cacheControl'=> 'public,max-age=86400'],
        ],

with copied config file

        'gcs' => [
            'driver' => 'gcs',
            'key_file_path' => null,
            'key_file' => [
                'type' => 'service_account',
                'project_id' => 'alien-device-123123',
                'private_key_id' => '12312',
                'private_key' => "-----BEGIN PRIVATE KEY-----...",
                'client_email' => '123.iam.gserviceaccount.com',
                'client_id' => '123',
                'auth_uri' => 'https://accounts.google.com/o/oauth2/auth',
                'token_uri' => 'https://oauth2.googleapis.com/token',
                'auth_provider_x509_cert_url' => 'https://www.googleapis.com/oauth2/v1/certs',
                'client_x509_cert_url' => 'https://www.googleapis.com/robot/v1/metadata/x...'
            ],
            'projectId' => null,
            'bucket' => 'BUCKET',
            'pathPrefix' => '',
            'apiEndpoint' => null,
            'visibility' => 'private',
            'metadata' => ['cacheControl'=> 'public,max-age=86400'],
        ],

with exported env var

export GOOGLE_APPLICATION_CREDENTIALS=/app/gcs.json plus
        'gcs' => [
            'driver' => 'gcs',
            'key_file_path' => null,
            'key_file' => [],
            'projectId' => null,
            'bucket' => 'BUCKET',
            'pathPrefix' => '',
            'apiEndpoint' => null,
            'visibility' => 'private',
            'metadata' => ['cacheControl'=> 'public,max-age=86400'],
        ],

Works perfect to me...

it also works well for me locally.
the interesting thing is that when I built it on the docker image and put it in the kubernetes cluster then the problem happens.

Might help if you tell us how you configure the package 😄
Do you use GKE and creds from automatically mounted GOOGLE_APPLICATION_CREDENTIALS?

i don't use GOOGLE_APPLICATION_CREDENTIALS
in filesystems
 'gcs' => [
            'driver' => 'gcs',
            'key_file_path' => resource_path(env('GOOGLE_CLOUD_KEY_FILE', null)), // optional: /path/to/service-account.json
            //'key_file' => [], // optional: Array of data that substitutes the .json file (see below)
            'project_id' => env('GOOGLE_CLOUD_PROJECT_ID', 'your-project-id'), // optional: is included in key file
            'bucket' => env('GOOGLE_CLOUD_STORAGE_BUCKET', 'your-bucket'),
            'path_prefix' => env('GOOGLE_CLOUD_STORAGE_PATH_PREFIX', ''), // optional: /default/path/to/apply/in/bucket
            'apiEndpoint' => env('GOOGLE_CLOUD_STORAGE_API_URI', null), // see: Public URLs below
            'visibility' => 'public', // optional: public|private
            'metadata' => ['cacheControl' => 'public,max-age=86400'], // optional: default metadata
        ],

in .env
GOOGLE_CLOUD_PROJECT_ID=xxx
GOOGLE_CLOUD_KEY_FILE="xxx/service-account.json"
GOOGLE_CLOUD_STORAGE_BUCKET=xxx
GOOGLE_CLOUD_STORAGE_API_URI="https://xxxx/"

env loaded correctly in pod
# env | grep 'GOOGLE'
GOOGLE_CLOUD_STORAGE_API_URI="https://xxxx/"
GOOGLE_CLOUD_STORAGE_BUCKET=xxxx
GOOGLE_CLOUD_PROJECT_ID=xxxx
GOOGLE_CLOUD_KEY_FILE="xxxx/service-account.json"

the configuration is the same as in 2.0.2 where it works without a problem.

Changes i made fixes passing apiEndpoint into the client, so in your case i think it makes sense to remove GOOGLE_CLOUD_STORAGE_API_URI="https://xxxx/". It doesn't make sense to override it usually, only when you use some side gcs-compatible storage.
If won't help - some questions:

Why do you override default api url?
Is your GOOGLE_CLOUD_PROJECT_ID matching with project id from creds file?
Key file path is relative? Makes sense to check if that correct

@hotrush I can confirm removing the GOOGLE_CLOUD_STORAGE_API_URI from env resolved this issue for me.

@hotrush I can confirm removing the GOOGLE_CLOUD_STORAGE_API_URI from env resolved this issue for me.

resolve the issue but if you use a custom domain for cloud storage?
without this parameter, the domain is https://storage.googleapis.com/...
but I want to use my custom domain ex https://cdn.custom-domain.com/...
how to set my own domain?

Same problem of @veneliniliev here, and with a custom domain as bucket name, too.
After several hours trying to fix it, it worked when i downgraded to 2.0.2, too.


Anonymous-Auth=true still required?

I was reviewing the default settings and wondering if this is still required? It mentions haproxy, but i believe this was never used in this project
https://github.com/k8s-at-home/template-cluster-k3s/blob/main/provision/ansible/inventory/group_vars/master/k3s.yml#L49
  # Required for HAProxy health-checks
  kube-apiserver-arg:
    - "anonymous-auth=true"

Not required, but doesn't hurt to enable it if people wanted to use HAProxy as a external LB for kube API

Okay, I was wondering as CIS benchmarks recommend against it (although they don't score it if it's enabled), see also: https://rancher.com/docs/k3s/latest/en/security/self_assessment/#1-2-1

Makes sense, but I am not too concerned about security as this project is more oriented towards running this all at home where we are usually more relaxed on certain aspects of security.

Gotcha, I can still remove it in my fork. Thanks for the quick answer on this topic! :+1:


FFI Client Mock + Data Mock + API tests

API Tests can use TestResources (see Squeak by Example) + Test Spy Pattern

#362


changed React.PropTypes -> PropTypes

ran this codemod: https://github.com/reactjs/react-codemod/blob/master/transforms/React-PropTypes-to-prop-types.js to the entire ./lib dir

we're using this sparingly now but it's still being used nonetheless 😞

still?! ugh 😞
just die already

some legacy modal in .com

weird...this didn't fail even though prop-types is not in package.json. I'm updating just so that it doesn't fail further down the line.


Update Format.yml

JuliaFormatter currently adds a lot of noise to every PR conversation.
This makes the workflow less invasive and similar to the tests workflows.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 72.55%. Comparing base (1aa6087) to head (fb338fa).
Report is 3 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1428       +/-   ##
===========================================
- Coverage   94.07%   72.55%   -21.52%
===========================================
  Files           8       29       +21
  Lines         388    10611    +10223
===========================================
+ Hits          365     7699     +7334
- Misses         23     2912     +2889

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


doc: Better version handling

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[x] Documentation Update
[ ] Other: Replace this with a description of the type of this PR

Description
TODO
Related Issues & Documents

Documentation (check all applicable)

[ ] This PR requires changes to the Wiki documentation (describe the changes)
[x] This PR requires changes to the documentation inside the git repo (please add them to the PR).
[ ] Does not require documentation changes

Done by #3030


Add missing config option to oc scale commands

Currently scale commands fail:
00:51:31         "2019-04-18 00:51:30 | + oc scale deployment -n openshift-machine-api machine-api-controllers --replicas=0",
00:51:31         "2019-04-18 00:51:30 | error: Missing or incomplete configuration info.  Please login or point to an existing, complete config file:",
00:51:31         "2019-04-18 00:51:30 | ",
00:51:31         "2019-04-18 00:51:30 |   1. Via the command-line flag --config",
00:51:31         "2019-04-18 00:51:30 |   2. Via the KUBECONFIG environment variable",
00:51:31         "2019-04-18 00:51:30 |   3. In your home directory as ~/.kube/config",
00:51:31         "2019-04-18 00:51:30 | ",
00:51:31         "2019-04-18 00:51:30 | To view or setup config directly use the 'config' command."

Sigh, we should just set KUBECONFIG instead of duplicating this everywhere


Remove Python specific syntax from all occurrences of itemsList

At the moment the templates include the following logic for iterating over an itemsList:
{% for item in (params.itemsList if params.itemsList is iterable else params.itemsList.items()) %}

This is because with Python a dictionary value is being provided for itemsList:
{{
  onsList({
    itemsList: someDictionary
  })
}}

We could reduce the complexity of our macro templates by reducing our iteration logic to the following:
{% for item in params.itemsList %}

The user code can then add .items() as shown below:
{{
  onsList({
    itemsList: someDictionary.items()
  })
}}

Can we actually do this? it would require that user code be updated.

@rmccar - we would like to move forward with this, can you test moving the python specific code into eQ?

@rmccar - we would like to move forward with this, can you test moving the python specific code into eQ? We will look to issue comms and discuss this in the next monthly forum

I have tested this change in eQ and it doesn't actually seem to be a breaking change for us


Check for binary deps, handle errors gracefully, and offer to install

elm-oracle
elm-format
etc.

Fixed by 6e2afbf586fbd42d208113299dbdc515c3a0afcf


Custom dates parsing error

Due to the changes in parsing, in some cases it would not be possible to use a custom format with length different than 10 characters.
Changes:

For asSingle input, use whole input to parse the date
For not asSingle input, use separator or try to split input in half.
It doesn't hide date picker by default when a correct date is entered - ENTER key has to be pressed. This is due to the fact that if someone is using a single day or month, and they want to type 10, typing 1 would close the date picker and remove the focus from the input. Now they can type 1, then 2 to get 12 and press enter. Example: If using format YYYY-M-D if you want to get 10th January 2023, after typing 2023-1-1 it would close the picker and lose focus for input. Now you can type 2023-1-10, on change will fire twice for 2023-1-1 and 2023-1-10, but only after pressing enter or clicking outside you will close date picker and lose focus.
separator and displayFormat are not nullable, there is always some value for them (either default or user input)

Checked with multiple date formats, both single and not.
Works also for case described by @poshiemaaat here.
We could look into adding unit testing? @onesine

Yes!!!!!
The only thing I’d change is adding the onChange to fire when tabbing as well. The user should be able to click enter or tab for it to calculate but this is minor in the scheme of things. I’m so excited to get home and play with this. Thank you!
What are the chances we see a minor version update with this change?! 😂

@t0m3k @onesine what can I do to get these PRs merged? This one in particular is top of mind but I'd love to volunteer my time to get this in as well as some of the other features. This is stopping us from using a custom displayFormat - I'd like to help!

Hi @t0m3k & @poshiemaaat 👋
Sorry for the delay. I'll have a quick look at the PR.
Thanks @t0m3k for this PR. It would be interesting, as you said, to set up unit tests. I'll be happy to see what you can suggest in that regard.
Also thanks to @poshiemaaat for the interest in this package. We would really appreciate your help. It would be very interesting if you could help us with some issues.


Intermittent CI failure: restore_working_directory FileNotFoundError

e.g. in https://github.com/simonw/datasette/runs/2754772233 - this is an intermittent error:
__________ ERROR at setup of test_hook_register_routes_render_message __________
[gw0] linux -- Python 3.8.10 /opt/hostedtoolcache/Python/3.8.10/x64/bin/python

tmpdir = local('/tmp/pytest-of-runner/pytest-0/popen-gw0/test_hook_register_routes_rend0')
request = <SubRequest 'restore_working_directory' for <Function test_hook_register_routes_render_message>>

    @pytest.fixture
    def restore_working_directory(tmpdir, request):
>       previous_cwd = os.getcwd()
E       FileNotFoundError: [Errno 2] No such file or directory

Stack Overflow: https://stackoverflow.com/a/49367679/6083

The answer was that os.chdir() had been set to the deleted directory by accident. The directory was missing, but the error happened (it seems) at the attempt to get it with os.getcwd().

I'm going to try removing that restore_working_directory fixture entirely.

That broke things. Here's how pytest-cov fixed a similar issue: https://github.com/pytest-dev/pytest-cov/commit/7ccb1783bf8290447df58deeb41eae74296a6d9b
See also https://github.com/nedbat/coveragepy/issues/881 and https://github.com/pytest-dev/pytest-cov/issues/306 and https://github.com/nedbat/coveragepy/issues/824

That seems to have fixed it. I'd love to get rid of this restore_working_directory hack entirely.


Add an example of an unsafe fix

Per review in #7901 adds an example of an unsafe fix.

That's a great example!


Fix fxml JavaFX xmlns versions, hopefully permanently

git hook should automatically fix bad xmlns versions
closes #153

Codecov Report

Merging #212 (7e42752) into master (e9cda09) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #212   +/-   ##
=========================================
  Coverage     65.09%   65.09%
  Complexity      596      596
=========================================
  Files           111      111
  Lines          2484     2484
  Branches        262      262
=========================================
  Hits           1617     1617
  Misses          755      755
  Partials        112      112

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 e9cda09...7e42752. Read the comment docs.


(Crossposting Issue) Installing Plotlykt-Core as dependency causes project build to fail (without any new code added)

I have a Kotlin/JS for browser targeted program that builds fine how it is, until I add Plotlykt as a Gradle dependency. This causes one of my files, which contains Kotlinx Serialization, to fail to compile. I created an issue report for the Kotlinx Serialization repo first, but nothing suggests the issue is necessarily on that end.
Please see this issue for a detailed error:
https://github.com/Kotlin/kotlinx.serialization/issues/1030
and this project for a reproducible error (contains everything from the Intellij project to the Build Script):
https://github.com/CaptainZidgel/Erroring_Project
I hope this finds you well, please let me know if there are any other details I can give you.

Thank you for your report. The current version is not compatible with 1.4 and new serialization. I will fix it shortly. Right now you can stick with 1.3.72 or try to remove transitive dependency on serialization runtime. I will add appropriate warning to the readme. You also do not need serialization itself to run plotly.kt. it only uses json API, not the plugin.

Thank you for the swift response.
I will try to convert my project to 1.3.7 code and dependencies that support 1.3.7 (I did not add Serialization as an imagined dependency for Plotly, I need it for my project). If I cannot do that, I will shelve the project until Plotlykt supports 1.4 or I can find another JS-capable Kotlin plotting library.

1.3.72 should work (alongside with serialization 0.20.0). The demo project is here: https://github.com/mipt-npm/plotly.kt/tree/master/js-demo.
I will make a dev build with 1.4 in a few days. All the necessary libraries are already converted. But you should remember that kotlin-js broken compatibility on 1.4 and pre-1.4 libraries are in general not compatible with post - 1.4 libraries. I've added a compatibility note to address the problem.

@CaptainZidgel I've updated the dependencies, but sadly I get https://youtrack.jetbrains.com/issue/KT-39088 on JS compilation, so we will have to wait for 1.4.20. You can safely use the current version with 1.3.72.

Thank you for your help!

@CaptainZidgel The bug prevention Plotly core from working was fixed in 1.4.20, so we will migrate as soon as EAPs are out. For now just use 1.3.72.

https://github.com/mipt-npm/plotly.kt/releases/tag/v0.3.0-dev-3 is published. It requires Kotlin eap build, but works fine otherwise.


Add tasks

[o] Authentication

[x] Google Authentication
[ ] unit tests
[ ] e2e

[o] Task CRUD

[x] Add Task
[x] Remove Task
[ ] Filter Tasks
[ ] unit tests
[ ] e2e

[o] Block Task

[x] Show Blocked Tasks
[ ] Block/Unblock Task
[ ] unit tests
[ ] e2e

[o] Member Logged In

[x] Authenticated User
[ ] Team Member
[ ] unit tests
[ ] e2e

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


problem: instructions are outdated and broken

solution: append v2 to the import path.
See #795 for context.
Description
Reference issue
Fixes #...

Fantastic, thank you @gazhayes for making things better for everyone. Merged with 63b2d4da428988ecb7dc3e97556760ceeeeac39e
Sorry the project has rough edges :( if you have any other suggestions on how to make the project more friendly for people joining I would love to hear!
I am on Slack and excited to chat anytime.
thanks

Hi @Sean-Der you may wish to consider this contribution protocol: http://socialarchitecture.science/c4/ (a deeper explanation for each line is here: http://socialarchitecture.science/c4-deep/). It essentially turns the development process into a decentralised hill-climbing algorithm and makes things a lot easier for maintainers.


Add get and set for Env

This PR add three things :

Add get and set in Env.
Implement fixture meta for with_single_file.
Add env meta in fixture.

bors r+

rebased.
bors retry


Beta Suggestion: do - while loop

Challenge iterate-with-javascript-while-loops has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
Since the loop mechanism is taught differently, this seems to be a great place to teach the do while loop.

I'll work on this!


[React + Next.js]: Error: Plugin/Preset files are not allowed to export objects

Describe the bug
ERROR in ./.storybook/config.js
Module build failed: Error: Plugin/Preset files are not allowed to export objects, only functions. In /[pathname]/node_modules/babel-preset-stage-0/lib/index.js
Storybook won't load after a fresh install.
Install with directions from here:
https://storybook.js.org/basics/guide-react/#create-the-config-file
Add additional suggestions for a custom install because I'm using Next.js here:
https://gist.github.com/metasean/cadd2becd60cc3b295bf49895a56f9b4
To Reproduce
yarn storybook
"storybook": "start-storybook -p 9001 -c .storybook",
Code snippets
Package.json
"devDependencies": {
    "@storybook/react": "^3.4.11",
    "babel-jest": "^23.6.0",
    "babel-plugin-module-resolver": "^3.1.1",
    "babel-preset-react-app": "^5.0.5-next.c662dfb0",
    "casual": "^1.5.19",
    "enzyme-to-json": "^3.3.4",
    "graphql-tools": "^4.0.0",
    "jest": "^23.6.0",
    "jest-transform-graphql": "^2.1.0"
  }

System:

OS: Macbook Pro, Node v8.12.0
Framework: React.js
Version: 3.4.2

Additional context
Using Next.js, which I believe has all it's own config files. Someone here must have gotten Storybook setup with it before, so checking. :)
Also tried installing Babel-loader 8 as per previous tickets, and it didn't help.

This error means a mismatch between babel 6 (which is used in SB v3,4) and babel 7 (probably used in Next) the v3.4 will never use babel 7 out of the box.
You can:

Replace SB js rules with the full control mode
Upgrade to SB v4.

Thank you! I upgraded my package.json manually to 4.0.0, deleted my node_modules folder, and re-installed via yarn and everything worked.


Tabs don't update column counter

Repro:

Hit the tab key
Column count in gutter doesn't update

Forgive the GIF quality.

Hi, can I work on this bug?

@boyP yes we do accept pull requests :).

@alexandrudima not sure the fix is right, I can also change the editorStatus.ts to react on any types.

Indeed, I think the status bar should reflect the cursor state, regardless of the reason the cursor changed its state. So the fix should be made in editorStatus

Thanks for fixing this!

:heart:

:tada: Thanks!


Each named import/export in its own line

This style rule helps with rebasing. And also makes a decision for us. So one less decision for us to make.
https://github.com/gmsorrow/eslint-plugin-modules-newline

Hmmm, I'm not exactly sure what it does, would be nice with some examples...
If it disallows this:
import { Alert, Platform, Text, View } from 'react-native'

in favour of:
import { Alert } from 'react-native'
import { Platform } from 'react-native'
import { Text } from 'react-native'
import { View } from 'react-native'

Then I would personally be very much against that rule 😅

It's
import {
  a,
  b,
} from 'foo'

Each named import or export in its own line. That's a lot of lines, possibly. But I'd pay that happily, to have easier rebasing.


ERROR (MainThread) [homeassistant.components.broadlink] Failed to connect to device

Home Assistant version 0.101.3
SmartIR version 1.7.3
SmartIR configuration
smartir:
switch:

platform: broadlink
host: <IP_ADDRESS>
mac: '24:df:a7:36:76:6e'

climate:

platform: smartir
name: Condizionatore Salotto
unique_id: salotto_ac
device_code: 1121
controller_data: <IP_ADDRESS>
temperature_sensor: sensor.temperature
humidity_sensor: sensor.humidity
power_sensor: binary_sensor.ac_power

fan:

platform: smartir
name: Ventola Cucina
unique_id: cucina_fan
device_code: 1000
controller_data: <IP_ADDRESS>
power_sensor: binary_sensor.fan_power

Add configuration here

Describe the bug cannot connect to device, tested even with "broadlink.learn"
A clear and concise description of what the bug is.
Debug log
2019-11-12 12:20:37 ERROR (MainThread) [homeassistant.components.broadlink] Failed to connect to device
Add your logs here.

**Additional context**

Add any other context about the problem here.

Anyone on that ?
Thank you,
Simone

It looks like a problem on your Broadlink and not on SmartIR

Hi @smartHomeHub, device works fine with both IHC for EU and Broadlink apps.
Is just with SmartIR that it doesn't work.
Is there a way to get more debug and analyze the issue ?
Thx,
Simone

SmartIR is using the broadlink.send service. Could you try if the service sends commands to your broadlink device?

SmartIR is using the broadlink.send service. Could you try if the service sends commands to your broadlink device?

Nope, it doesn't. Same error in log: Failed to connect to device

Not sure why this is still(?) marked as closed.
Simone still has problems, and I am seeing the exact same issue when I try to call the service with broadlink.send
I've got two different devices - both working perfectly fine through the Broadlink app, both showing in my UniFi as connected with proper IP addresses.
Neither of them are responding - same error as above
'<IP_ADDRESS>' Traceback (most recent call last):   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/components/websocket_api/commands.py", line 130, in handle_call_service     connection.context(msg),   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/core.py", line 1234, in async_call     await asyncio.shield(self._execute_service(handler, service_call))   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/core.py", line 1259, in _execute_service     await handler.func(service_call)   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/components/broadlink/__init__.py", line 48, in _learn_command     device = hass.data[DOMAIN][call.data[CONF_HOST]] KeyError: '<IP_ADDRESS>'
'<IP_ADDRESS>' Traceback (most recent call last):   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/components/websocket_api/commands.py", line 130, in handle_call_service     connection.context(msg),   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/core.py", line 1234, in async_call     await asyncio.shield(self._execute_service(handler, service_call))   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/core.py", line 1259, in _execute_service     await handler.func(service_call)   File "/Users/admin/homeassistant/lib/python3.7/site-packages/homeassistant/components/broadlink/__init__.py", line 48, in _learn_command     device = hass.data[DOMAIN][call.data[CONF_HOST]] KeyError: '<IP_ADDRESS>'


Chapter 14: Basic RNN results differs from "Static Unrolling"

Hi,
Can you please explain little bit why the simulation code (basic RNN) is differ from using "Static Unrolling" as they are given the same input. However, you have mentioned that both creates the "exact same model".
Best,
Achintya

Hi @achintyasarkar ,
Thanks for your question.  The weights are initialized randomly, so the output will vary.  But the math equations are the same.
Hope this helps,
Aurélien

Thanks for your reply. It's now clear.
Best,
Achintya


Data entry: Corrected Rainbow fumarole

Wrong damage at choice 3 yellow.

Would you mind clarifying what is being corrected? PR titles should also be in imperative mood.


Missing icon on KDE panel

Step to reproduce problem:

Install Oracle Linux 9.4 with option "Minimal Install"
Enable CodeReady repository
Install oracle-epel-release-el9
Install group kde-desktop-environment
switch default target to graphical.target for systemctl
reboot

Everything works fine except for the lack of an icon on the application launcher button:

On Rocky Linux and CentOS, the same steps will get you a panel with an icon on the button.

Oracle Linux customers, please file your issue at https://support.oracle.com
Thanks for filing an issue with Oracle Linux.
GitHub Issues is not an official support channel and we don't offer
product support here. If you're not yet an Oracle Linux customer,
consider signing up at https://linux.oracle.com.
Even if you're not a customer, if we can confirm that an issue is a
bug we will do our best to fix it and to update this issue
once it has been fixed. We don't guarantee a fix or feedback and
for now, we will close this issue. If you have Oracle Linux support,
please use support.oracle.com to report issues.

I'm not a customer.
Could you explain where exactly I can post issue at https://support.oracle.com?

You can't if you are not a customer.  As the comment above states, "if we can confirm that an issue is a bug we will do our best to fix it and to update this issue once it has been fixed."

I don't actually see kde-desktop-environment as an option after enabling EPEL.  Can you provide the exact commands you are running?

sudo dnf config-manager --set-enabled ol9_codeready_builder
sudo dnf install oracle-epel-release-el9
sudo dnf install @kde-desktop-environment
4.1. sudo dnf install plasma-milou - krunner doesn't work without this package
sudo systemctl set-default graphical.target
reboot
After these commands you will get KDE desktop environment. Everything works fine except "launch application button on the panel"

Thanks, not sure why this wasn't working for me before.


Service_L4 persistenceMethods blank none error

I'm trying to have a runtime init yaml install and declare AS3 with a Service_L4 service. I want persistence None, but the default for Service_L4 is sourceIP. No matter what combination I try, runtime init keeps failing.
From reading the docs on AS3, you can supply an empty array with empty brackets [], but I tried and it failed.
BIG-IP version = <IP_ADDRESS>
Runtime init = 1.3.2
Expected Behavior
I want to deploy a Service_L4 with persistence none in runtime init with this type of AS3 json. Notice a blank array for persistence methods.
{
    "class": "ADC",
    "schemaVersion": "3.0.0",
    "tenant": {
        "class": "Tenant",
        "application": {
            "class": "Application",
            "service": {
                "class": "Service_L4",
                "virtualPort": 1701,
                "virtualAddresses": [
                    "<IP_ADDRESS>/0"
                ],
                "persistenceMethods": []
            }
        }
    }
}

Actual Behavior
Tried various ways to set persistence none in runtime init as3 yaml, but error.
              forwarder_tcp_ipv4_1701-to-25:
                class: Service_L4
                remark: Forwarding port 1701-to-25 IPv4
                allowVlans:
                  - internal
                virtualAddresses:
                  - '<IP_ADDRESS>/0'
                virtualPort: 1701
                layer4: tcp
                profileL4: basic
                snat: auto
                persistenceMethods: []
                translateServerAddress: false
                translateServerPort: true

tried with empty brackets

persistenceMethods: []
zzznextthingzzz:

error = error: {"code":422,"errors":["/Tenant1/App1/forwarder_tcp_ipv4_1701-to-25/persistenceMethods: should be array"],"declarationFullId":"","message":"declaration is invalid"}

tried with 'none'

persistenceMethods:
 - none
zzznextthingzzz:

error = error: {"code":422,"errors":["/Tenant1/App1/forwarder_tcp_ipv4_1701-to-25/persistenceMethods/0: should be equal to one of the allowed values ["cookie","destination-address","msrdp","source-address","tls-session-id"]"],"declarationFullId":"","message":"declaration is invalid"}

tried leaving blank

persistenceMethods:
zzznextthingzzz:

error = error: {"code":422,"errors":["/Tenant1/App1/forwarder_tcp_ipv4_1701-to-25/persistenceMethods: should be array"],"declarationFullId":"","message":"declaration is invalid"}

tried with blank array

persistenceMethods:
 -
zzznextthingzzz:

error = error: {"code":422,"errors":["/Tenant1/App1/forwarder_tcp_ipv4_1701-to-25/persistenceMethods: should be array"],"declarationFullId":"","message":"declaration is invalid"}
Workaround
I use Service_Generic which has a default persistence of none. That way, I do not have to set 'none' which apparently cannot be set properly right now. Hence, new ticket :)

Thanks for the detailed writeup. Something is definitely not working right here. We are now tracking this bug internally with ID ESECLDTPLT-2890.

@JeffGiroux Hi Jeff, I just attempted to repro this, but was unable to.  I was using runtime init v1.4.0 and AS3 3.33.0.
Worked both with a HTTP and L4 service.  Would you mind retrying with those versions?
thanks


chore(dev-deps): update dependency @semantic-release/github to v10.0.5

This PR contains the following updates:

Package
Type
Update
Change

@semantic-release/github
devDependencies
patch
10.0.4 -> 10.0.5

Release Notes

semantic-release/github (@semantic-release/github)
v10.0.5
Compare Source
Bug Fixes

normalize GitHub API URL (#839) (1e66974)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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


Also tweaks logging slightly.

Pull Request Test Coverage Report for Build 420

81 of 128 (63.28%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 52.79%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

webhook.py
1
8
12.5%

preprocessors/preprocessors.py
68
108
62.96%

Files with Coverage Reduction
New Missed Lines
%

webhook.py
1
9.33%

Totals

Change from base Build 417:
0.03%

Covered Lines:
3150

Relevant Lines:
5967

💛  - Coveralls


Plugin failed when no {port_specs}

When declaring the pc plugin in rebar.config without any {port_specs}, the compile task fails
Example:
rebar.config:
{plugins, [pc]}.
Run rebar3 compile

I failed to reproduce this with 1.3.0 and 1.4.0. What else is needed to provoke a crash? Tested with no sources, just Erlang sources, and Erlang plus C sources.

@jeanparpaillon, ping?

Closing, @jeanparpaillon feel free to reopen if the issue persists.


Why is Dev Home automatically installed after a Windows 11 update

I just noticed that this tool has been installed on my computer without any prior consent. In my case the tool might be helpful. But why does it have to be installed on each and every Windows 11 installation? Most users are not developers and have no use for such a tool.

You did consent

@BenS13 Maybe indirectly through some agreement text when I installed Windows 11. But I never explicitely agreed to install dev home. I mean I wasn't even aware it existed until I noticed it in my start menu.

/dup #1615

@cinnamon-msft Why are preview releases of this pushed to all windows 11 users?
Why not only push to users that have enabled developer mode?
Shouldn't the project become GA before pushing to every windows 11 user?

/dup #1615

@cinnamon-msft This other issue has already been closed. But since there is no satisfying response as to why this is pushed to each and every client, the issue still remains.

It's almost end of February 2024 and this issue appears to be ongoing.  Just had to remove this app from a vm I was testing win11 23h2 in.


Godot4 editor performance regressions when removing/moving 2D nodes

Tested versions
Reproducible in 4.3 stable, 3.5.3 stable works fine
System information
Godot v4.3.stable - Windows 10.0.19045 - Vulkan (Forward+) - integrated Intel(R) Iris(R) Xe Graphics (Intel Corporation; <IP_ADDRESS>87) - 12th Gen Intel(R) Core(TM) i7-1255U (12 Threads)
Issue description
I have a 430 nodes big scene with a lot of houses that have say 8 tilemaps. This scene works completely fine in Godot3, no lag. I converted the scene to Godot4 and everytime I remove or add a node, I get a 4s lag. If I duplicate a house node (that has many tilemaps) say 40 times and then remove a node, I get significantly prolonged lag. There are no tool scripts. Doing the same thing in the Godot3 editor doesn't result in any noticeable lag.
Why is this relevant?
As a Godot 2D engine developer, I want to be able to create bigger scenes for bigger games productively and not limit my creativity.
Now, I get that there's a limit to the amount of nodes that Godot should support for most devs. Also Godot as an engine doesn't seem to care about performance that much (yet). But throughout versions the performance shouldn't worsen like that imo.
Steps to reproduce

Add big enough nodes to the scene tree, with a lot of tilemaps
Duplicate them
Add a simple, new node (Node2D is enough)
See the freeze duration go up when moving or removing this node in the scene tree the more tilemap container nodes you add.

Minimal reproduction project (MRP)
laggyeditorgames.zip

It's not just you. There's been huge regressions since 4.2.2, my 3d scene freezes up for 7 seconds when adding/deleting any node and I get constant blue screen of death on 4.3. I don't know why no one else is talking about this. I'm sticking with 4.2.2/ testing other engines.

Tested on win10 i7-12700 GTX1050ti, adding a Node2D is almost instant, but duplicating subscene with lots of tilemaps is much slower compared to 3.5.
I've noticed this scene suffers fps drop when you moving the camera, while the GPU usage remains low. Switch to Compitability renderer solve this problem. Since you are using integrated GPU I'd recommand you check Compitability renderer to see if things go better.

Adding a Node2D is almost instant for me too, but not removing or moving a Node2D.

Reproduced in my environment, so the problem is about remove node.
A quick profile below:

When you try to remove a node, the editor needs to check one by one whether other nodes needs to be renamed (seems related to undo/redo), during this it access the property_list of the node.
TileSetAtlasSource::get_property_list spent a lot of time formatting strings, which cause the lag.

Related code: https://github.com/godotengine/godot/blob/af77100e394dcaca609b15bef815ed17475e51ed/scene/resources/2d/tile_set.cpp#L4933-L4955


Update links in README.md

This PR updates the links on the README to point to rust-lang/ctcft

lgtm!


Question about OpenBMC -> Board integration

What documentation I should use to learn how to integrate OpenBMC to host motherboard ?
Most of the documents seems to be about OpenBMC embedded in new MC hardware. Am I wrong?
Sincerely,
Jorge Ventura
PS: I am posting here because I don't know where to do that and I apologize if it's the wrong place.

I don't understand what you are asking.  Maybe you're looking for this?  https://github.com/openbmc/docs/blob/master/development/add-new-system.md
That document probably isn't 100% up to date but it is a decent starting point.  After that the suggestion tends to be to look at existing machines in https://github.com/openbmc/openbmc and try to match up your requirements with one of those reference implementations.  Feel free to join the Discord community to ask more specific questions if you start on an effort to port OpenBMC to your machine.


Make webms play inside app

It would be nice to play webms inside Dashchan without having to use an external media player app.

Built-in system features don't allow to make a good WebM player. It's impossible to implement video streaming. Even without streaming some videos will play without sound or image. It is also very dependent on the device.
The only way to make a good player is using software decoding. I must use external libraries for this. It's really not easy and it will take a lot of time. The app will weight 5 times more in result.
There are a lot of players that can play WebM by themselves. The goal does not justify the means.

Yeah external webm players are way better than inline webm viewing.

Which external app would y'all recommend for use with the kind of webm files posted on imageboards? One that loops them by default would be nice.

MX Player, for example. You can enable loop in Menu - Play - Loop one.

Even using MXPlayer isn't the same thing that see on PC and others 4chan(loops seems weird) tbh.
But I respect your opinion dev, this app is top tier and the best characteristic is be lightweight, well optimized, and can be personalized by the user, with 800kb.

Clover managed to implement this.

Clover uses system MediaPlayer, not software decoding.

Webm player was added in last version. It's experimental feature now.
You need to download this extension with codecs to make player work.

Link is 404 to webm extension please update thx

README contains a correct link but wiki doesn't. Fixed.


Update set repr

Is there already a test for this?

Sort of covered by test_setcomps but it's disabled since it's failing some closure related stuff.

I guess it'll also be covered by test_set since apparently I have to fix it...


thread safety ?

Hi,
I'm interested in this library but have a few questions regarding thread safety. The readme says it is "thread-safe", but what happens when a thread writes a log message while the output callback is already in progress (called from a another thread) ?
I don't see any explicit locking / queuing in the built-in output callbacks...

stderr with WriteFile (win) : I admit I can't find much documentation on using WriteFile with stderr, but how can you tell if FILE_APPEND_DATA  or FILE_WRITE_DATA are set ?

stderr with write() (*nix); the code comments that the usage is atomic, but POSIX says "A write is atomic if the whole amount written in one operation is not interleaved with data from any other process."  -- so, OK for multiple processes, but multiple threads ?

OutputDebugString on win32 is probably async safe, I read somewhere that it uses a mutex internally? Not 100% sure about this though, and it's not quite what I'm looking for.

I know there has been some testing with multiple threads, so maybe I'm just missing something ? I'd like to hear more about this.
Thanks!

Hi, good question. Library doesn't have any explicit locking around output callback, because not all output callbacks need one. So, if output callback needs any synchronization it can provide its own.

stderr with WriteFile (win) : how can you tell if FILE_APPEND_DATA or FILE_WRITE_DATA are set ?

You can verify with GetFileInformationByHandleEx() or something like that, that returned handle has FILE_APPEND_DATA set. And if you think about that, it should, since you can't seek() stderr anyway. Yes, they don't document  that explicitly, but since it's the only sane thing to do I'm OK with relying on it in default implementation. Once there will be a windows version that doesn't do that, we can think about what can be done about that. But I would agree that unit test that checks for that would be nice.

stderr with write() (*nix)

Experience shows that it's atomic for most Unix flavors you can find around. When some *nix will be found where it's not atomic, it can be addressed specifically. I would prefer to have fast code for common case and additional locking for outliers.
See this post from Linus (search for "with a chainsaw laughing maniacally"): http://yarchive.net/comp/linux/wakekill.html
Also see this SO and links from it:
http://stackoverflow.com/questions/12942915/understanding-concurrent-file-writes-from-multiple-processes/12943431#12943431

OutputDebugString

It better be async safe, otherwise it's unusable. OutputDebugString() can be used from different subsystems (like 3rd-party libraries) that have no idea about each other. No way they can synchronize  properly.
So, to sum up. Probably you can find some formal loop holes that can indicate that writes possibly can be non-atomic, but reality is that they are. If there will be a need to use this library on system where writes are non-atomic we can fix library specifically for this system (maybe even without introducing a lock, since we will know what we are dealing with).

Hi,
thanks for the detailed answers; all valid points.
Are you aware if anyone made an output callback that queues + buffers writes to a logfile ? Basically a FIFO so the callback return almost immediately without waiting on file IO.
This wasn't really an "issue" anyway, just a question (wasn't sure how else to contact you), so I guess I can close this.
Cheers

Are you aware if anyone has made an output callback that queues + buffers writes to a logfile ?
Basically a FIFO so the callback return almost immediately without waiting on file IO.

No, there is none AFAIK. It's not a trivial thing to do though. The hardest question is how much to buffer and what to do when logs are going in faster when IO is capable of writing them out. In the end, you either will have a chance to block anyway or will drop (newer or older) log lines when buffer is full. And to be honest, write() already does exactly that. It doesn't wait for physical IO to be completed. Though because of sys call involved overhead could be suboptimal, yes.
In general, I think it would be nice to have a companion library with most popular output callback implementations. It's just nobody needed one so far...

Indeed, it adds a failure point in the case of buffer overrun. In my case I could deal with an occasional blocking call, but I think either behavior could be OK, as long as it's documented.
I'm considering porting the buffering code from "zlog" ( https://github.com/zma/zlog/blob/master/zlog.c ) to zf_log for my own use (with some modifications to use wrapper functions around pthread / winAPI mutexes). If you have ideas on how to integrate it nicely enough for this project, I'd be glad to eventually submit a PR.
I'm thinking your "companion library" could just be an extra set of .c files that contain a bunch of possible callbacks; pick one at compile-time and that's it. Simple for you (no runtime selection to debug), versatile for users.

I'm thinking your "companion library" could just be an extra set of .c files that contain a bunch of possible callbacks;

Yes, I was thinking about something along this lines. I even have a local branch with skeleton for that. The basic idea is to export custom callbacks the same way zf_log exports stderr output callback. But instead of static struct instance, maybe have a function that can create one dynamically (since you need to specify a file to open, etc.). I'll post what I have today evening PST.

So I was thinking about something like that basically:
/* Create log output */
zf_log_output output_open(args...);
/* Release associated resources */
void output_close(zf_log_output *output);

/* Usage example */
void foo() {
    zf_log_output out = output_open("/tmp/log");
    zf_log_set_output_p(&out);
}

who implements output_open() and _close()  , each individual "driver"?

Makes sense, moved discussion to https://github.com/wonder-mice/zf_log/issues/9.


Update 0 A.D. version

oops

Thank you for the contribution. It needed some fixes, so they were made in f55136b3117fb300b587fdc3c07162f9e74b47c4. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Videos don't play till I click in scene

If a scene includes a VideoPlayer, the video doesn't start playing until the player clicks at least once, anywhere in the scene.
This can include the click to lock the cursor.
This behavior can be confusing and feel broken.
Here's a scene done with the Web Editor where you can see this
The_Theatre.zip

No longer relevant in 2.0 client


issues with assigning a binding to open calcurse

When I try to bind MODKEY+c to open calcurse, calcurse flashes and closes.
Steps to reproduce:

in ~/.local/src/dwm/config.h add the line:
{ MODKEY,          XK_c,  spawn,  SHCMD(TERMINAL " -e calcurse") },
right below the hotkey assignment for "htop", namely:
{ MODKEY|ShiftMask,  XK_r,  spawn,  SHCMD(TERMINAL " -e htop") },
recompile with sudo make clean install

You'll see that calcurse briefly opens and closes instantly. Any ideas? If you are not able to reproduce this issue, please let me know. :)

Issue is resolved. It just started working. I likely already had an instance of calcurse running causing this.


Request to Install dependencies automatically by declaring them

@frostney : Hi, congratulations on the module.
It would be interesting to give the possibility that during the creation of the module, the possibility to automatically install dependencies.
For example:
 --dependencies <dependencies> (the name of each dependency separated by a comma: 'com.name0, com.name1, com.name2, ..')

@frostney , @maicki , @thorbenprimke, @rozele : are there any news?

@Angelk90 Hey! What would be the use case for this? Are these dependencies for development or are these dependencies of the library. If these would be dependencies of the library it should be added to the peerDependencies of the libraries package.json.


Model returns different results for scipy sparse matrix and scipy.sparse.toarray()  numpy ndarray.

I have a pipeLine that includes TfidfVectorizer AND XGBoost model.
class Text2TfIdfTransformer(BaseEstimator):

    def __init__(self):
        self._model = TfidfVectorizer()
        pass

    def fit(self, df_x, df_y=None):
        df_x = df_x.apply(lambda x : clean_text(x))
        self._model.fit(df_x)
        return self

    def transform(self, df_x):
        return self._model.transform(df_x)

xgb_tf_idf = Pipeline(steps=[('tfidf',Text2TfIdfTransformer()),
                         ('xgboost', xgb.XGBClassifier(objective='multi:softmax', n_jobs=8))])

now, when I obtain intermediate tfidf result and pass  it to the model:
vec = tfidf.transform(['include stdio h int main'])
arr = vec.toarray()

vec == arr                 # True

xgb_tf_idf .predict(vec)    !=     xgb_tf_idf .predict(arr)

The predict proba method also differs and results are off by a lot. Am I doing something wrong or is this a bug ?
Thanks

I remember solving a similar issue before.  toarray() returns a dense array with 0 filling missing values.  But for XGBoost 0 is valid value.  So when used with return np array, you need to specify missing= 0 for XGBoost.

Thanks @trivialfis setting missing  = 0 solved the problem !
Closing the issue.


Add link to qbittorrent-gluetun-port-update

Thank you for maintaining this project! I really wanted to use qBittorrent with Gluetun, but I couldn't find any way to automatically update the forwarded port, so I made one myself. If you're using docker-compose, it's really easy to add this additional container to the deployment and keep the port up to date.
https://hub.docker.com/r/technosam/qbittorrent-gluetun-port-update
https://codeberg.org/TechnoSam/qbittorrent-gluetun-port-update
Hopefully this can be helpful to others who find themselves in my situation. Please let me know if there's any changes you'd like to see!

Hey @TechnoSam - Can I ask does this container find the open port for any provider and update it into qbit? As that would be great!

@DrFrankensteinUK What do you mean by provider? VPN provider?
It will work with anything Gluetun supports. It uses the control HTTP API. As long as Gluetun is making it available there, it will work. I don't think that's every VPN, but it is most of them. I'll have to check the documentation, but you can check too, it may take me a minute.

OK I think I get it, it's asking Gluetun what the forwarded port is and then applying that to qbit.
So if for example I set the firewall port forward variable for Gluetun of 44444 your container would map that into qbit saving me a step in setup.
Or is there some kind of additional activity taking place to find an open port on the vpn connection which can be used for qbit.
I suppose this matters more for vpn providers who randomise which port you are given on connection vs something like AIRVPN who give you a dedicated port.

I'm not sure about the firewall, I would have to do some research to figure out if that's relevant.
But yes, forwarding is the primary use cases. Services like ProtonVPN and PIA don't give you a fixed port like AirVPN or Windscribe. If you don't know what the forwarded port will be, Gluetun exposes that in the control server, and my container reads that and sends it to qBittorrent.

Not to detract from your work in any way, but I recently came across a bash script that does something similar: https://gist.github.com/fizzxed/958d8cdfa4dad8a97651afce4d74efce

Thank you for maintaining this project! I really wanted to use qBittorrent with Gluetun, but I couldn't find any way to automatically update the forwarded port, so I made one myself. If you're using docker-compose, it's really easy to add this additional container to the deployment and keep the port up to date.
https://hub.docker.com/r/technosam/qbittorrent-gluetun-port-update
https://codeberg.org/TechnoSam/qbittorrent-gluetun-port-update
Hopefully this can be helpful to others who find themselves in my situation. Please let me know if there's any changes you'd like to see!

Seems like everyone is getting in on varied implementations of this.  I just spun up my own docker image to do this as well forking someone elses repo as I saw it in a forum post to get the firewall issue fixed when using Proton.  Then I realized gluetun had the control server and the API, so I wrote a very similar script.  Guess it was a good way to learn how to build docker images, and get some learning about Github.
Guess I should have checked the pull requests first  :rofl:

It's a bit off-topic, but I've done somehing similar, but I took a different approach. Instead of a separate container, I've made a docker mod for Linuxserver's qbittorrent image. Here's the link if someone is interrested : https://github.com/t-anc/GSP-Qbittorent-Gluetun-sync-port-mod.
And it seems that qdm12 is thinking about a built-in solution for that, so I guess it will make other external ones obseletes (https://github.com/qdm12/gluetun-wiki/issues/75#issuecomment-2256301757).


Fix for sessions controller

fix #4736

I believe this was already fixed in master through the addition of () around arguments. The use of do/end isn't necessary AFAICT.

Already fixed by
https://github.com/plataformatec/devise/commit/1009096172f2cbc86bcd54d053c89a09be67fb9f

@tpitale @s01ipsist yep. This was just me jumping the gun. I didn't know this was already fixed. I will close this request.


[Question] Is there a way to constrain or limit the amount of rotation for a joint?

I would like to know if we can limit the amount a joint can rotate per axis, similar to a Configurable Joint Angular Limit options.

An option on the JiggleRigbuilder that affects only the root or on the Jiggle Settings that affect all the joints in the chain (except for the ignored transforms) would work really well.

There's no such option unfortunately, however you can get very similar results by doing things like-- if you want a 90 degree limit you could set Blend to 0.5.


Improved issue form by added JS validation issue#1143 resolved

🛠 Fixes Issue
Add JS validation in Issue_book form
📷 Screenshots

@SauravMukherjee44  check it !!


Elasticsearch Connector some properties need to be documented

hi, when i using es connector in my presto, I found property named 'elasticsearch.ignore-publish-address' in source code.
It work for me, but I can't find it in document.
I think it need to be documented, isn't it ?

Thank you for that!!! I was going crazy about this connector and only now I see that it uses the _nodes/http endpoint to get the publish_address of all nodes.
There is zero documentation about this functionality and the elasticsearch.ignore-publish-address property.
In addition, PrestoSQL just released the user & password auth in v337, again without updating the docs about the new functionality and properties:
elasticsearch.security=PASSWORD
elasticsearch.auth.user=elastic
elasticsearch.auth.password=changeme

It's expected from an open-source this size to update docs along with code at the same release version.

cc @mosabua

I am going to assign this to myself for now. If anybody wants to get to it before me link a PR to here... otherwise I hope to get to it next week

Doc PR merged. Will be published with next release


Configuration: areas and filesystems

We need a way to configure the areas and filesystems DOSS uses. Each area is made up of one or more filesystems. Each filesystem stores a copy of every blob in the area. In normal production usage there will be 2 areas: area.staging and area.master; and 3 filesystems: fs.staging, fs.display and fs.preservation.
See #29 for more background about the staging area. For the master area DOSS needs to make at least two copies of each blob, one on a disk array for online access and one on a tape-backed preservation filesystem (currently running SAM-QFS). The preservation filesystem actually itself makes multiple tape copies, some of the tapes get sent to an offsite location for disaster recovery, others sit in a tape robot onsite and can be used to restore the display copy in the event something happens to it. We had to make use of this recovery mechanism in 2011 after an operating system bug corrupted the display filesystem, it's not just paranoia.
[defaults.put]
target = area.staging

[area.staging]
fs = staging

[area.master]
fs = display, preservation
container = tar

[fs.staging]
path = /staging

[fs.display]
# online spinning disk backed filesystem
path = /display

[fs.preservation]
# nearline tape backed filesystem
path = /sam/master

I really want to use something like Python's configparser format for this. ini4j's ConfigParser class appears to be a reasonable Java implementation of it.
Commons Configuration seems to be the other popular choice, it does a lot more but looks a bit too overengineered for our needs.

Implemented using ini4j


Add support for retry option?

We're integrating sidekiq-cron as a replacement for cron jobs in part for the automatic retry support. However we don't want jobs to retry indefinitely. For example, in our case a job that runs every 15 minutes doesn't need to retry at all, and a job that runs nightly shouldn't retry after the end of the day.
We can use sidekiq_options retry: 5 or the like in the Job class (see doc), but the retry value is dependent on the scheduling, so we'd rather include it in schedule.yml, like:
parse_emails_job:
  cron: 'every 15 minutes'
  class: 'ParseEmails'
  retry: false # or 5, or retry_for: 15.minutes

The case for supporting this configuration option seems similar to the case for supporting queue.
Would you be open to a PR?

So this would overwrite the existing and/or default retry settings set for a job just like happens currently with the queue setting?

So this would overwrite the existing and/or default retry settings set for a job just like happens currently with the queue setting?

That's right.

Looks like an interesting feature! Please feel free to send a patch 🤝


Show current library in library list

Changes
This is a quick PR to provide a solution for the discussion in #751 and show the current library at the top of the library list.
I agree with Jon that it would be beneficial to see the current library - it is after all a part of the complete library list and is used during compile and other commands.
The current library has been given a special context value of currentLibrary to mark the object and omit the actions for moving up and down. Also the description is (current library).
Checklist

[x] have tested my change
[ ] updated relevant documentation
[ ] Remove any/all console.logs I added
[x] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[ ] for feature PRs: PR only includes one feature enhancement.

I thought of it - having a command for the list item. But maybe it is too much having the command available next to the current library as well as next to the USER LIBRARY LIST header?

@worksofliam Is this what you had in mind?

@chrjorgensen that's good! Sorry if I'm being annoying, but perhaps instead of a right click option, we could make it an in-line button (like move up / move up) instead of a right click.

@worksofliam Okay, I have now moved the button for change current library to the library list item for current library - no need for two identical buttons that close.

@chrjorgensen i think have both a right click and inline button is ok.
Make that commit and then I will merge!

@worksofliam Done

Looking good! Will merge later on today.


Graceful handling of Agent connection failures

Environment

OS / version: Fedora 29
Processor architecture: x86_64
TPM Manufacturer: N/A
Keylime version: master

Description
Should the agent not be able to connect to its assigned register, a stack trace is displayed and nothing is logged to cloudagent.log - this would make it challenging to debug at a later stage for adminstrators of keylime agents
Stack trace
2019-04-17 12:02:25.375 - keylime.cloudagent - INFO - Agent UUID: D432FBB3-D2F1-4A97-9EF7-75BD81C00000
Traceback (most recent call last):
  File "/usr/bin/keylime_agent", line 11, in <module>
    load_entry_point('keylime==1.2', 'console_scripts', 'keylime_agent')()
  File "/usr/lib/python2.7/site-packages/keylime-1.2-py2.7.egg/keylime/cloud_agent.py", line 519, in main
    keyblob = registrar_client.doRegisterAgent(registrar_ip,registrar_port,agent_uuid,tpm_version,ek,ekcert,aik,ek_tpm,aik_name)
  File "/usr/lib/python2.7/site-packages/keylime-1.2-py2.7.egg/keylime/registrar_client.py", line 139, in doRegisterAgent
    context=None)
  File "/usr/lib/python2.7/site-packages/keylime-1.2-py2.7.egg/keylime/tornado_requests.py", line 38, in request
    response = http_client.fetch(request)
  File "/usr/lib64/python2.7/site-packages/tornado/httpclient.py", line 118, in fetch
    self._async_client.fetch, request, **kwargs))
  File "/usr/lib64/python2.7/site-packages/tornado/ioloop.py", line 581, in run_sync
    return future_cell[0].result()
  File "/usr/lib64/python2.7/site-packages/tornado/concurrent.py", line 260, in result
    raise_exc_info(self._exc_info)
  File "<string>", line 3, in raise_exc_info
socket.error: [Errno 111] Connection refused

Instead we could perform a connection test, implement a retry and log the result to cloudagent.log for later debug information.

keep alive

Keep


feat(spell-check): support local dictionary

Description

It is useful if we can use local dictionaries that are in the target repository.
Users can define repository-specific words.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Test results:

Without the local dictionary: There was an error.
https://github.com/autowarefoundation/autoware-github-actions/runs/8088038593?check_suite_focus=true

With the local dictionary: There was no error.
https://github.com/autowarefoundation/autoware-github-actions/runs/8089950258?check_suite_focus=true


Security issue

I'm not saying u are but if your using your own url to check the tokens it's unnecessary and can mean your stealing the tokens people are checking

I'm not saying u are but if your using your own url to check the tokens it's unnecessary and can mean your stealing the tokens people are checking

you should check that please star fr
https://github.com/TheKindDeveloper/Dickscord-Python-Package


Strife Support

Not tagging this to any particular milestone. It's a stretch goal; probably post-v2.0.0.

Tagging Eriguns1 on this too since that probably makes sense in Strife-land. Still hella low-priority though.


Concept of looping through emails

Is there such thing as looping through current emails by label?
Lets say I have a label "Info Wanted".  I search for this label, and get back  10 emails.  So I'm looking at 10 emails all labeled "Info Wanted".  Now I'd like to respond to each of these emails with my typical "More Information" email. I I get the composing piece of this, but how do I automate this task as described?
Karl..

Old issue is old. Closing.


Switches focus indication left in place when moving focus quickly

Problem description
When quickly moving focus using tab, the grey rectangle around a checkbox, radio button or toggle can be left displayed when the checkbox is not focused.
I have no knowledge of the internals, but the focus highlight seems to be animated, so perhaps this is due to losing focus before the animation/transition is complete?
Steps to reproduce
Go to checkbox docs page and hold down tab to move quickly through focus. Hold tab for a few seconds, then go back to the checkboxes, and some (most?) of them will look like they are focused:

Tabbing back through the glitched checkboxes will fix the issue.
The same happens for radio buttons and toggles.
Versions

Material-UI: as per materialui.com
React: as per materialui.com
Browser: Chrome Version 53.0.2785.143 (64-bit), OS X 10.11.6

There's actually more to this - if you press space to toggle the actually focused checkbox while other checkboxes appear focused, they all toggle...

We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.
Still, we will accept PR fixes until v1-beta takes over the master branch.


create requirements.txt

Create a requirements.txt file to list project dependecies.

It should be effectively simpler because of some errors.


feat: Create ribbon component in React

Create a ribbon component to render the list of items like below -

I want to work on this.. may I?

this would be great for my 1st hacktoberfest event and I would like to work on this. is it possible?

@SINJINI-G is already working on this issue... you can work on the upcoming new issues! Thanks for your interest!


fix: pass updated keyword args to Publisher/Subscriber client in goog… #161

…le/pubsub

Codecov Report

Merging #164 (7717d76) into master (df0c28a) will decrease coverage by 69.89%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master     #164       +/-   ##
===========================================
- Coverage   86.18%   16.28%   -69.90%
===========================================
  Files          26       26
  Lines         702      706        +4
  Branches       65       65
===========================================
- Hits          605      115      -490
- Misses         76      584      +508
+ Partials       21        7       -14

Impacted Files
Coverage Δ

testcontainers/google/pubsub.py
0.00% <0.00%> (-95.46%)
:arrow_down:

testcontainers/kafka.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/mssql.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/neo4j.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/mongodb.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/keycloak.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/postgres.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/clickhouse.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/core/config.py
0.00% <0.00%> (-100.00%)
:arrow_down:

testcontainers/google/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 13 more

Continue to review full report at Codecov.

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

Seems like another fix could be a #446 release


update zh-Hans

Although the direct translation of "Switch primary and secondary buttons" in English does mean "交换主要和次要按键", this will make most people confused, what are the primary and secondary buttons, if translated as "交换鼠标左右按键" will make it easier to understand

Thanks!


feat: implement JsonRpc transport for batch requests

Introduces JsonRpc batching requests described in #593

Thank you @tcoratger for the review. All review comments addresses in the latest commit.

@thetheveloper can you rebase to trigger the ci again? I think this should be good now

@thetheveloper can you rebase to trigger the ci again? I think this should be good now

@tcoratger rebase done. I tried to include the test fixes (hence last commit), but from what if the tests are not run one after another try everything is fine. I suppose RPC node needs to process transactions from the previous iteration of testing, so the next one doesn't fail

Should be fine now. Latest base is in place @tcoratger

@tcoratger @xJonathanLEI anything else needed? thanks!

@tcoratger @xJonathanLEI anything else needed? thanks!

For me this is ok to merge but as the PR is quite big and important I would appreciate another review from @xJonathanLEI if possible?

Squashed and rebased.

@tcoratger maybe you can review as well? ty ty

While this implementation probably works, I'm really not a fan of adding serde_json::Value to a supposedly neutral trait of Provider. I mean it's probably acceptable to add it to JsonRpcTransport, because as its name suggests, it deals with JSON.
I know this is a bit not very pragmatic - after all, Provider is modelled after the JSON-RPC standard. However, with the current design any Provider user would now have to deal with deserialization... which is far from ideal.
Anyway, I just prefer a design where we don't have to pollute Provider with JSON stuff.
The diffs are also a bit all over the place:

Why do we have a get_block_with_tx_hashes_batch method? Wouldn't that be covered by send_requests?
Same goes for JsonRpcRequestParams - this type only covers 2 methods.

Closing this as I'll work on an alternative version. Thanks for working on this.


bug: creates .supermaven/config.json on every startup

Recently, this plugin has begun creating ./.supermaven/config.json if it doesn't exist (relative to CWD).
It also forces the user to call :SupermavenUseFree each time this file is created, rather than creating/using ~/.supermaven/config.json.
Note that downloaded binaries are still stored in ~/.supermaven/config.json.
I suspect this is an upstream problem (with sm-agent), however there's no available bug tracker for the upstream.

I can confirm this bug. It started happening after the latest plugin update.

same here, I almost pushed the config.json to my public repo.
I get that it might be useful to have a config for each project, but maybe let us set a default path to the plugin config that it can use if there is none in the project directory. and dont force it

This just made me disable the plugin. It is way too disruptive and caused quite some work

Maybe this can help others, until this is fixed, you can define the exact version of the plugin. If you are also using Lazy, then add this specific version from the commit before SM_EDITOR = "neovim" was set:
  {
    'supermaven-inc/supermaven-nvim',
    commit = 'df3ecf7',
    config = function()
      require('supermaven-nvim').setup {

Same bug almost committed it to repo and this prompts for me in every folder i open.

also your aggressive popup, reminding me of this issue whenever i open nvim in a new folder 👎
its not only pissing me off, there's also no need to constantly force open that popup.
you might want to consider changing that behaviour

Maybe this can help others, until this is fixed, you can define the exact version of the plugin. If you are also using Lazy, then add this specific version from the commit before SM_EDITOR = "neovim" was set:
  {
    'supermaven-inc/supermaven-nvim',
    commit = 'df3ecf7',
    config = function()
      require('supermaven-nvim').setup {

@jsnanigans I tried that, but it doesn't seem to work for me. That's why I suspect this was changed in sm-agent, rather than the nvim plugin directly.

@aarondill works for me, make sure to update the plugin, maybe delete the install folder and reinstall.
the commit which fix it for most of us (https://github.com/supermaven-inc/supermaven-nvim/commit/c5bb0490bdfc36a7841e5b82c8f6b92b5b8beef3) just pass a new environment variable to sm-agent, so indeed it's probably that

@aarondill Hey mate, could you please tell me what you did to fix the issue? I updated the plugin but it still wants to log in or tell me to use the free version.

just running Lazy update works too
Pankaj Garkoti, CTO @ Mavex.ai http://Mavex.ai
On Wed, 17 Jul 2024 at 3:47 PM, Aaron Dill @.***> wrote:

I added the commit from jsnanigans' comment, then opened lazy and deleted
the plugin (press x), then exited nvim and ran rm -r ~/.supermaven/,
reopened nvim and ran :SupermavenUseFree.
—
Reply to this email directly, view it on GitHub
https://github.com/supermaven-inc/supermaven-nvim/issues/74#issuecomment-2232951952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKMPOJ7JJRC3ONGDC7B3T3ZMZACBAVCNFSM6AAAAABK5ZQPW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSHE2TCOJVGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I still have this issue

the faulty commit has been reverted.

Closing thanks to d71257f


Smart Contract - Pausable feature

Description
Pausable feature can be useful to freeze a contract under attack for instance.
Acceptance Criteria

[ ] Relevant contracts can be pausablle

Done in migration


Revert "Add owners file for tools/run_tests/"

Reverts grpc/grpc#14355

Feel free to merge as soon as the tests pass.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


Revert "Revert "Couple typos, testing instruction extract""

Reverts Bloc/workshop-curriculum#16
Reverting the revert to trigger Blync.

Blync Deploy!
The following files were deployed

courses/ruby-primer/04-level-arrays/02-array-index/body.md
courses/ruby-primer/06-level-loops/02-each-with-index/body.md

The following files were not deployed

resources/ruby/reading-specs.md - Exception: GET https://api.github.com/repos/bloc/workshop-curriculum/contents/resources/ruby/reading-specs.md?ref=master: 404 - Not Found // See: https://developer.github.com/v3


Confuse travis less Fix Clickup integration - See https://app.clickup.com/t/dctafn  CU-dctafn

Confuse travis less Fix Clickup integration - See https://app.clickup.com/t/dctafn  CU-dctafn

Dont Confuse CU-dctafn

Task linked: CU-dctafn Fix logicmoo_utils CI Tests


Sharded MongoDB connection string is not supported

MongoDB has a specific connection string format for sharded setups which lists the hosts explicitly: mongodb://user:password@my-shard-00.atlas.mongodb.net:27017,my-shard-01.atlas.mongodb.net:27017,my-shard-02.atlas.mongodb.net:27017/database?ssl=true&replicaSet=atlas-xxxxx-shard-0&authSource=admin&retryWrites=true
Although this is not super common, it is part of the MongoDB specification:
mongodb://[username:password@]host1[:port1][,...hostN[:portN]][/[defaultauthdb][?options]]
(via Standard Connection String Format)
(The listing of the individual hosts can be avoided by using the "DNS Seed List Connection Format", which uses mongodb+srv:// instead and then executes some logic to get the individual hosts)
Currently you can not use Prisma with this format.
Preproduction
generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["mongoDb"]
}

datasource db {
  provider = "mongodb"
  url      = "mongodb://user:password@my-shard-00.atlas.mongodb.net:27017,my-shard-01.atlas.mongodb.net:27017,my-shard-02.atlas.mongodb.net:27017/database?ssl=true&replicaSet=atlas-xxxxx-shard-0&authSource=admin&retryWrites=true"
}

model User {
  id    String  @id @default(auto()) @map("_id") @db.ObjectId
  email String  @unique
}

This from DEBUG="*" npx prisma db pull CLI
❯ DEBUG="*" npx prisma db pull
[...]
Introspecting based on datasource defined in prisma/schema.prisma …
  prisma:introspectionEngine:rpc starting introspection engine with binary: /Users/j42/Dev/repro/11684/node_modules/@prisma/engines/introspection-engine-darwin +0ms
  prisma:introspectionEngine:rpc SENDING RPC CALL {"id":1,"jsonrpc":"2.0","method":"introspect","params":[{"schema":"generator client {\n  provider        = \"prisma-client-js\"\n  previewFeatures = [\"mongoDb\"]\n}\n\ndatasource db {\n  provider = \"mongodb\"\n  url      = \"mongodb://user:password@my-shard-00.atlas.mongodb.net:27017,my-shard-01.atlas.mongodb.net:27017,my-shard-02.atlas.mongodb.net:27017/database?ssl=true&replicaSet=atlas-xxxxx-shard-0&authSource=admin&retryWrites=true\"\n}\n"}]} +7ms
  prisma:introspectionEngine:rpc {
  prisma:introspectionEngine:rpc   jsonrpc: '2.0',
  prisma:introspectionEngine:rpc   error: {
  prisma:introspectionEngine:rpc     code: 4466,
  prisma:introspectionEngine:rpc     message: 'An error happened. Check the data field for details.',
  prisma:introspectionEngine:rpc     data: {
  prisma:introspectionEngine:rpc       is_panic: false,
  prisma:introspectionEngine:rpc       message: 'invalid port number in database URL',
  prisma:introspectionEngine:rpc       backtrace: null
  prisma:introspectionEngine:rpc     }
  prisma:introspectionEngine:rpc   },
  prisma:introspectionEngine:rpc   id: 1
  prisma:introspectionEngine:rpc } +6ms

Error: Error: invalid port number in database URL

    at Object[...]

and for the client
[...]
  prisma:client:libraryEngine sending request, this.libraryStarted: false +29ms
  prisma:client:libraryEngine Searching for Query Engine Library in /Users/j42/Dev/repro/11684/node_modules/.prisma/client +1ms
  prisma:client:libraryEngine loadEngine using /Users/j42/Dev/repro/11684/node_modules/.prisma/client/libquery_engine-darwin.dylib.node +0ms
  prisma:client:libraryEngine library starting +12ms
  prisma:client:fetcher Error: The provided database string is invalid. Unable to parse URL. in database URL. Please refer to the documentation in https://www.prisma.io/docs/reference/database-reference/connection-urls for constructing a correct connection string. In some cases, certain characters must be escaped. Please check the string for any illegal characters.
  prisma:client:fetcher     at /Users/j42/Dev/repro/11684/node_modu
[...]

The current error message is probably caused by us using an URL library to check the connection string for validity: https://github.com/prisma/prisma-engines/blob/b70a888e76ca5f3bd704f64afd8d03745a5c9205/query-engine/connectors/mongodb-query-connector/src/interface/mod.rs#L32-L37


OpenRPC JSON schema monaco editor hover tooltips

Is your feature request related to a problem? Please describe.
Show JSON schema tooltips on hover in result payload monaco editor. Good example of this is the current implementation of showing results in inspector.
see Inspector->helpers-> openrpcDocumenToJsonSchema(Result?)

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

GitHub release
1.1.11
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


When resetting/restoring etcd, only listen on loopback

Proposed Changes

Only listen on loopback when resetting

Types of Changes
bugfix
Verification
See linked issue
Linked Issues

TBD

User-Facing Change
The embedded etcd instance no longer accepts connections from other nodes while resetting or restoring.

Further Comments

needs additional work to address either of the resolutions suggested in rancherlabs/rancher-security#1049


python312Packages.google-cloud-bigquery: 3.24.0 -> 3.25.0

Changelog: https://github.com/googleapis/python-bigquery/blob/v3.25.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 322892 run on x86_64-linux 1

8 packages built:

python311Packages.dbt-bigquery
python311Packages.dbt-bigquery.dist
python311Packages.google-cloud-bigquery
python311Packages.google-cloud-bigquery.dist
python312Packages.dbt-bigquery
python312Packages.dbt-bigquery.dist
python312Packages.google-cloud-bigquery
python312Packages.google-cloud-bigquery.dist


:white_check_mark: Write HTML

Write HTML for partials and home.handlebars

you should use  tags instead of forms


Piece images not displayed

Hi there, congrats on the new plugin! I just tried installing, but the pieces are coming through as missing images:

My setup:

Obsidian 0.13.2 on macOS
obsidian-chess 1.1.3 installed via community plugins

Hmm, weird.
Can you show me your .obsidian/plugins/obsidian-chess folder?

I never tested on Mac, only Windows, and Linux... the issue might be there.

Hmm, just a main.js and manifest.json file. Maybe the assets didn't come through with the installer?
$ ls -al
-rw-r--r--@  1 dbyler  staff  89541 Nov 18 06:30 main.js
-rw-r--r--@  1 dbyler  staff    269 Nov 18 06:30 manifest.json

Workaround:

Manually download the plugin from your releases page
Copy the Assets folder into the live plugin folder
Restart Obsidian

Yeah, there seems to be a problem with obsidian copying the assets... I will add that workaround to the README file for now.
I haven't had much free time to continue development in the plugin, you can use other chess plugins like chesser and
chessboard.

Just a heads up, Obsidian doesn't download any additional assets other than the main.js file. So to actually fix the loading, you'll need to rollup the SVG assets into the main.js file.


HDDS-5831. Remove empty TaskQueue in ContainerStateMachine.

What changes were proposed in this pull request?
In ContainerStateMachine, remove the container TaskQueue if the queue is empty.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5831
How was this patch tested?
A new test is added in https://issues.apache.org/jira/browse/RATIS-1414 .

Thanks @szetszwo for the contribution.


Local Links don't work correctly

I'm trying to create a linked table of contents but when I set it up to link locally to the corresponding header tag, it links to the web page.
For example:

Local Link

 some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text some text

 Local Link

When the PDF is created in your JSFiddle it linked to: https://fiddle.jshell.net/_display/?editor_console=true%23locallink
fiddle.pdf

I'm also trying to make a table of contents for a html2pdf-generated PDF doc, and would love it if the ToC items were clickable for navigating (the document is fairly long).
I did a little poking around in jsPDF to see if I could find anything about how to do this, but haven't come across anything yet. Hoping this functionality will be possible at some point!

I'm also trying to make a table of contents for a html2pdf-generated PDF doc, and would love it if the ToC items were clickable for navigating (the document is fairly long).
I did a little poking around in jsPDF to see if I could find anything about how to do this, but haven't come across anything yet. Hoping this functionality will be possible at some point!

Me too, whenever I add a link it this way it opens up in the browser or in a new tab, not in the current PDF file.

me three

I have the same question, has anyone found a solution?

Same issue


unexpected end of JSON input

I'm trying to use your library to write a docker-machine proxmox driver and wrote a very simple test to see if the basics work:
package main

import (
	"fmt"
	"github.com/joernott/go-proxmox"
	"log"
)

func main() {
	proxmox, err := proxmox.NewProxMox("<IP_ADDRESS>", "root", "xxx")
	if err != nil {
		log.Fatal(err)
	}

	nodes, err := proxmox.Nodes()
	if err != nil {
		log.Print("Error listing nodes")
		log.Fatal(err)
	}
	for node := range nodes {
		fmt.Println(node)
	}
}

This doesn't work against Proxmox VE 4.1:
$ ./nodes
2016/07/04 13:42:24 Error listing nodes
2016/07/04 13:42:24 unexpected end of JSON input```

After much debugging turned out the Hostname parameter needs to match the name of the node you are accessing! Who knew :) I kept getting 401 No ticket. Good thing I have proper DNS. Sorry for the noise!


JS error

Trying to search in the TOC, typing anything throws this error:
filtered-toc.js:16 Uncaught TypeError: fuzzysort.highlight is not a function
    at HTMLLIElement.<anonymous> (filtered-toc.js:16:33)
    at jquery.min.js:2:24485
    at Function.grep (jquery.min.js:2:3421)
    at j (jquery.min.js:2:24454)
    at S.fn.init.filter (jquery.min.js:2:25133)
    at HTMLInputElement.<anonymous> (filtered-toc.js:10:55)
    at HTMLInputElement.dispatch (jquery.min.js:2:43064)
    at v.handle (jquery.min.js:2:41048)

and the search does not function.

@coke oops. Not sure when this stopped working. However,

the filter in ToC is different to the filter for the search button. But the functionality is essentially the same, so it might make sense to load only one JS library.
is the filter in ToC useful? Perhaps we should remove it altogether?

I have found it useful myself over the past few weeks.

We don't have automated testing on it, so I don't know. We would probably have to do a git bisect with a locally running copy of the site to find out. I can try this if it would help.

@coke some git magic would be appreciated

Need some help from @dontlaugh  - I have instructions for running the site locally, but it's always using latest, not the checked out version (so doing a git bisect doesn't help in that case)

@coke @finanalyst  Here is a way to do it. After ./rendered_html has been generated by the build process, run the following from the root of the repo
podman run \
  -v $PWD/rendered_html:/usr/share/caddy \  # mount host_path:container_path (dir)
  -v $PWD/Caddyfile:/etc/caddy/Caddyfile \  # mount host_path:container_path (single file)
  -p 8080:80 \                              # map host port 8080 to container 80
  docker.io/caddy:latest                    # use the upstream caddy image

I use podman, but you can replace it with docker and the rest of the parameters are the same.
Normally, when we build a docs site container, we pack all these files in, so all we need to do is ship a container to the server.
But in this case, we mount the config file and the static assets into the container at the default paths. That's /etc/caddy/Caddyfile and /usr/share/caddy, respectively.
Leave this running in a terminal, and you should be able to update the rendered html static assets, reload the webpage, and see your changes.
Let me know if it doesn't work. I think it should be pretty portable as long as you have docker or podman installed.

https://github.com/farzher/fuzzysort

For the record. The ToC filter uses fuzzysort, and ver 3.0.1 changed the API. Previously, the library was sourced from git, so we got the latest version. A cdn with link<EMAIL_ADDRESS>has the old API, which allows us to pin the library to a working version.


Update Screenshots in Documentation

This PR updates several screenshots to reflect the visual changes in version 0.17.

Well done, I like the dark mode and text-size comparison.


Use ctx2 instead ctx.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Looks ok - and it looks trivial to me from a CLA perspective.
@Richard-Koenning if you agree please can you amend your commit message to include the text "CLA: tivial" in a line on its own the body of the message.

Looks okay to me, too, and I agree it is trivial.

@mattcaswell I added the 1.1.1 label since the bug is present there as well; your comments so far are ambiguous as to whether it should be merged back to that branch as well.

Yes. 1.1.1 too. Whoever merges will have to remove the merge commit.

I will plan to do the merge, to save Matt's time for 3.0 stuff

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Pushed to master and 1.1.1; closing.
Thanks for submitting the fix!


adds support for registering descriptions and example imports for artifact classes

Adds support for registering descriptions and import examples for artifact classes. Artifact class is a new term that we're using to refer to semantic types with a registered directory format. Previously these would be registered with Plugin.register_semantic_type_to_format, and that is still possible for backward compatibility. These can also now be registered with Plugin.register_artifact_class, which optionally takes a description and a dict of usage examples.
Addresses #649.

Thanks @gregcaporaso!

Thanks for the reviews @ebolyen!


Schedule Service Refactor (Edinburgh Roadmap Item)

Scheduler service (and the metadata data) will be altered to associate schedule events/schedules to a particular service owner and allow the Scheduler Service to exercise those “owned” by the Scheduler Service on the appropriate service.

Duplicate of #796


Multicontext learning

Adds multicontext embedding model
Also updates LSTM for new Snorkel data model
Fixes #387

@ajratner Everything looks pretty bad to me

@henryre yeah...


The splunk logging target UI issue

What kind of request is this (question/bug/enhancement/feature request): bug
Rancher version: rancher/rancher:master 0226
Steps to reproduce (least amount of steps as possible):

Configure splunk logging
Enable SSL Verify and enter the CA Certificate
Click on TEST or SAVE

Result:
After clicking TEST or SAVE, the page will slide up
Before clicking the button:

After clicking the button:

related issue: https://github.com/rancher/rancher/issues/17782
It is SSL config being clear made the page slide up.

Version: Master 2.2 (2/28/19)
Confirmed issue. Test with both Elasticsearch and Splunk and when click save and test page no longer slides upwards.


[Benchmark] Add block_size option to benchmark_throughput.py

Add block_size and max_num_seqs options to benchmark_throughput.py
BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

Can we actually remove this parameter and let each hardware or attention backend choose their own?

Can we actually remove this parameter and let each hardware or attention backend choose their own? @liangfu Does this sound good to you if we make such a change?

I actually don't think block size is tightly coupled with attention backend especially when we have prefix caching. For example, we could intentionally enlarge block size to reduce hashing overhead, or reduce block size to increase the hit rate. However if this script is not supposed to benchmark prefix caching we could just hard coded block size for each backend.


FIX: tl.program_id() does indeed not have a cast method in triton2.3.1

Summary
https://github.com/linkedin/Liger-Kernel/pull/251 casted program_id to int64 in SwiGLU/GeGLU, but it's not compatible for triton2.3.1.

Testing Done

Hardware Type:
[x] run make test to ensure correctness
[x] run make checkstyle to ensure code style
[x] run make test-convergence to ensure convergence

consistent with https://github.com/linkedin/Liger-Kernel/blob/main/src/liger_kernel/ops/cross_entropy.py#L42, lgtm


Deployment fail AnnotationRegistry not found

Hi,
I have got a problem when i deploy my app :
*** [err :: toto.com] PHP Fatal error:  Class 'Doctrine\Common\Annotations\AnnotationRegistry' not found in /srv/www/toto/releases/20120217153948/app/autoload.php on line 36
command finished in 173ms
*** [deploy:update_code] rolling back

executing "rm -rf /srv/www/toto/releases/20120217153948; true"
servers: ["toto.com"]
[toto.com] executing command
command finished in 156ms
failed: "sh -c 'cd /srv/www/toto/releases/20120217153948 && php app/console assets:install web --env=prod'" on toto.com

I can explain this. One idea ?
Thanks in advance.

Hi,
Having exacty the same problem. When trying to execute the command "php app/console assets:install web --env=prod" manually it runs fine.
Anyone?


Test Sully HL7 sample data with HL7 ingest

We have received a new sample data from Sully and we need to run it through the HL7 ingest process to see if we find any issues with the ingester.
Sully HL7 file
Acceptance criteria:

Add the HL7 data to the Observation Message Tests and verify the expected values for the data.

@jorg3lopez this was on Cameron to complete but he has rolled off. I'm giving to you to complete. There's already a PR for it, so you just need to checkout the branch and verify that this works. The PR is here: https://github.com/CDCgov/prime-reportstream/pull/1920


NX Dependency Override - remove whenever possible

once the dependency for NX fixes this issue, remove the override.
          We should note in comments why and when this override is no longer needed if possible

Originally posted by @Nana-EC in https://github.com/hashgraph/hedera-json-rpc-relay/pull/1634#discussion_r1291857566

@AlfredoG87 is this still applicable?

we still have the dependency, not sure why

@AlfredoG87 is this still applicable?

I've created a PR for this, seems like the issue is resolved now. 👍
https://github.com/hashgraph/hedera-json-rpc-relay/pull/2365


Add a log config when starting SAQ

It would be nice to have a way to pass a log config in SAQ like in ARQ like this: https://github.com/samuelcolvin/arq/blob/main/arq/cli.py#L41

I believe this was addressed here https://github.com/tobymao/saq/pull/28.
So we won't be supporting passing in a log config, but you can override it like in the discussion.


Non text support

to enable copy of binary data. eg files, images, etc
@mfkl do u think this is achievable?

I think so (probably cleaner on iOS than Android though).
Android:
https://stackoverflow.com/questions/30660771/how-to-copy-images-files-to-clipboard-in-android-any-alternative-methods-steps
https://developer.android.com/guide/topics/text/copy-paste.html#Clipboard
iOS
https://developer.apple.com/documentation/uikit/uipasteboard/1622069-image?language=objc

Should this warrant a new package? Thinking TextCopy might be confusing for a package name that does copying of non-text things, might be fine though... what do you think?

yeah happy with a rename. but to what?

You seem much more inspired than me at naming nugets :D AnyCopy, XCopy, AllCopy, CrossCopy, ClipboardCopy...

dont know why i closed this

Worth noting that non-text will be a pain on Linux, given xsel is text only https://github.com/kfish/xsel/blob/master/xsel.c#L736 whereas xclip worked for any data type with its -target option - certainly it would be possible to do something gross and base64 encode binary data.
It feels like, going forward, using something like https://github.com/ajnewlands/X11.Net and doing the selector manipulation in process in stead of shelling out could be preferable.

Can we get at least partial support for some platforms? Like on Windows?