nixos/duckdns: init module

Description of changes
Added a Systemd service for updating DuckDNS domains
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

You updated at the same time I commented 😀
However, the secret is still visible in process listings.

I think "curl -K ./secret_params.txt ..." where secret_params.txt contains lines of "-d key=value" is the way to go. "replace-secret" in nixpkgs might help with securely preparing the file. (Untested.)

Used the -K parameter and some shell redirection to hide the secrets in the process list

Used the -K parameter and some shell redirection to hide the secrets in the process list

Thanks!
Please rebase / squash the commits and I can take another look. (I think those 11 commits are just fixups to the one "init" commit?)

Used the -K parameter and some shell redirection to hide the secrets in the process list

Thanks!
Please rebase / squash the commits and I can take another look. (I think those 11 commits are just fixups to the one "init" commit?)

Done :)

@bjornfor @h7x4 Are there any other changes I should make before this can be merged?


Problem: No implementation found for method configure on channel tpay

Hello,
I'm trying to implement the tpay library for Flutter, but I'm having trouble with it. Every time I try to use it, I encounter the following error:
No implementation found for method configure on channel tpay
The same issue occurs when I try to run the example provided in your repository. This happens with every action I try to perform, such as fetching available payment methods or making standard payments, using BLIK, Google Pay, etc.
I would appreciate your assistance.

Close.


error starting dnscrypt  2.0.14 on android 4.4.2

root@localhost:/data/local/entware/root/android-arm# ./dns* --version
CANNOT LINK EXECUTABLE: cannot locate symbol "sigfillset" referenced by "./dnscrypt-proxy"...
and at the same time
root@localhost:/data/local/entware/root/linux-arm# ./dns* --version
2.0.14
this is starting/working ok, but cannot be installed as service on android

Android 4.4 is so old. I don't think it's supported by Go.

jedisct1,
if there are no workaround for 4.4, than mark the required android's version in the description, pls.
jfyi: on 4.4.2 linux-arm version of dnscrypt works fine when starts from /etc/init.d, but one had to know this way and avoid starting it as service.

I think that error is to do with the Android NDK binaries, they are built with --api 21 in .travis.yml. We could try api 14, that should give us Android 4.0 compatibility.
Also the service feature won't work on any Android version, that vendor module simply doesn't support Android. /etc/init.d or the like must be used.

Sporif, jedisct1,
once again: linux-arm version works fine, but on android another feature is required (and i cannot make it work in a bash script) - forking from root to nobody after successful start of dnscrypt-proxy.
otherwise one had to open internet access on firewall to all programs with root permission, which is totally undesirable.


get error when use Recombee in java

Hi,I used the sample code in this repository,but when I run this line:
client.send(new Batch(addUserRequests));
I get this error:
java.lang.NoSuchMethodError: No static method encodeHexString([B)Ljava/lang/String; in class Lorg/apache/commons/codec/binary/Hex; or its super classes (declaration of 'org.apache.commons.codec.binary.Hex' appears in /system/framework/org.apache.http.legacy.boot.jar)
please help me...
thanks

Hi,
sorry for late reponse, I must have somehow overlooked the email :(
I just tried both examples using Java 8 (OpenJDK) and they worked for me.
Which version of Java do you use?


update zh-hans

Proposed changes
Update zh-hans

Types of changes

Bugfix (non-breaking change which fixes an issue)

Linked Issues

Further comments

needs rebase


Problems with CreateSrsMedia.ps1 script - Fails on downloading driver

I am needing to load the Skype Room System software on a handful of Surface Pro 6 tablets for a customer of ours. I downloaded the latest version of the script, v 1.4.1, from the Microsoft website (https://docs.microsoft.com/en-us/skypeforbusiness/deploy/deploy-clients/console). The script runs fine until it gets to the step to download the driver. I can see the file it attempts to download when it creates the temp copy in the directory is "SurfacePro6_Win10_17134_1808507_3.msi"
Below is the saved transcript from the execution of the script and below that the output of the Powershell session. Any help would be appreciated.

Windows PowerShell transcript start
Start time:<PHONE_NUMBER>2551
Username: DOMAIN1\stonebea
RunAs User: DOMAIN1\stonebea
Machine: AVIPF0WBQRN (Microsoft Windows NT 10.0.15063.0)
Host Application: C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
Process ID: 9164
PSVersion: 5.1.15063.1563
PSEdition: Desktop
PSCompatibleVersions: 1.0, 2.0, 3.0, 4.0, 5.0, 5.1.15063.1563
BuildVersion: 10.0.15063.1563
CLRVersion: 4.0.30319.42000
WSManStackVersion: 3.0
PSRemotingProtocolVersion: 2.3
SerializationVersion: <IP_ADDRESS>

Transcript started, output file is C:\Users\stonebea\Documents\PowerShell_transcript.AVIPF0WBQRN.4RBQN7KL.20190215082551.txt
Script version 1.4.1
Downloading CreateSrsMedia...
done
Validating signature for update.ps1...
success.
What type of customer are you?
Please make sure you have all of the following available:

A USB drive with sufficient space (16GB+).
The contents of this drive WILL BE LOST!
Windows 10 Enterprise media that matches your SRSv2 deployment kit.
Enterprise customers can access Windows 10 Enterprise media from the Volume Licensing Service Center:
http://www.microsoft.com/licensing/servicecenter
Any language pack (LP and/or LIP) files to be included.
MPSA customers:         http://go.microsoft.com/fwlink/?LinkId=125893
Other volume licensees: http://www.microsoft.com/licensing/servicecenter

Please do not continue until you have all these items in order.
Downloading deployment kit...
done
Validating signature for SRSDeploymentKit-<IP_ADDRESS>.msi...
success.
Extracting the deployment kit...
done.
This deployment kit is built for Windows build 10.0.17134.1.
This deployment kit draws its drivers from %configsetroot%\AutoUnattend_Files\DistShare\Out-of-Box Drivers.
Please indicate what drivers you wish to use with this installation.
[ 0] Logitech SmartDock
[ 1] Crestron SR
[ 2] Polycom MSR
[ 3] Lenovo Hub 500
[ 4] LRS
[ 5] HP Slice 2.0
[ 6] None
Selected option 0.
[ 0] Surface Pro 4
[ 1] Surface Pro (2017)
[ 2] Surface Pro 6
Selected option 2.
Downloading driver...
PS>TerminatingError(Save-Url): "Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out""

TerminatingError(Save-Url): "Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out""
TerminatingError(Save-Url): "Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out""
TerminatingError(Save-Url): "Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out""
TerminatingError(Save-Url): "Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out""

Windows PowerShell transcript end
End time:<PHONE_NUMBER>2941

Windows PowerShell
Copyright (C) 2016 Microsoft Corporation. All rights reserved.
PS C:\windows\system32> cd ....\SkypeRoomSystem
PS C:\SkypeRoomSystem> ls
Directory: C:\SkypeRoomSystem

Mode                LastWriteTime         Length Name

-a----        2/15/2019   7:55 AM          62349 CreateSrsMedia.ps1
PS C:\SkypeRoomSystem> .\CreateSrsMedia.ps1
Transcript started, output file is C:\Users\stonebea\Documents\PowerShell_transcript.AVIPF0WBQRN.tITBBfia.20190215180304
.txt
Script version 1.4.1
Downloading CreateSrsMedia... done
Validating signature for update.ps1... success.
What type of customer are you?
OEM or Enterprise: Enterprise
Please make sure you have all of the following available:

A USB drive with sufficient space (16GB+).
The contents of this drive WILL BE LOST!
Windows 10 Enterprise media that matches your SRSv2 deployment kit.
Enterprise customers can access Windows 10 Enterprise media from the Volume Licensing Service Center:
http://www.microsoft.com/licensing/servicecenter
Any language pack (LP and/or LIP) files to be included.
MPSA customers:         http://go.microsoft.com/fwlink/?LinkId=125893
Other volume licensees: http://www.microsoft.com/licensing/servicecenter

Please do not continue until you have all these items in order.
Downloading deployment kit... done
Validating signature for SRSDeploymentKit-<IP_ADDRESS>.msi... success.
Extracting the deployment kit... done.
This deployment kit is built for Windows build 10.0.17134.1.
This deployment kit draws its drivers from %configsetroot%\AutoUnattend_Files\DistShare\Out-of-Box Drivers.
Please indicate what drivers you wish to use with this installation.
[ 0] Logitech SmartDock
[ 1] Crestron SR
[ 2] Polycom MSR
[ 3] Lenovo Hub 500
[ 4] LRS
[ 5] HP Slice 2.0
[ 6] None
Selection: 0
Selected option 0.
[ 0] Surface Pro 4
[ 1] Surface Pro (2017)
[ 2] Surface Pro 6
Selection: 2
Selected option 2.
Downloading driver... Transcript stopped, output file is C:\Users\stonebea\Documents\PowerShell_transcript.AVIPF0WBQRN.t
ITBBfia.20190215180304.txt
Save-Url : Exception calling "DownloadFile" with "2" argument(s): "The operation has timed out"
At C:\SkypeRoomSystem\CreateSrsMedia.ps1:865 char:26

            $file = (Save-Url $URL "driver")

                     ~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Save-Url], MethodInvocationException
FullyQualifiedErrorId : WebException,Save-Url

PS C:\SkypeRoomSystem>

Tried on my home computer, had no issues. Must be some weird policy issue on my work machine. Sorry for the trouble.


"let" colorization on Javascript (ES6) changed with update to Version 0.10.10 (0.10.10)

After I got the update to Version 0.10.10 (0.10.10) for OSX, the Javascript "let" keyword is now white (used to be blue, recognized as a reserved word).
I tried adding back the jsconfig.json (which I understand is no longer needed to enable ES6) and the colorization is still no good.
I wonder if other keywords are being colorized incorrectly.

duplicate of https://github.com/Microsoft/vscode/issues/3903

Workaround is to switch to the Light+/Dark+ themes


Implementation of observe on root.

This commit include:

implementation of observe operation on root
test of observe on root

I looked at it but didn't test it yet.
At first sight, this sounds good. :+1:
Only little detail maybe we can change the test a little by adding :

do a modification  which will a notification
check the notification content is OK.

To trigger the notification you can either  :

send a WriteRequest.
or Look at TestObject (Id 3442), you can use leshan-server-demo and leshan-client-demo to see how it works
(and see resource /3442/0/0, /3442/0/1, /3442/0/2)

Note that we are currently in code freeze for the 2.0.0-M14 release so we will be able to integrate it in master only after the release. :slightly_smiling_face:

@mgdlkundera do you plan to modify the test ? If not let me know, so I do it :slightly_smiling_face:

Ok I will try to work on this tomorrow.

I also fixed an issue in test where you used ObserveRequest instead of ReadRequest :
https://github.com/eclipse-leshan/leshan/blob/55f008fa838bc05d4b2c8556dee54ba23bbd087e/leshan-integration-tests/src/test/java/org/eclipse/leshan/integration/tests/observe/ObserveCompositeTest.java#L302-L303
If you are curious, do not hesitate to look at the code
@mgdlkundera  Thx for you contribution :pray:

I'll look at the code. Thank you for your help.


Issue in configure a multer middleware.

Hello Author,
I need to use multer package in this application.  below is my code:-
const upload = multer({ dest: 'uploads/' })

upload_router.use('/file-uploads', upload )
upload_router.post('/file-uploads', async (request, response) => {
const file = request.file.path

console.log('file') // undefined

response.send("ok")
})

I am uploading a file via http request body(form-data) using postman with a 'file' key. but getting file as undefined in the console after the upload. Please help me to find what went wrong.

Hi, this could potentially mean that the multer package is not compatible with HyperExpress out of the box.
Have you tried using the built-in multipart file uploading within HyperExpress?

Thanks for your reply.  built-in multipart file is working out of the box. But in documentation it is not clear that how can we enhance  file handling limitation.I am going to handle heavy files of minimum size 5gb. However in multer we can define maximum file size  to be handled                                                                                const upload = multer({   storage: storage,   limits: { fileSize: maxSize /* bytes */ } })                                                                                Can we handle it like below? what is your opinion?                                                  ```
const fileSize = parseInt(request.headers["content-length"])
if ((file.mimetype === "image/png" || file.mimetype === "image/jpg" || file.mimetype === "image/jpeg" || file.mimetype === "application/octet-stream") && fileSize <=<PHONE_NUMBER>0) {
    cb(null, true)
} else if (file.mimetype === "video/mp4" && fileSize <=<PHONE_NUMBER>0) {
    cb(null, true)
} else {
    cb(null, false)
}

Thanks for your reply.  built-in multipart file is working out of the box. But in documentation it is not clear that how can we enhance  file handling limitation.I am going to handle heavy files of minimum size 5gb. However in multer we can define maximum file size  to be handled
const upload = multer({
  storage: storage,
  limits: { fileSize: maxSize /* bytes */ }
})
```                                                                                Can we handle it like below? what is your opinion?

const fileSize = parseInt(request.headers["content-length"])
if ((file.mimetype === "image/png" || file.mimetype === "image/jpg" || file.mimetype === "image/jpeg" || file.mimetype === "application/octet-stream") && fileSize <=<PHONE_NUMBER>0) {
    cb(null, true)
} else if (file.mimetype === "video/mp4" && fileSize <=<PHONE_NUMBER>0) {
    cb(null, true)
} else {
    cb(null, false)
}

Thanks for your reply.  built-in multipart file is working well for normal cases. But multipart does not work well during heavy lifing of files,I think may be we need to define file size in bytes for big files. I am going to handle big files(minimum size 5gb). However in multer we can define maximum file size  to be handled and multer works well with big files. Can we handle multipart like below? what is your opinion?
```
const fileSize = parseInt(request.headers["content-length"])
if ((file.mimetype === "image/png" || file.mimetype === "image/jpg" || file.mimetype === "image/jpeg" || file.mimetype === "application/octet-stream") && fileSize <=<PHONE_NUMBER>0) {
    cb(null, true)
} else if (file.mimetype === "video/mp4" && fileSize <=<PHONE_NUMBER>0) {
    cb(null, true)
} else {
    cb(null, false)
}


simplified ut-s

Description of changes

Improvements

simplified ut-s

cleaned up a typing import

Test plan

[+] Tests passed successfully locally with pytest for python, yarn test for js

Documentation Changes
N/A

Thanks @leo-gan


Improve OkHttp documentation.

Combine the marketing page (index.md) with the GitHub README.md.
Fix links pointing to 3.x documentation.
Use Markdown tables instead of HTML tables.

Merging docs changes optimistically! “LGTM” Reviews still welcome.


BUG fix:  Balance Contract commands

The crash problem was fixed by: bc2a8d12a217325403858f3f466ed72dad9aa687.
As the PR was merged manually I'm going to close this PR


Replacement is susceptible to memory leaks. Send an event when a node is replaced so things can be cleaned up

I noticed when using jQuery that if there is a bound function used as a callback for a jQuery event on a node that is replaced, the object the function belongs to never gets garbage collected. It's happening because when using native DOM manipulation jQuery event listeners do not get cleaned up; we need to use specific functions in jQuery to ensure that.
For example, this leaks an instance of Leak:
class window.Leak
  constructor: (node, @data) ->
    $(node).on "change", @onNodeChanged

  onNodeChanged: =>
    console.log "node changed with data", @data

node = $('

')[0]
document.body.appendChild(node)

new Leak(node)
$(node).trigger('change')

node2 = $('')[0]
document.body.replaceChild(node2, node)
node = null

If you do $(node).remove() before node = null then there's no leak.
This change won't actually clean up the memory leak, but if someone uses jQuery in their project then adding something like the following will fix the leak:
document.addEventListener 'page:after-node-removed', (event) ->
  $(event.data).remove()

See the leak happening here. To see the leak (in chrome)

Run the snippet
In the dev tools, open profiles and make a heap snapshop
At the top, enter Leak in the Class filter input. You'll see that there's an instance in memory

To see it fixed, uncomment the remove line at the bottom and repeat.

Great find, but we'll need to tweak this a little bit compared to Turbograft:

we shouldn't trigger the event on permanent nodes (L176)
we should trigger the event on the original nodes, not clones (L179)
when we're cloning nodes, we need to make sure we don't trigger the event on a body element that contains the original of those nodes
no need for removeNode
this needs tests (Turbolinks has some now :P)

I started working on this yesterday and have a PR almost ready to go.

We also need to make sure that this plays well with back/forward navigation.

Closing this in favor of #485.


Renamed test files in a standardised way

Fix related to #6

Can you please add some more testcases suitable for testing, according to your cognizance in this PR ?

@Commit2Cosmos why this issue is closed ?

Hi, I was trying to figure out how to merge and squash commits (relatively new to more advanced git stuff), didn't want it to break anything, I'll just open a new PR.


[WIP] fix test

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f3cc63b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage          ?   43.43%
=========================================
  Files             ?       43
  Lines             ?     1630
  Branches          ?        0
=========================================
  Hits              ?      708
  Misses            ?      922
  Partials          ?        0

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 f3cc63b...fefe5fd. Read the comment docs.

@jindulys Done. Please review.


add autohotkey2-lsp

Another attempt to add autothokey2-lsp: https://github.com/thqby/vscode-autohotkey2-lsp. There was a PR #3537  about this. The PR was closed due to unusual installing script. This PR uses the standard process of npm compile to build the lsp.

Any update on this PR?


RSunstone advanced filters special characters issue

Description
When using the advanced filters in RSunstone is not doing correctly for strings with special regex characters such as +, *, etc.
To Reproduce
Try to use advanced filters with texts that have special regex characters.
Expected behavior
Filters to bring the current data.
Details

Affected Component: Sunstone
Hypervisor: [e.g. KVM]
Version: 6.4.2, development

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

Code: https://github.com/OpenNebula/one-ee/pull/2357
Docs: https://github.com/OpenNebula/docs/pull/2427


Use guess_assess to determine confidence?

In compat.detect, the confidence is always 0.99: https://github.com/john-parton/chardetng-py/blob/f90b454b90558003824dafbd6d5c5d95be9a2b50/python/chardetng_py/compat.py#L16-L18
It’s possible I’m misunderstanding things, but it seems like you could use guess_assess() instead of guess() to at least determine high vs. low confidence. From the chardetng docs:

Same as guess(), but also returns a Boolean indicating whether the guessed encoding had a higher score than at least one other candidate. If this method returns false, the guessed encoding is likely to be wrong.

On a related note, it would be nice if shortcuts.detect() returned the boolean from guess_assess, too (or had another method that did so), since the aliases buried in the shortcuts module seem pretty important, and possibly prone to change. Otherwise a user has to know about them if they want to safely use guess_assess instead. (Or the aliases could be documented and moved to a more accessible place.)

Thanks for taking the time to comment.
If you're integrating this into a new project, my recommendation is to directly use the class bindings: https://chardetng-py.readthedocs.io/en/latest/class_reference.html
Your code should make a determination of how important the higher_score return value is.
However, I do agree that we could return two different confidence values depending on the value of higher_score. Perhaps is higher_score is False, the confidence value can be 0.01 and if it's True it can be 0.99. Thoughts?
I would happily accept a pull request that updates the documentation to drive people to use the class bindings. They're actually really nice, and you can do some things with it that other libraries can't even do. For instance, you can read a file in fixed sized chunks and feed them into the detector chunk-by-chunk, get a detected encoding, and then put the cursor back at the start of the to start reading it as text. In that way, you can cap the memory use of the application.The other python libraries that do charset detection require you to read the entire file into memory.

Basic outline of the proposed change. https://github.com/john-parton/chardetng-py/pull/110/files
Needs docs

However, I do agree that we could return two different confidence values depending on the value of higher_score. Perhaps is higher_score is False, the confidence value can be 0.01 and if it's True it can be 0.99. Thoughts?

That makes sense to me!

If you're integrating this into a new project, my recommendation is to directly use the class bindings: https://chardetng-py.readthedocs.io/en/latest/class_reference.html

I guess my concern here was that, if I do this, I lose the aliases support from the shortcut: https://github.com/john-parton/chardetng-py/blob/65e0a5aee04136ed885ef903354255e79f0b567a/python/chardetng_py/shortcuts.py#L7-L9
The code and discussion in #11 read like those are important because some_bytes.decode(encoding_name) might not work like you’d expect without them. (Am I getting that wrong?) It also seems like that’s a list that might change, and I’d like to not have to remember to check the source on every release to see if they are different and update my code that doesn’t use the shortcut to match.
Maybe the right answer there is to get the aliases out of the shortcut module and into src/lib.rs? (Or some new Python code that wraps the Rust EncodingDetector instead of directly exposing it?)

Also: I didn’t realize there were better docs on ReadTheDocs! 👍  (I had figured out how to use EncodingDetector by using the source.) It might help more people find them if you call that out directly in the text of the README (not just the little status shield at the top, which also currently lists ReadTheDocs as failing) and maybe set it as the website URL for the repo on GitHub and as a project link on PyPI (you can set up the links on PyPI using [project.urls] in pyproject.toml — urllib3 is a good example).

Perhaps it would be best if the aliasing were done in rust, in the glue code I have.  Good points in making it clearer that there's docs on readthedocs. I'll go create some separate issues for those.

Here's a PR that moves the ALIASES logic into lib.rs https://github.com/john-parton/chardetng-py/pull/112

I merged in some changes to get the docs building again and added a clear link to the documentation in the readme.

Closed by https://github.com/john-parton/chardetng-py/pull/110
Documentation has been updated to reflect the change. I'm not sure 0.99 and 0.01 are the best values, but if your application really cares, it should probably use the EncodingDetector class directly and use the bool it returns in guess_assess


added ArcGIS Online provider

This adds a provider for authenticating with ArcGIS Online using a named user account using the User logins via PHP, JSP, ASP.NET, and other server-based web apps workflow.
If you go to developers.arcgis.com and sign in you are able to create a new application to get the client Id and secret. You need a named user account to log in though, this is either an organizational account or a Portal for ArcGIS account.
Note that there is no state token passed so I had to set the AuthenticationProperties RedirectUri to the expected account controller callback method.

@davetimmins We are looking for help moving to ASP.NET 5. Please read http://www.jerriepelser.com/blog/calling-contributors-for-oauth-providers


Compatible with 1.8?

So is it compatible with v1.8?

All Tests are passing with 1.8. Please give it a try and report any problems you encouter.


feat: expose metrics port in helm charts

Reason for Change:

Exposing port to scrape metrics in PodMonitor.
Similar to https://github.com/kubernetes-sigs/secrets-store-csi-driver/blob/22b661d0bbaa896127726dbf9fd1c9c1e3d27230/manifest_staging/deploy/secrets-store-csi-driver.yaml#L69-L75

Requirements

[x] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[x] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

Done, PTAL

Isn't it 8095 according to this wiki?
https://learn.microsoft.com/en-us/azure/aks/csi-secrets-store-driver#the-secrets-store-csi-driver

/azp run pr-e2e

Codecov Report

Merging #1119 (00d052a) into master (d40addb) will not change coverage.
The diff coverage is n/a.


CIP-????: First class errors

@michaelpj @effectfully
Rendered in branch
Initial discussion: https://github.com/cardano-foundation/CIPs/pull/459#issuecomment-1443904223

Reply to https://github.com/cardano-foundation/CIPs/pull/459#issuecomment-1450854400 :
I hope the behaviour is more obvious given the explanation in this CIP?
The factorial implementation blows up because everything is still evaluated strictly.
The ifThenElse example doesn't because passing an error to it is entirely valid.

I realised another benefit: This allows optimising expensive1 || expensive2 into if expensive1 then True else expensive2, the reason being that the language is not truly pure (modulo traces, which we don't care about in optimisations).
This is evident by the fact that evaluating something doesn't matter wrt. transaction validity (modulo budgets, but optimisation already messes with that), so we can in fact in general entirely eliminate unused terms with this change.
We also fix the other issue I noted in https://github.com/input-output-hk/plutus/issues/4826, since const can now truly ignore the error!
This might be another (quite big?) reason to introduce this change to Plutus.

@L-as

I hope the behaviour is more obvious given the explanation in this CIP?
The factorial implementation blows up because everything is still evaluated strictly.
The ifThenElse example doesn't because passing an error to it is entirely valid.

I understood the proposed behavior, but I was failing to understand the motivation for it. Now that I've read this CIP I do indeed understand the motivation, thanks a lot for writing it out.
I think I agree that doing what you suggest would be an improvement over the status quo. I wouldn't like the resulting system, but less so than the current one.
However to be completely frank, I believe chances of anybody being able to convince our academics to give up on canonicity and to convince our managers to overhaul the whole evaluation semantics of the language (i.e. the implementation, the spec and the formalization) are pretty slim. As such, I don't feel like it's a battle worth fighting. Plus, if we were to overhaul the whole evaluation semantics, I would prefer some other solution, perhaps something based on call-by-push-value. Or maybe even just making the language lazy.
Overall, I think that ship has sailed, unfortunately.

I believe chances of anybody being able to convince our academics to give up on canonicity

I don't see why it would be any different from the status quo wrt. canonicity.

I would prefer some other solution, perhaps something based on call-by-push-value. Or maybe even just making the language lazy.

I think we will likely find a better solution in the future, however, this is a very minor change that can be done without changing most of the ecosystem.

to overhaul the whole evaluation semantics of the language (i.e. the implementation, the spec and the formalization)

Well that's precisely what it isn't, I'd say! This is a minor change to the implementation, the spec, and the formalisation.

Haskell and GHC had introduced pretty major overhaul to its errors and introduced more complex exceptions. So may there be a chance to do everything in a monad?

What do I need to do to get this in now with the new structure?

I wrote this last year:

I understood the proposed behavior, but I was failing to understand the motivation for it. Now that I've read this CIP I do indeed understand the motivation, thanks a lot for writing it out.

I wish I spelled out what convinced me, because rereading the CIP I'm not convinced that it would be worth doing that even given infinite resources for development. I was probably just sloppy last year.
One of the proposed changes is this:

Erase type abstractions and type applications entirely when going from TPLC to UPLC rather than turning them into delay and force.

But that just can't work. Take the following TPLC expression:
let factorial n = ifThenElse {forall a. integer} (\{_} -> n == 0) 1 (\{_} -> factorial (n - 1)) {()}
in factorial 2

If we erase all type abstractions and type applications from it while going from TPLC to UPLC, we'll get
let factorial n = ifThenElse (n == 0) 1 (factorial (n - 1))
in factorial 2

which is an infinite loop as the proposal acknowledges, unlike the original expression.
The proposal says

Very few people will make their own loopforever function to replicate the old error to write impure scripts.

but if you start erasing type abstractions and type applications left and right you'll get those loopforever functions all over the place without bothering to put them into the program manually.
@L-as

I don't see why it would be any different from the status quo wrt. canonicity.

You're proposing to add an axiom that every type is inhabited by the special ERROR construct, it's a textbook example of how to break canonicity, see e.g. this.
Can we have a formal model where that rule isn't an axiom, but rather an inherent part of the system, like "every type has an ERROR constructor, including even ->" and then elaborate TPLC/UPLC into that system? Maybe, but I doubt anybody is going to do it. It's a major change to the semantics of our languages. And having implemented and recently refactored the way we do error handling in the UPLC evaluator, I can assure you it's plenty of work, so I very much disagree with this:

Well that's precisely what it isn't, I'd say! This is a minor change to the implementation, the spec, and the formalisation.

If you believe it's a minor change anyway, we'd appreciate a branch implementing the proposed changes to handling of ERROR. We'll implement erasure of forces and delays ourselves just to meet you halfway.

Thanks, that’s a very concrete path forward. Maybe I’ll look at it when I have time.

@L-as the last comment (after a pessimistic review + a still-unaddressed request to update to the canonical document structure), now that a couple more months have gone by, suggests that you don't have the time to pursue this.
Ordinarily we would close it when an author doesn't have time to complete a CIP but I don't want to bury the work you & @effectfully have done to document this issue... so in the meantime I'm tagging this Abandoned which will close it later... if & when editors don't see any progress as apparently agreed after https://github.com/cardano-foundation/CIPs/pull/469#issuecomment-2158542726.

Thank you. I wouldn't necessarily say this will never happen, you should have a long-term view. I'm busy right now with some other open source stuff but when I have that done it's possible I will do this.

Rip


TODO tracker for ongoing training loop refactors

📚 TODOs around the loop refactor
This issue tracks the work planned for and after the loop refactor #7700.
Core contributors feel free to update the description here directly.
Hygiene and code quality

[ ] Typing for full mypy compatibility (see ignored files in setup.cfg)
[x] Documentation for the public interface + address https://github.com/PyTorchLightning/pytorch-lightning/pull/8065#discussion_r656160991
[x] Evaluate which methods should be public and which should be protected
[x] Rename Trainer.train_loop to Trainer.fit_loop and deprecate the former access.
[x] Formalize how arguments are forwarded to lower loops (especially in connect

Feature integration and follow-up refactors

[x] Extract a OptimizerLoop, potentially a loop for TBPTT  #9191

[x] extract utility for _run_optimization
[x] for manual optimization, replace closure call with direct call to _training_step()

[x] Integrate progress tracking #6429, this will also increase clarity and minimize attack surface for bugs!
[x] Determine who should own the dataloaders and responsibility of resetting them https://github.com/PyTorchLightning/pytorch-lightning/pull/7871#discussion_r649680923. Can references fall out of sync? https://github.com/PyTorchLightning/pytorch-lightning/pull/7871#discussion_r649680539
[x] Add a teardown, let loops move their results to CPU instead of LoggerConnector: https://github.com/PyTorchLightning/pytorch-lightning/pull/8245
[x] The EvaluationDataLoaderLoop.predictions are currently accumulated inside the inner evaluation loop instead of per epoch and then combined over all dataloaders. #8850
[x] Rename inner loops to epoch_loop, batch_loope etc to distinguish top level loop access. Also propose file structure organizing these. Resolved by #8077

Customization

[x] Formalize how loops get instantiated, set on the trainer, and connected to each other. #8226

Miscellaneous

Several todo's are annotated as TODO(@awaelchli)
Some todo's are annotated as TODO(@justusschock) as well.

cc: @justusschock @carmocca @ananthsub @tchaton

Closing as everything looks done here


fix(rules): Fixed an issue where rule handling would fail when the Pl…

…ex collection was manually removed. The collection will now get recreated.

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


occasional screwup in send_activity without clear error messaging

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
What package version of the SDK are you using.
4.14.1
Describe the bug
Give a clear and concise description of what the bug is.
This is the traceback, there is no proper error message.  So for starters the bug is the lack of clear exception messaging.
File "/usr/local/lib/python3.7/dist-packages/botbuilder/core/turn_context.py", line 174, in send_activity
result = await self.send_activities([activity_or_text])
File "/usr/local/lib/python3.7/dist-packages/botbuilder/core/turn_context.py", line 226, in send_activities
return await self._emit(self._on_send_activities, output, logic())
File "/usr/local/lib/python3.7/dist-packages/botbuilder/core/turn_context.py", line 304, in _emit
return await logic
File "/usr/local/lib/python3.7/dist-packages/botbuilder/core/turn_context.py", line 221, in logic
responses = await self.adapter.send_activities(self, output)
File "/usr/local/lib/python3.7/dist-packages/botbuilder/core/bot_framework_adapter.py", line 733, in send_activities
raise error
File "/usr/local/lib/python3.7/dist-packages/botbuilder/core/bot_framework_adapter.py", line 719, in send_activities
activity.conversation.id, activity.reply_to_id, activity
File "/usr/local/lib/python3.7/dist-packages/botframework/connector/aio/operations_async/_conversations_operations_async.py", line 525, in reply_to_activity
request, stream=False, **operation_config
File "/usr/local/lib/python3.7/dist-packages/msrest/async_client.py", line 115, in async_send
pipeline_response = await self.config.pipeline.run(request, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/msrest/pipeline/async_abc.py", line 159, in run
return await first_node.send(pipeline_request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/dist-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/dist-packages/msrest/pipeline/async_requests.py", line 106, in send
return await self.next.send(request, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/dist-packages/msrest/pipeline/async_abc.py", line 79, in send
response = await self.next.send(request, **kwargs)  # type: ignore
File "/usr/local/lib/python3.7/dist-packages/msrest/pipeline/async_requests.py", line 85, in send
await self.driver.send(request.http_request, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/msrest/universal_http/async_requests.py", line 91, in send
return await super(AsyncRequestsHTTPSender, self).send(request, **requests_kwargs)
File "/usr/local/lib/python3.7/dist-packages/msrest/universal_http/async_requests.py", line 79, in send
await future
concurrent.futures._base.CancelledError
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

this is not reproducible consistently, it happens sporadically.
Expected behavior
Give a clear and concise description of what you expected to happen.
bot sends a message or at least gives a clear exception for why it can not do so.
Screenshots
If applicable, add screenshots to help explain your problem.
N/A
Additional context
Add any other context about the problem here.
web server is aiohttp, straight from your MS code samples.

Hi @master0v,
I understand you're having trouble with the error messages, but it would be helpful if you could provide more information about exactly what behavior you're experiencing, and the conditions under which it occurs.

Are messages simply not getting sent?
Specific types of activities, or all of them?
Which channel?
Have you deployed to Azure? If yes, how did you do it?
How are you testing the bot?
Are you using a sample bot? If so, which one and what changes did you make to it (if any)?
Have you tried debugging the bot?

Hi Anish, thanks for your reply and your questions.  Let me address all of them

Are messages simply not getting sent?

yes, the message doesn't get sent when this error happens

Specific types of activities, or all of them?

doesn't look like it has to do with activity type. I am simply calling "TurnContext.send_activity()" function

Which channel?

Microsoft Teams

Have you deployed to Azure? If yes, how did you do it?

No, and not planning to.

How are you testing the bot?

I have it deployed in my development msTeams environment.  I found that it behaves differently in Teams than in other channels, or in testing, so I don't see the point in testing it with anything other than the Teams itself, since the final deployment is going to MS Teams.

Are you using a sample bot? If so, which one and what changes did you make to it (if any)?

yes, using modified EchoBot from BotBuilder-Samples

Have you tried debugging the bot?

yes, of course

Have you noticed any patterns to the errors?

No

Teams does indeed behave differently. Have you tried using one of the Teams bot samples instead? They're written slightly differently. Give the Teams Conversation Bot a try as-is and let me know if the same issues persist, and error messages appear, if any.

Hi Anish, thanks for the suggestion.  I've examined that sample as well.  My use case is really very simple: it is a 1:1 conversation with the bot.  User says something, the bot replies.   It would've been good if Teams supperted Suggested Actions, but it doesn't so I am forced to use a HeroCard with buttons.  Other than that, I don't see anything in that sample, that I can use to improve my bot.

As your bot is not currently functioning correctly, I would like to use the unmodified Teams sample to identify where the issues are coming from.
At this point, we don't know if the issue is in your bot, in the Azure Bot service, or in Teams. If you test connecting an unmodified sample written specifically for Teams to your Teams environment and we don't see the same behavior, we'll know the bug is in the code or vice versa.

ok, I will try that and report back.  However, I must point out that even in my code the error doesn't happen consistently or frequently.  I will have to write a stress test in order to really push that sample to break.

I haven't been able to reproduce the issue thus far over the various Teams test bots I have. It could be specific to your environment, or it could be some other issue. Until we narrow it down, we can't begin to fix it.

Could you at least add some additional messaging, and make sure all of the exceptions get thrown with good/clear messaging?

btw, it looks like I am not the only one having this issue: https://github.com/microsoft/BotBuilder-Samples/issues/2738

That's an old post on a much older version, and it has no resolution, so unfortunately it may not be of much help to us. Regardless, were you able to test a clean sample? What were the results?

@master0v - FYI, going forward, I'll be assisting you with this issue as @anishprasad01 will be unavailable for an extended amount of time.
To be clear, is the goal here to improve the error messaging, to identify what is causing your bot to sporadically fail sending messages, or both?
Also, is it possible for you to zip and post your project files here for me to review (minus any secrets/keys)? This way I can rely on your source files versus trying to accurately repro your setup.

I don't think we can achieve the latter without the former.  Iets focus on improving the error messaging, and hopefully that will help identify and solve what is causing the failure.
I can't send you the code, it really is no different from what is in the sample from the bot standpoint.  I did do some experiments and added additional debugging information to it though, and I can report the following" The message actually gets sent to the bot, the Exception gets thrown despite the user receiving the message.   I can't ignore this exception from my standpoint though, because there are other use cases for the when the message cannot be sent and I do need to know about it.

@master0v - I don't mean to put the onus on you, but as repro'ing your issue hasn't worked, are you able to try building and accessing the botbuilder-python library locally in your bot with the following change? If the code change is successful and returns a proper error message, then we can look at getting the library updated.
The following code is located in botbuilder-core/botbuilder/core/turn_context.py, line 156 (ref code here):
async def send_activity(
        self,
        activity_or_text: Union[Activity, str],
        speak: str = None,
        input_hint: str = None,
    ) -> ResourceResponse:
        """
        Sends a single activity or message to the user.
        :param activity_or_text:
        :return:
        """
        try:
            if isinstance(activity_or_text, str):
                activity_or_text = Activity(
                    text=activity_or_text,
                    input_hint=input_hint or InputHints.accepting_input,
                    speak=speak,
                )
            result = await self.send_activities([activity_or_text])
            return result[0] if result else None
        except BaseException as err:
            print(f"Unexpected {err=}, {type(err)=}")
            raise

Naturally, let me know if you have any questions.

Closing due to inactivity.

What "inactivity".  Is there any kind of requirement to reply during a certain time window?  I was not aware of that.  Please re-open, as the problem is still there, and don't close it until solved.
I've read up some more on the nature of concurrent.futures.CancelledError and it looks that a likely cause is task.cancel()  In fact I am able to consistently reproduce this error by replacing the await localTurnContext.send_activity call with await asyncio.sleep(60).
Is there anything in the botbuilder library that requires TeamsActivityHandler.on_message_activity to return within a certain timeframe, and cancels the co-routine if it doesn't complete within that timeframe?
If that is there case, how do I increase that timeframe or eliminate it altogether?


Provide sample data

We should provide sample data!
@LorenzLamm @rdrighetto do ya'll have some thoughts about what we can provide? Maybe some nice thylakoids and some curved surfaces?
We could provide it two ways:

napari sample data plugin
expose it on the CLI that @kevinyamauchi just added

Both would use pooch for the download (and ideally the same code path).

Yes, definitely! On Friday, @lorenzlamm uploaded some demo data to the hackathon google drive folder. I will try to get it on zenodo this week so that we can make a sample data plugin/loader.
Regarding the CLI, I was thinking a surforama --demo flag would be a nice route. As you said, it would just call the sample data loader function behind the scenes.

Yes, in our shared GDrive folder, the sample thylakoid membranes are in the ./membranorama/showcase folder.
They are from the previous thylakoid paper, where also original Membranorama was introduced.
I also realized that the Covid tomogram we were playing with is already on the data portal, so we could also us this?

We can have both!

I can put together some sample data on Zenodo this week. These two should be fine (thylakoids and covid). What files are needed exactly? Just the tomogram and the mesh in .obj format?

@rdrighetto I added the thylakoid example just now (#18). If you'd like to contribute a covid example, that would be awesome! We just need the tomogram (mrc file) and corresponding mesh (obj file) on zenodo.

If you'd like to checkout the new demo, you can run surforama --demo from main!


Bug in provided code

[Enter feedback here]
The code downloaded from Github contains a bug. Data\SampleWeatherData.cs method GetSampleData omits populating field ObservationTime. Consequently, when the user attempts to populate the table from the sample data, an Azure.RequestFailedException is thrown from line 110 of Services\TablesService.cs (the UpsertEntity statement at the end of method UpsertTableEntity) with message "The 'RowKey' parameter of value '2021-07-01' is out of range." because the RowKey expects a DateTime but is only being supplied a date (the time is null).
The fix is to insert the following before line 27 of SampleWeatherData.cs:

ObservationTime = x.ObservationTime,

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

ID: 83987349-2735-b9f5-8df2-80b65a9ff41d
Version Independent ID: 4f423457-b2ac-d90b-6104-a16e1aa92079
Content: Quickstart: Table API with .NET - Azure Cosmos DB
Content Source: articles/cosmos-db/table/create-table-dotnet.md
Service: cosmos-db
Sub-service: cosmosdb-table
GitHub Login: @DavidCBerry13
Microsoft Alias: daberry

@MichaelBrooks-Informatyx Thank you for the feedback. We are actively investigating and will get back to you soon.

@MichaelBrooks-Informatyx Thanks for your patience. I was able to reproduce the issue. I have created the Pull Request and waiting for it to be merged. It could take few days, below is the screenshot of change:

Please let us know if you have any further questions.

@MichaelBrooks-Informatyx We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Add dynamic IPolicyProvider AuthZ sample

https://github.com/aspnet/Security/issues/1689#issuecomment-372711031

In case it's useful, I have a basic IAuthorizationPolicyProvider sample here.

Being worked on by @mjrousos in PR: https://github.com/aspnet/AuthSamples/pull/31

Merged https://github.com/aspnet/AuthSamples/pull/31


Link documentation to webpack.js.org

Replaces webpack.github.io links with links to the new documentation.

Thanks


feat: Support Generic and TypeVar in doorcls.TypeHint

This turned out to be easier than I thought!  Please give this a critical look (and perhaps consider any tests that you think I've overlooked).
Turns out treating TypeVar as a special case of Union and Generic as a plain old _TypeHintSubscripted was all that was needed to make a lot of useful test cases now pass.
I think the different between bounds and constraints in TypeVars really only has meaning for static type checkers, but not so much for the sake of determining "is_subhint" compatibility.  what do you think?
This does not yet tackle covariance and contravariance

Clever and salubrious. Sadly, I made conflicts by merging prior pull requests. It's all my fault!
Oh, and as you surmise above...

I think the different between bounds and constraints in TypeVars really only has meaning for static type checkers, but not so much for the sake of determining "is_subhint" compatibility. what do you think?

Exactly. We even have a magic get_hint_pep484_typevar_bound_or_none() getter buried deep within the beartype._util subpackage that conveniently handles that on everyone's behalf by returning whichever of the bounds or constraints are defined on the passed TypeVar and, in the latter case, dynamically synthesizing and returning a new typing.Union[...] from those constraints.
I am amazed, stunned, and breathless that you independently invented the exact same wicked scheme. Like, pretty sure you're a superior doppelganger to me at this point:

Turns out treating TypeVar as a special case of Union and Generic as a plain old _TypeHintSubscripted was all that was needed to make a lot of useful test cases now pass.

:open_mouth:

conflicts fixed!  thanks!

Wunderbar. Let's merge this, people!
Also, I have a polite inquiry for you. Millions of devout Pythonistas want to know:

Where do you find the time? Seriously. I'd like some too, please.
What remains to be done? That is to say, are there any type hints or PEP standards that are supported by the @beartype decorator but not yet supported by the TypeHint API? If you can't think of anything, that means we're close. We might even be there already.

Oh – and don't worry. When someone says that, worry. I'm not trying to slyly enlist your mercenary services for further PRs. I'm just trying to grok how desynchronized these two competing APIs still are. Ideally, I'd love for TypeHint to hit feature parity with @beartype in time for our upcoming beartype 0.11.0 release.
My undying adulation for this immense investment of volunteer energy, @tlambert03.  Just kidding! I'm not creepy. Seriously, people. You gotta believe me. :zipper_mouth_face: ← creepiest emoji I seen today

Where do you find the time? Seriously.  That is to say, are there any type hints or PEP standards that are supported by the https://github.com/beartype decorator but not yet supported by the TypeHint API?

I feel confident that there are things.  Just to cover the bases, we should add at least a test or two for most of the abstract base classes in typing ... I suspect that most of them will just work by nature of being straightforward parameterized types, but just in case there's a hintsign unaccounted for in __new__ or something.
to really test it, i think it would be fun to try to "apply" it to a big codebase... just a massive smoke test to call TypeHint on every annotation found.  i'll try to do that sometime soon

my undying adulation for this immense investment of volunteer energy, @tlambert03.

it's been a real pleasure!  you are a continuous source of truly novel humor in my inbox... and I've also learned a lot from digging through the beartype codebase.  so it's a win/win

Just to cover the bases, we should add at least a test or two for most of the abstract base classes in typing...

Brilliance erupts. I should do this. Let's see if I do this...
Oh – wait. I'd been meaning to refactor our statement-level runtime type-checkers (so, beartype.abby.die_if_unbearable() and is_bearable()) to internally defer to TypeHint.die_if_unbearable() and TypeHint.is_bearable() methods. Once we do, unit tests will automatically explode for any type hints currently supported by @beartype but not by TypeHint. Phew! I can now splash around in a kayak and not do anything tomorrow. :relieved:
Houston, We Have Typing
It sounds like we're on the cusp of ready, which is both greatly relieving and yet paradoxically draining. Surmounting that final mile up to the icy-cold summit of production-ready release status is easier than bicycling with five rabid chihuahuas latched onto your leg. ...not again
If we don't hear from you this Summer, have a spectacular August and September.  We can't wait to begin stalking following you on your upcoming social portfolio of GitHub blogs, Twitter feeds, Twitch streams, and Discourse forums. So much :heart_eyes_cat: !


Ruby 3.0 support

Ruby 3.0 splits arguments into position arguments and keyword arguments. This needs to be taken into account if arguments are passed from an array.

Also tested with an example app and it works.


dex(rpc): implement candlestick streaming

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Sorry misclicked before writing the description, editing


fix manifest link in readme

The old link of Prow's manifest on this repo was 404 due to the file location changed. This PR fixed this problem.

/assign @shaowenchen

Thanks for your contributions.
/approve
/lgtm


[AzureTools][Aspire] Fail to publish the project using azd with the location "East US STG (eastusstg)" or "South Central US STG (southcentralusstg)"

Clean machine: Win11 x64 23h2 ENU
VS Version: VS 17.12.0 Preview 2.0 [35205.153.main]
Aspire Version: 8.1.1-preview.1.24405.5
azd Version: 1.9.5
Apply NuGet Feeds
REPRO STEPS:

Create an Aspire starter project
Open Command Prompt and navigate to the solution path of this project.
Run 'azd init', select 'Use code in the current directory' -> Confirm and continue initializing my app, type a new environment, press enter
Run 'azd up', and select Subscription to Azure Tools Manual Testing 1/WTE and select Location to "East US STG (eastusstg)", press enter.
Check whether succeeds to publish the project.

Expect:

Succeed to publish the project without any error.
It would be better to hide or remove unsupported locations if these locations are not supported

Actual:
Fail to publish the project with an error.

More Info:

This issue also reproduces for the location "South Central US STG (southcentralusstg)".
No location "East US STG (eastusstg)" or "South Central US STG (southcentralusstg)" in the dropdown list when publish the project using VS.

From the bug https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2183586

@vhvb1989 @ellismg can we hide these staging locations which are not valid for deployment?

Verified on the azd version 1.10.0-beta.1-pr.4097989, no "East US STG (eastusstg)" or "South Central US STG (southcentralusstg)" location shows for now.

Got it, thanks for the update.

Chatted with @ellismg and confirmed that we don't own the change/code in VS. Will create this issue on VS side.

@anvillan @abpiskunov can you create an issue on VS side for this?

@rajeshkamal5050
We already have bug 2183586 in area path DevDiv\Web Tools\Web\Aspire


tour: Cast weekday to int

Context: https://tour.golang.org/flowcontrol/10
Hi golang
I could not get this example to work until i cast'ed Weekday to int - like so:
package main

import (
	"fmt"
	"time"
)

func main() {
	fmt.Println("When's Saturday?")
	today := int(time.Now().Weekday())
	switch int(time.Saturday) {
	case today + 0:
		fmt.Println("Today.")
	case today + 1:
		fmt.Println("Tomorrow.")
	case today + 2:
		fmt.Println("In two days.")
	default:
		fmt.Printf("Saturday is weekday %v\n", int(time.Saturday))
		fmt.Printf("Today is weekday %v\n", int(time.Now().Weekday()))
		fmt.Println("Saturday is too far away.")
	}
}

Ah, not unsurprisingly I was confused. I suppose you know this, but here it goes.
I thought it was the arithmetic operator that coerced the type to int. But it turns out that it returns a new Weekday and that this is evaluated integer context in the comparison.
Which means that I could not print the week-day-number like this - because the Weekday returned from the arithmetic operation is evaluated in string context when it is printed out.
fmt.Printf("Today is weekday number %v\n", today + 0)
However this works
fmt.Printf("Today is weekday number %v\n", int(today))
This example sets today to thursday.
package main

import (
	"fmt"
	"time"
)

func main() {
	fmt.Println("When's Saturday?")
	today := time.Thursday
	switch time.Saturday {
	case today + 0:
		fmt.Println("Today.")
	case today + 1:
		fmt.Println("Tomorrow.")
	case today + 2:
		fmt.Println("In two days.")
	default:
		fmt.Println("Too far away.")
	}
	fmt.Printf("Today is %v\n", today)
	fmt.Printf("Today is weekday %v\n", today + 0)
	fmt.Printf("Today is weekday number %v\n", int(today))
	fmt.Printf("Is today Thursday? %v", (today == 4))
}


publish and release utils module

I would suggest moving this piece to indix/utils and adding it as dependency to this project.


Bring in the new SPO category from revamp branch

This is a draft to bring in the new SPO category from revamped branch
(this replaces #1061)

Thanks @rphair. Build log also available here: https://github.com/cardano-foundation/developer-portal/actions/runs/4978308037/jobs/8908435772 🤓

yes @katomm the build logs are great, but whenever possible I try to look at the locally built web site when 57 files are changed & the whole outline structure is rearranged 🤩
Note my local build still can't fetch the CIP content, upon which the rest of the local build depends.  My geolocation or IP range is apparently being blocked by the CDN which is serving that content.  Can someone familiar with the build scripts give me an idea how to pursue this?  cc @oversize @Mercurial @rdlrt @os11k

Just in case anyone has seen this before: my ISP is blocking raw.githubusercontent.com ... or maybe GitHub is blocking my ISP's addresses for that subdomain.  It affects other GitHub procedures besides the CIP content download (e.g. any time you hit the raw button & a lot of linked content for the dev portal & CIP repo).
This is my first really active year on GitHub & maybe someone with longer experience has seen this before & might know who at GitHub might help follow up about it. 😕🙏

Well might be something we could discuss elsewhere - but since you mention subdomain - if the issue is with DNS CNAME resolution, I'd suggest try using alternate DNS (cloudflare/google/opendns being most popular ones) or a proxy (or simpler shell scripts to update hosts file mapping with nslookup or IPs fetched from elsewhere)

DNS-over-TLS + cloudlfare is my preferred setup.


feat(documentation): documentation contribution add more details

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Sorry if this sounded harsh, thank you so much though!

Hey @LilyYin17,
Thanks for making these changes, they look great!
But can you apply the above changes suggested?
Have a nice day and happy coding.

Did the revisions based on your suggestions, and pull the latest code.

@LilyYin17 Thank you, for making those changes. However, you appear to have pushed changes from other commits, as opposed to rebasing this branch with main. Could you please revert the last few commits to this branch?

I'm not sure these changes are needed. The README currently links to the contributing documentation, where the landing page has the list this PR adds with the relevant links to the documentation.
Rather than linking to the GitHub issues tab in the readme, I think the current flow of encouraging contributors to start with the docs is ideal.

@nhcarrigan Makes a decent point. After the recent suggestions, this PR adds information already on the main - what I call - contributors page - where we actually want contributors to go.
As it stands, I find the amount of stuff in the README too much.
Thank you, for all your time on this, and sorry about the to-and-fro, but I think we are going to close this PR.


Add integration spec for timeline fragments [PEOPLE-17]

https://netguru.atlassian.net/browse/PEOPLE-17

specs still fail :(

Yes, unfortunately we still have problems with feature specs on circle. We tried a few things, but we did not find a solution yet. If code looks good you can sign off on it. I won't merge it yet though - untill we figure something out.


JRTB-2: add telegram bot to the project

Create stub telegram-bot for the project.
Acceptance criteria:
-Stub answer via telegram-bot added to the project

added in #16 pull-request


kernel: reduced memory allocations when mapping memory.

Codecov Report

Merging #74 (a1693b6) into main (47720f4) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main      #74   +/-   ##
=======================================
  Coverage   61.64%   61.64%
=======================================
  Files          82       82
  Lines        2988     2988
=======================================
  Hits         1842     1842
  Misses       1146     1146

Impacted Files
Coverage Δ

sources/libs/brutal/ds/map.h
100.00% <ø> (ø)

sources/libs/brutal/ds/vec.h
56.00% <ø> (ø)

sources/libs/brutal/ds/map.c
79.03% <33.33%> (ø)

sources/libs/brutal/ds/range.c
68.57% <100.00%> (ø)

Continue to review full report at Codecov.

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


av release in v2

When running av release in v2 (any NODE_ENV setting from what I can tell), the git commit doesn't seem to be including deleting files in the dist directory; should we modify the following in dev/version.js?
shell.exec('git add .');
to
shell.exec('git add -u .');

i wasn't aware of that git 🎏.  please submit a PR if it fixes your issue.

used --all as it's a bit cleaner and accomplishes the same thing.

done


switch to background kudo manager

Signed-off-by: Ken Sipe<EMAIL_ADDRESS>

to reviewers:   I'm not excited about the sleep 10 command (I'm expect to get comments on it :)... however I feel this is an ok compromise until we have a real solution which we should expect next week.
There is 1 more issue here that is worth mentioning... manager has a linux download option off our releases but does NOT have a darwin download option.   This means that local make test will fail on macs until the release process for kudo is updated (which is already on a PR for inclusion).   Monday I will also look into backporting the darwin release for the current version of kudo.

@porridge thanks for your eagle eyes! and review.
The kuttl-test.yaml is what we are switching to... however we should have tests that are running kudo and need the kudo-test.yaml.   It is likely this is temporary until we have complete moved over.   I am not against keeping both.  It depends on if we want to continue to support kudo test function (which calls kuttl... we should also have kudo test recognize kuttl-test.yaml).   Regardless, this is temp needed.
Regarding the order of things... you are correct but I would rather not reorder things.  If the manager gets the advantage of loading during the sleep... that is preferred.  If we are testing resilents.. than we should have a specific test for that.. not an accident of test design (I'm do not mean to insinuated that you mean that... just thinking out loud).   The sleep is offensive to me and likely naive.  It is temporary.   We have several feature in mind to remove this all together.   The end result I hope will be more satisfactory to both of us.   (Also... temp regarding this is this week... I expect to work on it this week)

If the manager gets the advantage of loading during the sleep

FTR the idea was:

start manager
post CRDs
sleep

so that manager still has a chance to load during the sleep...


Cannot build HIP with HIP-clang

step1:
git clone https://github.com/llvm/llvm-project.git
mkdir -p build && cd build
cmake -DCMAKE_INSTALL_PREFIX=/opt/rocm/llvm -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=1 -DLLVM_TARGETS_TO_BUILD="AMDGPU;X86" -DLLVM_EXTERNAL_LLD_SOURCE_DIR=../lld -DLLVM_EXTERNAL_CLANG_SOURCE_DIR=../clang ../llvm
make -j
sudo make install
step2:
export PATH=/opt/rocm/llvm/bin:$PATH
git clone -b amd-stg-open https://github.com/RadeonOpenCompute/ROCm-Device-Libs.git
cd ROCm-Device-Libs
mkdir -p build && cd build
CC=clang CXX=clang++ cmake -DLLVM_DIR=/opt/rocm/llvm -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_WERROR=1 -DLLVM_ENABLE_ASSERTIONS=1 ..
make -j
sudo make install
step3:
git clone -b master https://github.com/ROCm-Developer-Tools/HIP.git
cd HIP
mkdir -p build && cd build
cmake -DCMAKE_INSTALL_PREFIX=/opt/rocm/hip -DHIP_PLATFORM=clang -DCMAKE_BUILD_TYPE=Debug ..
report cmake error:
CMake Error at tests/hit/HIT.cmake:375 (elseif):
given arguments:
"NOT" "_exclude_compiler" "AND" "IN_LIST" "_exclude_runtime"

Unknown arguments specified
Call Stack (most recent call first):
tests/hit/HIT.cmake:430 (hit_add_files)
CMakeLists.txt:637 (hit_add_directory_recursive)
Can anyone figure out what't reason?  It's hard to build HIP now...

Here is my building scripts for ROCm-3.9, please refer.
https://github.com/xuhuisheng/rocm-build

Here is my building scripts for ROCm-3.9, please refer.
https://github.com/xuhuisheng/rocm-build
There are atleast 9 steps to build hip, you have just 3 steps.

my env is new ubuntu 20.04 as well, add video and rend groups, then follow your 11~19 steps, trying rocminfo gets below:error:
sudo ./rocminfo
ROCk module is loaded
Unable to open /dev/kfd read-write: Bad address
zecai is member of video group
hsa api call failure at: /home/zecai/riyue/rocm/rocminfo/rocminfo.cc:1142
Call returned HSA_STATUS_ERROR_OUT_OF_RESOURCES: The runtime failed to allocate the necessary resources. This error may also occur when the core runtime library needs to spawn threads or create internal OS-specific events.

The reason to fail to open /dev/kfd is "amdgpu: Failure to set tba address. error -1." from dmesg. Anyone can figure out why?

@Chunming-Zhou, sorry for the lack of response. Please try latest ROCm 6.0.2 (HIP 6.0.32831) to see if your issue still exists? If resolved, please close the ticket. Thanks.


after install with GO, it just doesn't appear to be installed?

hey, after I installed it via go, everything looked smooth and all required packages were installed without problems... but if I type shodanx on cli, it doesn't appear to be installed.

Hi @KB-Hicham
Please elaborate your issue by what command you used to install
Kind regards,
@RevoltSecurities

shodanx -h


proto: represents validators with key_hashes rather than keys

Depends

[x] #277
[x] #283

Problem
Validators on Deku are currently represented as public keys rather than as hashes of their public keys. As @d4hines noted in the comments, it's true that validators must inevitably reveal their public keys to sign blocks. However, according to @EduardoRFS, it's generally desirable among users for their public keys to be obscured wherever possible even if at some point they are unavoidably revealed.
Solution
This PR

changes any occurrences of Wallet.t (represented by public keys) that can be replaced by Address.t (represented by key_hashes) to avoid revealing public keys unnecessarily if/when possible
changes the consensus contract to

represent validators as LIGO key_hashes instead of keys
add an additional check that validator keys in a root_hash_action correspond to validator key_hashes in storage
account for the case where some keys are not present by representing validator keys as option key list

It's worth noting that a certain change made in the consensus contract swaps the position of two match cases due to a bug in the LIGO compiler that stopped the contract from compiling (the compiler would mismatch types and get confused); swapping the position of the match cases was proposed as a temporary work around.

Update; state.re is now creating an Address.t from the Address.key field for the purpose of passing public keys to those two functions above; this seems to work, and there are no more type errors reported by the compiler. However, one unit test, Consensus > Hash Validators is failing. This is somewhat to be expected, the test body was updated to be consistent with the new representation but the assertion was not. Can someone confirm for me that the "received" value here aligns with what we now expect the test to result in?

@rosalogia you need to recalculate that, my tip, look on how the hashing works on the consensus.mligo, this PR will require you to fix that also.
You can play with ligo at > https://ligolang.org/
Also if you want please enhance the test description.

Looking at this bit of consensus.mligo, it seems like we just hash the packed list of public keys right now. I wonder what makes most sense for us if we stop representing validators with public keys and represent them with public key hashes instead:

Packing a list of key hashes and hashing that
Simply concatenating the list of key_hashes after stringifying them
Something else?

Currently in tezos_interop.re we're doing option 1. It's not really obvious from the contract (since it needs to be changed too) which way of doing things is "correct", nor can I tell if this is something we have a choice over, so I'm deferring to you guys to determine which way of hashing validators (once they're represented by key_hashes) makes the most sense.
I'd rather not hastily change the expected value of the test, lest I render the test totally useless in preventing issues going forward :smile: thanks for your time and help in advance!

Packing a list of key hashes and hashing that

This is actually the only feasible options as michelson doesn't expose ways to operate on key_hash.

@rosalogia is this ready for review? I remember you mentioned you had some questions about ligo - feel free to ask them here or on slack and I'll do my best to answer

Rebase is finished :smile: all yours, @EduardoRFS. Hopefully I didn't get confused and mess anything up, but if so, point it out and I'll fix it quickly.

Actually, in my testing, the chain isn't working - it won't get past block height 1 :(

I retested systematically with Ambika and we could not reproduce the error i was seeing. It was probably a mistake on my end. This PR lgtm!


Add button to delete undeleted repositories from failed migrations

This PR adds a button to delete failed repositories if there has been a
failure during migration and for whatever reason the repository doesn't
get deleted automatically.
Fix #16154
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>Please check the following:

I know we're in freeze but I think this is small but useful ui improvement and could be added.

Codecov Report

Merging #16197 (f64cd02) into main (59f2558) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main   #16197      +/-   ##
==========================================
+ Coverage   44.54%   44.56%   +0.01%
==========================================
  Files         701      701
  Lines       83242    83207      -35
==========================================
- Hits        37084    37079       -5
+ Misses      40140    40116      -24
+ Partials     6018     6012       -6

Impacted Files
Coverage Δ

routers/web/repo/view.go
41.23% <0.00%> (-0.08%)
:arrow_down:

services/mailer/mail_comment.go
77.77% <0.00%> (-7.41%)
:arrow_down:

modules/git/repo_base_nogogit.go
82.85% <0.00%> (-2.86%)
:arrow_down:

modules/git/utils.go
68.05% <0.00%> (-2.78%)
:arrow_down:

modules/process/manager.go
72.83% <0.00%> (-2.47%)
:arrow_down:

modules/queue/queue_channel.go
95.00% <0.00%> (-1.67%)
:arrow_down:

models/user.go
52.32% <0.00%> (-1.07%)
:arrow_down:

services/pull/pull.go
43.37% <0.00%> (ø)

models/error.go
39.53% <0.00%> (+0.31%)
:arrow_up:

modules/queue/workerpool.go
54.58% <0.00%> (+0.76%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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

Done

POST /test/brokenmig 405 Method Not Allowed <- create @zeripath

ah I knew there was a reason why I didn't break out the modal.

Sorry @noerw we can't do that.

we can, just make sure to pass the post link as argument to sub template ?

we can, just make sure to pass the post link as argument to sub template ?

Nope. that will break the template.


Added my name to team members

Thank you @msarit I can't add it to project. I will need to rewatch yesterday's section and see how it's done.


yarn command fails during install of @project-backend/db packages and generating prisma schema

After following the instructions in the readme, I am having an issue with installing the backend dependencies. Please see the output from terminal below.
$ yarn
yarn install v1.22.5
warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized loc
k files. To clear this warning, remove package-lock.json.
[1/4] Resolving packages...
[2/4] Fetching packages...
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.11.3 || ^0.12.0 || ^0.13.0 || ^14.0.0".
warning "apollo-server-express ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^14.0.0 || ^15.0.0".
warning "apollo-server-express ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^15.0.0".
warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.10.0 || ^0.11.0 || ^0.12.0 || ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express > apollo-server-core ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>- 14".
warning "apollo-server-express > graphql-tools ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.12.3 || ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "apollo-server-express > graphql-tools ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^0.12.0 || ^0.13.0 || ^14.0.0 || ^15.0.0".
warning "workspace-aggregator-7d3ab4bd-a8ea-40e1-aad1-ca4a9144a6ec > @project-backend/graphql ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "workspace-aggregator-7d3ab4bd-a8ea-40e1-aad1-ca4a9144a6ec > @project-backend/graphql > graphql-tools > @graphql-tools/links ><EMAIL_ADDRESS>has unmet peer dependency "subscriptions-transport-ws
@^0.9.0".
[4/4] Building fresh packages...
[9/9] ⠈ @project-backend/db
[-/9] ⠁ waiting...
[-/9] ⠁ waiting...
[-/9] ⠁ waiting...
error C:\Users\Dev\Desktop\fullstack-starterkit-master\backend\node_modules\@project-backend\db: Command failed.
Exit code: 1
Command: yarn generate
Arguments:
Directory: C:\Users\Dev\Desktop\fullstack-starterkit-master\backend\node_modules\@project-backend\db
Output:
yarn run v1.22.5
$ yarn prisma generate --schema=./prisma/schema.prisma
$<EMAIL_ADDRESS>generate --schema=./prisma/schema.prisma
internal/modules/cjs/loader.js:800
    throw err;
    ^

Error: Cannot find module<EMAIL_ADDRESS>    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:797:15)
    at Function.Module._load (internal/modules/cjs/loader.js:690:27)
    at Function.Module.runMain (internal/modules/cjs/loader.js:1047:10)
    at internal/main/run_main_module.js:17:11 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}
error Command failed with exit code 1.

thanks for sharing, jeez windows always comes up with an new issue with yarn workspaces. I will try to fix it

hey @dustingauthier I was able to reproduce this on my windows machine and hence fixed it in PR #22 let me know if it works for you now, thanks

Works great. Thank you!


Added workload response handler tests

What type of PR is this?

What this PR does / why we need it:

Abstract xds clients to simplify ut
Added handle workload tests

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report
Attention: Patch coverage is 78.31325% with 18 lines in your changes missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

unittests
36.98% <78.31%> (?)

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

Files
Coverage Δ

pkg/controller/xdstest/fake_xdsserver.go
82.50% <100.00%> (ø)

pkg/controller/workload/workload_processor.go
42.57% <0.00%> (ø)

pkg/nets/nets.go
26.08% <0.00%> (ø)

pkg/controller/xdstest/fake_xdsclient.go
78.57% <78.57%> (ø)

pkg/controller/workload/bpfcache/fake_map.go
82.97% <82.97%> (ø)

/lgtm
/approve


Change GUI file loading

GUI will need to be able to read in the format of the output file from filename checking instead of .csv table.

File selection GUI can now read in .yaml files.  Have not removed the .csv code just yet though.


When will you release the code?

Excellent work!
Could you please release the code  asap!
Thanks!

Hi, it should be available.


GMS Test Page - "Reload of xx" Uses Legacy Value

Change to Governed Metrics Application instead of Metrics Libarary

This is resovled in 2.0.5


⚠ DDoS-Relay US-E-nyc1-01 has degraded performance

In 27cd7f4, DDoS-Relay US-E-nyc1-01 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 24 ms

Resolved: DDoS-Relay US-E-nyc1-01 performance has improved in 779a399 after 57 minutes.


Fix inner sum in BGV

Refactor of the inner sum API in lattigo. Fixes #510.
Issue
bgv.Evaluator.InnerSum was not working properly when n*batchSize = N (where N is the number of slots) as InnerSum was acting separately on the "rows" of the underlying plaintext.
It has been decided to refactor the whole InnerSum API in lattigo to be more coherent.
Changes

Refactoring of the InnerSum methods:

rlwe.Evaluator.InnerSum has been replaced by rlwe.Evaluator.PartialTrace to reflect that at the level of rlwe the notion of inner sum does not exist.
Introduction of the bgv.Evaluator.InnerSum and ckks.Evaluator.InnerSum methods, which have the same behaviour as the old InnerSum method for parameters n and batchSize s.t. 0 < n*batchSize <= ctIn.Slots() divides the number of slots. Parameters not satisfying these conditions are rejected.
Introduction of the bgv.Evaluator.RotateAndAdd and ckks.Evaluator.RotateAndAdd methods, which have the same behaviour as the old InnerSum method for all parameters.

New tests have been added.

In summary, #510 is fixed, legacy code is still supported except when n*batchSize does not divide the number of slots (or if n*batchSize is not in the interval [1, ctIn.Slots()]). In that case, RotateAndAdd must be used.

I would rename rlwe.Evaluator.PartialTrace to rlwe.Evaluator.PartialTracesSum or something similar because here we have an accumulator accQP that sums HW(n) partial traces. It's only when n is a power of two (where HW(n) = 1) and offset divides n that we have a true partial Trace (in the sens that full trace is from 0 to LogN and partial means from a to b where 0<= a < b <= logN). In other words, .Trace is a special case of PartialTracesSum.

Yes I agree, it is a partial trace in the sense that it is a sum with a subset of the automorphisms, but you're right that Trace can also be partial in that sense. I've changed to PartialTracesSum for now, we can rename it a later time if someone finds a better suited name.


Allow for default_locale to be nil

Feature request:

If default_locale is false, (or we didnt found a default_locale in `available_locales), then:

if object has already some translations, just set as default the first one found
if object has no translations (f.e. when its a new_record?), it should show only available_locales list (we need to explicitly add first translation, to make the translatable_inputs) visible

Changes would be need to be done propably in https://github.com/fabn/activeadmin-globalize/blob/develop/lib/active_admin/globalize/form_builder_extension.rb .
I might to do a PR if interested.

Might be dependant on https://github.com/globalize/globalize/pull/328 , checking it out now.

It's working good, but when trying to save the object, globalize raises validation error for the translation object of default locale (which shouldnt be created at all). More info in globalize/globalize#328 - we'll need to wait for the fix over there.


pdf download fails in cost analysis page

Hello one of our customer is experiencing issue while downloading pdf in cost analysis page.
Cost Explorer > Cost Analysis
When click 'PDF' button. No response on pdf rendering page as below

In developer tool error message appears when I click 'pdf' button

https://github.com/cloudforet-io/console/pull/487


mount: /mnt/xyz: unknown filesystem type 'juicefs'

I added the line to my fstab
mysql://juicefs:juicefs@(<IP_ADDRESS>:3306)/mnt_xyz  /mnt/xyz  juicefs  _netdev  0 0

When I try to mount the folder by following command
mount /mnt/xyz

The error appears
mount: /mnt/xyz: unknown filesystem type 'juicefs'.
       dmesg(1) may have more information after failed mount system call.

I tried both install methods, by a script and manually.
Environment:

juicefs version 1.1.1+2023-11-28.437f4e6
Debian GNU/Linux 12 (bookworm)
Linux pve-ge-01 6.5.11-4-pve #1 SMP PREEMPT_DYNAMIC PMX 6.5.11-4 (2023-11-20T10:19Z) x86_64 GNU/Linux
ZFS
mariadb 10.11.4
network: linux bridge

A symlink needs to be created from /sbin/mount.juicefs to the JuiceFS executable, e.g.
ln -s /usr/local/bin/juicefs /sbin/mount.juicefs


Add support for reasonML

Fixes #1854
Changes proposed:

[X] Add
[ ] Delete
[X] Fix
[ ] Prepare

Added the reason interface icon for .rei extension and fixed the extension for .re.

https://github.com/vscode-icons/vscode-icons/issues/1854#issuecomment-462169121

@JimiC, I'm not sure if this PR is completely unusable taking into account the support for Reason Interfaces it provides.
It seems that the extensions are only providing a common language id for these two kind of files thus making both having the same icon.
I think that having a separate icon would make a lot of sense for this case. Instead of changing the  Reason icon with the REI extension in it I think it would be better to slightly change the background color. After implementing this, we could maybe contact the Reason extension authors so they could provide support for a second language id.

I'm going to reopen this until we arrive to a final conclusion. :)

My opinion is to contact the Reason extension author first, await the decision and then act accordingly. I don't want to have to deal with two PRs for one thing.

Btw this should be tracked on the issue and not on the PR. It's best to reopen #1854 and close this one.

Well, we have some resolution at least. 😉 I'll put all this in the issue.


[WFLY-12402] Clean up test deployment use of management classes and t…

…hereby increase number of tests that can run in the ts.layers profile with Galleon slimmed servers

Eliminate no-longer needed dependencies on the org.jboss.as.cli module, which typically is not provisioned in ts.layers provisioning.
General cleanup of tests that used this or ones near those (primarily jca/ra/ds related tests) to better clarify between code that needs 'management' functionality and code that does not.
a) Deployments usually do not, particularly those in @RunAsClient tests. So remove the management code from the deployments as much as possible.
b) However, if the test class goes in the deployment and a @ServerSetup is on that class, the ServerSetupTask impl class and its superclasses must be present, even though they are not executed on the server side. Server side code processing the deployment will try and examine them.  Other classes used in the setup tasks do not need to be in the deployment though, as the setup task executes on the test driver side.
c) A number of tests were subclassing AbstractMgmtTestBase directly or transitively but were not using any of its functionality, instead relying on a ServerSetupTask for such things. In such cases the test class was changed to no longer subclass unnecessarily.
Following from this a large number of tests that previously could not run in the ts.layers profile now can, so they have been enabled.

https://issues.redhat.com/browse/WFLY-12402

I put the 'hold' label on this because it's currently based on the tmp branch I used for batch PR test #13292. Once I merge those I'll rebase on master.


Implementation of the NSPointerArray class

This is a proposal for the implementation of the NSPointerArray class.

Hi @bSr43, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

:clock1:

@bSr43 This is now merged. Thanks for your contribution!


After starting the server, it dies and does not respond

Start server.. generate new map..
Stop server.
Repeat start server... waiting starting... R.I.P.
crash-2024-03-04_10.55.08-server.txt
latest.log
debug.log

Problem in redstone
i changed in server.properties this value
max-chained-neighbor-updates=0
and my console:
and server R.I.P.

and Threading Detector
latest.log

try with new version (just redownload 1.19.2-1)

try with new version (just redownload 1.19.2-1)

ok

If an error appears, I will open the topic again.


Parse TRAVIS_PYTHON_VERSION to get python version

Using sys.version_info is a good fallback, but it doesn't allow TRAVIS_PYTHON_VERSION to be set explicitly in scenarios where Travis doesn't deal with Python versions, such as on multi-os builds under osx instead of linux.
Recent comments on #4 suggest support for multi-os builds, but that support is very limited currently. This should allow manually set TRAVIS_PYTHON_VERSION to behave identically to using the python configuration for versions.
References #4

@jayvdb: Thanks for the review. I really appreciate it!

@jayvdb: If your up for it, I've got another commit, and could use another round of review.

Looks good other than the version parsing, which shouldnt be a problem any time soon ;-)


Fixup RPM build

Fixes #1063.
Signed-off-by: Joe Grund<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1064   +/-   ##
=======================================
  Coverage   95.42%   95.42%
=======================================
  Files           2        2
  Lines         153      153
=======================================
  Hits          146      146
  Misses          7        7

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 02f02aa...7146d55. Read the comment docs.

I feel like tarring it all up as an "extra" tarball feels kludgy, and we should be putting it in the source tarball, but it's better than it was I guess.

The reason for the extra tarball is we don't need things like .service files in docker containers, ideally we should be able to put the python package directly into the container without the extra cruft (admittedly we're not there yet).
If you have a solution that allows for this approach, I'd be willing to implement it.


Spec does not define expected status code when required fields are absent

While I'm not sure we need to say when to use all possible http error codes, I think this is a good one to call out and I'm thinking that a "400 Bad Request" is appropriate.

@duglin and that is how I've implemented it in the broker I am currently working on (because that seems the most common sense response), but like you, I do believe the spec should be explicit in this regard.

PR:  https://github.com/openservicebrokerapi/servicebroker/pull/275


japan contents commit

adds the following contents translated Japanese

Getting Started with Data Engineering and ML using Snowpark for Python
Getting Started with Snowpark in Snowflake Python Worksheets
Tasty Bytes - Snowpark 101 for Data Science


Docs should be versioned

Currently website seems to not follow the recommended docusaurus versioning strategy https://docusaurus.io/docs/versioning
This makes it difficult to stage changes before release and to maintain diff versions

Note #38 adds versioning doc support HOWEVER we need to remove the hardcoded 1.0 doc version in the docusaurus.config.js file prior to 1.1 release.


Streaming service request wakanim.tv

#119
There you go :)

@ITSYV Thank you. I would test it on weekends

@dm3ch can you verify this and close this issue if it works? :)

TY. I haven't yet found enough time to build the project myself to test. But I would try to find time for that.


Top-level await is not supported

In jiti v1.12.9 (via nuxt 2.15.8), top-level await doesn't seem to be supported, although it is in Node.js v14+.
See also https://v8.dev/features/top-level-await
When starting my Nuxt app (with top-level await in a server middleware ESM module), I get the following error:
 ERROR  ServerMiddleware Error: await is only valid in async function                                                                                21:52:20

  const schemas = await Promise.all([
  ^^^^^

  SyntaxError: await is only valid in async function
  at new Script (vm.js:102:7)
  at createScript (vm.js:262:10)
  at Object.runInThisContext (vm.js:310:10)
  at g (node_modules/jiti/dist/jiti.js:1:54973)
  at tests/fixture-valid.js:3:44
  at g (node_modules/jiti/dist/jiti.js:1:55111)
  at ui/api/routes/fixtures/from-editor.js:12:21
  at g (node_modules/jiti/dist/jiti.js:1:55111)
  at ui/api/routes.js:3:19
  at g (node_modules/jiti/dist/jiti.js:1:55111)

Jiti works with CommonJS api (while top level await is an esm-only feature). Support is possible but might be tricky. In the meantime, use a main wrapper:
async function main() {
  // Move async logic here
}

main().catch(console.error)

As the world is (very slowly) moving more into esm, are there any plans on having jiti work with top-level await?
SST for example uses top-level await to load environment variables into the server. At least currently, trying to inject runtime config in Nuxt 3 using top-level await does not work and throws SyntaxError: await is only valid in async functions and the top level bodies of modules.

+1 for top-level await support. Tailwind is now using Jiti to load tailwind.config.mjs, but unfortunately, the choice to use Jiti also means that tailwind user can't leverage import.meta and can't use top-level await in tailwind config files.

We are on it! This feature require a new API from jiti jiti.import(id) (which returns a promise unlike jiti(id) that returns a sync result and is blocker for top level await)

Very glad to hear it! Thanks a lot for the follow-up 🤜💥🤛

Hi. Thanks for looking into this. Is there any update on progress?

Looking to work on this; @pi0 it seems to me that the main problem to solve is when a CommonJS module imports an ESM module with top-level await in it, because at that point the synchronous require is dependant on an async workload.
Reading through the code I see that jiti transpiles everything to CJS, why is that?
Wouldn't it be possible for ESM modules to stay ESM, unless required by a CJS module?

Top level support is now available in v2 (currently beta)


LocationWhenInUsePermission is incompatible

[!] CocoaPods could not find compatible versions for pod "PermissionsKit/LocationWhenInUsePermission":   In Podfile:     PermissionsKit/LocationWhenInUsePermission (fromhttps://github.com/sparrowcode/PermissionsKit`)
None of your spec sources contain a spec satisfying the dependency: PermissionsKit/LocationWhenInUsePermission (from https://github.com/sparrowcode/PermissionsKit`).

Related to #332. Use LocationPermission  after 10x version.


Conhost.exe CRASH when changing zoom with mouse cursor

Using Windows Store version 0.5.2762.0
Zipped DMP file attached.
conhost.exe.21732.dmp.zip

This may be a /dupe of #1856.


The JumpTo category name are appear in lowercase in the left sidebar of the example child page.

Most appropriate sections of the p5.js website?
Examples
What is your operating system?
None
Web browser and version
No response
Actual Behavior
The JumpTo category name are appear in lowercase in the left sidebar of the example child page.
(It will not be translated in any language other than English.)
Expected Behavior
Display the same as it appears on https://p5js.org/examples
Steps to reproduce
open https://p5js.org/examples/shapes-and-color-shape-primitives/
see JumpTo link's category in left side bar.
compare with https://p5js.org/examples/
Would you like to work on the issue?
I would create a pull request for suggestions.

Fixed with #586


Handle multiple telluric stars in mmirs_pipeline_taskfile

The selection of telluric stars is done in a manner of finding all telluric stars with the same configuration.  For MOS that should be fine; however, for longslit, multiple telluric stars with the same slit-filter-grism configuration is possible for a given night or for programs that receive multiple targets in the same night.
There needs to be at least two ways to filter telluric stars:
First, using the Prop ID.  This will help distinguish telluric star taken before science for a different program than those taken specifically for said program.  However, the one downside is that if telluric data is taken and the queue observer forgets to change the Prop ID, it would not work (hypothetically).
Second, generally telluric data is taken either before science and/or after.  One can determine this by grouping telluric data together and seeing if the last or first exposure of has FITS IMAGETYPE value of 'object'.  This would handle skipped frames such as alignment data, and calibration data (comps and flats).

A handle_tellurics() function has been developed (8477b8d) and is called by get_tellurics (1ac9372).
Several minor improvements were made: bf38ff2, e850f11, 64c65fe, 08d5e29.
Also, we have implemented a check to find nearest telluric data (before and after) and whether data is bracketed: 32aed86.
handle_tellurics() will provide an updated list of tellurics to get_tellurics(): b7968c0, a2d2838
A few additional minor commits: 131b44d (logging) 593c04a (array definitions), 10ef1e1 (bug fix)
Note: I've tested handle_telluric() for one rawdir folder with two science targets with the same configuration and bracketed telluric data and it appears to work.