[CSM-485] Fasten history fetching more

This includes:

Fetching limited history from database. Now time for history endpoint is approx. min(totalTxsNum, limit) * 7 ms, which looks ok taking into account (if I understood it correctly) that passed limit is usually a number of 1-2 order of magnitude for bittrex and up to 1000 for daedalus.

History update is performed as single modifying db transaction, not as get + set.
In fact, I thought that the latter works for O(|whole modified structure|), but guys made it clear to me that actually it's O(|modified part|). And accidentally I amended this part to point 1, so it will take an effort to revert - but I'll do if anyone ask me to.
Note that we do not benefit from atomicity in this case, because updateHistoryCache is in fact always called under lock

Still need to test correctness of this

I think we can just throw this PR to trash because

is made incorrectly
I assumed that fetching from db takes time, which is not true

I'm sorry for your time wasted on reviewing it

Extracted useful part to #1722


Use filename instead of the class name for drag & drop Mesh Instance.

Fixes https://github.com/godotengine/godot/issues/24061
Before:

After:

Seems like a good idea, thanks!


Is project alive?

Just curious are there any plans for this project to continue?

@mustafaakin thanks for reaching out; the short answer is yes.
We've identified the first deliverables and currently working on implementation for these deliverables.  We're expecting to get the first round of PRs out in the next few weeks.


New User Creation

Are you in process of creating a New User functionality or its already done. If done can you tell how can i enable it.

@umakant1979 i will be add soon.

Could you please make multi level user login systems such as employee login, admin login or even add access layers according to the roles ?


Release DataFusion 16.1.0 (patch release)

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Ballista cannot upgrade to DataFusion 16.0.0
Follow on to https://github.com/apache/arrow-datafusion/issues/4776
Describe the solution you'd like
Now that https://github.com/apache/arrow-datafusion/pull/4933 is merged, I would like to create a 16.1.0 release so that Ballista can upgrade.

[x] https://github.com/apache/arrow-datafusion/pull/4951

Describe alternatives you've considered
None
Additional context
None

cc @jonmmease as I think you you may have some ideas of what would be good to backport too
Also cc @maxburke who I think was working with an upgrade

FWIW, here are the post 16.0 PRs I'm currently backporting for VegaFusion (https://github.com/jonmmease/arrow-datafusion/pull/126). It's basically #4873 and a handful of PRs to finish wiring things up so that mean and variance-based aggregates work in windows with custom frames.
Thanks for the ping!

@jonmmease I reviewed those PRs, and it seems reasonable to apply them all to the patch release. Could you create the necessary PR(s) to apply those changes to the maint-16.x branch?

Sure thing

@andygrove, turns out the window aggregated fixes did make it into 16.0.0, so there's only one backport PR needed for #4873:  Here's the PR: https://github.com/apache/arrow-datafusion/pull/4959


New playsounds system

Don't merge yet. Missing is the alembic migration file because i couldn't get alembic to work for some reason, or I don't know how to make it work in pajbot specifically.


File info not auto update

What system are you running Yazi on?
Linux X11
What terminal are you running Yazi in?
Windows Terminal
Yazi version
0.2.3
Did you try the latest main branch to see if the problem got fixed?
Tried, but the problem still
Describe the bug
不通过yazi的manager命令来修改文件，修改后，文件信息没有更新
Expected Behavior
auto update file info
To Reproduce
比如：在外部使用 chmod 修改文件权限后，yazi显示的不是最新的权限
Configuration
No response
Anything else?
No response

I can't reproduce this issue in the latest main branch.
I noticed you mentioned trying the latest main branch - which commit are you trying? Please do a yazi -V and paste the version here.
https://github.com/sxyazi/yazi/assets/17523360/b9191a46-ddc0-4f4e-8b88-91a098d81281

抱歉，没有贴全部的版本信息
我目前更新到 0.2.4 问题依旧
yazi -V 结果：yazi 0.2.4 (Arch Linux 2024-03-10)
以下是我基于这个版本的视频：
https://github.com/sxyazi/yazi/assets/45219522/eb09c1fa-3e7f-45f1-a9b1-107d09dc2489

yazi --debug：
Yazi
    yazi 0.2.4 (Arch Linux 2024-03-10)

Environment
    OS: linux-x86_64 (unix)
    Debug: false

Emulator
    Emulator.via_env: ("xterm-256color", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect: Unknown([])

Adaptor
    Adaptor.matches: KittyOld

tmux
    TMUX: false

Zellij
    ZELLIJ_SESSION_NAME: None

Desktop
    XDG_SESSION_TYPE: None
    WAYLAND_DISPLAY: None
    DISPLAY: None

Ueberzug
    Version: Err(Os { code: 2, kind: NotFound, message: "No such file or directory" })

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: true

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/sky03/.local/state/yazi" directory.

You are in WSL right? What's your file system?

Yes, I'm use ArchLinux in WSL, and WSL2 only support the ext4.


Labels for scatter plot points

I'm looking out for how to add individual labels to the scatter plot that is generated by gmscatter. It would be great if anyone could assist me on this.
Thanks,
Chinmay

Which OS you are using?


Menu items in settings are not visbile

Text for the menu items in settings are not visible but they are clickable.
This is both for dark and light appearance.

ios v14.4

Same issue


Add images to spec.md

Proposed fix for #5671.

This won't work when the spec needs to get regenerated from the docx.


Installation error

I'm trying to install on Ubuntu Server 22.04 LTS from under docker. What could be the problem and how to fix it?
[+] Building 1.5s (7/11)
 => [internal] load .dockerignore                                          0.0s
 => => transferring context: 2B                                            0.0s
 => [internal] load build definition from Dockerfile                       0.0s
 => => transferring dockerfile: 499B                                       0.0s
 => [internal] load metadata for docker.io/library/python:3.8-slim         1.0s
 => [1/7] FROM docker.io/library/python:3.8-slim@sha256:f4efb39d02df8cdc4  0.0s
 => [internal] load build context                                          0.0s
 => => transferring context: 3.05kB                                        0.0s
 => CACHED [2/7] RUN apt-get update                                        0.0s
 => ERROR [3/7] RUN apt-get install -y python3 python3-pip python-dev bui  0.4s
------
 > [3/7]  RUN apt-get install -y python3 python3-pip python-dev build-essential python3-venv ffmpeg
#0 0.354 Reading package lists...
#0 0.367 Building dependency tree...
#0 0.369 Reading state information...
#0 0.372 E: Unable to locate package python3
#0 0.372 E: Unable to locate package python3-pip
#0 0.372 E: Unable to locate package python-dev
#0 0.372 E: Unable to locate package build-essential
#0 0.372 E: Unable to locate package python3-venv
#0 0.372 E: Unable to locate package ffmpeg
------
Dockerfile:13
--------------------
  11 |     RUN apt-get update
  12 |     #RUN apt-get install -y python3-pip
  13 | >>> RUN apt-get install -y python3 python3-pip python-dev build-essential python3-venv ffmpeg
  14 |
  15 |     RUN mkdir -p /code

ERROR: failed to solve: process "/bin/sh -c apt-get install -y python3 python3-p                                                                                ip python-dev build-essential python3-venv ffmpeg" did not complete successfully                                                                                : exit code: 100
ERROR: Service 'chatgpt_telegram_bot' failed to build : Build failed

Has it been solved? @exeigor

This is no good solution for sure, but for me commenting a line on Dockerfile worked.
RUN apt-get install -y python3 python3-pip python-dev build-essential python3-venv ffmpeg
to
RUN apt-get install -y python3 python3-pip python-dev build-essential python3-venv ffmpeg
Let me know if this works for you @art1313 and @exeigor

I updated Dockerfile in the latest commit. Try it again


esm support to fix bundler incompatibility

This package dist is currently not compatible with bundlers.
tsc outputs a commonjs format, but that is for node environments, not browser. It causes Object.defineProperty(exports, "__esModule", { value: true }); to be emitted, which causes an error in a browser.
I updated tsconfig.json to output esnext instead, and that fixes the problem.
See for more info: https://blog.rendall.dev/posts/2019/1/14/problem-typescript-adds-objectdefinepropertyexports-esmodule-value-true/

👋🏻  @benallfree
Thanks for your work!
Do you also have this problem ?

@jorisre Yes, this fixes #71. Thanks for catching that!

Thanks @jorisre shall we do a release with it today?

Thanks @jorisre shall we do a release with it today?

Yes you can :)

@jorisre sent you a private message at twitter.

Any thought on when the updated npm package will be published? I have to maintain my own branch with a build folder until the package is published


Fixes #426

Skip minibatches of size < 3

Codecov Report

Merging #430 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #430      +/-   ##
==========================================
+ Coverage   88.73%   88.77%   +0.04%
==========================================
  Files          29       29
  Lines        3372     3376       +4
==========================================
+ Hits         2992     2997       +5
+ Misses        380      379       -1

Impacted Files
Coverage Δ

scvi/inference/jvae_trainer.py
96.25% <100%> (+0.04%)
:arrow_up:

scvi/inference/trainer.py
90.26% <100%> (+0.08%)
:arrow_up:

scvi/inference/autotune.py
69.09% <0%> (+0.25%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f58fcea...c0fa60d. Read the comment docs.


Voluptuous statsd

Description:
Migration of the configuration check to voluptuous.
Related issue (if applicable): fixes 127528299
Example entry for configuration.yaml (if applicable):
statsd:
  host: localhost
  port: 8125
  prefix: DB_TO_STORE_EVENTS
  rate: 1
  log_attributes: true

@michaelkuty, would be nice if you could take a look at the changes and run a quick test. Thanks.

LGTM !

StatsD tests are broken.


Response headers are not displayed

When I run bootdev cli I don't see headers. For example for routing lesson:

I can see headers in any other rest clients. Also, the lesson submission does work.

we filter the headers to only show the ones being tested for
the behavior is kinda confusing, I will try to see if I can think of a way to make it more obvious 👍

Sure. As an idea, I think you can highlight the headers that are being tested

Would something like this make sense? We iterate through the result headers and use lipgloss to style any matches from the expected headers as green otherwise output them as grey. Then we can also loop through the expected headers and output any missing ones as red.
		for k, v := range result.Headers {
			if req.Request.Headers[k] == v {
				fmt.Print(green.Render(fmt.Sprintf("   - %v: %v", k, v)) + "\n")
			} else {
				fmt.Printf(gray.Render(fmt.Sprintf("   - %v: %v", k, v)) + "\n")
			}
		}
		for k, v := range req.Request.Headers {
			if _, found := result.Headers[k]; !found {
				fmt.Printf(red.Render(fmt.Sprintf("   - %v: %v", k, v)) + "\n")
			}
		}

@Waldeedle that looks nice!
I'm going to rewrite the render functions for the HTTP tests soon to match the fancy ones we have for the commands now, I'll keep this idea in mind


ARROW-16892: [Dev][Release] Fix version sorting on merge_arrow script

I have tested injecting FakeVersions and seeing how it prompts the correct one.
If there would be an 8.0.0 today it will prompt for it:
(Pdb) versions[0].name = "8.0.0"
(Pdb) c
Enter comma-separated fix version(s) [8.0.0]:

If there would be a 10.0.0 it would prompt the 9.0.0 as expected:
(Pdb) versions[0].name = "10.0.0"
(Pdb) c
Enter comma-separated fix version(s) [9.0.0]:

@amol- @kou are you ok with this fix for the merge script?

Is everyone ok with merging it? I'll merge it if I don't hear any opposition by tomorrow :D


ci: Java snapshots are not deployed for v0.40.0-rc.2

Java release CI requires to have v0.40.0-rc2

Interesting. I'm asking on https://github.com/orgs/community/discussions/67498. This should not happen unless the trigger logic is different from Regexp I know.

Tag I pushed is v0.40.0-rc.2 which is different from v0.40.0-rc2.

So I guess this is not an issue but a tagging mistake?
IIRC we instruct the RM to tag vX.Y.Z-rcN without the extra ..

IIRC we instruct the RM to tag vX.Y.Z-rcN without the extra ..

I prefer following the semver specs that use rc.N instead: https://semver.org/
Do the ASF or Java bindings have specific requirements for this?

Do the ASF or Java bindings have specific requirements for this?

No. We can change all the reference and the trigger.

No. We can change all the reference and the trigger.

Cool, let's keep consistency in all the references. I will update all the scripts.


Stack using Linked List in JavaScript

Stack using Linked List
Implementing a Stack data structure using Linked List, where following operations can be performed -

Insert items in the stack ( push )
Delete items from the stack ( pop )
Get the first top element of the stack ( peek )
Get all the items of the stack ( printStackElements )

Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
The goal is to have an understanding of how a stack can be implemented in different ways.

/assign


NDArray reshapes

xtensor has support for reshapes, so it should be easy to return a reshaped copy.

Actually, consider this a duplicate of the umbrella #14.


0.8.0 Roadmap

[x]  Rework the burn-import module to simplify ONNX operations implementation.
[x]  Develop a new backend for Burn, burn-wgpu (WebGPU).
[ ]  Implement the necessary ONNX operations to import BERT trained models.
[x]  Upgrade the burn-dataset module to use SQLite for storing datasets, encompassing both HuggingFace datasets and user-generated datasets.
[x] LSTM/GRU modules

v0.8.0 has been released:  https://github.com/burn-rs/burn/releases/tag/v0.8.0 Closing it and moving  "Implement the necessary ONNX operations to import BERT trained models." into its own issue: https://github.com/burn-rs/burn/issues/534


Project page fixing button placement, enhancing design and making it responsive

For project cards, the button placement is wrong and styling could be improved.
Page Link

I would like to work on this issue. Please assign this to me.

@sn2606 Assigning this issue to you. Happy Coding !


LoadLocalJSON is converting curly apostrophes to â€™ because of encoding issue

I am loading JSON from disk with LoadLocalJSON node, but it is converting curly apostrophes ’ to â€™. This is probably an encoding error; on my system it defaults to opening the file in cp1252 (Windows-1252) encoding rather than UTF-8.
It looks like you are not explicitly setting the encoding when you open the JSON file, which is probably the source of the issue. Maybe you could detect the encoding of the file first with the chardet library before setting the explicit encoding type when loading the JSON file, like this:
import chardet
....

        # Detect the encoding
        with open(file_path, 'rb') as file:
            raw_data = file.read()
        result = chardet.detect(raw_data)
        encoding = result['encoding']

        # Open and read the file with detected encoding
        with open(file_path, 'r', encoding=encoding) as file:
            data = json.load(file)

Unfortunately, this detection would require another package on install (chardet). If you don't want to require an additional package, you could just set the encoding type explicitly as UTF-8 when you read the JSON file, which I think is the standard encoding format for JSON.
with open(file_path, 'r', encoding='utf-8') as file:
            data = json.load(file)

I tried both of these methods, and my JSON file was properly read in with curly apostrophes intact. Btw, your SaveJSON node already writes JSON to file with explicit UTF-8 encoding. It is just the reading the JSON file back in that is the issue.

It makes absolutely sense to add the utf-8 encoding, I didn't think about it!
I added the fix on-the-fly while making the new release.
I did quite a big refactoring, if something broke let me know (also if the fix is ok)!

The fix works great! I'll let you know if anything else broke.


$> breaks operator symmetry

g    $    x -- function over value
g   <$>   f -- function over Functor
g x <$    f --    value over Functor
-- flipped
  x  #  g   --    value over function
  f <#> g   --  Functor over function
  f  $> g x --  Functor over value

I feel strongly $> should be #>. Am I missing something?

@garyb @hdgarrood Agreed? I'm not familiar with the deprecation process or I'd PR.

I don't know... ($>) is the functor counterpart to (*>), and works totally naturally in that regard.

I get that. The difference being that <*> accepts the same type on both sides whereas anytime $/<$> is flipped it becomes #/<#>. The documentation (and code correspondingly) reads: a version of (<$) with its arguments flipped. I don't believe changing the symbol to #> breaks symmetry with *>. But it does break what I see as beautiful intuition between operators and types. I stumbled on it outlining basic operators for an Egghead lesson.

Ah I see. I don't feel particularly strongly about it because there's also a bit of a problem in that there's a mixing of two metaphors with these operators - angled brackets and $ vs # are both signifying "direction". I guess ideally there should be (<$), ($>), (<#), (#>), but that wouldn't really help the matter as one of the one-sided $ and # varieties are always going to violate the intiution you're appealing to here.

The angled bracket direction thing coming from (<<<) and (>>>), if that wasn't clear.

I agree that the current choices of $, # <$>, <#> and how they relate makes sense, but you haven't persuaded me that the same idea should extend to <$ and $> in the way you suggest.
I do think changing it in this way would break a correspondence between <$, $>, <*, and > unless we also were going to add a flipped version of <> and rename *> in the same way (but I'm not really keen on that).

I see your point (maybe) too. <*> doesn't have a flip (<*>) corollary (unlike <$>/<#>). Therefore although (*>) = flip (<*) it semantically conveys choice. So although #> looks more consistent in my outline (because of types) it could undermine the notion of choice with <$>. Am I close? I prefer #> for pedagogical reasons. My goals when I teach operators will be instilling intuition so that <$> (like $ on a thing in a <box>) connects cognitively more than map (kinda like a loop). The choice operators and >>= (with Kleisli composition >=>) all imply passing the result of a function (or Functor of functions for <*>) on one side. This intuition is what breaks with <$> because it never accepted a function on that side (<#> did). See where I'm coming from?

As an aside I almost wish we'd never introduced # and <#>, since I only ever write code in $ style I find it hard to flip my thinking when I encounter the opposite, and don't get me started on code that mixes the two...

My point about angle brackets denoting direction is that Kleisli compose and bind also have =<< and <=< variants though, which flip the function arrows just as <$ does:
(>=>) :: (a -> m b) -> (b -> m c) -> a -> m c
(<=<) ::            ^ flip this

($>) :: f a -> b -> f b
(<$) ::     ^ flip this

I personally love # and use it predominantly for its use case. It's more intuitive when pipelining data (like F#//LiveScript/Elm's |> or OOP's .). I agree they shouldn't be used in conjunction however. My Cons example from purescript/purescript-foldable-traversable#44 illustrates the use case as well as @taylorfausak's SVO stuff in Neon.

Yeah, I understand the use case, it's the same argument for (>>>). I was just commenting on a problem I have with it personally.
The reason I prefer (<<<) and $ is they preserve the normal ordering of function application:
f $ g $ h x = f (g (h x))
f <<< g <<< h $ x = f (g (h x))

Even if it does mean you have to read things "backwards" in a pipeline situation. I'd argue the only reason we think it's "backwards" is we happen to come from a culture that writes left-to-right.

As a compromise, what if we did add <# and #>? There is some redundancy there, but you could argue that about any of our flipped operations.

I deleted two responses as I realized more deeply what you're saying. For documentation purposes: although <*> receives a Functor on both sides it's much closer in shape to <$> because one side is just functions lifted into the respective Functor. Therefore <* and *> effectively choose between results as if the left side (of functions) were evaluated. <$ and $> likewise choose as if the left side (a function) were evaluated. #> and <# would simply be aliases choosing the opposite side without a corollary in Apply. For that reason maybe it's better left well enough alone?

But naively <* and *> just pick the left or right result independently of which side of <*> is evaluated. That's why I thought (now uncertain either way) it made sense for #> to be the flip of <$

That's why I said "I don't know" in my first reply :smile:, I didn't mean it in the sense of doubting your conclusion, I just don't know what the best approach is given all the above.
<$ and $> are slightly strange operators in this scheme, as unlike <* and *> which in some sense evaluate both sides and discard a result, they are more like "evaluate one side and then inject a different result value". The reason it makes sense to use $ in the operator is it's functor-related, but since we have two different symbols for functor operators we end up with this problem.

Because <$ and $> imply sidedness instead of flippedness would (#>) = ($>)? That seems worse. I think $> is more correct but would still prefer $> for teaching in that scheme. It might be worth doing in Preface instead though.

@hdgarrood I concluded above. TLDR; I see your point about <*> but given that #> would mean precisely #> I prefer the latter for its type symmetry illustrated in my original description and accompanying gist.

What I'd like to happen, in order of preference:

Everything stays the same (sorry),
We remove $> and replace it with #>,
We add #> which is the same as $>.

You've categorised these six operators into two groups of three, but I'm just not convinced that $> and <$ fit into your first and second group respectively. As far as I can see there's no common property shared by every element of either group (that isn't also shared by one or more elements of the other).

For the avoidance of doubt: I'm really not very keen on either of my options 2 or 3, I just wanted to make it clear that introducing two different operators that do the same thing is :-1: :-1: :-1: whereas changing $> to #> is just :-1:.

For instance, I think it's at least as sensible that $> should correspond to $, as both put the "value" (the thing to which a function is going to be applied) on the right, and likewise <$ to #.

Everything stays the same (sorry),

Status quo isn't that bad. Worst case I'm amenable to doing it in Preface which is intended to be more beginner friendly. But after writing it out below I again prefer #> here.

there's no common property shared by every element of either group

They are symmetrical progressions to satisfy type transformations (probably wrong word). Their structure is:

ID
Expression
Left
Right

1
pure g <*> pure x
function, lifted
value, lifted

2a
g $ x
function
value

2b
g <$> pure x
function
value, lifted

2c
g x <$ pure x
function, applied
value, lifted, discarded

2d
pure (g x) <* pure x
function, applied, lifted
value, lifted, discarded

3a
x # g
value
function

3b
pure x <#> g
value, lifted
function

3c
proposed: pure x #> g x
value, lifted, discarded
function, applied

3d
pure x *> pure (g x)
value, lifted, discarded
function, applied, lifted

[Previous outline]

NB…

Anywhere I write function, applied is purely conceptual but nonetheless holds true for the relationship between >=> (monadic functions), >>= (applied on one side), and >> (in Haskell applied on both sides).
As each of us noted, the shape of <*> is most like <$> thus unlike <#>.
 If <&> existed I might likewise expect *> to be called &> (kinda thankful it doesn't). But a flipped <$> does exist hence my intuition about #>. That #> would be implemented identically to $> reinforces my notion that $> doesn't fit semantically (any more than <#).
Final snippet with naive implementations:
(<#>) = flip (<$>)
( #>) = flip (<$ )
( *>) = flip (<* )
-- but hypothetically in some twisted prelude
(<&>) = flip (<*>)
( &>) = flip (<* )

(that isn't also shared by one or more elements of the other).

Gist with alternate grouping? If neither of you is convinced it's not that big a deal and we can close.

I'm brewing a response but in the meantime I think it's worth noting that *> is not flip <*: https://gist.github.com/hdgarrood/41c8f423a64ab218d681

(@paf31 should http://try.purescript.org/?gist=41c8f423a64ab218d681 work?)

Right my mistake. It nonetheless holds true *> is like <* if <*> were flipped which <$> does have a flipped variant. To reemphasize my motivation these operators are degenerate forms of one another:

Need to combine a function with a value? $
Your value is wrapped in a Functor? <$>
Your function is already evaluated? <$
Your value is wrapped in a Functor? <*

The same progression applies flipped except $> breaks symmetry with <#> which is bad for pedagogy. They are exactly the flip of one another and in both other flips marked by changing the symbol.

@hdgarrood Your file has to be called Main.purs but it's on my todo list to work on if I get to it.

@hdgarrood It works for me ...
I think everything is consistent as it is. I think of the $ functions as being derivable in terms of the * functions and pure:

x <$ fa = pure x <* fa
fa $> x = fa *> pure x
f <$> x = pure f <*> x

Another way is to think in terms of const, but that's a bit more confusing IMO.
Now, x <#> f = (#) <$> x <*> pure f, so it's less clear what <# and #> should mean. I think we should just omit them.

Haha, sorry - @AppShipIt was correct, it was broken until I changed the filename to Main.purs (previously it was something else).

@paf31's reasoning makes sense to me especially after @hdgarrood's gist.


Add Node2D.get_global_rot method ?

Hi,
Node2D has get_pos and get_global_pos to get position relative to the parent node or global to the scene.
On the other hand it seems it only have get_rot, so why not add a get_global_rot ?
Beside it implementation would be pretty similar to get_global_pos:
float Node2D::get_global_rot() const {

    return get_global_transform().get_rotation();
}

I can submit a PR if you're interested ;-)

Note that if such a method is added, the natural question that follows is "Would we have get_global_scale in Node2D?"
I personally don't really see the need of it, i.e. when would one want only the rotation, and not scale and so on. Also, for some transformations (like top-down -> isometric), the rotation information is a bit speculative.

My need was pretty simple: I wanted to send a projectile from a moving/rotating character.
To do so I get the position and rotation of the firing character and apply them to a new instance of the projectile:

extends "base_weapon.gd"

export(int) var speed
export(PackedScene) var projectile_scene

func _fire():
	var projectile = projectile_scene.instance()
	projectile.set_pos(self.get_global_pos())
	var orientation = get_global_transform().get_rotation()
	projectile.set_rot(orientation)
	var linear_speed = Vector2(cos(orientation) * speed, -sin(orientation) * speed)
	projectile.set_linear_velocity(linear_speed)
	get_tree().get_root().add_child(projectile)

Given I could really easily get the global position with get_global_pos, it seems pretty legit to me you can get the same for the rotation...
But maybe I'm missing the obvious way to do that in Godot :wink:

Since there is already a get_global_pos() function, the way I see it get_global_rotation() should be added as well as get_global_scale().

@EdwardAngeles Slight nitpick, get_global_rot is better.
Also, we would have setters of the same properties.
@touilleMan Sounds legit, though you can just make projectile.set_global_transform(get_global_transform()) and be done with it.
Sounds good to me though, better to have those 4 methods than not, given that they are pretty easy to implement and would lead to more API coherence.

I've added a PR on this :+1:

Well this would revolutionize my coding experience! I can't count how many hours I've wasted fighting with global vs. local transforms.

implemented by c402ac13df1a945336faa009b469ac4f0528770f


UrlRule Problem with trailing slash (/)

Hi i have added a UrlManager rules like given below added to my config
  '<project:[-a-zA-Z]+>/<controller:\w+>' => '<controller>',
  '<project:[-a-zA-Z]+>/<controller:\w+>/<action:\w+>' => '<controller>/<action>',

If i then call a url like
/projectname/controllername/action            (works)
/projectname/controllername           (works pointing to the default action in the controller)

/projectname/controllername/              (with trailing slash doens't work)

But the Url with a trailing slash produces a 404 error on the SiteController.
This is because the UrlRule doesn't match.
if i change the UrlRule to the code below (\w* instead of \w+) the default controller action is called.
  '<project:[-a-zA-Z]+>/<controller:\w+>/<action:\w*>' => '<controller>/<action>',

I also tested the UrlRules in yii1 but there the UrlRules from the top are also working with a trailing / in the Url but not in yii2.
So I guess there is some kind of "bug".
I Don't Know if the trailing slashes are removed in the Request in yii1 but not in yii2.
But there is clearly some difference between the 2 frameworks.
Regards Horizons

Yes, there's difference and it's intentional.

@yiisoft/core-developers it's not the first time people are asking about it. Maybe we should redirect to proper URL instead of not allowing to use incorrect one altogether?

is it somehow possible that something is added to the pattern so that a optional trailing / is in the pattern.
In my project case it would be something like
$this->pattern ='#^(?P[-a-zA-Z]+)(?:[/])?$#u';
instead of
$this->pattern ='#^(?P[-a-zA-Z]+)$#u';
maybe this trailing slash pattern is added to the pattern in the UrlRule init when some placeholder is in the pattern or some variable is set.

#1807

The documentation is wrong. It says "the leading and ending slashes are ignored": http://www.yiiframework.com/doc-2.0/guide-runtime-routing.html#url-rules

It seems to a bigger problem than I've thought. Wordpress is adding / at the end of all links posted.

While it could be solved serverside, it's better to do in Yii itself else there will be lots and lots of unexpected 404s.

Could be done app-side like the following:
$config = [
    // ...
    'params' => require(__DIR__ . '/params.php'),

    // redirect to the page with the trailing slash
    'on beforeRequest' => function () {
        $app = Yii::$app;
        $pathInfo = $app->request->pathInfo;
        if (!empty($pathInfo) && substr($pathInfo, -1) !== '/') {
            $app->response->redirect('/' . rtrim($pathInfo) . '/', 301);
        }
    },
];

This leads to problems with search engines, as URI with and without ending slash may be different documents. All site should use only one style and return 301 or 404 for wrong ones.

301 is better.

@samdark then it should be an option to choose between with/without ending slash. And option to allow both for people who don't care about seo. And maybe 404 as default option (for BC)?

There's no problem with serving 301 instead of 404.
To choose with or without / one could configure UrlManager even now.
"Don't care" would not make sense if there will be 301 i.e. people would be redirected no matter which URL is used.

This redirect rule works for me in htaccess:
RewriteCond %{THE_REQUEST} \s/+(.+?)/+[?\s]
RewriteRule /$ /%1 [R,L]

Add a route at last line '<url:.+>/' => 'site/redirect',
and add a action
public function actionRedirect($url = '')
{
    if (substr($url, -1) == '/') {
        return $this->redirect('/' . substr($url, 0, -1));
    } else {
        throw new NotFoundHttpException;
    }
}

right, thanks gerpayt

Not sure where all the slash/no slash ended up but ran into something as well when porting some Yii1 code over to Yii2.
Docs have it stated incorrectly if the a requests  getPathInfo()  is used.
In the docs it states -
A path info refers to the part that is after the entry script and before the question mark (query string). The starting and ending slashes are both removed.
When doing something like $request->getPathInfo(); it returns the trailing slash.  This is not so much about handling the route, but different results expected for getPathInfo().
No big deal either way, just want the docs to match what it really is supposed to be,  solved by rtrim() the slash prior to using the getPathInfo() value in my code, again not a huge one, just letting folks know the docs may not match functionality.
Thanks
Sandy

I use this code in frontend/config/main for redirect to the page without the trailing slash
'on beforeRequest' => function () {
        $pathInfo = Yii::$app->request->pathInfo;
        if (!empty($pathInfo) && substr($pathInfo, -1) === '/') {
            Yii::$app->response->redirect('/' . substr(rtrim($pathInfo), 0, -1), 301)->send();
        }
    },

But I have HTTP/1.1 302 Found istead of HTTP/1.1 301 Permanent Redirect

@sauron918 please create a separated issue for your problem

@gerpayt:
slash should be inside regular expression:
'<url:.+/>' => 'site/redirect'
some improvements: 301 redirect, baseUrl, absolute url, one redirect in case of many slashes
public function actionRedirect($url)
{
	if (substr($url, -1) == '/') {
		return $this->redirect(Url::to(Yii::$app->getRequest()->getBaseUrl() . '/' . rtrim($url, '/'), true), 301);
	} else {
		throw new NotFoundHttpException;
	}
}

@samdark could you please take a look at my pull request? I hope it will help to solve the issue. I use this approach on sites I develop.

Still waiting for write answer. Why I will use 301 redirect for trailing slash?. As you guys know in yii1 better for url manager. Some guys has post like that
/projectname => projectcontroller/index
/projectname/ => projectcontroller/index
/projectname/action => projectcontroller/action
/projectname/projectspecifc_controller => projectspecifc_controller/index
/projectname/projectspecifc_controller/ => projectspecifc_controller/index
/projectname/projectspecifc_controller/action => projectspecifc_controller/action
It's not perfect answer. I have configure like that
'baseUrl' => '/',
'enablePrettyUrl' => true,
'showScriptName' => false,
'suffix'          => '/',
'enableStrictParsing' => true,
My code working fine.
Thanks

Because search engines think a/ and a are different URLs and since these contain the same content, your website is getting penalized for duplicated content. 301 solves the issue.

Another .htaccess rewrite rule before checking directory.
    # see http://stackoverflow.com/a/11880879/2485734
    RewriteCond %{REQUEST_URI} !(/$|\.)
    RewriteRule (.*) %{REQUEST_URI}/ [R=301,L]

and the config for UrlManager, enabling suffix
return [
    'class' => 'yii\web\UrlManager',
    'enablePrettyUrl' => true,
    'showScriptName' => false,
    'suffix' => '/',
    'rules' => [
        '/' => 'site/index',
        'product/<slug>' => 'product/view',
        'product-category/<name>' => 'product-category/index',
        'search/<keyword>' => 'search/result',
    ],
];

I've experienced similar issue with IIS.  So if anyone still experiencing trailing slash issue,
Please use following URL rewrite Rule in web.config
<rule name="SEO - Remove trailing slash" stopProcessing="false">
                    <match url="(.*)/$" />
                    <conditions>
                        <add input="{REQUEST_FILENAME}" matchType="IsDirectory" negate="true" />
                        <add input="{REQUEST_FILENAME}" matchType="IsFile" negate="true" />
                    </conditions>
                    <action type="Redirect" url="{R:1}" />
                </rule>

just use
'rules' => [               ...             ],             'normalizer' => [                 'class' => 'yii\web\UrlNormalizer',                 'action' => null, // No redirection by default                 'normalizeTrailingSlash' => true, // This option is available in Yii 2.0.10 or higher             ],


Convert old WASM keyword to versions 1.6 and 1.7 ?

WASM keywords that were created for versions below 1.5 don't work with 1.5 and since the format was changed again for 1.7 I'd expect they are not compatible as well.
Expected behaviour
Offer a tool to convert keyword files to new versions.
Actual behaviour
I can't upgrade my Porcupine version because my keyword does not work.
Steps to reproduce the behaviour
Try to use a WASM keyword created before version 1.5 in version 1.5+.
I'm writing this here because I've tried to contact Picovoice via Email 2 times, but got no reply so far :-(. The Keyword is "Hey SEPIA" and was created around October 2018 for SEPIA open-source assistant. Any help would be greatly appreciated.

Hi there, GitHub issues are reserved for software issues or suggestions for improvement. For commercial license inquiries, please fill out the contact form on our website:

In the description, please let us know if you have purchased an annual service maintenance package and include the order number.


IPv6 addresses are not recognized

It seems that IPv6 addresses are not recognized properly:
>>> $w = new LayerShifter\TLDExtract\Extract()
=> LayerShifter\TLDExtract\Extract {#197}
>>> $res = $w->parse('2bf2:eaa0::7:314:5474')
=> LayerShifter\TLDExtract\Result {#205}
>>> $res->isIp()
=> false
>>> $res->isValidDomain()
=> false

@supriyo-biswas IPv6 should be passed in square brackets.
var_dump(tld_extract('[2bf2:eaa0::7:314:5474]')->isIp());
// bool(true)

That kinda defeats the point, isn't it? Recognizing IPv6 addresses natively would allow for verification of domains and IPs from other invalid values in a single step without requiring to pass them through filter_var() first.

Current TLDExtract has part of parse_url functionality, so when we deal with IPv6 it wants to receive it in square brackets (RFC 2732).
The next major release will drop this functionality, so you will can pass an IP addresses and domain names, while links will throw an exceptions.


Run cypress in test environment and still get changes picked up?

We are aware of the suggestion to use ENV['CYPRESS']  in development mode, but we find this this option non-functional/unappealing for a number of reasons.
We have a lot of differences between development mode and test mode: environment keys, storage buckets, firebase emulators etc. We want all these to be configured properly as they are for our rspec suite.
Is there an alternative to still get rails and javascript changes to reload cypress in test mode? It seems that would be first thing on the list of a gem like cypress-on-rails, but I don't see it anywhere. Perhaps I missed it?
The alternative is to make our own environment system so we can run with our test env config in development mode (and just to get code reloading), which seems like an anti-pattern.
All help appreciated!

@benlieb would you be open to having my ShakaCode team help you? If so, please book a call or email me.

@justin808 Just to confirm, is this a paid service your offering? Or just really good developer relations, lol?


db-backup

In your mariadb image, you included the backup script. Will this happen in the postgres image as well?

I think it would be fairly easy to shoehorn the same functionality! I'll add it to the list and get to it in a week-ish.

outdated ...


Feature/simulator redesign

I figured I might as well get you to look at what I have so far and start changing things that need work!
PR for #289 .
PR Status
READY FOR REVIEW
Ready for PR?

[x] I have updated the CHANGELOG with an entry including a description, a link to my profile, and a link to the issue ticket. This change is filed under an Enhancement or Bug Fix, is shown under the relevant release tag name, and is included in my PR branch.
[x] I have run lein run-lint to check for linting errors.
[x] Debugger replacing code on error
[x] Ace editor boxes need to update on every game move or state update
[ ] When the page changes the simulator window goes gray until an update occurs

Changes Proposed:

Updated simulator based on the newest designs
Simulator has a progress bar to allow viewing previous moves by the bot

Screenshot of feature:

@emilyseibert @oconn @dehli

@dehli This is probably ready for an initial review!

@dehli Check this out and see how it looks!

@dehli Lmk if this looks better to you - it should be a lot better

You were right! I think that I must have just had some other coding error when I wrote them the first time which made me think that the trailing variable was necessary.

Alright, I've updated based on your comments again!

Looks good to merge!


docs: fix typo in SSL functions

fixing a typo in some documentation.
CLA: trivial

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Just a note: you don't need to check the tests are added or updated box since there is no test cases update in this PR :-)

It  also seems worth a backport to 1.1.1

It also seems worth a backport to 1.1.1
Anything I need to do for this?

Anything I need to do for this?

Not necessarily. We can do that when this PR is being merged.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

Merged to master (9f44e96e24) and 1.1.1 (004f570821)

It looks like the travis error was just the usual timeout, so this one would be ready to merge.  @InfoHunter I guess that's you?


ZEPPELIN-3846. Code Completion is broken

What is this PR for?
Code completion is broken due to the shaded jar in zeppelin-interpreter-api, this PR fix this issue and also add unit test for it.
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?

https://jira.apache.org/jira/browse/ZEPPELIN-3846

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

LGTM

Merge if no more comments


Add DDS feeder

@AkhilTThomas

tests are missing as of now. Will be added in another commit.

Looks good, only some minors that we possibly could ignore.
Possibly we should also mention ddsprovider on top level README (https://github.com/boschglobal/kuksa.val.feeders/blob/feature/DDS_feeder/README.md)
I did start ddprovider natively as well as from docker and did test from dds2val directory and all succeeded.

@SebastianSchildt should we merge or would you like to see a config file?


:0: error: unable to load standard library for target 'x86_64-unknown-linux-gnu'

Following error produced by /usr/lib/sourcekitdInProc.framework/sourcekitdInProc on Swift 3.1:

:0: error: unable to load standard library for target 'x86_64-unknown-linux-gnu'

This is probably a mistake in the Swift packaging, since .frameworks aren't a convention on Linux.

This issue has been fixed on https://github.com/norio-nomura/docker-sourcekit-builder/releases/tag/311


Code Quality: Pin WinRT server object to prevent object reference being invalid

Resolved / Related Issues
The potential AV coming from FileTagsDatabase and LayoutPreferencesDatabase can be caused by CsWinRT who may not pin the server object, so that the object reference holding by the client can become invalid which can lead to access violation and crash the process.
This is a workaround to CsWinRT. Let's see if this resolves the COMException crashing issue.
/cc: @yaira2 @hishitetsu

LGTM


Clarify how Schema Objects require full-document parsing (3.1.1)

This is the next step in un-blocking OASComply.  Unlike the previous OASComply-related PR, this PR does not apply to v3.0.4.  What, if any, wording about document parsing needs to go in 3.0.4 is something we can sort out later.
The implied requirement might be surprising to some, but is an unambiguous result of normatively citing JSON Schema 2020-12.  Resolution of references to $id values that do not match the document's location (e.g. are resolved as URIs rather than URLs, including by noticing them in already-parsed content) was affirmed multiple times by TSC decisions during the work on OAS 3.1:

https://github.com/OAI/OpenAPI-Specification/pull/1977#issuecomment-571665656
https://github.com/OAI/OpenAPI-Specification/issues/2092#issuecomment-585881781
https://github.com/OAI/OpenAPI-Specification/issues/2159#issuecomment-601283402

Unfortunately, it seems like we never did get around to adding explicit wording highlighting this.  Which does not change the fact that a.) it was decided and re-affirmed, and b.) normatively citing JSON Schema 2020-12 mandates this behavior as the only possible way to satisfy its requirements.
There might be further changes related to this in the "Resolving Relative References in URIs", but I'll deal with that when I fix #3545, which covers both that section and "Resolving Relative References in URLs".
I also expect we'll want to add further guidance for both implementors and description authors on the Learn site.  I tried to keep the text here as minimal as possible while heading off the misunderstandings we've seen as folks implement 3.1.

Commit message:
JSON Schema draft 2020-12 includes numerous keywords that require parsing the entire document prior to deeming a reference unresolvable. This makes that more clear and outlines several approaches.
The practice of embedding OpenAPI fragments in other formats is deemed to have implementation-defined (non-interoperable) behavior, as the potential complications that might arise are not predictable.

This PR IMHO is very clear. I am very much tempted to merge this before the TDC call. @lornajane  , @ralfhandl thoughts?

@miqui this one has some pretty deep impacts so I'd rather leave it open for longer.

Discussed during TDC 5/2/2024 - It was decided we need 2 more TSC approvals.

@ralfhandl I've pushed a new commit with the following commit message:

Rework the guidance around fragmentary parsing.
This goes into more detail and uses "undefined" instead of
"implementation-defined" as the behavior is likely to be
incorrect (rather than just a different interpretationof an
ambiguous requirement), and may result in security concerns
as well.

I don't know that it's a simplification, but I think it's more explicit and should be easier to follow.  Please do continue to suggest simplified wording - it's something I struggle with sometimes.

Looks good 👎


the system cannot find the path specified

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/hello-world-tutorial/create
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you paste a screenshot of what you're trying to do/run?


KNX Project file "Upload failed" - "Unknown error"

The KNX project file some how doesn't work for me.

I can browse & select a KNX project file and enter the password
The processing starts (= wheel starts turning)
The processing passes the password check (if enter a wrong password I get an "Invalid password" pop-up message faster approx. 10 seconds)
After approx. 30seconds I get an "Upload failed" pop-up with the message "Unknown error".

What could be the issue?
Log Details
Logger: homeassistant.components.websocket_api.http.connection
Source: components/knx/project.py:108
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 22:57:16 (1 occurrences)
Last logged: 22:57:16

[548110576848] Error handling message: Unknown error (unknown_error) admin from <IP_ADDRESS> (Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36)
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 26, in _handle_async_response
    await func(hass, connection, msg)
  File "/usr/src/homeassistant/homeassistant/components/knx/websocket.py", line 100, in ws_project_file_process
    await knx.project.process_project_file(
  File "/usr/src/homeassistant/homeassistant/components/knx/project.py", line 110, in process_project_file
    project = await self.hass.async_add_executor_job(_parse_project)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/knx/project.py", line 108, in _parse_project
    return xknxproj.parse()
           ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/xknxproject/xknxproj.py", line 41, in parse
    project = XMLParser(knx_project_content).parse(self.language)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/xknxproject/xml/parser.py", line 63, in parse
    group_address_links = [
                          ^
  File "/usr/local/lib/python3.11/site-packages/xknxproject/xml/parser.py", line 64, in <listcomp>
    _ga_id_to_address[link] for link in com_object.links
    ~~~~~~~~~~~~~~~~~^^^^^^
KeyError: 'GA-2980'

Context

All HA updates are installed

I have a approx. 10 KNX entities set up manually in a KNX.yaml file which work fine

The connection type is 'Tunnel UDP'

KNX-bus connection works, e.g. the group monitor works

XKNX Version 2.10.0

KNX-Frontend Version 2<IP_ADDRESS>540

Connected to bus Yes

Individual address 1.0.1

The KNX project file was exported from ETS 5.7.7 (Build 1428; latest 5.x version)

Reimporting the KNX project file into the ETS (and opening it) works fine

Please add logs from HA, if there was an error during parsing it will be there.
Other than that, are you using a reverse proxy?

I added the log details and the connection type is 'Tunnel UDP'.

It seems there is an orphaned ComObjectInstanceRef left in your project. This has a single link to a GroupAddress Id that is not present in the project file (0.xml). I also couldn't find the description text of that ComObjectInstanceRef anywhere in ETS.
I can't say if this is an issue of ETS, MDTs application or if that is somewhat expected behaviour used for some ETS feature.


try to get scn cloud mask working

seeing a TypeError: 'ProteinVocabulary' object is not callable error

Solved in last commit. Also prepares for adding sidechain before going into refinement.


[BUG] ToBuilder  and Singular on Maps

Hello,
I believe this is not only for Maps, but my issues was the following:
I have an object that has a Singular map. With the builder I only add one item (singular makes that possible) to the map.
I build the object, then I turn it back to builder with toBuilder.
When I want to add one more object to the map I get UnsupportedOperationException, because the Builder created the object with SingletonMap, making it impossible the add any more items, even in ToBuilder.

I'm using Intellij here, and even if you pass more than one element to the collection/map it does create an Immutable variation of the variable.
So you can never change it no matter how many objects you add/put in the builder.
I believe this is intended as in you should have added everything you needed via the builder interface before calling the build method.
Maybe if you require to add more objects later you should consider using @With(er) to create a copy of the instance allowing you to change the @With annotated field.


[BUG] Cairo show font, utf8 issue

the original function,
void cairo_show_text(cairo_t *cr, const char *utf8), not a c char, but a utf8 array
and cairo_show_glyphs()
cairo_scaled_font_text_to_glyphs() etc.
but in rust bind code
    #[doc(alias = "cairo_show_text")]
    pub fn show_text(&self, text: &str) -> Result<(), Error> {
        unsafe {
            let text = CString::new(text).unwrap();
            ffi::cairo_show_text(self.0.as_ptr(), text.as_ptr())
        };
        self.status()
    }

    #[doc(alias = "cairo_show_glyphs")]
    pub fn show_glyphs(&self, glyphs: &[Glyph]) -> Result<(), Error> {
        unsafe { ffi::cairo_show_glyphs(self.0.as_ptr(), glyphs.as_ptr(), glyphs.len() as c_int) };
        self.status()
    }

convert &str to CString, and passed an standard c char*, not utf8 buffer array this will lead some issue in windows with CJK fonts.
I just approach the problem by convert to c_utf8 array
            let c_str = CString::new(String::from(c).as_str()).unwrap();
            let c_utf8 = c_utf8::CUtf8::from_c_str(c_str.as_c_str()).unwrap();
            if let Ok((glyphs, clusters)) = scaled_font.text_to_glyphs(x, y, c_utf8.as_str()) {
                // if index == 0 , then glyph not exist
                if glyphs[0].index > 0 {
                    return Some((glyphs, clusters));
                }
            }

I'm not sure what exactly the problem here is. Rust strings are guaranteed to be valid UTF-8 and CString adds the \0-terminator to the end. Any char * strings in cairo (except for filenames/paths) are required to be / guaranteed to be valid UTF-8 and \0-terminated.

Can you maybe provide a minimal testcase for something that fails and explain how it fails?

ok, only on windows, with CJK strings. In windows, the char* may not always utf8 array. I'll upload a sample later.

Why would it not always be UTF-8, in which API? From the Rust side, all strings would always be UTF-8 but also the c_utf8 crate assumes every string that goes over FFI is UTF-8.
This sounds to me like somewhere something else is broken or wrong. Waiting for the sample and explanation what exactly is passed where that causes things to break. Maybe it's a bug in cairo :)

try this demo.
cairo_font.zip
how to reproduct:
install font FZSTK.TTF that's in the source code root into windows system.
install vcpkg in windows
https://github.com/microsoft/vcpkg
and install cairo lib
vcpkg install cairo --triplet x64-windows
set environment var
SET PKG_CONFIG_PATH=[vcpkg root]/installed/x64-windows/lib/pkgconfig
cargo run --release
output.png and output.pdf will generated.
the output.png is right, but the output.pdf is wrong. some character gone.

only in windows, in linux, work fine.
the result:

output.pdf
t

And the change with c_utf8 from the first comment actually makes this work for you? I would expect that to behave exactly the same.

And the change with c_utf8 from the first comment actually makes this work for you? I would expect that to behave exactly the same.
not utf8 issue. I found that it's behave the same after I try again.

the pdf issue only occured when set slant to italic and weight bold.

Considering that the very same code works for PNG but not PDF, I'd consider this a bug somewhere inside cairo. There's nothing on the bindings side that could break this in such a way.
Can you report it to https://gitlab.freedesktop.org/cairo/cairo/-/issues/new ?


Add gradient

It would be useful to have some gradient implementation in the spec. Possibly starting with np.gradient.
Though it is worth noting that np.gradient returns separate arrays in a list, but this can be an issue if users want them combined (since this entails a copy of potentially large arrays). To alleviate this, it would be helpful to have some option to return them pre-stacked. Usually it is pretty trivial to split them up later if needed list(a_g).
Edit: Another possible option would be an out parameter. Though this may need some clarification given the current list behavior.

Here is the data on who implements it: https://github.com/data-apis/array-api-comparison/blob/main/data/join.csv. Looks like:

yes: NumPy, CuPy, Dask, JAX
no: PyTorch, TensorFlow, PyData Sparse

Of course PyTorch and TensorFlow also have functions to calculate gradients, and probably a lot more accurate than np.gradient - just under different names. Not sure if other libraries use the same algorithm as numpy or not, or what should be required there in case we indeed do want to add a gradient function.
np.gradient doesn't seem to be used at all in SciPy or scikit-learn, nor in NumPy itself outside of tests. So I'l curious, what you need it for @jakirkham?

It comes up a lot in image processing. Often as an input to other operations (finding local minima/maxima, feature extraction, segmentation, surface construction, optical flow, etc.).
There are a few direct usages of np.gradient in scikit-image. In some cases it appears they have custom kernels.

I'm weakly opposed here.
We do use this inside Xarray (inside the differentiate method) but there are many ways to approximate gradients, and np.gradient only implements one of them (2nd order finite differences).
Instead, I think this would be a good example of a "recipe" that could be implemented on top of the array API. Maybe it would be worth collecting many of these recipes and putting them into a library?

Wonder if there is some simpler functionality we could capture that would be a useful building block. For example diff or similar

I mean, the simplest version of finite differences for an array representing the values of a function y(x) is literally (y[1:] - y[:-1]) / (x[:1] - x[:-1]).

The outcome of a conversation in the 4 Aug consortium call was in line with the above discussion. gradient doesn't seem suitable for standardization, however it would be nice to have a good high-quality implementation of a finite differences function on top of the array API.
SciPy just deprecated its two derivate functions in scipy.misc. For np.gradient, it could be improved. There's other packages like numdifftools and findiff for this.
Image processing is probably where this is most often needed. It'd be great if someone could synthesize the best of what is already floating around and putting it up somewhere as a separate function.
@jakirkham can we close this issue? I don't think it's actionable here.

As this discussion hasn't been active in a while and the general consensus is against adding a gradient function to the array API specification, I'll close this out.


fixes #58 Running dao tests runs forever

Using first() instead of toList() to fix the issue. Please check the reason as to why usign toList() , the test never completes.
https://stackoverflow.com/questions/62499073/android-room-library-with-kotlin-flow-tolist-doesnt-work

Thanks, @raghunandankavi2010 for fixing this. Feel free to contribute again. Happy to merge this! 🚀

Thanks, @raghunandankavi2010 for fixing this. Feel free to contribute again. Happy to merge this! 🚀


Replace Puppeteer with jsdom

Since we don't do any automation with Puppeteer and only use it to parse/evaluate the DOM, jsdom is a much better option since it's a fraction of Puppeteer size and is much faster at what we're doing.

Done.


Sidebar falls out of sync when hard navigating to pages outside of the current personas scope

Steps to recreate:

select "Administration" view
navigate directly to a page associated with "Migration" view i.e. /controls/job-functions
expected results: the sidebar shows "Migration" view as selected view (with corresponding pages)
actual results: the sidebar for "Administration" view is show (see below)


sbt client (--client) does not print error output on repeated commands, just quits with "[error]..."

steps

Setup minimal project with sbt 1.5.5
with some main method that fails, lets say at runtime (but also happens sometimes with compiletime error output)
have the sbt server powered down
run sbt --client runMain <your Main class> and observe error output
run the same command again -- and just get the message that there was an error, but no details
run sbt --client shutdown, and repeat: the first time -- when the server boots up -- the error message is there, the second time run, it is not.

problem
Reproducible on my machine only on recurring runs of sbt --client as described above, the thin client sometimes doesn't produce any error output on runtime / compiletime errors. Sometimes it works (on a running sbt instance) and then stops working later.
Example output: -- first time, error gets printed, second time, it doesn't get printed.
should print
[error] (run-main-0) java.lang.AssertionError: assertion failed
[error]
[error] obj1 == obj2
[error] |    |  |
[error] |    |  IS(4,blub)
[error] |    false
[error] IS(3,hellooooo)
[error] java.lang.AssertionError: assertion failed
[error]
[error] obj1 == obj2
[error] |    |  |
[error] |    |  IS(4,blub)
[error] |    false
[error] IS(3,hellooooo)
[error]
[error]         at com.eed3si9n.expecty.ExpectyBase$ExpectyListener.expressionRecorded(Expecty.scala:41)
[error]         at com.eed3si9n.expecty.RecorderRuntime.recordExpression(RecorderRuntime.scala:40)
[error]         at bci.logic.Stuff$.StuffMain(Stuff.scala:21)
[error]         at bci.logic.StuffMain.main(Stuff.scala:7)
[error] Total time: 1 s, completed Oct 21, 2021 6:18:05 PM
snuc@snuc-NUC8i7BEH:~/sandbox/bct/bci-logic$ sbt --client runMain bci.logic.StuffMain
[info] entering *experimental* thin client - BEEP WHIRR
[info] terminate the server with `shutdown`
> runMain bci.logic.StuffMain
[error] Total time: 0 s, completed Oct 21, 2021 6:18:07 PM
snuc@snuc-NUC8i7BEH:~/sandbox/bct/bci-logic$

expectation
it should just print errors every time.
notes
Maybe the client pipe gets closed too early or not flushed?

Thanks for the report!

Noticed with joy people were working on this. However, using sbt 1.5.7, the problem is not solved for me.

In my case, I am not shown the problem of a build-level failure.
It is not, as the issue title would suggest, a problem of repeated running of sbtn — the server was shutdown before. running sbt --client compile the acquits with just error.
If I run sbt without client, I get the full problem (a stacktrace and stuff explaining that there are conflicting cross-versions in my case)

seems like I will have to fall back to 1.5.2 again — that version works for me.
Please let me know how to provide further info so that this can get solved :) Thanks for the work already done!

sbt 1.5.7, the problem is not solved for me

how about in 1.6.1?

This fix was not backported to 1.5.7 but it should work in the 1.6.x series.


Do not send un-managed volume infos to master

Currently agent sends all volume infos to master and master tries to match those to existing volume instances. It creates two mongo queries per volume plus pub-sub "overhead".
Fixed agent to send only info on volumes where the proper io.kontena.volume_instance.id label is in place.

Could the server's Rpc::NodeVolumeHandler#set_state now also be improved to fail and log a warning if the volume_instance was not found?

It could fail, but that actually makes me think why do we even send then the info at all? I mean if server does not find the VolumeInstance and it doesn't create it and instead fails with a warning, what's the point to even send those volume infos?

Even though it is the server that decides which VolumeInstances to create right now, and the /node_volumes/set_state RPC doesn't actually do anything at all, I think there are two reasons to keep the /node_volumes/set_state RPC:

Reporting the actual state of the volume back to the server, which could update the VolumeInstance

Discovery of Docker volumes that show up via the volume plugins on other nodes, after the first volume instance is scheduled, and the volume is created
This would require other changes to the volume management, though, since the volume labels do not propagate.

This is an important fix for 1.2, because we've seen cases where host nodes might have several thousand orphaned anonymous Docker volumes, and sync'ing those to the master on startup causes RPC issues.

This is an important fix for 1.2, because we've seen cases where host nodes might have several thousand orphaned anonymous Docker volumes, and sync'ing those to the master on startup causes RPC issues.

Makes me think whether there should be a worker on agent that does docker prune for the volumes periodically? Not part of this PR for sure

This is an important fix for 1.2, because we've seen cases where host nodes might have several thousand orphaned anonymous Docker volumes, and sync'ing those to the master on startup causes RPC issues.

Makes me think whether there should be a worker on agent that does docker prune for the volumes periodically? Not part of this PR for sure

AFAICT the agent has always done Docker::Container.remove(v: true), so I'm not entirely sure why those dangling volumes have accumulated.


[ISSUE #8108] Fix check MetadataProvider when enable acl2.0

Which Issue(s) This PR Fixes

Fixes #8108
Brief Description

How Did You Test This Change?

LGTM


1048 second attempt

Summary
References #1048
A clean-up of the previously attempted pull request; commits have been cherry-picked and merge conflicts fixed accordingly. All commits are in regards to fixing style violations.
Let me know if there's anything else I need to do!
How Has This Been Tested?
bundle exec rake was performed and all checks passed.
Types of changes

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

Checklist:

[ ] I've added tests for my code
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Awesome, thanks @phoebeclarke. I'm just about to board a flight but I'll take a look ASAP, unless someone beats me to it :)

Thanks for your work @phoebeclarke ! Really appreciate your effort.  Please can you check my comment? There are some problems with Style/LeadingCommentSpace violations fixes, unfortunately there are some utf-8 chars around that should be removed as part of the style fix.
@mattwynne feel free to add your review 😄

In general @phoebeclarke I think that in future having more smaller PRs would make it easier to review these changes, and merge the un-contentious ones :)

Thanks again for all your effort @phoebeclarke 👍


clifm: 1.17 -> 1.18

Description of changes
Changelog: https://github.com/leo-arch/clifm/releases/tag/v1.18
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thank you, done.
Unrelated question: Is it fine to reformat packages with the now official formatter in the course of updates like this? With a separate commit I mean.

Unrelated question: Is it fine to reformat packages with the now official formatter in the course of updates like this? With a separate commit I mean.

100% fine.

Please update the commit log message as such:
clifm: format with nixfmt-rfc-style

Of course. Thank you for pointing that out.


Deploy PUDL Docker image to GCE

Zane has successfully dockerized the PUDL ETL (#1605, #1606, #1610).  I tested building and deploying an image to GCE via Github actions #1603. Time to put the two together! Deploy the PUDL container to a GCE VM.
TODO:

[x] Create service accounts for action and VM.
[x] Create a VM instance for PUDL to run on. Associate with a service account.
[x] Add gcloud update container step to docker-build-push.
[x] Set env vars
[ ] Mount volumes

Things to consider

[x] The docker-build-push action pushes to docker hub but I tested the deployment using Google Artifact Registry. How can we deploy an image from docker hub? Use docker.io/catalystcoop/pudl-etl:gce-deploy url.
[x] Zane's been using a docker-compose.yml to specify the container configuration. Container Optimized VMs aren't very docker-compose friendly. What is the best way to specify container configuration for CO VMs? Seems like you configure the container via the VM settings.

[x] How to mount volumes?
[x] Set env vars?

The pudl_etl.sh script is running inside a docker container on a VM instance! The ETL finishes but there is no way to view the outputs because I haven't got volume mounting working. I'm running into permissions issues because when I add volume mounts, the pudl_in and pudl_out directories on the host are created as root and the catalyst user in the dockerfile does not have permissions to the pudl_in and pudl_out directories. I'm going to pause on the volume mounting issue because I think once our inputs and outputs are connect to cloud buckets there won't be a need for mounting volumes on the VM disk.
Changing the default image command was a pain in the butt. Each argument needs a separate --container-arg flag. The final gcloud deploy command:
gcloud compute instances update-container "$GCE_INSTANCE" \
            --zone "$GCE_INSTANCE_ZONE" \
            --container-image "docker.io/catalystcoop/pudl-etl:${{ needs.push_to_registry.steps.docker_metadata.outputs.tags }}" \
            --container-command "conda" \
            --container-arg="run" \
            --container-arg="--no-capture-output" \
            --container-arg="-p" \
            --container-arg="/home/catalyst/env" \
            --container-arg="bash" \
            --container-arg="./docker/pudl_etl.sh" \
            --container-env-file="./docker/.env" \

We could also just replace the unit test command in the dockerfile with the full etl bash script command.
Here are some helpful resources when working with Container Optimized VMs:

How to deploy container on VMs.
How to configure the containers.
gcloud compute instance update-container documentation.
Container Optimized VM Disk and file system overview.

sudo journalctl -r -u konlet-startup

Command for viewing container startup logs.

I don't think gcloud compute instances update-container restarts the container restarts if there are dangling containers that use the same image. I ran the image a couple of times via ssh without the --rm tag. I made a change to pudl_etl.sh but it wasn't reflected on the VM. I tried to delete the image via ssh but it told me a container was still using the image. I deleted all of the containers using the image and the image itself. I ran the gcloud compute instances update-container command and the new changes we reflected.
This isn't a bug just something to be aware of.


Sprite and label tweaks

@karimnaaji, could you comment on this?
I've change the use of GLvoid* and GLintptr to size_t as these are treated as offsets into the vbo when a vbo is bound during the call to glVertexAttribPointer [https://www.khronos.org/opengles/sdk/docs/man/xhtml/glVertexAttribPointer.xml]. And I find this conceptually more clear than to provide a pointer type just because the c api has no overloads :)
Edit: I'm also ok with GLintptr instead of GLvoid* for the getAttribOffset. Though I would prefer to use the types provided by c++ than to pull in GL headers into everything just for some typedefs.
GLintptr is http://en.cppreference.com/w/cpp/types/ptrdiff_t

Concerning GL types, we've had some discussions around it some time ago, and decided to use GL types if used for GL calls, but your point on headers totally makes sense.

the issue with the most :+1: to be resolved is :drumroll: https://github.com/isaacs/github/issues/18


Chapter Files

Hi Rene,
how hard would it be to generate one file per Chapter from the current .md files?
Thanks for an answer
Deutsch ginge auch ;-)

Ah ok I managed to start the project and I am going to generate my own


NPE in protocolConverter.asHover

The Java language server fails with
    at new e (/usr/share/code-insiders/resources/app/out/vs/workbench/node/extensionHostProcess.js:4:227228)
    at asHover (/home/aeschli/.vscode-insiders/extensions/redhat.java-0.4.0/node_modules/vscode-languageclient/lib/protocolConverter.js:63:16)
e.onUnexpectedError @ workbench.main.js:29

when hovering over a symbol without hover.
The reason is that it returns an empty hover object, without the 'contents' field.
I guess strictly speaking this might be a server issue, but IMO it would be ok to be tolerant in the language client and just ignore all hovers without the 'contents' field.
https://github.com/Microsoft/vscode-languageserver-node/blob/master/client/src/protocolConverter.ts#L176

Looking at the code, it seems the intention is to return null (or undefined) when there is no hover to be returned, not a hover without contents.
The problem is that onHover is defined to respond with Hover:
onHover(handler: RequestHandler<TextDocumentPositionParams, Hover, void>): void;

Maybe this should be:
onHover(handler: RequestHandler<TextDocumentPositionParams, Hover | null, void>): void;

@martijnwalraven I fixed this a couple of days ago to make it more clear. With be in the next release.
@aeschli I am closing the issue. I am against swallowing an incorrect hover since it will lead to new issue claiming that a server returns a hover but the UI is not showing it. It would be like asking the VS Code API allow Hovers without content and not showing it.
I would recommend that the Java server implements data checking on their end to better discover this mismatches.

@dbaeumer Thanks! It seems master still has onHover without null. Or is this in a separate branch?

(I also think onSignatureHelp has the same issue.)

Good catch. I only updated
https://github.com/Microsoft/vscode-languageserver-node/blob/master/protocol\src\protocol.ts#L1279
Will do the others as well.

Done.


Update events.yaml

title: Energy Drone & Robotics Coalition 2024
link: https://www.edrcoalition.com/energy-drone-robotics-summit-2024
start_date: 2024-06-10
end_date: 2024-06-12
city: Houston
country: United States
category: Conference

Thanks!

Ah that happens during the night here in europe. As you can see, it has already appeared in the Readme.md, that happens with every merge. But the website regenerates once a day

Got it! Thank you for the update. Appreciate all your help!
-Rachel
On Fri, Dec 8, 2023 at 2:46 PM Kimberly McGuire @.***>
wrote:

Ah that happens during the night here in europe. As you can see, it has
already appeared in the Readme.md, that happens with every merge. But the
website regenerates once a day
—
Reply to this email directly, view it on GitHub
https://github.com/WeeklyRobotics/robotic_events/pull/22#issuecomment-1847824165,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEJEOWIZJ5OOPUD6SY7QJ63YIN4CFAVCNFSM6AAAAABAKC7VUOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXHAZDIMJWGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Rachel McPherson
Customer Experience Expert
InnovateEnergy https://innovateenergynow.com
832-741-1993


Not being able to sort the calculation

Thank you for doing the coding,
I have tried doing it but in a different board (nodemcu v3) because that was the one available in the region, but I'm not being able to sort out the temperature result, of your code I only removed the rgb parts as the board doesn't have it and put a println for the analog readings, and it is as follow:
19:29:56.285 -> 15.00 -  reading 1
19:29:56.285 -> 12.00 -  reading 2
19:29:56.285 -> 13.00 -  reading 3
19:29:56.285 -> 13.00 - reading 4
19:29:56.285 -> 13.00 - reading 5
19:29:56.285 -> 170.12 - temp
Any possible direction of the problem source I would really be appreaciated!
Current code:
#include <SPI.h>
#include <math.h>

// the value of the 'other' resistor
#define SERIESRESISTOR 10000
// What pin to connect the sensor to
#define THERMISTORPIN A0
#define THERMISTORNOMINAL 10000
// temp. for nominal resistance (almost always 25 C)
#define TEMPERATURENOMINAL 25
// how many samples to take and average, more takes longer
// but is more 'smooth'
#define NUMSAMPLES 5
// The beta coefficient of the thermistor (usually 3000-4000)
#define BCOEFFICIENT 3950

#define SECOND 1000

void setup() {
  Serial.begin(9600);
}

float take_reading() {
  float average = 0.0;
  for(byte i = 0; i < NUMSAMPLES; i++) {
    float currentRead = analogRead(THERMISTORPIN);
    Serial.println(currentRead);
    // convert the value to resistance
    currentRead = 1023 / currentRead - 1;
    currentRead = SERIESRESISTOR / currentRead;
    average += currentRead;
  }
  return average / NUMSAMPLES;
}

float calculate_temp(float resistance) {
  float steinhart = resistance / THERMISTORNOMINAL;     // (R/Ro)
  steinhart = log(steinhart);                  // ln(R/Ro)
  steinhart /= BCOEFFICIENT;                   // 1/B * ln(R/Ro)
  steinhart += 1.0 / (TEMPERATURENOMINAL + 273.15); // + (1/To)
  steinhart = 1.0 / steinhart;                 // Invert
  steinhart -= 273.15;                         // convert absolute temp to C
  return steinhart;
}

void send_temp(float temp) {
  Serial.println(temp);
  delay(SECOND);
}

void loop()
{
  float reading = take_reading();
  float temp = calculate_temp(reading);
  send_temp(temp);
}

Any news? I am in the same situation, Trinket m0 isn't available in my country and nodemcu v3 is available, unfortunately I have no idea at coding, can you share your work?

I'm not familiar with this unit, maybe pins need to be set in analog mode first?


Send all locations in JET stacktraces

Hi! It seems Runic.jl was updated since the last time I ran it on the code. Can you run it once on the whole repository to fix the CI? Thanks!

I think that looks correct, it is indented as a continuation of the previous line. However, perhaps that expression would look better as a single line one:
frames = [(frame.file, frame.line, frame.func) for frame in stackframes]

I think that looks correct, it is indented as a continuation of the previous line. However, perhaps that expression would look better as a single line one:
frames = [(frame.file, frame.line, frame.func) for frame in stackframes]

Ok, I see the point. In that case I prefer the one-line indeed.

Looking good, thanks!


PR: Wait until console It's ready before executing code.

Fix 1 and 2 of #4856
Fix error of banner and prompt being deleted when executing in a dedicated console.
Now It looks as expected when running in a dedicated console:

@rlaverde does this mean that the code execution is now slower somehow, because of this timeout? or we take the same time, but simply display info at a later step?

does this mean that the code execution is now slower somehow, because of this timeout?

Only when opening new consoles, It'll wait until console is ready before executing the script, the timeout It's to avoid waiting too much time.

the timeout It's to avoid waiting too much time.

What do you mean by this? What happens if the timeout if finished?

What do you mean by this?

It the console is never ready (maybe for an error in the initialization), continuing this infinite loop could hang spyder, the timeout is the maximum time to wait for the console to be ready

What happens if the timeout is finished?

It'll return and the code will executed, Maybe deleting the prompt, or mixing the output with the prompt

@ccordoba12 I moved the code, although I don't think it will be a difference, because the wait (for the banner) is necessary for commands that print something.

I was not too happy with this approach because I found it a bit too complicated. So I developed #5301, which I find simpler to understand.


CardView in ListView

Hi,
could you please provide a possible implementation of the IOrdinateHandlerParentView for a listview?
It would be nice to see how somebody could handle the cardview inside the listview.
Thanks

@grammatnMcK hi
Unfortunately, i don't have it.
And honestly speaking think that it can be very difficult for implementing =/

https://github.com/AndreiMisiukevich/CardView/issues/79#issuecomment-429854389
hmm, we need to investigate it.. relay it's strange
I will appreciate if you create small sample project (the best solution would be fork my repo and create small sample in forked repo for faster debugging)

@grammatnMcK are you sure that IsPanInteractionEnabled ins't working?

But how to use it with panning feature -- have no idea

IsPanInteractionEnabled only :(

Hi Andrei,
either IsPanInteractionEnabled is true of false, I cannot scroll the listview to bottom or to the top when I try to scroll on then cardview. What exactly did you tested?

CardsViewInListView.zip

@grammatnMcK i tested with my sample) I will check yours

@AndreiMisiukevich  actually adding the following line doesn't fix the problem:
new CardsViewRenderer(); // or CardsViewRenderer.Preserve();

@grammatnMcK incredible.. I tested on your sample

@grammatnMcK do you still use IsPanInteractionEnabled=false?

@AndreiMisiukevich this doesn't work by just adding the new CardsViewRenderer.
Did you edit anything else too?

sorry.. ok with this change and the IsPanInteractionEnabled = false is working.

@grammatnMcK super! =) Enjoy


Minor links refactoring in README.md

Fixed "Writing" heading and converted "here" into links for clarity

Thanks so much!
@bundlerbot r+


byzanz: 0.2.3.alpha -> unstable-2016-03-12

Description of changes
ZHF: #172160
cc @NixOS/nixos-release-managers

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Builds, nixos x86_64 & aarch64. Seems sensible to go to an unstable release.

Result of nixpkgs-review pr 172912 run on x86_64-linux 1

2 packages built:

byzanz
yaxg


push to Azure Pipelines?

https://azure.microsoft.com/en-us/services/devops/pipelines/
See https://github.com/pydata/xarray/blob/master/azure-pipelines.yml

This is an alternative to Travis

I am fine with travis. What’s the advantage of azure?

No idea. Was following the dev of xarray when I created it and noticed it popped up there. AFAIA conda uses this and tests builds on windows etc. so if we are encouraging useage of the conda download (which we are) probably not need. Welcome to close.


[Azure ML SDK v2] Issue while registering new data asset and then getting it back right after registration

Package Name: azure-ai-ml
Package Version: 1.0.0
Operating System: Windows Server 2022 Standard
Python Version: 3.9.13

Describe the bug
When I try to register a new data asset using MLClient.data.create_or_update() and then try to get newly registered entity by name using MLClient.data.get() right after creation, it throws me an error like:
ValidationException                       Traceback (most recent call last)
---> 12 registered_data_asset = ml_client.data.get(name='new_dataset_name', label='latest')

File ...\site-packages\azure\ai\ml\operations\_data_operations.py:135, in DataOperations.get(self, name, version, label)
    126     raise ValidationException(
    127         message=msg,
    128         target=ErrorTarget.DATA,
   (...)
    131         error_type=ValidationErrorType.INVALID_VALUE,
    132     )
    134 if label:
--> 135     return _resolve_label_to_asset(self, name, label)
    137 if not version:
    138     msg = "Must provide either version or label."

File ...\site-packages\azure\ai\ml\_utils\_asset_utils.py:797, in _resolve_label_to_asset(assetOperations, name, label)
    790     msg = "Asset {} with version label {} does not exist in workspace."
    791     raise ValidationException(
    792         message=msg.format(name, label),
    793         no_personal_data_message=msg.format("[name]", "[label]"),
    794         target=ErrorTarget.ASSET,
...
    700         error_type=ValidationErrorType.RESOURCE_NOT_FOUND,
    701     )
    702 return latest

ValidationException: Asset new_dataset_name does not exist in workspace workspace_name.

However, after couple of seconds get method will work.
Seems that data asset creation is asynchronous and there is a small time lag between data asset creation and ability to get this data asset from the workspace.
CLI version works fine, but it az ml data create... takes significantly more time than SDK version.
To Reproduce
Steps to reproduce the behavior:
dataset = Data(
            path='azureml://datastores/...',
            type='uri_folder',
            description='Test',
            name='new_dataset_name',
        )
dataset = ml_client.data.create_or_update(dataset)

registered_data_asset = ml_client.data.get(name='new_dataset_name', label='latest')

Expected behavior
The last command in the sequence should finish successfully and return registered dataset details.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Service Bus:0.11162622,Storage:0.06302417,Tables:0.0483478'

@azureml-github

@glebrh thanks for reporting this. We're working on a fix now, in the mean time you can add a small (1s) sleep between create and get


[ALLUXIO-2039] Fixing mounting of Swift directories.

Addresses: https://alluxio.atlassian.net/browse/ALLUXIO-2039

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

LGTM

LGTM
As a separate note, maybe we should come up with better integration tests for various UFSs.

@jsimsa Thanks for the fix! I left a few questions.
Also, what is the root cause for the mounting issue?

@jsimsa I left a minor comment, I don't feel strong about it, I will leave it up to you. Otherwise, LGTM

@apc999 @gpang addressed your comments, PTAL

Merged build finished. Test PASSed.

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

LGTM

LGTM


Incomplete handling of complex f-strings

Pasta, which is required for the tf_upgrade_v2 script of Tensorflow, throws an error on a specific f-string. I believe support for f-strings has been added recently, but this seems to be incomplete.
This concerns the following minimal code example
import os
path = ["path", "to", "file"]
c = f"/{(os.sep).join(path)}"

When running tf_upgrade_v2 (pip3 install tensorflow==tensorflow==2.0.0-alpha0)
Related: https://github.com/google/pasta/pull/61, https://github.com/google/pasta/pull/58, https://github.com/google/pasta/pull/59, https://github.com/tensorflow/tensorflow/issues/26486

Oops, I saw the bug in TF first. It's fixed in f26107d. Thanks!


Developed setting to loop search from the beginning or end of document in Find Widget

This PR fixes part of #84304.
Developed setting which controls whether the search automatically restarts from the beginning (or the end) when no further matches can be found.
Default: true (to mantain the same behavior as before).

Here you can see that, when the value of this setting is false, 'Previous' and 'Next' buttons are sometimes disabled in Find Widget so one cannot go back or forward to further matches.

This setting is under Find group of settings and then do not affect Diff Editor. Maybe we can create another setting, only for Diff Editor.

@rebornix When you have a free time can you take a look on this? 😃
If you agree with these changes I will keep testing on Insiders.
After that I'll probably make a new PR developing an equivalent setting under Diff editor group of settings, which will affect looping navigation on next/previous changes, according to the image below:

It makes sense to me, thanks for your help!

@rebornix I'm happy you appreciate! :smile:
Could you merge this into master, please? Or you'd prefer someone else giving a second opinion before doing so?
When you merge this I'll keep testing on Insiders.
Thanks in advance!

@rebornix any news on merging this?
I need to test a little bit more on Insiders before endgame...

I'll take a look in debt week of Apri.

@rebornix thanks! :smile:

@GustavoASC thanks for your contribution! It would be in tomorrow's Insiders.

Hi @rebornix ! Since it's a new feature, could a GIF of this new setting be added to 1.45 release notes? There are people looking for this VSCode behavior and I'm afraid they won't have knowledge about this.
E.g.: https://github.com/microsoft/vscode/issues/46504 and https://stackoverflow.com/questions/57158475/disable-search-wrap-in-vs-code
Thanks in advance!


maximum sum increasing subsequence | CPP/C++ | Dynamic Programming | GSSoC'21

💥 Proposal
maximum sum increasing subsequence in CPP/C++
Have you read the Contributing Guidelines on Pull Requests?
YES

/assign


Add IDEA code style profile to the repo

https://github.com/pgpainless/pgpainless/commit/39e87f9ce48f8d01a638e19e3cbed00a5dc6f097#commitcomment-49355117

I added a .editorconfig file in the root directory. Let me know if your IDE automatically picks up on this and if it works :)

@vanitasvitae Thanks. it works great!
One more question. How do you think could we add the following header to be able to apply it automatically?
/*
 * Copyright 2018 Paul Schaub.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

I'd have to do some research on how to do that.

It appears that adding headers is not easily possible without additional tooling.


layer-swipe-js not support arcgis js api 3.18

hi, @driskull , this layer-swipe-js seems not support arcgis js api 3.18?
i found there is a LayerSwipe widget in esri/dijit module in 3.18, but the code is mixed, but i want to update some function to meet my need

Hi @dyh333, yes it's supported in 3.18. It just hasn't been updated on GitHub for a while. I'll update the code here.

hi, @driskull , when this layer-swipe-js will support arcgis js api 4.x?

@dingkesong there should be a widget in the API at some point to do this but not an external widget on GitHub. I don't have a date for you just yet. sorry!

@dingkesong there should be a widget in the API at some point to do this but not an external widget on GitHub. I don't have a date for you just yet. sorry!
thank you


Razor components/Blazor doesn't iterate correctly inside a for loop of a List

Describe the bug
I made a simple razor component that just renders a list of strings in buttons.  Each button is clickable so when I click one, I add the text "AddedText" to the string inside the button.
This doesn't work since the counter variable always sets to the last position, throwing an out of range exception.
This seems to happen only in the onclick binding, because the list renders correctly in html.
To Reproduce

Using the latest ASP.NET CORE 3 PREVIEW 2
Run this code (this is just a modified Counter.cshtml from the template)

@page "/counter"
@for (int c=0;c<stringlist.Count;c++)
{
    <button onclick=@(()=>addText(c))>
        @stringlist[c]
    </button>
}

@functions {
    List<string> stringlist = new List<string>();

    void addText(int index)
    {
        stringlist[index] += "AddedText";
    }

    protected override void OnInit()
    {
        stringlist.Add("STRING1");
        stringlist.Add("STRING2");
        stringlist.Add("STRING3");
        base.OnInit();
    }
}

Expected behavior
I expected the button when clicked, to change the string in the correct position of the List.
FIX
To circumvent this issue, I've found that if I use an intermediate counter variable in the FOR LOOP it works,  so here's the FOR LOOP change:
@for (int c=0;c<stringlist.Count;c++)
{
    int c2 = c; //Intermediate counter
    <button onclick=@(()=>addText(c2))>
        @stringlist[c2]
    </button>
}

This way the loop not only renders correctly, but reacts correctly to the click event modifying the string in the List.

@Andrzej-W Thank you!! I changed the description of the issue, now looks better!,   also I've found in your posts that this isn't a bug per se, but how C# behaves in these scenarios with binding and lambdas.   Now I don't know how to proceed here, I guess someone will set this as "Not a bug".
Thanks again, regards!

Assuming you agree that this is not a bug, you should be able to close it
yourself.
Even though there is probably nothing to fix here, you brought some
attention to a confusing C# feature and it's relation to Blazor, and
hopefully made the solution more searchable. So thanks for your
contribution!

Hopefully this behaviour will be documented soon:
https://github.com/aspnet/Blazor.Docs/issues/371
I have created a few helpful examples in this issue - you can take a look.
As @dmorrison42 said you can close this issue yourself. You should see the Close issue button to the left of the green Comment button.


inference a batch images in one time

how can i using efficientnet to inference a batch images in one time, not circle inference in every image.

What do you mean? Something different from normal pytorch batching?

If you use Dataloader, the simplest way is to change the batch_size to the size of your dataset. Be sure that you have enough memory to do that.


chore: update project structure to 201e2aa0

Pull Request Test Coverage Report for Build 580722463

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

Totals

Change from base Build 359069531:
0.0%

Covered Lines:
22

Relevant Lines:
22

💛  - Coveralls


It doesn't work

Hey ! What's your Spotify and Spicetify version ?
I would need more info, all of the following should be ran in DevTools while on Spotify (when you're playing a song)

Spicetify.Player.data.item.metadata.artist_uri what's the output ?

You should have something looking like this spotify:artist:xxxxxxxxxxxxxxxxxxxxx

Now save the xxxxxxxxxxxxxxxxxxxxx part from 1. and drop it in the following command : await Spicetify.CosmosAsync.get("https://api.spotify.com/v1/artists/xxxxxxxxxxxxxxxxxxxxx"), replacing xxxxxxxxxxxxxxxxxxxxx by the ID from part 1.

If everything went well you should be able to see genres in the output, like this :

What's happening on your end ?

I pushed a new update that fixes https://github.com/Vexcited/better-spotify-genres/issues/7.
Your issue may be related to it, can you please try to update the extension and see if it changes anything ?

Closing as stale. Feel free to reopen if the issue persists on your end.