import declarations may only appear at top level of a module

Versions
Nuxt 3.13.2
nuxt-vite-legacy 1.3.1
node v18.19.1
Trying to generate legacy bundles for Firefox v60 but I am getting
import declarations may only appear at top level of a module 460cJYcf.js
This was the line it highlighted when i went into more details on the console
function NR(){import.meta.url,import("_").catch(()=>1),async function*(){}().next()}/**
Is there some polyfill that i need to specify or some additional config i missed?
I checked on caniuse.com and import.meta is not supported by Firefox v60
This is my nuxt.config.js config for the library
legacy: {     targets: ["defaults", "firefox >= 60"]   },

This is a documented caveat, see "Legacy browsers that had just begin to support modules" in README.
You can try modernPolyfills:
{
  legacy: {
    targets: ["defaults", "firefox >= 60"],
    modernPolyfills: true, // <--- this
  },
}

but I haven't tested that.
Vite itself does much more than that in the upstream, including something about import.meta.url: https://github.com/vitejs/vite/blob/f65d6ae5a762708d2412da37fb4efc0256f1b535/packages/plugin-legacy/src/snippets.ts and I suspect it must be for a reason.


Update jackal to v3.0.0

Proposal: https://ping.pub/jackal/gov/9
Upgrade planned at block https://ping.pub/jackal/block/4074200
Estimated Time: 24/08/2023, 00:19:27

ghcr.io/akash-network/cosmos-omnibus:v0.3.45-rc4-jackal-v3.0.0

Resolves #613


Error while deploying to Fermyon Cloud "The key value service is temporarily unavailable"

Hey 👋🏻
when I try to deploy a Spin application to Fermyon Cloud using spin deploy, I receive the following error message:
Uploading sample-for-the-issue version 0.1.0+r412cba...
Deploying...
Error: Unable to create app

Caused by:
    The key value service is temporarily unavailable.

Learn more at https://developer.fermyon.com/cloud/faq

For reference see the actual spin.toml:
spin_manifest_version = "1"
authors = ["Thorsten Hans<EMAIL_ADDRESS>description = ""
name = "sample-for-the-issue"
trigger = { type = "http", base = "/" }
version = "0.1.0"

[[component]]
id = "sample-for-the-issue"
source = "target/wasm32-wasi/release/sample_for_the_issue.wasm"
allowed_http_hosts = ["https://api.foo.com"]
[component.config]
some_config = "some_value"
[component.trigger]
route = "/bar"
[component.build]
command = "cargo build --target wasm32-wasi --release"
watch = ["src/**/*.rs", "Cargo.toml"]

Subsequent Deployment attempts
When I try to deploy the same app again using spin deploy, I receive the following output
Uploading sample-for-the-issue version 0.1.0+r5b15b3b0...
Deploying...
Error: No channel with app_id b65b8241-dade-44b5-a9b9-a5aa47cc5007 and name spin-deploy

Learn more at https://developer.fermyon.com/cloud/faq

Spin CLI Version
I tested with spin 1.2.1 and spin 1.1.0

@ThorstenHans Thanks for reporting this - I think Spin is surfacing an error from Fermyon Cloud, so I will transfer the issue to the feedback repo which is where we track Cloud issues.


added best practice for crate setup on ec2

i'm not sure if the conclusion is really useful, but it's ok to let it there.
otherwise ack

@bogensberger agree, I removed the "conclusion"


refactor memory tracking

refactor memory for tracing
track memory for tasks
Description

Testing Instructions

I think the thread local nature of with_local_counter is going to cause problems if/when work stealing happens.

It reads the allocation counters before and after every poll, and only keeps the diff. So that should be fine.

added not send for allocation counters


payment went through but invoice not show up after payment

I did my first test charge and this showed up

but I don't see it on my list (the one in pic is a old transaction)

logs

https://api.stripe.com/v1/Webhook - invalid_signature
No signatures found matching the expected signature for payload

oh shit I put wrong webhook secret

Issue solved after adding right webhook secret


CLI issue

$ python pummel featurize --seed 1 --skip-feats POWSP regions
usage: pummel featurize [-h] [--chunksize LINES] [--skip-rbf]
[--n-freqs N_FREQS] [--bandwidth BANDWIDTH]
[--rff-orthogonal | --rff-normal] [--seed SEED]
[--skip-feats FEAT_NAME [FEAT_NAME ...]]
dir [outfile]
pummel featurize: error: too few arguments

This works:
$ python pummel featurize --seed 1 regions

As does this:
$ python pummel featurize --skip-feats POWSP --seed 1 regions

It's a slightly unintuitive behavior of argparse; it's trying to interpret regions as another argument to --skip-feats. You can do it with
python pummel featurize --seed 1 --skip-feats POWSP -- regions.


Framework 2.0 for CLR

I would like to write a stored procedure using Tweetinvi. I understand to do this it would need to be a CLR SP, correct?
And to do that, CLR still uses Framework 2.0, at least on MSSQL 2012 and it's  "kind of a thing" if you know what I'm talking about unless it's changed.
But I cannot install Tweetinvi if I'm setup for Framework 2.0 in my CLR app inside of Visual Studio.
Can someone (who specifically has CLR experience, not just winging stuff at the wall) answer me? Thank you for your help in advance.

Hi,
We use SQL Server 2008 R2, which is limited to CLRv2 so I haven't tried this, but I was under the impression that SQL Server 2012+ supported CLRv4.
Have you tried building something simple targeting .NET4+ and installing it?
Cheers,
Josh

Woops, we are running that too.
So question: Can I do CLR with it needing .NET v2?

OR.... what is the best way to integrate Tweetinvi into MSSQL server so I can use it in TSQL? Thanks.

.NET 3.5 was the last version to target CLRv2, so any code you want to run on SQL Server pre-2012 will need to target .NET 3.5 (or lower).
So there's 3 options:
1:
I don't know how much work it would take for Tweetinvi to be compiled against .NET 3.5. To find out, just download the source code & change each project to target .NET 3.5. Then you can try and compile it & count the errors!
2:
Realistically though, I'd ask yourself why whatever it is you want to do belongs in SQL Server in the first place. You can store data in SQL Server 2008 R2 and process it in a separate application tier (which could be .NET 4.6, or whatever you wanted). There are overheads in getting the data out of SQL Server & then back in again after processing it, but unless you have billions of rows to process this shouldn't be too much of an issue (and even then it's generally better to move processing out of SQL Server due to licencing costs).
3: Buy SQL Server 2012+. If the deadline's tomorrow, you'd better have deep pockets! JK... This would be silly :p If you're buying it anyway in a couple of months (staying nicely ahead of 2008 R2's EOL) then you could just bump up the upgrade schedule?

?? No I want to do something simple: Have the database think for me like any other application. A stored procedure should be able to interact on its own, pulling information. Can I do that in 2008 R2 running now  .NET 4.6?

.NET 3.5 was the last version to target CLRv2, so any code you want to run on SQL Server pre-2012 will need to target .NET 3.5 (or lower).

Does that answer your question?

I saw that, I was just wondering if Tweetinvi would somehow work on 2008 R2. It's ok though as I got that 2.0 DLL (system.web32 I think?) to compile. I just  tested and it works so I can use another library I have. Thanks.

Yes, it's definatelty possible to run Tweetinvi on SQL Server 2008 R2, although not without changes to the Tweetinvi codebase. It's the first option I mentioned in the earlier post :)

Thanks.

Hi just as an addition to this thread, Tweetinvi is targeting .NET 4.0 and above.
I had multiple requests to support more versions but based on the quantity of target that library supports (PCL, .NET Core, W10, Xamarin...) and the important changes that it requires, Tweetinvi will not support .NET 3.5.


net: mgmt: Protect processing of iface data

Use lock while accessing iface data.
One of the use case is while setting regulatory, it access iface info, while from some other place also it can be accessed. Protected the data processing.

Use lock while accessing iface data. One of the use case is while setting regulatory, it access iface info, while from some other place also it can be accessed. Protected the data processing.

I'm not entirely sure what you're trying to fix exactly? The pointer value is passed to the function and will not change while looping. I think the lock does not help, unless I'm missing something.
Can you explain why this is needed?

Use lock while accessing iface data. One of the use case is while setting regulatory, it access iface info, while from some other place also it can be accessed. Protected the data processing.

I'm not entirely sure what you're trying to fix exactly? The pointer value is passed to the function and will not change while looping. I think the lock does not help, unless I'm missing something.
Can you explain why this is needed?

The issue here is linked to

Use lock while accessing iface data. One of the use case is while setting regulatory, it access iface info, while from some other place also it can be accessed. Protected the data processing.

I'm not entirely sure what you're trying to fix exactly? The pointer value is passed to the function and will not change while looping. I think the lock does not help, unless I'm missing something.
Can you explain why this is needed?

Hi,
Thanks for info.. I will update the locking mechanism.

Minor comment: please update the commit subject to use net: l2: wifi: prefix instead.

Minor comment: please update the commit subject to use net: l2: wifi: prefix instead.

Even the commit log refers to iface data, it should be updated to nm


Garbage collection/memory optimization

This deletes unused references in parallelism/dynamism. That said, the python garbage collector occasionally has to be told what to do, so the user should be aware.
Fixes #373
[Short description explaining the high-level reason for the pull request]
Changes
How I tested this
Notes
Checklist

[ ] PR has an informative and human-readable title (this will be pulled into the release notes)
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.

This works nicely. See this script:
import numpy as np
import pandas as pd

from hamilton import driver
from hamilton.ad_hoc_utils import create_temporary_module
from hamilton.function_modifiers import parameterize, source

NUM_ITERS = 100

import pandas as pd
import numpy as np

def foo_0(memory_size: int = 100_000_000) -> pd.DataFrame:
    """
    Generates a large DataFrame with memory size close to the specified memory_size_gb.

    Parameters:
    memory_size_gb (float): Desired memory size of the DataFrame in GB. Default is 1 GB.

    Returns:
    pd.DataFrame: Generated DataFrame with approximate memory usage of memory_size_gb.
    """
    # Number of rows in the DataFrame
    num_rows = 10 ** 6

    # Calculate the number of columns required to make a DataFrame close to memory_size_gb
    # Assuming float64 type which takes 8 bytes
    bytes_per_row = 8 * num_rows
    target_bytes = memory_size
    num_cols = target_bytes // bytes_per_row

    # Create a DataFrame with random data
    data = {f'col_{i}': np.random.random(num_rows) for i in range(int(num_cols))}
    df = pd.DataFrame(data)

    # Print DataFrame info, including memory usage
    print(df.info(memory_usage='deep'))
    return df

@parameterize(
    **{f"foo_{i}" : {"foo_i_minus_one" : source(f"foo_{i-1}")}  for i in range(1,NUM_ITERS)}
)
def foo_i(foo_i_minus_one: pd.DataFrame) -> pd.DataFrame:
    print(f"foo_{i}")
    return foo_i_minus_one * 1.01

if __name__ == '__main__':
    mod = create_temporary_module(foo_i, foo_0)
    dr = driver.Builder().with_modules(mod).build()
    output = dr.execute(
        [f"foo_{NUM_ITERS-1}"],
        inputs=dict(memory_size=1_000_000_000)
    )

Using mprof, we can easily run it, and get the following plot. My computer dies without that code:

If you comment out the new code, you'll notice that things act strangely. The caveat here is that mprof doesn't record swap space. So, you see a large alocation, then things start to act quite junky (my computer's music skips 😆). Note this is 100mb increments instead of 1gb. 1gb will just give up.

Now, with the code added, it caps out at 500mb before GC -- I think this is storing a few pointers to the node + general python memory usage.  Its keeping a few extra ones around -- Hard to tell why, as there are likely more than one reference being kept around due to the way the del function works with dicts/when GC is happening:


Processing of media content stopped working

Hello!
Here's what I noticed 1-2 days ago. The media content that would normally be loaded from column "L" of GoogleSheets database for the timeline stopped being processed and as a result the pictures / videos do not show up. Also there's another strange glitch hard to explain that appeared the same time. Here's a screen capture video I recorded to explain what I mean: https://drive.google.com/file/d/1h2d_cwTqFZ2V2_DKg6AE9Qdrpq7uAi6B/view?usp=sharing. The glitch I report takes place only in case of entries with non-blank entry in column "L". If the cell is empty, there's no problem with scrolling or closing the popup.
No other timeline I know is as good as the TimeExplorer. It's simply the best. I hope the issue can be easily fixed and the project will not be abandoned. I've been building my TimeExplorere timeline for more than a year now. Even if the project is not going to be developed anymore I truly hope it will remain operative.
Thank you again for this fantastic tool :-)
Best regards
Jakub Pleksus

Hi! I've finally had an opportunity to see what's happened with the media not loading, and it looks like something has changed with the TimelineJS library I use to load the media. I'll have to implement a better fix eventually, but for now media is loading again, as I've pointed the timeline page to an older local copy of that library. Thanks for bringing this to my attention!

Thank you so much Jeremy! You made my day :-)
I appreciate very, very much 😊
Pleksus

W dniu 2020-09-14 21:07:00 użytkownik Jeremy Guillette<EMAIL_ADDRESS>napisał:

Hi! I've finally had an opportunity to see what's happened with the media not loading, and it looks like something has changed with the TimelineJS library I use to load the media. I'll have to implement a better fix eventually, but for now media is loading again, as I've pointed the timeline page to an older local copy of that library. Thanks for bringing this to my attention!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Proposal - Run E2E tests in CI

We currently have nightly E2E tests which take 4 minutes.
I was wondering if there is a particular reason why we don't run them in every CI per PR? Can't hurt I think?

Currently it takes 13-14mins as we keep adding more scalers to E2E tests, but I think that it is still feasible to run them during each PR.
But the main issue is, that the tests are runing on shared kubernetes instance. So in case there are 2 opened PRs in approx the same time, the test will interfere.

They run on every merge to master as well. We will need to only trigger them manually for external contributions since it'll be deploying code to the test cluster. However, the point @zroubalik brought up about shared cluster stands, but at least the manual triggering can be coordinated.


[WFLY-19889] Propagate the actual merge commit to cloud-tests

Followup for https://issues.redhat.com/browse/WFLY-19889 / https://github.com/wildfly/wildfly/pull/18329
Remove the push case which isn't relevant here.
Propagate the merge commit to remote CI (was accidentally using the pr head).
This has been verified via a PR in my repository. https://github.com/kabir/wildfly/pull/59
The head of the PR is 6faf72591546dc66caba1ba4252548debc77ec79
At the time of writing the main branch sha is 2ed62572194a8f09dde55aca3ca4d80571629b17
The trigger job https://github.com/kabir/wildfly/actions/runs/11575716317 outputs
PR_NUMBER=59
GITHUB_SHA_FOR_PR=db1b5fc50d3e3008feffe64acdf44a24b710fc1e
GITHUB_EVENT_PULL_REQUEST_HEAD_SHA=6faf72591546dc66caba1ba4252548debc77ec79

The workflow run job outputs
 curl -X POST -s https://api.github.com/repos/kabir/wildfly-cloud-tests/dispatches -H 'Accept: application/vnd.github.v3+json' -H 'Content-Type: application/json' -H 'Authorization: ***' -d '{"event_type": "trigger-cloud-tests-pr", "client_payload": {
  "triggerRepo": "kabir/wildfly",
  "githubSha": "db1b5fc50d3e3008feffe64acdf44a24b710fc1e",
  "prHeadSha": "6faf72591546dc66caba1ba4252548debc77ec79",
  "pr": "59"
} }'

And on the cloud tests side I see https://github.com/kabir/wildfly-cloud-tests/actions/runs/11575728467/job/32223011050
*****
{
  githubSha: db1b5fc50d3e3008feffe64acdf44a24b710fc1e,
  pr: 59,
  prHeadSha: 6faf72591546dc66caba1ba4252548debc77ec79,
  triggerRepo: kabir/wildfly
}

githubSha is what gets cloned when building wildfly https://github.com/wildfly-extras/wildfly-cloud-tests/blob/main/.github/workflows/wildfly-cloud-tests-callable.yml#L94-L102
And prHeadSha is used to communicate back about the progress (I know this works from the status on the test PR)

Thanks @kabir


Support for platform indipendent lockfile and SHA-pinning

Hi there,
thank you for the great effort put into this amazing project.
I'm asking whether you plan in the future (maybe when uv support won't no longer be experimental) to overcome the current limitations in order to being able to:

Generate a platform-indipendent lockfile
Pin the dependency version by SHA. You can already do that with pip-tools (option --generate-hashes), but apparently this is not available inside the rye CLI
Check for outdated dependencies (like you can do with pip: pip list --outdated)

Yep, this is planned. But the plumbing for this will be in uv so until then I will close this as there are already issues for it.

is there an issue on uv for this? i couldn't find one so i raised https://github.com/astral-sh/uv/issues/2679


[ISSUE] Enabling ingress over zeebe gatway service fails

Describe the issue:
Enabling the ingress over zeebe-gateway through the helm values configuration file creates an k8s ingress but on port 80. We cannot connect to the ingress from a zeebe client like zbctl. We get a connection refused error. We are using k3s as our kubernetes distribution which comes by default with traefik.
Actual behavior:
Below is the screenshot of executing the zbctl command to connect to the cluster. It gives a connection refused error.

below are the logs printed for the traefik pod
time=“2023-02-08T10:09:16Z” level=error msg=“Skipping service: no endpoints found” serviceName=dev-zeebe-gateway servicePort="&ServiceBackendPort{Name:,Number:26500,}" namespace=default providerName=kubernetes ingress=dev-zeebe-gateway
Expected behavior:
Traefik ingress should be created with port 26500 exposed and zbctl (any zeebe client) should be able to connect with the zeebe cluster without any errors.
How to reproduce:
Please update your helm values file with the below snippet for the zeebe-gateway and install the helm chart. Below is the screenshot of the services, pods and ingress created.

The issue is possibly because the zeebe-gateway ingress template does not expose the grpc port as defined here.
Logs:
Below are the logs on the traefik pod
time=“2023-02-08T10:09:16Z” level=error msg=“Skipping service: no endpoints found” serviceName=dev-zeebe-gateway servicePort="&ServiceBackendPort{Name:,Number:26500,}" namespace=default providerName=kubernetes ingress=dev-zeebe-gateway
Environment:

Platform: Local laptop
Helm CLI version: 3.10.3
Chart version: 8.1.5
Values file: below is the snippet of the zeebe-gateway from the values file

zeebe-gateway:
  ingress:
    enabled: true
    host: localhost
    className: traefik
    annotations:
      ingress.kubernetes.io/protocol: h2c
  replicas: 2
  affinity:
    podAntiAffinity: null

Thanks

Hi @shahamit
There are 2 issues here.

The TLS should be setup in the Ingress to use it with Zeebe (more details in the official docs: Zeebe Ingress (gRPC)).
The TLS port 443 should be provided either via --port or setting the whole address via --address (please check zbctl --help for more details).

@aabouzaid - sorry for the delayed response. I tried with nginx ingress controller too. I am unable to connect from zbctl client.

I do not have tls enabled. I need ingress only over the zeebe-gateway. Hence I have enabled it in the values.yaml only in the zeebe-gateway and not in the global element
I have installed nginx ingress controller through this helm command helm install nginx-ingress ingress-nginx/ingress-nginx
Ingress does get created correctly

When I try to access it through zbctl or java client I get errors. With this zbctl command zbctl status --address localhost:80 --insecure I get an error as "Error: rpc error: code = Unavailable desc = connection closed before server preface received" while I get the below log statement in the nginx pod
<IP_ADDRESS> - - [12/Oct/2023:11:56:31 +0000] "PRI * HTTP/2.0" 400 150 "-" "-" 0 0.001 [] [] - - - - 8bceac8badbe6243b31635e2305aaa25

The error indicates some problem with the ingress configuration. It is returning HTTP 400.
What could be going wrong?
Thanks.

Hi @shahamit
It's actually mentioned in the docs link above. :grinning:

Is this comment still valid? Is there a reason why GRPC over plain http is not available?

That's an Ingress Nginx implementation matter (not something from our platform side), and as far as we know, it's still the same.

Would ingress on zeebe-gateway work with traefik as the ingress controller over HTTP?

We didn't test that, so cannot confirm. But generally speaking, it should work.
The only component in our platform that requires HTTPS is Identity working in sub-context  (if works on a separate without sub-context it will work with HTTP normally).

Closing this since there have been no updates for a while.


Fastnlight: Watch Morrison[\W_]*+recovery

Fastnlight requests the watch of the watch_keyword Morrison[\W_]*+recovery. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Morrison[\W_]*+recovery has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


NumberBox Significant Figures not respected

Describe the bug
NumberBoxes don't seem to respect high number of significant figures, despite number formatter settings.
Steps to reproduce the bug

Have a NumberBox bound on its Value property to a double
Set the NumberFormatter up as follows:

            InitializeComponent();
            SignificantDigitsNumberRounder rounder = new()
            {
                SignificantDigits = 24
            };

            DecimalFormatter formatter = new()
            {
                IntegerDigits = 1,
                FractionDigits = 24,
                SignificantDigits = 24,
                NumberRounder = rounder,
            };
            MyNumberBox.NumberFormatter = formatter;

Have the bound property evaluate to a sufficiently precise number, e.g. 0.01234567890123456

Expected behavior
NumberBox will show full number.
Actual behaviour
NumberBox rounds number 0.01234567890123
Version Info

NuGet package version:
[Microsoft.WindowsAppSDK (1.0.0-experimental1)]
[Microsoft.WindowsAppSDK.Foundation (1.0.0-experimental1)]
[Microsoft.WindowsAppSDK.WinUI (1.0.0-experimental1)]

Windows app type:

UWP
Win32

No
Yes

Windows version
Saw the problem?

Insider Build (xxxxx)

May 2021 Update (19043)
Yes

October 2020 Update (19042)
Yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

Additional context

@dtynewydd do you know if this issue is exclusive to winui3?


[UI] tooltip should not fix after click.

Raise issue to Clarity team, pls refer https://github.com/vmware/clarity/issues/681

Verified, issue will be gone if upgrading Clarity to version >0.8.14

We have upgrade to 0.9.X. Let's close this issue.


Consider Making Swipe Action Wording More Consistent

Please consider changing the wording for the Read/Unread swipe action from Mark as Read/Unread to Read/Unread.
This would make it consistent with the other swipe actions (Star/More) - which are also single words.
Thanks

Those labels double as accessibility labels. We changed Read to Mark as Read so that it’s more clear for people using accessibility.


Template Schema Generator For Categorical Image Data

🚨🚨 Feature Request

[ ] Related to an existing Issue
[x] A new implementation (Improvement, Extension)

improve HUB using by Defining a new Schema Generator
Various Image Recognition Datasets which have a pattern to identify a object,Like these kind  of categorical data have similar structure like
|
|__root folder
   |
   |
   |__label1
   |  |
   |  |__images
   |
   |__label2
   |  |
   |  |__images
   |
   |__ label_n
   |  |
   |  |__images
   |
   |

this kind of folder structure with images ,we need to define schema each time to upload the dataset which can be solve by one defined schema structure.
Description of the possible solution
we create a template schema generator for them for easy usage to upload to hub don't need to think so much on defining schema on this specific dataset which have a folder under that many  subfolders containing images.
Now a example schema can look like this :
def schema_generator(classlabel=[None],shape=(None,None)):
      classlabel = ClassLabel(name=classlabel)
      schema = {
    "labels": classlabel,
    "image":Image(
                  dtype="uint8",
                  shape=(shape)
       }
     return schema

Basically a schema_custimsor class with different schema customising utility for this label categorising problem.
An alternative solution to the problem can look like
A Template library for where users can store the schema for later use.
Teachability, Documentation, Adoption, Migration Strategy
More like c++ template library.

@davidbuniat assign it

@sparkingdark what about adding Dataset.from_directory() that will generate the schema and upload the dataset like other convertors from_pytorch() or from_tensorflow()?

Okay that's a good idea
On Wed, 9 Dec 2020, 13:41 Davit Buniatyan<EMAIL_ADDRESS>wrote:

Assigned #293 https://github.com/activeloopai/Hub/issues/293 to
@sparkingdark https://github.com/sparkingdark.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/activeloopai/Hub/issues/293#event-4088616564, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AL7XHCATBZQXG3D5XBI5DXLST4WMBANCNFSM4UTCUXVQ
.

@davidbuniat more like a getting the directory structure and generate the based on that or first analyze the directory structure to generate schema to build the Dataset.

@sparkingdark exactly!

need some light up @davidbuniat

@sparkingdark ofc, how can I help?

@sparkingdark let me know if you need support here :)

@sparkingdark let me know if you need support here :)

@mynameisvinn this has been moved to https://github.com/activeloopai/Hub/pull/370

@mynameisvinn this has been moved to https://github.com/activeloopai/Hub/pull/370


Gas Optimizations

abort() - line 150-156

status global variable could be used more cheaply with a temp variable
Estimated gas saved: 2 * 800 SLOAD = 1600

bulkPayout() - line 194-245

status global variable could be used more cheaply with a temp variable
bulkPaid global variable could be used more cheaply with a temp variable
Estimated gas saved:

4 * 800 SLOAD = 3200
1 * 5000 SSTORE = 5000
1 * 800 SLOAD = 800
1 * 5000 SSTORE = 5000
Total: 14000 max
Note: Due to conditional statements, it will not always be this high of gas savings. This is simply the most amount of gas that may be saved in a transaction.

finalizePayouts() - line 247-259

global variables used in loop
temporary variables would save lots of gas
Estimated gas saved:

(4 * 800 SLOAD) * _amounts.length

Note: Instead of benchmarks, I simply added estimations for the possible gas savings.

Hi @KadenZipfel and thanks for the PR, want to add an issue with some of the gas optimization findings? Can add to the project board that way. :)

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

CI seems to fail due to 0.4.24 related issues with assigning memory to variables. @KadenZipfel  are you still taking a stab at this?

@rbval My bad, thanks for pointing that out.

Hmm... Looks like there's a stack to deep exception now. I suppose the _bulkPaid variable pushed it over the top. It seems like we can either scrap _bulkPaid and eat the gas costs, or split the function up. @rbval @posix4e what do you think?

Yea I think I faced the same issue when trying that out. Feel free to split it in a meaningful  way. Probably makes sense to make the helper methods internal to limit the attack surface.

@rbval What's the current state on this?

Hi @KadenZipfel , just noticed this. Will return a comment by tomorrow. For further notice: do a re-request, helps me skim through the noise faster :)

 All committers have signed the CLA.

This needs a rebase

This needs a rebase

I'm still waiting on bounty payment. Will rebase once I've been compensated.

This needs a rebase

I'm still waiting on bounty payment. Will rebase once I've been compensated.

Wow I am so sorry I missed this update. Please reach out to me at<EMAIL_ADDRESS>and we get the payment going asap. Sorry for the delay.

@KadenZipfel updates?

@KadenZipfel
  1) Contract: Escrow

       calling abort

         succeeds if caller is a trusted handler:

     AssertionError: Unspecified AssertionError

      at Context.it (test/Escrow.js:95:9)

      at process._tickCallback (internal/process/next_tick.js:68:7)

Why is that failing?

Adding @uivlis to the review


Hey all,In PDFrequestOptions can you tell how to include view filters.

Looks like a duplicate of #643


Timezone config doesn't work?

Version: 0.999.0dev
Expected results
__time = 2020-03-04T10:00:00.000Z
Actual results
__time = 2020-03-04T02:00:00.000Z
How to reproduce the bug

in druid
__time:2020-03-04T02:00:00.000Z

/usr/local/src/incubator-superset# docker exec -it ef023b203bb9 /bin/bash

superset@ef023b203bb9:/app$ date -R
Sat, 14 Mar 2020 19:18:48 +0800
superset@ef023b203bb9:/app/superset$ cat config.py | grep DRUID_TZ
#DRUID_TZ = tz.tzutc()
DRUID_TZ = tz.gettz('Asia/Shanghai')

in superset
Go to Sources->Tables->Edit Table
Offset:8

Run Query
get __time = 2020-03-04T02:00:00.000Z
but not 2020-03-04T10:00:00.000Z
Environment
(please complete the following information):

superset version:  0.999.0dev
python version: 3.6.9
node.js version: node -v
npm version: npm -v

"Z" in the timestamp actually means UTC time (+00:00), so whatever time zone you set, as long as the data type is timestamp, it will always return in UTC time. You can probably create a calculated column with time offset instead.
There are significant complexity revolving timezone configs. Superset decided to always UTC time to keep things simple. It'd be quite some work if we want to add custom timezone support.


[mesheryctl] Token location is hardcoded.

See mesheryctl Command Tracker for current status of commands.
Current Behavior

contexts:
  local:
    endpoint: http://localhost:9081
    token: Default
    platform: kubernetes
    components: []
    channel: stable
    version: latest
current-context: local
tokens:
- location: ~/Downloads/test-2.json
  name: Default

failed to write the token to the filesystem: open /Users/shabana/.meshery/~/Downloads/test-2.json: no such file or directory
Desired Behavior

Token should be generated at path provided in config.
Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker Kubernetes

Contributor Resources

mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Let me see if I can tackle this one...


NW routes and GledOpto mode 007

2 updates (because I couldn't roll back....)

colored network routes on the map
my Gledopto GL-C-007 now reports as GL-C-007-2ID so I updated the adapter

Thanks!


Suggestion to remove duplicated validation text

When we are adding validation, we can see the error message. And after pressing for search validation text is written 2. times.
Also, it would be great to have the possibility to validate dropdown after form submit button was pressed(I mean when we complete full form with all fields)
I know it's a well-known issue I read tickets in searchable_dropdown, but looks like it's an important issue as lots of people need this widget as part of the form.

Hello @geriby23 ,
Sorry for the late reply. Times are quite busy these days. I tried to cover your first request (duplicate validation in case of menu) with release 1.0.16 . Can you please let me know if the result is ok for you?
I like your idea regarding the form validation. Can you please let me know whether you think that the following changes should lead to the desired behavior?

replace extends StatefulWidget by extends FormField<T> at the SearchChoices class
add a key to all constructors
add a call to super to all constructors

Somehow, it seams too easy.

Hi @lcuis , thanks for your reply! I have just checked your new release it's seems 1.0.17. Yup duplications were fixed, looks great now. About auto validation, before the user press form button to validate the form. I don't know if these steps would help, I'm not so familiar with flutter now. I think maybe you are calling somewhere validation method with setState.

Thanks for your reply. I'll leave this issue open for now until we get a clearer understanding on how to solve this second request.

And thanks for your confirmation @geriby23 !

I believe your second request would be a great next move for this plugin.
However, in order to start, I would need an example on how to use and what is expected to happen with a SearchChoices within a form.
Do you think you could propose such a usage example @geriby23 ?

Closing due to inactivity. Feel free to reopen if needed.


Add a Docker healthcheck

Seems reasonable to add this functionality:
https://docs.docker.com/engine/reference/builder/#healthcheck

Thanks for the suggestion. As I understand it a healthcheck would place requirements on the base image which we might not want or need. Most people are going to delegate health checks to a platform anyway (k8s, cloudfoundry etc.) so I'm not sure it has a huge value in a "getting started" guide.


Bootstrap 4 support?

Hey guys - are there any plans to support Bootstrap 4?

I think all of the form class names are the same in Bootstrap 4 so the validator should still work.

They aren't - has-error has been dropped. Also, Bootstrap 4 no longer ships with Glyphicons. Therefore the feedback icons won't show unless the user installs Glyphicons:
feedback: {
      success: 'glyphicon-ok',
      error: 'glyphicon-remove'
    }

It would be nice if there would be a parameter for specifying success and error icons, for example.

Aw man. I guess I could also add .has-danger along with .has-error to keep both versions compatible.
I'm not sure what you're suggesting about the icons though. There already is an option to specify the icon classes that are applied.

Ah, that would be cool :+1:
Sorry about the icons - I hadn't spotted this before. Nice!!!

This should perhaps go into a separate branch? As BS4 is still in alpha. Then set a tag for the current BS3. There is no need to have universal support to complicate source code. IMO

It's a tiny change, just adding .has-danger along with .has-error. It'd be more complicated to have to manage two versions.

Bootstrap V4 is incompatible with bootstrap-validator due to https://github.com/twbs/bootstrap/pull/19141#issuecomment-233172275.  Please re-open until then.

I think is no a good idea make further changes to support an alpha package... At least not yet, until v4 get stable.

Any update on this?

Bootstrap 4 is still in alpha. I'm not gonna start officially supporting it until it it's at least in beta.

Has the Bootstrap 4 Beta been released, any updates?

Bootstrap is Currently v4.0.0-beta, any updates?

v4 is out. Waiting for the updates. :)

I'm looking forward to this as well!!

me too

So is it compatible now?

I made changes locally to get it working on my app. I'm not confident it's good enough for a PR, but it's working. Maybe someone else can tackle it?


Implement readthedocs builds

Closes #29
As of April 2022, it is now possible to build Jupyter books using readthedocs.
This is an important feature, as this makes it possible to have readthedocs generate the docs for each individual PR.

@kanderso-nrel Would you recommend that the solarstations.csv file go inside the docs folder or at the root? Since it's the core of the repository I figured it might be nice to have it at the root, but that makes it a bit harder to access from the files in the docs folder I suppose.

I think either is fine, but I'd lean towards keeping it in the root and accepting the small inelegance of having to do pd.read_csv('../solarstations.csv')


xteve  [ERROR] CreateFile .\<IP_ADDRESS>:34400: The system cannot find the file specified

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
after downloading the xteve
as soon i started the program into cmd
i get this error
2021/08/11 06:20:33 [xTeVe] System Folder:          C:\Users\racha.xteve
2021/08/11 06:20:33 [xTeVe] Load Settings:          C:\Users\racha.xteve\settings.json
2021/08/11 06:20:33 [xTeVe] Temporary Folder:       C:\Users\racha\AppData\Local\Temp\xteve\2021-06-27QW-R3XBGB
2021/08/11 06:20:33 [xTeVe] GitHub:                 https://github.com/xteve-project
2021/08/11 06:20:33 [xTeVe] Git Branch:             master [xteve-project]
2021/08/11 06:20:33 [xTeVe] SSDP / DLNA:            true
2021/08/11 06:20:33 [xTeVe] UUID:                   2021-06-27QW-R3XBGB
2021/08/11 06:20:33 [xTeVe] Tuner (Plex / Emby):    16
2021/08/11 06:20:33 [xTeVe] EPG Source:             PMS
2021/08/11 06:20:33 [xTeVe] Plex Channel Limit:     480
2021/08/11 06:20:33 [xTeVe] Unfiltered Chan. Limit: 480
2021/08/11 06:20:33 [xTeVe] Backup Path:            C:\Users\racha.xteve\backup
2021/08/11 06:20:33 [xTeVe] Backup file:            C:\Users\racha.xteve\backup\xteve_auto_backup_20210811_0620.zip
2021/08/11 06:20:33 [xTeVe] Open:                   <IP_ADDRESS>:34400
2021/08/11 06:20:33 [xTeVe] [ERROR] CreateFile .\<IP_ADDRESS>:34400: The system cannot find the file specified. () - EC: 0

FATAL ERROR * * * * *
OS:   windows
Arch: amd64
Err:  interface conversion: interface {} is nil, not float64

panic.go:965 runtime.gopanic
iface.go:261 runtime.panicdottypeE

figured something out
C:\Users\racha>nssm install xteve
Administrator access is needed to install a service.
C:\Users\racha>nssm start xteve
Can't open service!
OpenService(): Access is denied.
dont know how make it adminstrator tried everthing it keeps saying access denied

Follow this... https://github.com/xteve-project/xTeVe-Documentation/blob/master/en/configuration.md#xteve-as-a-windows-service


Remove static_cast

Fixes #2307

In case you are trying to make it compile on a platform where the static_assert doesn't compile, removing the lock-free int check is not the right fix. If a platform does not have a lock-free int, we must either write a special version that relies on a smaller lock-free type, or on a lock, or a code that disables logging on such rare platform.

The static_cast is unnecessary.  I can add back a check that the underlying type is an int.

@RickWinter
g_logLevel HAS to be atomic. Given that, comparison and initialization mixing of int and enum is not going to compile:
atomic<int> g_logLevel = ...;
Logger::Level level = ...;

level >= g_logLevel; // error: no match for ‘operator>’ (operand types are ‘enum Logger::Level’ and ‘atomic<int>’)
g_logLevel = Logger::Level::Warning; // error: cannot convert ‘enum Logger::Level’ to ‘atomic<int>’ in initialization

Nowhere in the standard it says that std::atomic<T> where sizeof(T) == sizeof(int) HAS to use the same template specialization. It is pretty reasonable to assume, but it does not say so.
And why remove that. We explicitly use int, and that is why it is better implementation, and static_cast removal does not compare in value.
There's nothing that would prevent implementations from doing this:
#include <iostream>

enum class Enum : int { A, B, C };

template <typename T>
struct Otomic
{
    static const constexpr bool IsLockFree = false;
};

template <>
struct Otomic<int>
{
    static const constexpr bool IsLockFree = true;
};

int main()
{
    std::cout << "Otomic<Enum>::IsLockFree: " << Otomic<Enum>::IsLockFree << "\n";
    std::cout << "Otomic<int>::IsLockFree: " << Otomic<int>::IsLockFree << "\n";
}

Output:
Otomic<Enum>::IsLockFree: 0
Otomic<int>::IsLockFree: 1

https://github.com/Azure/azure-sdk-for-cpp/issues/2307#issuecomment-916313877
This, to me, reverses the meaning of #2307 being necessary to do in the first place.

Billy:

Wait, I read this as the opposite - that before C++17, there is no way to be guaranteed that atomicLogger::Level is atomic, so we are limited to int, for which we can guarantee.

That's right. is_always_lock_free was added in '17. It is not necessarily true, particularly on implementations that chose the "base classes" version of atomic, that atomic is in any way related to atomicLogger::Level.

https://github.com/Azure/azure-sdk-for-cpp/issues/2307#issuecomment-916388721

The extra typedef and static_casts are not necessary.  That code is defending against terrible QoI which is a waste of time.  If the underlying type is an int but the implementation chooses to use some other implementation than atomic the implementation is likely to have much bigger problems.

@RickWinter, can you at least remove the sizeof(Logger::Level) <= 8 && (sizeof(Logger::Level) & (sizeof(Logger::Level) - 1)) == 0 assertion? We do still check that the underlying type is int, and we expect that the implementation will pick the same implementation of atomic internally. Given all that, no need for this additional check sizeof being power of two.


feat: swap out raw-loader for html-loader for html files

#* angular html components as well as index.html now have the benefits of html-loader

Fixes #2231

LGTM. I'll wait for tests to pass.

Looks like tests failed on basehref, I'll give this another whirl

Almost there, would be great to have this fixed in the coming update!
Thank you, @TheLarkInn

TBH if you wanna replace our custom base href plugin by base functionality of HTMLPlugin, I can't say I mind. Might be the sort of stuff that would happen in a different commit/pr.

Will this PR be merged soon? :)

@TheLarkInn Have you tried running ng build --base-href /url in a new project and look if the base is there? To me it seems like this breakage is legit.

@hansl I noticed that in your PR you had html-loader (the AOT one), so I hadn't revisited it.

I'll take a look at this next and see what errors are being caused.

Closed since we removed this in https://github.com/angular/angular-cli/pull/2537


[Bug] sglang template import issue

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.
[X] 4. If the issue you raised is not a bug but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[X] 5. Please use English, otherwise it will be closed.

Describe the bug
I run sglang with the following standard zephyr.jinja template:
{% if messages[0]['role'] == 'system' %}
    {% set offset = 1 %}
{% else %}
    {% set offset = 0 %}
{% endif %}

{% for message in messages %}
    {% if (message['role'] == 'user') != (loop.index0 % 2 == offset) %}
        {{ raise_exception('Conversation roles must alternate user/assistant/user/assistant/...') }}
    {% endif %}

    {{ '<|' + message['role'] + '|>\n' + message['content'] | trim + eos_token + '\n' }}
{% endfor %}

{% if add_generation_prompt %}
    {{ '<|assistant|>\n' }}
{% endif %}

But I got an error of:
"RuntimeError: Chat template configs/chat_templates/zephyr.jinja is not a built-in template name or a valid chat template file path."
I tried to write a template in .json file myself, since your sglang reads the template in:
Conversation(
  name=template["name"],
  system_template=template["system"] + "\n{system_message}",
  system_message=template.get("system_message", ""),
  roles=(template["user"], template["assistant"]),
  sep_style=sep_style,
  sep=template.get("sep", "\n"),
  stop_str=template["stop_str"],
)

But I still got the above error. Do you have any template example file in .json format or jinja format?
Reproduction
Code example:
sglang_command = f"python -m sglang.launch_server --model-path {model_path}" + \
                   f" --port {port}"

 if chat_template is not None:
     sglang_command += f" --chat-template {chat_template}"
 elif tokenizer_path is not None:
     sglang_command += f" --tokenizer-path {tokenizer_path}"

 print("running sglang server with command: ", sglang_command)

 with open(result_save_path, 'w', encoding='utf-8') as log_file:
     pass
 log_file = open(result_save_path, 'a', encoding='utf-8')

 ## somehow this will hang if we don't output things to a FILE
 sglang_process = subprocess.Popen(
     sglang_command,
     shell=True,
     start_new_session=True,
     stdin=log_file,
     stdout=log_file,
     stderr=log_file,
     text=True
 )

Model: fine tuned "alignment-handbook/zephyr-7b-sft-full".
Environment
sglang state-of-the-art version

SGLang supports jinja and json formats.
Jinja: the same as HF tokenizer
JSON: https://github.com/sgl-project/sglang/blob/main/docs/references/custom_chat_template.md
code: https://github.com/sgl-project/sglang/blob/16eb33ffe2afc9c043e4f80f18e15ea57b984944/python/sglang/srt/openai_api/adapter.py#L124-L160
For your error, please check the file path.


pkger silently fails to apply patches (uses wrong location)

Environment

Platform: Linux (ubuntu 20.04), docker 20.10.21 in rootless mode
pkger version: pkger 0.10.0

recipe contains list of patches like:

  patches:
    - gcc11-alt-compat-test.patch
    - gcc11-d-shared-libphobos.patch
   ...

During build I can see in output:

[INFO ][2022-12-01T18:08:10.576Z][sfx-gcc11][11.3.0][rocky8] applying patches
[ERROR][2022-12-01T18:08:10.599Z][sfx-gcc11][11.3.0][rocky8] /bin/sh: /tmp/sfx-gcc11-tmp-1669918080/patches/gcc11-alt-compat-test.patch: No such file or directory
[WARN ][2022-12-01T18:08:10.600Z][sfx-gcc11][11.3.0][rocky8] applying patch Patch { patch: "gcc11-alt-compat-test.patch", strip: 0, images: None } failed, reason = command failed with exit code 1
Error:
/bin/sh: /tmp/sfx-gcc11-tmp-1669918080/patches/gcc11-alt-compat-test.patch: No such file or directory
...

... and so on for all patches.
And build continues like nothing wrong happened.
As I looked into container, patches are uploaded to /tmp/sfx-gcc11-tmp-1669918080/patches.tar/ directory, /tmp/sfx-gcc11-tmp-1669918080/patches is empty
I see 2 problems here:

patches are uploaded or fetched to/from wrong location in container
failure during patch apply does not stop build => it can be easy overlooked

first part of problem (using wrong patch location) is probably in pkger-core/src/build/patches.rs:87 - fix here made patches work

Should be resolved in #91. I'll leave the issue open for now to confirm that it works on your end.


Cache hit rate per URL

Right now I can get the hitrate per client created, but I'd like to see how the cacherate is per URL as well, kinda like a label in prometheus (which is what we'd use it for). Would that be possible to add?
We query this thing: https://github.com/httpcache4j/httpcache4j/blob/4b26a55286b851ac124c20c8167a78aad860b30b/httpcache4j-core/src/main/java/org/codehaus/httpcache4j/cache/CacheStatistics.java

Sure, but to avoid having a major memory impact is has to be limited somehow.
It should probably be based on a concurrent LRU map and not introduce external dependencies.
PR ?

Another possibility is to add a callback handler that deals with different states of cache.
Then the memory impact is not handled in the library, but by clients.
Use of the callback must be optional, so there must be a Noop handler.


Selected Kernel does not point to the right interpreter

Originally reported by @muser840
https://github.com/microsoft/vscode-jupyter/issues/5214#issuecomment-817510157
Just to confirm that Jupyter uses the wrong python, which is confirmed by
import sys
print(sys.executable)
! which python

For me the output of the above prints:
/usr/bin/python3
/home/user/venv/bin/python

It should be the same! These is some mess with switching between python versions by the extension. Somehow switching to old and already deleted environments make it work. That makes extension totally unusable...
The problem is present on versions:
v2021.5.702919634
v2021.6.732501275
update: sometimes it randomly starts to work...

@muser840
Please could you provide the following information:

VS Code version: XXX
Jupyter Extension version (available under the Extensions sidebar): XXX
Python Extension version (available under the Extensions sidebar): XXX
OS (Windows | Mac | Linux distro) and version: XXX
Python and/or Anaconda version: XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Attempt to repro the issue & then copy the output from the Jupyter output panel.

I think I see the same issue:
Version: 1.55.2 (system setup)
Commit: 3c4e3df9e89829dce27b7b5c24508306b151f30d
Date: 2021-04-13T09:35:57.887Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 6.1.7601
Jupyter Extension : v2021.5.745244803
Python 3.8.6 64-bit
Environment: venv
With venv selected - Jupyter Notebook output:
import sys
print(sys.version)
print(sys.executable)
3.8.6 (tags/v3.8.6:db45529, Sep 23 2020, 15:52:53) [MSC v.1927 64 bit (AMD64)]
C:\Program Files\Python38\python.exe
Command line - same venv activated:
(reporting-env) D:\projects\utils>python
Python 3.8.6 (tags/v3.8.6:db45529, Sep 23 2020, 15:52:53) [MSC v.1927 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import sys
print(sys.executable)
D:\projects\reporting-env\Scripts\python.exe

@bardal Thanks for the feedback:

Are you using notebooks or interactive window?
Please could you re-load VS Code, run the same code in a cell and prvide the following:

Cell Output
Output from the Jupyter output panel (you might have to copy th eoutput to a text file & upload that text file)
What kernel did you pick (i.e. what were you expecting to see)
Screenshots

With the cell output
With the kernel displayed in the bottom right

I'm sorry you're running into this and would appreciate the replying with the above info, I'd like to unblock you and others running into this.

I am loading a .ipynb in Visual Studio Code - not sure if that means notebooks or interactive window!
Jupyter Cell:
import sys
print(sys.version)
print(sys.executable)
3.8.6 (tags/v3.8.6:db45529, Sep 23 2020, 15:52:53) [MSC v.1927 64 bit (AMD64)]
C:\Program Files\Python38\python.exe
I selected reporting_env as the kernel
Screenshot and Jupyter output panel screenshot:

jupyter_log.txt

Thanks for the information,
Is reporting_env a conda environment?

reporting_env is a standard Python venv (not conda).

@bardal Thanks for the information
Please could you do me yet another favor:

Please install VS Code  Insiders
Install the Jupyter extension
Open the same notebook and try this agaon
Once again, please provide the logs & the screenshot.

FYI - In VS Code Insiders, you'll get the new Notebook UI & I believe this has been resolved there (See here for more information https://devblogs.microsoft.com/python/notebooks-are-getting-revamped/).
Thanks again for your help.

Thanks for providing the information.

Unfortunately this is on a work PC so I won't be able to install VS Code
Insiders.
On Mon, 26 Apr 2021 at 18:04, Don Jayamanne @.***>
wrote:

@bardal https://github.com/bardal Thanks for the information
Please could you do me yet another favor:

Please install VS Code Insiders
Install the Jupyter extension
Open the same notebook and try this agaon
Once again, please provide the logs & the screenshot.

FYI - In VS Code Insiders, you'll get the new Notebook UI & I believe this
has been resolved there (See here for more information
https://devblogs.microsoft.com/python/notebooks-are-getting-revamped/).
Thanks again for your help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-jupyter/issues/5489#issuecomment-827001544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLXLESXD24X6OH2YKVBFWDTKWMJ3ANCNFSM42ZWDDFA
.

Still reproes

Validated. Lots of different ways to set up environments, but works with my venv.


could you please assist me with using json2csv in "vanilla" JS

Hi!
I assume, based on the documentation, that it's possible to use json2csv as a library in a project which is not using node, i.e. contains only old-school (like myself) js. I found this older 5.0.7 library, which is well, old and non-existing in your current release list. I'm trying to use the newest version of it inside our platform with no luck. is there any way to achieve this?

okay, I think I can now answer my own question:
using rollup, it's possible to create a library for those who are stuck in the 2000s. here is how I achieved this. I'll probably fork this library and create a PR in the next few days...

you'll obviously need rollup for node, which I have installed globally for myself, but you can either have it installed globally or locally, dealer's choice...

npm i -g rollup

install the necessary rollup packages using npm:

npm i --save-dev @rollup/plugin-commonjs @rollup/plugin-node-resolve rollup-plugin-polyfill-node

add rollup.config.js to project root. please note that the output file will be stored in ./bundle.js

//rollup.config.js
import { nodeResolve } from '@rollup/plugin-node-resolve';

import nodePolyfills from 'rollup-plugin-polyfill-node';

import commonjs from '@rollup/plugin-commonjs';

export default {
  input: 'packages/plainjs/src/index.js',
  output: {
    file: './bundle.js',
    format: 'umd',
    name: "json2csv"
  },
  plugins: [
    nodeResolve({
      browser: true
    }),
    commonjs(),
    nodePolyfills()
  ]
};

run rollup -c, and voila!

Alternatively, one could also add "rollup": "rollup -c" to scripts in package.json for automation or other purposes.

Hi @arashdalir ,
The current version of json2csv only support esm modules and cjs. I use esbuild instead fo rollup but it should be the same.
Can you elaborate a bit on how are you using the library and what error you are getting?

hi! sorry for the late reply; was really busy here.
we are using json2csv in our application to export large amounts of data (100k rows) with around 50 columns. the library is integrated into an old code which has no support for cjx or esm, so the library needed to be "converted" so we could use it in our tool. additionally, you are using lodash and json/streamparser which created additional issues for us because our app is in an offline environment and I couldn't find a way to load these libraries...
to achieve the result I needed, I rolluped the library and created an old-fashioned UMD-like bundle out of it, which resolves both of my issues: it automatically fetches the needed external resources and embeds them into the bundle, and the result is a plain javascript code which we are now using in our application.

That makes sense.
I'll close this as it is resolved.
Please let me know if there is anything else that you need.


Naming consistency in dev-cluster

I decided upon some naming conventions after having already done the celestia-local work, so I need to go back and make some things consistent with the newer work.

consistent naming for PersistentVolumes and PersistentVolumeClaims

{{pod name}}-{{service name}}-pv
{{pod name}}-{{service name}}-pvc

use plural version of words for kubernetes yaml filenames, even if there is only one resource in the file at the moment

volumes.yaml, storageclasses.yaml, etc

consistent naming for storage values in values.yaml

We've had a lot of work done to our charts. I don't think this is relevant anymore. We can reopen if it is.


Upgrade dependencies

Upgraded

rubocop to 0.49.1
simplecov to 0.15.1
rake to 12.1.0

Note
ruby would crash during testing if upgrade minitest to 5.10.2/3

This change is

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

LGTM


ConfigurationModel is slow

Related to https://github.com/Microsoft/vscode/issues/34353 and #34282. When you profile start and look at the time spend in the configuration service you'll notice that it's still very slow.
Paste require('vs/base/node/startupTimers').ticks().forEach(tick => console.log(tick.name, tick.duration)) in the dev-tools-console and see that ~100ms is spend in building/merging/deduping the configuration model.
Apart from clone it also seems that a lot of time is spend in arrays#distinct which as a potential runtime of O(n2)

@jrieken Thanks for the report.
May I know if it is slower than before ?

May I know if it is slower than before ?

No clue, you can measure it with an old version of VS Code

This is the new picture after doing following improvements:

Merge, clone and freeze only when the configuration is requested.
Fix from @jrieken - #37539
Register and trigger configuration and default configurations all at a time.
Merge user configuration service and workspace configuration service to remove duplicate merges and triggering.


[PE-D][Tester C] Urgency Data when running analytics returns invalid percentage

After adding a single loan with the start date 1900-01-01 and the return date 1990-01-02, the analytics command returns an urgency percentage in the negatives.

Labels: type.FunctionalityBug severity.High
original: jovantanyk/ped#3

Fixed by PR #201


Use PhysicalModel for the default canvas Material.

It might be easier to review the diff for each commit separately.
The first commit is just a revert of the previous RRect workaround.

@HansMuller - I updated buttons_test to expect clipPath instead of clipRect, please take a look at the last commit in this PR

LGTM too. Sorry to have taken so long to chime in.


Timeout of 5000ms exceeded calling find on pages

Hi! How can I fix this issue?

It was just the version of nuxtJS.. (^1.0.0-rc11)


On reloading my page ReactPlayer doesn;t work.

On reloading my page ReactPlayer doesn;t work. It gives the following error.
YT.Player is not a constructor

@iamjamielobo What OS and browser are you using? Do you have more steps to reproduce this? Or an example repo?

It;s happening on all browsers.
It happens rarely when you reload the page.
I've implemented it here.
https://in.bookmyshow.com/mumbai/videos/trailers/dunkirk/ET00044852

Ok I think I know what's going on. If you have several different things loading the youtube iframe API at once, the global YT object can sometimes be {loading: 1, loaded: 0} with no Player yet. When this happens, the getSDK promise is resolved with this object here and the whole thing breaks. Should be a simple fix just to ensure that loaded is 1 as well before resolving the array.


KeyPair on WSL2 (probably WSL too) not working

Got public / private keypair working using snowflake connector for python and snowsql both on WSL2 but having difficulty recognising the private key file via the Snowflake Connecto on SQLTool.
I've copied the key  file from WSL to /mnt/c to see if that is the problem but the same message with a different directory. In both cases I pick the file from the drop down instead of typing it in directly.
Error log shows as:
[1622678960563] ERROR (ls): Connecting error: {"code":-32001,"data":{"driver":"Snowflake","driverOptions":{}},"name":"Error"}
ns: "conn-manager"
[1622678960565] ERROR (ext): ERROR: Error opening connection ENOENT: no such file or directory, open '\wsl$\Ubuntu\home\itsme\snowflake\snowflake_keys\snowflake_test.rsa_key', {"code":-32001,"data":{"driver":"Snowflake","driverOptions":{}}}
ns: "error-handler"
[1623272025619] INFO  (ext): EXECUTING COMMAND => sqltools.showOutputChannel

Hi, I got a fix: In short the key full filepath needs to be in Native Linux format as you would get to it from your WSL disstro and do  not use the SELECT FILE Button as it will build the path in Windows format.
An enhancement would be if WSL is being used that the file button Select would use the filepath for the Linux distro.
I don't know how to change the status to enhancement. Sorry. Could the owner please change it.
Details:
Don't pick the private key from the SELECT FILE button if you are on WSL2.
This is because the file path is in Windows format and you need it in Linux format.
The egs below are as if I has the key in both Windows and Linux and was to pick one of them. Personally if I'm
using wsl I'd leave the key in the wsl distro especilly if you're using the same ksy for sqnowsql or the python connector.
eg: BAD
WINDOWS wrong format  although it looks okay (If I pick from the SELECT file button)
A) Use KEY on Windows dir:    C:\itsme\snowflake_test.rsa_key
B) Use KEY on Linux dir:            \wsl$\Ubuntu\home\itsme\snowflake\snowflake_keys\snowflake_test.rsa_key
eg: GOOD
LINUX right format enter it directlyv(eg is with Ubuntu 18.04.03 distro)
A) Use KEY on Windows dir:     /mnt/c/itsme/snowflake_test.rsa_key
B)  Use KEY on Linux dir:          /home/itsme/snowflake/snowflake_keys/snowflake_test.rsa_key

Could you show how you configured this? @anujghai


Admin permission

I am getting "Need admin permission" while testing the connection.  I created a test bot using the Portal UI and associated the auth to it.  How do I fix this error?

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

ID: a9a43e68-83a3-1adc-0c55-e277ddb98c6b
Version Independent ID: 5c46848d-bc20-480e-4571-00a283d930a2
Content: Add authentication to your bot via Azure Bot Service - Bot Service
Content Source: articles/v4sdk/bot-builder-authentication.md
Service: bot-service
Product: bot-framework
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

Post support questions on Stack Overflow using the botframework tag.


fix BUG:https://github.com/rs/SDWebImage/issues/1484

New Pull Request Checklist

[ ] I have read and understood the CONTRIBUTING guide

[ ] I have read the Documentation

[ ] I have searched for a similar pull request in the project and found none

[ ] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[ ] I have added the required tests to prove the fix/feature I am adding

[ ] I have updated the documentation (if necessary)

[ ] I have run the tests and they pass

[ ] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
...

Codecov Report

Merging #2249 into master will decrease coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2249      +/-   ##
==========================================
- Coverage   78.53%   78.48%   -0.05%
==========================================
  Files          36       36
  Lines        3741     3742       +1
  Branches      343      342       -1
==========================================
- Hits         2938     2937       -1
- Misses        780      783       +3
+ Partials       23       22       -1

Impacted Files
Coverage Δ

SDWebImage/SDWebImageCodersManager.m
70.14% <100%> (+0.45%)
:arrow_up:

SDWebImage/SDWebImageDownloaderOperation.m
91.53% <0%> (-0.66%)
:arrow_down:

Continue to review full report at Codecov.

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

@LululuSir thanks for submitting the PR, but please detail a bit:

why do you think this is the proper fix for the issue #1484
how did you test? can you write some unit test?

Reiteration path:
Decompress the same image in multi-thread at the same time.
One way to increase the probability of crash , is to decompress an animated image(gif/apng).
Reason(Speculation):
This function is not thread safe:
CGContextDrawImage(CGContextRef c, CGRect rect, CGImageRef image);
Verification:
It was very easy to crash when decompress the same imageRef in two thread at the same time. And it was fixed when I add the thread protection.

@LululuSir thanks for the details. Could you also add a unit test?
@dreampiggy what do you think? It sounds valid to me

@dreampiggy I think it will work too.
@bpoplauschi I find it in product environment, and it fix that crash.But, I can't reiteration in a demo or unit test

Got it. I like @dreampiggy's suggestion

@bpoplauschi @LululuSir 😅 Well. I miss one thing. Just mark coders as atomic may non-thread safe in this case.
If two thread call addCoder: with two different coders. There may be a posibility that each of two threads get the original coder array, and they both modify to add one. The final result will contains only  one of the desired added coder.
// Time begin

<--- Thread 2 here
NSMutableArray<id<SDWebImageCoder>> *mutablecoders = [self.coders mutableCopy];
if (!mutablecoders) {
    mutablecoders = [NSMutableArray array];
}
<--- Thread 1 here
[mutablecoders addObject:coder];

// Time passe

<--- Thread 2 here, mutable coders is nil, then it create empty array @[]
if (!mutablecoders) {
    mutablecoders = [NSMutableArray array];
}
[mutablecoders addObject:coder];
self.coders = [mutablecoders copy];
<--- Thread 1 here, the coders is @[coder1]

// Time passe

self.coders = [mutablecoders copy];
<--- Thread 2 here, the coders is @[coder2]

So actually, we need a lock to guard this instead of using atomic. I can create a PR for this.

@dreampiggy you are correct, we need to lock the part where we edit the array. That is a common pattern, which we have used to do with a serial queue that is the recommended method from Apple to synchronising access to a non-thread safe resource. We can also use a lock. But you are right, we can't just rely on the atomic flag.

I'll provide a fix in 4.x.
This decompressedImageWithImage:data:options is totally removed in 5.x because the fact that force decode a image is not related to image format, so it's not part of custom coder to provide a implementation. Instead, we use SDWebImageCoderHelper to do this force decode process. It's thread-safe.

@LululuSir See #2274

Done by #2274....Which use a lock to guard thread-safe for this coders manager.


Dockerfile generation fails with Cluster not found for context 'k8s.KubeConfigModels.Context' in kubeconfig

If I generate a new operator using the template, I get the following build failure:
Generating Dockerfile
Unhandled exception. System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> k8s.Exceptions.KubeConfigException: Cluster not found for context `k8s.KubeConfigModels.Context` in kubeconfig
   at k8s.KubernetesClientConfiguration.SetClusterDetails(K8SConfiguration k8SConfig, Context activeContext)
   at k8s.KubernetesClientConfiguration.InitializeContext(K8SConfiguration k8SConfig, String currentContext)
   at k8s.KubernetesClientConfiguration.GetKubernetesClientConfiguration(String currentContext, String masterUrl, K8SConfiguration k8SConfig)
   at k8s.KubernetesClientConfiguration.BuildConfigFromConfigObject(K8SConfiguration k8SConfig, String currentContext, String masterUrl)
   at k8s.KubernetesClientConfiguration.BuildDefaultConfig()
   at DotnetKubernetesClient.KubernetesClient..ctor()
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeConstructorInfo.Invoke(BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitConstructor(ConstructorCallSite constructorCallSite, RuntimeResolverContext context)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSiteMain(ServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitDisposeCache(ServiceCallSite transientCallSite, RuntimeResolverContext context)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor`2.VisitCallSite(ServiceCallSite callSite, TArgument argument)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.Resolve(ServiceCallSite callSite, ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.DynamicServiceProviderEngine.<>c__DisplayClass1_0.<RealizeService>b__0(ServiceProviderEngineScope scope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngine.GetService(Type serviceType, ServiceProviderEngineScope serviceProviderEngineScope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngineScope.GetService(Type serviceType)
   at McMaster.Extensions.CommandLineUtils.CommandLineApplication.ServiceProvider.GetService(Type serviceType)
   at McMaster.Extensions.CommandLineUtils.CommandLineApplication.System.IServiceProvider.GetService(Type serviceType)
   at McMaster.Extensions.CommandLineUtils.Conventions.ConstructorInjectionConvention.FindMatchedConstructor[TModel](ConstructorInfo[] constructors, IServiceProvider services, Boolean throwIfNoParameterTypeRegistered)
   at McMaster.Extensions.CommandLineUtils.Conventions.ConstructorInjectionConvention.ApplyImpl[TModel](ConventionContext context)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)
   at McMaster.Extensions.CommandLineUtils.Conventions.ConstructorInjectionConvention.Apply(ConventionContext context)
   at McMaster.Extensions.CommandLineUtils.CommandLineApplication.Builder.McMaster.Extensions.CommandLineUtils.Conventions.IConventionBuilder.AddConvention(IConvention convention)
   at McMaster.Extensions.CommandLineUtils.ConventionBuilderExtensions.UseConstructorInjection(IConventionBuilder builder, IServiceProvider additionalServices)
   at KubeOps.Operator.HostExtensions.RunOperatorAsync(IHost host, String[] args)
   at CloudFormationStackOperator.Program.Main(String[] args) in <redacted>
   at CloudFormationStackOperator.Program.<Main>(String[] args)

Hey @mmccord-mdbuyline
Quick question: do you have any extra fields in the context? Like "non standard" fields that may interfere with the parser?
This sounds like an issue with your kubeConfig.
But, in any case, it should not need the kubernetes client to generate the dockerfile.

No non-standard fields. It reads the kubeconfig fine. Has to do with the fallback mechanism in BuildDefaultConfig(). In fact if I add the following fall back mechanism to my ConfigureServices method to override the default behavior it works:
               bool tryFallback = false;
                try
                {
                    services.TryAddSingleton(KubernetesClientConfiguration.InClusterConfig());
                }
                catch (Exception)
                {
                    tryFallback = true;
                }

                if (tryFallback)
                {
                    try
                    {
                        services.AddSingleton(
                            KubernetesClientConfiguration.BuildConfigFromConfigFile(UserKubeConfigPath, "docker-desktop"));
                    }
                    catch (Exception ex)
                    {
                        throw;
                    }
                }

However, obviously this is not ideal, particularly from the stand point of a template that theoretically should work out of the box. I might suggest a different approach for your generators: bundling .NET executables with the nuget package that perform these operations instead of building these operations into the application produced by the template. If you do that you can build out purpose built executables that do not need to even include the kubernetes client.

Side note... UserKubeConfigPath from above looks like this:
private static readonly string UserKubeConfigPath =
            Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.UserProfile), ".kube", "config");

I see...
I don't get the point of your suggestion clearly, do you mind explaining which executables you mean?
I use the kubernetes client quite heavily in the operator sdk, so there is no replacing it. I had built a software once that used kubectl, but this is error prone as hell when using it over sub-processes.
When we have a look at the method that builds the kube config out of "default":
https://github.com/kubernetes-client/csharp/blob/master/src/KubernetesClient/KubernetesClientConfiguration.ConfigFile.cs#L47-L71
Is any of this "if" 's true on your machine, or can you think of any reason why it does use the wrong config?
The steps are:

Check if KUBECONFIG environment var is set, if yes, use this file (or multiple files) as config
Check if the <user home>/.kube/config file exists, if yes, use this file
Check if the running app is "in - cluster" (meaning is KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT set, and does the service account token as well as the root ca exist on the machine), if yes, build the kube config inside the cluster
use a default location of http://localhost:8080

I'm asking this, because - despite the usage of docker and stuff - I've got a classical "works on my machine" problem. I cannot reproduce the error.
BUT, what you could do:
as soon as you added the operator stuff with the AddKubernetesOperator, you may remove the transient binding for IKubernetesClient and add it with your specific config:
services.AddKubernetesOperator();
services.RemoveAll(typeof(IKubernetesClient));
// use your specific config here
services.AddTransient(_ => new KubernetesClient(KubernetesClientConfiguration.BuildConfigFromConfigFile()));

Did you manage to find something @mmccord-mdbuyline ?

FWIW, I was able to reproduce by breaking my kubectl config; if the active context is set to use a cluster that isn't defined in the file, it throws this error.
This looks like a non-issue for this project. An issue could be opened on buehler/dotnet-kubernetes-client to change the behavior to match kubectl (which attempts to just connect to localhost:8080), but I prefer it throwing an error like this anyways.
apiVersion: v1
clusters:
- cluster:
    server: "https://{{REDACTED}}/"
  name: master
contexts:
- context:
    cluster: InvalidClusterNameHere
    user: admin
  name: master
current-context: master
kind: Config
preferences: {}
users:
- name: admin
  user:
    token: "{{REDACTED}}"

[2021-05-31T12:42:53.8011607-05:00] [INF] (Controller/Program) Starting up
[2021-05-31T12:42:54.6253442-05:00] [FTL] (Controller/Program/1) Application start-up failed
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> Autofac.Core.DependencyResolutionException: An exception was thrown while activating DotnetKubernetesClient.KubernetesClient.
 ---> Autofac.Core.DependencyResolutionException: An exception was thrown while invoking the constructor 'Void .ctor()' on type 'KubernetesClient'.
 ---> k8s.Exceptions.KubeConfigException: Cluster not found for context `k8s.KubeConfigModels.Context` in kubeconfig
   at k8s.KubernetesClientConfiguration.SetClusterDetails(K8SConfiguration k8SConfig, Context activeContext)
   at k8s.KubernetesClientConfiguration.InitializeContext(K8SConfiguration k8SConfig, String currentContext)
   at k8s.KubernetesClientConfiguration.GetKubernetesClientConfiguration(String currentContext, String masterUrl, K8SConfiguration k8SConfig)
   at k8s.KubernetesClientConfiguration.BuildConfigFromConfigFileAsync(FileInfo kubeconfig, String currentContext, String masterUrl, Boolean useRelativePaths)
   at k8s.KubernetesClientConfiguration.BuildConfigFromConfigFile(FileInfo kubeconfig, String currentContext, String masterUrl, Boolean useRelativePaths)
   at k8s.KubernetesClientConfiguration.BuildConfigFromConfigFile(String kubeconfigPath, String currentContext, String masterUrl, Boolean useRelativePaths)
   at k8s.KubernetesClientConfiguration.BuildDefaultConfig()
   at lambda_method100(Closure , Object[] )
   at Autofac.Core.Activators.Reflection.BoundConstructor.Instantiate()
   --- End of inner exception stack trace ---


ECATT scripts pull request

When I backup ECATT script on the GitHub using ABAPGIT client. Post deletion of ECATT script from SAP when I refresh in ABAPGIT I see D against the D(Local Deleted) but I don't get the pull request to pull back the ECATT script which is deleted evenif the ECATT script is present in the GitHub repository.

how did you generate the ECATT xml? via transaction SECATT?
ECATT objects are not supported by default in abapGit, see https://github.com/larshp/abapGit/wiki/Supported-object-types
also see https://github.com/larshp/abapGit/issues/163

Out tester created ECATT scripts and saved them in ZTEST1 package. When I clone the repository, it gives shown me the ECATT scripts objects as well to stage to GitHub repository.
I thought if abagit doesnt support any object it will not appear in the abapgit to transfer to GitHub repository :-)

you can try it at your end as well, create some ECAT scripts via SECATT capture them in any Z package and try to clone that package via abapgit, it automatically creates xml file of ECAT object

yes, ECATT scripts are currently not supported by abapGit
abapGit lists all files in the repository, so it is possible to ignore the file

I committed .xml files from one Dev environment and tried importing to another Dev environment it successfully imported them, I can send you my test case document if you are interested to see :-)
However in your issue #163 you mentioned below which I did:
try using https://github.com/larshp/abapGit-Plugins it might support eCATT objects

okay, plugins, can you add a step by step guide on how to reproduce the issue?

Follow below steps:

Create any ECATT test object and capture it in any package
Clone that package to any repository using ABAPGIT
Stage and commit the ECATT test objects using ABAPGIT
Delete the ECATT test object from SE80
Run the ABAPGIT again, we will see D against the local system
As we have deleted the object from local system but it still present on GitHub/Remote system when I refresh I was expecting the pull option enabled in ABAPGIT which I don't see in ABAPGIT

Kindly advise

Hi,
If you like to remove the object from the remote repo, then click "Stage", this follows the same approach/logic as command line git.
If you like to have the object created again from remote then click "Advanced" -> "Reset local", again same logic as command line

I tried above option, it works for other supported objects except ECATT objects :-)
in case of ECATT it doesn't do anything.

oh, okay, might be a bug in abapGit-Plugins, will take a look

@prashantagl fixed, try again, latest abapGit and abapGit-Plugins versions

fixed in https://github.com/larshp/abapGit-Plugins/commit/4b6f611a22d1b309fa307ab1aa290e27eb7e2c67

Thanks!
Will update the code and update you

Tested with updated code, still no pull request but reset local option worked to restore the deleted ECATT scripts.
Pull option could have been better, as reset to local will be dangerous in scenarios where there is mismatch between local and remote code as it will reset all local objects.

pull and reset local actually triggers the same code, and there are(should work) warnings if mismatches exist

In case of command line git, the git pull does not restore deleted files, see below, will close this issue


Add codecov badge

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

Comparison is base (65e84d2) 93.06% compared to head (0020e79) 93.06%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master       #7   +/-   ##
=========================================
  Coverage     93.06%   93.06%
  Complexity      774      774
=========================================
  Files            49       49
  Lines          2350     2350
  Branches        335      335
=========================================
  Hits           2187     2187
  Misses          100      100
  Partials         63       63

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


[Hotfix] Case summaries for lead assessors

This is a hotfix to allow the lead assessors download the case summaries
It also updates the views.

 [QAE16IMP] Add case summary review to LEAD assessor reports view


How can i change the batch-size?

How can i change the batch-size?

you can configure the batch size and the associated learning rate and training iters in the config file. It is easy to find :)


Pipeline fails to start with "could not read Username for ..."

Component
server, agent
Describe the bug
Possibly related to #1905 since I see some similar logs.
This is a fresh install, and the pipelines fail at the clone stage with:
fatal: could not read Username for https://[forgejo_domain] No such device or address
exit status 128

OAuth works fine. I can log out and back in with no issues. Only the agent running the workflow seems to be having the issue.
Steps to reproduce
The setup is the following on two hosts.
Host A
This host houses Forgejo, the Woodpecker server, and the Caddy proxy for tls.
services:
  services:
  woodpecker:
    image: woodpeckerci/woodpecker-server:latest
    container_name: woodpecker
    ports:
      - 3001:8000
      - 9000:9000
    restart: unless-stopped
    networks:
      - forgejo
    extra_hosts:
      - [forgejo_domain]:<IP_ADDRESS>
    environment:
      # generic
      - WOODPECKER_OPEN=true
      - WOODPECKER_HOST=https://[woodpecker_domain]
      - WOODPECKER_ADMIN=...
      - WOODPECKER_AUTHENTICATE_PUBLIC_REPOS=true
      # forgejo
      - WOODPECKER_FORGEJO=true
      - WOODPECKER_FORGEJO_URL=https://[forgejo_domain]
      - WOODPECKER_FORGEJO_CLIENT=...
      - WOODPECKER_FORGEJO_SECRET=...
      # agent
      - WOODPECKER_MAX_WORKFLOWS=1
      - WOODPECKER_AGENT_SECRET=...
    volumes:
      - /mnt/ssd/woodpecker:/var/lib/woodpecker
      -
  forgejo:
    image: codeberg.org/forgejo/forgejo:9.0
    container_name: forgejo
    environment:
      - USER_UID=1000
      - USER_GID=1000
      - FORGEJO__database__DB_TYPE=postgres
      - FORGEJO__database__HOST=postgres-forgejo:5432
      - FORGEJO__database__NAME=forgejo
      - FORGEJO__database__USER=forgejo
      - FORGEJO__database__PASSWD=forgejo
    restart: unless-stopped
    networks:
      - forgejo
    extra_hosts:
      - [woodpecker_domain]:<IP_ADDRESS>
    volumes:
      - /mnt/ssd/forgejo:/data
      - /etc/timezone:/etc/timezone:ro
      - /etc/localtime:/etc/localtime:ro
    ports:
      - "3000:3000"
      - "222:22"
    depends_on:
      - forgejo_db

Host B
This host is on the same LAN, and runs the woodpecker agent.

Additional context
The relevant logs when the pipeline starts:
Server logs
{"level":"error","repo_id":"1","pipeline_id":"7","workflow_id":"7","error":"sql: no rows in result set","time":"2024-10-22T07:40:19Z","message":"queue.Done: cannot ack workflow"}
{"level":"error","repo_id":"1","pipeline_id":"7","workflow_id":"7","error":"stream: not found","time":"2024-10-22T07:40:19Z","message":"done: cannot close log stream for step 38"}
{"level":"error","repo_id":"1","pipeline_id":"7","workflow_id":"7","error":"stream: not found","time":"2024-10-22T07:40:19Z","message":"done: cannot close log stream for step 39"}
{"level":"error","repo_id":"1","pipeline_id":"7","workflow_id":"7","error":"stream: not found","time":"2024-10-22T07:40:19Z","message":"done: cannot close log stream for step 40"}
{"level":"error","repo_id":"1","pipeline_id":"7","workflow_id":"7","error":"stream: not found","time":"2024-10-22T07:40:19Z","message":"done: cannot close log stream for step 41"}
{"level":"error","repo_id":"1","pipeline_id":"7","workflow_id":"7","error":"stream: not found","time":"2024-10-22T07:40:19Z","message":"done: cannot close log stream for step 42"}

Agent logs
{"level":"warn","repo":"haath/rust-template","pipeline":"7","workflow_id":"7","error":"rpc error: code = Unknown desc = workflow finished with error uuid=01JASJPRDXKEBT5A0V7PP73QCR: exit code 128","time":"2024-10-22T07:40:19Z","message":"cancel signal received"}

Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Checked that the bug isn't fixed in the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]

Nevermind, I had some local DNS issue.


Add more theorems about Array.swap's basic properties

Here is a description of each theorem being added. Given some array:

If you swap element at position i with that at j, then element at j in the new array (after swap) will be the element that was at i in the old array (before the swap)
If you swap element at position i with that at j, then element at i in the new array will be the element that was at j in the old array
If you swap element at position i with that at j, then all elements that are neither at positions i nor j remain unchanged from what they were before the swap
If you swap element at position i with that at j, get a new array, then in this new array if you swap element at position i with that at j again, what you get is the original array you had before any of the swaps
If you swap element at position i with that at j, what you get is the same as if you instead had swapped element at position j with that at i

awaiting-review

WIP

awaiting-review

WIP

awaiting-review

WIP

awaiting-review

WIP

@semorrison awaiting-review

awaiting-review

lgtm


copy-formatted-for-elm-repl: copy selected code into clipboard formatted for pasting into repl

I'd like to be able to copy selected multiline Elm code into the clipboard and be able to paste it into the elm repl formatted to be able to be successfully evaluated.
I'm curious whether this is possible ... and how hard it would be -- or maybe it already exists??
For example this code copied and pasted into the elm repl generates an error:
type alias Option =
    { id : Int
    , name : Stri

Adding  \ at the end of each line except for the last allows the repl to evaluate it successfully:
type alias Option = \
    { id : Int \
    , name : String \
    }

It seems like copying with newlines escaped could be useful for a text editor in general. Maybe it could be made into its own package? (I didn't see an existing package like this during a quick search).

@stepheneb, the feature shouldn't be hard to implement, but it would be a special copy command (e.g. Elmjutsu: Copy For Repl).  Are you okay with that?

That would be fine!
Also if you want to outline where changes should be made I'm also happy to
try and implement the feature and generate a pull request.
On Tue, May 21, 2019 at 4:24 AM halohalospecial<EMAIL_ADDRESS>wrote:

@stepheneb https://github.com/stepheneb, the feature shouldn't be hard
to implement, but it would be a special copy command (e.g. Elmjutsu: Copy
For Repl). Are you okay with that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/halohalospecial/atom-elmjutsu/issues/167?email_source=notifications&email_token=AAABYFFV4RUQACUXHOJ6BGDPWOWSJA5CNFSM4HL6ATHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV3EWYQ#issuecomment-494291810,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABYFB4O5DC33O4JV3YJKLPWOWSJANCNFSM4HL6ATHA
.

--
Be curious and kind whenever possible, https://learnmaketeachshare.org/

@stepheneb, you can add the command in lib/main.js
Somewhere here:
this.subscriptions.add(
      atom.commands.add(
        'atom-text-editor:not([mini])[data-grammar^="source elm"]',
        {
          'elmjutsu:quick-fix-file': () => this.quickFix.quickFixFileCommand(),
          'elmjutsu:set-main-paths': () =>
            this.elmMakeRunner.setMainPathsCommand(),
          'elmjutsu:set-compile-output-path': () =>
            this.elmMakeRunner.setCompileOutputPathCommand(),
          'elmjutsu:go-to-definition': () => this.goTo.goToDefinitionCommand(),
          'elmjutsu:go-to-symbol': () => this.goToSymbol.goToSymbolCommand(),
          'elmjutsu:find-usages': () =>
            this.findAndRenameUsages.findUsagesCommand(),


Slider: aria-label for the handles

Add the aria-label for the handles via a prop.
This PR re-activate the axe tests for the slider (which failed without it).

Thanks @quentinderoubaix !


CSS reset

Is it possible to enable some CSS reset?
Like:
-webkit-tap-highlight-color:transparent;
To disable the blue color on click on gmail android/ios:

And:
-webkit-appearance:none;
to remove on gmail  iOS:

We discussed this in January's meeting and decided to allow these, so both -webkit-tap-highlight-color and -webkit-appearance should now be supported in Gmail.

Great! Thank you for your support and attention!


how to dispatch actions

in a.vue page
eg :how to dispatch actions
created(){
   this.$store.dispatch('refreshToken') //unknown action type: refreshToken
}

answer by my  self:
this.$store.dispatch('user/refreshToken')

answer by my self:
this.$store.dispatch('user/refreshToken')

you also can dispatch action by util function,like this
dispatchActions<UserActionsType>('user','refreshToken')