Can't compile/build via nix flake with homemanager | udis86

Regression?
No
System Info and Version
 System/Version info
This is my "RUNNING" version - see problem description below.
Hyprland, built from branch main at commit 918d8340afd652b011b937d29d5eea0be08467f5  (flake.lock: update).
Date: 2024-06-25
Tag: v0.41.2, commits: 918d8340afd652b011b937d29d5eea0be08467f5

flags: (if any)

System Information:
System name: Linux
Node name: XXX
Release: 6.6.36
Version: #1-NixOS SMP PREEMPT_DYNAMIC Thu Jun 27 11:49:15 UTC 2024

GPU information:
03:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Cezanne [Radeon Vega Series / Radeon Vega Mobile Series] [1002:1638] (rev c5) (prog-if 00 [VGA controller])

os-release: ANSI_COLOR="1;34"
BUG_REPORT_URL="https://github.com/NixOS/nixpkgs/issues"
BUILD_ID="24.11.20240703.9f4128e"
DOCUMENTATION_URL="https://nixos.org/learn.html"
HOME_URL="https://nixos.org/"
ID=nixos
IMAGE_ID=""
IMAGE_VERSION=""
LOGO="nix-snowflake"
NAME=NixOS
PRETTY_NAME="NixOS 24.11 (Vicuna)"
SUPPORT_URL="https://nixos.org/community.html"
VERSION="24.11 (Vicuna)"
VERSION_CODENAME=vicuna
VERSION_ID="24.11"

plugins:

======Config-Start======

<skipped>

======Config-End========

Description
While building in nixos (home-manager). This happens in the configuration stage.
hyprland>   The source directory
hyprland>
hyprland>     /build/source/subprojects/udis86
hyprland>
hyprland>   does not contain a CMakeLists.txt file.
hyprland>

How to reproduce
Add a plugin to hyprland (or disable build cache). It does not compile, it does not find udis86.
My relevant flake inputs:
    hyprland = {
      url = "github:hyprwm/Hyprland/v0.41.1?submodules=1";
      #url = "github:hyprwm/Hyprland";
      inputs.nixpkgs.follows = "nixpkgs";
    };
    # This does not even build yet, since hyprland fails before, but just for completeness sake:
    Hyprspace = {
      url = "github:KZDKM/Hyprspace";
      inputs.hyprland.follows = "hyprland";
    };

My hyprland homemanager nix:
  wayland.windowManager.hyprland = {
    enable = true;
    plugins = [
      inputs.Hyprspace.packages."x86_64-linux".Hyprspace
    ];

If I skip the plugin setup, it uses a cache and I can use it successfully - so just compiling locally for some reason does not work.
Crash reports, logs, images, videos
No response

See https://github.com/hyprwm/Hyprland/issues/5891. github:-style URLs don't work.


Search and Add Button align

#5
Buttons are not aligned properly - Improvement
Resolved: Remove Margin From Add Opportunity to make it align.

Good job, @NPriyajit! Just merged your pr.


Feat/sunmao start

Features

Move the modules' schemas to individual files
Use the color font as the demo component
Use the animation trait as the demo trait
Support i18n
Support router
Support creating the application, component, trait, and util method by NPM scripts

Preview
https://user-images.githubusercontent.com/25414733/177118106-908a6f9c-c97c-4c24-af6d-72e0862b83ef.mov
https://user-images.githubusercontent.com/25414733/177118740-6a02c235-d827-4b1d-9c27-58b7646891f2.mov

Very good!


Use FancyZones on Multi Monitor Setup

Summary of the new feature/enhancement

Currently when I'm using more than one monitor, it is not possible to set individual Zones for each monitor. It would be a nice feature if i coud set up fancy zones for each Monitor.
Proposed technical implementation details (optional)

Yes I can only seem to add zones to my primary monitor, As I use a lot of apps in work it would be ideal to assign zones on multiple displays

Hi @GeorgHoffmeyer, @Grichar89
we are tracking the multi monitor support in a dedicated issue https://github.com/microsoft/PowerToys/issues/195.
Feel free to add your contribution there, I'm closing this one as duplicate.
Thank you.


Add CTA label to Teaser links

Part of #277 and #309
Create input component for adding a CTA label to the Teaser component
Render the CTA label in the component
Have fallback in the link component with original 'Read more' text
Acceptance criteria

Can add a label for the link/download.
Label is mandatory in validation.
~Label text is rendered in teaser component~

Duplicate of #332 (oops)

Works fine. Will probably be even more refined with #332


Adjust query API URL in docs depending on context

 Update GraphQL documentation hosted on non-production API so it shows the other endpoints, preferably dynamically #small


[labs/analyzer] More analysis coverage for gen-manifest

This PR includes the following:

CLI improvements:

Print generate command diagnostic errors more gracefully to CLI - https://github.com/lit/lit/pull/3529/commits/8d7b0e976d7eace4b8ff94b61e72994d0bef9c5a
Add support for --exclude options (e.g. exclude tests): https://github.com/lit/lit/commit/7fac64ce9efad4852bc1c5fd592aa81708bf28b0) - important for excluding test files from e.g. manifest or wrapper generation

Add more analysis support and CEM emit:

TS enum type variables: https://github.com/lit/lit/commit/bb8ba3151e559cc197061aa2da8dc23dd61c2fda
description, summary, and deprecated for all models: https://github.com/lit/lit/commit/de76b2e891190c0d187c97388aaebb9a0d97b4c3
module-level description & summary: https://github.com/lit/lit/commit/32145ba9ef972c002874c2557cf97f4ce8bf0cf6
ClassField and ClassMethod: https://github.com/lit/lit/commit/3fd322c250f620feccd57c688044811820d8b71d

Also introduces ...IfNotEmpty() helpers in gen-manifest to trim down the JSON size by only including optional fields when they have with meaningful values (e.g. non-empty string, non-empty array, true boolean, etc.)

This PR successfully builds the following manifest for MWC: custom-elements.json, after making the following minor MWC source changes:

menu/lib/menu.ts

export ActionDetail interface
remove unused? {SelectedDetail} type from @fires annotation

Addresses much of remaining items in #2993, with exception of:

FunctionDeclaration
MixinDeclaration / CustomElementMixinDeclaration
Attributes (of LitElementDeclaration)

I noticed that some of the privacy labels in custom-elements.json are incorrect. For example firstUpdated is noted as public. Is that because the source is not properly including override or does privacy need to be brought forward from the base class?


Commenting range doesn't show on untitled file sometimes

https://github.com/microsoft/vscode/issues/176498#issuecomment-1526240359

@TylerLeonhardt did you make any changes outside those in the repro repo for https://github.com/microsoft/vscode/issues/176498#issuecomment-1526240359? I'm having trouble reproing it. Also, did you do extension debugging in a Codespace?

No changes outside of the repro repo... and yes extension debugging through a Codespace. Just tried again and was able to repro.
Let me know how I can help!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

Closing as I was never able to repro.


Water material: Fix support for mesh instances

See https://forum.babylonjs.com/t/water-material-fix/41609/5
It also fixes use of bumpSuperimpose, fresnelSeparate and bumpAffectsReflection, which were not reset when switching their state from true to false.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13968/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13968/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13968/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13968/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13968/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13968/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13968/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13968/merge#BCU1XR#0


[Bug]: Missing utility to perform import and export of data from files.

Name and Version
rapidfort/mysql-official:latest
Which runtime are you using to reproduce this issue?

[ ] Kubernetes
[ ] Docker Compose
[X] Docker

Is this issue reproducible on the original source image?
Reproducible
Could you please identify the category? Details in TROUBLE_SHOOTING.md
Coverage missing
What steps will reproduce the bug?

Create a database(assume "DB") and tables(assume "tbl") inside it.
Inside MySQL CLI execute SHOW VARIABLES LIKE "secure_file_priv"  to get the path at which storage access is granted.
The path(result of previous execution) will be NULL. So, import and export of data will not happen and the following command to export data to csv will throw error.

USE DB;

TABLE tbl
INTO OUTFILE '/path/shown/by/secure_file_priv/backup.csv'
FIELDS TERMINATED BY ',';

Are you using any custom parameters or values?
No response
What is the expected behavior?
Data inside table must be written in csv format to a file named backup.csv.
What do you see instead?
ERROR 1290 (HY000): The MySQL server is running with the --secure-file-priv option so it cannot execute this statement
Additional information
No response

Compared the wrong OS images. Rapidofort's hardened image was based on Debian. But the functionality comparison was done with Fedora based mysql image.


Update api.tsx

Just a small test to make sure I’m doing it right. This weekend I will contribute much more

amazing! thank you very much @brigolini


External flash mx25l2564 issue with stm32 Nucleo_h723zg

Hi All,
I am interfacing the external flash mx25l2564 with nucleo_h723zg in Zephyr version 3.2 , Have connected the external flash Qspi in MCU,
Since in stm32h723vgtx-pinctrl.dtsi file those pins are mapped to octospi pins I have added this in my dts file ,
&octospi1{
pinctrl-names = "default";
pinctrl-0 = <&octospim_p1_clk_pb2 &octospim_p1_ncs_pb6
&octospim_p1_io0_pc9 &octospim_p1_io1_pc10
&octospim_p1_io2_pe2 &octospim_p1_io3_pd13>;
status = "okay";
mx25l2564: mx25l2564@0 {
    compatible = "st,stm32-ospi-nor";
    reg = <0>;
    ospi-max-frequency = <8000000>;
    size = <DT_SIZE_M(256)>; /* 32 MBytes */
	data-rate = <OSPI_STR_TRANSFER>;
    status = "okay";
    spi-bus-width = <4>;
    writeoc = "PP_1_4_4";
	sfdp-bfp = [
		53 46 44 50 06 01 02 ff
		00 06 01 10 30 00 00 ff
		c2 00 01 04 10 01 00 ff
		84 00 01 02 c0 00 00 ff
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		e5 20 fb ff ff ff ff 1f
		44 eb 08 6b 08 3b 04 bb
		fe ff ff ff ff ff 00 ff
		ff ff 44 eb 0c 20 0f 52
		10 d8 00 ff d6 49 c5 00
		81 df 04 e3 44 03 67 38
		30 b0 30 b0 f7 bd d5 5c
		4a 9e 29 ff f0 50 f9 85
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		00 00 00 00 00 00 00 00
		7f ef ff ff 21 5c dc ff
	];
	partitions {
		   compatible = "fixed-partitions";
		   #address-cells = <1>;
		   #size-cells = <1>;

		   partition@0 {
		       reg = <0x00000000 DT_SIZE_M(32)>;
		   };
	};
};

};
and also have added these below lines in stm32h723.dtsi file.
clock-names = "ospix", "ospi-ker", "ospi-mgr";
            clocks = <&rcc STM32_CLOCK_BUS_AHB3 0x00004000>,
                <&rcc STM32_SRC_PLL1_Q OSPI_SEL(1)>,
                <&rcc STM32_CLOCK_BUS_AHB3 0x00200000>;

Getting error in running flash_shell sample writing and erasing to the flash.
Below is the output which i am getting :
uart:~ *** Booting Zephyr OS build   ***
Found flash controller mx25l2564@0.
Flash I/O commands can be run.
[00:00:00.100,000]  flash_stm32_ospi: OSPI flash config is SPI|DUAL|QUAD / STR
uart:~
uart:~$ flash erase mx25l2564@0 0 1000
Erase Failed, code -16.
[00:00:24.599,000]  flash_stm32_ospi: OSPI AutoPoll failed
[00:00:24.599,000]  flash_stm32_ospi: Erase failed : flash busy

Environment : nucleo_h723zg board with QSPI  mx25l2564 NOR flash  connected on the

Converting to discussion until it is confirmed that issue is a genuine driver bug (potential dts config issue)


site specific announcements can also be locale specific

The Canada site wants different announcements if the locale is French or English. Currently, if the Site is set, it fetches announcements of all locales for that site. This change is meant to only fetch announcements with a specific locale for a site if announcements have both site and locale set.

@kueda can you check this out. It works, but not sure its the most elegant way to grab announcements with a given site param with locale = nil or locale = current locale


TextBoxComponent cannot wrap correctly when using Chinese

Current bug behaviour
TextBoxComponent cannot wrap correctly when using Chinese
Expected behaviour
Can wrap correctly when using Chinese
More environment information

Flame version: 1.2.0
Platform affected: windows

Please provide more details here. Like, what are some of the examples of the Chinese text, and how it's supposed to wrap, and how does this compare to the current behavior.
Sorry, but my knowledge of Chinese is somewhere around 0, which is probably true for most other Flame developers as well.

Please provide more details here. Like, what are some of the examples of the Chinese text, and how it's supposed to wrap, and how does this compare to the current behavior.
Sorry, but my knowledge of Chinese is somewhere around 0, which is probably true for most other Flame developers as well.

English

Chinese

text: 炉石传说新版本的战士橙卡公布了，卡牌原型是魔兽世界中德纳修斯大帝的武器，人气非常高的病娇剑娘蕾茉妮娅。
I think it is caused by no spaces between Chinese characters.

Ok, so here's the Unicode line-breaking algorithm, for future reference: https://unicode.org/reports/tr14/#Algorithm

Ran into the same isssue, any workaround here?

And I think it's not only for Chinese, but also Korean and Japanese and most East Asian languages which is not seprated by SPACE between two words.
So for the delevelopers who's target thoese markets it should be a problem here.

Have to add the SPACE into the sentence manually, doable for TextBoxComponent with fixed size.
Pretty sloppy, need fix.

Agree that it should be fixed, if anyone wants to have a look at how to implement the unicode line breaking algorithm we're open for PRs, like always. :)

We have run into the same problem.
Flutter widgets always handle wrapping properly. So we use overlays to avoid the problem temporarily.

I have found another workaround. Call toTextPainter on your flame TextPaint class to get the flutter TextPainter class and then call layout method on the instance to get the same layout as  flutter text widget.
_yourTextComponent.text = "[your CJK text]";
final tp = yourTextPaint.toTextPainter(_yourTextComponent.text);
tp.layout(maxWidth: 200);

Obviously, you will also need to set text box size manually.
Maybe we can fix the updateBounds method on text box component to use layout instead of manually compute the bound.

Maybe we can fix the updateBounds method on text box component to use layout instead of manually compute the bound.

As long as it is not a lot more computationally expensive to do that it sounds like a good solution, do you want to work on a PR for that?

Maybe we can fix the updateBounds method on text box component to use layout instead of manually compute the bound.

As long as it is not a lot more computationally expensive to do that it sounds like a good solution, do you want to work on a PR for that?

I looked into the codebase, and found out that TextRenderer is blocking me from making changes.
If the TextRenderer used by TextBoxComponent is guaranteed to be a TextPaint, then everything is easy. I can just delete most of the code in updateBounds and _recomputeSize, and have TextPainter.layout method to do the calculation.
However, TextRenderer also have another implementations, DebugTextRenderer and SpriteFontRenderer.
Any ideas on how to make those class auto compute layout or how to make the TextRenderer easier to work with?

Hi! Was this fixed or remains open? I would like to contribute :)

Hi! Was this fixed or remains open? I would like to contribute :)

It is still open indeed, I'll assign you to the issue! :)

I actually think we should be reusing much more of flutter layouting, when working on our text rendering pipeline I noticed we are basically reinventing the wheel of a lot of flutter stuff that is completely widget agnostic


on session destroy add a callback function

I want to trigger a event as soon a session is destroyed, something like this:
var store = new MongoStore()

store.on('destroy', function (sess) {
  // react to your store destroying a session
})

app.use(expressSession({ store: store }))

Is this supported?

Not currently supported, but PRs are welcome.

the event should be triggered where you do req.session.destroy() rather


Update hibernate-validator to 7.0.1.Final

Updates org.hibernate:hibernate-validator from 6.2.0.Final to 7.0.1.Final.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.hibernate", artifactId = "hibernate-validator" } ]

labels: library-update

@kelnos I'm struggling with this one -- Even just attempting to isolate the dependencies themselves, the construction of
Validation
    .byProvider[HibernateValidatorConfiguration, HibernateValidator](classOf[HibernateValidator])

now fails to compile, but manually tracing out the various parameters suggests that this should be a valid construction.
[error] .../guardrail/modules/sample-akkaHttpJackson/src/test/scala/core/TestImplicits.scala:50:16: type arguments [org.hibernate.validator.HibernateValidatorConfiguration,org.hibernate.validator.HibernateValidator] do not conform to method byProvider's type parameter bounds [T <: javax.validation.Configuration[T],U <: javax.validation.spi.ValidationProvider[T]]
[error]     .byProvider[HibernateValidatorConfiguration, HibernateValidator](classOf[HibernateValidator])
[error]                ^
[error] one error found

Even attempting to just copy/paste the exact construction from hibernate-validator tests in Java...
package core;

import javax.validation.Validation;
import javax.validation.Validator;
import org.hibernate.validator.HibernateValidator;

class TestImplicitsSupport {
    static Validator buildValidator() {
        return Validation.byProvider(HibernateValidator.class)
            .configure()
            .buildValidatorFactory()
            .getValidator();
    }
}

yields
[error] .../guardrail/modules/sample-akkaHttpJackson/src/test/java/core/TestImplicitsSupport.java:9:1: method byProvider in class javax.validation.Validation cannot be applied to given types;
[error]   required: java.lang.Class<U>
[error]   found: java.lang.Class<org.hibernate.validator.HibernateValidator>
[error]   reason: inference variable U has incompatible bounds
[error]     equality constraints: org.hibernate.validator.HibernateValidator
[error]     upper bounds: javax.validation.spi.ValidationProvider<T>
[error] Validation.byProvider

I don't know how to proceed here, the underlying issue seems to be due to the different ways that Java and Scala handle existential types, admittedly one of my weak points.

I would not proceed, as this version isn't compatible with DW1.x :-P

Fortunately, merging is blocked. Gonna close this to reduce noise.


Reverse playedBefore check. New players will never receive the keys.

I noticed the check for new player keys seems wrong.
I fixed it with what I think will fix the issue

Yeah I'm sorry. The hasPlayedBefore is only to check if this player is actuallt known on the server and not to check if he is a new player. So I think you can disregard this PR


Update your devcontainer environment

As a developer
I need my devcontainer environment to download and install IBM Cloud CLI
So that they dont have to repeat the effort manually everytime
Details and Assumptions

we are using IBM Cloud and have setup Python Cloud Flask

Acceptance Criteria

devcontainer automatically downloads and installs IBM Cloud CLI

Given I setup devcontainer environment
When I am working on IBM Cloud
Then IBM Cloud CLI is automatically downloaded and installed

@JShimota you did this already I think when you updated the manifest and the procfile etc


Allow support for lists of enums via many=True kwarg

As currently implemented, when defining an EnumField(MyCoolEnum, many=True), the many kwarg is not respected.
This PR allows for the EnumField to represent  list of str enum values.

@justanr let me know if anything else is needed to get this PR merged, thank you!

Any reason to do this over using the List field?
http://marshmallow.readthedocs.io/en/latest/api_reference.html#marshmallow.fields.List

The issue I've encountered with this is that jsonschema generation tools, such as marshmallow-jsonschema do not play nicely with fields.List(EnumField(SomeEnum)), and end up rendering jsonschema like
{
  type: "array"
}

instead of something reasonable like

{
    "type": "array",
    "items": {
        "type": "string",
        "enum": ["one", "two", "three"]
    }
}

or
"my_enum": {
    "oneOf": [
        {
            "type": "string",
            "enum": [
                  "A",
                  "B",
                  "C"
               ]
           },
           {
                "type": "null"
           }
        }
    ]
}

Seems like an issue that should be brought up against that project rather than making concessions here. Alternatively, an extension package that maps this and json schema closer.


Observable secondary Dockerfile

- What I did
Added Dockerfile to create an observable secondary.
- How I did it

Single stage build based on our usual build image with the Dart SDK
No need to compile an AOT binary
Fixes permissions so that non root user can run dart and access root .pub-cache
Entrypoint is running:

/usr/lib/dart/bin/dart run --observe=8181/<IP_ADDRESS> /app/at_secondary_server/bin/main.dart
NB binding to <IP_ADDRESS> so that Docker compose can map ports into it
- How to verify it
Build the image:
sudo docker build . -f Dockerfile.observe -t atsigncompany/secondary_obs

Modify a dess docker-compose.yaml like this:
image: atsigncompany/secondary_obs
    networks:
      second: {}
    ports:
    - published: 6767
      target: 6767
    - published: 8181
      target: 8181

Then (re)create a service from that docker-compose.yaml
The start of the service logs will contain the authentication URL for the observatory.
- Description for the changelog
Observable secondary Dockerfile

Taking a look today thanks Chris


Remove ADAL Code

Removed all references to ADAL code, as MSAL will be the only option going forward.
Still working on the testing but so far basic functionality works fine

Marking as draft while I fix the build & tests

Couple of additional changes needed:

SimpleTokenCache was ADAL specific only? Why does it continue to exist?
Why is accountTreeNode.test.ts still testing ADAL? I would expect all ADAL instances should be cleared.
Please review codeblocks in ADS Core that mention comment in connectionWidget and connectionManagementService:For backwards compatibility with ADAL, we need to check if the account ID matches with tenant Id or just the account ID
The OR case can be removed once we no longer support ADAL

Remove ADAL handling from findAccountIndex in accountManagementService.
Remove code blocks from other extensions that read the auth library configuration as it should no longer be supported, e.g. getAzureAuthenticationLibraryConfig() in import extension.

Overall, please do a thorough search in all files in entire repository and clear off anything that is ADAL specific, including comments, localized strings, constants, and any existence of authenticationLibrary configuration.


TRUNCATE_TARGET_PARTITION

That command is deprecated, on SQLDW Gen2 the correct sintax is TRUNCATE_TARGET.

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

ID: 43f9b7ed-90e8-b928-edc5-166ef12582e7
Version Independent ID: e64dbaaa-7066-be71-569f-1467d142eb94
Content: Azure SQL Data Warehouse Release Notes April 2018
Content Source: articles/sql-data-warehouse/release-notes-april-2018.md
Service: sql-data-warehouse
GitHub Login: @anumjs
Microsoft Alias: anjangsh

@brunofeldman  Thanks for bringing this to our attention. Your feedback  has been shared with the content owner for further review.

@XiaoyuMSFT could you please have a look at this

@brunofeldman  The Release Notes April 2018 article has been deprecated and removed from the doc set. The article Alter Table has the correct syntax.  I will close this issue. The @ notification mechanism continues to work even after an item is closed, in case further communication is needed. Thank you for submitting this issue. We hope to hear from you again.

#please-close


Use go-cmp for comparison in integration_test

Currently integration_test is using reflect.DeepEqual() to check equivalency.
There is at lease one case that we know of where DeepEqual() is not working as expected and is when float values have NaN. And there might be other cases too.
For now we have a customized cmpOption for FloatType to cover the above case, and other types are using the regular cmp.Equal(), except for DecimalType and IntType.
In order to have cmp.Equal() working for DecimalType, we might need to explicitly track precision.
As for IntType more investigation is needed, but I probably the issues is with 0 and null.Int cases

Closing -- Not ready yet, it was supposed to be a PR into Armanbqt/ion-go fork


subctl verify doesn't handle multiple kubeconfig files

What happened:
Running subctl verify --help one can see --kubeconfig string             absolute path(s) to the kubeconfig file(s), however running with several files produces an error:
[...]$ subctl verify --kubeconfig /path/to/mkolesni-0.12-testday1/auth/kubeconfig,/path/to/mkolesni-0.12-testday2/auth/kubeconfig --kubecontexts mkolesni-0.12-testday1,mkolesni-0.12-testday2
? You have specified disruptive verifications (gateway-failover). Are you sure you want to run them? No
Performing the following verifications: connectivity, service-discovery
Running Suite: Submariner E2E suite
===================================
Random Seed:<PHONE_NUMBER>
Will run 37 of 41 specs

Non-recoverable error: context "mkolesni-0.12-testday1" does not exist[...]$

What you expected to happen:
It should work given a list of files, or the help text should be updated
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Diagnose information (use subctl diagnose all):
Gather information (use subctl gather)
Cloud provider or hardware configuration:
Install tools:
Others:

The separator for kubeconfigs is “:”, not “,”, same as for the KUBECONFIG variable. The documentation should indicate that...
$ subctl verify --kubeconfig awsocp1/auth/kubeconfig:awsocp2/auth/kubeconfig --kubecontexts admin-1,admin-2

The documentation should indeed but from a UX perspective we should be consistent with kubecontexts flag and convert commas to colons internally.

Or the other way round, so users can specify either colons or commas.

BTW when using colon separator theres a different issue where it can only access one cluster.

It works for me, the command above is what I used for my tests...

It works for me, the command above is what I used for my tests...

IDK when I run with the colon I get an error "unauthorized" but when running with two configs in arguments it works.
The error:
[mkolesni@mkolaptop openshift]$ subctl verify --kubeconfig /path/to/mkolesni-0.12-testday1/auth/kubeconfig:/path/to/mkolesni-0.12-testday2/auth/kubeconfig --kubecontexts mkolesni-0.12-testday1,mkolesni-0.12-testday2
? You have specified disruptive verifications (gateway-failover). Are you sure you want to run them? No
Performing the following verifications: connectivity, service-discovery
Running Suite: Submariner E2E suite
===================================
Random Seed:<PHONE_NUMBER>
Will run 37 of 41 specs

STEP: Creating kubernetes clients
Failure [0.457 seconds]
[BeforeSuite] BeforeSuite
github.com/submariner-io/shipyard@v0.12.0-rc1/test/e2e/e2e.go:42

  Failed to list nodes
  Unexpected error:
      <*errors.StatusError | 0xc0005408c0>: {
          ErrStatus: {
              TypeMeta: {Kind: "", APIVersion: ""},
              ListMeta: {
                  SelfLink: "",
                  ResourceVersion: "",
                  Continue: "",
                  RemainingItemCount: nil,
              },
              Status: "Failure",
              Message: "Unauthorized",
              Reason: "Unauthorized",
              Details: nil,
              Code: 401,
          },
      }
      Unauthorized
  occurred

  github.com/submariner-io/shipyard@v0.12.0-rc1/test/e2e/framework/framework.go:513
------------------------------
Failure [0.000 seconds]
[AfterSuite] AfterSuite
github.com/submariner-io/shipyard@v0.12.0-rc1/test/e2e/e2e.go:55

  Unexpected error:
      <*errors.StatusError | 0xc00080a460>: {
          ErrStatus: {
              TypeMeta: {Kind: "Status", APIVersion: "v1"},
              ListMeta: {
                  SelfLink: "",
                  ResourceVersion: "",
                  Continue: "",
                  RemainingItemCount: nil,
              },
              Status: "Failure",
              Message: "Unauthorized",
              Reason: "Unauthorized",
              Details: nil,
              Code: 401,
          },
      }
      Unauthorized
  occurred

  github.com/submariner-io/shipyard@v0.12.0-rc1/test/e2e/framework/gateways.go:159
------------------------------

Ran 37 of 0 Specs in 0.604 seconds
FAIL! -- 0 Passed | 37 Failed | 0 Pending | 0 Skipped
[] E2E failed

subctl version: v0.12.0-rc1

Are your kubeconfigs valid for combined use? You need to check that their contents have unique names (in particular, the user). I use
yq -i '.contexts[0].context.user = "admin-'$c'" | .contexts[0].name = "admin-'$c'" | .users[0].name = "admin-'$c'" | .current-context = "admin-'$c'"' awsocp${c}/auth/kubeconfig

where c is set to the appropriate value (h, 1 or 2 in my setup).

That must be it, but this isn't documented entirely on the AWS guide - I updated the context name and current context as it detailed, but didn't touch the user name as it wasn't specified.
I opened a bug to fix this doc - https://github.com/submariner-io/submariner-website/issues/707


aves libre sometimes access geomobileservice-pa.googleapis.com

Fore someone this is a bug, for most this this is a feature request.
When displaying the Infobox for a picture or a detailed view in the map with press on the picture the real address is calculated from the coordinates using the service of "geomobileservice-pa.googleapsis.com". This interfere with a google-free usage.
After blocking the access to "geomobileservice-pa.googleapsis.com" "aves libre" works fine and perfect - only a near address is displayed.
Two Proposal:
For OpenStreetMap maps please use an other method for calculation the address from the coordinates (reverse geocoding) like "Nominatim" but which is not for heavy users but an entry point for related services.
Alternative make a switch in the section privacy to control the call of the google function. The price for a google-free usage is displaying only the near address.
But please don't drop the displaying of the real address.

Aves Libre does not reference proprietary code from Google (or from anybody else), but Aves uses the standard Geocoder from Android. Apparently, on your device, you have the Google Mobile Services installed and they take care of responding to the Geocoder API calls.
The way your device responds to this Geocoder call is out of scope of Aves.

I hope you will change the decision and you will implement other way to translate gps coordinate to address.
It sad, but when i started to use aves, i receive spam mails like 'buy flat/house etc in [city, where i was]'.
It's always, when i  back from travel.
Ps. Your app is the best 🙂


[Bug] First run broken

Describe the bug
On clean install and first run tunnel.json does not have TunnelName property.
Maybe the new version of cloudflared creates the file diferently?
After copy pasting tunnel.json from the logs, adding "TunnelName":"homeassistant" and restarting it works fine.
To Reproduce
Clean install, config and enable debug logging, first run.
Authorized correctly, then error:
DEBUG: Tunnnel name read from file: null
WARNING: Tunnel name in file does not match config, removing tunnel file
and exit.

Logs
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 00-banner.sh: executing...
-----------------------------------------------------------
 Add-on: Cloudflared
 Use a Cloudflared tunnel (formerly Argo Tunnel) to remotely connect to Home Assistant without opening any ports
-----------------------------------------------------------
 Add-on version: 0.5.2
 You are running the latest version of this add-on.
 System: Home Assistant OS 7.4  (amd64 / qemux86-64)
 Home Assistant Core: 2022.3.1
 Home Assistant Supervisor: 2022.01.1
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 00-cloudflared-log.sh: executing...
[cont-init.d] 00-cloudflared-log.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
Log level is set to DEBUG
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] 10-cloudflared-config.sh: executing...
[18:01:02] INFO: Checking Add-on config...
[18:01:03] INFO: Checking for existing certificate...
[18:01:03] NOTICE: No certificate found
[18:01:03] INFO: Creating new certificate...
[18:01:03] NOTICE:
[18:01:03] NOTICE: Please follow the Cloudflare Auth-Steps:
[18:01:03] NOTICE:
Please open the following URL and log in with your Cloudflare account:
https://dash.cloudflare.com/argotunnel?callback=https%3A%2F%2Flogin.cloudflareaccess.org%2F63dYTIhGuV#######redacted#######
Leave cloudflared running to download the cert automatically.
You have successfully logged in.
If you wish to copy your credentials to a server, they have been saved to:
/root/.cloudflared/cert.pem
[18:01:45] INFO: Authentication successfull, moving auth file to the '/data' folder
[18:01:45] INFO: Checking for existing certificate...
[18:01:45] INFO: Existing certificate found
[18:01:45] INFO: Checking for existing tunnel...
[18:01:45] NOTICE: No tunnel file found
[18:01:45] INFO: Creating new tunnel...
Tunnel credentials written to /data/tunnel.json. Keep this file secret. To revoke these credentials, delete the tunnel.
Created tunnel homeassistant with id #######redacted#######-b94a-b74e0eb05301
Tunnel Token: eyJhIjoiOGQyOTQwYTUzODE#######redacted#######
[18:01:46] DEBUG: Created new tunnel: {"AccountTag":"#######redacted#######8e5e1815","TunnelSecret":"#######redacted#######1DTiKc6Eacqw63A=","TunnelID":"#######redacted#######-b94a-b74e0eb05301"}
[18:01:46] INFO: Checking for existing tunnel...
[18:01:47] INFO: Existing tunnel with ID #######redacted#######-b94a-b74e0eb05301 found
[18:01:47] INFO: Checking if existing tunnel matches name given in config
[18:01:47] DEBUG: Tunnnel name read from file: null
[18:01:47] WARNING: Tunnel name in file does not match config, removing tunnel file
[18:01:47] FATAL: Failed to create tunnel

Thanks @miguelangel-nubla for pointing this out. Seems to be an issue related to the latest cloudflared version. Should be fixed with #64

@elcajon any idea when you guys will be able to push the fix?

@darrylgibbs they are actively working on it. And usually @elcajon and @brenner-tobias are quite fast.

@felipecrs apologies if that came across rude, I'm just really keen to try this thing..ha ha.  I discovered it after hours of searching for the solution when I had the same issue.
Thanks for the amazing work all! @felipecrs @elcajon

PR is ready, just needs to be re-approved. 😉

Should be fixed in v0.5.3, please re-open if this does not help. Thanks a lot @elcajon


Accept named parameters after positional arguments #219

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

@voidint The named arguments will be ignored. Can you add a few test cases to verify that?

Sorry, it really doesn't work.

Since this patch does not work, we'd like to close it, so other people can work on it.


Cannot serialize function that accesses namedtuple

Description
Repro:
from collections import namedtuple
import jax.numpy as jnp
from jax.experimental.export import export, serialize, deserialize

TestTuple = namedtuple('TestTuple', ['a', 'b'])

def test_fn(x):
    return x.a

exp = export(test_fn)(TestTuple(jnp.array([1]), jnp.array([2])))
serialized = serialize(exp, vjp_order=0)
deserialized = deserialize(serialized)

Traceback:
Traceback (most recent call last):
  File "/Users/dion/codes/jax/repro.py", line 11, in <module>
    serialized = serialize(exp, vjp_order=0)
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 59, in serialize
    exported = _serialize_exported(builder, exp, vjp_order)
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 75, in _serialize_exported
    in_tree = _serialize_pytreedef(builder, exp.in_tree)
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 244, in _serialize_pytreedef
    children_vector_offset = _serialize_array(
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 144, in _serialize_array
    element_offsets = [serialize_one(builder, e) for e in elements]
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 144, in <listcomp>
    element_offsets = [serialize_one(builder, e) for e in elements]
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 244, in _serialize_pytreedef
    children_vector_offset = _serialize_array(
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 144, in _serialize_array
    element_offsets = [serialize_one(builder, e) for e in elements]
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 144, in <listcomp>
    element_offsets = [serialize_one(builder, e) for e in elements]
  File "/Users/dion/codes/jax/jax/experimental/export/_serialization.py", line 263, in _serialize_pytreedef
    raise NotImplementedError(f"serializing PyTreeDef {node_data}")
NotImplementedError: serializing PyTreeDef (<class '__main__.TestTuple'>, None)

Since namedtuples are natively recognized as pytree containers by JAX this should probably be in scope?
System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.24
jaxlib: 0.4.24
numpy:  1.26.1
python: 3.10.13 (main, Aug 24 2023, 12:59:26) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
jax.devices (1 total, 1 local): [CpuDevice(id=0)]
process_count: 1

It is true that JAX's pytrees support namedtuple natively, but it is not obvious that namedtuple, or any custom pytree node type, should be serializable directly. Deserialization may happen in a different Python process (or even one that is not Python ) and the particular namedtuple class may not be available there.
I'll think some more about it, but for now I recommend wrapping the function to be serialized such that it takes a dictionary argument.


cpu-usage: cpu usage not working with nagios user.

This issue respects the following points:

[X] This is a bug, not a question or a setup/configuration issue.
[X] This issue is not already reported on Github (I've searched it).
[X] I use the latest release of the Monitoring Plugins (https://github.com/Linuxfabrik/monitoring-plugins/releases).
[X] I agree to follow Monitoring Plugins's Code of Conduct.

Bug description
When excute :
/usr/lib64/nagios/plugins/cpu-usage --count 5 --critical 90 --warning 80 with "nagios" user on Icinga2 server. I can use this plugin with other user.
Steps to reproduce - Plugin call
/usr/lib64/nagios/plugins/cpu-usage --count 5 --critical 90 --warning 80
with "nagios" user on Icinga2 master

Steps to reproduce - Plugin Version
cpu-usage: v2022071801 by Linuxfabrik GmbH, Zurich/Switzerland
Steps to reproduce - Data
Query failed: INSERT INTO "perfdata" (cpu_usage,guest,guest_nice,iowait,irq,nice,softirq,steal,system,user) VALUES (:cpu_usage,:guest,:guest_nice,:iowait,:irq,:nice,:softirq,:steal,:system,:user);, Error: attempt to write a readonly database, Data: {'cpu_usage': 0.2, 'guest': 0.0, 'guest_nice': 0.0, 'iowait': 0.0, 'irq': 0.0, 'nice': 0.0, 'softirq': 0.0, 'steal': 0.0, 'system': 0.1, 'user': 0.1}
Python version
Python 3
Environment
Ubuntu 22.04
Linux  5.15.0-48-generic #54-Ubuntu SMP Fri Aug 26 13:26:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Application
Icinga2
Build information:
Compiler: GNU 11.2.0
Build host: runner-hh8q3bz2-project-298-concurrent-0
OpenSSL version: OpenSSL 3.0.2 15 Mar 2022
Python modules
Package                Version

antiorm                1.2.1
attrs                  21.2.0
Automat                20.2.0
Babel                  2.8.0
bcrypt                 3.2.0
blinker                1.4
certifi                2020.6.20
chardet                4.0.0
click                  8.0.3
cloud-init             22.2
colorama               0.4.4
command-not-found      0.3
configobj              5.0.6
constantly             15.1.0
cryptography           3.4.8
db                     0.1.1
dbus-python            1.2.18
distro                 1.7.0
distro-info            1.1build1
gpg                    1.16.0-unknown
httplib2               0.20.2
hyperlink              21.0.0
idna                   3.3
importlib-metadata     4.6.4
incremental            21.3.0
jeepney                0.7.1
Jinja2                 3.0.3
jsonpatch              1.32
jsonpointer            2.0
jsonschema             3.2.0
keyring                23.5.0
launchpadlib           1.10.16
lazr.restfulclient     0.14.4
lazr.uri               1.0.6
MarkupSafe             2.0.1
more-itertools         8.10.0
netifaces              0.11.0
nftables               0.1
oauthlib               3.2.0
pexpect                4.8.0
pip                    22.0.2
ptyprocess             0.7.0
pyasn1                 0.4.8
pyasn1-modules         0.2.1
PyGObject              3.42.1
PyHamcrest             2.0.2
PyJWT                  2.3.0
pyOpenSSL              21.0.0
pyparsing              2.4.7
pyrsistent             0.18.1
pyserial               3.5
python-apt             2.3.0+ubuntu2.1
python-debian          0.1.43ubuntu1
pytz                   2022.1
PyYAML                 5.4.1
requests               2.25.1
SecretStorage          3.3.1
service-identity       18.1.0
setuptools             59.6.0
six                    1.16.0
sos                    4.3
ssh-import-id          5.11
systemd-python         234
Twisted                22.1.0
ubuntu-advantage-tools 27.10
ubuntu-drivers-common  0.0.0
ufw                    0.36.1
unattended-upgrades    0.1
urllib3                1.26.5
wadllib                1.3.6
wheel                  0.37.1
xkit                   0.0.0
zipp                   1.0.0
zope.interface         5.4.0
Additional info
No response

Hi, the problem is probably that the sqlite database for caching values was created by a different user (for example while running the plugin as root). Now the nagios user has no permissions to write to it. You can try removing it:
rm -f /tmp/linuxfabrik-plugin-cache.db
rm -f /tmp/cpu-usage.db

Hi, the problem is probably that the sqlite database for caching values was created by a different user (for example while running the plugin as root). Now the nagios user has no permissions to write to it. You can try removing it (whatever you'll find on your system, one of those):
rm -f /tmp/linuxfabrik-monitoring-plugins-cpu-usage.db
rm -f /tmp/linuxfabrik-plugin-cache.db
rm -f /tmp/cpu-usage.db

Hello.
You are right. I followed your guide and it worked. I feel strange why only this plugin have .db files.

Only the plugins have db-files that need data for comparisons from the past.

Thank you much.


website certificate expired and is not reachable

Describe the bug
Your let's encrypt certificate expired today. 05.05.2021 14:00
Although I currently also get too many redirects error in browser.
How to Reproduce
Just open website
Expected behavior
certificate is updated and website can be used again.

Thanks for the issue @holomekc, we are aware and it is being worked on.

fixed 😅

We're having the same issue again. Investigating, thanks for your patience 🙏


Improve Spice REPL show tables & row limit

Improves the Spice REPL to rewrite the query for show tables to just list the tables in the public schema.
Also limits the output to the terminal to 500 lines.
Also fixes a bug with spice init creating an empty dir.
Closes #952
Closes #874
Closes #839

Also to close https://github.com/spiceai/spiceai/issues/839 properly, based on my test, the query needs to have about ~120M decoding.
The batch we are getting from datafusion stream is 8K, so the size of the records need to smaller than that.
Unless people using a small limit or filter the selected columns; it's hard to get this fix properly.


Update defaults with Create and Enable SCM_DO_BUILD... by default with Deploy

Fixes #1357

I believe we discussed that we only want to enable the setting on web app creation rather than on every deploy.
If users already have a working project, we don't want to risk breaking their existing projects by changing an app setting when they deploy.


encoding/json: float output change breaks BigQuery API

This isn't a bug in Go, but it does cause breakage elsewhere. I'm not sure what the fix should be (if any).
What version of Go are you using (go version)?
1.7.5, 1.8
What operating system and processor architecture are you using (go env)?
Linux amd64
What did you do?
Ran this program:
package main

import (
    "encoding/json"
    "fmt"
    "log"
)

func main() {
    b, err := json.Marshal(5e20)
    if err != nil {
        log.Fatal(err)
    }
    fmt.Println(string(b))
}

In 1.7.5, the output is 5e+20.
In 1.8, the output is<PHONE_NUMBER>00000000000.
Both are legal JSON, but the latter breaks BigQuery:
InsertAll googleapi: Error 400: Invalid JSON payload received. Unable to parse number.
[{"json":{"MyFloat":50000000000000000000
                    ^, badRequest

Seems like a BigQuery bug. That is how Chrome formats it too. We explicitly switched formatting so we'd behave like Node.js & Chrome etc.
In Chrome:


[Common] max memories in nns-edge

nnstreamer now supports max 256 tensors in the buffer.
Increase max memories in nnstreamer-edge handle.
TODO:
We will fix the util functions in edge elements of nnstreamer, to handle memories in gst-buffer.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #194. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

@myungjoo I updated commit message.
(nnstreamer now supports 256 memories in the buffer, and I will fix util functions in edge elements of nnstreamer.)


Show nothing

I have install the latest chatgpt-advanced-3.2.7 version. When i typeing all the info and press enter it will show nothing. However i remove this WebchatGPT the chatGPT will be working as normal. When i install back again WebchatGPT it will not function again.
not function means "When i typeing all the info and press enter it will show nothing."

Same here. It seems the problem correlates with latest Chrome version 114.0.5735.90

same!!!

same issue!

Same issue with chrome and brave browsers

same for me on chrome and mozilla

Works now.
just updated to chrome version 114.0.5735.91 by clicking on the 3 dots upper right corner of chrome  --> Help --> About Chrome" then it updates when showing the current version.
Also updated the WebChatGPT extension to version 3.3.4.
All good now!

I am using the Edge browser, but it doesn't seem to be working.

Yeah, doesn't work on firefox either

This issue is still occurring on brave :(

Time to move to another extension, too bad WebChatGPT has been acquired.


cannot find installed packages - integrate install, CentOS x64

I am having problems with vcpkg and cmake under CentOS to build a C++ static library project that has a dependancy on boost and cpprestsdk.  I'm targeting Unix Makefiles and I cannot appear to get include paths set in order for header files location to be resolved.
I thought that after an 'integrate install'  a cmake toolchain file is created that sets up everything that cmake needs in order to pass include and link paths into (in this case) a unix Makefile?
Thought I would start the diagnosis from the vcpkg side first.
$ vcpkg install boost cpprestsdk openssl
$ vcpkg integrate install
Applied user-wide integration for this vcpkg root.
CMake projects should use: "-DCMAKE_TOOLCHAIN_FILE=/vcpkg/scripts/buildsystems/vcpkg.cmake"

$ cmake -DCMAKE_TOOLCHAIN_FILE=/vcpkg/scripts/buildsystems/vcpkg.cmake -G "Unix Makefiles"
[INFO] VCPKG CMAKE_TOOLCHAIN_FILE = /vcpkg/scripts/buildsystems/vcpkg.cmake
-- Configuring done
-- Generating done
-- Build files have been written to: /Devel/Domain

$ make
 [INFO] VCPKG CMAKE_TOOLCHAIN_FILE = /vcpkg/scripts/buildsystems/vcpkg.cmake
-- Configuring done
-- Generating done
-- Build files have been written to: /Devel/Domain
[  7%] Building CXX object CMakeFiles/Domain.dir/BaseDataFactory.cpp.o
In file included from /Devel/Domain/stdafx.h:4:0,
                 from /Devel/Domain/BaseDataFactory.cpp:1:
../Common/Common.h:75:26: fatal error: cpprest/json.h: No such file or directory
 #include <cpprest/json.h>

Here's my CMakeLists.txt file
cmake_minimum_required(VERSION 2.8.9)

if(DEFINED ENV{VCPKG_ROOT} AND NOT DEFINED CMAKE_TOOLCHAIN_FILE)
  set(CMAKE_TOOLCHAIN_FILE "$ENV{VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake"  CACHE STRING "")
endif()

message(" [INFO] VCPKG CMAKE_TOOLCHAIN_FILE = ${CMAKE_TOOLCHAIN_FILE}")

set(CMAKE_CXX_STANDARD 11)
# Side note: WHY DO WE NEED TO SPECIFY cpprestsdk DIR EXPLICITLY?
set(cpprestsdk_DIR "/vcpkg/installed/x64-linux/share/cpprestsdk")

find_package(OpenSSL REQUIRED)

find_package(cpprestsdk CONFIG REQUIRED)
add_compile_options(-std=c++11 -I../ -m64)
file(GLOB SOURCES "*.cpp")
add_library(Domain STATIC ${SOURCES})
install(TARGETS Domain DESTINATION ../lib)

Comments?  Anyone?

This issue hasn’t been updated in a year, if it is still an issue, please reopen.


Can't build on EAS since the node version is locked on v18 but EAS is using v16.18.1

Description

https://github.com/jpudysz/react-native-unistyles/blob/af5256c3ad8539072a6c8728220da86eac5be4d8/package.json#L95
We really need v18 for this? :)
Steps to reproduce
eas build

Snack or a link to a repository (optional)
No response
Unistyles version
1.1.1
React Native version
0.72.6
Platforms
iOS

Hello Albert,
Node 16 has been deprecated since September 11. You can now use Node 18 in your EAS. Please follow this link: https://expo.dev/changelog/2023/11-03-node-default.
Enjoy your weekend! 🙌


How to generate an article in 4v?

Hello guys. I reaed documentation, but a command middleman article TITLE doesn't work(Could not find command "article".).
My gemfile:
source 'https://rubygems.org'
gem 'wdm', '~> 0.1.0', platforms: [:mswin, :mingw]
gem 'tzinfo-data', platforms: [:mswin, :mingw, :jruby]
gem "middleman", "~> 4.0.0"
gem "middleman-blog"
gem "middleman-livereload"
gem 'redcarpet', '~> 3.3', '>= 3.3.3'
gem "builder", "~> 3.0"
Thanks.

Yes, that command has been removed. I'd suggest creating the article text file manually. The command wasn't really doing that much. Will update the docs soon.

Mmmh,
@tdreyno I really like the helper and still find it useful and use it regularly.
Now one needs to ...

remember how a blog article is structured
add the title to the front matter yourself
convert the title to a valid file name
have a template for a new article somewhere else - quite annoying if you've got a complex layout for articles.

What was the reason to remove it? Can we extend/change it to make it more useful to you/other contributors/others?
Cheers,
Dennis

We could bring it back. I asked on Twitter and nobody admitted to using it.

Mmh... Ok... I "just" started to use twitter and most times I'm overwhelmed by the tweets found there. Maybe I just oversaw it. Wouldn't it be better to ask those things via Github and reference this issue in a tweet in the future?

I really like the helper and use it every time I create a new article. It's quite convinient to have it around.

I'm going to ask for feedback on gitter.im as well.

Using either date in path or front matter is v4 only correct? In v3 I get an exception:
== Request: /articles/killing-me-softly-1-4-cleanup-ruby-scripts-after-sigterm.html
       error  build/articles/killing-me-softly-1-4-cleanup-ruby-scripts-after-sigterm.html
       undefined method `date' for #<Middleman::Sitemap::Resource:0x0000000291d6e0>
       /home/user/data/projects/fedux_org-site/lib/fedux_org_site/articles/article.rb:123:in `method_missing'

I don't think so... seems easy to write a test for this. Will have to investigate.

Mmmh... See middleman/middleman-blog#272

im also missing this helper.
was very useful in creating new articles

I just started using Middleman and have been racking my brain trying to figure why this command wasn't working. Is there a place online that we can directly go to that will list deprecated items for future reference? Thanks

I'm not aware of such a list. I asked @tdreyno if he would reconsider removing this helper.

That list is here: https://middlemanapp.com/basics/upgrade-v4/
But as this is an extension (blog) not core, it isn't mentioned.
As @dg-ratiodata mentioned, I'll likely be adding it back soon. Didn't realize so many relied on it.

@tdreyno I loved that command. I don't follow you on twitter, so there's no way I would have seen that post, but from a workflow standpoint it's kind of vital to blogging using middleman. I've patched together an aText expansion that replaces the functionality, but I would also find it useful to be back in middleman-blog.
Also, if there are any documentation changes needed in response to putting it back, let me know. I'd love to find a way to help out the project now that I'm using it regularly.
Also, thanks for helping build a great product.

@trevortwining It'll definitely be back in v4.1.0 really soon now.
We'd love to have any help you're willing the give. Usually, it's best to scratch your own itch. Keeping track of the issues and features you care about and helping us solve and document them.

@tdreyno Thanks for taking it back!

@tdreyno :+1: for bringing it back!

Just tried Middleman 4.1.1 with middleman-blog which is still 4.0.0. And middleman article 'test article' leads still to Could not find command "article"

will show up in blog 4.1, whenever that goes out.

@tdreyno I am using middleman (4.1.2) and middleman-blog (4.0.0) but that command is still not working, and the documentation is still outdated. How can i create article? How's the format? are there any tutorial or post?
I am new to middleman, since 2 days ago
Peace :) Middleman is great staticgen

Blog 4.1, not core 4.1. Blog is still 4.0 for now

is there any other solution for now? at least an easy way for the readers.. or a link to another way to make articles? saves time... thanx

If you've got some Github Chops, recoving that code from v3 and making a PR would help. Otherwise, I'm the only person working on this and I'm in crunch time at the day job right now

@tdreyno I pulled the article command and what I think is its wiring (it's still mentioned in extension.rb:42), though I still get «Could not find command "article".» when trying to run it.
Code is here: https://github.com/dinomite/middleman-blog/tree/features/new-article-command

Wow what a real bummer, I just started using middleman from using octopress. With Octopress/Jekyll I always generated the boilerplate article.

@dinomite I think v4 requires the commands to be registered, rather than happening automatically. If you turn that commit into a PR, I'll take a look at getting it working.

Created: https://github.com/middleman/middleman-blog/pull/284
@tdreyno Thanks for being so quick!  I looked around and couldn't figure out how to do that registration—interested in seeing your PR for how it's done.

Any eta on this, even current documentation shows this as the command to use?

@farishkash https://twitter.com/middlemanapp/status/720288267648696322

Thank you,
Weird I had 4.0 by using gem "middleman-blog", I forced it to 4.0.1 by using gem 'middleman-blog', '~> 4.0.1'
Thanks again.

Bundler will lock down versions unless you run bundle update

@tdreyno Thanks for doing the work to get this back in!

Hi there...
I am in trouble , I read lots of review but can't fix it
here is what I stil have :(
Could not find "source/new-article.erb" in any of your source paths. Your current source paths are: /Users/nelly/.rvm/gems/ruby-2.3.0/gems/middleman-blog-4.0.1/lib/middleman-blog/commands

I upgraded to middleman-blog 4.0.1 but still get Could not find command "article".

I think @tdreyno meant blog-4.1.0 not blog-4.0.1. It's a new feature so the version number should be really a 4.x.0 not 4.0.x:
http://semver.org/:

MAJOR version when you make incompatible API changes,
MINOR version when you add functionality in a backwards-compatible manner, and
PATCH version when you make backwards-compatible bug fixes.

Ah ok. Unfortunately even with
gem "middleman", "~> 4.0.0"
gem 'middleman-blog', '~> 4.1.0'
gem "middleman-sprockets", "~> 4.0.0.rc"

and a bundle update I get:
Could not find gem 'middleman-blog (~> 4.1.0)' in any of the gem sources listed in your Gemfile or available on this machine

Changing
gem "middleman", "~> 4.0.0"
gem 'middleman-blog', '~> 4.1.0'

to
gem "middleman"
gem "middleman-blog"

worked for me.


access to root Vue instance

Version
4.0.2
Reproduction link
https://jsfiddle.net/ludanxer/gy9m72xf/1/
Steps to reproduce
router.beforeEach((to, from) => {
  console.log(router.app)
})

What is expected?
To be able to access root Vue instance via router.app. Or document the breaking change if it's expected.
What is actually happening?
router.app undefined

I add firebase as a global vue property for authentication, but I can't seem to access it inside router.

Vue Router can be installed in multiple Vue apps, so having app could be misleading while having an array of app seems overly complicated. However, this can be added very easily:
app.use(router)
router.app = app

A PR to add this to the migration guide is welcome!

Vue Router can be installed in multiple Vue apps, so having app could be misleading while having an array of app seems overly complicated. However, this can be added very easily:
app.use(router)
router.app = app

A PR to add this to the migration guide is welcome!


Naming suggestion: OphydDevice -> Device

It is currently ophyd.device.OphydDevice. I think ophyd.device.Device is sufficient. Only worth doing this if we do it before we begin to deploy.

Might even be worth using importing to flatten the name space so we get from ophyd import Device as OD

I'm OK with this. sed away, @danielballan.


customized wrapper class support

This commit  brings up the require autoloader code and makes some changes in the way to load a wrapper class ( fully qualified class name is acceptable), which allows developers to implement there own wrapper

The reason why I use an autoloader other than composer's, is you can use php artisan laravoole reload to gracefully reload the composer's autoloader. So the job must in worker start callback, which called prepareKernel. You may add wrapper_map in config/laravoole.php and read it.

I see, it make sence. 😄


[ARM] Foundation-Dev tests URL Session - resume doesn't seem to work

On the most recent master build - 20170228.1
Download, Cancel and then Resume – doesn’t resume the download, it stays in the cancelled state

@ashvarma the text field below the download table should be populated with details about the task when you cancel it and attempt to resume it. Could you share those details?

This is by design; the default URL in the Foundation URL Session test view controller gives a very, very small response (47 bytes). By the time the download is created, it's finished. If you cancel it before it starts, the headers specify that it cannot be resumed.

Also: we have automated functional tests https://github.com/Microsoft/WinObjC/blob/develop/tests/functionaltests/Tests/NSURLSession.mm#L656 for this scenario.  The test app itself was written for a demo.


Better fail message

Pull request
Please confirm that this pull request has done the following:

[x] Tests added
[x] Documentation added (where applicable)
[x] Example added (either to an existing notebook or as a new notebook, where applicable)
[x] Description in CHANGELOG.rst added

Adding to CHANGELOG.rst
Please add a single line in the changelog notes similar to one of the following:
- (`#XX <https://github.com/znicholls/silicone/pull/XX>`_) Added feature which does something
- (`#XX <https://github.com/znicholls/silicone/pull/XX>`_) Fixed bug identified in (`#YY <https://github.com/znicholls/silicone/issues/YY>`_)

We can do, although currently there are rather a lot of reasons why it may fail with a ValueError. This sort of warning will still need to be caught anyway. Do we want to ignore all ValueErrors from a single run of silicone in those cases? Quite possibly we do, I suppose.

Do we want to ignore all ValueErrors from a single run of silicone in those cases?

So I think given this PR is just about what happens if there's no data, let's think about that first. As a user I'd expect an error with the message you have.
Then when we're running Silicone elsewhere (e.g. AR6 stuff) we can put the error handling in there. However in this PR I think an error is still most appropriate. If we wanted to get really fancy, we could create our own ValueError sub-class, NoData, or something.

I've changed the error message, however I still get this weird tagging problem that means that half of the notepad checks fail. Am I just using a different package for something?

Am I just using a different package for something?

No the order of columns in dataframes isn't stable. Just put # NBVAL_IGNORE_OUTPUT at the top of each of the failing cells.

Thanks a lot @Rlamboll @znicholls  - this is useful - so we will use a try/except block - but at least the warning will indicate why the particular scenario fails.


MNT bump codecov uploader

Closes https://github.com/skops-dev/skops/pull/408

Finally green. Seems like we needed a new codecov token. Not sure what all those matplotlib issues are though, and they don't make the CI fail, and I can reproduce locally.

cc @BenjaminBossan

It's there in the 3.9 build in the test log. But we can ignore it it seems.


Copy of LICENSE in repo root

- Why?
Make the LICENSE visible at the top level of the repo, which will also appear in GitHub sidebar like:

- What I did
Copy of LICENSE from one of the projects into repo root
- How I did it
cp at_backupkey_flutter/LICENSE .
- Description for the changelog

Copy of LICENSE in repo root

@murali-shris I'm still waiting on your approval for this

Some of the pubspec.yaml files already had homepage and repository set thus:
homepage: https://atsign.dev
repository: https://github.com/atsign-foundation/at_widgets

This PR now makes them all consistent, which fixes #53 (it wasn't only at_events_flutter, there were a bunch of others pointing to the master branch).

@sarika01 these packages will need to be pushed to pub.dev for the docs changes to take effect. Anything that's going to be updated in the next week or so can be swept along with the pushes that will happen anyway. For anything else can you please do a version+1 push.


Experiment Question

How to compare different paper if their source code using different language(python, c) and different framework(Tensorflow, PyTorch, Caffee).
!! Better mentioned this in Weekly report

Natasa Reply:
Regarding some of your questions:
There are good sides of both having all the methods implemented in the same framework, and having each method implemented as described in the papers where they are first proposed.
A benefit from using the code supplied by the authors (in this case, to have the Caffe -version) is that you can make sure that you have the exact replication, you can check the results and make sure that you get the same as proposed (i.e., that you got it right) and can make further claims regarding what the original authors said.
A benefit from having all the methods in the same environment (in this case, to reimplement - based on the code/description in the a paper) is very clear when it comes to comparative studies: You do have the same conditions for all the compared methods on one same dataset (e.g., oral cancer), and the differences you may observe come from the methods, not the implementations or the particular environments.
In my opinion, considering that you observe difficulties with the first method, and its Caffe implementation, it would be by far best to have all the three methods running in PyTorch (do look at papers citing the Kanazawa paper if any later ones supply code that may be easier to work with). If I misunderstood that you actually did get the Caffe version running, and you would prefer to stay with it - that may be an option too, especially in case you foresee a lot of work to reimplement.
Difference in the structure of the networks: Preferably stay with what is proposed in the papers, and evaluate. If you believe that some changes in the structure may improve performance, do test that and report on the performance improvement achieved from the change. This step would then be an original contribution. The main project task remains to compare existing methods as they are presented in resp. papers.


added input handling

@whuiras Accidentally branched off of issue-78 for my maven.yaml and pom.xml changes, so they're a part of this PR now. Oops! :tired_face:


immich-go: 0.19.1 -> 0.20

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for immich-go is: Immich client tool for bulk-uploads
meta.homepage for immich-go is: https://github.com/simulot/immich-go
meta.changelog for immich-go is: https://github.com/simulot/immich-go/releases/tag/0.20
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.20 with grep in /nix/store/wvhj2bcxhwijciaa5jj425hd4q5asyix-immich-go-0.20
found 0.20 in filename of file in /nix/store/wvhj2bcxhwijciaa5jj425hd4q5asyix-immich-go-0.20

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

immich-go

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wvhj2bcxhwijciaa5jj425hd4q5asyix-immich-go-0.20 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/c8ef4484228a4d45db8c14c2fc77d53cd16808e6#immich-go

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wvhj2bcxhwijciaa5jj425hd4q5asyix-immich-go-0.20
ls -la /nix/store/wvhj2bcxhwijciaa5jj425hd4q5asyix-immich-go-0.20/bin

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

1 package built:

immich-go

Maintainer pings
cc @kai-tub for testing.

@NixOS/nixpkgs-merge-bot merge

Second try:
@NixOS/nixpkgs-merge-bot merge


Question about MSE Codec tests MediaSourceDuration

In MSE Codec tests MediaSourceDuration item, I have a question about test function
https://github.com/youtube/js_mse_eme/blob/master/media/mseCodecTest.js#L855
Why it uses checkEq?
Why not uses checkApproxEq as previous line to do the check?

Fixed in 9edb3984f98a37ccf6620bd59d4e83eba8fc23a4


Update docs for Selenium Grid Scaler

Provide a description of what has been changed
Update docs for Selenium Grid Scaler following https://github.com/kedacore/keda/pull/6368
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #

Close as no config changed in upstream fix ticket!


Exception when used in combination with sphinxcontrib-spelling

The sphinx-data-viewer extension throws an exception when running the spelling checker from sphinxcontrib-spelling. The following error is produced when running make spelling:
Extension error (sphinx_data_viewer.sphinx_data_viewer):
Handler <function install_lib_static_files at 0x7f63c586d260> for event 'env-updated' threw an exception (exception: 'SpellingBuilder' object has no attribute 'script_files')

This only happens when using sphinx-data-viewer version 0.1.3 in combination with recent versions of Sphinx tooling. This is a fragment of my Pipfile:
[packages]
sphinx = "==7.2.6"
sphinxcontrib-spelling = "==8.0.0"
sphinx-needs = "==2.0.0"
sphinxcontrib-plantuml = "==0.27"

When sphinx-data-viewer is downgraded to version 0.1.2 the error disappears.
Output when running sphinx-build with -v:
$ make spelling
Running Sphinx v7.2.0
Initializing Spelling Checker 8.0.0
making output directory... done
locale_dir /home/user/doc/locales/en/LC_MESSAGES does not exist
Ignoring wiki words
Ignoring acronyms
Ignoring Python builtins
Ignoring importable module names
Ignoring contributor names
Looking for custom word list in /home/user/doc/spelling_wordlist.txt
Scanning contributors
locale_dir /home/user/doc/locales/en/LC_MESSAGES does not exist
building [mo]: targets for 0 po files that are out of date
writing output...
building [spelling]: all documents
updating environment: locale_dir /home/user/doc/locales/en/LC_MESSAGES does not exist
[new config] 8 added, 0 changed, 0 removed
reading sources... [ 12%] hardware/hdd
reading sources... [ 25%] hardware/hrs
reading sources... [ 38%] hardware/hts
reading sources... [ 50%] index
reading sources... [ 62%] software/sdd
reading sources... [ 75%] software/srs
reading sources... [ 88%] system/ssdd
reading sources... [100%] system/ssrs

Traceback (most recent call last):
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx/events.py", line 96, in emit
    results.append(listener.handler(self.app, *args))
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx_data_viewer/sphinx_data_viewer.py", line 68, in install_lib_static_files
    if extra_path.endswith("js") and str(web_path) not in app.builder.script_files:
                                                          ^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'SpellingBuilder' object has no attribute 'script_files'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx/cmd/build.py", line 298, in build_main
    app.build(args.force_all, args.filenames)
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx/application.py", line 354, in build
    self.builder.build_update()
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx/builders/__init__.py", line 290, in build_update
    self.build(['__all__'], to_build)
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx/builders/__init__.py", line 313, in build
    updated_docnames = set(self.read())
                           ^^^^^^^^^^^
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx/builders/__init__.py", line 426, in read
    for retval in self.events.emit('env-updated', self.env):
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.local/share/virtualenvs/doc-_SZa7Qpc/lib/python3.11/site-packages/sphinx/events.py", line 107, in emit
    raise ExtensionError(__("Handler %r for event %r threw an exception") %
sphinx.errors.ExtensionError: Handler <function install_lib_static_files at 0x7f5120210f40> for event 'env-updated' threw an exception (exception: 'SpellingBuilder' object has no attribute 'script_files')

Extension error (sphinx_data_viewer.sphinx_data_viewer):
Handler <function install_lib_static_files at 0x7f5120210f40> for event 'env-updated' threw an exception (exception: 'SpellingBuilder' object has no attribute 'script_files')
make: *** [Makefile:27: spelling] Error 2

Thanks for the detailed error report. :+1:
It is fixed by checking the current builder and its capability to register js/css files.


reverse-geocoding-<IP_ADDRESS> fails Haddock phase

Haddock cannot process your source code, apparently because of some syntaxerror in the markup. At http://hydra.cryp.to/build/418515/nixlog/2/raw, you'll find a complete build log that shows the error message.

Hi... I had actually fixed this on a previous commit, but then I got distracted when I merged another branch and this creeped back in.
It should now be fixed, at least running cabal haddock in my system does not report errors.
I've pushed it here and to hackage, thanks!

Very cool, thank you!

Well, version <IP_ADDRESS> has exactly the same problem:
src/Data/Geolocation/Reverse/Providers.hs:12:19:
    parse error on input ‘-- ^ Only one supported right now’

A complete build log is at http://hydra.cryp.to/build/525396/nixlog/1/raw.

Yes, my fault... it should be version <IP_ADDRESS>: I uploaded the package candidate to hackage, forgot to publish.
Should be ok now, please confirm.
Cheers

Works fine now: http://hydra.cryp.to/build/525788. Thank you!


Unable to obtain body of a post request.

So I have a simple function that makes a simple post request with some parameters:
    async with aiohttp.ClientSession(raise_for_status=True) as session:
        async with session.post(trigger.url, data=params) as response:
            return await response.json()

When I try to test the parameters passed:
        aresponses.add("test.url.com", "/trigger/path", "POST", aresponses.Response(body=json.dumps({"some": "response"}), status=201, headers={"Content-Type": "application/json"}))
        response = await utils.invoke_trigger(trigger, variables)
        print(await aresponses.history[0].request.post())

I get following error:
self = <StreamReader 75 bytes eof e=PayloadAccessError()>

    async def readany(self) -> bytes:
        if self._exception is not None:
>           raise self._exception
E           aiohttp.web_protocol.PayloadAccessError

.venv/lib/python3.8/site-packages/aiohttp/streams.py:386: PayloadAccessError

Thanks for catching that @takeda ! I believe I've fixed this now and it's available as version 2.0.2
@yaaminu I believe you were looking for a feature like this?


Issue in load_dataset_to_es in v0.1

I got a general out of index error in step2_export_to_elasticsearch as below,
Caused by: java.lang.IndexOutOfBoundsException: 0
File "/opt/seqr/hail-elasticsearch-pipelines/hail_scripts/v01/utils/elasticsearch_client.py", line 373, in export_kt_to_elasticsearch
kt.to_dataframe().show(n=5)
I debugged it a little bit, and found this line return an empty KeyTable. And I dig a little bit and found domains = va.domains[va.aIndex-1] in site_fields_list prevents the application to generate a KeyTable and thus df is empty as well. I had to manipulate the list to skip this field and was loaded successfully . I believe the split_multi() has already been run since I saw the message "Hail: WARN: called redundant split on an already split VDS" somewhere else.
Any thoughts on this issue?

It looks like generate_vds_make_table_arg assumes that if is_split_vds is true then every array field in the table contains one value for each alt allele. Depending on that dataset, that may not be true.
https://github.com/macarthur-lab/hail-elasticsearch-pipelines/blob/92526f1183e0c7054f6b7bb5ae688da6629a4417/hail_scripts/v01/utils/elasticsearch_utils.py#L206-L207
I think you could work around this by passing is_split_vds=False to export_vds_to_elasticsearch. This would require you to handle any array fields containing per-alt allele values yourself instead of relying on generate_vds_make_table_arg to do it automatically.

This looks like the same issue as https://github.com/macarthur-lab/hail-elasticsearch-pipelines/issues/123

This looks like the same issue as #123 . Converting the Set to an Array triggers this downstream issue.

I suspect these are two different issues though they are pointing to the same (similar) field of domains. I can see the schema here is still an Array data structure, the other one #123  was converted to a Set which caused the problem.

It looks like generate_vds_make_table_arg assumes that if is_split_vds is true then every array field in the table contains one value for each alt allele. Depending on that dataset, that may not be true.
hail-elasticsearch-pipelines/hail_scripts/v01/utils/elasticsearch_utils.py
Lines 206 to 207 in 92526f1
if is_split_vds and isinstance(field_type, basestring) and field_type.startswith("Array"):
expr += "[va.aIndex-1]"
I think you could work around this by passing is_split_vds=False to export_vds_to_elasticsearch. This would require you to handle any array fields containing per-alt allele values yourself instead of relying on generate_vds_make_table_arg to do it automatically.

I don't know what exactly happens here (I mean va.domains[va.aIndex-1], I do see domains filed is An Array in VDS schema). This step didn't throw any exception but return an empty KeyTable silently. And the exception was threw until the dataframe tries to show the first 5 rows which is due to a empty key table.

And the exception was threw until the dataframe tries to show the first 5 rows which is due to a empty key table.

I doubt that's what is actually throwing the exception. An empty dataframe can be shown.
~$ pyspark
>>> from pyspark.sql.types import *
>>> field = [StructField("field1", StringType(), True)]
>>> schema = StructType(field)
>>> df = sqlContext.createDataFrame(sc.emptyRDD(), schema)
>>> df.count()
0
>>> df.show(5)
+------+
|field1|
+------+
+------+

Spark transformations are lazy, so these expressions aren't actually run until something (like show) requires their results. So an index out of bounds error won't show up when select or annotate is first called since it's not immediately evaluating the select/annotate expression.

I don't know what exactly happens here (I mean va.domains[va.aIndex-1], I do see domains filed is An Array in VDS schema)

A VDS imported from a VCF may contain multi-allelic variants. In addition, some info fields may contain an array with one value per alt allele.

chrom
pos
alleles
info.AC

1
100
["A", "C"]
[20]

1
200
["C", "T", "G"]
[5, 10]

split_multi splits those multi-allelic variants into multiple rows, but it doesn't split other fields. So after split_multi, that example would look like:

chrom
pos
alleles
info.AC
aIndex
wasSplit

1
100
["A", "C"]
[20]
1
false

1
200
["C", "T"]
[5, 10]
1
true

1
200
["C", "G"]
[5, 10]
2
true

That's where the va.fields[va.aIndex - 1] annotations come in. They are used to filter the fields containing per alt-allele values down to the one value for that row's variant. So replacing info.AC with info.AC[va.aIndex - 1] results in:

chrom
pos
alleles
info.AC

1
100
["A", "C"]
[20]

1
200
["C", "T"]
5

1
200
["C", "G"]
10

So if your original dataset contains an array field (domains) that doesn't have one value per alt-allele, attempting to read domains[va.aIndex - 1] would result in an index out of bounds error.

I agree with Nick that would be one thing that could cause this error.
For load_clinvar_to_es specifically, I was able to reproduce the error with the current clinvar release downloaded by the script.
When I then changed this line
https://github.com/macarthur-lab/hail-elasticsearch-pipelines/blob/master/hail_scripts/v01/utils/computed_fields.py#L330
from
.map( x => x.domains)
back to
.map( x => x.domains.map( domain => domain.db+":"+domain.name ) )
it got rid of the error, and ran to completion successfully.
Let me check with @hanars  if we should make this change in the repo or fix it some other way.

I agree with Nick that would be one thing that could cause this error.
For load_clinvar_to_es specifically, I was able to reproduce the error with the current clinvar release downloaded by the script.
When I then changed this line
https://github.com/macarthur-lab/hail-elasticsearch-pipelines/blob/master/hail_scripts/v01/utils/computed_fields.py#L330
from
.map( x => x.domains)
back to
.map( x => x.domains.map( domain => domain.db+":"+domain.name ) )
it got rid of the error, and ran to completion successfully.
@hanars @nawatts What do you think - should we go back to .map( x => x.domains.map( domain => domain.db+":"+domain.name ) ) or fix this another way?

Hi Ben, sorry I just have time to verify this. Your fix did solve the issue in #123, but for this specific load_dataset_to_es script, it gives a different compliant. No doubt We will move forward to 0.2 in future, not sure if this is worthy for your team to look into this. BTW, I haven't tried the workaround mentioned above is_split_vds=False.
Hail version: 0.1-105a497
Error summary: HailException: String' has no field or method db'
:3:        .map( x => x.domains.map( domain => domain.db+":"+domain.name ))

@hanars @nawatts What do you think - should we go back to .map( x => x.domains.map( domain => domain.db+":"+domain.name ) ) or fix this another way?

@knguyen142's comment on #131 identified with the issue with the ClinVar script.
get_expr_for_vep_protein_domains_set was originally designed to be called on the unaltered VEP transcript consequences, not the sorted transcript consequences created by get_expr_for_vep_sorted_transcript_consequences_array. That was changed in #87, which updated the call in load_dataset_to_es.py but not load_clinvar_to_es.py.
@bw2

I think the issue came from the VCF files before and @nawatts  was right. After I moved away from that VCF, I don't have the issues any more. But I believe ClinVar script is a different issue as @bw2  already re-produced the issue. I will close this one. Thanks for all the helps!


fix the schema version error in sync schema when latest schema diff is empty

What problem does this PR solve?
Issue Number: close #5499 close #5397
Problem Summary:
What is changed and how it works?
In schemaBuilder.apply(schema diff), we will update the storageDeltaMerge schema version based on the global schema version.
Thus we need to check whether the latest schema diff is empty before apply diff (in this case we need to update version = version - 1 first)
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @lidezhu @JaySon-Huang

/run-all-tests

/run-all-tests

/merge

/run-unit-test

/run-integration-test

/run-all-tests

/merge


Support for Python 3.10 with Astro Python SDK

I think we do support Py 3.10 for Astro Python SDK -- but we should officially test it out and add a test in CI

Look like currently we run tests in Python3.8 env only. I was thinking to enable test for other Python versions too but CI time will increase drastically.

for now, as discussed we are putting this on hold cc: @phanikumv

Since now we have some separation between unit and integration tests and units tests take relatively less time so I have started on it.  We will run unit tests for Python-{3.8,3.9,10}


onBindViewHolder not always called for inserted children

Sometimes when I do collapseParent B and then expandParent A, B's children do not get recycled and show up in the A's children.
I tried calling notifyChildItemRangeChanged after the expandParent and it didn't do anything.
Looking at it with a debugger, onBindViewHolder wasn't called for every new child. I don't really know why it happens, but I guess this doesn't do the job, and notifyItemRangeInserted should be called instead.
If I call notifyItemRangeInserted instead of notifyChildItemRangeChanged, then it works fine.

Interesting, are these calls happening back to back?
Shouldn't need to call any notify methods because we are calling those for the expand and collapses. I'll take a look into this, any chance one of the samples can be edited to show this bug?

Yes, I call them back to back.
I will try to reproduce the issue this weekend.
On Fri, Feb 5, 2016, 00:42 Paul Turner<EMAIL_ADDRESS>wrote:

Interesting, are these calls happening back to back?
Shouldn't need to call any notify methods because we are calling those for
the expand and collapses. I'll take a look into this, any chance one of the
samples can be edited to show this bug?
—
Reply to this email directly or view it on GitHub
https://github.com/bignerdranch/expandable-recycler-view/issues/158#issuecomment-180086346
.

Alright, here's the sample and the video on how to reproduce the issue.
In the video first I show how it's supposed to work, and then the bug itself. The 99999 seems to be moved out of screen completely.
This is what happens when I click on the first parent. I was wrong -- it's not about the collapseParent and expandParent, it's about the notifyParentItemRangeRemoved and expandParent. The order doesn't matter.
I don't remove any items from the mParentItemList, but the bug remains if I do, so I left it out of the commit.

@egor-n Thanks for the source code! This may help us get to the bottom of this.
Your last sentence "I don't remove any items from the mParentItemList, but the bug remains if I do, so I left it out of the commit." is one that I'm curious about.
As the sample stands now, the call to notifyParentItemRangeRemoved is incorrect because of the fact that no parents have been removed. The notify does not make any modifications to the list, it simply updates the recycler view assuming that those changes have been made. So right now it moves items around from position 1 and 2 out even though those are still present in the adapters data (most likely causing the wrong children to appear when expanding position one, and scrolling it off screen to cause a different item to reload.).
Could you show the case where the bug remains even when the removal of items is happening? I want to see your approach before seeing where I would put said code to better understand the confusion. Hope this isn't too much trouble!

@paul-turner Thank you for your response.
Here's the code. The bug is still present with these changes, the 99999 item still appears instead of the 22222.

@egor-n found the problem! (At least with this specific case).
mAdapter.getParentItemList().subList(1, 2) will return only the second item, because the sublist function is a start and ending index, where the start is inclusive and the end index is exclusive.
This means when we do a clear, we are only deleting item at position 1, so the notify code would need to change to: mAdapter.notifyParentItemRangeRemoved(1, 2); or mAdapter.notifyParentItemRemoved(1);
Or leave the parentItemRangeRemoved as is and change the clear to: mAdapter.getParentItemList().subList(1, 3) so that it actually removes the last two.
Let me know if that clears up the issue, or maybe we still have a bug somewhere just need to update the sample a little bit more :)

@paul-turner you are right about the subList - it is, in fact, exclusive with the end parameter.
I have just tried changing it to .subList(1, 3) and the problem is still present, though :(

Ran it again on my computer with the change, everything seems to be ok on my end.
Maybe you ran the old apk by accident? (the link is pointing to the old commit, though I see the new commit with the changes to 1,3)

@paul-turner I've done a clean build and am positive that I am running the latest .apk. The undesired behavior is still present.
I've attached the debugger and stepped through the code. Here's what I see:

The value of mName in Ingredient@4410 is 22222.

I was able to further isolate this bug. Take a look at the code here, please.
I am not using your library at all at this point. No parent/child item types but the logic remained the same. I do everything manually as would ExpandableRecyclerAdapter do.
I am not sure why you can't reproduce this bug. I can say that if the last item does not scroll off the screen, then the bug doesn't appear.

I see the bug now, turns out you have to scroll down and then scroll back up (I was simply hitting the expand button on the first item).
I've stepped through the code trying to figure out what may be going wrong, based on your last example it feels like it is either a bug with RecyclerView or a misuse of the notify calls. I'm going to make a simple app that demonstrates the problem and see if I can get other peoples opinions and probably submit a bug report after that.

@paul-turner thanks for spending your time on this! Let me know if I can help you somehow.

Here is the sample https://github.com/paul-turner/RecyclerViewNotifyTest showing the bug and the bug report here: https://code.google.com/p/android/issues/detail?id=201055&thanks=201055&ts=1455575559
I did see that calling notifyItemRangeRemoved(1, 4) resulted in the correct behaviour for my example (instead of saying notifyItemRangeRemoved(2, 4) which takes into account the added row). We will see if it is correct or not to shift the indices based on previous data set changes.

Well, I cannot reproduce this bug using the 23.2.0 version.
Too bad it wasn't looked at by a Google dev, who was assigned to the issue. I hope it won't appear again because of that.


feat: add link to repo to SSR shell

@brillout what do you think of this?

👍 Good idea, feel free to merge.


[FLINK-14878]Support use  catalog through sqlUpdate() method in TableEnvironment

What is the purpose of the change
Support USE CATALOG catalogName through sqlUpdate() method in TableEnvironment
Brief change log

c2375e0 Add useCatalogOperation and support it both in flink/blink planner
442d683 add converter test and make sqlUseCatalog sql kind to other ddl to s
ea05725 add related ITcase in CatalogTableITCase to both planner

Verifying this change
This change added tests and can be verified as follows:

CatalogTableITCase#testUseCatalog
SqlToOperationConverterTest#testUseCatalog

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: ( no )
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (not applicable )

cc @bowenli86 @xuefuz @lirui-apache @KurtYoung to have a review.

sorry for the late comment, but I happened to notice that UseCatalogOperation resides in package /table/operations/ddl/. Since it's not ddl, should it just be relocated to /table/operations/?

sorry for the late comment, but I happened to notice that UseCatalogOperation and UseOperation reside in package /table/operations/ddl/. Since they are not ddl, should they be relocated to /table/operations/?

Make sense, addressed.


MM-37608 and MM-37416 issue fixes

Summary

Fixed the MM-37608 issue
Fixed the MM-37416 issue

Ticket Link

https://mattermost.atlassian.net/browse/MM-37608
https://mattermost.atlassian.net/browse/MM-37416

Checklist

[ ] Added or updated unit tests (required for all new features)
[x] Has UI changes
[ ] Includes text changes and localization file updates

Device Information
This PR was tested on:

iPhone 12 Simulator - iOS 14.4
Android Emulator - Pixel_3a_API_30_x86 - Android 10

Release Note
Fixed the MM-37608 and MM-37416 issues

Hello @manojmalik20,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Building app in separate branch.

Cherry pick is scheduled.


Deprecate unused trade methods/add "baseline buy" method

Changes

Comments

#289  👈
master

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

Join @forstmeier and the rest of your teammates on  Graphite


Detection results trainval

Hi, thank you for the nice work!
Would it be possible to get the detection results of your method also for the trainval set of nuScenes?

All the model results are there in modelzoo https://github.com/tianweiy/CenterPoint/blob/master/docs/MODEL_ZOO.md
under Nuscenes_3d tracking in json format

Yeah, you can find val set result in the Google drive. The train set is too large to upload for now and you can easily generate it yourselves.


Add permission checks to save_or_overwrite_slice

This PR is meant to resolve issue #3498

Add Slice permission check for saveas to explore request
Add Dashboard permission check for add_to_dash == existing to explore request
Add Dashboard permission check for add_to_dash == new to explore request

Sorry our linter configuration is picky about ' over ".
As a sidenote (may be out-of-scope for this PR) we need to write a solid test suite around accessibility/security. I'll create an issue for it.


Allow usage to switch between different views of the recent usage table (js)

Closed in favour of https://github.com/Energy-Sparks/energy-sparks/pull/2825 (see more info in ticket)