Hashing time.Time does not generate unique hashes

See included ginkgo test case.  2 Different times return the same hash
It("hashes time differently", func() {
			t1 := time.Now()
			t2 := t1.Add(1 * time.Second)

			hashedT1, err := hashstructure.Hash(t1, nil)
			Expect(err).Should(BeNil())

			hashedT2, err := hashstructure.Hash(t2, nil)
			Expect(err).Should(BeNil())

			Expect(hashedT1).ShouldNot(Equal(hashedT2))
		})

Hm, we tested this awhile back and it worked but I wonder if this is due to the recent (somewhat, a couple Go releases ago) changes to time to support monotonic clocks... Thanks for reporting this I'll take a look.

I wonder if this is because time.Time does not have any exported properties, so a hash of the struct it not able to be generated.
In the tests, there is a test to test time.Time without forcing it to a string using a tag. Should the expected result be false rather than true?
I don't think this is an issue with the change to monotonic clocks, but rather the time struct does not have any exported properties and since hashstructure will not be able to read those properties, the generated hash is always the same.
A possible solution might be to add a special case to handle time.Time as strings.
If the time.Time is a property in a struct, it's possible to work around it by forcing it to a string using hash:"string", however this does not help if you're interested in passing a time.Time directly to Hash().


How to install?

Hello,
I am running Yosemite 10.10.1 and downloaded the Zip file. There is a lack of documentation on this page that describes the install process and no readme in the Zip file.
There are a bunch of files in the directory but no mention of how to install the application.
Please help me understand the install process. Thanks.

Duplicate of #14.


[GEMM] Fix functional issues with bfloat16 result format

The 09-experimental-block-pointer.py tutorial should support matrix multiplication with float32 accumulation and subsequent truncation to bfloat16.
After addressing #1266, it should be enough to add (torch.bfloat16, torch.float32, torch.bfloat16) to the list of testcases in the tutorial code.

Update: Fix prepared locally; waiting for #1282 to land


Velocity genes and data preprocessing

Hi,
I have been trying to use LatentVelo with my data. I have managed to install the tool and use it with one example dataset from scVelo. However, when I use it with my data , I run into the following error
Warning, folder already exists. This may overwrite a previous fit.
0 velocity genes used
Traceback (most recent call last):
  File "/rna-velocity/src/scRNA_latentvelo.py", line 106, in <module>
    epochs, val_ae, val_traj = ltv.train_anvi(model, adata, batch_size = batch_size,
  File "/lib/python3.9/site-packages/latentvelo-0.1-py3.9.egg/latentvelo/trainer_anvi.py", line 104, in train_anvi
  File "/lib/python3.9/site-packages/latentvelo-0.1-py3.9.egg/latentvelo/models/annot_vae_model.py", line 223, in loss
  File "/lib/python3.9/site-packages/latentvelo-0.1-py3.9.egg/latentvelo/utils.py", line 22, in unique_index
  File "/lib/python3.9/site-packages/latentvelo-0.1-py3.9.egg/latentvelo/utils.py", line 22, in <listcomp>
RuntimeError: max(): Expected reduction dim to be specified for input.numel() == 0. Specify the reduction dim with the 'dim' argument.

I guess that the error results from the 0 velocity genes that were used in the model . Could you maybe explain what the role of the velocity genes in the calculation is and how they are determined?
For the data preprocessing, I followed the examples in your repository and have tried both ltv.utils.anvi_clean_recipe and ltv.utils.standard_clean_recipe. Would you recommend any other preprocessing steps for the data ?
Any help would be really appreciated. Your tool looks very interesting, so it would be great to use it!

Generally this happens when the genes are very noisy. I would try something like sc.pp.filter_cells(adata,min_genes=30) to see if that fixes it. I have some updates soon that might also be useful, for example increasing the number of neighbors when computing moments for unspliced and spliced would also reduce noise. In the examples I also always run scv.pp.filter_genes(adata, min_shared_counts=20) or similar for real data, when you run this how many genes are left over? If it removes almost all of the genes then the noise is likely the issue.
Alternatively, you can disable requiring velocity genes by setting 'corr_velo_mask=False' when specifying the model, then it should run without requiring velocity genes.

Thanks for the suggestions! After running scv.pp.filter_genes(adata, min_shared_counts=20) a lot of genes, approximately 60% have been filtered out. So the data does seem somewhat noisy. With the filtering step it still does not work.
I also tried disabling the velocity genes in the VAE model with both methods that you suggested. When I use corr_velo_mask=False or correlation_reg=False, I get either a Runtime error RuntimeError: max(): Expected reduction dim to be specified for input.numel() == 0. Specify the reduction dim with the 'dim' argument. or a Value error ValueError: With n_samples=1, test_size=0.1 and train_size=None, the resulting train set will be empty. Adjust any of the aforementioned parameters..
I only disabled the velocity genes in the model definition. Do I need to change something else as well in the the training?

I had the same problem when I tried to integrate multiple samples that were too different from each other. I would guess that this tool doesn't work for data that is too noisy and complex, and your best bet is running it on a subset of the data -- but this is just my guess, and I have only experimented with LatentVelo for a week.


sass compilation fails on linux (tmp noexec)

Hi,
maven build on windows with eclipse works fine using the plugin
version of plugin 0.2.25
but the build fails on linux with jenkins
i get this error:
Caused by: java.lang.UnsatisfiedLinkError: /tmp/libjsass-7568725547703726790/libjsass.so: /tmp/libjsass-7568725547703726790/libjsass.so: failed to map segment from shared object: Operation not permitted
at java.lang.ClassLoader$NativeLibrary.load (Native Method)
at java.lang.ClassLoader.loadLibrary0 (ClassLoader.java:1941)
at java.lang.ClassLoader.loadLibrary (ClassLoader.java:1824)
at java.lang.Runtime.load0 (Runtime.java:809)
at java.lang.System.load (System.java:1086)
at io.bit3.jsass.adapter.NativeLoader.loadLibrary (NativeLoader.java:48)
at io.bit3.jsass.adapter.NativeAdapter. (NativeAdapter.java:28)
at io.bit3.jsass.Compiler. (Compiler.java:35)
at wrm.libsass.SassCompiler.compileFile (SassCompiler.java:42)
after google for this error, maybe the problem is that the /tmp directory is mounted with noexec
i'm not sure if it's a good idea to mount the directory without the noexec for security reasons and if company policy allows it.
has anyone else the problem? is there a solution?
Thanks!
Maven home: /products/maven/apache-maven-3.5.2
Java version: 1.8.0-adoptopenjdk, vendor: Oracle Corporation
Java home: /products/java/adopt/jdk8u172-b11/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "3.10.0-1127.13.1.el7.x86_64", arch: "amd64", family: "unix"

@ck443 Thanks for raising the issue.
Imho this is caused by the jsass lib the plugin uses.
The most obvious solution would be to grant exec rights on the /tmp folder.
That being said, which version of jenkins are you using?
With jenkins you should be able to start the build in a docker container which essentially eliminates the need to mount the /tmp folder with noexec because a new container is created for each build.

@123Haynes thanks for your answer


How to create keyboard navigation example like hotmail email box.

How to create keyboard navigation example like hotmail email box.

I'm sorry, you'll need to research this yourself. Try google & stackoverflow for help, if needed.


Update Composer dependencies (2020-08-05-00-11)

Visual regression test passed!
View the visual regression test report


Serbian language leader

Hello everyone.
I'm Marko Todorić and I would like to become a Serbian language leader . I am a native speaker, currently working as a DevOps engineer and beginner Python developer.
I have free time to contribute to this project and would very much like to be involved. Consider myself to r a good fit as I'm very familiar with home assistant as I've been using it fore little over two years now.

Thanks @maretodoric! Let's continue the discussion here: https://github.com/home-assistant/intents/discussions/400


Add Windows Symlink builder script

Fixes #1062
This PR is ready for review.
Risk
This PR makes no API changes.
Testing Plan

Open the Android project in Android Studio on a Linux/Mac machine, verify it builds
Open the Android project in Android Studio on a Windows machine, run the gradle task buildWindowSymLinks,  verify it builds

Summary

Added a script to build Windows type symlinks
Added a gradle task to call script
Added a gradle conditional to point to correct folder based on OS

Changelog
Bug Fixes

Windows users can now build the project from source if they wish

CLA

[x] I have signed the CLA

Codecov Report

Merging #1081 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #1081   +/-   ##
==========================================
  Coverage      46.05%   46.05%
  Complexity      3692     3692
==========================================
  Files            433      433
  Lines          20260    20260
  Branches        2190     2190
==========================================
  Hits            9331     9331
  Misses         10445    10445
  Partials         484      484

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 a1c9407...a2ab690. Read the comment docs.


Fix issues reported from Crashlytics

Token update sanity check.
Browser history item sanity check & back/forward logic update.

@JamesSmartCell please write these messages clearly, it is not clear what this PR actually does.

It's a small PR that patches an issue reported on crashlytics (GPF), and fixes an inconsistency in the same code I noticed while exhaustively testing the fix. The back forward buttons on the dapp broswer didn't quite work consistently. The code was fixed so they work correctly and have tests at each stage in case of edge conditions. Real world use on multiple devices and OS's throws unexpected things at you. Ideally we should be testing on at least the major OS variants: Samsung Huawei Motorola and Xiaomi.


Install script breaks on non-debian systems

The install script assumes a lot of things, one of them is that the apt tool is installed on the target system. It might be a good idea to either check that the OS is debian/debian-based, or to print a warning beforehand about this script only working on debian-based systems.

Yes will do.
Thanks!

Added Debian-based an Arch compatibility.
Still working on Fedora since i dont know the package names for yum (and i dont have a VM right now).


Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Good idea, thanks!


Can never get calendar grid to display: Unknown props

I've copied and pasted the code in the basic demo and the 'Getting Started' doc exactly, and I keep getting the message:

warning.js:44 Warning: Unknown props startAccessor, endAccessor, onView, onNavigate, views, titleAccessor, allDayAccessor, dayFormat, selectRangeFormat, eventTimeRangeFormat, timeGutterFormat, monthHeaderFormat, dayHeaderFormat, dayRangeHeaderFormat, agendaHeaderFormat, agendaDateFormat, agendaTimeFormat, agendaTimeRangeFormat, formats, events, components, onHeaderClick, onShowMore on  tag. Remove these props from the element. For details, see https://fb.me/react-unknown-prop
in div (created by MonthView)
in MonthView (created by Calendar)
in div (created by Calendar)
in Calendar (created by Uncontrolled(Calendar))
in Uncontrolled(Calendar) (created by MyCalendar)
in div (created by MyCalendar)
in MyCalendar (created by App)
in div (created by App)
in div (created by App)
in App

My App.js is very basic:
import React, { Component } from 'react';
import logo from './logo.svg';
import './App.css';
import BigCalendar from 'react-big-calendar';
import moment from 'moment';
import '../node_modules/react-big-calendar/lib/css/react-big-calendar.css'

// Setup the localizer by providing the moment (or globalize) Object
// to the correct localizer.
BigCalendar.momentLocalizer(moment); // or globalizeLocalizer

const myEventsList = [];

const MyCalendar = props => (

    <BigCalendar
      events={myEventsList}
      startAccessor='startDate'
      endAccessor='endDate'
    />

);

class App extends Component {
  render() {
    return (

Welcome to React

          <MyCalendar />

    );
  }
}

export default App;

Here is what is rendered:

you have to set an explicit height on the calendar

Ah, the warnings in the console made me think it was broken. Would be nice to get rid of those, but this solved the issue. Thanks!


[Event Change Request] Codeunit 7312 "Create Pick" - OnFindBWPickBinOnBeforeApplyBinCodeFilter

Describe the request
I want to change the BinCodeFilterText based on the current location. Therefore, I request to add CurrLocation as a new Parameter to the OnFindBWPickBinOnBeforeApplyBinCodeFilter event:
local procedure FindBWPickBin(
        LocationCode: Code[10]; ItemNo: Code[20]; VariantCode: Code[10]; ToBinCode: Code[20]; UnitofMeasureCode: Code[10]; QtyPerUnitofMeasure: Decimal;
        QtyRoundingPrecision: Decimal; QtyRoundingPrecisionBase: Decimal; DefaultBin: Boolean; CrossDockBin: Boolean; var TotalQtyToPick: Decimal; var TotalQtyToPickBase: Decimal;
        var TempWhseItemTrackingLine2: Record "Whse. Item Tracking Line" temporary; WhseItemTrackingSetup: Record "Item Tracking Setup"; UserDefaultBin: Boolean)
var
	FromBinContent: Record "Bin Content";
	QtyAvailableBase: Decimal;
	QtyToPickBase: Decimal;
	QtytoPick: Decimal;
	BinCodeFilterText: Text[250];
	IsSetCurrentKeyHandled: Boolean;
	IsHandled: Boolean;
	EndLoop: Boolean;
begin
	// ...
	if not IsHandled then begin
		if CurrLocation."Require Pick" and (CurrLocation."Shipment Bin Code" <> '') then
			AddToFilterText(BinCodeFilterText, '&', '<>', CurrLocation."Shipment Bin Code");
		if CurrLocation."Require Put-away" and (CurrLocation."Receipt Bin Code" <> '') then
			AddToFilterText(BinCodeFilterText, '&', '<>', CurrLocation."Receipt Bin Code");
		if ToBinCode <> '' then
			AddToFilterText(BinCodeFilterText, '&', '<>', ToBinCode);
		// NEW BEGIN
		OnFindBWPickBinOnBeforeApplyBinCodeFilter(BinCodeFilterText, CurrLocation);
		// NEW END
		if BinCodeFilterText <> '' then
			FromBinContent.SetFilter("Bin Code", BinCodeFilterText);
		if WhseItemTrkgExists then begin
			WhseItemTrackingSetup.CopyTrackingFromWhseItemTrackingLine(TempWhseItemTrackingLine2);
			FromBinContent.SetTrackingFilterFromItemTrackingSetupIfRequiredWithBlank(WhseItemTrackingSetup);
		end;
	end;
	//...

[IntegrationEvent(false, false)]
local procedure OnFindBWPickBinOnBeforeApplyBinCodeFilter(var BinCodeFilterText: Text[250]; CurrLocation: Record Location)
begin
end;

Additional context
I want to change the BinCodeFilterText based on the current location bin codes (Service & To-Production).
Internal work item: AB#544588

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 23054.


The depth start of get_distance() function ?

Required Info

Camera Model
D400

Issue Description
When I use get_distance() to get the distance from camera to the surface in its front. There is always about 4mm error if I set the depth start at the imager, so I guess the function return the distance to front cover glass.
Is that so ? The depth starts from front cover glass ?

Hi @gywhitel   Yes, on the cased versions of RealSense 400 Series cameras, depth is measured from the front glass.  On the caeless Depth Kit versions of RealSense cameras, depth is measured from the lens or from the back of the module.
You can get the ground truth of measurements by adding a minus value to them.  On the D435 and D435i models, that is -4.2 mm

Thanks ! Problem solved.

Thanks very much @gywhitel - I'm pleased I could help.  I have updated the table graphic to show the value for the D455 model too.


resolveConsistentlyWith can't downgrade directly declared dependencies

Given configurations A and B, where B extends A, B is set to resolve consistently with A. If B directly declares a dependency on a module from A in a higher version compared to what was resolved in A, Gradle can't resolve the version for B.
Expected Behavior
Given the request for the "consistent" version resolution, the same version as A should be enforced.
Current Behavior
Resolution fails.
Context
We are trying to develop a plugin to allow users easily build modules for a specific java-based product. As such, we inject some dependencies through the plugin for test and compilation. Users can add their own dependencies. The test harness has a number of dependencies, but we'd like the test classpath to eventually use the same versions as the compile classpath. Tried using the consistent resolution and it works as long as it has to upgrade the dependencies. Downgrade though does not work.
Steps to Reproduce
Minimal build:
plugins {
    id 'java'
}

repositories {
    mavenCentral()
}

configurations {
    testCompileClasspath.shouldResolveConsistentlyWith(configurations.compileClasspath)
}

dependencies {
    implementation 'org.apache.commons:commons-text:1.9'
    testImplementation 'org.apache.commons:commons-lang3:3.12.0'
}

Here commons-text has a transitive on commons-lang3:3.11.0. But specifying a dependency on 3.12.0 breaks the build:
> Task :dependencyInsight
org.apache.commons:commons-lang3:3.11 FAILED
   Selection reasons:
      - By constraint : version resolved in configuration ':compileClasspath' by consistent resolution
   Failures:
      - Could not resolve org.apache.commons:commons-lang3:{strictly 3.11}.
          - Cannot find a version of 'org.apache.commons:commons-lang3' that satisfies the version constraints:
               Dependency path ':gradletest:unspecified' --> 'org.apache.commons:commons-lang3:3.12.0'
               Constraint path ':gradletest:unspecified' --> 'org.apache.commons:commons-lang3:{strictly 3.11}' because of the following reason: version resolved in configuration ':compileClasspath' by consistent resolution
               Dependency path ':gradletest:unspecified' --> 'org.apache.commons:commons-text:1.9' (compile) --> 'org.apache.commons:commons-lang3:3.11'

I believe this is part of the strictly logic - if people declared a certain dependency there must be a reason, so we should raise an issue. But I don't find it particularly intuitive. The downgrade instead works just fine if the downgrade happens for a transitive dependency, i.e. this couple of dependencies
    implementation 'org.apache.commons:commons-text:1.7'
    testImplementation 'io.bitbucket.5faces:common:1.0.4'

causes commons-lang3 3.12.0 requested by the test dependency to be downgraded correctly to the 3.9 version resolved in the compile classpath

That's a design decision of the dependency management engine. We want you to fix your dependency declarations.


Generate Unit Tests for Benchmarks

Closes #5222
This PR introduces automatically generated unit tests for each runtime benchmark.
From the docs:
Test functions are automatically generated for each benchmark and are accessible to you when you run cargo test. All tests are named test_benchmark_<benchmark_name>, expect you to pass them the Runtime Trait, and run them in a test externalities environment. The test function runs your benchmark just like a regular benchmark, but only testing at the lowest and highest values for each component. The function will return Ok(()) if the benchmarks return no errors.
You can construct benchmark tests like so:
#[test]
fn test_benchmarks() {
  new_test_ext().execute_with(|| {
    assert_ok!(test_benchmark_dummy::<Test>());
    assert_err!(test_benchmark_other_name::<Test>(), "Bad origin");
    assert_ok!(test_benchmark_sort_vector::<Test>());
    assert_err!(test_benchmark_broken_benchmark::<Test>(), "You forgot to sort!");
  });
}

The last CI thing seems to be an issue with the VM


Install docker desktop with power shell

I use power shell to install docker desktop with command 'Start-Process $dockerInstallerPath -ArgumentList "install --quiet" -Wait -passthru', but this way use the default configuration, how can i install without wsl2 like this

Thanks for sharing the issue.
If you have a paid Docker subscription, please raise a ticket through Docker Desktop support.
If you do not have a paid subscription,  you can get support through our Github repo for-win, where we respond on a best-effort basis.


[docs]: incorrect components nav order

Description of the issue
Components nav menu should be alphabetic e.g. Accordion, Alert, Avatar, Badge, etc
Instead, we have Avatar, Badge, Accordion, Button, etc.

Bug currently displays at: https://patternflyelements.com/components/


CSS and JS inside HTML

Describe the bug
Often there is CSS and JS code within an HTML snippet. Kmdr needs to recognise this and parse for that language.

An example of this


Add skill progress bar in skill nav-tree

Fixes #230
However the missing folder icon seems to be present :thinking:

However the missing folder icon seems to be present 🤔

I have it now as well ... 🤷♂


Get size and origin for all controls

During my development of drag & drop I am continuously confronted by trying to obtain the correct position of the dropped item. This is possible some, but the inconsistency in being able to set the sizes of controls, but not obtaining the current size and origins is frustrating, necessitating own functions (on the Mac, in Objective-c, or modifying NAppGUI itself).
I propose that the following be added:
S2Df size = guicontrol_size(<control>)and
V2Df origin = guicontrol_origin(<control>)
This way we would have a consistent way of obtaining these across NAppGUI, and, together with the existing gui_mouse() function may calculate any drop position accurately. These already exist for windows, and to a lesser degree for views.
For some controls, like listboxes, tables, etc. a similar row and header height function should also be added. For instance, now you can set the row and header height in a table but not obtain the current ones.

Hi @SamSandq. This can be done with
window_control_frame()
window_client_to_screen()

You are right, the following code will obtain the mouse position in a control:
V2Df gui_mouse_pos_in_view(Window *w, void *v) {
    V2Df mp = gui_mouse_pos();
    R2Df frame = window_control_frame(w, (GuiControl *) v);
    V2Df posX = window_client_to_screen(app->window, frame.pos);
    V2Df pos = {mp.x - posX.x, mp.y - posX.y};

    return pos;
}

Consider this issue partly resolved; we still have to be able to obtain the row and header heights, as applicable, in order to position correctly.

You may close this issue as solved.


Auto-accept steam invites and gamesPlayed problem

Hi!
Today I launched my steam messages bot. But two parts of my js aren't working:
// accept users that added you when you were offline.
steamClient.on('relationships', function() {
  for (var friend in steamClient.friends) {
    if (steamClient.friends[friend] == 2) {
      steamClient.addFriend(friend);
      console.log('SteamID ' + friend + ' added!')
    }
  }
});

// accept users that add's you while you're online.
steamClient.on('friend', function(SteamID, Relation) {
  if (Relation == 2) {
    steamClient.addFriend(SteamID);
    console.log(SteamID + " Accepted")
  }
});

and
steamClient.gamesPlayed([220, 440, 570, 730]);
What am I doing wrong? Can someone help me?

To explain I paste console problem-info:
    steamClient.gamesPlayed([220, 440, 570, 730]);
                ^

TypeError: steamClient.gamesPlayed is not a function

when I remove steamClient.gamesPlayed([220, 440, 570, 730]); from js everything works just fine and console prints standard Logged in! but program isn't accepting any invites on steam :/

Create a steam user from the client. It's in the main help docs, along with here. https://github.com/seishun/node-steam/tree/master/lib/handlers/user
steamClient != steamUser.

^

Okay I changed steamClient.gamesPlayed([220, 440, 570, 730]); to steamUser.gamesPlayed([220, 440, 570, 730]); and I have var steamUser = new Steam.SteamUser(steamClient); in js file but console prints this error:
                throw Error("Illegal value for "+this.toString(true)+" of type "
+this.type.name+": "+val+" ("+msg+")");
                ^

Error: Illegal value for [object Object] of type message: number (object expected)

Please read the documentation. This is not a help desk.


Consider another color (blue) for Extension view Enable button

VSCode Version:1.3.0-insiders
OS Version:Windows 10

The beige yellow Enable button does not look compelling IMO.  I can see that you wanted to contrast the green Install button (green means go) but yellow means yield and isn't attractive.
Consider using blue which is a very common button color and will have good contrast on both light and dark themes.  We generally use blue for highlighting and in this case we want to highlight that you need to take an action to enable the extension.

We talked in UX meeting about making this button the default blue color.

I like how blue blends into the viewlet - thanks for the nice suggestion.
Check it out nad let me know what you think


Integrate display to terminal in F3D

Context
@snoyer recently added a small feature in the libf3d to transfrom an image into displayable terminal (#1219 ) and created a python app for it: https://github.com/snoyer/f3d/blob/ef9d6c0c64909712ed10e840229890dcd0bb1255/examples/libf3d/python/render-terminal/render_terminal_mouse.py
Let's discuss how to cleanly integrate that in F3D

a python app for it

that's a generous way to phrase it, I would have called it a "janky POC" myself :)
There was a quick mention of making terminal output a type of window and then just plugging it into the existing F3D app; while we could probably forward keyboard and mouse events the limited resolution of the output mean it couldn't properly support all of the features of the app (unreadable text actors for example).
Should it be looked at more like the android version, ie. as a custom, platform-specific front-end?
Maybe it depends on how much you expect out of it compared to the actual F3D?

I think it has value to be integrated in the F3D executable, even if there is limitations.
Alternatively, it could be a separate executable or pure python app, but it would not help with integration into distribution packages and such.

Another idea for the short term: Have a collection of python tools in the repository. You can see that as ready-to-use and useful examples.
We can make it part of the installation process and package it in the wheels.
I think it's be still useful to have this feature in the F3D application itself too tho


[MEV-Boost\Builder] Unhide hidden params for mev-boost\builder

Unhide arguments:
--eb-endpoint
--validators-registration-default-enabled
--validators-proposer-blinded-blocks-enabled
Extended proposer-config json referenced by the existing --validators-proposer-config
A draft note covering the usage of the changes: https://hackmd.io/@StefanBratanov/BkMlo1RO9
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

The eb-endpoint name seems confusingly similar to ee-endpoint and it even took me some time to work out what eb was.  Might be better to make it explicitly --external-builder-endpoint and have an alias --eb-endpoint.
The other two options possibly should be named so they're tied together (so they'd be --external-builder-registration-default-enabled.  Maybe we drop the external and just have --builder-registration-enabled and --builder-endpoint etc.
Do we need to expose --validators-proposer-blinded-blocks-enabled? I'd like to make that the only way it works at some point in the future so would be good if we could avoid exposing it.  Possibly enable it automatically when a builder endpoint is specified?

The eb-endpoint name seems confusingly similar
yeah, agree, not particularly user friendly.

Maybe we drop the external and just have --builder-registration-enabled and --builder-endpoint etc.
I like more this option. Btw the e was for execution :-)

Do we need to expose --validators-proposer-blinded-blocks-enabled?
the problem is that this option is for the VC. So we automatically enable it when using --validators-registration-default-enabled, but its complicated if you use --validators-proposer-config, also because it can be changed at runtime.

An option could be to simply enable the blinded block flow by default, and warn if user disable it (which could prevent mev when using validators-proposer-config)

An option could be to simply enable the blinded block flow by default (which saves bandwidth for Bellatrix blocks), and warn if user disable it (which could prevent mev when using validators-proposer-config)

If we're really confident in our fallback that would be reasonable. Otherwise it could easily break for a lot of people.

a beginning could be to enable it now in all our merged testnets


[DOCFIX] Add collectInfo command to doc

This adds collectInfo command to Troubleshooting.md.
The improvements to command printouts are addressed in #11037

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8260/
Test PASSed.

@ns1123 PTAL especially on the security implications. Thanks!

The code change part moved to https://github.com/Alluxio/alluxio/pull/11035

Removed the change to command printouts in code. This PR focuses on the doc fix.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8511/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8512/
Test PASSed.

ping @ns1123

@ZacBlanco Do you mind merging this for me please? Thanks!

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.2 please


"Terminal Delete Word Left" sends ^W to terminal instead of deleting the word (powershell)

VSCode Version: 1.15
OS Version: Windows 10

Steps to Reproduce:

Open the powershell integrated terminal and write something.
Hit ctrl+backspace or execute the "Terminal Delete Word Left" command.

A ^W will show up instead of deleting the word:

Also, "Delete Word Right" sends a D.
Reproduces without extensions: Yes

Duplicate https://github.com/Microsoft/vscode/issues/31821

Having the same issue, ctrl+backspace inserts a ^w in the current position in the terminal, ctrl+delete will insert a d. The functionality works in the powershell.exe that ships with windows. Checked #31821 & #39312 to no avail and found no solutions. Hopefully there's some sort of solution as I would use the WSL bash as my go to terminal however there's poor support for using dependencies installed within the WSL, ie. having vscode use the git.path point to the git installed in the wsl /usr/bin/git


Do not download isolinux folder for http boot

Folder isolinux is not required by http boot

/test-cs8-virt

/test-cs9-virt

/test-rhel-8-6-virt

/test-rhel-8-7-virt

/test-rhel-9-0-virt

/test-rhel-9-1-virt

All green now. Land it.


Enable the v4 core/health EndpointTest

Looks like Eclipse has reordered the imports a bit, let me know if the original order needs to be preserved

If you can preserve the original order that would be great


Do not define HMR overlay if HMR is disabled

Is your feature request related to a problem? Please describe.
When the HMR is disabled at vite.config.ts, it is still being loaded in @vite/client
const overlayId = 'vite-error-overlay';
customElements.define(overlayId, ErrorOverlay);

I'm not using the HMR and it is breaking Vite for me:
Uncaught DOMException: Failed to execute 'define' on 'CustomElementRegistry': the name "vite-error-overlay" has already been used with this registry
    at http://localhost:9000/base/path/@vite/client:176:16

I'm proposing to change the code to detect if hmr overlay is enabled/disabled or prevent defining vite-error-overlay twice.
// The code above should be replaced with this:
const overlayId = 'vite-error-overlay';
if (!customElements.get(overlayId)) { customElements.define(overlayId, ErrorOverlay); }

Using Vite 50 on WSL, Edge 88

It seems the client.ts file is loaded twice. I will try to reproduce it an open proper issue for it.

It seems the client.ts file is loaded twice. I will try to reproduce it an open proper issue for it.


Merge subsidence into dycore tendencies

This PR:

moves subsidence computations into the dycore tendencies
Removes TC-internals from subsidence calculations
Moves the subsidence cache into the dycore cache (which is now optional)
Removes total_enthalpy and uses the Thermodynamics total_specific_enthalpy

Subsidence is not always a function of time, however this PR changes things so that it's always computed. This is a bit simpler in that there's no need for an initialization and the update. Also, the cache is now tied to the tendency (which is now optional) and in closer proximity to the tendency.
This PR is a step towards #949.

bors r+


What does status in narrations.json mean?

There are 3 status, "redacted", "redacted_partial" and "complete"
There are 466 videos narrations status="complete" but metadata has_redacted_regions=True
There are 651 videos narrations status="redacted" but metadata has_redacted_regions=False

Status represents whether narrations were redacted (i.e. removed - either fully or partially) for those videos since those intervals are in the test sets.  We'll update the annotation schemas in the docs to represent this.

Thanks for reply! So IIUC, narration redaction is irrelevant to video redaction?


Guillotine the Built-ins

Move everything that the TC doesn't have to care about to a lib/prelude.ml file. This file is looked up in:

AMC_PRELUDE
AMC_LIBDIR/prelude.ml
relative to the compiler

this was pain

@SquidDev help
> open import "%prelude"

> open import "%option"

> (+1) <$> Some 1
=stdin:33: attempt to call a nil value (global 'Functor')

nvm


coverage is described in make help but has no implementation

https://github.com/mozilla/puente/blob/master/Makefile#L9

I removed some things from the Makefile that shouldn't have been there plus updated the help.
Thank you for writing this up!

fast response, a+++, would file again ⭐ ⭐ ⭐ ⭐ 🌟


[WIP] Re-organize playbook

As this re-write/re-organize gives these features

[ ] Easily integrate new OS support
[x] Make updates more smoothly
[x] Manage systems more safer
[ ] Add Windows support (in plan)
[x] Cleaner structure and becomes easy to read source code

What can be made, a already did, can't more than that, just need to merge

What can be made, a already did, can't more than that, just need to merge


feat(pixels): added my pixel (#5785)

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

merged


Updates API calls

Adds DKG Get info to get the Info of the status of an election (DKG part)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 64.132%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3079

Relevant Lines:
4801

💛  - Coveralls


No tables or views in tree using Databricks driver.

I am able to query as expected, but when I go to the tree it says Nothing here beneath Tables and Views.

I am on the newest version of SQLTools and the Databricks driver.

Please raise this with the author of the Databricks driver. One way is to use the "Report Issue" option from VS Code's "Help" menu, then choose "An extension" in the second dropdown and the Databricks driver on the third one.

Understood. Thank you!


[v2] data urls in css break parsing

Version
@next, so recent v2.0. I can't post a reproduction, webpackbin just hangs when trying to add styled-components@next or @2.0.
Steps to reproduce
styled.div`
    background-image: url(data:image/png;etc); /* quoting the url doesn't help */
`

Expected Behavior
it works like in v1
Actual Behavior
it breaks:
.gMVerY {
    background-image: url(data:image/png;}.other-component-1hm8ero {}.kWlwJm {position: absol…: 3px;left: -300px;width: 1290px;background-color: rgba(120, 120, 120, 0.8);
}

This is exactly why we need people to get to try v2 before we release it. 👍  @thysultan this sounds like a bug in stylis, could you look into that?

@mxstbr:  @thysultan already fixed it in  v1.0.1 :) Can you upgrade?

Ohh nice 👌 Sure, PR welcome!

@mxstbr I tried installing stylis v1.0.1 in SC's node_modules folder but that didn't fix matters - could there be something else at play?

Hmm, the es build actually already includes stylis, which is why it doesn't use the new one. Building a fresh one does indeed fix the problem, but perhaps the es build should not package deps

Closing since #436 fixed it.

As I reached you in the morning at Twitter, I can confirm this. Currently, v2 does not parse style definition when data url is in place.


Left menu not showing

I use your framework to create a left menu only in my main view, I add the reveal button, but when I click inside nothing happens. What's up ? I link the init view with the left menu (sw_rear) and the main view (sw_front) with the Navigation controller between that. I delcared my init view like that:
@interface MainViewController : SWRevealViewController
I'm missing something ? No log message is showing and the app doesn't break, only is not appear the left slide menu.

In the front controller, you have to add the target when you tap the button:
        [self.revealButtonItem setTarget: self.revealViewController];
        [self.revealButtonItem setAction: @selector( revealToggle: )];

Yes I do this:

(void)customSetup
{
SWRevealViewController *revealViewController = self.revealViewController;
NSLog(@"Tenemos reveal: %@", revealViewController);
if ( revealViewController )
{
[revealButtonItem setTarget: self.revealViewController];
[revealButtonItem setAction: @selector(revealToggle:)];
[self.navigationController.navigationBar addGestureRecognizer: self.revealViewController.panGestureRecognizer];
self.revealViewController.delegate = self;
}
}
And I see revealViewController is null !!! Is it possible because @interface MainViewController : SWRevealViewController is not the entry point of my stroyboard ? How can I init SWRevealViewController ?

Thanks.

You do not have to subclass SWRevealViewController. Instantiate it and set frontViewController and rearViewController

https://github.com/iDevelopper/PBRevealViewController


Android build on macOS Cross-Compile host: Fixes wrong Ninja rules generation.

Cmake generates rules.ninja and build.ninja with macOS specific default linker options. That default options (like -dynamiclib -Wl,-headerpad_max_install_names) cause link errors when performing cross-compilation for Android.
Similar fixes for Window host already done. This PR provided fixes for macOS host.

@swift-ci please test

@compnerd Will be nice if you find a time to look on this PR.

CC: @Rostepher @gottesmm @gwynne @jrose-apple
The build complexity is getting pretty absurd at this point.  I think that we need to speed up the changes to break the standard library builds into sub-configures so that we can let CMake deal with the flags for the cross-compilation.

Again, I have no objections as long as the different target platforms continue to build in parallel.

Thank you!
Requested changes seems done.

@swift-ci please test

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 74ec877eef2ad28baf2591f88d1301173b21b193

Build failed
Swift Test OS X Platform
Git Sha - 74ec877eef2ad28baf2591f88d1301173b21b193

I believe that this is breaking the android SDK builds on Windows :-(.  We no longer pass -soname which means that the library soname is wrong.

Seems like another if/else block needed .)

@vgorloff - found the issue, its a dereferencing behavior change in CMake, PR25973 addresses that.


add typescript type declarations

This change set adds type declarations to allow using this package in TypeScript projects while having full support for autocompletion and integrated API documentation.
It also fixes  some bugs I discovered along the way, mostly missing parameters/documentation in code comments.

@mast: shamelessly pushing this PR again

@mast Are you planning to add TypeScript support for this package? :)


Ampersand is being encoded improperly when using addCompany method

I am using this library to build vcards on law firm websites. Most them have ampersands in their names & which is being used for the site title on the website.
I am then grabbing this site site to populate the Company field in the vcard.
But when I view the contents of the vcard, the & is being shown as &amp
Is there something I can do to prevent this? Do I need to change the char set for my implimentation? Thanks for the help.

Anybody out there?

I'm at the moment unable to check what is happening there, it will be a few weeks until I will be able to check if I can fix this.
I don't know if @jeroendesloovere is available?

We have problems with encoding special characters as well - i.e. first name "Grüter" will become "GrÃ¼ter" on Windows and "Gr√°ter" on MacOS.

I am having the same problem with the ampersand.


Show/Hide UI issue in samsung phones

Even though i added ShowVolumeUI.HIDE  , for some phones, particularly samsung phones, the volume change UI is shown on the screen

@Vaibhavch2001 Did you had fix on this.I'm facing the same issue

@Vaibhavch2001 Did you had fix on this.I'm facing the same issue


Remove deprecated KotlinDslOptions.experimentalWarning

Was deprecated in Gradle 7.0 and scheduled for removal in Gradle 8.0.

Done in https://github.com/gradle/gradle/pull/21909 for Gradle 8.0


Implement shortnames feature

Add support for resolving model shortnames. Parse shortnames from /etc/ramalama/registries.conf.d/000-shortnames.conf . Modify New() function to use resolved shortnames. Improve user experience by allowing use of familiar shorthand names for models. This change allows users to use shortnames instead of full model names, similar to podman functionality. The feature enhances usability while maintaining compatibility with existing model naming conventions.

@rhatdan it's not clear to me how podman/bats handles tests that depend on certain files like "/etc/containers/registries.conf.d/000-shortnames.conf"
I can see tests in podman/bats testing the shortnames features, but they seem to test failure cases, rather testing successful cases.

Linked issue: https://github.com/containers/ramalama/issues/38

I still have to do directory precedence style code here. For example on macOS rootless installs are more common, so probably don't want to edit /etc at all:
etc/ramalama/registries.conf.d/000-shortnames.conf

We want to read these files in /usr/share/ramalama/aliases.conf, /etc/ramalama/aliases.conf $HOME/.config/ramalama/aliases.conf
For the in the container use case we should prepare a file out of this content with the last ALIASES winning and then mount it into the container.
cat  /usr/share/ramalama/aliases.conf /etc/ramalama/aliases.conf $HOME/.config/ramalama/aliases.conf > /tmp/aliases.conf and then process it, removing duplicates.

Makes sense to me, although instead of this one:
$HOME/.config/ramalama/aliases.conf
we could do:
$HOME/.local/share/ramalama/aliases.conf
$HOME/.local/share/ramalama is a directory we already use and it kinda fits with /usr/share/ramalama/aliases.conf
But I'm easy about directory structures, happy to go completely as you described also @rhatdan ...

Well $HOME/.local/share is for app content, whereas $HOME/.config/ is for user custom configuration.
As far as the mount within the container,  I think we will need to process all of the aliases on the users host system into one aliases.conf file which we mount into the container.  So assemble it from /usr, /etc/ and $HOME into something in /tmp and then mount into container.  I don't think we want to mount all of the mounts from the host into the container. This way the container itself could have some aliases.conf files which are not on the users host system.

Well $HOME/.local/share is for app content, whereas $HOME/.config/ is for user custom configuration.
As far as the mount within the container, I think we will need to process all of the aliases on the users host system into one aliases.conf file which we mount into the container. So assemble it from /usr, /etc/ and $HOME into something in /tmp and then mount into container. I don't think we want to mount all of the mounts from the host into the container. This way the container itself could have some aliases.conf files which are not on the users host system.

I moved the resolving of short-names to long-names to occur outside of the container, so now no temporary files required, seemed cleaner... Less difference between container and no-container case and no temporary files to clean up. And now no bind mounts also.

LGTM


Please invite me to the community

Name
akashogics
Additional Context
i am interested in joining to learn in open source

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


UUID conversion

A complete conversion from usernames to UUIDs should be done as part of the conversion to the Sponge configuration format.
This way, there are no stray uuid-using permissions configurations lying around (and hopefully users don't edit the file by hand to put names in)
This involves

[ ] Converting names to UUID on conversion
[ ] Setting name option on user login

SquirrelID seems like a good option.

and done with bf21cf92d5ad4897d1a861993bb9551e563f7bfa


Windows: Create product name from vendor and app name

Previously the app name was used for product name as well. But there is
a limitation for the latter, it does not accept a name made only of
numeric digits. To mitigate this, always create the product name by
combining vendor and app name.
BUG=APPTOOLS-338

@pozdnyakov ptal

rs=me


Error peek view triangle not correct color

When using the error peek view, the triangle tip does not respect the correct error color and is always yellow:

Looks like the rules are not getting deleted and overridden.
Arrow widget deletes the old rule and adds new rule with new colours and which is not working.
@jrieken Moving this back to you.

fixed by reverting https://github.com/microsoft/vscode/commit/5565ddd88c6350f90e1031be4ce0d6089ac73a57


Lint/UnneededSplatExpansion: wrong unnecessary splat expansion.

Expected behaviour
Hello! I'm having a issue with Rubocop warning this code:
[*'A'..'H', *%w(J K M N), *'P'..'Z', *'0'..'9'] Unnecessary splat expansion in *%w(J K M N)
Which gives me something like ['A', 'B', 'C', 'D' 'E' ..  'J', 'K', 'M', 'N', 'P' .. '0', .. '9'].
Actual behaviour
When i use rubocop -a filename , the cop just removes the * from *%w(J K M N) and the returned data appears to be something different like:
['A', 'B', 'C', ['J', K', 'M', 'N'], 'P' ... '0']
It creates a new array inside. I'm trying # disable instead but would be good if i could keep both running, rubocop and *%w(J K M N)
RuboCop version
$ rubocop -V
0.44.1

I tried putting together a bugfix, but came across this:
[1] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.source
=> "[*[1, 2, 3, 4, 5]]"
[2] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node
=> s(:array,
  s(:splat,
    s(:array,
      s(:int, 1),
      s(:int, 2),
      s(:int, 3),
      s(:int, 4),
      s(:int, 5))))

[1] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.source
=> "*[1, 2, 3]"
[2] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node
=> s(:array,
  s(:splat,
    s(:array,
      s(:int, 1),
      s(:int, 2),
      s(:int, 3))))

This^ should be splat-array instead of array-splat-array. Is this some issue with the parser?

@tejasbubane, I am not seeing the same issue with the code parsing that you are with the given example. What are versions of parser and ruby that you are using? Mine are 0.44.1 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-darwin14).
[9] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.source
=> "*[1, 2, 3]"
[10] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node
=> s(:splat,
  s(:array,
    s(:int, 1),
    s(:int, 2),
    s(:int, 3)))
[11] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.parent.source
=> "[*[1, 2, 3]]"
[12] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.parent
=> s(:array,
  s(:splat,
    s(:array,
      s(:int, 1),
      s(:int, 2),
      s(:int, 3))))

I was able to reproduce what you saw with the foo = *[1, 2, 3]. In this case, I assume that the array-splat-array has something to do with how Ruby will treat the array once it has been expanded.
[1] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.source
=> "*[1, 2, 3]"
[2] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node
=> s(:splat,
  s(:array,
    s(:int, 1),
    s(:int, 2),
    s(:int, 3)))
[3] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.parent.source
=> "*[1, 2, 3]"
[4] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.parent
=> s(:array,
  s(:splat,
    s(:array,
      s(:int, 1),
      s(:int, 2),
      s(:int, 3))))
[5] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.parent.parent.source
=> "foo = *[1, 2, 3]"
[6] pry(#<RuboCop::Cop::Lint::UnneededSplatExpansion>)> node.parent.parent
=> s(:lvasgn, :foo,
  s(:array,
    s(:splat,
      s(:array,
        s(:int, 1),
        s(:int, 2),
        s(:int, 3)))))

@victorfeijo, this doesn't look like it will be difficult to fix. We just need to change the correction if we are in an array.


Lots of confusing errors when you have a comma instead of a semicolon on a field declaration

This snippet results in a three error messages:
class A	{
  A(this.a);
  final A a,
}

analyzer:
  error • The final variable '' must be initialized • test.dart:2:3 • final_not_initialized_constructor_1
  error • Expected an identifier • test.dart:4:1 • missing_identifier
  error • Expected to find ';' • test.dart:4:1 • expected_token

compiler:
test.dart:4:1: Error: Expected an identifier, but got '}'.
}
^
test.dart:4:1: Error: Expected ';' after this.
}
^
test.dart:2:10: Error: 'a' isn't an instance field of this class.
  A(this.a);
         ^

[ ] Analyzer shouldn't give the bogus message "The final variable '' must be initialized".
[ ] Compiler and analyzer should both decide if they expect an identifier or a semicolon, and not request both in the same location. (Presumably that's a parser thing.)
[ ] Compiler and analyzer should agree on whether a is an instance field of the class.

I suspect that the parser is recovering by inserting a synthetic identifier (which has an empty name) and a semicolon after the comma. What I think it should be doing is reporting that it expected a semicolon instead of a comma. If it did that, then I believe that would be the only diagnostic users would see from analyzer. And I suspect that if it did that the compiler would agree that a is an instance field.
Given that analyzer and cfe share the same parser, I'm not sure why the parse errors from 'compiler' are different.

Still reproduces


VSCode creates duplicate files

I am designing an Extension with quite a few Reports, which slows down the realtime warnings. There have been a lot of issues with my txt2al converted Reports which were basically the same issue over and over again so I got quite fast at solving this.
It happend around 10 times that VSCode copied my file (with an Suffix with my local machine Name) and the original file was empty or in an old Version of my code. VSCode switched the files opened in my Editor without me noticing first, so that I had to go back and compare those files and delete one of them.
Is this a known issue? Can I surpress this behavior somehow?

Hi @JulianTillmann,
Please open a new issue about the Txt2AL convertion and include a .txt file that reproduces the problem.
What files are copied and have their filename changed? I'm not aware that we are doing anything like that.
The compiler will keep your .rdl and word files updated so when you change your reports dataset then the files also have the correct view of what the system provides them.

This is not a Problem with a single file but rather seems a strange behaviour of VSCode.
I have, for example, a Report file "Rep56789.SomeReport.al" and I'm searching and replacing some Code, that the Txt2al provided. The real time warning thingy (I don't know what to call it) is checking all the changes I'm doing and suddenly I have a new file called "Rep56789.SomeReport-localmachinename.al". The old file is empty or has a previous state with Code I already changed.

Are you sure this is not the effect of some other extension you have installed in VSCode. Try disabling all other extensions and see if the behavior is still there,

Nothing I would think: German Language for VSCode, AL Language, a XML Formatter and a GUID Creator. But I'll try.
Is there a way to disable or slow down the real time warning thingy? I'd suspect this would be to blame

In the last months, we've been working hard on adding new features that increase your productivity and on improving the quality of the tools. Unfortunately, this meant that we were not able to follow up on as many GitHub issues as we would have liked. Is this issue still a problem for you? Can you provide a sample project to reproduce the issue? Does it still reproduce on the Wave 1 2020 release?

hasn't happened in months I think. so I guess this can be closed

I will close it then, thank you for your answer!


Unifying docker management to the docker-compose file. Idea/question.

I haven't seen the full stream yet, but it makes me wonder why you are using the handcrafted build.sh, run.sh, and stop.sh scripts instead of the docker-compose file. You can specify the network pretty easily there, and it would be more approachable for other people too.
But thanks for that anyway.

I wrote this in 2018-ish, and at the time, I couldn't figure out how to set a static IP for each instance via compose. I'm not actually sure you can do it now, since Compose very much wants to have flexible networking.

I don't know for sure, but the docker-compose and docker CLI switches should be identical in these capabilities. But you are right, it is preferable to make it flexible. Thanks for the answer!


Bioc2022Anno

Please supply the following information:
Short title: Introduction to Bioconductor annotation resources
Workshop URL: eg., https://github.com/jmacdon/Bioc2022Anno
Workshop docker image: eg., ghci.io/jmacdon/bioc2022anno
Workshop port: 8787
Workshop memory request: 2GB
Workshop description: There are various annotation packages provided by the Bioconductor project that can be used to incorporate additional information to results from high-throughput experiments. This can be as simple as mapping Ensembl IDs to corresponding HUGO gene symbols, to much more complex queries involving multiple data sources. In this workshop we will cover the various classes of annotation packages, what they contain, and how to use them efficiently.

Thanks, @jmacdon.  I added 2022 to the short title to distinguish from similar named workshops in the past.


update xrefs from camel 3.12.x to 3.13.x

See apache/camel-website#670

Current plan seems to be to leave camel 3.12.x on the website until cq 2.4.x is retired: this would replace the need for this PR.


query returns weird result for non existing elements

Querying the document with non existing elements should returnnull looking at the SelectorEngine implementations. But instead they do return an element that simply not exists.
I have no option to find out if these elements do exist or not. The isEmpty method of WebKitElement also is no option because it always returns false.

A workaround is to queryAll and check for empty

Yes, W3CSelectorEngine.query return null but Ui4j intercepts all public methods of WebkitElement. If selector engine return value is null instead of null value an empty WebkitElement produced by WebKitEmptyElementProxy and isEmpty#L48 method of proxy return true instead of false. I am going to document these kind of magical things with JavaDoc after 2.0 release.
Here is sample code to test isEmpty method:
package com.ui4j.test;

import com.ui4j.api.browser.BrowserEngine;
import com.ui4j.api.browser.BrowserFactory;
import com.ui4j.api.browser.Page;

public class IsEmptySample {

	public static void main(String[] args) {
		BrowserEngine webKit = BrowserFactory.getWebKit();
		Page page = webKit.navigate("about:blank");

		page.getDocument().getBody().setInnerHTML("
my content
");

		System.out.println(page.getDocument().query("#valid-id").isEmpty()); // print false
		System.out.println(page.getDocument().query("#invalid-id").isEmpty()); // print true
	}
}

The empty element logic influenced by JQuery. JQuery doesnt return null or throw any exception if the query result is empty.

ok. So, the solution is to always check isEmpty(). I'm fine with it but it's not very convenient to use. You have already changed API with 2.0. So, what about let query return an Java 8 Optional?

Although ui4j-webkit requires Java 8, ui4j-api is compatible with Java 7. Because of this requirement i didnt use Optional feature. But, Its better to drop Java 7 compatibility and use Optional feature.

Element.isEmpty method removed and SelectorEngine.query() method returns Optional instead of Element.


Docs: Add instructions for infrastructure operator

The cluster-api-provider-agent requires the Infrastructure Operator to be installed.
THis PR add instruction on how to install it on OCP.

/lgtm

/hold

Moving this to assisted service repo


"updateTimestamp: false" is not working properly

First of thank you such an awesome plugin :+1:
There is just one little thing that bugs me. POT-Creation-Date is updated every time even if there are no changes. This is a bit annoying in a version controlled environment, where you don't want to see unnecessary changes.
Edit: Just found updateTimestamp in the documentations. I think it is exactly what I'm looking for. But when I tried it, it stops updating POT-Creation-Date, altogether even if there changes in the translation strings.

I have the issue where updateTimestamp: false in the Gruntfile is just ignored, and the timestamp is updated every time even if no strings have changed.

FWIW, this is applied here.

There has been a test for the updateTimestamp option for quite awhile and the underlying node module has its own tests.
My only guess is that this was an issue with an earlier version, but if anyone is still having an issue, please feel free to reopen with more information -- a failing test would be even better.


Import kurbo via vello, not glazier.

This makes this consistent in terms of where kurbo things are imported from.

I had previously opened a discussion in Zulip about this: https://xi.zulipchat.com/#narrow/stream/354396-xilem/topic/Importing.20from.20kurbo


jira open fails in dockerized version

Describe the bug

Hi, I am running the dockerized version of jira-cli. Setup and connection to Cloud is succesful, am able to list projects. But then trying "jira open", I get:
~ # jira open XYZ
Error: exec: "xdg-open,x-www-browser,www-browser": executable file not found in $PATH
Please provide following details

JiraCLI Version:

(Version="v1.4.0", GitCommit="eb842e18014303a575fde8e19508b3c7c13982c5", CommitDate="2023-05-09T07:09:45+00:00", GoVersion="go1.19.9", Compiler="gc", Platform="linux/amd64")

Are you using Jira cloud or on-premise jira server?

Version:         1001.0.0-SNAPSHOT
Build Number:    100234
Deployment Type: Cloud
Default Locale:  en_GB

What operating system are you using? Also mention version.

Ubuntu 22.04. Note: running dockerized jira-cli

What terminal are you using? Also mention version.

Bash shell.
To Reproduce
I am running the dockerized version of jira-cli. Setup and connection to Cloud is succesful, am able to list projects. But then trying "jira open", I get:
~ # jira open XYZ
Error: exec: "xdg-open,x-www-browser,www-browser": executable file not found in $PATH
Expected behavior
Navigate to the project.
Screenshots
None
Additional context
None

This is isn't really possible out-of-the-box with Docker, because it would have to communicate with your OS to open the browser, and it can't because it's isolated.
So just install it natively would be my advice.
If you don't mind getting your hands dirty, you can try something like this:
https://stackoverflow.com/questions/54437534/docker-open-a-url-in-the-host-browser
Though you are on your own with the latter option, as it's more of a power user option.


dont render well

I get some example here(WebcamPanelExample). In  netbeans it works fine but when I build it as executable jar file and open it works except for the camera. It seems very slow first it was "device initialization" after a plenty seconds (33s) it shows but black video and repaint value is 0.2 and fps 0.0 and doesn't change. I don't know what is the problem.
In desktop works fine. The problem arises on laptop

I suggest you try this driver: https://github.com/eduramiba/webcam-capture-driver-native

I suggest you try this driver: https://github.com/eduramiba/webcam-capture-driver-native

will this work in java ant?? My project was not in maven

Sure it should work if you download the jars from maven central: https://repo1.maven.org/maven2/io/github/eduramiba/webcam-capture-driver-native/1.0.0/
Then, if using windows, remember to copy the correct DLLs from https://github.com/eduramiba/webcam-capture-driver-native/tree/master/natives to where your program will run

I get some example here(WebcamPanelExample). In netbeans it works fine but when I build it as executable jar file and open it works except for the camera. It seems very slow first it was "device initialization" after a plenty seconds (33s) it shows but black video and repaint value is 0.2 and fps 0.0 and doesn't change. I don't know what is the problem.
In desktop works fine. The problem arises on laptop

In logs
2023-12-05 09:57:24,183 [AWT-EventQueue-0] INFO  com.github.sarxos.webcam.Webcam - WebcamDefaultDriver capture driver will be used
2023-12-05 09:57:24,353 [atomic-processor-1] INFO  com.github.sarxos.webcam.ds.cgt.WebcamOpenTask - Opening webcam USB2.0 HD UVC WebCam 0
2023-12-05 09:57:57,730 [atomic-processor-1] ERROR com.github.sarxos.webcam.ds.buildin.WebcamDefaultDevice - Timeout when requesting image!
2023-12-05 09:58:03,225 [frames-refresher-[0]] ERROR com.github.sarxos.webcam.ds.buildin.WebcamDefaultDevice - Timeout when requesting image!
2023-12-05 09:58:08,814 [atomic-processor-1] ERROR com.github.sarxos.webcam.ds.buildin.WebcamDefaultDevice - Timeout when requesting image!
2023-12-05 09:58:14,381 [frames-refresher-[0]] ERROR com.github.sarxos.webcam.ds.buildin.WebcamDefaultDevice - Timeout when requesting image!


Build failed with gcc 7.2 on Amazon Linux AMI

$ npm install

><EMAIL_ADDRESS>install /home/ec2-user/app/node_modules/hashtable-patch-valeriansaliou
> node-gyp configure build

make: Entering directory `/home/ec2-user/app/node_modules/hashtable-patch-valeriansaliou/build'
  CXX(target) Release/obj.target/native/src/hashtable.o
In file included from ../src/hashtable.cpp:1:0:
../src/hashtable.h:6:64: error: missing binary operator before token "("
 #if defined __APPLE__ && defined __has_include && __has_include(<tr1/unordered_map>)
                                                                ^
In file included from ../src/hashtable.h:15:0,
                 from ../src/hashtable.cpp:1:
../src/v8_value_hasher.h:7:64: error: missing binary operator before token "("
 #if defined __APPLE__ && defined __has_include && __has_include(<tr1/unordered_map>)
                                                                ^
In file included from ../../nan/nan_new.h:189:0,
                 from ../../nan/nan.h:222,
                 from ../src/hashtable.h:14,
                 from ../src/hashtable.cpp:1:
../../nan/nan_implementation_12_inl.h: In static member function ‘static Nan::imp::FactoryBase<v8::StringObject>::return_t Nan::imp::Factory<v8::StringObject>::New(v8::Local<v8::String>)’:
../../nan/nan_implementation_12_inl.h:340:37: warning: ‘static v8::Local<v8::Value> v8::StringObject::New(v8::Local<v8::String>)’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:5052): Use Isolate* version [-Wdeprecated-declarations]
   return v8::StringObject::New(value).As<v8::StringObject>();
                                     ^
In file included from ../src/hashtable.h:14:0,
                 from ../src/hashtable.cpp:1:
../../nan/nan.h: In constructor ‘Nan::Utf8String::Utf8String(v8::Local<v8::Value>)’:
../../nan/nan.h:1066:53: warning: ‘v8::Local<v8::String> v8::Value::ToString() const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:10040): Use maybe version [-Wdeprecated-declarations]
       v8::Local<v8::String> string = from->ToString();
                                                     ^
../../nan/nan.h:1080:74: warning: ‘int v8::String::WriteUtf8(char*, int, int*, int) const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:2658): Use Isolate* version [-Wdeprecated-declarations]
         length_ = string->WriteUtf8(str_, static_cast<int>(len), 0, flags);
                                                                          ^
In file included from ../src/hashtable.h:15:0,
                 from ../src/hashtable.cpp:1:
../src/v8_value_hasher.h: In member function ‘bool v8_value_equal_to::operator()(CopyablePersistent*, CopyablePersistent*) const’:
../src/v8_value_hasher.h:43:24: warning: ‘bool v8::Value::Equals(v8::Local<v8::Value>) const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:2481): Use maybe version [-Wdeprecated-declarations]
         if (a->Equals(b)) {          /* same as JS == */
                        ^
../src/hashtable.cpp: In static member function ‘static Nan::NAN_METHOD_RETURN_TYPE HashTable::Constructor(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/hashtable.cpp:50:43: warning: ‘int32_t v8::Value::Int32Value() const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:2478): Use maybe version [-Wdeprecated-declarations]
         int buckets = info[0]->Int32Value();
                                           ^
../src/hashtable.cpp: In static member function ‘static Nan::NAN_METHOD_RETURN_TYPE HashTable::Rehash(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/hashtable.cpp:231:42: warning: ‘int32_t v8::Value::Int32Value() const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:2478): Use maybe version [-Wdeprecated-declarations]
     size_t buckets = info[0]->Int32Value();
                                          ^
../src/hashtable.cpp: In static member function ‘static Nan::NAN_METHOD_RETURN_TYPE HashTable::Reserve(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/hashtable.cpp:249:43: warning: ‘int32_t v8::Value::Int32Value() const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:2478): Use maybe version [-Wdeprecated-declarations]
     size_t elements = info[0]->Int32Value();
                                           ^
../src/hashtable.cpp: In static member function ‘static Nan::NAN_METHOD_RETURN_TYPE HashTable::MaxLoadFactor(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/hashtable.cpp:266:39: warning: ‘double v8::Value::NumberValue() const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:2475): Use maybe version [-Wdeprecated-declarations]
         factor = info[0]->NumberValue();
                                       ^
../src/hashtable.cpp: In static member function ‘static Nan::NAN_METHOD_RETURN_TYPE HashTable::ForEach(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/hashtable.cpp:289:33: warning: ‘v8::Local<v8::Object> v8::Value::ToObject() const’ is deprecated (declared at /home/ec2-user/.node-gyp/10.13.0/include/node/v8.h:10046): Use maybe version [-Wdeprecated-declarations]
         ctx = info[1]->ToObject();
                                 ^
make: *** [Release/obj.target/native/src/hashtable.o] Error 1
make: Leaving directory `/home/ec2-user/app/node_modules/hashtable-patch-valeriansaliou/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2

After some time I figured that it's because of gcc version
When I switched to gcc 4.8 instead of 7.2 installation succeeded

Good to hear 👍


chore: Bump v037 core

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 78.13%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
90.5%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.01%

Covered Lines:
5841

Relevant Lines:
7476

💛  - Coveralls


[Feature-3311][alert] Add alert plugin instance function

Brief change log

Add alert plugin instance to bind alertgroup and alertplugin

Verify this pull request
This pull request is code cleanup without any test coverage.

This is a temp branch, there is no need to review the code at present, thx~

Part of the function is not completed, because of time, I merge this PR first, and for the remaining functions, I will create a new PR to complete it.


Timer needs to be reinitialized after being reset.  How can I just set the timer to zero?

I save time the user spends working on a project on my page.  Because saving to my database isn't that reliable just on unload, I also save the time spent every time the user saves their project.
So, after a user saves their project, I send that time to be saved to my database, and want to reset the timer to zero.
I thought that's what ResetRecordedPageTime(my page name) would do.  What it actually seemed to do is remove the timer altogether - if I reset the page time, then wait, then try to retrieve time spent, I get NaN.
What I'm doing is calling ResetRecordedPageTime(mypage), then re-initializing the timer (same page name every time).  This seems to work and have the behavior I want.
  The docs should indicate this, if so.  And it would be nice to have a function that lets you set a timer value to whatever you want so you can zero it out after saving the time saved.

I just thought of another thing it would be nice to have - instead of just being able to reset a timer, I'd like to be able to set its value.
then I could call that to set the timer to zero after a save.
On a page refresh, I can save the old timer value to localStorage, and on page initialization I could pull out that value and add it to the timer.  Then I would update times only on project save.
Any hope of a "setTimerToValue('page_name', value) function?

Ah, I read the code and found out my problem (one of my problems).  After calling ResetRecordedPageTime, you don't want to reinitialize the timer - you need to call startTimer('timer-name').  If you re-initialize the timer again with the same name, the idle time gets all screwy.


added silent flag for detached run

This PR adds a --silent flag that fixes the detached process crashing due to the write the the std i/o of a parent process already ended.

its enough if we check for the --detach flag


⚠ Bofrost.de has degraded performance

In 6ad5d44, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1229 ms

Resolved: Bofrost.de performance has improved in 5e3613c after 15 minutes.


Allow user to set the symbol lookup date

Tests added

One note about a variable name, but other than that this LGTM! :+1:


ci(compile): and publish

I suggest to do a squash merge as I did various tests.
This PR automatically compile, generate a zip, create a new release and attach that file.
It is executed when a new tag is configured in the repository.

So compiles with leveldb libraries so it should works also where is not installed.
Do you think that an appimage is needed? as it is a simple binary like innoextract etc

Do you think that an appimage is needed? as it is a simple binary like innoextract etc

innoextract is taken from the Arch repos only for SteamOS and only for SteamTinkerLaunch. This project is much more general, and will happen to be used by SteamTinkerLaunch.
In terms of building with LevelDB, are there any licensing concerns? This project is licensed under MIT, LevelDB is licensed under BSD-3 Clause. This is a general concern I had with packaging the project, including for an AppImage, as I am not a lawyer and don't know what restrictions may apply (it's also why no work was done on packaging for a long time!).
Licensing issues aside that would apply no matter how we package, for use with STL, I would prefer to use an AppImage, just like with Yad, yes. :slightly_smiling_face:

Looks good then, just one more question (sorry if it seems like I'm splitting hairs here...):

The inbound licence is a permissive licence: provide you respect and include all existing copyrights or trademarks and [...]

Since we're simply including the leveldb header and not modifying the copyright notices in LevelDB itself, we should also be good on this front I imagine?

Approvals aren't set up for this repo (since it's just me as a code owner) but pressing the "Approved" button is satisfying both to do and to receive, so I like doing it when I remember to :-)
I checked a bit more out about the BSD-3 license and I think just as you said we are good. I will merge this.
Thank you!

It is enough that you mention in the readme that leveldb is a dependence to compile it.


Let contributors know not to run npm i from within packages

It's not obvious that running npm install from within a package breaks the development environment.
There should be an mention of this fact and an explanation why. Something along the lines of:

tinacms uses lerna to manage dependencies when developing locally. This allows the various packages to reference each other via symlinks. Running npm install from within a package replaces the symlinks with references to the packages in the npm registry.

I'll add that as a tip on this page.

Great! The README is where I was thinking


Center all slider's elements

I want to show all the elements within the slider centered, but I try to put more classes to center, only the first element centered me, what do I do?

Closing pending edit of this issue to conform to guidelines detailed in https://github.com/kenwheeler/slick/blob/master/CONTRIBUTING.markdown


[CSL-441] Incorrect threshold is used in calculateSeedQ for GodTossing

@gromak
It uses richmen and certificates for new slot. But it should use old threshold.
For example, suppose 10-th epoch has just started. We need to calculate seed based on data from previous epoch. We should use threshold which was used for secrets generation for 9-th epoch.

@georgeee
Features needed for non-genesis addresses support

@gromak
Don't forget that deep enough certificates must be used.


Add option to add 3rd party libraries to classpath

When testing extensions with .withExtensionClassesFrom() it might become necessary to add 3rd party libs as dependencies to the classpath, as the extensions rely on these libraries. Currently that's not possible.
A method like .withExtensionLibsFrom() might be a good solution.
(this issue is deviated from discussion #57)

Hi @dasniko,
Can I take this issue?
A need this right now e I wold like to contribute in your project too.

Hi @piantino,
if you like, you can send a PR for this, I didn't start yet.
But I think we should name it .withProviderLibsFrom() as the method for custom provider code is called .withProviderClassesFrom().
Looking forward for your suggestion.

WDYT about something like .withProviderClassesFromMavenRepository("groupId:artifactId:version")?
We discuss about the naming, I just wanted to proof the idea of referencing libs/dependencies from (local) maven repo.

For my necessity, the absolute path is enough.
Bellow a sample after my change (is working):
keycloak = new KeycloakContainer();
keycloak.withProviderLibsFrom("./target/my-provider-jar-with-dependencies.jar");

I will research how to look for local maven dependencies, but if the project was not a maven?

I will research how to look for local maven dependencies, but if the project was not a maven?

We should start with a Maven version. It's still the most used dependency management tool. If it works and there are people needing other approaches, I still can think about it.
Having a way to include deps from a repo is way more flexible than just using paths. When I have a dependency from a maven repo in my project, I dont't want to copy the lib into my project, just for the test cases. Additionally, the path on my machine may be different (if it's not part of the project directory) to other machines.

This might be a friend (and I already use other ShrinkWrap libs in the project):
https://github.com/shrinkwrap/resolver

The more I think about this, the more I would go back to the first approach to just add a list of files/paths to the /providers folder in the container. Everything else seems to re-implement another opinionated API (Shrinkwrap resolver). As already mentioned in #60, I'd do the Maven deps part with an extra documentation (e.g. wiki page).
This approach would be flexible enough for any usecase. If one needs some local libs, it's possible, same with Maven deps resolved via another way.
Let's use a method with a signature like this:
.withProviderLibs(List<File> files)
Or would List<Path> be better as attribute?
Can you please update your PR #60 accordingly? Sorry for the overhead with Shrinkwrap resolver. You can skip the part with the docs/readme and leave it untouched (resp. revert your current changes), I'll add an appropriate documentation afterwards. Thanks.

I agree that accepting files is a better approach because this library will not simplify the use of containers, wherein most cases are just copying files from the host to the image/container.
I didn't know Shrinkwrap, but is very simple to use, so, I agree that providing samples and documentation is very nice. Also, this has support to Gradle, may be other in the future.
Finally, is easier to implement and test if the file was copied. Take the sample of a custom template, today this library doesn't support it, but most simply are just copied to the container and calls the build action, Why complicate that!

Thanks for this amazing feature!
We also just encountered this problem and would be thankful for a release.
Could you tell me when are you planning to release this in 2.0.0? Are you also planning to backport this for older versions? If not, may I contribute?
Thanks in advance!

@benbeu
This will be released with 2.1.0 hopefully soon.
I won't implement this in older versions, as older versions would mean v1.x which is Wildfly based and deprecated, so I won't publish new features for this branch. See this as a chance to upgrade your Keycloak environment to the new Quarkus based version. You'll have to do it anyhow latest in a few months.

Thanks for the quick response!
You are probably right and we will discuss this.
Have a great day!


Update error-handling.md corrected mistake with example

The example supplied example did not return a promise, only created a promise and returned nothing.
I suspect this was originally wrote as (req, res, next) => Promise.resolve()..... but was changed to => {} for more readability, but failed to notice the change in behavior that this caused.

Thanks for the suggestion @WORMSS, but the code is correct. As express is using callbacks, we should pass res and next to the promise to handle the async operation.
In this example, the res is not used as throw will occur first (check the next snippet).
app.get('/', (req, res, next) => {
  Promise.resolve().then(() => {
    throw new Error('BROKEN')
    res.send("hello")
  }).catch(next) // Errors will be passed to Express.
})

As you can see the next function is passed to the .catch() function to handle the errors generated in the previous .then(). Maybe we can improve the example to make it more clear, but not sure how.
Feel free to re-open this PR with a different suggestion 🙂


Travis: Add Mono build on X11/GCC with tools

This should help catch build issues for the Mono module too.
Apparently it's possible, though undocumented, to install the dependencies conditionally for given jobs of the build matrix, so I'll rework it tomorrow to install mono only for the mono build (and clang-format-5.0 only for the static checks): https://github.com/travis-ci/travis-ci/issues/3505

Cherry picked into 3.0.6


CA - MSH-4 confusion - Resubmission

CA expected CSV uploaders to have CDPH000085 in MSH-4, they were initially being reported with CDPH000087, then during the portover we confirmed that the MSH would change from that 87 to 85 identifier.
The issue has been fixed. Now we need to look to resubmission. The full list is >3000. Spliting that list into multiple tickets of 500 each.

There are 1198 records that have been purged from RS. Of those 113 are positive
equipment_model	count
BioFire COVID-19 Test | count = 19
BioFire Respiratory Panel 2.1 (RP2.1)*@ | count = 2
LDT: PacificDx Covid-19 | count =  74
Xpert Xpress SARS-CoV-2 | count = 18

Email sent to CA asking if we need to resend the full remaining list (1197) or only the positives (113)

Update: Response from CA State on 9/29/2022
Thank you, Brick & team!
For CA COVID reporting, if results are negative antigens, we do not need these, if the remaining 1197 have positive & negative PCR results, CA does require these to be reported.
Confirmation on PCRs vs Antigens can help, thanks & let me know if there are any other questions?
Thanks!
Vanessa Burkhart

Of the 1197, 1174 are molecular tests and need to be resubmitted according to CA rules. I asked Nathan with SR what the process is for manually pulling in the needed files.

CA agreed on the sync today that we only need to send the 113 positive results. I reached out to Laura King with SR to coordinate


Regrading config["input_frames"]

I wish to pass multiple images with background (same for all images) to the model and obtain the translation and quaternion for each. How does the config["input_frames'] work, can it help in achieving this?

Please refer to our follow-up work for multiple images: https://github.com/rozumden/MotionFromBlur


Some questions about the UI

What does the "WAS only" mean?
Why the "Inbound" and "Outbound" are limited smaller than 4?
And what does ths points mean in the red circle of the following picture?

Hi @JackeyLee007
Selecting WAS only removes terminal nodes such as database, cache or unknown rpc nodes from the server map. It might be helpful to select when you have too many such nodes cluttering up the server map.
Inbound and outbound are limited to 4 currently because of complex server maps may be too large with 4 depth. However, this might change in the future with more optimization.
The red circle's description is under the help menu on the top right corner of the screen.

Renders inbound/outbound nodes for each and every node (within limit) even if they are not directly related to the selected node.
Note that checking this option may lead to overly complex server maps.

Hi @Xylus,
That's clear! Thanks for your answer.
Regards


Replace HTTP with HTTPS in README.md and CONTRIBUTING.md

Replaced HTTP links to HTTPS links for consistency
What kind of change does this PR introduce?
Refactor HTTP links in some Markdown files to HTTPS :)
Did you add tests for your changes?
I don't think changes to Markdown files need tests? If I'm wrong please guide me on how to write one :sweat_smile:
Does this PR introduce a breaking change?
No, it's just minor changes.
What needs to be documented once your changes are merged?
Nothing that I know of, since it's just minor Markdown changes.

Thanks


Mock Response Returns with empty "url"

Describe the bug
When using mock-fetch to intercept a fetch, the response it returns with does not have a "url" field - it returns blank. Example below.
Example Test:
Deno.test("getEndpoint should return the correct endpoint URL", async () => {
  const url = "https://webmention.test/test/1";
  const expectedEndpoint = "https://webmention.test/test/1/webmention";

  mf.intercept(url, { method: "GET" }).response("ok", {
    status: 200,
    headers: { "Link": `<${expectedEndpoint}>; rel="webmention"` },
  });

  const endpoint = await getEndpoint(url);

  console.log(endpoint);
  assertEquals(endpoint, expectedEndpoint);
});

Example Response:
Response {
  body: ReadableStream { locked: false },
  bodyUsed: false,
  headers: Headers {
  "content-type": "text/plain;charset=UTF-8",
  link: '<https://webmention.test/test/1/webmention>; rel="webmention"'
},
  ok: true,
  redirected: false,
  status: 200,
  statusText: "",
  url: ""
}

Expected behaviour According to the MSN Docs, the url field should include the final url after any and all redirects (Redirects which could be simulated using the mock library, say). Instead, it comes up completely empty.
I've noticed other mock fetch libraries (namely this one) also have this issue - is this an intended result and I'm just misunderstanding the URL field?
Thanks.

Hey @chilledtonic - thanks for submitting this issue and sorry for my delayed reply, I was on holiday last week! Yeah it definitely looks like the URL should be populated - I'll double check this tomorrow! If confirmed as a bug, would you be interested in contributing a PR to fix this?

Hey @chilledtonic - thanks for submitting this issue and sorry for my
delayed reply, I was on holiday last week!

No worries at all, it's no crisis - I'm actually also on Holiday right
now!

Yeah it definitely looks like the URL should be populated - I'll double
check this tomorrow! If confirmed as a bug, would you be interested in
contributing a PR to fix this?

I'd be happy to contribute. I'll read up on the docs and get back to
you. Thanks.

I'd be happy to contribute. I'll read up on the docs and get back to
you. Thanks.

Awesome! Any questions, give me a shout :)

If confirmed as a bug,

And to confirm, this is a bug - good spot!


Unexpected memory usage for efficientnetb1?

I can train much larger models than efficientnetb1 without nsl (moreover, only the decoder is trainable here) without using much RAM. NSL's memory usage seems to be growing without end. Is this expected behaviour to use 13+ GB of RAM for this?
Edit: Using nsl.keras.AdversarialRegularization
Using this implementation: https://github.com/qubvel/segmentation_models/tree/master/segmentation_models

Edit: Uses 2.6 GB of RAM when training with a single layer Conv2D (with adverserial model) and around 4-5 GB when running efficientnetb1 without nsl

Works with Unet + mobilenetv2 using 10 GB of RAM so it's probably at least not growing without bound.

Memory usage will always be higher in adversarial training to some degree, but that does seem excessive. I wonder if some of the overhead is coming from the separate gradient tape. In an image model the activations are rather large, and the tape caches all intermediate activations. Ideally, common subexpression elimination should catch this, though, when the training loop is being compiled. What version of TensorFlow are you using? If you can provide a working code example, we may be able to debug further.
We don't have easy access to the tape that Keras uses. You could try using a custom training loop with a single tape.

@samedii, sorry for the super late reply.
I managed to reproduce the issue with EfficientNet using TensorFlow 2.0. As @ppham27 mentioned, the intermediate activations for the adversarial examples did cost some extra memory. Also Keras model subclassing (which AdversarialRegularization does) seems to be taking some space as well.
Thankfully the situation is much improved with the recent TensorFlow 2.1-rc0 release. In my experiment, the adversarial regularized model now takes 4G of RAM when using TensorFlow 2.1-rc0, comparing to 12G using TensorFlow 2.0. The base EfficientNet model takes 3G of RAM regardless of TensorFlow versions.
I am closing this issue for now. Feel free to reopen it or create a new one if you need anything else.


Update for Halogen 5

Formless is not compatible with the release candidate for Halogen 5 and will prevent users and libraries that depend on Formless from upgrading when Halogen is released.

Looks great! Is there anything missing for halogen 5 support?

#46 closes this issue; I haven’t merged it because Halogen hasn’t had an official release yet, but since plenty of people are on the release candidate perhaps I should, and just tag it when Halogen 5 does have its official release.
In the meantime you can point at that branch and use this with Halogen 5 just fine. Halogen RealWorld does so.


When I off the internet connection of mobile, then send the push. After it, I on the internet connection. I didn't receive any push.

    final ApnsClient<SimpleApnsPushNotification> apnsClient = new ApnsClient<>(
            new File("C://Users//Ajay//Downloads//iphone_dev.p12"), "1234");

    final Future<Void> connectFuture = apnsClient.connect(ApnsClient.DEVELOPMENT_APNS_HOST);
    connectFuture.await();

    // Once we're connected, we can start sending push notifications.
    final SimpleApnsPushNotification pushNotification;

    {
        final ApnsPayloadBuilder payloadBuilder = new ApnsPayloadBuilder();
        payloadBuilder.setAlertBody("Example!");

        final String payload = payloadBuilder.buildWithDefaultMaximumLength();
        final String token = TokenUtil.sanitizeTokenString("yyyyyyyyyyyyyyyyyyyyyy");

        pushNotification = new SimpleApnsPushNotification(token, "com.example.myApp", payload);
    }

    final Future<PushNotificationResponse<SimpleApnsPushNotification>> sendNotificationFuture =
            apnsClient.sendNotification(pushNotification);

    try {
        final PushNotificationResponse<SimpleApnsPushNotification> pushNotificationReponse =
                sendNotificationFuture.get();
        if (pushNotificationReponse.isAccepted()) {
        	System.out.println("Message Send Sucessfully");
            // Everything worked! The notification was successfully sent to
            // and accepted by the gateway.
        } else {
            // Something went wrong; this should be considered a permanent
            // failure, and we shouldn't try to send the notification again.
            System.out.println("Notification rejected by the APNs gateway: " +
                    pushNotificationReponse.getRejectionReason());

            if (pushNotificationReponse.getTokenInvalidationTimestamp() != null) {
                // If we have an invalidation timestamp, we should also stop
                // trying to send notifications to the destination token (unless
                // it's been renewed somehow since the expiration timestamp).
            }
        }
    } catch (final ExecutionException e) {

        System.err.println("Failed to send push notification.");
        e.printStackTrace();

        if (e.getCause() instanceof ClientNotConnectedException) {

            apnsClient.getReconnectionFuture().await();
        }
    }
    final Future<Void> disconnectFuture = apnsClient.disconnect();
    disconnectFuture.await();

Result is -
[nioEventLoopGroup-2-1] INFO com.app42.push.apns.ApnsClient - Connected to api.development.push.apple.com/<IP_ADDRESS>:443.
com.app42.push.apns.ApnsClient - Disconnecting.
Message Send Sucessfully
But I didnt get notification on mobile

If the notification was accepted by the gateway, I'm not sure why it wouldn't have appeared on the destination device. I'd encourage you to take a look at this tech note from Apple, though, and particularly the parts about enabling push notification logging on your phone.
That said, it sounds like Pushy itself is doing its job as expected: it's sending notifications to the APNs server and reporting the server's reply. Since this does not appear to be a bug with Pushy itself, I'm going to close this issue.
If you need help for things other than bugs or feature requests, I'd encourage you to try the mailing list, which is much better suited for general support questions.
Cheers!


[BETA] ESM Orbit Camera

Renamed old orbit camera to legacy-orbit-camera.js
Added new orbit camera dervived from the multi-camera

Preview

Its likely that I will follow up this PR with the Multi-camera extending this class since they are very similar

Closing as new orbit camera example part of GH-7111


Allow color to also be an iterable

Currently, color has to be Optional[str]. I request that an iterable also be allowed for it, with the same length as x and y. This is intended to conveniently allow piecewise coloring. For example, I may want to sometimes specify color=['green', 'green', 'red', 'yellow'], etc.

Interesting idea. I don't think I've ever had to make each point in a plot a different color (without also needing labels for each, at which point you need to make each point a separate plotting call anyway). What's your use case?

That sounds pretty unique. If more people express interest in such a convenience feature, I will look into it. For now, you should be able to make what you want by plotting each color as a separate call with something like this:
temperature = np.array([16, 17, 18, 16, 15, 18, 21, 19])
going_up_indices = [0,1,2,5,6]
going_down_indices = [3,4,7]
fig = tplot.Figure()
fig.scatter(x=going_up_indices, y=temperature[going_up_indices], color="red")
fig.scatter(x=going_down_indices, y=temperature[going_down_indices], color="blue")
fig.show()

Doing it for a line plot is similar: define the color for each line segment (two subsequent temperature values) and plot each segment separately in a for loop.


what to do if it says "you have no power here"

its working on my other server but not on main one

what to do?

Read Wiki || #67

If you want to give other users ability to configure a bot, give them a role and place it's name in discord_bot_role.


Can't edit the Email TextInput

"prop-types": "^15.6.0",
"react": "16.0.0-beta.5",
"react-native": "0.49.1",
"react-redux": "^5.0.6",
"redux": "^3.7.2",
"redux-form": "^7.1.0"
The React Native - Redux Form Implementation Guide
3. Connect the form fields to the store using the Field wrapper
Following the guide, when i arrive at this step and add the 'value' property to my InputText, i can't edit it in my iOS Simulator and so my form doesn't work.
Anyone facing this issue ?

These might be related:

https://github.com/erikras/redux-form/pull/3460 (in v7.1.0)
https://github.com/erikras/redux-form/pull/3496 (merged today, not yet released)

Confirmation that one of these is the cause or solution would be much appreciated.

Unfortunately, i haven't found any issue nor solution related to mine in your links.
No error is triggered.
This is my form which is called by reduxForm
  <View style={styles.container}>
   <Text>Email:</Text>
   <Field name="email" component={renderInput} />
   <TouchableOpacity onPress={handleSubmit(submit)}>
     <Text style={styles.button}>Submit</Text>
   </TouchableOpacity>
 </View>

And this is what my component looks like
 const renderInput = ({ input: { value, onChange, onBlur, onFocus }}) => {
   return <TextInput
                 style={styles.input}
                 onChangeText={onChange}
                 onBlur={onBlur}
                 onFocus={onFocus}
                 value={value}
              />
}

The value property freezes the input and the onChangeText doesn't seems to work at all.

@erikras, I don't have this issue, if necessari I can provide a gist.
@LouisJS, you don't seem to be passing the onChange prop to

@jkomyno Hi. I'm passing the input.onChange method to TextInput with onChangeText.
It is such a simple case, i don't get why it doesn't work properly.
If there any chances you could provide a working example. That would help me understand my mistake.

Solved.
If it ever happens to you. You might have something wrong with your Redux Store setup (very hard to debug without Redux Dev Tools working on Simulator JS debugger)
Or checkout if your react-native is not too recent. (had to downgrade to 0.46.x <=