Style guide suggestion: Avoid duplicate links

https://github.com/python/cpython/pull/92199#discussion_r863506149:

It is not related to this PR, but if some item is mentioned multiple times in one paragrafph, I prefer to keep link only in one of them (usually at the first occurrence). :class:`!OpenerDirector` or ``OpenerDirector``. Too many duplicated links do not make the text more readable.

I agree with @serhiy-storchaka; this would be good to mention in the docs style guide.

+1.
I'd put this in the same conceptual group as introducing initalisms & acronymns once and then using the shorter form.
A

What is the scope for avoiding repeated links?
For long pages, it could be useful to use links in different sections.
The nature of webpages is you don't read the whole page, but skip to what is relevant, and may even arrive part way down at an anchor link.

I'd agree with @JelleZijlstra that, as a general rule, you should link to the same thing only once per paragraph. If the paragraph is long enough that you start wondering if you need to add another link, it's probably time to break up the paragraph.

What is the scope for avoiding repeated links?

Per paragraph I'd say, as with Alex / Serhiy / Jelle. There are edge cases (enumerations, single sentence paragraphs followed by a full paragraph), but any style guide is incomplete.
A

Like for AmE/BrE spelling, Wikipedia's MoS specifies not repeating wikilinks in close proximity. IIRC in that case it is per-section, but once per paragraph-ish seems fine, particularly for reference docs, with room for context-appropriate discretion (very short paragraphs, etc.).

I've thought a fair bit more about this and put it into practice various places; I typically do so per lowest-level section (or equivalent unit, e.g. class, function or method in API docs) since the reader may have jumped directly to the section via a link, or otherwise only consulting that specific section. In particular, readers of reference docs may only be reading at a high level of granularity and each unit (function, method, etc) should stand on its own, and readers of how-to guides are likely only reading the specific section covering their specific task, same with an individual explanation section, etc.
So long as one makes proper use of Sphinx's very powerful linking and cross-referencing capabilities (even between different Sphinx sites with intersphinx), it is very cheap to link/reference things, so the main constraint becomes oversaturating the reader with duplicate links as long as they would be expected to read both (e.g. same paragraph/section).

It’s not just duplicates. Sometimes just the sheer number of links in a paragraph can be distracting. (Making the links stand out less does not really answer this, it’s no use if there are links but you can’t see them.)

This recently came up in https://github.com/python/cpython/pull/94636#discussion_r933885060 too.

Having a link per paragraph sounds good to me, with some wiggle room  (e.g. sometimes it might be enough to have the link once in a 2-paragraphs-long function documentation).
Links to the module/class/method/function itself are generally not very useful, so those could be omitted in all cases.

However, if additional occurrences are not linked, they will render differently, potentially being more distracting.  For example:

with ``...`` and it will look like:

The type system of the sqlite3 module is extensible in two ways: [...] you can let the sqlite3 module convert SQLite types to Python types via converters.

with no markup it will look like:

The type system of the sqlite3 module is extensible in two ways: [...] you can let the sqlite3 module convert SQLite types to Python types via converters.

This recently came up in https://github.com/python/cpython/pull/94636#discussion_r933885060 too.

Just to clarify for others, more or less these guidelines (linking only on first usage within a unit, and not linking something we're inside of already) is what @erlend-aasland (and by extension, myself) have been following on the sqlite3 docs Diataxis-inspired revamp, and they seem to be working well.
If we're codifying this in the style guide, I would suggest making the general rule "one link per least readible unit", or whatever we want to call the minimum unit a reader would be expected to read, per the docs type. Generally speaking, this means the lowest-level addressable block, e.g. the lowest-level section in a How-To Guide, or per-function parameter in an API Reference.
This guideline is more flexible, generally applicable and tailored to the Diataxis type of the documentation while helping guide authors on how to judge this—plus, a lot of places don't have actual "paragraphs" at all. Furthermore, a more rigid application of the fixed "paragraph rule" could lead to overly high link density in some places (successive dependent paragraphs in the same lowest-level How-To section), while missing links in some places they are crucial (each parameter in an API Reference).

However, if additional occurrences are not linked, they will render differently, potentially being more distracting. For example:

I assume you're referring specifically to :py:-domain roles that are linked to their respective objects, given it is fairly well accepted to only hyperlink items in normal prose text within a certain unit, and unlike the former, is both more cognitively and mechanically expensive for authors.
I see two possibilities, both of them with benefits and tradeoffs:

Ref on first usage within a unit:
+ Reduces link density, and allows spending link budget where its more valuable than just repeating the same link
+ Makes it clearer to readers what is a new linked term within a unit, and which they've already seen
+ Only formatting change (under our current theme) is the link, which follows convention for prose text
- May look inconsistent to some readers
- May modestly increase cognitive load for the author to determine and remember when to use which

Ref on every usage:
+ Individual usages look consistent with each other
+ Easier for authors to remember and follow
- Not consistent with practice in prose text
- May lead to excessive link density for readers, or come at the expense of much more useful links

This isn't a super-strong preference, but while conceptually I like the idea of being consistent on every usage, given the practical tradeoffs for readers I favor linking first usage, mostly to reduce link density for readers and ensure link budget is spent on more reader-useful links, and since the formatting difference is just of the link (which provides meaningful information to readers) and consistent with how prose links are treated.

For me, this boils down to is this reference going to make life easier/better for the user[^1]?
IMO, I do not see how duplicate links and a high link density can possibly improve life for the user; I seems to me an anti-pattern. We should not add refs just because we can (i.e. there is something to link to => I need to link to it). We should only add refs if they improve the docs for the end user. IMO, reducing the link density and making sure that there is only unique refs/links within a paragraph is a great improvement for the end user. Consistency? Yes, making sure that there is only unique links within a paragraph is a schema; following a schema leads to consistency.
IMO, in the documentation for method x, we should not link to the documentation for method x[^2]; ditto for functions, classes, and even modules[^3].
[^1]: The same question can be applied to everything we do; is this feature going to make life better for the user? Is breaking this API going to make life better for the user? Etc.
[^2]: Again, is the link to method x when you are reading about method x going to improve the life of the end user? I don't think so. Actually, I think it might even be confusing or even irritating.
[^3]: How does linking to the sqlite3 module from within the sqlite3 module docs help the user? My answer: it does'tn; it only leads to the user clicking the link, losing the flow of what they were reading, realising that the link only linked to the top of the very page they were reading, then navigating back to whatever they were reading. Was that forward/backward dance needed for the user? No, it was totally unneeded. Did it improve the docs experience? No. Did it further enlighten the user? No. Did it cause frustration? Possibly. Did it cause irritation? Possibly.

Consistency? Yes, making sure that there is only unique links within a paragraph is a schema; following a schema leads to consistency.

Just to note, my argument for making the guidance "first use in atomic unit" follows the same basis as yours, and just takes them further—it is focused specifically on ensuring readers see one and only one link to a given target within each "atomic unit" of the docs (e.g. section, admonition, parameter description, etc), which:

better serves readers by aligning with the scope in which reader is highly likely to have seen the link, or at the very expect to find it (e.g. inside the lowest-level section they are presumed to have read in a guide, inside an admonition that they may be reading on its own, or an individual function parameter rather than an unrelated parameter they may not have read)
reduces link density to the minimum required by not repeating links any more than necessary (since paragraphs inside lowest-level guide sections generally presume having read the previous paragraphs anyway, there's no need to repeat a link in each one)
more consistently encompasses the different syntax and semantics used through docs and in different Diataxis categories, beyond just paragraphs, allowing for a more generally applicable rule

A possible counterargument is that the markup we are adding (e.g. :func:`...`) is there to add semantics to the element -- the fact that it also generates a link might be seen as secondary.  By removing the roles and using ``...``, we lose semantic information, even though practically speaking I'm not sure it really matters to anyone.
I wonder if, instead, it would be possible to use roles consistently (thus reducing the cognitive load of the writer, and making updates easier) and have Sphinx automatically add links only to the first occurrence within an atomic unit. This will also save us for rewriting all repeated references in all the documents.

I wonder if, instead, it would be possible to use roles consistently (thus reducing the cognitive load of the writer, and making updates easier) and have Sphinx automatically add links only to the first occurrence within an atomic unit. This will also save us for rewriting all repeated references in all the documents.

If that's possible, that would be wonderful!

Actually, from reading the devguide section on inline markup for #57 , I've come up with a better idea that retains all the advantages of "ref on first usage within an atomic unit" while keeping the semantic meaning and any distinct formatting of the role in question (aside from not being a link), and makes usage and conversion mentally and mechanically simpler. Per said section on cross referencing roles:

If you prefix the content with !, no reference/hyperlink will be created.

Therefore, instead of doing ..., we can simply do role:`!...` and it won't link the text but otherwise assign it the same doctree node type, and therefore the same formatting (though it also won't check them for correctness with -n, it seems, or elide any intersphinx prefix, as it doesn't attempt to actually resolve the reference at all—perhaps not ideal for catching typos, but conversely it means that its easy to add references that we know won't resolve just for formatting consistency, like we did in the What's New, without generating noisy Sphinx warnings).
The pros/cons:

I wonder if, instead, it would be possible to use roles consistently (thus reducing the cognitive load of the writer, and making updates easier) and have Sphinx automatically add links only to the first occurrence within an atomic unit. This will also save us for rewriting all repeated references in all the documents.

Possible? It should be; you'd add a

Actually, from reading the devguide section on inline markup for #57 , I've come up with a better idea. Per said section on cross referencing roles:

If you prefix the content with !, no reference/hyperlink will be created.

Therefore, instead of doing  ... , we can simply do role:`!...` and it won't link the text but otherwise assign it the same doctree node type, and therefore the same formatting (to note, it won't check them for correctness with -n, it seems, or elide any intersphinx prefix, as it doesn't attempt to actually resolve the reference at all—perhaps not ideal for catching typos, but conversely it means that its easy to add references that we know won't resolve just for formatting consistency, like we did in the What's New, without generating noisy Sphinx warnings).
This has a superset of the advantages of the  ...  on non-first-usage approach, while also ensuring syntactic, formatting and semantic consistency with linked usage and only requiring a one-character change to implement on existing text, minimizing any impact of inconsistency on both authors and readers. Therefore, this seems to be obviously preferably over either previously proposed alternative.
Also, the same approach can be used for references to an object (module, class, function, etc) inside itself, with the same benefits and essentially no downsides relative to either.
Source:
* If you prefix the content with ``!``, no reference/hyperlink will be created.

  For example, :class:`sqlite3.Connection`
  vs. :class:`!sqlite3.Connection`.

Result:

I wonder if, instead, it would be possible to use roles consistently (thus reducing the cognitive load of the writer, and making updates easier) and have Sphinx automatically add links only to the first occurrence within an atomic unit. This will also save us for rewriting all repeated references in all the documents.

It quite likely is possible; we could e.g. write a Transform that iterates through xref doctree nodes and either sets them to non-resolving or subs in the appropriate subclass of xref nodes to not resolve. However, it would be quite non-trivial, mostly due to having to programmatically define at the doctree level exactly what constitutes an "atomic unit"—which again, would be theoretically possible (since there should be enough semantic information to fairly reliably conclude this) but it would require handling a fair number of edge and corner cases due to the number of different node types and nesting, and if it gets one wrong there's no real way for an author to use their judgement and override this.
Also, particularly if substantial time is not spent on optimization, it could be quite expensive since it requires iterating through a large number of low-level nested nodes in every single document, perhaps multiple times; given that doc build times (given the large size and even larger build matrix) are already a serious concern, even a few percent slowdown could have significant implications.
So, while it possibly could be explored, IMO given it only requires adding one extra character (and the far more time-intensive part is properly linking things to begin with, which this could of course not solve) it might be better to just use manual discretion here, at least for now.

At least once I intentionally made the second occurrence a link, not the first one. Also, sometimes the unit is larger than paragraph, so only human can make a decision.

At least once I intentionally made the second occurrence a link, not the first one.

Indeed, very recently I did the same for some carefully thought out reasons

Also, sometimes the unit is larger than paragraph, so only human can make a decision.

There is theoretically enough semantic information in the doctree to determine appropriate unit size (hardcoding it to paragraph nodes would be a nonstarter as it wouldn't work in a large number of common cases), but as you mention there's still enough special cases this wouldn't handle, and wouldn't provide an easy means to override without even more complexity, that I don't think that aspect is worth it given it is easy to do manually following the newly-suggested above approach.

Using :role:`!...` seems a nice compromise, so I think we should just recommend that.  :role:`~...` could also be documented nearby as "Avoid repeated module/class names".

Can you combine the two modifiers?

* :meth:`Cursor.execute`
* :meth:`~Cursor.execute`
* :meth:`!Cursor.execute`
* :meth:`~!Cursor.execute`
* :meth:`!~Cursor.execute`

:role:`~...` could also be documented nearby as "Avoid repeated module/class names".

@ezio-melotti Its already the very next bullet point :)

Can you combine the two modifiers?

@merwok You could also just do :meth:`!execute`, which is simpler and less to type, but :meth:`~!Cursor.execute` is preferred from a semantic perspective, clearer in the source text, more consistent across usages and and will still work if you remove the !.

I know, that’s why I asked!  Great that the combo works, thanks Ezio for checking 🙂

Sorry, just wanted to clarify :)

Actually, :meth:`~!Cursor.execute` only partially works, and accidentally at that, as it makes Sphinx try to resolve !object.extend, which fails, so only the extend part is displayed due to ~ and it is unlinked due to the resolution failure. However, unlike a true !, the name is still looked up and attempted to be resolved, and it still generates a -n broken reference warning, so presumably :meth:`!execute` is actually preferable.
This could be changed in Sphinx by a few different methods (overriding create_non_xref_node in PyXrefMixin to run parse_reftarget first and use that as the title text, in order to support !~ (probably the best approach I see), modifying parse_reftarget and doing a separate check for ! in PyXrefMixin to support ~!, and possibly others.
However, none of them are trivial to implement and test, and would only come with a yet-unreleased Sphinx version, so I'm not sure if it is worth it for a case that could just be handled by manually omitting the undesired parts of the dotted name.
@AA-Turner , thoughts?

Tooling aside; I see a consensus here, would someone like to write it up for the https://devguide.python.org/documentation/style-guide/?

I've made a pull request for the devguide: https://github.com/python/devguide/pull/1294

In https://github.com/python/cpython/pull/117005#discussion_r1532502844, @AlexWaygood suggests using
:meth:`!__init__`

instead of
:meth:`~!object.__init__`

because it renders the same, but is less clutter in the .rst file.  Semantically, it's a link-suppressed reference to a non-existent method, but the readers' experience will be the same.  Thoughts?

Thoughts?

Never mind: I am now seeing that this has been discussed above, and CI fails with ~!, so I will switch to the shorter form.

I think we can close this now https://github.com/python/devguide/pull/1294 is merged, please re-open or comment if there's more to do.


easy_dna functionality port.

I had a great call the other month with the developer of easy_dna and he suggested that even though it isn't by far his most popular library it is his most useful and that poly should have similar functionality. Below is a list of functions that I will accept PRs for if ported. See above link for documentation and links to original source code. Please make sure they are well tested!
all_iupac_variants
anonymized_record
copy_and_paste_segment
cut_and_paste_segment
dna_pattern_to_regexpr
list_common_enzymes
random_dna_sequence
random_protein_sequence
replace_segment
reverse_segment
swap_segments
Thanks,
Tim
P.S some clarifying info that Davian asked for.
Most functions will likely fit into the scope of either sequence.go or transformations.go and their associated test files. For those that don't feed feel to make a utils.go and utils_test.go in the projects main directory to include with your pull request.
Most functions can be written as standalone string functions and then be wrapped by a method to use with poly's main sequence struct. That's as complex as integration with the main library will have to be in most cases.

@TimothyStiles this matches the sequence-level tools I was talking about. I can imagine another layer of abstraction that can utilize these functions for part-level tools that can take advantage of these functions.
If your roadmap includes the part level operations and being able to use both MoClo and Biobrick parts for different types of assembly. I think iGEM students will be able to use poly for the day-to-day.

I'd like to work on some of these! I can raise an issue for each function I decide to work on so that I can be assigned to it, if that would help out with tracking things.

Jeff, we'll do that as well

I can work on dna_pattern_to_regexpr!

I'll work on random_dna_sequence :)

Perfect! I'll start with all_iupac_variants. I'll raise an issue this evening.

I can work on list_common_enzymes!

Since all functions have been claimed by someone or another I'm going to close this issue. Please open a pull request and cite this issue to merge.


pandas requires installation from source on python 3.10

This makes the python 3.10 CI job take about 15 minutes longer than it should. We may want to switch to a conda-based stack where possible to obtain these dependencies more quickly.

It's because we have pandas pinned in guild/tests/requirements.txt to <1.2, and there are no wheels for newer python with the older version. We should try to remove that pin and fix whatever the new pandas might break.

I unpinned pandas and hit this error:
Exception raised:
    Traceback (most recent call last):
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/doctest.py", line 1346, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest None[0]>", line 2, in <module>
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/pandas/core/frame.py", line 3488, in __getitem__
        return self._slice(indexer, axis=0)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/pandas/core/generic.py", line 3929, in _slice
        result = result.__finalize__(self)
      File "/home/msarahan/mambaforge/envs/guildai/lib/python3.10/site-packages/pandas/core/generic.py", line 5549, in __finalize__
        self.flags.allows_duplicate_labels = other.flags.allows_duplicate_labels
    AttributeError: 'function' object has no attribute 'allows_duplicate_labels'

Seems like it will take some time to investigate and fix.


Produce a typings file for TypeScript users

We should produce a typings file for our externally facing APIs

Does not need to be tracked in regards to V2 GA


Ban IPs that request sus resources

I'm seeing IPs request things like /.ftpconfig and /.vscode/sftp.json. Look into banning IPs after an obviously malicious request.

List:
/.vscode/sftp.json
/ftpsync.settings
/.ftpconfig
/.remote-sync.json
/sftp-config.json
/.git/config
/.env
/.vscode/ftp-sync.json
/deployment-config.json

Rocket just server a 404 for these files, but it would be nice to just block these guys so they don't have the chance to find something.


Add a C++ program [word counter]

Description :
I want to add a C++ program in "Hacktoberfest2021_awesome-projects/Unique C++ Projects/" that counts the number of words in a sentence.
Use :
This C++ program takes a sentence as input from the user and stores it in the form of a String.
Then counts the number of words in considering each word is separated by a blank space (" ").
Concepts used:

Strings
Loop
Basic input and output.

Great description Arpit, but there is no need to raise an issue.
You can directly raise a PR and after going through the README.md file.

Thank you. I have opened a pull request for this program.


Enhancement: Add clientSideComponentId option to spo customaction get to be able to get by id

Add an extra option to spo customaction get to be able to get customactions by clientSideComponentId (if they are modern spfx extensions and have such an id)

Option
Description

-c, --clientSideComponentId [clientSideComponentId]
clientSideComponentId of the user custom action to retrieve information for. Specify either id, title or clientSideComponentId.

Examples
Return details about the user custom action a clientSideComponentId 3a607934-ebe3-4605-aedd-aae67710d8d8 located in site or site collection _https://contoso.sharepoint.com/sites/test_
m365 spo customaction get --clientSideComponentId 3a607934-ebe3-4605-aedd-aae67710d8d8 --webUrl https://contoso.sharepoint.com/sites/test --scope Web

@waldekmastykarz, I'm struggling with one thing though on this one.
the idea is that we can call spo customaction get from within (among others) spo applicationcustomizer get.
The point is though, when you retrieve by title you cannot be sure that spo customaction get returns a customaction that is an applicationcustomizer. You also don't want to force the user to choose the right id from a list of mixed customaction types.
That would suggest we'd better use spo customaction list --location '' like we're thinking about in #4381. But you'll be querying too many records there. I'm not sure thats really a problem, (as the amount of applicationcustomactions is always quite limited. But still, from performance considerations, I would not want to query the entire batch.
As a solution, we could implement a --filter option on spo customaction list, to make it broader than just filter by location. Just like we have in spo site list..
Whats your opinion on this?

If the underlying API supports filter, we should definitely use it to offer more flexibility. Let's not forget updating the underlying command's docs to reflect this change.

Thinking of it some more... filter is an advanced property that requires you to know the exact syntax. If there are common properties that we expect people to filter by, we could expose them as options so that users don't need to both with the proper syntax, escaping nested quotes, etc. There's no harm in doing both: exposing location and filter as options.

Thanks @waldekmastykarz, I'll rework it to expose them both 👍 that's the best solution across the board.

When looking at this issue, it seems that the filter for the UserCustomActions api ignores the clientSideComponentId eq 'guid'.
Is it the intention to first retrieve all custom actions here and then filter them in the code based on the clientSideComponentId?

If it's not possible to get a single one, then I'd say let's get all of them and filter client-side. It is what it is.

Thank you for your feedback! Can you assign me to this one? Would love to work this one out further

When looking at this issue, it seems that the filter for the UserCustomActions api ignores the clientSideComponentId eq 'guid'.

Hi @nicodecleyre, I had to try this for something else that I'm implementing, it turns out that you can in fact filter on clientSideComponentId, but as it's a guid value, you have to prepend the word guid:
This will work:
$filter=ClientSideComponentId eq guid'b41916e7-e69d-467f-b37f-ff8ecf8f99f2'
while this does not:
$filter=ClientSideComponentId eq 'b41916e7-e69d-467f-b37f-ff8ecf8f99f2'
Could you update your PR based on this new intel?

When looking at this issue, it seems that the filter for the UserCustomActions api ignores the clientSideComponentId eq 'guid'.

Hi @nicodecleyre, I had to try this for something else that I'm implementing, it turns out that you can in fact filter on clientSideComponentId, but as it's a guid value, you have to prepend the word guid:
This will work: $filter=ClientSideComponentId eq guid'b41916e7-e69d-467f-b37f-ff8ecf8f99f2'
while this does not: $filter=ClientSideComponentId eq 'b41916e7-e69d-467f-b37f-ff8ecf8f99f2'
Could you update your PR based on this new intel?

Hi @martinlingstuyl, sorry, somehow I missed your post regarding this issue. Thank you for your feedback! Tested it and works like a charm, so updated the PR! Thank you very much 💪


create overwrite middleware named "acceptOverwrite"

Hi @zhanghao-zhoushan,
Please create a middleware named "acceptOverwrite" and add unit test for it.
The behavior of the middleware:
When ctx.options.overwrite is truthy, set ctx.overwrite to true. When ctx.options.overwrite is falthy, do nothing.
Pass through the next callbacks. Do not block or terminate middleware passthrough process.
The test cases of the middleware:
when ctx.options.overwrite is truthy, alters context's overwrite value.
when ctx.options.overwrite is truthy, pass through middleware process.
when ctx.options.overwrite is falsy, do nothing.
when ctx.options.overwrite is falsy, pass through middleware process.

The implementation is similar to #18.

Nice Job @zhanghao-zhoushan!


Post Junior Developer Tips

This is my first blog post, so I'm hoping I formatted correctly. I wasn't sure how to do sub headings? Comment welcome.

This is my first blog post, so I'm hoping I formatted correctly. I wasn't sure how to do sub headings? Comment welcome.

@FionaDL I haven't reviewed it yet, but here you can see how the formatting looks:

Regarding markdown syntax, you can check the official documentation: https://www.markdownguide.org/cheat-sheet/


Missing footer translations

The French footer is missing a few translations.

Date modified (Date de modification)
Contact Information - Terms and conditions - Privacy (Contactez-nous / Avis / Confidentialité)
The button "Report a problem on this page" (Signaler un problème sur cette page)

#39 is a duplicate

@MrDeshaies The translations have been fixed. Also fixed their related links.


[Bug]: Not able to Compress grass Block

Minecraft Version
1.18
Mod Version
1.8.0
Is Compressium Loaded
Loaded
What happened?
Im using Minecraft 1.20.1 but can't select it here!
I'm not able to compress grass blocks into there compressed form ...

Instead you're able to compress grass instead ... Why ?

latest.log
No log

omg thats so bad of me its really common to miss one word, so instead of minecraft:grass i should have put minecraft:grass_block

All good.
Atleast it's a easy fix 😄


Taskbar icon vs Tray icon

On windows, the taskbar item can have a different status from the tray icon.
So we can have an mention in the tray, but it doesn't show in the taskbar. Maybe the taskbar is only reading data from the channel that is currently open for some reason?
I think it would be nice to have both based on the same data.

I have thought that they are based on same data(Tray icon doesn't indicate the number of mentions because of its size). Could you tell detail?

It's the same data, just had a look. At least with dev branch.
So it has to either be a bug or was updated since 1.2.1 which I run at work.

(Can't compile the dev branch at work, but this is with 1.2.1)
For replecation:

Login with account a and b
Write a DM with account a to account b
Account b's desktop app won't show that there is a mention, on the taskbar.

Hmm...I could get correct indication after the steps.

taskbar: Red (1) badge
tray icon: Red dot (means mention)

Windows 10 x64
Mattermost desktop: 1.2.1
Mattermost server: 3.0.0 (3.0.3)

Sorry, ignore me. I got a dual taskbar programm for windows 7 and I forgot, that one of the two taskbars just shows the app icon, nothing else. Sorry.


fixing last reboot in ks

fixing esxi 5.5 so it passes in jenkins continuous-os-installs
@geoff-reid

Build finished.

BUILD on-http #2645 : FAILURE

** BUILD build-config-setup  #122782: SUCCESS
** BUILD unit-tests  #9567: SUCCESS
** BUILD smoke-test  #3519: FAILURE


chore: use winston labels and topics for cleaner code

Hello is it possible to have your twitter or email for a question?

@mopmop682 Discord: Abady#5793


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/26222

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/34828

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/103125


draft custom CA steps for K8s deployment

Advances #4696. Arguably blocked by #42173.
Custom CA steps are working for authenticating and initializing the CRDB cluster and running SQL shell (steps 1-5) in the K8s deployment guide.
The Monitor step (6) still isn't compatible: Our current Prometheus and Alertmanager YAMLs only request certs from K8s.
I also ran into an ImagePullBackOff error with the Upgrade step (7), which may be unrelated.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/98c43470b531944d3a5730878adc00ed429751e9/
Edited pages:

v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/26a65a21cfee1a4a5cac27e44bb7efec340d3078/
Edited pages:

v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/80620397fddcb7c8c5a1f56bdbee32749d8105a8/
Edited pages:

v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/861b858048ea4c3a6ffe2fe47274e5f4e22c6562/
Edited pages:

v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/75775ea828751a320ecc9c392287b8e19ce58487/
Edited pages:

v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0499c3efee84ef8222d7f50e2b302573e90a8263/
Edited pages:

v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5569cc943aff01183fc83e54dc02b0adc6bb5ae8/
Edited pages:

v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bc18f42735d50b938f6b7508c80c719934c97dcb/
Edited pages:

v19.2/orchestrate-a-local-cluster-with-kubernetes-insecure.md
v19.2/orchestrate-a-local-cluster-with-kubernetes.md
v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-a-local-cluster-with-kubernetes-insecure.md
v20.1/orchestrate-a-local-cluster-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5bd256be57a7f320d4be1b17d38d7a6a01579bee/
Edited pages:

v19.2/orchestrate-a-local-cluster-with-kubernetes-insecure.md
v19.2/orchestrate-a-local-cluster-with-kubernetes.md
v19.2/orchestrate-cockroachdb-with-kubernetes.md
v20.1/orchestrate-a-local-cluster-with-kubernetes-insecure.md
v20.1/orchestrate-a-local-cluster-with-kubernetes.md
v20.1/orchestrate-cockroachdb-with-kubernetes.md


Setting scope for image reader in video_demo

Otherwise running it with mmyolo models triggers error, trying to find that module undery mmyolo scope

@RangiLyu could you take a look


Connects to #227. Reset all search filters.

Key Changes:

When a search submit event is invoked at the top-level, all search filters are reset so that previously selected filters are not persisted. This change applies to the DEA search and Gene-centric View.

The top level search submit now resets the previous filters.
I found another issue with reset filters: when a user clicks on the "Reset filters" button, ideally, the search results should reflect the changes. This can be accomplished by submitting the query in the query box without any filters to the backend API to fetch the results, alternatively, if you cache the results without filters, you could re-use it for the rendering.

@jyu-sm thanks for reviewing the PR and your comments. As we discussed, it's probably suitable to defer your recommendation of "Reset filters" button behavior to the next version.


Make new retriever for file testing from gin.

Actual implemention retrieve data with http for tesing IOs.
This is inherited by an old file repo from gnode before gin platform (with ftp deposit and http retrieve)
Because of that, we need to explicitly make a list of any file in folders that need to be downloaded, for each IO.
This is tedious and error prone.
two solution to avoid this:

make a index on al all files inside the gin repo. this would be update at each time we push and changes files.
use datalab (there is no gin python client) to retrieve lazily the repo for file testing.

I don't known work it is but I think the datalad solution is better no ?

Closed by #984


mapping option valued function yields list containing null

I have installed mono and f# via Xamarin Studio (stable channel).
$ fsharpi
F# Interactive for F# 4.0 (Open Source Edition)
Freely distributed under the Apache 2.0 Open Source License
For help type #help;;

(fun x -> if x then Some x else None) false;;
val it : bool option = None

List.map (fun x -> if x then Some x else None) [false];;
val it : bool option list = [null]

I believe that this is just an artefact of the compiler using null as the representation for None. The same happens in Visual Studio. For some discussion see https://stackoverflow.com/questions/11959321/why-is-none-represented-as-null

Yes, that's correct (and we'd love to see printing fixed to track types properly!)


Consider changing target SD script to the official Deforum script

Rather than having a separate SD script that does similar (but less / less mature) stuff compared to deforum, the UI could generate parameters for deforum (would require deforum script to have a mode that accepts a parseq-style JSON manifest)
https://github.com/deforum-art/deforum-for-automatic1111-webui

Exploring here: https://github.com/deforum-art/deforum-for-automatic1111-webui/issues/13#issuecomment-1294489498

Done in https://github.com/rewbs/sd-parseq/pull/21


Preprocessing the test set and the ID scores

Hello, thanks for the great work! We have several questions:
The paper states to have used 2824 CelebA-HQ images as the test set. However, 4 of them could not pass the pre-processing step; hence, 2820 of them were utilized while calculating metrics. We were trying to replicate the results for our data, but unfortunately, the pre-processing scripts you provided yielded 2808 in our case, not 2820. We were wondering if there is a problem with the pre-processing scripts. Maybe you could share the idx's of 2820 images you have utilized so we can double-check our dataset.
As a further result, we achieved an ID score of 0.79 with our data. Table 2 states that it should be 0.88, so we were wondering if it is a typo or due to the dataset.
Thanks in advance

Hi,
Regarding the pre-processing script there is a problem with the pre-processing script. We adapted the pre-processing script from EG3D but later found out that there was a problem. If you look at lines 27 and 28 in dataset_preprocessing/ffhq /crop_images_in_the_wild.py, the code sorts both image files and landmark files. However, if the face cannot be detected in some images and no landmark file is present, then this code will yield incorrect cropping of the images. Therefore, I would recommend fixing this before running the evaluation. Unfortunately, we found out this after the paper release. If you still want those incorrectly cropped images we used in our evaluation, I can share them via google drive.
Regarding the ID score, 0.79 seems correct, as we have specified in the "Major Changes" section in the readme.md that we drop normalization in the code. With normalization, 0.79 is equivalent to 0.89, which is in a similar range as reported in the paper.
Let me know if you have any further questions.

Hi again,Thanks for the quick response


test: llvm-cov

Proposal

Add coverage with llvm-cov

Links

https://github.com/taiki-e/cargo-llvm-cov

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4ad9eea). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage        ?   79.59%
=======================================
  Files           ?       14
  Lines           ?      147
  Branches        ?        0
=======================================
  Hits            ?      117
  Misses          ?       30
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


corrected package name

reported by a customer. the package name in public python packager is google-cloud-videointelligence
tested

@lukesneeringer I can confirm on PyPI and in the setup.py config.
It's also worth noting that the README isn't valid RST for PyPI (see the content on PyPI). This is why we use
python setup.py check --restructuredtext --strict

It is in the nox.py for the sub-package but we don't run it in our CircleCI config.
https://github.com/GoogleCloudPlatform/google-cloud-python/blob/9c48ace907bf60f581352ed66e570ba8dbce849e/.circleci/config.yml

@lukesneeringer What say you about my comment?


chore(demo): site fixes based on UX feedback

update demo site to address UX issues raised
update docs only as necessary to address above

fixes #269

@Foxandxss is fixing the build - typings didn't get installed.

Reverted change to dropdown....

@icfantv I don't think we are in a better shape with the proposed changes, sorry :-/
There are number of issues with the new proposal:

we shouldn't remove side-nav (it is fine to keep it in addition to top-nav)
we shouldn't have drop-down with Directives - just "Directives" link
not sure what the "Previous docs" link is supposed to do - is it for the future?
I think that we have misunderstood each other regarding GitHub buttons. What I was getting at is that the "Code on GitHub" button is too big (maybe we shouldn't have it at all?)

All in all I don't think we want to land it in the current form. As an alternative we can do 2 things:

you can send another proposal
we can have a hangout together and work interactively on this
I can send a proposal with changes and you can take it from there.

Let me know what works best for you.

@pkozlowski-opensource, @Foxandxss updated per meeting and comments.  using the multi-slot routing functionality was broached by Jesús but after seeing the URL syntax, I'm not so sure...
It would, however, buy us some DRY across files...

@icfantv unfortunately it doesn't merge cleanly after I've landed #259. Could you please rebsae?

Rebase it and I want to review it before we merge it.

@pkozlowski-opensource i'm on it.  @Foxandxss, will ping when updated.

@pkozlowski-opensource, @Foxandxss, ok, this is rebased w/ master and alert is merged.


Update known-issues.md

Docs Build status updates of commit 3b67983:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

support/mem/intune/known-issues.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off


Meaning of macros is compilation order dependent

Consider this Mirah code:
class Foo1
	macro def somemacro
		quote do
			puts "Foo1.somemacro"
		end
	end
end

class Bar
	attr_accessor foo:Foo1

	def somemethod(foo:Foo2)
		foo.somemacro
	end
end

class Foo2
	macro def somemacro
		quote do
			puts "Foo2.somemacro"
		end
	end
end

Bar.new.somemethod(Foo2.new)

The expected output of this code is
Foo2.somemacro

but the actual output is
Foo1.somemacro

Note that a reordered code
class Foo1
	macro def somemacro
		quote do
			puts "Foo1.somemacro"
		end
	end
end

class Foo2
	macro def somemacro
		quote do
			puts "Foo2.somemacro"
		end
	end
end

class Bar
	attr_accessor foo:Foo1

	def somemethod(foo:Foo2)
		foo.somemacro
	end
end

Bar.new.somemethod(Foo2.new)

produces the expected output.

Hmm. That seems like it's just a straight up bug. I mean it looks like it's expanding a macro based on the foo method rather than the foo local var.


teamspeak-server: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change
New teamspeak server version.
Things done
Could not test. Please validate.

cc @Shados

Just tested this out on unstable, appears to work fine.


Problem installing on FreeNAS 10.2 jails

I've followed the instructions here (https://github.com/nfarina/homebridge/wiki/FreeNAS-9.10-BSD-Jail) on a clean jail, downloaded from the linked template. But I'm consistent errors on using npm on the jail. When I try to 'npm install bignum' it always fails with an error message that says the current versions of node.js and npm are incompatible with each other.
Homebridge Log / Command Output
root@homebridge:/ # npm install bignum
npm WARN npm npm does not support Node.js v9.1.0
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 4, 6, 7, 8.
npm WARN npm You can find the latest version at https://nodejs.org/
WARNING: You are likely using a version of node-tar or npm that is incompatible with this version of Node.js.ode_modules/.staging
Please use either the version of npm that is bundled with Node.js, or a version of npm (> 5.5.1 or < 5.4.0) or node-tar (> 4.0.1) th
at is compatible with Node.js 9 and above.
/usr/local/bin/node[72959]: ../src/node_zlib.cc:437:static void node::<anonymous namespace>::ZCtx::Init(const FunctionCallbackInfo<v
8::Value> &): Assertion `args.Length() == 7 && "init(windowBits, level, memLevel, strategy, writeResult, writeCallback," " dictionar
y)"' failed.
 1: node::Abort(void) [/usr/local/bin/node]
 2: node::Assert(char const* const[4]*) [/usr/local/bin/node]
 3: _ZNSt3__16vectorIPN4node14SigintWatchdogENS_9allocatorIS3_EEE21__push_back_slow_pathIRKS3_EEvOT_ [/usr/local/bin/node]
 4: v8::internal::FunctionCallbackArguments::Call(void(*)(v8::FunctionCallbackInfo<v8::Value> const&)) [/usr/local/bin/node]
 5: v8::internal::Isolate*<v8::internal::Object> v8::internal::Builtins::InvokeApiFunction(v8::internal::Isolate*, bool, v8::interna
l::Handle<v8::internal::HeapObject>(int, v8::internal::Object*, v8::internal::HeapObject) [/usr/local/bin/node]
 6: v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*) [/usr/local/bin/node]
Abort

Install Node.js and npm via portsnap.
portsnap fetch
portsnap extract

/usr/ports/www/node
make install clean

/usr/ports/www/npm
make install clean

Seems like pkg install has an older version of Node.js.

When i try root@homebridge:/ # /usr/ports/www/node6      i get the error
/usr/ports/www/node6: Permission denied.


fix(css): headline-05 updated to correct value

Describe pull-request
Updated values for headline-05 to correct values according to figma.
Solving issue
In case of GitHub issue add # plus the number of the issue (for example #123) OR if it is Azure then AB# and number of ticket
Fixes: CDEP-3034
How to test

Go to storybook
Check so that headline-05 text is 16px instead of 14px

Suggested test steps

[x] Browser testing (Chrome, Safari, Firefox)
[x] Design/controls/props is aligned with other components

Additional context
We need to check that modal title still wants to have tds-headline-05 as class after this change.

Strangely enough it does not work. It still gets the 14 pixels... It does not build the css after my updates and I don't know why :(

I think this one is lacking: https://github.com/scania-digital-design-system/tegel/blob/fix/CDEP-3034-headline-05-wrong-size/typography/scania-fonts-vars.scss#L18

I think this one is lacking:
https://github.com/scania-digital-design-system/tegel/blob/3a04f3499bdd2e4a50a4b4d4ef452653bd1fbe39/typography/scania-fonts-vars.scss#L18
Also make sure to npm run build and npm run storybook. Sometimes it does not pick up changes.

I knew I was missing something!! What is the difference between the core one and /typography ones?
Thanks alot @timrombergjakobsson

I think this one is lacking:
https://github.com/scania-digital-design-system/tegel/blob/3a04f3499bdd2e4a50a4b4d4ef452653bd1fbe39/typography/scania-fonts-vars.scss#L18
Also make sure to npm run build and npm run storybook. Sometimes it does not pick up changes.

I knew I was missing something!! What is the difference between the core one and /typography ones? Thanks alot @timrombergjakobsson

Basically what I said on standup, typography is a separate package, structural wise. Although in the same package, when you do a download. In the old tegel package it was entire separate one https://www.npmjs.com/package/@scania/typography. Now bundled together in one.

I think this one is lacking:
https://github.com/scania-digital-design-system/tegel/blob/3a04f3499bdd2e4a50a4b4d4ef452653bd1fbe39/typography/scania-fonts-vars.scss#L18
Also make sure to npm run build and npm run storybook. Sometimes it does not pick up changes.

I knew I was missing something!! What is the difference between the core one and /typography ones? Thanks alot @timrombergjakobsson

Basically what I said on standup, typography is a separate package, structural wise. Although in the same package, when you do a download. In the old tegel package it was entire separate one https://www.npmjs.com/package/@scania/typography. Now bundled together in one.

makes total sense what I just said hahah

makes total sense what I just said hahah

Haha totally :) it works anyway :)


Cannot redeclare class with isCrossRef="true" and multiple foreign keys

In this example with isCrossRef="true" and multiple foreign keys the script fails with
Fatal error: Cannot redeclare Base\User::$collCinemas in /home/sandbox/generated-classes/Base/User.php on line 126

If isCrossRef="true" or the second foreign key created_by is removed the script runs without error.

Maybe this is related to #473?

Probably, but I'm not sure. Looks pretty much like a bug.

fixed?


Improve Combine Support

This PR makes Request a Publisher. This allows you to do the following:
let cancellable = Request {
  Url("https://jsonplaceholder.typicode.com/todos")
}
.sink(receiveCompletion: { ... }, receiveValue: { ... })

instead of:
Request {
  Url("https://jsonplaceholder.typicode.com/todos")
}
.onData { ... }
.call()

You can use Combine to apply all sorts of operations to the Request:
let cancellable = Request {
  Url("https://jsonplaceholder.typicode.com/todos")
}
.map(\.data)
.decode([Todo].self, decoder: JSONDecoder())
.sink(receiveCompletion: { ... }, receiveValue: { ... })

Or, even more concisely:
let cancellable = AnyRequest<[Todo]> {
  Url("https://jsonplaceholder.typicode.com/todos")
}
.objectPublisher
.sink(receiveCompletion: { ... }, receiveValue: { ... })

There are also stringPublisher and jsonPublisher, which publisher those respective types.
RequestGroup also conforms to Publisher:
let cancellable = RequestGroup {
  Request {
    Url("https://jsonplaceholder.typicode.com/todos")
  }
  Request {
    Url("https://jsonplaceholder.typicode.com/posts")
  }
  Request {
    Url("https://jsonplaceholder.typicode.com/todos/1")
  }
}
.sink(receiveCompletion: { ... }, receiveValue: { ... })

Codecov Report

Merging #34 into master will increase coverage by 2.41%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   85.69%   88.11%   +2.41%
==========================================
  Files          23       25       +2
  Lines         748      917     +169
==========================================
+ Hits          641      808     +167
- Misses        107      109       +2

Impacted Files
Coverage Δ

Sources/Request/Request/Extra/RequestChain.swift
100.00% <ø> (ø)

Sources/Request/Request/Request+Combine.swift
80.43% <80.43%> (ø)

Tests/RequestTests/RequestTests.swift
96.97% <95.71%> (+3.44%)
:arrow_up:

...s/Request/Request/Extra/RequestGroup+Combine.swift
100.00% <100.00%> (ø)

Sources/Request/Request/Extra/RequestGroup.swift
100.00% <100.00%> (ø)

Sources/Request/Request/Request.swift
93.10% <100.00%> (+0.05%)
:arrow_up:

Sources/Request/Request/RequestParams/Body.swift
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


[Doc-Enhancement] Markdown is not supported for SignIn card

Oauth signin card in bot doesn’t support markdown. So, please add note for the same.

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

ID: 001a12ea-30fa-d626-18a9-02a518bfb37f
Version Independent ID: e5f5f69e-66e9-d1b2-e19a-485d0ffcd6fa
Content: Text formatting in cards - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-format.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@SubbaReddi - Thanks for reporting your issue.
We will check this at our end and update the document accordingly.


Provide NeRF Eval Instructions

Update documentation to include NeRF eval command i.e. to populate NeRF sigma grids, required for training second phase semantic module (include instructions to install additional required packages, e.g. kubric).

Acknowledged. Working on it.

Clarification: Provide documentation for

Training NeRF models (in parallel)
Evaluating the quality of said NeRF models
Training NeSF semantic modules (requires pretrained NeRF models)
Evaluating NeSF semantic module


Develop Map Page as an alternative Home Screen. Show Followers, Following, Tweets of Following on the map

http://test.loklak.net/map Please add the following features:

Followers on the map (in one color)
Following (people) on the map (in another color)
Last 100 Tweets of following
New tweets pop up automatically (and we keep the last three tweets open. If a new tweets comes up the oldest tweets closes.)

Show an overlay box on the right, where users can show if they display 1-4.

Just a thought: I would like to see where the followers of Justin Bieber come from. There is usually a location tag in users profile.

am able to get the list of followers from the Twitter using hello.js api however the location in the response is the name of the place.To plot on the map I need the coordinates.I tired using Google Maps API to geocode the location but that's too slow since they put a cap of 5 results per second.Thus we need loklak to provide Geocoding Service.A issue has been created
Requires https://github.com/loklak/loklak_server/issues/62

the geocoding servlet is now available. See https://github.com/loklak/loklak_server/issues/62 for details.

Let's keep this issue open and keep info here to smoothen this feature and implement all details.

Yes

Let's experiment with the size of the image of followers/following. Can you reduce the size to 30x30 pixel, please?

Implemented with #313


Ignore packages.config files in File Anzalyzer

Fixes Issue
As a side effect of introducing the new Nugetconf Analyzer scans packages.config, the old File Analyzer started picking up all analyzed packages.config and adding them to the dependencies list. This behavior creates a lot of noise and doesn't carry any valuable information.
Description of Change
Added "packages.config" to the ignore list of the File Analyzer.
Have test cases been added to cover the new functionality?
no

Can one of the admins verify this patch?

Thank you for the PR!


test: enable mixed beacon-validator sim tests

Motivation
At the moment mixed client tests are disabled because the assumption was that those will only work once https://github.com/ChainSafe/lodestar/issues/5553 is fixed but that is not the case as the issue is not an actual problem which would prevent Lodestar BN <> Lighthouse VC combinations from producing attestations and aggregates.
Description

Enable mixed client tests added in https://github.com/ChainSafe/lodestar/pull/5885
Fix host resolution issue
Rename from "multi client" to "mixed client"

Further considerations
We could update sim tests to run all containers with --network host flag and not create a custom docker network

host.docker.internal workaround could be removed
less isolation of running process, potential port collisions

@nazarhussain are there any other reasons / trade offs for using a custom docker network instead of host network?

@nflaig Using custom network allows us to set a submit and then we can assign any IP to our services in that range from code. There is no drawback I see using it.

There is no drawback I see using it

Yeah, the only drawback is that we have to use host.docker.internal but that's not really a big deal, should work on all platforms now.

:tada: This PR is included in v1.12.0 :tada:


jshint errors for date functions: 1 fixed, 1 ignored.

AQL_DATE_LEAPYEAR() slightly rewritten to avoid "Confusing use of '!'". It passes your tests.

Are all 3 comments needed to disable line length check?
Or does either work?
Does // ignore maxlen jshint check affect the subsequent line only?

Oh and how do you want to be mentioned as contributor? Github user name, actual name, ?

This line does absolutely nothing, and could be removed
//ignore maxlen jshint check

While this
//jshint -W130`

turns off the maxlen validation until
//jshint +W130`

turns it back on.
They've done similar exceptions earlier, so I think they'll approve.
friday will do :)

Should I close this and remove //ignore maxlen jshint check?

I made the changed locally, you can kick back and wait for the update :)
BTW: if you add .patch to a github PR url, then you get the diff:
https://patch-diff.githubusercontent.com/raw/CoDEmanX/ArangoDB/pull/3.patch

Cool :)


fix(i3): Reconnect i3 IPC socket on restart/error (#762)

This fixes issue #762 by reconnecting the socket if an error occurs.

Works fine for me too. Thanks!


added adapters for DSPy

-adapters for general LMs (DavinciAdapter), chat LMs (TurboAdapter) and Llama models (LlamaAdapter)

just a reminder to revoke that key if you haven't already!

Hi @arnavsinghvi11, this looks pretty neat as it would allow a lot of flexibility for prompting for different models (or even the same model but using different ways to state your instructions). I wonder what is holding you back from merging it?
If there are issues regarding merging the entire thing, then would you consider a more immediate but less drastic change:
As signature_to_template() in
https://github.com/stanfordnlp/dspy/blob/d8b8909773fc31e72cec093db2f26109590e524e/dspy/predict/predict.py#L137-L140
has an optional adapter parameter, which actually takes a template as opposed your adapter, I propose that we can add a setting to change the default template, so that the behavior of the template could still be customized to some extent (albeit a bit less modularized).
Additionally, I still see a bit of inflexibility with TemplateV2.guidelines(), which could be addressed in the future, though. I believe that adapters should also have the ability to customize the format of the guidelines.


Top-align text within attributed string

If I have text of different sizes in an attributed string, is it possible to top align them? Or do I have to create the labels and top align them with BONTextAlignmentConstraint?
Here is an image for reference:

Currently, your best bet is going to be do tweak the baseline offset. It would be cool to have an automatic way to do this within a label, though. I wonder if there's a way to use the metrics of the font to calculate the appropriate baseline shift? Otherwise, @mikewdeasy, your instinct to use BONTextAlignmentConstraint sounds like the right way to do this. Beware: this will affect the VoiceOver accessibility, because you'll end up with three separate labels. If that's something you care about, you might want to do something special in that case.


skewness formula precedence error

return Math.sqrt( N*N-1)*g / (N-2);
should be
return Math.sqrt( N*(N-1))*g / (N-2);
so that (N*N) -1 becomes N*(N-1)
(I am basing this on comparison to results from Excel and Python Pandas library)

@jlp765 Thanks for finding this bug. Will patch later today.


Small Phi Typo

Hey!
Thanks a lot for these notes, they are awesome and are helping me a lot. I found a small error in the Simple Monte Carlo part. I think it was a typo but let me know if it's not correct! I checked the book and seems that estimator for Phi is the one that we want not the function phi.

Glad the notes are useful 😄 You are right, that is a typo. Thanks for catching that!


Fix blog-wide account when WordPress is in subdirectory

Fixes #741
Proposed changes:

Other information:

[ ] Have you written new tests for your changes, if applicable?

Testing instructions:

Go to '..'

@edent Can you verify that it works?

@pfefferle seems to work. If you try to follow<EMAIL_ADDRESS>it responds appropriately.


Remove version: Amazon.Chime version 4.38.10186.1

Reason

Installer unavailable. See https://github.com/microsoft/winget-pkgs/issues/171560#issue-2505208830

Resolves #171560

Microsoft Reviewers: Open in CodeFlow

URL: https://clients.chime.aws/win/releases/Chime.4.38.10186.exe
Status Code: 200
(Automated message - build 894)


Using PSGallery to Install Pester in AppVeyor File

This is for PowerShell/DscResources#147.

This change is

PSSA errors and missing newlines failing the tests have been submitted under #15.


[DON'T MERGE] VerifySigner() now uses CredentialPojo as extra param

Summary

Concisely describe the issue and your solution:
Link to existing issue (type "#" to check):
Specify the tests done so far:

Solutions (Optional)
Please describe the details of your solution if it might be too complicated for reviewers.

Codecov Report

Merging #219 into develop will decrease coverage by 0.03%.
The diff coverage is 32.00%.

@@              Coverage Diff              @@
##             develop     #219      +/-   ##
=============================================
- Coverage      39.69%   39.66%   -0.04%
- Complexity      1506     1507       +1
=============================================
  Files            232      232
  Lines          12862    12879      +17
  Branches        1603     1609       +6
=============================================
+ Hits            5106     5108       +2
- Misses          7121     7135      +14
- Partials         635      636       +1

Impacted Files
Coverage Δ
Complexity Δ

.../webank/weid/service/impl/EvidenceServiceImpl.java
30.15% <16.66%> (-1.34%)
26.00 <1.00> (+1.00)
:arrow_down:

...k/weid/service/impl/CredentialPojoServiceImpl.java
44.34% <33.33%> (-0.20%)
133.00 <0.00> (-2.00)

...java/com/webank/weid/util/CredentialPojoUtils.java
51.62% <100.00%> (+0.42%)
105.00 <2.00> (+2.00)

Continue to review full report at Codecov.

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


"[Kingfisher] Failed to create CG context for blurring image."

I am getting this assertion causing my app to crash line 479 of Image.swift. In log the following error was also visible before the assertion:
CGBitmapContextCreate: unsupported parameter combination: set CGBITMAP_CONTEXT_LOG_ERRORS environmental variable to see the details
I have deleted the assertion and though I am not getting any blur, at least the app no longer crashes.

Hey.
It might be caused by some strange format of that image. May I know the image url (or the image itself) crashes your app? So I could have a look at what happen indeed.
Thanks.

Sure here is the image url: https://s3.amazonaws.com/winering-production/10906-313f1f875b62f3a-full

Should be fixed in #567.
There are some other work to do before a new version could be released. I will tag it later ASAP.

@onevcat Tomorrow I migrate Kingfisher to 5.0, after I received this bug.

Hello, I'm getting this issue on last Kingfisher 5.1.0

Same issue in here , while scrolling it getting crashed

@anhhtbk @salyasev @Eldhopj May I know what image causes this problem? Is there a link or an image I can use to reproduce this issue?

I am facing the same issue while scrolling. However it only crashed when I use

imageView?.kf.setImage(
            with: viewModel.thumbnailURL,
            placeholder: nil,
            options: [
                .processor(DownsamplingImageProcessor(size: imageView!.bounds.size)),
                .scaleFactor(UIScreen.main.scale),
                .cacheOriginalImage
            ])

@onevcat
crash happens when the imageview size is zero also

@seanliu1 Not quite sure about it. The image itself seems fine.
Is there any crash log for it?
@Eldhopj The crash of DownsamplingImageProcessor for zero or negative size should be already fixed in #1074, can you confirm that you are using the latest version?

This problem also occurs when using ResizingImageProcessor with a referenceSize of CGSize.zero

The ResizingImageProcessor should be failing with an assertionFailure, which is intended as a warning of incorrect usage of this API. It behaves as a reminder when developing, and will not crash in a release build.

I received this bug in Kingfisher 6.0 with follow images:


Eventually consistent writes

I wrote an eventually consistent LRU earlier this week by leveraging crossbeam with evmap. This concept can be easily adapted for non-LRU use as to facilitate concurrent eventual writes with a version tailored for bounded use via crossbeam::queue:ArrayQueue and an unbounded version via crossbeam::queue::SegQueue. If you want this as part of evmap, you can run with the idea or I can push up a PR and we collaborate, otherwise I'll release this as a separate crate.

Hi there! I'd generally recommend keeping anything built on top of left-right as separate crates. If there's anything I've found with evmap, it's that concurrent datastructures tend to work best if they're tailored to a fairly specific use-case, and aiming for generality tends to sacrifice either performance or ergonomics. Nice that you found a good way to do it though, and thanks for checking!


[Proposition] SpaceX

Make another one about the latest SpaceX Dragon mission :rocket: :smile:

Hmm, i will think about it 🛰  👨🚀


Roll engine 98487198ba65..e96900df2f27 (3 commits)

https://github.com/flutter/engine/compare/98487198ba65...e96900df2f27
git log 98487198ba65cc7c4031d1ed3effe1ec915e567a..e96900df2f273920f7da46ae589b859d5a1fccb3 --no-merges --oneline
e96900df2 Roll src/third_party/skia afd2c10c989c..e4c88bb1f848 (1 commits) (flutter/engine#9577)
618b193fb Roll fuchsia/sdk/core/linux-amd64 from hKK9H_q6Wr4qc_LULh6yowFyaqv8Ha-3JsuE05J4tpwC to F9oY0d56X3aONxwv5vw9Gco--AHmC8SHuDXdhdWCnnMC (flutter/engine#9576)
2a848c505 Roll fuchsia/sdk/core/mac-amd64 from jofgNhNmp-u51vpEXxTPuR89aL5DbLb-KNyK_Whh-ZAC to cFEtOUFvy2XYSKuSmCMDaMQV8_Y7GOif521L3x86J_4C (flutter/engine#9575)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (chinmaygarde@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Add support for O365 instrumentation headers

client-request-id and return-client-request-id

Closing as no longer pertinent.


Cargo packages duplicate files on case-insensitive file systems

Problem
It seems that Cargo is excluding an already-packaged files using exact name comparison, which doesn't always match how the file system sees name equality.
   Archiving Cargo.lock
   Archiving Cargo.toml
   Archiving Cargo.toml.orig
   Archiving README.md
   Archiving readme.Md
   Archiving src/main.rs

Example crate:
https://docs.rs/crate/rosu/0.6.0/source/
Steps
[package]
…
readme = "README.md"

echo case > readme.Md
cargo package

The same applies to license-file, cargo.lock.
Possible Solution(s)
Theoretically there could be other gotchas of this kind, e.g. HFS+ file system on macOS forces file names to use NFD Unicode form, while most text has NFC form, which makes codepoint-by-codepoint comparisons not equal. However HFS+ is on its way out, so perhaps a simple case-insensitive comparison will suffice.
Notes
No response
Version
cargo 1.79.0-nightly (499a61ce7 2024-03-26)

In the same vein, if there's a TARGET/ directory, it doesn't get excluded when packaging.
Caused by:
  Source directory was modified by build.rs during cargo publish. Build scripts should not modify anything outside of OUT_DIR.
  Added: /private/tmp/bla/target/package/testx-0.0.0/TARGET/.rustc_info.json
  	/private/tmp/bla/target/package/testx-0.0.0/TARGET/debug
  	/private/tmp/bla/target/package/testx-0.0.0/TARGET/debug/.cargo-lock
  	/private/tmp/bla/target/package/testx-0.0.0/TARGET/debug/.fingerprint

@rustbot label Command-package

so perhaps a simple case-insensitive comparison will suffice

While that sounds lovely, in what locale? For the languages I speak it is relatively straight forward, but my understanding is that case handling is lossy in some languages, such as German (ẞ is Ss in upper case I think?) and Turkish (I believe they have the letter "i" both with and without a dot, and the uppper/lower case there isn't straight forward, but don't ask me how exactly).
As a Swedish/English speaker this is all hearsay though, and I don't know how e.g. Windows or Mac OS handle these, though I think I heard that NTFS store a case normalisation table at creation time based on the locale set at that point?

On Windows, the NTFS up case table is initialized when the drive is first formatted. So it'll depend on the Windows version that did that. It is however language neutral and only acts on the Basic Multilingual Plane.
Also, depending on the configuration, NTFS can be case sensitive. In Windows this can even be set differently for each directory.

Ah yes FAT32 is indeed a mess. But then I'm also not sure how well Cargo and rustc support it as it lacks a lot of filesystem features that may be expected. Probably it does at least work if it's only read from (e.g. the target directory is on another drive).

While that sounds lovely, in what locale?

It is a messy problem, but fortunately the detection algorithm doesn't need to produce user-facing text, so it doesn't need to be perfect from linguistic perspective. It only needs to detect potential collisions between file names. Crates that work with only a specific combination of Windows locale and NTFS vintage are not generally useful, so the detection can also err on the side of over-normalizing (e.g. normalize all dotless ı's to i, check against both lower and upper case, treat codepoints with multiple transliterations/decompositions as a wildcard, etc.).
but for a start, even a simple .lowercase() will handle more than enough for the accidental variations of Readme.Md.


Allow use of captured regex group in files-to-include and files-to-exclude fields of global search

In global search pane, one can use regex to search for something. You can also use the captured group in the "replace by" field. However, currently it is not possible to use these captured groups in the "files to include" and "files to exclude" fields.
Example use case:
I want to be able to edit all #include lines in different modules so that:
// file: module1/file1.h
#include "moduel1/file2.h" // want to replace with #include "file2.h"
// file: module2/file1.h
#include "module1/file2.h // don't want to replace this

Sorry I don't understand why you wouldn't set "files to include" to "module1" or something, can you give an example of how this would work? It seems like you are asking for something self-referential becuase you have to decide which files to search in before you can search in them.

Hi roblourens thanks for your response. Because it can be module1 or any moduleN. It's just that the filename and the string have to match. You are right that it is self-referentiual but I think it is possible. Essentially I want to only accept the results if the matched group name appears in a certain way in the filename.
// file module1/a1.h
#include "module1/a1.h" // change to #include "a1.h"
#include "module2/b1.h // don't change
--------------------------------------------------
// file module2/b1.h
#include "module1/a1.h // don't change
#include "module2/b1.h // change to #include "b1.h"

So if the feature was available I would do:
Search(regex mode): #include "(.+?)/(w+?.h)"
Replace: #include "$2"
Files to include: $1

Ok, I see the concept but it's not really practical to implement it with the way search works right now, sorry.


PLT-5518 RN: Allow user to upload + send image file attachments

Summary
This PR adds the ability for images to be attached to posts.
Ticket Link
https://mattermost.atlassian.net/browse/PLT-5926
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Added or updated unit tests (required for all new features)
[ ] All new/modified APIs include changes to the drivers

Test Information
This PR was tested on:
iPhone 6 iOS 10
iPhone 7 iOS 10
Galaxy s7 - Android 6.0 API 23

@jarredwitt also take a look at why tests are failing

@jwilander added tests.


fix(searchfield): add onClear callback handler for clear button

When using the SearchField component, I would like to know when the clear button is clicked. Right now, this isn't possible.

Coverage remained the same at 98.986% when pulling 7a978ee07ca1ac729a2688dbea78fcbd84bf0fbe on astankiewicz/search-field-onclear into 4458a19f113a2201790a772f74934393da1a61d2 on master.

Coverage remained the same at 98.986% when pulling 7a978ee07ca1ac729a2688dbea78fcbd84bf0fbe on astankiewicz/search-field-onclear into 4458a19f113a2201790a772f74934393da1a61d2 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add timeout in running test shell script step

Update workflow with the following timeout added to cancel hang job ASAP.
jobs.<job_id>.steps[*].timeout-minutes
The maximum number of minutes to run the step before killing the process.

Done by commit https://github.com/virt-s1/rhel-edge/commit/66fc4edc416b09ea8fd2a5e03e914466093d10b6


Show/hide Form component

Value Proposition
As a user, I want the form on top of the page not to affect my view on the page so that I have a good overview of the content.
Description

Acception Criteria

[ ] Display Form when needed.
[ ] Hide Form when not needed or when done with entering data.

Tasks

[ ] Write function for showing and hiding Form component.
[ ] Add a button for showing the Form.
[ ] Implement function on Form and button.

Looks good!
I'd rephrase the value proposition a bit:
As a user,
I want to be able to create new nest boxes
without navigating away from the homepage.
I'm eagerly awaiting the new feature 🥳


How to subscribe for a download

The sentence "We recommend that you subscribe to Azure IP Ranges and Service Tags for any changes or updates." does not make very much sense, since the link points to a download for a json-document. How can that be subscribed?
Also I would expect a way to subscribe for when the integration has occurred and to the required steps to leverage that integration (when available). For cosmosDB there is a notification for when an outdated SDK is used through azrue monitor - something like that would be good.

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

ID: f7be424c-f540-a753-6ecc-6ff3913368d5
Version Independent ID: dc80b25f-374d-8775-16e7-ae7d57b4b94f
Content: Azure Front Door Service - Frequently Asked Questions for Front Door
Content Source: articles/frontdoor/front-door-faq.md
Service: frontdoor
GitHub Login: @sharad4u
Microsoft Alias: sharadag

@DOliana, We are looking into this query and will update you as soon as possible.

@DOliana , Appreciate your patience. As Azure frontdoor backend IP space changes often. It can be a challenge in configuring IP ACLing for your back-ends. So, team is working towards its integration and till then it should be refereed and adjusted manually. As Azure IP Ranges and Service Tags JSON file gets updated every weekly, there is a need to download them every 7 days for the updated list. So, they might used the term subscription here for regular updates. But as far as i know there is exactly no feature of subscribing for this service here. Let me loop the respective content author for better insights.
@sharad4u ,Could you please provide your insights here ?

I fully understand the reason for which it is written in that way. The problem is that it suggests that an automatic solution that updates the IPs that need to be filtered is easy to implement, which is not the case.

I'm also facing this issue - I can't lock down access to back end IP addresses that might change, but I won't know if it changes unless I manually download the ranges document and manually check it against what I have configured, weekly.  Sorry, but this is a horrible workaround.  It makes any service protected by AFD brittle, while NOT implementing this completely nullifies any traffic protections that AFD might provide the back end.  Please expedite a solution to this strange (and huge) limitation.

@SubhashVasarapu-MSFT   Can I please have an official word on this?  We are preparing to launch a new service with AFD, and the lack of both a stable backend IP address range and a method of subscribing to updates is extremely concerning.  Please advise.

Any news on this? Would be good to have a smaller range than "gets updated once a week". Does it mean that once a week the website just might go down from certain POP's, until some kind of script downloads the file, sees the changes, and updates the ip rules? How's that acceptable for anything in production?
Is there any better way for a website hosted on App Service, considering I'm using two managed services from a same provider, I would expect them to actually work together.

#reassign: gunjanj

@DOliana I apologized for how long it took for you to get a response on this. We recently went GA with the Service tag discovery API, you should now be able to programmatically retrieve the list of service tags and IP addresses using either REST, Azure PowerShell or Azure CLI to accomplish this. For more information please see: https://docs.microsoft.com/en-us/azure/virtual-network/service-tags-overview#service-tags-on-premises.  If you have any further questions please feel free to reach back out. For now I will be archiving this issue. Thank you. #please-close


"Sârbii-Măgura", "Vitănești" are listed in the drop-down menu

Description:
When user writes "Sârbii-Măgura" and "Vitănești" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Sârbii-Măgura"
Expected results
The location appears in the drop-down menu as "Comuna Sârbii-Măgura, OLT"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Vitănești"

Expected results
The location appears in the drop-down menu as "Sat Vitănești, OLT (Sârbii-Măgura)".


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test

Docs Build status updates of commit ca36684:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-26-08-12-38 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-26-08-12-38

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat: customize name of Shield Tables w/o constant

Ref https://github.com/codeigniter4/shield/pull/628#issuecomment-1424971414

Please update the branch, we have recently made changes:
https://github.com/codeigniter4/shield/pull/629/files#diff-72d57325d612cc25a56eec53a2f8c721d8ef92fd14b20e2f83446f226972f715R476

Added docs.


Update used versions of CP and xcodeproj

Since CocoaPods 0.37 has been released, the used version of xcodeproj needs to be bumped to make the CP plugin usable/loadable again.

I don't normally +1, but please merge this because all the things are broken until slather gets bumped :)

@jhersh as a workaround for the meantime, you can apply this patch to a local checkout and rake install the gem or point your Gemfile to my fork.

That works great locally, but my bigger challenge is with projects that have a Travis/Circle CI script including e.g. sudo gem install cocoapods xcpretty obcd slather -N. It will happily install incompatible versions of slather and cocoapods, then pod install and the rest of the build fails.

@jhersh you should use a Gemfile instead of gem install :) The you can use GH forks and properly pin versions, e.g. https://github.com/contentful/product-catalogue-ios/blob/master/Gemfile#L4

You're darn right I should! For pods, I like the simplicity of keeping all dependency commands in .travis.yml or circle.yml. It's great until something like this happens :sob:

:shipit:, @marklarr should do a release :trollface:

:+1:

I'll release these afternoon :) also, I should give @kylef access on rubygems :trolface:

Slather 1.7.1 is out

Finally! :trollface:


Fix box-sizing values

@adamwathan I think you remove a part of box-sizing values -box

Whoops, thank you! 😅


dataview rework and Kibana OAS generation

The idea here is to take the work put into the Fleet OAS schema, and start preparing to migrate the Kibana APIs to it as they become available in the elastic/kibana oas folder.

Delete the generated/data_views spec, pull data_views from kibana instead (as a start). On this note, the data_views subset of the schema still appears to be created from a bundled file, rather than auto-generated. The schema needed minimal modifications to support what was being already being done. I can try creating an issue upstream for those like before.
Create a kibana2 client and config for lack of a better name. Once libs/go-kibana-rest is no longer needed, just rename the whole mess to kibana. I imagine there will be some discussion on this. Most of it is just copied from the Fleet client.
At some point way after all this, merge the generated fleet and kibana files since they're pulling the same spec down.
Reworked the kibana_dataviews resource. Was using this two stage tfsdk model and regular type model combo that is a bit overkill.

Notable tidbits for the resource:

Computed, Optional fields with no default set can have unknown values, for strings this is Pointer(""). So using somefield.ValueStringPointer() ends up sending a value that is not desired to the API. Added a helper for it. Surprisingly this did not crop up in the Fleet APIs as they all have defaults.
Existing functionality allows for semanticEqual between null configs and empty results for the SourceFilters, FieldAttrs, RuntimeFieldMap, and FieldFormats fields. Defaults could be set to be empty maps/slices, or keep the functionality I migrated.
There was logic for field_attrs.count being int or float from the API that I need to verify. right now its set to int only. The current docs say integer here

It looks like the operationId field has been fixed up with https://github.com/elastic/kibana/pull/198132. The published docs now include this change which should mean we can rip the operationId re-writing out of this code.

Already had it pending. I'll add it to this one
On Sun, Nov 3, 2024, 5:49 PM Toby Brain @.***> wrote:

It looks like the operationId field has been fixed up with
elastic/kibana#198132 https://github.com/elastic/kibana/pull/198132.
The published docs now include this change which should mean we can rip the
operationId re-writing out of this code.
—
Reply to this email directly, view it on GitHub
https://github.com/elastic/terraform-provider-elasticstack/pull/881#issuecomment-2453606292,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMTBDDD776QTF7ACWMFUDZ62R73AVCNFSM6AAAAABQXTTI4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGYYDMMRZGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Pulled a fresh copy again, the Delete EPM package body (force) was removed, apparently it was deprecated the entire time. Along with a bunch of kafka topic bits that aren't implemented here yet. Also included some import formatting and a golangci-lint finding in the factory_test file.

kibana2

I don't feel incredibly strongly here. There's benefits to using this naming in that we'll definitely rename it when the OG kibana client is removed.
Let's assume this exists in the codebase for 'a while' though which doesn't seem unreasonable. I'd prefer something more descriptive for example kibana_oapi or gen_kibana, at least then it's obvious what the difference between the clients is, though maybe less obvious which to choose for new code.
I'm happy to leave it as is if you do feel strongly here. I hate it, but that's not necessarily a bad thing :)

changed to kibana_oapi. I played around with merging them together and some other ideas, which only made the spaghetti worse.

No problem, its a big architectural change after all. 👍


CLI makes keystore required when truststore is specified

Which version of WireMock you're using
git: eb70b240169fcf1abbd86af36482d1c49826fa0b;

How you're starting and configuring WireMock, including configuration or CLI the command line
java \
-cp /var/wiremock/lib/*:/var/wiremock/extensions/* \
com.github.tomakehurst.wiremock.standalone.WireMockServerRunner \
--root-dir=/stubs \
--port=80 \
--https-port=443 \
--https-truststore=/truststore.jks \
--truststore-password=password \
--https-require-client-cert \
--verbose \
--global-response-templating

Expected behavior
Server starts with specified https-truststore while using the default https-keystore.
Actual behavior
Exception in thread "main" joptsimple.MissingRequiredOptionsException: Missing required option(s) [https-keystore]
	at joptsimple.OptionParser.ensureRequiredOptions(OptionParser.java:426)
	at joptsimple.OptionParser.parse(OptionParser.java:400)
	at com.github.tomakehurst.wiremock.standalone.CommandLineOptions.<init>(CommandLineOptions.java:140)
	at com.github.tomakehurst.wiremock.standalone.WireMockServerRunner.run(WireMockServerRunner.java:52)
	at com.github.tomakehurst.wiremock.standalone.WireMockServerRunner.main(WireMockServerRunner.java:125)

Same problem here

Same problem here

Same with last version 2.27.2.

Same with last version 2.27.2.


line 90 -> making 'Business Owner' bold

by chance found a typo, which resulted in BO being not bold.

 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.


WIP: Separate EKS-A releases into version-based folders

We want to be able to update and manage each individual EKS-A release separately. This would help us bump up bundle version for one release independent of the other, instead of one bundle version that making all releases use the latest bundle release. It also allows for performing bundle updates for releases that have already gone out.
/hold
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/ok-to-test

/test eks-anywhere-release-tooling-presubmit

/test eks-anywhere-release-tooling-presubmit

@abhay-krishna: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

eks-anywhere-cluster-controller-tooling-presubmit
60355049bbd5567c17bfabe1c00153faac5708b7
link
/test eks-anywhere-cluster-controller-tooling-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


🔧 Example app crashed on ios release build

How were you trying to build the app?
Example app crashed on ios release build.
Full build logs
2023-09-28 18:21:53.238066-0400 VisionCameraExample[25171:6547125] [native] Running application VisionCameraExample ({
    initialProps =     {
    };
    rootTag = 1;
})
2023-09-28 18:21:53.260174-0400 VisionCameraExample[25171:6547433] [javascript] Running "VisionCameraExample
2023-09-28 18:21:53.261669-0400 VisionCameraExample[25171:6547433] [javascript] Re-rendering Navigator. Camera: undefined | Microphone: undefined
2023-09-28 18:21:53.261991-0400 VisionCameraExample[25171:6547433] [javascript] Loading react-native-worklets-core...
2023-09-28 18:21:53.262525-0400 VisionCameraExample[25171:6547433] [javascript] Worklets loaded successfully
2023-09-28 18:21:53.262922-0400 VisionCameraExample[25171:6547433] VisionCameraProxy: Creating Worklet Context...
2023-09-28 18:21:53.263693-0400 VisionCameraExample[25171:6547433] VisionCameraProxy: Worklet Context Created!
2023-09-28 18:21:53.267375-0400 VisionCameraExample[25171:6547433] [javascript] Re-rendering Navigator. Camera: granted | Microphone: undefined
2023-09-28 18:21:53.267914-0400 VisionCameraExample[25171:6547433] [javascript] Re-rendering Navigator. Camera: granted | Microphone: granted
2023-09-28 18:21:53.318013-0400 VisionCameraExample[25171:6547433] [javascript] Re-rendering camera page with active camera. Device: "Back Triple Camera" (4224x2376 photo / 1920x1080 video @ 60fps)
2023-09-28 18:21:53.334218-0400 VisionCameraExample[25171:6547433] [javascript] Re-rendering camera page with active camera. Device: "Back Triple Camera" (4224x2376 photo / 1920x1080 video @ 60fps)
2023-09-28 18:21:53.385445-0400 VisionCameraExample[25171:6547433] [javascript] Camera initialized!
2023-09-28 18:21:53.385680-0400 VisionCameraExample[25171:6547433] [javascript] Re-rendering camera page with active camera. Device: "Back Triple Camera" (4224x2376 photo / 1920x1080 video @ 60fps)
libc++abi: terminating due to uncaught exception of type facebook::jsi::JSINativeException: Compiling JS failed: 1:1:invalid empty parentheses '( )' Buffer size 3 starts with: 280a29
(lldb)

Project dependencies
the package.json from the example app

VisionCamera Version
3.0.0
Target platforms
iOS
Operating system
MacOS
Can you build the VisionCamera Example app?
No, I cannot build the Example app either
Additional information

[ ] I am using Expo
[X] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

hey @basukiwinot there's a few duplicates of this issue in #1776 #1714

Maybe related but not the same. The issue I'm having is on ios release build. Debug build works fine.


Portal history in IITC

To build functionality for the new portal history in IITC.
https://community.ingress.com/en/discussion/13451/portal-history-now-available-on-intel-map/p1?new=1

https://github.com/iitc-project/ingress-intel-total-conversion/blob/a34df32640fbec85a245198451e0d5141c711927/code/entity_decode.js#L97-L111
when portal level is none a.length may be 14. but when portal was visited or captured, scout controlled, a.length may be 19.
I infer a[18]  means level.

Duplicate of #1315


Consultation with SSO Team

User Story

As a new organizational user in progress
I want to work directly with the SSO team to help our organization understand our capabilities
So that we all understand if the current solution is an appropriate fit.

Acceptance Criteria or Constraints

I'll see this or
The solution will conform to this or
Non-functional requirements

Acceptance Tests
Scenario 1: xxxxxx

GIVEN ...
WHEN ...
THEN ...

Scenario 2: xxxxxx

GIVEN ...
WHEN ...
THEN ...

Dependencies?
Team or ideally a ticket
Risks

Description
Impact
Strategy
Contingency Plan

If this does or doesn't occur by...
Then this will happen, costing $?
Mitigate, Transfer, Accept, Avoid
We will do this, if it occurs

Process Guidance
User Story
A short introductory section with the following structure:
As a: the person or role who will benefit from the feature;
I want: the feature;
so that: the benefit or value of the feature.
Acceptance Tests or also called Behaviour Driven Development Acceptance criteria
A description of each specific scenario of the narrative with the following structure:
Given: the initial context at the beginning of the scenario, in one or more clauses;
When: the event that triggers the scenario;
Then: the expected outcome, in one or more clauses.

this is what our tool will do.


Use a front-end framework

I believe that using a front-end framework (like NextJS or Remix) helps both the user experience and the developer experience.
Today the project lacks some useful features, such as a dropdown in the search and some CSS improvements that would help the results become more readable.

Thanks for bringing this up. Maybe I am old, but I see some value in low tech solutions where they are sufficient, and hence am hesitant to add complex frameworks and client-side logic to the mix.
CSS improvements are certainly possible without a framework, and I appreciate contributions.
What kind of drop down do you envision? My browser already adds suggestions based on the history of searches.
Note also that Loogle has a simple JSON API, so if you feel inspired to create a more fancy frontend, you can! If it sees clear traction that's a good way to convince me.


riak-debug relative data directory path crash

Running Riak from make devrel...
The storage has been configured with mult, and there is a leveldb datastore named big_leveldb.
The big_leveldb datastore has been configured to use data/big_leveldb and Riak is fine with this, loading data in and out to the associated bucket-type..
However riak-debug is not, it crashes with the following error:
[/basho/riak/dev/dev1%]bin/riak-debug
........E......EE..............Unable to locate <<"big_leveldb">> LevelDB data directory. Aborting.
Using riak_kv_eleveldb_backend data_root: data/big_leveldb

The problem goes away when an absolute path is given.

Two pull requests that relate to this, #676 will fix this bug in the 2.0 branch and #678 will fix it in the 1.4 brach.

As the above PRs have already been merged, I'm going to go ahead and close this issue. Feel free to re-open if there's more to discuss.


Clean install not prompting

Clean installed windows recently, just reinstalled ofscraper, installation went smoothly using pipx, enter "ofscraper" in cmd, it starts to open, but when it would usually prompt me for a task it instead just says "script finished" and ends.
Log
Screenshot
Config is currently default
Windows 10
Python 3.13.1 (installed using microsoft store)

looked slightly harder and found the solution in #484  (run pipx inject ofscraper aiolimiter==1.1.0 --force)
leaving open in case someone else doesn't identify the other thread as this problem

thats what mine would do before i ran that command you quoted.
Only thing i could think would be that you have two instances of ofscraper installed and pipx injected the wrong one. that or its something with the difference between powershell and command prompt.
im no pro tho

Having the same issue on fresh install ofscraper v3.12.9 installed via pipx. Tried to inject with aiolimiter but still haven't found a work around yet.

thats what mine would do before i ran that command you quoted. Only thing i could think would be that you have two instances of ofscraper installed and pipx injected the wrong one. that or its something with the difference between powershell and command prompt. im no pro tho

yup! that was it https://i.imgur.com/D9Qb8fv.png
I unistalled both (pip and pipx had 2 different instances) and only re-installed the pipx one and while it still doesn't work as current version I had to do the fix #484 (run pipx inject ofscraper aiolimiter==1.1.0 --force) to have it running.
still fails to download videos due to cdrm2 down, but better than nothing atm

CDRM hasn't been working for a while now. You will need to either try keydb or manual DRM keys (which is the preferred method IMHO) https://of-scraper.gitbook.io/of-scraper/cdm-options

Having the same issue on fresh install ofscraper v3.12.9 installed via pipx. Tried to inject with aiolimiter but still haven't found a work around yet.

If you can't get it to work with pipx you are going to want to uninstall everything for this scraper. ofscraper, pipx, aiolimiter, and python. Then install python 3.11.6, pip, aiolimiter (version 1.1.0). Then install ofscraper using pip. Doing all that should get the scraper working.