Fails to build on FreeBSD

$ rustc -Vv
rustc 1.72.0 (5680fa18f 2023-08-23) (built from a source tarball)
binary: rustc
commit-hash: 5680fa18feaa87f3ff04063800aec256c3d4b4be
commit-date: 2023-08-23
host: x86_64-unknown-freebsd
release: 1.72.0
LLVM version: 16.0.5

$ cargo build
[...]
error[E0308]: mismatched types
    --> src/freedesktop.rs:706:43
     |
706  |     let count = unsafe { libc::getmntinfo(&mut fs_infos, libc::MNT_WAIT) };
     |                          ---------------- ^^^^^^^^^^^^^ expected `*mut *mut statfs`, found `&mut *mut statvfs`
     |                          |
     |                          arguments to this function are incorrect
     |
     = note:    expected raw pointer `*mut *mut statfs`
             found mutable reference `&mut *mut statvfs`
note: function defined here
    --> cargo-crates/libc-0.2.149/src/unix/bsd/freebsdlike/freebsd/mod.rs:5222:12
     |
5222 |     pub fn getmntinfo(mntbufp: *mut *mut ::statfs, mode: ::c_int) -> ::c_int;
     |            ^^^^^^^^^^

error[E0609]: no field `f_mntfromname` on type `&statvfs`
   --> src/freedesktop.rs:714:20
    |
714 |         if fs_info.f_mntfromname[0] == 0 || fs_info.f_mntonname[0] == 0 {
    |                    ^^^^^^^^^^^^^ unknown field
    |
    = note: available fields are: `f_bavail`, `f_bfree`, `f_blocks`, `f_favail`, `f_ffree` ... and 6 others

error[E0609]: no field `f_mntonname` on type `&statvfs`
   --> src/freedesktop.rs:714:53
    |
714 |         if fs_info.f_mntfromname[0] == 0 || fs_info.f_mntonname[0] == 0 {
    |                                                     ^^^^^^^^^^^ unknown field
    |
    = note: available fields are: `f_bavail`, `f_bfree`, `f_blocks`, `f_favail`, `f_ffree` ... and 6 others

error[E0609]: no field `f_fstypename` on type `&statvfs`
   --> src/freedesktop.rs:718:45
    |
718 |         let fs_type = c_buf_to_str(&fs_info.f_fstypename).unwrap_or_default();
    |                                             ^^^^^^^^^^^^ unknown field
    |
    = note: available fields are: `f_bavail`, `f_bfree`, `f_blocks`, `f_favail`, `f_ffree` ... and 6 others

error[E0609]: no field `f_mntonname` on type `&statvfs`
   --> src/freedesktop.rs:719:52
    |
719 |         let mount_to = match c_buf_to_str(&fs_info.f_mntonname) {
    |                                                    ^^^^^^^^^^^ unknown field
    |
    = note: available fields are: `f_bavail`, `f_bfree`, `f_blocks`, `f_favail`, `f_ffree` ... and 6 others

error[E0609]: no field `f_mntfromname` on type `&statvfs`
   --> src/freedesktop.rs:726:48
    |
726 |         let mount_from = c_buf_to_str(&fs_info.f_mntfromname).unwrap_or_default();
    |                                                ^^^^^^^^^^^^^ unknown field
    |
    = note: available fields are: `f_bavail`, `f_bfree`, `f_blocks`, `f_favail`, `f_ffree` ... and 6 others

Some errors have detailed explanations: E0308, E0609.
For more information about an error, try `rustc --explain E0308`.

This shouldn't happen with the most recent version of libc which is enforced via manifest, so I don't know what's going on here.
The assumption here is that you are using the latest version which seems to be the case.
CC the author of this port @0323pin

@Byron I use NetBSD and not FreeBSD. So, I don't have a test environment to play with this.
Apparently, FreeBSD supports both the more recent statvfs and the older statfs.
A quick look seems to indicate that these functions in libc are using statfs 😞
Two options here, either a PR on libc to make these function use statvfs instead of statfs or, conditioning my previous change here to NetBSD.
It's easy to test a FreeBSD specific patch for libc to see how that works, @jbeich are you willing to try that?

I should add that in NetBSD, statfs is not an option since,

The statvfs(), statvfs1(), fstatvfs(), and fstatvfs1() functions first appeared in NetBSD 3.0 to replace the statfs() family of functions which first appeared in 4.4BSD

This is ages ago.
As for FreeBSD there's support for both, so why not switching to statvfs by default?

either a PR on libc to make these function use statvfs instead of statfs

How would this work? libc (Rust) crate imports a subset of libc (C) where FreeBSD getmntinfo(3) uses statfs. Switching to statvfs would break existing consumers and require statfs -> statvfs polyfill in a crate that is not supposed to abstract away OS differences.

@jbeich As mentioned, I don't use FreeBSD. I thought statfs existed only for backwards compatibility.
Although, these things should perhaps have been wrapped in a non-OS specific way, in this case the easiest way forward is probably to split this into two different sections.  One for target_os = netbsd and another for target_os = freebsd and target_os = openbsd.

@jbeich Can you please run the following and confirm it builds on FreeBSD?
git clone https://github.com/0323pin/trash-rs
cd trash-rs
cargo build --release

Thanks!

@Byron I don't think we really need to wait for @jbeich to actually answer this. All I did was restore what was there previously for FreeBSD and the new section for NetBSD has been tested locally.
Unfortunately, nushell had a new release during this time. So just an heads-up that building this new release on FreeBSD or, on OpenBSD will fail.
Sorry for the inconvenience. Please refer to this issue or, to the PR, if you hear complains.
Regards

A new release was made that should fix this issue: https://github.com/Byron/trash-rs/releases/tag/v3.1.1 .

I confirm, v3.1.1 builds fine on FreeBSD.


Space- Jeta

React Tic Tac Toe
Congratulations!  Please reflect on the assignment with these questions.
Reflection

Prompt
Response

How are events / event handlers and useState connected?
In App.js it’s a function called onClickCallback, first I am passing that function down to the tree as a props. In the Square.js I build a function onSquareClick that will get called when the event is triggered and will pass the id to onClickCallback and when the id gets there, this function will check if this id is equal to the square that is clicked and if the value of that square is empty. If this is true will assign either X or O and will setSquares(newSquare);  or that Array will get changed after every “true” click.

What are two ways to do "dynamic styling" with React? When should they be used?
It's Inline CSS that uses style as an attribute and we have an External Style that uses className as an attribute.

Much like Rails works with the HTTP request->response cycle, React works with the browser's input->output cycle. Describe React's cycle from receiving user input to outputting different page content.
When the user clicks on the button is kind of the input, when this event gets triggered  the onClickCallback method will be called and will change the states and the changes will be rendered through DOM in the browser(output)

CS Fundamentals Questions

Question
Answer

What do you think is the BigO complexity of the method you use to compute a winner?
(O)n I thinnk.

Consider what happens when React processes a state change from setState -- it must re-render all of the components that now have different content because of that change.What kind of data structure are the components in, and what sort of algorithms would be appropriate for React's code to "traverse" those components?Speculate wildly about what the Big-O time complexity of that code might be.
Components  are in tree data structure, each component is a nodo and each of them may have a parent but not necessary child. And I think that time complexity is O(n) where n is the number of the nodes in a tree.

  Instead describe the functionality added, like, "completed calculateWinner function

Uses existing stylesheets to render components
✔

Functional Requirements

Functional Requirement
yes/no

The Square component renders properly and executes the callback on a click event.
✔

The Board component renders a collection of squares
✔

The App component renders a board and uses state to maintain the status of the game.
✔

Utilizes callbacks to UI events to update state
✔

Overall Feedback

Overall Feedback
Criteria
yes/no

Green (Meets/Exceeds Standards)
5+ in Code Review && 3+ in Functional Requirements

Code Style Bonus Awards

Was the code particularly impressive in code style for any of these reasons (or more...?)

Quality
Yes?

Perfect Indentation
✅

Descriptive/Readable
✅

Concise
✅

Logical/Organized
✅

Summary
Well done, you hit the learning goals here.  Nice work!


permission to access “lists” not avaliable when trying to add sources

hello, i am not able to add any sources to my package manager because of some error

the refreshing icon cache maybe?


serial port support

@homunkulus r+

:pushpin: Commit d392fba has been approved by japaric

:hourglass: Testing commit d392fba with merge d392fba...

:pushpin: Commit c35bc19 has been approved by japaric

@homunkulus r+

:hourglass: Testing commit c35bc19 with merge c35bc19...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing c35bc197e14d8f7f25e685114138b804718a6c35 to master...


Data privacy of Slack archive

Hello, thanks for your great tool! I just have one concern: could anyone access my slack files by "guessing" the token? Wouldn't it more safe to also include a password?

In theory yes. However, since there are so many combinations it's practically impossible. There are 122 random bits - i.e. 2^122 possible permutations which is a number greater than the number of atoms in our universe from what I could find.
See https://security.stackexchange.com/a/53477
The UUIDs I'm using fall into the last category listed in the answer.


Validate profiles before writing them to disk

We could add a pre validation step to parse the seccomp profiles before writing them to disk. This way we can ensure that only valid profiles are written down. I would report errors with events as well as directly at reconciliation level. WDYT? Would this be suitable for v1?
/area security
Refers to https://github.com/seccomp/containers-golang/issues/34

Let's close this now
/close


Document how to host ASP.NET Core in a Windows Service

Using https://docs.microsoft.com/en-us/aspnet/core/api/microsoft.aspnetcore.hosting.windowsservices

Note, this currently only works on full .NET.

https://github.com/lankaapura/MusicStoreAsWindowsService

@muratg Who can help out with the outline and technical details for this one?

@Tratcher Can you help out with the outline and technical details for this one?

https://github.com/aspnet/Hosting/issues/771#issuecomment-270608527

@Tratcher  what are the scenarios for running in a windows service? Why would you do that?

When you want it to always be running. Windows will start it for you on boot, restart it if it crashes, etc..

Proposed outline for the doc:

Overview:

Article explains how to run in Windows Service.
Scenario:

You want to run on Windows,
You don't want to use IIS,
You want automatic restart after reboot or crash

Prerequisites

Runs only on .NET Framework

(.csproj example showing TargetFramework and RuntimeIdentifier)
When creating a project in VS, use the ASP.NET Core Application (.NET Framework) template.

Use WebListener rather than Kestrel for edge deployments.  See When to use Kestrel with a reverse proxy

Getting started

Install package Microsoft.AspNetCore.Hosting.WindowsServices.
In Program.Main:

Use an absolute path to the publish location for UseContentRoot
Call host.RunAsService instead of Run
(code snippet example)

Publish
Create the service (sc create servicename binPath="path to executable")
Start the service (sc start servicename or use Services MMC)
Can now browse to same path as when running as console app

Provide a way to run outside of a service

Check command-line args and/or debugger attached and call RunAsService only if running as a service
(code snippet example)

Handle stopping and starting events

Create an implementation of WebHostService and IWebHost extension method.
In Program.Main change the host.RunAsService command to use new extension method
(code snippet example)

Sample app

Create from MVC web app template, add package and code needed for example snippets.

@Tratcher please review, plus a question:  can you point me to an example that shows how to get services (e.g., a logger) in a WebHostService derived class?

Looks good.
As for services, they're on the IWebHost, and your constructor should have access to that.
https://github.com/aspnet/Hosting/blob/dev/src/Microsoft.AspNetCore.Hosting.WindowsServices/WebHostService.cs#L30-L32

Review URL


test: add integration test

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


storage: Allow merging multiple segments in the segment API

What problem does this PR solve?
Issue Number: ref #5237
Problem Summary:
What is changed and how it works?
This is one of a series of PRs for #5237, where you can see the overall roadmap.

Segment - Adjusted interface to accept merging multiple segments.
Note: The path of merging multiple segments is covered in tests. It is not used in production in this PR, as checkSegmentUpdate will always merge two segments.

Segment - Added [[nodiscard]] to the new segments returned in segment APIs to avoid forgetting using them.

DeltaMergeStore - Added sleep to the flush inside segment merge, in case of a flush is already in progress.

Segment - Added utility function Segment::simpleInfo and Segment::info for pretty-printing a vector of SegmentPtrs. They are also used in merging segments.

SegmentTest - Added new unit tests:

TestSegmentMemTableDataAfterSplit
TestSegmentMergeTwo
TestSegmentMergeThree
TestSegmentMergeInvalid

SegmentTest - Updated the segment test framework to test with merging multiple segments. The randomized test will randomly pick 2 ~ 6 continuous segments.

SegmentTest - Added a probability config for different segment operations in the randomized segment test, so that there is more chance to test with merging multiple segments.

SegmentTest - Changed write_rows from 8192 to default 100 when writing rows with deleted pack, because when writing rows with data we also write 100 rows by default.

SegmentTest - Write, Ingest and WriteWithDeletedPack now write continuous rows randomly in the segment range, instead of always write at the beginning of the segment.

SegmentTest - Randomized write now write 20 ~ 100 rows randomly, instead of always write 100 rows.

SegmentTest - Changed to use the random in c++11, so that we can use different distribution utilities.

SegmentTest - Fix: Now the segment range generated by the test conforms with the check range->isStartInfinite or range->isEndInfinite when is_common_handle = true. This prevents possible bug that only happens in the test due to a wrong segment range. For example, the check whether this is the last segment will not work correctly in previous test framework, but will work in production.

SegmentTest - Fix: Prevent writing 0-sized block when int segment range cannot contain any valid rows. This may happen with large number of splits in the test.

SegmentTest - Improved some loggings to better track operations and understand what happened.

[ ] This PR is based on https://github.com/pingcap/tiflash/pull/5835. Merge it first to get a nicer diff.

Check List
Tests

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

Side effects

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

Documentation

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

Release note

None

/rebuild

/run-all-tests

/merge


Server address patch

Moving parsing to ServerAddress where it can be tested and doesn't bloat ServerInformation.
Not sure why the project.json.lock files are included in this...

Hi @ardalis, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Can you fix the commit? The lock file and other changes shouldn't be included. Maybe you did a merge before.

I accidentally started on master with this PR. Delete and use PR #121 instead.


Multi gpu model, low GPU utilization

Training using GPU only utilize 1% of my GPU, anybody able to enhance it?
Maybe using keras.utils, multi_gpu_model?

使用GPU的训练仅使用我的1％的GPU，谁可以增强它？
也许使用keras.utils，multi_gpu_model？

Most of time is spent in MCTS algorithm while training, so gpu won't help much unless you use a deeper
neural network.
训练时绝大多时间花费在蒙特卡洛树搜索上面，所以用gpu加速效果不明显，除非用更深的神经网络。


Delay tomcat startup

Can we somehow delay Tomcats startup until Oracle is up and running? Had an issue today where I had to manually restart tomcat as the database wasn't up and running when ORDS started up and thus /ords returned a 404 error (until restarting Tomcat)

Maybe a post_install.sh? I have had a different issue with ORDS 3.0.6. Had to touch /etc/ords/tomcat before restarting Tomcat.

Was thinking one solution could be to convert oracle to a systemd service, then I believe you can have a dependency chain. Haven't investigated how easy this is for the oracle-xe sysV init script.
https://fedoramagazine.org/systemd-converting-sysvinit-scripts/

@tschf I've had the same issue that after the initial OXAR reboot am getting the ORDS 404 error. Fix is to restart again. Would be good if we didn't have to do this and also a lot less confusing for others.

Yes, it's on my radar. I made a start trying to convert the SysInit script a while ago, but didn't get anywhere. Haven't had time to carry on.

Actually, I think converting it is an unnecessary step.
We can try this. Edit /usr/lib/systemd/system/tomcat.service. and modify the line to be: After=syslog.target network.target oracle-xe.service.
I'll set this up soon - not sure of a good way to reliably reproduce the problem though.

@tschf only way I've encountered it is after an OXAR installation (right after the countdown reboot).

As per PR, I've added oracle-xe to the After setting in a new tomcat service file<EMAIL_ADDRESS>and for Debian, added to the Should-Start setting. Tomcat should now be started with tomcat@oxar rather than tomcat (the tomcat service is disabled in favour of tomcat@oxar).


Mquezada/SWFlows

Non intrusive PR for running Shallow Water problems easily. The idea is to share the _so, _p and _n files for all the problems.

@ejtovar I've updated the branch to get this ready to  review/refactor but there are a few tests failing now for SWE, for  simple reasons, I think. Could you take look?
[gw0] [ 47%] FAILED proteus/tests/SWFlows/test_SWFlows.py::TestSWFlows::test_solitary
[gw0] [ 48%] FAILED proteus/tests/SWFlows/test_SWFlows.py::TestSWFlows::test_parab1D
[gw0] [ 48%] FAILED proteus/tests/SWFlows/test_SWFlows.py::TestSWFlows::test_dam3Bumps

@ejtovar and @manuel-quezada the SWCV.h implementations have the same issues as the TwoPhaseFlow PR in terms of allocation of large arrays on the stack (e.g. register double[numDOFs]).

The tests are initially failing in travis because the SWE app tests are written in such a way that they can only be called from inside that specific directory.The tests need to be written so they can be called from anywhere (e.g. py.test FULL_PATH_TO_TEST.py).  I put in a fix for that, but the tests are still failing for the app. @manuel-quezada and @ejtovar can you take a look at tests/SWFlows and see why just those three tests fail to match the reference data?

I figured the problem about the location of the tests in the TwoPhaseFlow PR and fixed it there but not here. About the tests, they might be failing just because of known changes on the code. I think this PR was open since before I went the last time to TAMU to incorporate convex limiting so I would not worry much about it. But maybe it is safe to run all tests for longer times and make sure that the solutions look fine. @ejtovar can you take care of that?

Yes, I can look into it @manuel-quezada and @cekees.


vdk-heartbeat: cover requirements.txt automatic installs

Upon vdk deploy the Control Service is responsible for installing all external dependecies defined in requirements.txt of the data job so they can be used during a cloud run.
A recent outage in one of our customers caused data jobs to ignore their requirements.txt and not install any external libraries. It was trace to be due to a release of data job base
image
CICD of the open source VDK did not catch this.
This change aim to address this gap and to cover this important functionality for correct installation of data job's python requirements. And also we can use it as a way to reproduce the issue and further root cause
See also https://github.com/vmware/versatile-data-kit/issues/2391

Is this PR just a test or a fix for the issue too?
I dont see a fix for the issue?

Is this PR just a test or a fix for the issue too? I dont see a fix for the issue?

It is only a test. But it fails to reproduce the issue. So I am trying to figure out why ...
It's still valuable coverage. So likely I will keep it.

Is this PR just a test or a fix for the issue too? I dont see a fix for the issue?

The fix is in https://github.com/vmware/versatile-data-kit/pull/2429

It is only a test. The fix(es) would be separate PR(s). But it fails to reproduce the issue. So I am trying to figure out why ...

Wait, I'm confused now. 😄 If you're just testing something, please mark it as draft.

It is only a test. The fix(es) would be separate PR(s). But it fails to reproduce the issue. So I am trying to figure out why ...

Wait, I'm confused now. 😄 If you're just testing something, please mark it as draft.

No, I mean , this is not fixing the issue , The fix is in separate PR (#2429).
I am extending VDK heartbeat to test for the issue so we can catch it in the future.


Wait for known USB device -t timeout is not working

It seems like Wait for known USB device timeout stopped working between version 1.4.127 and 1.4.138.
uuu -m 2:841 -t 2 FB: ucmd
uuu (Universal Update Utility) for nxp imx chips -- libuuu_1.4.127-0-g08c58c9

Success 0    Failure 0    Wait for Known USB Device Appear... at path 2:841

Error: Timeout: Wait for USB Device Appear

Timeout is after 2s as it should, but from version >=1.4.138, timeout is not working and uuu keeps waiting forever. It never times out.

Thx, would it be possible to get a prerelease with this included?


Add idempotence sub command

A molecule v2 tracking issue.
Add the idempotence sub command and modules into the v2 branch.

Implemented in #691.


Query and Mutations are not generated when the GraphQL API compiled in AWS-Amplify

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
14.20.1
Amplify CLI Version
10.0.0
What operating system are you using?
Ubuntu
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
I used all the default configurations and used amplify add-api to add the graphQL api
Amplify Categories
auth, function, api, hosting
Amplify Commands
configure, publish, pull, push, status, update
Describe the bug
I created graphQL api and made a couple of models in the schema.graphql and compiled the app and pushed in to amplify and checked out the application in AppSync. But the folders are not creating for mutations
Expected behavior
I created graphQL api and made a couple of models in the schema.graphql and compiled the app and pushed it in to amplify and checked out the application in AppSync. Its should create a mutations and subscriptions folder in the src folder
Reproduction steps

Create a amplify api
Create data models to save in the schema.graphql
Compile the API

GraphQL schema(s)

input AMPLIFY {
  globalAuthRule: AuthRule = { allow: public }
} # FOR TESTING ONLY!

type CommonUsers @model {
  id: ID!
  name: String!
  email: String!
  address: String!
  gender: String!
  phone: String!
  userType: String!
  totalLeaves: String!
  leaves: [Leave] @hasMany
}

type Leave @model {
  id: ID!
  name: String!
  startDate: String!
  endDate: String!
  days: String!
  phone: String!
  userID: ID!
  user: CommonUsers @belongsTo(fields: ["userID"])
}

Project Identifier
44530c55f18864b8fb3bbad07c88dd90
Log output

# Put your logs below this line

Additional information
No response

Hi @PulasthiAbey 👋, We are closing this issue as we have not received any response from you. However, if you are still facing the same issue and require our assistance, please don't hesitate to leave a comment. This way, we can reopen the issue and provide you with better assistance.
Thank you!

I still have this issue. I'll update you if I find a solution I'll update


Fix Hercules CI issue post-IFD change

See https://hercules-ci.com/github/purs-nix/purs-nix/jobs/81
Apparently triggered by https://github.com/purs-nix/purs-nix/commit/5db6ec5b6efbe26e8efcdc00ffd36ef632ba02fc

Close to minimal repro: https://github.com/purs-nix/hci-segfault-repro/tree/fb6ce74b317f7c3676d5a5999580882a5aa5d297
Removing just one line makes HCI work: https://github.com/purs-nix/hci-segfault-repro/commit/1799bd77cd660ac62001caf84cbf3f7953a2ce2d

https://github.com/hercules-ci/hercules-ci-agent/issues/445#issuecomment-1272111039

Setting a large stack size fixed it, and CI now works. The only pending issue is to fix this in Nix upstream.


chore: 升级官网行政地图 L7Plot 版本

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] commit message follows commit guidelines

Description of change

冲突了


Incompatible Node versions

Hi,
I'm getting this error when trying to install directory-import:
Your Node version is incompatible with "registry.npmjs.org/directory-import/3.0.12".

Expected version: >=18.17.0
Got: v18.12.1

This is happening because the package's manifest has an engines.node field specified.
To fix this issue, install the required Node version.

I'm using nvm to manage my node version and it seems that their newest v18 version is v18.16.1.
Is your code really relying on v18.17.0 or is it maybe possible to loosen your requirements a bit?
Thanks in advance :)

UPDATE:
I had to install v18.16.1 to install v18.17.0.
So I was able to install the package now. But my point stays valid, maybe some cannot update to the current Node version.


[Question] Handling a very large amount of items in lists

Maybe this was discussed previously, but I've tried to search github issues and didn't find anything related.
I'm curious how Monomer will perform when given a very large data set to render in vscroll (vstack hugeItemList). For example 1000, 10_000, 100_000 items. For each of them there will be a WidgetNode, so there will be not only a sequence of data items, but also a same-sized sequence of widget nodes which will consume memory (and cpu/gpu?).
Or am I missing some obvious way to handle this, or maybe this won't be a problem at all? (for example due to Haskell's laziness getting into play here, but I doubt this will be the case, seems like strictness is preferred when rendering).
I guess this also relates to the question of rendering infinite lists where you can produce new items and widget nodes as soon as user scrolls closer to the current list's end.
I know that mobile APIs invented ways to cope with this, for example Android's Jetpack Compose has LazyColumn and LazyRow. They have a special "dsl" which allows user to provide a lambda index -> WidgetNode which this component calls whenever it builds a new items and it keeps only a handful of nodes on the screen (say, 6) which are then reused for rendering of all the items.
What's the situation for Monomer here?

Hi @dimsuz!
Currently there is no special handling, and performance will degrade once the volume is too high. If the data you display in the list does not change often, and the volume is not too high, it should work well.
I ran simple tests with thousands (tens?) and it worked well in terms of rendering/scrolling; using a million rows may be problematic when data changes, since each widget in the list will be updated, but otherwise it should work fine too. You need to be careful to avoid merging that list every time the model changes (i.e., parts of the model not related to the list itself), as explained in this example and shown here.
It is possible to create a custom container that is smarter about row handling, and there is a short discussion here. The truth is, since I have not have a need for this, I have it in my mind as a thing I'd like to have but it's not one of my priorities. Eventually, whenever I get to make the library work on mobile, I will need to implement it to have decent performance in shorter lists too.

I see! Good to know about mergeRequired, I will remember this and will try it if I ever run into perf problems. So far it's a purely theoretical question for me :)
I want to say that I'm really enjoying writing a small gui app in Monomer, it feels great to use and very well thought out. As a Haskell (intermediate) beginner I also learn by reading bits of its source code from time to time.

Great! Thanks for the kind words!
I'll close the issue for the time being, since the possible solution will not be handled in isolation. Feel free to re-open if needed. Thanks!

@Dretch I missed this, sorry (I got the notification, but since the PR was closed, I forgot about it).
That sounds great! I'll take a look and see how I can incorporate the same idea into the existing widgets.
One option I was thinking about some time ago is the case when all the rows have the same height (which is quite common!); in this scenario, it is easier to figure out what needs to be visible, etc. But, of course, having a general solution is much better. I guess, as you mention, that having this functionality in stack/grid would be better than only having it in selectList (which already incorporates a scroll).
I'll finish a couple of pending things, make a 1.6 release, and take a look into this.


makes quoted status cards optional

is it possible to have the "quoted status cards" introduced in v3.1.1 to be optional?
imo it makes the feed to bulky.

in my case, only the feed.

Version 3.1.6 has the enhancement.
This in an option that can be activated in the general settings.
The option name is Leaner feeds, at the bottom of the general settings:

thank you very much jean-francois.
unfortunately i can not confirm the feature as i see no visible difference.
but that may be totally my fault as i thought that it was related to the enormous link previews.
v3.1.0 did not show the link previews embedded with every tweet if it contained a link.
so v3.1.0 was a mostly clean feed with text only.
all later version one tweet fills more than a half screen because of link-previews.
sorry for any inconvience i created. i hope you did not suffer to much from my incompentence.


Missing navPosition into typescript definition

Before this PR is being merged, we can hack with something like the followings to get rid of missing type.
const tnsSettings: TinySliderSettings = Object.assign({
  // ...
  navPosition: 'bottom'
});


'int' object is not callable

Deck 1: Dragonling Mechanic, River Crocolisk, Silverback Patriarch, Defender of Argus, Wisp, Bloodlust, Deathlord, Flametongue Totem, Chillwind Yeti, Flametongue Totem, Argent Squire, Deathlord, Frostwolf Warlord, Wolfrider, River Crocolisk, Wolfrider, Azure Drake, Nerub'ar Weblord, Ogre Magi, Fire Elemental, Ironforge Rifleman, Argent Squire, Stormpike Commando, Zombie Chow, Sludge Belcher, Gelbin Mekkatorque, Stormpike Commando, Lord of the Arena, Zombie Chow, Kobold Geomancer
Deck 2: Twisting Nether, Youthful Brewmaster, Bloodmage Thalnos, Feugen, Fen Creeper, Silverback Patriarch, Summoning Portal, Sense Demons, Demolisher, Stranglethorn Tiger, Sludge Belcher, Nightblade, Argent Commander, Coldlight Oracle, Corruption, Young Dragonhawk, Old Murk-Eye, Stoneskin Gargoyle, Nerub'ar Weblord, Wailing Soul, Nightblade, Target Dummy, Kobold Geomancer, Wisp, Razorfen Hunter, Acidic Swamp Ooze, Loatheb, Frost Elemental, Mana Addict, Lord Jaraxxus
Traceback (most recent call last):
  File "C:\Users\Ragowit\Documents\GitHub\mcts\hearthbreaker\uct.py", line 140, in DoMove
    self.game.play_card(self.game.current_player.hand[move[3]])
  File "C:\Users\Ragowit\Documents\GitHub\mcts\hearthbreaker\hearthbreaker\game_objects.py", line 2015, in play_card
    card.use(self.current_player, self)
  File "C:\Users\Ragowit\Documents\GitHub\mcts\hearthbreaker\hearthbreaker\game_objects.py", line 909, in use
    minion.battlecry(minion)
TypeError: 'int' object is not callable

This was only happening when Jaraxxus made an infernal, then youthful brewmaster brewed it back, and then it was played again.  Anyways, fixed now.


Save my internship applications

As a user I can save my internship applications so that I can keep track of my applications.

No changes needed


Use hashtag

Would be great if the bot could use hashtags when retweeting if it notices this. Many of my retweets require this in addition to the retweet itself. aslo some retweets require "Like comment and retweet."

I don't think I fully understand. You mean something like "retweet to win #includeThisHashtag"? Like some contest owners require you to include the hashtag in the RT in order to be eligible?
I understand that some contests require more, but this is a Contest Bot based on retweets. It could be an improvement, though.

Thats what i meant, yeah.

For likes/comments I imagine it won't be very hard. The program would have to check for words like "comment" or "like". But what about hashtags? Most posts on Twitter use hashtags. But very few contest organizers actually require adding the hashtags in the RT. How do you know when to add hashtags and when not?
Furthermore, when you Retweet you can't change the text. So you would have to create a new tweet. I haven't seen anything like this and don't really consider it useful. If you could elaborate more or come up with an idea for it to work, that'd be great.
I could add an improvement for Liking and Commenting.


HBASE-25469 Add detailed RIT info in JSON format for consumption as metrics

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  7s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m  2s
master passed

+1 :green_heart:
compile
3m 33s
master passed

+1 :green_heart:
checkstyle
1m  7s
master passed

+1 :green_heart:
spotbugs
2m  5s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 44s
the patch passed

+1 :green_heart:
compile
3m  8s
the patch passed

+1 :green_heart:
javac
3m  8s
the patch passed

-0 :warning:
checkstyle
1m  3s
hbase-server: The patch generated 1 new + 89 unchanged - 0 fixed = 90 total (was 89)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
18m 52s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 13s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 16s
The patch does not generate ASF License warnings.

49m 31s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3535

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux d7591fe20994 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 332d9d13b1

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
96 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 44s
master passed

+1 :green_heart:
compile
1m 14s
master passed

+1 :green_heart:
shadedjars
8m 26s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 44s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 27s
the patch passed

+1 :green_heart:
compile
1m 15s
the patch passed

+1 :green_heart:
javac
1m 15s
the patch passed

+1 :green_heart:
shadedjars
8m 47s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 41s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
146m 29s
hbase-server in the patch passed.

179m 19s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3535

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 2ffbb32b75b5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 332d9d13b1

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/testReport/

Max. process+thread count
4121 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 43s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 25s
master passed

+1 :green_heart:
compile
1m  7s
master passed

+1 :green_heart:
shadedjars
8m 45s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 41s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 57s
the patch passed

+1 :green_heart:
compile
1m  6s
the patch passed

+1 :green_heart:
javac
1m  6s
the patch passed

+1 :green_heart:
shadedjars
8m 58s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
160m 31s
hbase-server in the patch passed.

194m 15s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3535

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 93a691ef7490 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 332d9d13b1

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/testReport/

Max. process+thread count
3703 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@bharathv I considered consolidating, but mainly wrote it this way so that 1) the url isn't so complex, and 2) the code is not dependent on this page/the complex url (we can also attach the servlet elsewhere if we want).

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
5m 56s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 21s
master passed

+1 :green_heart:
compile
3m 13s
master passed

+1 :green_heart:
checkstyle
1m 11s
master passed

+1 :green_heart:
spotbugs
2m 12s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  3s
the patch passed

+1 :green_heart:
compile
3m 17s
the patch passed

+1 :green_heart:
javac
3m 17s
the patch passed

-0 :warning:
checkstyle
1m  9s
hbase-server: The patch generated 1 new + 89 unchanged - 0 fixed = 90 total (was 89)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m  9s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 25s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

56m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3535

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 5baad65097aa 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / de84c1765a

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

The problem is that the two places can now diverge. If someone is not familiar with the other page, they just update one of them. As an operator it is now confusing whether to use /rit or /rit.jsp or both or how they differ apart from the output type is not obvious. Is it difficult to clean up rit.jsp or move rit.jsp logic into /rit servlet (and add a redirect in branch-2) ?

@bharathv there already are multiple places where RIT info can be found: the main /master-status page and the rits.jsp page. I don't think it's that confusing what the differences between the different RIT info sources are; the /master-status gives an overview of the RIT, the rits.jsp gives more details, and the /rit servlet displays the metrics we want as JSON. and, each one links to the next. there are indeed overlaps between these places, but each has its own purpose that is independent of the other ones. we may choose to display different info in the JSON than on the rits.jsp page (I asked for feedback on this JIRA comment about whether we should consider displaying only RITs over threshold in the JSON, or consider grouping by server name or state in the JSON -- if you or @apurtell have any thoughts there, please do share). that is just my personal opinion though, any thoughts @apurtell?

actually, I think it makes more sense to link /rit from the /master-status RIT section as a "view as JSON" link rather than the rits.jsp page because the rit.jsp page may display RITs only for a certain table, depending on the filter, while the JSON always returns all RITs.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  2s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 28s
master passed

+1 :green_heart:
compile
1m 11s
master passed

+1 :green_heart:
shadedjars
8m 18s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 43s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 13s
the patch passed

+1 :green_heart:
compile
1m 13s
the patch passed

+1 :green_heart:
javac
1m 13s
the patch passed

+1 :green_heart:
shadedjars
8m 16s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
142m 43s
hbase-server in the patch passed.

174m 59s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3535

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux fe34390c66f9 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / de84c1765a

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/testReport/

Max. process+thread count
3852 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 41s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 14s
master passed

+1 :green_heart:
compile
1m  3s
master passed

+1 :green_heart:
shadedjars
8m 12s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 40s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 41s
the patch passed

+1 :green_heart:
compile
1m  0s
the patch passed

+1 :green_heart:
javac
1m  0s
the patch passed

+1 :green_heart:
shadedjars
8m 10s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
144m 51s
hbase-server in the patch passed.

175m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3535

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e361ccdfe72f 4.15.0-151-generic #157-Ubuntu SMP Fri Jul 9 23:07:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / de84c1765a

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/testReport/

Max. process+thread count
4025 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3535/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 59s
Docker mode activated.

 I don't think it's that confusing what the differences between the different RIT info sources are; the /master-status gives an overview of the RIT, the rits.jsp gives more details in a human-friendly format, and the /rit servlet displays the metrics we want as JSON. there are indeed overlaps between these places, but each has its own purpose that is independent of the other ones. we may choose to display different info in the JSON than on the rits.jsp page (I asked for feedback in this JIRA comment about whether we should consider displaying only RITs over threshold in the JSON, or consider grouping by server name or state in the JSON

I'm not convinced by the explanation. I can see how /master-status has a summary in the footer and is a bit different from the other pages but it is usually a common pattern to do something like /rits?type=html /rits?type=json  /rits?type=txt etc and all are served by the backing page (especially when they are serving the same information presented in different ways). Not a big deal though, don't want to block the progress here.


Feature Request: Open tabs in new or existing groups

Request to "open selected links" in new or existing groups in addition to same or new window. Thanks!

This should work as of the latest commit, 77024e1.  This also switches to MV3 (for reasons), so I had to add some weird special cases to support Opera (which I mostly use), and the permissions changes may make the Chrome web store review harder. But we will see!

(This is currently under review as v. 1.5)

I downloaded the new version and neither option of open selected links in existing nor new window work, and there is no option for opening in a new or same tab group. I restarted Chrome/Edge and issue still persists. Am I doing something wrong?

The tab group UI is in the extension's popup; I have it pinned in my browser to make that easier to get at. That seems to work consistently.
Is the not working you see for the context menu options? I have started getting that intermittently now too.... let's make a new issue for that.
(I've just submitted updated screenshots to the Chrome Web Store.)

Ohhh, I found the tab group UI. I normally hide most extensions in the extensions menu to prevent clutter, that's why I didn't notice.
As far selecting and opening links, neither right-click option works. Something broke in the last update. Is there more information I can send to help?

I found it -- it worked for me since I had moved the installing the listener for the context menu items into the installed listener, and I had just installed. Moving that back to the top level seems to have fixed it.  I committed that as 1.5.1, but have to wait for my updated screenshots to be approved before I can upload a new version to the chrome web store. 😬
While I was thinking about it, I added a context menu option to open things in the current window in a fresh tab group, to better support use cases like yours. I've committed that as 1.6.0 and will upload instead of 1.5.1, once I can submit a new version.

(the diagnosis is in #3)


Add unittests for the remangler, particularly to cover error handling.

We currently don't have any tests for remangler error handling, because in asserts builds it asserts. This patch adds a flag that only exists in !NDEBUG builds to turn off assertions when inside the unit tests.

@swift-ci test

@swift-ci test


Update the deprecated Ansible collection "community.kubernetes" to the renamed collection "kubernetes.core"

The Ansible collection "community.kubernetes" has been deprecated and the new collection is named as "kubernetes.core". We need to update the project to use the latest maintained and supported Ansible collection.
See the deprecation notice at: https://docs.ansible.com/ansible/5/collections/community/kubernetes/docsite/deprecation.html

The code update has been checked in, and README updated too.


Widget display on lastUpdated

PR has below changes

Repo widget to display on lastUpdated
Code analysis widget -> Test tab -> removed testJobNames for test results display

Need some minor cleanup, remove obsolete references to "testJobNames". Thanks


feat: Transifex app remove the default locale restriction

Purpose
Remove the check for contentful space default locale match with transifex project source language, since they can be different now
Approach
Testing steps
Try to connect a transifex project with source language different from contentful space's default locale
Breaking Changes

Dependencies and/or References

Deployment

Hello from Transifex! Is there an ETA on when the Contentful team will review this PR?


Routing for polymorphic relationships

Are registering of routes for polymorphic relationships supported? In the docs for routing, this is what is said:

The related resource type needs to be known so that the query parameters for requests to relationship routes can be validated correctly. For example, for a to-many relationship, the filters that are allowed will be those allowed for the related resource type. So a request to posts/1/author will need to validate the filters for those allowed for users resources, not the posts resource.

This isn't fully tested at the moment. The way I've got things set up it in theory works but there's likely to be some bugs in it - particularly how to go about validating query params for polymorph relations. This is because query params on relation routes are validated by the inverse resource type (i.e. the resource that is in the relationship), but for polymorphs there are many inverse resource types.
So support is a bit patchy at the moment. I need to add tests for this scenario so that I can check it actually works and iron out any potential bugs (which I suspect there might be a few).

Hey there – I was wondering if there was any progress on this? It doesn't look like JSON:API really supports this, so did you mean you were working on something that would act as a bridge between Eloquent's polymorphic relationships and the JSON:API specification @lindyhopchris?

@osteel can you let me know what you're trying to do with polymorphic relations, then I can advise whether it's supported in this version or not?
I'm working on the new version at the moment, which will include full support for polymorphic resources.

@lindyhopchris I was just being curious really, as I'm currently working with polymorphic relationships and realised JSON:API doesn't really support them. For the time being, it looks like the fact that the HasOne and HasMany rules accept multiple resource types does the trick for me (although I still need it to test it), but yeah I'm curious to see how you're intending to support this further!
Feel free to ignore this though 😉

Ah great, thanks for confirming!

Now tracking polymorphic issues in the new package on this issue:
https://github.com/laravel-json-api/laravel/issues/8


Named locations in config files

Short Description
We can predefined a number of named locations in the config file (or in a separated file), then use their name for "location" config parameter.
Possible solution
A few line of code will do the trick
How it would help others
The location will make sense to the user and allow us to quickly switch between our favourite locations. Even though copy & paste from a text file may do the trick but it's still worth to implement for the sake of user friendliness.

Need to chamge alltitude min/max for every location too


cannot load PDB helper DLL

cv2pdb.exe -C MyApp.exe

and get this error:
MyApp.pdb: cannot load PDB helper DLL

More information is needed. What VS installation are you using?
I suspect that VS 2022 might be a problem, because it is running as a 64-bit process. You might have to start cv2pdb in an environment that uses the x86 executables of the VC tool chain.

One possible solution is to put cv2pdb into the same directory with VS directory that contains mspdbsrv.exe.


Access scope from text property

basically i'm trying to add an input field inside the text property and access the scope value :
    $scope.deleteTask = function (id, row) {
        SweetAlert.swal({
            title: "Are you sure?",
            text: "<input ng-model='deleteAll' type='checkbox' /><h6>Delete All Tasks Created By The Same Operation </h6> ",
            type: "warning",
            html : true,
            showCancelButton: true,
            confirmButtonColor: "#DD6B55", confirmButtonText: "Yes, delete it!",
            cancelButtonText: "No, cancel !",
            closeOnConfirm: false,
            closeOnCancel: false,
            showLoaderOnConfirm: true
        },
        function (isConfirm) {
            if (isConfirm) {
                console.log($scope.deleteAll);
            } else {
                SweetAlert.swal("Cancelled", "Your task is safe :)", "error");
            }
        });
    }

$scope.deleteAll returns undefined :/
Edit :
I figured out that the "divs" were created outside the wrapper which has ng-scope reference,i think it would be great if we can specify the id of the container that we want to use to create the divs inside

Any news on this? +1

since it's created outside the scope there is no ng model, what i've done is that i declared global vars  that i update on input change using javascript

Thanks for the workaround, @badreal .

Don't use globals use .bind($scope) and use it within the function as this. instead of $scope.
function (isConfirm) {
if (isConfirm) {
console.log(this.deleteAll);
} else {
SweetAlert.swal("Cancelled", "Your task is safe :)", "error");
}
}.bind($scope));


refactor: create Occurrence class

Types of changes
Internal changes:

[x] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)

Description

Extract occurrence-related data to from Reccurrence to Occurrence class

More PRs will follow (I already have the changes in my local repos, submitting slowly)

Motivation and Context

Separate concerns in Recurrence - the recurring part itself and the dates (occurrence)
Better code readability

How has this been tested?

Unit tests

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follows this project's contributing guide
[x] I agree to follow this project's Code of Conduct

I'm really happy to merge this - but am sending this PR now so you have a chance to think about the comments before we talk later, if you wish...
I appreciate you may have ideas for where to go with this next.
However, it hasn't quite had as many encapsulation benefits as what I had imagined (perhaps because of less familiarity with the code..)

As I mentioned in the PR, I already have the code where this all should/could evolve and I'm submitting PRs as chunks of commits that are easy to review =)
All of the points mentioned above have been addressed =)

As I mentioned in the PR, I already have the code where this all should/could evolve and I'm submitting PRs as chunks of commits that are easy to review =)

Sorry for missing that bit of the comment!


Why ignore unknown module error?

When getModuleRecord results in an error this error isn't logged, why?
https://github.com/capaj/systemjs-hot-reloader/blob/master/hot-reloader.js#L212-L215
Would be useful to add a bit more comments to why this isn't logged.
I just spend half an hour trying to figure out why a module wasn't hot reloading and this error would have been very useful.

Better error handling is coming in systemjs-hmr


Implement Auto Top Up UI Setting

Depends on https://github.com/akash-network/console/issues/395
Objective
Implement a new UI setting in the existing settings page that allows users to enable or disable automatic top-ups for Akash Network deployments under custodial wallets via AuthZ deployment grants.
Scope
The task infers changes to be done to deploy-web Settings page. New setting should utilise existing transaction signing components.
General Flow

User navigates to Setting page
Either on the initial tab or under Authorizations a new set of settings is added: Automatic Top Up for uAKT and USDC deployments.
Once enabled user is prompted with setting params: funds limit and expiration data.
Once submitted user is required to sign a relevant transaction: /cosmos.authz.v1beta1.MsgGrant for /akash.deployment.v1beta3.DepositDeploymentAuthorization.
The setting state should now reflect it's enabled.
User can disable the setting.
Once disabled user is required to sign relevant revoke transaction.
The setting state should now reflect it's desabled.

Acceptance Criteria

New setting is implemented.
User should be able to enable or disable auto top ups.
Setting state should be reflected in UI.
Setting should utilise existing transaction signing components.
Authorization state managed by the setting should be also reflected in the Authorizations list in Settings.
The feature should be hidden via env var toggle.
The feature should be covered by e2e tests.

Design Chart For Reference

Link

Hello @ygrishajev
I am interested in this issue.
I want to handle this issue.
Thank you

tagging @aktdenis for settings page UI to enable/ disable this feature


Making Record.Fmt public

Making Fmt public as mentioned in #82. Thanks again!

@op - fixed merge conflicts


Default account?

Hi,
I'm happy to try your work, I'm on the login page, what is the default account or how to create it?
Thanks ;)

Ok, sorry: docker exec -it artalk artalk admin
Thanks Google Translate ;)
Have a nice day!


fplll: 5.4.3 -> 5.4.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/fplll/fplll/releases.
meta.description for fplll is: Lattice algorithms using floating-point arithmetic
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.4.4 with grep in /nix/store/fswak147mm019n5wwkbf6sx7mmv1vnxp-fplll-5.4.4
found 5.4.4 in filename of file in /nix/store/fswak147mm019n5wwkbf6sx7mmv1vnxp-fplll-5.4.4

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

9 total rebuild path(s)

9 package rebuild(s)

First fifty rebuilds by attrpath
flopoco
fplll
python310Packages.fpylll
python310Packages.paranoid-crypto
python39Packages.fpylll
python39Packages.paranoid-crypto
sage
sageWithDoc
sollya

Instructions to test this update (click to expand)

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

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

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

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

2 packages failed to build:

sage
sageWithDoc

7 packages built:

flopoco
fplll
python310Packages.fpylll
python310Packages.paranoid-crypto
python39Packages.fpylll
python39Packages.paranoid-crypto
sollya

Maintainer pings
cc @timokau @omasanori @7c6f434c @collares for testing.

I've run Sage tests locally and they pass. Let's see if ofBorg agrees. @ofBorg build sage


Showcase fresh demo store website

Hello everyone! , I'm new to programming, this is my first deno/fresh website and first time contributing.  hope I didn't break anything. have a nice day.

Is the site fully working? Clicking on any of the buttons does nothing. I cannot put items into the cart 👀

Good Evening sir, I'm still working on website state management. Sadly yes the images is screenshot of other sites. I'll change it now.


Deprecate CTypes Backend

Currently, Charm4Py uses 3 different backends to interface with the Charm++ runtime library: Cython, CFFI, and Ctypes. In the manual, Cython and CFFI are recommended for different Python builds, but Ctypes is not recommended. It also doesn't look like there is a way to build Charm4Py with the Ctypes backend in the installation script.
Given this, I think it would be appropriate to deprecate the Ctypes backend to reduce the amount of work required to make changes to the Charm++/Charm4Py interface.

Mentioning @rbuch and @matthiasdiener as I think you're the two within PPL that are involved in Charm4Py development.

It would be good to know what exactly is bad about ctypes before deprecating it.

I think this probably makes sense to do. Based on a cursory investigation, it seems like CFFI is basically a better replacement for Ctypes, the only caveat being that Ctypes is in the standard library, so it's perhaps a bit more portable, but I think that's moot given the state of Python package management.
It might also be worth looking into cppyy or pybind11, if they improve performance or ease of use.


more information link error when trying to change the logo height

not allowing me to save it unless i pick a more information link there , not sure why

Use issues to report bugs, please post your questions on the forums or search the forums for existing solutions.


Create contact list of board members

Board members would like contact details for all members of the board and committees.

ref #11  ref #32

This could be exported from SFDC.  What is the format this is done in?

This is completed


Specified attribute encoding to "UTF-8" when creating instance of xmlB...

...uilder

:+1:

@baalexander @patricklodder

@mathieug Sorry, this is taking some time because I cannot merge this as-is. The XMLRPC spec leaves encoding open to implementation so we cannot start hard-coding UTF-8 as that might break implementations other than yours. We'll need to make this configurable and optional.

@patricklodder Any news?


Icon for napari in web-browser

Can we make it so napari logo appears in my web-browser as a little icon instead of the default gray globe when you're at napari.org - like how the github logo appears when you're on github.com

It's just a detail, but i think it's a nice touch :-)
cc @codemonkey800 @melissawm

Agree with @codemonkey800.
It looks quite simple https://stackoverflow.com/a/54665517/7475772

Great - maybe we move this issue over to that repo then. Thanks!!


Implemented Imports Manager

Adding an Imports Manager to make it easier to find and re-import animations imported via Inspector Dock.
This screen can be accessed via Project -> Tools -> Aseprite Wizard -> Imports Manager....

It shows all nodes with animations imported via Inspector Dock.
It allows re-importing any file or files.
Because of limitations, the scene of the animation being imported will be open in the editor.
Also, when importing multiple nodes at same time, it requires manually triggering save.

It shows basic information about the file and configuration associated to the node:

Allows importing all resources from a scene:

And all scenes in a folder:

It also allows selecting multiple files at same time:

When importing more than one resource a confirmation message will show up notifying you of how many resources will be re-imported:

Bonus features:
Filtering:

And docking to the bottom dock:

To do:

[x] load test Linux
[x] Load test windows
[x] Load test MacOS

Feature request: #134
This PR adds support to Godot 4. I'm still assessing if it's work to port to Godot 3 as well.

After importing the root folder, I found some errors in the Output tab of the editor:
res://addons/AsepriteWizard/interface/imports_manager/import_helper.gd:77 - Invalid get index 'keep_anim_length' (on base: 'Dictionary').
Also a lot of these:
Node not found: "./Sprite2D" (relative to "/root/@EditorNode@17171/@Panel@13/@VBoxContainer@14/@HSplitContainer@17/@HSplitContainer@25/@HSplitContainer@33/@VBoxContainer@34/@VSplitContainer@36/@VSplitContainer@62/@VBoxContainer@63/@PanelContainer@110/MainScreen/@CanvasItemEditor@9462/@VSplitContainer@9281/@HSplitContainer@9283/@HSplitContainer@9285/@Control@9286/@SubViewportContainer@9287/@SubViewport@9288/WaterSplash").
In this case, the scene root is a AnimatedSprite2D itself.

The Output File Name setting is used as a prefix actually, when entering MySprite.png it outputs to MySprite.pngMountains.png in my case, where the Aseprite file is called Mountains.aseprite.

I'm occasionally seeing weird things happening, like this for instance:

Godot Editor output says aseprite generated bad data file
The output files are written the wrong directory, as shown in the following diff. UI\Dialogs\Player is not supposed to be part of the path.

Thanks a lot for testing this. I haven´t had time to look into these errors yet and it might be slow in this couple of weeks, but I'll check it out.
I've seen the aseprite generated bad data file error when trying to import from a new scene that has never been saved before, so the output path is not available. I guess the bug might be around that, maybe the path being passed to the import is outdated or just straight up incorrect.
Let me know if you identify any pattern in these issues.


Shouldn't filesize be a cached property?

https://github.com/nficano/pytube/blob/c052b5cb7d6a0280e46dbd2c495a1995b78e27c7/pytube/streams.py#L153-L162
Apparently it conducts requests on every property call. Note that in python even hasattr calls property code.

Yes, I also thought about it. For example, if a user wants to track download progress, he has to call filesize() method and each time it will make a request, which is inefficient. @nficano Please look into this.

Based on the pattern seen in the class's init, I think it would be better to just set _filesize = None and when filesize is called check if it is defined, if yes then use that values, if not then get and set that value.


Fix date and datetime interval, ranaroussi#125

Pandas' index.date drops timezone information, so conversion into
exchange timezone has to happen in intra-day intervals only.

@ranaroussi It seems the CodeFactor fails unrelated to my fix. Might want to clean that up in a separate commit, or 'lax the settings.


API: Index.get_nearest method

xref #3004
xref #841
xref #7873
xref #7223
xref #8815
Building on @immerrr's excellent refactor in #8753, I would like to propose adding a get_nearest method to pandas.Index that does nearest neighbor lookups. The idea is that nearest neighbor lookups are usually the desirable/sane thing to do when using inexact indexes. Eventually, we might want to add an alternative "wrapper index" (like IntervalIndex), e.g., NearestNeighborIndex which switches the default behavior; this would be an intermediate step in that direction.
The implementation would be a simple wrapper that calls Index.get_slice_bound twice, once to the left and once to the right. Ideally, this would this would even work for array-like arguments, though perhaps there should be separate get_nearest_loc and get_nearest_indexer methods.

Hey Stephan,
Nice to see this functionality being built-in.  I have been used hacked together version of this for scikit-spectra for a while, and really think anyone who uses float-indexed data will find this extremely useful.
Just to add my two cents, I think that the "nearest nearest" mentatility makes the most sense.  The keyword side is superfluous because if the user is aware enough to use the keyword, they are also aware enough to just change the value.  For example, if I was trying to get data close to 130.5 but I needed to get a value less than 130.5, I'd just index:
get_loc_nearest(130.49)
As opposed to
get_loc_neareset(130.51, side='left')
Or am I misunderstanding?
I also think the name get_nearest() would work, but also like get_loc_neareset()
One issue we ran into was dealing with is what to do when the user oversteps the bounds of the data, do you raise an error or just return the nearest value?  For our purposes, it made more sense to throw and error, but the data was strictly monotonically increasing and had clear upper and lower limits.  I guess the more general case would be that index floats would have no clear limits and would not necessarily be sorted/monotonic.
What would happen in the case of duplicate values in the index?

The keyword side is superfluous because if the user is aware enough to use the keyword, they are also aware enough to just change the value. For example, if I was trying to get data close to 130.5 but I needed to get a value less than 130.5, I'd just index: get_loc_nearest(130.49) as opposed to get_loc_neareset(130.51, side='left')

Not sure I follow. Suppose the index in your example is given by pd.Index([129, 131]). get_nearest(130.49) would still return the value corresponding to 131. Basically, this feature is useful for irregularly spaced data. For example, you could use it to return the last record of each hour.

I also think the name get_nearest() would work, but also like get_loc_neareset()

Hmm. We could certainly do get_loc_nearest and get_indexer_nearest instead of a single function. That might make the functionality more obvious.

One issue we ran into was dealing with is what to do when the user oversteps the bounds of the data, do you raise an error or just return the nearest value? For our purposes, it made more sense to throw and error, but the data was strictly monotonically increasing and had clear upper and lower limits.

For this use case, I think you'll want an IntervalIndex. Currently we just return the nearest value, though we could hypothetically add something like a max_distance argument.

What would happen in the case of duplicate values in the index?

Not supported in my current PR (the result is ambiguous for looking up an indexer). The index needs to have unique and sorted values (either ascending or descending).

Cool.  Sorry, I haven't had a chance to use the IntervalIndex because I'm still bogged down in 0.14.
I see what you mean about the side argument now.  I was stuck in my own use cases I guess, where we generally know our index, but the float rounding is the pain.  IE our data is  Float64Index([120.540, 121.483, ...]) and we just want something like get_loc_nearest(121.0).  So I see why the side argument is necessary.
Are you planning to have a nearest indexer that would work like .loc or .iloc eg:
data.locnearest[130:140, 30:35.33]
IE 2D nearest indexing?

@hugadams IntervalIndex hasn't been merged yet -- still sitting in a PR :). I think something like .loc_nearest as an indexer would be a nice addition, though I don't have any concrete plans yet.

nice question to show perf of nearest
http://stackoverflow.com/questions/28612773/how-to-speed-up-nearest-search-in-pandas

@jreback  yes, but do that sanely we'll need the MultiIndex version (#9365).


Smoother interpolation of basin profile and Q(h) relation

DataInterpolations.jl, which we currently use for linear interpolation of Q(h) relations, supports all sorts of other interpolation types which are explained very minimally in the docs. We can modify the code so that it becomes very easy to test different types of interpolation, currently primarily for the stability/convergence of large models.
For Q(h) relations the interpolation type can probably easily be changed, although care has to be taken with extrapolation behavior.
The interpolation of the basin profile can be done much cleaner than it is now, getting rid of get_area_and_level. I propose that we use an interpolation object for the level as a function of the storage $h(S)$, and then the area for a given level can be obtained as $\frac{1}{h'(S)}$, either by the derivative function of DataInterpolations.jl or via ForwardDiff.jl. Also here care has to be taken with extrapolation.

Also update numerics.qmd (which has not been merged at the time of writing).

This may help: https://github.com/SouthEndMusic/SmoothInterpolation.jl

It mentions that this will be updated once this issue is resolved, is that still correct?
If so, can we add more explanation in https://deltares.github.io/Ribasim/concept/numerics.html#basin-profiles ?


Update CRUD methods

POST, GET and DELETE are enabled in example project's API Explorer.

Thanks.


Add py.typed to ship typing hint

Added py.typed file in /spylib

interesting.

@ponty33 Have you run locally (see README for development environment) to test that mypy is happy now?

@hillairet I will test that


[CARBONDATA-1053] supported char type for hive

tested manually using hive cli

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2047/

retest this please

Build Failed  with Spark 1.6.2, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2070/


Unexpected Model Warning for gpt-4o-mini Despite No Specification: Possible Default Configuration Issue

Aider version: 0.58.1
Python version: 3.12.3
Platform: Windows-10-10.0.19045-SP0
Python implementation: CPython
Virtual environment: No
OS: Windows 10 (64bit)
Git version: git version 2.45.2.windows.1
Aider v0.58.1
Main model: openrouter/openai/gpt-4o with architect edit format
Editor model: openrouter/anthropic/claude-3.5-sonnet with editor-diff edit format
Weak model: openrouter/openai/gpt-4o-mini
Git repo: .git with 7,655 files
Warning: For large repos, consider using --subtree-only and .aiderignore
See: https://aider.chat/docs/faq.html#can-i-use-aider-in-a-large-mono-repo
Repo-map: using 1024 tokens, auto refresh
Command and Output
PS C:\Qt\5.15.2\Src\qtbase\src> aider --model openrouter/openai/gpt-4o --editor-model openrouter/anthropic/claude-3.5-sonnet --map-tokens 1024 --architect
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
Warning for openrouter/openai/gpt-4o-mini: Unknown context window size and costs, using sane defaults.
Did you mean one of these?
- openrouter/openai/gpt-4
- openrouter/openai/gpt-4o
- openrouter/openai/o1-mini
For more info, see: https://aider.chat/docs/llms/warnings.html
You can skip this check with --no-show-model-warnings

Proceed anyway? (Y)es/(N)o [Yes]: n

What exactly is going on here? I never specified gpt-4o-mini. My guess is on startup aider's source code is attempting to set some defaults and one of the defaults is conditionally set based on if i'm using openrouter-related models, and it just happens to have a typo?
More importantly I can't seem to find a ocmmand to change the weaker model.
If I am missing something obvious I do apologize.

Workaround: --weak-model openrouter/anthropic/claude-3-haiku seems to work.

Thank you for filing this issue.
You can ignore this warning or use a different model.


Plugin fails to build since the latest update

Probably because some of the resource ObjectID's changed. See e.g. https://github.com/rdutta/runelite-gauntlet/compare/bed17cdb2c8097e47a010b3b02dad4d84eec9b94...rdutta:9af84577f865330dd62f76081470d62d2765fa09

Fixed in runelite/plugin-hub#4340


Training the gym-hybrid using hppo

Hi
I am new to DI-engine and trying to train the gym hybrid "thomashirtz" agent using HPPO. could you please explain me what are config,entry, envs folders are mentioning under DI-engine/dizoo/gym_hybrid/??

envs: the wrapper which transform original gym env into DI-engine's format
config: the configuration file which contains the setting of Algorithm + Env (nested dict) and the default main function to run this config
entry: another main functions for running corresponding config

we suggest that you should implement your own main function for your demand, and you can imitate  the following two files:

dizoo/gym_hybrid/entry/gym_hybrid_ddpg_main.py(for env)
ding/entry/serial_entry_onpolicy.py(for HPPO on policy algorithm)

Could you provide a hppo main function file ,I am new to DI-engine and H-PPO, thank you very much.

envs: the wrapper which transform original gym env into DI-engine's format
config: the configuration file which contains the setting of Algorithm + Env (nested dict) and the default main function to run this config
entry: another main functions for running corresponding config

we suggest that you should implement your own main function for your demand, and you can imitate the following two files:

dizoo/gym_hybrid/entry/gym_hybrid_ddpg_main.py(for env)
ding/entry/serial_entry_onpolicy.py(for HPPO on policy algorithm)


Usage of in-memory database can be problematic

Without a persistent database, we will reuse addresses because we don't remember the index we last used to derive an address.
Additionally, if the application runs for a long time, we might go past the look-ahead value (currently 100 by default in BDK) and not discover certain coins (because we start looking at 0 again).

We don't have an inmemory db anymore.


"route" flagged as unknown block inside "aws_route_table" resource

latest plugin 0.4.1
resource "aws_route_table" "something"
{
  vpc_id = "vpc-something"
  route {
    cidr_block = "<IP_ADDRESS>/0"
    instance_id = "id-something"
  }
}

the route line gets yellow flagged by inspector
unknown block type route

Similarly, I am seeing the tags block flagged within an as_instance resource

Will release new plugin a bit later (on weekend, probably), as workaround you could disable inspection.

@SpencerBrown Published new plugin version (0.4.2). Try it please.

Just got prompted to install new plugin. Works like a champ. All my .tf files are now proudly showing green checkmarks. And the previously missing blocks are now showing as options when I hit ctrl+space.
Thanks!!

@SpencerBrown  Glad to hear that,
If you have ideas what to improve in plugin feel free to share them here, by email, or create new issues.


Atom Families support - TypeError: fn is not a function (it is Object)

I'm getting an error when unmounting the ScopeProvider when just using atoms, and no atom families.
I think that the spreading of this set, when empty, is adding an object to the clean up functions which cannot be called:
https://github.com/jotaijs/jotai-scope/commit/a1d7c909c46322b90cfcda73b376022420022621#diff-d6a2b7535e5c937402c991bbcff9db5a2998ec232cf52500d35a2086289cc8d4R79-R82

Could you share a codesandbox example of this issue?

Resolved with https://github.com/jotaijs/jotai-scope/pull/55
Please update jotai-scope to 0.7.2.

Resolved with https://github.com/jotaijs/jotai-scope/pull/55
Please update jotai-scope to 0.7.2.

Amazing, thank you @dmaskasky - just updated and it's working now!


Read only versioning endpoints

PR for the following rest contract: https://github.com/DSpace/Rest7Contract/pull/97

@tdonohue The "VersionRestController" & "EPersionVersionLinkRepository" were deleted, we were exposing the entire EPerson record which is not something we want (in DSpace 6 we currently only expose the name).
We are now exposing only the name of the submitter for admins (or to the public if the versioning is configured in that manner).
These changes also required some changes to the rest contract, which can be found here: https://github.com/DSpace/Rest7Contract/pull/112


Login UI fixes

Fixes https://github.com/nearprotocol/near-wallet/issues/493 and some other improvements

Something went wrong when I tried to login creating new account.
https://gitpod.io#snapshot/e6cb166a-1b7c-4a74-90b8-ddefe5a28753

Looks like doesn't work correctly when new account is created in process of login (see attached screenshot).

Looks like it's the extra / in the config causing it

URL becomes (note //login):
https://near-wallet-pr-504.onrender.com//login/?title=Who+was+the+last+person+to+say+%22Hi%21%22%3F&contract_id=dev-1586223108122&success_url=https%3A%2F%2F1234-fa2f9cf3-4c3e-443b-afb8-7d998e2caa48.ws-us02.gitpod.io%2F&failure_url=https%3A%2F%2F1234-fa2f9cf3-4c3e-443b-afb8-7d998e2caa48.ws-us02.gitpod.io%2F&app_url=https%3A%2F%2F1234-fa2f9cf3-4c3e-443b-afb8-7d998e2caa48.ws-us02.gitpod.io&public_key=ed25519%3ABzeosjEV4E38k1r4R4JuUJFHrCLh41VKuh739P6TbSv
I tested removing it and it worked fine. Was wondering how my PR could have caused that issue :)


the echo handle should match the function declaration.

There will be a panic panic: echo: unknown handler after running go test.
The reason is the declaration of HandlerFunc is in echo.go, line 40:
HandlerFunc    func(*Context) *HTTPError
It will not match the declaration func(*Context) error in the test.

Thank you!


Publish to Arch User Repository

Hi, I love Feroxbuster and wanted to help more users access it so as an Arch Linux user, I published it to the AUR (Arch User Repository) under the package name feroxbuster.
This pull requests just appends to the installation instructions of the README.md telling users how they can install it from Arch Linux.
Thank you for making an awesome project! If you have an AUR account and would like me to transfer ownership of the package to you, I'd be happy to do that as well.

Thank you for the submission, this is great!
I have a few questions, as I'm unfamiliar with Arch and AUR (I don't use Arch btw :wink:).
It looks like git and cargo are dependencies, which to me implies that when a user runs yay -S feroxbuster, the package manager clones the repo and builds from source using cargo.  Is that accurate?
Is there any config file / metadata specification file tied to the published package, if so, is it publicly visible anywhere?
I guess my questions boil down to, is this kept in sync with the latest release and does anything except git clone / cargo build happen underneath the hood?

Follow up question, is there a way to track installs from AUR?  If so, can you check for a badge on shields.io for it?

Good questions!

It looks like git and cargo are dependencies, which to me implies that when a user runs yay -S feroxbuster, the package manager clones the repo and builds from source using cargo. Is that accurate?

Yes this is accurate. After cloning and building using cargo build --release, it installs the binary in /usr/bin/ so that Arch users can run feroxbuster from the command line.

Is there any config file / metadata specification file tied to the published package

Yes! The Arch User Repository uses a specification called PKGBUILD and you can see the specification for it here. PKGBUILDs are a set of instructions telling user's computers how to find, build and install a project.

Is it publicly visible anywhere?

Yep! You can see feroxbuster's PKGBUILD file here. This one is quite simple and hopefully each part should make sense. The build() function is run first which executes cargo build --release under the hood and then following the AUR specification installs it to the user computer in the package() function.

I guess my questions boil down to, is this kept in sync with the latest release

Yes this will install directly from the most recent commit.

does anything except git clone / cargo build happen underneath the hood?

Nope, that's really it.

Follow up question, is there a way to track installs from AUR? If so, can you check for a badge on shields.io for it and add it to the readme ?

Hmm I'm not sure if you can track the number of installs but you can view the "Popularity" and "Votes" field on the package page. Shields.io does have a badge for the "votes":

Should I add that to the README?

Is this kept in sync with the latest release?
To add, for it to reinstall the code and tell users an update is available, I'd have to update the package version.

Is this kept in sync with the latest release?

To add, for it to reinstall the code and tell users an update is available, I'd have to update the package version so it matches the most recent release version.

haha, was just about to ask how pkgver=1.0.5 comes into play.

Actually, I'll include a pkgver() function like the one below which should register an update whenever a commit is made meaning the AUR package maintainer can just set it and forget it (I won't have to change the pacakge version each time a commit/release is made).
pkgver() {
	cd "${_pkgname}"
	( set -o pipefail
		git describe --long 2>/dev/null | sed 's/\([^-]*-g\)/r\1/;s/-/./g' ||
		printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
	)
}

git describe --tags | awk -F- '{print $1}' | cut -c2- appears to work

Thanks, that looks good! Though that will be in sync with tags rather than commits, is this what you are going for? Would you like me to make a second Arch package for feroxbuster-git (-git is standard for AUR packages that sync to Git commits) which would be in sync with commits rather than tags? I'd be happy to do this and this would give users both a stable and bleeding edge option.

brb, but tldr is that master is always in sync with tags as far as the binary itself is concerned

well, I'm back much quicker than i thought... lol.
so, i work in branches and only merge to master when complete.  after merge i publish a new release.  The release tags will always be in the vX.X.X format. The only time that master is out of sync with the latest tagged commit is when the readme / CI config / etc is updated.  Anything that affects the binary will be captured in a tagged commit.

As far as the readme badge, don't sweat it.  If one were available for installs, I'd like it, but it's ok that there's not.

Gotcha, yeah that all makes sense. Alright I've updated the PKGBUILD so feroxbuster on the AUR is now in sync with tagged commits.

Is it ok to have both a static pkgver and a pkgver function defined?

Yeah, the pkver() function will always override the pkgver variable.

Also, the function actually self-edits (updates) the variable.

Good stuff!  Thanks for this PR, and thanks for checking out the tool.  Merging! :partying_face:

Awesome, thanks! 🙂 Keep up the great work with feroxbuster!


Parsing of dates in "ar" locale fails when typed

Describe the bug
There is a problem with parsing type dates in "ar" locale (see: https://github.com/flutter/flutter/issues/138635)
To Reproduce
see examples in https://github.com/flutter/flutter/issues/138635
Alternatively:
import 'package:intl/date_symbol_data_local.dart';
import 'package:intl/intl.dart';

void main() {
  initializeDateFormatting("ar");
  // works
  DateFormat('yMd', "ar").parseStrict(String.fromCharCodes(
      [1633, 1633, 8207, 47, 1633, 1633, 8207, 47, 1634, 1632, 1632, 1633]));
  // fails (codeUnits of typed date)
  try {
    DateFormat('yMd', "ar").parseStrict(String.fromCharCodes(
        [1633, 1633, 47, 1633, 1633, 47, 1634, 1632, 1632, 1633]));
  } catch (e) {
    print(e);
  }
}

Did some digging and my current theory is, that date_time_patterns.dart contains some unneeded "RLM" characters. Are they supposed to be generated there?
If I change keyboard to arabic and type the date there will be no such characters and as such parsing fails.

 Date parsing is unfortunately very fragile.

RLMs are also in ar.xml in CLDR
https://unicode-org.atlassian.net/jira/software/c/projects/CLDR/issues/CLDR-14076 suggests to me that is supposed to be like that. (I am personally not sure since the RLM is for strings with mixed text directions as far as I can see, which should not normally be the case if someone is simply using arabic locale, right?)
I did not manage to get RLM into the string by typing. (Also had a look at some arabic dates I found online and they also did not include RLMs)
Question remains where this should be mitigated. Does every caller of parseStrict need to handle adding the RLMs when in arabic locale? Or should DateFormat handle this?

I don't know what the best strategy is - parsing is inherently fragile exactly because of cases like this. Maybe the logic of parsing could be special-cased for arabic parsing. I am open for reviewing a PR!


Fix for mitsuhiko/babel#133.

Fixes the tests that failed with the newer versions of pytz. Fixes #133

No news is bad news, and this bug prevents the fix for many other bugs. What can be done, @benselme?

@jrfern I don't understand your comment fully.
@benselme can you rebase the PR onto current master, correct the reference to point to bug https://github.com/mitsuhiko/babel/issues/106 (as it was a duplicate, please use full URL, avoid relying on users viewing the commit message via GitHub), add an explanation so the commit message is valid on its own (it should be, the issue link should provide more information about the issue while the commit message itself should provide enough information to explain the change (not the fix) to the code and justify it.) Take a look at http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits/ for a summary of common commit message guidelines.

@benselme still around? I can understand frustration of your PR being ignored for such a long time. We took over maintenance shortly ago because the old maintainer (obviously) didn't have enough time to maintain this repo properly. I can assure you, that we will work together with you to get this merged.
@jrfern you seem to be interested in this issue, if @benselme is not around anymore, would you like to ramp this one up to our current standards and submit a replacement PR? We currently don't have the manpower to do more than triaging issues and reviewing and merging PRs because they stacked up pretty high.

I just came back from a long trip. I'll try to fix that PR this week.

@benselme cool, many thanks!

Hasn't this been fixed in https://github.com/mitsuhiko/babel/commit/9f099d39553188d9bc56f7721ea40ece9f0959cd already ?

@benselme fixed indeed.  Closing.


'observe' mapping options doesn't work with sub-object

Hi devs,
It looks like the 'observe' mapping doesn't work quite well for objects which contain sub-object.
Here is a  sample:
// Working scenario
var data = { "toObserve": "foo", "toIgnore": "bar" };
var mapping = { "observe": ["toObserve"] };
// it is all good on next line
var vm = ko.mapping.fromJS(data, mapping);

// Failing scenario
var data = { "toObserve": "foo", "toIgnore": { /*...*/ } };
var mapping = { "observe": ["toObserve"] };
// next line fails fails on attempt to treat "toIgnore" object as observable
var vm = ko.mapping.fromJS(data, mapping);

I think that in visitPropertiesOrArrayEntries callback (latest revision, line 467) the "value" must be unwrapped from observable as it is actually done few lines below for writable arrays. By doing that I get expected view model even for data objects with deep sub-object hierarchy. Could you please have a look on that? Thank you.
Cheers,
Tavriets

Hi, there
Any chance of pushing this fix in a new release any time soon? I run into this issue today and I was lucky to stumble to a stackoverflow thread that mentioned this bug.
It will help people save time and frustration
Thanks

@bellostom the author says: "Due to lack of time this project is currently not actively maintained."
Check the many forks that this project has, maybe someone has already fixit!