UnicodeError: label empty or too long

I'm getting this error:
/lib/python3.6/encodings/idna.py", line 165, in encode
    raise UnicodeError("label empty or too long")
UnicodeError: label empty or too long

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "cloudfail.py", line 266, in <module>
    subdomain_scan(args.target)
  File "cloudfail.py", line 167, in subdomain_scan
    target_http = requests.get("http://"+subdomain)
  File "/tmp/env/lib/python3.6/site-packages/requests/api.py", line 72, in get
    return request('get', url, params=params, **kwargs)
  File "/tmp/env/lib/python3.6/site-packages/requests/api.py", line 58, in request
    return session.request(method=method, url=url, **kwargs)
  File "/tmp/env/lib/python3.6/site-packages/requests/sessions.py", line 502, in request
    resp = self.send(prep, **send_kwargs)
  File "/tmp/env/lib/python3.6/site-packages/requests/sessions.py", line 612, in send
    r = adapter.send(request, **kwargs)
  File "/tmp/env/lib/python3.6/site-packages/requests/adapters.py", line 440, in send
    timeout=timeout
  File "/tmp/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 600, in urlopen
    chunked=chunked)
  File "/tmp/env/lib/python3.6/site-packages/urllib3/connectionpool.py", line 356, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib/python3.6/http/client.py", line 1239, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.6/http/client.py", line 1285, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.6/http/client.py", line 1234, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib/python3.6/http/client.py", line 1026, in _send_output
    self.send(msg)
  File "/usr/lib/python3.6/http/client.py", line 964, in send
    self.connect()
  File "/tmp/env/lib/python3.6/site-packages/urllib3/connection.py", line 166, in connect
    conn = self._new_conn()
  File "/tmp/env/lib/python3.6/site-packages/urllib3/connection.py", line 141, in _new_conn
    (self.host, self.port), self.timeout, **extra_kw)
  File "/tmp/env/lib/python3.6/site-packages/urllib3/util/connection.py", line 60, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/lib/python3.6/socket.py", line 743, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
UnicodeError: encoding with 'idna' codec failed (UnicodeError: label empty or too long)```

Hi thanks for reporting this issue, can I just get a bit more info regarding this issue?
What domain are you trying to scan? Have you modified the subdomain list?

I'm running this command:
 python cloudfail.py --target geenstijl.nl --tor
I've not modified the subdomains list.

I also saw this error with the default data files from revision 125169c.
It turns out it fails when trying to request m..<target> (an invalid address, notice the two periods with an empty string between them). The m. entry is in the data/subdomains.txt file (line 1275), and as of that commit it's the only entry ending in a period.

Brilliant find @shello that is most likely the cause of this error (although I am not too sure why it seems to work sometimes) but I will most likely remove this subdomain in the next commit.

So... the solution is removing "m." at line 1275. I worked for me. Thanks

This should now be resolved as of commit a9ef9dc95e972ed7992b7db54fd1abd1406f6349


Possible memory leak with Greatwood Trees

Version: Thaumcraft-1.12.2-6.1.BETA17
It looks like greatwood trees work the same way as vanilla BigTree and suffer the same memory leak where references to worlds can hang around. You can see the bug report for the vanilla big tree's leak here: https://bugs.mojang.com/browse/MC-101232
And forge's patch for it here: https://github.com/MinecraftForge/MinecraftForge/blob/ab228832f24e53436d058a96ff9483376a917b02/patches/minecraft/net/minecraft/world/gen/feature/WorldGenBigTree.java.patch#L36

Yeah, greatwoods are just a modified version of the bigtree's. Thanks, I'll take a look


sct_label_vertebrae -discfile: adding top disc

Fix #2967
This PR aims at fixing the problem reported in #2967.
This modifies the code from function label_disc in spinalcordtoolbox/vertebrae/core.py. This function now adds a label for a disc at the top of the labeled segmentation.
There is a new workflow to produce sub-xxx_seg_labeled-disc.nii.gz which is:

from the straight space, fetch the z corresponding to each disc (you have it from the variable list_disc_z)
generate a single voxel label at this z location, centered in the cord
apply the un-straightening to these blobs using label interpolation with sct_apply_transfo (which dilate the label to be a 2x2x2 blob and uses nn interpolation afterward, and retrieve the center of mass of the blob)
this could create some changes in  sub-xxx_seg_labeled-disc.nii.gz. Some labels might be a voxel below their previous position.

This PR also introduce a new flag -clean-labels which takes a boolean argument.  When -clean-labels 1 is used, the output sub-xxx_seg_labeled will go through the clean_labeled_segmentation function.

@jcohenadad I slightly changed the workflow to this.

from the straight space, fetch the z corresponding to each disc (you have it from the variable list_disc_z)
generate a single voxel label at this z location, centered in the cord
apply the un-straightening to these blobs using label interpolation with sct_apply_transfo (which dilate the label to be a 2x2x2 blob and uses nn interpolation afterward, and retrieve the center of mass of the blob)
Do you think using trilinear interpolation instead of nn would be better? if so maybe we could change the label interpolation?

Looks like the new behaviour (fa3648dc1468d3587fe4ba87c07391b80dcd4f34) places the discs one slice below, compared to the old behaviour (5.0.0):

@lrouhier any reason for this new behaviour?

On the sct_example_data/t1, the first disc label (C1) appears 3 slices below the top part of the labeled segmentation:

Could be due to unreliable straightening warping field at the edge of the image (or it could be something else...)

On the sct_example_data/t1, the first disc label (C1) appears 3 slices below the top part of the labeled segmentation:
Could be due to unreliable straightening warping field at the edge of the image (or it could be something else...)
UPDATE 2020-11-11 17:55:02: I notice the same behaviour on the t2:

I am looking into it, but in the "straight" space we have the correct label

In the straight space.

In the curve space.

I am looking into it, but in the "straight" space we have the correct label

what i find strange, then, is that un-straightened labeled segmentation does not seem to be affected in the same way as the un-straightened labels. I think there is something fishy there...

what i find strange, then, is that un-straightened labeled segmentation does not seem to be affected in the same way as the un-straightened labels. I think there is something fishy there...

Update:  What seems to happen is that there are two extra slices at the top of the segmentation.
Example with this experiment (to reproduce files are here with my results and the modified label):

I used sct_label_vertebrae -i sub-amu01_T2w.nii.gz -s ../../derivatives/labels/sub-amu01/anat/sub-amu01_T2w_seg.nii.gz -c t2 -discfile ../../derivatives/labels/sub-amu01/anat/sub-amu01_T2w_labels-disc-manual.nii.gz -r 0 -ofolder keep_files_original
Then open fsleyes and erase label number 3 (Position=(34,145,177)), save this file as sub-amu01_T2w_labels-disc-manual-modified.nii.gz
run  sct_label_vertebrae -i sub-amu01_T2w.nii.gz -s ../../derivatives/labels/sub-amu01/anat/sub-amu01_T2w_seg.nii.gz -c t2 -discfile ../../derivatives/labels/sub-amu01/anat/sub-amu01_T2w_labels-disc-manual_modified.nii.gz -r 0 -ofolder keep_files_modified

you should obtain the result presented in this gif which show the seg_labeled obtain with "original" discfile and then the seg_labeled obtained with "modified" discfile.

You will see that on label 4 the segmentation is different which seems weird. The position of the discs (seg_labeled_disc file) are the same in both instances. So This actually comes from a weird behavior from the segmentation, as you may notice on the gif there seem to also have a difference of two slices between the original and the modified.
I guess something is adding slices to the segmentation at the top. I think it may come from "clean_labeled_segmentation". I am looking. I'll update my comment later.

UPDATE: I definetly think clean_labeled_segmentation is the culprit (see here ).
The first line actually creates a dilation with factor 2, which would explain the two added slices at the top of the segmentation.
This gif is here to explain:

here you see what we obtain with the previous  run sct_label_vertebrae -i sub-amu01_T2w.nii.gz -s ../../derivatives/labels/sub-amu01/anat/sub-amu01_T2w_seg.nii.gz -c t2 -discfile ../../derivatives/labels/sub-amu01/anat/sub-amu01_T2w_labels-disc-manual_modified.nii.gz -r 0 -ofolder keep_files_modified

in pale green is the segmentation obtained with the current code.
In yellow you can see what we get when we comment this line so we don't use clean_labeled_segmentation.

I think the code change in 6fc58eb fix the 'extra slice' issue

The goal of clean_labeled_segmentation is:
(i) to remove voxels that are in the in sub-XXX_seg_labeled but not in sub-XXX_seg
(ii) to "fill" missing voxels (voxels that are in  sub-XXX_seg, sub-XXX_seg_labeled)
To that effect the strategy would be:

To multiply sub-XXX_seg with sub-XXX_seg_labeled. This would accomplish  (i)
To find the difference between sub-XXX_seg and sub-XXX_seg_labeled. This will give us the voxels that are missing. We then find the value of these voxels by dilating sub-XXX_seg_labeled.
The problem with this method is that it adds slices on top of the labeled segmentation, because of the dilation.

After discussion, the best solution would be to deactivate clean_labeled_segmentation. To do that and keep the code (which can be useful in certain cases), we'll create a flag --clean-labeled. If the flag is present the labeled segmentation will go through clean_labeled_segmentation, if not we will skip this step.

we'll create a flag --clean-labeled.

to follow our convention, it should be single dash. Also, i suggest the following wording: -clean-labels.
also, to be consistent with the concept of "cleaning", we should only do the following multiplication if this flag is on.
https://github.com/neuropoly/spinalcordtoolbox/blob/afc5be3fe0c9da431789f41a3988739883c230c8/spinalcordtoolbox/vertebrae/core.py#L322

I think this is fairly ready!

@lrouhier pls create a unit test

@jcohenadad I added a test to check if the function run with the new flag, is this what you had in mind?

@jcohenadad I think this is ready!
The test now computes the labeled_segmentation with and without the clean-labels flag. Then we compare the labeled_segmentation with the original segmentation in both cases. The 'cleaned' labeled_segmentation is supposed to be closer to the original one.

Problem:
Discs missing:
Position=(156,222,7) -- Value= 1.0
Position=(156,208,7) -- Value= 2.0
Position=(156,191,7) -- Value= 3.0
Position=(162,119,6) -- Value= 7.0
Position=(166,102,6) -- Value= 8.0
Position=(184,52,7) -- Value= 10.0
Position=(192,27,7) -- Value= 11.0

To reproduce:
Version: 5bb113c5a7f17b10eadc9a1ebb01f50b94514f53
Data: @lrouhier TO DOCUMENT --> ASK ALEX

Just adding a note that I have an in-progress review (I didn't forget!), but I'll finish the review after https://github.com/neuropoly/spinalcordtoolbox/pull/2990#issuecomment-743398182 is addressed, as things might change. :stuck_out_tongue:

@jcohenadad I Think this is ready.
There were two problems:

The first one is that the dilation in label interpolation was a bit too small for spines that were really 'curved'.
The second one was when using disc file. The disc file was straightened with nn interpolation which leads to some error.


Footer column close and open feature broken

Description:
If you reduce the browser window size in the storefront and scroll down to the footer, a plus or minus symbol appears depending on whether you have opened or closed the footer window. If you make the window larger again, reload the page and repeat the process several times. Eventually, you can no longer close the footer window, and the minus symbol will remain. The behaviour occurs irregularly
Environment: Tested in Version <IP_ADDRESS> as well in <IP_ADDRESS>
Steps to reproduce:

Open the storefront

Resize the browser window to a smaller size until the plus or minus symbol appears

Expand and collapse the footer column

Resize the browser to a regular width

Expand and collapse the footer column

Resize it back to a small width

Try and collapse the footer column

Expected result:
The plus or minus symbols should still open or close the footer
Actual result:
The plus or minus symbols should remain and doesn't open or close the footer anymore

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-34228.
[created from NEXT-34228, comment 485914]

[public] Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know. Thank you again for your valuable feedback.
[created from NEXT-34228, comment 501225]


Pachyderm fundamentals docs

Here are the main Pachyderm fundamentals docs.  I'm moving on to examples now (and eventually to the release post).

It looks like CI references the pipeline spec doc for some reason.  Need to update the location or add a copy.


[$250] [Dupe detection] Tax code review step shows empty option when workspace does not have tax rate

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.23-0
Reproducible in staging?: Y
Reproducible in production?:  Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

Workspace A has tax rates.
Workspace B does not have Taxes feature (disabled)

Go to staging.new.expensify.com
Go to workspace chat (workspace A)
Submit an expense with tax rate
Go to workspace chat (workspace B)
Submit the same expense as Step 3 but without tax rate
Submit another expense (any amount) to workspace chat (workspace B)
Open transaction thread of the expense submitted in Step 5
Click Review duplicates
Click Keep this one (any)
Proceed to Tax code review page

Expected Result:
Tax code review page will not have blank option
Actual Result:
Tax code review page shows blank option
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/29d537cb-7059-4c95-9a27-bcda44aa8383
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01bb1c0676c6145d31
Upwork Job ID:<PHONE_NUMBER>730671209
Last Price Increase: 2024-09-03
Automatic offers:

brunovjk | Reviewer | 103811978
Krishna2323 | Contributor | 103811979

Issue OwnerCurrent Issue Owner: @brunovjk

@adelekennedy  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@brunovjk one existing proposal to review above!

Dupe detection is in #wave-control, moving this over. CC: @pecanoro @kubabutkiewicz @parasharrajat @dylanexpensify

I am going to put a HOLD on this issue because I have the feeling the proposal here will fix this.

Hello @pecanoro, I tested both proposals (test branches) separately, and it seems that each one resolves its specific problem, but neither addresses the other's issue. They appear to be distinct problems with different root causes.
For the current problem, the test branch is working well, and I can no longer reproduce the issue. However, I still need to do a bit more investigation into the underlying cause. I’ll provide an update on this tomorrow morning. Thanks

After reviewing the issue in depth, yeah, we can remove the HOLD in this one. @brunovjk Feel free to review the proposals!

Thanks, I will finish the review soon.

Apologies for the delay. @Krishna2323's proposal seems good to me. The approach of using ONYXKEYS.REVIEW_DUPLICATES instead of route.params.threadReportID to correctly fetch the transactionID is sound. Additionally, the suggestion to skip the field review step when no field options are available is logical and improves the flow.
🎀👀🎀 C+ reviewed

Sounds good, assigning @Krishna2323!

Sorry for the delay 🙏🏻, I have been sick from last 3 days 🥲. Will work on this today.

Feeling better now? Let me know if I can assist in anyway

PR under review.

Weekly Update: PR is still in review

Same as above

PR is still in review

I am doubling the reward of this one since it took longer than anticipated!

BugZero Checklist:

[x] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[x] 1b. Mistake during implementation
[ ] 1c. Backend bug
[x] 1z. Other: Additional to the issue reported here, during the PR, we introduced an improvement to skip the review page if no fields (e.g., tax codes) are enabled in the policy and extracted from different transactions. This enhancement prevents unnecessary steps when reviewing duplicates.

Where bug was reported:

[x] 2a. Reported on production
[ ] 2b. Reported on staging (deploy blocker)
[ ] 2c. Reported on both staging and production
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[x] 3z. Other: Applause - Internal Team

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Workspace A has tax rates, categories, and tags enabled.
Workspace B does not have the taxes, categories, and tags features enabled.

Test:

Open the chat for Workspace A.
Submit an expense in Workspace A with a tax rate, category, and tag (use the same amount and merchant).
Submit another expense in Workspace A with all fields left empty (defaults).
Open the chat for Workspace B.
Submit an expense in Workspace B with the same amount and merchant as the previous entries.
Open the transaction thread for the expense submitted in Workspace B.
Click "Review Duplicates."
Choose "Keep this one" for the first transaction and verify that the tax, tag, and category review step is skipped since Workspace B does not have these features enabled.
Return to the initial duplicate review step (where you select the transaction to keep).
Choose "Keep this one" for the second transaction and verify that the tax, tag, and category review step is not skipped, as Workspace A has these features enabled.
Repeat steps 9 and 10 for the third transaction, ensuring the tax, tag, and category review step is not skipped since it belongs to Workspace A.
Return to the chat for Workspace A and open any transaction thread for an expense submitted there.
Click "Review Duplicates."
Choose "Keep this one" for the first transaction and verify that the tax, tag, and category review step is not skipped, as Workspace A has these features enabled.
Return to the initial duplicate review step.
Choose "Keep this one" for the second transaction and verify that the tax, tag, and category review step is not skipped since Workspace A has these features enabled.
Return to the initial duplicate review step again.
Choose "Keep this one" for the third transaction and confirm that the tax, tag, and category review step is skipped, as this transaction belongs to Workspace B, which does not have these features enabled.

Do we agree 👍 or 👎

@pecanoro the test steps above make sense to me - any additional thoughts?

Payouts due:

[ ] Contributor: $500 @Krishna2323 Upwork
[ ] Reviewer: $500 @brunovjk Upwork


KOGITO-5072: DMN Dev Sandbox

How to use and test
Check out KIE live 40 (don't forget to like the video 😃)

Live version
Online Editor - Staging (last update Aug 3rd @ bf8b71ee368539ea761a895eb0e10865a56ef541)

New environment variables

KOGITO_TOOLING_BUILD_docker: Whether or not to build docker images as part of build:prod.
Default: false

DMN_DEV_SANDBOX__baseImageRegistry: The registry of the DMN Dev Sandbox base image.
Default: quay.io

DMN_DEV_SANDBOX__baseImageAccount: The account of the DMN Dev Sandbox base image.
Default: kogito_tooling_bot

DMN_DEV_SANDBOX__baseImageName: The name of the DMN Dev Sandbox base image.
Default: dmn-dev-sandbox-deployment-base-image

DMN_DEV_SANDBOX__baseImageTag: The tag of the DMN Dev Sandbox base image.
Default: latest

DMN_DEV_SANDBOX__onlineEditorUrl: The URL to open the online editor in the DMN Dev Sandbox form.
Default: https://kiegroup.github.io/kogito-online

DMN_DEV_SANDBOX__formGtmId: The GTM ID for analytics purposes on the DMN Dev Sandbox form.
Default: undefined

New packages

@kogito-tooling/dmn-dev-sandbox-deployment-base-image: Contains the Dockerfile to build the base image.
@kogito-tooling/dmn-dev-sandbox-form-webapp: Contains a typescript webapp in which is deployed in the Dev Sandbox. The output is placed inside the docker image.
@kogito-tooling/dmn-json-schema-generator: Contains a java code to extract the form schema out of a DMN file. The resulting jar is placed inside the docker image to be used at build time.

Build and test locally
Make sure to export the following environment variables:
export DMN_DEV_SANDBOX__onlineEditorUrl=http://<IP_ADDRESS>:9001
export DMN_DEV_SANDBOX__baseImageAccount=caponetto
export ONLINE_EDITOR__kieToolingExtendedServicesCompatibleVersion=0.11.0

Note: Currently, the image is published only on my account (caponetto).
Then, in the root:
yarn bootstrap && yarn build:dev
cd packages/online-editor
yarn start

Go to http://<IP_ADDRESS>:9001 and start using it!

Workflows

CI :: Monorepo will only build the docker image if the package dmn-dev-sandbox-deployment-base-image needs to be built.
CI :: Daily dev version will build and publish the docker image with tag daily-dev
Release :: Draft will build and publish the docker image with tag <version>-prerelease
Release will build and publish the docker image with tags latest and <version>

Great stuff, @caponetto! I'll have fun reviewing this one 😝

Hi @tiagobento, thanks a lot for the review! :)
Requested changes are in, please take a look when possible.

@tomasdavidorg This is ready for review, can you please take a look?

Branch rebased to fix a conflict.

All green @tiagobento 😃


Sometimes LDAPS connection dont work to AD server

Describe the bug
Sometimes I get this error message on login:
An unknown error occurred
Note:
This problems only occure when I am using LDAPS on LDAP connection everything works fine
After i enabled the debug mode I get followign stacktrace (Changed user and pass):
[2020-04-02 14:41:36] production.ERROR: ldap_bind(): Unable to bind to server: Can't contact LDAP server {"exception":"[object] (ErrorException(code: 0): ldap_bind(): Unable to bind to server: Can't contact LDAP server at /var/www/BookSt ack/app/Auth/Access/Ldap.php:93) [stacktrace] #0 [internal function]: Illuminate\\Foundation\\Bootstrap\\HandleExceptions->handleError(2, 'ldap_bind(): Un...', '/var/www/BookSt...', 93, Array) #1 /var/www/BookStack/app/Auth/Access/Ldap.php(93): ldap_bind(Resource id #558<EMAIL_ADDRESS>'xxxxxxxxx') #2 /var/www/BookStack/app/Auth/Access/LdapService.php(165): BookStack\\Auth\\Access\\Ldap->bind(Resource id #558<EMAIL_ADDRESS>'xxxxxxxxx') #3 /var/www/BookStack/app/Auth/Access/LdapService.php(46): BookStack\\Auth\\Access\\LdapService->bindSystemUser(Resource id #558) #4 /var/www/BookStack/app/Auth/Access/LdapService.php(80): BookStack\\Auth\\Access\\LdapService->getUserWithAttributes('rodekev', Array) #5 /var/www/BookStack/app/Auth/Access/Guards/LdapSessionGuard.php(71): BookStack\\Auth\\Access\\LdapService->getUserDetails('rodekev') #6 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Auth/AuthenticatesUsers.php(82): BookStack\\Auth\\Access\\Guards\\LdapSessionGuard->attempt(Array, false) #7 /var/www/BookStack/app/Http/Controllers/Auth/LoginController.php(113): BookStack\\Http\\Controllers\\Auth\\LoginController->attemptLogin(Object(BookStack\\Http\\Request)) #8 [internal function]: BookStack\\Http\\Controllers\\Auth\\LoginController->login(Object(BookStack\\Http\\Request)) #9 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Controller.php(54): call_user_func_array(Array, Array) #10 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php(45): Illuminate\\Routing\\Controller->callAction('login', Array) #11 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Route.php(219): Illuminate\\Routing\\ControllerDispatcher->dispatch(Object(Illuminate\\Routing\\Route), Object(BookStack\\Http\\Controllers\\Auth\\LoginController), 'login') #12 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Route.php(176): Illuminate\\Routing\\Route->runController() #13 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(681): Illuminate\\Routing\\Route->run() #14 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(130): Illuminate\\Routing\\Router->Illuminate\\Routing\\{closure}(Object(BookStack\\Http\\Request)) #15 /var/www/BookStack/app/Http/Middleware/CheckGuard.php(25): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #16 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): BookStack\\Http\\Middleware\\CheckGuard->handle(Object(BookStack\\Http\\Request), Object(Closure), 'standard', 'ldap') #17 /var/www/BookStack/app/Http/Middleware/RedirectIfAuthenticated.php(40): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #18 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): BookStack\\Http\\Middleware\\RedirectIfAuthenticated->handle(Object(BookStack\\Http\\Request), Object(Closure)) #19 /var/www/BookStack/app/Http/Middleware/GlobalViewData.php(25): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #20 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): BookStack\\Http\\Middleware\\GlobalViewData->handle(Object(BookStack\\Http\\Request), Object(Closure)) #21 /var/www/BookStack/app/Http/Middleware/Localization.php(76): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #22 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): BookStack\\Http\\Middleware\\Localization->handle(Object(BookStack\\Http\\Request), Object(Closure)) #23 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/VerifyCsrfToken.php(76): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #24 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Foundation\\Http\\Middleware\\VerifyCsrfToken->handle(Object(BookStack\\Http\\Request), Object(Closure)) #25 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Middleware/ThrottleRequests.php(59): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #26 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Routing\\Middleware\\ThrottleRequests->handle(Object(BookStack\\Http\\Request), Object(Closure)) #27 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/View/Middleware/ShareErrorsFromSession.php(49): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #28 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\View\\Middleware\\ShareErrorsFromSession->handle(Object(BookStack\\Http\\Request), Object(Closure)) #29 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Session/Middleware/StartSession.php(56): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #30 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Session\\Middleware\\StartSession->handle(Object(BookStack\\Http\\Request), Object(Closure)) #31 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Cookie/Middleware/AddQueuedCookiesToResponse.php(37): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #32 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Cookie\\Middleware\\AddQueuedCookiesToResponse->handle(Object(BookStack\\Http\\Request), Object(Closure)) #33 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Cookie/Middleware/EncryptCookies.php(66): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #34 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Cookie\\Middleware\\EncryptCookies->handle(Object(BookStack\\Http\\Request), Object(Closure)) #35 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(105): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #36 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(683): Illuminate\\Pipeline\\Pipeline->then(Object(Closure)) #37 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(658): Illuminate\\Routing\\Router->runRouteWithinStack(Object(Illuminate\\Routing\\Route), Object(BookStack\\Http\\Request)) #38 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(624): Illuminate\\Routing\\Router->runRoute(Object(BookStack\\Http\\Request), Object(Illuminate\\Routing\\Route)) #39 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Routing/Router.php(613): Illuminate\\Routing\\Router->dispatchToRoute(Object(BookStack\\Http\\Request)) #40 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(170): Illuminate\\Routing\\Router->dispatch(Object(BookStack\\Http\\Request)) #41 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(130): Illuminate\\Foundation\\Http\\Kernel->Illuminate\\Foundation\\Http\\{closure}(Object(BookStack\\Http\\Request)) #42 /var/www/BookStack/vendor/barryvdh/laravel-debugbar/src/Middleware/InjectDebugbar.php(58): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #43 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Barryvdh\\Debugbar\\Middleware\\InjectDebugbar->handle(Object(BookStack\\Http\\Request), Object(Closure)) #44 /var/www/BookStack/vendor/fideloper/proxy/src/TrustProxies.php(57): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #45 /var/www/BookStack/app/Http/Middleware/TrustProxies.php(39): Fideloper\\Proxy\\TrustProxies->handle(Object(BookStack\\Http\\Request), Object(Closure)) #46 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): BookStack\\Http\\Middleware\\TrustProxies->handle(Object(BookStack\\Http\\Request), Object(Closure)) #47 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/TransformsRequest.php(21): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #48 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Foundation\\Http\\Middleware\\TransformsRequest->handle(Object(BookStack\\Http\\Request), Object(Closure)) #49 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/ValidatePostSize.php(27): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #50 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Foundation\\Http\\Middleware\\ValidatePostSize->handle(Object(BookStack\\Http\\Request), Object(Closure)) #51 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/CheckForMaintenanceMode.php(63): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #52 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(171): Illuminate\\Foundation\\Http\\Middleware\\CheckForMaintenanceMode->handle(Object(BookStack\\Http\\Request), Object(Closure)) #53 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(105): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}(Object(BookStack\\Http\\Request)) #54 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(145): Illuminate\\Pipeline\\Pipeline->then(Object(Closure)) #55 /var/www/BookStack/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(110): Illuminate\\Foundation\\Http\\Kernel->sendRequestThroughRouter(Object(BookStack\\Http\\Request)) #56 /var/www/BookStack/public/index.php(54): Illuminate\\Foundation\\Http\\Kernel->handle(Object(BookStack\\Http\\Request)) #57 {main} "}
Steps To Reproduce
Steps to reproduce the behavior:

Go to login page and insert the login credentials
Click on login
Get the error message like described above
After I hit 2-3 times F5 the login works. Sometimes the login also works on the first try

Expected behavior
Logged in
Screenshots
If applicable, add screenshots to help explain your problem.
Your Configuration:

Bookstack Version: v0.28.3
PHP Version: v7.2.15
Hosting Method: Nginx v1.12.2
Using Active Directory

Additional context
.env LDAP config:
AUTH_METHOD=ldap LDAP_SERVER=ldaps://adserver.mydomain.com:636 LDAP_BASE_DN=OU=USERS,DC=syncdomain,DC=com<EMAIL_ADDRESS>LDAP_PASS=xxxxxxxxx LDAP_USER_FILTER=(&(sAMAccountName=${user})) LDAP_VERSION=3 LDAP_EMAIL_ATTRIBUTE=mail LDAP_USER_TO_GROUPS=true LDAP_GROUP_ATTRIBUTE="memberOf" LDAP_REMOVE_FROM_GROUPS=true LDAP_TLS_INSECURE=true
If you need any more infos let me know. Hope you can help and thank you in advance.
Reference to: #1069, #1220

I started experiencing a similar issue on my install today too however even when refreshing several times the authentication will not complete.  The only way to resolve has been to fall back to LDAP rather than LDAPS as mentioned in #1220

I've got the exact same problem with version 0.30.4 and a Samba4 AD DC. The first try to login fails reproducible but then either by hitting F5 or entering the same credentials again I am able to login. Is there anything I can do to help solve this issue?

Since the last comment on this issue is relatively old I'm going to close this. If the issue remains and is something you still require to be fixed please open a new issue, referencing this one.


python 3.10 CI failing due to conda-forge package

This issue should fix itself.
python 3.10 on all OSs fails with:

griffe 0.48.0 requires backports-strenum, which is not installed.

Mostly likely cause is mkdocs-material which was updated in the last 24hrs
https://anaconda.org/conda-forge/mkdocs-material

Nope, the cause is Griffe ^^ We started depending on backports.strenum, but it seems the Conda package did not catch up on this.

ah gotcha. I don't depend on griffe directly so I was trying to see which of my dependencies had it as a sub-dep. Thanks for the info!
Looks like the griffe feedstock maintainer already merged a fix, so this is now fixed: https://github.com/conda-forge/griffe-feedstock/pull/112


Fix : Klockwork fixes for unit tests

Signed-off-by: Nabendu Maiti<EMAIL_ADDRESS>

We are working on fixing unit-test failure reporting. This PR can be merged once that pipeline is fixed.


A question about spurious shifts experiments

Thanks for sharing the code!
I'm interested in this project and I'm trying to run the experiments based on the code and paper.
However, in the spurious shifts experments, there is an interesting phenomenon.  In the experiments of λ (figure 4 in the paper), my result is opposite from that in the paper, which means that λ=1 reaches the lowest ADE and λ=5 reaches the highest ADE.
I think I have followed all the settings in the paper and code (e.g. version of pytorch, hyperparameters), so I'm a little confused about this phenomenon. Could you please give me some advice about this? Is there anything special that you think should be focused on when runing the spurious shifts experiments?
Sorry for bothering you and looking forward to replys.

Thanks for your reply! I'm trying to

Hi @mmdzb, thank you for your interest and reporting the issue.
I have just rerun the experiment using seed=1 as we left by default in the script spurious/scripts/add_confidence/train.sh.
Here's our obtained result
The global pattern looks pretty similar to our previous figure, while there are still some local variations. We conjecture that the local variations are attributed to some sources of randomness, e.g., random sampling for prediction. However, running experiments over multiple seeds should reduce the variations, especially when considering a broader spectrum of λ.
Hope this helps. If you have any further questions, please don't hesitate to let me know.

Thanks for your reply! I will try to figure the problem out.


Added support for Python 3.6.9

which is compatible with NVIDIA Jetson devices.

Added this feature in a new PR. Builds for python > 3.6.1 are all in a single branch (master) now.


Az acr build command info needed

az acr build --image sample/hello-world:v1
--registry myContainerRegistry008
--file Dockerfile .
The above command builds and pushes to registry. What if i  need only build and then push based on my interest .Because i have to scan image before pushing it.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4f357648-1ce0-04eb-fab1-add2e2bce02e
Version Independent ID: 982ac987-43f5-7fdd-6853-a7fe823040cc
Content: Quickstart - Build a container image on-demand in Azure - Azure Container Registry
Content Source: articles/container-registry/container-registry-quickstart-task-cli.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

Thanks for the question. We are currently investigating and will update you shortly.

will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


feature Add an http action for visibility of compaction score on each tablet (#38489)

pick: #38489
Usage:

curl http://be_ip:be_host/api/compaction_score?top_n=10 Returns a json object contains compaction score for top n, n=top_n.

[
    {
        "compaction_score": "5",
        "tablet_id": "42595"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42587"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42593"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42597"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42589"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42599"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42601"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42591"
    },
    {
        "compaction_score": "5",
        "tablet_id": "42585"
    },
    {
        "compaction_score": "4",
        "tablet_id": "10034"
    }
]

If top_n is not specified, return all compaction score for all tablets. If top_n is illegal, raise an error.
invalid argument: top_n=wrong

curl http://be_ip:be_host/api/compaction_score?sync_meta=true sync_meta is only available on cloud mode, will sync meta from meta service. It can cooperate with top_n.
If add param sync_meta on non-cloud mode, will raise an error.

sync meta is only available for cloud mode

In the future, this endpoint may extend other utility, like fetching tablet compaction score by table id, etc.

Proposed changes
Issue Number: close #xxx

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run p0


feat: add function and compute task states in user doc

Companion PR

https://github.com/Substra/substra-backend/pull/823
https://github.com/Substra/orchestrator/pull/366
https://github.com/Substra/substra-tests/pull/319
https://github.com/Substra/substra-frontend/pull/297
https://github.com/Substra/substra/pull/397

Part of FL-1397

thanks @guilhem-barthes!
Could you please add the CP state machine as well?

Could you please add the CP state machine as well?

Ok, I will add the previous one and update to the one we discussed in the future batch of PR which update the compute plan flow


Fix bicep in VS build

Fix bicep in VS build
Microsoft Reviewers: Open in CodeFlow

@majastrz, fyi, fixed below as part of this change:

Updated to emit a binlog
Updated vsix name to vs-bicep.vsix
Copying language server bits to a folder to avoid overwriting


The ALB Target Group Name is not set to the 'name' Variable

Description
The ALB target group that is created by the module has no name property specified, so it is being created with a Terraform default name.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version : 4.0.7

Terraform version: 1.5.5

Expected behavior
Name the ALB target group according to the module name variable.
Actual behavior
The name of the ALB target group is set to a default Terraform name.
Terminal Output Screenshot(s)

This issue has been resolved in version 4.0.8 :tada:


Update of Microsoft.Identity.Client packages

In order to fix High security issues present in the Veracode report my security team advise is to update this package in all nuget packages that my main solution reference

@microsoft-github-policy-service agree


Feature: store images as strings in the Hive box

Major Bug Fix
Fixes #242
Changes

Added utils directory to the codebase which contains the functions for converting an image to a string and vice-versa.
Updated HiveDB.
Now the pdfs and starred box store a string instead of a file on index 2.

Results

The console doesn't throw an error of unsupported type in Hive Box.
The PDFs don't disappear from the app once the user restarts it.

By this images get saved in your gallery, later pr u will resolve the problem with pdf vanishing

By this images get saved in your gallery, later pr u will resolve the problem with pdf vanishing

The PDF vanishing bug has also been resolved in this PR. The lint checks are failing due to changes in #239,  I will fix them.


Score not showing properly

First of all, amazing library. But, I guess the score property of result object is always zero (even when it hasn't found any matches).
Example:
void testFuzzy() {
    final fuse = Fuzzy(
      [
        Person('John', 'Doe', 29),
        Person('Jane', 'Doe', 32),
        Person('Peter', 'Parker', 36),
        Person('Bruce', 'Wayne', 30),
        Person('Clark', 'Kent', 50),
        Person('Tony', 'Stark', 34),
        Person('Clint', 'Barton', 35),
      ],
      options: FuzzyOptions(
        keys: [WeightedKey<Person>(name: 'firstname', getter: (p) => p.firstname, weight: 1)],
        shouldNormalize: true,
        isCaseSensitive: false,
        shouldSort: false,
        findAllMatches: false,
        verbose: true,
      ),
    );

    var result = fuse.search('John');

    if (result.isNotEmpty) {
      result
          .forEach((r) => print('${r.item.firstname} ${r.item.surname} (${r.item.age}) | score: ${r.score}'));
    } else {
      print('no results');
    }
  }

class Person {
  final String firstname;
  final String surname;
  final int age;

  Person(this.firstname, this.surname, this.age);
}

output shows:

flutter: Full text: "John", score: 0.0
flutter: Score average (final): 0.0
flutter:
Check Matches: true
flutter: Full text: "Jane", score: 0.5
flutter: Score average (final): 0.5
flutter:
Check Matches: true
flutter: Full text: "Peter", score: 1.0
flutter: Score average (final): 1.0
flutter:
Check Matches: true
flutter: Full text: "Bruce", score: 1.0
flutter: Score average (final): 1.0
flutter:
Check Matches: true
flutter: Full text: "Clark", score: 1.0
flutter: Score average (final): 1.0
flutter:
Check Matches: true
flutter: Full text: "Tony", score: 0.5
flutter: Score average (final): 0.5
flutter:
Check Matches: true
flutter: Full text: "Clint", score: 1.0
flutter: Score average (final): 1.0
flutter:
Check Matches: true
flutter:
Computing score:
flutter: John Doe (29) | score: 0.0
flutter: Jane Doe (32) | score: 0.0
flutter: Tony Stark (34) | score: 0.0

We can see that on verbose output the score is taken correctly, but when printing after search, is aways zero.
I can workaround it doing:
    result.forEach((r) => r.score = r.matches[0].score);
    result.sort((a,b) => a.score.compareTo(b.score));

But I'm not happy with this approach, is there a solution for this? Or perhaps I'm doing something wrong, please help.
Thanks in advance

I forgot to mention that results are not sorted by score as well (even with shouldSort: true), due to the same problem, score always equal to zero, so sorting will not take effect.

Hello @blinkn! Sorry for taking a long time to answer this.
This library was a hacky implementation of Fuse.js in Dart and those sort of errors could definitely occur, sorry about that.
I'm not sure that I'll be able to take a look at this soon, but if you could, please update us in the process, and open a PR here so other could benefit from it as well!


Enforce usage is kept up to date

Rebuild locally with:
bundle exec script/sync-readme-usage README.md

cc @fancyremarker

Codecov Report

Merging #228 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #228   +/-   ##
=======================================
  Coverage   92.84%   92.84%
=======================================
  Files          46       46
  Lines        1719     1719
=======================================
  Hits         1596     1596
  Misses        123      123

Continue to review full report at Codecov.

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


CI failures

failures:
---- src/lib.rs - (line 120) stdout ----
Test executable failed (exit code 101).
stderr:
thread 'main' panicked at 'called Result::unwrap() on an Err value: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }', src/lib.rs:31:36
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
---- src/lib.rs - (line 34) stdout ----
Test executable failed (exit code 101).
stderr:
thread 'main' panicked at 'called Result::unwrap() on an Err value: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }', src/lib.rs:26:35
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
---- src/lib.rs - (line 98) stdout ----
Test executable failed (exit code 101).
stderr:
thread 'main' panicked at 'called Result::unwrap() on an Err value: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }', src/lib.rs:21:35
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
---- src/lib.rs - (line 79) stdout ----
Test executable failed (exit code 101).
stderr:
thread 'main' panicked at 'called Result::unwrap() on an Err value: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }', src/lib.rs:18:35
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
failures:
src/lib.rs - (line 120)
src/lib.rs - (line 34)
src/lib.rs - (line 79)
src/lib.rs - (line 98)
test result: FAILED. 15 passed; 4 failed; 0 ignored; 0 measured; 0 filtered out; finished in 6.84s
error: test failed, to rerun pass '--doc'
Error: The process '/usr/share/rust/.cargo/bin/cargo' failed with exit code 101

I'm going to close this one as I haven't seen it lately. We can reopen if it recurs

This seems to be caused by the default constructor for Reedline creating a history with file access which probably fails in the ci with this rather weird message. I stumbled accross this again when fixing another doccomment and forgetting to set the no_run marker


Take IsTruncated field of S3 ListParts API response into account

Fix for issue #147

There is actually a ListPartsPages function in the API, which does the iterating for you. You got a callback function, which is executed for each page and can be used to append the page contents to a parts array for example. Should simplify the code.
https://docs.aws.amazon.com/sdk-for-go/api/service/s3/#S3.ListPartsPages

As @trickkiste said (https://github.com/tus/tusd/issues/147#issuecomment-322761255), the ListPartsPages function offers a more elegant code but is very tough to test with our current setup. Therefore we came to the conclusion that ListPartsPages in not optimal. However, this does not affect your patch, @koenvo.
Going to have a deep look into this PR now.

@koenvo would you care to rename you ListParts method to ListAllParts to be more descriptive and differentiate it from the S3API method.

Done :-)

Thank you very much again :)


KCL parser: allow comments in multi-line arrays

Part of #1528. Handling comments in multi-line object expressions will require a separate PR.
This makes the following comment structure valid (it was a parse error before):
[
  // X
  1,
  // Y
  2,
]

There are definitely edge-cases where formatting your KCL will do something unexpected, e.g.
[
  1, // X
  2, // Y
]

becomes
[
  1,
  // X
  2,
  // Y
]

but I think this PR is valuable for unblocking users. We can make it smarter at a later point.

i think the parser test failures are real

For record I gave the green light because @iterion should be getting Windows green today. All other tests passed.


add operation id to documentation

I added this field to be able to personalized methods of the sdks generated with swagger-codegen and zf-apigility-documentation-swagger.

Thanks, @garygitton!


Better Transcription Support

Let's utilize the functions after https://github.com/CanCLID/ToJyutping/pull/3 and https://github.com/nk2028/ToMiddleChinese/pull/4 are settled!

Ping @ayaka14732


New, empty branch considered as merged after parent pull

Bug description
Assume, we have a develop with unpulled changes from remote. Then we slide-in new branch as a child of develop. Next, we pull the develop. Now edge between develop and newly created branch is gray, but should be red.

I would not consider this one as a bug; this is rather an expected behavior...
you create a branch B from A
you pull A, A becomes A', B still points to A (which is supposed to be behind A')
branches pointing commits behind other branches are considered to be merged
that's all
However, in the first moment, it seems to be a nice feature to automatically "rebase" 0-commit branches (cause this is what it is about actually).
On the other hand, this is not something that a user would expect to happen after the pull (possibly we should hide it behind some git config).

I'd still considering this as a bug - branch B (from your example) was not merged into A but edge indicates it was.

branches pointing commits behind other branches are considered to be merged

and probably this is a problem (bug). Probably we should take into consideration not only current commit graph but maybe also branches reflog or something else.

I agree with @micpiotrowski; this should be relatively easy fix (since @mkondratek: rebasing 0-commit branches is indeed implemented by git rebase as fast-forwarding)

Currently, the algorithm for selecting between green or gray edge is (P = parent branch, C = child branch):

if P is descendant or equal to C:

if P is descendant of C: gray edge
if P is equal to C:

if filtered reflog (reflog without branch creation and resets) of C is empty: C has just been created, so green edge
else: there has been some non-trivial activity on C, hence C is merged, so gray edge

The first case (if P is descendant of C: gray edge) could just leave the Rebase action available (alongside Slide Out), even without marking the edge as red :thinking:

Possibly adding a new SyncToParentStatus will be necessary to handle that case... although I can't say at the moment, maybe it can be done in other way

leave the Rebase action available

I'd actually only make the rebase action available from context menu, but not suggest it from the toolbar (since slide-out is most commonly needed in that case; rebase isn't the default choice)

Closed by #622.


Text in search boxes not visible in Todoist dark theme (and possibily other integrations as well)

🐛 Describe the bug
When searching for projects and tags, the text is not visible inside the text input:

Steps to reproduce
Steps to reproduce the behaviour:

Open Todoist
Click the Toggl Button inside the Todoist UI
Open the Projects (or Tags) popdown
Type something in the search box

Hi there, I can fix this one

Should be resolved in 1.44.5 🚀


Using node 0.6.7 using npm to install fails

Following your Readme instructions
npm install dreadnot -g
Fails
npm ERR! error installing<EMAIL_ADDRESS>npm ERR! error rolling back<EMAIL_ADDRESS>Error: UNKNOWN, unknown error '/usr/local/lib/node_modules/dreadnot'
npm ERR! Unsupported
npm ERR! Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! Required: {"node":">= 0.4.1 < 0.5.0"}
npm ERR! Actual:   {"npm":"1.1.0-beta-10","node":"0.6.7"}
npm ERR!
npm ERR! System Linux 3.0.4-linode38
npm ERR! command "node" "/usr/local/bin/npm" "install" "dreadnot" "-g"
npm ERR! cwd /usr/local/src
npm ERR! node -v v0.6.7
npm ERR! npm -v 1.1.0-beta-10
npm ERR! code ENOTSUP
npm ERR! message Unsupported
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /usr/local/src/npm-debug.log
npm not ok

I also have the debug log file posted here: http://<IP_ADDRESS>/npm-debug.txt

Should be fixed in 5e84c58

still a problem, this time because of jade -><EMAIL_ADDRESS>which depends on 0.4.x

I can change the commander dep

nvm I have "commander": "0.2.x",

Bumped the jade dependency to 0.20.0, I think this should fix it. Is there some npm command I can run to show the tightest engine restrictions in the dependency tree?
In any case, just published 0.1.3, hopefully that'll help.


Add hsqldb in-memory DB support

It might be useful for testing and some other cases when having a persisted DB is not needed (e.g. following tutorials). It was originally added to #219 but due to lack of time for creating documentation/writing tests/etc was not merged.

In #232 @zikolach added this file: https://github.com/iiasa/ixmp/blob/399e1b04db32b8fb6f090dba611a900fd4c421ea/tests/data/testdb/test.properties.sample#L1-L7
…used in: https://github.com/iiasa/ixmp/blob/399e1b04db32b8fb6f090dba611a900fd4c421ea/tests/backend/test_jdbc.py#L39-L43
This is a proof-of-concept & test that in-memory databases can be used. However, it is still not advertised, documented, or configurable (without use of a dbprops file).

Note that as part of #270 we now make extensive (almost exclusive) use of this in the test suite, so it's very well-supported. The documentation/public API/command-line configuration still needs work.


Data Ingestion - Emergency room data

Create a scheduler job to get num emergency rooms into GCS bucket
Emergency room: https://healthdata.gov/dataset/hcup-nationwide-emergency-department-database-neds-restricted-access-file (unclear how to gain access)

This issue will now also include a function/service to get the data from GCS into bigquery and any data standardization that needs to be done within bigquery.

It looks like these are paid datasets: https://www.distributor.hcup-us.ahrq.gov/Databases.aspx
Suggest skipping this or we look for a comparable dataset.


Cannot edit invalid pre-filled e-mail address on Register Microsoft account screen

Description
Attempting to log in using Microsoft account. After entering credentials, it goes to https://www.nuget.org/account/register, where it asks me to provide a username. The email field is already filled in and is not editable, but since it is my Microsoft account name and is not a valid e-mail address, I'm stuck. Attempting to go forward gives the error message: This doesn't appear to be a valid email address.
To Reproduce
Steps to reproduce the behavior:

Go to this URL: https://www.nuget.org/users/account/LogOn?returnUrl=%2F
Click Sign in with Microsoft button
Enter Skype id (I've also tried logging out of skype.com, and coming back to nuget using the e-mail address associated with the Skype/Microsoft account - same results.)
When prompted, I enter my password. That opens this URL: https://www.nuget.org/users/account/authenticate/return?ReturnUrl=%2F&error=&errorDescription=
As shown in the screen shot, there is no way to edit the e-mail address. When I click Register there, I get the error: This doesn't appear to be a valid email address.

Expected behavior
I either need nuget to correctly get the e-mail address associated with my Skype/Microsoft account or have it allow me to edit the e-mail address field manually.
Screenshots

Hi @tombogle - it looks like the email address claim for your account doesn't actually return an email id but your skype id(?). Can you verify you have an email address linked to your Microsoft account(https://account.microsoft.com/profile/)? Perhaps, would it be possible for you to update your microsoft account(https://account.microsoft.com/profile/) and add email address as login mechanism?

OK, I managed to reproduce the issue and work around it.
Workaround:

Go to https://account.microsoft.com
Click on your profile picture in the top right corner and select "Sign out";

In my case, clicking that profile icon opened the account card that had my Skype name specified there, so I expect it would be the same for you;

Page reloads and your profile picture gets replaced with the generic "Sign in" icon;
Click the "sign in" icon;
Make sure you use your email (and not your Skype name) in the login process;
You should get back to your account page and clicking on the profile picture in the top right corner will open a card where it would have your email specified.
Go to https://www.nuget.org, click the "Sign in" button, followed by "Sign in with Microsoft";
Below the log in prompt you should see a "We found an account you can use here" message (with your email specified below your name):

Click it.
You should get to the "Register Microsoft account" page that will have your proper email specified.

@tombogle - does the workaround specified by @agr above work for you? I would like to get you unblocked and then look into the appropriate fix here.

The issue has been fixed and is live in production.


Update page titles for all App services bundle pages

Update page titles for all App services bundle pages

lgtm!


Upgrading encryption protocol

As of now we use AES-Advanced encryption system to encrypt the files but expecting some new salts to add in features for the system.

@rexdivakar Do you want to add other algorithms other than AES? If yes, any suggestions for this

I believe we can add asymmetric RSA encryption

Sure we can. I would like to work on this then

Issue assigned to @SanchitJain123
🌟 Star it 🍴Fork it 🤝 Contribute to it!
Happy Coding

@rexdivakar RSA is not suitable to encrypt larger files so it is recommended to encrypt AES encryption key with RSA rather than the file content. Should I proceed with that?

Well we need a asymmetric method of encryption got any ideas on that ? @SanchitJain123

This will be asymmetric in the sense that the AES encryption key will be encrypted through RSA (asymmetric encryption), while the data will be encrypted through symmetric one. Kinda hybrid encryption :). Also, I have read various sources on the stack as well all are suggesting this. But if we would like that the data is also encrypted through the asymmetric methods then we will have to maybe encrypt chunks of data through the same public key and encrypt the whole file. The first approach will be faster and the second approach will be slower. @rexdivakar

right now we need a asymmetric method so we can share the keys to the another user via ssh so others can use it @SanchitJain123
Do u think that could be done ?

Yes, the procedure would be this,

We will generate the AES encryption key and encrypt it through asymmetric RSA. New public and private keys will be generated through this.
We will save privateKey along with the encrypted AES encryption key in a dedicated file.  (let's say this file is named key.pem).
Then the user can send the key.pem file through ssh connection to another user and save it along the encrypted file/folder or the user can specify the path for the same in the command line.
Then, the user will have a decrypted file/folder using the same using our program.

yup, exactly what i need in @SanchitJain123

I'll start implementing this then.

yes go ahead @SanchitJain123

Hey @SanchitJain123  ur decryption is failing for folders, can u please check it !

@rexdivakar sure will check it if there is some issue. When I tried it, it was working fine.

please ping me @SanchitJain123  over discord username: root#5998
https://discord.gg/e47wmTW


Inventory modifier not working

Add Inventory not working after CSGO update

Me too.

Me too.

Me too.

我突然发现都是我们国人在等...🤣

damn i was scared cus i thought i got banned or something

how long will it take to fix this approx?

yes unfortunately. i thought it might be a problem on my computer so i reinstalled the game. everything works fine except the inventory changer :(

+1 for this, it is because of the latest game update i believe


BRTFLUT.png uses SRGB and it should be interpreted as linear (RGB)

Pretty much anything that is an image (base color and also emissive) should be SRGB and the rest should be interpreted as linear (RGBA)
Also the shader should not be using SRGBtoLINEAR() for every single texture, only for the albedo and the emissive colors.
References:

https://forum.allegorithmic.com/index.php?topic=8745.0
http://polycount.com/discussion/193930/substance-designer-linear-vs-srgb-output

We're not using SRGBtoLINEAR on every texture.  The metal/rough and occlusion textures are excluded from this conversion.
There's an open question about the BRDF lookup in #64.  There's also a request to upgrade the BRDF to 16-bit in #45.
Thanks for the additional references!  I'm going to close this as duplicate with #64, but please do subscribe to that issue for updates on this.

Sounds good! BTW thanks for this great sample, I had so much fun with it! :)

Yes, thanks for this code, it really helped us jumpstart PBR!


[AJ-1296] Move UriViewer into data table directory

Continuing to organize data table code. This moves the UriViewer component into src/data/data-table along with other data table code. UriViewer is used to show information about arbitrary URLs in data table content.
It also moves LogViewer, which is unrelated to UriViewer, into src/workflows-app/components since it is only used by workflows-app components.
The many other modules that import isXUri from uri-viewer-utils suggests that those functions should be located in a more central place, but that's a project for another PR.

This is awesome cleanup + refactoring. Thank you!


deps: bump core version

The latest core https://github.com/celestiaorg/celestia-core/releases/tag/v1.8.0-tm-v0.34.20 fixes the race in #1252, so we should update to it in our replace

We have to wait for upgrade of celestia-app because the latest version isn't compatible with new core.


Using Node.js as Backend

Hey, I would like to start working on backend using Node.js. Please assign me this issue as a part of GSSoC 2021.

Sorry, DJANGO did the job, maybe next version might use it:)


Log region name to game_state

Forgot to give this a description yesterday;
In game_state, we should have a current_region element.
When the player is in a region, the region name should be there, and if the player is outside all of our custom-defined regions, it can just be "None"

79d12b9b438d64436df10a78b57c48a68188b1cd

Note: I think changes are in 8e475f754316eb050c2cd510455b2a850e257b2d


Grid support

Added methods to generate Bootstrap grid.
Code example:
echo $this->Html->gridContent($this->Form->input('field1'))
    ->gridConfig(['size' => 4])
    ->gridContent($this->Form->input('field2'))
    ->gridConfig(['size' => 4])
    ->gridContent($this->Form->input('field3'))
    ->gridConfig(['size' => 4])
    ->gridRender();

I like the idea, but the interface looks very alien and "uncakelike". I'd rather see something like tableCells.

Wouldnt this maybe be an own GridHelper instead of putting all that into the Html one?

Don't use private properties or methods
Don't uses HTML fragments like in gridRender(). Either use string templates and/or elements.
As @dereuromark sadi better to have a separate helper instead of adding more methods to HtmlHelper

I agree with almost everything, I just think it should be part of the Html Helper because it will render html code, but that's my opinion.
I think the project could implement other Bootstrap's components inside the HtmlHelper, such as: Navs (tabs and pills), List Group, Panels, Wells, Grid, etc.

Almost every helper generates HTML :) See Form or other helpers.
The question is more: Is it a large enough subpart to be its own helper or is it OK to through into the existing one.
If you say "Navs (tabs and pills), List Group, Panels, Wells, Grid, etc" then this sounds to me like a new separate one rather than bloating the HtmlHelper IMO.

Sorry, I think I explained wrong. I think the HtmlHelper of the Bootstrap's project, should know how to write the html components of Bootstrap.

@golgher Having it there makes sense to me

I don't think the HtmlHelper should be bloated with grid nor components. I started things in here because it was simpler but if there is a need to add other features, they are complex domains that would require their own respective helpers IMO.

Closing this due to lack of activity.
@golgher You are welcome to reopen or make a new PR with changes as requested.


Fix comparison of last deployed and revision fields in chart CR status

Towards https://github.com/giantswarm/giantswarm/issues/14642
We recently made last deployed and revision optional fields in the chart CR status but I screwed up the comparison in the status resource when it got integrated.
This caused chart-operator to OOM on exodus. As the status resource was updating the CR status on each loop and having problems calling the status webhook.
Checklist

[x] Update changelog in CHANGELOG.md.

Tested on ginger.


Bless nightly-2023-10-31 output

Automatically created by https://github.com/Enselic/cargo-public-api/actions/runs/6701734965/jobs/auto-bless

Looks like https://github.com/rust-lang/rust/pull/112463


Data Zoom min and Max Values are not correct when yAxis scale option true.

Version
5.4.3
Link to Minimal Reproduction
https://codepen.io/abdurrahmanyildirim/pen/GRzwzGp
Steps to Reproduce
Check the minimal reproduction. Data zoom min value is 150 but the y-axis starts from 140. Max value also doesn't match with y-axis max value.
Current Behavior
Max and min values of datazoom doesn't match with axis.
Expected Behavior
Max and min values should be same
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

In the linked example you are not applying any min / max restriction to dataZoom.
In case you are referring to the yAxis labels not starting and ending exactly with the min and max values of your data, this has nothing to do with dataZoom but is automatically done by the scale option. If you want your axisLabel set to your minimum and maximum data values you can use the axis property min and max:
yAxis: {   min: 'dataMin',   max: 'dataMax' }

Yes, it doesn't work correctly when scale is true. Is there a way to apply scale values from dataZoom?

Any solution?

best i can come up with is setting yAxis min and max to the minimum an dmaximum data values: example

Okay, thanks for your time.

Actually this might work.


Replace all methods of promote_type by promote_rule

Resolves https://github.com/JeffreySarnoff/ArbNumerics.jl/issues/67.

The comparison tests in https://github.com/JeffreySarnoff/ArbNumerics.jl/blob/1bd8472c3bfb2607a3d876fff261f11ce113dc70/test/compare.jl#L10-L57 already do a great job in testing that comparisons still hold after promoting. Nevertheless, I added a few small tests that check the expected output type of some basic arithmetic operations. Is this what you had in mind?

yes thanks
On Tue, Jun 11, 2024 at 10:13 PM Lars Göttgens @.***>
wrote:

The comparison tests in
https://github.com/JeffreySarnoff/ArbNumerics.jl/blob/1bd8472c3bfb2607a3d876fff261f11ce113dc70/test/compare.jl#L10-L57
already do a great job in testing that comparisons still hold after
promoting. Nevertheless, I added a few small tests that check the expected
output type of some basic arithmetic operations. Is this what you had in
mind?
—
Reply to this email directly, view it on GitHub
https://github.com/JeffreySarnoff/ArbNumerics.jl/pull/70#issuecomment-2161532796,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM2VRSHXQL3XBTKFIPEIFDZG5K7LAVCNFSM6AAAAABJEBBPV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRRGUZTENZZGY
.
You are receiving this because you commented.Message ID:
@.***>


[Loss] Fix loss layer were allocating a new mem

[Tensor/Clean] Relocate tensor methods
[Tensor] rearrange methods
[Loss] Fix loss layer were allocating a new mem

As allocation is managed in `manager`, layer shouldn't be allocate a new
memory that are managed by memory.
However, loss layer was allocating a new memory. This patch fixes the issue.

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>Cc: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #876. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #876. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/876-202101221716550.77814888954163-806b6f6edf538b67f24cb6084150c24d4584cebc/.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/876-202101221716550.77814888954163-806b6f6edf538b67f24cb6084150c24d4584cebc/.

@zhoonit The reason for the failure of the PR has been identified and it is due to yet another bug in in-place layer optimization.
This will be fixed in a couple of days as it requires some major changes.

@zhoonit The reason for the failure of the PR has been identified and it is due to yet another bug in in-place layer optimization.
This will be fixed in a couple of days as it requires some major changes.

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/876-202102011505350.64488792419434-4130fb0ebc85b053e58039972e0d0bee2f8419a0/.

@zhoonit Now that #880 has been merged, can you please rebase this and check if any issue still exists?

@zhoonit Now that #880 has been merged, can you please rebase this and check if any issue still exists?

Done.


chore(dep): update to helix-fetch v2

Please ensure your pull request adheres to the following guidelines:

[ ] make sure to link the related issues in this description
[ ] when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

Related Issues
Thanks for contributing!

:tada: This PR is included in version 1.1.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


enable/disable Sticky Scroll by language

I really liked the sticky scrool feature when working with Python, Javascript and C++.
But, for me, it gets confuse in a HTML code.
Now iis only possoble to enable or disable it globally and I think it will be a good idea to make this feature language dependent.
Thanks for your attention,
Henrique

"[html]": {
	"editor.stickyScroll.enabled": false
}

Thanks @RedCMD

Thank you!!!

Hi @hgf777-br if you are using the django extension, then you must be working in a python file. In that case you can change the html tag above to python


Replace deprecated aws_s3_bucket_object with aws_s3_object to stop warnings

 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.


Consider adding a note about troubleshooting for the blog

This idea came during morning stand up. @kristenwomack suggested to have some notes for forks about how auth logout and auth login unblocks users after a recent change to subscription roles (like the one that happened to the Playground subscription not so long ago).
We have a troubleshoot section on azd's official docs: https://learn.microsoft.com/azure/developer/azure-developer-cli/troubleshoot
Should we add the note in the troubleshoot and then either keep a constant footer at the end of each release blog post about this troubleshoot page?, or at least mention it from time to time when there might be new entries in the troubleshoot docs?

We already had this section - https://learn.microsoft.com/en-us/azure/developer/azure-developer-cli/troubleshoot?tabs=Browser#updated-azure-account-permissions-do-not-refresh-in-azd

@gkulin added a link at the end of the blog post. Marking this complete.


Separate Dashboard

Currently, the Dashboard replaces the home/start page, when an user is logged in.
This should be separated, so the home/start page is available to logged in users, too. In addition, a navigation menu entry for the dashboard must be available.

Achieving this is simple:

Modify  \Core\Controller\IndexController: Move code dedicated to the dashboard functionality to the dashboardAction() method.
Create a route to the dashboard action model.config.php:81
Create an entry in the navigation menu config module.config.php:250.
Add acl rules to enable/disable access to the dashboard route module.config.php:156
Fix unit and behat tests.

Changes are available in the Demo. So we close this issue.


dotnet templates

It would be nice to have a couple of dotnet templates
I mainly care for wasm bolero, but I guess other might want server bolero/blazor
If I have a chance during the week I'll try to get you a PR

I made a repo for couple of simple usecases: https://github.com/slaveOftime/Fun.Blazor.Samples
Those can be treated as templates.
Yes if we have cli templates, it will be better.

Hey hope all is well, have you had a change to take a look at
https://github.com/slaveOftime/Fun.Blazor.Samples/pull/1 ?

Sorry, I did not noticed the email for that. I will take a look in the next few days.

@AngelMunoz thanks for your work!!! 👍
It is now merged. Github actions also setup for it. It is now published on nuget.


Obfuscate telephone number

Additionally to the email the phone number should also be obfuscated to prevent calls from "Microsoft Support India" pretending PCs of the phone numbers owner are infected with malware.
This obfuscation should work without clownflare dependencies

Support this


Fix readthedocs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.97%. Comparing base (bb8d4ca) to head (36b3060).

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   80.97%   80.97%
=======================================
  Files          23       23
  Lines        1388     1388
=======================================
  Hits         1124     1124
  Misses        264      264

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


refactor: simplify __VLS_IsAny

Thus we don't have to use a hack to solve https://github.com/microsoft/TypeScript/issues/54630

LGTM, thanks!


Ability To Set Cookie Domain On Sticky Sessions

What does this PR do?
Simply added domain to cookie configurations for sticky sessions
Now a Dev can apply the label:
- traefik.http.services.backend.loadbalancer.sticky.cookie.domain=my.desired.cookie.domain.com
or comparable config in toml/yml/json, to specify what cookie domain is set for the sticky session cookie
Motivation
traefik sits between my frontend and backend. So the cookie domain set by traefik does not match the domain of the frontend
ie fontend domain - "memehub.lol". and backend domain - "backend.memehub.lol"
Traefik sets the cookie domain to "backend.memehub.lol" by default itself since that is the domain it sits on.
When the frontend is refreshed, if the cookie domain is not matching the current domain/subdomain then the cookie is lost on the refresh.
After traefik sets the initial sticky cookie, if I manually change the domain of the cookie to the desired cookie domain ".memehub.lol" then the cookie persists through refreshes without issue, BUT, if the cookie domain is left as traefik self sets it "backend.memehub.lol" then the cookie is lost on refresh.
When the cookie is lost, then when replicas for the backend service is more than one, the requests dont get routed to the same server each time as desired, but get defaulted to round robin
Before this PR, Traefik had no way set to the cookie domain.
The package used by traefik for round robin accepts a cookie domain.
Just needed to add a couple lines to add domain to the traefik config and have traefik pass it on through.
Additional Notes
I am a fairly noob coder, less than one year under my belt.
So if I am missing something about this PR process, go lax on me.
I built my pr fork with the exp.Dockerfile and tested it was functional on my site.

idk why make docs failed when I didnt change the docs?

Hello @Distortedlogic,
You've done nothing wrong we had an issue with our CI and we have restarted your builds.

Hello @Distortedlogic,
Thanks for your pull request and your interest in Traefik!
We tried to reproduce the issue you are facing but without any success. When the frontend page is refreshed the sticky cookie is not removed by the browser and sent on refresh to the backend through the AJAX request.
Allowing the configuration of the cookie Domain could lead to misconfigurations, e.g. if the Traefik service with the sticky cookie is used by different routers. We are also thinking that sharing the sticky cookie between subdomains does not really make sense.
According to that, we will close this PR but https://github.com/traefik/traefik/issues/7463 will remain open if you want to provide us with a reproducible case.

Did you have traefik on a different domain than the frontend? The issue occurred for me both on chrome and brave and I provided screen shots on the issue I opened for this prior to the pr. Regardless, why not just state the caveat for any worries in docs like anything else n give the option to use or not to the devs instead of choosing for them?

We are using multiple identical instances of traefik:

running the same synced config
hosting multiple routers, 1 for each subdomain,
routers grouped; cookie name for each router in a group set the same e.g. routerid.md5(groupname),
routers ungrouped; cookie name set to e.g. routerid.md5(fqdn),
subdomains referencing/linking/redirecting to each other.
running over multiple uplinks using multiple a-records+rfc2136

We'd like to share the cookies between subdomains, to be able to make sure a user stays with the same real server as much as possible, no matter which subdomain they are using.
@kevinpollet, I do not see any real issues concerning misconfiguration as one should set the name to something unique for each router/group. So it would be nice if the setting of 'domain' on the cookie was made possible.

Hello @mdklapwijk and thanks for your interest in Traefik,
Please, could you open a feature request describing the use case?

Years later, I still cant believe such a small straightforward useful feature with the code already PR'ed with only 3 lines of code that literally only passes through one additional setting that affects nothing else, and another dev laying out their need requires them to create a feature request? What a clown show 🤡 . Glad I dropped Traefik from my tech stack.


'kubectl get all' command support added

i don't find any alias for "kubectl get all" , so i added it.

You also need to edit the generation script.

done in commit : 145a434ec08c4789843be09eeb528bc1fae84c31 @ahmetb

You also need to edit the generation script.


Image details function

changed function name from image_details to get_image_details
updated function doc string in .py file for get_image_details
updated readme function section for all three functions

Great job! It is better to update Readme in master branch instead of function branch next time.

Good catch, thanks!

I agree with all the changes. thanks!

@olivia-lin I noticed that, in your "test_get_image_details.py", your function documentation has no appropriate indent. pytest reports error due to this problem. please double check and modify it.


CallbackDataField[type] node

@dp.callback_query()
async def my_handler(ticket_id: Field[int], action: Field[enums.TicketAction]):
    ...

need discussion about of validators.
proposed implementation syntax:
async def validator(value: Any) -> list[int]: ...

name: Field[str] # typed value
# ^^^^^ name have "str" type annotation

name: Field[lambda any_value: None] # validate value with any type
# ^^^^^ name have "None" type annotation

name: Field[int, lambda integer: integer ** 2] # vaidate with typed value
# ^^^^^ name have "int" type annotation

name: Field[validator] # allow async validators
# ^^^^^ name have "list[int]" type annotation


Weird install preview message in a yarn repo

Just wanteds to report his as I'm trying this tool out right now - something is wrong with the comment:
See https://github.com/phryneas/remark-typescript-tools/pull/15#issuecomment-2336716697

Oh
What do you think @brc-dd ?


Added test pipeline

Just so we're all clear, this is not running tests in the same way we do locally. May cause problems at some point.

Just so we're all clear, this is not running tests in the same way we do locally. May cause problems at some point.

Discussed, we are OK deferring this risk to later and working through any issues that might come up.


Add examples for common project types (ERC20, ERC721)

The projects that are currently in the examples/ directory aren't very useful. We should add examples that demonstrate how to use Ponder for common/well-understood protocols and standards.
We could also add smaller examples that showcase specific features like custom log filters and proxy contracts.

token-erc20
token-erc721
with-custom-log-filter
with-proxy-contract
project-ens
project-uniswap
etc

Added token-erc20 in this PR: https://github.com/0xOlias/ponder/pull/225


feat: added a link to OSCR tooltips pointing to the OSCR docs

Description
Adds a link to the OSCR docs in OSCR info tooltips.
Related Tickets & Documents
Closes #3920
Mobile & Desktop Screenshots/Recordings
Before

After

Note that the after screenshot is using our standard tooltip making it uniform with our other tootips.
Steps to QA
Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[ ] Tier 3
[x] Tier 4

[optional] What gif best describes this PR or how it makes you feel?

Note that the after screenshot is using our standard tooltip making it uniform with our other tootips.

Note that the after screenshot is using our standard tooltip making it uniform with our other tooltips. I can put it back to the same sizing.

@jpmcb

Note that the after screenshot is using our standard tooltip making it uniform with our other tooltips. I can put it back to the same sizing.

Gotcha - I had assumed the original had some special design, but it seems not: https://github.com/open-sauced/app/pull/3857
Dealers choice!! 🃏


HSEARCH-3883 NullPointerException when building a SimpleQueryString predicate

https://hibernate.atlassian.net/browse/HSEARCH-3883
I didn't add a test because it turns out that the code triggering the NPE was relying on unsupported behavior, but I think this cleanup is a good think regardless. We can ask ourselves whether we want to officially support that use case later.

Thanks Fabio, I applied your suggestions. Will merge as soon as the CI build succeeds.


There is a big bug in value set

Did you try to set a value with a plus character? The result will surprise you:
.env file:

<EMAIL_ADDRESS>MAIL_PASSWORD=hy6gGbS+
MAIL_ENCRYPTION=tls

Executing command: php artisan env:set MAIL_PASSWORD ABCDEF+
Result file looks like this:

<EMAIL_ADDRESS>MAIL_PASSWORD=ABCDEF+
+
MAIL_ENCRYPTION=tls

If your old value has other charaters, you get error:
.env file:

<EMAIL_ADDRESS>MAIL_PASSWORD=hy6^gGbS+
MAIL_ENCRYPTION=tls

Executing command: php artisan env:set MAIL_PASSWORD ABCDEF+
got error Could not set the value in your .env file, reverting...
The problem is preg_replace function...

Ah, well spotted! Seems like you've already identified the bug, can you submit a PR to fix this? If not, I'll have a look at this later next week 🙂

No worries @svytas! I'll pick it up as soon as I can make some time 😄

fixed by #38

Hi, still it works strange - looks like I cannot change value if there are no numbers in it:
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD 2
Successfully set [MAIL_PASSWORD] to [2] in your .env file.
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD 2A
Successfully set [MAIL_PASSWORD] to [2A] in your .env file.
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD A2
Successfully set [MAIL_PASSWORD] to [A2] in your .env file.
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD X2
Successfully set [MAIL_PASSWORD] to [X2] in your .env file.
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD X2X
Successfully set [MAIL_PASSWORD] to [X2X] in your .env file.
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD X2+X
Successfully set [MAIL_PASSWORD] to [X2+X] in your .env file.
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD XX
Could not set the value in your .env file, reverting...
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD XXDD
Could not set the value in your .env file, reverting...
vagrant@homestead:~/test$ php artisan env:set MAIL_PASSWORD AA
Could not set the value in your .env file, reverting...

That's... very  strange. This package desperately needs a rewrite / big refactor anyway, I'll see what I can do!


Screenshot of installing a NuGet package

image for Add usings for types in reference assemblies and NuGet packages

@dixonsj : Thanks for your contribution! The author, @, has been notified to review your proposed change.

@dixonsj Can you give some more context for adding this image? Which topic does it appear in? Thanks.

Hi.
It was meant to be with a seperate or for PR for https://github.com/MicrosoftDocs/visualstudio-docs/issues/1614
I updated the section on using directives but wanted to add a snippet to show the add reference to nuget package.
Thanks
Shane.

From: Genevieve Warren<EMAIL_ADDRESS>Sent: Saturday, October 6, 2018 5:31:15 AM
To: MicrosoftDocs/visualstudio-docs
Cc: Shane Dixon; Mention
Subject: Re: [MicrosoftDocs/visualstudio-docs] Add files via upload (#1619)
@dixonsjhttps://github.com/dixonsj Can you give some more context for adding this image? Which topic does it appear in? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/visualstudio-docs/pull/1619#issuecomment-427475114, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AP5_wjmV3yyB_M6UG478DiE3ShC_w8PAks5uh7ODgaJpZM4XJmxE.

This is the PR  the image goes with, however the PR is only in the user's fork at this point.

@dixonsj Can you submit a new PR that combines the screenshot with the text you want to add, or add the text to this PR? Thanks.

Sorry about that mess.
I have created a new PR https://github.com/MicrosoftDocs/visualstudio-docs/pull/1668
Thanks,
Shane.
From: Genevieve Warren<EMAIL_ADDRESS>Sent: Tuesday, 9 October 2018 4:55 AM
To: MicrosoftDocs/visualstudio-docs<EMAIL_ADDRESS>Cc: Shane Dixon<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/visualstudio-docs] Screenshot of installing a NuGet package (#1619)
@dixonsjhttps://github.com/dixonsj Can you submit a new PR that combines the screenshot with the text you want to add, or add the text to this PR? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/visualstudio-docs/pull/1619#issuecomment-427924489, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AP5_wmR6u9iHOI0GVmewQaBFsEMRCbdGks5ui5GFgaJpZM4XJmxE.


New Tutorial and Comparing Built in

Issue #, if available:
Description of changes:
I created a CloudFormation template that builds a common set of resources for evaluating Amazon Forecast(S3 bucket, IAM policies, and SageMaker Notebook Instance).
Screenshot walkthrough
3 notebooks that illustrate how to prepare data, get a forecast, and evaluate it
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.


[prql-js] Very bad template literal example

What happened?
In the prql-js bindings readme, one can find the following usage example:
const prqljs = require("prql-js");
const prql = (string) => prqljs.compile(string[0] || "");

const sql = prql`from employees | select first_name`;
console.log(sql);

Here, a custom template literal formatter "prql" is defined and then used.
Such template literal formatters are often seen in ORMs to handle automatically escaping variables, e.g.
let username = "this_will_be_escaped"
sql`select * from users where username = ${username}`

the misguided use in the example was presumably picked up from such usecases and then implemented incorrectly.
A template literal formatter will get the raw strings as an array via the first argument, then the variables to insert as spread arguments, e.g.
const sql = (strings, ...variables) => // escape variables and insert them between elements of strings array

The example implementation simply returns the first string element. This means that if, for example
prql`hello ${variable} world`

is called, only hello  will be passed to prqljs.compile.
A template literal function has no purpose here in the first place, as template literals can just be passed to functions directly (where they will first get joined into a string):
compile(`none
of that
nonsense`)

this is not the same as the prql example above; here, the compile function will simply always be called with one string (the result of the template literal).
TLDR the template literal formatter function in the example is not only broken, but also useless and should be avoided due to the convention of template literal formatter functions preprocessing what is passed to them
PRQL input
-

SQL output
-

Expected SQL output
No response
MVCE confirmation

[X] Minimal example
[X] New issue

Anything else?
the error catching (example) of parsing json out of the error message is also pretty horrendous

Thanks for the issue @leumasme
Does anyone here who knows more about JS than me have a view?

I agree, using a template in the example is an overkill, even if prql`from x` looks cool.
@leumasme, do you care opening a PR?


Suggestion: gitter channel

Create a gitter channel to discuss roadmap, issues, etc

A gitter already exists at this url ;) https://gitter.im/tleunen/react-mdl


Issue #318 and #319

See my comment, also looks like you have a build or test error

Addressed by #322


Feature Addition: URL RDM API from ReactMap

I would like to look into having url generated based on a cell pokemon that was identified that could be clickable from a link that would take a user to reactmap and go ahead and have a scanZone calculated for the L15 cell that a cell pokemon is in.  That way, one click from poracle (or other notifier) and one click on the website would enable a user to FIND a cell pokemon.

For simplicity sake, just automatically populate a scanzone level2 at the middle of the l15 cell?

Not sure this is a good enhancement. Would need to make sure the backend properly handles queues. What if 30 users got the discord alert and all decided to click on the link?

Could React store the api calls that it processes for some short period of time, and if a api call matches within X time, ignore it and notify the user that that request is already in queue?

That's completely possible. You'd be moving the queuing problem to the frontend instead of backend. Could store a hash of the URI in the database as a basic shared cache.
If you only have one ReactMap instance running it's possible to do this all in memory but ideally this wouldn't have that limitation. Node scales quite poorly as one instance so a basic cache would be a requirement. Others might have comments.


Cannot access items of a class_name="SysHeader32" control

Hi guys. Can someone please help with the following issue?
I have this control:
TcxCustomInnerListView - ''    (L494, T71, R1428, B868)
['', 'TcxCustomInnerListView', '0', '1']
child_window(class_name="TcxCustomInnerListView")
   |
   | Header - ''    (L494, T71, R494, B71)
   | ['2', 'Header']
   | child_window(class_name="SysHeader32")

And I've added screenshot with output of inspect.exe

And I cannot access the members(Export Setup and Test mode) of this class TcxCustomInnerListView in any way.
I tried to use HeaderWrapper but I got errors:
1.
self.setupList = app.get_main_window().child_window(class_name="TcxCustomInnerListView")
sys_header = self.setupList.child_window(class_name="SysHeader32")
header = HeaderWrapper(sys_header.wrapper_object())
header.click_input(coords=header.get_column_rectangle(1).mid_point())
Error:
pywinauto.findwindows.ElementNotFoundError: {'class_name': 'SysHeader32', 'top_level_only': False, 'parent': <win32_element_info.HwndElementInfo - '', TcxCustomInnerListView, 7867452>, 'backend': 'win32'}

2.
self.setupList = app.get_main_window().child_window(class_name="TcxCustomInnerListView")
sys_header = self.setupList
header = HeaderWrapper(sys_header.wrapper_object())
header.click_input(coords=header.get_column_rectangle(1).mid_point())
Error:
OSError: [WinError 6] The handle is invalid.

Anyone knows what I'm doing wrong? Thanks in advance

Issue solved!
If anyone else has this problem I managed to find the solution by using ListViewWrapper instead of  HeaderWrapper.
The solution that works:
from pywinauto.controls.common_controls import ListViewWrapper

self.setupList = app.get_main_window().child_window(class_name="TcxCustomInnerListView")
item = "Export Setup"
list_view_wrapper = ListViewWrapper(self.setupList.wrapper_object())
list_view_wrapper.select(item)
or
item = list_view_wrapper.item(item)
item.click_input(double=True)

If you're using backend="win32" (default one in Application() object), you should stick to Spy++. Inspect.exe corresponds to Application(backend="uia"). By the way "UI Automation" mode in Inspect.exe shows more info than "MSAA mode" (this is older technology).

How about trying sys_header = self.setupList.child_window(class_name="SysHeader32", visble_only=False) ?

Hi @vasily-v-ryabov Thanks for looking into this. Yes I'm using backend="win32". I'll give Spy++ a try.
I tried sys_header = self.setupList.child_window(class_name="SysHeader32", visible_only=False)  but it returns only
Header - ''    (L494, T71, R494, B71)
['', 'Header']
child_window(class_name="SysHeader32")

It may be because TcxCustomInnerListView is a DevExpress element and I saw that those cannot be access like the others.

DevExpress controls is a commercial product. They have closed source implementation and only TestComplete or Squish can work with all their features. Maybe trying HeaderWrapper from "win32" backend can help partially.