Tasks waiting even if some tasks have already been assigned resources

This is more like a feature request/clarification rather than an issue.
Currently, I've been experiencing the situations like this:

Due to the nature of my experiment, I want to run an experiment for, let's say, 20 times, for the assessment of stability. But if I set it to be 20, I have to wait for a long time before I see any of my tasks start running. That's terrible if there is a problem in the experiment (need to wait for a long time to see it failed).
What I want to see, is that some instances have already started running, or have succeeded, even though other instances have been waiting. I know this could be a problem, because in some situations there could be need for synchronization (but in case of that, the user should be aware to hang the job). Another solution is to add a minTasksReady, to let the user decide.

Framework launcher in PAI uses gang scheduling, so only when all tasks are ready will they start to run.
Maybe you can consider to submit 20 jobs instead of 20 tasks.
/cc @yqwang-ms

Launcher supports to enable/disable gang scheduling per job, so could PAI restserver can also let user make his choice? @abuccts

https://github.com/microsoft/pai/blob/b94e63894bc831a2a9ddb421c2263e3f8f0f51b7/subprojects/frameworklauncher/yarn/src/main/java/com/microsoft/frameworklauncher/common/model/PlatformSpecificParametersDescriptor.java#L83

@ultmaster - does parallel submit 20 jobs serve your problem?

Yes. My workaround is to write a script to submit multiple jobs in batch. Working like a charm :)

Yes. My workaround is to write a script to submit multiple jobs in batch. Working like a charm :)

why not use NNI?


Default metadata initialization function.

Added new environment variable swan_default_metadata_db.
Thanks to it it is possible to have default metadata constructor.

run integration tests


swaggo is not performing well

Either downgrade or change the library

Or Collapse cache would be set as Default so API will work faster


path names and indexing

right now vg index is run on every chromosome.  --path_name argument only used to infer if chromosome prefixed with chr or not.   fine for now, but we'll eventually need to accept arbitrary names

I suppose I could run vg view [reference_graph.vg] -j | jq .path | jq 'map(.name)'. And then parse out the path names from there to infer the path names. If there is discordance between the path names the user provides and the path names that are contained within the reference graph, then an exception should be thrown.

was also thinking something along the lines of accepting a file like
http://genome.ucsc.edu/goldenPath/help/hg19.chrom.sizes
for chromosome information.
On Tue, Dec 6, 2016 at 4:07 PM, Charles Markello<EMAIL_ADDRESS>wrote:

I suppose I could run vg view [reference_graph.vg] -j | jq .path | jq
'map(.name)'. And then parse out the path names from there to infer the
path names. If there is discordance between the path names the user
provides and the path names that are contained within the reference graph,
then an exception should be thrown.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil-vg/issues/57#issuecomment-265272538,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2_7nfxksHUqtctipP_PjkS43s_96Cfks5rFc6dgaJpZM4LF3sV
.

Somewhat cleaned up in #92.  Will close for now and re-open new issue if ever want to add option to automatically treat all paths as reference.


It still work fine. But left search bar can't show any users name and Search anymore..

I have been running this program for more than two months.
and I appreciate the author’s amazing idea.
It was always working fine.
But recently, the left search bar can not show user's name and Searching anymore..
So I have to change the user's ID from URL for looking someone's data records. (As shown below)

Maybe facebook change the API recently, so it can not link the user's name?
or just only I have this problem... is there anyone have the same problem with me?

Been running this for over an year now. Worked near flawlessly.
Yes, friends list won't load anymore but changing uid from url still works.
Maybe as a result of "App Review Process"?

As for the uid workaround, I use this script to make things easier. (Of course you can bookmark users)
greasyfork[DOT]org/en/scripts/38824-get-facebook-uid
And is it just me or facebook have stopped including "lat" from pull requests for friends that we don't actively chat on fb? In other words, most people(not all) that I have not contacted in weeks/months does not show any last seen time on fb web ui nor mobile messenger therefore this app itself does not work for such people. (Pretty sure these people hasn't blocked/ignored my conv. or unfriended)

please can you explain more how to solve this problem


Select trigger is not aligned correctly in IE 11

IE 11
demo

possible duplicate of https://github.com/vitmalina/w2ui/issues/1268

Closing issue: no response, no reproducer, most likely duplicate


Is startup_pane a string?

https://json.schemastore.org/tmuxinator.json specifies that startup_pane is a string.
Your documentation shows it as an integer:
# Specifies (by index) which pane of the specified window will be selected on project startup. If not set, the first pane is used.
# startup_pane: 1

There is absolutely nothing that can be done about that here.
I opened an issue at https://github.com/SchemaStore/schemastore/issues/3308. If it doesn't get fixed by iteself in a few weeks, I might fix it myself.

Fixed by https://github.com/SchemaStore/schemastore/pull/3311


NavigationMenu not working in Astro

Hi,
I tried starting up a quick project with Astro React, Tailwind and of course shadcn-ui. When I copy the NavigationMenuDemo from
https://ui.shadcn.com/docs/components/navigation-menu and import it into my layout (or index.astro) it only highlights the buttons on hover, no dropdown appears.
Code:
https://github.com/polooner/diverse-stem

Have you tried hydrating it? By default, all library components in Astro are not interactive.

Have you tried hydrating it? By default, all library components in Astro are not interactive.

I am new to Astro and did not know it has a client directive, adding client:load to the component solved it.

Have you tried hydrating it? By default, all library components in Astro are not interactive.

I am new to Astro and did not know it has a client directive, adding client:load to the component solved it.

do you have any issues with hydration? I am currently battling it since I am using same component and seems like it won't go away and has something to do with either radix or shadcn


Cleanup viewBox feature request

I would like to propose a feature called something like: simplifyViewbox that updates the viewBox attribute AND all the child elements. I am more than happy to submit a PR to this. I just wanted to see if there was any interest / get some insight into the value of this request.
The use case for this is if you have a really ugly SVG file like:
<!-- This is very unreadable -->
<svg viewBox="0 0 70591.91181 35296.6681">
  <rect x="11294.48" y=" 3529.525" width="42354.3" height="28237.334" />
</svg>

<!-- This is essentially the equivalent to the above one, but easily editable by hand -->
<svg viewBox="0 0 500 250">
  <rect x="80" y="25" width="300" height="200" />
</svg>

I think the API would allow the user to specify a height AND/OR width value.

If both are specified then that would be destructive to the aspect ratio.
If just a height is specified then the width will be calculated based on the already existing aspect ratio.
If just a width is specified then it would be the opposite if just a height is specified.

Very interested in hearing any thoughts you have about this.

It's a known optimization technique, but unfortunately it's hard to automate, especially in more complex cases. E.g. how to you find these numbers? You need to scale by a certain value, but how do you know this value and that it's good, other then trying many possibilities one by one which is ineffective and slow? See #293.


Memory leak using downloader(), what am I missing?

I have a part of my app where I use a ViewPager to load various full screen images.  It was working fine before, but I needed to add ability to use authentication headers, so I found this library.  My problem is if I use the downloader() method, my app ends up running out of memory because all the images are kept in memory, but if I don't use the downloader() then memory is released after the next image is displayed.  Here is a sample of code:  Am i missing something?
Not working:
OkHttpClient.Builder clientBuilder = new OkHttpClient().newBuilder();
OkHttpClient okHttpClient = clientBuilder.build();

Picasso picasso = new Picasso.Builder(container.getContext())
                    .downloader(new OkHttp3Downloader(okHttpClient))
                    .build();

picasso.load(pages.get(position))
                    .into(photoView);

Working:
Picasso.with(container.getContext())
                    .load(pages.get(position))
                    .into(photoView);

I'm not sure if this is a workaround, or best practice.  But I was able to get around this issue by initializing the picasso object in the constructor of the PagerAdapter instead of creating a new one in each call to instantiateItem()

You should use a single Picasso instance for your whole app, not just for the PagerAdapter.

But yes the problem is you were creating new instances over and over instead of reusing them.

In case if anybody is using multiple instances of Picasso then don't forget to call picasso.shutdown() when no more required. You may call this method in onDestroy() of activity.


[Paginator] Unclear in docs about how to set language

Bug, feature request, or proposal:
I want to set the paginator to german language. How can I do it???

See #5573 Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are Stack Overflow, gitter, mailing list, etc.

@katrina-engelbrecht the docs briefly mention it at the bottom of the paginator overview, but I don't feel like it's entirely clear.  Here's an example
https://plnkr.co/edit/Zy0kOh78CBelkBH8VEug?p=preview
Note that you must be using the latest builds or wait until the next beta release since MdPaginatorIntl was just recently exposed.

I'll add something to the docs similar to the example provided by @willshowell so its clearer about how to use Angular's injection dependency to do this

@andrewseguin maybe additional docs will show a better way but I feel like it's clunky to override all the properties or extend the whole class when you're trying to change the text for all 4 labels. I'm wondering if MdPaginatorIntl could benefit from a constructor,
export const customPaginatorIntl = new MdPaginatorIntl(
  'Items per page',
  'Next page',
  'Previous page',
  (page, size, length) => `Page ${page} of ${length / size}`
);

@ViewChild(MatPaginator) paginator: MatPaginator;
ngOnInit() {
...
this.paginator._intl.itemsPerPageLabel = 'My translation for items per page.';
...
}

seriously this whole thing has to be reworked. its just insane how much code you need to translate 4-5 strings.

Hi there, I've something that worked in past (angular 8.x) to translate paginator strings:
import { MatPaginatorIntl } from '@angular/material/paginator';
// PAGINATOR TRANSLATIONS
import { PAGINATOR } from '../../../../i18n/languages';

export class CustomPaginator extends MatPaginatorIntl {
  constructor() {
    super();
    this.itemsPerPageLabel = PAGINATOR.itemsPerPageLabel;
    this.lastPageLabel = PAGINATOR.lastPageLabel;
    this.nextPageLabel = PAGINATOR.nextPageLabel;
    this.previousPageLabel = PAGINATOR.previousPageLabel;
  }
}

that stopped working in the new version of material 9.1.3 and angular 9.0.7

Currently if I use that I'm receiving the following error:
ERROR TypeError: Cannot read property 'subscribe' of undefined
    at new MatPaginator (paginator.js:235)

I'm using angular 9.1.0 and material 9.2.0.
It's working for me:
// Custom class:
import { Injectable } from '@angular/core';
import { MatPaginatorIntl } from '@angular/material/paginator';

@Injectable({
  providedIn: 'root'
})
export class CustomMatPaginatorInt extends MatPaginatorIntl {

  itemsPerPageLabel = 'Itens por página';
  nextPageLabel = 'Próxima página';
  previousPageLabel = 'Página anterior';

  getRangeLabel = (page: number, pageSize: number, length: number) => {
    if (length === 0 || pageSize === 0) {
      return '0 de ' + length;
    }
    length = Math.max(length, 0);
    const startIndex = page * pageSize;
    // If the start index exceeds the list length, do not try and fix the end index to the end.
    const endIndex = startIndex < length ?
      Math.min(startIndex + pageSize, length) :
      startIndex + pageSize;
    return startIndex + 1 + ' - ' + endIndex + ' de ' + length;
  };

}

// Import on providers of app module:
providers: [
    ...
    { provide: MatPaginatorIntl, useClass: CustomMatPaginatorIntl}
  ],

I find this problem really intresting,
why it's needed to build a whole component, to change the text of another component, which the main purpose is to not need more code ?
Is there any solution except using: private properties of the paginator, third party implementation of the translation, building yourself the implementation solving the problem ?

@MuMaestro

I find this problem really intresting,
why it's needed to build a whole component, to change the text of another component, which the main purpose is to not need more code ?
Is there any solution except using: private properties of the paginator, third party implementation of the translation, building yourself the implementation solving the problem ?

@marcelo-3d's example is the go. Here's a working example. This method works with almost all the material components. As far as I remember, mat-datepicker has some other implementation.

This is not what I was talking about. Why my language needs a reimplementation of the component ? Why any language other than english should implement extra code to deal with localization ?
I know how to implement code, there are many examples online; I'm asking why I need to add another file to change it ? shouldn't this be possible with a simple input of label ?

Well I see, but this is just a small class. When they implemented it there was no easy way to do the localization of a library that also supports runtime changes. They come up with this. You write it once in the project, can be integrated with any custom localization library and does not need a lot of code.
You want a simplet input, but there are 6 labels.
Do you want 6 different inputs?
<mat-paginator itemsPerPageLabel="Items per page" nextPageLabel="Next page" ....></mat-paginator>

I don't think you want this.
Do you want a single input?
<mat-paginator [labels]="labels"></mat-paginator>

You will have an object that looks like the service you have to implement in every component you use the paginator.
Do you want to put it in the MAT_PAGINATOR_DEFAULT_OPTIONS so you can configure it with the other options?
You loose the ability of runtime changes.
I think this is the price we have to pay, but with this we can have our cake and eat it too.

Duplicate of #8239


chore(npm): use require.resolve when possible to avoid hard coded mod…

Cleaned up and rebased version of https://github.com/angular/angular.js/pull/13320.
I've tested the affected commands locally, and they all went through.

@mgol I've updated the PR with your suggestions, can you take a look?

LGTM.


Row component with d-- utility classes not working

Issue description
I would like to display a row using the following code:
<Row className="d-none d-lg-block">Foo</Row>

components: Row
reactstrap version #7.1.0
import method es
react version #16.8.2
bootstrap version #4.3.1

What is happening?

The element is not rendered, regardless of the window size.
What should be happening?
It should be rendered as of the lg breakpoint.

Any idea what could be done to fix this?

Never mind, please. All working, please delete this issue if possible. Thanks.


Use correct rails api for data attributes on register to vote promo

The rails API for defining data attributes on a tag doesn't walk through nested
hashes so if we want:
Hi

we might expect to be able to write:
link_to 'Hi', 'http://hi.com/', data: { nested: { attributes: 'please' } }

but that doesn't work, and would instead produce:
Hi

Which is obviously wrong.  It will however translate underscores to dashes so we
are able to use ruby symbols as follows:
link_to 'Hi', 'http://hi.com/', data: { nested_attributes: 'please' }

and this generates what we want.
I don't think any documents actually use the register to vote promo so it's unlikely this has ever broken in production, but this acts as an example of how to set data attributes for tracking via ruby rather than directly in HTML so it's better to fix it now.

Nice 👍


Support getting coefficient from operator string

This line creates an operator term from a string.
https://github.com/jlapeyre/QuantumOps.jl/blob/a27da4b03d46734f183903a1f170057e046c52e6/src/op_term.jl#L125
We can first split the string on whitespace and parse the first result (if there are two) as a ComplexF64 by default.
This is quite fast if there is no coefficient, so this is not a great efficiency penalty.
We could support other ways to parse the coefficient. For example have a method that takes the type of the coefficient as the first arg.
OpTerm{Pauli}(Int, "3 XXY")

I did some work and left some comments in the code. If and how to support this is an open question.


One-time crash on quit (3.0a)

Operating system or device, Godot version, GPU Model and driver (if graphics related):
OS X 10.12.6 (16G29) on a MacBookPro12,1 (retina).
Cloned git deefc2a63d36b6659630df703a9b98adffb893e9 (3.0, master). Built with scons platform=osx -j4 and executed with ./bin/godot.osx.tools.64.
Issue description:
On my second run, when I quit the editor it crashed (I opened the project manager, selected a project, opened the editor, made some changes, ran both normally and via F5, then quit by clicking the close button, I don't remember if I got the "quit?" dialogue or not). I got a stack trace and the message:
   At: core/vector.h:137.

Attached is the STDOUT and the stack trace.
crashreport.txt
stdout.txt

Had the same crash today. First time I saw this.
Godot version:
a663dbfdd8465d45a03e4880c0d907334236b260
OS/device including version:
macOS 10.13.2
Issue description:
During quit Godot crashed. Happened only once, not reproducible.
Steps to reproduce:
n/a
Minimal reproduction project:
n/a
crash-audio.txt

Should be fixed by #15153


Fix the warning message passed by HiveServer2 JDBC Driver

I think it is necessary to track and fix the Warning passed in by HiveServer2 JDBC Driver. The source of this issue is on the Apache Hive side.

[INFO] --------< io.github.linghengqian:hive-server2-jdbc-driver-thin >--------
[INFO] Building hive-server2-jdbc-driver-thin 1.7.0-SNAPSHOT              [2/3]
[INFO]   from hive-server2-jdbc-driver-thin/pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[WARNING] 2 problems were encountered while building the effective model for org.apache.yetus:audience-annotations:jar:0.5.0 during dependency collection step for project (use -X to see details)
[INFO]

I am working on the current issue.

I forgot about https://github.com/linghengqian/hive-server2-jdbc-driver/issues/1 . What a surprise, I thought it was fixed.


chore: bdd-spring-1573214274 to 0.0.1

chore: Promote bdd-spring-1573214274 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


tests: set _file to avoid <unknown-file> messages

Set _file in test-case modules, to avoid <unknown-file> messages.
Since the module system is importing an anonymous fnOrAttr module, instead of a path type module, it doesn't have enough context to figure out the module's file automatically.
Usually such non-path modules would just inherit their importer's _file, however this doesn't apply when passing modules directly to lib.evalModules since there is no importer to inherit from.
Example test failurues
Adding foo = "" to a test case in tests/test-sources/examples.nix produces the following errors:
Without the change:
error: The option `foo' does not exist. Definition values:
- In `<unknown-file>': ""

With the change:
error: The option `foo' does not exist. Definition values:
- In `/nix/store/ss06gh6ig3nk0pvh1a7n5x1a3g47x91j-test-sources/examples.nix': ""

@mergifyio queue

@mergifyio dequeue

@mergifyio queue

@mergify refresh

@mergify queue


1662 support asynchronous browsing in opc ua adapter

Purpose

Improves OPC-UA browsing in te OPC-UA adapter (see #1662)
Remarks

PR introduces (a) breaking change(s): yes
Requires update of the OPC-UA adapter model
PR introduces (a) deprecation(s): no

PR introduces (a) breaking change(s): yes
Requires update of the OPC-UA adapter model

can we have a short description on how to achieve this or (even better) provide a link to the documentation?


new gradle build system should put apk in /build directory

The old flutter build apk would put the new .apk file directly off the root project build directory and this was very convenient. The new gradle based build system puts it deep down where it is not as easy to find <app dir>/android/app/build/outputs/app-release.apk.
If this issue is accepted and gradle build tool is changed to do the old behavior, make sure to also update the path provided in the docs here:
https://flutter.io/android-release/#building-a-release-apk

Thanks for the bug.  @mit-mit or @jakobr-google would know.

Just want to point out that while the old behavior is more convenient and ergonomic I'm not too familiar with the standard Android development workflow. If most Android devs are already used to finding their .apks in /android/app/build/outputs/ directory then maybe this is an ok default.
On the other hand, for someone with no prior Android experience it would definitely make it easier to find the APK (especially if you're clicking around for it in an attempt to upload it to the Play Store).

We're currently using the standard Android build layout, but I do agree it isn't pretty. Especially because flutter build clean doesn't actually clean anything, currently.


feat: updated readme

The readme is pretty bare right now, I know it's a pain creating them though so I took the liberty of updating it.

Thanks heaps for this @Brisklemonade! I wanted to keep the README minimal (subjective) but this PR forced me to at least pretty it up a bit. Took what I could and updated it.


WebUI, NasUI Enhancement Backlog

July Candidate Items

[ ] Adopt a whole new hyper-parameter chart. @ultmaster
[ ] Improving code and data model structure. @ultmaster
[ ] Adopt a whole new intermediate result chart. @ultmaster
[ ] Refactor trial jobs table. @ultmaster
[ ] Pagination. @colorjam
[ ] Searching by parameters is too difficult to use. (v1.3 feedback) @colorjam
[ ] Log files drawer cannot be closed via clicking overlay area. @Lijiaoa
[ ] Weird spacing, e.g., top margin of "display top 10 trials". @Lijiaoa
[ ] Support metrics other than default in intermediate result chart. #2661 @Lijiaoa
[ ] Weird capitalization, e.g., "Top Maximal trials". @Lijiaoa
[ ] Unable to deal with infinity. (discussed in meeting 07/13) @Lijiaoa
[ ] Show waiting trials in overview page. (v1.0 feedback) @yangou1988
[ ] Update duration and max trial numbers. @yangou1988
[ ] Should we make all columns center aligned? @yangou1988

Backlog
WebUI
Overview

[ ] Log files drawer cannot be closed via clicking overlay area. @Lijiaoa
[ ] Weird spacing, e.g., top margin of "display top 10 trials". @Lijiaoa
[ ] Weird capitalization, e.g., "Top Maximal trials". @Lijiaoa
[ ] Show waiting trials in overview page. (v1.0 feedback) @yangou1988
[ ] Update duration and max trial numbers. @yangou1988

Default metric

[ ] Zooming in is weird.

Hyper-parameters Parallel Coordinates
@ultmaster

[ ] Unable to deal with unsupported search space and unmatched search space (with trials). #2668
[ ] Unable to drag axes. #2642
[ ] Support nested search space. #2555
[ ] Customizable search space axes and metrics axes. #2555 #1483
[ ] The second choice type can not select data from yAxis when search space have 2 choice types. #1398

Trial duration

[ ] Is this chart really useful? Or do users rather want to see the timeline?

Intermediate Result Chart

[ ] Unable to deal with infinity. (discussed in meeting 07/13) @Lijiaoa
[ ] Compare trials: incorrect data. Show 4 lines when selecting 3 trials. Unreproducible. #2568
[ ] Support metrics other than default. #2661 @Lijiaoa
[ ] Lines are weirdly hidden when the chart is zoomed in. @ultmaster
[ ] Customize the scale that is used for a specific metric. #1566 @ultmaster

Trial Jobs Table
@ultmaster

[ ] Trial ID cannot be copied. #2672
[ ] Sort by additionally added keys. #2580
[ ] Collapse all and expand all. #1465
[ ] Separate click event from selection event.
[ ] Pagination. @colorjam
[ ] Should we make all columns center aligned? @yangou1988
[ ] Show finished trial logs. (needs API support from backend)
[ ] Searching by parameters is too difficult to use. (v1.3 feedback) @colorjam

Connection between charts

[ ] When certain trials get filtered out, the table should only show those trials.
[ ] Redirect to corresponding trials in table when certain trials have been selected.

Misc

[ ] Auto refresh doesn't save state.

Code and Development Experience

[ ] No mock API server. Developers have to wait for the experiment to run into a certain state.
[ ] Redundant logic for search space parsing.
[ ] Naming is too complex and confusing. e.g., acc vs accuracy, parameters vs description.parameters, TableObj vs TableRecord.
[ ] Redundant logic for legend and deprecated features, e.g., multi-phase. The support was actually quite limited back then and almost useless now.
[ ] Upgrade to Fluent UI (originally Fabric UI).

NasUI
Known Issues

[ ] Performance issue.
[ ] Cannot focus on the key components, i.e., searched components, in the whole network.
[ ] Support for structures are limited, e.g., recognition of the parameters in Conv2d.
[ ] Scalability to different applications.

Code and Development Experience

[ ] Introducing extra effort to maintain a separate codebase from WebUI.

@ultmaster Could you help update this issue status? Looks like many items had done...

@ultmaster Could you help update this issue status? Looks like many items had done... (again


NPOI 2.5.4 Release Notes

Planned Release Date: July 31, 2021
#589 fileRecoveryPr should not appear after extLst
#563 Word tables change height row of the template
#183 picture tag in content control is written without a namespase in output word document
#582 Table of Contents - Completion
#180 CT_Text.Write throws null exception
#554 ReplaceText may throw null exception
#504 Docx File corruption due to the image wrapping
#402 broken xlsx using conditional formatting..
#558 NPOI.SS.UserModel.RangeType missing namespace
#520 Locale on conditionalcolor scale
#470 ColumnIndex negative if culture is tr-TR
#186 NPOI removes background color from conditional formattings
#315 Can't open default Word template docx file
#597 Relax version bounds for NPOI's dependency on SharpZipLib
#489 make GetCTWorksheet method public

@tonyqus - would it be reasonable to do #597 when you actually publish, making sure sharpziplib >=1.2.0 vs. = 1.2.0 (2.5.3's nupkg is more strict than <= 2.5.2)

5 days later and NuGet still has 2.5.3 as latest version. Could you please update it in there as well? Thanks.

@grumpytm PLEASE READ THIS RELEASE NOTE CAREFULLY. THERE IS A 'Planned Release Date'. Thanks!

@malhotrar I'm planning to setup as sharpziplib >=1.3.1. Don't worry.

I'm not implying anything, I asked cos of this issue: Unable to resolve dependencies. 'SharpZipLib 1.3.2' is not compatible with 'NPOI 2.5.3 constraint: SharpZipLib (= 1.2.0)'.
Got an app and can't update it cos of this issue. That's all.

NPOI 2.5.4 released


Ellipsis documentation fix

Example for ellipsis included a comma but it shouldn't

@calummacuisdean thanks for the PR!
The README is generated and the code examples come from the code comments for the specific helper. You can find the ellipsis helper comments here. Would you update those too so the README will be correct when it's generated?


New version: VariantVisualization v0.3.6

Registering package: VariantVisualization
Repository: https://github.com/compbiocore/VariantVisualization.jl
Created by: @fernandogelin
Git reference: register/VariantVisualization/v0.3.6
Version: v0.3.6
Commit: c92e899ed90a4d978e710ad14ced8645d936fe85
Reviewed by: @fernandogelin
Reference: https://github.com/compbiocore/VariantVisualization.jl/commit/c92e899ed90a4d978e710ad14ced8645d936fe85#commitcomment-33492839

Thanks for the feedback. I'll make those changes.
But why shouldn't I tag a release before registering? My understanding is that the TagBot would create a release but I'm not using that in favor of automated semantic release. That creates the tag, release, and changelog based on my commit messages, it also changes the version on my Project.toml, so I don't need to worry about version numbers or creating releases by hand.

The problem is that if you need to update something, like I suggested now, then you have to delete the tags, since they won't correspond to the release in the registry otherwise.

Bump @fernandogelin, did you want to change the [compat] section?

Closing as stale.


backend/udp: Use a /32 prefix for the flannel0 interface

Description
I discovered that bug #533 still affects my team, since the fix (#576) only applies to the vxlan backend, but the bug is also present in the udp backend. This pull requests fixes the same bug in the udp backend.
Specifically, I found that whichever host was assigned the zeroth subnet (<IP_ADDRESS>/24 out of <IP_ADDRESS>/16 in our test cluster) would not be able to communicate from the host level to the container level of any other host, because the receiving host would think that a message from <IP_ADDRESS> was from the broadcast address. By changing the flannel interface to have 172.18.x.0/32 bound instead of 172.18.x.0/16, this problem was avoided.
I've deployed these changes to our test cluster, and the problem seems to be resolved without breaking anything else.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Since this is a small bugfix, and the udp backend doesn't appear to have any existing unit tests, I'm assuming that I don't need to add any tests. Documentation also seems unnecessary.
Release Note
None required

Thanks @celskeggs


network module: option for traffic in bit/s

At the moment the network module seems to calculate byte/s values for the bandwith values.
It is more common to measure traffic in bit/s.
An option to configure this case would be awesome.
A token like %upspeed-bit% or %downspeed-bit% could be an idea. Otherwise an option in the basic setting of the network module?

I rarely see download speed being measured in bits when it comes to user applications. I'm aware of the difference but to me it wouldn't serve any purpose - other than knowing the max speed.
I do however welcome a pull-request in case you want this implemented.

I'm closing this as a won't fix.

i agree with x70b1, mbps would be great


Installation clarifications

This is currently the biggest blocker for my review in https://github.com/openjournals/joss-reviews/issues/7288.
Installation instructions structure
First and foremost, considering that you give very detailed instructions on how to install your package on several OS, and specifically different MacOS processors, I am missing a top-level overview of the installation process for users who don’t need the detailed instructions. For example, this could be:

The installation of gastli requires Python version 9 or higher, with the dependencies ipython, numpy and jupyter, as well as a Fortran compiler. The gastli package itself can be installed with pip. You will also need to download a zip file of input data required by the package.

Then I would add a warning/note about the processor details, so that people know up front that this is something to look out for.
Only then would I go into detailed installation instructions, where I think you can separate:

The conda environment setup with the installation of Python, its dependencies, and gastli itself - this should be pretty much the same on any OS, provided that a conda distribution is already installed.
The input data download, again the same across all OS.
A Fortran compiler installation - here is where you need to separate between Linux, MacOS with Intel chips and MacOS with Apple chips.

The result of this would be that users who are experienced enough with similar installations can just get the gist of things and move on. You also warn about the processor bifurcation immediately in the beginning, so users on a MacOS will know to be mindful when starting the installation. And finally, it still gives you the space to provide all the details about the different installation procedures.
General comments
Apart from the above, below I am providing a couple of review questions on things that are still unclear to me. If I simply didn't understand something well, I apologize - I will be happy to rectify my comments with any new information you can give me.

You recommend to install Python, but then you proceed to do a new Python installation in a conda environment for gastli. Why? The system will only ever use one or the other as far as I know, which means that you just want the Python installation in the conda environment I believe?
Have you considered providing a conda recipe for the part that requires a conda environment setup? For example something like an environment.yml file that would install all the conda installations at once instead of having to type them manually? Since you have very few requirements, you could also just provide the one line (instead of a full environment.yml file) to create the conda environment with the right dependencies automatically: conda create -n gastli python=3.10 ipython numpy jupyter, and then the line that installs gastli: pip install gastli==0.9.1. If you were to use a environment.yml file, it would be able to run both of these things at once - the two steps are so simple though that the use of such a file is not necessarily warranted.
I also just realized that your gastli pip package already lists requirements, so you might also just put any dependencies in there instead of having the user install them one by one. Or have a conda recipe that includes your pip package, as I mentioned in 2). Either way, it would make the installation easier.
In the section where you describe the installation with Rosetta, you don’t even mention a dedicated conda environment, you just say to “install the following packages in miniconda3” - package installations should always be directed to a specific environment, in your case the gastli/GASTLI_env environment.
You recommend to work with a conda env but you also recommend to install gcc with homebrew instead of conda. Why? There could be a good reason to have a system-wide gcc install for your purposes with homebrew, but at first sight, it seems more logical to contain all dependencies for GASTLI in the same conda env. gcc seems to be installable through conda-forge, have you tried that?
You claim: “The installation of GASTLI on Apple machines with the M1/M2/M3 chip requires Intel emulation with Rosetta.” It would be useful to have a concrete example that triggers the mentioned error so that the user can quickly determine if the Rosetta installation is indeed required for them.
Also, does the above mean that option 2) with the disk image doesn’t actually work on M1/M2/M3 machines?
I am on a MacOS 12.2 Monterey using an M1 and tried to use the disk image solution because it seems so much easier than to set up Rosetta. However, the disk image installation doesn’t seem to be recognized when I run import gastli.Coupling as cpl in an ipython terminal from my gastli conda environment. Did you try an installation like this yourself, is there more to it than just running the installer? For reference, the error message I get:

ImportError: dlopen(/Users/ilaginja/opt/miniconda3/envs/gastli3/lib/python3.10/site-packages/gastli/fortinput.cpython-310-darwin.so, 0x0002): Library not loaded: /usr/local/opt/gcc/lib/gcc/current/libgfortran.5.dylib
  Referenced from: /Users/ilaginja/opt/miniconda3/envs/gastli3/lib/python3.10/site-packages/gastli/fortinput.cpython-310-darwin.so
  Reason: tried: '/usr/local/opt/gcc/lib/gcc/current/libgfortran.5.dylib' (no such file), '/usr/local/lib/libgfortran.5.dylib' (no such file), '/usr/lib/libgfortran.5.dylib' (no such file)

Thanks @ivalaginja for the feedback. I start by addressing comment 8):

I am on a MacOS 12.2 Monterey using an M1 and tried to use the disk image solution because it seems so much easier than to set up Rosetta. However, the disk image installation doesn’t seem to be recognized when I run import gastli.Coupling as cpl in an ipython terminal from my gastli conda environment. Did you try an installation like this yourself, is there more to it than just running the installer? For reference, the error message I get:

This is the message we get if we try to install GASTLI in Apple Silicon with the gfortran native ARM64 installation. This is not explicitly said in the disk image subsection of the installation documentation, but the version I think you need to install of gfortran is the Intel one, not the ARM64. I need to confirm this with the co-author that wrote this sub-section of the documentation. In the meantime, can you try to uninstall your current ARM64 gfortran and install the Intel one? If that does not work, please let me know.
I continue to address the other questions in the following comments.

Great, will do! Thanks for the fast feedback!

I changed the installation page in the documentation to address the suggestions for the structure. The installation page now follows the structure:

overview of requirements
table of content for Fortran installation for different OS
warning for Apple Silicon Users
usual installation with conda and pip. We recommend the users to use these.

Answer to comments 1), 2) and 3):

You recommend to install Python, but then you proceed to do a new Python installation in a conda environment for gastli. Why? The system will only ever use one or the other as far as I know, which means that you just want the Python installation in the conda environment I believe?

Exactly. We recommend the user to use conda or venv environments to avoid conflicts, but it is up to the user to decide this. I have modified that part of the text as "If you decide to create a new conda environment (for example, “GASTLI_env”), [...]" just to clarify that the use of conda is optional.

Since you have very few requirements, you could also just provide the one line (instead of a full environment.yml file) to create the conda environment with the right dependencies automatically: conda create -n gastli python=3.10 ipython numpy jupyter, and then the line that installs gastli: pip install gastli==0.9.1.

ipython and jupyter are not required to use GASTLI, I removed these from the docs. Now the user just needs to 1) create a conda/venv environment (if they choose to) and 2) run pip install gastli==0.9.1

I also just realized that your gastli pip package already lists requirements, so you might also just put any dependencies in there instead of having the user install them one by one.

Yes, this is why the .yml file is not so necessary. numpy, which was mentioned in the documentation before, is already in the requirements. I removed the mention of numpy together with ipython and jupyter.

Answer to comment 4):

In the section where you describe the installation with Rosetta, you don’t even mention a dedicated conda environment, you just say to “install the following packages in miniconda3” - package installations should always be directed to a specific environment, in your case the gastli/GASTLI_env environment.

First the steps to install miniconda3 need to be followed. Then once the user has miniconda3 working with Rosetta, they can install GASTLI as usual. I removed the package part and added a clarification at the end that the user can create a conda/venv environment to avoid conflicts.

Answers to comments 6) and 7):

You claim: “The installation of GASTLI on Apple machines with the M1/M2/M3 chip requires Intel emulation with Rosetta.” It would be useful to have a concrete example that triggers the mentioned error so that the user can quickly determine if the Rosetta installation is indeed required for them.

I added instructions on how to check if the user needs to follow the instructions for Apple Silicon in the very first warning in the installation front page. This includes an example of the error message that we get when trying to install GASTLI in the arm64 architecture.

Also, does the above mean that option 2) with the disk image doesn’t actually work on M1/M2/M3 machines?

No, the disk image solution is an alternative to Rosetta for Apple Silicon users, so it should work on the M1-M3 machines.

Answer to comment 5):

You recommend to work with a conda env but you also recommend to install gcc with homebrew instead of conda. Why? There could be a good reason to have a system-wide gcc install for your purposes with homebrew, but at first sight, it seems more logical to contain all dependencies for GASTLI in the same conda env. gcc seems to be installable through conda-forge, have you tried that?

The installation of gcc/gfortran is quite tricky. If the user chooses to install gcc through conda-forge instead of homebrew, the compilation of the fortran backend done by Python's packages (that is meson and pip in the case of GASTLI) may not work, depending on the OS. We noticed that using homebrew is more successful in these cases, so this is what we recommend.

@ivalaginja I will close this issue once you confirm that you could install it in your machine. I also want to wait for the co-author that wrote the Apple Silicon installation with disk images to clarify this part of the documentation. He'll get back next week.

@lorenaacuna thanks a lot for the detailed answers! I will look at them in detail shorty, it looks like all of my questions were addressed. I'll let you know!

I changed the installation page in the documentation to address the suggestions for the structure. The installation page now follows the structure:
* overview of requirements

* table of content for Fortran installation for different OS

* warning for Apple Silicon Users

* usual installation with conda and pip. We recommend the users to use these.

This is so much better now, amazing! :D

Answer to comment 4):

In the section where you describe the installation with Rosetta, you don’t even mention a dedicated conda environment, you just say to “install the following packages in miniconda3” - package installations should always be directed to a specific environment, in your case the gastli/GASTLI_env environment.

First the steps to install miniconda3 need to be followed. Then once the user has miniconda3 working with Rosetta, they can install GASTLI as usual. I removed the package part and added a clarification at the end that the user can create a conda/venv environment to avoid conflicts.

This is also a lot clearer in the docs now, thank you!

Answer to comment 5):

You recommend to work with a conda env but you also recommend to install gcc with homebrew instead of conda. Why? There could be a good reason to have a system-wide gcc install for your purposes with homebrew, but at first sight, it seems more logical to contain all dependencies for GASTLI in the same conda env. gcc seems to be installable through conda-forge, have you tried that?

The installation of gcc/gfortran is quite tricky. If the user chooses to install gcc through conda-forge instead of homebrew, the compilation of the fortran backend done by Python's packages (that is meson and pip in the case of GASTLI) may not work, depending on the OS. We noticed that using homebrew is more successful in these cases, so this is what we recommend.

Aha! Good to know. Thanks for explaining!

I am currently going through the installation with Rosetta  for a M1/M2/M3 Mac and have a couple of questions and comments:

Twice in the installation instructions, you instruct to "cut and paste" -> do you really mean cut, or do you mean copy and paste? If it's indeed cut, it might be worth to clarify in parentheses (copy and remove), if you don't actually mean cut, then replace the word with paste. I have to admit that as someone who has never used Rosetta before, it is unclear which of the two is needed.

The two lines are:

https://github.com/lorenaacuna/GASTLI/blob/de99c5ca4851d1b54d6cdf155bb1ce562b02cad2/docs/source/macos_arm64_install.rst?plain=1#L70
https://github.com/lorenaacuna/GASTLI/blob/de99c5ca4851d1b54d6cdf155bb1ce562b02cad2/docs/source/macos_arm64_install.rst?plain=1#L103

Here, you say you'll assume people already have anaconda installed in their main terminal. Since most people are very confused between anaconda, miniconda and conda, here again it would be good to specify something like: "[...] you already installed anaconda or miniconda  in your M1/M2/M3 terminal [...]".

Claim: https://github.com/lorenaacuna/GASTLI/blob/de99c5ca4851d1b54d6cdf155bb1ce562b02cad2/docs/source/macos_arm64_install.rst?plain=1#L129 - but how will I actually know? Might be worth adding half a sentence describing how exactly the user will know.

The most important comment is, actually, that when I ran softwareupdate --install-rosetta, I get the following output - note the error message in the second to last line before it claims it finished successfully:

maintenncesMBP6:~ ilaginja$ softwareupdate --install-rosetta
I have read and agree to the terms of the software license agreement. A list of Apple SLAs may be found here: http://www.apple.com/legal/sla/
Type A and press return to agree: A
2024-11-24 20:33:44.639 softwareupdate[39894:12294479] Package Authoring Error: 002-57052: Package reference com.apple.pkg.RosettaUpdateAuto is missing installKBytes attribute
Install of Rosetta 2 finished successfully

The result is that I do not have the iTerm app on my machine. I tried googling this but wasn't able to understand what is happening. Would you have any clues?

This is the message we get if we try to install GASTLI in Apple Silicon with the gfortran native ARM64 installation. This is not explicitly said in the disk image subsection of the installation documentation, but the version I think you need to install of gfortran is the Intel one, not the ARM64. I need to confirm this with the co-author that wrote this sub-section of the documentation. In the meantime, can you try to uninstall your current ARM64 gfortran and install the Intel one? If that does not work, please let me know.

My co-author got back about the disk image solution: If you have the M1/M2/M3 Mac OS, you need to install the ARM64 (not Intel, as I said in a previous message). For them, the disk image worked just "out of the box", following the instructions in FX Coudert's repository.
@ivalaginja we suggest to check what fortran binary is being called by default by running the command which gfortran on your terminal. If it is not the the arm64 version that you installed, then you need to change the path of the default gfortran to the arm64. I'm adding a few words about this in the documentation. I'll be back to this thread to address the comments about the Rosetta installation.

I address the Rosetta installation questions:

I am currently going through the installation with Rosetta for a M1/M2/M3 Mac and have a couple of questions and comments:

Twice in the installation instructions, you instruct to "cut and paste" -> do you really mean cut, or do you mean copy and paste? If it's indeed cut, it might be worth to clarify in parentheses (copy and remove), if you don't actually mean cut, then replace the word "cut" with "copy". I have to admit that as someone who has never used Rosetta before, it is unclear which of the two is needed.
The two lines are:

https://github.com/lorenaacuna/GASTLI/blob/de99c5ca4851d1b54d6cdf155bb1ce562b02cad2/docs/source/macos_arm64_install.rst?plain=1#L70
https://github.com/lorenaacuna/GASTLI/blob/de99c5ca4851d1b54d6cdf155bb1ce562b02cad2/docs/source/macos_arm64_install.rst?plain=1#L103

We mean cut, i.e copy and remove. I clarified this in the documentation.

Here, you say you'll assume people already have anaconda installed in their main terminal. Since most people are very confused between anaconda, miniconda and conda, here again it would be good to specify something like: "[...] you already installed anaconda or miniconda  in your M1/M2/M3 terminal [...]".

Ok, added to the documentation.

Claim: https://github.com/lorenaacuna/GASTLI/blob/de99c5ca4851d1b54d6cdf155bb1ce562b02cad2/docs/source/macos_arm64_install.rst?plain=1#L129

but how will I actually know? Might be worth adding half a sentence describing how exactly the user will know.

It is displayed in the terminal when you open it. When I open my iTerm_Rosetta, this is the first message that is shown:
Last login: Mon Nov 25 10:04:04 on ttys000
Running on Rosetta using miniconda3

The default interactive shell is now zsh.
To update your account to use zsh, please run `chsh -s /bin/zsh`.
For more details, please visit https://support.apple.com/kb/HT208050.
(base) mac-n-176:~ acuna$

I added this to the documentation.

The most important comment is, actually, that when I ran softwareupdate --install-rosetta, I get the following output - note the error message in the second to last line before it claims it finished successfully:

maintenncesMBP6:~ ilaginja$ softwareupdate --install-rosetta
I have read and agree to the terms of the software license agreement. A list of Apple SLAs may be found here: http://www.apple.com/legal/sla/
Type A and press return to agree: A
2024-11-24 20:33:44.639 softwareupdate[39894:12294479] Package Authoring Error: 002-57052: Package reference com.apple.pkg.RosettaUpdateAuto is missing installKBytes attribute
Install of Rosetta 2 finished successfully

The result is that I do not have the iTerm app on my machine. I tried googling this but wasn't able to understand what is happening. Would you have any clues?

I have Rosetta installed in the terminal in my OS Monterey M2 chip and I never had this message. Not having the iTerm app in the directory "Applications" of your Mac should not be a problem (I don't have it either myself). Use the default terminal app instead. In my case it is in Applications > Utilities, and it is named "Terminal". If you have the same, follow the instructions in the documentation with "Terminal" instead of "iTerm".
Then for the installation of Rosetta, you may already have Rosetta 2 installed in your system if you tried to run an Intel app before (it is downloaded automatically, see: https://support.apple.com/en-us/102527). If that's the case, you should be able to see the option "Open using Rosetta" under "Get Info" when your right click on the Terminal app.
Let me know if that works.

This is the message we get if we try to install GASTLI in Apple Silicon with the gfortran native ARM64 installation. This is not explicitly said in the disk image subsection of the installation documentation, but the version I think you need to install of gfortran is the Intel one, not the ARM64. I need to confirm this with the co-author that wrote this sub-section of the documentation. In the meantime, can you try to uninstall your current ARM64 gfortran and install the Intel one? If that does not work, please let me know.

My co-author got back about the disk image solution: If you have the M1/M2/M3 Mac OS, you need to install the ARM64 (not Intel, as I said in a previous message). For them, the disk image worked just "out of the box", following the instructions in FX Coudert's repository.
@ivalaginja we suggest to check what fortran binary is being called by default by running the command which gfortran on your terminal. If it is not the the arm64 version that you installed, then you need to change the path of the default gfortran to the arm64. I'm adding a few words about this in the documentation. I'll be back to this thread to address the comments about the Rosetta installation.

Unfortunately, this still doesn't work. When I run which gfortran, it now returns /usr/local/bin/gfortran as expected, and I made sure to install the ARM64 version. Not sure what's happening there.
Still working on the installation with Rosetta.

I have Rosetta installed in the terminal in my OS Monterey M2 chip and I never had this message. Not having the iTerm app in the directory "Applications" of your Mac should not be a problem (I don't have it either myself). Use the default terminal app instead. In my case it is in Applications > Utilities, and it is named "Terminal". If you have the same, follow the instructions in the documentation with "Terminal" instead of "iTerm".
Then for the installation of Rosetta, you may already have Rosetta 2 installed in your system if you tried to run an Intel app before (it is downloaded automatically, see: https://support.apple.com/en-us/102527). If that's the case, you should be able to see the option "Open using Rosetta" under "Get Info" when your right click on the Terminal app.
Let me know if that works.

This worked, thanks for the guidance. Maybe you could put this information in a "Warning" or "Note" box in the documentation too? Also, I first had to close all my open Terminal windows to make this work.
For the docs, I would urge you to remove the note about the conda env for gastli being optional which you have at the end of the Rosetta installation section. You go to great lengths having people install conda, so imho they should also use it. It is a good coding practice anyway so let's endorse it :)
That being said, I would like to see a clarification on the Installation page about the sequence of things. YOu currently have: "If you do not have either of the first two requirements, we offer an installation guide for different OS below: [...]"
Instead I would suggest to give very clear instructions for installation:
"To get everything installed, first make sure to follow the Fortran installation process appropriate for your OS, then proceed with the GASTLI package installation." -> I would also make a title for the Fortran installation section here on the same level like the title "Installation of GASTLI’s Python package". This way it is clear that these are two separate steps that need to be done in sequence to get everything done.
I will report back with my further tests.

This worked, thanks for the guidance. Maybe you could put this information in a "Warning" or "Note" box in the documentation too? Also, I had to quite my Terminal to make this work which is logical to me once I realized you cannot run both architectures in separate windows at the same time apparently.
For the docs, I would urge you to remove the note about the conda env for gastli being optional which you have at the end of the Rosetta installation section. You go to great lengths having people install conda, so imho they should also use it. It is a good coding practice anyway so let's endorse it :)
That being said, I would like to see a clarification on the Installation page about the sequence of things. YOu currently have: "If you do not have either of the first two requirements, we offer an installation guide for different OS below: [...]"
Instead I would suggest to give very clear instructions for installation:
"To get everything installed, first make sure to follow the Fortran installation process appropriate for your OS, then proceed with the GASTLI package installation." -> I would also make a title for the Fortran installation section here on the same level like the title "Installation of GASTLI’s Python package". This way it is clear that these are two separate steps that need to be done in sequence to get everything done.

I'm adding these suggestions to the documentations now. I'm glad you could make everything work with Rosetta

Everything seems to run as expected. Do you have an idea though why the pytests don't run when I call them locally?

It works for me. The exact steps I followed are:

Clone the repository with git. From your terminal: git clone https://github.com/lorenaacuna/GASTLI
Create a new virtual environment (or conda, whichever your prefer). In my case: virtualenv GASTLI_forIvatest
Activate the environment: source GASTLI_forIvatest/bin/activate
Let's check what packages are installed by default in this new environment: pip list. The output is:

Package    Version
---------- -------
pip        24.3.1
setuptools 75.3.0
wheel      0.44.0

Go to GASTLI main folder (cd GASTLI). If you run ls from this directory, you should see the meson.build and pyproject.toml files. I run pip install . to install GASTLI and its dependencies, including pytest.
6. Go to the the test directory (cd test) and run pytest. The output for me was:
(GASTLI_forIvatest) (base) mac-n-176:test acuna$ pytest
========================================================================= test session starts ==========================================================================
platform darwin -- Python 3.10.14, pytest-8.3.4, pluggy-1.5.0
rootdir: /Users/acuna/Desktop/MPIA2024/Code dev/JOSS GASTLI/Review/ivas test/GASTLI
configfile: pyproject.toml
collected 4 items

test_Coupling.py .                                                                                [ 25%]
test_GASTLI.py .                                                                                [ 50%]
test_Thermal_evolution.py .                                                                                [ 75%]
test_atm_models_interp.py .                                                                                [100%]

==================================================================== 4 passed in 109.43s (0:01:49) =====================================================================
 [i] Allowed maximum number of iterations reached.

 [i] Convergence reached.
 [i] Convergence reached.
 [i] Convergence reached.
 [i] Allowed maximum number of iterations reached.

It takes a while for all tests to run (almost 2 minutes) but it run without interruptions.
I'm going to close this issue since we resolved the installation problems. If you still have problems with pytest and testing it, please open a new issue


Keep track of what host a credential set came from

Definitely useful to keep track of on long engagements

Done in 3cb0e7704306758865f21e6fae7777645fcd1496


fix: Sum of zero elements equals zero

I believe sum of zero elements equals zero. Current implementation produces NaN for this case.

In plain JavaScript, how does the length of a set correlate with its type?

Here, I don't think that convention would be useful.
If no set of elements is passed then there is no number of elements to count or sum, and so there is no number that can be returned.
Anyway, the current behaviour is consistent with other operations, e.g. BigNumber(1).plus()  // NaN`.

I don't think that convention would be useful here. I think it is better to indicate a probable mistake with NaN.
The current behaviour is consistent with other operations, e.g. BigNumber(1).plus() // NaN.

I don't think that convention would be useful here. I think it is better to indicate a probable mistake with NaN.
The current behaviour is consistent with other operations, e.g. BigNumber(1).plus() // NaN.

Big difference here is that .plus expecting exactly one number, while sum expects set of numbers. And it is consistent with mathematical definition of sum

Big difference here is that .plus expects exactly one number, while sum expects set of numbers.

Sum expects one or more numbers.

Imagine we have a cart and products in it.

Right, it's shopping, not formal mathematics.
How about:
sum = BigNumber.sum(...cart.products.map(p => p.price));
if (sum.isNaN()) {
  if (cart.products.length === 0) handleEmptyCart(cart);
  else handleInvalidPrice(cart);
}

LGTM!

For empty cart it's NaN. What kind of error we want to indicate here?

NaN is returned as a sentinel value which can be interpreted multifariously by code at a higher level of abstraction.

Still if we pass undefined as first argument we would receive NaN because it's an error.

Well, undefined is not a number, so that seems reasonable.

Big difference here is that .plus expects exactly one number, while sum expects set of numbers.

Here, sum expects one or more numbers.

Imagine we have a cart and products in it.

Right, it's shopping, not formal mathematics.

For empty cart it's NaN. What kind of error we want to indicate here?

That there is nothing to sum?

Still if we pass undefined as first argument we would receive NaN because it's an error.

NaN is returned because undefined is not a number.
I agree that in some situations it would be useful for zero to be returned, but it would need to be applied consistently so:
BigNumber();           // 0
BigNumber(1).plus();   // 1
BigNumber.sum();       // 0
// etc.

I will consider it further but I am not convinced atm.


management

[Enter feedback here]

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

ID: c7d9e962-1eea-a7f6-74a9-baa20fd37977
Version Independent ID: 8a6c1479-8b1e-e0c7-d1ab-631323b239f7
Content: Visual Studio Projects - C++
Content Source: docs/build/creating-and-managing-visual-cpp-projects.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

@Samuyuththa
Thanks for entering an issue. Unfortunately, it doesn't appear to be actionable. If you could explain the problem in more detail, that would be useful. Until then, I'm closing this as not fixable given the information available.


Update common files.

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
41a0acd0fa8f5b6709d5a9a230d9774905512560
link
/test gencheck_operator

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

I think you need to update some of the targets in Makefile.core.mk.  Here's an excerpt of a prototype change I did last night.
mesh: vfsgen
go build ./cmd/mesh.go
GOARCH=$(TARGET_ARCH) GOOS=$(TARGET_OS) go build -o $(TARGET_OUT)/mesh ./cmd/mesh.go
NOTE: docker targets only work with local builds.
controller: vfsgen
go build ./cmd/manager
docker: controller
mkdir -p build/out
cp $(GOBIN)/istio-operator build/out/.
docker build -t $(HUB)/operator:$(TAG) -f build/Dockerfile .

thanks @geeknoid will address


Configuration for disabling ParenthesesAroundCondition in multi-line conditions

Hello!
I don't like extra parentheses, like many of us.
But sometimes they seem to me appropriate. Especially, in cases with multi-line conditions.

Expected behavior
I don't want to get offense from ParenthesesAroundCondition for multi-line conditions.
Actual behavior
I'm getting.
Steps to reproduce the problem
if (
  one.update(one_values) &&
  two.update(two_values)
)
  return true
end

I can't write so. I need to align:
if one.update(one_values) &&
   two.update(two_values)
  return true
end

RuboCop version
Include the output of rubocop -V. Here's an example:
$ rubocop -V
0.54.0 (using Parser <IP_ADDRESS>, running on ruby 2.5.1 x86_64-linux)

Thank you! :hearts:


fix retrieving endian length

The endian length was not being parsed properly so the length comparison check was failing. By specifying the byteLength to readUIntBE, it meant that the totalLength parameter was set properly.
Tested under node.js v7.3.0

Cool, thanks for this! I just confirmed on another branch so LGTM


plz help me with language translation

Screenshot

Description
as you can see from the image, I've created language file but it won't work.
This happens not only in Row level security filter page, it happeds oftenly in other components as well.
thank you and all the best

Are ANY of the translated strings loading? It looks like the global navigation and Refresh buttons aren't translated either, so I worry your problem is not just the string, but the loading of strings in general.

Also your PR doesn't say what version of Superset you're using. Is this an issue in 3.0 or newer?

Closing since we no longer even support 3.0. This issue seems to have gone stale.


Let normal display of Chinese

Hi, thanks for making a PR! I'm sorry to say, but I don't think this is an acceptable solution. sys.setdefaultencoding('utf8') is a hack which has always been discouraged, as it can break some things and cause silent bugs that are hard to spot. A better approach would be to use the unicode data type to store the LDAP group name (and anything else that might need to be unicode).
Alternatively, Python3 uses utf8 everywhere, and I need to convert the codebase anyway since Python2 is no longer officially supported. So this might be a good time for that to happen.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.k3441 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

not merging. closing PR. If you like please re-open with the above fixes as mentioned by @DarthFennec


Use container runtime as docker for doc build

There is an issue with podman storage side which stops us to build
the docs and produce following error.
make GOPATH="/home/circleci/go" build_docs

podman run -v /home/circleci/project/docs:/docs:Z --rm quay.io/crcont/docs-builder:latest build_docs -b html5 -D /docs/build -o index.html /docs/source/getting_started/master.adoc
Trying to pull quay.io/crcont/docs-builder:latest...
Getting image source signatures
[...]
Writing manifest to image destination
Storing signatures
  Error processing tar file(exit status 1): operation not permitted
Error: Error committing the finished image: error adding layer with blob "sha256:d2320253299952d6acf67a183191a320661feeec9a7c4f1cb5107ab23cef7d8e": Error processing tar file(exit status 1): operation not permitted
make: *** [Makefile:125: build_docs] Error 125

Exited with code exit status 2

https://github.com/containers/buildah/issues/2326

fixes: #2267

/hold
Till build doc job on circle ci passes.

circle-ci passed
/unhold

This could probably have been fixed with:
    sed -i '/^driver = "overlay"/ c\driver = "vfs"' /etc/containers/storage.conf


Mirror Shield

Is it normal that the mirror shield can not be enchanting with repair?

Yes, it’s not enchantable at all. You could change this with /mconfig wand mirror_shield enchantable true if you want.


Placement Driver

Wouldn't it be worthwhile to integrate the 'Placement Driver' into tikv?

no @doublex

PD is written with Go, tikv is Rust, it is hard to combine them together.
PD is the control center, and scheduler TiKVs, they are two different levels in TiDB cluster.

Can we use TiKV without PD? If not, why isn't PD part of TiKV?
Different languages shouldn't be a problem.

yes @reinventions  you can start a single standalone TiKV without PD, but cluster must use.
In our design, we should keep PD -> TiKV -> TiDB separately and won't combine them into one binary. Of course, this may cause deployment not easily, but the architecture is elasticized and resilient and the service hierarchy is clear.

@siddontang
Thanks for your answer.

Do you have any example for  a single standalone TiKV without PD?   I just want to run some tests and needn't PD .


[type: feature] Support "Spring Cloud Alibaba Dubbo".

Upgrade dubbo to 2.7.8, fix potential ConcurrentModificationException (apche-dubbo#6031).

Codecov Report

Merging #2171 (8523f01) into master (c852ed5) will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #2171      +/-   ##
============================================
+ Coverage     60.41%   60.43%   +0.02%
+ Complexity     4069     4059      -10
============================================
  Files           607      608       +1
  Lines         18097    18093       -4
  Branches       1734     1732       -2
============================================
+ Hits          10933    10935       +2
  Misses         6270     6270
+ Partials        894      888       -6

Impacted Files
Coverage Δ

...henyu/plugin/alibaba/dubbo/AlibabaDubboPlugin.java
51.28% <0.00%> (ø)

...in/alibaba/dubbo/cache/ApplicationConfigCache.java
44.66% <ø> (ø)

...a/dubbo/handler/AlibabaDubboPluginDataHandler.java
66.66% <ø> (ø)

.../alibaba/dubbo/proxy/AlibabaDubboProxyService.java
68.42% <ø> (ø)

...bbo/subscriber/AlibabaDubboMetaDataSubscriber.java
52.94% <ø> (ø)

.../shenyu/plugin/apache/dubbo/ApacheDubboPlugin.java
50.00% <0.00%> (ø)

...gin/apache/dubbo/cache/ApplicationConfigCache.java
47.05% <ø> (ø)

...he/dubbo/handler/ApacheDubboPluginDataHandler.java
27.90% <ø> (ø)

...apache/dubbo/proxy/ApacheDubboGrayLoadBalance.java
0.00% <ø> (ø)

...in/apache/dubbo/proxy/ApacheDubboProxyService.java
58.33% <ø> (ø)

... and 8 more

Continue to review full report at Codecov.

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

what is the difference between spring cloud alibaba with alibaba dubbo?

"alibaba dubbo" is based on com.alibaba.dubbo.
"SpringCloudAlibaba dubbo" is based on org.apache.dubbo with different Registry.
https://spring.io/projects/spring-cloud-alibaba

For changes like this, please create an issue first

The example is good, but I think the starter for springcloud alibaba is meaningless. User just use the apache dubbo is same.


[BUG]: Type Error when trying to create a connection to a neon instance

What version of drizzle-orm are you using?
0.30.1
What version of drizzle-kit are you using?
0.20.14
Describe the Bug
When trying to create a connection using Neon, I have a type error in the drizzle() function.
My code:
import { neon } from "@neondatabase/serverless";
import { drizzle } from "drizzle-orm/neon-http";
import { env } from "../env";

const connection = neon(env.DATABASE_URL);

export const db = drizzle(connection);

Expected behavior
Since I'm following the docs, the code snippet above should throw no errors.
Environment & setup
Node - 20.10.0
Bun - 1.0.30
NextJS - 14.1.3
Drizzle-orm - 0.30.1
Drizzle-kit - 0.20.14
@neondatabase/serverless - 0.9.0
Macbook air M1

const connection: NeonQueryFunction<boolean, boolean> = neon(env.DATABASE_URL);
this coded fixed the error, Its a very small TS problem, no big deal, I just think you guys should know about it

Having the same issue, following exactly the drizzle docs but still getting a TypeScript error.

Having the same issue, following exactly the drizzle docs but still getting a TypeScript error:

Argument of type NeonQueryFunction<false, false> is not assignable to parameter of type NeonQueryFunction<boolean, boolean>

Yes, if you follow the exact documentation you'll still get the error, kinda weird no one is talking about it

This resolved the error for me. I do think either the docs need to change or a fix should be made.
Copy pasting from docs should not result in errors or warnings.
import { neon, type NeonQueryFunction } from "@neondatabase/serverless";
import { drizzle } from "drizzle-orm/neon-http";

const sql: NeonQueryFunction<boolean, boolean> = neon(process.env.DATABASE_URL!);
const db = drizzle(sql, { schema: schema });

Docs received a big update recently. If this issue still persists, feel free to reopen this issue.


404 Pages on Clean Install

Operating System:Mac OS X
Vagrant Version: 1.8.6
Virtual Box Version: 5.1.6
Install goes fine with the exception of the known bugs : #249
failed: [vccw.dev] (item=wordmove) => {"cmd": "/usr/bin/gem install --version 2.0.2 --user-install --no-document wordmove", "failed": true, "item": "wordmove", "msg": "ERROR:  Error installing wordmove:\n\tphotocopier requires Ruby version ~> 2.4.", "rc": 1, "stderr": "ERROR:  Error installing wordmove:\n\tphotocopier requires Ruby version ~> 2.4.\n", "stdout": "", "stdout_lines": []}
and
failed: [vccw.dev] (item=wp-coding-standards/wpcs:) => {"failed": true, "item": "wp-coding-standards/wpcs:", "msg": "./composer.json has been updated Loading composer repositories with package information Updating dependencies (including require-dev) Your requirements could not be resolved to an installable set of packages. Problem 1 - Conclusion: remove squizlabs/php_codesniffer 3.0.0 - Conclusion: don't install squizlabs/php_codesniffer 3.0.0 - wp-coding-standards/wpcs 0.3.0 requires squizlabs/php_codesniffer ~2.0 -> satisfiable by squizlabs/php_codesniffer[2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.4.0 requires squizlabs/php_codesniffer ~2.0 -> satisfiable by squizlabs/php_codesniffer[2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.5.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.6.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.7.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.7.1 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.8.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.9.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.10.0 requires squizlabs/php_codesniffer ^2.6 -> satisfiable by squizlabs/php_codesniffer[2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1]. - wp-coding-standards/wpcs 0.11.0 requires squizlabs/php_codesniffer ^2.8.1 -> satisfiable by squizlabs/php_codesniffer[2.8.1, 2.9.0, 2.9.1]. - Can only install one of: squizlabs/php_codesniffer[2.0.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.1.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.2.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.3.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.3.1, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.3.2, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.3.3, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.3.4, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.4.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.5.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.5.1, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.6.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.6.1, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.6.2, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.7.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.7.1, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.8.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.8.1, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.9.0, 3.0.0]. - Can only install one of: squizlabs/php_codesniffer[2.9.1, 3.0.0]. - Installation request for squizlabs/php_codesniffer (locked at 3.0.0, required as *) -> satisfiable by squizlabs/php_codesniffer[3.0.0]. - Installation request for wp-coding-standards/wpcs * -> satisfiable by wp-coding-standards/wpcs[0.10.0, 0.11.0, 0.3.0, 0.4.0, 0.5.0, 0.6.0, 0.7.0, 0.7.1, 0.8.0, 0.9.0]. Installation failed, reverting ./composer.json to its original content.", "stdout": "./composer.json has been updated\nLoading composer repositories with package information\nUpdating dependencies (including require-dev)\nYour requirements could not be resolved to an installable set of packages.\n\n  Problem 1\n    - Conclusion: remove squizlabs/php_codesniffer 3.0.0\n    - Conclusion: don't install squizlabs/php_codesniffer 3.0.0\n    - wp-coding-standards/wpcs 0.3.0 requires squizlabs/php_codesniffer ~2.0 -> satisfiable by squizlabs/php_codesniffer[2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.4.0 requires squizlabs/php_codesniffer ~2.0 -> satisfiable by squizlabs/php_codesniffer[2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.5.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.6.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.7.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.7.1 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.8.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.9.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.10.0 requires squizlabs/php_codesniffer ^2.6 -> satisfiable by squizlabs/php_codesniffer[2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].\n    - wp-coding-standards/wpcs 0.11.0 requires squizlabs/php_codesniffer ^2.8.1 -> satisfiable by squizlabs/php_codesniffer[2.8.1, 2.9.0, 2.9.1].\n    - Can only install one of: squizlabs/php_codesniffer[2.0.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.1.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.2.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.3.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.3.1, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.3.2, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.3.3, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.3.4, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.4.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.5.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.5.1, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.6.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.6.1, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.6.2, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.7.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.7.1, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.8.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.8.1, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.9.0, 3.0.0].\n    - Can only install one of: squizlabs/php_codesniffer[2.9.1, 3.0.0].\n    - Installation request for squizlabs/php_codesniffer (locked at 3.0.0, required as *) -> satisfiable by squizlabs/php_codesniffer[3.0.0].\n    - Installation request for wp-coding-standards/wpcs * -> satisfiable by wp-coding-standards/wpcs[0.10.0, 0.11.0, 0.3.0, 0.4.0, 0.5.0, 0.6.0, 0.7.0, 0.7.1, 0.8.0, 0.9.0].\n\n\nInstallation failed, reverting ./composer.json to its original content.\n", "stdout_lines": ["./composer.json has been updated", "Loading composer repositories with package information", "Updating dependencies (including require-dev)", "Your requirements could not be resolved to an installable set of packages.", "", "  Problem 1", "    - Conclusion: remove squizlabs/php_codesniffer 3.0.0", "    - Conclusion: don't install squizlabs/php_codesniffer 3.0.0", "    - wp-coding-standards/wpcs 0.3.0 requires squizlabs/php_codesniffer ~2.0 -> satisfiable by squizlabs/php_codesniffer[2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.4.0 requires squizlabs/php_codesniffer ~2.0 -> satisfiable by squizlabs/php_codesniffer[2.0.0, 2.1.0, 2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.5.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.6.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.7.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.7.1 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.8.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.9.0 requires squizlabs/php_codesniffer ~2.2 -> satisfiable by squizlabs/php_codesniffer[2.2.0, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.5.0, 2.5.1, 2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.10.0 requires squizlabs/php_codesniffer ^2.6 -> satisfiable by squizlabs/php_codesniffer[2.6.0, 2.6.1, 2.6.2, 2.7.0, 2.7.1, 2.8.0, 2.8.1, 2.9.0, 2.9.1].", "    - wp-coding-standards/wpcs 0.11.0 requires squizlabs/php_codesniffer ^2.8.1 -> satisfiable by squizlabs/php_codesniffer[2.8.1, 2.9.0, 2.9.1].", "    - Can only install one of: squizlabs/php_codesniffer[2.0.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.1.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.2.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.3.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.3.1, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.3.2, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.3.3, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.3.4, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.4.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.5.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.5.1, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.6.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.6.1, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.6.2, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.7.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.7.1, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.8.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.8.1, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.9.0, 3.0.0].", "    - Can only install one of: squizlabs/php_codesniffer[2.9.1, 3.0.0].", "    - Installation request for squizlabs/php_codesniffer (locked at 3.0.0, required as *) -> satisfiable by squizlabs/php_codesniffer[3.0.0].", "    - Installation request for wp-coding-standards/wpcs * -> satisfiable by wp-coding-standards/wpcs[0.10.0, 0.11.0, 0.3.0, 0.4.0, 0.5.0, 0.6.0, 0.7.0, 0.7.1, 0.8.0, 0.9.0].", "", "", "Installation failed, reverting ./composer.json to its original content."]}
I have since installed wordmove manually as described here: https://github.com/vccw-team/vccw/issues/248
vccw will load correctly for one or two loads, depending on how fast I move. However, it then reverts to serving 404 pages. I wanted to make sure this was not a separate issue, and alert you if it was.

I have verified this is a duplicate of the above mentioned issues.
A temporary user fix for this is vagrant ssh in and remove the .composer/composer.lock file, and change codesniffer to 2.9.1 or simply rerun composer install. Also, follow the steps mentioned here while sshed in to install ruby 2.4.


oblt-cli/setup: version input precedence

It's not a breaking change because https://github.com/elastic/oblt-actions/pull/69 has not been released yet.
This change makes the action prefer the version input if both version and version-file are defined.
Additionally, it will add a warning annotation.

It will also add a notice annotation informing you about the downloaded version.

I think this change is a breaking change

For the reason mentioned in https://github.com/elastic/oblt-actions/pull/73#discussion_r1662228671
I don't think this is a breaking change.
Also, the version-file input has not yet been released.


removed a duplicate

@oliver-zehentleitner,
Nice catch! Thanks 😎

also the small things should be fixed :)
you welcome!

Thank you


Demo Error

Running test : API.get_douyin_profile_videos()
Traceback (most recent call last):
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\tenacity_asyncio.py", line 51, in call
result = await fn(*args, **kwargs)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\tikhub\api.py", line 254, in get_douyin_profile_videos
async with session.get(
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\aiohttp\client.py", line 1141, in aenter
self._resp = await self._coro
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\aiohttp\client.py", line 637, in _request
break
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\aiohttp\helpers.py", line 720, in exit
raise asyncio.TimeoutError from None
asyncio.exceptions.TimeoutError
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "F:\软件\douyinProject\douyin_monitor.py", line 81, in
asyncio.run(async_test())
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\asyncio\runners.py", line 44, in run
return loop.run_until_complete(main)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\asyncio\base_events.py", line 642, in run_until_complete
return future.result()
File "F:\软件\douyinProject\douyin_monitor.py", line 62, in async_test
aweme_list = await api.get_douyin_profile_videos(douyin_user_url, 20)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\tenacity_asyncio.py", line 86, in async_wrapped
return await fn(*args, **kwargs)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\tenacity_asyncio.py", line 48, in call
do = self.iter(retry_state=retry_state)
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\tenacity_init_.py", line 363, in iter
raise retry_exc from fut.exception()
tenacity.RetryError: RetryError[<Future at 0x25d2dd71c40 state=finished raised TimeoutError>]
Process finished with exit code 1

Looks like Douyin API has an issue, I will fix it later.

Looks like Douyin API has an issue, I will fix it later.

OK


Add support for building Fedora 37 images

Enumerate all supported F37 image types in the distribution configuration, but only allow iot-raw-image in the API for now.
@Gundersanne can you have a look at the Fedora repository URLs?  I used metalinks which I don't see being used for any other distro.

I think this is duplicating https://github.com/osbuild/image-builder/pull/430

I think this is duplicating #430

It is.  My bad :)  Closing


Lady Tsunade Added

Hi! There is a merge conflict. Kindly Sync your forked repo and then open the pull request.


added isjdelasalle

@AndreyJz Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Stream isolation tests

Confirm that connections are isolated to a Tor circuit keyed to first party

I submitted a request: https://trac.torproject.org/projects/tor/ticket/26832

Fixed

Fixed


fix(frontend): previous value changed after new item created

Fix: issue-4043736

:tada: This PR is included in version 1.0.0-alpha.15 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.16 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.13 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:


Accordion:  Restrict expanding to button click in heading

I have an <accordion-heading> tag in my accordion with a couple controls inside (inputs, buttons, etc.), but I only want the accordion to expand when one specific button is clicked.  Nothing else should trigger the expand/collapse.  It doesn't seem there is a way to do this.
I've tried:

Setting is-open to false and setting it to true in the ng-click of my button.  Doesn't work because clicking anywhere inside the accordion-heading toggles this boolean.
Setting is-disabled to true and then setting it to false when the button is clicked.  Also doesn't work, the accordion stays disabled
Wrapping ONLY my button with the <accordion-heading> tag.  This causes the rest of the markup in my heading to part of the 'expand/collapse' detail.

https://github.com/angular-ui/bootstrap#support

What is the resolution?  I did not see any further information in the link: https://github.com/angular-ui/bootstrap#support

I figured it out....
set the is-disabled to true for the accordion-group, such as:

Then create an accordion-heading and add an ng-click to a specific text or element, such as:

 Accordion Heading Open

@pedro-meza The solution works like a charm:):):) How does is-disabled in accordion-group relate to ng-click in accordion-heading?
Thank you

@aswini-19aug The status-open is just a scope variable that is set to true or false in the open() function.

@pedro-meza it works fine, thanks for snippet


Route regex not working for numbers

Hi, I'm trying to set up this route:
http://localhost:3002/#2016-12-16/report
by using this:
router.add('(:num)-(:num)-(:num)/report', function(year, month, day) {
console.log(year, month, day);
});

but it's not working. It acts as if the URL doesn't exist. Any idea what am I doing wrong please?

Fixed


DynamicSurfacesDemo is missing nestedScrollingEnabled on ScrollView

ScrollView in activity_main.xml is missing the android:nestedScrollingEnabled="true" property. So the image is not hidden during scrolling.

Intended behavior. Committed the non-scrolling version.


add resnet50_trt example

What is this example adding that is not already demonstrated by the other examples?
@jackyh  have you signed the CLA. We cannot accept your PR until you do: https://github.com/triton-inference-server/server/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla

What is this example adding that is not already demonstrated by the other examples?
@jackyh have you signed the CLA. We cannot accept your PR until you do: https://github.com/triton-inference-server/server/blob/main/CONTRIBUTING.md#contributor-license-agreement-cla

Lots of customers they use python backend for image or language preprocessing, and, typically, will add a DL model backend to process inference, which will make it as an ensemble pipeline. We don't have examples to demonstrate how to use python backend to do such kind of preprocessing which includes data format transformation, normalization of input data or even more complex preprocessing, and then about how to add a DL backend to do inference to form an ensemble pipeline.


Use hight memory. Should use FLAnimatedImage

https://github.com/Flipboard/FLAnimatedImage

Yes, I agree. If you're playing large GIFs, you should definitely use that instead.


C++ modules not working yet

I was under the assumption that ninja already supported modules, but i cant get the simplest of module based programs to work. I'm new to ninja as well as cmake modules so maybe i'm doing something wrong?
Here's the module code i cant get to compile:
cpp_modules.zip
I've tried with Ninja 1.12.0 as well as the master branch but neither work. I'm using these commands (replace the path to where master branch is built):
cmake -DCMAKE_BUILD_TYPE=Debug -DCMAKE_MAKE_PROGRAM=PATH_TO_MASTER_BUILD/ninja.exe -G Ninja -B build/
cmake --build build/

This fails with:
helloworld.ixx:2:8: error: header file iostream cannot be imported because it is not known to be a header unit
Using these commands it does work:
cmake -DCMAKE_BUILD_TYPE=Debug -G "Visual Studio 17 2022" -B build/
cmake --build build/

Ninja doesn't directly support C++ modules, or other language-specific features. It provides lower-level abstraction that generators like CMake can use to support them.
What you describe seems to be an issue with the Ninja build plan generated by CMake, which supports C++ modules, starting from version 3.28, according to https://cmake.org/cmake/help/latest/manual/cmake-cxxmodules.7.html. Note that header units are not supported, but that doesn't seem the case here.
I recommend you raise an issue on the CMake bug tracker instead.

Will do, thanks for looking at it 👍


Reconfigure worker pod capabilities (#537)

Removing privileged and adding SYS_MODULE capability
Worker application in worker pod will run as User 0 in order to
be able to run modprobe

Fixes 729

Codecov Report

Merging #732 (e90a594) into main (b456aba) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #732   +/-   ##
=======================================
  Coverage   77.45%   77.45%
=======================================
  Files          49       49
  Lines        4714     4716    +2
=======================================
+ Hits         3651     3653    +2
  Misses        892      892
  Partials      171      171

Files Changed
Coverage Δ

internal/controllers/nmc_reconciler.go
70.74% <100.00%> (+0.11%)
:arrow_up:

/lgtm


Have sql db proj publish command return publishing promise

Right now the command is returning immediately - but callers should be able to wait on completion of the command to know when the publish is actually finished.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.26%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/controllers/projectController.ts
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14747

Relevant Lines:
29016

💛  - Coveralls