Automatic hide on long list of tags

Describe your suggested feature

As you can see the list of tags in the bar i ban (standard) using the save-filter feature is too long and I would like to make it easier the eventual addition of  more tags by making space in the tag bar (but also the other bars)
i suggest to make the app hide automatically in a little colored square:

in this way one could add all the tags wanted without obstructing the space of the bar.
The automatic part should be triggered by the number of words typed in the bar (also other bars).
Also, clicking on the colored squadre should make the tags appear but i don't have a proper suggestion of how it should happen,  or at least, not a comfortable one.
Add a toggle option to disable this feature.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.8.1.
[X] I will fill out all of the requested information in this form.

I personally done see a need for this, its working as its supposed to in the material spec


fix: run rector fixer on codebase

Issue
Rector had one open issue
Solution
Run rector:fix
Impact
Quiet rector
Usage Changes
no
Considerations
no
Testing
no

coverage: 88.014% (-0.003%) from 88.017%
when pulling d99745a19c95fad5ef38844280864e37b850cc3a on fix-rector
into 48dc3fc5a9104251f440af6b65f6a622660a91dc on 2.x.


using an existing julia environment

Hi, thanks for your great packages including PythonCall and this one!
I've recently got started with PythonCall, especially the JuliaCall package from Python.
As my requirement is to access an existing Julia environment instead of creating a fresh one, I tried specifying PYTHON_JULIAPKG_PROJECT. Unfortunately,  juliapkg just rewrites the existing Project.toml, and I haven't got a success without modifying the source code of PythonCall.jl.
I wonder if there can be a way to specify the current project without any hardcoding.

It works smoothly. Awesome project!


ci/win: add ARM, ARM64 and x86

We could also add containers to run testcode on ARM.
old example: https://github.com/baresip/retest/blob/main/.github/workflows/run-on-arch.yml

there are some errors in the build:
C:\Program Files\OpenSSL\lib\libssl.lib : warning LNK4272: library machine type 'x64' conflicts with target machine type 'x86'
C:\Program Files\OpenSSL\lib\libcrypto.lib : warning LNK4272: library machine type 'x64' conflicts with target machine type 'x86'

perhaps the wrong Arch for the OpenSSL libs ?

Why building arm on windows?

Why building arm on windows?

We had recently some build errors for Windows ARM and RE_ATOMIC. The complete ARM ecosystem is growing, so it I think its useful to catch these ones early.

perhaps the wrong Arch for the OpenSSL libs ?

Yes we need platform builds for OpenSSL, maybe we found a good source of prebuilt ones, or use cached builds.

https://github.com/chocolatey/choco/issues/1803
it is also possible to compile on ARM with OpenSSL disabled.

Implemented in: https://github.com/baresip/re/pull/855


Suggestion about files removal when storage is almost full

Hi,
I like very much this project. I think it is a great one!! I plan to use it with my Home Assistant server.
One thing I noticed that I think can be improved (user experience aspect) -
I run kerberos.io on docker and suddenly witnessed all files on the capture folder being removed every 1 min or so.
It turned out that my 450GB SSD had only ~32GB free space. Once I cleaned the recycle bin (~80GB), the phenomenon disappeared.
I guess Kerberos checks free space by percentage - I suggest to do it by actual GB. 32GB are a lot of free space and should not cause such aggressive files deletion.
I also suggest to eliminate older files only. Maybe leave always the newest 50 files or so? Instead of delete them all.
thanks in advanced and cheers!

we'll take that into account in new release (this repo). The issue however is related to kerberos-io/machinery. Thanks @quammy


chore: bump datafusion version to fix last_value regression

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
... to the datafusion commit with PR https://github.com/apache/datafusion/pull/10783.
Now last_value with string arg works:
mysql> select last_value('3');
+-----------------------+
| last_value(Utf8("3")) |
+-----------------------+
| 3                     |
+-----------------------+
1 row in set (0.04 sec)

Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

It's better to add a sqlness test to https://github.com/GreptimeTeam/greptimedb/blob/main/tests/cases/standalone/common/function/expression.sql

This PR also fixed #4096


refactor: increase batch size to get validator indices to 64

Motivation
The current batch size of 10 is quite conservative. The rationale behind choosing the value https://github.com/ChainSafe/lodestar/pull/2730#issuecomment-866749083 seems outdated and there has been more discussion on this https://github.com/ethereum/beacon-APIs/pull/328 in the meantime.
The value 64 is supported by all implementations and reducing the amount of queries reduces load on beacon node, especially during startup.
Description

Increase batch size (PUBKEYS_PER_REQUEST) to get validator indices to 64
Update tsdoc of batchItems, previous comment is misleading and inaccurate

:tada: This PR is included in v1.10.0 :tada:


The driveId parameter must be specified if and only if corpora is set to drive.

  "errors": [
   {
    "domain": "global",
    "reason": "teamDriveIdRequiresTeamDriveCorpora",
    "message": "The driveId parameter must be specified if and only if corpora is set to drive."
   }
  ],

Google does return this error. But it should not say that because.
Corpora can never be set to drive.
--
From this endpoint https://developers.google.com/drive/api/v3/reference/files/list?apix_params={"corpus"%3A"user"%2C"includeItemsFromAllDrives"%3Atrue%2C"teamDriveId"%3A"0AAGLxubfP1gqUk9PVA"}

Compare to
"message": "Invalid string value: 'drive'. Allowed values: [domain, user]"

Answer -> corpus is deprecated, use corpora


[Proposal] : Better handling for bad worker nodes.

Problem :
In case we have a bad worker having environment issues e.g disk issues and all tasks being assigned to that worker starts failing, If configured with FillCapacityWorkerSelectStrategy, Overlord keeps assigning new tasks to the bad worker since it is the most likely the one with the least used capacity.
Solution :
Approach 1 : Add periodic sanity check to the workers.

while starting up a worker will fork and run a sanity check task and announce itself if the task succeeds. If it fails it will retry it n (configurable) times and exits if the task fails repeatedly.
worker will periodically run sanity checks on its own by forking a small test-task
If the dummy task fails repeatedly, the worker disables itself and overlord will not assign any more tasks to this.

Approach 2  : Make overlord smart to identify faulty workers and blacklist them.

maintain a count of failed tasks, and blacklist a worker when it exceeds a configurable threshold. the counter will need to be reset when a task succeeds on a worker.
send a small NoOp dummy task to the worker before blacklisting it in order to distinguish between issues in task itself or worker environment.
Optionally also verify all the newly added workers too by assigning a dummy task.
Autoscaling should be able to replace the blacklisted worker.

I like Approach 1 and I think its better to have the worker do sanity checks on itself instead of overlord asking workers to do it.
Thoughts ? Suggestions?

I agree with #1, it fits in better with my Tiering proposal for tasks as well.

Just to be different, I like #2 because it would catch more problems. A worker that is broken in some other way than just all tasks immediately failing won't be caught by method #1. But either one is better than nothing so I am not picky.

IMHO the worker should be responsible for reporting its own status. And different types of workers would have different criteria for "healthy".
Obviously the overlord should have some level of "Can't even reach the middle manager" case handling.

I like both. But, I would go for 1 because it is more self-contained within the worker and simpler to implement. Also, It will be great to increase the scope of 1 a little bit to track count of consecutive failures (for any tasks assigned to it and not just the sanity task, and disable itself if there were too many consecutive failures.
In future, We can evolve towards 2 if/when we notice cases not caught by 1.

What is the status of this within the upcoming 0.15.0 release?

sort of handled by https://github.com/apache/incubator-druid/pull/3643 and subsequent related changes.


Double html snippets when extending with html snippets

While trying to set up neovim for htmldjango files, I was unable to get my html-lsp working inside htmldjango files. Not really a issue with this repo of course.
The only solution I came across was extending htmldjango files with html snippets. However, upon doing so, I get double html snippets.
I've tried uninstalling lsps to narrow down the issue and it seems to only happen after I extend these snippets like so:
local luasnip = require("luasnip")

-- loads vscode style snippets from installed plugins (e.g. friendly-snippets)
require("luasnip.loaders.from_vscode").lazy_load()

-- Include snippets not in global snippets for friendly-snippets
luasnip.filetype_extend("htmldjango", { "html", "loremipsum", "djangohtml" })
luasnip.filetype_extend("html", { "loremipsum" })
lluasnip.filetype_extend("python", { "django" })

Here is how it looks like:

Is anyone else getting the same or is it just me?

FWIW, I was unable to replicate this issue locally, but I have in the past had duplicate snippets. I'll leave this open for a few more days in the event I figure something out in my spare time OR if someone else encounters the issue too :thinking:

I run into the exact same issue when adding luasnip.filetype_extend("htmldjango", { "html" }) to my config.


vdc-open helm chart - add ranger plugin & impersonation module

When Gaian is configured as the virtualizer component to provide a sample VDC environment, we need to ensure the ranger plugin is installed (to implement data governance access rules), and the impersonation module (so that we can use NPAs)

The VDC environment will need significant re-work to adapt to the many changes made in Egeria to better support
metadata integration. Additionally the helm charts used for the lab & a simple base config have evolved to better support
different types of services, exposing of ports, persistent storage etc.
As such specific incremental changes to the current - now old - charts do not really add value.
As such closing for now


Carts in_cart not an attribute of the schema.

Status is not in the schema. I designed it so that you would have to join it with products and see that the quantity > 0.
    @staticmethod
    def get_user_cart(userid, status='in_cart'):
        # Fetch items based on the status (either 'in_cart' or 'saved')
        rows = app.db.execute('''
        SELECT userid, productid, productname, quantity, unit_price, status
        FROM Carts
        WHERE userid = :userid AND status = :status
        ''', userid=userid, status=status)
        return [Cart(*row) for row in rows]

There are a lot of functions in carts.py being called with this argument. Plz fix and check that after you log in you can access the carts page.

Modified schemas to allow for this.


[Port][Expression] Floor, Ceiling, Round, TicksToDays,TicksToHours, TicksToSeconds, DateTimeDiff pre-built functions

closes: #2284
closes: #2297
closes: #2337

Pull Request Test Coverage Report for Build 138884

49 of 51 (96.08%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 80.975%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/adaptive-expressions/src/expressionFunctions.ts
42
44
95.45%

Totals

Change from base Build 138635:
0.03%

Covered Lines:
12887

Relevant Lines:
15199

💛  - Coveralls


E: You don't have enough free space in /var/cache/apt/archives/

Thanks for the commands to switch to the community repos and remove the nag. I've executed both, checked for upgrades and found 32 updates. I'm getting a message that I don't have sufficient free space.
In theory, I should have lots of available disk space on the machine. Is there a way to increase the space allocated to Proxmox itself?

Looks like I only allocated 100Gb of 500 to Proxmox and the 100Gb is 99.7% full so what I'm really needing to do is understand how to assign the remainder of the disk so Proxmox can use it

@philgree By default, Proxmox will will setup two primary storage areas. One should be used for Proxmox and data needed to setup a VM or LXC. The other should be used for the virtual disks for the VMs and LXCs. Check the storage location for the VMs and LXCs, if they are using local consider moving them to the other storage pool. To help with the setup of new VMs and LXCs, I would change the Content parameter in the local storage pool to equal VZDump backup file, ISO image, Container template.

@whiskerz007 Thank you very much. I struggled with this problem for a few days. Ultimately my Googlefu failed me, I went back to a bare metal install and it setup exactly as you describe. I obviously did something strange on my first install - I remember specifying just a 100Gb partition to 'dip my toe in the water' rather than letting it take the whole disk.
As is so often the case, messing up is a great way to learn! I'm sure I (and others) will pass this way again.


Add a RedCall user : trim() the input value

on https://redcall.minutis.croix-rouge.fr/admin/pegass/create-user
Searching for "43XX87P " wont find anything while
"43XX87P" will.
useful to trim with copy/paste from excel etc...

will be done at next release (middle of next week)


docs: guide to deploy Kubeapps using Helm

Kubeapps install guide for BKPR

we have to point out that these instructions are not sufficient for production clusters

You are right. That was a poor choice of words. I have edited the two instances and re-worded those to sections.

@vikram-bitnami Thanks for the excellent review. I've updated the PR to apply your suggestions. PTAL.

@vikram-bitnami can you please take a look?

updated!

@anguslees

(Eg: perhaps install tiller locked down with in-cluster TLS certs, so it's only exposed via kubeapps' auth/authz proxy)

Not sure if there's a chicken-egg problem here given BKPR is deployed using Helm (in this guide), but cert-manager should be able to manage the PKI assets for this for you (using a combination of the 'selfsigned' and 'ca' issuer type)

Thanks for the info @munnerz. We intend to experiment on this and come up with some elegant solution. At the same time in Helm 3 tiller will be removed so were keeping an eye out for those developments as well.


Update prod-docker.rst to add Git installation step.

Git must be installed to run git clone command.
wget is also forced to install in the first step, but I recommend creating a pre-step to install all these requirements before installation steps begin.

Thanks @bcalik for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

@pichouk Would you be able to take a look if the changes made here look correct?

LGTM :+1:


Use watch API to wait for autoinstantiated builds

Closes #216.
There's no timeout, we don't have it for any other watch methods either - should we?

Polling had 60 seconds, that sounds reasonable. Not sure about implementation, does pycurl support timeout? (or do we have to start new thread with timer?)

The last time I tested this the 60 seconds were sometimes not enough. PyCURL seems to support timeouts. Should I open a separate ticket for this?

I would stuff it into this one. But to be honest, I'm not sure we need the timeout (the build should be scheduled no matter what, right?). @twaugh Tim, what do you think?

I don't think it requires a timeout really.

LGTM, and it works fine here.

Also FWIW it took 117s for a build to get scheduled here just now.

117s for a build to get scheduled here just now.

oh my, +1 for not doing timeout then


chore(main): release 2.1.1

:robot: I have created a release beep boop
2.1.1 (2024-01-09)
Bug Fixes

main: termination sequence. (b679d44)
output: possibility of deadlock (4e3718f)
output: update interface comments (229662c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/northeye/chissoku/releases/tag/v2.1.1 :sunflower:


How many websockets are using when syncing multiple databases?

I'm writing a chat program, so I use one DB per session pattern. If using one DB per user pattern, the database will be more, and the program logic will be harder.
Every user has multiple sessions, so PouchDB will use multiple long polling, this is very resource consuming. I'm just curious about if this library will solve the problem?

Each PouchDB object you create that uses the socket adapter represents one single web socket. And yes, this sounds like something that can improve performance in your use case, assuming your connection to CouchDB/PouchDB on the server side is still not using too many HTTP connections. Hope that helps!

Thanks for your reply!


fix: article translate original

Fixed the issue where clicking on the original text after translation fails.

The PR environment is ready, please check the PR environment
[Attention]: This environment will be automatically cleaned up after 6 hours, please make sure to test it in time. If you have any questions, please contact the author of the PR or the community team.


ERROR TypeError: Cannot read property 'marktype' of undefined

We have this error when hovering over the svg. Looks like a tooltip issue? How can we resolve?

@giorgi-ghviniashvili Hmmm I might need to investigate it further.
Though, I'm taking personal time (some health issues + preparing for my graduation) so that I might not be able to look at this. Feel free to upload a PR or fork this repo for your purpose!

@yhoonkim thanks for the response, I will try to dive into the code and create a PR.

@yhoonkim thank you for the response, feel well and congrats on finishing up.


ActivityDefinition InteractionType

ActivityDefinition has been extended to include:
interactionType
correctResponsesPattern
choices
scale
target
step
This as been based on the work carried out by Paul Carpenter and is my first GitHub submission.

Sorry for the delay on this, I'm trying to give our OSS libs some love.

This code works.  I did follow @brianjmiller suggestion that constant names should be capitalized, and changed the code to use the defined constants.


Remote debugging of python steps with PyCharm or alternative

I am not sure if this is possible at all but my current workflow is too painful so it's worth asking. I have some python scripts inside a docker image that I call sequentially in different steps of a workflow. Locally they work and I can easily debug them, but when running in the cluster there are different sort of issues (configurations, paths, etc.) that are really hard to debug. What I would like to do is to connect using PyCharm (https://www.jetbrains.com/help/pycharm/remote-debugging-with-product.html#remote-debug-config) but I don't know if that's possible at all.
Any hints or recommendations? How do you debug your steps?

I use kubectl get $podname -o yaml to verify that things look ok, and then I start up the same code locally with those same settings, possibly using kubectl port-forward to get access to any services needed. But mostly the error is clear from looking at the yaml. Also, use python logging module, and log extensively. Use an environment variable to set the log-level of the logger so you can toggle debug mode in the workflow by an environment variable.


Where can I find open source video data?

Thanks for your great work, I want to know where can I find those generated videos mentioned in the paper?

Hi, we have organised and uploaded the sampled videos here: https://drive.google.com/drive/folders/13pH95aUN-hVgybUZJBx1e_08R6xhZs5X?usp=sharing
You can follow the example scripts here for automatic download and evaluation: https://github.com/Vchitect/VBench?tab=readme-ov-file#example-of-evaluating-videocrafter-10

Hi, more details regarding open-sourced video data is provided here: https://github.com/Vchitect/VBench/tree/master/sampled_videos#sampled-videos

Hi, we're closing this issue as it appears your questions have been addressed. However, feel free to open a new issue or reopen this one, if you have further questions or if anything else comes up related to this issue.


bugfix: add missing LICENSE in vendor

Signed-off-by: Wei Fu<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
github.com/prometheus/client_golang vendor package miss the LICENSE. We
should add it back.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
NONE
Ⅳ. Describe how to verify it
# -> before
# vagrant @ ubuntu-xenial in ~/go/src/github.com/alibaba/pouch on git:master o [13:19:02]
$ govendor status
The following packages are missing or modified locally:
        github.com/prometheus/client_golang
Error: status failed for 1 package(s)

# <- after
# vagrant @ ubuntu-xenial in ~/go/src/github.com/alibaba/pouch on git:bugfix_make_govendor_status_correct o [13:25:55]
$ govendor status

# vagrant @ ubuntu-xenial in ~/go/src/github.com/alibaba/pouch on git:bugfix_make_govendor_status_correct o [13:33:09]
$ echo $?
0

Ⅴ. Special notes for reviews

LGTM


evaluatio scripts

Hi, thanks for your great work.
Is there any plan to release the evaluation scripts for the results reported in the radar plot?

Yes - all evaluation code is in the evaluation repository: https://github.com/TRI-ML/vlm-evaluation
The repo has scripts for automating all the dataset downloads, and evaluating any of the Prismatic models


invalid flag: --export=malloc

Hi, when I use v0.4.0, I meet this error
github.com/wasilibs/nottinygc@v0.4.0/init.go:9:14: invalid flag: --export=malloc

How can I fix it?

Hi @rinfx - are you using TinyGo 0.28.x? v0.4.0 requires it as the minimum version and I believe using an older version would cause that error message.

Thanks! Switch to TinyGo 0.28.x can fix this problem


chore(main): release 2.0.1

:robot: I have created a release beep boop
2.0.1 (2024-11-23)
🐛 Bug Fixes

deps: bump @rollup/rollup-linux-x64-gnu from 4.24.3 to 4.24.4 (#422) (29046b8)
deps: bump @rollup/rollup-linux-x64-gnu from 4.24.4 to 4.25.0 (#434) (2b692d1)
deps: bump @rollup/rollup-linux-x64-gnu from 4.25.0 to 4.26.0 (#436) (0cca3a0)
deps: bump @rollup/rollup-linux-x64-gnu from 4.26.0 to 4.27.2 (#438) (6edc137)
deps: bump @rollup/rollup-linux-x64-gnu from 4.27.2 to 4.27.3 (#441) (bde5b09)
deps: bump @swc/core-linux-x64-gnu from 1.8.0 to 1.9.1 (#426) (30d9dae)
deps: bump @swc/core-linux-x64-gnu from 1.9.2 to 1.9.3 (#445) (956b31b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.0.1
:sunflower:


[feature-request] Create a production ready CardinalKit web portal.

Describe the solution you'd like
Must make use of #4 (including rule-sets).
Create a ReactJS-powered production ready CardinalKit web portal.
Here are some key features we're looking for:

It should be easy to set up (i.e. the only parts a potential team should worry about are replacing Firebase credentials and hosting the portal).
Two-factor authentication for access to data in portal (required for HIPAA compliance).
It should provide easy access to survey data.
Show the user customizable ways to visualize the survey data.
All survey consent forms can be aggregated into Firebase Storage from the app so clinicians can view them from the portal.
(nice to have) CRUD operations on the database through the portal (no need to ever touch the Firebase database through their website anymore!)
(nice to have) Customize user interface colors to match project.
(nice to have) Easy to understand documentation for customizing/setting up the portal.

See https://github.com/cs342/CardinalKit-CS342-Web
as an example!
NOTE: interest on features is gaged using reactions on this first post: e.g.  👍

Hi @varunshenoy  & @ssgutierrez42
I am interested in contributing to this feature. I need a few clarifications on this, can we connect on call to discuss this?

Hi @sivakb!
Are you available next Friday @ 2PM to connect (Sep 25th)? would love to chat!

@ssgutierrez42  Yes,I am available on Sep 25th. This is my email address<EMAIL_ADDRESS>Looking forward to meet you.

This is being addressed at: https://github.com/CardinalKit/CardinalKit-Web-Dashboard


Rename to genuinetools

[X] Releases are moved over
[ ] Needs reactivation of a travis project

thanks!


honk e2e: put tube vk into vk tree

VK verification got in and we're still trying to merge the initial honk e2e pass, we for now just make a special index for tube which always passes vk checks

I believe the tube vk is now being handled? @sirasistant


error: ';' expected in Generated classes

in an example generated class from concrete class called FeedModel, FeedModel$Container.java loadFromCursor() has several errors
there are several errors in these generated classes where a semicolon ';' is placed after 2 closing parenthesis ')' when there should only be 1 closing parenthesis.

Same problem here. dk why, when removed private worked perfectly

Not sure how that happened but what does your model look like? Just curious about the ; after 2 closing parenthesis issue.

i've removed usage of dbflow because of a few issues (generated files were causing compile time errors as described above, one to many relationships seemed to work sometimes but not always) but the example model causing issues was:
public class FeedModel extends BaseModel {
public interface Columns extends BaseColumns {
    String NAME = "col_name";
}
@Column(name = Columns._ID)
@PrimaryKey(autoincrement = true)
public long id;

@Column(name = Columns.NAME)
public String name;

}
i basically had to remove all getters/setters and change the private fields to public which was not fun.

my model:
@Table(databaseName = GAUDIDatabase.DB_NAME, tableName = AQL.ENDPOINT)
@TableEndpoint(name = AQL.ENDPOINT, contentProviderName = "GAUDIContentProvider")
@ModelContainer
public class AQL extends BaseModel{
public static final String ENDPOINT = "AQL";

@ContentUri(path = AQL.ENDPOINT,
        type = ContentUri.ContentType.VND_MULTIPLE + ENDPOINT)
public static Uri CONTENT_URI = ContentProviderUtils.buildUri(ENDPOINT);

public interface Columns {
    String AQL_ID = "aql_id";
    String AQL_NAME = "aql_name";
    String AQL_SERVER_ID = "aql_server_id";
    String AQL_TIMESTAMP = "aql_timestamp";
}

@Column(name = Columns.AQL_ID)
@PrimaryKey(autoincrement = true)
Long aql_id;

@Column(name = Columns.AQL_NAME)
String aql_name;

@Column(name = Columns.AQL_SERVER_ID)
Long server_id;

@Column(name = Columns.AQL_TIMESTAMP)
DateTime timestamp;

all were private fields...had to remove them cz of exception

If you want you're fields as private, you can add accessors of the fields in the format of:
fieldName => getFieldName(), setFieldName()
As long as those methods are accessible.

i'm sorry but i don't think it's possible... even with accessors my containers in toModel() using model.fieldName that gives me access error and still getting ; error in modelContainer.put(), it generates more ")" then should.
ex: modelContainer.put("aql_id",null));
Thank you!

Same issue here. It seems to always happen when I attempt using @ModelContainer

Same issue here, as OptimistNot stated, there is an unnecessary ")" that makes this compile error.

will check this out for 2.2

Are sure that it is fixed in in 2.2 ? I've just checked out this version and the same problem happen when I annotate a BaseModel class with @ModelContainer annotation.

Sorry guys! it will be in 2.2.1.

Hi, this bug still exists in 2.2.1 .
It occures with @ModelContainer annotation


PROC_ARST can not identify the async reset

Steps to reproduce the issue
Create a file called test.v with the following contents:
module top (clk, rst_n, in, sel, out);
input clk, rst_n, sel, in;
output out;

reg test[0:1];

always @(posedge clk or negedge rst_n) begin
  if (!rst_n)
    test[sel] <= 0;
  else
    test[sel] <= in;
end

assign out = test[0] & test[1];

endmodule

Then run yosys -p 'read_verilog test.v; proc;'.
( Yosys 0.17+0 (git sha1 d562bfd16, gcc 9.3.0-17ubuntu1~20.04 -fPIC -Os))
Expected behavior
Complete with no issues.
Actual behavior
The following error printed:
2.5. Executing PROC_ARST pass (detect async resets in processes).
Found async reset \rst_n in `\top.$proc$test.v:7$3'.
ERROR: Async reset \rst_n yields non-constant value 1'm for signal \test[0].

This is not a bug — this code assigns to a dynamically-selected bit in the "reset" branch of the if, and thus doesn't actually describe an async reset circuit.

This is not a bug — this code assigns to a dynamically-selected bit in the "reset" branch of the if, and thus doesn't actually describe an async reset circuit.

I have tried this code with Design Compiler and there is no errors reported.


Add an activity graph for each package

In order to measure the degree of improvement in metrics, we need to show how active the package is. If the metrics are not dropping even though the frequency of changes is high, it is a sign that it is being maintained.
To accomplish this, create the following action items:

[x] Add an activity graph on summary page
[x] Show top three contributors on summary page
[ ] Add an activity graph on package detail page
[ ] Show top three contributors on package detail page

Activity graph looks like this:

https://github.com/tier4/ros-metrics-reporter/graphs/contributors
Reference:
https://github.com/PyGithub/PyGithub
https://docs.github.com/en/rest/reference/projects#list-project-collaborators

https://github.com/hoxu/gitstats
https://github.com/morucci/repoxplorer

Test implementation for summary page

For gitlab support, consider to use gitlab-python when PyGithub throws an error.
https://python-gitlab.readthedocs.io/en/stable/


Is it possible to add classes and data attributes?

Hi,
I know you don't get automatic active class bindings as you with {{#link-to}} but is it possible to add class names and data attributes?

href-to is a helper, not a component. So just do like you normally would in HTML.
Go to home

Sorry - should have mentioned I have already tried this - the class and data attributes are not rendered when you do this.....

That's weird, they do render here. Could you post a full code snippet?

Hi,
Sorry - please bear with me - not sure how to show you everything short of putting the complete app up.
This is my mark up example
Students
The link displays and works fine, but if I inspect the html in Firefox, the class and data attributes are just missing ??
Is there anything else you need?

Hi,
Very sorry - I totally ignored the fact that you had the class and data attributes outside of the {{href}} block - apologies !!!!

No worries, had the exact same misunderstanding when I first refactored from link-to to href-to…


Customer feedback | az aks create | CLI example incorrect #4596

Describe the bug
This GitHub issue is being moved from MicrosoftDocs/azure-docs-cli GitHub issue 4596.

Type of issue
Typo
Feedback
The example in this document is incorrect.
az aks create -g MyResourceGroup -n MyMC --kubernetes-version 1.20.9 --node-vm-size VMSize --assign-identity "subscriptions/SubID/resourceGroups/RGName/providers/Microsoft.ManagedIdentity/userAssignedIdentities/myID" --enable-managed-identity --crg-id "subscriptions/SubID/resourceGroups/RGName/providers/Microsoft.ContainerService/CapacityReservationGroups/MyCRGID"
Page URL
https://learn.microsoft.com/en-us/cli/azure/aks?view=azure-cli-latest
Content source URL
https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/aks.yml
Screen print

Related command
az aks create
Errors
autogenerated example has syntax error, it is missing a /
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
Azure CLI 2.64.0
Additional context
No response

Thank you for opening this issue, we will look into it.


Export more TypeScript types at package root level

Is your feature request related to a problem? Please describe.
When we use near-api-js in TypeScript, we have to find out individual sub-directories inside the package to import types.
 import

 {Account, KeyPair, Near, utils}

 from 'near-api-js';
 import

 { InMemoryKeyStore, UnencryptedFileSystemKeyStore, }

 from 'near-api-js/lib/key_stores';
 import

 {NearConfig}

 from 'near-api-js/lib/near';
 import

 { FinalExecutionOutcome, JsonRpcProvider, }

 from 'near-api-js/lib/providers';
 import

 { ExecutionStatusBasic, FinalExecutionStatusBasic, Finality, }

 from 'near-api-js/lib/providers/provider';
 import

 { addKey, createAccount, fullAccessKey, transfer, }

 from 'near-api-js/lib/transaction';
 import

 {PublicKey}

 from 'near-api-js/lib/utils';
 import

 {parseNearAmount}

 from 'near-api-js/lib/utils/format';
 import

 {base_decode}

 from 'near-api-js/lib/utils/serialize';

Describe the solution you'd like
We would like to import such types from the package directly, such as:
 import

 {Account, KeyPair, Near, utils, InMemoryKeyStore, UnencryptedFileSystemKeyStore, NearConfig, FinalExecutionOutcome, JsonRpcProvider, }

 from 'near-api-js';

If you really want to separate such types, please use TypeScript namespaces:
 export namespace keys {
 export class PublicKey {}
 }

Describe alternatives you've considered
See my imports above.
Additional context
Add any other context or screenshots about the feature request here.

I'm not sure if it's a great idea to export all classes, interfaces, and functions to index.ts.
Have you considered using this approach?
https://github.com/ref-finance/ref-ui/blob/b077c129cc5169c8a5ddc08e95b42c31aac91a78/src/services/wrap-near.ts#L59

@volovyk-s I'm not proposing to flatten all classes/interfaces/functions to index.ts. As I stated in the issue, the primary concern is that the current typing forces users of near-api-js to understand the internal directory structure as we have to import types as follows:
import {NearConfig} from 'near-api-js/lib/near';
import {
  FinalExecutionOutcome,
  JsonRpcProvider,
} from 'near-api-js/lib/providers';
import {
  ExecutionStatusBasic,
  FinalExecutionStatusBasic,
  Finality,
} from 'near-api-js/lib/providers/provider';

There are two problems with the approach above:

It makes the internal path part of the api contract and any refactoring can lead to breaking changes
It makes it hard for VSCode to help import such types

If you prefer to have some hierarchy/isolation for typing, namespace is the way to go, such as:
export namespace utils {
  export function parseNearAmount(...) {};
  // or export const parseNearAmount = parseNearAmount;
}

@austinabell in your opinion what is the best practice?

@austinabell in your opinion what is the best practice?

Not quite sure what would be best tbh, probably best to discuss the tradeoffs. I haven't kept up to date with js API standards very closely recently but I will try to share an opinion.
My intuition would be that this could be a good approach, but it might depend on some internals I haven't looked into yet. @raymondfeng brings up a good point that these imports are based on the internal directory structure, which not only could be difficult for a new user to pick up but could mitigate the breaking changes if there ever was a need to refactor in the future.

Just to chime in, I'm not a fan of using namespaces in modern Typescript, and from my memory, the TS team doesn't recommend them in most cases for modern TS/JS either.  Namespaces are represented by objects in the global scope so you can end up with naming conflicts, and can make it impossible to do tree-shaking across the codebase.
Modules provide similar benefits as far as scoping types, but without any possibility of name conflicts at the global scope.
We can accomplish the same basic result as namespaces do, by doing a little re-structuring of the directory structure and creating a single file that imports our internal modules, and exports those for consumption by users at the root level of the package (e.g. index.js), to give them a consistent place to import types from that (a) doesn't include file paths and (b) doesn't create globals to do so.
To see what this looks like as far as a real codebase goes, rxjs is a pretty fine example...
https://github.com/ReactiveX/rxjs/blob/master/src/index.ts
This way consumers of the package can import whatever types they want, while those types are still defined in their individual modules, and no global namespace is declared for near-api-js types to 'live' in. :)

My first preference is to export public facing types at root level. Namespaces can be used for more isolations if needed. Please note that exported artifacts from a package does not create global conflicts as they be imported as a different name. For example:
import {Account as NearAccount} from 'near-api-js';

I think that flattening interfaces would be pretty good idea for both near-api-js and near-sdk-as.
Like there isn't that much stuff in these libraries to create separate subnamespaces, they exist mostly for historical reasons and generally just result in extra inconvenience. And if anything it's best to avoid name clashes for public classes / functions in one library in any case, as it would confuse users.
I pretty much would prefer to import stuff like formatNearAmount directly on top level instead of having to look where it is located every time.

Hey everyone, I'm currently conducting a clean-up on the issues of NAJ. Is exporting types at package root level still interesting to NAJ? Please let me know if that's the case; otherwise, I'll proceed to close the issue in 7 days.
cc: @frol

I believe that after the great effort of modularizing NAJ, we should not go back into flattening everything to near-api-js. Instead, let’s focus on making sure that the individual modules make sense to use and have intuitive APIs and types provided next to the functions/classes that users will use or export them at the top level of the module.

Closing issue as it's not relevant anymore


soapui: 5.7.0 -> 5.7.2

Description of changes
Updated to version 5.7.2
The url of the tar.gz changed, is not anymore in s3.amazonaws.com
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Welcome to nixpkgs! Generally we try to keep a clean git history, which means during PR revisions we prefer you revise the commits and force push. Please squash the two commits

@pbsds Do you know how long can take a PR to be merged?

Thanks for the ping. In the future feel free to request reviews either thorugh the github ui, the Discourse thread or on Matrix. :)


Remove GOV.UK CDN Logs Monitor

This app was previously disabled in https://github.com/alphagov/govuk-puppet/pull/7739
This completes the process by deleting the associated puppet files and
configuration of the application.

@kevindew I pushed a fix for the test failure so this can go out. Can you thumb this please, then I'll merge and deploy.

Thanks for that @boffbowsh - I can't approve since I opened this so I'll just merge it.


tailwind.config.js file created but ignored

Changes to the tailwind.config.js file do not seem to be applied, even after restarting dev server (using 1.1.2). Here are the contents of my config:
module.exports = {
  theme: {},
  variants: {
    backgroundColor: ['responsive', 'hover', 'focus', 'group-hover'],
    visiblility:  ['responsive', 'hover', 'focus', 'group-hover'],
    cursor: ['responsive', 'hover', 'focus', 'group-hover'],
    display: ['responsive', 'hover', 'focus', 'group-hover'],
  },
  plugins: [],
  screens: {
    xs: '380px',
    sm: '640px',
    md: '768px',
    lg: '1024px',
    xl: '1280px'
  },
}

And now it's working... without change.

Hey @chadananda, I'm having the same issue. Any ideas why or what made it work?

Having same issue here, can't find anything

If you created config file while --watching, you should rerun that command.


Fixed duplicate RECURRENCE-ID when expanding events

Fixes https://github.com/python-caldav/caldav/issues/394
There is an issue with Xandikos: when searching for the events a 500 Internal Server Error is returned. This might need some investigation.

  File "/home/runner/work/caldav/caldav/.tox/py/lib/python3.10/site-packages/xandikos/icalendar.py", line 1039, in asutc
153
    return dt.astimezone(pytz.utc).replace(tzinfo=None)
154
AttributeError: 'datetime.date' object has no attribute 'astimezone'
155

This seems to be a problem with Xandikos.

@jelmer - do you have time to look into the broken test here?

Sorry for forgetting about this one for so long.  Now I can't figure out how to rerun the tests.  Hm.

So the xandikos-problem is with your test code, it is using dates for DTSTART, while datetime probably would be more appropriate.  At the other hand, date is a valid type for DTSTART, so perhaps the problem is with xandikos.  Hm.

I have cherry-picked the commit and worked around the xandikos issues.  I will raise an issue on the xandikos issue tracker about the breakage.


Improvement: logger middleware to the javascript boilerplate

Hello :wave:
There should be a logger middleware implemented with the javascript boilerplate.

You can choose from any popular logging libraries like morgan or winston.
logs should be stored on the /logs directory, in the project root.

Please refer to CONTRIBUTING.md for setting up the project on your local machine.
Feel free to ask for help, clarification, or doubts in the comments!

hi , I would like  to help @gunvantsr

Hey, @ryuusama09 Please go ahead.

Hi , I like to work on this @gunvantsr

Hi @gunvantsr , I'd like to work on this

Hello @Harshwardhan-Patil, you can take this up!

Can I help?

Hi @gunvantsr , I would like to work on this


Inferentia lab in AIML module

What this PR does / why we need it:
Create the lab for Inferentia in the AIML module.
Quality checks

[x] My content adheres to the style guidelines
[x] I ran make test or make e2e-test and it was successful

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The PR has been copied from #182 to enable he new netlify build automation.

Closed for replacing with #433 .


How to download files from blob after CreateSPAsyncReadJob Job is completed

I have gone through Migration Asynchronous Read API  and able to successfully create CreateSPAsyncReadJob which stays in "Queued" state for sometime than turns its state to "Processing" and finally returns its state as "None" when completed.
Sample:
Refer Thread
However there is not much information about how we download the Blobs generated and placed by the completed job.
Have tried some sample code as depicted below but it throws Microsoft.WindowsAzure.Storage.StorageException: 'The remote server returned an error: (403) Forbidden.'
        using (var clientContext = new ClientContext(siteUrl))
        {
            var securePassword = new SecureString();
            foreach (var c in password) securePassword.AppendChar(c);
            clientContext.Credentials = new SharePointOnlineCredentials(userName, securePassword);

            var result = clientContext.Site.CreateSPAsyncReadJob($"{siteUrl}/Lists/SampleGenericList", new AsyncReadOptions() { });
            clientContext.ExecuteQuery();
            var jobData = result[0];

            MigrationJobState state;
            do
            {
                Thread.Sleep(5000);
                var status = clientContext.Site.GetMigrationJobStatus(jobData.JobId);
                clientContext.ExecuteQuery();
                state = status.Value;
            } while (state != MigrationJobState.None);

            var blobUri = new Uri(jobData.ManifestContainerUri);
            var blobAccountName = blobUri.Host.Split('.')[0];
            var containerName = blobUri.LocalPath.Replace("/", "");

            var storageCredentials = new StorageCredentials(blobAccountName, jobData.EncryptionKey);
            var cloudStorageAccount = new CloudStorageAccount(storageCredentials, true);
            var cloudBlobClient = cloudStorageAccount.CreateCloudBlobClient();
            var containerReference = cloudBlobClient.GetContainerReference(containerName);
            var blobs = containerReference.ListBlobs();
            List<string> blobNames = blobs.OfType<CloudBlockBlob>().Select(x => x.Name).ToList();
            foreach (var blob in blobNames)
            {
                //Download blob
            }
        }

Any direction would be of great help?
Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 39174cb2-71d0-b58a-6edc-3fcabd922e8e
Version Independent ID: fae882c6-c489-6992-a343-6f0d657fb212
Content: Migration Asynchronous Read API
Content Source: migration/SPO-asynchronous-read-API.md
Product: sharepoint-server-itpro
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@upadhyayadhaval thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.
@officedocsbot close

@officedocsbot close

Would it maybe be an option you update the documentation as there is nothing anywhere about this topic. Then we dont have to create a support case all of us?


WARN level logging when text/fallback is missing in chat.* API method calls

Hello, i am currently using Node.js with TypeScript with slack-bolt.
"@slack/bolt": "^3.4.0",
And keep getting this WARN logs in DataDog, it's weird because in chat.postMessage i do have the text argument.
The second log is related to fallback argument, i didn't see this argument in the Slack API documentation, though i tried adding it, but this warning still comes up.

[WARN]  bolt-app The text argument is missing in the request payload for a chat.postMessage call - It's a best practice to always provide a text argument when posting a message. The text is used in places where the content cannot be rendered such as: system push notifications, assistive technology such as screen readers, etc.

[WARN]  bolt-app The fallback argument is missing in the request payload for a chat.postMessage call - It's a best practice to always provide a fallback argument when posting a message. The fallback is used in places where the content cannot be rendered such as: system push notifications, assistive technology such as screen readers, etc.
`

How can i fix this so the warnings don't show up?
Also, i would like to customize the logs, currently the WARN in Datadog is labelled with red color and treated as an error log, i would like to colorize it as yellow, probably add my custom logger (winston/bunyan) to log this? How can i achieve this in slack-bolt?
Thanks in advance.

Hi @erkand-imeri, thanks for asking the question!
The text warning means that your chat.* API method call has only blocks in the parameters. For better user experience, having top-level text as well is highly recommended.
const response = await client.chat.postMessage({
  channel: "C123",
  text: "fallback text message",
  blocks: [ ... ],
});

The fallback warning means that any of your attachments do not have its fallback field for it. For the same reason, having fallback is highly recommended. Can you check the payload that you use for chat.* method API calls in your app?
const response = await client.chat.postMessage({
  channel: "C123",
  text: "fallback text message",
  attachments: [{ blocks: [], fallback: "attachment fallback" }],
});

For your information, here is the logic of the detection: https://github.com/slackapi/node-slack-sdk/blob/%40slack/web-api%406.5.1/packages/web-api/src/WebClient.ts#L643-L664
As for the logger customization, please refer to this document section: https://slack.dev/bolt-js/concepts#logging
You can click "Sending log output somewhere besides the console" to view more details.
Also, the interface to implement is available here: https://github.com/slackapi/node-slack-sdk/blob/%40slack/logger%403.0.0/packages/logger/src/index.ts#L14-L63
I hope this helps!

Hi @seratch thanks a lot for the explanation.

When i add the fallback text then the text is getting displayed on slack thread something which i don't want to, if i leave it as a parameter yet as an empty string i still do get the message.

As for the customization, i tried doing it that, but since i am using TypeScript

What should i do with getLevel, setName and setLevel, i mean i can put ignore for setName and setLevel but getLevel will give me an error no matter what?
    debug: (...msgs) => { logWritable.write('debug: ' + JSON.stringify(msgs)); },     info: (...msgs) => { logWritable.write('info: ' + JSON.stringify(msgs)); },     warn: (...msgs) => { logWritable.write('warn: ' + JSON.stringify(msgs)); },     error: (...msgs) => { logWritable.write('error: ' + JSON.stringify(msgs)); },     setLevel: (level) => { },     getLevel: () => { },     setName: (name) => { },

When i add the fallback text then the text is getting displayed on slack thread something which i don't want to, if i leave it as a parameter yet as an empty string i still do get the message.

You can set " " (a whitespace char in the string) if you don't want to set any meaningful text there.

As for the customization, i tried doing it that, but since i am using TypeScript

You may want to do something similar to the default console logger: https://github.com/slackapi/node-slack-sdk/blob/%40slack/logger%403.0.0/packages/logger/src/index.ts#L68-L151

Thanks @seratch , that worked it looks, but i get the (edited) notification, but it's still a lot better than actual text.

Great to hear that it worked for you! Let us close this issue now.


Document internal tables partitioning scripts

As discussed in https://github.com/cockroachdb/docs/issues/2957, we want to document internal tables for partitioning use cases.
It is not documented but users can run this query for tables:
Select * from crdb_internal.partitions
join crdb_internal.tables on partitions.table_id=tables.table_id
where tables.name='mytable'

It is also not documented but users can run this query for databases:
Select partitions.name
from crdb_internal.partitions
join crdb_internal.tables on partitions.table_id=tables.table_id
where database_name='mydb

We should add this to our various docs including https://www.cockroachlabs.com/docs/v19.1/demo-geo-partitioning.html#step-3-provision-vms

@awoods187, can you add some context about why and when you would use this?

Developers often set up scripts. We know of several customers who wanted to know how to capture the status of partitioning as a part of scripting their environment. Instead of using the new show statements, as they are porcelain, we will recommend they gain information on partitions from this source.

This was fixed by #4978.


early_stopping() min_delta kwarg unexpected

Description
In lightgbm 3.3.5, passing min_delta with a float value should work, but is throwing an "unexpected keyword argument" error. Removing the kwarg works, but the default value is 0.0 which means early stopping won't occur in that case.
Reproducible example
import lightgbm

print(lightgbm.__version__)  # confirm version is 3.3.5

lightgbm.early_stopping(
                    stopping_rounds=100,
                    first_metric_only=False,
                    verbose=True,
                    min_delta=0.00001,
                )

Returns:
3.3.5
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Untitled-1.ipynb Cell 1 in <cell line: 5>()
      [1](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=0) import lightgbm
      [3](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=2) print(lightgbm.__version__)
----> [5](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=4) lightgbm.early_stopping(
      [6](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=5)                     stopping_rounds=100,
      [7](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=6)                     first_metric_only=False,
      [8](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=7)                     verbose=True,
      [9](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=8)                     min_delta=0.00001,
     [10](vscode-notebook-cell:Untitled-1.ipynb?jupyter-notebook#W0sdW50aXRsZWQ%3D?line=9)                 )

TypeError: early_stopping() got an unexpected keyword argument 'min_delta'

Environment info
import platform as pf
pf.machine(), pf.version(), pf.platform(), pf.uname(), pf.system(), pf.processor()

('x86_64',
 '#1 SMP Debian 5.10.162-1 (2023-01-21)',
 'Linux-5.10.0-21-cloud-amd64-x86_64-with-glibc2.29',
 uname_result(system='Linux', node='kmb', release='5.10.0-21-cloud-amd64', version='#1 SMP Debian 5.10.162-1 (2023-01-21)', machine='x86_64', processor=''),
 'Linux',
 '')

LightGBM version or commit hash: 3.3.5
Command(s) you used to install LightGBM
pip install lightgbm

virtualenv on Linux cloudvm, tested in both python scripts and Jupyter notebooks with lightgbm 3.3.5.
Additional Comments

None.

Hi @kylebeni, thanks for using LightGBM. You're probably looking at the latest version of the documentation. If you look at v3.3.5 you can see that min_delta isn't present in that release, it will be available in the next major release (v4.0.0) which you can track in #5153.

Thanks for the quick response @jmoralez. In the future the delineation between dev (latest) and prod docs could be clearer.
It isn't obvious that "latest" means "unreleased". Thanks.


Sqitch connecting to local instead of remote host

When trying sqitch deploy on a remote host (Google Cloud SQL Postgresql instance) I'm getting the following error:
$ sqitch deploy
Adding registry tables to dev
psql: FATAL:  database "dev" does not exist
"psql" unexpectedly returned exit value 2

& no sqitch schema is created on the database.
If I create a local database of the same name, I get:
$ sqitch deploy
Adding registry tables to dev
ERROR:  relation "changes" does not exist

& sqitch schema is created on the local database instead of the remote.
connection uri is:
db:pg:///dev?hostaddr=[IP Address]&host=[Project]:[Instance]&user=admin&sslmode=verify-full&sslrootcert=../certs/server-ca.pem&sslcert=../certs/client-cert.pem&sslkey=../certs/client-key.pem
Server logs from the deploy only show a failed select query on table changes (which does not exist), so it is clear connection is happening properly at some point, just not when creating the initial schema.
$ sqitch --version
sqitch (App::Sqitch) 0.9995

Try db:pg://[IP Address]/dev.

Solved (sort of):
Putting db:pg://[User]@[Host] (with a url encoded ':') and adding the host/ip to /etc/hosts seems to be the only thing that works.

Thanks for the prompt reply.

you…you have a colon in the host name? Yeash.


how to debug bluetooth problem for mybmw connect to car's app function

Hi,
Today, I upgraded my phone to Android 14 QPR3,
build from LineageOS 21.
After the upgrade, the MyBMW app can no longer connect to the car's app function.
The app checkbox is not shown checked in the car. (All permissions are set correctly for the MyBMW app, and it worked well before the upgrade.)
It always stuck on open app if necessary when I repair phone on car.
Before the upgrade, everything worked well.
So, it may be a problem with the Bluetooth in the new Android system.
How can I debug the problem? Is there any checkpoint for MyBMW to make the app function connect to the car?
Thanks.

Following. Same issue with my S24 Ultra

Unfortunately, the MyBMW>Car connection is relatively opaque, and I'm not sure what the car looks for before showing the Apps checkbox. The only suggestion I've heard work for other people is to enable all the permissions and background execution options (especially for Samsung), which it sounds like you've done. I've heard other people work well on Android 14 after setting these settings, and my own Sony phone hasn't had any issue with Android 14.
If you'd like to debug further, I have notes on how to patch the Connected/MyBMW apps to enable debug logging, which can be combined with decompilation to try to trace through what it's doing. I haven't experienced the issue of the Apps checkbox not showing, so I can't help in this regard.

Unfortunately, the MyBMW>Car connection is relatively opaque, and I'm not sure what the car looks for before showing the Apps checkbox. The only suggestion I've heard works for other people is to enable all the permissions and background execution options (especially for Samsung), which it sounds like you've done. I've heard other people work well on Android 14 after setting these settings, and my own Sony phone hasn't had any issues with Android 14.

I know the permission setting problem on Android 14, when I first upgraded to Android 14 beta I found it needed to give nearby device permission to the mybmw app which may be needed by connecting a Bluetooth device from the mybmw app.
Then all things going well.

If you'd like to debug further, I have notes on how to patch the Connected/MyBMW apps to enable debug logging, which can be combined with decompilation to try to trace through what it's doing. I haven't experienced the issue of the Apps checkbox not showing, so I can't help in this regard.

Thanks for the info.
Cuz I'm using a self-build AOSP which may be the QPR3 version missing some Bluetooth functions.
I'm going to instigate the communication between mybmw and car and will come back if any progress.

maybe useful info from Logcat:
1719818404.550  1002  2610  3070 W ActiveDeviceManager: A2DP connection timeout. Activate HFP for XX:XX:XX:XX:D9:4F

is there any way to bypass the login view of BMW Connected_<IP_ADDRESS>49_Apkpure.apk
I followed the info and opened the log, but the app needed a login to open.

BMW Connected will connect to the car even without logging in, and even add its old Audioplayer and Calendar apps.

BMW Connected will connect to the car even without logging in, and even add its old Audioplayer and Calendar apps.

yes,
and I just confirm that the old 'BMW Connected_<IP_ADDRESS>49_Apkpure.apk' can works without problem.
so it maybe new MyBMW and Android 14 QPR3 has something wrong.

Oh sorry, I missed this email:
The only negative effect I've seen is that older headunits get confused about both BMW Connected and MyBMW connecting at the same time and crash and restart. I've mainly seen this with the USB connection and IDrive4, but perhaps they are not correlated together.

I know the permission setting problem on Android 14, when I first upgraded to Android 14 beta I found it needed to give nearby device permission to the mybmw app which may be needed by connecting a Bluetooth device from the mybmw app. Then all things going well.

NB! The newest release of My BMW (4.9.2 release date Sept 11 2024) resets all the given permissions for the app - and "Nearby devices" permission brakes the AAIdrive connection !!! Looks like all affected users need to manually give permission for Nearby devices in order to get the "App" show up in the iDrive bluetooth device properties, for making the AAIdrive work again

I'm not sure if an app upgrade would reset permissions, as I understand the Android upgrade process.
I do know Android has a feature to clear permissions of apps that aren't used in a long time, for "security". Perhaps this is what people have been experiencing?

I'm not sure if an app upgrade would reset permissions, as I understand the Android upgrade process. I do know Android has a feature to clear permissions of apps that aren't used in a long time, for "security". Perhaps this is what people have been experiencing?

Yes, it might have been a Android security updatethat did it as well - anyhow, I discovered this recently, so please check this if you encounter problems.  (it might be that timeout you're referring to as well - although I doubt that, since I use My BMW every day)


Update and rename content/github/collaborating-with-issues-and-pull-r…

…equests/about-pull-requests.md to محتوا/گیتاب/همکاری با درخواستها و موضوعات/about-pull-requests.md

Why:
Closes [issue link]

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for the deploy-to-heroku link in your pull request, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Thank you for your pull request. We are unable to accept this change at this time.


Convert Project to Maven

This PR converts the project to maven with support to be able to publish to Maven Central

I signed it!

@NeilFraser I have taken care of comments.
About

Also, are all those sections of pom.xml needed? There seems to be a lot of boilerplate copied from some other project. The smaller the footprint the less likely it is to break.

You are right. I copied most of the template from my another project google-analytics-java. I copied with intention that sometime you would want to push this to library to maven central so that folks can easily consume it.
However, Google may have different way of publishing to central (guava team might know about it). if that is the case, I can remove all that additional sections.

@NeilFraser I have converted your tests to Junit based tests so that you can just execute them with command mvn test, which will take care to setup classpath etc.

Converting the tests to JUnit (or any other test framework) will increase maintainability, not decrease. It is easier to spot issues given the fact that Most IDEs are optimized for it. Also the error message would be more readable; assertEqual would print both the expected value and the actual value, useful if we have Continuous Integration enabled in future. It may need discussion about which one to use, but using a test framework is a good idea overall, for tests larger than 10 testcases.

@franklinyu Reason nexus related stuff is there with assumption that Neil would want to push this library to Maven Central so that is can be easily adopted by others.
However since this google, they might have another internal approach they are following for other google libs like guava. If Neil wants to follow internal approach, I will remove all Nexus related stuff.

Not familiar with the plugin, I assume that it automates pushing to Maven Central. However this repository does not seems to revolve that fast. I would suggest switching to Nexus only when manually pushing to Maven Central becomes a burden over @NeilFraser.

Was there any development on this issue? I would be extremely advantageous to have the possibility to use this project via Maven Central.

Removed the nexus related config from pom.xml
Removed the conflicts with latest
Integrated latest diff_match_patch_test.java to run as Junit tests

Whats stopping this from getting merged ?

Please rebase your branch and also I would highly recommend upgrading the java version from 1.6 to atleast 1.8.

About 1.8: If code is not using any 1.8 features, why make it that version as minimum?
About rebasing: Unless Neil confirms that he will review and merge, there is no point in rebasing as he updates the code, it will get out of sync.

Its not just about if the code does not use any 1.8 features. java 6 has had pretty much end of life for public updates a long time ago. Open source code does not have to keep using a version which will not have public updates anymore.

Also,
diff_match_patch.java:[1813,28]
Use @Deprecated with the javadoc @deprecated
The javadoc element only marks it as deprecated where as the java annotation lets the compiler know, so any consumer using that method gets to know if they are using a deprecated function/method.

@mithuns I'd understand the merits of 1.8. But it is upto Neil to make that call.

Ping? @NeilFraser

And? Does the library now show up in maven central which is the objective if I understand right?

@WolfgangFahl How is the group ID fun.mike related to @NeilFraser?

@franklinyu
you asked: How is the group ID fun.mike related to @NeilFraser?
Answer: I have no clue. My question was about the compatibility. I did not find the source repository for that release anywhere but the class file content looks somewhat o.k. I am using that release for a workaround of this issue not being closed yet and an "official" maven release being available.

Any update on this? I'd be happy to take a look if there is anything else that's needed. @NeilFraser @brsanthu

I'd love to see this merged. It would improve our adoption of this project.

Please let me know if you plan on publishing regular releases to Maven Central. If not, I will update https://bitbucket.org/cowwoc/google-diff-match-patch/ to do exactly that. I already published the google-code version a few years back but now there are requests to publish your newer releases. Obviously it's better for you to handle this in-house than have someone do this externally.
Let me know...

For those of you who are interested, I just pushed the latest version of this code to Maven Central under the name org.bitbucket.cowwoc:diff-match-patch:1.2
See https://bitbucket.org/cowwoc/google-diff-match-patch/ for the project website.
NOTE: I am not developing this library. I am just releasing the latest code to Maven Central.

Thanks @cowwoc for your efforts! It's great to have this library available as a regular gradle dependency - hopefully this PR moves forward but until then I've reviewed the cowwoc re-package and it appears to be a clean release of this code. Cheers

@NeilFraser we're waiting on your response

The author doesn’t seem interested any longer. I’m using a fork: https://github.com/java-diff-utils/java-diff-utils

@NeilFraser - A bit of a tragedy is happening here! If this Maven PR could only merge, it would mean your labor of love would become instantly more appealing to a bigger swath of the Internet and your code would be more widely used and appreciated. But instead it's about to undergo a fork, splintering the user base and reducing the impact of your mainline efforts here. You've already done some work to review this PR-- can't we bring it back to life?

What's the point of maintaining a java library if you're not going to make it accessible to the majority of java developers?

Sad this is never taken care of ...

@cowwoc Thanks for submitting the code to Maven.
Would you (or anyone else) in this thread interested in the following modification: change the input type from String to CharacterSequence. It might be helpful when you want to compare two really large strings (say 2GB each) and you want to implement some logic hidden by CharacterSequence interface that allows you to offload such a string to a file for example and then use in-memory-mapped file to access it. Support of multi-byte characters in this case would be tricky as it would need an intermediate translation layer between file bytes and characters sequence (I don't have such implementation), but for one-byte charsets like ISO-8859-1 the implementation is rather simple (my case).
If String is passed instead of CharacterSequence there will be some minor performance drop, but I don't expect much.
Let me know if somebody is interested in the patch / code.
For above reason I cannot use java-diff-utils project, as in line 150  provided that String original has size X, I would expect that List<String> origList would have at least size X, and Patch<String> patch might also have size X depending on how many are differences and what JVM policy on String.substring() (create a lightweight copy or clone part of internal character buffer). So we have so heavy memory pressure...

@dmak did you mean CharSequence? If not, to what class are you referring? Is there a link?

Support of multi-byte characters in this case would be tricky as it would need an intermediate translation layer between file bytes and characters sequence

diff-match-patch doesn't currently specify input encoding. to that end, yes, it's complicated because if you feed it input with different code unit lengths then you will get different output and data corruption if you mix the deltas between systems. (for example, you can't apply in Java a delta generated from Python3 if there are surrogate-pairs in the input stream).

but for one-byte charsets like ISO-8859-1 the implementation is rather simple (my case).

one-byte charsets like ISO-8859-1 are also problematic for this library until some specification on index/length units are made (I proposed one such backwards-compatible declaration in #83) for the output deltas. this is more or less to say, I agree with you that introducing a character stream reveals the complexity of representing human text, though technically it doesn't introduce complexity that wasn't already present-yet-hidden.

You're right, I meant CharSequence – just corrected my post above. And your post lead me to another idea that maybe writing such a layer between byte and character translation would be more difficult rather than first doing the diff on byte level and then mutating (extending / shrinking) a bit the resulting diffs so that they do not eventually stop in the middle of multi-byte sequence so that every range in the result could be converted to a string without a problem.
Anyway, just an idea that passing String into the algorithm is too limiting as this class cannot be extended / it's behavior cannot be changed, and converting string to an array of characters is too wasteful. So CharSequence is a good trade off, as it could be String, StringBuilder or something else backed up by a file or another storage.


Remove member variables for number of states

This PR removes the member variables representing the number of requested states in class Nfa and the number of states present in Delta.
It was decided that we will represent the number of states in an automaton with a union of initial, final and delta states, where delta states mean the number of allocated Post. Each state used in delta (either as a source or a target state) will have allocated its Post in delta. Therefore, the number of allocated Post in delta corresponds to the number of states in the whole delta (both source and target states).
Furthermore, this PR renames member variable post of type vector of Post to posts which now correctly explains, that the variable holds, indeed, multiple Post classes, not just one Post. This was a typo done in the past, and it seems that no one noticed.
Mata::Nfa::Delta::posts_size() now represents the number of states in the whole delta, including both source and target states. Therefore, the function does no longer need to have a unique name different from the number of states in delta. This function was renamed previously to posts_size() only because it represented only the number of source states. Now, when it represents the number of all states in delta again, it can be renamed back to its original meaning.
Resolves #189 and #167.

I optimized the check for exact identity of automata. I will merge the PR soon, unless anyone has anything to add.

I think the reimplementation is better. I would do it differently though (bool init = lhs.get() == rhs.get(); if (!init) { return false;} ...), but it is up to taste I guess.
I also wonder, if lhs.trans == rhs.trans is already implemented efficiently, and could be simplified to just return lhs.trans == rhs.trans.

I also wonder, if lhs.trans == rhs.trans is already implemented efficiently, and could be simplified to just return lhs.trans == rhs.trans.

We do not have any such method. Apparently, there is no need for one such as this. You do not compare for equality of deltas usually, you compare whether the languages of automata are equal.

I think the reimplementation is better. I would do it differently though (bool init = lhs.get() == rhs.get(); if (!init) { return false;} ...), but it is up to taste I guess.

I can rewrite it as you suggest. I contemplated doing that earlier, too, but I decided that the level of indentation was still acceptable. Nevertheless, we do not strive for a single exit point here, so both approaches are equivalent. It is up to us to decide what is more readable.

I also wonder, if lhs.trans == rhs.trans is already implemented efficiently, and could be simplified to just return lhs.trans == rhs.trans.

We do not have any such method. Apparently, there is no need for one such as this. You do not compare for equality of deltas usually, you compare whether the languages of automata are equal. But regardless, this function is meant mainly for testing purposes. We can allow ourselves some maybe-not-so-efficient operations. That is, there is no need to implement this function only for exact identity check.

Oh, it is fine then.

I think the reimplementation is better. I would do it differently though (bool init = lhs.get() == rhs.get(); if (!init) { return false;} ...), but it is up to taste I guess.

I can rewrite it as you suggest. I contemplated doing that earlier, too, but I decided that the level of indentation was still acceptable. Nevertheless, we do not strive for a single exit point here, so both approaches are equivalent. It is up to us to decide what is more readable.

I would advise to get a second opinion from someone other (maybe even Lukas)

Resolved in PM.


rss template: Set correct lastBuildDate

The embedded rss template does not use the current date for the lastBuildDate:
Example: https://gohugo.io/index.xml:
<rss xmlns:atom="http://www.w3.org/2005/Atom" version="2.0">
<channel>
<title>Hugo – The world’s fastest framework for building websites</title>
<link>https://gohugo.io/</link>
<description>Recent Hugo news from gohugo.io</description>
<generator>Hugo -- gohugo.io</generator>
<language>en-us</language>
<lastBuildDate>Thu, 02 Mar 2017 12:00:00 -0500</lastBuildDate>
...

Sorry for all these commits, tested and confirmed ok now.


[Feature Request]: support farm runtime code split info a single file

What problem does this feature solve?
When I develop a browser extension, inline scripts are not permitted. after I write a plugin to transform HTML, a new issue emerges. Perhaps there should be a correct registration order. Maybe farm should be supported by default?.
inline error

transform html
{
      name: "transform-html",
      configResolved(config) {
        targetpath = path.join(config.compilation?.root ?? process.cwd(), config.compilation?.output?.path ?? 'dist');
      },

      buildStart: {
        executor() {
          fileMap = {}
        }
      },
      transformHtml: {
        order: 2,
        async executor({ htmlResource }) {
          const htmlCode = Buffer.from(htmlResource.bytes).toString();

          const reg = /<script>([\s\S]*?)<\/script>/g
          const content = reg.exec(htmlCode);

          const targetJsFile = path.join(targetpath, path.dirname(htmlResource.info?.name ?? ''), path.parse(htmlResource.info?.name ?? '').name + '-inject-runtime.js')
          const name = path.parse(targetJsFile).base
          fileMap[targetJsFile] = content?.[1] as string

          const newHtmlCode = htmlCode.replace(reg, `<script src="./${name}"></script>`);
          htmlResource.bytes = [...Buffer.from(newHtmlCode)];
          return htmlResource;
        }
      },
      writeResources: {
        executor() {
          Object.keys(fileMap).forEach(filepath => {
            fs.ensureDir(path.dirname(filepath))
            fs.writeFile(filepath, fileMap[filepath])
          })
        }
      },
    }

new issue

What does the proposed API look like?
{
  html: {
   moduleRegister: "inline" | "single"
 }
}

Can runtime.isolate property solve the problem?

Can runtime.isolate property solve the problem?

It seems not work。 There still inline scripts in html template


Literals don't accept inputs with whitespace when str_strip_whitespace is True

Initial Checks

[x] I confirm that I'm using Pydantic V2

Description
When I use a string Literal like suit: Literal["Clubs", "Diamonds", "Hearts", "Spades"] and set str_strip_whitespace=True, I expect validation to remove whitespace before checking the literal value.
However, validation yields this error when Literal input includes whitespace:
1 validation error for Card
suit
  Input should be 'Clubs', 'Diamonds', 'Hearts' or 'Spades' [type=literal_error, input_value='  Spades  ', input_type=str]
    For further information visit https://errors.pydantic.dev/2.5/v/literal_error

May be related to #8577.
Example Code
from typing import Literal
from pydantic import BaseModel, ConfigDict, ValidationError

class Card(BaseModel):
    model_config = ConfigDict(str_strip_whitespace=True)
    suit: Literal["Clubs", "Diamonds", "Hearts", "Spades"]

try:
    card = Card(suit="  Spades  ")
except ValidationError as exc:
    print(exc)

Python, Pydantic & OS Version
pydantic version: 2.5.3
        pydantic-core version: 2.14.6
          pydantic-core build: profile=release pgo=true
                 install path: /Users/me/dev/other-tools/kafka/shaw_datafile/.venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.6 (v3.11.6:8b6ee5ba3b, Oct  2 2023, 11:18:21) [Clang 13.0.0 (clang-13<IP_ADDRESS>)]
                     platform: macOS-13.6-arm64-arm-64bit
             related packages: typing_extensions-4.9.0 pydantic-extra-types-2.4.1 pydantic-settings-2.1.0 email-validator-2.1.0.post1

@davidpeckham,
Indeed, the logic for stripping whitespace occurs after the validation of a Literal value.
This isn't something that we can change easily with a deprecation warning, as it's a subtle change that would be breaking for a lot of users. Thus, this isn't something that's on our roadmap to change.
Luckily, you can write a pretty simple before validator to do what you're asking:
from typing import Annotated, Literal

from pydantic import BaseModel, BeforeValidator, ValidationError

StrippedSuit = Annotated[str, BeforeValidator(lambda v: v.strip())]

class Card(BaseModel):
    suit: StrippedSuit

card = Card(suit="  Spades  ")
print(repr(card))
#> Card(suit='Spades')

Marking this as closed 👍 bc the change isn't planned.


Drain path cleanup

Moving more code into microservice/handlenodedrain.go files.  Docs drain path cleanup.

Does this fix any bugs or just a clean up code ? Asking if we need this for next build to QA


chore: fix typos/grammar mistakes in various places

Hi, I appreciate your work and like the idea of managing game dependencies with a CLI tool. I was thinking about writing a simple protonup "clone" in Rust and then I found your repository. I'm still just a beginner when it comes to Rust, but maybe I can contribute in the future.
In this commit I just fixed some spelling/grammar mistakes that I found by looking through the source code. I hope it helps :grinning:
In main.rs I also found these lines:
SubCommand::with_name("proton")
    .about("ProtonGE gestion (Install / Remove / List)")

I don't know what you meant with the word gestion. Might be another typo.
BR

Hi, thanks for the corrections, and it's help a lot.
I'm also beginner with Rust and it's not a problem your contribution is welcome is this repo.
And for proton subcommand, i'm write just the word in french and the correct word is more management.


Error

Hi Admin.
Can help Me Fixed Error.
Collecting nltk==3.6.5
Downloading nltk-3.6.5-py3-none-any.whl (1.5 MB)
|████████████████████████████████| 1.5 MB 568 kB/s
Collecting numpy==1.19.5
Downloading numpy-1.19.5-cp38-cp38-manylinux2010_x86_64.whl (14.9 MB)
|████████████████████████████████| 14.9 MB 2.2 MB/s
ERROR: Could not find a version that satisfies the requirement webdrivermanager==3.5.2 (from -r requirements.txt (line 3)) (from versions: 0.2.0, 0.3.0, 0.4.0, 0.4.1, 0.4.2, 0.5.0, 0.6.0, 0.7.1, 0.7.2, 0.7.3, 0.7.4, 0.8.0, 0.9.0, 0.10.0)
ERROR: No matching distribution found for webdrivermanager==3.5.2 (from -r requirements.txt (line 3))

Hi Ahriman, thanks for noticing.  Removed requirements for "webdrivermanager" directly as mentioned on SO thread.  Feel free to let me know if anymore problems.


Lowering the tps

/radon fix-block-access-forceload

I was wrong, it's the opposite, it increases production a lot.


d3 Logo?

I'm adding JS detection for various new libraries to the Library Detector for Chrome. I noticed d3 does not seem to have a favicon or logo of any kind on their site. I wanted to confirm there was no such thing since it would be nice to have artwork for the Chrome extension. Thanks.

Is this the official logo now?
<svg width="96" height="91" style="position:relative;top:22px;">
    <clipPath id="clip">
      <path d="M0,0h7.75a45.5,45.5 0 1 1 0,91h-7.75v-20h7.75a25.5,25.5 0 1 0 0,-51h-7.75zm36.2510,0h32a27.75,27.75 0 0 1 21.331,45.5a27.75,27.75 0 0 1 -21.331,45.5h-32a53.6895,53.6895 0 0 0 18.7464,-20h13.2526a7.75,7.75 0 1 0 0,-15.5h-7.75a53.6895,53.6895 0 0 0 0,-20h7.75a7.75,7.75 0 1 0 0,-15.5h-13.2526a53.6895,53.6895 0 0 0 -18.7464,-20z"></path>
    </clipPath>
    <linearGradient id="gradient-1" gradientUnits="userSpaceOnUse" x1="7" y1="64" x2="50" y2="107">
      <stop offset="0" stop-color="#f9a03c"></stop>
      <stop offset="1" stop-color="#f7974e"></stop>
    </linearGradient>
    <linearGradient id="gradient-2" gradientUnits="userSpaceOnUse" x1="2" y1="-2" x2="87" y2="84">
      <stop offset="0" stop-color="#f26d58"></stop>
      <stop offset="1" stop-color="#f9a03c"></stop>
    </linearGradient>
    <linearGradient id="gradient-3" gradientUnits="userSpaceOnUse" x1="45" y1="-10" x2="108" y2="53">
      <stop offset="0" stop-color="#b84e51"></stop>
      <stop offset="1" stop-color="#f68e48"></stop>
    </linearGradient>
    <g clip-path="url(#clip)">
      <path d="M-100,-102m-27,0v300h300z" fill="url(#gradient-1)"></path>
      <path d="M-100,-102m27,0h300v300z" fill="url(#gradient-3)"></path>
      <path d="M-100,-102l300,300" fill="none" stroke="url(#gradient-2)" stroke-width="40"></path>
    </g>
  </svg>


perf: more load tweaks

Some more loading stuff and fix to the erroring call on empty quilts.

Nice. From 10k feet it looks like we're aggressively choreographing the loading sequence and being smart about initializing things in the natural order of the UX.
Seems good to me!


added dynamic routes explanation and example to README file

I have added a simple explanation for how the dynamic routes work and have added a screenshot as an example

Hello,
Please open the pull request against the develop branch and add the image to the repo, not linking externally. Thank you


HTTPS bypass

I want to bypass HTTPS certificate validation for some API. It'd be nice if there's a switch like that in there.

Sorry. I got on the wrong foot here. It wasn't HTTPS issue.


Toothpick 3 doesn't inject dependencies after obfuscation

I'm dealing with pro guard configuration at production app. When I'm working in debug variant, everything works fine but I've tested that, after obfuscation class' members are null.
I've got following class:
public class SplashActivity extends BaseActivity {

    @Inject
    DataFacade dataFacade;

    @Inject
    LoginFacade loginFacade;

    @Inject
    SharedPreferencesManager sharedPreferencesManager;

Toothpick inject() is called at onCreate()
This is my proguard-rules.pro:
#Toothpick 3
-dontwarn javax.inject.**
-dontwarn javax.annotation.**
-keep class javax.inject.**
-keep class javax.annotation.**
-keepclassmembers class * {
	@javax.inject.Inject <init>(...);
	@javax.inject.Inject <init>();
	@javax.inject.Inject <fields>;
	public <init>(...);
}
-keepnames @toothpick.InjectConstructor class *
-keepclasseswithmembernames class * { toothpick.ktp.delegate.* *; }
-keepclassmembers class * {
    toothpick.ktp.delegate.* *;
}

After the app launch the stack trace tells:
java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String com.example.app.j.a.a.b()' on a null object reference

A decompiled code looks like:
public class SplashActivity extends c {
    @Inject
    d dataFacade;
    @Inject
    e loginFacade;
    @Inject
    com.example.app.j.a.a sharedPreferencesManager;

Am I missing something at configuration?

Resolved.
proguard-rules.pro didn't have following lines
-keep class * extends toothpick.MemberInjector
-keep class * extends toothpick.Factory

In my case, I didn't use the Smoothie.

Thx
Le ven. 12 juin 2020 à 00:05, Arkadiusz<EMAIL_ADDRESS>a écrit :

Closed #412 https://github.com/stephanenicolas/toothpick/issues/412.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stephanenicolas/toothpick/issues/412#event-3437182843,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN7PXLHAS5UBPALKFIUKYTRWHHVFANCNFSM4N2FN5QQ
.


Support for logging libraries other than Logback

I tried using this library in my project at work. Alas, the project uses Log4j as the logging library - which leads to an exception in the constructor of the LogRecorder (a NoClassDefFound in case of the test project attached below):
public LogRecorder(Class<?> type) {
    this.logger = (Logger) LoggerFactory.getLogger(type);
}

I tried changing the Logger type from the Logback (ch.qos.logback.classic.Logger) one to the SLF4J interface (org.slf4j.Logger), but that doesn't have the addAppender method :/
Since this could be quite an undertaking, I suggest adding to the Readme that only Logback is supported for now, to avoid confusion.
I have attached an example project that uses Log4j2 below.
Log4j2Project.zip

I was afraid of that. I'll have a look.

When I look at the actual error, it is a
java.lang.NoClassDefFoundError: org/slf4j/spi/LoggingEventAware. That is not a problem with Logsert using logback, it is a problem with Logsert using slf4j-api:2.0.7 That library contains the LoggingEventAware while version 1.7.36 (the one you are using in you project) does not (and maven uses 1.7.36 as an override in the dependency management).
Now you could update that version and make the test green, but then your application won't log because your log4j-bindings still need slfj4 version 1.7.x.
So until Logsert can handle slf4j-api:1.7.x as well, I am afraid you cannot use it.
So I looked at MessageAssert. Except for a ch.qos.logback.classic.Level (log level), which could easily be changed to an org.slf4j.event.Level, the interface is already logging-library-agnostic.
What we could do is create a class e.g.
public class ListAppender implements org.slf4j.spi.LocationAwareLogger {

    public List<LoggingEvent> getLoggedEvents() {
        return events;
    }
}

and implement the interfaces so we have our own in-memory slf4j implementation. Then we can get rid of the logback dependencies of logsert altogether.


Once created and deleted component can't be created again

[Enter feedback here]
This is the message error:
(UserError) Not found component Example_Train.
Code: UserError
Message: Not found component Example_Train.
How can I solve this?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 73a70fdb-96e1-a30f-bd42-450eba903109
Version Independent ID: f9b84ffe-95d0-f2e8-184c-70361de4aa66
Content: What is a component (preview) - Azure Machine Learning
Content Source: articles/machine-learning/concept-component.md
Service: machine-learning
Sub-service: core
GitHub Login: @likebupt
Microsoft Alias: keli19

Thanks for the feedback! We are currently investigating and will update you shortly.

@likebupt Hello Blanca, could you please check on this issue? I can't find any related information about deleted component.  Thanks.

@mx-iao @likebupt Can you please take a look.

Hi @richardvlas , sorry for the inconvenience caused.
In order to maintain proper lineage and reproducibility, we introduce tombstoning for deleted components and other assets including dataset/model/env, etc. If you delete a component with specific name+version, you cannot create a new one with the same name+version, but you should be able to create same name and different version.
This will prevent scenarios like the following: user deletes an ComponentVersion used to run a job. User then creates a new ComponentVersion with the same name/version, but with a different logic. User then reruns the job, and because an component with the same name/version is found, the job is successfully submitted. However, the job will ultimately fail if the new component does not have the necessary dependencies for running the code.

@richardvlas we haven't heard from you. We will now proceed to close this thread. If you have additional questions on this topic, please comment here.
#please-close


docs: correcting typo "available" -> "unavailable"

".. if UNavailable, uv will download Python for you."

Summary

Test Plan

Thanks!


add neo4j

Codecov Report

Merging #4 into develop will increase coverage by 2.16%.
The diff coverage is 94%.

@@             Coverage Diff              @@
##             develop      #4      +/-   ##
============================================
+ Coverage      78.04%   80.2%   +2.16%
- Complexity       150     162      +12
============================================
  Files             45      49       +4
  Lines            542     576      +34
  Branches          14      14
============================================
+ Hits             423     462      +39
+ Misses           114     109       -5
  Partials           5       5

Impacted Files
Coverage Δ
Complexity Δ

...l/EmbeddedMemSqlDependenciesAutoConfiguration.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

.../EmbeddedMariaDBDependenciesAutoConfiguration.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...mbeddedAerospikeDependenciesAutoConfiguration.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

.../couchbase/EmbeddedCouchbaseAutoConfiguration.java
100% <ø> (ø)
4 <0> (ø)
:arrow_down:

...mbeddedCouchbaseDependenciesAutoConfiguration.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...ion/camel/EmbeddedKafkaCamelAutoConfiguration.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...ika/test/redis/EmbeddedRedisAutoConfiguration.java
100% <ø> (+19.35%)
5 <0> (ø)
:arrow_down:

...is/EmbeddedRedisDependenciesAutoConfiguration.java
100% <ø> (+25%)
2 <0> (ø)
:arrow_down:

.../aerospike/EmbeddedAerospikeAutoConfiguration.java
100% <ø> (ø)
5 <0> (ø)
:arrow_down:

...configuration/ZookeeperContainerConfiguration.java
100% <100%> (ø)
5 <1> (ø)
:arrow_down:

... and 9 more

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 f71d3c3...9a8e710. Read the comment docs.


Fix chatcraft selecting wrong model per issue #660

Fix chatcraft selecting wrong model per issue #660

This was driving me insane on mobile

I wonder if we should add some UI hints about what a model can do, so people are forced to be LLM experts to use various features...


Which IP should change the DNS to.

Should it be my public IP or my local IP

The IP address of the network your switch is connected to, so your local IP.

Should I put it for both Primary and Secondary?

If it's setup correctly you should only need to set primary, but it doesn't hurt setting both.


Applying SecurityGroup by subdividing #888

#888 적용 범위가 넓어 자원별로 세부화한 반영 필요
SecurityGroup 적용 필요

completed by https://github.com/cloud-barista/cb-spider/commit/c80a66deb007718f1302d4383a0cbeb10c5d8ce6


docs(vitals) remove stale vitals spec

Summary
Remove stale vitals spec.
Full changelog

Removed stale vitals spec

Checklist:

[x] Commit message & atomicity checked
[ ] Documentation  N/A
[ ] Spellchecked my updates N/A
[x] Ready to be merged

@h3xar0n or @Jlawlzz could you review this? It's just one link to click. Thanks!


Add speech sync JSON/REST system tests.

This adds the speech sync_recognize() JSON/REST system test.
Based from https://github.com/GoogleCloudPlatform/google-cloud-python/pull/2648

I'm not sure how https://github.com/GoogleCloudPlatform/google-cloud-python/pull/2649/commits/63cb2fa42e07746a45720324327c7a3edc352742 got in here? I rebased master in...

There we go!

Actually..this is pointless now since the switch to the env usage.


Review workflow overhaul

This PR is a full overhaul of the review workflow to reflect latest changes to the wireframes: https://www.figma.com/file/wo6SjH2DBuXf4DgL8rDBOX/PREreview?node-id=521%3A313
If fully working, all options will take a user successfully through the review process, whether or not they chose to fill out a longform review.

Fixes #122
Fixes #136

On a second check, it turns out I was able to submit a second review, so I'm working on refactoring the checks in place for whether a user has reviewed and/or requested.

On a second check, it turns out I was able to submit a second review, so I'm working on refactoring the checks in place for whether a user has reviewed and/or requested.

Hi @rudietuesdays I pushed up some refactoring of the useEffect hook that checks for the user's latest draft & whether the user has long reviewed -- now a user shouldn't be able to submit multiple long reviews & should only get drafts that they themself wrote. Would you mind testing this or giving another look?

Hi @rudietuesdays I pushed up some refactoring of the useEffect hook that checks for the user's latest draft & whether the user has long reviewed -- now a user shouldn't be able to submit multiple long reviews & should only get drafts that they themself wrote. Would you mind testing this or giving another look?

Thank you for this review @harumhelmy ! It looked great, but I was experiencing a bug where on submit of a rapid review, it would successfully submit in the back end but on the front end the stepper wouldn't move forward. That should be fixed now if you can take another look!

Thank you for this review @harumhelmy ! It looked great, but I was experiencing a bug where on submit of a rapid review, it would successfully submit in the back end but on the front end the stepper wouldn't move forward. That should be fixed now if you can take another look!

Okay so I just started seeing that bug (wherein a rapid review is submitted in the backend but the frontend stepper isn't moving forward, and you end up submitting multiple reviews) too @rudietuesdays! It's still happening on my end on both Chrome and Firefox, and I realized that it only happens when a user is newly logged in and hasn't refreshed the browser, for some reason it doesn't happen when I am newly logged in and then refresh the browser, if that is a clue.

@harumhelmy I wonder if this would be fixed by the user context updates I added to develop? I'd love to try to merge this if it's ok otherwise, and create a new issue for the bug if it persists!

Okay so I just started seeing that bug (wherein a rapid review is submitted in the backend but the frontend stepper isn't moving forward, and you end up submitting multiple reviews) too @rudietuesdays! It's still happening on my end on both Chrome and Firefox, and I realized that it only happens when a user is newly logged in and hasn't refreshed the browser, for some reason it doesn't happen when I am newly logged in and then refresh the browser, if that is a clue.

@harumhelmy I wonder if this would be fixed by the user context updates I added to develop? I'd love to try to merge this if it's ok otherwise, and create a new issue for the bug if it persists!


patch: Added support for passing params as keyword arg

Added support for passing params as keyword arg

init:

chat = ChatWatsonx(
    model_id=MODEL_ID_TOOL,
    url=URL,
    project_id=WX_PROJECT_ID,
    max_tokens=10
)

method:

resp_1 = chat.invoke(prompt_1, max_tokens=10)

Integration test passing successfully -> https://github.com/langchain-ai/langchain-ibm/actions/runs/11741957098