Mocha testing socketio middleware won't finish suite

Prerequisites

[x] Checked that your issue isn't already filed by cross referencing issues with the common mistake label
[x] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[x] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[x] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with:
node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend avoiding the use of globally installed Mocha.

Description
I am testing an nodejs-app with a server and a client component on nodejs 8.9.0 with mocha 4.0.1 and chai 4.1.2.
For mocha to end properly, I have to make sure that all socketio and http-servers are closed after the tests have been run. This works fine with normal tests, but as soon as I register a middleware to the socketio-server, the mocha-process won't close and stay open forever.
Steps to Reproduce
copy into a minimal test-file:
// test.spec.js
'use strict'

const Express = require('express')
const Http = require('http')
const io = require('socket.io')
const ioclient = require('socket.io-client')

const NODE_PORT = process.env.NODE_PORT || 3000

describe('Client', function () {
    beforeEach(() => {
        const express = new Express()
        this._http = Http.Server(express)
        this._socketio = io(this._http)
        this._http.listen(NODE_PORT)
    })

    // this test works perfectly, even when I copy it and run it
    // multiple times in this suite
    it('should connect to a socketio-server', (done) => {
        const client = ioclient.connect(`http://localhost:${NODE_PORT}`)
        this._socketio.on('connection', () => {
            client.close()
            done()
        })
    })

    // this test also finished, but the suite hangs afterwards - as if
    // a socket-client or socket-server was not closed properly.
    it('should finish the test suite even with a middleware', (done) => {
        const client = ioclient.connect(`http://localhost:${NODE_PORT}`)

        this._socketio.use((socket, next) => {
            return next()
        })

        this._socketio.on('connection', () => {
            client.close()
            done()
        })
    })

    afterEach(() => {
        this._socketio.close()
        this._http.close()
    })
})

Install dependencies: npm i<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Run the Testfile: npx mocha test.spec.js
Expected behavior: [What you expect to happen]
All tests to run, and mocha exiting properly
Actual behavior: [What actually happens]
All tests run, mocha does not exit
Reproduces how often: [What percentage of the time does it reproduce?]
100%
Versions
see above

Note: This is a cross-post from https://stackoverflow.com/q/47094615/898999

So, the problem was, that the server closed the client connection on a successful connection event. The client did not get any information on that, but instead saw a failed connection and tried to reconnect. This opened a socket to the server again and because the server was already closed, the connection error kept coming.
This behavior stopped node from properly destroying all objects, which in turn explaines the hanging. The solution is to call done() only after the client has declared a connection open, not after the server has declared a connection open like so:
'use strict'

const Express = require('express')
const Http = require('http')
const ioserver = require('socket.io')
const ioclient = require('socket.io-client')

const NODE_PORT = process.env.NODE_PORT || 3000

describe('Client', function () {
	beforeEach(() => {
		const express = new Express()
		this._http = Http.Server(express)
		this._ioserver = ioserver(this._http)
		this._http.listen(NODE_PORT)
		this._client = null
	})

	it('should connect to a socketio-server', (done) => {
		this._ioserver.on('connection', () => {
			done()
		})

		this._client = ioclient.connect(`http://localhost:${NODE_PORT}`)
	})

	it('should finish the test suite even with a middleware', (done) => {
		this._ioserver.use((socket, next) => {
			return next()
		})

		this._client = ioclient.connect(`http://localhost:${NODE_PORT}`)

		// if we wait for the server and kill the server socket,
		// the client will try to reconnect and won't be killed
		// by mocha.
		this._client.on('connect', () => {
			done()
		})
	})

	afterEach(() => {
		// this last call forces the client to stop connecting
		// even if tests failed
		this._client.close()

		this._ioserver.close()
		this._http.close()
	})
})


Set up CI/CD pipeline to external servers

We require a continuous integration pipeline for this project.
Ideally this should be done using GitHub actions rather than our Jenkins server.
Every new commit to "develop" should trigger a build and deploy to our QA server.

The build is automatically executed upon push to develop - however the deploy is not working correctly


Merge develop into master

Merge recent changes from #6 into master.

Meant to merge the other way around.


Question about socket and event in case of scenes

Hello
Thanks very much the Wiser Api ist perfect and very useful. I have please one question. Till now i dont know how to solve..
We have a button with a special scene.. lets say all off and you press before you leave the house
How is it possible to receive the event that this scene or button was pressed? Websocket delivers only information about on of dimm states i dont see one for pressing button.
Thanks very much

Since version 5.1.19 we support user scripts. The scripts can be executed by SmartButtons events. You will find examples on the scripts webpage. Maybe this will help you.

Thanks for answer.. but where exactly I can find this examples how to do my own scripts and add it as an action to be executed by smartbuttons? Did find nothing in the public normal api section.

Sorry for the "schäbig" explanation!

Go to the web interface of your µGW ("http://" or "http://wiser-XXXXXXXX.local")
Click on tab scripts
Click on button [Create File]
Click into text-box and choose "hue_scene.py"
Click on save button
Click on pencil button behind the script
Now you can customize the script for your use case
Maybe you only need the function onButtonEvent() with the call aiocurl()
Click on ✅ to save the file
Now you can click on  button [Link to SmartButton]
Choose your script
Choose Button Type "scene"
Click on  button [Link to SmartButton]
Press the SmartButton on the wall

Use this link to play efficiently with the API 😉


fix: @babel/preset-env v7.13.0 moved utils.js

@babel/preset-env moved lib/utils.js to lib/polyfills/utils
I think this patch should fix it,
Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Adjusted the corresponding import in @nuxt/babel-preset-app
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Nope .. this was more complex than I thought


Save and display summary stats from source-wide exports

Prerequisite: #373 (summary stats)
This applies to both percent coverage and calcification rate stats.

When a stats export is done over all images in the source, save the results to a database field. (Perhaps a field in the Source database table.)
Display the latest source-wide export statistics in a box on the source main page. Include the export date and number of images used in the stats computation, to give an idea of how up-to-date it is.
If no source-wide exports were done yet, we still display a small box on the source main page, indicating that export results would go there. This should help the discoverability of the export features.
In the export form, indicate whether or not the export stats will be saved, based on whether the current selection includes all images in the source.

The source main page display requires saving stats from past exports, because it takes a while to compute stats over many images, and we don't want to slow down the source main page significantly.
Since there are endless ways to filter a source to a subset of its images, I thought the simplest thing to do (at least for a start) would be to limit this save feature to the 'all images' case.

Decided not to do this task right away, since I wasn't entirely sure about the current performance of image covers / calcification rates exports, and wasn't sure if it'd be wise to add more incentive to do full-source exports yet. The performance situation might be better than I thought, but I'll err on the safe side for the moment.


Feature: Add connection config for GCP managed Postgres

GCP has a special connector that integrates their gcloud auth with their postgres managed service.

Will add those in a follow-on PR. Thanks for the review @eakmanrq !


Ruleset listing endpoint: return latest version only

This PR replaces https://github.com/heetch/regula/pull/57.
So the work is done in two steps:

first, we retrieve the path to the latest version for each ruleset
then, we retrieve the rulesets entries in a transaction

Fix #21.

Closing it in favor of https://github.com/heetch/regula/pull/60.


Improve QA checks: black, re-format code, commit message check, explain in docs

We've been spending a lot of time on code formatting during reviews lately.
In the past we discussed introducing black, I think we could start introducing it, I leave some notes here:

add it to the QA requirements
add it to the QA checks (ensure there's a way to skip the check)
add a way to trigger the reformatting using isort and black (eg: openwisp-qa-format), so we can format the code with one command (hence making it easy for new contributors)
update the python code convetions docs to suggest the use of openwisp-qa-format and openwisp-qa-check: http://openwisp.io/docs/developer/contributing.html#python-code-conventions
add a flag to force the commit message check and add the suggestion of using this feature in the guidelines at: http://openwisp.io/docs/developer/contributing.html#commit-message-style-guidelines

I'll try to provide a PR for this issue

[x] update the python code convetions docs to suggest the use of openwisp-qa-format and openwisp-qa-check

This is done in https://github.com/openwisp/openwisp2-docs/pull/101


Windows github action failing on create release step

Conditon create  release step on steps.publish_nuget.outputs.VERSION presence

fixed https://github.com/b3b00/csly/commit/bcf7a1ea7f49a3b2e5ebdb819e4354281ada4281

fixed https://github.com/b3b00/csly/commit/bcf7a1ea7f49a3b2e5ebdb819e4354281ada4281


Document that Swift increases app size significantly

With current beta 0.8.2 and latest dev, flutter build ios --release produces app which is 5 times the size of the app built with 0.7.2.
with 0.7.2 the size was around 22mb
with 0.8.2 the size is around 100mb
I've tested with basic flutter app by creating new flutter app and building it. Even basic app size is around 61mb
Steps to Reproduce

Create new flutter app.
Run flutter build ios --release
Check the size of the app

Images for reference
aot and app size

contents of the app

Logs
[✓] Flutter (Channel dev, v0.9.2, on Mac OS X 10.13.2 17C88, locale en-IN)
    • Flutter version 0.9.2 at /Users/hari/development/flutter
    • Framework revision 85b4670b2a (6 days ago), 2018-09-19 14:59:23 -0700
    • Engine revision 2e8e96fad1
    • Dart version 2.1.0-dev.4.0.flutter-4eb879133a

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/hari/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 28.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] VS Code (version 1.27.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.18.0

[!] Connected devices
    ! No devices available

! Doctor found issues in 1 category.

I've tested with 0.7.5 and 0.6.2. even this version created basic app of size 69mb which is higher than current beta. I do remember that the older beta was producing app with less size. is this expected? did something changed in the tools which is resulting in huge size of the app?
@zoechi could you please escalate this, as this effects app store downloads. I feel this should be top priority.

@goderbauer Any thoughts in how this might be related to your work?

This doesn't reproduce for me on the latest master (see below). Can you tell me what exactly you measured the size of?
~/dev/ $ flutter create size_test
<output omitted>
~/dev $ cd size_test/
~/dev/size_test $ flutter build ios --release
Building com.example.sizeTest for device (ios-release)...
Signing iOS app for device deployment using developer identity: "iPhone Developer: Micharl Goderbauer (KY4Q2ZN4ME)"
Starting Xcode build...
 ├─Building Dart code...                             18.6s
 ├─Generating dSYM file...                            0.8s
 ├─Stripping debug symbols...                         0.0s
 ├─Assembling Flutter resources...                    1.3s
 └─Compiling, linking and signing...                 10.5s

Xcode build done.                                           32.8s
Built /Users/goderbauer/dev/size_test/build/ios/iphoneos/Runner.app.
~/dev/size_test $ du -hs /Users/goderbauer/dev/size_test/build/ios/iphoneos/Runner.app
 24M	/Users/goderbauer/dev/size_test/build/ios/iphoneos/Runner.app

The produces app has a size of 24 MB, which seems about right.
Note: The size of Runner.app does not tell you anything about the download size your app will have on the store or the size it is going to occupy on user's devices. When downloaded from the store, Apple applies a process called app thinning that removes all artifacts and architectures from the app that are not needed for the target device. The inly way to determine the true app size is to upload the app to app store connect and see what numbers apple reports there.

@goderbauer sorry my bad. The app size is ~27mb for me with objc and ~59mb with swift. In screenshot above, swift libraries contribution was about half of the app size.
Would it be useful for anyone who is new to iOS, if we mention about swift effect on final app size somewhere in the flutter website?
thanks for the help

Yes, Swift apps are known to be bigger because they currently have to ship their own Swift runtime - this effect is unrelated to Flutter.
I'll turn this bug into a request to update documentation to mention that fact for Flutter as well.

@swingywc if you can switch to objective-c you can reduce your app size by almost 50%. App store also compresses your app before sending it to the target device, also it does app thinning to reduce the bytes downloaded on to the device. The size seems to be normal for swift apps.
for me, generated IPA file is around 100mb but the download size from the app store is around 45mb, this is a fat binary which supports both 32 and 64 bits.  If you can upload different IPAs for different architecture, which is possible, then the download size can further be reduced by about 35% to 40%. These numbers are based on my app sizes.

Issue moved to flutter/website #2011 via ZenHub


remove the unecessary header guard

It is not necessary to use the header guard because #import will ensure that the header is included at most once.
It also causes metamacro_xxx not found error when ReactiveObjC is imported by @import ReactiveObjC. I don't understand how objc modules work, so I cannot explain why it causes such errors.

Thanks!


support mysql8 cluster

hello, I use liquibase with mysql8 cluster,  start error, report DATABASECHANGELOG lack primary id. I check master branch, it still exist this problem
hope modify CreateDatabaseChangeLogTableGenerator like this
.addColumn("ID", DataTypeFactory.getInstance().fromDescription(charTypeName + "(" + getIdColumnSize() + ")", database), null, null, new NotNullConstraint())
->
.addPrimaryKeyColumn("ID", DataTypeFactory.getInstance().fromDescription(charTypeName + "(" + getIdColumnSize() + ")", database), null, null, null, new NotNullConstraint())
now, I extend CreateDatabaseChangeLogTableGenerator to do this, tnks!

@little-circle-circle I am not following your issue. I think you are requesting we enhance the product to have a primary key in DATABASECHANGELOG - is that what you are requesting? Thanks.

@kataggart  yes, tables without a primary key can not be successfully created in MySQL Cluster, tnks.

I've experienced this issue far downstream as well. keycloak is using Liquidebase, and the missing primary key breaks kubernetes deployments into MySQL. This goes all the way back to https://liquibase.jira.com/browse/CORE-1909.


daemon: improve error message when SocketLB is required

Clarify that the SocketLB is only required for the combination of overlay routing and BPF masquerade. And be more helpful by saying what value needs to be set for the --bpf-lb-sock option.

The amazing @brb removed this whole chunk in https://github.com/cilium/cilium/pull/33728.


Possible Race Condition Loading in RTE

I have an application where the LabVIEW APIs work fine in the IDE but not as a built application.
My suspicion is that the ctor functions run too early to catch the labview symbols and we don't have a second shot.
I'm going to look to replace the ctor create with the new LazyLock API. This does require a rather high msrv of 1.80 but at this stage of the crate I think that is probably OK for the future benefit of using std functions.

Can confirm your original observation. Have you verified that the refactoring to LazyLock solves the issue?

No turns out it wasn't a race issue, the loading method for the APIs in the IDE doesn't work in the runtime engine. I'm just about to merge a fix for that.


Add BitBucket support to the Commit Time Exporter

Enhance the commit exporter to support BitBucket

Duplicate of #106
Closing.


it has no PK

I have been having this issue for months on both Android and iOS using Xamarin.  The table has a primary key.  I have checked the code definition and it list a primary key field.  I have checked the database on the physical phone and the scheme shows a primary key with auto increment.  The table very large about 2000 records.  Any ideas on how to fix this issue?   The code below is from a de-compiled version on iOS in release mode.
    [AutoIncrement]
    [PrimaryKey]
    public int Id
    {
        [MethodImpl(8)]
        get
        {
        }
        [MethodImpl(8)]
        set
        {
        }
    }

Very strange! When do you receive the error? When you call CreateTable? Or when you Insert?

It is when I call delete or deleteall. happens 3x more on iOS than Android.  Get same behavior outside a transaction. Unsure why this error being reported daily.

You have to be careful with Delete -- there are two overloads with pretty different behavior. I've been meaning to rename one of the overloads to clarify. Would you be willing to post the lines of code with the Delete?

I post the code later or tomorrow when I have access to it.

if (dirtyEvents != null)
{
	foreach (var item in dirtyEvents)
	{
              // this line is throwing the Exception.
		obj.Delete(obj.Table<shared.Tables.Events>().Where(x => x.LinkID == item));
	}
}

if (newEvents != null && newEvents.Count > 0)
{
	hasData = true;
	foreach (var item in newEvents)
	{
		obj.Insert(new shared.Tables.Events
		{
			LinkID = item.LinkID,
			AllDayCell = item.AllDayCell,
			End = item.End,
			EventType = item.EventType,
			LongTitle = item.LongTitle,
			ShortTitle = item.ShortTitle,
			Start = item.Start,
			Text1 = item.Text1,
			Text2 = item.Text2,
			AllDay = item.AllDay,
			SearchText = item.ShortTitle.ToLower()
		});
	}
}

The Table:

using System;
using SQLite;
using Xamarin.Forms.Internals;

namespace shared.Tables
{
    [Preserve]
    [Table("Events")]
	public class Events
	{
		public Events ()
		{

		}

        [PrimaryKey, AutoIncrement]
        public int Id { get; set; }

		/// <summary>
		/// PSEA Database ID
		/// </summary>
		/// <value>The type of the event.</value>
		public Int32 LinkID { get; set; }

		/// <summary>
		/// Event Type (Local - L, Region - R, StateWide - S)
		/// </summary>
		/// <value>The type of the event.</value>
		public string EventType { get; set; }

		/// <summary>
		/// Event Short Title (Calendar Title) .max 10
		/// </summary>
		/// <value>The title.</value>
		public string ShortTitle {get;set;}

		/// <summary>
		/// Calendar Search String (make lowercase)
		/// </summary>
		/// <value>The search text.</value>
		public string SearchText {get;set;}

		/// <summary>
		/// Event ListView Title
		/// </summary>
		/// <value>The title.</value>
		public string LongTitle {get;set;}

		/// <summary>
		/// Event Start
		/// </summary>
		/// <value>The start.</value>
		public DateTime Start {get;set;}

		/// <summary>
		/// Event End
		/// </summary>
		/// <value>The end.</value>
		public DateTime End {get;set;}

		/// <summary>
		/// All Day Event
		/// </summary>
		/// <value><c>true</c> if all day; otherwise, <c>false</c>.</value>
		public bool AllDay {get;set;}

		/// <summary>
		/// All Day Cell Event
		/// </summary>
		public bool AllDayCell { get; set; }

		/// <summary>
		/// addition field
		/// </summary>
		/// <value><c>true</c> if all day; otherwise, <c>false</c>.</value>
		public string Text1 {get;set;}

		/// <summary>
		/// addition field
		/// </summary>
		/// <value><c>true</c> if all day; otherwise, <c>false</c>.</value>
		public string Text2 {get;set;}

		[Ignore]
		public Xamarin.Forms.Color EventToColor
		{
			get {
				if (String.IsNullOrEmpty (EventType) == false) {

					var u = EventType.ToUpper ().Trim ();
					if (u == "S") {
						return Xamarin.Forms.Color.Blue;
					} else if (u == "L") {
						return Xamarin.Forms.Color.Green;
					} else if (u == "R") {
						return Xamarin.Forms.Color.FromHex ("A52A2A");
					}
				}
				return Xamarin.Forms.Color.Blue;
			}
		}
	}
}

Ah, I see the error: you are using Delete incorrectly.
Delete takes an object to delete. You are passing it a query:
obj.Delete(obj.Table<shared.Tables.Events>().Where(x => x.LinkID == item));

Should be:
obj.Table<shared.Tables.Events>().Delete(x => x.LinkID == item);

Soon you will also be able to write:
obj.Table<shared.Tables.Events>().Where(x => x.LinkID == item).Delete();

Thanks for the fix. Did not know I was doing it wrong.

You should better document these things. :)


Use __attribute((malloc)) not __attribute((malloc)) in macros.h

#   define OSSL_CRYPTO_ALLOC __attribute__((malloc)) in include/openssl/macros.h interacts badly with anything that defines malloc like cmocka's cmocka.h .  GCC recommends the use of __attribute__((__malloc__)) in header files to  prevent this issue https://gcc.gnu.org/onlinedocs/gcc-4.7.2/gcc/Function-Attributes.html.
You may also specify attributes with '__' preceding and following each keyword. This allows you to use them in header files without being concerned about a possible macro of the same name. For example, you may use __noreturn__ instead of noreturn.
% grep malloc /opt/local/include/cmocka.h
 *      int *answer = malloc(sizeof(int));
 *      int *answer = malloc(sizeof(int));
 * tested by cmocka should replace calls to malloc(), calloc() and free() to
 * test_malloc(), test_calloc() and test_free() respectively. Each time a block
 * @brief Test function overriding malloc.
 * extern void* _test_malloc(const size_t size, const char* file, const int line);
 * #define malloc(size) _test_malloc(size, __FILE__, __LINE__)
 *     int * const temporary = (int*)malloc(sizeof(int));
 * @see malloc(3)
void *test_malloc(size_t size);
#define test_malloc(size) _test_malloc(size, __FILE__, __LINE__)
/* Redirect malloc, calloc and free to the unit test allocators. */
#define malloc test_malloc
void* _test_malloc(const size_t size, const char* file, const int line);
%

This is regression in 3.1.

Hi @Anis-cpu-13,
I'm no maintainer, but submitting OpenSSL's CLA is a good start for contributing non-trivial stuff.
Once you have yours submitted, go ahead and work on any issue you think you are fit for. If it is not anything big (which stuff marked with "good first issue" is not), you can simply do the work without prior consultation and then open a pull-request.
If the maintainers do have problems with your commits, they will just tell you and you can either argue for your cause or do the necessary changes to comply.
Don't forget to mention the issue your pull-request solves in the description so it can be closed :)

This would likely fall under the trivial clause.  Submit a pull request and we'll decide.  Or better start with your ICLA...

Thanks a bunch for the useful tips on getting started with contributing to the OpenSSL project! I really appreciate you taking the time to share this info. I'll be sure to follow your suggestions, like submitting the CLA and mentioning the issue in my pull-request.
Can't wait to dive in and learn from this experience! Thanks again for your help, mate

Fix in #20869


Update the theme of scrollbar to be consistent on other pages

Description
Update the theme of scrollbar to be consistent on other pages as the homepage and also make it more visible on light theme.
Screenshots

Enviroment:

OS: Mac Linux Windows
Browser: Chrome Safari Firefox
Version:
Device: Desktop Mobile

Contributor Guides and Handbook
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

📚 See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

please assign me this issue @Savio629

Is anyone working on this issue @Savio629 , if not then I would like to be assigned to work on this.

@ranjanmangla1 As you are working some other issues, let @Faizan711 take up this issue 😄

@Faizan711 Please let us know of your progress so far

@vishalvivekm I have changed the scroll bar for home and catalog page, more pages left.
Will raise a PR when whole site is done.

Closed as completed via https://github.com/meshery/meshery.io/pull/1361


Support hanami (ruby)

https://github.com/hanami

https://guides.hanamirb.org/v2.0/routing/overview/


Pock is still running but not working after waked up from sleep.

Describe the bug
After macbook waked up from sleep. Pock is running but nothing happen with the touch bar, need to manually quit then relaunch to fix.
How to reproduce
Steps to reproduce the behavior:
1. Open pock everything works fine.
2. Close macbook lip let it go into deep sleep.
3. Open lip , login.
4. Pock is still running, the icon show up on status bar, configuration work fine, but the touch bar is not modified.
5. Quit pock, and launch it again fix the problem.

Expected behavior
Pock work after mac waked up from sleep.
Versions:

macOS version: 10.15
Pock version:  0.6.2

Hi, could you please provide more information about your Touch Bar and Pock setup?

All my settings is at default, only Launch  at login checked, i noticed that this isn't only way happening, sometimes it works fine after wake up, sometimes it doesn't.

I have the same problem.

I would give that issue a +1, although I can not provide more information.
Pock Version: 0.7.2
macOS: 10.14.6 (18G103)
Preferences:

Thanks for your great work!


Cannot Create User on Initial Setup

I've managed to launch the application via docker, but there doesn't seem to be any support to initially sign in, or create a new user. How is someone supposed to set up users for this application?

I experience the same thing
I have successfully installed and am presented with the login screen
I have tried the logins mentioned in the readme but I'm told they are invalid login credentials
@laxsrbija Can you advise?


added new Contributor

please let me know if any changes are required thanks  fixes.   #153

@rithviknishad hey please check if any changes are required thanks.

@rhithesh any updates on this?

Closing this PR as inactive


WPFTweaksLoc

Why would you do
https://github.com/ChrisTitusTech/winutil/blob/2354645b479b9ca7bcf377ea4e199b262f4b6bb4/config/tweaks.json#L80-L116
When you can do
if((Test-Path -LiteralPath "HKLM:\Software\Policies\Microsoft\Windows\LocationAndSensors") -ne $true) {  New-Item "HKLM:\Software\Policies\Microsoft\Windows\LocationAndSensors" -force -ea SilentlyContinue };
New-ItemProperty -LiteralPath 'HKLM:\Software\Policies\Microsoft\Windows\LocationAndSensors' -Name 'DisableLocation' -Value 1 -PropertyType DWord -Force -ea SilentlyContinue;

TLDR: Maintainability and Error Handling & Logging and Scalability
Maintainability:
It's easier to add/modify a new registry key if all the information you need to provide is simply supplied via an Object containing the Path, Name, Type, Value, and Original Value. Those values can then be used dynamically as a function argument to set the values. It is also way easier for code review to check if a new change e.g. a new registry key makes sense and all the values are provided if you simply have to check if the provided values are correct without needing to verify all the logic around changing or removing a key for every change.
Error Handling & Logging
The object of the registry key is handed over to a function that handles the actual changing of the key. The function (Set-WinUtilRegistry) works the same for every registry key object in the script and does some error handling as well, like checking if the key even exists. Also, the central function writes output in a standardized way, so if you ever want to change the logic on how to set or remove registry keys, you would only have to edit a single file instead of every occurrence of a registry key change individually.


Roll Engine from 1d537824d6d2 to 0561a988e3b1 (36 revisions)

Roll Engine from 1d537824d6d2 to 0561a988e3b1 (36 revisions)
https://github.com/flutter/engine/compare/1d537824d6d2...0561a988e3b1
2021-02-15<EMAIL_ADDRESS>Add missing header guard, namespace (flutter/engine#24423)
2021-02-15<EMAIL_ADDRESS>Roll Dart SDK from 648d5f951915 to 39dc63439396 (1 revision) (flutter/engine#24421)
2021-02-15<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from JmB58d-3R... to KKD5xVYLO... (flutter/engine#24420)
2021-02-15<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from OHm_snJzB... to H0v5Ox4Vs... (flutter/engine#24419)
2021-02-15<EMAIL_ADDRESS>Roll Skia from 56a8fbd21ee2 to 7cb0f6e9702a (1 revision) (flutter/engine#24418)
2021-02-15<EMAIL_ADDRESS>Roll Dart SDK from c9b47adb178e to 648d5f951915 (1 revision) (flutter/engine#24416)
2021-02-15<EMAIL_ADDRESS>Roll Skia from b44fbb3392f8 to 56a8fbd21ee2 (3 revisions) (flutter/engine#24415)
2021-02-15<EMAIL_ADDRESS>Roll Skia from c3e152b9f795 to b44fbb3392f8 (3 revisions) (flutter/engine#24414)
2021-02-15<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from uJ8jPHroy... to JmB58d-3R... (flutter/engine#24411)
2021-02-15<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from NQL9o1B8n... to OHm_snJzB... (flutter/engine#24410)
2021-02-15<EMAIL_ADDRESS>Roll Dart SDK from 56fa015ca3ec to c9b47adb178e (1 revision) (flutter/engine#24408)
2021-02-15<EMAIL_ADDRESS>Roll Skia from 7a2ff98ce68b to c3e152b9f795 (2 revisions) (flutter/engine#24404)
2021-02-15<EMAIL_ADDRESS>Revert "Roll buildroot to e7857d269381e23d1a89a3cbf017854992bda15a (#24399)"
2021-02-14<EMAIL_ADDRESS>Roll buildroot to e7857d269381e23d1a89a3cbf017854992bda15a (flutter/engine#24399)
2021-02-14<EMAIL_ADDRESS>Roll Skia from 554aabbaa8e8 to 7a2ff98ce68b (1 revision) (flutter/engine#24403)
2021-02-14<EMAIL_ADDRESS>Roll Dart SDK from 4c8147b7b41e to 56fa015ca3ec (1 revision) (flutter/engine#24402)
2021-02-13<EMAIL_ADDRESS>Roll Skia from 4f065e286df6 to 554aabbaa8e8 (1 revision) (flutter/engine#24398)
2021-02-13<EMAIL_ADDRESS>Roll Dart SDK from e9693f1b2401 to 4c8147b7b41e (1 revision) (flutter/engine#24397)
2021-02-13<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from yyQmi032P... to NQL9o1B8n... (flutter/engine#24396)
2021-02-13<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from AjjJVX_95... to uJ8jPHroy... (flutter/engine#24395)
2021-02-13<EMAIL_ADDRESS>Roll Dart SDK from 6eafd1856eb5 to e9693f1b2401 (1 revision) (flutter/engine#24394)
2021-02-13<EMAIL_ADDRESS>Windows: linker compatibility with AppContainer for winuwp target (flutter/engine#24318)
2021-02-13<EMAIL_ADDRESS>Roll Skia from ba4c3086ba80 to 4f065e286df6 (2 revisions) (flutter/engine#24393)
2021-02-13<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from WIqUEANCH... to yyQmi032P... (flutter/engine#24391)
2021-02-13<EMAIL_ADDRESS>Roll Dart SDK from add1eac3622c to 6eafd1856eb5 (1 revision) (flutter/engine#24390)
2021-02-13<EMAIL_ADDRESS>Roll Skia from 6e73404a782c to ba4c3086ba80 (3 revisions) (flutter/engine#24388)
2021-02-13<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from YMY-cajLa... to AjjJVX_95... (flutter/engine#24381)
2021-02-13<EMAIL_ADDRESS>[canvaskit] fix Path.from (flutter/engine#24382)
2021-02-12<EMAIL_ADDRESS>Roll Skia from 2494942f0d8d to 6e73404a782c (3 revisions) (flutter/engine#24383)
2021-02-12<EMAIL_ADDRESS>Roll Skia from 3354f8c4f8cb to 2494942f0d8d (4 revisions) (flutter/engine#24380)
2021-02-12<EMAIL_ADDRESS>Fix vulkan surface leaks. (flutter/engine#24372)
2021-02-12<EMAIL_ADDRESS>Roll Skia from 28698696f1b3 to 3354f8c4f8cb (2 revisions) (flutter/engine#24378)
2021-02-12<EMAIL_ADDRESS>Roll Dart SDK from 1e0fec6e48cf to add1eac3622c (1 revision) (flutter/engine#24375)
2021-02-12<EMAIL_ADDRESS>[canvaskit] support adding leaf layers w/o container layers (flutter/engine#24357)
2021-02-12<EMAIL_ADDRESS>Roll Skia from ec24154521f3 to 28698696f1b3 (4 revisions) (flutter/engine#24374)
2021-02-12<EMAIL_ADDRESS>Replace Flutter surface only after all platform views are destroyed (flutter/engine#24363)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from YMY-cajLa0mb to KKD5xVYLOuDq
fuchsia/sdk/core/mac-amd64 from WIqUEANCHaur to H0v5Ox4Vs2Ys
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
...

Commit queue failed; closing this roll.


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #31.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #31

@conda-forge-admin, please rerender


feat(build): upgrade deps

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Initial version of Ramen motivation

Signed-off-by: Veera Deenadhayalan<EMAIL_ADDRESS>

Closing older PRs that are not applicable at present (or stale). Reopen if required.


[BEAM-2774] Added loose failure mode to allow individual VCF record reads to fail

Added configuration parameter allow_malformed_records that allows reads for records to be logged and returned as MalformedVcfRecord objects so that the pipeline can continue if some records are malformed.
@chamikaramj, @jkff can you please take a look?

@chamikaramj Thanks for reviewing! Please take another look when you get the chance.

@chamikaramj thanks for reviewing! Please take another look when you get the chance.

LGTM. Thanks.

Run Python PreCommit

Run Python PreCommit


[BEAM-9679] Add Partition task to Core Transform katas

This pull requests adds a Partition lesson to the Go SDK katas.  I would like to request the following reviewers:
(R: @lostluck )
(R: @henryken )
If accepted by both reviewers, please wait until the Stepik course is updated before finally merging this PR.

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

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

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Samza
Spark

Go

---
---

---

Java

Python

---

---

XLang
---
---
---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@henryken should I go ahead and update the Stepik course?

@lostluck / @henryken Thank you for reviewing.  I'll update the stepik course and provide the updated *-remote.yaml files in the next PR planned for Side Input.

I am surprised to see a 859 line PR for what amounts to a 20-line example accompanied by about 20 lines of docs. Is there some way to have less auto-generated code checked in?

I can't speak to the Kata generated code, but the go.mod and go.sum files are strongly recommended to be submitted as is to ensure  full transitive dependency of the right and correct package versions are included. They aren't intended for human consumption for the most part.

IntelliJ Edu autogenerated a go.mod in each lesson so that they are independently runnable when the course files are provided to the user. I had been thinking of deviating from IntelliJ Edu's method and instead just having one go.mod in learning/katas/go.  Should I start another pull request if I'm successful at this?

The Kata generated metadata files are important and only contain few lines of code.
What I understand is that each lesson is run like a mini Go project/module.
@damondouglas, you can give a try having one go.mod file for all lessons, if it's possible.

@henryken / @lostluck I've started work on consolidating the module by beginning work at this branch: https://github.com/apache/beam/compare/master...damondouglas:BEAM-10428-single-module-kata-go
Essentially the final goal is to make one go.mod at the root of learning/katas/go.  So far I've only changed the Hello Beam section so you can preview what I intend for the rest.
Would it make sense to start a PR for this branch without the intent to merge initially?

That works fine. Given how mechanical the change is, it wouldn't be unreasonable to do the change all at once in one PR.
However, I do recommend doing all the directory renames in separate commits from the deleting the old mods so that we can check the relatively smaller changes on a per commit basis (rather than just all at once). Won't matter to the final merge, since it will get squashed, but it will make it easier to review.


[release-4.17] OCPBUGS-38483: Bump to k8s.io v0.30.3, controller-runtime v0.18.4

This is a manual cherry-pick of https://github.com/openshift/coredns-ocp-dnsnameresolver/pull/15.

/assign @rfredette

This k8s.io and controller-runtime bumps are required for compatibility with other components in 4.17.  The risk is higher without this bump than with it.
/label backport-risk-assessed

/label qe-approved
/label cherry-pick-approved

/jira refresh

No changes from original #15 and all tests passed.
/approved
/lgtm

/approve


How to get started

Hi there,
I am new to app building and the software world. I know how to code and all but haven't tried end-to-end yet. Can you pls tell me how and where I can run this so that I can get started in doing some edits of my own?

Follow the Flutter official documentation. It will help you set up a flutter SDK on your PC, then try to make an app on your own which is given in the documentation itself, and then fork this repository.


Resolve the issue for "adding a version control for excel file and the regression testing for a excel file"(#2278)

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
#2278. [TASK] Add a version control for excel file and the regression testing for a excel file
What are the significant changes in functionality due to this change request?
Two separated folders are included under Raven/tests/reg_self_tests
/D_01_Excel_Version_Control_Tool
/D_02_Excel_Python_Regression_Testing_Tool
There is probably a need to place these folders in a appropriate location.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

The new pull request has been created and this one is removed.


Support querying olap table with StarOSTablet

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] others

Problem Summary(Required) ：

Select the all replicas in olap scan node for query plan.
The information of all replicas is obtained from StarOS.

[FE PR Coverage check]
:disappointed: fail : 3 / 17 (17.65%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/catalog/StarOSTablet.java
0
7
00.00%

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
2
8
25.00%

:large_blue_circle:
com/starrocks/sql/optimizer/Utils.java
1
2
50.00%

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 11 / 18 (61.11%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
2
8
25.00%

:large_blue_circle:
com/starrocks/sql/optimizer/Utils.java
1
2
50.00%

:large_blue_circle:
com/starrocks/catalog/StarOSTablet.java
7
7
100.00%

:large_blue_circle:
com/starrocks/catalog/LocalTablet.java
1
1
100.00%

[FE PR Coverage check]
:disappointed: fail : 14 / 27 (51.85%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/planner/OlapScanNode.java
2
8
25.00%

:large_blue_circle:
com/starrocks/planner/MetaScanNode.java
3
9
33.33%

:large_blue_circle:
com/starrocks/sql/optimizer/Utils.java
1
2
50.00%

:large_blue_circle:
com/starrocks/catalog/StarOSTablet.java
7
7
100.00%

:large_blue_circle:
com/starrocks/catalog/LocalTablet.java
1
1
100.00%


fix story saving for PBS

Resolves
This fixes https://github.com/LLK/scratchjr-private/issues/597
Proposed Changes
make currentProject can be set outside of ScratchJr.js
Reason for Changes
Without this setter method, we couldn't save the story project
Test Coverage

[x] don't save story project without changing
[x] Works well for PBS version
[x] lint the code

[ ] Don't save story starters project if not changed.
[ ] Don't save story starters project if only the title changed.
[ ] Can't share the story starters if not changed.
[ ] Can close the info dialog after open

There is another small issue that if you set the name to Surprise and return to the lobby, the name will be Surprise 1 and it should be Surprise.


iOS device doesn't receive the push from my dashboard

Issue Description

I follow the instruction https://github.com/ParsePlatform/parse-server/wiki/Push to setup my parse server

I follow the instruction https://github.com/ParsePlatform/PushTutorial/blob/master/iOS/README.md#1-creating-the-ssl-certificate to setup my iOS code

The log in dashboard

There was a warning about: 'warning: error while sending push'
I don't know where is going wrong.
iOS device doesn't receive the push form my dashboard.

iOS version: 10.1.1
device: iphone 7 plus

Environment Setup

Server

parse-server version  : 2.2.13
Operating System:    unbuntu
Remote server? : Aliyun

Database

MongoDB version: 3.2.9
Storage engine:  wiredTiger
Remote server? : Aliyun

Logs/Trace
I set the VERBOSE=1 to get this log

use distribution certificate

You should probably run with VERBOSE=1 and probably check the documentation of node apn as this is the underlying module that sends the push notifications


Test stale issue

Describe the issue you're reporting
Testing an action that reacts to an issue being given the Stale label.

/never-stale

/never-stale

/never-stale


Dialog preferences not saved

Hi,
It seems that the "don't show this again" preference for popup dialogs are not saved.
I just tried with a fresh install, and I can't manage to make those preferences stick (for example the popup when going to the advanced setting keep showing up even if I had check the box previously).

Do you mean that preferences are not saved after you restart the container?

Just the checkbox setting to show again (or not) the popups (like the one presented when you go to the advanced settings). If I go to the advanced settings, the popup appear, I check the "don't show this again", go back to another part of the settings (like 'general') come back to the advances settings, the popup appear again with the "don't show this again" unchecked. And it's the same for every popup dliagos (like the one that appear when you click on 'add new link' in the new grabber, or the one when forcing a download to start, etc.). Other settings are saved correctmly, but not the preference to show again or not the popups.

How did you created the container?
I just did a quick test by creating the container like this:
docker run --rm -p 5800:5800 jlesage/jdownloader-2

I access the advanced settings, get the the popup.  I make sure to check "don't show this again".  I go back to general settings then come back to advanced settings and I don't get the popup.

It's on un Unraid server via the template via the community apps.


Marketplace: broken endless scrolling

When viewing my profile, on a large-enough or zoomed-out-enough screen I can't see all my extensions because the endless scrolling algorithm is broken.
I guess it assumes a scroll event will happen, which is not always the case, and it doesn't check if a new page should be loaded.

Thanks @fabiospampinato for reporting this. We are working on the fix.

Hi @fabiospampinato - this issue should have been fixed (actually a few weeks back but I missed updating this thread). Can you please confirm this?

I no longer see the problem, thanks.


[Help] I've created a new category, but posts dont display on the category view.

Is there some kind of cache or something? Why wouldn't it display?
No errors in console...

I have this problem too @amouratoglou
Can you let me know what the solution is?
Thanks in advance


SSM model integration instead of Flash Attention

Hi, very nice work!
How easy is it to integrate SSM state space model here instead of Flash Attention?
Would it even help in anything? They claim they have O(N) complexity while this has O(N**2), the question is whether this can even outperform the existing approach with Flash Attention memory bank.

Hi, the idea is promising. SSM has lower computation complexity than our attention block. However, I think there should be a lot of effort and specific design for the memory bank.


what's means depth = 0.33?

Hello community,
What is mean depth = 1.0 or depth = 0.33?

It means scale factor for depth/width of Network, you might refer this code, we will add docstring later.


[PE-D][Tester E] Add a glossary to the UG.

The User Guide uses quite a lot of technical jargon to explain commands. I assume that your application is curtailed for all prospective job and internship applicants.
Users that are not from SOC (and I would argue that some students in SOC) may find the technical jargon very confusing to refer to and this may impede the User Guide's utility as a quick and easy guide for users.
I suggest adding a glossary to make the features and explaining the technical jargon for users to make it easier for them to understand the commands and some terms.

Labels: severity.Medium type.DocumentationBug
original: proto-aiken-13/ped#4

Closed by #163


Weird tx history and balance when force-resyncing

Context
The force-resync endpoint allows artificially rolling back the wallet to (SlotId 0 0).

Information
-

Version
includes the #1559 fix

Platform
mac

Installation
Official FC with cardano-wallet manually replaced

Steps to Reproduce

Launch Daedalus FC with a wallet

curl  -vX PUT https://localhost:50078/v2/byron-wallets/021028cf0b109aab8f4fde7dd124a89efeeb4ed8/tip \
       -H "Content-Type: application/json; charset=utf-8" \
       -d '{ "slot_number": 0, "epoch_number": 0 }' --http1.1 --cert $TLS/client.pem --cacert $TLS/ca.crt  |  jq

Expected behavior

I'd expect:

All tx history disappears

But otherwise:
2. All of the tx history becomes pending but otherwise don't change
And always:
3. Tx history should be in sync with the wallet balance
4. The wallet balance should (here) be 0 immediately after force-resyncing
Actual behavior

Incoming txs disappear from history. Outgoing txs are turned to pending.
The pending outgoing txs have the wrong dates
The wallet balance does not match the tx history
The wallet balance is not 0 at the start of the chain. The balance decreases to become more accurate while syncing, despite the wallet essentially only having incoming transactions.

Resolution

QA

Incoming txs disappear from history. Outgoing txs are turned to pending.

This is the actual expected behavior. When rolling back, incoming txs are forgotten (and will eventually be re-discovered). On the contrary, outgoing txs may still be in a mempool and be accepted somewhere. Possibly, they'll end up invalid because they reference a no-longer existing UTxO. But it is so far up to users to cancel them.
Later, we will add automatic re-submission of transaction, using a transaction scheduler.

The pending outgoing txs have the wrong dates.
This is also expected. The transaction is becoming pending at the precise time the rollback happened.

The wallet balance does not match the tx history

The wallet balance is not 0 at the start of the chain. The balance decreases to become more accurate while syncing, despite the wallet essentially only having incoming transactions.

These two are actually much more interesting and do not look right. Although there are still pending transaction in the history, the checkpoint UTxOs should still be correct and so should the balance.

The funds I see must be the change from the pending transactions: https://github.com/input-output-hk/cardano-wallet/blob/5333afc469294bfa74e07de3317578545661ea05/lib/core/src/Cardano/Wallet/Primitive/Model.hs#L321
Wondering:
Say we rollback a tx1, and spend its inputs on the new fork in tx2. tx1 is now invalid. Will tx1 still be pending, and will its change incorrectly contribute to the available balance?

Yes. Or at least, in Daedalus, because Daedalus shows the total balance, not the available one :/

A similar-looking occurrence on the ITN without force-resync: https://input-output-rnd.slack.com/archives/CDM8B0AN9/p1591216845381400

@Anviking The logic for keeping track of submitted transactions has changed a while ago. Is this issue still present?

@HeinrichApfelmus It's difficult to test, both because I currently don't have a local synced node, and because the force-resync endpoint has been removed. Do you have an idea of how the effects of the changes in this context?
Off the top of my head:

Force-resyncing has been removed, but perhaps something similar will be introduced in the future for DB migration. I'm not sure how the DB changes would interact with this.
I imagine multiple pending txs spending the same TxIns still could lead to misleading (total) balances

the force-resync endpoint has been removed.

I see. In this case, I would simply like to close the issue in order to tidy up our issue list.

but perhaps something similar will be introduced in the future for DB migration.

If we were to reintroduce this functionality again, I think the correct solution is to treat it as a rollback. The DB is now almost fully subservient to the WalletState data structure, where such behavior can be reasoned about.

Say we rollback a tx1, and spend its inputs on the new fork in tx2. tx1 is now invalid. Will tx1 still be pending, and will its change incorrectly contribute to the available balance?

I think the available balance is computed from the total balance by subtracting unspent outputs can are inputs to pending transactions. This means that if an output has already been spent, it will not be subtract again.

Fair enough, though I'm not convinced something similar could not happen again through long rollbacks, real or artificial ones.

@Anviking and I have discussed this in a call and came to the following conclusion:

In the presence of rollbacks, the definition of totalBalance actually needs to be adapted to account for sequence of pending transactions that spend each other's outputs. This is discussion in Section 7 of the wallet specification.
In the presence of expiration dates and rollbacks, totalBalance ceases to make sense — it becomes possible to create pending transactions that attempt to spend the same outputs even though each transaction spends from the available balance at the time of submission. Expiration dates are mentioned in Section 10.4 of the wallet specification, but no full treatment is given.

After some thinking, I believe that a new notion of "minimum anticipated balance" in lieu of "total balance" might be suitable. I have created ticket ADP-3106 to revisit this.


feat(viem): add withdrawSuperchainWETH

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#540  👈
#539
#538
main

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

Join @tremarkley and the rest of your teammates on  Graphite


CSS squeezing

Do you know where cssqueeze lives?

You mean https://github.com/ianbogda/CSSqueeze ? A friend of mine wrote it, but I can't tell you anything more about it, you should ask there...

Thank you!


test against a real flagd setup

We should have a pre-release check which will spin up a flagd provider and run some java code to validate that the thing actually works. I don't know if this is quick enough to be part of the PR flow (ideal), but it should at least happen before we release new versions.

cc @toddbaert

I will take this one (though likely I won't get to it until next week).
I will implement something like what I researched here, centralizing as much as possible.

Resolved with https://github.com/open-feature/java-sdk/pull/77


[ WIP ] Task 3 Tile builder

@smburdick  do you know what Gian's GitHub is? I was going to update this myself to help me catch up and jump in with the current progress, but it might be better if he writes it up.

@smburdick  do you know what Gian's GitHub is? I was going to update this myself to help me catch up and jump in with the current progress, but it might be better if he writes it up.

https://github.com/giangiac

We should add more details to the task flow to specify the user stories.


recursive relations are making the admin slow

the recursive relations in the models are slowing things down.  Need to balance ui need to select an Item against need for data in the site.

Create save() method on Item, create Link object with deep_id, title and whatever else needed.  Then add relations to Links not self


Modify Install directions to include permissions rewriting

Hey, awesome project, really impressed. Some install feedback that could go a long way cause I was getting really frustrated -- indicate that grav needs ownership / write access to itself in order to work. This is currently in the docs but it's buried in the "other problems" area. The directions for someone to do a self install, git pull down or zip upload is almost always going to give ownership permissions to the person putting the files up as opposed to apache/iis/etc.
With your bash prompt copy and paste directions for installations something like:

You may not have to but it wouldn't hurt to make sure permissions to correct, as a step right after upload
chmod 775 -R gravcms
chown ${USER}:apache -R gravcms

At minimum a direct link to the part of the docs from right after install about needing to change permissions.

That step is quite specific to your setup.  On most shared hosting setups, the user account  is the same as the PHP process due to suPHP.  Also on most local setups the webserver is also running with the user account rather than root, so these permissions issues don't come up.  I do outline these permissions issues in the troubleshooting area, but really it's only about 10% of people that are running their own linux servers that face this.


Add Node 7 tests

Coverage remained the same at 93.243% when pulling be1f3890a42674d238abe3ef2e250838d49a690c on alippai:patch-3 into 5014cad15cba26db3bd7c4ae395ebaad32955978 on urish:master.

Coverage remained the same at 93.243% when pulling be1f3890a42674d238abe3ef2e250838d49a690c on alippai:patch-3 into 5014cad15cba26db3bd7c4ae395ebaad32955978 on urish:master.

Coverage remained the same at 93.243% when pulling be1f3890a42674d238abe3ef2e250838d49a690c on alippai:patch-3 into 5014cad15cba26db3bd7c4ae395ebaad32955978 on urish:master.

Thanks!


Further News Item

Hi, I tried to create a new news item, hopefully it works ...

Can you redo the PR (or just add any commit), so it will trigger the new CI which will check if everything was done correctly

Tried that by changing the file: "PR Build Check / build (pull_request) Successful in 43s" - did it run as expected?

yes perfect thanks! Will merge 🙂


CSS menu stuff

@andralex

please rebase

Rebased. So far this is without Matt's gray style, fixes only


Create setColor

What problem does this feature solve?
There is a duplication, setColor function has been declared in all components where is used.
What is expected?
Global setColor function in utils. Refactor all usage of this function and replace it to the global one.

#189


Add Goat Cash (GOAT) icon and metadata

Adding goat.svg for icon and updating associated contract-map.

Will re-do PR.


See if the SCL versions can be made into a template

In this directory: https://github.com/centerforaisafety/cerberus-cluster/tree/master/playbooks/roles/packages/tasks

Do we still want to keep this issue open?

@steven-basart
I don't think this would make the code cleaner. There are only a few SCL versions and adding more abstraction wouldn't simplify the code. Just my two cents.

Happy to close it. It's really low priority anyways.


Str 678 replace UUID with proof

Description

Currently, the prover-client uses Uuid to track proving tasks, which makes it difficult to understand the nature of the task at a glance. The objective of this ticket is to replace the Uuid-based tracking system with a ProofContext.
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Documentation update
[X] Refactor
[X] New or updated tests
[X] Dependency Update

Notes to Reviewers

Checklist

[ ] I have performed a self-review of my code.
[ ] I have commented my code where necessary.
[ ] I have updated the documentation if needed.
[ ] My changes do not introduce new warnings.
[ ] I have added tests that prove my changes are effective or that my feature works.
[ ] New and existing tests pass with my changes.

Related Issues

The term "handler" might be incorrect for all the types in the handlers mod, since it implies some inversion of control where you construct the handler and pass it to a runtime that invokes it like a callback. It seems more like you could call them "providers" or "repository interfaces" something?

Renamed all handlers -> operators.

Why "operators"?

Why "operators"? I don't really understand the reasoning.

The trait is defined as ProvingOperation which is meant to define the operations required for proof generation.
Since this trait is being implemented renaming it to Operators.


build warnings about auth

"service_account_key" has been deprecated. Please switch to using google-github-actions/auth
from build log


allow fractional CPU usage

For small jobs that only consume very few resources it may be desirable to specify a fractional number of CPU resources (e.g. 0.2). Mesos already supports this, so maybe only changing the type of cpu parameters from int to float may be sufficient...?

As Retz was originally designed for CPU intensive jobs in our use cases and aimed for CPU pinning, historically it doesn't allow fractional CPU usage. I understand there are tiny jobs that ranges from 0.1 CPU to ~0.5 CPU, but do we really need float, like 16.725 CPUs? AFAIK non-CPU intensive jobs like memory, network, or disk-intensive jobs are mostly single threaded and sometimes wants 100% CPU usage. I'd like to know more about use cases and to find out right answer to your problem.

I agree that there is no need for 16.725 CPUs :-)
My use cases for a tiny job look as follows:

I want to launch a small server where jobs can register when they are started and unregister when they exit. This small server is used for coordination between multiple jobs, but it doesn't do any real work.
I want to launch a proxy server that is used by multiple jobs. It may or may not cache downloaded files, but it doesn't do any real work.
I want to launch a job that provides some service to the other jobs, but does not consume a lot of resources.

As per my understanding, it is not easily possible to actually limit the CPU consumption for a Mesos container (at least not if it is a Docker container), so my major issue is that if I have 4 jobs on one machine that are mostly idle, then I would have to block 4 CPUs for them, so I would like to specify less than 1 CPU for each job.

I agree that there is no need for 16.725 CPUs :-)

Good news :)
Thank you for giving a few use cases. My current thinking is to add tiny flag as a tag of a job, making Retz server ignores all resource specs given by clients and force minimal, fixed and predefined job size. That said, an extension it could be possible to create any arbitrary flag-as-a-tag to make fixed resource sizes defined in server side retz.properties, or could be processed by user defined Planner.


[Question] What's the purpose of the classical parametrization and classical training ?

Hello, I was wondering what is the purpose of including the classical training and parametrization facilities in a quantum natural language processing library.
I'm not sure how to properly elaborate more on this question. The only thing that comes to my mind is that having a classical equivalent of those two steps of the pipeline is useful to properly compare the result, but I'm fairly sure I'm missing something.
P.s. I suggest enabling "Discussions" from the repository's settings  so that people can post there their questions instead of opening an issues.

Hi @mspronesti, thanks for the question. We have enabled Discussions now, so feel free to move your question there and we'll copy the response.
In general, including classical training in lambeq is useful for many reasons:

First, lambeq is something more than a QNLP toolkit, it's a language modelling tool capable of representing language in many different levels of abstraction (syntax trees, pregroup diagrams, string/monoidal diagrams, quantum circuits, ZX-diagrams (this one is coming soon). For example, the abstract representation given by a string diagram is equivalent to a tensor network, which can be executed on classical hardware. Not giving this ability to the user would leave the toolkit incomplete in our opinion.
As you mentioned, comparison and benchmarking between quantum models and similar classical implementations is always helpful. In fact, all of our development in CQ follows this pattern, initial exploratory experiments on classical hardware, then simulations and translation to purely quantum models.
Access to quantum hardware is still limited. When it's available, today's quantum computers are still weak, so it's not possible to run large-scale experiments on them. In many cases, your only option is a "quantum-friendly" classical model.

Hope this helps, let us know if you need any clarifications.


Update telegram to 3.5.2-107970

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Closing in favour of https://github.com/caskroom/homebrew-cask/pull/39808
Looks like this has been updated in-place because the sha256 has changed.
Thanks for the PR nonetheless.


Locality weighting in Maglev inconsistent with other LBs

The way Maglev currently implements weighting for any given host and locality is: effective_weight = host_weight * locality_weight. This is nice and simple, but it's inconsistent with how the other LBs implement locality weighting. A simple example:

Locality A: locality_weight = 1

contains 100 hosts, each with host_weight = 1

Locality B: locality_weight = 1

contains 50 hosts, each with host_weight = 1

The way Maglev figures it, the effective weight for all hosts is 1: they all get equal traffic. But the way all other LBs figure it, and what users expect (per documentation, assuming healthy hosts) is that A should be getting the same traffic as B, no matter how many hosts happen to be in each locality: consequently, each of the 50 hosts in B should be taking twice as much traffic as each of the 100 hosts in A.
Judging by https://github.com/envoyproxy/envoy/issues/2725#issuecomment-370586299, we meant to do this at some point, but didn't.
The correct math is to sum the locality weights, and sum the host weights within each locality, to get the right proportions. For example, given:

Locality A: locality_weight = 2

Host a: host_weight = 3
Host b: host_weight = 5

Locality B: locality_weight = 7

Host c: host_weight = 11
Host d: host_weight = 13

The sum of locality weights = 9, the sum of host weights in A is 8, and the sum of host weights in B is 24, which gives us effective weights:

Host a: (2/9) * (3/8) ~= .083
Host b: (2/9) * (5/8) ~= .139
Host c: (7/9) * (11/24) ~= .356
Host d: (7/9) * (13/24) ~= .421

Yeah, I think this is legit broken; @snowp do you have any thoughts?

I suspect no one has ever used weighted maglev in production but I'm not sure.

Yeah I agree, this seems like a bug. I assume Maglev doesn't allow for the tiered host selection we do for the other LBs (priority -> locality -> host), but we should still make sure the overall host weighting is consistent when using locality weighting.

Cool, I'll work on this. I'm actually doing it for RingHashLB first, and I plan fix it in Maglev after.

There's two ways we could do it with Maglev/Ring hash LB. We could either use two tables, doubling memory use, but allowing more granular weighting for a given ring size, or what we do today, a single table with an effective weight function that expresses locality weight correctly. I think the latter is a more reasonable evolution of what we have, but maybe one day someone would prefer the two table solution.
What we can't do is have an EDF WRR scheduler for the first-level locality pick, since this needs to be deterministic and hash based.


[HOLD for payment 2024-04-25] [$500] Help - Send a message button opens help.expensify.com in a new tab

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.57-2
Reproducible in staging?: y
Reproducible in production?: y
Issue reported by: Applause - Internal Team
Action Performed:

Go to help.expensify.com
Click Send a message button under "Didn't find what you were looking for?".

Expected Result:
User is redirected to Concierge chat as it says "Concierge is here to answer all your questions.".
Actual Result:
User is redirected to help.expensify.com in a new tab.
Workaround:
n/a
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/28efd27b-186d-4a7d-a97b-df1647d264a8
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01761bb2ae86cc484e
Upwork Job ID:<PHONE_NUMBER>823450624
Last Price Increase: 2024-03-28
Automatic offers:

rojiphil | Reviewer | 0
gijoe0295 | Contributor | 0

@CortneyOfstad I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

I am able to recreate this as well — getting eyes on it!

@rojiphil we have a proposal above for this issue here – feel free to share your feedback when you have a chance! Thanks!

@gijoe0295 proposal here LGTM.
The RCA is correct and the proposed changes will set the main site url and launch the concierge page.
🎀👀🎀 C+ reviewed

Thanks for the review @rojiphil, assigning @gijoe0295 🚀

PR deployed to staging a week ago, so still have to wait a bit

BugZero Checklist:

[x]  [@rojiphil] The PR that introduced the bug has been identified. Link to the PR
[x]  [@rojiphil] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment
[x]  [@rojiphil] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: I don't think a discussion is required here as the current checklist itself is good enough to catch such regressions
[x]  [@rojiphil] Determine if we should create a regression test for this bug. Since the functionality is required to reach out to the concierge team, it may be good to have a regression test for this bug
[x]  [@rojiphil] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Test Steps

Go to Expensify Help Home
Press Send a message button
Verify user is directed to Concierge chat

Payment Summary
@gijoe0295 — paid $500 via Upwork
@rojiphil — paid $500 via Upwork

Regression test here — https://github.com/Expensify/Expensify/issues/390863
Thanks all — closing!


📝 Clarifying documention - Change from "Top Category" to "Topic"

Cc: @adamcogan @bradystroud
Hi @BrookJeynes
Pain
As per my conversation with @jaydenalchin we noticed people are often confused when referring to "Categories" and "Top Categories" on SSW.Rules
We agreed the best approach is to change from "Top Category" to "Topic"
Suggested Documentation
This way we have:

SSW.Rules

Topic (Management)

Category (Rules to Successful Projects)

Rule (Do you understand the value of consistency?)

Tasks

[ ] Update the Wiki instructions to change from "Top Category/Categories" to "Topic/Topics"

Thanks!

@tiagov8 @bradystroud @BrookJeynes
Please also note that ideally Top Category / Top Categories should change to Topic / Topics across some SSW.Rules and SSW.Rules.Content files in addition to wiki docs. Examples below.

Figure: SSW.Rules files need renaming

Figure: SSW.Rules.Content files need renaming


Output folder for generated sources

Is there any way to specify output folder for generated sources? (-out option for thrift)
It is use case when needed to generate sources in src/gen folder
I looked at the code but seems the target-path option is always overriden by merge

lein-thrift puts its stuff into a subdirectory thrift-java of your project's :target-path - there is currently no special option within the :thriftc map.
Do you really need this functionality, i.e. do you really need fine-grained control over where on the classpath they end up?

That may be useful when we need to generate sources and commit them, and someone is able to compile the code even without thrift compiler (target folder is often ignored in git)
Though it's not big deal, I can generate sources using thrift and just copy them to the place I want.

I see. I'm personally not a fan of committing auto-generated code but I'll include the possibility to customize the output path.

Released as
[lein-thriftc "0.2.3"]

(I finally managed to get a list of all open tickets in my repositories - needless to say, a few were forgotten...)


[Don't merge] Bandit test

Ayo!

is this a thing we ever want to merge? are we getting rid of this? make it a draft at least?

Nah.. I scrap it. I was in case anyone wanted to test bandit git action.


pythonPackages.regex 2019.12.20 -> 2020.8.1

Motivation for this change
The black code formatter was broken under the previous regex version and works with this version.
I updated regex because I got this error message from black:
ModuleNotFoundError: No module named 'regex._regex'
Traceback (most recent call last):
  File "/usr/local/bin/black", line 5, in
    from black import patched_main
  File "/nix/store/2mq7rd88rnj3knsg16j3q8x2lg3c0lc2-python3.7-black-19.10b0/lib/python3.7/site-packages/black.py", line 15, in
    import regex as re
  File "/nix/store/gf7y6r2fphfp7mp48khs73b3gw7g38lm-python3.7-regex-2019.12.20/lib/python3.7/site-packages/regex/__init__.py", line 1, in
    from .regex import *
  File "/nix/store/gf7y6r2fphfp7mp48khs73b3gw7g38lm-python3.7-regex-2019.12.20/lib/python3.7/site-packages/regex/regex.py", line 402, in
    import regex._regex_core as _regex_core
  File "/nix/store/gf7y6r2fphfp7mp48khs73b3gw7g38lm-python3.7-regex-2019.12.20/lib/python3.7/site-packages/regex/_regex_core.py", line 21, in
    import regex._regex as _regex
ModuleNotFoundError: No module named 'regex._regex'

Things done

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

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

LGTM

This has been superseded.

This has been superseded.

@risicle: superseded by what?


Compatibility of pack to create api driven projects

Hi there! Loved the project, this really reflects the maturity of data science projects and where we are standing. So good!
I rise this issue as I was wondering if the current structure can be adapted to an api-driven project. This is, a project in which the analysis and data flow may be related to an api definition.
If yes, what would it be? So we can document it (or point me out where it is)
If not, why?  Some books have recommended having an api flow for analysis and process so our results and analysis are available for our mates in engineering. Even allowing for an easy scale up.
Thank you so much!

@ricalanis can you give a better description of what is the flow of an API driven project?
or link to some resources about it? just to maintain a clean discussion.

Of course @MrOutis
I would say that a Data Science project is api-driven when an API is defined throughout the data science process to:

Make raw and intermediate data available in an endpoint
When an output is directed to an API to be interactive (From a dataviz to a simple data product)

Maybe I am skewed, but with Flask it is relatively easy to make this leap and I would like to have a clearer view on this can be included on the flow that this project proposes.
Two great sources that cover this:
http://www.datacommunitydc.org/blog/2014/02/flask-mega-meta-tutorial-data-scientists

So far, when setting up an API (or web app, or streaming data pipeline, microservice, etc) under this structure, we have been putting it in its own folder within src/. Let's say I was going to write a small API with Flask/Flask-RESTful. I'd create a directory for it and put my source code files in there:
/
└── src
    └── api
        ├── app.py
        └── __init__.py

Then to run it you could python src/api/app.py from the project root. You could even create a rule in the Makefile to run this, or create a Dockerfile to run it in a container — whatever entry point makes the most sense for your project.
Does that cover the use case you have in mind @ricalanis or do you think more changes are necessary to the existing structure?

Seems legit! Really covers my use case, thanks!
I wonder if such /src/api folder should be in the default folder structure, as such scripts that make your models or data discoverable can be, for me, as useful as the viz generating code. Of course one runs the risk of growing the folder structure bigger and bigger to cover such use cases. What do you guys think?

Interesting! This is not a common use case for us but we're very open to hearing more from the community.
Instead of closing this issue, I'm going to create a needs-discussion label so we can see if others are interested in providing feedback. We'll revisit the issue after some time has passed to figure out whether to close it or take action.
Sound good?

Sounds perfect! That way I will be able to test it more and more and have a more solid opinion of this and how it fits on the flow that the project proposes.

Hi @MrOutis and @isms,
Just came across the directory structure and the issue. I guess @ricalanis made a point. Many times, you want your model to shift to the production environment and there you need an endpoint which can communicate with third party applications and adding  /src/api folder in default structure make sense if I talk about my use case too.

Closing but still open to hearing more about this if people have specific improvement proposals.


Broken Links to schemas on the Docs>API Overview

https://ga4gh-schemas.readthedocs.io/en/latest/api/index.html
All links to schemas through here are broken.

Thanks @Arc39 moving this over to schemas! https://github.com/ga4gh/schemas/issues/643


Route prioritization should disregard (foo) segments

Describe the bug
When the routes are being prioritized, they should disregard (foo) segments. Currently, something like (foo)/[...catchAll] is prioritized above bar because it has more segments. (foo) segments shouldn't be counted when doing this sorting.
Reproduction
none, currently
Logs
No response
System Info
System:
    OS: Linux 5.10 Ubuntu 20.04.4 LTS (Focal Fossa)
    CPU: (8) x64 Intel(R) Core(TM) i7-10510U CPU @ 1.80GHz
    Memory: 10.36 GB / 12.32 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Yarn: 1.22.19
    npm: 8.15.0

Severity
blocking an upgrade
Additional Information
No response

I'm confident this is an easy fix — just need to change this line...
https://github.com/sveltejs/kit/blob/128340cdc4a9307dfa59bbc378cf9c644310bdaa/packages/kit/src/core/sync/create_manifest_data/index.js#L112
...to .filter(segment => segment !== '' && affects_path(segment) instead of .filter(Boolean) — but I haven't been able to come up with a failing test yet

Alrighty let me try to extract a failing test from my app.

Okay, I've added (foo)/(bar)/[...catchAll]/+page.svelte and foo/[bar]/+page.svelte to the basics test app, and visiting /foo/something is handled by the former instead of by the latter.


MonthView cellBuilder - titleColor not showing correct colour

I'm trying to implement dark mode in my app, but when I set the titleColor in my custom FilledCell to Colors.White, it's not showing as fully white - is there a way to change that?

This is my code:
cellBuilder: (date, events, isToday, isInMonth) => FilledCell(
                date: date,
                events: events,
                shouldHighlight: isToday,
                titleColor:
                    isInMonth ? Colors.white : Color.fromRGBO(0, 0, 43, 10),
                backgroundColor:
                    isInMonth ? Color.fromRGBO(0, 0, 43, 10) : Colors.grey,
              )

Thanks!

@ronny-strath You don't have to add the condition for titleColor. In FilledCell there is a condition where, if isInMonth is true, it will display the color same as titleColor else, it will show the titleColor with 40% opacity.
If you need other changes, you can create your own widget and use it instead of FilledCell.
I'm closing this issue for now. Please create another issue if required.


Images are not rendered

percollate pdf https://hpbn.co/primer-on-latency-and-bandwidth/
Generates this:
Networking-101:-Primer-on-Latency-and-Bandwidth-High-Performance-Browser-Networking-(O'Reilly).pdf
Some of the image tags have svg sources, and they are not rendered. This page also has a png which is not rendered either.

Hi, thanks for reporting! Can you let me know your percollate version by running percollate --version? It seems to be working for me on 0.2.12 / macOS.

$ percollate --version
0.2.12

Could it be related to something else then? 🤔

Sorry, the Github downtime is making it hard to respond.
I've just published<EMAIL_ADDRESS>Can you try upgrading to see if the issue still reproduces? If it does, can you perhaps look at the temporary html file to see how the HTML markup around images looks? (or you can attach it here, I think, as a .zip file)

Works perfectly! Thanks!


Dockerfile: Suppress wget progess bars

Can you sign the CLA ?

@vavanade  If you have some trouble with CLA, @gs-olive  can integrate this change as a part of  https://github.com/pytorch/TensorRT/pull/1732

Closing since its been 3mos and we cannot accept changes without CLA


Submit mini-bio to SC repo - Drew

In addition to posting mini-bios here, we also need to submit them to SC repo via a pull request as follows:
"People who wish to teach Software Carpentry must submit a small pull request to a specified GitHub repository. Your pull request should contain one file (and one file only) called "your_name.txt" (e.g., marie_curie.txt or alan_turing.txt). This file should contain a sentence or two that describes the type of work you do. You will submit the name of this file as part of your application.

@drewleonard42 Have you done this please?

I have.

Done in https://github.com/swcarpentry/training-prep-2015-09/pull/140


OVA: map vm configuration from extraConfig in ovf file

In some of the OVF file the configuration can be found under ExtraConfig rather that Config, currently the first option isnt supported in the OVA server and we may miss some impotent information,  this PR adds it.
example:
<vmw:Config ovf:required="false" vmw:key="cpuHotRemoveEnabled" vmw:value="false"/>
<vmw:ExtraConfig ovf:required="false" vmw:key="firmware" vmw:value="efi"/>

@ahadas reviewed the changes you posted, looking good to me.

@ahadas created https://github.com/kubev2v/forklift/pull/809


Add deploy to heroku button

Makes hosting the API easier for others.

Can you move the button down to the Install section? doesn't really line up nicely with the status things

@jomo Done and pushing now

Coverage remained the same when pulling 5653ba21cf395ecd378b7cc060b21aa2f3b3e978 on Tama63:master into 04b5161285d3f9a27e0d745c92ab047792231256 on Jake0oo0:master.

thanks!


Animation channel cannot target WEIGHTS when mesh does not have morph targets.

    Ok, here is:

A minimal blender file exhibiting my problem (only 1 object visible in the collections tree, this is my very old and large private file with pretty much everything deleted apart from a single cube obejct, that being said when I append from it I see two other objects which is suspicious in itself).
A GLTF export from that file missing the meshes[].primitives[].targets key
A fresh 3.3.1 blender file with the object I am testing with appended into it
A GLTF export from that file with working morph targets

https://drive.google.com/drive/folders/1m1Ng4aFuSYsRCCKyqZRV5wSnaTd7fK_F?usp=share_link
I'm piggy-backing on this issue as I'm seeing a very similar symptom, I apologise if it turns out to be unrelated.
Originally posted by @willstott101 in https://github.com/KhronosGroup/glTF-Blender-IO/issues/1785#issuecomment-1310438237

Let's separate your test case from the original one.

Thanks, sorry for the extra admin.

ah interesting... thanks for digging. Would it be feasible to check if that specific object has modifiers, and allow the export if it has none?

@willstott101 Let's first fix this, as proposed in my PR.
Your suggestion looks interesting, I will investigate in a second time

Done!
Thanks :)


Notifications on frontend

We need to notify user in some cases:

Fetching test suites source - now we silently send a request to fetch test suites from source
Run and rerun requests - now we display that run/rerun request was sent, but we do not display that an execution is finished.

The first case is more important now than the second, but we still need to do something with that later (maybe with #106 👀)

@0x6675636b796f75676974687562 it's your good choice :)


Handle interrupts when applet handler is not running.

This fixes lack of USB control interface and watchdog servicing when an applet is not running.

Looks like it'll need a manual rebase...


Compiler encountered an internal error

Hi, I was trying to port my code to macos that I faced the following error. You can see my code using this link:
Source of shader with compile issue
2022-06-05 14:52:51.167451+0430 TechDemo[5590:77764] Compiler failed with XPC_ERROR_CONNECTION_INTERRUPTED
2022-06-05 14:52:51.167556+0430 TechDemo[5590:77764] MTLCompiler: Compilation failed with XPC_ERROR_CONNECTION_INTERRUPTED on 1 try
2022-06-05 14:52:51.227904+0430 TechDemo[5590:77766] Compiler failed with XPC_ERROR_CONNECTION_INTERRUPTED
2022-06-05 14:52:51.227994+0430 TechDemo[5590:77766] MTLCompiler: Compilation failed with XPC_ERROR_CONNECTION_INTERRUPTED on 2 try
2022-06-05 14:52:51.300977+0430 TechDemo[5590:77764] Compiler failed with XPC_ERROR_CONNECTION_INTERRUPTED
2022-06-05 14:52:51.301080+0430 TechDemo[5590:77764] MTLCompiler: Compilation failed with XPC_ERROR_CONNECTION_INTERRUPTED on 3 try
[mvk-error] VK_ERROR_INITIALIZATION_FAILED: Render pipeline compile failed (Error code 2):
Compiler encountered an internal error.

-----------ERROR------------
File: /Users/mohammad.fakhreddin/Documents/c-plus-plus-projects/vulkan-engine/src/engine/render_system/RenderFrontend.cpp
Line: 114
Function: DebugCallback
"Message code: 0\nMessage: VK_ERROR_INITIALIZATION_FAILED: Render pipeline compile failed (Error code 2):
Compiler encountered an internal error.\nLocation: 0\n"
---------------------------
Assertion failed: (false), function DebugCallback, file /Users/mohammad.fakhreddin/Documents/c-plus-plus-projects/vulkan-engine/src/engine/render_system/RenderFrontend.cpp, line 114.
Assertion failed: (false), function DebugCallback, file /Users/mohammad.fakhreddin/Documents/c-plus-plus-projects/vulkan-engine/src/engine/render_system/RenderFrontend.cpp, line 114.

I use version 1.1 of vulkan in this project

It was a bug related to OS and drivers that have been resolved after the upgrade to newer version of MacOs.

Seeing the same issue on macOS 12.6. May I ask which version of macOS fixed the error for you?

Hey @AmesingFlank
I upgraded my OS and Xcode to macOS 12 Monterey that fixed my problem. But you have the same version.
I wish they at least tried to answer people. MoltenVK is not very stable. If you have other options don't use it. Try to run your project on Linux instead.


Making adjustment on the pickerWheel will fail in some scenario

This step here When(I: adjust(menuScreen.picker.wheel(0), to: "Example")) does not work in scenario when self.view.endEditing(true) is implemented for the pickerWheel in the app code.
The reason is that the pickerWheel is set to disappear from the screen after the user makes an adjustment on the wheel. Therefore when running the UI test, it succeeds in making the adjustment on the pickerWheel but fails on asserting the expected value that has been set.

This step here Then(I: adjust(otherElementsScreen.slider, to: 1)) provided in the example fails.
The reason for the assertion failure is Element did not have the right value before timing out! Expected: 1.0, actual: 0.9971751570701599

@TABanniefung these seem like two separate issues?

Issue #23 covers the slider issue (although I can't reproduce it!)