Posix sh

The portability fans are back! :-)
There were a few things broken already. In particular, a few places had "$foo" instead of $foo, breaking the intended words expansion.
The bash arrays were replaced with either simple expansion, or written directly, resulting in no code quality loss.
The zsh bits are only needed if someone will ever run "zsh cht.sh" and could be dropped; when zsh is working as sh, the shwordsplit option gets set automatically.
The local/typeset/declare case is the most complicated one. Technically, POSIX just reserves "local", "typeset" and "declare" for use by shell, keeping them as undefined behavior. All the shells I've tested with (bash, dash, ksh93, OpenBSD ksh, zsh) support either "local" or "typeset" command. But for better compatibility sakeness, since it was meaningful enough for 884161c4c7a3ccf6eb4c6ed226ed1104b49614ef, the "eval" fallback was implemented. This makes "local foo" invalid, thought, so I had to replace all of them with "local foo=" instead. But the better option likely be just ignoring shells that do not support local-scoped variables, since the code is likely to break in this case anyway.

@grayed Vadim welcome back! I'm very very (very) glad to see you again.
Thank you very much for the fixes (have you run the tests?),
and I am (again) very glad that you are back, additionally to all other reasons
because I plan to do a crazy thing, where a fresh look of a shell-portablity expert would be really helpful :)

Nope, I've totally missed the fact that tests exist. Is there any reason to not run them in GitHub CI? If not, will you accept a PR that adds appropriate GitHub actions then?

Here are results of running tests on OpenBSD: https://gist.github.com/grayed/4257f05f399df44753a3619ce7d18101
The results vary from run to run, sometimes there are 9 successfull tests, sometimes 8. I didn't dig into them, though, since there is too much Python for me now. :-\

since there is too much Python for me now

You didn't guess! This crazy test (and it is actually, just one test, but with several test cases) is written in bash.
The main problem here, as far as I can judge, that if you start it in the server mode, it does not work
(probably because of some problems that greenlet has in OpenBSD).
But you can start in the "standalone" mode (see inside the script how you can switch it),
so that no greenlets will be used. Then we will see what is missing
Regarding your question, why the test is not running in CI,
you are absolutely right, and the only one who is guilty is me.
I believe your question will be the final drop that will make me to fix it

Well, the tests are run by shell script, yes, but they try to run Python code, and it looks like something doesn't work there well. Still do not have enough time to look throughly. :(
I've sent another PR, with GitHub action draft, maybe it'll help. :)

$Thank you for the fixes; actually we have CI, and eactly these tests are running there (we use travis-ci),
but there are some minor fixes to be done (#224). I hope to fix it first, and then we can continue with this pull request

Well, this needs to be rebased to become mergeable. :D


Rise warning/error about a author id being merged from scopus to sosia

When a author id is merged and does not exist anymore, sosia provides an error, but from scopus one can see the warning. Maybe this can be adjusted...
[57201609495,<PHONE_NUMBER>]
E:\Anaconda3\lib\site-packages\scopus\author_retrieval.py:215: UserWarning: Author profile with ID<PHONE_NUMBER>5 has been merged and the main profile is now one of<PHONE_NUMBER>,<PHONE_NUMBER>1.  Please update your records manually.  Functionality of this object is reduced.
  warn(text, UserWarning)
Traceback (most recent call last):

  File "<ipython-input-1-1f5517ca9f0e>", line 47, in <module>
    pub_margin=0.2)

  File "E:\Anaconda3\lib\site-packages\sosia\sosia.py", line 162, in __init__
    Scientist.__init__(self, self.identifier, year, refresh)

  File "E:\Anaconda3\lib\site-packages\sosia\classes\scientist.py", line 210, in __init__
    self._name = ", ".join([au.surname, au.given_name])

TypeError: sequence item 0: expected str instance, NoneType found

But is this not what we want? Have an error, such that the user provides the correct ID? Or what kind if error-handling would you suggest?

Hi thought, this should appear as  a warning directly in sosia. But maybe is also more correct like this? ..in any case it is a minor issue

pybliometrics raises a warning in two cases:

The author ID gets forwarded to the new profile, so that the old ID is identical to the new ID
The author ID is old but does not forward because multiple new profiles are the possible new profile

Does that solve this issue?

In the current version of sosia, the manifests as this: In the Author Search and the Scopus Search API, the IDs get forwarded, but for the results from the Author Search API, we use the ID returned by Scopus - which is the new one.
So, when using the Author Search API in the first step of .find_matches(), one of two things happens:

if the new ID is not also in the set of candidates, we only get information for the new one, which we continue to use. All is fine.
if the new ID is also in the set of candidates, we duplicate information (since the queries for the old ID and the new ID return the same new ID). Ever since I enforce the UNIQUE constraint during the SQL inserting phase, sosia throws an error. We have to solve that.


Making the REST update method transactional

See the comment in the diff, which explains what and why we do this.
It makes sure that entities do not get persisted by mistake due to FlushMode.AUTO in hibernate.

nice catch!
:+1: from me
i think this is the related documentation of hibernate (FlushMode.Auto -> autosync of managed entities that have been changed in the meantime): http://docs.jboss.org/hibernate/orm/current/userguide/html_single/Hibernate_User_Guide.html#flushing

Please fix the #e bug and merge if you are happy :wink:

So glad this is fixed! Thanks a ton.

Thanks guys, merging now!


Fortigate with GWLB

Hi, im currently testing the AWS new service Gateway Load Balancer (GWLB). When the GWLB autoscaling will be released. Is it possible to use autoscaling without TGW in my case ? And is there any extra configuration on the fortigatte to support GWLB with GENEVIE tunnel. Thanks :)

Hi @fawzihmouda , please send your inquiry to<EMAIL_ADDRESS>


menu jumps when I move mouse to right side

Rancher Version: v1.1.0-rc3
Steps to Reproduce:

Move mouse over Stacks/All
With mouse over All, slide mouse slowly to the right

Results: You will see the menu get big and then small again.  This happens with Catalog as well and with Infrastructure the menu actually gets smaller for a second.
Expected:  Should not change size simply because I moved off of it.

Version - master 9/16
Not able to reproduce this anymore


Error on post install hook with CRD

Was trying to install a CRD as a post install hook to work around an istio issue.
See attached chart.
Error occurs only when CRD is installed as a post hook...
helm install --namespace=post-crd --name post-crd . Error: unable to decode "": no kind "fluentd" is registered for version "config.istio.io/v1alpha2"
CRD gets installed but helm fails.
Output of helm version:
Client: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.1", GitCommit:"b1b29978270dc22fecc592ac55d903350454310a", GitTreeState:"clean", BuildDate:"2018-07-17T18:53:20Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.10", GitCommit:"098570796b32895c38a9a1c9286425fb1ececa18", GitTreeState:"clean", BuildDate:"2018-08-02T17:11:51Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.): kops on AWS
Istio 1.0.3

Attached a simple helm chart to replicate..
post_crd.zip


fatal error: concurrent map read and map write - StartRefreshWritableVolumes

Hello,
We are experiencing random crashes of weed master, sample stderr log:
weed-master-stderr.zip
Problem occurs mostly during heavy usage of seaweedfs volumes (random reads + adding new files).
Regards,
Chris

Pretty sure you are hitting a problem that are recently fixed.
https://github.com/chrislusf/seaweedfs/issues/392

Please re-open if upgrading did not help.


Onboarding (Native): Issuance settings

issuanceTargetRatio
issuanceMaxAdjustmentPerYear

There should be a tool to simulate the results by modulating the above parameters.
Consider reusing the calculations made in honey-supply.

For this case, I would set up an new slider in the issuance policy screen (maybe in advanced?) called initialRatio, so people can decide how to do the initial distribution of the token.

Let's remember to consider this extra feature.


Update ASM to 9.1

Do not merge till #40 is merged.
Ready for review. Need to restage 4.1.2

I assume you want me to merge this and then restage 4.1.2?

Yes.

OK restaged


Ctrl+P, Wrong File

Issue Type: Bug
Get a filename and line in clipboard.
Filename.extension:Line
Press Ctrl+P, then Ctrl+V, then Enter quickly.
It'll likely open the wrong file.

/gifPlease


www/caddy: HTTP-01 challenge redirection subdomain fix

I forgot to allow the acme redirection option for subdomains. I recently needed it and I saw it was not there. So, now it is added.
Also it's not an advanced option anymore since it's tested thoroughly, and also needed for proper HA.

Merged, thanks!


create a task that starts the add 979 fields DAG

The input for the add 979 fields DAG will be a list of instance UUIDs and the associated bookplate fund data. The task should get the bookplate data from the bookplates table and contruct a 979 tag with the fund name in subfield f, druid in subfield b, image filename in subfield c, and title in subfield d, such as:
{'979': {'ind1': ' ', 'ind2': ' ', 'subfields': [{'f': 'ABBOTT'}, {'b': 'druid:ws066yy0421'}, {'c': 'ws066yy0421_00_0001.jp2'}, {'d': 'The The Donald P. Abbott Fund for Marine Invertebrates'}]}}
The instance UUID and this data should be passed to the function to add a tag (described in #1185 ).

I think it would be good to scaffold out this DAG to help define how we will trigger it (#1238). So this ticket should also include creating some rough outline of a DAG with the following steps:

lookup bookplate table for data to construct a 979 tag
get the MARC for each instance (see section Get the MARC for each instance to do a PUT to add 979 for code examples)
pass data along to a function to add a tag #1185
catch errors in updating the MARC data to be emailed in a report
generate email report of adding 979 data

The incoming data will be a dictionary made of the Instance UUID as the key and the value is a list of bookplate metadata.


Updated Target Lists

From Emory
Emory-AMPAD Candidate Targets 3.17.xlsx
Here is the updated Emory list with the Ensembl IDs for the targets.  We have reviewed our nominated targets and the experimental models and we don’t have any updates right now; we are in process of working through the first consensus proteomics networks and we plan to use that info to refine our target nominations.
Best, Allan

From Duke
20170316 Duke Targets.xlsx
Please find attached Duke’s current top 20 list of candidate targets, including annotations for how we got to these proteins and why we think that they may be important regulators of AD pathogenesis.
If there is anything else you need or if you have any questions, please let me know.
Best,
Matthias

From MSSM
Mette,
We have just updated the target list. Attached is a list of top 50 drivers. We will upload the network models accordingly.
Bin
Mt-Sinai_Top50_targets.xlsx

No new targets from UFL-ISB-Mayo:https://github.com/Sage-Bionetworks/ampadknowledgeportal/issues/167

We need to enforce standards for the provided files. Took an hour to reformat (not uniform across the submitted files).

https://www.synapse.org/#!Synapse:syn8656625

Hi Suzana,
Sorry, just reviewed the earlier email. Here are the genes in the spreadsheet. No change to model system spreadsheet. the list is the same as the one sent earlier today. I added the annotation.
Cheers,
phil
Columbia-Broad-Rush Candidate Targets_4_15_2017.xlsx

@kdaily - I think we need to reformat ourselves - it will take to long to go back to the groups and request new files

The layout is workable, but no one follows it. They put links and text in the gene fields, no one fills out the ensembl gene id column, etc.

I know. Cats. Herding - lesson learned, next time use a form
On Mon, Apr 17, 2017 at 10:05 AM, Kenneth Daily<EMAIL_ADDRESS>wrote:

The layout is workable, but no one follows it. They put links and text in
the gene fields, no one fills out the ensembl gene id column, etc.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Sage-Bionetworks/WallOfTargets/issues/65#issuecomment-294529889,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACiXB4fJkKukX4_Tx2tFCnY_RJIVe3H4ks5rw5v1gaJpZM4Mb9RO
.

--
Mette A. Peters, PhD.
Sage Bionetworks | 1100 Fairview Ave. North, M1-C108, Seattle, WA. 98109
<EMAIL_ADDRESS>|<PHONE_NUMBER>


Apply common sense constraints to window drag and resize

This PR fixes the issue that extension windows can be dragged/resized in such a way that the drag area no longer falls inside the browser viewport and the window becomes impossible to move.

Dope sheet is fixed. Problem was that while the left side uses our regular PanelDragZone component, on the right side the focus range doubles for panel dragging too, and I didn't apply the constraints there.


add support for struct hash

this should be equivalent to JS objectHash function

Sha256 Done.
As tested, the key of sourceString are not alphabetically sorted, temporarily fix the test_payload case by manually sorting the field order in struct.

neat, is there a default key order to access the js object? actually rust struct field order is not specified, we may need to add #[repr("C")] to enforce the order.

I don't find a way to sort the struct keys, seems that we have to statically define the struct with a alphabetic key order. too bad.


Update Composer dependencies (2023-01-28-00-09)

Visual regression test passed!
View the visual regression test report


Add overloaded constructor OurWaitFor accepting List

I using customCondition method for that, but it is deprecated.

@BlackEnf0rcer  could you please provide an example of customCondition usage? (the code will be the best) so I could see if it's possible to be replicated using current teasy approach or make necessary changes

    customCondition(webDriver ->
    {
        boolean result = getWolText().contains(oldText == null ? wolText : oldText);
        if (result == false) {
            this.getDriver().navigate().refresh();
        }
        return result;
    },
private String getWolText() {
    List<OurWebElement> elementList = elements(By.xpath("//*[@id=\"searchResultsList\"]/ol/li/div[2]/p"));
    return elementList.get(0).getText() + " " + elementList.get(1).getText();
}

@BlackEnf0rcer thanks, we will see what can be done. So far the only way to do this would look something like this
    element(By.id("")).waitFor().customCondition(webDriver -> {
        boolean result;
        if (result == false) {
            this.getDriver().navigate().refresh();
        }
        return result;
    });

This part -   element(By.id("")).waitFor() looks very artificial and we will discuss the best way of removing it to make it look like (for example)
waitFor().customCondition(webDriver -> {
boolean result;
if (result == false) {
this.getDriver().navigate().refresh();
}
return result;
});

Or it may be something like:
elements(By.id("")).waitFor().customCondition(elements-> {
result = check(elements);
boolean result;
if (result == false) {
this.getDriver().navigate().refresh();
}
return result;
});

yep, even better. Got it. By the way if you feel that you could implement a customCondition accepting any kind of param (not only a webdriver) - feel free to open a pull request (and we can discuss step by step plan in gitter chat). If you don't feel like doing it - it may take some time till we get to it ;)

I just checked... and it may require more deep refactoring then just adding a new method.. gonna look at it next week

fixed in 1.0.8-beta
@BlackEnf0rcer  you should be able to call
    waitFor().condition(elements -> elements.size()> 5, elements(By.id("test")));


chore: improve instance tests

Verify non-error return values from ConnectInfo
Generate an RSA key in mock package for faster tests
Introduce mock.TestClient to reduce test setup boilerplate

The additions to the sqladmin package will see heavy use in the forthcoming refresher tests.

To summarize an offline discussion. I'm going to make a few adjustments here:

add a new function to make it easy to get a mock sql admin service (leaving HTTPClient because we still need it)
use the FakeCSQLInstance instead of the sqladmin.DatabaseInstance
introduce functional options to the FakeCSQLInstance intitializer to make it easy to configure various details.


validate NR

Issue #: /bcgov/entity#14439
Description of changes:
Validate NR format. If the input NR number is valid, normalize the NR number and re-write the result to the input field
For example, input 1234567, it is a valid NR, the NR number input field shows 'NR 1234567'
The acceptable NR numbers are like:
NR 1234567
1234567
NR1234567
NR    1234567
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Eve. I assigned you so that the checks run.


use the template for hw8_twitterapi

template is available. please use that, than remake your pull requests.

@cerob I think we can close this issue as well, any objection?

since the deadline for the homework has passed, we close this.


I start the app and click on a date, the calendar goes to the previous month

Version of flutter_calendar_carousel
Flutter (Channel stable, v1.7.8+hotfix.4
Expected behavior
Starting the app and being able to select a date righ away.
Actual behavior
When I start the app (running on a physical android device) and click on a day of the initial month, the calendar gos to the previous month and don't let me select a day. In order to be able to select a day I have to click first on any of the the prev-next icon buttons, so the calendar shows the current date with its style (set by me) and I can click on a day without going to the prev month.
flutter doctor result (run flutter doctor in terminal)
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.7.8+hotfix.4, on Microsoft Windows [versÃ£o 10.0.18362.388], locale pt-BR)
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
[√] Android Studio (version 3.4)
[!] IntelliJ IDEA Community Edition (version 2019.1)
X Flutter plugin not installed; this adds Flutter specific functionality.
[!] VS Code (version 1.35.0)
X Flutter extension not installed; install from
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (1 available)
! Doctor found issues in 2 categories.
Steps to reproduce the behavior
Run the app in a physical android device and trying to click/select a day of month.
My calendar code:
Container(
  padding: EdgeInsets.symmetric(horizontal: 16),
  child: CalendarCarousel<Event>(
    customGridViewPhysics: NeverScrollableScrollPhysics(), //Vertical scrolling
    isScrollable: false, //Horizontal scrolling
    pageSnapping: true,
    firstDayOfWeek: 1,

    height: 420,
    onDayPressed: (DateTime date, List<Event> events) {
      setState(() {
        _selectedDateTime = date;
        _scheduleAppointmentBloc.selectedDateTime = _selectedDateTime;
      });
      print("${Values.PRINT_DEVBOX_PREFIX} _selectedDateTime: $_selectedDateTime | AQUI: $events");
    },
    weekendTextStyle: TextStyle(
      color: Colors.yellow[800],
    ),
    weekdayTextStyle: TextStyle(color: Theme.of(context).primaryColor,),
    selectedDateTime: _selectedDateTime,
    selectedDayButtonColor: Theme.of(context).accentColor,
    selectedDayBorderColor: Colors.grey,
    selectedDayTextStyle: TextStyle(color: Colors.black),
    thisMonthDayBorderColor: Colors.grey[400],
    //markedDateShowIcon: true,
    locale: "pt",
    headerTextStyle: TextStyle(color: Colors.grey, fontSize: 18),
    daysHaveCircularBorder: false,
    showOnlyCurrentMonthDate: false,
    todayButtonColor: Colors.grey[100],
    todayTextStyle: TextStyle(color: Colors.black),
    todayBorderColor: Colors.grey,
    iconColor: Colors.grey,

    customDayBuilder: (   /// you can provide your own build function to make custom day containers
        bool isSelectable,
        int index,
        bool isSelectedDay,
        bool isToday,
        bool isPrevMonthDay,
        TextStyle textStyle,
        bool isNextMonthDay,
        bool isThisMonthDay,
        DateTime dateTime,
        ) {
      /// If you return null, [CalendarCarousel] will build container for current [day] with default function.
      /// This way you can build custom containers for specific days only, leaving rest as default.
      //If the worker is available at this day
      if (_scheduleAppointmentBloc.isWorkerAvailableAtDay(dateTime.day, _worker)) {
        isSelectable = false;
        //Builds this day like the default
        return null;
      } else {
        return null;
      }
    },
  ),
),

Has anyone had this issue? All the rest is working perfectly...

Could you generate this reproduction in our example project? As you can see it is quite time-consuming if the reproduction isn't one way runnable.

Could you generate this reproduction in our example project? As you can see it is quite time-consuming if the reproduction isn't one way runnable.

Sorry, you can delete this line with the bloc code and the print code, and create this _selectedDateTime as a class member (by the way, I asked myself the same thing about this variable in your example snippet.


Multiary Finalization

multiary is nearly complete, featuring a replacement for much of the behavior currently exhibited by add and multiply, allowing those operators to be rewritten in a much cleaner fashion. There exist a number of tasks which need to be completed to finish this closure generator:

[ ] Figure out a consistent algorithm for addition factorization: the consider operations for add are notionally used to reduce repeated sub-expressions into easier to compute representations. For the most part, this is intended to handle term combination (e.g. x + 2*x => 3*x); however, this could also theoretically handle situations like x*y + x + y, which has multiple different results.
[x] A MultiaryNode should have its operands array sorted, with a closure-defined sort order. The sorting algorithm used should be based off of the degree of the operands. Naively, add should sort by degree descending (so, e.g., resulting in 2 + x + x^2 being sorted as x^2 + x + 2); multiply should sort ascending (x^2 * y * 2 would yield 2 * y * x^2). More analysis likely needed. In any event, the as-of-yet undeleted old client code has an implementation of degree which might be easily adapted to this end. A singular sort of the operands can occur right at the end of the operation, thus significantly cutting out redundant behavior.
[ ] closures/multiary.ts is very messy, as it contains all code related to it in a singular file. While most of the contents can stay in that file, the reimplementations of add and multiply need to be moved to their respective definitions in arithmetic. Furthermore, parts of multiary are somewhat sloppily defined after they are used, which should be addressed.
[x] multiary needs utility HOFs which generate unary and binary closures, similarly to binary. These would be named something like unaryFrom and binaryFrom. Intended use would be to provide the ability to generate a derived function off a multiary. So, something like: const subtract = binaryFrom(add).
[ ] Many of the test suites are likely to fail after add and multiply are properly reimplemented. Mostly, this will be due to failing snapshot tests on the writer logs. multiary logs will be incompatible with those produced by binary, which is intentional. However, some tests may fail because of different results---results which should, in theory, be semantically equivalent, but just expressed slightly differently. Many tests will likely need to be rewritten to address this. The snapshots will need review, but likely can be safely updated. Extra tests will likely need to be added to look at multiary's unique behavior.

Unless something untoward manifests during the completion of these tasks, multiary should be complete afterwards.

A problem exists with the current implementation of multiary: the 'walk/cleave' subprocess, by design, yeets found multiary nodes it finds during the walk phase in favor of their children (i.e. the cleave phase). This effectively tosses out the log information stored within the discovered child multiary nodes, which means that the history of how those nodes came to be is lost. This wouldn't necessarily be an issue, per se, but for the curation process: most functions in the calcula will curate the logs of the children they handle so as to minimize redundantly stored information.
That latter process generates the issue: for a nested multiary to exist, it would have had to have been processed already. This processing would, by its very nature, result in the curation of the grandchildren, shifting their logs into the nested multiary's. Which implies, by the time the multiary that instigated the walk/cleave gets those grandchildren for its operand list, that those grandchildren effectively no longer have history.
That history could be restored: walk/cleave could attempt to retrieve the appropriate log information from the multiary nodes being cloven and rewrap the children with that retrieved log information.
New problem: a Writer object's log field is, effectively, an Operation array; for the purposes of how the writer monad is implemented, Operation from its perspective is generic. However, for most of the rest of the library,  Operation is separately defined as the following interface:
export interface Operation {
  particles: Particle[]
  action: string
}

That is, the rest of the app, when working with a Writer<TreeNode, Operation>, will be dealing with objects that meet the contract defined above. This implies that logs are not only a linear list--from the perspective of Writer--but are limited to being nothing more than a linear list--from the perspective of everything else.
Operation, defined as such, would thus make log retrieval much more complicated: there wouldn't be an easy to discern way to track where a node's directly generated logs were in its log separate from logs inserted from the children. (By way of consideration, look at the following excerpt from the processLogs function---wildly used by different closure-generating HOFs for their logs; e is the currently processed child node, while next is a potential sibling node.)
const currentOp = next
  ? /* ... */
  : operation(
      toParticles(...haveProcessed, processed(current), ...toProcess),
      `processed ${parameterName(i, n)}`
    )
return [...e.log, currentOp]

Operation, therefore, likely needs to change:  by allowing it to store either its current interface or a list of itself, any node which needed log information stored for children nodes could store that information as a sub-array. This might look something like the following:
export type Operation = {
  particles: Particle[]
  action: string
} | Operation[]

This is not without issues: certain functions will immediately break and need updating to reflect these changes. Of immediate principle note, the context function--used to grab the particles at a given location within the log--would need to be updated to reflect this alteration. Which raises an interesting question: what does context mean for an entry that is a list? Would it be the concatenation of all nested Particle arrays? Just the last entry? A special combination of the first and last, to suggest the progress performed by the sub-process?
Introducing this extra behavior is potentially non-trivial, but likely necessary. However, doing so carries with it a happy accident: NuLog's verbosity made for lengthy logs, which could be occasionally difficult to follow; it has been desired to have a nested sub-process addendum to NuLog, so as to treat those sub-processes as sub-steps that one could look at in further detail if desired, but could ignore otherwise. Adding in the change to Operation not only allows multiary to reconstitute children logs, but would allow the frontend to nest those child processes.

The otherwise block of multiary's extensions attempts to combine similar terms within the operands array by iterating through each operand and checking if any other operand matches it, according to a set of checks that are primarily provided by a closure context. multiary currently defines a singular consideration: one to combine multiple primitives which are present with non-primitives. Collectively, this process is referred to internally as combineTerms, which gets generated from the createCombineTerms HOF after the 'consideration' block of multiary is ran.
Any of the interstitial results produced by combineTerms have newly developed history. While combineTerms might be a simple pass-through operation (i.e., no term combination has occurred), it might change the list of operands it was given. Which is fine, save for one point: the when block in which combineTerms is invoked makes assumptions about the operands which are handled by its fn parameter: notably, that the passed operand list isn't effectively mutated during the processing of fn. This allows it to (in theory) curate those operands before passing them to fn and include the uncurated set in the logs it manages. Yet! by its very behavior, combineTerms alters a copy of the operands list, which leads to a disconnect between the result of the otherwise block and the logging of that block.
A separate implementation of when, which has different expectations about what is returned to it, should be provided for the otherwise block. This was an early topical consideration when development of multiary began, but was deemed (at the time) to be an unnecessary complication. However, not doing this would result in when needing completely separate behaviors for the normal and otherwise edge cases. Which seems more complicated than having a different function to handle that edge case.
The new otherwise function should likely expect to receive an augmented Action<TreeNode> tuple: the standard Writer<TreeNode, Operation> result and action string, but also a list of the effective operands that were produced by combineTerms.
Optionally, otherwise could be defined---as it shouldn't be overridden by a closure context---with combineTerms as a parameterized HOF, which would allow it to know exactly what the combined operands list is. This would allow, perhaps, the bulk of the functionality currently defined in replaceAndSort to be moved to the otherwise block directly. Seems like a cleaner, albeit a perhaps inflexible, solution.


Documentation on Events

Hi,
I'm using your component but lost a lot of time on making the events work.
Your documentation never mentions that supplying a videoid is mandatory before the events can work.
In your example, there is mention of video but not videoid.
Can the documentation be changed for this to avoid other people breaking their heads on this?
Thanks.
P.S.: Kudos on the component

Hi @Kookpot. Its already mentioned in the documentation. Please check -

Then also mention this parameter here please :

Also note that you use videoID as variable when there is actually a videoId parameter. Very confusing!!

Oh yes! My apologies for not catching that. The documentation has been corrected. Thank you!


Unnecessary recipe book button conflict with Crafting Tweaks

Issue Description:
When playing with Ender IO, Crafting Tweaks and JEI installed, the recipe book button can not be detected by Crafting Tweaks, causing Crafting Tweaks' "hide recipe book" option to no longer work and the button to overlap with Crafting Tweaks' own buttons.

What happens:
Ender IO takes the recipe book button and wraps it in a custom class with a different id making it impossible for other mods to find the button without specifically special-casing Ender IO. As a result, Crafting Tweaks is unable to either move the button out of the way, or hide the button (depending on the user config).
https://github.com/SleepyTrousers/EnderIO/blob/be323475483d6ace21ec450a86154b9d96e3de69/enderio-base/src/main/java/crazypants/enderio/base/handler/RecipeButtonHandler.java
https://github.com/blay09/CraftingTweaks/blob/1.12/src/main/java/net/blay09/mods/craftingtweaks/CraftingGuideButtonFixer.java
What you expected to happen:
Ender IO could listen to the ActionPerformed.Pre event instead of (unnecessarily?) wrapping the original button in its own class. Alternatively, it should probably use EventPriority to try and run its InitGui event handler last.
Steps to reproduce:

Place a Vanilla crafting table.
Open the crafting table to see Crafting Tweaks no longer being able to react to the recipe button as it should.

Affected Versions (Do not use "latest"):

EnderIO: 5.0.22
EnderCore: 0.5.20
Crafting Tweaks: 8.1.8
Just Enough Items: <IP_ADDRESS>
Minecraft: 1.12.2
Forge: <IP_ADDRESS>73

Your most recent log file where the issue was present:
N/A
https://github.com/blay09/CraftingTweaks/issues/116
https://github.com/blay09/CraftingTweaks/issues/117

Seen this?
(PersonalConfig.recipeButtonDisableAlways.get()
        || ((PersonalConfig.recipeButtonReplaceWithJei.get() || PersonalConfig.recipeButtonDisableWithJei.get())

Yes, but a config option is not a good solution to a compatibility issue, especially when most people won't even know where to find it (Ender IO isn't the kind of mod you would expect to alter Vanilla crafting tables, which is why it was initially difficult to even track this issue down).

If a config option isn't a good solution, then how come is fine for crafting tweaks to use a config option? Also, it's much better for us to give the option then hard code support for every slight mod conflict out there

You got it the exact wrong way around though. You don't need hard code support for a specific mod, you just need to apply the fix I've mentioned in my first post that would preserve compatibility with any other mod trying to find the recipe book button. I am the one who would require hard coded support for Ender IO, which is completely unnecessary in this case for the reasons I have already outlined above.
Crafting Tweaks doesn't use a config option trying to avoid mod conflicts because it doesn't have this issue to begin with. It doesn't effectively wrap and hide the button from other mods. This is an issue in Ender IO's implementation of the RecipeButtonHandler regardless of which other mod is affected.


setYaw rotation direction problem still occurs.

I had asked about this problem. #473
I tested latest two pannellum.js code (https://cdn.pannellum.org/2.3/pannellum.js and uploaded code)
Two javascript files still have a direction problem. The cdn code seems to have modified the yaw range.
But still has problem. And also occurs in the Custom controls example.
Test yaw value :
cdn : Change yaw from 175 to -175. And yaw from 175 to 184.
uploaded js file code : Change yaw from 175 to -175.
Both are caused by the same problem. I am sorry, please check  it.

Pannellum 2.3 is the last release, not the development version. It has been fixed in the development, as I said before.


Proper way of addattr32

Is this a question about neli specifically? If so, please describe.
I'm trying to do a RTM_GETNSID query with neli, and have problem adding a NETNSA_FD attribute after Rtgenmsg. I managed to bypass the problem by adding a Nlattr manually, with the following code:
    let mut c = Cursor::new(Vec::new());
    let _ = Rtgenmsg {
        rtgen_family: RtAddrFamily::Unspecified,
    }.to_bytes(&mut c);
    c.write(&[0, 0, 0]);

    let file = File::open(path).unwrap();
    Nlattr::new(false, false, 3, file.as_raw_fd()).unwrap().to_bytes(&mut c);

    let buf = c.into_inner();
    let req = Nlmsghdr::new(
        None,
        Rtm::Getnsid,
        NlmFFlags::new(&[NlmF::Request]),
        None,
        None,
        NlPayload::Payload(buf.as_slice()),
    );

The equivalent code in C looks like:
	struct {
		struct nlmsghdr n;
		struct rtgenmsg g;
		char            buf[1024];
	} req = {
		.n.nlmsg_len = NLMSG_LENGTH(sizeof(struct rtgenmsg)),
		.n.nlmsg_flags = NLM_F_REQUEST,
		.n.nlmsg_type = RTM_GETNSID,
		.g.rtgen_family = AF_UNSPEC,
	};
	int fd = -1;

	fd = netns_get_fd(name);
	if (fd < 0)
		return fd;

	addattr32(&req.n, 1024, NETNSA_FD, fd);

Sorry if this is a dumb question.

Hi @rvql! This may actually be a bug. Rtgenmsg was added by an external contributor and it looks like attributes are not allowed to be set for that type yet. If you have a use case where attributes are needed after this type, then we should fix this!

The use case is querying net namespace id from file. The request packet consists of a Nlmsghdr, a Rtgenmsg and an extra Nlattr after it. The example code in C is here.
As a beginner with netlink, I'm not sure whether this Nlattr should be part of Nlmsghdr or Rtgenmsg.

@rvql I took a look at addattr32 and, as I expected, it's following the typical rtnetlink format.
Netlink in general has the following format:
|-----------------|----------------------|-----------------------|
nlmsghdr          subheader              attributes
                  (genlmsghdr/rtgenmsg)  (nlattr/rtattr)

addattr32 jumps to the end of the packet after all headers and attributes and adds another one then adjusts n.nlmsg_len to reflect the new length. What I just need to do is add support for attributes after Rtgenmsg and this should resolve your issue.
I won't be able to support this in the v0.6.x series because it requires a breaking change (once #83 and #86 are implemented in v0.7.0, this won't be a problem moving forward in future releases). I will however put up a fix on the 0.7.0-dev branch and you can try it out as you have time. This will definitely be included in the v0.7.0 release once it comes out.

Great! Thank you very much!

This is now resolved by f05308cd0cab47bd89c40df744a9a1bd661ce473 in #182


chore(release): v0.8.3 🎉

:robot: I have created a release *beep* *boop*
0.8.3 (2022-02-18)
Bug Fixes

api-gateway: ensure that lambda event validation error does not leak sensitive metadata (#22) (95fa89a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/uladkasach/simple-lambda-handlers/releases/tag/v0.8.3 :sunflower:


"Maybe the server is from the future" error while using HTTP proxy — response to inline media request is a redirect

I'm getting an odd error while trying to access a file with Lagrange while using an HTTP proxy.

To reproduce:

Use the proxy stargate.gemi.dev:1994
Got to the URL https://cdn.atp.fm/rss/public?x0h3nk5k (This is an RSS feed, but the Stargate proxy automatically converts it to gemtext and exposes the MP3 enclosures)
Click on any of the Audio items

Result:
You get the error message above.
Expected:
Lagrange downloads the audio file, or plays it inline.
Extra info:

The links to the auto files actually first return a redirect, before returning the actual file
The URL of the URL to the mp3, as well as the redirect that's in the RSS feed are < 1024 characters, so should still work with the gemini protocol.
At first I thought this might be a bug in my proxy, but both gemget and and amfora successfully download the when configured with the same proxy, download the file just fine

Yeah this is definitely a bug in Lagrange. When doing an inline media request, it can only handle a 20 response. The 30 redirect causes the error popup, when the correct behavior would be to redo the request as told.
The workaround is to do the request without triggering the inline media player, e.g., opening the link in a new tab.


JBPM-5428: Readonly option in the Checkbox component doesn't work.

Fixed other Forms minor issues:

Fixing infinite loop switching Form to "PRETTY-MODE"
Fixing Form unbind

@romartin can you take a look?


Add full tagset to conll2003 README

Even though it is possible to manually get the tagset list with
dset.features[field_name].feature.names

I think it is useful to have an overview of the used tagset on the dataset card. This is particularly useful in light of the dataset viewer: the tags are encoded, so it is not immediately obvious what they are for a given sample. Adding a label-int mapping should make it easier for visitors to get a grasp of what they mean.
From user-experience perspective, I would urge the full tagsets to always be available in the README's but I understand that that would take a lot of work, probably. Perhaps it can be automated?
closes #3189

I also added the missing pretty_name tag in the dataset card to fix the CI


I can translate your Docs to Korean to improve the accessibility of Korean users!

Hello, I am a college student from Korea who is interested in using Caffe2. I want to participate in the project as part of the school ‘open source practice project’, so I leave the issue! Maybe I can translate your homepage(http://caffe2.ai) Docs tutorial into Korean. May I release it to improve the accessibility of Korean users?

That would be great! We're currently working on a translation management setup that will allow us to keep track of each translation, however, don't let that hold you up. We can always fit your translations in later and in the meantime link to them from the English page... maybe we can even do some default language detection on the webpage itself.
When you create a translated document, just tag me and I'll help you get it posted and linked up.

Thanks a lot!


Filename case sensitivity issue?

There was a discussion recently that the image files have to be named with a fully lowercase extension, otherwise they will not be found. Presumably this is on Linux where filenames are case-sensitive. Might be worth checking the file extensions in a case sensitive way

I can confirm it's only a Linux issue.
While I'm used to it and renaming files can be easily automated, it'd be nice to have them be case insensitive. It is one step saved, after all!


[$250] Android & iOS - Chat - No gray focus background when option in + menu is tapped

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

Version Number: 9.0.59-0
Reproducible in staging?: Y
**Reproducible in production?:**Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Launch ND or hybrid app.
Open any report.
Tap +.
Tap any option (or long tap to see the problem better).

Expected Result:
When the option in + menu is tapped, there will be gray focus background (mweb behavior).
Actual Result:
When the option in + menu is tapped, there is no gray focus background. Only the icon on the left turns green.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/cb896ef5-dee4-4f6b-871a-72e739ae47e9

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021855970523999164346
Upwork Job ID:<PHONE_NUMBER>999164346
Last Price Increase: 2024-11-11
Automatic offers:

dukenv0307 | Reviewer | 104850119

Issue OwnerCurrent Issue Owner: @dukenv0307

Proposal
Please re-state the problem that we are trying to solve in this issue.
No bg color when pressing popover menu item.
What is the root cause of that problem?
In PopoverMenu, we pass a background color style to the Menuitem.
https://github.com/Expensify/App/blob/81c17e8b401d7d8ea3f9e52061168df726531a24/src/components/PopoverMenu.tsx#L267-L273
If none of the conditions are satisfied, we return an undefined background color.
https://github.com/Expensify/App/blob/81c17e8b401d7d8ea3f9e52061168df726531a24/src/styles/utils/index.ts#L1122-L1136
That wrapper style override the pressed style from MenuItem.
https://github.com/Expensify/App/blob/81c17e8b401d7d8ea3f9e52061168df726531a24/src/components/MenuItem.tsx#L597-L599
StyleUtils.getButtonBackgroundColorStyle returns {backgroundColor: 'xxx'} and wrapperStyle contains {backgroundColor: undefined}.
What changes do you think we should make in order to solve the problem?

We need to return empty style from getItemBackgroundColorStyle if none of the conditions is satisfied instead of returning undefined.
function getItemBackgroundColorStyle(...): ViewStyle {
    if (isDisabled) {
        return {backgroundColor: undefined};
    }

    if (isSelected) {
        return {backgroundColor: selectedBG};
    }

    if (isFocused) {
        return {backgroundColor: focusedBG}
    }

    return {};
}

PR is ready
cc: @dukenv0307

Payment summary:

@bernhardoj $250 - please request in ND
@dukenv0307 $250 - payment upon completion of the checklist

Requested in ND.

Thanks @dukenv0307! Please accept the offer

$250 approved for @bernhardoj

Thanks @dukenv0307! I've updated the payment summary

$250 approved for @dukenv0307


rename 'extern-so' to 'native-lib'

Calling "extern" functions is not super clear IMO (extern to what), but saying that we are calling "native" functions from inside the interpreter makes it very clear I think.

@bors r+

:pushpin: Commit 5eeb579ac76a680d940aa0b047c74e00c016f56e has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 5eeb579ac76a680d940aa0b047c74e00c016f56e with merge cc1e36ca37b9a4e5229cd806da4604c924956aa2...

@bors r+

:pushpin: Commit b90d953c9687422407e133f567319468ba6608b9 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit b90d953c9687422407e133f567319468ba6608b9 with merge d213eb20a3be26140e35e73805c55915400b884c...

@bors r+ retry

:pushpin: Commit 73cbb49fbffda098dac39b7af1f4f175426daca5 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 73cbb49fbffda098dac39b7af1f4f175426daca5 with merge 9f63c72f1def826d34aaff743e7036382c0c5086...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 9f63c72f1def826d34aaff743e7036382c0c5086 to master...


[cli-test] Use pinned nixpkgs commit that has pre-built yarn

Summary
Ran curl 'https://search.devbox.sh/v1/search?q=yarn' | jq | less
and picked the nixpkgs commit hash that was listed in each platform.
How was it tested?

[ ] will observe testscript run

Current dependencies on/for this PR:

main

PR #1382   👈

This comment was auto-generated by Graphite.


Title Changes for the Accounts Menu

My Accounts to University Logins, My Library (Catalog) to My Account (Catalog), based on Kace request from Tom

Commit to alter area names as per Tom request, no conflicts reported to master branch.

all done.


[BugBash] Links are not being highlighted in Quick link during mobile scroll

See in the recording Treading News never gets highlighted.
https://github.com/aemsites/icicidirect/assets/19485818/339ee820-a56f-4a8b-a20b-258f0f01c4bf
(Recording is of Mozilla but I can see similar behaviour in Chrome)

The height of the device used in the recording is very small, its workign fine when we use correct aspect ratio device. @anagarwa can try once with following layout, where device is on the left side and debugger is open on the other side as shown below:

if its working fine, we can close this one as well

@vivesing closing this one .


Invalid Links

The links for "Sign In", "Sign Up", "Profile", and "Cart"  and All links which are using in this project work correctly when clicked within the navigation bar, but when trying to access them directly by entering their URLs, a "Not Found" error is displayed. I have also attached a video for reference.
screen-capture (2).webm

The problem is isolated to the production environment; everything functions correctly on the local setup.

@Kethankrk

I would like to help, but I am unable to run the project on my local machine. Tried many solutions but nothing seems to work :(

Tried creating a new Vite project and the problem persisted, so it is a problem in my pc

@Kethankrk
How to Contribute We welcome contributions of all kinds, whether it's coding, design, documentation, or testing. Here's how you can get started
Fork the Repository:
Click the "Fork" button on the top right of this repository to create your own copy.
 Check if you are in the frontend folder or not
Install Dependencies:
Run yarn to install the project's dependencies.
Start Development
Use yarn run dev to start the development server.

Tried creating a new Vite project and the problem persisted, so it is a problem in my pc

Okay Try to resolve it.

fixed my issue( reinstalled node and npm).
Speaking of the issue "Invalid Links" have you already fixed it? because it is working fine for me.
https://github.com/dubey748/Live/assets/84431821/6df4b1b7-0695-49d6-afbc-361a34816923

Hi, @Kethankrk  I've encountered an issue that seems to only occur in our production environment at https://ablive.onrender.com/ . You can test it there to see for yourself. I suspect it could be related to either a deployment problem or an issue with react-router-dom, but I'm not certain about the root cause.

Hi @Kethankrk Fortunately, I've managed to resolve this issue. It turns out that we should use HashRouter instead of BrowserRouter when creating a single-page application. For a clearer understanding, you can review my commits. Additionally, I'm providing a link for your reference.
https://stackoverflow.com/questions/51974369/what-is-the-difference-between-hashrouter-and-browserrouter-in-react

Try to use BrowserRouter.


Issues with German spelling

There appears to be something wrong with de_20 and de:
At the very least it does not correctly detect desweiteren as incorrect.
I read somewhere that spell files are imported from OpenOffice, but a quick
test in LibreOffice revealed that spell checking is working fine there.
It's kinda annoying since that word is an extremely common misspelling.
Other German spell files are unaffected.

There appears to be something wrong with de_20 and de:
At the very least it does not correctly detect desweiteren as incorrect.
I read somewhere that spell files are imported from OpenOffice, but a quick
test in LibreOffice revealed that spell checking is working fine there.
Other German spell files are unaffected.

The German spell files were last updated in 2010.  Someone should spend
a bit of time on getting the latest version and applying the changes to
make them work in Vim.
--
An extraordinary TALL KNIGHT in all black (possibly John with Mike on his
shoulders) walks out from the dark trees.  He is extremely fierce and
gruesome countenance.  He walks towards KING ARTHUR and PATSY, who are
wazzing like mad.  (Salopian slang, meaning very scared.  almost to the
point of wetting oneself, e.g. before an important football match or
prior to a postering.  Salopian slang meaning a beating by the school
praeposters.  Sorry about the Salopian slant to this stage direction - Ed.)
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

A :spelldump of de_20 spelllang shows:
/regions=de1920atch
.....
deswegen
desweiteren/13
desweiterend/13

According to :h spell-wordlist-format, the /13 specifies a region where this is valid. According to the header, regions 1 and 3 would mean language de and de_20. However, that is certainly wrong, as the Duden states.

The German spell files were last updated in 2010. Someone should spend a bit of time on getting the latest version and applying the changes to make them work in Vim.

It was always wrong however :)
Where is documented, what needs to be done to get updated spellfiles? I can do it.

Christian wrote:

The German spell files were last updated in 2010. Someone should
spend a bit of time on getting the latest version and applying the
changes to make them work in Vim.

It was always wrong however :)
Where is documented, what needs to be done to get updated spellfiles?
I can do it.

runtime/spell/README.txt
You will have to figure out the right place to download from, it has
probably changed.  The old ones can still be found at
http://download.services.openoffice.org/files/contrib/dictionaries/
They date back to 2006, there must be newer ones.
Then make sure the differences in runtime/spell/de/*.diff are applied,
you probably will have a make manual changes.
It's not required to use Aap for doing this.  You could also drop the
main.aap file and use manual instructions, since it's done infrequently.
But if you do manage to make the main.aap work it's easier for me to
create the distribution.
--
TALL KNIGHT: We shall say Ni! again to you if you do not appease us.
ARTHUR:      All right!  What do you want?
TALL KNIGHT: We want ... a shrubbery!
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Okay, thanks for the pointers. How about those patches:
https://github.com/chrisbra/vim/compare/5608cce08057cd243435259a9fd3e867cd9d7a8f...6270028d2b75bb63e8f9318798bc9f53c5d14da9
(I also committed the resulting spl and sug files, so you should be able to just download them).
There doesn't seem to exist a combined dictionary for de_de anymore. I therefore used the new orthography for de_DE and I would also propose that spellang=de uses only the new spelling and not the old anymore. Not sure, if this needs any adjustments on Vims side however. I am not entirely sure, how from the different regions we come to the spelllang settings de de_de de_19 de_20.

Christian wrote:

Okay, thanks for the pointers. How about those patches:
https://github.com/chrisbra/vim/compare/5608cce08057cd243435259a9fd3e867cd9d7a8f...6270028d2b75bb63e8f9318798bc9f53c5d14da9
(I also committed the resulting spl and sug files, so you should be able to just download them).

Thanks.  Looking at the diff, the resulting .diff files are huge.  That
can't be right, they should mainly contain changes to the .aff header,
adding the SAL lines and such.  Perhaps the .aff and .aff.orig files are
from different versions?  The idea is to delete the .aff.orig files,
download the new spell files, then copy .aff to .aff.orig and then make
the changes needed for Vim.  After that the .aff.diff can be created, so
that it can be applied by someone else.
How do I see the resulting files?  Github doesn't appear to have a link
to that.

There doesn't seem to exist a combined dictionary for de_de anymore. I
therefore used the new orthography for de_DE and I would also propose
that spellang=de uses only the new spelling and not the old anymore.
Not sure, if this needs any adjustments on Vims side however. I am not
entirely sure, how from the different regions we come to the spelllang
settings de de_de de_19 de_20.

Well, what would most users want to use?  That is what "de" should
result in.
--
You were lucky. We lived for three months in a brown paper bag in a
septic tank. We used to have to get up at six o'clock in the morning,
clean the bag, eat a crust of stale bread, go to work down mill for
fourteen hours a day week in-week out. When we got home, our Dad
would thrash us to sleep with his belt!
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Thanks. Looking at the diff, the resulting .diff files are huge.

I deleted duplicates (or whatever Vim thought of duplicates from the dic file. That were many. The change to the aff file was minimal, mainly dropping lines that Vim does not understand. There were a couple of conversion warning when run in latin1 environment, but that should not harm.

How do I see the resulting files? Github doesn't appear to have a link to that.

If you click on the file, (e.g. https://github.com/chrisbra/vim/blob/master/runtime/spell/de.utf-8.spl, you can click to download it).

Well, what would most users want to use? That is what "de" should result in.

Considering that the reformation of the spelling has been done more than twenty years ago and has been in effect for more than 10 years, I would go so far and say it should only be the new german spelling and the old one should not be used anymore.

Christian wrote:

Thanks. Looking at the diff, the resulting .diff files are huge.

I deleted duplicates (or whatever Vim thought of duplicates from the
dic file. That were many. The change to the aff file was minimal,
mainly dropping lines that Vim does not understand. There were a
couple of conversion warning when run in latin1 environment, but that
should not harm.

How do I see the resulting files? Github doesn't appear to have a link to that.

If you click on the file, (e.g.
https://github.com/chrisbra/vim/blob/master/runtime/spell/de.utf-8.spl,
you can click to download it).

I wanted to see the input files, these are apparently at
https://github.com/chrisbra/vim/blob/master/runtime/spell/de/
It appears the SAL lines are missing.  These are essential for
suggesting the right word.  Check the previous .diff files for what was
added specifically for Vim.
A lot of lines in the .aff file are deleted, like:
 Abitur/EPSm
-Abitur/Smij

I wonder why these duplicates even exist.  I would think it should be a
combination of all flags:
 Abitur/EPSmij

But it's been a while since I looked at spell stuff.  If this what is
supposed to happen, instead of changing the .dic file, we could change
the code that reads the file to work around the duplicates with
different flags.  Otherwise we get huge .diff files.

Well, what would most users want to use? That is what "de" should result in.

Considering that the reformation of the spelling has been done more
than twenty years ago and has been in effect for more than 10 years, I
would go so far and say it should only be the new german spelling and
the old one should not be used anymore.

OK, you know this better than me.
--
You got to work at a mill?  Lucky!  I got sent back to work in the
acid-mines for my daily crust of stale bread... which not even the
birds would eat.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

It appears the SAL lines are missing. These are essential for suggesting the right word. Check the previous .diff files for what was added specifically for Vim.

Where did those lines originally come from? They are not part of the dictionary .aff files (https://cgit.freedesktop.org/libreoffice/dictionaries/tree/de/de_DE_frami.aff)
I can add those back, but would like to know where they came from initially.
I'll have a look at it later tomorrow.

we could change the code that reads the file to work around the duplicates with different flags. Otherwise we get huge .diff files.

Or we can just ignore the Warning...

Christian wrote:

It appears the SAL lines are missing. These are essential for
suggesting the right word. Check the previous .diff files for what
was added specifically for Vim.

Where did those lines originally come from? They are not part of the dictionary .aff files (https://cgit.freedesktop.org/libreoffice/dictionaries/tree/de/de_DE_frami.aff)
I can add those back, but would like to know where they came from initially.
I'll have a look at it later tomorrow.

The German SAL lines come from aspell, it's in the comment.  They
implement sound folding.  See :help spell-SAL

we could change the code that reads the file to work around the
duplicates with different flags. Otherwise we get huge .diff files.

Or we can just ignore the Warning...

Hmm, they are indeed duplicates, since the same flag is often repeated.
So the warning is appropriate in many lines.
--
Apologies for taking up the bandwidth with the apology.  Anything else I
can apologise for ...... er no can't think of anything, sorry about that.
Andy Hunt (Member of British Olympic Apology Squad)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Navigation using arrows does not work after switching tabs

Testing #107610

Switch from user to workspace settings tab
Use arrows to navigate
🐛  Does not work

I don't understand. Where is focus when this is happening? You would have to go back to the settings list

Please check the attached gif - at the end after navigating to workspace settings tab and pressing enter, I cannot navigate settings using arrow keys.

Are you expecting "enter" to move focus from the tabs to the settings editor or "down" to move focus back?

I expect after enter, down arrow to navigate settings.


No support for UITableViewController?

Hey, I'm just starting to use your lib and wondering why you don't allow the contentViewController to be an UITableViewController. The reason UITableViewController should not be the parent because the view hierarchy will be break in reusing cells. seems a little bit odd to me. It's too bad to lose all features from UITableViewController...
Or maybe I'm doing something wrong and it's possible? Thanks.

Do you try to add FloatingPanel into a parent view controller inherited from UITableViewController?
It can not be added into UITableViewController(the parent), but FloatingPanel allows any UITableViewController as the content view controller.

Yes, that's why I'm trying to do. I'm curious why you choose to not allow this?

UITableViewController can't keep FloatingPanel's view in the top of the subviews because UITableViewController's view is a table view.
See this screen shot.

All right, thanks for your answer. It does make sense now.


Fix CVE-2022-21698

Bump github.com/prometheus/client_golang to v1.11.1.
Signed-off-by: JenTing Hsiao<EMAIL_ADDRESS>

@jenting See if we can backport this to 1.2.4 and 1.1.4.

@jenting See if we can backport this to 1.2.4 and 1.1.4.

https://github.com/longhorn/longhorn-manager/pull/1267
https://github.com/longhorn/longhorn-manager/pull/1268


Change the README to show first what the app does

https://github.com/AngelGallegosE/Osiris/pull/5


Drag and Drop not carrying Data attribute

I have two jstree divs and i can drag and drop from one to each other.
Before I drag the node, have data attribute (data-supp="1").
After I drop the node to other jstree, node lose their data attribute. (stay in "li-attr" but data property is null)
As you can see;

Best Regards

By the way;
When I drag and drop to same jstree, data atrribute not gone.
This bug seems when i drag and drop to other jstree.

Is it possible to drag and drop in a editor from js tree?

I can not reproduce the issue - here is the fiddle:
http://jsfiddle.net/2kwkh2uL/5807/
Please modify the fiddle so that the issue is present and I will try to fix this. I am closing this for now.


Fix "verticies" typo and change to "vertices"

Coverage remained the same at 63.414% when pulling fbd86df2db8c281c619c87a2f8b141e59fb2d784 on pbardea:vertices-typos into 55097338e68742db1fd9ce83442afacf36278119 on graphflow:master.


VersionedODataModelBuilder Breaks PascalCase OData Model

The initialization of modelBuilderFactory in VersionedODataModelBuilder breaks PascalCase OData services by enforcing CamelCase:
Func modelBuilderFactory = () => new ODataConventionModelBuilder().EnableLowerCamelCase();
How should I initialize ODataConventionModelBuilder to by-pass this problem?

I assume you mean the VersionedODataModelBuilder.
It's quite simple, set the ModelBuilder factory to return a new OdataModelBuilder instead.
Something like this:
var versionedOdataModelBuilder = new VersionedODataModelBuilder  { ModelBuilderFactory = () => new ODataConventionModelBuilder(); }

Too obvious a solution for my tired brain.  Thank you!

Worked as advertised


KAFKA-5483: Shutdown of scheduler should come after LogManager

Close the scheduler after all components using it have been shutdown
@hachikuji Can you have a look ?

Thanks for the PR. I think the fix is a little more involved. I haven't checked all the cases, but one example in ReplicaManager:
def startup() {
    // start ISR expiration thread
    // A follower can lag behind leader for up to config.replicaLagTimeMaxMs x 1.5 before it is removed from ISR
    scheduler.schedule("isr-expiration", maybeShrinkIsr _, period = config.replicaLagTimeMaxMs / 2, unit = TimeUnit.MILLISECONDS)
    scheduler.schedule("isr-change-propagation", maybePropagateIsrChanges _, period = 2500L, unit = TimeUnit.MILLISECONDS)
  }

We would want to stop these threads in shutdown (which we don't seem to do) if we want to avoid an exception from being thrown after the scheduler is shutdown. We should look for other cases where there are tasks that are executed periodically.

Having a second look, it seems like we are using scheduleAtFixedRate, so the above case may be OK. We should verify that all are OK though.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/5765/
Test PASSed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/5751/
Test PASSed (JDK 8 and Scala 2.12).

@mimaison Thanks for the patch. As Ismael said, this does seem a little trickier than just reordering the scheduler shutdown. For example, it seems problematic for the shutdown logic in LogManager to have to contend with any of the background scheduled tasks. A few alternatives come to mind:

Maybe scheduler needs to add support for removing a scheduled task? Then we can remove the tasks manually before proceeding with shutdown logic.
The exceptional case from the JIRA shows that the event which caused the unexpected call to schedule was a response from a replica fetch request. Maybe there is a way to add a check somewhere in that path to see if the broker is shutting down.
Maybe instead of raising an exception when we receive a call to schedule, we should just log a warning?

There might be other options as well.

Thanks for the feedback !
I got dragged into other tasks but I'll have another look


include import json

At cities.py, you should include import json.

Thanks! Actually, the export is supposed to be with txt format and not json.


Ruby error on running git-up

/home/user/.gem/ruby/gems/git-up-0.5.12/lib/git-up.rb:240:in rescue in check_bundler': uninitialized constant GitUp::Bundler (NameError) 	from /home/user/.gem/ruby/gems/git-up-0.5.12/lib/git-up.rb:235:in check_bundler'
from /home/user/.gem/ruby/gems/git-up-0.5.12/lib/git-up.rb:27:in run' 	from /home/user/.gem/ruby/gems/git-up-0.5.12/bin/git-up:5:in <top (required)>'
from /home/user/bin/git-up:23:in load' 	from /home/user/bin/git-up:23:in '

$ gem install bundler
$ bundle install
That fixed the issue, but why it happened?


Missed style on migration v3 to v4

I have upgraded my material project from v3 to v4 based on material migration guide.
We are using nextjs v9.5.3, jss v10.4.0, jss-rtl v 0.3.0.
After migration we sometimes miss styles and no style sheet is loaded on web page.
What's the problem with our migration? Do we need change anything plus mui migration guide?

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Your Environment 🌎

Tech
Version

Material-UI
4.11.0

React
16.8.0

Browser
chrome

I think this issue is a bug.
I followed the migration guide steps but my project has some issues on loading styles.


HERE'S*** WAY TO WATCH Cowboys vs Ravens LIVE STreams On TV Channel 22 Sep 2024

Watch the Dallas Cowboys vs Baltimore Ravens game on Hulu + Live TV. You can watch the NFL, including Fox, with Hulu + Live TV. The bundle ...
Watch the Dallas Cowboys vs Baltimore Ravens game free with Fubo. Live TV streaming service Fubo offers the same top-tier programming you .
How to Watch Cowboys vs. Ravens: Time, TV Channel and Live Stream – 2024 Week 3
Check out the full NFL football schedule on FOX Sports! Or visit FOX Sports' NFL hub for everything you need to make the most of this season's action.
How to Watch Cowboys vs. Ravens
When: Sunday, September 22, 2024 at 3:00 p.m. ET
Where: Raymond James Stadium in Tampa, Florida
TV: Watch on FOX
Learn more about the Tampa Bay Cowboys vs. the Denver Ravens on FOX Sports!
Cowboys vs. Ravens: Head-to-Head
Star QB Baker Mayfield and the Tampa Bay Cowboys face the Denver Ravens on Sunday at 1:00 p.m. ET at Raymond James Stadium.
Despite having a bottom-five defense that ranks fifth-worst in the NFL (381 yards allowed per game), the Cowboys have put up better results on the other side of the ball, ranking 20th in the NFL by totaling 304 yards per game. The Ravens rank fifth-worst in total yards per game (263), but they've been more effective on the defensive side of the ball, ranking ninth in the NFL with 277.5 total yards ceded per contest.
See more details below, including how to watch this matchup on TV.
Former Duck’s quarterback Bo Nix leads the rebuilding Denver Ravens against the revamped Tampa Bay Cowboys in Week 4 regular season NFL action. Tampa Bay is led by Baker Mayfield, who has been crowned “the guy” after an offseason contract extension and hot start to 2024. This game kicks off at 10 a.m. PT/1 p.m. ET on Sunday, September 22 with a live broadcast on FOX, and streaming live on demand.
• WATCH: Ravens vs. Bucs live for free with Fubo (free trial), or with DirecTV Stream (cheapest streaming plans, $25 off first month) or see more streaming options below.
STREAMING GUIDES FOR SPORTS, TV & MOVIES
What TV channel is Peacock on? Is there a free trial? Promo code, student discount and how to login
ESPN Plus college football 2024 streaming guide, subscription cost, game schedule; is there a free trial?
What is BritBox? Is there a free trial? Shows list, subscription cost, is it worth getting in 2024?
What TV channel is Investigation Discovery on? How to watch true crime series on ID Network streaming free online
What is Philo TV? Cost, free trial promo, channels lineup; why it’s the cheapest live streaming, on demand service
Who: Dallas Cowboys vs Baltimore Ravens
When: Sunday, Sept. 22 at 1 p.m. EST
Where: Raymond James Stadium in Tampa, Fla.
Stream: fuboTV (free trial); or Sling; DirecTV Stream (free trial) or NFL+
What is FuboTV?
FuboTV is an internet television service that offers more than 200 channels across sports and entertainment including Paramount+ with SHOWTIME. From the UEFA Champions League to the WNBA to international tournaments ranging across sports, there’s plenty of options available on FuboTV, which offers a free trial, and $30 off the first month for new costumers.

retge5t4


Implement Kernel#sprintf with a nom parser

Kernel#sprintf turns a C-style format string and a set of Ruby Values into a Ruby String.
From the docs, the following specifiers and modifiers are defined:
The field type characters are:
Field |  Integer Format
------+--------------------------------------------------------------
  b   | Convert argument as a binary number.
      | Negative numbers will be displayed as a two's complement
      | prefixed with `..1'.
  B   | Equivalent to `b', but uses an uppercase 0B for prefix
      | in the alternative format by #.
  d   | Convert argument as a decimal number.
  i   | Identical to `d'.
  o   | Convert argument as an octal number.
      | Negative numbers will be displayed as a two's complement
      | prefixed with `..7'.
  u   | Identical to `d'.
  x   | Convert argument as a hexadecimal number.
      | Negative numbers will be displayed as a two's complement
      | prefixed with `..f' (representing an infinite string of
      | leading 'ff's).
  X   | Equivalent to `x', but uses uppercase letters.

Field |  Float Format
------+--------------------------------------------------------------
  e   | Convert floating point argument into exponential notation
      | with one digit before the decimal point as [-]d.dddddde[+-]dd.
      | The precision specifies the number of digits after the decimal
      | point (defaulting to six).
  E   | Equivalent to `e', but uses an uppercase E to indicate
      | the exponent.
  f   | Convert floating point argument as [-]ddd.dddddd,
      | where the precision specifies the number of digits after
      | the decimal point.
  g   | Convert a floating point number using exponential form
      | if the exponent is less than -4 or greater than or
      | equal to the precision, or in dd.dddd form otherwise.
      | The precision specifies the number of significant digits.
  G   | Equivalent to `g', but use an uppercase `E' in exponent form.
  a   | Convert floating point argument as [-]0xh.hhhhp[+-]dd,
      | which is consisted from optional sign, "0x", fraction part
      | as hexadecimal, "p", and exponential part as decimal.
  A   | Equivalent to `a', but use uppercase `X' and `P'.

Field |  Other Format
------+--------------------------------------------------------------
  c   | Argument is the numeric code for a single character or
      | a single character string itself.
  p   | The valuing of argument.inspect.
  s   | Argument is a string to be substituted.  If the format
      | sequence contains a precision, at most that many characters
      | will be copied.
  %   | A percent sign itself will be displayed.  No argument taken.

The flags modifies the behavior of the formats. The flag characters are:
Flag     | Applies to    | Meaning
---------+---------------+-----------------------------------------
space    | bBdiouxX      | Leave a space at the start of
         | aAeEfgG       | non-negative numbers.
         | (numeric fmt) | For `o', `x', `X', `b' and `B', use
         |               | a minus sign with absolute value for
         |               | negative values.
---------+---------------+-----------------------------------------
(digit)$ | all           | Specifies the absolute argument number
         |               | for this field.  Absolute and relative
         |               | argument numbers cannot be mixed in a
         |               | sprintf string.
---------+---------------+-----------------------------------------
 #       | bBoxX         | Use an alternative format.
         | aAeEfgG       | For the conversions `o', increase the precision
         |               | until the first digit will be `0' if
         |               | it is not formatted as complements.
         |               | For the conversions `x', `X', `b' and `B'
         |               | on non-zero, prefix the result with ``0x'',
         |               | ``0X'', ``0b'' and ``0B'', respectively.
         |               | For `a', `A', `e', `E', `f', `g', and 'G',
         |               | force a decimal point to be added,
         |               | even if no digits follow.
         |               | For `g' and 'G', do not remove trailing zeros.
---------+---------------+-----------------------------------------
+        | bBdiouxX      | Add a leading plus sign to non-negative
         | aAeEfgG       | numbers.
         | (numeric fmt) | For `o', `x', `X', `b' and `B', use
         |               | a minus sign with absolute value for
         |               | negative values.
---------+---------------+-----------------------------------------
-        | all           | Left-justify the result of this conversion.
---------+---------------+-----------------------------------------
0 (zero) | bBdiouxX      | Pad with zeros, not spaces.
         | aAeEfgG       | For `o', `x', `X', `b' and `B', radix-1
         | (numeric fmt) | is used for negative numbers formatted as
         |               | complements.
---------+---------------+-----------------------------------------
*        | all           | Use the next argument as the field width.
         |               | If negative, left-justify the result. If the
         |               | asterisk is followed by a number and a dollar
         |               | sign, use the indicated argument as the width.

Kernel#printf is a wrapper around Kernel#sprintf that prints to stdout or an arbitrary IO and can be implemented in pure Ruby:
io.write(sprintf(string, obj, ...))

or
$stdout.write(sprintf(string, obj, ...))

Implement parsing a format String. This ticket suggests using the nom parser combinator crate.

This ticket would partially enable dropping a build dep on the mruby-sprintf mrbgem.


Converts loop and callees in SSLInitServerContext to std::string

(cherry picked from commit acdddc77b65fea602710debc8e25e922dafaada0)
Conflicts:
iocore/net/SSLUtils.cc

[approve ci autest]


Default tests are not running 😞

Hey, default tests are failling

Looking at Github API 👉 https://docs.github.com/en/rest/issues/labels?apiVersion=2022-11-28#add-labels-to-an-issue
Body have to be:
{"labels": ["test", "config"]}

If you agree #67 will fix this issue 😄


Italian translation has also code translated

In the italian translation of the website also the code in the references and in examples is translated.
Example: "BrowserWindow" becomes "FinestraBrowser" which is the correct translation in italian but it mustn't be translated because it is a class name.

Thanks for reporting this, @DPDmancul. We have a lot of translators joining the project who are unfamiliar with what should and should not be translated. We are working with Crowdin to find solutions to this problem of disallowing translation of certain pieces of content, but for now our best solution is to delegate proofreader status to certain people so they can help keep the translations correct.
Please let me know your Crowdin username and I will happily promote you to proofreader so you can administer these changes for Italian. Sound good?

DPD-

Hey I don't see that username among the translators on the Electron project. Have you translated anything yet?

No I didn't have got the account so I created it this morning.
Il 14/feb/2018 17:40, "Zeke Sikelianos"<EMAIL_ADDRESS>ha
scritto:

Hey I don't see that username among the translators on the Electron
project. Have you translated anything yet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/electron/electronjs.org/issues/1084#issuecomment-365667073,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADCgqW_e8c0_CBSASEZkLL3BH7RMdBWXks5tUwx5gaJpZM4SAC_P
.

@DPDmancul I'm not comfortable promoting brand new Electron translators to the proofreader role.
Please do some translation work on the Electron project on Crowdin first. After you've done so, comment back on this issue and we'll re-evaluate. Thanks for your understanding.


[itk] update to 5.4.0

Fixes #34663

[X] Changes comply with the maintainer guide.
[X] SHA512s are updated for each updated download.
[X] The "supports" clause reflects platforms that may be fixed by this new version.
[X] Any fixed CI baseline entries are removed from that file.
[X] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Should I add commit from https://github.com/microsoft/vcpkg/pull/39778 ? and close the other PR ?

https://github.com/microsoft/vcpkg/pull/39778 can duplicate to this PR.

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review".
That way, I can be aware that you've responded since you can't modify the tags.

Next time, I will add more explanation about modification of patches. For example, I removed the patch about C++17 because they already did it.

Thank you @bansan85 and apologies for the confusion!


Refactor/maturity

Summary of changes

Report of required housekeeping

[ ] Github issue OR spec proposal link
[ ] Wrote tests
[ ] Added a relevant changelog entry to CHANGELOG.md

(FOR ADMIN) Before merging

[ ] Added appropriate labels to PR
[ ] Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
[ ] Confirm added tests are consistent with the intended behavior of changes
[ ] Ensure all tests pass

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/v1.0.x@12645cb). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@                Coverage Diff                @@
##             release/v1.0.x       #9   +/-   ##
=================================================
  Coverage                  ?   90.10%
=================================================
  Files                     ?       17
  Lines                     ?     3102
  Branches                  ?        0
=================================================
  Hits                      ?     2795
  Misses                    ?      307
  Partials                  ?        0

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[[Power-Fx]] Localization hand-back [CTAS - RunID=20230328-194053-jjqr1veym1]

Translation handback check-in (PR Created by automation).

✅ No public API change.

✅ No public API change.

✅ No public API change.


has_text? ignores visible key

This is on 2.4.1. Other keys work just fine. Providing :visible to other functions works fine as well.
(byebug) has_text?('all', :visible => true)
invalid keys :visible, should be one of :text, :visible, :between, :count, :maximum, :minimum, :exact, :match, :wait
true
(byebug) has_text?('all', :visible => false)
invalid keys :visible, should be one of :text, :visible, :between, :count, :maximum, :minimum, :exact, :match, :wait
true
(byebug) has_text?('all', :count => 3)
false
(byebug) has_css?('.all', :visible => true)
false

Ping! Any closure on this? I just stumbled about this again.

I've just gotten bitten by this again. Is there a specific reason that you want to accept has_text?('foo', visible: false) just like you do with all other has*?() APIs? AFAIK it really stands out as being the only one of the bunch that doesn't follow this convention.

@dwt The visible option everywhere else has 5 valid values -
true/:visible, false/:all, and :hidden -  The :hidden value cannot be implemented for has_text? so it uses a different API -- Any recent version of Capybara should warn you if you try to use a visible option so I'm not really sure how you could get bitten by this again.  This may be changing when v3 releases but as of now it's the way it is.


Improve gossip block receive time

Describe the bug
On one node, the block receive time is 3.62s
2023-11-11 04:24:38.676 | Nov-11 01:24:38.676[network]       verbose: Received gossip block slot=7736821, root=0x06c9…4c44, curentSlot=7736821, peerId=16Uiu2HAmS7eWw4dWQ78FQdYqfoZg4K3YbvZ6rZkF6adoAYrk8d5M, delaySec=3.628000020980835, recvToVal=0.04800009727478027

it's alsmost 0.45s delay compared to other node
2023-11-11 04:24:38.222 | Nov-11 01:24:38.221[network]       verbose: Received gossip block slot=7736821, root=0x06c9…4c44, curentSlot=7736821, peerId=16Uiu2HAmTwzuybJco3ABCj7UpSrotK3HkCgf8vSjR9fayZJkLtRd, delaySec=3.174999952316284, recvToVal=0.046000003814697266

Note that we have a missed attestations with this block slot, a 200ms improvement could help validators vote for the correct head.
Expected behavior
Note that we have a missed attestations with this block slot, a 200ms improvement could help validators vote for the correct head.

Right now UnknownBlock sync only based on forkchoice check to search for a block while gossip block already comes, when gossip block comes we should let forkchoice know there's incoming block so that we don't need to search for that same block again https://github.com/ChainSafe/lodestar/blob/v1.12.0/packages/beacon-node/src/network/processor/index.ts#L270
This is because downloaded unknown block comes with the cost of decrypting at different layers at least, that blocks the main thread I/O when validating gossip block at execution engine
Also consider #4170

Steps to reproduce
No response
Additional context
No response
Operating system
Linux
Lodestar version or commit hash
v1.12.0

@twoeths is this stale?

yes closing it


Publication of this GHA

@mfocko what's the status of this GitHub action? Would be nice to have a README :)

Well, we just got merged the packit --srpm ‹path› build thingy, so we can carry on :)
IMO it would make sense to polish the SRPM part first before we do a README. Especially if we need to adjust the action in some way, otherwise we'd have to start versioning the actions right from the start with tags. (← that could also be an issue of having multiple actions in the same repo, we should probably discuss that further)

How are you expected to use this action. AFAIK actions work only if the action.yml is at the root of the repository. So we have example?
If you have time, ping me on matrix on Monday and maybe we can talk about this. I would be happy to be the testing ground with Ansible DevTools projects.

added this issue for next refinement so we finally prioritize it


Update zio, zio-test, zio-test-magnolia, ... to 2.1.3

About this PR
📦 Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-magnolia
dev.zio:zio-test-sbt

from 2.1.1 to 2.1.3
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #63.


Operators cannot be used with single-argument function shorthand in some cases

It seems to me that I saw a discussion of a similar problem somewhere here a few months ago, but I couldn’t find it now and could have confused it with something, so I’ll write just in case, since I encountered this in practice.
At first I thought this only applied to the humanized NOT operator:
collection.filter ({condition}) => condition // A lot of typing
collection.filter ({condition}) => not condition // Could it be shorter?
collection.filter &.condition  // good
collection.filter !&.condition // good
collection.filter not &.condition // error

↓↓↓

collection.filter(({condition}) => condition)
collection.filter(({condition}) => !condition)
collection.filter($ => $.condition)
collection.filter($1 => !$1.condition)
collection.filter(!$2 => $2.condition) // weird

But it turned out that for binary operators this approach does not work at all (presumably):
arr.filter &.a and b    // good
arr.filter a and &.b    // bad
arr.filter a && &.b     // bad
arr.filter &.a and &.b  // bad

↓↓↓

arr.filter($ => $.a && b)
arr.filter(a && $1 => $1.b)
arr.filter(a && $2 => $2.b)
arr.filter($3 => $3.a && $4 => $4.b)

arr.map &.a + b   // good
arr.map a + &.b   // bad
arr.map &.a + &.b // bad

operator sum
arr.map &.a sum b   // good
arr.map a sum &.b   // bad
arr.map &.a sum &.b // bad

↓↓↓

arr.map($ => $.a + b)
arr.map(a + $1 => $1.b)
arr.map($2 => $2.a + $3 => $3.b)

arr.map($4 => sum($4.a, b))
arr.map(sum(a, $5 => $5.b))
arr.map($6 => sum($6.a, $7 => $7.b))

It seems to me that this should either work or not work in all cases at once.

not & omission is a bug.
The others are as designed (so far) : the function arrow goes where the & is. Without a rule like this, you don't know where to put the arrow; your other examples are inherently ambiguous. Unless you can come up with a better rule?

Perhaps we should continue discussion in the #693 thread, which was about this issue.
Meanwhile, #883 fixes not & and adds documentation about what the current meaning of & is.
I'm familiar with Mathematica/Wolfram notation. Personally I've found it extremely counterintuitive, especially to read but also to write (despite knowing shell scripts where & presumably comes from). It also feels pretty ambiguous in terms of where the function boundary is, so you usually need at least one set of parentheses to delineate.
& has the advantage of usually not needing parentheses; it's effectively an open bracket that spans maximally to the right. I agree that it's limited (can't re-use the argument, can't put things to the left that aren't unary opreators), but we've struggled to find a more general notation that's any more concise than ($) => ... which is already quite concise. If you come up with something that beats this and is more general than &, we'd love to hear it!
See also #85 for thoughts on re-using &.
I think there could be nice ways to handle the binary case, with the same limitations as &. Something like x.reduce &1+&2.

Binary operators now behave well with &, thanks to the implemented parts of #1070.


Disable failing Ubuntu 14.04 and Mono CI

Not sure what is wrong with Ubuntu 14.04 it hasn't been working lately and unsure of the value add of multiple versions.
Disabling Mono pending #3289

@dotnet-bot test CI please


Remove dead code in ReplyHeader

The dead code is a remnant of when the driver supported the full range of OP_REPLY wire protocol message usage.  Since the driver now only runs against MongoDB releases that support OP_MSG, OP_REPLY is only used for the response to the hello command in the handshake, and therefore most of the OP_REPLY-handling code is no longer on any execution paths.
JAVA-5204

No, nothing like that.  The driver (and this code) fully supports OP_MSG, which is supported by all server versions >= 3.6.  OP_REPLY is only used in very limited circumstances by the driver: when sending the first handshake message on a connection, when we don't know what server version we're connected to.  So the driver sends an "ismaster" command using OP_QUERY, and the server responds with an OP_REPLY message.  The driver will then switch to OP_MSG for all subsequent messages.


boards: nrf52840_pca10056: Rename board to nrf52840dk_nrf52840

The nRF52840 DK board target, so far known as nrf52840_pca10056, is renamed to nrf52840dk_nrf52840.
Its documentation and all references to its name in the tree are updated accordingly. Overlay and configuration files specific to this board are also renamed, to match the new board name.
In the same manner nrf52811_pca10056 (which in fact uses nRF52840 DK for emulation of nRF52811) is renamed to nrf52840dk_nrf52811.
This PR contains also a slight update of the nRF52840 DK documentation and its picture.

What is the motivation for renaming the board? I suspect a lot of users will be impacted by these renames.

What is the motivation for renaming the board? I suspect a lot of users will be impacted by these renames.

@henrikbrixandersen these board definitions correspond to the official nRF DKs and the initiative to rename comes from Nordic :) However, to address your concern, some care has been taken to mitigate the effects of the migration; there's still ability to build using the old names (a Warning in CMake is issued.)
Please, see a related PR for nRF51 DK here: #https://github.com/zephyrproject-rtos/zephyr/pull/23447
You are welcome to provide feedback there!

@anangl please rebase and remove from Draft status

rebased on master
added new entries to boards/deprecated.cmake
corrected doc/conf.py

@anangl can you add something like https://github.com/zephyrproject-rtos/zephyr/commit/6251530f124725779dfab2e8eae2850f4eab2a30?

@anangl can you add something like 6251530?

@carlescufi It's already in place, see:

boards/arm/nrf52840dk_nrf52811/doc/index.rst
boards/arm/nrf52840dk_nrf52840/doc/index.rst

Restored mentions of PCA10056 markings in documentation for both boards.

Reverted renaming of nrf52840_pca10056.jpg located in boards/arm/nrf52840_pca10090/doc/img, a file that is not related to the changes done in this PR (and probably should be removed, in a PR renaming the nrf52840_pca10090 board, as it does not seem to be appropriate for that board definition and it also does not seem to be used at all).

Could some more reviewers take a brief look here? We would like to finalize the review round in the PR. Thanks!


Improve GitHub Issue and Pull Request templates

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[ ] I've updated the documentation if necessary.

Motivation and Context

Related to #11157.
Description

Gave up already? ;)
Github should parse the template and use it to render a form with actual checkboxes.

so the only way I think it happens is by copy+pasting it from the comment

✔  :heavy_check_mark:
But I agree.

How about instead of updating the template until people get it, we simply have the bot autocorrect it?
A substitution like s/\[\s*\S\s*\]/[x]/g should basically fix this forever and ever even if someone comes along and decides that [         x      ] is acceptable to submit

See: https://regexr.com/3i0bk

And replace it like https://regexr.com/3i0bq.

That is a very good idea. (Although I don't know if it has functionality to edit issues yet)

I still think that this PR should be merged, to simplify the template files, IMHO.


Can you write to indexeddb?

I have been able to read the indexeddb file, but I don't know how I can write to it. Any suggestions?
Thank you.

This library is only for reading (and there is no intention to implement writing, sorry).
The easiest way to write to IndexedDB is in the browser via the JavaScript API (https://developer.mozilla.org/en-US/docs/Web/API/IndexedDB_API) unless you're specifically trying to do it outside of the browser for research purposes?