Get the sum to attempt to parse before it gives up

Previously the sum helper would ignore anything that wasn't a number and move on, this is a problem as some helpers pass as a string when used like this - {{sum (now "%Y") 10}}
As Lodash is already being used in here, I've taken advantage of the parseInt function, which passes through NaN if it can't turn it to number.

Sorry @hariadi, my mistake.
I've added in effectively {{sum "20" 10}} as that was the problem and the test is now passing.
What do you think?

Thanks @hariadi

@hariadi - When is the next NPM update?

@hariadi has there been another NPM publish, I want to use your version of this again on my projects rather than my fork


Skyline/work/20210111 inspect code for model dependency on UI

This checks for code in the model namespace that uses UI namespace (Skyline..*UI, Skyline.Controls, Skyline.Alerts).
I've picked off some of the easier refactorings.

I'm going to go with Nick's suggestion.

I'm going to go with Nick's suggestion.


Can't open app on Samsung Galaxy S20

Bug
The app isn't opening on Samsung S20. I am using the Version 0.0.11 of the Odysee app. Following are the Phone details:
Model name: Galaxy S20
Model number: SM-G980F/DS
One UI version: 3.1
Android version: 11
Kernel version: 4.19.87-22520152
To Reproduce
Steps to reproduce the behavior:

Go to phone's home screen
Click on the Odysee app icon
App opens with dark screen, just to crash 2 seconds later

Expected behavior
I expected to see a splash screen with the Odysee logo. After loading I should enter the app and be able to interact with the UI. However I can't even open the app, as it always crashing.
Screenshots
I have no screenshots

Which app are you referring to? Did you install the APK for Odysee or the Odysee web app installed via a browser? Try removing and re-adding the PWA app if so.


Error E0713 when building substudy

when I tried to build substudy: git clone https://github.com/emk/substudy.git cd substudy cargo build
I got this error:
Compiling submodel v0.1.0 (/Users/me/substudy/submodel)
error[E0713]: borrow may still be in use when destructor runs
--> /Users/me/.cargo/registry/src/index.crates.io-6f17d22bba15001f/url-1.6.0/src/form_urlencoded.rs:261:40
|
259 | impl<'a> Target for ::UrlQuery<'a> {
|      -- lifetime 'a defined here
260 |     fn as_mut_string(&mut self) -> &mut String...
261 |     fn finish(self) -> &'a mut ::Url { self.url }
|                                        ^^^^^^^^ - here, drop of self needs exclusive access to *self.url, because the type UrlQuery<'_> implements the Drop trait
|                                        |
|                                        returning this value requires that *self.url is borrowed for 'a

For more information about this error, try rustc --explain E0713.
error: could not compile url (lib) due to previous error
warning: build failed, waiting for other jobs to finish...
warning: vobsub (lib) generated 26 warnings
warning: subtitle_ocr (lib) generated 2 warnings
warning: substudy (lib) generated 10 warnings (run cargo fix --lib -p substudy to apply 4 suggestions)
and when I tried to fix it as suggested: cargo fix --lib -p substudy
I got this:
error: the working directory of this package has uncommitted changes, and cargo fix can potentially perform destructive changes; if you'd like to suppress this error pass --allow-dirty, --allow-staged, or commit the changes to these files:

Cargo.lock (dirty)

What can I do? Thanks!

Please try the latest version—the version you tried to build hadn't been updated for many years. There are also pre-compiled binaries available for Mac, Linux and Windows, though the Mac and Windows binaries are unsigned.

Hi Eric, thanks for your reply. I tried again with the latest version, without error but a warning showed up:
   Compiling substudy v0.6.10 (/Users/me/substudy/substudy)
warning: field `0` is never read
   --> substudy/src/video.rs:108:21
    |
108 | pub struct Fraction(Ratio<u32>);
    |            -------- ^^^^^^^^^^
    |            |
    |            field in this struct
    |
    = note: `Fraction` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
    = note: `#[warn(dead_code)]` on by default
help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field
    |
108 | pub struct Fraction(());
    |                     ~~

warning: `substudy` (bin "substudy") generated 1 warning


The 'sha256' password method is deprecated and will be removed in Werkzeug 3.0.

/code/kubedash/functions/user.py:84: UserWarning: The 'sha256' password method is deprecated and will be removed in Werkzeug 3.0. Migrate to the 'scrypt' method.
  password_hash  = generate_password_hash(password, method='sha256'),
/code/kubedash/functions/user.py:94: SAWarning: Object of type <User> not in session, add operation along 'Role.users' won't proceed (This warning originated from the Session 'autoflush' process, which was invoked automatically in response to a user-initiated operation.)
  kubectl = KubectlConfig.query.filter(KubectlConfig.name == username).first()

To me it seems like in the method 'generate_password_hash' you have to change the method parameter to "scrypt". Alternatively you can also provide some parameters n,r and p like so 'scrypt:n:r:p'. I think they have something to do with the salt, but there i'm not entirely sure. The default is "scrypt:32768:8:1". So in the end you can just write it new like this: "generate_password_hash(password, method='scrypt')". I hope this helps.

THX @SimonSchneider06

Thank you!


Automated cherry pick of #8059: fix: avoid serial vm creation

Cherry pick of #8059 on release/3.4.
#8059: fix: avoid serial vm creation

/lgtm
/approve


No longer works

The JSON in this example no longer works because the pipeline().parameters.subscription now sends back the subscription name and not the subscription ID, which is required to build the link for the "View pipeline run" button.

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

ID: b1a2fafe-d860-19b0-8440-95f4347e8ba2
Version Independent ID: 80b17f1a-b974-dfbe-cf0b-c92ce5cee346
Content: How to send notifications to a Microsoft Teams channel - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/how-to-send-notifications-to-teams.md
Service: data-factory
Sub-service: tutorials
GitHub Login: @nabhishek
Microsoft Alias: basitfarooq

@francisapel
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.


Feature request: epic timeline

Hi,
This is a great extension. Are you planning to allow for an epic timeline similar to what it does for features? We'd like to be able to see our epics and control how they span from start iteration to end iteration.

At this time it is not on the radar, did you try Epic Roadmap pivot? There you can see the progress of entire Epic across multiple teams, does that satisfy your requirement?

I did try it but no, it doesn't really satisfy this requirement (nor does it really work well at all yet).
Your extension has something vital which deployment plans don't...explicitly spanning multiple sprints (although it'd be great if I could set it without dragging.) We'd like to do that for epics same way you enable us to do for features.

Thanks I will think about it.
Can you share specific feedback about Epic Roadmap on what does not work at all?

Thanks for your feedback.
Closing as we shipped a feature where you can set start/end iteration without dragging it.
At this point there is no plan to have epic displayed in calendar view. We will reconsider if more people ask for it.


Migrate to using TinyXML2

This is the forward port of #241 for the current master branch sdf10.

there are some conflicts here, partially due to deprecation of the API's in parser_urdf.hh in the following bitbucket PR:

https://osrf-migration.github.io/sdformat-gh-pages/#!/osrf/sdformat/pull-requests/658/page/1

@azeey, we marked it as deprecated as of 9.2, and I think we are planning to remove it in 10.0? let's make a separate PR to move that header file from include to src, and then it will clarify the API impact of this PR

PR to not install parser_urdf.hh: https://github.com/osrf/sdformat/pull/276

@mjcarroll Do you know if you need a stable release for Ignition Dome (libsdformat10), or will y'all be fine with using master?
Asking in the context of #278, whether we should target libsdformat11 instead of libsdformat10.

@EricCousineau-TRI I would like to target dome.

OK, thanks!

do you mind rebasing on top of sdf10 and changing the target of this PR?

https://github.com/osrf/sdformat/commits/sdf10

do you mind rebasing on top of sdf10 and changing the target of this PR?

Done! Thanks for getting that set up.

this PR is huge, but I'm not sure if there's a way to split up the changes. once you start parsing with tinyxml2, all the functions are going to need to read those data structures

this PR is huge, but I'm not sure if there's a way to split up the changes.

One thing to reduce the size could be to do the conversion, and then the removal of the vendored TinyXML in a followup PR

@scpeters How do I go about updating sdformat-ci-pr_any-ubuntu_auto-amd64 to install tinyxml2?

Codecov Report

Merging #264 into sdf10 will decrease coverage by 0.20%.
The diff coverage is 78.74%.

@@            Coverage Diff             @@
##            sdf10     #264      +/-   ##
==========================================
- Coverage   86.66%   86.45%   -0.21%
==========================================
  Files          59       60       +1
  Lines        9071     9103      +32
==========================================
+ Hits         7861     7870       +9
- Misses       1210     1233      +23

Impacted Files
Coverage Δ

src/SDFExtension.hh
100.00% <ø> (ø)

src/XmlUtils.cc
62.50% <62.50%> (ø)

src/parser_urdf.cc
79.42% <74.13%> (-0.79%)
:arrow_down:

src/parser.cc
77.37% <83.82%> (-0.16%)
:arrow_down:

src/Converter.cc
91.57% <92.94%> (+0.11%)
:arrow_up:

src/Types.cc
100.00% <100.00%> (ø)

src/Param.cc
92.06% <0.00%> (-0.10%)
:arrow_down:

include/sdf/Types.hh
100.00% <0.00%> (ø)

... and 1 more

Continue to review full report at Codecov.

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

It appears that the example is failing due to a trailing quote in the simple.sdf file: https://github.com/osrf/sdformat/blob/master/examples/simple.sdf#L6
Is this intentional, or is it that TinyXML2 happens to be a bit more strict?

How do I go about updating sdformat-ci-pr_any-ubuntu_auto-amd64 to install tinyxml2?

the logic for the ubuntu build is in release-tools:

https://github.com/ignition-tooling/release-tools/blob/master/jenkins-scripts/lib/dependencies_archive.sh#L55-L107

for homebrew, we'll need a new formula with different dependencies; I'll work on that

for homebrew, we'll need a new formula with different dependencies; I'll work on that

https://github.com/osrf/homebrew-simulation/pull/1054

example is failing due to a trailing quote in the simple.sdf file

that was definitely not intentional; thanks for fixing it

for homebrew, we'll need a new formula with different dependencies; I'll work on that

osrf/homebrew-simulation#1054

osrf/homebrew-simulation#1054 is merged

@osrf-jenkins run tests

the homebrew CI is working, but it's failing to compile on Ubuntu, any ideas?

the homebrew CI is working, but it's failing to compile on Ubuntu, any ideas?

I believe it's because we force using the internal urdfdom on the Jenkins CI, I'll get that fixed up.

@osrf-jenkins run tests please

the build farm is backed up, but I expect it to pass, so I'll approve
let's wait to merge until the last job finishes

ok, it looks it just needs an update to the migration guide and changelog, and then we can squash and merge
@mjcarroll do you want me to update these?

ok, it looks it just needs an update to the migration guide and changelog, and then we can squash and merge
@mjcarroll do you want me to update these?

updated in 7155942

updating dependency in release repo: https://github.com/ignition-release/sdformat10-release/pull/1


Add Feature model

Adds support for https://github.com/gnosis/safe-client-gateway/issues/675

Add a new model – Feature
Feature represents features that are enabled for specific chains
It is represented by a single name (its primary key)
Add a migration that adds the following features – CONTRACT_INTERACTION, DOMAIN_LOOKUP, ERC721, ERC1155, SAFE_APPS and SPENDING_LIMIT

I quickly looked at how django handles many to many relationships. I am just left with the question, what happens when you update a Feature db row, as we don't have id's I suppose a new entry is created in the db, right? Are we ok with this?

Feature does have an id but it's hidden on the model side as it's mostly used for internal purposes. However this is not the case with Wallet since Wallet has the key as the primary key so changing it leads to a creation of a new entry. This is not what we want but a fix is on the way 👍


My first submission

Check List:

[x] You have run ./mvnw verify and the project builds successfully
[x] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[x] All formatting changes by the build are committed
[x] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[x] Output matches that of calculate_average_baseline.sh
[x] For new entries, or after substantial changes: When implementing custom hash structures, please point to where you deal with hash collisions (line number) ->

Execution time: 3800 ms
Execution time of reference implementation: 222823 ms

@gunnarmorling Hope you're having a great evening and that all is ready to be tested.
Thank you for creating such a magnificent challenge. My colleagues and I (and java community) learned a lot!

This produces incorrect output for the 10K keyset test (see create_measurements_3.sh):
+ timeout -v 300 ./test.sh JurenIvan measurements_10K_1B.txt
Validating calculate_average_JurenIvan.sh -- measurements_10K_1B.txt
87c87
< Alc;5.0;19.0;31.5
---
> Alc;-11.6;19.8;59.2
133c133
< Az ZubayrCaucaiaVitsyebskShinjukuNicolás Ro;6.1;14.8;29.6
---
> Az ZubayrCaucaiaVitsyebskShinjukuNicolás Ro;-15.4;14.4;43.3
294c294
< Chon;10.9;18.4;28.5
---
> Chon;-12.4;17.0;47.9
...

Yes, you can. As we're after the cut-off time, you'll have two more changes you can make to this PR (see note at the top of the README). If it's not working or valid after that, I'll have to close it unfortunately. Updates should be pushed quickly, so I can evaluate all the pending entries. Thx!

Getting this exception now for the 10K keyset:
Benchmark 1: ./calculate_average_JurenIvan.sh
Exception in thread "main" java.lang.IllegalArgumentException: Size exceeds Integer.MAX_VALUE
	at java.base/sun.nio.ch.FileChannelImpl.map(FileChannelImpl.java:1293)
	at dev.morling.onebrc.CalculateAverage_JurenIvan.processSegment(CalculateAverage_JurenIvan.java:54)
	at dev.morling.onebrc.CalculateAverage_JurenIvan.lambda$main$0(CalculateAverage_JurenIvan.java:43)
	at java.base/java.util.stream.IntPipeline$1$1.accept(IntPipeline.java:180)
	at java.base/java.util.stream.Streams$RangeIntSpliterator.forEachRemaining(Streams.java:104)
	at java.base/java.util.Spliterator$OfInt.forEachRemaining(Spliterator.java:712)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:556)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:546)
	at java.base/java.util.stream.ReduceOps$ReduceTask.doLeaf(ReduceOps.java:960)
	at java.base/java.util.stream.ReduceOps$ReduceTask.doLeaf(ReduceOps.java:934)
	at java.base/java.util.stream.AbstractTask.compute(AbstractTask.java:327)
	at java.base/java.util.concurrent.CountedCompleter.exec(CountedCompleter.java:759)
	at java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:507)
	at java.base/java.util.concurrent.ForkJoinTask.invoke(ForkJoinTask.java:676)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateParallel(ReduceOps.java:927)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:264)
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:702)
	at dev.morling.onebrc.CalculateAverage_JurenIvan.main(CalculateAverage_JurenIvan.java:45)

Note you can push once more to this PR before I'll either merge it (if valid) or have to close it. Thx!

Ok, this looks good now. 00:08.157 for the standard key set.


Querying the broker for online (connected) clients in realtime

Hi
Is there a way to query the broker for a specific clientID,
whether it is online or not (connected or not)?
The use case is that we need to send a push notification to wake our clients if they
are disconnected, which means we need a way to ask the broker
if the client is connected or not (across the cluster if possible, or just a single broker instance)
Thanks a lot

@syberkitten We will release more management APIs including clientId lookup in 2.3-beta.2 :)

@syberkitten http://emqtt.io/docs/v2/rest.html#clients

@syberkitten More HTTP Management APIs have been added in 2.3-beta.2 release.


nicer/shorter "share" urls

right now the JS code aggressively encode everything so we get
https://grol.io/?c=%2F%2F+Fibonacci+number
func+fib(x){if+x<%3D1{x}else{fib(x-1)%2Bfib(x-2)}
fib(35)
while this works too and it a lot shorter:
https://grol.io/?c=//+Fibonacci+number%0Afunc+fib(x){if+x<=1{x}else{fib(x-1)%2Bfib(x-2)}%0Afib(35)

< might need escaping in addition to  ->+, +-> %2B and \n-> %0A

Getting better:
https://grol.io/?c=//+Fibonacci+number
func+fib(x){if+x<=1{x}else{fib(x-1)%2Bfib(x-2)}
fib(35)


Add Vertical Timeline to Blogs

Something Like https://codyhouse.co/gem/vertical-timeline?

A similar design is here:
https://wordpress.com/design/type/free/eric1016.wordpress.com


Colorscheme should work for colorblind.

Using the same general colors with simple hue alterations is not useful for color blind individuals especially when the colors used are exactly on the lines of confusion.

Needs stated goals to abide by the guidelines for colorblindness. Under no circumstances should color be the only indication of the state of something.
https://uxdesign.cc/color-blindness-in-user-interfaces-66c27331b858
This problem more or less comes out of whisperer where the blue for vector engrave and the grey for raster are basically the same thing and no indications are made, beyond color to figure out which is which.

Added this to design guide, and design guide to wiki.
https://github.com/meerk40t/meerk40t/wiki/UI-and-UX-Design-Guide


Square max size limit if a player is in rec mode

Since the max split limit is squared as well

@Agarian Looks good 👍


treeView.message = undefined should not block viewsWelcome from appearing

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.82.2
OS Version: MacOS 13.5.2

Steps to Reproduce:

Use VSCode treeView with a viewsWelcome, make sure the treeView is empty
Set the treeView.message to undefined
The viewWelcome is not shown

A minimal reproducible example is accessible on here: https://github.com/JulesFaucherre/vscode-views-welcome-issue
Expected behaviour:
Setting the treeView.message to undefined should make VSCode consider the message as empty and show the viewsWelcome

We also tried to run delete treeView.message. This does bring back the viewsWelcome but the treeView.message do not show anymore when we set its value after that


interstitial issue/misundestunding

Hi there
I am facing the following scenario in an ionic/cordova app:

start the app
initialize the the ads unit ids
creates the interstitial
then, in the controller of the initial view I show the interstitial
this is repeated every time the app is run

I know the interstitial is created (or, at least, the prepareInterstitial method is called and its success callback is run).
Then, in the initial view's controller, I call a showInterstitial and I know its success callback is run. Also, the onAdPresent event is triggered.
The issue is, when I close the app and then rerun again, the startup flow is pretty much the same but the interstitial is not showing: its success callback is called but the onAdPresent is not triggered.
This scenario only is happening in android!!!, in iOS everything goes smooth
Environment:

cordova 6.2
platform<EMAIL_ADDRESS>cordova-plugin-admobpro 2.19.1 👍
cordova-plugin-crosswalk-webview 1.5.0
(...) more plugins (...)

Any hint is very welcome
Cheers in advance and very good plugin

I just guess, in android, when you close the app, the process does not really exit.
Note: Before each showInterstitial, you need call prepareInterstitial() once again.

hey, thanks for the reply!!!
Actually, now I am seeing it is happening also in iOS :-/.
I can provide the code which is run. This is an angular factory exporting the functionality. The piece of code in the factory where I prepare the interstitial is:
function initInterstitial(autoShow) {
      var deferred = $q.defer();
      var defaultOpts = {
        adId: admobid.interstitial,
        autoShow: typeof autoShow !== 'undefined' ? autoShow : true,
        isTesting: true,
        success: function() {
          console.log('Interstitial created!!!');
          deferred.resolve();
        },
        error: function() {
          console.log('Failed to created interstitial');
          deferred.reject();
        }
      };

      $window.AdMob.prepareInterstitial(defaultOpts);

      return deferred.promise;
    }

I know this is run (and, apparently, successfully) as I can se the 'Interstitial created!!!' message in the console.
Then, the part where the interstitial is shown is easier:
function showInterstitial() {
      $window.AdMob.showInterstitial(function() {
        console.log('Interstitital showed ok');
      }, function() {
        console.log('Failed to show the interstitial');
      });
    };

Also, I can see the in the console 'Interstitial showed ok' message.
Before preparing the interstitial, I register to the onAdPresent event, as:
$document.bind('onAdPresent', function(data) {
        console.log('$document.bind(onAdPresent): ' + JSON.stringify(data));
        if (data.adType == 'rewardvideo') {
          console.log(data.rewardType);
          console.log(data.rewardAmount);
        }
});

Despite I always can see the 'Interstitial showed ok' message, I can't see the event triggered along with it, which is confusing me more. I dunno whether or not there is some scenario where the ads are not shown despite following all these steps, but I don't get any other clue/error.
I am trying to modify my codebase in the case I can miss some events which can provide more information or just the flow is wrong, but the fact the interstitial seems to be prepared and then not shown is driving me quite confused.
Cheers!!!!


dotnet release notes for the 2024-05 release

This is an automated pull request to gather the dotnet release notes for the 2024-05 release. See Producing Release Notes for details of the processing.

/azp run azure-sdk - ci


nmstatectl: Fix the support of multiple state files

Old code return on first state file, but the test case
test_set_command_with_two_states indicate it should support multiple
state files.
Fixed by only return when failure when iterating state files.
Test case updated to reveal this problem.

The reason only master tier2 is failing:
With NM master, we support querying additional linux bridge options including the read only gc-timer which keep changing between queries which cause the verification failure after rollback.
Then it reveal the left over linux-br0 interface.

This is high priority.
Once we switch to NM 1.26, the failure will block our CI.


Support className to make styled-components usage easier

Feature: What is your use case for such a feature?
You can style directly third party components with styled-components this way:
import React from 'react';
import styled from 'styled-components';
import {Hits} from 'react-instantsearch';

const StyledHits = styled(Hits)`
  border: 1px solid black;
  background: red;

  .hit {
    display: block;
    border: 1px solid green;
  }
`;

[...]

return (


Engine crash when GDScript "is" keyword is tested against a variable containing a string

Godot version
3.4.stable.flathub.206ba70f4
System information
Ubuntu 21.04 x64, Intel(R) Core(TM) i5-3317U CPU @ 1.70GHz, Mesa 21.0.3, GLES3
Issue description
The GDScript is keyword is supposed to be tested against either a native class name or GDScript instance. If, however, a variable would hold something other than a reference to either of these then the engine crashes silently. This is the resulting stack trace:
Running: /app/bin/godot-bin --path /home/niklas/Projects/TestProject --remote-debug <IP_ADDRESS>:6007 --allow_focus_steal_pid 17 --position 288,150
Godot Engine v3.4.stable.flathub.206ba70f4 - https://godotengine.org
OpenGL ES 3.0 Renderer: Mesa Intel(R) HD Graphics 4000 (IVB GT2)
OpenGL ES Batching: ON

Doesn't crash if code below is commented out.

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v3.4.stable.flathub (206ba70f4b2a3eac4455bb21bd7c8af76285bd86)
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /usr/lib/x86_64-linux-gnu/libc.so.6(+0x39530) [0x7f37b3a75530] ()
[2] /app/bin/godot-bin() [0xabb0c0] ()
[3] /app/bin/godot-bin() [0xa3beb5] ()
[4] /app/bin/godot-bin() [0x9fe449] ()
[5] /app/bin/godot-bin() [0x1cec9e8] ()
[6] /app/bin/godot-bin() [0x1cec95d] ()
[7] /app/bin/godot-bin() [0x1d1bca7] ()
[8] /app/bin/godot-bin() [0x1d38ebf] ()
[9] /app/bin/godot-bin() [0x94d19a] ()
[10] /usr/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0) [0x7f37b3a5fb90] ()
[11] /app/bin/godot-bin() [0x9612ce] ()
-- END OF BACKTRACE --
================================================================

Steps to reproduce
I've been able to reproduce this consistently using the code included under "minimal reproduction project", simply attach it to a node and it should trigger the crash. The first print call is visible in the terminal but not in the editor output pane.
Minimal reproduction project
extends Spatial

func _ready():
	var test_class = Spatial

	if self is test_class:
		print("Doesn't crash if code below is commented out.")

	# If this below is uncommented then the code above doesn't even run.
	test_class = "Spatial"

	if self is test_class:
		print("Crashes without a message and no warning in editor.")

Sorry about that! Thanks for confirming the issue and providing one!

The issue comes from casting a variant pointer to an Object pointer.
The faulty line is at:
https://github.com/godotengine/godot/blob/eef08ea3e06a3c4ca6fefb63a7bab113b42a3265/modules/gdscript/gdscript_function.cpp#L485
obj_B is a null pointer that later gets derefenced at:
https://github.com/godotengine/godot/blob/eef08ea3e06a3c4ca6fefb63a7bab113b42a3265/modules/gdscript/gdscript_function.cpp#L512
causing the crash if DEBUG_ENABLED is true.
Otherwise, obj_B will make the nc pointer null at line:
https://github.com/godotengine/godot/blob/eef08ea3e06a3c4ca6fefb63a7bab113b42a3265/modules/gdscript/gdscript_function.cpp#L508
which will get dereferenced at line:
https://github.com/godotengine/godot/blob/eef08ea3e06a3c4ca6fefb63a7bab113b42a3265/modules/gdscript/gdscript_function.cpp#L516
causing a crash due to a null pointer dereference.

Fixed by #56791.


Add @preferredOutputFormat directive.

@preferredOutputFormat rez;
@preferredOutputFormat ndat;
@preferredOutputFormat kestrel;

This is now effectively implemented, but has taken a different form ultimately.
@require classic;
@require extended;
@require rez;

The reasoning is that the format requirement should be provided in KDL if necessary for certain features, and the actual format should be specified, rather than a potential use case.
The user should also be expected to specify the desired format on the command line using the --format option, with the same types specified. If the option is not specified, then KDL will take it best assumption (based on the requirements specified in KDL type files)


Improve a11y of the locale chooser

What: see #184
Why: To make it accessible
How:

Highlighting: re-added outline on button, changed background to the link instead of the list item
Aria tags: added aria-labels to Links and also alt to flag icons
Close on click outside: used componentDidMount and componentWillUnmount to bind a click event to the document. Maybe there are some improvements here.

Note:
Also changed getLocaleAndHost to getHost, because the locale wasn't needed anymore (was a left over from #177)

Thanks for the review and comments @Anujan!

This is looking great. A few minor comments that I think could improve this even further.
Perhaps I got this wrong since the button seems a bit funky. However I'd add a label to the button that opens the selector with something like "Change language localization" or something similar.
Upon opening the selector, focus either the list of the first element (with focus) because the focus remains within the button.
Span within the list element should probably have aria-hidden="true" so it's not read.  The anchor already has the correct label: Change localization to English reading English would be redundant.

Close with ESC would be a nice touch. Return focus to button after that.

Great feedback! Much appreciated!
If you notice anything else on the site, feedback would likewise be appreciated!
https://rc.glamorous.rocks

Navigating through the menu using the keyboard I kinda expected the menu to close when pressing Escape. ¯\_(ツ)_/¯

@kentcdodds can you put this somewhere for testing?
It appears that you're intending this to behave as a menu. If that's the case, you'll want to adhere to
https://www.w3.org/TR/wai-aria-practices-1.1/#menu
Pay close attention to the keyboard interaction patterns defined.  The version of this menu that is currently on the site does not adhere to the necessary patterns.

Thanks for the feedback everyone!
@karlgroves Once I corrected all of the stuff, I'll try to spin up a testing environment with now.

I think I changed most of the remarks. Thanks for the feedback!

removed alt tags on the flag svg's
added aria-label on the button
added lang on anchor
added aria-hidden on div around svg and display span, is this OK? (comments were that the SVG flag needed to have it)
escape closes the locale picker and focuses the button
refactored the click outside
when opening the locale picker, the first item is being opened. Can this be reviewed please, I think there must be a better way of doing it...

@karlgroves When trying to set up the test environment the build process kept spitting out errors. I don't know why at the moment, but I'll take a look at it tomorrow if its still needed.

I added a script for building and deploying. See the CONTRIBUTING.md for instructions 👍

@kentcdodds I'm not able to build.
First had an issue with integrations/index (cannot read property 'type' of undefined @ babel-generator\lib\whispace.js:50:19).
And now I'm getting the error Error: EEXIST, file already exists C:\...\glamorous-website\out\en\_next\0d3594bb-ff23-4d69-8c1a-9f70af64ff30\page\_document\index.js     at fsError (C:\...\glamorous-website\node_modules\recursive-copy\lib\copy.js:377:15)     at C:\...\glamorous-website\node_modules\recursive-copy\lib\copy.js:252:16     at <anonymous>.
Tried reinstalling everything in a clean directory but without any success, any thoughts?

Hmmm... That's pretty weird. Ummm.... Yeah, I'm not sure what to try, sorry 😕
I'll try to pull your branch and deploy it myself later.

Here it is: https://en-glamorous-hhttuorgyn.now.sh
I've rebased on master as well...
I think that choosing a language and redirecting is broken.. 🤔
Thanks!

This will still need some work.
Here is the markup for a menu on the Tenon.io website:

	<button aria-haspopup="true" aria-owns="myAccountMenu" aria-label="Open Account menu" aria-expanded="true">

	</button>

			Dashboard

			Admin

			Profile

			Notifications

			Logout

```

Note the menu/ menuitem roles, etc.

Also, take special care to read the Keyboard Interaction section of this:
https://www.w3.org/TR/wai-aria-practices-1.1/#menu

@kentcdodds Do I take the localization of the menu bar (with the input of @karlgroves) in this PR?

Is there a reason why the locale picker is being shown at the top of the side menu?
Because on the homepage and on the mobile menu its being shown as the last item.
If it has to be shown at the top, when how does it have to behave when being opened (over the other menu items, other menu items need to be pushed to the bottom, ....)?
I changed it locally to see how it would look like if it was the last menu item:

At first, I thought it was better at the top, but now I think it's better at the bottom so I'm good with your changes

Going to push the changes to move it to the bottom.
Tested it quickly because I have to go. I'll test it thoroughly tomorrow.

I think this is ready for another review.
The menu recommendations from @karlgroves deserves an own PR in my opinion.
@kentcdodds For the snapshot testing I used mount instead of render because with the render method the list items in the locale selector weren't in the snapshot.


Create card component that supports logos/icons/images

Describe the feature
Currently, the card components in Figma do not allow you to add logos/icons/images to them without detaching the component from the design system. Is it possible to add this ability? Thanks!

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate requests

Hello,
Thanks for reaching out to us. I added this ticket to our board for further investigation.
Thanks,
Anna

This is a great request; we will be updating the library later this year and we plan to include it then.

We are releasing a new Figma library in Q2 which is a big upgrade from the existing one. We plan to add a placeholder component to the cards so that you can put anything you want in there. I'll close this for now, but feel free to reach back out if the new library doesn't meet your needs.


thread 'tokio-runtime-worker' panicked at 'attempt to multiply with overflow'

I get a lot of these when running the eth-portal bridge client:
thread 'tokio-runtime-worker' panicked at 'attempt to multiply with overflow', trin-core/src/utp/stream.rs:849:9

Unclear if this is trin <> trin or cross client behavior.

I could fix this and refactor the utp stream implementation to use tokio::time::Duration across the board, instead of the current somewhat bespoke implementation. @ogenev objections? :)

No objections, feel free to open a PR with anything you feel suitable. A lot of things can be improved in utp 😄

@erwanor Any progress on this? I'm still seeing a lot of these errors occurring. If not, no worries at all, we have plenty of other issues to tackle, hahaha. And in the meantime, maybe @ogenev can knock this one out

I'm still seeing a lot of these errors occurring.

Just curious, do you use the latest master (with the FindContent uTP fix)?

@ogenev Yup, it's still occurring on the latest master for me, also seeing a lot of WARN trin_core::utp::stream: Received STATE packet for an unknown active uTP stream: Packet {} errors

Admittedly, I'm not intimately familiar with the utp spec, but it seems like this could be the source of the issue?
https://github.com/ethereum/trin/blob/master/trin-core/src/utp/stream.rs#L878
As it stands now, the congestion time is squared every time this method is hit, causing it to rise exponentially and eventually overflowing the multiplication buffer. Is it supposed to be squared every time? Or should it just be doubled?

As it stands now, the congestion time is squared every time this method is hit, causing it to rise exponentially and eventually overflowing the multiplication buffer. Maybe it would be better to simply increment it with a constant value - like the MIN_CONGESTION_TIMEOUT?

Yes, incrementing with MIN_CONGESTION_TIMEOUT and saturating_add should solve the overflow panic which is the quickest fix.
However, why is this method called multiple times in a row?
We call this method every time we send a uTP packet and don't receive a response from the remote socket. This is part of the resend uTP logic which works as follows:

We send uTP packet.
We listen on the uTP socket for a response.
If we don't get a response by a set time of a timeout, we resend the uTP packet.

The problem is that we don't have a resend limit, for example, Fluffy tries to resend a packet 4 times and then gives up but we keep resending the packet forever :face_exhaling:.
So a more fundamental fix to the problem would be to implement a resend limit on lost uTP packets.


CUDA error when using Mamba2 with long context

Hi, I am benchmarking inference speed on long sequences and encountering CUDA-related errors specifically with the Mamba2 models at longer sequence lengths (>200k). This issue does not occur with Mamba1 models.
For example running:
python benchmark_generation_mamba_simple.py --model-name "state-spaces/mamba2-1.3b" --promptlen 300000 --genlen 1

produces the error:
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/utils/generation.py", line 260, in generate
output = decode(
File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/utils/generation.py", line 221, in decode
scores.append(get_logits(sequences[-1], inference_params))
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/utils/generation.py", line 184, in get_logits
logits = model(
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1520, in _call_impl
return forward_call(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/models/mixer_seq_simple.py", line 279, in forward
hidden_states = self.backbone(input_ids, inference_params=inference_params, **mixer_kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1520, in _call_impl
return forward_call(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/models/mixer_seq_simple.py", line 194, in forward
hidden_states, residual = layer(
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1520, in _call_impl
return forward_call(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/modules/block.py", line 67, in forward
hidden_states = self.mixer(hidden_states, inference_params=inference_params, **mixer_kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1511, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1520, in _call_impl
return forward_call(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/modules/mamba2.py", line 242, in forward
y = mamba_chunk_scan_combined(
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/ops/triton/ssd_combined.py", line 581, in mamba_chunk_scan_combined
return MambaChunkScanCombinedFn.apply(x, dt, A, B, C, chunk_size, D, z, dt_bias, initial_states, seq_idx, cu_seqlens, dt_softplus, dt_limit, return_final_states, return_varlen_states)
File "/usr/local/lib/python3.10/dist-packages/torch/autograd/function.py", line 553, in apply
return super().apply(*args, **kwargs)  # type: ignore[misc]
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/ops/triton/ssd_combined.py", line 540, in forward
out, out_x, dt_out, dA_cumsum, states, final_states, *rest = _mamba_chunk_scan_combined_fwd(x, dt, A, B, C, chunk_size, D=D, z=z, dt_bias=dt_bias, initial_states=initial_states, seq_idx=seq_idx, cu_seqlens=cu_seqlens, dt_softplus=dt_softplus, dt_limit=dt_limit)
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/ops/triton/ssd_combined.py", line 312, in _mamba_chunk_scan_combined_fwd
dA_cumsum, dt = _chunk_cumsum_fwd(dt, A, chunk_size, dt_bias=dt_bias, dt_softplus=dt_softplus, dt_limit=dt_limit)
File "/usr/local/lib/python3.10/dist-packages/mamba_ssm/ops/triton/ssd_chunk_state.py", line 675, in _chunk_cumsum_fwd
_chunk_cumsum_fwd_kernel[grid_chunk_cs](
File "/usr/local/lib/python3.10/dist-packages/triton/runtime/jit.py", line 326, in
return lambda *args, **kwargs: self.run(grid=grid, warmup=False, *args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/triton/runtime/autotuner.py", line 156, in run
timings = {config: self._bench(*args, config=config, **kwargs) for config in pruned_configs}
File "/usr/local/lib/python3.10/dist-packages/triton/runtime/autotuner.py", line 156, in
timings = {config: self._bench(*args, config=config, **kwargs) for config in pruned_configs}
File "/usr/local/lib/python3.10/dist-packages/triton/runtime/autotuner.py", line 133, in _bench
return do_bench(kernel_call, warmup=self.num_warmups, rep=self.num_reps, quantiles=(0.5, 0.2, 0.8))
File "/usr/local/lib/python3.10/dist-packages/triton/testing.py", line 108, in do_bench
torch.cuda.synchronize()
File "/usr/local/lib/python3.10/dist-packages/torch/cuda/init.py", line 801, in synchronize
return torch._C._cuda_synchronize()
RuntimeError: CUDA error: an illegal memory access was encountered
This issue seems to only occur with Mamba2 models and is present across models of all different sizes. Mamba1 however works well and i am able to do inference on prompt lengths of up to 1m on the 1.4b model.
I am using a single h100 (80gb) card.
Thanks!

Are any tensors of size >= 2GB?
We use int32 for indexing, it's possible that it wraps around the max of int32 and produce negative index, causing IMA.

Yes i have several intermediate output tensors exceeding 2GB in size. However this is also true for lower context lengths that do not produce the error. For example, context length 250k which runs fine has almost all layer outputs exceeding 2GB but it runs fine.
Noted on the indexing causing the issue, is one solution to lower the models dimension?

I think I found the problem, I have submitted a PR.

+1.

Try the fix I did in the PR, it's 4 lines of code change, lmk if that works?

+1. I tried the fix in the PR, not working either.

I found this is related to triton issue. Modify tl.program_id(*) to tl.program_id(*).to(tl.int64) can skip this error: https://github.com/triton-lang/triton/issues/1058.


Don't clear invite counts on resetting store

Fixes invite count disappearing on switching users. cc @keybase/y2ksquad

Probably superseded by #23263


Support cached event handlers with varies (custom headers)

Context: https://github.com/unjs/nitro/discussions/1023#discussioncomment-5226630
By default, we drop all incoming request headers from being passed to the cached event handler. This is to ensure cache is not leaked and users get the same response.
However, a valid usecase might be actually intentionally supporting multiple cached responses based on headers (language, variant, multi domain, etc). We might support an opt-in feature varies: [host, x-domain] that creates cache segments for each variant and using hashed values to map.
Related/Similar spec to inspire: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Vary

This feature would be very helpful, our nuxt app is deployed on different domains with different locales and we would like to use routeRules for caching.
I would like to help to implement it. Do i just have to extend the getKey method? Should i also add the headers which are defined in varies to the request? Is there something else to consider?


Example for COPY INTO with MSI

I would like to get this example added for COPY INTO using MSI. Customer has tried to go through our documentation, and was unable to find the exact syntax to have copy only use MSI for credentialing.  Worked with Synapse Team (Artur Vieira<EMAIL_ADDRESS>provided this example) to get code to work with the customer.
F. Load using MSI credentials
Set the Synapse SQL to use “-AssignIdentity”
Set-AzSqlServer -ResourceGroupName $resourcegroupname -ServerName $servername -AssignIdentity
Create a Scoped credential on your Database
CREATE DATABASE SCOPED CREDENTIAL msi_cred WITH IDENTITY = 'Managed Service Identity';
COPY INTO dbo.myCOPYDemoTable
FROM 'https://storlabarturv.blob.core.windows.net/test/bcpDemo.txt'
WITH (
FILE_TYPE = 'CSV',

CREDENTIAL = (IDENTITY = 'Managed Identity'),

FIELDQUOTE = '"',

FIELDTERMINATOR=','

)

@thesqlpro : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Any updates on this?

@MightyPen Could you or someone on your team review this pull request? Thanks.

Thanks for providing this example! We don't need scoped credential for COPY:
Create a Scoped credential on your Database
CREATE DATABASE SCOPED CREDENTIAL msi_cred WITH IDENTITY = 'Managed Service Identity';
Other than that, it looks good.

@julieMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#assign:julieMSFT

#sign-off

@julieMSFT: I'm sorry - only the author of this article, @kevinvngo, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ktoliver HI Kristine, I spoke with Kevin and updated the PR. This can be merged.

Thanks, all


Drop Python 2 support

Currently the build is failing due to Python 2 incompatibility. I can patch it, but my plan was to make the current release (2.30) the last one supporting Python 2. But it might be easier just to drop it, since this has already happened with matplotlib, scipy, numpy, etc.

Either option is fine with me.
As of last week, conda-forge has officially deprecated Python 2.7.  I think the next time we "rerender" this package, Python 2.7 support will be dropped automatically, unless we explicitly add it back.  It would be possible to maintain a Python 2 build for just a little longer, since existing packages will not be deleted (just not maintained).  Whatever you prefer is fine with me.

As a separate matter: The typical practice for maintaining conda-forge feedstocks is to avoid pushing to the master branch directly.  Instead, we always open a PR, even for relatively trivial changes.  (I have been gently scolded for this in the past, so now I've fallen in line 😄 )
So, do you mind if I open a new PR for the 2.30 upgrade?  (I'll start by reverting the master branch via force-push, then I'll use your recent commits as the basis for the PR.) We can discuss build issues in that thread. We'll also be able to make requests to the conda-forge admin bot, if necessary.

Ok, so let's just drop python 2. It's a good idea anyway.
I'm sorry for pushing to master directly, I did it by mistake. I was using the github interface, and after a few revisions I pressed the wrong button in the end... I'll be more careful next time.
Please feel free to revert and open a PR.

Please feel free to revert and open a PR.

Done!

I'm sorry for pushing to master directly, I did it by mistake.

Ah, sorry for the lecture, then.  But while I'm at it, I'll point out a non-obvious conda-forge rule: they ask us not to create temporary branches on the main repo -- we are supposed to use our own fork as the basis for all PRs.  The only permissible branches are the master branch, and any long-term maintenance branches we wish to be built built (and published) by the conda-forge CI.
Accordingly, I've deleted today's temporary branches from this repo.

Ok, thanks for the info. This was my first contribution to conda-forge ever, and did not bother to read anything, so my bad.


Block List fields are displayed backwards

I expected sleep 10 first, then echo hello second:

The dropdown and results page show it properly though:

@zhen0 I see you closed then reopened. I think this is fixed? Would you mind verifying when you get a chance. Absolute lowest priority.

Pretty confident this is fixed. Closing because of inactivity.


Firefox segfault when the hardened profile activated

Describe the bug
Firefox segfault on launch when using the hardened profile.
To Reproduce
Steps to reproduce the behavior:

Install firefox
Activate the hardened profile
launch firefox

Notify maintainers
@taku0
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.59, NixOS, 20.09pre-git (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.7`
 - channels(root): `"nixos, nixos-hardware"`
 - channels(j4m3s): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixos modules affected by the problem
module:
  firefox-bin

This might be a duplicate of #97898

Indeed, closing.


active low reset

can openfpga fabric has active low reset?
I have a benchmark it is active low reset and I ran on k4n4 but the no correct value is showing on the out_gfpga.
is there a pin assignment or anything that can let the fpga fabric know this is a active low reset?
Thanks.

I got it, it's in the openfpga_arch file.


Timezone changes not being reflected correctly

I am facing an issue while using moment in Cefsharp browser on a Cordova hybrid application on Windows. On changing the device timezone, moment does not respect the change while converting to UTC.
Moment is loaded on IST Indian Standard Time (which is UTC+0530). When I run the below test, it works fine.
test(‘utc value conversion’, () => {
expect(
moment("2019-03-23 07:30:00").utc().format("YYYY-MM-DD HH:mm:ss")
).toBe("2019-03-23 02:00:00"));
});
Expected 2019-03-23 02:00:00
Received 2019-03-23 02:00:00
When the device timezone is changed to Myanmar UTC+0630 - so now the date value in local time becomes 26/3/2019 08:30 am.
test(‘utc value conversion’, () => {
expect(
moment("2019-03-23 08:30:00").utc().format("YYYY-MM-DD HH:mm:ss")
).toBe("2019-03-23 02:00:00"));
});
Expected 2019-03-23 02:00:00
Received 2019-03-23 03:00:00
The same code works fine while using moment in Safari browser on the same Cordova hybrid application on iOS.

Hi,
Can we please have an update on this?

This is a great question for Stack Overflow.


feat(doc): edgex doc update for EdgeX Ireland

Jiyong, the docs in EdgeX Foundry should be updated as well
https://github.com/edgexfoundry/edgex-docs/blob/master/docs_src/microservices/support/Kuiper/Ch-Kuiper.md


Error with Angular 15

./node_modules/frappe-gantt/src/gantt.scss?ngGlobalStyle:1:0 - Error: Module parse failed: Unexpected token (1:0)
File was processed with these loaders:

./node_modules/resolve-url-loader/index.js
./node_modules/sass-loader/dist/cjs.js
You may need an additional loader to handle the result of these loaders.

.gantt .grid-background {
|   fill: none;
| }

With Angular 14 work with it: https://github.com/frappe/gantt/issues/203#issuecomment-1315502431

I have the same issue as well.
As a workaround you can go to node_modules/frappe-gantt/src/index.js and comment out "import './gantt.scss';" on line 7.
Then import the styles in your angular.json file.

this is because package.json has wrong configuration ("main": "src/index.js",), try to import directly from dist js and css (via css imports - styles or in .scss)
import Gantt from 'frappe-gantt/dist/frappe-gantt';

Thanks you @mmospanenko, i have a folk that work! https://github.com/alvarosinmarca/ngx-gantt

Loading from dist doesnt seem to work for me.
Is there any other work around or another way to fix this?

Loading from dist doesnt seem to work for me. Is there any other work around or another way to fix this?

Comment out line 7 in the node_modules/frappe-gantt/src/index.js, which says "import './gantt.scss';"
Alternatively this package is forked off of the frappe-gantt package and the import gantt.scss is already commented out. frappe-gantt-angular15

Loading from dist doesnt seem to work for me. Is there any other work around or another way to fix this?

Comment out line 7 in the node_modules/frappe-gantt/src/index.js, which says "import './gantt.scss';"
Alternatively this package is forked off of the frappe-gantt package and the import gantt.scss is already commented out. frappe-gantt-angular15

Thank you!


Invalid JSON chars in text strings crash

If a show has a character that is not a valid JSON character, for example:
text: Brooks & Dunn

It will throw a JSON decode error when it's decoded by the MC from the JSON encoding of BCP.
Also this doesn't work:
text: "Brooks & Dunn"

We should figure out how to identify and safely encode these.

urlencode/decode?

Does this still exist?

Reopen if you experience this


GEODE-5629: Changing GMSEncryptJUnitTest to use ConcurrencyRule

This test was not capturing exceptions from the background threads.
Using ConcurrenyRule to execute the background tests.

@balesh2

I just noticed that LoginHandlerInterceptorJUnitTest failed. I think that's a JMock test that was just recently changed.


fix build error while cross compiling linux x64 gnu from mac aarch64

I'm using cargo-zigbuild to cross compile x86_64-unknown-linux-gnu from an aarch64 mac (mac m1). Doing so hits this error:
   Compiling netstat2 v0.9.1 (/Users/kurt/work/netstat2-rs)
warning: `tcp_info` is ambiguous
   --> src/integrations/linux/netlink_iterator.rs:192:53
    |
192 |             let tcpi = &*(RTA_DATA!(attr) as *const tcp_info);
    |                                                     ^^^^^^^^ ambiguous name
    |
    = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
    = note: for more information, see issue #114095 <https://github.com/rust-lang/rust/issues/114095>
    = note: ambiguous because of multiple glob imports of a name in the same module
note: `tcp_info` could refer to the struct imported here
   --> src/integrations/linux/netlink_iterator.rs:4:5
    |
4   | use libc::*;
    |     ^^^^^^^
    = help: consider adding an explicit import of `tcp_info` to disambiguate
note: `tcp_info` could also refer to the struct imported here
   --> src/integrations/linux/netlink_iterator.rs:1:5
    |
1   | use crate::integrations::linux::ffi::*;
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = help: consider adding an explicit import of `tcp_info` to disambiguate
    = note: `#[warn(ambiguous_glob_imports)]` on by default

error[E0609]: no field `state` on type `&libc::tcp_info`
   --> src/integrations/linux/netlink_iterator.rs:193:40
    |
193 |             return TcpState::from(tcpi.state);
    |                                        ^^^^^ unknown field
    |
    = note: available fields are: `tcpi_state`, `tcpi_ca_state`, `tcpi_retransmits`, `tcpi_probes`, `tcpi_backoff` ... and 26 others

For more information about this error, try `rustc --explain E0609`.
warning: `netstat2` (lib) generated 1 warning
error: could not compile `netstat2` (lib) due to 1 previous error; 1 warning emitted

The latest libc (0.2.160) added new tcp structs to the namespace, so the use libc::* import shadows the linux::ffi::* imports.
This PR is a minimal change to fix the build issue, as the explicit import from linux::ffi::* prevents the collision from ambiguous * imports.
It might be smart to only use explicit imports to prevent breakage in the future, but it's probably extremely rare that new structs are added, and there are a lot of imports. Alternatively, maybe the ffi struct can be deleted, now that libc exports a similar struct.
I previously filed a bug with libc before realizing the root cause: https://github.com/rust-lang/libc/issues/3987

Thanks @kurtbuilds, thanks for the detailed writeup 🥳🐛
Seems like this issue also makes the CI fail for the regular builds - I'd be happy to merge this if you can make everything pass.
There also seems to be a lot of bad warnings / errors - seems like a little 🧹  is overdue anyway.

These appear to be all test issues, rather than library issues.
Are you willing to bump the rust version? Your CI runs 1.69. A bunch of test compilation bugs are assertions about alignment of fields within structs, and rust 1.77 introduced offset_of, which would be the simplest way to fix those tests.

When will this PR be merged? I have the same problem. cc @ohadravid @kurtbuilds

cargo-zigbuild should work now @kurtbuilds 😄

Added a zigbuild step to the CI in #19, so closing this for now 😄

Confirmed the new version works in our build process! Thank you!


example?

hey, anyone got an example?
I don't have a .terminal.customColorSchemes[] in my config

I've never used Tabby, but I'd assume normal YAML syntax?
terminal:
  customColorSchemes:
  - name: Catppuccin Mocha
    foreground: HSL(226, 64%, 88%)
    background: HSL(240, 21%, 15%)
    selection: null
    cursor: HSL(10, 56%, 91%)
    cursorAccent: null
    colors:
      - HSL(234, 13%, 31%)
      - HSL(343, 81%, 75%)
      - HSL(115, 54%, 76%)
      - HSL(41, 86%, 83%)
      - HSL(217, 92%, 76%)
      - HSL(316, 72%, 86%)
      - HSL(189, 71%, 73%)
      - HSL(226, 64%, 88%)
      - HSL(234, 13%, 36%)
      - HSL(343, 81%, 80%)
      - HSL(115, 54%, 81%)
      - HSL(41, 86%, 88%)
      - HSL(217, 92%, 81%)
      - HSL(316, 72%, 91%)
      - HSL(189, 71%, 78%)
      - HSL(226, 64%, 93%)

i have it like this:
  customColorSchemes:     - name: Catppuccin Mocha       foreground: HSL(226, 64%, 88%)       background: HSL(240, 21%, 15%)       selection: null       cursor: HSL(10, 56%, 91%)       cursorAccent: null       colors:         - HSL(234, 13%, 31%)         - HSL(343, 81%, 75%)         - HSL(115, 54%, 76%)         - HSL(41, 86%, 83%)         - HSL(217, 92%, 76%)         - HSL(316, 72%, 86%)         - HSL(189, 71%, 73%)         - HSL(226, 64%, 88%)         - HSL(234, 13%, 36%)         - HSL(343, 81%, 80%)         - HSL(115, 54%, 81%)         - HSL(41, 86%, 88%)         - HSL(217, 92%, 81%)         - HSL(316, 72%, 91%)         - HSL(189, 71%, 78%)         - HSL(226, 64%, 93%)
but tabby always overwrites this, it won't let me select this theme


some checks fail on void linux

I was looking at the package on Void linux. I made some small changes (latest version pending a go update):
 # Template file for 'gopass'
 pkgname=gopass
-version=1.15.11
+version=1.15.12
 revision=1
 build_style=go
-build_helper=qemu
 go_import_path=github.com/gopasspw/gopass
 go_package="${go_import_path} ${go_import_path}/cmd/..."
-makedepends="gnupg"
 depends="gnupg>=2 git"
+checkdepends="gnupg git age"
 short_desc="Slightly more awesome standard unix password manager for teams"
 maintainer="Felipe Nogueira<EMAIL_ADDRESS> license="MIT"
 homepage="https://www.gopass.pw/"
 changelog="https://raw.githubusercontent.com/gopasspw/gopass/master/CHANGELOG.md"
 distfiles="https://github.com/gopasspw/gopass/archive/v${version}.tar.gz"
-checksum=f85610a4f114125bd21e1100d6a2970c7ab76f09a7e094aa6be378018979eb56
+checksum=803d779bc55aab3c1e0ade85612f287b3007cd5316a7a571314e08b015a95426
+
+do_check() {
+       go test -v ./...
+}

and wanted to also run tests. Right now some are failing (and some are skipped):
=== RUN   TestInvalidYAML
    yaml_test.go:63: Failed to stat GOPASS_BINARY : stat : no such file or directory
--- SKIP: TestInvalidYAML (0.00s)
=== CONT  TestListRegressions1628
    list_test.go:60: Failed to stat GOPASS_BINARY : stat : no such file or directory
--- SKIP: TestListRegressions1628 (0.00s)
PASS
ok  	github.com/gopasspw/gopass/tests	(cached)
=== RUN   TestPubring
=== PAUSE TestPubring
=== CONT  TestPubring
--- PASS: TestPubring (0.01s)
PASS
ok  	github.com/gopasspw/gopass/tests/can	(cached)
FAIL
=> ERROR: gopass-1.15.12_1: do_check: 'go test -v ./...' exited with 1
=> ERROR:   in do_check() at srcpkgs/gopass/template:19

Does anyone know how to fix these?

If I only do certain tests, then no failure:
do_check() {
GOPASS_BINARY=${GOPATH}/bin/gopass go test -v ./tests
}

What are the suggested tests to be run in the build chroot? only the tests folder (passing), or the full ./... (failing)?


Tweaked c++ side naming

Summary of the Pull Request
Made similar tweaks on the C++ side.

Can we use the following two elements from the resources file instead writing the text into C++ code like @crutkas proposed in the main PR?

KeyboardManager_RemapKeyboardHeader.Text
KeyboardManager_RemapShortcutHeader.Text

As far as I know that's not possible since those are in resw files, which C++ doesn't natively support like C# does. One issue we already have tracked is to make sure at least within the C++ side we move all the strings to a single file (.rc is the format used there). I guess a second issue we should make is have some framework where we can use a common file for both C++ and C# projects.
I can create an issue for this, though I don't think we'll be able to get it fixed for 0.19

Then I merge the changes.
And we should open the issue you mentioned. But I think it could already existing.


Custom user model migration fail. SettingsReference object has no attribute '_meta'

ISSUE SUMMARY
When creating an custom user model with an extra field migration fail.
STEPS TO REPRODUCE
As described in Oscar's documention:

Create a user app
Subclass customer.AbstractUser to create the new User model
Add new field
Add AUTH_USER_MODEL = "user.User" settings
Run python manage.py makemigrations

The output should be:
Traceback (most recent call last):   File "manage.py", line 18, in <module>     execute_from_command_line(sys.argv)   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/core/management/__init__.py", line 381, in execute_from_command_line     utility.execute()   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/core/management/__init__.py", line 375, in execute     self.fetch_command(subcommand).run_from_argv(self.argv)   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/core/management/base.py", line 323, in run_from_argv     self.execute(*args, **cmd_options)   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/core/management/base.py", line 364, in execute     output = self.handle(*args, **options)   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/wagtail_modeltranslation/management/commands/makemigrations_translation.py", line 27, in handle     super(Command, self).handle(*args, **options)   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/core/management/base.py", line 83, in wrapped     res = handle_func(*args, **kwargs)   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/core/management/commands/makemigrations.py", line 168, in handle     migration_name=self.migration_name,   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/db/migrations/autodetector.py", line 43, in changes     changes = self._detect_changes(convert_apps, graph)   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/db/migrations/autodetector.py", line 186, in _detect_changes     self.generate_altered_fields()   File "/home/thanasis/projects/naturaldoctor/natural-doctor/virtualenv/lib/python3.6/site-packages/django/db/migrations/autodetector.py", line 919, in generate_altered_fields     new_field.remote_field.model._meta.app_label, AttributeError: 'SettingsReference' object has no attribute '_meta'
TECHNICAL DETAILS
OS: Ubuntu 18.04
Python 3.6.9
Oscar 2.04
Postgres 10

This is not originating from Oscar, but from wagtail-modeltranslation. See https://github.com/infoportugal/wagtail-modeltranslation/issues/240 .


Add support in tmt report --how reportportal add metadata

This pr add support in report --how reportportal --metadata
The following values accepted
metadata: [{ key: key1, value: value2 }, { key: key2, value: value2 }, { value: value3}]

as per this link
https://reportportal.io/docs/dev-guides/ReportingDevelopersGuide/#start-launch

There is additional request to receive the launch_id from launch_uuid
And then update the launch with metadata

resolves [#2062](https://github.com/teemtee/tmt/issues/2062)

#2088 Cover this solution


test: grep versions with --sort

/cc @heyitsanthony @xiang90
Otherwise, it only greps v3.0.9

Semaphore doesn't seem to work with this command. Will fix.

Should work. Think it's just not grepping the v3.0.10 because my branch is outdated.
git tag -l returns only up-to v3.1.0-alpha.0
v3.0.8
v3.0.9
v3.1.0-alpha.0

Manually running that command correctly greps v3.0.10.

lgtm


Added video title and thumbnail url extraction

Hi guys. Thank you for your contributions! Could you add support for meta extraction to the delegate based method also? I'll merge and bump a version with this in place. :wink:

Sure! Also, I forgot a detail... merging this will make existent code using YTVimeoExtractor break because of the new parameters.
How about I remove the parameters I added here, and instead add new methods for getting all the information as a dictionary? That would be best I think.

You are right, we should try to keep backward compatibility. A dedicated method for metadata extraction is definitly better.

Merged df200d5319a5322f28f14f0e609b63f1a61ee1ce


test(concurrency): test finish_execution_during_commit

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.84%. Comparing base (7df2b9f) to head (541bc6d).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1904   +/-   ##
=======================================
  Coverage   77.84%   77.84%
=======================================
  Files          61       61
  Lines        8744     8744
  Branches     8744     8744
=======================================
  Hits         6807     6807
  Misses       1497     1497
  Partials      440      440

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

+reviewer:@meship-starkware


ui, ux?

Hi,
What if the graphical interface was something like this https://pg.datawan.id/?

img 1
img 2

Best Regards.

@stokebreakup Im not sure which part of the UI you're referring to. Can you elaborate?

If you have a theme that'll with accessibility, please submit a PR. Right now there's no way of customizing the themes, but i feel like a basic UI customization feature should do the trick.


[BEAM-13650][Playground] Add link for examples

[BEAM-13650]
Add link for examples.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @KhaninArtur @ilya-kozyrev

Don't we need also to read this link and return to the frontend?

It's already done.

LGTM!


Actions are no longer experimental, so enable them by default

This PR makes the actions enabled by default, so people will find it easier to enable actions in repository setting.

I'm not sure if this should be enabled by default. You still need a runner, so this will be not working by default.

I'm not sure if this should be enabled by default. You still need a runner, so this will be not working by default.

Yes, but this change will reduce people asking why they cannot get the actions feature.

Should we add it as a default unit to repo_units too?

I'm also not convinced it should be enabled by default, considering it takes additional setup to get it working.
What about instead adding it to the install page? It's getting a little big, but overall I think that might make more sense.

I think it may instead increase the amount of "why aren't actions working" questions if they are enabled but have no runners by default.

In that case we could add a warning to repo owners with actions pending that they need to add runners?

I think it may instead increase the amount of "why aren't actions working" questions if they are enabled but have no runners by default.

In that case we could add a warning to repo owners with actions pending that they need to add runners?

I think that may be a worthwhile change in itself, but it seems like of the two main install types:

Users who use the install page, a checkbox for enabling actions along with a link to "how to set up runners" may be a good solution, or
Users who deploy with a ready config should likely be reading and understanding the configuration for actions/runners. Anyone in this category would also already have them enabled and so unaffected by this PR, but anyone who doesn't have them enabled will likely want to review before it is turned on for them.

It just seems like flipping the toggle from false to true doesn't improve the situation as a default, it just moves the work elsewhere.
As another alternative, a db-setting may work fine here as well, so it becomes easier to just go flip the switch when the admin is ready.

And update app.example. ini as usual.
I guess I don't mind it enabled by default if it does nothing. What we should eventually do is enable it automatically in a repo as soon as any workflow files are detected in the repo and at least 1 worker is registered. This will eventually require that we also enable it by default per-repo.

Should we add it as a default unit to repo_units too?

I think that may have a bigger impact for users which can be leave to other PRs. This PR will not affect anything, the actions tab will not be enabled except they enable it in repository setting.

I'm also not convinced it should be enabled by default, considering it takes additional setup to get it working. What about instead adding it to the install page? It's getting a little big, but overall I think that might make more sense.

Yes, but this change will reduce people asking why they cannot get the actions feature.

I think it may instead increase the amount of "why aren't actions working" questions if they are enabled but have no runners by default.

This global configuration will not enable repository actions by default. In fact, after this PR, every repository still needs to enable actions in the setting.


Discarding staged files is not possible

In the status view I'm not able to discard staged files with the d-key. For unstaged files this works perfectly.

This was intentional - discarding a staged file seems ambiguous, especially if that file is partially staged.
For example, let's say half of file.ext is staged, I select file.ext in the unstaged section, and then I hit d.  The outcome would be as expected - the staged portion would remain staged, and the file in the working directory would revert to HEAD.  file.ext in the index would be unchanged.
However, if instead I select file.ext in the staged section and hit d, the outcome is not clear.  Would the file be unstaged?  It would have to be, if the action did anything.  But it would have to do more than simply unstage, so it would perhaps unstage and then revert the file in the working directory to HEAD.  However, in that scenario, the action taken effects the file both in the index and in the working directory, which (in my mind) is not the expected outcome of selecting the file in the index and performing an action.
I'm open to discussing it, however - so if you have additional thoughts or ideas of what d should do to staged files, please let me know.


prisma deploy will fail if "Create new database" is selected and Docker is not running

Describe the bug
When not having an endpoint in my prisma.yml and running prisma deploy (or prisma deploy --new when I have an endpoint) and selecting the Create new database option, the CLI walks me through the process of creating a Docker setup for a DB and Prisma server which will be running on http://localhost:4466.
The problem is that after I select the database (MySQL, Postgres or MongoDB), the CLI fails with the error:
$ prisma deploy
? Set up a new Prisma server or deploy to an existing server? Create new database
? What kind of database do you want to deploy to? MySQL

Written endpoint `http://localhost:4466` to prisma.yml

 ▸    Could not connect to server at http://localhost:4466. Please check if your server is running.

Get in touch if you need help: https://www.prisma.io/forum/
To get more detailed output, run $ set -x DEBUG "*"

This is confusing and the flow should be handled more gracefully. I suggest the following:
After I selected the database type, the CLI should:

Check if the Docker CLI is installed

If no -> Fail with error message: You need Docker to run Prisma locally. Please install Docker for your operating system from https://store.docker.com/search?offering=community&type=edition. Find more info in the Prisma docs: https://www.prisma.io/docs/prisma-server/deployment-environments/docker-rty1/

(Docker is installed) Create Docker Compose file on file system
Run docker-compose up -d and print message similar to Launching the Docker containers for Prisma and your database ...
Run prisma deploy

Closing this in favor of https://github.com/prisma/prisma/issues/3578


[TASK] Replacing ET with TreeStructure for xml parsing

Issue Description
Is your feature request related to a problem? Please describe.
The exclusive use of xml.etree.ElementTree (ET) in parsing input files may in the future lead to bugs or limit functionality if other non-xml trees are used for input.
Describe the solution you'd like
Replacement of the ET parsing with the broader TreeStructure module in RAVEN (RAVEN/ravenframework/utils/TreeStructure.py) will limit future difficulties. Not all ET functions are available in TreeStructure, and naming of functions differs, so more than a simple find-and-replace will be necessary. One such ET function, used in FORCE/tests/unit-tests/test_heron.py, that has no counterpart in TreeStructure, is ET.fromstring(). This function parses a string with xml formatting into ET structure and is particularly useful for unit tests.
Describe alternatives you've considered

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[ ] 1. If the issue is a defect, is the defect fixed?
[ ] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[ ] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[ ] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[ ] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

@caleb-sitton-inl I may not understand fully, but from your description, it sounds like we should continue using ET over TreeStructure so we have access to all the functions provided by ET also, it seems to me that we should rather rely on a python native library as much as we can before using an in-house wrapper. Also it appears that ET seems to be more widespread used throughout RAVEN and HERON.
@PaulTalbot-INL @GabrielSoto-INL Thoughts?

We did discuss this a little on a PR. While using ET is a great way to determine how the XML is used, as we look to use EDDI formats as well (with Workbench for .heron files), we may need to consider the more agnostic approach. I agree, for now ET is just fine.


Occasional ThreadLocalRandom failures on Kokoro

Our tests occasionally fail in a puzzling way on Kokoro.  I've appended an extract from a recent test failure on MacOS (note that this test is apparently running JDK 1.8.0_152).
Starting process 'Gradle Test Executor 3'. Working directory: /Volumes/BuildData/tmpfs/src/github/jib/jib-plugins-common Command: /Library/Java/JavaVirtualMachines/jdk1.8.0_152.jdk/Contents/Home/bin/java -Djava.security.manager=worker.org.gradle.process.internal.worker.child.BootstrapSecurityManager -Dorg.gradle.native=false -Dfile.encoding=UTF-8 -Duser.country=US -Duser.language=en -Duser.variant -ea -cp /Volumes/BuildData/tmpfs/tmp/.gradle/caches/4.6/workerMain/gradle-worker.jar worker.org.gradle.process.internal.worker.GradleWorkerMain 'Gradle Test Executor 3'
Successfully started process 'Gradle Test Executor 3'
Gradle Test Executor 3 started executing tests.
com.google.cloud.tools.jib.builder.steps.BuildAndCacheApplicationLayerStepTest > testRun_emptyLayersIgnored STANDARD_ERROR
    SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
    SLF4J: Defaulting to no-operation (NOP) logger implementation
    SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

com.google.cloud.tools.jib.plugins.common.ZipUtilTest > testZipSlipVulnerability_windows SKIPPED
Gradle Test Executor 3 finished executing tests.

com.google.cloud.tools.jib.tar.TarStreamBuilderTest > testToBlob_stringsAndTarArchiveEntries FAILED
    java.lang.NoClassDefFoundError: Could not initialize class java.util.concurrent.ThreadLocalRandom
        at java.util.concurrent.ConcurrentHashMap.fullAddCount(ConcurrentHashMap.java:2526)
        at java.util.concurrent.ConcurrentHashMap.addCount(ConcurrentHashMap.java:2266)
        at java.util.concurrent.ConcurrentHashMap.replaceNode(ConcurrentHashMap.java:1166)
        at java.util.concurrent.ConcurrentHashMap.remove(ConcurrentHashMap.java:1097)
        at org.mockito.internal.util.concurrent.WeakConcurrentMap.expungeStaleEntries(WeakConcurrentMap.java:127)
        at org.mockito.internal.util.concurrent.WeakConcurrentMap$WithInlinedExpunction.containsKey(WeakConcurrentMap.java:260)
        at org.mockito.internal.creation.bytebuddy.MockMethodAdvice.isMock(MockMethodAdvice.java:116)
        at org.mockito.internal.creation.bytebuddy.MockMethodAdvice.isMocked(MockMethodAdvice.java:121)
        at java.io.OutputStream.write(OutputStream.java:75)
        at org.apache.commons.compress.archivers.tar.TarArchiveOutputStream.writeRecord(TarArchiveOutputStream.java:566)
        at org.apache.commons.compress.archivers.tar.TarArchiveOutputStream.putArchiveEntry(TarArchiveOutputStream.java:387)
        at com.google.cloud.tools.jib.tar.TarStreamBuilder.writeEntriesAsTarArchive(TarStreamBuilder.java:50)
        at com.google.cloud.tools.jib.blob.WriterBlob.writeTo(WriterBlob.java:36)
        at com.google.cloud.tools.jib.tar.TarStreamBuilderTest.verifyBlobWithoutCompression(TarStreamBuilderTest.java:193)
        at com.google.cloud.tools.jib.tar.TarStreamBuilderTest.testToBlob_stringsAndTarArchiveEntries(TarStreamBuilderTest.java:78)
306 tests completed, 1 failed, 1 skipped

I hit similar errors a couple of weeks ago, but on Linux, and which magically cleared up.  Unfortunately our build environments seem to be running different versions of the JDK as those tests ran on a JDK 1.8.0_171).

About using different JDK versions, we should talk the Kokoro folks and ask why.
And assuming this is JDK-8150014 as you said, I guess there's nothing much we can do except waiting for JDK >= 9?

Perhaps Kokoro provides knobs for tweaking the versions used?
20 more days to JDK11! http://www.java-countdown.xyz

Haven't seen this for a while.


Linter for Java

CheckStyle
CLang-Formatter

An analysis is required to specify which params would fit initially better

Given CLang-Formatter, it was included an initial configuration for Java


Get command does not only uses local cache

In the readme it is mentioned that using skate sync only reads data from the local cache. But when I do a skate get XXX, a call to my remote charm server is made.
I do not know if this is a bug or if the readme should be changed.

Hi, the data is only synced on write, but on read we're currently making a network call to fetch your encryption keys. The keys are fully encrypted for your ssh public key before being stored on our server, then decrypted on fetch and used to decrypt the BadgerDB. You're correct that the documentation should be clearer though! We also plan to cache the encryption keys locally to make skate and charm kv work in a truly offline, read-only mode. For now, we'll update the docs. Thanks for the catch!

I had to stop using a way to store some local tokens because it introduced seconds of delays to my profile load when pulling several entries. I hope this local caching becomes an option eventually.

yes this is a big blocker to usage anywhere other than on a lan


Reorder structure Dofs into free and fixed dofs blocks

Thanks to PhD.  Jaime Gil a researcher in Structural Analysis helped me to undersetand the order and reorder algo. This can be used to split the tangent matrix into constrained and unconstraind dofs bolcks. Once the assemble process is started then each element can access to the reordered dofs to store the their contribution. Then once the displcements are obtained in the order base  the reordered vector can be used.
order = [ 3, 4, 7, 1, 2, 5, 6 ]
indices = 1:7
reorder=zeros(Int,length(indices))
reorder[order] = indices
This is to reorder the dofs for a pobrlem with free dofs = [3,4,7] and nodes with dofs = [n1 1,2, n2 34, n5 567]

A usefull package for this feature is https://github.com/JuliaArrays/BlockArrays.jl


Correct "seperated" to "separated"

fix typo

Hi @shengyu7697, Can you please sign CLA , thank you !

Hi @shengyu7697, Can you please sign CLA , thank you !

I have already signed the CLA.


Support For In Memory Database

We need support for an in memory database for integration testing. We don't really mind what in memory database is supported.

The DBOs module currently supports only MySQL (and compatibles) and PostgreSQL. In general, I'd say that it's best to test your app against the same database type that's going to be actually used. Having said that, MySQL has a Memory storage engine. We never tested the DBOs module against it, but you can definitely try (would love the feedback!). We also plan to implement support for SQLite next.

Do you guys have an ETA on the SQLite driver, or has work been started on it?

The work on it has not started yet. But we plan to have it by September.

Closing for now. Stay tuned for the SQLite driver implementation.

#3


select radio when corresponding select is changed, addresses #696

Addresses https://github.com/jupyterhub/kubespawner/issues/696
Does:

Adds js- prefixed classes to the Select boxes and the Label containers - this is not strictly necessary, but I find it good practice to separate out classes used by Javascript
Adds a <script> tag to the end of the template with simple jQuery to react to changes to the select tag and ensure that the parent Input Radio is selected.

Stylistic choices:

I've gone ahead and used jQuery since it existed - happy to remove the dependency, but it would mean a bit more verbose code.
There's probably a few different ways to handle traversing the DOM and selecting the correct radio, probably with different trade-offs. I've gone for the simplest approach here: this does mean that this function would need to be looked at in case the HTML structure of the page drastically changes, but hopefully it's simple enough to tell what's going on here that it would be easy to fix.

Let me know if this works / if there's anything else that would be good to do here.
cc @yuvipanda

Closing here - going to open PR against jupyterhub/kubespawner, based on conversation.


Video App Skill won't exit

Hi Steve,
I downloaded your video app skill and ran it locally. The sample video plays, but it won't exit after I say "Alexa Quit" or "Alexa Stop" . I see the "SessionEndedRequest" being invoked, but the sample video continues to play. Only after saying "Alexa Stop" a few times the skill quits. Would you have any insight how I could fix this problem?
Thanks,
Moses

Nevermind. I figured out a workaround


ValidationsValidateVoucherResponse interface incomplete

Shouldnt the discount field in ValidationsValidateVoucherResponse have a discount for fixed type vouchers?
Currently it looks like this: discount?: DiscountAmount | DiscountUnit | DiscountPercent;
It is missing DiscountFixed that would use DiscountVouchersTypesEnum.FIXED

Dear @Luifr  thank you for reporting this issue.
Required changes has been introduced in this PR: https://github.com/voucherifyio/voucherify-js-sdk/pull/207
PR that triggered new version release: https://github.com/voucherifyio/voucherify-js-sdk/pull/208
Fix is included in version 2.2.1 and it should be available in NPM registry shortly.
I will close this issue now, if you feel like something is still missing, please feel free to reopen


Implemented expandtabs method in Bytes with suitable test cases.

The method expandtabs inside the Bytes.java file from python/common/org/python/types has been implemented with suitable test cases inside tests/datatypes/test_bytes.py
This method expands the tabs according to the tabsize(default=8) with spaces.
It is to be noted that this method does not replace the tabs with space as many times the tab character ('\t') is encountered but replaces with position parameter for spaces with the input length.
eg
b'test\t'.expandtabs() will not be b'test        ' but instead will be b'test   '
All test cases run properly, and further details can be noted through the test_bytes.py file.

@freakboy3742 merge?


Develop dependabot to maintain, update Github actions

All existing github action regressions have been fixed. We could start developing dependabot to maintain the github actions

PR created : https://github.com/iocost-benchmark/resctl-demo-image-recipe/pull/12

Merged https://github.com/iocost-benchmark/resctl-demo-image-recipe/pull/12


Feat: Overload for specifying multiple types for a query (e.g. for items query)

Done using [in] cloud url paratemer. New types filter was created for this purpose


Deprecation warning in Rack 3

On 404 responses, I'm getting deprecation warning in logs:
ActionController::RoutingError (No route matches [GET] "/login.html"):

/usr/local/bundle/ruby/3.2.0/gems/rollbar-3.4.1/lib/rollbar/middleware/rails/show_exceptions.rb:10: warning: Request#[] is deprecated and will be removed in a future version of Rack. Please use request.params[] instead

Rails 7.1.1
rack 3.0.8

I confirm this issue on Rails 7.1.2. Same error.
rails (7.1.2)
rack (3.0.8)

Hey @morgoth @jonmchan, Thanks for reporting this issue. I opened an internal bug ticket for this issue (131090). I'll get back to you after we schedule it.

Hey @morgoth @jonmchan, We scheduled work on the Ruby SDK after Thanksgiving. Our Ruby dev is currently headed down on another crucial initiative. I was informed that another fix will probably require a breaking change; hence expect a major version bump.

Thanks for the update and thanks for addressing this! Will be on the look out for the new version.


Feature: Support repeated filenames

Feature Target Details
Currently file names must be unique or conflicts will occur. Add support to leverage fullpaths & immutable indexing so that repeated filenames don't break the process.
Why is this change needed?
With the addition of immutable indexing for tracking file moves this is now possible.
Additional context
Move-FileToMediaFolder will need addressing and probably additional functions.

[ ] Move Update-Lastindex to Invoke-MEDIA(Movies/Shows)ToProcess

[ ] Add the file index into the table when copying the file to transcode directories and suffix the copied file with the index at the end of the filename

[ ] Invoke-Process(movie/show) - Check for any needed considerations
[ ] Start-Transcode(movie/show) - Check for any needed considerations
[ ] Move-FileToMediaFolder - Account for the previously appended index suffix when updating the table.

[ ] Move-FileToMediaFolder - Update to reference probed comment data in the database and update that entry specifically.
[ ] Move-FileToMediaFolder - Remove the previously appended index suffix when moving the item back to the original directory.

Replacing a file is failing because the newly transcoded file doesn't match any table entries. Need to add handling for that.


Can't call typecheck in try-pact

When I try (typecheck 'payments) for the payments example at http://kadena.io/try-pact/, I get: :88:0: No such module: "payments"

You just need to commit the tx first, typecheck resolves modules from the RefStore in the environment.


Forward port 4-to 5 20220815

➡ Forward port
Port ign-fuel-tools4 to ign-fuel-tools5
Branch comparison: https://github.com/gazebosim/gz-fuel-tools/compare/ign-fuel-tools5...ign-fuel-tools4
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

We don't support the ign-fuel-tools5 branch anymore, that EOL'ed with Dome. I'll close this an open a port to Fortress (ign-fuel-tools7).

https://github.com/gazebosim/gz-fuel-tools/pull/288


doc: document Scope

resp. extend existing documentation.

As discussed on zulip, CC @kmill.

Thanks for the suggestions and explanations; these were very helpful. I have addressed them all.

I was not sure about two details of the "how to contribute":

I made separate commits for my changes (so you can check more easily what changed)
I aimed to wrap all comments at 100 characters (do you use semantic line wrapping, hard wrapping at 100 chars or a mixture?)

I'm very happy to follow the house style, if I know what it is. CONTRIBUTING and the information linked therein does not seem to mention it.

Thanks! I've submitted it to the merge queue.


Tests for the new Rects class

Temporary PR to add commits to https://github.com/useblocks/libpdf/pull/36.
As https://github.com/useblocks/libpdf/pull/36 is merged, this is not needed anymore.


Added get method to Map-like objects

Description
Map-like objects didn' have any get method for the underlying dict member.
List of changes

added Each Map has now a get method with the default value of None.

Thank you!


Respect restricted and account confirmation system defaults when creating new user

The DefaultUserIsRestricted, RegisterEmailConfirm and RegisterManualConfirm system configuration options are not always enforced when creating a new user.
This PR configures these system defaults in user_model.CreateUser, and also enhances the CreateUserOverwriteOptions  to allow overwriting the system defaults when needed.

There are a lot of places where IsActive is currently set to true instead of
!(setting.Service.RegisterEmailConfirm || setting.Service.RegisterManualConfirm).
I'm not sure which of these need to remain true by design?

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@7b4c3c7). Click here to learn what that means.
The diff coverage is 39.20%.

:exclamation: Current head f747cdd differs from pull request most recent head a795e55. Consider uploading reports for the commit a795e55 to get more accurate results

@@           Coverage Diff           @@
##             main   #19310   +/-   ##
=======================================
  Coverage        ?   47.44%
=======================================
  Files           ?      942
  Lines           ?   131414
  Branches        ?        0
=======================================
  Hits            ?    62346
  Misses          ?    61555
  Partials        ?     7513

Impacted Files
Coverage Δ

cmd/admin.go
0.00% <0.00%> (ø)

routers/api/v1/admin/user.go
40.00% <0.00%> (ø)

routers/install/install.go
1.79% <0.00%> (ø)

routers/web/auth/linkaccount.go
0.00% <0.00%> (ø)

routers/web/auth/oauth.go
20.66% <0.00%> (ø)

routers/web/auth/openid.go
0.00% <0.00%> (ø)

services/auth/reverseproxy.go
0.00% <0.00%> (ø)

services/auth/source/pam/source_authenticate.go
0.00% <0.00%> (ø)

services/auth/source/smtp/source_authenticate.go
0.00% <0.00%> (ø)

models/user/user.go
54.78% <33.33%> (ø)

... and 4 more

Continue to review full report at Codecov.

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

Could anyone advise on where setting.Service.RegisterEmailConfirm and/or setting.Service.RegisterManualConfirm can be bypassed and where they cannot?

Could anyone advise on where setting.Service.RegisterEmailConfirm and/or setting.Service.RegisterManualConfirm can be bypassed and where they cannot?

I think gitea command line could ignore these options. Otherwise they should be obey.

I don't really think external authentication sources should obey to these rules also, probably they could have option to them to added then to activate user or not but currently I don't think that's a good idea to have to activate them

My initial reason for this PR was that DefaultUserIsRestricted is currently not taken into account in multiple places where new user accounts are created.
When discussing this on discord @zeripath mentioned "you might wanna check isManualConfirm too - appears I may have missed some", so I broadened the scope to also look into RegisterEmailConfirm and RegisterManualConfirm as well.
But it looks like there's no consensus as to where these settings should apply?

Should I maybe leave the RegisterEmailConfirm and RegisterManualConfirm settings AS-IS, and only address DefaultUserIsRestricted in this PR?

Should I maybe leave the RegisterEmailConfirm and RegisterManualConfirm settings AS-IS, and only address DefaultUserIsRestricted in this PR?

I'm OK for that.


Set constructor does not accept Union type of Set | undefined when strictNullChecks is enabled

TypeScript Version<EMAIL_ADDRESS>
Search Terms:
set union constructor strict null undefined readonly array strictNullChecks
Code
const params = undefined as Set<any> | undefined;
const ok = params ? new Set(params) : new Set(params);
const ng = new Set(params);

compile command
tsc test.ts -target ES2017 -strict

Expected behavior:
It should be compiled successfully
Actual behavior:
Line const ng = new Set(params) raise type error:
error TS2345: Argument of type 'Set<any> | undefined' is not assignable to parameter of type 'readonly any[] | null | undefined'.
  Type 'Set<any>' is missing the following properties from type 'readonly any[]': length, concat, join, slice, and 11 more.

This error occurs only when strictNullChecks is enabled
Playground Link:
http://www.typescriptlang.org/play/#src=const params %3D undefined as Set<any> | undefined%3B
const ok %3D params %3F new Set(params) %3A new Set(params)%3B
const ng %3D new Set(params)%3B

Toggle strictNullChecks on/off from "Option" menu
Related Issues:
I could not find any related issues.

So there are two overloads of SetContructor
https://github.com/Microsoft/TypeScript/blob/4505eea92d6509ffea479128cb7e72706ae7f090/lib/lib.es2015.collection.d.ts#L67-L70
https://github.com/Microsoft/TypeScript/blob/afa94c527cd407fb0eed815d070f66406485a03d/src/lib/es2015.iterable.d.ts#L179-L181
I think the second one should be
interface SetConstructor {
  new <T>(iterable?: Iterable<T>): Set<T>;
}

When you split them up into parts, the resolution gets split, that is why it is working.
const ok = params ? new Set(params) : new Set(params);`
                            ^Iterable         ^ReadonlyArray | undefined


Refactor random generators

We currently have random generators distributed over the source code. It would be good to use one central generator instead (maybe in util).

done


Content for  multi-port deployments, services, and routes

Also, seed SO with this question
@0x777 : Assign it back to me once you are done so I can take care of the above.

https://stackoverflow.com/questions/47770895/how-can-i-expose-my-hasura-microservice-on-multiple-ports


Is Jerry Nixon on Vacations??

Hi,
I know Jerry is the owner of Template 10 and also a very busy man, but it has been weeks since we last heard of him.
Is he on vacations?

Yes. Back now, though. Thanks @Quico20 for keeping track.

Glad you are back!!!
Hopefully you release soon T<IP_ADDRESS> for falls creators and higher!!!