fix: render CurrencySelect component only when currencies are available

Hey, I just made a Pull Request!
Fixes: https://github.com/spotify/backstage/issues/3134
The cost insights page is not loading because we are getting currencies as undefined and we are trying to loop over the undefined using map method in CurrencySelect component, now we will only show CurrencySelect component if currencies area available from config.
Screenshots
Before

After

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)

Oof. Nice catch @abhishekjakhar. Currencies should never be undefined as we ship a set of currencies "pre-configured" (loaded in state but not actually derived from the config itself). These are initialized in the default state of the useConfig hook.
Seems like there is a bug related to the import resolution of defaultCurrencies from its type file, as well as enums and other non-type definitions writ large, as all of these imports are now returning undefined. This can be reproduced for other non-type definition imports in other files.

Oof. Nice catch @abhishekjakhar. Currencies should never be undefined as we ship a set of currencies "pre-configured" (loaded in state but not actually derived from the config itself). These are initialized in the default state of the useConfig hook.
Seems like there is a bug related to the import resolution of defaultCurrencies from its type file, as well as enums and other non-type definitions writ large, as all of these imports are now returning undefined. This can be reproduced for other non-type definition imports in other files.

If set of currencies are always "pre-configured" then I don't think we require this check. I was confused between adding this check or using defaultCurrencies from types as fallback.

The feature should still be behind a feature flag but yes, the nullish checks aren't necessary as config.currencies should always be defined.

The feature should still be behind a feature flag but yes, the nullish checks aren't necessary as config.currencies should always be defined.

In the useConfig hook we are exporting products, metrics, engineerCost and icons. But, are not exporting currencies. We have imported currencies from types but I guess we forgot to export them from the hook. I will export the currencies form hook and remove the nullish checks.

@abhishekjakhar The useConfig hook exports the hook, context and type definition. Products, metrics, etc. are properties of the config context. defaultCurrencies is defined and exported in Currency.ts.
The underlying issue seems to be that the export of defaultCurrencies from Currency.ts is returning undefined in useConfig - in fact, all non-type exports from type definition files are returning undefined - so the config state which is defined outside of the component, is being initialized incorrectly. At runtime when the component mounts, these references resolve but the Provider mounts with an incorrect state.
It appears this issue affects components who have similar dependencies on these non-type exports. Exports should always resolve, so the remediation for this will likely require migrating these non-types (enums, consts, etc.) to their own files.

@abhishekjakhar The useConfig hook exports the hook, context and type definition. Products, metrics, etc. are properties of the config context. defaultCurrencies is defined and exported in Currency.ts.
The underlying issue seems to be that the export of defaultCurrencies from Currency.ts is returning undefined in useConfig - in fact, all non-type exports from type definition files are returning undefined - so the config state which is defined outside of the component, is being initialized incorrectly. At runtime when the component mounts, these references resolve but the Provider mounts with an incorrect state.
It appears this issue affects components who have similar dependencies on these non-type exports. Exports should always resolve, so the remediation for this will likely require migrating these non-types (enums, consts, etc.) to their own files.

Could I take sometime and work on this?

This issue is being take care of in https://github.com/spotify/backstage/pull/3147 PR

Closing this PR because this issue is being take care of in #3147 PR


Capture UnauthorizedServiceException by AuditableEnforcer

Adds UnauthorizedServiceException to the list of Exceptions captured by RegisteredServiceAccessStrategyAuditableEnforcer  specifically for the cases where a service is disabled in the service registry.  This will allow all calling methods to handle enforcer calls the same by checking the returned result for a captured exception and eliminates the need to code a try/catch block.

... for the cases where a service is disabled in the service registry.

Could you add a few test cases to demonstrate this?

Will do


More sophisticated version selection

The user might specify allowable Julia versions using a semver syntax. We could borrow code from PythonCall.jl

I think this was actually done a couple of weeks after this issue was opened.
See https://github.com/jlapeyre/julia_semver


Which service should GatewayServer use, do I have to configure  first?

Yes, you need to configure forwarding.

是的，要配置转发吧。


Puppet6

Coverage decreased (-0.5%) to 94.828% when pulling 6a906aedda0e8b0b6ade6d3c1422a23efc838f69 on raphink:puppet6 into d5b70e400d1f2dbb5f022a74fa23048e89b26c01 on hercules-team:master.


Exception thrown when you change encoding and get a non-buffer in the data event

@mikeal @isaacs this breaks nano's binary tests - returns an exception
Here is the stack trace plus instructions to reproduce:
https://gist.github.com/ff40d9ce9b73d9394702
Is this my mistake, did some assumption in request changed without me noticing it?
Nuno

this still an issue?


using other alternative relays

Hi, I tried using other relays just to see how it works but when using other relays it doesn't. it seems it's only with your default relay that works?.. is this using a specific thing that other relays don't understand?

Senstr only uses kind 4 events currently so it should work with all relay implementations. What relay did you try? I tested with wss://nostr-pub.wellorder.net and it's working for me.

somehow now it's not working with any of the relays to me. I tried the one you mentioned and the default one. maybe it's because it's all exploding now with so much demand?

I'm not able to reproduce the issue with the provided details so closing this issue. I set up an issue template for bugs so you can open another issue with device/browser/relay details and screenshots if you want.


wrap stdio of workload containers

Instead of redirecting workload container stderr and stdout to vhive stderr and stdout, it is now logged as:
time="2021-07-02T16:52:41.467607817Z" level=info msg="ERROR:root:testing logging2" vmID=15

Tested with pyaes code / image@docker hub.

Changelog updated. I can't check your marks though (by clicking them).
Why the test is failing, i don't understand yet.

please add a spelling exception to configs/.wordlist* file to fix the linter error


Dont set docker DNS

what
doing a assume-role in a geodesic container while on a network that does not allow using any external DNS servers directly, which is very common in corporate networks, results in a timeout for sts.amazonaws.com
-> Run 'assume-role' to login to AWS
 ⧉  users.aws.example.org
 ✗   (none) ~ ⨠  assume-role
Enter passphrase to unlock /conf/.awsvault/keys/:
Enter token for arn:aws:iam::883075610859:mfa/bootStrap: 395604
2018/12/05 13:55:44 Request body type has been overwritten. May cause race conditions
2018/12/05 13:55:54 Request body type has been overwritten. May cause race conditions
2018/12/05 13:56:04 Request body type has been overwritten. May cause race conditions
aws-vault: error: Failed to get credentials for example (source profile for example-users-bootstrap): RequestError: send request failed
caused by: Post https://sts.amazonaws.com/: dial tcp: lookup sts.amazonaws.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:40888-><IP_ADDRESS>:53: i/o timeout

I would not set DNS to anything as it makes the assumption that one can access google DNS.
https://github.com/cloudposse/geodesic/blob/master/rootfs/templates/wrapper#L261

@jdn-za agreed - I'm going to drop that. I think it's left overs from a long time ago. Had some issues with DNS resolution in docker.

For the record, it was implemented due to https://github.com/docker/docker/issues/24344

Thanks gents


Update CMake to make pybind_python automatically

#307  fixes this issue

@LakshmiKumar23 please test this with the PR and let me know if this works for you.

Fixed on TOT


Artemis will sometimes use more than 50 mA

https://forum.sparkfun.com/viewtopic.php?f=170&t=59681

the same program will use 4 mA on the regular redboard Artemis, so this is a bug somewhere!
#124

Some are just bad.


Allow width of the clipboard window to be set

Allow width of the clipboard window to be set
Thanks for consideration

Why?

@p0deje it's just a visual thing.
it would be very helpful too be able to set the width higher for high resolution displays especially.
I'd like to see more of the clipboard contents for long texts.
Also it is suitable to adjust to all my other applications where I have set a bigger width than the defaults.
Thank you for consideration.


Bug(eos_cli_config_gen): BGP vlan config should not have redistribute igmp when belonging to a VRF with evpn multicast

having redistribute igmp for a svi/vlan belonging to a VRF with evpn multicast is an issue because it could cause conflicts in the control-plane as far I know.

Originally posted by @krikoon73 in https://github.com/aristanetworks/avd/issues/3462#issuecomment-1884638318

We may have to implement this behavioral change behind a knob, or wait for 5.0.


Routing for static pages/routes (SSR deployments)

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[x] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d2nv66ftvk25sd
AWS Region
us-east-1
Amplify Hosting feature
SSR
Question
Hi I have an Astro adapter using the SSR deployment specification.
Is there a way of handling static pages (routes)? For example, an /about/ page served from .amplify-hosting/static/about/index.html. I've played around with various routes (in the deploy manifest) but haven't been able to find a good solution. If you can check the appId I've listed - click on the "Static" link in the production URL.
At the moment, the workaround I'm using is a manual rewrite for each route e.g. for a static about page with /about/ /about/index.html 200 (Rewrite) - so once it gets rewritten, it hits the static route pattern and works as expected:
"path": "/*.*",
"target": {
    "kind": "Static"
},

This would be ok if rewrites could be set from a file (so during the build process, I could just configure these known static routes automatically). But based on this issue, that's not possible yet. Or is there a single rule I could use for all these cases? Thanks for any assistance.

I see, that's great to hear. Thanks.


Output for version

Hi there,
I use the phpmd.phar I downloaded from http://static.phpmd.org/php/2.1.3/phpmd.phar.
The xml-output gives me something like this:
[...]
<pmd version="@project.version@" timestamp="2014-11-27T15:21:22+01:00">
[...]

Is there something I have to configure to make the output of the version correct?

I suspect this could be set in the build script for the PHAR file.

For the record, it's not replaced either using ./vendor/bin/phpmd

#743

@kylekatarnls this issue can be closed


nbdserver: support NBD_CMD_TRIM (TrimAt) in backend

Currently the interface (Backend).TrimAt method is not implemented by the ardb.Backend implementation. This doesn't cause any errors, as the command is more meant as a hint rather than a command, and thus the server is free to ignore it. However it could allow us to safe space in case clients do call it. Here is what the documentation says about NBD_CMD_TRIM:

NBD_CMD_TRIM (4)
A hint to the server that the data defined by len and offset is no
longer needed. A server MAY discard len bytes starting at offset, but
is not required to.
After issuing this command, a client MUST NOT make any assumptions
about the contents of the export affected by this command, until
overwriting it again with NBD_CMD_WRITE or NBD_CMD_WRITE_ZEROES.
A client MUST NOT send a trim request unless NBD_FLAG_SEND_TRIM
was set in the transmission flags field.
source: https://sourceforge.net/p/nbd/code/ci/master/tree/doc/proto.md#request-types

In short, it would mean:

we can delete all full blocks given on the offset;
write zeroes to the partial blocks given on the offset;

This would be implemented in: https://github.com/g8os/blockstor/blob/3bb9e3b25b09949eb1d5c56f032d0c308b16ef0e/nbdserver/ardb/backend.go#L181-L184
It's fairly easy to implement, shouldn't take much time, and it's an official nbd protocol command, so I don't think we have any reason to ignore it. The nbd connection code (https://github.com/g8os/blockstor/blob/3bb9e3b25b09949eb1d5c56f032d0c308b16ef0e/gonbdserver/nbd/connection.go#L481-L507) is already prepared for it, so it would just mean that we have to implement the steps mentioned above in the backend.go file.
While playing around with qemu, it did never trigger NBD_CMD_TRIM, but again, it is an official command, so although a server can ignore it, I don't see why we would do so.
Note that according to the protocol documentation the "client MUST NOT send a trim request unless NBD_FLAG_SEND_TRIM was set in the transmission flags field". At the moment this flag is not checked in the gonbdserver code, so we might as well ensure that this command is only supported in case that flag is set during the negotiation phase.

The implementation in #307 create regression. I'm unable to do these anymore
sudo nbd-client -b 4096 -name default localhost 6666 /dev/nbd1
sudo mkfs.ext4 -b 4096 /dev/nbd1
sudo mount /dev/nbd1 /mnt/sharedvolume

reverting the commit make it work again.

As discussed on chat, if you don't immediately see a solution, feel free to bump this issue back to the roadmap, after you reverted the commit on master. As it's done via a PR, you can even do that via GitHub.

Commit is reverted, moved this issue to roadmap


How to query the output?

How can we search amongst the output . for example querying all the techniques or tests which contain the word of "SMB".
Is there a way to do it directly without saving the output file?
I want to find all the Tests that are related to "SMB" for example

My suggestion is to use the search feature provided on GitHub. In this case you could go to the Atomic Red Team Project and search for SMB.


Intellisense Not Working in my ES6 project

Its been a week since intellisense stopped working on my projects. It doesn't predict any functions for the installed npm modules.
I have restarted VScode several times and nothing happened. Right now I am using jetbrains fleet to do my work.
Is there any solution to it?

VS Code Version: 1.72.2
OS Version: Windows 11

Does this reproduce in the latest VS Code insiders build with all extensions disabled?
If it does, please share a minimal example project that demonstrates the issue

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Ceremony should not try to sync from PR

Description

https://github.com/sigstore/root-signing/blob/main/.github/workflows/sync-ceremony-to-main.yml#L38 fires when a push occurs from a ceremony branch and updates repository metadata. However, the workload identity pool is configured to only accept requests from the main branch, rather than ceremony branches. We could loosen this restriction, but it opens us up to insider risk, if a maintainer creates a change on the ceremony branch without a review.
We should remove https://github.com/sigstore/root-signing/blob/main/.github/workflows/sync-ceremony-to-main.yml#L66-L127 and create another GHA like https://github.com/sigstore/root-signing/blob/main/.github/workflows/sync-main-to-preprod-and-prod.yml to sync main to preprod (and only preprod) after a ceremony change.

This was resolved.


title not switched to titleSortified for ordering

I'm not sure if this is actually a bug, but on CCD indexes weren't switching title to titleSortified for index ordering. Here's my fix for this: https://github.com/punkave/ccd/pull/243/commits/7c7f09867d1585f74569353e49c4345778e8131e
Note that these pages have a page setting to order the page alphabetical or by publishedAt. I think that the issue was simply that this was set explicitly to title, but if it was still supposed to switch to titleSortified that might be a bug.

If sort('title') is getting called at the end of the day, and yet the
sort comes out case sensitive, it's a bug all right.
On Thu, Nov 30, 2017 at 5:43 PM, Alex Bea<EMAIL_ADDRESS>wrote:

I'm not sure if this is actually a bug, but on CCD indexes weren't
switching title to titleSortified for index ordering. Here's my fix for
this: punkave/ccd@7c7f098
https://github.com/punkave/ccd/commit/7c7f09867d1585f74569353e49c4345778e8131e
Note that these pages have a page setting to order the page alphabetical
or by publishedAt. I think that the issue was simply that this was set
explicitly to title, but if it was still supposed to switch to
titleSortified that might be a bug.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe/issues/1139, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fRibIY8318uyVG-zxeMT5YcVmOcLks5s7y-vgaJpZM4QxUSb
.

--
THOMAS BOUTELL, CHIEF SOFTWARE ARCHITECT
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

... Unless somebody redefined title with addFields and left off
sortify: true.
On Fri, Dec 1, 2017 at 10:29 AM, Tom Boutell<EMAIL_ADDRESS>wrote:

If sort('title') is getting called at the end of the day, and yet the
sort comes out case sensitive, it's a bug all right.
On Thu, Nov 30, 2017 at 5:43 PM, Alex Bea<EMAIL_ADDRESS>wrote:

I'm not sure if this is actually a bug, but on CCD indexes weren't
switching title to titleSortified for index ordering. Here's my fix for
this: punkave/ccd@7c7f098
https://github.com/punkave/ccd/commit/7c7f09867d1585f74569353e49c4345778e8131e
Note that these pages have a page setting to order the page alphabetical
or by publishedAt. I think that the issue was simply that this was set
explicitly to title, but if it was still supposed to switch to
titleSortified that might be a bug.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/apostrophe/issues/1139, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fRibIY8318uyVG-zxeMT5YcVmOcLks5s7y-vgaJpZM4QxUSb
.

--
THOMAS BOUTELL, CHIEF SOFTWARE ARCHITECT
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

--
THOMAS BOUTELL, CHIEF SOFTWARE ARCHITECT
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

@abea can you confirm whether the project in question has overridden title via addFields? If someone did that and left out sortify: true, we would see this behavior and it would not be an Apostrophe bug.


Authorization for importing repositories

I need to import Git repositories from other DevOps organizations that require authorization. How do I structure my request for this?

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

ID: 330cdef3-6483-e7a5-09e1-5689836984d6
Version Independent ID: 66256ae3-7e9a-ba6e-a2a7-3dd02dc0117d
Content: Import Requests (Azure DevOps Git)
Content Source: docs-ref-autogen/5.0/git/Import%20Requests.yml
Product: devops
Technology: devops-ref
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

Hey, I have the same question, how do I migrate my git repos to azure devops with authentication?

do we have any update on this?

To import repositories from another DevOps organization (or another Git host in general) you have to take two steps:

First you have to create a Git endpoint to that specific repository. You can find this operation at https://docs.microsoft.com/en-us/rest/api/azure/devops/serviceendpoint/endpoints/create?view=azure-devops-rest-5.1. The details for the correct request you can reverse engineer from the UI or take a peek at what I use:

var json = JsonConvert.SerializeObject(new
            {
                Name = name,
                Type = "git",
                Url = gitUrl,
                Authorization = new
                {
                    Scheme = "UsernamePassword",
                    Parameters = new
                    {
                        Username = username,
                        Password = password
                    }
                },
                isReady = true
            });

Once the service connection is created and you have captured it's id that is returned. You can start an import request (https://docs.microsoft.com/en-us/rest/api/azure/devops/git/import requests/create?view=azure-devops-rest-5.1) like this:

            var json = JsonConvert.SerializeObject(new
            {
                Parameters = new
                {
                    DeleteServiceEndpointAfterImportIsDone = true,
                    GitSource = new
                    {
                        Overwrite = false,
                        Url = sourceGitUrl
                    },
                    ServiceEndpointId = serviceEndpointId
                }
            });


Type checking stops after spread operator

Bug Report
🔎 Search Terms
spread
spread operator
spread any
spread object assign
🕗 Version & Regression Information
This is the behavior in every version I tried, and I reviewed the FAQ for entries about the spread operator
⏯ Playground Link
Playground link with relevant code
💻 Code

const x: any = { a: 5 }

interface I {
  b?: string
}

let spread: I;
spread = {
  ...x,
  b: 5
}

🙁 Actual behavior
Type mismatch is missed after use of spread operator.
🙂 Expected behavior
I expect a "Type 'number' is not assignable to type 'string | undefined'.(2322)" error for b: 5.
Similar bug #46128 indicates that after spread the type is not kept.
The main difference between the bugs is that in this case, the type is declared beforehand.

Edit: removed a line of code and changed some wording to make things clearer

I'm using Angular forms which are not typed.
Now that I'm aware of this issue, I can add an extra step to the process and add a type to the object I'm spreading.
Maybe I'm wrong, but I still expect the rest of the assignments to be checked.

You have two assignments, and both are checked. { ...x, b: 5 } is typed any, and is assignable to I. Assigning any to a typed variable does not change the type of the variable. So spread is typed I, which is not assignable to never.

The never part is obvious. It just demonstrates the type is of spread is I (as expected). I can remove that part if it'll make the issue clearer.
My expectation is that { ...x, b: 5 } will be checked as two assignments: ...x which will be treated as any and b: 5 which will be treated as number.
This is what happens when you do something like this:
const x: any = { a: 5 }

interface I {
  a: string;
  b?: string;
}

let spread: I;
spread = {
  a: x.a,
  b: 5
}

The fact that any values are assigned does not interfere with following type checks.

What it looks like is that by using spread syntax, the object literal as a whole is inferred as typeof x & { b: number } (similar to the typing of Object.assign).  Since typeof x is any, the whole type collapses to any before the usual assignability check can catch it.  The non-spread version is inferred as { a: any, b: number }, so the any doesn’t take over the type in that case.

@fatcerberus this really explains it.
I guess the questions now are:

Can it be considered a bug?
Can the process be changed so it will offer more type safety?
Should I open a feature request instead of this bug?
I failed to find any documentation regarding the type inference of the spread operator (only this.
Is there any documentation about this?
Maybe adding it, or moving it to a more visible place will be sufficient.

I would say less “bug” and more “design limitation” - TS has to pick a type for the type of the object literal in order to check it (assignability checks are done between types, not values; one assignment = one typecheck), and intersecting anything with any gives back any.  There’s no mechanism for typechecking individual properties of an object assignment, even with a spread.
One could make the case that T & any should be T for all T, but this would actually prevent your spread entirely - it would be treated as { b: number }.  So IMO the current behavior is the lesser of two evils given that any is meant to be the “shut up and leave me alone” type.

Treating it as { b: number } is actually not a bad idea IMHO. Because if you want to get any, you can always cast it to.
spread = {
  ...x,
  b: 5
} as any

It's just unexpected when there's no "temporary" variable that could be thought of as an any type.
let x: { foo: number } = { foo: 1, baz: 2 }; // Error, excess property `baz`

will error while following works:
let baz = { baz: 2 }
let x: { foo: number } = { foo: 1, ...baz}; // OK

Why would object with spreading be any even if it uses types?


I get an error when doing meteor npm install

2017-12-08T14_54_07_251Z-debug.log
I have this log when I tried to open it with meteor npm install

Ah, it was some problem with package-lock.json. If you delete it it should work.

OK I got it working maybe you shouldn't commit this package.json file ! or at least tell to remove it in instructions.
Otherwise, I launched the beast, and it feels really good, thanks for providing this !!!

You're supposed to include it to make sure that everyone uses exactly the same package versions. I removed it from this repo though, since it's just a simple demo anyway.


React warning on first time npm start

After initial npm start I'm getting this warning in browser console.
VM28315:36 Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
 (client) <!-- react-empty: 1 -
 (server) <div data-reactroot="

Look at the end of the readme.md
Client and Server Markup Mismatch
This warning is visible only on development and totally harmless. This occurs to hash difference in react-router. To solve it, react router docs asks you to use match function. If we use match, react-hot-reloader stops working.

There is a workaround to make the warning go away, as explained here #149


ruff server does not support E30* errors?

ruff version: 0.3.5
lsp ocnfig:
return {
  filetypes = { "python" },
  single_file_support = true,
  cmd = { "ruff", "server", "--preview", "--config", "~/.config/nvim/pyproject.toml" },
  settings = {
    args = options.ruff_args,
  },
}

pyproject.tmol
[tool.ruff.lint]
# Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`)  codes by default.
# Unlike Flake8, Ruff doesn't enable pycodestyle warnings (`W`) or
# McCabe complexity (`C901`) by default.
select = ["E4", "E7", "E9", "F"]
ignore = ["E402","E501","N802","N803","N806","N801","N813","N815","N816","RUF001","RUF002","RUF003","RUF012","ARG001"]
extend-select = ["E", "N", "W", "ARG", "RUF"]

# Allow fix for all enabled rules (when `--fix`) is provided.
fixable = ["ALL"]
unfixable = []

# Allow unused variables when underscore-prefixed.
dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"

but i did not get any errors

Can you try adding preview = true to your pyproject.toml, under [tool.ruff]?

@charliermarsh
[tool.ruff]
preview = true
# Exclude a variety of commonly ignored directories.
exclude = [
    ".bzr",
    ".direnv",
    ".eggs",
    ".git",
    ".git-rewrite",
    ".hg",
    ".ipynb_checkpoints",
    ".mypy_cache",
    ".nox",
    ".pants.d",
    ".pyenv",
    ".pytest_cache",
    ".pytype",
    ".ruff_cache",
    ".svn",
    ".tox",
    ".venv",
    ".vscode",
    "__pypackages__",
    "_build",
    "buck-out",
    "build",
    "dist",
    "node_modules",
    "site-packages",
    "venv",
]

# Same as Black.
line-length = 120
indent-width = 4

It's still not working.

@a1401358759 did you keep your existing lint configuration?
[tool.ruff]
preview = true
# Exclude a variety of commonly ignored directories.
exclude = [
    ".bzr",
    ".direnv",
    ".eggs",
    ".git",
    ".git-rewrite",
    ".hg",
    ".ipynb_checkpoints",
    ".mypy_cache",
    ".nox",
    ".pants.d",
    ".pyenv",
    ".pytest_cache",
    ".pytype",
    ".ruff_cache",
    ".svn",
    ".tox",
    ".venv",
    ".vscode",
    "__pypackages__",
    "_build",
    "buck-out",
    "build",
    "dist",
    "node_modules",
    "site-packages",
    "venv",
]

# Same as Black.
line-length = 120
indent-width = 4

[tool.ruff.lint]

# Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`)  codes by default.
# Unlike Flake8, Ruff doesn't enable pycodestyle warnings (`W`) or
# McCabe complexity (`C901`) by default.
select = ["E4", "E7", "E9", "F"]
ignore = ["E402","E501","N802","N803","N806","N801","N813","N815","N816","RUF001","RUF002","RUF003","RUF012","ARG001"]
extend-select = ["E", "N", "W", "ARG", "RUF"]

# Allow fix for all enabled rules (when `--fix`) is provided.
fixable = ["ALL"]
unfixable = []

# Allow unused variables when underscore-prefixed.
dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"

@MichaReiser this is my whole config
[tool.ruff]
preview = true
# Exclude a variety of commonly ignored directories.
exclude = [
    ".bzr",
    ".direnv",
    ".eggs",
    ".git",
    ".git-rewrite",
    ".hg",
    ".ipynb_checkpoints",
    ".mypy_cache",
    ".nox",
    ".pants.d",
    ".pyenv",
    ".pytest_cache",
    ".pytype",
    ".ruff_cache",
    ".svn",
    ".tox",
    ".venv",
    ".vscode",
    "__pypackages__",
    "_build",
    "buck-out",
    "build",
    "dist",
    "node_modules",
    "site-packages",
    "venv",
]

# Same as Black.
line-length = 120
indent-width = 4

[tool.ruff.lint]
# Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`)  codes by default.
# Unlike Flake8, Ruff doesn't enable pycodestyle warnings (`W`) or
# McCabe complexity (`C901`) by default.
select = ["E4", "E7", "E9", "F"]
ignore = ["E402","E501","N802","N803","N806","N801","N813","N815","N816","RUF001","RUF002","RUF003","RUF012","ARG001"]
extend-select = ["E", "N", "W", "ARG", "RUF"]

# Allow fix for all enabled rules (when `--fix`) is provided.
fixable = ["ALL"]
unfixable = []

# Allow unused variables when underscore-prefixed.
dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"

[tool.ruff.format]
# Like Black, use double quotes for strings.
quote-style = "double"

# Like Black, indent with spaces, rather than tabs.
indent-style = "space"

# Like Black, respect magic trailing commas.
skip-magic-trailing-comma = false

# Like Black, automatically detect the appropriate line ending.
line-ending = "auto"

# Enable auto-formatting of code examples in docstrings. Markdown,
# reStructuredText code/literal blocks and doctests are all supported.
#
# This is currently disabled by default, but it is planned for this
# to be opt-out in the future.
docstring-code-format = false

# Set the line length limit used when formatting code snippets in
# docstrings.
#
# This only has an effect when the `docstring-code-format` setting is
# enabled.
docstring-code-line-length = "dynamic"

Hm okay. This works for me but I use the old LSP (and vscode).
Can you try removing the
settings = {
    args = options.ruff_args,
  },

because the new server doesn't yet support options (but I'm just guessing at this point).
@snowsignal would you mind helping out?

@MichaReiser  I try removing the
settings = {
    args = options.ruff_args,
  },

but it's still not working,ruff-lsp works good for me.

Hey @a1401358759, I don't think the new server supporst configuring Ruff via the global --config flag or a local configuration file.
@snowsignal Correct me if I'm wrong here.

@a1401358759 Right now the new server has a few limitations, especially around configuration. At the moment, we don't support configuring the server with the --config flag, nor do we support the args field in settings (like @dhruvmanila and @MichaReiser mentioned). Alternative configuration is something we're actively working on, but at the moment the configuration just defaults to any pyproject.toml / ruff.toml files at the root of your workspace or in parent folders.
Thank you for opening this issue - I'll follow up once we have a setting like this available.

@snowsignal Okay, thank you very much for your answer.

@a1401358759 Once https://github.com/astral-sh/ruff/pull/11225 lands in the next release, you should be able to specify a custom configuration file!
Here's what your LSP config could look like:
return {
  filetypes = { "python" },
  single_file_support = true,
  cmd = { "ruff", "server", "--preview", },
  settings = {
    configuration = "~/.config/nvim/pyproject.toml"
  },
}

The configuration setting is what you should use to specify the configuration file.
Let me know if you run into any more issues!

my lsp config
return {
  filetypes = { "python" },
  single_file_support = true,
  cmd = { "ruff", "server", "--preview" },
  settings = {
    configuration = "~/.config/nvim/pyproject.toml",
  },
}

my pyprohect.toml
[tool.ruff]
preview = true
# Exclude a variety of commonly ignored directories.
exclude = [
    ".bzr",
    ".direnv",
    ".eggs",
    ".git",
    ".git-rewrite",
    ".hg",
    ".ipynb_checkpoints",
    ".mypy_cache",
    ".nox",
    ".pants.d",
    ".pyenv",
    ".pytest_cache",
    ".pytype",
    ".ruff_cache",
    ".svn",
    ".tox",
    ".venv",
    ".vscode",
    "__pypackages__",
    "_build",
    "buck-out",
    "build",
    "dist",
    "node_modules",
    "site-packages",
    "venv",
]

# Same as Black.
line-length = 120
indent-width = 4

[tool.ruff.lint]
# Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`)  codes by default.
# Unlike Flake8, Ruff doesn't enable pycodestyle warnings (`W`) or
# McCabe complexity (`C901`) by default.
select = ["E4", "E7", "E9", "F"]
ignore = ["E402","E501","N802","N803","N806","N801","N813","N815","N816","RUF001","RUF002","RUF003","RUF012","ARG001"]
extend-select = ["E", "N", "W", "ARG", "RUF"]

# Allow fix for all enabled rules (when `--fix`) is provided.
fixable = ["ALL"]
unfixable = []

# Allow unused variables when underscore-prefixed.
dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"

[tool.ruff.format]
# Like Black, use double quotes for strings.
quote-style = "double"

# Like Black, indent with spaces, rather than tabs.
indent-style = "space"

# Like Black, respect magic trailing commas.
skip-magic-trailing-comma = false

# Like Black, automatically detect the appropriate line ending.
line-ending = "auto"

# Enable auto-formatting of code examples in docstrings. Markdown,
# reStructuredText code/literal blocks and doctests are all supported.
#
# This is currently disabled by default, but it is planned for this
# to be opt-out in the future.
docstring-code-format = false

# Set the line length limit used when formatting code snippets in
# docstrings.
#
# This only has an effect when the `docstring-code-format` setting is
# enabled.
docstring-code-line-length = "dynamic"

but ruff server still doesn't check E30* errors

Is there still a problem with my configuration?


Votes not recorded

Discord bot isn't registering any votes

Vote now recorded, but validation isn't working :hammer:


proposal: errors: add NewJoinedErrors

Description
Sometimes I miss the capabilities of the standard library to handle errors. Wrapping an error in context with fmt.Errorf("some context: %w",err) is a great feature. But there are situations where having multiple errors of the form
var errMyError1 = errors.New("my error 1")
var errMyError2 = errors.New("my error 2")

I need to pack them both into an error followed by the ability to detect this fact with
errors.Is(err, errMyError1)
errors.Is(err, errMyError2)

Example Implementation
At the moment, I'm using my own type to solve such problems:
type joinedErrors struct {
	errExt error
	errInt error
}

func (e joinedErrors) Error() string {
	return fmt.Sprintf("%s: %s", e.errExt, e.errInt)
}

func (e joinedErrors) Unwrap() error {
	return e.errInt
}

func (e joinedErrors) Is(target error) bool {
	return errors.Is(e.errExt, target)
}

func (e joinedErrors) As(target interface{}) bool {
	return errors.As(e.errExt, target)
}

Using the following function
func NewJoinedErrors(errExt error, errInt error) error {
	return joinedErrors{errExt: errExt, errInt: errInt}
}

i can get what i want.
Example Using
someErr1 := errors.New("my error 1")
someErr2 := errors.New("my error 2")
err:=fmt.Errorf("error occured: %w", someErr1)

// errNegative1 := fmt.Errorf("%w: %w", someErr2, err) - will cause the error
// using the standard library
errNegative1 := fmt.Errorf("%w: %s", someErr2, err)
fmt.Printf("%q, %v, %v\n",errNegative1, errors.Is(errNegative1,someErr1), errors.Is(errNegative1,someErr2))
errNegative2 := fmt.Errorf("%s: %w", someErr2, err)
fmt.Printf("%q, %v, %v\n",errNegative2, errors.Is(errNegative2,someErr1), errors.Is(errNegative2,someErr2))

// using the NewJoinedErrors
errPositive := NewJoinedErrors(someErr2, err)
fmt.Printf("%q, %v, %v\n",errPositive, errors.Is(errPositive,someErr1), errors.Is(errPositive,someErr2))

You can try it here
Conclusion
I think a function like NewJoinedErrors might be useful for someone and should probably be included in the errors package.
The only drawback I see is some inconvenience in managing the text context. The context can be added for the overridden errors separately or later for the resulting error.
As a user, it would be more convenient for me to use fmt.Errorf("some context %w: and more context: %w", errExt, errInt) notation instead of the proposed construct. In this case, the errExt, errInt ratio under the hood is specified by the order of variables, which may not be quite obvious (meaning which error will be retrieved first during Unwrap).
Another option is to add a specifier for external error: fmt.Errorf("some context %w: and more context: %W", errInt, errExt), so the user knows that errExt will be extracted first, though I cannot immediately imagine a situation where this really matters.
I believe that the above changes will not spoil backward compatibility and will give error handling a more generic look.

There are various existing multierr packages, such as https://pkg.go.dev/go.uber.org/multierr.  It's not clear why this needs to be in the standard library.
This may be a dup of #47811.


CI: Set PYTHON_COLORS=0 to disable coloured CPython tracebacks

Possible alternative to https://github.com/nedbat/coveragepy/commit/75b22f0e24559446ff551acb6ddbaa92bce8add3.
Instead of stripping colour escape sequences from the Python 3.13 tracebacks, set PYTHON_COLORS=0 to disable only the CPython colour output on the CI.
https://docs.python.org/3.13/using/cmdline.html#envvar-PYTHON_COLORS

I thought about this (after I made the "ignore color" commit).  Is there a reason to prefer one over the other?

Fix it at the source, rather than cleaning up the output.
One line added in one file, vs. ~30 lines in four files.
I expect CPython will add colour in other places too at some point (I don't know if in 3.13), which could need without_color peppering elsewhere too.

An embarrassing admission: until I looked at this PR's diff, I didn't realize I had set FORCE_COLOR=1, I thought GitHub had done it!  You are probably right: less code is better code.

It might be better to set PYTHON_COLORS=0 in a more central place, like tox.ini, so that local test runs will also be immune to local settings.

An embarrassing admission: until I looked at this PR's diff, I didn't realize I had set FORCE_COLOR=1, I thought GitHub had done it! You are probably right: less code is better code.

Yeah, GitHub Actions is not a tty so tools' usual colour auto-detection doesn't work there (and we don't want escape codes when piping output), so FORCE_COLOR is needed. (I think colour worked back in the day on Travis CI.)

It might be better to set PYTHON_COLORS=0 in a more central place, like tox.ini, so that local test runs will also be immune to local settings.

Good idea, updated!


Attempt to fix StateLayout: Collection was modified

Description of Change
Attempt to fix StateLayout crash.
Bugs Fixed

Fixes #784

API Changes
None
Behavioral Changes
None
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[ ] Has samples (if omitted, state reason in description)
[X] Rebased on top of main at time of PR
[X] Changes adhere to coding standard
[ ] Updated documentation

Do not merge until the fix is properly validated.

@jfversluis hm, bot didn't generate a NuGet (

@jfversluis hm, bot didn't generate a NuGet (

I disabled it for now since it was getting a life of its own and I don't want the AI to take over this repo...
Have to take a look what's going on with it :)

I disabled it for now since it was getting a life of its own and I don't want the AI to take over this repo...
Have to take a look what's going on with it :)

This PR doesn't make sense anymore since the issue is gone

This PR doesn't make sense anymore since the issue is gone


Does this plugin require 4D Write, and can it preview EML files?

Hi and thank you for this plugin, it looks awesome and exactly what we needed.
Per the subject, I have two questions: does this plugin require 4D Write (it appears to do so, while the light version doesn't, but in my testing does only generate an image preview of the passed in document)?
Can it preview .eml email files, like macOS can in QuickLook? That format doesn't seem listed among the supported formats on Apple's own docs, and indeed testing a QuickLook preview with the light plugin only generates an image of the eml file icon.
Thanks in advance!

4D Write is not required.
the difference between light and regular is that light uses official Apple API, while regular uses undocumented API which changes from time to time.
v2 may work better on newer OS.

https://github.com/miyako/4d-plugin-ql-v2

Thank you so much!
We're doing some testing with the v2, sadly it seems .eml files only preview with an icon and not the actual email content; it had to be expected considering Apple is not listing that file format in the documentation for QLPreviewController, but yours is nevertheless an amazing plugin, thank you for your work!


Update golangci-lint, sync go-toolset/vale/pre-commit

Changes

Update go-toolset image to latest
Update vale styles to latest
Enable new golangci-lint rules
Enable a bunch of new golangci-lint, commented the ones we don't use so
to make it easier to get new linters in the future.
Corrected a lot of errors
Update pre-commit hooks
Refactor getInstallationIDFromPayload function
Don't do twice marshal/unmarshal in the function when it's not needed
Remove black from linter.yaml and use ruff format to format check
Check if the files are formatted with ruff format --check instead of
black since we know use it for linting we may as well use it for
formatting (and faster).

Submitter Checklist

[X] 📝 A good commit message is important for other reviewers to understand the context of your change. Please refer to How to Write a Git Commit Message for more details how to write beautiful commit messages.
[X] ♽ Run make test before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI. (or even better install pre-commit and do pre-commit install in the root of this repo).
[X] ✨ We heavily rely on linters to get our code clean and consistent, please ensure that you have run make lint before submitting a PR. The markdownlint error can get usually fixed by running make fix-markdownlint (make sure it's installed first)
[X] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[X ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[X] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[X] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

lgtm


Avoid empty or noninformative authors or other instance of person

Related issues
Refs: N/A
Describe the changes made in this pull request
In CFF 1.2.0, the following is considered valid:
cff-version: 1.2.0
message: the message
title: the title
authors:
- {}
- name: second author

or
cff-version: 1.2.0
message: the message
title: the title
authors:
- country: NL
- name: second author

This is not really a big deal, even if it is a bit odd. Anyway, this PR changes the minimum requirements on instances of "person" such as element of authors, contact, element of editors, etc. It requires at least one of the following fields to be present on each person:

alias
email
given-names
family-names
orcid

With the changes in this PR, neither of the examples above would be valid anymore.
In constrast, the following would be considered valid
cff-version: 1.2.0
message: the message
title: the title
authors:
- alias: jspaaks
- name: second author

or
cff-version: 1.2.0
message: the message
title: the title
authors:
- alias: jspaaks
  given-names: Jurriaan H.
- name: second author

Note: depending on how pedantic we feel, this is a breaking change. Personally, I choose to think of this as a bugfix.
Review checklist

[x] Please check if the pull request is against the correct branch
(format/schema/semantic documentation changes: develop; typos, meta files, etc.: main)
[ ] Please check if all changes are recorded in CHANGELOG.md and adapt if necessary.
[x] Please run tests locally.

cd $(mktemp -d --tmpdir cff.XXXXXX)
git clone https://github.com/citation-file-format/citation-file-format .
git checkout <branch>
python3 -m venv env
source env/bin/activate
pip install --upgrade pip wheel setuptools
pip install -r requirements.txt
pytest
<do other things>

[ ] <do other things>

Thanks for the review!


Falcon fails in 0-rawreads/tan-runs/

I tried falcon multiple times and always failed in 0-rawreads/tan-runs step. When I used only a small proportion of the real data, some of the tan-runs succeeded but still failed eventually on other chunks.
Any help is greatly appreciated!
This is the error file from 0-rawreads/tan-runs/tan_013 run-P2d88a2c0762e02.bash.stderr.txt I was able replicate the error by running bash user_script.sh in the tmp/pypetmp/0-rawreads/tan-runs/tan_013 folder. Error started at
LAsort<EMAIL_ADDRESS>LAmerge TAN.raw_reads.56.las<EMAIL_ADDRESS>LAcheck raw_reads TAN.raw_reads.53.las TAN.raw_reads.54.las TAN.raw_reads.55.las TAN.raw_reads.56.las
+ LAcheck raw_reads TAN.raw_reads.53.las TAN.raw_reads.54.las TAN.raw_reads.55.las TAN.raw_reads.56.las
[WARNING]Call 'bash -vex run_datander.sh' returned 256.
Traceback (most recent call last):
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 1443, in <module>
    main()
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 1439, in main
    args.func(args)
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 990, in cmd_tan_apply
    tan_apply(args.db_fn, args.script_fn, args.job_done_fn)
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 355, in tan_apply
    io.syscall('bash -vex {}'.format(os.path.basename(script_fn)))
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/pypeflow/io.py", line 29, in syscall
    raise Exception(msg)
Exception: Call 'bash -vex run_datander.sh' returned 256.

It seems all the input files are in place. build/, run-split/ and run-chunks/ were done successfully. I don't know how to proceed from here.
my cfg file:
[General]
input_fofn = input.fofn
input_type = raw

pa_fasta_filter_option=pass
pa_DBsplit_option = -a -x500 -s200
ovlp_DBsplit_option = -a -x500 -s200

ovlp_HPCTANmask_option =
pa_REPmask_code = 0,300;0,300;0,300

genome_size = 800000000
seed_coverage = 60
length_cutoff = -1
length_cutoff_pr = 1500

pa_HPCdaligner_option = -v -B4 -M16
pa_daligner_option =  -e.70 -l1000 -s1000

falcon_sense_option = --output_multi --min_idt 0.70 --min_cov 4 --local_match_count_threshold 2 --max_n_read 200 --n_core 6
falcon_sense_skip_contained = False

ovlp_HPCdaligner_option = -v -B4 -M32
ovlp_daligner_option = -h60 -e.96 -l500 -s1000

overlap_filtering_setting = --max_diff 100 --max_cov 100 --min_cov 20 --bestn 10
[job.defaults]
njobs = 12
use_tmpdir = ./tmp
stop_all_jobs_on_failure = true
job_type = slurm
job_type = string
pwatcher_type = blocking

submit = srun -J ${JOB_ID} -o ${JOB_STDOUT} -e ${JOB_STDERR} --time=10:00:00 --mem=100g --cpus-per-task=${NPROC} --partition=production ${JOB_SCRIPT}

[job.step.da]
NPROC=8
[job.step.la]
NPROC=8
[job.step.cns]
NPROC=12
[job.step.pda]
NPROC=8
[job.step.pla]
NPROC=8
[job.step.asm]
NPROC=24

Typically, this is due to out-of-memory.
Try
cd -rawreads/tan-runs/tan_013/
bash -vex user_script.sh

If that works, try to watch peak memory while you run. Then adjust pypeflow settings. Or if it fails, maybe you can debug there.

I ran 0-rawreads/tan-runs/tan_013/user_script.sh as you advised. It failed and gave the same error message.
LAsort<EMAIL_ADDRESS>LAmerge TAN.raw_reads.56.las<EMAIL_ADDRESS>LAcheck raw_reads TAN.raw_reads.53.las TAN.raw_reads.54.las TAN.raw_reads.55.las TAN.raw_reads.56.las
+ LAcheck raw_reads TAN.raw_reads.53.las TAN.raw_reads.54.las TAN.raw_reads.55.las TAN.raw_reads.56.las
[WARNING]Call 'bash -vex run_datander.sh' returned 256.
Traceback (most recent call last):
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 1443, in <module>
    main()
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 1439, in main
    args.func(args)
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 990, in cmd_tan_apply
    tan_apply(args.db_fn, args.script_fn, args.job_done_fn)
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/dazzler.py", line 355, in tan_apply
    io.syscall('bash -vex {}'.format(os.path.basename(script_fn)))
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/pypeflow/io.py", line 29, in syscall
    raise Exception(msg)
Exception: Call 'bash -vex run_datander.sh' returned 256.
WARNING:root:Call '/bin/bash user_script.sh' returned 256.
WARNING:root:CD: 'uow-00' -> '/share/vandeynzelab/scratch/Shiyu/Fonio_Assembly/Temporary_data/Falcon/tmp/shyche/pypetmp/share/vandeynzelab/scratch/Shiyu/Fonio_Assembly/Temporary_data/Falcon/0-rawreads/tan-runs/tan_013'
Traceback (most recent call last):
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/generic_run_units_of_work.py", line 117, in <module>
    main()
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/generic_run_units_of_work.py", line 113, in main
    run(**vars(args))
  File "/afs/genomecenter.ucdavis.edu/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/falcon_kit/mains/generic_run_units_of_work.py", line 66, in run
    pypeflow.do_task.run_bash(script, inputs, outputs, params)
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/pypeflow/do_task.py", line 187, in run_bash
    util.system(cmd)
  File "/software/anaconda2/5.0.1/lssc0-linux/lib/python2.7/site-packages/pypeflow/io.py", line 29, in syscall
    raise Exception(msg)
Exception: Call '/bin/bash user_script.sh' returned 256.

The peak memory use before it died was around 7g.

It looks like LAcheck failed. That's not actually our code, but let's try to figure out if that's the problem anywyay. If you cd into the "uow" run directory, you can run LAcheck directly. Just type that whole line (after +) into your shell. If that crashes, Gene Myers will definitely want a failing testcase. Hopefully it's not too much data to send him. Let's see.

LAcheck raw_reads TAN.raw_reads.53.las TAN.raw_reads.54.las TAN.raw_reads.55.las TAN.raw_reads.56.las

Also try to learn the file sizes of those .las files. (They might be symlinked.) And also of the .raw_reads.* database files.

I ran LAcheck raw_reads TAN.raw_reads.53.las TAN.raw_reads.54.las TAN.raw_reads.55.las TAN.raw_reads.56.las -v. Error output as following:
  TAN.raw_reads.53: Duplicate LAs (1034526 vs 1034526)
  TAN.raw_reads.53: Not OK, see stderr
  TAN.raw_reads.54: Duplicate LAs (1068337 vs 1068337)
  TAN.raw_reads.54: Not OK, see stderr
  TAN.raw_reads.55: Duplicate LAs (1090695 vs 1090695)
  TAN.raw_reads.55: Not OK, see stderr
  TAN.raw_reads.56: Duplicate LAs (1110284 vs 1110284)
  TAN.raw_reads.56: Not OK, see stderr

I'm not sure which stderr to check.
The file sizes are
18G  ./0-rawreads/build/.raw_reads.bps
53M ./0-rawreads/build/.raw_reads.dust.anno
253M ./0-rawreads/build/.raw_reads.dust.data
262M ./0-rawreads/build/.raw_reads.idx

658K ./temp/0-rawreads/tan-runs/tan_013/uow-00/TAN.raw_reads.53.las
648K ./temp/0-rawreads/tan-runs/tan_013/uow-00/TAN.raw_reads.54.las
524K ./temp/0-rawreads/tan-runs/tan_013/uow-00/TAN.raw_reads.55.las
686K ./temp/0-rawreads/tan-runs/tan_013/uow-00/TAN.raw_reads.56.las

There used to be a bug in DALIGNER which caused similar LAcheck failures. My guess is that you are not using our version of DALIGNER. Please post conda list.

conda list
# packages in environment at /software/anaconda2/5.0.1/lssc0-linux:
#
# Name                    Version                   Build  Channel
falcon-kit                1.2.6                    pypi_0    pypi
falcon-phase              1.0.0                    pypi_0    pypi
falcon-unzip              1.1.7                    pypi_0    pypi

Also
pb-assembly               0.0.4                    py27_5    bioconda
pb-dazzler                0.0.0                h470a237_0    bioconda
pb-falcon                 0.2.6            py27hc0aa232_3    bioconda
pbalign                   0.3.1                    py27_0    bioconda
pbbam                     0.18.0               h6678c95_4    bioconda
pbccs                     3.3.0                         0    bioconda
pbcommand                 1.1.1            py27h24bf2e0_1    bioconda
pbcopper                  0.4.2                h02d93b8_0    bioconda
pbcore                    1.5.1                    py27_1    bioconda
pbcoretools               0.2.4                    py27_1    bioconda
pbmm2                     0.8.1                ha87ae23_0    bioconda

pb-dazzler                0.0.0                h470a237_0    bioconda

That should be fine. Maybe you have LAcheck or daligner in your $PATH from elsewhere. Let's see:
which daligner
which LAcheck

I use a falcon module installed by my system administrator.
$ which daligner
/software/falcon/2018.31.08/lssc0-linux/bin/daligner
$ which LAcheck
/software/falcon/2018.31.08/lssc0-linux/bin/LAcheck

Is that the same path as the Bioconda tools? Or could those be old versions?
which samtools
which python
which nucmer

$ which samtools
/software/anaconda2/5.0.1/lssc0-linux/bin/samtools
$ which python
/software/anaconda2/5.0.1/lssc0-linux/bin/python
$ which nucmer
/software/anaconda2/5.0.1/lssc0-linux/bin/nucmer

LAcheck and daligner are not from the path /software/anaconda2/5.0.1/. Does it mean that I'm not using the versions from bioconda? I did find fc_run in /software/anaconda2/5.0.1/lssc0-linux/bin/, but when I run which fc_run, the path is /software/falcon/2018.31.08/lssc0-linux/bin/fc_run.

Does it mean that I'm not using the versions from bioconda?

Correct.

Hi, it took me a while to finally figure out which one is the latest bioconda pb-assembly. I still can't run through the tan-run step but with slightly different/better problem.
 They look like the same error as I got before. But when I ran bash -vex run_datander.sh, it went through without any error.
Then I try the command inside user_script.sh
python3 -m falcon_kit.mains.dazzler --config=/share/vandeynzelab/scratch/Shiyu/Fonio_Assembly/Temporary_data/Falcon/General_config.json --db=/share/vandeynzelab/scratch/Shiyu/Fonio_Assembly/Temporary_data/Falcon/0-rawreads/build/raw_reads.db  tan-apply --script=../../../../../../../../../../../../../../0-rawreads/tan-split/tan-scripts/tan_000/run_datander.sh --job-done=job.done
it gave me the error I saw from .err file, which is:
LAsort<EMAIL_ADDRESS>LAmerge TAN.raw_reads.4.las<EMAIL_ADDRESS>LAcheck raw_reads TAN.raw_reads.1.las TAN.raw_reads.2.las TAN.raw_reads.3.las TAN.raw_reads.4.las
+ LAcheck raw_reads TAN.raw_reads.1.las TAN.raw_reads.2.las TAN.raw_reads.3.las TAN.raw_reads.4.las
[WARNING]Call 'bash -vex run_datander.sh' returned 256.
Traceback (most recent call last):
  File "/share/vandeynzelab/local/conda_envs/my_pb_env_conda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/share/vandeynzelab/local/conda_envs/my_pb_env_conda3/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/share/vandeynzelab/local/conda_envs/my_pb_env_conda3/lib/python3.7/site-packages/falcon_kit/mains/dazzler.py", line 1531, in <module>
    main()
  File "/share/vandeynzelab/local/conda_envs/my_pb_env_conda3/lib/python3.7/site-packages/falcon_kit/mains/dazzler.py", line 1527, in main
    args.func(args)
  File "/share/vandeynzelab/local/conda_envs/my_pb_env_conda3/lib/python3.7/site-packages/falcon_kit/mains/dazzler.py", line 1056, in cmd_tan_apply
    tan_apply(args.db_fn, args.script_fn, args.job_done_fn)
  File "/share/vandeynzelab/local/conda_envs/my_pb_env_conda3/lib/python3.7/site-packages/falcon_kit/mains/dazzler.py", line 394, in tan_apply
    io.syscall('bash -vex {}'.format(os.path.basename(script_fn)))
  File "/share/vandeynzelab/local/conda_envs/my_pb_env_conda3/lib/python3.7/site-packages/pypeflow/io.py", line 27, in syscall
    raise Exception(msg)
Exception: Call 'bash -vex run_datander.sh' returned 256.

I also tried to run LAsort, LAmerge and LAcheck separately with success.
I'm really puzzled here. Any help is greatly appreciated.
Shiyu

When others have seen a crash in tan-masking, it is usually out-of-memory. If you adjust your config, hopefully you can reduce the memory required per job, or increase the memory requested from the machines you use.
The way to debug this is to go into the run-directory of the failing task and run the task yourself. You can try bash user_script.sh for a start. Observe the process to see how much memory it actually uses. Good luck!


feat(serverless-offline): Add plugin support

Optimize and transform handles paths before serverless-offline does its magic.
PR's are welcomed! 👍

This functionality is redundant since serverless-offline offers Babel integration.


Report llvm-3.8-dev package broken

What platform is this for, and in what way it is broken?   Thanks!

Ubuntu/Debian
On Wed, Apr 20, 2016, 20:32 Bob Summerwill<EMAIL_ADDRESS>wrote:

What platform is this for, and in what way it is broken? Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/ethereum/evmjit/issues/42#issuecomment-212548312

That package doesn't build?    Our code is broken when built against it?
What's the exact issue?

CMake shared files of llvm-3.8-dev are incorrect and build configuration of evmjit will fail. It is known issue but the maintainer is fixing it for years...
I fixed some of the issues some time ago, but I don't have direct access to the deb package SVN.

Blergh ...


LB-IPAM switch CiliumLoadBalancerIPPool.Spec.AllowFirstLastIPs from default Yes to default No

Old LB-IPAM behavior was to always reserve the first and last IP of a CIDR, this feature has now been made optional with the CiliumLoadBalancerIPPool.Spec.AllowFirstLastIPs field. If not specified, we still default to Yes in v1.15. However, it is desirable to have this default to No.
Once the v1.15 branch has been forked we can apply this change. Docs and CRD comments should be updated accordingly and a note should be added to the upgrade guide that users should explicitly set this value on their policies if they do not want the changed default behavior.

Minor note on this one. The CiliumLoadBalancerIPPool does not seem to calculate the available IPs correctly if the prefix length is /32.
apiVersion: "cilium.io/v2alpha1"
kind: CiliumLoadBalancerIPPool
metadata:
  name: public
spec:
  cidrs:
  - cidr: "<IP_ADDRESS>/32"

I created a CiliumLoadBalancerIPPool for the Gateway LoadBalancer and it stays in Pending. I see this:
$ k get ippools
NAME     DISABLED   CONFLICTING   IPS AVAILABLE   AGE
public   false      False         0               28m

Not stale, have not gotten to it yet


vagrant up causes no space left on device

See also docker/docker#8804. I'm not sure whether this is primarily a Docker issue, a Vagrant issue, or a Vagrant CoreOS issue. Please reassign this issue as necessary.
Running vagrant up on coreos-vagrant causes the error scp: /var/tmp/hostname.yml: No space left on device (RuntimeError). See below for details. The volumes do not appear full.
sh-4.2# df
Filesystem     1K-blocks     Used Available Use% Mounted on
rootfs          16793600 12362164   3910668  76% /
devtmpfs         2014476        0   2014476   0% /dev
tmpfs            2026352        0   2026352   0% /dev/shm
tmpfs            2026352<PHONE_NUMBER>   1% /run
tmpfs            2026352        0   2026352   0% /sys/fs/cgroup
/dev/sda9       16793600 12362164   3910668  76% /
/dev/sda3        1032088   294556    685104  31% /usr
tmpfs            2026352        0   2026352   0% /media
tmpfs            2026352        0   2026352   0% /tmp
/dev/sda6         110576       88    101316   1% /usr/share/oem
/dev/sda9       16793600 12362164   3910668  76% /var/lib/docker/btrfs

I'm seeing exactly this with CoreOS alpha (633.1.0) on Vagrant using the VMWare provider. Did you ever find a solution?

No, I didn't. I blew away the machine and started over.

I came across the same problem. When I inspected the disk usage I saw that mongo was filling up the disk. I deleted mongo files and uninstalled it since I didn't need mongo. This fixed the issue in my machine.

Moved to https://github.com/coreos/bugs/issues/1359


Remove optional chaining for backwards compatibility

This PR would refactor the only 2 parts of optional chaining to help older browsers make use of the package (namely Safari < v13)

Hey @Syphini! 👋
Thanks for the PR, I appreciate that! 🙏
I'm currently working on a v2 release of tailwind-merge that will transpile to a more modern version of JS, so the removal of optional chaining will not be enough to support Safari 13 anymore. But I'm also adding an ES5-compatible build that will work in Safari 13 (optional chaining gets transpiled away there). You can read about that in the migration guide.
I think I'll release v2 in about 3 weeks (I'm traveling in between, so can't release sooner unfortunately). I suggest closing this PR and use the ES5-compatible bundle in v2.
If you can't wait until v2, you could also install<EMAIL_ADDRESS>which is the latest commit on the main branch which is essentially v2. I don't plan any more changes in code for v2, only some docs are missing (you can check out the v2 roadmap). But keep in mind that those dev versions are unstable between releases, so I'd be cautious using those in high stakes production apps.

Sounds good I'll be happy to wait for v2.
Noticed that there wasn't any transpiling for this while testing so I put through a PR but if you're all over it then no worries.
Thanks for the response!


Support abstract domain sockets using epoll on linux

Version: 5.0.0.Alpha2
Context:
I am trying to write a client for dbus on linux and need to support abstract domain sockets to properly connect to the native dbus servers.
It is currently not possible to create an abstract domain socket using the epoll selector. While creating normal domain sockets works fine, an attempt to prepend a \0 to open an abstract socket leads to an IOException when binding or connecting since the GetStringUTFChars used in the source will encode the \0 character. There could also be issues with the strlen and strncpy operations if this was moved to a different encode operation.
$ java -version
openjdk version "1.8.0_51"
OpenJDK Runtime Environment (build 1.8.0_51-b16)
OpenJDK 64-Bit Server VM (build 25.51-b03, mixed mode)

Operating System: Arch Linux 64-bit
$ uname -a
Linux ylt 4.1.4-1-ARCH #1 SMP PREEMPT Mon Aug 3 21:30:37 UTC 2015 x86_64 GNU/Linux

@yawkat can you tell me what you mean with "abstract domain socket" ?

See this man page. Basically, domain socket names allow \0 bytes in them, and on linux, a domain socket starting with \0 does not appear in the file system and is called abstract.
socat supports these through a special ABSTRACT address family. dbus  supports them through an abstract property in their address system.
Sorry for not linking the exact paragraphs, I'm on mobile.

@yawkat so I guess using 'GetStringChars' would be right here... I'm correct ?

Wouldn't that return an array of 16-bit numbers? Casting that to 8 bit would just give you lots of 0s.
I'm guessing the right way would either be decoding it as ascii or passing through the byte array directly instead of the string. Also need to replace the string operations because they rely on a trailing 0.

@yawkat good point.

Okay, had nothing else to do so I tried my hand on a fix: #4092

Fixed by https://github.com/netty/netty/pull/4092


Update most of VS2017 samples to VS2022 and make sure samples are run on the proper runner on GitHub

A Vs2022 sample must run on a windows-2022 runner. Same for Vs2019. GitHub runner image no longer support multiple VS build tools. See related announcement: https://github.com/actions/runner-images/issues/9701.
I believe there is more cleanup to do in SamplesDef.json to make sure solutions generated by samples are ran on corresponding runner image and VS versions. But this is a first pass that focus on fixing GitHub build.

Looks fine to me.
There are indeed more samples to change to normalize everything but I'm okay with fixing GitHub first.
Maybe we should also get rid of .NET 4.7.2 and move on to a recent version.


Is DML a wrapper for Intel DSA?

I've read the document, but I'm still confused about the hardware and software options.
Is DML a wrapper for Intel DSA?
Since DMA also supports asynchronous data movement, does DML support DMA as well?
And the dml::software is implement by starting another thread (or corountine)?
Thanks in advance!

Hello,
What document are you referring to? Is it DML documentation or DSA specification?
DML is a software layer that enables DSA, that is correct.
We've added Library Architecture Overview for the similar library, called QPL (for IAA support), I wonder if this could be helpful to get a better overall understanding.
Just to clarify, hardware_path (Low-Level API) or dml:hardware (High-Level API) is for offloading operation fully to accelerator. There could be portion of work done on the host, but in most of the cases it is just some preparatory work before offloading. Asynchronous execution on hardware_path is basically about submitting a job and periodically checking whether it is completed (with submit and check done on users side), synchronous interface does the same, but on library side (as if under single API we call submit and periodically check completion).
software_path / dml::software is for executing the operation on the host only. So for this path, there is no asynchronous API (submit).
automatic_path / dml::auto currently is very straightforward and submits job to DSA first, and in the case of error, do re-try (or continue) on the host. This could be potentially extended to include heuristics for choosing whether to execute on host or accelerator, or extended to divide work between those.
Hope this helps!

Thank you for your quick response!

@mzhukova  I apologize for reopening this issue, but I still have some related follow-up questions that I would like to ask you.
DSA is a target for offloading many functions. However, before DSA, there was also on-chip DMA in I/OAT similar to DSA. I understand that DSA has more features than DMA, but if we only consider memory copy, is DSA faster than DMA? Additionally, will DML support DMA "hardware"?
Thank you once again!

For those who are still interested, this paper is a good read, A Quantitative Analysis and Guideline of Data Streaming Accelerator in Intel® 4th Gen Xeon® Scalable Processors


brew upgrade command shows 'Warning' when installed is up-to-date, even with --quiet

brew gist-logs <formula> link OR brew config AND brew doctor output
HOMEBREW_VERSION: 3.0.1
ORIGIN: https://github.com/Homebrew/brew
HEAD: c951be8d3c7b339c7e759b7c40aec859e09a70a5
Last commit: 10 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 647e215ff8f6d2ecd096f8436fde6fe74f59f56c
Core tap last commit: 31 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_BROWSER: open
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: /usr/bin/vim
HOMEBREW_MAKE_JOBS: 4
Homebrew Ruby: 2.6.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3_2/bin/ruby
CPU: quad-core 64-bit westmere
Clang: 10.0 build 1000
Git: 2.30.1 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
macOS: 11.2.1-x86_64
CLT: <IP_ADDRESS>.1.1539992718
Xcode: 10.1

[X] The brew doctor above contains no "Warning" lines.

What were you trying to do (and why)?
Upgrade a package (any package, the package is not important)
I am using 'go' purely as an example, below...
What happened (include all command output)?
brew upgrade --quiet go
Warning: go 1.15.8 already installed

What did you expect to happen?
I know that the package is already installed - I'm trying to upgrade it...
Without the --quiet switch, it's acceptable to provide some feedback, although I would expect something like:
Warning/Info: go 1.15.8 already up-to-date

However, with the --quiet switch, I would not expect a warning when everything is up-to-date?
Step-by-step reproduction instructions (by running brew commands)
# again, any package, the package is not important
# just ensure that the package is already installed
# e.g. brew install go

brew upgrade --quiet go

  -q, --quiet                      Make some output more quiet.

It doesn't say “suppress all output”. I think it's fair that a warning is emitted: you're asking explicitly, by name, to upgrade a formula that will not be upgraded. A warning seems in order, in all cases.

Almost, hence the suggestion of "already up-to-date" rather than the same message that's given when trying to actually install something that's "already installed". Without the --quiet I'd certainly expect to see something like that as there is helpful information there. However, it's not really adding anything, by informing me know that nothing's going to change, when I've specifically requested some 'quietness' :-)

I didn't suggest that it should “suppress all output”. I agree with you that this would be the outcome (in this instance), but I'm not saying that should always be the case (other warnings for example, where they provide useful information, may be helpful). It's just that in this case, it doesn't add anything helpful. If an update were available, we would see the update occur (regardless of the --quiet switch). I wasn't expecting the --quiet switch to mute everything (as you've highlighted, it's merely there to "Make some output more quiet", which is what I had expected - that 'some' being the warning which I don't care about... hence, wanting to quiet down the resulting output)

Passing on this for now given the documentation, sorry.


FireFox, Chrome, Edge, Opera Drivers are not working

I have installed SeleniumBasic-<IP_ADDRESS>.exe and did full installation.  Only IE and PhantomJs drivers are working and others are not working. I have Excel 2016 64 bit and Excel 2007 32 bit running on Windows 10 64bit . Same issue on both excel versions.

I have been having problems getting mine to work for the week or two.  It just goes to a blank screen when trying to open any page in chrome.  It was working fine for months before that.

Not sure about other browsers but chrome driver in the latest version of selenium Basic is outdated.
I looked into the chrome driver version and found out that it’s v 2.21while the latest version with bug fixes for chrome 54 is v 2.25.
Selenium wrapper installer comes with v 2.21 so we have to manually update the driver.
First download chrome driver 2.25 from here – https://sites.google.com/a/chromium.org/chromedriver/downloads then go to your selenium path – C:\Users\username\AppData\Local\SeleniumBasic and replace the chrome driver with the latest one and it should work fine.

I updated the chromedriver.exe file as suggested in one of the other issues posted but that did not fix it.  I had to update my call to the driver as well.  SeleniumWrapper.webdriver         Set driver = New SeleniumWrapper.webdriver         driver.Start "chrome", "https://account.authorize.net/ui/themes/anet/Transaction/VirtualTerminal.aspx"         driver.Open ("https://account.authorize.net/ui/themes/anet/Transaction/VirtualTerminal.aspx")

Glad it worked, I don't know how you were calling it previously. You can find more examples here -

Browser automation using Selenium VBA

Scrape website using Selenium VBA

I posted how to get Edge working.  Go here.
To get Firefox working you need to downgrade to Firefox 46.0.1.  Disable auto updates.
This is a great product.  I would pay for it.  I am hoping it is updated soon.

Thank you for the update.
Dear author,
please keep this product updated and I am willing to help it by coding
support / donation.
Dharmesh
On Thu, Dec 1, 2016 at 11:25 AM, mogulman52<EMAIL_ADDRESS>wrote:

I posted how to get Edge working. Go here
https://github.com/florentbr/SeleniumBasic/issues/126.
To get Firefox working you need to downgrade to Firefox 46.0.1. Disable
auto updates.
This is a great product. I would pay for it. I am hoping it is updated
soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/florentbr/SeleniumBasic/issues/133#issuecomment-264219525,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtGQR4-eNjNKtVr-UtBPPMFLXBZYe0kks5rDvTfgaJpZM4K9_Rz
.

--
Thanks,
Dharmesh Jajal
203.526.6189

Hi,
I am getting some issue when i use my macro on one of my computer :
Firefox opens but it doesnt open the URL I gave and I get an error.
On my other computer it works perfectly.
The difference between these the computer is the Firefox version.
On the working one, FF version is 39
On the NOT working one, FF version is 49.
I read that upper FF 47 Webdriver is no more supported. The new driver is "Marionette" but I can't find how to make it work with VBA.
Have you any idea to make my automation work on my both computer ?
Here my code :
`Public Sub Mozilla()
Dim driver As New Selenium.WebDriver

Dim i As Integer, j As Integer
Dim dems As String

driver.Start Sheets("Paramètres").Range("POD_NAV")
driver.Get Sheets("Paramètres").Range("POD_URL")
driver.FindElementById("P101_USERNAME").SendKeys Sheets("Paramètres").Range("POD_LOGIN")
driver.FindElementById("P101_PASSWORD").SendKeys Sheets("Paramètres").Range("POD_PWD")
driver.FindElementById("P101_LOGIN").Click

End Sub`

You need to downgrade to FF v.46.0.1.  I looked at the code in hopes of updating it but I am lost on how to change/debug it.

I dreaded this answer...
so sad to donwgrade the system in order to make it work. It sounds so bad, to hear that you have to not follow the progress but look at the back.
Anyway I will do it in order to make it work.
Otherwise if someone find anything, they are welcome. By the way no marionette for selenium vba ?

ChromeDriver does not work on Windows 7 x64.
Run-time error ''33'
SesssionNotCreatedError
session not created exception
from uknown error: Runtime.executionContextCreated has invalid 'context':
{"auxData":{"frameId":1596.1","isDefault":true),"id":1,"name":"",....

Please read all comments in this post.  Upgrade Chrome driver and downgrade to FF 46.0.1

the "only" problem seems to be the gecko driver, isn´t it possible to handle it by adding some kind of
GeckoDriverService service =new GeckoDriverService.Builder(firefoxBinary)
.usingDriverExecutable(new File("path to geckodriver"))
.usingAnyFreePort()
.usingAnyFreePort()
.build();
service.start();
in the .....\Selenium\Drivers\FirefoxDriver.cs, as seen on [http://www.seleniumhq.org/docs/04_webdriver_advanced.jsp]...?!?!?

I tried this a while back.  I only spent a few hours on it.  I wasn't able to get NUnit tests working.  I don't really understand some things so I was guessing.  If I get motivated I might spend some more time on it.
`    [ProgId("Selenium.FirefoxDriver")]
[Guid("0277FC34-FD1B-4616-BB19-14DB1E4916D4")]
[ComVisible(true), ClassInterface(ClassInterfaceType.None)]
public class FirefoxDriver : WebDriver, ComInterfaces._WebDriver
{
const string BROWSER_NAME = "firefox";
    /// <summary>
    ///
    /// </summary>
    public FirefoxDriver()
        : base(BROWSER_NAME) { }

    internal static IDriverService StartService(WebDriver wd)
    {
        ExtendCapabilities(wd, false);

        var svc = new DriverService();
        svc.AddArgument("--host=localhost");
        svc.AddArgument("--port=" + svc.IPEndPoint.Port.ToString());
        svc.Start("geckodriver.exe", true);
        return svc;
    }

    internal static void ExtendCapabilities(WebDriver wd, bool remote)
    {
        Capabilities capa = wd.Capabilities;

        Dictionary opts;
        if (!capa.TryGetValue("firefoxOptions", out opts))
            capa["firefoxOptions"] = opts = new Dictionary();
    }
}

}`

Any update on this please?

I am have the same problem, any idea plz

If you update the Chrome and Edge drivers it seems to work fine.  SeleniumBasic installs at:
C:\Users<user>\AppData\Local\SeleniumBasic
Get correct Chrome webdriver and replace it.  It has to have same name. https://sites.google.com/a/chromium.org/chromedriver/downloads
Get correct Edge webdriver and replace it.  You'll need to rename it.  https://developer.microsoft.com/en-us/microsoft-edge/tools/webdriver/#downloads
Firefox doesn't work unless you use version 46.1.0 (very old).

Dear mogulman52.
The issue on FF 48+ was submitted one year ago with title: "Firefox 48+ not working with SeleniumBasic (#162)".
Your reply at that time was:
"The last FF release that works with SeleniumBasic is 46.0.1. When FF switched to the new GeckoDriver it broke. I tried to adapt SB to the new FF driver but there are several things I don't understand and I wasn't successful. I haven't seen any postings from Florent Breheret so I assume he has lost interest in this product. It is a great product and I hate to see it abandoned. I'll probably try again to get it working. If I can't maybe I'll do a dll from Excel to .Net. Selenium works with C# and FF. "
Can you please give an update on this (dll or anithing else)?.

Dear mogulman52.
The issue on FF 48+ was submitted one year ago with title: "Firefox 48+ not working with SeleniumBasic (#162)".
Your reply at that time was:
"The last FF release that works with SeleniumBasic is 46.0.1. When FF switched to the new GeckoDriver it broke. I tried to adapt SB to the new FF driver but there are several things I don't understand and I wasn't successful. I haven't seen any postings from Florent Breheret so I assume he has lost interest in this product. It is a great product and I hate to see it abandoned. I'll probably try again to get it working. If I can't maybe I'll do a dll from Excel to .Net. Selenium works with C# and FF. "
Can you please give an update on this (dll or anithing else)?.

Do we know if there was ever a fix for using selenium basic with FF 48 or greater?

Thanks for the response and attention ..
I'm looking for a tool that deals with Office software and Internet browsers
The programming language is VBA
Sent from Mail for Windows 10
From: mde00435
Sent: 12 تموز, 2019 08:19 م
To: florentbr/SeleniumBasic
Cc: mouhammadzaheralshaar; Comment
Subject: Re: [florentbr/SeleniumBasic] FireFox, Chrome, Edge, Opera Driversare not working (#133)
Dear mogulman52.
The issue on FF 48+ was submitted one year ago with title: "Firefox 48+ not working with SeleniumBasic (#162)".
Your reply at that time was:
"The last FF release that works with SeleniumBasic is 46.0.1. When FF switched to the new GeckoDriver it broke. I tried to adapt SB to the new FF driver but there are several things I don't understand and I wasn't successful. I haven't seen any postings from Florent Breheret so I assume he has lost interest in this product. It is a great product and I hate to see it abandoned. I'll probably try again to get it working. If I can't maybe I'll do a dll from Excel to .Net. Selenium works with C# and FF. "
Can you please give an update on this (dll or anithing else)?.
Do we know if there was ever a fix for using selenium basic with FF 48 or greater?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

This email has been checked for viruses by AVG.
https://www.avg.com

I've given up on FF.  There is too much I don't understand.  If I need FF I use C#.  I got dragged into a big project and didn't have time.  I did get SB to work with EDGE.  It is a convoluted process.  You start Edge, get process id and connect to Edge.

For those waiting on this issue, do you have to or must use VBA? Because there's a VBScript alternative that may be close enough for your needs (I didn't check the exact functionalities between the two though, or what bugs each may have).
http://htejera.users.sourceforge.net/vbswebdriver/
and in case not, perhaps it might be easier to fix that solution for FF (and Chrome) support than this project.

Dear Mr. mougulman52,can you better explain how you connected SB to EDGE having their process id?With my regards.
Inviato da Yahoo Mail su Android
Il Lun, 15 Lug, 2019 alle 19:34<EMAIL_ADDRESS>ha scritto:
I've given up on FF. There is too much I don't understand. If I need FF I use C#. I got dragged into a big project and didn't have time. I did get SB to work with EDGE. It is a convoluted process. You start Edge, get process id and connect to Edge.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

For those waiting on this issue, do you have to or must use VBA? Because there's a VBScript alternative that may be close enough for your needs (I didn't check the exact functionalities between the two though, or what bugs each may have). You might also be able to call the VBScript code from VBA too.
http://htejera.users.sourceforge.net/vbswebdriver/
and in case not, perhaps it might be easier to fix that solution for FF (and Chrome) support than this project.

I still need to use with VBA.

Here is hoe you connect to EDGE using SB.  I  didn't create this solution.
`
Option Explicit
Public Sub Sample()
Dim pid As Long
Dim pno As Long: pno = 17556
pid = StartEdgeDriver(PortNo:=pno)
If pid = 0 Then Exit Sub
With CreateObject("Selenium.WebDriver")
.StartRemotely "http://localhost:" & pno & "/", "MicrosoftEdge"
.Get "https://www.bing.com/"
.FindElementById("sb_form_q").SendKeys "abcdefg"
MsgBox "pause", vbInformation + vbSystemModal
.Quit
End With
TerminateEdgeDriver pid
MsgBox "done.", vbInformation + vbSystemModal
End Sub
Private Function StartEdgeDriver( _
Optional ByVal DriverPath As String = "C:\Windows\System32\MicrosoftWebDriver.exe", _
Optional ByVal PortNo As Long = 17556) As Long
Dim DriverFolderPath As String
Dim DriverName As String
Dim Options As String
Dim itm As Object, itms As Object
Dim pid As Long: pid = 0
With CreateObject("Scripting.FileSystemObject")
If .FileExists(DriverPath) = False Then GoTo Fin
DriverFolderPath = .GetParentFolderName(DriverPath)
DriverName = .GetFileName(DriverPath)
End With
'check already running process
Set itms = CreateObject("WbemScripting.SWbemLocator").ConnectServer.ExecQuery _
("Select * From Win32_Process Where Name = '" & DriverName & "'")
If itms.Count > 0 Then
For Each itm In itms
pid = itm.ProcessId: GoTo Fin
Next
End If
'execute WebDriver
Options = " --host=localhost --jwp --port=" & PortNo
With CreateObject("WbemScripting.SWbemLocator").ConnectServer.Get("Win32_Process")
.Create DriverPath & Options, DriverFolderPath, Null, pid
End With
Fin:
StartEdgeDriver = pid
End Function
Private Sub TerminateEdgeDriver(ByVal ProcessId As Long)
Dim itm As Object, itms As Object
Set itms = CreateObject("WbemScripting.SWbemLocator").ConnectServer.ExecQuery _
("Select * From Win32_Process Where ProcessId = " & ProcessId & "")
If itms.Count > 0 Then
For Each itm In itms
itm.Terminate: Exit For
Next
End If
End Sub
`

@mogulman52 do you know if the latest versions of chrome still work with selenium basic?
I've written a whole tonne of code for Firefox in but I guess that it should work with chrome with a few small mods.
The issue I'm having with Firefox is i'm automating listing items on eBay; however when I want to save the listing Firefox crashes with an 'unresponsive script' message. Is this symptomatic of the outdated browser or is this something else?

I can confirm that all my SeleniumBasic code, which goes back 3 to 4 years, still runs perfectly fine in my version of Chrome (75.0.3770.142) which is not the absolute latest.
Occasionally I had some unexpected issues in the past in code that was working fine before. That usually happened after a new version of Chrome was released but to fix it, I just downloaded the latest ChromeDriver and overwrite the old version in the SeleniumBasic installation directory and all was well again.
Also heck out the ChromeDriver download page for recommended version against each new Chrome release.
https://sites.google.com/a/chromium.org/chromedriver/downloads

Dear mr. Mogulman, your SeleniumBasic works fine for Chrome, Edge and Opera too, including the latest versions of these web browsers.
But Firefox doesn't works with the geckodriver.exe available on web for the recent releases of this browser (after release 69 I guess). Could you please update us on the status of code implementing Firefox latest release?


fix : URI detector/template loading from windows fs

Fixed in https://github.com/zeldigas/text2confl/pull/116. Differences:

more accurate resolution of templates
fixed regex instead of maintaining multiple ones for external links detection


I2C example

There is no I2C example available in the examples folder. This would be a welcome addition for those attempting to learn how to use the STM32 and this crate at the same time.

This is a somewhat known problem also discussed in #24. Looks like some progress has been made in that area, so I might try to add an example later today.
For now, you can also have a look at https://github.com/TheZoq2/weather/blob/master/rtic_hardware/src/main.rs#L190 which at least shows how to set up i2c and pass it to a driver crate

This is a somewhat known problem also discussed in #24. Looks like some progress has been made in that area, so I might try to add an example later today.
For now, you can also have a look at https://github.com/TheZoq2/weather/blob/master/rtic_hardware/src/main.rs#L190 which at least shows how to set up i2c and pass it to a driver crate


RFC: Include BinExport as a native exporter format in Ghidra

Is your feature request related to a problem? Please describe.
In order to use BinDiff for program diffing, one needs to export binary disassembly first.
For Ghidra, there is the a version of the companion project BinExport, which exports into the protobuf based format the BinDiff expects. This is implemented as a Ghidra extension and is available here:
https://github.com/google/binexport/tree/main/java
Now, due to the way Ghidra handles extension versions, it is difficult for BinDiff to ship a precompiled extension that works out-of-the-box with multiple Ghidra versions, even when there should be no relevant API differences (like 10.3.2 -> 10.3.3).
Users often complain that they have to manually modify the extension.properties with the current version.
Describe the solution you'd like
Since BinExport (and BinDiff) are open source, in an ideal world, Ghidra would just include BinExport. What this might look like is up for discussion, but it could be as simple as shipping it as one of the default plugins, similar to "sample" or "MachineLearning", but have it enabled by default.
If this is accepted, I'd be happy to prepare the necessary pull requests to make this happen.
Describe alternatives you've considered
Instead of shipping BinExport as a default plugin, include it as a default exporter:

Additional context

https://github.com/google/binexport/issues/91 - example of the issues users are facing when there is a new Ghidra version, but no pre-compiled binaries yet.
https://github.com/quarkslab/quokka - Another exporter, by Quarkslab. I can imagine that they might also want to be included after porting it to Ghidra.

That is good to know :)
I'd personally still like if BinExport would be natively supported, though. Not sure how complicated it is for you to take on another dependency (protobuf and the authoritative binexport2.proto file from the main BinExport repo - the latter could simply be copied, as it changes very infrequently and never in wire-incompatible ways).

Shall I prepare a PR adding BinExport so we have something more concrete to talk about?

We'll discuss it, probably on Thursday, and I'll let you know how we'd like to proceed.

@cblichmann, we talked about it, and agreed a PR would be a good next step to further evaluate the possibility of integration.  We noticed things like
https://github.com/google/binexport/blob/5795afc727e7ab66072ea12b38f3e9c978bfa046/java/src/main/java/com/google/security/binexport/BinExport2Builder.java#L431-L434
and it would be helpful to have something closer to a final version in hand.

Sounds good, I'll try and come up with a PR for this around next week.

@ryanmkurtz I uploaded a first version - obviously took a bit longer than a week :)

Thanks!

@cblichmann, is the exporter currently the only integration that exists between Ghidra and BinDiff?  Or are there other plugins that allow for things like visualization within Ghidra?

Currently the only integration, I haven't had time to do more. But it'd be useful to integrate with Ghidra's graph views eventually.

We can reopen this if you return to it one day.


Failed to build on macOS

I used Homebrew https://brew.sh/ to install Qt; it installed Qt6.
By default, it left PKG_CONFIG_PATH blank, so I ran the build like this:
PKG_CONFIG_PATH=/opt/homebrew/opt/qt/libexec/lib/pkgconfig cargo build --release

The log said:
  cargo:warning=c_src/image.cpp:26:16: error: no matching constructor for initialization of 'CImage'
  cargo:warning=    return new CImage(std::move(image));
  cargo:warning=               ^      ~~~~~~~~~~~~~~~~
  cargo:warning=c_src/image.cpp:14:8: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from '__libcpp_remove_reference_t<QImage &>' (aka 'QImage') to 'const CImage' for 1st argument
  cargo:warning=struct CImage {
  cargo:warning=       ^
  cargo:warning=c_src/image.cpp:14:8: note: candidate constructor (the implicit move constructor) not viable: no known conversion from '__libcpp_remove_reference_t<QImage &>' (aka 'QImage') to 'CImage' for 1st argument
  cargo:warning=struct CImage {
  cargo:warning=       ^
  cargo:warning=c_src/image.cpp:14:8: note: candidate constructor (the implicit default constructor) not viable: requires 0 arguments, but 1 was provided
  cargo:warning=1 error generated.

Then I installled qt@5 and that failed, too.
export LDFLAGS="-L/opt/homebrew/opt/qt@5/lib"
export CPPFLAGS="-I/opt/homebrew/opt/qt@5/include"
export CXXFLAGS="-I/opt/homebrew/opt/qt@5/include"
export PKG_CONFIG_PATH="/opt/homebrew/opt/qt@5/lib/pkgconfig"

  cargo:warning=c_src/image.cpp:26:16: error: no matching constructor for initialization of 'CImage'
  cargo:warning=    return new CImage(std::move(image));
  cargo:warning=               ^      ~~~~~~~~~~~~~~~~
  cargo:warning=c_src/image.cpp:14:8: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from '__libcpp_remove_reference_t<QImage &>' (aka 'QImage') to 'const CImage' for 1st argument
  cargo:warning=struct CImage {
  cargo:warning=       ^
  cargo:warning=c_src/image.cpp:14:8: note: candidate constructor (the implicit move constructor) not viable: no known conversion from '__libcpp_remove_reference_t<QImage &>' (aka 'QImage') to 'CImage' for 1st argument
  cargo:warning=struct CImage {
  cargo:warning=       ^
  cargo:warning=c_src/image.cpp:14:8: note: candidate constructor (the implicit default constructor) not viable: requires 0 arguments, but 1 was provided
  cargo:warning=1 error generated.

This is a C++17 vs C++20 thing; the build script should require C++20, but either it doesn't, or your compiler is reporting C++20 mode without supporting it fully (or, heck, something else entirely).  But there's zero reason for the code to require C++20; one constructor call can easily be changed and it's safe for C++17 (the minimum for Qt6).  I've just pushed a change, please give it a shot.

That worked!


Increase code coverage

Code coverage is at a measly 35%? Fix this asap before moving on...

Adding unit tests for piece has increased this to 46. But still a long way to go.

We should wait to move out move generation out of the piece classes before achieving higher code coverage

81% code coverage so far. Good going. Let's step it up!

Fixed in #36


Added cli version of the script to Generate Demo Events for SharePoint Events List

🎯 PR aim
The aim of this PR is to add an CLI for Microsoft 365 Equivalent to https://pnp.github.io/script-samples/spo-generate-demo-events/README.html?tabs=pnpps
✔ What was done

[X] updated readme with script

🔗 Related
#74
📸 Result
the result of running the script are items added to the list

@Adam-it - its the metadata for the sample for the Site to show what and who the sample is. In future hopefully will be also showing samples on https://adoption.microsoft.com/sample-solution-gallery which would be cool. This site would continue but folks can find samples in both places.
For now, if you like, Ill take care of the updates and you can see what i've changed in the PR to get some insight.
I probably need to update docs to include this info. So its good thing you asked the question :-)

Btw - did you know you qualify for the Community Badges if you haven't already. If you register here, you can apply for it:
https://aka.ms/m365pnp-badge-registration

@pkbullock  ok thanks for the explanation 👍👍. Please fell free to update it (for sure you know way better what should be done 👍) I will check your commit to learn for future what should be done as well 😉.
Thanks for the tip 👍.
Also thanks for the comment with the badge info. I already got my contributor badge ❤


Any spa using aspnet spa generator ends up in this error...

could you please provide dotnet -v, verbose the command you are using and buildOptions from your project.json

works fine now. dotnet cli version was old. Thanks @laskoviymishka


download from crates.io fails with SSL error

Cargo fails when trying to download log from crates.io:
$ cargo build --verbose
 Downloading log v0.3.1
Unable to get packages from source

Caused by:
  Failed to download package `log v0.3.1` from https://crates.io/api/v1/crates/log/0.3.1/download

Caused by:
  SSL connect error
$ curl -OL https://crates.io/api/v1/crates/log/0.3.1/download
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:03 --:--:--     0
curl: (35) error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol

(I'm using a public WLAN, so this might be a network problem on my end.)

hmm https://github.com/rust-lang/cargo/issues/1608

@vks what version of curl are you using? I believe very old versions don't support things like SNI which are required for crates.io right now

@alexcrichton It is actually a quite recent version:
$ curl --version
curl 7.42.0 (x86_64-unknown-linux-gnu) libcurl/7.42.0 OpenSSL/1.0.2a zlib/1.2.8 libidn/1.30 libssh2/1.5.0
Protocols: dict file ftp ftps gopher http https imap imaps pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp
Features: AsynchDNS IDN IPv6 Largefile GSS-API Kerberos SPNEGO NTLM NTLM_WB SSL libz TLS-SRP UnixSockets

I think it was a network problem, because I cannot reproduce the problem on a different network.

Ah ok, sounds like it may be an intermittent firewall issue that may have been legitimate!