Poor performance

Same https://github.com/born05/craft-assetusage/issues/32
We will deactivate this plugin because we have timeouts and the asset list doesn't load when the plugin is activate.
Enabling the plugin and switching on the usage columns causes the asset index screens to load very slowly.
Craft CMS 4.4.6 | Asset Usage 3.1.0 | PHP version 8.2.2

Found the same thing, massive timeouts with some folder that make this plugin unusable, sorry to say!

Same here. As soon as the usage column is enabled, a folder with <100 files goes from a couple hundred miliseconds to multiple seconds load time.
@roelvanhintum I had a look how the count is implemented, pretty sure this is the culprit:
https://github.com/born05/craft-assetusage/blob/craft4/src/services/Asset.php#L30-L39
The additional call to Craft::$app->elements->getElementById means that every single element related to any asset in the list is queried and loaded individually.
The service uses a direct query against the relations table. Maybe instead of filtering the results after this query, the function can join the elements table to the initial query? This one has the canonicalId, draftId and revisionId fields, should be able to exclude revisions this way, right?

Same here, on sites with a lots of content, it really makes the CMS unusable. We ran into performance issues because of running a LIKE '%__%' with no index.


Clarify project setup steps

We had a learner in Reactiflux who accidentally deleted the entire my-project/src/ folder because they mis-typed the "delete all files in src" step.  I tweaked the instructions to hopefully make it less likely that others would do that in the future.

This is how it looks btw.

To avoid this kind of weird formatting please check deployment previews. :-) In this case, the fix is to add newlines after "Note"

(The change is good though. I just want to make sure we're mindful of formatting)


version identification

We need to identify the version installed, so that updates are possible later. Its just good practice anyway to do this.
With a golang app it would be:
go build -ldflags="-X 'github.com/dawenxi-tech/2fa/commands.Version=$(git describe --tags)'" -o "2fa-$(git describe --tags)-windows-amd64.exe"

For gio, I think its possible too but need to work it out and do it in the Makefile.
This will setup the basis for releases and updates to Desktop versions.

screen shot showing how you can open a GUI to see things like version, logs , etc if we want to also do something like that later.
Logs would be really useful actually.

am closing this as its done.
will make a new issue for showing the version in the GUI.


Jupyter interactive plots are very slow / laggy when opened in separate tab (e.g seaborn pairplot)

https://github.com/microsoft/vscode/issues/101238#issue-646955985

@sws144 Thanks for reporting this, and sorry about the issue. Do you have a code example that you could share that exhibits this behavior?  I do know that we have issues with generating very large file sizes with dense plots since we generate both SVG and PNG versions of the plot. The SVG versions can get very large causing some performance issues. You can skip generating SVG versions with a setting, but turning off that setting turns of the plot viewer since that uses SVGs for display.
https://github.com/microsoft/vscode-python/issues/12388

Here are the two files to run (up until row 135 for the sns.pairplot). This is a fairly big pairplot, but renders plot viewer unusable.
https://github.com/sws144/titanic-model-app/blob/master/titanic-logistic-backup.py
Source data file to create plots:
https://github.com/sws144/titanic-model-app/blob/master/titanic_train.csv
Perhaps we can allow PNG versions of the plot viewer as well? Thank you!

I am encountering this issue as well on RedHat Linux 7, even with a fairly beefy computer (albeit moderate CPU clock speeds).


Skip examples

Hi. Are there any Skip examples out there? Programs which have:

FFI: JS/native interop
networked code
reading/writing files (is bundler app the best one?)

You are right that those are missing. Thanks for flagging.

The only example app is the bundler, but note that the compiler, standard library, and many of the tools are all written in Skip. I’d encourage you to poke around the src/ directory to see more skip code (src/project/ is the project system and demonstrates reading files - also see the Filesystem module in stdlib).
Note that we dont have support for user defined FFI, and don’t have networking APIs in stdlib yet. Those are definitely things we’d like to support though.

Is there no network api yet?


Improve punctuation

:+1:

:-1: i'm afraid, because you've changed application wide to application-wide only in one place, which is inconsistent.

I've fixed the typo everywhere, since the punctuation changes need to be discussed separately. https://github.com/kraih/mojo/commit/8ea4618cf9a6c7c717b4062d282b353f6d2af086

We've talked about this on IRC, and there's many different opinions on how punctuation should look in that sentence. Not sure how to settle this.

No problem!  Usually, FWIW, that's a sign (different opinions) that reducing the sentence to shorter and more sentences is in order.  Longer sentences can often be more efficient and clever, but readability (I think specifically in the case of this project) is likely the number one goal.  That might be just something worth keeping in mind, not necessarily striving for.
For uninterrupted clear readability, I do believe that "for example" should be offset with commas.  I think this is a case of "misplaced modifier".
"it allows you for example to customize" -> "it allows you to, for example, customize"


feat: azure publish plugin

Description

Task Item
close #2461
close #2757
Screenshots

@VanyLaw We need to simplify this in a few ways.
do not provision in the plugin
The provisioning part (botProjectDeploy.create) should not be part of the Composer publish action. This should happen outside of the plugin, via a command line tool.  We need to figure out how to make this an easy to use script -- possibly bundle it with the ejected code.
Ideally, we would be able to do something like this:
provisionComposer --name <bot> --location westus --environment dev --appPassword <password> --luisAuthoringKey --<key> --token <token> --subscription <subid>

This would perform the provision, and after completion, output a configuration profile, like:
Your Azure hosting environment has been created! Copy paste the following configuration into a new profile in Composer's Publishing tab.

{
accessToken: <token>,
subscription: <subid>,
environment: <environment>,
luisAuthoringKey: <key>,
luisAuthoringRegion: <region>
}

make sure the plugin does not expect the declarative assets to be local
Right now, the plugin is doing a simple recursive copy of the files from the bot project's source folder.  However, this will not work for all storage providers.
Instead, you should use the project.files array to access the file content and write them to the appropriate location.
Consider alternative mechanisms for building/deploying
Right now, the plugin copies the declarative assets from the bot project along with the c sharp code project from the template into a single TEMP folder.  Inside this temp folder, it updates an appsettings.deployment.settings file after provisioning, and then uses that for the deploy.
There are a number of flaws in this approach, though a big one is that this requires provisioning to occur immediately before deploying and will only work for one project at a time, and only work once since provisioning status is held in the plugin's memory and not serialized anywhere.

where should information about the provisioned resources live? in the appsettings.json file? In the publishing profile?
How will this work with an ejected runtime?
When using an ejected runtime, should the build occur in a temp folder or should it occur in the normal project folder?

Error handling during provisioning
Right now, a lot of the errors just return and cause no error to be raised. These critical errors should probably throw an error and cause the promise to reject so that they can be handled properly.

@benbrown Can't agree more. I did some update today.

To handle the token expire more user-friendly, I split the login into a js script which return a credential. user can simpliy run this before start composer. And no need to update the profile even the token expired. The credential will be read before composer start and refresh in memory.
Instead of the a single TEMP folder, I used the hask key of profile config(name, location,env,````) as a folder name to save those declarative assets and runtime. and add the field create to defined provision or not. If user use the same config to publish different bots, it will only replace the    composerDialogs folder in it and use the same config and appsettings.Deployment.json to publish.

I will keep doing the rest and move the provision out of the plugin next.

@benbrown  Hi Ben, I fix the comments about use the project.files instead of dataDir. And I know we can simply implement the pluginLoader addRuntimeTemplate function to support eject, but I don't know where is the link we can click to eject the deploy runtime?  Is it the customize runtime page is only for local publish plugin?

@benbrown Hi Ben, I fix the comments about use the project.files instead of dataDir. And I know we can simply implement the pluginLoader addRuntimeTemplate function to support eject, but I don't know where is the link we can click to eject the deploy runtime? Is it the customize runtime page is only for local publish plugin?

The runtime eject page is now in master - it is part of the settings.

@VanyLaw I updated the provisionComposer.js file to format the JSON response and also include the accessToken as well as the full credential object.
I have several pieces of feedback on the current state of things:

As discussed, the publish profile should accept the access token and use this to do the zip deploy.  I was unable to figure out how to do this - do you know? Is it possible?
The current code uses the WebsiteManagementClient class to change usernames and passwords using a predictable pattern, which seems very insecure and probably a bad practice since that deploy user might be used for other purposes.  We have to find a way to do this that does not involve changing user credentials.  Hopefully, by using the access token.
After some consideration, we have decided to go with the Node versions of the provisionComposer script and login script.  We will copy these into the project at create time as part of the boilerplate material -- I still need to implement that in #2451 -- but for now it would be good to get the necessary class out of the @bfc/bot-deploy package and made available locally to that provisionComposer script.  That is, we should get that stand-alone javascript package ready.
we should add to the schema all the fields that will be created by a successful provision instead of leaving it defined as a generic object

@VanyLaw small update - my latest change to this branch switches the zipdeploy to using the bearer token. It works!
However, it is still using the large profile.credentials blob to generate this instead of using an accesstoken in the profile instead.  Can you please make this change?  This will likely require a bit of refactoring to remove expectation that this.creds exists inside the botDeploy class. As I mentioned in the previous comment, we need to split the DEPLOY piece from the PROVISION piece anyways.

@VanyLaw this is looking pretty good.
A few things:

[ ] When the token has expired, the first error it hits is a 400 error from LUIS. We should handle this and present a nice error message that tells the user to use az account get-access-token
[ ] In provisionComposer: There is a particular error that happens when the provision fails because your subscription is full. It comes back as an InvalidTemplateDeployment.   Perhaps we can provide a better error message, or at least print out the details array which may have info in it.

@VanyLaw My progress report today:

I was able to repeatedly provision and publish successfully! Hooray!
I moved the location of the provision scripts into /assets/shared.  These get copied into every new bot project on create.  I also added a README file with instructions on using it, copying the profile, and refreshing the token.
I made a change so that the azurePublish plugin will now clean up the publishBots folder after it completes.
I made a change so that if the bot project has a customized runtime path, it will copy the runtime from this path instead of the default template (support ejected runtime)
I turned off writing to publishHistory.txt by default. I think it is unsafe to write this history because it could contain sensitive information. The history is kept in memory for now.

A few things I could use your help on:

[ ] If I do NOT have LUIS configured, the first error about the invalid token comes out of the zipDeploy step.  Please catch that error and make sure it responds properly. It currently responds with a VERY VERBOSE error with a lot of binary data in it that is impossible to read.
[ ] In the provision script, we should default to provisioning ALL the resources. Currently these default to false, but they should default to true.
[ ] The result of the provision includes the app password field TWICE.  This should only be present once.
[ ] If I disable certain resources during the provision, the resulting profile should exclude those keys.  Currently it includes the keys with blank values.  For example, if I do not provision cosmosDB, there should be no cosmosDB key in the config.


issue #1071 fixed

🛠 Fixes Issue #1071
👨💻 Changes proposed
Fixed the bug of home page which was preventing the maze image to load.
✅ Check List (Check all the applicable boxes)

[x] You have linked this PR to the correct issue.
[x] My code doesn't break any part of the project (Zero Octave-Javascript-Projects).
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

@Astrodevil hey is there something wrong? Can u tell me why my pull request was closed? Please let me know ,so that i can do better next time


docs: Update README.md

added https://reporoster.com/ contributor acknowledgment/shout out

Which problem is this PR solving?

Short description of the changes

providing acknowledgment to open source contributors

Tbh I don't think that providing this information to readme will give us any benefit as this info is already available through buttons on top right corner anyway.

Tbh I don't think that providing this information to readme will give us any benefit as this info is already available through buttons on top right corner anyway.

Agree

I also don't really want to link out to an external site for functionality already easily available on github. Based on @obecny comment and @vmarchaud thumbs up on it, I think maintainers are united on this and I'm closing this as won't do.


roch_simulator: 1.0.7-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository roch_simulator to 1.0.7-0:

upstream repository: https://github.com/SawYer-Robotics/roch_simulator.git
release repository: https://github.com/SawYerRobotics-release/roch_simulator-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.23
previous version for package: null

roch_gazebo
* Modify some websites to correct.

roch_simulator
* Modify some websites to correct.

I just merged the PR for version 1.0.6 therefore this conflicts now. Please close obsolete PRs yourself.


Initial working implementation of generator

@hanwavefront I think I've resolved all the issues you identified, please take another look when you have a moment.


How to get object features from each detection.

Is there a convenient way to get the object features used for classification (clip embeddings), as outputs along with the bounding boxes?

Hi,
The features are here. It may need some changes in code to pass it to the outputs.
Best,
Xingyi

Thanks. I am trying to make changes to the code to pass it along with other outputs like bounding boxes and class predictions. But I am unsure of one section of the code. In the inference mode, inside detic_roi_heads.py, I am not clear what is really happening. While having outputs from different stages (head_outputs), the predictor, predictions and proposals are only considered from the finals stage (head_outputs[-1]). Would be glad if you can clarify that part. Thanks again!


Fix code for getting top features

Change the index used in feature_selection method for getting the top feature_weights. data shape is (n_samples, n_features). Currently used index is 0 which points to the n_samples.

Thanks. It still worked because ZIP truncates to longest list to the shortest list's size, but definitely a bug.


Describe hie in the digest of Gabriel Gonzales (author of pipes etc)

https://github.com/Gabriel439/post-rfc/pull/126
@alanz @wz1000
I don't know the names/urls of hie servers for atom/sublime/etc

ok it's largely done (by me)


Error in latest docker image: SELECT id FROM users

I am getting an error with the latest docker image and the logs say it's having problem with running queries such as SELECT id FROM users. The latest docker image was published earlier today

Here is the full error:
Traceback (most recent call last):                                                                        │
│     File "/code/run.py", line 6, in                                                                 │
│       from func.interactions import *                                                                       │
│     File "/code/func/interactions.py", line 68, in                                                  │
│       allowed_ids = load_allowed_ids_from_db()                                                              │
│                     ^^^^^^^^^^^^^^^^^^^^^^^^^^                                                              │
│     File "/code/func/interactions.py", line 63, in load_allowed_ids_from_db                                 │
│       c.execute("SELECT id FROM users")                                                                     │
│   sqlite3.OperationalError: no such table: users

I just tried it and this came up. Same error.

I just executed part where it creates db manually:
import sqlite3
def init_db():
    conn = sqlite3.connect('users.db')
    c = conn.cursor()
    c.execute('''CREATE TABLE IF NOT EXISTS users
                 (id INTEGER PRIMARY KEY, name TEXT)''')
    c.execute('''CREATE TABLE IF NOT EXISTS chats
                 (id INTEGER PRIMARY KEY AUTOINCREMENT,
                  user_id INTEGER,
                  role TEXT,
                  content TEXT,
                  timestamp DATETIME DEFAULT CURRENT_TIMESTAMP,
                  FOREIGN KEY (user_id) REFERENCES users(id))''')
    conn.commit()
    conn.close()

init_db()

I executed part where it creates db manually, and it started normally after:
import sqlite3
def init_db():
    conn = sqlite3.connect('users.db')
    c = conn.cursor()
    c.execute('''CREATE TABLE IF NOT EXISTS users
                 (id INTEGER PRIMARY KEY, name TEXT)''')
    c.execute('''CREATE TABLE IF NOT EXISTS chats
                 (id INTEGER PRIMARY KEY AUTOINCREMENT,
                  user_id INTEGER,
                  role TEXT,
                  content TEXT,
                  timestamp DATETIME DEFAULT CURRENT_TIMESTAMP,
                  FOREIGN KEY (user_id) REFERENCES users(id))''')
    conn.commit()
    conn.close()

init_db()

Thank you! Worked for me

I am using the docker image, so will this be included in the docker build?

@zahidhanif it's included, but it doesn't run for some reason.

Clone Repository
Copy this code  from run.py:

    c = conn.cursor()
    c.execute('''CREATE TABLE IF NOT EXISTS users
                 (id INTEGER PRIMARY KEY AUTOINCREMENT, name TEXT)''')
    c.execute('''CREATE TABLE IF NOT EXISTS chats
                 (id INTEGER PRIMARY KEY AUTOINCREMENT,
                  user_id INTEGER,
                  role TEXT,
                  content TEXT,
                  timestamp DATETIME DEFAULT CURRENT_TIMESTAMP,
                  FOREIGN KEY (user_id) REFERENCES users(id))''')
    conn.commit()

to interactions.py after:
def load_allowed_ids_from_db():
    conn = sqlite3.connect('users.db')

Build and run docker compose up --build -d

I've mapped the users.db volume in yml and made my own empty db, it worked that way.

I've mapped the users.db volume in yml and made my own empty db, it worked that way.

Can you please send me an example?

Here's the empty users.db (just extract it to bot folder):
users.zip
Docker compose I'm using:
version: '3.8'
services:
  ollama-telegram:
    container_name: ollama-telegram
    image: ruecat/ollama-telegram
    restart: on-failure
    env_file:
      - ./.env
    volumes:
      - ./users.db:/code/users.db

  ollama-server:
    image: ollama/ollama
    container_name: ollama-server
    volumes:
      - ./ollama:/root/.ollama
    restart: always
    ports:
      - '11434:11434'

Once started, I had to execute ollama pull on the ollama-server container like this:
docker exec -it [container_id] ollama pull llama3.2:1b

replace [container_id] with container id from docker ps command of the ollama-server.
And the .env file looks like this:
TOKEN=
ADMIN_IDS=
USER_IDS=
ALLOW_ALL_USERS_IN_GROUPS=1
INITMODEL=llama3.2:1b
TIMEOUT=3000

# UNCOMMENT ONE OF THE FOLLOWING LINES:
# OLLAMA_BASE_URL=localhost # to run ollama without docker, using run.py
OLLAMA_BASE_URL=ollama-server # to run ollama in a docker container
# OLLAMA_BASE_URL=host.docker.internal # to run ollama locally

OLLAMA_HOST=${OLLAMA_BASE_URL}

# Log level
# https://docs.python.org/3/library/logging.html#logging-levels
LOG_LEVEL=DEBUG

Here's the empty users.db (just extract it to root folder of the project): users.zip
Docker compose I'm using:
version: '3.8'
services:
  ollama-telegram:
    container_name: ollama-telegram
    image: ruecat/ollama-telegram
    restart: on-failure
    env_file:
      - ./.env
    volumes:
      - ./users.db:/code/users.db

  ollama-server:
    image: ollama/ollama
    container_name: ollama-server
    volumes:
      - ./ollama:/root/.ollama
    restart: always
    ports:
      - '11434:11434'

Once started, I had to execute ollama pull on the ollama-server container like this:
docker exec -it [container_id] ollama pull llama3.2:1b

replace [container_id] with container id from docker ps command of the ollama-server.
And the .env file looks like this:
TOKEN=
ADMIN_IDS=
USER_IDS=
ALLOW_ALL_USERS_IN_GROUPS=1
INITMODEL=llama3.2:1b
TIMEOUT=3000

# UNCOMMENT ONE OF THE FOLLOWING LINES:
# OLLAMA_BASE_URL=localhost # to run ollama without docker, using run.py
OLLAMA_BASE_URL=ollama-server # to run ollama in a docker container
# OLLAMA_BASE_URL=host.docker.internal # to run ollama locally

OLLAMA_HOST=${OLLAMA_BASE_URL}

# Log level
# https://docs.python.org/3/library/logging.html#logging-levels
LOG_LEVEL=DEBUG

Thanks, that's worked for me

hm..  i'm still getting this after the latest update:
Traceback (most recent call last):   File "/code/run.py", line 6, in <module>     from func.interactions import *   File "/code/func/interactions.py", line 68, in <module>     allowed_ids = load_allowed_ids_from_db()                   ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/code/func/interactions.py", line 63, in load_allowed_ids_from_db     c.execute("SELECT id FROM users") sqlite3.OperationalError: no such table: users
mapping the db-file just change the error message to something like "can't read users.db"..

@hhenne1 sounds like permission issue to me. Does docker user has rw access to that file?

oh yea.. im dumb.. my bad, sry.. had a typo in the mount path.. should have realized that docker was mounting an empty folder instead of the file..

May be related to the comment I left in https://github.com/ruecat/ollama-telegram/pull/61/commits/d3fb767f384b682737ee7cfb537f90bc980d0a47

Unfortunately, the only existing Docker image is tagged latest (https://hub.docker.com/r/ruecat/ollama-telegram/tags?ordering=-last_updated), so there is no possibility to rollback.

I just tried to install this from scratch... I'm still getting the errors:
$ docker-compose logs -f
ollama-telegram  | Traceback (most recent call last):
ollama-telegram  |   File "/code/run.py", line 6, in <module>
ollama-telegram  |     from func.interactions import *
ollama-telegram  |   File "/code/func/interactions.py", line 68, in <module>
ollama-telegram  |     allowed_ids = load_allowed_ids_from_db()
ollama-telegram  |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^
ollama-telegram  |   File "/code/func/interactions.py", line 63, in load_allowed_ids_from_db
ollama-telegram  |     c.execute("SELECT id FROM users")
ollama-telegram  | sqlite3.OperationalError: no such table: users
ollama-telegram  | Traceback (most recent call last):
ollama-telegram  |   File "/code/run.py", line 6, in <module>
ollama-telegram  |     from func.interactions import *
ollama-telegram  |   File "/code/func/interactions.py", line 68, in <module>
ollama-telegram  |     allowed_ids = load_allowed_ids_from_db()
ollama-telegram  |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^
ollama-telegram  |   File "/code/func/interactions.py", line 63, in load_allowed_ids_from_db
ollama-telegram  |     c.execute("SELECT id FROM users")
ollama-telegram  | sqlite3.OperationalError: no such table: users
ollama-telegram  | Traceback (most recent call last):
ollama-telegram  |   File "/code/run.py", line 6, in <module>
ollama-telegram  |     from func.interactions import *
ollama-telegram  |   File "/code/func/interactions.py", line 68, in <module>
ollama-telegram  |     allowed_ids = load_allowed_ids_from_db()
ollama-telegram  |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^
ollama-telegram  |   File "/code/func/interactions.py", line 63, in load_allowed_ids_from_db

Has the changes been pushed to docker hub?

Here is hotfix patch based on @fredix solution.
Use with git apply hotfix-db-users.patch.txt
hotfix-db-users.patch.txt


--grace 1 prevents screenlock

Problem
The --grace option doesn't work as expected. The screen doesn't lock. To me it seems like the white default lock screen pops up for a split second but it never actually locks or instantly unlocks itself again without asking for a pw
swaylock -C /dev/null --grace 0: screen locks as expected and I can unlock it by typing the password
swaylock -C /dev/null --grace 1 (or any val > 0): screen seems to lock for a split second and instantly unlocks itself without asking for pw
Any idea why this is happening? Sth with pam maybe?
Info
 ~> swaylock --version
swaylock version <IP_ADDRESS>
 ~> hyprctl version
Hyprland, built from branch main at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: 2024-08-07
Tag: v0.42.0, commits: 9a09eac79b85c846e3a865a9078a3f8ff65a9259

flags: (if any)

 ~> uname -a
Linux lt-t14 6.6.47 #1-NixOS SMP PREEMPT_DYNAMIC Mon Aug 19 04:04:32 UTC 2024 x86_64 GNU/Linux

Trace
 ~> swaylock -C /dev/null --trace --grace 0
2024-08-27 22:18:25 - [main.c:1870] Found config at /dev/null
2024-08-27 22:18:25 - [main.c:1880] Parsing CLI Args
2024-08-27 22:18:25 - [main.c:453]: trace: handle_wl_output_geometry
2024-08-27 22:18:25 - [main.c:657]: trace: handle_wl_output_name
2024-08-27 22:18:25 - [main.c:658] output name is eDP-1
2024-08-27 22:18:25 - [main.c:469]: trace: handle_wl_output_scale
2024-08-27 22:18:25 - [main.c:669]: trace: handle_wl_output_done
2024-08-27 22:18:25 - [main.c:453]: trace: handle_wl_output_geometry
2024-08-27 22:18:25 - [main.c:657]: trace: handle_wl_output_name
2024-08-27 22:18:25 - [main.c:658] output name is DP-4
2024-08-27 22:18:25 - [main.c:469]: trace: handle_wl_output_scale
2024-08-27 22:18:25 - [main.c:669]: trace: handle_wl_output_done
2024-08-27 22:18:25 - [main.c:453]: trace: handle_wl_output_geometry
2024-08-27 22:18:25 - [main.c:657]: trace: handle_wl_output_name
2024-08-27 22:18:25 - [main.c:658] output name is DP-5
2024-08-27 22:18:25 - [main.c:469]: trace: handle_wl_output_scale
2024-08-27 22:18:25 - [main.c:669]: trace: handle_wl_output_done
2024-08-27 22:18:25 - [main.c:535]: trace: handle_screencopy_frame_buffer
2024-08-27 22:18:25 - [main.c:535]: trace: handle_screencopy_frame_buffer
2024-08-27 22:18:25 - [main.c:535]: trace: handle_screencopy_frame_buffer
2024-08-27 22:18:25 - [main.c:562]: trace: handle_screencopy_frame_flags
2024-08-27 22:18:25 - [main.c:610]: trace: handle_screencopy_frame_ready
2024-08-27 22:18:25 - [main.c:562]: trace: handle_screencopy_frame_flags
2024-08-27 22:18:25 - [main.c:610]: trace: handle_screencopy_frame_ready
2024-08-27 22:18:25 - [main.c:562]: trace: handle_screencopy_frame_flags
2024-08-27 22:18:25 - [main.c:610]: trace: handle_screencopy_frame_ready
2024-08-27 22:18:25 - [main.c:1970] Using ext-session-lock-v1

 ~> swaylock -C /dev/null --trace --grace 1
2024-08-27 22:18:32 - [main.c:1870] Found config at /dev/null
2024-08-27 22:18:32 - [main.c:1880] Parsing CLI Args
2024-08-27 22:18:32 - [main.c:453]: trace: handle_wl_output_geometry
2024-08-27 22:18:32 - [main.c:657]: trace: handle_wl_output_name
2024-08-27 22:18:32 - [main.c:658] output name is eDP-1
2024-08-27 22:18:32 - [main.c:469]: trace: handle_wl_output_scale
2024-08-27 22:18:32 - [main.c:669]: trace: handle_wl_output_done
2024-08-27 22:18:32 - [main.c:453]: trace: handle_wl_output_geometry
2024-08-27 22:18:32 - [main.c:657]: trace: handle_wl_output_name
2024-08-27 22:18:32 - [main.c:658] output name is DP-4
2024-08-27 22:18:32 - [main.c:469]: trace: handle_wl_output_scale
2024-08-27 22:18:32 - [main.c:669]: trace: handle_wl_output_done
2024-08-27 22:18:32 - [main.c:453]: trace: handle_wl_output_geometry
2024-08-27 22:18:32 - [main.c:657]: trace: handle_wl_output_name
2024-08-27 22:18:32 - [main.c:658] output name is DP-5
2024-08-27 22:18:32 - [main.c:469]: trace: handle_wl_output_scale
2024-08-27 22:18:32 - [main.c:669]: trace: handle_wl_output_done
2024-08-27 22:18:32 - [main.c:535]: trace: handle_screencopy_frame_buffer
2024-08-27 22:18:32 - [main.c:535]: trace: handle_screencopy_frame_buffer
2024-08-27 22:18:32 - [main.c:535]: trace: handle_screencopy_frame_buffer
2024-08-27 22:18:32 - [main.c:562]: trace: handle_screencopy_frame_flags
2024-08-27 22:18:32 - [main.c:610]: trace: handle_screencopy_frame_ready
2024-08-27 22:18:32 - [main.c:562]: trace: handle_screencopy_frame_flags
2024-08-27 22:18:32 - [main.c:610]: trace: handle_screencopy_frame_ready
2024-08-27 22:18:32 - [main.c:562]: trace: handle_screencopy_frame_flags
2024-08-27 22:18:32 - [main.c:610]: trace: handle_screencopy_frame_ready
2024-08-27 22:18:32 - [main.c:1970] Using ext-session-lock-v1

Have the same issue, here is my system info
❯ swaylock --version
swaylock version v<IP_ADDRESS>-1-g496059a (" __DATE__ ", branch 'master')
❯ uname -a
Linux 6.10.6-arch1-1 #1 SMP PREEMPT_DYNAMIC Mon, 19 Aug 2024 17:02:39 +0000 x86_64 GNU/Linux
❯ hyprctl version
Hyprland, built from branch  at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: Wed Aug 7 19:17:10 2024
Tag: v0.42.0, commits: 5069

flags: (if any)

Logs
❯ swaylock -C /dev/null --trace --grace 1
2024-08-28 13:40:08 - [main.c:1870] Found config at /dev/null
2024-08-28 13:40:08 - [main.c:1880] Parsing CLI Args
2024-08-28 13:40:08 - [main.c:453]: trace: handle_wl_output_geometry
2024-08-28 13:40:08 - [main.c:657]: trace: handle_wl_output_name
2024-08-28 13:40:08 - [main.c:658] output name is eDP-1
2024-08-28 13:40:08 - [main.c:469]: trace: handle_wl_output_scale
2024-08-28 13:40:08 - [main.c:669]: trace: handle_wl_output_done
2024-08-28 13:40:08 - [main.c:535]: trace: handle_screencopy_frame_buffer
2024-08-28 13:40:08 - [main.c:562]: trace: handle_screencopy_frame_flags
2024-08-28 13:40:08 - [main.c:610]: trace: handle_screencopy_frame_ready
2024-08-28 13:40:08 - [main.c:1970] Using ext-session-lock-v1

I have the same issue. By adding --grace-no-mouse, the grace option works as intended, so it seems to have to do with that setting.

I added some debug prints into the swaylock_handle_mouse [1] function and it seems like this is called even if the mouse is not moved.
[1] https://github.com/jirutka/swaylock-effects/blob/496059a8565c2d5eed672c2e5bc5e1edd14b3de8/password.c#L96
This could work as a quick and dirty fix: only if the pointer is moved more than 10, the function to unlock during grace is called:
static void wl_pointer_motion(void *data, struct wl_pointer *wl_pointer,
		uint32_t time, wl_fixed_t surface_x, wl_fixed_t surface_y) {
		static int last_x = 0;
		static int last_y = 0;
		int x = wl_fixed_to_int(surface_x);
		int y = wl_fixed_to_int(surface_y);

		if (last_x > 0 && (x - last_x > 10 || y - last_y > 10)) {
		    swaylock_handle_mouse((struct swaylock_state *)data);
		}

		last_x = x;
		last_y = y;
}

The real fix would be to find out why the motion function is called even though the pointer is not moved.

Ok, after some fiddling with WAYLAND_DEBUG=1 I can pinpoint it to 2 function calls:
https://github.com/jirutka/swaylock-effects/blob/496059a8565c2d5eed672c2e5bc5e1edd14b3de8/main.c#L417
and
https://github.com/jirutka/swaylock-effects/blob/496059a8565c2d5eed672c2e5bc5e1edd14b3de8/main.c#L439
The first one is no wonder, because render_frame ultimately calls wl_surface_commit().
If I comment those, the motion function is not called periodically without pointer movement anymore. So the actual drawing of the surface by wayland triggers the motion event.
Maybe it is better to use the fix from above then changing the inner workings of swaylocks frame drawing.

This is a Hyprland issue, and I've managed to figure out the root cause there after much troubleshooting and debugging. I've made https://github.com/hyprwm/Hyprland/pull/8584 there which fixes the issue for me - if anyone could test it that would be much appreciated :)

Iirc in the PR you did for hyprland, swaylock still needs to check for the actual pointer movement like in my quickfix above because you did remove that one commit from your PR. Correct?

Iirc in the PR you did for hyprland, swaylock still needs to check for the actual pointer movement like in my quickfix above because you did remove that one commit from your PR. Correct?

Yep - I've made #70 for that.

an confirm that the combination of hyprwm/Hyprland#8584 and #70 resolves the issue. Only applying the hyprland patch wasnt enough

Awesome, thanks. I changed my Hyprland PR, so it doesn't fix the issue anymore. #70 should fix the issue with or without the Hyprland patch.


Added blog post links for litreview0 and vidreview0

@fortnight be sure to git pull and merge upstream changes

Whoops, I meant LitReview2

@fortnight make sure you git pull and merge upstream.

:8ball: :ok:


fix parsing of long reason phrases in CONNECTION_CLOSE frames

This is the same fix as I did in #883.

Codecov Report

Merging #889 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #889      +/-   ##
==========================================
- Coverage   90.74%   90.71%   -0.03%
==========================================
  Files          90       90
  Lines        6212     6212
==========================================
- Hits         5637     5635       -2
- Misses        371      372       +1
- Partials      204      205       +1

Impacted Files
Coverage Δ

internal/wire/connection_close_frame.go
93.33% <100%> (-6.67%)
:arrow_down:

Continue to review full report at Codecov.

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


Feature request: consider adding a license

Summary
Hi @ourzora! Would you be open to adding a license to this project? 💭
Without one, this project is under exclusive copyright by default and unless otherwise specified, nobody else can copy, distribute, or modify your work without being at risk of take-downs, shake-downs, or litigation. Once the work has other contributors (each a copyright holder), “nobody” starts including the @ourzora team. 😅
I humbly suggest the MIT License or GNU GPL v3.0 (like what's done for ourzora/v3) as an option.
If you're up for it and know which one y'all would like, I wouldn't mind cutting a PR for it too! ✌

great callout, added https://github.com/ourzora/offchain/blob/main/LICENCE


Plex package stopped occasionally

Every couple of days the Plex package is stopped after running this script every night.
I just log in a hit Run. Any way to make sure the script re-starts it if necessary?

Strange; I've been using the script for more than a year (nightly) and I don't recall ever experiencing this. Do the stoppages happen during the day or immediately after the auto-update cycle?
I'd suggest enabling logs on the Plex Server itself and looking into them when the server stops again. Additionally, I'd try disabling the script for some time to see if using it correlates with occurrences of the issue.
Another workaround might be to create another scheduled task that just runs synopkg start PlexMediaServer on a regular basis.

I'm gonna wait for the next update and see if that caused it. Thanks for all the tips!


kodiPlugins: import addons from kodi repositories

Motivation for this change
It is difficult to package kodi plugins because of the dependencies that most come with. This PR aims to improve the situation by automatically importing every plugin from the kodi repository that we reasonably can automatically import.
I'm not sure if this PR is a good idea or not:

this adds over 1,000 packages to nixpkgs
not all kodi plugins can be packaged for nixpkgs because some plugins require read/write access to other plugins source code at runtime... see kodiPlugins.skin-arctic-zephyr-mod which I have packages as an example as such (hint: the plugin crashes on startup)
plenty of other reasons :man_shrugging:

On the plus side this allows you to create an instance of kodi with plugins that won't get mangled by other kodi plugins, and can avoid the headache of having to manually create custom kodi builds, which I'm in the habit of doing every 6 months or so. In other words this PR has high value to me... but maybe only me :man_shrugging:
This PR is heavily work in progress and I'm looking for feedback on whether this even belongs in nixpkgs or not
ping @spiderbit @tyrion
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

this adds over 1,000 packages to nixpkgs

Would it be a smarter idea to manually add and verify the plugins people want to use?

Yeah I was considering selectively adding plugins as needed... it might be challenging to teach the @r-ryantm bot to update them, though (I don't know haskell to add the functionality).

Yeah I was considering selectively adding plugins as needed... it might be challenging to teach the @r-ryantm bot to update them, though (I don't know haskell to add the functionality).

r-ryantm respects update scripts so if you can write a script in bash, python or maybe perl the bot can update it. There is an update script in the maintainers directory which I think the bot uses.

r-ryantm respects update scripts so if you can write a script in bash, python or maybe perl the bot can update it. There is an update script in the maintainers directory which I think the bot uses.

I didn't know that. I'll give my approach some thought. Thanks.


Add a tsconfig.json file to project

More information here.

Also find example in Interval Resizer:
interval-resizer/tsconfig.json

One more resource:
https://github.com/parcel-bundler/parcel/issues/168

Schema:
http://json.schemastore.org/tsconfig


More descriptive error on Client.Timeout

When I'm on spotty wifi / poor internet (such as where I am currently in northeastern iceland), I often get client timeout errors on single files when performing a download.
The error message currently looks like this:

Would be great if it just dropped the name of the file it errored on, then I could acquire it manually more easily.  Realize this is a silly use case in some respects, but I travel a lot while working for clients 😸

Nevermind, realized I can jack the timeout up on my end... so needed to RTFM.


Handle already existing release hash

As discussed in https://github.com/40thieves/webpack-sentry-plugin/issues/16#issuecomment-289450544, currently creation of releases will crash if the release hash stays the same.
Potential options:

Crash with better error message
Overwrite the existing release's files (how does Sentry api handle this?)
Delete the existing release and try again

We just ran into this after a failed deploy. It looks like the Sentry API returns a 409 - Conflict status code when this happens. I'm not sure what the best thing to do is here, but an option to ignore this type of error (and not others) would solve our issue. The current suppressErrors option swallows this error, and others that we would want to raise on.

Similarly, we have a use case where we would like to tie multiple development builds to a specific development 'release' and use this 'release' to bundle sentry caught exceptions.  This is difficult when we can't harmlessly register the same 'release'.  +1 for an ignore duplicate release error flag.


GRPC/http2

What is the problem your feature solves, or the need it fulfills?
I'm trying to create a proxy for GRPC, but I don't know how.  So I created an HTTP proxy to use http2 and followed the examples and I'm receiving an error of invalid HTTP version. Would someone be able to help me with an example? Maybe It needs a different configuration...
Describe the solution you'd like
Maybe a new example and docs will solve the problem.
Additional context
Log from Pingora proxy
ERROR pingora_proxy: Fail to proxy: Downstream InvalidHTTPHeader context: buf: "PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n\0\0\u{12}\u{4}\0\0\0\0\0\0\u{2}\0\0\0\0\0\u{4}\0 \0\0\0\u{5}\0\0@\0\0\0\u{4}\u{8}\0\0\0\0\0\0O\0\u{1}" cause: invalid HTTP version
Log from GPRC client using tonic
Error: Status { code: Unknown, message: "h2 protocol error: http2 error: connection error detected: frame with invalid size", source: Some(tonic::transport::Error(Transport, hyper::Error(Http2, Error { kind: GoAway(b"", FRAME_SIZE_ERROR, Library) }))) }

Is the grpc service https or h2c ?

Is the grpc service https or h2c?

Hi, thank you for your answer. I'm trying to create a grpc service https. But can you explain about h2c and what is the difference between them?

HTTP/2 over TCP: https://httpwg.org/specs/rfc7540.html#rfc.section.3.1

H2C is not supported, #66

@vicanso Does it mean Pingora doesn't support grpc?

Do you have an example using Pingora?

I think gRPC proxying requires your service to support tls.

tls: https://github.com/hyperium/tonic/blob/master/examples/src/tls/server.rs#L39
h2c: https://github.com/hyperium/tonic/blob/master/examples/src/h2c/server.rs#L42

@vicanso but in those cases, I need to use the proto files in the proxy as well and I wouldn't like it... Do you know a way?

Do you want a grpc gateway like: https://github.com/grpc-ecosystem/grpc-gateway ?
I thinks pingora doesn't support it.

even with tls, the httpproxy fails to proxy grpc, the request goes upstream but never comes back totally to the client.
upstream peer is: "<IP_ADDRESS>:50055"
upstream_response: ResponseHeader { base: Parts { status: 200, version: HTTP/1.1, headers: {"server": "nginx/1.18.0 (Ubuntu)", "date": "Thu, 11 Apr 2024 20:24:17 GMT", "content-type": "application/grpc", "transfer-encoding": "chunked", "connection": "keep-alive"} }, header_name_map: Some({"server": CaseHeaderName(b"Server"), "date": CaseHeaderName(b"Date"), "content-type": CaseHeaderName(b"Content-Type"), "transfer-encoding": CaseHeaderName(b"Transfer-Encoding"), "connection": CaseHeaderName(b"Connection")}) }
2024-04-11T20:24:17.479479Z  WARN pingora_core::connectors: Unexpected data read in idle connection

on the client:
Received RST_STREAM with code 0

maybe something to do with trailers?
im using basically the default example of proxy with tls, and my grpc server is behind an nignx to provide tls.

@jpvolt I'm trying, but I configured pingora and now downstream works, but now I'm trying to understand how I can proxy to the service request. Loot my code.
`rust
use async_trait::async_trait;
use dotenv::dotenv;
use pingora::protocols::ALPN;
use pingora::server::{configuration::Opt, Server};
use pingora::Result;
use pingora::{
proxy::{ProxyHttp, Session},
upstreams::peer::HttpPeer,
};
use tracing::Level;
pub struct GrpcProxy;
fn main() {
dotenv().ok();
tracing_subscriber::fmt().with_max_level(Level::INFO).init();

let opt = Opt::default();
let mut server = Server::new(Some(opt)).unwrap();
server.bootstrap();

let mut grpc_proxy = pingora::proxy::http_proxy_service(&server.configuration, GrpcProxy);

let mut tls_settings =
    pingora::listeners::TlsSettings::intermediate("cert/localhost.crt", "cert/localhost.key")
        .unwrap();

tls_settings.enable_h2();

grpc_proxy.add_tls_with_settings("<IP_ADDRESS>:8000", None, tls_settings);
// grpc_proxy.add_tcp("<IP_ADDRESS>:8000");
server.add_service(grpc_proxy);

server.run_forever();

}
#[async_trait]
impl ProxyHttp for GrpcProxy {
type CTX = ();
fn new_ctx(&self) -> Self::CTX {}
async fn upstream_peer(
    &self,
    _session: &mut Session,
    _ctx: &mut Self::CTX,
) -> Result<Box<HttpPeer>> {
    let mut peer = Box::new(HttpPeer::new("<IP_ADDRESS>:50051", false, String::default()));
    peer.options.alpn = ALPN::H2;
    Ok(peer)
}

}
`

I can get the trailers {"grpc-status": "0", "grpc-message": ""} in response_trailer_filter, but I don't know how to encode it.
/// When a trailer is received.
    async fn response_trailer_filter(
        &self,
        _session: &mut Session,
        _upstream_trailers: &mut header::HeaderMap,
        _ctx: &mut Self::CTX,
    ) -> Result<Option<Bytes>>
    where
        Self::CTX: Send + Sync,
    {
        Ok(None)
    }

I can get the trailers {"grpc-status": "0", "grpc-message": ""} in response_trailer_filter, but I don't know how to encode it.
/// When a trailer is received.
    async fn response_trailer_filter(
        &self,
        _session: &mut Session,
        _upstream_trailers: &mut header::HeaderMap,
        _ctx: &mut Self::CTX,
    ) -> Result<Option<Bytes>>
    where
        Self::CTX: Send + Sync,
    {
        Ok(None)
    }

@vicanso yeah, I have the same problem here. I dont know http2 spec enough to encode, and the header map does not seem to have any trait that lets us return Bytes, assuming that the bytes on the results are the actually trailer bytes.

This is a bug, we're not propagating upstream h2 trailers downstream, will have a patch for this soon.

Just ran into this error. Happy to help.

@andrewhavck , do you want any help? I'm available to contribute to this patch.

We merged the fix internally last week it should sync this week.

thanks!

The fix @andrewhavck mentioned is available on the main branch. We are not planning on publishing a release with that for at least another week. If using this feature of the main git branch is a problem, let us know here.

@andrewhavck @johnhurt
Thank you very much, It's working!
is it possible to track each "message" in a stream connection? Because I need to metrify each event. I can only get the first event connection in the logging function but the event in the stream no.

@paulobressan we don't support this right now, feel free to open another issue. Closing this one as the original problem is now fixed.


custom filter not work when qurey callback function first params not use specific key name

Package
filament/filament
Package Version
v3.1.27
Laravel Version
v10.38.2
Livewire Version
v3.0.0
PHP Version
PHP 8.3.0
Problem description
when I use Filament\Tables\Filters\Filter::query() function then set the callback function first param name is not $query, the query will not work
Expected behavior
expect the parameters can set with any name and query can work
Steps to reproduce

use a table custom filter
the filter use query function
set first callback parameter name is not $query
reproduce~

Reproduction repository
https://github.com/popdaul/filament-issue
Relevant log output
No response

This is not a necessary issue. But the document doesn't say it's necessary to set $query parameter name.
Conside add Note or Tips to document, or compatible with any key name about callback parameter.
I have tried resolv this issue.
src/Concerns/EvaluatesClosures.php
    protected function resolveClosureDependencyForEvaluation(ReflectionParameter $parameter, array $namedInjections, array $typedInjections): mixed
    {
        $parameterName = $parameter->getName();

        if (array_key_exists($parameterName, $namedInjections)) {
            return value($namedInjections[$parameterName]);
        }
        ## is here
        if (isset($namedInjections['query'])
            && $parameter->getType()->getName() === \Illuminate\Database\Eloquent\Builder::class) {
            return value($namedInjections['query']);
        }

But it may have too much impact.

This is the same for every parameter in Filament, they all have a strict naming convention. If you want to add a new feature (resolution based on Builder type), we'd love to see a PR, thanks!


data-click does not return function value

Example <a href=# data-click=doSomething> test </a>
Value returned from doSomething is ignored!

v1.1.0


[RESOLVED] Unavailable by ICMP ping

Problem started at 02:21:14 on 2023.07.11
Problem name: Unavailable by ICMP ping
Host: ZO-FG
Severity: High
Operational data: Down (0)
Original problem ID: 32044
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23123&eventid=32044

Problem has been resolved in 3h 27m 0s at 05:48:14 on 2023.07.11
Problem name: Unavailable by ICMP ping
Host: ZO-FG
Severity: High
Original problem ID: 32044


How do I use the WindowSystem?

The WindowSystem exists, but there's no real visibility into it in the FAQ or SamplePlugin. Would be nice to have an entry in the FAQ and to explain why it exists.

See example here:
https://github.com/goatcorp/Dalamud/blob/master/Dalamud.CorePlugin/PluginImpl.cs#L64
https://github.com/goatcorp/Dalamud/blob/master/Dalamud.CorePlugin/PluginWindow.cs


Fixed the animation issue

When you click the right arrow It slides right to left and vice-versa for the left arrow click.

@HosseinShabani Can you review this PR? It would be great if we could see this issue fixed.


test-pipeline does not work with file upload

Describe the bug
When using the test a pipeline it does not work with a file upload in the new GUI
Reporter
JJansenvr
Category

Frank Console

To Reproduce
Steps to reproduce the behavior:

Go to test-a-pipeline
Click on file upload button
Select your xml or zip file
see error "Please specify a file or message!" at the top of the page

Expected behavior
selecting a file should result in testing the pipeline with the contents of the file
when a zip is selected every file from the zip should be tested with test a pipeline
Screenshots

Environment:

Browser Chrome
tested on local (IAF 7.6-20200608.143400)
also tested on test environment (IAF 7.6-20200529.175744)
also tested on production IAF 7.5-RC2:

As far as I am aware it does work, although it should be in a .zip file.
This way you could upload multiple messages at once and run them, although I do think that it should be possible to upload a single (xml) file aswell.


Change website for holoviz intro

Changed the name from pyviz to holoviz also for the website. I also built it locally and tested it.

cheers! I think this tutorial could be adapted  / enhanced to feel a bit more integrated into OGGM-Edu one day!


Database connection not being saved at version 6.2.5

System information:
Mac OS
DBeaver 6.2.5

Describe the problem you're observing:
Every time I start dbeaver I have to configure the database connection again
Steps to reproduce, if exist:
Start dbeaver, follow the wizard to configure a PostgreSQL connection. Connect, maybe make some query, close dbeaver. Next time you start dbeaver, it presents the same wizard again. This issue raise again, its happen before in https://github.com/dbeaver/dbeaver/issues/6960.
I should reinstall DBeaver to version 6.2.2 to make this issue gone.

thanks for the bug report.
fixed in scope of #7251
you could test Early Access version (https://dbeaver.io/files/ea)


Update default mempool max gas per tx to reflect current mainnet

Background
The current default mempool max gas per tx value is the old 25M value:
https://github.com/osmosis-labs/osmosis/blob/c6697a5e4b2ccdf216f213a33db7e97b0dd46399/cmd/osmosisd/cmd/root.go#L151
Along with localosmosis instances, this apparently impacts edgenet? So  transactions that exceed the 25M limit fail:
https://github.com/osmosis-labs/osmosis/blob/c6697a5e4b2ccdf216f213a33db7e97b0dd46399/x/tokenfactory/types/params.go#L16
Suggested Design
Please update to 120M to reflect current mainnet.
Acceptance Criteria
When spinning up a localosmosis container, $HOME/.osmosisd-local/config/app.toml's "max-gas-wanted-per-tx" should equal "120000000", like below:
ubuntu@ip-xxx-xx-xx-xxx:~/.osmosisd-local/config$ cat app.toml | grep "gas"
minimum-gas-prices = "0uosmo"
  arbitrage-min-gas-fee = ".005"
  max-gas-wanted-per-tx = "120000000" #<-- this guy
  min-gas-price-for-high-gas-tx = ".0025"

CC: @DongLieu

ok, tkank you

No this conclusion isn't right

Oh I see; I misunderstood the max gas aspect.
Then yea, the problem behavior of the tokenfactory create denom still exists due to the gas cost of 40M violating the 25M per-tx limit.

Thanks for flagging the problem, great catch!
Definitely concerning this got so far along / denom creation has been bricked on testnet for people

added fix in tokenfactory (https://github.com/osmosis-labs/osmosis/pull/5468)
@ibjc could you please check if the issue is resolved with this PR

thanks, looks good to me!
i generated this issue by spinning up a localosmosis container, and then dispatching the tokenfactory's create denom message.
for testing https://github.com/osmosis-labs/osmosis/pull/5468, i just pointed the repo to the sis/tokenfactory-reduce-gas branch, and rebuilt the container.
the readme.md below roughly outlines the steps i took (except i didn't manually update the gas per tx config):
https://github.com/zodiac-protocol/tokenfactorytest/blob/04f2b6188063bb62a2004213606566b687afbbc7/README.md?plain=1#L5-L53


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27304

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27304

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27306

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27306

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27346

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27346

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27365

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27365

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/35563


Document freeze_versions into top-level dataset

The new (recommended) freeze_versions allows the author to freeze into their dataset, rather than needing to modify (fork, modify, and host)
the ///repronim/containers dataset.
Closes #90

great, thank you!


🎮 Game Request
Game logic and basic description
This is a 3D remake of the famous Chrome Dino Game. Run as far as possible or race with one of your friends, but be careful not to bumb into any obsticles!

Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @Durgesh4993 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Provide option for conda installation parameters

First, thanks for the great work 👍
Currently, agents with package_manager.type = conda will install silently using --quiet:
Executing Conda: /home/h4dr1en/miniconda3/bin/conda install -p /home/h4dr1en/.trains/venvs-builds/3.6 -c defaults -c conda-forge -c pytorch pip==20.1 --quiet --json

For debugging purposes, it would be great if instead one could specify other parameters acceptable by conda install, notably -v

@H4dr1en that sounds like a good idea, there is already a --debug flag, but it will not output the conda results to screen. Should not be a problem to add :)
btw:
--quiet --json will just remove the upgrade banner, the actual full return results are in the json format, so just printing it out should give you an insight on what's going on.


Dimension Stack broken/functioning incorrectly with Amplified Nether mod installed?

My guess as to why is that the Nether's ceiling height is fixed at 128 in the code, and so because the Amplified Nether mod changes the ceiling height of the Nether, the Immersive Portals mod does not recognize that change and so still acts as if the bedrock ceiling of the Nether is at 128.
Is there any way of manually changing the height of the ceiling global portal in the Nether, or would that have to be fixed on the programming end of things?

No, you are wrong, this mod's dimension stack y level is not fixed in 128 at all. This mod gets height by the dimension's logical_height. Amplified nether mod does not actually change the logical height of the dimension type.
It's Amplified Nether's issue.
There is also a datapack called Amplified nether which does not change the logical height either.
This mod's latest version in 1.18.1 already supports customizing the Y level of dimension stack.


use Hugging Face notebook login instead of putting tokens in plain text in notebook

actually currently does not work on google colab


About BM25 hard negatives

Hi :)
Thank you for your great work!
I read this issue (https://github.com/texttron/tevatron/issues/66#issuecomment-1473047705).
In this description,
This downloads the cleaned corpus hosted by RocketQA team, generate BM25 negatives and tokenize train/inference data using BERT tokenizer. \
The process could take up to tens of minutes depending on connection and hardware.

does it means "Tevatron/msmarco-passage" is from cleaned msmarco? @MXueguang
I would like to know whether it uses just BM25 hard negatives or hard negatives with some added treatment.

Hi @aken12,
"Tevatron/msmarco-passage" is using BM25 hard negatives but with 2 additional treatment

if the positive passage is not shown in top200 BM25 hits, the example is dropped, so results in ~400k examples rather than original 500k.
We use the augmented corpus from RocketQA, the passage is augmented by title

@MXueguang
Thank you for your kind response!
In examples/coCondenser-marco/get_data.sh, I cannot find the code to select a positive passage.
1. if the positive passage is not shown in top200 BM25 hits, the example is dropped, so results in ~400k examples rather than original 500k.

Do we need to add a process to select positive passages, in addition to running get_data.sh?
Or, is qidpidtriples.train.full.2.tsv.gz already processed? (I guess the latter is not correct.)
Thanks :)

it seems qidpidtriples.train.full.2.tsv.gz only has 400k queries.

Oh, yes, that's true.
I understand now, thank you!!


add double/triple buffer pbo API to memtransfer

The new OpenGL ES 3.0 support will allow asynchronous double or triple buffered PBO reads.  This requires some restructuring and an API modification in memtransfer.  It will allow for more efficient GPU reads at the expense of a frame or two of latency.

PR #18


jobs/java-offline-buildpack points to non-existing file

I checked out the v268 from cf-release.  Then I ran
bosh2 create-release
This failed with the following error ...
Building a release from directory '/Users/aedgar/workspace/cf-release':

Constructing jobs from manifest:

Reading job from '/Users/aedgar/workspace/cf-release/jobs/java-offline-buildpack':
stat /Users/aedgar/workspace/cf-release/jobs/java-offline-buildpack: no such file or directory

Exit code 1
I looked and the java-offline-buildpack is a symbolic link to ../src/java-offline-buildpack-release/jobs/java-offline-buildpack.
But src/java-offline-buildpack-release does NOT exist in the v268 build.   The symbolic link in jobs needs to be removed and the v268 tag needs fixing.

Hey @andrew-edgar. Thanks for opening this issue. The java-offline-buildpack was indeed removed from cf-release, but looks like we failed to remove some of the dangling references. We'll take them out.
In the meantime, if you're not making any local modifications to your release, you should be able to upload CF 268 to your bosh director by uploading via the release manifest:
bosh2 upload-release releases/cf/cf-268.yml

@dsabeti   we are making local modifications which is why we need the bosh2 create-release to work.   I updated the bosh2 cli to the latest and got even more errors
bosh2 create-release
Building a release from directory '/Users/aedgar/workspace/cf-release':

Constructing jobs from manifest:

Reading job from '/Users/aedgar/workspace/cf-release/jobs/cloud_controller_clock':
Fingerprinting source files:
Fingerprinting file '/Users/aedgar/workspace/cf-release/jobs/cloud_controller_clock/templates/newrelic.yml.erb':
lstat /Users/aedgar/workspace/cf-release/src/capi-release/shared_job_templates: no such file or directory
Reading job from '/Users/aedgar/workspace/cf-release/jobs/cloud_controller_ng':
Fingerprinting source files:
Fingerprinting file '/Users/aedgar/workspace/cf-release/jobs/cloud_controller_ng/templates/blobstore_waiter.sh.erb':
lstat /Users/aedgar/workspace/cf-release/src/capi-release/shared_job_templates: no such file or directory
Reading job from '/Users/aedgar/workspace/cf-release/jobs/cloud_controller_worker':
Fingerprinting source files:
Fingerprinting file '/Users/aedgar/workspace/cf-release/jobs/cloud_controller_worker/templates/blobstore_waiter.sh.erb':
lstat /Users/aedgar/workspace/cf-release/src/capi-release/shared_job_templates: no such file or directory
Reading job from '/Users/aedgar/workspace/cf-release/jobs/java-offline-buildpack':
stat /Users/aedgar/workspace/cf-release/jobs/java-offline-buildpack: no such file or directory

Exit code 1
aedgar1macbook:~/workspace/cf-release ((v268))$
Looks like the bosh2 cli does not correctly traverse all the symbolic links in cf-release to capi-release.   Can you validate this and see why the bosh2 cli cannot create-release in cf-release.
The errors above about /Users/aedgar/workspace/cf-release/src/capi-release/shared_job_templates is interesting because the newrelic.yml.erb is a symbolic link to "../../../shared_job_tempaltes/newrelic.yml.erb"  but this is not working as
cf-release/src/capi-release/jobs/cloud_controller_clock" is a symbolic link to "../src/cloud_controller_ng/bosh/jobs/cloud_controller_clock"
So when traversing the second symbolic link it's using the actual path not the path relative to the first symbolic link.   Definitely a bug in bosh2 cli or cf-release or capi-release having all these relative symbolic links.
Thanks!
Andrew

@andrew-edgar We've seen different versions of this issue before. You're right that the issue are the symlinks in cf-release. It was my understanding that the BOSH team was trying to add symlink support to the CLI. In the meantime, you'll need to use the old ruby CLI if you want to create a local release.
@cppforlife, is there a bug in the BOSH tracker to address this?

@andrew-edgar. As I noted in the other Github issue, I've added documentation to describe the problem of using the new bosh CLI to create-release. Also, we've removed the dangling symlink. Is there anything else to think about, or should we close the issue?

@dsabeti No that should be good.  thanks!
Andrew

does this work ?  getting error while create-release
Building a release from directory '/home/custom/cf-release':

Constructing jobs from manifest:

Reading job from '/home/custom/cf-release/jobs/cloud_controller_clock':
stat /home/custom/cf-release/jobs/cloud_controller_clock: no such file or directory

Exit code 1

@parthasarathi204, I think the BOSH team has decided not to implement the full support of symlinks necessary to work with cf-release. As such, you'll need to use the Ruby CLI to run bosh create release, and then you can use the Go CLI for the subsequent operations.

Any way I was doing the same staff  , but did not work .
bosh1 create release --force && bosh1 upload release && bosh1 -n deploy --recreate
Then with out any change i have used
bosh  create-release  --tarball dev_releases/cf/cf-release.tgz --force
bosh -e vbox upload-release dev_releases/cf/cf-release.tgz
retried the above steps once more. Gives same error
L Error: Missing template file 'newrelic.yml.erb' for job 'cloud_controller_clock'
15:25:17 | Error: Missing template file 'newrelic.yml.erb' for job 'cloud_controller_clock'
Started  Tue Aug 22 15:25:17 UTC 2017
Finished Tue Aug 22 15:25:17 UTC 2017
Duration 00:00:00
Task 26 error
Uploading release file:
Expected task '26' to succeed but state is 'error'
Exit code 1

You have to use the old CLI for "create release", and the new CLI for "upload release":
bosh1 create release --force --tarball ...
bosh2 -e vbox upload-release ...

Okay but I could not do  that as I am trying to use a new blobstore_client 'foo'. If I provide the credential details in config/private.yml and config/final.yml as per http://bosh.io/docs/reference/blobs.html , bosh1 commands refuse to work . Errors out as
Cannot initialize blobstore: Unknown client provider 'foo', available providers are: ["dav", "simple", "s3", "local", "s3cli"]
Though I have added the new 'foo' cli  in bosh (https://github.com/cloudfoundry/bosh/blob/master/src/bosh-director/lib/bosh/blobstore_client/client.rb) .
diff --git a/src/bosh-director/lib/bosh/blobstore_client/client.rb b/src/bosh-director/lib/bosh/blobstore_client/client.rb
index 4b26411..45c18da 100644
--- a/src/bosh-director/lib/bosh/blobstore_client/client.rb
+++ b/src/bosh-director/lib/bosh/blobstore_client/client.rb
@@ -1,13 +1,13 @@
module Bosh
  module Blobstore
     class Client
-      PROVIDER_NAMES = %w[local s3cli gcscli davcli]
+     PROVIDER_NAMES = %w[local foocli s3cli gcscli davcli]
Am I missing something ?

@parthasarathi204, I think you're facing a different issue than the original issue here about compatibility between BOSH CLI's and cf-release. To add a custom blobstore, you'll need to ask help from the BOSH team: https://github.com/cloudfoundry/bosh/issues


"Oțelu" is listed in the drop-down menu

Description:
"Oțelu" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Oțelu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Oțelu, Argeș (Berevoești)".


Async federated training

We introduce async training scheme to the federated setting:
The main class is the AsyncWeightHolder, this contains a buffer of currently loaded weights.
It works as follows:
The current model has an associate version.
When a user posts weights, these are added to the buffer only if they were trained from the current version (we can easily relax this to allow weights from versions that are not older than n iterations of it). Once the buffer reaches capacity, we average the weights in the buffer, update the latest model and increment the version by one.
We also added several tests for the class and corresponding server requests.
TODO:

[x] Renamed version to round

Did some refactoring of the helper classes/folders to make it more intelligible what they do, and refactored accordingly s.t. each file unit has a theme of functions that are similar in what they do

Async is now default for federated since the sync is buggy, we can later add the option sync/async to tasks for example as suggested by @martinjaggi or in the UI.
When testing on e.g. titanic, the output in the server console is as follows:
************************************************************
-> Adding new weights of LXyFxBoyQl to buffer.
-> Adding new weights of leKwHW1H5G to buffer.

Buffer is full; Aggregating weights and starting round: 1

-> Adding new weights of LXyFxBoyQl to buffer.
	Updating weights of LXyFxBoyQl to buffer.
	Updating weights of LXyFxBoyQl to buffer.
	Updating weights of LXyFxBoyQl to buffer.
-> Adding new weights of leKwHW1H5G to buffer.

************************************************************
Buffer is full; Aggregating weights and starting round: 2

-> Adding new weights of LXyFxBoyQl to buffer.
This indicates that LXyFxBoyQl just contributed to the current round for the first time, if this is followed by:

excellent work. just some cosmetics for later:

rename holder to buffer
mark the old weights as sth like weights update based on round x
add a time-cutoff which can be different from one. so if then if x < now - cutoff discard them? (you already added a comment)
clarify the logic documentation a bit at the main parts (in the buffer there can be max one weight/parameter per client.
check if the round number is already attached as meta-date for each model received anyway? (we had this for federated statistics on the server).
remove the non-async scheme, then code becomes easier to read, and no switches needed

thanks, good suggestions, some questions:

Currently we do not store old weights, in a current round if a new user updates new weights we override them. We could store the old ones, but I think we risk getting in memory troubles.
If we want to have the cryptographic stack that Vincent worked I think we need to have a sync version of federated. Although maybe it can be modified for async, but I'm not sure.

no need to store old weights. only the freshest per worker makes sense

for secure MPC, the buffer will be compatible (there is a recent facebook paper). so it's ok to fully drop sync

relevant paper, just for future reference:
Federated Learning with Buffered Asynchronous Aggregation
https://arxiv.org/abs/2106.06639


Port the baroclinic channel test group from compass

Checklist

[x] User's Guide has been updated
[x] Developer's Guide has been updated
[x] API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
[x] Documentation has been built locally and changes look as expected
[x] Testing comment in the PR documents testing used to verify the changes
[x] New tests have been added to a test suite

Testing
So far, I have only tested the 10 km test cases. On my Ubuntu laptop, I verified that all of the following are bit-for-bit with compass:

default
decomp_test
restart_test
threads_test
I also verified that rpe_test ran successfully and produced the following images that look qualitatively similar to compass output:

I have not yet verified that the RPE tests run at 4 and 1 km resolution.  I also haven't compared the RPE output quantitatively with compass.  I believe some algorithmic changes I made might affect the results and I think we likely are willing to live with that.

@cbegeman, when you have time after you get back from vacation, could you run the RPE tests for both compass main and this branch, and verify that the output looks similar enough for your taste?
I would appreciate you having a careful look at the code and documentation, too, since this will likely serve as our template for a lot of future development.

@cbegeman, just a quick note to say that I have something I'm happy with in #28 and will use this branch to test it but haven't got that far yet.  I'll ping you again when #28 and then this are ready to review.

I noticed that the 1km rpe test ended up with a config_btr_dt = 1s instead of compass's 2s. I think the consistent scaling of dt with resolution is best, just wanted to make you aware.

Yes, I believe I did that on purpose for consistency of the algorithm for computing config_btr_dt.  I agree that the change is desirable but worth noting.

@cbegeman, I believe I've made all the changes you requested in your review.  I'll go ahead and merge after CI completes. Obviously, we will want to continue to test this and make changes as issues emerge, since this is too much code to hope to port over and modify without bugs.


[BUG] Wrong init and no DE

It's the package from edge:
fastfetch -v
fastfetch 2.27.1 (aarch64)

OS: postmarketOS v24.06 aarch64
Init System: busybox

I believe that the init is openrc and not busybox am I wrong?
In the config I have DE but it does not show it.
Is that the file that launch the session?
For example /usr/share/xsessions/bspwm.desktop.

This happens for all userlands which have a shell built into them, including busybox and toybox.

The InitSystem module in fastfetch reports the first process started by kernel. In your case, it's busybox. This process is not necessarily the real process that initializes your system, because it can delegate the task of initializing the system to another process. See /etc/inittab.


Update python version

Doing the MIMIC-IV example in Python 3.11 gives me:
Traceback (most recent call last):
  File "/Users/robin/Documents/git/MEDS_transforms/MIMIC-IV_Example/pre_MEDS.py", line 266, in main
    relative_in_fp = fp.relative_to(out_fp.resolve().parent, walk_up=True)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: PurePath.relative_to() got an unexpected keyword argument 'walk_up'

Summary by CodeRabbit

New Features

Updated minimum supported Python version to 3.12.

Bug Fixes

Adjusted compatibility requirements for existing environments using Python 3.11 or earlier.

@rvandewater as we deprecate these example folders and move to the ETL template repos like this for MIMIC, do we still need to address this issue? https://github.com/mmcdermott/MIMIC_IV_MEDS

In case the MIMIC_IV pipeline has not changed we would need to specify python 3.12 for that specific etl. Otherwise, no we do not.


Working testPubSub with ready for review.

This has a test case for inter-thread and one for inter-process. Each is just point to point. Each does a thousand trials of random data messages of randomly generated length from 1 byte to 1400.

One last note: please remember to rebase this PR to main once #23 is merged


Followup to 2497. Addressing comments from PR 2497

Description
Addressing comments made in PR 2497.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ X] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ X] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2628 (aa0ae39) into main (1f90f38) will decrease coverage by 0.08%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #2628      +/-   ##
============================================
- Coverage     93.46%   93.39%   -0.08%
- Complexity     2185     2186       +1
============================================
  Files           258      258
  Lines          6107     6114       +7
  Branches        497      499       +2
============================================
+ Hits           5708     5710       +2
- Misses          268      270       +2
- Partials        131      134       +3

see 3 files with indirect coverage changes


demo.py and AssertionError np.all(0 < np.linalg.eigvalsh(S))

Hi,
When running python examples/demo.py I get the following error:
Traceback (most recent call last):
File "examples/demo.py", line 107, in
model.resample_model()
File "/usr/local/lib/python2.7/dist-packages/pyhsmm/models.py", line 430, in resample_model
self.resample_parameters()
File "/home/someuser/code/pyhsmm-slds/pyslds/models.py", line 36, in resample_parameters
self.resample_lds_parameters()
File "/home/someuser/code/pyhsmm-slds/pyslds/models.py", line 41, in resample_lds_parameters
self.resample_dynamics_distns()
File "/home/someuser/code/pyhsmm-slds/pyslds/models.py", line 58, in resample_dynamics_distns
for s in self.states_list])
File "/home/someuser/.local/lib/python2.7/site-packages/pybasicbayes/distributions/regression.py", line 257, in resample
*self._natural_to_standard(self.natural_hypparam + stats))
File "/home/someuser/.local/lib/python2.7/site-packages/pybasicbayes/distributions/regression.py", line 97, in _natural_to_standard
assert np.all(0 < np.linalg.eigvalsh(S))
AssertionError
My numpy version is 1.8.2.
What could be the problem?
Thanks!

Can you try upgrading numpy? I'm on 1.11.0 on this laptop and I don't see that error. Also try moving to the most recent version of pylds (clone the github master branch and do a pip install -e .).

Thanks, solved it!


Fix default Java parameter types

Summary

Renames bigint -> biginteger
Replaces INTEGER_REGEXPS  -> FLOAT_REGEXPS for bigdecimal
Enables snippet generation for bytes

Motivation and Context
https://github.com/cucumber/cucumber-jvm/pull/1248
Types of changes

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

Checklist:

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

I already did the first 2 of those.
For little changes like this, let's stick to tbd. I'll merge this so we get the byte fix too.


Target framework questions

From old docs issues list:
https://github.com/NuGet/Home/issues/358 (Targeting)
https://github.com/NuGet/Home/issues/720 (PCLs)
https://github.com/NuGet/Home/issues/1160 (multi-targeting)
https://github.com/NuGet/Home/issues/4331 (.NET compact)
From customer feedback:

What is the equivalent of the deprecated packages? when I create a .netcore library it import dnxcore50 which is deprecated? Should I replace it with netcoreapp1.0 or netcore50?

Need some examples

Rolled up to #778


Import using Gradle doesn't work

Hi,
I have followed your readme file to import the library into my project.
I put the dependency statement into build.gradle file:
dependencies {
compile 'com.antonyt.infiniteviewpager:library:1.0.0'
}
When I do the the project sync with Gradle files I get the following error:
Failed to resolve: com.antonyt.infiniteviewpager:library:1.0.0
What can I do to import correctly your library into my project?

I am having the same issue.

@cvongrim This didn't work for me at all.


Add support for minLen.  Connects to issue #316.

Adds new option, minLen, as requested in issue #316.

:+1:


add unit test for SPDM 1.3 MEL algorithm negotiate

please add one new sub test case to trigger LIBSPDM_STATUS_INVALID_MSG_FIELD


Adding aria-hidden for accessibility

I'm using a modified version of this library to embed ASCII animations and someone had suggested that adding the aria-hidden attribute would be useful avoid tripping up screen readers. Would you be interested in me creating a pull request to add that to the library?

I was thinking about it last week… Where would you add it?
The target element for the rendering can freely be chosen and is not created at runtime… I don’t think it’s not the job of the library itself to add this attribute to it (you never know how it will be used).
But it’s a good idea to point out the isse so I'd rather add it to the embed examples and add an “accessibility” chapter in the manual – but not impose it; what do you think?

Exactly, that rings true to me, especially since it's so easy to add the attribute to a parent element; affecting all children.


The issue happens when the board is a bit too small. Try it with a row size of 100 and a column size of 100. Hope this helps.

yep, works with larger sizes, thanks


Fix membership::change_key for keys that maintain the sort order

Fix bug introduced here: https://github.com/paritytech/substrate/pull/4300

Superseded by https://github.com/paritytech/substrate/pull/430


Feature request: Change tag names

Thanks for this really helpful project! :-)
I want to use it to generate JSX code but with custom tag names.
F.ex.:
import { htmlToJsx } from "html-to-jsx-transform";

const jsx = htmlToJsx('
Hello World!
',
  {changeTagNames: (id: string) => id === "h1" ? "MyHeaderComponent" : id});

// jsx === '<MyHeaderComponent tabIndex={0}>Hello World!</MyHeaderComponent>';

If you are interested I could write a PR for this otherwise I would fork the source.

Sounds like a useful addition, feel free to add it.

Sorry for my long break. :-(
In the meantime the requirements of our project changed, we don't use html-to-jsx-transform anymore; so I will close this feature request.
But html-to-jsx-transform is still a source of inspiration for us, thanks for your great work!


What's cost function behind DualVariableWarmStartOSQPInterface

System information

Linux Ubuntu 16.04:
Apollo installed from (source ):
Apollo version (5.5):

When doing warm startup in open-space planning task,  DualVariableWarmStartProblem is used to solve l_warm_up, n_warm_up
The objective function is norm(A' * lambda) in OSQP solver, any detailed explanation about why?
The reference material 'Optimization-Based Collision Avoidance' doesn't match with code very well.

It is not the same dual warm up strategy from OBCA, generally speaking, it wraps the l2 norm constraints into the cost function, since OSQP is not good at solving QCQP.

Hi, @quhezheng  thanks for your interests and contributions to improve the Apollo platform. We will close the issue for now. If you have any additional question, please feel free to open a new issue. Our engineer teams are more than happy to help that.
Thank you for supporting Apollo!


Don't serialize values in toPlainObject()

Because if an object value implements toJSON then the result of
toPlainObject() is going to have toJSONified value as opposed to the
underlying object. This is unexpected as all we really want from
toPlainObject() is to get rid of type properties.

Hey @artemave I'm not sure about adding the dependency on clone. It's a lot of code to solve a problem I don't fully understand. Could you elaborate on this?

This is unexpected as all we really want from toPlainObject() is to get rid of type properties.

If I understand, you have an object that implements toJSON(), which isn't a value-object, and you don't want toPlainObject() to call your toJSON() (which is the default behaviour when it doesn't know what else to do)... correct?

Can you implement a method in your object, like "toPlainObject()" which returns its properties?

toPlainObject() is entirely about serialisation :) — though clearly it could be named better!
You’re right, it was historically called toJSON({ typeAnnotations: false })
The only reason that changed is that it was incorrect because toJSON() already takes a key argument, so we can’t overload it.

Fair enough! Can we have another method for this? Let's call it.. hmm.. something like maybe perhaps... toObject()?

I actually like the idea. Just not the dependency, because I'm trying to keep value-object as slim as possible.
How about if we accept clone as an argument?
.toPlainObject(clone)

Great, thanks @artemave!

Nice one! Could you document it? I was struggling to come up with a good explanation ;)


ENH: Addition of Poisson Means Test (E-test).

This PR builds on gh-10561 and implements several remaining comments.
There have been numerous requests on the broader internet for an easily accessible and fast implementation of the "E-test", or the poisson means test. The publishing paper is somewhat popular but there are few, if any, implementations other than the original authors' Fortran. This Fortran has some noted faults, which this implementation mends.
The name poisson_means_test is somewhat vague, but the idea is that while the current implementation is for the E-test, a method parameter could be added that allows the user to specify if they want E-test or C-test, where the default is E-test so it maintains backwards compatibility.
It's a bit more complex to review since this builds off of a relatively stale PR, but I've done my best to address the common concerns in that PR. I have not removed the original contributors commits since I don't want to abstract the credit away, but for merging I think this will be a squash into one merge.
https://github.com/scipy/scipy/pull/10561#discussion_r688991083

avoid the use of np.divide(..., where)/np.multiply(..., where) in favor of indexing as needed and using the usual operators

This has been done with indexing.

We should also only compute the PMFs where the indicator is actually true

I agree that this would be ideal, but the likeliehood that any one of the PMFs is not used seems somewhat low. If the two arrays that result from a pmf calculation are
x1 = array([0, 1, 2, 3, 4, 5, 6, 7, 8, 9])
x2 = array([[0],
            [1],
            [2],
            [3],
            [4],
            [5],
            [6],
            [7],
            [8],
            [9]])

and then they are broadcast together, the indicator does not usually take out entire rows or columns (from my brief testing, not proof based) so I don't think that trying to check if certain rows or columns don't need to be calculated.

Switching the arguments has the downside of changing the test statistic

I agree. Instead of switching the arguments I limited it to swapping out the indicator.

rename them to n1 and n2. If we were to change count1 and count2 to k1 and k2,

Being able to compare to the paper is important, so I think this is a good choice.

n1 and n2 be floats?

I think it is reasonable that these can be floats.  https://github.com/scipy/scipy/pull/10561#discussion_r688987209 They are a "count" in some sense, but as long as the unit of measure is maintains across both "counts" then it is fine. However there seems to be some interesting results when you continue to make the "counts" smaller.
stats.poisson_means_test(0, 125.0, 3, 125.0)
PoissonMeansTestResult(statistic=-1.7320508075688772, pvalue=0.08837900945483518)

stats.poisson_means_test(0, 12.5, 3, 12.5)
PoissonMeansTestResult(statistic=-1.7320508075688772, pvalue=0.08837900945483518)

stats.poisson_means_test(0, 1.25, 3, 1.25)
PoissonMeansTestResult(statistic=-1.7320508075688772, pvalue=0.08837900945483518)

stats.poisson_means_test(0, 0.125, 3, 0.125)
PoissonMeansTestResult(statistic=-1.7320508075688774, pvalue=0.08837900945483518)

stats.poisson_means_test(0, 0.0125, 3, 0.0125)
PoissonMeansTestResult(statistic=-1.7320508075688774, pvalue=0.032362623816529226)

stats.poisson_means_test(0, 0.00125, 3, 0.00125)
PoissonMeansTestResult(statistic=-1.7320508075688774, pvalue=0.08837900945483518)

stats.poisson_means_test(0, 0.000125, 3, 0.000125)
PoissonMeansTestResult(statistic=-1.7320508075688774, pvalue=0.08837900945483518)

For some reason the p-value is different when the sample size is 0.0125. Kind of perplexing, could be safer just to leave it as an int. If the argument is valid that the unit doesn't matter, the user should be convert their data to an integer in order for the test to work.  The author uses sample "size"  which is ambiguous, but I think to be conservative we could leave it as an integer input in case floats are indeed not allowed. I definitely think that k1 and k2 (the observed values) are intended to be integers because they are described as counts.
Additional information

I expect there to be one test failure relating to a division error.
It was previously suggested that

Other, unrelated notes that I would put in the code, but they'd disappear once mdmahendri#1 merges: we need to compute the statistic and p-value even if lmbd_hat2 <= 0.

I suspect the reason that the authors indicated that you do not need to calculate the p-value in this scenario is that it leads to ZeroDivisionErrors down the road. Often (but not always) the variance can be 0, which is a common denominator in divisions. I'm still thinking about the best way to put this into action.
Also -- I see the lint errors. My pep8 checker locally just doesn't work I guess. Will mend on next push.

@swallan Maybe finishing this up should be next?

To sum up what is going on with this PR:

There are no external implementations of this method outside of a fortran method that has noted faults. If there are other implementations, they are well hidden.
there is some ambiguity relating to what the input types should allow. I think that is appropriate that the inputs n1 and n1 any real value, and that k1 and k2 should be integers. See comment https://github.com/scipy/scipy/pull/10561#discussion_r688987209 for discussion. To sum it up, ns are "observations" in the sense that they are an observed amount. It could be 100 grams or .1kg. However ks should be integers since the poisson distribution handles discrete values. In the post describing the PR I show how the units can be changed and the results are the same.

Thanks, Sam. LMK when you've summarized the agreement between this implementation and the Fortran code and are ready for me to take a look.

I took a look at the results in test_fortran_authors and see that the maximum relative difference for the p-values was 5.84296954e-06.
It was for this value:
0.008053640402974236  # fortran
0.008053687460149835  # scipy

so a difference in the 8th decimal place. Even for a small value I think that is pretty good.
The PMF for poisson is relatively simple to evaluate, so differences from fortran aren't likely caused by any inaccuracies in Scipy.

SciPy's poisson PMF utilizes the logPMF, which I assume was done to get better stability. It was done a long time ago so I can't immediately see why this edit was made, though.

Great. Are you happy with this then? Shall I review more carefully?

Yes please!

@swallan would you send an email to the mailing list asking whether we should add this test even though there's already a version in statsmodels?
@chrisb83 @josef-pkt what do you think? @josef-pkt I ask you from the statsmodels perspective, and @chrisb83 I ask you because you were involved in the original PR https://github.com/scipy/scipy/pull/10561.

Update from statsmodels:
statsmodels has currently only the test for the ratio of poisson, not for the difference of 2 poisson rates.
I spent the last month getting almost everything for one and two-sample Poisson
https://github.com/statsmodels/statsmodels/pull/8166
which includes now etest for ratio and diff, with either score or wald test statistic.
I skipped the original Krishnamoorthy and Thomson moment estimate (at least for now) since there is no advantage to it compared to score, AFAICS.  (I might still add it for completeness)
The score constrained MLE and pvalue also also explicit, analytical expression and don't need numerical rootfinding as does the score confidence interval.

I think the test would fit well into scipy.stats. I personally do not mind the duplication.

Thanks @josef-pkt @chrisb83.
@swallan please send an email to the mailing list inviting feedback. Please mention that statsmodels may soon offer enhanced functionality for this test, but that we are considering adding this (the basics) in SciPy nonetheless.

Email sent 5/3.


Not working...?

Sorry if my question is dumb...I just started learning React a couple days ago.
When I run the code below,
I can drag "Drag me!", but I can not drag it onto "I'm a valid target...".
Also when I load the page, console instantly shows "test", instead of after dragging and dropping "Drag me!"
import React from 'react';
import ReactDOM from 'react-dom';
import { DragDropContainer, DropTarget } from 'react-drag-drop-container';

class Game extends React.Component {
    render(){
        return(

                 <DragDropContainer targetKey="foo" onDrop={console.log("test")}>

Drag Me!

                </DragDropContainer>
                <DropTarget targetKey="foo" >

I'm a valid drop target for the object above since we both have the same targetKey!

                </DropTarget>

            )
    }
}

ReactDOM.render(
  <Game />,
  document.getElementById('root')
);

Your code immediatily shows 'test' because you are invoking a call to console.log on the definition of the component. onDrop must receive a function reference (not a function call) change your code to
onDrop={() => console.log("test")}

Thank you very much, it worked!! I need to study more.


Implement catalog-rebuilder so it will automatically rebuild if catalogs are empty

Now the an environment variable CATALOG_REBUILDER_ENABLED have to be set, to make the catalog-rebuilder execute the job.
This can be automated, so that the catalog_rebuilder will automatically rebuild the catalog if csw/solr is empty.
To check this, DMCI will implement a status-endpoint which the catalog-rebuilder then can use to check if it is empty, and then start the job.

After some thought the solution we have will work better, and using dmci sidecar gives us some benifits with multiple workers etc.

This can be closed. Does not apply anymore. The new catalog-rebuilder have admin posibilites to clean the csw and solr, and then invoke manual rebuild


Regex "*" token only captures one match with RegEx.search_all()

Godot version
4.0.beta10
System information
Windows 10
Issue description
RegEx in godot only captures one, exactly one, using search_all() on a regex that contains "*". I don't know how to explain it in words, but I put a code to reproduce the problem below.
Why am I consider this as a bug? the same regex, "a*", in godot has different result of what RegExr does.
Steps to reproduce
var regex = RegEx.new()
regex.compile("a*")

var str = "a captain eats crew"
var results = []

for result in regex.search_all(str):
  results.append(result.get_string())

print(results) # got ["a"], expected to be ["a", "a", "a", "a"]
# RegExr.com also gives me ["a", "a", "a", "a"] and not ["a"]

Minimal reproduction project
No need

Beta 10 was two months ago, can you test this with an up to date version

@AThousandShips sure, I'll test out on a newer version, rq.

@AThousandShips yes, I tested with newer version, 4.0.rc4. And the bug is still there with the exact same code as above.

I am not sure why this breaks but if you just search for "a" not "a*" it works, I suspect because the search pattern causes some confusion as it can be empty therefore causes some kind of issue with restarts or anchors, for example it breaks with the string " a captain eats crew", indicating that when you add a repeat that yields an empty result you get some weirdness like that.

To clarify what happens:
In the example you gave it searches and finds "a", then it starts after "a", finding "" as it matches the pattern, and the search_all function then detects that the end of this search is at the same point as the end of the previous one, and then stops.
In my example it instead finds "", and then stops by the same reason.
While it can be said to be counterintuitive it also is not a very useful search, and there would have to be a decision over how to change this behavior, do we add all the empty cases? Do we add none?
Doubt that any "fix" will be made for this but if it is desired I can write it.

Yes, I thought the same that a* could even match empty strings and probably did an infinite loop. However, as far as I know, RegEx doesn't make the a* match empty string. Though, I can fix it by changing * to +.
Anyway, the "a captain eats crew" is just my brief example. I tried using \S* to search every word (not using \w), and it still returns one exact match using search_all().
Well, I just think this is a bug, because it works on the other place. If somehow this is an intended feature of the godot, I might use + instead of *.

Not all regex engines are the same, but I'm wondering, what do you intend to achieve with using *, why do you want to match an empty result?

Yeah, that's my fault of using * instead of +. I always thought that a* will match if there are a. I don't know whether I need to close this issue or not, but I think I found another bug, I'll post in a separate issue.

I fixed this issue while trying to fix a different bug: https://github.com/godotengine/godot/pull/85783
The short version is that search_all wasn't able to handle zero length results very well.
I did a test with the above "a*" and "a captain eats crew" here: https://regex101.com/r/ynUQra/1
According to the website, there are 20 matches. 4 of them matches a while the remainder are empty strings. Trying the Godot engine with the above pull request, I was able to get the exact same result:

Switching back to a version of the engine before my branch, I got this, showing that it was indeed my branch that fixes this issue:

Amended my PR to include an Asterix search_all test just like the one above. It confirmed my PR fixes issues like this, so if it's accepted this issue should be closed.

So this is a bug... T_T