Is it appropriate to name GO:0022900 as "electron transport chain"?

The electron transport chain is commonly defined as "a series of protein complexes ..." in literature.
https://www.ncbi.nlm.nih.gov/books/NBK526105/
https://en.wikipedia.org/wiki/Electron_transport_chain
In other words, it is not a process per se. It just enables the process of transporting electrons.
But GO BP defines it as a process ("A process in which a series of electron carriers ...") which is inconsistent with the literature. For this reason, it sounds like it should be renamed to something else that is not in conflict with the commonly accepted definition in the literature.

But two paragraphs on [(https://www.ncbi.nlm.nih.gov/books/NBK526105/)]:

Oxidative phosphorylation is made up of 2 parts: the electron transport chain (ETC) and chemiosmosis. The ETC is a collection of proteins and organic molecules, which electrons pass through in a series of redox reactions, and release energy. The energy released forms a proton gradient, which is used in chemiosmosis to make a large amount of ATP.

How should we parse this statement? Is it asserting that oxidative phosphorylation is the sum of a process (chemiosmosis) and a supercomplex (the electron transport chain), or two supercomplexes (the ATPase complex and coupled transporter complexes that mediate chemiosmosis plus the electron transport supercomplex), or should we read it as an ordinary-language statement with ordinary-language imprecision as asserting that the overall process has two sub-process parts? Unlike the first two readings, this one looks like a really good fit to the known biology that GO is trying to capture.

@deustp01 The author clearly used ETC to refer two things in this passage: 1) the process that GO BP refers, 2) the corresponding complex.
When this kind of ambiguity occurs, GO should not just use one of the possible usages. The meaning used in GO of the phrase should be consistent with the common meaning of the phrase in the literature. Based on the PubMed search with the query "electron transport chain"[tiab],  it mostly refers to the complex instead of the process. Using "electron transport chain" to refer to the process will cause confusion when people mostly use it to refer to the complex.
One should also look up the dictionary definition of the root word of the phrase. In this case,chain is defined as "a sequence of items of the same type forming a line" in New Oxford American Dictionary. It doesn't have the meaning of process, which may explain why its most common usage refers to the complex.
For these reasons, "electron transport chain" is not the best term name. Something more appropriate and less ambiguous is preferred.

I'm with Peter here :)
Ontology semantics = precise
Human semantics = very ambiguous
ETC: "A process in which a series of electron carriers operate together to transfer electrons from donors to any of several different terminal electron acceptors to generate a transmembrane electrochemical gradient."
I think ETC chain = series of components AND processes:
the processes of multiple redox reactions facilitated by a chain of complexes resulting the establishment of a proton gradient across a membrane
--> overall, what's meant to be defined is the series of processes but a chain of complexes is facilitating the process.
Just my 2p...

I agree, I couldn't think of a better label for the precise we are trying to describe here:
definition:
"A process in which a series of electron carriers operate together to transfer electrons from donors to any of several different terminal electron acceptors to generate a transmembrane electrochemical gradient."
And since this is a BP term  it shouldn't really be misinterpreted. Labels are ambiguous. Definitions are precise.

@ValWood @bmeldal Just having a definition is not necessarily always sufficient. Let's put it in another way. If a phrase can mean both a complex and a biological process, it has to be disambiguated. If the molecular complex meaning of ETC were to put in GO, how would you disambiguate the molecular complex ETC term name and the biological process ETC term name?

As Val said, the definition clearly describes this term as a process. Yes, the title is a little ambiguous in this case and that's why we always remind people to check the definition first.
Personally, when I hear "ETC" I immediately think of the process, not just the chain of complexes that facilitate the process.

GO:0098803    respiratory chain complex ?

...is the closest component term (for this reason).
The constitutive complexes are clearly defined

Plus the term is a child of biological_process. There are many many cases where a term name is ambiguously used in the community. That is why it is very important to look at the definition of the term and how the term is placed in the ontology. We try to use term names that are used frequently in the community and as @deustp01 points out above, this label is used to describe the process.

@ValWood So there is not a term for the ETC complex. My question is: if the ETC complex were in GO, how you would disambiguate the names of the term for the complex and the term for the process? Or if there were some other phrase that could mean both a complex and a process, how would the exact phrase be assigned to one of the two terms? I think that it better to make the names consistent with the common usage in the literature, i.e., to assign the phrase to the term for which that is most commonly used in the literature.
@ukemi @bmeldal I think this ambiguity was caused by the overloading of the same phrase. The most commonly accepted definition of "electron transport chain" is still the corresponding complex rather than the process. This can be seen in the widely used textbook Molecular Biology of the Cell (5/ed): "The entire set of proteins in the membrane, together with the small molecules involved in the orderly sequence of electron transfers, is called an electron-transport chain." If such an influential textbook define "electron transport chain" as a complex, why GO would use "electron transport chain" for the less used and overloaded meaning of the ETC process? Having a definition sentence of a term is not a substitution for a good name of the term.
Most of the ambiguous phrases should have a major meaning and one or more minor meanings. So the ambiguity can be resolved by checking which meaning is used most frequently in the literature and using that meaning. With this relatively objective principle, the determination of the ambiguous term name will be very clear. Only when a phrase is equally used for multiple meanings, which should be rare, there might need a discussion.

My question is: if the ETC complex were in GO, how you would disambiguate the names of the term for the complex and the term for the process? Or if there were some other phrase that could mean both a complex and a process, how would the exact phrase be assigned to one of the two terms?

As I explained above "ETC complex" is not in GO, because it isn't a single complex. There was a long discussion about ETC process vs. component a  but I can't find it. Maybe it was back on the SF tracker.
We would not use the same primary name for 2 terms (although one phrase could be a synonym). But more importantly we do not annotate to term labels, we annotate to the definitions. A curator would need to figure out the meaning and make the correct assignment. Curation isn't text-mining, we are interpreting the data and making the correct selection after reading the definition.
GO:0070469    respirasome Definition
The protein complexes that form the electron transport system (the respiratory chain), associated with a cell membrane, usually the plasma membrane (in prokaryotes) or the inner mitochondrial membrane (on eukaryotes). The respiratory chain complexes transfer electrons from an electron donor to an electron acceptor and are associated with a proton pump to create a transmembrane electrochemical gradient.
has exact synonym "membrane electron transport chain" (but this should probably be related because of the ambiguity)

@pengyubio
If you provide a better term name for ETC process that is less ambiguous to the user community we can change that and make ETC a synonym.
Similarly, GO:0070469 respirasome (synonym: membrane electron transport chain) looks like the "ETC complex" you are looking for. Does it need a better name? Looks like the term name got swapped at some point (see 'comment').
You will find GO:0098803 respiratory chain complex as a part_of child of respirasome as they are the constituent parts of the respirasome, while respirasome is a bucket term.
NB: Most complex terms end in 'complex' or "-some" and are disambiguated from their processes that way.

closing. If there is a specific action,  (i.e renaming suggestion idea), open a new ticket and link back to this one for context.


Require Completed Sets

Requirements:

All existing sets should be required to be completed before adding new sets
Sets should complete Left -> Right, with sets on the right greyed out until they are available.

Completed.


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Current coverage is 87.40% (diff: 100%)

Merging #149 into master will not change coverage

@@             master       #149   diff @@
==========================================
  Files             1          1
  Lines           262        262
  Methods          42         42
  Messages          0          0
  Branches         42         42
==========================================
  Hits            229        229
  Misses           33         33
  Partials          0          0

Powered by Codecov. Last update 37fbb72...2bb7cd9

Current coverage is 87.40% (diff: 100%)

Merging #149 into master will not change coverage

@@             master       #149   diff @@
==========================================
  Files             1          1
  Lines           262        262
  Methods          42         42
  Messages          0          0
  Branches         42         42
==========================================
  Hits            229        229
  Misses           33         33
  Partials          0          0

Powered by Codecov. Last update 37fbb72...2bb7cd9


Adding multiple authors requires an unintuitive pre-filled form deletion

When adding multiple authors, one must go back into the addition dialog for each author.  Further, the user must delete the previously-added user, even though the user is not actually being removed from the authors list.  This then clears up the search box to add another author.
001.zip

I think clearing the searchbox after adding an Author is a good improvement. However, having the modal window persist after this action is a kind of a trade-off. For resource owners who want to add a single Author it is ideal to have the modal window dismissed automatically.
I'll wait for @dtarb's say in this matter to proceed.

I agree with @Maurier.  Fix the problem of the search box not being clear when the modal is opened.  I do not think it is a problem for the modal to be dismissed and a user have to click + for each author they want to add.  However I do not feel strongly and would support improvements (but not at high priority).
Also - there should be consistency in the behavior between adding authors and adding contributors.  Right now there is not.  Groups can not be added as contributors (they should be).  The contributors modal does not remember the previous user (which is what we want for authors).  Similarity between the manage access and adding authors and contributors would also be advantageous for a consistent user experience.

This is also related to https://github.com/hydroshare/hydroshare/issues/3700 and will be worked out as part of the same PR.


modify checks for convergence

Options for convergence other than rho_dne can be used when iperiodic=3 and domain_parallel = 'y'. This PR enables for uses to use the options.

Thank you for merging, @ayamada224 san.


legcord: 1.0.1 -> 1.0.2

Updated legcord to v1.0.2: https://github.com/Legcord/Legcord/releases/tag/v1.0.2
Added updateScript to allow r-ryantm to submit update PRs for this package.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 350121

x86_64-linux

:white_check_mark: 1 package built:

legcord


Add remoteBaggageRestrictionManager

[x] Tests

Coverage decreased (-0.04%) to 85.355% when pulling 28607fce18a5f006f38e6ae8fe9530c716df83a8 on Add_RemoteBaggageRestrictionManager into 08628f9e69418b067587bb7cd0f6c1968808a786 on master.

This design affects all languages, and the API visibility issues are similar. Can the following work?

jaeger/

interface BaggageSetter { SetBaggage() }
span checks if baggageSetter == null then it stores the baggage manually, no additional logic, otherwise calls baggageSetter.SetBaggage(self, key value)

internal/baggage

struct DefaultBaggageSetter
deals with metrics and logging
may want to rename baggageSetter to baggageActor (whatever)
contains pointer to BaggageRestrictionManager that it uses for the rules

default BRM is - allow all
RemoteBaggageRestrictionManager pulls the rules from the agent

config

BaggageManagerConfig
creates BaggageRestrictionManager with RemoteBaggageRestrictionManager

This way the two API additions are BaggageSetter interface and BaggageManagerConfig struct. Everything else is either private or stashed under internal/. There don't seem to be any circular dependencies.
We could also go a bit further and move RemoteBRM under internal/baggage/remote. This way if people are not using config package, and don't want the RemoteBRM, they don't even need to compile that package into their app. Right now we're forcing everyone to compile almost everything from the client, we should really be doing a better job at isolation.

What we can do instead:

jaeger/

interface BaggageRestrictionManager { GetRestriction(key) (bool, int) }

private baggageSetter initialized by tracer that has ptr to BaggageRestrictionManager (baggageSetter handles metrics and logs)

internal/baggage

struct Default BRM - allow all

struct Remote BRM pulls the rules from the agent

config

BaggageManagerConfig

creates BaggageRestrictionManager with RemoteBaggageRestrictionManager

interface BaggageRestrictionManager { GetRestriction(key) (bool, int) }

well my goal was to come up with a unified API across all languages. (bool, int) won't work in Java

my bad
interface BaggageRestrictionManager { GetRestriction(key) Restriction }

type Restriction struct {
    valid bool
    maxValueLength int
    whatever in the future we want to add
}

and restriction is generated once in the DBRM and once a minute in RBRM

So one drawback I see in your approach is "(baggageSetter handles metrics and logs)" - meaning there isn't any way to actually NOT log baggage without introducing more options to the Tracer (and increasing its surface). In my proposal the side-effects of setting baggage are encapsulated into another struct that can be replaced if needed, and Tracer is only exposed to its SetBaggage interface. Although when you mentioned circular dependency on the call, I see the potential for it here since the setter might need access to private Span API - :-( ...

I addressed some of the concerns and moved as much of the code over to the internal directory. I'll add the design decisions inline

Coverage decreased (-4.2%) to 81.231% when pulling 17e056517923ff5e5cbfa60a23aadb16e08ac021 on Add_RemoteBaggageRestrictionManager into 08628f9e69418b067587bb7cd0f6c1968808a786 on master.

span calls tracer.setBaggage, but that was already done in a previous PR. Information hiding ftw

Coverage decreased (-0.3%) to 85.057% when pulling ecc0680493e2920a3d1ca51512652f60dce83370 on Add_RemoteBaggageRestrictionManager into 08628f9e69418b067587bb7cd0f6c1968808a786 on master.

Coverage increased (+0.2%) to 85.573% when pulling fe802ec470f203c7754e5f449385cd780ca1514f on Add_RemoteBaggageRestrictionManager into 08628f9e69418b067587bb7cd0f6c1968808a786 on master.

Coverage increased (+0.2%) to 85.578% when pulling 5cc274eb81f36fb8e2c39d430d2fb6baf26d977e on Add_RemoteBaggageRestrictionManager into 08628f9e69418b067587bb7cd0f6c1968808a786 on master.

Coverage increased (+0.2%) to 85.578% when pulling 7ed2a48a1cc27b579765aa23ddf478c85099cd9c on Add_RemoteBaggageRestrictionManager into 08628f9e69418b067587bb7cd0f6c1968808a786 on master.


Unable to do large batch requests

Hello!
I am doing batch requests in the size of 1000-2000~ which works fine on my other nodes with the params
--rpc.batch-request-limit=4000 \
--rpc.batch-response-max-size=100000000

but on my base archive node with mostly stock settings with the above it is still giving me exceeds
{'jsonrpc': '2.0', 'id': 1, 'error': {'code': -32600, 'message': 'batch too large'}} with a payload size of 1400~ items
for context i am batch grabbing tx info + tx receipt +  tx trace for all transactions in a block for indexing purposes
Using version node-0.8.1 in archive mode
node-081-node   latest    8e37bf9fc631   2 weeks ago     975MB
node-081-geth   latest    e8feba8d8fcd   2 weeks ago     975MB

@anonymous-sushi
Thanks for reporting this issue. Confirming this is still in our backlog, pending to be reviewed.


Reduced CodeStyle Warnings

@scopeInfinity resolve conflicts and restart failing builds, we can then review and merge

@mananwason Resolved merge conflicts.

@scopeInfinity I don't think this is required right now. Let's just close this.


chore: Fix vulnerability in chromedriver

Details
https://github.com/microsoft/axe-sarif-converter/security/dependabot/26 calls out a command injection vulnerability in chromecriver versions earlier than 119.0.1. This PR adds a resolution for chromedriver to force usage of the version with the fixed vulnerability.
Motivation
Address https://github.com/microsoft/axe-sarif-converter/security/dependabot/26
Context

Pull request checklist

[x] PR title respects Conventional Commits (starts with fix:, feat:, etc, and is suitable for user-facing release notes)
[x] PR contains no breaking changes, OR description of both PR and final merge commit starts with BREAKING CHANGE:
[x] (if applicable) Addresses issue: https://github.com/microsoft/axe-sarif-converter/security/dependabot/26
[n/a] Added relevant unit tests for your changes
[x] Ran yarn precheckin
[x] Verified code coverage for the changes made

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sticky IngressRouteTCP based on source IP

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Hello,
I would like the ability of IngressRouteTCP to allow sticky routing based on source IP (compatible with PROXY protocol), this affinity ensures the client is served on the same pod.
In our use case, the client established multiple connections (mTLS, TLS handshake) to 3rd party services. Since not all services support Session Ticket/TLS PSK yet, it's costly and slow to restart connections (Require full handshake). When you have undred pods, it's just loop to warm each pods, previous warmed become cold (connection are closed) so it's very inneficient. We would like to reroute the same user to this pod.

Hello @jjsaunier, and thanks for your suggestion.
We are interested in this issue and the traction it will receive. We will leave the status as kind/proposal to give the community time to let us know if they would like this idea. We will reevaluate as people answer.
The conversation is time-boxed to one month.

Hello there,
We have set the status to enhancement.
However, this would not make it to our roadmap as we are focused elsewhere. If a community member would like to build it, let us know, and we will work with you to ensure you have all the information needed to merge it.
We prefer to work with our community members at the beginning of the design process to ensure that we are aligned and can move quickly with the review and merge process. Let us know here or create a PR before you start, and we will work with you there.
Don’t forget to check out the contributor docs and link the PR to this issue.


Plugin works now without decompiler plugin.

Also version number has been increased to 2.0.1

Many thanks, @erazor-de! The changes LGTM. 🚀
Related issue: https://github.com/giladreich/ida_migrator/issues/1


Added Shortest Palindrome problem and solution

This PR closes issue #241

Hi there!
I am the Mentor of this project .
Although this repo is not part of Hacktoberfest 2024, we have some great alternatives for you. Welcome to SkillShow, a repository focused on skill enhancement!
Guidelines:
Please adhere to the existing guidelines.
No additional rules, thanks!
Check it out here: SkillShow.
Note: Start by creating an issue. Once it’s assigned to you, feel free to open a pull request (PR). Directly opening a PR will lead to an invalid label.
Looking forward to your contributions!


Custom icon not showing in multiple pages UI in python

i am getting this error after 3rd pages
self.tk.call('wm', 'iconphoto', self._w, *args)
_tkinter.TclError: can't use "pyimage7" as iconphoto: not a photo image

I have a multipage UI python program
I am not destroying the previous page when I navigate to new page
due to this I am getting an error for icon and program stops working
any solution ??
Note: I cannot destroy the pervious pages as they are dependent as holds processing data

Could you share a simplified version of your code that reproduces this issue, please?

Thank you for your reply, but now it is working I am changing the code with false to true and applied this code only in main file and rest of pages icon also work shown.
#set the path to set icon
p1 = PhotoImage(file = r'sc.png')
Setting icon of root window
root.iconphoto(True, p1)//False
but for applied this code logo is not very clear, is there any other option to change icon?

I'm sorry, I really don't understand the issue you're having. Are you trying to use a custom png as the page icon, instead of an emoji? If so, I wouldn't expect that to work, but if you have a way to make it work I would love to see how you are accomplishing that (with enough code to reproduce it).


fix

file path arg is in class constructor.

There seems to be some problem with the instructions at that link:

The conflict is caused by:
    datatrove 0.4.0 depends on numpy>=2.0.0
    datatrove[dev] 0.4.0 depends on numpy>=2.0.0
    datatrove[all] 0.4.0 depends on numpy>=2.0.0
    datatrove[quality] 0.4.0 depends on numpy>=2.0.0
    datatrove[testing] 0.4.0 depends on numpy>=2.0.0
    datatrove[cli] 0.4.0 depends on numpy>=2.0.0
    datatrove[io] 0.4.0 depends on numpy>=2.0.0
    datatrove[multilingual] 0.4.0 depends on numpy>=2.0.0
    spacy[ja] 3.8.2 depends on numpy>=1.19.0; python_version >= "3.9"
    datatrove[processing] 0.4.0 depends on numpy>=2.0.0
    datatrove[s3] 0.4.0 depends on numpy>=2.0.0
    datasets 3.1.0 depends on numpy>=1.17
    spacy 3.8.2 depends on numpy>=1.19.0; python_version >= "3.9"
    tensorflow 2.16.2 depends on numpy<2.0.0 and >=1.26.0; python_version >= "3.12"
    datatrove 0.4.0 depends on numpy>=2.0.0
    datatrove[dev] 0.4.0 depends on numpy>=2.0.0
    datatrove[all] 0.4.0 depends on numpy>=2.0.0
    datatrove[quality] 0.4.0 depends on numpy>=2.0.0
    datatrove[testing] 0.4.0 depends on numpy>=2.0.0
    datatrove[cli] 0.4.0 depends on numpy>=2.0.0
    datatrove[io] 0.4.0 depends on numpy>=2.0.0
    datatrove[multilingual] 0.4.0 depends on numpy>=2.0.0
    spacy[ja] 3.8.2 depends on numpy>=1.19.0; python_version >= "3.9"
    datatrove[processing] 0.4.0 depends on numpy>=2.0.0
    datatrove[s3] 0.4.0 depends on numpy>=2.0.0
    datasets 3.1.0 depends on numpy>=1.17
    spacy 3.8.2 depends on numpy>=1.19.0; python_version >= "3.9"
    tensorflow 2.16.1 depends on numpy<2.0.0 and >=1.26.0; python_version >= "3.12"

But I ruff formatted it.

LGTM, ty :)


[CT-2496] [Spike] Determine best practices for composing nodes with python Protocols

Spike
For 1.6 we'll be dropping support for python 3.7. With python >= 3.8 we can being to count on the existence of python's Protocol. We have a desire to use Protocol for defining shared interfaces between dbt-core and MetricFlow for the integration of the two. As such we need to develop a better understanding of what our best practices should be when using Protocols in the composition of python classes like dbt-core nodes.

Spike Report
I reviewed the existing work Quigley has done in the dbt-semantic-interfaces repo toward extracting interfaces which will be shared between Core and MetricFlow.
Recommendations

We should not use the runtime_checkable decorator on protocols. Although Gerda raised valid concerns that isinstance() will not work without this decorator, it has severe limitations which could bite us. It verifies the presence of member variables and methods, but not their types or signatures. Fortunately, there are only four places we use isinstance() to check a node's type, and they would all be improved by refactoring the code to avoid its use.

We should not support default methods on these Protocols (effectively making them mixins) at least for now. The added complexity is not justified by any immediate use case, and functionality can be provided in the default Protocol implementations in the semantic interfaces library.

Since the protocols in semantic interfaces will constitute a cross-product API and because changes to them will have versioning implications, we should organize them separately from the rest of the code in that library. They should be contained entirely within the Protocols directory of the repository, and the types used by their member variables and function signatures should also be defined in the same directory. Some of these types (e.g. enums) might be concrete types, but they are part of the expected interface just the same. The work of isolating the protocols to their own directory is largely complete, but there are some exceptions which should be removed as we move forward with MetricFlow integration.

Loose Ends

Since the protocols will exist in a separate repository from Core, we will need to have a versioning strategy and release schedule for that repository. I don't know enough about the roadmap now to recommend a complete strategy, but we should look to establish a version 1.0.0 of dbt-semantic-interfaces by the time we release dbt 1.6.

Since the semantic interfaces repo is shared by Core and MetricFlow, who will own repo maintenance, releases, and how will we coordinate changes?


pypi tarballs are missing tests directory

The tests are missing from the pypi tarballs for at least a few releases.
Oddly, when I run python setup.py sdist from a git checkout, the resulting tarball does have the tests; I'm not sure how that is working.

Which releases are affected?

@Lukasa, I believe this is related to moving the tests into their own directory starting in 2.10.
#4015 probably should have been to change test_requests.py to 'tests' in MANIFEST.in rather than removing the bad filename.

I checked the tarballs for the following releases, they are all missing the tests directory.
2.11.1
2.12.3
2.13.0
2.14.2

That sounds right @nateprewitt. Want to fix that up with a PR?

Yep, I'll throw the PR up when I get back to a computer, unless you have any interest @floppym.

Oddly, when I run python setup.py sdist from a git checkout, the resulting tarball does have the tests; I'm not sure how that is working.

This was working for me because I had the hgdistver python package installed. This installs a setuptools plugin that auto-detects any files under version control.


Iced applications fail to compile under wasm target on master

On current master, Iced applications fail to compile under the wasm target, because Subscription is not implemented in iced_web:
error[E0432]: unresolved import `crate::Subscription`
 --> C:\Dev\iced\src\application.rs:1:41
  |
1 | use crate::{Command, Element, Settings, Subscription};
  |                                         ^^^^^^^^^^^^ no `Subscription` in the root

error[E0432]: unresolved import `crate::Subscription`
 --> C:\Dev\iced\src\sandbox.rs:1:54
  |
1 | use crate::{Application, Command, Element, Settings, Subscription};
  |                                                      ^^^^^^^^^^^^ no `Subscription` in the root

Maybe there could be a wasm build in the GitHub actions to highlight these issues?

Given we are still exploring, I don't think we should currently focus on offering a fully compatible cross-platform API when it comes to widgets in master. We should be able to batch work on each side and avoid jumping back and forth too much.
However, I agree iced itself should always compile when targetting Wasm. I fixed this in #131 and added a check in the CI pipeline.

Right, it's actually iced that failed to compile on wasm, not necessarily an iced application. I agree about not having compatibility on master, that would be too limiting. Thanks for fixing and adding the check!


Add support for GitHub sign-in via Generic IDP

Approved with one small comment. This is great! Double-checking but this isn't breaking at all right?

Not breaking as far as I can tell :smile:


Type_clock Translation it_IT - Fixed filename extension

Type_clock Translation it_IT - Fixed filename extension

Already fixed . pls be careful next time

sorry and thanks :)


Couldn't see devices

I've installed the SonoffLAN add-on, devices appeared in the "Developer tools" but not in the Devices when creating automation.
Here are some strings from the configuration.yaml
sonoff:
    username: MyLogin
    password: MyPassword
    reload: always
    devices:
        switch.sonoff_10007b86a8_1:
          device_class:
            - light  # zone 1 (channel 1)
            - light  # zone 2 (channel 2)
          name: Sonoff 2CH Pow

What I'm missing? Thanks in advance.

#83


Git modified_files detect whole folder content changes

What did you do?
ℹ Please replace this with what you did.
Dangerfile:
git.modified_files.include?("some/folder/*")
Run danger local
What did you expect to happen?
Detect any changes in files inside certain folder, as the content of the folder is dynamic. Should there be any changes inside this folder, I want danger to detect and warn it
What happened instead?
It seems current function only support for a certain file such as build.gradle. Is there a way to achieve a folder level changes detection?
Your Environment

Which CI are you running on? Bitrise
Are you running the latest version of Danger? 7.0.1

include? is just a glob matcher across any files which git shows as changed.
If you're getting unexpected results, I'd look into either:

danger isn't picking up the git details right
git's modified files aren't showing what you expect


EVEREST-793: first Everest version

Deploy first Everest version in dev environment
EVEREST-793

Could you please elaborate why the recommended and supported sections are empty for 0.8.0 and what will be the normal process of adding a new version to the service?

They're empty because they're not needed in dev environment.
See dev vs. prod in https://github.com/Percona-Lab/percona-version-service?tab=readme-ov-file#how-to-publish-your-changes
It also describes the "normal" process of adding a version to the service.

Could you please elaborate why the recommended and supported sections are empty for 0.8.0 and what will be the normal process of adding a new version to the service?

They're empty because they're not needed in dev environment. See dev vs. prod in https://github.com/Percona-Lab/percona-version-service?tab=readme-ov-file#how-to-publish-your-changes
It also describes the "normal" process of adding a version to the service.

I'm still a bit confused, if they are not needed why do we need to have this file at all in dev? Can you please provide more details on how this will be used in dev vs prod?

The version service runs in dev and prod deployments. We can use the dev deployment for integration tests without having to update the prod deployment.
At the moment the integration tests are stuck because they cannot retrieve the latest version of Everest. By deploying this change I hope to resolve part of the issues with integration tests.

Ok so you don't need the versions of the components yet, you just need to have an everest version present. I'm expecting that you will need to add the component versions soon though. It doesn't seem to make much sense to have a version without the component versions but since this will unblock you for the rest of the work I don't see any issues.


[Extension] Can't log in to google account when using Floccus.

https://floccus.org/
Used to work with the old Kiwi browser but doesn't work since the chromium update. Hitting 'Login with google' after trying to setup a google drive sync does nothing.
Expected behavior: Login with google would open a browser window that lets you login, to successfully link accounts.
Here's an open issue with Floccus, where they pass the blame for Kiwi for not working, despite them listed Kiwi as one of their supported browsers:
https://github.com/floccusaddon/floccus/issues/921#issuecomment-886474011

I can't log in either

I have the same problem

It's not just Floccus - The google sign in does not work with in others extensions either like Tab Session Manager.
This does seem to be a bug in the Kiwi Next source code.

@kiwibrowser are you still aware of this issue? Was reported first on floccus' GitHub so wanted to make sure it's not lost.
Last comment in August was 'it will be fixed very soon'.

@kiwibrowser We all would be extremely grateful for the fix. The sad irony is that @d3ward just recently recommended Floccus as a substitute for the discontinued /sync project.
I've already seen people whith bookmarks tattoos, save the humanity!

Still a problem

Ya the issue still there :sweat:

Still a problem

U can try the old kiwi browser (kiwibrowser clasic) and then export account config

workaround: login into floccus on a desktop browser (using gdrive option), then export the floccus config, copy the config file to phone, then import the config in floccus. I've tried some month ago and it was working.


I can't configure the docker.io image repository

curl -sfLk https://get.k3s.io | INSTALL_K3S_VERSION=v1.28.9+k3s1 sh -s - --disable traefik --disable-default-registry-endpoint
centos7
vi /etc/rancher/k3s/registries.yaml
mirrors:
  "docker.io":
    endpoint:
      - "https://uph96ue9.mirror.aliyuncs.com",
      - "https://mirror.ccs.tencentyun.com",
      - "https://hub-mirror.c.163.com",
      - "https://mirror.baidubce.com"
  "ccr.ccs.tencentyun.com":
    endpoint:
      - "https://ccr.ccs.tencentyun.com"
  "registry.cn-hangzhou.aliyuncs.com":
    endpoint:
      - "https://registry.cn-hangzhou.aliyuncs.com"
configs:
  "registry.cn-hangzhou.aliyuncs.com":
    auth:
      username: ihezebin
      password: xxxx
  "ccr.ccs.tencentyun.com":
    auth:
      username:<PHONE_NUMBER>56
      password: xxxx

sudo systemctl restart k3s
cat /var/lib/rancher/k3s/agent/etc/containerd/config.toml not have any changes
[root@hezebin-k3s-master ~]# k get nodes
NAME                 STATUS     ROLES                  AGE     VERSION
hezebin-k3s-master   NotReady   control-plane,master   2m50s   v1.28.9+k3s1

but if remove the docker config, like this:
mirrors:
  "ccr.ccs.tencentyun.com":
    endpoint:
      - "https://ccr.ccs.tencentyun.com"
  "registry.cn-hangzhou.aliyuncs.com":
    endpoint:
      - "https://registry.cn-hangzhou.aliyuncs.com"
configs:
  "registry.cn-hangzhou.aliyuncs.com":
    auth:
      username: ihezebin
      password: xxxxxx
  "ccr.ccs.tencentyun.com":
    auth:
      username:<PHONE_NUMBER>56
      password: xxxx

What I want to achieve is something like docker's
sudo mkdir -p /etc/docker

sudo tee /etc/docker/daemon.json <<-'EOF'
{
  "registry-mirrors": [
    "https://uph96ue9.mirror.aliyuncs.com",
    "https://mirror.ccs.tencentyun.com",
    "https://hub-mirror.c.163.com",
    "https://mirror.baidubce.com"
  ],
  "exec-opts": ["native.cgroupdriver=systemd"]
}
EOF

sudo systemctl daemon-reload
sudo systemctl restart docker


Add support for adding images anchored to one cell

Addresses #738 #650 and is related to #702.
There is a bug in cell-position-xform.js that assumes that columns are 640,000 EMU wide and 180,000 EMU tall. This assumption is broken if you customise cell width or height, and the bug appears if you use fractional columns or rows to try to position your image.
This PR adds support for adding images using just a tl and then providing an ext that contains a width and height in pixels. The width and height are converted internally to EMU using a conversion at 96dpi. That seems to be the right thing to do, but something I'm a little unsure about!
This change should read as well as write the one-cell-anchor-xform, although I have only used writing so far but there is a test for reading and writing.

You can test it by adding https://github.com/karlvr/exceljs#integration to your package.json.

I just added it to my package.json and works like a charm. Is ext compulsory?

I think you need to provide either ext or use the existing method. I guess we could allow ext to be omitted if we just used the image width and height?

@karlvr definitely. If ext is not specified, we should use the original image's dimensions. One might get the image programmatically somewhere without knowing the exact width and height. Obviously there are libraries out there that help with finding images' sizes, but still would be nice if your PR does it out of the box

It sounds reasonable. I wonder if there is other image loading code in this library. Might well be!
On 5 Apr 2019, 12:03 AM +1300, Brian Le<EMAIL_ADDRESS>wrote:

@karlvr definitely. If ext is not specified, we should use the original image's dimensions. One might get the image programmatically somewhere without knowing the exact width and height. Obviously there are libraries out there that help with finding images' sizes, but still would be nice if your PR does it out of the box
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Looking into this now. Looks good, my only comment so far would be to offset tl by -1, -1 like the two-cell anchor. I'll look into this

Hi @karlvr , I have an issue using this functionality - exporting images looks great but they aren't getting deleted when I deleted a row from the Excel file. When I use use the default (tl: br:) the attribute editAs: 'oneCell' resolved this issue, but it doesn't seem to work when I use tl: ext: Did anyone else came across this problem?

If you set ext with height and width it works but you get the MS error and repair. I did a diff on the drawings xml of the original and repaired. No changes what-so-ever were made other than on the root xdr:oneCellAnchor where excelJS sets an attribute 'editAs="absolute"' or 'editAs="oneCell"', Microsoft wipes that out (no attribute at all). That is the only change. So very confused on what the heck we need to do to make this work. Should you not be setting this attribute? Is it defined elsewhere where MS is barfing?

如果您使用高度和宽度设置ext，它可以工作，但会出现MS错误并修复。我对原件的图纸xml进行了比较并修复了。除了在根xdr：oneCellAnchor上之外，没有进行任何更改，其中excelJS设置了一个属性“editAs=”absolute“或”editAs=“oneCell”“，Microsoft将其清除（根本没有属性）。这是唯一的变化。所以非常困惑我们需要做什么才能完成这项工作。您不应该设置此属性吗？它是否在 MS 所在的其他地方定义？

+1

If you set ext with height and width it works but you get the MS error and repair. I did a diff on the drawings xml of the original and repaired. No changes what-so-ever were made other than on the root xdr:oneCellAnchor where excelJS sets an attribute 'editAs="absolute"' or 'editAs="oneCell"', Microsoft wipes that out (no attribute at all). That is the only change. So very confused on what the heck we need to do to make this work. Should you not be setting this attribute? Is it defined elsewhere where MS is barfing?

+1


Pod issue

Your latest controller version (1.0.2) cannot be taken from pod.
Running pod install with pod 'SARate', I took version (1.0.1). Trying pod 'SARate', "~> 1.0.2", I had the following error:
[!] Unable to satisfy the following requirements:

SARate (~> 1.0.2) required by Podfile

Could you please update your pod project?
Thank you very much.
P.S.
Has promptForNewVersionIfUserRated property gone? Why?

Any comments about this?
Thank you.

+1

+1

No news yet?

Fixed. Sorry for delay.


Cart sampling index int64

trajectory is float32 which is not sufficient for accurate indexing of large datasets. Needs to be int64

good catch!


Query execution is not committed properly

Describe the issue:
The following query execution
INSERT INTO UM_UUID_DOMAIN_MAPPER ( UM_USER_ID , UM_DOMAIN_ID , UM_TENANT_ID ) VALUES ( ? , ( SELECT UM_DOMAIN_ID FROM UM_DOMAIN WHERE UM_DOMAIN_NAME = ? AND UM_TENANT_ID = - ? ) , ? )
is not committed due to this in large concurrencies we observe deadlocks. When adding defaultAutoCommit=true in user store pool options we can avoid this issue. But this is not recommended. This needs to be fixed from the code level.
[1]. https://github.com/wso2/carbon-kernel/blob/63ce7570a7c5be8fcaba291a59c9e26d0d82783e/core/org.wso2.carbon.user.core/src/main/java/org/wso2/carbon/user/core/common/UserUniqueIDDomainResolver.java#L194
How to reproduce:

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version:IS 5.10.0
Database: MySQL, H2
Userstore:  JDBC

Duplicate issue https://github.com/wso2/product-is/issues/8893


Speed up more tests with gem caching

Cache our bundle installs to S3 so we don't have to compile the gems
each time
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

@lbakerchef it this in combination with https://github.com/chef/chef-server/blob/master/.expeditor/verify.pipeline.yml#L3-L4 that makes it work.


Bump x/net to 0.23.0 to resolve CVE-2023-45288 for CNI plugin

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

/lgtm

/approve


Adding Linux commands

Hello sir, since Linux for Pirates is for learning about the Linux operating system then why doesn't it have anything about the linux commands? Don't you think it would be a good idea to include the linux commands in this repo as well? Please let me know if my suggestion is valid or not.

It does have some here, and the exercises already go over them quite extensively. We plan to cover them further and will be adding a lot more about Linux and the different commands you would expect to find.


M07-LAB17: formatting visual studio installation

Contact Details
<EMAIL_ADDRESS>What happened?
Exercise : Instructions
Step: Review applications required for this lab
Description of issue: the formatting of the selected workloads show still some Markdow. This is not the case in AZ400_M04_Feature_Flag_Management_with_LaunchDarkly_and_Azure_DevOps. So I synced them so they have the same content.
Lab
Lab 17 Package Management with Azure Artifacts
Relevant screenshots

Do you want to help us? 👏

[X] Create a Pull Request by following this project's Contribution Guide

it is related to how LODS renders it compared to GitHub Pages.
the actual syntax is needed for LODS

but it does what the issue mentions in GitHub Pages

it is related to how LODS renders it compared to GitHub Pages.
the actual syntax is needed for LODS
but it does what the issue mentions in GitHub Pages

Strange for me is why it looks ok for this one, since there the same is achieved: https://github.com/MicrosoftLearning/AZ400-DesigningandImplementingMicrosoftDevOpsSolutions/blob/master/Instructions/Labs/AZ400_M04_Feature_Flag_Management_with_LaunchDarkly_and_Azure_DevOps.md

I think the problem is LODS systems render each lab in a different way, they use some REGEX expressions for modifying instructions. We will need to sync with them @LuizMacedo

MOC-Labs-Review did not experience this Issue.

We need to sync with LODS to take a decision. Not critical as we may replace this lab

MOC Labs Review did not experience this issue


[REQUEST] ignore Response Source / Limit Response Source at specified Bytes

Many times, we don't need Resp. Source in our configs
is it possible to not download Response Source (not only hiding source from logs)
i think this ability was there in BB or an old V. of OB1
also it's needed on some targets
https://github.com/openbullet/openbullet/issues/491
and also it's good if we could limit getting Resp. source to specified Bytes

https://github.com/openbullet/OpenBullet2/issues/52

@openbullet
this issue is not duplicate of that
i mean not downloading the source, not only hiding it from debugger log
if i wanted to only hide debugger thing, i could use data.Logger.Enabled
But you said why we don't use HEAD method instead, when we don't need the SOURCE

Because a HEAD request does not accept post data.
HEAD may not be allowed [405]
Using HEAD instead of GET makes our requests detectable..

Okay fine I'll add it


Ensure we throw an error during startup when MVC services haven't been added to DI

This is the counterpart to https://github.com/aspnet/Routing/issues/276.

Dupe of #3907


Implement more robust modbus RTU answer checks

This implementation assumes correct implementation by Modbus RTU slave, only CRC is checked. If the answer contains less registers than the request, array out of bound issue happen.
This was not noted with real device but with simulated Modbus RTU answers.

Actually, it does not matter because the transport layer (Bluetooth LE) is already including CRC, so I will not improve it further unless real serial comms will be needed.


Add documentation on the strain code

https://tomographicimaging.github.io/iDVC/executable.html

not until we make it available!

I thought it was available in the executable


Swift 4 Support

Is there going be Swift 4 release anytime soon?

New release with Swift 4 support have just been submitted to CocoaPods https://cocoapods.org/pods/AuthorizeMe


Pass the whole object to the generate property of a slug field

Hey! Awesome product and tool. Really. Great job. I will consider donating a few coffees later on :)
As for the feature request, I think it would be nice if we could generate an entry's slug based on all its fields (language, for example). It should be fairly easy to implement.
Thanks!

+1 for overall more slug generation options like combining fields


vue-tsc: errors after update to version 0.31.0 or higher

Hi, after updating vue-tsc to version 0.31.0 i have following errors:
❯ npm run build

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

src/App.vue:8:1 - error TS1232: An import declaration can only be used in a namespace or module.

8 import { LibInput, LibHeadline } from 'vite-component-library';
  ~~~~~~

src/App.vue:8:39 - error TS2307: Cannot find module 'vite-component-library' or its corresponding type declarations.

8 import { LibInput, LibHeadline } from 'vite-component-library';

the same happens on version 0.31.1
Steps to reproduce:
git clone git@github.com:deathmaz/vue3-boilerplate.git
cd vue3-boilerplate
git checkout vue-tsc-update
./setup.sh
cd vite-project
npm run build

Last version that worked for me is 0.30.6
You can also switch to main (which has 0.29.8 version of vue-tsc) branch and run:
./setup.sh
cd vite-project
npm run build

There it should work without errors.
The only difference between main and vue-tsc-update branches is the version change of vue-tsc
Thank you for your work!

Hi @deathmaz, due to the limitation of volar, you need to move the import statement to the top in script setup. See https://github.com/johnsoncodehk/volar/issues/897#issuecomment-1021916449.
<script setup lang="ts">
import { defineAsyncComponent, ref } from 'vue';
+ import { LibInput, LibHeadline } from 'vite-component-library';
// This starter template is using Vue 3 <script setup> SFCs
// Check out https://v3.vuejs.org/api/sfc-script-setup.html#sfc-script-setup
const HelloWorld = defineAsyncComponent(
  () => import('./components/HelloWorld.vue')
);
- import { LibInput, LibHeadline } from 'vite-component-library';
const color = ref('red');
</script>


bump sentence-transformers

Bumping to v2.3.0

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

Comparison is base (3a39a88) 86.15% compared to head (a259819) 86.46%.

:exclamation: Current head a259819 differs from pull request most recent head 3fcc9b5. Consider uploading reports for the commit 3fcc9b5 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
+ Coverage   86.15%   86.46%   +0.31%
==========================================
  Files          27       27
  Lines        1242     1249       +7
==========================================
+ Hits         1070     1080      +10
+ Misses        172      169       -3

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


A continuation

Describe your problem here.
So basically, #2414 the code ain't working
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
I puted the code in gameoversubstate between line 74 & 77 with a empty line at 73

I have code for lua do u want it?

Yes and you didn't show me any errors or whatever in console.


Edit Custom monitor

This means adding a feature to be able to edit a custom monitor that was created by the user, just to be clearer


Optimize hot path in textBufferCellIterator

Summary of the Pull Request

References
The += operator is an extremely hot path under heavily output load. This PR aims to optimize its speed.

PR Checklist

[ ] Supports #10563
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

This PR only drops the time needed for "1 million yes" from ~8.0s to 7.0s, even 6.6s if it's in a good mood.
Before:

After:

Sheesh. I had thought the only way of improving this one was going to be a multi-pass operation where we pre-pass some of the data to identify things that were uncomplicated and then bulk fill them into the destination via memcpy instead of handling them like this one-by-one. Perhaps that's still in the future cards, but I'm glad to see this method improve.

I tested these changes out and they work very well. Almost unreasonably well.
This PR drops the time to print big.txt by over 30% down below 1s in WSL2. I mean... wow!
As much of a hack this code is I'm 100% in favor of merging it before we ultimately rewrite the console buffer anyways. I'm in favor of retaining _SetPos for operator-= as I believe that operator-= isn't used that much. Because if we do that, we can clearly mark the "hacky" code as being an "inlined" version of _SetPos, which should make the intent clear enough that this can be merged.

The ApiGetConsoleOriginalTitleA failure seems to be transient.

It's absolutely wild that you should say this today. On the same day, you hit a random ApiGetConsoleOriginalTitleA test failure and the OS build hit the same failure and filed a blocking bug on me.
The same day.
We traced it back to an issue in https://github.com/microsoft/terminal/pull/8621.

Now that you mentioned it, I do remember seeing the same failure several times both locally and on CI pipeline. What do you know. The failure loves me 😃
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: Dustin L. Howett @.>
发送时间: Tuesday, July 20, 2021 3:06:44 AM
收件人: microsoft/terminal @.>
抄送: Chester Liu @.>; Author @.>
主题: Re: [microsoft/terminal] Optimize hot path in textBufferCellIterator (#10621)
The ApiGetConsoleOriginalTitleA failure seems to be transient.
It's absolutely wild that you should say this today. On the same day, you hit a random ApiGetConsoleOriginalTitleA test failure and the OS build hit the same failure and filed a blocking bug on me.
The same day.
We traced it back to an issue in #8621https://github.com/microsoft/terminal/pull/8621.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/pull/10621#issuecomment-882787988, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABD6BL653VC3D5WPYIZGWQDTYRZUJANCNFSM5AF66GBA.


ip tables setup

To have some extra security, I added some iptables rules.  But haven't tested them yet.  And perhaps there are better ways to add extra security in iptables.  Would be nice if users could share some expertise here.

Long time since I used IPTABLES. 😊
Obviously, the general rules apply. Block everything inbound then only open what you know you need to. With most remote service tools, you don't need any firewall settings because you are running a local daemon service that reaches out. Default firewall configs usually allow this without changes. However, you could also consider blocking all outbound traffic as well except for things you know are needed.

Hi @Paul-Reed,
Thanks for the tip!  I also tried UFW while I was writing the tutorial, and it indeed is less cryptic compared to iptables.
However Tailscale also installs some iptables rules.  And UFW only shows the rules which you have added via UFW, but not the rules that have been added via iptables.  As a result UFW does not show all Tailscale related rules, which I found confusing.
So this why I added this "help wanted" issue, in case anybody with iptables knowledge could create some rules.  I think this is something that would be useful:

Wireguard traffic arriving on port 41641
Funnel traffic arriving on port 8443, which should only arrive from the ip range of the Tailscale Funnel relay servers (not searched yet which range that is).
Traffic on other ports (1880, 443, ...) should only be available from the localhost, i.e. requests forwarded by the Tailscale agent's reverse proxy.

That way all the local services on the Raspberry are only accessible via the Tailscale agent's reverse proxy, all over https with LetsEncrypt certificates.  And no direct access to them around the agent.

Ignore if you're already aware... But just found quite an active & knowledgeable Tailscale community on Reddit - https://www.reddit.com/r/Tailscale/
I just searched for iptables in the Tailscale context and it returned quite a few interesting posts https://www.reddit.com/r/Tailscale/search/?q=Iptables&cId=4e8e59fc-7231-402a-b2a3-4c1ca6a030c2&iId=b9935ef0-12f6-4550-862a-6ef28cb9c312


Error on mobile : Can't set headers after they are sent

Hello !
I'm using streaming-room on Windows 10 and I love it !
But I have some troubles with the mobile version and I can't find the origin of the problem.
7 times ouf of 10, the livestream won't load on the mobile version, the player show : "The media playback was aborted due to a corruption problem or because the media used features your browser did not support", and the NodeJS console show :
Error: Can't set headers after they are sent.     at SendStream.headersAlreadySent (C:\Users\illus\Desktop\streaming-room-master\node_modules\send\index.js:390:13)     at SendStream.send (C:\Users\illus\Desktop\streaming-room-master\node_modules\send\index.js:619:10)     at onstat (C:\Users\illus\Desktop\streaming-room-master\node_modules\send\index.js:732:10)     at FSReqWrap.oncomplete (fs.js:154:5)
I guess it's a problem with sending headers twice or something like that in the webserver part. Do you have any idea ? Thanks :)

Ok I figured out the problem is with the protection of the '/videos' directory.
When I delete   app.use('/videos', protect); from the webServer.js file, it works perfectly on mobile too, but the stream is now unprotected. Sooo it seems that on mobile the access to the playlist is denied, even if we're logged in.
Signed cookies not sending to the server at one point ?
🤔

On NodeJS the signed cookie "username" is "undefined" when I try to reach the stream on mobile. Any ideas why ? It's really annoying :/

Hey @illusdidi is your mobile device a Samsung?


Disabling a key not working (Remap a key)

ℹ Computer information

PowerToys version: 0.27.1
PowerToy Utility:
Running PowerToys as Admin: yes
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

… I disabled a key (semi colon) on Keyboard Manager, but I am still able to use it
…
…

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

screenshot of the settings please.  Right now this isn't enough to go off of.


Lost  page content when reload

Please take a look in this photo, when reload it lost all content in windows VS

I have the same issue. Was working great all day, until this.

@tnolan2 @huyjack178 could you please provide small sample?

Probably you had mistakes in xaml during designing?

I use prism and that page is in the modules of prism

@huyjack178 could you please create small sample project with this issue?
i will debug it :) 👍

@AndreiMisiukevich I did a little bit of debugging on this. In my case I only see this issue when I have a page with a converter inside my page resources.
<ContentPage.Resources>
<converters:NegateBooleanConverter x:Key="Inverter" />
</ContentPage.Resources>
I have also noticed when I have a toolbaritem in my page it will duplicate when saving. I am using prism on xamarin 4 preview. I can submit a sample later. I can probably swap most converters out for data triggers to alleviate the issue on my end.

I fixed the issue by wrapping the converters in a resourcedictionary. After looking at the xamarin docs it looks like they should be wrapped in a resource dictionary regardless.
Xamarin doc reference: https://docs.microsoft.com/en-us/xamarin/xamarin-forms/app-fundamentals/data-binding/converters
ContentPage.Resources
ResourceDictionary
converters:NegateBooleanConverter x:Key="Inverter"
ResourceDictionary
ContentPage.Resources

v 1.2.1


[ENH] Generate glossary page from schema

Closes #900.
This is just a first draft. It does not include links from the glossary back to the specification text, or even collect info about where the terms are used in the text.
I've included anchors for each of the object definitions, so it should be fairly easy to link to the appropriate place in the glossary from things like tables in the text.
Changes proposed:

Create new appendix, Glossary, with all schema objects rendered.
Replace single underscore used to delineate metadata fields with the same name, but different definitions, with two underscores, to match the convention proposed in #827.
Add descriptions for a couple of objects where that field was empty.
Add functions to generate the glossary from the schema.

Alternatively, I could organize them by type first, instead of including the type after the term, in parentheses. Not sure which would be preferable.

Had a quick look and am generally in favor of this, as it might help users in general.
Does that make the entities appendix page rerundant though?

I could organize them by type first, instead of including the type after the term, in parentheses. Not sure which would be preferable.

A way to dynamically sort / filter them by type or whatever other criterion would be the best but not sure how feasible it is in MkDocs and would have to be nuked for the PDF rendering.
I would start with alphabetical and progressively improve later.

Does that make the entities appendix page rerundant though?

I don't think this makes the "entities" appendix redundant - these are essentially different pieces of information, aren't they?

I would start with alphabetical and progressively improve later.

agreed!

nuked for the PDF rendering.

@tsalo could you please rebase this PR on master (or merge master) so that we can see how the rendered PDF looks like?
Thanks!

A way to dynamically sort / filter them by type or whatever other criterion would be the best

@Remi-Gau that's definitely in the wishlist, along with links back to sections of the specification that use each term.

I would start with alphabetical and progressively improve later.

Sounds good. I'll keep it as-is then.

@tsalo could you please rebase this PR on master (or merge master) so that we can see how the rendered PDF looks like?

@sappelhoff done!

Does that make the entities appendix page rerundant though?

I don't think this makes the "entities" appendix redundant - these are essentially different pieces of information, aren't they?

Well let's just say that the entities page is a subset of the glossary. See the example of the hemisphere entity below.

https://bids-specification--923.org.readthedocs.build/en/923/99-appendices/09-entities.html#hemi

https://bids-specification--923.org.readthedocs.build/en/923/99-appendices/14-glossary.html#hemisphere-entities

I don' think it is necessarily an issue though.

Eventually, it would be great to have the glossary replace the Entities appendix. Imagine hovering your mouse over a term in the specification and the glossary entry pops up! However, at the moment, I think the glossary is just too big and difficult to navigate for users to rely on it for something as common as checking entity definitions.

@sappelhoff would you rather I wait on @effigies's review or can we merge now that there are two approvals?

Let's merge :-) :rocket: Thanks @tsalo


Entry Expiry Status Not Cleared

Running KeePass 2.48.1 (64 bit, Portable) with v<IP_ADDRESS> of the plugin, I've tried to update a password of an expired entry using the 'Generate New Password...' option.
The password is changed successfully and the expiry date is set correctly.
The issue is that the entry is still shown as expired with a X over the icon and strikethrough font. To clear the expiration status, I had to open the entry and then press OK.
Linked to: https://github.com/ihanson/KeePass-Rule-Builder/issues/3#issue-729316428

This is now fixed in the Version 3.1 release. Thanks!


[v1.1.0 rc0][v1.0.2] MDB_BAD_VALSIZE: Unsupported size of key/DB name/data, or wrong DUPFIXED size

Describe the bug
I use thai language to search but it error "MDB_BAD_VALSIZE: Unsupported size of key/DB name/data, or wrong DUPFIXED size".
My dataset format is [{"product_id":xxxxx,"product_title":"yyyyyy"}] "yyyyyy" is thai language
My setting index is default setting
I can send simple data to you
To Reproduce

Import data
Use meilisearch dashboard
Open browser devtools
Search "วริ" , "น้"
See error

Expected behavior
Indexation process completes successfully.
Screenshots

Meilisearch version:
v1.0.2
v1.1.0 rc0
Additional context
Docker official image

Hey @Plong-Wasin 👋,
Thank you very much for this bug report. Could you please send us your dataset in this issue or via email to<EMAIL_ADDRESS>It is an internal issue that must not be thrown and be correctly handled by the engine.
Have a nice day!

Hey @Plong-Wasin,
I did receive your email with the dataset and was able to reproduce the bug.
I found out that the error originates from this line. I think the Charabia tokenizer is returning an empty string when it parses the input query and tries to search for it in the internal database, however, LMDB returns an error when we use an empty string to get a value. We will fix that in the next v1.1.0 release.
Thank you very much for your help!

Fixed by https://github.com/meilisearch/meilisearch/pull/3577
will be released in RC1 today 😇


ESP Lite Sleep support in Tickless mode

Enable the following:

ESP Tickless mode
ESP power management

Add two functionality to enter and leave the light sleep mode

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


Storing & querying nested resources

All Atomic Data Resources that we've discussed so far have a URL as a subject.
Unfortunately, creating unique and resolvable URLs can be a bother, and sometimes not necessary.
If you've worked with RDF, this is what Blank Nodes are used for.
In Atomic Data, we have something similar: Nested Resources.
Let's use a Nested Resource in the example from the previous section:
["https://example.com/john", "https://example.com/lastName", "McLovin"]
["https://example.com/john https://example.com/employer", "https://example.com/description", "The greatest company!"]

By combining two Subject URLs into a single string, we've created a nested resource.
The Subjet of the nested resource is https://example.com/john https://example.com/employer, including the spacebar.
So how should we deal with these in atomic_lib?
Approaches
Store nested in their parent as Enum
In both Db and Store, this would mean that we make a fundamental change to the internal model for storing data. In both, the entire store is a HashMap<String, Hashmap<String, String>>
We could change this to:
HashMap<String, Hashmap<String, StringOrHashmap>>, where StringOrHashMap is some Enum that is either a String or a hashmap. This will have a huge impact on the codebase, since the most used method (get_resource_string) changes. Don't think this is the way to go.
Store nested in parent as Value
An alternative is to not store the string representations, but store the Values in the store. Currently, all Atom values in the store are strings. We could changes this to store Values. Some performance implications:

Serializing the string representations (ad3) would be slower.
Serializing to non-string representations would be faster (e.g. JSON)
Using the data in some structured way would be faster.
Adding data to the store from highly-optimized serialized formats (AD3) would be slower.

This would also
Store as new entities, with path as subject
In this approach, the nested resources are stored like all other resources, except that the subject has two URLs with a spacebar. This has a couple of implications:

When deleting the original resource, all its nested ones will not be deleted (but should be), so this requires some extra logic
When iterating over all resources, we can no longer assume that every single Key (subject) is a valid URL.

Store inside parent resource, with path in Property URL
Similar to the approach above, but in this approach we use the Property URL to store nested paths. Implications:

Iterating over the Properties will not result valid Properties - these must be split up.
Finding some nested value needs a range query: select all properties that start with some string

Store all Atoms as BtreeMap<Path, Value>
Perhaps it makes sense to store all Atoms in something such as BtreeMap<Path, Value>, where the path is the subject followed by a property path (one or more property URLs). This should work by using BtreeMap's (and Sled's) range function to select all the right properties.
API design
And what should the API for the library user look like? Should a nested resource be a special Value? This seems sensible. However, in reality it is just a regular AtomicURL.
Serialization
Let's go over serialization by example. Let's assume a Resource of a person with some nested friends.
JSON
This is the easiest. Simply nest an object!
{
  "@id": "https://example.com/arthur",
  "name": "Arthur",
  "friends": [{
     "name": "John"
  }, {
    "name": "Suzan"
  }]
}

Note that these nested resources don't need to have an @id field, contrary to the root resource. Their identity is implicit.
AD3
JSON has nice nesting, but AD3 is originally designed to be very flat. If we use the Subject field to store paths, we get quite long subjects. This gets a bit awkward:
["https://example.com/arthur", "https://example.com/friends", ["https://example.com/arthur https://example.com/friends 0", "https://example.com/arthur https://example.com/friends 1"] ]
["https://example.com/arthur https://example.com/friends 0", "https://example.com/name", "John"]
["https://example.com/arthur https://example.com/friends 1", "https://example.com/name", "Suzy"]

The first Atom seems entirely redundant - it provides no more information than the second two. However, leaving it out might cause issues down the line: imagine if I'd GET https://example.com/arthur, but the first atom didn't exist. It would return no atoms - it would be empty. In order to prevent this, we could tweak the store a bit, so that a GET will search for all subjects that either are the URL, or start with the URL followed by a spacebar.
Another approach might be to nest triples in AD3, too:
["https://example.com/arthur", "https://example.com/friends", [[["https://example.com/name", "John"]],[["https://example.com/name", "Suzy"]]]

But this, too, is ugly and not human readable. JSON might be the way to go.

Perhaps it makes sense to start with the atomic_lib::Resource API - which does not support nested resources yet.
I think something like this makes sense:
let resource = Resource::new();
let nested_resource  = resource.new_nested('https://example.com/someProp');
nested_resource.set_prop_shortname('description', 'me is nested');

For now, I think I'm going to try storing Values instead of Strings in the store. This means refactoring quite a bit of code.

Currently, nested resources (at least in the Commit.rs impl) are serialized using Hashmap's auto conversion... Not nice.
This needs more thought. I think the AtomicURL datatype should be removed, and should be replaced by a Resource datatype, which can be either a URL or a Nested Resource.
The Nested Resource should be serialized as a JSON object in JSON, and perhaps in some other way in AD3.

Should I allow both Nested and URL resources in an array?
/// An individual Value in an Atom, represented as a native Rust enum.
#[derive(Clone, Debug, Serialize, Deserialize)]
pub enum Value {
    AtomicUrl(String),
    Date(String),
    Integer(isize),
    Markdown(String),
    ResourceArray(Vec<ResourceValue>),
    Slug(String),
    String(String),
    /// Unix Epoch datetime in milliseconds
    Timestamp(i64),
    NestedResource(PropVals),
    Boolean(bool),
    Unsupported(UnsupportedValue),
}

#[derive(Clone, Debug, Serialize, Deserialize)]
pub enum ResourceValue {
    NestedResource(PropVals),
    AtomicUrl(String),
}

Should I allow both Nested and URL resources in an array?
/// An individual Value in an Atom, represented as a native Rust enum.
#[derive(Clone, Debug, Serialize, Deserialize)]
pub enum Value {
    AtomicUrl(String),
    Date(String),
    Integer(isize),
    Markdown(String),
    ResourceArray(Vec<ResourceValue>),
    Slug(String),
    String(String),
    /// Unix Epoch datetime in milliseconds
    Timestamp(i64),
    NestedResource(PropVals),
    Boolean(bool),
    Unsupported(UnsupportedValue),
}

#[derive(Clone, Debug, Serialize, Deserialize)]
pub enum ResourceValue {
    NestedResource(PropVals),
    AtomicUrl(String),
}


Unable to withdraw liquidity

Describe the bug
Withdrawing liquidity is not working for multiple reasons. See
https://www.awesomescreenshot.com/video/13471955?key=06aebc9110b32964d90b4c80fba0a454

Clicking the % button shortcuts doesnt do anything
Typing in a number small enough (less than my amount pooled) works but doesnt allow me to 'Remove Liquidity', as the button is still unavailable
Typing in a number more than the liquidity I have throws this error

call revert exception; VM Exception while processing transaction: reverted with reason string "exceed total supply" [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ] (method="calculateRemoveSwapLiquidity(bytes32,uint256)", data="0x08c379a00000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000001365786365656420746f74616c20737570706c7900000000000000000000000000", errorArgs=["exceed total supply"], errorName="Error", errorSignature="Error(string)", reason="exceed total supply", code=CALL_EXCEPTION, version=abi/5.7.0)

We've fixed all the above issues. I was able to add and withdraw liquidity. Please try.

I'm getting this error on BSC. Is it due to https://github.com/connext/nxtp/issues/2814 ?

Due to #2859


Dashboard: Fix dashboard does not get icons from kubernetes cluster

Signed-off-by: Matias Bavera<EMAIL_ADDRESS>
run the example-deployment/get-icon.sh script to populate the  ws/rmf-web/packages/dashboard` with the icons.
build the dashboard's docker.

What's new

Self-checks

[ ] I'm familiar with and follow this Typescript guideline
[ ] I added unit-tests for new components
[ ] I tried testing edge cases
[ ] I tested the behavior of the components that interact with the backend, with an e2e test

Discussion

thank you for your help :bow:
i think there might be a few differences in how we are using rmf-web and the script,
could it be possible to use environment variable to specify paths for the rmf-web workspace? we put rmf-web in $HOME, not in ws.
In addition, we are using custom icons not available online, how do we modify the .resources file accordingly?

I believe the example deployment git clones the rmf sources as part of the instructions, so you can point to script to the rmf directories via relative paths.

i think there might be a few differences in how we are using rmf-web and the script,
could it be possible to use environment variable to specify paths for the rmf-web workspace? we put rmf-web in $HOME, not in ws.
In addition, we are using custom icons not available online, how do we modify the .resources file accordingly?

In that case, I don't think you should be using the example scripts, you can run the setup from the dashboard before building it as part of your deployment script.

I believe the example deployment git clones the rmf sources as part of the instructions, so you can point to script to the rmf directories via relative paths.

You mean by using the js "ament"?

In that case, I don't think you should be using the example scripts, you can run the setup from the dashboard before building it as part of your deployment script.

Yeah, sounds good. So there is no need for the get-icons-locally script then. So the deploy script should show the resources prompt then?
I'll create a new issue with this and let's merge just the icon fix then


Active registry roots

Possible replacement of latest_active_index_roots with latest_active_registry_roots to allow for significantly more efficient light clients (see item 17 in #1054). Key changes:

Validator information (including indices) is pre-shuffled and grouped by committee.
Validator information relevant to light clients (validator.effective_balance and validator.slashed) is kept "close" to the validator index to avoid accessing validator_registry.
Validator information (index, effective_balance, slashed) is compactified into a uint64 which is friendly to SSZ packing.
Shuffled pubkeys are available but kept separate from compact validator information. Light clients with pubkeys cached do not need to re-download and re-authenticate them.

"pre-shuffling": could this simplify the spec more? I.e. we compute a list of shuffled active indices after the randao (or effectively the shuffling seed) is updated, and then just have the data (committees, active indices) available when we need it? Clients can already optimize it themselves, but it may actually reduce the spec complexity (if we can have this index list somewhere around...).

"pre-shuffling": could this simplify the spec more?

We actually used to store the shuffling entirely in state, but after much debate removed it to (1) reduce state size and (2) give optionality to clients esp in resource restricted environments.

Close in favour of #1219.


Trying to parseSQL with character '-' returns empty rules array

Hi @jakeboone02,
i'm trying to parse expression with parseSQL and getting result with an empty rules array (a key with '-' character).
expression: "(somthing-somthing = '')"
result: {rules: Array(0), combinator: 'and'}
will appreciate your help..

Can you tell me what SQL dialect you're using? Normally identifiers like that have to be quoted with double quotes (e.g. Oracle) or backticks (e.g. MySQL).

Im using js with expression "(somthing-somthing = '')" which im trying to parse with parseSQL.
I tried also double quotes ("somthing-somthing" = '') and single quotes, but still getting an empty rules array.

Can you try it with backticks, like this?
parseSQL("(`test-test` = '')")

it works! thank you for the help

Great! Want to close this issue?


Audio input quality seems bad

I've noticed that the audio input quality is quite poor, which results in the AI struggling to understand what's being said. When using the transcribe function from OpenAI, I observe numerous transcription errors. These issues don't occur in the OpenAI Playground that often.
The errorrate is so high I don't think I can put this live, especially considering people are often not that clear as I am within the testing environment.
Possible Cause:
I suspect this might be related to the audio input format (g711_ulaw). The Playground uses PCM16, which seems to offer much better quality. However, the difference in transcription quality shouldn't be this significant, regardless of the format used I hope?
Request:
Could it be possible to review or adjust the audio input format to PCM16, or investigate why there's such a significant difference in transcription quality between the current implementation and the Playground?
Is there a way to increase input quality somehow?

I totally agree with you @brainvine. I've noticed the same issue, especially when dealing with local dialects. While he can somewhat handle English (not Good), it struggles significantly with dialects like Darija or other regional languages. The difference in transcription accuracy between the current implementation and the OpenAI Playground is very noticeable.

@pkamp3 Do you have any ideas? Can we work with PCM16 rather than G.711 u-law?

same issue here
also the ai voice is crackling a little bit now

Does this work for anyone? I can't achieve the same results as in the tutorial. Somehow, the audio quality is poor, which makes the Agent not understand what I'm saying. When I use direct integration with RealtimeAPI, everything works perfectly.

I don't fully understand what the issue is, as the problem doesn't seem clearly defined. Is the problem related to Twilio or OpenAI? I used Twilio with Azure's Speech-to-Text, and it worked quite well. However, when using Twilio with OpenAI, the results seem poor.
I suspect there may be an issue with the audio encoding between Twilio and OpenAI. Twilio might not be supporting OpenAI's default audio encoding format, or OpenAI may need to add broader support for more formats. Currently, we have to perform a transformation in between: we receive the audio data from Twilio, transform it, and then send it to OpenAI.

Hey all, I hear your concerns but we're limited by the channel and this is out of scope for this repo. G.711 A-law or u-law are what we need to use for calls on the PSTN, and that's what Media Streams supports.
Let me point you at a few ideas, though:

You might be able to make the conversion better yourself. As I'm writing this, OpenAI wants PCM16 24 kHz little-endian if you make the conversion with some optimizations, filters, or your other ideas.
Our Voice client supports OPUS, might work better for you – but it depends on whether your call crosses the PSTN. In this case, you'd still need to transcode before sending audio between the two WebSockets. My colleague Michael Carpenter wrote a bit about considerations here.


Port lasair-deploy from Heat --> Terraform

28/JUN/23
@RoyWilliams reported

the terraform port works
a test cluster was built during @genghisken 's visit to Edinbugh in w/b 19/JUN
this can now be  closed and the performance testing can start


Fix 500 error if there is a name conflict when edit authentication source

Codecov Report

Merging #23832 (fe95827) into main (f521e88) will decrease coverage by 0.14%.
The diff coverage is 28.82%.

@@            Coverage Diff             @@
##             main   #23832      +/-   ##
==========================================
- Coverage   47.14%   47.00%   -0.14%
==========================================
  Files        1149     1158       +9
  Lines      151446   153187    +1741
==========================================
+ Hits        71397    72006     +609
- Misses      71611    72678    +1067
- Partials     8438     8503      +65

Impacted Files
Coverage Δ

cmd/dump.go
0.66% <0.00%> (-0.01%)
:arrow_down:

cmd/mailer.go
0.00% <0.00%> (ø)

cmd/manager.go
0.00% <0.00%> (ø)

cmd/manager_logging.go
0.00% <0.00%> (ø)

cmd/migrate_storage.go
5.76% <0.00%> (-0.12%)
:arrow_down:

cmd/restore_repo.go
0.00% <0.00%> (ø)

cmd/web.go
0.00% <0.00%> (ø)

models/actions/run.go
1.63% <0.00%> (-0.10%)
:arrow_down:

models/actions/runner.go
1.44% <ø> (ø)

models/packages/package.go
45.45% <0.00%> (-1.13%)
:arrow_down:

... and 65 more

... and 66 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


feat: add more consensus metrics on rejected proposals

Description
This adds two counters to the consensus metrics:

ApplicationRejectedProposals
TimedOutProposals

It also adds a third StartHeight which is set once upon start so we know how many heights the metrics have been running for

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use
unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Do we want to backport v0.34.x-celestia?

Yup


Support Paging

Closes #250

Any progress on this PR?

@OliverRhyme Hey there - Sorry to be slow. No progress. This is not on our roadmap for Store5 stable. We will keep thinking about this. Completely open to contributions

Released earlier today!


Cannot install package: "the hash of the archive has changed since publication"

Thank you @fabhof for your work on Elm!
When installing I get the following error:
-- CORRUPT PACKAGE DATA --------------------------------------------------------

I downloaded the source code for fabhof/elm-ui-datepicker 4.0.0 from:

    https://github.com/fabhof/elm-ui-datepicker/zipball/4.0.0/

But it looks like the hash of the archive has changed since publication:

  Expected: a196df6cf7e31b523f53094e2c5fff870897bab8
    Actual: b73b6a1259666a106c78a4fb730232ea8facbd57

This usually means that the package author moved the version tag, so report it
to them and see if that is the issue. Folks on Elm slack can probably help as
well.

In order to reproduce:
mkdir elm-test
cd elm-test
elm init
elm install fabhof/elm-ui-datepicker

(I did rm -r ~/.elm in order to force the download)
I found mentions of this happening if the package author changes their GitHub username but this doesn't seem to be the case.
Also I found this issue from beginning of February that reports similar issues on other packages due to Github internal changes but this seems to be resolved by GitHub already: https://discourse.elm-lang.org/t/potential-breaking-change-in-github-zipballs-codec/8121/2. And my Elm project has around 30 dependencies and this is the only package that gives the error if I remove the ~/.elm folder and then run elm make.  So it doesn't seem like the above issue coming up again.
I'm at a loss here on what I can do/try on my side. Tested on 2 machines (Windows & Linux) and this happens on both of them.

That was interesting to find out. Try again please and let me know if it is still an issue for you.

@mackdunkan Still get it?

Works here now! Thanks for the fix, what did you do?

Thanks for fixing @FabHof


Switching between tabs "Tree View" and "XSL Output"

It seems that switching between tabs "Tree View" and "XSL Output" always transforms the output from scratch. For small files it is not important, but for monster files it takes a lot of time.
Maybe you can check if the existing output can be used or if it really needs to be recalculated?

Yeah, definitely could optimize that better.  Doing it or not is easy enough, based on whether you've modified or reloaded the XML document, but doing an incremental update is much harder since the xslt transform engine doesn't support such a concept.

The XSLT has to be checked for modification too, right?

From: Chris Lovett @.>
Sent: Wednesday, May 19, 2021 9:53:21 PM
To: microsoft/XmlNotepad @.>
Cc: Pierre-de-la-Verre @.>; Author @.>
Subject: Re: [microsoft/XmlNotepad] Switching between tabs "Tree View" and "XSL Output" (#92)
Yeah, definitely could optimize that better. Doing it or not is easy enough, based on whether you've modified or reloaded the XML document, but doing an incremental update is much harder since the xslt transform engine doesn't support such a concept.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/XmlNotepad/issues/92#issuecomment-844421438, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADFJV6HMUOGSRYCHG76XVE3TOQJLDANCNFSM45EPAECQ.

Right and that part is already implemented.  It checks the file last modified time and reloads the XSLT if necessary.

Closing for now since what can be optimized is already.

Do I see it right that the "... Button" to define the output does not invoke a "save as .." dialogue, but an "Open File " dialogue? Is it indented?


VSECM_LOG_LEVEL field has been edited to a default value of 3 for each location

Log Level States Update
Description
VSECM_LOG_LEVEL field has been edited to a default value of 3 for each location.
Checklist

[x] You have read the contributing guidelines and
especially the test policy.
[x] You have followed all the contributing guidelines for this project.
[x] You understand and agree that your contributions will be publicly available
under the project’s license.

Thank you for your reviews 🙏

Thanks for your contribution 🙏 .


Message Tracker - Multiple Results

Problem statement
As an internal admin, I would like the ability to search for multiple msgIDs to understand what happened to it and why it did not make it to the expected state
The purpose of this is to help with addressing support tickets.
What you need to know
Mural with initiative break-down https://app.mural.co/t/primedatainput6410/m/primedatainput6410/1660849046968/e257020850ab71f57b31120e27ec06203ae9c22b?sender=u3fea607b46e84058d3224917
Wireframe for the ongoing work on this feature
https://www.figma.com/file/fKl2M96ylkxYezwjVVUSIX/Internal-Admin-%26-Tools?node-id=411%3A1069
Acceptance criteria
Internal admin user is able to search for multiple  test results to get information about

success in being delivered or the reason why it was not delivered
all associated dates, facility, organization, report ID and other related data

To do

Design Wireframes, flow and IA
Scope out requirements from O&O for display and export of data

@HeatherRienks

@fmccaf1 - Please check with O&O as part of 6987 (usability testing) to see if this is needed. Thanks!

@CnoteNava @HeatherRienks @chris-kuryak Labeled stale as we are not sure whether this upgrade is needed. Usability research on the mvp will discover this need if it exists. However, being a plausible next iteration, I'll leave it open for us!

@kevinhaube Thanks for this comment! Really helps with the clarity for me.

Moving back to New Items as it is awaiting scope and priority from our results with the Single Message epic #6992

Unanswered questions from ticket 6953

What is the complete lifecycle of a ReportStream message? And, does a message's lifecycle always follow the same steps, or are there situations in which a lifecycle may differ from the norm?

Can we (are we technically able to) include all steps of the message's lifecycle in the message detail view of this tool?

Will "transport results" field be filled in when the tool is in prod? If not, what needs to happen to fill this field consistently?

Lastly, the link to report id was not functioning at the time of these tests. Once the bugs present during testing are addressed, it would be wise to invite O&O engineers back to look at these screens and provide feedback.

I think we can move forward with designs for now, and save thinking through how to represent the message at different stages of the lifecycle. Question for the team is whether I should move forward with answering these questions next sprint, or if we should save it further down the road.


[SPARK-24350][SQL] "array_position" error fix

What changes were proposed in this pull request?
Fixes ClassCastException in the array_position function - SPARK-24350
When calling array_position function with a wrong type of the 1st argument an AnalysisException should be thrown instead of ClassCastException
Example:
select array_position('foo', 'bar')

java.lang.ClassCastException: org.apache.spark.sql.types.StringType$ cannot be cast to org.apache.spark.sql.types.ArrayType
	at org.apache.spark.sql.catalyst.expressions.ArrayPosition.inputTypes(collectionOperations.scala:1398)
	at org.apache.spark.sql.catalyst.expressions.ExpectsInputTypes$class.checkInputDataTypes(ExpectsInputTypes.scala:44)
	at org.apache.spark.sql.catalyst.expressions.ArrayPosition.checkInputDataTypes(collectionOperations.scala:1401)
	at org.apache.spark.sql.catalyst.expressions.Expression.resolved$lzycompute(Expression.scala:168)
	at org.apache.spark.sql.catalyst.expressions.Expression.resolved(Expression.scala:168)
	at org.apache.spark.sql.catalyst.analysis.Analyzer$ResolveAliases$$anonfun$org$apache$spark$sql$catalyst$analysis$Analyzer$ResolveAliases$$assignAliases$1$$anonfun$apply$3.applyOrElse(Analyzer.scala:256)
	at org.apache.spark.sql.catalyst.analysis.Analyzer$ResolveAliases$$anonfun$org$apache$spark$sql$catalyst$analysis$Analyzer$ResolveAliases$$assignAliases$1$$anonfun$apply$3.applyOrElse(Analyzer.scala:252)
	at org.apache.spark.sql.catalyst.trees.TreeNode$$anonfun$transformUp$1.apply(TreeNode.scala:289)
	at org.apache.spark.sql.catalyst.trees.TreeNode$$anonfun$transformUp$1.apply(TreeNode.scala:289)
	at org.apache.spark.sql.catalyst.trees.CurrentOrigin$.withOrigin(TreeNode.scala:70)
	at org.apache.spark.sql.catalyst.trees.TreeNode.transformUp(TreeNode.scala:288)

How was this patch tested?
unit test

Can one of the admins verify this patch?

@ueshin can you trigger tests?

ok to test.

Test build #90990 has started for PR 21401 at commit 8e8f623.

Test build #90990 has finished for PR 21401 at commit 8e8f623.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90990/
Test FAILed.

retest this please

retest this please

Fixes ClassCastException in the array_position function - this seems like a better PR title than the current one.

Test build #91005 has finished for PR 21401 at commit 8e8f623.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91005/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

good catch, thanks

Test build #91015 has started for PR 21401 at commit 8e8f623.

Test build #91015 has finished for PR 21401 at commit 8e8f623.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91015/
Test FAILed.

Jenkins, retest this please.

LGTM

Test build #91020 has finished for PR 21401 at commit 8e8f623.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91020/
Test FAILed.

Test build #91045 has finished for PR 21401 at commit 8e8f623.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91045/
Test FAILed.

retest this please

LGTM

Test build #91058 has started for PR 21401 at commit 8e8f623.

Test build #91058 has finished for PR 21401 at commit 8e8f623.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91058/
Test PASSed.


Enhancement - Delay to blink

ENHANCEMENT
Would love to see delay added to the blinking of the target view.
Just so we can give more control to the component.

This is likely something that wont be supported, it seems to go against the guidelines. Sorry.


Data Table fails to render time as 00:00

If the time captured is 00:00, the dataTable does not display the time, it leaves it blank
Steps to reproduce the behavior:

Render this URL link: https://function-adminportal-test.shesha.dev/dynamic/Boxfusion.SheshaFunctionalTests.Common/Books-Table
Search for TestTime Picker 4
Check the time column
Click on the magnifying glass to drill down to details view
Check the time value
Actual Results: The dataTable does not display the 00:00 value on the table

Expected behavior
The exact value sent to the back should be rendered on the dataTable
NB: The display component used is 'default'. If you use a timePicker display component, the dataTable is able to render 00:00, however it breaks the other values

Related Item

#621
#533
#532

Hi @hlayisanimazuze. I've fixed it, please check


Apply java library plugin in tests, samples, javadoc

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test qfl without pts

I've triggered the following builds with parameters: -DenablePredictiveTestSelection=false for you. Click here to see all build failures.

QuickFeedbackLinux build

@bot-gradle help

Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN> [without PTS]
Add without PTS to run the build with full test coverage.

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle squash

Squash the current pull request into a single commit. The squash message will come from the pull request body, with:

All Signed-off-by: lines kept.
All authors being Co-Authored-By:

@bot-gradle merge

Enqueue this PR into GitHub merge queue for merging.

GitHub will create a merge commit from your PR branch HEAD and the target branch
A GitHub Merge Queue Check Pass build will be triggered on the merge commit
When the build passes, the target branch will be fast-forwarded to this merge commit (i.e. merge the PR)

@bot-gradle squash and merge

Squash the current pull request into a single commit as described in squash command above,
then enqueue this PR into GitHub merge queue as described in merge command above.

@bot-gradle cherrypick to <branch>

Cherrypicks the current PR to another branch.
A new PR will be created and a build will triggered automatically if there is no conflict.

@bot-gradle cancel

cancel a running build in GitHub merge queue and remove the PR from the queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test prf ACT

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build
AllCrossVersionTestsReadyForNightly build

@bot-gradle test prf ACT

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build
AllCrossVersionTestsReadyForNightly build

@bot-gradle test prf ACT

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build
AllCrossVersionTestsReadyForNightly build

Keep this