Feature Request: Team public folders

I'm sorry if this has been suggested before elsewhere; I searched but didn't find anything.
I think it would be really nice if there was a way for teams to have a public folder in addition to the current private folder under /keybase/team/.
This would allow teams to have a homepage served at keybase.pub and would allow them to put out public documents as well.  (I'm mostly thinking of larger open-source dev teams when I suggest this, but I'm sure other teams would have a use for this.)
With that said, I'm not sure where this public folder would go.  The most straightforward way to structure it (I would think) would be to have:
/keybase/team/public/[teamName]
/keybase/team/private/[teamName]
But that wouldn't be easy to switch to since we've already got:
/keybase/team/[teamName]
for private folders right now.
Is this something that you would be interested in doing?
Edit: grammar

On our internal roadmap. We would also need a notion of a public team whose membership is visible to all, to verify writers and writes.  Please :+1: if your team also would use such a feature. Thank you!

Thanks so much for the quick response; very happy to hear this is on the internal roadmap!

We would also need a notion of a public team whose membership is visible to all, to verify writers and writes.

I guess I didn't realize that teams are currently completely private (makes sense, just didn't think about it).  I'm assuming this means in the future there will be two types of teams (public/private) and you'll have to specify which type of team you're creating during the creation process?
If so, I'm assuming there will be restrictions on whether or not a team can switch to private (or public) after they've been created.
Maybe that's too much to ask at this point.  I realize there's probably still very much to do before the exact behavior of that part of the system can be nailed down.

This seems like a good use case for subteams methinks.
You could have a few people join a subteam for outward facing public folders, and the public wouldn't know of all the members of the parent team.

Yeah, I think that would work pretty well.
The only issue I see with that is that it might be an annoyance for "public" teams to have to create a sub-team and add everyone to it if what they really want is for the whole team to be public.
Which also means there'd be no way for a team to prove that there are no "private"/"hidden" team members.
Not sure if either of those are worth being concerned about at this point though.

there'd be no way for a team to prove that there are no "private"/"hidden" team members.

The point is that I, as a consumer of the data, need to know that a member of the public team signed the data. If I see data signed by an unknown user, I just ignore it. No need to prove “no hidden members”

That's true, and that totally makes sense.
I really just meant that point to be considered from a transparency perspective instead of a security perspective.

Any update on this? Would love to use public team folders for a project!


Clarify caching inputs

The documentation mentions shared-key and cache-key as an input, but the latter doesn't seem to exist. I've changed the wording to refer to prefix-key and shared-key instead.

Anything more I can do here? I'm assuming the failing CI tests are unrelated to my docs change.

Indeed. Thanks for the ping!
It's an interesting failure due to GitHub Actions caching that caused an older commit that should have failed CI to incorrectly pass CI by using a previous run's cache:
https://github.com/obi1kenobi/cargo-semver-checks-action/pull/79
After that PR merges, this CI should be green again. I'll keep an eye on it now and make sure it's all good.


Tests for high number of bound variables (JAVA-515)

On my machine, the test PreparedStatementTest.prepared_statements_with_less_than_65k_parameters_should_be_accepted runs forever, please do not merge yet.

CASSANDRA-8276 is fixed in 2.2.0, what about merging this ticket in 3.0? @pingtimeout @olim7t any thoughts on that? I can take care of the remaining work.


refactor optimize the agg function of count(1)

Proposed changes
after PR #12803,
like count(*) in dup table will use the segment static,
now add count(const value) also use that optimization.
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.05 seconds
stream load tsv:          430 seconds loaded<PHONE_NUMBER>9 Bytes, about 165 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          60 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230417095803_clickbench_pr_130089.html

run buildall

run p1


Dark theme does not seem comfortable. What can I do?

Hello everyone,
I started Jetpack Compose 🥳 but My Android Studio has Dark theme. Elements are not read when I preview.

How can I prevent this?
I would be glad if you could help.

You can add showBackground = true argument to @Preview annotation

Closing as this is not an issue with the samples in this repo and would be better suited for Slack/StackOverflow.


Change when on_mouse_up closure is called

Context
The feature will be used when a developer wants to call a function on a widget when a mouse button is released while hovering it.
Problem description
I'm using the on_mouse_down closure in MouseHandler to know when the user has pressed the mouse on a widget, I use that to tell a handler widget that "this is now being dragged-and-dropped". Then the user drags the mouse and lets go of the mouse on another widget, this tells the handler widget that "it was dropped on this widget".
Currently to achieve the "release" behavior I'm using on_mouse_up_global. This calls a function on every widget that has it any time the user lets go of their mouse button. However, this is wasteful as I then have to calculate if the mouse is over the widget. What I actually want—and what I expected on_mouse_up to do—is to call a closure only on the widget the mouse is released on.
Solution
I would suggest changing on_mouse_up to work as I expected (to be called whenever the mouse is released when hovering a widget), because it then mirrors the functionality of on_mouse_down, like its name suggests.
The current functionality of on_mouse_up could be renamed to something like on_mouse_up_sticky, to clearly show that this is a "sticky" mode that will call on the widget it was pressed on once the button is released.
Examples and MockUps
Here is an example of how I imagine it working:
self.name("MyWidgetView")
    .width(50.)
    .height(50.)
    // This closure is called when the mouse button is released
    // while hovering the widget:
    .on_mouse_up(move |states, p| {
        states
            .get_mut::<SlotState>(id)
            .mouse_action(MouseAction::MouseReleased, _);
    })
    // This closure is called when the mouse button is released after
    // first being pressed while hovering the widget:
    .on_mouse_up_sticky(move |states, p| {
        states
            .get_mut::<SlotState>(id)
            .mouse_action(MouseAction::MouseReleasedSticky, _);
    })
    .child(MouseBehavior::create().enabled(id).target(id.0).build(ctx))

@lukors I would suggest to only have one mouse_up callback and to remove only the restriction that the element must be pressed before.

Done.


[WIP] Run the legacy test suite on windows

Close to avoid triggering useless appveyor builds.

Will reopen when ready to review.

Ready to be tested by appveyor.

:+1: @OznOg @dawagner @cc6565 please review

:-1: but great job :)

Please resubmit against the master branch.

:+1:


Trying to disable text editor with shortcut opens a new tab

Originally reported on Google Code with ID 650

What steps will reproduce the problem?

Write any umple code so we can see a diagram
Press ctrl+t to hide the text editor

What is the expected output? What do you see instead?
Expected: text editor hides.
Actually: a new tab opens.
What version of the product are you using? On what operating system?
Latest UmpleOnline, Ubuntu 14.04.
Please provide any additional information below.
I am using Chrome Version 38.0.2125.111 (64-bit). Ctrl+t is a Chrome short-cut for
opening a new tab, which interferes with Umple's shortcut.

Reported by gabriel.tapuc on 2014-11-07 17:17:38

Control-t works fine in Mac OS (on Chrome and other browsers. Keep that.
Detect OS and make it something else that doesn't conflict on Linux and Windows, (e.g. something else with the 't'). Check the other shortcuts too.

I can take this issue.

great.

Make sure you leave the key bindings on Mac OS in tact. The key is to use key binding combinations that work on other platforms (e.g. control-option-t) might be a good choice in Windows and Linux instead of (control-t). The latter would still work on Mac. This needs testing in multiple browsers on multiple OSes. And all the key bindings need changing. key bindings are set in two separate places ... one for the text pane and one for the graph pane.

The two key bindings that failed are control-t and control-n. It looks like this issue isn't because of Windows/Linux since it works in Firefox for both Windows and Linux. Instead, it fails in Google Chrome because the two key bindings are reserved.
For control-n, I was able to get it working as control-alt (option)-t in both Windows and Linux. However, while I was able to get ctrl-alt-t working in Windows, in Linux (Ubuntu) it will open a terminal rather than be captured by the javascript.

I feel like working with ctrl + t or cmd + t is just a bad idea. This shortcut is commonly used in many applications. What about using cmd + h or ctrl + h. It'd take some research to confirm, but I don't think those key shortcuts are used. It'd take some investigation, but I feel like using t is just too common and will run into problems.
This is horrendous to look at, but should prove useful: Keyboard Comparison.

I would prefer to leave the 't' for text. Yes, lots of apps use t, but for many different purposes.

Because of ctrl + alt + t opening the terminal in Ubuntu, and ctrl + shift + t also being reserved in Chrome, would need to use ctrl + alt + shift + t in Ubuntu for Chrome (or not use t).
ctrl + h doesn't work in Chrome because it is reserved for history, but ctrl + alt + h works for Windows and Ubuntu.

I suggest using ctrl on the Mac and ctrl alt shift for the others. Hiding the diagram with ctrl-d or command-d on the mac will be much more of a common operation than hiding the text. And as long as the tooltip appears correct, all should work.

Which tooltip are you referring to? I understand there is the tooltip for diagrams, but I haven't seen any
related to the shortcuts.
Also, should I create a user manual page for all of the shortcuts by OS?

All the commands operable by keyboard shortcuts have menu items. So go to options menu and hover over each menu item, and you will see the various tooltips with the d t a m e g s keys.
You are right, The webpage http://cruise.eecs.uottawa.ca/umple/UsingUmpleOnline.html should be updated to include the key bindings. Inputs to the manual are in build/reference
The manual is built using ant -Dmyenv=local -f build.umple.xml packageDocs


Add Rating Feature for Anime

-Introducing a rating feature for anime would enable users to share their opinions and may contribute to the platform's community-driven content evaluation systems maybe.

Implement moderation tools to ensure the integrity and accuracy of anime ratings, such as flagging inappropriate or misleading ratings.
In conclusion, I believe that adding a rating feature for anime content would significantly enrich the user experience

I believe this is something you should implement in your own application.
This is only a scraper adding such functionality to this api will require it to have its own database and user authentication.


Question: Locking across Heroku Dynos

My Heroku App is running on multiple dynos. Is this package able to lock across dynos?

I don't know. I only maintain the package because the author disappeared, and I did not have that use case.  In fact, the quality is so low and the code so unreadable that I would advise you not to use this at all.

@rogierschouten , thanks for your feedback and for maintaining this package. I'll explore alternatives.


Add Calendar to the Core

Surprise! For concrete5 version 8.3 we are adding our calendar add-on to the core for all to use. This add-on has many improvements even over the add-on published in the marketplace! Including:

The ability to add multiple calendars to your site
Join pages to calendar events
Calendar events are a separate data model from pages.

Detailed permissions at the calendar level.
Workflow integration with calendar events.
Version control for calendar events (!)
A powerful recurring event model that works even with event versioning.

All this and a lot more.
Important
Upgrading with this is not yet working. You must test this on on a clean concrete5 install. This is not ready to merge.

*salivate*
Utmost thanks!

@aembler
Adding the Document Library and this calendar are very useful additions.


[Question] Custom metric type

Hi,
Is there a way to define custom metric that would output dictionary or tuple?
I need to create a metric that will output Scorer values aggregated by the value and display occurrence of each value.
The values in my case are strings.
It seems that the only output type from the "metric" that is supported is "float".
Here is example "metric" which is currently not supported:
@metric
def item_count() -> Metric:
    """
    Currently not working. Inspect expects a float value for the metric.
    """
    def metric(scores: list[Score]) -> tuple:
        count_dict = {}
        for score in scores:
            score_value = str(score)
            if score_value in count_dict:
                count_dict[score_value] += 1
            else:
                count_dict[score_value] = 1
        return tuple(count_dict.items())
    return metric

Thank you.

cc @dragonstyle

What version of Inspect are you using? Metrics in recent versions of Inspect are allowed to return any of the following types:
str | int | float | bool
list[str | int | float | bool]
dict[str, str | int | float | bool | None]

The elements of the system downstream should automatically handle metrics that return lists or dicts, hopefully behaving as you'd expect them to.

I'm using inspect_ai==0.3.42, let try the latest version. Thank you.

I'm using latest inspect, version: inspect_ai==0.3.42. I have custom scorer that returns following class (example):
       # Debug
        scorer_value = "test-value"
        return_score = Score(
                value=scorer_value,
                answer=state.output.completion,
                explanation=scorer_explanation,
                metadata=metadata,
            )
        return return_score

The "value" is properly reflected in the json and in inspect view, however if I use custom metric, by the time Scorer.value reaches it, the value is converted to float and is showing as 0.0. Is there a way to send string values to custom metric for processing/

I'm reverting the original fix to this as it caused additional regressions elsewhere. I will attempt to fix again today.


bugfix tests/rackhd20/test_rackhd20_api_pollers.py

This is a correction for the pollers test.
@hohene @johren @jimturnquist @diontje @larry-dean

+1

@keedya

The user field is not always present.


Make spline_knots a parameter

Making the splot Knots a tunable knob.
Default is 5, if 0, returns model without this corrector.
@TomWagg, does this work for you?
Question: Why is the  "full_model_normalized" a "self." object, and the other models are not?
This will close #54

This will solve #54

Good call on the none vs 0

I got the None conditions working. One caveat: without the spline fit some of the flux values seems to be negative, so we get warnings about converting them to magnitudes, waiting to talk to discuss what to do about this.


Add likert question clustering

mapping function agree = 1, neutral = 0 etc
umap with cosine similairity
HDBSCAN for clusters
heat map plot to visualise what the question clusters mean

Use this data: https://data.austintexas.gov/dataset/Community-Survey/s2py-ceb7/about_data
10K rows and public domain


Excalidraw support

If you like excalidraw, and whould like to implement excalidraw as alternative whiteboard enging for this plugin, please rise an issue about it.

It would be great to be able to use Excalidraw as the whiteboard engine. Tldraw has some features missing that I consider basic, like two-headed arrows or library objects.

https://news.ycombinator.com/item?id=29266164 Excalidraw based on canvas, is not good at rendering wikitext.


Integrate ChatGPT

I can work on integrating this for you I am going to leave some fields anonymous such as API keys but all the code will be there

It was decided to not integrate chatgpt with the discord bot


Improve README.md

The readme file should have enough instructions for people to get what graphql-tools is for, and to get started without having to refer to the documentation.

Fixing in #183


To-do items toward a minimum viable product

[x] replace assert False, "TODO" with raise NotImplementedError (the mypy constraint no longer exists).
[x] creation functions arange (from NumPy/CuPy)
[x] creation functions empty (from NumPy/CuPy)
[x] creation functions eye (from NumPy/CuPy)
[x] creation functions from_dlpack (from NumPy/CuPy)
[x] creation functions full (from NumPy/CuPy)
[x] creation functions linspace (from NumPy/CuPy)
[x] creation functions ones (from NumPy/CuPy)
[x] creation functions zeros (from NumPy/CuPy)
[x] data type functions can_cast (from NumPy)
[x] data type functions finfo (from NumPy)
[x] data type functions iinfo (from NumPy)
[x] data type functions result_type (from NumPy)
[x] data type functions astype (from Awkward)
[x] creation functions empty_like (from Awkward)
[x] creation functions full_like (from Awkward)
[x] creation functions ones_like (from Awkward)
[x] creation functions zeros_like (from Awkward)
[x] manipulation functions broadcast_arrays (from Awkward)
[x] manipulation functions concat (from Awkward)
[x] searching functions nonzero and where (from Awkward; the 1-arg and 3-arg cases)
[x] array object __dlpack__ and __dlpack_device__ (only for rectilinear arrays)
[x] array object __getitem__
[x] array object __len__ (and declare it to be a Sequence or Sized or something)
[x] array object __setitem__
[x] array object __iter__
[x] array object __init__'s copy argument should do a deepcopy
[x] manipulation functions expand_dims (from a slice)
[x] manipulation functions squeeze (from a slice)
[x] indexing functions take (from a slice)

This would reduce our 51 TODOs down to 20 (by filling in 31).

Now
% fgrep -r 'NotImplementedError("TODO ' src | wc -l
33

The remaining 8 check-boxes should bring it down by 7, down to 26 TODOs. (The accounting changed when I converted assertions into NotImplementedError.)

Now
% fgrep -r 'NotImplementedError("TODO ' src | wc -l
23

Yes, I got rid of a few others along the way (arguments in ragged.array functions).
All the rest of the NotImplementedError TODOs are in issues. They'll be addressed one at a time, as needed.


[v2] Rewrite Slider without mdlUpgrade

Currently, the Slider component requires the javascript code from material design to work properly. We should rewrite the component as a true React component without that dependency.
It should also move in its own directory, with its tests. See Badge as an example.

Try the new implementation using the new MDL v2 code: https://github.com/google/material-design-lite/blob/master/src/slider/slider.js


Update crition names to have correct level (A, AA)

Some of the levels were incorrect for some of the criterion so I've gone through and updated these in this PR.

@BNewing unfortunately there's now a merge conflict – would you be able to rebase?
I'm happy to sort this out for you on your behalf if you prefer.

Rebased :) thank you for looking through these PRs!


ARROW-8394: [JS] Upgrade to TypeScript 4.0.2, fix typings for TS 3.9+

Upgrades to TypeScript 4.0.2
Fixes typings for TypeScript versions 3.9+

Closes https://issues.apache.org/jira/browse/ARROW-8394

@kszucs recent node versions changed behavior of one of the stream properties we're testing, so node 11 will fail now.
I tried updating all the places I saw node 11 being specified, but I'm not sure how to update the Ursabot / AMD64 Debian 9 NodeJS 11 build. Is this something we can update and include in this PR?

Thanks for the update! I'll handle the ursabot builder.

Thanks Paul! I should have time to take a look over the weekend

@ursabot build

The Ursabot / AMD64 Debian 9 NodeJS 11 builder is no longer enabled in favor of Ursabot / AMD64 Debian 9 NodeJS 14, so it should be good to go from the buildbot side.

@trxcllnt Is this ready to merge?

@nealrichardson yeah I believe so

Thanks a lot for your work on this, @trxcllnt! Really excited for this to land in a release.


[Bug (minor)] <IP_ADDRESS>: Burn optimised sequence is sub-optimal

I have no idea why but:

Burn sequence of 2 images and 2 raster elements is sub-optimal on first element;
Burn sequence is not always the same

I suspect that the starting point for the burn is not being initialised properly.

In the above image you can see that it goes first to the stars image and only does the raster words at the top at the end - I would expect it to start with the words at the top and then move downwards to the next element.
I didn't capture the simulation as an image (and haven't managed to recreate it), but having done an actual burn with the above sequence, when I opened the simulation window the second lower set of words was shown as burning first.

Here is an example of the second issue.
I started a burn for two of these, then deleted one and opened simulation.
What is even weirder is that not all the burn paths for the deleted item are in the simulation - only some of them.
Before delete:

After delete:

Simulation:

Acknowledged

These sorts of intermittent problems can be very difficult to track down. IMO it is better to prevent code from having incorrect input and raising an exception (which shows exactly what was wrong and why) than to allow it to happen and give false output and then have to try to track down where it is going wrong. That is where strong typing comes in - be explicit about what you expect to be passed into a method and let there be show-stopping exceptions when it doesn't happen. (David was a talented coder, but in this respect he has left a legacy of unreliable code.)

On the first matter - do these operations contain by any chance a definite coolant setting? This would cause this behaviour as the operations would not be able to merge. E.g.

With coolant setting
Without

No - I have no coolant control and haven't ever set this to my knowledge. But I can check the actual file.


Fix ANSI colors in chrome and tracy traces

Objective
Fix ANSI colors in chrome and tracy traces, at the cost of losing ANSI colors in logs to stdout when trace_chrome or trace_tracy features are used.
Fixes #3563
Solution
Add .with_ansi(false) to the "fmt layer" when trace_chrome or trace_tracy are enabled.
I'd appreciate someone checking the tracy output, as I wasn't able to test that (except to see that bevy still builds with that feature)

ok in tracy

Note: adding the trace_chrome or trace_tracy feature also affects logging output to stdout in other ways. This PR doesn't address that.
With:
2022-01-08T00:48:40.761898Z  INFO bevy_app:frame:stage{name=PostUpdate}:prepare_systems:system{name="bevy_diagnostic::log_diagnostics_plugin::LogDiagnosticsPlugin::log_diagnostics_system"}: bevy diagnostic: frame_count                     :   98.000000  (avg 98.000000)

Without:
2022-01-08T00:49:02.431899Z  INFO bevy diagnostic: frame_count                     :   75.000000  (avg 75.000000)

Closing because I can't reproduce the issue anymore


[Discussion]: Any plans to make Flow more interpopble with TS ecosystem?

The problem is that flow doesn't have any tool to convert TS to Flow even though most TS features are now possible. This breaks the formula of upstart success: performance x interoperability = success that we see in projects like Bun(to Node), Go+Rust+Zig(to C).
Right now anyone using flow has to go through the painful process of converting the TS definitions of almost any NPM module they want to use by hand and then maintaining them. I think the fact that flow is written in OCAML which is reasonably fast and portable compared to TS is quite a big win but without easy interoperability with the ecosystem it's a no-go choice. Which is a shame as a week of TLC to a project like flowgen to bring it up to date (almost all features it says are missing or has polyfilled are now supported, I added support for most of them in a fork in 1-2 hours but project maintainers are inactive, there are still more e.g. type guards), and add it to CLI e.g. flow convert <npm-package> would make flow a more viable option.

Do you mind if I reopen this issue? I think this is a valuable discussion to have.
This year we have started converging with TS where it makes sense. e.g. we now have conditional types, mapped types and type guards. We are also in the process of changing some syntax to align with TS, such as the casting syntax (https://github.com/prettier/prettier/pull/15130) and some smaller items (https://github.com/prettier/prettier/pull/15429, https://github.com/prettier/prettier/pull/15466). You can follow some progress in #8989.
The next logical step is .d.ts support. This is an umbrella project that contains a lot of smaller items (e.g. declaration merging, namespace, TS syntax, etc). That being said, I cannot promise anything concrete or a timeline here. In the internal prioritization process, OSS is unlikely to make it to the top. However, I am more optimistic this time. We finally finished big architecture changes (types first) and big rewrites (LTI) so that our performance and type checker are no longer on fire, so we can focus more time on DevX issues like this. Lack of interoperability with the TS ecosystem is also an internal developer pain.
I appreciate your passion for Flow even though a lot of things are stacked against us now. I do have a personal passion for this, since I was on your side before. In 2018-2019, I used Flow but eventually gave up due to lack of well-maintained flow-typed definitions. I believe there is something we can do here.

Thanks for the great answer. I was regretfully a bit passive aggressive in places cause I was in the middle of converting some d.ts 😅 Actually tbh I just renamed the files and merged them together and there wasn't much work besides that which was cool to see - definitely getting closer.
That's encouraging to hear and the closing feature parity is noticeable, understandable if you're not at the stage to make stable automatic conversion yet/don't have the time.  Side note: cool I find the as syntax more visually pleasing anyway :)

Sorry to not really bring much to the table with this, but wanted to say this would be an absolute game changer for us as well. Flow is a fantastic tool, we particularly appreciate the integration with relay, but the lack of support for typescript defs really causes us headaches on a weekly basis. I truely believe this is the only major headache we have with it and adding def support would save an enormous amount of time.

We planned next year that we are going to tackle this. More specifically, I will look into the following aspects, in order of priorities:

namespaces (only for type definitions)
declaration merging
some cosmetic syntax changes to directly understand TS definitions

By the end of next year, our goal is not to understand every single .d.ts file without issue, but hopefully as we progress, you will need fewer and fewer manual edits to an upstream d.ts file to make it usable in Flow

Once we make the next release, it will contain https://github.com/facebook/flow/commit/d8bba3cfbb5f68a675dd6b5532f745b050157cc1, which will make it less hostile to use TS type names.
Note that this is best effort support (e.g. eslint and prettier support might not be ready when we allow more stuff under the flag), but if you actually managed to use other tools only designed for TS but use flow for type checking, we'd like to hear from your experience.

Every once in a while I revisit Flow to see how the interop story is moving along, because the awesome type system and tooling around Flow are so encouraging. Are there resources where I can read about progress in the past year? Thanks for all your efforts.

The experimental.ts_syntax flag is the major progress this year. I also implemented some experimental d.ts to .js.flow translator in the flow-api-translator package.


How to know what reason caused the imageproxy server down?

I'm usingimageproxy --addr myip.1.1.1:8080 -cache /tmp/imageproxy
to start my imageproxy server on Ubuntu 16.04 and try to test its limit and performance.
The server has only 300MB available memory. And I request images sequently image  by
xargs -n 1 curl -O < urls.txt The urls.txt file has 300 more images urls.
After about 20 images I downloaded successfully to my Mac, the imageproxy server shut down with no error.
I've checked the following logs:

/tmp/imageproxy.ERROR
/tmp/imageproxy.INFO
/tmp/imageproxy.WARNING

Above logs only shows image: unknown formatand some other has nothing to do with the reason shutting down the server.
And I also check the running console log, it shows some Go lang info:
goroutine 14 [IO wait]:
net.runtime_pollWait(0x7f00847c4e60, 0x72, 0x7)
	/usr/lib/go-1.8/src/runtime/netpoll.go:164 +0x59
net.(*pollDesc).wait(0xc42001ad88, 0x72, 0x8db480, 0x8d75e8)
	/usr/lib/go-1.8/src/net/fd_poll_runtime.go:75 +0x38
net.(*pollDesc).waitRead(0xc42001ad88, 0xc42013d000, 0x1000)
	/usr/lib/go-1.8/src/net/fd_poll_runtime.go:80 +0x34
net.(*netFD).Read(0xc42001ad20, 0xc42013d000, 0x1000, 0x1000, 0x0, 0x8db480, 0x8d75e8)
	/usr/lib/go-1.8/src/net/fd_unix.go:250 +0x1b7
net.(*conn).Read(0xc42000e158, 0xc42013d000, 0x1000, 0x1000, 0x0, 0x0, 0x0)
	/usr/lib/go-1.8/src/net/net.go:181 +0x70
net/http.(*persistConn).Read(0xc420073200, 0xc42013d000, 0x1000, 0x1000, 0x4, 0x450310, 0xc420057500)
	/usr/lib/go-1.8/src/net/http/transport.go:1316 +0x14b
bufio.(*Reader).fill(0xc420057380)
	/usr/lib/go-1.8/src/bufio/bufio.go:97 +0x117
bufio.(*Reader).Peek(0xc420057380, 0x1, 0xc42003bbe5, 0x1, 0x0, 0xc420057620, 0x0)
	/usr/lib/go-1.8/src/bufio/bufio.go:129 +0x67
net/http.(*persistConn).readLoop(0xc420073200)
	/usr/lib/go-1.8/src/net/http/transport.go:1474 +0x196
created by net/http.(*Transport).dialConn
	/usr/lib/go-1.8/src/net/http/transport.go:1117 +0xa35

goroutine 15 [select]:
net/http.(*persistConn).writeLoop(0xc420073200)
	/usr/lib/go-1.8/src/net/http/transport.go:1704 +0x43a
created by net/http.(*Transport).dialConn
	/usr/lib/go-1.8/src/net/http/transport.go:1118 +0xa5a

I check memory and cpu usage by top then I found out a suspicious process called kswapd0 always shows before the server crashed.
After search I think it's because running out of memory. But there is no Error log for this.
Can this ERROR be logged in the log files? Or is there any way I can make sure the reasons it crashed?

kswapd0 => not a suspicious process. That mean the computer's swap space (disk RAM) is being used (probably intensely in your case).

I have the same error but I'm not using any cache
goroutine 1364 [IO wait]:
internal/poll.runtime_pollWait(0x7f750aeb6388, 0x72, 0x0)
        /usr/local/go/src/runtime/netpoll.go:173 +0x57
internal/poll.(*pollDesc).wait(0xc4f5434b98, 0x72, 0xffffffffffffff00, 0x1060a60, 0x105b410)
        /usr/local/go/src/internal/poll/fd_poll_runtime.go:85 +0xae
internal/poll.(*pollDesc).waitRead(0xc4f5434b98, 0xc433968c00, 0x1, 0x1)
        /usr/local/go/src/internal/poll/fd_poll_runtime.go:90 +0x3d
internal/poll.(*FD).Read(0xc4f5434b80, 0xc433968c11, 0x1, 0x1, 0x0, 0x0, 0x0)
        /usr/local/go/src/internal/poll/fd_unix.go:125 +0x18a
net.(*netFD).Read(0xc4f5434b80, 0xc433968c11, 0x1, 0x1, 0xba4, 0x3f948ff87a89d04c, 0xba5)
        /usr/local/go/src/net/fd_unix.go:202 +0x52
net.(*conn).Read(0xc42d463a70, 0xc433968c11, 0x1, 0x1, 0x0, 0x0, 0x0)
        /usr/local/go/src/net/net.go:176 +0x6d
net/http.(*connReader).backgroundRead(0xc433968c00)
        /usr/local/go/src/net/http/server.go:660 +0x62
created by net/http.(*connReader).startBackgroundRead
        /usr/local/go/src/net/http/server.go:656 +0xd8

goroutine 1972 [select]:
net/http.(*persistConn).writeLoop(0xc422927e60)
        /usr/local/go/src/net/http/transport.go:1759 +0x165
created by net/http.(*Transport).dialConn
        /usr/local/go/src/net/http/transport.go:1187 +0xa53

What can it be?

can folks confirm whether they are still seeing this issue?  I'm finally getting around to trying to go through the issue backlog, and want to confirm whether this still happens.  Thanks.

I wrote my own image proxy meanwhile https://ramiel.gitlab.io/caravaggio/ :)

can folks confirm whether they are still seeing this issue? I'm finally getting around to trying to go through the issue backlog, and want to confirm whether this still happens. Thanks.

I have no longer this server so I cannot reproduce it for now. thanks your work anyway.

cool, thanks for following up.  And that looks like a really cool project, @ramiel!
Closing as "not reproducible" for now.  If anyone sees this behavior again, feel free to reply here or open a new issue.


Suggestion: Move lookup logic

Description
After taking a look at #64, I think it would be a good idea to move the lookup logic to the Store interface, and force all of the different stores to implement their own lookup. This change will probably be necessary if the project is looking at having different approaches for checking stock at different stores (apis, full page scraping, element scraping).
I don't mind implementing this; just looking for feedback/a green light if I choose to do continue so I don't waste anyone's time.

I'm actually doing this now as we speak! Haha, thanks :)

Sounds good!


Upgrade to K8s 1.23

Description
Fixes #313
Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)

Reading through the changelog it doesn't seem like there's anything actionable for us to do.

bors r+


calver or semver?

I'm open to either. I do think that this package has some core APIs that we will only want to break very carefully, so semver seems better. Otherwise, we need to go around and pin it.

Agreed on semver.


trying to run elk_apache samples - Error importing kibana dashboard

Error: Could not locate that index-pattern-field (id: geoip.location)
at FieldAggParamFactory.FieldAggParam.deserialize (http://localhost:5601/bundles/kibana.bundle.js?v=9889:99917:16)
at http://localhost:5601/bundles/kibana.bundle.js?v=9889:102896:29
at Array.forEach (native)
at AggConfigFactory.AggConfig.fillDefaults (http://localhost:5601/bundles/kibana.bundle.js?v=9889:102874:28)
at new AggConfig (http://localhost:5601/bundles/kibana.bundle.js?v=9889:102788:13)
at http://localhost:5601/bundles/kibana.bundle.js?v=9889:103492:19
at Array.map (native)
at new AggConfigs (http://localhost:5601/bundles/kibana.bundle.js?v=9889:103490:35)
at VisFactory.Vis.setState (http://localhost:5601/bundles/kibana.bundle.js?v=9889:99388:20)
at SavedVis._updateVis (http://localhost:5601/bundles/kibana.bundle.js?v=9889:104372:17)

@scanlonjts what version of Kibana are you running ?

kibana 4.5.0  -- i may have found issue  will let you know

my bad - this is all set (copied wrong file) -and had not recreated index in kibana so it did not recoginze all fields

Great. Thanks for giving the examples a go. If you have suggestions for other examples that might be helpful, let us know.


RMT driver (TX) does not work properly if the inverted GPIO output is used (IDFGH-3638)

Environment

Development Kit:  [any ESP32]
Module or chip used:  [ESP32]
IDF version (run git describe --tags to find it): v4.2-dev-2243-gcf056a7d0
Build System: idf.py
Compiler version (run xtensa-esp32-elf-gcc --version to find it): xtensa-esp32-elf-gcc (crosstool-NG esp-2020r2) 8.2.0
Operating System: Linux
Using an IDE?: No
Power Supply: USB

Problem Description
TL;DR: RMT driver (TX) does not work properly if the inverted GPIO output is used.
Details: I'm trying to write an alternate firmware for Xiaomi Universal IR controller (MJYKQ01CM). This device is based on a single core ESP32 chip with BLE and WiFi, 4MB of SPI flash.
It has IR receiver that connected to GPIO_NUM_27, and seven IR transmitters, they connected to GPIO_NUM_26.
Some pictures - https://imgur.com/a/qWfD7xt
The thing is that GPIO_NUM_26 connected as the inverted output. But RMT drivers doesn't have an API to set that, it contains a hardcoded function call:  https://github.com/espressif/esp-idf/blob/master/components/driver/rmt.c#L490
.. rmt_set_pin() ...
   if (mode == RMT_MODE_TX) {
...
      esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 0, 0);
...
   }

As result, the transmitter does not work properly (and that's how I burned half of the diodes on that board because they remains in the ON state after transmitting :D )
To solve my issue I changed that line to:
 esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 1, 0);

After that change all peripherals works as expected.
Expected Behavior
An API to tell RMT driver that GPIO is inverted or not.
Actual Behavior
There is no such API.

Thanks for reporting, we will look into.

Hi,
sorry about your board :( Would enabling the idle output of RMT and setting it to high (both are in the rmt_tx_config_t) give you the correct output?

Hey @wilhelmw201
My tx init function looks like:
esp_err_t ir_init_tx(void)
{
	rmt_config_t config = RMT_DEFAULT_CONFIG_TX(DEVICE_GPIO_IR_TX, DEVICE_RMT_CHANNEL_TX);
	esp_err_t err;

	ESP_LOGI(TAG, "TX: gpio #%d, channel %d", DEVICE_GPIO_IR_TX, DEVICE_RMT_CHANNEL_TX);

	config.clk_div = 255;
	config.tx_config.loop_en = false;
	config.tx_config.carrier_duty_percent = 50;
	config.tx_config.carrier_freq_hz = 38000;
	config.tx_config.carrier_level = 1;
	config.tx_config.carrier_en = true;
	config.tx_config.idle_level = RMT_IDLE_LEVEL_LOW;
	config.tx_config.idle_output_en = true;

	err = rmt_config(&config);
	if (err != ESP_OK) {
		ESP_LOGE(TAG, "TX: rmt_config failed: %s", esp_err_to_name(err));
		return err;
	}

	err = rmt_driver_install(DEVICE_RMT_CHANNEL_TX, 1000, 0);
	if (err != ESP_OK) {
		ESP_LOGE(TAG, "TX: rmt_driver_install failed: %s", esp_err_to_name(err));
		return err;
	}
	return ESP_OK;
}

I changed config.tx_config.idle_level to RMT_IDLE_LEVEL_HIGH, and reverted the GPIO change (esp_rom_gpio_connect_out_signal was called with 0, 0).
Result: transmitter is always ON, and it cannot produce correct IR commands (my LED lamp ignores all of them).

Hi,
can you elaborate on this sentence: "The thing is that GPIO_NUM_26 connected as the inverted output."? I assume you mean that you have inverted the GPIO prior to calling the listed ir_init_tx, but this was overwritten by the RMT init function?
(If yes, it is not necessary to inverse the output, you can set the RMT idle level to the idle level you want and inverse your signal data.)

What I tried:

esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 1, 0) and RMT_IDLE_LEVEL_HIGH: always on, doesn't work
esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 1, 0) and RMT_IDLE_LEVEL_LOW: works as expected
esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 0, 0) and RMT_IDLE_LEVEL_HIGH: always on, doesn't work
esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 0, 0) and RMT_IDLE_LEVEL_LOW: always on, doesn't work

So it is not just the inverted signal itself, LED should stay in the OFF state if there is no transmission at the current moment.

I see, did you enable the carrier? Then in this case this is problematic.
We'll put an inverse function in the GPIO, but for now you may have to call esp_rom_gpio_connect_out_signal manually :(

did you enable the carrier?

I did :)

may have to call esp_rom_gpio_connect_out_signal manually
That's what I'm doing now. Hope it will fixed by some flag/argument in RMT driver API.

esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 1, 0) and RMT_IDLE_LEVEL_LOW: works as expected
esp_rom_gpio_connect_out_signal(gpio_num, RMT_SIG_OUT0_IDX + channel, 0, 0) and RMT_IDLE_LEVEL_HIGH: always on, doesn't work

What seems strange to me is that this two cases should have the same output if carrier mode is on.

Invertion of GPIO output signal was added :

in the legacy driver, there's a flag RMT_CHANNEL_FLAGS_INVERT_SIG to control this
in the new driver, there's a invert_out flag in rmt_tx_channel_config_t configuration structure.

See doc for how to use it.


Implement Transaction proposal from ZIP-321 URIs for Swift

Feature was left out of 2.0.11. Rebased and started to test again

Created a new PR after rebasing and retesting
https://github.com/Electric-Coin-Company/zcash-swift-wallet-sdk/pull/1390


[BR] Cannot use general.storage.volumes

Describe the bug
The operator seems to ignore general.storage.volumes
To Reproduce
Steps to reproduce the behavior:

Install the operator
Create a K8SMediaserver resource with this in spec.general/storage:

...
spec:
  general:
    storage:
      pvcName: mediaserver-pvc
      size: 5Gi
      pvcStorageClass: "nfs-pvcname-storageclass"
      subPaths:
        tv: media/tv
        movies: media/movies
        downloads: downloads
        transmission: transmission
        sabnzbd: sabnzbd
        config: config
      volumes:
        persistentVolumeClaim:
          claimName: ext01-mmedia
...

Expected behavior
I would expect the operator to create a Deployment for plex that mounts a new volume using nfs-pvcname-storageclass and also an existing volume which pvc name is ext01-mmedia.
Environment:

K8s version: v1.22.6+k3s1
CNI Plugin: flannel
CSI Type:

Hi @j-fuentes, thank you for reporting this, can you please try setting spec.general.storage.customVolume to true?

@kubealex that works, it mounts the volume.
However, I think I misunderstood the feature. From the name of the setting (volumes) I thought it would allow to mount additional volumes. But it just replaces the default volume.

I think I can use projected volumes to achieve what I want, mounting more than one volume.

@j-fuentes yep, the volume, since all components rely on the same 'space' for downloading/moving/archiving media, is unique.

👍🏽 I am going to close the issue now, as the problem I reported wasn't such a problem but something not supported that I misunderstood.
Thank you so much for the explanation.


[SPARK-14744][examples] Clean up examples packaging, remove outdated examples.

First, make all dependencies in the examples module provided, and explicitly
list a couple of ones that somehow are promoted to compile by maven. This
means that to run streaming examples, the streaming connector package needs
to be provided to run-examples using --packages or --jars, just like regular
apps.
Also, remove a couple of outdated examples. HBase has had Spark bindings for
a while and is even including them in the HBase distribution in the next
version, making the examples obsolete. The same applies to Cassandra, which
seems to have a proper Spark binding library already.
I just tested the build, which passes, and ran SparkPi. The examples jars
directory now has only two jars:
$ ls -1 examples/target/scala-2.11/jars/
scopt_2.11-3.3.0.jar
spark-examples_2.11-2.0.0-SNAPSHOT.jar

Test build #56430 has started for PR 12544 at commit ccb73e8.

Test build #56430 has finished for PR 12544 at commit ccb73e8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/56430/
Test PASSed.

Is there any documentation that needs to be updated with the addition of --packages or --jars to run the examples (e.g. running-on-yarn.md) ?

Is there any documentation that needs to be updated

There isn't any explicit documentation about how to run streaming examples (that I can find). The pyspark bindings will print out an error message with instructions if the needed classes are not found, though. AFAIK Scala examples would just fail with a class not found error.
In any case, running streaming examples is, in general, more complicated than just running a single spark app in any case. In my view they're meant more as a code example than something you should really try to be running.

I already got a review from Sean, but in case anyone else wants to chime in... @JoshRosen @rxin

Can we have them as non-runnable files? I don't think people would run these, but it might be good to have them so people can write code based on the templates.

Do you mean the HBase and Cassandra examples? My view is that they're not good examples of how to integrate with those systems, and leaving them around would send the wrong message.

Merging this to master.


是否预加载VSSM权重会对模型精度有影响

作者你好，文章里模型的效果非常好，但是在复现WHU-CD数据集上进行训练发现预加载vssm_small_0229_ckpt_epoch_222.pth进行训练能够达到文章里的效果，；但是如果选择不使用vssm_small_0229_ckpt_epoch_222.pth权重文件在数据集上训练F1-score会从文章里的94.变成90.，请问这是什么原因导致的呢，正常来说在imageNet数据集上训练得到的权重并不适应遥感数据集

Hi,
Thank you for your interest to our work. We assume that pre-trained weights on Imagenet provide a very good initial value compared to if you were to train from scratch. Besides mamba, a similar situation exists with transformer. If you train transfomer from scratch on remote sensing tasks, its accuracy cannot match many CNN networks.
Your statement that the weights obtained from training on the ImageNet dataset are not adapted to the remote sensing dataset is a bit inaccurate. We can say that the weights obtained by training on the ImageNet dataset may not be as good as the weights obtained by pre-training on some large remote sensing datasets, but we can't say that they are not applicable to remote sensing datasets. Especially for optical high resolution images, the weights obtained by pre-training on ImageNet dataset can be well extracted for features applicable to downstream tasks.
Best,

Thank you for your answer, it was very helpful in solving my confusion, this article is really awesome.


"svn info" command return an error

Hi Sir
I'd like to execute the "svn info" command. But it returns an error.
I use robo 0.6.1 and my code is:
	$result = $this->taskSvnStack($svn_user, $svn_password)
	->dir($dir)
	->exec("info")
	->run();

The error is:
ERROR: proc_open(): CreateProcess failed, error code - 267
in [...] vendor\symfony\process\Process.php:283

I found it. The error means that the path I use ($dir), doesn't exist.

@burzum:
Think we could throw an error in Robo even before executing such tasks to have a better user experience.
Like doing a realpath in https://github.com/Codegyre/Robo/blob/master/src/Common/ExecCommand.php#L34 and if false throw an Exception?

@boedah feel free to fix it. You can use the ResourceCheckerTrait I've recently added for exactly this purpose.

Yes, will do.

With 1.0.0-beta1 things changed. Now the task is completed succesfully, but I get the following error:
ERROR: fopen(C:\Users\repository): failed to open stream: Permission denied in phar://C:/.../robo.phar/src/Runner.php:249
The same svn command from the cli works fine.
What about the error ?


Problem in concating bootstrap and other libraries through bower

Through bower-install gulp task, libraries like bootstrap is not concating css and js file due to bower.json not present in bower_components/bootstrap
Bootstrap and some other libraries in bower contains .bower.json file instead bower.json. In bower task we have check this also

The user should be allowed override such configs.
In https://github.com/Swiip/generator-gulp-angular/ there is a gulp conf.js file where the user can config the src, dist directories etc.


Error: Bad Signature (when trying to install in Spacemacs)

I followed the setup instruction for spacemacs, but after restarting, installation fails with (Spacemacs) Error:  An error occurred while installing copilot (error: (bad-signature jsonrpc-1.0.25.tar.sig))
I'm running Spacemacs 27.2 on Ubuntu 20.04.
Am I doing something wrong? Thanks

It seems that Spacemacs had an issue while trying to install jsonrpc (version 1.0.25), which is a dependency of copilot.el.
You could try manually installing jsonrpc.


Failure to parse syntax for a pretty large shell script

At line 1940 of my shell script the BashSupport plugin starts to report syntax error where there are none.
The error message says "Internal parser error: Maximum level of nested calls reached. Please report this."

Is the script available for inspection?

Unfortunately, my employer doesn't allow access to source code outside the organization. Anything else I could do to identify the issue?

Hm, this is difficult to analyze :)
Is the script deeply nested, i.e. does it have deeply nested expressions or statements, e.g. "if" in "if" in "while" an so on?
As this is the first report of this kind I'll probably be unable to fix this without a script, though.

How many lines does the script have, by the way? Up to 5000 lines shouldn't be a problem. Might depend on the nesting level, though.
Which version of BashSupport do you use?

The script has 2400 lines. The partifular function has a lot of small for loops and in these for loops, bash expressions like ((n=n+1)) occur. I wouldn't say it is deeply nested - max 1 or 2 levels deep.

Same issue.
Script: https://github.com/rusxakep/docker-percona/blob/develop/5.7/docker-entrypoint.sh

Thanks for the sample script!
The next version will be less restrictive while parsing long command lists, i.e. your script will be parsed without errors. A proper fix to support really long lists won't be done anytime soon, though.

Am 25.03.2016 um 20:13 schrieb Mikhail Baykov<EMAIL_ADDRESS>Same issue.
Script: https://github.com/rusxakep/docker-percona/blob/develop/5.7/docker-entrypoint.sh
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

hi, this was "milestoned" to 1.5.6. I have now 1.5.6 and the error persists. I updated to 1.5.7 and the same result. Maybe this issue should be re-opened?
Great work. Your effort is very appreciated.

Hm, this is supposed to be fixed. It might break for really long and nested scripts, though. Which message do you see?
If you like you can submit a new issue, a sample script to reproduce this with 1.5.7 helps a lot.
I won't be able to work on it for quite a while, see the note on https://github.com/jansorg/BashSupport
--
<>< Re: deemed

Am 23.05.2016 um 22:39 schrieb Oscar aka Elvis<EMAIL_ADDRESS>hi, this was "milestoned" to 1.5.6. I have now 1.5.6 and the error persists. I updated to 1.5.7 and the same result. Maybe this issue should be re-opened?
Great work. Your effort is very appreciated.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub

Yes, same issue here. Take my script as an example of the error using 1.5.7
https://github.com/v1s1t0r1sh3r3/airgeddon/blob/master/airgeddon.sh
Regards.


Add annotations passthrough for host process containers

This changes adds in a way for annotations specified for a sandbox
container to be passed through to every container in the pod.
K8s only passes annotations to the RunPodSandbox call and not to individual
containers unfortunately. To accomplish this I cache the pod sandboxes OCI
spec on the pod object as well as expose a method that passes through
specific annotations from the pod spec through to an individual containers.
This is useful for a couple of host process containers annotations, like
microsoft.com/hostprocess-rootfs-location which specifies a non-default
path for the rootfs of the container to show up at.

cc @marosset If you want to try this out

@anmaxvl ptal

@ambarve PTAL when you get a chance


Added petting to Holoparasites

Learning GitHub just to make holos pettable, lets goooo :D
Changelog
add: made Holoparasites pettable

Other one is more recent so using that.


[Roots 3] Living Quiver voided some arrows when shift-clicking into it

I'm not sure how to explain this properly other than I was making a lot of living arrows, shift-clicked a partial stack into the nearly-full stack in the quiver, and instead of filling the next slot in the quiver it filled the existing slot in the quiver and voided the rest.
Then upon opening the quiver again immediately after to double check what had just happened, the stack size in that slot wasn't full anymore and I still didn't have all of those arrows back. I'm pretty sure there were less living arrows, even, than when I'd started - I don't have the exact amounts memorized because this happened a little while ago, but it occurred to me I should probably report it.

This is the latest version, I presume? I know this was an issue with with both the quiver and the component pouch after I converted them from being capability-based into being NBT-based. It's possible that there are still some outliers -- stack overflows from one into a new slot is something I haven't tested extensively.

The version on the Devcraft server - that's where this happened.
The jar's named this:
Roots-1.12.2-3.0.0-14
If that helps.

Thanks, that definitely helps. I'll have to re-examine the code, I must've missed something.

I'm worried that this might still be the case. We'll need to test it.

I'm gonna close this as there were a significant number of changes since it was opened. If it's still the case, someone will notice and can file a new report.


Notebooks are no longer working in JuypterLab without extension

In a JupyterLab environment without the extension the following error is displayed:

works


Out-off-proportion CPU load

Constantly about 30% CPU load (2x1.9GHz, 4 threads) when regarding any map in 2x2 view (still 20-25% in single view), peaks to 50% when moving the mouse over the map selection screen (each zooms and unzooms), drops back to idle when another tab is selected. This is on Win 7 64-bit on Firefox 49.
IE seems to be limited to one thread, caps at 25% with stuttering when trying to pan.
This might not be that noteable on gaming desktops.
X-posted this here from reddit

I'm okay with a bit of a performance hit when panning or zooming with 4 maps, but not okay that it is constant CPU usage - gotta be hurting battery life...
I'm not seeing anything that drastic, even on midrange hardware, but I'll look into this more soon.

Same problem here on win 10 64-bit firefox 50.

It sits at a constant 35% CPU on firefox, between 8% and 10% constant on chrome for me. (win10).
definitely makes it unusable to have loaded on a second screen for me.

I'll start looking at this in more detail, I am seeing the issue in Firefox as well.  Chrome seems fine for me, but still higher than I think it should be.

  I'm really sorry I didn't look at this closer sooner. :(
So the issue is the loading spinner I'm using - it is using css animations and it is sort of always on screen.  Wow.  So much battery power wasted, so many users inconvenienced because I wanted to be fancy and smart with CSS.  Sigh.
Okay, so I'll definitely have a fix in today, you should notice the site runs a TON better.  It will still have some janky performance when you're doing something intense like zooming with 4 panels up, but that shouldn't be nearly as concerning as this obviously.
Yar.  Thanks for raising this again.

fantastic thanks for taking a look

thanks again, it works perfectly now. just played a ton of games with it on my second screen and it's a literal game changer :D


Local files

Does it work only with local files ? Can I open files by link without downloading ?

No, this library doesn't have any function to directly open file by link. You can reach the same result combining this library with react-native-fs as described here.
Remember that opening a file using a link means that the file needs to be downloaded anyway, doesn't matter if this process is done under the hood. :)

@vinzscam Got it! Thanks!


Master

fix merge conflict

I'm not including an auto-clicker in this script.

Oh man, wrong fork. I'm on Windows and Git isn't playing nice. Sorry.


Is the m40_ilrl is the trained model

we download the pre-trained model from https://drive.google.com/drive/folders/1NyuwzIiJGyJsCkvEOnrziwrGN56uoTTj/m40_ilrl.zip and test it on modelnet40 with noise, but we get unreasonable results (MAE R is about 15). We also test m40_ilrl.zip on modelnet40 without any noise and the MAE R is about 1.1.

Hi,
Yes, m40_ilrl.zip is the model trained for a total of 100 epochs ("ours IL+RL" in the paper). I've just tried to replicate your issue by downloading the weights from Google Drive but running
python registration/eval.py --mode=ilrl --dataset=m40-model
yields the results reported in the paper for me with a MAE R of 1.47 and 1.34deg for m40-model and m40-cat, respectively. I've also tested without noise (i.e., changing the last parameter in eval.py:L127 to "clean") and get a MAE R of 1.18 and 0.82deg for m40-model and m40-cat, respectively. So those results seem consistent with yours and there might be an issue with your results using noise ("jitter").
Could you confirm that the settings in RandomJitter are scale=0.01 and clip=0.05? Might there be any other changes to the code? Are you using the ModelNet40 version provided here?

Thanks for your patient answer, I checked the code and I did change the noise parameters by mistake.


Show all possible configuration options with CLI

Hi!
When I do poetry config --list, poetry shows no output.
I've tried to dig a little bit, and found out that I have an empty ~/.config/pypoetry/config.toml which poetry probably created during the installation. I also found that --list reads this file and tries to get config variables from there.
So it seems like an expected behavior, but it's very unintuitive. How about showing the list of the possible configuration values without the need to go to the documentation site or compiling docs myself? What do you think?

All available settings are now documented on this page, but I think you shouldn't have to leave the terminal to see them, so I agree with you.
Now, we have to discuss precisely how we're going to do this. Perhaps --list shows all available settings and --show shows the ones you've set? Or should we try to leave --list unchanged?

Yeah, eventually I found the docs section inside the web site, but not leaving the terminal would be great.
I think that approach of showing available options with their values (defaults in case they weren't set yet) is the best one and multiple existing tools are doing that. Git is one of those
I'm talking about something so you can do poetry config --list > conf.toml. And maybe it worth writing them explicitly in the default config file as well.
What do you think?

Version 0.11 was just released and this feature is in it ;)

Great! Much more helpful :tada:


InfluxQL tests failing in algo W branch

There are two InfluxQL transpiler tests failing in the feat/use-algo-w branch:

ors.influxql
ands.influxql

They are failing with the message:
failed to run query: could not get handle from JSON AST: recursion limit exceeded at line 1 column 8056

The tests have been commented out with this issue URL.

In the case of ands.influxql, the test case involves transpiling and then evaluating a simple SELECT query with 200 conjunctive predicates in the WHERE clause. ors.influxql is symmetrical.
The issue occurs in the evaluation of the transpiled flux AST. The initial AST resides in Go. It is then serialized as JSON where Rust is invoked to deserialize it and perform the rest of semantic analysis. In JSON the WHERE predicate is expressed as 200 nested JSON objects, and the recursion limit is exceeded when serde_json is deserializing the object into a Rust AST.
It is possible to remove serde_jsons recursion limit, however care would need to be taken to ensure we did not explode the stack. A more permanent solution is to remove JSON serialization from the parsing process entirely. That is outlined here.
I'm going to keep these tests skipped at this time and move this issue to the backlog. serde_json has a default recursion depth of 128 https://github.com/serde-rs/json/blob/master/src/de.rs#L48 which seems reasonable for now.


⚠ PrivateHD has degraded performance

In 3be1e5f, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3924 ms

Resolved: PrivateHD performance has improved in facb71e.


Reshaper extension load order for Repeater

Hi, I just spent quite a bit of time debugging why Reshaper would not work in the Repeater tool, even with the Repeater tool enabled in the Reshaper settings. In the logs I could see that the rules were triggering and executing properly, but their output was not reflected in the Repeater response pane.
I found that if the popular extension Logger++ is loaded, and Logger++ is found below Reshaper in the load order, Reshaper will not be able to modify the Repeater response pane.
Example rule:

JSON export
{"version":1,"rules":[{"whens":[],"thens":[{"@class":".ThenSetValue","useMessageValue":false,"sourceMessageValue":"SourceAddress","sourceIdentifier":"","sourceIdentifierPlacement":"Last","sourceMessageValueType":"Text","sourceMessageValuePath":"","useReplace":false,"regexPattern":"","text":"foobar","replacementText":"","destinationMessageValueType":"Text","destinationMessageValuePath":"","destinationMessageValue":"HttpResponseBody","destinationIdentifier":"","destinationIdentifierPlacement":"Only"}],"enabled":true,"autoRun":true,"name":"test"}],"webSocketRules":[],"variables":[]}

Scenario 1
Load order

Repeater

Scenario 2
Load order

Repeater

I'm not sure how to fix this, I think its a regression in Burp because I always have Logger++ as the very last extension in the load order. Might be something to document in the extension?

Hi @ldionmarcil ,
Thanks for the nice write-up. This issue is not expected functionality. The same issue has been reported by users of Logger++ in combo with other extensions in the Portswigger Discord. See https://discord.com/channels/1159124119074381945/1164175825474686996/1315709434261147721. Current speculation by the Logger++ dev is that it is a Burp Suite bug as you've mentioned. I'm keeping an eye on the situation.


[ci] Add JDK 24 early access to CI build

coverage: 87.175%. remained the same
when pulling ef4070db66969c9f4dfc31b15e7ce15f1ac9d05e on harawata:jdk-24-ea
into 1d58e3f0f49cef691e6c009e3e83947ab41635ab on mybatis:master.


Update dev-ko branch by merging main branch

This PR will update dev-ko branch by merging the latest commit of main branch.

Hi @gy-ulbak96
Thank you for the contribution to maintain dev-ko branch !! :)

Note
Whereas DCO fails, I'd make DCO checking passed manually. (as a maintainer)
It is based on the observation from https://github.com/cncf/glossary/pull/1290#issuecomment-1237991211
We are hard to avoid DCO failure for PRs like this PR. (merging a branch into another branch)


Add Google Authentication for Login and Register. {GSSoC '23}

Description
Is your feature request related to a problem? Please describe.

It simplifies the registration process by allowing users to sign up using their existing Google accounts, eliminating the need for a new account and reducing the registration steps.
It enhances security by leveraging Google's robust security measures, reducing the risk of unauthorized access or data breaches.
It increases user trust as many users are familiar with and trust Google's authentication services.

Describe the solution you'd like
The feature entails enabling users to authenticate using their Google account with the assistance of Auth0. By implementing this functionality, we can enhance the user experience and provide a seamless onboarding process.
Describe alternatives you've considered
Alternatives include authentication through other popular social media platforms like Facebook or Twitter, giving users a choice of platforms for logging in.
Additional Information
Context
Provide any additional context or information that helps explain the feature request.
Potential Implementation (optional)
If you have any ideas or suggestions on how the feature could be implemented, please share them here.
Screenshots or Mockups (optional)
If applicable, include any screenshots, mockups, or visual representations of the requested feature.
Priority (optional)
Please indicate the priority level of this feature request (e.g., High, Medium, Low).
Additional Notes (optional)
Add any other notes or comments about the feature request here.

not required now.


Checklist for declaring stability for messaging semantic conventions

Separate experimental parts from stable messaging semantic conventions
When messaging semantic conventions are declared stables, some parts of the conventions should be excluded and remain in an experimental state.
This issue collects all parts that should be marked as experimental prior to declaring messaging semantic conventions stable:

All references to "Process" spans (open-telemetry/semantic-conventions#657)
All references to "Settle" spans (open-telemetry/semantic-conventions#621)
The following attributes:

messaging.message.body.size (open-telemetry/semantic-conventions#474)
messaging.message.envelope.size (open-telemetry/semantic-conventions#474)
network.protocol.name (open-telemetry/semantic-conventions#504)
network.protocol.version (open-telemetry/semantic-conventions#504)
messaging.destination.temporary and messaging.destination.anonymous (open-telemetry/semantic-conventions#1126)

Finally, also any stability present on any stabilized messaging documents need to be removed.
Open tickets against instrumentations
As suggested in this comment, tickets against all existing messaging instrumentations in OpenTelemetry repositories need to be opened, outlining the changes and required actions

The last item in this list was done https://github.com/open-telemetry/semantic-conventions/pull/1442, so closing this one.


Update traccar manager inconsistent

Hi Anton,
According my topic on https://www.traccar.org/forums/topic/access-traccar-manager-inconsistent/, follow the step by step where screen Traccar Manager still not update:

Open Traccar manager and click on device
Open Chrome , type server address and click on device
stay until the standby on the cellphone Android (black screen)
wait a time (1, 2, 3 ... 5 minutes,... don't matter)
unblock the cellphone and look the Traccar manager ( I have a old information)
look the google chrome ( I have a updated information)

follow the test doing considering time equal 1 minute:
https://drive.google.com/file/d/0B1mhLFkpr3AwWlFneWtHOEh4VVE/view?usp=sharing
Is there a way (syntax - variable to be set on traccar.xml, etc. - or change Traccar manager code) to force Traccar  manager  update  each 1 minute?
Best regards
Alex

Most likely the issue is that session has expired, so you have to re-login. Traccar Manager doesn't allow you to refresh a page at the moment. Next version of Traccar will automatically redirect you to login screen, so you won't need to refresh anything manually and this issue will go away.

Hi Anton,
is it possible set/growing up the expired session time ? or instead automatically redirect you to login screen, automatically reflesh the page before expired session time?
I use the beep web notification to advise the people about geofence events.
it does works as if it were  push notifications (workaround).
Best regards

Have you actually read my comment?

Oh,so  sorry, it is "instead of", translate wrong
my apologices
yes, if growing up, the problem continues, but like I sad, it is a
workaround, because you tell / write that push notification is not to easy
to implement.
traccar has 3 option of alert:
web - beep only sounds when the sesison is active, if go to login page it
woulbe like as deactive session, so, beep will not works
e-mail: Brazil is poor country, some of person are old and do not
understand smartphone, image smartphone with e-mail
SMS: I am trying find in brazil a smpp provides, unfortunately I am search
yet
so, so sorry for english, and more once time thank you for the patient
Alex
2017-05-20 20:57 GMT-03:00 Anton Tananaev<EMAIL_ADDRESS>
Have you actually read my comment?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/3177#issuecomment-302906152,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aakdi_crWlqzOEe9FVREo3uk274rtJiOks5r7331gaJpZM4NgeRs
.

what you suggest?
2017-05-20 21:09 GMT-03:00 aecsalome<EMAIL_ADDRESS>
Oh,so  sorry, it is "instead of", translate wrong
my apologices
yes, if growing up, the problem continues, but like I sad, it is a
workaround, because you tell / write that push notification is not to easy
to implement.
traccar has 3 option of alert:
web - beep only sounds when the sesison is active, if go to login page it
woulbe like as deactive session, so, beep will not works
e-mail: Brazil is poor country, some of person are old and do not
understand smartphone, image smartphone with e-mail
SMS: I am trying find in brazil a smpp provides, unfortunately I am search
yet
so, so sorry for english, and more once time thank you for the patient
Alex
2017-05-20 20:57 GMT-03:00 Anton Tananaev<EMAIL_ADDRESS>
Have you actually read my comment?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tananaev/traccar/issues/3177#issuecomment-302906152,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aakdi_crWlqzOEe9FVREo3uk274rtJiOks5r7331gaJpZM4NgeRs
.

I suggest to use email or SMS notifications if you want guaranteed delivery.


Initial commit

OK, I've done a first pass over it all. Thoughts?

Well, I reviewed a little of this, am afraid I didn't get through the whole thing yet.

Ok, I've had a look through all of this.  Not in a lot of detail, but an overview.  Impressions:

There's an impressive amount of functionality here!  I can't imagine wanting much more from a rotations package right now.
The public exported interface is pleasingly small, only comment for the future would perhaps be to merge EulerAngles and ProperEulerAngles, or create them both out of a composable set of rotations in 2D planes, as @andyferris tried in CoordinateTransformations.
I feel the amount of code generation is rather out of control, and it makes the implementation of the constructors and conversions very opaque. This could probably be cleaned up with some carefully chosen generic functions.  The matlab code is... unfortunate ;-)
I have misgivings about the number of conversions/constructors defined, particularly between types outside the package - I think this may cause confusion to third party users of, eg, Quaternions and Rotations.  If an obvious constructor is missing, I'd say it'd be better trying to get it into the original package.

Overall I'm happy to merge this mostly unchanged, as I think most of the problems are things we can improve without disrupting API users.


Extend VaraTestRepoSources for git submodule usage

Introduces handling of local libraries used as GitSubmodules in benchbuild projects.
Also closes: se-passau/VaRA#677

@boehmseb I think we need a BB version bump for this, do we already have a BB release that includes this or are we waiting for the container fix?

The following issues on benchbuild side need to be resolved first: PolyJIT/benchbuild#345, PolyJIT/benchbuild#346, and PolyJIT/benchbuild#347. The last one asks for a new release.


moving static resources into bower dependencies

I'm already using bower and grunt-wiredep in different projects with keystone. I
was wondering if you think that this feature will be useful to the generator i can
make a PR for that

Alternatively, you can pull Bootstrap in from npm. If you npm install -S bootstrap, it has the Less source files. Or you can do npm install -S bootstrap-sass for the Sass version.
I didn't want to commit Bootstrap's source to my project, and I wanted to use Sass, so I used npm and wired it up with the following changes:
keystone.js:
keystone.init({
  // ...
  'sass options': {
    includePaths: [
      'node_modules',
      'node_modules/bootstrap-sass/assets/stylesheets'
    ]
  },
  // ...
});

public/styles/site.scss
// ...
@import "bootstrap";
// ...

from the bower home page: ...psst! While Bower is maintained, we recommend using Yarn and Webpack for front-end projects


Updates and change to METACTL_API_URL.

Searching for METALCTL_URL in the whole organisation gives > 100 occurrences https://github.com/search?q=org%3Ametal-stack+METALCTL_URL&type=code

Yep, but almost only metalctl documentation. In fact it affects ~5 repositories or something.


Include references in Indications data table on drug profile page

As noted in #929, indication_refs will be available in the ElasticSearch index after the pipeline has run for 20.09.
Once the data is available, we should update the GraphQL API and front-end UI to expose this data.

The field has the following format:
          "indication_refs" : [             {               "ref_type" : "DailyMed",               "ref_url" :  "https://dailymed.nlm.nih.gov/dailymed/lookup.cfm?06c485cd-ee30-44f2-be63-bb98f8d1a907",               "ref_id" : "06c485cd-ee30-44f2-be63-bb98f8d1a907"             },  ... ,             {               "ref_type" : "ClinicalTrials",               "ref_url" :  "https://clinicaltrials.gov/search?id=%22NCT01908452%22",               "ref_id" : "NCT01908452"             }]
There is a VM cm-jb-es in the open-targets-eu-dev project. It has an Elasticsearch instance which has the updated drug index (jb_drug-data) in case you needed to test against it.

@andrewhercules to provide more details on indications refs

@andrewhercules to provide more details on indications refs

Closed by https://github.com/opentargets/platform-app/pull/285


ci: Bump ubuntu-next image

The bump is to include [1] which is required to properly fix #14956.
[1]: https://github.com/torvalds/linux/commit/e8b9eab99232c4e62ada9d7976c80fd5e8118289

test-me-please

test-me-please

test-me-please

test-me-please

test-1.16-netnext

The EKS conformance suite has nothing to do with the VM image bump. Marking as ready to merge.

test-me-please

CI hit - https://github.com/cilium/cilium/issues/11560 (did some investigation)

test-me-please

test-1.16-netnext had a weird failure:

Test report file /home/jenkins/workspace/Cilium-PR-K8s-1.16-net-next/src/github.com/cilium/cilium/test/k8s-1.16.xml was length 0

test-1.16-netnext


Sqlite: Custom Aggregation function returning none.

Hi,
Re: the following code in the tutorial.
@Engine("sqlite");
AnonymizedCodeContribution(cl_lengths: [110, 220, 405], org: "ads");
AnonymizedCodeContribution(cl_lengths: [30, 51, 95], org: "ads");
AnonymizedCodeContribution(cl_lengths: [10, 20, 1000], org: "games");

HarmonicMean(x) = Sum(1) / Sum(1 / x);

OrgStats(
    org:,
    mean_cl_size? Avg= cl_size,
    harmonic_mean_cl_size? HarmonicMean= cl_size) distinct :-
  AnonymizedCodeContribution(cl_lengths:, org:),
  cl_size in cl_lengths;

The output from BigQuery has values for harmonic_mean_cl_size

  | org | mean_cl_size | harmonic_mean_cl_size
-- | -- | -- | --
ads | 151.833333 | 75.402468
games | 343.333333 | 19.867550

but when I run the above code for sqlite, I get none for the same column

  | org | mean_cl_size | harmonic_mean_cl_size
-- | -- | -- | --
ads | 151.833333 | None
games | 343.333333 | None

Generated SQL Query:
The following query is stored at OrgStats_sql variable.
WITH t_0_AnonymizedCodeContribution AS (SELECT * FROM (

    SELECT
      JSON_ARRAY(110, 220, 405) AS cl_lengths,
      'ads' AS org
   UNION ALL

    SELECT
      JSON_ARRAY(30, 51, 95) AS cl_lengths,
      'ads' AS org
   UNION ALL

    SELECT
      JSON_ARRAY(10, 20, 1000) AS cl_lengths,
      'games' AS org

) AS UNUSED_TABLE_NAME  )
SELECT
  AnonymizedCodeContribution.org AS org,
  AVG(x_5.value) AS mean_cl_size,
  ((SUM(1)) / (SUM(((1) / (x_5.value))))) AS harmonic_mean_cl_size
FROM
  t_0_AnonymizedCodeContribution AS AnonymizedCodeContribution, JSON_EACH(AnonymizedCodeContribution.cl_lengths) as x_5
GROUP BY org;

Another issue caused by SQLite's integer division.
We should update it to:
%%logica OrgStats
@Engine("sqlite");
AnonymizedCodeContribution(cl_lengths: [110, 220, 405], org: "ads");
AnonymizedCodeContribution(cl_lengths: [30, 51, 95], org: "ads");
AnonymizedCodeContribution(cl_lengths: [10, 20, 1000], org: "games");

HarmonicMean(x) = Sum(1.0) / Sum(1.0 / x);

OrgStats(
    org:,
    mean_cl_size? Avg= cl_size,
    harmonic_mean_cl_size? HarmonicMean= cl_size) distinct :-
  AnonymizedCodeContribution(cl_lengths:, org:),
  cl_size in cl_lengths;

👍 . please close at will.


Add --diff-only argument.  Closes #5

Coverage remained the same when pulling 01d019c95348fb659ec022b58b4a0c139b1c7d6f on diff_only_issue_5 into<PHONE_NUMBER>5e894a18927d87363fe609825ee627 on master.

Coverage remained the same when pulling 4b5f2bb33bf1eaa28537afe0beb7ae5339067852 on diff_only_issue_5 into 52537b506d07fdf55dc37f8647cf1c32e9f9ab1c on master.


Make keyboard parsing zoom

We can make keyboard_func zoom a little more.
The fastest way to do it is with a nasty map thing.
Put this at the top of display.c. We only want to build it once or it will be way slower.
#define ESC 27
...
static vec3 lowercase_to_vec3[26][3] = {
    {-0.01, 0, 0},  // 'a'
    {-1}, {-1},
    {0.01, 0, 0}, {0, 0.01, 0,},  // 'd' and 'e'
    {-1}, {-1}, {-1}, {-1}, {-1}, {-1},
    {-1}, {-1}, {-1}, {-1}, {-1},
    {0, -0.01, 0},  // 'q'
    {-1},
    {0, 0, -0.01},  // 's'
    {-1}, {-1}, {-1},
    {0, 0, 0.01},  // 'w'
    {-1}, {-1}, {-1}
};

Now the logic becomes super easy.
/**
 * Exit program when escape is pressed.
 * Handle camera movement input.
 */
void keyboard_parsing(unsigned char c) {
    char lowercase_char = c - 'a';
    vec3 *offset;

    if(c == ESC)
        exit(0);
    offset = lowercase_to_vec3[lowercase_char];
    if(*offset != -1)
        move_camera(CTX, *offset);
}

The old implementation would not be able to make use of branch prediction, but this should just step forward like an idiot and cache it on the way.
Also I have no idea if this works, I haven't run it, but it should be pretty close.

fixed in: 83e94e8716a39c01a1a662d96aba44d8e300a5f1
we're absolutely zoomin,it only took me a year


Does not build because of release of nan 2.0.0

npm install<EMAIL_ADDRESS>Fails to build since<EMAIL_ADDRESS>is released 4 days ago.
A snippet of the error log
  CXX(target) Release/obj.target/node_sleep/sleep.o
In file included from ../sleep.cc:2:0:
../node_modules/nan/nan.h: In constructor ‘Nan::Utf8String::Utf8String(v8::Local<v8::Value>)’:
../node_modules/nan/nan.h:1178:27: error: ‘REPLACE_INVALID_UTF8’ is not a member of ‘v8::String’
                           v8::String::REPLACE_INVALID_UTF8;
                           ^
../sleep.cc: In function ‘Nan::NAN_METHOD_RETURN_TYPE Sleep(Nan::NAN_METHOD_ARGS_TYPE)’:
../sleep.cc:36:12: error: ‘NanScope’ was not declared in this scope
   NanScope();
            ^
../sleep.cc:38:7: error: ‘args’ was not declared in this scope
   if (args.Length() < 1 || !args[0]->IsUint32()) {
       ^
../sleep.cc:39:54: error: ‘NanThrowError’ was not declared in this scope
     return NanThrowError("Expected number of seconds");
                                                      ^
../sleep.cc:39:54: error: return-statement with a value, in function returning 'void' [-fpermissive]
../sleep.cc:42:9: error: ‘args’ was not declared in this scope
   sleep(args[0]->Uint32Value());
         ^
../sleep.cc:44:22: error: ‘NanReturnUndefined’ was not declared in this scope
   NanReturnUndefined();
                      ^
../sleep.cc: In function ‘Nan::NAN_METHOD_RETURN_TYPE USleep(Nan::NAN_METHOD_ARGS_TYPE)’:
../sleep.cc:48:12: error: ‘NanScope’ was not declared in this scope
   NanScope();
            ^
../sleep.cc:50:7: error: ‘args’ was not declared in this scope
   if (args.Length() < 1 || !args[0]->IsUint32()) {
       ^
../sleep.cc:51:59: error: ‘NanThrowError’ was not declared in this scope
     return NanThrowError("Expected number of microseconds");
                                                           ^
../sleep.cc:51:59: error: return-statement with a value, in function returning 'void' [-fpermissive]
../sleep.cc:54:10: error: ‘args’ was not declared in this scope
   usleep(args[0]->Uint32Value());
          ^
../sleep.cc:56:22: error: ‘NanReturnUndefined’ was not declared in this scope
   NanReturnUndefined();
                      ^
../sleep.cc: In function ‘void init(v8::Handle<v8::Object>)’:
../sleep.cc:61:16: error: ‘NanNew’ was not declared in this scope
   exports->Set(NanNew<String>("sleep"), NanNew<FunctionTemplate>(Sleep)->GetFunction());
                ^
../sleep.cc:61:29: error: expected primary-expression before ‘>’ token
   exports->Set(NanNew<String>("sleep"), NanNew<FunctionTemplate>(Sleep)->GetFunction());
                             ^
../sleep.cc:61:64: error: expected primary-expression before ‘>’ token
   exports->Set(NanNew<String>("sleep"), NanNew<FunctionTemplate>(Sleep)->GetFunction());
                                                                ^
../sleep.cc:61:74: error: request for member ‘GetFunction’ in ‘Sleep’, which is of non-class type ‘Nan::NAN_METHOD_RETURN_TYPE(Nan::NAN_METHOD_ARGS_TYPE) {aka void(const Nan::FunctionCallbackInfo<v8::Value>&)}’
   exports->Set(NanNew<String>("sleep"), NanNew<FunctionTemplate>(Sleep)->GetFunction());
                                                                          ^
../sleep.cc:62:29: error: expected primary-expression before ‘>’ token
   exports->Set(NanNew<String>("usleep"), NanNew<FunctionTemplate>(USleep)->GetFunction());
                             ^
../sleep.cc:62:65: error: expected primary-expression before ‘>’ token
   exports->Set(NanNew<String>("usleep"), NanNew<FunctionTemplate>(USleep)->GetFunction());
                                                                 ^
../sleep.cc:62:76: error: request for member ‘GetFunction’ in ‘USleep’, which is of non-class type ‘Nan::NAN_METHOD_RETURN_TYPE(Nan::NAN_METHOD_ARGS_TYPE) {aka void(const Nan::FunctionCallbackInfo<v8::Value>&)}’
   exports->Set(NanNew<String>("usleep"), NanNew<FunctionTemplate>(USleep)->GetFunction());
                                                                            ^
make: *** [Release/obj.target/node_sleep/sleep.o] Fout 1

npm install<EMAIL_ADDRESS>Did not work either
  CXX(target) Release/obj.target/node_sleep/sleep.o
In file included from ../sleep.cc:2:0:
../node_modules/nan/nan.h: In constructor ‘Nan::Utf8String::Utf8String(v8::Local<v8::Value>)’:
../node_modules/nan/nan.h:1178:27: error: ‘REPLACE_INVALID_UTF8’ is not a member of ‘v8::String’
                           v8::String::REPLACE_INVALID_UTF8;
                           ^
make: *** [Release/obj.target/node_sleep/sleep.o] Fout 1

Which version of node are you using? You need a version of node which has the REPLACE_INVALID_UTF8 patch it seems.

We are using node v0.10.24

Seeing the same on Node v0.10.38.

Looks like the REPLACE_INVALID_UTF8 patch is in Node 0.10.29 and above: https://github.com/joyent/node/commit/881ac26f27f4ac9585d66c8d8a67d5b246a23d1b

Yes if you want to use node v0.10 you need to patch to v0.10.29 or later.


Drawing text with graphical tileset

There doesn't seem to be an easy way to render text when using a graphical tileset.
If I try and render text it errors saying it cannot find a matching tile for that glyph, which is correct as I have only defined a small subset of graphical tiles, and also have not added any text glyphs to my tileset.
I was wondering if there's a way to get around having to text with a graphical tileset? Could you create two Display objects, one for the tileset and one for text rendering, then position absolute the two canvas containers over top of each other?

Hello, we tried to solve the issue.
This is what we did:
Added support for text rendering on top of graphical tilesets by creating two separate canvas layers - one for the tileset and one for text. Modified the Display class to manage both layers and updated relevant methods to handle text rendering separately.
You can review changes in this commit: https://github.com/mark-addams001881/ondras-rot.js-224/commit/d9ea687358ec7b6927f8bede373c5620b600d87e.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

Latta AI seeks to solve problems in open source projects as part of its mission to support developers around the world. Learn more about our mission at https://latta.ai/ourmission . If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Favour January timeline

[ ] Create a tutorial on how to use the Mindee API
[ ] Write blog posts at least 2 or more
[ ] January newsletter
[ ] Read through the docs and make changes where necessary
[ ] Review the documentation architecture
[ ] Come up with plan on how to move the computer vision SDK
[ ] Review and suggest changes to Doctr Documentation

What is Create a tutorial on how to use the Mindee API? Don't have some already in the doc?
Also, can you be more specific Read through the docs and make changes where necessary, like a timeline plan to help us focus and see where it's going. Something like:
Week 1: Review getting started section
Week 2: Review Mindee's REST API section
...
Maybe creating the issues for each of these steps or even pages: seems overkill, but it would be nice to have in the milestones, and see that we are making progress.


Feature/89 fuzzy calendar

Edit of granule_size does not work for me. UI component behaves not the way I expect.
It always shows 1 second despite the fact that I put number different than 1 (e.g., 60) and the system allowed me to save it.


python3Packages.qiskit: 0.24.1 -> 0.25.0

Motivation for this change
https://github.com/Qiskit/qiskit/releases/tag/0.25.0.

Adds tweedledum, an optional-ish dependency of qiskit-terra
Updates pylatexenc

Things done

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

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

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

python3.8-qiskit-aqua> Requirement already satisfied: numpy<=1.20.1,>=1.17 in /nix/store/pwjsgyqgsvqi5wg5h7rqk57i9qv9lqyc-python3.8-numpy-1.20.1/lib/python3.8/site-packages (from qiskit-aqua==0.9.0) (1.20.1)
python3.8-qiskit-aqua> Requirement already satisfied: yfinance<=0.1.55 in /nix/store/rl858rjjm13385hs9v5ksh3nhkcvpiai-python3.8-yfinance-0.1.55/lib/python3.8/site-packages (from qiskit-aqua==0.9.0) (0.1.55)
python3.8-qiskit-aqua> ERROR: Could not find a version that satisfies the requirement h5py<=3.1.0 (from qiskit-aqua)
python3.8-qiskit-aqua> ERROR: No matching distribution found for h5py<=3.1.0

python3.8-qiskit-aqua> Requirement already satisfied: numpy<=1.20.1,>=1.17 in /nix/store/pwjsgyqgsvqi5wg5h7rqk57i9qv9lqyc-python3.8-numpy-1.20.1/lib/python3.8/site-packages (from qiskit-aqua==0.9.0) (1.20.1)
python3.8-qiskit-aqua> Requirement already satisfied: yfinance<=0.1.55 in /nix/store/rl858rjjm13385hs9v5ksh3nhkcvpiai-python3.8-yfinance-0.1.55/lib/python3.8/site-packages (from qiskit-aqua==0.9.0) (0.1.55)
python3.8-qiskit-aqua> ERROR: Could not find a version that satisfies the requirement h5py<=3.1.0 (from qiskit-aqua)
python3.8-qiskit-aqua> ERROR: No matching distribution found for h5py<=3.1.0

This should be fixed now. I hadn't realized that qiskit-aqua had pinned all of its dependencies to <= the current versions, which obviously doesn't work with nixpkgs's rolling releases. Fixed by unpinning all requirements in requirements.txt.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 118729 run on x86_64-linux 1

18 packages built:

python38Packages.Quandl (python38Packages.quandl)
python38Packages.pylatexenc
python38Packages.qiskit
python38Packages.qiskit-aer
python38Packages.qiskit-aqua
python38Packages.qiskit-ibmq-provider
python38Packages.qiskit-ignis
python38Packages.qiskit-terra
python38Packages.tweedledum
python39Packages.Quandl (python39Packages.quandl)
python39Packages.pylatexenc
python39Packages.qiskit
python39Packages.qiskit-aer
python39Packages.qiskit-aqua
python39Packages.qiskit-ibmq-provider
python39Packages.qiskit-ignis
python39Packages.qiskit-terra
python39Packages.tweedledum


feat: Implement queryWithBinds for bind variables

What does this PR do?
What issues does this PR fix or reference?
This PR implements queryWithBinds  in Spring 23 where applicable instead of using bind variables in SOQL
Notes for review - The new Security method replacements for queries and DML will be implemented in separate PR.
The PR fulfills these requirements:
[ ✔] Tests for the proposed changes have been added/updated.
[ ✔] Code linting and formatting was performed.
Functionality Before
<insert gif and/or summary>
Functionality After
<insert gif and/or summary>

Codecov Report
Base: 94.41% // Head: 94.47% // Increases project coverage by +0.06% :tada:

Coverage data is based on head (c1ebaa5) compared to base (59bd239).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@                   Coverage Diff                   @@
##           prerelease/spring23     #388      +/-   ##
=======================================================
+ Coverage                94.41%   94.47%   +0.06%
=======================================================
  Files                       58       58
  Lines                     1968     1992      +24
=======================================================
+ Hits                      1858     1882      +24
  Misses                     110      110

Flag
Coverage Δ

Apex
94.47% <100.00%> (+0.06%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...efault/classes/Data Recipes/DynamicSOQLRecipes.cls
100.00% <100.00%> (ø)

...ain/default/classes/Files Recipes/FilesRecipes.cls
84.52% <100.00%> (+0.77%)
:arrow_up:

...asses/Invocable Recipes/InvocableMethodRecipes.cls
92.85% <100.00%> (+1.55%)
:arrow_up:

...pp/main/default/classes/LDV Recipes/LDVRecipes.cls
75.00% <100.00%> (+4.62%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


org.knowm.xchange.kucoin  GET /api/v1/fills

Could you please add orderId support for public Pagination getKucoinFills in package org.knowm.xchange.kucoin
Something like:
public Pagination getKucoinFill(String orderId) throws IOException {
checkAuthenticated();
return classifyingExceptions(
() ->
fillApi.queryTrades(
apiKey,
digest,
nonceFactory,
passphrase,
null,
orderId,
null,
null,
null,
null,
10,
1));
}

Hello, checking the FillApi.java interface and it looks like pagination is supported. @earce or whomever else might want to close this. :+1:


fix: remove decodeURI from boostagram message

the message is a random string provided by the user

Thanks!
Adjusted the tests.

ah, thanks for the test!!


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In daaf17d, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 15868 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in 1997ed0 after 1 hour, 57 minutes.


Duplicate entries in Gazetteer for Devonport

There are 2 correct entries for Devonport DEV in the Gazetteer - Devonport and Plymouth Docks.
These two entries have been included in the Gazetteer under Plymouth as well.
Can the search links for each of these plces under Plymouth please be moved to the same entries under Devonport and the Plymouth entries removed.
Geoff

Done


Use useEffect to avoid unsafe side effects for useAccount callbacks

Description
When using the onConnect and onDisconnect callbacks in an app with react strict mode on, the current implementation ends up causing the app state to become unstable.  This has something to do with the double rendering of components when a react app, in strict mode, is running the dev server.
Additional Information

[x] I read the contributing docs (if this is your first contribution)

Your ENS/address:
ajmay.eth
Related Issues
I believe these issues are related to this fix:

https://github.com/rainbow-me/rainbowkit/issues/837
https://github.com/rainbow-me/rainbowkit/issues/836
https://github.com/rainbow-me/rainbowkit/pull/891

As these issues usually occur after an onConnect or onDisconnect callback is called and when logging the state, you can see that it reverts on the second render.

@jxom @tmm would be happy to get your thoughts on this one.  I got the tests to run locally.  I'm getting 3 errors but none related to this functionality.  Also, tested in the playground and all seems to be fine.

Very excited to not need to remove strict mode from projects where rainbow kit is used

Appreciate the patience! Will have some time to take a look soon.

@aj-may spent some time looking into this and it doesn't seem like anything is wrong here. The dev playground has used strict mode since April 2022. Can you describe some more what you mean by "app state to become unstable"? I'm seeing onConnect and onDisconnect callbacks fire appropriately in the dev playground and test suite.
(Side note: The issues you linked for RainbowKit are definitely issues, but quickly poked around there and they seem unrelated to wagmi.)

@tmm Ahh, I don't think this bug will rear its head unless there is a component that is using these callbacks to modify its own state.  in that case the dependent components state becomes unstable, or reverts to its previous state on the second render under react strict mode.
Let me see if I can make a quick branch to add something to the playground that can replicate the bug.

@tmm I was able to add an example to reproduce here: https://github.com/aj-may/wagmi/tree/side-effect-playground

@tmm Have you had a chance to pull down the branch I created that reproduces the issue?   Would love if we can revisit this and potentially merge this in.

@tmm The changes you pushed cause the connect and disconnect callbacks to be called twice in dev mode with react strict mode on and still shows some state fluctuations over the multiple renders in the console.

Prior to that the console looked like this:

👍 I'll take this for a spin in a couple hours

Looking good!


Fix the typos in quick start guide and adding the namespace explicitly

Without the -n sparkoperator I wasn't able to see the deployment.
wrt. kubectl describe sparkapplication spark-pi it just looks better, although the previous form was also working

Thanks for fixing!


[Suggestion] Allow us to convert loot capacitors into stackable versions that take a slight performance hit

Most of these capacitors don't even work in an Anti-barrel and it gets dangerous to have too much NBT data stored in one area. Keeping all of the loot capacitors that came out of my mob farm/lootbags corrupted a chunk of mine. I'd like to put them through a crafter or something that can strip the NBT data and stack them without losing all of the benefits entirely.

But the nbt data is how they are useable in machines. If you remove that they will be no longer usable in machines.

Could we make a machine that eats loot capacitors and spits out weaker generics? This would be a better solution than trashing all loot capacitors which is currently the only thing I can do to prevent chunk corruption from storing massive amounts of them.

You could make a shapeless recipe using CraftTweaker and then use the EIO Crafter or another auto crafting system.

Yeah that's the best option. They really shouldn't be able to cause chunk corruption by themselves, unless you are storing thousands of them. It doesn't really make sense to add a mechanic to make them more generic since really, you can jue take a regular capacitor then.


[5.5] Router - Minor changes and some tests added

Fyi: Nothing functional was changed at these methods.

fyi, can even do 1-liners in the future if it is the end of the method:
return ($this->current()) ? $this->current()->getAction()['controller'] : null;


Feature request: view log output in admin interface

It would be useful to be able to see recent log output through the admin web interface (tail -n 50 or something) for each of mailserv's log files.

Any chance of getting this pulled in?