Manifests need logo

Something from here: https://github.com/pulibrary/pul_graphics/tree/master/pul/png

This will be important for cicognara.

Having the rights statements in the manifests will be useful for CDH projects making use of PUL materials, so we can be sure we're using materials appropriately and not exposing or reusing content that we shouldn't.


Support running under other UID/GID/user/group

Is your feature request related to a problem? Please describe.
Requiring UID 1000 for docker is not possible due to our admin's policy.
Describe the solution you'd like
Allow the container to run with a different UID / GID.

Not sure if this is the cause the this issue.
/dropbox $ maestral start
Traceback (most recent call last):
  File "/usr/local/bin/maestral", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/maestral/cli/common.py", line 35, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/maestral/cli/cli_core.py", line 177, in start
    if is_running(config_name):
  File "/usr/local/lib/python3.10/site-packages/maestral/daemon.py", line 325, in is_running
    return maestral_lock(config_name).locked()
  File "/usr/local/lib/python3.10/site-packages/maestral/daemon.py", line 282, in maestral_lock
    path = get_runtime_path("maestral")
  File "/usr/local/lib/python3.10/site-packages/maestral/utils/appdirs.py", line 201, in get_runtime_path
    return to_full_path(runtime_path, subfolder, filename, create)
  File "/usr/local/lib/python3.10/site-packages/maestral/utils/appdirs.py", line 30, in to_full_path
    os.makedirs(path, exist_ok=True)
  File "/usr/local/lib/python3.10/os.py", line 215, in makedirs
    makedirs(head, exist_ok=exist_ok)
  File "/usr/local/lib/python3.10/os.py", line 225, in makedirs
    mkdir(name, mode)
PermissionError: [Errno 13] Permission denied: '/.cache'

Here a little workarround (!) to run Maestral with UID different than 1000:
Create own Dockerfile (based on original Dockerfile)
FROM maestraldbx/maestral

####  set your target UID here #####
ARG UID=2000

# switch back to root to allow deluser/adduser
USER root

# remove old dropbox user
RUN deluser dropbox

# re-add dropbox user with your target UID
RUN adduser -D -u ${UID} -h /dropbox dropbox
RUN chown dropbox:dropbox /dropbox

# switch back to run user
USER dropbox

For initial setup
docker build -t my/maestral .
run -it -v /mnt/dropbox:/dropbox my/maestral ash

$ maestral start

Running Maestral daemon
docker run \
  -d \
  --name maestral \
  --rm \
  -v /mnt/dropbox:/dropbox \
  my/maestral

/mnt/dropbox will be accessed as UID 2000 now


Database mapping macro

Mapping macro, similar to YAML or JSON mapping, to populate an instance of a class from a resultset.

This depends on Hash been ordered, which may change in the near future https://github.com/crystal-lang/crystal/issues/2331

I don't see where it depends on the hash being sorted, as it matches the column name with the mapping key (exactly like the YAML mapping does)

As for the travis check failure, it seems to be a change in Crystal 0.14 (it was working fine in 13)

I meant that https://github.com/bcardiff/crystal-db/pull/2/files#diff-89ed3becdd76dae7d10377988fac60a5R91 iterates the fields in certain order.

@spalladino could you rebase & update for the removal of read?. Maybe the last changes in JSON mapping to support nillable/unions offers some guidance to deal better with unions.


⚠ Bofrost.be has degraded performance

In 4f0b14e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 887 ms

Resolved: Bofrost.be performance has improved in f398209 after 2 minutes.


Create role Mayan EDMS

Refs:

https://gitlab.com/mayan-edms/mayan-edms

For LDAP authentication:
https://forum.mayan-edms.com/viewtopic.php?t=1606
https://forum.mayan-edms.com/viewtopic.php?t=1961


Latest Change in Contact.list results in Account Name not listed..

Was running a test on the latest dojo code and found AccountName not in the list w/teh Contact Name.
Before:
QuerySelect was obtaining the AccountName from the Contact record
Now:
QuerySelect obtaining the Accountname from the related Account
ItemTemplate was not changed:
IS:
itemTemplate: new Simplate([
'{%: $.NameLF %}',
'{%: $.AccountName %}'
Should be:
itemTemplate: new Simplate([
'{%: $.NameLF %}',
'{%: $.Account.AccountName %}'
Question.. why was the querySelect changed?  Sometimes it is valid for the Contact to have a different AccountName than the Account has. This change makes that no longer possible.

My bad, sorry about that, I just added AccountName back in.

NO Problem!... just glad I caught it... and that you fixed it. ;-)
--
RJLedger – www.rjlsystems.net
Blog: blog.rjlsystems.net
Phone: +1<PHONE_NUMBER>
-----Original Message-----
From: carlos tirado<EMAIL_ADDRESS>Sent: Thursday, February 23, 2012 14:02
To: Robert Ledger
Subject: Re: [argos-saleslogix] Latest Change in Contact.list results in Account Name not listed.. (#37)
My bad, sorry about that, I just added AccountName back in.

Reply to this email directly or view it on GitHub:
https://github.com/SageSalesLogix/argos-saleslogix/issues/37#issuecomment-4142693

Thanks! - I assume the Account.AccountName has to do w/the fix in Activity so that one can select the Contact first and then get the Account data filled automatically.

Exactly. Now you can select Contact, or Ticket, or Opportunity first before Account, the Account gets automatically filled then the dependency restored in the other fields that are also related to Account.

Carlos,
Great productivity (and fewer "clicks") change! I've the latest 2.0 up on out 7.5.4 production system and using it daily.
rjl
--
RJLedger – www.rjlsystems.net
Blog: blog.rjlsystems.net
Phone: +1<PHONE_NUMBER>
-----Original Message-----
From: carlos tirado<EMAIL_ADDRESS>Sent: Thursday, February 23, 2012 21:32
To: Robert Ledger
Subject: Re: [argos-saleslogix] Latest Change in Contact.list results in Account Name not listed.. (#37)
Exactly. Now you can select Contact, or Ticket, or Opportunity first before Account, the Account gets automatically filled then the dependency restored in the other fields that are also related to Account.

Reply to this email directly or view it on GitHub:
https://github.com/SageSalesLogix/argos-saleslogix/issues/37#issuecomment-4150401


add pytest linter

likely requires @mgedmin 's copyright approval and @tpict 's for ftplugin default

thanks, but can you please update the failing test?

@mgedmin Would you mind hinting at why the environment variable to disable warnings is needed, instead of say simply passing --disable-warnings?

please squash in the following change:
diff --git a/src/testdir/test_compiler.vim b/src/testdir/test_compiler.vim
index b7315e1b8..0a841bc06 100644
--- a/src/testdir/test_compiler.vim
+++ b/src/testdir/test_compiler.vim
@@ -71,10 +71,10 @@ func Test_compiler_completion()
   call assert_match('^"compiler ' .. clist .. '$', @:)

   call feedkeys(":compiler p\<C-A>\<C-B>\"\<CR>", 'tx')
-  call assert_match('"compiler pandoc pbx perl\( p[a-z_]\+\)\+ pylint pyunit', @:)
+  call assert_match('"compiler pandoc pbx perl\( p[a-z_]\+\)\+ pylint pytest pyunit', @:)

   call feedkeys(":compiler! p\<C-A>\<C-B>\"\<CR>", 'tx')
-  call assert_match('"compiler! pandoc pbx perl\( p[a-z_]\+\)\+ pylint pyunit', @:)
+  call assert_match('"compiler! pandoc pbx perl\( p[a-z_]\+\)\+ pylint pytest pyunit', @:)
 endfunc

 func Test_compiler_error()

Failures:
	From test_alot.vim:
	Found errors in Test_compiler_completion():
	command line..script /home/runner/work/vim/vim/src/testdir/runtest.vim[613]..function RunTheTest[57]..Test_compiler_completion line 6: Pattern '"compiler pandoc pbx perl\\( p[a-z_]\\+\\)\\+ pylint pyunit' does not match '"compiler pandoc pbx perl perlcritic php pip_compile podchecker powershell pylint pytest pyunit'
	command line..script /home/runner/work/vim/vim/src/testdir/runtest.vim[613]..function RunTheTest[57]..Test_compiler_completion line 9: Pattern '"compiler! pandoc pbx perl\\( p[a-z_]\\+\\)\\+ pylint pyunit' does not match '"compiler! pandoc pbx perl perlcritic php pip_compile podchecker powershell pylint pytest pyunit'

I don't grasp immediately what this check tries to ensure. Must pytest be placed before pylint somehow?

Oh, I misread. I rather have to fix the test than the code in the PR. Why not let it end with pyunit instead ?

Oh, thank you, @chrisbra , I read your squash request to late, but I think we came to the same conclusion

yes, I think it is fine.

likely requires @mgedmin 's copyright approval and @tpict 's for ftplugin default

For the &errorformat bits from my .vimrc?  I hereby place them in public domain, or whatever wording works for "use them as you see fit".

In the above-mentioned PR to downstream, I added a refinement to unsetting the compiler


DTSCCI-938 remove solicitor CU permissions for respondent1 flags

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/DTSCCI-938
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

no longer needed


[DNM] tikv-3.0: use region end_key as compaction guard

Signed-off-by: zhangjinpeng1987<EMAIL_ADDRESS>only for test

/release

/release

/release


Export - makefile templates - Wextra

I would like to enable flag -Wextra to CC_FLAGS. The library is build with this flag, as defined in gcc.py script.
But templates for gcc_arm does not contain this option (I checked few). I might look at this, and start fixing warnings.

I could add this flag in #1159. Only one file must be adapted for that. Maybe -pedantic would be good, too.

:+1: for -Wextra
:-1: for -pedantic

@chrta Send separate PR for that extra flag, after the current refactoring of gcc make templates is on master. Thanks

This issue is fixed after merging #1251


DOCS-1668

Added PEM and BCFKS support to PingData image certificates

@marcodelas These doc files are generated from our docker-builds repository. I'll make these changes there, and comment here when that's done.

Well, nevermind. I for some reason assumed this doc was the generated file, but looking into it is this one isn't generated. And our generated docs already include the other keystore types. I'll go ahead and merge this, my bad.


Update mm-whats-new.md

Added coming soon section.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@JoanneHendrickson PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Complete lint for cmd/influx

[ ] CHANGELOG.md updated
[ ] Rebased/mergable
[ ] Tests pass
[x] Sign CLA (if not already signed)

Part of #4098.

Thanks @pablolmiranda -- you must sign the CLA before we can merge this. https://influxdb.com/community/cla.html

@otoolep CLA signed.

+1.
@corylanou

Looks good.  Needs a rebase to resolve the conflicts and then we can merge. Thanks!

@otoolep @corylanou can you double check this PR?

Looks good @pablolmiranda -- can you please squash your commits and I will then merge. A comment-only change like this should be a single commit.
Also please send me an e-mail with your address so we can send you a shirt.


New version: Microsoft.Edge.Beta version 112.0.1722.33

Result: Installation Successful

Manifest
Add/Remove Programs

Name
Microsoft Edge Beta
Microsoft Edge Beta

Version
112.0.1722.33
112.0.1722.33

Publisher
Microsoft Corporation
Microsoft Corporation

ProductCode

{0C051B7E-0A20-315E-8348-223400D8EA88}

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 7163
Microsoft Reviewers: Open in CodeFlow

Automatic Validation failed with:

Installation failed with exit code -1978335226
2023-04-05 16:47:57.219 [CLI ] ShellExecute installer failed: 1603 CustomAction DoInstall returned actual error code -2147219199 (note this may not be 100% accurate if translation happened inside sandbox) MSI (s) (B0:44) [16:47:56:590]: Product: Microsoft Edge Beta --
Error 1722. There is a problem with this Windows Installer package. A program run as part of the setup did not finish as expected. Contact your support personnel or package vendor.
Action DoInstall, location: C:\Windows\Installer\MSIFA5B.tmp, command: /silent /install "appguid={2CD8A007-E189-409D-A2C8-9AF4EF3C72AA}&appname=Microsoft Edge Beta&needsAdmin=True&usagestats=0&ap=beta-arch_x64" /installsource enterprisemsi /appargs "appguid={2CD8A007-E189-409D-A2C8-9AF4EF3C72AA}&installerdata=%7B%22distribution%22%3A%7B%22msi%22%3Atrue%2C%22system_level%22%3Atrue%2C%22verbose_logging%22%3Atrue%2C%22msi_product_id%22%3A%22AD8BEC5F-3889-3EAC-9670-462F333FA07A%22%2C%22allow_downgrade%22%3Afalse%2C%22do_not_create_desktop_shortcut%22%3Afalse%2C%22do_not_create_taskbar_shortcut%22%3Afalse%7D%7D"
MSI (s) (B0:44) [16:47:56:971]: Product: Microsoft Edge Beta -- Installation failed. MSI (s) (B0:44) [16:47:56:971]: Windows Installer installed the product. Product Name: Microsoft Edge Beta. Product Version: 112.0.1722.33. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 1603.

Error 1603 sometimes means "elevation needed but not requested" while needsAdmin=True shows within the appguid. But these should be running within elevated sessions.


Radio mode documentation

Radio mode is an amazing feature, it would be nice to have it documented in a more defined way (especially the per-provider behaviour such as a track from Spotify using the recommendations endpoint to get Spotify to suggest similar tracks)

Are you referring to the developer docs here?

Ok. MA only ever uses the provider to construct the queue which is why it only works with certain providers. Can you provide more info about what else you would like to know about it? We have two sentences about it currently on here https://music-assistant.io/ui/

Checking in again as to what additional info you think is needed?

I hadn't seen the radio part in this documentation, good to see it's in there! It would be nice to get more details about This will populate the queue with tracks based on the selected track, as behaviour could be different for different providers, but I'm happy to close this as it's not really essentialy.

Cool I will just add a bit more to indicate that behaviour will be dependant on the provider as suggested,


Getting ##[error]Failed to parse with json file

Hi,  I'm working in Azure Gov and I'm trying to push my config json file to an App Config service and I keep getting an error:
##[error]Failed to parse: .......path to file
the file looks like this:
{
"Key1":  "value"
}
I am using Azure Pipelines in our DevOps instance.  The pipeline is using a service principle that has permissions to the resource group (it is successfully deploying to an app service) and has the "App Configuration Data Owner" role on the config service.
The error is a bit non-specific, do you have any ideas on what I can do to further troubleshoot this?

Hi @mslentz, could you share the format of your json file, which could have key and value masked and I can use the file to repro the issue?

@mslentz, also check if the config json file contains any trailing commas,  can be the cause of the error sometimes

I discovered my issue was with spaces in the path to the file.  Thanks for your help.

@MaryanneNjeri thanks for looking into this. Just curious, won't it work when there are spaces in the path?

I tried it on my end, if the space is present in the file path something like this $filepath  /configFile.json, the error presented  is
##[error]An unexpected error occurred. ENOENT: no such file or directory, open 'path to file' , but if the space is present in the file name, config File.json and the path is $filepath/config Files.json it works, maybe @mslentz  could confirm where the spaces are in the filepath ?

I did have spaces in my file path.  I changed to an AZ CLI command and was able to publish a simple json file: {"Key1": "value"}
I am using this command:
az appconfig kv import -s file --format json --path $file   --separator . --depth 2  --label Development --auth-mode login --endpoint https://xxxxxx --yes
I just added my actual config file:
{
"ActionCenterDiscussionFetchDays": "30",
"ActionCenterNotificationFetchDays": "14",
"AllowedHosts": "*",
"EnableHelpPage": "true",
"Logging:LogLevel:Default": "Information",
"Logging:LogLevel:Microsoft": { "Hosting": { "Lifetime": "Information" } },
"ResultReplayer": {
"Enable": "false",
"RemoveFromKey": "...."
},
"api": {
  "Password": "",
  "UserPropertyCacheLengthInMinutes": "0",
  "orgServiceUri": "https://...",
  "username": "xxxx"
},
"email": {
  "MailFrom": "xxxx",
  "SMTPServer": "xxxx",
  "SMTPServerEnableSsl": "false",
  "SMTPServerPort": "xxx",
  "SMTPServerUsername": "xxxx",
  "Support": "",
  "disableWorkerService": "false",
  "mailBody": "xxxxxxxxx",
  "mailSubject": "xxxxxxx"
},
"form2": { "version": "2" },
"form3": { "version": "2" },
"form4A": { "version": "2" },
"ida:ADFSMetadataCRMApi": "https://xxxxxx",
"ida:WtrealmCRMApi": "https://xxxxxxxx",
"owin:AutomaticAppStartup": "true",
"web": {
  "ApplicationInsights:InstrumentationKey": "xxxxx",
  "AzureKvUri": "https://xxxxxx",
  "AzureWebAppClientId": "xxxxx",
  "AzureWebAppTenantId": "xxxxx",
  "EfsapDbConnStringUrl": "xxxxxxxxxx",
  "HomeRealm": ""
}

}
and the CLI is giving an error that says: The input is not a well formatted json file.  I'm not seeing the problem, can you help?

@mslentz, I tried importing the exact contents of your JSON file using the same command that you are using, and I'm not getting any error. Can you try to format the file you're using with any JSON formatter and see if there's a syntax issue on your side?

Thanks for bringing this to our attention @mslentz. The error turned out to be due to the presence of a non-ASCII character in the JSON file being imported. We are working on supporting the import of all unicode characters using CLI.

Great. @MaryanneNjeri, you will fix the 'space in path" issue too?

Can you share how the task looks? That is the exact JSON I'm pushing, with a few strings substituted. Again, it's valid JSON and it works pushing from the CLI.

Yes let me share how the task looks like from my end.

version: 1.3.4
displayName: 'azure app config task'
inputs:
azureSubscription: 'service principal '
ConfigstoreName: 'config store name’
Configuration file path: 'config1.json'
Separator: ':'
Depth: '2'
Strict: false

Hey Maryanne, that's exactly how the task looks on my end. Very weird; I ended up doing this manually, which is obviously not ideal for production, but had to be done. I'd love to revisit this if you all find there's a bug in here, I'm happy to help you all battle-test this if need be.

@ctvanzandt42 will definitely reach out if we find a bug. Please feel free to reach out incase of anything.

Hi, @ctvanzandt42  a new release is out for Azure App Configuration Task version - 3.5.16 and Azure App Configuration Push Task version-1.4.4, with the fix that gives more information about the error message above, please feel free to check it out. In case of any questions please feel free to reach out :).
Thanks,
Maryanne

Wonderful, Maryanne! Let me know when the release notes are posted, I don't see them. Looking forward to implementing the fix.

Awesome will let you know once the release notes are posted.

@ctvanzandt42  the release notes for this fix were posted, AzureDevOpspipelineExtension and AzureDevOpsPushPipelineExtension

@MaryanneNjeri can this be closed?

@MaryanneNjeri can this be closed?

Yes, thank you so much!


Chinese translation improvement

DAYS = "天",
HP = "生命值",
IF_COOKED = "如果烤熟：",
MINUTE = "分",
SECOND = "秒",
USES = "次",

About USES you can compare this English wiki page with this Chinese wiki page, "33 uses" is spoken as "33 次".
I've checked on Google Translate, zht variant (Traditional Chinese) is the same.
Also, zhr below seems to be a typo.
https://github.com/gimme/item-meta/blob/cc30ef09c91eb7fdf783329d6d2504d16ae99a72/scripts/itemmeta/language/strings.lua#L27

Appreciate it! What about the other ones, do they look ok?

the other ones

Translations? I don't really know about those languages.

I mean the other strings in Chinese

About zhr, in languages/loc.lua it looks like it refers to the Rail platform. Just to be safe, I force it back to zh since that's what the correct translation file is called in this case.

Oh, my mistake then. However after my improvement, zht.lua seems all the same as zh.lua, no difference.

the other strings in Chinese

All checked, should be ok. I'll reopen if any issue found later.

Thank you so much!

Ooops, you missed IF_COOKED...

Oh shoot. That makes more sense. On it...


Support for E164 formatted number

It would be nice if this lib could parse ExPhoneNumber.parse("+977123456789")  without passing the country.

it turns out if I pass  E164 formatted number and an arbitrary string then it can extract its country code

@sushant12 Thanks for making the effort to check this and sharing here.
I see that this is not documented, the API does also not make this clear. I assume that you have found out about this by experimentation and/or reading the code.
I think that documenting this (e.g. providing an example usage) would be beneficial. A PR with such is welcome.

Hi @szymon-jez
Instead of passing an arbitrary string as the second parameter in parse/2 , should not  it accept single parameter for e164 formatter number? eg: ExPhoneNumber.parse("+977123456789")

Yes, calling the function with "" or nil as the second argument can feel awkward. Therefore adding ExPhoneNumber.parse_e164("+977123456789") could be an explicit way to deal with this.
An alternative – to widen the API – would be to document how the parse function deals with e164 numbers. Here are a few examples how it behaves:
iex(46)> {:ok, phone_number} = ExPhoneNumber.parse("+48 123456789", "PL")
{:ok,
 %ExPhoneNumber.Model.PhoneNumber{
   country_code: 48,
   country_code_source: nil,
   extension: nil,
   italian_leading_zero: nil,
   national_number: 123456789,
   number_of_leading_zeros: nil,
   preferred_domestic_carrier_code: nil,
   raw_input: nil
 }}
iex(47)> {:ok, phone_number} = ExPhoneNumber.parse("+48 123456789", "")
{:ok,
 %ExPhoneNumber.Model.PhoneNumber{
   country_code: 48,
   country_code_source: nil,
   extension: nil,
   italian_leading_zero: nil,
   national_number: 123456789,
   number_of_leading_zeros: nil,
   preferred_domestic_carrier_code: nil,
   raw_input: nil
 }}
iex(48)> {:ok, phone_number} = ExPhoneNumber.parse("+48 123456789", nil)
{:ok,
 %ExPhoneNumber.Model.PhoneNumber{
   country_code: 48,
   country_code_source: nil,
   extension: nil,
   italian_leading_zero: nil,
   national_number: 123456789,
   number_of_leading_zeros: nil,
   preferred_domestic_carrier_code: nil,
   raw_input: nil
 }}
iex(49)> {:ok, phone_number} = ExPhoneNumber.parse("+48 123456789", "GB")
{:ok,
 %ExPhoneNumber.Model.PhoneNumber{
   country_code: 48,
   country_code_source: nil,
   extension: nil,
   italian_leading_zero: nil,
   national_number: 123456789,
   number_of_leading_zeros: nil,
   preferred_domestic_carrier_code: nil,
   raw_input: nil
 }}
iex(50)> {:ok, phone_number} = ExPhoneNumber.parse("+48 123", "PL") # invalid
{:ok,
 %ExPhoneNumber.Model.PhoneNumber{
   country_code: 48,
   country_code_source: nil,
   extension: nil,
   italian_leading_zero: nil,
   national_number: 123,
   number_of_leading_zeros: nil,
   preferred_domestic_carrier_code: nil,
   raw_input: nil
 }}

So parsing an e164 number does not even look at the provided country code.


Simplify health checks

@zhfeng feel free to merge this onto you branch when you are satisfied with the changes. This should also trigger a re-run of you branch's pipeline (which failed for some technical reason).


⚠ Fetchers has degraded performance

In e5ab4db, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 380 ms

Resolved: Fetchers performance has improved in b2bdcb2.


Add support for continue-on-error flag for build and add tests cases

Description
With option to continue to build even if a single component fails a recent change was merged in build repo. See https://github.com/opensearch-project/opensearch-build/pull/3834
This change adds the support for same from CI side of things.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @gaiksaya please bump version if needed.
Thanks.


Switch from localstack to minIO

Proposed Changes

fixes https://github.com/ohcnetwork/care/issues/2634
Replaced docker localstack configuration with minIO configuration. Added script to creates facility and patient buckets. Converts facility bucket to public to allow public access as minIO doesn't support ACL.

Associated Issue

Switches from localstack to minIO for local dev s3 since localstack is a very heavy and unreliable at times and most features are blocked behind a paywall

Merge Checklist

[ ] Tests added/fixed
[ ] Update docs in /docs
[ ] Linting Complete
[ ] Any other necessary step

Only PR's with test cases included and passing lint and test pipelines will be reviewed
@ohcnetwork/care-backend-maintainers @ohcnetwork/care-backend-admins

Summary by CodeRabbit

New Features

Introduced a new MinIO service for object storage, replacing the previous LocalStack service.
Added initialization and entrypoint scripts for setting up MinIO.

Bug Fixes

Updated environment variables for bucket credentials and endpoints to ensure compatibility with the new MinIO service.

Documentation

Updated setup documentation to reflect the transition from LocalStack to MinIO in the development environment.

@vigneshhari Please review.
Would like to add that minIO doesn't seem to support "public-read" ACL set during upload, hence 'facility-bucket' is initialized as public.


updates to Web Server deployment

Issue #, if available:
Description of changes:

Fix-up how we use bootobject parameter (make it optional)

follows on https://github.com/setheliot/aws-well-architected-labs/pull/34 and https://github.com/setheliot/aws-well-architected-labs/commit/6d246bf88b88ad03b40bb30631f452c655678685

Change Web server autoscaling group minsize from 2 back to 3

it was changed it for testing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing this PR and replacing it with https://github.com/setheliot/aws-well-architected-labs/pull/48


Confirm locale codes

Locale codes might have to be changed depending on i18n approach and tooling.
Currently, the google i18n (Chrome browser) locale codes are considered in #83
Are there specific requirements for Desktop and mobile builds?

I think we just need to make sure the locale codes are compatible with Intl and give the expected output, so that numbers and dates can be formatted correctly.
This can be tested with Intl.NumberFormat('en').format(1234.123) and Intl.DateTimeFormat('en').format(new Date()) in the console


collection_start value is now saved when a plot is flagged

src/clj/org/openforis/ceo/db/plots.clj
148, drop the nil here.  Karen said we will never need the confidence for a flagged plot.
147-149 in clojure, when we can, we column align similar statements.  this includes the right side of the let assignments.
(defn flag-plot [{:keys [params]}]
  (let [plot-id          (tc/val->int (:plotId params))
        user-id          (:userId params -1)
        collection-start (tc/val->long (:collectionStart params))]
    (call-sql "flag_plot" plot-id user-id nil (Timestamp. collection-start))
    (unlock-plots user-id)
    (data-response "")))

src/sql/changes/update_2021-01-01_flag_collection_start.sql
Drop this file.  clojure -A:build-db only-functions will drop and recreate all functions.  I run this every time that I re-deploy the server.
src/sql/functions/project.sql
1141, 1154, 1156 remove _confidence
1162, set confidence to NULL on conflict


Fix import attribute translation with underscore

1. Why is this change necessary?
2. What does this change do, exactly?
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfil them.

hey @hlohaus
thanks for your PR and nice to hear from you again :blush:
but the contribution guidelines also apply for you :wink:

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://developers.shopware.com/contributing/contribution-guideline/.


Updates from ADS - Module

Updates to the module data source.

additions make sense, meta will definitely be relevant for T1574 + T1129. Not yet sure about technique usage of content but conceptually makes sense (comparable to Binary file metadata, but similar detections could potentially be based on meta 🤷)


[MINOR] When a Jenkins Control tool is focused, color should be inverted

This is a cosmetics issue
Jenkins Control tool open, focus on the code editor:

Jenkins Control tool open, focus on the tool:

For reference, this is how it looks for a Maven tool:

IntelliJ IDEA 2023.3 EAP (Ultimate Edition)
Build #IU-233.8264.8, built on September 28, 2023
Licensed to IntelliJ IDEA EAP user: Jacek Gajek
Expiration date: October 28, 2023
Runtime version: <IP_ADDRESS>+7-b1063.1 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 6.2.0-33-generic
Jenkins Plugin 0.13.19-2023.2
GC: G1 Young Generation, G1 Old Generation
Memory: 8048M
Cores: 24
Registry:
  debugger.new.tool.window.layout=true
  ide.experimental.ui=true
Non-Bundled Plugins:
  com.github.leomillon.uuidgenerator (4.5.1)
  de.netnexus.camelcaseplugin (3.0.12)
  IdeaVIM (2.5.1)
  org.turbanov.run.configuration.as.action (2.2)
  de.kontext_e.idea.plugins.autofill (1.2)
  com.github.stevejagodzinski.akka-log-inspection-plugin (1.2.9)
  Jenkins Control Plugin (0.13.19-2023.2)
  String Manipulation (9.11.3)
  com.jetbrains.packagesearch.intellij-plugin (233.8264.8)
  woaini.liquibase.plugin (1.3-SNAPSHOT)
  PlantUML integration (7.2.0-IJ2023.2)
  com.intellij.ml.llm (233.8264.9)
  commit-prefix-plugin (1.2.1)
  Pythonid (233.8264.8)
  com.intellij.bigdatatools.core (233.8264.8)
  org.intellij.scala (2023.3.2)
  com.rspn.git (1.9)
  mobi.hsz.idea.gitignore (4.5.1)
Kotlin: 233-1.9.0-release-358-IJ8264.8
Current Desktop: KDE

Sound great

I've changed the title because actually it SHOULD be inverted when focused, but it's not.

Part of #506

can be tested with eap: https://plugins.jetbrains.com/plugin/6110-jenkins-control/versions/eap/412498

released with 0.13.20


Colored Usernames not colored

Hi, was just chatting with someone, they also said they were facing this issue, so I figured I'd bug report.
It seems like the Colorized Usernames feature is no longer working. They just appear white. I noticed it in the main and nightly builds. I checked my settings to verify they were proper, and yeah. I tried disabling the settings, re-enabling them, still bugged.
I thought I saw them work the other night, but I honestly I think I was using the wrong version on accident when I first booted..
Screenshots
https://i.imgur.com/dkgSiQc.png
https://i.imgur.com/LZdQcHF.png
https://i.imgur.com/EV7IH1c.png
OS and Chatterino Version
    Windows 10, Chatterino 7.3.2
&
    Windows 10, Chatterino Nightly 7.3.2

Duplicate of god knows how many issues. Prior to opening a new issue search for existing ones to avoid opening duplicates, it's not that hard.
Duplicates on this repository:

Duplicates on upstream Chatterino repository:
Chatterino/chatterino2#2873
Chatterino/chatterino2#2822

Download chatterino7 portable version, works in there. Little workaround for now


creating math_constants.h file

This the HIP equivalent of the CUDA file of the same name.
Currently the contents are limited only to what is needed when
compiling Eigen with HIP. More contents can be added as/when needed.

@deven-amd Any ETA on addressing the review comments? I could have fixed them myself, but looks like you have deleted the branch from which the PR was made. So I am unable to submit additional commits to this PR.

You can use different branches in your fork for different PRs.
On Thu, Jul 5, 2018 at 10:53 PM, Deven Desai<EMAIL_ADDRESS>wrote:

@deven-amd commented on this pull request.
In include/hip/math_constants.h
https://github.com/ROCm-Developer-Tools/HIP/pull/512#discussion_r200429098
:

+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.
+*/

+#ifndef HIP_INCLUDE_HIP_MATH_CONSTANTS_H
+#define HIP_INCLUDE_HIP_MATH_CONSTANTS_H
+
+// Some standard header files, these are included by hc.hpp and so want to make them avail on both
+// paths to provide a consistent include env and avoid "missing symbol" errors that only appears
+// on NVCC path:
+
+
+#if defined(HIP_PLATFORM_HCC) && !defined(HIP_PLATFORM_NVCC)
+#include <hip/hcc_detail/math_constants.h>
+#elif defined(HIP_PLATFORM_NVCC) && !defined(HIP_PLATFORM_HCC)
+//#include <hip/nvcc_detail/math_constants.h>
BTW, what is workflow to submit a second PR from my fork, while another is
still outstanding (#541
https://github.com/ROCm-Developer-Tools/HIP/pull/541)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ROCm-Developer-Tools/HIP/pull/512#discussion_r200429098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD_A95b_99DHYtWn5COMcIDtLwDqBjD8ks5uDkujgaJpZM4Ui7GL
.

Filed pR #551...closing out this one


Documentation of Regex.new

The documentation for Regex.new at Regex.html has the line:
Regex.new("^a-z+:\s+\w+")                     # => /^a-z+:\s+\w+/

As does the source code for Regex.initialize in src/regex.cr
That is not correct, because the parsing of string values is not the same as the parsing of Regex values, particularly when it comes to escaped characters like \d.
dumb_rx = Regex.new( "\s+\d+\s+" )
if "aa 22 cc" =~ dumb_rx
  printf "single-slash Match\n"
else
  printf "single-slash No Match\n"
end

dumb_rx = Regex.new( "\\s+\\d+\\s+" )
if "aa 22 cc" =~ dumb_rx
  printf "double-slash Match\n"
else
  printf "double-slash No Match\n"
end

produces:
single-slash No Match
double-slash Match

(which is what I would expect it to do, of course).

Also, in Ruby one can use a regexp-value as the first parameter to Regexp.new(), to avoid the extra level of escaping that you'd need when using a string-value for that parameter.  Crystal does not allow that, although I do not know if that was intentional.  If the user tries:
dumb_rx = Regex.new( /\s+\d+\s+/ )
if "aa 22 cc" =~ dumb_rx
  printf "single-slash Match\n"
else
  printf "single-slash No Match\n"
end

they get the compile-time error of:
Error in line 1: instantiating 'Regex:Class#new(Regex)'

instantiating 'Regex#initialize(Regex)'

in /usr/lib/crystal/regex.cr:224: undefined method 'gsub' for Regex

    source = source.gsub('\u{0}', "\\0")
                    ^~~~

================================================================================

Regex trace:

  /usr/lib/crystal/regex.cr:222

      def initialize(source, @options : Options = Options::None)

You might wonder why anyone would bother doing Regexp.new( /patrn/ ) instead of simply using /patrn/, and I must admit I do not know why I started doing that.  Maybe it had something to do with the state of ruby back in 2002.  But if crystal does intend to not allow that, then maybe the definition for Regex.initialize should be:
def initialize(source : String, @options : Options = Options::None)

instead of:
def initialize(source, @options : Options = Options::None)

?


[SPARK-47277] PySpark util function assertDataFrameEqual should not support streaming DF

What changes were proposed in this pull request?

The handy util function should not support streaming dataframes, currently if you call it upon streaming queries, it throws a relatively hard-to-understand error:
>>> df1 = spark.readStream.format("rate").load()
>>> df2 = spark.readStream.format("rate").load()
>>> from pyspark.testing.utils import QuietTest, assertDataFrameEqual
>>> assertDataFrameEqual(df1, df2)
/Users/wei.liu/oss-spark/python/pyspark/pandas/__init__.py:43: UserWarning: 'PYARROW_IGNORE_TIMEZONE' environment variable was not set. It is required to set this environment variable to '1' in both driver and executor sides if you use pyarrow>=2.0.0. pandas-on-Spark will set it for you but it does not work if there is a Spark context already launched.
  warnings.warn(
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/wei.liu/oss-spark/python/pyspark/testing/utils.py", line 936, in assertDataFrameEqual
    actual_list = actual.collect()
  File "/Users/wei.liu/oss-spark/python/pyspark/sql/dataframe.py", line 1453, in collect
    sock_info = self._jdf.collectToPython()
  File "/Users/wei.liu/oss-spark/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/java_gateway.py", line 1322, in __call__
  File "/Users/wei.liu/oss-spark/python/pyspark/errors/exceptions/captured.py", line 221, in deco
    raise converted from None
pyspark.errors.exceptions.captured.AnalysisException: Queries with streaming sources must be executed with writeStream.start();
rate

Because the function calls collect which is not supported on streaming dataframes. It'd be good if we can catch this earlier.
Why are the changes needed?

Improve usability
Does this PR introduce any user-facing change?

No
How was this patch tested?

Unit test
Was this patch authored or co-authored using generative AI tooling?

Generated-by: Github Copilot
It helped me to pick the error class UNSUPPORTED_OPERATION

I'm having some local build issue, since this is a small change I want to defer the test to remote CI

cc @itholic

CI error is still related, i'll rebuild locally and verify tomorrow

Merged to master.
@WweiL would you mind opening a backporting PR to branch-3.5?

@HyukjinKwon Sure


[bug] App not working with iOS Safari

In the latest version, the app does not work in the Safari browser on iOS.
After adding 4 player names and clicking "Start", the names are not present in the teams. There are only the placeholder names.

fixed in commit https://github.com/tomboe311/KICKET/commit/7d5d8572a9852df1d0f63ac16b6a37ffd3d3622f


Wrong ACK RUri

ACK has wrong request Uri
The reason - different format for Record route options: <<"lr">> or {<<"lr">>,<<"on">>}
NKSIP Version v0.4.0
Quick fix -
-            case lists:member(<<"lr">>, RouteOpts) of
-                true ->
-                    DestUri = TopRoute#uri{
+            Lr = lists:member(<<"lr">>, RouteOpts) or lists:member({<<"lr">>,<<"on">>}, RouteOpts),
+            case Lr of
+                true ->
+                  DestUri = TopRoute#uri{

https://kamailio.org/docs/modules/3.1.x/modules_k/rr.html#id2691315

Ok, I see now. I will fix that when I can go back yo nksip.
Thanks


Update main.py

Don't know if this was correct. Was getting the error "RuntimeWarning: coroutine 'GPTShell.default' was never awaited, when I tried to run a single command, and this fixed it for me.

Yep, good catch, thanks!


bitwarden-desktop: 2024.8.0 -> 2024.8.1

Diff: https://github.com/bitwarden/clients/compare/desktop-v2024.8.0...desktop-v2024.8.1
Changelog: https://github.com/bitwarden/clients/releases/tag/desktop-v2024.8.1
Description of changes

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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

Can you copy these lines?
https://github.com/NixOS/nixpkgs/blob/1a88164cc03ff2b7f3e79c042bee932bce7f036d/pkgs/by-name/bi/bitwarden-cli/package.nix#L23-L26


[reportFormat] Question about formats

What different report formats exist and what does it influence?

Hey, there are 2 report formats, UNICODE and ASCII.
You should use ASCII if UNICODE is not supported by your terminal.
You can find the examples here https://github.com/fabriciorby/maven-surefire-junit5-tree-reporter#output


docs scraper not handling markdown with omitted optional yaml front matter

What happened?
While investing scraper for https://github.com/crossplane-contrib/provider-jet-equinix/pull/18 and processing https://raw.githubusercontent.com/equinix/terraform-provider-equinix/master/docs/resources/equinix_ecx_l2_connection.md,
scraper: error: Failed to scrape Terraform provider metadata: cannot scrape Terraform registry: failed to scrape resource metadata from path: ../.work/equinix/equinix/docs/resources/equinix_ecx_l2_connection.md: failed to find the prelude of the document using the xpath expressions: //text()[contains(., "description") and contains(., "page_title")]

According to https://developer.hashicorp.com/terraform/registry/providers/docs#yaml-frontmatter, page_title is optional.
Does scraper know how to handle the Markdown documents and formatting described here? https://developer.hashicorp.com/terraform/registry/providers/docs#format

How can we reproduce it?

Checkout https://github.com/crossplane-contrib/provider-jet-equinix/pull/18/commits/97eb823e70fd751ff0265128a6db4dfbad9d8909
run make generate

Same issue here with Confluent Provider.

I'm also hitting this issue, is there a way around it?

Hi @displague,
Could you please try overriding the default value of the --prelude-xpath command-line argument in apis/generate.go with something like:
//go:generate go run github.com/upbound/upjet/cmd/scraper -n ${TERRAFORM_PROVIDER_SOURCE} -r ../.work/${TERRAFORM_PROVIDER_SOURCE}/${TERRAFORM_DOCS_PATH} -o ../config/provider-metadata.yaml --prelude-xpath "//text()[contains(., \"subcategory\")]"

@ulucinar This seems to have unblocked the build.  I do some some new code documentation bugs after making this change. Some lines are duplicated and some lines are pulled from the wrong section of the Terraform docs.

https://github.com/crossplane-contrib/provider-jet-equinix/pull/18/commits/433bd889641490b0ce8ce595cf65822c216bbe27#diff-305bc62735706da4b7f680400fca095704d4a3ceb03e994c41fce4391430390aR276
https://github.com/crossplane-contrib/provider-jet-equinix/pull/18/commits/433bd889641490b0ce8ce595cf65822c216bbe27#r1125691520

Hi @displague,
We had a recent change in the scraper that was motivated by fixing a case in upbound/provider-gcp. I don't expect it to address the duplication or the wrong-section issue you mentioned above but still, if you would like to give it a try, you may do so by updating your upjet dependency and adding the optional command-line argument --resource-prefix equinix to the generate comment like we do here. Sorry for the inconvenience. It's quite challenging for one scraper to be able to handle all the cases.

Getting this issue trying to generate from the Fastly provider. It is erroring on empty markdown files.
https://github.com/fastly/terraform-provider-fastly/blob/5c16f5639f7ebbb4ea637172131bfbb16453959b/docs/resources/arguments/package.md
https://github.com/fastly/terraform-provider-fastly/blob/5c16f5639f7ebbb4ea637172131bfbb16453959b/docs/resources/components/footer.md
The only workaround I have at the moment is to delete these files.


removed library

Blocked pending a response to my question on the related PR https://github.com/arduino/library-registry/pull/2433

Blocked pending a response to my question on the related PR #2433
I made changes to the library today around 15:30 PM IST, but those are not being shown in the Arduino IDE in different versions. So couldn't find any source to do so, thus thought of deleting the library and updating the library-registry.txt for doing this.

We have determined the removal of the library is not necessary in the other thread so I will close this PR and continue to provide assistance in the other thread..


Fix winff package

The download link does no longer work

Fixed: https://chocolatey.org/packages/winff/<IP_ADDRESS>50503


ActiveForm events how to

I can't seem to find in the docs how to add "Custom events" in yii.activeForm.js, basically I need to work on 'beforeSubmit' and 'afterSubmit' in yii.activeForm.js

https://github.com/yiisoft/yii2/commit/cc40c55a6ba14b30be69e340cb415939981c0070


Dependency updates

Another day, another dependency update 🍻  Cheers.

awesome! @phated I added you as a collab and owner on npm, feel free to merge and publish, thank you!


How to obtain both values?

How to obtain both values (minimum and maximum values)? I'm not sure how to get from it.

Minimum value is lowerValue
Maximum value is upperValue


Gtest already imported with external LLVM

Check List

[N/A] The binary I am trying to run has been compiled statically for either RV64 or AArch64.
[N/A] The compiled binary is a Linux Elf file.
[N/A] I have provided both a config file and a binary to SimEng as runtime arguments.

Binary File Information
Please run file on the binary used and paste the output below (i.e. file myBinary.elf).
N/A
System Description
Please provide the following as a list:

The Operating System of the system you are running SimEng on
Ubuntu 22.04
The compiler used to compile SimEng and its version
gcc 11.4.0 + clang-14.0.5
The compiler used to compile the static binary and its version
N/A
The ISA or specific processor that the binary was compiled for

For example, if -march=armv8.4-a+sve was used, then present armv8.4-a+sve
If -mcpu=neoverse-v1 or similar was used, then present neoverse-v1
N/A

The processor of the system you are running SimEng on
N/A
The main memory capacity of the system you are running SimEng on
64 GB

SimEng Version
Provide the SimEng repository branch, commit hash, and version tag (if relevant) that the issue is present on.
Any, but specifically tested on dev
SimEng CMAKE Options Used
Provide a bullet list of all CMAKE options used. E.g. -DCMAKE_BUILD_TYPE=Release.

-B build
-S .
-DCMAKE_BUILD_TYPE=Release
-DCMAKE_INSTALL_PREFIX=$HOME/SimEng/build/
-DSIMENG_ENABLE_TESTS=ON
-DSIMENG_USE_EXTERNAL_LLVM=ON
-DLLVM_DIR=$HOME/clang-14.0.5/lib/cmake/llvm/
-GNinja

Binary Compilation Instructions
Provide a bullet list of how the binary in question was compiled, including all compiler flags used.
N/A
SimEng Command Line Expression
Provide the command line expression used to run SimEng e.g. ./simeng /path/to/configs/a64fx.yaml /path/to/myBinary.elf
N/A
SimEng Metadata Output
If your simulation begins to execute the binary, please provide the metadata that SimEng prints at the start of execution.
N/A
Problem Description
Simeng fails to build with tests enabled when using an external clang/llvm build that has been built with unit tests enabled (the default build option). It errors with the following:
CMake Error at build/_deps/googletest-src/googletest/cmake/internal_utils.cmake:152 (add_library):
  add_library cannot create target "gtest" because an imported target with
  the same name already exists.

This appears to be because LLVM builds its unit tests with googletests as well, and if a version doesn't yet exist on the system (or isn't specified), then it installs version release-1.11.0, the same used in SimEng. This release version was found in llvm-project/third-party/benchmark/cmake/GoogleTest.cmake.in.
I expect a fix would be in SimEng's CMakeLists.txt to search for an existing gtest install, and if a compatible one is found to instead use this rather than attempting to still install.
In the meantime, users (and myself) can fix this by one of the following options:

Recompiling clang/llvm with the flag -DLLVM_INCLUDE_TESTS=OFF to disable unit tests (and therefore not installing gtest),
Building SimEng without using an external LLVM,
Building SimEng with tests disabled.

Any of these three options should suffice for the short term.

Should be resolved in https://github.com/UoB-HPC/SimEng/pull/433


Cross-compile to windows

Closes #11
The app builds (on macos) but fails on runtime with:
The code execution cannot proceed because WebView2Loader.dll was not found.

Using x86_64-pc-windows-msvc instead of x86_64-pc-windows-gnu target, with https://github.com/rust-cross/cargo-xwin did the job.
Thanks to: https://github.com/DioxusLabs/dioxus/issues/899#issuecomment-1686497135

Time to package cargo-xwin in nixpkgs: https://github.com/NixOS/nixpkgs/issues/262780

For time-being, I will just use buildRustPackage and package it locally while I package it in parallel to upstream nixpkgs.

While trying to use cargo-xwin as part of the buildPhaseCargoCommand in crane, I get the following error:
failed to create directory `/homeless-shelter/Library/Caches/cargo-xwin/xwin`
       >
       > Caused by:
       >     Read-only file system (os error 30)

That is because cargo-xwin uses the default cache_dir given by https://crates.io/crates/dirs. Although cargo-xmin provides a way to configure this cache with XWIN_CACHE_DIR, when crates that depend on cmake are used XWIN_CACHE_DIR is not chosen, primarily because of: https://github.com/rust-cross/cargo-xwin/blob/28cd46e8c5aff2411d7982288b8e392e73b8a82c/src/common.rs#L398-L403.

To solve the CI issue, I am guessing we have to download the manifest deps of cargo-xwin in a prior phase so that during the build it returns without trying to download, cf. https://github.com/rust-cross/cargo-xwin/blob/28cd46e8c5aff2411d7982288b8e392e73b8a82c/src/common.rs#L271-L273

Something I'd like to figure out is how to use crates using libraries, like sqlite. felicity uses sqlx and I might as well change dioxus-desktop-template to use a similar library for demonstration.


Review RSA jwk format

As per https://tools.ietf.org/html/rfc7517#section-9.3 we supposed to be using e, n and d key values to avoid timing attacks, and in some cases, discard the e public factor and use only n and d.
Manuale uses only e and n.
Let me test if adding all key values will impact transactions.

Because we only want to publish public aspects of the key. That's it.


ByteString interface

We could have an interface to work with ByteStrings as a sequence of Word8s, or of ASCII chars, or maybe both.
Implementation would be similar to Text because I expect we want to support slicing.
I'm open to adding this should someone ask for it.

Yes please!

How do you plan to use it?
I haven't given much thought to the interface here, so it would help to consider some use cases.


Assure no SM survives plugin factory deactivation.

Currently plugins sometimes destroy continuations used for hooks
in TSRemapDeleteInstance. To make sure no SM is still alive when
TSRemapDeleteInstance is called "deactivating" the remap plugin
factory as late as possible right before its destruction.
Fixed minor memory leak when a corrupted DSO file loading is
attempted during config reload, added few extra debug messages.

[approve ci autest]

Thank you, @jrushford, for the review and testing it as well!
I will be glad to make the extra change while I am fixing things in that function, also renamed the variable from msg to msg_format because it seems someone did not realize that msg is actually the format string.

This looks like it's depending on the @strategy commit (128507ae5a69ab0cbaa6d9707cfe2ad1d95cbed8).

@zwoop, although this PR has nothing to do with @strategy commit 128507a my attempt to cherry-pick surprisingly failed for me too, hence created 9.0.x specific PR #6299.


[BUG] ModuleNotFoundError: No module named 'typing'

[x] I've checked docs and closed issues for possible solutions.
[x] I can't find my issue in the FAQ.

Describe the bug
I am getting the following error when attempting to use rich:
ModuleNotFoundError: No module named 'typing'
File "/home/user/Text_Gen/text-generation-webui-main/installer_files/env/lib/python3.11/site-packages/rich/__init__.py", line 4, in <module>
  from typing import IO, TYPE_CHECKING, Any, Callable, Optional, Union
ModuleNotFoundError: No module named 'typing'

Platform

Click to expand
Linux, ssh from windows cmd prompt
I may ask you to copy and paste the output of the following commands. It may save some time if you do it now.
If you're using Rich in a terminal:
python -m rich.diagnose
pip freeze | grep rich

(/home/user/Text_Gen/text-generation-webui-main/installer_files/env) user@userserver:~$ python -m rich.diagnose
Traceback (most recent call last):
  File "<frozen runpy>", line 189, in _run_module_as_main
  File "<frozen runpy>", line 112, in _get_module_details
  File "/home/user/Text_Gen/text-generation-webui-main/installer_files/env/lib/python3.11/site-packages/rich/__init__.py", line 4, in <module>
    from typing import IO, TYPE_CHECKING, Any, Callable, Optional, Union
ModuleNotFoundError: No module named 'typing'

(/home/user/Text_Gen/text-generation-webui-main/installer_files/env) user@userserver:~$ pip freeze | grep rich
Traceback (most recent call last):
  File "/home/user/Text_Gen/text-generation-webui-main/installer_files/env/bin/pip", line 7, in <module>
    from pip._internal.cli.main import main
  File "/home/user/Text_Gen/text-generation-webui-main/installer_files/env/lib/python3.11/site-packages/pip/__init__.py", line 1, in <module>
    from typing import List, Optional
ModuleNotFoundError: No module named 'typing'

I'm afraid you have a problem with your Python environment, this isn't a Rich issue. Notice that trying to run just the pip freeze command gave you the same error.

The typing module is in the standard library. If that doesn't exist then something is amis with your environment, as Tom pointed out. Try recreating your virtual environement?


error with executing malware

what does the rest say?

Traceback (most recent call last):
File "resources\discord_token_grabber.py", line 5, in
File "PyInstaller\loader\pyimod02_importers.py", line 385, in exec_module
File "requests_init_.py", line 43, in
File "PyInstaller\loader\pyimod02_importers.py", line 385, in exec_module
File "urllib3_init_.py", line 42, in
ImportError: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'OpenSSL 1.1.0i  14 Aug 2018'. See: https://github.com/urllib3/urllib3/issues/2168

https://github.com/urllib3/urllib3/issues/2168
They state that you need a newer version of open SSL since python 3.10+ and some version of their package. Go download a newer version of OpenSSL

Ok

did that help?

no i updated my openssl to 3.1.1 and added it to path but it still give that error

Could u please make an issue in urllib telling them that you have code compiled using pyinstaller and you would like to know how to fix it?

yeah i fixed it

What did u do to fix it?


IJAL upload webapp  - introductory video

@davidjamesbeck
Just  a proof of concept - I hope I can count on you to redo this, with better audio, fewer "umm"s, and more linguistic knowledge!
IJAL upload video

Very nice. I’ll redo it, but maybe once we have a prettier interface and have the text display down pat.
David

On Feb 23, 2019, at 4:30 PM, Paul Shannon<EMAIL_ADDRESS>wrote:
@davidjamesbeck https://github.com/davidjamesbeck
Just a proof of concept - I hope I can count on you to redo this, with better audio, fewer "umm"s, and more linguistic knowledge!
IJAL upload video https://www.youtube.com/watch?v=ZBA62dCpyXI&feature=youtu.be
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/paul-shannon/slexil/issues/8, or mute the thread https://github.com/notifications/unsubscribe-auth/ApvDhkfWDb5h8SRHT3_9CLFDOK6xggtTks5vQc8AgaJpZM4bOOOc.


Multiplayer P1 Can Still Control P2 Using IJKL Keys

In multiplayer, player 1 can still control player 2's character using the I, J, K, and L keys on the keyboard. We should probably disable that, if possible.

A mix of local and networked multiplayer might be interesting to support one day, but for now lets disable 2nd player input entirely in the multiplayer sim.


Types: Clean up token type definitions

The token types defined in @namada/types haven't been actively maintained, and could likely be significantly simplified or removed. These types are created by mapping relevant data to supported tokens in the slip44 package.

Review existing Tokens & CosmosTokens types and determine what is actually required
Once any changes are in, regenerate documentation and update apps where applicable

Closing as we have new ways of configuring these


Proposal: Support for console AQUA_PUBSUB_DBSSL environment variable

This is a proposal for the support of configuring  the AQUA_PUBSUB_DBSSL environment variable in the same method as SCALOCK_DBSSL with db.ssl and SCALOCK_AUDIT_DBSSL with db.auditssl for consistency.
Currently, this can be configured with web.extraEnvironmentVars but is inconsistent.
I'll try to submit a PR this weekend, but I'm a helm authoring noob.

Thanks for reporting.
Added requested support in all active branches. Closing the issue.


add imgService

Save local image from monster, cards and equipment.

@ovniks thank you for your contribution, apologize for the delay in merging.


Update library.py with vbox 6.0.x xidl

ENVIRONMENT

Operating System: Any
Python version: Any
VirtualBox version: 6.0.4
VirtualBox SDK version: 6.0.4
Location where VirtualBox SDK is installed: any
pyvbox version: 1.3.2
[v] Happens in latest master branch?

SUMMARY
VirtualBox 6.0 contains new constants in enums, for example VirtualSystemDescriptionType. This breaks some api call, for example working with appliance (unknown values on conversion from numeric to enums)

I'm working on this currently.

Just released support for 6.0.x in virtualbox==2.0.0 (or pyvbox==2.0.0)


Unable to Use Starbusrt Trino ODBC to connect

as soon as i try to set the ODBC Connection or even attempt to connect i get the
FAILED!
[Starburst][Trino] (1010) Error occurred while trying to connect: SYNTAX_ERROR : USER_ERROR : line 1:1: mismatched input '{'. Expecting: 'ALTER', 'ANALYZE', 'CALL', 'COMMENT', 'COMMIT', 'CREATE', 'DEALLOCATE', 'DELETE', 'DENY', 'DESC', 'DESCRIBE', 'DROP', 'EXECUTE', 'EXPLAIN', 'GRANT', 'INSERT', 'MERGE', 'PREPARE', 'REFRESH', 'RESET', 'REVOKE', 'ROLLBACK', 'SET', 'SHOW', 'START', 'TRUNCATE', 'UPDATE', 'USE',

Thanks for your comment, @jlriverayuso. This issue has been assigned to the doc author for review.

Thanks for your comment, @jlriverayuso.
This looks like a support request or product question rather than documentation feedback.
You might find a solution in the Power BI Community. If not, try our support page.


fix disposition permission check

coverage: 84.867%. remained the same
when pulling da6ea1c54222740c69d6f48642468ec82f771272 on fix-disposition-link
into ad6c523168a91d55c714b0bb10fc87cd160b76df on main.


Cached result is not instantly returned when fetchPolicy is set to "cache-and-network"

Describe the bug
When settings the query options with { fetchPolicy: "cache-and-network" }, the cached result is not instantly returned.
Instead, it waits until the data is fetched from the server to return the result.
To Reproduce
Steps to reproduce the behavior:

Setup useQuery with the following options { fetchPolicy: "cache-and-network" }
Setup 2 query variables: variables1 and variables2
Execute the query with variables1, then variables2, then variables1 again — e.g. const { result } = useQuery(query, variable1, { fetchPolicy: "cache-and-network" })
On the second time you query with variables1, the change in result won't be instantaneous. Instead, it will wait for a few seconds before updating.

Expected behavior
The cached result should be returned instantly.
Versions
vue: "^3.2.16"
@vue/apollo-composable: "^4.0.0-alpha.16"
@apollo/client: "^3.5.8"

We have identified the same issue.
In the meantime, for this to be corrected, it would seem that if we add
notifyOnNetworkStatusChange: true,
at the useQuery level it works. But impossible to put it at the level of the config of apollo in defaultOptions.
Wait and see :)

some problem

notifyOnNetworkStatusChange: true does fix it for fetching the query the first time. But if your component refetches the query (because of reactive variables changing or whatever), it will have the same issue again. So it's a workaround for some simple use cases, but definitely not for all use cases (and it's pretty common to use reactive variables in queries).

If you're affected by this, could you please try #1388? You can setup linking manually or put "@vue/apollo-composable": "npm<EMAIL_ADDRESS>in your package.json. Let me know if it fix (or does not fix) your issues!

@ishitatsuyuki fixes the issue for me.

Heads up: there's now a release that ships the fix of @ishitatsuyuki https://github.com/vuejs/apollo/releases/tag/v4.0.0-beta.1

This is fixed in vue-apollo 4.x, but the same logic exists in vue-apollo 3.x, see:
https://github.com/vuejs/apollo/blob/fdc6fb869202e49cd1194a71876394a742deed1d/packages/vue-apollo/src/smart-query.js#L115-L123
Removing the code also fixes the issue, but I found that at least in 3.x, the code does something.
Consider the following:

parent component executes query against network, and then renders child component
child component executes a query against cache-first

with the above code, the child component has the cached data before the component tries to render the template. Without the code, it doesn't (and you have to use v-if etc. to wait for the data)
This brings me to the point that vue-apollo is still not well tested. The pull request was merged without an accompanying test case, and the code that was removed apparentely wasn't covered by a test case either (but this may be because it was copied from the 3.x branch). As long as the testing strategy is subpar, errors like this will pop up again and again.

I ended up patching the @vue/apollo-option directly inside vue-apollo-option.esm.js for now, I'm not 100% sure of the logic, but it solved the issue for us with "cache-and-network" not returning any cached results. Also network-only and cache-only seemed to work as expected.
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index e5a7f2f..efb1d95 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -783,7 +783,8 @@ var SmartQuery = /*#__PURE__*/function (_SmartApollo) {
         var currentResult = this.retrieveCurrentResult();
         if (this.options.notifyOnNetworkStatusChange ||
         // Initial call of next result when it's not loading (for Apollo Client 3)
-        this.observer.getCurrentResult && !currentResult.loading) {
+          (this.observer.getCurrentResult && this.options.fetchPolicy !== 'network-only') ||
+          (this.observer.getCurrentResult && this.options.fetchPolicy === 'network-only' && !currentResult.loading)) {
           this.nextResult(currentResult);
         }
       }


add analyses endpoint

This is the second PR replacing #309

Thanks for splitting these up, @rentzso!

No problem @nagem
Thank you for reviewing this two monster PRs !

Incidentally, because this change is large, if you could avoid force-pushing that will make it a lot easier to review. Up to you, I know I force-push a lot haha :)

Agreed, no force-pushing!

I'll take another pass at reviewing this on Monday but it LGTM as a first implementation of Analyses. My only comment from an architectural standpoint would be that we consider moving Analyses out of a subdocument space and into their own container so any additional enhancements aren't limited by what mongo can do in a list of a list (or having to pull the object into python and send it back). Unfortunately if we do that we'll break a lot of cont_name + s code 😉.
Nice work, @rentzso.

I think @nagem makes a good point regarding nested subdocument arrays.
@rentzso IIRC, Elasticsearch considerations went into the current design. What do you think about promoting analyses to their own collection?
Also, I think we should keep the + s convention alive by choosing a different name, such as computation.

Yes. Elasticsearch was the main factor as having analyses external to containers increases the complexity of some searches on analyses, at least in its current design. The feature is not in any PR yet, but it probably will.
However, Changing this now could be a big time investment on something that (from first tests) seems to work fine.
I understand the concern for the double nested list, but mongo doesn't block updates on nested lists. I was concerned too at first. :)
But with mongo aggregation pipelines, it doesn't seem a big deal...
Let me know though if you have performance concerns.
Another thing that needs to be evaluated is if for each analysis there will be always a related container. If not the nested option could be problematic.
About the singular vs. plural issue I don't understand exactly why singular names arrived in the API.
Mongo Collections are plural so why can't we make everything plural? :)
It could be grammatically incorrect sometimes, but it should save us some headaches. Or am I missing something?
A big thank to @nagem and @kofalt for reviewing this!
On May 20, 2016, at 4:31 PM, Gunnar Schaefer<EMAIL_ADDRESS>wrote:

I think @nagem makes a good point regarding nested subdocument arrays.
@rentzso IIRC, Elasticsearch considerations went into the current design. What do you think about promoting analyses to their own collection?
Also, I think we should keep the + s convention alive by choosing a different name, such as computation.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

As stated before, I am firmly opposed to compromising on the modeling and implementation of our primary database in order to simplify the integration of external features, such as Elasticsearch.
Btw, two main takeaways from my visit to MongoDB office hours:

design your data model such that common operations are simple and elegant
stay away from nested arrays

👍 on grammatically incorrect, but simplified level naming.

For what it's worth Renzo, I'm certainly the cause of some of the plural versus singular mix - a job.input.x.type would be acquisition not acquisitions, for example, and that's reflected in various parts of the code. If the team decides otherwise we could change things, but that's low ROI :/
My personal assumption has been that, eventually, plural nouns will only be used for the URL routes (with plurals then stripped before app logic) and for mongo table names (added at the very end, ideally by helper functions), and be singular everywhere else.
But I've largely left it alone because we have larger fish to fry :)

@kofalt thanks for the explanation. If we all agree, we should track this on a tech-debt issue to "make everything plural".
@gsfr the main reason was Elasticsearch, but it wasn't the only reason. 😄
Having analyses as a subdocument was the simpler solution. Permission handling was trivial and, as analyses at the beginning were immutable, also their sublists were supposed to be immutable. Unfortunately, now, we have notes (and tags) that are not.
I am not opposed (in principle) to putting analyses in a separate collection, but please note that the current implementation seems to work well and refactoring will take time (3-4 man-days).
I am definitely interested in understanding more the reasons why we should avoid nested arrays.
Can we have another mongodb office hours when I am back or after OHBM?
It could also be a good time to ask if what I did with the mongodb aggregation is a good/acceptable or bad practice. I have already followed this advise (cc: @nagem @kofalt): https://docs.mongodb.com/manual/core/aggregation-pipeline/#early-filtering
Let me know what you think should be the next step on this issue.

@rentzso I agree that currently we don't hit any issues with analyses being a subdocument (other than a slightly awkward file lookup with aggregations). This is the particular bit of mongo doc that would make editing anything in these subarrays difficult:

The positional $ operator cannot be used for queries which traverse more than one array, such as queries that traverse arrays nested within other arrays, because the replacement for the $ placeholder is a single value

Some potential tricky situations that may or may not apply depending on how immutable analyses are:

With the addition tags, if they follow the group tagging pool logic (a tag renamed in the group pool is renamed everywhere), we would have to pull the tag and reinsert it.
Any modification to a file (name, type, measurement, ...)
Modifying the contents of a note
Schema upgrades already have issues with updating an array of objects (must be done in code), but we can currently treat all containers the same (see this update script)

After a discussion with @gsfr and @rentzso, the plan for edits to the first iteration (this PR) of Analyses is as follows, and I would appreciate @kofalt, @lmperry, and @dpuccetti to weigh in on any problems they see.
One major point is there are two types of analysis, one is tied to a job internally and one whose outputs are created externally and uploaded by a user. This PR should support both. One major outstanding question is whether we denormalize data from the job object to the analysis object for internal analyses.
Major changes:

An analysis object is going to be a standalone container with its own Mongo collection and based on the base container in our data model
Move AnalysesHandlerto its own handler file with plans to merge it into ContainerHandlerin the future, but leaving the endpoints the same for now
Use existing logic in ListHandler where possible
Require POST to have parent container type and parent container id
outputs field will become files to match base class

Analysis object:
Italicized fields are inherited from the base container.

Field
Type
Description

_id
string
database ID

parent_id
string
parent's database id ID

parent_cont
string
projects/collections/sessions

label
string
label

description
string
description

user
string
user id of the submitter (automatically set)

version
string
version id (unknown if this is necessary or helpful)

job
string
identifier of the job that generated the analysis

inputs †
[File, …]
array of File objects with the analysis file inputs*

created
timestamp
creation time (automatically set)

modified
timestamp
modification time (automatically updated)

notes
[Note, …]
array of Note objects

files †
[File, …]
array of File objects that represent analysis file outputs*

views
integer
not supported in v1 of analyses

downloads
integer
not supported in v1 of analyses

permissions
[Permission, …]
array of Permission objects inherited from parent container

public
boolean
not supported in v1 of analyses

archived
boolean
not supported in v1 of analyses

tags
[string, …]
array of tags

info
object
map of arbitrary information

† These will exist on the job object if internal analysis.
API endpoints:

API endpoint
Method
Behavior/Use

/api/analysis/<id>
GET
retrieve analysis by id

/api/analysis
POST
create an analysis, must include parent_id and parent_container

/api/analysis/<id>
DELETE
delete analysis

/api/analysis/files
GET
retrieve file info for analysis

/api/analysis/files/<filename>
GET
download file

/api/analysis/notes/
POST
add note

/api/analysis/notes/<note-id>
DELETE
delete note by id

`/api/<collections
projects
sessions>//analysis/`

/api/upload/analysis
POST
upload files and metadata related to analysis output. Used by engine (results of job) or UI (external analysis)

Another option for dealing with duplicated info from the job object is to merge the info from the job into the inputs and files arrays during a GETcall to an analysis object, and/or just inflate the job object from the id and add it to the returned analysis object.

Based on the PR so far, it seems as though rules will not run on analysis containers. If that's the case, I request that the changes to Placer.save_file be reverted, and the new Placer implementation can just call files.move_form_file_field_into_cas(field) directly. That will keep things clearer, IMO.

I dislike duplicating object data, but I've also never heard of this "internal" use case before now. I don't have many comments on that because I don't feel I understand the feature.
If we do copy-paste, a.inputs should match the job doc, IMO. I would press for outputs to do the same, but I guess that doesn't make sense as this container will actually hold the files itself. That means, implicitly, and file on the doc is a gear/whatever output. Should we disable "Add Data"?
Overheard some rumbling about immutability. Where did we land on this? Which keys can be modified? Will permissions work differently in that regard? Whichever keys are mutable should not be duplicates.
Suggest removing user and adding origin. More flexible & consistent.

After a talk with @gsfr,

No, rules will not run on analyses
We will not denormalize data (copy/paste) from the job, but will fill out those fields from the job for GETs. Internal (job) analyses will be a lot more sparse than external analyses but they should both look nearly the same when returned from the api
Internal (job) analysis will have immutable inputs and outputs

Rebased.

I think we have a lot to think about for tweaks we can make to inputs and outputs and how they align with the inputs and outputs of jobs. What we already have is a great start and should probably be merged in as is, with incremental tweaks (and any necessary schema upgrades) happening as we make more decisions on how analyses will be used and stored.
Overall, great work @rentzso!

Rebased and removed some debug logging, ready to merge when CI is green.


why get bad result?

however in diffusers, it works well. same inference config.
sdxl

did you implement original code completely?

Hi! Sadly I don't think there's any better implementation of HiDiffusers for ComfyUI at the moment (pretty sure the HiDiff dev said they're working on implementing it to UI's, or at least making it easier to do so). However, while I don't ALWAYS get good results, the most stable results for SDXL models I've been getting were with the following settings (res: 1024x1536):

I know they're different from recommended (especially start and end times), but they're way more reliable than defaults or the Simple node implementations. Furthermore I'm unsure why but the documentation says to use output blocks "4,5" however, in code it's "5,4" and that also for whatever reason affected the end results in my tests, not entirely sure if order should matter or not as I haven't looked into the original HiDiff code.

did you implement original code completely?

like the README says: "My implementation may or may not work correctly."
i tried to understand the code and i made a best effort attempt to implement it correctly, but there's definitely no guarantee that it's perfect. if anyone finds a specific part that i didn't implement correctly and points it out, i'd naturally try to fix it.
i use it with SD 1.5 and it works well. i haven't seen very good results with SDXL, i'm not sure what the difference is if it works fine in the diffusers implementation.


Soil/Sediment Visual Metrics

Update: will include in future versions of the data package. Will need help standardizing this

Update 2023-12-20
All columns have been standardized except cohesion_post_lyopholization and notes:

cohesion_post_lyopholization: based on the entries, it seems like this could be broken into two columns (aggregated and easy to break apart), but I'd like to talk to @amyerspigg or @kaizadp about what this column means before I do so

notes: good amount of repeats in here from what the other columns address, but since this is a qualitative analyte, the notes seem important here. I'd also like to go through these with AMP or KP and decide whether to remove or clean up.

Kits that need attention:

[ ] K010 sediment: no data but metadata says it was collected
[ ] K001 and K007 need to be removed
[ ] K046 transition and wetland switched (we know how to handle this, #60)

cohesion_post_lyopholization: based on the entries, it seems like this could be broken into two columns (aggregated and easy to break apart), but I'd like to talk to @amyerspigg or @kaizadp about what this column means before I do so

I don't know the value of this column. I say drop, unless @amyerspigg thinks otherwise. Need to fix the spelling - "lyophilization"

Example entries:

@amyerspigg and @kaizadp agree to drop cohesion_post_lyopholization and notes


Menu for Angular components library

Pull Request Template
Description
Implements AgnosticUI Menu for Angular Component Library, accepting content for the menu items.
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Checklist:
Please delete options that are not relevant.
- [ ] Is this a bug fix
- [ ] Does the change require CSS updates to existing components? If so, have you updated all frameworks and verified?

[x] Is this a new feature
[x] Have you submitted new feature or bugfix issue? Unless it's a small bugix, we generally prefer to track PRs against an issue which starts a dialogue to give us context.
[x] Does this add a new component? If so, have you updated all framework's copystyles.js to pull in said CSS and verified it's working properly?
[ ] Tests passing?
[x] Have you added to the framework's /examples apps? e.g. agnosticui-react/examples? These are kitchen sink sanity checks which help to verify a component is working and derive documentation snippets.
[ ] Have you updated the docs? These live in site/docs

We realize above checklist is pretty intimidating, reach out with an at mention on your issue if you're interested in contributing but need some clarifications!

@roblevintennis I took a couple of swings at cleaning up the commit log with an interactive rebase, but I kept getting locked up. Let me know if you would like that to be done and I can work through it (not something I have much experience with).
Biggest change from the last time you reviewed this is on the Angular Dependency Injection side. I previously was trying to use the MenuComponent as a type in ng's DI. That leads to a circular dependency. ng-packagr would still build it, but Storybook wants nothing to do with it. So I've abstracted the MenuComponent to an interface, which the component then implements. It'll be a little bit trickier to maintain, but since the component implements the interface, there will be errors

Thanks for the updates🙏🏽
Don't worry about the commit log for now but thanks for trying.
I had a pretty quick look through and nothing stood out egregious so that's good 👍🏽
It's a pretty big PR so I think it's a good idea for me to fetch your remote and do a round of spot checks. My weekend is is shot with sick kiddos to watch so I'll try to get to it early next week. Thanks again!

@vitale232 Thank you so much for your efforts on this tricky component! I went ahead and did a pretty thorough QA and created https://github.com/AgnosticUI/agnosticui/issues/181 for follow up fixes. I think this is a significant step forward and worth it to just merge it for now. There's another reason to…
I'm midway on a fairly major refactor and I'd like to get your PR here merged so I can do the minor rewiring of some of the style copies from copystyles.js. I've left details on the follow up issue but basically I started to break out some of the buttons.css in ways that will sort of break all framework implementations 😋  if I don't do this rewiring. It's an interesting problem and perhaps drawback of AgnosticUI's shared CSS approach that I'll have to contemplate but I think it's fine for now.
Let me know if you feel compelled to take on https://github.com/AgnosticUI/agnosticui/issues/181 or some of the items (I can break that up into separate issues if it helps). Well done here though! I'm so happy you contributed this nice improvement to the Angular package! 💯 🔥 🙌 🐨 😉

@vitale232 Thank you so much for your efforts on this tricky component! I went ahead and did a pretty thorough QA and created #181 for follow up fixes. I think this is a significant step forward and worth it to just merge it for now. There's another reason to…
I'm midway on a fairly major refactor and I'd like to get your PR here merged so I can do the minor rewiring of some of the style copies from copystyles.js. I've left details on the follow up issue but basically I started to break out some of the buttons.css in ways that will sort of break all framework implementations 😋 if I don't do this rewiring. It's an interesting problem and perhaps drawback of AgnosticUI's shared CSS approach that I'll have to contemplate but I think it's fine for now.
Let me know if you feel compelled to take on #181 or some of the items (I can break that up into separate issues if it helps). Well done here though! I'm so happy you contributed this nice improvement to the Angular package! 💯 🔥 🙌 🐨 😉

Thanks so much for the thorough review! Glad we were able to move the libs forward a bit.
I'm definitely interested in helping out with #181. I'll probably only have a few hours per week to dedicate to this, so as long as that's not a problem I'm on board!
I've also been thinking about adding some structure into the ng lib. Specifically, I think it would be nice to start splitting the components into their own little modules. ag-ui is pretty well positioned to be one of the first Angular libs to embrace standalone components, assuming that lands soon. So, perhaps instead of having a kitchen sink AgModule, we would have AgMenuModule. Then, when Angular lands standalong components, we'd be positioned to migrate to the public API becoming  imports: [AgButtonComponent, AgMenuComponent] in the consumer's metadata. If that's something you're interested in, too, I can give it a bit more thought an open an issue.

Wow that RFC is huge lol. I know that I got tripped up on NgModule when I started my job (not quite a year ago) where we use Angular. Interesting that they're looking into these "standalone components" which is really much closer to other framework ecosystems. I have no idea which is or will be "better" in the Angular landscape. Honestly, I'd assume you have a better sense of that pulse. I'd definitely be open to the approach 👍

ag-ui is pretty well positioned to be one of the first Angular libs to embrace https://github.com/angular/angular/discussions/43784

Maybe you're trying to say this could be a bit of a differentiator? Ok, cool 😎 👍
In terms of helping with #181 a few hours per week is wonderful and appreciated :)
I don't know if I mentioned but I'm really hoping folks such as yourself with more Angular experience can help to shape the Angular package. I did it primarily because 1. Angular is still very popular and used in a lot of important apps so I wanted to support it 2) It helps me to slowly get better at Angular myself which in turn helps me at work.
But I'm still not very good at it so it really helps to have someone come in and create a directive, mention upcoming RFCs and interesting directions! 🙏


General dependency update

Most of the current dependencies use quite old versions:

JUnit 4.11 was released on November 2012.
The Jackson version (2.1.4) currently used was released on February 2013.
Apache Commons Codec v1.7 was released on September 2012.
The Joda time library used was released in February 2012. Furthermore, from Java SE 8 onwards, users are asked to migrate to java.time (JSR-310).
(The Jetty client is also outdated in the current version, but hopefully it will be updated soon)

So, would not be convenient to update these?

This is an ongoing need, but I'm going to close this issue as I think they've all been bumped since the versions listed. Feel free to open another issue/pull request when there is an important update to a dependency, recognizing that we have to maintain backwards compatibility within major versions.


Problem with imports in angle brackets

I installed the latest version of Motif (0.3.1) through cocoapods but I'm seeing the following error when building:
.../Pods/Motif/Carthage/Checkouts/libyaml/src/yaml_private.h:6:10: 'yaml.h' file not found with <angled> include; use "quotes" instead

This header is marked as Public in the produced framework, should it really be so? (the name seems to indicate otherwise :) )
The same thing also happens with the config.h import in the same file. If I change both to quotes as the error message suggests, it works.

Fixed in #88 / 0.3.3


yara: 4.0.1 -> 4.0.5

Motivation for this change
Update from 4.0.1 to 4.0.5
Includes a fix to work around https://github.com/VirusTotal/yara/issues/1459
Things done

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

[ ] NixOS
[ ] macOS
[X] other Linux distributions

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

Result of nixpkgs-review pr 115084 at a30bbce1 run on x86_64-linux 1

1 package built:

yara

1 suggestion:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/tools/security/yara/default.nix:40:3:
   |
40 |   meta = with lib; {
   |   ^

Looks like a duplicate of #114562

Looks like a duplicate of #114562

You are correct, I will close this PR.


Azure DevOps CI/CD - Invalid YAML file in documentation

The following task is invalid:
  - task: Install .NET Aspire workload
    inputs:
        inlineScript: |
            dotnet workload install aspire

Error is:
/.azdo/pipelines/azure-dev.yml (Line: 22, Col: 11): Task reference must be in the format <NAME>@<MAJOR_VERSION> or<EMAIL_ADDRESS>For example MyTask@2 or<EMAIL_ADDRESS>The following task reference format is invalid: 'Install .NET Aspire workload'

Source: https://learn.microsoft.com/en-us/dotnet/aspire/deployment/azure/aca-deployment-github-actions?tabs=windows%2Cvisual-studio&pivots=azure-pipelines

Potential fix
  - task: Bash@3
    displayName: Install .NET Aspire workload
    inputs:
      targetType: 'inline'
      script: |
        dotnet workload install aspire

There is no azure-dev.yml in this repository... Are you sure you're in the right repository?

This issue refers to the AZD documentation, not the Git repo. Please click on the link provided above.


(C#) #endregion directive is not being highlighted in the editor

VSCode Version: 0.10.11
OS Version: OSX 10.11.4 (El Capitan)

Steps to Reproduce:

Open Visual Studio Code in a C# project.
Create a #region, #endregion block.

You'll notice that #endregion is not highlighted in the same way that #region is.

This issue was moved to OmniSharp/omnisharp-vscode#174


in heron what means storm taskid?

in storm tuple, I can get which task it come from, then according to this taskid, i can hash this tuple to diffrent thread, but in heron Tuple no longer supports getSourceTask, so what should i do?

We have considered whether to keep this interface a long time ago and decided not to support it. Can I know what the benefits are to "hash this tuple to different thread" basing on the taskid, instead of its value , i.e. tuple.hashCode()?

@daxwang - should we close this, if this is not an issue?


Add lint: commands in commands

Discussed on Discord.
This is bad because it's pointless indirection and results in confusing ordering. Generally you should operate using the World directly in your command. Command::apply can be used to call any command as an ordinary function.

I'm curious what a valid use case for World::commands() would even look like?

Yeah, that method is 100% intended for tests as far as I'm aware. Maybe it would be useful with a weird remote scripting / dev tools API?

I personally use it in combination with flush because I'm too lazy to find out which struct enqueued by Commands' convenience methods that are not available on World.

After a quick search within bevy itself, I see the following use classes:

Tests
Component on-add and on-remove hooks
In SceneSpawner::spawn_queued_scenes, with the comment "defer via commands otherwise SceneSpawner is not available in the observer"

Number 2 is on DeferredWorld though, not World. Calling commands there is very natural imo.

I use commands to access normal-system APIs that use commands from exclusive systems. I'm against this lint.

My suggestion for this lint is to only lint on World::commands within an impl Command for Foo block.


The meta-updater layer brakes the cron daemon

The 'cronie' recipe creates the directory /var/spool/cron/ (by this command).
This directory is removed somewhere during the OSTree image creation.
As a result, during boot the crond daemon fails to start with the error:
Starting crond: /var/spool/cron: No such file or directory
/var/spool/cron: mkdir: No such file or directory
FAIL

Why does it happen?
How can we make the cron daemon work with the meta-updater layer?
Thanks.

Hi Guy,
The root cause of this is because /var special. It is persisted between updates, and thus isn't updated by OSTree.  Ideally recipes would create any temporary directories they need automatically at run time--this is required if the package isn't in the initial image that is flashed to the device, because we don't know about them when writing the initial image.
There are some special cases handled in image_types_ota.bbclass, which are not really a good solution to the problem, but are required until 'stateless' becomes the norm.
Are you using systemd?  If so, I would propose changing the cronie recipe to use systemd-tmpfiles and create this directory at boot time.

My usage for the daemon is for recipes that have pre-defined tasks (like ntp). For that, lets move the directory to a path managed by ostree and not a temporary one.
I'll try to modify the cronie recipe to use /usr/share/cronie-spool/.
Additionally, I'll add step on image_types_ostree to move the files from /var/spool/cron.
Does this make sense?

Closed on #132


Design: Update the LEOS screen showing the coins can be vested or staked (locked)

Iteration 1
Probably requires some discussion, the vesting feature is more complicated then I expected

New iteration


Document features that required 19.2 upgrade finalization

Fixes #5646.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3cbc2b3157eb5cd1b1d568c606d3811d9a05249a/
Edited pages:

v19.2/upgrade-cockroach-version.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0532ef7f5d21764c416cf438272e54fb22343e5f/
Edited pages:

v19.2/upgrade-cockroach-version.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/434d40f1136f7f1dc6857db41a84e9b5fb6aa226/
Edited pages:

v19.2/backup-and-restore.md
v19.2/upgrade-cockroach-version.md


Increment version for servicebus releases

Increment version for servicebus releases

Can one of the admins verify this patch?


[Bug] Aborted (core dumped) error on concurrent requests in lmdeploy v0.0.14 with qwen-14b-chat-4bit model

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.

Describe the bug
I am experiencing a critical issue with lmdeploy version 0.0.14, where the service crashes with an "Aborted (core dumped)" error when handling concurrent requests.
I noticed there is a PR related to this bug. However, it seems like this issue hasn't been resolved in version 0.0.14.
I would appreciate any assistance in resolving this issue, as it significantly impacts the usability of the service in high-traffic scenarios.
Reproduction
lmdeploy serve api_server ./workspace --server_name <IP_ADDRESS> --server_port 5000 --instance_num 64 --tp 1
Environment
- lmdeploy Version: 0.0.14
- Model Used: qwen-14b-chat-4bit
- GPU: NVIDIA A100 40GB

Error traceback
Aborted (core dumped)

Which PR?

Which PR?

#676 #686

@lin-xiaosheng Hi，how do you quantize to get the qwen-14b-chat-4bit model? Is the accuracy of your qwen-14b-chat-4bit model similar to the official qwen-14b-int4 (gptq)?
I am using https://huggingface.co/lmdeploy/qwen-chat-14b-4bit

It's been a while since we release v0.0.14.
One suggestion is to upgrade to v0.1.0, which will be released this week.
Another is to apply the related PR to a branch v0.0.14


qqmusic: init at 1.1.5

Description of changes
Tencent QQ music is a music streaming service in China.

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
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[X] Fits CONTRIBUTING.md.

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

My Chinese font seemed to be unhappy.

My Chinese font seemed to be unhappy.

I cannot reproduce this issue. Which Chinese fonts do you have installed?


Project error

Hi,
Thanks for your project but error showing
./configure: error: no /tmp/ngx_pagespeed-<IP_ADDRESS>-beta/config was found

I admit I should update it, it's just very painful to maintain this one.

FWIW, there's no one with proper docker knowledge/experience involved in maintaining the apache ones,
and help is surely welcome there :-)

(For master, I suspect taking https://hub.docker.com/r/envoyproxy/envoy-alpine as a base could be convenient)

I'm really reluctant to use a base image that is not an official OS of Docker Hub and that doesn't even show its sources, for a web server. That's a clear no no.
The Dockerfile you pointed out is useful, however it's 2 years old. After trying a bit, the main issue is that the documentation regarding how to build manually fails with weird errors on Alpine. It says that it cannot find PSOL even though I could extract it. I wonder if it's trying to find the PSOL for my Alpine and cannot find that one specifically. Building PSOL is badly documented and relying on so many things it should not: bash, lsb_release...

I can add that, but it's also possible to propose it over at:
https://github.com/apache/incubator-pagespeed-mod/tree/master/html
(modpagespeed.com mirrors that)

Regarding Envoy's docker images, I think they are available via https://github.com/envoyproxy/envoy/tree/master/ci (the continuous integration pushes them). But I wasn't lying when I said I'm not expert here, so just pointing it out; if it's useful -- I'll leave that up to you :-)

What if ngx_pagespeed as provided as an AppImage? I'm not sure but it might work on Alpine then without almost any work. https://github.com/AppImage/AppImageKit/issues/1015
I'll update the dev branch with what I currently have.

One more issue regarding build PSOL: The latest release does not seem to have submodules defined: https://github.com/apache/incubator-pagespeed-ngx/blob/v<IP_ADDRESS>-stable/.gitmodules
So it looks like either need to guess a compatible PSOL, or build from latest snapshot. Neither are good options.

@oschaaf Once there is a release with PSOL source code, I'll try again to compile PSOL.

Well, there's psol-<IP_ADDRESS>-apache-incubating-x64.tar.gz

Sorry - that went out too early; I wanted to point to https://dist.apache.org/repos/dist/release/incubator/pagespeed/<IP_ADDRESS>/x64/

Thank you. Btw. I replied on https://github.com/apache/incubator-pagespeed-mod/pull/2021


feat: Add autosuggest quick navigation (second try)

Addresses CX-1761
Description
Adds quick navigation items ("Artworks" & "Auction Results") to the search bar's autosuggest items for artists.

Note that this search bar code is a great candidate for a complete rewrite

@damassi I've got a branch where i rewrote it already :p

oh what! Damn 👌 👌

https://github.com/artsy/force/compare/dzucconi/refactor/autocomplete-integration?expand=1#diff-e310f1cc3bb59bb47b8f02ad047e93ffbe464faf299a09afa405fa5c3d1a0d8d — but I need to port this work over

Needless to say, your version simpler 😄