Validation: Fixed Absolute Links and Duplicate Descriptions

This PR fixes 33 issues in 20 files.

Docs Build status updates of commit 36bdf97:
:warning: Validation status: warnings

File
Status
Preview URL
Details

api/overview/Library-Reference/pickerdialog-members-office.md
:warning:Warning
View
Details

api/Outlook.ContactItem.Close(even).md
:white_check_mark:Succeeded
View

api/Outlook.ExchangeUser.GetPicture.md
:white_check_mark:Succeeded
View

api/Outlook.Folder.GetCustomIcon.md
:white_check_mark:Succeeded
View

api/Outlook.OlAttachmentBlockLevel.md
:white_check_mark:Succeeded
View

api/overview/Access.md
:white_check_mark:Succeeded
View

api/overview/Access/object-model.md
:white_check_mark:Succeeded
View

api/overview/Excel.md
:white_check_mark:Succeeded
View

api/overview/Excel/object-model.md
:white_check_mark:Succeeded
View

api/overview/Language-Reference.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverterapplicationpreferences-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverter-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverterpreferences-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverteruicallback-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/labelinfo-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/reference-object-library-reference-for-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/textframe2-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/textrange2-members-office.md
:white_check_mark:Succeeded
View

api/overview/Outlook.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/pickerdialog-members-office.md

Line 35, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'f/dotnet/api/microsoft.office.core.pickerdialog'.

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 Docs 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 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 docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5a2059a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Outlook.ContactItem.Close(even).md
:white_check_mark:Succeeded
View

api/Outlook.ExchangeUser.GetPicture.md
:white_check_mark:Succeeded
View

api/Outlook.Folder.GetCustomIcon.md
:white_check_mark:Succeeded
View

api/Outlook.OlAttachmentBlockLevel.md
:white_check_mark:Succeeded
View

api/overview/Access.md
:white_check_mark:Succeeded
View

api/overview/Access/object-model.md
:white_check_mark:Succeeded
View

api/overview/Excel.md
:white_check_mark:Succeeded
View

api/overview/Excel/object-model.md
:white_check_mark:Succeeded
View

api/overview/Language-Reference.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverterapplicationpreferences-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverter-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverterpreferences-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/iconverteruicallback-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/labelinfo-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/pickerdialog-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/reference-object-library-reference-for-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/textframe2-members-office.md
:white_check_mark:Succeeded
View

api/overview/Library-Reference/textrange2-members-office.md
:white_check_mark:Succeeded
View

api/overview/Outlook.md
:white_check_mark:Succeeded
View

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


Invalid session due to incorrect API marks the order as complete

Describe the bug
When an order is placed using thawani and there is an issue with the API credentials. The order is being marked as processing instead of on hold.
To Reproduce
Steps to reproduce the behavior:
Placing an order with an invalid API credential
Expected behavior
Order should be kept on hold and the thawarni session log should display payment error

hey @PupaClic ..
I'll keep WooCommerce manage order page as a reference for why we are using failed status and pending status when the payment is created!
On-hold is used for offline payment ( most of the e-commerce system use this status when the order is created).
When the order is being created
The  default status will be pending and not processing.
if the credentials are incorrect  then the payment status will be failed! and the reason is the way Thawani Pay integration works.

Hey @PupaClic is your question is answered ?


Fix dark theme

Probably add .text-muted to delete/restore transaction buttons
Change background color of participants list to some variable, like color-primary or so. See paperCSS for details

Funding

You can sponsor this specific effort via a Polar.sh pledge below
We receive the pledge once the issue is completed & verified

fixed in #76


fix bug in CopyRequest & CopyResponse

Codecov Report

Merging #648 (ee3a5fc) into pipeline (e409b6e) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           pipeline     #648      +/-   ##
============================================
- Coverage     79.97%   79.91%   -0.06%
============================================
  Files            94       94
  Lines         10378    10378
============================================
- Hits           8300     8294       -6
- Misses         1599     1603       +4
- Partials        479      481       +2

Impacted Files
Coverage Δ

pkg/cluster/cluster.go
59.62% <0.00%> (-0.94%)
:arrow_down:

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 e409b6e...ee3a5fc. Read the comment docs.


Review system architecture for existing Blockchain ETL projects.

Reviewing the system architecture for existing Blockchain ETL projects is an essential first step in the Stellar ETL project. This will build an intuition for the system as a whole, help understand how the various pieces fit together, and contribute to the design of a minimal yet effective command-line tool.
The necessary information is in section 2 here.

Finished and discussed on Wed 6/23, but retroactively added to account for Isaiah's work.


Suggestion for on-responded on prompts

Fixes #3947
Description
On issue #3947, relying on the responded flag todecide whether to show the prompt interferes with logic that could be happening either
a. Concurrently
b. On some middleware
c. As part of how we reached the prompt
I believe the intention is to prevent sending a prompt twice to the user, using a symbol may be enough for that, without relying on global status
Specific Changes

Instead of using the "has responded" global flag, create one just for the prompt. My suggestion is per-prompt, but it could be made per all prompts.

Testing
I don't believe this adds a new feature to the library

Notice how I don't expect this to be merged as-is, but I hope to instead bring a conversation on how we can address #3947 while keeping as many of the design decisions made as possible.


wasm-themis: TypeError: TextEncoder is not a constructor | Webpack

Describe the bug
When importing wasm-themis in a webpack project I get the following error:

I think this is because of an incorrect usage of the fast textencoder api: https://github.com/anonyco/FastestSmallestTextEncoderDecoder#api-documentation
To Reproduce
import * as themis from "wasm-themis";
themis.initialized.then(function () {
//
// Now you can use "themis" functions
//
});

Expected behavior
The polyfill should be used and no exception thrown.
Environment (please complete the following information):

OS: Firefox
wasm Themis version: 0.13.1
Installation way:

[x] via package manager
[ ] built from source

Hi @maxammann! Thank you for letting us know.
We are currently testing wasm-themis only in nodejs environment, and haven't seen this error before.
As I understood from docs, the fix might as easy as changing the utils.js#L38 line from
const utf8Encoder = new TextEncoder()

to
const utf8Encoder = (new TextEncoder).encode;

(I'm not sure, but potentially we might also need to change the import line #L21 from const {TextEncoder} = require('fastestsmallesttextencoderdecoder') to const {TextEncoder, TextDecoder} = require('fastestsmallesttextencoderdecoder')).
However, we are not experienced with webpack, @maxammann would you mind creating a small reproducible project with Themis import? It would help us to test the fix faster.
I also wanted to suggest you to try to change this line inside node_modules themis code to make sure that the fix works, but I've read this article "I want to change the code in my node_modules folder" that explains that webpack is not okay with manually tweaking node_modules even for testing.

I'm already using patch-package (https://www.npmjs.com/package/patch-package) which would make patching it trivially. I already tried yesterday to exclude that polyfill which works fine.
As we are an open source project you can checkout this project and change into the web directory: https://github.com/cyla-app/cyla-app/tree/main/web
From there run:
yarn and yarn start.
Would you be interrested in a PR with a better packaging for web for this library? There are actually different "bundle formats" which can be targeted like commonjs, umd, es6 etc which can be quite confusing.

I already tried yesterday to exclude that polyfill which works fine.

Sounds amazing! You mean, that you already patched that line in wasm-themis for your local build and it works fine?
Can you please create a PR with this fix (for master branch), so we can make sure that other tests are passing, and merge it? 🥺👉👈

Would you be interrested in a PR with a better packaging for web for this library

Of course, we would :) We'd be forever grateful for this, and add your name to the list of contributors and Hall of Fame :)

I fixed the TextEncoder issue. In the long run we should package the node library using webpack or rollup for better comparability.
I also suggest a TypeScript rewrite and then bundling it for node and browsers.

Oh great, GitHub automatically closed the issue before the fix has been made available.
@maxammann would you be okay with this fix released in the upcoming 0.14 (some undefined time in the future), or you need to unbreak stuff soon(ish) and would prefer some 0.13.2 to be made available?

@ilammy Its fine if it gets released in the undefined future. Right now I plan to use a custom build for the near future as there are maybe some other things I will need to change.
Did you already try whether this package works using webpack in a browser? I did not manage to load the .wasm correctly. I also do not see a way of doing it right now. I suspect that were wasn't much testing with webpack (which is fine).
I plan to do a PR as soon as I manage to get it working in the browser (while keeping node suppport). Is that fine?
And sorry for the amount of communication right now :) but I really want to get this working for our university project :)

Okay. So I guess “fixes for WasmThemis in browser” will be one of the highlights for 0.14, @vixentael.

Did you already try whether this package works using webpack in a browser?

Not yet :(

I suspect that were wasn't much testing with webpack

You're right.
WasmThemis is known to be working with Node. Stable versions certainly work with Electron (and is probably working on master). But as for browsers... Let me put it like this: I think I remember that someone once told me that at some point they saw WasmThemis working in some browser.
It's not tested on CI. It's not tested manually during the release (as far as I know). I wouldn't be surprised if it does not work at all, contrary to what documentation claims.

I plan to do a PR as soon as I manage to get it working in the browser (while keeping node suppport). Is that fine?

It's not only fine – it would be goddamn amazing ❤ (if you ask me).
Unfortunately, I personally don't really have the capacity at the moment for more than keeping WasmThemis on life support. Neither do other maintainers, I believe. So any help here would be much appreciated.
If you fancy a TypeScript rewrite some time in the future then you have a green light from me on that. I believe that would be a valuable addition to WasmThemis.

“fixes for WasmThemis in browser” will be one of the highlights for 0.14

Makes sense for me 💪

I plan to do a PR as soon as I manage to get it working in the browser (while keeping node suppport). Is that fine?

Thank you @maxammann :) Looking forward!

It's not only fine – it would be goddamn amazing heart (if you ask me).

Thank you @maxammann :) Looking forward!

I just got it working with small changes. I copied the .js files to my project for now. There I will transfer it to TypeScript and fix issues. Afterwards I'll contribute it upstream by adding proper bundling.
I added this to the makefile which makes it easier to include the wasm lib:
$(BIN_PATH)/libthemis.js: LDFLAGS += -s MODULARIZE=1

Alright, I did the TypeScript migration: https://github.com/cyla-app/cyla-app/tree/main/web/src/themis
Next step would be to run the old tests against the new bundled version :)

https://github.com/cossacklabs/themis/pull/792

I suggest to close this, as no discussions is happening right now, see #792


Merge to publish

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/add-and-remove-attachments-to-an-item-in-a-compose-form.md
:white_check_mark:Succeeded

docs/add-ins/add-in-commands-for-outlook.md
:white_check_mark:Succeeded

docs/add-ins/addin-tutorial.md
:white_check_mark:Succeeded

docs/add-ins/compare-outlook-add-in-support-in-outlook-for-mac.md
:white_check_mark:Succeeded

docs/add-ins/images/quick-start-button.PNG
:white_check_mark:Succeeded

docs/add-ins/images/quick-start-task-pane.PNG
:white_check_mark:Succeeded

docs/add-ins/images/quick-start-yo-prompts.PNG
:white_check_mark:Succeeded

docs/add-ins/index.md
:white_check_mark:Succeeded

docs/add-ins/limits-for-activation-and-javascript-api-for-outlook-add-ins.md
:white_check_mark:Succeeded

docs/add-ins/outlook-mobile-addins.md
:white_check_mark:Succeeded

docs/add-ins/outlook-on-send-addins.md
:white_check_mark:Succeeded

docs/add-ins/quick-start.md
:white_check_mark:Succeeded

docs/add-ins/tips-for-handling-date-values-in-outlook-add-ins.md
:white_check_mark:Succeeded

docs/add-ins/toc.yml
:white_check_mark:Succeeded

docs/add-ins/troubleshoot-outlook-add-in-activation.md
:white_check_mark:Succeeded

docs/add-ins/use-regular-expressions-to-show-an-outlook-add-in.md
:white_check_mark:Succeeded

docs/breadcrumb/toc.yml
:white_check_mark:Succeeded

docs/rest/javascript-tutorial.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ngspice 36 moved to old-releases/36

In iic-osic-setup.sh, the folder old-releases/ must be inserted in the download path for ngspice 36, i.e, the following works:
https://sourceforge.net/projects/ngspice/files/ng-spice-rework/old-releases/36/ngspice-36.tar.gz

Resolved by updating ngspice to latest version 42.


API

Is your feature request related to a problem? Please describe.
大佬，咱们这个wallet-core有api文档吗？
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

See https://developer.trustwallet.com/wallet-core and unit tests in this repo


Change the path sameness check in FalcorController to be deep quality

I think when it compares paths right now, its a shallow comparison. There's a deep equals function already built into BaseComponent, we just need to expose it.

Nothing to see here, understood _.isEqual incorrectly.


umami api client test ko with docker umami version 2.13.3

Hi,
https://github.com/boly38/action-umami-report/issues/59
I'm trying to play basis example  (ex. I first would like to list websites)

server with umami docker self-hosted
self-hosted umami version 2.13.3

env
export UMAMI_API_CLIENT_USER_ID=admin
export UMAMI_API_CLIENT_PASSWORD=xxx-replace-by-admin-password-here
export UMAMI_API_CLIENT_SECRET=xxx-replace-by-app-secret-set-in-docker-compose
export UMAMI_API_CLIENT_ENDPOINT=https://myumami.example.fr/api

node
import {getClient} from '@umami/api-client';

const umamiUsername = process.env.UMAMI_API_CLIENT_USER_ID;
const umamiPassword = process.env.UMAMI_API_CLIENT_PASSWORD;
let client;
const getMeFixed = false;
{
    client = getClient();
    const {ok, data, status, error} = await client.login(umamiUsername, umamiPassword);
    console.log(`login: ${JSON.stringify({ok, data, status, error}, null, 2)}`);
}
if (getMeFixed) {
    const {ok, data, status, error} = await client.getMe();
    console.log(`getMe: ${JSON.stringify({ok, data, status, error}, null, 2)}`);
}
{
    const {ok, data, status, error} = await client.getWebsites();
    console.log(`getWebsites: ${JSON.stringify({ok, data, status, error}, null, 2)}`);
}

result
login: {
  "ok": true,
  "data": {
    "token": "==REDATED==",
    "user": {
      "id": "==REDATED==",
      "username": "admin",
      "role": "admin",
      "createdAt": "2024-07-12T15:52:00.340Z",
      "isAdmin": true
    }
  },
  "status": 200
}
getWebsites: {
  "ok": false,
  "status": 504,
  "error": "\r\n<title>504 Gateway Time-out</title>\r\n\r\n
504 Gateway Time-out
\r\n
nginx\r\n\r\n\r\n"
}

but I got 504 error with server side error logs :
PrismaClientKnownRequestError:
Invalid `prisma.user.findUnique()` invocation:

Inconsistent column data: Error creating UUID, invalid character: expected an optional prefix of `urn:uuid:` followed by [0-9a-fA-F-], found `m` at 3
    at _n.handleRequestError (/app/node_modules/@prisma/client/runtime/library.js:121:7749)
    at _n.handleAndLogRequestError (/app/node_modules/@prisma/client/runtime/library.js:121:7057)
    at _n.request (/app/node_modules/@prisma/client/runtime/library.js:121:6741)
    at async l (/app/node_modules/@prisma/client/runtime/library.js:130:9355)
    at async /app/.next/server/chunks/405.js:1:17597 {
  code: 'P2023',
  clientVersion: '5.17.0',
  meta: {
    modelName: 'User',
    message: 'Error creating UUID, invalid character: expected an optional prefix of `urn:uuid:` followed by [0-9a-fA-F-], found `m` at 3'
  }
}

NB: it was the same kind of error for getMe use case.
this issue and error logs are close to umami-software/umami/issues/1964
I just try to create a !admin new user and switch env to use this new creds, login is ok but I got same issue for the getWebsites step .. :'(
any help / hint appreciated
regards

uin pgsql db I didnt find any ``urn:uuid:`


Update USE_CASES.md to include Attachments

Reference Issue #347

Thanks again for your contribution!
When you have a moment, please fill out this form so we can send you some swag. Thanks!


feat: fee estimation for forc-deploy and point --testnet to devnet

Description
Estimation is hard to test as it is dependent on the network configuration but this is nearly the same thing we have been doing for scripts.

--testnet points to devnet.
For the new max_fee requirement, adds an estimation phase with dry runing the transaction
Various consts are pointed to relevant devnet endpoints.

Most visible outcome is that forc-deploy --testnet can deploy to devnet without any other flags etc.

Requesting compiler review as well as this is blocking the release.


[v1.15] author backport: lrp: update LRP services with stale backends on agent restart

This PR is an author backport for https://github.com/cilium/cilium/pull/36036

/test-backport-1.15


Incorrect port for BluetoothListener.win32

This was the underlying issue for #99. According to the docs, for Bluetooth server applications calling bind (i.e., a Bluetooth listener), "the port member must be a valid port number or BT_PORT_ANY". Since the port is being set to the default (0, which is fine for clients), attempting to receive a connection via accept will never return.
Setting the port to BT_PORT_ANY in BluetoothListener.win32 (here) resolves the issue.

Fixed in 4.0.15


Improve the usage of advanced features around Digital Ocean

Is your feature request related to a problem? Please describe.
A user in slack indicated that they wanted to use Digital Ocean's own cloud provider so that they could make use of the available load balancer. The conversation seems to indicate that others have had this desire and that the solution is involved and error prone.
Here's the conversation https://rancher-users.slack.com/archives/C3ASABBD1/p1677662554917629
Describe the solution you'd like
If there's enough demand I think it might be useful to make this particular workflow easier to work with within rancher.

Closing due to traction.


Merge develop to main for CI publish (asset service)

Merge develop to main for CI publish (asset service)

I'm not sure how to do git flow on a monorepo since every package has a different version number. Normally we'd cut a release branch and make the PR from the release branch into main so people can continue to push to develop without affecting what gets merged into main.


Add examples in readme file

Show what features it has in readme file with examples.

I've added some examples to the readme. Do you have suggestions for other concrete examples we should add there?


[Feature Request:] Service map based on hostnames or host headers

Contact Details
<EMAIL_ADDRESS>Is your feature request related to a problem? Please describe.
I'm currently looking at a few captures and the service map is not extremely useful, because most of the traffic seems to be shown with ips.
It makes for a more complex graph than it needs to be (especially if you run more than 1 pod per service, for replication)
I'd prefer to "group" the graph by using the hostnames, either by the DNS resolution, or maybe the http Host header ?
Original Thread
No response
Describe the solution you'd like to see
A way to change the service map mode to use a header-based or dns name, something like that ? Even making it customizable by selecting a header, if you're using an API gateway it might help a lot to remove some weird mapping issues.
Provide additional context
No response
Code of Conduct

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

@Gby56 can you check which version you are using? Over the weekend, we added DNS support.  DNS functionality is supported starting at V38.0. Can you check the latest release and let us know what you think, and what you think is missing.

@Gby56 if this topic is still of interest, please check out the latest release as we completely changed the service map since this issue was brought up. Specifically, we added a lot more identity related information


htaccess file / shared hosting install

Hi There
First off thanks a million for creating this a hugely beneficial tool and I would have to agree that RSS / Atom feeds should be open to all outside the "Social" walled gardens.
TBH I think this is more a question than an issue but im having difficulty running this on a shared web hosting service.
I have downloaded and run the RSS-bridge on my local MAMP stack and it works a treat however when I try to upload to hosting provider it 500 errors out
I have created a .htaccess file and added the following lines
AddHandler application/x-httpd-php56.php php_value allow_url_fopen On
However it errors out. I also created a php.ini file and added it to the root folder with the .htaccess file containing the following code
allow_url_fopen = On;
I get the following error in my log file

.htaccess: Invalid command 'php_value', perhaps misspelled or defined by a module not included in the server configuration

and

AddHandler requires at least two arguments, a handler name followed by one or more file extensions

Is this an issue with the code im using in the .htaccess file or something to do with the shared hosting im using?
For now I have deleted it off my Shared hosting and using it quite happily on my local MAMP :-)
Thanks a million for your help in advance and feel free to delete / remove this as an issue if its not relevent

Closing issue hosting provider does not allow fopen to be set to 1
"allow_url_fopen" is not set to "1". Please check "php.ini


fix(bug97): modify setAlwaysOnTop() to somehow allow theme switches

How is it connected?
Anyway, closes #97

I'll add styling here?

Nice idea!

And also, believe it or not, this was done on my Raspberry Pi and MobaXTerm, so I can't really guarantee if it works on Windows.

Also, formatted files, so plus point?

Also, I have a Twitter too, so I could promote TimerX. Hopefully, someone might look at this. Wy not enable the blog part on Docusaurus so that we could post changelogs and some more things to users?

If you allow me to post on Twitter, I can post only during the evening, school.

its fine if you promote the app on twitter

Hmm, then it will be posted during the evening since I'll be back from school by then. What do you think of a Slack or Discord community? I think it's cool here.

Hmm, then it will be posted during the evening since I'll be back from school by then. What do you think of a Slack or Discord community? I think it's cool here.

Discord sounds good i think, but i dont have it. Still think it would be the best option.

I will return to discord … someday … pls dont ask about it ok?

Then I'll definitely ask about it. Human psychology?

My parents dont like it when i talk to random ppl on the internet and thats why i dont have discord. They also dont know that i talk to you guys.

I will return to discord someday, when they cant tell me what to do anymore. In 5 years.

oh... ok np

Even my parents don't know I talk to you all 😜 guess we are on the same coin.

Well, but my parents locked down my internet on my pc (and disabled the browser on my phone) and they monitor my phone. So i cant go on discord. Your parents atöeast don't monitor your pc and disable your internet. You can do what you want. I cant. :(

Hope you break that barrier one day tho...

Well, my parents don’t know that I talk to you guys as well, so…

If you don’t have internet how do you access GitHub?

If you don’t have internet how do you access GitHub?

Downloading Chrome on my phone for every commit.

And for talking i use github mobile

Hope you break that barrier one day tho...

Me too :(

anyway, when r you starting work on calcx? After 1.1 there isnt much we can do for tiimerx

how does adding packages on Pypi wotk because i want to make a module which automaticallay makes the windw svttk And adds mica and forces dark titlebar and binds unmaximize to redraw titlebar

how does adding packages on Pypi wotk because i want to make a module which automaticallay makes the windw svttk And adds mica and forces dark titlebar and binds unmaximize to redraw titlebar

If you want, you can add it to ntkutils! It already has dark titlebar forcing and all the packaging is done! (And it needs more features lol) If you dont, i can set it up for you, just give me your source code!

Great!


Instance does not initialize to the correct todo type type after restart

Describe the bug
My second instance is set to "TODO" but always apears as "Calendar" (actually broken without any entries). Checking/Clicking the already checked TODO item manually fixes the problem.
To Reproduce
Secondary instance with TODOs. Force restart (kill, start)
Expected behavior
Initializes as correctview type

Windows Version
Windows 11 23H2
Additional context and log file
2024-01-15 08:52:53.5252 INFO Command Line Args:
2024-01-15 08:52:53.6050 INFO Found 15.0 key
2024-01-15 08:52:53.6050 INFO Found 16.0 key
2024-01-15 08:52:53.6050 INFO Found C2RSvcMgr key
2024-01-15 08:52:53.6050 INFO Found ClickToRun key
2024-01-15 08:52:53.6050 INFO Found Common key
2024-01-15 08:52:53.6050 INFO Found DmsClient key
2024-01-15 08:52:53.6050 INFO Found Outlook key
2024-01-15 08:52:53.6050 INFO Office version 16 detected
2024-01-15 08:52:53.6050 INFO Outlook path reported as C:\Program Files\Microsoft Office\Root\Office16\ and Outlook.exe found in that path
2024-01-15 08:52:53.6050 INFO Outlook Bitness is: x64
2024-01-15 08:52:54.4263 DEBUG Checking to see if there is an instance running.
2024-01-15 08:52:55.4410 INFO Starting the instance manager and loading instances.
2024-01-15 08:52:56.0920 DEBUG Loading app settings from registry
2024-01-15 08:52:56.0920 DEBUG Settings Found.
2024-01-15 08:52:56.0920 DEBUG Multiple instances to load
2024-01-15 08:52:56.3329 DEBUG Instantiating instance Default Instance
2024-01-15 08:52:56.7276 INFO Outlook folder path: IPM.Appointment
2024-01-15 08:52:57.0429 DEBUG Showing Instance Default Instance
2024-01-15 08:52:57.0645 DEBUG Instantiating instance TODOs
2024-01-15 08:52:57.3259 INFO Outlook folder path: IPM.Task
2024-01-15 08:53:00.2948 DEBUG Showing Instance TODOs

Thanks for the report! I'm not able to reproduce. Does that only happen with todo's? Do any of the other views stick?

It happens with views other than the calendar from what I tested.


Iphone Flashing

Is there any reason why, after developing it locally, on iPhone, both 6s and 10, the screen is flashing?

My bad, I was implementing a framework with pub.js. I got confused, I will delete this topic. Sorry

I don't think that I can actually cancel the issue. Could you do it for me? In the meantime, I will take advantage to ask you a question. I've never used golang, and I was trying to compile the code somewhere locally with a tool like node.js. Do you think it is possible to compile a code locally, using html, javascript and css using go and a compiler? Or could you explain to me how? Thank you for your time

@orso081980 It's fine.
As for compiling a golang app, you basically just download golang, set up the GOPATH environment variable, then go get the project to compile, or go build a local one.

Could you be interested in doing some freelance job for my company? We are struggling in doing a project and we need a reliable reader for epub files. Bookbrowser is a fantastic solution but so far we haven't figured it out how to use it. We could be more than happy to hire you in helping us with this project.
Thank you in advance, we're looking forward to hearing from you
Get Outlook for Androidhttps://aka.ms/ghei36

From: Patrick G<EMAIL_ADDRESS>Sent: Monday, March 5, 2018 3:36:52 PM
To: geek1011/BookBrowser
Cc: Marco; Mention
Subject: Re: [geek1011/BookBrowser] Iphone Flashing (#39)
@orso081980https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Forso081980&data=02|01||07bb1e62a57d40908de908d582d8d5d9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636558790160449958&sdata=YVup5JiWa8GZatXXwPSV37SlDolYT542DHMzWXMavPQ%3D&reserved=0 It's fine.
As for compiling a golang app, you basically just download golang, set up the GOPATH environment variable, then go get the project to compile, or go build a local one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgeek1011%2FBookBrowser%2Fissues%2F39%23issuecomment-370556261&data=02|01||07bb1e62a57d40908de908d582d8d5d9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636558790160606232&sdata=DBVs52jxNvicbcdALARRID1nVQ7y5L%2Ff1zXk2YZoPYo%3D&reserved=0, or mute the threadhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHytmoZ5DFZBbi9RjRmNIhrelG-39AIqks5tbaHkgaJpZM4SW7jt&data=02|01||07bb1e62a57d40908de908d582d8d5d9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636558790160606232&sdata=gGKaYflb1d5Gmu00EWz6%2F1oEaTOlTLP%2FjjNYSiN9Z44%3D&reserved=0.

Hey Patrick. How are you? What Do you think about my offer? Your reader is fantastic and we are really interested in your paid service. Please come back whenever you've got time. Have a good day
Get Outlook for Androidhttps://aka.ms/ghei36

From: Patrick G<EMAIL_ADDRESS>Sent: Monday, March 5, 2018 3:36:52 PM
To: geek1011/BookBrowser
Cc: Marco; Mention
Subject: Re: [geek1011/BookBrowser] Iphone Flashing (#39)
@orso081980https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Forso081980&data=02|01||07bb1e62a57d40908de908d582d8d5d9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636558790160449958&sdata=YVup5JiWa8GZatXXwPSV37SlDolYT542DHMzWXMavPQ%3D&reserved=0 It's fine.
As for compiling a golang app, you basically just download golang, set up the GOPATH environment variable, then go get the project to compile, or go build a local one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgeek1011%2FBookBrowser%2Fissues%2F39%23issuecomment-370556261&data=02|01||07bb1e62a57d40908de908d582d8d5d9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636558790160606232&sdata=DBVs52jxNvicbcdALARRID1nVQ7y5L%2Ff1zXk2YZoPYo%3D&reserved=0, or mute the threadhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHytmoZ5DFZBbi9RjRmNIhrelG-39AIqks5tbaHkgaJpZM4SW7jt&data=02|01||07bb1e62a57d40908de908d582d8d5d9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636558790160606232&sdata=gGKaYflb1d5Gmu00EWz6%2F1oEaTOlTLP%2FjjNYSiN9Z44%3D&reserved=0.


Collapse show property is wrong

The collapse shows when the property expression is false

Please post a link to a reproduction in Plunker.

you can see it in Plunker preview used for collapse, the original value of the property is false, which is making the collapse to show on false, this could make some confusions when creating expressions for collapse

So, let's say what the dictionary says about collapse:

Fold or be foldable into a small space

So to collapse something, means that you fold it, in bootstrap terms, you make it disappear.

So in this case, is this folded? False, it is not collapsed.
Then you click it:

Is this now folded? True, it is collapsed.
So TL;DR this works as intended, a false expression means that the content is shown and true means that it is collapsed. Think about this like ng-hide. If true it hides, if false it shows.

This is also consistent with the TWBS collapse plugin, the presence of the collapse class hides content.


json.Marshall returning a wrong empty result when provided an object from yaegi

The following program sample.go triggers an unexpected result:
package main

import (
    "encoding/json"
    "fmt"
)

type Foo struct{}

func (f Foo) MarshalJSON() ([]byte, error) {
    return []byte(`{"test": "foo"}`), nil
}

func main() {
    i := Foo{}
    marshal, err := json.Marshal(i)
    if err != nil {
        panic(err)
    }
    fmt.Println(string(marshal))
}

Expected result:
$ go run ./sample.go
{"test":"foo"}

Got:
$ yaegi ./sample.go
{}

UnmarshalJSON has a similar issue: The following snippet runs fine in if compiled but yaegi fails with the following error:
parsing time ""2021-01-15T19:29:57"" as ""2006-01-02T15:04:05Z07:00"": cannot parse """ as "Z07:00"..
type data struct {
	First      timestamp `json:"first"`
	Last       timestamp `json:"last"`
}

type timestamp time.Time

func (j *timestamp) UnmarshalJSON(b []byte) error {
	s := strings.Trim(string(b), "\"")
	t, err := time.Parse("2006-01-02T15:04:05", s)
	if err != nil {
		return err
	}
	*j = timestamp(t)
	return nil
}

@ldez Any news for this issue ?

@flemzord, thanks for asking this. This issue does not occur anymore with yaegi-v0.14.2, so I'm closing it.
Cheers.


Respect skip_root option of cascade

Maybe I just got the meaning of the skip_root option wrong.
My understanding is that it should skip looking up keys in the root of the locale tree.
So if there is only one translation foo: 'bar' it should not be found by
I18n.t :foo, cascade: {skip_root: true}

So far the test for skip_root did not have any effect (that i could observe).
The cascade tests also passed with skip_root: true.
I included a test to illustrate my understanding of how skip_root should work.
If that's not the way it's meant to work tests for it should be included or it should be removed entirely.

The skip_root option was added in e2b8a766d69923da1582d364704ce3928833adf3 and it's not clear to me what this was intended for.
I think this one will need to be handled by @svenfuchs.


Clean up the tests

Signed-off-by: Alexey Roytman<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 786776769

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.932%

Totals

Change from base Build 774653870:
0.0%

Covered Lines:
677

Relevant Lines:
880

💛  - Coveralls

LGTM except that the commit message could be more specific, e.g. Clean up encoding_test.

Thanks @hzhou8 ,

the commit message could be more specific, e.g. Clean up encoding_test

Fixed


Fix linking issues on Windows

closes https://github.com/noib3/nvim-oxi/issues/77, https://github.com/noib3/nvim-oxi/issues/120

I've added those linker flags to the .cargo/config.toml file to see if it would fix CI on Windows.
Unfortunately all the integration tests fail because Neovim exited with non-zero exit code: -1073741819, which from what I call tell means that some access was denied.
Does the same happen to you locally if you do
cargo build --features=neovim-nightly
cd tests
cargo test --features=neovim-nightly

?

I've added those linker flags to the .cargo/config.toml file to see if it would fix CI on Windows.
Unfortunately all the integration tests fail because Neovim exited with non-zero exit code: -1073741819, which from what I call tell means that some access was denied.
Does the same happen to you locally if you do
cd tests
cargo build --features=neovim-nightly
cargo test --features=neovim-nightly

?

Looks like I still need to add the build.rs mentioned in this comment to make it work. So, looks like we need to have a lua51.dll and lua.lib file at the time of testing in CI

So it works locally if you add that build.rs to tests? Seems weird since that's only relevant at build time, and building already works in CI.

Closed by #149.


mockReturnValue is not typesafe

This compiles, but it shouldn't:
import { createSpyFromClass } from 'jest-auto-spies';

class Foo {
  bar(param: string): string {
    return '...';
  }
}

const foo = createSpyFromClass(Foo);
foo.bar.mockReturnValue(1);
foo.bar.calledWith(1).mockReturnValue(1);

When there is a strong type system we shouldn't write tests for invalid cases.
typescript version: 4.2.4
jest-auto-spies version: 1.6.5

Hi Shai and Guillermo,
I am interested to take over this issue.
Sofiane
Le mer. 11 août 2021 à 14:41, peterreisz @.***> a
écrit :

This compiles, but it shouldn't:
import { createSpyFromClass } from 'jest-auto-spies';
class Foo {
bar(param: string): string {
return '...';
}}
const foo = createSpyFromClass(Foo);foo.bar.mockReturnValue(1);foo.bar.calledWith(1).mockReturnValue(1);
When there is a strong type system we shouldn't write tests for invalid
cases.
typescript version: 4.2.4
jest-auto-spies version: 1.6.5
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hirezio/auto-spies/issues/51, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLIX6TOP663RZXULPNMCW3T4J4XPANCNFSM5B6RC7PQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Thanks @peterreisz for the report
And thanks a lot @meksof for the help!
I've assigned this issue to you
Please let us know if you need anything
Thanks again! ❤ 💪

Has this been resolved? Just noticed this today.

I think I resolved it in my pull request #73
Perhaps @shairez could give some feedback. 😉

Thanks for the PR @mopinsk ! 🙏


add past executions for failed flow in failure email

This PR adds following information to the failure email template:
overall failed execution among last 72 hours' execution(e.x 3/25 failed)
past 72 hours execution tables(status, start/end time).
The motivation is to capture people's attention if the flow has been failing too many times recently.
Eg:

Pull Request Test Coverage Report for Build 3889

26 of 35 (74.29%)  changed or added relevant lines in 4 files are covered.
23 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 35.812%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-common/src/main/java/azkaban/executor/JdbcExecutorLoader.java
0
1
0.0%

azkaban-common/src/main/java/azkaban/executor/ExecutionFlowDao.java
3
5
60.0%

azkaban-common/src/main/java/azkaban/utils/Emailer.java
4
10
40.0%

Files with Coverage Reduction
New Missed Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/JobRunner.java
2
76.89%

azkaban-exec-server/src/main/java/azkaban/execapp/event/JobCallbackRequestMaker.java
21
56.25%

Totals

Change from base Build 3888:
-0.01%

Covered Lines:
10303

Relevant Lines:
28770

💛  - Coveralls


Improve and extend launch file tutorial

Replaces https://github.com/ros2/ros2_documentation/pull/876.

To start with the new tutorials, I have migrated the existing one to a XML launch file.
The python examples are still there, in a different tab.
I have also added a few sections showing more advance launch features (in progress).
I have deleted the explanation of how to create a folder/file. I think that's a bit unnecessary, but I can add that again.

Closed in favor of #1468.

Thanks @audrow for taking over this!!


[FLINK-24719] Update Postgres test dependencies

What is the purpose of the change

Update Postgres and Postgres test dependencies to latest version, currently used by JDBC connector

Brief change log

Updated dependencies in POM

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@AHeise Please merge


Use Object.hasOwnProperty

I noticed that this module is dependent on the hasown module. Node has had Object.hasOwnProperty since 0.10.0 - could this use the built-in method to slim its contribute to node_modules? I'd be happy to provide a PR if that helps.

Technically it’s Object.prototype.hasOwnProperty.call, and that relies on both Object.prototype and Function.prototype being unmodified.
Using hasown ensures that later mutation of builtins doesn’t break things, so it’ll be relevant and necessary and beneficial forever.


Enhancement: include flows from solutions in 'flow list'

Description

Im using the CLI to export the flows in my environments. For one of our costumers i needed the action "run a child flow". So i created the flow in a solution in the environment. At the moment i can't list the flow with the m365 flow list --environment <environment> --output json and can't export the flows in solutions.

Steps to reproduce

Create a Solution in PowerAutomate
Create or add a flow to this solution
Run the Command m365 flow list --environment <environment> --output json

Expected result

List all flows in the environment included flows in a solution.

Actual result

List all flows in the environment excluded flows in a solution

Environment

CLI-Version: 3.8.0
PS-Version: 7.1.3

Thank you for reporting the limitation @joshua-probst. The current functionality is by design and is tied to the API that we have available to retrieve flows. We could research if there is a way for us to retrieve flows from solutions in a consistent way. If we can, it would make a great extension for the existing command

Retrieving a flow from a solution is possible, but this requires a different endpoint than the one in m365 flow list . See Power Automate Web API docs
I've tested it with a manual flow on my tenant.

This one interests me, but i feel it needs design first.
To start with, the properties that we get back from the command m365 flow list are different then the ones we get back from the api https://ENVIRONMENT/api/data/v9.1/workflows?$filter=category eq 5. So how can we fit this into this command? or should we rather look at this as a separate command, for example m365 pp solution flow list ?

Good point! If it's a different API and different data shape, then it would be confusing to mix it in one command. Having a separate command specific for this purpose would be a good place to start and we could see if there's way to combine them in the future.

A couple of questions:

in the description you use the spo prefix. I suspect that it's copy-paste from another spec and not the correct value, right?
in examples, you're not using any prefix, which is also incorrect, let's update them
in the first example we don't specify either the solutionName or -id. I don't think that's intentional, is it?
in the additional info, you refer to dynamicsApiUrl: how do we get to it?
in the additional info call, in $select you only specify solutionid. Will it include name and workflowidunique as well or do we need to adjust the call?
does the command need any additional permissions that we're not using yet and which we should add to the app reg?

A couple of questions:

in the description you use the spo prefix. I suspect that it's copy-paste from another spec and not the correct value, right?

yes, totally a copy-paste error

in examples, you're not using any prefix, which is also incorrect, let's update them

sorry about that..

in the first example we don't specify either the solutionName or -id. I don't think that's intentional, is it?

It is, when you don't provide solutionName or -id, it list the flows from all the solution. When you provide solutionName or -id it only list the flows from that particular solution

in the additional info, you refer to dynamicsApiUrl: how do we get to it?

We get the dynamicsApiUrl by using the common function powerPlatform.getDynamicsInstanceApiUrl where you provide the environment and the adAdmin parameter

in the additional info call, in $select you only specify solutionid. Will it include name and workflowidunique as well or do we need to adjust the call?

No, the additional call only happens when you provide the option solutionName, only then we have to get the solutionid with which we make the final call that returns all the information from the flows

does the command need any additional permissions that we're not using yet and which we should add to the app reg?

Tested the api's with the request command, and it doesn't seem that we need additional permissions

Sorry for just getting back this enhancement now.
I think that we can close this enhancement, because the command already list all the flows inside solutions. Thank you all that this was fixed.


Make sure integer types in protocols/horizon are consistent with XDR

Originally posted by @leighmcculloch in https://github.com/stellar/go/pull/3845#discussion_r697659878:
I understand the motivation is because whilst the XDR is int32, Core will only use positive values.
I think this inconsistency in types could come back to bite us at some point. If the values are inconsistent it is always possible for the protocol to change in a way that breaks the Horizon API. And it's very common in applications to know that an int value will only be positive, so I don't see the problem this solves.
Another way to think about this is that we're trying to constrain the possible values the field will have to only positive values, however by using an uint32 all we've done is shifted the possible values into another range that also won't be used, positive values greater than what a int32 will hold.
If the goal is to capture the space of usable values, the positive side of int32 is uint16.
However, inconsistencies in Horizon's API and the XDR are inconvenient and confusing to developers using both, so I'd vote for just matching the XDR.

The positive side of int32 is uint31, not uint16. The sign bit only causes the representation to be divided by 2.

@2opremio 👍🏻 Ah you're right, thanks for correcting me.

@bartekn @leighmcculloch to be clear, the scope of the current issue is only the new AMM code?
I was thinking we should probably do the same for all the code, although it will entail a breaking change.

I think we should do it for all fields in the API but I'm not sure when. Looks like Protocol 18 upgrade could be a good moment because everyone will need to upgrade and update their clients anyway. Let's discuss in Horizon meeting.

OK, I will start with the protol18 changes

We discussed this today in the Horizon team meeting. I'd rather not proceed with this for now.
The change is breaking for clients but doesn't solve a problem anybody has. It does reduce surprise for readers of the XDR, but that is more of a nice to have. It is possible the protocol might someday change in a way that affects this, but if we haven't fixed it by then, that would seem to be a forcing function.
Even after these changes, there are places where we will still continue to map types (most notably, strings for large numbers).
This topic is arising now because we are adding more fields. However, as long as we continue to be consistent with existing type approaches there is no problem changing these (and all the others) in the future. An ideal opportunity for this refactor would be when we implement API versioning.
For future context: Core originally switched to signed types in order to avoid issues with their internal SQL data store (https://github.com/stellar/stellar-core/issues/1960). The relevant code change was https://github.com/stellar/stellar-core/commit/d9e80975d5b8edcca54e9eb0143fd0b4796f05fc.

Makes sense.

as long as we continue to be consistent with existing type approaches there is no problem changing these (and all the others) in the future

However, does this have the knock-on affect that Horizon will use different types for new fields? I don't think there's much value in continuing this approach of using uint32 instead of int32 for new fields.

I'm proposing we make the new fields uint32 as well, for consistency. The value is that they are all the same, no surprises. Changing to new types going forward adds complexity for the user.
Since we want to refactor the others anyway eventually, fixing these at that time seems fine.

@ire-and-curses do we want to close it or keep it open in the Backlog? Currently it's open but in Done section.

We can leave it open for now. @paulbellamy and I started discussing how to clean up the backlog, we'll adjust status more then if needed.


Release depends on generated proto code

Since the googleapis Status files were removed from source control (and are created using 'make generate'), we need to update the release workflow to run generate prior to releasing code.

Update needed to release workflow after #86.


i18n support

Add i18n support

Just for the note, partially was implemented here: https://github.com/vitkarpov/grunt-nunjucks-2-html/issues/13

i18n of pages was implemented based on node-gettext.
Special wrapper around grunt-nunjucks-2-html provides page with neseccary for i18n handling functions, filters, etc.
i18n of JavaScript files isn't supported yet.


Scene based animations

Hi,
first of all, these glTF Tools are great, I have jsut started using it and it's a must have tool for gbl/gbTF editing, validation and creation.
I have come across a few glb files which include animations, however, these are not stored under the animations object but rather as multiple scenes.
In other words, I have a glb with 20 scenes which should be iterated (unloaded and loaded) in order to create the animation.
Inspecting the file in Code, I can clearly see all the scenes and their nodes but is there a way to animate such files using glTF Tools?
If not, it would be great to add such functionality.
Regards
Tim

Closing this until more info is known about these kind of glTFs.


Update countdown to 2.0.1

RE-CHECK!

Automated adapter checker
ioBroker.countdown

[ ] :heavy_exclamation_mark: [E606] Current adapter version 2.0.1 not found in README.md
[ ] :eyes: [W156] Adapter should support admin 5 UI (jsonConfig) if you do not use a React based UI
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.countdown

[ ] :heavy_exclamation_mark: [E606] Current adapter version 2.0.1 not found in README.md
[ ] :eyes: [W156] Adapter should support admin 5 UI (jsonConfig) if you do not use a React based UI
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W145] Each "common.news" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[ ] :eyes: [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package

Add comment "RE-CHECK!" to start check anew

E606 seems to be false positive

@jack-blackson E606 is caused by an incorrect fiormatting if revision headers. It should be level three (### 1.2.3 ...) but is level two (## 1.2.2 ...) so it is not detected by checker. Please fix with next release (or use @alcazone/release-script).
Everything's fine for now

Thanks for checking. I updated the readme file.


Import skas simulator tests

I've written a "client simulator" for reference-kbs that imitates how a VMM host + encrypted guest would interact with the attestation server. Import these tests to reference-kbs for CI + compatibility.
https://github.com/tylerfanelli/skas

@slp My one question is how to get the server started for tests. Would I simply run rocket() in main.rs and then begin interacting with port 8000?


TF2 porting: text feature

Text feature now passes pytest with 2 warnings on precision\recall ill defined (I suppose due to demo\test data)
I'm doing pytest tests\integration_tests\test_experiment.py::test_experiment_seq_seq (which contains a text input and text output)
This seems to be specular to the audio feature, although I'm mentally missing the output feature call method, along with some doubts on the datatypes (I did not expect text to be loaded as int8, might need to widen the assert to allow other datatypes if necessary)
As usual, let me know if this is useful at all :)

I guess both you and @jimthompson5802 worked on the same, my bad for not syncying the work of both. Jim could you please check this and see if it is the same code you did? If not could you post here the differences?

@w4nderlust
re:

Jim could you please check this and see if it is the same code you did?

Yes, @carlogrisetti and I worked on the same module: text_feature.py.  Here is a diff showing the changes I made.
text_feature_diff.txt
Since Text is a subclass of Sequence and dependent the recent changes, I'm just waiting for the clean-up work on Sequence classes before pushing the Text Feature in case there are last minute breaking api changes to work out.

now that I merged the sequence decoder PR, could you two merge tf_porting in this and then merge both your  efforts into a single PR?

@carlogrisetti if you merge the updated tf2_porting branch into your branch should bring you up-to-date on the other changes and allow you to finish this PR.  If you need help or have questions, let me know.

And this passes tests again... seems even too simple to be true

Thank you!


Fix calling reducer after subEffect

Fix calling reducer after subEffect.
More information about problem in Issue #613

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


Fix typo

"er" is undefined - my bad!

I've merged some of this via https://github.com/alexa-js/alexa-app-server/commit/25789e16111946bfc8553bdb6c00a1cffba97764. I don't understand the use change, can you please explain?
I've also added tests in https://github.com/alexa-js/alexa-app-server/pull/23. Would you be so kind to write tests for all the changes, the error one would be a great start.

This change:
`-                        var ip = req.headers['x-forwarded-for'] || req.connection.remoteAddress;

                   if (ip=="<IP_ADDRESS>" || ip=="::ffff:<IP_ADDRESS>" || ip=="::1")

                   var ip = req.connection.remoteAddress;

                   if (!req.headers['x-forwarded-for'] && (ip=="<IP_ADDRESS>" || ip=="::ffff:<IP_ADDRESS>" || ip=="::1"))
                   {`

was a security fix.  A browser could provide a header like:
X-Forwarded-For: <IP_ADDRESS>
which would allow it to bypass the alexa verification process. This change ensures that X-Forwarded-For is only observed if it's sent from a local server (such as a proxy). Otherwise, the real IP is used.

That's fine, and that whole logic moved to alexa-verifier.
I was asking about this:

Ah yes - well, that is because bodyParser.urlencoded doesn't necessary work for all the user server modules.
In my project, for example ( https://gyazo.com/bc5b88a444b8ff334b78a5771b3d80c0 ) there are many express routes added which don't use urlencoded forms, so blanket-applying bodyParser.urlencoded to every request causes problems.

Oh I see, that makes a lot of sense! I opened https://github.com/alexa-js/alexa-app-server/issues/35 and will close this.
If you have time, would love some tests and a pull request for that issue.


Add ui for section ordering

Codecov Report

Merging #10 (986d16c) into master (efa17c0) will increase coverage by 0.04%.
The diff coverage is 92.00%.

@@             Coverage Diff              @@
##             master      #10      +/-   ##
============================================
+ Coverage     91.80%   91.85%   +0.04%
- Complexity      303      304       +1
============================================
  Files            29       29
  Lines          1221     1228       +7
============================================
+ Hits           1121     1128       +7
  Misses          100      100

Impacted Files
Coverage Δ
Complexity Δ

src/Model/Table/TasksTable.php
93.26% <92.00%> (+0.25%)
51.00 <0.00> (+1.00)

Continue to review full report at Codecov.

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


Reporting errors on partial matches

We need to report errors occurred when creation of violations when a rule partially matches or otherwise. Windup has an Optional category for violations that address this.

We need to re-design the output format also to cover #38. At this point, lets use this issue to track the entire re-work.

This is fixed via #114


Bump framework version

Purpose
$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7277792368

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7277792368
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7284097711

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7284097711
Status: failure

Closing since version has been already updated with https://github.com/wso2/product-is/pull/18711


Support for loading SVG from Blob or resources (Android)

I did refactoring and added other source for SVG like Blob.
Can you extend iOS functionality in this way?
With this functionality we can set and update svg without storing it to a file.
Change color:
var file = Ti.Filesystem.getFile('images/myimage.svg');
var blob = file.read();
var source = blob.text;
source = source.replace("#315b09", "#000000");
$.svg.image = Ti.createBuffer({value: source}).toBlob();

Or set image value directly from HTTPClient response or generate it in application.

@falkolab. I think this module needs refactoring as it does not fail gracefully if the image file is not found - basically crashes the app. I am struggling to get it to work with Tishadow (actually I can make it work on iOS but not Android at the moment) as I believe the module accepts local file paths and not global and therefore causes a problem in tishadow (which changes absolute paths of files in the compile). Maybe your new source type will solve that problem so I will try it out.

hey i didn't saw this PR !! Thanks a lot  ! I will have a look at the code and release an updated package/ I am so sorry that these time i do not have time for open source. So i can't work on modules.
Thanks a lot @falkolab  !! :)
@icecandy : You have to know that this module is based on open sourced lib's that i can't change myself ! :/

@GeraudBourdin Please update ios module to support blobs as source and level of modules versions

That would be nice :) I think Geraud is very busy though.

Сan anyone help him?


Can not see any the screenshots on macos

Hi guys,
I follow the guide in the link: https://www.htmlhifive.com/conts/web/view/pitalium-tutorial/test-basic-run
But I can not see any screenshots in the test result folder.
Please help me!!! Thank you so much.

I also can not see the currentExpectedIds.json from the hifive-pitalium folder that downloaded by git commands.

Hi. Thank you for trying Pitalium!
Could you give me information below?

test code (SampleTest.java)
config files(environmentConfig.json, capabilities.json, testAppConfig.json and persisterConfig.json)
test log (as shown in the "Console" tab on your screenshot)

Also I will confirm if the tutorial document has any flaw.
Are you reading the English version of the doc?

Hi, thank you so much for your kind.
I am reading the English version.
I got struggle to deal with MacOs due to the fact that the doc is wirte for windows.
Do you have any doc for MacOs?
I have attached the file here Files.zip. Please help me to check it.
Thank you so much!

Sorry we don't have MacOS version. I'm glad if you tell me which part of doc was confusing.
I've checked your configuration files.
In environmenConfig.json, I saw execMode property is set to "RUN_TEST" mode. But you should set it to "SET_EXPECTED" to collect screenshots as the expected(correct answer) image of the view in your first try.
If you run test in RUN_TEST mode without expected images then all the assertView fail.

@Haro10 Hi, Did my suggestion solve the problem?

Hi!
Sorry for replying late. I have resolved it. I don't know exactly how I fixed it, but it luckily was fixed.
If I have more information, I will contact you!
Thanks for your kind!

Got it. I'll close this issue.
If you find another problem, don't hesitate to reopen it.
Thank you!


comment linting and test refactoring

This is a superficial PR cherry-picking some refactoring and linting done working on another upcoming PR. It only touches comments and test code. The main change is extracting various test helpers into a new testutil package to improve the signal-to-noise of the tests themselves.

I'm going to insta-merge this because it's low impact / low risk and I'd like to base an actually-useful PR on it.


Use ceph-iscsi 3.x package names

In ceph-iscsi 3.x ceph-iscsi-config and ceph-iscsi-cli are merged into a single ceph-iscsi package.
Fixes #3872

Should be backported to stable-4.0 as well

@dsavineau yes, there is more work pending on ceph ansible in regards to ceph-ansible and iscsi - but I don't know the status of the patch that Mike Christie mentions in #3870

Fixed in #3977


Provide WidevineCdm folder as zip file for Windows users

Widevine installation could be made faster and hassle-free for Windows users if the latest correctly configured Widevine module (WidevineCdm folder) is provided. Others could then just extract and copy-paste this folder in their User Data directory. This should work without any problems and also solve the issues I mentioned in #1439.

merged with #1439


Removal of extra_zones flag breaks tutorials

Removal of the extra zones flag breaks all the .yaml configs in
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/tree/master/tutorials/inter_region_reports/data
This was very useful for allowing two zone axes for multi-zone tests.
It is unclear how to achieve a similar result now without having several dozen configs files.

It looks like the resolution is some use of flag_zip and zone no?

Working on this


Android bad animation

I was testing and in the browser works excellent but in the device the animation have issues :(

Same problem here. On iOS is working perfect.

Try it:
td-card {    -webkit-transform:translateZ(0); }


Create a more useful contributing document

Add info about how to get involved to Contributing.md and reference it from the readme. More background in #26

Do we still need that? I guess by now the reason of this fork is well known


How have min_sizes and max_sizes been computed (data/config.py)?

I'm interested in understanding exaclty how min_sizes and max_sizes have been computed. More specifically, in data/config.py the above are defined as lists under the corresponding keys of the dictionaries voc and coco.
I know that they correspond to the minimum and maximum sizes of objects in the different scales, but I do not understand how exactly they have been computed. For instance, for the feature map of dimensions 38x38, for the coco dataset, the minimum size of the objects is 21 and the maximum 45. What does this mean?

Have you understood how min_sizes and max_sizes have been computed? @chi0tzp

I also want to know how to set exact value.Perhaps from the distribution of bounding box size?

Hi, has anybody figured this out? @chi0tzp @vickersmith @LZP4GitHub

I also so curious about this


Chore: Apply Emitter in simulator and fix cleanup when terminated

@tonilastre Applied the standardized naming convention and updated docs. Feel free to take one final glance. :slightly_smiling_face:


New Version of the Microsoft 365 learning pathway Not Working after upgrade

Describe the bug
Upgraded the SharePoint app catalog with the new version of the Microsoft Learning Pathways app. Installed it into Teams, and when I click on the app, I get an error message
"Sorry, something went wrong
Can't load the application on this page. Use the browser Back button to retry. If the problem persists, contact the administrator of the site and give them the information in Technical Details.
TECHNICAL DETAILS
Error details: Error: Error initializing application. Error: TypeError: Cannot read properties of undefined (reading 'componentType')
[Troubleshoot issues with Microsoft SharePoint Foundation.](javascript:HelpWindowKey('WSSEndUser_troubleshooting'))
Correlation ID: 82c8f4a0-20f5-7000-bf31-fac6d1bc5a65
Date and Time: 12/5/2023 6:21:54 AM"
To Reproduce

Go into Microsoft teams and find the Microsoft Learning teams app click open which produces the error message.

Expected behavior

To load the Microsoft learning path apps

Screenshots

Learning Pathways version number

Please share the version number of learning pathways in your tenant app catalog 4.4.0.

Additional context

Before the upgrade, the owner of the Learning Pathways changed the URL name in the SharePoint communication site. We ran the PowerShell URL script to reconfigure the app, which installed without errors, but we're not sure if this is also causing the errors. The old app is still present along with the new app.

@JoNico5, Thank you for reporting this. Please allow us sometime to analyse the issue and we will get back to you ASAP.

@JoNico5, We tried from our end but we are not able to reproduce the issue. We tried to upgrade from v4.0+ to latest version.
Can you please provide below details to help us understand the root cause of this issue?

What version you are upgrading from?
Do you have any custom config settings?
Is multilingual settings enabled?

If you are upgrading from version 4.0+, can you please try to clear all entries from custom config list and try to check if you are getting same issue?

@JoNico5 you can try to remove the config files and any custom items that point to the old URL. I would expect that if that was the case that they would be erroring out in the browser as well. It's the same code running in Teams. To clear the cache you need to go to the Learning Pathways site => Site Contents => CustomConfig. Delete everything with the title CustomConfig.
Depending on what is in the CustomAssets or CustomPlaylists if there is data in there with the old URL you should clear that out as well.
Try to do that and let us know if that fixes the issue.

@dcashpeterson I have removed all custom config that contained the old name and it is still producing the same error.  Is there a way to reload the learning path but still keep the content that is on the comms site?

@JoNico5 if you want to delete everything then you can remove all items from the CustomConfig list and the customasset and customplaylist lists. This will however remove any custom playlists you have created. It will also remove the custom assets from Learning Pathways but not the pages that were created so if you have a page in the new site in the Site Pages library that page will still be there. You would need to create a new asset and link it to that page.
That said, because this works in the browser and not Teams. Without doing the above, you can uninstall the SPFx package and make sure that you empty the trash in the app catalog, remove the app from Teams and make sure you can't see it from the Teams admin center. Then once you can confirm that it was deleted you can reinstall the SPFx package and sync the package to Teams.

OK lovely I will give that ago

Closed. If this is still an issue please open a new issue and reference this one in the comments.


Error in example.py

Custom Bot is now running...
Traceback (most recent call last):
File "C:\Users\Iiro Hei\Downloads\ClashRoyaleBuildABot\example\main.py", line 18, in
main()
File "C:\Users\Iiro Hei\Downloads\ClashRoyaleBuildABot\example\main.py", line 14, in main
bot.run()
File "C:\Users\Iiro Hei\Downloads\ClashRoyaleBuildABot\example\custom_bot.py", line 42, in run
self.set_state()
File "C:\Users\Iiro Hei\AppData\Local\Programs\Python\Python39\lib\site-packages\clashroyalebuildabot\bot\bot.py", line 116, in set_state
self.state = self.detector.run(screenshot)
File "C:\Users\Iiro Hei\AppData\Local\Programs\Python\Python39\lib\site-packages\clashroyalebuildabot\state\detector.py", line 63, in run
self._draw_text(d, v['bounding_box'], str(v['number']))
File "C:\Users\Iiro Hei\AppData\Local\Programs\Python\Python39\lib\site-packages\clashroyalebuildabot\state\detector.py", line 35, in _draw_text
text_width, text_height = self.font.getsize(text)
AttributeError: 'FreeTypeFont' object has no attribute 'getsize'

I have the same problem as well

@all-contributors-bot please thank @llgorrrrr for reporting a bug

@all-contributors-bot please thank @llgorrrrr for reporting a bug

@all-contributors please thank @IIgorrrrr for reporting a bug

@all-contributors please thank @IIgorrrrr for a bug report

Didn't work :(

Didnt work, please reopen this issue


zotero.lua: Add support for generating transferable live ODT document

commit c6b62dfc8f4a09e8a44ba83cd119934354bc5e10
Author: Jambunathan K<EMAIL_ADDRESS>Date:   Thu Jul 30 15:36:51 2020 +0530

    zotero.lua: Add support for generating transferable live ODT document

    * pandoc/pandoc-zotero-live-citemarkers.lua (config): New field
    `transferable'.

    (Meta): Set `config.style' from metadata variable
    `zotero_transferable'.  Update meta.xml, only if the doc is
    non-transferable.

    (zotero_docpreferences_odt, zotero_bibl_odt_banner): New
    functions.

    (zotero_bibl_odt, zotero_ref): Handle `config.transferable'.

    (Doc): Transferable documents require a banner (and document
    preferences) right at the beginning of body text.

    * pandoc/Makefile (ZOTERO_TRANSFERABLE): New variable.

    (odt-scannable-cite.odt, odt-with-csl-style.odt)
    (odt-without-csl-style.odt): Use above variable.

    Add support for generating transferable documents.  See 'Moving
    Documents with Zotero Citations Between Word Processors'
    (https://www.zotero.org/support/kb/moving_documents_between_word_processors).

    A document can be transferred only if it has a prior style.

    You can generate transferable documents with

        pandoc -s --metadata=zotero_csl_style:chicago-author-date \
        --metadata=zotero_transferable:true  -L zotero.lua ...

    For discussions surrounging this feature see

        https://github.com/retorquere/zotero-better-bibtex/pull/1586
        https://github.com/retorquere/zotero-better-bibtex/pull/1592

Looks like there is some cross-talk.  Let me resolve the conflcts on my side.

It looks like you haven't fetched upstream maybe?

I have resolved the conflicts.  I have checked that lua file is good.   Double-check if the Makefile is OK.

Feel free to make any changes that you deem necessary.

I can again merge and make the change requests I marked or leave those to you, whatever you prefer.

I can again merge and make the change requests I marked or leave those to you, whatever you prefer.

Do the changes yourself.
Any functionality-related questions I will answer here, so that you can refer back to why certain things are the way they are.

I suggest that you do a quick unit test i.e., build the different odt files, open it in LibreOffice and check.  If you are satsfied, you can merge.
Any minor issues that you may run into can always be taken up as bug-fixes.
(Don't worry.  I will not run away.  I will be around to answer questions.)

Ball is in court.  I will not make further changes.  (I am saying this so that we don't run in to each other's shoes as we make changes independently).  3-way merges aren't really that pleasant for any developer ...

I think I have all the changes now. I've just merged to master; I just won't bundle until I've done the docx version and some tests.

I think I have all the changes now. I've just merged to master; I just won't bundle until I've done the docx version and some tests.

Thanks.  I will be around to answer any questions ....

I won't have further time until tomorrow

Neither am I in hurry nor would I insist to get the changes in to bundle.
(I have my local changes  which I can experiment with)

I won't have further time until tomorrow

Ok. Good day.


VR doesn't work

Issue description
VR doesn't seem to work as of opening this issue.
While Oculus and SteamVR (in my case) do open while OpenXR is enabled in the project's settings, the functionality just doesn't seem to be there.
Even if I open my respective VR software before running, it will just run on Flat Mode instead.
Steps to reproduce
Run V-Sekai with a VR headset connected, before or after booting up its respective software.
Minimal reproduction project (For SDK or engine issues)
No response
(closest - https://github.com/V-Sekai/v-sekai-game/commit/328c01ec1b52de34572fba0d1af35e26bafbebeb)
Using a released version?
No
What OS? Any other relevant information? (V-Sekai version, Godot revision, System information)
Windows 10, Oculus Rift

I'm checking this as soon as I can.

I am testing an alternative fix, and a map upload fix.

I tried to run it in VR just now and it works!
The controllers don't work well for some reason, but I'll just open a new issue for that.


fix(UsageText): consistent indent for help UsageText output

What type of PR is this?

[X] bug
[ ] cleanup
[ ] documentation
[ ] feature

What this PR does / why we need it:
This PR addresses the bug found by @codefromthecrypt and noted in #1278
Which issue(s) this PR fixes:
Fixes #1278
Testing

Added multiple unit tests that test the functionality for multi-line UsageText values for App, Command and SubCommand types.
Spot tested with a local CLI tool integration.

Release Notes
fix(UsageText): consistent indent for help UsageText output

only thought is might want to also backfill a test that MarkdownDocTemplate does not indent any of the lines (maybe overkill..)

only thought is might want to also backfill a test that MarkdownDocTemplate does not indent any of the lines (maybe overkill..)

The markdown is a bit different. The values passed into the template are manipulated before the execution of the template render. Note the prepareCommands on line 59 as an example. https://github.com/urfave/cli/blob/master/docs.go#L56-L62
As a result, there shouldn't be a need to adjust the template or test for the MD generation. A part of me wants to refactor the MD generation to use template functions more consistently, but that is an out of scope effort that I think warrants more conversation before implementing.

out of scope comment so hope it isn't too distracting. I found indeed markdown could be better handled differently, though to each their own on this. Particularly, there's formatting bias towards help output which doesn't work so well in markdown. I ended up using this which means I don't personally need the markdown feature at the moment:
	cli.MarkdownDocTemplate = `# GetEnvoy CLI Overview
{{ .App.UsageText }}

# Commands

| Name | Usage |
| ---- | ----- |
{{range $index, $cmd := .App.VisibleCommands}}{{if $index}}
{{end}}| {{$cmd.Name}} | {{$cmd.Usage}} |{{end}}
| --version, -v | Print the version of GetEnvoy |

# Environment Variables

| Name | Usage | Default |
| ---- | ----- | ------- |
{{range $index, $option := .App.VisibleFlags}}{{if $index}}
{{end}}| {{index $option.EnvVars 0}} | {{$option.Usage}} | {{$option.DefaultText}} |{{end}}
`


Remove the default project if it exists

:nut_and_bolt: Description
Cleaning up the default project.
:athletic_shoe: Demo Script / Repro Steps
start_all_services
Create a project with ID default in the UI.
rebuild components/authz-service
See the default project go away.
:chains: Related Resources
:white_check_mark: Checklist

[ ] Necessary tests added/updated?
[ ] Necessary docs added/updated?
[ ] Code actually executed?
[ ] Vetting performed (unit tests, lint, etc.)?

@bcmdarroch Nope! We set up cascade delete for all project assignment relational tables you've listed above :) Let's walk through it:
We have set up cascade deletes on iam_projects
Referenced by:
    TABLE "iam_policy_projects" CONSTRAINT "iam_policy_projects_project_id_fkey" FOREIGN KEY (project_id) REFERENCES iam_projects(id) ON DELETE CASCADE
    TABLE "iam_role_projects" CONSTRAINT "iam_role_projects_project_id_fkey" FOREIGN KEY (project_id) REFERENCES iam_projects(id) ON DELETE CASCADE
    TABLE "iam_statement_projects" CONSTRAINT "iam_statement_projects_project_id_fkey" FOREIGN KEY (project_id) REFERENCES iam_projects(id) ON DELETE CASCADE

Let's test that it works!
Put the default project into a policy, a statement, and a role
chef_authz_service=# insert into iam_projects (id, name, type, projects) values ('default', 'default-name', 'chef-managed', '{default}');
INSERT 0 1
chef_authz_service=# select * from iam_projects
chef_authz_service-# ;
 db_id |        id        |        name        |     type     |      projects
-------+------------------+--------------------+--------------+--------------------
    27 | ~~ALL-PROJECTS~~ | All Projects       | chef-managed | {~~ALL-PROJECTS~~}
    28 | (unassigned)     | Unassigned Project | chef-managed | {(unassigned)}
    31 | default          | default-name       | chef-managed | {default}
(9 rows)

# cat test.json
{
  "name": "Projects Test",
  "id": "projects-test",
  "members": [ "token:*" ],
  "statements": [
    {
      "effect": "ALLOW",
      "actions": [ "*" ],
      "projects": [ "default" ]
    }
 ],
 "projects": [ "default" ]
}

[63][default:/src:0]# curl -H "api-token: $TOK" -XPOST -d @test.json https://localhost/apis/iam/v2beta/policies --insecure | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   433  100   218  100   215    150    148  0:00:01  0:00:01 --:--:--   298
{
  "policy": {
    "name": "Projects Test",
    "id": "projects-test",
    "type": "CUSTOM",
    "members": [
      "token:*"
    ],
    "statements": [
      {
        "effect": "ALLOW",
        "actions": [
          "*"
        ],
        "role": "",
        "resources": [
          "*"
        ],
        "projects": [
          "default"
        ]
      }
    ],
    "projects": [
      "default"
    ]
  }
}

[68][default:/src:0]# curl -H "api-token: $TOK" -XPOST -d '{"id":"test-role","name":"test", "projects":["default"], "actions":["*"]}' https://localhost/apis/iam/v2beta/roles --insecure | jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   169  100    96  100    73     65     50  0:00:01  0:00:01 --:--:--   115
{
  "role": {
    "name": "test",
    "id": "test-role",
    "type": "CUSTOM",
    "actions": [
      "*"
    ],
    "projects": [
      "default"
    ]
  }

We see that each relational table contains the default project
chef_authz_service=# select * from iam_role_projects;
 role_id | project_id
---------+------------
      13 | default
(1 row)

chef_authz_service=# select * from iam_policy_projects;
   policy_id   | project_id
---------------+------------
 projects-test | default
(1 row)

chef_authz_service=# select * from iam_statement_projects;
             statement_id             |    project_id
--------------------------------------+------------------
 407e3fad-4a34-4e84-bfdc-7945c77e55dc | default
(1 rows)

Delete default project and it cascade deletes itself everywhere
chef_authz_service=# DELETE FROM iam_projects WHERE id='default';
DELETE 1
chef_authz_service=# select * from iam_statement_projects;
             statement_id             |    project_id
--------------------------------------+------------------
(0 rows)

chef_authz_service=# select * from iam_policy_projects;
 policy_id | project_id
-----------+------------
(0 rows)

chef_authz_service=# select * from iam_role_projects;
 role_id | project_id
---------+------------
(0 rows)

Setting up cascade delete earlier means this PR is pretty straightforward for the project assignment tables!
@bcmdarroch @msorens is there somewhere else I'm missing that we need to remove the default project to where it actually got added? I couldn't find anywhere else we actually got around to populating values with the default project but let me know if I'm missing anything else!


Bug bounty testing... Please do not approve. jkhvmx

Bug bounty testing... Please do not approve. jkhvmx

@malshinim: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

@malshinim Thanks for the PR! From what we can tell, it looks like this PR was checking for misconfigurations in our build pipelines that would allow for exfiltration of sensitive info. Based on what we can see, we believe our configuration to be safe. However, if you did find something, please let us know at<EMAIL_ADDRESS>Thank you!


Prototype kcp apiserver -> pcluster egress

The kcp apiserver has to talk to the pcluster for

kubectl exec|attach|logs
webhooks in CRD conversion
webhooks in admission.

For that kubernetes uses egress, compare https://kubernetes.io/docs/tasks/extend-kubernetes/setup-konnectivity/.
The task here is to enable kcp to make use of this mechanism, configured for each shard (not customizable for the users).
Acceptance criteria:

kubectl exec/attach/logs works
webhooks work

I will give this a try!

@pweil- @sttts do you recall if we put this on the roadmap, and if so, which prototype?

At least the logs portion of this has been discussed as priority for prototype 4 as part of the need for basic live observability with no history.    Might consider if we need the full package or can implement items in the description incrementally.

Thanks, added to 4 for now.

/cc

From my experience with WebRTC (few years ago) it is very complex (need to set up stun/turn servers to really bypass NAT, and it wasn't ready for enterprise grade systems. Throughput was quite low, which was OK for p2p where each peer was over "slow" internet connections, but definitely not for datacenter connectivity.
Maybe the syncer can be configured to open a persistent connection to KCP, which will allow KCP to make on-demand requests to the cluster api server? Similar to how websockets work, just using grpc...

Maybe the syncer can be configured to open a persistent connection to KCP, which will allow KCP to make on-demand requests to the cluster api server? Similar to how websockets work, just using grpc...

That's my initial thinking as well

From my experience with WebRTC (few years ago) it is very complex (need to set up stun/turn servers to really bypass NAT in different network levels), and it wasn't ready for enterprise grade systems. Throughput was quite low, which was OK for p2p where each peer was over "slow" internet connections, but definitely not for datacenter connectivity.

Yeah, it is too complex, I've managed to have it working https://github.com/aojea/webrtc-p2p but it requires a custom signaling protocol and integrating it in a kubectl exec/attach/get  workflow has a bad UX, the time to setup a connection is considerable.

and it wasn't ready for enterprise grade systems.

Slipping into v0.7.

A drawing showing the plumbing into the handler chain: https://docs.google.com/drawings/d/1KCqpg5XJPa7QTJPBKY2RO2Na6HGVspJtVepyqjBsu5o/edit

I have the implementation working https://github.com/kcp-dev/kcp/pull/873, however, I can not find the right place to plumb it in the handlers chain.
If someone can pair program with me for 1 or 2 hours we can have it in,

Please set up a time with me today or tomorrow if that works for you

@aojea could you please put this behind a feature gate this is off by default?

@aojea could you please put this behind a feature gate this is off by default?

yes


Detekt plugin not able to download from Maven central

The bug you're experiencing might have already be reported!
Please search in the issues before creating one.
Expected Behavior
It should download plugin from Maven central repo
Observed Behavior
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:1.8:run (detekt) on project agathe: Execution detekt of goal org.apache.maven.plugins:maven-antrun-plugin:1.8:run failed: Plugin org.apache.maven.plugins:maven-antrun-plugin:1.8 or one of its dependencies could not be resolved: Could not find artifact com.beust:jcommander:jar:1.74 in central (https://repo.maven.apache.org/maven2) -> [Help 1]
Steps to Reproduce

Context
I am working with Kotlin and want to do static analysis using Detekt plugin. My problem is, when I want to download it from Maven central repository, it is giving me below error. But when I use company's repository (Nexus), it does not give me any error and downloads it smoothly. I don't understand this.
Can someone tell me why this is happening? I want to use this plugin and have not found out any other alternative.
Your Environment
I am using Detekt for Kotlin with Maven.
Version of detekt is 1.8

Duplicate #1568

Jcommander isn't published to maven central anymore.

The examples provided on the detekt documentation site do ask that users add jcenter to the list of repositories

The last release tag for jcommander on github is from 2017. I can't find the release notes for the 1.74 release. 1.72 is latest on Maven central and 1.71 is latest on github.
As detekt is now fully published to maven central, we should use 1.72 of jcommander instead.
Maybe even migrate away from jcommander ...

As detekt is now fully published to maven central, we should use 1.72 of jcommander instead.

Agreed.

Maybe even migrate away from jcommander

Jcommander does a good job for detekt. I don't think it's worth the effort to migrate away.


Create a spreadsheet of all dependencies on kie-tools with their respective licenses

Due to the Apache move and productization process, we need to know the licenses of all our dependencies on kie-tools.
Open questions:

What licenses are allowed?
Are restrictions the same for development/testing and runtime?

https://www.apache.org/legal/resolved.html

From the root of kie-tools:
license-checker --csv
"module name","license","repository"
"@babel/core@7.16.12","MIT","https://github.com/babel/babel"
"@babel/preset-env@7.16.11","MIT","https://github.com/babel/babel"
"@eslint/eslintrc@0.4.1","MIT","https://github.com/eslint/eslintrc"
"@kie-tools-scripts/bootstrap@0.0.0","UNLICENSED",""
"@kie-tools-scripts/build-env@0.0.0","Apache-2.0","https://github.com/kiegroup/kie-tools"
"@kie-tools-scripts/check-junit-report-results@0.0.0","UNLICENSED",""
"@kie-tools-scripts/run-script-if@0.0.0","Apache-2.0","https://github.com/kiegroup/kie-tools"
"@kie-tools-scripts/sparse-checkout@0.0.0","UNLICENSED",""
"@kie-tools-scripts/update-version@0.0.0","UNLICENSED",""
"@nice-move/prettier-plugin-package-json@0.6.1","MIT","https://github.com/nice-move/prettier-plugin-package-json"
"@pnpm/filter-workspace-packages@5.0.27","MIT","https://github.com/pnpm/pnpm/blob/main/packages/filter-workspace-packages"
"@pnpm/find-workspace-packages@4.0.27","MIT","https://github.com/pnpm/pnpm/blob/main/packages/find-workspace-packages"
"@pnpm/logger@4.0.0","MIT","https://github.com/pnpm/logger"
"@types/eslint-scope@3.7.3","MIT","https://github.com/DefinitelyTyped/DefinitelyTyped"
"@types/eslint@7.2.10","MIT","https://github.com/DefinitelyTyped/DefinitelyTyped"
"@types/node@18.13.0","MIT","https://github.com/DefinitelyTyped/DefinitelyTyped"
"@types/prettier@2.2.3","MIT","https://github.com/DefinitelyTyped/DefinitelyTyped"
"@typescript-eslint/eslint-plugin@4.24.0","MIT","https://github.com/typescript-eslint/typescript-eslint"
"@typescript-eslint/experimental-utils@4.24.0","MIT","https://github.com/typescript-eslint/typescript-eslint"
"@typescript-eslint/parser@4.24.0","BSD-2-Clause","https://github.com/typescript-eslint/typescript-eslint"
"@typescript-eslint/scope-manager@4.24.0","MIT","https://github.com/typescript-eslint/typescript-eslint"
"@typescript-eslint/types@4.24.0","MIT","https://github.com/typescript-eslint/typescript-eslint"
"@typescript-eslint/typescript-estree@4.24.0","BSD-2-Clause","https://github.com/typescript-eslint/typescript-eslint"
"@typescript-eslint/visitor-keys@4.24.0","MIT","https://github.com/typescript-eslint/typescript-eslint"
"eslint-config-prettier@8.3.0","MIT","https://github.com/prettier/eslint-config-prettier"
"eslint-plugin-react-hooks@4.2.0","MIT","https://github.com/facebook/react"
"eslint-plugin-react@7.23.2","MIT","https://github.com/yannickcr/eslint-plugin-react"
"eslint-scope@5.1.1","BSD-2-Clause","https://github.com/eslint/eslint-scope"
"eslint-utils@2.1.0","MIT","https://github.com/mysticatea/eslint-utils"
"eslint-visitor-keys@2.1.0","Apache-2.0","https://github.com/eslint/eslint-visitor-keys"
"eslint@7.26.0","MIT","https://github.com/eslint/eslint"
"fast-xml-parser@4.1.2","MIT","https://github.com/NaturalIntelligence/fast-xml-parser"
"filemanager-webpack-plugin@7.0.0","MIT","https://github.com/gregnb/filemanager-webpack-plugin"
"graph-data-structure@2.0.0","MIT","https://github.com/datavis-tech/graph-data-structure"
"graphviz@0.0.9","GPL-3.0","https://github.com/glejeune/node-graphviz"
"husky@6.0.0","MIT","https://github.com/typicode/husky"
"jest-junit@14.0.0","Apache-2.0","https://github.com/jest-community/jest-junit"
"jest@26.6.3","MIT","https://github.com/facebook/jest"
"kie-tools-root@0.0.0","UNLICENSED",""
"markdown-table@3.0.2","MIT","https://github.com/wooorm/markdown-table"
"postinstall-postinstall@2.1.0","MIT","https://github.com/ds300/postinstall-postinstall"
"prettier@2.3.0","MIT","https://github.com/prettier/prettier"
"pretty-quick@3.1.0","MIT","https://github.com/azz/pretty-quick"
"rimraf@3.0.2","ISC","https://github.com/isaacs/rimraf"
"run-script-os@1.1.6","MIT","https://github.com/charlesguse/run-script-os"
"typescript@4.5.4","Apache-2.0","https://github.com/Microsoft/TypeScript"
"yargs@17.3.1","MIT","https://github.com/yargs/yargs"

There's a lot more.... working on those.

Surprisingly, the only dependency that I have found that's a problem for us is node-graphviz. It says MIT in the repo, the package.json file says GPL-3, and there is one file that contains a GPL-3 header.

@LightGuard @porcelli Not sure who should be reviewing this.

@LightGuard do we have at least a list based on current codebase of the libraries that we need work to replace?

@LightGuard I can review.. but I'm going to need instructions on where to look.

Understood. I'll get those instructions to you.
On Tue, Apr 4, 2023 at 10:11 AM Tiago Bento @.***>
wrote:

@LightGuard https://github.com/LightGuard I can review.. but I'm going
to need instructions on where to look.
—
Reply to this email directly, view it on GitHub
https://github.com/IBM/kie-roadmap/issues/24#issuecomment-1496246431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHKE25GTK2MOGGEA6USATW7RBZXANCNFSM6AAAAAAUTBKBJQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Jason Porter
http://en.gravatar.com/lightguardjp

@LightGuard @porcelli
FYI, Mend has an utility to export found licenses in excel:
IBM_Business_Automation_Manager_Open_Editions_kie-tools-inventory-report_Page_1.xlsx

@gitgabrio Would you like to manage this one? I never figured out how to get the right permissions to MEND. We'll need it for Apache. Of course, there seem to be a lot of duplications in the spreadsheet, but we can get that figured out.

@LightGuard What exactly would be required ? There is already some code bit, in our pipeline, that downloads all the licences for a given repo. I also have some postman script to do that. Sad note: right today we find out that the kie-tools repo/dashboard has some issue

@LightGuard
I've downloaded json of kie-tools-9 licenses. I'm gonna transform it to excel and let you know

Done? :D

There are several items listed as "Requires Review." We'll need to get those sorted out. A quick check looked like many of those are our jars, but we'll need to ensure we have those licenses figured out, but this is a great start. I wouldn't call it finished just yet though.

what is remaining on this task?

From an Apache perspective, we need to know if the licenses we're using are Category A, B, or X. If there are any Category X, we need to find alternatives.

I'm good to move this to done. We can reopen or create a new ticket if we need to address this again for an ASF release.


Deployment on Aura network

Chain Name
Aura Mainnet
Chain ID
6322
RPC URL
https://jsonrpc.aura.network
Block Explorer URL
https://aurascan.io
Deployment Funds
Yes

Done, thanks!