Backward compatibility policy

Using this issue to allow public discussion on Backward Compatibility Policy - this is to be a "companion policy" to the LTS issue. https://github.com/zowe/zlc/issues/72
Proposal is to post both at the same time.
Zowe LTS Backwards Compatibility.docx

Posting comments from email exchange:

what defines public vs non-public APIs? anything we published to the API catalog today out of the box? if this is what we have in mind, there's probably work to be done here since I don't think we document existing APIs in our Zowe docs today nor do we have Swagger docs for all APIs OOTB

We should do a full review APIs, but my starting point would be all APIs which are consumed by both "extenders" (building plugins) as well as "plain old users" (say, using out-of-the-box USS APIs or CLI commands). This type of categorization would include CLI commands but exclude ZSS APIs - ZSS APIs are consumed just within Zowe itself to my knowledge.

what options do we have wrt governance? How do we ensure we stay true to our commitments wrt non-breaking API changes? e.g. require automated test cases for all public APIs?

Test suites against APIs would be the gold standard and part of acceptance criteria for new code. When APIs go end-of-life, remove the test cases. Until then - incremental test coverage, code review, and planning/communicating...any other ideas for the short-term are welcome...

any particular reason why we decided to propose n-1 as opposed to n-2? (not saying it has to be n-2 necessarily, just want to hear your rationale)

Two main reasons:
'n-1' is the minimal compatibility guarantee we can give and is probably a good starting point. I'm cautious about statements which in practice would extend 3-5 years from now.
I feel like we're trying to toe a line between framework stability and evolvability. I have some concerns, potentially unfounded, that extended 'n-2' or 'n-3' compatibility guarantees would limit our capacity to evolve or remediate flaws in design that appear after extended adoption/growth. I also get the feeling that developers don't want to be locked into supporting decisions and APIs made, say, 4 years ago if we declare 'n-2' support. I may be wrong, and/or we need to become comfortable with that kind of approach.

wrt the Conformance program, personally I struggle with how its set up today. IMO, any certification program should be designed around major version boundary (1st digit) and not by year, it is the version (V1 vs V2 vs V3 ...) that uniquely defines the characteristics of a given release of project. Thoughts?

Agreed, I think our LTS and 'n-1' statements can lead the direction for the conformance program + conformance versions. Conformance can evolve within a major release (say Active LTS) to leverage new features or functions, and we should think about how to version and communicate that - but I like generally tying conformance to major releases at Active LTS+ stage. Current development stage - YMMV and nothing is promised (except n-1 backwards compatibility?  )

For completeness - in email exchange positive comment to the "alternate proposal"
I like the alternate proposal. It gives us an opportunity to learn without making a public commitment. The squads should work as if n-1 was already the rule during this time. It should be in their working agreement/manifesto.

ZLC today felt this has been addressed based on description of Current, Active LTS - closing as complete.


IITC stomps on the jqueryUI resizable option

Our plugin needs to have the ability to resize alert and other dialog windows inside IITC, but even though the code is correct, IITC seems to be preventing resizing of alert and dialog windows.

This repo is unmaintained, haven't you noticed?


No documentation for where clause in cds.ql

Looking at https://cap.cloud.sap/docs/node.js/cds-ql#where, there seems to be missing documentation for the where clause.
What I'm looking for:
e.g:

Current where doco:

Hi @sammok83,
the content below having() is also valid for where(). Got also confused about this, the first time when looking at it.
Here it is the same but a bit more obvious: https://cap.cloud.sap/docs/node.js/events#req-error
BR,
Nico

Hi @sammok83 ,
yes, @nocin described it correctly. I agree, that we don't make it as clear here as it should be. I'll fix that.
All the best,
René

In addition we changed the styling, so that it's visually more clear:

(to be released most probably end of November)


bug(@vtmn/css): enhancement needs to be done for Tabs component

The icons do not take the colour of the link as well as the focus:

To fix add color: inherit;  for small and medium :

There is an alignment problem when mixing with icon, badge or just text:

There is no active status for the tabs (not focus)

Hi @Mig-l,
Thanks! I close your PR because:

For the first point, it is fixed: https://github.com/Decathlon/vitamin-web/commit/f510e433f1c269260d1251d37c9cb9ea296c13de
For the second, we don't have this problem here on @vtmn/css showcase: http://localhost:6006/?path=/story/components-navigation-tabs--overview
For the third, it is fixed: https://github.com/Decathlon/vitamin-web/commit/f510e433f1c269260d1251d37c9cb9ea296c13de

@lauthieb for the second point it's because the VtmnIcon have a default variant. So, it apply --vtmn-semantic-color_content-primary


Brian/split

adds spend for https://github.com/mlabs-haskell/hedgehog-plutus-simple/issues/7

Closing this cause it should have just been part of #8


Inconsistencies, sometimes, on similar characters. Is there a list of mistakes for characters that are very similar?

Current Behavior
Hello, I am using Tesseract to extract some data from screenshots.
I've noticed that sometimes there are mistakes in interpreting characters like '0' and 'O', 'P' and 'R' or '-' and '—' or the other way around. This happen with the same font.
Expected Behavior
No expected behaviour since it's understandable that without OCR preprocessing such mistakes can occur.
Suggested Fix
Is there a comprehensive list/map of all characters that are usually mistaken recognised that are very similar?
I need that list in order to calculate effective string distance with Levenshtein.
tesseract -v
5.3.1
Operating System
macOS 13 Ventura
Other Operating System
No response
uname -a
No response
Compiler
No response
CPU
No response
Virtualization / Containers
No response
Other Information
No response

Please respect the guidelines for posting issues: use tesseract user forum for asking questions/support


Add CONTRIBUTING.md

I think we should indicate what format commitlint enforces, etc.

Sure, I'd like to add them soon.


Add the build type to ponyc -v output

This adds the type of build after the version number when running ponyc -v.
e.g.
0.2.1-944-ga9585df [debug]

@Perelandric this breaks the windows build:
"main.c
....\src\ponyc\main.c(255): error C2065: 'PONY_BUILD_CONFIG' : undeclared identifier

@SeanTAllen forgot about Windows. Not sure where to set that but I'll poke around a bit. First need to set up Pony in a Windows VM.

Don't think this is going to happen right now. I can just drop this PR.

I asked on the pony+dev list for Windows assistance. Hopefully we can bring this one home. It's a nice feature I'd like to merge and eventually see expanded. Thanks for the initial legwork @perelandric

Thanks @SeanTAllen. Much appreciated.

Ok, my ponyc fork () has a small patch to set this on Windows.  Should probably set it via premake, since it is set in the Makefile on linux, but I can't immediately see how to do this, so I just set it in the platform.h header.
diff --git a/src/common/platform.h b/src/common/platform.h
index 598e671..d85a1c0 100644
--- a/src/common/platform.h
+++ b/src/common/platform.h
@@ -284,6 +284,13 @@ typedef __int128_t dw_t;

 #if defined(PLATFORM_IS_WINDOWS)
 #  include "vcvars.h"
+
+#ifdef NDEBUG
+#define PONY_BUILD_CONFIG "release"
+#else
+#define PONY_BUILD_CONFIG "debug"
+#endif
+
 #endif

 #endif

Is there a procedure for merging from different forks?
Wish I had a Windows newer than XP. May need to remedy that. Eyeballing the premake file, seems like maybe each configuration just needs another defines, like...
defines "PONY_BUILD_CONFIG=\"release\""

Yeah, that works much better.
diff --git a/premake5.lua b/premake5.lua
index 3a29454..afd30de 100644
--- a/premake5.lua
+++ b/premake5.lua
@@ -89,15 +89,18 @@
       targetdir "build/debug"
       objdir "build/debug/obj"
       defines "DEBUG"
+      defines "PONY_BUILD_CONFIG=\"debug\""
       flags { "Symbols" }

     configuration "Release"
       targetdir "build/release"
       objdir "build/release/obj"
+      defines "PONY_BUILD_CONFIG=\"release\""

     configuration "Profile"
       targetdir "build/profile"
       objdir "build/profile/obj"
+      defines "PONY_BUILD_CONFIG=\"profile\""

     configuration "Release or Profile"
       defines "NDEBUG"

I don't know how to deal with multiple forks in github either.

Perhaps it would be easiest for you to add this to your pr branch rather than trying to merge from mine.

Sounds good, @kulibali. I'll do that.

@SeanTAllen Tests are passing. Made a mistake while updating and ended up with everything in a single commit while correcting, so this most recent change is just the premake5.lua file.

Thanks, @Perelandric and @kulibali!


feat(frontend): gather some anonymous analytics about transactions duration

Motivation
We discussed with @DSharifi if there was an easy way to gather analytics and insights about the duration of processes as perceived by users. As a result, Daniel had the idea to persist the duration of function executions as anonymous metadata in the events we save in Juno analytics. Basically could potentially be a quick win. This PR provides some of the first metrics.
Changes

Add two new functions in the analytics modules to initiate events and ultimately submit those once a function is executed
Track those kind of events for ETH, IC, ICRC and ERC20 transactions

Thanks for the quick work! @peterpeterparker


Third Party Domain Verified, but Timesout ERR_CONNECTION_REFUSED When Visiting

App Id: d20hiszf5421db
Region: us-east-2
Step: Complete
Status: Verified but timesout
Issue/question
I have successfully validated my domain which is hosted on Siteground. I only added a subdomain, as the primary domain points to a website hosted elsewhere. I did this entirely through the Amplify console, I did not create a hosted zone on Route 53 and did not do anything with CloudFront.
However when typing in the url, it just timesout, it doesn't load my site. I get:
This site can’t be reached.
www.portal.example.com refused to connect.
ERR_CONNECTION_REFUSED
For example, my primary domain is example.com and I've only added the subdomain: portal.example.com.
I have tried deleted the linked domain, and starting over. It validated again, but still timesout. It has been >24 hours since validating the domain.

Found my answer here: https://stackoverflow.com/questions/59240000/aws-amplify-subdomain-redirect-to-www-admin-mydomain-com-showing-err-connectio
I had to go to App Settings, at the bottom, Rewrites and Redirects, and remove the redirect from portal.example.com to www.portal.example.com


Change color showPageIndicator

Question
how can i change the color of showPageIndicator in ios

Currently, we don't support this kind of feature.

This would be a nice add on, especially for dark mode.


Create Shell Backend

Create a shell backend for "ShellTensors" that only store their shape.

I would be interested in looking into this and/or #75. I guess the plan is to have the current BaseBackend as a reference and replace all its methods by "shape methods". Perhaps we might need to modify that later depending on the needs of the ContractionNetwork. Let me know if you have any other ideas.

Great!
I would recommend doing this first as #75 will require this.


Adding 2-way sync (SD-to-HAss) for state_entity_id

Basic skeleton for updating the state_entity_id in HAss when the streamdeck is manually turned on or off.
For now, I am only doing this if the state_entity_id is an input_boolean. In the future, I would like to make this configurable by adding another global config setting (e.g. state_bidirectional_sync).
But before moving forward, basnijholt needs to confirm I didn't manage to break anything :)

Awesome! Thanks a lot, just pushed a small change.


chore: Build via microbundle, add ESM for real

Expose caip library as ESM, CJS, UMD.
Rationale: Apparently, ESM is advertised in package.json and is not really built. It definitely breaks applications built on Vite. Only CJS and UMD builds are available. Adding ESM for real is slightly more than just building with tsconfig.esm.json. This PR adds ESM build with no changes to the codebase.
Tested ESM availability using import.meta.resolve('caip') and import('caip').
Tested CJS availability using require.resolve('caip') and require('caip').

Currently running into this problem.
Importing caip-js or packages that depend on it into my svelte project causes Vite to an error on build, with the following message:


First pass at docs for flags and commands

Feedback welcome, please use in the PR inline suggestion tool.

Feedback welcome, please use in the PR inline suggestion tool.

I eventually found the suggestion tool, but before that I ended up creating a commit to fix a typo.


Support for o1-mini and o1-preview

Since there is now API access to these models, could you please add support for them. I've tried generating a new API key, however I still get the following error:  state=finished raised KeyError

Doc: https://platform.openai.com/docs/guides/reasoning

There is no compatible issue i think. Do you have the required tier to use this model?

Ah, that is probably the issue. I am only tier 1. I have however recently become able to interact with the API via the playground:

Thanks for taking the time to respond.

Can you share me your config with sensitive information redacted.

The container log:

2024-11-20 12:15:07,453 - INFO - matrix chatgpt bot start.....
2024-11-20 12:15:10,841 - INFO - Successfully login via password
Mismatch in keys payload of device MatrixChatGPTBot (xxxxx) of user @bot:matrix.org (@bot:matrix.org).
2024-11-20 12:15:25,930 - INFO - Message received in room XXXX
User | !gpt hello
2024-11-20 12:15:30,729 - gpt - ERROR - RetryError[<Future at 0x7fb025e2f790 state=finished raised KeyError>]
2024-11-20 12:15:30,729 - ERROR - RetryError[<Future at 0x7fb025e2f790 state=finished raised KeyError>]
2024-11-20 12:15:32,003 - INFO - Message received in room XXXX
bot | > <@user:matrix.org> !gpt hello
Something went wrong, please try again or contact admin.

My config:

HOMESERVER="https://matrix.org"
USER_ID="@bot:matrix.org"
PASSWORD=xxxxx
DEVICE_ID="MatrixChatGPTBot"
ROOM_ID="!xxxxx:matrix.org"
OPENAI_API_KEY="xxxxxxxxxxxxxx"
GPT_API_ENDPOINT="https://api.openai.com/v1/chat/completions"
GPT_MODEL="o1-preview"
MAX_TOKENS=4000
TOP_P=1.0
PRESENCE_PENALTY=0.0
FREQUENCY_PENALTY=0.0
REPLY_COUNT=1
SYSTEM_PROMPT="You are ChatGPT, a large language model trained by OpenAI. Respond conversationally"
TEMPERATURE=0.8
TIMEOUT=120.0

Yes, other models including gpt-3.5-turbo work fine. The curl request returned the following:
BadRequestError: Error code: 400 - {'error': {'message': "Unsupported value: 'messages[0].role' does not support 'system' with this model.", 'type': 'invalid_request_error', 'param': 'messages[0].role', 'code': 'unsupported_value'}}
Which in turn, lead me to stackoverflow. And subsequently the creation of the following python script - which does work with the o1-mini and o1-preview models:
import os
import requests
import json

# Retrieve your OpenAI API key from an environment variable
api_key = os.getenv("OPENAI_API_KEY")

# Check if the API key is available
if not api_key:
    raise ValueError("Please set the OPENAI_API_KEY environment variable.")

# Set up the URL and headers for the request
url = "https://api.openai.com/v1/chat/completions"

headers = {
    "Content-Type": "application/json",
    "Authorization": f"Bearer {api_key}"
}

# Define your prompt
prompt = "Can you tell me a joke?"

# Structure the messages accordingly
messages = [
    {
        "role": "user",
        "content": prompt  # Use the prompt directly as a string
    }
]

# Define the data payload for the request
data = {
    "model": "o1-preview",
    "messages": messages
}

# Send the POST request
response = requests.post(url, headers=headers, json=data)

# Check if the request was successful
if response.status_code == 200:
    # Extract the assistant's reply content
    reply_content = response.json()["choices"][0]["message"]["content"]

    # Print the assistant's reply (since it's already a string)
    print("Response from GPT:", reply_content)
else:
    # Print the error
    print(f"Status Code: {response.status_code}")
    print("Response:")
    print(response.json())

Try: ghcr.io/hibobmaster/matrixchatgptbot:sha-c2d6ecfaf62f539f9e5ac0d321aa6c56002ec437

That version works great with both o1 models. Thank you.


Python support

I'd love to have python support so someone could clone a typical Django project and run strappy bootstrap, strappy server, strappy test, and strappy console and have the project running without any further knowledge of Python.

[ ] Pip (see bundler for inspiration)
[ ] Django (see rails for inspiration)
[ ] … what else?

See the docs for more info on defining new languages & frameworks.

Bootstrap:

.python-version => Use pyenv for to install Python
requirements.txt => Use pip to install dependencies
Pipfile => Use pipenv to install dependencies

Server:

manage.py => This project might be Django: python manage.py runserver

Test:

manage.py => This project might be Django: python manage.py test
pytest.ini => Call pytest to run tests instead?

Console:

manage.py => This project might be Django: python manage.py shell


Provider config variable adminApiKey is obsolete

What happened?
The Pulumi New Relic provider has a config option called adminApiKey. However, the upsteam Terraform provider has no such option (at least in the docs). I even went back a major version and did not see such an option in provider config.
I'm working on a New Relic workshop, and my code appears to work fine by only specifying accountId and apiKey, so this leads me to believe adminApiKey is obsolete. It's also very confusing because there is no description for what it's even supposed to represent: not on Installation and Configuration and not on the provider's API docs page.
Expected Behavior
n/a
Steps to reproduce
n/a
Output of pulumi about
n/a
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @jkodroff - thank you for filing this! We'll clean this up as soon as we can.
I verified with the NewRelic documentation that this key is in fact deprecated since 2020.
https://docs.newrelic.com/docs/apis/intro-apis/new-relic-api-keys/#key-details --> "Other API Keys"

@guineveresaenger Note that the registry docs will also have to be updated for the Install and Config page: https://www.pulumi.com/registry/packages/newrelic/installation-configuration/

Upvoting as we have another upcoming New Relic workshop and would be great to have this addressed by then (October 2024)


Description on archived Javascript

I noticed the following descriptions on executing archived JS.

Note: Only the wget extractor method executes archived JS when viewing snapshots, all other archive methods produce static output that does not execute JS on viewing. If you are worried about these issues ^ you should disable the wget extractor method using archivebox config --set SAVE_WGET=False.

Source: https://github.com/ArchiveBox/ArchiveBox#security-risks-of-viewing-archived-js

Workarounds
Disable the wget extractor by setting archivebox config --set SAVE_WGET=False, ensure you are always logged out, or serve only a static HTML version of your archive.

Source: https://github.com/ArchiveBox/ArchiveBox/security/advisories/GHSA-cr45-98w9-gwqx
I think the the SAVE_DOM archive method could also lead to the similar issue. When viewing Chrome > HTML ./output.html, any remote javascript will be loaded and executed.
Is that right? If so, we should also document this and remind users to disable this option if they should worry about the XSS/CSRF issue.

You're right, it used to be stripped before we expanded it to be the full outerHTML with <head>, but it it's included now. Thanks!
I'll update the CVE and notices.


Update to iTwin 4.7.2 (and newer appui)

For some unknown reason, this caused our three remaining class-based React components to fail to compile, so those were converted to functional components.

@toddsouthenbentley Please carefully review the class to functional React component conversions. I think I tested them, but I'm somewhat uncomfortable about things like the fact that the useEffect in TouchCaptor triggers every time isTouchStarted changes. It needs to do that, but since that removes the old listeners and creates new ones, I'm a little leery of it.

@toddsouthenbentley Please carefully review the class to functional React component conversions. I think I tested them, but I'm somewhat uncomfortable about things like the fact that the useEffect in TouchCaptor triggers every time isTouchStarted changes. It needs to do that, but since that removes the old listeners and creates new ones, I'm a little leery of it.

It looks like appui-react now has a usePointerCaptor hook that works for both mouse and touch events. I would suggest using that and getting rid of the Touch* classes/functions.
If you're still leery of the new code then I'd suggest staying with the old code and figuring out why it no longer builds.

It looks like appui-react now has a usePointerCaptor hook that works for both mouse and touch events. I would suggest using that and getting rid of the Touch* classes/functions.

It's not at all obvious how I would update the code to use that, and it looks like that would be an even bigger rewrite than converting a class to a function.

If you're still leery of the new code then I'd suggest staying with the old code and figuring out why it no longer builds.

I am investigating that, but the reality is that changing the class components to function components is a good thing. Given that everything works (resizing the resizable bottom panels), I think my conversion is correct.

Updating @types/react to 17 seems to fix the problem that prevented class-based components from being used. I can revert the class-to-functional changes, but since they are generally a good thing, I'm a bit hesitant to do so.

@toddsouthenbentley As you noticed, I reverted the components I was leery about to their original class versions after fixing the problem causing the compilation errors. I left ActionSheetButton as a functional component because it was much simpler and I don't have reservations about it.


App version to be changed in Github release notes

App version is 1.3.15 but, the github release notes is v1.3.14

In Github release notes

App version


Hamburger Menu doesn't collapse when you click on the page

Expected Behavior
When the user clicks on the page (outside of the menu) the menu should collapse.
Current Behavior
The menu stays exposed until the user selects the "in" arrow at the top of the menu.
Possible Solution
Not sure.
Steps to Reproduce

Navigate to https://www.groa.us/explore
Expand the hamburger menu.
Click on the page outside the hamburger menu
Observer the menu remains expanded.

Context (Environment)
The menu that appears when you click the User icon on the right side of the page does disappear when you click anywhere else on the page.

solved issue with menu. It now closes when you tap on page. will create a pull request


SSL_load_client_CA_file unable to load "TRUSTED CERTIFICATE"

Make SSL_load_client_CA_file api (https://www.openssl.org/docs/man1.1.1/man3/SSL_load_client_CA_file.html) load 'TRUSTED CERTIFICATE' created using https://www.openssl.org/docs/man1.1.1/man1/x509.html with -trustout option.
CLA: trivial

Travis failed with "An error occurred while generating the build script." twice on MacOS. seems a Travis build error.
Should pr be re-submitted?

This change would have to be done on master branch as this is not just a bug fix.


Feature: Support CMYK Color Mode

CMYK Support
This PR adds support for the CMYK color mode. It adds CMYK to RGB (and vice versa) conversion functions, and handles parsing CMYK input when initializing a new instance. Test cases have also been added.
Use Case
I came across this project while working on web based design software in which the end result is a print file. CMYK is the industry standard when it comes to print, so supporting this inside tinycolor would be very useful.
Example

@scttcper, do you have any interest in this? If not, we can close this out, and I can maintain a fork with it.

@ChandlerVS Just difficult to review without learning it myself. Let's mash it


Remove gosu

@kalitad Could you show me how your modified docker entrypoint looks like, removing gosu is causing some permission problems with the entrypoints commands... Might also be the mounted volume.
docker run -i -P alpine-postgres
The files belonging to this database system will be owned by user "postgres".
This user must also own the server process.

The database cluster will be initialized with locale "en_US.utf8".
The default database encoding has accordingly been set to "UTF8".
The default text search configuration will be set to "english".

Data page checksums are disabled.

fixing permissions on existing directory /var/lib/postgresql/data ... initdb: could not change permissions of directory "/var/lib/postgresql/data": Operation not permitted
sed: /var/lib/postgresql/data/postgresql.conf: No such file or directory
...

Hi @kiasaki , check attached files. For me it's working.
docker-entrypoint.TXT
Dockerfile.TXT

Ah, I see, this Dockerfile uses the VOLUME instruction and chowning the PGDATA dir in the Dockerfile doesn't seem to do the trick, I need to do it in the docker-entrypoint.sh (where it is done ATM using gosu)
You get away with it by having the data reside in the container using: RUN mkdir -p $PGDATA && \chown -R postgres $PGDATA. I am guessing most people will prefer a folder to be mounted so that their data persists after the container gets killed.

Just a note, you no longer have to install and remove curl, further simplifying the Dockerfile.

Thanks @Remco is that new in Alpine Linux version 3.3 or is it only added in the "official" alpine:3.3 docker image.
We are using gliderlabs/alpine:3.3 as a base image in this repo, does it still apply?

It has little to do with the base image. In the original Dockerfile the package curl was installed, then curl was used to download gosu and then curl was removed again. Since this patch removes the only use of curl, there is no need to install and remove it.
Like this commit.

Ha, sorry totally misunderstood the comment. Indeed, curl is not needed anymore where we used to fetch gosu

Why remove gosu?

Mostly one less dependency / install steps.
But at this point it would just be a disservice as to support uses cases where you mount volume we need to call chmod using gosu from in docker-entrypoint.sh.


Upgrade protobuf to >=5.0

resolves #218
Description
Upgrade protobuf to >=5.0,<=6.0
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

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

Project coverage is 68.50%. Comparing base (1c9da9f) to head (4c0f2fd).

Files with missing lines
Patch %
Lines

dbt_common/events/types_pb2.py
60.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #219      +/-   ##
==========================================
+ Coverage   68.48%   68.50%   +0.01%
==========================================
  Files          52       52
  Lines        3389     3391       +2
==========================================
+ Hits         2321     2323       +2
  Misses       1068     1068

Flag
Coverage Δ

unit
68.50% <60.00%> (+0.01%)
:arrow_up:

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

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


Getting Subscription SKU display names via API

Is there any way to pull this list of products via the graph API (or other API) so an application could always have an up to date list of SKU display names?
All the types that are connected to subscription skus in microsoft graph (ex: https://docs.microsoft.com/en-us/graph/api/resources/subscribedsku?view=graph-rest-1.0) seem to include the SkuId and skuPartNumber, but not a display name. Am I missing anything or is this documentation page the only place where this mapping of skuPartName to display name exists?
Thanks

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

ID: 7c245452-161d-89a1-c026-99a0721cf071
Version Independent ID: 16dec59a-eb83-1837-4efd-4645e7aeaed2
Content: Product names and service plan identifiers for licensing - Azure AD
Content Source: articles/active-directory/users-groups-roles/licensing-service-plan-reference.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @kkern
Microsoft Alias: kakern

@SHAGGAR
Thanks for your feedback! We will investigate and update as appropriate.

@SHAGGAR
Thank you for your time and patience!
This documentation is the only place where you can find the mapping of skuPartName to display name. As stated within the article:

If you'd like to create a feature request you can do so using our User Voice forum or the Microsoft Graph Feature Request forum.
Thank you for your time and patience throughout this issue.

@SHAGGAR in case this is of any use to you, I came across this link in the past:
https://go.microsoft.com/fwlink/p/?linkid=834456
If you can figure out the source they used to create that spreadsheet, you might have what you're after.  And in case my PSD1 file is of any user, I created it from the linked spreadsheet:
https://github.com/JeremyTBradshaw/PowerShell/blob/master/psd.MSOnlineServicesSkuPartNumberFriendlyNames.psd1
I can't wait until this stuff is available through MS Graph.


NetworkX version issue

What happened?
Hi, I am encountering a problem about networkX versions.

If not using the docker image but manually install by Python, networkX 3.0 will be used, and the function to_scipy_sparse_matrix has been renamed to to_scipy_sparse_array, and it will crash the program.
Please specify that the networkX dependency version to 2.8.2 in later releases, or to update the function name in fmriprep.
Thanks.
What command did you use?
fmriprep /data /out participant -w /tmp

What version of fMRIPrep are you running?
22.1.1
How are you running fMRIPrep?
Local installation ("bare-metal")
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
No response
Additional information / screenshots
No response

This is an issue with nipype. You can either install networkx<3 or you can install nipype from the latest master.
There is nothing to do in fmriprep.


Dev App throws error after change made to file

PhoneGap dev app working fine for me (win7), but after making a change to a file, I see the following in the dev app screen:
Uncaught node.js Error
Error: watch EPERM
at exports._errnoException (util.js:742:11)
at FSEvent.FSWatcher._handle.onchange (fs.js:1045:26)

Hi there @CCDSystems
Just tested on my windows machine and it seems fine there. What version of phonegap are you running? If you aren't on 4.1.2-0.22.12, you might want to try and upgrade phonegap by doing npm install -g phonegap.
Otherwise, it seems like you're running into a permission issue. You might want to try running the command prompt or your git bash as an administrator and then try again. Let me know if that helps.

Closing since we haven't seen very much activity on this issue. Feel free to reopen.


[QUESTION] unable to build on windows (CURL 32)

HI.
I'm trying to build on windows 10 and I've this error, somebody knows what could be?
I'm using VSC 2019 on x64 native Tools Command prompt
If anyone could help me I'll be grateful.

We build on Windows in our Travis CI:
https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/blob/master/.travis.yml#L108-L117
Can you please try that in case you're doing something different?

We build on Windows in our Travis CI:
https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/blob/master/.travis.yml#L108-L117
Can you please try that in case you're doing something different?

I still have the same error and I had to reset the pc, currently I was able to execute the command "Nmake MakeFiles" and the files were created correctly, however, when I running the command "cmake .. -DBUILD_GSTREAMER_PLUGIN = ON -DBUILD_JNI = TRUE" it gives me an error this error Could NOT find JNI (missing: JAVA_AWT_LIBRARY JAVA_JVM_LIBRARY
can anybody help me?

@Lepm11 ,
We recently updated CMake version in this PR which could solve the issue: https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/pull/748
It has not made to the release yet. Can you see if updating CMake helps your case?

So far i have not been able to install on windows 10, i tried on ubuntu 20.04 and it worked fine. Windows generates many errors


100% CPU usage after a time and unable to get RTSP stream anymore

First of all, thank you for your solution, it's very helpful.
I am using your code in docker container: https://github.com/humpedli/docker-xiaomi-aqara-rtsp
Unfortunately I discovered that the CPU usage is increasing to 100% after a time and I am unable to get RTSP stream anymore. This time means like 8-16 hours. Could you please suggest me something how to debug this issue? There is no error on console output.

I'm running this in a dedicated VM (to stop the 100% CPU bug from taking down my home server) and my workaround has been to have a cron script that kills the script and re-runs it every 3 hours. It means ~3 seconds of downtime every 3 hours, which is fine by me.
I did have a poke through the code, but I know nothing about C (I'm a PHP / Node.js guy) so I can't help much there, other than thinking that it might be an out of control loop or something.


Non-utf8 filenames permanently prevent cloud storage source from syncing

Describe the bug
If the cloud storage importer encounters any filenames which contain non-utf8 characters, they will cause the importer to permanently fail to import any files from cloud storage.
To Reproduce
Steps to reproduce the behavior:

Create a file in cloud storage which contains non-utf8 Surrogate pairs, such as an emoji or certain Chinese, Japanese, and Korean characters
Attempt to sync cloud storage

You will see that the sync operation fails:

The following error message appears from the Python django backend:
Error logs for localfiles storage 5 in project 1 and job null:

  File "/usr/local/lib/python3.10/dist-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
UnicodeEncodeError: 'utf-8' codec can't encode characters in position 48-71: surrogates not allowed

meta = {"attempts":2,"duration":0.321591,"time_queued":"2023-10-05 18:13:08.252656+00:00","time_failure":"2023-10-05 18:13:08.577589+00:00","time_last_ping":"2023-10-05 18:13:08.255998+00:00","time_in_progress":"2023-10-05 18:13:08.255998+00:00"}

And any filenames containing non-utf8 characters will permanently be added as entries to the task and io_storages_localfilesimportstoragelink tables and are not removed upon deletion of the cloud storage object. The presence of these entries will cause any future import operations to fail:
SELECT * FROM io_storages_localfilesimportstoragelink WHERE NOT (key ~ E'[\\uD800-\\uDFFF]');

SELECT * FROM task WHERE NOT (data::text ~ E'[\\uD800-\\uDFFF]');

Expected behavior
The program should drop entries which have non-utf8 Surrogate pairs and provide an appropriate error message to the user. This would prevent these entries from causing all future cloud storage operations to fail
Screenshots
See above
Environment (please complete the following information):

OS: AlmaLinux release 8.8 (Sapphire Caracal), official label-studio docker container
Label Studio Version 1.9.0

Additional context
This issue may be considered high priority for triage given that it may frequently occur for non-English language users for any filenames containing emojis or Chinese, Japanese, and Korean characters, and such an occurrence will permanently impair the operation of label-studio.

If any other users encounter this issue, here is my current mitigation:

shell into to the postgres docker container:

docker exec -it 7eba5a17b80a bash

backup your datbase:

pg_dump -U postgres -d postgres > backup.sql

use the psql utility as the user postgres:

psql -U postgres

connect to the postgres database:

\c postgres

List the relations in the database using \dt:

\dt

Determine if any key in the io_storages_localfilesimportstoragelink table contains invalid characters:

SELECT * FROM io_storages_localfilesimportstoragelink WHERE NOT (key ~ E'[\\uD800-\\uDFFF]');

Remove the invalid entries from io_storages_localfilesimportstoragelink table:

DELETE FROM io_storages_localfilesimportstoragelink WHERE NOT (key ~ E'[\\uD800-\\uDFFF]');

Determine if any data in the task table contains invalid characters:

SELECT * FROM task WHERE NOT (data::text ~ E'[\\uD800-\\uDFFF]');

Remove the invalid entries from task table:

DELETE FROM task WHERE NOT (data::text ~ E'[\\uD800-\\uDFFF]');

Delete the cloud storage object in the settings interface on the label-stuidio webui settings for your project

Back up your dataset files

Rename all files in your dataset which contain non-ascii characters in their filename:
remove-non-ascii.py

import os
import sys

def remove_non_ascii(string):
    return ''.join(i for i in string if ord(i) < 128)

def rename_files_in_dir(dir_path):
    for root, dirs, files in os.walk(dir_path):
        for file in files:
            new_file_name = remove_non_ascii(file)
            if new_file_name != file:
                old_file_path = os.path.join(root, file)
                new_file_path = os.path.join(root, new_file_name)

                # Rename the file
                os.rename(old_file_path, new_file_path)
                print(f"Renamed: {old_file_path} -> {new_file_path}")

if __name__ == "__main__":
    if len(sys.argv) != 2:
        print("Usage: python script_name.py [root_directory_path]")
        sys.exit(1)

    root_dir = sys.argv[1]
    if not os.path.exists(root_dir):
        print(f"Error: The directory {root_dir} does not exist.")
        sys.exit(1)

    rename_files_in_dir(root_dir)

python3 remove-non-ascii.py /home/user/DATASET-DIRECTORY-HERE

Add your dataset to project cloud storage using a new cloud storage object

Thanks for the detailed report. I've brought this to the engineering team, and we're looking into solutions to fix this issue.

This fix worked for me!
On my app (version 1.12.0_post0) the postgres database was empty, and the one holding everything was called labelstudio.
For some reason the SELECT command flagged every file and every task in the io_storages_localfilesimportstoragelink and tasks tables as containing surrogates. All apart from 1 seemed like normal filenames to me, and I couldn't see many source files getting renamed by the renaming script afterwards either. Maybe all the 'okay' filenames got tainted by one bad entry?
Well, it works now (somehow)!


[FancyZones] 0.21.1 FancyZones no longer functional

ℹ Computer information

PowerToys version: 0.21.1
PowerToy Utility: FancyZones
Running PowerToys as Admin: No
Windows build number: [run "winver"] 20H2
Monitor setup: 1 x 5120x1440 Samsung CRG9 49" Ultrawide, DisplayPort connected

📝 Provide detailed reproduction steps (if any)

Launch Power Toys, Enable FancyZones
"Allow Zones to span across monitors" NOT ticked (single monitor setup)
Launch Zones Editor - comes up with "Focus" mode
Select any other zone layout, or try and edit to create your own, click apply
Zone Editor appears to crash, and the selected zones are not retained - open Zone Editor and you are back at "Focus" layout
No zone layout is actually active - shift and drag, no highlight, no zones to activate, and while dragging the window around to find the zone Explorer silently crashes (Fault Module Name: ucrtbase.dll).

FZ is definitely "active" at least in part as the dragged window reacts to "Make dragged window transparent" setting in FZ.
✔ Expected result
What is the expected result of the above steps?
Zone Editor retaining setup, and usable zones.
❌ Actual result
What is the actual result of the above steps? See repro steps above - no usable FZ, and Explorer crashes.
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

@Swallowtail23

"Allow Zones to span across monitors" NOT ticked

To clarify: the option is not selected, correct?

@Swallowtail23

Zone Editor appears to crash

If it crashes, there should be a crash report in the Event log, open Event Viewer and go to Windows Logs -> Application

@Swallowtail23

FZ is definitely "active" at least in part as the dragged window reacts to "Make dragged window transparent" setting in FZ.

If you turn off the option to make the window transparent, Explorer does it still crash while dragging the window?

@Swallowtail23

"Allow Zones to span across monitors" NOT ticked

To clarify: the option is not selected, correct?

Correct, the option is NOT selected.

@Swallowtail23

Zone Editor appears to crash

If it crashes, there should be a crash report in the Event log, open Event Viewer and go to Windows Logs -> Application

No crash report in Event Viewer, so Zone Editor is not crashing. And checking the json file it is writing the selected layout to json (e.g. "type":"priority-grid"},"editor-show-spacing":true,"editor-spacing":4 etc.).

@Swallowtail23

FZ is definitely "active" at least in part as the dragged window reacts to "Make dragged window transparent" setting in FZ.

If you turn off the option to make the window transparent, Explorer does it still crash while dragging the window?

Yes, Explorer still crashes, with transparency on OR off. I can post Event viewer details if needed?

Some more troubleshooting - it is DisplayFusion causing the Explorer crash, not FancyZones. It has not been doing it before, and Display Fusion has not been updated - but I can reproduce the crash with DF not FZ.
So on this single display PC, FZ works OK with DisplayFusion not running, I'll close this one. Separate issue for span across monitors but that's in the other ticket.


Implementing Searching YouTube videos

How should the implemention for searching YouTube videos look like?
Should it be an implemention of the Youtube Search API (https://developers.google.com/youtube/v3/docs/search) oder more like a scrapper like these inofficial APIs (i.e. https://github.com/damonwonghv/youtube-search-api)?

Done using https://github.com/Nolkaloid/godot-yt-dlp/pull/12


Make manager emails send again

The manager emails that were updated in #2296 are no longer sending.

On at least some of the methods (and possibly all), it looks like the code that sends email deliberately excludes the depositor if it the same as the collection manager:
https://github.com/sul-dlss/happy-heron/blob/main/app/services/collection_observer.rb
In other words, if the collection manager themselves makes a deposit to that collection and then makes changes that would trigger an email, they won't get the email (since they are both the depositor and the manager).  If there was a second collection manager, they would get the email.  If there is only one manager and that matches the depositor, then no emails go out.
Can we confirm if this is the desired behavior and could possibly be what's happening.

@peetucket Ah, okay. Thanks for clarifying that. I'll have to find another tester to help me out with this, so might take a bit.

Note: we may also have some issues when owner is not the same as depositor with some of the logic in https://github.com/sul-dlss/happy-heron/blob/main/app/services/collection_observer.rb (i.e. it may be excluding depositors instead of owners).  See #2654

@peetucket I got Cathy to help me test this and these are indeed working.
Should we leave this ticket open but change it to reflect the note you made above? Or maybe open a new ticket to look into that?

Great!  I'd suggest leaving ticket as is for now and briefly discussing after standup.  I think it makes sense to switch to using the owner in the related spots of the code too -- to me, once you switch a work to a new owner, that's a signal that the original depositor should not be used for any more of emails anymore.

Should be fixed by #2676


DOC: special: add parameters/returns sections for Bessel integrals

Reference issue
None
What does this implement/fix?
The functions

itj0y0
it2j0y0
iti0k0
it2i0k0
besselpoly

are missing parameters/returns sections in their docstrings; add
those. Also write the definitions using LaTeX, which makes them much
easier to read.
Additional information
None

The CI failures appear to be unrelated.


perf: use v2 to write the sorted IVF partition files

The old v1 approach created a row group for every partition (including empty partitions).  This approach converts the input into a List<Struct<...>> array where each row is a partition of data.  Empty partitions are not included in the array.
This yields significant performance benefits.  I'm not 100% sure if the benefit is changing to v2 or simply changing the format in which we write the data.

Testing locally I see a bunch of failures from test_create_ivf_hnsw_with_empty_partition.  I suspect the issue is that the call to shuffle_dataset is running more quickly than it did before and this is making it possible to write to more files which is causing the open files error to be more prevalent but I don't actually know.

Testing locally I see a bunch of failures from test_create_ivf_hnsw_with_empty_partition. I suspect the issue is that the call to shuffle_dataset is running more quickly than it did before and this is making it possible to write to more files which is causing the open files error to be more prevalent but I don't actually know.

shuffle_dataset would be called only once during the entire indexing progress.
is that a recall assertion failure? I fixed such an issue before

We never got a chance to look at this, but Rob was claiming just the act of concatenating batches improved performance, but I was skeptical. Maybe there's something to that?

Yeah, I have to concatenate for v2 (v2 can write multiple arrays but I can't make a list array from multiple arrays) so that might explain it.

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

Please upload report for BASE (main@9703e50). Learn more about missing BASE report.
Report is 7 commits behind head on main.

Files
Patch %
Lines

rust/lance-index/src/vector/ivf/shuffler.rs
84.33%
39 Missing and 16 partials :warning:

rust/lance-index/src/vector/ivf/builder.rs
0.00%
26 Missing :warning:

rust/lance/src/index/vector/ivf.rs
47.50%
21 Missing :warning:

rust/lance-file/src/v2/writer.rs
87.50%
0 Missing and 2 partials :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2492   +/-   ##
=======================================
  Coverage        ?   79.58%
=======================================
  Files           ?      208
  Lines           ?    59368
  Branches        ?    59368
=======================================
  Hits            ?    47249
  Misses          ?     9394
  Partials        ?     2725

Flag
Coverage Δ

unittests
79.58% <76.62%> (?)

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

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

Closing as this will be tackled piece by piece in other PRs.


Json check

Todo: tests

Codecov Report

Merging #470 into master will decrease coverage by 1.05%.
The diff coverage is 40.29%.

@@            Coverage Diff             @@
##           master     #470      +/-   ##
==========================================
- Coverage   80.11%   79.05%   -1.06%
==========================================
  Files          37       37
  Lines        2509     2574      +65
  Branches      155      158       +3
==========================================
+ Hits         2010     2035      +25
- Misses        453      493      +40
  Partials       46       46

Impacted Files
Coverage Δ

cabot/urls.py
96.77% <ø> (ø)
:arrow_up:

cabot/rest_urls.py
91.48% <100%> (+0.18%)
:arrow_up:

cabot/templates/base.html
99.45% <100%> (ø)
:arrow_up:

cabot/cabotapp/models.py
69.66% <14.63%> (-4.04%)
:arrow_down:

cabot/templates/cabotapp/_statuscheck_list.html
80.8% <60%> (-1.17%)
:arrow_down:

cabot/cabotapp/views.py
68.88% <82.35%> (+0.5%)
:arrow_up:

Continue to review full report at Codecov.

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

Oh, interesting !

Can you elaborate in the commit message on the reasons why this new check is useful ?
Looks like there is some overlap with HTTP check. Could that have been done as part of the HTTP check ?

@JeanFred main use case was for stuff like celery stats endpoint (which we have a horrrible regex hack for), and some other hypothetical i can't remember right now. But it's useful for basically anything that exposes a json endpoint
There is a lot of overlap with httpcheck... I was actually thinking of refactoring to have

(Abstract?) http check that only has status code check
Regex http check
Json http check

Alternatively it could be 1 type of check with a drop-down for check type? would be more front-end work though

This looks very interesting indeed. I am curious to see if this allows me to use events from sensu-api as cabot checks.

Closing as it's super out of date.


Move OUTPUT_MODE logic from rcS to rc.interface.

Hi,
This PR migrates the remaining OUTPUT_MODE logic block from the rcS script and places it into rc.interface.  OUTPUT_MODE logic blocks in rc.interface have also been grouped together at the top of the script.  This PR is copy/paste from rcS into rc.interface, and copy/paste from the bottom of rc.interface higher up to group logic together.
Testing was conducted on a pixracer.  No observed changes to the startup script output result and a startup script output compare is below for convenience.
Let me know if you have questions on this PR.
-Mark

Thanks for catching that @bkueng , I just replaced that logic block back at the bottom of rc.interface where it was previously located.  I'll look for any remaining opportunities to refactor that logic.
The serial console output diff shows the same line differences as posted above, no changes between upstream and this PR.  Rebased and pushed up!  Thanks again.


ZRANGEBYLEX does not handle min of "+" or max of "-"

The code for handling the special values + and - in zrangebylex only handles - for min and + for max, not vice versa. For example, the following code prints [b'bar'] instead of an empty list:
import asyncio
import aioredis

async def main():
    client = await aioredis.create_redis_pool('redis://localhost')
    await client.zadd('foo', 0.0, 'bar')
    result = await client.zrangebylex('foo', b'+', b'+')
    print(result)
    client.close()
    await client.wait_closed()

loop = asyncio.get_event_loop()
loop.run_until_complete(main())
loop.close()

This also affects zrevrangebylex and zremrangebylex.
Incidentally, the API design is problematic because it's impossible to pass strings like (- (meaning, strings lexicographically greater than -) into redis.

@bmerry -
Thanks for this submission! Would you be willing to contribute a potential fix to the API? Maintenance of this library is back on and the next release will be a breaking change, so updating the API to work as expected would be a great boon.

Yes, I may be able to contribute something, depending on the timeframe. When are you thinking of releasing the next version?

@seandstewart I can think of two possibilities for eliminating the ambiguity. I'm going to start on the first, but let me know if you'd rather I did the second.

Eliminate the include_min and include_max arguments, and make it the user's responsibiity to prepend a [ or (. This will break almost all code that uses these APIs, but it will break them in a way that gives an error message. It will also make the implementation simpler as it will just be forwarding to redis.

Introduce singleton constants meaning "from the start" / "from the end", similar to ZSET_AGGREGATE_SUM etc; and treat "-" and "+" as literal strings which will get a prefix. This will keep a lot of code working as is (any code that doesn't explicitly pass a - or +), but code that breaks will break subtly (e.g. will start excluding anything that comes before "-" lexicographically.


Update value_count serialization/deserialization to be consistent with original schema

Goal

Ensure that the Schema (ColumnSchema) value_count property is restored to the same value when serialized/deserialized.
Update default value of is_list/is_ragged to infer from value_count property.

Details
value_count property serialization/deserialzation:

fails to serialize to json

value_count = {'max': 1, 'min': 1}
AttributeError: 'int' object has no attribute '_serialized_on_wire'

silently fails to re-construct the same schema object (serialize -> deserialize)

value_count = {{'max': 0, 'min': 0}, {'max': 2, 'min': 1}}

~These value_count examples may not be practically an issue~. It opens up some questions about where validation should take place for the Schema (at schema construction time or at serialization time). And where there are certain assumptions or constraints in serialation.
is_list/is_ragged default value

value_count max > min

implies that is_ragged=True, is_list=True

value_count max == min

implies that is_ragged=False, is_list=True

value_count max=0

implies that is_ragged=False, is_list=False

This PR is a replacement for #169 since it removes the line being changed in that PR. #169 could be merged, but not strictly required to if this one is.


Got the 404 error when i got the pretrained ".pth" file of MMDetection from model zoo

When i use https://github.com/open-mmlab/mmdetection/blob/master/docs/MODEL_ZOO.md url,
I got the 404 error，
what should i do?
.

The correct URL should be https://github.com/open-mmlab/mmdetection/blob/master/docs/model_zoo.md
We will fix it.

https://github.com/open-mmlab/mmdetection/pull/2641


Domain expired

When i click on dashboard tel me the domain is expired: mobileminerapp.com

MobileMiner was shuttered a couple of years ago.

Is possible remove a link to mobileminerapp.com from the MultiMiner?

It's an Open Source project - you are free to remove it and submit a PR.


.pre-commit-config.yaml vs github actions linter fix

Commit hash / release tag
https://github.com/nanlabs/python-reference/actions/runs/3316005966/jobs/5477289911
OS
Ubuntu 20.04.5 LTS
What did you do?
Trying to run pre-commit hooks to fix any possible linter issue.
What did you expect to see?
We expect to see the same errors that github actions linter check.
What did you see instead?
github action linter checks for different errors that pre-commit hooks don't check

we apparently fixed it in a previous pull request. I'm gonna close it, we can reopen it if the problem happens again


chore(release): use shared semantic release config

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Data path should be configurable

You are reading environment.depth.belowKeel my Garmin Intelliducer GAR-010-00703-00 exposes the depth at environment.depth.belowTransducer

Fixed it using the signalk-derived-data plugin.
But it would still be nice if this would be configurable.


Cloudflare!!

undetectable-chromedriver v2 works.
probably should use multiple browsers, check if need to restart browsers (check cloudflare blocks).
also might need to start caching wiimmfi, we'll see.
just need to weave it into the rest of the application.
this will take a while

Nah


I corrected some grammatical errors

@Aderogba2003  try this year for Hacktoberfest and have a look at the updated repo and contribute.Make sure to follow me and star the repo
HAPPY HACKING


feat: Add memif2memif example

Signed-off-by: Denis Tingaikin<EMAIL_ADDRESS>Closes https://github.com/networkservicemesh/deployments-k8s/issues/262

Requires https://github.com/networkservicemesh/sdk-vpp/pull/89


Detecting image as a live person

I am using this project in my application but the problem is that its detecting image as a live person. How to resolve this issue?

The SDK is from Huawei, you have to contact them. We (Doubango AI) have an accurate implementation developed by us but it's not open source.


Allow access to /dev/net/tun inside containers

This allows containers to access /dev/net/tun by adding an option containers.<name>.enableTun that enables the correct capabilities and permissions for the container.
Also adds two generic options containers.<name>.additionalCapabilitites and containers.<name>.allowedDevices that help adding things like FUSE access (/dev/fuse) later down the road.
Motivation for this change
This change is useful for programs running in containers trying to set up tunnel interfaces. The current DevicePolicy prevents containers from accessing most device files thus explicitly allow /dev/net/tun.
NOTE: CAP_NET_ADMIN has to be granted in addition to the changes described above. This can be done by setting containers.<name>.privateNetwork = true on the container for example.
Fixes #18708.
/cc @Mic92
Things done

[x] Tested using nixos-rebuild test
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Hi,
could you please also make the list of allowed devices a configuration thing? I actually don't want all my containers to work with tun/tap devices. Only the single instance that is running openvpn (in the future).
And you might want to write a test for that. There are quite a number of container-networking tests already. You could add one that has one host with two containers, one with tun/tap enable, one without and then check whether access to tun device works or not. Also Maybe the openvpn module could make use of a test that uses two containers instead of two VMs? (Okay, openvpn could actually use a test.)

If DeviceAllow were configurable from the container definition, could the OpenVPN module add the required device itself? That would solve the problem of requiring extra container-level configuration when running/not running this service.

In the first iteration this would mean that admins creating the container with openvpn inside would have to add the option. But it would prevent all other (existing) admins from opening their containers via undesired tun/tap. So this is firstly a security issue for me.
The openvpn module/service in nixos could then detect whether its inside a container (I think boot.isContainer == true) and maybe add the relevant option.
I could also help you with writing the test. Somehow I started liking system tests in NixOS so much I started writing them a lot on my spare time :wink:

I'm not sure if OpenVPN should implicitly give itself the necessary privileges (capability+device access) when it detects that it's inside a container.
Basically, what I was thinking was to merge this PR with the other one (#18823 - capabilities) to make things easier. Than I'd add options containers.<name>.additionalCapabilities for configuring capabilities and containers.<name>.allowedDevices for giving the container access to device files. Having these generic options would make things easier when adding things like FUSE access later down the road.
I would than add a third option containers.<name>.enableTun that would add CAP_NET_ADMIN and /dev/net/tun to the previously mentioned options. An assertion or lib.warn would be added to the openvpn module indicating to the admin that containers.<name>.enableTun = true is required when it detects that it runs inside a container. That way, the admin has to explicitly enable tunnel support in the container.
What do you think?

I have updated the branch according to the plan I described above. Please have a look!

Sadly no new container test. How do we know it actually works. And how do we ensure that it still works in 17.03?

As I said, I'm still working on tests. I've tested against the master snapshot of nixpkgs with nixos-rebuild test -I nixpkgs=nixpkgs-master which currently corresponds to 17.03 as I understand it.
Do you mind hoping on IRC, I got a couple of questions concerning testing in nixpkgs, but don't wont to pollute the issue discussion.

Ah, I am sorry. The new review-feature of github irritated me and let me believe this is already merged. Sorry. Keep on testing 😉

Thanks!


Add CTA to signup for IPFS Newsletter

We should prompt visitors to join the weekly newsletters so we can keep them up to date on project and community news.
I can provide an embed or integration via Mailchimp.

Bumping this again.

This is a good idea. Could maybe use the callout CSS that @jessicaschilling made to promo on the main page - and then link to a new page where folks can configure their subscription (subscribe/unsubscribe)?
There's already a link for the RSS on the blog page, could also add a pop-up or page expansion on mouse-off to suggest folks sign up for the newsletter. @cwaring or @rafaelramalho19 might have ideas for something fast and lightweight to let people know this is an option.

The header callout is pretty big and splashy -- it might be best to keep that reserved for major news/releases rather than something we want to have live more permanently on the site.
We could add a similar mini-section to the "who's using IPFS" content:

The button could go to the Mailchimp signup form, or whatever other means we're using for signup.

Thanks for asking for my ideas @momack2 😄
I like Jessica's suggestion, and we could add some animations to call the attention of the user to that specific area, here's a rough example (made in 5 minutes so don't expect much 😅)
https://codepen.io/rafaelramalho19/pen/NWGoJbo
Video showing this:
https://dl.dropboxusercontent.com/s/esyrsksakebeqad/XjQCJ3Qmry.mp4

@renrutnnej -- For the sake of keeping this rolling (we can add something splashier later), please see https://github.com/ipfs/website/pull/380 - thanks!

@jessicaschilling this works for me! Thank you. Also I can't merge to the website repo for some reason, but I'll +1 over there too


asdc

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Comment

/close

Random comment

oajhsdf

/close

test

test2

/close

/close


Change * (asterisk) position in generator declarations in the docs

In the docs, we are declaring generators like this: function *() { ...
However, function and * are part of the same keyword. So, the correct way to declare a generator is like this: function* () {... or like function*() {...
Reference:
Mozilla Developer Network JS Docs
Airbnb's JavaScript Style Guide

Both function* and function * works the same.
This is just a matter of style, not functionality, and since Koa 2.x is using async/await instead of generators in my opinion this is not important at all.

agreed, these kinds of issues aren't helpful or productive sorry!


Leave out key administrators when using default KMS key policy

Is your request related to a problem? Please describe.
When using kms_key_enable_default_policy = true, the generated KMS key has a policy granting kms:* to all identities in the account. However, it will also always add the KeyAdministration policy for the identity performing the plan/apply, even if the kms_key_administrators variable is empty, as it uses a datasource to always append the current identity. As the KeyAdministration permissions are redundant for any identity inside the account in this case, we would like not to add them at all, because we have different identities running TF.
Describe the solution you'd like.
I see two possible solutions:

Add a variable kms_key_administrators_add_current to turn off the automatic addition of the current identity to the key_administrators variable of the KMS module.
Do not add the current identity at all in case kms_key_enable_default_policy is enabled as it would be redundant (Breaking change!)

Describe alternatives you've considered.
Alternatives would require changes to the KMS module
I'd be happy to submit a PR if this is considered useful.

could you provide a reproduction and then also the desired output

For example, just follow https://developer.hashicorp.com/terraform/tutorials/kubernetes/eks and then add kms_key_enable_default_policy = true you'll see this diff:
  # module.eks.module.kms.aws_kms_key.this[0] will be updated in-place
  ~ resource "aws_kms_key" "this" {
        id                                 = "***"
      ~ policy                             = jsonencode(
          ~ {
              ~ Statement = [
                  + {
                      + Action    = "kms:*"
                      + Effect    = "Allow"
                      + Principal = {
                          + AWS = "arn:aws:iam::***:root"
                        }
                      + Resource  = "*"
                      + Sid       = "Default"
                    },

Nice boit :+1:


delivery: Generate library and source from Artifact Registry

The client libraries at /client-libs/python is generated using the OpenAPI.yaml spec. Should the generated libraries be created at each deploy, or maintained in GitHub?

Discussed in team sprint meeting (refer 8/22/22) - currently only leveraged in the flask website. This may no longer be necessary if the flask website container gets replaced with the new lit client app.
Requires further discussion on scope of work and impact of current CI/CD flow.


Slack invite link is not working

Hello,
Thank you for your contribute.
It seems that the invite link to Slack is no longer active and working.
This is the error message I get when following:
https://join.slack.com/t/redhat-certs/shared_invite/zt-7ju3rz7b-_G3Njp3PDwdBG_81SwPeLA
********* SNIP ***********
This link is no longer active
To join this workspace, you’ll need to ask the person who originally invited you for a new link.
********* SNIP ***********
thank you

This is because Slack changed its policy how long a link will be valid (default is now 30 days).
Here is a new link: https://join.slack.com/t/redhat-certs/shared_invite/zt-k6ew2jxh-fnBYw2Pw4~PQQE~Nse~wCQ


POC: to batch plugin responses this has reduced webworker scripting by 2 seconds

batching updateAction Data basically batching the results of plugin responses
Description

[!TIP]
Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team).
Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR.

Fixes #Issue Number
or
Fixes Issue URL

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!CAUTION]
🔴 🔴 🔴 Some tests have failed.
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/12056385315
Commit: 9bd05cebb4d8fe8c0a5affd3d87c92a2da90abaa
Cypress dashboard.
Tags: @tag.All
Spec:
The following are new failures, please fix them before merging the PR:
cypress/e2e/Regression/Apps/CommunityIssues_Spec.ts
cypress/e2e/Regression/Apps/ImportExportForkApplication_spec.js
cypress/e2e/Regression/Apps/MongoDBShoppingCart_spec.ts
cypress/e2e/Regression/ClientSide/ActionExecution/NavigateTo_spec.ts
cypress/e2e/Regression/ClientSide/Autocomplete/JS_AC1_spec.ts
cypress/e2e/Regression/ClientSide/Binding/TableTextPagination_spec.js
cypress/e2e/Regression/ClientSide/Binding/TableV2TextPagination_spec.js
cypress/e2e/Regression/ClientSide/Binding/TableV2_Api_spec.js
cypress/e2e/Regression/ClientSide/Binding/Table_Api_spec.js
cypress/e2e/Regression/ClientSide/BugTests/DS_Bug28985_spec.ts
cypress/e2e/Regression/ClientSide/BugTests/JS_Bug29131_spec.ts
cypress/e2e/Regression/ClientSide/Debugger/Widget_property_navigation_spec.ts
cypress/e2e/Regression/ClientSide/ExplorerTests/Renaming_spec.js
cypress/e2e/Regression/ClientSide/Git/ExistingApps/v1.9.24/DSCrudAndBindings_Spec.ts
cypress/e2e/Regression/ClientSide/OneClickBinding/TableWidget/OneClickBindingMysql_spec.ts
cypress/e2e/Regression/ClientSide/OneClickBinding/TableWidget/Table_MongoDB_spec.ts
cypress/e2e/Regression/ClientSide/OneClickBinding/TableWidget/Table_postgres_spec.ts
cypress/e2e/Regression/ClientSide/PeekOverlay/PeekOverlay_Spec.ts
cypress/e2e/Regression/ClientSide/Widgets/Button/Button2_spec.ts
cypress/e2e/Regression/ClientSide/Widgets/ListV2/Listv2_BasicServerSideData_spec.js
cypress/e2e/Regression/ClientSide/Widgets/ListV2/Listv2_Meta_Hydration_ServerSide_spec.js
cypress/e2e/Regression/ClientSide/Widgets/Migration_Spec.js
cypress/e2e/Regression/ClientSide/Widgets/Modal/Modal_spec.ts
cypress/e2e/Regression/ClientSide/Widgets/Radio/Radio2_spec.ts
cypress/e2e/Regression/ClientSide/Widgets/Select/Select3_Spec.ts
cypress/e2e/Regression/ServerSide/ApiTests/API_Bugs_Spec.js
cypress/e2e/Regression/ServerSide/ApiTests/API_MultiPart_Spec.ts
cypress/e2e/Regression/ServerSide/GenerateCRUD/MongoURI_Spec.ts
cypress/e2e/Regression/ServerSide/GenerateCRUD/MySQL2_Spec.ts
cypress/e2e/Regression/ServerSide/GenerateCRUD/Postgres2_Spec.ts
cypress/e2e/Regression/ServerSide/MySQL_Datatypes/Basic_Spec.ts
cypress/e2e/Regression/ServerSide/OnLoadTests/OnLoadActions_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/Array_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/Binary_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/BooleanEnum_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/Character_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/DateTime_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/Json_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/Numeric_Spec.ts
cypress/e2e/Regression/ServerSide/Postgres_DataTypes/UUID_Spec.ts
cypress/e2e/Regression/ServerSide/QueryPane/Mongo1_spec.ts
cypress/e2e/Regression/ServerSide/QueryPane/S3_1_spec.js
cypress/e2e/Regression/ServerSide/QueryPane/S3_2_spec.ts
cypress/e2e/Sanity/Datasources/SMTPDatasource_spec.js
List of identified flaky tests.
Wed, 27 Nov 2024 19:54:36 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[ ] No

Summary by CodeRabbit

New Features

Introduced a new mechanism for capturing action data updates within a specified 10-second window, allowing for more efficient processing of actions.
Added a new action type TRIGGER_EVAL_BATCH for batch evaluations, enhancing the handling of multiple evaluations in a single action.

Bug Fixes

Improved handling of action data updates by modifying the way parameters are accepted in the action handling functions.
Enhanced error handling during execution data processing, improving robustness.

/build-deploy-preview skip-tests=true


Draw a polyline with x,y

Hi, I get from GPS signal coordinates (etc. 44.3456, 23.876 etc. ... ...). How I can show it on the ground? Also how I can make ground bigger (I dont want to see edges) ?
I want to use your library to draw route - to make some kind of navigation app.
p.s. Really good html5 library

i had the same question before (for GPS coords, except also with altitude), @kevinroast added test6 [1] as an example for polyline drawing. you can easily adapt this for your use.
[1] http://www.kevs3d.co.uk/dev/phoria/test6.html

but some other examples?


Fr: Bad Ceiling beam Description

The description of Ceiling beam indicate them as being light source, which they aren't.

Could you please edit the relevant file here https://github.com/bartwe/StaxelTranslations/tree/main/fr-FR and create a Pull Request with the fix ?


Add support for AmazonRegion to Billing

The amazon_region field will be introduced in version 2.18 and this PR will remain on hold until after its release. The scheduled release date for 2.18 is January 29, 2019.
Update (02/20): v2.18 was released last week (02/14). Releases to add support PHP and Ruby were created yesterday. See below:
PHP:

https://github.com/recurly/recurly-client-php/releases/tag/2.11.2

Ruby:

https://github.com/recurly/recurly-client-ruby/releases/tag/2.17.6

Official documentation for v2.18:
https://dev.recurly.com/docs/create-an-accounts-billing-info-using-external-token

@cristiangraz Please re-review this PR at your earliest convenience. 2.18 was released last week (02/14) and updates to the PHP, Python and Ruby client libraries to support 2.18 was released yesterday.
PHP:

https://github.com/recurly/recurly-client-php/releases/tag/2.11.2

Python:

https://github.com/recurly/recurly-client-python/releases/tag/2.9.4

Ruby:

https://github.com/recurly/recurly-client-ruby/releases/tag/2.17.6

@cristiangraz FYI. there are features between 2.15 and 2.18 that this PR does not support. Please refer to these release notes ➡ https://dev.recurly.com/page/api-release-notes

Thanks for the reminder. PR looks good, but we need to make these two changes before merging:

bump X-API-Version to 2.18
update User-Agent to Blacklight/2019-02-20

@cristiangraz The changes above were made in https://github.com/blacklightcms/recurly/pull/98/commits/0e8bef6629120adbed00bc5d426934bcdddf4e8a. Please merge at your earliest convenience.

@jpgnotgif Thanks for the reminder, I looked through the release notes previously and it appears to just be new/optional fields. This shouldn't cause any issues, I'll open up a separate PR with the additional fields added.


🔎 Add section with dependencies

I noticed that the documentation is missing a section listing all dependencies.
Including this information would be very helpful for users. For example, you could add this information in the installation section, similar to how it is done by xarray: https://docs.xarray.dev/en/stable/getting-started-guide/installing.html
JOSS: https://github.com/openjournals/joss-reviews/issues/6742

closed by #451


Ubuntu repo maybe broken?

VSCode Version: 1.49.1
OS Version: Ubuntu 20.04.1 LTS

Steps to Reproduce:
$ sudo apt update
Get:6 http://packages.microsoft.com/repos/vscode stable/main amd64 Packages [201 kB]
Err:6 http://packages.microsoft.com/repos/vscode stable/main amd64 Packages
File has unexpected size (201047 != 200914). Mirror sync in progress?
Hashes of expected file:
  - Filesize:200914 [weak]
  - SHA512:a91d388504b56c875620f73b618813be1c127d6bfe96406cc6e9a39fdf33f1dd2c8ecbda4a636f0c86bc18005e4f5d0683da0c02ba13c555cef8d11eb83412fe
  - SHA256:1d39334cb2779572dd17869a05d3ed3f1d57992cc9755a34a4d2622132b19f57
  - SHA1:d5224a98d6adfb4ab65d66aa803f9688a6503080 [weak]
  - MD5Sum:2fc601c3aa23f067ef305bfb7f7086e7 [weak]
 Release file created at: Mon, 28 Sep 2020 06:53:06 +0000
Reading package lists... Done
E: Failed to fetch http://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  File has unexpected size (201047 != 200914). Mirror sync in progress?
  Hashes of expected file:
   - Filesize:200914 [weak]
   - SHA512:a91d388504b56c875620f73b618813be1c127d6bfe96406cc6e9a39fdf33f1dd2c8ecbda4a636f0c86bc18005e4f5d0683da0c02ba13c555cef8d11eb83412fe
   - SHA256:1d39334cb2779572dd17869a05d3ed3f1d57992cc9755a34a4d2622132b19f57
   - SHA1:d5224a98d6adfb4ab65d66aa803f9688a6503080 [weak]
   - MD5Sum:2fc601c3aa23f067ef305bfb7f7086e7 [weak]
  Release file created at: Mon, 28 Sep 2020 06:53:06 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.

Does this issue occur when all extensions are disabled?: Yes
Previous version of this problem:  #41094

Same issue here since the last 2 hours approximately - just did a clean install of Ubuntu 20.04.01 and installed VS code using the .deb file.
Region - Australia.

After 8 hour broken repo now Confirm fixed in Indonesia.

Restored (Belgium, this morning, debian buster). Thanks !

Guess this can be closed as well, issue has now been fixed. Thanks

/duplicate https://github.com/microsoft/vscode/issues/107793


fix: Self types as results are translated to class names

Closes #86
Summary of Changes
Selfs as result types are now translated to the corresponding class names.

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

v0.3.0
GitHub release

Your semantic-release bot :package::rocket:


symcrypt: upgrade to 103.5.1

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary
Upgrade SymCrypt to version 103.5.1 to fix bugs related to FIPS certification.
Also changed how we handle environment variables used to embed git information into the binary so it will cause a failure during rpmbuild if we don't update them, and provided a script to update them.
Change Log

Update SymCRypt to version 103.5.1 to fix bugs related to FIPS certification.

Does this affect the toolchain?
NO
Test Methodology

Build and tested in containerized buld.
Used image from https://dev.azure.com/mariner-org/mariner/_build/results?buildId=661519&view=results to create hyperv vm to test.

Auto cherry-pick results:

3.0-dev :white_check_mark: -> https://github.com/microsoft/azurelinux/pull/10815

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=661830&view=results


Add option to center contents with limited text-width

I believe one of the main selling points of feed readers is their distraction-free nature and comfort of use. In my experience, having text centered with narrow width limit in maximized window is the most enjoyable way to read. Currently I was able to only limit the text width.

Hi, thanks for opening the issue!
I think the option makes sense. I'll accept a pull request adding it.
A few thoughts:

if html-renderer is set, text-width doesn't apply — the new setting probably shouldn't apply either;
we should make sure that the text stays centred even when the window is resized.

Hi, it's quite an old issue so I wanted to ask whether it's been implemented yet.
Unfortunately, I don't have any programming skills but ever since I started using miniflux, web client on mobile (which centers text) and newsboat as a client on desktop, I felt that it's an important feature that many users would appreciate

@Mati20187 Nope, it hasn't been implemented yet. We don't do any sort of planning, so I can't tell you when it will get done — it will be implemented when someone decides to implement it and send a pull request.

This has been bothering me too. On a wider screen it just looks odd to have the column "stuck" to one side. Especially coming from a decent terminal epub reader like epy, where the column is where it might be expected: the middle.
Pity about the PR above. What about a basic solution left-margin x which just shifts everything rightwards by however much the user wants to risk?

@Friptick, the problem with introducing new settings is that it's nigh impossible to take them out.

Fair enough. Your project, your call about how or if to proceed. I can only second what OP describes so succinctly:

I believe one of the main selling points of feed readers is their distraction-free nature and comfort of use. In my experience, having text centered with narrow width limit in maximized window is the most enjoyable way to read. Currently I was able to only limit the text width.

Any news about this? @Minoru

No. Please see my earlier comment here for an explanation on how we operate.


Clearer error message when user does not have permissions on target path

I was recently troubleshooting the following error in an Azure Automation runbook which exports some data to an xlsx-file:
Exception calling "Save" with "0" argument(s): "Error saving file C:\temp\report.xlsx"
At C:\Program Files\WindowsPowerShell\Modules\ImportExcel\5.3.4\Export-Excel.ps1:1024 char:30
else           { $pkg.Save() }
~~~~~~~~~~~
CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : InvalidOperationException
When run manually, it worked fine.
It turned out to be lacking NTFS permission for the executing user (Azure Automation runbooks by default runs under the Network Service context).
Would it be possible to catch the lack of necessary permissions and return information about this in the error message?

The save() method of the ExcelPackage object doesn't return very descriptive errors. We can add a check for a valid path but that won't pick up a lack of write permissions.
I'd recommend putting these checks in the script that calls Export-excel  - that way catches all the things which can go wrong in a given environment.

That`s a good point - I'll go ahead and close the issue.


Importing 'MultiplexedPairedEndBarcodeInSequenceDirFmt' requires a directory, not manifest.txt

Hello! I have a problem with qiime tools import...
#script:
qiime tools import
--type MultiplexedPairedEndBarcodeInSequence
--input-path manifest.txt
--output-path import.qza
--input-format  MultiplexedPairedEndBarcodeInSequenceDirFmt
#problem
There was a problem importing manifest.txt:
manifest.txt is not a directory.
I tried to remove the --input-format and encountered similar issues：
#problem
There was a problem importing manifest.txt:
Importing 'MultiplexedPairedEndBarcodeInSequenceDirFmt' requires a directory, manifest.txt
And I tried to modify the file suffix to .tsv/.csv/...  useless...
by the way, my manifest.txt contains {sample-id, forward-absolute-path, reverse-absolute-path}.
I used this file to import as the type of 'SampleData[PairedEndSequencesWithQuality]', its sucessful.
However, I want use the function of 'qiime cutadapt demux-paired', it requires '--i-seqs' : ARTIFACT PATH MultiplexedPairedEndBarcodeInSequence, The paired-end sequences to be demultiplexed.  [required]
So, I need to solve the import issue.
I would appreciate some assistance.
Many thanks,
Chiker

oh!
I tried creating a new file and imported two fastq.gz files named forward. fastq.gz and reverse. fastq.gz, and it was successful.
But I have a lot of data, how can I import it together?

Hi @Chiker7,
Please post this on the QIIME 2 User Forum, as this is an error message you are receiving while attempting to use QIIME 2 (rather than an actual bug in the software, or feature request, etc). Thanks!


Allowing datastore system test to run on emulator with no connection.

Currently, the GOOGLE_APPLICATION_CREDENTIALS is used and an access token is minted even though requests to the emulator require no authentication.

By allowing the default behavior to occur, an authorized http instance tries to [get a new] access token even if it doesn't need one.

@pcostell Is it intended that the emulator doesn't check the value of the Authorization header (or even if  the header has been sent?)

LGTM


Link to feed

Link to the atom feed from the head.
I was worried I'd have to do this for every file. Nope! Thanks 11ty 😄
Resolves: #3
Makes happy: Me
Solves: People forgetting to check your website everyday due to supposed lack of RSS feed, and therefore missing The Quest Continues for a 3 weeks 😉

@kaleidawave, ping :)
(I hope you don't mind, but you aren't listed as a watcher on https://github.com/kaleidawave/kaleidawave.github.io/watchers so I figured you probably hadn't seen this at all.)

Whoops, somehow my watch settings were only on participating? So only just seeing this. Didn't think people would make contributions to the blog!
Thanks for this addition, didn't know about this! Haven't tested this branch but assume it works okay.


cryptomator: 1.6.14 -> 1.8.0

Description of changes
https://github.com/cryptomator/cryptomator/releases/tag/1.8.0
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

This was closed due to a wrong Pull request where the

@discite I think there are more changes needed to get cryptomator 1.7+ working. See https://github.com/NixOS/nixpkgs/pull/200590

Superseded by #230757.


[UI] - Great key/value editor

Editing key/Value is painful where value became long : especialy in ConfigMap

Why not adding an "Edit" button on the top right of the textarea

Which expand the textarea

** Important notes **
Please : Put the textarea "value" css to :
    overflow: scroll;
    font-family: monospace;

Editing value in Lato font kills all users eyes :)
Thanks !

I'd like to be assigned to this one because I already created a monospace key-value component like this as part of the questions.yml enhancements.


Uncaught TypeError in repo.py line 299

Aider version: 0.56.0
Python version: 3.12.5
Platform: macOS-14.6.1-x86_64-i386-64bit
Python implementation: CPython
Virtual environment: Yes
OS: Darwin 23.6.0 (64bit)
Git version: git version 2.23.0
An uncaught exception occurred:
Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "main.py", line 698, in main
    coder.run()
  File "base_coder.py", line 735, in run
    self.run_one(user_message, preproc)
  File "base_coder.py", line 778, in run_one
    list(self.send_message(message))
  File "base_coder.py", line 1202, in send_message
    saved_message = self.auto_commit(edited)
                    ^^^^^^^^^^^^^^^^^^^^^^^^
  File "base_coder.py", line 1885, in auto_commit
    res = self.repo.commit(fnames=edited, context=context, aider_edits=True)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "repo.py", line 95, in commit
    diffs = self.get_diffs(fnames)
            ^^^^^^^^^^^^^^^^^^^^^^
  File "repo.py", line 222, in get_diffs
    if not self.path_in_repo(fname):
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "repo.py", line 359, in path_in_repo
    return self.normalize_path(path) in tracked_files
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "repo.py", line 299, in normalize_path
    path = str(Path(PurePosixPath((Path(self.root) / path).relative_to(self.root))))
                                   ~~~~~~~~~~~~~~~~^~~~~~
TypeError: unsupported operand type(s) for /: 'PosixPath' and 'NoneType'

Thanks for trying aider and filing this issue.
The fix is available in the main branch. You can get it by installing the latest version from github:
aider --install-main-branch

# or...

python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you. I'm going to close this issue for now, but feel free to add a comment here and I will re-open. Or feel free to file a new issue any time.

Thanks for trying aider and filing this issue.
The fix is available in the main branch. You can get it by installing the latest version from github:
aider --install-main-branch

# or...

python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you. I'm going to close this issue for now, but feel free to add a comment here and I will re-open. Or feel free to file a new issue any time.

I've updated to 0.57.1 now and hoping the problem fixed. Thank you.


i saw how beautiful this repo is, in terms of parallelism / numa stuff etc.

i understand this is intel repo but curious will amd work as well or... what kind of architecture / intel chip set is best used with this repo?
about to upgrade hardware and looking forward to suggestion with use with this repo.
have yet to run it. but hope to know what chipset is best used with this repo

i understand this is intel repo but curious will amd work as well or... what kind of architecture / intel chip set is best used with this repo?

For now, Intel Xeon CPUs are the most optimized, especially those with AMX instructions.
#209 is optimizing Intel's AVX2 client CPUs, especially the hybrid CPUs that will benefit most.


MenuItemModel lacks text and link properties

Describe the bug
In strongly typed typescript setups, using the properties 'link' and 'text' on instances of MenuItemModel fails with the following error
  Object literal may only specify known properties, and 'text' does not exist in type 'MenuItemModel'.

To Reproduce
Steps to reproduce the behavior:

Use with the example below

@Component({
  selector: 'ds-navbar',
  styleUrls: ['./navbar.component.scss'],
  templateUrl: './navbar.component.html',
  animations: [slideMobileNav]
})
export class NavbarComponent extends BaseComponent implements OnInit {
  scholarsMenu: MenuSection = {
    id: 'scholars_menu',
    visible: true,
    active: false,
    model: { type: MenuItemType.LINK, disabled: false, text: 'Scholar Profiles', link: '/communities/fb79-513', },
    index: 0,
    shouldPersistOnRouteChange: true,
  };

  ngOnInit() {
    super.ngOnInit();
    this.menuService.addSection(this.menuID, this.scholarsMenu);
  }
}

Expected behavior
All properties should be defined on the model interface
Related work
I have created a PR #3750

Sorry @tdonohue, it's my first PR here. What am I supposed do at this stage?

@otuoma : At this stage, your PR has made it onto our Project Board (https://github.com/orgs/DSpace/projects/31) and is waiting on a volunteer to review it.  So, there's nothing for you to do but be patient.  Please do keep in mind though that we have a backlog of reviews currently (and it's the end of the year) so, unfortunately, it may be a while until a volunteer is able to review your work.
If you wanted to try to speed up the process, you are also allowed to ask a different developer if they would be willing to trade reviews (where you would review/test one of their PRs in exchange for them reviewing/testing yours).  See https://wiki.lyrasis.org/display/DSPACE/Trading+reviews+on+Pull+Requests


Hungarian translation

Thank you!

Could I get you to submit another PR with new translations? See: https://github.com/primefaces/primelocale/blob/main/en.json