Multi cluster support

Hi, interesting project you got there. Are you guys planning for multi cluster support? I noticed you guys are using singletons for clusters. This makes it hard to do dependency injections as well

Please elaborate, what is the usecase and what do you want to accomplish
Some context that might be worth knowing:
There are some deliberate design decisions at play when it comes to singletons in Proto.Actor.
One of the main painpoints when we developed Akka.NET was that in order to have ActorSystems and related primitives, the serialization layer must be able to re-attach deserialized ActorRefs (PID's in our case).
In essence this means that if there is an ActorRef/PID somewhere inside a message with a deep object structure, any such instance must be found and re-attached to the actorsystem.
This can be done, but requires custom serializers and quite a bit of black magic.
Early on in the design phase of the project, we decided to not go down that route, instead keeping

One possibility is the idea of "microservices", where each service has it's own clusters. But I guess this is solved by the cluster name if i am not wrong. While working with this though i noticed another major issue with using static classes. I can't seem to be able to mock Cluster nor PID. I suppose i can wrap another layer above this but is there any "TestKit"?

Multiple clusters could potentially be used so that a single application can act as a control point between a cluster of GPU enabled devices (that will have some form of specialized processing) and a cluster of standard non-GPU computers (for general processing). The singleton clusters do help with ease of use, but do not allow for separation of hardware. If there is a better way for me to go about this, please let me know. Otherwise, this framework is excellent.

This discussion has been dead for a while and it's not clear what is being proposed. From my POV the purpose of Proto.Cluster is to enable a set of processes to act as a single unit scaled across multiple identical hosts (VMs, pods, containers) where all processes are identical and replaceable.  Hope this clarifies something.
Closing. Please re-open the discussion if you disagree.

I think that Cluster should be scoped inside the ActorSystem and by doing so we'll resolve this issue.

@alexeyzimarev That could probably be done - although I do find the idea of running multiple clusters across a single process as well as running a single cluster across multiple processes... strange :)

I agree that it's a bit weird conceptually, although should be perfectly doable technically :)

Running multiple remotes and cluster instances on a single process is possible with the code in #498.
Actually in a project I changed the remote unit tests to run multiple remotes from the same process.
It can be easily done for the cluster too.

I didn't even think of this, being able to run multiple nodes in the same process would make life so much easier for cluster and remote test and debugging.


Android SDK 25 and Lower Fix for java.nio.file

Hi,
I have made a few changes just to get rid of java.nio.file because it is not supported lower than android SDK 26 (o). It was very quick and I haven't really tested it but it seems to be working. If anyone wants to see it just comment.
Cheers.

This is primarily a Java api, so we will be using relatively recent java features. Personally I would use a kotlin api for android development. Though do feel free to fork and apply android specific fixes.


Define SecOps processes in relation to managing cloud firewalls

Describe the issue
CPF is strong in IaC and DevOps around cloud based infra, whereas SecOps bring to the table strength in policy updates and cross firewall management.  We want to combine the two without affecting production stability.
Additional context
Add any other context, attachments or screenshots
Definition of done

Meet with the CPF team for 30min (Jon, Bruce, Max, Warren, Julian, Gary)
Create a draw.io diagram outlining the proposed approach
PO to bring the approach forward in the tri-weekly secops meeting, gather feedback.

Jon is away for first 2 week of Aug and Julian will be on leave in 3,4th week of Aug as well.

Let's have a mural meeting tomorrow Aug 11th.


[Feature-Request] {Add unit tests}

Start with the why:
Well, it's a general good practice.
Move to the what:

depthai_bridge - Adding tests to verify Converters properly output messages based on some mock input DAI data.

depthai_ros_driver - Adding several test cases to verify bare C++ code for PipelineGenerator to verify that parameters/objects are properly created.

Move to the how:

depthai_bridge: For each converter add a test case with some example data. Verify that different options work.
depthai_ros_driver: Test out pipeline generation based on different devices (mock device sensor list).

I guess best way to start is to start with one test case for each of those packages and add more later.
cc @saching13

Yes. We can start with unit for each of the converters.
Follow s by integration tests where we run it on our devices with rpi, Jetson platforms.


Hide dialog category based on permission

Hi all
Couldnt find anything about this. Is there an option to hide a form category based on permission?
I have a situation for example where i have a field inside the form dialog. but if a user doesnt have permission to view this field the category above it is still visible, with no fields under it. looks kinda weird and also shows the user he is not seeing something.
Is there any way to hide categories based on permission or any other conditional?
Thanks

    if (!Q.Authorization.hasPermission("yourPermissionKey)) {
        this.form.SomeFieldInCategory.element.closest('.category').toggle(false);
    }

perfecto!


fix(deployment): Using latest image in helm chart

Summary
Using the latest image for helm deployment on the main branch.
Fix: #3479, maybe lit bit the same as ongoing #3480
Does this close any open issues?
Closes #3479
Screenshots
N/A
Other Information
Currently, Mysql-8 uses oracle based image, it seems not to work well with the KinD pipeline, so keep Debian-based MySQL image in the CI pipeline.

Some discussions with @warren830:

Need to use the latest image for MySQL-8, and need also support arm64/v8 arch, as guys may use arm base mac.
Currently, the official image with mysql:8 does not work well with the KinD pipeline, so let's keep using the Debian-based MySQL image.

Great work, thank you @matrixji


23-cell pattern support

App should support 23-cell pattern

https://xd.adobe.com/view/e7fe32c6-047a-42b3-b3fe-fe92b4d9e11b-b2ad/screen/8f5e1ec4-4886-4b55-9e38-624af515d624/

Completed


More filters on free packages page.

Filter out ignored games, games with content that the user has asked not to see, games that have an age
check, and games that don't support the users language or OS, including no VR support, VR only, and no Steam Deck support.

I really don't want to have a bunch of furry and henti games in my steam library.


[Bug]: @ command selection effect problem under light theme

What happened?
In the light theme, you can't see the selected shadow effect, so you can't know which option is selected

Steps to Reproduce
Input @ under light theme, you can't find which item be selected
What browsers are you seeing the problem on?
Microsoft Edge
Relevant log output
No response
Screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is it solved?
I git pull and docker-compose pull and docker-compose up
The problems still exist

Is it solved?
I git pull and docker-compose pull and docker-compose up
The problems still exist

it is, ensure you have the latest commit with git log
if not, use git fetch origin

@magicdmer sorry about that, missed one item, it's merged now, the docker image may need about 20 minutes to build: https://github.com/danny-avila/LibreChat/pull/2675

@magicdmer sorry about that, missed one item, it's merged now, the docker image may need about 20 minutes to build: #2675

check ok


[Bug]: Variant configuration is not being saved when changed

Bug Description
When creating an LLM app, changing the system prompt, running an evaluation, and navigating to the playground, the new prompt is not visible. Instead, the old system prompt is displayed.
To Reproduce
Steps to reproduce the behavior:

Create an application and navigate to the playground
Change the system prompt
Run an evaluation, then return to the playground; the variant will revert to the default/old prompt.

Expected Behavior
Variant configuration when changed should be saved when the user saves the changes.

@aybruhm what's the status here ? have you started working on it ? can you assign this to another one ?

fixed


[ General question ] functional API, RxJS-like

Hello,
I was just wondering if you ever considered to provide RxJS-like API? Something like:
use function Rx\from;
use function Rx\op1;
use function Rx\op2;

from(1)
    ->pipe(
        op1(),
        op2(),
        ...
    )
    ->subscribe();

Thank you!

In JS, the main reason they switched to pipe was for tree-shaking, which isn't necessary in PHP.  Is there something that you're trying to do that can't be done with the current API?


Added Dream Code Italian localization - legacy

We won't release an other update from the legacy branch. Anyway, I merged your commit because it would be silly to close a working PR.
Thank you and sorry for the long time of waiting.


Upgrade Kafka library to 0.10.1

This issue is obsolete as per moving confluent to 3.2.0  - can be closed but PR https://github.com/wepay/kafka-connect-bigquery/pull/89 needed to correct connect-distributed startup


Chain only() and except()

Use chain only() and except() has two options:
1. addOnly()

only(): force set only keys
addOnly(): add only keys

except() same.
PR: #31
2. forceOnly()

only(): add only keys
forceOnly(): force set only keys

except() same.
PR: #32

@fourstacks What do you think?

Hey @ycs77 - thanks for the PRs.
I've looked over both and whilst I kinda get what's happening, I can't really work out the use case for these new methods?  I'm also a little uncertain as to whether two two PRs are two different suggestions for tackling some problem or whether you're suggesting them both for inclusion?
Perhaps you could give me a few examples of how they would be used and what problems they help to overcome?  It might be that there's a legitimate use case here that I'm not seeing.  Thanks

The example has UserPresenter and different only() usage:
class UserPresenter extends FlexiblePresenter
{
    public function values(): array
    {
        return [
            'id' => $this->id,
            'name' => $this->name,
            'email' => $this->email,
            'description' => $this->description,
            'avatar' => $this->avatar,
        ];
    }
}

1. addOnly()
Force set only() (only()) in presetProfile(), and add only (addOnly) postsCount, likesCount:
public function presetProfile()
{
    return $this->only('name', 'avatar');
}

public function index(User $user)
{
    $user->loadCount('posts', 'likePosts');

    return Inertia::render('User/Profile', [
        'user' => UserPresenter::make($user)
            ->preset('profile')
            ->addOnly('postsCount', 'likesCount')
            ->with(function (User $user) {
                return [
                    'postsCount' => $user->published_posts_count,
                    'likesCount' => $user->like_posts_count,
                ];
            })
            ->get(),
    ]);
}

2. forceOnly()
Force set only() (forceOnly()) in presetProfile(), and add only (only) postsCount, likesCount:
public function presetProfile()
{
    return $this->forceOnly('name', 'avatar');
}

public function index(User $user)
{
    $user->loadCount('posts', 'likePosts');

    return Inertia::render('User/Profile', [
        'user' => UserPresenter::make($user)
            ->preset('profile')
            ->only('postsCount', 'likesCount')
            ->with(function (User $user) {
                return [
                    'postsCount' => $user->published_posts_count,
                    'likesCount' => $user->like_posts_count,
                ];
            })
            ->get(),
    ]);
}

After finishing the above example, I seem to think that forceOnly() is not needed anymore, just a simple chain only feature.
3. Only use chain only()
public function presetProfile()
{
    return $this->only('name', 'avatar');
}

public function index(User $user)
{
    $user->loadCount('posts', 'likePosts');

    return Inertia::render('User/Profile', [
        'user' => UserPresenter::make($user)
            ->preset('profile')
            ->only('postsCount', 'likesCount')
            ->with(function (User $user) {
                return [
                    'postsCount' => $user->published_posts_count,
                    'likesCount' => $user->like_posts_count,
                ];
            })
            ->get(),
    ]);
}

Hey @ycs77 - thanks for the examples.  Sorry to be a bit slow here but I'm still not entirely clear what problem these new methods are trying to solve?  What is is that you want to do that you're unable to do with the current set of api methods?

The problem I want to solve is that only() is defined in preset, but I want after chaining other only() keys.


Recommended to use interface instead of struct in xxxManager

From auth0 created by starsz: go-auth0/auth0#226
Description
Hello, I found that the xxxManager was defined as a struct.
And it was referenced in management.
https://github.com/go-auth0/auth0/blob/0ed82d242c8bfacb02db3352ef6688591108b371/management/management.go#L85-L106
https://github.com/go-auth0/auth0/blob/0ed82d242c8bfacb02db3352ef6688591108b371/management/user.go#L280-L282
Affected Resources

management.Xxx
management.XxxManager

Potential Sample Code
I recommend to use interface instead of struct in xxxManager
Like this:
type UserManager interface{
 Create(u *User, opts ...RequestOption) error
 Read(id string, opts ...RequestOption) (u *User, err error)
 Update(id string, u *User, opts ...RequestOption) (err error)
 List(opts ...RequestOption) (ul *UserList, err error)
...
}

So that we can mock the Management.User and do some unit tests.
The same as other managers.

We'll be closing this issue down as this is not something we're currently gonna tackle in v0 of the SDK, but potentially in an upcoming v1.
It is still possible to unit test any of the funcs even without having the interface, similar to:
func TestNew_WithInsecure(t *testing.T) {
	h := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		switch r.URL.Path {
		case "/api/v2/users/123":
			w.Write([]byte(`{"user_id":"123"}`))
		default:
			http.NotFound(w, r)
		}
	})
	s := httptest.NewServer(h)

	m, err := New(s.URL, WithInsecure())
	assert.NoError(t, err)

	u, err := m.User.Read("123")
	assert.NoError(t, err)
	assert.Equal(t, "123", u.GetID())
}

Hello, when is this planned for?

Hello,
I wanted to know when the change to interface is planned.
It would also be good if there is an extensive and generic http.HandlerFunc for the approach suggested above in TestNew_WithInsecure.

Hey @maheshmedam 👋🏻
Thanks for inquiring into this! We plan to tackle that in the upcoming year.

It would also be good if there is an extensive and generic http.HandlerFunc for the approach suggested above in TestNew_WithInsecure.

Appreciate the feedback! We'll keep this on our radar.


transfer to textlint-ja

[ ] transfer to textlint-ja
[ ] update readme
[x] add npm access

done


Added support for hammerblade stats tag and printing

DRAMsim3 will now print out stats with corresponding tags from hammerblade

alright, i split the epoch stats and tag stats, so now there are two .json files that get printed out

This looks good to me. @mrutt92 what say you?

@marcusnchow Can you add a few comments (or a meta-comment) about what you did? It seems like you have a naming structure here so just describe what your added methods do for posterity

Added this to the commit message
The goal of this commit is to align Dramsim3 stats logging with vanilla_stats in
the manycore

This commit creates two sets of Counters (Epoch and Tag) and associated Printing
Functions (PrintEpochStats and PrintTagStats) within each class
memory_system->dram_system->controller->simple_stats

Within each set there are three types:
{set}_counters_ {set}_vec_counters {set}_histo_counts_

The only difference between the two sets of Counters is the period when they
print to a file and reset their counters

Epoch - prints every "epoch_period" cycle which is set in each dramsim3 config
file after it prints a period, the stats are accumulated in the total counter
and then reset to 0

Tag - prints whenever "print_stat_v_i" in manycore_tb_top is set high, along
with the associated tag in "print_stat_tag_i". This is the same with how
vanilla_stats.csv gets logged. After it prints the counter is reset to 0, but
they are not accumulated.

Would be good to put that in the source somewhere, too.

added message in simple_stats.h

added message in simple_stats.h

I don't see it?


Global quit hotkey

Add a new Settings option for defining a "Task quit" keyboard shortcut. All tasks can then reference this shortcut when quit functionality is needed

Potential issue: all tasks would need to implement the hotkey check/quit code, otherwise the "global" Quit hotkey is no longer truly global. Not sure how enforceable this is in practice...


Permission error when running project

Bug summary
I'm facing an issue when running the project with this lib. I have two PC's. In one of them just works perfeclty. In the other one, this error is displayed.

The workaround to make it work was to remove the import com.zoontek.rnpermissions.RNPermissionsPackage; and  packages.add(new RNPermissionsPackage());  from the MainApplication.java.
Is there an explanation or a right way to solve this?
Thank you all in advance 😉
Library version
3.8.0
Environment info
Environment info in the "working" PC:

System:
    OS: Windows 10 10.0.19044
    CPU: (4) x64 Intel(R) Core(TM) i5-3470 CPU @ 3.20GHz
    Memory: 5.50 GB / 15.90 GB
  Binaries:
    Node: 12.22.6 - ~\AppData\Local\Temp\yarn--1683555139979-0.31997181848243983\node.CMD
    Yarn: 1.22.10 - ~\AppData\Local\Temp\yarn--1683555139979-0.31997181848243983\yarn.CMD
    npm: 6.14.15 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK:
      API Levels: 23, 28, 29, 30, 31, 33
      Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.2, 30.0.3, 32.0.0
      System Images: android-29 | Google Play Intel x86 Atom, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom
      Android NDK: Not Found
    Windows SDK: Not Found
  IDEs:
    Android Studio: Version  <IP_ADDRESS> AI-202.76<IP_ADDRESS>51085
    Visual Studio: 16.5.30011.22 (Visual Studio Community 2019)
  Languages:
    Java: 1.8.0_262
    Python: 3.7.3
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.63.5 => 0.63.5
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

System requirements in the NOT working PC
System:
    OS: Windows 10 10.0.19044
    CPU: (4) x64 Intel(R) Core(TM) i3-3250 CPU @ 3.50GHz
    Memory: 1.71 GB / 7.95 GB
  Binaries:
    Node: 14.20.0 - ~\AppData\Local\Temp\yarn--1683555283336-0.1649267494845914\node.CMD
    Yarn: 1.22.11 - ~\AppData\Local\Temp\yarn--1683555283336-0.1649267494845914\yarn.CMD
    npm: 6.14.17 - C:\Program Files\nodejs\npm.CMD
    Watchman: Not Found
  SDKs:
    Android SDK:
      API Levels: 23, 29, 30, 31, 33
      Build Tools: 29.0.2, 30.0.2, 30.0.3, 31.0.0, 33.0.0
      System Images: android-30 | Google Play Intel x86 Atom, android-33 | Google APIs ARM 64 v8a, android-33 | Google APIs Intel x86 Atom_64, android-33 | Google Play Intel x86 Atom_64
      Android NDK: Not Found
    Windows SDK: Not Found
  IDEs:
    Android Studio: Version     2<IP_ADDRESS> AI-203.7717.56.2031.7678000
    Visual Studio: Not Found
  Languages:
    Java: 1.8.0_211
    Python: 3.10.11
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.63.5 => 0.63.5
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

### Steps to reproduce

1. …
2. …

### Reproducible sample code

```js
I have no a sample for this...

React Native packages are autolinked since 0.60.
What happen here is that you have a currently broken setup (the computer where you have to manually link the package), and one correct (the computer where the package is correctly autolinked + linked manually - which cause the issue: the package is linked twice).
Fix your setup in order to remove manual linking.
I'm closing this since this is not a library issue.


use extension class from di component instead of the kernel

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets
fixes #453

Documentation
-

License
MIT

What's in this PR?
Use the correct Extension class to be future proof
Why?
The class from HttpKernel namespace becomes deprecated.

⚡  Thank you! Would be great to have this tagged 🙏

here you go: https://github.com/php-http/HttplugBundle/releases/tag/1.33.1


fix: update vllm to v0.4.0 hotfix version

Update vllm to v0.4.0 hotfix version, remove --enforce-eager parameter
Fix typo error

@0xff-dev Test failed due to README changed in https://modelscope.cn/models/qwen/Qwen1.5-0.5B/summary.
We do not need to do explicitly check on content.


TOK-446 Return empty fees if response has none of them

Signed-off-by: Nikita Khateev<EMAIL_ADDRESS>

Could you add a unit test for parse_response_with_fees and no fees?

Could you add a unit test for parse_response_with_fees and no fees?

https://github.com/sovrin-foundation/libsovtoken/pull/318/files#diff-761baa00539510de622bfbe579e620ffR94
It is here. Or you mentioned something else?

That's an integration test. I would like a test inside parsers::parse_response_with_fees_handler.


White Cane for Blind People

About the PR
The attached video shows every angle of the sprites I made in the right and left hand, and wielded in right and left hand. It also shows the item description, item icon, and the fact it does 5 blunt damage and then 8 blunt damage when wielded. Later in the video it shows that if you join the character as a permanently blind character, you spawn with the cane. Currently, the only way to obtain the item is upon spawning with the blind trait. I tried using the admin menu (debug>view variables>server components>add component>PermanentBlindness) to make my character blind mid-round and it did not spawn the cane. So I don't think we will have to worry about blind people ahelping out of cloning to be re-blinded and suddenly duping canes.
Credit to Finket for modifying the .cs files, and I did the spriting and yamling.
Media

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

Changelog

:cl: Tristan Thomas

add: Added white cane for blind characters

https://user-images.githubusercontent.com/91704530/236366040-53b85dea-418c-4512-834a-96811865dc49.mp4

oh damn, I was reviewing my PR and noticed I accidentally left an extra png file in there called "oldwieldleft.png" that should not be used or included in the PR. My bad. This also failed for the RSI Validator check. Do I have to redo the entire PR or is there some way to remove that?

You can just delete the file and commit it to your repo, that'll also rerun the checks

I am new to this whole coding/github/etc thing, there's unfortunately still fixes that need to be made before this is ready


Cannot find extension in VSCode

VSCode version: 1.50.1
Commit: d2e414d9e4239a252d1ab117bd7067f125afd80a
Date: 2020-10-15T10:42:48.064Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.18-1-MANJARO
Steps to Reproduce:

Open VSCode
Select Extensions: Marketplace
Search latex language support

I copy this issue from my github repo vscode-LaTeX-support/issues/10. The issue happen in Germany and France.

Does this issue occur when all extensions are disabled?: Yes/No

Duplicate of https://github.com/microsoft/vscode/issues/107997#issuecomment-703245178

Duplicate of #107997 (comment)

Thanks.


Jamie/4186 block beta flags

:nut_and_bolt: Description: What code changed, and why?
Users may opt in to accessing beta, legacy, or other unfinished features by turning the services on or off through our feature flag.  In order to do this, they type feat, lega, or beta, to open the option box.  This becomes problematic if a user is searching for, filtering by, or for any other reason, typing something related to any of these key strings.
This branch updates the feature flag window to only open if the user does not actively have focus on an input, textarea, button, or select element.
:chains: Related Resources
Addresses: https://github.com/chef/automate/issues/4186
:+1: Definition of Done
When user is actively focused on an input, textarea, button, or select element and types the code keys.  The feature flag modal does not open.
When a user is not actively focused on an input, textarea, button, or select element and types the code keys, the feature flag window opens as expected.
:athletic_shoe: How to Build and Test the Change
cd into root folder: build components/automate-ui-devproxy && start_all_services
cd into components/automate-ui: make serve
Click or tab to focus inside an input element (blinking cursor for input/text area) or on a button.
Type feat or lega or beta -> no modal should show
Focus out/off of any of the input elements
Type feat or lega or beta -> modal should show
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

@kagarmoe / @susanev
Now that a user must not be focused on an input element to open the feature flag modal, is this change significant enough to require a documentation update?

@SEAjamieD i dont think we document how to open this modal anywhere in the docs, besides the data feed page. @kagarmoe @mjingle did you want to update it there?


Please update your installation documentation

Hi,
I installed Docker for Windows, confirmed that it's running, and ran your code from this page: https://ipsingh06.github.io/seedsync/install/
But it appears that the code is either incorrect or obsolete. Can you please double check and update?
I get unrecognized command errors in CMD when I type this:
docker run \    -p 8800:8800 \    -v <downloads directory>:/downloads \    -v <config directory>:/config \    ipsingh06/seedsync
(Note: the directories were specified)
And this:
docker-machine ip <IP_ADDRESS>
thanks

Just asking,
Dude you change  with your actual local download directory? Also ?
Those are mapping fields. The dir "/downloads" inside the Docker container will be mapped to the local directory you define. Same goes for config.

Can you provide the exact Docker run command you entered?

Ok, so, first of all, I realized after starting this thread that the code format was not copy-paste ready and I had to remove all of the "". I'm sure this has thrown off many users who are not used to programming, so I recommend to the author to provide the code without those breaks, or at least to instruct users to remove them.
And second, when I re-run the code, I get an error saying the config is incomplete. Here's the output:
C:\Users\xxx>docker run -p 8800:8800 -v C:\Users\xxx\Downloads:/downloads -v "C:\xxx files\install files\Docker_config":/config ipsingh06/seedsync
2021-02-16 05:13:03,881 - INFO - seedsync (MainProcess/MainThread) - Debug mode is disabled.
2021-02-16 05:13:03,888 - INFO - seedsync (MainProcess/MainThread) - Starting SeedSync
2021-02-16 05:13:03,890 - INFO - seedsync (MainProcess/MainThread) - Platform: x86_64
2021-02-16 05:13:04,550 - INFO - seedsync.WebApp (MainProcess/MainThread) - Html path set to: /app/html
2021-02-16 05:13:04,559 - ERROR - seedsync (MainProcess/MainThread) - Config is incomplete

And from what I've read, the docker-machine ip command is obsolete.

It looks like you're missing a \ in your config mapping out is "xxx files" correct?
I'm wondering if the spaces may be causing issues. If I recall, Linux systems don't like spaces in file names or directories.

No, the directories are specified correctly; I just replaced my name with 'xxx'. If you have spaces without quotes, such as C:\docker config:/config instead of "C:\docker config":/config, it will of course not run any of the code, but as I posted earlier, it does run the code, only it says the configuration is incomplete.
I can also confirm that docker will create your specified directories if they don't exist already-- or at the very least the config directory.
Here are the contents of the configuration file that is created:
`[General]
debug = False
verbose = False
[Lftp]
remote_address =
remote_username =
remote_password =
remote_port = 22
remote_path =
local_path =
remote_path_to_scan_script = /tmp
use_ssh_key = False
num_max_parallel_downloads = 2
num_max_parallel_files_per_download = 4
num_max_connections_per_root_file = 4
num_max_connections_per_dir_file = 4
num_max_total_connections = 16
use_temp_file = False
[Controller]
interval_ms_remote_scan = 30000
interval_ms_local_scan = 10000
interval_ms_downloading_scan = 1000
extract_path = /tmp
use_local_path_as_extract_path = True
[Web]
port = 8800
[AutoQueue]
enabled = True
patterns_only = False
auto_extract = True`
Perhaps someone can point out what's missing.
Then, there's the issue with finding the IP, as "docker-machine ip" is obsolete. Plus, there is no set IP in the configuration file, if that's where it should be.

Ok, have your pulled up seedsync yet? Can you?
To me it looks like the config just needs your seedbox details.

Yeah, I finally got it to work. All I had to do is go to localhost:8800 and fill out the seedbox details.
It's unfortunate that the instructions don't make that clear. They imply that you need to first find the docker/container ip.
Open <ip address>:8800 in your browser. In this example that would be http://<IP_ADDRESS>:8800
I would suggest to the author to please clean up the instructions, because if I wasted hours trying to figure out what's wrong, I'm sure that there are plenty of far less experienced users who wouldn't have gotten this far.
And, what's worse, for me, is that this application does not do what I thought it would. 😩 Somebody on Reddit suggested it for transferring files between two seedboxes (can't find the link anymore), whereas this is clearly remote-->local transfers only.
Anyways, thanks for your help @quadcom.


gaze inference

How to load the model separately for gaze inference?

Hi, please see https://github.com/swook/GazeML/blob/master/src/elg_demo.py and https://github.com/swook/GazeML/blob/master/src/core/model.py#L275 for a somewhat complex example...

Please refer to the code at https://github.com/swook/GazeML/blob/master/src/util/gaze.py#L45
Best wishes,
Seonwook


Fixing uninitialized gotData_

Set gotData_ to false on new procedures and make sure it is copied
correctly for copied procedures.

Thanks for fixing this!


DBZ-2412 Avoiding NPE in logical router

https://issues.redhat.com/projects/DBZ/issues/DBZ-2412
@rk3rn3r, can you review + merge this one? Best reviewed commit by commit. Thx!

This should also be back-ported to 1.2.

Unrelated test failure is tracked at https://issues.redhat.com/browse/DBZ-2398

Merged to master & back-ported to 1.2. Thanks!


Deleting a memo also deletes the resources of other memos

Describe the bug
After deleting a memo, memos with resources (like a picture) are deleted. The memo intended to be deleted is removed, but the resources in other memos are also removed. The text in other memos are still visible.
The image is built using the latest codebase pulled for the delete issue: https://github.com/usememos/memos/commit/6bda64064e4dc8a6401e48f83f282833121e8afe
This setup uses a postgresql DB.
Steps to reproduce

Create a memo with text and a resource (a picture)
Create another with text
Delete the memo from step 2
Refresh the page
The memo's resource from step 1 is also deleted (also verified in the "resources" tab that no resources exists)

Screenshots or additional context
https://github.com/usememos/memos/assets/13276370/47fc08f9-3242-4f77-b11e-0f023ecf3fbe

I can confirm the bug is there:
https://github.com/usememos/memos/blob/b20e0097cfd7c84546ceb9daff38e82b330b6223/store/db/postgres/resource.go#L188

Fixed in https://github.com/usememos/memos/commit/2d9c5d16e1a38dc9a225ee8d56bf862b41e7f3a8


App crashing on IOS (Prod) blinking/flickering in development

Describe the bug
When using Draggable Flatlist in production the app first is freezing when I try to drag an Item. After a minute the App crashes. When I'm using the list in development I noticed a flicker/blinking of the list every time I drag an item to the new position.
 const onDragEnd = useCallback(({ data }: DragEndParams<WorkoutExercise>) => {
    dispatch(
      workoutActions.updateWorkoutExercises(
        [...data].map((item, index) => {
          return { ...item, sort: (index + 1).toString() };
        })
      )
    );
  }, []);

 <DraggableFlatList
      contentContainerStyle={{
        paddingBottom: 160,
      }}
      keyboardShouldPersistTaps="handled"
      // style={{ flex: 1 }}
      showsVerticalScrollIndicator={false}
      data={workoutExercises}
      onDragEnd={onDragEnd}
      renderItem={renderItem}
      keyExtractor={(item) => item.id}
    />

To Reproduce
Link a snack if possible.
Platform & Dependencies
Please list any  applicable dependencies in addition to those below (react-navigation etc).

react-native-draggable-flatlist version: 4.0.0
Platform: IOS
React Native or Expo version: 47.0.8
Reanimated version: 2.12.0
React Native Gesture Handler version: 2.8.0

Additional context
Add any other context about the problem here.

@hraschan , were you able to solve this issue ? I'm currently facing the same bug :(

Facing the same issue

Yes I also notice the flicker/blinking on android as well

Nope. I didn’t find any solution. This issue doesn’t come up every time 🥲

Yup same issue, it's constantly flashing


poetry: 1.5.1 -> 1.6.1

Description of changes
Diff: https://github.com/python-poetry/poetry/compare/refs/tags/1.5.1...1.6.1
Changelog: https://github.com/python-poetry/poetry/blob/refs/tags/1.6.1/CHANGELOG.md

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

8 packages built:

poetry
poetry.dist
poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
poetryPlugins.poetry-plugin-export
poetryPlugins.poetry-plugin-export.dist
poetryPlugins.poetry-plugin-up
poetryPlugins.poetry-plugin-up.dist


Missing public accessors on MultiSearch Response

Elastic.Clients.Elasticsearch version:
8.9.2
Elasticsearch version:
8.9.0
.NET runtime version:
6.0
Description of the problem including expected versus actual behavior:
When using the MultiSearch API there seems to be missing public accessors for the response. The "responses" property on MultisearchResponse is a union of MultisearchItem and ErrorResponseBase which are stored in internal fields named Item1 and Item2.
Steps to reproduce:

Make a MultiSearch Query
Inspect the Responses property on the response.

Expected behavior
Results of type T should be publicly available (should be a property that exposes the results of "MultiSearchResponseItems".Select(x => x.Item1))
Provide DebugInformation (if relevant):
The results of the query/queries are non-public.

Hi @Alkaersig,
it's possible to access the union members like this:
var responses = result
    .Responses.Select(resp => resp.Match(first => first, second => (MultiSearchItem<Person>?)null));

The above example returns the first variant for all valid responses and maps to null for all error cases. This is nothing that I would recommend running in production as it simply ignores all error responses, but it simply shows a way how to access the individual variants.

Thanks @flobernd
It seems that it will do the job, although it's not a very intuitive way of getting the data compared to exposing a DictionaryList, Tuble or wrapping object (the old client just had a GetResponses method for getting the results).

I think the new way is inspired by functional programming languages. Personally I like that approach as it provides a type-safe way to to match the desired variant without causing extra allocations.
I'll make sure to add documentation of the Union type so that the Match() method is more present to everybody.
For a custom GetResponses() method, I would suggest to implement a simple extension method like this:
return result.Responses
    .Select(resp => resp.Match(first => first, second => (MultiSearchItem<Person>?)null))
    .Where(x => x is not null);


Simplify contributing docs

CC @simon04, @skipjack. Please have a look.
Closes #587.

@simon04 I did the suggested changes.


Indexing doesn't finish

Describe the bug
Indexing my files does not complete.  When indexing, it stops around the 4k files mark, when in reality there is approximately 50k photos.
re-indexing does not solve the problem.
Photo/video (optional) that causes the bug
If error is in connection with a particular photo/video, It would help a lot with the debugging if you can attach that file too.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Kubuntu 18.04
Browser: Opera

Used app version:

docker-nightly

I am running the latest docker container, with MySQL backend.  The photo structure is is volume mounted directly into the container in the default location.
I have logging set to 5 & 3 for app and MySQL respecitvely and the last photo that is indexed is shown below.  After that file, there is no further logging output to the console.
pigallery       | query: ROLLBACK
pigallery       | { QueryFailedError: ER_DUP_ENTRY: Duplicate entry 'IMG_0995.jpg-76' for key 'IDX_332f316a46fcb158ac71210109'
pigallery       |     at new QueryFailedError (/pigallery2-release/node_modules/typeorm/error/QueryFailedError.js:11:28)
pigallery       |     at Query.<anonymous> (/pigallery2-release/node_modules/typeorm/driver/mysql/MysqlQueryRunner.js:167:45)
pigallery       |     at Query.<anonymous> (/pigallery2-release/node_modules/mysql/lib/Connection.js:502:10)
pigallery       |     at Query._callback (/pigallery2-release/node_modules/mysql/lib/Connection.js:468:16)
pigallery       |     at Query.Sequence.end (/pigallery2-release/node_modules/mysql/lib/protocol/sequences/Sequence.js:83:24)
pigallery       |     at Query.ErrorPacket (/pigallery2-release/node_modules/mysql/lib/protocol/sequences/Query.js:90:8)
pigallery       |     at Protocol._parsePacket (/pigallery2-release/node_modules/mysql/lib/protocol/Protocol.js:278:23)
pigallery       |     at Parser.write (/pigallery2-release/node_modules/mysql/lib/protocol/Parser.js:76:12)
pigallery       |     at Protocol.write (/pigallery2-release/node_modules/mysql/lib/protocol/Protocol.js:38:16)
pigallery       |     at Socket.<anonymous> (/pigallery2-release/node_modules/mysql/lib/Connection.js:91:28)
pigallery       |   message:
pigallery       |    'ER_DUP_ENTRY: Duplicate entry \'IMG_0995.jpg-76\' for key \'IDX_332f316a46fcb158ac71210109\'',
pigallery       |   code: 'ER_DUP_ENTRY',
pigallery       |   errno: 1062,
pigallery       |   sqlMessage:
pigallery       |    'Duplicate entry \'IMG_0995.jpg-76\' for key \'IDX_332f316a46fcb158ac71210109\'',
pigallery       |   sqlState: '23000',
pigallery       |   index: 0,
pigallery       |   sql:
pigallery       |    'INSERT INTO `media_entity`(`id`, `name`, `type`, `directoryId`, `metadataCaption`, `metadataCreationdate`, `metadataFilesize`, `metadataKeywords`, `metadataOrientation`, `metadataBitrate`, `metadataDuration`, `metadataSizeWidth`, `metadataSizeHeight`, `metadataCameraDataIso`, `metadataCameraDataModel`, `metadataCameraDataMake`, `metadataCameraDataFstop`, `metadataCameraDataExposure`, `metadataCameraDataFocallength`, `metadataCameraDataLens`, `metadataPositionDataCountry`, `metadataPositionDataState`, `metadataPositionDataCity`, `metadataPositionDataGPSDataLatitude`, `metadataPositionDataGPSDataLongitude`, `metadataPositionDataGPSDataAltitude`) VALUES (DEFAULT, \'IMG_0995.jpg\', \'PhotoEntity\', 76, DEFAULT,<PHONE_NUMBER>000, 2198584, \'\', 1, DEFAULT, DEFAULT, 2592, 1936, 80, \'iPhone 4\', \'Apple\', 2.8, 0.058823529411764705, 3.85, DEFAULT, DEFAULT, DEFAULT, DEFAULT, -32.04533333333333,<PHONE_NUMBER>3333333, 31.19884726224784)',
pigallery       |   name: 'QueryFailedError',
pigallery       |   query:
pigallery       |    'INSERT INTO `media_entity`(`id`, `name`, `type`, `directoryId`, `metadataCaption`, `metadataCreationdate`, `metadataFilesize`, `metadataKeywords`, `metadataOrientation`, `metadataBitrate`, `metadataDuration`, `metadataSizeWidth`, `metadataSizeHeight`, `metadataCameraDataIso`, `metadataCameraDataModel`, `metadataCameraDataMake`, `metadataCameraDataFstop`, `metadataCameraDataExposure`, `metadataCameraDataFocallength`, `metadataCameraDataLens`, `metadataPositionDataCountry`, `metadataPositionDataState`, `metadataPositionDataCity`, `metadataPositionDataGPSDataLatitude`, `metadataPositionDataGPSDataLongitude`, `metadataPositionDataGPSDataAltitude`) VALUES (DEFAULT, ?, ?, ?, DEFAULT, ?, ?, ?, ?, DEFAULT, DEFAULT, ?, ?, ?, ?, ?, ?, ?, ?, DEFAULT, DEFAULT, DEFAULT, DEFAULT, ?, ?, ?)',
pigallery       |   parameters:
pigallery       |    [ 'IMG_0995.jpg',
pigallery       |      'PhotoEntity',
pigallery       |      76,
pigallery       |<PHONE_NUMBER>000,
pigallery       |      2198584,
pigallery       |      '',
pigallery       |      1,
pigallery       |      2592,
pigallery       |      1936,
pigallery       |      80,
pigallery       |      'iPhone 4',
pigallery       |      'Apple',
pigallery       |      2.8,
pigallery       |      0.058823529411764705,
pigallery       |      3.85,
pigallery       |      -32.04533333333333,
pigallery       |<PHONE_NUMBER>3333333,
pigallery       |      31.19884726224784 ] }

Hi,
Did this work before? (I made some changes that should not effect this, but might happen).
Also is it possible that you somehow have (?) two photos with the same name in one directory?
Patrik

Yeah - heaps with the same name, just different case.
I haven't used for 6 months or so, so I'm not sure. I was looking for an
obvious reason for it stopping, but the logs are not indicating anything.
I might try add some logging to see each file that is being checked.
On Fri, 19 Jul 2019, 21:48 Patrik J. Braun<EMAIL_ADDRESS>wrote:

Hi,
Did this work before? (I made some changes that should not effect this,
but might happen).
Also is it possible that you somehow have (?) two photos with the same
name in one directory?
Patrik
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/93?email_source=notifications&email_token=ABST2ICO5I4IE3M3MDZ6KHTQAHA3ZA5CNFSM4IFBXO62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2LVT2Q#issuecomment-513235434,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABST2IA3SPR75UHSZ6PYN7LQAHA3ZANCNFSM4IFBXO6Q
.

Hi,
Can you make sure that your nightly is up-to date?
I have fixed the case sensitive issue a while ago.
Patrik

Hahah, yeah I know, you fixed it for me! I don't think it is the file name,
as you had fixed that, hence why it says that it is skipping the duplicate.
But not sure how to see where it gets up to before stopping
On Sat, 20 Jul 2019, 15:33 Patrik J. Braun<EMAIL_ADDRESS>wrote:

Hi,
Can you make sure that your nightly is up-to date?
I have fixed the case sensitive issue a while ago.
Patrik
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/93?email_source=notifications&email_token=ABST2IA32JKARVNPRZFYKO3QAK5U5A5CNFSM4IFBXO62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NI23Y#issuecomment-513445231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABST2IF77EKIKOAN5SL2FDLQAK5U5ANCNFSM4IFBXO6Q
.

Hi,
You can set the logging at config.json to
"log": {
            "level": 5,
            "sqlLevel": 3
        }

From your log it looks like that you have the exactly same file name (same casing) in the same folder.
My educated guess that you might have something like this:
folder1
   IMG.jpg
Folder1
   IMG.jpg

So I made a fix and test for folder names and path to be case sensitive. You can try it with the latest nightly.

Yep - that is a good point.
I have:
- 2018
  |- 01 Jan
  |- 02 Feb
  |- ...
  |- 12 Dec
- 2019
  |- 01 Jan
  |- 02 Feb
  |- ...
  |- 12 Dec

So that sounds like a perfect guess.  I'm trying now

Yep - here it is:
pigallery       | error: { Error: ER_DUP_ENTRY: Duplicate entry '12 Dec-2011/' for key 'IDX_05d748647d27bf887ef0826de7'
pigallery       |     at Query.Sequence._packetToError (/pigallery2-release/node_modules/mysql/lib/protocol/sequences/Sequence.js:47:14)
pigallery       |     at Query.ErrorPacket (/pigallery2-release/node_modules/mysql/lib/protocol/sequences/Query.js:77:18)
pigallery       |     at Protocol._parsePacket (/pigallery2-release/node_modules/mysql/lib/protocol/Protocol.js:278:23)
pigallery       |     at Parser.write (/pigallery2-release/node_modules/mysql/lib/protocol/Parser.js:76:12)
pigallery       |     at Protocol.write (/pigallery2-release/node_modules/mysql/lib/protocol/Protocol.js:38:16)
pigallery       |     at Socket.<anonymous> (/pigallery2-release/node_modules/mysql/lib/Connection.js:91:28)
pigallery       |     at Socket.<anonymous> (/pigallery2-release/node_modules/mysql/lib/Connection.js:502:10)
pigallery       |     at Socket.emit (events.js:198:13)
pigallery       |     at addChunk (_stream_readable.js:288:12)
pigallery       |     at readableAddChunk (_stream_readable.js:269:11)
pigallery       |     --------------------
pigallery       |     at Protocol._enqueue (/pigallery2-release/node_modules/mysql/lib/protocol/Protocol.js:144:48)
pigallery       |     at PoolConnection.query (/pigallery2-release/node_modules/mysql/lib/Connection.js:200:25)
pigallery       |     at MysqlQueryRunner.<anonymous> (/pigallery2-release/node_modules/typeorm/driver/mysql/MysqlQueryRunner.js:158:44)
pigallery       |     at step (/pigallery2-release/node_modules/tslib/tslib.js:136:27)
pigallery       |     at Object.next (/pigallery2-release/node_modules/tslib/tslib.js:117:57)
pigallery       |     at fulfilled (/pigallery2-release/node_modules/tslib/tslib.js:107:62)
pigallery       |     at process._tickCallback (internal/process/next_tick.js:68:7)
pigallery       |   code: 'ER_DUP_ENTRY',
pigallery       |   errno: 1062,
pigallery       |   sqlMessage:
pigallery       |    'Duplicate entry \'12 Dec-2011/\' for key \'IDX_05d748647d27bf887ef0826de7\'',
pigallery       |   sqlState: '23000',
pigallery       |   index: 0,
pigallery       |   sql:
pigallery       |    'INSERT INTO `directory_entity`(`id`, `name`, `path`, `lastModified`, `lastScanned`, `mediaCount`, `parentId`) VALUES (NULL, \'12 Dec\', \'2011/\',<PHONE_NUMBER>687, NULL, 3, 194)' }
pigallery       | { QueryFailedError: ER_DUP_ENTRY: Duplicate entry '12 Dec-2011/' for key 'IDX_05d748647d27bf887ef0826de7'
pigallery       |     at new QueryFailedError (/pigallery2-release/node_modules/typeorm/error/QueryFailedError.js:11:28)
pigallery       |     at Query.<anonymous> (/pigallery2-release/node_modules/typeorm/driver/mysql/MysqlQueryRunner.js:167:45)
pigallery       |     at Query.<anonymous> (/pigallery2-release/node_modules/mysql/lib/Connection.js:502:10)
pigallery       |     at Query._callback (/pigallery2-release/node_modules/mysql/lib/Connection.js:468:16)
pigallery       |     at Query.Sequence.end (/pigallery2-release/node_modules/mysql/lib/protocol/sequences/Sequence.js:83:24)
pigallery       |     at Query.ErrorPacket (/pigallery2-release/node_modules/mysql/lib/protocol/sequences/Query.js:90:8)
pigallery       |     at Protocol._parsePacket (/pigallery2-release/node_modules/mysql/lib/protocol/Protocol.js:278:23)
pigallery       |     at Parser.write (/pigallery2-release/node_modules/mysql/lib/protocol/Parser.js:76:12)
pigallery       |     at Protocol.write (/pigallery2-release/node_modules/mysql/lib/protocol/Protocol.js:38:16)
pigallery       |     at Socket.<anonymous> (/pigallery2-release/node_modules/mysql/lib/Connection.js:91:28)
pigallery       |   message:
pigallery       |    'ER_DUP_ENTRY: Duplicate entry \'12 Dec-2011/\' for key \'IDX_05d748647d27bf887ef0826de7\'',
pigallery       |   code: 'ER_DUP_ENTRY',
pigallery       |   errno: 1062,
pigallery       |   sqlMessage:
pigallery       |    'Duplicate entry \'12 Dec-2011/\' for key \'IDX_05d748647d27bf887ef0826de7\'',
pigallery       |   sqlState: '23000',
pigallery       |   index: 0,
pigallery       |   sql:
pigallery       |    'INSERT INTO `directory_entity`(`id`, `name`, `path`, `lastModified`, `lastScanned`, `mediaCount`, `parentId`) VALUES (NULL, \'12 Dec\', \'2011/\',<PHONE_NUMBER>687, NULL, 3, 194)',
pigallery       |   name: 'QueryFailedError',
pigallery       |   query:
pigallery       |    'INSERT INTO `directory_entity`(`id`, `name`, `path`, `lastModified`, `lastScanned`, `mediaCount`, `parentId`) VALUES (?, ?, ?, ?, ?, ?, ?)',
pigallery       |   parameters: [ null, '12 Dec', '2011/',<PHONE_NUMBER>687, null, 3, 194 ] }

Hm. This one should not actually happen.
Can you try 2 things:

Making sure that you have the latest (the admin page now shows the version next to the 'mode selection button)
reset your database (to make sure: manually deleting all tables) then rerunning the indexing.

Thanks,
P

Ok, i did a full removal of the DB & Pig2 images, volumes and networks.  Ran through and they are all good now!
That said, i still cannot see the version number in the admin section
cheers

Hi,
thanks. Probably there was some inconsistency with the DB.
Also, I suspect for some reason docker does not download the latest image for you. You should see the version here (you can check it on the heroku demo):

As an alternative, you can try using 1.7.0-stretch instead of nightly-stretch image.

I don't think it would be the dB, I deleted and restarted it quite often.
And forced redownloads by deleting the images.
Anyway, all up and running with 65k Images
Thanks - closed
On Mon, 22 Jul 2019, 16:44 Patrik J. Braun<EMAIL_ADDRESS>wrote:

Hi,
thanks. Probably there was some inconsistency with the DB.
Also, I suspect for some reason docker does not download the latest image
for you. You should see the version here (you can check it on the heroku
demo https://pigallery2.herokuapp.com/):
[image: ver]
https://user-images.githubusercontent.com/7512182/61618572-6789af00-ac6d-11e9-947c-66544598c010.jpg
As an alternative, you can try using 1.7.0-stretch instead of
nightly-stretch image.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/93?email_source=notifications&email_token=ABST2IF75O3IECEHFNGU3R3QAVXPDA5CNFSM4IFBXO62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2PG6FQ#issuecomment-513699606,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABST2IECH2CO7TDRKQYY7ALQAVXPDANCNFSM4IFBXO6Q
.

Okay. Let me know if you have this issue again.
P

There was indeed a bug that causes very similar Foreign key error that you also reported:
If you indexed a sub folder before its parent folder.
Namely if you reset the DB then open the /gallery/subdir first, then /gallery/ you get a foreign key error.
The last commit should fix it.
Also the docker nightly should also work now. (It had caching enabled that prevented using the latest commit for build)

Cheers I'll try it out again.
And I'll pull a new container
On Fri, 26 Jul 2019, 05:42 Patrik J. Braun<EMAIL_ADDRESS>wrote:

There was indeed a bug that causes very similar Foreign key error that you
also reported:
If you indexed a sub folder before its parent folder.
Namely if you reset the DB then open the /gallery/subdir first, then
/gallery/ you get a foreign key error.
The last commit should fix it.
Also the docker nightly should also work now. (It had caching enabled that
prevented using the latest commit for build)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/93?email_source=notifications&email_token=ABST2IAAUCSJCB6DDUNNX7DQBIM3VA5CNFSM4IFBXO62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD223QNA#issuecomment-515225652,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABST2IBLWEZSP3LK3V2GQMTQBIM3VANCNFSM4IFBXO6Q
.


Add sys.gettimeofday? and std/bench

Commit 1
Microsecond-timing available via sys.gettimeofday? (added tests/posix.bn too):
Bone Lisp 0.4.0-pre
@0: (sys.gettimeofday?)
(1467210688 959696)
@1: (sys.gettimeofday?)
(1467210694 231724)

Commit 2
Add std/bench module to allow measuring elapsed time of code:
Bone Lisp 0.4.0-pre
@0: (use std/bench)
(std/bench)
@1: (mysub (long-op x) (unfoldr =0? id -- x))
#sub(id=0x7f993fee00d5 name=long-op argc=1 take-rest?=#f)
@2: (long-op 10)
(1 2 3 4 5 6 7 8 9 10)
@3: (say-time (long-op 10))
Running (long-op 10) ... Elapsed: 30 usecs
#t
@4: (say-time (long-op 100))
Running (long-op 100) ... Elapsed: 233 usecs
#t
@5: (say-time (long-op 1000))
Running (long-op 1000) ... Elapsed: 2341 usecs
#t
@6: (say-time (long-op 10000))
Running (long-op 10000) ... Elapsed: 8607 usecs
#t

As for the second commit, I'm not sure the names I chose (usec-diff, measure, say-time) are the best fit. Suggestions for changes are welcome.

This is cool... and looks good so far. A few thoughts:
usec-diff seems generally useful in combination with sys.gettimeofday?, maybe it should be in a posixprelude.bn file that gets loaded by boneposix.c, what do you think?
Since sys.gettimeofday? may return false on failure, we should check for that. Probably posixprelude.bn should contain a convenience subroutine like:
(defsub (gettimeofday)
  "..."
  (aif (sys.gettimeofday?)
       it
    (err "...")))

How about renaming usec-diff to timeofday-diff? This suggests the relation to sys.gettimeofday?.
Maybe rename measure to measure-time?

By the way, what about copyright notices and licensing info in the files?

All suggestions sounds good, I'll try them (not sure I'll get to it today).
As for the copyright notices, I'm not sure what I should do. How is it different than adding some lines to an existing file? I don't mind handing you the copyright to the code (it's gonna stay in your project...). Let me know if I should copy the template from the other files.

BTW - I have a hidden agenda with all these small contributions - I want to try to implement Mal ( https://github.com/kanaka/mal ) in bone-lisp. I'm still not sure how can I implement something like env_set (mutable key-value map) using bone-lisp (and there's the memory mgmt thing to wrap my head around, but that's the whole idea of this exercise :-).

Copyright: The correct thing would probably be to add your name to all files that you have changed, and put only your name in files that you created. Copyright assignment is not possible in all countries.  But if you prefer to stay anonymous, you can also just copy the header with my name from other files. The most important thing is that there is a license statement.
Mal: That's going to be interesting. If you use a tree to store an environment, you don't have to copy too much data when creating a new version of the environment. You could try to use reg-loop as a copying garbage collector (but currently, copying does not recognize shared subtrees).
The easy solution would be to add mutable hash tables that need to be deleted manually. There is a hash table implementation already, so it's trivial to make it available to Bone code. But then you've got nothing to wrap your head around anymore. ;)
Feel free to always discuss your ideas about implementing Mal here (you could open an issue for such a discussion, for example).

I just realized: Since bone_init() does not load prelude.bn, bone_posix_init() also should not load posixprelude.bn. That should be done in main() to be consistent. I got that wrong above.

We can have (use posixprelude) at the end of prelude.bn, or add a bone_load("posixprelude"); in main.c. Identical effect, I believe.

Re: mutable hash table - I thought all bone data types are immutable... So I assumed you wouldn't want to add it.
I don't mind performance, so the env map can be alist and set(k,v) can return a new alist, and then use reg-loop to replace the "current-env". Then comes the atom data type (similar to Clojure atom) which is mutable... The Erlang implementation of Mal resorts to holding a separate erlang process for each mutable object (mal env or mal atom).

Another try at gettimeofday and std/bench (force-pushed to same branch).

Re posixprelude: This should not be in prelude.bn, because the POSIX module is supposed to be optional. main() is a good place because that's where we decide whether we activate the POSIX bindings. I.e. a program that wants to include Bone Lisp as a scripting language would not include main.c anyway and should be able to use prelude.bn unaltered.
Re immutability: Well, I am not dogmatic about it at all (in fact, there is already _var! which changes a global binding, though it's intended for internal use only), but there is a reason for chosing immutability: It's the simplest way to ensure that there won't be any dangling pointers. For example:
(with x 0
  (in-reg (set! x (list 0)))
  x

This would return a dangling pointer. For the same reason, Bone does not have reg-new/reg-destroy! but restricts the use of regions to in-reg and reg-loop.
However, there is nothing preventing us from providing an unsafe module. (But note that the current VM implementation does not allow the implementation of set! because closures have their own copys of bindings; this avoids a bit of indirection when accessing free variables).
I did not work on this so far because I want to see how far I can get without it, and don't want to be tempted to use these features all the time. :) But I expect that eventually they will probably be added anyway.

Thanks for the sys.errno fix.
Re: immutability: According to the classic Lisp process, eventually you'll have to write a Bone Lisp evaluator in Bone Lisp :-) The question is whether you have the tools to do it. I'm also interested in the "how far can you go without mutability".


internal/poll: provide method to change maxSpliceSize

https://github.com/golang/go/blob/c5d7f2f1cbaca8938a31a022058b1a3300817e33/src/internal/poll/splice_linux.go#L20
I suffer from poor performance caused by the default maxSpliceSize 4MiB on my embedded platforms. Would it be beneficial that a method is provided to change it?
After I changed the value from 4MiB to 32KiB on my OpenWRT router, I got a twice performance.

CC @acln0
If 32K is better than 4M on your system, perhaps we should just use that everywhere.

Seconded @ianlancetaylor if there is a better value, let's make it the default rather than a configuration option.

If 32K is better than 4M on your system, perhaps we should just use that everywhere.

In principle, I don't see why not, but I would need to run benchmarks to make sure that the change doesn't represent a regression for code that transfers a huge volume of data per unit of time through splice (the kind of systems that saturate links). Such code was the motivation for introducing the splice code in the first place, and a regression would be undesirable. If there is a singular better value, then, of course, that should be the default. Trying to sneak configuration down to the splice code seems to be prohibitively difficult to me anyway, so I think that option is off the table.
I do wonder why a maximum transfer size of 4M per splice round makes things significantly slower, though. We also have a large round size here:
https://github.com/golang/go/blob/c5d7f2f1cbaca8938a31a022058b1a3300817e33/src/internal/poll/copy_file_range_linux.go#L15
Should we change this one too? Or is this one different because it's copy_file_range and not splice?
Even if we changed the 4M to 32K today, the change wouldn't make it into the 1.15 tree, so I think it would be better to investigate it properly, by reading kernel source code, running more benchmarks, and so forth. I am going to do that at some point in the 1.16 development cycle.
@march1993
Sorry for the trouble. If you don't want to maintain a patched Go tree for a cycle until this is sorted out, try using, instead of io.Copy(dst, src), io.Copy(struct{ io.Writer }{dst}, src).
But while you are at it, and have patched your tree already, can you try a few additional intermediate transfer sizes and report the performance you observe, please? Perhaps 64K, 256K, and 1M? Thanks.

The length should be under MAX_RW_COUNT which based on PAGE_MASK
https://github.com/torvalds/linux/blob/d76913908102044f14381df865bb74df17a538cb/fs/splice.c#L783-L784
I can send a CL for this and done some tests on my rpi etc.

@march1993 What is the contents of the file /proc/sys/fs/pipe-max-size on your system?  Thanks.

@march1993 What is the contents of the file /proc/sys/fs/pipe-max-size on your system? Thanks.

It was 1048576.

@march1993 What is the contents of the file /proc/sys/fs/pipe-max-size on your system? Thanks.

It was 1048576.

Were you transferring a mass of data via splice(2) or small amount of data？and what's your kernel version? thanks.
@march1993

Were you transferring a mass of data via splice(2) or small amount of data？and what's your kernel version? thanks. @march1993

A mass of data.  The kernel version was 5.4.154.


doc: add k8s control nodes on burrito 1.4.x

I installed burrito 1.4.1 with a single control node for a test.
Now I want to add two control nodes for high availability but I do not know
how to do it.
Do you have any document about how to add control nodes?
Thanks,

I added a document - Add control plane nodes in ReadTheDocs.
https://burrito.readthedocs.io/en/aster/operation/add_control_plane_node.html


Add missing HTML code

Added missing HTML Doctype
Added missing HTML header

Looks good. Thanks for your contribution. Merging this.

You're welcome dude, I'm glad I could contribute to your project ;)
Feel free to take a look to mine :)


Implement require-specific-component rule

Resolve #24
Example of incorrect code for this rule:
import { Box } from "@chakra-ui/react";

<Box display="flex" margin="2" paddingTop={4}>
  Hello
</Box>;

Example of correct code for this rule:
import { Box, Flex } from "@chakra-ui/react";

<Flex margin="2" paddingTop={4}>
  Hello
</Flex>;

Thank you for creating this.  I feel sick and I haven't been able to scrutinize it, but overall it looks good.
I left some comments, but it's my personal preference. It's okay to reject.
If there is no problem with the self-review, please merge.

In the future, there may be multiple conversion candidates. In that case, we need to think about which one we want to convert to.
Even now <Box as='img' d='flex'/> has two candidates, but it seems non-sense because it can't have child nodes..

@no-yan  Thank you for reviewing despite the sick! Make sure you rest. Really no problem to be inactive for a while since this is just a volunteer project. your health is most precious.

Arigato!


CF::Data::Find + Tests

Coverage increased (+0.0004%) to 99.644% when pulling 7056cf5c06a729576bd5702f25ccfb9901ee2a49 on DigiDNA:master into 39a4e21dbc946bdfca17899f3661fa037410c540 on macmade:master.


Implement best practices for prometheus-operator-app app

Coming from here.
The goal of this issue is to have the App eventually adhere to standard cloud-native best-practices. The points below are to be considered non-exhaustive. Feel free to add some that make sense and remove some that don't. If this issue does not make sense (e.g. is a duplicate or covers a deprecated app) please close it.
Expected outcome:

[ ] the app has properly set PDBs (as percentages)
[ ] the app can scale horizontally based on Custom Metrics (if it makes sense)
[ ] the app runs with more than 1 replica by default (if it needs HA)
[ ] the app does not use emptyDir volumes if not strictly necessary

As this is quite a lot of effort but only minimal value gained at this time we won't be doing it for now. So closing it out of scope for now.


client side of multi TUF file atomic update

Depends on https://github.com/endophage/gotuf/pull/16
Includes #56
Signed-off-by: David Lawrence<EMAIL_ADDRESS>(github: endophage)

LGTM but the API makes me sad.

@NathanMcCauley what makes you sad?

FYI, now that we have multi-update I'm planning to remove the single file update and make this the only way to update TUF files. Net change in number of HTTP endpoints is then 0. This is a more robust solution than the existing "UpdateHandler" as it requires more strictly structured data.

It makes me sad that we're passing in a map from string->raw json to gotuf in order to update the remote files.
I wish it took a list of TufFiles where TufFile was something closer to this object: https://github.com/endophage/gotuf/blob/master/data/roles.go#L54
Agreed that its a step forward as its more structure than we had before.

What you're calling TufFile is currently represented as the partially parsed Signed, and the fully parsed objects SignedRoot, SignedTargets, SignedSnapshot, and SignedTimestamp.
It's interesting you reference Role because I'm quite unhappy with that. The code that handles them is ugly, especially root.json roles.

You're right, it should actually probably be something like this:
type TufRepo struct {
	Targets []Target
	Root     Root
	Timestamp      Timestamp
	Snapshot      Snapshot
}

Then, json encoding/serialzing would happen inside gotuf.

I could simply add a ToJSON method on Signed if it's the marshalling in the notary client you want to remove. I have a similar ToSigned method on the SignedRoot etc... structs.

I think the real problem is I need to bulk up the gotuf client. Currently we have a gotuf client that handles reading only really (a design I carried over from flynn, although the client has been fully rewritten), and a notary client that handles writing, and mostly just calls the gotuf client for reading.

Makes sense, I'm fine with spiffying this up later if you are.

Yeah, I've been planning to put write functionality into the gotuf client for a while. It just hasn't been critical and at this point is going to be a cross-repo refactor, moving stuff out of notary's client and into gotuf's.

@endophage any more functionality you want to add here?

Just tests. Working on wrapping up the caching in gotuf right now. Then tests here.

Looks like we would not be able to merge this PR because of merge conflicts. Please rebase, fix conflicts, and force push to your branch.


confusing error when executing %sqlcmd

When executing %sqlcmd, I get this:

But the error is unclear. Instead, we should throw an UsageError and tell the user what are the available commands, along with some examples

Request to assign this issue (#262)

Hi Ido/ Eduardo,
I would like to know the control flow of the above command '%sqlcmd'.
I have forked the repo and created a new branch 'dev-262'. I changed the file magic_cmd.py to debug the issue. The issue occurs at line 44. So after line 43, I added some print statements to check the split of the line argument.
However, when I committed the file and tried to install the repo as a pip package using:
(pip install git+https://github.com/raghavSharmaCode/jupysql.git@dev-262)
It did not reflect the changes that I was expecting when I ran the '%sqlcmd' command in a test .ipynb file.

Hence I would like to learn about the OOPS implementation of this package so that we are able to debug and resolve the issue on a low-level design basis.

did you setup with the invoke setup command. Once you fork and clone the repo, you can run:
invoke setup

This will install JupySQL in editable mode, meaning all changes should reflect (if you're using Jupyter, you need to enable autoreload or restart the kernel, though)
To check if JupySQL is correctly installed in the current environment:
In [1]: import sql; sql
Out[1]: <module 'sql' from '/Users/eduardo/dev/jupysql/src/sql/__init__.py'>

You can see that sql prints the path where I cloned the repo, you should see something similar

Hi Eduardo,
Please confirm if the below image explains what you asked for.

The latest code is in the dev-262 repo.
Please confirm if I should raise a pull request for you to verify.
Thanks and regards,
Raghav

Looks good!
Some feebdback:
If the user passes
%sqlcmd

The error should say:
Missing argument for %sqlcmd
Valid commands are: tables, columns

If they pass a wrong argument:
%sqlcmd stuff

The error should say:
stuff is not a valid argument for %sqlcmd
Valid commands are: tables, columns

As requested,

Also, I would like to know the significance of the default line argument as an empty string. An empty string is a use case but how much is it relevant as the default argument?
I will be raising a pull request for you to verify the changes. Looking forward to your response.
Best,
Raghav

for reference, the PR is: https://github.com/ploomber/jupysql/pull/271


chore(main): release 0.9.0

:robot: I have created a release beep boop
0.9.0 (2024-05-16)
Features

Created stepper input (#67) (ca51b1f)
Created Time Input (#75) (fb7b629)
Global Header (zebrafed#38) (0822d4f)

Bug Fixes

breadcrumb rounded icon (c46788d)
Button update (zebrafed#42) (0822d4f)
Correct colors on Radio button hover / focus states (c46788d)
enable mouse region on desktop for selectable areas (c46788d)
navigation bar text color (c46788d)
Remove calls to zeta in button that are not needed (c46788d)
UX-1090 - Add expansion option for TopAppBar (#73) (ab36e8e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ZebraDevs/zeta_flutter/releases/tag/v0.9.0 :sunflower:


New package: MyExample v0.1.0

Registering package: MyExample
Repository: https://github.com/ErwanDemairy/MyExample.jl
Created by: @ErwanDemairy
Version: v0.1.0
Commit: 5a97776cbbd77b0b4b9804cc4e4b916fff89caea
Reviewed by: @ErwanDemairy
Reference: https://github.com/ErwanDemairy/MyExample.jl/issues/1#issuecomment-1414012202
Description: example from "Developing Julia Packages" video

please close


SQSclient http issue under highload

SQS client cause socket exceptions under high load in .net core.
Expected Behavior

Sqsclient should able to manage the httpclient lifetime with the applications.

Resused the httpclients and not consume all of the ports. 8124
Current Behavior
Clientconfig httpclient factory is null. Therefore no disposable httpclients. Cache is increasing to the maximum.
Possible Solution
    public class SqsHttpClientFactory : Amazon.Runtime.HttpClientFactory
    {
        private readonly IHttpClientFactory _httpClientFactory;

        public SqsHttpClientFactory(IHttpClientFactory httpClientFactory)
        {
            _httpClientFactory = httpClientFactory;
        }

        public override HttpClient CreateHttpClient(IClientConfig clientConfig)
        {
            return _httpClientFactory.CreateClient();
        }
    }

            services.AddAWSService<IAmazonSQS>();
            services.AddSingleton<IAmazonSQS>(x => new AmazonSQSClient(new AmazonSQSConfig
            {
                HttpClientFactory = new SqsHttpClientFactory(x.GetRequiredService<IHttpClientFactory>())
            }));

Steps to Reproduce (for bugs)
Create ecs fargate cluster. Run a task where the container runs a .net core app on linux.
Context

Services goes against high load. Not supports the SLAs which are defined for sqs.

Are you sure about the http a best protocol for messaging?
Your Environment

AWSSDK.Core version used: newest
Ecs Fargate via linux containers.

.NET Core Info

.NET Core version used for development:  2.2
.NET Core version installed in the environment where application runs: 2.2

What do you have set in the client config around this topic?  Is CacheHttpClient true?

        /// <summary>         /// <para>         /// This is a switch used for performance testing and is not intended for production applications          /// to change. This switch may be removed in a future version of the SDK as the .NET Core platform matures.         /// </para>         /// <para>         /// If true, the HttpClient is cached and reused for every request made by the service client          /// and shared with other service clients.         /// </para>         /// <para>         /// For the .NET Core platform this is default to true because the HttpClient manages the connection         /// pool.         /// </para>         /// </summary>         bool CacheHttpClient { get; }

        /// <para>
        /// This is a switch used for performance testing and is not intended for production applications
        /// to change. This switch may be removed in a future version of the SDK as the .NET Core platform matures.
        /// </para>
        /// <para>
        /// If true, the HttpClient is cached and reused for every request made by the service client
        /// and shared with other service clients.
        /// </para>
        /// <para>
        /// For the .NET Core platform this is default to true because the HttpClient manages the connection
        /// pool.
        /// </para>
        /// </summary>
        bool CacheHttpClient { get; }```

I am also having this issue.  Is there a suggested way to use the SQSClient without consuming all the connections?

I spoke with the aws support through my company, and they wont solve it,bc there is workaround for it.
What I mentioned above. SqsHttpClientFactory with asp.net core httpclientfactory
pros.: solve the issue
cons: asyncblock the client (if u want to consume and produce) So everything which is could be event-sourcing
https://github.com/aws/aws-sdk-net/issues/565
Implementation guide:
https://www.stevejgordon.co.uk/httpclient-connection-pooling-in-dotnet-core
same issues:
https://github.com/aws/aws-sdk-net/issues/1339
https://github.com/aws/aws-sdk-net/issues/971

This was really useful for adding poly policies and using reflection for adding tests. Thanks.


Added PassiveAggressiveClassifier

SOLVES ISSUE
This PR solves issue #149
CHANGES MADE
Added PassiveAggressiveClassifier
Made changes to
https://github.com/blobcity/autoai/blob/main/blobcity/config/classifier_config.py
NOTE
Please consider this PR as a submission towards Hacktoberfest 2021 and add the hacktoberfest-accepted label to it.

@RajuUwU change the key from
"PassiveRegressor":[
linear_model.PassiveAggressiveClassifier
to
"PassiveAggressiveClassifier"


Token Expiry is not set during creation

In this library, I see that token expiry is set after generation. Is it the expected behavior?

@hansiemithun - Thanks for your question. Not sure what you mean with "set after generation".
If you intend to use the exp claim, you are supposed to include it in the payload before you invoke the generate method.  At verification time, if present in the payload, the exp claim is automatically processed. In summary:
iat claim:

Automatically inserted in the payload at generation time;
Processed at verification time only if you chain the parse method with the setTokenLifetime method.

exp claim:

Up to the user to include it in the payload before generation;
Automatically processed at verification time if present.

I hope this clarifies.

I could not see in an example from the documentation to set before creating token. Could you please provide here a small piece of program to set.
I was playing with this and I created a small program using this module here: https://github.com/hansiemithun/jwe-example
I am trying to understand even after the token is expired, I could still use the same token inside jwt.io to decode the payload. Is there a way to delete the token or make the existing token invalid for decode and see it externally? Or is it the expected behavior? Do we need to internally handle it in our application after expired?

@hansiemithun - The exp claim goes in the payload. There is nothing magic about it. For example:
let payload = {
  iss: 'auth.mydomain.com',
  aud: 'A1B2C3D4E5.com.mydomain.myservice',
  sub<EMAIL_ADDRESS>  info: 'Hello World!',
  list: [1, 2, 3],
  exp: Math.floor(Date.now() / 1000) + 3600   // will expire in one hour, i.e. 3600s is the life cycle
};

The block of code with setTokenLifetime is related to iat, not to exp. Again:
iat is a fact of life. The generate method knows the exact moment when you're generating the token. On the other hand, the iat verification depends on how long you want to life cycle to be. You must puss this info at verification time with the setTokelifetime method.
The situation for exp is the reverse.  You must set it explicitly at generation time since it depends on how long you want the life cycle to be. On the other hand, at verification time it is a fact of life... if it is there, it immediately possible to know if the token has expired or not by comparing exp with the current time.
Nothing of this has anything to do with whether the token is usable or not! Signature (JWS) and decryption (JWE) have nothing to do with either the iat or exp claim (which are both part of the payload). It's your application policy to decide what you do with expired tokens.


renaming a directory to its own subdirectory deletes it instead

gfal2-util 1.5.3
gfal-rename /path/to/source /path/to/source/sub
seems to permanently remove the source directory

Hello,
I'm confused on what the expected result should be for this.
Linux mv would refuse this renaming.
Gfal2 creates a POSIX-like file object and sends the request to the server.
The servers I've tested (EOS via root and https protocols) refused this renaming.
What is the storage endpoint you've tried?
Cheers,
Mihai

Thanks for the reply - big mistake on my part, I was trying to move a bunch of files one directory deeper and didn't realize the obvious.


Add /services/Achievement/findByCodingamerId

When this endpoint is called ?

URL : https://codingame.com/home

 Achievement


Switching submission collection during workflow does not work

Describe the bug
Switching the submission collection during workflow does not work.
To Reproduce
Steps to reproduce the behavior:

Login as administrator.
Take a task from the task pool and try to change the collection.

Expected behavior
The workflow item should change its owning collection to the collection selected in the dropdown.
Related work
https://github.com/DSpace/dspace-angular/issues/1234

Collection change in workflow is not trivial. The collection could have different workflow definition associated or also if the same workflow definition is used it is possible that the flow will change due to custom workflow steps.
A first "workaround" would be to disable the feature in workflow and allow to change the collection only during the submission (workspace phase)


Log command in REPL does not properly toggle settings

The log command in kore-repl behaves unexpectedly:

the logType field is required, meaning it must be specified every time you want to adjust which logs are printed
the logEntries field is required, meaning you must repeat the logs you want printed every time you change a different setting (stderr or file? standard format or oneline? severity threshold...)
the optional fields use a default value if one is not provided. For example, this means that if you do not specify a severity and the current setting is Info, it will be reset to Warning. If you do not specify a timestamp setting, they will be enabled even if the current setting is disabled.

These issues could be fixed by (1) separating each component of the log command into its own command, e.g. enable-log-timestamps (2) creating a much more general set-option command, or (3) making GeneralLogOptions hold Maybe things, and modify generalLogOptionTransformer to only modify using the fields that are present.
Option (2) seems like overkill.
Personally, I think it makes sense for all of these controls to be part of the log command (keeping them all together). Option 1 would lead to having 5 separated but closely related commands floating around. In practice, I think it's easier to do
log stderr
log [DebugTransition]
log oneline

compared to
log-type stderr
log-entries [DebugTransition]
log-format oneline

This is just my opinion. I'm opening this issue so we can discuss before taking any action if any.

@JKTKops suggests we also rename GeneralLogOptions


cant close youtube tv popup

Describe the bug
when watching any video a popup of "try youtube premium/ try youtube tv" pops up and cant close it.
To Reproduce
Steps to reproduce the behavior:

watch video
wait a while
ad try's to play
popup appears

Expected behavior
popup doesn't show/ can close it
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 120.0.6099.110 (Official Build) (64-bit)

I figure this is the same as what I'm having.

The last "modal" popup window gets painted at weird places. Maybe the element wasn't properly dismissed so that it could be deleted? 🤷
Refreshing the page clears the DOM, but some new popup will get "stranded in limbo" soon enough! 😆


Add in some manual gc.collect()/torch.cuda.empty_cache() to the nodes

I've tried various different comfyui settings like --lowvram and with or without the keep_model_loaded true/false and it seems like a chunk of vram stays permanently tied up. It's probably a comfyui thing.
I added:
if not keep_model_loaded:
    print("Offloading Lumina model...")
    model.to(offload_device)
    gc.collect()
    torch.cuda.empty_cache()

to the LuminaGemmaTextEncode and LuminaGemmaTextEncode classes, within nodes.py, and now they seem to work fine and properly flush the vram
EDIT: oh and you might need to import gc but I'd assume it might be loaded from somewhere else already.
Also, you might want to add some kind of cleanup like this for in the event that a user cancels the sampler early. It doesn't seem to free up the space again until you rerun it, which might cause issues on the gemma sampling step if a large chunk of vram is now being held up by the cache still in vram.

I don't really know how to handle the case of the cancel, comfy automatic model management seems to require using the comfy model patcher object and I don't know how to wrap custom models like this to it.
The rest I added in latest update.


Improvements to minimal flattening

fixes #1621
fixes #1614
improves #1602 and #1614 (flatten consistent with validate regarding pointers to responses/parameters as schemas)

Carry out vendor update, including:

go-openapi/swag (comparing floats)
mailru/easyjson

Stumbled on yet another bug/regression. This one will have to wait another fix to pass.

all fixes applied in go-openapi dependant packages

codecov/patch non-significant here

Where is appveyor check gone? Why does this new licensing check never complete?

Looks like appveyor is run but the CI report is dropped


Updated allowed lists. (#3687)

Stack:

#3709
#3708
#3707 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.


Add repeating reminders

It would be awesome to have repeated reminders  e.g. remind every monday, second day, once per month or once per year.
It is usefull for stuff you have to do more than one time. Then you do not need to crate a reminder every day.
Love the app, thanks!

This is probably the only feature missing that keeps me ising Tasks Astrid clone instead of this app full time.
I hope this is on the roadmap.


mapproxy: 3.0.1 → 3.1.0

https://github.com/mapproxy/mapproxy/releases/tag/3.1.0
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354957

x86_64-darwin

:white_check_mark: 2 packages built:

mapproxy
mapproxy.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354957

x86_64-linux

:white_check_mark: 2 packages built:

mapproxy
mapproxy.dist


Can't submit jobs back-to-back with same input.star file

Can't submit jobs to AWS in successive back to back submissions - you get an error that is related to the data not being downloaded correctly on the AWS instance from S3.

User had the example line missing columns. Added a col number check into script.

I think this was related to the cidr log files created by launch_AWS_instance.py. I've updated to the code to use timestamps for the cidr log files, and this has appeared to fix the problem.


activityStreamsEmailNotifications check in cronjob.yaml

Currently, in the readme, it says that activityStreamsEmailNotifications is a string, and it is also like that in the default values.yaml. However, the {{- if .Values.ckan.activityStreamsEmailNotifications -}} will also return true if you change its value to "false", so maybe it should be a boolean (or the check should be changed)?

@matthiasdg the check has been modified to check for "true" in #15.
Note that email notifications are broken in 2.9 and we are working on a fix.

Thanks! I didn't need the email notifications and noticed the job crashing, so wanted to turn them off, that's how I discovered the faulty check :smile:

Ok then! Closing as done.

@matthiasdg in case you will need email notifications, they are now fixed for 2.9 with this PR #16


Rotating the textures on an illusion block turns them inside out.

This applies to all forms of illusion blocks, every rotation value other than 0 causes the block to appear inside out.

Seems like something broke, when I tried to fix the model coordinates in one of the latest updates... Thanks for reporting!
Regards, Manu


Remove api v1

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue

This PR removes API version 1 entirely, including documentation and tests.
Which Traffic Control components are affected by this PR?

Documentation
Traffic Ops Client (Go)
Traffic Ops
CI tests

What is the best way to verify this PR?
verify existing API tests pass for all versions greater than 1, make a request to APIv1 and verify it doesn't succeed.
The following criteria are ALL met by this PR

[x] Tests are unnecessary
[x] This PR includes documentation
[x] This PR includes an update to CHANGELOG.md
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

This can't be done until #5541 is closed

The tests all pass now, but we should not merge it until #5377 is done.

I made this not a draft because someone had told me that was done... I'll investigate to see if #5377 can be closed

Usages in cache-config/ are in GoDoc comments - it looks safe to change them since the information they're conveying isn't actually specific to that API version. I can do that if you want, but it shouldn't affect anything.
Usage in docs example has been updated
Usage in the ORT.py package is irrelevant, since that is due for removal. Pointless to change now, since it can't possibly work now that atstccfg has been removed, and whatever I do will be deleted anyway.
Usage in Grove is in a comment that wouldn't even appear in package documentation. I can get rid of it if you want, but it's not affecting anything.
I don't tend to touch anything in infrastructure/docker (except in the build subdirectory) because those Dockerfiles are archaic, they're parsing that API output in a way that may not be trivial (e.g. getting/setting server IP addresses), and their use is nowhere recommended or even documented. I don't even know if those work. I can change it, but it might break things in weird and unexpected ways, which doesn't seem worth it for something we don't even really support.
Usage in lib/go-tc is in GoDoc comments, which I can change if you want.
Usage in load-test.jsx is not referring to a Traffic Ops host - you can tell because it's passing a hostname for a Traffic Ops instance as a query parameter. test/router/server.go is the server serving that API, so the only real change necessary should be in there. But on the other hand - that test suite doesn't work. All browsers I've tried (Firefox, Chrome, Opera, all at latest versions) will refuse to load the .jsx file on the grounds of CORS request failure. If those tests are intended to work, they'll need to be reworked anyway, so the changes could just take place then.
cfg_test.pl is a script that, I believe, has not worked for longer than your involvement in the project. Possibly longer than mine. It was replaced by the compare tool, which has since been removed because it no longer serves any purpose. This script should not exist.
Usage in restapi.py is for a generic API, an API base URL containing an api/1 segment is perfectly valid in that context. In tosession.py it's in a comment on an example - which itself works afaik - and doesn't appear in any documentation.
Usage in copy-pgdata.sh has been updated. I thought that was for migrating from a mysql database to postgresql, but the comment header implies otherwise. Regardless, the usages are trivial enough to change easily.
Usage in traffic_ops/testing/ is in non-documenting comments, which I can change if you want but it's not affecting anything.
Usage in traffic_ops/traffic_ops_golang/ in GoDoc comments has been updated, as it was incorrect - except for one instance, which is still technically correct. One usage as a hard-coded value in a response header and one usage in a test have both been updated. The usages in the routing test are specifically checking for APIv1 not existing, and so should remain. The URL used in the DS Stats test is arbitrary, I can change it, but it won't be any more or less valid; only the headers on the request actually matter.
Swagger is not actually supported by Traffic Ops. The traffic_ops/traffic_ops_golang/swaggerdocs/ directory and any and all sub-packages thereof should be deleted, in my opinion. Unless someone is going to make them work, in which case they ought to update them to be correct themselves. But I don't think that's actually possible, in general, without significant code changes and some way to integrate the output with our existing documentation system (which doesn't exist afaik).
Usage in TP was previously removed, but must have been re-introduced during rebase with the commit that added/updated the stable API version.

Usages in cache-config/ are in GoDoc comments

Usage in lib/go-tc is in GoDoc comments

In tosession.py it's in a comment on an example - which itself works afaik - and doesn't appear in any documentation.

Usage in traffic_ops/testing/ is in non-documenting comments

The URL used in the DS Stats test is arbitrary

Updating all of these makes sense to me, IMO.

cfg_test.pl is a script that, I believe, has not worked for longer than your involvement in the project. Possibly longer than mine. It was replaced by the compare tool, which has since been removed because it no longer serves any purpose. This script should not exist.

Sounds like it could have been removed in the Perlectomy. Removing it with the rest of API v1 is the next best thing, right?

Usage in Grove is in a comment that wouldn't even appear in package documentation. I can get rid of it if you want, but it's not affecting anything.

https://github.com/apache/trafficcontrol/blob/1c72915665192d9ffa4d08e5ba202e044b38c2be/grove/grovetccfg/grovetccfg.go#L195
I'm seeing more API v1 stuff lower down:
https://github.com/apache/trafficcontrol/blob/1c72915665192d9ffa4d08e5ba202e044b38c2be/grove/grovetccfg/grovetccfg.go#L232-L265
Removing the Grove API v1 stuff would make more sense in a separate PR.

Usage in the ORT.py package is irrelevant, since that is due for removal.

I don't tend to touch anything in infrastructure/docker (except in the build subdirectory) because those Dockerfiles are archaic, they're parsing that API output in a way that may not be trivial (e.g. getting/setting server IP addresses), and their use is nowhere recommended or even documented.

Usage in load-test.jsx is not referring to a Traffic Ops host - you can tell because it's passing a hostname for a Traffic Ops instance as a query parameter. test/router/server.go is the server serving that API, so the only real change necessary should be in there. But on the other hand - that test suite doesn't work. All browsers I've tried (Firefox, Chrome, Opera, all at latest versions) will refuse to load the .jsx file on the grounds of CORS request failure. If those tests are intended to work, they'll need to be reworked anyway, so the changes could just take place then.

Swagger is not actually supported by Traffic Ops. The traffic_ops/traffic_ops_golang/swaggerdocs/ directory and any and all sub-packages thereof should be deleted, in my opinion. Unless someone is going to make them work, in which case they ought to update them to be correct themselves.

Agreed that taking care of these outside of #5339 makes sense.

Oops, missed a response.

Usage in restapi.py is for a generic API, an API base URL containing an api/1 segment is perfectly valid in that context. In tosession.py it's in a comment on an example - which itself works afaik - and doesn't appear in any documentation. I can change it if you want, but it's not affecting anything.

https://github.com/apache/trafficcontrol/blob/b64fcd5bf675c73ea1d085ee40245fcbd92e0df5/traffic_control/clients/python/trafficops/restapi.py#L141
https://github.com/apache/trafficcontrol/blob/b64fcd5bf675c73ea1d085ee40245fcbd92e0df5/traffic_control/clients/python/trafficops/restapi.py#L147
Updating these makes sense to me, too.

The parameter documentation for api_base_path in restapi.py doesn't need to be updated. That is not referring to the Traffic Ops API.

The parameter documentation for api_base_path in restapi.py doesn't need to be updated. That is not referring to the Traffic Ops API.

You are referring to
https://github.com/apache/trafficcontrol/blob/b64fcd5bf675c73ea1d085ee40245fcbd92e0df5/traffic_control/clients/python/trafficops/restapi.py#L141-L148
?

Yes. RestApiSession is a class that abstracts making request to an arbitrary REST API, it is not specific to Traffic Ops in any way. TO-specific code is in tosession.py.
class RestApiSession(object):
	"""
	This class represents a login session with a generic REST API server. It provides base
	functionality inherited by :class:`TOSession`.
	"""
	...

I'm seeing more API v1 stuff lower down:

I don't know what I can do about that. The comment says to remove that block when the API is updated to 1.3, but we're moving past that here. Plus from what I can tell the information it's gathering is actually necessary for the utility to do its work, so idk how I can remove it without breaking everything. It seems like Grove may need to be refactored, only @rob05c could possibly know for sure.

The parameter documentation for api_base_path in restapi.py doesn't need to be updated. That is not referring to the Traffic Ops API.

Changing 1.2 to a version that Traffic Ops has never used still seems like an improvement, even if that means changing it to a version Traffic Ops will never use.

Changing 1.2 to a version that we have never removed from Traffic Ops still seems like an improvement, even if that means changing it to a version Traffic Ops will never use.

If we do change it, it should specifically be to a version Traffic Ops would never use. I don't want this thing that has nothing to do with Traffic Ops to be a part of the upkeep of eliminating a Traffic Ops API version that happens to have path segments that look similar to it. So if we're going to change it, it should be to a version that isn't a valid TO API version (getting rid of it wouldn't be good, it's including an example that shows an API version in the base path segment in addition to an example that doesn't so that the reader understands that they may or may not include a version in the base path at their discretion) like, say, /api/v1.2?

[I]f we're going to change it, it should be to a version that isn't a valid TO API version (getting rid of it wouldn't be good, it's including an example that shows an API version in the base path segment in addition to an example that doesn't so that the reader understands that they may or may not include a version in the base path at their discretion) like, say, /api/v1.2?

IMO it still looks similar to a TO API version. /api/latest?

I can't change those without making breaking changes to structures that are still in use after the removal of APIv1

The traffic_router/core/src/test/resources/api/1.3 directory can be removed.