fix: use lspconfig pyright exposed commands so user config is retained

This seems to have fixed the issue. I looked into lspconfig's code and found a command that is exposed to set python path. So I did that and call LspRestart to enforce the change.

just discover one issue. After this change, a python file is required to be opened before VenvSelect can be executed. Let me know what is your take on this.

I wonder if there is a way to get the changes to be active without LspRestart? I was so happy using the callback function (before_init) since it doesn't require a full restart of all lsp servers.
But I will try your changes this weekend, maybe it's not such a big deal if it works better for all use cases.
It's not so good that at python file needs to be opened, and the reason is because the plugin will search for venvs based on what file you are editing. So let's say you are using editing a Readme and you want to select a python interpreter. I guess then it wouldn't work and the user would have to find and open a python file first?
But I will have to try all of this myself, see how it feels. :)

after a few attempts, I managed to get it do so that we don't need to open a python file to set python path. Also, now I am not using LspRestart. This snippet is code for code from lspconfig pyright command.
Probably good to use some testing before merging.

That sounds extreamly promising! Im so happy you are taking the time and effort to look through this, specially this week when im very busy at a new job, learning new stuff. :) But it will pass and once the learning period is over, I will again have more time to work with you and others on this plugin. I will try your PR on saturday, looking forward to it!

Might be a busy weekend for you do to some testing on this project. 😂 I have added caching of the venv with reference to the CWD.

oh btw, congrats on your new job! For me, I am still waiting for an answer from my job applications. So, I have some time for this. 😊

Perfect! I actually look forward to the weekend now to test the new features. It sounds like it will add a bunch of new functionality without making anything worse, which is perfect. :)

Added a new commit for support to update venv for debugger. But because this require a few more plugins installed, I created a config setting and is off by default. Users can turn it on after adding the dependencies and put 'dap_enabled' to true

I had to add this line to the code also to avoid an error about config being nil in telescope.lua, line 46:
local config = require("venv-selector.config")
If it works for you without that line, im not sure whats going on. :)

oh btw, congrats on your new job! For me, I am still waiting for an answer from my job applications. So, I have some time for this. blush

I loved being off and just working on plugins and neovim stuff, it was so much fun. Enjoy every minute of it. :)

I had to add this line to the code also to avoid an error about config being nil in telescope.lua, line 46:
local config = require("venv-selector.config")
If it works for you without that line, im not sure whats going on. :)

Hmm.. interesting. I did not do any additional configuration to telescope, and I am just using whatever configuration already provideed by Astronvim.

Hmm.. interesting. I did not do any additional configuration to telescope, and I am just using whatever configuration already provideed by Astronvim.

Interesting... Anyway we better add that line to make sure it works on vanilla configs. :)

As expected, there are some merge conflicts now after I merged in https://github.com/linux-cultist/venv-selector.nvim/pull/13. To support PyLSP the plugin now have changed a bit how it activates the venv - either with pyright or pylsp depending on what the user has running.
I was also thinking it would possibly be better to have the "use_cache" functionality as an option to setup() instead of as a separate VenvSelectCached command to run.
The cached functionality could also be a bit confusing in combination with the other options, specially the auto_refresh option that will keep previous search results in the telescope picker when set to false. If the auto_refresh is true (so it reloads the telegram picker every time you use VenvSelect), then I guess "use_cache" could be useful to avoid searching a certain path again. But then you have to cache the path being searched, and that is not the cwd in most cases - its the directory that is found when doing a parent search (current file open in buffer and then go up parent amount of steps). So that is the directory you would have to cache to make it work in the same way as the rest of the code.
I think it could be quite useful, specially if its something like the home directory with tons of files to search, but we have to be a bit careful to make sure it works with the other options. If a user presses Ctrl-r to refresh, the plugin should also not use the cache for example.

I am not sure how we should resolve the conflicts. As there are 3 features that are involved here. Pylsp, DAP and cache. Let me know which one you intend to keep for now so i know better.

as for the cache. I don't know if we are inline here. My idea is not to use telescope at all after I have set my python interpreter. That means, for the first time, I set my interpreter, every other time when I use the same CWD into nvim, I will the most need a keymap to activate the interpreter I choose the first time. For any case if I need to change the interpreter, I can still use the VenvSelect command to pick another interpreter. Which at that point it will again be updated into the cache. And the workflow continues. This feature to me is very important as it allows me to automate it to everytime when nvim is open, I call VenvSelectCached using autocmd so I got my interpreter without any more keystrokes. Unless the first time in my new project or when I have updated my python verions, etc.

Contrary to which I read from you, I think you are more interested in caching the telescope results. Which doesn't really make much sense to be as I want to avoid using the venv search as much as possible. So caching the search results is not so useful. Even if it takes a longer time to search my whole system, I don't really care because I don't have to do it everytime I launch my project.

Lets keep all those things, now that I understand better what you intended. There should be pylsp support merged on the main branch, so its just a matter of integrating your code with what exists on main somehow. :)

Now I understand better why you made it a separate command. Thank you for explaining, and yes, i was thinking of caching the telescope results, and you were thinking of caching the python interpreter based on cwd. So its different features that dont interfere with eachother. Then lets keep it as a separate command, like you did.

Okay, I have resolved the conflicts to keep all features. I think it is an eventful PR. So many exciting new features!

Minor thing I notifed when trying your new command, it prints the venv twice. :)

Feels really nice we got all this into main, thank you!


Retrieve latitude and longitude from methods calls instead of static properties

The library is currently broken: it appears that Google changes the name of keys for geometry.location's latitude and longitude randomly, probably on some deploys.
Calling lat() and lng() ensures that the location values are always retrieved.

Excellent ! :+1:

@vskosp up

+1


Fix for #17 : Incorrect data encoded in json when numeric support is enabled

Fix for #17 : Incorrect data encoded in json when numeric support is enabled

I cannot reproduce this somehow. master branch produces correct results for me.

I cannot reproduce this somehow. master branch produces correct results for me.

Hello @tgn3000  , the issue is when we enable numeric support and try to convert the xml.
Kindly change it in xml2json.hpp (line 32) and try again
static const bool xml2json_numeric_support = true;
Now try with the following input :
Input XML
<?xml version="1.0"?>
<Book>
	<name>Sample Book Name</name>
	<author>New Author</author>
	<ISBN>99921-58-10-7</ISBN>
	<price>102.50</price>
</Book>

Output JSON
{
    "Book": {
        "name": "Sample Book Name",
        "author": "New Author",
        "ISBN": 99921,
        "price": 102.5
    }
}

Value like 99921-58-10-7 is encoded as 99921 in json


Installation instructions are ambiguous.

The installation instructions are ambiguous.   It would be helpful if the paths and folder references were explicit, rather than vague.  They could even include links to the correct references.  As they exist, they assume many things that are not evident

Things are still a little vague, indeed.
Right now, all of it works with WSL in Windows, or any mainline version of Linux that has a copy of a RISC-V GCC.
But there is not currently a downloadable-from-here copy of the RISC-V compiler.
All building is done by cd'ing into the folder with the project or demo you want to run and typing make
Any idea how I could word that better for the readme?

Ok, I've updated the instructions for Windows, directing people to use https://gnutoolchains.com/risc-v/


Memoize Rosetta API GraphQL requests for genesis info

Due to the increased size of the genesis ledger post-hard fork, GraphQL queries for genesis info are significantly taking more time to be responded to. This affects Rosetta API requests that rely on these queries, to the point that concurrent Rosetta requests make the GraphQL endpoint unresponsive.
To mitigate this, this PR implements the preexisting memoization mechanism in all Rosetta API GraphQL queries that retrieve genesis info. After caching the GraphQL requests, subsequent Rosetta requests are responded to in a similar time as compared to when the genesis ledger is smaller.
This has been tested by running an updated Rosetta node connected to the post-hard fork test network and validating it with rosetta-cli.

!ci-build-me

!ci-nightly-me

!approved-for-mainnet


Fix the readout error value

The ErrorProfile for the IQMFakeDeneb backend had readout fidelity numbers in place of readout error. This PR fixes that problem.

Can you update the CHANGELOG.rst with a description of your change and a newer version.
For example:
Version 13.17
=============

* Fixed Deneb Readout errors to closer resemble reality. Fidelities were reported as errors. `#125 <https://github.com/iqm-finland/iqm-client/pull/125>`_

If you want to make more involved changes in the future, you can use tox -e format to automatically fix the files to fit our formatting guidelines. After that, you can run tox to automatically check if the tests pass locally on your computer. Later, Github will run these tests for other supported OSs before the changes can be merged.

The tests are currently failing, this is because some tests were using private fields in iqm_client which they were not supposed to use and now no longer exist. This problem is fixed by another PR that is currently under review. The plan is to wait for those changes, then update the branch with latest, and then rerun the tests. This might take a few days.

The other PRs are merged. You will have to merge main into your branch. This will create a merge conflict in the CHANGELOG.rst that you will have to resolve manually. The result should be the changelog from main with your change added as a new version, which is now version 15.1

I've incorporated these changes into #116 so this is obsolete.


Push docs to github pages on publish

Ideally also for embedded-svc

Embedded SVC already has docs hosted on docs.rs because it doesn't depend on any idf crates: https://docs.rs/embedded-svc/0.16.5/embedded_svc/


[µTVM] Raise a better error when project_dir does not exist

Found this problem while fixing the µTVM documentation. If project_dir (specified to ZephyrCompiler constructor) points to a non-existent directory, you get a cryptic compiler error.

@tqchen

@tqchen


Supernova is an open source tool licensed under MIT.
Written with <3 by @nickvourd, @Papadope9 and @0xvm.
Please visit https://github.com/nickvourd/Supernova for more...

[+] Payload size: 510 bytes
[+] Converted payload to GO language
[+] Generated XOR key: byte(0x2d) => 45, byte(0xec) => 236, byte(0xb9) => 185, byte(0x91) => 145, byte(0xb9) => 185, byte(0x7c) => 124, byte(0x79) => 121, byte(0x78) => 120, byte(0xd7) => 215, byte(0x4b) => 75
[+] The encrypted payload with XOR:
shellcode := []byte{0xd1, 0xa4, 0x3a, ........................,0x9e};
[+] Payload encryption with XOR completed successfully! (Completed in 190.611µs)
[+] The encrypted shellcode saved to /home/kali/Desktop/Supernova/payload_xor10.bin file.
┌──(kali㉿kali)-[~/Desktop/Supernova]

I try use it with : icecold didnt work

What is icecold?

Okay, what isn't working with Supernova?

Ok no worries! Please keep in mind this issue chat for Supernova. We are not responsible for the Compatibility with other tools!

Ok but it's still out of the scope


wip: refactor caching options and require own cache instance

To enable caching, you must pass a cache store instance:
class UserSerializer
  include FastJsonApi::ObjectSerializer

  # enable caching
  cache_store ActiveSupport::Cache::MemoryStore.new
end

To disable it, simply pass a falsy value:
class AdminUserSerializer < UserSerializer
  # disable caching
  cache_store false
end

Set race condition ttl and namespace:
class UserSerializer
  include FastJsonApi::ObjectSerializer

  # setting all cache options is now possible
  cache_store ActiveSupport::Cache::MemoryStore.new(
    race_condition_ttl: 20.seconds, namespace: 'my_caching_namespace'
  )
end

Very explicit where cache is stored
No Rails.cache automagic with possible colliding keys
Rails 5.2+ cache versioning support by using a cache that supports it
Implicit namespace support (by cache instance, e.g.
ActiveSupport::Cache::MemoryStore.new(namespace: 'jast_jsonapi')
All cache options can be set directly, no need to for fast jsonapi to handle them

refs #51
Checklist
Please make sure the following requirements are complete:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes /
features)
[ ] All automated checks pass (CI/CD)

@doits I'd like to get this merged soon. Please let me know if you're ok me picking up the work you left. Ty :bowing_man:

@stas I had a week off and try to work on it this week, adding specs at least.
I still was not satisfied with breaking backwards compatibility though. While thinking about it I had the following idea to make everything a little bit simpler and 100 % backwards compatible: Now that we only have cache_store_instance and cache_store_options without any logic behind it, maybe having a new setter for them would be easier?
class UserSerializer
  include FastJsonApi::ObjectSerializer

  def cache_store(store, options = {})
    self.cache_store_instance = store
    self.cache_store_options = options
  end

  # enable caching
  cache_store ActiveSupport::Cache::MemoryStore.new, { namespace: 'fastjson' }
end

This would make the old cache_options deprecated much easier with 100 % backwards compatibility:
      def cache_options(cache_options)
        warn('DEPRECATION WARNING: `cache_options` is deprecated and will be removed soon, please use `cache_store`. See <link to docs>')

        self.cache_store_instance = cache_options[:enabled] ? Rails.cache : nil
        self.cache_store_options = {
          expires_in: cache_options[:cache_length] || 5.minutes
          race_condition_ttl: cache_options[:race_condition_ttl] || 5.seconds
        }
      end

With that we get:

100 % backwards compatibility, e.g. setting the same old defaults or stuff like cache_options enabled: Rails.env.production? will continue to work exactly as before with cache_options
Simpler deprecation logic: Old method will be removed, new method will stay
With new naming of setter it is clear the logic behind it changed, no confusion with old docs etc.
Every option can be passed to .fetch as is, we don't have to remove the old cache option keys from it

What do you think?

@doits I'm not sure I'm sharing your concern. It's pretty normal to deprecate a param/option of a method interface. Folks have a very simple and straight-forward way to upgrade or stick to the current version.
Again, I see the current interface a result of a poor API design and we need to start moving away from it. There's literally 0 benefit in keeping things the way they are now and I did not see any argument that would change my mind (still very open to it though, but I don't want to keep this PR open forever...).

@stats this is exactly how I think, too, though the current implementation gave a false sense of backwards compatibility which was really bothering me. Because it used Rails.cache as a fallback but not the other options, and especially ignored enabled: false. Even if this was a poor API, it should be fully backwards compatible until removed. Or removed right now, because like you say users can simply stick to the old version until they adapt to the new API.
But you are right we do not need another method for this, fallback can be implemented inside the old cache_options method. Which I did just now:

If no store: option is passed, we assume the user is using the old API and behave exactly like it was before but with deprecation messages.
If a store: is passed, we assume the user is using the new API and do not have to care about old behavior

I think this is now the best of both worlds: Simple implementation for the new API and a 100 % backwards compatible fallback.

If this implementation is OK for you, I will rebase all this into one meaningful commit with a proper description and update the docs.
Do you think we need specs for the old cache options behaviour / Rails.cache fallback? And do we need some more specs for new caching options?

... I just added one spec to test that cache_options enabled: true still works. I made it a self sustaining block so it can be easily removed later.

@doits are we ready here? I'd like to merge it :)

@stas yeah I think so if I remember correctly, specs and docs are added and it should work ;-)


Support for Arch Linux

Do you have any plan for nomad desktop in arch linux?Atleast if you make a mkiso script for arch like ubuntu based, we can generate arch iso with the script. Nomad is awesome and very simple.

Hi @Neil-seu.
I was planning to base our ISOs on Arch Linux. Currently we are a quite busy working on other areas of the system, and the Ubuntu based one is fairly stable. If you can (and want, and are so kind to :grinning: ) make us a pull-request and I will review it and continue improving it.

Thats amazing :) But I don't know where to start. I am just hyped with your eye-candy and can't wait to see the nomad on arch repo officially. But don't know how to start. i saw the mkiso part. Is it possible to bring those packages on arch?? so that we can make a nice nomad arch iso?

I think if nomad comes to arch repo, it will be used vastly. as it is much simpler than default kde. i was trying to do something with it. But no progress :(

We could build the WHOLE distro based on Arch (I like Arch too :wink:), but, as I said, currently we have no available time to work with the mkarchiso scripts. We would have to port all of our development environment from an apt-based system to PKGBUILDs (sadly, a lot of work).
We (the Nitrux team) would have to discuss this in a more extensive way.

I understand man :) Its okay :) At least I'm glad to see that you appreciate it :grinning: I will also keep digging though i have very little idea about these porting :laughing:

Hi @Neil-seu,
as you can see Nomad Desktop is not only one app but a collection of carefully integrated apps. So in order to have it running in Arch Linux you will have to pack all of them. Also there could be some changes in the way the configurations are done, so there will be need of some patches.
About including it in the repositories, we will be glad to see our software in such distribution but right now we are busy pushing forward new things. So as @luis-lavaire said if you step up and make the packing we will assist you in what ever you need. Also we could host a repo with the packages.
Cheers

I second @azubieta.
If you need assistance just raise an issue @Neil-seu.

Thanks Guys.. :grin:


Added GO program for Linear Search

Why:
Closes #6
What's being changed:

programme/linear-search/linear-search.go
programme/linear-search/README.md

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

LGTM 👍


GenerateChangelogCommandStep always reports having written a Changelog file

Search first

[X] I searched and no similar issues were found

Description
If you have an empty table or your includeObjects/excludeObjects result in no matches - because of a typo, there is no result set to generate a changelog file.
This is both true for SQL ans XML changelog generation.
Haven't tried a JSON variant.
Steps To Reproduce
Perform a generateChangeLog on an empty table.
In the console, you will see
Generated changelog written to mytable.oracle.sql

But there is no such file.
This is not Oracle-specific.
Expected/Desired Behavior
If no changelog file was produced, do not state it was generated.
Liquibase Version
4.29.2
Database Vendor & Version
No response
Liquibase Integration
No response
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
No response
Additional Context
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR (Thank you!)

This bug is due to only checking if the a changelog file is defined, without checking its existance, see here.
            if (StringUtil.trimToNull(changeLogFile) != null) {
                Scope.getCurrentScope().getUI().sendMessage("Generated changelog written to " + changeLogFile);
            }

@catull thanks for filing this and pointing out where it occurs. We should definitely not show this message if nothing was generated.

Hi @catull,
Yes, you are right this is an issue in the IF block you have highlighted in your last comment. I think we should change that IF block to something like:
if (!changeLogWriter.generateChangeSets().isEmpty()) {
                Scope.getCurrentScope().getUI().sendMessage("Generated changelog written to " + changeLogFile);
            }
            else {
                Scope.getCurrentScope().getUI().sendMessage(String.format("Changelog not created. There are not changesets to write to %s changelog", changeLogFile));
            }

Are you keen to implement these changes?  It would be nice to also add some tests to validate these scenarios, could you also do that?
Thanks,
Daniel.

Dear @MalloD12
Yes, I can implement the change, and add some test cases along with it.
Reagrds,
@catull


Add feedback faces into the guides

Add the feedback faces from openliberty.io to the end of the cloud-hosted-guides section.

These will have to be connected using the GA (Google analytics) and will have to contain JS script to be reactive.
Another possible way is to have the face when clicked send a POST request to the google analytics.

close

add Give us feedback link to the last step


Shell Script output "Build Issues" report

I've got a shell task in a VSTS build, running on a linux agent, that outputs the following in the logs

2018-06-11T18:04:51.9611244Z ##[section]Starting: Shell Script build/yay-lint.sh
2018-06-11T18:04:51.9940268Z ==============================================================================
2018-06-11T18:04:51.9952160Z Task         : Shell Script
2018-06-11T18:04:51.9963624Z Description  : Run a shell script using bash
2018-06-11T18:04:51.9975070Z Version      : 2.1.3
2018-06-11T18:04:51.9986531Z Author       : Microsoft Corporation
2018-06-11T18:04:51.9998119Z Help         : More Information
2018-06-11T18:04:52.0009392Z ==============================================================================
2018-06-11T18:04:53.3767744Z [command]/bin/bash /home/me/myagent/_work/1/s/build/yay-lint.sh
2018-06-11T18:04:53.9738204Z site/profile/file1.ext - WARNING: class not documented on line 1
2018-06-11T18:04:53.9764612Z site/profile/file2.ext - WARNING: class not documented on line 1
2018-06-11T18:04:53.9778096Z site/profile/file3.ext - WARNING: class not documented on line 1
2018-06-11T18:04:53.9905226Z ##[error]/bin/bash failed with return code: 1
2018-06-11T18:04:54.0008722Z ##[error]Bash failed with error: /bin/bash failed with return code: 1
2018-06-11T18:04:54.0672972Z ##[section]Finishing: Shell Script build/yay-lint.sh

I'm running the linting command in a "fail on warning mode" that fails with return code 1.
When I look at the build summary, I see the follwoing,

I'd like to see the text "site/profile/file1.ext - WARNING: class not documented on line 1"  Is this possible?  I've tried all the stdout redirections that I can think of and can't seem to get it to work.
I felt like I got pretty close with

echo "##vso[task.logissue type=error]$(yay-lint site/profile --fail-on-warnings)"

But it only gave 1 of the 3 output lines

Seems more like a bash scripting question :)
Haven't tried this but I think you want to run it and then pipe to grep (to filter) and then to sed which is a stream editor.
Something like this?
https://unix.stackexchange.com/questions/143798/why-does-the-same-sed-regex-after-grep-fail-when-run-in-a-bash-script-vs-bash
To do that, you would have to use a cmd line task or a a script in between.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days


Loader_5

a simple loader is added with html, css and javascript

Hello @dhawalbisht
Hope this finds you well and hope you are doing great.. Just checking up on you. We can get your code in if you make the changes as indicated above, We would also be closing this PR soon.
Thank you for your understanding.


[Rector] Apply Rector: RenameParamToMatchTypeRector

Rename parameter variable to match typed parameter in methods with RenameParamToMatchTypeRector rector rule.
Checklist:

[x] Securely signed commits

This will break in php8 with regard to named parameter usage. https://3v4l.org/qnTGX

Ok 👍 , closing then


Automation: Remoting ScriptBlock is not convertible from string

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I am trying to execute a simple pipe with an foreach-object remotely from a c# net6
The ScriptBlock parameter get converted to a string and throws an conversion exception on the remote site.
I found no possibility to do.
The remote site is linux pwsh 7.2.5
cmd = new Command("Get-ChildItem");
            cmd.Parameters.Add("Path", "/dev/disk/by-id/scsi-*");
            commands.Add(cmd);

            cmd = new Command("Foreach-Object");
            cmd.Parameters.Add("Process", ScriptBlock.Create("Join-Path -Path $_.Directory -Child $_.Target -Resolve"));
            commands.Add(cmd);

            //this is executed on a linux remote runspace
            var path = await power.InvokeAsync(commands).FirstOrDefaultAsync();

´´´

### Expected behavior

```console
execute the foreach-object script block remotely

Actual behavior
error

Error details
.Management.Automation.RemoteException : Cannot bind parameter 'Process'. Cannot convert the "Join-Path -Path $_.Directory -Child $_.Target -Resolve" value of type "System.String" to type "System.Management.Automation.ScriptBlock".

Environment data
net6, Microsoft.PowerShell (SDK.7.2.5)

Visuals
No response

Unfortunately and bewilderingly, this behavior was declared to be by design - see duplicate

https://github.com/PowerShell/PowerShell/issues/11698

grandiose design,
It is possible to enter-pssession into the remote system and execute everything
But with c# and a Runspace and Pipeline its impossible to Get-ChildItems + Join-Path in a single pipe

An alternative that works in more scenarios is using ps.AddScript() like so
PowerShell ps = PowerShell.Create();
ps.AddScript("Get-ChildItem -Path $args[0] | ForEach-Object { Join-Path -Path $_.Directory -Child $_.Target -Resolve }");
ps.AddArgument("/dev/disk/by-id/scsci-*");

var path = await ps.InvokeAsync(commands).FirstOrDefaultAsync();

Unless I'm mistaken you can also just do | Select-Object -ExpandProperty FullName rather than using Join-Path here.

@jborean93  thx for the example
Something changed from pwsh6 to pwsh7 the Target is now suddenly relative to directory property value
I used it here:
https://github.com/Zetanova/hyperv-csi-driver/blob/726552dc1716ec96996727bd394cdc8bad4b1663/src/hyperv-csi-driver/Infrastructure/HypervNodeService.cs#L102
Because of some bugs in the RunspacePool, I worte my own simple runspace pool management
and view Runspace utility extensions. Maybe you have interest/idea for it.

Weird the FullName should be the absolute path, I know the behaviour of ToString() on the FIleInfo object changed from the file name to the absolute path but the FullName property stayed the same.

I need the target of the symlink and not the path of the symlink to discover the block device by VHDX ID (guid)
It is working now by a workaround, I query Directory and Target first and after execute a second pipe with them.
The Join-Path needs to be executed remotely because maybe the local system is running under windows.
PS /root> Get-ChildItem -Path /dev/disk/by-id/scsi-*47f90bea* | fl

    Directory: /dev/disk/by-id

Name           : scsi-36002248047f90bead727987e9c2393f2
Length         : 9
CreationTime   : 7/20/2022 12:54:42 PM
LastWriteTime  : 7/20/2022 12:54:42 PM
LastAccessTime : 7/21/2022 7:57:31 PM
Mode           : l----
LinkType       : SymbolicLink
Target         : ../../sdb
VersionInfo    :

Name           : scsi-36002248047f90bead727987e9c2393f2-part1
Length         : 10
CreationTime   : 7/20/2022 12:54:42 PM
LastWriteTime  : 7/20/2022 12:54:42 PM
LastAccessTime : 7/21/2022 7:57:31 PM
Mode           : l----
LinkType       : SymbolicLink
Target         : ../../sdb1
VersionInfo    :

PS /root> Get-ChildItem -Path /dev/disk/by-id/scsi-*47f90bea* | select FullName

FullName
--------
/dev/disk/by-id/scsi-36002248047f90bead727987e9c2393f2
/dev/disk/by-id/scsi-36002248047f90bead727987e9c2393f2-part1

PS /root>

Ah my apologise you are dealing with symlinks and Target that's my fault. Please ignore the -ExpandProperty FullName stuff.

If memory serves, .Target was changed to report the target path as defined rather than what full path it resolves to.
To resolve the target to a full path:
Since (at least?) 7.2, you can now use:
# $true means: follow to the ultimate target, if the target is itself a symlink.
(Get-Item /proc/self).ResolveLinkTarget($true).FullName

Since 7.3.0-preview.2, there's now an ETS .ResolvedTarget code property that reports the target's full path as a string; it simply calls .ResolveLinkTarget($true).FullName.
(Get-Item /proc/self).ResolvedTarget

If memory serves, .Target was changed to report the target path as defined - which is often relative - rather than what full path it resolves to.

Just to link to the issue about this change: https://github.com/PowerShell/PowerShell/issues/13365

The change on now the extension of features are fine,
But that someone can't use Foreach-Object Command remotely with a pipeline of Microsoft.PowerShell.SDK looks like a regression.
I could really find no option to pass the parameters to Joint-Path from the pipeline
And I am working with
AppContext.SetSwitch("System.Runtime.Serialization.EnableUnsafeBinaryFormatterSerialization", true);`

@mklement0
ResolveLinkTarget() is working like it should be:
PS /root> (Get-ChildItem -Path /dev/disk/by-id/scsi-*47f90bea*).ResolveLinkTarget($true).FullName
/dev/sdb
/dev/sdb1


Automatically adjust buffer capacity in TestEm3

This change automatically determines the buffer capacity for the electron, positron and gamma buffers based on the total available global memory. This change is a derivation of the first commit in #197.
The choice of using 1/4th of the global memory size as Capacity for each of the 3 buffers is rather arbitrary and smaller to what @amadio proposed before. However, 2/7th did not work on my workstation as I seem to have more VRAM occupied for my desktop environment. It is still important though that most of the available VRAM is also available to the example being run and not just a hard-coded constant that people tweak depending on their GPU memory size.

It is still important though that most of the available VRAM is also available to the example being run and not just a hard-coded constant that people tweak depending on their GPU memory size.

Why not just use the default one and be done with it? We had many discussions on variations of this patch already, I don't see what changes to past proposals of this. It's just not realistic to use the maximum memory and then expect the simulation to provide enough particles in one event to fill that.

It's just not realistic to use the maximum memory and then expect the simulation to provide enough particles in one event to fill that.

I explained that yesterday during the talk. We are doing R&D work and while, apparently (because I have only heard about this), Geant4 cannot easily provide particles from more than 1 event now, it might be able to do so a few years down the road. So it is worthwhile to consider relaxed requirements for doing R&D work. And we need the bigger batch sizes and thus capacities to explore how to utilize GPUs efficiently.

I explained that yesterday during the talk. We are doing R&D work and while, apparently (because I have only heard about this), Geant4 cannot easily provide particles from more than 1 event now, it might be able to do so a few years down the road. So it is worthwhile to consider relaxed requirements for doing R&D work. And we need the bigger batch sizes and thus capacities to explore how to utilize GPUs efficiently.

I must have missed that part and can't find it in the slides. And we do have a solution for the utilization problem today: use multiple threads on the CPU to submit work to the GPU(s). That is realistic because Geant4 can handle one event per each thread, and their EM parts together can keep the GPU busy. That doesn't need any of this unrealistic increase of capacity / memory / batch size.

I must have missed that part and can't find it in the slides.

I think I mentioned it around slide 20 when I discussed the separate interactions kernel implementation.

And we do have a solution for the utilization problem today: use multiple threads on the CPU to submit work to the GPU(s). That is realistic because Geant4 can handle one event per each thread, and their EM parts together can keep the GPU busy. That doesn't need any of this unrealistic increase of capacity / memory / batch size.

There are of course multiple solutions to get the GPU busy. I am just trying a different approach, where AdePT can just buffer up tracks from many events and thus provide bigger batches to the GPU. I know that that does not fit Geant4's current workflow, but then maybe Geant4 also needs to adapt at some point if this route proofs to be successful. We are doing R&D after all and we should explore different strategies.

Another argument against this kind of dynamic sizing is that it will lead to slightly different simulations on different machines with different amounts of memory when magnetic field propagation is enabled and we potentially kill particles at the tails...

That may be and I don't have the amount of in-depth knowledge on magnetic field propagation to know whether there is a solution to that. In my mental model, I can process tracks independently from each other so it does not matter for the correctness or determinism whether I put 100 or 10'000 of them on the GPU at the same time. But it matters for the throughput.
Anyway, I will move my effort to the new example created by @hageboeck now in #216.

And we do have a solution for the utilization problem today: use multiple threads on the CPU to submit work to the GPU(s). That is realistic because Geant4 can handle one event per each thread, and their EM parts together can keep the GPU busy. That doesn't need any of this unrealistic increase of capacity / memory / batch size.

There are of course multiple solutions to get the GPU busy. I am just trying a different approach, where AdePT can just buffer up tracks from many events and thus provide bigger batches to the GPU. I know that that does not fit Geant4's current workflow, but then maybe Geant4 also needs to adapt at some point if this route proofs to be successful. We are doing R&D after all and we should explore different strategies.

Sure, but this topic was again discussed last week during the Celeritas visit, and right now all data clearly shows that multiple threads with reasonably sized batches of particles (within expectations of LHC experiments) is actually more efficient. I'm pasting some numbers for TestEm3MT on my 2070 SUPER that I also showed last week:
-particles 10000 -batch 1000 -threads 1
  16.5937

-particles 10000 -batch 500 -threads 2
  15.9731

-particles 10000 -batch 250 -threads 4
  15.7286

-particles 10000 -batch 400 -threads 1
  16.6332

-particles 10000 -batch 200 -threads 2
  15.9354

-particles 10000 -batch 100 -threads 4
  15.7649

-particles 10000 -batch 200 -threads 1
  17.1891

-particles 10000 -batch 100 -threads 2
  16.3435

-particles 10000 -batch 50 -threads 4
  16.2854

-particles 10000 -batch 100 -threads 1
  18.5353

-particles 10000 -batch 50 -threads 2
  17.3869

-particles 10000 -batch 25 -threads 4
  17.8254

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

-particles 10000 -batch 100 -threads 1
  18.5353

-particles 10000 -batch 100 -threads 2
  16.3435

-particles 10000 -batch 100 -threads 4
  15.7649

-particles 10000 -batch 100 -threads 8
  15.6614

I'd like to point out that this is without any hadronic interactions, which we know scales linearly with more threads in Geant4.

Sure, but this topic was again discussed last week during the Celeritas visit, and right now all data clearly shows that multiple threads with reasonably sized batches of particles (within expectations of LHC experiments) is actually more efficient.

That is a good insight to have! But the difference between the various thread counts is not that big and this may shift due to subsequent optimizations of the GPU kernels. Nonetheless, I think we should also have support for multiple threads in the SplitKernels example eventually.


Add application/version in appengine-web.xml support

The values in the deployment config will take precedence, but we should support users that choose to configure their application by including these values in appengine-web.xml

as @patflynn said, nothing to do here


Add blockchain backend to Enerbot

For future releases of this bot, we (i) need a backend, the proposal for this it would be something like this:

Create a database with privileged users of this community.
Store useful information and some related to events.
Provision a proper way to keep a registry of credentials like channels or admins instead of using env variables.

Kind regards.
Post Scriptum: You can use and backend like:

Firebase Database
Mysql (please no)
PostgreSQL
Whatever your heart feels right.

If you ask me I would rather use Firebase than another MySQL with VOs. That and because you can make it happen, so go ahead and make ENERGON great again @siorellana.


Test enhancement

Changed log

Add php-7.1 to php-7.3 tests in Travis CI build.
Let this package require php-7.1 version at least.
Using the latest stable PHPUnit version.
Add the filter tag to let PHPUnit can do the code coverage work.
Add the missed type hint in some method annotations.
Remove require_once('DetectorBase.php'); because of useless. The vendor/autoload.php will load these classes in tests folder automatically.
Add the missed type hint for some method annotations.

@tivie Could you also review this one ?


fix simple table tag

Questions
Answers

Description?
tix table tag

Type?
bug fix

BC breaks?
no

Deprecations?
no

Possible impacts?
Custom css on first table th that now is td

Th in first cell is a legitimate syntax for bootstrap 4.6:
https://getbootstrap.com/docs/4.6/content/tables/
I will wait for @NeOMakinG to state his opinion.

Good to know! 👍🏻

Also: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/tr

It's a valid syntax yes.
@drc0 can you please give us more information on why you are trying to change this? I don't get it like that, without issues and description
If you feel like your contribution is relevant, create an issue and let's talk about it.

@NeOMakinG thanks for asking, the issue raised from a bug we had on our custom theme derived from classic-theme, we styled that table with a selector like tbody td, so it was a styling bug and not a semantic one. Your comments in this PR are valid though and having [code] has his validity. I'm not feeling changing this to td has his pros anymore, and I'm going to close this PR.
thanks


Provide profiling example

Does silk actually perform profiling? Or only log sql queries?
This is a nice looking project, and didn't take long to set up, but I didn't see anything I would call a profile, and don't see any screenshots of one in the readme. There is a nice log of all database queries, but when I hear about a profile in python, I think of something like this:
https://docs.python.org/2/library/profile.html

I decided to use http://jiffyclub.github.io/snakeviz/. If silk could be combine with snakeviz, so silk put the profile in the database, then somehow had a link for snakeviz to view it, that would be awesome. Clearly this would not be a trivial undertaking, snakeviz wants to read a file, and doesn't know anything about django models.

@AgDude yep silk's profiling is limited to profiling time taken to execute blocks of code & num. database queries executed in that block of code and allows you to trace back those db queries to their origin. @JoshData contributed a cProfile integration a while back though, which sounds more like what you're wanting.
If you fancy contributing the snakeviz integration I'll happy add you to the django-silk org - this would be an awesome enhancement & i'd be around for any questions you may have wrt. the code base. Doubt I'll have time to implement this myself though - which you can probably tell by my awful response time :(

When I add SILKY_PYTHON_PROFILER = True to my settings files I get a nice cProfile output in the Profiling tab under Python Profiling.
I copied that text and saved it in a .prof but when I run it through snakeviz i get the following error:
snakeviz: error: the file ~/Desktop/test1.prof is not a valid profile. Generate profiles using:
python -m cProfile -o my_program.prof my_program.py

Any ideas?

@AgDude @hanleyhansen Was enhancement possibly pushed out in a recent release?

Yes. The PR i submitted adds a download button to the Profiling page where you can download the binary .prof file and use it in snakeviz.
See: https://github.com/django-silk/silk/pull/123
That was merged and uploaded to pypi.

@hanleyhansen Thanks for confirming with me. I will close this issue and say it was released in v0.6.0 as an exportable feature.


Add B Souty to Contributors list

Thanks for this guide

Hi  @B-Souty, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


"Delete" function deletes more than what's selected when using "Select all"

Link to reproduction
No response
Describe the Bug
I wanted to delete a subset of all entries in a upload collection: the filenames starting with some prefix. There were hundreds of matching entries so I used the "Select all" function on the top right to select all entries (not only the entries on the current page). It is my understanding that this should select all matching entries, not simply ALL entries... and the number of selected entries after clicking on this function seems to reflect that.
Despite selecting only a subset of all entries in a collection, and despite the deletion confirmation modal displaying the right amount of selected entries, the deletion process actually deletes more entries than expected (all entries in this case).
To Reproduce
https://github.com/payloadcms/payload/assets/29893320/396384a6-243f-45a0-b68d-31ab24ccef95
(It seems the video fails to load on Firefox... please try on a Chromium-based browser)

Create multiple entries in a collection (here, 2 entries in the Scan collection)
Use filters or the search function to get a subset of all the entries (here, 1 entry that contains the substring "1").
Use the "Select all X [name of the collection]" to select all entries matching our search
Click on the Delete button
The "Confirm deletion" modal also reports the right amount of entries to be deleted.
After confirming, the success toast at the bottom indicates that ALL entries have been deleted.

Notes
I've tested other cases such as:

Manually selecting some subset using the checkboxes in each entry's row.
Selected a subset using the checkbox in the header of the table (which selects all entries in the current page)

In both cases, the right amount of element were deleted when I used the Delete function. It seems the problem only arise when using the "Select all X [name of the collection]" button.
Payload Version
2.16.1
Adapters and Plugins
"@payloadcms/bundler-webpack": "1.0.6",     "@payloadcms/db-mongodb": "1.5.1",     "@payloadcms/richtext-lexical": "0.10.0"

I can confirm this issue (Payload 2.28.0) - this will delete every entry in the collection by accident

I can confirm this in 3.0.0-beta.111 as well


update Class vs Object

ISSUE TYPE

[ ] Typo
[ ] Bug
[x] Feature

WHERE / WHAT

--> Class vs object question in technical question
EXPECTED BEHAVIOR

Add more information about class vs object as the newbies won't understand a thing by reading that one line
REPRODUCTION STEPS

#

Looks like you made a few PRs for this. I'll go ahead and close it.


NPM Dev Run issue - Urgent

Hi Support,
I have installed the Node.js, NPM and while running dev it shows error as per attached image (react.png). Pls assist us to fix this issue.
Thanks,
Suhanya.M

You need to install the dependencies. Change to the directory package.json is in and npm install.  If you already did that (which you didn't appear to), make sure to npm install -g concurrent.


Fixing linting issues

Running black on file

@simondeziel  @stgraber Fixing the linting from my previous PR. The checks didn't run when I opened the previous PR for some reason.

Codecov Report

Merging #522 (9cf52de) into master (859af24) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #522   +/-   ##
=======================================
  Coverage   97.03%   97.03%
=======================================
  Files          56       56
  Lines        4250     4250
=======================================
  Hits         4124     4124
  Misses        126      126

Impacted Files
Coverage Δ

pylxd/models/tests/test_network.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@simondeziel @stgraber Fixing the linting from my previous PR. The checks didn't run when I opened the previous PR for some reason.

Right, new contributors need to be approved for CI to kick in. Looks like you are on the approved list now.


Can't see tables in Connection Explorer - get "Nothing here"

Description
Connecting to an SQLite database succeeds, however I can't see any of the tables in the CONNECTIONS explorer.
All I see is: Nothing here
I've tried with a full absolute filename, relative file name to workspace directory (with and without the ${workspaceFolder} variable.
I've installed Node.js/npm.  vscode asked me to install sqlite, which I did.
Expected behavior
A list of table names :)
Screenshots

Desktop (please complete the following information):

SQLTools Version 0.23.0
VSCode Version: 1.49.2
OS: Windows 10 Pro - Microsoft Windows [Version 10.0.19041.508]
Driver: SQLite
Database version: SQLite 3

Additional context
Am using Python 3.7, with SQLAlchemy 1.3.19 for an application.
Can view the tables and data ok with SQLStudio 3.2.1 (though I'm having trouble delete a table and some columns in other tables due to some FK error).  I don't think it is related, but I thought I'd mention it.  I can still see the tables regardless.

Got the same issue.
Same versions as OP.

I think the problem is in resolving the SQLite file's location. I went into my workspace's settings and changed the sqltools.connections object from
{
  "sqltools.connections": [
    {
      "previewLimit": 50,
      "driver": "SQLite",
      "name": "Dev",
      "database": "${workspaceFolder:client-backend}/src/db/dev.db.sqlite"
    }
  ],
}

to
{
  "sqltools.connections": [
    {
      "previewLimit": 50,
      "driver": "SQLite",
      "name": "Dev",
      "database": "./src/db/dev.db.sqlite"
    }
  ],
}

and then the connection worked fine.

@KerimG Thank you!!! This fixed the issue for me too -- I wonder why they use that weird notation if it doesn't work?

This fixed the issue for me too

This fixed the issue for me too

@KerimG  Thanks a lot, I really solve this issue

@KerimG  Thanks a lot, I really solve this issue

@KerimG , I can confirm that this workaround works for me, but it's not a permanent solution.
@mtxr, is this easy to fix?

@KerimG , I can confirm that this workaround works for me, but it's not a permanent solution.
@mtxr, is this easy to fix?

I think this is a problem of Windows paths. I've just found an empty C:\c%3A\__Data\...\data\database.sqlite file, which I assume was created by sqltools.

I think this is a problem of Windows paths. I've just found an empty C:\c%3A\__Data\...\data\database.sqlite file, which I assume was created by sqltools.

On windows, using SQLTools v0.24.0 and SQLTools SQLite v0.3.0 I can also confirm that this is an issue.
The new connection wizard wants to use the VSCode variable and it does not work with the tool.


Ensure packaged IaSQL container runs locally end-to-end

https://discord.com/channels/920684574231646229/920885549668114472/1072705282624454716
It seems like it might not have enough memory? If so, we need to increase the memory docker gives the container and get it below the default memory requirement by fixing craco or perhaps https://github.com/iasql/iasql/pull/1940
iasql main % yarn docker-build
yarn run v1.22.19
$ docker build -t iasql:latest .
[+] Building 18.1s (28/46)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 2.27kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 34B                                                                                0.0s
 => [internal] load metadata for docker.io/library/debian:bullseye                                                                                0.3s
 => [internal] load build context                                                                                0.1s
 => => transferring context: 59.46kB                                                                                0.1s
 => [ 1/42] FROM docker.io/library/debian:bullseye@sha256:92277f7108c432febe41beffd367dbb6dac60b9fbfe517c77208e6457eafe22b                                                                     0.0s
 => CACHED [ 2/42] RUN apt update                                                                                0.0s
 => CACHED [ 3/42] RUN apt install curl ca-certificates gnupg build-essential git jq -y                                                                                0.0s
 => CACHED [ 4/42] RUN ["bash", "-c", "curl https://www.postgresql.org/media/keys/ACCC4CF8.asc | gpg --dearmor | tee /etc/apt/trusted.gpg.d/apt.postgresql.org.gpg >/dev/null"]                0.0s
 => CACHED [ 5/42] RUN ["bash", "-c", "echo 'deb http://apt.postgresql.org/pub/repos/apt/ bullseye-pgdg main' > /etc/apt/sources.list.d/postgresql.list"]                                      0.0s
 => CACHED [ 6/42] RUN apt update                                                                                0.0s
 => CACHED [ 7/42] RUN apt upgrade -y                                                                                0.0s
 => CACHED [ 8/42] RUN apt install postgresql-client-14 postgresql-14 postgresql-14-cron pgbouncer postgresql-server-dev-14 libcurl4-openssl-dev git make g++ locales-all -y                   0.0s
 => CACHED [ 9/42] RUN git clone --depth 1 https://github.com/pramsey/pgsql-http                                                                                0.0s
 => CACHED [10/42] RUN cd pgsql-http && make && make install                                                                                0.0s
 => CACHED [11/42] RUN ["bash", "-c", "curl -fsSL https://deb.nodesource.com/setup_16.x | bash -"]                                                                                0.0s
 => CACHED [12/42] RUN apt install nodejs -y                                                                                0.0s
 => CACHED [13/42] RUN npm install -g yarn                                                                                0.0s
 => CACHED [14/42] WORKDIR /dashboard                                                                                0.0s
 => CACHED [15/42] COPY dashboard/package.json package.json                                                                                0.0s
 => CACHED [16/42] COPY dashboard/yarn.lock yarn.lock                                                                                0.0s
 => CACHED [17/42] RUN yarn install                                                                                0.0s
 => CACHED [18/42] COPY dashboard/assets assets                                                                                0.0s
 => CACHED [19/42] COPY dashboard/public public                                                                                0.0s
 => CACHED [20/42] COPY dashboard/tsconfig.json tsconfig.json                                                                                0.0s
 => CACHED [21/42] COPY dashboard/tailwind.config.js tailwind.config.js                                                                                0.0s
 => CACHED [22/42] COPY dashboard/craco.config.js craco.config.js                                                                                0.0s
 => [23/42] COPY dashboard/src src                                                                                0.0s
 => ERROR [24/42] RUN yarn build                                                                                17.6s
------
 > [24/42] RUN yarn build:
#28 0.447 yarn run v1.22.19
#28 0.463 $ craco --max_old_space_size=8192 build
#28 2.777 Creating an optimized production build...
#28 17.28
#28 17.28                     The build failed because the process exited too early.
#28 17.28                     This probably means the system ran out of memory or someone called
#28 17.28                     `kill -9` on the process.
#28 17.28
#28 17.46 error Command failed with exit code 1.
#28 17.46 info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
------
executor failed running [/bin/sh -c yarn build]: exit code: 1
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


Improve burger menu

Hi,
Today, I used Bootstrap v4 alpha 6 and wanted to change color of burger menu (3 lines).
But I discovered that burger menu was a button which contained a span with a background-image.
This background-image is an SVG.. So developers can not change color...
I think it would be a good idea to:
FIRSTLY, change svg to three span to enable developers to modify burger menu color.
SECONDLY, in full css (keyframes) and/or with javascript/jquery, propose few animations like these codepen examples:
https://codepen.io/designcouch/pen/Atyop
Thanks!

You can customize the navbar toggler icon via Sass by changing the $navbar-dark-color or $navbar-light-color, which is used to generate the color on the SVG.

Which essentially means bootstrap isn't usable for dark backgrounds without compiling your own? :(

I mean you can jump to that conclusion, or you can take a look at the source code, issues, and PRs that led to the code's current status and provide more valuable feedback in the form of suggested changes.
That said, I imagine you would need to recompile everything to build a completely dark site. Bootstrap's variables are setup to support that—it's why we extend the $body-bg and $body-color variables into other components.

I get that you have to make decisions balancing features against each other. The problem now is that a lot of components has -dark variants for their classes which gets you almost all the way towards a dark page straight from a CDN bootstrap. The hamburger menu is the only component I've noticed so far that breaks it.


Fibonacci list generating script

(Put an X inside the [ ] to denote check mark [X].)

[X] If creating a new file :

[X] added links to it in the README files ?
[ ] included tests with it ?
[X] added description (overview of algorithm, time and space compleixty, and possible edge case) in docstrings ?

[ ] if done some changes :

[ ] wrote short description in the PR explaining what the changes do ?
[ ] Fixes #[issue number] if related to any issue

[ ] other

Pull Request Test Coverage Report for Build 768

0 of 10 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 71.443%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

algorithms/maths/fibonacci.py
0
10
0.0%

Totals

Change from base Build 764:
-0.1%

Covered Lines:
4308

Relevant Lines:
6030

💛  - Coveralls

@keon I made the changes to the script and added a unit test. It passed when I compiled it.


[WIP] Dockerfile changes to build for ROCm

Description
This PR changes the Dockerfile to build for HipBLAS
Notes for Reviewers
I'm not sure about referencing the rocm/dev-ubuntu-22.04:6.0-complete container, I couldn't find anything publicly available that included the libs. I have one locally that I will look at publishing.
Signed commits

[ X] Yes, I signed my commits.

I also updated the go builder to from bullseye to bookworm , there was a glibc dependency for ROCm that didn't work on bullseye

does this actually build?
looking at https://github.com/mudler/LocalAI/issues/1592 atm

It works for me on llama.cpp at least, I havent tested the rest yet. It doesn't include stablediffusion IIRC. I'm still testing is why I still have it WIP

great work, I love to see this one

it does build however I still seem to have the same issue as I had when following the arch build instructions in the localai wiki. where the model will load without a problem, then for some reason the api query is never executed and the model just sits there in vram, this issue also persists when running this method in CPU mode also :|

This works with stablediffusion now. The diffusers backend is still giving me trouble

This works with diffusers now

@mudler I removed the WIP header,. Is there a good way to test this? I did some manual testing and it seems to work.

for acceleration tests we have a specific workflow but for now is disabled as it waits for HW.
However that doesn't cover container e2e tests (which are still manual) for all the backends

@fenfir with #1689 I'm actually making the base image configurable - maybe it is now as easy as pointing to the ROCm images? any chance you can give that a shot?

@fenfir I've updated the PR and based the Dockerfile directly on the rocm images, can you try it out?

ok, let's give a shot to the images on master


Use docs.microsoft.com links

Fix https://github.com/MicrosoftDocs/windowsserverdocs.zh-cn/issues/17

#sign-off


Fresh install - Tutorial

What version of Remix are you using?
1.7.0
Steps to Reproduce

npx create-remix --template remix-run/indie-stack blog-tutorial
Typescript
npm install
npm run dev (FAIL)

Expected Behavior
I should be able to installl tutorial template and follow along
Actual Behavior
Can't run dev server for tutorial template
> dev
> run-p dev:*

> dev:css
> npm run generate:css -- --watch

> dev:remix
> cross-env NODE_ENV=development binode --require ./mocks -- @remix-run/dev:remix dev

node:internal/modules/cjs/loader:498
      throw e;
      ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib' is not defined by "exports" in /Users/..../remix/blog-tutorial/node_modules/headers-polyfill/package.json
    at new NodeError (node:internal/errors:393:5)
    at throwExportsNotFound (node:internal/modules/esm/resolve:340:9)
    at packageExportsResolve (node:internal/modules/esm/resolve:619:3)
    at resolveExports (node:internal/modules/cjs/loader:492:36)
    at Module._findPath (node:internal/modules/cjs/loader:532:31)
    at Module._resolveFilename (node:internal/modules/cjs/loader:941:27)
    at Module._load (node:internal/modules/cjs/loader:803:27)
    at Module.require (node:internal/modules/cjs/loader:1021:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/Users/.../remix/blog-tutorial/node_modules/msw/lib/node/index.js:656:18) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

Node.js v18.9.0

> generate:css
> tailwindcss -o ./app/styles/tailwind.css --watch

Rebuilding...
Done in 213ms.

You can bump msw version, works for me.
"msw": "^0.47.0",

@romkor Thanks for heads up! It indeed was the problem! Now working.


[TexField] [Tabs] tabs value changes to R…s.c…s.Constructor on input change

I just put:
Contact = React.createClass
  render: ->
    <div>
      <TextField
        hintText="Name"   />
      <TextField
        hintText="E-mail" />
      <TextField
        hintText="Typ een bericht..."
        multiLine={true}  />
    </div>

Inside:
     <Tabs
<EMAIL_ADDRESS>            onChange={@tabsChange}>
            <Tab label="Home" value="b" >
              Home
            </Tab>
            <Tab label="Contact" value="c">
             <Contact />
            </Tab>
      </Tabs>

Where tabsChange is:
  tabsChange: (value, e, tab) ->
     @setState({tabsValue: value})

But typing in the TextField, it will set the value of Tabs to some SyntheticEvent somehow. Thereby changing the tab to none selected, and not showing the tab contents anymore.
I have to do:
tabsChange: (value, e, tab) ->
    if typeof value is 'string'
        @setState({tabsValue: value})

To prevent the Tabss value from changing to a function instead of what it's supposed to be: a string.
It looks like the tabs's onChange is getting triggered with each keystroke

This is still an issue on the latest 0.14.4.
I have to use e.stopPropagation() every time to prevent Tabs from going blank on every keypress in a TextField (like described above)

Since you understand the problem pretty well, do you think you could you have a dig into the code and see of you can find the cause?

@TrySpace : Is this still an issue? Care to submit a PR?

same issue here. I believe the issue is caused by the valuelink. Tabs onChange is invoked for the bubble up event 'react-change'.
Tabs should not use the valuelink to handle the tab activation.

This issue should have been fixed by #5219.


Newsletter sign-up is unresponsive and unstyled

Describe the bug
Newsletter sign-up is unresponsive and unstyled. Make the button look the other buttons on the website, style the card and make it responsive.
https://user-images.githubusercontent.com/54665036/120535293-72343400-c400-11eb-95df-8e12d1866f52.mp4

Note:

If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.

Hello! Can I work on the issue as I am a participant of LM-SoC

/assign

@yashraj-02-techie Are you working on this?


Add new resource provider Microsoft.EngagementFabric

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

The API spec was review and merged via Private repo PR 569

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

@xzsheng Please link the PR that was approved in the private repo
Were there any additional changes made to the specs since the PR was approved?

@xzsheng Please link the PR that was approved in the private repo
Were there any additional changes made to the specs since the PR was approved?

@dsgouda The approved PR in private repo is https://github.com/Azure/azure-rest-api-specs-pr/pull/569
There is no change since the PR was approved.

@jhendrixMSFT could you take a look at the Go generation failures, not sure why it's failing

From the log:
System.InvalidOperationException: codegen for preview swagger Microsoft.EngagementFabric/preview/2018-09-01/EngagementFabric.json must be under a preview subdirectory

Also I don't see the readme.go.md file.  Please be sure to put Go SDK config in its own config file not readme.md.

Thanks @jhendrixMSFT, @xzsheng please address the comments. You can refer to some existing readme.go.md files in the repo.

Thanks @jhendrixMSFT, @xzsheng please address the comments. You can refer to some existing readme.go.md files in the repo.

@dsgouda Readme.go.md added. All checks are green now. Thanks.


Summarize the content of a certain title

Hello, I would like to ask if this plugin can help count the content under specific headings in files under specific folders. I have tried multiple times to write code using GPT but failed. Can you provide a similar example? Thank you very much. The dataview plugin seems to be doing well. I would like to ask if this plugin can help count the content under specific headings in specific folders and provide a similar example. Thank you very much, The dataview plugin seems to be unable to perform this function

Yes, you can. That's the exact reason why I created this plugin. Here is a code I'm been using, and what it looks like.
scope:
    - ReadingNotes/
matches:
    - regex: '[\S\s]+'
      template: >-
        {{eval "return data.result.content.contains('#summary')? '✓': 'x'"}}
order:
    fields: mtime
    orders: desc
fileIndecator: >-
    |{{result.index}}|[[{{result.basename}}]]|{{eval "return moment.unix(data.result.mtime/1000).format('YYYY-MM-DD')"}}|{{template}}|
joinString: "\n"
decorator: |-
    Notes: {{eval "return (data.templates.match(/✓/g)).length"}}/{{eval "return data.summaries.length"}}

    | ID  | Note | ModifyTime | Done        |
    | --- | ---- | ---------- | ----------- |
    {{templates}}

Yes, you can. That's the exact reason why I created this plugin. Here is a code I'm been using, and what it looks like.是的， 你可以的。这就是我创建这个插件的确切原因。这是我一直在使用的代码，以及它的外观。
scope:
    - ReadingNotes/
matches:
    - regex: '[\S\s]+'
      template: >-
        {{eval "return data.result.content.contains('#summary')? '✓': 'x'"}}
order:
    fields: mtime
    orders: desc
fileIndecator: >-
    |{{result.index}}|[[{{result.basename}}]]|{{eval "return moment.unix(data.result.mtime/1000).format('YYYY-MM-DD')"}}|{{template}}|
joinString: "\n"
decorator: |-
    Notes: {{eval "return (data.templates.match(/✓/g)).length"}}/{{eval "return data.summaries.length"}}

    | ID  | Note | ModifyTime | Done        |
    | --- | ---- | ---------- | ----------- |
    {{templates}}

Amazing, I succeeded, just a simple learning of the regex， I can match anything I want, thank you for your help!


Add files via upload

Added new docs for site configuration administration settings. Posts, localization, file sharing and downloads, emojis, public links, users and teams, notifications.

Summary

Ticket Link

Closing in favor of adding these files to https://github.com/mattermost/docs/pull/4036.


How can I share a Golang object between different phases quickly?

I am developing a rate limiting plugin in Golang. In the plugin's logic, I need to generate a rate limiting object that is related to the current request URL during the access phase for QPS calculation. Then, in the response phase, I need to access this request-specific rate limiting object for further computation. How should I share this object between the two phases?

We encourage questions like this in the Discussions forum in the main Kong repo: https://github.com/Kong/kong/discussions.


Undefined instance variable value while using arrow functions

🐛 Bug Report
Using arrow functions in mockImplementation function the class instantiation does not work.
To Reproduce
Steps to reproduce the behavior:
let mockFunction = jest.fn().mockImplementation((a) => {
    this.temp = a;
});

Expected behavior
let mockFn = new mockFunction(6);
console.log(mockFn.temp) //this gives me undefined instead of 6

A clear and concise description of what you expected to happen.
Link to repl or repo (highly encouraged)
Please provide either a repl.it demo or a minimal repository on GitHub.
Issues without a reproduction link are likely to stall.
Run npx envinfo --preset jest
Paste the results here:
npx: installed 1 in 2.374s

  System:
    OS: macOS High Sierra 10.13.6
    CPU: x64 Intel(R) Core(TM) i5-7267U CPU @ 3.10GHz
  Binaries:
    Node: 10.4.1 - /usr/local/bin/node
    npm: 6.1.0 - /usr/local/bin/npm
  npmPackages:
    jest: ^23.5.0 => 23.5.0

Arrow functions don't have a constructor. Use regular function instead.


[Feature] Admin - Remove Contributor from Node

Proposal
Allow the admin to remove the contributor from a node.
Repercussions
No way to add them back easily, without cooperation from other users?
Based on
#5056, also uses #5047

Other than the logging issue, the feature-specific code in this PR looks good. (There are other issues with Desk, but I'll put those in the Desk PR) :mount_fuji:
Please also add a test to make sure the most recent log isn't for removing a contributor after you fix the logging issue. Back to you. @Ghalko

@mfraezz I am going to take the desk stuff out and base this off of #5013 and bring in registrations rather than slowing this PR down with desk stuff. Bad initial decision.

Made the commit fiasco much better @mfraezz and the last admin can't be removed.

Merged into release/0.66.0. :+1:


Installing robbyrussell/oh-my-zsh... Error! Activate logging and try again.

Description
Fresh install antigen, cannot install plugins. It shows
Installing robbyrussell/oh-my-zsh... Error! Activate logging and try again.

Steps to reproduce
I clone antigen repository to local:
git clone git@github.com:zsh-users/antigen.git $HOME/antigen

and with this zshrc config.
Software version
antigen version
Antigen develop (d1dd78b)
Revision date: 2018-01-15 14:37:21 -0300
zsh 5.4.2 (x86_64-ubuntu-linux-gnu)
Linux instance-2 5.0.0-1014-oracle #19-Ubuntu SMP Tue Mar 17 04:50:21 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Configuration

no .antigenrc config
.zshrc config.

More information

I have the same issue with a simpler zshrc. I theorize somewhere robbyrussell/oh-my-zsh needs to change to ohmyzsh/ohmyzsh ?
https://github.com/ohmyzsh/ohmyzsh

You can use this instead
antigen use ohmyzsh/ohmyzsh

@einverne @gee1k @jleaders I have a PR that does all of that (changes robbyrussell/oh-my-zsh -> oh-my-zsh/oh-my-zsh).

@solkaz I tested your PR on my fork but it doesn't seem to be working

@solkaz I tested your PR on my fork but it doesn't seem to be working

You can use this instead
antigen use ohmyzsh/ohmyzsh

This BREAKS the ohmyzsh plugins. For example following will stop working
antigen bundle git


Add SubtractAssetQuantity command

What is this pull request?
Add SubtractAssetQuantity command implements/tests
How to use the features provided in the pull request?

Add SubtractAssetQuantity to schema and model
Add SubtractAssetQuantityExecutor
Add Test

Sorry I miss rebase

Does git rebase origin/develop work?

@MizukiSonoko you basically can force push to your own branch after rebase, it's fine


Add some missing explanations and some minor improvements

Description
Closes #285, #273 and #372.
Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[x] Code is formatted according to CONTRIBUTING.md
[x] Self-reviewed the diff

That comment is a bit weird, since mint is user input. But we don't have other token algebras than Coin, so you cannot put anything interesting there at the moment anyway.


Flutter Clock from scratch

is it mandatory to use the flutter clock helpers or download the flutter clock repository.  Can i start from scratch?

You can start from scratch...
But its recommended to use flitter_clock_helper

if I am starting from scratch then do I have to include the flutter_helper directory in my project although I am not using it anywhere. I can see in the submission requirements it's mentioned something like this
"In your /flutter_clock folder, zip these two folders:
flutter_clock_helper (unedited)
_clock (your clock)"

You can submit one directory if you create your own clock that don't use flutter_clock_helpers

flutter_clock_helper is not required for your submission to be valid. It is there to give you access to the optional parameters, like light or dark mode. However, you can submit a project that does not use it at all.


GEODASH - Use templates or geodash conbfigurations from other projects

Normal/Beginner users will be at a loss when trying to configure GeoDash.
It would be much easier if the user is given a few options when starting the geodash configuration (as we do when starting a new Collect Earth project)
The options should be (more or less, we will need to define exactly what the pre-existing options should be)

Geodash with NDVI charts and Landsat 8, Landsat 7 and Sentinel-2 imagery

2 Geodash with Sentinel-1 and comparisons
-3 Copy geodash from another project (import geodash json configuration)
-4 Start from scratch

Thank you, this is actually in development currently and should be rolled out in the near future

This has been completed, to use at the top of the Create Project in the section:
Use Project Template (Optional)
Select the project that you would like to copy the GeoDash from and check the box: Copy Template Widgets


New branch: low-contention

This builds on my previous pull-request.
Removing unneeded barriers and optimizing with XCHG instead of CAS doesn't make things worse,
as far as I can tell.
This is just really really hard to get right, and your current HP[tid] idea might have fundamental design flaws.
With garabage collection to solve the reclamation problem, a linked list works pretty well.  But without garbage collection, it's a huge problem.  (See http://www.yebangyu.org/LockFreeProgrammingPractice.pdf for a collection of some papers, including one showing just how complicated a lock-free linked list is, if you really want to free nodes instead of just using a free list.  ABA problems make it extra hard.)
There's a reason everyone uses circular buffers like the MPMC queue analyzed in https://stackoverflow.com/questions/45907210/lock-free-progress-guarantees.  (From https://liblfds.org/).  It should be much more efficient than this linked-list thing, because it doesn't have to scan an array in the consumer.
In your version, readers contend heavily for access to that array as well as other shared data.  malloc/free probably contend across threads, too, so that's another big reason for a fixed-size buffer.

I hit assert in this branch too.
CPU: Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz
Program: bin/test_p100c10
Old version (branch FSHW) do not use any memory-barrier, just use CAS operator.
In new branch (HP), i'm trying to implement Hazard Pointers algorithm. And i found double free bug.
I use these memory-barrier because solve double free bug. And I do many research to find out why it's happened. But manual always tell me malloc/free is thread-safe.
This is why the lastest version fill with mb to prevent this bug. I know memory-barrier spend more CPU, and make program more slower. But crash is more unbearable than slower performance.
If you know how to resolve the  memory-barrier with crash, please let me know.
Thanks.

I wasn't seeing any double-free bugs, just use-after-free.  That's a separate thing.

This is why the lastest version fill with mb to prevent this bug.

But they don't prevent the bug.  My tests branch shows that just adding more checks to your code without removing any memory barriers does detect problems.
I see the assert trigger in that branch on my i7-6700k (gcc7.3 -O3).  I made sure to keep all your memory barriers and stuff you were doing in that version, and just added the poison + assert checking.
Therefore you have a bug somewhere.  I don't know where; I'm not that interested in finding it because this whole thing looks very inefficient.  It's also not in a usable state for other reasons: the free list can grow very large, like 2.5GiB memory usage for the p100c10 test.
Most of the changes in this branch are functionally equivalent (I think all of them are unless I made mistakes).  If you remove the assert checking in this version (compile with make CPPFLAGS=-DNDEBUG), the tests pass consistently, just like they did before I added the assert to your original.
Definitely replacing CAS loops with XCHG is a win, in the 2 cases I did that, and I'm sure that's safe.
This would be easier to read using C11 stdatomic; then you could see seq-cst stores vs. release stores clearly.  IDK if you want to rewrite it that way.

I remove assert and hit this crash.
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Core was generated by `./bin/test_p100c10'.
Program terminated with signal 11, Segmentation fault.
#0  free_pool (freeall=false, ctx=0x7ffe513df600) at lfq.c:45
45			if ( (!p->can_free) || (!p->free_next) || inHP(ctx, (struct lfq_node *)p) )
Missing separate debuginfos, use: debuginfo-install glibc-2.17-196.el7.x86_64
(gdb) l
40			return; // this pool free is not support multithreading.
41		volatile struct lfq_node * p;
42
43		for ( int i = 0 ; i < MAXFREE || freeall ; i++ ) {
44			p = ctx->fph;
45			if ( (!p->can_free) || (!p->free_next) || inHP(ctx, (struct lfq_node *)p) )
46				goto exit;
47			ctx->fph = p->free_next;
48			free((void *)p);
49		}
Test:
 while ./bin/test_p100c10; do echo success ; done
It happened while i running 3 hour.
Your code has better performance than me, but hit another bug.
coredump

Interesting.  I wonder if the node got freed after being added to the free list.
We know the code has use-after-free bugs, and segfaults are an obvious consequence if the whole page gets unmapped.  But this is in a different part of the code from the one the assert is checking.  Would be interesting to add an assert there, too, to look for use-after-free in that case without the whole page having to be unmapped for you to detect it.

Or possibly my idea of using a union {next, free_next} was a problem.
Hmm, maybe another thread can be looking at (or preparing to write) the next pointer when we add it to the free-list.  I don't think the producer could do that, because new nodes get added with next=NULL, and the consumer will never free the last node.


interactive publish: skip silo auth if there's a bridgy publish backlink

...since the backlink proves that they intend to publish it with bridgy. requested by @tantek.

Thinking about this. I like it, and it would help us work around eg #911.
I'm worried about one scenario, though. Say a post includes a Bridgy Publish backlink, but doesn't actually trigger the publish. Someone else (ie not that site's owner) could publish that post as that silo user at some arbitrary point in the future.
Realistically, I do know some sites (eg @jamietanna and maybe @jalcine, among others) that automatically include Bridgy Publish backlinks for some silos on many or all posts.
@tantek et al, open to thoughts on the threat model here!

(heh, this issue number)
I actually disabled automatic syndication until I supported https://github.com/indieweb/micropub-extensions/issues/4 (by way of https://git.jacky.wtf/indieweb/koype-next/issues/11) to do automatic syndication more accurately. That said, if I notice a failure on syndication, it follows an exponential retry flow (and I tend to check the logs often after I post, so I'd be able to debug).
That said, is it more of an idea/request to implicitly authenticate people if they have everything in place to publish posts?
(Originally published at: https://jacky.wtf/2022/6/YDYq)

That said, is it more of an idea/request to implicitly authenticate people if they have everything in place to publish posts?

Right, this.

Interesting! I figure some level of user intervention was required for things like Twitter or Tumblr. If not, that would be very nice to have (and would be one less step for people).
(Originally published at: https://jacky.wtf/2022/6/F65c)

Looked at this a bit. It's not easy right now, due to the way we handle the OAuth check for interactive publish requests. We do the OAuth redirect and dance first, before we fetch the source page. To do this, we'd need to add a new server-side handler before the OAuth dance that fetches the source page and checks for the backlink. Doable! But nontrivial.


Can't install php-cs-fixer through composer: Your requirements could not be resolved to an installable set of packages

$ composer require friendsofphp/php-cs-fixer --dev
Using version ^2.16 for friendsofphp/php-cs-fixer
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Restricting packages listed in "symfony/symfony" to "5.1.*"
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - friendsofphp/php-cs-fixer v2.16.0 requires symfony/console ^3.4.17 || ^4.1.6 -> no matching package found.
    - Conclusion: don't install friendsofphp/php-cs-fixer v2.16.4
    - Conclusion: don't install friendsofphp/php-cs-fixer v2.16.3
    - Conclusion: don't install friendsofphp/php-cs-fixer v2.16.2
    - Conclusion: remove composer/semver 3.2.0
    - Installation request for friendsofphp/php-cs-fixer ^2.16 -> satisfiable by friendsofphp/php-cs-fixer[v2.16.0, v2.16.1, v2.16.2, v2.16.3, v2.16.4].
    - Conclusion: don't install composer/semver 3.2.0
    - friendsofphp/php-cs-fixer v2.16.1 requires composer/semver ^1.4 -> satisfiable by composer/semver[1.4.0, 1.4.1, 1.4.2, 1.5.0, 1.5.1, 1.5.2, 1.6.0, 1.7.0].
    - Can only install one of: composer/semver[1.4.0, 3.2.0].
    - Can only install one of: composer/semver[1.4.1, 3.2.0].
    - Can only install one of: composer/semver[1.4.2, 3.2.0].
    - Can only install one of: composer/semver[1.5.0, 3.2.0].
    - Can only install one of: composer/semver[1.5.1, 3.2.0].
    - Can only install one of: composer/semver[1.5.2, 3.2.0].
    - Can only install one of: composer/semver[1.6.0, 3.2.0].
    - Can only install one of: composer/semver[1.7.0, 3.2.0].
    - Installation request for composer/semver (locked at 3.2.0) -> satisfiable by composer/semver[3.2.0].

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.
 - It's a private package and you forgot to add a custom repository to find it

Read <https://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.

Installation failed, reverting ./composer.json to its original content.

Same here. Basically friendsofphp/php-cs-fixer doesn't support symfony/console:^5.0  yet.

Basically friendsofphp/php-cs-fixer doesn't support symfony/console:^5.0 yet.

@caugner really? How did you figure this out?
The problem is with composer/semver - support for ^2 and ^3 was added and will be available with next release.

@kubawerlos https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v2.16.0/composer.json#L24

@caugner and how from that you figured the generalisation? Version 2.16.1 supports symfony/console:^5 yet you wrote:

Basically friendsofphp/php-cs-fixer doesn't support symfony/console:^5.0 yet.

This is not true - not "basically", but "only in version 2.16.0"...
In the first comment we can see composer require friendsofphp/php-cs-fixer --dev which would try from latest  release, not 2.16.0.

I ran that command in my project today and had the same error regarding symfony/console, hence my deduction. 🤷♂
Great if it works now. 👍

I strongly advise not to install PHP CS Fixer as part of your project's dependencies though. PHP CS Fixer is a tool, not a library you need to build something (unless you build some library for PHP CS Fixer). You may have e.g. a dedicated tools/php-cs-fixer/composer.json file which allows you to install it without messing with your own dependencies.
Dependencies of PHP CS Fixers should remain an internal implementation detail of the tool, not something we have to keep updated for the purpose of making it installable in every Symfony app out there.

I strongly advise not to install PHP CS Fixer as part of your project's dependencies though

Yes!
and, depends, no :}
I did for this reason use the phar, so no clashes.
But, if you want to use custom rules from external packages, this approach doesn't work, see https://github.com/kubawerlos/php-cs-fixer-custom-fixers/issues/368 😢

if you want to use custom rules from external packages, this approach doesn't work

This is why I keep using Composer. I had similar issues with PhpStan and extensions. Using Composer to install each tool in its own isolated place works pretty well for me.

@caugner and how from that you figured the generalisation? Version 2.16.1 supports symfony/console:^5 yet you wrote:

Basically friendsofphp/php-cs-fixer doesn't support symfony/console:^5.0 yet.

This is not true - not "basically", but "only in version 2.16.0"...
In the first comment we can see composer require friendsofphp/php-cs-fixer --dev which would try from latest release, not 2.16.0.

I am not sure if that is correct. I tried installing the version 2.16.0 but that threw me the same error:
Enter the version constraint to require (or leave blank to use the latest version): 2.16.0
Search for a package:
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Restricting packages listed in "symfony/symfony" to "5.1.*"
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for friendsofphp/php-cs-fixer 2.16.0 -> satisfiable by friendsofphp/php-cs-fixer[v2.16.0].
    - friendsofphp/php-cs-fixer v2.16.0 requires symfony/console ^3.4.17 || ^4.1.6 -> no matching package found.

Can you try installing 2.16.1 and not 2.16.0? Or maybe more easy is to pick the constraint ^2.16


receipt_old view unneeded?

Hello, it looks like the laravel-cashier-mollie/resources/views/receipt_old.blade.php isn’t needed and has sneaked in by mistake.

Yes, I remove this file from the package.
https://github.com/mollie/laravel-cashier-mollie/pull/9


Add config to OpenSuse Leap 15.3

Please can you add config to distro OpenSuse Leap 15.3

I am sorry but support for niche distributions with small user bases is usually not worth it for me in terms of the amount of time required for maintenance.


print

somehow no error and nothing print on the screen using the readme example

For me it works. Can you post your python version, and requests lib version?


HashMap in Java

Can i work on this issue under hacktoberfest2022?

@RaniK27 Could you assign to me? Thank you :)

I am good with Hashmap in Java . Can you please assign me this issue .


Way to create pages based on FrontMatters in other files?

I'm wondering how one would go about implementing the generation of pages based on FrontMatters that are located within other files with Still.
As a real-world example, my Jekyll page has blog posts, each has a category defined. Jekyll does some magic and generates one page for each category, containing the list of blog posts with this category. I'm moving my blog and would like to keep these category pages.
My current idea would be to have a preprocessor for eex files (I'm using eex instead of slime), which adds all blog posts into a list (kinda like the collections for tags are built). From there, using a hook I could add a compiler that takes this list and creates category pages.
Is this the right approach? Is there something I'm missing?

Hello 👋
I think you're close, but that solution would only allow you to have one page for all categories. Unless you manually created a page for each category. This is because each input file can only generate one output file. So to have multiple "category" pages (one per category) on the output, you would need multiple "category" pages on the input. But, for reasons not related to having read your issue, I just finished the first step to allow for scenarios precisely like the one you just described 😎 https://github.com/still-ex/still/pull/176 The last step is to implement something like 11ty's pagination (it's not pagination as we usually think of it). Can you hold just for a bit? I'll finish it in a couple of days

I see #176 is done. Does this mean it's usable? If it is, I'd volunteer to add some documentation as soon as I was able to understand the code as I couldn't find any docs in the PR?

Hey @Sgoettschkes sorry I was on vacation and then catching up with work. https://github.com/still-ex/still/pull/176/files is one of the core changes to allow for pages based on frontmatter. I'm working on it here, would you have a look? I think functionality is working, and documentation is ongoing but I still need to write a better example and tests. I could use some feedback

No worries, I didn't have time to work on this as well ;) I'll have a look and give feedback.


Bio: Macromolecule sem type detectors: false positives

To reproduce:

open SPGI dataset

Expected: Cblood nM column is string
Actual: Cblood nM column gets detected as a macromolecule
Here's a sample cell value:
54.0

43.3

90.9

687.0```

In detector tests, perform multiple attempts on the same sample data (due to the non-deterministic nature of detectors).


List->Members->Events

This feature is missing https://mailchimp.com/developer/reference/lists/list-members/list-member-events/
Should be able to do:
$mailchimp     ->lists('1a2b3c4d')     ->members('a1167f5be2df7113beb69c95ebcdb2fd')    ->events(['name' => 'your_event_name'])

@Jhut89  Thanks. I  Added pull request #77 to address this.

👏 👏 👏

You should be good. Your PR is merged and a minor release was made once unit tests passed. You should be able to pull the update library through composer version 3.2.1

Great. Thank you.


Fix issue #18

I don't think that we need all these checks here.
This seems to resolve #18 .

It not only support call between php and php,I want to support php and golang, and so on.

I made a mistake，is TraceIDFromString not SpanIDFromString

I fix this problem.you can chenckout branch fix_traceid_len to test.

Yes it works as expected now. Thanks!

Done in PR 20


Renders QR code into String

I wanna print QR code in terminal.
let code = QrCode::new(b"01234567").unwrap();
let s = code.render::<Luma<u8>>()
    .to_string("\x1b[49m  \x1b[0m", "\x1b[7m  \x1b[0m");
println!("{}", s)

Oh my God! It works.

Coverage decreased (-75.6%) to 17.886% when pulling 74a048285732d30398bdb59a6e37a67fe33f43d6 on ARwMq9b6:master into efc390f685eca2a16f79c74acd8d6073701d7748 on kennytm:master.

Coverage decreased (-75.6%) to 17.886% when pulling 74a048285732d30398bdb59a6e37a67fe33f43d6 on ARwMq9b6:master into efc390f685eca2a16f79c74acd8d6073701d7748 on kennytm:master.

In 0.4 it is implemented as
let code = QrCode::new(b"01234567").unwrap();
let s = code.render().light_color("\x1b[7m  \x1b[0m").dark_color("\x1b[49m  \x1b[0m").build();
println!("{}", s);


RFM - duckietv

I am mostly stepping away from chocolatey as I do not really use it anymore and don't really have time to upkeep my packages (fixing issues or checking and seeing if everything is still working as it should be) I have been maintaining for a good while now, so I am currently on the lookout for someone to take over pretty much all my packages (I'll be creating an issue for each package).

Current Maintainer

[x] I am the maintainer of the package and wish to pass it to someone else;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://community.chocolatey.org/packages/duckietv
Package source URL: https://github.com/JourneyOver/chocolatey-packages/tree/5046f4d352f51a07a95bd40762275339949f6b81/automatic/duckietv

Hello,
I'll take this package.
It ready on my git.
Can you add me as maintainer ? (tunisiano)
Thank you
Have a nice day

@tunisiano187 I have added you as a maintainer (that you need to accept), thanks for taking over the package.

Hello,
That have passed the process (it's a nightly build so no need to be moderated)
So this issue can be closed as complete
Have a nice day


Radial menu position incorrect when aspect ratio != 4:3

It is related to this change
Happens in 3/4 players MP or 2 players MP with vertical split.

The HUD aspect ratio correction isn't meant to work in split screen yet, looks like I forgot some checks for that. Fixed I think.

Yes is correct now.
Not sure it's related, but the in-game window has a similar problem:

It is related, but not quite as much of a problem, so I haven't fixed it.


Missing query object while redirect by locale

🐛 Bug Report
When load url with another locale, do redirect. For example:
http://localhost:3000/en/oauth2/connect?foo=foo
redirects to http://localhost:3000/oauth2/connect?foo=foo (if locale equal defaultLocale)
But in this case next-js router missing query params.

Locale ru contains
query: {
    "foo": "foo"
}

But locale en after redirect has empty query object even asPath = "/oauth2/connect?foo=foo"
To Reproduce

Set default locale.
Create url with search params
Go url with different locale

// Paste your code here

import { useRouter } from 'next/router';

export const App: React.FC = () => {

  const router = useRouter();
  console.log(`locale ${router.locale}`, router);

  return null;
}

Expected behavior
react.query before and after redirect is equals.
Your Environment

<EMAIL_ADDRESS>

I'm not sure this is an issue of next-i18next, but it may be an issue of next.js.
Anyway, please provide a reproducible repository or codesandbox example.

Hi there,
Assuming that you're using router.push(), an you try not enabling shallow routing ?
More info in the link: https://nextjs.org/docs/routing/shallow-routing#caveats

not stale


sbt-scripted fails when using sbt 1.0.1 and coursier

I'm running:

sbt 1.0.1
coursier 1.0.0-RC11

When using sbt-scripted, I systematically get the following failure:
info] Error during sbt execution: Could not finder sbt launch configuration.  Searched classpath for:
[info] 	/sbt.boot.propertiesUnspecified
[info] 	/sbt/sbt.boot.propertiesUnspecified
[info] 	/sbt.boot.properties
[info] 	/sbt/sbt.boot.properties

When I remove coursier from my list of global plugins, everything seems to behave as it should.
Here's a repository containing the smallest reproduction I could make, and there's the travis log that shows the same failure.
By contrast, this is the travis log when coursier is not enabled.

Adding the following lines to build.sbt solved the problem for me. There should be a (probably quite ad hoc) way to have sbt-coursier generate update reports that don't require that workaround, but for now, one has to add these lines.

Jesus, SBT, why do you have to make it so hard to love you...

Any more info on what the root cause is, @alexarchambault?

@dwijnand The default implementation of sbtLauncher makes assumptions about how the order of things in the update report - in particular, it seems to expect the jar of org.scala-sbt:sbt-launch upfront. sbt-coursier doesn't provide guarantees about the order of things in the update report (for now, at least).

That should be fine in 1.0.0-RC12 (with or without the workaround). Re-open if ever you still run into this with it.

Confirmed, at least for my user case.