[Feature request] Multicast UDP

In many multi-robot applications, we need few up-downlink for data to the central system and some from the central-system to all the robots which is inconvenient if we are not using Multicast/Broadcast.

I haven't used this feature in a long time, but you should be able to use broadcast UDP by specifying a broadcast address as the destination.
The problem with multicast/broadcast over WiFi is that most WiFi routers turn off all low-level retransmission mechanisms for multicast packets (since they would need to keep track of multiple receivers). In our experiments, this increased the packet drop rate so much that it simply was not worth it.
TLDR: We won't work on this, but PRs are welcome.

Did you try IPv6 multicast during your tests? The internet is very enthusiastic about IPv6 for multicast but I only experimented IPv4.
As ROS2 is using DDS, it seems to be capable of UDP multicast. What do you think of the following architecture?
ROS1 using a local master on every machine.
ROS bridge (1 to 2 and 2 to 1) on every machine handling the communication.

Did you try IPv6 multicast during your tests? The internet is very enthusiastic about IPv6 for multicast but I only experimented IPv4.

No, we did not test IPv6 multicast. I expect the same problems as with IPv4 multicast.

As ROS2 is using DDS, it seems to be capable of UDP multicast. What do you think of the following architecture?
ROS1 using a local master on every machine.
ROS bridge (1 to 2 and 2 to 1) on every machine handling the communication.

It depends on the task you are trying to solve. nimbro_network is designed for high-latency, high-loss networks. DDS will not work well in this context, since it does not have forward error correction (to my knowledge). If your network has low latency, you can certainly try it.

Thanks @xqms for the fast answer!
We are working on both use-cases: mobile network (using Zerotier as a VPN) and single WiFi access points. It is good to know that it might not work to use DDS with a mobile network.


Black Vulkan Rendering with Intel HD

Rendering seems to be almost completely black when using an Intel HD GPU & the Vulkan API. Oddly enough, a very small section in the upper left seems to render colors related to the scene, like if the buffer is a few pixels wide.

When post processing filters are disabled, the screen is grey (seems to be the background color). There is still a small rectangle of scene-rendered pixels in the upper left.

Might be related to image layouts?
https://www.reddit.com/r/vulkan/comments/8p5hye/intel_vs_nvidia_issue/

RenderDoc captures of Intel HD frame & NVIDIA frame. Obvious problems arise during the first draw color pass. NVIDIA renders the scene fine at that step, while Intel makes the little dark blob in the corner:
RenderDoc-captures.zip

Asking for help on Reddit here: https://www.reddit.com/r/vulkan/comments/bab4hc/small_rectangle_rendered_on_intel_hd_works_fine/

Reddit to the rescue: "You appear to be ignoring Out Of Date / Sub Optimal returns from presentation.
Are you actually getting these return codes at all? Because if so you need to recreate your framebuffers etc. at the new size."
https://vulkan-tutorial.com/Drawing_a_triangle/Swap_chain_recreation#page_Suboptimal_or_out_of_date_swap_chain

The result of presenting & getting the next image is masked if it isn't an error here:
https://github.com/phr00t/xenko/blob/master/sources/engine/Xenko.Graphics/Vulkan/SwapChainGraphicsPresenter.Vulkan.cs#L154
... and "suboptimal" swap chains are not errors. So, I modified SharpVulkan to see the result of these functions on Intel HD hoping for an answer -- unfortunately, these functions are returning "Success".
The idea was to recreate the swap buffers if there was a problem.. so I've been trying to recreate them anyway. Also unfortunately, recreating the swap buffers doesn't help (nothing changes in the rendering behavior).

I can confirm that this also happens on a intel uhd 620.

Fixed with https://github.com/phr00t/xenko/commit/44b9dee8ec2cc8df9de193e52ad3ef98b16a9ef5


Make self-hosted GitHub runners ephemeral (and gracefully deregister)

Our self-hosted GitHub runners autoscale in a Managed Instance Group, but are currently not ephemeral. This means that state from one build can influence a subsequent build. We take some steps to mitigate this (deleting the workdir), but this is incomplete. It also doesn't help for security purposes, which means we've still got GitHub actions set to require approval for non-collaborators.
To make the runners ephemeral, we need to make the VM shut down when the runner exits. If the ephemeral runner performs a job, it will deregister itself, but if shutdown another way (e.g. via MIG scale-in) it won't (https://github.com/actions/runner/issues/1364). We can attach deregistration to a shutdown hook (calling the token proxy to fetch a deregistration token). Right now, I'm just periodically deregistering offline runners via the API so we keep the UI clean (GitHub only does it after 30 days).
Here's a gist I stole from a coworker that contains approximately the setup we want. It needs to be integrated into our runners and the current setup.sh script: https://gist.github.com/GMNGeoffrey/26d1546ccb5563d715fcaa47e826f36d

(out of curiosity, what's the latency of a shutdown/startup? would it be worth having a pool that doesn't suicide after every build for collaborators vs ones for public?)

(out of curiosity, what's the latency of a shutdown/startup? would it be worth having a pool that doesn't suicide after every build for collaborators vs ones for public?)

Probably, but we'd need to convince security. GitHub doesn't have very good tools for isolation between runners, so we're rolling our own. I'd rather start with them uniformly ephemeral and then add back a pool of persistent runners later. I think it would also be better to use shared caches rather than persistent runners. My thought was that we could have postsubmit runners that have read/write access to the caches and presubmit could be read-only.

Thanks for filing this with the links! I'll start looking into implementing similar systemd services to handle this.

So can we close this issue since it's marked as Done?

There's actually an issue on the GPU runners and they keep shutting down as soon as they start. I need to investigate what's going on.

The issue is that the runner on these machines is out of date. They're running 2.293.0 and it's been more than 30 days since 2.294.0 was released. That means that GitHub won't let them connect:

Aug 15 22:09:24 github-runner-presubmit-gpu-us-central1-j2ss start.sh[1511]: An error occurred: Runner version v2.293.0 is deprecated and cannot receive messages.
Aug 15 22:09:24 github-runner-presubmit-gpu-us-central1-j2ss systemd[1]: github-actions-runner-start.service: Main process exited, code=exited, status=2/INVALIDARGUMENT

So that's annoying... So the runner service fails, triggering shutdown, which causes the MIG to replace the instance and we bootloop. Even if we remove it from the MIG, the service for the runner will run on startup, fail, and cause shutdown. I was only able to get in to see the logs by removing an instance from the instance group and editing the startup script to disable the GitHub actions runner service. We're going to need to find a better way to monitor the versions. I'm subscribed to release notifications on the GitHub actions runner repo, but I missed the email. Having configurations for our images checked in rather than being just manually configured would help with this...

For now, I'll just update and create a new image


Question regarding SSH key secret

I'm new to Github so apologies if this is a bad question. Am I supposed to upload the private key as a secret for the checkout@v3 action? Tried it with the public key but it failed with "Permission Denied (publickey)". The obvious solution would be to upload the private one but I'm asking this just to be safe.

Yes just like when committing locally on your private pc with ssh you need the private key to checkout & commit (this would be your secret in the action) and the public key has to be registered as deploy key in the repo.
But you don't even need the ssh key!
You can remove these two lines from your config:
        with:
          ssh-key: ${{ secrets.SSH_KEY }}

actions/checkout and stefanzweifel/git-auto-commit-action will automatically use ${{ github.token }} secret to checkout & push to the repo.
I didn't knew that when I created the action (I was also quite new to gh actions back then) so some old docs might still tell you to use an ssh key.  The latest example is on the README.md.
I'll close the issue but if you have more questions feel free to reopen it and ask!


Check Chinese name tags in tiles

At the moment we're only using the name:zh tag, we probably also need to look at name:zh-Hant and name:zh-Hans (although name:zh is the most popular).

Of course, for users, only name:zh-Hans and name:zh-Hant are needed, and name:zh is not a useful option because it will mix the two, but name:zh should be used as a fallback for both.


Show correct port in log message

This slightly threw me when starting up 😄

Whoops! Sorry about that. Good catch @michaelvigor! 🙏


Initial Recipe of ConferencingSpeech2021 multi-ch speech enhancment

Conferencing2021.

Codecov Report

Merging #3 (5fd49bc) into master (55e0478) will decrease coverage by 0.34%.
The diff coverage is 23.00%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
- Coverage   81.81%   81.47%   -0.35%
==========================================
  Files         339      337       -2
  Lines       29244    29110     -134
==========================================
- Hits        23925    23716     -209
- Misses       5319     5394      +75

Impacted Files
Coverage Δ

espnet2/train/preprocessor.py
23.32% <5.19%> (-5.74%)
:arrow_down:

espnet2/tasks/enh.py
95.23% <80.95%> (-3.59%)
:arrow_down:

espnet2/enh/espnet_model.py
88.23% <100.00%> (-0.96%)
:arrow_down:

espnet/nets/pytorch_backend/transformer/mask.py
87.50% <0.00%> (-12.50%)
:arrow_down:

...ets/pytorch_backend/fastspeech/length_regulator.py
96.00% <0.00%> (-4.00%)
:arrow_down:

espnet/transform/transform_interface.py
66.66% <0.00%> (-3.34%)
:arrow_down:

espnet/nets/mt_interface.py
66.66% <0.00%> (-2.90%)
:arrow_down:

espnet2/asr/specaug/specaug.py
97.14% <0.00%> (-2.86%)
:arrow_down:

espnet2/layers/time_warp.py
94.73% <0.00%> (-2.64%)
:arrow_down:

espnet2/enh/layers/beamformer.py
69.87% <0.00%> (-2.57%)
:arrow_down:

... and 61 more

Continue to review full report at Codecov.

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


java.lang.ArrayIndexOutOfBoundsException: 64. Bulk delete vertex error

OrientDB Version: 3.1.17
Java Version: 8
OS: Linux
Using OrientDB gui I need to delete a lot of vertex using delete query but I have a strange error

This is my orientdb server log trace (Analytics is my database name)
Exception 33BEB057in storageplocal:/orientdb/databases/Analytics: 3.1.17 - Veloce (build 1b0146ad0a560a69f7eb7a157843718f8d9a5440, branch UNKNOWN) com.orientechnologies.orient.core.exception.OStorageException: Internal error happened in storage Analytics please restart the server or re-open the storage to undergo the restore process and fix the error. 	DB name="Analytics" 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.checkErrorState(OAbstractPaginatedStorage.java:4769) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.checkOpenness(OAbstractPaginatedStorage.java:4760) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.getClusterIdByName(OAbstractPaginatedStorage.java:2068) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentAbstract.getClusterIdByName(ODatabaseDocumentAbstract.java:805) 	at com.orientechnologies.orient.core.metadata.OMetadataDefault.init(OMetadataDefault.java:129) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.loadMetadata(ODatabaseDocumentEmbedded.java:332) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.init(ODatabaseDocumentEmbedded.java:210) 	at com.orientechnologies.orient.core.db.OrientDBEmbedded.openNoAuthenticate(OrientDBEmbedded.java:432) 	at com.orientechnologies.orient.core.db.OrientDBEmbedded.openNoAuthenticate(OrientDBEmbedded.java:79) 	at com.orientechnologies.orient.server.OServer.openDatabase(OServer.java:983) 	at com.orientechnologies.orient.server.OServer.openDatabase(OServer.java:954) 	at com.orientechnologies.orient.server.OConnectionBinaryExecutor.executeDatabaseOpen37(OConnectionBinaryExecutor.java:1371) 	at com.orientechnologies.orient.client.remote.message.OOpen37Request.execute(OOpen37Request.java:75) 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.sessionRequest(ONetworkProtocolBinary.java:355) 	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.execute(ONetworkProtocolBinary.java:239) 	at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:67) Caused by: com.orientechnologies.orient.core.exception.OStorageException: Exception during execution of component operation inside of storage Analytics 	DB name="Analytics" 	at com.orientechnologies.orient.core.storage.impl.local.paginated.atomicoperations.OAtomicOperationsManager.calculateInsideComponentOperation(OAtomicOperationsManager.java:242) 	at com.orientechnologies.orient.core.storage.impl.local.paginated.atomicoperations.OAtomicOperationsManager.calculateInsideComponentOperation(OAtomicOperationsManager.java:229) 	at com.orientechnologies.orient.core.storage.impl.local.paginated.base.ODurableComponent.calculateInsideComponentOperation(ODurableComponent.java:96) 	at com.orientechnologies.orient.core.storage.index.sbtree.singlevalue.v3.CellBTreeSingleValueV3.remove(CellBTreeSingleValueV3.java:492) 	at com.orientechnologies.orient.core.index.engine.v1.OCellBTreeSingleValueIndexEngine.remove(OCellBTreeSingleValueIndexEngine.java:130) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.removeKeyFromIndexInternal(OAbstractPaginatedStorage.java:3029) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.removeKeyFromIndex(OAbstractPaginatedStorage.java:2991) 	at com.orientechnologies.orient.core.index.OIndexAbstract.doRemove(OIndexAbstract.java:628) 	at com.orientechnologies.orient.core.index.OIndexAbstract.doRemove(OIndexAbstract.java:602) 	at com.orientechnologies.orient.core.index.OIndexAbstractDelegate.doRemove(OIndexAbstractDelegate.java:442) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.applyTxChanges(OAbstractPaginatedStorage.java:2487) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.commitIndexes(OAbstractPaginatedStorage.java:2467) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.commit(OAbstractPaginatedStorage.java:2382) 	at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.commit(OAbstractPaginatedStorage.java:2192) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentAbstract.internalCommit(ODatabaseDocumentAbstract.java:2479) 	at com.orientechnologies.orient.core.storage.impl.local.OMicroTransaction.doCommit(OMicroTransaction.java:357) 	at com.orientechnologies.orient.core.storage.impl.local.OMicroTransaction.commit(OMicroTransaction.java:173) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.endMicroTransaction(ODatabaseDocumentEmbedded.java:928) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.executeDeleteRecord(ODatabaseDocumentEmbedded.java:918) 	at com.orientechnologies.orient.core.tx.OTransactionNoTx.deleteRecord(OTransactionNoTx.java:246) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.delete(ODatabaseDocumentEmbedded.java:1103) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.delete(ODatabaseDocumentEmbedded.java:120) 	at com.orientechnologies.orient.core.record.ORecordAbstract.delete(ORecordAbstract.java:288) 	at com.orientechnologies.orient.core.record.impl.OVertexDocument.delete(OVertexDocument.java:358) 	at com.orientechnologies.orient.core.record.impl.OVertexDocument.delete(OVertexDocument.java:31) 	at com.orientechnologies.orient.core.sql.executor.DeleteStep$1.next(DeleteStep.java:35) 	at com.orientechnologies.orient.core.sql.executor.CountStep.syncPull(CountStep.java:52) 	at com.orientechnologies.orient.core.sql.executor.OSelectExecutionPlan.fetchNext(OSelectExecutionPlan.java:34) 	at com.orientechnologies.orient.core.sql.executor.OUpdateExecutionPlan.executeInternal(OUpdateExecutionPlan.java:41) 	at com.orientechnologies.orient.core.sql.parser.ODeleteVertexStatement.execute(ODeleteVertexStatement.java:71) 	at com.orientechnologies.orient.core.sql.parser.OStatement.execute(OStatement.java:72) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentEmbedded.command(ODatabaseDocumentEmbedded.java:647) 	at com.orientechnologies.orient.server.network.protocol.http.command.post.OServerCommandPostCommand.executeStatement(OServerCommandPostCommand.java:219) 	at com.orientechnologies.tinkerpop.command.OServerCommandPostCommandGremlin.executeStatement(OServerCommandPostCommandGremlin.java:47) 	at com.orientechnologies.orient.server.network.protocol.http.command.post.OServerCommandPostCommand.execute(OServerCommandPostCommand.java:104) 	at com.orientechnologies.orient.server.network.protocol.http.command.post.OServerCommandPostCommandGraph.execute(OServerCommandPostCommandGraph.java:34) 	at com.orientechnologies.orient.server.network.protocol.http.ONetworkProtocolHttpAbstract.service(ONetworkProtocolHttpAbstract.java:253) 	at com.orientechnologies.orient.server.network.protocol.http.ONetworkProtocolHttpAbstract.execute(ONetworkProtocolHttpAbstract.java:811) 	... 1 more Caused by: java.lang.ArrayIndexOutOfBoundsException: 64 	at com.orientechnologies.orient.core.storage.impl.local.paginated.wal.OWALPageChangesPortion.updateData(OWALPageChangesPortion.java:374) 	at com.orientechnologies.orient.core.storage.impl.local.paginated.wal.OWALPageChangesPortion.setIntValue(OWALPageChangesPortion.java:51) 	at com.orientechnologies.orient.core.storage.impl.local.paginated.base.ODurablePage.setIntValue(ODurablePage.java:262) 	at com.orientechnologies.orient.core.storage.index.sbtree.singlevalue.v3.CellBTreeSingleValueBucketV3.updateKey(CellBTreeSingleValueBucketV3.java:518) 	at com.orientechnologies.orient.core.storage.index.sbtree.singlevalue.v3.CellBTreeSingleValueV3.rotateNoneLeafLeftAndRemoveItem(CellBTreeSingleValueV3.java:794) 	at com.orientechnologies.orient.core.storage.index.sbtree.singlevalue.v3.CellBTreeSingleValueV3.deleteFromNonLeafNode(CellBTreeSingleValueV3.java:697) 	at com.orientechnologies.orient.core.storage.index.sbtree.singlevalue.v3.CellBTreeSingleValueV3.balanceLeafNodeAfterItemDelete(CellBTreeSingleValueV3.java:579) 	at com.orientechnologies.orient.core.storage.index.sbtree.singlevalue.v3.CellBTreeSingleValueV3.lambda$remove$3(CellBTreeSingleValueV3.java:532) 	at com.orientechnologies.orient.core.storage.impl.local.paginated.atomicoperations.OAtomicOperationsManager.calculateInsideComponentOperation(OAtomicOperationsManager.java:237) 	... 38 more
After that I can only rester the server but the vertex are not delete

UPDATE
it seems related to a class unique index
If I try to remove o rebuild it I have the same error

Removing index...
Error: com.orientechnologies.orient.core.exception.OStorageException: Exception during execution of atomic operation inside of storage Analytics
DB name="Analytics"
DB name="Analytics"
Error: com.orientechnologies.orient.core.exception.OStorageException: Exception during execution of component operation inside of storage Analytics
DB name="Analytics"
Error: java.lang.ArrayIndexOutOfBoundsException: 64

The index is present on my class Account as described below
Account.subjectFactory_userId  UNIQUE  ["subjectFactory","userId"]  CELL_BTREE

Is there something that I can do to fix this issue?

Hi,
fix  for similar problem are in the 3.2.9 release and 3.1.x branch not yet released, will be released with 3.1.20, to fix this problem you may try to rebuild the index after upgrade, if even the index rebuild fail, the suggested solution is to export and import the data on a version with the fix.
Regards

Thanks @tglman
do you know when the 3.1.20 fixed version will be released?

Hi @tglman
I updated to 3.2.9 version as you suggested me.
So after that I run **REBUILD INDEX ***
but I have the follow error
Rebuilding index(es)...
Error: com.orientechnologies.orient.core.exception.NotEmptyComponentCanNotBeRemovedException: Recommendations.versioning : Not empty index can not be deleted. Index has 2019 records

The Class Recommendations has an index on the versioning property

but there no data are present in that property and if I run select count(*) from Recommendations I have a different value: 2059

So what I have to do to fix it?
I run REPAIR DATABASE --fix-graph too but the problem is not resolved
Is there something that I can do?
Gianlica

Hi,
The best way to work around a broken index is export and import the database.
fixes for avoid that the index get broken have been releases, we are working on making the remove of a broken index easier, at the end can be rebuilt from scratch.
Regards

Hi @tglman
thanks a lot. I will try to export and import the database

Hi @tglman
I tried to export and import the database but I have always the follow error:
Error: com.orientechnologies.orient.core.exception.OSerializationException: Error on unmarshalling JSON content for record #21:2
        DB name="Analytics2"

Error: java.lang.IllegalArgumentException: Argument '9a5335db-7a7f-4806-9eb8-18b051b3921a' is not a RecordId in form of string. Format must be: <cluster-id>:<cluster-position>

Looking the database the record #21:2 is a valid vertex

Argument '9a5335db-7a7f-4806-9eb8-18b051b3921a' is a value present in the embedded class field

where is stored a  valid json (I can read/write and update it)

So I don't understand the error during import
Can you help me? What I can to do now?
Thanks
Gianluca

Hi @tglman
any news about my issue?

Hi,
It seems that the 3.2.x series has issues with json api and import, we are addressing them, this issue are not present in 3.1.x series and the 3.1.20 has been released last week, so if is possible for you I would suggest to do the import using that version.
Regards

Hi @tglman
I'm a little confused. You know that is not feasable to move always database version up and down.
I hope that coming back to 3.1.20 will fix this issue definitely

Hi @tglman
I don't find the 3.1.20 docker in
https://github.com/orientechnologies/orientdb-docker/tree/d95d46e21052dbaa36c7c8cd63abde0e6826749d/release/3.1.x
Where I can find it?

Hi @tglman
I see the 3.2.20 release. Does it fix the export/import bug?

Hi,
well is actually 3.2.10 the last release of 3.2.x series, and yes I reverted the changes that caused the problem, so it should be solved, but I could not reproduce the bug myself so I cannot confirm the fix directly.
Regards

Hi @tglman
afer 2 migration and export import, finally with 3.2.10 index and export/import problems are fixed
Thanks
Gianluca

Hi,
Thanks, closing

Hey @tglman, we've got the same root exception but after a failure to remove a compound index.
Can you look at the stacktrace and let me know if the fixes you've made also cover my case? Thanks!

Hi,
Yes this should have been already fixed in a recent version of the 3.2, I think in 3.2.9
Regards


Update antv.json

Pull request motivation(s)
same as https://github.com/algolia/docsearch-configs/pull/1121
What is the current behaviour?
If the current behaviour is a bug, please provide all the steps to reproduce and screenshots with context.
What is the expected behaviour?
NB: Do you want to request a feature or report a bug?
NB2: Any other feedback / questions ?
Is this the right way to do it?

https://github.com/afc163/docsearch-configs/blob/0afe54ec86cc7b40433d6b06d166e0ebf2a4bb68/configs/antv.json
Is this config works now? I prefer to use this one.


Go to definition should be able to handle ctx.attr(s).foo

Currently, go to definition for baz in a dotted foo.bar.baz finds a (top-level) struct definition for foo with attribute bar. The implementor of LspContext should be able to provide hints to the LSP about e.g. ctx.attr.foo, which has something of a special meaning in Buck(2) and Bazel, but not necessarily in Starlark itself. This issue is mainly to discuss what the API might look like, as briefly discussed in https://github.com/facebookexperimental/starlark-rust/pull/92#issuecomment-1700739048.
As a first idea, I came up with this:
// on LspContext
fn get_dotted_identifier_resolve_options<'a>(
    &self,
    current_file: &LspUrl,
    within_function: Option<&'a str>,
    segments: &[&'a str],
) -> anyhow::Result<Vec<ResolveOptions<'a>>>;

// outside LspContext
pub struct ResolveOptions<'a> {
    scope: ResolveScope,
    matcher: ResolveMatcher<'a>,
}

pub enum ResolveScope {
    OnlyTopLevel,
    Current,
}

pub enum ResolveMatcher<'a> {
    OneOf(Vec<ResolveMatcher<'a>>),
    Ident(&'a str),
    Assignment {
        lhs: Option<Box<ResolveMatcher<'a>>>,
        rhs: Option<Box<ResolveMatcher<'a>>>,
    },
    FunctionCall {
        function_name: Option<Box<ResolveMatcher<'a>>>,
        parameters: Option<Box<ParameterMatcher<'a>>>,
    },
    ObjectLiteral {
        key: Option<&'a str>,
        value: Option<Box<ResolveMatcher<'a>>>,
    },
}

enum ParameterMatcher<'a> {
    PositionalParameter {
        index: Option<usize>,
        value: Option<ResolveMatcher<'a>>,
    },
    NamedParameter {
        name: Option<&'a str>,
        value: Option<ResolveMatcher<'a>>,
    },
    // Maybe also something for *args and **kwargs
}

This would allow the LspContext to check for the ctx.attr pattern, and instruct the LSP to look for the appropriate AST nodes. For Bazel, that might look something like this:
fn get_dotted_identifier_resolve_options<'a>(
    &self,
    _current_file: &LspUrl,
    within_function: Option<&'a str>,
    segments: &[&'a str],
) -> anyhow::Result<Vec<ResolveOptions<'a>>> {
    let mut result = vec![];
    if within_function.is_none()
        || segments.len() < 3
        || segments[0] != "ctx"
        || segments[1] != "attr"
    {
        return Ok(result);
    }

    for function_name in ["rule", "repository_rule"] {
        result.push(ResolveOptions {
            scope: ResolveScope::OnlyTopLevel,
            matcher: ResolveMatcher::FunctionCall {
                function_name: Some(Box::new(ResolveMatcher::Ident(function_name))),
                parameters: Some(vec![
                    ParameterMatcher::NamedParameter {
                        name: Some("implementation"),
                        value: Some(ResolveMatcher::Ident(within_function.unwrap())),
                    },
                    ParameterMatcher::NamedParameter {
                        name: Some("attrs"),
                        value: Some(ResolveMatcher::ObjectLiteral {
                            key: Some(segments[2]),
                            value: None,
                        }),
                    },
                ]),
            },
        })
    }

    Ok(result)
}

The current LSP implementation to look for the struct definition would be written as follows (though I think it shouldn't be up to the LspContext to have to define this—it would only define additional lookup patterns).
ResolveOptions {
    // The currently impl is actually `ResolveScope::OnlyTopLevel`, but
    // it really should be `::Current`.
    scope: ResolveScope::Current,
    matcher: ResolveMatcher::Assignment {
        lhs: Some(Box::new(ResolveMatcher::Ident(segments[0]))),
        rhs: Some(Box::new(ResolveMatcher::FunctionCall {
            function_name: Some(Box::new(ResolveMatcher::Ident("struct"))),
            parameters: Some(vec![ParameterMatcher::NamedParameter {
                name: Some(segments[1]),
                value: None,
            }]),
        })),
    },
}

Another option would be to make the AST public (#77), and simply provide callbacks that take in the AST node and some information about the identifier being looked up, and simply returns true or false, or perhaps the matching AST node (so you can match on a function call, but jump to some specific argument). Actually, I suppose the proposal above needs some mechanic as well to do the same. Any thoughts?

I don't have any great ideas here - it seems a really hard problem. The definition is miles away and in a very different format. As such it seems like a hook that by default does nothing, but can be customized arbitrarily, seems like the sensible way of going. But I wonder - how valuable is this? For the amount of work required, it should probably be very valuable.

Does the LSP not have type information? I'd expect this to be based on (for buck2) seeing an AnalysisContext type so that we could pass that around in different ways without losing ide functionality. It still needs special handling for the attrs type since the fields on it are context dependent.
I'd think that would be single dotted things each time though, i.e. we'd use information related to the AnalysisContext type to understand that ctx.attrs is an AnalysisAttrs (idk what type it actually is) and then understand ctx.attrs.foo as just the .foo on that type.
I don't see how we really do it once we are passing around the ctx to other functions. Maybe we need the type to take arguments about the attrs that it has, but then maybe we'd want a more structured api around attr definitions... but you could imagine something like:
def some_cxx_utility(ctx: AnalysisContext[CxxToolchainAttrs], ...):
   cxx_toolchain = ctx.attrs._cxx_toolchain
   ...

Now, we should provide stronger guidance that we shouldn't be passing around ctx like that at all (and I'd argue we should split out the attrs entirely and then maybe the whole issue is moot).

The LSP was written after the type checker, so there isn't any type information used in the LSP. We should definitely fix that one day.


MINOR: Replace EasyMock with Mockito in test-utils module

More detailed description of your change
Replace EasyMock with Mockito in streams:test-utils

ping @ijuma @chia7712 @mjsax to have a look.

Thanks for the PR @dengziming! Merged to trunk.


feat(teamgantt): organize my tasks by status

Description
My Tasks command is now organized by status instead of project.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Nice!


Questions about AI Training & Implementation Experience

Hi @Gongsta,
I've been exploring your Poker-AI project with great interest and have some questions after working with it:

Training Process:

I'm currently running the training process and watching the model files generate. What indicators should I look for to know when the training has reached an optimal point?
I notice the strategy files being created in the models directory - is there a target size or number of files that indicates complete training?

Practical Questions:

After completing the training, what's the best way to evaluate if the AI is performing as intended?
Are there specific behaviors or patterns in the AI's play style that would indicate successful training?

Implementation:

I'm particularly interested in how you approached the separation between pre-flop and post-flop training. What were the key considerations that led to this design decision?

Next Steps:

For someone who's gotten the basic implementation running, what would you recommend as the most interesting aspects to explore or modify?

Thanks for creating and sharing this fascinating project! The approach to handling imperfect information games is really interesting.

I was intrigued by the tournament results you mentioned in your video - specifically where the AI won against one friend but lost to another due to a bottom pair all-in call. This raises some interesting questions about AI decision-making and player skill levels:
Could you elaborate on how you evaluate the AI's performance against different skill levels? For example:

What specific traits differentiated the "not strong player" from the more skilled opponent?
When the AI made that seemingly incorrect bottom pair call, was this a pure mathematical decision by the AI, or did it reveal a limitation in how the AI evaluates all-in situations?

Have you considered creating an interactive assistant/chatbot interface for the AI? Something like:

Using the trained model's knowledge to create a real-time poker advisor that could:

Analyze specific game situations
Provide betting suggestions
Explain its reasoning for different actions
Help players understand optimal strategies

For example, a player could input:
"I have AK suited on the button, facing a 3-bet from the big blind, pot is $200"
And the AI could suggest actions based on its trained strategies.
Questions:

Is the current model's knowledge structured in a way that could be adapted for this kind of interactive consulting?
Would the existing card and bet abstractions be useful for real-time advice?
Have you considered any ways to make the AI's decision-making process more explainable to humans?

I think this could be a fascinating educational tool for poker players trying to understand GTO (Game Theory Optimal) concepts in real situations
This idea is interesting because:

It would make the AI more accessible for learning
Could help bridge the gap between pure math and human understanding
Turn it from just a playing engine into a teaching tool
Help players understand the reasoning behind certain decisions


FTP Implementation of Plot system

I don't like the amount of comits there are just to get up to track

Second part needs to be done on Plot-Terra / Plot-Companion


Translate the code from C# to C++

400 RUB reward can be claimed by anyone who completes this task.

What do you propose replace System.Random?

What do you propose replace System.Random?

Something with similar characteristics: https://docs.microsoft.com/en-us/dotnet/api/system.random?view=net-5.0#remarks

We can basically use std::mt19937 or std::mt19937_64 (for 64-bit numbers) with uniform_int_distribution where we can put our Ranges

The  header has pretty much all we need

If needed, we can wrap the PRNG into another class (but I don't really think it's worth it)

charp version do not use System.Random in the return statements
We can use next style:
namespace ...
{
     bool NextBool()
     {
          auto generator = make_mt19937();
          std::uniform_int_distribution<> random(0, 1);
          return random(generator);
     }

     bool NextBool(const std::mt19937& generator)
     {
          std::uniform_int_distribution<> random(0, 1);
          return random(generator);
     }
}

OR)))
     bool NextBool()
     {
          return rand() % 2;
     }

Do we need to use std::knuth_b?

This might make the user dependent std::knuth_b

Do we need to use std::knuth_b?

I don't really think so. The default engines are pretty good at generating random numbers

default engines?

I mean, the ones which are defined in the  header without any additional shuffling

I want to return the reward

I want to return the reward

Thank you. This reward will be used for other LinksPlatform priority issues.


Fix fire of global show event for IE

For IE create new Event object and set details separately because initEvent doesn't support this and we expect lately to have detail property on event object.

Thanks 👍


Vscode crashes on WindowsPE at opening files

VSCode Version: 1.20.1 Date 2018-02-12T 15:34:36:336Z
OS Version: Windows PE based on Windows 10240

Steps to Reproduce:
1.Download VSCode-win32-x64-1.20.1.zip and unpack to x:\program files
2. start code.exe (its starting - yipee!)
3. press ctrl-o or ctrl-k /ctrl-o and it will crash
Does this issue occur when all extensions are disabled?: Yes
I found some Logs in X:\windows\temp which i´m attaching here.
temp.zip
I will try to reproduce this on a newer PE version, but not today.
edit
Same with Windows 10.0.17093.1000

I'm not familiar with WindowsPE, but from reading this https://docs.microsoft.com/en-us/windows-hardware/manufacture/desktop/winpe-intro it seems it may not be suited for running VSCode. What's your use case?
Here's our wiki on collecting crash data: https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues#windows

@RMacfarlane ; @bpasero
on the winpe-intro-site is written:
Install Windows by using apps or scripts from a network or a local drive.
Add your own custom shell or GUI to automate these kinds of tasks.
Thats exactly what I´m doing. Until now the scripts was batch but Powershell is way more powerful.
For the moment i´m using notepad++. There i write the script mark one or more lines copy them to the powershell console and execute them. Yes, its working, but....
Also on the site you mentioned are some limitations of PE
.MSI installation files.
I will try to install debug and diagnostic tool to provide an proper debug log, but....

Hi
as expected/feared the debug diagnostic tool isn´t working proper with the limitations of PE..
So i ´m not able to provide any additional and logs.
Now I don´t know - on one hand I have to use powershell, which is supported by ms.
On the other hand working with Powershell on PE is quite annoying. There are some tools - also provided by MS - but none of this are working properly.
At this time i have an editor and have to copy line by line and function by function to the powershell window and execute them. This is sadly slow and annoying, but the only working way.
You decide if you want to close this issue - I would love to use VSCode for daily work, but its not in my capabilities to do this,
However - thanks!

I am not sure what Windows PE is, but we do not officially support it. Closing.

Windows PE is Windows PreinstallEnvironment
https://docs.microsoft.com/en-us/windows-hardware/manufacture/desktop/winpe-intro?f=255&MSPPError=-2147217396
and thats the environment I have to work.
But its ok to close it, because its a PE (or to be exact dotnet core issue) which will be fixed with dotnetcore 3.
Maybe.
Probably.


chore: remove wrongly placed file #199

Tbh I don't think so. The weblate PR was green before this was merged. A re-run helped. This is just to clean things up. 🤷♂


gs backend misbehaves with backpressure

if you are copying from gs to a slow destination (such as bigml), then gsutil's network connection can time out, which is not recoverable

This is normally only triggered using --stream-size and slow destinations like BigML, because --stream-size causes a mismatch between input stream boundaries and output stream boundaries. This mismatch can lead to long pauses partway through an input stream.
Fixing this depends on fixing #112.


add models & JsonAdapters

Added models and its JsonAdapters for /evaluations and /events endpoints.
I've create bucketeer2 module as a temporary workspace.
package structure
main package: io.bucketeer.sdk.android
I'm planning to put everything internal into io.bucketeer.sdk.android.internal and its sub packages.

I've added some missing fields(Event.environment_namespace & GetEvaluationRequest.user_evaluations_id) and missing RegisterEventsResponse

thanks!


Ensure page selector works with page aliases

Right now, page aliases fire the original page ID back to the page selector, making it impossible to choose a page alias from the page selector widget. We should make sure that original page alias ID is the one fired into the page_selector.js JS, if you click on an alias.

I would like to have this fixed. We are using the page selector and aliasses and eventually ran into this issue.


[function]regexp_instr

Feature request
as mentioned in #15385, we need a function to search for the pattern and return its index. However, the regexp expression is more flexible, so we better support this function: https://dev.mysql.com/doc/refman/8.0/en/regexp.html#function_regexp-instr

Also, because like in sr is vectorized, is faster than regex, so it's better to support a like_instr as well so we can provide better performance in some cases.

@wangsimo0 could you please assign this issue to me? thank you!

@blanklin030 sorry we missed your message, now it's assigned to you. Do you still want to work on it?🙏🏻

@kateshaowanjou yep, i want to work on it, please assign this issue to me, thank you!

@wangsimo0 could you please assign this issue to me? I wanna try,  thank you!

@kateshaowanjou Hi kate, could you please assign this issue to me?

#36958


Mark trailing whitespace after type name in directives as markup

#1617
Also verified this in VS.

@NTaylorMullen

Add/use SpanKindInternal.None / SpanKind.None and see if the editor code properly ignores it.

Nice idea! It worked. I wonder if there are other places we can use this where we don't want any intellisense.


New Secure Password Generator Command

Creates new password and converts certain characters to numbers and symbols.

@WillBurnham we moved the repo, can you PR here: https://github.com/pyhoneybot/honeybot, thank you!


Revert "IBM Provider: convert the fetched secret to map to access the properties"

Reverts external-secrets/external-secrets#2597

/approve


form-js-heroes

studying, but i can't understand localStorage and Json.
sofrendo com javascript - episódio 3

studying, but i can't understand localStorage and Json.
sofrendo com javascript - episódio 3


[New Feature] Runner: Do not scroll up summary

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
While a collections runs the runners shows a summary at the top:

During execution the summary scrolls up. In the end the runner looks like this:

It is possible to see the summary always during the execution of a collection.
The auto scroll is very useful. I just want to see the summary while the runner executes the collection.
Mockups or Images of the feature
The runner view should always look like this:

This will be a valuable addition. Right now, we're focused on fixing essential onboarding issues.
We value quality improvements and understand their importance, but we need a bit more time before we can get to them.


Add wpa_supplicant to rf-sombox installer

Added installer for crypto dependencies which are stored for now in https://github.com/emaclean-adi/wpa_supplicant.git
We probably want to move that repo somewhere else

can you squash all of these?
Thanks
-Robin

Do you want me to just close this pull request?

I can review atomic requests, but patches on top of patches, make things hard.
So, yes, if you can fix things up, and submit again, it would be great.


Doesn’t work for iPhone 12 Pro Max iOS 14.4.1

Doesn't work for iOS 14.4.1

It works on my iPad Air 4 (which has the same chip as the 12 pro max) on 14.5 so it should work for you.
Can you tell me:
Which websites you tried
Is dark mode turned on your phone?

Closing due to lack of update. If you're still experiencing issues let me know.


WIP Build index image with latest NHC + SNR for megred PRs to main

Add build step for building and pushing versioned NHC + SNR + index images
Do not merge yet! Needs to be moved to postsubmit once it works

/hold
needs to be moved to postsubmit

login to Quay does not work because of missing secrets in the presubmit job (it can't read the medik8s secrets in pre submit for security reasons). I will move to postsubmit and merge, let's see how it goes. Will ask for review afterwards.

/hold cancel


Updated x2Go scripts for xubuntu and xfce

I've updated the scripts for setting up X2Go with xubuntu and xfce - the current versions don't run successfully.  Also, I've added info for xubuntu on how to disable compositing which significantly impacts performance of remote desktop, even with X2Go.  Unfortunately, I can't find a way to include this in the script because it must be run within a terminal within the graphical desktop - I've added a short set of steps at the end of the ReadMe that explains the steps for this.

LGTM


Support version message in _info_handler function. Solve #98.

Reading the documentation I found the first message contains the actual version of the websocket stream.
The expected message is:
{
   "event":"info",
   "version": 1
}

The patch check if the message have the 'version' key in the message, log the version and exit the function.

Thanks!


Get port working

Not 100% related, but https://github.com/conduit/conduit-sprint/blob/get_port_working/lib/conduit/sprint/parsers/query_port_status.rb#L41 has it as COMPlETED with a lowercase L, where I'm assuming it should be all uppercase?

:+1:


Segmentation fault when running gen_nn_ops.fractional_avg_pool

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
2.10.0
Custom Code
Yes
OS Platform and Distribution
Ubuntu 22.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
segfault happens with negative list elements.

Standalone code to reproduce the issue
import tensorflow as tf
import os
import numpy as np
from tensorflow.python.ops import gen_nn_ops
try:
  arg_0_tensor = tf.random.uniform([5, 20, 30, 3], dtype=tf.float64)
  arg_0 = tf.identity(arg_0_tensor)
  arg_1_0 = 2
  arg_1_1 = -5.267949192431123
  arg_1_2 = -52.58578643762691
  arg_1_3 = 1
  arg_1 = [arg_1_0,arg_1_1,arg_1_2,arg_1_3,]
  arg_2 = True
  arg_3 = True
  deterministic = True
  seed = 87654321
  seed2 = 341261001
  out = gen_nn_ops.fractional_avg_pool(arg_0,arg_1,arg_2,arg_3,deterministic=deterministic,seed=seed,seed2=seed2,)
except Exception as e:
  print("Error:"+str(e))

import tensorflow as tf
import os
import numpy as np
from tensorflow.python.ops import gen_nn_ops
try:
  arg_0_tensor = tf.random.uniform([1, 10, 10, 1], dtype=tf.float64)
  arg_0 = tf.identity(arg_0_tensor)
  arg_1_0 = True
  arg_1_1 = -0.35668935305391647
  arg_1_2 = -0.7209753581353426
  arg_1_3 = -87
  arg_1 = [arg_1_0,arg_1_1,arg_1_2,arg_1_3,]
  arg_2 = True
  arg_3 = True
  deterministic = True
  seed = 87654321
  seed2 = 341261001
  out = gen_nn_ops.fractional_avg_pool(arg_0,arg_1,arg_2,arg_3,deterministic=deterministic,seed=seed,seed2=seed2,)
except Exception as e:
  print("Error:"+str(e))

### Relevant log output

```shell
2023-01-07 13:44:10.489552: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.493914: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.494017: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.494307: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-01-07 13:44:10.494924: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.495025: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.495113: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.840688: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.840834: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.840928: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:980] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2023-01-07 13:44:10.841010: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1616] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 4263 MB memory:  -> device: 0, name: NVIDIA GeForce GTX 1660 Ti, pci bus id: 0000:01:00.0, compute capability: 7.5
Error:{{function_node __wrapped__FractionalAvgPool_device_/job:localhost/replica:0/task:0/device:CPU:0}} Fractional average pooling is not yet supported on the batch nor channel dimension. [Op:FractionalAvgPool]
Error:{{function_node __wrapped__FractionalAvgPool_device_/job:localhost/replica:0/task:0/device:CPU:0}} Both seed and seed2 should be 0 if deterministic is false. [Op:FractionalAvgPool]
Error:Expected bool for argument 'pseudo_random' not -69.0.
Error:Value for attr 'T' of uint32 is not in the list of allowed values: float, double, int32, int64
	; NodeDef: {{node FractionalAvgPool}}; Op<name=FractionalAvgPool; signature=value:T -> output:T, row_pooling_sequence:int64, col_pooling_sequence:int64; attr=pooling_ratio:list(float),min=4; attr=pseudo_random:bool,default=false; attr=overlapping:bool,default=false; attr=deterministic:bool,default=false; attr=seed:int,default=0; attr=seed2:int,default=0; attr=T:type,allowed=[DT_FLOAT, DT_DOUBLE, DT_INT32, DT_INT64]> [Op:FractionalAvgPool]
Segmentation fault

</details>

@nimashiri,
I tried to execute the mentioned code and the issue was fixed in the latest stable v2.11 and nightly. Kindly find the gist of it here.

@nimashiri,
As it was the bug on tensorflow v2.10, it has been resolved on the latest stable tensorflow v2.11 where we can observe that Segfault error has not been raised. Thank you!

Also this one:
import tensorflow as tf
import os
import numpy as np
from tensorflow.python.ops import gen_nn_ops
try:
  arg_0_tensor = tf.random.uniform([5, 20, 30, 3], dtype=tf.float64)
  arg_0 = tf.identity(arg_0_tensor)
  arg_1_0 = 1
  arg_1_1 = -15.267949192431123
  arg_1_2 = 0.0
  arg_1_3 = -1
  arg_1 = [arg_1_0,arg_1_1,arg_1_2,arg_1_3,]
  arg_2 = True
  arg_3 = False
  deterministic = True
  seed = 87654319
  seed2 = 341261001
  out = gen_nn_ops.fractional_avg_pool(arg_0,arg_1,arg_2,arg_3,deterministic=deterministic,seed=seed,seed2=seed2,)
except Exception as e:
  print("Error:"+str(e))

Also this one:
import tensorflow as tf
import os
import numpy as np
from tensorflow.python.ops import gen_nn_ops
try:
  arg_0_tensor = tf.random.uniform([5, 20, 30, 3], dtype=tf.float64)
  arg_0 = tf.identity(arg_0_tensor)
  arg_1_0 = 1
  arg_1_1 = -15.267949192431123
  arg_1_2 = 0.0
  arg_1_3 = -1
  arg_1 = [arg_1_0,arg_1_1,arg_1_2,arg_1_3,]
  arg_2 = True
  arg_3 = False
  deterministic = True
  seed = 87654319
  seed2 = 341261001
  out = gen_nn_ops.fractional_avg_pool(arg_0,arg_1,arg_2,arg_3,deterministic=deterministic,seed=seed,seed2=seed2,)
except Exception as e:
  print("Error:"+str(e))

This issue was already tracking with #59198. And the initial issue for Segmentation fault was not able to reproduce in tf v2.11 as mentioned in the above comment. Kindly find the gist of it here where we can observe that Segfault error has not been raised and executed without any error and let us know if it serves the problem. Thank you!

Looks like an issue on cleanup side.
@learning-to-play

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The code snippet provided has returned the error which is catch through the exception, and doesn't crash with latest TensorFlow version. Please find the gist here.
The TensorFlow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


update sigtest to 2.2 everywhere

I think but am not certain that this is needed.

There are some other areas in the tck-dist where this shows up.

Ah, OK, got 'em. Cheers.


how to get the promise result

I have a async code, but I can not get the promise result. And I have not found some explame for it.Could you help me?

No description provided.

Hi lgl2017,
Sure, here goes.. most simple example I could think of
#[tokio::test]
    async fn test_promise_async() {
        let rt = init_test_rt();
        let jsvf = rt
            .eval(
                None,
                Script::new("test_prom_async.js", "Promise.resolve(123)"),
            )
            .await
            .expect("script failed");
        if let JsValueFacade::JsPromise { cached_promise } = jsvf {
            let res = cached_promise
                .get_promise_result()
                .await
                .expect("promise resolve send code stuf exploded");
            match res {
                Ok(prom_res) => {
                    if prom_res.is_i32() {
                        assert_eq!(prom_res.get_i32(), 123);
                    } else {
                        panic!("promise did not resolve to an i32.. well that was unexpected!");
                    }
                }
                Err(e) => {
                    panic!("prom was rejected: {}", e.stringify())
                }
            }
        }
    }

So, when the script passed to rt.eval return a Promise the QuickJsRuntimeFacade (rt) return a JsValueFacade::JsPromise which in turn has a CachedJsPromiseRef which has a method get_promise_result
What that method actualy does is open a channel and return the receiver, the sender is invoked by closures added to the promise reactions (then and catch) so a Result or Error is sent when the promise resolves.
since get_promise_result is an async function you can .await the result in async code


No metadata provided to the system on macOS Big Sur

Checklist:

[x] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES AS WELL

[x] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[x] I have double checked and can reproduce the issue

OS: macOS 11 Big Sur Developer Beta 9 (20A5384c)
GPMDP Version: 4.7.1
Issue Descriptions: macOS Big Sur introduces a media control centre that displays album art and other track info. This is all set to "undefined" in GPMDP.
Here's GPMDP compared to my own Pandora client, Epimetheus:

Steps to Reproduce:

Play some music on macOS Big Sur

Looks like the same issue as #3881 and #3888

Ah yep, looks like #3888 describes this exact issue. #3881 is probably related. Closing.


Have "changing" and "changed" signals in Range controls

Currently, there is only one value_changed signal in Sliders, which is called continuously as the user drags the thumb. This is fine to be able to see the change as we change the value, however when the change is heavy, having it to update continuously results in lag.
The idea is to have two signals then:
changing which is called when the value changes,
changed when the user finished to drag the slider, OR, has clicked once without dragging (having also changed the value).
Names aren't specially important, just pick some that make sense.
Note: I think it would also make UndoRedo easier to implement because we would know when editing finishes.

I would keep value_changed and add two new signals like drag_started and drag_stopped.
1st this would be backwards compatible and actually the value is being changed while you are dragging.

personally, I'd prefer released() as suggested in closed issue 7056

Is this being addressed at all?

@Favorlock It seems nobody is currently working on this feature. Pull requests are welcome :slightly_smiling_face:

I slightly updated the first post as I had this use case again but had to clarify something

For mouse detection I'd add slider_pressed and slider released.
Maybe we could approach this the Qt way and implement a 'tracking property, if tracking is false value_changed is only emitted when we change the value with keyboard or when we finish moving the slider with the mouse. With tracking set to true the signal would behave the same as the actual implementation.

Can I request the slider documentation be updated with the list of signals as well? I have great difficulty finding which signals are available for controls.
https://docs.godotengine.org/en/3.2/classes/class_slider.html#class-slider

@Giwayume All signals are in the documentation. Slider doesn't have specific signals, all its signals are inherited from Range, Control, etc.
Click the classes in the "Inherits" line for the documentation of each parent class.

@Giwayume They are only listed in the Range documentation, due to how inheritance works :https://docs.godotengine.org/en/3.2/classes/class_range.html#class-range

I found a GDScript solution, which works fine with mouse dragging (not tested on a touchscreen though)
extends VSlider

signal drag_started
signal drag_stopped

func _input(event):
	if event is InputEventMouseButton:
		if event.is_pressed():
			emit_signal("drag_started")
		else:
			emit_signal("drag_stopped")

reference: https://stackoverflow.com/a/51687109

This is going to emit even when the value didn't change.


statistics calculation out_of_range error

Hi, I got an error trying to do the first step of statistics calculation.
reseq illuminaPE -j {threads} -b {input.bam} --refIn {input.refin} --statsOnly --statsOut {output}

>>> 24-04-21 12:38:06: info:  Running ReSeq version 1.1 in illuminaPE mode
>>> 24-04-21 12:38:06: info:  Reading reference from data/mtro_02.fa
>>> 24-04-21 12:38:28: info:  Read in 28220 reference sequences.                     [15/1832]
>>> 24-04-21 12:38:29: info:  Reading mapping from data/tros_v7.bam
>>> 24-04-21 12:38:29: info:  Storing real data statistics in results/mtro_ref_stats.reseq
>>> 24-04-21 12:38:29: info:  Tiles will be ignored and all statistics will be generated like all reads are from the same tile.
>>> 24-04-21 12:38:43: info:  Starting PreRun
>>> 24-04-21 13:05:09: info:  Finished PreRun
Total number of reads: 903097225
Read length: 1 - 150
Read length on reference: 19 - 221
Quality: 2 - 37 ( 33-based encoding )
Mapping Quality: 0 - 60
Maximum InDel Length: 70
>>> 24-04-21 13:05:09: info:  Detected non-extendible sequence 'ACCCTAACCCTAACC' as adapter for read 2
>>> 24-04-21 13:05:09: info:  Detected non-extendible sequence 'TGTTTGTCTTGTGGTCGTGTTCTGTAT' as adapter for read 2
>>> 24-04-21 13:05:09: info:  Detected non-extendible sequence 'TTAGGGTTAGGGTT' as adapter for read 2
>>> 24-04-21 13:05:09: info:  Detected adapter 2: ATGTATGATGGGGTGGTGTAATAGGTAGATGTATGAT
>>> 24-04-21 13:05:09: info:  Detected adapter 1: AGATCGGAAGAGCACACGTCTGAACTCCAGTCAC
>>> 24-04-21 13:05:11: info:  Starting main read-in
>>> 24-04-21 13:11:49: info:  Read 5% of the reads.
>>> 24-04-21 13:18:36: info:  Read 10% of the reads.
>>> 24-04-21 13:25:43: info:  Read 15% of the reads.
>>> 24-04-21 13:33:10: info:  Read 20% of the reads.
>>> 24-04-21 13:40:50: info:  Read 25% of the reads.
>>> 24-04-21 13:49:01: info:  Read 30% of the reads.
>>> 24-04-21 13:57:22: info:  Read 35% of the reads.
>>> 24-04-21 14:05:54: info:  Read 40% of the reads.
>>> 24-04-21 14:14:26: info:  Read 45% of the reads.
>>> 24-04-21 14:22:56: info:  Read 50% of the reads.
>>> 24-04-21 14:31:27: info:  Read 55% of the reads.
>>> 24-04-21 14:39:54: info:  Read 60% of the reads.
>>> 24-04-21 14:48:42: info:  Read 65% of the reads.
>>> 24-04-21 14:52:11: info:  Finished 5% of the bias fits.
>>> 24-04-21 14:54:47: info:  Finished 10% of the bias fits.
>>> 24-04-21 14:57:11: info:  Finished 15% of the bias fits.
>>> 24-04-21 14:58:55: info:  Finished 20% of the bias fits.
>>> 24-04-21 14:58:58: info:  Read 70% of the reads.
>>> 24-04-21 15:00:21: info:  Finished 25% of the bias fits.
terminate called after throwing an instance of 'std::out_of_range'
  what():  vector::_M_range_check: __n (which is 29095) >= this->size() (which is 29091)

Hi,
thank you for reporting the issue. Would it be possible to provide me with the input files? That would make it much easier for me to find what is causing the crash and to fix the bug.

Hi, possible but this is more than 65Gb of data, is that fine with you?

As long as you can provide a download link, that is no problem at all.

Could you provide me with an email address where I can send you the data?

<EMAIL_ADDRESS>

Hi, did you get my email?

I pushed the bugfixes to the devel branch.

Unfortunately, I am getting a very similar error:
terminate called after throwing an instance of 'std::out_of_range'
  what():  vector::_M_range_check: __n (which is 23868) >= this->size() (which is 10000)

what caused this in the first place?

I get the exact same error when using the development version, by the way


Prometheus metrics output

Hi,
Thanks for this great tool.
When running it for a kubernetes cluster, it could be nice to output the result as prometheus metrics, so it could be scraped, allow graphing and generate alerts.

Hi @jfrabaute ,
This is a great idea. I worked on something like this for a bit that reported the number of high, medium, and low vulnerabilities. It would definitely be cool to add more metrics beyond these and could be a feature I or anyone else in the community adds to flan in the future.


Mixing up Models, Views and Controllers

Hey
First of all, it's not directly an issue but something I noticed: It seems that on some points the MVC-Pattern is violated. For example, the widget in "views/widgets/duplicated_computernames_widget.php" contains SQL statements which, I believe, belong into the corresponding model.
However, this isn't really an issue but just something I noticed and wanted to mention.
I do have some more questions about the tool in general (not about the useage but more about contributing and the general "direction" it's heading) but I couldn't find an address to write to?
Regards
Hansfreddy

You're right, I'm converting the current codebase to better match the MVC design pattern. My goal is to get most of the ui to use rest queries. There's a munkireport-dev mailing list to discuss future development. For a quick back-and-forth you could also check out the macadmins.org which has a #munkireport slack channel


HTTPAltRequestValidator fetches 401, but original request is not repeated

Thanks for this awesome library, would love to use it in production!
The problem I'm currently facing is that when my original request returns a 401, the HTTPAltRequestValidator works as expected (i.e. it does the login call and sets it on the client), but my original request is not repeated as I see in a proxy running on my mac.
this is my request:
HTTPRequest {
                $0.url = URL(string: "https://...")
                $0.maxRetries = 3
                $0.method = .get
            }

and this is my setup:
        let client = HTTPClient.shared

        let authValidator = HTTPAltRequestValidator(statusCodes: [.unauthorized]) { request, response in
            // If triggered here you'll specify the alt call to execute in order to refresh a JWT session token
            // before any retry of the initial failed request.
            return try! Endpoint.login("...", "...").httpRequest()
        } onReceiveAltResponse: { request, response in
            // Once you have received response from your `refreshToken` call
            // you can do anything you need to use it.
            // In this example we'll set the global client's authorization header.
            if let authorizationHeader = response.headers[.authorization] {
                client.headers.set(.authorization, authorizationHeader)
            }
        }

        client.validators.insert(authValidator, at: 0)

I get to the the client.headers.set(.authorization, authorizationHeader) - but then the original request is not repeated... what am I missing here?
Thanks a lot for your help!

I've solved it in https://github.com/immobiliare/RealHTTP/commit/8bbe0fb9a4d95a8fafc1f1c77f68c0c3a9611720 while the https://github.com/immobiliare/RealHTTP/commit/45cea68e41fbfabb55ef903dadbff7ab7f879a82 introduces several unit tests for the validator itself.
I'll release it shortly.

Hi @malcommac , AWESOME!
With the new branch .package(name: "RealHTTP", url: "https://github.com/immobiliare/RealHTTP.git", branch: "fix/27-alt-validator"), my test just turned green, thanks a lot!


Ensure country with 0 priority will get matched before country with or 2 priority

Right now a query to +61 return "cc" country instead of "au" (which it should be). This pull request address this.

Thanks for your contribution! Can you add a test to test.js to demonstrate this change? That would also make it easier for me to understand what's going on, since I didn't write this component.
Can you also remove dist/bc-countries.min.js from the PR? I'll generate that for the next release.


Support for Angular 1.X

Hello,
In our project we are using Angular 1.4 and we've started to use new router as we want go in component based approach with flux architectural pattern. We are hoping with this approach we could migrate more easily to Angular 2.0.
So far all work great, still there are some issues with the router. Like we cannot have multiple named viewports at one view and so on. We have address them some time ago and were hoping they would be fixed in next router release.
Now we discovered that router source are moved to Angular 2.0 repository and seems like there will be no further releases of angular-new-router, or?
Can you support us now how shall we proceed here to be able to use new router with bug fixes with Angular 1.X ?
Is it still possible to have angular-new-router released for Angular 1?
Shall we use somehow modules from Angular 2?
Thanks in advance.
Regards,
Beata

"Is it still possible to have angular-new-router released for Angular 1?"
The answer for this is a definitive yes, that is their announced plan. But I am really hoping that it will not wait until the 1.5 (it was suppose to be a part of 1.4)...

I got answer from the angular team on angular/angular repo:
Check here:
https://github.com/angular/angular/issues/3065#issuecomment-121687595
Brian (btford) wrote that he is working on it this week. Seems to me it shall be released soon.

Any progress?

This is why I didn't bother with the new router.  promises, promises, no delivery for 1.X

Any progress?  1.5 is on the way, so is router included?  I can't really find any good guidance on this.


CTD When Entering Character Menu after Uninstall

After Uninstalling Banner Kings Clicking on the Character Menu causes CTD
this is on the Current 1.0.2 Bannerlord Build as of 1/1/23 & the Current Banner Kings Build (last updated 12/21/22)

The mod is not removable from campaigns.


Single Image "filepond--item" preview for multiple files.

First of all, amazing work pqina and Rik Schennink for this amazing plugin. Love it!
Is there a way to remove the little preview ("filepond--item") for multiple images in FilePond please? If not can this be implemented within the plugin please?
Something similar to the following example (but without the .txt obvs xD) :

Say for example I need to upload a folder which has 150 attachments. Currently that would create 150 "filepond--item" which would create a lot of clutter on the page. Like so:

Once again I thank you for your work!

Do you mean grouping them under one item?

Yes and like for example have: 15 Attachments Uploaded. Instead of one for each item.


[NWPS-1135] Feature/nwps 1135 be new featured content component

@smanivasag Done, I add the reference


A small request - Please add a warning to the homepage

First, thank-you for the great tutorial/workshop you have put together here. It is immensely helpful to those of us trying to explore running K8 on AWS.
I do, however, believe it would be very beneficial to add a warning, either on the homepage or even throughout the sections, that many of the EC2 instances spun up during this workshop do not qualify for the free tier on AWS. They will get billed for usage while running through the workshop. A repeated footer section reminding people to destroy their clusters after each section would also be very helpful, in case people skip over this initial warning.
In hindsight, I realize this (cost) is probably obvious to those writing the workshop, but I foolishly started going through the workshop without paying close enough attention to what instance sizes Kops was using and got a bill the next day. Granted, it was not for a lot - only a couple dollars as I was diligent about tearing things down quickly after completing a section - but I still received a bill despite my diligence. I believe making people aware of this fact up front will help ensure no one is surprised, especially those not familiar with how AWS pricing works.
I would hate for people to get surprised by a bill like I did, or worse yet forgetting to tear down the cluster after each section and getting a much larger bill, as it may give people an (undeserved) bad impression of both the workshop and AWS.
Again, thank-you very much for all your hard work and I truly do appreciate the effort you have put into this!

Hi Jason, I think we have someone working on this issue here https://github.com/arun-gupta/kubernetes-aws-workshop/issues/88
But you are correct, we should at least provide a warning as the work is being done to cover it. Do you think if it makes sense to add this at https://github.com/arun-gupta/kubernetes-aws-workshop
or in https://github.com/arun-gupta/kubernetes-aws-workshop/blob/master/prereqs.adoc ?

this is resolved

@dalbhanj Thanks for the update. Sorry for the late reply.


Add HelmRelease images to cluster manifest

#443
Example usage
$ flux-local get cluster --path tests/testdata/cluster/ -o yaml --enable-images
---
clusters:
- path: tests/testdata/cluster
  kustomizations:
  - name: apps
    namespace: flux-system
    path: tests/testdata/cluster/apps/prod
    helm_repos: []
    helm_releases:
    - name: podinfo
      namespace: podinfo
      chart:
        name: podinfo
        repo_name: podinfo
        repo_namespace: flux-system
      images:
      - public.ecr.aws/docker/library/redis:7.0.6
      - ghcr.io/stefanprodan/podinfo:6.3.2
    cluster_policies: []
  - name: flux-system
    namespace: flux-system
    path: tests/testdata/cluster/clusters/prod
    helm_repos: []
    helm_releases: []
    cluster_policies: []
  - name: infra-configs
    namespace: flux-system
    path: tests/testdata/cluster/infrastructure/configs
    helm_repos:
    - name: bitnami
      namespace: flux-system
      url: https://charts.bitnami.com/bitnami
      repo_type: default
    - name: podinfo
      namespace: flux-system
      url: oci://ghcr.io/stefanprodan/charts
      repo_type: oci
    - name: weave-charts
      namespace: flux-system
      url: oci://ghcr.io/weaveworks/charts
      repo_type: oci
    helm_releases: []
    cluster_policies:
    - name: test-allow-policy
  - name: infra-controllers
    namespace: flux-system
    path: tests/testdata/cluster/infrastructure/controllers
    helm_repos: []
    helm_releases:
    - name: metallb
      namespace: metallb
      chart:
        name: metallb
        repo_name: bitnami
        repo_namespace: flux-system
      images:
      - docker.io/bitnami/metallb-controller:0.13.7-debian-11-r29
      - docker.io/bitnami/metallb-speaker:0.13.7-debian-11-r28
    - name: weave-gitops
      namespace: flux-system
      chart:
        name: weave-gitops
        repo_name: weave-charts
        repo_namespace: flux-system
      images:
      - ghcr.io/weaveworks/wego-app:v0.24.0
    cluster_policies: []

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

Comparison is base (b77ba60) 92.42% compared to head (af813cd) 92.95%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #445      +/-   ##
==========================================
+ Coverage   92.42%   92.95%   +0.52%
==========================================
  Files          19       19
  Lines        1783     1817      +34
==========================================
+ Hits         1648     1689      +41
+ Misses        135      128       -7

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


ADR-15: Make reverse relations aware of versions.

ADR for fixing reverse relations – please read the text and discuss :)

Codecov Report

Merging #477 (15088d0) into master (e030e41) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #477   +/-   ##
=======================================
  Coverage   91.80%   91.80%
=======================================
  Files         289      289
  Lines       17619    17619
  Branches     1425     1425
=======================================
  Hits        16175    16175
  Misses       1183     1183
  Partials      261      261

Continue to review full report at Codecov.

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

It sounds like a sane default (even an exception being thrown would be better than what happens now).
Are there any circumstances where you wouldn't want this, and would want to opt for an older version for instance ?

Are there any circumstances where you wouldn't want this, and would want to opt for an older version for instance ?

I can't think of any – this should always just work. As it notes there is a TODO around whether those reverse relations should include magic versioning, but I'm not including that for now.


Unable to download

I have atom 1.28.2 x64, and a Mac OS High Sierra 10.13.6 (17G65)

@peterth3geek
Sorry, the theme is deprecated. You can download and install the original version: nucleus-dark-ui.
A long time ago the nucleus-dark-ui had released an immersive version.
Thx.


Improvements to streams usages

Various improvements of current usage of streams.

I do tend to prefer Stream.of over Arrays.stream, only because it's fewer characters, but either way works fine

I did not know those two are equivalent. I think I like Stream.of better as well so I'll change them to that.

In 80eddf7 I converted Arrays.stream to Stream.of. I also made some simplifications to PropertyTypeTest


v-ref is not working with

Don't know if it's working as planned, but I would like v-ref to work in both cases :)
Also, can't find any references in the docs about the case
http://jsfiddle.net/4ty2v4fc/52/
P.S. Thank for implementing 'wait-for' mechanism. Saved me from a lot of trouble!

It is weird behavior but then you are trying to do something weird by having a v-component on a <template> tag. It doesn't really make sense, because <template>s are explicitly hidden from the DOM, while components are designed to be shown. And if you look at the DOM after your code runs appears as this:

gfd

gfd

So for some reason the <template> completely disappeared. Anyway you might want to rethink that. If you want a component, don't put it on a <template>, and if you want a reference to the template element, use v-ref instead.

I don't see anything about using v-component with

<template> tags are treated as symbolic wrappers for block v-if and v-repeat, it doesn't work with v-component.

Okay, understood.
Another question: would you consider to make <template> support those directives, please? :)
It's really nice to use it sometimes, especially in such costructions as table/tr/td.
My guess is you could just redirect this.$el from <template> to top level element of component's template with check for just one of it. And it would render as there was no <template> at all.

I'm not sure if I understand the use case for table/tr/td - can you give an example?

http://jsfiddle.net/4ty2v4fc/56/
It's about templates' placing. I keep templates in .tpl files near .js files.
From time to time I use v-component on a plain div without any classes serving as template holder. It looks like this:

 <<== this one is just a holder without any classes or visual decoration

...

But in the example case I'm forced to keep component item template container separated from its content (tbody is the container).
The thing is I want to keep tbody with its content in one tpl file.
But in the case I can't use extra divs in table structure.
It's the same with components whose containers are display: inline-block;

Why are you using v-component and v-ref to do that though? It seems to me you simply want to access that raw template?
You can do something like
<template id="item-template">

</template>

childComponent: {
  template: '#item-template'
}

Or use v-el on the template:
<template v-el="template">
</template>

// access it
this.$$.template

Nah. Accessing template itself is no the case.
The case is about decomposition.
I want to decompose a page for several components: Content, TopPanel, Header, Footer, etc.
Also, there are smaller components. Using

Main

Articles

Feedback

The block has its container -
. Notice its a actual part of TopPanel with its visual decoration.
So now I want to mount TopPanel to my page. And there are, actually, two ways:
First one:
...

...

Here we will get extra wrapper - div without any actual role or use.
And the Second one:
...

...

Now we've got rid of extra wrapper, but TopPanel container/starting point got detached from its content. It makes a mess in templates and preventing TopPanel from normal reusing.

I guess you are now using replace: true :)


AttributeError: 'loci.formats.ImageReader' object has no attribute 'pages'

Hello,
I successfully ran the program with the 2D sample data to get an evaluation metric in WSL2, new conda env, python 3.8.  I am getting an error with my 2D CODEX images and masks you can find here:
https://drive.google.com/file/d/1gVcyTuSwLg0-bTMXKL3D9fD55AWzoING/view?usp=sharing
https://drive.google.com/file/d/1zH_ETFekRvnabLy_ESleKq-QP2QHyKm-/view?usp=sharing
I am using from terminal:
python seg_eval_main.py example_data/imgs/spleen.ome.tiff example_data/masks/stack.ome.tiff
The error I am getting is:
04-Dec-23 16:56:06 - bfio.bfio.BioReader - WARNING  - Python backend only works for tiled OME Tiff files with minimum tile size of 1024x1024, switching to bioformats backend.
04-Dec-23 16:56:07 - bfio.bfio.BioReader - WARNING  - Python backend only works for tiled OME Tiff files with minimum tile size of 1024x1024, switching to bioformats backend.
Traceback (most recent call last):
File "seg_eval_main.py", line 28, in
seg_metrics = read_and_eval_seg (img_path, mask_path, PCA_model, output_directory)
File "/home/smith6jt/CellSegmentationEvaluator/SimpleCSE/read_and_eval_seg.py", line 24, in read_and_eval_seg
img["data"] = aimg.get_image_data()
File "/home/smith6jt/anaconda3/envs/SegEval/lib/python3.8/site-packages/aicsimageio/aics_image.py", line 711, in get_image_data
return self.data
File "/home/smith6jt/anaconda3/envs/SegEval/lib/python3.8/site-packages/aicsimageio/aics_image.py", line 529, in data
return self.xarray_data.data
File "/home/smith6jt/anaconda3/envs/SegEval/lib/python3.8/site-packages/aicsimageio/aics_image.py", line 469, in xarray_data
and dimensions.DimensionNames.MosaicTile in self.reader.dims.order
File "/home/smith6jt/anaconda3/envs/SegEval/lib/python3.8/site-packages/aicsimageio/readers/reader.py", line 532, in dims
self._dims = Dimensions(dims=self.xarray_dask_data.dims, shape=self.shape)
File "/home/smith6jt/anaconda3/envs/SegEval/lib/python3.8/site-packages/aicsimageio/readers/reader.py", line 359, in xarray_dask_data
self._xarray_dask_data = self._read_delayed()
File "/home/smith6jt/anaconda3/envs/SegEval/lib/python3.8/site-packages/aicsimageio/readers/bfio_reader.py", line 208, in _read_delayed
self._tiff_tags(),
File "/home/smith6jt/anaconda3/envs/SegEval/lib/python3.8/site-packages/aicsimageio/readers/bfio_reader.py", line 218, in _tiff_tags
for code, tag in self._rdr._backend._rdr.pages[0].tags.items()
AttributeError: 'loci.formats.ImageReader' object has no attribute 'pages'
It would appear that the reason I was able to run the example data, but not my data is because my data requires the use of Bioformats, but when I do
conda install -c bioconda bioformats
I get:
Solving environment: /
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed
UnsatisfiableError: The following specifications were found
to be incompatible with the existing python installation in your environment:
Specifications:

bioformats -> python[version='2.7.|3.5.|3.6.*']

Your python: python==3.8
If python is on the left-most side of the chain, that's the version you've asked for.
When python appears to the right, that indicates that the thing on the left is somehow
not available for the python version you are constrained to. Note that conda will not
change your python version to a different minor version unless you explicitly specify
that.
The following specifications were found to be incompatible with your system:

feature:/linux-64::__glibc==2.35=0
feature:|@/linux-64::__glibc==2.35=0

Your installed version is: 2.35
If you have a workaround for the dependencies or perhaps reformatting our images I would appreciate any help.
Thanks,
Justin

This was probably a WSL issue and not anything to do with the code.  There have been a lot of issues with WSL lately.  Updating and upgrading WSL as well as a fresh install of CellSegmentationEvaluator fixed the problem for me.

great, closing issue.


Steps should be mentioned to enable EnableJWTGeneration prior changing the parameters under JWTConfiguration

Description:
EnableJWTGeneration should be enabled before changing the other properties in apim-manager.xml JWTConfiguration section, as per [2]. So better to add that step as the first point before mentioning other parameters in[1]
<EnableJWTGeneration>true</EnableJWTGeneration>

[1] https://docs.wso2.com/display/AM210/Configuring+WSO2+Identity+Server+as+a+Key+Manager
[2]

Affected Product Version:
apim.2.1.0

We will not be fixing this as this issue is based on an older version of the product.


Provide timestamp in microseconds value

Description
Can you provide a value such as Node's process.hrtime, very useful when playing with sound for instance.
Reproduction
Solution
Additional Information

React Native version: 0.34.0
Platform: both
Operating System: MacOS

Hmm, how would this be implemented? You couldn't use the typical native modules setup because it requires asynchronous bridging between js <-> native.

@lacker Exactly, and I don't know enough of the inner mechanisms of react native to make a pull request with my own solution, hence why I'm asking in case someone has a solution :)
Thanks

This actually exists already in React Native since v0.24. I've been using it for awhile but its not documented anywhere.
const currentTimeInSecondsToNanoPrecision = global.nativePerformanceNow();

It's used in the internal PerformanceLogger module.

why is there not more information about that ?
I really need that kind of tool to monitor my own function / performance.
When you're dealing with animations, with 16ms for a frame, milliseconds are not small enough.
Nobody cares for performance ?
PS: in a navigator performance.now() is given relative to a time origin which is generally the page creation. In React Native what is the time origin ? (simple curiosity)

@jniac: After digging around in the source code for a while, I found out that global.nativePerformanceNow() seems to call the native functions CACurrentMediaTime() on iOS, and clock_gettime() on Android.
Both of these native functions appear to have up to nanosecond resolution, and from some quick testing it seems like global.nativePerformanceNow() returns a floating point number representing the amount of milliseconds since device boot on both platforms.
Good question as to why it's not documented, it seems like it would be useful in a lot of cases. Maybe there's something that we're missing here from the whole picture.


fix: external secrets regression, missing team-admin values

fixes #827

@staticvoid255 if you have code that is missing from this PR, please add it. I am kindly asking to diff what is here with what you have, both for learning purposes, as well as completion.

Nothing that I put in that you haven't, but please describe the test you performed. Below are my thoughts which will prevent the environment breaking if deployed for GU:

Deploy latest release of Otomi (or preferrably 0.16.6 like GU are using)
Create a secret in Console
Verify ExternalSecret is synced to a k8s secret
Upgrade Otomi to this branch version
(optional) Ensure kubernetes-external-secrets.yaml or external-secrets.yaml is created in the values repo
(important) Ensure that creating such a file does not break things
Modify secret
Ensure sync
Create new secret
Ensure sync

I cannot do the upgrade for GU until this path has been tested given how many new features will be added, so it is imperative that this test be performed. Please confirm results here.

If it is still the case that adding kubernetes-external-secrets.yaml or external-secrets.yaml then running apply causes things to break, then it confirms something else that's in the back of my mind. So please do this step and confirm results.

I did the tests and this PR does indeed result in secrets getting synced without errors.

Great thanks! 👍


Bug report: getting JWT token now has PyJWT issues

I'm currently using PyJWT 1.7.1 as packaged in the docusign-esign python client, and recently in our dev environment after a deploy we are getting the following issue when attempting to get a JWT token via DSClient.get_token_jwt()
NotImplementedError: Algorithm 'RS256' could not be found. Do you have cryptography installed?

This is using the Docusign example, and the specific code this is failing at is
api_client.request_jwt_user_token(
                client_id=DS_JWT["ds_client_id"],
                user_id=DS_JWT["ds_impersonated_user_id"],
                oauth_host_name=DS_JWT["authorization_server"],
                private_key_bytes=private_key,
                expires_in=3600,
                scopes=use_scopes,
            )

Any thoughts as to what's going on? We aren't able to use Docusign in any form without fixing this.

^ Sorry, should've been more specific. I had the cryptography package already installed, and I was getting this error. The version I was using was cryptography 38.0.1, which I verified based on pip list.
I proceeded to uninstall and reinstall cryptography via pip, and it looks like the version that was installed was version 2.8. I then reran this command and got the same output. I'm not sure what's going on at this point/why Docusign isn't picking up the pip installation of cryptography. For context, this is packaged in a Serverless framework Lambda function on AWS

Yes, my application is already configured with an RSA keypair, so I don't think this is the issue. This was basically working for a while (and is still working in our production stages), but once I made a new deployment in a dev stage to add features, this error came up

@vineetparikh  Please reinstall requirements from the requirements.txt file if you have updated to the latest version. We updated the code to support the latest pyjwt > 1.7 so the dependencies are having a miss-match.

^ After attempting to reinstall from pip, it looks like docusign-esign's pip wheel is not updated to support pyjwt>1.7.1: could you please fix that?

When I set the requirements file and attempt to install, I run into the issue described in https://github.com/docusign/docusign-esign-python-client/issues/153

^ in my case I don't have a local setup of docusign-esign-python-client, I'm setting this up via pip and because I'm using Serverless, I can't actually make changes locally and see them on AWS. I would need this PR to be merged in


An example of connecting actor system's event bus to an SSE source

No merging please. Just an example for now.
@aquamatthias the example is working now.
You can connect to the server through curl localhost:9000/connect. By accessing localhost:9000/postmessage, new messages appear in the terminal.

Thanks! Serves as example.


add : new profile

I have added my profile to the list in accordance with Hacktoberfest !

The PR is getting blocked from getting merged due to conflicting files, please sync the repository @rohitroy-github

I have solved the error.


Implement GNU/Linux elevation dialog

Make use of sudo-prompt Linux (beta) support, which attempts to use
gksudo, kdesudo or pkexec.
Fixes: https://github.com/resin-io/etcher/issues/277
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Pending extensive testing.

There is an issue in sudo-prompt being fixed here: https://github.com/jorangreef/sudo-prompt/pull/16

This won't be merged given the explanation on the linked sudo-prompt issue. We'll create a new module wrapping pkexec for GUI applications.


Meson encountered an error in file meson.build, line 298, column 1

I am on linux mint 8.1
I also want to say thanks for making a program like this. Not having a setpoint like program is the one thing that is keeping me from moving full time to linux.
'check' with pkg-config executable '/usr/bin/pkg-config'
Meson encountered an error in file meson.build, line 298, column 1:
Native dependency 'check' not found

you need to install check-devel or whatever it's called on mint. See this post here, it's still configure but the same principle applies:
http://who-t.blogspot.com.au/2014/05/configure-fails-with-no-package-foo.html

Thanks for the help! Check just didn't sound like a package to me. Once
again thanks for making this software.
On Jun 13, 2017 12:59 AM, "Peter Hutterer"<EMAIL_ADDRESS>wrote:

you need to install check-devel or whatever it's called on mint. See this
post here, it's still configure but the same principle applies:
http://who-t.blogspot.com.au/2014/05/configure-fails-with-
no-package-foo.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/libratbag/libratbag/issues/181#issuecomment-308013582,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab-u_ExZ0oSYV17tpEKIhqzvWyDaMuZOks5sDiMBgaJpZM4N4Aov
.

Can you indicate what on earth "check" actually is? I'm having a hell of a time finding anything about the library/project to say nothing of actually finding a package with the headers for it.

https://libcheck.github.io/check/


ResourceManager.LoadPriFiles() throws InvalidCastException

Describe the bug
Loading ResourceMap failed: System.InvalidCastException: Specified cast is not valid.
  at System.Runtime.InteropServices.Marshal.ThrowExceptionForHR(Int32 errorCode)
  at WinRT.IObjectReference.As[T](Guid id)
  at ABI.System.Collections.Generic.IEnumerable`1.CreateMarshaler(IEnumerable`1 obj)
  at ABI.Windows.ApplicationModel.Resources.Core.IResourceManager.LoadPriFiles(IEnumerable`1 files)
  at Windows.ApplicationModel.Resources.Core.IResourceManager.LoadPriFiles(IEnumerable`1 files)
  at TestWinUI3.App.LoadMyPriFiles()

Steps to reproduce the bug

Steps to reproduce the behavior:

just call the ResourceManager.Current.LoadPriFiles()...

Expected behavior
pri file loaded successfully
Screenshots

Version Info

NuGet package version:
[Microsoft.WinUI 3.0.0-preview2.200713.0]

Windows app type:

UWP
Win32

yes

Windows 10 version
Saw the problem?

Insider Build (20226)
yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
yes

Xbox

Surface Hub

IoT

Additional context

@stevenbrix or @alwu-msft thoughts here? my hunch is a cs winrt bug?

Yeah, most likely a CsWinrt bug. @Scottj1s

Closing as this does not repro with WinAppSDK 1.3.


Include opt in tar functionality?

Just want to get some thoughts on whether you would want to include tar'ing as part of this loader, given it's suggested in the README. I'm personally not fan of having to write a post build script for all my remote apps to tar the output and copy it into my assets folder.
I imagine it could look something like
use: [
  {
    loader: "dts-loader",
    options: {
      name: nextFederationConfig.name, // The name configured in ModuleFederationPlugin
      exposes: nextFederationConfig.exposes,
      typesOutputDir: ".wp_federation", // Optional, default is '.wp_federation'
      tarOutputDir: "public", // and only tar if this exists, default to not tar
    },
  },
],

After a bit of digging, it looks like it may not be possible with a loader? happy to close if so

Thanks for the suggestion, it may be challenging, but anyway, I'd keep a single responsibility of the loader, and then it's up to the consumer to decide how to use these types.


Ability to run a specific global scenario test

Hello,
Following on from #163 I converted all my features which use installsAfter to use the global scenarios test functionality and removed them from specific feature tests since they always fail for the default test.
Whilst in my example for #163 I only had one feature, my work project has over 20 features.
The issue now is that the GitHub Actions build times out (>6 hours) running the global scenarios for 20+ features.
This issue could be solved in one of the following ways:

Allow running a specific global scenario test, providing it's name on the command line, such as with a command line switch --global-scenario=<name-of-scenario>.
This would enable use of a build matrix for global scenarios too, such as the one used in feature-template to run specific feature tests individually. This would also aid debugging failed scenarios and allow them to be run in parallel.

Use a scenarios.json within the features's test directory. If the file is present, then the "default" feature test should not run, since it won't pass anyway, and the included scenarios be run instead. Global scenarios wouldn't be needed in this case.

I would think option 2 would be the preferred approach, as it shouldn't break any existing implementations and as I understand global scenarios are intended for testing whether a combination of features work together instead of for features which simply use installsAfter. Having a scenarios.json file within a features test directory seems more obvious and natural place.

It's kind of the same for testing a single feature with a "big" scenarios.json. It would be nice to run  those in parallel. I would like to / I do run this with every line changed of the feature, but it easily takes 20+ minutes to go through all the scenarios from scenarios.json.
I have another solution proposal: run all the scenarios tests in parallel, and if/once a test fails, print its output (and abort the others).

As far as I can see this (2.) is possible now.
https://github.com/devcontainers/cli/pull/124


How to render templates into local files without "--- #Source: helm/templates/xxx.yaml" lines

I just use helm to create local yaml files for docker swarm.
$ helm template --output-dir ./app ./app

But there are at least two lines like this:
---
# Source: helm/templates/docker-compose.yaml

How to remove these comment lines? Thanks.

You could try using sed to strip those lines from the output.

source: https://github.com/helm/helm/blob/8ab18f7567cedffdfa5ba4d7f6abfb58efc313f8/pkg/action/action.go#L172-L177

Thank you for the solution:)
I suggest add a command option to skip outputting these lines.


chore: add auto labeler

:tada: This PR is included in version 1.0.49 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket: