Update threads pool libraries to benchmark to latest.

Signed-off-by: Jérôme Benoit<EMAIL_ADDRESS>
PR Checklist

[x] Please add a description of your changes.
[ ] We need your changes to come with unit tests in order to keep this project in quality and easy to maintain.
[ ] If your changes contain any new feature please be sure to document it.
[ ] Please add a link to the open issue or task that this pull request will resolve.
[ ] Add a note to the changelog to indicate the change.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If yes please indicate it and migration info into the CHANGELOG.md file.

Wait, you should also update the benchmarks, should you?

Wait, you should also update the benchmarks, should you?

It should be done by @pioardi, the documentation reference his computer. If he can't do it anymore, he needs to tell before you or I take over the benchmark runs.

But the benchmarks should be always match to their versions
cause they can differ in results from version to version

But the benchmarks should be always match to their versions
cause they can differ in results from version to version

Theoretically, the merge of that PR shall be done after @pioardi benchmark run from that branch and the commit of the result in it.

@pioardi you must do a benchmark run on top of that branch and commit the result.

@pioardi you must do a benchmark run on top of that branch and commit the result.

@pioardi ping?


use graphics::layout?

Hi,
Congrats on an awesome tool!
Something caught my attention that might be of relevance: Just tried maftools within a Shiny application, and when the Shiny application is launched the plotly library is attached, with its own layout function (masking the base graphics layout function). As a consequence, when i try to call oncostrip, the Shiny environment will attempt to use plotly's layout function rather than the graphics::layout function. When altering maftools layout calls to graphics::layout, it seems to work fine.
thanks,
Sigve

Hi,
Sorry for late reply, and thanks for reporting. I can reproduce the issue. Just loading plotly after maftools causes oncoplot/oncostrip functions to throw an error. I'll add the namespace to layout
function soon.
p.s: just checked pcgr - looks pretty cool! Will definitely try it out..

No worries, and do not be sorry. You are not late:-)
Cheers,
Sigve

Could you please check if the fix works? I have tested it before and after loading plotly but havent tested it within shiny app. Please let me know.

Works like a charm! Thanks for the fix.

Hi @PoisonAlien, I am having this same issue within a shiny app. I have updated to the latest github version of maftools but the problem persists.
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS Mojave 10.14.6

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/3.6/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] parallel  stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] shiny_1.4.0       kairos_<IP_ADDRESS>00 testthat_2.3.1

loaded via a namespace (and not attached):
  [1] colorspace_1.4-1     ggridges_0.5.2       rsconnect_0.8.16
  [4] rprojroot_1.3-2      markdown_1.1         fs_1.3.1
  [7] rstudioapi_0.11      roxygen2_7.0.2       farver_2.0.3
 [10] yesno_0.1.0          remotes_2.1.0        DT_0.12
 [13] golem_<IP_ADDRESS>01     fansi_0.4.1          xml2_1.2.2
 [16] codetools_0.2-16     splines_3.6.0        doParallel_1.0.15
 [19] knitr_1.28           maftools_2.2.20      pkgload_1.0.2
 [22] jsonlite_1.6.1       packrat_0.5.0        gridBase_0.4-7
 [25] cluster_2.1.0        shinydashboard_0.7.1 BiocManager_1.30.10
 [28] readr_1.3.1          compiler_3.6.0       httr_1.4.1
 [31] backports_1.1.5      assertthat_0.2.1     Matrix_1.2-18
 [34] fastmap_1.0.1        lazyeval_0.2.2       cli_2.0.1
 [37] later_1.0.0          htmltools_0.4.0      prettyunits_1.1.1
 [40] tools_3.6.0          gtable_0.3.0         glue_1.3.1
 [43] reshape2_1.4.3       dplyr_0.8.4          Rcpp_1.0.3
 [46] Biobase_2.44.0       NMF_0.22.0           vctrs_0.2.2
 [49] iterators_1.0.12     crosstalk_1.0.0      xfun_0.12
 [52] stringr_1.4.0        ps_1.3.2             mime_0.9
 [55] lifecycle_0.1.0      rngtools_1.5         shinycssloaders_0.3
 [58] scales_1.1.0         hms_0.5.3            promises_1.1.0
 [61] RColorBrewer_1.1-2   yaml_2.2.1           curl_4.3
 [64] ggplot2_3.2.1        pkgmaker_0.31        stringi_1.4.5
 [67] desc_1.2.0           foreach_1.4.8        BiocGenerics_0.30.0
 [70] pkgbuild_1.0.6       attempt_0.3.0        bibtex_<IP_ADDRESS>
 [73] rlang_0.4.4          pkgconfig_2.0.3      evaluate_0.14
 [76] lattice_0.20-38      purrr_0.3.3          htmlwidgets_1.5.1
 [79] processx_3.4.2       tidyselect_1.0.0     plyr_1.8.5
 [82] magrittr_1.5         wrapr_1.9.6          R6_2.4.1
 [85] pillar_1.4.3         withr_2.1.2          survival_3.1-8
 [88] tibble_2.1.3         crayon_1.3.4         wordcloud_2.6
 [91] plotly_4.9.2         rmarkdown_2.1        usethis_1.5.1
 [94] grid_3.6.0           data.table_1.12.8    callr_3.4.2
 [97] forcats_0.4.0        digest_0.6.24        xtable_1.8-4
[100] tidyr_1.0.2          httpuv_1.5.2         munsell_0.5.0
[103] registry_0.5-1       viridisLite_0.3.0

Update to https://github.com/PoisonAlien/maftools/issues/387#issuecomment-586027004, explicitly calling shiny::renderPlot helped fix the problem. Thanks!

Hello @jaybee84 ,
Sorry for the delay. Good to know that the issue was fixed. Please let me know if there is anything I can help you with..


Options to set limit on number and size of packages to download.

It is pretty easy when mirroring from large repos to inadvertently get more than you really need.
One way to address this would be to add limits on both the number of packages and the total
size of packages to be downloaded and to not download anything if the limit is exceeded. We
can pick some suitable default value and require users to override it for larger mirroring jobs.

Thanks, has this been implemented? Am running into space issues when mirroring Python3.


CO-1904_First_field_does_not_have_input_focus_when_manually_adding_new_CO_person_role

Enable autofocus for the first field of CoPersonRole add/view form

Thanks Ioannis - note that if there are no COUs, we need to focus the Affiliation instead - can we update this to include that?

@spartanicus suggestion is now implemented

The move to the HTML5 attribute is a particularly good call (the original implementation of this goes back a while). Merging.


Allow loading checkpoints with superfluous state-keys

Change the default checkpoint loading behavior, such that we allow
loading a checkpoint that has state-dict entries that do not
appear in the original (Distiller) model.

@nzmora - Didn't we already merge this in #300? When I merge master into this branch, it shows no diffs. Keeping open at the moment in case I'm missing something...

@guyjacob ,
Yup - we did merge this.  I don't know what happened here, but i'm closing.


Possible Vulnerabilities

Issue type:
I checked some files used in the netbox, some of them have CWE known, below the list and files.
This alert may be a false positive, manual confirmation is required.
/static/jquery-ui-1.12.1
https://cwe.mitre.org/data/definitions/352.html
Cross-site request forgery, also known as a one-click attack or session riding and abbreviated as CSRF or XSRF, is a type of malicious exploit of a website whereby unauthorized commands are transmitted from a user that the website trusts.
/static/jquery-ui-1.12.1/package.json
https://cwe.mitre.org/data/definitions/538.html
A configuration file (e.g. Vagrantfile, Gemfile, Rakefile, ...) was found in this directory. This file may expose sensitive information that could help a malicious user to prepare more advanced attacks. It's recommended to remove or restrict access to this type of files from production systems.

Python version: 2.7.12+
NetBox version: 2.0.1

This might be true, however netbox is clearly not intended to be used in a public / client facing environment and has no significant user structure (other than authorised or unauthorised and the secret management platform where signed key = full access), as such it is clear that security within this project is not a significant designed intent.
However I'm sure that if someone where to put up a pull request that fixed the issues you point out that it would indeed be accepted.


cdn handlebars tag doesn't work in production

The cdn handlebars tag does not work in production.

Hey @pfcodes, unfortunately that is indeed the case, widgets do not currently support stencil's custom handlebars helpers (listed (here)[https://github.com/bigcommerce/widget-builder/issues/78]).

Hey @pfcodes, unfortunately that is indeed currently the case. Widgets don't currently support stencil context variables (listed here), which unfortunately the cdn handlebars helper relies on.


Fix - Avoid usage of JsonConvert in test host process

Related issue
https://github.com/Microsoft/vstest/issues/1757

@dotnet-bot test this please


Crazy Idea: min-margin

Hi,
I don't know how crazy / impossible / bad it is, but I suggest it anyway.
As front-end developers, we often have to deal with cases where we don't know what margin to put on something because:

either it's badly sticked and we have to wrap the component with a box
either it conflicts with another element and make a really big margin (10px right for A + 10px left for B)
either we have to use some complex selectors to avoid this but it's not very component friendly / thinking.

What I suggest is having a min-margin (with min-margin-left etc) attribut, that can automatically takes the min-margin of the differents components around, to automatically set the lowest min-margin between two components.
For example, if you have a component A and a component B, placed side by side, with min-margin: 10px on A and min-margin: 20px on B, the margin between both won't be 30px, but 20px.
I don't know what are the current limits of the DOM / browsers to do this, but maybe you can find a solution for having this. It would drastically improve component reusability, component thinking and component readability.
Maybe you have another idea to solve this issue ?
Tell me what you think about it ! :)
Thank you !

IMO – glamorous shouldn't really be inventing new non-standard CSS properties. This is definitely a pain point but might be solvable through clever componentization. Using your second example–10px right for A, 10px left for B–maybe they should both have no margin by default and then a parent HorizontalList component should overrides the styles of A and B and inject a margin rule.

I agree with what's been said. Another thing I've done is used helpers like spaceChildrenHorizontally:
import React from 'react';
import { render } from 'react-dom';
import glamorous from 'glamorous';

function spaceChildrenHorizontally(space) {
  return {
    '& > *': {
      marginLeft: 0,
      marginRight: 0,
      '&:not(:first-child)': {
        marginLeft: space / 2,
      },
      '&:not(:last-child)': {
        marginRight: space / 2,
      },
    },
  };
}

const Container = glamorous.div(
  {
    display: 'flex',
    justifyContent: 'center',
    backgroundColor: 'gray',
    padding: 20,
  },
  spaceChildrenHorizontally(20),
);

const Child = glamorous.div({
  backgroundColor: 'lightgray',
  padding: 10,
});

// so you don't have a bunch of output in
// the console every time you make a change
console.clear();

function App() {
  return (
    <glamorous.Div maxWidth={600} margin="auto" fontSize={24}>
      <Container>
        <Child>Child 1</Child>
        <Child>Child 2</Child>
        <Child>Child 3</Child>
        <Child>Child 4</Child>
      </Container>
    </glamorous.Div>
  );
}

render(<App />, document.getElementById('root'));

Try it here.
I think that the general feeling is correct though. This seems like something another library should solve and not something glamorous should be getting into. Thanks though!


address cc

Address issue in perl configuration that full paths are stored in variious config places by storing just basename of CC/GCC.
Additionally takes care the we do not have still build-environment paths in final package configuration.
Mention gdbm's run dependency explicit in run-section

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_bez6_w14lw/clone/recipe/meta.yaml:18: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
ERROR: /tmp/abs_bez6_w14lw/clone/recipe/meta.yaml:19: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_19yhq21h_n/clone/recipe/meta.yaml:18: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
ERROR: /tmp/abs_19yhq21h_n/clone/recipe/meta.yaml:19: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4evfxa247z/clone/recipe/meta.yaml:19: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
ERROR: /tmp/abs_4evfxa247z/clone/recipe/meta.yaml:20: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_27mf3u_8a6/clone/recipe/meta.yaml:19: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
ERROR: /tmp/abs_27mf3u_8a6/clone/recipe/meta.yaml:20: compilers_must_be_in_build: The recipe requests a compiler in a section other than build
Errors were found


延迟？

声音先出来，后面视频延迟

For latency issue, I think you could use third-party SDK, https://github.com/daniulive/SmarterStreaming for example.


Fixed wrong and undefined stylings

Signed-off-by: Agha Saad Fraz<EMAIL_ADDRESS>I have fixed LESS styling and removed the undefined ones.

Yeah @wetneb though I have checked, but I think it would be better somebody else review the changes as well.


Number.isFinite(...)

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/isFinite

@mudcube, thanks for the PR. The code looks good. Before I can merge this in, I need to check that you have read and agreed to our contribution terms:

All contributions to the polyfill service are made under the MIT licence. It is important therefore that you agree to the following when contributing code to the service:

Any contribution I make to the polyfill service by way of Pull Request contains only data that is:

wholly my own work, upon which I am legally permitted to grant a licence; or
wholly the work of individuals within the same organisation to which the copyright in all work created by employees is assigned, of which I am a current employee and authorised representive for the purposes of this contribution, and which is legally permitted to grant a licence; or
licensed by the original author under the MIT licence or equivalent terms

In respect of any part of my contribution that is not already available under an MIT or compatible licence, I hereby grant to the Financial Times Limited (FT) and to recipients of software distributed by the FT a perpetual, worldwide, non-exclusive, no-charge, royalty-free, irrevocable license to make, have made, use, offer to sell, sell, import, and otherwise transfer the data.

It is helpful to us if you can clearly state your agreeement to the above terms when creating a pull request. If you don't, you may be asked to post a followup comment to accept the terms, before we can merge your PR.

@JakeChampion Np. Those terms sound fine to me.

Thanks for contributing this polyfill. It will be released in version 3.11.0.
In case you are not aware, this project has a set of contribution rewards awarded to community members who have a pull request merged into the project.

Contribution rewards
We want to encourage more people to get involved in the polyfill service, so the Financial Times is pleased to offer one month's free premium access to all FT content (normally $50) to anyone who has a pull request merged in this repository. Claim your free access.

Again, thanks for this pull request :-)


Weird behavior after changing routes

Reproduction

Click Start Tour button. Now the tour flow works well and all buttons work as expected.
Click the About router link above at navigation and then back to home page.
Click Start Tour button again. Except close button at the top right conner,
there will be something wrong with other buttons in step 1. If try to click them one by one, you will get the result as screenshot shown below.

Demo
https://codesandbox.io/s/guide-tour-65yrm

Edited:
It seems that there are multiple tour instance. If there is anything I can do to destroy the old tour instance when leaving the page(component)? How to fix it?
Thank you!

@NevenLiang yes, probably need to destroy when you change pages.

@rwwagner90 Do you have any clues about why it happen and any advices to handle it with or without Angular? I can't find a related method in the docs of tour.
I have tried to add the code as following, but it doesn't work.
export class HomeComponent implements OnDestroy {
  constructor(private guideTour : ShepherdService) {}

  ngOnDestroy() {
	this.guideTour = null;
  }

  startTour() {
    // set up and start guide tour
  }
}

@NevenLiang I just took a look at the codesandbox, and I am unsure of what the problem is. I am not an Angular dev, so I am not sure if we have something setup wrong or what, but addSteps should be setting tourObject to a new tour, so there should be no way multiple get triggered.
It almost seems like Angular registers the service multiple times and somehow call the method to create new tours multiple times.

I have noticed that the tour flow is fine, but the problem is step-1 keeps showing on the screen and only clicking on the cancel(X) icon can make step-1 disappeared.
If we click the Next button in step-1, step-1 should disappear. If we click on the Exit button or the Next button on step-2, the mask does disappear, which means cancel() or complete() have been called, but step-1 is still on the screen.

@NevenLiang sorry, perhaps I was not clear in my previous message. I agree with you there is a bug, but I am not an Angular dev, so I do not know what we may have done wrong in this library.
addSteps should be setting tourObject to a new tour, so there should be no way multiple get triggered.
It almost seems like Angular registers the service multiple times and somehow calls the method to create new tours multiple times.

@rwwagner90 OK.
I am still trying to fix it. I have found some new clues.

There are two step-1 popups, which are popup-1 and popup-3 (You can change the left and bottom property of popup-1 to make it clear). The popup-1 is like a controller, it can only be closed by clicking cancel icon. The popup-2 and popup-3 is the normal step.

@NevenLiang Is there new information there? We know Angular is creating multiple instances here. I am not an Angular dev, so it would be helpful if someone who knew how Angular services work could chime in here. It appears the service instantiates itself every time you enter the route, instead of once for the application. We need it to only instantiate one time, and I need guidance from someone who is good with Angular to help us remedy this in the library.

@NevenLiang I believe I got it to work: https://codesandbox.io/s/guide-tour-8znbk
I think the issue is you need to setup the service in your app.component so there is a single service created. I believe doing it in child components causes there to be a new service instantiated every time the component is rendered.
We probably need to make the docs more clear about this.

@rwwagner90 Sorry for the late response.
Yes, I set it as component's provider because I use shepherd only in that component now, which leads to instantiate every time the component is rendered.
Thank you so much!

@NevenLiang you're welcome! Perhaps we should mention this in the README?


Updating Other Ras genes figure 4

Updating script generating the plotting data and actual figures.
also removing mention of "hyperactive" in README and updating paths for scripts folder.

dropping samples that had mutations or CNAs in the Ras genes before calculating number of Ras events.

dropping Ras genes columns so they are not calculated in total mutation count per sample.

@dhimmel can you give a quick look?

@gwaygenomics : can confirm that I looked at the pandas docs and axis = 1 refers to columns. Up to you if this is sufficient to merge


Handle attribute values that are boolean or blank

When trying to serialize a DOMElement that has an attribute set to a boolean or is blank, the current behaviour doesn't handle this well.

The existing behavior seems more in line with how browsers generally behave—attribute values are coerced into strings, and you need removeAttribute to make them disappear. Outputting the attribute name alone for true also seems wrong for things that aren't actual boolean attribute.

Unfortunately that isn't how the spec says they should behave (or any browser I've used).
Here's the relevant info:
http://www.w3.org/TR/2008/WD-html5-20080610/semantics.html#boolean

I think we're confusing boolean properties with boolean attributes, since this code applies to both. Here's an example of where your patch behaves incorrectly:
var document = require('min-document');

var div = document.createElement('div');
div.setAttribute('data-foo', true);
div.toString();
// WRONG: got '

', expected '

'

var div = document.createElement('input');
// The spec says if a boolean attribute is present, it should be set to either "" or the name of the attribute ("checked"), but browsers will let you set it to anything.
input.setAttribute('checked', true);
div.toString();
// WRONG: got '

', expected '

'

Here's where your patch probably behaves correctly:
var document = require('min-document');

var div = document.createElement('input');
input.checked = true;
div.toString();
// OK?: got '

'

Note that browsers won't even serialize the checked property as an attribute like that, but it might make sense here.


Bois, PR #20 is broke af

Basically this:

This isn't issue worthy :squint:

no u

whomst are closing this

fix the damn


Update updates.txt (v.0.1.4 - v.0.1.6)

if there are many additional information I should know, lemme know and I will update it

Thank you for putting the update details in the update text file


rangeUntil extension function import marked as unused

I have an extension function like this to allow you to say something like now..<5.minutes to get an open ended range of instants:
/**
 * Convenience method to convert an instant and a duration into an open-ended range of instants. For
 * example: `now..<5.minutes` produces an open-ended range from now until 5 minutes from now
 *
 * @receiver The starting instant of the range
 * @param duration The amount of time to add to the starting time to get the end of the range
 * @return An open-ended range that starts at `this` and ends before `this + duration`
 */
operator fun Instant.rangeUntil(duration: Duration): OpenEndRange<Instant> = this..<this + duration

Expected Behavior
When I add an import in another file to use this operator, ktlint should know that ..< corresponds to the rangeUntil function name and should not report the import of the rangeUntil name as unused
Observed Behavior
Ktlint reports an unused import on the rangeUntil operator
Steps to Reproduce
Define a rangeUntil extension operator function in one file and import it and use it in another file

Ktlint is smart enough to do this for rangeTo function and the .. operator.

I believe all that is needed is adding rangeUntil to this line: https://github.com/pinterest/ktlint/blob/master/ktlint-ruleset-standard/src/main/kotlin/com/pinterest/ktlint/ruleset/standard/rules/NoUnusedImportsRule.kt#L294

Tnx for suggestion. It was an easy fix.


Add: TestSystemOperator to test the ^system op

Tests if the call to the ^system op are done correctly.
Is not testing if the result is ok.
Test with the bash script moved into the root directory of the process. It is now in opennars-parent

#!/bin/bash
echo "true"
exit 0 # success

As we discussed earlier, there are some outstanding issues with the implementation. These are listed here for the record:

How to handle a hung system process - timeout?
How to handle return values that require parsing to terms?
How to specify the directory where the system call is executed?
How to handle error code and return value?

I would suggest that it doesn't parse to terms, maybe we need a string term type so anything can be enclosed with "" or '' and ' itself can be escaped with ' (same for " as ").

GMPOW2, for some reason I can't see your email comment here regarding not parsing the return string. I agree that in the short term this will be ok as the use cases we have planned don't require  statements to be returned. However, future use cases may require statements to be returned and this would require parsing the return string. For example
(^system, (SELF}, func $1, ?1)
where ?1 should bind to <a --> b>
I can see this being a common requirement which is not currently supported.

Can you address the issues raised by the Codacy quality review please :)

Can you add some test cases that show vars being unified before the system call ?
Something like: (^system, (SELF}, func {$1}, #1)
and provide a statement that can unify with the operator var
I realise this is outside the scope of the ^system operator call but it would be helpful to show it being able to handle vars and unification in one place.

It should be able to without a problem.
A test is always good though, to avoid surprises. :)
Am Mo., 7. Okt. 2019 um 19:05 Uhr schrieb Tony Lofthouse <
<EMAIL_ADDRESS>
Can you add some test cases that show vars being unified before the system
call ?
Something like: (^system, (SELF}, func {$1}, #1
https://github.com/opennars/opennars/pull/1)
and provide a statement that can unify with the operator var
I realise this is outside the scope of the ^system operator call but it
would be helpful to show it being able to handle vars and unification in
one place.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/opennars/opennars/pull/598?email_source=notifications&email_token=AB7GUBOJ77NBXVQLC4JFGWTQNOCAZA5CNFSM4I53UAKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAROSZI#issuecomment-539158885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7GUBNEFSFBI5I27GE63O3QNOCAZANCNFSM4I53UAKA
.

In a break with convention I will merge the PR as it is needed urgently.


[BTRX-416][risk=no] Add Startup Logging

Addresses
https://broadinstitute.atlassian.net/browse/BTRX-416

Coverage decreased (-0.05%) to 62.325% when pulling ccbc008e5aa016cd998c3f99b374e46d98757f97 on BTRX-416 into 3d8cd299580ce729d1cd57652843ef9b8448047d on develop.


Need an example

How do I use this?

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

ID: d358d6f1-d12f-1d2a-2f84-0be0a9b30cd8
Version Independent ID: 43356929-8f15-a2d1-dbe9-504078ed4986
Content: IconSourceElement Class (Windows.UI.Xaml.Controls) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.controls.iconsourceelement.yml&version=GBlive&_a=contents
Product: uwp
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

Hi, sorry for the delayed response to this issue, and thanks for the suggestion. The doc has been updated with the missing information.


Draft point cloud STAC extension

A draft point cloud STAC extension based on PDAL info output for geospatial point clouds.

By the way, should any of the fields be required or are these all optional (which they are currently)?

Some of them should be required. I'll go through and denote.

Some of them should be required. I'll go through and denote.

We usually do be prefixing the descriptions with the following markdown **REQUIRED.**  - Thanks.


Notebook fails: lightweight_functions_component_io_kfp.ipynb

Expected Behavior
Notebook doesn't fail
Actual Behavior
Step #4: lightweight_functions_component_io_kfp.ipynb                      FAILED    00:00:16    ---------------------------------------------------------------------------
Step #4:                                                                                Exception encountered at "In [20]":
Step #4:                                                                                ---------------------------------------------------------------------------
Step #4:                                                                                KeyError                                  Traceback (most recent call last)
Step #4:                                                                                /tmp/ipykernel_31031/4032352611.py in <module>
Step #4:                                                                                1 from kfp.v2 import compiler
Step #4:                                                                                2
Step #4:                                                                                ----> 3 compiler.Compiler().compile(
Step #4:                                                                                4     pipeline_func=pipeline, package_path="component_io_job.json"
Step #4:                                                                                5 )
Step #4:
Step #4:                                                                                /usr/local/lib/python3.9/site-packages/kfp/v2/compiler/compiler.py in compile(self, pipeline_func, package_path, pipeline_name, pipeline_parameters, type_check)
Step #4:                                                                                1134       kfp.TYPE_CHECK = type_check
Step #4:                                                                                1135       kfp.COMPILING_FOR_V2 = True
Step #4:                                                                                -> 1136       pipeline_job = self._create_pipeline_v2(
Step #4:                                                                                1137           pipeline_func=pipeline_func,
Step #4:                                                                                1138           pipeline_name=pipeline_name,
Step #4:
Step #4:                                                                                /usr/local/lib/python3.9/site-packages/kfp/v2/compiler/compiler.py in _create_pipeline_v2(self, pipeline_func, pipeline_name, pipeline_parameters_override)
Step #4:                                                                                1070
Step #4:                                                                                1071     with dsl.Pipeline(pipeline_name) as dsl_pipeline:
Step #4:                                                                                -> 1072       pipeline_func(*args_list)
Step #4:                                                                                1073
Step #4:                                                                                1074     self._validate_exit_handler(dsl_pipeline)
Step #4:
Step #4:                                                                                /tmp/ipykernel_31031/2896638886.py in pipeline(message)
Step #4:                                                                                14     train_task = train(
Step #4:                                                                                15         dataset_one=preprocess_task.outputs["output_dataset_one"],
Step #4:                                                                                ---> 16         dataset_two=preprocess_task.outputs["output_dataset_two"],
Step #4:                                                                                17         imported_dataset=importer.output,
Step #4:                                                                                18         message=preprocess_task.outputs["output_parameter"],
Step #4:
Step #4:                                                                                KeyError: 'output_dataset_two'
Step #4:
Step #4: === END RESULTS===
Step #4:

That error was already fixed early last week and merged into master. This is some earlier version of the notebook. I did double check on master that this notebook indeed has the fix.

Okay, closing as we discusssed.


update co-organizer section

pls replace eejot.org logo with the logo of www.wdac.org.np. here's their image: http://www.wdac.org.np/wp-content/uploads/2016/03/wdac-logo-1.png (also hyperlink the image.)

@studenton -- sorry shashi seems to be away from computer for other stuff. amit - would you mind updating this too. thanks so much

@924231278564d9de65fb8b30083a8dcae44bbde1 fixes it.


Fix api_change.rst -- incorrect after merge of #578

This is a documentation fix for the api_change.rst
This file did not get updated in release v2.4.0 to state what changes occured between v2.3.0 and v2.4.0.  Subsequently, changes in dev were added to the file in the wrong location during the merge of #578.  This PR corrects this.

Compare the section OpenFAST v2.3.0 to OpenFAST dev in readthedocs master and dev


confusing log message

This log message, while helpful once you know what it means, is initially confusing:
2023-06-29 16:25:31,554 INFO success: quit_on_failure entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)

suggest removing the substring quit_on_failure.

well apparently this error message is coming from Flask or uwsgi, but not from PloverDB. Closing.


Handle ResourceQuotas for the migrate unattached PV case

@sseago please include some detail on this issue.
Jira: MIG-204

Fixed in https://github.com/konveyor/mig-controller/pull/536


SheetContentScaffold overlaps content and does not expand

When using SheetContentScaffold with just a ListView or any other scrollable as the body, the sheet will not expand to the full extent. Additionally, it is impossible to scroll to the last few items in the listview.
When using the SheetContentScaffold with a bottom bar, the bottom bar will always float on top of the body, and the last few items can't be reached as they are underneath the bottom bar.
Version:  smooth_sheets: ^1.0.0-f3<IP_ADDRESS>
Here is a minimal example code alongside a screen recording of the issue:
    var listView = ListView.builder(
      shrinkWrap: true,
      itemBuilder: (context, index) {
        return ListTile(
          title: Text('Item $index'),
        );
      },
      itemCount: 5,
    );
    final content = SheetContentScaffold(
        body: listView);
    // final content = listView;

    final withBg = SafeArea(
      // bottom: false,
      // top: false,
      child: Material(
        borderRadius: const BorderRadius.vertical(top: Radius.circular(16)),
        clipBehavior: Clip.antiAlias,
        child: content,
      ),
    );

    return ScrollableSheet(child: withBg);

Recording of the usage of the SheetContentScaffold, last item is unreachable and not fully expanded:
https://github.com/user-attachments/assets/fdf5d936-ea01-4064-acd0-e9389e1fa331
Recording without using the SheetContentScaffold, all items are accessible, and the sheet is fully expanded:
https://github.com/user-attachments/assets/301f2ee8-5dee-463a-9e46-a4589c3ef007

@Clon1998 Try removing shrinkWrap:true in the listview.

I encountered the same problem as you, but I only need these few lines of code to reproduce it!
class DemoWidget extends StatelessWidget {
  const DemoWidget({super.key});

  @override
  Widget build(BuildContext context) {
    return ScrollableSheet(
      minPosition: SheetAnchor.proportional(0),
      maxPosition: SheetAnchor.proportional(0.9),
      initialPosition: SheetAnchor.proportional(0.9),
      child: ListView.builder(
        itemCount: 50,
        itemBuilder: (context, index) {
          return ListTile(
            title: Text('Item $index'),
          );
        },
      ),
    );
  }
}

I encountered the same problem as you, but I only need these few lines of code to reproduce it!

class DemoWidget extends StatelessWidget {

  const DemoWidget({super.key});

  @override

  Widget build(BuildContext context) {

    return ScrollableSheet(

      minPosition: SheetAnchor.proportional(0),

      maxPosition: SheetAnchor.proportional(0.9),

      initialPosition: SheetAnchor.proportional(0.9),

      child: ListView.builder(

        itemCount: 50,

        itemBuilder: (context, index) {

          return ListTile(

            title: Text('Item $index'),

          );

        },

      ),

    );

  }

}

When I set "maxPosition" and "initialPosition" to "SheetAnchor.proportional(1)" it's ok but it's not the height I expected.
Version Info:

smooth_sheets: ^1.0.0-f3<IP_ADDRESS>

Flutter 3.24.3 • channel stable •

https://github.com/flutter/flutter.git

Framework • revision 2663184aa7 (5 周前) • 2024-09-11

16:27:48 -0500

Engine • revision 36335019a8

Tools • Dart 3.5.3 • DevTools 2.37.3

I actually creates a different issue for that:
https://github.com/fujidaiti/smooth_sheets/issues/265

@Clon1998

    final withBg = SafeArea(
      // bottom: false,
      // top: false,
      child: Material(
        borderRadius: const BorderRadius.vertical(top: Radius.circular(16)),
        clipBehavior: Clip.antiAlias,
        child: content,
      ),
    );

    return ScrollableSheet(child: withBg);

I suspect that the Material inside the SafeArea is causing the issue. I recommend wrapping the ScrollableSheet with the SafeArea instead.
return SafeArea(child: ScrollableSheet(...));

@Clon1998

    final withBg = SafeArea(
      // bottom: false,
      // top: false,
      child: Material(
        borderRadius: const BorderRadius.vertical(top: Radius.circular(16)),
        clipBehavior: Clip.antiAlias,
        child: content,
      ),
    );

    return ScrollableSheet(child: withBg);

I suspect that the Material inside the SafeArea is causing the issue. I recommend moving the SafeArea outside of the ScrollableSheet instead.
return SafeArea(child: ScrollableSheet(...));

This seems to solve the issue. Will test the API further but for now it solves it


How do I use this? Great package!

I was wondering what this means? I'm not sure what to do with this: passiveSupport({/.../})
Do I wrap React components in this?
Thank you! Awesome job making this :)

https://stackoverflow.com/questions/51345929/react-js-violation-added-non-passive-event-listener-to-a-scroll-blocking-mou
This shows an example of how to use it. You wrap the event type in passiveSupport()
E.g.:
import { passiveSupport } from 'passive-events-support/src/utils'
passiveSupport(['mousewheel'])


Add new defer_relation argument to mock_model

Description
defer_relation is an optional argument that was added to ModelNode as part of the dbt clone work. Despite it being optional, MagicMock was trying to mock it up, which failed validation tests on Path. This new argument was added to the mock parameters with a value of None to avoid creating it.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

@mikealfare so it wasn't a actual mashumuro failure good to know great catch


Postcondition DNFer shouldn't run in-place on Memalloy results

We might want to reuse those results for something else, or check to make sure the DNFer is actually doing its job properly.  When DNFing doesn't work, it's a bit odd that we're having to mv the files out of the way.
Instead of the DNF operation happening in-place on results/_latest, we could make a new directory results/_latest/act/dnf and save the results there.
This might, in fact, make it worth dropping the in-place ability of make_dnf, which I originally added specifically to overwrite the memalloy results.
This'll need some care: the script that's handling running the DNFer should run a round of DNF if a _latest directory exists but an act/dnf doesn't, but should not run it unless necessary or forced.

As far as I know, it now does use a separate directory.


Error "Sync in progress initiated by different client"

Hi,
when I try to sync I get "Sync in progress initiated by different client" error.
Recently, I've reset my Logseq cache. After that, the error started to appear.
I've tried to reset Readwise plugin but it didn't fix the error.
If I watch the developer console, an error is throwed

Hi,
Readwise has since released an official plugin. Do you want to try that
instead?
On Tue, 16 Aug 2022 at 6:53 PM, rubenvillar @.***>
wrote:

Hi,
when I try to sync I get "Sync in progress initiated by different client"
error.
Recently, I've reset my Logseq cache. After that, the error started to
appear.
I've tried to reset Readwise plugin but it didn't fix the error.
If I watch the developer console, an error is throwed
[image: image]
https://user-images.githubusercontent.com/4112766/184862242-952f3c95-4c2c-4586-8b2b-c9ef3078768e.png
—
Reply to this email directly, view it on GitHub
https://github.com/hkgnp/logseq-readwise-plugin/issues/26, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALNXKGNJSKCIKA53UEUKMETVZNXKZANCNFSM56VK57GQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi,
you are totally right.
I'm using the official plugin. I though this was the official plugin repo.
¡Sorry!


Mailgun Issue

I'm using Mailgun to send my emails. Unfortunately, I seem to be running into a problem. This is the error I get.
Client error response [url] https://api.mailgun.net/v2/https://api.mailgun.net/v2/sandboxde2ccd631106433f9069b43856977dfb.mailgun.org/messages.mime [status code] 404 [reason phrase] NOT FOUND
Is this something vendor-related or me?

Why is your url https://api.mailgun.net/v2/https://api.mailgun.net/v2.

@crynobone This is what I don't get. The only changes I made for Mail is adding the username and password in the mail.php config file.

Your host at mail config should be: sandboxde2ccd631106433f9069b43856977dfb.mailgun.org
Refer to MailgunTransport

@rscafi Done that, still the same problem.


Embedded version no longer installs

The change introduced in https://github.com/tecbot/gorocksdb/commit/17991d3138a879b166adebf86f7c84da3c1517a7 is incompatible with the CockroachDB rocksdb version used for the embedded build, so go get -tags=embed github.com/tecbot/gorocksdb now fails with a compile error:
# github.com/tecbot/gorocksdb
../../tecbot/gorocksdb/options.go:436: cannot use C.double(value) (type C.double) as type C.int in argument to _Cfunc_rocksdb_options_set_max_bytes_for_level_multiplier

I'm not sure whether that's fixable by this package since you don't control the cockroach version, but thought you'd want to know.

It's also incompatible with the the latest RocksDB v4.11.2 release, meaning that if you're on OSX and brew install rocksdb per their installation instructions, compiling against the shared library (rather than c-rocksdb) also won't work.
IMO, this library should always stay compatible at least with the latest RocksDB release rather than the HEAD of master.

Yea, I think having some tags in there is definitely better than nothing. I was able to get it build locally by just checking out an older commit and then running CGO_CFLAGS ... CGO_LDFLAGS ... go install github.com/tecbot/gorocksdb, so instructing users to checking out and build from the desired RocksDB version should be sufficient.

What is the recommended version? I tried a few and couldn't find the magic one.

@duane, I built it against v4.11.2, though looks like RocksDB has recently released v4.13.4

I just changed it...works ok 💯
@@ -433,7 +433,7 @@ func (opts *Options) SetMaxBytesForLevelBase(value uint64) {
 // SetMaxBytesForLevelMultiplier sets the max Bytes for level multiplier.
 // Default: 10
 func (opts *Options) SetMaxBytesForLevelMultiplier(value float64) {
-       C.rocksdb_options_set_max_bytes_for_level_multiplier(opts.c, C.double(value))
+       C.rocksdb_options_set_max_bytes_for_level_multiplier(opts.c, C.int(value))
 }

 // SetMaxBytesForLevelMultiplierAdditional sets different max-size multipliers

I just discovered that this issue "fixed itself" - the cockroach c-rocksdb repo recently updated to a newer version that's compatible with this repo.  I'll go ahead and close this issue, since go get -tags=embed github.com/tecbot/gorocksdb works again.


Link to "build a classic WAR file" goes to a 404 page

https://spring.io/guides/gs/accessing-data-jpa

We removed that file a while back. The link in the guides should be removed, too. This is a good one for someone who wants a place to get started on contributing to Spring.

Thank you, @blackcoder41, @Buzzardo, and @robertmcnees, for your helpful insights and guidance on this issue! I’ve submitted a PR to remove the broken link. I really appreciate the direction you provided, which made it easier to tackle this fix. Looking forward to contributing more to the project!
My PR


module: getExe instead of /bin/minecraft-server

Use lib.getExe instead of assuming the executable is named minecraft-server.

I have no idea why I didn't know about this before, good catch!


chore: now testing on Aspect Workflows RBE

We can remove the existing buildbuddy bazelrc configuration as well, and rename our aspect_rbe config to just rbe.


custom page break separator with new lines doesn't respect \n in frontend

Note: we're aware of some missing content in the output and layout issues on tables. Please refrain from opening new issues on this topic unless if you think it's different from what has already been reported.
Describe the bug
say i change the separator to \n----\n there will be literal \n's in the output instead of new lines. only happens on frontend.
Screenshots
Feel free to also provide screenshots if relevant.
Client:
Please remove untested options:

Frontend (cloud.llamaindex.ai)

Options
standard options

Fix was fixed but we didn't close this issue.


enhancing the ui of blogs read more page

pls assign me this issue as i would like to contribute in enhancing the ui of read more blog page
issue:

Do a fresh fetch , blog page is updated.


Add support to specify the start and end of a date range for sequence collection dates

We will typically just need to get clade assignments (and summarize to counts of clade assignments) for samples that were collected within a particular date range.  We should be able to specify those dates as part of a call to assign_clades.  This is related to discussion in #3 in that we'll need to be sure that when we pull the sequence data, we get everything that has a collection date within the specified range.
For more specificity, here's a suggestion that I'm not at all committed to: we could introduce command line arguments seq_start_date and seq_end_date and keep anything with seq_start_date <= collection_date <= seq_end_date.

@elray1 quick clarification: how would these start and end dates interplay with the --released-after parameter we're already using when getting sequence data via the NCBI API?
Is the starting range of the sequence collection date the same date we'd use as --released-after on the API call, or a different parameter altogether?

I believe it should at least be closely related (maybe released-after = seq_start_date - 1??). But I'm not sure.  Do you know of a place where these dates are documented?

I couldn't find anything definitive on the relationship between the API's released-after parameter and the collection-date in the metadata.
collection-date definition from the metadata schema:

The collection date for the sample from which the viral nucleotide sequence was derived

reference to "released after" property of NCBI's virus dataset downloads:

genomes released after

Let's chat about how to get a definitive answer. In the meantime, I'll use your released-after = seq_start_date - 1 to get started.


Adjust /etc/ssh/sshd_config automatically

We noticed that if you do not have PermitRootLogin set in /etc/ssh/sshd_config, upgrades are blocked.
I suggest that instead, the file should be updated if the setting is missing to say: PermitRootLogin yes, which is the CentOS 7 default. Then you don't need to block on this.

In technical terms, this can be done without much difficulty.
That being said, I think that it should only be done when there was no PermitRootLogin present in the global config.
We wouldn't want to override a global PermitRootLogin no or PermitRootLogin prohibit-password automatically.
So in cases with an explicit PermitRootLogin we would preserve the current behavior, and with an implicit default, we would set a new value automatically and let the user know through a non-blocking report message.
@toddr Does that sound about right?

Correct. The blocker is when it is not, explicitly, set. My suggestion is that the blocker should never assert because if it is not set, then we should set it to the default for RHEL 7

It seems like PermitRootLogin is not blocker anymore at least in leapp 0.14.0 / leapp-repository 0.16.0 when sshd_config is untouched.
Have you tried latest packages from http://repo.almalinux.org/elevate/el7/x86_64/ ?

Actually the change is now:
        if not config.permit_root_login and config.modified:

Which means the user won't be notified if permit_root_login is unmodified. This means that on upgrade, they will switch logic from PermitRootLogin yes to PermitRootLogin no and potentially get locked out. Isn't that bad?

That makes sense if we're expecting default config values provided to change with the upgrade - but I don't think that's something that can be relied upon.


SHIRO-693 - Update plugins

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Shiro-pr/95/

Thanks @coheigea !!


Add getters and setters for kundensystemId (#453)

Adds getters and setters for kundensystemId, see #453.

I have a sense that this would allow implementing an imperfect workaround for #458 outside the library, i.e. in the user application code. Let's try to address #458 inside the library instead.

Merge? :thinking:


Adapt to angstrom 0.7.0 API changes (when it's out)

https://github.com/ocaml/opam-repository/pull/10286
Instructions

replace parse_only with parse_string

All tests in test suite passed.
Closing issue.


Fix incompatability with loader version 15

Fixes #566 as well as updating dependencies

Thanks for the PR!


Recognize Github discussions activity

I think it would be great to incentivize answering discussions in the GitHub discussion tab to boost activity there. Especially since the author there can actually mark specific comments as solutions.
We could consider implementing multipliers, like 3x for answering a question, and 1x or 0.5x for the author marking an answer as the solution. This would encourage people to accept answers more, since a lot of people don't do that, which can make it look like a question is still unresolved.

Linking https://github.com/nuxt/nuxters/issues/4


Feature request: Support jsdoc style typescript comments in js files

Typescript supports a way to add jsdoc-style comments (that indicate type information) within js files (https://github.com/Microsoft/TypeScript/wiki/JSDoc-support-in-JavaScript). This is quite a valid way to use Typescript without moving to ts files.
But these files don't seem to get picked up or processed by typedoc to generate relevant documentation. Is it possible to support this?
(TS compiler api should already have ways to understand this comments, the way VSCode does)

@bendtherules TypeDoc should already support this? Are you passing the allowJS compiler option? If so, please let us know how we can reproduce your issue. Thanks!

Repro repo: https://github.com/bendtherules/repro-typedoc-issue-jsdoc/tree/without-ext-types
Trying to run: npx typedoc --tsconfig ./tsconfig.json --out ./docs ./src/
But it doesn't have any methods of Calculations in the output.
In master branch, the ,js files import stuff from other .ts files, whose types do show up in output (but not from js files).

Hi, just checking back.
Is this problem reproducible on someone else's end?
If this doesn't work, is there any simplified example which is known to work?

I was able to get it to work with some slight modifications.
I changed the input file to explicitly be the js file. The TS compiler was complaining about overriding the file so I added an outDir to the tsconfig.json. With this configuration TypeDoc was able to generate the Calculations docs.
npx typedoc --tsconfig ./tsconfig.json --out ./docs ./src/index.js


Add support for sensor install on Debian 12 platform

Debian 12 is supported via BPF user-mode. Although our API returns results for Deb only for versions 9/10/11, this may just be an oversight on our api side to add 12 to the versions of the sensor in which it is supported.
Keeping this here for tracking purposes before adding any code.

API changes have been updated. This work now for Debian 12 platform to pull in the correct sensor version.


Dynamic Extras

Hi,
I think I have found a bug, or it's just me not understanding Svelte 😬
I'm trying to create a list where the items are both droppable and drop zone. When dropping, the items get reordered in the list. Unfortunately, this doesn't update the Extras injected into use:asDroppable. I made a full example of my problem in a svelte repl. Just let me know if the problem is unclear. 🙏🏻

have you tried my svelte-sortable-flat-list-view?


allow master & slave to work in parallel for RocksDB WAL tailing

Allow master and slave to work in parallel for RocksDB engine WAL tailing.
The slave can send a request for more WAL data to the master asynchronously, before applying the WAL data it just received. This way we can make the master work too while the slave is applying the data locally, increasing the likes of the next batch being ready to be processed already when the slave needs more data from the master.
The feature is turned on for the RocksDB engine only at the moment, as it does not work when sending the list of open transactions (which MMFiles uses).

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/517/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/518/


[Question] Dynamically set FloatWindow's MaximizeBox and MinimizeBox

I'm farily new to DockPanel Suite. I have created a custom FloatWindow and IFloatWindowFactory like this:
public class MapFloatWindowFactory : DockPanelExtender.IFloatWindowFactory
{
    public FloatWindow CreateFloatWindow(DockPanel dockPanel, DockPane pane, Rectangle bounds)
    {
        return new MapFloatWindow(dockPanel, pane, bounds);
    }

    public FloatWindow CreateFloatWindow(DockPanel dockPanel, DockPane pane)
    {
        return new MapFloatWindow(dockPanel, pane);
    }
}

public class MapFloatWindow : FloatWindow
{
    public MapFloatWindow(DockPanel dockPanel, DockPane pane) : base(dockPanel, pane)
    {
        FormBorderStyle = FormBorderStyle.Sizable;
    }

    public MapFloatWindow(DockPanel dockPanel, DockPane pane, Rectangle bounds) : base(dockPanel, pane, bounds)
    {
        FormBorderStyle = FormBorderStyle.Sizable;
    }
}

However with this approach, the MaximizeBox and MinimizeBox property of my DockContent is not respected. How would I implement this?

I'm trying to get this to work as well. DPS hides all the buttons on a FloatWindow when none of its contents have an enabled and visible close button. There's no simple way to disable or hide just the close button without using the WinAPI or messing with CreateParams, so this is a good way to hide it if the minimize/maximize buttons are already hidden, which is the default case.
Unfortunately there doesn't seem to be a good extension point in DPS to change this behavior. This is handled in FloatWindow.RefreshChanges, which is internal. I think that should be changed to protected internal virtual.
https://github.com/dockpanelsuite/dockpanelsuite/blob/b9e958c7b98d7838b183493191acfb9744f65fd8/WinFormsUI/Docking/FloatWindow.cs#L284-L315
The alternative is to mess with underlying Form so that the ControlBox property is partially ignored. This actually works surprisingly well. To unhide the buttons, override the CreateParams to set the styles WS_SYSMENU and WS_CAPTION.
const int WS_SYSMENU = 0x00080000;
const int WS_CAPTION = 0x00C00000;
var cp = base.CreateParams;
cp.Style |= WS_SYSMENU | WS_CAPTION;
return cp;

All references within the framework to ControlBox and FormStateControlBox should be reviewed to insure that adding those styles back in doesn't break anything. To me it appears that everything is ok if the FormBorderStyle is not None and HelpButton is not enabled.
You also want to make sure the close button is disabled in the cases where it was previously hidden. It turns out that setting ControlBox to false disables the close button in addition to hiding all the buttons, so nothing special needs to be done to accomplish this.

The current code base was written for VS2005 theme mainly, and hasn't be touched much ever since.
If you do want a better float window (to work with VS2015 themes for example), you really should consider custom rendering of the window, and add your own buttons, like VS itself does.

I too wish to promote a floating window to top level for better multi-monitor support.
Did anyone have I working solution or any progress?
Going to play with the above suggestions....

`        public MapFloatWindow(DockPanel dockPanel, DockPane pane) : base(dockPanel, pane)
{
MinimumSize = ((Form)dockPanel.ActiveDocument).MinimumSize;
MaximumSize = ((Form)dockPanel.ActiveDocument).MaximumSize;
}
    public MapFloatWindow(DockPanel dockPanel, DockPane pane, Rectangle bounds) : base(dockPanel, pane, bounds)
    {
        MinimumSize = ((Form)dockPanel.ActiveDocument).MinimumSize;
        MaximumSize = ((Form)dockPanel.ActiveDocument).MaximumSize;
    }`

Try this.


Icon request: Greenhost

 Greenhost

URL of the site or app (required): https://greenhost.net/
Logo resources for the icon (optional, but recommended):

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/Greenhost.svg
❓ How to add icon(s) to Aegis?

Because if you already haven't noticed, all the logos in the icon here are one-colored from the day one and we are not changing that guideline ever.

Some logos have one-colored versions and some don't. We try to make logos with no official one-colored version to one-colored as fatefully as we can.
If you don't like, well I'm sorry but breaking our design guidelines would make our icons not visually consistent anymore and it would look like a mess.
Note also that some logos are really hard convert to one-colored.


WebApp:cannot import name 'BlockBlobService' from 'azure.storage.blob' while running anything related to az webapp

Describe the bug
Command Name
az webapp create
However, running even az webapp --help causes this issue.
Errors:
cannot import name 'BlockBlobService' from 'azure.storage.blob' (/usr/local/lib/python3.8/dist-packages/azure/storage/blob/__init__.py)
Traceback (most recent call last):
python3/dist-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 528, in execute
    self.commands_loader.load_arguments(command)
azure/cli/core/__init__.py, ln 299, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
...
cli/command_modules/appservice/custom.py, ln 39, in <module>
    from azure.storage.blob import BlockBlobService, BlobPermissions
ImportError: cannot import name 'BlockBlobService' from 'azure.storage.blob' (/usr/local/lib/python3.8/dist-packages/azure/storage/blob/__init__.py)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az webapp create -g {} -p {} -n {} --runtime {} --deployment-local-git

Environment Summary
Linux-5.4.0-29-generic-x86_64-with-glibc2.29
Python 3.8.5
Shell: bash

azure-cli 2.0.81

Extensions:
webapp 0.3.1
vmware 1.0.0
azure-devops 0.17.0

Additional Context
Output of pip freeze
azure-common==1.1.26
azure-core==1.12.0
azure-nspkg==3.0.2
azure-storage-blob==2.1.0
azure-storage-common==2.1.0
azure-storage-nspkg==3.1.0
certifi==2020.12.5
cffi==1.14.5
chardet==4.0.0
cryptography==3.3.2
enum34==1.1.10
futures==3.3.0
idna==2.10
ipaddress==1.0.23
isodate==0.6.0
msrest==0.6.21
oauthlib==3.1.0
pycparser==2.20
python-dateutil==2.8.1
requests==2.25.1
requests-oauthlib==1.3.0
six==1.15.0
typing==<IP_ADDRESS>
urllib3==1.26.3

route to appropriate team

@streaak - I am unable to repro the issue. could you please try with the latest CLI version - 2.20.0.

closing due to lack of response. Please file a new issue if this is still a problem.


[Bugfix] Rewriting the base helper logic for HTMLBars to use streams and...

... fixing 1.9 application rerender bug

CLA is valid!


RFC: add support for UNIX timestamp input

Background
Often by default streaming data will have DATE/TIME types converted to UNIX timestamp. For example, when creating a table in Postgres with a TIMESTAMP type field...
simple_db=# \d t
                           Table "public.t"
 Column |            Type             | Collation | Nullable | Default
--------+-----------------------------+-----------+----------+---------
 id     | bigint                      |           | not null |
 t      | timestamp without time zone |           |          |
Indexes:
    "t_pkey" PRIMARY KEY, btree (id)
Publications:
    "dbz_publication"

simple_db=# select * from t;
 id |             t
----+---------------------------
  1 | 2022-09-27 08:32:57.14096
(1 row)

...and having a debezium connector creating a topic with the following (read) event...
{"schema":{},"payload":{"before":null,"after":{"id":1,"t":1664267577140960},"source":{"version":"1.9.5.Final","connector":"postgresql","name":"postgres","ts_ms":1664268325363,"snapshot":"true","db":"simple_db","sequence":"[null,\"24558624\"]","schema":"public","table":"t","txId":1334,"lsn":24558624,"xmin":null},"op":"r","ts_ms":1664268325364,"transaction":null}}

...you can see that the t timestamp data is converted to a UNIX timestamp:
"after":{
   "id":1,
   "t":1664267577140960
}

This event will be skipped in RW since we do not accept parsing UNIX timestamps at the moment:
WARN risingwave_source::connector_source: message parsing failed internal error: failed to parse column 't' from json: expect timestamp, but found<PHONE_NUMBER>140960, skipping

There are workarounds to deal with this, for example Kafka Connect has a timestamp converter that can convert the UNIX timestamp back to a formatted date string. However, in my opinion this is not very user friendly. That's why I suggest to add support for UNIX timestamp input (for TIMESTAMP, DATE and TIME) along our current formatted date string parsing.
Design
Obviously, we already support casting str to TIMESTAMP/DATE/TIME:
#[inline(always)]
pub fn str_to_date(elem: &str) -> Result<NaiveDateWrapper> {
    Ok(NaiveDateWrapper::new(parse_naive_date(elem)?))
}

#[inline(always)]
pub fn str_to_time(elem: &str) -> Result<NaiveTimeWrapper> {
    Ok(NaiveTimeWrapper::new(parse_naive_time(elem)?))
}

#[inline(always)]
pub fn str_to_timestamp(elem: &str) -> Result<NaiveDateTimeWrapper> {
    Ok(NaiveDateTimeWrapper::new(parse_naive_datetime(elem)?))
}

In order to parse the UNIX style timestamps we can utilize the unix-ts crate. This crate supports parsing to DateTime and NaiveDateTime:
"If the chrono feature is enabled, unix-ts also supports converting to chrono::DateTime and chrono::NaiveDateTime. This is done through the to_datetime and to_naive_datetime methods."
This way we can simply add the following casting functions:
fn unix_to_date(elem: &str) -> Result<NaiveDateWrapper>
fn unix_to_time(elem: &str) -> Result<NaiveDateWrapper>
fn unix_to_timestamp(elem: &str) -> Result<NaiveDateWrapper>

Future Optimizations
No response
Discussions
No response
Q&A
No response

Will Debezium convert date/time to UNIX timestamps (integers) as well?
I think the function argument of unix_to_timestamp should be integer, since it uses json integer type.

"after":{
   "id":1,
   "t":1664267577140960
}

As the json above suggests, that field is a number ("t":1664267577140960) rather than a string ("t":"1664267577140960").
Shall we implement the to_timestamp [function](https://www.postgresql.org/docs/current/functions-datetime.html#:~:text=39%3A53.662522-05-,to_timestamp,-(%20double%20precision ) and invoke it automatically in source when the input is a json number?

invoke it automatically in source when the input is a json number?

I don't think we should infer a timestamp from a number in normal JSON. But we can do that when it's Debezium JSON.

Will Debezium convert date/time to UNIX timestamps (integers) as well?
I think the function argument of unix_to_timestamp should be integer, since it uses json integer type.

"after":{
   "id":1,
   "t":1664267577140960
}

Yes both TIME and DATE are converted to integers as well:

               {
                  "type":"int64",
                  "optional":true,
                  "name":"io.debezium.time.MicroTimestamp",
                  "version":1,
                  "field":"ts"
               },
               {
                  "type":"int32",
                  "optional":true,
                  "name":"io.debezium.time.Date",
                  "version":1,
                  "field":"d"
               },
               {
                  "type":"int64",
                  "optional":true,
                  "name":"io.debezium.time.MicroTime",
                  "version":1,
                  "field":"t"
               }

Indeed, I forgot to change the argument type to integers 😅

By the way, we should clearly define the unit: is it in seconds, milliseconds or microseconds?

That's right it should be clearly defined. The above example is for date/time/timestamp in Postgres. So, Postgres -> Debezium:

TIMESTAMP: microseconds since epoch
DATE: days
TIME: microseconds

However, I am not sure yet how Debezium converts time/date datatypes from other relational databases. For example in MySQL, DATETIME(3) is in milliseconds, while DATETIME(6) is in microseconds.

I don't think we should infer a timestamp from a number in normal JSON.

Why not for JSON? Is it because for JSON in many cases there won't be a spec for converting timestamps to integers and vice versa? If so, then yes I also think that casting UNIX to TIMESTAMP should be limited DEBEZIUM JSON in the source connector. And then I guess implementing to_timestamp like @xiangjinwu suggested is the best solution 👍

done


ARROW-12701: [Website][Release] Include Rust and DataFusion commits, contributors, changes in release note

Note: this PR does not yet include DataFusion commits, although it should be trivial to add them once we ask what tagging conventions they are going to use.

Thanks for doing this @thisisnic! Could you also please change how n_commits is calculated? It should iterate over directories and total up all the commits.

Thanks for doing this @thisisnic! Could you also please change how n_commits is calculated? It should iterate over directories and total up all the commits.

How do you mean, sorry?  I thought that's what it currently does.

Thanks for doing this @thisisnic! Could you also please change how n_commits is calculated? It should iterate over directories and total up all the commits.

Done

Created a follow-up ticket to port the release note generation to archery at some point: https://issues.apache.org/jira/browse/ARROW-13483

Regenerated the release not for 5.0.0 with this patch: https://github.com/apache/arrow-site/pull/131

Regenerated the release note for 5.0.0 with this patch: apache/arrow-site#131

Thanks! @thisisnic there's a problem with the Rust committers; please see https://github.com/apache/arrow-site/pull/131/files#r678709259

@thisisnic I manually fixed the problem with the Rust committers and applied the fix in https://github.com/apache/arrow-site/pull/133. So we're OK for the 5.0.0 release and there's no great urgency, but let's fix the problem before we merge this PR.

@ianmcook Do you remember what you did to get the correct list on this ticket? I've been looking into it but a lot of what I find online is people complaining that they can't find that information after a squash & merge.

Closing this ancient PR and starting again in a new one


Babel runtime dependency will crash node (esm modules)

Issue
Running tests without any transpilation with nodejs will lead to issues with react-flow v9+. This wasn't the case before.
Reproduce
Repository with steps how to reproduce this issue:
https://github.com/andys8/react-flow-node-issue-example
Error
internal/modules/cjs/loader.js:1092
      throw new ERR_REQUIRE_ESM(filename, parentPath, packageJsonPath);
      ^

Error [ERR_REQUIRE_ESM]: Must use import to load ES Module<EMAIL_ADDRESS>require() of ES modules is not supported.
require() of<EMAIL_ADDRESS>from /home/user/dev/repository/react-flow-node-issue-example/node_modules/react-flow-renderer/dist/ReactFlow.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename extends.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from<EMAIL_ADDRESS>
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:13)
    at Module.load (internal/modules/cjs/loader.js:940:32)
    at Function.Module._load (internal/modules/cjs/loader.js:781:14)
    at Module.require (internal/modules/cjs/loader.js:964:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at Object.<anonymous> (/home/user/dev/repository/react-flow-node-issue-example/node_modules/react-flow-renderer/dist/ReactFlow.js:6:1)
    at Module._compile (internal/modules/cjs/loader.js:1075:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1096:10)
    at Module.load (internal/modules/cjs/loader.js:940:32)
    at Function.Module._load (internal/modules/cjs/loader.js:781:14) {
  code: 'ERR_REQUIRE_ESM'
}
npm ERR! Test failed.  See above for more details.

Based on this issue https://github.com/rollup/plugins/issues/381, this
https://github.com/wbkd/react-flow/blob/42019a6a6659b201beefb873d950db2983493fdf/rollup.config.js#L52
might be  a workaround. In that case, maybe skipPreflightCheck: true as suggested in this comment could be an alternative.
I tested that both workarounds result in react-flow building, while it wouldn't without. I don't know if this change would resolve the initial issue, but it could very well do so.

Thanks for the report. Which node version are you using?

Latest lts (v14). See: https://github.com/andys8/react-flow-node-issue-example#versions

This is really strange. We haven't change the babel plugins or the rollup config. The problem is that these packages are required by the current production build:
require('@babel/runtime/helpers/esm/extends');
require('@babel/runtime/helpers/esm/objectWithoutPropertiesLoose');

In v8.8.0 the weren't. Maybe there is a new option for one of the babel or rollup plugins. I will try to fix it.

can you try v9.0.3?

can you try v9.0.3?

Fixed the issue for me. Thanks!

Awesome, the issue is fixed. Great work!
https://github.com/andys8/react-flow-node-issue-example/blob/master/README.md#update-issue-fixed


bluez: 5.40 -> 5.43

Motivation for this change
Update Bluez to latest version
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Is there anything I should do abbout the failing tests?

No, the test failure appears to be unrelated. Thanks!


Issue while deploying API on remote server  (IIS)

Exception Info: uPLibrary.Networking.M2Mqtt.Exceptions.MqttConnectionException: Exception connecting to the broker ---> System.ComponentModel.Win32Exception (0x8009030D): The credentials supplied to the package were not recognized at System.Net.SSPIWrapper.AcquireCredentialsHandle(SSPIInterface secModule, String package, CredentialUse intent, SCHANNEL_CRED scc)

Generally an MQTT Device Client is design to run as a background process. If you host it in IIS or any web server, the process become stateless and hence won't be able to connect to MQTT server endpoint for a longer period of time. Deploying to IIS is also out of scope of this component. If you want a web page to ingest to MQTT broker, you need to look at ingesting messages from web page using MQTT over Websockets channel. I have implemented that also. You can take a look at the below repository.
https://github.com/aws-samples/aws-iot-core-dotnet-app-mqtt-over-websockets-sigv4
Hope this helps.


Add Support for Custom Language Grammars in processMarkdown

Hey there, I initially thought about forking the package to add a small feature but decided it would be cleaner to refactor it a little instead. I've already created a PR with the changes but thought I'd open an issue first to discuss.
I've added support for consumers of the package to pass custom language grammars via a langConfig option. This allows extending the syntax highlighting without modifying the core library.
Usage
import langLua from "shiki/langs/lua.mjs";

const { html } = processMarkdown({
  md: mdWithLua,   // Your markdown string with Lua code blocks
  langConfig: {
    langs: [langLua],  // Add custom Lua language grammar
    // aliases: {} can be added too if needed
  },
});

If this feature aligns with the direction of the project, I’d be happy to finalise the pull request. Let me know if you'd like any changes.
Thanks for considering!

Great idea! Thanks for making the PR I'll take a look


ERROR TypeError: this.dropdown.hostElement.nativeElement.firstChild.getBoundingClientRect is not a function

template:
                  <form class="form-material">

                        <ng-select [ngClass]="'ng-select'" [options]="countries"  [ngModelOptions]="{standalone: true}" [allowClear]="true" [(ngModel)]="selectedCountry">
                          <ng-template #optionTemplate let-option="option">

                            {{option?.label}}
                          </ng-template>
                        </ng-select>

                  </form>

error:
ERROR TypeError: this.dropdown.hostElement.nativeElement.firstChild.getBoundingClientRect is not a function
at SelectComponent.updatePosition (ng-select.es5.js:1807)
at eval (ng-select.es5.js:1634)
at ZoneDelegate.invokeTask (zone.js:421)
at Object.onInvokeTask (core.js:4740)
at ZoneDelegate.invokeTask (zone.js:420)
at Zone.runTask (zone.js:188)
at ZoneTask.invokeTask (zone.js:496)
at ZoneTask.invoke (zone.js:485)
at timer (zone.js:2025)
happens when you open select

This looks very similar to issue #287 and #289, which has been fixed in version 1.0.0-rc.5. Just to be sure which version are you using? If you are using rc.4, then pleas update to rc.5 to resolve this issue.

Since there has been no response, I assume that this has been resolved by upgrading to a newer version.


Cleaning a bit the search module and converting files to Kotlin

This branch was rebased on top of fm/search_not_working fix, so it can be actually tested.

CI is now green


Modernjs: doesn't work with latest version v2.60.3 because of dev/peer dependencies

Describe the bug
When upgrading modern-js packages to latest version the modernjs process end up giving us weird errors (like it doesn't find the .env file). After testing, we found that this is because module-federation depends on 2.60.6
Notice that modern-js packages are not in peer dependencies.

Would be good to add the modernjs packages as peer dependencies and let the consumer use the correct range version.
Thanks!
Reproduction

Used Package Manager
pnpm
System Info
System:
    OS: macOS 15.0.1
    CPU: (8) arm64 Apple M1
    Memory: 73.98 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.13.1 - ~/.nvm/versions/node/v20.13.1/bin/node
    Yarn: 1.22.22 - ~/.nvm/versions/node/v20.13.1/bin/yarn
    npm: 10.5.2 - ~/.nvm/versions/node/v20.13.1/bin/npm
    pnpm: 9.12.0 - ~/.nvm/versions/node/v20.13.1/bin/pnpm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 131.0.6778.86
    Safari: 18.0.1

Validations

[X] Read the docs.
[X] Read the common issues list.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Module federation issue and not a framework-specific issue.
[X] The provided reproduction is a minimal reproducible example of the bug.

@GiancarlosIO do you have a repo we can look at

Can you provide the reproduce repo ? Because the devDeps only related with types , i want to know which package cause this

@GiancarlosIO dev deps should not install on your box, only on ours.
Can you supply repo? the packages we have in dependencies have no weird interdependences that would load another version of modernjs in to it.


Include pylupdate commands

Could you include commands concerning UI translation management please?

Implemented in v 0.2.0

Thanks!


Hindi Doesn't Work.

When i select hindi from language section and press on generate i get the following error message -
Using ready reference
Using folder reference
None  test
C:\Tools\xtts-webui\temp\resampled_audio_8657cc3a-7662-40ec-b5f9-d0f93f283953.wav Test
Traceback (most recent call last):
File "C:\Tools\xtts-webui\venv\lib\site-packages\gradio\queueing.py", line 489, in call_prediction
output = await route_utils.call_process_api(
File "C:\Tools\xtts-webui\venv\lib\site-packages\gradio\route_utils.py", line 232, in call_process_api
output = await app.get_blocks().process_api(
File "C:\Tools\xtts-webui\venv\lib\site-packages\gradio\blocks.py", line 1561, in process_api
result = await self.call_function(
File "C:\Tools\xtts-webui\venv\lib\site-packages\gradio\blocks.py", line 1179, in call_function
prediction = await anyio.to_thread.run_sync(
File "C:\Tools\xtts-webui\venv\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "C:\Tools\xtts-webui\venv\lib\site-packages\anyio_backends_asyncio.py", line 2144, in run_sync_in_worker_thread
return await future
File "C:\Tools\xtts-webui\venv\lib\site-packages\anyio_backends_asyncio.py", line 851, in run
result = context.run(func, *args)
File "C:\Tools\xtts-webui\venv\lib\site-packages\gradio\utils.py", line 678, in wrapper
response = f(*args, **kwargs)
File "C:\Tools\xtts-webui\modules\text2voice\generation.py", line 592, in generate_audio
output_file = XTTS.process_tts_to_file(
File "C:\Tools\xtts-webui\scripts\tts_funcs.py", line 443, in process_tts_to_file
raise e  # Propagate exceptions for endpoint handling.
File "C:\Tools\xtts-webui\scripts\tts_funcs.py", line 433, in process_tts_to_file
self.local_generation(
File "C:\Tools\xtts-webui\scripts\tts_funcs.py", line 335, in local_generation
out = self.model.inference(
File "C:\Tools\xtts-webui\venv\lib\site-packages\torch\utils_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "C:\Tools\xtts-webui\venv\lib\site-packages\TTS\tts\models\xtts.py", line 526, in inference
text = split_sentence(text, language, self.tokenizer.char_limits[language])
KeyError: 'hi'
I have reinstalled the entire program 3-4 times with all necessarcy files and dependencies and it still gives me same error. All the other languages are working fine only hindi is not working. Please look into this and fix the error.

Same here I’m facing this issue but no update from developer

Same here I’m facing this issue but no update from developer
i manually installed using the methods provided in the github.. i managed to bypass this error..but the generated hindi audio is total gibberish... its just random sounds..like umm or ahh ...

Same here I’m facing this issue but no update from developer
i manually installed using the methods provided in the github.. i managed to bypass this error..but the generated hindi audio is total gibberish... its just random sounds..like umm or ahh ...

Have the same issue.
Gibberish output on models 2.0.2 and 2.0.3.


Remove carriage return from WSL distribution names.

Summary of the Pull Request
Checks the Distribution name for \r and removes it if necessary.

References
#1168

PR Checklist

[x] Closes #1168
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
One line to check for and remove any carriage return characters.

Validation Steps Performed
I've installed a second distribution and tested this to ensure that it now functions properly.

Thanks for fixing this!


Remove osd-operators-registry targets

osd-operators-registry is obsolete; and aws-account-operator was never integrated into it anyway. Remove make targets that existed solely for its use.

/lgtm
Straightforward.


Allow MB of RAM/HDD

This issue has been moved here from https://github.com/tredly/tredly-host/issues/38. Please see the original thread for more details but continue discussion here.

I think that Tredly currently only allows a minimum of 1GB of RAM and 1GB of HDD. People should be able to set less than this.

@pointybeard i believe this was implemented with 1.0 json schema?

This has been implemented.


HHH-9707 - Nulls first behavior on SQLServer database

Proposed fix for HHH-9707.

@maedabr, thanks for the pull request. I will make the changes I mentioned in the comments on your second commit.

I just tested on SQL Server 2012 and 2014 and nulls first/last is still not supported. Did you actually test using SQL Server 12?
The issue you mentioned is closed as "Won't fix":
https://connect.microsoft.com/SQLServer/feedback/details/329337/nulls-ordering-nulls-first-nulls-last-sql-2012
I've confirmed that your fix for SQLServer2008Dialect does work for SQL Server 2012 and 2014 though.
I've pushed to master and will push to 4.3 shortly.
@Jona5, if you are still having problems, then please open a new Jira.


Add prophecy to ignore files in writeError

Before

After

Not sure if 5.x is still supported by I'm still on symfony 5.

Please pull request agains stable.


Process explorer should remember it's location and dimensions

It seems to always launch in the center of the active window, so whenever I'm open it I need to drag it out to the other monitor.

Do this request still accept PR? I would like to try it, we can record the last position and size and create it with this value the next time we open it.
and I found that the current code expects the same window as vscode every time it starts, If needed, we can add a switch to control whether to follow the parent view's window or not?
https://github.com/microsoft/vscode/blob/dd93163b73418600c48ede98aa78b072e7a0cde4/src/vs/platform/issue/electron-main/issueMainService.ts#L352

@weartist sounds good 👍, I think just  a single global position is probably the best as the process explorer is shared across all windows.

@weartist sounds good 👍, I think just  a single global position is probably the best as the process explorer is shared across all windows.

let me gave it a try

Position is good but size seems pretty random
https://user-images.githubusercontent.com/1794099/214634753-b1f0e2e9-0d5c-406d-a374-e59770a0ba58.mov

@jrieken do you have multiple monitors? What's the devicePixelRatio of your monitors?

yes, multi-monitor and no clue what devicePixelRatio is. Is that zoom related or how do I find that out?

@jrieken yeah that's the scaling factor of the window, open devtools and run window.devicePixelRatio when the window is on each monitor.

Thanks for the intro - it prints 2 on both screens
Also looping in @bpasero since this is working fine for normal code windows

Hm, not sure there is something specific I do. Getting the bounds for persistence is done here:
https://github.com/microsoft/vscode/blob/0785b1c28204ca0f0584fb2fe8810142ab989069/src/vs/platform/windows/electron-main/windowImpl.ts#L1125
And restoring here:
https://github.com/microsoft/vscode/blob/0785b1c28204ca0f0584fb2fe8810142ab989069/src/vs/platform/windows/electron-main/windowImpl.ts#L1219

@bpasero suggested to share the logic for the regular Windows which is probably the fix here, I didn't want to invest much effort into this though.

@jrieken Hi, can you still reproduce the problem, I tested it with multiple monitors and did not find the size problem, slittle strange

Was able to catch another important issue - if 2 monitors are used on windows, and then the second monitor disconnected, the explorer window gets inaccessible as if it is "stuck" on the disconnected monitor.
Repro steps:

use windows machine and a laptop with an external monitor
open vscode workbench
open process explorer and drag the window to the second monitor
close the process explorer
disconnect the second monitor, open the process explorer again
⚠ notice that the process explorer window is not accessible as if it is left on the other monitor, the only way to get it back is to connect the external monitor again; this issue is persists even after an OS reboot


Single quotes or double quotes?

See https://github.com/apple/swift/pull/40888.

There's no preference for single or double quotes - the efficiency of either operation is going to be a wash, especially when compared with the cost of flushing the ostream they're being dumped into.
In the interest of minimizing code churn, I would take the typo fix but I don't think it's worth it to go through and regrade all the usages of string/char literals in the printers.

I understand that enforcing a single/double quote policy everywhere is unmanageable. Could you still merge the quote changes along with the typo fix?

Pull request is not mergeable.


write post-condition assertions without introducing a new step

@hvanz reported an issue where he is trying to have an assertion after an action, and that produces a stuttering step which is not wanted.
He thought the built in assert could be interpreted in such a way where the step with the assertion doesn't make it through the trace. This is not true, but it is a valid idea.
Also https://github.com/informalsystems/quint/issues/1236 might help solving this issue.

introduce post-conditions inside runs.
Yes, probably that what's needed here.

So, given that a typical way of writing tests is:
run fooTest =
  init
    .then(action1)
    .then(all { assert(property1), unchangedAll })
    .then(action2)
    .then(all { assert(property2), unchangedAll })

then the generated trace from this run will contain the two undesired steps from unchangeAll. I couldn't find a good way to rewrite this test to avoid generating stuttering steps.
I thought ideally one could write something like the following, with .assert(...) (or other name) as post-condition:
run fooTest =
  init
    .then(action1)
    .assert(property1)
    .then(action2)
    .assert(property2)

Then internally it would be easier to distinguish which states must be discarded when generating the trace.

The proposed post-condition assert makes since to me.
But isn't unchangedAll only needed in the last step of a run, in cases where you want to assert something in a final state? E.g., couldn't the first example be
run fooTest =
  init
    .then(action1)
    .then(all { assert(property1), action2 })
    .then(all { assert(property2), unchangedAll })

This is equivalent but the problem is still there in the last state, right? Also I think it could be confusing to have property1 and action2 in the same line, with property1 referring to the action in the line above.

Just thinking about it. Probably the easiest solution would be to introduce an operator that works similar to  then in the sense then it shifts primed variables to non-primed, but it evaluates the predicate and discards the new state.


Prettify all of common test output.

This uses cth_readable to:

silence error_logger output to the shell unless a test fails
silence ct:pal output to the shell unless a test fails

I have currently not baked in any way to disable this behaviour, but I
figured if it is required, there is time to do it before the final
3.0.0 release.
What it looks like

Cool, +1 if I read this correctly and you do plan to add a way to switch it off before we go 3.0.0.

Yeah. It's just that I want us to cut the last beta soon and get this through for it. I have a few shell bugs to tackle in the upcoming days too.

That being said, what format should the switch take? {ct_prettify, false}, {ct_readable, false} or something else?

Yea, probably {ct_readable, false} so it is nearly the same name as the app.
I'd also like to include eunit formatters. I can quickly add that and make a PR if you agree that would be good to include in a similar fashion to this.

Yeah that could be good to have too.

Should add the app to rebar.app.src applications

@tsloughter done with the switch
@talentdeficit interested in your review of this.

+1
i like the idiom you used for ensuring things were only present in a list once, i'll have to remember that

Well if the entry was there twice, it will be there twice again. It actually ensures they are there once or as many times as they were before. But yeah, it's nifty.


Correct spacing typo between "if" and "you"

Challenge Escaping Literal Quotes in Strings has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
In the instructions section, there appears to be no space between if and you in this text:
"...so that ifyou were to print..."
Just need the space to render between these 2 words.

I think this can be resolved by simply removing the emphasis on if (it is not used in a similar case earlier in the instructions), since the resulting string will likely be logged to the console for the user anyway in upcoming changes (see #9363).

That's not an uncommon problem with italics - typically that's where the use of an em-space (] [) is handy.
Just removing the the emphasis, which I find slightly distracting anyway, solves the visual problem.

We can go ahead and remove the <em> tags on the "if". I add them there because this challenge had lots of confusion on interpreting what that last string printed to the console actually means.

I'll take this up.

Haven't seen this one updated. I can implement this fix if you haven't already @Manish-Giri

@dpcleveland sorry, it's almost done. Just need to submit the PR.

@Manish-Giri No worries! Sounds good

var myStr = "I am a "double quoted" string inside "double quotes"."; // Change this line

@BariqFitri
This place is not for put solution on here. You could help people at FreeCodeCamp Chat Room or Forum. GitHub Issues are for reporting bugs on the website only.
Happy Coding!


Prove generated code with different provers separately

While CVC4 is able to prove all code on its own, Z3, Alt-Ergo or both together are not capable to do that. We should analyse for which code parts they are unable to show correctness and what the reasons are.


Move OSX AzDO CI leg off "DotNetCore-Mac" pool

The DotNetCore-Mac pool only has 5 agents and is usually congested:
https://github.com/dotnet/source-build/blob/a8ca959fc8a74f3730fa49e2554436706657bb08/.vsts.pipelines/jobs/ci-osx.yml#L6
When I try switching pools, CoreCLR native build fails with:
CMake Error at src/corefx/System.Globalization.Native/CMakeLists.txt:13 (message):
  Cannot find utypes.h, try installing libicu-dev (or the appropriate package for your platform)

OSX instructions for this dependency are at coreclr Documentation/building/osx-instructions.md#icu.
The correct pools are documented at https://github.com/dotnet/arcade/blob/master/Documentation/VSTS/VSTSOnboarding.md#agent-queues

Completed by https://github.com/dotnet/source-build/pull/860.


Monster Statblock pdf Bleeds off of page

I was working on a monster statblock and I have to check the print to pdf every so often because the formatting doesn't always match for my computer, and this is an extreme case. The statblock seems fine in the live-updater, but it bleeds to the end of the page in the print to pdf. I also checked other monster statblocks I have, and this one is the only one that has this problem.
Edit page picture.
Print-to-pdf picture.
EDIT: Homebrewery Link.

I have no idea what I did. But it's fixed now.


Adding module update commands in README.md

Code of Conduct

[X] I have read and agree to the project's Code of Conduct.

Module Version
v<IP_ADDRESS>2
Motivation
For customers with already deployed Report Module it will be good to include Upgrade of the Module, maybe after Installation part.
Suggestion
Adding section in Readme file Upgrading the Module after Installing the Module section, with brief description how to upgrade Reporting Module and if required other depended modules.
Update-Module -Name VMware.CloudFoundation.Reporting

Pull request  #57 opened to add an Updating the Module section for both the module and its dependencies.


Add protegrity

Hey @Kaustubh-Natuskar , Can I fix this issue .

Try to fix all issues where we have to add single companies 😃

hey @Kaustubh-Natuskar , Kindly close this issue ( Issue is already solved ) .


not every Docker image uses requirement.txt in ibm_cf

Some Docker images in ibm_cf doesn't use requirements.cf, for example Dockerfile.python36
Should all images we consistent?

@JosepSampe what is your input on this?

Those I created yes (Dockerfile.python38 and Dockerfile.slim), the other ones, I don't know. I never used them.
This one you refer (Dockerfile.python36) can be deleted, it is the same as the Dockerfile. The only difference is the build time. With Dockerfile you have to install everything, and with Dockerfile.python36 everything is installed from ibmfunctions/action-python-v3.6, however it might contain different python packages versions.

@JosepSampe which one we need there? I actually using Dockerfile.python36 all the time :)

If you were using Dockerfile.python36, you can use: https://github.com/pywren/pywren-ibm-cloud/blob/master/runtime/ibm_cf/Dockerfile


Various minor docs fixes

Remove defaults to pull everything from conda-forge and pyviz/label/dev
Fix rendering the Plotly plot on the extension guide

Ok so the only way I found to display the plotly plot is to load require.js via sphinx.


Passwords get lost

Hello @bnjbvr,
I got this error in my logs:
Error when polling accounts: when retrieving operations: Weboob error: Traceback (most recent call last):
  File "py/endpoint.py", line 111, in <module>
    content = BankHistoryHandler(id, password, website).post(bankuuid)
  File "py/endpoint.py", line 84, in post
    return self.load_from_connector(name)
  File "py/endpoint.py", line 46, in load_from_connector
    raise Exception("Unexpected Weboob error: %s" % unicode(e))
Exception: Unexpected Weboob error: Backend(bp): Configuration error for field "password": Value "undefined" does not match regexp "^(\d{6}|)$"

And the import of operations didn't work.
It seems that the password for the bp module (banque postale) is not defined, but it appears in the databrowser.
Any idea?

Thanks for the report. I have no idea what's going on with these lost passwords. It's troublesome, because it tends to happen more and more often... I'll doublecheck the password path, to see if there can be some race condition somewhere.

Re,
It seems to be another thing than just the password loss. After the reboot, I get this on manual imports (same bank):
Error -  when fetching operations for access: when retrieving operations: Weboob error: Traceback (most recent call last):
  File "py/endpoint.py", line 111, in <module>
    content = BankHistoryHandler(id, password, website).post(bankuuid)
  File "py/endpoint.py", line 84, in post
    return self.load_from_connector(name)
  File "py/endpoint.py", line 46, in load_from_connector
    raise Exception("Unexpected Weboob error: %s" % unicode(e))
Exception: Unexpected Weboob error:

What if you try by hand?
cd weboob
source ./env/bin/activate
python ./py/endpoint.py bank_id login password

I tried. The script always freezes here :
https://github.com/bnjbvr/kresus/blob/master/weboob/py/endpoint.py#L90
The loop is never entered, I tested with a print without condition.

My bad, the module is not up to date. It seems the weboob-config does not work in a virtual env.

Can you try this to update weboob and weboob modules:
cd weboob
source ./env/bin/activate
pip install -U -r requirements.txt

I did a full reinstall ...
Now I have all the passwords of by bankaccounts deleted in the databrowser....

Well I lost ALL my passwords in cozy

Sounds like quite a general issue with the cozy data-system. Have you opened a bug on the DS repo?
If it's identified as a general cozy issue, can we close this one?

I did open on the DS repo ;)
Let's close this one


add changelog for v1.10.0 release

Summary

add changelog for v1.10.0 release

Release Note
n/a
Documentation
n/a

Codecov Report

Merging #2087 (145edf4) into main (3e49138) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2087   +/-   ##
=======================================
  Coverage   26.33%   26.33%
=======================================
  Files         129      129
  Lines        7564     7564
=======================================
  Hits         1992     1992
  Misses       5317     5317
  Partials      255      255

Help us with your feedback. Take ten seconds to tell us how you rate us.

Merge and release in the AM!