Add translation option using https://hosted.weblate.org/

I started to translate, but it is very uncomfortable to translate in the current situation.
Can you add your project to this site? It is very convenient to translate like that!
hosted.weblate.org

I started to translate, but it is very uncomfortable to translate in the current situation. Can you add your project to this site? It is very convenient to translate like that! hosted.weblate.org

Thanks for your suggestion, I will spare some time to try to add the translated content to this website.

I started to translate, but it is very uncomfortable to translate in the current situation. Can you add your project to this site? It is very convenient to translate like that! hosted.weblate.org

Recently I tried a new idea to change the resw resource file to a rejson resource file. Compared to resw resource files, rejson resource files look more intuitive and can be manipulated in lightweight compilers such as Notepad, vscode, etc.
Online translation site https://hosted.weblate.org/ I'm sorry I didn't figure out how it works.
If you understand how the site works, you can participate in the project and propose your implementation, which is very much appreciated.
---------------------------------
最近我尝试了一种新的想法，将 resw 资源文件改为 rejson 资源文件。相比于 resw 资源文件，rejson 资源文件看起来更为直观，可以在轻量化的编译器中进行操作，比如 Notepad，vscode 等等。
在线翻译网站 https://hosted.weblate.org/ 我没有搞清它是怎么操作的，很抱歉。
如果您了解该网站是如何操作的，您可以参与到该项目中，并提出您的具体实行方案，非常感谢。
效果图

@Gaoyifei1011 It seems that you have already started a new venture. Want to switch to email correspondence? I have no experience, but it seems that in the "Create a component" window, you need to click on "Start from scratch", and upload a ZIP file.

Thanks for the suggestion! I have some other things to attend to during this time, so I'm sorry I couldn't check your response in time. After reconsideration, I decided that I would prefer project contributors to leave traces of their contributions in my project. In addition, since I am not very familiar with the use of an online translation hosting platform, I do not plan to host the translated content of my project to this platform for the time being.
Previous .resw files required tools and plug-ins such as Visual Studio or Visual Studio Code to provide a more user-friendly interface, but were not very convenient to use for users who did not have these tools. Therefore, I decided to switch the translation format of my project to a more convenient .resjson format, so that you can translate directly in a text editor such as Notepad.
The general availability of .NET 9 is expected this week, and updates to other related toolchains, such as the Windows App SDK and WinUI 2, may also occur. After the official release of these tools, I plan to update the project's dependencies and release a new version. In addition, this new version will also include some fixes that I have made during this time, which will be mentioned in the release notes. If you are still interested in the project, welcome to submit your translation this week, and thank you again for your support!
-----------------------------------------
感谢您的建议！这段时间我有些其他的事情需要处理，因此未能及时查看您的回复，感到抱歉。经过重新考虑，我决定更希望项目贡献者能够在我的项目中留下他们的贡献痕迹。此外，由于我不太熟悉在线翻译托管平台的使用，因此暂时不打算将项目的翻译内容托管到该平台。
之前的 .resw 文件需要 Visual Studio 或 Visual Studio Code 等工具和插件才能提供更为友好的界面，然而对于没有这些工具的用户来说，使用起来并不十分方便。因此，我决定将项目的翻译格式切换为更为简便的 .resjson 格式，这样您可以直接在记事本等文本编辑器中进行翻译。
预计这周 .NET 9 将正式发布，其他相关工具链，如 Windows App SDK 和 WinUI 2，也可能会有更新。在这些工具正式发布后，我计划更新项目的依赖并发布新版本。此外，这个新版本还将包含我这段时间做的一些修复，具体更新内容将在发布说明中提及。如果您仍然对项目感兴趣，欢迎在本周提交您的翻译内容，再次感谢您对项目的支持！


SheetJS plugin

It would allow to load data from any spreadsheet document.
This is a demo with Lume in the website: https://docs.sheetjs.com/docs/demos/content/#lume-demo

Yes, data loaders are independent of the template engine.

Done in a2e5680b2ff1f02949bd0364788b55159befd2c6


Any docs about katashared filesystem?

Which feature do you think can be improved?
Want a detail doc about katashared filesystem.

Hi @YongHaoWu - could you provide us a few more details? Are you meaning how Kata maps mounts and volumes from the host into the guest for instance? The only specific mention of katashared in the code base is in the kata agent code that handles those mounts I believe.
I think I agree though, a nice short-ish document explaining and showing how the different container volume/mount types are handled and mapped into kata would be nice. I don't think we currently have that info in one clear place.

@grahamwhaley  Aha, I see. Yeah, I want to know how the different container volume/mount types are handled and mapped into kata .
Thanks.


SSE game session operation

Using SSE to let users play the game and can quit during the game

done with 0d4e464


[BUG] getting_started requirements.txt

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Create new python 3.10 conda environment
pip install archai
Try and run the getting-started notebooks

Expected behavior
should work
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [Windows 11]
Virtual Environment [conda,]
Python Version [3.10.1]

Additional context
Add any other context about the problem here.

never mind, my pip install archai didn't work properly.


Comment icon in PR tree has black outline

Repro:
Make a comment and view the PR tree

cc @misolori

Couldn't find one so created https://github.com/microsoft/vscode/issues/135591

I'll switch over to the proposed codicon support, though that means we'll only show one decoration.


Frozen app with pyinstaller  failes with schwifty

I created issue at pyinstaller where is also description how to reproduce the problem. As part of solution there is suggestion for schwifty:

Alternatively, you could modify schwifty\registry.py to use regular os functions (listdir, etc.) instead of resource_listdir and resource_filename.

I don't think that I will do something in that regard, especially since this problem is going to be fixed in an upcoming release of pyinstaller (after this pull-request is merged).

Just checked with pyinstaller 6.5 and the issue is still there.


server: improve initial peer bootstrapping

In this commit, we address an existing issue with regards to the inital
peer bootstrapping stage. At times, the bootstrappers can be unreliable
by providing addresses for peers that no longer exist/are currently
offline. This would lead to nodes quickly entering an exponential
backoff method used to maintain a minimum target of peers without first
achieving said target.
We address this by separating the peer bootstrapper into two stages: the
initial peer bootstrapping and maintaining a target set of nodes to
maintain an up-to-date view of the network. The initial peer
bootstrapping stage has been made aggressive in order to provide such
view of the network as quickly as possible. Once done, we continue on
with the existing exponential backoff method responsible for maintaining
a target set of nodes.
We also randomize the order of the different bootstrappers to prevent
from always querying potentially unreliable bootstrappers first.

Needs a rebase to fix conflicts as well, also we can manually implement the shuffle using rand.Perm.

Rebased.


releases aren't really releases

For some reason I cannot list releases on this repo.  API calls like https://docs.github.com/en/rest/reference/repos#get-the-latest-release fail.  I can list tags, but not releases.  Maybe they are not really releases somehow?  I'm writing something to make sure I keep the latest release in my yaml for this :)

I think you are right, the latest release is not actually marked as latest according to GitHub.
I have opened an issue https://github.com/pnikosis/semtag/issues/11 in semtag repository to track what can be done there.

@antonbabenko I'm building a replacement using go https://github.com/1efty/semtag
It's not feature complete yet, but hoping to get there shortly. Maybe this request could be incorporated into v1.

@1efty I don't think I am eager to replace working solution with existing semtag to anything else in the near future.

@grimm26 I think you just need to use pre-commit autoupdate command, not try to write your own wrappers

@MaxymVlasov yes i'm doing that now


Add use case notebook

Add a use case notebook explaining different queries and diagnostic workflows

@scollis when you get a chance, can you take a look here?


Created misc folder, and created my own problem.

This problem is called HiveMaker, and I have suddenly thought of it. I shall still create solution.py

By the way, the reason why there is no solution.py is that I cannot put it anywhere. I shall wait for it. Thank you!


CORS Issue with the Assets

I'm not sure if this is the right place to report this, but i'm getting CORS issues whether it be from the playground, or from any side-app that uses Babylon.JS

CORS are being enabled, will take a few minutes. Thanks!

That was fast, thank you @RaananW !


Use classes instead of objects

Coverage decreased (-0.06%) to 88.534% when pulling 1f3c7783c8c3f0fb6e8869d7beeed572d6d8c486 on jansav:v2.0.0 into ae8ea0d93bf4dfbc572b00fae23d5191b99eeb0f on Houston-Inc:v2.0.0-beta.


Apple Logo looks squished

Describe the bug
The Apple Logo ASCII art looks weirdly compacted and squished.
To Reproduce
Steps to reproduce the behavior:

Use macchina on a mac.

Expected behavior
It has a normal aspect ratio.
Screenshots

macchina --doctor output
We've collected a total of 19 readouts including 2 failed read(s) and 1 read(s) resulting in a warning.

Readout "LocalIP" failed with message: Unable to get local IPv4 address.
Readout "GPU" failed with message: This metric is not available on this platform or is not yet implemented by libmacchina.

1 of the 3 unsuccessful read(s) resulted in a warning:
 Readout "Distribution" threw a warning with message: Since you're on macOS, there is no distribution to be read from the system.

I don't see any squishiness...

Maybe something like this would be better:

Feel free to make a pull request, the relevant section is the following:
https://github.com/Macchina-CLI/macchina/blob/e7da33025060b7b6536d8991474b3608f8936b90/src/ascii.rs#L86-L105

gonna try!


[minor] Updated the readme for readability

Math rendering
The math rendering in github is supported by mathjax. Updated to support that.
The reason it was not working before (given the comment) is that the mathjax does not expect blank lines after and before $ or $$.
Link to fork
Original link was referring to the master branch of the parent repository. This is misleading, as current repository is behind, and is not compatible with the master branch anymore. In addition to that, the default branch name might change, which would cause a broken link.
Developer guide
Restructured the developer guide to be more readable. This will eventually need to be removed to CONTRIBUTING.md file.

Thank you for your contribution @z-a-f feel free to reach-out if you plan to use DP-XGBoost.


chore/update-dependencies

I've updated some dependencies.
Tests seem to pass, but I'm not sure about snapshots. They seem different, added html classes have changed.
Otherwise I've tested on my Gatsby website. It did work. Probably requires more testing to be sure.

Thanks for the pull request. Will check and merge the pull request if everything works fine :)

Let me know if any changes are needed, I'll be happy to help.


Publishing a boolean variable from Unity to Ros topic

I have a question!

[x] I searched the Wiki, open and closed issues for an answer. I tried my best to find the answer by myself without success. I believe that the discussion we will have in this issue, and the solutions we might find, will help me, and likely other community members who have a similar problem.

Here is my question:
I am quite new in unity. I need to synchronize my recording video on Unity with the data frame analysis in ROS with RosConnector. I thought to create a boolean variable to publish in a ROS topic for notifying to my nodes that my recording in unity it stared. It may be shameful but I can't understand how to do it. I tried to create a C# script to handle the publishing but i don't know how to get my boolean variable. I'm sorry if i did't understand something it would be obvious in the wiki.
I attached here the script that i wrote:
namespace RosSharp.RosBridgeClient
{
public class Status : UnityPublisher<MessageTypes.Std.Bool>
{
public bool messageData;
    private MessageTypes.Std.Bool message;

    protected override void Start()
    {
        base.Start();
        InitializeMessage();
    }

    private void InitializeMessage()
    {
        message = new MessageTypes.Std.Bool
        {
            data = messageData
        };
    }

    private void Update()
    {
        message.data = messageData;
        Publish(message);
    }
}

}

Hi @andreb-88 ,
without having tested it, your class looks correct. All you have to do now is to set your 'messageData ' boolean value.
Since 'messageData' is a public field you can set directly via the unity inspector, or from any other method in this class or some other.

Thank you! It works. I attached the script if it helps someone else.
using System.Collections;
using UnityEngine;
namespace RosSharp.RosBridgeClient
{
public class DroneArmedPublisher : UnityPublisher<MessageTypes.Std.Bool>
{
private bool status;
private MessageTypes.Std.Bool message;
    protected override void Start()
    {
        base.Start();
        InitializeMessage();
    }

    private void FixedUpdate()
    {
        status = UDPControl.GetDroneArmed();
        //Debug.Log(status);
        UpdateMessage();
    }

    private void InitializeMessage()
    {
        message = new MessageTypes.Std.Bool
        {
            data = status
        };
    }

    private void UpdateMessage()
    {
        message.data = status;
        Publish(message);
    }
}

}


Improve errors when getting images that don't exist in the DB

When getting an image that doesn't exist from the data/images/{id} endpoint in the main app, we get the error
...rows are closed...

We should probably improve this error.

No longer necessary with #79


Guess GPU type on DX11 where it cannot be easily discovered

Partially addresses #3489
From what I can tell there's no way to discover this via the DX11 APIs, so guess based on the vendor and device IDs. This can be expanded on later, but initially I've just set it to guess "integrated" for Intel GPUs and "discrete" for everything else, which probably covers >90% of the cases where this applies...

Added 👍

Updated

Thanks!
bors r+

@Diggsey please make the same PR against master, just exclude commit that had a version bump and changelog updates


Add installation source data

Use the new agent_installation_source and add new graph:

(Using dummy data)

@CDimonaco @fabriziosestito @rtorrero This has been unblocked by https://github.com/trento-project/web/pull/728
Could you have a short check so we can definitely close this agent installation source telemetry thingy?


#19046 Support user-defined keybinding for pinning/unpinning active results tab

Also please add copyright header to the created files.

#19046

Verified


Build receipts

Depends on #1578

We can drop the last commit and merge directly to main. Thoughts?

We can drop the last commit and merge directly to main. Thoughts?

That works too


Add vLLM provider support

This PR contains:

[x] New features
[ ] Changes to dev-tools e.g. CI config / github tooling
[x] Docs
[ ] Bug fixes
[ ] Code refactor

This PR adds support for using vLLM to run local model inference as an alternative to the huggingface backend. The main benefit of vLLM is that it is much faster than huggingface and automatically manages GPU memory so you don't have to find the optimal batch size. Here is a small example where vLLM is 5x faster than huggingface on an A100 (for larger datasets I've found vLLM can be around 30x faster):
inspect eval examples/popularity.py --model hf/meta-llama/Meta-Llama-3-8B-Instruct -M device='cuda:0'
inspect eval examples/popularity.py --model vllm/meta-llama/Meta-Llama-3-8B-Instruct -M device=0

The other benefits is that it has support for some of the parameters in GenerateConfig that huggingface doesn't support such as best_of, frequency_penalty, presence_penalty, and num_choices.
Another thing to consider is that vLLM is in the process of implementing tool call/guided generation capabilities into the library so once that is released it will be very easy to add function-calling support for local models (I have my own personal implementation for local models, not included in this PR, but it relies on the model getting the syntax right which isn't always reliable).
Feedback is very appreciated!

Hi @kaifronsdal, this PR looks fantastic, thanks so much for contributing it!
A couple of follow up items:

There are a few failing MyPy, Ruff, etc. checks on the PR, could you resolve those?

I think we should add an additional model provider that provides support for the vLLM OpenAI compatible server (https://docs.vllm.ai/en/latest/getting_started/quickstart.html#openai-compatible-server). This would be something like the TogetherAI provider (which just derives from OpenAIAPI): https://github.com/UKGovernmentBEIS/inspect_ai/blob/main/src/inspect_ai/model/_providers/together.py).  We could call this provider e.g. vllm-server.   Alternatively, we could have the vllm provider itself delegate to OpenAI when e.g. the model_base_url parameter is provided (that approach might be a bit cleaner).  What do you think?

Great to hear that tool use is coming soon to vllm! (hopefully they'll also implement this for the server endpoint). Do you know whether they have plans to add multi-modal support (rumor is that Lllma 3 405B will be multimodal).

Think I fixed them. Turns out the workflows weren't running on my fork by default.
I think having the option for vllm-server is a great idea. I don't have a strong preference for how to do it though. The easiest solution would be to just use vllm's OpenAI compatible server and run the server manually, but perhaps it might be nice to have a seperate provider implementation for completeness (I think vLLM tries to match OpenAI's format exactly so it might be even simpler than the TogetherAI implementation).
They already have experimental support for VLMs!

This worked for me:
To start the vLLM server:
export CUDA_VISIBLE_DEVICES=0
python -m vllm.entrypoints.openai.api_server --model meta-llama/Meta-Llama-3-8B-Instruct --dtype auto --api-key token-abc123 --host <IP_ADDRESS> --port 8080

To use with inspect:
export OPENAI_API_KEY=token-abc123
inspect eval examples/popularity.py --model openai/meta-llama/Meta-Llama-3-8B-Instruct --model-base-url http://localhost:8080/v1


[DOC] fix import errors of the code in "user guide"

Description
(Brief description on what this PR is about)
Fixed several import errors of the code presented in the docs/wiki/2_user_guide/Train-Model.md file
Checklist

[ x ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

cc @awslabs/raf-reviewer

Thanks @zarzen


When roaming status is true, is it mandatory to return both countryCode and countryName?

Context is the response object for POST /roaming v0.5.0:
    RoamingStatusResponse:
      type: object
      required:
        - roaming
      properties:
        roaming:
          $ref: "#/components/schemas/ActiveRoaming"
        countryCode:
          $ref: "#/components/schemas/CountryCode"
        countryName:
          $ref: "#/components/schemas/CountryName"

When roaming is returned as true, is it mandatory to also return both countryCode and countryName?
Could an operator choose to omit countryCode and countryName for privacy reasons?

Hi @trehman-gsma , as per specification , only 'roaming' (true or false) field  is required and other two (countryCode & countryName) are optional.

Hi @sachinvodafone,
The in-line documentation in the spec currently states: when the device is in roaming situation, visited country information is send back in the response.
Some may imply this to mean that country information has to be returned when the device is roaming. Do you think the doc needs updating?

Hi @trehman-gsma  Thank you for pointing out the oversight. As per the documentation, it makes sense to include countryName and countryCode in the response when the device is in roaming. I can't recall any privacy usecase where the API server should omit these values. In such situations (if any), the API response should adhere to the specific "purpose of accessing the data," outlined in the “IdentityandConsentManagement” documentation. This might involve defining two different scopes—one to send only the roaming status and the other to send all three attributes—or alternatively, defining two separate endpoints. However, it's essential to highlight and assess these use cases (if any) to determine their relevance, and real-world applicability to make any change in our current design/specification. So as per my view , there's no need to make any changes at this point, considering that the API should consistently sends countryName and countryCode when the device is in roaming.

This issue was discussed during DeviceStatus meeting on 13-03-24. It was agreed that country information is optional in cases of roaming=true. PR #127 created to update documentation.

Hello @sachinvodafone @bigludo7 @akoshunyadi,
I have a related question on top of what was discussed during the DeviceStatus meeting on 13-03-24. I can create a new issue if required.
When roaming status is false, can country information also be optionally returned?

Hello @trehman-gsma
Looking in the API documentation:

Country : Country code and name - visited country information, provided if the device is in roaming situation.

So I do not expect country in not roaming situation but this is perhaps unfair to deny conformance.
The major point for me is that country information is optional (even for roaming situation) and we need to be very clear to the API consumer that this API cannot be considered as a sure thing to get the country where the device is.


Reload command not working?

When I change the Welcome message in the config save it then reload in game with the reload command it does not show the new msg it still shows the old message. So I don't think the reload command is working as intended.
To Reproduce
Steps to reproduce the behavior:

Change the config print out.
Save the config
Use reload command in game
old message still shows and reload did not work.

Expected behavior
It should use the new config settings after a reload and not the old settings anymore.
Reload command bug only seems to be affecting the login msg. All other events work fine with a reload besides the login msg that one takes a reload to take effect.

I also seems to be placing a 'A' where every color code is used.
For instance I have color code "§6 BlahBlah" and it will print "A BlahBlah" and BlahBlah will be gold like it should. So not sure why it is putting an A.

Thanks for reporting this bug! I'll fix it as soon as possible

I have fixed the reload command. The A may be an encoding issue, does your config file use UTF-8?

Hmm I am not to sure what it is using to be honest. I am just using whatever file it made for me. I open it with Notepad++

Does it appear if you change the § with \u00a7

I will give it a try, I know I did try & also and that was the same also. But have not yet had a chance to try the code you sent.


uses old dovecot stats module

This exporter relies on the old style dovecot stats module (https://wiki2.dovecot.org/Statistics/Old), which has been replaced with a new (built-in) system as of version 2.3 (https://wiki2.dovecot.org/Statistics)

Thanks for your issue.
Feel free to send a pull request, unfortunately we are currently not working on this.

I never got it working either way, with the old or new stats modules. Always received a message roughly like this:
dovecot_exporter[13321]: 2018/10/16 16:37:37 Failed to scrape socket: Input does not provide any columns```

@ageis does sudo doveadm stats dump user give the right output?

I'm receiving the same output as @ageis
Is there any fix for that?
2.2.36 (1f10bfa63)```

Thanks

Same error than @ageis for dovecot 2.3.4

@L0ric0
How did you got it to work ?
I've been trying to get it to work but it just wont export the statistics.

FWIW, it looks like dovecot gets native support for prometheus-style metrics soon: https://github.com/dovecot/core/commit/d22a32424b7646e8a7a42987f8e58958323031ee

@L0ric0
How did you got it to work ?
I've been trying to get it to work but it just wont export the statistics.

in conf.d/10-old-stats.conf
service old-stats {
    fifo_listener old-stats-mail {
        user = vmail
        mode = 0600
    }
}

plugin {
    old_stats_refresh = 30 secs
    old_stats_track_cmds = yes
}

i think that was all, as it was some time ago i don't know if there was more

For anyone else coming across this, it is officially supported in dovecot from version 2.3.11 https://doc.dovecot.org/configuration_manual/stats/openmetrics/
Can't install it on focal, but I got it working on hirsute. It's very simple to configure now.


Introduce ImageProvider class

The new ImageProvider class allows images in QML to be loaded using QPixmaps.
The last commit is a demo for our fellow designers.
For testing this PR one can run bitcoin-qt with different values of the -chain=... command-line option.

However, I wonder if we want to instead ship with an SVG for each chain?

Does using SVG make ImageProvider unneeded? How about caching?

Something like Image { source: "qrc:/mainnet.svg" } should work without custom image provider. What cache?

Let's discuss whether we want to use SVG. Qt support is detailed at https://doc.qt.io/qt-5/svgrendering.html.

Something like Image { source: "qrc:/mainnet.svg" } should work without custom image provider.

It should work for any image, even non-svg.

What cache?

https://doc.qt.io/qt-5.12/qquickimageprovider.html#image-caching

@Bosch-0 has an opinion on the usage of SVG, per Icon Policy Documentation:

"Both an icon source file, in Scalable Vector Graphics (SVG) format,
and an optimized production file, in Portable Network Graphics (PNG) format"

"SVGs are used as source files because they can scale while retaining image quality."

Being base open OpenGL and hardware acceleration, Qt Quick could have a native support for SVG images without any performance penalties. Unfortunately, I have no enough expertise in this area for now.
The only drawback is introducing a new Qt SVG module.

Rebased after the recent sync.


added custom help command

this closes #6

Add the ability for the syntax of the command to be shown when the help command is initiated. Also Beautify Embed, ;)


move flake8 to after

If we do the check after, it will still fail the test suite... but you'll get to see all the tests that fail as well, instead of just a flake8 failure.

I'm not against changing it, but it was done by design, precisely to avoid an useless full test run.
If you do the check after, you need to wait for a full test run, find out there were flake8 errors, fix them, and re-run the full test, that's wasteful and takes a lot more time.


Bug fix in FFTDF and molecular DFMP2

This pullreq fixes two bugs:

loop() in class 'pbc.df.fft.FFTDF' raises a shape mismatch issue when self.blockdim is smaller than the block size defined by blksize (or ngrids). Fixed by consistently using blksize.

The molecular DFMP2 kernel gives incorrect energy when loop_ao2mo() yields more than one chunk of qov, e.g. when FFTDF is used as the DF type or when a small blksize is fed into with_df.loop(blksize=blksize). The MP2 energy requires products of 4-center integrals, like <ij|ab>* <ij|ab>, which means we need double sum of auxiliary index over each integral. This can't be accomplished by a single-layer loop over chunks of Q (as in b^Q_{ia}). So a simple solution is to build the full Lov tensor first.

Codecov Report

Merging #555 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #555   +/-   ##
=======================================
  Coverage   82.29%   82.29%
=======================================
  Files         574      574
  Lines      109978   109983    +5
=======================================
+ Hits        90509    90514    +5
  Misses      19469    19469

Impacted Files
Coverage Δ

pyscf/mp/dfmp2.py
68.10% <100.00%> (+1.43%)
:arrow_up:

pyscf/pbc/df/fft.py
88.28% <100.00%> (ø)

Continue to review full report at Codecov.

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


Reducing the resource usage of readWAVECAR() and ReciprocalSpaceWavefunction{D,T}

When readWAVECAR() is called, it suffers from very high memory allocation, as reported by BenchmarkTools.@benchmark (run on a Surface Pro 7):
julia> @benchmark readWAVECAR()
BenchmarkTools.Trial: 1 sample with 1 evaluation.
 Single result which took 36.330 s (17.70% GC) to evaluate,
 with a memory estimate of 25.12 GiB, over 584372609 allocations.

This may not be as a big an issue as the amount of memory used to store the resulting ReciprocalSpaceWavefunction{3,Float32}:
julia> data = readWAVECAR()
...
julia> varinfo(r"data")
  name        size summary
  –––– ––––––––––– ––––––––––––––––––––––––––––––––––
  data 547.466 MiB ReciprocalWavefunction{3, Float32}

The size of the WAVECAR used as input is 150 MB, so clearly a good amount of space is being wasted with the ReciprocalWavefunction{3,Float32} struct. The question is what is wasting the space. One possibility is that HKLData{D,T} is wasteful, because not all of the HKL indices are being assigned data. If that's the case, it might be wise to change the data field to a Dict{NTuple{Int64}, T} to prevent memory from being cluttered with useless zeros. We could also use a similar strategy to what WAVECAR files use (implicit indexing based on whether the reciprocal space vectors have a magnitude below the energy cutoff) but I think this is probably too complicated/not worth the savings.
On the other hand, the issue might be that the ReciprocalWavefunction{D,T} struct is wasteful in storing data in Vector{Vector{HKLData{D,T}}} structs. It might be better use Matrix{HKLData{D,T}} instead, with matrix dimensions nband * nkpt.

In pull request #42 I altered the field to be a Matrix{HKLData{3,T}} but this does not appear to have a significant effect on the size of the wavefunction that's read in.

I'm going to try replacing HKLData with HKLDict, which uses a Dict instead of an Array for backing. For wavefunction files that are usually specified to some energy cutoff, effectively only an ellipsoid bounded by the maximum extent of the G-vectors may contain non-zero values - so only about half of an Array is actually being used.

Well, would you look at that. Here's the result of readWAVECAR() when using an HKLData:
julia> varinfo(r"ans")
  name        size summary
  –––– ––––––––––– ––––––––––––––––––––––––––––––––––
  ans  547.466 MiB ReciprocalWavefunction{3, Float32}

And here's the result from the new HKLDict:
julia> varinfo(r"ans")
  name      size summary
  –––– ––––––––– ––––––––––––––––––––––––––––––––––
  ans  1.481 GiB ReciprocalWavefunction{3, Float32}

Looks like HKLData is still more efficient in terms of space! I didn't try timing the two, but there didn't seem to be a significant difference in the runtime.

I think with the new PlanewaveWavefunction type this is far less of an issue and can be closed.


Support for extensions based on PerpetualCache

I would like to make a simple extension based on PerpetualCache, but I not want to lose the default decorators.
It can be implemented in this way?

I should implementation by decorator.


Decimal precision truncates to 2 if decimal is nullable

Moving this issue from https://github.com/aspnet/EntityFramework/issues/7775
Model:
public class Site
{
    public decimal? Lon { get; set; }
    public decimal? Lat { get; set; }
}

DbContext:
public class MyDbContext: DbContext
{
    public DbSet<Site> Sites { get; set; }
    protected override void OnModelCreating(DbModelBuilder modelBuilder)
    {
        modelBuilder.Entity<Site>().Property(x => x.Lon).HasPrecision(9, 7);
        modelBuilder.Entity<Site>().Property(x => x.Lat).HasPrecision(9, 7);
    }
}

using(var db = new DbContext())
{
    var s = new Site() { Lon = 40.1234567, Lat = 50.124567 };
    db.Sites.Add(s);
    db.SaveChanges(); // in DB there are ok, non-truncated values

   var s1 = db.Sites.First(); // s1 have truncated Lon = 40.12, Lat = 50.12
}

The problem occurs only if decimal is nullable (decimal?). If decimal is not nullable - all fine.
EF 6.1.3

The behavior should be the same for nullable and non-nullable decimals. Could be related to SqlProviderServices.TruncateDecimalsToScale. @AndriySvyryd can you investigate?

@lansman I have not been able to reproduce this issue; my test code is below. Can you please post a runnable project/solution or complete code listing that demonstrates the behavior you are seeing?
namespace SixTwo
{
    public class Site
    {
        public int Id { get; set; }
        public decimal? Lon { get; set; }
        public decimal? Lat { get; set; }
    }

    public class SixTwoContext : DbContext
    {
        static SixTwoContext()
        {
            Database.SetInitializer(new DropCreateDatabaseAlways<SixTwoContext>());
        }

        public SixTwoContext()
            : base(@"Server=(localdb)\mssqllocaldb;Database=SixTwo;ConnectRetryCount=0")
        {
        }

        public DbSet<Site> Sites { get; set; }

        protected override void OnModelCreating(DbModelBuilder modelBuilder)
        {
            modelBuilder.Entity<Site>().Property(x => x.Lon).HasPrecision(9, 7);
            modelBuilder.Entity<Site>().Property(x => x.Lat).HasPrecision(9, 7);
        }
    }

    public class Program
    {
        public static void Main()
        {
            using (var context = new SixTwoContext())
            {
                var s = new Site() {Lon = (decimal)40.1234567, Lat = (decimal)50.124567 };
                context.Sites.Add(s);
                context.SaveChanges();

                var s1 = context.Sites.First();
            }

            using (var context = new SixTwoContext())
            {
                var s1 = context.Sites.First();
            }
        }
    }
}

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Add test command

This PR adds a new command to invoke to enable simpler loading of testing data. It also makes gitpod.io loading faster.

Seems like a reasonable addition, thanks @matmair


sbt server does collectAnalyses on initialize

Followup from #4110
steps
Connect to sbt server in a freshly cloned repo like https://github.com/non/spire
problem
sbt server runs collectAnalyses during the initialize handshake, triggering compilation in all projects + platforms locking my sbt shell for a few minutes.
expectation
I expect sbt server not to trigger collectAnalyses during the initialize handshake. A client can trigger collectAnalyses via sbt/exec after initialize, if the client is interested in that action.
Judging from the discussions in #4110, it seems the motivation for running collectAnalyses is to support textDocument/definition. However, only a few sbt server clients use that functionality. Additionally, since collectAnalyses may take a long time in large projects (in scalameta/scalameta I suspect it'll be >10 minutes since we cross-build ~17 modules), the client may want to refine the scope with something like core/collectAnalyses in order to provide a snappier UX.
notes
There is no workaround for clients, this step is hardcoded in sbt server https://github.com/sbt/sbt/blob/187c50c78d66c6cee655fc4c0f35dfdd16f0f0d5/main/src/main/scala/sbt/internal/server/LanguageServerProtocol.scala#L61
We want to use sbt server to perform automatic installation in metals (https://github.com/scalameta/metals), but this sbt server behavior is extra unfortunate in our case since the metals client runs a semanticdbEnable command right after initialize, which enables a compiler plugin causing the project to re-compile throwing out the zinc cache from collectAnalyses.
sbt version: 1.1.1

I don't know the details of what collectAnalyses does. From my perspective, sbt server provides a generic solution for clients to trigger tasks and commands via sbt/exec. I would leave it to clients to decide what to do.

Maybe fold it into compile?

Only if it's negligible cost to all other non-LSP, direct and indirect users of compile.


Add Browserify & Babel to Grunt build

For JavaScript, one for combining JS files, the other for using ES6.

:+1: Can merge


Update peer dependencies

Hi, thanks for the PR! However ^15.0.0-0 includes 15.0.0 and any future 15.x versions, so this is not necessary.


[question] Preview lints: shouldn't autofixes be unsafe?

As the title says, wouldn't the autofixes on in-preview lint-rules, inherently be unsafe?
Take the following example (no pyproject.toml or alike):
mlar@cettisanger /tmp/ruff-preview-test> cat test.py
# E201 (preview, "safe" autofix)
print( "E201")
# E703 (not preview, safe autofix)
print("E703");
# RUF015 (not preview, unsafe autofix)
list([1, 2])[0]

mlar@cettisanger /tmp/ruff-preview-test> ruff check test.py --extend-select "RUF" --extend-select "E"
test.py:4:14: E703 [*] Statement ends with an unnecessary semicolon
test.py:6:1: RUF015 Prefer `next(iter([1, 2]))` over single element slice
Found 2 errors.
[*] 1 fixable with the `--fix` option (1 hidden fix can be enabled with the `--unsafe-fixes` option).

mlar@cettisanger /tmp/ruff-preview-test [1]> ruff check test.py --extend-select "RUF" --extend-select "E" --preview
test.py:2:7: E201 [*] Whitespace after '('
test.py:4:14: E703 [*] Statement ends with an unnecessary semicolon
test.py:6:1: RUF015 Prefer `next(iter([1, 2]))` over single element slice
Found 3 errors.
[*] 2 fixable with the `--fix` option (1 hidden fix can be enabled with the `--unsafe-fixes` option).

What is being said here, is that:

E201: The lint rule is "considered experimental or unstable", but the associated autofix is safe
E703: The lint rule and associated autofix are both stable (perfect)
RUF015: The lint rule is stable but the associated autofix is unsafe (this makes sense)

It makes sense that an autofix can be considered safe (that is, it requires no changes before going out of preview), but if you attach it to a lint-rule that has, as an example, too many false-positives (which I assume falls under "experimental and unstable"), it is really that safe?
I was at least under the assumption that in order to use an in-preview autofix, I would have the also specify --unsafe-fixes

Hi!
Fix safety has a different meaning than preview. Fix safety is akin to the confidence that the fix will not change the meaning of your code. Preview is used for new changes, both to explore the viability of rules and reduce the number of new diagnostics users have resolve on patch releases. The safe fixes on these preview rules are intended to meet the standards of a safe fix. If they do not, we need that feedback before the rules are promoted to stable. I don't see an additional opt-in benefiting users. We use unsafe fixes for preview rules when the fix is actually unsafe.
I'd recommend reading the applicability documentation at https://github.com/astral-sh/ruff/blob/main/crates/ruff_diagnostics/src/fix.rs#L12-L24
The preview documentation at https://docs.astral.sh/ruff/preview/
And the versioning discussion at https://github.com/astral-sh/ruff/discussions/6998

Thanks for the fast and detailed response!
The terminology just confused me. Being in preview and still considered "safe" was not working in my head :-)
Again thanks, also for ruff!


[Bug]: Deploy to Environment

Using: AL-Go-Preview (current)
We have environments configured in GitHub (UAT, for instance) and have an environment secret "AUTHCONTEXT" created with the proper Json in there. However, when we run the CI/CD pipeline, it throws the following errors when trying to deploy the apps to the environment.
Did anything change in the way this is working or has to be defined? It used to work (not sure exactly when last).
Waiting for other process executing ReadSecrets
Other process completed ReadSecrets
Masking value for ClientSecret
Trying to get the secret (UAT-AuthContext) from the github environment.
Trying to get the secret (UAT-AuthContext) from Key Vault (*****).
Module Az.KeyVault is available in version 6.0.1
Importing module Az.KeyVault (minimum version 5.2.0)
Connecting to Azure using clientId and clientSecret.
Selecting subscription ***
Successfully connected to Azure
Trying to get the secret (UAT_AuthContext) from the github environment.
Trying to get the secret (UAT_AuthContext) from Key Vault (*****).
Error: Unexpected error when running action. Error Message: Error trying to get secrets from Azure Key Vault. Error was Operation returned an invalid status code 'BadRequest' Code: BadParameter Message: The request URI contains an invalid name: UAT_AuthContext, StackTrace: at GetKeyVaultSecret, C:\1\_work\_actions\microsoft\AL-Go\5eeeec54ab7e6a9d97701effe[42](https://github.com/***/***/actions/runs/9858901855/job/27290709587#step:6:44)0ef74bda08c67\Actions\ReadSecrets\ReadSecretsHelper.psm1: line 150 <- at GetSecret, C:\1\_work\_actions\microsoft\AL-Go\5eeeec54ab7e6a9d97701effe420ef74bda08c67\Actions\ReadSecrets\ReadSecretsHelper.psm1: line 186 <- at <ScriptBlock>, C:\1\_work\_actions\microsoft\AL-Go\5eeeec54ab7e6a9d97701effe420ef74bda08c67\Actions\ReadSecrets\ReadSecrets.ps1: line 88 <- at <ScriptBlock>, C:\1\_work\_temp\ed2defd3-3cb5-4bde-bb87-26a618228d5e.ps1: line 4 <- at <ScriptBlock>, <No file>: line 1
Error: Process completed with exit code 1.

Yeah, we did some changes to this area, which shouldn't have affected the functionality though.
Do you have an AZURE_CREDENTIALS defined with access to a keyvault?
Are you using the keyvault for secrets?

I changed it to use microsoft/AL-Go-PTE@main and yes, it is working. I also have the AZURE_CREDENTIALS defined with access to a keyvault and am using the keyvault for secrets, but not for the environment secrets.

The reason for this issue is that this powershell command:
Get-AzKeyVaultSecret -VaultName "BuildVariables" -Name 'UAT-AUTHCONTEXT'

will return the secret if it exists or null if it doesn't, whereas this command
Get-AzKeyVaultSecret -VaultName "BuildVariables" -Name 'UAT_AUTHCONTEXT'

Will throw due to an illegal secret name.
GitHub secrets allows '', but not '-'
Azure DevOps keyvault allows '-', but not ''
Therefore, we look for both, but it obviously shouldn't throw.

Shipped in preview


adding Mastodon to social profiles in config.yml

This is an enhancement or feature.
Summary

In the _config.yml I included the possibility to add your Mastodon profile in both the sidebar and footer. You can add your full Mastodon profile URL to the list.

Closing as the theme allows you add any link/icon to the author sidebar and footer via author.links and footer.links in _config.yml
Check the docs for more info and examples.
https://mmistakes.github.io/minimal-mistakes/docs/configuration/#site-author

I know you can do this with the author.links and footer.links. But why would there be a couple of defaults already then, like Gitlab and Facebook? Why not add one of the newer platforms as well?  Because, you might as well remove the defaults from Twitter, Facebook, Github and Instagram as well since these can be added as well with author.links and footer.links.

Those are deperecated and will eventually be removed.


Assignment Part 2

Review OSS project documentation for security-related configuration and installation issues. Summarize your observations.
The purpose of this task is to review security-related documentation of the project and find ways to improve it. Open source projects are always looking for contributors for their documentation. It may also allow you to get to know the open source community and procedures to contribute.

Dillon added this portion. Larissa added extra information. The rest edited and approved.


Enable remote db connections

Oops, we forgot to pass down the database hostname and port to the underlying sequelize module. This PR enables us to talk to a remote database.

That will help :smile:  :+1:

Ooppsie Daisy: :+1:


Small API usabilty improvments

Add shortcut .getParent() to PathAddress
allow also sub types of AttributeDefinition as paramter to AbstractBoottimeAddStepHandler ctor

Linux Build 1137 is now running using a merge of e54a4dec789410e3a80b3a44e47814fa07931fca

Windows Build 835 is now running using a merge of e54a4dec789410e3a80b3a44e47814fa07931fca

Core - Full Integration Build 413 is now running using a merge of e54a4dec789410e3a80b3a44e47814fa07931fca

Windows Build 835 outcome was SUCCESS using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03
Summary: Tests passed: 2984, ignored: 95 Build time: 0:18:04

Linux Build 1137 outcome was SUCCESS using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03
Summary: Tests passed: 2984, ignored: 95 Build time: 0:18:32

Core - Full Integration Build 413 outcome was FAILURE using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03
Summary: Tests failed: 2 (2 new), passed: 2853, ignored: 378 Build time: 0:49:04
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptMultipleClientsTestCase.encryptedAndSignedRequestFromAlice: javax.xml.ws.WebServiceException: Could not send Message.
	at java.net.SocketInputStream.socketRead0(Native Method)
	at java.net.SocketInputStream.read(SocketInputStream.java:152)

Windows Build 836 is now running using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03

Linux Build 1138 is now running using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03

Core - Full Integration Build 414 is now running using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03

Windows Build 836 outcome was SUCCESS using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03
Summary: Tests passed: 2984, ignored: 95 Build time: 0:17:56

Linux Build 1138 outcome was SUCCESS using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03
Summary: Tests passed: 2984, ignored: 95 Build time: 0:18:34

Core - Full Integration Build 414 outcome was FAILURE using a merge of 6abadfa82d6a5927aa9f392fbb1ba6074ae5ce03
Summary: Tests failed: 3 (1 new), passed: 2852, ignored: 378 Build time: 0:49:23
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.smoke.webservices.WSTestCase.testManagementDescriptionMetrics: java.lang.AssertionError: null
	at org.junit.Assert.fail(Assert.java:86)
	at org.junit.Assert.assertTrue(Assert.java:41)

org.jboss.as.test.integration.jpa.cfgfile.CfgFileTestCase: org.jboss.arquillian.container.spi.client.container.DeploymentException: Cannot deploy: jpa_cfgfile.jar
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:83)
	at org.jboss.as.arquillian.container.ArchiveDeployer.deployInternal(ArchiveDeployer.java:64)

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptMultipleClientsTestCase.encryptedAndSignedRequestFromAlice: javax.xml.ws.WebServiceException: Could not send Message.
	at java.net.SocketInputStream.socketRead0(Native Method)
	at java.net.SocketInputStream.read(SocketInputStream.java:152)


APEX-83 Fixed race condition for storing the applicationWindowCount…

… and checkpointWindowCount

This is an important fix which corrects a race condition where the incorrect application window count is stored with a checkpoint.
@PramodSSImmaneni please review.


Functions in Kotlin: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-functions?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-functions#4
In which task and step of the codelab can this issue be found?
In the last image there is error.
https://developer.android.com/static/codelabs/basic-android-kotlin-compose-functions/img/45ad1199b078b4aa.png
Describe the problem
String and Int are type whereas name and age are parameters but in the image "Int" is pointed as a parameter.
Additional information
Include screenshots if they would be useful in clarifying the problem.

@tehami94
Thank you for reaching out to us and sharing your concern. The arrows were omitted in the parameter list, so they won't be confusing anymore:


Vendor is not specified in RxIssueRegistry

Hey folks!
When I run the lint checks in my project, I see the following report on my console:
nl.littlerobots.rxlint.RxIssueRegistry in /home/david/.gradle/caches/transforms-3/5e006e2dd79821b780f390d44e831beb/transformed/rxlint-1.7.7/jars/lint.jar does not specify a vendor; see IssueRegistry#vendor

Could you please specify the vendor, so this warning is removed? Nothing urgent, this is just a "warning", the build finishes without errors.

What version of AS are you seeing this? The AGP dependencies should probably be bumped again to the latest stable version for this.

I'm on Android Studio Arctic Fox | 2020.3.1 Patch 2.


Filter and DropMissing

This PR add Filter and DropMissing transformations.

Codecov Report

Merging #28 (89a6e49) into master (efd8914) will increase coverage by 0.28%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   92.00%   92.28%   +0.28%
==========================================
  Files          14       15       +1
  Lines         375      402      +27
==========================================
+ Hits          345      371      +26
- Misses         30       31       +1

Impacted Files
Coverage Δ

src/transforms.jl
97.82% <ø> (ø)

src/transforms/filter.jl
96.29% <96.29%> (ø)

Continue to review full report at Codecov.

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

As usual a great PR 👍🏽 we just need to double check the testmissing helper function, maybe we could avoid introducing a function if all we need is to check the positions of the missing values with findall.

Also, we need to add these two new transforms to the README table 👍🏽

This PR is ready for a final review @juliohm!


Field "to" not present in transaction receipt for contract creating transactions

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (2d22d51 2022-06-08T00:06:46.392116828Z
What command(s) is the bug in?
anvil
Operating System
Linux
Describe the bug
Link to issue on brownie tracker: https://github.com/eth-brownie/brownie/issues/1549
Deploying contracts via brownie using anvil currently fails due to field "to" not being present in transaction receipt received from anvil.
Field "to" should be present in transaction receipt, set to null when it's a contract creation transaction.

bug is here: https://github.com/gakonst/ethers-rs/blob/master/ethers-core/src/types/transaction/response.rs#L389-L390, should probably add a #[serde(default)] @mattsse ?

ah I misread it, #1876
it's missing completely, so yeah we need to fix upstream ethers, on it.

after having another look at the ethers code, I was wondering why it's not included because by default serde deserializes None as null
checked against anvil test in #1876
curl -X POST --header 'content-type: application/json' --data '{"jsonrpc":"2.0","method":"eth_getTransactionReceipt","params":["0x2aa3a1aefa199c991a4544e7319b360af2d4f346db71e4c0af26e0047b595887"],"id":1}' --url http://localhost:8546/

{
  "jsonrpc": "2.0",
  "id": 1,
  "result": {
    "blockHash": "0x08a54bb05bef8c24406f00898e4d1ea27c834b265b0dbce21d2718043884a4b9",
    "blockNumber": "0x2",
    "contractAddress": "0xe7f1725e7734ce288f8367e1bb143e90bb3f0512",
    "cumulativeGasUsed": "0x0",
    "effectiveGasPrice": "0xee6b2800",
    "from": "0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266",
    "gasUsed": "0x3c5d3",
    "logs": [],
    "logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
    "root": null,
    "status": "0x1",
    "to": null,
    "transactionHash": "0x2aa3a1aefa199c991a4544e7319b360af2d4f346db71e4c0af26e0047b595887",
    "transactionIndex": "0x0"
  }
}

and to is null here, so not sure what we're missing

it seems like this is not an issue based on our 2 tests, closing and please re-open if you think we misunderstood


Add xDFSNamespaceServerConfiguration resource - Fixes #16

This has been primarily added to enable FQDN mode to be turned on for DFS Namespace servers. Two other DFS Namespace server parameters were also exposed.
LdapTimeoutSec
SyncIntervalSec
Because I am using files.trimTrailingWhitespace in VS Code, some trailing white space was also trimmed from the Readme.md.
This relates to issue #15, but I will still need to determine if the other DFSN resources require changes to support FQDN mode, so I'm leaving #15 open until I've checked this.

This change is

Reviewed 13 of 13 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

DSCResources/MSFT_xDFSNamespaceServerConfiguration/MSFT_xDFSNamespaceServerConfiguration.schema.mof, line 5 [r1] (raw file):

{
    [Key, Description("Specifies the resource is a single instance, the value must be 'Yes'"), ValueMap{"Yes"}, Values{"Yes"}] String IsSingleInstance;
    [Write, Description("Specifies a time-out value, in seconds, for Lightweight Directory Access Protocol (LDAP) requests for the DFS namespace server.")] Uint32 LdapTimeoutSec;

should we spell out seconds?

Comments from Reviewable

@HemantMahawar What do you think?

Review status: all files reviewed at latest revision, 1 unresolved discussion.

DSCResources/MSFT_xDFSNamespaceServerConfiguration/MSFT_xDFSNamespaceServerConfiguration.schema.mof, line 5 [r1] (raw file):
Previously, PlagueHO (Daniel Scott-Raynsford) wrote…
> @TravisEz13, I defined it as Sec to match the underlying cmdlet property, but if it you think it would be clearer as Seconds, I can change it - I'd take your advice on this one.

@hemant

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

DSCResources/MSFT_xDFSNamespaceServerConfiguration/MSFT_xDFSNamespaceServerConfiguration.schema.mof, line 5 [r1] (raw file):
Previously, HemantMahawar (Hemant Mahawar) wrote…
> Sec is fine. We use "Sec" in PowerShell instead of seconds as a suffix to time related properties. Also built-in WaitFor* resources follow the same pattern on its time related property (RetryIntervalSec)

Thanks @hemant - @TravisEz13 - it should hopefully be ready for your final review! Thanks!

Comments from Reviewable


qiskit-cold-atom not compatible with qiskit-nature 0.7.1

Informations

Qiskit Cold atoms version:

current branch

Python version:

3.10

Operating system:

all
What is the current behavior?
Trying to run tests and install qiskit-cold-atom requires to fix qiskit-nature to <= 0.6.2. Otherwise it raises several errors in linting and testing. The main problem seems to be:
No name 'operators' in module 'qiskit_nature'
Steps to reproduce the problem
All problems are produced in this test run.
What is the expected behavior?
We should be able to cleanly work with qiskit-nature in its most current version.
Suggested solutions

I tried to get started on this one in this branch. However, we have to jump from the syntax of 0.4 to the syntax of 0.7 and this seems to be a fairly major shift. I think that I might have found most of the operators, but some problems and questions remain:

It would seem that SparseLabelOp is the new name of SecondQuantizedOp ? Not sure what has to be changed to make it work.
The register_length seems to not exist anymore and had to be updated to num_spin_orbitals or num_spins.
For the moment the tests still fail with 'Gate rlx has no defined generator'. No idea why.
I think that we should also enable dependabot updates to avoid another massive version jump like this one in the future.

I was not sure if I should open a PR draft or just get your feedback here. Whatever you guys prefer. Ideas how to solve the issue @eggerdj or @LaurinFischer ?

Hi Fred, I think a PR would be good. Like that we can see where the tests fail and comment.

Hi @fretchen,
I've opened a new branch on my fork where I'm working on this (didn't have permission to push directly onto your PR).
Things are working fine now, but I still have to update the tutorials and some tests.
I'll ping you when everything is working, then I suggest you merge this into your open PR and perhaps ask Daniel to review it.
In the latest qiskit nature, the syntax to initialize SpinOp and FermionicOp changed significantly, which required some refactoring. This means that older versions of qiskit nature will no longer be compatible.


breakpoints are not verified when remote debugging over ssh

When starting a (gdb) debugging session on the remote over ssh (so specifying the ssh object in the launch.json properly), all of my breakpoints turn into a "Unverified Breakpoint" state... but still work correctly

What does "Unverified Breakpoint" mean? And why are they "Unverified" while they just work, my program breaks correctly on the breakpoints and I can add / remove breakpoints on the fly...

I have also this problem. I thought that is compilation problem (some flags or whatever) but checked on target and everything works as should. I wondering does it not a problem that I having a different start point.  Still, I will try to figure it out, start maybe from gdb args.

The same for me with remote target. I've not seen this problem with previous VSCode versions. Maybe, they changed something.
Also, there is a big problem. The breakpoints are inserting immediately AFTER -exec-run. If they insert BEFORE, they are visible via (gdb) info b. This is a random issue.
For example,
1-gdb-set target-async on
2-environment-directory "..."
3-file-exec-and-symbols "...firmware.elf"
4-break-delete
source .pioinit
Running executable
5-exec-run
6-break-insert -f "C:\\Users\\User\...main.cpp:7"
i b
(no breakpoints)

@WebFreak001 do you have any ideas? Why does extension insert breakpoints after exec?

uh vscode gives me the breakpoints after starting debugging iirc. Plus it needs to work afterwards to dynamically change breakpoints

@WebFreak001 Who does send a this command -exec-run?

https://github.com/WebFreak001/code-debug/blob/master/src/gdb.ts#L106

Should we wait when autorun commands will be finished before running? https://github.com/WebFreak001/code-debug/blob/master/src/gdb.ts#L100

it is.

https://github.com/WebFreak001/code-debug/blob/master/src/backend/mi2/mi2.ts#L737
I don't see any await/Promise.then(). So, this.miDebugger.start() will be run before end target (microcontroller) is ready for debugging. Am I right?
Normally, autorun with custom commands for embedded target requires 5-10sec depending on the device (flashing, halting, reseting).

it isn't waiting for a response before starting, but it will send all commands in order so they should execute in that order according to the docs

Ok, you know that better. Do you have any ideas how to resolve the issue with breakpoints?

of course that could be a feature improvement for example for flashing etc, if you want it you can make an issue and/or a PR, shouldn't be too hard, just adding a few things to package.json and to the 3 base debug servers

no idea, would need to check the cause first, I have a feeling I will eventually work on this some more soon anyway


Authorization in ASP.NET Core

Authorization refers to the process that determines what a user is able to do.
Authorization is orthogonal and independent from authentication. However, authorization requires an authentication mechanism.
Authentication is the process of ascertaining who a user is. Authentication may create one or more identities for the current user.
ASP.NET Core authorization provides a simple, declarative role and a rich policy-based model. Authorization is expressed in requirements, and handlers evaluate a user's claims against requirements. Imperative checks can be based on simple policies or policies which evaluate both the user identity and properties of the resource that the user is attempting to access.
Authorization components, including the AuthorizeAttribute and AllowAnonymousAttribute attributes, are found in the Microsoft.AspNetCore.Authorization namespace.

Registered users can view all the approved data and can edit/delete their own data.
Managers can approve or reject contact data. Only approved contacts are visible to users.
Administrators can approve/reject and edit/delete any data.

Setting the fallback authorization policy to require users to be authenticated protects newly added Razor Pages and controllers. Having authorization required by default is more secure than relying on new controllers and Razor Pages to include the [Authorize] attribute.
The AuthorizationOptions class also contains AuthorizationOptions.DefaultPolicy. The DefaultPolicy is the policy used with the [Authorize] attribute when no policy is specified. [Authorize] doesn't contain a named policy, unlike [Authorize(PolicyName="MyPolicy")].
An alternative way for MVC controllers and Razor Pages to require all users be authenticated is adding an authorization filter:
builder.Services.AddControllers(config =>
{
    var policy = new AuthorizationPolicyBuilder()
                     .RequireAuthenticatedUser()
                     .Build();
    config.Filters.Add(new AuthorizeFilter(policy));
});

The preceding code uses an authorization filter, setting the fallback policy uses endpoint routing. Setting the fallback policy is the preferred way to require all users be authenticated.
The SeedData class creates two accounts: administrator and manager.
dotnet user-secrets set SeedUserPW <PW>
Services using Entity Framework Core must be registered for dependency injection using AddScoped. The ContactIsOwnerAuthorizationHandler uses ASP.NET Core Identity, which is built on Entity Framework Core. Register the handlers with the service collection so they're available to the ContactsController through dependency injection. Add the following code to the end of ConfigureServices
ContactAdministratorsAuthorizationHandler and ContactManagerAuthorizationHandler are added as singletons. They're singletons because they don't use EF and all the information needed is in the Context parameter of the HandleRequirementAsync method.
Resource-based authorization must be imperative.
"Roles are stored in the Identity cookie. Changes made to user roles are not persisted to the cookie until the cookie is regenerated or the user signs out and signs in. Applications that add users to a role should call SignInManager.RefreshSignInAsync(user) to update the cookie."
When the user is not authenticated, a ChallengeResult is returned. When a ChallengeResult is returned, the user is redirected to the sign-in page.
When the user is authenticated, but not authorized, a ForbidResult is returned. When a ForbidResult is returned, the user is redirected to the access denied page.
if (!User.Identity!.IsAuthenticated) {
  return Challenge();
}

if ( !isAuthorized ) {
  return Forbid();
}

rMyXAHzkT5FeEMZ


buy players automatically

What would be really awesome for me is if there is a way to automatically buy players. So, I would set some criteria in this script (for example: rare gold players from bundesliga < 1000) and then buy 30 players that fit the criteria.

I have found this "controllers.items.ItemDetails.prototype._onBuyNow" for buyNow, but i don't know how call this function ...

For bid 1000 on the first result, we can't just bid with this ? =>
gNavManager.getCurrentScreenController()._controller._listController._viewmodel._collection[0]._auction.bid(1000);

That's an option as well. But you would still need to determine whether you should bid or not.
I did some further research to find out if it is possible to implement. In order to search the market and get the trade IDs you could do the following:
var t = new transferobjects.SearchCriteria();
t.maxBuy = 3000; // or any other option you want
var o = new communication.SearchAuctionDelegate(t);
o.addListener(communication.BaseDelegate.SUCCESS, this, function marketSearch(sender, response) {
  // response.auctionInfo is an array that contains all auction data, including tradeId
});

This combined with the previous snippet could work for an initial version.

Made a UI for the proof of concept. This inherits from the pages.Search screen. Seems to work for BIN sniping. The rest of the app stays usable while the buyer works.
Current limitations are:

Unable to search for rare flag (for example to search rare gold player contracts). This would require paging
Not listing bought items instantly (requires more input fields in the search criteria area)
Only one type of item can be searched at once
Based on Buy Now price only (otherwise we have to keep track of trade status and implement bidding logic. This is doable but way more complex.)
Causes Captcha verification every few hours

Is something like this what you had in mind?

That looks good. Too bad about the rare players, but there can be worked around that, by simply searching for a specific rare player.
I am missing the ability to bid. From the mockup it seems you can only buy (correct me if i'm wrong). However the ability to bid would be what makes this thing really powerful and handy. So ideally I would expect a radio button, forcing the user to choose between bidding and buying.

Yes, as I've stated in my previous comment, this is based on Buy Now only for now.

Based on Buy Now price only (otherwise we have to keep track of trade status and implement bidding logic. This is doable but way more complex.)

About the rare players: being rare is not an issue, but the variants of Specials might be useful to search for. For example: a player can have many special forms like IF (multiple), OTW, Halloween, etc...

When will this be realized and is there any chance of getting a ban from EA for using JS scripts?

@Qeekly: Of course there is always a chance of getting banned by EA by using any tool that helps you trade. I think it's stated in their ToS. Probably I should mention in the readme that the use of these scripts is are at the users own risk.
I am still testing this locally, it works great if used for short amounts of time. I need to fix the rate limiting before I want to push this to the public in order to prevent bans.

Check the change that I proposed for one of your scripts.
I think this is not a bad decision. Tell me how to change the script so that I get the price twice (or better in one) when I click the button, and not three, as now. That is, the problem is this: I click on the button and the price is received, then after a time there are two more price updates. How to fix it? I'm strong in php, java, but not strong at js.

For myself, I rewrote a bit of code for my style and added a few useful features that I need. This is an addition to the previous comment with the question.
(function () {
  'use strict';

	$(document).bind('DOMNodeInserted', function(event) {

		var playerImageUrl;
		var playerId

		if($(event.target).hasClass('DetailPanel')) {

			if($(event.target).find('#getMinBuyNowPrice').length === 0) {

				$(event.target).find('.bidOptions').append('<button id="getMinBuyNowPrice" class="list">Buy Now (min.)Get Price</button>');
				$(event.target).find('.QuickListPanel').append('<button id="getMinBuyNowPrice" class="list">Buy Now (min.)Get Price</button>');

			}

			$('.listFUTItem').click(function () {

				$('#minBuyNow').text('Get Price');

			});

			$('#getMinBuyNowPrice').click(function () {

				if($('body').find('.squadSlot').length === 0) {

					playerImageUrl = $('.listFUTItem.selected').find('.photo').attr('src');

				} else {

					playerImageUrl = $('.squadSlot.ui-slot-selected').find('.photo').attr('src');

				}

				playerId = playerImageUrl.substr(playerImageUrl.lastIndexOf('/') + 1).replace('.png', '');
				search(playerId,<PHONE_NUMBER>).observe(this, handleSearch);

			});

		}

	});

	var minimumBIN =<PHONE_NUMBER>;
	var maskedDefId = 0;
	var handleSearch = function handleSearch(t, data) {

		if(data.items.length > 0) {

			var newMinimumBIN = Math.min.apply(Math, data.items.map(function (o) { return o._auction.buyNowPrice; }));

			if(newMinimumBIN < minimumBIN) {

				minimumBIN = newMinimumBIN;
				$('#minBuyNow').text(minimumBIN);

				var random_time = Math.random()*(17000 - 8000 + 1) + 8000;
				setTimeout(function () { search(maskedDefId, minimumBIN).observe(this, handleSearch); }, random_time);

			} else {

				var lowestBIN = Math.min.apply(Math, data.items.map(function (o) { return o._auction.buyNowPrice; }));
				var player = repositories.Item.getStaticDataByDefId(maskedDefId);

				$('#minBuyNow').text(lowestBIN);

		  }

		} else {

			var lowestBIN = Math.min.apply(Math, data.items.map(function (o) { return o._auction.buyNowPrice; }));
			var player = repositories.Item.getStaticDataByDefId(maskedDefId);
			$('#minBuyNow').text(lowestBIN);
		}

	};

	var search = function search(playerId, maxBuy) {

		minimumBIN = maxBuy;
		maskedDefId = playerId;

		if(maxBuy ===<PHONE_NUMBER>) {

			maxBuy = 0;

		}

		var searchCriteria = new transferobjects.SearchCriteria();
		searchCriteria.type = enums.SearchType.PLAYER;
		searchCriteria.defId = [maskedDefId];
		searchCriteria.maxBuy = maxBuy;

		return repositories.TransferMarket.search(searchCriteria);

	};

})();

@Qeekly: Please do not pollute issues with unrelated comments. Create a new issue and move the above comments to that issue for a new discussion.

@Mardaneus86 Can you paste a code how you created a new item (autobuyer) in nav#footer?

I will post an article on the wiki this evening to show how you can do that.

Keep in mind that the auto buyer is still in a highly experimental state and might cause some issues. It's also not very customisable yet.


Issue 6810 type annotate sqlalchemy.sql.selectable

Description

Checklist

This pull request is:

[ ] A documentation / typographical error fix

Good to go, no issue or tests are needed

[ ] A short code fix

please include the issue number, and create an issue if none exists, which
must include a complete example of the issue.  one line code fixes without an
issue and demonstration will not be accepted.
Please include: Fixes: #<issue number> in the commit message
please include tests.   one line code fixes without tests will not be accepted.

[ ] A new feature implementation

please include the issue number, and create an issue if none exists, which must
include a complete example of how the feature would look.
Please include: Fixes: #<issue number> in the commit message
please include tests.

Have a nice day!

@dzmitar @g-kisenkov I thought we needed to get rid of all mypy errors because the project doesn't currently have any, but with this PR there are a few new ones, even ones that are not related to the selectable.py file.

Some of the fixes for issues not related to selectable.py are shown here: https://github.com/GodelTech/sqlalchemy/pull/2/files. For example, the fix for lib/sqlalchemy/util/compat.py: 117: error: Name "anext" is not defined [name-defined] can be found there. I'm not completely sure so if anybody have other opinions, do not hesitate to fix it other ways

@dzmitar @g-kisenkov I thought we needed to get rid of all mypy errors because the project doesn't currently have any, but with this PR there are a few new ones, even ones that are not related to the selectable.py file.

Some of the fixes for issues not related to selectable.py are shown here: https://github.com/GodelTech/sqlalchemy/pull/2/files. For example, the fix for lib/sqlalchemy/util/compat.py: 117: error: Name "anext" is not defined [name-defined] can be found there. I'm not completely sure so if anybody have other opinions, do not hesitate to fix it other ways

I think either way is fine. In my opinion, though, it's OK to leave out errors in other modules. Other modules can be partially typed if some definition from them is used in a typed context in the main module you're typing out.
Overall, I think that concentrating on one module at a time saves context switching and improves productivity. On the other hand one can get fixated on complicated cases in one module while there can be low-hanging fruits in others. Also, reviewing changes dispersed throughout a plethora of modules is a bit harder, in my view. But again, the choice is ultimately yours.

@dzmitar
The PR is cleared on my end, I can't see anything I could comment on. So, if @mlatysh is done with reviewing, then you can open a PR into the base SQLA repo. Please, ensure that you reference #6810. Also, if you think there are any changes that are worth to comment on to ease the reviewing process from author's side, then don't hesitate to do so.


Update PHPMailer latest version

Hi @saosangmo, thanks for your pull request! This has just been implemented in the latest release.


qlik with Python for Machine Learning

Hi Nabeel,
I am trying to integrate the Python with Qlik sense desktop with the steps you have mentioned in the Github (https://github.com/nabeel-oz/qlik-py-...)
After installing the required prerequisites (Python 3.6 version, Qlik Desktop and Microsoft Visual C++ Build Tools), I am still facing the below issue, after I execute the Qlik-Py-Init.bat, windows.bat file

"error: Command "C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\VC\Tools\MSVC\14.16.27023\bin\HostX86\x64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -Ic:\users\skumar\appdata\local\temp\temp5_qlik-py-tools-4.0.zip\qlik-py-tools-4.0\qlik-py-env\include -IC:\Users\skumar\AppData\Local\Programs\Python\Python37\include -IC:\Users\skumar\AppData\Local\Programs\Python\Python37\include -IC:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\VC\Tools\MSVC\14.16.27023\include -IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\ucrt -IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\shared -IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\um -IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\winrt -IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\cppwinrt /EHsc /Tppystan/_misc.cpp /Fobuild\temp.win-amd64-3.7\Release\pystan/_misc.obj" failed with exit status 2"

"Command "c:\users\skumar\appdata\local\temp\temp5_qlik-py-tools-4.0.zip\qlik-py-tools-4.0\qlik-py-env\scripts\python.exe -u -c "import setuptools, tokenize;file='C:\Users\skumar\AppData\Local\Temp\pip-install-9fy4r3pa\pystan\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record C:\Users\skumar\AppData\Local\Temp\pip-record-1qbg_m01\install-record.txt --single-version-externally-managed --compile --install-headers c:\users\skumar\appdata\local\temp\temp5_qlik-py-tools-4.0.zip\qlik-py-tools-4.0\qlik-py-env\include\site\python3.7\pystan" failed with error code 1 in C:\Users\skumar\AppData\Local\Temp\pip-install-9fy4r3pa\pystan"

Please suggest any solution, if you could help me out on this!!
Thank you so much for the support and great work
Regards
Subash Kumar
<EMAIL_ADDRESS>

Hi Subash,
The second one is about a gcc error to compile pystan.  In documentation page from the pystan module, there is a workaround for this problem. For me, using Anaconda, I just type:
conda install pystan -c conda-forge
Besides that, you mentioned that you're using Python 3.6 but your log error says python3.7

--compile --install-headers c:\users\skumar\appdata\local\temp\temp5_qlik-py-tools-4.0.zip\qlik-py-tools-4.0\qlik-py-env\include\site\python3.7\pystan" failed

Check if are any problems with your envs.
Regards

Thanks for your contribution @HeinRique !
@subash28kumar As mentioned above the error shows that you're using Python 3.7, which is not yet supported by all the libraries used in this project. You will need to uninstall your current version of Python and install 3.6.8 to be able to use PyTools.
https://www.python.org/downloads/release/python-368/

Hi Nabeel/Hein
I am still seeing this error, while trying to uninstall the 3.7 and installing 3.6.8 python
Installing collected packages: pystan
Could not install packages due to an EnvironmentError: [Errno 2] No such file or directory: 'c:\users\skumar\appdata\local\temp\temp6_qlik-py-tools-4.0.zip\qlik-py-tools-4.0\qlik-py-env\Lib\site-packages\pystan\stan\src\test\test-models\good\function-signatures\distributions\univariate\continuous\scaled_inv_chi_square\scaled_inv_chi_square_ccdf_log.stan'
Any thoughts on this
Regards
Subash Kumar

Hi Nabeel/Hein,
We are able to integrate the Python with Qlik Sense based on your instruction,
Now am trying to bring the pandas and numpy  python modules into the qlik
I just added the pandas in the Qlik-Py-Init batch file and started the Qlik-py-Start batch file, is it something correct approach
I am still not able to see pandas and numpy modules in qlik
Thank you so much for your input, really interesting and helpful
regards
Subash Kumar

I think you may have misunderstood the capabilities of this Server Side Extension (SSE). It provides functions built in Python which provide a specific capability to be used in Qlik. For e.g the Prophet function can produce a forecast. There is Python code behind the function that uses several libraries like fbprophet, pandas and numpy, but in Qlik you simply call the function in the expression editor or load script to produce a forecast without having to use any Python code.
It is possible to write Python script in Qlik and have it evaluated in the SSE, but this type of usage is not available with PyTools as mentioned in the notes on the approach.
If you want to build your own SSE the best place to start is here: https://github.com/qlik-oss/server-side-extension

Thank you so much Nabeel for the explanation, I am trying to bring the pandas and numpy libraries into qlik, but not sure how to start in github
Regards
Subash Kumar

Hi Nabeel,
I am just trying to setup the model with the below load script
Model_Name,
EstimatorArgs,
ScalerArgs,
ExecutorArgs

FROM [lib://qlik py testing/MODEL_INIT.xlsx]
(ooxml, embedded labels, table is MODEL_INIT);
Model_Name,
EstimatorArgs,
ScalerArgs,
ExecutorArgs

EXTENSION PythonTools.sklearn_Setup(In{Model_Name,EstimatorArgs,ScalerArgs,ExecutorArgs});
I am getting below error:
Started loading data
In << MODEL_INIT
Lines fetched: 1
The following error occurred:
Connector reply error: grpc::StatusCode::UNKNOWN: 'Exception iterating responses: dictionary update sequence element #0 has length 1; 2 is required'
The error occurred here:
Data has not been loaded. Please correct the error and try loading again.
Apologize if disturbing again
regards
Subash Kumar

The arguments you are passing are probably not in the required syntax. Check them against the documentation to be sure you are providing the correct keywords and separators.

Hi Nabeel,
Thank you for the response, just wanted to check with you on the Prophet function for forecasting, with below syntax, we tried connecting sample superstore data in qlik and tried to forecast the values based on ship date on monthly frequency), the formula is working but unable to see any predicted line
PyTools.Prophet([Ship Date],Sum(Profit),'freq=M')
Thank you for all your support and help
regards
Subash Kumar

Please look at the usage documentation for applying the function correctly. The expression looks ok but your app may miss some essential requirement such as passing the future periods to the SSE.

Hi Nabeel,
Thank you so much for your guidance, still I am not able to plot the prediction chart in qlik
Apologize, I am new to qlik and I am troubling you again and again, sorry for that. If you could help me on this, this would be really helpful.

Tried creating a new app having guidance of your app while data loading,
We do not see any issues while doing data loading, but while ploting the chart with required dimensions and measures, unable to see chart
Dimensions: Order_date
Add Measures: Sum(Sales)
Add Measures: PyTools.Prophet(if(FORECAST_MONTH <= AddMonths(Max(Total [Order Month & Year]), $(vforecastperiod)), FORECAST_MONTH),

Also tried Dimensions: =if(FORECAST_MONTH <= AddMonths(Max(Total [Order Month & Year]), $(vforecastperiod)), FORECAST_MONTH)
Measures: Count({$<FORECAST_LINK_TYPE = {'Actual'}>} Distinct 'Order ID')
Add Measures: PyTools.Prophet(if(FORECAST_MONTH <= AddMonths(Max(Total [Order Month & Year]), $(vforecastperiod)), FORECAST_MONTH),
Count({$<FORECAST_LINK_TYPE = {'Actual'}>} Distinct [Order ID]),
'freq=MS, return=yhat')
Count({$<FORECAST_LINK_TYPE = {'Actual'}>} Distinct [Order ID]),
'freq=MS, return=yhat')
if you can share me your email id, I could share the qvf app I created in qlik.
Regards
Subash Kumar
9046147652

Hi Nabeel,
I feel the app is working with  future forecast dates are being passed to the SSE correctly, because I am able to view the future and historic dates based on the current date in the layout.
The [Date Column] is extend into the future to cover the periods you want to forecast
Report frontend layout:
Dimension: =if(FORECAST_MONTH <= AddMonths(Max(Total [Order_Date]), $(vforecastperiod)),FORECAST_DATE)
Measures=PyTools.Prophet(FORECAST_MONTH,sum(Sales),'freq=M')
load script:
// Here we set the maximum number of years to be forecasted
SET vMaxForecastYears = 5;
// We then get the minimum and maximum dates in the key date field
[TempDateMinMax]:
LOAD
min([Order_Date]) as minDate,
max([Order_Date]) as maxDate
//sum([Sales]) as Sales
RESIDENT [Orders];
// We store the minimum date as a variable. This will be the start date for our forecast calendar
LET vCalendarStartDate = Peek('minDate', 0, 'TempDateMinMax');
// We also store the start date for the forecasting (the first 'future' date). No historical data exists from this date onwards
LET vForecastStartDate = Peek('maxDate', 0, 'TempDateMinMax')+1;
// Finally store the the last date for the forecast
LET vForecastEndDate = $(vForecastStartDate) + 365 * $(vMaxForecastYears);
// Now that we have the variables the temp table can be dropped
drop table TempDateMinMax;
// Create a calendar from the earliest date in the data to the maximum forecast date
// Read from the third load statement below to the top as this is a preceding load
[Forecast Calendar]:
LOAD
FORECAST_DATE,
FORECAST_MONTH,
FORECAST_YEAR,
FORECAST_DATE_TYPE,
'' as FORECAST_KEY;
// Create month and year fields as well as a field to tell us if the date is historic or in the future:
LOAD
FORECAST_DATE,
Dual(Year([FORECAST_DATE]) & '-' & Month([FORECAST_DATE]), MonthStart([FORECAST_DATE])) as FORECAST_MONTH,
Year([FORECAST_DATE]) as FORECAST_YEAR,
if(FORECAST_DATE < $(vForecastStartDate), 'Historic', 'Future') as FORECAST_DATE_TYPE;
// Autogenerate dates for the required range:
LOAD
Date($(vCalendarStartDate) + IterNo() - 1) as FORECAST_DATE
AutoGenerate 1
While $(vCalendarStartDate) + IterNo() - 1 <= $(vForecastEndDate);
// Notice the  in the load above. In our first run we create a generic key that will link each date in the forecast calendar to all actual dates.
// In the load below, we create a copy of the data in the Forecast Calendar where the key is the actual Date
// This will help us separate selections on actual dates/ months/ years with forecast dates and avoid gaps in our forecasting.
Concatenate([Forecast Calendar])
LOAD
FORECAST_DATE,
FORECAST_MONTH,
FORECAST_YEAR,
FORECAST_DATE_TYPE,
FORECAST_DATE as FORECAST_KEY
RESIDENT [Forecast Calendar];
// The values to be forecasted are linked to the forecast calendar by the key date
[Actual to Forecast Link]:
LOAD DISTINCT
Order_Date,
'Actual' as FORECAST_LINK_TYPE,
Order_Date as FORECAST_KEY
//Sales
RESIDENT Orders;
// Each key date is also linked to all possible dates in the forecast calendar
// This is to allow the timeseries to be plotted without any gaps
Concatenate([Actual to Forecast Link])
LOAD DISTINCT
Order_Date,
'Generic' as FORECAST_LINK_TYPE,
'' as FORECAST_KEY
//Sales
RESIDENT Orders;
Not sure though, what I am missing from the above code, once I add the Pytool.Prophet(FORECAST_MONTH, sum(Sales),'freq=M') to the measures in the front end layout in the app, I am seeing the error "The Service is currently unavailable.You may be able to correct this by retrying and using backoff"
Thought to check with you, if anything I am missing
Thank you for the support
Regards
Subash K

Hi Subash, in your latest post you're using FORECAST_DATE as the dimension, yet using FORECAST_MONTH in the measure. That is incorrect as the granularity of the data should match.
Also, if the error mentions the service not being available, you need to confirm that the connection to the SSE is working.
You can attach files to issues in GitHub so if your app can be shared publicly you can post it here.

Hi Nabeel, thank you for the swift response, sorry for the wrong granularity, I have changed the Dimension from FORECAST_DATE to FORECAST_MONTH, I ran the batch files and restart the connection to SSE again, now am not seeing service not available error, but still after the adding the measure as PyTools.Prophet(FORECAST_MONTH,sum(Sales),'freq=M'), unable to see any values, it is showing blank, attached the screenshot below

Dimension: =if(FORECAST_MONTH <= AddMonths(Max(Total [Order_Date]), $(vforecastperiod)),FORECAST_MONTH)
Measure: PyTools.Prophet(FORECAST_MONTH,sum(Sales),'freq=M')
Data load script:
[Orders]:
LOAD
"Row ID",
"Order ID",
"Order_Date",
"Ship Date",
"Ship Mode",
"Customer ID",
"Customer Name",
Segment,
Country,
City,
State,
"Postal Code",
Region,
"Product ID",
Category,
"Sub-Category",
"Product Name",
"Sales",
Quantity,
Discount,
Profit
FROM [lib://sample/Sample - Superstore.xls]
(biff, embedded labels, table is Orders$);
// Here we set the maximum number of years to be forecasted
SET vMaxForecastYears = 5;
// We then get the minimum and maximum dates in the key date field
[TempDateMinMax]:
LOAD
min([Order_Date]) as minDate,
max([Order_Date]) as maxDate
//sum([Sales]) as Sales
RESIDENT [Orders];
// We store the minimum date as a variable. This will be the start date for our forecast calendar
LET vCalendarStartDate = Peek('minDate', 0, 'TempDateMinMax');
// We also store the start date for the forecasting (the first 'future' date). No historical data exists from this date onwards
LET vForecastStartDate = Peek('maxDate', 0, 'TempDateMinMax')+1;
// Finally store the the last date for the forecast
LET vForecastEndDate = $(vForecastStartDate) + 365 * $(vMaxForecastYears);
// Now that we have the variables the temp table can be dropped
drop table TempDateMinMax;
// Create a calendar from the earliest date in the data to the maximum forecast date
// Read from the third load statement below to the top as this is a preceding load
[Forecast Calendar]:
LOAD
FORECAST_DATE,
FORECAST_MONTH,
FORECAST_YEAR,
FORECAST_DATE_TYPE,
'' as FORECAST_KEY;
// Create month and year fields as well as a field to tell us if the date is historic or in the future:
LOAD
FORECAST_DATE,
Dual(Year([FORECAST_DATE]) & '-' & Month([FORECAST_DATE]), MonthStart([FORECAST_DATE])) as FORECAST_MONTH,
Year([FORECAST_DATE]) as FORECAST_YEAR,
if(FORECAST_DATE < $(vForecastStartDate), 'Historic', 'Future') as FORECAST_DATE_TYPE;
// Autogenerate dates for the required range:
LOAD
Date($(vCalendarStartDate) + IterNo() - 1) as FORECAST_DATE
AutoGenerate 1
While $(vCalendarStartDate) + IterNo() - 1 <= $(vForecastEndDate);
// Notice the  in the load above. In our first run we create a generic key that will link each date in the forecast calendar to all actual dates.
// In the load below, we create a copy of the data in the Forecast Calendar where the key is the actual Date
// This will help us separate selections on actual dates/ months/ years with forecast dates and avoid gaps in our forecasting.
Concatenate([Forecast Calendar])
LOAD
FORECAST_DATE,
FORECAST_MONTH,
FORECAST_YEAR,
FORECAST_DATE_TYPE,
FORECAST_DATE as FORECAST_KEY
RESIDENT [Forecast Calendar];
// The values to be forecasted are linked to the forecast calendar by the key date
[Actual to Forecast Link]:
LOAD DISTINCT
Order_Date,
'Actual' as FORECAST_LINK_TYPE,
Order_Date as FORECAST_KEY
//Sales
RESIDENT Orders;
// Each key date is also linked to all possible dates in the forecast calendar
// This is to allow the timeseries to be plotted without any gaps
Concatenate([Actual to Forecast Link])
LOAD DISTINCT
Order_Date,
'Generic' as FORECAST_LINK_TYPE,
'' as FORECAST_KEY
//Sales
RESIDENT Orders;
Regards
Subash Kumar
<EMAIL_ADDRESS>

Hi Nabeel,
I am not sure how to share the app through github, that is the reason, I shared the above code for your view.
Regards
Subash Kumar

Hi Nabeel,
I am able to plot the values, from the below uploaded image, the first 2 columns are actual date(Order date) and sum(Sales) from sample superstore data, the next 2 columns are forecasted columns using the SSE extension, but still I feel the data is not populating correct, looks like join issue, Need your thoughts, if you could help me on this.

Regards
Subash Kumar
<EMAIL_ADDRESS>(if you feel comfortable, could you please share me your email id), please apologize, if you feel disturbing.

You are missing the set analysis expression described in the usage documentation: https://github.com/nabeel-oz/qlik-py-tools/blob/master/docs/Prophet.md#use-prophet-with-your-own-app

Hi Nabeel,
Sure Nabeel, will check as per your guidance but unable to attach the app file for your view in github, I will look into the code again and see what am missing, any idea how can I share the app in github, tried to drag and drop the app, it populates that we don't support the file type
please confirm, what should be my dimension, is that I have to add Order_date which is the original column from the input shet or I should have the forecast date which is derived in qlik as dimension,
similarly measures will be sum(sales) from the original input sheet and I have to add one more measure with pytool library as the forecasted measure.
regards
Subash K

Hi Nabeel,
I am able to do forecasting  now based on different frequencies I am trying to implement logistic regression, any thoughts which documentation I would need to refer now from SSE
Thank you so much for your guidance
Regards
Subash Kumar

You would need to refer to the machine learning section of the usage documentation.
https://github.com/nabeel-oz/qlik-py-tools#usage
I'm closing this issue as all questions have been addressed.


Optionally use funcback pattern if callback is omitted

Create remote methods with "funcbank" pattern to reduce code complexity and make available for use with NoDent.
The pattern is optional (only used if "callback" is omitted) and used only if the callback is NOT specified to the rmeote method.
Note: This also fixes a bug which lost the "options" if the "args" were omitted.

I'm closing this PR as there was no activity for more than a year now...


Peer dependency only on minor and major.

Any chance of changing peer dependencies to only be on major and minor version but exclude patch since it should not introduce breaking changes.

Duplicate of https://github.com/valor-software/ng2-bootstrap/issues/1028 ?


[BUG] Not Return 400 with invalid token

🙁 Actual behavior
When validating a token a 400 gets returned in case the token is invalid.
I tried to implement this feature yesterday to the bot - Problem is when returning a 400 its treated like an exception which means that we need to catch it in the Bot.
🙂 Expected behaviour
In my opinion we should return 200 as the operation on the server was executed successfully, but only with the result, that the token is not valid
🔢 Steps to Reproduce the Problem
No response

Oh yes, good spot 🤓  yep returning a 200 with an invalid message is probably best 👍


Read input registers and basic support of little Indian values

To support Solax X1 G4 inverters, one need to decode little Indian numbers and read Input registers. Sample config file included..

Last minute bug fix on some little Indian values..

Many thanks for contributing @PhilippeRigaux! Can you please check the output from the linter workflow and rename indianess/indian to endianness/endian?

What is the "linter workflow"?

OK, saw the emails, currently fixing them..

Le 28 févr. 2024 à 10:51, Marcus Zoller @.***> a écrit :
What is the "linter workflow"?
You can run make check-code to run the linter. The linter is blocking a merge if there are code formatting issues. It's a bit picky but helps to ensure readability...
—
Reply to this email directly, view it on GitHub https://github.com/mazocode/modbus2mqtt/pull/1#issuecomment-1968721374, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEDPZ2NVUDB3JZ6XUIT2ET3YV4D2HAVCNFSM6AAAAABD44TA52VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYG4ZDCMZXGQ.
You are receiving this because you were mentioned.

Looks good, thanks!