The xgboost.DMatrix function can not be silent by setting "silent=True"

For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System: linux
Compiler: gcc
Package used (python/R/jvm/C++):
xgboost version used: latest
If you are using python package, please provide

The python version and distribution
python 2.7

Steps to reproduce
I trained data by python api. However, I didnt want the function xgboost.DMatrix output anything.
What have you tried?

dtest2 = xgb.DMatrix(sys.argv[1],silent=True)
I set silent to be True.But there is info "[23:28:44] 1441x10 matrix with 11528 entries loaded from file_name".
How can I close the output info.

I think it's related to this: #1216

@ungil  Yes! We are meeting the same problem.


How to get a DataNode On a CallContract

Thanks for your wonderful work
It's realy helpful on working with unity.
I want to get a datanode object on calling a method from contract.
but now I got a transaction from which nothing usedfull data is included.
Any advice is appreciated.

Sadly at least for now when calling a contract the only thing you obtain back is a transaction. I recommend writing to the storage and then read the storage back with api.GetStorage

Got it, thanks so mush!


Privileged Access Group - Programatic membership removal - Graph/Powershell

Hi there,
I am trying to develop a way to programatically remove users (members) from Privileged Access Groups. I am currently looking at using graph modules within powershell. I can pull transistive memberships etc. but have not found a way to remove those memberships etc. Is there an idea on when powershell/graph support for this feature will be available?
Cheers

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

ID: f4da66b2-36f2-753f-4170-bcbf51824033
Version Independent ID: c9e72e4f-effc-37f0-f418-c92af00f0523
Content: Assign eligible owners and members for privileged access groups - Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/privileged-identity-management/groups-assign-member-owner.md
Service: active-directory
Sub-service: pim
GitHub Login: @curtand
Microsoft Alias: curtand

@Cagey-One Thanks for your feedback. We will investigate and update the thread.

@Cagey-One Unfortunately, currently there is no API support for PIM alerts and privileged access groups. Since this feature is currently in preview, PowerShell or GraphAPI for Privileged access groups is not supported.
If you'd like the ability to be able to create PIM enabled Azure AD groups and edit their settings using the Graph API, I'd recommend leveraging our User Voice forum and creating a feature request, so our engineering team can look into implementing this.

@Cagey-One Unfortunately, currently there is no API support for PIM alerts and privileged access groups. Since this feature is currently in preview, PowerShell or GraphAPI for Privileged access groups is not supported. If you'd like the ability to be able to create PIM enabled Azure AD groups and edit their settings using the Graph API, I'd recommend leveraging our User Voice forum and creating a feature request, so our engineering team can look into implementing this.

Thanks for the information. I understand this is in Preview but I believe the support tools should be pretty high on the priority list for a feature to have appropriate support tools available.
I have commented on https://feedback.azure.com/d365community/idea/643cf69b-e087-ec11-a81b-0022484bfd94 which was raised 5 months ago and and has now just received a comment from Microsoft (which was blank...).
If there is anyway that the feature could be given a push from within that would be great.


Add get_mapping_allclose

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (12da90e) 100.00% compared to head (9b099cc) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #198   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           20        20
  Lines          961       988   +27
  Branches       192       197    +5
=========================================
+ Hits           961       988   +27

Impacted Files
Coverage Δ

src/coola/comparators/__init__.py
100.00% <100.00%> (ø)

src/coola/comparators/equality.py
100.00% <100.00%> (ø)

src/coola/comparators/numpy_.py
100.00% <100.00%> (ø)

src/coola/comparators/pandas_.py
100.00% <100.00%> (ø)

src/coola/comparators/polars_.py
100.00% <100.00%> (ø)

src/coola/comparators/torch_.py
100.00% <100.00%> (ø)

src/coola/comparators/utils.py
100.00% <100.00%> (ø)

src/coola/comparators/xarray_.py
100.00% <100.00%> (ø)

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


latte-dock: init at 0.5.98

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

[x] NixOS
[ ] macOS
[ ] Linux

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

fixed.


DisplayObjectContainer click method not working

The mouse (click) events don't seem to fire on DisplayObjectContainer for me. Am I missing something? I even included a hitArea just in case.
    var DOC = new PIXI.DisplayObjectContainer();
    DOC.width = 10;
    DOC.height = 10;
    DOC.interactive = true;
    DOC.buttonMode = true;
    DOC.hitArea = new PIXI.Rectangle(0, 0, 10, 10);
    DOC.click = function(e) {
      console.log('Clicked');
    };
    stage.addChild(DOC);
    requestAnimationFrame(animate);

Is the stage marked as interactive as well?

ran into this issue as well, but there is a trick to make it work:
if you fill the DisplayObjectContainer with an invisible Rectangle Graphic object, your click handler will work:
this.backgroundLayer = new PIXI.DisplayObjectContainer();
this.background = new PIXI.Graphics();
this.background.beginFill(0x000000, 0.0);
this.background.drawRect(0, 0, this.stageSize.x, this.stageSize.y);
this.background.interactive = true;

....

this.background.click = function (e) {
  // do something with the event
}.bind(this);

I don't have any problem running your code @Arduinology here's a fiddle showing it: http://jsfiddle.net/nikkikoole/ngma7snq/18/

Closing thus since interactions are being rewritten and I can't reproduce this anyway.


Error:  Dark Mode toggle button is not working

Describe the Error
The dark mode toggling button is not working
When i am clicking on it the the mode is not toggling
Expected behavior
The dark mode must change to light mode
https://github.com/user-attachments/assets/ed762526-7733-43f9-9440-ca49fa8bf532

Its working for me.


Add DB Browser for SQLite

Add of DB Browser for SQLite, a tool to parse sqlite3 databases.
Code : https://github.com/sqlitebrowser/sqlitebrowser

DB Browser for SQLite was already present as sqlitebrowser, merging in last commit.


run in order

-c config.yaml

set the patterns, by glob syntax
order:
    - case*.yaml
    - stage2*.yaml
    - stagex.yaml

then the run order

patterns hit the config
the rest of non-hit

[ ] run in order (glob? the pattern is based on the binary root dir)


Defining a Parent in the design does not create parent parameters for swagger specs

Hi,
I think there is an issue with the swagger.json generated through design.
Let's say you have something like:
  var _ = Resource("parent", func() {
    //...
    BasePath("parents")
    Action("show", func() {
		Routing(GET("/:parentID"))
		Params(func() {
			Param("parentID", String, "Parent ID")
		})
                //...
    })
    //...
  }

  //...

  var _ = Resource("child", func() {
    //...
    BasePath("children")
    Parent("parent")
    Action("show", func() {
		Routing(GET("/:childID"))
		Params(func() {
			Param("childID", String, "Child ID")
		})
                //...
    })
    //...
  }

Then, you have following endpoints open:
GET /parents/{ParentID}
GET /parents/{ParentID}/children/{ChildrenID}

If you use the auto-generated swagger.json with the official swagger-UI, for the case of GET parent, everything works fine.
But for the case of GET children, the UI does not offer the parameter parentID. However it does, correctly, the parameter childID.
Running the action, takes you to a failure, since you cannot supply the parentID, any how.
Therefore, my assumption is that, in design, when you state Parent('parent') in design and auto-generate the code, the parent parameters are not checked to properly generate the swagger.json file, for the children case.
Not sure if the explanation is clear.
Any tip on this?
Thanks in advance!

My bad. This is already fixed. Closing ticket


Add suport to flush via API

The AWS Lambda Plugin as the requirement to flush data at the time the invocation of the function is done. See https://github.com/open-telemetry/opentelemetry-js-contrib/blob/b128dae70cf5723162ef72ed0ed83924f3edf4dc/plugins/node/opentelemetry-instrumentation-aws-lambda/src/aws-lambda.ts#L199-L206
As once can see this creates a binding from instrumentation to a specific SDK. Usually an instrumentation should have no need to require SDK components.
Currently API offers no way to get the TracerProvider used (it returns a ProxyTracerProvider) nor does it offer a way to force flush the trace data.
Should we enhance the API with some functionality regarding this?

This is a recurring spec question. AFAIK it has been shot down every time. Maybe a member of the @open-telemetry/technical-committee can weigh in?

@dyladan  I think it is a reasonable request to have a Flush API. What were the arguments against? Perhaps time to submit a proposal to the spec? That will be a backward-compatible addition to the API. Exporters are already supposed to have a Flush API, so it is a matter exposing it in the user-facing API.

I am not clear how this is not addressed by the ForceFlush that is already in the spec? https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk.md#forceflush

@yurishkuro It's specified as part of the SDK but an instrumentation should usually have no need to access the SDK, they should only need the API.
As of now having the requirement to access the SDK even results in a direct binding to a specific SDK implementation. So one of the OTel goals to allow custom SDKs is also broken.
I see following possibilities to solve this (thinking only of JS here as we are in JS repo) on API level (but maybe there are more)

exposing TracerProvider#forcedFlush on API
adapt API that api.trace.getTracerProvider() returns the concrete TracerProvider instance installed instead a ProxyTracerProvider. Users can then do a instanceof or typeof(provider.getSpanProcessor) === 'function'check (would always fail on ProxyTracerProvider.

(1) is quite generic but will most likely result in followups to add more and more APIs which is not my intention here
(2) sounds quite reasonable as I would in general expect that api.trace.getTracerProvider() returns what I passed to api.tracer.setGlobalTracerProvider() (assuming success here)
Another option is clearly to ignore this and let an instrumentation use SDK APIs as needed and just document hat they are not fully compatible with 3rd party SDKs.

How is the SDK installed in the first place? The default impl in the API is no-op, so there needs to be some code that is aware of the SDK, and it's usually that code that's responsible for flushing.

The instrumentation code currently only accesses the API directly and does not use any sdk-specific methods.

Javascript is dynamically typed and there is nothing like a build step to link together to an app. It's quite easy to create an app having an SDK and some instrumentations as direct dependency and one of these instrumentions may decide to add parts of the default SDK as transitive dependency (maybe the same as the direct dependency, maybe a different one, maybe just version differs).
If API offers all needed functionality for instrumentations the API needs to match. This is by far easier/stable done.

Is there any movement on this topic? I have ran into this too as I'm loosing spans if using the batchprocessor.
The stop-gap solution for me has been this snippet, but it is far from ideal:
  const traceProvider = api.trace.getTracerProvider();
  if (traceProvider instanceof NodeTracerProvider) {
    await traceProvider.shutdown();
  } else if (traceProvider instanceof ProxyTracerProvider) {
    const delegateProvider = traceProvider.getDelegate()
    if (delegateProvider instanceof NodeTracerProvider) {
      await delegateProvider.shutdown()
    }
  }


Request to add left-shift operator

The MACN11 cross-assembler on the PDP-10 implemented the underscore character as a left-shift binary operator.  This PR is a request to accept my implementation of it in macro11.
The operation is a left shift of the left side value by the number of bit positions indicated by the right side value.  As for the arithmetic operators, both values must be literal (numeric constants or symbols equated to a numeric constant).
If the command line option yus to allow the underscore character in symbols is selected then this new interpretation of underscore is disabled.
I have added a few test cases to test_prec.mac and test_prec.lst.ok.
Sorry for filing a PR right after your Release 0.6 announcement.  I actually implemented this change a few years ago, but then I saw the "MOVED TO GITLAB" statement so I didn't follow through.  Then today I received email about the release since I am watching your fork here on github, so it looks like you are maintaining both -- Thanks!
Steve Casner

Hi Steve! Thanks for your pull request. I'm indeed mainly maintaining this on gitlab but I keep a copy here, for possible addition to / updating in the simh tools at some point.
It looks like this addition is compatible with my goal to accept all sources that Macro11 05.05 would accept. And a shift operator is certainly useful. I guess the underscore  _ was used because that character used to be a left arrow ← character.
So I will merge this, and I can actually think of some further changes which generalize the idea a bit more.

Right shift. If there is no obvious character to use as the operator (which seems likely), a negative shift value could be a right shift.
I suppose the limitation to constant values is there because even the "complex relocations" of the linker don't contain a shift operator. But since a shift is equivalent to the appropriate multiplication (or division), such an operator could be used instead. I think that would allow arbitrary labels to be used on either side of the operator.
maybe even the _ operator does not have to be 100% disallowed when the -yus option is used. Since it is a binary operator it can only appear in between expressions, not at the start. .word 1 _ 2 3 could still be acceptable and not mean the same thing as .word 1,_,2,3 (4 3 resp. 1 <value of _ label> 2 3. Maybe some extra careful parsing checks need to be added to avoid confusion here and there.
Some optimizations: x _ 0 is the same as x, x _ 16 is 0.

Can I invite you to propose some additional changes along such lines (in a new merge request)? I can also do it myself if you don't want to.

Correction: for the complex relocations, the right hand side of course still needs to be constant, otherwise the multiplication factor cannot be determined.

Thanks for merging my changes.  You are right that when the left shift operator was initially defined the ASCII character graphic was a left arrow rather than underscore.  Also the MACN11 implementation did interpret a negative shift value as a right shift; the files I was wanting to assemble didn't include any examples of that so I overlooked it.  Looking again at the MACN11.DOC file I got from SNAILDART:
 Logical shift operator ("←")

      An additional binary operator is  recognized  in  expression
      evaluation.   "a←b"  is evaluated as "the value of a shifted
      left b bits".  If b is negative, the term is shifted  right.
      Both  a  and  b may be any valid term, including a bracketed
      expression.

      A common use for  this  operator  is  in  defining  symbolic
      values  for  bit masks.  For example, "INTENA = 1←6" defines
      an interrupt enable mask as a 1 in bit 6.

(That document was written in the 1970s but as I see it now it contains UTF-8 LEFTWARDS ARROW characters; there must have been some automatic character translation done somewhere.)
I have implemented your suggestions except I think trying to mix underscores in symbols with the shift operator would likely be more confusing than helpful.  I'll submit another PR.

Thanks!
We'll see if people complain about the lack of both extensions at the same time.


Leetcode company frequency asked Question

List of Questions
The questions in the pdfs inside folder are sorted by frequency of appearance. For locked premium questions, please search the problems in Leetcode website which are available for free.
The PDFs have leetcode companies tagged. Those problems are good practice to be familar with company's mostly asked problems.
Leetcode Company Tag
can I upload this and work for this issue in GSSOC

Please assign me this issue so that I can resolve it under GSSOC-23

Please assign me this issue so that I can resolve it under GSSOC-23

You can do this issue create me so I will work for it
You create own issue like top rated probom with solution then you work your issue

List of Questions
The questions in the pdfs inside folder are sorted by frequency of appearance. For locked premium questions, please search the problems in Leetcode website which are available for free.
The PDFs have leetcode companies tagged. Those problems are good practice to be familar with company's mostly asked problems.
Leetcode Company Tag
can I upload this and work for this issue in GSSOC

Please reply back I will start work for this issue


Run image build using kaniko credential

What this PR does / why we need it:

gsutil command within the pyfunc and batch predictor docker image build failed with GCS access error when running on workload identity node pool even though kaniko pod has been given admin privilege to GCS. Apparently, the gsutil credential is not properly initialize and run without using the kaniko's credential.  This PR properly initialize gsutil credential to use the one provided to the kaniko pod. (similar to https://github.com/caraml-dev/turing/pull/291)
Does this PR introduce a user-facing change?:

None

Checklist

[ ] Added unit test, integration, and/or e2e tests
[ ] Tested locally

LGTM


⚠ Bofrost.at has degraded performance

In f9b334d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2398 ms

Resolved: Bofrost.at performance has improved in 9843a14.


How to set up only one request to use the proxy?

I found
rp, err := proxy.RoundRobinProxySwitcher("socks5://<IP_ADDRESS>:1337", "socks5://<IP_ADDRESS>:1338")
	if err != nil {
		log.Fatal(err)
	}
c.SetProxyFunc(rp)

but if i have ten urls need request,  this way is all request to use proxy,  i only want one request to use the proxy, What should I do?

Ohi,
Proxying only one request can be achieved multiple ways.
You can use two collectors, one with proxy and another without proxy:
c := colly.NewCollector()
c2 := colly.NewCollector()
c2.SetProxy("http://<IP_ADDRESS>:8080/")
// ...

or you can unset the proxy after the first request:
c := colly.NewCollector()
c.SetProxy("http://<IP_ADDRESS>:8080")
c.Visit("http://example.com/")
c.SetProxyFunc(nil)
// ...

Ok i try it, thanks


Add gn rules to build iOS tests

Currently many iOS tests in the engine are run via Python scripts. We should support building these to targets to an output that is executable on the host machine and that does whatever test/simulator setup is required, then co-ordinates the running of the test, logging output, and reporting pass/fail status. Rule output could be as simple as a shell script.
Ideally these should resemble as much as possible, in terms of target attributes and configuration, the equivalent macOS and C++ rules.
Once supported, these rules can then be wired into et so that tests can be queried and launched from the tool.
See https://github.com/flutter/flutter/issues/147013 for an example of how this was done for Dart.

many iOS tests in the engine are run via Python scripts.

And bash 😞 ...  https://github.com/flutter/flutter/issues/143953


Fix typo in clear() method of Node

Thanks 👍

Thanks for merging. Do you mind cutting a new NPM release with this patch?

Published. Sorry for my late reply. I'm working now with KaTeX which is much faster than MathJax. For example I have a lot of problem with unexpected errors in MathJax if you try to update equation based on inputs. With KaTeX I don't have any problems. It just works and it hase much simpler API for use with React.
P.S.: I'm using custom KaTeX component which is based on react-katex.

Thanks!
I'm not too familiar with KaTeX. I'll look into it. I write math in LaTeX myself so that's not a problem.


Find related services near arbitrary location

Open the app on some topic, click find relates services, observe that the list of results is sorted by distance from the device location. Desired is to be able to enter the name of a location such as "Surrey" in order to get results sorted by the distance to the named location, as consistent as possible with the results from the Algolia search.
Acceptance Criteria

[ ]  There is a field on the service listing page that allows the user to enter a location
[ ] If the location is specified, the app hits geocoder to get the lat/long for the location
[ ] If the field is empty, the device location is used [to be confirmed, see mocks when they're ready]
[ ] The location from geocoder or from the device is submitted in the request to the server

Note that we are in the process of doing a fair amount of refactoring around the search. This will affect the code for calling the geocoder endpoint. The plan is to merge your PR first. So use existing code to access geocoder, but don't refactor it or make a lot of changes outside the component.

Moving @cherrielam 's ticket for this here:
We would like to add the location search function to the topic-services page. This is because: As a user, I would like to adjust the service results on a topic in order to search in the location that is of interest to me. This is particularly important for users who want to use AA pre-arrival.
Mock is here
Acceptance Criteria

[ ]  Topic>Services List has location search function as reflected in mocks, and service list updates according to the input location.

Steps for QA:

QA: Open the service search for a topic, observe many services are listed, focus on search input field, enter "Kelowna", hit Search, observe fewer services are listed, focus on search input field, clear the field by hitting X, hit search, expected is that many services are listed, observe that the listing still shows results for Kelowna.

Thanks, the fix has been pushed. I've noticed another potential issue. Because this component only triggers the loading screen when we actually start searching for services there is a delay in displaying it while the location is being fetched. There are two potential fixes:
A) Easy approach: leave the search expanded, which displays the buttons as inactive while the location lookup completes, collapse it after to indicate we've found the location
B) Longer/Refactor approach: Move the isFetchingLatLng state up into the parent component so we can display the loading screen when child component performs the lookup
Let me know your thoughts, thx!

@bensinjin I see what you mean now, I agree that this is an issue because it gives the user the impression for a split second that the search has run/not run/nothing happened
My preference is B... we should give the users the sense that something is happening (loading) sooner, rather than A which might give the user the feeling that the search has frozen or something while the lookup is happening? But either sounds like an improvement from currently. @rasmus-storjohann-PG may be a better judge of which one is better worth the time vs impact

Thx @cherrielam! We've decided to leave this for now and circle back if it becomes an issue later as more pressing work needs to be addressed.

Tested on Staging V1.25.0, server V1.3.24 on iOS 7s in Burnaby:

Selected topic "Applying for a driver's licence" > clicked "Find related services near me" > services in Burnaby showed up at the top > entered "Surrey" in the location field > services in Surrey then appeared on the  top of the list.
Selected topic "Register for English classes " > clicked "Find related services near me" > services in Burnaby show up on top of the list" > entered "Victoria" in the location field > only services in Victoria showed up.

Acceptance Criteria:

[X] Topic>Services List has location search function as reflected in mocks, and service list updates according to the input location.


flutter analyze is confused about running pub get

Trying out the instructions from here:
https://flutter.io/setup/
I get analysis errors until I run flutter upgrade, which doesn't seem to be mentioned anywhere.
Transcript as follows:
Last login: Mon Oct 31 16:56:31 on ttys004
leafp-macbookpro:~ leafp$ git clone https://github.com/flutter/flutter.git
Cloning into 'flutter'...
remote: Counting objects: 67773, done.
remote: Compressing objects: 100% (33/33), done.
remote: Total 67773 (delta 8), reused 0 (delta 0), pack-reused 67740
Receiving objects: 100% (67773/67773), 20.64 MiB | 1.44 MiB/s, done.
Resolving deltas: 100% (48658/48658), done.
leafp-macbookpro:~ leafp$ export PATH=`pwd`/flutter/bin:$PATH
leafp-macbookpro:~ leafp$ flutter doctor
Downloading Dart SDK 1.21.0...
######################################################################## 100.0%
Building flutter tool...
Downloading Material fonts...                       1616ms
Downloading package sky_engine...                   945ms
Running 'flutter packages get' in sky_engine...     353ms
Building Dart SDK summary...                        1510ms
Downloading engine artifacts android-arm...         7802ms
Downloading engine artifacts android-arm-profile... 2401ms
Downloading engine artifacts android-arm-release... 2302ms
Downloading engine artifacts android-x64...         3505ms
Downloading engine artifacts android-x86...         8819ms
Downloading engine artifacts ios...                 13667ms
Downloading engine artifacts ios-profile...         12624ms
Downloading engine artifacts ios-release...         9491ms
Downloading darwin-x64 tools...                     4004ms
Downloading android-arm-profile/darwin-x64 tools... 3475ms
Downloading android-arm-release/darwin-x64 tools... 1073ms
[✓] Flutter (on Mac OS, channel master)
    • Flutter at /Users/leafp/flutter
    • Framework revision 977a25f29c (37 minutes ago), 2016-12-15 13:54:53
    • Engine revision 2f32e8054d
    • Tools Dart version 1.21.0

[x] Android toolchain - develop for Android devices
    x Android Studio / Android SDK not found. Download from https://developer.android.com/sdk/
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[-] iOS toolchain - develop for iOS devices (Xcode 8.1)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.1, Build version 8B62
    x Brew not installed; use this to install tools for iOS device development.
      Download brew at http://brew.sh/.

[✓] IntelliJ IDEA Ultimate Edition (version 2016.3.1)
    • Dart plugin version 163.9166.22
    • Flutter plugin version 0.1.6

[✓] Connected devices
    • None
leafp-macbookpro:~ leafp$ flutter doctor
[✓] Flutter (on Mac OS, channel master)
    • Flutter at /Users/leafp/flutter
    • Framework revision 977a25f29c (37 minutes ago), 2016-12-15 13:54:53
    • Engine revision 2f32e8054d
    • Tools Dart version 1.21.0

[x] Android toolchain - develop for Android devices
    x Android Studio / Android SDK not found. Download from https://developer.android.com/sdk/
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[-] iOS toolchain - develop for iOS devices (Xcode 8.1)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.1, Build version 8B62
    x Brew not installed; use this to install tools for iOS device development.
      Download brew at http://brew.sh/.

[✓] IntelliJ IDEA Ultimate Edition (version 2016.3.1)
    • Dart plugin version 163.9166.22
    • Flutter plugin version 0.1.6

[✓] Connected devices
    • None
leafp-macbookpro:~ leafp$ flutter analyze flutter/packages/flutter/test/animation/animation_controller_test.dart
Analyzing 846 files...
[warning] Undefined name 'FontWeight'. (flutter/dev/benchmarks/complex_layout/lib/main.dart, line 431, col 62)
[warning] Undefined name 'FontWeight'. (flutter/dev/benchmarks/complex_layout/lib/main.dart, line 350, col 83)
[warning] Undefined name 'FontWeight'. (flutter/dev/benchmarks/complex_layout/lib/main.dart, line 352, col 82)
[error] Undefined class 'Point'. (flutter/dev/benchmarks/microbenchmarks/lib/gestures/data/velocity_tracker_data.dart, line 927, col 21)

/// Many more errors elided

leafp-macbookpro:~ leafp$ flutter upgrade
Upgrading Flutter from /Users/leafp/flutter...
Already up-to-date.
Running 'flutter packages get' in flutter...        5298ms
Building unlinked bundles for flutter...            1966ms
Running 'flutter packages get' in flutter_driver... 5112ms
Building unlinked bundles for flutter_driver...      84ms
Running 'flutter packages get' in flutter_markdown... 4717ms
Building unlinked bundles for flutter_markdown...    48ms
Running 'flutter packages get' in flutter_test...   4477ms
Building unlinked bundles for flutter_test...        89ms
Building unlinked bundles for flutter_tools...      305ms
Building unlinked bundles for sky_engine...          25ms

Upgrading engine...
Already up-to-date.

Flutter • channel master • https://github.com/flutter/flutter.git
Framework • revision 977a25f29c (38 minutes ago) • 2016-12-15 13:54:53
Engine • revision 2f32e8054d
Tools • Dart 1.21.0

Running flutter doctor...
[✓] Flutter (on Mac OS, channel master)
    • Flutter at /Users/leafp/flutter
    • Framework revision 977a25f29c (38 minutes ago), 2016-12-15 13:54:53
    • Engine revision 2f32e8054d
    • Tools Dart version 1.21.0

[x] Android toolchain - develop for Android devices
    x Android Studio / Android SDK not found. Download from https://developer.android.com/sdk/
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[-] iOS toolchain - develop for iOS devices (Xcode 8.1)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.1, Build version 8B62
    x Brew not installed; use this to install tools for iOS device development.
      Download brew at http://brew.sh/.

[✓] IntelliJ IDEA Ultimate Edition (version 2016.3.1)
    • Dart plugin version 163.9166.22
    • Flutter plugin version 0.1.6

[✓] Connected devices
    • None
leafp-macbookpro:~ leafp$ flutter analyze flutter/packages/flutter/test/animation/animation_controller_test.dart
Analyzing 846 files...
No analyzer warnings! (ran in 23.6s; 56 public members lack documentation)
leafp-macbookpro:~ leafp$

The problem is that pub get doesn't get run in the flutter repo by default (since for most people it's a waste of time).
What I don't understand is what the flutter tool thinks it's analyzing in your case. If you'd run it from inside the package you wanted to analyze a file from, it would have run pub get for you and then only analyzed that package. But because you weren't in a package, it just crawled the whole directory and analyzed every dart file it found (other than those in directories with .dartignore files). I think it's creating a temporary file and a temporary .packages from the .packages files it finds, but it's not clear to me why it wouldn't then run pub get in those directories first.
Another bug here is that the argument is ignored. You can't use flutter analyze to analyze a single file. We should show a message to that effect (or support analyzing a single file or path).
Another bug is that flutter upgrade for some reason does run pub get in the flutter directory. It shouldn't do that (or we should always do it, but that seems wrong).
So in summary:

[ ] flutter analyze should either complain about being given a file argument or do something with it.
[ ] flutter analyze should run pub get in the directories it scans that contain pubspec.yaml files.
[ ] flutter upgrade should not run pub get in flutter repo directories.

The instructions on flutter.io are aimed at people writing flutter apps, not people writing flutter. For the latter, please see https://github.com/flutter/flutter/blob/master/CONTRIBUTING.md.
I've filed separate bugs for the issues I listed above, so I'm going to close this bug. Please don't hesitate to reopen if you think I've missed a key issue here.

No problem, thanks for looking into it.


Broken links in documentation

The documentation for backtrackable at https://package.elm-lang.org/packages/elm/parser/latest/Parser#backtrackable points at the missing https://package.elm-lang.org/packages/elm/parser/latest/TODO. There are some more links like this on the same page.

Also the link to inContext in the README:
https://github.com/elm/parser/blob/master/README.md#tracking-context

Also:

README explains delayedCommit in detail and links to it in the docs, but I believe it has been replaced with backtrackable
README has two <br> tags at the bottom which are displayed as plaintext on package.elm-lang.org
backtrackable should link to semantics.md(?)
there should be an example for commit (in semantics.md?) based on its documentation
Parser.Advanced is referenced in the docs of Parser but not exposed in elm.json


Change copy of rewards screen

Description
Just copy changes to the rewards screen

Other changes
N/A
Tested
Manually
How others should test
Go to the rewards screen and make sure you see the right copy.
Related issues

Fixes #998

Backwards compatibility
N/A

Hey @MuckT We have verified this issue using latest Test flight build V 1.19.0 and Android internal build V1.19.0 & updated status in Gsuite.
Verified On Devices: Samsung Galaxy J5 Prime (8.0), Samsung Galaxy M01 Core (10.0)
GSuite: https://docs.google.com/spreadsheets/d/1bWi6smdTMClreG96sGjYxb2lee9Ja6eT4-Lm7eXm1gM/edit#gid=1206218866&range=J3
Query:

Also we have observed that, below "Earn without commitments" section we are able to see  "Earning is stress free: Connect your phone number to start earning" text but in attached snapshot we are able to see "Earning is stress free: Add cUSD or cEUR to your account and start earning on your balances" text. So is it expected or we need to raise ticket for same?
Attachment: EarnRewardDescriptionSomewhatDifferenceThanSnapshot

CC: @gnardini

Oh, the copy needs to change depending on wether the user connected their phone number or not.
Yes, please do raise a P2 ticket, thanks!

Hey @gnardini thanx for more information. Here are our observation for both cases:

[ ]  For verified number we are able to see "Earning is stress free: Add cUSD or cEUR to your account and start earning on your balances" text &
[ ] For unverified number able to see "Earning is stress free: Add cUSD or cEUR to your account and start earning on your balances" text on "A new way to earn" page.
Verified On Devices: Samsung Galaxy J5 Prime (8.0), Samsung Galaxy M01 Core (10.0)
Attachment:
AbleToSeeExpectedTextForVerifiedNumber
AbleToSeeExpectedTextForNonVerifiedNumber
Thanks..!!

Hey @gnardini,  We have verified this issue using latest Test flight build V 1.19.0 and Android internal build V1.19.0 & updated status in Gsuite.
Sheet link: https://docs.google.com/spreadsheets/d/1bWi6smdTMClreG96sGjYxb2lee9Ja6eT4-Lm7eXm1gM/edit#gid=1206218866
Devices: iPhone 12 (14.7.1), Redmi Note 8 (10.0)

https://valora.testquality.com requirement R86.

Hey @MuckT We have verified this issue using latest Test flight build V 1.19.0 and Android internal build V1.19.0 & updated status in Test Quality Tool
Test Quality Tool link:
Android (9.0):

https://valora.testquality.com/project/12128/plan/21816/run/150277/result/2485119

Verified On Devices: Redmi Note 8 (9.0)
CC: @gnardini


Update provincial_mp_cumulative.csv

OK, I changed the format. I'm not sure how to resolve the conflict though.

@lizelgreyling you seem to have deleted the file from thie PR.  Are you wanting to start again or was that a mistake?

@dmackie: I completely redid the file. It should be correct now.


As VALIDATOR, I want to cross-check my study data against the study define.xml for null values in variables that are mandatory so that I can stay compliant with SD1229

This check applies to SEND, SDTM, and ADaM data.
This check is IG-version-agnostic.
This check applies to both dataset definitions (ItemRef within ItemGroupDef) and value list definitions (ItemRef within def:ValueListDef)

SD1229 in FDA VR v1.6:

Rule message: Variable value is null when value-level condition occurs
Rule description: Variable value must be populated when value level attribute Mandatory='Yes' as specified in define.xml file.

During validation i made the VSORRESU variable order number 1 value null in dataset. The define file defines it as mandatory. After running the rule I observed that the null value was not caught by the engine, which fails the Acceptance Criteria. I have attached the generated report and rule, dataset and define file, to reproduce the issue.

test file.zip
CORE-Report-2023-07-28T16-00-42.xlsx


Dynamically generate font name

this prevents IE from caching the font values and clashing
deeper integration with fontello?

Currently there isn't a way to automate this process.  Probably best to document the process in the README and move on.

I have documented the process in the README and a pull request has been created.


Don't allow a linefeed to scroll when outside DECSTBM margins

Summary of the Pull Request
When the DECSTBM margins are set, scrolling should only allowed within those margins. So if the cursor is below the bottom margin, it should just be clamped when it tries to move down from the bottom line of the viewport, instead of scrolling the screen up. This PR implements that restriction, i.e. it prevents scrolling taking place outside the DECSTBM margins, which was previously allowed.
PR Checklist

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

Detailed Description of the Pull Request / Additional comments
This simply adds a condition in the AdjustCursorPosition function to check if the cursor is moving below the bottom of the viewport when the margins are set. If so, it clamps the y coordinate to the bottom line of the viewport.
The only time it's acceptable for scrolling to happen when margins are set, is if the scrolling is taking place within those margins. But those cases would already have been handled earlier in the function (in the fScrollDown or scrollDownAtTop conditions), and thus the y coordinate would have already been prevented from moving out of the viewport.
Validation Steps Performed
I've added some screen buffer tests to confirm this new behaviour, and I've also checked the test case from the initial bug report (#2657) and made sure it now matches the results in XTerm.

Thank you!


kubernaut claim should print a reminder to set the KUBECONFIG env var

Rather than just telling you where the configuration was written, it should remind you to set KUBECONFIG. Something like
$ kubernaut claim
Warning: Kubernaut does not currently support LoadBalancer services!
Wrote kubernetes config to /home/ark3/.kube/kubernaut
Reminder: export KUBECONFIG=/home/ark3/.kube/kubernaut

This will be in 0.2. It supports detection and assistance for Bash, Fish and Zsh.


Optimize into_component for float_to_uint, u8 to f32/f64

Implement speedup for component conversion of f32/f64 to uint.
Add IntoComponent implementation for u8 to f32/f64.
The float to uint conversion doesn't handle NAN or infinity.

I recently picked up Hacker's Delight, a book with some programming arithmetic tricks and algorithm implementations. On page 377, there's a table with conversions between IEEE floats/doubles and uint32/64s. I implemented some of the conversions for the component conversions and there was a slight speedup for u8 to f32, and a considerable speedup for f32 to u8.
Rgb family/rgb_u8 to linsrgb_f32
                        time:   [8.0802 us 8.1011 us 8.1223 us]
                        thrpt:  [17.237 Melem/s 17.282 Melem/s 17.326 Melem/s]
                 change:
                        time:   [-6.0041% -5.6794% -5.3362%] (p = 0.00 < 0.05)
                        thrpt:  [+5.6370% +6.0214% +6.3876%]
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild
Rgb family/linsrgb_f32 to rgb_u8
                        time:   [9.0137 us 9.0414 us 9.0701 us]
                        thrpt:  [15.435 Melem/s 15.484 Melem/s 15.532 Melem/s]
                 change:
                        time:   [-42.937% -42.707% -42.467%] (p = 0.00 < 0.05)
                        thrpt:  [+73.812% +74.542% +75.246%]
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  3 (3.00%) high mild
  1 (1.00%) high severe

The formulas don't work for the entire range of floats/doubles, but I believe they cover everything needed by palette. The code sacrifices clarity for performance/magic but I've added comments to the relevant sections.

Interesting. Luckily we are working with unsigned values within the ranges, so no loss in that sense. What would the result be if a float value happens to be out of range?

In the current implementation, it would panic on a value less than some small, negative fractional number and return 0 for everything above the limit. Instead, we can use wrapping_sub or overflowing_sub which don't seem to have any substantial impact on performance. I originally wrote it with overflowing_sub which passes back a bool on whether overflow occurred, I don't remember why I changed it to a regular subtraction.
Here's a playground link with the regular and wrapping subtraction, and their results.
12845056 - Regular sub: 0
340282350000000000000000000000000000000 - Regular sub: 0
-340282350000000000000000000000000000000 - Regular sub: 0
NaN - Regular sub: 0
inf - Regular sub: 0
-inf - Regular sub: 0
-0.001 - Wrapping sub: 255
12845056 - Wrapping sub: 0
340282350000000000000000000000000000000 - Wrapping sub: 0
-340282350000000000000000000000000000000 - Wrapping sub: 0
NaN - Wrapping sub: 0
inf - Wrapping sub: 0
-inf - Wrapping sub: 0

I know components can be unclamped so this may be an issue.

For example, here's a program straight from the docs.
use palette::convert::FromColorUnclamped;
use palette::{Lch, Srgb};

fn main() {
    let rgb = Srgb::from_color_unclamped(Lch::new(50.0, 100.0, -175.0));
    dbg!(rgb);
    dbg!(rgb.into_format::<u8>());
}

which implicitly converts an unclamped to a clamped value:
[src\main.rs:6] rgb = Rgb {
    red: -3.2585966353532334,
    green: 0.5920742313913836,
    blue: 0.5165314331478271,
    standard: PhantomData,
}
[src\main.rs:7] rgb.into_format::<u8>() = Rgb {
    red: 0,
    green: 151,
    blue: 132,
    standard: PhantomData,
}

Using wrapping subtraction, of course we don't get the same results:
[src\main.rs:6] rgb = Rgb {
    red: -3.2585966353532334,
    green: 0.5920742313913836,
    blue: 0.5165314331478271,
    standard: PhantomData,
}
[src\main.rs:7] rgb.into_format::<u8>() = Rgb {
    red: 130,
    green: 151,
    blue: 132,
    standard: PhantomData,
}

Should converting from an unclamped value to another component format require an explicit clamping to a valid color?

Hmm, I would like to avoid making it too surprising. I guess it's generally expected to clamp, or rather saturate the bits, when converting to unsigned ints. Float to float can still preserve the value as good as possible. I would think it's the more useful of the behaviors, as it's much more predictable and also follows the analogy of light intensity. Otherwise I'm also afraid people would still have to clamp or check themselves to be sure.
I made a modified version that combines clamping to <= 255 and saturating_sub: playground link. I have no idea what the performance impact is, but it seem to have the desired behavior. Including for the extra values I added.
fn into_component_saturate(n: f32) -> u8 {
    let max = u8::MAX as f32;
    let scaled = (n * max).min(max);
    let f = scaled + f32::from_bits(C23);
    f.to_bits().saturating_sub(C23) as u8
}

Removing the clamping to max makes it behave more like with wrapping.
I'm afraid we may have to sacrifice a bit of performance to get the behavior right, but it may still be better than the current state or the alternatives.

Yes, saturating is what I would expect. I wasn't aware of that function.
I figured I might have to add the clamping back but there was some design involved to make the right choice, thanks for that. Only a slight regression from the unclamped version I first pushed.
Rgb family/linsrgb_f32 to rgb_u8
                        time:   [9.4328 us 9.4483 us 9.4638 us]
                        thrpt:  [14.793 Melem/s 14.817 Melem/s 14.842 Melem/s]
                 change:
                        time:   [+3.5184% +3.9171% +4.2800%] (p = 0.00 < 0.05)
                        thrpt:  [-4.1043% -3.7695% -3.3988%]
                        Performance has regressed.

Compared to master, still a big improvement.
Rgb family/linsrgb_f32 to rgb_u8
                        time:   [9.5660 us 9.5841 us 9.6032 us]
                        thrpt:  [14.579 Melem/s 14.608 Melem/s 14.635 Melem/s]
                 change:
                        time:   [-39.454% -39.247% -39.048%] (p = 0.00 < 0.05)
                        thrpt:  [+64.064% +64.600% +65.164%]
                        Performance has improved.

And we get proper clamping
[src\main.rs:6] rgb = Rgb {
    red: -3.2585966353532334,
    green: 0.5920742313913836,
    blue: 0.5165314331478271,
    standard: PhantomData,
}
[src\main.rs:7] rgb.into_format::<u8>() = Rgb {
    red: 0,
    green: 151,
    blue: 132,
    standard: PhantomData,
}

Yes, it's still a great improvement! Well done! Would you mind adding a few tests that compares to what round gives? Just to be on the safe side, as usual. I'm especially thinking cases like below 0, above 1, 0.5, and 0.99 (or something close to 1 that should become the max value, to make sure #117 doesn't come back), but the special ones like NaN and +/-Inf can also be added for good measure.

Yes, I think it's a good idea to add more tests. As I was writing this I realized there wasn't much of a safety net for the special cases.
The new behavior doesn't have the same rounding due to the rounding mode of towards nearest even instead of just towards nearest. Here's a playground link of the tests I came up with.
The uint to float behavior looks good. Float to uint has small differences
Float to Uint conversion:

-800 - New: 0, Old: 0
-0.3 - New: 0, Old: 0
0 - New: 0, Old: 0
0.005 - New: 1, Old: 1
0.024983 - New: 6, Old: 6
0.15 - New: 38, Old: 38
0.3 - New: 76, Old: 77 /* difference */
0.5 - New: 128, Old: 128
0.6 - New: 153, Old: 153
0.7 - New: 178, Old: 179 /* difference */
0.8 - New: 204, Old: 204
0.8444 - New: 215, Old: 215
0.955 - New: 244, Old: 244
0.999 - New: 255, Old: 255
1 - New: 255, Old: 255
1.4 - New: 255, Old: 255
12845056 - New: 255, Old: 255
340282350000000000000000000000000000000 - New: 255, Old: 255
-340282350000000000000000000000000000000 - New: 0, Old: 0
NaN - New: 255, Old: 0 /* difference */
inf - New: 255, Old: 255
-inf - New: 0, Old: 0

NaN doesn't make sense in either case, so I don't know if it matters. The rounding behavior is probably good enough too. May be surprising in a few cases, compared to school math, but I would bet it disappears in the noise when doing anything more advanced. Some seem to even recommend that behavior for floats.

I thought the same thing about NaN. As I've learned more about this, I've come to find out that is the default rounding mode for IEEE floating points. A benefit is that it eliminates bias towards/away from 0 even though it distributes numbers towards even more than odd. So as you said, it may be surprising sometimes compared to school math but it's normal for the purpose of rounding IEEE floats.

I added comments to notate the special cases as we've discussed here. I'm not sure how helpful they are, but I also added tests for the current expect results of rounding from f32/f64 to u8, if not helpful now then for the future.

It's always good to make notes of decisions and reasoning for future adventurers. :slightly_smiling_face: Even if it's just us coming back to it.
I think this looks good! I don't have anything to add, so feel free to prepare for merge if you don't have anything more. :+1:

I think I should delete this section in the Packed docs about how 0.5f32 gets rounded to 0x80. That's still true but it's not true how it's done anymore.
/// // The second assert is essentially how library components are converted
/// let float = 0.5f32 * 255.0;
/// assert_eq!(0x7F, float as u32);
/// assert_eq!(0x80, float.round() as u32);

This is ready to go if you're okay with the changes :+1:

Yes, let's do this. Thank you!
bors r+


Adding service log message for unsupported workload on master/infra node

This message is raised out of findings from OHSS-3928 where customer scheduled pods on infra nodes. This is against OSD Service Policy.

/lgtm


element-{web,desktop}: 1.10.12 -> 1.10.13

Description of changes
https://github.com/vector-im/element-desktop/releases/tag/v1.10.13

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 174280 run on x86_64-darwin 1

1 package built:

element-desktop


Moving operation for resource types 'Microsoft.Web/staticSites' across subscriptions or across ResourceGroups failed with InternalServerError

Issue Description
Moving operation for resource types 'Microsoft.Web/staticSites' across subscriptions or across ResourceGroups failed with InternalServerError because it exceeded the allowed time limit of 1 hour.
User Role and Permissions
I am the subscriptions owner, and I also have both Owner and Contributor permissions over the resources. My role management over the Static Web App that I want to move is: contributor, reader, admin, anonymous, authenticated.
For Static Web App creation was used
Static Web App Deployment Source:
- Github
- Other
- have not tried using as source AzureDevops
Static Web App Azure Functions and staging details
- Tried with different regions like WestEurope, CentralUS, Est US 2
Reproduction Steps
Scenario 1: Move Static Web Site across subscriptions (from subsA to subsB)
Scenario 2: Move Static Web Site  across Resource Groups (in another RG, same subscription and region)
Scenario 3:  Try  to create Static Web Apps with  Azure Functions and staging details in different regions and try to move then across ResourceGroups in same subscription, or in other subscritions
Expected Behavior
The Static Web App should be moved successfully without exceeding the allowed time limit.
Actual Behavior
In all the scenarios, the move operation failed with an InternalServerError (in Portal log) and with the following message via CLI:
CLI Error Message
Resources '/subscriptions/xxxx/resourceGroups/RG-A/providers/Microsoft.Web/staticSites/demo-static-app' move actions failed. The correlation Id is 'yyyyy'.
Exception Details: (ResourceMoveTimedOut) Move resources for provider 'Microsoft.Web' did not finish within allowed time '01:00:00'.
- Code: ResourceMoveTimedOut
- Message: Move resources for provider 'Microsoft.Web' did not finish within allowed time '01:00:00'.
- Target: Microsoft.Web/staticSites

Additional Information
Please perform the reproduction steps using the CLI as it provides a more detailed output.

Updates
This issue  is on MS side and potentially caused by a change made to the control plane business logic for Static Web Apps.
Action Plan
Waiting for mitigation from MS side.
Alternative
Recreate the Static Web App in the desired location, please use CLI (from Portal might be an issue at the moment, see details here: https://github.com/Azure/static-web-apps/issues/1095)

Hi @dummy-andra, yes this was caused by a regression introduced into our Move Resources logic. We are currently rolling out a fix for this. Thank you for your patience.

Hi @dummy-andra, the fix is now rolled out and you should no longer be experiencing the issue. Thanks


Fix hardcoded links to understand deployed environment

For: https://trello.com/c/R4wibPPq/478-make-find-your-local-council-breadcrumbs-and-related-links-link-to-current-environment-2
We cache Plek.new.website_root in Frontend.govuk_website_root and use that when building the links.  Also we tidy up all the other places using Plek.new.website_root to use the new thing.  Some extra notes in the commits if you're interested.

Looks good 👍


Ignore failures detected by junit

The junit step is detecting known failures and marking the jobs as
unstable, which github interprets as a failure. By setting the
health-scale factor to zero, we disable the test result contribution
to the build score, and the jobs should no longer be marked as unstable.
This should be reverted once the tempest tests are passing reliably.

Is this needed know that we have the blacklist? The point is to make the tests always mandatory so we dont break stuff with our changes, so tests should contribute to the job status, otherwise we have no assurance that we are not breaking stuff :)
BTW, those blacklisted tests should be removed pretty soon as we are working on fixing those

No, this was meant as an alternative approach, it's no longer useful now that the blacklist change has merged.

No, this was meant as an alternative approach, it's no longer useful now that the blacklist change has merged.

Sorry I did miss your comment in 756 before merging!


Use updated types-psycopg2 from typeshed

This finishes the work done in #126. Commits are cherry-picked to give the original author credit.
I also removed psycopg2 tests as they are typeshed's problem now.
Fixes #74
Closes #126

Looks great, thank you!!


Numbered list not rendered correctly in docs when using code block

This is how the current Installation section looks in the generated docs:

1 and 1 instead of 1 and 2.
Even if this is actually in the README:
## Installation

1. Add the dependency to your `shard.yml`:
```yaml
dependencies:
  library:
    github: user/library
```
2. Run `shards install`

That's probably a bug with the markdown renderer

The bug is very likely here: parser.cr:253.

Another solution is to stop investing time for this in the stdlib and use e.g. markd

Let's just ignore the error for now and change the list to be unordered. A fix can be delayed until we have some final decision on how to provide proper markdown support to the compiler. Or maybe we don't even need that if the compiler just outputs plain markdown and an external doc generator renderes it to HTML or whatever format.

There are battle tested markdown libraries, that's not a problem if we'd like to go in that direction.

We've been through the logistics of the compiler depending on external shards before and it's not going to happen any time soon.

@bcardiff shouldn't this be rather fixed before releasing 0.27.0?

I'd say not necessarily, the readme is usually read from GitHub, etc. The docs are used for the API (yes, the readme appears there too but someone will have read GitHub before, or having 1., 1. isn't a big deal).

Unfortunately now after #7118, the Installation section is rendered even buggier:

## Installation

1. Add the dependency to your `shard.yml`:

   ```yaml
   dependencies:
     library:
       github: user/library
   ```

2. Run `shards install`

IMHO the only sane way to proceed is to invest in a proper markdown implementation. I don't think it makes sense apply makeshift patches to the Markdown library every now and then when a new flaw is detected. This continuously requires attention and effort.
The stdlib Markdown implementation is very rudimentary and flawed. It's safe to assume it would require a complete rewrite to push it to an acceptable Markdown support level.
I proposed #4613 a long time ago, but it has been sidetracked from the main issue to discussions about the organization of the compiler code etc.. So it's still inconclusive and lacks any path to action.
Until there is any progress in that issue, I think it's best to simply avoid any flaws in the Markdown parser. I.e. change ordered list to unordered and revert #7118.


defibrilator doesnt work/nerf/buff idea

hello, i was excited to try out the new items as i am always, but somehow the defibrilator wasnt working sadly, i tried left clicking holding left click, spamming nothing worked. id like for it to be fixed and for it to be a little bit morebalanced perhaps? my input would be to make it more expensive, and also limit where it can be used, i suggest 1 charge reviving one guy but he can be on critical health or less than 100, that way you arent able to revive 100 health out of the get go ( maybe this could work if the defib had 2 charges) and another nerf to it would be for example, you cannot revive players killed by coil heads, cannot revive scorched players (oldbird grab), cannot revive snipped players (barber) you could add more of course but yeah. its a really interesting item and this is my contribution to makiing more fun/balanced. but yeah currently it doesnt work and i wasnt able to find anything in the config to enable it to work. thank you!

The defib is a new item and therefore may take a bit before I get it working 100%, with that being said I was pretty tired and made some simple mistakes while coding it that I have since fixed. I really like your balance ideas/changes and am happy to implement some of them after I get it working smoothly first. I would personally disable it for now as it will not be very balanced but after a few updates I should get it in a good spot balance wise. Thanks! If you'd like credit for the changes/ideas in the changelogs let me know.

hey thank you so much for your reply, i see now well i hope you get to fix it im really having fun with the mod its really briliant,  oh and also maybe you could also add cocooned players to that list of unrevivables haha, i also had one more change maybe, maybe youd need to use it a couple times for it to work since normal defibs can take multiple tries to revive a patient, kinda like the cruiser ignition(so if you decide to add this maybe the defib could use extra charge bars like 3 or 4 so  you can either be super lucky and revive a player on the first hit or  be unlucky and use all charges to revive, if ur lucky u get to revive more players with the extra charges which sounds fun), defib could also use some sounds (of course not need to add this if you find it difficult) oh and also it would be so nice to give some particles effects/sfx to the lifeline, its a cool item but its silent, it also seems somewhat really good in my opinion? being able to grab loot with it and beign able to save yourself from danger or falls, maybe it needs an extra risk? thats just me though, anyways id love to see more updates yorue doing really good work and no need to credit me of course, youre the master mind im just here to give my input its the least i can do:D

I'm fairly sure I can't differentiate the methods that players have died in code very well but I'll see what I can do, the other ideas sound good after I get it working. Sounds and effects are being added, I'm just picky so it take a while to find the perfect ones I want.

awesome sauce, thank you

sigma, i think ima go try this revive thing now

This is fixed/implemented and will release in a day or so, so I'm going to close this.

Hey, sorry for reviving this closed issue but do you have any idea what could be wrong with the defibrillator? I updated to the latest version and the logs say "no corpse found" when trying to use it, no matters what. Is there a way to make the mod spit out some more logs that could be useful for debugging?
Cheers

It just doesn't work at all in the most recent released version. But luckily I have 100% fixed it and tested it in my next version so keep an eye out for that. Right now I'm just trying to polish the update while I still have time.


Test PR

:star: PR built and available in a preview environment michaelwallett-node-http-pr-5 here


Motion and HSV recording

I have camera with hardware motion sensor, exposed through MQTT.
I enabled HSV, and want to use recording on person detection.
Currently, the only way to get that working is to set videoanalysis.active = true.
Is there a way to have HSV detecting motion using MQTT camera sensor instead of videoanalysis? My understanding is that object detection is done by Apple in the cloud, so I think videoanalysis only does motion detection and not object recognition?

I think object detection is done on your network and object classification is done in the cloud.
Ref: https://support.apple.com/guide/security/camera-security-sec525461d19/web
HomeKit decrypts the audio and video streams on the home hub and analyzes the video frames locally for any significant event.

Yes by object detection and object recognition I meant classification, vs motion detection.
So the question is the same: Is there a way to have HSV detecting motion using MQTT camera sensor instead of videoanalysis?

@mihailescu2m Does the mqtt client functionality work and trigger HSV classification even when the video analysis is off?

@utezduyar yes, and my cpu usage now is low

There is no MQTT broker support in camera.ui right, only client? You must be using an external broker.

@mihailescu2m can you share the conf for onvif2mqtt, I am trying to implement this for my camera.

@Rahulsharma0810
$ cat config.yml
mqtt:
  host: <mqtt_broker_ip>
  port: 1883
onvif:
  - name: <camera1_name>
    hostname: <camera1_ip>
    port: <camera1_onvif_port>
    username: <camera1_username>
    password: <camera1_password>
  - name: camera2
    hostname: <IP_ADDRESS>
    port: 2020
    username: camera2user
    password: camera2pass

@mihailescu2m thanks for it can you share your whole docker-compose setup, in my case even though both services are up, it is not detecting any event.

@Rahulsharma0810 I don't use docker-compose for this, just started a container with onvif2mqtt ...

Thanks for the quick reply, I made it work, can you share your cameraui mqtt conf.
Thanks @mihailescu2m

      "mqtt": {
          "motionTopic": "onvif2mqtt/<camera1_name>/motion",
          "motionMessage": "ON",
          "motionResetTopic": "onvif2mqtt/<camera1_name>/motion",
          "motionResetMessage": "OFF"
      },

Probable solution: #855


Popup description when placing the cursor

When reviewing the damage / healing of two classes with equal branches for example 2 fury warriors, when clicking on the damage done the pop-up description shows the correct information but having two damage windows, and buying the damage of both, a detail occurs when place the mouse to see the pop-up box shows the information of the other player not the main one.
In this example it was a dungeon (I had found the detail with 2 heals shamans)

I have two skada windows.
On the left I place one and on the right the other
When hovering over to see the pop-up information, the pop-up information of the warrior on the right  appears in the warrior on the  left.

Note: I have the last update of the skada with cache and wtf files deleted
Left warrior

Right warrior

The screenshots aren't really readable and I didn't catch the full problem!

Left warrior

Right warrior

So you mean it's the same warrior, same fight and same mode but showing different numbers?

When there are 2 characters with the same branch, the floating information only shows that of 1 character, but not the other.
In the example the warriors are
Bakusayga and the other is Shuhu, when comparing the damage done by them, the floating information on the Bakusayga detail shows Shushu's, but not Bakusayga's in the window where their damage detail is. In the window where the other warrior is if it shows its corresponding information

I figured the problem and i tested it. I am looking for the issue as I am typing. I will let you know when it's fixed!
Thanks for reporting this.

After checking the code since yesterday and trying to find the problem, it turned out it wasn't a problem but this is how the addon is. I tried for hours several solutions without altering the code but sadly to do so the whole code needs lots of changes.
When you are on a mode, for example Damage, you see the list of players. When you click on a player, the module stores the player's ID internally to itself self.playerid = id .. So now the module itself remembers the player. When you click on a spell, it does the same thing self.spellname = label ... Keep this in mind to understand what's said below.
When you have multiple windows, showing the same mode (Damage, Damage taken, Healing ...etc), the last clicked window is the one holding the data to show because as I said above it's stored per module, no per window.

If you click on Player B on Window 2, you change the player of the module.
Now if you click on a spell, the module has 2 data to look for : PlayerID and SpellID (or spell name) ... So no matter what you hover on the other windows, the module is looking for the what you clicked last, thus showing wrong tooltip if players have same spells, or showing absolutely nothing if the spells are different.
I hope I explained it in a easy way and unfortunately, this is a wontfix issue, so I am closing this issue.
PS: I will keep searching and trying things, if I come up with a good method without modifying the code too much I will let you now.

Skada used to store details about modes/players ..etc per module, this caused window tooltips to be misbehave.
This is how to reproduce the issue:

Have two windows showing the same set and the same mode.
Click on a player on one then a player on the other window.
Hover the spells and you will notice that the last clicked window shows proper tooltips while the other doesn't show unless the player is of the same class or they have common spells (i.e : Auto Attack).
Right-click on the window that doesn't work and it will swap to what's the active window is showing.

This is not a bug but rather how Skada works, at least so far.
What's on the next update?
The next update will allow players to freely have multiple windows showing the same set/mode and the tooltips will show the appropriate info because data is stored/cache per window instead of being stored per module.
How is this useful?
Player tend most of times to compare their characters to the same classes in the raid, and because Skada is not Details, the comparison window doesn't and won't be built-in BUT, if you want to compare, here is an example of how I do it:

Have you regular windows for UI purposes just like you are used to do (damage, healing, threat ... etc).
Create as many comparison windows as you want, have them stacked to each other and of course hidden.
The feature child-parent that was built-in but disabled will, after the update, be available and will allow you to attach a window to a parent (one-to-one relation). Whatever you do and wherever you navigate on the parent, the child will follow.
Let's assume I have three hidden windows: Window A parent of Window B which is a parent of Window C. The three windows are snapped to each other and hidden, only shown when I click Skada minimap button.
If I want to compare, after clicking the minimap button, I select the set/mode I want on Window A, which causes Window B to follow, which by consequence causes Window C to follow.
I select a player on Window A, both windows B & C will follow as well, but that's not a problem.
I right-click Window B to select a different player, Window C will follow but again, it's not a problem.
I right-click on Window C to select a 3rd player (preferably if the same class) and here I have 3 windows showing 3 players of the same class and their spells.
Now, any spell I click on Window A will cause the other windows to follow and show that spell info for the other 2 players.

If it is complicated to understand, You can always ask me on Discord to explain it more.


⚠ Bofrost.be has degraded performance

In e08394c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2593 ms

Resolved: Bofrost.be performance has improved in 96f41cf.


python37Packages.gssapi: 1.6.9 -> 1.6.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.gssapi is: "Python GSSAPI Wrapper"
meta.homepage for python37Packages.gssapi is: "https://pypi.python.org/pypi/gssapi"
meta.changelog for python37Packages.gssapi is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.6.10 with grep in /nix/store/xg15n87f2zrrfssqhfa50p45vh0sp50f-python3.7-gssapi-1.6.10

directory tree listing: https://gist.github.com/a6af6e93ac2c103c76df2b7b29f90365

du listing: https://gist.github.com/e49c373b20a7798fdbfab0a8cdd26fa4

Rebuild report (if merged into master) (click to expand)

16 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.asyncssh
python37Packages.gssapi
python38Packages.asyncssh
python38Packages.gssapi

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xg15n87f2zrrfssqhfa50p45vh0sp50f-python3.7-gssapi-1.6.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.gssapi https://github.com/r-ryantm/nixpkgs/archive/d6decf82dd6638a159a662298af1bda485f5ba15.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xg15n87f2zrrfssqhfa50p45vh0sp50f-python3.7-gssapi-1.6.10
ls -la /nix/store/xg15n87f2zrrfssqhfa50p45vh0sp50f-python3.7-gssapi-1.6.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

2 packages failed to build:

python37Packages.asyncssh
python38Packages.asyncssh

2 packages built:

python37Packages.gssapi
python38Packages.gssapi

Maintainer pings

Build failures are fixed in #104724 and because they are unrelated to the version bump this can be merged.


Feat/azurem version upgrade

what

upgrade module version to support on Azurerm version 4.11.0

Terraform Security Scan Failed
Show Output

Terraform Security Scan Failed
Show Output


Experimental integration with Achoreographer

This is an alternative to #674.
It has significantly better latency performance, with a few peculiar behaviours I can't quite reason about. In particular, it seems to like falling back to 45 fps, but not consistently. More experimentation is needed, I think.
Overall, for the amount of code, the latency reduction is excellent. With this:

Total latency: ~33ms-43ms
vs without this:

Total latency. Highly variable: ~43ms-53ms
In both of these cases, 16ms is due to compositing, and ~10ms is due to rendering. That is, this integration still leaves some latency on the table, because the callback arrives at vsync, but our rendering doesn't take all of the time the next compositing time.

I've been doing some more thinking about this - this actually needs to be a component of/complement to #674.
The Android side docs are kind of terrible, but AChoreographer is extremely useful for us to have.


Show "diet" field by default

Hi again :)
Whether or not a place to eat offers vegetarian, vegan or other dietary foods is one of the most important pieces of information about a place to eat for a lot of people. But since this field is hidden in iD by default, the majority, who don't mostly care about this, is more likely to not add this information, even if they know or very easily can find out.
I would very much appreciate it if you could make this visible by default.
Thank you again for all your great work!

Hi @forteller. I agree this info is at least as important as cuisine, so I'm okay making it more visible. This will also help mappers avoid common mistakes like cuisine=vegan.

Could you please add this to amenity:fast_food too? Thanks! :)

Hi again. Diet type is still not shown by default for fast food.

openstreetmap/id-tagging-schema@11f552486ca946bb6ecf2fef6d4448d12dd68db9 went into v3.1.0, which iD hasn’t been able to upgrade to yet: #8600.

I see. Then I want to request that it's also added to the current branch :)


Multithread

Hi,
I have found this script and have modified it to:

allow concurrent connections
replace ansi escape codes to allow connecting to aruba/HP devices
add argument parser to allow customization of script call.

I have also added other files to the repo.

add setup.py and Makefile to help build a package
add version control vía bump2version.
systemd unit service to automatically start proxy as a service.

I have also change demo_server.py name to ssh_to_telnet_proxy, so it is named after the repository.
I am very grateful for your work. I would not have been able to implement a proxy like yours. I hope you find this modifications useful.
Best regards

Thanks for sharing your work! It makes this a whole lot more useable!


Refine PeerServerSegmentFinder

Separated from #12886 for easier review

Refine and simplify the logic of PeerServerSegmentFinder
Fix the bug of using netty port when admin port is not explicitly configured

Codecov Report
Attention: Patch coverage is 0% with 47 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (246d07f).
Report is 303 commits behind head on master.

Files
Patch %
Lines

...pache/pinot/core/util/PeerServerSegmentFinder.java
0.00%
24 Missing :warning:

...ata/manager/realtime/RealtimeTableDataManager.java
0.00%
7 Missing :warning:

...pinot/common/utils/fetcher/HttpSegmentFetcher.java
0.00%
5 Missing :warning:

.../pinot/core/data/manager/BaseTableDataManager.java
0.00%
5 Missing :warning:

...spi/utils/retry/ExponentialBackoffRetryPolicy.java
0.00%
3 Missing :warning:

...pinot/common/utils/fetcher/BaseSegmentFetcher.java
0.00%
2 Missing :warning:

.../core/realtime/PinotLLCRealtimeSegmentManager.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12933       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2428        -8
  Lines        133233   132836      -397
  Branches      20636    20568       -68
=============================================
- Hits          82274        0    -82274
- Misses        44911   132836    +87925
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
0.00% <0.00%> (-27.73%)
:arrow_down:

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Ssh in docker.

Hi i like to see how it is working. is there way connect to docker with admin rights and how is going... It will be good ideA. im newbiew and like computer science i want to learn. Man this project is super. PLease help make more features that nginx max. support. Include if you can incorporate main server, so peoples who have apache dont have to use it. Example:
I have apache2 and i have firefly iii on /var/www/html/fireflyiii so for that to work i need have apache2. But i dont know how apache2 work. So insalling nginx proxy manager can do all work. Be like fully web server and dont use apache2 server. Now when it run. have explanation. Im sure that when every day peoples use it. They will have more ideas and with that help you. And donate. with sshin docker i mean. I want enter in docker like root in ubuntu. MUACKS

One should never, ever ever run a ssh server inside docker.
You can enter a shell inside a docker container easily. I highly recommend taking an introduction to docker course online, there are plenty of free ones out there.
This project does not support hosting files within itself, only proxying to other services, so you will be able to run your apache stuff separately (even in another docker container) and proxy to it with NPM.
However, support for that is outside of this project.

One should never, ever ever run a ssh server inside docker.
You can enter a shell inside a docker container easily. I highly recommend taking an introduction to docker course online, there are plenty of free ones out there.
This project does not support hosting files within itself, only proxying to other services, so you will be able to run your apache stuff separately (even in another docker container) and proxy to it with NPM.
However, support for that is outside of this project.

i dont wajt ssh server inside docker. i want enter in docker with root access. like you use your computer i try with this command:
docker-compose run -u 0  bash name of containder and it didn't work.

docker exec -ti name_of_container bash


"show_result" MethodError

In the course of testing https://github.com/RelationalAI/raicode/issues/6046 I came across this issue wherein it seems the show_result method is broken.
Using Julia version 1.7.3 and using Pkg; Pkg.add(url="https://github.com/RelationalAI/rai-sdk-julia.git") per the documentation.
julia> rsp = exec(
           ctx,
           database,
           engine,
           "def a = 1;2 def b = 3;4 def output:one = a def output:two = b"
       )
Dict{String, Any} with 4 entries:
  "metadata"    => JSON3.Object[{…
  "problems"    => Union{}[]
  "results"     => Pair{String, Arrow.Table}["/:output/:two/Int64"=>Arrow.Table with 2 rows, 1 columns, and schema:…
  "transaction" => {…

julia> show_result(rsp)
ERROR: MethodError: no method matching show_result(::Dict{String, Any})
Closest candidates are:
  show_result(::IO, ::JSON3.Object) at ~/.julia/packages/RAI/nBTqc/src/results.jl:61
  show_result(::JSON3.Object) at ~/.julia/packages/RAI/nBTqc/src/results.jl:62
Stacktrace:
 [1] top-level scope
   @ REPL[27]:1

👍 you are correct. The show_result() function was only written for the v1 outputs, and we haven't rewritten it for the v2 outputs yet.

@NRHelmi has offered to implement this from the v2 response. (@NRHelmi: can you do it off the proto metadata? @bachdavi is working right now on adding proto support to this repo; can you coordinate with him on this?)

Fix verified in https://github.com/RelationalAI/rai-sdk-julia.git (main).
Thanks @NHDaly / @NRHelmi .

Note: This issue has been migrated to https://relationalai.atlassian.net/browse/RAI-5676.
This link is only accessible to employees of RelationalAI.


Support for pointers to incomplete types

First of all, thank you for providing this implementation of std::variant. It's great to have  a lightweight alternative to boost::variant available in C++11.
I just stumbled over an issue with recursive type definitions, and I'm not sure whether it's a limitation of C++11 or the variant implementation. The following struct definition compiles fine with GCC's and Clang's C++17 support:
struct S {
    std::variant<
        std::map<int, S>*
    > var;
};

When replacing std::variant with mpark::variant, I get an error message regarding the incomplete type S of std::pair<int, S>::second. Is it possible to enhance your implementation so that this kind of recursive type definition compiles with mpark::variant under C++11 as well?

Thanks for the report! I just tried it out and confirmed the issue. It either looks like my fallback implementation of is_swappable is instantiating too eagerly, or I'm checking for is_swappable where I shouldn't be. I'm going to be attending CppCon this week, so I'll be sure to fix this after the conference!

Awesome. Thanks for the confirmation and for having a look into the issue. Have a great time at CppCon.

The following commits address this issue:

https://github.com/mpark/variant/commit/e567a710ea6429b921fd42fa415d12232eab7e46
https://github.com/mpark/variant/commit/9e06a9d6a17fb31f2e4aabd42e59f48bd0ee0ed8
https://github.com/mpark/variant/commit/ea542a0fef86fb8c1696238b95bdaddb8ff2337c

Great! Thanks for fixing this issue. It works flawlessly now.

I'm experiencing something similar with gcc 7, seems to work with 8 https://godbolt.org/z/0fb-bV
(also works if I use standard variant)
Thanks!


#2359 continued

re: #2359 that issue was locked for some reason.
I have the latest versions of everything (node, npm, gulp), so i'm not sure that's the issue.
the transpilation docs you linked to liks to a 'more advanced dive', here: https://gulpjs.com/docs/en/documentation-missing, which is a 404. I would submit a PR, but i wouldn't know what to write. i'd love to learn, but i can't find the docs.

The team doesn't currently have the bandwidth to write any advanced docs. If you want to submit a PR, we will review.
I'm going to close this because that documentation is already tracked in https://github.com/gulpjs/gulp/issues/2256

ok, so the original issue ('need ES6 gulpfile example') is probably valid then? or do you consider that fixed?


Fix policy creation for multi-word names

Would love to see an example multi-word action in todos to ensure this is working

@andrepinho or @danielweinmann I'll need authorization to publish to rubygems<EMAIL_ADDRESS>


ApiException class having only internal constructors makes unit testing difficult

In order to unit test that my code handled an error response correctly, I wanted to have a mock/fake implement the interface for my api, and simply throw an ApiException when the method in question is called.
Unfortunately, that's not possible with ApiException constructors being internal, so instead I had to research how to customize and override HttpClient and HttpMessageHandler to actually form a 500 status code response message, without actually making a request.  This was made made even more complicated by RestEase.Implementation.Request.ConstructUri insisting that the HttpClient have a valid BaseAddress (otherwise generating a NullReferenceException), even when the HttpClient itself would have worked fine without one.
I strongly recommend making the constructors for ApiException public to facilitate testing.  There needs to be an easier way to force an api call to fail (without actually using the network), for the purposes of unit testing.

In order to unit test that my code handled an error response correctly, I wanted to have a mock/fake implement the interface for my api, and simply throw an ApiException when the method in question is calle

That's a valid request: I'm happy to make that change.

This was made made even more complicated by RestEase.Implementation.Request.ConstructUri insisting that the HttpClient have a valid BaseAddress (otherwise generating a NullReferenceException), even when the HttpClient itself would have worked fine without one.

That's incorrect behaviour: it should never throw an NRE.

Can you give me an example of a URI which HttpClient accepts, but which causes RestEase to throw an NRE, please?

The problem is when its a custom HttpClient doesn't have (or need) a BaseAddress (i.e. set to null).  This causes an exception in RestEase when attempting to make a call to the API.  Minimal example as a .NET Core console project is attached.  RestEaseNRE.zip

Aah, a null BaseAddress, not an "invalid" one. Gotcha.

Released as v1.2.3.


:green_heart: fix badge pointing at AucaCoyan not nushell org

I created the badges on README.md pointing to my repo AucaCoyan/nufmt.
This PR fix it and points correctly to nushell/nufmt

good catch!


Alpha support for RKE2 kubernetes distribution

Testgrid Run(s) Executing @

I need to fix this so you don't get an empty message. There are no templates so it didn't run anything.

Testgrid Run(s) Executing @

I need to fix this so you don't get an empty message. There are no templates so it didn't run anything.


Consider removing type in materialAssociations

Is it really necessary? It could probably be inferred from the material type.

Done in #97 .
The type was somewhat redundant and was a potential cause for problems when some mismatching happened.


Source mode docs in UI

Add Docs tab with reference list in Studio under Settings, similar to in style mode with CartoCSS reference. Include basic definitions of PostGIS database fields, example SQL code snippets for queries, and general trouble-shooting tips.

/cc @springmeyer

@amyleew any news on this?
We want to release a new version soon.
https://github.com/mapbox/mapbox-studio/tree/vs2015 is the branch I'm working on.

@BergWerkGIS - It's good to go! Working branch: https://github.com/mapbox/mapbox-studio/pull/1387. New PR to merge with vs2015

Thanks, @amyleew looks nice.


chore: update multiaddr dns prefix for bootstrap nodes

Summary of changes
The list of bootstrap nodes was provided using the /dns4 multiaddr prefix, which limits libp2p connections to IPV4 only. Since the ChainSafe Infrastructure team has added IPV6 support to their bootstrap nodes, this change introduces a node's ability to join using either IPV4 or IPV6 through the DNS path. This change supports nodes that are IPV4 only, as well as those that support IPV4 and IPV6 (ChainSafe & glif.io nodes).
Changes introduced in this pull request:

Multiaddr prefix changed from /dns4 to /dns

Reference issue to close (if applicable)

Closes
Other information and links

Change checklist

[X] I have performed a self-review of my own code,
[ ] I have made corresponding changes to the documentation,
[ ] I have added tests that prove my fix is effective or that my feature works (if possible),
[ ] I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Are you positive that, e.g., glif hostnames are resolvable to ipv6? From https://github.com/multiformats/multiaddr/blob/master/protocols.csv
dns -	domain name resolvable to both IPv6 and IPv4 address

❯ host -t AAAA calibration.node.glif.io
calibration.node.glif.io is an alias for a2ad5f028b78e4f628b8280d2482e5fa-9805736762487b63.elb.ap-northeast-1.amazonaws.com.

~
❯ host -t AAAA a2ad5f028b78e4f628b8280d2482e5fa-9805736762487b63.elb.ap-northeast-1.amazonaws.com.
a2ad5f028b78e4f628b8280d2482e5fa-9805736762487b63.elb.ap-northeast-1.amazonaws.com has no AAAA record


bamoe-issues#74: Documentation: "Decision Microservice in 15 minutes" example is missing how to deploy

issue
Closes https://github.ibm.com/dba/bamoe-issues/issues/74
On this PR
Add the Deploy section to the Getting started example

I like the copy, good work. I'd like to see the images about half the size they currently are. They're big both in file size and pixel size.
Also, we should have BAMOE Canvas branded stuff instead of Sandbox, yes?

Should I update the https://github.com/IBM/bamoe-docs/pull/19 images to BAMOE Canvas instead of KIE Sandbox too?

Yes, please do.
On Fri, Jun 16, 2023 at 13:34 Luiz João Motta @.***>
wrote:

Should I update the #19 https://github.com/IBM/bamoe-docs/pull/19
images to BAMOE Canvas instead of KIE Sandbox too?
—
Reply to this email directly, view it on GitHub
https://github.com/IBM/bamoe-docs/pull/29#issuecomment-1595193806, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHKE4M5ZSV476AYRJYKQLXLSYKXANCNFSM6AAAAAAZJUA3WQ
.
You are receiving this because your review was requested.Message ID:
@.***>

--
Sent from Gmail Mobile


is_between() method

Godot version:
89bcfa4
Issue description:
There are many useful built-in methods that do different kinds of things. One time, when writing a condition inside _process(), I thought that we could have something like is_between(a, min, max), which would be just short for if a >= min and a <= max. This might be useful when you want to put an expression for a, so you don't need a new variable and don't need to write the expression twice.

I agree, that sounds very usefull

I don't see much point, is_between(a, min, max) is not much shorter than a >= min and a <= max, and the latter is more explicit, especially regarding whether the bounds are considered in the interval or not. If we add this, some users will want is_strictly_between for a > min and a < max or is_between_min_and_strict_max for a >= min and a < max.

This might be useful when you want to put an expression for a, so you don't need a new variable and don't need to write the expression twice.

I don't understand that, why do you need a variable? The proposed is_between(a, min, max) would be the same as a >= min and a <= max, no variables are needed.

I don't understand that, why do you need a variable?

If a is a result of some complex calculations, like some_non_trivial_function()
then you either repeat it twice
if some_non_trivial_function() >= -10 && some_non_trivial_function() <= 20
or "cache" the result
var a = some_non_trivial_function()
if a >= -10 && a <= 20

With is_between() this becomes just is_between(some_non_trivial_function(), -10, 20). That's literally why I thought about this function.

I agree with @akien-mga on this one, the inclusion of the boundaries is not clear (or would need other arguments) and the gain isn't that significant.

Maybe min <= a <= max, like Python?

Closing in favor https://github.com/godotengine/godot-proposals/issues/889


Major docs overhaul (remove asciidoc and switch to MDX)

Description of change
This is a pretty large PR so apologies in advance. In short I've ripped asciidoc out of the website and switched to MDX as it's much better supported by all the Gatsby plugins (especially i18n solutions).
Checklist

[x] npm test passes
[x] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[x] contribution guidelines followed here

What sort of challenges were we seeing with Asciidoc and gatsby?  This seems like a pretty extreme solution...

What sort of challenges were we seeing with Asciidoc and gatsby? This seems like a pretty extreme solution...

 A lot of problems were caught with the recent Gatsby 5.x migration that required me to write a whole load of custom plugins that aren't going to be easy to maintain. We also introduced MDX when we merged the blog into the main website so having two different languages in a single website feels a little strange.

One theme was the handling of superscript and (e.g. code) annotations - would be good to have an md equivalent for those. The HTML handling for the buttons page made me feel a bit ill too ;-) Otherwise a fair recreation of what we have. We might choose to tweak a few pieces when the basic framework is in place.

Thanks @tellison, weirdly MDX should handle superscript so I've raised a bug to understand why it's not rendering on our website. Other than that the buttons page also made me feel a bit 'meh'. One of the advantages of MDX is that we can write our own custom components so I might make a side-by-side component that would allow us to mimic what we had in asciidoc.

Uh, I was happy to see many docs written in asciidoc. I'd appreciate to stay with it, from my point of view it is much more powerful than Markdown, in particular if it comes to developer documentation.
Maybe you would like to re-think the choice of gatsby as microsite tool, despite dropping asciidoc? Are design/site architecture decisions like this documented somewhere? I'd like to understand what particular requirements led to such decisions.

Uh, I was happy to see many docs written in asciidoc. I'd appreciate to stay with it, from my point of view it is much more powerful than Markdown, in particular if it comes to developer documentation.

@ascheman remember we're not dropping it for markdown but rather MDX which is much more powerful than asciidoc. For starters, we can write custom components for pretty much anything that markdown is missing. I'm happy to have more conversations around this but from experience to date, asciidoc is not as well supported in Gatsby, often more complex when trying to do simple things (tables and code blocks for example) and less familiar to the GitHub cohort of regular contributors that make changes to our project.

Uh, I was happy to see many docs written in asciidoc. I'd appreciate to stay with it, from my point of view it is much more powerful than Markdown, in particular if it comes to developer documentation.

@ascheman remember we're not dropping it for markdown but rather MDX which is much more powerful than asciidoc. For starters, we can write custom components for pretty much anything that markdown is missing. I'm happy to have more conversations around this but from experience to date, asciidoc is not as well supported in Gatsby, often more complex when trying to do simple things (tables and code blocks for example) and less familiar to the GitHub cohort of regular contributors that make changes to our project.

Well, my mileage varies on this. For instance, I prefer AsciiDoc tables over MD tables. On the one hand they are slightly harder to create (I leave that to the IntelliJ AsciiDoctor plugin) and to maintain. On the other hand they are harder to maintain.
If it comes to more complex stuff you want to express, you always have to learn one way or the other how to do that in your given tool context. Maybe people are more used to MD at first sight. But what does it help if they have to learn MDX and Gatsby to write elaborated docs.
I am not familiar with Gatsby, but it seems to be based on TypeScript and NPM?
So folks contributing to the docs may end up having to learn that? I'd like to argue to stay in the Java world here, e.g., by using jbake (or something similar) which makes use of template engines from the JVM universe.
I mean, in the end, the Adoptium web page is a statically rendered site. What makes Gatsby the tool of choice over other alternatives? Did you consider JVM based alternatives?
Having said that, I am always happy to get a deep insight into other universes besides the Java world :-)

@gdams Will need a rebase


UnicodeEncodeError with to_sql and unicode column names

From SO: http://stackoverflow.com/questions/33337798/unicodeencodeerror-when-using-pandas-method-to-sql-on-a-dataframe-with-unicode-c
To reproduce:
from sqlalchemy import create_engine
engine = create_engine('sqlite:///:memory:')
df = pd.DataFrame([[1,2],[3,4]], columns = [u'\xe9',u'b'])
df.to_sql('data', engine, if_exists='replace', index=False)

which gives UnicodeEncodeError: 'ascii' codec can't encode character u'\xe9' in position 0: ordinal not in range(128), because of this line: https://github.com/pydata/pandas/blob/master/pandas/io/sql.py#L856, where it stringifies the individual columns names with str (which fails on python 2 with unicode).

I suppose we should just use compat.text_type instead of str


test creating cache dir

We need an out-of-tree cache dir for svg files. A symlink to this will need to be created.

@dontlaugh leaving it to you to merge. let me know what I need to do

I still have some testing to do, but the gist is that the build code will the reading from a directory that is symlinked to something like /home/builder/cache. This normally would give me pause, but there are no on disk secrets accessible to the user. There are a few directories in $HOME that are left over from the from source builds of moarvm, nqp, rakudo. Those builds happen during the construction of the CI environment itself.
I'll need to play with this a bit after work.

@dontlaugh  FYI I just outlined changes needed to typegraph to use cache in #393