The type org.apache.spark.sql.api.java.JavaSQLContext cannot be resolved.

I was not able to find the version 1.0.0 as mentioned in ReadMe through maven.
groupId: com.databricks
artifactId: spark-csv_2.10
version: 1.0.0

So, I tried to version 0.1 in my pom.xml.
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
	<modelVersion>4.0.0</modelVersion>
	<groupId>com.spark.test</groupId>
	<artifactId>SparkTest</artifactId>
	<version>0.0.1-SNAPSHOT</version>
	<dependencies>
		<dependency> <!-- Spark dependency -->
			<groupId>org.apache.spark</groupId>
			<artifactId>spark-core_2.10</artifactId>
			<version>1.3.0</version>
		</dependency>
		<dependency>
			<groupId>org.apache.spark</groupId>
			<artifactId>spark-sql_2.10</artifactId>
			<version>1.3.0</version>
		</dependency>
		<dependency>
			<groupId>com.databricks</groupId>
			<artifactId>spark-csv_2.10</artifactId>
			<version>0.1</version>
		</dependency>
	</dependencies>
</project>

And when I try to parse a csv, it results in a compilation error.
The type org.apache.spark.sql.api.java.JavaSQLContext cannot be resolved. It is indirectly referenced from required .class files
My Java code
SparkConf conf = new SparkConf();
JavaSparkContext sc = new JavaSparkContext(conf);
SQLContext sqlContext = new org.apache.spark.sql.SQLContext(sc);
DataFrame cars = (new JavaCsvParser()).withUseHeader(true).csvFile(sqlContext , "cars.csv");

I'm not able  to understand why the JavaCsvParser is looking for  org.apache.spark.sql.api.java.JavaSQLContext, since in Spark 1.3 they have merged both JavaSQLContext and  SQLContext into SQLContext Again please let me know if I have not configured my pom.xml correctly.
(I'm new to spark and I'm not sure if I'm talking sense here)
Thanks

We have not officially released version 1.0.0 on Maven. This will be done within coming few days.

1.0.0 is released.

Hi,
My pom entries are  :

org.apache.spark
spark-core_2.10
1.4.1

org.apache.hadoop
hadoop-client
2.7.1

org.apache.spark
spark-sql_2.10
1.3.0

com.databricks
spark-csv_2.11
1.0.3

I get this error on build :
[ERROR] COMPILATION ERROR :
error: cannot find symbol
[ERROR]   symbol:   class DataFrame
The java code where SQLContext & DataFrame is used :
private static final SQLContext sqlContext = new SQLContext(sc);
DataFrame cityDF = sqlContext.sql("select CITY,count(*) as Total_Customers from customers group by CITY");
Is it because the jars of spark sql doesnt contain DataFrame ? which is the right sparksql jar i should be using in that case ?
Thanks in advance.

It's because you are using spark-csv_2.11 but spark-sql_2.10. Could you
also please change 2.11 to 2.10 for spark-Csv?
On Aug 12, 2015 6:56 PM, "Vinod Jayakumar"<EMAIL_ADDRESS>wrote:

Hi,
My pom entries are :
org.apache.spark
spark-core_2.10
1.4.1
org.apache.hadoop
hadoop-client
2.7.1
org.apache.spark
spark-sql_2.10
1.3.0
com.databricks
spark-csv_2.11
1.0.3
I get this error on build :
[ERROR] COMPILATION ERROR :
error: cannot find symbol
[ERROR] symbol: class DataFrame
The java code where SQLContext & DataFrame is used :
private static final SQLContext sqlContext = new SQLContext(sc);
DataFrame cityDF = sqlContext.sql("select CITY,count(*) as Total_Customers
from customers group by CITY");
Is it because the jars of spark sql doesnt contain DataFrame ? which is
the right sparksql jar i should be using in that case ?
Thanks in advance.
—
Reply to this email directly or view it on GitHub
https://github.com/databricks/spark-csv/issues/27#issuecomment-130506602
.

Thanks for the quick reply.
this didnt work :(
I changed spark-csv to 2.10 to match with spark-sql version 2.10.

org.apache.spark
spark-sql_2.10
1.3.0

com.databricks
spark-csv_2.10
1.0.3

Could you also match your spark-sql and spark-core version? To 1.4.1
perhaps?
On Aug 12, 2015 7:10 PM, "Vinod Jayakumar"<EMAIL_ADDRESS>wrote:

Thanks for the quick reply.
this didnt work :(
I changed spark-csv to 2.10 to match with spark-sql version 2.10.
org.apache.spark
spark-sql_2.10
1.3.0
com.databricks
spark-csv_2.10
1.0.3
—
Reply to this email directly or view it on GitHub
https://github.com/databricks/spark-csv/issues/27#issuecomment-130508076
.

In fact you don't need spark-core once you have spark-sql.
On Aug 12, 2015 7:11 PM, "Burak Yavuz"<EMAIL_ADDRESS>wrote:

Could you also match your spark-sql and spark-core version? To 1.4.1
perhaps?
On Aug 12, 2015 7:10 PM, "Vinod Jayakumar"<EMAIL_ADDRESS>wrote:

Thanks for the quick reply.
this didnt work :(
I changed spark-csv to 2.10 to match with spark-sql version 2.10.
org.apache.spark
spark-sql_2.10
1.3.0
com.databricks
spark-csv_2.10
1.0.3
—
Reply to this email directly or view it on GitHub
https://github.com/databricks/spark-csv/issues/27#issuecomment-130508076
.

it worked now. Sorry it was a silly miss from my side. I was importing scala.row and other older imports.
i replaced them with these
import org.apache.spark.SparkConf;
import org.apache.spark.api.java.JavaSparkContext;
import org.apache.spark.sql.DataFrame;
import org.apache.spark.sql.Row;
import org.apache.spark.sql.SQLContext;
@brkyvz Thanks a lot for your replies


Increase zombie prying time for bolted doors

About the PR

Closes #23165
Why / Balance

It's strange that it takes the same time for zombies to pry open bolted versus unbolted doors. This encourages survivors to bolt doors to possibly buy them some extra time against the horde.
Technical details

Simply adds a new pry time modifier for bolted status. This currently only affects zombies, as nothing else seems to have the ability to pry open bolted doors.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

tweak: Zombies now take longer to pry open bolted doors.

changelog is missing :cl:

as nothing else seems to have the ability to pry open bolted doors.

jaws of life :godo:

removed


Update goodsync to <IP_ADDRESS>

After making all changes to the cask:

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

This was a mistake, version was up to date.


Hyprlock kills hyprland session when power cycling monitor

Brief summary of the problem:
Turning off one of my three monitors, waiting a few seconds and then turning said monitor back on, leads to hyprlock crashing and taking my hyprland session with it, which throws me back to my display manager.

Hardware description:

CPU: R7 7800X3D
GPU: AMD RX 7900 XTX
Display: Main 5120x1440 120hz DP, secondary 2560x1440 75hz HDMI, third 1920x1080 60hz HDMI

Distro info:

Name: NixOS 24.11.20240613.e9ee548 (Vicuna) x86_64
Kernel version: Linux 6.9.4
Mesa version: 24.1.1
Hyprland and hyprlock package: nixos-unstable

How to reproduce the issue:

Lock system with all monitors active
Shutoff one of the monitors and wait a few seconds
Turn on said monitor
This should result in being booted back into the display manager

sounds like a hyprland issue, clients should not crash the compositor. Try hyprland-git.


Document CONNECT use-case with parse_authority

See: https://datatracker.ietf.org/doc/html/rfc7230#section-5.3.3

We already have an example like that in https://develop.url.cpp.al/url/parsing/authority.html#url.parsing.authority.authority_view

Nice! Since this is the use-case that 99% of people will need, I think we should reflect that in the section heading so they can find it in the table of contents.

We did have something like that, where authority_view had its own section. I merged the sections in some previous PR to make the TOC look a little better:

(the overview could still improve though)
Maybe we have a different idea for that, but isolating authority view in the TOC looked quite ugly:

This previous layout also made the relationship between authority and authority_view more complex to explain, less natural to the reader, and I'm not sure it's so helpful, since authority is already visible and it's the natural place to look for that kind of use case.  But I don't know, maybe there's a better way I haven't considered yet.

isolating authority view in the TOC looked quite ugly:

Yes that image of the ToC looks bad. It repeats the word Authority. How about the heading "HTTP CONNECT" since that's usually what it is used with?

To be honest, the repeated word is something we can improve but not something I mind that much because this is likely to happen in this kind of document anyway.
What I don't like is

visually the snaggle-tooth as an exception for something that is really just a use case among many, and
how it conceptually affects the section hierarchy, because "authority view" is not even the last subsection of this section and this becomes a mess: the "Authority" section would include text that should come before and after the "Authority view"

But we can do anything. Just say the word.

"Documentation can always be improved."
Get the information into the docs, and we can worry how to make it pretty later.

Get the information into the docs, and we can worry how to make it pretty later.

The information is in the docs already. We are just discussing the presentation now.

"Documentation can always be improved."

At this point, we're not discussing an improvement. I think we're discussing whether this would make it worse.

What I mean is we can live with it the way it is now...and worry about making it better later.

I support that :)


Label suggestion

BEGINNER :)
-> https://github.com/search?utf8=✓&q=state%3Aopen+label%3ABEGINNER&type=Issues

@PeterDaveHello This is already a label in issuehub

oops, thanks @pablopunk !


How to make PrivateGPT to translate everything into English language before store and process inputs

Hi, I want to use PrivateGPT for Slovak documents, but it's not possible, because there is no LLM model that can work with Slovak language. I tried to make a small testing python script that will

read the txt file
translate it with Helsinki-NLP/opus-mt-sk-en into English
summarize that English text with Falconsai/medical_summarization
translate that summarization into the Slovak language with Helsinki-NLP/opus-mt-en-sk

And the result was actually very good. So my suggestion is that PrivateGPT could have possible configuring translators so LLM and Vector database will have to work only with English text. What do you think about this? In settings.yaml it will be something like this
translations:
  from_prompt:
    model: Helsinki-NLP/opus-mt-sk-en
    # source: eng_Latn # Could be commented when this arguments are not required to the translation model
    # target: slk_Latn
  from_response:
    model: Helsinki-NLP/opus-mt-en-sk # could be commented when it's same as from_prompt
    # source: slk_Latn
    # target: eng_Latn
  to_store:
    model: Helsinki-NLP/opus-mt-sk-en
    # source: eng_Latn # English
    # target: slk_Latn # Slovak
  from_store:
    model: Helsinki-NLP/opus-mt-en-sk # could be commented when it's same as to_store
    # source: slk_Latn
    # target: eng_Latn

What do you think about this? Im a noob in Python so I'm not able to make it by myself, thx.

Hi, I want to use PrivateGPT for Slovak documents, but it's not possible, because there is no LLM model that can work with Slovak language. I tried to make a small testing python script that will
* read the txt file

* translate it with `Helsinki-NLP/opus-mt-sk-en` into English

* summarize that English text with `Falconsai/medical_summarization`

* translate that summarization into the Slovak language with `Helsinki-NLP/opus-mt-en-sk`

And the result was actually very good. So my suggestion is that PrivateGPT could have possible configuring translators so LLM and Vector database will have to work only with English text. What do you think about this? In settings.yaml it will be something like this
translations:
  from_prompt:
    model: Helsinki-NLP/opus-mt-sk-en
    # source: eng_Latn # Could be commented when this arguments are not required to the translation model
    # target: slk_Latn
  from_response:
    model: Helsinki-NLP/opus-mt-en-sk # could be commented when it's same as from_prompt
    # source: slk_Latn
    # target: eng_Latn
  to_store:
    model: Helsinki-NLP/opus-mt-sk-en
    # source: eng_Latn # English
    # target: slk_Latn # Slovak
  from_store:
    model: Helsinki-NLP/opus-mt-en-sk # could be commented when it's same as to_store
    # source: slk_Latn
    # target: eng_Latn

What do you think about this? Im a noob in Python so I'm not able to make it by myself, thx.

In order for this recipe to work, is it necessary to first convert Helsinki-NLP/opus-mt-sk-en to gguf and put it in the models folder privategpt?


Home page image refresh trying a picture element instead of next/image

This uses a standard <picture> html element instead of next/image for responsive sizing based on aspect ratio
Preview

That is a cleaner solution on the HTML/JS side, the downside is that we don't get the automated asset pipeline and compression that the next/image approach gives us. Loading both your and my PRs in the same browser at the same size, the next/image approach loads an optimized 1.1MB image, while this PR loads the original 3.5MB image.
Is there maybe a way to tap into that asset pipeline for the picture element so we get the best of both worlds?

do we need to change the favicon too @GBKS

I think the favicon is ok for now, we did just change that one pretty recently


Returning enum named c_int in main causes C++ error

This program causes the C++ compiler to throw an error:
enum c_int: u64 {
    Abc = 1
}

function main() {
    return c_int::Abc
}

There is a mismatch in main's return value. The expected result here is for the error to be handled by Jakt's front-end instead of the C++ back-end.


test,Nangate45,cell.v: fix QN in DFF_X2

Thanks for the fix.  Please note that commits need to be signed (git commit -s).  See the Details link by the DCO action for steps to fix that.

@maliberty I think you need to hit the button to let them run the CI

Thanks!


Favorites for charts are visible in different game modes

Describe the bug
If a chart has difficulties between two different game modes (say, a dance difficulty and a solo difficulty), favoriting the song in one game mode will make it show up as a favorite in other game modes.
To Reproduce
Favorite "Bullet Timing" from Dark Chanchellors in the dance game mode.
Switch to the solo game mode; the song will show up as favorited.
Expected behavior
This is unintuitive behavior.
If I favorite a dance difficulty of a song, it doesn't mean that I love the song in solo as well where the difficulty that made me favorite the song in the first place isn't even available.
Ideally favorites of a song would track what game mode the song was favorited in, or (even better) should be tied to individual difficulties of a song, not the song as a whole.
Doing the latter would also make combinations of filters + favorite sort / playlist more intuitive as well.

c803c09eae8d08218b167c2bb39b6a1fbcc5278f


0.3.3 breaks Linux Mint compatibility again

Just like 0.3.1 broke compatibility with Linux Mint (which was fixed in 0.3.2), it is once again broken in 0.3.3.
Expected behavior
deb-get 0.3.3 works when run unter Linux Mint 20.3
Actual behavior
deb-get shows this error message when run:
[!] ERROR! Linux Mint Una is not supported because it is not derived from a supported Debian or Ubuntu release.
Steps to reproduce
just run "deb-get" with none or any parameters
System information
Run the following a paste the output below:
/etc/os-release
NAME="Linux Mint"
VERSION="20.3 (Una)"
ID=linuxmint
ID_LIKE=ubuntu
PRETTY_NAME="Linux Mint 20.3"
VERSION_ID="20.3"
HOME_URL="https://www.linuxmint.com/"
SUPPORT_URL="https://forums.linuxmint.com/"
BUG_REPORT_URL="http://linuxmint-troubleshooting-guide.readthedocs.io/en/latest/"
PRIVACY_POLICY_URL="https://www.linuxmint.com/"
VERSION_CODENAME=una
UBUNTU_CODENAME=focal
/etc/lsb-release
DISTRIB_ID=LinuxMint
DISTRIB_RELEASE=20.3
DISTRIB_CODENAME=una
DISTRIB_DESCRIPTION="Linux Mint 20.3 Una"
deb-get version
prints error message, as shown above (but it is 0.3.3)
deb-get update
prints error message, as shown above
Just an idea...
Would it be possible to add some kind of CI tests, which run deb-get under different distros? That way regressions like this could be prevented.

I was taking a look to add something extra for Mint (not sure if that have been a good idea, anyway) but this may have been addressed with the merged commit https://github.com/wimpysworld/deb-get/commit/f091a1334d34dbd8777fef24480523ba4c602e2e  .. the deb-get team just hasn't released a 0.3.4 version as of yet, but if that commit works then this issue may already be fixed

Duplicate of #384. Fixed via https://github.com/wimpysworld/deb-get/commit/fe4cb91c60535c438af685894a4093e58eb241d4.


Support uninitialized Git repo rendering

When submitting a pipeline from an uninitialized Git repo, information about the repo may be successfully collected, even though the first commit does not exist. This will produce a rendering error:

By requiring a commit to exist for the purposes of rendering the Git information, the issue is fixed:

Should we instead detect that the repo is uninitialized and not collect the Git info?

@neutralino1 I can't find anything in the GitPython documentation for checking if a repo is initialized or not in addition to a check we already do. There are code snippets in that documentation that do things like check if there are any remotes configured. So it's just a matter of checking the individual attributes that you care about. In our case, we care about the commit SHA.

@tscurtu and can we make this commit check on the backend?
The reason I am pushing for this, is that it would ensure higher quality data, so that clients do not need to do those checks.


Fix integration test helpers

Hey @maddin2016 sorry for the stuff around :cry:
This PR should fix up the integration helper contexts to work with github.com and GHE without needing special/duplicate versions of them
I also fixed up your AddPermission test :+1:

Many thanks for fixed up AddPermission


feat: add file input component

This PR adds fileInput component by implementing following specifications : https://github.com/okp4/ui/issues/300

Codecov Report

Merging #308 (3a35b70) into main (3351c31) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #308   +/-   ##
=======================================
  Coverage   97.93%   97.93%
=======================================
  Files          60       60
  Lines         631      631
  Branches      101      101
=======================================
  Hits          618      618
  Misses         12       12
  Partials        1        1

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

:tada: This PR is included in version 1.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Predicted hit always white unless overridden

Idk if I'm missing something, after the recent fix to the <col #####> now my predicted hit text is always white unless overrridden.  Previously it would adopt your base XP drop colours/active prayer colours

Should be fixed with https://github.com/runelite/plugin-hub/pull/3486.


Ubuntu 18.04

Should this work on Ubuntu 18.04? After upgrading from 16.04 to 18.04 it doesn't seem to be running. What can I do to troubleshoot?

Installed a clean Ubuntu 18 box and ran everything again and now it's working.


Immovable Rod visual variations

About the PR

This PR adds 7 rare visual variations to the Immovable Rod, making it possible for other wacky items to replace the basic metal rod.
The current items have been added:

Mop
Hammer
Shark (Plushie)
Banana
Clown
Throngler
Gibstick

Why / Balance

Gives the immovable rod some more flavor and fits the announcement warning text, which is non-specific as of what item hits the station. The variations will not change the effect of the immovable rod.
To keep the variations rare and interesting when they happen, the rod has only a 1 in 10 chance to be replaced by a variation. All variations have an equal chance to be chosen.
Technical details

Uses a similar pick system to the Wilhelm scream easter egg, though with prototypes instead of sound files.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

no cl lots of fun

clown is amazing.
make it honk.

All of this can be done without modifying C#
make another rod entity with RandomSpriteComponent, where you customize all possible sprites. And make a spawner entity, where you customize the drop chances via Prototypes and RarePrototypes fields in RandomSpawnerComponent

I looked into this and while it would work, it also has some drawbacks that still requires modifying C#;

Custom descriptions wouldn't show with the RandomSpriteComponent. Probably not a great loss though.
The rod that gets spawned is the version that doesn't have velocity or direction. The rod entity is given those properties in ImmovableRodRuleSystem once spawned. If it were to be replaced with a spawner, the function would have to request the spawned rod's entity to apply those properties. But from what I can tell, there's no way to get the entity spawned with RandomSpawnerComponent?

I feel like most of these are kinda uninteresting and detract from the flavor of the rod. The clown is kinda funny but the other stuff is just random objects without anything else meaningful.

+1. IMO the only ones to keep are Clown (funny) and The Throngler (adds to the "mythos" surrounding the item).

I took the items in two conceptual categories; javelin-like items (Mop, Hammer, Throngler) and joke items (Clown, Gibstick, Shark Plushie, Banana). These items were sourced from suggestions by other SS14 players.
Due to the rarity of the variations I do not think there's a risk they will de-value the original rod. 9 times out of 10 it's still going to be the classic old reliable steel rod.

Weh plush rod for insta merge :trollface:

I feel like most of these are kinda uninteresting and detract from the flavor of the rod. The clown is kinda funny but the other stuff is just random objects without anything else meaningful.

Doesn't have to be meanigful. There is no lore. For all we know the rod is a magic being that is able to have any form. But I agree. Some of these are not that interesting (mop or shark).

I like the clown, throngler, and shark. The broom is kinda cool. The rest are just okay, except for the banana, which I don't think works at all.

Doesn't have to be meanigful. There is no lore. For all we know the rod is a magic being that is able to have any form. But I agree. Some of these are not that interesting (mop or shark).

Mop was the original inspiration actually. Someone who had been antagonising a janitor all round got hit by a rod while in perma, and people joked it was the janitor who had thrown their mop at them through the wall. 😂

If we are doing this, any variations should only have a 5% chance to be rolled at all. ie 95% of the time it's a regular rod.

5% chance? Come on people. I've been playing for almost 300 hours and I don't think I've actually ever seen one of these directly. I'd say make it 50% chance, it's funny and they put some good work into this.

I don't like that the appearance and name randomization feature is built into the rod code.

I don't like that the appearance and name randomization feature is built into the rod code.

I've read up closer on how EntitySpawnEntry handles probability and now have a better grasp of it; I should be able to change the code to move more of the randomness to .yaml.

the fixed rod component should contain only the logic of destruction and movement.  random appearance tangent logic must be completely removed from this system and component

the immovable rod component/system should contain only the logic of destruction and movement. random appearance tangent logic must be completely removed from this system and component

This has now been implemented. Random appearance logic is now solely contained within the EntitySpawnEntry component property and can be removed by simply deleting the rod variations from that property in the .yaml. The values of the variations add up to 5%.

CI failure unrelated.


docs: mention request for Pgx

Sorry for the comment from out of the blue. Jumanji's sophisticated API is great, and its application to problems like TSP is really interesting.
Today, we released Pgx, a collection of JAX-based RL environments dedicated to classic board games like Go. We have implemented over 15 environments, including Backgammon, Shogi, and Go, and confirmed that they are considerably faster than existing C++/Python implementations. We also plan to implement Chess and Contract Bridge in the coming weeks.
We believe Jumanji and Pgx can complement each other as both implement JAX-based RL environments but focus on different domains. We would be happy if you could kindly mention Pgx in the README like other JAX-based RL environments if you like it. For example,

🎲 Pgx provides classic board game environments like Backgammon, Shogi, and Go.

🎲 [Pgx](https://github.com/sotetsuk/pgx) provides classic board game environments like Backgammon, Shogi, and Go.

Thanks!

Hey @sotetsuk! Thank you for raising this issue! Pgx sounds awesome - just checked out your repo and it looks super cool! Re mentioning Pgx in the README, we can absolutely do that! 🚀 We have created a PR for it #108 😄

Thank you for your quick response and comments on Pgx! 🙏


Dragging and sorting in both directions?

I've looked at the example sortable list and have managed to make it work horizontal, by having the divs display in inline-block, and then changing the "Draggable-XXX" file's cardTarget, hover() method, to have X variables instead of Y.
Is there a way to somehow make react-dnd detect both direction dragging? I am currently trying putting this:
    // Dragging downwards     if (dragIndex < hoverIndex && hoverClientY < hoverMiddleY) {       return;     }     // Dragging upwards     if (dragIndex > hoverIndex && hoverClientY > hoverMiddleY) {       return;     }     // Dragging left     if (dragIndex < hoverIndex && hoverClientX < hoverMiddleX) {       return;     }     // Dragging right     if (dragIndex > hoverIndex && hoverClientX > hoverMiddleX) {       return;     }
but to no avail, the dragging is really buggy, sometimes it won't even swap the elements, and I have to move the mouse around like crazy to get some registration/feedback

Here you go @jr69 https://github.com/OliverJAsh/react-dnd-sortable-simple/commit/9ebcabe0f8ade4fd97bf30bf95f011d611072464

What we did with our team is the following and it's working ok: (based on @OliverJAsh and the original, done some tweaks)
    // Get vertical middle
    const hoverMiddleY = (hoverBoundingRect.bottom - hoverBoundingRect.top) / 2;

    // Get horizontal middle
    const hoverMiddleX = (hoverBoundingRect.right - hoverBoundingRect.left) / 2;

    // Determine mouse position
    const clientOffset = monitor.getClientOffset();

    // Get pixels to the top
    const hoverClientY = clientOffset.y - hoverBoundingRect.top;

    // Get pixels to the left
    const hoverClientX = clientOffset.x - hoverBoundingRect.left;

    const upwards = dragIndex > hoverIndex && hoverClientY > hoverMiddleY;
    const downwards = dragIndex < hoverIndex && hoverClientY < hoverMiddleY;
    const leftwards = dragIndex > hoverIndex && hoverClientX > hoverMiddleX;
    const rightwards = dragIndex < hoverIndex && hoverClientX < hoverMiddleX;

    if (upwards && (leftwards || rightwards)){
      return;
    }

    if (downwards && (leftwards || rightwards)){
      return;
    }

Thanks to all!
Juan

thanks @juampick , @OliverJAsh  :+1:

I've managed to make it work as expected, inspired by @OliverJAsh's  commit:
// get hovered image vertical middle position
    const hoverMiddleY = (hoverBoundingRect.bottom - hoverBoundingRect.top) / 2;

    // get hovered image horizontal middle position
    const hoverMiddleX = (hoverBoundingRect.right - hoverBoundingRect.left) / 2;

    // determine mouse position
    const clientOffset = monitor.getClientOffset();

    // get pixels to the top
    const hoverClientY = clientOffset.y - hoverBoundingRect.top;

    // get pixels to the left
    const hoverClientX = clientOffset.x - hoverBoundingRect.left;

    // only perform the move when the mouse has crossed half of the items height/width
    // are we dragging right or left?
    const dragRight = dragIndex === (hoverIndex - 1);
    const dragLeft = dragIndex === (hoverIndex + 1);

    // are we dragging up or down?
    const dragUp = dragBoundingRect.top > hoverBoundingRect.top;
    const dragDown = dragBoundingRect.bottom < hoverBoundingRect.bottom;

    if ( dragRight && dragIndex < hoverIndex && hoverClientX < hoverMiddleX ) {
      return;
    }

    if ( dragLeft && dragIndex > hoverIndex && hoverClientX > hoverMiddleX ) {
      return;
    }

    if ( dragUp && dragIndex > hoverIndex && hoverClientY > hoverMiddleY ) {
      return;
    }

    if ( dragDown && dragIndex < hoverIndex && hoverClientY < hoverMiddleY ) {
      return;
    }

@Av1shay,
thanks for your code, can you maybe explain where you get the dragBoundingRect  from?
Cheers

I've managed to make it work as expected, inspired by @OliverJAsh's commit:
// get hovered image vertical middle position
const hoverMiddleY = (hoverBoundingRect.bottom - hoverBoundingRect.top) / 2;

// get hovered image horizontal middle position
const hoverMiddleX = (hoverBoundingRect.right - hoverBoundingRect.left) / 2;

// determine mouse position
const clientOffset = monitor.getClientOffset();

// get pixels to the top
const hoverClientY = clientOffset.y - hoverBoundingRect.top;

// get pixels to the left
const hoverClientX = clientOffset.x - hoverBoundingRect.left;

// only perform the move when the mouse has crossed half of the items height/width
// are we dragging right or left?
const dragRight = dragIndex === (hoverIndex - 1);
const dragLeft = dragIndex === (hoverIndex + 1);

// are we dragging up or down?
const dragUp = dragBoundingRect.top > hoverBoundingRect.top;
const dragDown = dragBoundingRect.bottom < hoverBoundingRect.bottom;

if ( dragRight && dragIndex < hoverIndex && hoverClientX < hoverMiddleX ) {
  return;
}

if ( dragLeft && dragIndex > hoverIndex && hoverClientX > hoverMiddleX ) {
  return;
}

if ( dragUp && dragIndex > hoverIndex && hoverClientY > hoverMiddleY ) {
  return;
}

if ( dragDown && dragIndex < hoverIndex && hoverClientY < hoverMiddleY ) {
  return;
}

This solution helped me a lot. You can drag and drop any direction without any hassle.
Also, to get dragBoundingRect directly in hover method without accessing the DOM you can return it from beginDrag method.
beginDrag(props, monitor, component) {
    // `getNode` comes from `useImperativeHandle` suggested by ReactDnD
    const node = component.getNode();
    const dragBoundingRect = node.getBoundingClientRect();

    return {
        // ....
        dragBoundingRect
    };
  }

Then easily retrieve it from monitor.getItem()

I've managed to make it work as expected, inspired by @OliverJAsh's commit:
// get hovered image vertical middle position
const hoverMiddleY = (hoverBoundingRect.bottom - hoverBoundingRect.top) / 2;

// get hovered image horizontal middle position
const hoverMiddleX = (hoverBoundingRect.right - hoverBoundingRect.left) / 2;

// determine mouse position
const clientOffset = monitor.getClientOffset();

// get pixels to the top
const hoverClientY = clientOffset.y - hoverBoundingRect.top;

// get pixels to the left
const hoverClientX = clientOffset.x - hoverBoundingRect.left;

// only perform the move when the mouse has crossed half of the items height/width
// are we dragging right or left?
const dragRight = dragIndex === (hoverIndex - 1);
const dragLeft = dragIndex === (hoverIndex + 1);

// are we dragging up or down?
const dragUp = dragBoundingRect.top > hoverBoundingRect.top;
const dragDown = dragBoundingRect.bottom < hoverBoundingRect.bottom;

if ( dragRight && dragIndex < hoverIndex && hoverClientX < hoverMiddleX ) {
  return;
}

if ( dragLeft && dragIndex > hoverIndex && hoverClientX > hoverMiddleX ) {
  return;
}

if ( dragUp && dragIndex > hoverIndex && hoverClientY > hoverMiddleY ) {
  return;
}

if ( dragDown && dragIndex < hoverIndex && hoverClientY < hoverMiddleY ) {
  return;
}

where can I find dragBoundingRect?


Add Rest of UI for Workspace Chats

@amyevans, please review when you get the chance,
CC: @marcaaron, @roryabraham
Details
Using correct subtitle for PolicyExpenseChats (a.k.a Worksapce Chats) and adding welcome text for PolicyExpenseChats as well.
Held on https://github.com/Expensify/App/pull/7852
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/195472
Tests
Same as Web QA but done locally

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

PR Review Checklist

Contributor (PR Author) Checklist

[x] I made sure to pull main before submitting my PR for review
[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I clearly indicated the environment tests should be run in (Staging vs Production)

[x] I wrote testing steps that cover success & fail scenarios (if applicable)
[ ] I ran the tests & they passed on all platforms
[ ] I included screenshots or videos for tests on all platforms
[x] I verified there are no console errors related to changes in this PR
[x] I followed proper code patterns (see Reviewing the code)

[ ] I added comments when the code was not self explanatory
[ ] I put all copy / text shown in the product in all src/languages/* files (if applicable)
[ ] I followed proper naming convention for platform-specific files (if applicable)
[ ] I followed style guidelines (in Styling.md) for all style edits I made

[x] I followed the guidelines as stated in the Review Guidelines

PR Reviewer Checklist

[ ] I verified the Author pulled main before submitting the PR
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the testing environment is mentioned in the test steps

[ ] I verified testing steps cover success & fail scenarios (if applicable)
[ ] I verified tests pass on all platforms & I tested again on all platforms
[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified there are no console errors related to changes in this PR
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified comments were added when the code was not self explanatory
[ ] I verified any copy / text shown in the product was added in all src/languages/* files (if applicable)
[ ] I verified proper naming convention for platform-specific files was followed (if applicable)
[ ] I verified style guidelines were followed

[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines

QA Steps
Perform this QA on staging

Log into an account that doesn't have a workspace yet.

Create a workspace.

Search for your workspace chat (it should have the same name as your workspace and it will have the same icon as the workspace)

Verify that you see the correct subtitle like this:

Check that the welcome text in the chat looks like this:

Click on the header and verify you see the details page like this (also be sure to click on settings to make sure it works)

Invite another user to this workspace and look for their workspace chat (It will have their name/email as the title). Verify the header looks like this:

Verify that the welcome text looks like this:

Click on the header and verify that the report details page looks like this (and click on settings to see that it works)

[ ] Verify that no errors appear in the JS console

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[ ] iOS
[ ] Android

This is ready for review, but I'm going to try and test the dang mobile platforms today if I can get the emulator working.

@amyevans @marcaaron Thanks for reviewing! I addressed/commented on your suggestions, so please give this another look when you get the chance!

🚀 Deployed to staging by @amyevans in version: 1.1.43-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.1.43-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Update to dart-sass?

With the deprecation of LibSass / node-sass https://sass-lang.com/blog/libsass-is-deprecated , do you intend to change the peer dependency to sass?
Both have the same JS Api.  https://sass-lang.com/documentation/js-api
And they seem to work fine with this importer, however you get a terminal warning about unmet peer dependencies.  Honestly not sure if there's a way to make it an either/or peer dependency.  OR perhaps a rebrand of this library is coming down the line.

I've been using this with dart-sass regularly and it seems to be working fine, apart from the warning. Worth splitting into a node-sass release and a dart-sass release?
Or, alternatively, what about not having any peer dependencies defined, and then include a check somewhere to only warn if the user has no version of Sass installed?


Dont Know How to use FetchMore Function for Posts with codegen

I have AllPosts query ,with Generated GQLFetchMoreQueryAllPosts function , I Get an UnImplemented Error: Please provide a parser function to support result parsing.
Without codegen i use like below
    fetchMore!(
        FetchMoreOptions(
           variables: {'endCursor': endCursor},
           updateQuery: (prev, moreRes) {
           final posts = [
              ...prev!['posts']['nodes']
                as List<dynamic>,
              ...moreRes!['posts']['nodes']
                as List<dynamic>
           ];
           moreRes['posts']['nodes'] = posts;
           return moreRes;
           },
        ),
      ),

Which version of graphql_codegen are you using? Are you calling parsedData on a query result anywhere in your code?

I'm Using the graphql_codegen: ^0.6.0 &  graphql_flutter: '^5.0.1-beta.5'. Yeah I use the result.parsedData & show them in ListView

Thanks for reporting this @tsrCodes! I've found the bug in graphql_flutter and submitted a PR: https://github.com/zino-hofmann/graphql-flutter/pull/1044

@tsrCodes, merged and released in the latest version of graphql: 5.0.2-beta.3

Kudos @budde377  , Works Well , But Missing Type Defs ,

@tsrCodes

But Missing Type Defs

What do you mean?

                                    fetchMore!(
                                      GQLFetchMoreOptionsQueryAllPosts(
                                        updateQuery: (Map<String, dynamic>?
                                                previousResultData,
                                            Map<String, dynamic>?
                                                fetchMoreResultData) {
                                          final posts = [
                                            ...previousResultData!['posts']
                                                ['nodes'],
                                            ...fetchMoreResultData!['posts']
                                                ['nodes'],
                                          ];
                                          fetchMoreResultData['posts']['nodes'] =
                                              posts;
                                          return fetchMoreResultData;
                                        },
                                        variables: VariablesQueryAllPosts(
                                          endCursor: endCursor,
                                          first: 3,
                                        ),
                                      ),

Is that right way??

@tsrCodes, looks about right! 😄


fix(material/datepicker): date range input not marked as required when using reactive forms

Fixes that the date range input wasn't styled as being required when using the required reactive forms validator.
Fixes #26654.

This seems to work for 'fill' form-fields, but not 'outline'


Array support for Data

would be nice to have
impl<T: Data, const Size: usize> Data for [T; Size]

or the equivalent macro for pre-const generics, like
macro_rules! impl_Data {
    () => {},
    ($this:expr $($rest:tt)*) => {
        impl<T: Data> Data for [T; $this] {
            fn same(&self, other: &Self) -> bool {
                self.iter().zip(other.iter()).all(|(a, b)| a.same(b))
            }
        }
        impl_Data($(rest)*);
    }
}

impl_Data { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 32, 64, 128 } // etc

The macro probably has errors in it but that's the gist.

would happily accept this as a PR for arrays up to some size (for now), say 32?

Fixed in #888


#2726 (Part1) Drop crate lazy_static, prefer std::sync::LazyLock

MSRV change
rust-version is now 1.80
This is just the smallest change possible the framework..
smallest patch easiest  to review. I want to do this as the implications of the MSRV change need carefull consideration.
There are lots of  bulky changes to the examples/project subdirectories which will need careful testing in themselves .. but that is part 2.

Looking at the errors ( indentation errors).
error: doc list item without indentation
   --> leptos_reactive/src/signal_wrappers_read.rs:892:5
    |
892 | ///   without reactively tracking it.

These errors are  being fixed here .. https://github.com/leptos-rs/leptos/pull/2728
So guess this PR should be delayed until this lands

@martinfrances107 If you want to patch this in the 0.7 branch too, that'd be appreciated :)

Closing this as 0.7 is now the main branch.


Doc: example app - client UI

Thanks for the work on eventually.  I'm trying to come to grips with the moving parts and encountered some trouble in getting a client to test the server setup I thought the following might be a useful addition to the documentation (README.md?)
To setup a Client

Go to 'examples/bank-accounting'
Execute cargo run
Start another shell, execute podman run -it --rm --network="host" -v $(pwd)/proto:/protos fullstorydev/grpcui -plaintext localhost:10437
See the gRPCui web page at http://<IP_ADDRESS>:8080/
How to test the functionality via this client....

Actually this is an error in starting gRPCui, but I wonder if this setup description is suitable as a Documentation PR to help fellow noobs?

Hey @taqtiqa-mark, thank you for reporting this 🙇🏻♂
I think more documentation is always better. We can definitely add more information in a relevant README.md file 👍🏻
I cannot spot any difference from the two commands you posted in your comments. Is it the same?

-insecure vs -plaintext


Float is not a double

I'm using typhoon for binary serialization and it's important for me to distinguish float from double as they have different binary layouts (4 bytes vs 8 bytes). This code treats them as one type. Could you please introduce types::double?

Hi! Thank you for the issue. I know that float is not a double. The problem is that in PHP it is considered the same. PHPStan also uses double as an alias for float (see https://github.com/phpstan/phpstan/issues/214).
We might consider introducing types::double and TypeVisitor::double() as first-class citizens, but that would require some understanding of how we will support it in other tools in future.
For now I would suggest creating a class Double (float $value) {} and then using types::object(Double::class). You can even have your own collection of binary enum types:
enum binaryTypes implements Type
{
    case int8;
    case int16;
    case int32;
    case int64;
    case double;

    public function accept(TypeVisitor $visitor): mixed
    {
        return match ($this) {
            self::int8 => $visitor->namedObject($this, Id::class(Int8::class), []),
            self::int16 => $visitor->namedObject($this, Id::class(Int16::class), []),
            self::int32 => $visitor->namedObject($this, Id::class(Int32::class), []),
            self::int64 => $visitor->namedObject($this, Id::class(Int64::class), []),
            self::double => $visitor->namedObject($this, Id::class(Double::class), []),
        };
    }
}

WDYT?

Done in https://github.com/typhoon-php/typhoon/commit/fe25e895dfdaa7b8884ebb9c927d7a16c8790880, https://github.com/typhoon-php/typhoon/commit/456058ebda3ca3ff683529245c27de908a782fba. See https://github.com/typhoon-php/typhoon/blob/0.4.x/docs/implementing_custom_types.md for docs.


[BUG] Target Reticules are black instead of orange

Module Version: v2.0.2
Not sure why, but when Border Control is active the target reticules are black instead of orange.

Should be Fixed with 2.1.1


Updated repositories.xml DOC file for reference section with examples of @Query

Included some examples of how adding @Query to a repository method will
work with the parameters passed into it.

Thanks a lot!


Implement rotate left and right

Just a simple implementation of rotate operations using what appears to be the most recommended method these days. It should hopefully satisfy the request in https://github.com/dcodeIO/long.js/issues/45.

It's a fair point. I'll take a look at it.

Perhaps something more along these lines?
I think I got the logic right. (Been on vacation, so my brain might be running in a lower gear.) I certainly added a couple more test cases.
Debated moving some of the bit count calcs to local vars, but wasn't sure if there would be a preference one way or the other.

@dcodeIO When you get a moment, I think I'm ready for more input. Thanks.

Thanks, looks good! Merging, and if there are still issues we can continue from here.

Hi @dcodeIO, are you going to publish a new version containing this commit?


Fix to is_eos_token variable

Hi Phil,
is_eos_tokens is not defined. Quick fix to the variable name in autoregressive_wrapper.py.
Thank you,
Enrico

🙏💯


[4.0] [StdlibUnittest] Avoid serializing references to ObjC runtime APIs

Explanation: Avoid crashes when mixing 3.2 and 4.0 modes in the Swift test suite under certain build configurations.
Scope: Affects our private StdlibUnittest only.
Radar: (none)
Reviewed by: (pending)
Risk: None. We don't ship this library or include it in downloadable toolchains.
Testing: It fixed the issue on master.

@swift-ci Please test

@swiftix or @moiseev, can you "review" this (i.e. confirm it will not affect end users)?

LGTM. This won't have any effect on end users, but will help our testing run smoother


Asset bundles behind load balancer with CDN

This seems to be a vey lightly touched scenario,however, I am dealing with multiple Yii2 servers behind a load balancer topped off with a CDN which caches the contents of each node.
I am finding that I am actually getting problems due to how the asset manager uses mtime to name asset folders, this is causing the compressed assets (css and js) on each server to not match due to having different text in them and so errors occuring on caching of some servers at times causing degraded user experience.
I cannot control the CDN to cache over a specific connection, it just calls up the static file straight from the load balancer when it is requested.
After quite a bit of searching around this scenario seems to not really be thought about when developing the assets. The closest I have found is some having made a version based asset manager that you need to manage manually.
What has been decided on this subject? Should I implement my own asset manager which most likely stores a common key between the servers based upon the first server in the group or has there been some thinking of supporting this within Yii2 core?

Are you referring to asset console command? It uses md5 hash, not modification time.

Using Amazon Cloud you should consider uploading compressed CSS and JS files to the single CDN server instead of relying on EC2 instances for this.
Then you should modify the compression target assets to use CDN URL as baseUrl.

No the asset console command is related but what I am taking about is the:
/assets/456gfd/

folder path that actually goes into the css and js files

@klimov-paul  Ok Kool thanks. Of course the system I use is slightly different.
It essentially caches my data by request.
So for example I have a file in my CSS of:
../assets/45/glyphicons.woff

and when I call:
cdn.site.com/assets/45/glyphicons.woff

it actually caches the first url.
It is not a CDN I have control over as such.

The only good way is usage of the central server like Amazon S3 for the asset files storage along with the corresponding modifications at asset bundles configuration.
I am unsure if we can make something to resolve problems related to load balancer usage.

Indeed, it seems I will have to make this relatively customised. Any kind of work around is probably too custom for Yii2 core really

Since this comes very high on Google search result, if not first, for this problem I will describe how I ended up using Yii here.
The way I did it was to actually have the load balancer/a control server prepare the Yii site by doing the asset compression and init commands etc and then just copy the files onto each of the app servers.
This meant that as my CDN, which is request based, went to find a file behind the load balancer it wuld alwasy get the same file over and over and not a file uniquely identified for that app sever under the load balancer.

Could be related: http://yii2-cookbook.readthedocs.org/structure-asset-processing-with-grunt/

Indeed similar though it doesn't propogate to multiple app servers, it is mostly a deploy script for a singular server. I used phing to solve to sovle my problem

@Sammaye it depends on how you configure it. Example if for a single server but you could do virtually anything in such way.

Relates #9245


A New Kind of Science by Stephen Wolfram

(only online reading)
http://www.wolframscience.com/nks/

already exists, this issue should be closed


fix(travis): add 5.0.0 release to matrix

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ Bofrost.lu has degraded performance

In 88e7a43, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1442 ms

Resolved: Bofrost.lu performance has improved in bbdad61 after 10 minutes.


[dv/cip] Fix and document mubi randomization

The weights of true, false, and others should treat others as a group and be uniform.
The current scheme is flawed since it under-samples true and false.

Running 1000 randomizations with the current code and weights False=4, True=2, Others=2 gives the following per value frequency:
mubi 00000000 count 0.016638
mubi 00000001 count 0.016647
mubi 00000002 count 0.016696
mubi 00000003 count 0.016588
mubi 00000004 count 0.016559
mubi 00000005 count 0.499670
mubi 00000006 count 0.020759
mubi 00000007 count 0.021234
mubi 00000008 count 0.020884
mubi 00000009 count 0.020863
mubi 0000000a count 0.250557
mubi 0000000b count 0.016607
mubi 0000000c count 0.016592
mubi 0000000d count 0.016563
mubi 0000000e count 0.016489
mubi 0000000f count 0.016654
Notice the values between False and True get a higher frequency than those above True and below False. This is because there are 5 for each of the latter, and only 4 of the former.

Sri, I found xcelium was failing and perhaps this is why it was reopened? I have a fix for it.

Yes, thats right. IMO the distribution will need to be an autogenerated piece of code.

@matutem @sriyerg has this been fixed in the meantime?

I am not aware of pending issues, and I just run an xcelium test that exercises the randomization code.
Closing this issue again, but reopen if I missed something.


Intermediate Certificates not installed

I don't believe this client installs the intermediate certificate, which means there are problems connecting to sites through certain security appliances and firewalls.

That matches my experience on Windows 2012 R2. I haven't found specific guidance related to this so I'll add it here.
I manually installed the Let’s Encrypt Authority X3 (IdenTrust cross-signed) certificate from https://letsencrypt.org/certificates/ to the Local Computer's Intermediate Certification Authorities store.
I installed this certificate using the .der file:

At this location:

This can be manually fixed by concatenating the intermediate certificate onto your own certificate. letsencrypt's certificates can be found here: https://letsencrypt.org/certificates/
basically just cat your-domain.com-crt.pem lets-encrypt-x3-cross-signed.pem >> chained-crt.pem and then use that certificate file instead. It destroys the beauty of having a fully automated solution though.

I think manually installing the intermediate certificate first (or after) is an easier solution though?

I assumed installing the intermediate certificate happened on the client's end, and therefore would have to be done for every client, which makes no sense. Was the certificate installed on the server?

Yes, on a new installation, the intermediate certificates are missing from the certificate store on the server. Once installed on the server, they then get sent to the client OK.

That sounds more robust indeed, thanks.


Wrong getTextWidth return with chinese Font

I am using getTextWidth to center and position my Text correctly in a screen. One is an outline and the other a normal Text.
My project switches font between latin and chinese.
My example uses NotoSansSC_400Regular. It looks like it's able to calculate the normal Text fine but has issues with the outline Text together with the Chinese font. No issues with the latin one.

Thanks for reporting this. Can you check if getGlyphWidths returns the correct result? It might be a bug in our getTextWidth() implementation. We will try to add a test case for it.

@wcandillon the above text is: 欢迎

Thks a lot! I added a text case at #916. Let me know if I am missing something.

@hirbod let me know if you would like to add more logic to the test case to make sure we support the example you are showing above.

@wcandillon my issue has been the stroke width only text example. Normal text has been calculated correctly, in my screenshot you can see that it's just the first word that has totally wrong dimensions and this is misaligned while it should be centered like the filled word. No issue with the Latin character.

can you share a small code snippet I could add to #916? I'm not sure what we should test for.

Sure, will do tomorrow :)

got curious, @wcandillon this should help
import * as skia from "@shopify/react-native-skia";

interface DualTextProps {
  strokeText: string;
  fillText: string;
  fontSize: number;
  y: number;
}

const DualText = ({ strokeText, fillText, fontSize, y }: DualTextProps) => {
  const font = skia.useFont(require("../assets/NotoSansSC-Regular.otf"), fontSize);

  return font !== null ? (
    <skia.Group>
      <skia.Text
        style={"stroke"}
        color={"black"}
        x={0}
        y={y + fontSize}
        font={font}
        strokeWidth={1}
        text={strokeText}
      />
      <skia.Text
        style={"fill"}
        color={"black"}
        x={font.getTextWidth(strokeText)}
        y={y + fontSize}
        font={font}
        text={fillText}
      />
    </skia.Group>
  ) : (
    <></>
  );
};

export const Root = () => {
  return (
    <skia.Canvas
      style={{
        width: 1000,
        height: 200,
      }}
    >
      <DualText strokeText={"THE"} fillText={"NOSE"} fontSize={40} y={0} />
      <DualText strokeText={"欢迎"} fillText={"欢迎"} fontSize={40} y={100} />
    </skia.Canvas>
  );
};

Thanks a lot, I definitely see the bug now :)


[bug 1139878] Change the size of textarea in reports description.

@johngian r?

LGTM r+

@johngian I also added a pictogram to the initiatives. I chose the medal so that we can have the same pictograms in both events and reports for these fields (the events pictograms are added in pr #948)

Pictogram looks good. r+

Coverage decreased (-0.01%) to 91.77% when pulling 4584a31563b2ba9e265f904c0cdc401ded48bdb3 on akatsoulas:1139878-reports into 14d654a772abc53549acbd0b63f76ece482c269b on mozilla:master.

Coverage remained the same at 91.78% when pulling 62d5474732d90e349d05a2602551eccaf0ece180 on akatsoulas:1139878-reports into 14d654a772abc53549acbd0b63f76ece482c269b on mozilla:master.


Datalabeling : Added system test

towards #5941

@chingor13 Build failed due to Datalabeling service is not enable or activate for this project.

@athakor Enabled, but can you reopen this PR on https://github.com/googleapis/java-datalabeling?


HHH-17017 Hibernate 6 cannot read VARCHAR column as INT

The test shows that using @JdbcTypeCode(SqlTypes.VARCHAR) fixes the issue;

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

https://hibernate.atlassian.net/browse/HHH-17017

I guess the source of user confusion here is that usually IntegerJavaType will automatically do conversion from String to Integer. EnumJavaType doesn't do such a conversion, it assumes Strings are names, I think because EnumJavaType doesn't actually properly know whether it is an ORDINAL or STRING enum mapping.
I agree with you guys that this is a pretty minor discomfort because the workaround is so easy.

I was wrong, as commented on the Jira, when using @JdbcTypeCode(SqlTypes.VARCHAR) the string value is persisted.
In the Jira I proposed other two solutions.
In previous version it worked because it seems  MySQL has no issues using an IntegerJdbcType even when the db column is a varchar, while it has issue when using a TinyIntJdbcType

Ah OK. I think this is slightly less than trivia to fix because as I said:

EnumJavaType doesn't actually properly know whether it is an ORDINAL or STRING enum mapping.

But I have not dug deep and so maybe that's wrong.

yes, probably we would need an OrdinalEnumType and a StringEnumType, anyway I think that persisting  an oridnal enum into a varchar column is really an edge case

I agree.


Can't use pod to get SpreadsheetView

When I add pod 'SpreadsheetView' into my Podfile and run pod install, it show me "[!] Unable to find a specification for SpreadsheetView"
And I run pod search, it also couldn't find it.
Please check this, thx.

@Smiacter You should run pod repo update.

Ok, I'll try it, thx

Closing. Seems to be resolved.

OK, update pod fixed my problem. THX.

Hi, I am still having this issue. Tried, pod repo update but still getting this error[!] Unable to find a specification for SpreadsheetView

@nipun0505 There is a problem with your environment. Needs more info to investigate what is wrong.

Thanks for the quick reply. I am using cocoapods 0.39.0. Could that be an issue?
 Is there any reason not to update?

source "https://github.com/CocoaPods/Old-Specs" seems a cause of the issue.


can't compile rust-lua on Windows

I ran into this problem while trying to reproduce issue #72 on Windows 7 64-bit.
I'm using the latest build:
rustc 1.12.1 (d4f39402a 2016-10-19)
cargo 0.13.0-nightly (109cb7c 2016-08-19)

I get the following error output:
   Compiling lua v0.0.9
error: failed to run custom build command for `lua v0.0.9`
process didn't exit successfully: `C:\Source\lua-embed-test\luahost\target\debug\build\lua-0442ce095437a50b\build-script-build` (exit code: 101)
--- stdout
OPT_LEVEL = Some("0")
PROFILE = Some("debug")
TARGET = Some("x86_64-pc-windows-gnu")
debug=true opt-level=0
HOST = Some("x86_64-pc-windows-gnu")
TARGET = Some("x86_64-pc-windows-gnu")
TARGET = Some("x86_64-pc-windows-gnu")
HOST = Some("x86_64-pc-windows-gnu")
CC_x86_64-pc-windows-gnu = None
CC_x86_64_pc_windows_gnu = None
HOST_CC = None
CC = None
TARGET = Some("x86_64-pc-windows-gnu")
HOST = Some("x86_64-pc-windows-gnu")
CFLAGS_x86_64-pc-windows-gnu = None
CFLAGS_x86_64_pc_windows_gnu = None
HOST_CFLAGS = None
CFLAGS = None

--- stderr
thread 'main' panicked at 'Error: The system cannot find the file specified. (os error 2)', C:\Users\GK\.cargo\registry\src\github.com-1ecc6299db9ec823\lua-0.0.9\build.rs:145

The buildscript currently depends on make and a C compiler (gcc for windows-gnu). Do you have both of these available?

I've got cygwin on that machine with both make and gcc, will it work if I run cargo from a cygwin shell?  Or will the cygwin environment confuse it further?
Also, is there a way to get better diagnostics on a failure?

Cygwin should work, or at least is worth a try; I use MSYS2 which is similar to cygwin in many ways and things work fine.
It would be possible to modify build.rs to be a bit more specific about where the failure is. It would also be possible to remove the dependency on make, but it's a bit trickier.

Cygwin gcc started spewing errors about undefined reference to __errno, __getreent, and what-have-you.
I installed MSYS2 with gcc 5.3.0 and GNU make 4.2.1.  I get a similar panic to the above, but this time the error is The directory name is invalid. (os error 267)
I'll try playing with build.rs this evening and see if I can figure out what's breaking.  (I won't have access to a Windows machine over the weekend though.)

Make sure you're running MSYS2 in the MINGW64 mode with mingw-w64-x86_64-gcc and mingw-w64-x86_64-make packages installed, not in the base MSYS2 mode with the bare gcc and make packages. Otherwise I expect things will not work as well. If you need it, I wrote a guide for a different Rust project, but it should still be mostly applicable (barring some out-of-date stuff).

I was using the ordinary packages, yes.  Thanks for the guide, I'll try on Monday and see if it works.

@SpaceManiac tried installing MSYS2 as per your guide, and then compiling rust-lua53
It says the build failed in build.rs:176 but with some println! debugging I found that it doesen't print anything past line 151.
This is on a fresh x86_64 install of MSYS2 with the only package installed being mingw-w64-x86_64-toolchain
$ RUST_BACKTRACE=1 cargo build
   Compiling lua v0.0.11 (file:///C:/Users/afonso/git/rust-lua53)
error: failed to run custom build command for `lua v0.0.11 (file:///C:/Users/afonso/git/rust-lua53)`
process didn't exit successfully: `C:\Users\afonso\git\rust-lua53\target\debug\build\lua-418791497191e787\build-script-build` (exit code: 101)
--- stdout
cargo:rustc-link-lib=static=lua
OPT_LEVEL = Some("0")
TARGET = Some("x86_64-pc-windows-gnu")
HOST = Some("x86_64-pc-windows-gnu")
TARGET = Some("x86_64-pc-windows-gnu")
TARGET = Some("x86_64-pc-windows-gnu")
HOST = Some("x86_64-pc-windows-gnu")
CC_x86_64-pc-windows-gnu = None
CC_x86_64_pc_windows_gnu = None
HOST_CC = None
CC = None
TARGET = Some("x86_64-pc-windows-gnu")
HOST = Some("x86_64-pc-windows-gnu")
CFLAGS_x86_64-pc-windows-gnu = None
CFLAGS_x86_64_pc_windows_gnu = None
HOST_CFLAGS = None
CFLAGS = None
PROFILE = Some("debug")

--- stderr
thread 'main' panicked at 'Error: The system cannot find the file specified. (os error 2)', build.rs:176
stack backtrace:
   0:           0x48cd9a - std::sys::imp::backtrace::_write::h9de14d9391a68d69
   1:           0x497437 - std::panicking::default_hook::{{closure}}::h35d735e0666e8a8c
   2:           0x496ff3 - std::panicking::default_hook::h0e6a0c6e63807aa3
   3:           0x4979be - std::panicking::rust_panic_with_hook::h6039b94a430bd351
   4:           0x497866 - std::panicking::begin_panic::he7a6c483d1b92bfe
   5:           0x497789 - std::panicking::begin_panic_fmt::h0eab1e13d28f1802
   6:           0x4129dd - build_script_build::main::hcf949c5bd932e661
                        at C:\Users\afonso\git\rust-lua53/build.rs:176
   7:           0x497648 - std::panicking::try::do_call::he7c18ae3ebb73a17
   8:           0x4a1658 - _rust_maybe_catch_panic
   9:           0x498163 - std::rt::lang_start::h27502d1630e59612
  10:           0x412aaa - main
  11:           0x4013b4 - _tmainCRTStartup
  12:           0x4014e7 - mainCRTStartup
  13:     0x7ffccd278363 - unit_addrs_search

@afonso360 I believe you are missing the make package. If that's not enough, here's full steps on a clean system:

Download and install msys2-x86_64-20161025.exe from website
Open "MSYS2 MinGW 64-bit" shell

pacman -Syu to update core
When prompted, close the shell window

Open "MSYS2 MinGW 64-bit" shell

pacman -Syu to update packages
pacman -S git make mingw-w64-x86_64-gcc
wget https://static.rust-lang.org/rustup/dist/x86_64-pc-windows-gnu/rustup-init.exe
./rustup-init --default-host x86_64-pc-windows-gnu --default-toolchain stable -y
export PATH=$USERPROFILE/.cargo/bin:$PATH
git clone https://github.com/jcmoyer/rust-lua53
cd rust-lua53
cargo test

@SpaceManiac Thanks it worked! Managed to compile it both as a crate and as a dependency to other projects, and with the latest commit I can compile it on MSVC

Is there any reason that gcc crate can't be used instead? It does all the heavy-lifting of finding msvc and works well for the msvc targets that are common on win32.
I've used it extensively on netcode.io and it works with with both win32 and linux.
I'm hitting the same error and not really looking to install a full cygwin toolchain to use this crate.

@SpaceManiac You shouldn't need to install Rust inside the MSYS environment. Rustup can install the x86_64-windows-pc-gnu target, and you just need to add the /bin/ folder for the MSYS environment to your PATH. Likewise, you could completely skip MSYS and just put MinGW onto your PATH.

In my experience, MSYS2 is the best way to install MinGW and keep it up to date, install C dependencies like libcurl, openssl, libsodium, and others that regularly come by when working with some Rust libraries, and to keep git up to date. This is why I recommend its use.

I'm not saying it's not a useful tool. It's just that you don't need to isolate your Rust installation from the rest of your PC's software ecosystem, making it much more fragile.

Running rustup-init within MSYS2 doesn't actually install it into MSYS2. It still goes to %USERPROFILE%/.rustup and is perfectly accessible from the rest of the system.

Does it? In that case, I retract my statement.


RSS Constellations: File collision

File           : GameData/RealSolarSystem/RSSKopernicus/Sun.cfg
Installing Mod : RSSExtrasolar 1.0
Owning Mod     : RealSolarSystem
CKAN Version   : v1.22.1-0-g174271a (beta)
Real Solar System 11.4.0 is installed.
RSS Extrasolar 1.0 won't install because it is trying to overwrite this file.
Is there a workaround?

The only workaround I can think is to manually delete the Sun.cfg file before installing RSS Extrasolar, but you'll have to mess with it every time you upgrade, unfortunately. Is the RSSExtrasolar version of the file substantially different to the RealSolarSystem version? If it's the same, we could filter that file out, since RealSolarSystem is a dependency for RSSExtrasolar.

What version of KSP are you running? Given that RSSExtrasolar 1.0 is for KSP 1.1.3, I don't think you'll have any worries about upgrades until you move to KSP 1.2.2, for which the whole RSS/Extrasolar installation is different.


bump dcos-cni

High-level description
This is backport of #2492
I have requested more information from @asridharan  on the late introduction of this feature. This may or may not be merged with 1.11.0 GA

@mesosphere-mergebot bump-ee

@orsenthil  this PR fixes : https://jira.mesosphere.com/browse/DCOS_OSS-2171
We discovered a issue with DNS in K8s during the bug hunt. The fix for that bug was done as part of : https://jira.mesosphere.com/browse/KUB-145
A side-effect of KUB-145 was that K8s wouldn't function on networks such as Big-switch network (an SDN provider) . This bump in the DC/OS CNI plugins introduces a fix to mitigate the side-affect of KUB-145.

@orsenthil thanks for creating this back-port.
@GoelDeepak need your reviews on this.

@mesosphere-mergebot label ready for review

@mesosphere-mergebot override-status teamcity/dcos/test/aws/onprem/static  https://jira.mesosphere.com/browse/DCOS_OSS-2115
@asridharan  / @urbanserj  / @jeschkies  - Can you help with the above^.

Also, please approve the EE PRs too.

@mesosphere-mergebot override-status teamcity/dcos/test/aws/onprem/static https://jira.mesosphere.com/browse/DCOS_OSS-2115


Add "Target app" to key remapping as well.

Description of the new feature / enhancement
I love the keyboard manager feature, helps me a lot with ms word for making some characters easier to type using the key remapping feature. But i wish it only works in a certain app(s) i choose, since it can be inconvenient to have to turn it on/off every time, so that would be a great addition
Scenario when this would be used?
For when you want to use key remapping for a certain app(s) without having to turn it on and off every time you need it.
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/8809


retry on packages and repositories failures

add register/until on all packaging related tasks to avoid non valid CI
failure.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test luminous-centos7_cluster

jenkins test dev-centos7_cluster

jenkins test pipeline


mkcloud: Remove cloud4 support

Since Cloud 5 is EOL we can drop the support for Cloud 4.

Looks good. Maybe we should verify with a parallel gating run that Cloud5 still works.


Generating docs for libs in NX Monorepo

Search terms
NX, NRWL, Libraries, Monorepo
Question
Hi there. I am trying to create a documentation for 2 Libs inside our NX Monorepo. Here´s how I tried it:

Set typedoc.json to entryPointStrategy "packages"
Pointed the entryPoints to the respective folders "libs/store" and "libs/model"
Inside the respective package.json, I added the entry for "main" to "./src/index.ts"
Added "typedoc": {      "entryPoint": "./src/index.ts",      "readmeFile": "./README.md",      "displayName": "XXX"  } like described in the docs to the libs package.jsons

When I try to create the doc, I only receive the error
Error: Entry point "C:\XXX\libs\model\src\index.ts" does not appear to be built by the tsconfig found at "C:/XXX/libs/model/tsconfig.json"
Has anyone an idea what I am missing?

That error means that libs/model/tsconfig.json specifies the include (or files) option such that the libs/model/src/index.ts file is not included in the program. If the file is included, this is probably a bug and I'll need a reproduction to try to fix it.

Okay, after I added "srx/index.ts" to the "include" of the respective tsconfig.json´s it is creating the documentation.
However, now I am facing a different problem. I receive the "Warning"
Warning: ModalDTO, defined at libs/model/src/lib/DTO/modal-dto.ts:47, is referenced by @deichmann/store.selectActiveModal but not included in the documentation.  and if i check the generated documentation of the entry 'selectActiveModal' (located in 'libs/store') there is no link to the entry of 'ModalDTO' (located in 'libs/model')
Do you any clue on how to fix this?

Okay, I keep an eye on the other issue. Thanks a lot so far


feat:  Is there any way to send the colors with text ?

Hi @hiteshshimpi-55,
We are planning to support colors as well in couple of releases. You can following this discussion and contribute if you have any ideas around the same.
https://github.com/rohanjsh/typeset/discussions/37#discussioncomment-7947117


Edge - Prevent bypassing Microsoft Defender SmartScreen prompts for sites

Hi,
first thanks for this awesome tool!
Can you please check list "finding_list_msft_security_baseline_edge_98_machine.csv" about "Prevent bypassing Microsoft Defender SmartScreen prompts for sites"
I set this policy through Edge GPO but the HardeningKitty report does not recognized it.
HardeningKitty checks for "HKLM:\Software\Policies\Microsoft\MicrosoftEdge\PhishingFilter,PreventOverride".
This seems for be right for the old edge legacy browser not for edge chromium.
correct should be "HKLM:\Software\Policies\Microsoft\Edge\PreventSmartScreenPromptOverride".
See here --> https://www.thewindowsclub.com/prevent-users-from-bypassing-smartscreen-warning-in-edge
Thanks in advance

You're right. Fixed in commit 738f3e3

Thanks for fixing!

Solved!


Autocompletion from get_node() still does not work for some cases

Tested versions
v4.2.1.stable.official [b09f793f5]
v4.2.1.stable.official [b09f793f5] + #85224
v3.5.3.stable.official [6c814135b]
System information
Ubuntu 20.04 LTS
Issue description
Since https://github.com/godotengine/godot/pull/79386, autocompletion improved, but it still does not work for other get_node() cases.
This is a regression (since GDScript 2.0 as far as I can tell) from Godot 3.5 since all cases (except 4, see NOTES in Steps to reproduce) autocomplete.
These related bugs differ slightly and are not using get_node() directly.
https://github.com/godotengine/godot/issues/74888 (custom function returns)
https://github.com/godotengine/godot/issues/71296 (autoload)
https://github.com/godotengine/godot/issues/78860 (superseded by this new bug, I also thought it's a little unclear)
https://github.com/godotengine/godot/issues/78820 (return from Dictionary)
I also thought it would be nice to have one MRP to exhaust all combinations for get_node() autocompletion since it's a common usecase.
Steps to reproduce

Load the MRP
In the script attached to node A.  Try typing out do_something() and observe similar behavior as noted in the comments.  I've pasted script A.gd below for reference.

extends Control

@onready var b = $B
@onready var b_get_node = get_node("B")
@onready var b_typed := $B
@onready var b_typed_get_node := get_node("B")

@onready var c = $C
@onready var c_get_node = get_node("C")
@onready var c_typed : C = $C
@onready var c_typed_get_node : C = get_node("C")

var d
var d_get_node

var e
var e_get_node
var e_typed : E
var e_typed_get_node : E

@onready var f = %F
@onready var f_get_node = get_node("%F")
@onready var f_typed := %F
@onready var f_typed_get_node := get_node("%F")

@onready var g = %G
@onready var g_get_node = get_node("%G")
@onready var g_typed : G = %G
@onready var g_typed_get_node : G = get_node("%G")

var h
var h_get_node

var i
var i_get_node
var i_typed : I
var i_typed_get_node : I

func _ready():
  # -- B (onready)

  # inline
  $B.do_something() # does not autocomplete
  get_node("B").do_something() # does not autocomplete

  # script vars
  b.do_something() # autocompletes
  b_get_node.do_something() # does not autocomplete
  b_typed.do_something() # autocompletes
  b_typed_get_node.do_something() # does not autocomplete

  # local vars
  var blocal = $B
  var blocal_get_node = get_node("B")
  var blocal_typed := $B
  var blocal_typed_get_node := get_node("B")
  blocal.do_something() # does not autocomplete
  blocal_get_node.do_something() # does not autocomplete
  blocal_typed.do_something() # does not autocomplete
  blocal_typed_get_node.do_something() # does not autocomplete

  # -- C (onready, class_name)

  # inline
  $C.do_something() # does not autocomplete
  get_node("C").do_something() # does not autocomplete

  # script vars
  c.do_something() # autocompletes
  c_get_node.do_something() # does not autocomplete
  c_typed.do_something() # autocompletes
  c_typed_get_node.do_something() # autocompletes

  # local vars
  var clocal = $C
  var clocal_get_node = get_node("C")
  var clocal_typed : C = $C
  var clocal_typed_get_node : C = get_node("C")
  clocal.do_something() # does not autocomplete
  clocal_get_node.do_something() # does not autocomplete
  clocal_typed.do_something() # does not autocomplete in 4.2.1-stable; autocompletes with 4.2.1-stable + #85224
  clocal_typed_get_node.do_something() # does not autocomplete 4.2.1-stable; autocompletes with 4.2.1-stable + #85224

  # -- D

  # script vars
  d = $D
  d.do_something() # does not autocomplete
  d_get_node = get_node("D")
  d_get_node.do_something() # does not autocomplete

  # -- E (class_name)

  # script vars
  e = $E
  e.do_something() # does not autocomplete
  e_get_node = get_node("E")
  e_get_node.do_something() # does not autocomplete
  e_typed = $E
  e_typed.do_something() # does not autocomplete
  e_typed_get_node = get_node("E")
  e_typed_get_node.do_something() # does not autocomplete

  # -- F (onready, unique name)

  # inline
  %F.do_something() # does not autocomplete
  get_node("%F").do_something() # does not autocomplete

  # script vars
  f.do_something() # autocompletes
  f_get_node.do_something() # does not autocomplete
  f_typed.do_something() # autocompletes
  f_typed_get_node.do_something() # does not autocomplete

  # local vars
  var flocal = %F
  var flocal_get_node = get_node("%F")
  var flocal_typed := %F
  var flocal_typed_get_node := get_node("%F")
  flocal.do_something() # does not autocomplete
  flocal_get_node.do_something() # does not autocomplete
  flocal_typed.do_something() # does not autocomplete
  flocal_typed_get_node.do_something() # does not autocomplete

  # -- G (onready, class_name, unique name)

  # inline
  %G.do_something() # does not autocomplete
  get_node("%G").do_something() # does not autocomplete

  # script vars
  g.do_something() # autocompletes
  g_get_node.do_something() # does not autocomplete
  g_typed.do_something() # autocompletes
  g_typed_get_node.do_something() # autocompletes

  # local vars
  var glocal = %G
  var glocal_get_node = get_node("%G")
  var glocal_typed : G = %G
  var glocal_typed_get_node : G = get_node("%G")
  glocal.do_something() # does not autocomplete
  glocal_get_node.do_something() # does not autocomplete
  glocal_typed.do_something() # does not autocomplete in 4.2.1-stable; autocompletes with 4.2.1-stable + #85224
  glocal_typed_get_node.do_something() # does not autocomplete 4.2.1-stable; autocompletes with 4.2.1-stable + #85224

  # -- H (unique_name)

  # script vars
  h = %H
  h.do_something() # does not autocomplete
  h_get_node = get_node("%H")
  h_get_node.do_something() # does not autocomplete

  # -- I (class_name, unique_name)

  # script vars
  i = %I
  i.do_something() # does not autocomplete
  i_get_node = get_node("%I")
  i_get_node.do_something() # does not autocomplete
  i_typed = %I
  i_typed.do_something() # does not autocomplete
  i_typed_get_node = get_node("%I")
  i_typed_get_node.do_something() # does not autocomplete

NOTES

These repro steps were intended to be an exhaustive list of get_node() that does not include custom functions, inlined classes, or returns from other data types (like Dictionaries).  It is likely I missed some cases though.
I ran the same test with https://github.com/godotengine/godot/pull/85224 cherry-picked to 4.2.1-stable.  It improves autocompletion slightly; the differences are noted in the comments.
I ran the same test with v3.5.3.stable.official [6c814135b] (in a 3.5.3 project; I didn't include the MRP), and all applicable cases autocomplete correctly except for these:

  blocal.do_something()
  blocal_get_node.do_something()
  blocal_typed.do_something()
  blocal_typed_get_node.do_something()

Minimal reproduction project (MRP)
getnodebug.zip

So i looked to the cases and splited them up into some categories:

get_node: everything using get_node instead of $/% won't work. This is hard to fix without bigger implementation changes. Furthermore whether this should be supported needs to be discussed first. In comparison to $/% the method may add a lot of edge cases that may feel very buggy if not handled correctly. (Talking about things like calling get_node on a variable. It might be another node, it might be a reference to self, ...)
Inline: Inline usage of $/% does not work. This is a regression from my orginial PR and was fixed in #86111. It will probably get cherry picked for the next release.
local untyped (or type infered): this results from wrong code for determining the assigned value of a local var inside of _get_subscript_type. This could be fixed there but would duplicate code from _get_expression_type so my preferred solution would be to remove _get_subscript_type and integrate it into the later. This however may break quite some stuff so before working on this I wan't to have solid unit tests for autocompletion.
property with local asignment (d): _get_subscript_type does not search for local assignments, _get_expression_type does. Another case were merging them would make sense.
local assignment of typed class property: Basically the same problem as #85224 but with properties instead of local variables

Thank you for the detailed response!
I've edited the original description to include results from #86111.  Looks like it fixes the inline cases as you've stated.

get_node: everything using get_node instead of $/% won't work. This is hard to fix without bigger implementation changes. Furthermore whether this should be supported needs to be discussed first.

Godot 3.5.3 had this working, but I am unaware of whether its implementation produced bugs considering how difficult it is.  This may be a hack, but would it be possible to hardcode these cases?  For example, if the code explicitly parses get_node, implicitly convert it to an inline case and handle that?  Though I'm finding this may not be viable because all cases cannot be converted.  For example, converting get_node("../A") to $"../A" doesn't work since the latter is invalid (it is valid syntax, but it errors at runtime).  Additionally this is similar to preprocessing which is an entirely different discussion.

I updated #85224 to fix case 5.

There goes my "No Godot on holidays plan". But to net let this get out of hand here is my quick response:
The problem you described in your edit was pointed out in this issue already. Recap: it is fixed, the fix is merged but it is not yet part of an official release.
Your original issue is from what I know a design decision. The problem is the following: While you have a specific scene in mind for your scrip, Godot has not. In theory you could attach the script to every node or scene, so Godot can't know which you are referring to. So the decision was made, to assume you are referring to the currently open scene (in many cases this holds true, when you open the script via the scene tree dock, but it becomes a problem if you tend to switch your scripts in the script editor). In theory Godot could check all scenes or all open scenes, but this won't scale well and lead to performance issues down the road.
There are some proposals to provide a tight coupling between a script and a certain scene/node. But I don't know what the status is on that front. Wouldn't expect it to be solved soon, but if it gets in it would provide a way to provide autocompletion beyond the currently open scene.
Let's end this with a quick disclaimer (not specifically targeting you, but more as a copy and pastable thing for all autocompletion issues, since there tends to be a lack of organization on autocompletion issues).

Autocompletion is a complex topic where a lot of internal things can go wrong. Just because the title of this issue is related to the same topic, does not mean that this is the same issue that you encountered. Even if the title sounds like it, this is not a multipurpose thread. The original poster of this issue provided a specific problem (or problems), which are being discussed here. Read the original issue carefully. If it aligns with your problem there is no need to add a comment just leave a emoji reaction to let us know the issue is a common problem. If your problem requires adding code that is not related to the OP, it is most likely worth a new issue. If you are sure that you provide additional info related to the issue: Great, go ahead and help us figure it out!
Let's keep the Github issues well organized together. :smiley:

https://github.com/godotengine/godot/issues/45414#issuecomment-2112010667

The problem is still present in version 4.3 beta1

The problem is still present in version 4.3 beta1

Please reread my first comment and be specific about your issue.

The problem is still present in version 4.3 beta1

Please reread my first comment and be specific about your issue.

#92185

#92185

This is not fixed yet and I wouldn't bet on it getting fixed any time soon. I updated my first comment to reflect which points are still open. Use dollar literals in the meantime.


Avoid passing null

Issue created from code comment with imdone.io

TODO: @Sandy Avoid passing null id:95
src/vs/workbench/services/configuration/node/configuration.ts:458
@imdone - Efficiently manage your project's technical debt. imdone.io

Issue closed by removing a comment.

TODO: @Sandy Avoid passing null id:95 gh:154
src/vs/workbench/services/configuration/node/configuration.ts:458
@imdone - Efficiently manage your project's technical debt. imdone.io


generate iglu-style scenes in cuberite

Description
adds a script to generate iglu-style scenes, and changes cuberite_process.py to more easily read/run them
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[x] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[x] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Testing
ran:
ipython -- droidlet/lowlevel/minecraft/small_scenes_with_shapes.py --NUM_SCENES 1 --save_data_path <path_to_blocks.json>
ipython -- droidlet/lowlevel/minecraft/cuberite_process.py --config really_flat_world --schematic <path_to_blocks.json>
also tried
ipython -- droidlet/lowlevel/minecraft/cuberite_process.py --config really_flat_world --random_shapes
Checklist:

[x] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I have verified that all scripts in tests/scripts runs on hardware.

Codecov Report

Merging #925 (e8f97b4) into main (0c60f2d) will decrease coverage by 0.17%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##             main     #925      +/-   ##
==========================================
- Coverage   32.37%   32.19%   -0.18%
==========================================
  Files         344      344
  Lines       34057    33885     -172
==========================================
- Hits        11025    10909     -116
+ Misses      23032    22976      -56

Impacted Files
Coverage Δ

lowlevel/minecraft/cuberite_process.py
49.13% <0.00%> (-5.87%)
:arrow_down:

agents/_agent.py

lowlevel/minecraft/small_scenes_with_shapes.py
19.69% <0.00%> (ø)

Are all mobs disabled in this IGLU style world (or you want mobs in it)? I tried to generate a world with the above commands and can still see mobs in it.

Also can we blank out all the areas that are not building zone?

ok, removed monsters and made it completely flat


Bonus tests

Hi, yesterday I tried to solve the bonus, but instead of &, in tests it's used #.

Fixed
On Wed, Apr 8, 2020 at 10:30 AM DianaScurtu<EMAIL_ADDRESS>wrote:

Hi, yesterday I tried to solve the bonus, but instead of &, in tests it's
used #.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/UPB-FILS/alf/issues/56, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVU5IN2YYDT2JZ3GDOY4SDRLQR2NANCNFSM4MDWQ6WA
.

--
Alexandru RADOVICI
Universitatea "Politehnica" din Bucuresti
e-mail<EMAIL_ADDRESS>telefon:<PHONE_NUMBER>
www.ipworkshop.ro

I have some questions:
if_then_else_error.alf:
the error is at line 9, in output file at line 10
if_then_error.alf:
the error is at line 7, in output file at line 10
For an error, which I get in main.js, how could I know that it was generated by an instruction like & if_then, to display the line number?
unregister_value.alf:

the last endif is at position 18, in output file at 20
why is function_call called 3 times?
for the first if P = 1, for the second one it goes on the else branch

A lot of line numbers were wrong due to our preprocessing, fixed.

I have some questions:
if_then_else_error.alf:
the error is at line 9, in output file at line 10
if_then_error.alf:
the error is at line 7, in output file at line 10
For an error, which I get in main.js, how could I know that it was generated by an instruction like & if_then, to display the line number?
unregister_value.alf:
* the last endif is at position 18, in output file at 20

* why is function_call called 3 times?

* for the first if P = 1, for the second one it goes on the else branch

It should be called only twice.


Support list of maps of tags in AD annotation *.instances value

What does this PR do?
it allows in annotations ad.datadoghq.com/<container_name>.instances to support several container configs for the same "check_name".
Motivation
Aim is to improve the UX on datadog-agent configuration thanks to annotations.
Additional Notes
none

@xvello,
commit updated with comments fix.

Did you have the chance to test the feature on a testing cluster?
Also just double checking, is there an impact on com.datadoghq.ad.logs & com.datadoghq.ad.logs

@mfpierre yes successfully tested on a test-cluster

@aerostitch IIRC you were relying on this feature on Agent5. This will be shipped in 6.11.0

Awesome, thanks a lot! :)


Report PDF Margin

As a superuser
I want the margins on all report PDFs to be a minimum of 1/4 inch on each side
so that reports can be printed on any modern printer
Acceptance Criteria
Scenario 1:
GIVEN the resource forecast
WHEN printing on 11 x 17 paper
THEN the margin must be 1/4 inch
Validation Rules?
(If yes, list here)
Screenshot
(Link above will always be the most up-to-date version.)

Added 3/4 inch margin on both sides. Please verify.


Docs

Document the interface and key assumptions about the pdesystem

I would like to take up this task, if no one else is already working on it. Pls do let me know if you have any suggestions or directions to keep in mind.


Deprecated: Using ${var} in WordPress 6.4.2

Deprecated: Using ${var} in strings is deprecated, use {$var} instead in G:\xampp\htdocs\grandinho\wp-content\themes\air-light-master\inc\hooks\scripts-styles.php on line 109
Deprecated: Using ${var} in strings is deprecated, use {$var} instead in G:\xampp\htdocs\grandinho\wp-content\themes\air-light-master\inc\hooks\scripts-styles.php on line 109
Deprecated: Using ${var} in strings is deprecated, use {$var} instead in G:\xampp\htdocs\grandinho\wp-content\themes\air-light-master\inc\hooks\scripts-styles.php on line 109

Thanks for the issue.

What air-light version are you using?
What PHP version are you using?

Air-light didn't have PHP 8 support in January. This is fixed in 9.4.0.


Replace libgit2 with gitoxide

Following #621, we wish to drop libgit2 and use gitoxide instead.
What's left:

[ ] status index/worktree
[ ] status untracked + recurse untracked dirs
[x] write index back (even though we shouldn't do it unless it's lossless, should be discussed at least

Also see the corresponding tracking ticket at gitoxide, showing which features are implemented already.
@o2sh Would you be OK if I contributed the most promising part related to traversing the commit-graph for a start? This would mean that build times increase due to building git2 and gitoxide (minimal, only the required crates at first). Otherwise we'd have to wait until all features are available, which is probably no more than a couple of months.
That said, I'd prefer to get started right away and am happy to see the transition through until completion.

Sure, the commit-graph traversal is indeed the most crucial part.
Looking forward to see your contribution.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

git status is going to happen soon and then the issue can be properly resolved.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

I have updated the description to be more precise, and hope that at least one of these checkboxes can be ticked soon.

Thanks!

even though we shouldn't do it unless it's lossless, should be discussed at least

Well, I'm open to discussion :laughing:. Could you elaborate on this one, though? I'm not 100% sure what this task means, or when it would have been completed.

The index file has a lot of state in it which is used to accelerate certain git operations. If a git status is run, it might update some file-stat information of entries (rare), and in order to not redo that work (usually very little) one would have to write these changes back. But the way this is currently implemented, it will drop most index-extensions as writing them isn't implemented, only reading is. This will not break the index, but it might make some operations which rely on these now missing extensions to be slower. An example for this is the untracked files extension which git uses to learn about untracked files faster.
Thus I propose not to write the index back at all, so onefetch is truly read-only without unintended side-effects, no matter how rare or noticeable they might be.


mode argument of dask.dataframe.DataFrame.to_csv does not work

The documentation of dask.dataframe.DataFrame.to_csv provide mode as an argument, but its value is always overwritten at dask/dataframe/io/csv.py:728-733.
(Please keep the argument. There are some cases where append mode makes sense, such as using get_partition to combine all partitions to a same csv file.)

Would you be interested in providing a PR with this change? You should note that not all file system backends necessarily allow append.

I have added a pull request. I am not sure whether you want to add a test or how you want it tested.

As explained in the pull request, this proposal has quite some flaws. I am closing it for now.


Added Pomegranate

The Pull Request resolves Issue #807
Description:
Added Pomegranate, since it wasn't taken by anyone.

Any idea why the build failed?

Okay, this is related with Issue #771


Remove display name limit of 32

There's no need to have a 32-character limit anymore

We moved it to 40th position. Do we need more? It can only reach 45 at most (without color symbol)

As of now, it does include color symbols towards the count

Color characters will not be included in the length in the next version


NodeJs and NPM Publishing Workflow

Update to #2407, same npm build workflow, but added npmpublish so that any release created on GitHub can be published to npm to easily be managed. @nfarina, @KhaosT, @alexindigo, @Maddox, @madmod, @plasticrake, @AppleTechy, or @ebaauw  Let me know what I need to change(or go ahead and make the modifications) to make it work correctly for homebrige.

Just ran a test of the npm publish and it successfully published to npm. npm_token will just have to be added to the repo.


Compilation failed when ESP_IDF_VERSION = "release/v5.1"

Compilation failed when ESP_IDF_VERSION = "release/v5.1" , the error message is as follows:
error[E0560]: struct `esp_idf_sys::ledc_timer_config_t` has no field named `deconfigure`
   --> /home/danc/.cargo/registry/src/index.crates.io-6f17d22bba15001f/esp-idf-hal-0.41.1/src/ledc.rs:133:13
    |
133 |             deconfigure: false,
    |             ^^^^^^^^^^^ `esp_idf_sys::ledc_timer_config_t` does not have this field
    |
    = note: available fields are: `speed_mode`, `duty_resolution`, `timer_num`, `freq_hz`, `clk_cfg`

For more information about this error, try `rustc --explain E0560`.
error: could not compile `esp-idf-hal` (lib) due to previous error

After inspection, it seems to have something to do with this:
https://github.com/esp-rs/esp-idf-hal/blob/c68856c4e4886abf1d328d2b46cb3663c0059565/src/ledc.rs#L129-L133
Can it be changed here to something like this? Although I can compile and pass after manual modification.
#[cfg(not(any(esp_idf_version_major = "4", esp_idf_version_major = "5")))]
deconfigure: false,

will be addressed in https://github.com/esp-rs/esp-idf-hal/pull/255

Fixed with https://github.com/esp-rs/esp-idf-hal/commit/2ab4c709f170643daf5e419ac91d6845f8567a68
Will release a new patch version shortly.


HIX calculation for pages with no real text

Describe the Bug

Pages with only html tags in its content are not handled as empty pages, and HIX calculation/refresh is activated and it fails to retriev HIX value.
Steps to Reproduce

Opne a new page
Insert per "View -> Source code" html tags (example below)
Click the HIX update button or save the page
See the error in the box "Text understandability"

Expected Behavior

Such a page is handled as empty page (and show "There is no page content")
or
Manage to fetch HIX value
Actual Behavior

HIX retrieving is activ but fails.
Additional Information

See this similar bug too #2916
Exmple1:

Exmaple2:

Error like HIX benchmark API call failed: <HTTPError 400: 'Bad Request'> was not observed but Failed to retrieve the hix score for <PageTranslation (.........)>

Traceback

Solution from the issue grooming: do not send the text to TextLab if the page doesn not contain real text.


Compile to ObjectiveC doesn't work

Get ARC issues when trying to integrate into a project.

You can use the protos from my swift PGoApi port.

Closing this issue, let me know if this is still relevant.