phpmd cannot run with parameters

I'm trying to integrate phpmd to run with PhpStorm, alas I'm getting the following error consistently:

PHP Mess Detector
phpmd: Can not correctly run the tool with parameters:
C:/Users/me/AppData/Local/Temp/phpmd_temp.tmp/folder/myclass.php
xml codesize,controversial,design,naming,unusedcode
Possible tool process hangup after 30 sec.

I found the error on Stack Overflow, but none of the solutions there would fix the problem.
I'm using PHP 5.6, phpmd 1.5.1 and PhpStorm 2016.3.1. I already tried maxing the tool process timeout, and I'm not using XDebug or any other debugger for that matter.
I'm running out of ideas, so any suggestion is appreciated.

Hi Martin,
Did you mark directories?

Hey,
thx for the answer. But no, I did not mark any directories.
I did configure phpmd as a PhpStorm inspection though, as described here. So the error is triggered when I save changes in my php class.
I should add that I can run phpmd without error and expected results by running it from the command line against my php class.

phpmd 1.5.1, really?

phpmd 1.5.1, really?

Yes, as per my composer.lock:

Is there something I'm not getting here...?!

This version is not supported anymore, please use a more current version, e.g. 2.5.0.

Oh shoot, that happens when you follow a documentation to the letter...
Alright, did the update to 2.5, and unfortunately the error still persists with the exact same error message...

@martin-edelmann though I also use PhpStorm, I normally use PHPMD only through the CLI or our Continuous Integration system (Jenkins).
But I just configured and checked one of our apps successfully without getting any errors.
I'm going to close this, as I think it's a local configuration issue.
Feel free to get in touch with me through mail or Skype (I speak German by the way).


reduce time boxen are vulnerable during deployment

We create, then manage packages, then do everything else, then run secure.yml, which takes tens of minutes.
We should run secure.yml first. We don't currently because it's hard to set up ansible/vagrant once we change the ports and restrict iptables. Maybe have an ansible flag of some kind to disable running that, explicitly disable it for the dev vm deploy process, default enabled.
We should manage packages second. Yum update, yum remove, yum install.
Maybe in the future make a base image and start with that.

do not wait for dns, make deploy process work with IP address

DO firewall tool might help, basically prebuilt iptables
https://blog.digitalocean.com/cloud-firewalls-secure-droplets-by-default/?utm_medium=email&utm_source=local&utm_campaign=CloudFirewallsAnnounce


doesn't start on Win10 with nvim-nightly

Hi, I wanted to check neovide but it doesn't start for me.
I use neovim-nightly, installed via scoop (it works ok as a standalone and embedded as a firenvim in firefox)
NVIM v0.5.0-330-g71ee46acc
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x64/cl.exe /DW...ects/neovim/src -IC:/projects/nvim-deps/usr/include -IC:/projects/neovim/build/src/nvim/auto -IC:/projects/neovim/build/include
Compiled by appveyor@APPVYR-WIN

Features: -acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM\sysinit.vim"
  fall-back for $VIM: "C:/Program Files/nvim/share/nvim"

Run :checkhealth for more info

When I start neovide.exe nothing pops up, I can see a process of neovide but that is it.

Logging is implemented, better error messaging is in, and a new window system is implemented. Could you try building from the sdl2 branch and see if that changes anything?

@habamax

Do you think now I would be able to build it from source code?

maybe. I added some more instructions in the readme in the sdl2 branch. Specifically

Install CMake. I use chocolatey: choco install cmake --installargs '"ADD_CMAKE_TO_PATH=System"' -y
Install LLVM. I use chocolatey: choco install llvm -y

Those dependencies may incidentally fix your issue

If you aren't able to build from source still though, I will ping this thread again once I have merged the sdl2 branch and made a release for you to try

@habamax sdl2 merged and the current release has it

@Kethku it works!
Thank you!

This happens when I install vim and nvim-nightly via scoop

Hm this still seems to happen for me for both neovide 0.5.0 (scoop) and 0.7.0 (from releases) with an empty init.vim (I added the line finish at the beginning to immediately exit). I tried running neovide --wsl from powershell and neovide.exe --wsl from wsl (for both versions), and both lead to similar results (window not appearing or appearing and immediately closing)

Facing the same issue on Windows 10.

@starptr @kabirnayeem99 Can you try running the latest build from actions (https://github.com/Kethku/neovide/actions/runs/947940077) instead? If the issue persists, please try running neovide with neovide --log, and send the results here.

I have installed this version. But the same issue persists. Double clicking on the .exe file opens neovide, but i can't open it from cmd.

Closing as stale. If you get back to this, please upload the file created when you run neovide with --log and reopen.

Hello  @Kethku i am having a prbl that is when i try  run neovide it just pop's a window for a second  then it disapperes
Can you pls help me out
i tried running neovide --log the output is
thread 'main' panicked at 'called Result::unwrap() on an Err value: NoAvailablePixelFormat', src\window\mod.rs:290:10
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
i have build neovide form source
waiting for your reply


typo in "QUERY_CAPTURE_MODE to AUTO and sets other recommended options in SQL Server 2017 (14.x) " example

There is a typo here.  The last comma after   WAIT_STATS_CAPTURE_MODE = ON, should not be there.
The following example sets QUERY_CAPTURE_MODE to AUTO and sets other recommended options in SQL Server 2017 (14.x) to include wait statistics:
SQL
Copy
ALTER DATABASE [QueryStoreDB]
SET QUERY_STORE = ON
(
OPERATION_MODE = READ_WRITE,
CLEANUP_POLICY = ( STALE_QUERY_THRESHOLD_DAYS = 90 ),
DATA_FLUSH_INTERVAL_SECONDS = 900,
QUERY_CAPTURE_MODE = AUTO,
MAX_STORAGE_SIZE_MB = 1000,
INTERVAL_LENGTH_MINUTES = 60,
SIZE_BASED_CLEANUP_MODE = AUTO,
MAX_PLANS_PER_QUERY = 200,
WAIT_STATS_CAPTURE_MODE = ON,
);

@campbellwarren -- thank you for catching this. I am looking into the article.

Happy to help.  The robust documentation for Query Store is appreciated!

Thank you for your feedback.  I have updated the documentation and it should be live by tomorrow.


Refactor vector storage to correctly handle relevancy scores

Description: This pull request aims to support generating the correct generic relevancy scores for different vector stores by refactoring the relevance score functions and their selection in the base class and subclasses of VectorStore. This is especially relevant with VectorStores that require a distance metric upon initialization. Note many of the current implenetations of _similarity_search_with_relevance_scores are not technically correct, as they just return       self.similarity_search_with_score(query, k, **kwargs)  without applying the relevant score function
Also includes changes associated with: https://github.com/hwchase17/langchain/pull/6564 and https://github.com/hwchase17/langchain/pull/6494
See more indepth discussion in thread in #6494
Issue:
https://github.com/hwchase17/langchain/issues/6526
https://github.com/hwchase17/langchain/issues/6481
https://github.com/hwchase17/langchain/issues/6346
Dependencies: None
The changes include:

Properly handling score thresholding in FAISS similarity_search_with_score_by_vector for the corresponding distance metric.
Refactoring the _similarity_search_with_relevance_scores method in the base class and removing it from the subclasses for incorrectly implemented subclasses.
Adding a _select_relevance_score_fn method in the base class and implementing it in the subclasses to select the appropriate relevance score function based on the distance strategy.
Updating the __init__ methods of the subclasses to set the relevance_score_fn attribute.
Removing the _default_relevance_score_fn function from the FAISS class and using the base class's _euclidean_relevance_score_fn instead.
Adding the DistanceStrategy enum to the utils.py file and updating the imports in the vector store classes.
Updating the tests to import the DistanceStrategy enum from the utils.py file.

@dev2049

Any plan to rebase this patch? I have tested it and it was working fine.

Nice work @raymond-yuan. Can you merge master to resolve conflicts and have a look at any Lint errors (I just kicked off tests again)?

I have merged master at https://github.com/flepied/langchain/tree/relevance_refactor and it seems to work well with my tests...

I have merged master at https://github.com/flepied/langchain/tree/relevance_refactor and it seems to work well with my tests...

Great. When @raymond-yuan is back online, I'll let him resolve the merge conflicts.
I had a quick look, but there were a few to work though, so I wanted to make sure they are resolved correctly.

@raymond-yuan I see tests for chroma, faiss, pgvector, singlestore. but a larger set of DBs (pinecone, etc) are modified; tests for some like pinecone are tricky b/c they are no in memory, but do we have confirmation (e.g., from users) that this is a no-op / backwards compatible? i see the the existing logic is typically moved to _select_relevance_score_fn, so it should be, but would be nice to verify it we could (e.g., a subtle change in scoring may confuse folks).

relevancy score function already implemented, these should be no-ops and completely backwards compatible

Ya, agreed. Cool, thanks for running some additional tests.
Going to merge this now, esp given the demand to get it in.
We can take up follow-up work on this in newer PRs.
Good work!

@raymond-yuan Looking into this for the supabase retriever. Awesome work! Any reason why you left supabase out of this? Was it implemented correctly?

@ShantanuNair ah sorry, no I missed this because it already had a custom implementation of similarity_search_with_relevance_score. Upon closer inspection, I believe this is correctly implemented for the default match_document being used which seems to be in line with the documentation. This code already converts the default distance fn (cosine distance) into a similarity score (1 - cos(x))

Looks like relevant issue with Pinecone Cosine Similarity, which is converted back to distance - filtering out relevant documents when using score_threshold -- #8207

I found the score_threshold also filters out the most relevant doc for the FAISS vectorstore. I think there are still bugs with this refactoring changes. `
First, the FAISS.similarity_search_with_score /similarity_search_with_score_by_vector  actually returns the score which already represents the similarity score instead of distance... (I validated with the embed_function with the same sentences and the dot product is 1 which matches the score from the function) but somehow the docstring says the lower score means more similarity?
Second, the _max_inner_product_relevance_score_fn  should just return the score itself for FAISS (at least) if the higher score from FAISS.similarity_search_with_score /similarity_search_with_score_by_vector represents the higher similarity... let me know if I miss anything.
I guess a fix is to implement the _max_inner_product_relevance_score_fn in the FAISS vectorstore file, like lambda x: x.


Merge 5-31 updates to live

Docs Build status updates of commit 67dad80:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Publisher.OLEFormat.md
:white_check_mark:Succeeded
View

api/Publisher.ObjectVerbs.md
:white_check_mark:Succeeded
View

api/Publisher.Options.md
:white_check_mark:Succeeded
View

api/Publisher.Page.md
:white_check_mark:Succeeded
View

api/Publisher.PageBackground.md
:white_check_mark:Succeeded
View

api/Publisher.PageSetup.md
:white_check_mark:Succeeded
View

api/Publisher.PageSize.md
:white_check_mark:Succeeded
View

api/Publisher.PageSizes.md
:white_check_mark:Succeeded
View

api/Publisher.Pages.md
:white_check_mark:Succeeded
View

api/Publisher.ParagraphFormat.md
:white_check_mark:Succeeded
View

api/Publisher.PhoneticGuide.md
:white_check_mark:Succeeded
View

api/Publisher.PictureFormat.md
:white_check_mark:Succeeded
View

api/Publisher.Plate.InUse.md
:white_check_mark:Succeeded
View

api/Publisher.Plate.md
:white_check_mark:Succeeded
View

api/Publisher.Plates.md
:white_check_mark:Succeeded
View

api/Publisher.PrintableRect.md
:white_check_mark:Succeeded
View

api/Publisher.Printer.md
:white_check_mark:Succeeded
View

api/Publisher.ReaderSpread.md
:white_check_mark:Succeeded
View

api/Publisher.Row.md
:white_check_mark:Succeeded
View

api/Publisher.Rows.md
:white_check_mark:Succeeded
View

api/Publisher.RulerGuide.md
:white_check_mark:Succeeded
View

api/Publisher.RulerGuides.md
:white_check_mark:Succeeded
View

api/Publisher.ScratchArea.md
:white_check_mark:Succeeded
View

api/Publisher.Section.md
:white_check_mark:Succeeded
View

api/Publisher.Sections.Add.md
:white_check_mark:Succeeded
View

api/Publisher.Sections.md
:white_check_mark:Succeeded
View

api/Publisher.Selection.md
:white_check_mark:Succeeded
View

api/Publisher.ShadowFormat.md
:white_check_mark:Succeeded
View

api/Publisher.Shape.md
:white_check_mark:Succeeded
View

api/Publisher.ShapeNode.md
:white_check_mark:Succeeded
View

api/Publisher.ShapeNodes.md
:white_check_mark:Succeeded
View

api/Publisher.ShapeRange.md
:white_check_mark:Succeeded
View

api/Publisher.Shapes.md
:white_check_mark:Succeeded
View

api/Publisher.Stories.md
:white_check_mark:Succeeded
View

api/Publisher.Story.md
:white_check_mark:Succeeded
View

api/Publisher.TabStop.md
:white_check_mark:Succeeded
View

api/Publisher.TabStops.md
:white_check_mark:Succeeded
View

api/Publisher.Table.md
:white_check_mark:Succeeded
View

api/Publisher.Tag.md
:white_check_mark:Succeeded
View

api/Publisher.Tags.md
:white_check_mark:Succeeded
View

api/Publisher.TextEffectFormat.md
:white_check_mark:Succeeded
View

api/Publisher.TextFrame.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add a solution for the Employee Storage exercise

Closes #6

Submitted to Base Goerli: https://goerli.basescan.org/address/0xc513F6B727c7FAD9b2D97C8483AB159fd07C7192#code


Add ability to specify operator version to install

Just for context: the reason the falcon and k10 operators can still be installed with automatic install plan approval is that they are both uncertified and used for negative test cases. falcon-operator is from the community-operators catalog and k10-kasten-operator hasn't been certified for any OCP version since 4.5, so whatever version of these we install will be fine.


Favicon overlap problem

When I select "Display favicon instead of pin" option, favicon and title overlaps, as seen in the picture.

I probably won't have time to look into it for a month or so, sorry for the trouble


Add Sentry Spans for Bash execution.

Related to #270

Additionally, we want to gain insights of the span nomad.execute.exec that is describing the Nomad execute request. To do so, we produce a special string in the bash command, scan the output for this string and convert it into a Sentry span.

It's really nice to have those insights and they look beautiful with CodeOcean run and score requests.

Those insights show the delay caused by the Container management (Nomad + Docker). Here, we might be ale to save up to 300ms (2 * (50ms + 100ms)) per run.

One that I still have is the difference between nomad.execute.pipe and nomad.execute.bash. The one is for piping the command to the tty and the other for invoking it (should we add some details about the difference to the description)?

:+1:

Why is the "end" command only shown once?

Because we have no additional information how long the "End"-span should last. Therefore, we just show the span until the "End"-message is received by Poseidon.

And would it make sense to add some more details (like [parts of] the actual command as a span description)?

:+1:

ToDo:

Clean up functions (dynamic nesting of bash commands)
(Static) renaming of span descriptions

I was quickly searching for the shell escape. For one, a package such as shellescape might be useful (and I would prefer some library if it exists). Alternatively, some manual escaping of all characters (except explicit safe ones) could be the way to go (here are some good replies for that topic).

For one, a package such as shellescape might be useful (and I would prefer some library if it exists).

This library is not useful in our case as it only surrounds the string with single-quotes [1]
I could not find another library for our use case.

Alternatively, some manual escaping could be the way to go (here are some good replies for that topic).

The proposed set of safe characters does not work in our case.  For example, the characters " " (sleep\ 2\ &&\ echo\ $(date)), "&" (echo $date && bash -vxc "sleep 2 \&\& echo \$(date)"), "(", ")" (echo $date && bash -vxc "sleep 2 && echo \$\(date\)"), "=" (echo $date && bash -vxc "ec\=1 && echo \$(date)") are not included in the safe set of characters and therefore would be escaped - (leading to errors when executing).
Instead, the manual of bash is very helpful. In the chapter "Quoting", it is explained, that the only characters with a special meaning inside a double-quoted text are $, backtick, " and \. We can use the go library to escape the backslash and double-quote correctly (fmt.Sprintf("%q", "abc")) and therefore just handle the two remaining characters.
All in all, although we had some complication during these changes, I think the code quality and simplicity has increased by introducing the dynamic nesting of the bash commands.

Thanks for having a look here and considering the two misleading suggestions I made; your solution looks fine and having a look at the manual always seems to be a good idea 👍
As discussed, let's shortly investigate:

Context: Add full command
Shorten span description to "make run", ... (but keep the full "user" command)
Span description: Include "unset ${!NOMAD_@}" (and escape)


Add zn-CN translation for all nodes

Description
I have added zn-CN translation for all nodes: Common, Function, Storage; network, parsers, sequence as well.

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Please can we confirm this PR is ready to merge?

Please can we confirm this PR is ready to merge?

I've reviewed the changes and it is ready to merge.

Hi @knolleary
This pull request is ready to merge now.

Thank you everyone!


Suggestion: Custom properties - Blacklist certain Keys

If a user defines a custom property he is free to decide how to name it and what key should be used. The key is used as property name in the data model of the asset, which is created based on this custom property. As there aren't any limitations to what keys the user can define, he could use keys, which are already used as built-in properties. This can result in strange side effects.
For example if the user defines a custom property, which has the key type, he could by accident override the built-in property type, which is used to identify the asset type. There should be a blacklist, which contains keys, which must not be used by users and are marked as built-in.

The implementation is complete. Here is the list of blacklisted keys:
'c8y_Notes',
'c8y_IsAssetProperty',
'c8y_JsonSchema',
'c8y_IsAssetType',
'c8y_IsAsset',
'c8y_IsDeviceGroup',
'c8y_Global',
'c8y_Position',
'additionParents',
'assetParents',
'allowedAssetTypes',
'id',
'icon',
'category',
'name',
'isNoneChildAssetsAllowed',
'properties',
'childAdditions',
'childAssets',
'childDevices',
'creationTime',
'description',
'deviceParents',
'label',
'lastUpdated',
'owner',
'self',
'type',
'customProperties',
'pagination',
'currentPage',
'nextPage',
'pageSize',
'key',
'assetModels',
'depth',
'assetProperties',
'tags'


Specify package version

Change:

Remove dev minimum stability
kornrunner/keccak ~1.0
phpseclib/phpseclib ~2.0

Fix:

undefined pow function in phpseclib 2.0

Codecov Report

Merging #58 into master will decrease coverage by 0.26%.
The diff coverage is 54.54%.

@@             Coverage Diff              @@
##             master      #58      +/-   ##
============================================
- Coverage     88.51%   88.24%   -0.27%
- Complexity      659      661       +2
============================================
  Files            47       47
  Lines          1445     1455      +10
============================================
+ Hits           1279     1284       +5
- Misses          166      171       +5

Impacted Files
Coverage Δ
Complexity Δ

src/Utils.php
94.81% <54.54%> (-2.45%)
86 <0> (+2)

Continue to review full report at Codecov.

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


Fix NoMethodError when (un)locking single packages in apt and zypper

Signed-off-by: Jonathan Hartman<EMAIL_ADDRESS>Description
These resources don't coerce package_name to an array so would raise an
exception when trying to call name.all? when package_name, and thus name,
was a string.
Before applying the provider changes:
✗ bundle exec rspec spec/unit/provider/package/apt_spec.rb
...
Failures:

  1) Chef::Provider::Package::Apt after loading the current resource when locking a package should not lock if the package is already locked
     Failure/Error: name.all? { |n| locked_packages.include? n }

     NoMethodError:
       undefined method `all?' for "irssi":String
     # ./lib/chef/provider/package/apt.rb:78:in `package_locked'
     # ./lib/chef/provider/package.rb:226:in `action_lock'
     # ./spec/unit/provider/package/apt_spec.rb:425:in `block (4 levels) in <top (required)>'

  2) Chef::Provider::Package::Apt after loading the current resource when unlocking a package should not unlock if the package is already unlocked
     Failure/Error: name.all? { |n| locked_packages.include? n }

     NoMethodError:
       undefined method `all?' for "irssi":String
     # ./lib/chef/provider/package/apt.rb:78:in `package_locked'
     # ./lib/chef/provider/package.rb:245:in `action_unlock'
     # ./spec/unit/provider/package/apt_spec.rb:446:in `block (4 levels) in <top (required)>'

✗ bundle exec rspec spec/unit/provider/package/zypper_spec.rb
...
Failures:

  1) Chef::Provider::Package::Zypper action_lock should lock if the package is not already locked
     Failure/Error: name.all? { |n| locked_packages.include? n }

     NoMethodError:
       undefined method `all?' for "cups":String
     # ./lib/chef/provider/package/zypper.rb:83:in `package_locked'
     # ./lib/chef/provider/package.rb:226:in `action_lock'
     # ./spec/unit/provider/package/zypper_spec.rb:284:in `block (3 levels) in <top (required)>'

  2) Chef::Provider::Package::Zypper action_lock should not lock if the package is already locked
     Failure/Error: name.all? { |n| locked_packages.include? n }

     NoMethodError:
       undefined method `all?' for "cups":String
     # ./lib/chef/provider/package/zypper.rb:83:in `package_locked'
     # ./lib/chef/provider/package.rb:226:in `action_lock'
     # ./spec/unit/provider/package/zypper_spec.rb:300:in `block (3 levels) in <top (required)>'

  3) Chef::Provider::Package::Zypper action_unlock should unlock if the package is not already unlocked
     Failure/Error: name.all? { |n| locked_packages.include? n }

     NoMethodError:
       undefined method `all?' for "cups":String
     # ./lib/chef/provider/package/zypper.rb:83:in `package_locked'
     # ./lib/chef/provider/package.rb:245:in `action_unlock'
     # ./spec/unit/provider/package/zypper_spec.rb:341:in `block (3 levels) in <top (required)>'

  4) Chef::Provider::Package::Zypper action_unlock should not unlock if the package is already unlocked
     Failure/Error: name.all? { |n| locked_packages.include? n }

     NoMethodError:
       undefined method `all?' for "cups":String
     # ./lib/chef/provider/package/zypper.rb:83:in `package_locked'
     # ./lib/chef/provider/package.rb:245:in `action_unlock'
     # ./spec/unit/provider/package/zypper_spec.rb:356:in `block (3 levels) in <top (required)>'

Check List

[x] New functionality includes tests
[x] All tests pass
~RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)~
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Hmm... The Travis jobs that failed are failing in other recent PRs as well, so don't seem to be related to this change, but holler if I'm missing something there.

This doesn't actually fix the behavior though since now you're comparing arrays and there's a third possibility which is that the array is a mix of locked and unlocked packages.
To fix this in the yum package provider i change the superclass code to look for package_all_locked? and packaged_all_unlocked? in the subclass:
https://github.com/chef/chef/blob/f6a4bf2fa6b47344acfc65b4f71d7952287a3840/lib/chef/provider/package.rb#L223-L227
And the implementation of that in the yum subclass is here:
https://github.com/chef/chef/blob/f6a4bf2fa6b47344acfc65b4f71d7952287a3840/lib/chef/provider/package/yum.rb#L181-L187

errrr  derp, i have really not had enough caffeine this morning..   yes, that's the right way to do things...

yep, i think i need a do-over on this morning...


There are no applicable practices to display. Check the assessment's target maturity level.

Version: <IP_ADDRESS>
After selecting "Maturity Models" --> "Ransomware Readiness Assessment" then the next button and then again the next button I get the following message: "There are no applicable practices to display. Check the assessment's target maturity level".
No "maturity levels" options are shown.

The method of creating new assessments changed to a card gallery layout a few releases ago.  Closing this issue.


[Issue-196] optimize speed get bitcoin balance

🚀 Deployed on https://pr-197--papaya-melba-f5c160.netlify.app


Where is M117.0.5938.157 - 53rd Release in the deb repo?

Add all the stuffz here
Hello, I'm using the deb repo to install Thorium and I see that there's a new release but version M117.0.5938.157 - 53rd Release is not available. I see that there is an auto updater included but I am still on Version 116.0.5845.169 when I click on the about in the browser. I just really want to stay up to date and having the deb repo updated every time a new release is out would be nice. Thank you very much for making this amazing project!!

@greengeckowizard It's not? It should be

@greengeckowizard You're right, it's not. @midzer Can you upload new .deb to repo?

Ooops, I forgot to upload before my vacation^^
Should be up now. Thanks for filing @greengeckowizard :)

Ooops, I forgot to upload before my vacation^^
Should be up now. Thanks for filing @greengeckowizard :)

Thanks much!


Rescript core

Imports @rescript/core and refactors the relevant code.

Tested and functional on Kraken by building with "@ninefx/bs-xml": "NineFX/bs-xml#rescript_core",


Wrong image appearing in the pandas profiling report

Describe the bug
A wrong image appears inside the pandas profiling report. It shows another image that is generated outside of pandas profiling library using matplot lib in the web application.

To Reproduce

Version information:

Additional context

Hi @jmehul99 could you provide more information on how to reproduce this?


add wrapper support for Starlette

The instrumentation itself is fairly trivial, we wrap Starlette.build_middleware_stack to insert our middleware.
Testing was a bit more difficult. Ultimately, I resorted to only test the wrapping, but without use of the wrapper script.
Related issues
Closes #1602

Don't forget CHANGELOG


Ignore any file-related IO errors and continue on

If any files or directories can't be read, skip them and continue onto the next in the list. May have to use catchIO and the like.

👍


Unexpected rounding of float value

I would have expected:
pp 1.005.round 2, mode: :ties_away # => 1.0

to round to 1.01, like it does in Ruby and PHP, as both of those are using their version of TIES_AWAY.
Ref: https://discord.com/channels/591460182777790474/591597160492171264/945597596947349525
\cc @HertzDevil

The code in question:
https://github.com/ruby/ruby/blob/37d5890e4941cedf6918821b29bb4a7e3a092e62/numeric.c#L109-L125
https://github.com/ruby/ruby/blob/37d5890e4941cedf6918821b29bb4a7e3a092e62/numeric.c#L2538
def round_half_up(x, s)
  xs = x * s

  f = xs.round
  return f if s == 1.0

  if x > 0
    f += 1 if (f.to_f + 0.5) / s <= x
    x = f
  else
    f -= 1 if (f.to_f - 0.5) / s >= x
    x = f
  end

  x
end

struct Number
  def round(digits : Number, base = 10, *, mode : RoundingMode = :ties_even)
    case mode
    in .ties_away?
      round_half_up(self, base ** digits) / (base ** digits)
    # ...
    end
  end
end

1.005.round(2, mode: :ties_away) # => 1.01

@HertzDevil Are you saying you don't see @Blacksmoke16 's 1.0, but actually see 1.01?  Trying locally I also see 1.0


Convert composite Types

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #[issue number]
Description
Clear and concise code change description.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Builds | Releases | Other)
Screenshots (if applicable)
Checklist

[ ] I have read and acknowledged the Code of conduct
[ ] I have read How to Contribute
[ ] I have read the Developer's guide
[ ] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[ ] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[ ] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

Codecov Report

:exclamation: No coverage uploaded for pull request base (gsoc/hapi-proto-converter@3494393). Click here to learn what that means.
The diff coverage is n/a.

@@                     Coverage Diff                      @@
##             gsoc/hapi-proto-converter     #680   +/-   ##
============================================================
  Coverage                             ?   54.15%
  Complexity                           ?      177
============================================================
  Files                                ?       44
  Lines                                ?      818
  Branches                             ?      117
============================================================
  Hits                                 ?      443
  Misses                               ?      308
  Partials                             ?       67

Continue to review full report at Codecov.

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

@epicadk , could you please update the PR description with the issue number it fixes?

@epicadk , could you please update the PR description with the issue number it fixes?

actually it depends on #604 so will create an issue soon after that pr is merged.

@epicadk can we close this? thanks


feat (event) Adds Event Constructor, Parameter, Field and Method

Constructor takes String as parameter and assigns it to title

getTitle() returns title as String

Thanks for your help Jean...this PR looks a lot better than #14.

Now that LGTM !
We'll still need more fields, if anyone's interested.

Sorry...got distracted and never ran the test before commit 37eca84. This commit, 690239b, corrects the code and now it actually works and all the tests pass.

LGTM

LGTM, looks like we all love immutability ?

LGTM


SvelteKit SSR issues

I can't seem to get SvelteKit SSR authentication to work. I've used the code provided in the documentation but the exportToCookie method does not work, even with secure and httpOnly set to false.
// src/hooks.server.js
import PocketBase from 'pocketbase';
import { PUBLIC_POCKETBASE } from '$env/static/public';

/** @type {import('@sveltejs/kit').Handle} */
export async function handle({ event, resolve }) {
    event.locals.pb = new PocketBase(PUBLIC_POCKETBASE);

    // load the store data from the request cookie string
    event.locals.pb.authStore.loadFromCookie(event.request.headers.get('cookie') || '');

    try {
        // get an up-to-date auth store state by verifying and refreshing the loaded auth model (if any)
        event.locals.pb.authStore.isValid && await event.locals.pb.collection('users').authRefresh();
    } catch (_) {
        // clear the auth store on failed refresh
        event.locals.pb.authStore.clear();
    }

    const response = await resolve(event);

    // send back the default 'pb_auth' cookie to the client with the latest store state
    response.headers.append('set-cookie', event.locals.pb.authStore.exportToCookie());

    return response;
}

What do you mean by "exportToCookie method does not work"?
If you ware testing on localhost, note that some browsers (eg. Safari) may require disabling the default secure cookie flag, aka.:
response.headers.append('set-cookie', event.locals.pb.authStore.exportToCookie({
    secure: false, // enable on prod
}));

If i console log the result from the .exportToCookie() method the expire date is set to 01-01-1970, i think chrome automatically removes this cookie. What would be the expected outcome of the .exportToCookie() method?

Could you provide a more complete code sample?
How and where do you authenticate in the first place?

Ofcourse. I have tried some other things but still can't seem to get it to work. Here is the code i'm using:
hooks.server.ts:
// src/hooks.server.js
import PocketBase from 'pocketbase';

export const handle: Handle = async ({ event, resolve }) => {
	event.locals.pb = new PocketBase('http://localhost:8090');

	// load the store data from the request cookie string
	event.locals.pb.authStore.loadFromCookie(event.request.headers.get('cookie') || '');

	try {
		// get an up-to-date auth store state by verifying and refreshing the loaded auth model (if any)
		event.locals.pb.authStore.isValid && (await event.locals.pb.collection('users').authRefresh());
	} catch (_) {
		// clear the auth store on failed refresh
		event.locals.pb.authStore.clear();
	}

	const response = await resolve(event);

	// send back the default 'pb_auth' cookie to the client with the latest store state
	response.headers.append('set-cookie', event.locals.pb.authStore.exportToCookie());

	return response;
};

routes/login/+page.svelte:
<script lang="ts">
</script>

<div class="w-full flex min-h-full justify-center items-center">
	<form method="POST" class="flex flex-col">
		<input class="rounded border-2 border-primary" type="text" name="username" placeholder="Gebruikersnaam" />
		<input class="rounded border-2 border-primary" type="password" name="password" placeholder="Wachtwoord" />
		<button class="bg-primary text-white" type="submit">Log in</button>
	</form>
</div>

routes/login/+page.server.ts:
import type { Actions } from '@sveltejs/kit';

export const actions: Actions = {
	default: async ({ locals, request }) => {
		const formData = await request.formData();
		const username = formData.get('username') as string;
		const password = formData.get('password') as string;

		locals.pb
			.collection('users')
			.authWithPassword(username, password)
			.then((res) => {
				console.log(res);
			})
			.catch((err) => {
				console.log(err);
			});
	},
};

You are missing a return in the action, aka. try with:
import type { Actions } from '@sveltejs/kit';

export const actions: Actions = {
	default: async ({ locals, request }) => {
		const formData = await request.formData();
		const username = formData.get('username') as string;
		const password = formData.get('password') as string;

		return locals.pb
			.collection('users')
			.authWithPassword(username, password)
			.then((res) => {
				console.log(res);
			})
			.catch((err) => {
				console.log(err);
			});
	},
};

If that doesn't resolve your issue, feel free to let me know I'll try to setup a local Svelte project to test it.


Add a --steady command line arg

Add a --steady command line arg so a config file isn't required for a simple setting.
Rename msi_perkeyrgb.py to main.py so the file name isn't the same as the package which caused Python to get confused.

Hi, thanks for that !
That looks pretty good, I'm going to merge it. But before that, would you mind splitting it in two commits ? (The new feature in one commit and the renaming in the other).


Refactor dev dependencies

This PR cuts down on dev dependencies:

np: use it through npx instead
eslint: replaced with biome

Some parts of the code were refactored for linting to pass, but functionality has not changed at all.

👍


Mutate elements of array in parallel example using Rayon library

The example in the commit demonstrates how difference on each element of array is calculated in parallel (instead of sequential computation) using Rayon library in Rust.

Thanks Alisha! I'll take a look at the code asap. The CI test failed due to some unrelated issue (which we should fix as well, but it's not necessary for you to do it here)

Thanks a bunch @alisha17! I left some notes. Please see what you think and make revisions.

This is great @alisha17. Thanks! I rebased your branch and pushed it to master.


Upgrade gradle version and dependencies

Upgrade gradle to a newer version, upgrade spring boot version, reformat code, fix some codestyle warnings.
Had to remove pool.start() call from ThreadPoolConfig class because the newer version of Jetty would throw an error on the start.
Cannot upgrade to gradle 6.1 because the js plugin won't work with it.

Wow! That's awesome! I'm terribly sorry @singleplayer88 I've just seen your pr... I'll review it on Mon
Thank you so much! Unfortunatelly my company dicided to continue this project as non-open source. But I must get a grip, dust off this repo and make it "great again" =)

Thank you! I think we have to consider removing that checkstyle, from my perspective it's more a headache than something valuable.

Thanks @QIvan !


Use RAF for animations

This makes things buttery smooth by greatly improving performance. It also fixes, once and for all, strange positioning issues that spur from display: none being toggled on the same repaint that animations take place.

Testing a component that uses requestAnimationFrame is not a good time o_O.
Will have to dig into why the 2.12 tests are failing.

Merging this and fixing tests later. There's no reason this would break things in prior versions of ember, we're just hitting limitations of the test framework.


Support Faker provider positional args

Adds ability to pass positional arguments to the underlying Fake-Factory provider.
This is necessary for some providers which use variable length positional arguments (ex, pylist for it's element types).
Addresses issue #216

Test failure on python 3 is a failure in the current version of Fake-Factory.  https://github.com/joke2k/faker/issues/267

Hi @derekjamescurtis ,
The problem with this approach is that users will get weird failures if they forget to set the locale arg:
A user could write bookmarked_pages = factory.Faker('pylist', 6, False, int) and get an unusual error message.
I'd rather go with the following: bookmarked_pages = factory.Faker('pylist', _args=(6, False, int)).
Would that suit your needs?

Yeah, I think that's a better suggestion.  I'll update the pull request.
On Jul 25, 2015 6:56 PM, "Raphaël Barrois"<EMAIL_ADDRESS>wrote:

Hi @derekjamescurtis https://github.com/derekjamescurtis ,
The problem with this approach is that users will get weird failures if
they forget to set the locale arg:
A user could write bookmarked_pages = factory.Faker('pylist', 6, False,
int) and get an unusual error message.
I'd rather go with the following: bookmarked_pages =
factory.Faker('pylist', _args=(6, False, int)).
Would that suit your needs?
—
Reply to this email directly or view it on GitHub
https://github.com/rbarrois/factory_boy/pull/217#issuecomment-124836860.

@derekjamescurtis Mind rebasing and updating the PR as mentioned above?

Hi @derekjamescurtis ,
May I ask what is the status of this PR?


Concourse API client assumes a "target" url that is authenticated

It looks like the target URL that is used in the concourse API client assumes an authenticated URL (meaning basic auth credentials embedded into the URL). What this means is that a user will have to provide resource configuration like the following to make this resource work:
resources:
- name: some-pipeline
  type: concourse-pipeline
  source:
    target<EMAIL_ADDRESS>    username: username
    password: password

We found this when we tried to use the resource against a concourse that is protected by basic auth. We were unable to do a put to create a new pipeline. We saw output like the following:
Logging to /tmp/concourse-pipeline-resource-out.log789647078
2016/05/10 00:06:57 Unexpected response status code: 401, expected: 200

And when we hijacked the put container, we saw this in the logs:
/tmp/build/put # more /tmp/concourse-pipeline-resource-out.log027887453
Concourse Pipeline Resource version: 0.2.2
Received input: {Source:{Target:https://my-concourse.com Username:username Password:***REDACTED-PASSWORD***} Params:{Pipelines:[{Name:pipeline ConfigFile:pipeline/pipeline.yml VarsFiles:[]}] PipelinesFile:}}
Performing login
Starting fly command: [-t concourse-pipeline-resource-target login -c https://my-concourse.com -u username -p ***REDACTED-PASSWORD***]
Waiting for fly command: [-t concourse-pipeline-resource-target login -c https://my-concourse.com -u username -p ***REDACTED-PASSWORD***]
Login successful
Input pipelines: [{Name:pipeline ConfigFile:pipeline/pipeline.yml VarsFiles:[]}]
Setting pipelines
Starting fly command: [-t concourse-pipeline-resource-target set-pipeline -n -p pipeline -c /tmp/build/put/pipeline/pipeline.yml]
Waiting for fly command: [-t concourse-pipeline-resource-target set-pipeline -n -p pipeline -c /tmp/build/put/pipeline/pipeline.yml]
Setting pipelines complete
Getting pipelines
Exiting with error: Unexpected response status code: 401, expected: 200

We think the issue is that the URL given to this GET request (https://github.com/robdimsdale/concourse-pipeline-resource/blob/14e4a76d1470c336047d9c912a37f6d406b0416c/concourse/api/pipelines.go#L31) does not include the basic auth credentials if you don't include them with the target like is shown above.

Firstly, sorry for the delay. I didn't realize I wasn't "watching" the repo - is that default github behavior when you create a repo? I assumed I was watching.
Anyway, to your issue, that's odd, as I see this work against my test concourse instance which requires authentication. My concourse is set up to allow both basic auth and github, so I haven't tried it with only basic auth? Other than I can't really see why this doesn't work.
The PR at #6 was merged, hopefully that fixes this issue


Container stuck freezing and blocks rest of system

Required information
Distribution:
Linux container-west1-plbw 4.10.0-32-generic #36~16.04.1-Ubuntu SMP Wed Aug 9 09:19:02 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

lxc info (condensed):
core.https_address: '[::]:8443'
core.trust_password: true
images.auto_update_interval: "0"
driver: lxc
driver_version: 2.0.8
kernel: Linux
kernel_architecture: x86_64
kernel_version: 4.10.0-32-generic
server: lxd
server_pid: 3123
server_version: "2.16"
storage: zfs
storage_version: <IP_ADDRESS>-2

Issue description
I've run into an issue during which freezing a container (while in the process of stopping it) gets stuck, which leads to a high load average (I usually notice the first symptoms around 400, but it'll climb to 1000 if the server stays in use).
The process that seems to get stuck the most is an Apache 2 web server.
I've noted the timeout here, and am running LXD version 2.16.
Strangely, I can still interact with the remote API (e.g. checking container states), but cannot interact with LXD locally.
Here is the state of a stuck container:
{:architecture=>"x86_64",
 :config=>
  {:"volatile.base_image"=>
    "c8d365c36c1a9f411e65b027e7e72a3e308fcbeb34e2c6a15d94343e08650cf1",
   :"volatile.eth0.hwaddr"=>"00:16:3e:4c:a7:c4",
   :"volatile.eth0.name"=>"eth0",
   :"volatile.idmap.base"=>"0",
   :"volatile.idmap.next"=>
    "[{\"Isuid\":true,\"Isgid\":false,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536},{\"Isuid\":false,\"Isgid\":true,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536}]",
   :"volatile.last_state.idmap"=>
    "[{\"Isuid\":true,\"Isgid\":false,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536},{\"Isuid\":false,\"Isgid\":true,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536}]",
   :"volatile.last_state.power"=>"RUNNING"},
 :devices=>
  {:root=>{:path=>"/", :pool=>"default", :size=>"128MB", :type=>"disk"}},
 :ephemeral=>false,
 :profiles=>["default", "small"],
 :stateful=>false,
 :description=>"",
 :created_at=>2017-08-13 13:31:09 UTC,
 :expanded_config=>
  {:"environment.http_proxy"=>"",
   :"limits.cpu"=>"2-7",
   :"limits.cpu.allowance"=>"25%",
   :"limits.cpu.priority"=>"5",
   :"limits.disk.priority"=>"5",
   :"limits.memory"=>"256MB",
   :"limits.memory.enforce"=>"soft",
   :"limits.processes"=>"100",
   :"user.network_mode"=>"",
   :"volatile.base_image"=>
    "c8d365c36c1a9f411e65b027e7e72a3e308fcbeb34e2c6a15d94343e08650cf1",
   :"volatile.eth0.hwaddr"=>"00:16:3e:4c:a7:c4",
   :"volatile.eth0.name"=>"eth0",
   :"volatile.idmap.base"=>"0",
   :"volatile.idmap.next"=>
    "[{\"Isuid\":true,\"Isgid\":false,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536},{\"Isuid\":false,\"Isgid\":true,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536}]",
   :"volatile.last_state.idmap"=>
    "[{\"Isuid\":true,\"Isgid\":false,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536},{\"Isuid\":false,\"Isgid\":true,\"Hostid\":100000,\"Nsid\":0,\"Maprange\":65536}]",
   :"volatile.last_state.power"=>"RUNNING"},
 :expanded_devices=>
  {:eth0=>{:nictype=>"bridged", :parent=>"lxdbr0", :type=>"nic"},
   :root=>{:path=>"/", :pool=>"default", :size=>"128MB", :type=>"disk"}},
 :name=>"production-ca1ac5e25d74777d-1502631069738",
 :status=>"Freezing",
 :status_code=>109,
 :last_used_at=>2017-08-13 13:31:19 UTC}

Steps to reproduce
This is different from how I'm using it now (which involves never freezing containers), but previously these steps seemed to exacerbate the issue:

Start a lot of containers (~100 on a system with 8 CPUs and 30GB memory)
Freeze them all
Try to use the system to start other containers

Information to attach
I lost my SSH shell to my last stuck server and it's not possible to regain it after you lose it, but will follow up with that information when another server dies.

Next time this happens, can you get:

dmesg output
/var/log/lxd/lxd.log

This would help us see if it's something that LXD did which is causing problems or if you're running into kernel scheduling bugs around the freezing logic (which is unfortunately not that uncommon).

Thank you for the fast response.
Not sure if relevant, but all schedulers (e.g. /sys/block/sda/queue/scheduler) appear to be set to none.
I'll followup with those logs once another server crashes, which seems to be occurring every few days.

Those are I/O schedulers, the scheduler that's relevant here is the process scheduler which on most systems is going to be CFS (completely fair scheduler) and is what the various CPU related cgroups (cpu, cpuset, cpuacct and freezer) all integrate with.

I had several servers trigger this issue overnight, Gist here with debugging info.
It looks like there's a BUG log line at 24585.875478 in the dmesg output, after which the output changes significantly.
I'll (try to) keep this SSH connection to this server open, in case there's other output that would be useful.

Ouch, yeah, that's quite clearly a kernel bug. Looks like something went wrong while a cgroup was destroyed, causing any further interaction with the cgroup subsystem to deadlock, slowly causing all your processes to hang...
I don't believe there's much we can do for this at the LXD level. I'd strongly recommend filing a kernel bug for this. For the Ubuntu kernel, you can run "ubuntu-bug linux" or file a report directly at https://launchpad.net/ubuntu/+source/linux/+filebug

Okay, thank you for the info. I'll file that with them.
In the meantime, any recommendations for ways to alleviate the issue? Maybe trying a different / older kernel or version of LXD?

We've not changed that behavior in LXD in quite a while, so I'm not sure that changing LXD versions would help. Changing kernel versions might.

To follow up, I have been running Linux 4.4.0-38 for 24 hours with no sign of an issue (currently running 7 servers, usually 1 would have triggered it by now).
Will report kernel bug to Linux and follow up here if anything changes or when I hear from them. Thanks for the help!


Edit command variant - Append

Allows user to add on groups/tags instead of overwriting the existing groups/tags

Completed. PR #59


Use Kanji for Japanese documents.

Summary
Documentation related.
Use Kanji for Japanese documents. Change "もじれつ" to "文字列".
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Thanks for fixing this!


Change markdown-style formatting in README.rst to rst

This consolidates all the README markup into rst, so it should read a bit better on GitHub.
generated via pandoc
:stroopwafel:

Thank you!


feat: add map function for landing page

merging to get a build, will address any comments if needed


Upgrade Spring Security to 5.3.3

There's a vulnerability in 5.3.2, and also some changes that switch to using lite @Configuration so that will help this project.

Fixed in 2ece27c


GROOVY-8965, GROOVY-10180, GROOVY-10668: STC: instanceof and ||

As discussed under 8965 and 10668, check LUB type first for common method (fixes testMultipleInstanceOf4).  Duck-typing (testMultipleInstanceOf1 and testMultipleInstanceOf5) is incomplete; currently reported as ambiguous method error.
https://issues.apache.org/jira/browse/GROOVY-8965
https://issues.apache.org/jira/browse/GROOVY-10180
https://issues.apache.org/jira/browse/GROOVY-10668

Updated to support duck-typing scenario.  I'm still trying to figure how to support 6790, 8142 and (x instanceof Map || x instanceof List) ? x.size() : 1 at the same time.
https://issues.apache.org/jira/browse/GROOVY-6790
https://issues.apache.org/jira/browse/GROOVY-8142


Android actionsheet does not contain message

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[] iOS
[x] Android
[] UWP
[] NET Standard 1.1

Version of ACR UserDialogs Library
6.3.8
Expected Behaviour
Calling UserDialogs.Instance.ActionSheet(config) with an ActionSheetConfig that contains a message, results in an action sheet that has a blank message.
Actual Behavior
The actionsheet should have a message displayed. The source code seems to ignore the message property of the ActionSheetConfig, once it goes into ActionSheetBuilder.Build , as SetMessage is not called on the Builder.
Code of Conduct

No Library Update Requests
No Timeline Requests
Please read through the documentation and FAQs
Please be respectful of my time. I put a lot of effort into my libraries.  I try to support the issue requests within a day.  I'm not psychic, so please take the time to fill out the above form as complete as possible

Hi
Title is shown and message is not shown in android. This had been fixed?
Thanks for help.

No this is still an issue, @aritchie has closed this issue and given reference to its duplicate #250 -  https://github.com/aritchie/userdialogs/issues/250 .


[SPARK-15053][BUILD] Fix Java Lint errors on Hive-Thriftserver module

What changes were proposed in this pull request?
This issue fixes or hides 181 Java linter errors introduced by SPARK-14987 which copied hive service code from Hive. We had better clean up these errors before releasing Spark 2.0.

Fix UnusedImports (15 lines), RedundantModifier (14 lines), SeparatorWrap (9 lines), MethodParamPad (6 lines), FileTabCharacter (5 lines), ArrayTypeStyle (3 lines), ModifierOrder (3 lines), RedundantImport (1 line), CommentsIndentation (1 line), UpperEll (1 line), FallThrough (1 line), OneStatementPerLine (1 line), NewlineAtEndOfFile (1 line) errors.
Ignore LineLength errors under hive/service/* (118 lines).
Ignore MethodName error in PasswdAuthenticationProvider.java (1 line).
Ignore NoFinalizer error in ThreadWithGarbageCleanup.java (1 line).

How was this patch tested?
After passing Jenkins building, run dev/lint-java manually.
$ dev/lint-java
Checkstyle checks passed.

Test build #57503 has started for PR 12831 at commit 678cb59.

Test build #57503 has finished for PR 12831 at commit 678cb59.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57503/
Test FAILed.

PySpark error is irrelevant to this PR. I'll rebase this to retrigger Jenkins.

Test build #57505 has started for PR 12831 at commit 3f1c6fa.

Test build #57505 has finished for PR 12831 at commit 3f1c6fa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57505/
Test PASSed.

Merged build finished. Test PASSed.

I agree with all the changes, though some look like they wouldn't have actually been lint errors? I assume we can't fix the PasswdAuthenticationProvider file because it's cloned from Hive.

Thank you for review, @hvanhovell and @srowen .
For PasswdAuthenticationProvider, I didn't change that due to the same reason mentioned by @srowen .

Merged to master/2.0

Thank you, @srowen .


pywal not finding i3 pid to reload i3

I am running raspbian lite with minimal installs. I have to manually reload i3 because the line in reload.py @ 47 returns false from util.get_pid('i3') Here is a sample of my htop filtered by i3 string

Where's the i3 process? I can see a bunch of i3 related utilities but the window manager itself seems to be missing.

Yeah, i3 must have been in a really weird state. Rebooting caused the whole WM to break. Reformatted and started over. You can close this out. Sorry for the inconvenience.


isDeviceConnected return BleErrorCode.DeviceDisconnected instead of false🐛

Prerequisites

[X] I checked the documentation and FAQ without finding a solution
[X] I checked to make sure that this issue has not already been filed

Expected Behavior
if (bleManager.isDeviceConnected(device.id)) should return false when is not connected.
Current Behavior
if (bleManager.isDeviceConnected(device.id)) throw the error BleErrorCode.DeviceDisconnected when not connected.
Library version
3.2.1
Device
iPhone SE (iOS 17.5.1)
Environment info
System:
  OS: macOS 13.6.6
  CPU: (12) arm64 Apple M2 Pro
  Memory: 76.55 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.20.3
    path: ~/.nvm/versions/node/v18.20.3/bin/node
  Yarn:
    version: 1.22.21
    path: /usr/local/bin/yarn
  npm:
    version: 10.7.0
    path: ~/.nvm/versions/node/v18.20.3/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /usr/local/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: 2023.3 AI-233.14808.21.2331.11842104
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.10
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.6
    wanted: 0.73.6
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Steps to reproduce

Scan for devices
Memorize the device
Call your connectToDevice function
Add a isDeviceConnected(device.id) before connecting your device
Verify if you have an error instead of false

Formatted code sample or link to a repository
const connectToDevice = async (device) => {

try {
      if (bleManager.isDeviceConnected(device.id)) {
        await bleManager.cancelDeviceConnection(device.id);
      }
    } catch (error) {

      if(error.errorCode === BleErrorCode.DeviceDisconnected) {
      vConsoleLog(
        fileName,
        `Error while disconnecting device: ${error.message}`
      );
    }

   // Connect your device if no error
}

Relevant log output
🟢 Creating JS object for module 'NativeModulesProxy'
🟢 Creating JS object for module 'ExponentConstants'
🟢 Creating JS object for module 'ExponentFileSystem'
🟢 Creating JS object for module 'ExpoDevice'
Unbalanced calls start/end for tag 19
Running "main" with {"rootTag":11,"initialProps":{}}
🟢 Creating JS object for module 'ExpoKeepAwake'
API MISUSE: <CBCentralManager: 0x300c68a00> has no restore identifier but the delegate implements the centralManager:willRestoreState: method. Restoring will not be supported
--> BLE permissions granted
--> State Manager : Default
[GESTURE HANDLER] Initialize gesture handler for view <RCTRootContentView: 0x10690ca60; frame = (0 0; 375 667); gestureRecognizers = <NSArray: 0x3033ad980>; layer = <CALayer: 0x303dc6ea0>> reactTag: 11; frame = {{0, 0}, {375, 667}}; layer = <CALayer: 0x303dc6ea0>
--> Start scan devices
PoweredOn
--> Device scanned:  MYDEVICENAME MYDEVICEID
--> new device scanned:  MYDEVICENAME
quic_conn_process_inbound [C<IP_ADDRESS>:2] [-cccc99356eb95e52] unable to parse packet
--> Initialize app 0,2,29
--> Device scanned:  MYDEVICENAME MYDEVICEID
--> new device scanned:  MYDEVICENAME
--> Stop scan devices
--> device id MYDEVICEID
--> State Manager : Connecting
--> Error while disconnecting device BleError: Operation was cancelled

Additional information
Bonjour!
I use the PLX library since January. Great library, really like it, thanks for the good work.
I know that is a weird verification to do before connecting, but I had to add this verification because after calling this :
deviceConnection = await bleManager.connectToDevice(device.id, {
          autoConnect: false,
        });

I get a deviceConnection.name and deviceConnection.localName to null after calling bleManager.connectToDevice (I will write another Issues for this one) and I try to understand if my bleManager is disconnecting correctly my device when I call bleManager.onDeviceDisconnected.
Thanks for you help.

Hi @philbourg
Thank you for kind words 😄
I can't reproduce the behaviour with the provided code.
  useEffect(() => {
    bleManager.startDeviceScan(null, null, async (error, device) => {
      if (device && device.name === 'someDeviceName') {
        bleManager.stopDeviceScan()
        connectToDevice(device)
      }
    })
  }, [])

  const connectToDevice = async (device: Device) => {
    try {
      if (bleManager.isDeviceConnected(device.id)) {
        await bleManager.cancelDeviceConnection(device.id)
      }
    } catch (error) {
      if (error.errorCode === BleErrorCode.DeviceDisconnected) {
        console.log(`Error while disconnecting device: ${error.message}`)
      }
    }

    device
      .connect()
      .then(device => {})
      .catch(error => {
        console.error('error', error)
      })
  }

I have noticed one issue in provided code. The isDeviceConnected() method is async so you need to await and this may be the couse of the error as Promise is truthful so the cancelDeviceConnection is always being run. The final code should look like:
      if (await bleManager.isDeviceConnected(device.id)) {
        await bleManager.cancelDeviceConnection(device.id)
      }

I have also question on what is the case for handling already connected devices by disconnecting them. Why you just don't ignore already connected devices? Do you have multiple devices you want to connect to at once? Handling this case properly may remove the need for checking connection state.
I hope it helps. If issue persist please provide more context of how you use the library:

how you initialize BleManager
how you scan for devices
how you connect to device

Hi @intent-kacper-cyranowski ! :)
Adding the await resolve it!
Thanks a lot!

I have also question on what is the case for handling already connected devices by disconnecting them. Why you just don't ignore already connected devices? Do you have multiple devices you want to connect to at once? Handling this case properly may remove the need for checking connection state.

I want to be sure that I have only one and one device at the time connected for now.  It's more a QA function for my app, I want to detect if I had problem with cancelDeviceConnection.


Lint request: Do not use optional new/const

Dart 2 introduces support for optional new/const. Some teams prefer to consistently only have required new/const keywords in their code, and leave out all the rest. To enforce this, a lint is requested.

cc @a14n any interest in looking at this?

@eernstg, @lrhn : quick question for the following example
class A {
  const A(o);
}
main() {
  A([]);
}

Is A([]) equivalent to const A(const []) or new A([]) ?
BTW it would be great to have examples on https://github.com/dart-lang/sdk/blob/master/docs/language/informal/implicit-creation.md.

@floitschG thanks for your explanations.

Fyi. Updated the comment again.

To elaborate on what Florian said, the logic behind const insertion is very similar to type inference.
In Dart 2 type inference, if the context requires a specific type, that is the type we try to make the expression have. If the context doesn't care, then we build the type of the expression from the types of the sub-expressions.
For const/new insertion, it's similar.
If the context requires a const value, we will try to make it one by inserting const in front of everything that needs it, and if the result still isn't a const value, it's an error anyway.
If the context doesn't require a const value, then we evaluate the sub-expressions normally, and if all the arguments to a const constructor are const, we make that a const construction too.
The distinction matters for list and map literals. For classes with const constructors, if the constructor arguments are constant, then the object is immutable anyway, but a list literal with constant elements is not naturally immutable, and we won't make it so by accident. So const C([]) inserts const in front of [] because it would be an error otherwise, but C([]) does not because you might want a mutable list stored in C. For C(const []), you will get an immutable C instance, the only difference between newandconstis the identity of the immutable instance. In that case, we just make itconstfor you, because it's convenient - no more writing theconstinconst Duration(seconds: 1)`.
(We have one exception: optional parameter default values. We reserve the option to allow non-const values in that position in the future, and for that reason, we won't consider it a "const requiring context" for const inference, even if it is currently still an error if the value isn't constant).

Interesting! One simple, syntactic rule of thumb might be useful: When we want opportunistically const values ("the more the better, but it's not required to be all const") we could put a const on every composite literal (list or map, currently) where it is possible and correct according to the intended behavior of the program; when we cannot tolerate anything else than a const value we would put a const at top level, if needed (in that situation we'd often be in a const context, anyway, like e in const x = e;). So the rule would be "best effort: bottom-up on composite literals; must be const: top-down".

Thanks for all your explanations.
Could you check the following list to validate if my expectations are good.
class A {
  const A([o]);
}
main() {
  A();                       // new A()
  A([]);                     // new A([])
  final v1 = A();            // new A()
  const v2 = A();            // const A()
  final v3 = A([]);          // new A([])
  const v4 = A([]);          // const A(const [])
  final v5 = A(const []);    // const A(const [])
  final v6 = const A([]);    // const A(const [])
  final v7 = A([const A()]); // new A([const A()])
  const v8 = A([A()]);       // const A(const [const A()])
  final v9 = const A([A()]); // const A(const [const A()])
}

Re-reading #32553 I'm not sure about my expectations.

Almost:
class A {
  const A([o]);
}
main() {
  A();                       // const A(), because it's possible.
  A([]);                     // new A([]), because `[]` is only const when forced by context.
  final v1 = A();            // const A(), because it's possible.
  const v2 = A();            // const A(), both possible and forced.
  final v3 = A([]);          // new A([]), because of non-forced `[]`.
  const v4 = A([]);          // const A(const []), forced by `const v4` context.
  final v5 = A(const []);    // const A(const []), because it's possible to add the outer `const`.
  final v6 = const A([]);    // const A(const []), the inner `const` is forced by the outer one.
  final v7 = A([const A()]); // new A([const A()]), the list is not forced `const`, stays mutable.
  const v8 = A([A()]);       // const A(const [const A()]), forced by being initializer for `v8`.
  final v9 = const A([A()]); // const A(const [const A()]), forced by outermost `const`.
}

Thanks!
Last one :
final v = A(A());   // const A(const A())  ?

We are not shipping the automatic const insertion part of this feature.  All constructor calls with no explicit new or const are made new unless they appear in a const context.  Updated examples below:
class A {
  const A([o]);
}
main() {
  A();                       // new A(), because not in const context.
  A([]);                     // new A([]), because not in const context.
  final v1 = A();            // new A(), because not in const context.
  const v2 = A();            // const A(), because in const context.
  final v3 = A([]);          // new A([]), because not in const context..
  const v4 = A([]);          // const A(const []), forced by being initalizer for `v4`.
  final v5 = A(const []);    // new A(const []), because not in const context..
  final v6 = const A([]);    // const A(const []), the inner `const` is forced by the outer one.
  final v7 = A([const A()]); // new A([const A()]), the list is not forced `const`, stays mutable.
  const v8 = A([A()]);       // const A(const [const A()]), forced by being initializer for `v8`.
  final v9 = const A([A()]); // const A(const [const A()]), forced by outermost `const`.
  final v = A(A());   // new A(new A()), because not in const context.
}

We are not shipping the automatic const insertion part of this feature.

Is it just a first step or is it totally removed?
What's the reason of this decision?

When it went to implementation, it became clear that there were serious concerns around how to make this work as specified with modular or platform independent compilation.  As specified, you have to do constant evaluation before you can insert new and const, which means that in some important scenarios you can't insert new and const until essentially link time.   We did some quick exploration of alternatives  (e.g. https://dart-review.googlesource.com/c/sdk/+/50080) but none of the options that were short term feasible were entirely satisfactory.  Given our extremely short time horizon for landing a change, we felt that changing the spec was a large implementation risk, and had a large risk of us specifying a bad feature.
I think it unlikely that we will ship the feature as originally specified in the future, but we may consider looking for an alternative way to get similar functionality at some point.

Hi @a14n are you still looking at this?

@mit-mit the lint is ready but not merged for now (see dart-lang/linter#936). Basically I tried to remove optional const in the flutter codebase and faced an issue with flutter analysis. I'm waiting for someone to figure out what's the underlying problem and fix it to go further (cc @pq @bwilkerson @devoncarew ).

(link to flutter issue updated to https://github.com/flutter/flutter/issues/17429)


[main] Update community files (#228)

Update community files
Signed-off-by: Knative Automation<EMAIL_ADDRESS>

OCF Webhook is merging this PR


Transfer Ownership to Official ?

Thank you for this great nomnoml integration with vsCode markdown! Yet, it hasn't been updated in 3 years and there are a few rendering bugs that the old version has.
I would hope maybe @skanaar could directly take ownership or fork this into his own organisation to make sure it stays in sync with the latest version.

That's a good proposal and I'm open to transfer the extension to another maintainer.
I'm no longer using the extension myself and don't plan to maintain it further.
Especially, if VS Code or a dependency would require a fix in this code base.
But I took your comment as motivation to publish a new version with the fixes from the PR plus some updated dependencies.
I hope it didn't break anything or added new rendering issues.
Had only limited time and sample files to test with.

There are some manual steps required to publish a new version to the VS Code Marketplace. The steps here are pretty much all that is needed.
I'm sure it can be automated. There are probably some marketplace extensions, that could be used as reference.

Hi guys.
I'm on vacation right now, but in a while I'll have a look and think about it.

Thanks, have a great vacation !


About the hyperparameter

There is a hyperparameter    during evaluation.
But the paper doesn't seem to mention how to set it up, and what was chosen in the experiment.
Is it possible to try to eliminate it with

where  is the position where anomaly ends

The hyperparameter is used to select what the maximum tolerable delay should be, and that depends on the application itself. The problem with the equation that you mention is that for cases where the anomaly lasts for several thousand instances (for eg. 100k), setting the delay as 100k seconds is not really reasonable since that is equivalent to ~ 30 hours. So in our experiments, we set it as ~10% of e_i from your equation. For example, in SWAT the longest anomaly sequence is ~50k instances, so we set delta max as 5k.


Add support for paths with spaces

Fixes #4

I have made my own fork: https://github.com/vanyasem/git-change-date
And I have uploaded a scoped npm package: https://www.npmjs.com/package/@vanyasem/git-change-date
You can install it by running npm install -g @vanyasem/git-change-date


python: execute pythonImportsCheckPhase in subshell

Motivation for this change
Execute in subshell so that the cwd returns back
to the original directory. This is important as
it enables pythonImportsCheck to work with checkPhase

pretty much you had to do doCheck = false; or else the checkPhase (or setuptoolsCheckPhase) would run and burn because setup.py (or test paths) were no longer available
good example is: https://github.com/NixOS/nixpkgs/pull/80444#discussion_r381058197
Tested that:

fails when an import isn't available
succeeds when it is
is able to be ran with a checkPhase

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build requests

@GrahamcOfBorg build python3Packages.requests
@GrahamcOfBorg build pythonPackages.requests
@GrahamcOfBorg build python38Packages.requests


This a test

Overview
This is a test ticket.

@myersjustinc Howdy. I created this ticket yesterday for demo purposes in a class, but it seems I may not have the ability to delete it! Do you know if my owner/admin perms have been deprecated? Or perhaps the delete feature is so well hidden that I'm not finding it? Either way, feel free to delete this ticket and sorry for any confusion!

None of us had permissions to do it, but I just looked up how to add those permissions, and now I have that option. (And so should you!) I'll delete now.


⚠ Bofrost.at has degraded performance

In d84ee3d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 948 ms

Resolved: Bofrost.at performance has improved in 19a1d0b.


There are no rules able to parse the input

Hi there,
I get the following exception when using Seabird to work with any of my .cnv files:
"CNVError: There are no rules able to parse the input. File: [filename]"
Here are a couple of the .cnv files that I'm using:
cnv_examples.zip

(I should add that I'm using Seabird 0.11.2)

Thanks for making those changes, however I'm still getting the same exception with 0.11.3. It occurs both in shell and in Python script. In the latter this is the code I'm using:
from seabird.cnv import fCNV
profile = fCNV(cnvfilepath)
Do you know why I could still be getting this error?

That works fine now, thanks!

Thank you for the CNV sample file. Please check if you're using the latest version (0.11.4).

Hi again,
Many of my other files still throw the same exception. They may differ slightly from the example files I gave above. Here is a sample of these files:
cnvexamples2.zip
I can attach more if needed.

Hello,
I am experiencing a similar error, "CNVError: There are no rules able to parse the input." when calling "from seabird.cnv import fCNV
file = fCNV(entry)"
Only a single file is returning this error, so it is not a terrible inconvenience, but it may be for others in the future. I have attached this file. FYI, I downloaded the seabird package in December 2019.
FK180624_STN002_CTD001_30_JUN_2018_PROC_ALL_Bin_1m_Dwn.zip
Thank you for the code!

Hello,
I have exception "CNVError: There are no rules able to parse the input. File: [filename]" too.
The header of my .cnv files is xml.
Can PySeabird parse these files?
my_data.zip

Thank you for this awesome package! I am getting this same error on the attached files (and many other probably similar ones)
d200504_001.zip

Hello,
I am just starting to look at some CTD data and am so glad I came across this package! I unfortunately also have the exception 'CNVError: There are no rules to parse the input.' Do you have a sense of what in the data file is creating this flag? Thank you so much for taking a look!
example_file.zip


Race condition in initialisation

The following two tests will often cause a hang on cargo test when started at the same time
    #[test]
    fn lock_part1() {
        h5call!(*H5P_ROOT);
    }
    #[test]
    fn lock_part2() {
        *H5P_ROOT;
    }

Yep :/ I've noticed the tests hanging as well - I guess that must be it.


Inline Critical CSS in html files

I have a working script that inlines critical above the fold CSS https://github.com/serverless/site/blob/master/scripts/build/inlineCSS.js (<-- runs on a post phenomic build npm script)
One caveat: it adds about a minute to the build time.
Perf > build time?  ¯_(ツ)_/¯
Related: https://github.com/MoOx/phenomic/issues/459
Reference: https://www.smashingmagazine.com/2015/08/understanding-critical-css/#what-is-critical-css

This is amazing, Can we add it as an option?
I agree with user performance is more important than build time :)

I think with a smarter approach we could handle that on the static rendering directly. Closing this one as it's a dup of #459


Rename live visualization

[x] LiveVisualization => StepVisualizer2D
[x] PathSampling.live_visualization => PathSampling.live_visualizer
[x] PathSampling.visualization_frequency => PathSampling.status_update_frequency
[x] Update notebooks used in tests
[x] Update remaining notebooks (requires #588)

See also #527.
Note that this now includes #588, which should be merged first.
Fixes this everywhere except in AD TPS examples; that will be added to #571.

Passes tests, ready for review. Includes #597, #588, so please handle those first (will try to update after #588 is merged, to make review here easier.)

@dwhswenson  #588 is merged you can update

Although I think there is not much to review...

@jhprinz: updated. You're correct: There is not much to review. :-) But doing the update definitely makes it easier to look over!
Tests still running as I type this, but barring 5000-step error, it should pass (other stochastic errors should be solved!)

Good. Merging...


Customize icon

My bar renders some icon incorrectly. Is there anyway to customize them? Thanks in advance

can you provide a pic about spaceline in your vim. maybe you can try to update your nerdfonts.

I used nerdfont already but some icons are still missing  :(

your pic is broken bro ..try to update your fonts .down it from nerdfonts website. because the fonts also update for fixed bug add feature and so on ..

Actually, the problem is my terminal. I use urxvt and it cannot render some icons.
Great job btw :)


Add Feed Model object Example in README.md

Instade of looking for the complete feed model with google and find it in a issue of the plugin (thx for that),
I think it could be better for every body to have it directly in the README file and on the home page on the plugin.
Regards,
Eric Fressange

This is way old, obviously, but this is better sourced from Instagrams API docs - it'll be out of date constantly otherwise.


avoid ambiguous exception handling in creating road map

The general exception handling will catch all errors and report "Unable to find map in map_source", which might hind some problems other than file not found. For example, when I switch to a new machine and did not setup SUMO properly, which caused ImportError of SumoRoadNetwork, however this error will also be caught by the outer 'catch'.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


add ufetch-nixos package

Motivation for this change
ufetch was not available in nixpkgs.
Things done
added ufetch-nixos to nixpkgs

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] 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)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-in-distress/3604/30

CLosing cause review feedback is not addressed since months and I don't think this package really benefits from being in Nix. Just download the script and execute it.


Custom texture format

In order to reduce VRAM usage and load times, I want to create a custom texture file format using BCN compression and storing mipmap levels.

If we decide to persue this, DDS already support BC and mips.


⚠ Bofrost.at has degraded performance

In 5aeca44, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1670 ms

Resolved: Bofrost.at performance has improved in 58cf476 after 7 minutes.


InjvmInvoker throw Exception that message is "No Provider ..." with consumer refer DubboService use "*"

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 3.1.6
Operating System version: mac 12.6
Java version: 1.8

Steps to reproduce this issue

two providers export injvm DubboService : com.test.TestService:1.0.0 and com.test.TestService
consumer refer injvm DubboService: com.test.TestService:*
consumer call DubboService com.test.TestService.test()

Pls. provide [GitHub address] to reproduce this issue.
Expected Behavior
consumer call provider success.
Actual Behavior
throw Exception and the message is "No Provider".

If there is an exception, please attach the exception trace:
Just put your stack trace here!

It support group, but InjvmInvoker.isAvailable() is not support the wildcard ("*"), because the code logic of InjvmInvoker.isAvailable() is inconsistent with InjvmInvoker.doInvoke().
the code of InjvmInvoker.isAvailable() :
public boolean isAvailable() {         InjvmExporter<?> exporter = (InjvmExporter)this.exporterMap.get(this.key);         return exporter == null ? false : super.isAvailable();     }
the code of InjvmInvoker.doInvoke():
public Result doInvoke(Invocation invocation) throws Throwable {         Exporter<?> exporter = InjvmProtocol.getExporter(this.exporterMap, this.getUrl());         if (exporter == null) {             throw new RpcException("Service [" + this.key + "] not found.");         } else {            ...         } }
PR https://github.com/apache/dubbo/pull/13716 is fix it. please

It support group, but InjvmInvoker.isAvailable() is not support the wildcard ("*"), because the code logic of InjvmInvoker.isAvailable() is inconsistent with InjvmInvoker.doInvoke(). the code of InjvmInvoker.isAvailable() :
public boolean isAvailable() {
        InjvmExporter<?> exporter = (InjvmExporter)this.exporterMap.get(this.key);
        return exporter == null ? false : super.isAvailable();
    }

the code of InjvmInvoker.doInvoke():
public Result doInvoke(Invocation invocation) throws Throwable {
        Exporter<?> exporter = InjvmProtocol.getExporter(this.exporterMap, this.getUrl());
        if (exporter == null) {
            throw new RpcException("Service [" + this.key + "] not found.");
        } else {
           ...
        }
}

PR #13716 is fix it. please

I think we should create a directory for it like those provider registered in registry.

e.g. org.apache.dubbo.registry.integration.RegistryDirectory#toMergeInvokerList

It support group, but InjvmInvoker.isAvailable() is not support the wildcard ("*"), because the code logic of InjvmInvoker.isAvailable() is inconsistent with InjvmInvoker.doInvoke(). the code of InjvmInvoker.isAvailable() :
public boolean isAvailable() {
        InjvmExporter<?> exporter = (InjvmExporter)this.exporterMap.get(this.key);
        return exporter == null ? false : super.isAvailable();
    }

the code of InjvmInvoker.doInvoke():
public Result doInvoke(Invocation invocation) throws Throwable {
        Exporter<?> exporter = InjvmProtocol.getExporter(this.exporterMap, this.getUrl());
        if (exporter == null) {
            throw new RpcException("Service [" + this.key + "] not found.");
        } else {
           ...
        }
}

PR #13716 is fix it. please

I think we should create a directory for it like those provider registered in registry.

goods idea, we should support consumer refer use wildcard either on "group" or "version" or both in the best.


fix sharding

Pull Request Test Coverage Report for Build 200

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build 198:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls


Handle nested table of contents

How should we display this on the TOC list? Indentation/font styles?

I would make it easy:

Main category in bold
Sub-categories indentated two spaces right and normal font


Backward convolution code performance improvement

The new backward code structure is similar to forward code. Furthermore, I have included possible optimizations specific to backward code.

Codecov Report

Merging #154 into master will increase coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #154      +/-   ##
==========================================
+ Coverage   12.43%   12.49%   +0.06%
==========================================
  Files         168      168
  Lines       23771    23699      -72
  Branches     4369     4318      -51
==========================================
+ Hits         2955     2961       +6
+ Misses      20500    20427      -73
+ Partials      316      311       -5

Impacted Files
Coverage Δ

src/generator_convolution_backward_avx512.c
0% <ø> (ø)
:arrow_up:

src/libxsmm_dnn.c
0% <0%> (ø)
:arrow_up:

src/libxsmm_dnn_handle.c
0% <0%> (ø)
:arrow_up:

src/generator_convolution.c
0% <0%> (ø)
:arrow_up:

.../libxsmm_dnn_convolve_st_bwd_custom_custom_1.tpl.c
0% <0%> (ø)
:arrow_up:

...late/libxsmm_dnn_convolve_st_bwd_nhwc_custom.tpl.c
0% <0%> (ø)
:arrow_up:

...mplate/libxsmm_dnn_convolve_st_bwd_nhwc_rsck.tpl.c
0% <0%> (ø)
:arrow_up:

src/libxsmm_main.c
47.36% <0%> (+0.13%)
:arrow_up:

src/generator_gemm_avx_microkernel.c
93.67% <0%> (-1.27%)
:arrow_down:

src/generator_gemm_common.c
47.4% <0%> (-0.2%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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

Good work!  I've taken all changes, but not the "old" files. We can keep some amount of dead code, but not at file level. Of course, one can do this kind "versioning" (or "todolist") locally (outside of the repository).


change name of reqs file to requirements

snyk looks for *requirements.txt pattern, so this file was not scanned for vulnerabilities

@HLasse is it on purpose you haven't named it similarly to the other req files?

Nope, unintended

hmm, i think maybe the dev container uses the same pattern, so because the files have different versions of mlflow, there are conflicting dependencies when trying to set up the container

I can clean up the synthcity-reqs to only be the diff to the main dependencies - that should make our lives easier

that would be 🥚cellent

On further thought, there was a reason ;) the synthcity-reqs are incompatible with some functionality in the rest of the library (hence the separate file and not just added to the core requirements), which means that there will be conflicts and some tests will fail if they are installed together.
I have manually checked the synthcity-reqs for vulnerabilities before installing on Overtaci. Perhaps the synthcity-reqs file should be moved into the scz-bp project folder, so no-one uses it "on accident" - its only purpose is for testing models that are augmented with synthetic data.

alrigthy 👍


Error with ajv/dist/compile/codegen Module After Installing @nuxtjs/eslint-module

Hello, I've recently set up a project using Nuxt 3 following these steps:

Installed Nuxt 3 using bunx nuxi@latest init .

Added ESLint and the related Nuxt module with bun add -D eslint @nuxtjs/eslint-module.

Configured the Nuxt config file as follows:

// https://nuxt.com/docs/api/configuration/nuxt-config
export default defineNuxtConfig({
  devtools: { enabled: true },
  modules: ['@nuxtjs/eslint-module'],
});

However, when I run bun dev, I encounter the following error:
ERROR  Cannot start nuxt:  Cannot find module 'ajv/dist/compile/codegen'

This issue is a new occurrence; I didn't face it in previous setups. Temporarily, installing the ajv package directly seems to solve the problem, but I'm uncertain if this is the optimal solution. Does anyone have insights into the root cause of this issue?
Furthermore, if this issue is widespread, it might be prudent for the @nuxtjs/eslint-module to include the ajv package as a dependency. Your opinions on this matter would be greatly appreciated.
For additional context, a similar issue has been documented in an existing GitHub thread: https://github.com/nuxt-modules/eslint/issues/88
Thank you for your assistance.

If I use the npm dedupe --force command, it resolves the issue. However, I'm still uncertain about why this problem occurs. Additionally, when I use Yarn and Bun.js, the problem persists.

Just install ajv@latest as dev dependencies can fix it.

I believe this is more like a package hoisting issue rather than a bug of this project.


Evaluating binomial & non-binomial metrics simultaneously with Experiment class & how to categorise metrics by type

I can't seem to find much info on how to exactly evaluate both binomial and non-binomial metrics at the same time within a dataframe that gets input within the Experiment class.
It seems that, even with the method column specified, that multiple_difference treats it as a binomial metric. You would obviously need different inputs to perform a t-test, so how would I  add and specify these columns? If so, how would I indicate these in Experiment?
Likewise, there's a really good paper you posted on your risk-aware product decision framework using multiple metrics - and I've seen mention of success metrics within the repository/q&a - however there's no documentation I could find that indicates how to specify success, deterioration, and guardrail metrics. I did see a method on sample ratio which is a form of a quality metric, so I suspect this has been considered but it's difficult to see how to implement the entire approach.
Do let me know if you need any further information. Thanks for your time!

Here's an example (that could be put in an example notebook or as a test case for the Experiment class):
columns = [
    "group_name",
    "num_user",
    "sum",
    "sum_squares",
    "method",
    "metric",
    "preferred_direction",
    "non_inferiority_margin",
]
data = [
    ["Control", 6267728, 3240932,<PHONE_NUMBER>2, "z-test", "m1", "increase", 0.15],
    ["Test", 6260737, 3239706,<PHONE_NUMBER>2, "z-test", "m1", "increase", 0.15],
    ["Test", 6260737, 38600871,<PHONE_NUMBER>9, "z-test", "m2", None, None],
    ["Control", 6225728, 35963863,<PHONE_NUMBER>9, "z-test", "m2", None, None],
    ["Test", 62607, 26738, None, "chi-squared", "m3", "increase", None],
    ["Control", 62677, 16345, None, "chi-squared", "m3", "increase", None],
]
df = pd.DataFrame(columns=columns, data=data)
test = spotify_confidence.Experiment(
    data_frame=df,
    numerator_column="sum",
    numerator_sum_squares_column="sum_squares",
    denominator_column="num_user",
    categorical_group_columns="metric",
    interval_size=0.99,
    correction_method="bonferroni",
    metric_column="metric",
    treatment_column="group_name",
    method_column="method",
)

diff = test.multiple_difference(
    level="Control",
    level_as_reference=True,
    groupby="metric",
    non_inferiority_margins=True,
)

display(diff)
test.multiple_difference_plot(
    level="Control",
    level_as_reference=True,
    groupby="metric",
    non_inferiority_margins=True,
    use_adjusted_intervals=True,
    absolute=False,
).show('html')

Guardrail metrics are specified by providing a NIM, non-inferiority margin as for the m1 metric. In the example we fail to reject the hypothesis "m1 Test is no worse than m1 Control" since some of the metric's CI is below the NIM.
Success metrics can be one sided or two sided as specified by preferred_direction. I guess the difference between success and deterioration metrics are that success metrics are what you're actually trying to improve, while deterioration metrics are metrics that you would hope stay neutral, often related to performance/latency/numbers crashes etc.

The way you’d implement the deterioration metrics is to take the same data as you’d use for your main results (like Pelle provided), but flip the preferred direction, not use any NIMs, and use a separate alpha.
For example, suppose you have one success metric that should improve and one guardrail metric (with a NIM) for which an increase is a good change. You would then use that as in Pelle’s example and set preferred direction to increase and set the NIM for the guardrail metric. Next, you would make a similar call but set preferred direction to decrease and not set a NIM. You will then test whether any of the two metrics has significantly moved in the wrong direction. In the paper, we also use a different alpha for this test - so it’s using a separate budget. You can also include a sample ratio mismatch test here via the chi-squared test.


feat(build): implement rollup builds

Terser minify bundles
Provide es6 modules bundle and commonjs bundle
Provide sourcemaps for each
Output ts declaration files to lib

Before submitting a pull request, please make sure the following is done:

Fork the repository and create your branch from master.
Run npm install in the repository root.
If you've fixed a bug or added code that should be tested, add tests!
Ensure the test suite passes (npm test).

Note. If you find a typo in the documentation, make sure to modify the corresponding source (docs are generated).

It looks like definitelytyped-header-parser is a dependency of dtslint but is not published to npm. Instead it is referenced as a git dependency. I don't have any issues with the tests locally, but I'm guessing travis is not resolving that dep correctly. @gcanti It looks like you've already forked dtslint and are looking into it. Let me know if you want further help.

@baetheus thanks for this PR but I'm not acquainted with rollup so I'm not sure what's the goal / benefit here.
Looks like this build will output only two .js files

lib/index.js
lib/index.es.js

However all fp-ts modules are supposed to be (also) consumed individually
import { Option } from 'fp-ts/lib/Option'

The goals I had with this PR were:

Provide a tree shakeable es6 bundle in addition to the current commonjs target, since es6 module exports are the spec and showing up in node and browsers.
Provide sourcemaps so debugging production apps is easier to reason about.
Ensure that the commonjs build is backwards compatible with previous releases.

I think this is covered by https://github.com/gcanti/fp-ts/pull/814, closing for now


Updated plugin for Matomo 5.x.

Updated the plugins.json, the validation code works with Matomo 5.x.
https://github.com/simivar/matomo-password-policy-enforcer/issues/15
Thank you for your plugin.

Thank you! 🙏


Update Survarium

Game Title
Survarium
Status of the Anti-Cheat
🚫 Denied
Tinkering Steps
Survarium is long gone now, closed over a year ago and not likely to return, I guess it should be removed.
Proof of Mention
https://survarium.com/

Should just be removed then

Yes, it is now removed, thanks!


Redirects not working in Kirby 4?

After upgrading to Kirby 4, any URL I have saved as a redirect is returning a 404.
Any advice on what I might be doing wrong?

Kirby 4.0.1
Redirects 1.12.0

https://github.com/bnomei/kirby3-redirects/releases/tag/v4.0.0

Thank you! I don't think I was using any placeholders, but I can't fully remember now! 😄


Bug in console-link-job-sa

The gitops-console-link-job in the 200-openshift-gitops BOM is creating a service account by the name of console-link-job when the process is looking for console-link-job-sa. This causes the matrix button to not populate the list of applications.

Updated openshift-gitops and devtools to latest . Have to test to verify the above issue related to service account name

@faheemshai when you execute the script, please verify the SA name from OCP console.  As discussed I have updated the gitops and devtools to latest, so assuming that the SA name should be matching with the expected value.