Empty JSON object returns size of 1

Description
An empty JSON object returns a size of 1rather than the expected size of 0.
This is problematic because:

accept() indicates this is valid JSON (because it is)
parse() parses this as valid JSON (because it is)
size() indicates 1 so we expect there to be at least one KV pair
Using the iterator results in an exception because there isn't really a value to parse.

Reproduction steps
Create an empty JSON object using the string "{}" or "{ }", and then check the size.
According to https://www.json.org/json-en.html these strings are the minimum valid JSON string.
Expected vs. actual results
I expect and empty JSON object to return a size of 0.
However it is returning a value of 1.
Minimal code example
**Code**

#include <stdio.h>
#include <iostream>
#include <nlohmann/json.hpp>

#define DEBUG_DUMP(x)    printf("%-3s : %s : %zu\n", \
                                #x, x.dump().c_str(), x.size())

int main(void)
{
    // This is valid JSON and it's empty, no objects.
    std::string emptyJson("{}");
    std::string emptyJson2("{ }");

    nlohmann::json    j1(emptyJson);
    DEBUG_DUMP(j1);

    nlohmann::json    j2(emptyJson2);
    DEBUG_DUMP(j2);

    // Parse nothing
    nlohmann::json    j3;
    DEBUG_DUMP(j3);

    // There isn't a way to see if this is an empty/invalid json object?
    if (j1 == j2) printf("j1 equals j2\n");
    if (j2 == j3) printf("j2 equals j3\n");

    // lets iterate over the objects that should be there, throw an excemption
    // and crash!
    for (nlohmann::json::iterator it = j1.begin();
         it != j1.end();
         ++it)
    {
        std::cout << "the key:'" << it.key() << "'" << std::endl;
        std::cout << "the value:'" << it.value() << "'" << std::endl;
    }

    return (0);
}

Output:
$ g++ -I ./json/single_include jrrtest.cpp
$ ./a.out
j1  : "{}" : 1
j2  : "{ }" : 1
j3  : null : 0
terminate called after throwing an instance of 'nlohmann::json_abi_v3_11_2::detail::invalid_iterator'
  what():  [json.exception.invalid_iterator.207] cannot use key() for non-object iterators
Aborted (core dumped)

### Error messages

_No response_

### Compiler and operating system

Ubuntu 20.04 w/ g++ 9.4.0; Windows 11, Visual Studio 2017 15.9.53

### Library version

3.7.0

### Validation

- [X] The bug also occurs if the latest version from the [`develop`](https://github.com/nlohmann/json/tree/develop) branch is used.
- [ ] I can successfully [compile and run the unit tests](https://github.com/nlohmann/json#execute-unit-tests).

You are not calling the parse function, but pass strings to the constructor. This creates string values which have a size of 1 by design, see https://json.nlohmann.me/api/basic_json/size/#examples


build: bump version to v0.08

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

Project coverage is 92.51%. Comparing base (fb52448) to head (f3d6a59).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
- Coverage   97.73%   92.51%   -5.23%
==========================================
  Files          14        4      -10
  Lines        1149      294     -855
==========================================
- Hits         1123      272     -851
+ Misses         17       13       -4
  Partials        9        9

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


Support translation for active record enum type

How about support translations for ActiveRecord enum?
Example:
en:
  enums:
    model_name:
       enum_field_name:
          enabled: Enabled
          disabled: Disabled
          ...

What would be awesome!

This works. Gotta use @ivanzotov repo in Gemfile for now. Thanks, Ivan.

@yurijmi You are welcome!

I got an error like NoMethodError: undefined method invert' for #Array:0x00000008183c30` around enum and this branch fixed it. I want this patch to be merged.

Looks nice, can be merged in after specs are added and conflict is resolved.

Please reopen on update.


Stop after a while on Windows

I have the same code running on Linux and Windows. On Linux it works flawlessly but on Windows, when adding the cron verbose logger, I see a "start" message followed by a "stop" one after a bit. This of course prevents the jobs to be run. Do you have an idea on how to fix this? Is it a known bug?

package main

import (
	"fmt"
	"github.com/robfig/cron/v3"
	"log"
	"time"
)

func main() {
	// Initialize a new cron scheduler
	c := cron.New(cron.WithLogger(cron.VerboseLogger(log.New(os.Stdout, "cron: ", log.LstdFlags))))

	// Define a sample cron job
	_, err := c.AddFunc("*/1 * * * *", func() {
		fmt.Println("Running cron job...")
	})
	if err != nil {
		fmt.Println("Error adding cron job:", err)
		return
	}

	// Start the cron scheduler in a goroutine
	c.Start()

	// Keep the program running to allow cron jobs to continue executing
	select {}
}


Personal results keep being asked

Hello,
I installed a nest hello and it works great overall, due to the wiring in my house, I couldn´t install the chime so, I received the notification on the phone, but I really wanted to add the chime and announcement feature, so I grabbed one of my raspberrypis and installed the google assistant, which is working now, however if i press the button on the doorbell, i dont hear anything on the raspberry google assistant, which I set as speaker, it does respond to the command for playing the video on the chromecast tv but doesnt chime or make any sound at all.
I read that I might enable the "personal results" which I did, but even if I ask "what's my calendar for tomorrow?" it will tell me I need to enable the personal results on the iphone, and in that screen that shows me I can see that "Personal results" is already enabled.
Any tips or tricks to make the doorbell sound through my custom google assistant speaker?

Hi @beckmx ,
I am also facing similar issue, did you get any workaround?
Thanks.


Qr mnemonic view

Priority
Normal
Description

Shows a QR code along with the mnemonic phrase

Addresses feature #108


Cannot import vrprojector into python command

HI I get this error
Python 3.6.0 (v3.6.0:41df79263a11, Dec 23 2016, 07:18:10) [MSC v.1900 32 bit (Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import vrProjector
Traceback (most recent call last):
File "", line 1, in
ModuleNotFoundError: No module named 'vrProjector'

@uniquelyesmel: In the way the package is currently structured, you might want to add the directory directory containing vrProjectorCmd.py to PYTHONPATH, then I think it should work.

I'm working on making this package pip installable so hopefully this issue should just go away.
Hang in there :]


Error with vault passwords

Hi There. I'm try install this repository in GKE for tests and I don't get to pass this error:
[WARNING]: Error in vault password file loading (default): Invalid vault password was provided from script (/usr/local/bin/vault-password)
ERROR! Invalid vault password was provided from script (/usr/local/bin/vault-password)
Where have I to put this password? I read the documentation several times and I don't find it.
Tank you

Hy There,
I just got it.


May I use some of your luasnip in my plugin?

I really love your snip setup for go. Great work.
I would like to use those code in my plugin and I have a PR here:
https://github.com/ray-x/go.nvim/pull/165
I noticed there is no licence agreement in your repo. So need to confirm may I copy those codes across to my plugin (MIT licence)?

Hi mate, thanks for your comment.
You have my blessing! I will add a license shortly.

Thanks! That will save me lots of time.


Enhancement: getUserDB method

Please add a getUserDB(user_id, dbName, type) method which returns a PouchDB instance with admin access. This will allow the server API to easily manipulate the user DB with admin privileges. Without this API, the only way is to share the CouchDB admin credentials separately with the rest of the server code.
This will also go in parallel with other PouchDB instances provided by superlogin, like usersDB and couchAuthDB.

OK. I'll work on this for the next release.


Make Compose box accessible before the chat loads

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/14223
PROPOSAL: https://github.com/Expensify/App/issues/14223#issuecomment-1485373560
Tests

Same as QA steps

[x] Verify that no errors appear in the JS console

Offline tests

Open any old chat or a chat that is still loading.
Type a message in the compose box.
Verify that the compose box is accessible and functioning properly as it would when the device is connected to the

QA Steps

Disconnect the device from the internet.
Open any old chat or a chat that is still loading.
Type a message in the compose box.
Verify that the compose box is accessible and functioning properly as it would when the device is connected to the internet.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://user-images.githubusercontent.com/75031127/228626885-935770c8-8c75-4a83-95e4-c770ed510c28.mov

Mobile Web - Chrome

https://user-images.githubusercontent.com/75031127/228630329-5db44e22-24e5-4869-8087-2a31df4badd1.mov

Mobile Web - Safari

https://user-images.githubusercontent.com/75031127/228632525-8b4caca0-56c4-40b6-a22f-baed28d75386.mov

Desktop

https://user-images.githubusercontent.com/75031127/228634659-11734fe0-3876-4727-b996-5364945856c0.mov

iOS

https://user-images.githubusercontent.com/75031127/228637745-285f14e3-588a-47ff-876d-8e38424ae7c1.mov

Android
https://user-images.githubusercontent.com/75031127/228643517-48d0c265-a88b-4119-8f70-65da7496c0a2.mov

@techievivek @sobitneupane One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]


MSC2002: Proposal for adopting MSC1884 as v4 rooms

Rendered: https://github.com/matrix-org/matrix-doc/blob/matthew/room_v4/proposals/2002-rooms-v4.md

@mscbot fcp merge
we want to get this and 1884 merged asap and released in prep for 1.0

#1884 - link for the lazy

I think what happens for room v5 / matrix 1.0 and beyond are somewhat orthogonal here. +1 to getting features out into the wild rather than sitting on them until they form a huge batch, so I am happy with this.

I think what happens for room v5 / matrix 1.0 and beyond are somewhat orthogonal here.

I think the proposed longer-term context is fairly important to try to understand how this fits in, but agreed it's technically orthogonal to the MSC itself.

@matrix-org/core-team I'd like to propose an expedited FCP for this, so we can merge this (and #1884) into the spec and ship it in synapse (https://github.com/matrix-org/synapse/pull/5210) asap, so that when 1.0 rolls around we have as many servers talking v4 rooms in the wild as possible and we can enable creating v4 rooms by default.
I suggest we reduce FCP to 1 day - i.e. aim to merge this around 15:00 GMT+1 tomorrow, unless someone finds a last minute problem with MSC1884 (or this).

Impl is at https://github.com/matrix-org/synapse/pull/5217, ready to merge when this completes FCP.

as per https://github.com/matrix-org/matrix-doc/pull/2002#issuecomment-494023657, we are declaring FCP to be complete here.

cancelling the bot's FCP to stop it "helping"
@mscbot fcp cancel

Spec PR: https://github.com/matrix-org/matrix-doc/pull/2019

merged 🎉


8_puzzle

8_puzzle
Fixes : #4223
I have fixed the issue #4223.
Added new game

Mark the task you have completed ✅

[✅] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[✅ ] I have performed a self-review of my own code or work.
[✅ ] I have commented my code, particularly in hard-to-understand areas.
[✅] My changes generates no new warnings.
[✅] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[✅] I have added screenshot for website preview in assets/images
[✅] I have added entries for my game in main README.md
[✅] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @manikumarreddyu ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

i have created readme.md file in my folder..check it once

In my opinion ,the bot which you hace created is not working fine..
it is my kind request to you that..make some edits to bot..

Hey @kunjgit check it once

hello @ishita-43  can you check it once

hello @kunjgit  can you please check it once ..

Thank you  @manikumarreddyu , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Update 051.1-astar.md

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

Closing see comment on #2782


Feature Request - Drop Columns

Currently, Delta Lake doesn't seem to support dropping columns without loading the entire dataset into Spark, dropping the columns from the DataFrame and overwriting the original table. If I correctly understand the way the transaction log works, it seems like it could support such a statement by making certain columns unavailable and altering the schema from the drop commit onwards.
Especially in combination with shallow cloning on Databricks (which is a brilliant feature, by the way) this could lead to big efficiency gains, for example when you want to do some dimensional modeling on a shallow clone of a (very denormalized) source table, where you replace multiple columns with one reference ID column without touching the original data.

This has been added to current master. cc @vkorukanti

This is implemented in 58b25108e1e275fa7e2ff4ba758184a19270cf15

This blog post explains how to drop columns in Delta Lake if you'd like more details.


add more contributors?

I think it would be great to add more contributors to close more issues and review pull requests

The development of this repository seems to have stagnated.
Maybe its time to try to talk to the owner, or do a fork.


About pre-training data of distilled qwen model

Is there any pre-trained model data or recommendation for the distilled qwen model? The effect has been verified on the llama model, but the performance on qwen is very poor. I suspect that the problem is the pre-trained data.

I think the dolly dataset is enough for qwen. For more high-quality data, you can try sharegpt. Note: Qwen's tokenization is much larger than LLaMA, which means the processed data should be stored in int32, not uint16 as in the codebase.


Feat/generate activity script

Please add Pino logging. Also it would be great to know neutron address that is using with passed mnemonic, and please add restrictions on the max amount of funds to stake/unbond etc.


✨ Simplify status condition reasons

Description
Simplifies status condition types per the v1 API Review RFC. Succeeded and Failed are used for all the status condition types (Unpacked, Installed, Resolved) to ensure consistency.
Fixes #1153

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

API diff is acceptable.


Update text analytics sample

Purpose

update samples\language\text_analytics_samples.py, now uses t2 text analytics sdk

Does this introduce a breaking change?

[ ] Yes
[x] No

Other Information

no change in other samples

hey @cxznmhdcxz some checks here failed - could you get that fixed?
cc: @jongio

Update text analytics from track1 to track2; file: samples\language\text_analytics_samples.py
Update anomaly detector from track1 to track2;  file: samples\anomaly_detector_samples.py

@jongio for notification.

Update text analytics from track1 to track2; file: samples\language\text_analytics_samples.py
Update anomaly detector from track1 to track2;  file: samples\anomaly_detector_samples.py

@jongio for notification.

@PatrickFarley - Can you review?

@PatrickFarley - Can you review?

The Text Analytics sample looks good - @mrbullwinkle for Anomaly Detector

The Text Analytics sample looks good - @mrbullwinkle for Anomaly Detector


Clicking GUI in another plugin instantly closing it

Magic is modifying many plugins with its default configurations.. first i had to spend hours preventing my sticks turning into broom sticks and now this issue of magic instantly closing a GUI when i clicked some stained glass panes, in another one of my plugins. It does not happen without Magic installed. can someone please help me figure out what I need to turn off inside of magic to fix this?
server version 1.12.2 paper, latest magic

It sounds you have some incompatible plugins, this can be very hard to track down. Since you know magic is one of them, you probably need to just remove it. Sorry!


add more right padding to the 'run' button

With the updates to the codemirror scrollbars, we should add slightly more right padding to the run button:

Hi can I work on this issue ?

For sure! I suspect we already have some padding around the container for the run and format button, so this issue is probably about adding enough additional padding on the right so the buttons look good in relation to the new scrollbar position.

how do we reproduce this issue ?

You can run the UI locally using this instructions: https://github.com/dart-lang/dart-pad/tree/main/pkgs/sketch_pad#how-to-run


Crashes on ART

Hi,
First thanks very much for this excellent library! This really solves my OOM issue.
This library will crash on ART, due to the tighter jni check rule. More specifically, we cannot pass 0 capacity to NewDirectByteBuffer() (around line 305 in JniBitmapOperationsLibrary.cpp).
I simply changed the capacity to 1 and it works. I'm not quite familiar with JNI, so you might better take a look yourself.
Hope this will help.
Best

Hi,
Sadly, even now I can't succeed importing my own library to Android-Studio, so it's very hard for me to fix things (even Eclipse started having issues).
However, now that I searched for this command ("NewDirectByteBuffer") , maybe I was wrong about how to use it? It was so long ago that I don't remember even what it is...
Can you please send me APKs of before and after this change? I also have Lollipop, and I remember it worked fine without this "1" value.

Well, I'm programming on a special device, so apk won't run on normal Android device.
Loading to Android Studio is pretty smooth on my side, the directory is as follows:

And add ndk.dir in local.properties, nothing special.

Can you please show a step-by-step way to import my sample&library, and be able to really edit C++ code and compile it on Android-Studio ?
No matter what I've tried - it didn't work.
About the APKs, I meant sending me the APKs of my own sample. Not yours.

Actually I couldn't get your sample work either... Seems that it cannot find the shared library.
For my own project, I just did the following steps:

Create a 'jni' folder under src/main/ folder and copy JniBitmapOprationsLibrary.cpp to this folder.
Add a package com.jni.bitmap_operations, and add JniBitmapHolder.java to this package.
Add your NDK path to local.properties
Import com.jni.bitmap_operations package where it will be used
And it should be fine, except the NewDirectBuffer issue on my side.

It didn't work now either. It says:
D:\android\ndk\build\core\add-application.mk
Error:(199) *** Android NDK: Aborting...    .  Stop.
Error:Execution failed for task ':app:compileDebugNdk'.

com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command 'D:\android\ndk\ndk-build.cmd'' finished with non-zero exit value 2

I will try out the new NDK version (10e).

I repalce your test.png to a lager jpeg.

@dinghu This looks like a question that should be in a new thread.
Anyway, OOM will be avoided only if you succeed decoding using the normal way, use the library, recycle the previous bitmap, do the things you want, and then create a new bitmap from the library.
The input and output still needs to stay in the heap memory.
However, you can modify the code to handle a bitmap with 565 config, which uses only half the memory (yet without transparency support).
Another solution is to avoid using this library, and use a different library instead, which allows you to decode the bitmap right inside JNI :
https://github.com/suckgamony/RapidDecoder


Add left-stripping functionality

This PR fixes https://github.com/in-toto/in-toto-rs/issues/13 and includes the following additions:

Introduces a new optional input lstrip_paths to in_toto_run addressing rebuilderd absolute paths & future usages.
Add private function apply_left_strip: Given an artifact path in &str format, left strip path for given artifact based an optional array of lstrip_paths provided, returning the stripped file path in String format wrapped in Result.

Please verify and check that the pull request fulfills the following requirements:

[x]  Tests have been added for the bug fix or new feature
[x]  Docs have been added for the bug fix or new feature

Thanks @joyliu-q, I'll try and take a look later today. Just to get it out of the way, can you tell me if it handles the issue seen here: in-toto/in-toto-golang#139?

I don't have anything to add to Santiago's review, though I think this PR isn't handling duplicate artifacts post left-strip.

Superseded by #19. Thanks for you work here, Joy!


Changing a platform's movement_controller doesn't change a sensor's movement_controller.

I think it is rare that the movement_controller property on a platform is changed. However it is worth noting that changing a platform's movement_controller doesn't change the attached sensors' movement_controller. I've a pasted a snippet below which demostrates this.
timestamp = datetime.datetime.now()
fixed_state = State(np.array([[2], [2], [0]]),
                    timestamp)
fixed = FixedMovable(states=fixed_state, position_mapping=(0, 1, 2))
platform = MovingPlatform(movement_controller=fixed)

platform.add_sensor(DummySensor())
assert platform.movement_controller is sensor.movement_controller

moving_state = State(np.array([[2], [1], [2], [-1], [2], [0]]), timestamp)
moving = MovingMovable(states=moving_state, position_mapping=(0, 2, 4), transition_model=None)

platform.movement_controller = moving

assert platform.movement_controller is sensor.movement_controller  # This fails/

Option 1: Make the movement_controller property read only.

Option 2: Add a setter for the movement_controller property.

What are your thoughts @erogers-dstl ?


where to find the synset of the published model

@jychoi84
Could  you please point where to find the synsets of the published model LocationNet on
http://mxnet.io/model_zoo/
Thanks!

@patricianing By 'synsets', do you mean the mapping from geo-coordinate (latitude, longitude) to geographical cell?
For instance, the mapping such as (lat:37.12, lon:122.23) --> (cell#12)?

@jychoi84 I mean the labels of the model. The model predicts a location of an image, I would like t find all labels of the locations. Thanks!

@patricianing
Oh right. We should've linked this tutorial that includes how to use the labels of the model (grids.txt).
https://github.com/multimedia-berkeley/tutorials/blob/master/predict_geolocation.ipynb

@patricianing  http://data.dmlc.ml/models/imagenet/synset.txt

@luoyetx LocationNet model is not trained with ImageNet :)
I added the link to the label file in my response above.

@jychoi84 Thanks a lot! So it is a different model from PlaNet? Do you know how to map the result vector (the probability of falling into each S2 cell) to the text info like city/country name of the corresponding location?

@patricianing
LocationNet is inspired by PlaNet but it's trained on different dataset, architecture, hyperparams.
The comparison of these two can be found here.
Examples in this tutorial will give you coordinate geo-tag (latitude, longitude). If you want to get a plaintext place name from this coordinate geo-tag, you can use one of the reverse geocoding services online such as Google's MAP API or this one on github.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Step 5: Testing of Implementation

Needs story point estimate. This is work for Max and @anvabr . Adding to Sprint 5


feat!: Use the hugr builder

This is an in-progress transplant of guppy from using an ad-hoc Hugr definition into the builder from the hugr library.
The main mismatch between the old and new builders is that guppy used to lazily add typed ports to the graph nodes as it needed. Since the new builder requires operations to define their signature, we end up having to edit the code everywhere from guppylang.compiler to the guppylang.prelude definitions.
My first goal is to get this working, and then I will try and split some changes from this PR (although most of it will have to be merged monolithically).
Current test status:

386 $\color{green}\text{passed}$
17 $\color{yellow}\text{skipped}$
0 $\color{red}\text{failed}$

These two issues currently breaking some tests. I'll fix them before merging the PR

https://github.com/CQCL/hugr/issues/1319
Required to store the module names in the hugr metadata
https://github.com/CQCL/hugr/issues/1424
Required for pytket and llvm integration
Release hugr and tket2 with the latest changes

BREAKING CHANGE: Removed guppylang.hugr_builder.hugr.Hugr, compiling a module returns a hugr.Hugr instead.

Codecov Report
Attention: Patch coverage is 96.03825% with 29 lines in your changes missing coverage. Please review.

Project coverage is 92.30%. Comparing base (d542601) to head (f9a7abd).

Files
Patch %
Lines

guppylang/prelude/_internal/compiler.py
75.86%
14 Missing :warning:

guppylang/prelude/_internal/util.py
96.33%
4 Missing :warning:

guppylang/compiler/expr_compiler.py
97.69%
3 Missing :warning:

guppylang/tys/arg.py
70.00%
3 Missing :warning:

guppylang/tys/param.py
62.50%
3 Missing :warning:

guppylang/compiler/core.py
95.45%
1 Missing :warning:

guppylang/error.py
83.33%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
+ Coverage   90.88%   92.30%   +1.42%
==========================================
  Files          48       48
  Lines        5341     5071     -270
==========================================
- Hits         4854     4681     -173
+ Misses        487      390      -97

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


improving docs left-nav labels

overview
this is changing our converter labels to their full names which should better help folks find what they are looking for
this also updates the heading on the get started section to be more clear, ive aligned on get started instead of getting started but can switch to getting started if folks feel strongly
there was a previous convo around also updating the get started guide labels, but we decided against that for now
related internal slack convo
screenshots
before

after

@cnunciato random additional thought, maybe Intro to Pulumi instead of only Intro, thoughts?

@susanev I actually kinda feel like "intro" isn't really the right title for that section, so I wouldn't be super inclined to add that particular change here.


is bug? the recv message can't read header

conn.go line 42
func (p *conn) Recv()(*Message, error)
msg = NewMessage(int(sz))
msg.Body = msg.Body[0:sz]

No, its not a bug.  We read the size from the transport header, and then the entire message afterwards.  The size (and if there were any, any other transport specific headers) are not part of the message that is delivered to the application.
Conceivably we could capture transport state here and pass it to the Pipe along with the message, but it turns out that for connected pipes there is no state that is interesting at this time.  (The interesting state is all gathered during connection establishment time.)
Note that Scalability Protocol headers (such as Req/Rep headers) are included in the size, and are delivered with the message.  The pattern specific code will parse and strip those from the message later in the code path.


Vickel: Watch pdu60.com

Vickel requests the watch of the watch_keyword pdu60\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

pdu60\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


S3 hosting bucket is empty

Please describe which feature you have a question about?
Hosting
Provide additional details
I'm unable to view the files written to the hostingbucket inside the S3 AWS Console. The UI says the hosting bucket is empty, even though the build and deploy succeeded. How can I verify what files were written to S3 during a deploy?
This made debugging (https://github.com/aws-amplify/amplify-console/issues/278) a pain when trying to verify what files were actually deployed.

My mistake, the empty hostingbucket was not created by the Amplify Console.


Can not write a model method for through(association) model

Given an through(association) model with a field where I need to add some model method.
association model c:
module.exports = {
    attributes: {
        quantity: { type: 'integer' },
        //Relations
        a: { model: 'a', type: 'date', foreignKey: true, columnName: 'a_id' },
        b: { model: 'b', foreignKey: true, columnName: 'b_id' },
    },
    incrementQ: function () {
        //Cool native request goes here
    }
};

When I try to access this method I've got an error: c.incrementQ is not a function
...
//Created the model in before etc. etc.
it('decrement by one', function (done) {
        console.log('Medel method:', c.incrementQ); //undefined
        c.incrementQ({id: 123}, {quantity: -1}).then((res) => { //the error: no such method
            console.log(res.quantity);
            done();
        }).catch(done);
    });
...

I've made a repository, and a question

@TylerBre @particlebanana @devinivy @elennaro - Hey everyone, we've moved this issue to the main Sails hub with an added label "orm." A resolution appears to have been patched in a newer version of Waterline. Feel free to update if it's still unresolved.  Thanks

Hi @TylerBre @particlebanana @devinivy @elennaro we are further cleaning up the Sails issues by closing older ones that have been patched with new updates.

This has been published.


chore: add the ability to handle new metrics event in persister

Rebased because of resolving conflicts.

lgtm👍

nit: chore: add handling for new metrics events in the persister service


Fix multiple fulltrust process instances running

After the modifications to the fulltrust process to add the recyclebin support, multiple instances of the fulltrust process will sometimes be launched. This PR should solve this.

@yaichenbaum Please wait a sec before committing, I'm waiting for this to build for a final check.

@gave92 No worries, our policy is to always wait for a build to succeed before merging.

Ok everything seems good!


Models not available

Hi!
Been trying to run Instanseg, and since a couple of minutes I get this issue:
Traceback (most recent call last):
  File "C:\Program Files\JetBrains\PyCharm Community Edition 2024.1.4\plugins\python-ce\helpers\pydev\pydevd.py", line 1551, in _exec
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "C:\Program Files\JetBrains\PyCharm Community Edition 2024.1.4\plugins\python-ce\helpers\pydev\_pydev_imps\_pydev_execfile.py", line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "[...]\cellstitch-instanseg\scripts\main.py", line 260, in <module>
    model = InstanSeg("fluorescence_nuclei_and_cells")
  File "[...]\envs\cellstitch\lib\site-packages\instanseg\instanseg.py", line 118, in __init__
    self.instanseg = download_model(model_type, verbose = self.verbose)
  File "[...]\envs\cellstitch\lib\site-packages\instanseg\utils\utils.py", line 858, in download_model
    response.raise_for_status()  # Raise an error for bad response
  File "[...]\envs\cellstitch\lib\site-packages\requests\models.py", line 1024, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.github.com/repos/instanseg/instanseg/releases/tags/instanseg_models_v1

I see there are some efforts to change the model downloading, could it have to do with that?
Cheers!

Can confirm!


Broken gamut

I created a Kotlin notebook and try to plot the hue slice, the boundary is vey weird, while we expect a continuous boundary.

Is it the expected behavior? Or there are some errors in the library's algorithm?

Oh, I think this is my problem


"Update manifests SCA Link "

This story updates the sca info link to direct to the appropriate resource

:exclamation: Outdated dependencies error

Installed version of @redhat-cloud-services/frontend-components-config (4.7.2) is older than required version (6.0.0)


php56: Fix URL/checkver

Addresses https://github.com/ScoopInstaller/Versions/issues/21#issuecomment-500088862

Move to archives means php.net has frozen this version (5.6 and 7.0), so please refer to php55 that remove autuupdate and checkver and lock its version number to 5.6.40 (for 7.0, to 7.0.33).
Yes, php70 is broken, too... So why not fix it here?

Fixing PHP 7.0 is fine too.

Seems I can’t approve this myself. @niheaven ?

Wow, but why I can...
It seems one cannot approve oneselves PR.


Fixed #33639 -- Enabled cached template loader in development.

ticket-33639

Hi @collinanderson, @ngnpope, @kezabelle — Follow-up to our discussion on https://github.com/django/django/pull/15140. Can I ask you to give it play and see if there are any blockers? Thanks! 🎁

@carltongibson I'll use a the branch for a while in development. Seems to be working well so far. Thanks.

Thanks for checking @felixxm! Updated. 👍


Ensure staging can run with NODE_ENV=production

We need to run staging with NODE_ENV=production because we want the build process to be exactly the same as production one.
However, we need to ensure there is no undesirable side effects to it.

The only issue was fixed in https://github.com/opencollective/frontend/pull/513


SPARK-6414: Spark driver failed with NPE on job cancelation

Use Option for ActiveJob.properties to avoid NPE bug

Can one of the admins verify this patch?

Jenkins, this is ok to test.

Test build #29009 has started for   PR 5124 at commit 687434c.

This patch merges cleanly.

It looks like this NPE bug has been around for a while, but it seems pretty hard to hit (which is probably why it hasn't been reported before).  I think that we should be able to trigger / reproduce this by creating a new SparkContext, ensuring that the thread-local properties are null, launching a long-running job, then attempting to cancel all jobs in some non-existent job group.  Can we add a regression test for this?  Shouldn't be too hard if my hunch is right.
It looks like we don't directly expose the Properties object to users, so if we wanted to we could go even further and convert all of the upstream nullable Properties into Options[Properties] as well.  If you look at the call chain leading to this use of properties, it looks like it can only be null if no local properties have ever been set the job submitting thread, its parent thread, or any of its other ancestor threads.  Therefore, maybe we can just eliminate the whole null / Option stuff entirely by ensuring that the thread-local has an initialValue instead of having it be null in some circumstances and not others.
Therefore, here's my suggestion:

Add a regression test and confirm that it reproduces the original bug.
Override SparkContext.initialValue to return a new empty properties object (since this is how we lazily initialize the properties in the existing code.  Update the other parts of SparkContext that set this to account for this change.
Add a few assert(properties != null) so that we catch errors up-front.  I'd add these checks at the entry points of the DAGScheduler, e.g. the private[spark] submitJob methods that are called from SparkContext.

Your patch looks good overall, but if we can I think we should just fix the underlying messiness.

(Sorry, that should have been SparkContext.localProperties.initialValue above; I've revised my comment)

Test build #29009 has finished for   PR 5124 at commit 687434c.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29009/
Test PASSed.

@JoshRosen thanks for the suggestions.  Let's me work on those tonight.

Test build #29266 has started for   PR 5124 at commit baea4fd.

This patch merges cleanly.

@JoshRosen , thanks for the help.  I updated this PR with

regression test
use Option[Properties] for the value of SparkContext.localProperties
add initialValue to SparkContext.localProperties

Test build #29266 has finished for   PR 5124 at commit baea4fd.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29266/
Test PASSed.

Test build #29334 has started for   PR 5124 at commit 6cfe888.

This patch merges cleanly.

@JoshRosen I updated PR, please review.  Thanks.

Test build #29335 has started for   PR 5124 at commit aef7ac6.

This patch merges cleanly.

Test build #29334 has finished for   PR 5124 at commit 6cfe888.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29334/
Test PASSed.

Test build #29335 has finished for   PR 5124 at commit aef7ac6.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29335/
Test PASSed.

Aside from two minor comments that I just left, this looks good to me.

Test build #29439 has started for   PR 5124 at commit 3d64166.

Test build #29439 has finished for   PR 5124 at commit 3d64166.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29439/
Test PASSed.

hi @JoshRosen , do you think this PR is ready to be merged?  Please let me know if you have more comments.  I'm happy to address them.

LGTM; looks like there's a merge conflict, but I'll fix it myself.

Looks like the Apache Git sever is down / inaccessible, so I'll have to commit this in a little bit.

@JoshRosen, thanks for the help.  I just rebased this PR against master.  It should also fix the conflict.

Test build #29620 has started for   PR 5124 at commit 2290b6b.

Test build #29620 has finished for   PR 5124 at commit 2290b6b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29620/
Test PASSed.

I've merged this into master (1.4.0), branch-1.3 (1.3.1), and branch-1.2 (1.2.2).  Thanks!

@JoshRosen, no problem.  I'm happy to contribute.


Using secret to protect database username - openmetadata.config.database.auth.username

Hello,
The chart is currently protecting database password using secret on these 2 propreties :
openmetadata.config.database.auth.password.secretRef
openmetadata.config.database.auth.password.secretKey

but there is no possibility to use secret for database username :
openmetadata.config.database.auth.username
Do you think we could implement something similar to share the same secret for username and password please ?
Thanks for your help.

Hello @achovelon .
This is not available out of the box though!
One such alternative is to disable database field from Helm Values and pass in the alternative Environments from the Secrets using helm values envFrom.
That way, you are in complete control over the database credentials which can be managed and maintained by you without upgrading helm charts values!
Example, you can do something like below with your helm values -
openmetadata:
  config:
    ...
    database:
      enabled: false
...
envFrom:
  - secretRef:
       name: openmetadata-db-config
...

In the above case, you will responsible to populate all the Environment variables required as database values for Application Configurations. You can view the corresponding environment variable for helm values in the docs here

I will close this issue the above concluded alternative for you to test out! Thanks.


fix: fixed javadoc errors

Update javadoc source version to 8 and modified configuration to ignore 'missing' javadoc comments since missing unnecessary @returns are getting flagged as an error.
Fixes #909  ☕

@aeitzman the fix is too coarse. Half of the warning are due to missing @params, we want those fixed. Same for @returns. It might be optional for builder etc, but not in general.
I suggest we better fix the docs, then disable the check. You can cover only ExternalAccountCredential, CredentialAccessBoundary and I will cover the rest.


fix: update call to refine

Summary
The signature of the refine method in PriceRanges has recently changed. But the PriceRanges instantiate an other component PriceRangesForm that also use the refine method as prop.
The call of refine inside PriceRangesForm hasn't be changed, and throw an error when we try to preventDefault on the event that is actually a number.
You can try the PriceRanges component in the documentation, press Go and the component will throw an error.

The change with https://deploy-preview-2377--algolia-instantsearch.netlify.com/widgets/priceranges seems correct, no more page reloading


Unorganized Files

Hi, I'm not even sure the API allows it, but I just discovered I have a load of unorganised files (not related to GDO) in Google Drive, only visible with the following search in Drive:
is:unorganized owner:me
Would it be possible to add a feature to show these files, perhaps an .Unorganized folder, similar to .Trash?
Thanks.

I just published v0.6.11 (to the beta PPA) where you can access your unorganized files setting the new config option lost_and_found to true. You will then see a new directory /lost+found where you should find all your unorganized files. Warning: fetching those files is slow because there is no direct filter to get them. I have to fetch all files and then filter (client side) to get those without parents. I've also added a shared_with_me option to get shared files (that you can see in "Shared with me" section of the web interface).

This is awesome, thank you so much!

this seems to time out (maybe?) with no results when there's a large number of files.

If you remount with -debug, do you get any error in the log files?


feat(train_test_split): Add "time-based column" warning

This implementation has been tested for X is a pandas or polars dataframe, and checks whether the dtype of any column matches the string "datetime".
In the future we might refactor this through a compatibility layer like narwhals.
Because the new warning's message is dynamic (it includes the name or index of the first offending column), I took the opportunity to simplify the TrainTestSplitWarning interface: now, only the check method is expected, and it returns either None or a warning message string.
Addresses #684

Did you get a chance to look into polars' data parsing? For example, they have this which seems cool:
df = pl.read_csv("docs/assets/data/apple_stock.csv", try_parse_dates=True)

Reference

https://docs.pola.rs/user-guide/transformations/time-series/parsing/

I know about the flag, but I can't find a simple way to try_parse_dates in the polars API (i.e. something that is not writing to a CSV and reading it back...).
In the interest of time I've made the assumption that the dtypes in the passed dataframe are accurate; we can add more clever parsing later. Unless someone finds a simple way to parse the dates before this gets merged of course.
On the other hand, the link reminded me that we need to detect "Date" columns as well as "DateTime" columns: this is now done.


simple issue to fix

when i follow anybody their content is immediately inserted into my timeline shifting the timeline and losing the place where i was browsing ...
the simple solution is to NOT do this until the next manual refresh of the timeline so i can finish browsing my timeline in order ...
another solution would be to remember the place in timeline where i was browsing but the problem with that is the new content is not added all at once, but rather if there are maybe 20 notes that are added it will be 20 separate events added so inevitably the position in timeline will be lost ...
even if all 20 events are added at the same time the images in them will not all render immediately ...
most reliable solution would be to simply do nothing until next refresh ...
or maybe only add events BELOW the point where i was browsing so the timeline doesn't shift ...

Unfortunately this isn't really easy to fix due to how noStrudel handles replaceable events under the hood. It tries to only keep a single version of the event in the background and all UI components subscribe to that source when they need it.
So when the following list is updated, its not very easy to tell a single UI component that it shouldn't use the latest event until a specific time :(


[$250] Attachement - Console log error appears when sending an attachment

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: 9.0.11-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/45366
Action Performed:

Go to the staging app
Open Console
Copy any image to your clipboard
Open any report
Paste the image to the composer input by Ctrl+V
Press the 'Enter' key to send the message
Observe the console tab while the action is being performed

Expected Result:
No console error will show up
Actual Result:
Console error shows up
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/67d79d34-228e-4473-83dd-eb2a3153c6e4
logs (2).txt
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0108693be973624c6a
Upwork Job ID:<PHONE_NUMBER>007178354
Last Price Increase: 2024-07-25

Issue OwnerCurrent Issue Owner: @Pujan92

We think that this bug might be related to #vip-vsp

I have reproduced this
https://github.com/user-attachments/assets/7e81e76f-d392-40d7-9836-44e496e25a3f

This is a chat attachment bug, better suited for #vip-vsb.

@Pujan92 can you take a look? It's not clear to me if this can be resolved by an external contributor or if it has to be resolved internally, and that will help me decide if we're going to prioritize this.

@johncschuster We can let it open for the external contributors as I see some error in Onyx saving.

@Pujan92 I'm confused by your last comment. Can you clarify that for me? Do you feel this can be resolved by an external contributor, or do you feel we will need to take this internal?

@johncschuster It seems more like a front-end issue only, so we can keep this open for external contributors.

It seems more like a front-end issue only, so we can keep this open for external contributors.

Got it! Thanks for clarifying that! In that case, let's see if we can drum up some proposals!

bump for melv

bump for melv

Thanks @ardent-dev.
@johncschuster I am also not able to reproduce this now, can you plz confirm?

@Pujan92

I am also not able to reproduce this now, can you plz confirm?

Sure! I've just asked for a retest. If the issue isn't reproducible, let's close the issue.

Ok! I'm going to go ahead and close this one then. Thanks for the retest!


fix(api): do not handle smoothie alarms from halt()

When changing the smoothie driver in 628c6c47c79a656f9babd552784a96d99deb4852 to
be more robust and have better locking, the changes to the main comms sequence
around write_with_retries and handle_return started trying to handle and recover
from all smoothie alarms, including the one generated by a call to
robot.halt() toggling the estop gpios. This interferes with (read: breaks
catastrophically) protocol cancel, which relies on this sequence:
run() rpc call         smoothie      stop rpc call
  |                       |                |
  smoothie cmd start -> executing          |
  |                       | (estop) <-   halt
  smoothie cmd raises <- ALARM             |
  |                       |                |
  exc raised, call done   |                |
  (cancel)                |                |
                         recover     <-   M999
                         executing   <-  home...

and if the smoothie cmd in run handles the alarm and tries to recover, then it
never cancels the protocol.
To fix this, raise a different exception that the normal error handling
machinery doesn't catch and therefore propagates all the way up.

FYI, still does not resolve the cancel bug in api V2


Use better format to output the board including piece orientation

e.g.   12n,234w,123e,.... in order by lines with a dot for en empty place.
Then I could also create an app to draw the board..

Also, write out the empty position options better way, less space consuming


Duplicate headers

Hello,
Why KafkaHeaders.EXCEPTION_CAUSE_FQCN is equal to KafkaHeaders.EXCEPTION_FQCN ? Is it a  typo ?
https://github.com/spring-projects/spring-kafka/blob/c0e210c61eb78bb2f8a21f2970e3a5af56ab60f2/spring-kafka/src/main/java/org/springframework/kafka/support/KafkaHeaders.java#L252
https://github.com/spring-projects/spring-kafka/blob/c0e210c61eb78bb2f8a21f2970e3a5af56ab60f2/spring-kafka/src/main/java/org/springframework/kafka/support/KafkaHeaders.java#L246
And why having 2 sets of headers for DLT (DeadLetterPublishingRecoverer and @RetryableTopic DLT) ?
KafkaHeaders.DLT_EXCEPTION_CAUSE_FQCN KafkaHeaders.EXCEPTION_CAUSE_FQCN

Thanks; fixed. The DLT_ headers are for "classic" (blocking) retries with the default DeadLetterPublishingRecoverer.
The ones without the DLT_ are for non-blocking retries; they don't have DLT in the name because the intermediate topics are not dead letter topics, they are delayed retry topics.
There's an argument that says the final publish to the retryable topic DLT should have the DLT prefix but they don't.
Bottom line; use the DLT_ headers when not using retryable topics and the others when you are.


Need hasSize property for TabBarView or dynamic sizing for CustomScrollView

I have a Layout I want to do with Flutter and I have found a solution that works but gives me an overflow error. It seems to be impossible to solve.
The layout looks like this:
https://i.stack.imgur.com/a7fiU.png
Also a minimal example on dartpad:
https://dartpad.dev/bda4cc5fd2aea292310fe05daa440760
I know that people solve similar problems with NestedScrollView, but from an UX experience I'm more interested on a single unified scroll view with sliver properties (like zooming the top element on overscroll).
Does anybody know of a proper workaround or is there some planned features to make this layout solvable?

I am not sure if CustomScrollView and TabView are supposed to go that way as TabView is not a scrollable item, instead it contains scrollable items and as you have added it inside a CustomScrollView, it is absorbing the hits for tabview as well. As of now there are no widgets to solve the problem.

I am not sure if this is what you are looking for.

Hi @ichitaka,
You are getting the overflow exception because you've used Column widget which expands to the maximum available height. You'll need to use ListView or Wrap instead of Column to accommodate the list that you are showing in the section.
Using ListView, the text widgets are displayed properly and the screen becomes scrollable.
See if it works for you and answers your question.
Thanks.

Thanks @darshankawar for your answer.
So yes I have tried ListView before and I do have updated my DartPad with it. To make my point clear, it is possible to do a nested scroll view this way, but I actually want my screen to behave like a single scroll view. So by scrolling with the content of my TabBarView, I also want to scroll my SliverAppBar and all the other members of my CustomScrollView. Right now with this solution, if I scroll the content of my TabBarView, only my ListView will scroll.

@ichitaka Thanks for clarification.

This issue doesn't appear to be actionable right now, the dartpad from the OP does not show code anymore. Can someone share an updated example of the issue?


clustering technology

clustering technology  and giving mark on course that no listed on udacity anymore

Definitely should remain posted as it, at the very least, allows the opportunity to continue utilizing Udacity's initial purpose. Understandably so, the course listing will naturally become outdatrd with time.

Finally getting back to making a few changes on this repo. At this time, I am going to focus on these changes before any additional reorganization is added. I will most likely incorporate your concept of listing the courses in a more sane fashion but likely won't group them. Feel free to update and reopen once my updates are live if you feel any of your changes still apply and will make for a positive change. Thanks!


Users should be able to log in from the landing page

Problem
Unlike new users wanting to sign up for an account, existing users are required to navigate away from the landing page by clicking on the 'Create an Account or Sign-in' button at the top right of the navbar if they wish to log in to their accounts.  This unintuitive and does not follow common patterns found on most mainstream web services.
Solution
Replace the 'Create an Account or Sign-in' button on the navbar with a sign-in widget on the landing page. All of the functionality that currently exists (standard username/password, two-factor input, and forgotten passwords) can be incorporated.

Notes

This will not remove the page users currently use to sign in. It can still be used for redirects when unauthorized access is attempted e.g. a user who is not logged in navigates to a bookmark he/she has for a private project/component.
Two-factor input -- there are two routes for allowing users to enter two-factor codes
Expand the new widget to include a third, optional field for two-factor codes like what currently exists.
Create a more standard widget (only username/password) and redirect users that have two-factor auth enabled to another page after entering username/password where they can enter their two-factor code. This seems to be a common pattern.
I plan on working on this feature request

Input requested!

Rather than replacing the nav bar widget, what if you made the sign up box into a tabbed area defaulting to sign up that you could switch to log in?

@brianjgeiger interesting idea. I think this (and most UI) things should go up for a vote related to what 'feels' the most intuitive. Let's get some of the others to weigh in before I start working.

@caneruguz likes to be involved in things like this.

I like to be able to log in to a site without having to do any navigation whatsoever, and I don't really see much of a functional difference between clicking a link and clicking a tab.

I agree with @fabianvf . Facebook, Twitter, Google all do it this way and we know they've done the research.

I think this update should involve generally rethinking the sign up process and should include the current login page as well that has three different purpose forms in the same place. I think an incremental small step doesn't go far enough to improve the user login experience and an overhaul would need more time.
Having said that I'm not sure to what extent this fix aligns with our 3 month priorities. I'm wondering whether it would be best to start work on this in the next 3 month sprint.  I would ask Jeff about his thoughts on the timeline for a project like this.

@HarryRybacki I think I mis-interpreted your original proposal, my apologies. That sounds like a good plan. I'll be interested in how you do the two-factor, though. Three fields?

@brianjgeiger Do we have a way of easily determining what percentage of users have two-factor enabled?

Should be a simple db query. I don’t know what that query is off the top of my head. I expect the number to be very small right now.

Josh is grabbing that info for me. If it's small I'll opt for two fields (username/password) with a forward for two-factor to keep the landing page as clean as possible. This can be changed later on if the percentage grows.

Based on past discussions, the login procedure will have to be re-worked more that you may be anticipating in order to do a two-stage login. @lyndsysimon has been considering doing that re-work, but hasn't had the time, so you might want to check with him about that.

Will do, thanks!

If that doesn't work out, you could have the two factor field drop down, possibly with a submit button, if focus is in the username/password field. Or something similar.

@HarryRybacki, issue #879 has some discussion about re-working the login process that you might be able to address as well.


Using Lravel propmts in panel builder commands.

[ ] Changes have been thoroughly tested to not break existing functionality.

As Laravel now ships with Laravel Prompts, we can use laravel prompts to take user inputs from terminal.
This PR will make uses of prompts in panel builder commands.

Please test your work, as there are syntax errors here in the code.

Please test your work, as there are syntax errors here in the code.

I dont know why, but github desktop was not pushing the changes, this is wired.

@danharrin Hi,
How to pass this test?
Error: Used function Laravel\Prompts\select not found.
Error: Used function Laravel\Prompts\text not found.

It's because prompts doesnt support L9. Livewire is also dropping L9, so I we will have to soon too.

Thanks


backup all pipe if user not specify input in pipe-promot

This is the functionality where the spinnaker will backup all applications and pipelines to the git repo when the user runs the backup pipeline without any input provided using pipeline promotion.

LGTM


[cspan] append 'Referer' header with the video URL

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Let C-Span info extractor insert 'Referer' header with the video URL into the format entries.
Fixes #25729

dstftw can you pls add this fix soon maybe next release as still showing error for c-span HTTP Error 403: Forbidden
the only way i can get working is using m3u8 i find and --referer
youtube-dl --no-check --no-check m3u8 --referer c-span url
my issue below id form time ago still not fixed latest version
https://github.com/ytdl-org/youtube-dl/issues/23148


Added Fibonacci In Scheme

Good afternoon!
For some info, check out the README, I have included some stuff in there.
It is my first time trying to contribute to a project on github, so I am learning as I go along.
One thing I noticed is that my scheme_fibonacci file, even though I deleted it from the fibonacci-series folder, still looks like it is there and I can't figure out how to get rid of it. (I want it to only be in the Scheme folder) so if you know how I would go about making those changes it would be greatly appreciated and I'll make the necessary changes.
Since I'm new to this, if there is anything I should be doing differently/you need me to do, I would greatly appreciate the pointers and I would be happy to make any changes/edits that you need.
-London

Well, I just realized that I could have added the test video and the read me file to the pull request instead of adding them to the folder.
To be honest, I can't seem to figure out how to delete them, so any help with that would be great.
I am attaching the video and the read me to this comment so that they will still be here after deleting them from the folder.
https://user-images.githubusercontent.com/91155492/136671044-9dad5f5d-dbb1-4946-b559-c2c44887a08e.mp4
Scheme_README.txt


fix/strain2-v208

This PR is required to align the xml files to the latest version of the strain2 board which is 2.0.8
The change to 2.0.8 was done only for those xml  files which requested the previous version 2.07, so that the system can be aligned.
Here is the log change of the new firmware build for strain2

fyi @isorrentino @Gianlucamilani @S-Dafarra


Isues installing HCXDUMPtools and hcxtools.

hi all. i am keep getting errors while installing hcxdumtools, hcxtools and hashcat.
the error i am gettitng for the hcxdumptool is as below:
cc -O3 -Wall -Wextra -std=gnu99  -o hcxdumptool hcxdumptool.c
/usr/bin/ld: cannot open output file hcxdumptool: Is a directory
collect2: error: ld returned 1 exit status
make: *** [Makefile:29: build] Error 1
can someone please specify the steps by steps how to install the above tools.
thank you

delete all directories with hcxdumptool, hcxtools and hashcat.
do a fresh git clone for every tool:
git clone https://github.com/ZerBea/hcxdumptool
git clone https://github.com/ZerBea/hcxtools
git clone https://github.com/hashcat/hashcat
cd into every folder and run make install:
cd hcxtools
make install
cd ..
cd hcxdumptool
make install
cd ..
cd hashcat
make install
cd ..
forget the youtube videos where all tools are copied together into one directory.
This is an issue tracker. If you have questions, please ask here for help:
https://hashcat.net/forum/
https://hashcat.net/forum/thread-6661.html
Closed, because it ins't an issue.

same problem?...

This issue was caused by a broken installation.
possible reasons:

user copied source code from all tools into one folder
user used source code folder as working directory

I assume OS is KALI!
@huitc you're running Android, but a required include file wasn't pulled. So you have to pull it and copy it to include folder.

Hi i am trying to clone, but it's asking for username and password. Where can I find those?

Use the https url instead of the SSH one.

No password required to clone hcxtools/hcxdumptool:
$ git clone https://github.com/ZerBea/hcxdumptool
Klone nach 'hcxdumptool' ...
remote: Enumerating objects: 46, done.
remote: Counting objects: 100% (46/46), done.
remote: Compressing objects: 100% (31/31), done.
remote: Total 1336 (delta 26), reused 34 (delta 15), pack-reused 1290
Empfange Objekte: 100% (1336/1336), 460.31 KiB | 1.09 MiB/s, Fertig.
Löse Unterschiede auf: 100% (879/879), Fertig.

Hi, thanks. I was able to clone hcxdumptool amd hcxtools and hashcat but when i complie;
1.hcxdumptool using
cd /usr/local/bin/hcxdumptool
make
make install
then it gives error " install: "hcxdumptool" and  /usr/local/bin/hcxdumptool are the same name file.
make***:[makefile:29: install error 1]

After installation of hcxtools
When i go to
/usr/local/bin/hcxtools
make
then it says: fatal error curl/curl.h no file or directory found.
#include <curl/curl.h>

After clonning hashcat;
when i go to cd /usr/local/bin/hashcat
then it gives error

" install: cannot overwrite directory /usr/local/bin/hashcat with non directory.
make***:[src/makefile:455: install hashcat] Error 1

Your workflow is an absolute no go!

first install dependencies:
To install requirements on Kali use the following 'apt-get install libcurl4-openssl-dev libssl-dev zlib1g-dev'
never clone to your binary directory /usr/local/bin/ !!!!!
Instead cd to your home directory and do a git clone, followed by a cd to the hcxtools folder. Than run make and make install (as super user)
cd $HOME
git clone https://github.com/ZerBea/hcxtools
cd hcxtools
make
make install

cd $HOME
git clone https://github.com/ZerBea/hcxdumptool
cd hcxdumptool
make
make install
For hashcat you must install GPU driver and opencl runtime!!!
https://hashcat.net/wiki/doku.php?id=frequently_asked_questions#installation
cd $HOME
git clone https://github.com/hashcat/hashcat
cd hashcat
make
make install
Read more here:
https://null-byte.wonderhowto.com/how-to/hack-wi-fi-cracking-wpa2-passwords-using-new-pmkid-hashcat-attack-0189379/

@ZerBea, thanks a lot. I did it. :)

Fine.

gracias amigo !!!!

No hay de qué.

I am getting an error while running the 'apt-get install libcurl4-openssl-dev libssl-dev zlib1g-dev' command:-
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
libssl-dev is already the newest version (3.0.13-1~deb12u1).
libssl-dev set to manually installed.
zlib1g-dev is already the newest version (1:1.2.13.dfsg-1).
zlib1g-dev set to manually installed.
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
libcurl4-openssl-dev : Depends: libcurl4 (= 7.88.1-10+deb12u6) but 8.8.0-1~bpo12+1 is to be installed
E: Unable to correct problems, you have held broken packages.
Actually, I want to run make and make install for hcxdumptools and hcxtools but get the following error:-
hcxdumptool.c:41:10: fatal error: pcap/pcap.h: No such file or directory
41 | #include <pcap/pcap.h>
|          ^~~~~~~~~~~~~
compilation terminated.
make: *** [Makefile:75: hcxdumptool] Error 1
Any fix for the situation?

@ChaarSaal
I'm not certain what documentation you are reading as current installation instructions should be as follows:

Update your system to it's latest: sudo apt-get update && sudo apt-get upgrade -y
Install required system packages: sudo apt-get install git gcc libpcap libpcap-dev

Note: These packages may come under different names! Make sure you're installing the correct packages!

Clone the repository: git clone https://github.com/ZerBea/hcxdumptool.git
cd into that directory: cd hcxdumptool
Compile and install: make -j $(nproc) && sudo make install

From here, the tool should be installed and ready for use.

@ChaarSaal
By default, Debian based distributions are not designed to compile and install additional tools (e.g. hcxtools, hcxdumptool).
If that's what you want, it is mandatory to install the development tools and the development files (as mentioned by @Jake-Grafton ).
Unfortunately your package manager is broken on your distribution and you got this error;
E: Unable to correct problems, you have held broken packages.
To fix that please take a look at the documentation of your distribution or e.g. here:
https://superuser.com/questions/1793779/parrot-os-unmet-dependencies-held-broken-packages.


dvorak layout is missing two remaps

I just tried using the new layout mechanism (which is great by the way!) and noticed that the shipped dvorak layout is missing the two remaps on the number row (- and = are mapped to [ and ], respectively). As you probably generate the layout files automatically, I didn't send a pull request.

Thanks. They are indeed generated (by scripts/dump-xkb-config). This should be fixed in the latest commit.


Tag replacement not working

From the forum: https://ghost.org/forum/bugs-suggestions/2411-editing-the-character-case-of-tags/
How to reproduce this bug

Create a new post with a tag of "example"
Publish this post
Edit this post by removing our "example" tag and adding "EXAMPLE" in its place
Reload the blog page to see that neither the "example" nor "EXAMPLE" tags exist on your post.
Edit the post once again to add back in our "EXAMPLE" tag which shouldn't have been removed.
Reload the blog page to see that now both "example" and "EXAMPLE" exist on the page.
Remove the "example" tag to fix the issue.

I initially published it with a lowercase "css", after noticing that this didn't look good I edited my post to remove the "css" tag and add in a new "CSS" tag. Upon clicking save neither of the tags had been added. When editing the post for a second time to once again add the "CSS" tag, both the "css" and "CSS" tags ended up being added to the post. Afterwards I was able to remove the "css" tag but keep the "CSS" tag.

I'm still having trouble with this. When I type in either cased tag, both show up after I save.
I'll do a screencast if needed. I don't mind if it will help.
I have /tags/tldr/ & /tags/tldr-2/. On my post it shows both tag:
https://www.dropbox.com/s/x39xsi9lnk6jt0t/Screenshot 2015-01-12 02.08.54.png?dl=0

@chancesmith That's because they are, essentially, both the same tag. In the next version of Ghost there will be a new interface where you can edit your tags to fix this problem.


add use=coverage option to Make to obtain coverage information

from runs of ponyc or the test commands
both for gcc (using gcov and lcov) and for clang (using llvm-cov).
gcc and gcov and lcov
# compile ponyc
CC=gcc CXX="g++" make config=debug use=coverage

# run some test or ponyc
./build/debug-coverage/libponyc.tests --gtest_filter='LexerTest.*'

# generate coverage report
lcov --directory .build/debug-coverage/obj –zerocounters
lcov --directory .build/debug-coverage/obj --capture --output-file ponyc.info
genhtml -o build/debug-coverage/coverage ponyc.info

# open build/debug-coverage/coverage/index.html

clang and llvm-cov
# compile ponyc for coverage
CC=clang CXX="clang++" VERBOSE=true make clean all config=debug use=coverage

# run the program for which you want to obtain coverage information
LLVM_PROFILE_FILE="build/debug-coverage/coverage.profraw" ./build/debug-coverage/libponyc.tests --gtest_filter='LexerTest.*'

# generate coverage data
llvm-profdata merge -sparse -output=build/debug-coverage/coverage.profdata build/debug-coverage/coverage.profraw

# show coverage data (only for lexer.c in this case)
llvm-cov show ./build/debug-coverage/libponyc.tests -instr-profile=./build/debug-coverage/coverage.profdata src/libponyc/ast/lexer.c

Question: I would love to provide the step by step guide in some kind of file. Does anyone have a preferred place fot this?

Hm, I'm not sure - it doesn't look like we currently have a place for such things.
@SeanTAllen any ideas?

I have used a DEVELOP.md at several projects now, containing all information for developers and people who want to be. Stuff like a C styleguide, how to test PRs locally, and useful stuff like this coverage instructions might have a cosy home there.

@mfelsche a lot of that info is in contributing. i think this could make sense there as well.


bug: the button that expands the followers list found at the top is not labeled according to my screen reader.

Bug
When tabbing or navigating wiht the arrow keys, you will hear "button" at the very top of the screen, if using a screen reader. when pressing this button, the list of who you are following will appear. The button has no label, thereby making it harder for a screen reader user to find the button and go to their follower list.
To Reproduce
I'm using nvda which can be found ere.

launch nvda.
launch the lbry app or the odysee website.
turn off your screen.
use the keyboard to navigate to the "button" It will say "button."
hit enter on the button.
tab or use arrows to find your followers list.

Expected behavior
I should hear what that button does. "see followers" o or what ever that label actually is supposed to be.
Screenshots
If applicable, add screenshots to help explain your problem.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Additional context
n/a

Thanks for reporting this @marrie
I will make sure this is fixed soon

I'm still able to reproduce this under mac and windows. Thanks so much and happy Sunday.

Thanks for the fix. That was fast.￼

Thanks for the fix. That was fast.￼

Thanks for closing the issue! I forgot about that. 🙂
If you run into any more screen reader issues please create another issue so we can fix it.

Thanks for closing the issue! I forgot about that. 🙂
If you run into any more screen reader issues please create another issue so we can fix it.


webapp: list-publishing-profiles valid query returns empty array

This is autogenerated. Please review and update as needed.
Describe the bug
The command works fine when not using any query.
When using the command with --query [?publishMethod=='FTP'], the tool returns an empty set.
Succsessfully tested query on http://jmespath.org
Command Name
az webapp deployment list-publishing-profiles
Errors:
It returns [].
To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az webapp deployment list-publishing-profiles --id {YOUR-ID-HERE} --query [?publishMethod=='FTP']

Expected Behavior
Returning just the profile for publishing with ftp.
Environment Summary
Windows-10-10.0.17134-SP0
Python 3.6.6
Shell: powershell.exe

azure-cli 2.0.64

Additional Context

cc @AzureAppServiceCLI @antcp

@JKamsker we are unable to repro the issue in latest CLI version (2.11.1) . could you please try with the latest CLI version and let us know if you still face the issue.

Please reopen if this still repros with latest - closing due to lack of response.


Added files and images

Description
I just added two files and some images
for **SPOOKY FUN ** :smile_cat:

😀😀😀


[experiment] Try use CMake interface library to simplify some of the cuda libs.

Firing a diff to see if this actually works across platforms.

Interesting finding. This only works with cmake3 because INTERFACE library is not available in 2.8.

@pietern Maybe we should enforce cmake 3 :)

@caffe2bot retest this please.

Ubuntu cmake3 install is pending a docker build trigger. Rest is passing properly (per centos7 build).

:heart:


Document TestBinder

As a developer, I want to be able to have a guide on using the TestBinder, so that I can write tests for my module that do not rely on specific middleware.


Whisper

A new function in the |tell series. Basically the same, but sends as PMs instead.
|whisper
Suggested by Trahsi

Sort of done by making Tell send as PMs if you PM them to MoronBot.


Describe Feeling View - Text box can go under keyboard

The height of the text box means that typing can go under the keyboard and even off screen

Closing as this behavior has now been improved


cherrypick of "KFP - Updated the documentation regarding metrics #2171"

/assign @Ark-kun @8bitmp3
website now shows 1.1-branch by default, so let's cherry pick

Forced cla because @Ark-kun is a Googler

Hi @8bitmp3! Thank you for the detailed review, but this is a cherry pick PR, so we shouldn't update anything here.
Do you want to submit a PR with your suggestions to master branch?

/lgtm


Toggle mode support

ToggleMode Support Added

[ ]  For developers who use phpstorm/webstorm .idea/* ignore added to ignorefile.

[ ]  If settings.animation parameter includes toggleMode arguman and it's true toggleMode comes active. To close animation you need to call setLoadingScreenStatus(false) function and to reactive it call setLoadingScreenStatus(true)

Useful and needed function/feature added


Schedule the remote start and stop

Hello everyone!
I have created a charging point and CSMS scripts successfully with a structure similar to the examples you have in the repository. Now what I want to do is to do the remote start and stop charging but at a specific hour: e.g. at 17:30 I want to do the remote start procedure and at 19:30 that the charging will be over according to a timetable, I want to do the remote stop charging. However the whole asynchronous logic is still a bit confusing to me. Do you have any ideas of how should I approach the subject?
Thanks!

Can you share those scripts?

Maybe you can make an API using fastAPI and then trigger a remote start request at a particular time interval using scheduler or asyncioscheduler.


Getting 'StructuredQuery.from cannot have more than one collection selector' from reactive repositories

Spring boot parent version  - 2.3.2.RELEASE
Spring cloud version - Hoxton.SR6
When running load test on my application getting below exception quite frequently.
org.springframework.cloud.gcp.data.firestore.util.ObservableReactiveUtil$StreamingObserver.onError(ObservableReactiveUtil.java:126) at io.grpc.stub.ClientCalls$StreamObserverToCallListenerAdapter.onClose(ClientCalls.java:453) at io.grpc.internal.ClientCallImpl.closeObserver(ClientCallImpl.java:426) at io.grpc.internal.ClientCallImpl.access$500(ClientCallImpl.java:66) at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl.close(ClientCallImpl.java:689) at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl.access$900(ClientCallImpl.java:577) at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$1StreamClosed.runInternal(ClientCallImpl.java:751) at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$1StreamClosed.runInContext(ClientCallImpl.java:740) at io.grpc.internal.ContextRunnable.run(ContextRunnable.java:37) at io.grpc.internal.SerializingExecutor.run(SerializingExecutor.java:123) at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) at java.base/java.lang.Thread.run(Unknown Source) Caused by: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: StructuredQuery.from cannot have more than one collection selector. at io.grpc.Status.asRuntimeException(Status.java:533)
INVALID_ARGUMENT: StructuredQuery.from cannot have more than one collection selector. at io.grpc.Status.asRuntimeException(Status.java:533)
I am not querying on multiple collections in a single query, in-fact I am extending FirestoreReactiveRepository in it's plain form for my collections and writing methods in it for query (just like JPA). I am getting these errors for 3 repositories in my 2 Spring boot services, so I assume it won't be problem with just one of my collection. This happens when there is huge load on service, running small bunch of requests work fine.

@dmitry-s PTAL.

@meltsufin @dmitry-s Any updates on this issue. As I am also getting same issue, although I am accessing single collection only?
When I attempt to access firestore, I first get error "{io.grpc.StatusRuntimeException: UNAVAILABLE: io exception}" and later when I retry same code then get error "{io.grpc.StatusRuntimeException: INVALID_ARGUMENT: StructuredQuery.from cannot have more than one collection selector.}"

Also this issue is not consistent, same code works fine for all cases but sometimes gives this in consistent behavior.
I am also using FirestoreReactiveRepository

@manishjain5238 Could you provide the source code so we can try to replicate the issue?

@dmitry-s , I experienced this issue and managed to resolve it, but I'm not sure exactly why my solution worked. Here's some code roughly the same to what was causing my problem:
  private Mono<SeasonMetadata> getSeasonMetadata(Timestamp timestamp) {
    Flux<SeasonMetadata> seasonMetadata =
        seasonMetadataRepository.findByEndGreaterThan(timestamp);
    Mono<SeasonMetadata> seasonMetadata =
        seasonMetadata
            .filter(metadata -> metadata.getStart().compareTo(timestamp) <= 0)
            .next();  // <- annotation 1
    return seasonMetadata;
  }

  public Flux<SeasonStats> updateResults() {
    Mono<SeasonMetadata> currentSeasonMetadata = getSeasonMetadata(Timestamp.now());
    Flux<Player> players = playerRepository.findAll();

    return players.flatMap(  // <- annotation 2
        player ->
            currentSeasonMetadata.flatMap(
                metadata -> {
                  Stats parent = new Stats();
                  parent.setId(metadata.getId());
                  SeasonStats playerStats = new SeasonStats();
                  playerStats.setPlayerId(player.getId());
                  return firestoreTemplate.withParent(parent).save(playerStats);
                }));
  };

The call to FirestoreTemplate::save  in updateResults() was not successful. When I changed the call of next() instead to publishNext() at the line marked annotation 1, then the problem went away. I realized also that I should invert the nested flatMap calls at annotation 2 to look like:
currentSeasonMetadata.flatMapMany(
        metadata ->
            players.flatMap(
                player -> {

^ And in that case, either next() or publishNext() worked fine.
However, since I'm somewhat new to Reactor, I'm not sure how this could cause the query to be built incorrectly. Any updates would be appreciated. Thanks!

@mhbarney I tried replicating the issue using your code, but it works fine for me.
Would you be able to provide an application that reproduces the error?
Thanks!

@mhbarney I finally was able to reproduce it. I'll let you know when we have updates.

@mhbarney the issue is caused by double subscribing to the currentSeasonMetadata. It happens because you have currentSeasonMetadata.flatMap inside of players.flatMap. That Mono doesn't currently support double subscription, and I will create a fix for allowing that.
But in this case I think you don't actually want to fetch new currentSeasonMetadata for every player, so my suggestion would be to call chache() method on that Mono, like so:
    Mono<SeasonMetadata> currentSeasonMetadata = getSeasonMetadata(Timestamp.now()).cache();

And that should fix your issue.
Let me know if that works for you.
Thanks!

@dmitry-s thank you, using cache() also worked. I'll also take a look at the code changes you made to learn more.


Algorithm runs exceptionally slow

With addition of 2 children per crossover, alg runs insanely slow. Looking into issue now

Selection was chunking parents every 1 step, doubling the number of parents necessary. Fixed with last commit.


⚠ Diverse AirCon & Mechanical has degraded performance

In 337d93c, Diverse AirCon & Mechanical (https://diverseaircon.com.au) experienced degraded performance:

HTTP code: 200
Response time: 13122 ms

Resolved: Diverse AirCon & Mechanical performance has improved in 2c5ef8d.


Pkg Extensions: EvoTrees, MLJDecisionTreeInterface, NearestNeighborModels, LaplaceRedux

Once the Data folder has been moved out to TaijaData, it will become possible to also use an extension for MLJBase. Noting this down here as a future TODO.

@RaunoArike Data folder has been just migrated to TaijaData


Fix dead link

The other link in the first post died too.

Hi there;
This is an automated reply, to let you know LineageOS doesn't merge pull
requests submitted directly to our github.com repositories.
You can submit your change requests through our public review system,
located at https://review.lineageos.org/
That review system is an installation of Google's Gerrit code review.
If you've never interacted with Gerrit before, we also suggest you take
a look at http://wiki.lineageos.org/usinggerrit-howto.html
Thanks!


Allow account_linking type event to propagate from webhook to allow handling

First of all thanks for the awesome library!! Nicely written and architected!
Facebook's latest Webhook reference (https://developers.facebook.com/docs/messenger-platform/webhook-reference/account-linking) supports account linking process to Link Messenger account to 3rd party accounts. It will be nice If Bootbot will allow account_linking type event to propagate.
This is a one line change, that I will be happy to submit, or feel free to add it or let me know if it does not make any sense.
In Bootbot.js   in _initWebhook function:
  if (event.optin) {
......
             } else {               console.log('Webhook received unknown event: ', event);             }
before the final else add a new line
    if (event.optin) {
....
 } else if (event.read) {
              this._handleEvent('read', event);
           }else if (event.account_linking){   // Add this to support account linking
            this._handleEvent('account_linking', event); // Add this to support account linking
           } else {
              console.log('Webhook received unknown event: ', event);
            }
Thanks in advance

Hey @ShoukriKattan, thanks for the suggestion! I've added support for account_linking events on the latest version: 1.0.6
Thanks again, and feel free to submit a PR next time :)
Diff: https://github.com/Charca/bootbot/commit/5feabaac3d1a313114504e1aa5ec8825bb88235d


Alias names to ".true" and ".false".

These are tokens, so in some usages they fail...
For instance with how I use AVA, I destructure the functions at the top.  This way I don't have to prefix every operation with t. (which looks ugly in my opinion).  So I do this:
test('test name', ({is, same, not}) => {
  // is(), same(), not()
});

But, whenever you when using functions with token-like names, it fails.  For example, this destructuring fails getting .true:
test('test name', ({is, true, not}) => {
  // is(), true(), not()
});

SyntaxError: Unexpected token true
  at requireFromString (~/projects/rpv/node_modules/require-from-string/index.js:26:4)

Would be nice if there were some alias functions for any token-similar named functions for cases like this...  .true() and .false()

I was playing around, and discovered if you reassign the name in the destruction, it doesn't fail.  For instance:
test('test name', ({is, true: isTrue}), {
  // is(), isTrue()
});

This passes.
So you can choose to merge this or not.

It seems like you have found a good solution to your problem. I don't think it is a good idea for us to start aliasing methods. Eventually we will introduce the ability to fully integrate custom assertions. When that happens, you will be able to modify our default assert plugin with the aliases you desire and apply it once across your entire project.

I don't see the point of using destructuring when it's more verbose than just doing t.true(). We're also not interested in adding aliases as we want to keep it consistent.
https://github.com/sindresorhus/ava/issues/114#issuecomment-152706993

Sounds good :+1:


ResourceManifest::Get should be optional

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest version
[X] Search the existing issues.

Steps to reproduce
With addition of Export functionality, that may be the only functionality that a resource implements;
however currently specifying 'Get' in the manifest is mandatory, which produces an error if resource does not support it:
PS C:\DSCv3> dsc resource export -r Microsoft/ProcessList
Error: Manifest: C:\DSCv3\bin\debug\process.dsc.resource.json
JSON: missing field `get` at line 9 column 1

Expected behavior
NA

Actual behavior
NA

Error details
No response
Environment data
NA

Version
v1
Visuals
No response

Should get be optional, or should the manifest have a property like export that validates against a subschema like:
title:       Resource Exporting
description: |
  Defines whether the resource can be used to export instances. When this value is `true`,
  the resource is processed normally with the `dsc resource export` and `dsc config export`
  commands. When this value is `false`, the resource is ignored. When this value is `only`,
  the resource is invalid for other operations, like `get` and `set`.
type:    [boolean, string]
enum:    [true, false, only]
default: false

And then an export-only manifest would define as either:

Top-level keyword:
{
  "export": "only",
  "get": {
    // get definition
  }
}

Keyword in get:
{
  "get": {
    "export": "only",
    // get definition
  }
}

Are there times where we would expect the export to be something different from get, except for whether instance properties are passed to the resource? Do I have to define export and get fully to support both, or can I do something like this?
{
  "get": {
    // get definition
  },
  "export": true
}

Instead of:
{
  "get": {
    // get definition
  },
  "export": {
    // same as get definition
  }
}

It's also not clear to me a real example where a resource would only support export.  It means it can't be used for auditing and if it implements export, it shouldn't be much more to support get.


How to setup cockpit to work only from its domain?

Hello,
I will use Cockpit API on frontend with visible API key (not full access) but i dont want developers to have ability to request my data from anywhere.

On root directory there's an index.php.
You can find:
header("Access-Control-Allow-Origin: *");
Remove the star and put your own domain there.
header("Access-Control-Allow-Origin: example.com");

@6feferonka9 it's possible to configure cors by editing the file config/config.yaml or in Settings > System Settings through the UI:
# Cockpit settings
cors:
  allowedOrigins:
   - example.com