Enable Powershell to support Channel Setting Permissions. -Infrastructure as Code Microsoft.  Come on Now.

really need the ability to set "only owners can post messages" in the channel settings via PowerShell.

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

ID: 76309c0b-2a9c-36a8-d7c7-f7d6f311d39b
Version Independent ID: bc69c0e3-0fa7-c86e-5e8c-067c8811e9bf
Content: Create an org-wide team in Microsoft Teams - Microsoft Teams
Content Source: Teams/create-an-org-wide-team.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@valmark1 Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Modifying the Github actions file

Hey @schristoff I have modified the action file and here the Golang version is used from env avoid installing it  everytime . Just a small question that should I seperate the file one on push and one on Pull Request?

Hey,
Yeah I think it makes sense to have one that runs on pull requests and one that runs on merge into main (push into main)
The push into main one would have the publish built into it
    - name: Publish
      if: success() && github.event_name != 'PullRequest'
      env:
        GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
      run: mage Publish

Hey, Yeah I think it makes sense to have one that runs on pull requests and one that runs on merge into main (push into main) The push into main one would have the publish built into it
    - name: Publish
      if: success() && github.event_name != 'PullRequest'
      env:
        GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
      run: mage Publish

Okay I will make the changes requried

Hey @schristoff I have seperated 2 files one for push and another on pull request also changed the file names .

This was implemented in https://github.com/getporter/skeletor/pull/51


Delete does not work

Using the api:
{{url}}/masterlib/delete/fromapp with the body of:
{
"appname": "Nøgletal Dev"
}
gives the response:
"delete complete!"
But the master data items are still present in the application.
Running Qlik Sense 3.2 SR1

Just noticed that the Master Data Items on the application have been distributed with an older version of GMS - but I guess it should not matter, the endpoint is supposed to delete all MDI's no matter origin?

@dauledk, I found some wonkiness as well but what I did is I restarted all Qlik Sense services, then restarted the powertools service.  I think the issue is notification service binding which kind of sucks because notification bindings are ephemeral.  They have to be reset everytime services get restarted.

@goldbergjeffrey I just tried restarting all services - but to no help.
This is the log output:

Status output:
{"level":"info","msg":"POST delete/fromapp for Nøgletal Dev","meta":{"module":"routes"}}
{"level":"info","msg":"deleteFromApp::Calling deleteFromApp","meta":{"module":"doWork"}}
{"level":"info","msg":"getDocId::running getdoc.getDocId","meta":{"module":"getdocid"}}
{"level":"debug","msg":"getDocId::qrsInteract::/app?filter=name eq 'Nøgletal Dev'","meta":{"module":"getdocid"}}
{"level":"debug","msg":"getDocId::qrsInteract::Nøgletal Dev id:f52b1ce9-9116-4c23-a8b1-948ecf8fb692","meta":{"module":"getdocid"}}
{"level":"info","msg":"deleteAllMasterItems::Deleting all MasterItem dimensions and measures from app: f52b1ce9-9116-4c23-a8b1-948ecf8fb692","meta":{"module":"deletemetrics"}}
{"level":"info","msg":"Destroyed 0 measures from Nøgletal Dev","meta":{"module":"deletemetrics","app":"Nøgletal Dev"}}
{"level":"info","msg":"Waiting for notification service to inform that the repository is clean","meta":{"module":"deletemetrics","app":"Nøgletal Dev"}}
{"level":"info","msg":"Destroyed 0 measures from Nøgletal Dev","meta":{"module":"deletemetrics","app":"Nøgletal Dev"}}
{"level":"info","msg":"Waiting for notification service to inform that the repository is clean","meta":{"module":"deletemetrics","app":"Nøgletal Dev"}}
{"level":"info","msg":"deleteAllMasterItems::saveObjects::Master Items removed.","meta":{"module":"deletemetrics"}}
{"level":"info","msg":"deleteAllMasterItems::Engine connection terminated","meta":{"module":"deletemetrics"}}
{"level":"info","msg":"deleteFromApp success","meta":{"module":"doWork"}}
{"level":"info","msg":"POST delete/all success::delete complete!","meta":{"module":"routes"}}

Do you have any idea of what I can try?

try restarting all Qlik Sense services, then restart the powertools service once all Qlik Sense services are back online.  This sometimes happens when the notification entry for GMS is not reset after a Qlik Sense services restart.  Unfortunately, the notification service guids are ephemeral.  They aren't stored in the repo so they need to be regenerated and synced everytime a restart happens.

@dauledk did restarting services resolve the delete issue?  I'd like to close this if I can.

@goldbergjeffrey no, unfortunately not.

GMS 2.0.5?  I ran through it just this week and it deleted on command.

Can you provide screenshots showing the delete and the result of the application to be deleted?

I've worked on converting from qsocks to Enigma.  Deleting is definitely working in #129.  This will be part of the next release targeted for end of July.


Remove facets control on mobile results view

The mobile results view should hide the search facets and show the list of facilities. It's not too hard to make it work this way by removing the facets from the DOM, the issue is that the desktop view needs to keep the facets around on the results page.
Removing facets from the DOM also means it's impossible to go back to the questionnaire (currently we solve this by completely reloading the page to start the questionnaire over).
How to split logic for mobile v desktop?
When rendering the results page, jQuery calls show() on the facets which overrides media queries setting mobile facets to display:none. Perhaps after show() we can attach a class to the facets that has a media query that hides mobile? Not sure if it's an order of operations issue or what. Any thoughts welcome.

There's a nice pull-request on CodeforBoston/finda that does this nicely. https://github.com/codeforboston/finda/pull/149

I actually tried that back in June but it had a couple of quirks IIRC. Worth another shot though -might just need light refining.
Anyone interested in merging in the PR code and trying it out?

Hey @ashleybot, I tried a merge myself and was getting a functional site, but with the survey contained entirely within the sidebar... looks like how the sidebar is being positioned in the mobile-layout PR is conflicting with how the survey is implemented. Let me know if I can do anything to help!


Proposed change to JCDP standard: Expanding the JCDP coverage

Introduced during WGJCDP meeting on April 2023.
Expanding the JCDP coverage; west Atlantic, Arctic, Mediterranean…further?

Coverage of the existing spatial remit of JCDP is patch and requires work to address, it is recommended that the WG focuses on improving engagement within the current spatial remit before expanding to other areas.

POST MEETING NOTE – feedback from JCDP discussion at the ECS conference highlighted potential to include the Mediterranean as a priority, given multiple NA Atlantic countries have coasts in the Med and some of the drivers e.g. MSFD reporting, would benefit from having a wider geographic scope.

Decision during July interim WG meeting
"Agreed: let it develop organically – don’t restrict, but don’t resource widening the net. Countries with Med and NE Atlantic coasts may wish to upload all data. Potential data management solution for overseas territories etc. "


[Feat] - Add password to mission

Why this feature ?
The objective is to restrict access to a mission for a user.
When I see the MCD a user is assign to many missions. We need o restrict the access to a mission throw this relation.
If a user is not assign to the mission the API return a 403 Forbidden error
This restriction will be apply to sub-ressources (hosts, vulns...)

You right, Idea was to generated public and private key to encrypt all data in dabatase ( all key different by mission )
and use the key to encrypt and display information about the mission, but its maybe  too heavy
the goal was to put an extra layer of protection as this is sensitive data that affects businesses


fix docker events message

Description
Fixing the problem of hanging process, that caused by waiting for the error message
Checklist

[ ] Unit/Integration test added (if applicable)
[ ] Documentation added/updated (wiki or md)
[x] Code style is consistent with the rest of the project

@stsvilik I think we need to update the packages. I see an opened PR for this, due to old packages CI failed

@stsvilik I think we need to update the packages. I see an opened PR for this, due to old packages CI failed

Next PR will address failing tests and dependencies upgrades.

@stsvilik if you want I can bump all wdio dependencies. In our automation project, we'd updated to 6.10.6 and all works good

@stsvilik I think we need to update the packages. I see an opened PR for this, due to old packages CI failed

Next PR will address failing tests and dependencies upgrades.

@stsvilik if you want I can bump all wdio dependencies. In our automation project, we'd updated to 6.10.6 and all works good

In this case I would like to keep PR's separate.


feat: allow setting podAnnotations

Description of the change
We use them in combinations with KIAM to allow the pod to assume a IAM role
Existing or Associated Issue(s)

Additional Information

Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[ ] JSON Schema generated.
[ ] List tests pass for Chart using the Chart Testing tool and the ct lint command.

@alexef unfortunately I can't merge this, unless your commits are signed. Can you please look into that?

https://docs.github.com/articles/about-gpg/


Multi Folder Workspace Config Issue

Most of the time my VSCode workspaces contain 2-6 folders and luckily only 1 has been a front-end project. So to deal with Windi I have just made the first folder in my workspace the one using Windi, and all is well. As it looks to only support the first workspace folder.
https://github.com/windicss/windicss-intellisense/blob/main/src/extension.ts#L15
However when I go to add two projects using Windi to a workspace, its a mess as its picking up the first projects config, and doesn't pick up any custom properties or info from its own config. I am guessing in order to make this work we would need some information about the currently opened file, and trace it back to a root folder in the workspace, then search for the windi config from there? Or Maybe VSCode already provides information on the root folder for the current working file?
I am guessing there is no workaround for this currently?

/cc @IgnisDa

I wonder how to approach the case when two or more windiconfigs are present in the same workspace. Do we use the first one or do we combine them both? For example, if we have a custom color defined in windi config of project 1, do we want it to be available in windi project 2 too?
@aphex and @alexanderniebuhr what do you think?

I would say not necessarily. Monorepos are mostly just separated projects in subfolders.
But getting the nearest windi.config seems to be not good for performance.
/cc @antfu tagging you to see if you have any idea how multiple windi.config files in one workspace could be handled.

I think get the nearest config sound like the correct way to do that (just like tsconfig.json). We could cache those configs so the performance shouldn't be affected too much.

Our use case is not a monorepo but a vscode workspace that is pulling together multiple repos. Difference is there is no top level folder open in VSCode.
For example we will have a project-back-end folder and a project-front-end. These are both separate repos and sibling folders. We then have a folder of workspace files that orchestrate various projects together. In this simple case its just 2. However as they get more complex and we collect more vue applications in a single "project" we would like to use windi across all of them.
So from our standpoint, to answer your question, it would be best if simply the nearest windi config was loaded. Merging the files is not on our radar, and if we wanted to share assets we would like just create a package that both projects could install and reference.

Wouldn't presets be the way to go here?

Any update for this issue?


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 0191a23:
:clock10: Full build: the average full build time is 7 min(s) 53 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 0191a23:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Login window double login fix.

This change fixes a bug with cookie validation that was
causing the first login attempt to fail. The login window will
not try to verify the cookie before successful login. This change also
adds login feedback.

Coverage remained the same at 55.348% when pulling f3abf3887c2b4a710c717d3a2d97d70b55fdbe52 on gopherstein:master into e1ecbf1975e360412715ece94787b2640b0d1020 on supergiant:master.


Update Mutations is not working in Serverless RDS GraphQL

Describe the bug
update mutations is not working with serverless RDS mysql, react-native and amplify.
CreateMutation is working fine.
UpdateCustomerDetails = async (sub) => {
    let UserDetails = await API.graphql(graphqlOperation(mutations.updateCustomer {updateCustomerInput:
         {
            UserID: sub,
            Name: "name"
        }
    }));
    console.log(UserDetails);
  };

This code doesn't generate any errors, and it's pretty much the same as the code I used to create the record in the first place which works.
but it returns this data and doesn't actually update the record as intended.
Object {
  "data": Object {
    "updateCustomer": null,
  },
}

Amplify CLI Version
4.29.0
To Reproduce
Create sample Serverless RDS and do Update Mutation
Expected behavior
UpdateMutation should work
Desktop (please complete the following information):

OS: Mac
Node Version. v10.21.0

@ammarkarachi do you have any update on this ?


Possibly redundant BatchNorm layer?

@xptree Thank you for the code. It seems that there are two cascaded BatchNorm layers in each GIN layer. I am wondering whether one of them is redundant.
Specifically,
In the UnsupervisedGIN class, BNs are instantiated (use_selayer is False, as in the code):

and called during forward:

Meanwhile,
In the ApplyNodeFunc class, a BN is again instantiated and called during forward  (use_selayer is False, as in the code):

So in each layer, there are two cascaded BNs, between which there is only a ReLU activation.
As a novice in GNN, I did not see such implementation (cascaded BNs) elsewhere.  Could you please explain why you did this? Does this implementation lead to better performance than keeping only one BN in each layer?
Thank you!

Hi @zhikaili ,
Thanks for point out. However, this is in GIN's implementation and we left it unchanged. It's not any more special in GCC. Please see https://github.com/dmlc/dgl/blob/master/examples/pytorch/gin/gin.py.


Data: project addition by HartzFrequency[GSSoC 23'][Discord API & Python]

Add a new project to the list
Wave
I want to add my Discord bot which is made using Python to resolve the problem which is an external software requirement for recording voice chats of DIscord channels
this discord bot will  allow you to record the conversations and can give provide you the recording
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

hey @HartzFrequency you don't need to create multiple issues at a time. You can add multiple projects in a single issue.


respondent parents' address info required vs. optional

tested in LHI (Oakland & Mason for all 3 petition types), and there, if we say the parent does not live with respondent, the parent's address is required.

Amy actually wants to change this to make it always optional (see #214), so canceling this.


won't work saving custom weights to saved_model.pb

tried with cpu & gpu tensorflow
python3 save_model.py --weights ./data/custom.weights --output ./checkpoints/custom-416 --input_size 416 --model yolov4
2020-11-15 10:47:19.227868: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2020-11-15 10:47:20.467105: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcuda.so.1
2020-11-15 10:47:21.073388: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-11-15 10:47:21.074016: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1716] Found device 0 with properties:
pciBusID: 0000:00:04.0 name: Tesla V100-SXM2-16GB computeCapability: 7.0
coreClock: 1.53GHz coreCount: 80 deviceMemorySize: 15.75GiB deviceMemoryBandwidth: 836.37GiB/s
2020-11-15 10:47:21.074059: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2020-11-15 10:47:21.076255: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcublas.so.10
2020-11-15 10:47:21.078071: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcufft.so.10
2020-11-15 10:47:21.078490: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcurand.so.10
2020-11-15 10:47:21.080445: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusolver.so.10
2020-11-15 10:47:21.082111: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusparse.so.10
2020-11-15 10:47:21.082262: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libcudnn.so.7'; dlerror: libcudnn.so.7: cannot open shared object file: No such file or directory
2020-11-15 10:47:21.082281: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1753] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
2020-11-15 10:47:21.082669: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2020-11-15 10:47:21.091500: I tensorflow/core/platform/profile_utils/cpu_utils.cc:104] CPU Frequency:<PHONE_NUMBER> Hz
2020-11-15 10:47:21.092099: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x6544520 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-11-15 10:47:21.092130: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-11-15 10:47:21.094041: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1257] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-11-15 10:47:21.094074: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1263]
Traceback (most recent call last):
 File "save_model.py", line 58, in <module>
   app.run(main)
 File "/home/ubuntu/.local/lib/python3.5/site-packages/absl/app.py", line 303, in run
   _run_main(main, args)
 File "/home/ubuntu/.local/lib/python3.5/site-packages/absl/app.py", line 251, in _run_main
   sys.exit(main(argv))
 File "save_model.py", line 54, in main
   save_tf()
 File "save_model.py", line 49, in save_tf
   utils.load_weights(model, FLAGS.weights, FLAGS.model, FLAGS.tiny)
 File "/home/ubuntu/yolov4-custom-functions/core/utils.py", line 143, in load_weights
   conv_weights = conv_weights.reshape(conv_shape).transpose([2, 3, 1, 0])
ValueError: cannot reshape array of size 4554552 into shape (1024,512,3,3)

setting up libcudnn of corresonding version doesn't impact the outcomes;
any ideas?

at another device
Traceback (most recent call last):
  File "save_model.py", line 58, in <module>
    app.run(main)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/absl/app.py", line 303, in run
    _run_main(main, args)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/absl/app.py", line 251, in _run_main
    sys.exit(main(argv))
  File "save_model.py", line 54, in main
    save_tf()
  File "save_model.py", line 49, in save_tf
    utils.load_weights(model, FLAGS.weights, FLAGS.model, FLAGS.tiny)
  File "/home/ubuntu/yolov4-custom-functions/core/utils.py", line 143, in load_weights
    conv_weights = conv_weights.reshape(conv_shape).transpose([2, 3, 1, 0])
ValueError: cannot reshape array of size 4554552 into shape (1024,512,3,3)

at another device
Traceback (most recent call last):
  File "save_model.py", line 58, in <module>
    app.run(main)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/absl/app.py", line 303, in run
    _run_main(main, args)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/absl/app.py", line 251, in _run_main
    sys.exit(main(argv))
  File "save_model.py", line 54, in main
    save_tf()
  File "save_model.py", line 49, in save_tf
    utils.load_weights(model, FLAGS.weights, FLAGS.model, FLAGS.tiny)
  File "/home/ubuntu/yolov4-custom-functions/core/utils.py", line 143, in load_weights
    conv_weights = conv_weights.reshape(conv_shape).transpose([2, 3, 1, 0])
ValueError: cannot reshape array of size 4554552 into shape (1024,512,3,3)

1

Hi!
Try to change line 14 of ./core/config.py file to custom.names before running save_model.py.

@jmudy
yes, but the file custom.names is not supplied with the package  https://github.com/theAIGuysCode/yolov4-custom-functions/tree/master/data/classes

@jmudy maybe you could attach the file custom.names?

Put this file in ./data/classes and it should work for you:
https://we.tl/t-yLRdRZK7ls

Thank you very much!
@jmudy

Hi I put my .name file in the ./data/classes folder but I still got the same error. Did you update anything else? Thanks!

the pointer should point tot he names file also as is referenced earlier in the thread

Поместите этот файл в ./data/classes, и он должен работать для вас:
https://we.tl/t-yLRdRZK7ls

Hello, link not working
Please post the contents of this file here
Many thanks

Поместите этот файл в ./data/classes, и он должен работать для вас:
https://we.tl/t-yLRdRZK7ls

Hello, link not working
Please post the contents of this file here
Many thanks

Ooook
custom.names contain one row "car" its work fine

Ooook
custom.names contain one row "car" its work fine


Trigger cache stampeed

The purpose of this PR is to trigger some pain to see if we can stampede the cache on Azure Pipelines.

Can one of the admins verify this patch?


ConfigMap mission / Spring Boot: incorrect instructions

Step 4 of procedure using oc cli should use file:
greeting-service/src/main/resources/application-local.properties instead of src/main/etc/application.properties (that last file doesn't exist anymore).

Note that this fix wouldn't be needed if we used instructions as detailed in #786

@metacosm will  change in  #786 work across all missions. If so we will  consider implementing this one based on decision on https://github.com/fabric8-launch/appdev-documentation/issues/786.


Prevent duplicate requirements with add_python_requirement

Describe the bug
When I run add_python_requirement and the requirement is already listed in config.yaml, it is listed again.
e.g.
requirements:
- joblib==1.0.0
- joblib==1.0.0
- joblib==1.1.0

To Reproduce
Run the add_python_requirement command twice with the same requirement
import truss

truss.init("test_truss")
tr = truss.from_directory("test_truss")
#tr.add_python_requirement("joblib==1.0.0")
#tr.add_python_requirement("joblib==1.0.0")
#tr.add_python_requirement("joblib==1.1.0")

Expected behavior
If the package is already a requirement, don't add it again.
If the package is already a requirement but I am specifying a different version, update the version rather than duplicating the entry.
Expected output of the above code sample:
requirements:
- joblib==1.1.0

Screenshots/Logs
If applicable, add screenshots or logs to help explain your problem.
Desktop (please complete the following information):

OS: MacOS
Environment versions: Python 3.7
Truss version: 0.0.30

additional information
Investigate whether the same thing happens for system requirements and fix it there, too!

Hi, I wonder if this is still an issue?

Hi, I wonder if this is still an issue?

I think it is.

I can work on this if it's okay

That would be wonderful, thank you so much!

This is great! thank you as well

Hi, I'm working on this but it is not allowing me to create a branch, is there anything I should do to gain access?

You can create a PR using a fork: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork

Sounds good! thank you

This issue has been resolved! It was fun :)

Thanks @tugrulguner


Oracle Cloud Object Storage as a Backend

Output of restic version
restic 0.16.0 compiled with go1.20.6 on darwin/amd64
What should restic do differently? Which functionality do you think we should add?
Add OCI (Oracle Cloud Infrastructure) as a backend.
What are you trying to do? What problem would this solve?
This will allow end users of Oracle Cloud to use restic for backing up data to OCI Object Storage , using various OCI auth mechanisms like:

user principal
instance principal
workload identity

From what I can tell, OCI Object Storage also offers an S3 API. We're not particularly eager to add new backends as these come with a significant maintenance burden. (We'd also need a way to continuously run tests for the backend).
I'll leave the issue open for now to see how interest there is, however, I think that using the S3 compatible API is the way to go.

I would love to contribute to this and noticed backends like azure were running tests by setting env variables. i feel this would be a great addition for OCI customers mainly due to auth mechanisms like instance principals and workload identity.

OCI Object Storage does support S3 compatibility, but to get a full feature set (like auth mechanisms, among other things), native OCI APIs should be used.

Hey @theabmitra,
I see that rclone supports Oracle Object Storage. If you have an immediate use case, you can use rclone as a backend with Oracle Object Storage configured.
However, there would be some performance hits, due to the overheads of rclone REST. We had done some tests when we proposed SMB as a backend for restic sometime back #4185, the differences between rclone+restic and direct SMB connections were significant.
We will draw more interest if we can create a draft PR to test and compare the performance between using Oracle Object Storage SDK in Golang and the rclone method.
I am happy to help with a draft PR.

@konidev20 we also want to support Velero as well, and since velero internally defaults to restic , we would prefer restic than rclone.

Looks like under the hood rclone also uses the S3-Compatible API Configure rclone for Object Storage

To configure rclone for Oracle Cloud Infrastructure Object Storage, you must use the Amazon S3 Compatibility API. First, you need to obtain proper credentials and the API endpoint and then set the correct environment variables.

I agree with:

OCI Object Storage does support S3 compatibility, but to get a full feature set (like auth mechanisms, among other things), native OCI APIs should be used.

From what I can tell, OCI Object Storage also offers an S3 API. We're not particularly eager to add new backends as these come with a significant maintenance burden. (We'd also need a way to continuously run tests for the backend).
I'll leave the issue open for now to see how much interest there is for this feature, however, I think that using the S3 compatible API is the way to go.

@MichaelEischer I understand the concern here. As @theabmitra mentioned the S3 compatibility layer is a compatibility layer and doesn't support a number of more advanced features available in OCI Object Store. A number of folks inside Oracle are using restic and the S3 layer, some others are using the rclone backend. But @theabmitra actually already wrote the necessary code for the backend and associated test cases. And I'm looking into what we would need to do to get it released to the wider world.
What would you need from us to accept a PR with that code?

The two main questions that have to be addressed are whether the improvement is worth the added maintenance effort and how the backend can be maintained.

S3 compatibility layer is a compatibility layer and doesn't support a number of more advanced features available in OCI Object Store.

Can you be a bit more specific which features relevant to restic are missing? What would be the benefits of having the OCI backend in restic compared to using the S3 backend?

A number of folks inside Oracle are using restic and the S3 layer, some others are using the rclone backend.

rclone at first glance also seems to support authentication methods like workload_identity_auth . So what's the benefit of integrating the backend natively into restic other than being integrated?

And I'm looking into what we would need to do to get it released to the wider world.

The other question is how this backend will be maintained. Who's going to look into bugs / problems related to the backend? Or maybe even add some extra features? While I can review PRs for the backend, those PRs would mostly have to be prepared by someone else.
All backends must also be tested in some way by our CI tests. We currently have two categories. Some backends start a local server for testing and interact with that and the others are run against the real cloud service (although not all authentication methods are tested), which requires corresponding credentials. Those latter tests are only run for the master branch and branches in the main repository to prevent credentials leaks (well, actually Github enforces that behavior). OCI would probably fall into the latter category? Leading to the question have we'd get the necessary credentials.
Having such automated tests is absolutely necessary, as from time to time we also make changes to the Backend interface which correspondingly requires changes to all backends. Without having a way to test those changes ourselves, the backend would probably just break over time.

i am also getting the restic issues while using velero  , as we are using OCI file share in pod and we need to take backup for those pods , currently getting below error when we preform the backup . not seen this issues with AKS
error running command=restic init --repo=s3:
https://xxxxxxx.compat.objectstorage.eu-frankfurt-1.oraclecloud.com/xxxxxxxvelero01/oke/restic/nginx-fss
--password-file=/tmp/credentials/velero/velero-repo-credentials-repository-password --cache-dir=/scratch/.cache/restic, stdout=, stderr=Fatal: create repository at s3:
https://xxxxxxx.compat.objectstorage.eu-frankfurt-1.oraclecloud.com/xxxxxxxxxvelero01/oke/restic/nginx-fss
failed: client.BucketExists: The secret key required to complete authentication could not be found. The region must be specified if this is not the home region for the tenancy.

@roshithrajan That sounds like the region isn't specified correctly.
From https://restic.readthedocs.io/en/stable/030_preparing_a_new_repo.html#s3-compatible-storage:

If needed, you can manually specify the region to use by either setting the environment variable AWS_DEFAULT_REGION or calling restic with an option parameter like -o s3.region="us-east-1".

Thank you @MichaelEischer  for the response and i have specified the region while installing the velero do we need to specify specifically for restic in somewhere ?. i have used below commands
velero install     --provider aws     --bucket okeprodvelero01     --prefix oke     --uploader-type=restic      --secret-file /home/opc/velero/credentials-velero --backup-location-config s3Url=https://axxxxxx.compat.objectstorage.eu-frankfurt-1.oraclecloud.com,region=eu-frankfurt-1,s3ForcePathStyle="true" --plugins velero/velero-plugin-for-aws:v1.7.1 --use-volume-snapshots=false --velero-pod-cpu-request "200m" --velero-pod-mem-request "512Mi" --velero-pod-cpu-limit "1" --velero-pod-mem-limit "2048Mi" --use-node-agent --namespace velero --node-agent-pod-cpu-request "200m" --node-agent-pod-mem-request "512Mi" --node-agent-pod-cpu-limit "1" --node-agent-pod-mem-limit "2048Mi"

when i changed the bucket  to home region its working now

i have specified the region while installing the velero do we need to specify specifically for restic in somewhere ?

That's a velero question, not a restic one, sorry.


Update method Program.PromptYesNo to consider the case when Input is redirected

In the case when the application is run by another application in redirect input mode, method Console.ReadKey throws an exception.
To work this around, the updated code checks for redirected input and if input is being redirected, it uses Console.Read() instead of Console.ReadKey.

If the input is redirected perhaps we should realise there's a good reason for the user to do this; like they don't want it to ask something of the user because they won't be there.
If the redirected input cannot be read a better solution would be to always return "Yes", and make sure "yes" is always the reasonable answer.

Perhaps a better idea would be the following?
(Disclaimer - I have not looked at the code...)
If PromptYesNo is used for confirming the user really wants to use what was entered, why not have a command line argument that specifically indicates it is being run in "quiet" or "unattended" or "noprompt" mode.  Because passing a value in an input file, and then passing confirmation that you "typed" it properly also in that input file seems silly...
Should be an easy change to check if "noprompt" parameter was passed, and if so, assume the input is correct and don't prompt at all...
Just be sure to test your input file by including --test the first time you run it.  If it looks good, remove --test and you are all set!

Check out the options for unattended use here:
https://github.com/Lone-Coder/letsencrypt-win-simple/wiki/Command-Line-Arguments


Missing error with recursive types and literals

alias RecursiveType = String |
                      Int32 |
                      Array(RecursiveType) |
                      Hash(String, RecursiveType)

extra_fields = {
  "teste" => "teste",
  "foo" => {
    "bar" => "blabla"
  } of String => RecursiveType,
  "array" => ["teste", "teste"]  of RecursiveType,
  "deep" => [
    {
      "foo" => {
        "foo" => "bar"
      }
    }
  ] of RecursiveType
} of String => RecursiveType

def deep_print(rc : RecursiveType)
  puts "#{rc} was not a RecursiveType" unless rc.is_a? RecursiveType

  case rc
  when Array(RecursiveType)
    rc.each { |itm| deep_print(itm) }
  when Hash(String, RecursiveType)
    rc.each { |key, itm| deep_print(itm) }
  end
end

deep_print(extra_fields)

Expected an error that {"foo" => {"foo" => "bar"}} was not a Hash(String, RecursiveType). Actual:
{"foo" => {"foo" => "bar"}} was not a RecursiveType

https://play.crystal-lang.org/#/r/1i8m
The case statement doesn't match any branches, but I think it would be possible to get a segfault from this somehow.
Crystal 0.20.3

Yeah, right now type restrictions and is_a? work a bit different, has to do with covariance and contravariance. This can't lead to a segfault because even thought the restriction lets the type pass, the compiler knows that it's not of that type so it won't let you use it as such.
Reduced:
class Foo
end

class Bar < Foo
end

def foo(r : Array(Foo))
  puts r.is_a?(Array(Foo)) # => false
end

foo(Array(Bar).new)

What should happen in the above code? The fundamental question is "is an Array(Bar) an Array(Foo)?". Well, if you want to read from it than the answer is probably yes. But if you want to write to it (if you want to put a Foo inside an Array(Bar)) then the answer is probably now. A type restriction is currently giving you the first answer, while is_a? is giving you the second answer.
Making both give the second answer would be an easy solution, except that things like a restriction being Range(Int, Int) will break.
I'll open a separate issue for this so we can close all related covariance issues and link to that one.

Closed in favor of #3803


Add a poll_listen() function

By embedding a singular Entry into the Event/Inner structure itself, we could probably add a poll_listen() function that allows a user to poll for an Event notification without needing to use an EventListener. Such a function would probably have the same caveat that poll_read()/poll_write() does in async_io::Async, where if it's called by different tasks it will cause a busy loop where one task wakes up another.

This would be incompatible with #42

At this point this would require a substantial amount of effort to integrate into the system, which probably wouldn't be worth it.


[BUG] Wrong changed-files returned for forked PRs

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
Lately, we upgraded to the latest version of changed files, and we are again facing problems with the script detecting too many files changed.
The expected outcome for this PR:
https://github.com/raycast/extensions/pull/3245
Will be to get the files listed in the files changed section, but instead, we get many extra files that have been changed on the main repository between the time this PR was created till the Check is running.
On this run you can see all those files being listed. They are coming from all commits that have been done on the main repository and probably coming from "merge" action during checkout.

To Reproduce

Setup main repository
Fork the main repository
Push some commits to main repository
Push some commits to forked respository
Run a files-changed on PR created from the forked repository and observe multiple files being reported on top of PR changes.

What OS are you seeing the problem on?
all
Expected behavior?
I would expect to only see files that are listed in PR.
Instead of comparing to the BASE_SHA, it should instead compare to the commit that was used to do the merge, in our case 4ef2e916978e1c645499e3a6c7a536bdc3153403.
The last version that was behaving properly was v14.6 and it was really solid in this case. It was using a git log --no-merge to get proper SHA to compare to.
Relevant log output
No response
Anything else?
Is there a way to achieve the same behavior as in v14.6 for PRs?
We need to upgrade script version due to GitHub deprecations.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@grzegorzkrukowski Can you confirm that this is still an issue?

hey @jackton1 - yes it's still the same case we discussed a long time ago here. We are using your awesome action from that time and we sticked to 14.3 version as it was working perfectly, but now due to GitHub deprecations I wanted to upgrade and I found out it's giving us completely different results than 14.3 :(

I'm guessing you mean v14.6 instead of v14.3 as highlighted here?

The last version that was behaving properly was v14.6 and it was really solid in this case. It was using a git log --no-merge to get proper SHA to compare to.

Yes, sorry, I fixed the comment.
That older version served us well for a long time, giving us proper results for quite some time after our last session of fixes in the PR I mentioned above.
When I upgrade now to the latest version, most of the things we talked about back then are gone, and I cannot figure out how to properly configure the newest version of this action to give me the expected results, which is: "give me all files changed by PR" after doing an action/checkout@v3 (GitHub is making a merge commit there).
In general, it's as simple as:  git diff --name-only -r HEAD^1 HEAD - so I am looking into my own custom solution for this, but would be great to have it working through generic action like this one.

@grzegorzkrukowski I created a demo repository here to replicate this issue can you create a PR there as the current PR with multiple runs shows only files that have been modified in the PR?
It would be really helpful to replicate the problem there and based on your comment about using git diff --name-only -r HEAD^1 HEAD I believe that's the default behaviour of using a push event instead of pull_request

Got it, that helps I was able to replicate it here https://github.com/jackton1/demo3/actions/runs/3379055121/jobs/5610106082

@jackton1 great to hear it's not be going crazy :)
Thanks for looking into this problem - I assume it's quite unusual, but for us, it's happening every day :(

I have same issue.
Basically, it returns difference between my "feature" branch and current master, but not only what I have changed in my branch.

@grzegorzkrukowski @ViacheslavKudinov can you verify that this works for you. Thanks

hey @jackton1 - thanks for a quick fix.
It worked for me. I left some comments on your PR - I think I found some "typo" in three places for error messages.
Also, the newest version gives some extensive logging - you can check it here:
https://github.com/raycast/extensions/actions/runs/3260964202/jobs/5660537605#step:4:164
It outputs all changed files for some reason - not sure if that's desired.

Glad to hear, thanks for reviewing the PR all typos were resolved in a subsequent PR. Let me know if you run into any other issue I’ll look into the diff output as that is handle in a different repository.

Hi,
I've tested on Friday 34.1.1, but now I see it is the newest version.
I need to test this new one then.
I will be back with results the next week.

Hi,
I think it works fine now.
Thanks for the fix.

Hi,
unfortunately, we get it again on 34.1.1 and update to 34.4.0 even worst as has this issue https://github.com/tj-actions/changed-files/issues/802

@ViacheslavKudinov can you verify that this is still an issue in the latest release?

Hi @jackton1
yes, i've updated for the new version and monitor result.
TBH, this situation with "wrong behaviout" was coming not all the time. It is difficult to catch the moment and the reason why it appears as we use it on pretty active monorepo.

We also upgraded today to 34.4.4 - will be monitoring the behaviour this week on our monorepo

Okay, I'll recommend posting any related logs with debugging enabled which would be helpful to resolve the issue. Thanks

Hi @jackton1
I've just got situation with wrong list of changed folders.
PR includes 2 folders we subscribe on: dev/developer-handbook and prod/developer-handbook

But Actions returned 3 folders: dev/developer-handbook prod/developer-handbook prod/hawkeye-CENSORED

Step was run  like this (from logs):
un tj-actions/glob@v16
2022-11-22T16:11:33.9231612Z with:
2022-11-22T16:11:33.9231845Z   files: dev/
prod/

2022-11-22T16:11:33.9232103Z   files-separator:

2022-11-22T16:11:33.9232565Z   excluded-files: prod/hawkeye-CENSORED/dashboards
prod/drone-api/
**/termination-lambda/
**/modules/
**/packer/
**/inspec/
.github
*.md

2022-11-22T16:11:33.9233039Z   excluded-files-separator:

2022-11-22T16:11:33.9233327Z   escape-paths: true
2022-11-22T16:11:33.9233585Z   working-directory: .
2022-11-22T16:11:33.9233891Z   base-sha: c705878f807045364fda9badf7ad6f78ddd5c89c
2022-11-22T16:11:33.9234240Z   sha: af7e12ea70b830ca06da9c899a8eccfe6cddc2f9
2022-11-22T16:11:33.9234546Z   include-deleted-files: true
2022-11-22T16:11:33.9234812Z   separator: |
2022-11-22T16:11:33.9235103Z   files-from-source-file-separator:

2022-11-22T16:11:33.9235639Z   excluded-files-from-source-file-separator:

2022-11-22T16:11:33.9235992Z   follow-symbolic-links: true
2022-11-22T16:11:33.9236282Z   match-directories: true
2022-11-22T16:11:33.9236567Z   strip-top-level-dir: true
2022-11-22T16:11:33.9237553Z   base-ref: master

Workflow  file looks like:
	jobs:
  get_changes:
    uses: CENSORED/git-changed-folders-workflow/.github/workflows/reusable-workflow.yml@v0
    with:
      excluded_folders: |
        prod/hawkeye-CENSORED/dashboards
        prod/drone-api/
        **/termination-lambda/
        **/modules/
        **/packer/
        **/inspec/
        .github
        *.md
      included_folders: |
        dev/
        prod/

it is reusable workflow which sends inputs into step like this:
      - name: Get changed dirs
        id: get_changes
        uses: tj-actions/changed-files@8a7336fb6f6bc00da867b745d3491de42ac0231b # tag v34.4.4
        with:
          dir_names: ${{ inputs.dir_names }}
          files_ignore: ${{ inputs.excluded_folders }}
          files: ${{ inputs.included_folders }}

What I see, change of folder prod/hawkeye-CENSORED was done in master before I've pushed my branch.
It looks  like Action catches difference between current default branch and my "feature" branch, but not only changes i've done in my branch.
We use squash merge approach in our default branch.

I tested it as well. Getting all the changes again from the main branch when it should be just PR changes.
Here is the PR:
https://github.com/raycast/extensions/pull/3171/files (11 files changed)
and related Check that returned all files from main branch:
https://github.com/raycast/extensions/actions/runs/3532228657/jobs/5926328721

@grzegorzkrukowski Curious to know if this is still an issue as I believe the bug was introduced here https://github.com/tj-actions/changed-files/issues/702 i.e we were previously using a two-dot diff in previous versions.

We started to use v34.5.1 and didn't get issues.
But it was not the all time the issue pop up.
So, I will monitor for some time and let you know if the issue is still here.
Thank you for your efforts @jackton1

We are using V42 and are seeing this issue with fork prs. We are having the same problem . The change files is doing comparison between the latest on main to the latest on feature branch, which includes changes made after the branch was created.

We are using V42 and are seeing this issue with fork prs. We are having the same problem . The change files is doing comparison between the latest on main to the latest on feature branch, which includes changes made after the branch was created.

Hi @NikhitaY22,
You'll need to update the checkout action if the changes files include changes outside the PR i.e made to the main branch after the PR was created.
See: Checkout pull request HEAD commit instead of merge commit
      - uses: actions/checkout@v4
        with:
          ref: ${{ github.event.pull_request.head.sha }}

@jackton1 we are already using Checkout pull request HEAD commit instead of merge commit

Hi @NikhitaY22 In that case I’ll suggest checking out the source repository also.
Example
- uses: actions/checkout@v4
  with:
    repository: ${{ github.event.pull_request.head.repo.full_name }}
    ref: ${{ github.event.pull_request.head.ref }}

@jackton1 Tested with adding repository to checkout step and the changed-files step has failed doing so on fork.

Sorry I had a typo in the ref input, it should be sha not ref
Change:
ref: ${{ github.event.pull_request.head.ref }}
To:
ref: ${{ github.event.pull_request.head.sha }}

We already have it set correctly.

name: Checkout PR head ref
uses: actions/checkout@v4
with:
repository: ${{ github.event.pull_request.head.repo.full_name }}
ref: ${{ github.event.pull_request.head.sha }}

Can you enable debug mode and share the logs. I can provide a more accurate solution based on the logs and also include the details of how your workflow is setup including details about the triggering event, the checkout action and changed-files actions setup.

logs_22056975141.zip

@jackton1 here are logs
logs_22056975141.zip

Hi @NikhitaY22 you previously posted the list of expected changed files, I'll need a full description that includes the list of the forked PR changed files for comparison and your workflow file to proceed.

Here are details
Fork PR actual changed files

changed-files detected files

logs_22079120820.zip
Workflow:
name: Check PR to main for blocking comments

on:
  pull_request_target:
    types:
      - opened
      - reopened
      - review_requested
      - synchronize
    branches:
      - main

jobs:
  pullrequest_to_main:
    runs-on: ubuntu-latest

    steps:
      # get changed files from PR
      - name: Checkout PR head ref
        uses: actions/checkout@v4
        with:
          ref: ${{ github.event.pull_request.head.sha }}
      - name: Get changed files
        id: changed-files
        uses: tj-actions/changed-files@v42
        with:
          files: files/**
      - name: Print Changed Files
        env:
          ALL_CHANGED_FILES: ${{ steps.changed-files.outputs.all_modified_files }}
        run: |
          echo "Changed files:"
          cat $ALL_CHANGED_FILES

@jackton1 here are the logs after adding the repository   and reproduced the issue
logs_22080063875.zip

Hi @NikhitaY22 @grzegorzkrukowski @ViacheslavKudinov this issue should resolved in the latest v44 release let me know if you notice any issues and I'll take a look.

@NikhitaY22 you don't need to explicitly set the ref and repository.

@jackton1 But the release notes for v44 suggests that we need to set repository .

To clarify you can omit the ref input. For simple cases like yours you the repository can also be omitted since you are checking out a single commit.


Stream priority

Priority range (1: Lowest - 255: Highest)
Default screen share priority: 255
Default video (camera) priority: 1
Consolidate functions for sorting and sort only when needed. Eliminate maintaining sorted list. That code is cumbersome and error prone. If the number of tracks are so high that sorting at more opportune time makes sense, we can revisit keeping sorted lists.

Thank you @davidzhao @cnderrauber for the review. I added one commit to trigger an allocate on priority change. Simple for now, but some room for improvement - https://github.com/livekit/livekit-server/pull/448/commits/7c12f80dd14c2b5a34eb9af67b0d1c9fc5319e64. Brain is not smart enough to think of clever ways to minimise impact of priority change of a single track :-) Will dig deeper when we have to cross that bridge.
Have tested it reasonably. Going to merge this. But, please do keep an eye out if you can.


it may be cool having a similar UI with the pannel for #%% from python

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
it may be cool having a similar side UI with the pannel for #%% from python, then there's a similar interactive env and no local file(.ipynb) needed.

Hi @NewUserHa, what file type would you expect instead if this experience were to exist?

I think that pannel would be great to test some statements, instead of always a new window for a notebook. So any file type and any trigger tags may be fine.
if that the notebook can support different languages can be better, but that seems to be vscode repo's issue.


Add w3school in exercise

[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 master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi
Thank you for your PR but currently, we do not merge PRs that only add to the More Information section.
You can refer to the Contributing Guide here to see why we close certain PRs.


Province changed proof-of-vaccination receipt format; 2nd dose card gen broken

Overnight, the province pushed a change to their proof-of-vaccination system which now combines all of the dose receipts into a single PDF, whereas previously every dose receipt was a separate PDF. This has (unsurprisingly) broken our card generation, and right now only dose 1 cards are being detected and generated.
A fix for this is in progress.

This has been properly fixed by https://github.com/billylo1/covidpass/pull/30 - thanks @jaspk06 !


How to install for different architectures?

When building for Electron it's possible to build for different architectures, for example:
electron-forge make --arch arm64

For this to work any native modules must be installed for the target architecture. This can be achieved with npm:
npm install --target_arch=arm64

or with yarn:
npm_config_target_arch=arm64 npm_config_target_platform=linux yarn install --force

This works for various modules such as serialport and lzma-native but not for @ngrok/ngrok. How can I force npm or (preferably) yarn to install another architecture such as @ngrok/ngrok-linux-arm64-gnu on a Linux x64 machine?
In our case the build is running on GitHub Actions. I do not have the same problem with building for multiple architectures on macOS because a Darwin "universal" binary is installed.

Hello, thanks for writing in!
At first glance it looks like both lzma-native and serialport include all prebuilds by default, rather than as separate subpackages like this SDK does, as you mention. One option to deal with this would be to reference the subpackages directly, and have it ignore the platform check with force. For instance npm i @ngrok/ngrok-darwin-x64 --force will pull the darwin package down on a linux host. We can look into what it would take for yarn to do a similar behavior, but hopefully the npm path would be able to solve for this use-case? Please let us know if that isn't the case or you run into additional hurdles!

I see! I tried a few things like using Docker run yarn install --force but I ran into a number of problems, especially on GitHub actions.
It seems the sub packages of @ngrok/ngrok include prebuilt .node binaries so actually using Docker isn't necessary. For now, my workaround is to use yarn install --ignore-platform to install the packages for all platforms and architectures.
I would have expected setting npm_config_target_arch to work but I don't understand enough about how NPM/Yarn work to know if I'm misunderstanding.

Ah, it looks like yarn may have stopped supporting npm_config_ settings? https://github.com/yarnpkg/yarn/issues/7629#issuecomment-753631296
It's great that you found a workaround that will get the job done, all the sub-packages do include the binary, and the main package determines which one to use. Let us know if you run into any other issues!

We'll go ahead and close this ticket out, feel free to reopen if you run into other issues. Thanks for writing in!


Unsupported machine type

When running terraform apply with the example libvirt.tf I get the following error:
Error applying plan:

1 error(s) occurred:

* libvirt_domain.terraform_test: Error crearing libvirt domain: [Code-1] [Domain-10] internal error: process exited while connecting to monitor: 2016-04-05T09:56:21.548490Z qemu-system-x86_64: -msg timestamp=on: Unsupported machine type
Use -machine help to list supported machines!

It seems to be caused because the defined machine pc-i440fx-2.4 in domain_def.go is not there for the following libvirt version:
$ virsh --version
<IP_ADDRESS>

Changing the value to an existing machine fixes the problem as expected.

I believe you need qemu 2.4 for the pc-i440fx-2.4 machine type.

I want to keep this provider simple and not expose all qemu features into the configuration but to give a more "cloud" feel.
At some point I even thought about using the standard EC2/OpenStack machine sizes as shortcuts.
But what we can do is to use virConnectGetDomainCapabilities to get the machine type of the host and use that.

Seems to work fine under Leap :)

great to hear that!


How to change App icon?

I've noticed that during the recent commit, the demo app has its icon displayed in the Windows tray. How to change/set that icon?
[I've also seen that you have put the demo app onto ms store, cool! But the icon has transparent problem with its background, have you noticed that?]

For setting the icon, it's as simple as setting the Icon property of the SukiWindow (or in Avalonia, just the Window).
Here is how we do it in the demo app, though this is probably a little unnecessary just a holdover from some old code.
As for the icon's transparent background I did notice and I have no idea why that is happening, it doesn't happen in debug on Windows only in the app store...


Design Istio service config

Istio config model v1 is the same as today. See https://docs.google.com/document/d/1PNzYnG0pl5Bubtx-rGwHFWDnBRkrT9upw5mhMA8EpM8/edit
Config model v2 is the new configuration flow with Config architecture: https://goo.gl/Jft4jf

[ ] Design service representation

[ ] Design service config

[ ] Design consumer config

cc @cmluciano

This is not actionable at this point. Closing.


repo.maven.apache.org no has lucene-analyzers-common 9.7.0 version

https://repo.maven.apache.org/maven2/org/apache/lucene/lucene-analyzers-common/

Here we go https://repo.maven.apache.org/maven2/org/apache/lucene/lucene-analysis-common/9.7.0/
dunno why, really.

The artifact was renamed for 9.x to reflect the directory structure in the repository.

how find 9.7.0 version lucene-analyzers-common jar??  url ?? thank you !
------------------ 原始邮件 ------------------
发件人:                                                                                "apache/lucene"                                                                                @.>;
发送时间: 2023年7月11日(星期二) 下午2:36
@.>;
<EMAIL_ADDRESS>主题: Re: [apache/lucene] repo.maven.apache.org no has lucene-analyzers-common 9.7.0 version (Issue #12432)
The artifact was renamed for 9.x to reflect the directory structure in the repository.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

There is no lucene-analyzers-common, it's been renamed to lucene-analysis-common, the JAR is here:
https://repo.maven.apache.org/maven2/org/apache/lucene/lucene-analysis-common/9.7.0/lucene-analysis-common-9.7.0.jar


Cosmetics on doc and some minor fixes.

Hi. Ive been using this awesome resource to build my virtualbox kubernetes environment.
While performing every step of the guide, I've been updating the docs to add clarity and some minor fixes.
Also tested etcd3.5.1 and working well, so updated that component in the masters.
Thanks for sharing!

Hi @chespix
We have now merged a major change to bring this to v1.24. In this we've also changed some of the wording. Feel free to suggest further changes.


Using copy_person creates duplicate IDs

Bug description
Person::copy_person currently also copies the ID of the person.
Currently, this is fine, as the function is used exclusively to copy an entire population in the World copy ctor.
However, using this function to duplicate a person in the same populitaion leads to non-unique IDs, which could cause issues elsewhere.
Version
Any
To reproduce

Create an instance of Person.
Call copy_person and obtain a different instance sharing the same ID

Relevant log output
No response
Add any relevant information, e.g. used compiler, screenshots.
No response
Checklist

[X] Attached labels, especially loc:: or model:: labels.
[X] Linked to project

This will probably be fixed along with #757

copy_person was deleted when merging #863.


Add comment column to training plan for increased LLM understanding

When using dbt, it is possible to persist column documentation as comments. By including that information in Vanna, it increases the probability of finding the correct query.

I love this! The one thing is that the column may not exist in all information schema and I believe as written it will fail. Would you be able to take a stab at making the comment column optional? No worries at all if you don't have time -- I can either do it myself or get someone else involved.

I love this! The one thing is that the column may not exist in all information schema and I believe as written it will fail. Would you be able to take a stab at making the comment column optional? No worries at all if you don't have time -- I can either do it myself or get someone else involved.

Alright, I did an updated take on it to have a list of optional columns, and I then moved some of the other ones there as well. This should make it easier to add columns later on as well.


Lint rules

Currently we have the following ESLint rules:

eslint-config-airbnb-typescript: a drop-in replacement for eslint-config-airbnb, decorating it with TypeScript support
eslint-config-react-app
eslint-config-prettier: we are using Prettier for formatting

However we changed the severity of several rules:
Core

linebreak-style: off
no-param-reassign: warning
no-prototype-builtins: warning

Import

named: warning
no-extraneous-dependencies: warning
no-named-as-default: warning
no-unresolved: warning
prefer-default-export: off

a11y

anchor-is-valid: warning
aria-role: warning
control-has-associated-label: warning
label-has-associated-control: warning
no-autofocus: warning
no-noninteractive-element-interactions: warning

React

button-has-type: warning
destructuring-assignment: warning
jsx-filename-extension: warning
forbid-prop-types: off
jsx-closing-bracket-location: off
jsx-closing-tag-location: off
jsx-props-no-spreading: warning
no-multi-comp: off
no-unused-prop-types: warning
no-array-index-key: warn
no-unescaped-entities: warning
prop-types: off
require-default-props: warning

Typescript

no-unused-vars: warning
no-use-before-define: warning

Should we keep them all? Should we simplify them? @youf-olivier @samuel-gomez @arnaudforaison

I'm agree with you, toolkit need strict rules but more simple implementation. Maybe only based on Airbnb rules

We have to validate our rules and fix all files in one pass.

I can work on that to fix them then switch the rules from warning to error.

Do we want to keep this rule: jsx-props-no-spreading? All our code use props spreading...

Do we want to keep this rule: jsx-props-no-spreading? All our code use props spreading...

This must be removed.


fix: Support Master Password authentication via env variable

The original PR for this wasn't tested and did not work this PR fixes it, I've tested the changes this time.

@Mikescops

Could we get a patch release as it's a blocker for the Raycast extension update

This will be in the next release, likely this friday or the next one.


Zero balance at fiat section and select currency is under the menu section.

Not sure if that is not a duplicate or maybe it's a new bug. At the web wallet at https://wallet.superhero.com/account there is two bugs:

Zero balance at fiat section.
List that helps to select currency is under the menu section.

Unfortunately, I can't reproduce these issues:

Please reopen if the issue persists

Please reopen if the issue persists

I will recheck it today's afternoon


Text composables: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-classes-and-objects?continue=https://developer.android.com/courses/pathways/android-basics-compose-unit-2-pathway-1%23codelab-https://developer.android.com/codelabs/basic-android-kotlin-compose-classes-and-objects#6
In which task and step of the codelab can this issue be found?
Describe the problem
Spelling error under the section "Reuse superclass code in subclasses with the super keyword"

To call the overridden method in the superclass from teh subclass, ...

Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hi @earlsab
Thank you for writing in with this issue.
It has been corrected and the lesson will be republished again in the near future.
Hope you are enjoying the course.
Regards.


Candle won't use half-gemm from cublas when doing fp16 matmul

This relates to https://github.com/huggingface/candle/issues/2136
Related to improving mistral.rs prompt processing speed https://github.com/EricLBuehler/mistral.rs/issues/153
Why does candle use
turing_fp16_s1688gemm_fp16_256x128_ldg8_f2f_tn kernel for F16 matmuls?
Llama.cpp uses
turing_h1688gemm_256x128_ldg8_tn for the same tensor.

If I understand it correctly from the docs https://docs.nvidia.com/cuda/cublas/index.html#cublas-lt-t-gt-gemm[/url]
h-gemm stands for half-gemm where as s-gemm stands for standard F32 gemm.
So, is it possible that candle is not using the best kernel, for some reason?
Is it possible that the candle version is doing the matmuls in F32, as the name would suggest, thus being slower than the other kernel?
Our benchmarks are:
Llama.cpp: ~1500t/s
mistral.rs: 1000t/s
And the major contributors are the kernels I mentioned above. Notice the proportion of time spent on each kernel pretty much matches our observed slowdown. More info here https://github.com/EricLBuehler/mistral.rs/issues/153#issuecomment-2081632685

Maybe candle needs to directly call hgemm_strided_batched https://github.com/coreylowman/cudarc/blob/64ddbc77a9a84ea4b8bb6b918b34c81beed71b24/src/cublas/result.rs#L222 here https://github.com/huggingface/candle/blob/main/candle-core/src/cuda_backend/mod.rs#L1654 ?

We're actually using this function which calls the generic gemm variant with sys::cublasComputeType_t::CUBLAS_COMPUTE_32F, a few lines below, so this means that it's a f16 kernel (as per the actual kernel name) but the accumulation is done using f32 as detailed here. Using f16 accumulation would indeed be faster but with lower precision so it's a bit unclear to me what the impact of this would end up being and if that would be good enough for most use cases.

Ah, I see, thanks.
We're currently trying to match llama.cpp speed using quantized models, so the loss of precision shouldn't matter for us.
But I can see how it matters for a regular F16 model...

Perhaps I could add this to my fork so we can try it out, and then we can merge it if we find an elegant solution?

@EricLBuehler I'd be glad to benchmark it, profile it etc if you implement it

I forked candle locally and hacked a call to the above function at https://github.com/huggingface/candle/blob/main/candle-core/src/cuda_backend/mod.rs#L1654
unsafe fn gemm_strided_batched<
    A: cudarc::driver::DevicePtr<half::f16>,
    B: cudarc::driver::DevicePtr<half::f16>,
    C: cudarc::driver::DevicePtrMut<half::f16>,
>(
    handle: sys::cublasHandle_t,
    cfg: StridedBatchedConfig<half::f16>,
    a: &A,
    b: &B,
    c: &mut C,
) -> std::result::Result<(), CublasError> {
    let alpha = cfg.gemm.alpha;
    let beta = cfg.gemm.beta;
    result::gemm_strided_batched_ex(
        handle,
        cfg.gemm.transa,
        cfg.gemm.transb,
        cfg.gemm.m,
        cfg.gemm.n,
        cfg.gemm.k,
        (&alpha) as *const half::f16 as *const _,
        *a.device_ptr() as *const _,
        sys::cudaDataType_t::CUDA_R_16F,
        cfg.gemm.lda,
        cfg.stride_a,
        *b.device_ptr() as *const _,
        sys::cudaDataType_t::CUDA_R_16F,
        cfg.gemm.ldb,
        cfg.stride_b,
        (&beta) as *const half::f16 as *const _,
        *c.device_ptr_mut() as *mut _,
        sys::cudaDataType_t::CUDA_R_16F,
        cfg.gemm.ldc,
        cfg.stride_c,
        cfg.batch_size,
        sys::cublasComputeType_t::CUBLAS_COMPUTE_16F,
        sys::cublasGemmAlgo_t::CUBLAS_GEMM_DEFAULT_TENSOR_OP,
    )
}

This matches the llama.cpp config, and it matches the same used kernels. Llama.cpp is at the bottom.

I perceived no difference in output quality.
Using these settings made it improve by 15% getting mistral.rs to ~1150t/s

Great that it works well with the reduced precision, I've looked a bit at the pytorch codebase and it seems that they use f32 accumulation by default. PyTorch provides an option to disable "reduced precision" here) (which is turned on by default) but this only impacts the truncation setting in SetMathMode. See this issue https://github.com/pytorch/pytorch/issues/123157 .
So to get around this, I've pushed #2141 , this provides a toggle to flip between the reduced precision accumulation and f32 accumulation - which remains the default. It's a global flag so not ideal but at least provides a way to test the reduced precision accumulation, the quantized example has been adapted to use it and indeed benefits from the speedup when using the f16 matmul for the prompt processing. Would that work for your use case?
When it comes to changing the default, it might be better to wait a bit for what happens on the PyTorch side. If models are trained with f32 accumulation, it's a bit unclear to me what the impact will be if one runs inference with a less precise accumulation.

I'm also not sure about making it the default.
The approach of https://github.com/huggingface/candle/pull/2141 fits our use case.
Thank you a lot!


[stable/heapster] Add annotations to service

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

/assign @lachie83

/assign
/ok-to-test

/lgtm

/lgtm


[plugin] flake8_pytest_style without config section (pyproject.toml) crashes

minimal reproducing config:
select = ["PT"]

output:
[2023-01-03][13:28:56][ruff::cache][ERROR] Failed to deserialize encoded cache entry: Io(Kind(UnexpectedEof))
[2023-01-03][13:28:56][ruff::cache][ERROR] Failed to deserialize encoded cache entry: Io(Kind(UnexpectedEof))
thread '<unnamed>' panicked at 'called `Option::unwrap()` on a `None` value', src/flake8_pytest_style/plugins/parametrize.rs:196:29
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Fixing now.


Mh3 24130 gsg

GSG HTML Pull Request
Description

Updated Doc file format and added GSG in HTML format.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I will raise a null pull request


move audit reports from old repo

:clipboard:
Overview
Copies audit-reports into main repo

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 23.50%. Comparing base (ab724bc) to head (7ce8a49).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #215       +/-   ##
============================================
- Coverage    41.31%   23.50%   -17.81%
============================================
  Files           74       40       -34
  Lines         5001     2076     -2925
  Branches       785      363      -422
============================================
- Hits          2066      488     -1578
+ Misses        2828     1513     -1315
+ Partials       107       75       -32

Flag
Coverage Δ

cannon-go-tests
?

chain-mon-tests
27.14% <ø> (ø)

common-ts-tests
26.72% <ø> (ø)

contracts-ts-tests
12.25% <ø> (ø)

core-utils-tests
44.03% <ø> (ø)

sdk-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 34 files with indirect coverage changes


"Premature end of stream" exception when decompressing re-compressed output

I was working on a modding tool for a game that uses ZStandard, the tool is capable of decompressing (exporting) and recompressing (importing) files compressed with zstd, it turns out that every version of ZstdSharp after 0.6.1 is unable to decompress back the buffer it has just compressed, throwing the aforementioned exception.
Example of decompressing
Example of compressing
0.6.2 introduced a new argument to DecompressionStream called checkEndOfStream, and setting that to false doesn't give an exception, but the files end up being corrupted (in my case that is textures, they appear cut off at the bottom), 0.6.1 does not give any exception and does not corrupt files, either
What I do is, I open the original file from the game, it decompresses fine, then I re-import the same file, it compresses with no errors, but gives the exception when trying to decompress the same compressed buffer it just produced, and loading it in-game looks broken by the end of the file

You can try to add Dispose / Close to finalize the stream data:
using (var bufferWriter = new ArrayPoolBufferWriter<byte>())
{
    using (var zss = new CompressionStream(bufferWriter.AsStream(), ZstdSharp.Unsafe.Methods.ZSTD_defaultCLevel()))
    {
        zss.Write(data, 0, data.Length);
        zss.Flush();
    }
    _data = bufferWriter.WrittenSpan.ToArray();
}

That appears to have fixed the issue, thanks!


Add Config.json file check

To ensure security, we need to store the token and prefix in a config.json file, so that we can use a .gitignore to ignore the file, so it isn't pushed.

Sorted!
Screw Python and its indentation man...


Dev

Codecov Report

Merging #102 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   98.99%   98.97%   -0.03%
==========================================
  Files          13       13
  Lines         698      682      -16
  Branches      174      171       -3
==========================================
- Hits          691      675      -16
  Misses          7        7

Impacted Files
Coverage Δ

src/utils/index.ts
100% <ø> (ø)
:arrow_up:

src/utils/params.ts
100% <ø> (ø)
:arrow_up:

src/provider/rpc-methods.ts
100% <100%> (ø)
:arrow_up:

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 64cebc9...fb0050d. Read the comment docs.


[optimization] Consider generating non-leaf nodes on the fly without storing

Try to imagine the request flow starting from data root up to leave and what the logic would look like for a responder that only has leaves.

Here is the naive way:
Case 1) requesting all data for a height for instance:
just return all leaves. requester can recompute everything locally and just compare against root.
Case 2) requesting one leaf (e.g. per index):
Load block of that height (all leaves), recompute the inner nodes of corresponding Merkle tree and respond with leaf + corresponding inclusion proof.
case 3) request all leaves according of one namespace:
like in 2) but return a bunch of leaves and a with the corresponding (recomputed) proof.
A less naive approach would acknowledge the fact that always loading all leaves to respond might be undesirable as well. So what could be a middle ground? Only store some inner nodes, e.g. store the tree in packages of smaller subtrees with their roots (aka some inner nodes) and only recompute the missing inner nodes when necessary.
I think the latter is how roughly trillian and diem handle tree storage if I understand correctly. IIRC, diem treats every 4 level binary tree as one inner node and in trillian this is configurable. Both are approaches are probably not only there to decrease storage but also to optimize for IO (matters for large trees).
Tendermint also doesn't store inner nodes for tx data storage at all but still can return inner proof nodes for each tx (always simply recomputes all though). For the state tree though, I do think all inner nodes are stored in a nodedb there. IO is purely tried to be improved via a cache IIRC.
Our decision to store everything is only the fact that this is how the used libraries do things. Which might not necessarily be the best for our use-cases and access-patterns. We don't need to worry much about it right now. But we should keep in mind that this might either be an IO bottleneck in the future, or, cause node operators to complain about disk usage (as far as I remember we store inner nodes for both rows and columns this roughly corresponds to a 4x increase corresponding to the raw data, additionally there might be a lot of padding involved to make the squares a power of two ... so it's probably fair to say on average the increase is about 8-10x the actual block data/txs). I suspect, if we don't make this efficient long-term node operators will simply try to game it and not run the DA part / node at all.
Originally posted by @liamsi in https://github.com/celestiaorg/celestia-node/pull/244#discussion_r761576295

Another example for the approach drafted above is the go transparency log btw (not surprising as it is based on trillian): https://research.swtch.com/tlog#tiling_a_log

A few thoughts:

If we're concerned with disk I/O and CPU usage (which are real costs), I think the best is actually to store the entire branch for each leaf in a single contiguous array. Note that with a 64x64 square, we're looking at 6x32 bytes = 192 bytes per 256-byte share, which is not even doubling storage costs.
If we're concerned with disk storage, then yes the inner nodes can be re-computed on the fly from just leaf nodes.

I imagine validator nodes won't even have to serve historical transactions or shares, so they should do the latter. For storage nodes though, or infrastructure nodes that serve lots of requests, the former is probably better.

Related to https://github.com/celestiaorg/celestia-node/pull/2038

Shwap fixed it


Add support for using toymodel in telescope frame

missing tests

Just because I had the script already from #2347
https://github.com/cta-observatory/ctapipe/assets/5488440/ad475013-53d6-4e23-a765-84db08397f69

Maybe we should just finish removing the CameraFrame versions of all of this... Then we only support TelescopeFrame, and you can transform into CameraFrame to see the result.

Hi @maxnoe can you explain what bits are actually about the new functionality? Most of this seems to be more style refactoring so I'm not sure what bit is where it is important to look closely...

Most of this seems to be more style refactoring so I'm not sure what bit is where it is important to look closely...

Actually, none of this is style refactoring, it's all directly related to support the two frames we can have image parameters in.
In the current main, we use u.quantity_input(x=u.m, ..) to make sure units are valid, however, image parameters can be in either CameraFrame which uses length units on the sensor or TelescopeFrame which uses angles on the sky (in degree).
The units checks prevented the toymodel to be used in the TelescopeFrame, now we support both.


Update descriptions for 2 parameters

@chrisda for review

Learn Build status updates of commit aa03810:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-PolicyConfig.md
:x:Error

Details

exchange/exchange-ps/exchange/index.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Set-PolicyConfig.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/52l1-s/exchange/exchange-ps/exchange/Set-PolicyConfig.md:206:(479) '```
Set-PolicyConfig -EnableLabelCoauth $true -En...'
 Invalid yaml: expected simple key-value pairs, please try to use PlatyPS validate the content

exchange/exchange-ps/exchange/index.yml

Line 119, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-DkimSigningConfig'.
Line 123, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-HostedContentFilterRule'.
Line 124, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-HostedOutboundSpamFilterPolicy'.
Line 129, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-IPBlockListConfig'.
Line 130, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-IPBlockListProvider'.
Line 131, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-IPBlockListProvidersConfig'.
Line 133, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-MalwareFilteringServer'.
Line 134, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-MalwareFilterPolicy'.
Line 135, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-MalwareFilterRule'.
Line 136, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-PhishSimOverridePolicy'.
Line 137, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-PhishSimOverrideRule'.
Line 138, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-QuarantinePolicy'.
Line 139, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-RecipientFilterConfig'.
Line 140, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-SecOpsOverridePolicy'.
Line 141, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-SecOpsOverrideRule'.
Line 142, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-SenderFilterConfig'.
Line 143, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-SenderIdConfig'.
Line 144, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-SenderReputationConfig'.
Line 146, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-TenantAllowBlockListSpoofItems'.
Line 147, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Test-IPAllowListProvider'.
Line 148, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Test-IPBlockListProvider'.
Line 149, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Test-SenderId'.
Line 150, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Update-SafeList'.
Line 179, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ImapSettings'.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit e44850d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Set-PolicyConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Setting user password from secret

Hi,
Is there a good way to define realm's users' password via kubernetes secrets?

Hello @nick4fake,
At the moment, the only way to establish a password is by using the spec.password property. However, we are going to add this feature request to our roadmap. So you will be able to define password from the existing Kubernetes secrets.
Thank you! cc @zmotso  @NikolayMarusenko

@nick4fake This feature is available from the latest release (1.16.0). The release was also published on OperatorHub


Set ensure absent on thomasleese user

I'm leaving GDS. 😢
Reverts https://github.com/alphagov/govuk-puppet/pull/5225 🙂
Trello Card

😿


Use Spotify instead of Rdio

Fix #4228
Since Rdio doesn't work anymore I've chosen to use Spotify here as a music streaming platform.
Attention: I'm not totally confident in these changes however so the PR requires a thorough review.
Description of the change: change Rdio to Spotify
Reason for the change: Rdio is dead
Link to original source: the source code and its corresponding web page

Hello @dagolinuxoid,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Hello @dagolinuxoid,Thanks again for the PR!                                         You have earned 4 out of the 5 points needed to receive glorious SendGrid Hacktoberfest swag.                                         Please take a moment to checkout this link to find more issues to get you past the required threshold or to simply continue the celebration.                                         Also, please be sure you have officially registered with us here by November 1, 2018 to qualify.                                         If you have any questions you can email us at<EMAIL_ADDRESS>                                        Thank you and Happy Hacktobering!                                         Team SendGrid


Invalid pointer free when error occurs inside of callback.msiSetKeyValuePairsToObj

Bug Report
iRODS Version, OS and Version
4.2.11 server, <IP_ADDRESS> plugin
What did you try to do?
Execute a dynamic PEP which calls a microservice from a configured rulebase in PREP (e.g. core.py). The rule looks like this:
def pep_api_touch_pre(rule_args, callback, rei):
    out = callback.msiString2KeyValPair('touch_pre_fired=YES', 0)
    if not out['status']:
        callback.writeLine('stdout', 'ERROR: ' + str(out['code']))
    kvp = out['arguments'][1]
    callback.msiSetKeyValuePairsToObj(kvp, '{0}', '-d') # this should produce an error

This can be triggered by simply executing itouch:
$ itouch foo

Expected behavior
An error code and python stacktrace in the log, like this:
Dec 21 17:00:21 pid:3215 NOTICE: checkAndGetObjectId cmlExecuteNoAnswerSql(rollback) succeeded
Dec 21 17:00:21 pid:3215 NOTICE: rsModAVUMetadata: rcModAVUMetadata failed
Dec 21 17:00:21 pid:3215 remote addresses: <IP_ADDRESS>, <IP_ADDRESS> ERROR: caught python exception: Traceback (most recent call last):
  File "/etc/irods/core.py", line 337, in pep_api_touch_pre
    callback.msiSetKeyValuePairsToObj(kvp, '{0}', '-d')
RuntimeError: [iRods__Error__Code:-817000] [-]  /irods_source/server/re/include/irods_re_plugin.hpp:326:irods::error irods::dynamic_operation_execution_manager<std::__1::tuple<>, RuleExecInfo *, irods::DONT_AUDIT_RULE>::call(std::string, std::string, OP, As &&...) [T = std::__1::tuple<>, C = RuleExecInfo *, Audit = irods::DONT_AUDIT_RULE, OP = std::__1::function<irods::error (const std::__1::basic_string<char> &, RuleExecInfo *&, irods::unpack &&)>, As = <const std::__1::basic_string<char> &, RuleExecInfo *&, irods::unpack>] :  status [CAT_UNKNOWN_FILE]  errno [] -- message [exec_microservice_adapter failed]
        [-]     /irods_source/server/re/src/irods_re_plugin.cpp:132:irods::error irods::default_microservice_manager<RuleExecInfo *>::exec_microservice_adapter(std::string, irods::default_ms_ctx, std::list<boost::any> &) :  status [CAT_UNKNOWN_FILE]  errno [] -- message [exec_microservice_adapter failed]

Observed behavior (including steps to reproduce, if applicable)
An invalid pointer free kills the connected agent. A sample of the output in the log:
Dec 21 17:29:21 pid:3419 NOTICE: rsModAVUMetadata: rcModAVUMetadata failed
free(): invalid pointer
Caught signal [6]. Dumping stacktrace and exiting

Dumping stack trace
<0>     Offset:         Address: 0x7fe183b8c9b8 /usr/lib/libirods_server.so.4.2.11(+0x15149b8) [0x7fe183b8c9b8]
<1>     Offset:         Address: 0x7fe18095a980 /lib/x86_64-linux-gnu/libpthread.so.0(+0x12980) [0x7fe18095a980]
<2>     Offset:         Address: 0x7fe180595fb7 /lib/x86_64-linux-gnu/libc.so.6(gsignal+0xc7) [0x7fe180595fb7]
<3>     Offset:         Address: 0x7fe180597921 /lib/x86_64-linux-gnu/libc.so.6(abort+0x141) [0x7fe180597921]
<4>     Offset:         Address: 0x7fe1805e0967 /lib/x86_64-linux-gnu/libc.so.6(+0x89967) [0x7fe1805e0967]
<5>     Offset:         Address: 0x7fe1805e79da /lib/x86_64-linux-gnu/libc.so.6(+0x909da) [0x7fe1805e79da]
<6>     Offset:         Address: 0x7fe1805eef0c /lib/x86_64-linux-gnu/libc.so.6(cfree+0x4dc) [0x7fe1805eef0c]
<7>     Offset:         Address: 0x7fe184f82222 /usr/lib/libirods_common.so.4.2.11(clearMsParam+0x92) [0x7fe184f82222]
<8>     Offset:         Address: 0x7fe17ce163b4 /usr/lib/irods/plugins/rule_engines/libirods_rule_engine_plugin-python.so(+0x75f3b4) [0x7fe17ce163b4]
<9>     Offset: 0x76    Address: 0x7fe17ce2c206 boost::python::detail::raw_dispatcher<boost::python::dict (*)(boost::python::tuple const&, boost::python::dict const&)>::operator()(_object*, _object*)
<10>    Offset: 0x34    Address: 0x7fe17ce2c104 boost::python::objects::full_py_function_impl<boost::python::detail::raw_dispatcher<boost::python::dict (*)(boost::python::tuple const&, boost::python::dict const&)>, boost::mpl::vector1<_object*> >::operator()(_object*, _object*)
<11>    Offset: 0x2fb   Address: 0x7fe17c48f85b boost::python::objects::function::call(_object*, _object*) const
<12>    Offset:         Address: 0x7fe17c491ad5 /opt/irods-externals/boost1.67.0-0/lib/libboost_python27.so.1.67.0(+0x25ad5) [0x7fe17c491ad5]
<13>    Offset: 0x47    Address: 0x7fe17c498e87 boost::python::handle_exception_impl(boost::function0<void>)
<14>    Offset:         Address: 0x7fe17c491766 /opt/irods-externals/boost1.67.0-0/lib/libboost_python27.so.1.67.0(+0x25766) [0x7fe17c491766]
<15>    Offset:         Address: 0x7fe17c063913 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyObject_Call+0x43) [0x7fe17c063913]
<16>    Offset:         Address: 0x7fe17c090eec /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(+0x1a1eec) [0x7fe17c090eec]
<17>    Offset:         Address: 0x7fe17c063913 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyObject_Call+0x43) [0x7fe17c063913]
<18>    Offset:         Address: 0x7fe17c00f31a /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(+0x12031a) [0x7fe17c00f31a]
<19>    Offset:         Address: 0x7fe17c063913 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyObject_Call+0x43) [0x7fe17c063913]
<20>    Offset:         Address: 0x7fe17bf8d951 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyEval_EvalFrameEx+0x41e1) [0x7fe17bf8d951]
<21>    Offset:         Address: 0x7fe17c0ce908 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyEval_EvalCodeEx+0x7d8) [0x7fe17c0ce908]
<22>    Offset:         Address: 0x7fe17c069649 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(+0x17a649) [0x7fe17c069649]
<23>    Offset:         Address: 0x7fe17c063913 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyObject_Call+0x43) [0x7fe17c063913]
<24>    Offset:         Address: 0x7fe17c0cde37 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyEval_CallObjectWithKeywords+0x47) [0x7fe17c0cde37]
<25>    Offset:         Address: 0x7fe17c03c381 /usr/lib/x86_64-linux-gnu/libpython2.7.so.1.0(PyEval_CallFunction+0xa1) [0x7fe17c03c381]
<26>    Offset:         Address: 0x7fe17ce1caf4 /usr/lib/irods/plugins/rule_engines/libirods_rule_engine_plugin-python.so(+0x765af4) [0x7fe17ce1caf4]
<27>    Offset:         Address: 0x7fe17ce0e1ca /usr/lib/irods/plugins/rule_engines/libirods_rule_engine_plugin-python.so(+0x7571ca) [0x7fe17ce0e1ca]
<28>    Offset: 0x834   Address: 0x7fe17ce0bb14 exec_rule(std::__1::tuple<> const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, std::__1::list<boost::any, std::__1::allocator<boost::any> >&, irods::callback)
<29>    Offset: 0x141   Address: 0x7fe17ced1b21 irods::error std::__1::__invoke_void_return_wrapper<irods::error>::__call<irods::error (*&)(std::__1::tuple<> const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, std::__1::list<boost::any, std::__1::allocator<boost::any> >&, irods::callback), std::__1::tuple<>&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, std::__1::list<boost::any, std::__1::allocator<boost::any> >&, irods::callback>(irods::error (*&&&)(std::__1::tuple<> const&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, std::__1::list<boost::any, std::__1::allocator<boost::any> >&, irods::callback), std::__1::tuple<>&&&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&&&, std::__1::list<boost::any, std::__1::allocator<boost::any> >&&&, irods::callback&&)

<snip>

See the attached log for the full output: prep_invalid_pointer_free.log.txt
Further investigation will be needed, but it is definitely related to fixes introduced for leaks occurring which were discovered in irods/irods#4649.
In particular, the line causing the invalid pointer free - seen in frame <7> - is here, added in <IP_ADDRESS>: https://github.com/irods/irods_rule_engine_plugin_python/blob/785b279cfc6e44d40b31c7bba78d2fe8593731a2/irods_rule_engine_plugin-python.cxx#L278
The issue seems to only manifest itself when a python exception is being thrown (notice frame <13>). When a proper data object path is given to the microservice, an error is not generated and the crash does not occur.

The clearMsParam() fixes the 32k leak - so we want to keep it.  It allows for multiple rules to be run within the same connection without making the server leak until collapse.
So, wrapping line 278 in some logic to detect whether we're already 'in an error case' would prevent this pointer free.
More investigation to whether that means that a leak would still occur, but maybe that doesn't matter since it's already in an exception case.
Another consideration is when !retVal.ok() but when not caused by an exception (just a regular iRODS error code).
If an exception is thrown, but caught and handled in the rule itself... how would this behave?
@stsnel, I think we can get this done pretty quickly for a <IP_ADDRESS>.

@trel Thank you, that sounds like a great solution 👍

We have confirmed that the solution works and prevents leaks for non-error cases.
I did notice that some microservices still leak and we will try to address this in the next server release in addition to the other observed leaks. See irods/irods#6044 and irods/irods#6108


Add the docs/resources/_gen folder to gitignore

Seems like running Hugo is causing some compiled assets to diff:
	modified:   docs/resources/_gen/assets/scss/src/sass/kube.scss_48b060fe05b0a273d182ef83c0605941.content
	modified:   docs/resources/_gen/assets/scss/src/sass/kube.scss_48b060fe05b0a273d182ef83c0605941.json

Need to ignore these or flag them as binaries.

This is solved by https://github.com/patternfly/patternfly-elements/pull/373