CredentialsError at /payments/new/

django-getpaid version: 2.2.0
django-getpaid-payu version: 0.2.0
Django version: 3.0.6
Python version: 3.7.5
Operating System: Ubuntu 19

I'm trying to finish the process of order with default settings of example app.
Action click  on checkout button (http://<IP_ADDRESS>:8000/order/3/) does POST to http://<IP_ADDRESS>:8000/payments/new/ with data
{"csrfmiddlewaretoken":"2Nmmhzp4MXWaR5cZxdFTHBnrfWdK5Vk0BOmr05g43o6VZfytiV7I85IFmE8Yw0bH","backend":"getpaid_payu","order":"4","amount_required":"199.99","description":"Lock,+Stock+and+Two+Smoking+Barrels","currency":"EUR"}

But response is 500
Request Method: 	POST
Request URL: 	http://<IP_ADDRESS>:8000/payments/new/
Django Version: 	3.0.6
Exception Type: 	CredentialsError
Exception Value: 	Cannot authenticate.
Exception Location: 	/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid_payu/client.py in _authorize, line 78
Python Executable: 	/home/m/Desktop/localhost/django-getpaid/example/venv/bin/python3
Python Version: 	3.7.5

Environment:

Request Method: POST
Request URL: http://<IP_ADDRESS>:8000/payments/new/

Django Version: 3.0.6
Python Version: 3.7.5
Installed Applications:
['django.contrib.auth',
 'django.contrib.admin',
 'django.contrib.messages',
 'django.contrib.sessions',
 'django.contrib.contenttypes',
 'django_fsm',
 'getpaid',
 'getpaid_payu',
 'orders',
 'paywall']
Installed Middleware:
['django.middleware.security.SecurityMiddleware',
 'django.contrib.sessions.middleware.SessionMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.csrf.CsrfViewMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'django.middleware.clickjacking.XFrameOptionsMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware']

Traceback (most recent call last):
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/django/core/handlers/base.py", line 115, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/django/core/handlers/base.py", line 113, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/django/views/generic/base.py", line 71, in view
    return self.dispatch(request, *args, **kwargs)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/django/views/generic/base.py", line 97, in dispatch
    return handler(request, *args, **kwargs)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/django/views/generic/edit.py", line 172, in post
    return super().post(request, *args, **kwargs)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/django/views/generic/edit.py", line 142, in post
    return self.form_valid(form)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid/views.py", line 24, in form_valid
    return payment.prepare_transaction(request=self.request, view=self)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid/models.py", line 343, in prepare_transaction
    return self.processor.prepare_transaction(request=request, view=None, **kwargs)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid/models.py", line 199, in processor
    self._processor = self.get_processor()
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid/models.py", line 219, in get_processor
    return processor(self)
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid/processor.py", line 47, in __init__
    self.client = self.get_client()
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid/processor.py", line 60, in get_client
    return self.get_client_class()(**self.get_client_params())
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid_payu/client.py", line 60, in __init__
    self._authorize()
  File "/home/m/Desktop/localhost/django-getpaid/example/venv/lib/python3.7/site-packages/getpaid_payu/client.py", line 78, in _authorize
    "Cannot authenticate.", context={"raw_response": self.last_response}

Exception Type: CredentialsError at /payments/new/
Exception Value: Cannot authenticate.

Credentials to PayU is default to sandbox
GETPAID_BACKEND_SETTINGS = {
    "getpaid_payu": {
        "pos_id": 300746,
        "second_key": "b6ca15b0d1020e8094d9b5f8d163db54",
        "client_id": 300746,
        "client_secret": "2ee86a66e5d97e3fadc400c9f19b065d",
        "confirmation_method": "PULL",  # required for local testing
    },
}

I'm new one with PayU, then probably I don't know something about PayU setup process.

Thanks for reporting. It looks like PayU doesn't allow its own test credentials that are used in their examples. I need to fix the example project and documentation.

Hi, credentials works. Just documentation leads to many misunderstanings.
Change client_id to oauth_id and client_secret to oauth_secret

I have a problem with payments, I set up my sandbox, I changed the credentials in the settings, and when I create a new order, it goes to payments, I use test cards from payu, in the sandbox the payment appears but on the django site and in the admin panel the transaction status does not change, how to approach this


FATAL EXCEPTION: pool-1-thread-1

When I execute the function ‘MediaPickerBuilder.getThumbnail(fileId: id, type: type, );’, the program will report an error.
Console information;
'E/AndroidRuntime( 8716): FATAL EXCEPTION: pool-1-thread-1
E/AndroidRuntime( 8716): Process: com.sisi.lovemusic, PID: 8716
E/AndroidRuntime( 8716): java.lang.RuntimeException: Methods marked with @UiThread must be executed on the main thread. Current thread: pool-1-thread-1'

Any update on this issue, Builder crashes app while loading media

It seems with the update to flutter several things might have changed. I fixed the issue on my side by adding a Looper Handler to MediaPickerBuilderPlugin class upon results like so.
                     val mainHandler:Handler = Handler(context.getMainLooper());                      val runnable:Runnable = object: Runnable{                         override fun run(){                             if (thumbnail != null)                                 result.success(thumbnail)                             else                                 result.error("NOT_FOUND", "Unable to get the thumbnail", null)                         }                     };

Hi @KasemJaffer , any chance you can take a look at this?

FYI, if you copy the solution from @samonkoba , remember to call
mainHandler.post(runnable)

Or if you need a temporal solution, you can use my fork:
  media_picker_builder:
    git: https://github.com/miquelbeltran/media_picker_builder.git

which contains @samonkoba solution.
I'd prefer that you @samonkoba send the pull request as you came up with the solution, but if you don't I am happy to do that.

@miquelbeltran Please do.

Thanks Kasem!


Add toilet sound I made

add toilet sound I recorded, about same length as original sound effect

We used @karst ’s sound instead, thanks for making the PR anyhow.


ERR_BLOCKED_BY_CLIENT in image editor

use "webview.experimental.useIframes": true
open an image
notice the error below

GET vscode-webview-resource://d92bca1d-43cd-4330-97a4-317e6889d688/file///Applications/Visual%20Studio%20Code%20-%20Insiders.app/Contents/Resources/app/extensions/image-preview/media/loading.svg net::ERR_BLOCKED_BY_CLIENT

Duplicate of #118181


adds retrieval from cold storage back

This PR adds retrieval from cold storage back. Fetching from cold storage is the default behavior of the retrieve command.  If you want to fetch from the cache you have to provide true to the --cache flag, e.g. --cache true.
One caveat: because of how we're building the car files now (not wrapping inside a Unix dir), the project https://github.com/ipld/go-car/ does not work for unpacking the car file, you have to use https://github.com/ipfs/go-cid

Maybe a silly question, but would it make sense to by default pull from cache if available, then fallback to cold?

Maybe a silly question, but would it make sense to by default pull from the cache if available, then fall back to cold?

Yeah, makes sense. This flag implementation comes from the assumption that most of the files wouldn't be stored in the cache (because storing in cache would be more expensive). And if that's true, making a call that most of the time fails would not make sense. But that assumption is probably old. I guess from current architecture thinking all files would go to a DA layer, right? So, checking the cache first would make sense

Hm yeah agreed.
A related, future feature idea—something else that would be nice is if there was another config file that lets you set default settings or contexts, like the vault name, path to (or value of) the private key, the default retrieval method (cache vs cold), etc. Then, commands that use --private-key, --account (from pk value), --vault, etc. can take these values by default. Idk the best approach for what's in this config file, but my use case is: I'm either creating new vaults and want to use the same settings every time w/o passing as flags, or I'm interacting with the same vault and want to reduce the flags I have to pass.

Maybe a silly question, but would it make sense to by default pull from the cache if available, then fall back to cold?

Yeah, makes sense. This flag implementation comes from the assumption that most of the files wouldn't be stored in the cache (because storing in cache would be more expensive). And if that's true, making a call that most of the time fails would not make sense. But that assumption is probably old. I guess from current architecture thinking all files would go to a DA layer, right? So, checking the cache first would make sense

You're assumption is still correct... we're still on track w/ the expensive cache idea (allowing people to optimize costs by trading off cold storage for cache). But even so, may as well get it from the cache assuming it's faster as long as they are paying for it?

Than again, I don't know how easy it is to check the cache and fallback

oh okay 👍 Well, which one is faster is tricky, because the cold storage retrieval is not really hitting Filecoin, it's hitting IPFS where the content is pinned by web3 storage. So, at the end of the day is it just as fast as hitting the cache? 😅 but I get your point, theoretically speaking the cold storage should be way slower
it's an easy change. i think it makes sense to do it


feat: ignore sign bit when all value are negative

Implementation for #2714 .

Codecov Report
Attention: Patch coverage is 65.51724% with 10 lines in your changes missing coverage. Please review.

Project coverage is 77.83%. Comparing base (3f935d5) to head (214a8b7).

Files with missing lines
Patch %
Lines

...t/lance-encoding/src/encodings/physical/bitpack.rs
65.38%
9 Missing :warning:

rust/lance-encoding/src/encodings/physical.rs
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2912      +/-   ##
==========================================
- Coverage   77.83%   77.83%   -0.01%
==========================================
  Files         231      231
  Lines       70199    70219      +20
  Branches    70199    70219      +20
==========================================
+ Hits        54641    54656      +15
+ Misses      12691    12679      -12
- Partials     2867     2884      +17

Flag
Coverage Δ

unittests
77.83% <65.51%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🐛 [BUG]: Custom edge getBezierPath curvature parameter does nothing

Is there an existing issue for this?

[X] I have searched the existing issues and this is a new bug.

Current Behavior
Changing curvature does absolutely nothing
Expected Behavior
Changing curvature should adjust the curvature of the Bezier path
Steps To Reproduce

Got to the Vue Flow Edges custom example
replace
const path = computed(() => getBezierPath(props))
with
const path = computed(() => getBezierPath({...props, curvature: 1}))
notice that nothing happens
change the curvature to any other number (e.g. 1000)
notice that nothing happens

Relevant log output
n/a

Anything else?
No response

Curvature does something but probably not what you think it does :)
Here's what it looks like:

Curvature 0

Curvature 1

So curvature influences the curve that an edge has at handles, not the bezier curve itself.

Thanks for the explanation 👍
Is there a way to edit the bezier curve curvature/elasticity with a few flow function or do I need to create my own / find an external curve creation function?

Currently, the bezier edge that is provided (+ the getBezierPath util) have a fixed curvature to them.
If you want something more custom, you'd have to implement it yourself but I am always open to taking PRs that improve on existing functionality, I just don't have the time right now to work on that as it's not really a highly requested feature 😅


Fix spelling

Fix some cases where "download" is misspelled. This should not be a breaking change, because all names modified are non-public.

Embarrassing! Thanks for it.


not working

not working, when i press left Alt i saw only mouse without anythins

Hi @xMeftu,
Thanks for reporting the issue. I'm sorry to hear that you're having trouble.
However, in order to help you effectively, I'll need more information. When you say it's "not working", could you please elaborate more?
Here are a few things that might help us identify the problem:

Are there any error messages displayed in the F8 console (FiveM's built-in error console) when you try to open the editor?
Have you followed the installation instructions exactly as mentioned in the README?
Can you confirm if any other mod or script is interfering with the key binding of Left Alt?
What is your current FiveM server version, and which OS are you running it on?

Please provide these additional details, so that I can further assist you.
Also, for future reference, providing as much information as possible when creating an issue helps us to quickly identify and rectify the problem. This might include steps to reproduce the issue, screenshots or video clips of the problem, error messages, or any other information related to the problem.
Looking forward to hearing back from you soon.
Best regards,
Kerm

the ui doesnt show up on newer versions of es extended


[Bug]:  Dino Runner :  Game starts automatically and  The dinosour can  not be seen on the screen if it  jumps and crosses the barriar of screen

Give a brief about the bug ✍
There should be a start button to start the game when the user wants
If the dino  jumps and it goes off the screen, it can not be back in the screen
After game over, the timer of 3 seconds should be live on the screen
What is the expected behavior? 🤔

a start button will be on the screen to start the game
if the dino jumps, it wont go off the screen  or there will be an option to get it back on the screen
the timer   should  be seen running on the screen after game over

Provide step by step information reproduce the bug 📄
-open the relevant js and html file
-modify them to fix the bugs
Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Tasnuva12 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

i could not tag this issue as gsoc . please consider it as a contribution of this program

/assign

Hey @Tasnuva12 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Idea: Self-documenting?

Would it be possible to (at app startup / schema generation), crawl the tree, figure out what validators would fire where, and add some kinda of documentation onto the input (at field level or otherwise).
Is there a FluentValidation built-in/community package that can generate docs like this? Could use that and then read from it.

Consolidating this issue and a few others under #39


Name-based rearrange: Refactor and fixes

Motivation and Context
Code changes for working with recent rearrange updates
How Has This Been Tested

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Sanity-check experiment: Name-based pick training success (Priors: Category, Oracle segmentation) ReplicaCAD scenes, Fetch Robot


Instrument Documentation

Improve docstrings and other documentation in older instrument routines.

Closing with merge of #208.


add speaker next to chromecast audio

Add a Speaker symbol next to Chromecast Audios
Wjile waiting for https://github.com/fardog/node-chromecast-osx-audio/pull/9 to be merged, I've pointed this project at my fork.
The soundcast process doesn't have to kill the node-chromecast-osx-audio process any more as it kills itself.

Good work @danjenkins! I'll test this and build a new release tonight. Thank you!

@andresgottlieb could you build the zip and push it as a release to github? I know other projects do this - I created a cask recipe for soundcast https://github.com/caskroom/homebrew-cask/pull/15776 - and other projects put their zips as github releases - this would also mean people could install an older version if they needed to etc

@danjenkins I released v1.5 as you proposed (thanks for the idea!). Is there a way to make Caskroom always use the last one available or we have to update it manually every time we release a new one?

Im not sure, as it was before, it was always taking the zip from Dropbox so
it would be the latest, now if I understand correctly you've put it on
github as a release, that will be versioned so we probably need to update
the cask version every time. Will double check.
On 12 Dec 2015 00:07, "Andrés Gottlieb"<EMAIL_ADDRESS>wrote:

@danjenkins https://github.com/danjenkins I released v1.5 as you
proposed (thanks for the idea!). Is there a way to make Caskroom always use
the last one available or we have to update it manually every time we
release a new one?
—
Reply to this email directly or view it on GitHub
https://github.com/andresgottlieb/soundcast/pull/24#issuecomment-164085426
.

@andresgottlieb did you upload the release as Soundcast_v1.5.zip or did Github add it? Just trying to see if theres a way to get the latest artefact

@danjenkins I uploaded it as Soudcast_v1.5.zip

I have a feeling the "correct" way to do this is to put in a PR every time you update and we probably should try and do it the correct way


Image overflow in the 2nd section and 3rd section of home page in mobile view

@Chaitalishetty I want to work on this issue under JWoC.

@Aniket-508 Assigning you this issue. Happy Coding!


Access via IAM roles for service account

Describe why this change is needed
Currently awspca configuration needs creating a dedicated IAM user with proper access and define a secret with access key - but since it is used in EKS it would be better to use dedicated service account with assigned IAM policy. It would be more natural way I think to integrate it. How do you think? (https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts.html)
Describe solutions and alternatives considered (optional)
No response
Is there anything else you would like to add?
No response

Thank you for raising this enhancement request with the AWS Private CA Issue plugin. We do support IAM Role for Service Accounts(IRSA) as mentioned in the Readme.  Please refer this example for steps on how to do this. We have resolved the query and are closing the issue. Please reopen if you have any issues or questions.

OK thanks, I missed it somehow, I will look at it.


[Event Request] Codeunit 5760 "Whse.-Post Receipt" function InitSourceDocumentLines

Dear ALAppExtensions Team,
We would like to request the following event publisher in codeunit 5760 "Whse.-Post Receipt" function InitSourceDocumentLines
OnBeforePurchLineModify(PurchLine, WhseRcptLine2, ModifyLine);
 if ModifyLine then
   PurchLine.Modify();
   // DINNL <<
     DINOnInitSourceDocumentLinesAfterModifyPurchLine(PurchLine, ModifyLine);
   // DINNL >>
 until PurchLine.Next() = 0;

local procedure DINOnInitSourceDocumentLinesAfterModifyPurchLine(var PurchaseLine: Record "Purchase Line"; ModifyLine: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Great Chart Package!

Hi,
Since the dataSet is a computed property, how do I update the values?
Thanks for the great work!
Blessings,
--Mark

Hi and thanks a lot! Data set is not computed property so you can just set new data set and chart updates.
Marek


QR scanner freezes if ATTESTING period ends while scanner open and scanning a claim too late

tested with @447f7139c6f7dafb7d80e43c9b3805933a897e5e

Can't we just close this a wont'-fix, when is this ever gonna happen in production?


Issue with make run at step 10/10

Hi
The setup keeps getting stuck at stage 10 with the error shown below, any tips on what the issue could be. The perf analysis was actually up and running for a while, but then somehow failed, and now this error is encountered each time -
Start an instance of our Ansible image to perform setup on the running instance
docker run --rm --network=container:grafana ntap-grafana/ansible:2.1
No config file found; using defaults
[WARNING]: provided hosts list is empty, only localhost is available. Note that
the implicit localhost does not match 'all'
PLAY [localhost] ***************************************************************
TASK [Gathering Facts] *********************************************************
ok: [localhost]
TASK [include_tasks] ***********************************************************
included: /home/tasks/grafana.yml for localhost
TASK [Wait for InfluxDB to startup completely] *********************************
ok: [localhost] => {"changed": false, "elapsed": 2, "match_groupdict": {}, "match_groups": [], "path": null, "port": 8086, "search_regex": null, "state": "started"}
TASK [Get influxdb datasource] *************************************************
fatal: [localhost]: FAILED! => {"cache_control": "no-cache", "changed": false, "connection": "close", "content_length": "42", "content_type": "application/json; charset=UTF-8", "date": "Mon, 04 Jan 2021 15:56:15 GMT", "elapsed": 0, "expires": "-1", "json": {"message": "Invalid username or password"}, "msg": "Status code was 401 and not [404, 200]: HTTP Error 401: Unauthorized", "pragma": "no-cache", "redirected": false, "status": 401, "url": "http://grafana:3000/api/datasources/id/WSP", "x_frame_options": "deny"}
PLAY RECAP *********************************************************************
localhost                  : ok=3    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0
make: *** [run] Error 2
[root@PHCGRAFANA02 eseries-perf-analyzer]#

Grafana can't access InfluxDB. Ensure that config details are correct (including datastore.json) and you executed from the right location as user who has permissions to these files. It could be many things but end result is Grafana can't access InfluxDB.

Grafana can't access InfluxDB. Ensure that config details are correct (including datastore.json) and you executed from the right location as user who has permissions to these files. It could be many things but end result is Grafana can't access InfluxDB.

Can this be closed?

I was leaving it for the creator to close, but I'll close it since it should be fixed.


What's the proper way to trigger scan the device again?

What's the proper way to let sriov device plugin to scan the VF again?
For example, forget to create VF before running sriov dp daemonSet
or change VF device numbers after sriov dp daemonSet's enumeration is done.
For now, I just delete the pods of sriov dp. so that the daemonSet will create the new pod to enumerate again.
Wondering if there is a proper way to do this.

I did the same by deleting daemon pod or re-create the daemon.
Ideally this would be triggered by a controller which monitors the change of VF provisioning and re-load device plugin daemon if necessary. There is a SR-IOV Operator example which makes use of Kubernetes Operator framework to implement such controller.

closing as stale


Can you please add an example file

Please add an example file/app so it gives a clearer understanding for people to use this package.

Sorry guys for delay, I don't have time for example app, however, I added example in README.md file in PR https://github.com/dotronglong/flutter-mailgun/pull/3


swift3 support?

Hey! Great library, would love to see swift3 supported.
Thanks!

Yes, it's in my todo. I'll work no that tomorrow for sure!

Awesome, you rock!

@philippeluickx done! sorry for the delay I've been kind of busy !
Master is now Swift 3.0 and updated pod version to 1.0.

 Thanks a lot!


RuntimeError: Cannot set version_counter for inference tensor AMD - w11

I am trying use  https://github.com/suno-ai/bark with DirectML over windows 11, changing the .to(device) to .to(dml) according to gpu-pytorch-windows Docs in the files generation.py in bark folder and build\lib\bark\ respectly. When I try run the project. I seen that the GPU started correctly but then i get the next error.
I am in Python 3.9.16
Could try help me to solve this bug, I dont know that could I do.
RuntimeError: Cannot set version_counter for inference tensor
Console Output:
python .\run.py
No GPU being used. Careful, inference might be very slow!
  0%|                                                                                | 0/100 [00:00<?, ?it/s]Traceback (most recent call last):
  File "C:\Users\NoeXVanitasXJunk\bark\run.py", line 13, in <module>
    audio_array = generate_audio(text_prompt)
  File "C:\Users\NoeXVanitasXJunk\bark\bark\api.py", line 107, in generate_audio
    semantic_tokens = text_to_semantic(
  File "C:\Users\NoeXVanitasXJunk\bark\bark\api.py", line 25, in text_to_semantic
    x_semantic = generate_text_semantic(
  File "C:\Users\NoeXVanitasXJunk\bark\bark\generation.py", line 460, in generate_text_semantic
    logits, kv_cache = model(
  File "C:\Users\NoeXVanitasXJunk\miniconda3\envs\tfdml_plugin\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Users\NoeXVanitasXJunk\bark\bark\model.py", line 208, in forward
    x, kv = block(x, past_kv=past_layer_kv, use_cache=use_cache)
  File "C:\Users\NoeXVanitasXJunk\miniconda3\envs\tfdml_plugin\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Users\NoeXVanitasXJunk\bark\bark\model.py", line 121, in forward
    attn_output, prev_kvs = self.attn(self.ln_1(x), past_kv=past_kv, use_cache=use_cache)
  File "C:\Users\NoeXVanitasXJunk\miniconda3\envs\tfdml_plugin\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Users\NoeXVanitasXJunk\bark\bark\model.py", line 50, in forward
    q, k ,v  = self.c_attn(x).split(self.n_embd, dim=2)
  File "C:\Users\NoeXVanitasXJunk\miniconda3\envs\tfdml_plugin\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Users\NoeXVanitasXJunk\miniconda3\envs\tfdml_plugin\lib\site-packages\torch\nn\modules\linear.py", line 114, in forward
    return F.linear(input, self.weight, self.bias)
RuntimeError: Cannot set version_counter for inference tensor
  0%|                                                                                | 0/100 [00:00<?, ?it/s]

I also encounter this issue when testing llama.
Environment:
Python: 3.10.11
PyTorch: 2.0
torch-directml: 0.2.0.dev230426

I downgraded to PyTorch 1.13 and Torch-directml 0.1.13 as documented. This error occured, too.

Same happens when trying to train yolov5 model using DirectML on AMD GPU(Radeon 6750XT 12GB). Windows 10,  torch-directml-0.2.0.dev.230426.
Command:
python train.py --data VisDrone.yaml --epochs 100 --cfg yolov5n.yaml  --batch-size 32
I see that it started to utilize my GPU, I see that it has finalized a first epoch and when trying to switch between epochs 0 to 1 it fails with error:
      Epoch    GPU_mem   box_loss   obj_loss   cls_loss  Instances       Size
       0/99         0G     0.1299     0.1296    0.04918        431        640: 100%|██████████| 405/405 [03:24<00:00,  1.98it/s]
                 Class     Images  Instances          P          R      mAP50   mAP50-95:   0%|          | 0/18 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "train.py", line 649, in <module>
    main(opt)
  File "train.py", line 538, in main
    train(opt.hyp, opt, device, callbacks)
  File "train.py", line 361, in train
    results, maps, _ = validate.run(data_dict,
  File "C:\ML\yolov5\venv\lib\site-packages\torch\utils\_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "C:\ML\yolov5\val.py", line 210, in run
    preds, train_out = model(im) if compute_loss else (model(im, augment=augment), None)
  File "C:\ML\yolov5\venv\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\ML\yolov5\models\yolo.py", line 210, in forward
    return self._forward_once(x, profile, visualize)  # single-scale inference, train
  File "C:\ML\yolov5\models\yolo.py", line 122, in _forward_once
    x = m(x)  # run
  File "C:\ML\yolov5\venv\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\ML\yolov5\models\common.py", line 56, in forward
    return self.act(self.bn(self.conv(x)))
  File "C:\ML\yolov5\venv\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\ML\yolov5\venv\lib\site-packages\torch\nn\modules\batchnorm.py", line 171, in forward
    return F.batch_norm(
  File "C:\ML\yolov5\venv\lib\site-packages\torch\nn\functional.py", line 2450, in batch_norm
    return torch.batch_norm(
RuntimeError: Cannot set version_counter for inference tensor

I also encounter this issue when testing llama.
Environment:
Python: 3.10.11
PyTorch: 2.0
torch-directml: 0.2.0.dev230426

I downgraded to PyTorch 1.13 and Torch-directml 0.1.13 as documented. This error occured, too.

Do you solve this error?

Hi all, thank you for submitting this issue. While I can't provide a timeline for resolution as the moment, please know that your feedback is valuable to us. We will follow up once we can review this issue.


packit to assist with creating an upstream release

How about we created a CLI command in packit to prepare for a new release the way we do it in our projects, the command would:
$ packit prepare-upstream-release 0.31.0

check out a new local git branch: git checkout -B {release-version}-release
update spec file: rpmdev-bumpspec -n {release} -c "- New upstream release {release}" {specfile_path}
run user defined commands - a new action (we could script %changelog prep here)

I wonder if this is the direction we want to go with packit: helping with upstream maintenance tasks

I wonder if this is the direction we want to go with packit: helping with upstream maintenance tasks

When I think about that. Since it manipulates spec-file it does make sense in Packit. And with all the code around, it shouldn't be a problem to do.

Leaving this closed on purpose - we have other stories to work on that have bigger prio. Also, this would be hard to do for everyone since projects do releases differently. Let's implement automation only for us in the meantime and scope this when it's actually requested by some of our users.


Networking part2 [WIP DO NOT MERGE]

Signed-off-by: baude<EMAIL_ADDRESS>

One nit to add --net as a valid option.
--net=none needs to be implemented, but that can be in a separate pr.


OVERLAPBLOCK (DESIGN 5)

Overlapblock (design 5)
Design a overlap block that looks like the button in this image...
Create a file "OVERLAP_DESIGN_5.html" in the modules/overlapblock folder.
Use simple HTML and CSS and JAVASCRIPT to create the overlapblock. You can use the interface development https://kwickerhub.com (This project) or you can write the lines of code yourself.
Step by Step Guide.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "OVERLAP_DESIGN_5.html" in the "modules/overlapblock" folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder  ••
"modules/overlapblock/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Make you Pull Request...
Good-luck.

Please assign it to me.


fix: Fixed a crash when opening the recipe view tab order GUI

allTabs was being initialized AFTER calls to allTabs.size(), and this led to unsavoury results

This project was deprecated in favour of AMI.


[BUG] Duplicate records loaded for same hashdiffs

Describe the bug
When loading a satellite incrementally, we can see duplicates come in if records with the same hashdiff are loaded twice.
Environment
dbt version: 1.1.0
dbtvault version: 0.8.3
Database/Platform:  All
To Reproduce
Steps to reproduce the behavior:

Load a Satellite
Load the same satellite again with the same records
See duplicates

Expected behaviour
Idempotent load, no duplicates
Additional context
Failing test:
@fixture.satellite
Scenario: [SAT-IM-10] Load data into a non-existent satellite, where we have two records with different payloads and same keys
  Given the SATELLITE table does not exist
  And the RAW_STAGE table contains data
    | CUSTOMER_ID | CUSTOMER_NAME | CUSTOMER_DOB | CUSTOMER_PHONE  | LOAD_DATE  | SOURCE |
    | 1001        | Alice         | 1997-04-24   | 17-214-233-1214 | 1993-01-01 | *      |
    | 1001        | Alice         | 1997-04-24   | 17-214-233-1215 | 1993-01-01 | *      |
  And I stage the STG_CUSTOMER data
  And I load the SATELLITE sat
  And the RAW_STAGE table contains data
    | CUSTOMER_ID | CUSTOMER_NAME | CUSTOMER_DOB | CUSTOMER_PHONE  | LOAD_DATE  | SOURCE |
    | 1001        | Alice         | 1997-04-24   | 17-214-233-1214 | 1993-01-01 | *      |
    | 1001        | Alice         | 1997-04-24   | 17-214-233-1215 | 1993-01-01 | *      |
  And I stage the STG_CUSTOMER data
  And I load the SATELLITE sat
  Then the SATELLITE table should contain expected data
    # We get 4 records here instead of 2
    | CUSTOMER_PK | HASHDIFF                                              | CUSTOMER_NAME | CUSTOMER_PHONE  | CUSTOMER_DOB | EFFECTIVE_FROM | LOAD_DATE  | SOURCE |
    | md5('1001') | md5('1997-04-24\|\|1001\|\|ALICE\|\|17-214-233-1214') | Alice         | 17-214-233-1214 | 1997-04-24   | 1993-01-01     | 1993-01-01 | *      |
    | md5('1001') | md5('1997-04-24\|\|1001\|\|ALICE\|\|17-214-233-1215') | Alice         | 17-214-233-1215 | 1997-04-24   | 1993-01-01     | 1993-01-01 | *      |

Why have we not seen this before?
We didn't have test coverage for this specific situation, as I believe it is something to do with how hashdiffs are compared vs the latest records.

This happens when we have more than 1 record per key for a given batch, which breaks the standard load pattern. Multi-active Satellites should be used in this case, however we want to find a way to prevent this in standard satellites, either with an error message or by doing some automatic filter for the user.
We also plan to release more guidance on this.

Fix released in v0.9.0. Let us know if you continue to have issues. Thanks!


fix: security configuration in methods, When the security definition is in the main configuration of the swagger definition

Thank you for this bugfix, I'll update swagger-typescript-api as soon as possible


Code Generation Patch Based Inference Bug

Facing an issue related to PR #26:

Traceback (most recent call last):
File "examples/vww.py", line 31, in
life_cycle_path="./lifecycle.png",
File "/Users/amahmed/Desktop/UMass/Spring_2022/Thesis/tinyengine/code_generator/CodegenUtilTFlite.py", line 70, in GenerateSourceFilesFromTFlite
code_generator.codeGeneration()
File "/Users/amahmed/Desktop/UMass/Spring_2022/Thesis/tinyengine/code_generator/CodeGenerator.py", line 131, in codeGeneration
self._genPatchInference()
File "/Users/amahmed/Desktop/UMass/Spring_2022/Thesis/tinyengine/code_generator/CodeGenerator.py", line 182, in _genPatchInference
last_patch_op_output_buffer_str_for_patch_inference = last_patch_op._getBufferstr(
AttributeError: 'NoneType' object has no attribute '_getBufferstr'

Deleting lines 179 to 181 in CodeGenerator.py removes the error.

Hi @Zerkclown,
Thanks for reporting the bug. The fix has been merged: https://github.com/mit-han-lab/tinyengine/pull/28.

Thank you, this fixed it!


Create a "Getting Started" guide and introduction project

Note that the current "Introduction" example can be removed once this is complete.

The introduction project has been created.

This should probably be created in the wiki rather than as a source document. README.md can link to it.

This is done.


-update 优化了README.md

DeveloperPullRequest
sun-yinuo other => sun-yinuo
2023.1.22 Request approval for merger

pass


Fail to collapse the function app when the 'Functions' node is refreshing

Repro Steps:

Right click one subscription and select 'Create Function App in Azure' to create a new function app.
Expand and right click the 'Functions' node then select 'Refresh'.
Check whether this function app can be collapsed or not when the 'Functions' node is refreshing.

Expect:
This function app can be collapsed and 'Refresh' action is canceled.
Actual:
Fail to collapse the function app. Please get the detailed video from
\trinlab2\BAQAFiles\CloudToolsTest\v-dantch\Video\Fail-to-callapse.mp4

This no longer repros for me. It was likely fixed by VS Code


Write a Julia program to print hello world

Description
Write a Julia program to print hello world
Desciption
Hello World !!!

How to contribute

Save the solution in print-hello-world.jl file
Add print-hello-world.jl file in print-hello-world folder

!assign


Fix markdown formatting

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Move away from commit-count in rewrite-clj versioning scheme

Problem/Opportunity
Using commit-count in the versioning scheme for rewrite-clj has had its merits but I'm finding the unpredictability of the next version number is annoying me enough to change.
For example, I'd like to be specific in my :added :deprecated metadata on vars.
That is hard to do with a commit count.
And it can be slightly futzy when getting the correct version in docs.
But this is arguably predictable enough in an automated release flow.
Proposed Solution
Move from commit-count to release-num.
Release-num increments by 1 for each release of rewrite-clj.
It spans the entire life of rewrite-clj.
Can be calculated by counting git tags.
Next release-num is (inc (git-tag-count)).
What will we lose?
I liked commit-count for local installs to my maven repo.
It usually distinguished them clearly from official releases.
But... since I also suffix local install versions with -canary for lib testing and -cljdoc-preview for cljdoc previews, they are distinguished anyway.
Other benefits
The commit-count numbers can get large and hard to remember making them less user-friendly.
Alternative Solutions
There are many.
I went over some of them here.
Additional context
None.
Action
I shall implement and document.

Next release-num is (inc (git-tag-count))

I propose to decouple this from tag counts and just create a version.edn where we generate git tags from and update.
Example here: https://github.com/babashka/cli/blob/91c836932b4f32f1eb1fb69e9feed418176ef926/bb.edn#L29

@lread My thinking is that you can have tags for more than just releases. And there is no reason to rely on git to get the next release number, if it's already stored in version.edn and one inc away.

Yeah, maybe I'm just swayed more by the current code and past thinking.
I don't think this one is worth a pros and cons grid. 🙂
So your version.edn:

is the version of the last release
is a version template for the next release

I can assimilate. I will not resist.


Add note about possible violation of max-pathlength on Windows

Just hit this.
I struggled with what to name my project for the template to work out (tried three or four times before landing on on something that felt like it fit the template).  Ended up following Nuke's Resharper pattern to shorten the name on Windows.


Corrected Scala:TwitterUtils parameters

Per Sean Owen's request, here is the update call for TwitterUtils using Scala :)

Can one of the admins verify this patch?

LGTM, will merge after a short time. I will run the doc build manually to double-check.

PS Usually you make the title like SPARK-6626 [DOCS] ..., could you update it?
https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark

Okay, updated. The system had added 5295 to my commit so I'm assuming that is the number to use :)

Actually this is not the right JIRA ticket. https://issues.apache.org/jira/browse/SPARK-5295 is about data types.


ENH add pava and isotonic_regression

Reference issue
Closes gh-17706
What does this implement/fix?
This PR adds isotonic regression via the pool adjacent violators algo. It adds to the API

[x] scipy.interpolate.isotonic_regression
[x] scipy.interpolate.IsotonicInterpolator

Additional information
For a long time, I missed writing semicolons at end of lines. So I went with C++ and pybind11:smirk:

An update to the docs explaining that the existing interpolators are only monotonic only if the original data is monotonic would be great.  From web searches it seems a lot of people are confused about this.

I moved the PAVA code and isotonic_regression to scipy.optimize. I guess I should also move and rename IsotonicInterpolator. I could also remove it from this PR and add it later. This would make this PR smaller and easier to review.

@ev-br Is their anything I can do to make the review easier?

As it seems to controversial for the moment, I removed the IsotonicInterpolator in https://github.com/scipy/scipy/pull/17722/commits/961558837f16f2b38ee8e698666351b8950f2a9c.
As info: I might not respond to further comments until June.

I wonder if this should return an OptimizeResult object like lst_linear or least_squares or at least some sort of results object rather than a bare tuple?

For the moment, https://github.com/scipy/scipy/pull/17722/commits/ebe9b2271f97661516991b11f63666eedc69fa96 is my last trial to make CI green.

CI is 🟢 , comments are addressed as good as I could.

Friendly ping at reviewers @ev-br @j-bowhay @amjames.
Anything I can do to make your review easier?

For the C++ / pybind11 part, I personally defer to @amjames . He had several suggestions above, not sure if they are addressed.
From a general "defensive driving" perspective, I'd suggest to fix the behavior for non-default dtypes (e.g.: what happens for float32 input? float16?) and fix the behavior with tests. Ditto for non-contiguous and readonly inputs.
Since this now adds things to scipy.optimize, maybe one of scipy.optimize regulars could take a look?

From a general "defensive driving" perspective, I'd suggest to fix the behavior for non-default dtypes (e.g.: what happens for float32 input? float16?) and fix the behavior with tests. Ditto for non-contiguous and readonly inputs.

Done in https://github.com/scipy/scipy/pull/17722/commits/651db47ebc7a963460a6c9a0b2c913d8a80f52fb and https://github.com/scipy/scipy/pull/17722/commits/804883c59187f152637990521ca38d9780146425.

My comments are also addressed, and this PR looks good to me. Would be great if scipy.optimize regulars could take a look. I'll ping @andyfaff   @mdhaber @dschmitz89 , hope it's OK.

@ev-br You requested changes some time ago. Could you approve this PR or direct my to the open points?

@ev-br Now I‘m confused very much. I changed this PR months ago, even changed the description: The whole interpolator thing is removed and only one public function added to the optimize module.
Could you please revisit.

Ugh, my apologies!
Apparently I was re-reading some long-cached version. And was indeed quite confused by reappearance of things I was vaguely remembering as having been removed.
LGTM on a refresh and reread!
I'm going to leave it for a few more days in case an optimize regular wants to take a final look-through, and merge otherwise.

Thanks @ev-br

One (last?) point that was asked for by reviewers is whether the user facing function should return an OptimizeResult instead of a tuple of arrays. I‘m happy to change that and it would make it consistent to the optimize module.
The boring part is that there is hardly any optimization info that could be reported apart of that tuple of arrays, e.g. success=True always (isn’t that nice), but no nit, nfev etc, even fun does not make sense (because there are infinitely many very different functions that are all minimized, quite a remarkable property).

One (last?) point that was asked for by reviewers is whether the user facing function should return an OptimizeResult instead of a tuple of arrays.

You mean this suggestion: https://github.com/scipy/scipy/pull/17722#pullrequestreview-1548958684
Agreed, this might be a good idea indeed, if only to make it more forward-facing. If at some point we find we want to add an extra return, it would be super-easy if the return value is an object. Otherwise we'd have to reuse tuple bunches from scipy.stats and those are not very nice.

I went with attributes x, weights and blocks.

@lorentzenchr do you want to squash the history into meaningful separate commits? Otherwise, I'm going to squash-merge soon (tm).

Looks like a small bump on one of the test tolerances is required

I‘ll have a look at the test tolerances.
@ev-br Don‘t you squash-merge by default? In that case no action on my side is needed.

Thanks for all the reviews and help to bring it over the finish line.


Improving haplotypes and primary/alternate genomes

Hello dear developers,
I have run PECAT on a diploid plant genome (heterozygosity=3.3) and obtained a primary (4.7Gb) and alternate (2.6Gb) set and haplotype1 (4.7Gb) and haplotype2 (2.8Gb). Do you think there is a possibility of improving the size of the genomes by moving some parameters?
I will greatly appreciate your response.
My script is this:
project=plant
reads= $READS
genome_size=<PHONE_NUMBER>
threads=20
grid=local
cleanup=1

prep_min_length=3000
prep_output_coverage=50

corr_iterate_number=1
corr_block_size=8000000000
corr_filter_options=--filter0=:al=5000:alr=0.5:aal=8000:aalr=0.5:oh=2000:ohr=0.2
corr_correct_options=--score=weight:lc=10 --aligner diff --filter1 oh=1000:ohr=0.01 --candidate n=300:f=20
corr_rd2rd_options=-x ava-pb -f 0.005 -I 20G
corr_output_coverage=50

align_block_size=4000000000
align_rd2rd_options=-X -g3000 -w30 -k19 -m100 -r500 -I 20G -f 0.005
align_filter_options=--filter0=l=3000:al=3000:alr=0.5:aalr=0.5:oh=1000:ohr=0.1 --task=extend --filter1=oh=100:ohr=0.01

asm1_assemble_options= --max_trivial_length 10000

phase_rd2ctg_options=-x map-pb -c -p 0.5 -r 1000
phase_use_reads=1
phase_phase_options= --phase_options icr=0.2
phase_filter_options= --threshold=1000

asm2_assemble_options= --max_trivial_length 10000 --contig_format dual,prialt

polish_use_reads=1
polish_map_options = -x asm20
polish_filter_options = --filter0 oh=1000:ohr=0.1
polish_cns_options =

@Karenmagh  Sorry for taking so long to get back to you!
The dataset is high heterozygosity.  PECAT failed to pair some contigs according to their haplotypes.  --contig_dup_rate=0.2 may work. Its default value is 0.3. The config can be modified as follows.
asm2_assemble_options= --max_trivial_length 10000 --contig_format dual,prialt --contig_dup_rate=0.2

Hello @lemene,
Thank you very much for your response, I have tried --contig_dup_rate=0.2 and my haplotypes adjusted a little more, I also tried 0.15 and now I have haplotype 1 of 4Gb and haplotype 2 of 3.2Gb, could you help me explain a little about the management appropriate of this parameter? Is there a problem if I lower it more?
I will greatly appreciate your help.

Hi @Karenmagh I'm glad this parameter is working.
If heterozygosity is low, two reads from different haplotypes might map to the same position in the initial assembly (3-assemble/primary.fasta).  PECAT can identify that the overlap between them is inconsistent using the SNP alleles. The pair of the reads is inconsistent. --contig_dup_rate=0.2 means  if 20% of the reads that make up one contig are inconsistent with the reads of another contig, then one contig will be placed in 5-assemble/primary.fasta and the other in 5-assemble/alternate.fasta.  If heterozygosity is high, reads from different haplotypes may not overlap, making it difficult for PECAT to identify inconsistent read pairs. Therefore, it is necessary to reduce this parameter. But I'm unsure of the outcome if it is set too low.

Hello @lemene,
Thank for you support,  I have two question more,
--diploid_rate=0.2 and
contig_dup_rate=0.2 is the same?
Regarding, --min_identity,  What does this parameter do?

Hi @Karenmagh  diploid_rate and contig_dup_rate have the same meaning, but diploid_rate is no longer in use. min_identity is the threshold. The overlap similarity used to construct the string graph must not be less than this value.


SqlServerDsc: Remove obsolete examples

Pull Request (PR) description

Changes to SqlServerDsc

The complete examples was removed. They no longer was accurate and some
referenced resources that no longer existed. Accurate examples can be found
in each specific resource example folder. Examples for installing Failover Cluster
can be found in the resource examples folders in the xFailOverCluster
resource module (issue #462).
A README.md was created under the examples folder to be used as reference how
to installed certain scenarios (issue #462).
Removed the local specific common test for compiling examples in this repository
and instead opt-in for the common test in the DscResource.Tests repository (issue #669).

This Pull Request (PR) fixes the following issues:
Fixes #462
Fixes #669
Fixes #373
Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[x] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[x] Examples appropriately updated?
[x] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

This change is

Codecov Report

Merging #976 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #976   +/-   ##
===================================
  Coverage    96%    96%
===================================
  Files        32     32
  Lines      3537   3537
===================================
  Hits       3417   3417
  Misses      120    120


Improves and fixes some test cases.

Adds float support (overrides some doubles that are not supported by nF default firmware)
Moves deserialize classes to private (where they should be).
Adds temporary workaround to ignore non properties in root class.
Fixes DateTime issues (updates corlib).

How Has This Been Tested?
Uses test application in lib.
Types of changes

[x] Improvement (non-breaking change that improves a feature, code or algorithm)
[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

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


Bug when saving

Hello,
Nice plugin but I have a problem when i want to save multi fields :
 Json::make("Attributes2", [
                Text::make(__('field1')),
                Text::make(__('field2')),
                Text::make(__('field3')),
            ]),

Form result :

When saving in database field:
{"0":"{\"0\":\"{\\\"field1\\\":\\\"value1\\\"}\",\"field2\":\"value2\"}","field3":"value3"}

I'm using Laravel 6 and Nova 2.10.
Thanks for help.
Bye.

Hi; you should cast your "Attribute2" attribute into JSON


202 op title

Resolves hydroserver2/hydroserver#202
I created this branch from the @200-test branch because I added sorting for the datastream form selection boxes in 200 that I wanted to use for this issue.
Checked that the formatted names can be as long as the enforced 200 maximum characters. If all three parts of the combined title are long, then the selection box will ... when it hits the full width of the page.

An image of normal use, sorted by the formatted title:

@horsburgh I added a tooltip displaying the full observed property title after a 1.5 second delay when the user hovers over one of the list items.

I like it. I think the tooltip solves the issue where there might be two observed properties that look the same when they get shortened based on the size of the drop down.


Some warnings in light example (CON-240)

Just cloned this respository and built light example. Build is successful but there are some warnings:
Are these from connectedhomeip and can these be fixed?
/Users/andy/esp/esp-matter/components/esp_matter/esp_matter_event.cpp:21:20: warning: 'TAG' defined but not used [-Wunused-variable]
 static const char *TAG = "esp_matter_event";
                    ^~~
[517/632] Building CXX object esp-idf/esp_matter/CMakeFiles/__idf_esp_matter.dir/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/EventManagement.cpp.obj
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/EventManagement.cpp: In member function 'CHIP_ERROR chip::app::EventManagement::LogEventPrivate(chip::app::EventLoggingDelegate*, const chip::app::EventOptions&, chip::EventNumber&)':
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/EventManagement.cpp:410:23: warning: 'writer' is used uninitialized in this function [-Wuninitialized]
     CircularTLVWriter checkpoint = writer;
                       ^~~~~~~~~~
[545/632] Building CXX object esp-idf/esp_matter/CMakeFiles/__idf_esp_matter.dir/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/util/attribute-storage.cpp.obj
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/util/attribute-storage.cpp: In function 'void emberAfEndpointConfigure()':
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/util/attribute-storage.cpp:117:40: warning: comparison is always true due to limited range of data type [-Wtype-limits]
     static_assert(FIXED_ENDPOINT_COUNT <= std::numeric_limits<decltype(ep)>::max(),
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/util/attribute-storage.cpp:142:21: warning: comparison is always false due to limited range of data type [-Wtype-limits]
     for (ep = 0; ep < FIXED_ENDPOINT_COUNT; ep++)
[578/632] Building CXX object esp-idf/esp_matter/CMakeFiles/__idf_esp_matter.dir/Users/and...connectedhomeip/connectedhomeip/src/app/clusters/door-lock-server/door-lock-server.cpp.ob
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/door-lock-server/door-lock-server.cpp: In member function 'bool DoorLockServer::HandleRemoteLockOperation(chip::app::CommandHandler*, const chip::app::ConcreteCommandPath&, chip::app::Clusters::DoorLock::DlLockOperationType, RemoteLockOpHandler, const chip::Optional<chip::Span<const unsigned char> >&)':
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/door-lock-server/door-lock-server.cpp:3348:38: warning: '*((void*)& pinCredIdx +2)' may be used uninitialized in this function [-Wmaybe-uninitialized]
         foundCred[0].credentialIndex = pinCredIdx.Value();
         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~
[583/632] Building CXX object esp-idf/esp_matter/CMakeFiles/__idf_esp_matter.dir/Users/and...atter/connectedhomeip/connectedhomeip/src/app/clusters/level-control/level-control.cpp.ob
In file included from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/ConcreteAttributePath.h:23,
                 from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/cluster-objects.h:27,
                 from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/attributes/Accessors.h:27,
                 from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/level-control/level-control.cpp:23:
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/lib/core/Optional.h: In function 'void emberAfOnOffClusterLevelControlEffectCallback(chip::EndpointId, bool)':
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/lib/core/Optional.h:72:13: warning: '*((void*)& transitionTime +2)' may be used uninitialized in this function [-Wmaybe-uninitialized]
             new (&mValue.mData) T(other.mValue.mData);
             ^~~
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/level-control/level-control.cpp:1074:40: note: '*((void*)& transitionTime +2)' was declared here
     app::DataModel::Nullable<uint16_t> transitionTime;
                                        ^~~~~~~~~~~~~~
[595/632] Building CXX object esp-idf/esp_matter/CMakeFiles/__idf_esp_matter.dir/Users/and...eip/src/app/clusters/operational-credentials-server/operational-credentials-server.cpp.ob
In file included from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/ConcreteAttributePath.h:23,
                 from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/cluster-objects.h:27,
                 from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/attributes/Accessors.h:27,
                 from /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/operational-credentials-server/operational-credentials-server.cpp:25:
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/lib/core/Optional.h: In function 'bool emberAfOperationalCredentialsClusterCSRRequestCallback(chip::app::CommandHandler*, const chip::app::ConcreteCommandPath&, const chip::app::Clusters::OperationalCredentials::Commands::CSRRequest::DecodableType&)':
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/lib/core/Optional.h:72:13: warning: '*((void*)& fabricIndexForCsr +1)' may be used uninitialized in this function [-Wmaybe-uninitialized]
             new (&mValue.mData) T(other.mValue.mData);
             ^~~
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/operational-credentials-server/operational-credentials-server.cpp:1054:31: note: '*((void*)& fabricIndexForCsr +1)' was declared here
         Optional<FabricIndex> fabricIndexForCsr;
                               ^~~~~~~~~~~~~~~~~
[617/632] Building CXX object esp-idf/esp_matter/CMakeFiles/__idf_esp_matter.dir/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/window-covering-server/window-covering-server.cpp.obj
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/window-covering-server/window-covering-server.cpp: In function 'bool chip::app::Clusters::WindowCovering::IsPercent100thsValid(chip::Percent100ths)':
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/window-covering-server/window-covering-server.cpp:42:51: warning: comparison is always false due to limited range of data type [-Wtype-limits]
 #define CHECK_BOUNDS_INVALID(MIN, VAL, MAX) ((VAL < MIN) || (VAL > MAX))
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/window-covering-server/window-covering-server.cpp:43:45: note: in expansion of macro 'CHECK_BOUNDS_INVALID'
 #define CHECK_BOUNDS_VALID(MIN, VAL, MAX) (!CHECK_BOUNDS_INVALID(MIN, VAL, MAX))
                                             ^~~~~~~~~~~~~~~~~~~~
/Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/src/app/clusters/window-covering-server/window-covering-server.cpp:341:9: note: in expansion of macro 'CHECK_BOUNDS_VALID'
     if (CHECK_BOUNDS_VALID(WC_PERCENT100THS_MIN_OPEN, percent100ths, WC_PERCENT100THS_MAX_CLOSED))
         ^~~~~~~~~~~~~~~~~~
[628/632] Generating ld/sections.ld
warning: ENABLE_ESP32_LOCATIONCAPABILITY (defined at /Users/andy/esp/esp-matter/connectedhomeip/connectedhomeip/config/esp32/components/chip/Kconfig:660) has leading or trailing whitespace in its prompt
[631/632] Generating binary image from built executable

/Users/andy/esp/esp-matter/components/esp_matter/esp_matter_event.cpp:21:20: warning: 'TAG' defined but not used [-Wunused-variable]
static const char *TAG = "esp_matter_event";
This warning is from esp-matter.
Other warnings are from connnectedhomeip.

We have fixed the warning from esp-matter.
Most of the warnings from  connectedhomeip are initialization warnings. For fixing them,we need to initialize the attributes which are Optional and Nullable.
But,initializing Optional and Nullable is wrong. The whole point of these classes is to be able to distinguish whether a given value is specified or not, so initializing them to 0 will likely change the behaviour of the code.
The one single warning which can be removed is the trailing whitespace warning for ENABLE_ESP32_LOCATIONCAPABILITY  in Kconfig file and that has been fixed.

Understood your points. Thanks!


Add svelte-check to the pre-commit hook

Currently, our pre-commit hook checks for ESLint issues and formatting problems. However, TypeScript errors go through freely. Remedy that by adding a pnpm exec svelte-check [...] to the .husky/pre-commit file.
P.S. Also an embarrassing detail: our pre-commit hook doesn't even run currently because it's not marked as executable. Please, mark it as executable while you're at it.

our pre-commit hook doesn't even run currently because it's not marked as executable

Are you referring to the x bit of the file permissions on Linux? If so, someone else would need to do that since I use Windows and it is working fine 😁

Are you referring to the x bit of the file permissions on Linux?

Yes


MCFD Personals Requests - Tagging and Scanning States

As an IAO MCFD User
I want to be able to flag requests for tagging or scanning
so that I can better understand what actions need to be taken on requests

Assumptions & Scope
This story is a need for the MCFD team to be able to better identify where their specific requests are at once physical copies of the records are received.
In this story, unlike the typical flow, IAO users will be able to update the request state from Call For Records to the relevant state to flag the next stag of processing. This will be specific to CFD and MSD requests only, as these are the only two Ministries with scanning activities for their records.
IAO will also be able to update the request state to "Records Review" from the scanning states, as scanning is often the final activity prior to the packages being ready to review.
Note: The CFR countdown clock will stop counting down when it moves to scanning or tagging
What is IN scope?
State change updates specific to CFD and MSD Personal requests
What is NOT in scope?
Any other Ministry
Acceptance Criteria
Scenario 1: MCFD or MSD Personal Request in Call For Records

GIVEN an IAO user is on a MSD or MCFD Personal Request in Call For Records
WHEN they activate the state change drop down
THEN they will see the following options: Open, Tagging, Ready to Scan, and Closed

Scenario 2: CFD/MSD Personal Request - Tagging

GIVEN an IAO user is on a MSD or MCFD Personal Request in Call For Records
AND the state change drop down is active
WHEN they click on 'Tagging'
THEN the state change modal will appear

Scenario 3: CFD/MSD Personal Request - Tagging - Activate Continue

GIVEN an IAO user is on a MSD or MCFD Personal Request in Call For Records
AND the state change modal for tagging is active
WHEN they click on 'Continue'
THEN the state change modal will disappear
AND the state of the request will update to tagging
AND the CFR countdown will no longer show on the left-nav tab

Scenario 4: Tagging - State Change Options

GIVEN an IAO user is on a MSD or MCFD Personal Request in Tagging
WHEN they activate the state change drop down
THEN they will see the following options: Call for Records, Ready to Scan, Records Review and Closed

Scenario 5: CFD/MSD Personal Request - Ready to Scan

GIVEN an IAO user is on a MSD or MCFD Personal Request in Call For Records or Tagging
AND the state change drop down is active
WHEN they click on 'Ready to Scan'
THEN the state change modal will appear
AND the CFR countdown will no longer show on the left-nav tab

Scenario 6: CFD/MSD Personal Request - Ready to Scan - Activate Continue

GIVEN an IAO user is on a MSD or MCFD Personal Request in Call For Records
AND the state change modal for 'Ready to Scan' is active
WHEN they click on 'Continue'
THEN the state change modal will disappear
AND the state of the request will update to 'Ready to Scan'
AND the request will auto-assign to the IAO Scanning Team

Scenario 7: Ready to Scan - State Change Options

GIVEN an IAO user is on a MSD or MCFD Personal Request in Ready to Scan
WHEN they activate the state change drop down
THEN they will see the following options: Call for Records, Tagging, Records Review and Closed

Scenario 8: Records Review - State Change Options

GIVEN an IAO user is on a MSD or MCFD Personal Request in Records Review
WHEN they activate the state change drop down
THEN they will see the following options: Call for Records, Tagging, and Closed

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

Took a run at these scanning/tagging states - tried to keep it relatively flexible so that states can be changed more easily. Note - these states will be limited to MSD and CFD Personal requests only. I added the last scenario to account for the potential of a file being strictly electronic records, Ministry would update to records review, and analyst can determine if e-tagging is required and flip it to tagging.

cc @JHarrietha-AOT @lmullane

Looks good, Matt.
Do we need to run this by anyone or are we good to proceed? Might be nice to loop in the processing and Intake managers.

CFR Clock information required

Tech Info

FE
BE

Migration script
WF behavior check

CFR clock will be keep running.

@JHarrietha-AOT, will need colour codes for the new states.

We need to update the ACs to show the new request states in the ministry queue and the IAO queue.
cc @m-prodan

Passed UX Assurance

@abin-aot When we move the request to 'Ready to scan' then request is not auto assigning to ' Scanning team'. Could you please check?
Private Zenhub Image

note from release planning - there is no need to put request on hold, assumption was more around the CFR countdown will no longer appear on left-nav tab, I have updated AC to make this more clear

@abin-aot Hey! All AC pass with the exception of scenario 8 where tagging needs to be added as a potential state change.
Private Zenhub Image

[Uploading... image.png]
Updated @liseandtea

Private Zenhub Image
Updated @liseandtea

Closing ticket - all AC pass.


feat: the log widget and the trace widget associate with each other, remove log tables on the trace widget

The log widget and the trace widget associate with each other, remove log tables on the trace widget.
Video
https://user-images.githubusercontent.com/20871783/181197768-0faacf83-b23e-496e-83d6-0a57c60390f7.mov
Signed-off-by: Qiuxia Fan<EMAIL_ADDRESS>

Please sync, and let's deploy it on demo.


JSONWidget updated with null value fix

Problem
JSONWidget gives error when there are null values due to ast.literal_eval which fails to evaluate null as values.
Solution
By default, now it will use json to loads and dumps. However if the json string has single quotes instead of double, it will first replace the single quotes to double quotes and then loads.
Acceptance Criteria
Tests have included for such cases.

Coverage increased (+0.009%) to 95.668% when pulling e82a7fe17d1c4d9bdbf2bf6379e222153fb96aff on shubhamdipt:master into 77816e4e6ee85b82e44d0769e37c76b0f948fcdf on django-import-export:master.

LGTM 👏
The only thing that I would say is missing is to expand the documentation. Any way you could add a couple of lines explaining this behavior? I think it would suffice to have the behavior summarized as discussed in the previous PR.
What do you think?

Thanks for your input. I added a couple of lines as docstring mentioning the two cases the current solution covers.

LGTM 👏
The only thing that I would say is missing is to expand the documentation. Any way you could add a couple of lines explaining this behavior? I think it would suffice to have the behavior summarized as discussed in the previous PR.
What do you think?

Thanks @shubhamdipt ! 👏


Finding loop in a linked list

Description
I want to make documentation of finding the loop in a linked list
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Guide on keeping app memory footprint low

Should I minify my JS? What type of optimization to do for assets? How many WebViews can I preload? And so forth.
(Migrated comment from @Leeds-eBooks:)

An obvious one (but may be worth mentioning, given that jQuery is included by default) is that Zepto.js can be a good alternative in certain use cases, and that only jQuery 2.x is needed, not 1.x. And that jQuery doesn't necessarily have to be used, as Steroids.js doesn't use it.
For myself, I'd like to know whether g-zipping is a good or a bad idea. I assume the former, but I know better than to assume anything in this field!

yeah, very very interesting!


Adds header based authentication to the mailbox routes.

Current dependencies on/for this PR:

main

PR #15

PR #16

PR #17  (this PR)

This comment was auto-generated by Graphite.


[Question❓] pymatting is slow running on A100

Hello all, very cool project. Just want to double check if pymatting is downloading any models on demand when the first time importing this library. For some reason, it takes ~30s to import these three lines on a clean Docker Build:
    from pymatting.alpha.estimate_alpha_cf import estimate_alpha_cf
    from pymatting.foreground.estimate_foreground_ml import estimate_foreground_ml
    from pymatting.util.util import stack_images

What did I miss? Thank you!

PyMatting uses the Numba library for JIT compilation. All JIT-compiled functions will be cached to make repeated calls are faster. I can think of three options to avoid repeated JIT compilation:

You could try preserving the cache directory. You can set the environment variable NUMBA_CACHE_DIR to specify its location manually. https://numba.pydata.org/numba-doc/latest/reference/envvars.html#envvar-NUMBA_CACHE_DIR
You could import the library when building the docker container. Make sure to import and run as the same user with the same hardware.
You could wrap PyMatting as a web service and keep the container alive forever. This way, you do not repeatedly import it.

You also mentioned that you are using an A100 GPU. Currently, only a few foreground estimation methods can make use of GPUs, for example estimate_foreground_ml_cupy if you install CuPy.

https://pymatting.github.io/foreground.html#estimate_foreground_ml_cupy
https://github.com/pymatting/pymatting/blob/master/pymatting/foreground/estimate_foreground_ml_cupy.py

However, CuPy is very slow on first import. I do not know whether it can be cached.

Thank you!

Glad I could help!
If those solutions should turn out to not work for you, we could think about other ways to solve it. For example, I considered replacing Numba with https://github.com/taichi-dev/taichi or a different JIT-compiler, but that would require rewriting most of PyMatting and might introduce backwards-compatibility issues, so I am not sure whether it is worth the effort.


Support the new notion of 'best' and not 'hot' being the default sort

Selecting 'hot' now merely shows 'best' and not actually 'hot' sort, and the default sort is still known as 'hot' instead of 'best'. The user should be able to actually use 'hot' sort.

#619


Add execl* functions on Windows

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

r=me with the style issue is fixed.

r=JohnTitor


wip: fix markergroup

I'd forgotten to push this commit.
Don't merge now, it's wip

Coverage decreased (-0.07%) to 58.861% when pulling 98ed6b33387bff2addb871ebf0dd5abb0d0e96b8 on mkborregaard:plotting_improvements into f5a09eaaba91b6c28129e2b7b3c113e98b9265bb on richardreeve:master.


Implicit Datasource Access for Dashboard RBAC

Hi all, I have done some reading on Dashboard Access Control and I can conclude from what I gather, that Dashboard access can now be automatically added to roles based on what you set in the UI, courtesy of https://github.com/apache/superset/pull/12680
It is noted that [SIP-51] is closed via https://github.com/apache/superset/pull/12680, addressing half of the issue.
However, this does not yet address the datasource/dataset access control with respect to Dashboard Access. I found this comment by @amitmiran137
https://github.com/apache/superset/pull/12680#issuecomment-770337622

we plan in a follow-up PR that user will be given temporary read access to all of the datasets within a dashboard as was mentioned in the development milestones in #10408

Is there an issue/PR for this yet? Please point me to it if there is already one! :)
Use Case:

Create a user role that has read-only access to dashboards, without access to the individual records in the underlying data (i.e. nothing shows up on Datasets menu)
Successful implementation would look like this: a user role could be set to have ONLY access to Dashboards, and not any datasets in the menu, yet dashboard is still able to render charts without permission errors.
This limits the user to accessing only what is set up for them via the Dashboard.
Filtering access should still be available.

As mentioned in https://github.com/apache/superset/issues/10408#issue-664324346

Airbnb and probably other orgs are fully dependent on dataset level access - they would not handle an extra dashboard level permission
In our org we will have hundreds of dashboards that will be based on the same dataset
Therefore there is no way to manage dashboard access for specific dashboards for specific users

Related issues:
[SIP-51]: https://github.com/apache/superset/issues/10408
https://github.com/apache/superset/issues/11198
https://github.com/apache/superset/issues/12910
https://github.com/apache/superset/issues/1799
https://github.com/apache/superset/issues/4127
https://github.com/apache/superset/issues/14260
etc.

Pretty sure this is implemented already. Have you tried enabling the feature flag DASHBOARD_RBAC?
https://github.com/apache/superset/pull/13992

You can find all the PRs related to this feature here: https://github.com/apache/superset/pulls?q=is%3Apr++dashboard_rbac

Yes , this is implemented as mentioned here

Thank you, that works like a charm! Before I close, can I ask where is the best source of documentation for superset? I'd prefer referring to that instead of trawling through the issue section or opening a ticket here unnecessarily :)
Can this DASHBOARD_RBAC feature flag be found here: https://superset.apache.org/docs/intro ? I wasn't able to locate it.

Documentation for this feature can be found here: https://superset.apache.org/docs/creating-charts-dashboards/first-dashboard#manage-access-to-dashboards
info about the various feature flags can be found here: https://github.com/apache/superset/blob/master/RESOURCES/FEATURE_FLAGS.md

Unfortunately, this feature breaks my dataset-based access. I can either use dashboard level OR dataset level. I've ran a db upgrade after setting the FF as well. What went wrong? (I am on 1.4.1)

Same here! I cannot have dataset level and dashboard level. I got a Forbidden Access if both are set up


Modifications on HashMap and HashSet

Changed the inner representation of entries in HashMap
Added the Keys() and Values() methods to both HashMap and HashSet data structures
Did benchmarks on the results and compared my fork with yours. This repository was used with custom modifications. Found that they did not differ in time and I think my implementation extends the interface making it a bit more useful.

Thanks you for your time.

What's the reason for changing the inner representation of entries? I think the addition of the Keys and Values methods is fine, but I'm not sure the other changes are necessary.


Moving subfields of array field mix their values

Describe the bug
When the input is still idle (I mean undefined) and you try to change its position it takes the value of other subfields. It messes things up.
Codesandbox
https://codesandbox.io/p/sandbox/focused-pond-s96spf?file=%2Fsrc%2FApp.js%3A169%2C8
Steps to reproduce

Try to move the empty subfields dragging and dropping in other position
Now try to move subfields with non-empty values

Expected behavior
I expect it to keep its value.

I think this is a bug with our moveValue method. I'll investigate in a bit

I think this is a bug with our moveValue method. I'll investigate in a bit

If it saves a little time, after some research, I wrote a test to check whether there was a problem with the moveValue function, but the test went well.
  it('should correctly move an array value, even with undefined values', () => {
    const form = new FormApi({
      defaultValues: {
        users: [
          {
            name: undefined,
          },
          {
            name: 'one',
          },
          { name: 'two' },
          {
            name: 'three',
          },
          {
            name: undefined,
          },
        ],
      },
    })

    const field = new FieldApi({
      form,
      name: 'users',
    })

    field.moveValue(3, 4)

    expect(field.getValue()).toStrictEqual([
      {
        name: undefined,
      },
      { name: 'one' },
      { name: 'two' },
      {
        name: undefined,
      },
      {
        name: 'three',
      },
    ])
  })

I wonder if the problem isn't the collision between an uncontrolled and a controlled field, and that screws up the ui.

Hey @Balastrong, was this fixed when you updated your move and insert stuff?

Unfortunately I think there's still something not behaving here. I made a short example: https://stackblitz.com/edit/tanstack-form-grdmqr?file=src%2Findex.tsx&preset=node
First issue is that indeed having <input value={subField.state.value} ... /> can't work if value changes from string to undefined and vice-versa when moving values, but this may be on React and subField.state.value ?? '' can help.
The second issue is that if we:

Change the text in the input from "one" to "onea"
Click the "move" button

The form state ends up with:
{"people":[{"name":"onea"},{"name":"one"},...]}
Instead of
{"people":[{"name":"onea"},{"name":undefined},...]}
It seems that the move operation (actually, I think the underlying setBy) didn't put undefined on a defined value, hence 'one' remained there.

This test on react-form fails because magically the second inputs gets back the value 'one'
The same operation (move then submit) on form-core passes.
it('should correctly move an array value, even with undefined values and not alter the state after submit', async () => {
    function Comp() {
      const form = useForm({
        defaultValues: {
          people: [
            {
              name: undefined,
            },
            {
              name: 'one',
            },
          ],
        },
      })

      return (

          <form
            onSubmit={(e) => {
              e.preventDefault()
              e.stopPropagation()
              form.handleSubmit()
            }}
          >
            <form.Field name="people" mode="array">
              {(field) => {
                return (

                    {field.state.value.map((_, i) => {
                      return (
                        <form.Field key={i} name={`people[${i}].name`}>
                          {(subField) => {
                            return (

Name for person {i}

                                  <input
                                    value={subField.state.value ?? ''}
                                    onChange={(e) =>
                                      subField.handleChange(e.target.value)
                                    }
                                  />

                            )
                          }}
                        </form.Field>
                      )
                    })}
                    <button type="button" onClick={() => field.moveValue(1, 0)}>
                      Move 1 to 0
                    </button>

                )
              }}
            </form.Field>
            <form.Subscribe
              selector={(state) => [state.canSubmit, state.isSubmitting]}
              children={([canSubmit, isSubmitting]) => (
                <>
                  <button type="submit" disabled={!canSubmit}>
                    {isSubmitting ? '...' : 'Submit'}
                  </button>
                </>
              )}
            />
          </form>

      )
    }

    const { getByText, findByLabelText, queryByText, findByText } = render(
      <Comp />,
    )

    const input = await findByLabelText('Name for person 0')
    expect(input).toBeInTheDocument()
    expect(input).toHaveValue('')

    const input2 = await findByLabelText('Name for person 1')
    expect(input2).toBeInTheDocument()
    expect(input2).toHaveValue('one')

    await user.click(getByText('Move 1 to 0'))

    expect(input).toHaveValue('one')
    expect(input2).toHaveValue('')

    await user.click(await findByText('Submit'))

    expect(input).toHaveValue('one')
    expect(input2).toHaveValue('') // This gets value 'one'
  })

I have run into this similar, if not same problem. I have an array of objects that represent other sub-fields of various depths. The internals of moveFieldValues did not work, throwing an error when attempting to slice off the fromIndex value. IIRC, the error was attempting to delete fromIndex from object Array which was a very strange error. I got around that by using [arrayMove utility from @dnd-kit/sortable])(https://github.com/clauderic/dnd-kit/blob/master/packages/sortable/src/utilities/arrayMove.ts). My only guess as to why this works is that it is making a copy of the array before performing the splice operations?
Once that was worked around, I noticed that if one of the sub-fields that has an undefined value is repositioned with a sibling that does have a defined value, the sub-field will assume that siblings value.
If I do not render the FieldApi instance, then the value does not get "absorbed".
If I ensure that the value is an empty string rather than `undefined, then the value does not get "absorbed"
I think the problem is that while the internal form store state and field instance states reflect the desired overall state of the form, the FormApi#fieldInfo values do not get updated along with the store state so everything gets "merged" on the next render and the undefined value is now defined from its sibling. 🤷♂


Pitch transform harmonics 5th produces incorrect result [BUG]

Describe the bug
This function transposes the sounding pitch down 2 octaves and then the touch note a fifth below that. The correct way to achieve the sounding pitch is to transpose it down 1 octave and then the touch note a fifth below that.
To Reproduce
Steps to reproduce the behavior:

Open a file with notes
Run Pitch Transpose Harmonics 5th script
See that the resulting harmonic sounds an octave lower than the original pitch selected.

Expected behavior
The result should be an octave higher.
Additional context
I found this while working on new transposition function. My plan is to submit a fix as part of that project, unless there is a concern about it.

I believe you fixed this with https://github.com/Nick-Mazuk/jw-lua-scripts/pull/59, so I'm closing the issue.
Feel free to reopen if that's not the case.
P.S. On GitHub, if a PR fixes an issue, you can make the issue automatically close when the PR is merged. For this, in the PR, just add close [issue-url]. For instance in this case:
close https://github.com/Nick-Mazuk/jw-lua-scripts/issues/58


OSError: [Errno 24] Too many open files

@tmack8001 is familiar with this.
I started having this issue after the TILT/NGINX/PicoStill changes recently. I ended up reverting to 41b2c83 and climbing each commit until I found the one that caused the issue. After slowly climbing the tree and giving it a few hours to hit the limit, I found that 4c7e112 is the one that causes this issue for me.
EDIT: I will add that when this issue arises, the entire picobrew server goes down. Also, when I was trying to just get it stable, I ended up raising my limit from 1024 to 8192 and still had this issue. That is when I reverted and started trying to find the bad commit. For trying to find the bad commit, I did revert back to 1024.

https://github.com/chiefwigms/picobrew_pico/commit/4c7e11244836ea0e9f7373dc2a758abc525bda3b can't be the commit that causes issues as that nginx change isn't even loaded by your server yet (as I hadn't made the modification to the git post script hook we added in a later commit). This is just staging for that so isn't active at all.
Must be something else.

To get more information here I'd like to get the number of open file descriptors at various times on your raspberrypi as related and coresponding with the python3 server.
$ ps aux | grep "python3
server.py"

root      7255  0.0  5.0<PHONE_NUMBER>0 ?        Sl   Mar16   4:01 python3 server.py <IP_ADDRESS> 80
pi       14448  0.0  0.0   7480   552 pts/0    S+   11:58   0:00 grep --color=auto python3 server.py

(note in the above "$" denotes where the command is vs the example output that follows the command)
After this command locate the "pid" (that first number after the username running the process). To locate the python3 server look for a line similar (not identical to) this one root      7255  0.0  5.0<PHONE_NUMBER>0 ?        Sl   Mar16   4:01 python3 server.py <IP_ADDRESS> 80 (the first in the above example output)
sudo ls -l /proc/<pid>/fd | wc -l

The above command will take that pid that was located in the first step and list out while doing a line count (each open file descriptor is on a line followed by a newline character). I'd be interested in how this count grows over time as this mostly.
Also would be interesting to get the list of actual file descriptors. For instance in my server I have a few open file descriptors to active sessions:
(below contains a subset of open file descriptors to the active devices that are logging to the server)
sudo ls -l /proc/7255/fd | grep session
l-wx------ 1 root root 64 Mar 21 04:10 3 -> /picobrew_pico/app/sessions/ferm/active/20210127_211301#2c3ae834b45b.json
l-wx------ 1 root root 64 Mar 21 04:10 40 -> /picobrew_pico/app/sessions/tilt/active/20210316_160024#Blue-C7F9F4A8651E.json

Just blew up, most of the descriptors are "pipe:[XXXXXX]" , but some are "socket:[XXXXXX]" (X = number). Lived ~ 3 hours on commit 4c7e112, I'm going to back down to bc15114 for now. I'll check on it after ~ 3hours to see where its number is.

Very odd, I went back to bc15114 and just saw it at 1021 and everything was good.. Until I went to about, then it blew up. Leaving about brings it back down to 1021 and allows the main page to show. But I bet it wouldn't hold a session from a device right now

Approaching an hour on 283f964 and have a solid 6 with no fluctuations.. It has to be a2cfa03 or bc15114

What I think is definitely happening here is that the Tilt scanning thread is spawning a new thread each and every time it async does work. What could make this better is just to forget the async nature and just process everything on a dedicated thread in a while session.active: loop instead of passing off to asyncio. Which would at least be similar to how the still polling is working.
I'll play around with it @cgalpin to see if this would work better.

@Intecpsp though if that is the case it should start right at https://github.com/chiefwigms/picobrew_pico/commit/bc151140ef0d8dbe59644cf57a552920f7427a5c or even the commit before if one manually sets up bluetooth and the config.yaml changes to activate the tilt processing threads.

I'm sure it's the tilt thread. I had initially tried to get it t use the current thread but had problems. Using asyncio.run worked, and did not see this issue, sorry. I will take a look as well.

No worries @cgalpin over having cold brew tonight I figured out a solution to this. Keeping the while True: essentially within the spawned async thread vs outside. Though made a few other changes in the process.

Thanks for fixing it. I like your solution!

I can confirm that after ~1 hour, I'm only showing 15 while running 98bb272 and tilt_monitoring: true. Looks like it's working!


Buildthedocs add versioning

Adds the ability to use the versioning system even when not on RTD.
Original patch by @1138-4EB
Closes #543

So, I'm -1 on these changes from a product perspective. I don't want to further generalize our theme and make maintaining this more work -- it's already a lot of work. If buildthedocs needs a special case to the theme, this project should extend it's own theme off of our theme. Adding more template blocks is something we can manage, but being responsible for the use case of an entirely different project is not on our roadmap.
For instance, we want to just remove this block and not be responsible for maintaining a list of versions in our theme -- this is the correct thing to do from Read the Docs perspective. Having a clash here is just going to be technical debt to us.
So sorry to close work here, but we discussed this use case and don't see the need for it. If we can add more template block extension points in our theme, that is probably fine. We just can't be responsible for downstream breakage.
F

I came upon this issue and would just share that I would have love to benefit from this feature (I was looking at this feature for QGIS Python API documentation to handle the different API versions).

if this interests anyone, this can be achieved by templating.
see https://stackoverflow.com/questions/49331914/enable-versions-in-sidebar-in-sphinx-read-the-docs-theme/63054018/#63054018


Adds D4 VASP modification

to support the research group of Dino Spagnoli.

@ddeeptimahanti @halfflat I cleaned up the PR code, please review.


⚠ Bofrost.de has degraded performance

In b6ffea7, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1437 ms

Resolved: Bofrost.de performance has improved in 2fe9561 after 20 minutes.


ImportError: cannot import name pem

I'm writting test case for my app hosted on Google app engine and when running the following command an import error occurs
nosetests -w tests --logging-level=ERROR --with-gae --gae-application=path_to_yaml_file --gae-lib-root=path_to_appengine
Following is the stacktrace of error message
Traceback (most recent call last): File "/home/daffodil/Desktop/museconnect/venv/local/lib/python2.7/site-packages/nose/loader.py", line 418, in loadTestsFromName addr.filename, addr.module) File "/home/daffodil/Desktop/museconnect/venv/local/lib/python2.7/site-packages/nose/importer.py", line 47, in importFromPath return self.importFromDir(dir_path, fqname) File "/home/daffodil/Desktop/museconnect/venv/local/lib/python2.7/site-packages/nose/importer.py", line 94, in importFromDir mod = load_module(part_fqname, fh, filename, desc) File "/home/daffodil/Desktop/museconnect/tests/api/test_groups.py", line 8, in <module> from main import app File "/home/daffodil/Desktop/museconnect/main.py", line 5, in <module> from apis import api_blueprint File "/home/daffodil/Desktop/museconnect/apis/__init__.py", line 4, in <module> from .clients import api as clients File "/home/daffodil/Desktop/museconnect/apis/clients.py", line 4, in <module> from model.museconnect import ( File "/home/daffodil/Desktop/museconnect/model/museconnect.py", line 2, in <module> from model.user import User File "/home/daffodil/Desktop/museconnect/model/user.py", line 10, in <module> from google.cloud import storage, exceptions File "/home/daffodil/Desktop/museconnect/venv/local/lib/python2.7/site-packages/google/cloud/storage/__init__.py", line 36,in <module> from google.cloud.storage.client import Client File "/home/daffodil/Desktop/museconnect/venv/local/lib/python2.7/site-packages/google/cloud/storage/client.py", line 19, in <module> from google.cloud.client import JSONClient File "/home/daffodil/Desktop/museconnect/venv/local/lib/python2.7/site-packages/google/cloud/client.py", line 18, in <module> from google.oauth2 import service_account File "/home/daffodil/Desktop/museconnect/appvendor/google/oauth2/service_account.py", line 77, in <module> from google.auth import _service_account_info File "/home/daffodil/Desktop/museconnect/appvendor/google/auth/_service_account_info.py", line 22, in <module> from google.auth import crypt File "/home/daffodil/Desktop/museconnect/appvendor/google/auth/crypt/__init__.py", line 39, in <module> from google.auth.crypt import rsa File "/home/daffodil/Desktop/museconnect/appvendor/google/auth/crypt/rsa.py", line 27, in <module> from google.auth.crypt import _python_rsa File "/home/daffodil/Desktop/museconnect/appvendor/google/auth/crypt/_python_rsa.py", line 25, in <module> from pyasn1_modules import pem ImportError: cannot import name pem
There are two pyasn1_modules... one in the google-cloud-sdk/platform/google_appengine/lib/ folder and other in appvendor folder of my app directory. The later one has pem.py file but the first one doesn't.
Tried to remove pyasn1_modules before adding appvendor from system path as mentioned in the following issue but this module is not present in sys.modules.
https://github.com/GoogleCloudPlatform/google-cloud-python/issues/2895

Hi there! These libraries are not supported on the App Engine standard 2.7 runtime. They are supported on the Python 3.7 runtime for standard.