Process expression data

Need to process expression data for each gene to for feature engineering.

Complete using gene expression notebook.


Deserialize Delta struct

I can easily deserialize Vec, but not whole Delta struct.
Is it possible to add Deserialize derive macro here to simplify this process?

Sure, missed that, will do

Done & published
Please let me know if it works as expected

Done & published Please let me know if it works as expected

Thanks, looks like it's working as it should!


URL in description and README broken

The URL in the repository description and README are pointing to githubschool's copy of the game instead of yours.
Please fix both so they point to your copy of the game at https://githubschool.github.io/github-games-poonam-dua

fixed the issue by changing the URL to plau GITGAmes


Harvestcraft trees are in Betweenlands

and they shouldn't be

1.10 Unabridged/Abridged discontinued.


IList map to return IList rather than Iterable

Thanks for this library, it is helping improve state management for some of our more complex widgets.
What is the reasoning that an IList<T>.map returns an Iterable<T> rather than the more specific IList<T>? If we return another IList we can chain it with other methods and pass it around. I find myself having to explicitly call .toIList() every time anyway.

As far as I remember, that choice was based on List.map returning an Iterable, it matches their APIs/impedances.

Ok thanks for the explanation.
Didn't consider the laziness. I suppose you want IList.map to be lazy just like List.map and if you return an IList it would be forced to evaluate everything.


Fix 'Duplicate Code' issue in multiple files

CodeFactor found an issue: Duplicate Code
It's currently on:
Graph\Tarjans_Algorithm\TarjansAlgo.c:4-109
Dynamic Programming\TarjanAlgo\Tarjan.c:4-109
Commit 5eb32fd6dfa66515f0b18b227eca2a69a0c21559

Please assign me this work.


WebResource, handlers (and my status)

Hi! So today I crudely tried to sledgehammer the Telerik AJAX control suite into this by adding fiddy (missing) org. files to the CoreWebForms + a few hacks and ended up with a data bound RadGrid on an ASPX page! So far so good, but one of the hacks was that WebResource (e.g. assembly CSS) seems to load without content type (Content-Type: application/octet-stream) so I hacked that in too (that part not done yet?).
Next up handlers - I spose this wont work:
<system.webServer>
	<handlers>
		<add name="Telerik_Web_UI_WebResource_axd" path="Telerik.Web.UI.WebResource.axd" type="Telerik.Web.UI.WebResource" verb="*" preCondition="integratedMode"/>

But I just found this: https://github.com/CoreWebForms/CoreWebForms/blob/9f41f142be383771825987b3e28a2155e3f787f4/docs/designs/Handlers.md - so I guess that's the way to go?
So far to very good! The Telerik suite does A LOT on top of it all, so I'm very please to have gotten so far! Nice work good people! :) There might be a bright .NET not-named-core in the future for all of us!

To answer my own question: Yes - but be aware if the handler uses Page it favors async.
Found this (I'll split to actual issues later if not resolved):
$get is missing from MicrosoftAjax.js dist (from DomElement.js)
var $get = (Sys.UI.DomElement.getElementById = function (id, element) {
becomes:
Sys.UI.DomElement.getBounds = function (element) {
and its not because of the terser

I clean a lot of the org. files that I added in the process - these are the only changes so far:
Added:

Extensions: JavaScriptSerializer + the needed classes - used org. files (until replaced by System.Text.Json as note says)
Extensions: AsyncPostBackTrigger
Extensions: ExtenderControl
WebForms: CustomValidator + 2 related

Hacked:

Extensions: ScriptResourceHandler + ScriptManagerExtensions - added content type in a return tuple. I see that its the basically the AssemblyResourceLoader that is not yet completed - so besides the content type - its also missing the  performSubstitution / WebResourceRegex for CSS going src: url('<%=WebResource("..
WebForms: Moved ApplyAppPathModifier to public public static class ExtensionsPublic
Changed a few var $x = to window.$x = so they won't get reduced

Besides that it looks very good / only minors! 🤓

thanks for the update! which version have you been testing on?

the latest source from here (from when I did it 2 weeks ago) so I could do the modifications


Type definitions incompatible with PIXI v5 rc2

The filter typings (types.d.ts) of 2.7.x extend from PIXI.Filter<{}> but it seems that PIXI.Filter is not generic (anymore?). Removing the brackets simply made the type checker happy.

Working on it: #195

Works with the latest typings. Closing.


Add testcase for Issue 1341

#1341

Codecov Report

Merging #1342 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1342      +/-   ##
============================================
- Coverage     86.05%   86.04%   -0.02%
+ Complexity     5478     5477       -1
============================================
  Files           128      128
  Lines         19162    19162
  Branches       4257     4257
============================================
- Hits          16490    16488       -2
- Misses         1715     1716       +1
- Partials        957      958       +1

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/alibaba/fastjson/util/TypeUtils.java
85.06% <0%> (-0.18%)
409% <0%> (-1%)

Continue to review full report at Codecov.

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


Fix visibility issue in jest, separate react dom & comp prop extension

from each other.
In cases that we have to add surface wrappers () elements to the DOM, JEST may get confuse the wrapper with the compoennt itself and some assertions may fail.
This commit ensures that jest related attributes cannot be added to the wrappers and they stay invisible to the framework. (Technically, we could just prevent any non-verified attributes on these wrappers, but that might be too drastic to add now).

Do we still need this @reshadi?


[Website] Resources add : Java In Education

During an MP community call (Nov. 2021), I shared the JCP initiative a few of us started working on under the JCP EC group about enhancing the reach of Java in Education.   This project is today a global effort JUG led and supported by the JCP program.  I stated that once ready to share. The resource links ought to be added under the MP Resources page.
The plan then was to continue improving the resources of the project by gathering global supporters and cases of use, etc. Today, the project has a good standing, and it continues to improve.
Task:
Under Resources:
Add Java In Education https://jcp.org/java-in-education
Once this is in the website, we will share it with the MP Community so that MicroProfilers who are members of JUG communities could either share their histories under the Videos section (Edwin already did) or find ways to become involved in the SharingIsCaring initiave.

It looks great, @rstjames 💯


HC-446: Added support for authentication in Otello

This PR updates Otello such that it now supports authentication so that end users wishing to use Otello outside of a cluster environment can do so. Authentication is done by using AWS' Secrets Manager feature, where the username and password information are stored.
When initializing Otello to use authentication, it will prompt you to enter an AWS Secrets Manager ID, which is the ID of the Secret that stores the username and password information that has access to a HySDS cluster. The base class was updated such that when the object is initialized, it will create a requests Session object and if authentication is set, it will go out to SecretsManager and set the Basic Authentication in the Session object. Furthermore, the base class initializer method was updated such that you can optionally pass in a session object in order to persist a single session throughout a program execution rather than creating a new one for each base class initialization.

@mcayanan  should we add an option for users to enter their password manually if they dont have a secret_manager_id

@DustinKLo I did initially go that route. So, the initial implementation was user enters password, then we base64 encode it to generate a token, which we put into the config.yml. Then, on the base class side, we decrypt this token, then set up basic auth. Not terribly hard to do, but I think that can be a future development ticket.

i was thinking of only storing the token as a class attribute and re-using it

@DustinKLo I did initially go that route. So, the initial implementation was user enters password, then we base64 encode it to generate a token, which we put into the config.yml. Then, on the base class side, we decrypt this token, then set up basic auth. Not terribly hard to do, but I think that can be a future development ticket.

i was thinking of only storing the token as a class attribute and re-using it

I was having a hard time figuring out how to basically pass that token along from the initializer step to the base class through an attribute. The only other way I can think of is basically when the base object gets initialized, that's when you'd get prompted for the password instead of at the initializer level. But the problem with that approach is that the way PODAAC wants to use it, it's not going to work for them. Basically, on their end, they will be calling otello from a lambda.


[CARBONDATA-1238] Decouple the datatype convert from Spark code in core module

Decouple the datatype convert from Spark code in core module.

Can one of the admins verify this patch?

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3193/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/598/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3194/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/599/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/613/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3208/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/616/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3211/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/631/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3227/

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3232/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/637/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3248/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/653/

LGTM


DOC: landing page is jinja, not rst

A variant of the doc build where the landing page is done via a jinja file (hence writing html), rather than a rst file.
This should make the life of a web-designed much easier: more flexibility.
By editing the jinja + CSS (we can insert landing-page specific css, eg to make the body wider)

This is unfinished: there is clearly some html + css work to be done. I don't think that I'll be able to finish it, but I hope that it helps people

I also added transparency to the gradients so that they give the right level of brightness both in white background and black background. I had to use inkscape to choose these colors as it has a more advanced colorpicker.

I've fixed some of the css. The cards remain not quite right

I improved the CSS a bit. Stopping here for lack of time. I think that this is quite OK and could be merged.
The lower part of the landing page still needs love.


Outside Collaborator Report

Add a GitHub report for outside collaborators across the org.

This was done with adc38d28c726e5c4efeebd73ac449cf3e6bf208f


pyyaml < 4.2b is vulnerable, see CVE-2017-18342

"In PyYAML before 4.1, the yaml.load() API could execute arbitrary code. In other words, yaml.safe_load is not used." - https://nvd.nist.gov/vuln/detail/CVE-2017-18342

Resolved by a4dcfd758e837db2482f2379c7a1312569e4f191


Add optional --static-php-version flag

The Issue

https://github.com/ddev/ddev/issues/4430

How This PR Solves The Issue
Adds optional --static-php-version value flag, that creates .ddev/.env.php-patch-build file with STATIC_PHP_VERSION="value".
Manual Testing Instructions
Install add-on using DDEV binary from:

https://github.com/ddev/ddev/pull/6406

ddev add-on get https://github.com/stasadev/ddev-php-patch-build/tarball/20240717_stasadev_optional_version --static-php-version=8.0.10
ddev restart

Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

Works great for me. can be pulled when we're ready.


Support AeroSpike Sink

add sink support of aerospike

(Jove Github Bot) added it to the current sprint.


evaluate eval set every n iterations

Hi,
I would like to track the training and validation loss in training. The only way I can do it is to provide train_data and valid_data in the eval set. However, since train_data is huge, adding train_data to the eval set significantly slows down the training. Is it possible to evaluate the eval set every n iterations.
Or is there better way to do this? Thanks!

metric_freq in cli.


RDB-26 and RDB-34

Fix for RDB-26 and RDB-34

Replicate reactions filter using attack filter from filterselection.elm / deckbuildselections.elm / search.elm ensuring references to attacktypes/filters/etc. are duplicated for reactions, test locally.

Separate 34 from 26, then we can resolve 34 on its own - far simpler.


strax.chunk.CannotSplit error

Describe the bug
I'm seeing an error like below during reprocessing of some data. It looks like the error occurs in merged_s2s.
Exception in thread build:merged_s2s
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/context.py", line 1215, in make
    for _ in self.get_iter(run_ids[0], targets,
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/context.py", line 1099, in get_iter
    generator.throw(e)
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/context.py", line 1072, in get_iter
    for n_chunks, result in enumerate(strax.continuity_check(generator), 1):
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/chunk.py", line 303, in continuity_check
    for chunk in chunk_iter:
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/processor.py", line 286, in iter
    raise exc.with_traceback(traceback)
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/mailbox.py", line 255, in _send_from
    x = next(iterable)
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/plugin.py", line 616, in iter
    yield from super().iter(iters, executor=executor)
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/plugin.py", line 427, in iter
    yield self.do_compute(chunk_i=chunk_i, **inputs_merged)
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/plugin.py", line 636, in do_compute
    _, result = result.split(t=self.sent_until,
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/chunk.py", line 170, in split
    data1, data2, t = split_array(
  File "/opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax/chunk.py", line 376, in split_array
    raise CannotSplit()
strax.chunk.CannotSplit

To Reproduce
Enter the latest container on Midway and try making, e.g., event_info_double for Run 23413:
st = straxen.contexts.xenonnt(<latest cmt global version>)
st.make('023413', 'event_info_double')

I also saw this error for Run 20318.
Versions
>>> straxen.print_versions()
Working on midway2-0413.rcc.local with the following versions and installation paths:
python  v3.8.10 (default, Jun  4 2021, 15:09:15) [GCC 7.5.0]
strax   v0.16.1 /opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/strax
straxen v0.19.3 /opt/XENONnT/anaconda/envs/XENONnT_development/lib/python3.8/site-packages/straxen

Duplication with https://github.com/AxFoundation/strax/issues/490


Can't run multiple instances of backstretch

Hi, when I try to add two different background to two different div or on the body and a div, only one of the two is displayed.
Like this page : http://geek1983.no-ip.org/caro_new/
The background is displayed, but not the #sliders div.
P.s. You can see the javascript on the bottom of the code.
Can someone help me please?
Thank you

I have still the same issue with Jquery 1.11.4 and Jquery 2.1.4. I try to setup batchstrected images on the body tag and then in 3 other div's on the same page.


feat: support nginx vars in ApisixRoute v2alpha1

Please answer these questions before submitting a pull request

Why submit this pull request?

[ ] Bugfix

[x] New feature provided

[ ] Improve performance

[ ] Backport patches

Related issues

#302

Features

This Pull Request aims at exposing the nginxVars field in ApisixRoute/v2alpha1, the apisix-ingress-controller will translate the nginxVars to vars in Route.

Need docs for the usage of  nginxVars.

@gxthrj Please take a look when you have time.

这个对应的是 route里面的 这个vars变量吧.  我没理解错吧.
  "vars": [
    [
      "scheme",    => subject
      "==",           => op
      "http"          => value
    ]
  ],

这个对应的是 route里面的 这个vars变量吧. 我没理解错吧.
  "vars": [
    [
      "scheme",    => subject
      "==",           => op
      "http"          => value
    ]
  ],

Right！


Question about FRM of conversions.

Hi, I wonder these following instructions:

vfcvt.f.xu.v vd, vs2, vm/vfcvt.f.x.v  vd, vs2, vm
vfwcvt.f.xu.v vd, vs2, vm/vfwcvt.f.x.v  vd, vs2, vm
vfwcvt.f.f.v vd, vs2, vm
vfncvt.f.xu.w vd, vs2, vm/vfncvt.f.x.w  vd, vs2, vm
vfncvt.f.f.w vd, vs2, vm

Do they depend on FRM (floating-point rounding mode register)
Thanks.

In general, widening conversions are unaffected by the rounding mode, but narrowing conversions are affected by the rounding mode.

Thanks. Andrew.
What about same element width:
vfcvt.f.xu.v vd, vs2, vm/vfcvt.f.x.v vd, vs2, vm
This is integer to float conversion, same width, for example, int32 to float.
Is it affected by rounding mode?

FP to integer conversions always depend on the rounding mode, regardless of width.
For integer to FP conversions, the answer depends on the widths.  For example, the FP64 mantissa is wide enough that all int32/uint32 values can be represented exactly.  But not all int32/uint32 values can be represented exactly in FP32.  In general, we need the significand (including the implicit leading 1 bit) to be greater than or equal to the integer width (excluding any sign bit) to avoid rounding.
As a side note, don't forget about this caveat in the spec: "All vector floating-point operations use the dynamic rounding mode in the frm register. Use of the frm field when it contains an invalid rounding mode by any vector floating-point instruction—even those that do not depend on the rounding mode, or when vl=0, or when vstart ≥ vl--is reserved."

Thanks Andrew. Closing it.


Password Reset

If a user forgets their password or if a user would like to reset the password we should provide them a link to reset it.

Link is already in login
can refer to this tutorial  looks like could be a good guideline

Password reset currently working, tutorial posted did not pan out, may need to refactor current reset password with more explicit instructions.


Skip testing debuginfo install on coreos images

This does not work, it's not an important use case and we currently don't have time to investigate this.

@psss would it make sense to add a "support matrix" to prepare/install docs? It starts to seem there are some combinations that are not supported, both yet and never, it might be nice to document them.

Yeah, sounds good to me. Probably just some simple form would be sufficient. Not spend too much time on that, 'cause, as you mentioned, the almighty install will deprecated that.

I was also thinking of moving parameters (package, expected message, ...) from this test file into respective package manager implementations, to keep them close to code

This also is quite reasonable, it took me a bit of time to find the right place for the exception, so +1 from me.

Oh! Green tests! Finally. Haven't seen this for quite some time! ;-)


Update search indexing for new templates

What changed, and why it matters
After updating to the new Furo theme, the change in page structure caused our search indexing to fail. This fixes that and the issues relating to it:

takes account of the new headerlink (it's a # rather than a paragraph symbol so we need to pick out the tag rather than parsing a symbol out of the text)
ignores the "back to top" link and the light/dark and menu item icons
does not index tables of contents (for example the lists on the "concept" pages)

Use the SEARCH instructions to create an opensearch and test what gets indexed.

LGTM, I've had it running locally and it indexes whereas current main does not.


Refresh Token revocation - all tokens for the associated subject are deleted instead the only one that was requested to revoke

Currently there is a bug with the way the token revocation works that was reported here  The issue is currently the token is revoked by ClientId and SubjectId instead of the Key(handler) so if the same user is logged in on multiple devices, if he logs out from one device it logs him out from all devices.
I know this has been fixed in the latest version and might be released sometime soon. But is it also possible to fix this in 1.5.3 since that is the version we are using and might not be able to upgrade soon?
It should be a one line change in the RevocationEndPoint.cs file.
https://github.com/IdentityServer/IdentityServer4/blob/1.5.3/src/IdentityServer4/Endpoints/RevocationEndpoint.cs#L183
In line 183
Instead of  await _refreshTokenStore.RemoveRefreshTokensAsync(token.SubjectId, token.ClientId);
Update to await m_refreshTokenStore.RemoveRefreshTokenAsync(handle);

We're following this model for support for old versions:
https://hueniverse.com/the-backwards-commercial-license-647290f7e38b
If your company wants this, then contact us directly for commercial support. Thanks.

Thanks. Can you tell me when the latest version of IdentityServer will be released which has this fix?

Check the milestones.


Add another ecommerce dataset

Can you share a bit more about the data you are planing to add. @neelshah2409

Can you share a bit more about the data you are planing to add. @neelshah2409

I have the dataset for understanding the product sale of flipkart.


CMakeLists.txt hardcodes /etc/... paths

Please use GNUInstallDirs[1] so that sysconfdir can be passed to cmake in a standard way.
(In Nixpkgs we now have to patch CMakeLists.txt, and that's pointless extra work for downstreams.)
[1] https://cmake.org/cmake/help/v3.0/module/GNUInstallDirs.html

This is only available in CMake 3.0 or higher, currently we support 2.6 and up. This will break compatability.

You may propose a PR/patch and try to not break compat with 2.6-3.0. Thanks!

Closing due to inactivity. Feel free to submit a PR which is compatible with 2.6 and above.

See also #899.

We should be able to deal with this now, as the minimum version requirement for cmake now is 3.4.2, so there should not be any worries about broken compatibilities any more.

We should be able to deal with this now, as the minimum version requirement for cmake has been raised to 3.4.2, so one does no longer need to worry about any broken compatibilities.

@bjornfor: I have added the GNUInstallDirs.cmake module locally, but are not quite sure yet how to properly integrate it into the build configuration we currently have. I'll open a new branch for this, so you or anyone else can have a look at it. I am sure there are people who know more about cmake than I currently do...

@Nightwalker-87 : GNUInstallDirs.cmake is distributed with cmake, you don't need to add it yourself to the codebase.
To use it just
include(GNUInstallDirs)

# Now you have access to all the variables mentioned in the docs
message("/etc files will be installed to ${CMAKE_INSTALL_FULL_SYSCONFDIR}")

I understand, but is this really all that has do be done unrelated to any custom configuration?

I think so. Heres an (untested) diff of how I'd replace two hardcoded /etc instances in stlink:
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 90378f9..e81f621 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -2,8 +2,9 @@ cmake_minimum_required(VERSION 2.8.7)
 set(CMAKE_USER_MAKE_RULES_OVERRIDE cmake/c_flag_overrides.cmake)
 project(stlink C)
 set(PROJECT_DESCRIPTION "Open source version of the STMicroelectronics Stlink Tools")
-set(STLINK_UDEV_RULES_DIR "/etc/udev/rules.d" CACHE PATH "Udev rules directory")
-set(STLINK_MODPROBED_DIR "/etc/modprobe.d" CACHE PATH "modprobe.d directory")
+include(GNUInstallDirs)
+set(STLINK_UDEV_RULES_DIR "${CMAKE_INSTALL_FULL_SYSCONFDIR}/udev/rules.d" CACHE PATH "Udev rules directory")
+set(STLINK_MODPROBED_DIR "${CMAKE_INSTALL_FULL_SYSCONFDIR}/modprobe.d" CACHE PATH "modprobe.d directory")
 set(STLINK_STATIC_LIB ON CACHE BOOL "Install static lib")

 if( IS_DIRECTORY ${LIB_INSTALL_DIR})

Ah, the remaining "/etc/" instances are in comments or documentation, so I think that patch above is al that's needed. (The patching in nixpkgs was done with sed and no GNUInstallDirs so it's not directly applicable.)

@bjornfor: Can you test it?

It does work, it played around with this script:
#!/usr/bin/env bash

set -x

cmakeFlags=
#cmakeFlags="$cmakeFlags -DCMAKE_INSTALL_SYSCONFDIR=/etc"

cmakeFlags="$cmakeFlags -DCMAKE_INSTALL_PREFIX=/local"
#cmakeFlags="$cmakeFlags -DCMAKE_INSTALL_PREFIX=/tmp/stlink-prefix"
#cmakeFlags="$cmakeFlags -DCMAKE_INSTALL_PREFIX=/usr"
#cmakeFlags="$cmakeFlags -DCMAKE_INSTALL_PREFIX=/usr/local"
#cmakeFlags="$cmakeFlags -DCMAKE_INSTALL_PREFIX=/opt"
#cmakeFlags="$cmakeFlags -DCMAKE_INSTALL_PREFIX=/opt/stlink"

nix-shell -p cmake pkgconfig libusb --command "mkdir -p build && cd build && rm -rf ../build/* && cmake $cmakeFlags .."

I also edited CMakeLists.txt to print out interesting variables.
I guess the default CMAKE_INSTALL_FULL_SYSCONFDIR might not be what everyone expects (not that I know what that is though), so perhaps it's a good idea to update documentation to suggest building with -DCMAKE_INSTALL_SYSCONFDIR=/etc to ensure no surprises in the common case LSB distro case. Background: testing shows that -DCMAKE_INSTALL_PREFIX=/usr results in CMAKE_INSTALL_FULL_SYSCONFDIR=/usr/etc.

Let me take a closer look at this related documentation on the web.
We should have an idea about how this works in detail.
I'll update our project documentation if necessary.

I think -DCMAKE_INSTALL_SYSCONFDIR=/etc should rather go into our cmake config as a default by referring to: https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html
However I'm not sure yet, whether we

should preset -DCMAKE_INSTALL_PREFIX= or
leave that flag to the user or
do a preset (first option) with allowing him to overwrite this preset flag.

@bjornfor: What's your opinion on this?

To have the least support issues, I think I would preset -DCMAKE_INSTALL_SYSCONFDIR=/etc and leave -DCMAKE_INSTALL_PREFIX= unset.
But I'm really unsure of presetting /etc (vs just documenting the option). I mean the option does have a default value already, so with /etc as overridden default it would no longer behave as documented in GNUInstallDirs.

Ok, thanks for your feedback. I'll add this to our documentation then and we can ensure the defaults as documented in GNUInstallDirs, which also appears to be in the sense of the opening post.

I think we should take a closer look at the previous configuration now, after including GNUInstallDirs to ensure there are no conflicts regarding directory paths. Otherwise we may find that old settings reconfigure standard settings of the former which clearly would not help...

@bjornfor: As proposed, I've added a note on cmake build options related to GNUInstallDirs to our compiling manual. Thus this issue is now fully addressed and resolved.

Thanks!


chore!: remove support for Telescope builtin file_browser

Closes #12

[x] Removed support for Telescope builtin file_browser
[x] Added section to README for using https://github.com/nvim-telescope/telescope-file-browser.nvim

Hi @jvgrootveld could you please review this? Thanks!

Ty for your contribution. I was a little absent lately due to different things. I just want to let everyone know will check all Open issues and PR's shortly.

@axieax I reviewed your PR, and I find it awesome that you created a PR to remove an (now) not-working feature and updated the readme with an alternative if others used it. I'm not sure about all the other changes in this PR which are also not described.
Could you explain these extra changes? To merge this feature asap I would recommend to change the PR so that only picker is removed and the Readme is updated.
I could just change it myself but I want to give you the choice to be added as a contributor 😄

@axieax I notice you are busy. I will wait for your change till next week. After that I will create a copy of your fork and update the code myself.
I am not sure if you will be added as a contributor if I update it but I still will mention you in the PR.


description about project

add a quick intro about what the project is about

fixed #3 and added project about


[Feature Request]: surfacing nested pipeline input parameters

Feature description
Components participating in a nested pipeline have their own custom configurations. For example, in one component, an English language model is loaded to process input data while in another component, it is configured to load a Spanish language model.
A flat layout of all rolled up configuration parameters from all components can be challenging to interpret by pipeline users. It may be worth considering to improve user experience with grouping or hierarchical layouts.
Describe alternatives you've considered
No response
Anything else?
No response

@yuanchi2807 When you refer to nested Pipeline input parameter... how "nested" are we talking here? The way I understand how Tekton works is:

(Top Level) Pipeline Parameters -- these are needed to start, or have a fallback

(Each) Task Parameters -- these can use the Pipeline Parameters, reference other values from other tasks

Here is the Tekton variable reference doc: https://tekton.dev/docs/pipelines/variables/
Could you provide more information?

@andrewballantyne Thanks for the link. On the DSP 'create run' configuration screen, how should I expect pipeline and task parameters be organized under sections?

@yuanchi2807 Creating a new Pipeline Run only has to answer the Pipeline Parameters. Not sure there is a good way to set internal task parameters. If you want some connectivity there, I imagine you write a Pipeline parameter and then your Task Parameter references the Pipeline Parameter for execution.
kind: PipelineRun
...
spec:
  params:
    - name: 'foo'
      description: 'Must be given before I can start'
  # this is probably nested under pipelineSpec or something
  tasks:
    - name: 'my-first-task'
      params:
        - name: 'bar'
          value: '$(params.foo)'
...

Something like that... this was typed without testing, so refer to the Tekton docs to make sure you get the right syntax when you're testing it out.
I may have misunderstood your question from Slack, so apologizes for that. Let me know if this clears things up.

@andrewballantyne Using SDK v1, can I expect each func_to_container_op or  load_component_from_file from kfp.components to have its own configuration section on the 'Create run' screen? Will try a couple of toy cases to confirm. Thanks.

@yuanchi2807 I can't speak much about the SDK in play here. @HumairAK @harshad16 might know more on that front.
I'm purely speaking from a Tekton point of view (the run aspect of DSP). It is my understanding that there is only 1 layer of input parameters, and that is of the Pipeline Run (defined from the Pipeline)

@yuanchi2807 , as a temporary workaround, we can use the fact that ODH GUI presents RUN input parameters according to ABC, so we might define a name convention for input parameters.
For example, ALL default Global parameters can have a prefix: global

All parameters for step A and the English language can have a prefix A_eng, the same step for Spanish will have a prefix A_sp, or something similar.

In the code, we can guarantee that a step-specific parameter has precedence.
In order to reduce mistake probabilities, we can add some checks of input parameter names.
It is not an ideal solution, but this approach allows us to define global parameters while also allowing for overwriting them on a per-step (nested step) basis. Furthermore, the ODH GUI will consolidate the step-related parameters.

How one wires up the Pipeline itself is entirely up to the creator of the Pipeline. But I don't think the Dashboard UI should help this out in anyway. We show top-level Pipeline params, and let the configuration manage itself imo.
Am I to understand that this is an issue we can close? This sounds data-driven and not something that the UI will manage outside of what it is currently doing today.

@andrewballantyne What @roytman proposed workaround works. This issue is certainly not as high priority as flowing logs back to DSP dashboard. Can reopen later.

We spoke more about the use-case here and it has come to my attention we are not talking about a small number of parameters for their use-case. Although yes, this seems to be the design intent by Tekton parameters, it seems it's a bit clunky in our UI. They have several dozens of parameters they are trying to inject into the two tasks they have, and in order to manage that they are prefixing their values and associating them in the Pipeline as expected (and noted in our previous conversations on this ticket).
This looks very wonky in our UI having to scroll through 40 some odd parameters dealing with all the various entires of partial relationship. I wonder if we can find a better way, maybe a grouping mechanism or something to help with large input-based parameters; some metadata we can adhere to help make collapsible sections.
@yannnz @kywalker-rh We might need to meet about this and discuss an approach to solving this from a UX point of view.
@HumairAK @gmfrasca we should consider investigating kubeflow pipelines for any existing functionality that would be expected that we follow. Your help here would be appreciated.
cc @jedemo FYI

Another workaround proposed by @blublinsky is to capture the dozen or so parameters as JSON. So at pipeline input page, several chunks of JSON are inserted by user. Not sure if they are consistent with UI design.

Another workaround proposed by @blublinsky is to capture the dozen or so parameters as JSON. So at pipeline input page, several chunks of JSON are inserted by user. Not sure if they are consistent with UI design.

This has a limitation I suspect -- we use inputs and not text areas -- so a large JSON blob will effectively be unmanageable in our UI as well.

Migrated to: https://issues.redhat.com/browse/RHOAIENG-569


sanitycheck: suppress warnings from EDT

We don't need sanitycheck's use of EDT to report warnings, we'll get
them from the build system.  So the warnings are just duplication and
noise, thus lets always suppress them.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

This should cause the slew of warnings like this to stop showing up:
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1
warning: unit address and first address in 'reg' (0x5008c000) don't match for /soc/peripheral@50000000/gpio@1


Shouldn't local_only be True by default?

I was writing a plugin system for pip that works across virtualenvs. (Install plugin once on global python, and then it works on all virtualenvs)
Naturally, I tried writing a plugin for pipdeptree.
The only problem is that pipdeptree puts local_only as False by default, which means invoking it from a virtualenv causes it to show all global packages.

This can be fixed by vendoring pipdeptree in the plugin itself, and running with python -I ... (isolated mode) but that looks too fragile.

It can also be fixed by the plugin, by just translating and hacking on the command line arguments.

But local_only=True feels like a sensible default, especially when get_installed_distributions() has this signature -
Signature: get_installed_distributions(local_only=True, skip={'python', 'argparse', 'wsgiref'}, include_editables=True, editables_only=False, user_only=False)

I am happy to author a pull request if that's okay :)

The --local-only flag is exactly the same as --local flag of pip freeze which also has false as the default.
Does your plugin system require the virtualenvs to have access to global site-packages? Because the --local/--local-only flag is only applicable in that case.

The --local-only flag is exactly the same as --local flag of pip freeze

Right, makes sense!

Does your plugin system require the virtualenvs to have access to global site-packages?

Not necessarily.
But it currently does that, so the plugin doesn't have to vendor pipdeptree.
Either that or we must integrate plugin with pipdeptree itself. But I guess that's too much to ask when it's not even merged into master yet!?
(FWIW, it's takes like a single line  and import for making the plugin).
We're currently having a discussion over at https://github.com/pypa/pip/issues/3999.

I just noticed https://github.com/naiquevin/pipdeptree/issues/2. Hopefully, this plugin system will solve user confusions like these...


fix only update cluster url after apply resources

issue: https://issues.redhat.com/browse/ACM-13722

LGTM

/lgtm


No Module name cleverhans.loss

I am trying to run the example https://github.com/tensorflow/cleverhans/blob/master/cleverhans_tutorials/mnist_tutorial_keras_tf.py
I am getting the following error

I wasn't able to reproduce this.
I expect that you have two or more versions of CleverHans installed. You're running the tutorials in the new version but the import command loads the modules from an older version.
To check this idea, run "python -c "import cleverhans; print cleverhans.file" and see if the path is different from the path to the tutorial script you were running.
Feel free to re-open the issue if this doesn't turn out to be what happened.

I get the following error running the command

I tried on two different machines

Windows

MAC

And is getting the same problem.

In the message the correct class member was rendered as file* (note the
bold font)*, but the correct member is file (note the two underscore
characters before and after "file". It seems that the response was
processed by some markup processor that uses double underscore to signify
bold font. Let's see if this response also shows the "file" as a bold
"file".
On Thu, Jul 12, 2018 at 4:16 PM Md Johirul Islam<EMAIL_ADDRESS>wrote:

I tried on two different machines

Windows
2.
MAC
And is getting the same problem.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/cleverhans/issues/453#issuecomment-404636887,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AImIscioqMBt4RIEKTJQT2t8tm26xFdtks5uF66LgaJpZM4VMjmi
.

--
Leo Mauro
Asesor
Dirección de Servicios Telemáticos
Universidad Simón Bolívar

I edited my previous comment so that the markdown processor doesn't remove the underscores. @johirbuet the command should work as displayed now

@goodfeli
I am getting the following output

You want
python -c "import cleverhans; print cleverhans.__file__

You want print not dir

@goodfeli
This is what i get

Is that the path to the directory where you are running the tutorials? If not, then the problem is that you have two versions installed, and the old version is getting imported.

I am facing the same issue as above, I downloaded the tar.gz file from PyPI for cleverhans https://files.pythonhosted.org/packages/e8/ea/c057263636b14c28923b42ed8ef52c0ba24d38e05fc3720f62df80a2f735/cleverhans-2.1.0.tar.gz. It doesn't contain loss.py

@npapernot
Now it makes sense to me. I was also installing the application using pip. And there was actually no module named cleverhans.loss. I checked manually in the package path.
Thanks

@chrisdonahue It's not cleverhans-related issue. It is intended behavior with pip -e. You can simply try other packages and get the same error. Or remove -e and success:
!pip install git+https://github.com/tensorflow/cleverhans.git#egg=cleverhans
from cleverhans.loss import CrossEntropy

pip -e (editable mode) installs package inside <current dir>/src. Individual package directory has to be added to sys.path to make it importable. Python will automatically fill sys.path at startup, but not at runtime. Of course pip has no capability to force running Python process to update its sys.path.
So with pip -e, you need either append package directory to sys.path, or restart python interpreter.


Allow auto-generated accessor functions to be defined in another module

This will allow us to fix https://github.com/NREL/PowerSystems.jl/issues/422.

@daniel-thom you can merge as soon as Travis passes


Enable a way to configure the Cache characteristics from the Ibex Core Top

Simply I noticed this in the Ibex. The cache is fixed always to:
module ibex_icache #(
  // Cache arrangement parameters
  parameter int unsigned BusWidth       = 32,
  parameter int unsigned CacheSizeBytes = 4*1024,
  parameter bit          ICacheECC      = 1'b0,
  parameter int unsigned LineSize       = 64,
  parameter int unsigned NumWays        = 2,
  // Always make speculative bus requests in parallel with lookups
  parameter bit          SpecRequest    = 1'b0,
  // Only cache branch targets
  parameter bit          BranchCache    = 1'b0
)

The cache is only instanced in the following line.
https://github.com/lowRISC/ibex/blob/master/rtl/ibex_if_stage.sv#L185
The parameters are not controlled definitely in any external way possible. It would be neat to control the size of the cache from the parameters. Is this the way intended?

You're right that this isn't currently wired out to the top-level. I think this is mostly because we haven't needed to do so yet. Do you have an application where it would help? If so, can you describe what you'd like to control?

I think this is partially to minimise the number of parameters fed out to top-level. Those that are there generally get fed into multiple different places. For the ICache however I think all of those parameters are purely internal to the ICache so you can modify them without effecting any other bits of Ibex (@tomroberts-lowrisc can confirm).
So if you're interested in different cache configurations I'd suggest simply altering the parameters used where the ICache is instantiated, though I'd caution that this may be the only tested parameter set so there may be new bugs to find in different parameter settings.

You're right that this isn't currently wired out to the top-level. I think this is mostly because we haven't needed to do so yet. Do you have an application where it would help? If so, can you describe what you'd like to control?

What I am doing is to instance the Ibex core in the Chipyard multi-core generator. This is for testing OpenTitan in Berkeley's hardware. They have an actual ICacheParams where some of the variables can be directly used from the configurations.

So if you're interested in different cache configurations I'd suggest simply altering the parameters used where the ICache is instantiated, though I'd caution that this may be the only tested parameter set so there may be new bugs to find in different parameter settings.

I will create a Fork then, and add the parameters directly by simply modifying the code then.
Thank you very much. Will close the issue.

A lot of the parameters in the I$ are fairly specific - but it seems reasonable to expose the cache size at the top level. I'll re-open this issue to track that enhancement.


module level multiline strings getting mangled into unreadability, forcing code changes

As a YAML library developer, I have quite a few module level multiline strings:
data = yaml.load("""\
a: 1
b: 2
""")

Black transforms this into
data = yaml.load(
    """\
a: 1
b: 2
"""
)

I can live with the two extra lines, but one wants to have the triple-quotes aligned (as this is not the same string, I understand that this is not done automagically):
data = yaml.load(
    """\
a: 1
b: 2
    """
)

That is of course unacceptable from a readability point of view, so one would tend to align the mapping keys in the document as well:
data = yaml.load(
    """\
    a: 1
    b: 2
    """
)

now for this YAML document that leads to the same data being loaded, but that is not always the case (e.g. not if the document starts with an explicit document marker).
I can provide some mechanism in the library for (auto-) dedenting of the input on calling load, but it seems to me that the transform of a multiline string to the next line as if it doesn't fit, is contrary to why one is using multiline strings in the first place: "so you don't have to cope with long strings\nwith embedded newlines that don't fit\non any reasonably sized line width")
Black should, for correct behavior, only consider the first line of a multi-line strings and check if that (with the preceding assignment and function call) on the line or not, whereas now it seems black takes the rather dumb approach of considering the whole of the multi-line string and then comes to the conclusion that it does not fit on a single line (which cannot really be a surprise), and so uglifies the code by pushing the starting triple quotes to the next line.

There is a "workable" alternative, that doesn't get mangled by black:
str_raw = """\
a: 1
b: 2
"""
data = yaml.load(str_raw)

but I rather not have to make code changes in all of the places I have module level strings are used to work around this issue.

Thanks for the thorough analysis and the report. I believe the root cause of this is the same as behind #256 so going to close this out in favor of that


Fixes UI bugs

Changes:

Fix null pointer exception causing command box not to clear after a get command
Fix schedule box overflow
Username label in table is no longer truncated
Schedule list in friend main card is now responsive

Issues

Resolves #175
Resolves #179
Resolves #172
Resolves #170
Resolves #167

Screenshots

Schedule overflow fixed
Table label is no longer truncated
Schedule list is now responsive

Codecov Report

Merging #201 (748387b) into master (a615048) will increase coverage by 0.55%.
The diff coverage is 3.44%.

@@             Coverage Diff              @@
##             master     #201      +/-   ##
============================================
+ Coverage     70.38%   70.93%   +0.55%
- Complexity      773      774       +1
============================================
  Files           122      122
  Lines          2367     2350      -17
  Branches        255      254       -1
============================================
+ Hits           1666     1667       +1
+ Misses          611      593      -18
  Partials         90       90

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/FriendCard.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/FriendListPanel.java
0.00% <ø> (ø)

...n/java/seedu/address/ui/FriendScheduleDayCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/GameCard.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/GameListPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/model/friend/Friend.java
88.88% <100.00%> (+4.96%)
:arrow_up:

Continue to review full report at Codecov.

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


[GFX-1223] Generate stack unwind tables

Jira ticket URL (Why?)
GFX-1223
Short description (What? How?) 📖
Apparently we can get usable stack traces by just enabling generation of stack unwind tables on iOS. We don't need to enable C++ exceptions. We still lost parts of the stack trace, but at least the stack frame above panic is preserved giving us a chance to understand (and group) Filament related crashes.
Crash report, before:

Crash report (of the same crash), after:

FYI, #40 couldn't preserve the entire stack trace either, but only the frame above panic. So this PR achieves the same result without enabling C++ exceptions in Filament libs.
Testing
Design review 🎨

Affected areas 🧭

Special use-cases to test 🧷

How did you test it? 🤔

Manual 💁♂
Automated 💻

I realized omitting stack frame pointers was the final issue preventing us from seeing usable stack traces in Crashlytics. Keeping the frame pointers finally fixes Crashlytics reports. See here


FATA[10:27:55] Error connecting to k8s cluster: Get http://localhost:8080/version?timeout=32s: dial tcp <IP_ADDRESS>:8080: connect: connection refused

Hi,
I am running a cluster in my local machine by kind, which runs k8s in a container.
When I kubefwd, the error like the below occurs.
➜  capa-viewer git:(main) ✗ (☸ |kind-kind:local) sudo kubefwd svc
[sudo] password for jjangga:
INFO[10:27:55]  _          _           __             _
INFO[10:27:55] | | ___   _| |__   ___ / _|_      ____| |
INFO[10:27:55] | |/ / | | | '_ \ / _ \ |_\ \ /\ / / _  |
INFO[10:27:55] |   <| |_| | |_) |  __/  _|\ V  V / (_| |
INFO[10:27:55] |_|\_\\__,_|_.__/ \___|_|   \_/\_/ \__,_|
INFO[10:27:55]
INFO[10:27:55] Version 1.17.3
INFO[10:27:55] https://github.com/txn2/kubefwd
INFO[10:27:55]
INFO[10:27:55] Press [Ctrl-C] to stop forwarding.
INFO[10:27:55] 'cat /etc/hosts' to see all host entries.
INFO[10:27:55] Loaded hosts file /etc/hosts
INFO[10:27:55] HostFile management: Original hosts backup already exists at /root/hosts.original
FATA[10:27:55] Error connecting to k8s cluster: Get http://localhost:8080/version?timeout=32s: dial tcp <IP_ADDRESS>:8080: connect: connection refused

I 'guess' this happens because kind runs k8s in a container, but not pretty sure.
If someone suggest a possible solution, I will do that and share the result here.
Thanks!

What happens when you sudo -E kubefwd? If your exporting KUBECONFIG, it may be getting lost without it.

I'll close this until we can get more info. Thanks

@balibebas @cjimti
The same issue happens with microk8s and k3d, which run k8s in the container, as like kind.

What happens when you sudo -E kubefwd?

➜  capa-viewer git:(ops/tracing) ✗ (☸ |kind-kind:linkerd) sudo -E kubefwd
INFO[11:51:51]  _          _           __             _
INFO[11:51:51] | | ___   _| |__   ___ / _|_      ____| |
INFO[11:51:51] | |/ / | | | '_ \ / _ \ |_\ \ /\ / / _  |
INFO[11:51:51] |   <| |_| | |_) |  __/  _|\ V  V / (_| |
INFO[11:51:51] |_|\_\\__,_|_.__/ \___|_|   \_/\_/ \__,_|
INFO[11:51:51]
INFO[11:51:51] Version 1.17.3
INFO[11:51:51] https://github.com/txn2/kubefwd
INFO[11:51:51]
Expose Kubernetes services for local development.

Usage:
  kubefwd [command]

Examples:
 kubefwd services --help
  kubefwd svc -n the-project
  kubefwd svc -n the-project -l env=dev,component=api
  kubefwd svc -n default -l "app in (ws, api)"
  kubefwd svc -n default -n the-project

Available Commands:
  help        Help about any command
  services    Forward services
  version     Print the version of Kubefwd

Flags:
  -h, --help   help for kubefwd

Use "kubefwd [command] --help" for more information about a command.

If your exporting KUBECONFIG, it may be getting lost without it.

There is no environment variable $KUBECONFIG on my local environment. Is this what you wanted to check?
➜  capa-viewer git:(ops/tracing) ✗ (☸ |kind-kind:linkerd) echo $KUBECONFIG
# nothing is printed.

Thanks!

@jjangga0214 Just like kubectl, kubefwd needs a cluster config file in order to connect to a cluster. If you are using sudo without sudo -E it might not be able to find the default in ~/.kube/config.. Use sudo -E or try specifying the path to a config directly. sudo kubefwd -c /path/to/config svc NAMESPACE.
Does kubectl work?

@cjimti Ah, now I got it. Thank you so much. It works like a charm, for example like sudo -E kubefwd svc. Thanks again!


Fix incorrect indentation in documentation

The selector field belongs to the ACMEChallengeSolver according to the api
documentation, and we should not add another entry in the list for the dns01 key
In the acme section the selector key was not properly indented.

/assign @jakexks


Bad rule condition and restriction test cases

They're just testing nonsense - which is fine for fuzzing, but bad for proper coverage - need to replace/augment them

Tests:
EXTENDEDRULECONDITION (7)
RESTRICTION (16)


Simultaneous job execution not prevented when using (thread-based) Que workers (due to DB connection pool)

Hey @airhorns,
In the process of creating our job serialisation functionality mentioned in #2, we discovered that the locking approach we both use falls over when using Que's inbuilt thread-based worker count mechanism. You have to use a database connection pool for that, which causes a connection reuse bug where, for example, with 10 workers and 10 connections, there exists a 10% chance that while one worker thread is executing a job, another thread will reuse the connection which has the lock, and so mistakenly share the lock. The result is that multiple jobs execute simultaneously, which is what we're trying to prevent.
We realised we probably shouldn't use Que's thread-based worker scaling anyway, since there's not really much concurrency there thanks to Ruby's GIL. To scale, we must use multiple Que worker processes only.
I've just demonstrated that this issue does indeed also affect que-locks. Below is an anonymised version of the manual testing code if you want to reproduce. It's basically an implementation of the classic bank account increment concurrency problem. Apologies for not having the time to put together a runnable demonstration. Edit: See link
We strongly recommend that you add a warning to the readme that this gem is incompatible with Que's thread-based workers, and that for scaling, users should run multiple process instead.
CC @ceralena, @smontanari, @chrisnankervis

Ok, I've put together a minimal code example to demonstrate the bug: https://github.com/ZimbiX/que-locks-issue-3-demo

🤦 That makes a lot of sense! I will definitely add the note.
I think it might be possible to make que-locks work fine with the connection pool, but it would have some tricky ramifications I will have to think through and test. If a locked job checked out and used only one connection from the pool for the whole duration of the job's execution, then I think the semantics of the postgres locks would re-align with the expected job lock semantics and things would continue to work. This'd be a big hit to concurrency though and might be kind of unexpected for operators.
FWIW, I do think the threaded worker style is important to support. Yes the GIL prevents simultaneous Ruby execution, but Que (and sidekiq and whatnot) take this threaded approach because background jobs are often doing IO heavy work that means they could yield the GIL to other threads that need to do a bit more CPU work before their next IO.

Damn, I just realised my demo repo was private - fixed now.

That makes a lot of sense! I will definitely add the note.

Cheers for adding the note; looks good. Perhaps it would actually be good also to raise an error at worker startup if the worker count is greater than 1? That way, incorrect behaviour is fully prevented.

I think it might be possible to make que-locks work fine with the connection pool, but it would have some tricky ramifications I will have to think through and test. If a locked job checked out and used only one connection from the pool for the whole duration of the job's execution, then I think the semantics of the postgres locks would re-align with the expected job lock semantics and things would continue to work. This'd be a big hit to concurrency though and might be kind of unexpected for operators.

Ahh right, we'd considered permanently reserving a connection from the pool for the worker, but it didn't seem like a sensible solution. Reserving a connection just for the duration of the job execution though sounds like it could totally work, if that's supported. Potentially, you could do it conditionally, based on whether there are multiple worker threads. Or better: require the user to set some config to explicitly enable this behaviour and avoid the startup error.

FWIW, I do think the threaded worker style is important to support. Yes the GIL prevents simultaneous Ruby execution, but Que (and sidekiq and whatnot) take this threaded approach because background jobs are often doing IO heavy work that means they could yield the GIL to other threads that need to do a bit more CPU work before their next IO.

Totally fair

Hey @oeoeaio, are you able to share here the code you wrote at GreenSync to solve this by reserving database connections?


Fix subscription due query.

@rjmk I got a little sidetrack as when I was checking that I had changed the email text properly I realised that I was not getting any results back. The error was Truncated incorrect datetime value. I think these are the right changes, the format we send the date up is the correct one for MySQL and the due_date field. What I am unsure of though, is why this has been working on live site.

@ivanmauricio Has it broken on the live site? If not, is it broken locally or on staging?

@rjmk Just locally. Although I was checking live site and I was getting results intermittently. Earlier I had a few people showing up for subscription due emails, now I am not getting any results with the same dates. It's strange though, I am not getting any errors, just an empty array as the results.

@ivanmauricio The latter case is because querying subscription due emails adds a subscription charge. Once that charge is added, the member will no longer satisfy the 'subscription due' test

@rjmk ahh okay. I guess I should not be doing these tests on the live site then?

Ideally not. You can always clean up after yourself by delete from payments where date_created=cur_date() (or something like that)

@rjmk okay will do!

@rjmk As to regards to this problem, what should we do? It does not work locally without the changes. However everything seems to be fine on live site. Should I make an issue?

Is there any more detail beyond the Truncated incorrect datetime value? Also, is this when you run with or without NODE_ENV set?

@rjmk without.

I typed the query in manually on the CLI and got -
+---------+------+----------------------------------------------------------------+ | Level   | Code | Message                                                        | +---------+------+----------------------------------------------------------------+ | Warning | 1292 | Truncated incorrect datetime value: '2011-02-01T00:00:00.000Z' | | Warning | 1292 | Truncated incorrect datetime value: '2017-12-12T00:00:00.000Z' | +---------+------+----------------------------------------------------------------+

Maybe new Date(end).toISOString().split('T')[0]?

It must be a version issue. Anyway, it can be fixed via
var date_string = d => (new Date(d)).toISOString().split('T')[0]

Beat me to it! I think that should be in the node_modules/app somewhere

Hehe

@rjmk This is the only file in node_modules/app I found it in. It was a bit tied up so thought I'd make a new helper.

@rjmk This looks good. Have tested 3 places, one where main format_date function is used, one whereformat_due_date is used and of course where the new date_only is used. All seems to be working fine.

@rjmk Btw have you seem Richard's email? Should I just get rid of the last few payments?

@rjmk Crisis averted. Think I have deleted relevant payments now. 😌

@ivanmauricio I'm still not comfortable with how that function will deal with invalid dates. I think I have a solution. Gimme a bit

Handling the possiblity for error got to be too painful to do by hand. Had to bring in Sanctuary.  It broke everything but I had a great time trying to figure out why. I have got everything working, not sure if it is the best solution though. You've inspired me to carry on my functional learning! Is it functors that I have to get a grasp on to better understand this code?

@ivanmauricio I'm so happy you enjoyed this code and it's kept you interested in FP! :)
So, we use both the functor and monad instances for Either in this code. That said, I don't think you need to be thinking about functors and monads to massively understand this code (though you should understand how map and chain work for Either). The important thing is more about general modelling of your state with powerful types and treating everything as a value (including, in this case, failure)

@rjmk Thanks for the tips! Once again had a really fun morning playing around with this. Think it is all working although not completely sure about some of my implementation. Will line comment.


Inference speed summary across all models under the same context (batch size)

If this is your first time, please read our contributor guidelines: https://gitee.com/mindspore/mindspore/blob/master/CONTRIBUTING.md
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

duplicate of #380


How to connect to Qfiled from local QFiledCloud?

Hello!
I deployed QFieldCloud on my computer using the Readme file from the repository, using Ubuntu. I have successfully launched Docker, Minio opens on localhost.
However, when I launch the QGis plugin to connect to the cloud, I don't fully understand which username/password I need to use. Are they listed in the .env file? Or is it something else?
Is there a Swagger in the containers so that I can send requests for testing?
BR!

Find your answer in the documentation attached: https://docs.qfield.org/get-started/tutorials/get-started-qfc/?h=double#change-the-default-qfieldcloud-server-in-qfield-and-qfield-sync


Resize image before upload

Hi,
I need to resize an image if is oversized before upload
I don't know if it is possible to integrate a such trick to resume.js
Example code to resize image in browser using a canvas:
https://jsfiddle.net/codeandcloud/xor3L8db/

I'm working to something similar... Using the event 'catchAll' I noticed that the event 'chunkingStart' fires right before the 1st chunk is made. At that time the ResumableFile still has no chunks. I'm planning to pause its upload process, check the image dimension and copy its bitmap data into a canvas if it exceed.
After the resize is done I will try to remove the original ResumableFile making a new one with the resized image data... this is it, in short...


fix the visitor && observer pattern don't show main bugs

Thanks and apologies for the very slow response. I'll be catching up on updating the site this coming weekend, so I'll take a look then.

This is intentional. The samples are not intended to be a complete example, but just a demonstration of how the visitor and observer patterns might be implemented. Usage is out of scope. Thanks for the contribution nonetheless.


Revise to the meaningful Farsi texts

I know that the translations are just examples of how RTL layout works, but as a tiny improvement, I've just refined the machine translations to make these awesome toolkits even better!

Hi @alinooshabadi — thanks so much for this!
I may not be able to get your suggestions into this project, but I'd like to use your suggestions for the upcoming version of this project: OfficeDev/.
I'll keep this PR around just in case, just wanted to give an update. If you'd like to see your changes in this project, you also need to sign Microsoft's CLA using the comment above.


refactor(linter): split up fixer code into separate files

refactor(linter): split up fixer code into separate files
refactor: mark Fix as non_exhaustive

#4223
#4222  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


There is no images in 'Fate Reforged' 2015 last set

There are only low res images in last set: 'Fate Reforged', an example:
http://api.mtgdb.info/content/card_images/391819.jpeg
http://api.mtgdb.info/content/hi_res_card_images/391819.jpg
Same Resolution.

yes there is delay in the his-res images, as it takes awhile to find hi-res scans. Some one has to physical scan the cards when I find a source they will be uploaded. Thank! I will leave this open until they are uploaded

Ok! Thanks!
El 04/02/2015 00:35, "Fritz (Fredrick Seitz)"<EMAIL_ADDRESS>escribió:

yes there is delay in the his-res images, as it takes awhile to find
hi-res scans. Some one has to physical scan the cards when I find a source
they will be uploaded. Thank! I will leave this open until they are uploaded
—
Reply to this email directly or view it on GitHub
https://github.com/planeswalkers/api.mtgdb.info/issues/93#issuecomment-72760044
.


Implement User Info Endpoint

Hey @jgrandja,
this is a very rough draft to verify I'm in the right direction. I'd love to get your feedback to see how I can deliver this ASAP.
Thanks

@idosal Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@idosal Thank you for signing the Contributor License Agreement!

Apologies for the delay @idosal. I will review within the next few days.

@idosal I'm pushing this out to the 0.2.0 as we still have a few tasks to complete for 0.1.2.

Closing in favor of #441

Damn, I completely missed the review. Sorry about that. Let me know if there's any way I can contribute.

@idosal, sorry about that, it's a bummer. Thanks so much for getting it started, it was a great help! Feel free to review #441, since you have invested in this feature already. I'd love to get your feedback on it if you have any thoughts.

Thanks, I'm glad it helped. I'll review it ASAP.


Hot Keys for JW Library and Zoom

Would it be possible to allow the software to also work with the zoom second window and have an editable shortcut to select which screen is on top.
A few congregations are using macro keypads that I have set up however we need an easier way to switch what is being shown, the best current way other than using OBS is to close JW library.
So having a keyboard shortcut that allows to change which application is on top would really help.
Below is the current keys for the macro keypad

Nice solution with the Macro Keypads
I know it is not specifically an answer to your question, but it may be of help. You can turn the second display off and on in JWLib without closing it.  Just toggle the option in settings.

I like the idea of JwlMediaWin have some sought of hotkey control to select different windows.
We are also investigating AutoHotKey as an option https://www.autohotkey.com/
HTH
Colin

Thanks @ColinAAvey I will have to look into that

You may be able to adjust the fork I have done here @OishiiCha to get what you need? https://github.com/rawdenhoff/JwlMediaWin/tree/zoomSwitcher
Sorry for the crude code @AntonyCorbett
WCL
Rawden.

@rawdenhoff This sounds like you have found a solution. Would it be possible for you to provide an EXE installer so that I can try it out?

@rawdenhoff This sounds like you have found a solution. Would it be possible for you to provide an EXE installer so that I can try it out?

I don't have an installer, but you can extract the files from this zip file...
https://github.com/rawdenhoff/JwlMediaWin/tree/zoomSwitcher/ZoomSwitcher/Exe

You may be able to adjust the fork I have done here @OishiiCha to get what you need? https://github.com/rawdenhoff/JwlMediaWin/tree/zoomSwitcher

This is looking really good thanks!

@rawdenhoff I don't see much info in the readme for this fork. How does it accomplish the switching between the 2 programs? is it just hotkeys?

You might find my autohotkey project useful:
https://github.com/JamesSwift/AutoHotKey-Zoom-Wrangler

@rawdenhoff I don't see much info in the readme for this fork. How does it accomplish the switching between the 2 programs? is it just hotkeys?

Sorry @Spenser715, @JamesSwift 's reply just now reminded me that I forgot to reply to your post!
It is just a Window Forms application that uses @AntonyCorbett code to show or hide both the JWLmedia (secondary) window and the Zoom secondary window. We use it all the time to control which one is displayed - saves having to drag the windows across monitors. Although I'd imagine you've found a solution by now. 😬
Once you launch the app, the Zoom and JWL checkboxes will enable or disable depending on whether the window handles can be located (i.e. if it can find the apps running).

@rawdenhoff Is there an update of your Zoomswitcher with the latest JwlMediaWin release?

Hi @AnthonyG83
I haven't had a chance to test these properly yet, but I have just pushed a commit in line with @AntonyCorbett 's recent one.
I'll try and test it at maintenance this morning and if all is well, try it out tomorrow afternoon for the meeting. Feel free to download the exe though.


Start with a pre-synced chain

with $$$

I think we will need this (or something like it) for Tabconf, especially if transaction creation is apart of any of the attacks. Adding the tabconf milestone.

tx_flood will still spread out the coins over time

I don't think this is true with how tx_flood is currently written. It looks like tx flood assumes the node is itself a miner, generates some random addresses for the node itself and then sends to itself.

We can also do that manually

Also fine with a manual script for pre-generating coins just for Tabconf if a pre-synced chain will take too long to get working. In general, I'm trying to find two birds one stone opportunities where if something is a) generally good for warnet and b) solves a problem for Tabconf, I'd prefer that over a custom script only useful for Tabconf. But if pre-synced chains is not the right approach or will take too long, then I can open an issue for manually populating tanks for warnet.

It spreads out the coins output-wise, but you are correct it's a self-send.
IIRC correctly (my mind is hazy), I checked out the regtest code from a pre-mined chain and it's ... actually just the first node doing a quick pre-mine of it's own, then adding the privkeys to nodes 0-3. No actual, pre-mined chain (as it fixed datadir) is going on. Will try and find the code again in a bit to double-check

Here it is. I do like how they mine first blocks in the past to get out of IBD quicker (we should do that!)
https://github.com/bitcoin/bitcoin/blob/0533e652355a68a2c52fb1524a8033bbb5bfa837/test/functional/test_framework/test_framework.py#L823-L895

Nice! Seems like we could port the _initialize_chain code, where a single node (controlled by us, e.g., the signet miner) starts with a pre-initialised chain, distributes the coins between N nodes and spits out M private keys. Those private keys can then be given to each of the tanks. This way, tanks come online, do IBD up to 199 blocks, and already have private keys loaded into their wallets (passed as a config argument). This would be slightly manual in that we need to run the initialisation step first and then pass the private key arguments to each of the tanks, but feels like we could easily automate in the future.

I don't think this is true with how tx_flood is currently written. It looks like tx flood assumes the node is itself a miner, generates some random addresses for the node itself and then sends to itself.

It spreads out the coins output-wise, but you are correct it's a self-send.

No, the logic is:

create a big empty list of addresses
for each node, create wallet (just so happens to be called "miner") generate addresses and add them to the list
in one thread per node, in an endless loop, check the node wallet balance and if there is any money there, send a tx to random addresses from that big list

So I think we can close this?

SO this can start at any point and will only send txs from nodes that have a wallet balance. even if there is only one miner, the money will eventually be spread out to everyone

I see, thanks for the explanation. I didn't look at the code for ensure_miner and assumed it was doing something more sophisticated. Also wasn't apparent to me that new addresses keep filling up self.addrs from all nodes.
TBH, still not convinced this is a good idea for seeding the network since I don't have an intuition on how long this takes to actually ensure every node has a sufficient amount of coins, and also seems hard to guarantee everyone has a similar sized amount. I'd be worried about a scenario where Team A gets way more utxos than Team B , so then Team B needs a top up at some point while team A gets to keep attacking. I also think we should avoid "we'll just manually do X" as much as possible, because this will make actually running warnet on the day of a nightmare. I also think we should be prepared for a catastrophic failure where we need to restart the cluster. If the only way to get everyone UTXOS is us manually doing a bunch the day transactions or waiting for tx_flood to fairly distribute coins, a restart is not gonna be pleasant.
I'd recommend we pursue the _initialize_chain route and perhaps time box it to a few days. If it doesn't work, we can use the tx_flood method or write a manual script. Happy to take a crack at this myself.

What would be the difference between that and what we have?

What would be the difference between that and what we have?

https://github.com/bitcoin-dev-project/warnet/issues/466#issuecomment-2324802706

What would be the difference between that and what we have?

#466 (comment)

So currently with miner_std:

Start all nodes
Mine mature blocks on all of them
They both recieve coins and sync at the same time

In #466 (comment):

Mine first
Sync next
Hand out keys last

What's the material difference I'm missing?

Mine mature blocks on all of them

this doesn't work for Tabconf if we a) limit access to generate commands for players or b) have a working signet. The whole idea of a presynced chain is that the UTXO set can be deterministically handed out to players without requiring players to be miners.

Mine mature blocks on all of them

this doesn't work for Tabconf if we a) limit access to generate commands for players or b) have a working signet. The whole idea of a presynced chain is that the UTXO set can be deterministically handed out to players without requiring players to be miners.

My question was about how -- in the absence of a backported signet, or limiting generate access -- the premined chain solution differs from the current solution in miner_std.
Naturally if we have either of those solutions then this whole discussion is moot, and we still don't need a premined chain, as we can just distribute funds/blocks at will.

Right, what's missing from this issue is some pre-conversation in the warnet chat where we were discussing whether or not to just use tx_flood/mining_std for distributing coins for Tabconf. As I've laid out here https://github.com/bitcoin-dev-project/warnet/issues/466#issuecomment-2326913937, I don't think either of those are viable options:

mining_std requires everyone to be a miner
tx_flood would take an indeterminate amount of time to distribute coins and no guarantees around distributing coins evenly

I think we should build towards having a signet miner, and in that case yes, we can write a manual script just for Tabconf for manually distributing coins. Personally, I still see value in have a pre-mined chain with coins pre-allocated as this allows for better reproducibility. If your argument is "we don't need this for Tabconf," great, let's remove the milestone and leave the issue open. If your argument is "we don't need pre-mined chains for warnet at all," I disagree.

I'm going to run a 100-tank warnet in GKE today with one miner and tx_flood and monitor block count and wallet balance over time, that should give us at least a little something to go on.

After 5 minutes, all 100 tanks have at least 4 BTC, scenarios initiated with minimal options:
(.venv) --> warnet run scenarios/tx_flood.py
(.venv) --> warnet run scenarios/miner_std.py --interval 5 --mature

100 tank graph looks like this x100:
nodes:
- connect:
  - tank-0001
  - tank-0017
  - tank-0018
  - tank-0023
  - tank-0027
  - tank-0079
  - tank-0035
  - tank-0089
  image:
    tag: '27.0'
  metricsExport: true
  metrics: balance=getbalance() blocks=getblockcount()
  name: tank-0000

After 10 minutes, every tank has at least 23 BTC. WARNET IS AWESOME!

the image mine as well also have a bitcoin.conf with the challenge, and then it mine as well also be deployed to every tank instead of IBD, I'm not sure what the drawback of that would be, the image should cache on the first install anyway

started https://github.com/bitcoin-dev-project/warnet/pull/555 as a POC, was pleasantly surprised at how simple it was and how its almost entirely a change to the helm chart templates (i.e., no extra docker images to manage).
The tricky bit is where to store the files externally, but if we following the url structure of https://storage/<signetchallenge>/blocks.tar.gz, doesn't really matter where it lives.
In the PR, I also include the wallets dir, so a clever person could get the url, unzip and get the wallet file. But for this, we can just add a passphrase to the wallet and the passphrase can be in a secret config when starting the signet miner.
Would love to hear if anyone disagrees with this approach, otherwise I'll try to finish it up tomorrow with docs and an end to end example.

Closed by #555


Support for App Extension target

When targeting App Extension, UIApplication is not available, which causes compilation error.

Added.
You may add YY_TARGET_IS_EXTENSION flag in Build Settings - Preprocessor Macros sections.

See also: https://github.com/ibireme/YYWebImage/issues/13


Fix new linter errors from golangci-lint 1.44.2

Fixes gofumpt and revive errors that show up when we upgrade from
golangci-lint 1.43.0 to 1.44.2.
Signed-off-by: Daniel Farrell<EMAIL_ADDRESS>

There's actually an even newer version of gofumpt that flags quite a bit more. I can run it instead?
https://pastebin.com/VhiUnPp5
https://github.com/mvdan/gofumpt/releases/tag/v0.3.0

After thinking about it more, I think it makes sense to only make the changes the current/latest linter flags. We should keep the standards consistent with the automation, driven by it. We can make the changes gofumpt 0.3 flags when/if golangci-lint pulls it in.

This does fix the issues flagged on https://github.com/submariner-io/shipyard/pull/734 (https://github.com/submariner-io/shipyard/runs/5324947094?check_suite_focus=true) and there are not any omitted there, I checked locally with golangci-lint 1.44.2.


port class 'process' to Windows

Why arent you using _popen function, which is the equivalent for windows? I would assume we could just write something like this:
#ifdef WIN32
#define MIGRAPHX_PCLOSE _pclose
#define MIGRAPHX_POPEN _popen
#else
#define MIGRAPHX_PCLOSE pclose
#define MIGRAPHX_POPEN popen
#endif

template <class F>
int exec(const std::string& cmd, const char* type, F f)
{
    int ec = 0;
    if(enabled(MIGRAPHX_TRACE_CMD_EXECUTE{}))
        std::cout << cmd << std::endl;
    auto closer = [&](FILE* stream) {
        auto status = MIGRAPHX_PCLOSE(stream);
#ifdef WIN32
        ec = status < 0 : -1 : 0;
#else
        ec = WIFEXITED(status) ? WEXITSTATUS(status) : 0; // NOLINT
#endif
    };
    {
        // TODO: Use execve instead of popen
        std::unique_ptr<FILE, decltype(closer)> pipe(MIGRAPHX_POPEN(cmd.c_str(), type), closer); // NOLINT
        if(not pipe)
            MIGRAPHX_THROW("popen() failed: " + cmd);
        f(pipe.get());
    }
    return ec;
}

The MSDN documentation for _popen states:
If used in a Windows program, the _popen function returns an invalid file pointer that causes the program to stop responding indefinitely. _popen works properly in a console application. To create a Windows application that redirects input and output, see Creating a child process with redirected input and output in the Windows SDK.

More _popen creates a pipe and requires translation from POSIX to WIN32 and vice versa. That is a performance penalty we want to avoid, considering the UAI project requirements.

If used in a Windows program, the _popen function returns an invalid file pointer that causes the program to stop responding indefinitely. _popen works properly in a console application.

I see, it would be better to use tiny-process-library for this. The popen has limitations on POSIX side as well(ie no bidirectional pipes). This probably doesnt need to be switched in this PR, though.


Pretty progress

Add ability to print progress in a fun way.
This adds the ability to specify both code/data fractions and item specifiers
TODO:

[ ] Add support for RELs having their own

Looks great, thanks again!


Question of "save" function.

jquery-bracket is very useful to me. I appreciate it.
When I use "save" function , the buttons of +,-,se/de at the upper left side are created automatically.
I want to use "save" function without these buttons.
Please let me know the solution.
Thanks.

Hi, great to hear you've found use for it!
At the moment, there is no built-in way of disabling the toolbar if edit is enabled. You could use CSS to hide it though.
div.jQBracket .tools {
  display: none;
}

I'll close this issue as it sounds like a duplicate for #59.


Out of the box support for load balancing based on CPU usage

In the HTTP probe section, it is vaguely mentioned that we can host a custom script on the health probe endpoint which checks for the CPU usage, and returns a non 200 status to deregister that instance from the Load Balancer.
Are there any links to get more information about whether or not an our setup needs this? Do scale sets have a role to play in this? Any pointers to concrete implementations of this approach?

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

ID: 0e1e7cb2-c5a7-7122-c729-dc259db99a40
Version Independent ID: 45c6fd72-5114-fcda-0bab-2d80ac6bed33
Content: Use Azure Load Balancer health probes to scale and provide high availability for your service
Content Source: articles/load-balancer/load-balancer-custom-probe-overview.md
Service: load-balancer
GitHub Login: @KumudD
Microsoft Alias: kumud

@gaurav5430 , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@gaurav5430 , You have PowerShell Scripts to check the CPU utilization of your VM. So you can add a trigger, when it reaches threshold, probe page will return non 200.

That's great, where can i find an example of such Powershell scripts?
Also, I am interested in understanding if such an optimization would actually be required in real life scenarios, or would it be somehow handled by LB and scale sets?

I don't have a script handy, but this is something which can be easily written with the logic which I have mentioned above. As of now, those logic has to be managed from backend resource and LB doesn't handle those optimizations.
I will go ahead and close this issue.
Please reopen if required.

Yeah, sorry.
I wasn't expecting you to provide a script.
Mostly expecting pointers to documentation (if exists) for whether or not this kind of functionality is required in day to day usage.
Thanks anyway.


ENT rework

Most ENTs share the same login system. It would be great if we had one master function, then other functions would just derive from it, basically wrapping it.
Also the cookiejar_from_dict methods all over the place could probably be done differently.

Todo : Support Educonnect, and all the other ways to connect to ENTs since not everyone has an ATEN account.

from what i understand the ents share the same login system but the way to get to the educonnect page is different.

From what i understand the ENTs share the same login system but the way to get to the Educonnect page is different, is it possible to manage all ENTs with one function?

On the pronote application, after connecting for the first time, it manages to reconnect automatically at each connection even with an ent. Maybe there is a way to do the same thing.
I tried to figure out how it worked but couldn't.

@OiseauDesPlages Yes but we already do it we save the hash of the password and I think the Pronote app does the same thing


Remove buildSrc

Only one function in buildSrc was actually required in the build, and moving that into build.gradle.kts was simple. Removing buildSrc will speed up build configuration.

Really nice catch 💪💡

Thank you for contributing these PRs! Things are getting very clean.

Removing buildSrc will speed up build configuration.

Definitely, especially as it is a configuration in Kotlin... Thanks.


Merge pull request #698 from Aguafrommars/master

4.6.1

:x: Build TheIdServer 4.6.1.build.2120 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/53d15667c2 by @aguacongas)

:x: Build TheIdServer 4.6.1.build.2122 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/a5c8ea5ae0 by @semantic-release-bot)

:white_check_mark: Build TheIdServer 4.6.1.build.2123 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/a5c8ea5ae0 by @semantic-release-bot)


Ranked scoreboard (WIP)

The scoreboard in the monitor is now ranked by trader balance, but due to how it is updated, it flickers regularly, and I'm not sure how to fix it.

Hi Jens. I see the flickering! I think I would try to solve the problem by sorting the traders by balance in the view-function backend instead of doing it with JS front-end after the table has already been generated. In the view function 'trader_table' you could sort the traders and send the sorted traders as context to trader_table.html.
Perhaps the only change needed is something like this?
traders = Trader.objects.filter(market=market).order_by('-balance')

You're right, that does seem to be much smarter. Thanks!

I merged the branch.  traders = Trader.objects.filter(market=market).order_by('-balance') really did the trick. I added the same ordering of "traders" to the monitor view to get the traders in the correct ordering immediately on the monitor interface (otherwise we would have to wait ~1 second for the htmx request to call the back end and do the reordering).


Fix the issue 10448: Azure storage file data lake Sample code is wrong

Fix the issue https://github.com/Azure/azure-sdk-for-java/issues/10448

Thank you for submitting this PR. We really appreciate the help!


Disabling AGC, AWB and friends

Hello !
I'm trying to get the more stable picamera image parameters as possible, in order to do cools things like colorimetry, that require constant sensitivity to light and color.
In order to achieve this, I installed the last commits and set these settings to a fixed value :
Capture parameters :

Shutter speed : 11111

Automatic Gain control :

exposure_mode : off
analog_gain: 8.0
digital_gain: 1.2

Auto-White Balance  :

awb_mode : off

Exposure time, awb and acg gains are now proudly fixed, but it seems I still have some color adaption algorithm running somewhere:
1 ) My white background looks snow-white.
2 ) I present a finger in front of the camera.
3 ) The background goes a bit redish (transition take around 1 second)
Did I missed to fix some setting somewhere ?
Is it just possible to achieve my goal ?
Personal note : Thank you 6by9, rwb27 and Dave, you are my everyday heroes !

Setting "awb" and "agc" to "off" is simply not enough for your case. You also need to get rid of the color processing blocks in ISP. Here are the things you can do:
1 - Capturing RAW images: RAW image data come directly out of the sensor, without any processing applied. Since you are interested in the response of the sensor to the incoming light, this is the ideal solution for your case. But, keep in mind that you may need to do certain post-processing (like lens-shading correction) to compensate for the nonuniformities.
2 - Disabling ISP blocks: This is the tedious trial and error approach. Just disable the ISP blocks one by one, until you get the desired output.

Disabling ISP blocks is something I need to add (as an advanced feature) in 1.14; not sure if I made a ticket for this already so this'll do to remind me for now!

FYI I'm adding an extra controllable bit to MMAL_PARAMETER_CAMERA_ISP_BLOCK_OVERRIDE that allows you to disable demosaicing as well. Bit 11.
It makes little sense with a Bayer sensor (you get an odd checkerboard effect), but some people may have managed to obtain mono versions of these sensors.

Implemented in 949421b231e0a0396e8dff2a348824e5fa5006ee


Uncaught Error: Remote FTP: getConfigPath returned false, but expected a string

[Enter steps to reproduce:]

When Atom starts, all is functionally. Then:
Open new window
Connect to remote FTP

Edit: Remote FTP works only with previous Project only, unable to connect when new created
Atom: 1.18.0 x64
Electron: 1.3.15
OS: Microsoft Windows 10 Pro
Thrown From: Remote-FTP package 1.0.0
Stack Trace
Uncaught Error: Remote FTP: getConfigPath returned false, but expected a string
At C:\Users\Andrew\.atom\packages\Remote-FTP\lib\client.js:122

Error: Remote FTP: getConfigPath returned false, but expected a string
    at Client.readConfig (/packages/remote-ftp/lib/client.js:122:47)
    at /packages/remote-ftp/lib/views/tree-view.js:212:30)
    at HTMLDivElement.dispatch (/packages/project-manager/node_modules/jquery/dist/jquery.js:4435:9)
    at HTMLDivElement.elemData.handle (/packages/project-manager/node_modules/jquery/dist/jquery.js:4121:28)

Commands
     -1:03.9.0 remote-ftp:toggle (input.hidden-input)

Non-Core Packages
atom-beautify 0.30.3
atom-bootstrap3 1.2.12
autoclose-html 0.23.0
laravel 0.7.1
project-manager 3.3.5
Remote-FTP 1.0.0

Related #621


Android Studio 3 Compatibility?

This is more of a question but, have you tested the aop-common.zip and aspectj.zip plugins from https://github.com/Archinamon/GradleAspectJ-Android/tree/master/IDE in Android Studio 3?
https://android-developers.googleblog.com/2017/10/android-studio-30.html
https://developer.android.com/studio/preview/features/index.html
https://developer.android.com/studio/index.html
I'm hesitant to upgrade if it means losing AspectJ on Android for a time.  Hopefully they didn't change how plugins work.  I'm on a Mac OS Sierra 10.12.6 (16G29).
Thanks!

I tried it on my friend's Android Studio 3 install and it looks like the GradleAspectJ-Android library has a dependency on Kotlin 1.1.51 maybe which is built into Android Studio 3 now.  He tried disabling the builtin plugin and adding the Kotlin library manually but couldn't get it to work.  I think that's the only part that didn't work, sorry this is secondhand info.

Hi! Sad but these plugins are not working properly within AS 3.0 no more. I did looking for updating them, but not sure I can do it without official sources. These plugins just a c/p that did work on parole.
To say about the second part of your question. I'll look into it in a few days, but that sounds strange. Builtin kotlin has nothing chained with a kotlin's stdlib that is using to compile the plugin's sources. Does any additional info or logs available?

Ok I bit the bullet and upgraded to Android Studio 3 for a client project.  If I enable the plugins:
AOP/@AspectJ Common
AspectJ Support

Android Studio opens without crashing.  But if I try to open my .aj file it just shows an error message so I had to disable the plugins.  Without them, we mainly lose syntax highlighting and the ability to step through AspectJ files in the debugger.
GradleAspectJ-Android still works however, and I'm able to see my trace at https://github.com/Archinamon/GradleAspectJ-Android/issues/59 being written to the console :-)
I had to comment out the lines for retrolambda but I'm not sure it was needed anyway.  From what I understand, it helps AspectJ detect anonymous functions, but I could be wrong about that?  I see anonymous class methods like this in the trace:
new Handler().postDelayed(new Runnable() {
	@Override
	public void run() {
		// do stuff here
	}
}, 1000);

They appear as MyActivity.1.1.run().
I've updated my tracer example at https://github.com/Archinamon/GradleAspectJ-Android/issues/59#issuecomment-331757834 to work with Android Studio 3.  Hope this helps someone.

You're right considering the retrolambda. It will generate (desugar, just like gradle 3.0.0 does) synthetics for every lambda expression. So we can easily find and pointcut them! :)
Talking about syntax plugin. That sad, but I have no idea how to support it w/o writing my own implementation of the plugin. The corner circumstances is this plugin is not official port for AS, just my own "adaption". It will work fine in IntelliJ Ultimate (trial too). My mind AS has some incompatible changes with base IntelliJ, that's why these plugins fails to run.


Add API for geometry cleanup operations

Identify the key model cleanup tools users require and expose them as design/component/body operations as appropriate.
Server-side support required before this can be implemented.

We'll link this to #648 for now. However, it is "half-way" implemented. We need to actually pass in Design, Component, Body objects... and not just strings that represent their IDs. Good thing is that this is an "independent" submodule of the package


Feat/connect to versioned search api - LESQ-581

Description
Music year: 2013

Removes construct elastic query (We now construct in the search api)
Adds version to endpoint in browser config

How to test

Go to https://deploy-preview-2223--oak-web-application.netlify.thenational.academy
search should work the same as before

Acceptance Criteria:

[x]  Release search api version to staging
[x]  Delete construct query from OWA and just send query object
[x]  Update staging config with search endpoint with version number (2)
[x]  Test search is working
[x]  Release search api version to production
[x]  Update production config with search endpoint with version number (2)
[x]  Test search is working

:tada: This PR is included in version 1.338.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


WFLY-12801 Upgrade JGroups to 4.1.8.Final

https://issues.jboss.org/browse/WFLY-12801

Looks like there is a regression with multicast binding on Windows (probably due to https://issues.jboss.org/browse/JGRP-2391).  Adding hold label.

This appears to be an issue with the routing table on the windows CI machines.  Keeping hold label until resolved.


wip new jedi tryPylance banner

Todo:

[ ] Copy previous tryPlance banner test cases

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #14697 (ab49cfb) into main (a678fbe) will decrease coverage by 0.14%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main   #14697      +/-   ##
==========================================
- Coverage   64.90%   64.76%   -0.15%
==========================================
  Files         541      548       +7
  Lines       25144    25683     +539
  Branches     3545     3632      +87
==========================================
+ Hits        16320    16633     +313
- Misses       8162     8364     +202
- Partials      662      686      +24

Impacted Files
Coverage Δ

...anguageServices/proposeLanguageServerBannerJedi.ts
24.56% <24.56%> (ø)

...rc/client/activation/jedi/multiplexingActivator.ts
17.64% <37.50%> (-0.10%)
:arrow_down:

src/client/activation/serviceRegistry.ts
76.74% <100.00%> (+0.55%)
:arrow_up:

src/client/common/experiments/groups.ts
100.00% <100.00%> (ø)

src/client/common/types.ts
100.00% <100.00%> (ø)

src/client/pythonEnvironments/base/locatorUtils.ts
60.46% <0.00%> (-37.58%)
:arrow_down:

src/client/common/utils/platform.ts
53.33% <0.00%> (-18.67%)
:arrow_down:

src/client/pythonEnvironments/base/locators.ts
55.26% <0.00%> (-18.08%)
:arrow_down:

...rc/client/pythonEnvironments/common/commonUtils.ts
85.93% <0.00%> (-14.07%)
:arrow_down:

src/client/pythonEnvironments/base/envsCache.ts
72.54% <0.00%> (-12.64%)
:arrow_down:

... and 40 more

Continue to review full report at Codecov.

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

Closing in favor of #14759.


NuGet Version out of date

The version of the dll currently on NuGet contains bugs fixed in later development versions, can you push a new release to NuGet?

Please see https://github.com/mausch/SolrNet/issues/114