Fix for batch size > 1 for reshape/shape layers

@paveltc This is the PR for which you need to run the smoke test

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 6

Complexity increasing per file
==============================
- model_compiler/python/nnir.py  2

Clones added
============
- model_compiler/python/nnir_to_clib.py  1
- model_compiler/python/nnir_update.py  1
- model_compiler/python/nnir_to_openvx.py  1

See the complete overview on Codacy

@LakshmiKumar23 @kiritigowda This PR passes the smoke tests.

@kiritigowda Resolved issues after we spoke yesterday about the * imports

@rrawther Rajy, @paveltc ran the smoke test on this PR


Install from the pip repo

Hello,
I'm currently writing a PR to integrate hloc in nerf studio. To get an easy/clean install, the best would be to install it automatically through the dependencies.
I saw this repo on PyPI : https://pypi.org/project/hloc-mod/
Is it an official/safe repo to use for dependency?
thanks

Hey, adding hloc to nerfstudio is great, thanks a lot! hloc is currently not compatible with PyPI but I will look into this soon. The hloc-mod package is broken and not maintained by me - I'm in the process of requesting to take it down.

Thanks for the answer. I modified my PR for nerfstudio so that hloc an optional dependency that the users can install it manually.
I looked at pixsfm, it seems interesting. I'll try to integrate it too and make a second PR for it.


Track and manage athlete settings changes over time

Please correct me if I'm wrong but I assume that this should include changes in TSS / TRIMP / HRSS calculation so that time-bounded athlete settings (FTP etc.) are reflected, right?

Exactly. Get the right weight, hr, ftp,... for the calculation of an
activity at a given date with the physical settings you had on that date.
On Apr 11, 2018 9:54 PM, "Kuba Wach"<EMAIL_ADDRESS>wrote:
Please correct me if I'm wrong but I assume that this should include
changes in TSS / TRIMP / HRSS calculation so that time-bounded athlete
settings (FTP etc.) are reflected, right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thomaschampagne/stravistix/issues/563#issuecomment-380575780,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJRpYLuPRjISvNlg25gPwf_qmXZe33vks5tnl9_gaJpZM4TDkqr
.

Great, I was just about to file an enhancement request for this. I'd like to volunteer for the task then. I have no experience with Stravistix source code but few years in Angular.js and Angular, so will need some time to get up to speed but should be rather productive after. What do you think?

This would be a very useful feature. I've had significant changes in my FTP over the last few months and if I use the current one it will underestimate the old rides but if I keep a previous FTP it is overestimating my current efforts.

Feature is live: closed by #625


feat: Filter functionality in dashboard section for circuits

Is your feature request related to a problem? Please describe.
If there are lots of projects its hard to find only private or public projects so- a filter button in each section(My circuits, Favorite Circuits, Collaborated Circuits) for projects to filter for specific access types.
Describe the solution you'd like

creating an option to select for access type in each section
Filtering the locals according to the selected filter option.

Are you working on this? (Yes/No)
Yes

Can I also help with working over this issue?

can I work on this issue


Asana board view is broken

MacOs Monterey 12.5
Chrome Version 103.0.5060.134
Extension Version 2.0.16
Relevant integration (if any):
Asana - https://app.asana.com
🐛 Describe the bug
Previously our Extension worked with the Asana Board view, however it appears to have broken. Asana still works on the list view.
Expected behaviour
The Toggl extension button should appear on board view.
Steps to reproduce
Switch into Asana board view, no Toggl button, switch back into List view and it is visible.
Steps to reproduce the behaviour:
As above
Other details or context
Case reference

Case reference

case reference

case reference

case reference

Case reference

case reference

case reference

Case reference

Case reference

Case reference.

Case reference

Case reference

Case reference

Case reference

Case reference

Case reference

Case reference

Case ref

case reference

case reference

case reference

Hey!
The extension is also not working in Calendar view anymore.
I used to use this every day, is there an idea of timeframe that it may be working again?


podman image prune fails on untagged images used by containers

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
podman fails to prune images if it encounters an untagged image used by a container.

Steps to reproduce the issue:

Pull an image: podman pull docker.io/library/ubuntu

Create a container out of it: podman run docker.io/library/ubuntu

Untag the image: podman rmi docker.io/library/ubuntu

Attempt to prune the images: podman image prune

Describe the results you received:
podman returns an error:
Error: failed to prune image: Image used by cf7ab1d066b4eb79b48b46469f3faacc41eb84682806d7305000cbe97658d9fc: image is in use by a container

Describe the results you expected:
Prune to succeed. Docker, for example, successfully avoids cleaning up untagged-but-still-used images.
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
Version:            1.4.5-dev
RemoteAPI Version:  1
Go Version:         go1.12.4
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.4
  podman version: 1.4.5-dev
host:
  BuildahVersion: 1.9.2
  Conmon:
    package: Unknown
    path: /usr/libexec/podman/conmon
    version: 'conmon version 1.0.0-dev, commit: 6f177c68ed989a6cc14ddbf8cec421819bb46401'
  Distribution:
    distribution: ubuntu
    version: "19.04"
  MemFree: 402661376
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    package: 'cri-o-runc: /usr/bin/runc'
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 4
  eventlogger: file
  hostname: eric-ubuntu
  kernel: 5.0.0-21-generic
  os: linux
  rootless: false
  uptime: 8h 27m 41.72s (Approximately 0.33 days)
registries:
  blocked: null
  insecure: null
  search: null
store:
  ConfigFile: /etc/containers/storage.conf
  ContainerStore:
    number: 30
  GraphDriverName: overlay
  GraphOptions: null
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 77
  RunRoot: /var/run/containers/storage
  VolumePath: /var/lib/containers/storage/volumes

Additional environment details (AWS, VirtualBox, physical, etc.):
Physical Ubuntu box.

This looks like a logic error in image.GetPruneImages. podman should ignore any image that has a container while all flag should only control the Names condition. So it would:

Delete unused images with no tag when all=false
Delete all unused images when all=true

Nevermind - looks like in normal flow podman rmi doesn't let you untag an image that has a container. I somehow got my podman storage in a weird state where I had a dangling stopped container.


PT road routes break when roads are altered

This seems to be a common issue:

Jorguś92: When I change the way, the vehicles stop driving it, and the lines of the public transport are interrupted ("the path can not be found"). After removing the TMPE_GlobalConfig file, the situation returns to normal, but when it changes again, you have to repeat the game restart and delete the file

I assume "change the way" = reverse road direction using upgrade tool? Or maybe just upgrading road to different type of road?
I've seen this happen too, on previous TMPE and believe it could possibly be a conflict between TMPE and IPT2 (unconfirmed). Normally I can just drag bits of the transport route (like move a single stop) and it sorts itself out, or wait for vehicles to get stuck on the route and then it sorts itself out.
I've asked for log file and savegame. If I can reproduce it I'll also post log and save.

Log file from Jorguś92:
https://drive.google.com/file/d/17WMZuv1obccHYcUDU-mm8mvSuA8GfHlM/view

Tagging #86 and https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/236

Note: This happens on rails too.
I'll try and provide steps to reproduce over next few days.

Looking back on this now, it's a mod error, likely IPT2 or SuperDemand mod.
As I forgot to add link to the steam discussion I can't contact user so closing this as it's not TM:PE issue.


encoding/xml: fix name spaces

There are many pending issues related to the handling of xml name spaces. We need to rethink this. One more try and then we're going to give up.

#7535 encoding/xml: Encoder duplicates namespace tags
#11496    encoding/xml: Serializing XML with namespace prefix
#9775 encoding/xml: Unmarshal does not properly handle NCName in XML namespaces
#12624    encoding/xml: brittle support for matching a namespace by identifier or url
#8167 encoding/xml: disallow attributes named xmlns:*
#11735    encoding/xml: empty namespace conventions are badly documented
#8068 encoding/xml: empty namespace prefix definitions should be illegal
#8535 encoding/xml: failure to handle conflicting tags in different namespaces
#11431    encoding/xml: loss of xmlns= in encoding since Go 1.4
#7113 encoding/xml: missing nested namespace not handled
#11724    encoding/xml: namespaced and non-namespaced attributes conflict
#7233 encoding/xml: omit parent tags if value is empty #2
#12406    encoding/xml: support QName values / expose namespace bindings
#9519 encoding/xml: support for XML namespace prefixes

then we're going to give up

Please don't give up :smile:

Seconding the comment above. Please don't give up on this. There is still value in handling namespaces properly in golang XML.

I did say we'd try one more time before giving up.

+1 for not giving up

+1 I was researching Go as a possible replacement for Python in processing ODT files and I was quickly hitting snags when dealing with multiple namespaces.

I feel your pain, I'm also stuck with SOAP-based web services and the namespace issue is a thorn here.  Going to generate XML w/ templates for now.  Please do not give up :)

+1

In general we don't pay attention to comments that only say "+1". In fact,
I think we have a bot that deletes them.
What would be very useful instead would be pointers to programs (ideally
self-contained ones on play.golang.org) illustrating what you are trying to
do with encoding/xml, what output you wish you were getting, and how the
current output differs/breaks your use case. That information would help
guide any eventual fixes we make here.
Thanks.

Point taken. I do not have code right now (because I decided to generate my SOAP requests throught text/template for the moment) but thet fact is, xml.Marshal generate tags w/o any namespace. I tried to use https://github.com/gabstv/go-soap (which use xml.Marshal) and even though there are "xmlns:" declaration for many tags, none of them are in the resulting XML file.  Many of the open tickets above are in the same situation.
xml.Unmarshal() is fine.

An example I'd like to see work properly without a lot of hacks (I normally pretend the namespace is part of the localname, which works well enough with XMPP because it's such a limited profile of XML, but won't work with more complex documents) is something like the following valid XMPP stream:
const xmlStream = `
<?xml version='1.0'?>
<stream:stream
    from='im.example.com'
<EMAIL_ADDRESS>    version='1.0'
    xml:lang='en'
    xmlns='jabber:client'
    xmlns:stream='http://etherx.jabber.org/streams'>
<stream:error>
  <see-other-host xmlns="urn:ietf:params:xml:ns:xmpp-streams">im.example.net</see-other-host>
  </stream:error>
</stream>

which, when decoded and re-encoded will result in something like:
<stream xmlns="stream"<EMAIL_ADDRESS>to="im.example.com" version="1.0" xmlns:_xml="xml" _xml:lang="en" xmlns="jabber:client" xmlns:_xmlns="xmlns" _xmlns:stream="http://etherx.jabber.org/streams">
…

Example here:  https://play.golang.org/p/XyMFO301HA
Note that the example reads a raw token because this is XMPP and it has to be streamed which the current XML implementation isn't very good at; that's a separate issue though.

An example I'd like to see work properly without a lot of hacks (I normally pretend the namespace is part of the localname, which works well enough with XMPP because it's such a limited profile of XML, but won't work with more complex documents) is something like the following valid XMPP stream:
const xmlStream = `
<?xml version='1.0'?>
<stream:stream
    from='im.example.com'
<EMAIL_ADDRESS>    version='1.0'
    xml:lang='en'
    xmlns='jabber:client'
    xmlns:stream='http://etherx.jabber.org/streams'>
<stream:error>
  <see-other-host xmlns="urn:ietf:params:xml:ns:xmpp-streams">im.example.net</see-other-host>
  </stream:error>
</stream>

which, when decoded and re-encoded will result in something like:
<stream xmlns="stream"<EMAIL_ADDRESS>to="im.example.com" version="1.0" xmlns:_xml="xml" _xml:lang="en" xmlns="jabber:client" xmlns:_xmlns="xmlns" _xmlns:stream="http://etherx.jabber.org/streams">
…

Example here:  https://play.golang.org/p/XyMFO301HA
Note that the example reads a raw token because this is XMPP and it has to be streamed which the current XML implementation isn't very good at; that's a separate issue though.

The issue in the above comment is effectively issue #7535

See #14407 .

Has there been any work done on this front?  I have a big need to work with a bunch of clunky SOAP API's, and these namespaces are just killing me everywhere with it.  Are there any libraries that work around this that I could make use of in the interim?

No, there has not.

Seems there hasn't been any progress on this. I know it's a dirty workaround, but I ended up just generating a string constant that I just replace with the proper namespaces. Probably just a tad safer than what someone in a duplicate issue did to workaround using text/template
https://play.golang.org/p/MlJEz9PSSO

A short summary of information submitted for the individual issues. Generally speaking, the namespace standard (https://www.w3.org/TR/xml-names) was not correctly enforced. Only #7113 fix is not yet available. No work on extending functionality has been done.
• #7535 Fix submitted
• #11496 Commented (invalid syntax)
• #9775 Commented (invalid syntax)
• #12624 Commented (invalid syntax)
• #8167 Improvement request to compact output
• #11735 Related to documentation. XMLName usage must be understood by reading Marshal/Unmarshal documentation jointly.
• #8068 Fix submitted
• #8535 Fix submitted
• #11431 No other fix required. Tests have been submitted. (duplicates #8535)
• #7113 No fix available yet.
• #11724 Fix submitted
• #7233 Commented
• #12406 Adding func - no work so far
• #9519 Improvement request to compact output. (duplicates #8167)

#7113 has also a proposed fix. All key issues related to the enforcement of namespace standard are solved.
This fix leaves active the handling of the nesting of the tags which would be a basis for other related improvements.

All above submitted fixes have minor incompatibilities when merged related to the handling of prefixes. A merged version has been submitted as one. The commit message contains the details previously submitted. Namespace rules are completely enforced. Declaring a namespace is done using xmlns="value w/o prefix" or xmlns:prefix="value with prefix".
The implementation is referring not to eventual prefixes but to their values. To improve XML compatibility, it was using a workaround for this behavior by adding attributes names (using _xmlns) to create chains and consequently almost valid XML. This whole business is now inactive for namespaces as it is not documented. Valid XML is now always returned but sometimes contains oddities as previously.
There are several ways to create namespaces and attributes and various possible conflicts. Further Marshal and Unmarshal share codes so exceptions arise from both types of operations. Idempotency (marshal/unmarshal in cycles w/o changes) is the objective for regular XML and has been mentioned several times in the issues. It is implicit to the documentation. It is not achieved as, for instance, prefix are not restored in tags (cf. TestIssue7535).
The XMLName of a struct must be unique. It designates either the override of the struct name together with an optional non-prefixed namespace. When unmarshaling, the value (the namespace URI usually) is stored into the XMLName of the struct. Without an XMLName field, it is lost.
When marshaling, it can only be an attribute as no prefix is available. A decoder has a strict property which can be false and allows to produce invalid XML. Unmarshal is always strict as it uses Token. Unmarshal can create a prefix only with an attribute.
A Token contains a token.Name.Value used as the value of the namespace when there is one which means that the prefix used is never available if it is not an attribute.

For everyone watching these XML issues in Go and hoping for a fix ASAP, I finally took the plunge and wrote an API-compatible binding to libxml2, which serves great as a temporary workaround for these namespace issues.
https://github.com/alexrsagen/go-libxml - I hope this helps someone!

Any ideas how to fix mix attributes with namespace and without?
<entity id="id" a:id="id_a" b:id="id_b"/>

E.g., this one will work fine:
	type Entity struct {
		IDA    		string   `xml:"a id,attr,omitempty"`
		IDB    		string   `xml:"b id,attr,omitempty"`
	}

but this one is conflicting:
	type Entity struct {
		ID    			string   `xml:"id,attr,omitempty"`
		IDA    		string   `xml:"a id,attr,omitempty"`
		IDB    		string   `xml:"b id,attr,omitempty"`
	}

@gunnsth did you figure out a more elegant way to handle the issue.

i just wanted add empty namespace, used quick fix Xmlns string `xml:"xmlns,attr"  but not solving other problems example, int, string types.
tried xml:"* categoryId" turns out ->    <categoryId xmlns="*">1001770</categoryId>
yet soap api not accepting * character wants <categoryId xmlns="">1001770</categoryId>
type Authentication struct {
	Xmlns string    `xml:"xmlns,attr"  json:"-"`
	AppKey string `xml:"appKey,omitempty" json:"appKey,omitempty"`

	AppSecret string `xml:"appSecret,omitempty" json:"appSecret,omitempty"`
}

type GetSubCategoriesRequest struct {
	XMLName xml.Name `xml:"http://www.n11.com/ws/schemas GetSubCategoriesRequest"`

	Auth *Authentication `xml:"auth,omitempty" json:"auth,omitempty"`

	CategoryId int64 `xml:"* categoryId,omitempty" json:"categoryId,omitempty"` <<<<<<<-------- i need xmlns=""
}

any help?

@ydnar Can you make a PR?

@ydnar Can you make a PR?

I did, 3 years ago: https://github.com/golang/go/pull/48641

Hi any plan to validate and merge the associated PR to fix namespaces soon ?

Unfortunately we don't have a plan for how to do that without breaking existing working programs.

Hi !
Thank you for this reply.
As we don't want to break existing programs, how about introducing encoding/xml/v2 but first go through x/encoding/xml/v2 to ensure no additional fixes or missing features are needed ?

@ydnar Can you make a PR?

I did, 3 years ago: https://github.com/golang/go/pull/48641

Hi @ianlancetaylor, what would be missing from this pull request for a first step ?
It seems to reference and fix all issues linked to this issue ?

Experience tells us that any change to encoding/xml that affects existing programs causes some of them to break.
This change (https://go.dev/cl/355353) may be the exception but I don't see why.

For encoding/xml/v2, the features I want to see are:

Never accept or produce ill-formed XML.
Never accept or produce namespace-ill-formed XML unless namespace processing has been explicitly disabled.
Reject DTDs by default, as they are a legacy feature with very large attack surface.
Correctly parse and serialize all well-formed XML that does not include a DTD.

There probably needs to be a community lead project that could work on encoding/xml/v2 for a few years to support what is generally needed and to find a new API that could be incorporated in the standard library. IMO nothing is ready to be included in the std lib (or even /x/ repos) right now.
There have been several efforts led by individuals to tackle specific problems with the current stdlib implementation. These should be combined into one project so additional improvements can be included in one module. Not to mention many commonly requested features when using XML in applications (XPath, digital signatures, XSDs, etc)

Many legacy systems use XML only.
Denying Golang stable compatibility with this format is an obstacle to its deployment in these companies.
As mentioned above, XML hardly ever evolves. Once compatibility is up to date, maintenance will be light.
It's not just a maintenance choice, it's also a strategic one.

workaround:
https://github.com/nbio/xml


Reference resolution on ingest for faster chained searches

Description
We currently defer the resolution of resource references to search time. This PR changes that to perform and maintain it on ingestion. We add a ResolvedResourceSurrogateId to the ReferenceSearchParam table and indexes so that the field can be read and updated efficiently.
The result is a improvement in performance for chained searches. This PR does not address _include performance.
The plan was to use an indexed view but performance hit was a lot higher than maintaining this column ourselves.
Semver Change (docs)
Patch

We are going to close this for now based on priorities but will leave the code for reference for when we have time to revisit.


BlobLeaseClient is missing a GetUrl getter

Is your feature request related to a problem? Please describe.
I need to know the URL of the BlobLeaseClient to produce good error message from my utility functions.
For convenience, a direct link to the header: https://github.com/Azure/azure-sdk-for-cpp/blob/main/sdk/storage/azure-storage-blobs/inc/azure/storage/blobs/blob_lease_client.hpp
Describe the solution you'd like
BlobLeaseClient should have a GetUrl getter.
blob_lease_client.hpp:
  std::string BlobLeaseClient::GetUrl() const
  {
    if (m_blobClient.HasValue())
    {
      return m_blobClient->GetUrl();
    }
    else if (m_blobContainerClient.HasValue())
    {
      return m_blobContainerClient->GetUrl();
    }
    else
    {
      AZURE_UNREACHABLE_CODE();
    }
  }

blob_lease_client.cpp:
    /**
     * @brief Get's the primary URL endpoint of the blob or container.
     *
     * @return A URL of a blob or container.
     */
    std::string GetUrl() const;

Describe alternatives you've considered
Passing a string as parameter everywhere in my code.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

Lease client is a convenient method for customers to do lease operations, unlike BlobClient, BlobContainerClient or BlobServiceClient, lease client doesn't have a corresponding real resource at server side. I think maybe we shouldn't have GetUrl for it.
@LarryOsterman What do you think?

The C++ Guidelines expect that your service clients should have a GetUrl method.
I'm not sure the utility of such a method, the guideline was added long before I was on the team. But the guideline exists.

At the moment I can't generate a meaningful string through the the public interface of BlobLeaseClient because there isn't a single public member function that can tell me anything about the resource the BlobLeaseClient targets.
GetUrl might be a bad name for it, but I believe there should be a way to ask for a string that identifies the resource receiving the lease operations that BlobLeaseClient can trigger.

These are the private data members of BlobLeaseClient
  private:
    Azure::Nullable<BlobClient> m_blobClient;
    Azure::Nullable<BlobContainerClient> m_blobContainerClient;
    std::mutex m_mutex;
    std::string m_leaseId;

It can be instantiated by either BlobClient or BlobContainerClient because blobs and containers can be leased. Once you give a client to the BlobLeaseClient you can never recover information about them anymore, you would have to keep a copy of the client and not loose track of the client/lease-client pairing.

@Jinming-Hu We should expose at a minimum the LeaseId.  I'm mixed on whether it makes sense to expose the URL of the targeted object.  Other language SDKs do expose both of these values.
see: https://learn.microsoft.com/en-us/dotnet/api/azure.storage.blobs.specialized.blobleaseclient?view=azure-dotnet

@RickWinter there is GetLeaseId already. It's only the target of the lease that is missing.

I'll leave this decision to @LarryOsterman

3 of the 4 P0 languages (.Net, Java, JavaScript) have a Uri property, Python does not. Go doesn't appear to have a BlobLeaseClient object.
I don't feel strongly about this, to be honest, but my take is that we should follow the pattern of the majority of the P0 SDK clients here.

Hi @microzchang, can you add GetUrl method for all lease clients?

Will add it.


Databricks deployment to custom vnet

Feature Request
Databricks on Azure supports deployment to custom virtual network. This is not yet supported on Azure sdk for go. Details and ARM templates for this can be found on these links:
https://docs.azuredatabricks.net/administration-guide/cloud-configurations/azure/vnet-inject.html
https://github.com/Azure/azure-quickstart-templates/tree/master/101-databricks-workspace-with-vnet-injection/

@botang-databricks do you know if this is planned to be added to the swagger for databricks?

Staled issue, closing


Add Getting started section to readme

README Update
Please add a getting started section to the README so blah

Added Getting started section to README.md


Using multiple instances on the same page?

Question
How do I use more than one instance of the Code Editor on the same page?
Additional context
I'm using your Code plugin to create snippets of sample code which are rendered in my front end template. If possible, I'd like to avoid adding additional code highlighters like Shiki, highlighter.js, etc and instead use craft-code-editor. I can output a single field using the example code provided:
<textarea id="myCode">{{ entry.codeEditor.value | raw }}</textarea>
{{ codeEditor.includeJs("myCode", "Frontend Editor", {
    language: entry.codeEditor.language,
    theme: "auto",
    "domReadOnly": true,
    "readOnly": true
}) }}

This works for the first instance – I'm seeing a VisualCode looking field on the front-end. But, only the first field renders properly. I assume this is because there's an ID conflict? So, I added a loop index like this:
<textarea id="myCode_{{ loopIndex }}">{{ entry.codeEditor.value | raw }}</textarea>
{{ codeEditor.includeJs("myCode_{{ loopIndex }}", "Frontend Editor", {
    language: entry.codeEditor.language,
    theme: "auto",
    "domReadOnly": true,
    "readOnly": true
}) }}

Which produces a unique ID for each textarea:
<textarea id="myCode_1">...</textarea>
<textarea id="myCode_2">...</textarea>

However, none of the fields render properly. I'm just seeing un-styled textarea fields.

This is an error in your Twig code. This code:
<textarea id="myCode_{{ loopIndex }}">{{ entry.codeEditor.value | raw }}</textarea>
{{ codeEditor.includeJs("myCode_{{ loopIndex }}", "Frontend Editor", {
    language: entry.codeEditor.language,
    theme: "auto",
    "domReadOnly": true,
    "readOnly": true
}) }}

Will render the first parameter to includeJs() to be myCode_{{ loopIndex }} because you're already in Twig code, so the {{ }} will just output those literal characters. You want something like this instead: myCode_#{loopIndex} that uses Twig string interpolation: https://twig.symfony.com/doc/3.x/templates.html#string-interpolation


Icon Request: test/spec JS

I'm using Visual Studio Code Version [1.6.1] [Stable]
I'm using the extension version [3.0.6]
[x] I've regarded to the FAQ and they did not help.
My system is: [Windows] [x64]
[x] I'm sure this issue is not a duplicate?
[x] I want to create an icon request:

Type: extension
Icon Name: Test/Spec
Extensions: .test.js, .spec.js
[x] I crossed out "Steps to reproduce", "Expected behaviour" and "Actual behaviour".

Description:
I keep test files ending with .spec.js in the same folder as the source files; having an icon for test files, in addition to folders, would be very helpful.
I can't think of an icon to use for this, I trust you can come up with a good one.

YES, this is the latest fashion! We have the tools now to avoid having to partition our code by type. Much better to partition by domain and have all the files related to a particular domain in one place.
I follow the same convention .test.js or .spec.js and I know that 100s if not 1000s of js developers are shifting to use this convention. Please add support for this :)
Thanks!

Or perhaps this is already possible? If so, please add documentation for how to achieve this - how to add custom mappings to icons using this plugin. Thanks :)

I made this PR: https://github.com/robertohuertasm/vscode-icons/pull/409

I hope to see these icons available by the end of this weekend :)
The PR is fine, just having problem placing the icons correctly, ie. centered vertically, one pixel down.


Background house-keeping & pre-existing thread

In our use-case of Cloudflare Workers, we already have a GC thread that's responsible
for house-keeping activities. To minimize having a different thread, because
ReleasePerCpuMemoryToOS is an internal API, & to be protected against future book
keeping improvements, I've split up the loop into an init & tick phase so that it can be
integrated into an existing book keeping thread (ours has a 0.2 Hz tick rate).
I'm not 100% sure about what to do about the marking of the thread as idle. For now I made
it the callers responsibility to determine if the thread is actually idle but the rules feel squishy
(e.g. should it basically be invoked before every invocation of the Tick method?)

@googlebot I signed it!


How to quantify mobilenet model in per_tensor mode?

I try to quantify the mobilenet in per_tensor mode by using the following configuration. I got the result model which just puts Relu activation into Conv. So if there is any way I can quantify the model in per_tensor mode?

python version: 3.9.12
neural-compressor version: 1.14.1
system version: Windows 11

#
# Copyright (c) 2021 Intel Corporation
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#   http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

version: 1.0

model:                                               # mandatory. used to specify model specific information.
  name: mobilenetv2
  framework: onnxrt_qlinearops                       # mandatory. supported values are tensorflow, pytorch, pytorch_ipex, onnxrt_integer, onnxrt_qlinear or mxnet; allow new framework backend extension.

quantization:                                        # optional. tuning constraints on model-wise for advance user to reduce tuning space.
  approach: post_training_static_quant               # optional. default value is post_training_static_quant.
  calibration:
    dataloader:
      batch_size: 1
      dataset:
        ImagenetRaw:
          data_path: /path/to/imagenet/val
          image_list: /path/to/imagenet/val.txt      # download from http://dl.caffe.berkeleyvision.org/caffe_ilsvrc12.tar.gz
      transform:
        Rescale: {}
        Resize:
          size: 256
        CenterCrop:
          size: 224
        Normalize:
          mean: [0.485, 0.456, 0.406]
          std: [0.229, 0.224, 0.225]
        Transpose:
          perm: [2, 0, 1]
        Cast:
          dtype: float32
  model_wise:                                        # optional. tuning constraints on model-wise for advance user to reduce tuning space.
    weight:
      granularity: per_tensor
      scheme: asym
      dtype: int8
      algorithm: minmax
    activation:
      granularity: per_tensor
      scheme: asym
      dtype: int8
      algorithm: minmax

evaluation:                                          # optional. required if user doesn't provide eval_func in lpot.Quantization.
  accuracy:                                          # optional. required if user doesn't provide eval_func in lpot.Quantization.
    metric:
      topk: 1                                        # built-in metrics are topk, map, f1, allow user to register new metric.
    dataloader:
      batch_size: 1
      dataset:
        ImagenetRaw:
          data_path: /path/to/imagenet/val
          image_list: /path/to/imagenet/val.txt      # download from http://dl.caffe.berkeleyvision.org/caffe_ilsvrc12.tar.gz
       transform:
        Rescale: {}
        Resize:
          size: 256
        CenterCrop:
          size: 224
        Normalize:
          mean: [0.485, 0.456, 0.406]
          std: [0.229, 0.224, 0.225]
        Transpose:
          perm: [2, 0, 1]
        Cast:
          dtype: float32
  performance:                                       # optional. used to benchmark performance of passing model.
    warmup: 10
    iteration: 1000
    configs:
      cores_per_instance: 4
      num_of_instance: 1
    dataloader:
      batch_size: 1
      dataset:
        ImagenetRaw:
          data_path: /path/to/imagenet/val
          image_list: /path/to/imagenet/val.txt      # download from http://dl.caffe.berkeleyvision.org/caffe_ilsvrc12.tar.gz
      transform:
        Rescale: {}
        Resize:
          size: 256
        CenterCrop:
          size: 224
        Normalize:
          mean: [0.485, 0.456, 0.406]
          std: [0.229, 0.224, 0.225]
        Transpose:
          perm: [2, 0, 1]
        Cast:
          dtype: float32

tuning:
  accuracy_criterion:
    relative:  0.02                                  # optional. default value is relative, other value is absolute. this example allows relative accuracy loss: 1%.
  exit_policy:
    timeout: 0                                       # optional. tuning timeout (seconds). default value is 0 which means early stop. combine with max_trials field to decide when to exit.
  random_seed: 9527                                  # optional. random seed for deterministic tuning.

Hi, according to your INT8 model, it is quantized in per-tensor mode actually. Please check the channel of scale and zero point of Conv weight, they are scalars rather than vectors.

Sorry, I uploaded a wrong model. This model is the result model: https://drive.google.com/file/d/1JcYu436evBvJwZJDpbkBeyaCIjKM8jjm/view?usp=share_link

Hi, we can get int8 model with 1.14.1 version on local machine. Can you share your log during quantization?

Thanks! I think the log was automatically deleted. By the way, when I tried to quantify the mobilenet, it took several hours. If you could get a  quantized model in a short time, I think maybe some of my configuration are wrong? I will retry to generate a log.

The time cost depends on the hardware and the number of data which is used to evaluate accuracy. It will do evaluation at least twice to get FP32 and INT8 accuracy, you can check if most time is spent on evaluation.

@mengniwang95 Thanks for your suggestion. I still can't quantify the mobilenet with high accuracy. But onnxruntime did solve my problem.


AddFinalizer clears Status subobject leading to delayed object readiness

What happened?
We create managed resources using only the Observe management policy and the provider will initially set the Synced condition to True but will not set the Ready condition at all.
We can see in the provider logs that the object seems to be successfully reconciled and is requeued according to the configured sync interval.
Once the sync interval is hit, the resource becomes ready.
How can we reproduce it?

Create any MR in the cluster with only the Observe management policy. We can easily reproduce this with provider-upjet-azure in any resource, for instance VirtualNetwork with the following spec:

apiVersion: network.azure.upbound.io/v1beta2
kind: VirtualNetwork
metadata:
  annotations:
    crossplane.io/external-name: vnet-test-crossplane
  name: vnet-test-crossplane
spec:
  deletionPolicy: Delete
  forProvider:
    resourceGroupName: rg-test-crossplane
  initProvider: {}
  managementPolicies:
  - Observe
  providerConfigRef:
    name: default

You will see the object become Synced almost immediately, but the Ready condition will stay nil. Also, the whole atProvider struct stays nil. Provider logs will show the following:

2024-12-06T20:57:33Z    DEBUG    provider-azure    Reconciling    {"controller": "managed/network.azure.upbound.io/v1beta1, kind=virtualnetwork", "request": {"name":"vnet-test-crossplane"}}
2024-12-06T20:57:33Z    DEBUG    provider-azure    Connecting to the service provider    {"uid": "bb757cb5-8934-471d-981b-7862513ac9b1", "name": "vnet-test-crossplane", "gvk": "network.azure.upbound.io/v1beta1, Kind=VirtualNetwork"}
2024-12-06T20:57:33Z    DEBUG    provider-azure    Instance state not found in cache, reconstructing...    {"uid": "bb757cb5-8934-471d-981b-7862513ac9b1", "name": "vnet-test-crossplane", "gvk": "network.azure.upbound.io/v1beta1, Kind=VirtualNetwork"}
2024-12-06T20:57:33Z    DEBUG    provider-azure    Observing the external resource    {"uid": "bb757cb5-8934-471d-981b-7862513ac9b1", "name": "vnet-test-crossplane", "gvk": "network.azure.upbound.io/v1beta1, Kind=VirtualNetwork"}
2024-12-06T20:57:34Z    DEBUG    provider-azure    Diff detected    {"uid": "bb757cb5-8934-471d-981b-7862513ac9b1", "name": "vnet-test-crossplane", "gvk": "network.azure.upbound.io/v1beta1, Kind=VirtualNetwork", "instanceDiff": "*terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"location\":*terraform.ResourceAttrDiff{Old:\"canadacentral\", New:\"\", NewComputed:false, NewRemoved:true, NewExtra:interface {}(nil), RequiresNew:true, Sensitive:false, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, RawConfig:cty.NilVal, RawState:cty.NilVal, RawPlan:cty.NilVal, Meta:map[string]interface {}(nil)}"}
2024-12-06T20:57:34Z    DEBUG    provider-azure    Skipping update due to managementPolicies. Reconciliation succeeded    {"controller": "managed/network.azure.upbound.io/v1beta1, kind=virtualnetwork", "request": {"name":"vnet-test-crossplane"}, "uid": "bb757cb5-8934-471d-981b-7862513ac9b1", "version": "1176637016", "external-name": "vnet-test-crossplane", "requeue-after": "2024-12-07T02:49:57Z"}

You can either wait for the requeue to trigger or you can restart the provider.
After that, the Ready condition will be set to true.

What environment did it happen in?
Crossplane version: 1.18.0
Additional information
After step-by-step debugging, we have found that the bug seems to come from the AddFinalizer function in the APIFinalizer struct. This struct is used by provider-upjet-azure and probably all upjet based providers (although I didn't validate).
The AddFinalizer function calls the kube client's Update function which will apply an object in the kube API, but also return the updated object returned by the API to the caller. This, in turn, nullifies any struct that is not applied by the Update function such as the Status sub-struct.
Here are the relevant parts of the code:

AddFinalizer function

This ends up calling Into from the kube client

Reconciler's call to AddFinalizer

I see 2 ways of fixing this:

In reconciler.go, we can save the status sub-struct and set it back after the finalizer call
In the AddFinalizer function, we can do the same

The benefit of 1 is that it applies to everyone, no matter the finalizer implementation they used while 2 only fixes that specific implementation.
We should probably also see if there are other places where this could happen.

This, in turn, nullifies any struct that is not applied by the Update function such as the Status sub-struct.

I remember that the quoted behavior requires providers to reconcile multiple times in order to apply a set of changes. The comment in the late initialization logic acknowledges the behavior:
https://github.com/crossplane/crossplane-runtime/blob/19d95a69cc03690c4b867ff91d89681fcf872a93/pkg/reconciler/managed/reconciler.go#L1276-L1290

You're right, that comment explains exactly the behaviour I'm seeing for the update call. However, it says that the update will implicitly requeue an immediate reconcile, but that is not what I'm seeing here, at least when the object is in Observe-only mode and when the update is a call to add the finalizer.
Either a behaviour changed in Kube client/kube controller runtime or something with Observe-only codepath makes it behave differently.
Having step-by-step debugged it, I can tell for sure that there was only 1 reconcile and the object was simply requeued for  time which in my case is 6 hours. The object stayed at Synced = true and Ready = nil with a nil atProvider struct. The only ways to get the object ready are to force a reconcile somehow, either by restarting the provider or by making a dummy modification on the object (adding an annotation or whatever). You can also wait , but that is not an acceptable solution in my case.
I think that for efficiency it would be better to handle these cases in a single reconcile to reduce load on providers pods, provider's external API and kubernetes API. On thousands of objects, this can make a significant impact. Also, assuming the requeue was actually done immediately, it would still potentially be behind hundreds of other objects in the queue. Again, for the observe-only use-case where a single reconcile would be enough, I feel like we can optimize it.
Are you against saving the Status sub-struct and restoring it after the update call? If so, what alternative do you suggest?

I would imagine that the comment assumes the typical scenario of full control by Crossplane which includes a subsequent Create call which will trigger the update, and in the Observe case there is no subsequent operator to trigger the status update.

Are you against saving the Status sub-struct and restoring it after the update call? If so, what alternative do you suggest?

I should have clarified that I don't have any opinions on the matter — I don't know enough about the managed reconciler design. I just wanted to provide reference for the described behavior.

I understand. Either way, this is a real bug and I want to propose a solution. Is there any maintainer that could help push this forward by suggesting an alternative or approving my suggestion so I can implement and propose a PR?


Hello world in Crystal, using type annotations

What does the program do ?
Hello, world! With type annotations
In what programming language it is written?
Crystal

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


notification tap event

It's impossible right now to detect notification click/tap event.

Hi @cainrus,
For handling the click event for the notification, to open a custom activity or window, or for tracking purposes, this will require you to change the implementation of the native push notification BroadcastReceiver.
Modify the PushReceiver.java file in the following path:
plugins/pushy-cordova/receiver/src/android/PushReceiver.java

Note: Do not accidentally modify plugins/pushy-cordova-receiver/src/android/PushReceiver.java or platforms/android/src/me/pushy/sdk/cordova/PushReceiver.java as your modifications will be lost when you reinstall the plugin.

Specifically, modify the following lines:
https://github.com/pushy-me/pushy-cordova/blob/54af4ccd7e961d3402914505d1b8f8cc1c333026/receiver/src/android/PushReceiver.java#L86-L92
You would need to add Extras to the Intent to track notification click:
launchIntent.putExtra("notificationClicked", true);

When you are done modifying the PushReceiver.java implementation, reinstall the plugin locally for changes to take effect by running the following commands in your root project directory:
cordova plugin rm pushy-cordova-receiver
cordova plugin add ./plugins/pushy-cordova/receiver/

Then, to fetch the notificationClicked extra from Cordova, you need to use another plugin like cordova-plugin-intent:
https://github.com/napolitano/cordova-plugin-intent
  //To get the intent and extras when the app it's open for the first time
    window.plugins.intent.getCordovaIntent (function (intent) {
        intenthandler(intent);
    });

    //To get the intent and extras if the app is running in the background
    window.plugins.intent.setNewIntentHandler (function (intent) {
        intenthandler(intent);
    });

    //To handle the params
    var intenthandler = function (intent) {
          if (intent && intent.extras && intent.extras.notificationClicked) {
        //Do something
          }
    };

HI @cainrus,
Great news! We have just finished integrating this functionality into the SDK.
Call the Pushy.setNotificationClickListener((data) => {}) method from your application to listen for when the user clicks your notifications:
// Listen for push notifications clicked
Pushy.setNotificationClickListener(function (data) {
    // Print notification payload data
    console.log('Clicked notification: ' + JSON.stringify(data));

    // Display an alert with the "message" payload value
    alert('Clicked notification: ' + data.message);

    // Navigate the user to another page or
    // execute other logic on notification click
});

This method is supported on both Android and iOS. Inside this method, you can then check the notification payload which was clicked and direct the user to the relevant page in your Cordova app or execute other custom logic on notification click.
To be able to access this method, please update the Pushy Cordova SDK in your app by running the following commands in the root of your Cordova project (these will delete any modifications you have made to pushy-push-receiver):
cordova plugin remove pushy-cordova-receiver
cordova plugin remove pushy-cordova

cordova plugin add<EMAIL_ADDRESS>cordova plugin add ./plugins/pushy-cordova/receiver/

Please let me know if you face any issues.


Rethink the design of private and public apis

Description
Rethink the design of what appears to be redundant in the implementation of private and public apis

It may be better to cut out the API for admin to a separate service instead of private.
Since the API for admin is assumed to be used only internally, it seems easy to separate it into a separate service as an internal API.


Multiple inputs per pin currently clear other inputs when you select some of the options at the bottom of the list.

Building from main and doing multiple inputs per pin will currently erase other mappings for that pin if you choose an option further down in the list like turbo or any of the extra buttons.  Once one of those buttons is selected, selecting an additional normal input like up will cause the line to clear as well.

Closing - this is intended with hotkeys that cannot be combined.


Files suddenly not opening at all

Operating system or device - Godot version:
Windows 10
Godot 3.0 pre-alpha
Issue description:
For whatever reason, when I fast-forwarded to master in my fork, files no longer opened properly. Every call to a File's .open method returns zero now. I've already tried it with a variety of file types and file modes.
Steps to reproduce:
var file = File.new()
print(file.open("user://save_game.dat", file.WRITE))
Output:
0

Could be totally unrelated but there were a few changes to the projects file format so your settings may not be listened to. If that results in it looking in the wrong location to open your files...
If that is the case, edit project settings to make sure the correct entries are added and then edit the project.godot file to clean up any old settings that are now unused. If not, then I'm barking up the wrong tree :)

AFAIK File.open() always returned 0 on success (it is equivalent to the OK constant).

Oh wow...that's right. I just completely forgot that the if statements should be comparing against OK instead of if file.open(...):. Nevermind me. I just had a massive brainfart all evening.


Ranges and more

LGTM but let's maybe split this in 2 PRs if possible:

cppcheck pinning and cppcheck related fixes (move these to #4625?)
ranges and stuff


Test flakiness - longer time wait

What this PR does / why we need it:
The test fails sometimes on this wait time
https://search.ci.kubevirt.io/?search=expected+phase+is+'WaitForFirstConsumer'+but+got+''&maxAge=336h&context=1&type=bug%2Bjunit&name=&excludeName=&maxMatches=5&maxBytes=20971520&groupBy=job
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test pull-kubevirt-e2e-k8s-1.21-sig-compute

/lgtm
@dhiller Would you have time for this small PR? Thank you in advance!

/lgtm

/retest-required

/test pull-kubevirt-e2e-kind-1.23-vgpu-nonroot


[Improve] e2e findElement imporvement

What changes were proposed in this pull request
Issue Number: close #xxx

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

This test case fails


Minor cleanup for the unnecessary tf.summary.record_if(True) (no_op)

There are more cleanups to come. Also I think there are some issue for the current TB callbacks (probably introduced when we port them from tf_keras)

https://github.com/keras-team/keras/issues/19121

Codecov Report
Attention: 279 lines in your changes are missing coverage. Please review.

Comparison is base (c8700f4) 80.14% compared to head (c9073ff) 53.83%.
Report is 22 commits behind head on master.

Files
Patch %
Lines

keras/backend/tensorflow/linalg.py
18.68%
74 Missing :warning:

keras/backend/jax/trainer.py
0.00%
41 Missing :warning:

keras/ops/linalg.py
89.17%
22 Missing and 7 partials :warning:

keras/callbacks/tensorboard.py
0.00%
27 Missing :warning:

keras/backend/jax/linalg.py
42.85%
24 Missing :warning:

keras/backend/torch/linalg.py
42.10%
22 Missing :warning:

keras/backend/tensorflow/numpy.py
0.00%
13 Missing :warning:

keras/backend/jax/core.py
0.00%
8 Missing :warning:

keras/backend/jax/numpy.py
0.00%
7 Missing :warning:

keras/backend/tensorflow/core.py
0.00%
7 Missing :warning:

... and 7 more

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19170       +/-   ##
===========================================
- Coverage   80.14%   53.83%   -26.31%
===========================================
  Files         341      362       +21
  Lines       36163    38871     +2708
  Branches     7116     7542      +426
===========================================
- Hits        28982    20928     -8054
- Misses       5578    16440    +10862
+ Partials     1603     1503      -100

Flag
Coverage Δ

keras
53.83% <58.97%> (-26.16%)
:arrow_down:

keras-jax
?

keras-numpy
53.83% <58.97%> (-3.25%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Describe the "procedures" section

this includes defining Procedure / Phase / Block etc.
this would be the most difficult part!

mostly done with this commit


Add usage instructions to README

While running the train_model.py file, I came across errors where it couldn't find files such as data/embeddings/model.txt etc. I was able to add the conllu dataset file but how do I get the rest of them?

Nevermind I missed reading the second file of the download_data script.


Test item counter

In this milestone/branch, I used Jest to test the item counter successfully.
I also refactored the code to modular JS.

great work man keep going


Separate Actions into several basis actions

Is your feature request related to a problem? Please describe.
All actions should be composed of several atomic actions so that user can combine several actions into single set of actions.
This will also allows developer to make a new actions easily.
Describe the solution you'd like
Make one single action into several atomic base actions.

Possible base action candidates.
Resource Preparations (This might be out of scope in the actions)

assign() ... assign single qubit to a variable
validate() ... resource validation in GOD layer.

Quantum Operations

gate() ...apply quantum gate to a qubit
measure() ... do measurement in some basis

Messaging

send() ... send message to proper partner

Post processing

free() ... free trash qubits
name() ... provide a new name to the resource
update() ... update some properties if it needs

Others?


Add Autocomplete for Cities [#6]

Context
Adds autocomplete for city names. We decided against using the Google Maps API (required billing), instead going with a city list @yyjlincoln found and doing the matching client side. The suggestions are pretty good from our tests.
Testing
Since the Netflify preview is not working too well, we'll need to test this locally:

Clone this PR locally (using Github CLI: gh pr checkout 20)
yarn install
yarn start
Play around with the search box.
Switch back to the branch you were working with: git switch -

Authors
@ditsuke @yyjlincoln

[x]  My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Closes #6

The auto-completion functionality is working fine.  You can display that in a dropdown list like general search bars. Like when we search at google. Maybe you can add a div block for dropdown list after this line <input type="text" value={city} onChange={event => setCity(event.target.value)} />.

The auto-completion functionality is working fine. You can display that in a dropdown list like general search bars. Like when we search at google. Maybe you can add a div block for dropdown list after this line <input type="text" value={city} onChange={event => setCity(event.target.value)} />.

We're yet to work on it, but we'll add that functionality soon before graduating this PR from a draft.

Ready for review
cc @harshsharma6401 @KhudaDad414 @CIPHERTron @KshitijDarekar @psahithireddy

Requesting review: @gmelodie @ditsuke @Dhruv-Sachdev1313 @Aniket762

It's much better. Getting a similar experience as we get on other websites. Its good to go.

Looks great! We should probably add a note to tell other contributors to do yarn again after pulling the latest changes - as we added a new dependency

agreed.

Updated with testing instructions!

LGTM. Also, the testing instructions helped.

Force pushed to revert a code-style change. No material changes.

Marking good to merge/ RTC.


JOSS Review: simplify and update figure 2 in paper

Purpose:
JOSS Review:
Additional comment added by reviewer: "The diagram shown Fig. 2 is quite complicated and it is not easy to relate it to the text you added in this section. I think it might be simplified to better show the link with the text (you have numbers in the text which can not be found in the figure for example)"
Related:
Review https://github.com/openjournals/joss-reviews/issues/4336
Resolved:
Resolve #8

Codecov Report

Merging #15 (a257df8) into main (84be817) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   87.05%   87.05%
=======================================
  Files          15       15
  Lines        3392     3392
=======================================
  Hits         2953     2953
  Misses        439      439

Flag
Coverage Δ

unittests
87.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 84be817...a257df8. Read the comment docs.


Testing Flutter code that uses a plugin and platform channel

I have a flutter plugin which uses the platform channel to do some native work.
How do I properly write tests for my application that requires this plugin?
Unit tests only are good for pure dart functions. I don't believe Widget testing will be able to test things that use the platform channel to native. So that leaves integration testing.
From what I understand is that integration testing will start your main application and you can control it around your app and test things.
For my case, I want to test just the code that uses the plugin (that uses the platform channel for native stuff).
Also what is important is the values that come back from the platform channel, so it is important to call the native side using a real platform channel and not a mock one.
Is that possible?  I don't want mock!!! thank you!

Hi @chiyw
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

How to use unit test in flutter plugin? Is there any document？

@pedromassangocode

I shared the links for the best communities out there for asking questions. Did you even asked there?
Sorry but this is not a place for asking questions like this.


Android & iOS - Categories - Import spreadsheet icon is missing

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.41-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Launch New Expensify app
Go to workspace settings > Categories
Tap 3-dot menu

Expected Result:
There will be icon for Import spreadsheet
Actual Result:
Import spreadsheet icon is missing
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/ecf1671f-1275-4f12-8080-946a2199059a
View all open jobs on GitHub

We think that this bug might be related to #wave-control

Proposal
Please re-state the problem that we are trying to solve in this issue.
Android & iOS - Categories - Import spreadsheet icon is missing
What is the root cause of that problem?
The file SVG included fill="inherit". The issue with fill="inherit" not showing up in React Native on iOS and Android is likely due to how React Native handles SVG properties. Unlike web browsers, React Native doesn’t automatically inherit styles from parent components in the same way.
https://github.com/Expensify/App/blob/71db3f19691069182447b6b1b9fff62ec93be86c/assets/images/table.svg#L2
What changes do you think we should make in order to solve the problem?
To resolve this, you can use fill="currentColor" (ex: fill="#8B9C8F") instead of fill="inherit". Example:
<svg width="20" height="20" viewBox="0 0 20 20" fill="none" xmlns="http://www.w3.org/2000/svg">
- <path fill-rule="evenodd" clip-rule="evenodd" d="M0 4C0 2.89543 0.895431 2 2 2H18C19.1046 2 20 2.89543 20 4V16C20 17.1046 19.1046 18 18 18H2C0.895431 18 0 17.1046 0 16V4ZM2 6V10H9V6H2ZM11 6V10H18V6H11ZM9 12H2V16H9V12ZM11 16V12H18V16H11Z" fill="inherit"/>
+ <path fill-rule="evenodd" clip-rule="evenodd" d="M0 4C0 2.89543 0.895431 2 2 2H18C19.1046 2 20 2.89543 20 4V16C20 17.1046 19.1046 18 18 18H2C0.895431 18 0 17.1046 0 16V4ZM2 6V10H9V6H2ZM11 6V10H18V6H11ZM9 12H2V16H9V12ZM11 16V12H18V16H11Z" fill="#8B9C8F"/>
</svg>

POC
![Simulator Screenshot - iPhone 16 Pro Max - 2024-09-28 at 22 34 34](https://github.com/user-attachments/assets/9ab16ce2-3cbc-434e-aa75-536a3c11ce8e)

I can take this since I reviewed the offending PR.

Is it safe to retest this on staging and close is out? @luacmartins

Confirmed fix on staging.


GetEmailAccount returns true even if scenario is not assigned.

In the codeunit 8893 "Email Scenario" procedure GetEmailAccount will return true even if the requested scenario is not assigned. Provided that the default one exists.
Please see Line 30 of codeunit 8892 "Email Scenario Impl.". The return value should be false, as this case is invoked if the requested scenario is not assigned.

@MakGM-FORTO that would be a breacking change, since the current behaviour is defined in the documentation.
/// <summary>
/// Gets the email account used by the given email scenario.
/// If the no account is defined for the provided scenario, the default account (if defined) will be returned.
/// </summary>
/// <param name="Scenario">The scenario to look for.</param>
/// <param name="EmailAccount">Out parameter holding information about the email account.</param>
/// <returns>True if an account for the specified scenario was found; otherwise - false.</returns>
procedure GetEmailAccount(Scenario: Enum "Email Scenario"; var EmailAccount: Record "Email Account"): Boolean
begin
    exit(EmailScenarioImpl.GetEmailAccount(Scenario, EmailAccount));
end;

It should return the default account, but why with 'true'?

Isn`t that why we have two procedures GetDefaultEmailAccount and GetEmailAccount?

By the way, as you mentioned 'True if an account for the specified scenario was found;' In my case it is not found or am I misunderstanding what 'found' means?

I just want to check if there is an email account for the given scenario. But I can`t find a way to do it

Availability update: We will publish a fix for this issue in the next update for release 24.
Build ID to track: 18930.


StyleGAN 2 ADA transfer learning sourcenet error

I trained a couple of models with the pytorch version of StyleGAN 2 ADA and want to use them with SeFa. Most of them run fine, but using models that are based on the transfer learning sourcenet models provided in the official ADA repo results in the following error:
RuntimeError: Error(s) in loading state_dict for StyleGAN2Generator:
Missing key(s) in state_dict: "mapping.dense2.weight", "mapping.dense2.bias", "mapping.dense3.weight", "mapping.dense3.bias", "mapping.dense4.weight", "mapping.dense4.bias", "mapping.dense5.weight", "mapping.dense5.bias", "mapping.dense6.weight", "mapping.dense6.bias", "mapping.dense7.weight", "mapping.dense7.bias".
Unexpected key(s) in state_dict: "synthesis.layer15.weight", "synthesis.layer15.bias", "synthesis.layer15.noise_strength", "synthesis.layer15.noise", "synthesis.layer15.filter.kernel", "synthesis.layer15.style.weight", "synthesis.layer15.style.bias", "synthesis.layer16.weight", "synthesis.layer16.bias", "synthesis.layer16.noise_strength", "synthesis.layer16.noise", "synthesis.layer16.style.weight", "synthesis.layer16.style.bias", "synthesis.output8.weight", "synthesis.output8.bias", "synthesis.output8.style.weight", "synthesis.output8.style.bias".
I know that those models have a different layer structure, but is there any way to run them with this code anyways?

Hello, is the model of transfer learning training wrong? Are other models of training normal?


track directives: one-shots, no note value

could save yet more space in the midi chunks with more directives:
oneshot
for drum tracks, it's rare to need note-offs. so don't emit them for one-shot instruments.
no note value
something like a kick drum track may not need note value at all. only emit a highly compressible trigger value.

a shame that these types of optimizations only eliminate the most easily compressible data, so the savings is not much. but it's a net win (code support only adds 8 bytes compressed), and many tracks fall into these categories.


Add plan for module split-up

the docs folder contains a proposal for separating the Metafacture modules which are currently in metafacture-core into different packages.
I am not really happy with this current version of the proposal as the metafacture-core-modules package contains way to many modules for my taste. Perhaps you have some ideas how we could further split up these modules. I think, it is desirable to have a small number of modules in the core-modules package as a number of other packages use modules from this package. Hence, it would be good if it were rather stable. That is much easier if it does not contain many modules.
Another open question is the naming of the Metafacture module packages. Should they using generic names or implementation specific ones? For instance, should the package that contains the modules for processing JSON with the jackson library be named metafacture-json-modules or metafacture-jackson-modules? The former would hide the implementation detail of which library was used to do the actual reading and writing. However, it is no longer possible to offer to packages for json
processing which are based on different implementations. The latter naming scheme would allow this.
Regarding the general progress of the reorganisation. All pull requests for metafacture-core are now merged. At the moment I am updating all dependencies of the core to their latest versions. Afterwards, I want to determine what needs to go into the Metamorph API package. That is
hopefully not to difficult as Metamorph is already quite independent from the rest of Metafacture. Only the xml-based test cases may cause some trouble. Once this is done, I thought of reorganising the package structure to reflect the new multi-module structure, so that the actual split up in the next release becomes easier. This last all-in-one version of Metafacture will be released as metafacture-core 4.0.0

I am not really happy with this current version of the proposal as the metafacture-core-modules package contains way to many modules for my taste. Perhaps you have some ideas how we could further split up these modules. I think, it is desirable to have a small number of modules in the core-modules package as a number of other packages use modules from this package. Hence, it would be good if it were rather stable. That is much easier if it does not contain many modules.

How about moving the things that others depend on out of core-modules, and into commons? That way, we would not have to try to keep core-modules small, and could even merge it with extra-modules (the core-extra split seems a bit arbitrary, unlike the modules for specific formats). From the proposal, it seems this would affect EventList.Event, StreamBuffer, StreamFlattener, and SingleValue.

Another open question is the naming of the Metafacture module packages. Should they using generic names or implementation specific ones? For instance, should the package that contains the modules for processing JSON with the jackson library be named metafacture-json-modules or metafacture-jackson-modules? The former would hide the implementation detail of which library was used to do the actual reading and writing. However, it is no longer possible to offer to packages for json processing which are based on different implementations. The latter naming scheme would allow this.

I think the generic, format-oriented names make sense, for the reason you mention (being able to swap out the implementation). The problem you mention (not being able to offer packages for JSON processing which are based on different implementations) can actually be an advantage: offering multiple packages for JSON processing could be rather confusing. I think we should offer only one solution for each format.
Another point about the names: the -modules suffixes seem a little odd, since almost all of them have that suffix. Would it make sense to actually name them metafacture-json, metafacture-csv, etc? And a related general question: these are meant to be separate GitHub repos, right?
For project management issues and practical reasons, I'd try to somewhat limit the number of new repos we introduce. I completely see and agree with your point about dependencies, and trying to limit the things you have to pull in. So I really think it makes sense to have these separate JSON, CSV, XML, etc. repos.
But what about a single metafacture-morph repo instead of separating metafacture-morph-api, metafacture-test-tools, and metafacture-morph-modules? And if we no longer have the issue of others depending on core-modules (due to moving stuff to commons as described above), we might as well move the stats-modules and file-modules into core-modules.
So we would get something like this: framework, commons, morph, modules, bib, csv, xml, rdf, json.

How about moving the things that others depend on out of core-modules, and into commons?

That's an interesting idea, I will think it through. It might simplify things quite a bit, indeed.

Another point about the names: the -modules suffixes seem a little odd, since almost all of them have that suffix. Would it make sense to actually name them metafacture-json, metafacture-csv, etc?

That makes sense indeed. I have updated the proprosal accordingly.

And a related general question: these are meant to be separate GitHub repos, right?

My plan was to put all packages into a single git repo to simplify maintaining the modules. When using multiple repositories I made the experience that it is easy to forget about the extra packages as it is currently the case with metafacture-mediawiki and metafacture-sql which are constantly behind the core.
That isn't meant to stop others from setting up additional repos for their own modules, though. I basically see the git repository as the place for modules which are adopted by the Metafacture maintainers. That means foremost that they make sure that the modules are updated if the core changes and that new versions are released with the core.

But what about a single metafacture-morph repo instead of separating metafacture-morph-api, metafacture-test-tools, and metafacture-morph-modules?

By splitting the API (which basically consists of the Function and Collector interfaces) it should become possible to update Metamorph without having to update all extension packages. I am not sure though whether we might be able to achieve the same by following a strict versioning scheme and ensure that we do not change API classes between major releases.
The metafacture-test-tools should be a separate Maven artifact as they are only needed as a test dependency. As they depend on junit, metafacture-core currently pulls junit as a production dependency. This is really not pretty :-)

In the original split-up plan the modules in the reader package were not included. I am not quite sure what do to about them. Apart from the BeaconReader none of them adds any actual functionality. They are merely combinations of a reader with a decoder.
I first thought to put these combinations in the same packages as the corresponding decoders. However, the readers all depend on the same base class which would introduce another big dependency with very little actual advantages (expect for saving three lines of code).
As far as I know, the readers are only used by metafacture-test-tools. Hence, I thought to move them there and make them an internal part of this package. So, in effect removing them.
What do you think? Is anyone using these modules?

What do you think? Is anyone using these modules?

Can you please provide a list of these modules?

I am not really happy with this current version of the proposal as the metafacture-core-modules package contains way to many modules for my taste. Perhaps you have some ideas how we could further split up these modules. I think, it is desirable to have a small number of modules in the core-modules package as a number of other packages use modules from this package. Hence, it would be good if it were rather stable. That is much easier if it does not contain many modules.

We could split them by data type. Data types identified are: Formeta, Streams, Objects, Pojos, Records, Triples, Entities, Literals, Regex, Strings, Lines, Lists, Maps, Sets, Events and XML. Grouping options would be:

pojos and objects
records and triples
collections (lists, maps and sets)
...

The readers are the classes in org.culturegraph.mf.stream.reader:

BeaconReader: actual implemenation of a beacon file reader (this class should be kept)
CGXmlReader: XmlDecoder + CGXmlHandler
CsvReader: LineReader + CsvDecoder
FormetaReader: FormetaRecordsReader + FormetaDecoder
GenericXmlReader: XmlDecoder + GenericXmlHandler
LidoReader: XmlDecoder + GenericXmlHandler with lido root tag
MabReader: LineReader + MabDecoder
MarcReader: LineReader + Marc21Decoder
MarcXmlReader: XmlDecoder + MarcXmlHandler
MetsModsReader: XmlDecoder + GenericXmlHandler with mets root tag
MultiFormatReader`: can dynamically instantiate the concrete reader implementations using a name or mime-type defined in metastream-readers.properties
PicaReader: LineReader + PicaDecoder
PicaXmlReader: XmlDecoder + PicaXmlHandler

Only the BeaconReader and the GenericXmlReader are available as Flux commands (read-beacon and generic-xml. The latter can be replaced with decode-xml + handle-generic-xml

I find these classes also kind of superflous but I am sure they are used by somebody (well, seems to be I am one of them ;) ). Removing these "abbrevation classes"  to the metafacture-test-tools is ok for me.

I'm sure I also use them somewhere, but I also remember being confused about what to use, so I think having one official way (reader + decoder) is actually an improvement. So +1 for moving them from me.

Thanks for the feedback on the reader package. I moved the classes as planned (see  commit 53edfdb)

I have started to experimentally split up metafacture-core. My aim is to see how the new structure feels and if there are any dependencies between classes that I have missed before. I will regularly update this pull request (using git push --force) to share my progress.

As metafacture-4.0.0 is now released this pull request is no longer relevant.


Draft new fiber bundle structure

@ninamiolane, @alebrigant here a draft implementation of our FiberBundle/QuotientMetric discussion. I'm comparing against the metric-refactor branch so  if is easier to see how much it changes from there.
Let's consider the two scenarios for FiberBundle. In scenario 1 we explicitly know the total space  and the base. In scenario 2 we don't have access to base. Let's start by scenario 2, which can be exemplified by discrete_curves and PreShapeSpace.
Following our discussion, now we can do for DiscreteCurves:
ambient_manifold = Euclidean(dim=dim)
space = DiscreteCurves(
    ambient_manifold, k_sampling_points=k_sampling_points, equip=False,
)

space.equip_with_metric(SRVMetric)
space.equip_with_group_action("reparametrizations")   # for now group actions are str -> to extend in the future

space.equip_with_quotient_structure()  # uses "map" approach

# we have now
space.quotient_metric  # SRVQuotientMetric
space.fiber_bundle  # can be used for e.g. `align`

FiberBundle also has access to space. It is not a Manifold anymore: it simply needs to implement the methods that arise from a given combination of compatible metrics and group actions.
For PreShapeSpace:
space = PreShapeSpace(k_landmarks, m_ambient, equip=False)
space.equip_with_metric(PreShapeSpaceMetric)

space.equip_with_group_action("rotations")
space.equip_with_quotient_structure()

# we have now
space.quotient_metric  # KendallShapeMetric
space.fiber_bundle  # can be used for e.g. `align`

So now, each of the spaces has two metrics: metric and quotient_metric. Imagine now we have an estimator that receives an equipped space. Within the estimator we do space.metric. This means if we use this equipped_space idea, we need to find out a way of telling the estimator which is the proper metric to use. (The alternative is to always pass space and metric, but it conflicts with our ideas of not using metrics in isolation; it will also be cumbersome and error prone doing so).
To overcome this, I think we should have what I (for now) call active_metric. By default active_metric is metric. This way, we can do something like:

estimator = FrechetMean(space)

# uses `metric`
estimator.fit()

space.set_active_metric("quotient_metric")
# uses quotient_metric
estimator.fit()

The estimators need, of course, to be aware of this change, which mean that they should call space.active_metric (this was the way I solved the "tests" problem we've discuss the other day).
Finally, let's come back to scenario 1. I'm using what I call GeneralLinearBuresWassersteinBundle which has GeneralLinear (with MatricesMetric) as total space and SPDMatrices as base. This is a good example, because SPDMatrices equipped with the corresponding QuotientMetric corresponds to SPDMatrices equipped with SPDBuresWassersteinMetric, which means we can tests one against the other.
space = SPDMatrices(n=n, equip=False)

total_space = GeneralLinear(n=n, equip=False)
total_space.equip_with_metric(MatricesMetric)

bundle = GeneralLinearBuresWassersteinBundle(total_space)

space.equip_with_metric(QuotientMetric, fiber_bundle=bundle)

# we only have one metric: `space.metric`

I'm keeping this example very generic (e.g. we could have created a metric that hides the creation of the bundle).
Overall I would say the solution is quite satisfying. I've already done tests (with new framework) to several of these changes and with exceptions in SRVQuotientMetric, everything is passing.
What are your thoughts?
@nguigs, I think we may also be interested in this.

I really like it!! Thank you for thinking through it and implementing it already!
Some initial comments:

FiberBundle also has access to space.

Here, "space" is the total space, that is "DiscreteCurves" in your very first code snippet, for example: correct?
2,

we need to find out a way of telling the estimator which is the proper metric to use.

I like the active metric idea. Could we do something like this instead:
space.equip_with_metric(space.quotient_metric)
estimator = FrechetMean(space)

# uses `quotient_metric`
estimator.fit()

It could be simpler? I'm worried about the duplication: space.metric and space.active_metric.

Thanks for the extremely quick feedback @ninamiolane!

Exactly. Though for this case we kind of can forget this distinction between total space and base, as they are the same (meaning even the same instance).

No. The reason is that we only create one DiscreteCurves space. Then we pass it to FiberBundle. The metric that is in DiscreteCurves is the one FiberBundle "sees". If we change it, then the total space metric corresponds to the quotient space metric which in turn needs access to the total space metric.

With this strategy only the following instances exist: a space, a metric (which for the fiber bundle is kind of the total space metric), a fiber bundle (which accesses the space, its metric, and a group), a quotient metric (which accesses space and fiber bundle - and the other stuff though them).
For the duplication, of metric and active_metric it does not happen it terms of code, because active_metric is just getattr(self,  active_metric_name), i.e. it is not a distinct object (just a mapping).

Thanks Luis, this is great !!
Regarding the active metric: Nina, when you suggest to create a quotient space even in scenario 1, do you mean create a new class ? (Eg for curves something like DiscreteShapes) In that case wouldn't that mean we finally don't use (for the estimators at least) all the tools in DiscreteCurves that we discussed and that are implemented here ? Or were you thinking of creating a quotient space accessible from DiscreteCurves, e.g. if space is DiscreteCurves, something like space.bundle.base.metric that gives the same result as space.quotient_metric ? That seems a little tedious but maybe there is something simpler.
The advantage of the set_active_metric solution is that it really allows us to use the space.quotient_metric, so I kind of like it.
I don't really have other comments on the PR, it all looks really good to me.

For completeness, here the above example updated with the changes we've decided:
ambient_manifold = Euclidean(dim=dim)
space = DiscreteCurves(
    ambient_manifold, k_sampling_points=k_sampling_points, equip=False,
)

space.equip_with_metric(SRVMetric)
space.equip_with_group_action("reparametrizations")   # for now group actions are str -> to extend in the future

space.equip_with_quotient_structure()  # uses "map" approach

# we have now
space.quotient
space.fiber_bundle  # can be used for e.g. `align`
space.quotient.metric # SRVQuotientMetric

I've updated tests and notebooks accordingly.
@ninamiolane, @alebrigant I'm merging this one in geomstats/geomstats#1836. Please let me know if you have any comment.


[fuzzing] Add a service config fuzzer

This is split off from https://github.com/grpc/grpc/pull/32956. Adding an independent fuzzer for just the service config will allow it converge on problems faster. Keeping the service config fuzzer inside the client channel resolver fuzzer as well may be useful, since there could be an interaction between systems that the larger fuzzer can find, which this small one can't.

This is currently blocked on proto challenges internally. I'd like to get this fuzzer in good shape while we work those out.

Also, I think this fuzzer should live in test/core/service_config.

Also, I think this fuzzer should live in test/core/service_config.

It has been moved.


fix: add installGlobals to server handler for node16.x

The app.arc file specifies the node16.x runtime, which doesn't have the Fetch globals (Request, Response, Headers, etc), so deploying this stack right after creating will have a ReferenceError: Request is not defined.

Looks like this solves my issue, I should have checked here first! https://github.com/remix-run/grunge-stack/issues/168

Remix v2 requires Node 18
I updated app.arc in #169


Liability contracts data API

There are tasks where I need to get data from liabilities past. For example:

Get all reports data from liabilities with given model,
Get all RobotLiability type contracts addresses by given promisor,
Get all reports data by given promisee.
Is it possible to implement ROS API for such data acquisition to let AIRA-developers no need to use raw web3 RPC?

Any updates for this topic?

Seems like robonomics-tools-v<IP_ADDRESS> solve this issuse by liability explorer.


Adding transition effects to dark/light theme changes

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
It it would be nice if theme changes are accompanied by transitions into each other. I know this request might sound like too small, but I think it would add little more elegance into the website.
Additional context
I would love to add the above changes ! Should I do so ? I am new to GitHub, so should I send a PR first or wait for this issue to get assigned ?

Hi Shashank and yes that would be great! you can go ahead with it. Let me know if you need any help :D!


Enable Learning Resources for IT-less

I noticed this isn't in static/beta/itless. Do you want to add it there as well?

@InsaneZein , That is a good catch. I'll add it


It doesn't work...

After waiting 10 minutes I get this message 🤷♂
Due to a bug fix in https://github.com/huggingface/transformers/pull/28687 transcription using a multilingual Whisper will default to language detection followed by transcription instead of translation to English.This might be a breaking change for your use case. If you want to instead always translate your audio to English, make sure to pass language='en'.
C:\Ai\hebrew_whisper\venv\Lib\site-packages\transformers\models\whisper\modeling_whisper.py:691: UserWarning: 1Torch was not compiled with flash attention. (Triggered internally at ..\aten\src\ATen\native\transformers\cuda\sdp_utils.cpp:455.)
attn_output = torch.nn.functional.scaled_dot_product_attention(

You can use this project instead:
https://thewh1teagle.github.io/vibe/

Thanks Shmuel, it looks promising. I wish I could make your version work :)

Thanks Shmuel, it looks promising. I wish I could make your version work :)

I need to make an updated version. trying to find time for this...


PhantomJS hangs around after tests

Hi,
I've started to use grunt-mocha for running our test suite in phantomjs but currently having an issue where phantomjs hangs around after the mocha task has finished, the major issue is that it is using 100% CPU and slowing the build process down. What's even more worrying is that it doesn't even quit when grunt has finished.

Can you try it with a trivial test suite with a fresh install and see if it still happens on your machine? If not, it is likely something to do with your tests.

Closing since can't repro


Expose server instance

ActionCable.server in Channel instances is a code smell. It makes testing more difficult than it should be (parallel tests might have more than one server).
I think Connection should expose server, e.g.
class TestChannel < ActionCable::Channel::Base
	def broadcast(data)
		# ActionCable.server.broadcast ... # bad!!!
		self.server.broadcast("testing-#{test_id}", data)
	end

	private def server
		@connection.send(:server)
	end

I don't think we should encourage users to write ActionCable.server except as a last resort.

Hello, I tried to solve the issue.
This is what I did:
Modified Connection::Base to expose server as a public method. Updated Channel::Base to access server through the connection. This change allows channels to access the server without using ActionCable.server, making testing easier and avoiding potential issues with parallel tests.
You can review changes in this commit: https://github.com/lattaai13/anycable-actioncable-next-5/commit/57114377e376dcc993d28e7d48020f4929735ebb.

[!CAUTION]
Disclaimer: The commit was created by Latta AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

This issue was tried to solve for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.

Agree on that. I never liked it.
I'd even prefer to have just broadcast within the channel, not even server.broadcast, but we already have broadcast_to that has a slightly different logic (prepend stream names with the channel class name).
Given current design and compatibility with the previous version, probably, having connection.broadcast(...) is the way to go.


Ghost "No categories matching" message

explore categories branch

In category tab, search for "banana"
In category tab, search for "owrgowirg"
Message "No categories matching owrgowirg found" appears
Press X
Recent searches appear
Press "banana"
Message "No categories matching owrgowirg found" appears, with throbble on top of it

Expected: At step 7, the message should not appear

Same issue was in image search.

@ujjwalagrawal17 still  present with this latest commits:

Thanks!


V6.4

These are some minor tweaks to the staging branch that happened this morning. It is the version currently running on my.farm.bot.

Coverage remained the same at 90.8% when pulling ab4015773deedfd4089cac6dcf2a19d558a823a4 on RickCarlino:v6.4 into 3aa39860d575bb4ea764308cca75596df426392d on FarmBot:staging.


AppsFlyer iOS SDK 6.13.0 required to comply with iAd framework removal

In your readme it says that the current version of iOS AppsFlyer SDK is v6.12.2.
Here, on your support page, it says that we need to be on AppsFlyer iOS SDK 6.13.0 to be compliant with the removal of iAd framework. If we are not, Apple will block our app within two years.
I kindly ask that you would bump your iOS AppsFlyer SDK to >=6.13.0 so that we can update our app to use it as soon as possible to avoid having it blocked by Apple.
Report
What did you expect to happen?
I expect the latest release of appsflyer-react-native-plugin to use AppsFlyer iOS SDK >=6.13.0.
What happened instead?
Latest release of appsflyer-react-native-plugin uses iOS AppsFlyer SDK is v6.12.2.

Hi @langest
We recently released the new iOS SDK version without any reference to the deprecated iAd framework. The version was released last week (v6.12.3) for the native environment. We do expect this release to be included in our next React Native plugin release. We currently have no concrete estimation on when this plugin version will be released but we are aware of this issue and it will be handled soon.
Please be advised that regarding Apple's point of view there is no immediate action and it can take up to 2 years to be enforced. Obviously we are working to have this iAd removal on all environments way before then.

Just to add to this we need version >=6.13.0 to send consent for DMA compliance on Android - received this notification via an email from AppsFlyer. Sounds like there needs to be a release before 6th March so anyone using this plugin can facilitate this?


Agent epoch monitoring

[x] Config (e.g., mutation/time thresholds),
[ ] Improve testing to demo consistency with concurrent writers (peers) when epoch is being created.

https://github.com/dxos/dxos/pull/4230


--host should be a top-level flag.

Right now, a few commands have a --host flag. We should probably just make it a top-level flag.

This might not be necessary, depending on @adamreese 's fix for tunnels.

Done


Inline suggestions support for line-by-line acceptance

I saw that the word-by-word acceptance feature for inline suggestions was released in the 1.75 release (though it seems there is still a bug https://github.com/microsoft/vscode/issues/172162).
I think it would be great if we could also have line-by-line suggestion acceptance with default keybindings of Ctrl+DownArrow and Ctrl+UpArrow.

You can bind a keybinding:
    {
        "key": "shift+tab",
        "command": "editor.action.inlineSuggest.acceptNextLine",
        "when": "inlineSuggestionVisible"
    }

Or use the hover:


Documentation on Usage

Add documentation of the different components and how to use in README.md

I just checked the storybook. I think that covers everything. Closing issue


Bumped Mesos to include operator API fix.

High-level description
This bumps DC/OS upstream to include the latest nightly Mesos build.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): Mesos diff
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee


New Repo?

Hi,
Is this same Vundle from gmarik? or is this a fork?
My vundle still works with Plugin "gmarik/vundle" but confused!

Hi. It's essentially the same, see #608 particularly: "Create an organization: VundleVim". It was transferred to that organization and redirects automatically.
"When a repository is transferred, all links to the previous location are automatically redirected to the new location."

This should probably be added somewhere like the wiki

@rudrab yes, it is. Docs should be up to date mentioning it…


Image Dump

Create an Image Dump Feature that allows you to grab all images from a certain amount of messages.

Added in #96


Update README.md

Update documentation about HttpPostJsonBody

Hi jpillias, thank you very much for your contribution!

You're welcom.
Thank you for your job too.
Julien Pillias
Responsable Technique R&D
31 rue des Jeuneurs, 75002 Paris
Tèl : 01 84 16 73 77
<EMAIL_ADDRESS>www.mail-metrics.com
www.key-performance-group.com
2016-08-11 11:10 GMT+02:00 Gabriel Anca Corral<EMAIL_ADDRESS>
Hi jpillias, thank you very much for your contribution!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LeaseWeb/LswApiCallerBundle/pull/27#issuecomment-239109400,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQgs054UcBTJq3bfUHsiGJCZgtwWMZ8iks5qeub6gaJpZM4Jh2DC
.


Lock emoji issue

Dear SDL_TTFs,
I am running the LEM editor with SDL2 GUI on OpenBSD. LEM uses a LOCK symbol in it's modeline to indicate a read-only buffer.
Unfortunately this does cause an error and I believe it is due to SDL_TTF.
Here is what's happening using the cl-sdl2-ttf package with sbcl (Steel Bank Common Lisp):
* (make-string 1 :initial-element #\LOCK)
"🔒"
* (SDL2-TTF:RENDER-UTF8-BLENDED (sdl2-ttf:open-font #P"/home/rpx/common-lisp/lem/frontends/sdl2/resources/fonts/NotoColorEmoji.ttf" 15) (make-string 1 :initial-element #\LOCK) 255 255 255 0)
debugger invoked on a SDL2-TTF::SDL-TTF-ERROR in thread #<THREAD "main thread" RUNNING {1001480003}>:    SDL Error (#<SDL-SURFACE {#X00000000}>): Text has zero width
So the error message is: SDL Error (#<SDL-SURFACE {#X00000000}>): Text has zero width
But, generally it works, as I can demonstrate with the sigma symbol:
* (make-string 1 :initial-element #\GREEK_CAPITAL_LETTER_SIGMA)
"Σ"
* (SDL2-TTF:RENDER-UTF8-BLENDED (sdl2-ttf:open-font #P"/home/rpx/common-lisp/lem/frontends/sdl2/resources/fonts/NotoColorEmoji.ttf" 15) (make-string 1 :initial-element #\GREEK_CAPITAL_LETTER_SIGMA) 255 255 255 0)
#<SDL2-FFI:SDL-SURFACE {#X929E5262360}>
Does anybody know why that is happening?
My setup:
tron$ uname -a
OpenBSD tron.fritz.box 7.4 GENERIC.MP#1544 amd64
tron$ doas pkg_info sdl2-ttf
Information for inst:sdl2-ttf-2.20.2p2
tron$ doas pkg_info sdl2
Information for inst:sdl2-2.28.5
tron$ sbcl
This is SBCL 2.3.11.openbsd.sbcl-2.3.11, an implementation of ANSI Common Lisp.
Any suggestions / advice appreciated!
Many thanks,
Robert

Can you recompile some stuff, emoji should work
Freetype needs to be compiled with FT_CONFIG_OPTION_USE_PNG
( cmake -DFT_DISABLE_PNG=OFF ../SDL_ttf_git )

Many thanks for your suggestion.
I think Freetype is part of OpenBSDs xenocara and also part of the base system.
Hmm..

you're welcome!
maybe it's faster for you to check:
also, you can try to compile,
SDL_ttf/examples/testapp.c

modify the source to only use your NotoFont emoji font
only use the string named "emoji"
set rendering blended ( F1 for help,  "m" to toggle the blend/shaded/text rendering)

it should display the emoji in color ! if not, then freetype hasn't been compiled with  png support ..

Thanks again and sorry, but I don't know how to do that.
I will ask on the mailinglist if this is in place.

Follow up to this: a couple things I found while researching the problem.

The issue is manifesting in SDL2, and there isn't a good quick way to build the example files. Additionally, at this point, SDL2_ttf is still using Autotools, so the CMake parameter provided doesn't actually fix the problem.
Apparently, SDL2's build of Freetype doesn't enable PNG support by default. I've found two ways to get around this issue.

Configure with CFLAGS="-DFT_CONFIG_OPTION_USE_PNG" LDFLAGS="-lpng" ./configure <options>. As far as I can tell, this is a functional workaround, but I haven't tested extensively.
Use the system's version of Freetype by using ./configure --disable-freetype-builtin <options>. For Arch, at least, Freetype is built with PNG support, so this fixes that issue.

@OrdoFlammae thanks a lot for commenting! Think OpenBSDs system freetype doesn't have the option enabled, but I'll check with maintainer / the mailing list.

Maybe we should look into adding PNG support for SDL_ttf 3.0?

Fix for arch seems this: link
Maybe adding PNG is the general fix?

@OrdoFlammae Tried Option ii for OpenBSD.
Recompiled Xenocara with FT_CONFIG_OPTION_USE_PNG enabled a
Recompiled sdl2-ttf port with --disable-freetype-builtin
Works, but unfortunately time consuming.
Adding this to the main build in OpenBSD is tricky because libpng is in ports and building base components can't depend on that, so libpng would need adding to xenocara in some way, which would be a lot of work to handle in ports, it would also make it more difficult to get libpng updated, and might not be accepted in the first place.
Source

FYI rebuilding with FT_DISABLE_PNG=off does somewhat work, we're getting color emoji support (see here), the problem is that said emoji can't be scaled (in NotoColorEmoji for example, we're stuck to size 109, see this for more info) and so users get strange font size mismatches (see here).
Tracking the size issue at #363, but I thought it would also be relevant to mention it here.

@madebr, can/should we enable FT_CONFIG_OPTION_USE_PNG in the vendored FreeType we use?