Add TLS support in scram-proxy

This requires https://github.com/neondatabase/helm-charts/pull/12 to be released first.

Cool!
I tried:
~$ psql postgres://hlinnaka%40zenith@start.stage.neon.tech/fragrant-rice-942513?sslmode=require
psql: error: connection to server at "start.stage.neon.tech" (<IP_ADDRESS>), port 5432 failed: server does not support SSL, but SSL was required

And:
~$ psql postgres://hlinnaka%40zenith@fragrant-rice-942513.cloud.stage.neon.tech/fragrant-rice-942513?sslmode=require
psql: error: connection to server at "fragrant-rice-942513.cloud.stage.neon.tech" (<IP_ADDRESS>), port 5432 failed: ERROR:  Internal error

There are more steps needed to have TLS working?

In proxy logs for your attempt
got message: SslRequest
got message: StartupMessage { major_version: 3, minor_version: 0, params: {"application_name": "psql", "": "", "user": "hlinnaka@zenith", "database": "fragrant-rice-942513", "client_encoding": "UTF8"} }
cplane request: http://console-staging.local/management/api/v2/proxy_get_role_secret?cluster=fragrant-rice-942513&role=hlinnaka%40zenith
error: Bad authentication secret

Looks like something is wrong with communication back to console, cc @kelvich.


Add query CPU time in verifier result

Fetch the job execution CPU time from Stats and record it in verifier result

Addressed the review.

Had a minor comment on naming, otherwise looks good!


Update Namespaces/APIs/Documentation for OpenSearch

Description

Replace documentation links with OpenSearch docs
Update API calls from _opensearch/_opendistro to _plugins
Updated namespaces from com.amazon.opendistroforelasticsearch to org.opensearch
Updated CI workflow branches for reports-scheduler dependencies

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is this PR ready to review? Could you check the CI failure of reports-scheduler?

Is this PR ready to review? Could you check the CI failure of reports-scheduler? @davidcui-amzn

Yes, looking now. Failed after I changed the version from beta1 to rc1

I noticed we still have some opendistro naming in README, could you also fix that in this PR if it's convenient?

https://github.com/opensearch-project/dashboards-reports/blob/dev/README.md#opensearch-dashboards-reports-for-open-distro
https://github.com/opensearch-project/dashboards-reports/blob/dev/README.md#opensearch-dashboards-reports-for-open-distro
Maybe just do a text search for distro

I noticed we still have some opendistro naming in README, could you also fix that in this PR if it's convenient?
* https://github.com/opensearch-project/dashboards-reports/blob/dev/README.md#opensearch-dashboards-reports-for-open-distro

* https://github.com/opensearch-project/dashboards-reports/blob/dev/README.md#opensearch-dashboards-reports-for-open-distro
  Maybe just do a text search for `distro`

Good catch, missed this earlier I only text searched for openDistro

Should the opensearch-job-scheduler-<IP_ADDRESS>-beta1.zip under test resource be renamed to rc-1?

Should the opensearch-job-scheduler-<IP_ADDRESS>-beta1.zip under test resource be renamed to rc-1?

Yes it should not be beta1 anymore, good catch. Will rename that and then merge.


Idea: pod-level health checks

All of our health checks are currently on containers.  If a check fails, we restart the single container.  I'm looking at the DNS pod, which runs etcd and kube2sky and skydns.  We have a healthcheck for skydns - do a DNS lookup.  But sometimes it fails because there's an etcd failure or a kube2sky bug.  what I really want is to restart the whole pod when the healthcheck fails.

Shouldn't a probe for kube2sky/etcd catch this and restart those? It sounds more like a e2e probe. If the problem is, say a kube2sky bug, how would restart help?

There once was a bug in kube2sky's watch loop.  It stopped updating DNS.
Hypothetically we could TTL all our etcd entries and then the healthcheck
would fail and a whole-pod kill would fix it.
But point taken, kube2sky would not add new records but it would have old
records.  Maybe we should have a sentinel DNS value that has a TTL that
kube2sky has to update periodically.
On Wed, May 13, 2015 at 10:26 AM, Rohit Jnagal<EMAIL_ADDRESS>wrote:

Shouldn't a probe for kube2sky/etcd catch this and restart those? It
sounds more like a e2e probe. If the problem is, say a kube2sky bug, how
would restart help?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/8186#issuecomment-101752477
.


I don't want to download file instead I want to show it as text file. Is it possible?

Version
{version or tag here}
File / Feature
{file or feature containing the issue}
Expected behaviour
{what should happen}
Actual behaviour
{what happens}
Stacktrace / Error message
{paste here}

Other info
{anything else that might be related/useful}

Yes. See also https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Disposition for background information.
from ranged_fileresponse import RangedFileResponse

def some_proxy_view(request):
    filename = 'myfile.txt'
    response = RangedFileResponse(request, open(filename, 'r'), content_type='audio/wav')
    response['Content-Disposition'] = 'inline'
    return response


Jest 29.5.0 and fix missing rootDir in config

npm run test + existing build -> gives the warning below. Easy fix + update jest
><EMAIL_ADDRESS>test
> NODE_ENV=test jest --maxWorkers=1 --silent

jest-haste-map: Haste module naming collision: next-i18next-create-client
  The following files share their name; please adjust your hasteImpl:
    * <rootDir>/src/createClient/package.json
    * <rootDir>/dist/commonjs/createClient/package.json

Checklist

[x] only relevant code is changed (make a diff before you submit the PR)
[x] run tests npm run test
[x] tests are included
[x] commit message and code follows the Developer's Certification of Origin

Checklist (for documentation change)

[x] only relevant documentation part is changed (make a diff before you submit the PR)
[x] motivation/reason is provided
[x] commit message and code follows the Developer's Certification of Origin

@adrai FYI this PR is very minor, I've merged it


Fix according new tests

fix work with huge step, restore padding(for case name_8, name_10, name_16)

Hi @ViacheslavYartsev what do you want to accomplish with this pull request ?
I'm glad to improve the library if you need.

I found some bugs: 1. restore name in case file_08, file_10, file_16 - forming sequence for file_08 and fiile_10, but file_16 - changed to file_6.
2. sequence step is char and check range is absent. So if step more then 127 will be incorrect sequence. I added testcases and some verification. Pull request not ready yet (need cleanup)

I'm restructuring the library right now can I ask you to create some bugs
instead ?
Judging from the PR I saw, I suspect the merge to be hard.
Le sam. 21 nov. 2015 à 07:04, ViacheslavYartsev<EMAIL_ADDRESS>a écrit :

I found some bugs: 1. restore name in case file_08, file_10, file_16 -
forming sequence for file_08 and fiile_10, but file_16 - changed to file_6.
2. sequence step is char and check range is absent. So if step more then
127 will be incorrect sequence. I added testcases and some verification.
Pull request not ready yet (need cleanup)
—
Reply to this email directly or view it on GitHub
https://github.com/gchatelet/light_sequence_parser/pull/6#issuecomment-158592755
.


Update crate link and fix the release action

@dilawar Can you make a crates.io account for yourself? I think it is best that I add you as owner there too.

Thanks. Version 0.3.2 is published now.


OS X + GPU NVIDIA nccl dependency missing

Comment migrated to a full issue here.

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): N/A
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): OS X 10.12.2 (Any)
TensorFlow installed from (source or binary): Source
TensorFlow version (use command below): Master @HEAD
Python version: N/A
Bazel version (if compiling from source): N/A (CMake)
GCC/Compiler version (if compiling from source): Xcode 8.3.3 (Any)
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: cmake -GXcode ...

Describe the problem
I'm trying to extend CMake-Linux support for GPU-build to finish CMake + OSX testing w/ (NVIDIA) GPU.   I had started with CUDA 8.0 and cuDNN 6.0, but given recent activity here, it seems better to target a recent master branch w/ CUDA 9.0 + cuDNN 7.0.  The OS X CUDA installation does not come with nccl (see find output below for contents).  NVIDIA has a (login based) nccl download for v2.0, but it only has Linux libraries.  I see three other options to support OS X + GPU Builds:

https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/nccl (build from bundled source v1.3.5)
https://github.com/nvidia/nccl (build from source v1.?)
add option(tensroflow_USE_NCCL "Use NCCL lib" ON) to make it nccl optional (I'm not sure how complicated this is) and update the CMake and source code w/ preprocessor definitions as needed

Shall I add the bundled nccl (v1.3.5?) files to the CMake build?  I see notes about migrating to v2.0 in commit comments, which seems to be closed source.  If that's the case, then it may be best to try to make any internal tensorflow nccl operations optional in order to support single GPU configurations to start with.
find /Developer/NVIDIA/CUDA-9.0/lib -name "*.a"
/Developer/NVIDIA/CUDA-9.0/lib/libcublas_device.a
/Developer/NVIDIA/CUDA-9.0/lib/libcublas_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libcudadevrt.a
/Developer/NVIDIA/CUDA-9.0/lib/libcudart_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libcufft_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libcufftw_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libculibos.a
/Developer/NVIDIA/CUDA-9.0/lib/libcurand_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libcusolver_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libcusparse_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppc_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppial_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppicc_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppicom_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppidei_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppif_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppig_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppim_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppist_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppisu_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnppitc_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnpps_static.a
/Developer/NVIDIA/CUDA-9.0/lib/libnvgraph_static.a

Source code / logs
N/A

We don't officially support OSX + Nvidia, so I'm marking this as community support.
Please feel free to put together a PR!

I will take a look.  Do you know which version of nccl is currently required for master?

I'm unfamiliar with the OSX build process myself, so I don't know, no.

Closing this issue due to staleness. Please use the latest version of TensorFlow and build again.
Feel free to report any issues you encounter with latest TensorFlow. Thanks!


when I set the light  theme, the cusor fade...........

when I edit in normal-mode and insert-mode, I can't find my cursor because the cursor and the background are all light. What should I do?

This is either an issue with your Terminal, Client (gvim, nvim-qt, etc.) or you might want :h guicursor.
Here's example's of different terminal behavior:

Kitty: white cursor in dark mode and light gray in light mode. Perfectly distinguishable all the time.
Alacritty: cursor color are reversed background and foreground. Very easy to find it.
Windows Terminal: Always white. I had to change it to something like a very light blue to make it visible in both dark and light theme.

This is either an issue with your Terminal, Client (gvim, nvim-qt, etc.) or you might want .
Here's example's of different terminal behavior::h guicursor

Kitty: white cursor in dark mode and light gray in light mode. Perfectly distinguishable all the time.
Alacritty: cursor color are reversed background and foreground. Very easy to find it.
Windows Terminal: Always white. I had to change it to something like a very light blue to make it visible in both dark and light theme.

Thank you very much. I've solved it.


STAT-171: Add parameter for license year

Don't merge until a new release of ssb-project is made (>1.0.4)

New release of ssb-project is made, safe to merge


Unexpected CUDA out of memory errors

System Info
ghcr.io/predibase/lorax:latest
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
First, run
docker run --gpus all --shm-size 1g -p 8000:80 -e "HUGGING_FACE_HUB_TOKEN=<token>" -v $volume:/data ghcr.io/predibase/lorax:latest --trust-remote-code --model-id meta-llama/Llama-2-7b-chat-hf
Then, send the following requests:
from concurrent.futures import ThreadPoolExecutor
import time
import requests

url = "http://<IP_ADDRESS>:8000/generate"
headers = {"Content-Type": "application/json"}

# Function to send a request
def send_request(payload):
    start_time = time.time()
    response = requests.post(url, headers=headers, json=payload)
    elapsed_time = time.time() - start_time
    return response.json(), elapsed_time

# Number of concurrent requests

adapter_list = [<list of 15 adapters>]
num_requests = len(adapter_list)

# Using ThreadPoolExecutor to send requests concurrently
with ThreadPoolExecutor(max_workers=num_requests) as executor:
    # Submit the requests
    futures = []
    for i in range(num_requests):
        payload = {
            "inputs": "Hello, my name is",
            "parameters": {"max_new_tokens": 100, "adapter_id": adapter_list[i]},
        }
        futures.append(executor.submit(send_request, payload))

    # Wait for all requests to complete
    results = [future.result() for future in futures]

print(results)

Expected behavior
First of all, awesome work with lorax! When sending requests at the same time like this, I'm receiving Cuda Out Of Memory errors on the server. I thought that actually, the server would check beforehand how many tokens can be handled and consequently enqueue requests that cannot be served. Have you encountered this before?

Hey @nearmax-p, thanks for the kind words!
Can you share the error message / stack trace you're seeing on the server? When stress testing LoRAX, we have occasionally seen issues where the block allocator for paged attention will complain that there's not enough GPU memory to allocate any more blocks. The error is transient, though, so the workaround for now is to resubmit the request, but we plan on properly addressing the issue with backoff logic in the server itself.
It's possible other situations could arise that would cause unexpected memory pressure, like trying to load adapters with very large ranks, but I haven't encountered such an issue in my tests.
In general, when CUDA OOMs arise, it's recommended to try varying some of the parameters the affect max batch size. In particular, these are the params we spend time tuning:

max-input-length
max-total-tokens
max-batch-total-tokens
max-batch-prefill-tokens

In the future, we also plan on investing in creating auto-tuning logic for these.

Thanks for the quick response! Sure, here is the error message, probably not too informative:
2023-11-22T15:43:35.808913Z ERROR HTTP request{otel.name=POST /generate http.client_ip= http.flavor=1.1 http.host=<IP_ADDRESS>:8000 http.method=POST http.route=/generate http.scheme=HTTP http.target=/generate http.user_agent=python-requests/2.31.0 otel.kind=server trace_id=6cbea2cc1ef6c755017450098ce24840}:generate{parameters=GenerateParameters { adapter_id: Some("adapter_id"), adapter_source: None, best_of: None, temperature: None, repetition_penalty: None, top_k: None, top_p: None, typical_p: None, do_sample: false, max_new_tokens: 500, return_full_text: None, stop: [], truncate: None, watermark: false, details: false, decoder_input_details: false, seed: None }}:generate{request=GenerateRequest { inputs: "Hi! my name is", parameters: GenerateParameters { adapter_id: Some("adapter_id"), adapter_source: None, best_of: None, temperature: None, repetition_penalty: None, top_k: None, top_p: None, typical_p: None, do_sample: false, max_new_tokens: 500, return_full_text: None, stop: [], truncate: None, watermark: false, details: false, decoder_input_details: false, seed: None } }}:generate_stream{request=GenerateRequest { inputs: "Hi! My name is", parameters: GenerateParameters { adapter_id: Some("adapter_id"), adapter_source: None, best_of: None, temperature: None, repetition_penalty: None, top_k: None, top_p: None, typical_p: None, do_sample: false, max_new_tokens: 500, return_full_text: None, stop: [], truncate: None, watermark: false, details: false, decoder_input_details: false, seed: None } }}:infer:send_error: lorax_router::infer: router/src/infer.rs:613: Request failed during generation: Server error: CUDA out of memory. Tried to allocate 2.00 MiB (GPU 0; 39.39 GiB total capacity; 37.87 GiB already allocated; 1.06 MiB free; 39.39 GiB allowed; 37.87 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Regarding the parameters for tokens: I've experimented a bit, and I do not believe that the issue here is due to the amount of tokens being processed, but rather due to keeping multiple adapters in GPU memory.
On an A100 (40GB) GPU, concurrent requests have been handled well for 10 adapters at the same time, but once I sent requests for 11 adapters, the OOM error came up again. This happened no matter the length of the prompt and no matter how high `max_new_tokens' was.
I've also tried sending substantially more concurrent requests (50+) to a group of 10 adapters (5 for each adapter), the error didn't occur here, so I assume it is really due to having multiple adapters loaded into GPU memory at the same time. The adapters I am using have rank 64 and target modules ["q_proj", "v_proj", "k_proj", "o_proj"], is this much larger than what you use? Happy to provide any more information here if needed :)

Hey @nearmax-p, apologies for the late response. Rank 64 is larger than what we typically use, but I have run some tests with adapters that large. As such, some of the default values we set might need to be adjusted to properly handle larger rank adapters.
In particular, we have a param called --max-active-adapters that controls how many adapters can be placed on the GPU at once. We default this to 128, but in your case it might be worth trying a much smaller number (like 8 or 10) and see if that addresses the problem.
The only downside to setting this value lower is that some adapters will then need to "wait their turn" to be placed onto the GPU. This shouldn't noticeably affect the overall throughput of the system, but can affect latency. One additional parameter you can use to tradeoff between latency and throughput is --adapter-cycle-time-s (default: 2). If you find that you'd rather decrease throughput in favor if lower latency, you can try decreasing this value.
Let me know if reducing max active adapters helps to address the memory pressure!

Hey @nearmax-p, just wanted to follow-up and see if configuring --max-active-adapters resolved your OOM issue. We're also working on making this more robust by providing a --max-activate-adapters-memory argument to let you configure the pool size without having to determine the exact number of adapters by hand.


chore: bdd-spring-1573502557 to 0.0.1

chore: Promote bdd-spring-1573502557 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Module imports without extension fail loading

I've configured regarding the documentation as follows
karma.config.js
    files: [
      { pattern: 'test/**/*.js', type: 'module' },
      { pattern: 'src/**/*.js', type: 'module', included: false }
    ],

Then when using a working mocha test with imports without .js extensions, like
import TestModule from '../../src/TestModule';

...the execution with karma fails with:
...
26 05 2021 15:09:57.267:WARN [web-server]: 404: /base/src/TestModule
Chrome Headless 90.0.4430.212 (Mac OS 10.15.7) ERROR
  Uncaught Error loading test/Test.js
  at http://localhost:9876/context.html:286:271

Importing '../../src/TestModule.js' does work, but is not a solution as the TestModule.js itself imports without extensions. The implementation of TestModule.js shall not be changed.
Is this a bug in Karma runner or did a missed an important configuration here? Help appreciated

Ok,  closing this one. I think it's a duplicate of https://github.com/karma-runner/karma/issues/3335

Sorry, not sure if this one is a duplicate of #3335
They do not mention extensions there. Re-opening. Any comments?

Seems to work as intended: https://stackoverflow.com/a/55289496/1377864

In a browser with support for ES Modules (ESM), no extensions will be added to the URL that you import - if your file for example has .js extension, you have to write import {x} from "./file.js".

Unless you told it to do so, Karma will not do any file processing, so it will just load the file "as is" in the browser.

Got it working as i needed it with karma-webpack . I believe it works there due to webpack's transformPath
If there is no way doing it in karma without any plugins then we can close this issue.

No, there is no way to do it without plugins. Vanilla Karma just loads your JS code in the browser without any extra processing and therefore JS should be in the format which the browser understands.


More neutrally bot avatar

Do you think you could change the bot's icon to a more neutrally one?

no


is there still a support server?

sorry for posting here, the bot was only available from this site: https://dsc.cool/bot/396466836331429889
I am trying to set it up in a new server but the bot never seems to respond to commands. is something wrong with it?

Sorry for late response. Support server was removed when I couldn't afford to host the bot (server fees were higher than I could handle at that time).
I'd like to bring it back soon, but I need to look into the code and rewrite some parts from scratch.

Weird, are you not the one hosting the current bot then? It's currently running as far as I can tell.

The bot itself is down. so it's impossible to add it to a new Discord server.
But the service which sends notifications with found games is up and running because it requires much fewer resources than the bot did.
As I said before, I'd like to bring the project back to life.
Discord.js seems to be not the best choice for a slightly bigger bot. I have plans to rewrite it with typescript and eris.js

I see! Thanks for clarifying. I've only ever made a bot that's up 24/7 so this is new to me.
Best of luck with the rewrite. I definitely like the typescript route and while I'm not familiar with eris.js,I have a framework I use for my bots that might be of interest: https://github.com/seiyria/DiscordBotExample
Though it really only covers the core. Not sure how well it scales since my bots are generally only in a few servers at most.


Add pyright to style group

We've added isort, black, and pylint. I'm using pyright for static type checking in VSCode. I'd like to add that to the style dev-dependencies group, CI, and DEVELOPERS.md as well.

CC @nahumsa

Awesome! If you don't mind, I can add that :)

Sounds good! Thanks!


Failed to run "pip3.9 install --disable-pip-version-check --target . werkzeug==1.0.1" | Error: spawn pip3.9 ENOENT

datasette, version 1.0a12
datasette-publish-vercel: latest version
Vercel CLI 33.6.1 (can successfully deploy any other python serverless projects, just doesn't work with datasette-publish-vercel)
# get test.db
echo '{"1": 2}' | sqlite-utils insert test.db test -
datasette publish vercel --project test-db test.db --debug
(node:39238) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
> [debug] [2024-04-03T12:37:23.772Z] Found config in file "/private/var/folders/km/6sczydd546n7xmy21z8yndzw0000gn/T/tmp4i1laanb/datasette-now-v2/vercel.json"
Vercel CLI 33.6.1
> [debug] [2024-04-03T12:37:23.793Z] user supplied a possible target for deployment or an extension
> [debug] [2024-04-03T12:37:23.836Z] failed to find extension command with name "vercel---confirm"
WARN! `--confirm` is deprecated, please use `--yes` instead
> [debug] [2024-04-03T12:37:23.875Z] Setting target to production
> [debug] [2024-04-03T12:37:23.881Z] Aborting search for repo root
> [debug] [2024-04-03T12:37:23.884Z] Spinner invoked (Loading scopes…) with a 1000ms delay
> [debug] [2024-04-03T12:37:23.900Z] #1 → GET https://api.vercel.com/v2/user
> [debug] [2024-04-03T12:37:23.903Z] #2 → GET https://api.vercel.com/v1/teams
> [debug] [2024-04-03T12:37:24.236Z] #2 ← 200 OK: sfo1::wzjtt-1712147844252-49579b8c26ac [333ms]
> [debug] [2024-04-03T12:37:24.239Z] #1 ← 200 OK: sfo1::pww66-1712147844254-52a280f48866 [339ms]
> [debug] [2024-04-03T12:37:24.245Z] Spinner invoked (Searching for existing projects…) with a 1000ms delay
> [debug] [2024-04-03T12:37:24.247Z] #3 → GET https://api.vercel.com/v9/projects/test-db
> [debug] [2024-04-03T12:37:24.372Z] #3 ← 200 OK: sfo1::pww66-1712147844364-ecba411e64f8 [125ms]
> [debug] [2024-04-03T12:37:24.381Z] Aborting search for repo root
🔗  Linked to tddschn/test-db (created .vercel and added it to .gitignore)
❗  The `name` property in vercel.json is deprecated (https://vercel.link/name-prop)
> [debug] [2024-04-03T12:37:24.389Z] Error while parsing repo data: ENOENT: no such file or directory, open '/private/var/folders/km/6sczydd546n7xmy21z8yndzw0000gn/T/tmp4i1laanb/datasette-now-v2/.git/config'
> [debug] [2024-04-03T12:37:24.464Z] Failed to get last commit. The directory is likely not a Git repo, there are no latest commits, or it is corrupted.
Error: this does not look like a git repo
> [debug] [2024-04-03T12:37:24.466Z] Spinner invoked (Deploying tddschn/test-db) with a 0ms delay
[client-debug] 2024-04-03T12:37:24.466Z Creating deployment...
[client-debug] 2024-04-03T12:37:24.467Z Provided 'path' is a directory.
[client-debug] 2024-04-03T12:37:24.469Z Found 24 rules in .vercelignore
[client-debug] 2024-04-03T12:37:24.469Z Building file tree...
[client-debug] 2024-04-03T12:37:24.473Z Found 4 files in the specified directory
[client-debug] 2024-04-03T12:37:24.475Z Yielding a 'hashes-calculated' event with 4 hashes
[client-debug] 2024-04-03T12:37:24.475Z Using provided API URL: https://api.vercel.com
[client-debug] 2024-04-03T12:37:24.475Z Using provided user agent: vercel 33.6.1 node-v21.7.1 darwin (x64)
[client-debug] 2024-04-03T12:37:24.475Z Setting platform version to harcoded value 2
[client-debug] 2024-04-03T12:37:24.475Z Creating the deployment and starting upload...
[client-debug] 2024-04-03T12:37:24.475Z Determining necessary files for upload...
[client-debug] 2024-04-03T12:37:24.476Z Creating deployment
[client-debug] 2024-04-03T12:37:24.476Z Sending deployment creation API request
[client-debug] 2024-04-03T12:37:25.618Z Deployment response: {"alias":["test-db-tddschn.vercel.app"],"aliasAssigned":false,"bootedAt":1712147845071,"buildingAt":1712147845071,"createdAt":1712147845071,"creator":{"uid":"ntFqYOqK9UsoheLT0RcyiOYx","username":"tddschn"},"id":"dpl_F8sfNbHFvnpYCPLiv87Sofdsb4cZ","lambdas":[{"id":"bld_eos928p0c","createdAt":1712147845473,"entrypoint":".","readyState":"READY","readyStateAt":1712147845473,"output":[]}],"name":"test-db","meta":{},"public":false,"readyState":"QUEUED","regions":["iad1"],"source":"cli","status":"QUEUED","type":"LAMBDAS","url":"test-f63rqqmp9-tddschn.vercel.app","version":2,"build":{"env":["CI","VERCEL","VERCEL_ENV","TURBO_REMOTE_ONLY","TURBO_RUN_SUMMARY","NX_DAEMON","VERCEL_URL","VERCEL_GIT_PROVIDER","VERCEL_GIT_PREVIOUS_SHA","VERCEL_GIT_REPO_SLUG","VERCEL_GIT_REPO_OWNER","VERCEL_GIT_REPO_ID","VERCEL_GIT_COMMIT_REF","VERCEL_GIT_COMMIT_SHA","VERCEL_GIT_COMMIT_MESSAGE","VERCEL_GIT_COMMIT_AUTHOR_LOGIN","VERCEL_GIT_COMMIT_AUTHOR_NAME","VERCEL_GIT_PULL_REQUEST_ID","VERCEL_DEPLOYMENT_ID","VERCEL_DISCOVER_FOLDER_SIZES","VERCEL_NEXT_BUNDLED_SERVER","VERCEL_SERVERLESS_FUNCTION_FAILOVER","VERCEL_ALLOW_RUBY32","VERCEL_LAYER_LAUNCHER","VERCEL_EDGE_FUNCTIONS_BUNDLE_BATCHES","VERCEL_IGNORE_BUILD_STEP_PACKAGE_MANAGER_DETECT","VERCEL_ENABLE_NPM_DEFAULT","ENABLE_VC_BUILD","VERCEL_BUILD_OUTPUTS_EDGE_FUNCTION","VERCEL_EDGE_FUNCTIONS_REGIONAL_INVOCATION","VERCEL_EDGE_FUNCTIONS_EMBEDDED_SOURCEMAPS","VERCEL_EDGE_FUNCTIONS_STRICT_MODE","USE_OUTPUT_FOR_EDGE_FUNCTIONS","NEXT_PRIVATE_MULTI_PAYLOAD","VERCEL_RICHER_DEPLOYMENT_OUTPUTS","VERCEL_EDGE_SUSPENSE_CACHE","VERCEL_SERVERLESS_SUSPENSE_CACHE","VERCEL_BUILD_MONOREPO_SUPPORT","VERCEL_USE_NODE_BRIDGE_PRIVATE_LATEST","VERCEL_USE_EDGE_FUNCTIONS_BRIDGE_LATEST","VERCEL_ENABLE_NODE_COMPATIBILITY","VERCEL_ENABLE_EXTENDED_FALLBACK_PAYLOAD","VERCEL_NEXT_PRELOAD_COMMON","VERCEL_WAKE_UP_DEPLOYMENT"]},"builds":[{"src":"index.py","use":"@vercel/python@3.0.7"}],"createdIn":"sfo1","env":["VERCEL","VERCEL_ENV","TURBO_REMOTE_ONLY","TURBO_RUN_SUMMARY","NX_DAEMON","VERCEL_URL","VERCEL_GIT_PROVIDER","VERCEL_GIT_PREVIOUS_SHA","VERCEL_GIT_REPO_SLUG","VERCEL_GIT_REPO_OWNER","VERCEL_GIT_REPO_ID","VERCEL_GIT_COMMIT_REF","VERCEL_GIT_COMMIT_SHA","VERCEL_GIT_COMMIT_MESSAGE","VERCEL_GIT_COMMIT_AUTHOR_LOGIN","VERCEL_GIT_COMMIT_AUTHOR_NAME","VERCEL_GIT_PULL_REQUEST_ID","DATASETTE_SECRET","VERCEL_DEPLOYMENT_ID"],"functions":null,"ownerId":"<id>","plan":"hobby","projectId":"prj_xvwubM3VOCXxRWmeg22IjYnCvUtK","routes":[{"src":"^(.*)$","dest":"index.py"}],"target":"production","inspectorUrl":"https://vercel.com/tddschn/test-db/F8sfNbHFvnpYCPLiv87Sofdsb4cZ"}
[client-debug] 2024-04-03T12:37:25.618Z Deployment created with a warning:  Due to `builds` existing in your configuration file, the Build and Development Settings defined in your Project Settings will not apply. Learn More: https://vercel.link/unused-build-settings
[client-debug] 2024-04-03T12:37:25.618Z Yielding a 'warning' event
[client-debug] 2024-04-03T12:37:25.618Z Deployment created
[client-debug] 2024-04-03T12:37:25.618Z Yielding a 'created' event
> [debug] [2024-04-03T12:37:25.619Z] Aborting search for repo root
🔍  Inspect: https://vercel.com/tddschn/test-db/<deployment_id> [1s]
✅  Production: https://test-<id>-tddschn.vercel.app [1s]
> [debug] [2024-04-03T12:37:25.684Z] Spinner invoked (Queued) with a 0ms delay
[client-debug] 2024-04-03T12:37:25.684Z Waiting for deployment to be ready...
[client-debug] 2024-04-03T12:37:25.686Z Waiting for builds and the deployment to complete...
[client-debug] 2024-04-03T12:37:26.994Z Deployment state changed to BUILDING
[client-debug] 2024-04-03T12:37:26.994Z Yielding a 'building' event
> [debug] [2024-04-03T12:37:26.994Z] Spinner invoked (Building) with a 0ms delay
[client-debug] 2024-04-03T12:37:31.512Z Yielding a 'error' event
> [debug] [2024-04-03T12:37:31.512Z] Error: Error: spawn pip3.9 ENOENT
Error: spawn pip3.9 ENOENT
    at Now.handleDeploymentError (/Users/tscp/.bun/install/global/node_modules/vercel/dist/index.js:165696:16)
    at processDeployment (/Users/tscp/.bun/install/global/node_modules/vercel/dist/index.js:165456:34)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Now.create (/Users/tscp/.bun/install/global/node_modules/vercel/dist/index.js:165585:28)
    at async createDeploy (/Users/tscp/.bun/install/global/node_modules/vercel/dist/index.js:165182:12)
    at async deploy_default (/Users/tscp/.bun/install/global/node_modules/vercel/dist/index.js:172734:22)
    at async main13 (/Users/tscp/.bun/install/global/node_modules/vercel/dist/index.js:199272:18)
Error: spawn pip3.9 ENOENT
Error: Command '['vercel', '--confirm', '--debug', '--prod', '--env', 'DATASETTE_SECRET=<sec>']' returned non-zero exit status 1.

I tried to deploy sqlite databases in other projects and I found that in new projects the plugin don't work anymore, while in old projects they still work. I don't know why, I haven't reach any usage limit on Vercel. I think one possible explanation is that my new projects are somehow marked to use a new build env on Vercel, but there's no way for me to know for sure.
All my new projects failed like this:
datasette publish vercel --project rick-n-morty-datasette-vercel data/rnm.db --install datasette-search-all --install datasette-render-timestamps --install datasette-render-images --install datasette-uptime --install datasette-render-html \
--install datasette-pretty-json --metadata data/metadata.yml
(node:48325) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
Vercel CLI 33.6.1
WARN! `--confirm` is deprecated, please use `--yes` instead
❗  The `name` property in vercel.json is deprecated (https://vercel.link/name-prop)
🔗  Linked to tddschn/rick-n-morty-datasette-vercel (created .vercel and added it to .gitignore)
🔍  Inspect: https://vercel.com/tddschn/<proj>/MUoFFHYUaqGqNVkEF7PYTntQ5eBY [4s]
✅  Production: https://rick-n-morty-datasette-vercel-h7l7l8hry-tddschn.vercel.app [4s]
Error: spawn pip3.9 ENOENT
Error: Command '['vercel', '--confirm', '--prod', '--env', 'DATASETTE_SECRET=<sec>']' returned non-zero exit status 1.

Failed to run "pip3.9 install --disable-pip-version-check --target . werkzeug==1.0.1" is in the deployment log on Vercel dashboard. I find it extremely weird since any of the dependencies listed in requirements.txt in the generated dir doesn't reply on werkzeug either directly or indirectly, and the deployment log doesn't show more information.

Confirming that changing Node.js to 18.x solved the problem.

I experienced the same issue and before reading this found a different solution, which was to change the vercel.json output in the plugin options to
"use": "<EMAIL_ADDRESS>(the default setting is current "use": "@vercel/python@3.0.7"). It doesn't seem like a great idea to supply my own vercel.json file because then any other vercel.json updates are not propogated but it works.
I chose 4.3.1 because it seemed to be the most popular option here: https://www.npmjs.com/package/@vercel/python?activeTab=versions
This would be a simple change to the _init_.py file... but not sure if it would break the solutions above?
Node version on Vercel remains 20.x

I have created #68 for this


Not working for indian currency INR and Country code INR

Not working for indian currency INR and Country code IN
When I changed the json configurations.
USD and US are working fine
But I want to collect payment in Indian Currency.
Please help

Closing. Feel free to reopen if this does not address your issue.


auth: fix token verification chain

There was a small regression introduced in
https://github.com/distribution/distribution/pull/4349.
Specifically, if the certificate chain verification succeeds we should return immediately instead of following up with further verification checks.
This commit fixes that: we only follow up with further token verifications if the x5c header is missing.
We've also refactored this method so it's hopefully clearer:

removed named returns -- they're unnecessary and add unnecessary cognitive load when reading the code

Closes #4299

There is a regression in the latest release PTAL @thaJeztah @Jamstah 😬

Once this is merged in, I'll cut a patch beta release

This is not currently in any release, right? I was trying out the 3.0.0-beta.1 release and ended up here after googling the problems we see

That's correct, @calmh. We will do another patch release before the stable, but no ETA.


how to use it

bro how to use it

@r00t7oo2jm you need visual studio to compile it...
first you need to compile GenerateRsrc, then use it to generate DataFile.terra file (that's your encrypted payload) you can drag it to here after its generated and compile terra project, Terra.exe is your loader


_load_gt_file too many values to unpack

I feel like a chump, but I have produced groundtruth images along with their natural counterparts and produced a train.txt and val.txt that appear to be the same format as the kittiseg versions. When I run train.py I get the following error:
File "train.py", line 131, in
tf.app.run()
File "/Users/wnas0001/anaconda3/envs/python2/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 44, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "train.py", line 127, in main
train.do_training(hypes)
File "incl/tensorvision/train.py", line 393, in do_training
modules['input'].start_enqueuing_threads(hypes, queue, 'train', sess)
File "/../inputs/kitti_seg_input.py", line 355, in start_enqueuing_threads
gen.next()
File "/../inputs/kitti_seg_input.py", line 155, in _make_data_gen
for image, gt_image in data:
File "/../inputs/kitti_seg_input.py", line 114, in _load_gt_file
image_file, gt_image_file = file.split(" ")
ValueError: too many values to unpack
I believe it has something to do with my txt file, which I've attached. Any help is greatly appreciated!
train.txt

Yes, it seams to have to do with your data file. Maybe an extra new_line at the end of file? The error would happen, if there is a line with more then two white spaces. Try
print(" File: {}".format(file))
To see which throws the error line fails.

Ah, yes there is something funny about the txt file I wrote in atom - I've simply edited your train3.txt and val3.txt files and it seems to work now.

...although it ends and reports "Killed 9"...?


default to ignoring autogenerated columns

Do not include changes to Shape_Length etc when reporting on changed attributes (unless otherwise specified).

List taken from existing geobc changedetector.py script
IGNORE_FIELDS = [
    "OBJECTID",
    "OID_",  # ArcPro adds this to csv files
    "FID",
    "GLOBALID",
    "GLOBAL_ID",
    "SHAPE_LENGTH",
    "SHAPE_LENG",  # .shp truncation
    "SHAPE_AREA",
    "GEOMETRY_LENGTH",
    "GEOMETRY_AREA",
]


gf implementation

it's can be implemented via adding current file include paths to 'path' for buffer
See more details in
:help gf

rj can be used to reach similar result.


file permissions not set when 'fileMode' set in assembly descriptor

i have the following in my assembly descriptor (it's just the relevant snippet)
<fileSet>
  <directory>${project.build.scriptSourceDirectory}</directory>
  <outputDirectory>/</outputDirectory>
  <filtered>true</filtered>
  <includes>
    <include>startup.sh</include>
   </includes>
   <fileMode>755</fileMode>
</fileSet>

when my container gets built, the startup script doesn't have the correct permissions. i'm not sure if this something that should work in the context of this plugin or not.
i also tried setting the permission directly on the local filesystem to see if they carry over but was unsuccessful there as well. right now i'm working around this by specifying bash startup.sh as the command but that somewhat less then idea.
is this a bug? if not, do you have any suggestions on how this could be handled? i want this contained within the data container that gets built rather then mucking w/ the image that will be running it (which in this case is the 'official' tomcat instance).

this was fixed in 0.11.0

Hi, i'm using version 0.11.2 and it seems that this issue is not fixed. i explored the temporary tar from assembly plugin which takes care of filemode, the tar created by maven-docker-plugin does not.

Perhaps its a windows issue since i'm building on a windows box

Thanks for reporting, I'll try to have a look for this.

i'm thinking this is windows related b/c the above works for me on a mac and linux machines.

also, what is the value of fileMode? i just realized what i posted above it slightly incorrect and it should be <fileMode>0755</fileMode>
trying to google but not coming up w/ much - it's possible this is a bug in the assembly plugin as well.

my assembly.xml was :
docker

${project.basedir}/src/main/docker/tagr
/

**/*

755

It is not a bug of the assembly plugin since creating a tar with mvn assembly:single -Ddescriptor=src/main/docker/assembly.xml creates the right attributes. And its surely a windows issue, since building on linux/macos creates the correct images.

The problem is that the fat/ntfs or any other file system like this ones which does not support linux like rights.
The proposed solution (#139) is to use as intermediary a system that does not rely on underling File system but the linux rights (ie: a tgz archive.)


Preparing for release

Preparing for release

ok, I meant also for the below changes on the image
On Thu, Nov 19, 2020 at 8:53 AM Andrew Block<EMAIL_ADDRESS>wrote:

@sabre1041 commented on this pull request.
In Makefile
https://github.com/redhat-cop/group-sync-operator/pull/46#discussion_r526903234
:

@@ -1,6 +1,6 @@
SHELL := /bin/bash

Current Operator version
-VERSION ?= 0.0.7
I updated it to 0.0.7 myself, so moving back to latest was resetting to
the default
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/redhat-cop/group-sync-operator/pull/46#discussion_r526903234,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPERXBWHOCQVRVMIODDGNDSQUPODANCNFSM4T2PJP3A
.

--
ciao/bye
Raffaele

@raffaelespazzoli im trying to create the most frictionless way to deploy the operator locally which is why I set those variables. You can of course override as needed


0.20.6 -> 0.20.7

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[indexer][watermarks][2/n] committer writes upper bounds to watermarks table

Description
Adds watermarks table, and has committer writing upper bounds to it.
The committer works on the checkpoints level, so to simplify what it needs to do, we can have the committer call store.update_watermarks_upper_bound() once when it completes its batch of work. The upper bound update function relies on PrunableTable to tell it which of epoch, cp, or tx should be used to update the upper bound.
Part of a stack of PRs for watermarks

simplify setting up test indexer: https://github.com/MystenLabs/sui/pull/19663
update pruner config: https://github.com/MystenLabs/sui/pull/19637
committer writes upper bounds https://github.com/MystenLabs/sui/pull/19649
pruner writes lower bounds: https://github.com/MystenLabs/sui/pull/19650
pruner prunes (wip)

Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

@gegaowp , extended to all tables and plugged watermark upper bound updates into the objects snapshot handler. lmk how this looks to you


xUnkown not found in logs

When running the PoC the external class loaded (xUnknown.class) from the http server (that the ldap server gives a reference to) doesn't show up as [MODIFIED] or [NOT WHITELISTED] in the logs.
Maybe something to do with "factory"-something something in ldap reference.

@LogFlames
Run the following commands to create the index

JDK: java -jar ~/Desktop/chains/sbom.exe/classfile-fingerprint/target/classfile-fingerprint-0.12.1-SNAPSHOT.jar jdk -o /home/aman/Desktop/chains/log4shell-poc/client/src/main/resources/log4shell.jsonl
Supply chain java -jar ~/Desktop/chains/sbom.exe/classfile-fingerprint/target/classfile-fingerprint-0.12.1-SNAPSHOT.jar supply-chain -i /home/aman/Desktop/chains/log4shell-poc/client/src/main/resources/log4shell.jsonl -s src/main/resources/bom.json

Then to monitor

java -javaagent:/home/aman/Desktop/chains/sbom.exe/watchdog-agent/target/watchdog-agent-0.12.1-SNAPSHOT.jar=sbom=/home/aman/Desktop/chains/log4shell-poc/client/src/main/resources/log4shell.jsonl,skipShutdown=true -jar target/log4shell-poc-1.0-SNAPSHOT.jar &> log.txt

We somehow fixed it.


Lint Programs as a Whole

Right now, TSLint lints files individually. This means that any information in one file (such as type information) is not available while linting other files.
One possible alternative is to, at startup, create a language services instance that has the source for the entire program and pass this to the rules. Full program information could then be used by the rules.

what additional features does this enable? we're not linting types yet anyway.

Possibly stuff like this: https://github.com/palantir/tslint/issues/577

basically any rule that needs to inspect types will need all the type information from the compilation path in order to work reliably

It'd at least be nice to pass a glob to tslint and lint multiple files.

@dallonf When using it programmatically I assume? (Since obviously a glob will work when using it via the CLI)

@JKillian I didn't know that was supposed to work ;) Created a ticket for my seperate issue: https://github.com/palantir/tslint/issues/688

Picking up  @aleksey-bykov's comment, I wonder if we can kick start some sort of effort on this.... We have a long list of rules that we want to write (let's ignore the fact many of these rules should in fact be part of TypeScript core)
@adidahiya @JKillian - are you aware of anyone who has tried to progress this?
As @aleksey-bykov points out, the effort involve in creating a custom linter that does have type information is not hard. The more complicated bit will be working out how to integrate this into tslint

Sorry for the delayed response here @myitcv. I'm not aware of any progress on this so far, feel free to go for it if you have the time! I'm happy to discuss how we can integrate this kind of thing into TSLint as it is now

How will this approach match the compiler options of tsc with those passed to tslint?
For example, we need the same input files, same typings, same moduleResolution flag, same baseUrl,rootDirs (https://github.com/Microsoft/TypeScript/issues/5039), --skipDefaultLibCheck and so on. Otherwise the typecheck will fail when tslint runs, and the user has to carefully duplicate these settings.
One option is to read tsconfig.json as an input to tslint, but some users may have configuration in another place (we often wrap the compiler API to workaround differences in filesystem layout).
Another option is to work with the typescript team to run additional static analysis checks in the compiler itself, using some extensibility mechanism.

@alexeagle - the approach we are following with tsvet is to reference the same tsconfig.json that is referenced by tsc.
As it stands, we only run tsvet once a compilation via tsc has succeeded. This obviates the need to ignore/dedup the compilation errors during the tsvet run.
Ultimately we could do away with the tsc step entirely and output compilation errors and our own custom errors in the tsvet step. This would need tsvet support for --watch and friends to become a reality practically speaking.

If the user will run a single program to type-check, run diagnostics,
produce custom errors, and emit, why not have tsc be that single program?
It seems simpler than replacing it by a different end-user application that
behaves the same as tsc.
On Wed, Mar 30, 2016 at 11:13 AM Paul Jolly<EMAIL_ADDRESS>wrote:

@alexeagle https://github.com/alexeagle - the approach we are following
with tsvet
https://github.com/palantir/tslint/issues/1048#issuecomment-199420447
is to reference the same tsconfig.json that is referenced by tsc.
As it stands, we only run tsvet once a compilation via tsc has succeeded.
This obviates the need to ignore/dedup the compilation errors during the
tsvet run.
Ultimately we could do away with the tsc step entirely and output
compilation errors and our own custom errors in the tsvet step. This
would need tsvet support for --watch and friends to become a reality
practically speaking.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/palantir/tslint/issues/680#issuecomment-203561476

@alexeagle - completely agree. That would be ideal. tsvet is something we can do today... and with relative ease, independent of the tsc compiler.
I see you've already commented on the issue that I believe is tracking this in TypeScript core.
Our work on tsvet is of course compatible with https://github.com/Microsoft/TypeScript/issues/3003 by  virtue of our 'rules' being defined in terms of the compiler API.

Full-program linting is now possible thanks to @ScottSWu's work in #1363! Please try out the new APIs, let us know how they work out, and file follow up issues as necessary ❤

Love it!


add separate channel arg to enable hedging

Pulling this piece out of #26766.
This guards the perAttemptRecvTimeout service config field with the new hedging channel arg, since we won't support that until we finish implementing hedging.

Known issues: #26595 #25100 #26843


Wrong source for eventTasks from XMLHttpRequest in certain browsers

In certain browsers (IE10, IE11, Safar 9), eventTask for XMLHttpRequest has the wrong prefix in source, something like undefined.addEventListener:readystatechange which should be XMLHttpRequest.addEventListener:readystatechange (like it is in chrome)
The reason is that, in those browsers target.constructor['name'] is undefined and that is used as the prefix for the source of the event

+1 Having issues with this as well on Safari and IE


x/build/gerrit: auth fails to look for netrc if git cookies missing

package gerrit intends to first look in the git cookies, then in netrc. But the exec to git returns an exit code of one and returns an error. By ignoring this error and just relying on the output, the correct fallback behavior can be observed.
I'll send a cl soon.

Thanks.


1368: add dialog for updating the course grade display

Delivers #1368 by adding an "Update Course Grade Display" menu option in the Course Grade column and exposing the display settings within new dialog.
I haven't implemented a preview yet.  I was thinking we should centralise all formatting of the course grade to a single component or helper as we display the course grade in several different contexts (instructor and student summary dialogs, student page, settings previews and the course grade column). Does that seem reasonable?  Maybe something along the lines of this https://github.com/payten/sakai/commit/72658e2acb54b37c168a92967b97c55d3d6b31c9 (but with some smarts added to buildGradeMatrix so the Course Grade column can use it efficiently too).

Yeah I was thinking the same thing in regards to the helper. Next up for me is adding total points to the course grade, then this can all be wired together.
For the panel though, students don't have access to get the GradebookInformation so we'll need to figure out a different way to do that. Or maybe just open er up to students ;)


fix_observer_location crashes when HAE{x,y,z} metadata not present

In GitLab by @dstansby1 on Nov 9, 2020, 09:04
AIA maps between 2020-05-03 and 2020-05-18 seem to be missing the hae metadata, which means that fix_observer_location fails with the following error:
  File "/Users/dstansby/github/solarsynoptic/aia_helpers.py", line 98, in prep
    m = fix_observer_location(m)
  File "/Users/dstansby/miniconda3/lib/python3.8/site-packages/aiapy/calibrate/meta.py", line 34, in fix_observer_location
    x=smap.meta['haex_obs'] * u.m,
  File "/Users/dstansby/miniconda3/lib/python3.8/site-packages/sunpy/util/metadata.py", line 49, in __getitem__
    return OrderedDict.__getitem__(self, key.lower())
KeyError: 'haex_obs'

I'm not sure whether the solution here is to:

Upstream this to whoever makes the AIA maps
Fallback to other metadata
Raise a slightly better warning message?

Anyway, thought I would report just in case it's helpful.

In GitLab by @wtbarnes on Nov 9, 2020, 09:08
This is concerning. It also means the derived observer coordinate on the sunpy map will be incorrect for this set of images.

In GitLab by @dstansby1 on Nov 10, 2020, 03:08
I've deleted and re-downloaded the AIA maps, and they seem to be fine now - I wonder if I was accidentally using an old lower level data product without realising. Anyway, all seems fine from my end, sorry for the noise!

In GitLab by @markcheung on Nov 9, 2020, 09:28
Thanks @dstansby1 will bring this to the attention of JSOC.

In GitLab by @markcheung on Nov 9, 2020, 09:34
When I spot check some files at the JSOC (http://jsoc.stanford.edu/ajax/lookdata.html) I find the HAE[XYZ]_OBS keywords are populated. Could you give an example of some images with this problem? The FSN keyword can be used as the unique identifier for each image.

In GitLab by @markcheung on Nov 11, 2020, 09:15
Thanks for checking @dstansby1. We appreciate contributions from the community.


Release Streams version 2.3.1

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Some not included under this PR


"Here, picture, picture, picture" error

[Hi,]
I have installed Piper (with Libratbag & Ratbagd) on Arch, but when I run Piper I am unable save device settings onto device (but I can change device settings & I can see the current loaded presets).
I am guessing the "Here, picture, picture, picture" is an error message, but I don't quite know where to trouble shoot from here.
Any help would be most appreciated, really love Libratbag! Great project!
Cheers,
Alex

"Here, picture, picture, picture" just means that no one created an image for your specific mouse.
Which mouse do you have?

Sorry, I should have stated that: Logitech G303

Ok. Piper is not as complete as I thought it was. It seems saving is indeed still todo:
https://github.com/libratbag/piper/blob/6d44a84e451a5ae9aa15e519bbc13f4b4a26d1f1/piper/__init__.py#L324
If you are comfortable with the terminal there is the ratbag-command which will save to the mouse.

Hey, thanks, Piper isn't working on my system correctly, (Mouse is detected but when I edit button values they are all showing up as button 0).
However ratbag-command is working fine. I was able to change resolutions and modes etc.
Cheers,
Alex

sorry, I barely have time to spend on libratbag these days, I mostly try to get to pull requests in something resembling reasonable time. Piper is almost abandoned right now.

Hey @whot, hmm, was looking forward to using Piper with my G303, (hence the PR for the SVG).
However ratbag-command is very useful and works well.
Are there plans to create a piper replacement? (or a GUI frontend to ratbag-command?)

Well, Piper is the Future :-)
ratbag-command needs root access, while Piper uses ratbagd, the systemd service which allows to configure the mouse through dbus, without root access. So basically, we need to enhance piper or have a different TUI if you want one. ratbag-command is first and foremost a developer tool that allows us to test the changes, but we don't really want end-users to use it (because of root access and not very friendly interactions).
To give you a rough idea, we are not GUI developers, but we wanted to have a starting point so others could jump in and improve the setting tool. This did not work as we expected, but we are gladly accepting PR on Piper too.
Note: As @whot said, I am too in a state where libratbag is very low priority compared to other projects I have at work. So sorry if you need to wait a bit for answers :(

No problem at all guys! I was able to do what I want with ratbag-command, change buttons resolution etc.  That's what I wanted to do, and it worked.
I would be happy to help out with more mice SVG's for the GUI (if that helps).

I would be happy to help out with more mice SVG's for the GUI (if that helps).

That will definitively help :) Any help is appreciated, and this part would be a great addition.

Ok! I already have the g900 done, (I did this as a top view as that thing has SOOOOO many buttons!).

afaict, commit e9f0c fixed this one?
The piper issues are still what they are, but @Hjdskes is working on that so it'll go away. Meanwhile, the SVG error should be fixed for this mouse now, so we're good here.


JsonObject.read_from_file error?

Perhaps because of some wrong version?
ERROR: 2019-04-15 07:46:26,182 {common.py:170}: Error reading /home/karthik/Documents/Chennai-2018.json :
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/sanskrit_data/schema/common.py", line 162, in read_from_file
    obj = cls.make_from_dict(jsonpickle.decode(fhandle.read()))
  File "/usr/local/lib/python3.6/dist-packages/sanskrit_data/schema/common.py", line 133, in make_from_dict
    recursively_set_jsonpickle_type(dict_without_id)
  File "/usr/local/lib/python3.6/dist-packages/sanskrit_data/schema/common.py", line 124, in recursively_set_jsonpickle_type
    some_dict[JSONPICKLE_TYPE_FIELD] = json_class_index[wire_type].__module__ + "." + wire_type
KeyError: 'Panchangam'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "./update_test_json.py", line 19, in <module>
    panchangam_actual = JsonObject.read_from_file(filename=os.path.join(ORIG_DATA_PATH, fname))
  File "/usr/local/lib/python3.6/dist-packages/sanskrit_data/schema/common.py", line 171, in read_from_file
    raise e
  File "/usr/local/lib/python3.6/dist-packages/sanskrit_data/schema/common.py", line 167, in read_from_file
    obj = cls.make_from_dict_list(jsonpickle.decode(fhandle.read()))
  File "/usr/local/lib/python3.6/dist-packages/sanskrit_data/schema/common.py", line 144, in make_from_dict_list
    assert isinstance(input_dict_list, list)
AssertionError

I'm basically trying to read a pre-written json file and dump it with floating_point_precision=4:
TEST_DATA_PATH = os.path.join(os.path.dirname(__file__), 'spatio_temporal/data')
ORIG_DATA_PATH = os.path.expanduser('~/Documents')
for fname in sys.argv[1:]:
    panchangam_actual = JsonObject.read_from_file(filename=os.path.join(ORIG_DATA_PATH, fname))
    panchangam_actual.dump_to_file(filename=os.path.join(TEST_DATA_PATH, fname), floating_point_precision=4)

How did you generate this file you're reading? Can you attach it here? A generally safe way to create a file is to dump a python object.

The file was generated using get_panchangam
Chennai-2019.json.gz
annual.py
1551:                 panchangam.dump_to_file(filename=fname)

This should have worked, right?

I think I got it.
The line 1525 in panchAngam should run in order for the reading to work.
common.update_json_class_index(sys.modules[__name__])

Which is why the reading happens properly in test_annual.py (which I updated and ran with your file) - it does:
from jyotisha.panchangam.spatio_temporal import City, annual

which runs the aforementioned line.
.


Compile error: missing module http

Description
I tried to use Fable.SignalR in my project with the steps given in the docs, but I am running into compile errors.
Steps to reproduce
I've added Fable.SignalR, SignalR.Elmish, SignalR.Feliz. I've even tried to add SimpleHttp...
Error message
.fable/Fable.SignalR.0.11.5/HubConnection.fs(332,46): (332,50) error FSHARP: The namespace or module 'Http' is not defined. Maybe you want one of the following:  HttpClient (code 39)

There is a fable compile warning which might be interesting:
"Error while reading metadata nodes in table itypars of assembly Fable.SignalR: the node has no matching declaration. Signal this warning, this assembly might has to be rebuilt"

Hi @Slesa,
I'm not sure what the cause of your compile error is, the Http module is in the Fable.SignalR package. I'd recommend checking your version of the Fable compiler is up-to-date, and try cleaning out your .fable directory. Perhaps looking at the demo app might help?

There is a fable compile warning which might be interesting:
"Error while reading metadata nodes in table itypars of assembly Fable.SignalR: the node has no matching declaration. Signal this warning, this assembly might has to be rebuilt"

This isn't an issue, you can safely ignore that warning.


fix(wizard): update ux styles for wizard

remove various title tooltips, update styles for consistency and clean up error messages

[test]

[test]

@mindreeper2420 snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1600-fabric8-ui.int.rdu2c.fabric8.io


Add remaining SCE sections to merged object docs

Closes #240
This PR adds the remaining SCE sections to merged_objects.md, including reducedDims, ADT considerations, and cellhash considerations. There are a couple things to be aware of:

I have 1 TODO for a relative link that is pending https://github.com/AlexsLemonade/scpca-docs/issues/221. Let me know if you think we should indeed keep & circle back to this, or get rid of it altogether
The specific cellhash columns are probably correct, but pending changes in this (Draft, at the time of writing) PR - https://github.com/AlexsLemonade/scpca-nf/pull/666 .

I also went back through the docs that were already written to replace some library --> sample wording.

I added language in https://github.com/AlexsLemonade/scpca-docs/pull/251/commits/4cba76821603e642e571ad69037a1f093b131fd0 to emphasize that statistics were calculated on individual libraries before merging. Any wording feedback?


Generate config for wskadmin scala tooling

New Scala based wskadmin tooling (#3722) needs configuration to access the DB details.
So for python based wskadmin script used to source this values from ${OPENWHISK_HOME}/whisk.properties which is generated via whisks.properties.j2 template via ansible task. For scala tooling we should also generate a similar config based on a jinja template and have it copied to a std location. Then new tooling can use that location per default simplifying the usage

Fixed via #3817


Add omitempty to EnvironmentConfiguration.env

This PR adds omitempty to the EnvironmentConfiguration.Env field for the Environment CR. Env being a required field was raised on slack by @skabashnyuk, and I see no reason for it to be required?
CC @maysunfaisal @jgwest for awareness, in case there's a reason it's required.

@johnmcollier Just one request, Can you link the Slack convo if possible, so that we know the reason when we return to the PR down the line?

Let's merge it, guys. We need that on stg better sooner than later.


Navbar - Messages route ----> icon button

This PR is one step of the Navbar refactor
This PR:

Moves Messages from the left routes section, to the right icons/menu section.
Changes Messages from a tab to an icon button similar to Notifications.
Fixes the Navbar width issue when on Documents - Tim pointed out in Issue #300.
Fixes the Navbar layout issue - KaHung pointed out in Issue #324.
Removes the mobile only "Navigate To..." dropdown - just leaves the Tabs now that Messages is gone (and Documents will be going away too.)

The three files this PR effects:
NavBar.jsx
NavbarLinks.jsx
NavMenu.jsx

https://github.com/codeforpdx/PASS/assets/71395931/efad5cd1-c02d-4585-9aa2-b7913a790e1e

There is a /test/Navbar folder with a test for OIDClogin component already. Should I make one Navbar test file and move that one into it (along with others I create)? Or a different file for each thing?
Ideas to test so far:

make sure Navbar renders each item of navbar (logo, routes, icons/menu)
make sure it renders the icons on right (larger than mobile view)
make sure it renders hamburger menu icon instead of the icons on right (mobile view)

@timbot1789

Yeah they can go next to the OIDC tests in the navbar folder. And those tests sould good. I'd also add:

When not logged in, we get the login component in the nav bar


Python

What is this issue even about?
And is this project still being maintained, contributed too? It doesn't seem like it if an issue like this is still open 2.5 months later.


test_response_to_triple_ack local test failure

Repeatable test failure (3 of 3 attempts on a fresh checkout of master at c00e99c6b
CI appears green at the commit, so I'm guessing it's some kind of configuration issue. I just added here for tracking (and also a reminder to add a README section about running the local test).
Captured output:
$ TRIN_INFURA_PROJECT_ID=1 RUST_LOG=debug cargo test -p trin-core
    Finished test [unoptimized + debuginfo] target(s) in 0.17s
     Running unittests (target/debug/deps/trin_core-19ec7be1499b1d43)

running 127 tests
test cli::test::test_custom_discovery_port ... ok
✂ ✂ ✂
test portalnet::types::content_key::test::test_distance_to_key ... ok
[2022-03-31T22:40:03Z DEBUG trin_core::portalnet::storage] Setting up RocksDB at path: "/tmp/trin.8RiPOGjQik5o/rocksdb"
[2022-03-31T22:40:03Z DEBUG trin_core::portalnet::storage] Setting up SqliteDB at path: "/tmp/trin.8RiPOGjQik5o/trin.sqlite"
[2022-03-31T22:40:03Z DEBUG discv5::discv5] Discv5 service already shutdown
[2022-03-31T22:40:03Z DEBUG discv5::discv5] Discv5 service already shutdown
test utp::stream::tests::test_response_to_triple_ack ... FAILED
test portalnet::types::content_key::test::test_should_store ... ok
[2022-03-31T22:40:03Z INFO  trin_core::portalnet::discovery] Starting discv5 with local enr encoded=enr:-IS4QJIjtY6mIYt5tnOIa89gnMsZu76Bd5Bg45gMaUKJW-RoAHq6nMg0r4G4APtTTduRlnekuWgp9QKyZKnIt78zBfsBgmlkgnY0gmlwhMCoAcCJc2VjcDI1NmsxoQJ9TpUJQOxBIiqmFdLWSXi81LLm9HrWkpwH_JTIaQuhU4N1ZHCCEJI decoded=ENR: NodeId: 0x5b61..bb5f, Socket: Some(<IP_ADDRESS>:4242)
[2022-03-31T22:40:03Z DEBUG trin_core::portalnet::storage] Setting up RocksDB at path: "/tmp/trin.8RiPOGjQik5o/rocksdb"
[2022-03-31T22:40:03Z DEBUG trin_core::portalnet::storage] Setting up SqliteDB at path: "/tmp/trin.8RiPOGjQik5o/trin.sqlite"
[2022-03-31T22:40:03Z DEBUG trin_core::portalnet::overlay_service] [History] Sending Ping request to node. Node: 0x08d3..fb5a
[2022-03-31T22:40:03Z DEBUG discv5::discv5] Service is already shutdown
test portalnet::overlay_service::tests::ping_node ... ok

failures:

---- utp::stream::tests::test_response_to_triple_ack stdout ----
thread 'utp::stream::tests::test_response_to_triple_ack' panicked at 'called `Result::unwrap()` on an `Err` value: Discv5 socket timeout', trin-core/src/utp/stream.rs:1623:37

failures:
    utp::stream::tests::test_response_to_triple_ack

test result: FAILED. 126 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 1.24s

Hmm, this is weird. I just tested it on a brand new laptop under Ubuntu and it pass all tests on multiple runs without any additional configurations.
@carver could you check please if you get the same issue with https://github.com/ethereum/trin/pull/283?

Suddenly, I also started to experience the same issue on my laptop, so I dig a little bit more.
On my side, the problem was related to discv5 packet timeout because of this warning:
Apr 05 16:58:48.782  WARN discv5::handler: Session has invalid ENR. Enr socket: Some(<IP_ADDRESS>:9601), Node: 0xe41d..8629, addr: <IP_ADDRESS>:9601

As we can see, there is some discrepancy between the Enr socket from the packet and the Node addr .
It looks like the issue is related to network interfaces, when we have more than one up and running interface, as this example:
ogenev:eth/ $ ifconfig -a
enx349971e17709: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
         .....

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>
        ....

wlo1: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        ...

This is because my machine was simultaneously running ethernet and wireless interfaces and when we pass --internal-ip flag
we always assign the first active interface IP address to discv5 but it seems discv5 is expecting the node address to be my wlan interface:
https://github.com/ethereum/trin/blob/fb5c3ed3542d0cda9aa0eb3a8f269be1ba0b3251/trin-core/src/socket.rs#L47
Not sure how we should deal with this, for now, for an appropriate fix, probably we should dig more into discv5 code to see how exactly the ENR socket address is created when sending auth challenge to a remote node
if we close one of the interfaces, there are no issues.

Weird, I thought the plan was that ENRs make a best effort to know their own IP, but that we don't reject a node for having a mismatch IP (for example, if they're behind a NAT).
For reference, here is where discv5 is enforcing that rule:
https://github.com/sigp/discv5/blob/506c9afe6a037dc3ba920f691068dac29f6cba7c/src/handler/mod.rs#L769
Interestingly, the test failures are random, as of e876d5b77b4. It also includes the tests trin-core/tests/overlay and test_duplicate_packet_handling:
thread 'overlay' panicked at 'Unable to respond to ping: The request timed out', trin-core/tests/overlay.rs:156:21

and
---- utp::stream::tests::test_duplicate_packet_handling stdout ----
thread 'utp::stream::tests::test_duplicate_packet_handling' panicked at 'assertion failed: `(left == right)`
  left: `SynSent`,
 right: `Connected`', trin-core/src/utp/stream.rs:1786:9

I'll keep poking around...

This seems most easily reproducible in trin-core/tests/overlay.rs, probably because it launches 3 nodes, so there are more opportunities for the IP to shift.
To add to @ogenev 's point: we always pick the "first" NIC, but the order that NICs are returned is unstable, even within the same test. So if the list of non-loopback NICs is longer than 1, then we launch with a random IP. In the overlay test above, there's a 75% chance that one of the 3 nodes has a different IP from the others. (This failure rate is observed roughly in my local runs)
My understanding here is that the problem is that when we send the data, we don't define an IP address. I believe the OS chooses that for us. So if we are sending from one local IP to another, the OS will (helpfully) choose to send from the same IP as the recipient IP. But then it won't match the chosen IP for the ENR.
But... whatever the reason, I believe that the data coming from a different IP shouldn't blow up the connection. If a peer misidentifies its IP in the ENR, we should just reply back with the IP we see and carry on, right? I thought that was in place to accommodate NAT traversal issues.
So we could go about trying to line up the IPs, but I'm currently more interested in adding resilience when the sending IP doesn't match the ENR.

r, I believe that the data coming from a different IP shouldn't blow up the connection. If a peer misidentifies its IP in the ENR, we should just reply back with the IP we see and carry on, right? I thought that was in place to accommodate NAT traversal issues.

If I'm wrong about this ^ then there are other paths to resolving this. Some options:

Currently, the sending IP is set with the same IP as the listen IP. If discv5 is meant to be strict about ENR IPs, then it would be reasonable to change discv5 to listen on <IP_ADDRESS>, but send with the same IP as the ENR.
If that becomes a black hole, we could just configure the node to both listen and send from the ENR IP.

So I've been able to verify locally that this patch prevents the overlay error across 5 consecutive runs:
diff --git a/src/handler/mod.rs b/src/handler/mod.rs
index 2aebd31..5a7769a 100644
--- a/src/handler/mod.rs
+++ b/src/handler/mod.rs
@@ -750,10 +750,8 @@ impl Handler {
     /// Verifies a Node ENR to it's observed address. If it fails, any associated session is also
     /// considered failed. If it succeeds, we notify the application.
     fn verify_enr(&self, enr: &Enr, node_address: &NodeAddress) -> bool {
-        // If the ENR does not match the observed IP addresses, we consider the Session
-        // failed.
+        // Verify the node public key (address) matches
         enr.node_id() == node_address.node_id
-            && (enr.udp_socket().is_none() || enr.udp_socket() == Some(node_address.socket_addr))
     }

     /// Handle a message that contains an authentication header.

This is almost certainly too lax, since it would add the ENR to the routing tables, and propagate it to other peers, even if the IP is unreachable. But it's nice to know that the change is sufficient.

To add to @ogenev 's point: we always pick the "first" NIC, but the order that NICs are returned is unstable, even within the same test. So if the list of non-loopback NICs is longer than 1, then we launch with a random IP. In the overlay test above, there's a 75% chance that one of the 3 nodes has a different IP from >=1 of the others. (This failure rate is observed roughly in my local runs

Is there a way to make that returned NICs stable? I think when the OS decides which NIC to use, it is looking at the IP routing table:
$ route -n                                                                                [14:51:19]
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>   <IP_ADDRESS>         UG    100    0        0 enx349971e17709
<IP_ADDRESS>         <IP_ADDRESS>   <IP_ADDRESS>         UG    600    0        0 wlo1
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS>     U     1000   0        0 wlo1
<IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS>   U     100    0        0 enx349971e17709
<IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS>   U     600    0        0 wlo1

When we have two interfaces pointing to the default route (<IP_ADDRESS>) as above, I assume it is always choosing the NIC with the lowest "Metric" value. So maybe if we can access this table in rust, get the interface with the lowest metric, and then lookup the NIC IP it should make this more stable?

Is there a way to make that returned NICs stable?

Yeah, this does seem possible, but I'm not sure it's worth prioritizing. As soon as we modify discv5 to accept connections from mismatched IPs, the NIC instability will be moot. It's actually kind of nice that the instability highlighted the bigger problem that discv5 is too strict.

So I believe I have the main thrust of the change in this PR: https://github.com/sigp/discv5/pull/109 . We'll see what sigp thinks...

Reopening as this failure surfaced again recently (noted in #377).

closing since this bug hasn't been reported in over a year


[Community] Abstract for GHC CFP - Community Presentation on Theme Builder, and Open Source Day project

Problem/Concern
The Call for Presentations/Projects is open for the Grace Hopper Celebration in October 2024!
Text from the announcement is below:
🌟 Save the Date! 🎉 GHC 24 is coming to Philadelphia, PA + Virtual on Oct 8-11, 2024, with Open Source Day kicking us off on Friday, Oct 4!
Get ready to experience insightful talks, cutting-edge workshops, and opportunities to connect with industry leaders and peers from around the globe!
📢 Our Call for Participation (CFP) is now open! Share your knowledge, experiences, and insights with a vibrant community of women and non-binary technologists. CFP
🏆 Plus, nominations for the prestigious Abie Awards are also underway. Recognize and celebrate the achievements of trailblazing women and non-binary people in technology. Abie Awards
Join us in the vibrant city of Philadelphia or virtually for an inspiring four-day event filled with incredible learning, networking, and career-building opportunities.
Proposed Solution
We can propose a speaking session or panel session on Theme Builder, and also propose Theme Builder as an Open Source Day project along with a Collaboration Summit (in person/hybrid) at GHC.
In our community calls we will comment on this issue and track ideas and decisions, closing the issue once abstracts are submitted-
@lwnoble

Hoping to submit the talk from OpenUK to GHC, and also submit the A11y Theme Builder Product to Open Source Day as a featured project - @lwnoble the CFP is open but we have some time. Can talk about this at OpenUK-

I want to help with this

@lwnoble will submit ghc 24 and @codyzu will submit for open source days.


Missing classes

I'm trying to use jReddit on my project, but when I add it via gradle into Intellij Idea there's a lot of classes missing. For example, the whole folders "comment" and "exception" are missing. The same thing when I download the jar from the maven repository. Am I doing something wrong?

closing this as duplicate of #102.


ACA service console log should reflect its enablement

Description
The membersrvc console log outputs the following statements of the ACA service regardless of whether it's enabled or not (as per aca.enabled settings):
INFO: 2016/08/01 18:32:11 Staring ACA services...
INFO: 2016/08/01 18:32:11 ACA PUBLIC gRPC API server started
INFO: 2016/08/01 18:32:11 ACA services started
should print a different message if the service is not enabled to avoid confusion
@diegomasini

@srderson can you assign me this issue?

@andresgaragiola I lost the ability to assign/close issues with the move to Gerrit. Maybe @christo4ferris still has this power, but he is on vacation until next week.

@andresgaragiola, same here.
But it's totally fine to just submit a Gerrit patch, referring to this issue.

@andresgaragiola that was fast!
@jimthematrix Andres' changes are merged to master. Please can you confirm that this is resolved and we'll close?

@jimthematrix The log correctly reflected that the ACA started. @andresgaragiola's fix was to actually not start the ACA when it was not needed. I think we can safely close this issue.

verified.

Thank you.


feat(iam): Check inline policies in IAM Users, Groups & Roles for admin priv's

First commit to proof code that resolves issue 2589 to check inline policies for admin privs.  Next steps include adding the new check to the compliance frameworks.
Context
See Prowler issue 2589.
Description
In prowler/providers/aws/services/iam/iam_service.py, updates made to update __list_inline_user_policies__ + 2 additional similar operations created for group and role inline policies + updated their models.
Also reordered the self.__list_inline_user_policies__ (and the 2 additional for groups and roles) to go after self.__list_policies* because of error:
[Module: iam_service]     ERROR: us-east-1 -- AttributeError[linenumber]: 'IAM' object has no attribute 'policies'
New directory and files create for new check:

prowler/providers/aws/services/iam/iam_inline_policy_no_administrative_privileges/
prowler/providers/aws/services/iam/iam_inline_policy_no_administrative_privileges/init.py
prowler/providers/aws/services/iam/iam_inline_policy_no_administrative_privileges/iam_inline_policy_no_administrative_privileges.metadata.json
prowler/providers/aws/services/iam/iam_inline_policy_no_administrative_privileges/iam_inline_policy_no_administrative_privileges.py

License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Something is failing in the IAM service and could be related with this moto issue https://github.com/getmoto/moto/issues/6712


Create a Docker image for ontop-cli

Complementary to the ontop-endpoint.
This image would focus on running Ontop commands, such as materialization and mapping conversion.
It could follow the style of https://github.com/stain/jena-docker, which allows to run the riot command.
See for example https://github.com/ontopic-vkg/MusicBrainz-R2RML#merging-the-mapping-files .

Addressed by #532 . Closing this issue as redundant.


Returning the latest resource from the Delete() call

Issue #, if available:
aws-controllers-k8s/community#836
Description of changes:
This PR is to related to https://github.com/aws-controllers-k8s/runtime/pull/21
It makes the code-generator changes for delete api as discussed under: https://github.com/aws-controllers-k8s/runtime/pull/21#discussion_r655729649
Testing:

make test passed.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Updated code-gen go.mod file to refer runtime 0.5.0

/lgtm

@kumargauravsharma @jaypipes we also need to set the version to 0.5.0 in the Makefile https://github.com/aws-controllers-k8s/code-generator/blob/main/Makefile#L9

@A-Hilaly thanks. I have updated the Makefile.

/lgtm


Data table: Scrolling truncation and text wrapping

Data table extension: Scrolling
Brief description
This issue covers the truncation and text wrapping behaviors of the data table scrolling pattern. This issue is broken out from #1977
Details
Link to design
Our current implementation of truncated text uses the browser default title attribute, but guidelines suggest we should use Carbon tooltip.
We’ll also need to add scenarios for wrapping text to two lines when space is narrow.

Tooltip on truncation
WKC table uses browser tooltips on truncation:

PAL uses Carbon tooltips:

Text should be truncated optically, not in the underlying code (technical truncation). This ensures that data is preserved and accessible to screen readers. On hover, truncation should display a Carbon tooltip (Carbon tooltips afford more control over delay and placement).

Links to other materials
e.g. sketch files, icons, images, prototypes
Design Maintainer(s)

Marion Brülls
Johanna Koval

Tasks
Before starting work on this epic, please review and complete the following.

[ ] Initial review of design/existing code

Working in Carbon for IBM Products package

[ ] Have you reviewed our
contribution guidelines?
[ ] Have you noted our
code guidelines?
[ ] Did you use the CLI to scaffold your new component?

@marion-bruells When we were working on our v11 migration, @matthewgallo saw that tooltips are now meant more for interactive child elements (e.g. a button). For consistency between v10 and v11, we were thinking it may be better to stick the browser default title attribute/tooltip in this case.
https://github.com/carbon-design-system/carbon/blob/main/docs/migration/v11.md#tooltip

@elycheea Agree, I think that makes sense to use a browser tooltip here.


ABAB designs

Please leave suggestions for single-case or small-n studies using an ABAB design. These could also include variants, such as with repeated reversals (e.g., ABABAB), the BABA design, and multiple-treatment reversal designs (e.g., ABAC).

Consider Smith & Ward (2006; https://doi.org/10.1901/jaba.2006.5-06), Behavioral interventions to improve performance in collegiate football. Here's the abstract:

Using a multitreatment withdrawal design, this study evaluated the differential effects of publicly posted plus verbal feedback, goal setting plus verbal feedback, and publicly posted feedback, verbal feedback, and goal setting together on the performance of 3 collegiate football players in practice scrimmages. Also assessed was whether the changes in practice behavior generalized to games. The dependent variables were performances on three wide receiver skills. The results show that public posting with verbal feedback, goal setting, and public posting with verbal feedback and goal setting were effective in improving player performance to a 90% criterion level during practice, and these changes generalized to game performance.

This is an n = 3 ABACABC multitreatment withdrawal design, where the last phase is BC, which combined elements of B and C. The criterion is a percent correct out of 10. However, putting the cap at 10 was arbitrary and could lead into a discussion of different n across participants when using the binomial likelihood. Also, this article was used in Kate's class.

Consider Greene et al (1981; https://doi.org/10.1901/jaba.1981.14-177), An analysis and reduction of disruptive behavior on school buses. Here's the abstract:

Thousands of children are injured or killed each year in school bus accidents. A significant number of these tragic incidents is precipitated by disruptive child behavior that distracts the drivers from their difficult task. Two experiments were conducted which addressed this problem. For both experiments an automated sound recording device (referred to as a Noise Guard) selectively responsive to frequencies above 500 Hz (i.e., unresponsive to bus drone) recorded both the duration and frequency of noise outbursts above a tolerable threshold. Additionally, an observer made in situ measurements of other disruptions including roughhousing and getting-out-of-seat. In the first experiment, following baseline measurements of these behaviors, middle-school students received feedback for noise outbursts. That is, when “Noise Guard'” was activated, it in turn operated one of several lights on a panel visible to all passengers. Each day students were allowed to listen to high-appeal taped music while riding the bus and to participate in a raffle for prizes, provided the number of outbursts on the preceding day remained below a specified criterion indicated on the light panel. This intervention resulted in drastic reductions of noise outbursts with a concomitant reduction in other disruptive behaviors. Comparable results were obtained in the second experiment which eliminated the raffle from the intervention.

This is an n = 2 multiple baseline ABABC design with multiple outcomes. For simplicity, you might just focus on the two outcomes for "Bus #70" presented in figure 1. They would give a nice example of single-case models using the count regression and response-duration regression (e.g., gamma, lognormal). This article was used in Kate's class (file path: /BA Seminar/3. Behavior Analysis & Schools/Greene, Bailey, & Barber, 1981.pdf).

Consider Lambert et al (2006; https://doi.org/10.1177/10983007060080020701), Effects of response cards on disruptive behavior and academic responding during math lessons by fourth-grade urban students. Here's the abstract:

The authors evaluated the effects of response cards on the disruptive behavior and academic responding of students in two urban fourth-grade classrooms. Two conditions, single-student responding and write-on response cards, were alternated in an ABAB design. During single-student responding, the teacher called on one student who had raised his or her hand to answer the question. During the response-card condition, each student was provided with a white laminated board on which he or she could write a response to every question posed by the teacher. Nine students were targeted for data collection because of their history of disciplinary issues in school and frequent disruptive behavior in the classroom. Data revealed substantial reductions in disruptive behavior and increases in academic responding during the response card condition compared to single-student responding. The findings are discussed in terms of the beneficial effects of direct, high-response strategies for urban, low-achieving learners.

This is an n = 9 ABAB reversal design with a bounded count outcome. Students are nested in two classes, 4 in one and 5 in the other, which makes for a nice complication. There's an additional complication that the initial A1 phase has very minor between-student differences, which affords a discussion of whether to keep the full random structure. There's also the issue that many other studies have used these data to compare analytic strategies (e.g., Manolov & Ferron 2020; Michiels & Onghena, 2019; Moeyaert et al, 2014; Peng & Chen, 2015; Shadish et al, 2014).

Consider Thorne & Kamps (2008; https://doi.org/10.1007/BF03391723), The effects of a group contingency intervention on academic engagement and problem behavior of at-risk students. Here's the abstract:

A successful learning environment can be characterized by actively engaged students displaying appropriate student behavior. We implemented a group contingency intervention as a novel component to a school-wide behavior management system to decrease the frequency of inappropriate behaviors and, conversely, increase the academic engagement of students in four elementary school classrooms. Twelve students with behavioral risks served as target students to monitor effects. A reversal design was implemented to evaluate behaviors across experimental conditions. Results indicated that the frequency of inappropriate behaviors decreased and academic engaged time increased for all 12 participants. These results suggested that the group contingency was an effective class-wide intervention. Implications for research and practice are discussed.

This is an n = 12 ABAB reversal design with two primary outcomes, a % on-task behavior (perhaps use ZIOB) and a rate (# behaviors within a 15-minute time frame, which could be transformed back to a count and modeled as Poisson or NB). Students are clustered into 4 classroom settings, making for a 3-level design. The data are also discussed in Swan & Pustejovsky (2018; https://doi.org/10.1080/00273171.2018.1466681).


[Module Proposal]: avm-res-network-dnsresolver (res) (bicep)

Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals

Check this module doesn't already exist in another project that is being evolved into AVM

[X] I have checked for that this module doesn't already exist in CARML or TFVM

Terraform or Bicep?
Bicep
Module Classification?
Resource Module
Module Name
avm-res-network-dnsresolver
Module Details
Migration of Module from CARML Library to BRM Repo for AVM
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username
ChrisSidebotham

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @ChrisSidebotham,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
We just want to ask you to double check a few important things before you start the development.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - to name your module
TelemetryIdPrefix - to be used in your module's telemetry

For your module's repository:

Repository name and folder path are defined in RepoURL
Create the GitHub teams defined in the ModuleOwnersGHTeam and ModuleContributorsGHTeam columns and grant them permissions as described here.

Grant the right level of permissions for  the AVM core team and PG teams on your GitHub repo as described here.
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment, once your module is ready to be published! 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team

NOTE:

This Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry. Do NOT close the issue before the successful publication is confirmed!
Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.

https://github.com/Azure/bicep-registry-modules/pull/574 < Module merged successfully


Catch errors in forAwaitEach

Coverage remained the same at 100.0% when pulling baed22bcb9a4a0666ef92b7050536f7d2e5eeea0 on NeoPhi:master into e6d5e9dc1aafe8d4712f22fb34d7781612e77127 on leebyron:master.

Thank you for adding additional tests!

Coverage remained the same at 100.0% when pulling 609ccd686252f2fb9cf7f9fc9963edbb361a4f77 on NeoPhi:master into 325e8ff9e7cd966156688d2012b16f3a588b886a on leebyron:master.


Unusable 'prepare -h ansible' in tmt minimal install

tmt doesn't require 'ansible' in minimal install, making this prepare step (and all features?) unusable.
# dnf install -y tmt
# tmt --feeling-safe  run prepare -h ansible --playbook playbook.yaml provision -h local
...
        prepare task #2: default-0 on default-0
        how: ansible
        playbook: playbook.yaml
        fail: Command 'sudo -E ansible-playbook -c local -i localhost, /playbook.yaml' returned 1.
...
        # stderr (1/1 lines)
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        sudo: ansible-playbook: command not found
        # ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I think this is about catching the error and printing "install tmt+provision-ansible" message, as is being done for other provisioning plugins.

@martinhoyer except we don't have such subpackage, nor 'tmt+prepare-ansible'.
The /usr/lib/python3.13/site-packages/tmt/steps/prepare/ansible.py is part of tmt package itself.

@martinhoyer except we don't have such subpackage, nor 'tmt+prepare-ansible'. The /usr/lib/python3.13/site-packages/tmt/steps/prepare/ansible.py is part of tmt package itself.

Aha, I see it's being pulled by provision-container and/or provision-virtual.


Add a way to specify naming scheme

Related to #220
Please let me know if this make sense.
Hopefully, there won't be a linting issue this time ;)

Arf, I didn't had a notification for this. Sorry for the delay, I'll review it today !
Thanks for the PR ☺

Sorry about the delay, it's been a rough week on my side.
It's good to go for me 😃

Yes, I'll close it ! Thanks for pointing it out :)


update to TF v2.6.0

Related to fedn/issue-245
Compute package updated with corresponding predict function supported in tensorflow 2.6.0.
Class labels prediction are obtained by the argmax on continuous predict probabilities scores.
requirements.txt is updated with strict version 2.6.0 to avoid future version conflicts.

Close issue fedn/245 once this PR is accepted

I have tried with tf >=2.3.2, see second commit

In total I have tested with version 2.3.2, 2.4.0, 2.5.0 and 2.6.0


Do we need a separate npm package for it?

Hey,
I'm a contributor of react-dropzone and just stumbled upon this repository by accident.
I'm wondering if world needs two libraries (and 2 npm packaged) for the same thing but with different syntax. Would you like to merge it back to original project and deprecate the npm package?
I was intendent to convert it to ES6 sooner or later myself so this would be a help and also less confusion for npm users I think.

Sure, I will deprecate this npm package.

Cool! Thanks for understanding.


Refactoring code

Hopefully someday coming 2d implementation is in separate file beam2d.jl.
Instead of import FEMBase: , use directly FEMBase.

Pull Request Test Coverage Report for Build 60

2 of 6 (33.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 98.473%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/beam2d.jl
0
4
0.0%

Totals

Change from base Build 57:
-0.4%

Covered Lines:
258

Relevant Lines:
262

💛  - Coveralls


Negative numbers for BInt

See #1
I don't expect you'll accept this pull request straight away. Feedback would be appreciated!

Hey @synlestidae, amazing work,
You're correct, I somehow completely overlooked the possibility of negative integers 😳 nice find.
I see you also formatted the code and written some good tests as well, which was very much needed.
The code looks good to me - although,  could you please add #[cfg(test)] for decode_str_to_i64, (maybe decode_bytes_to_i64 would be a better name for the function?), and test_decode_str helper functions as well as putting #[allow(unused_imports)] to the top of mod.rs?
Also, #![feature(inclusive_range_syntax)] can be completely removed from lib.rs now that it's not needed.
Thanks a lot for the great work, I really appreciate it! 👍

No worries, rustfmt takes care of the formatting! I've added the other changes you asked for now.

Yeah, I should use rustfmt myself.
Before I merge, could you perhaps squash the last 4 or so commits together into one, since they're all related to testing?
Thanks @synlestidae

Okay done, think it's good now

👍