Envoy for a cross cluster Thanos endpoint discovery

One response from @bwplotka in question #454 says 'you can use some proxy (we are doing this via envoy) to set up the connection between query and sidecars'. So I was trying the whole last week anything to configure Envoy to provide Thanos with endpoints based in a different cluster, but I am still unsuccessful. May I ask for clarification of this? Maybe some example for Envoy proxy. Maybe I am missing something and this is just not possible.
Here is our desired state:
Thanos Query
--store: remote.cluster:envoyport
Remote Cluster
Envoy with STRICT_DNS cluster discovery pointing to the headless service (DNS record with a multiple A records each for one Thanos StoreAPI).

Can you share what exactly issues you have?
cc @domgreen

This is indeed how we have our set-up internally. Could you share some more detail on your current issue?
It is most likely a config issue somewhere in the routing.

Sure,
Here is my Envoy configuration. thanos-store-gateway is a DNS record with multiple A records in cluster one.
Cluster One:
(Kubernetes LB with external IP -> Envoy -> thanos-store-zero)
Cluster Zero:
(Thanos -> Cluster One (Kubernetes LB))
I can connect to Envoy listener on port 10901 from Thanos running in Cluster zero, but the result is that Thanos can see only one endpoint at a time. When I refresh the Thanos Store GUI, I can see a response from different endpoint behind the proxy each time. But I want to have them all there.
static_resources:
  listeners:
  - name: listener_0
    address:
      socket_address: { address: <IP_ADDRESS>, port_value: 10901 }
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        config:
          stat_prefix: ingress_http
          route_config:
            name: local_route
            virtual_hosts:
            - name: local_service
              domains: ["*"]
              routes:
              - match:
                  prefix: "/"
                route:
                  cluster: service_backend
          http_filters:
          - name: envoy.router
  clusters:
  - name: service_backend
    connect_timeout: 0.25s
    type: STRICT_DNS
    dns_lookup_family: V4_ONLY
    lb_policy: ROUND_ROBIN
    http2_protocol_options: {}
    hosts: [{ socket_address: { address: thanos-store-gateway, port_value: 10901 }}]

Can you share what exactly issues you have?
cc @domgreen

node:  id: id_1  cluster: thanos-admin:  access_log_path: /tmp/admin_access.log  address:    socket_address: { address: <IP_ADDRESS>, port_value: 9901 }static_resources:  listeners:  - name: listener_0    address:      socket_address: { address: <IP_ADDRESS>, port_value: 10901 }    filter_chains:    - filters:      - name: envoy.http_connection_manager        config:          stat_prefix: ingress_http          route_config:            name: local_route            virtual_hosts:            - name: local_service              domains: ["*"]              routes:              - match:                  prefix: "/"                route:                  cluster: service_backend          http_filters:          - name: envoy.router  clusters:#  - name: service_backend#    type: EDS  #    connect_timeout: 0.25s#    eds_cluster_config:#      service_name: myservice#      eds_config:#        api_config_source:          #api_type: REST_LEGACY # GET /v1/registration/myservice          #api_type: REST # POST /v2/discovery:endpoints#          api_type: REST#          cluster_names: [eds_cluster]#          refresh_delay: 5s#  - name: eds_cluster#    type: STRICT_DNS#    connect_timeout: 0.25s#    http2_protocol_options: {}     #    hosts: [{ socket_address: { address: thanos-store-gateway,  port_value: 10901 }}]  - name: service_backend    connect_timeout: 0.25s    type: STRICT_DNS    dns_lookup_family: V4_ONLY    lb_policy: ROUND_ROBIN    http2_protocol_options: {}    hosts: [{ socket_address: { address: thanos-store-gateway, port_value: 10901 }}]

I wonder if that is not useful for you as well https://github.com/heptio/contour

For now, we will try to keep it as simple as possible but thank you for this point.
I think this can be closed now as we agreed on Slack that there is currently no way how to do endpoint discovery different way than DNS or Gossip.


errors of mingw makefiles

Note: If you use the fuzzy expressions such as 'can't use/no effect/problem/error', but don't give any relevant auxiliary information according to the template below, this issue will be not replied.

Describe the bug
I ran commands like "xmake create -l c -P ./hello" to create a simple project;
then I ran commands like "xmake f -p mingw; xmake project -k makefile " to create MinGW Makfiles。
All commands above ran in Powershell。
But when I ran "mingw32-make" in powershell  It throwed a error : " A subdirectory or file -p already exists "。
when I ran "mingw32-make" in git-bash it also throwed a error:"[makefile:26:build\mingw\x86_64\release\hello.ext] Error 1"。
Because this code is simple,you can try it self.
Expected behavior
I want to get a correct MinGW Makfiles.
Related Environment

xmake version: 2.5.1
os: windows
target platform: mingw

Currently does not support generating mingw makefile, maybe you need to wait for some time.

Currently does not support generating mingw makefile, maybe you need to wait for some time.

Will xmake of v2.5.2 support generating mingw makefile?Which will version support this function?

Will xmake of v2.5.2 support generating mingw makefile?Which will version support this function?

Temporarily arrange it at 2.5.2, depending on my time to decide
Of course, I also welcome to submit pr directly.

Temporarily arrange it at 2.5.2, depending on my time to decide
Of course, I also welcome to submit pr directly.

You can try
xmake f --toolchain=mingw -c --mingw=y:\llvm-mingw
xmake project -k make
mingw32-make.exe

It works for me.
C:\Users\ruki\Desktop\test2>xmake f --toolchain=mingw -c --mingw=y:\llvm-mingw
checking for platform ... windows
checking for architecture ... x64
checking for mingw directory ... y:\llvm-mingw

C:\Users\ruki\Desktop\test2>xmake project -k make
create ok!

C:\Users\ruki\Desktop\test2>y:\llvm-mingw\bin\mingw32-make.exe
compiling.release src\main.c
linking.release test2.exe


Update README.md - added windows binaries

Added a link to the nightly windows binaries hosted on Guido Tapia's (my) blog

please also add a line to the installation guide

done
On 22 September 2016 at 16:04, Tianqi Chen<EMAIL_ADDRESS>wrote:

please also add a line to the installation guide
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/xgboost/pull/1600#issuecomment-248819781, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAITHX6pl8g56KDNY1rX_YMWO4SRFIrxks5qshpigaJpZM4KDjXz
.


Setup structure of dash app

Here is what I envision the plots should be placed. I have already placed two sample plots for your reference, team.

How it works is simple. With the reference plots I have created, please add two more plots in order. The outputs and inputs must be in order to be used at the right place of the canvas.

Here are where you should be placing your plots. I have already structured the bootstrap html in a way that resembles our original app sketch.


Screen reader focus gets stolen when reviewing console output.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.68.1
OS Version: Windows 10 21H2 (x64) build 19044.1706

Steps to Reproduce:

Start NVDA
Open a terminal with ctrl+`
Type a few commands to generate output
use control up arrow to review the output
Try to type another terminal command

What happens here is the screen reader's focus is taken away from the terminal text field, and it has to be tabbed to again. This is not ideal, as this is not the behavior in CMD/Windows Terminal/PowerShell, just in VSCode.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.68.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

The issue still remains on VSCode 1.68.1

@TheQuinbox is this something new or has it been happening forever?
I just tried your steps and I can reproduce, but this might be expected since the focus is on the previous lines. Though I do agree that it might be a nice feature that you can type, even while the focus is on previous lines.
Forwarding to @Tyriar and @meganrogge since they are currently looking into terminal accessibility fixes.

You need to press escape currently to go back to the text input

@isidorn, It didn't happen in way earlier versions of Code, I remember that much. Not sure when it started, though. There are workarounds, like @Tyriar pointed out, and that's what I currently use, but it certainly would be useful to not have to press escape after reading previous lines too :).

Yep good feedback 👍

This no longer happens. We now have the accessible view for reviewing commands.


Split remote transcription into its own openai client

I run a local whisper server and a local openai compatible server for text based interactions, but they do not share a hostname. This change allows config to specify that a person would like to use one hostname, api key, and model for transcription, and another for scoring. I've been running this on my local setup for a couple weeks and opening #24 , it seems fine. I pulled in the latest from main, just this morning, but the changes for database don't seem to conflict so it should be relatively tested still.

I see the tests are ignored, do they work?

The transcriber tests worked when I wrote them and I would expect them to still work. They're just marked skip so that CI doesn't try to run them. You should be able to force them to run for local dev.

where can I pick up on this?

Hey! Sorry, I just saw this and was like "I can totally make that one change and get this merged" but then got really carried away and made things more complicated instead of less. I'm going to try to finish it out and ask you to test it for me on your setup tonight.

ok, it seems to work. the feed parser has a bug for the one feed that I live and die by, so I need to chase that down, but that's against the new database backed stuff in main. Thanks for hulk-smashing through!

I did have one question - those of us who aren't using openai, the API token is not very useful in the config. Maybe i'm that only person, but is there a reason to not Optional the key or default it to like "nothing" and only throw when the hostname isn't set?

So, it looks to me like the client implementation
https://github.com/openai/openai-python/blob/ea049cd0c42e115b90f1b9c7db80b2659a0bb92a/src/openai/_client.py#L102-L107
expects us to pass something here, or else falls back to trying to read it from an env var so I think we should require an API key.
As for the the default "nothing" I think having there be no default is a better balance between the self hosted and openai usecases.
The docs I found for faster-whisper-server
https://github.com/fedirz/faster-whisper-server/blob/cbb6c9284f626a7e708d5db8505297419a99154a/README.md?plain=1#L61
seem to acknowledge this wrinkle for the client, so if you get the "no API key" config error as a self-hoster it should be relatively easy to figure out.
For openai users, if you were to omit this field and use a default you could get some pretty puzzling 403s, especially since you specify API key on the root level.

Ok, cool. It seems like that's the direction the rest of the config is written towards. I guess I can't merge, that's you who does it? I have nothing more to add at this time.

Merged! thank you for the PR :)


Aric/deadline scope

Is there enough demand for this that you want me to back port it to production?  Or is after con good enough?

If you think you can port it safely now, let's do it, because we're getting
questions about it. If there's any concern about the backport, let it ride
and I'll just tell people we're being safe about it but it will be released
in mid-August.
On Fri, Jul 16, 2021 at 10:52, Aric Stewart @.***>
wrote:

Merged #567 https://github.com/CON-In-A-Box/CIAB-Portal/pull/567 into
master.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/CON-In-A-Box/CIAB-Portal/pull/567#event-5031693617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAISDXFDNVR5D53ALCWZDHDTYBIVTANCNFSM5APUUG2A
.

--
Michael Scott Shappe
Director, Systems Division
Convergence Events, Inc


This project should have a CHANGELOG.md

Is your feature request related to a problem? Please describe.
We currently do not have a way of keeping track of what is changing from release to release. This will become especially important once we release v1.0, as clients will start depending on functionality being stable and breaking changes to our API should be well documented.
Describe the solution you'd like
https://keepachangelog.com/en/1.1.0/
Describe alternatives you've considered
N/A
Additional context
N/A

Closest thing that we have to a change log right now is the on that's
generated for each release.
Usually I make a big BREAKING CHANGES header for what's going on but
something a little more refined could be good.
On Sat, Sep 23, 2023, 2:22 PM Willow Carretero Chavez <
@.***> wrote:

Is your feature request related to a problem? Please describe.
We currently do not have a way of keeping track of what is changing from
release to release. This will become especially important once we release
v1.0, as clients will start depending on functionality being stable and
breaking changes to our API should be well documented.
Describe the solution you'd like
https://keepachangelog.com/en/1.1.0/
Describe alternatives you've considered
N/A
Additional context
N/A
—
Reply to this email directly, view it on GitHub
https://github.com/TimothyStiles/poly/issues/369, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVXJVFWG3EJESSU7UUCC7LX35HHVANCNFSM6AAAAAA5EOI4VA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Allow attaching external drive as a backup directory

if you label a partition "BACKUP", it will be mounted as a backup folder for hassio

You need to define a full workflow, started from OS over to Supervisor/dbus that they cover a lot of user cases and can be used over the UI.
Please open an issue https://github.com/home-assistant/architecture and describe how a user-friendly backup system should look like on Hass.io | If they will be approved, you can start to implement your concept.
Thanks for your contribution.


select2 inside angularjs  ng-repeat not working

when select2 inside ng-repeat,it can not open the dropdown  list.

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


Make Card component for the cheatsheet page

Currently the url /Cheatsheets shows markdown stuff but that needs to be changed
Create a card component that will take the url and content as props

Need to shift the current content to a new url like /markdown or something like that

This card component will only be visible in mobile view


Unable to login to store when navigated to login page from download sdk.

Description:
Unable to login to store when navigated to login page from download sdk.

Suggested Labels:
APIM 2.1.0 update 4 , Bug, Priority/High Severity/Major
Affected Product Version:
APIM 2.1.0 update 4
OS, DB, other environment details and versions:
Ubuntu 14
Java version "1.8.0_51"
Version 63.0.3239.84 (Official Build) (64-bit)
DB - h2
Steps to reproduce:

Create an API and publish it
Go to store and click on the API (do not login to store)
Click on SDKs
Click on java or android
Provide valid credentials to the login page

Issue
Cannot login to store and no validation message is shown for incorrect login credentials.

Checked in the latests pack. Issue does not exists. Hence resolving the issue.
Note : You will be redirected to the login page if you try to download SDK without login. After a successful login (i.e - with valid credentials), you can download it.


ClockifyClient.FindAllTimeEntriesForProjectAsync "hydrated" parameter makes no sense

As the method can only return unhydrated time entries (TimeEntryDtoImpl) , the parameter "hydrated" is useless.
Suggestion: Separate method for hydrated time entries, which uses appropriate method for the use time entry queries (ClockifyClient.FindAllHydratedTimeEntriesForUserAsync) and returns HydratedTimeEntryDtoImpl

I will check it with current API. Thanks for suggestion!

I see I already have FindAllHydratedTimeEntriesForUserAsync. Is it enough for your needs?

Not really, no. The use case is finding all time entries for a task, regardless of the user, along with the names of the users who made those entries.

That, or FindAllHydratedTimeEntriesForProjectAsync for a wider usage in line with the current non-hydrated method.

Roger that, will do :)


[CI/CD] Build GitHub Action

Setup CI/CD GitHub action to build the project and to deploy to DockerHub the latest image version:

dirigiblelabs/xsk
dirigiblelabs/xsk-cf
dirigiblelabs/xsk-kyma

Fixed with:

https://github.com/SAP/xsk/commit/af085e39c54945d69677421040a1b45524c88ebf
https://github.com/SAP/xsk/commit/b038d5bd7178e6f68443444001b241727ca9d8e6
https://github.com/SAP/xsk/commit/37963ab40180c479efea0b9032a40d57708f7879

Fixed with:

https://github.com/SAP/xsk/commit/af085e39c54945d69677421040a1b45524c88ebf
https://github.com/SAP/xsk/commit/b038d5bd7178e6f68443444001b241727ca9d8e6
https://github.com/SAP/xsk/commit/37963ab40180c479efea0b9032a40d57708f7879


Compass App

Requirements for the app:-
As the phone is moved, the app should be able to show the direction with the degree
Requirements for the README.md file:-

Concepts used
Documentation and Resources used
Future Aspects of the app

@kunal299  Sir, I would like to contribute to this issue.
Can you please assign me this issue.

@avinash14022002 Sure, You can start this project.


docs: Compile list of best practices for developing programs

Such a list might include

setting up CI for verifiable builds + publishing build hashes with build artifacts
encourage repo per program (or logical set of programs) structure so that people can easily follow changes when upgrades occur
multisig as upgrade authority
common foot guns + security vulnerabilities
design patterns (e.g., transfer tokens directly instead of maintaining extra state variables that could potentially get out of sync with the actual token balances, balance isolation, event sourcing, pooling)

I'd love to read it. 👍


[enhancement] check dns pod state as part of crc status operation

This seems to be the root cause for #3851
When we initialize a microshift cluster we ensure the state of the cluster as running then we deploy and expose a service to test the connectivity, when we do it manually it works as expected, but when we run that through an automation it fails with:
time="2023-09-28T13:44:03Z" level=error msg="Post \"http://gateway/hosts/add\": dial tcp: lookup gateway on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:35808-><IP_ADDRESS>:53: read: connection refused"

The code for crc status on microshift checks oc get node but does not take care about the state of the pods running within the cluster.
On this scenario we expected dns is working fine as crc status is running but it is not the case, so it may worthwhile to check the state of the dns to ensure the running state of the cluster.

After adding a check for dns on the e2e now route is always added as it is expected, but we end up having issues with the test service deployed, ending on CreateContainerError due to:
Warning  FailedCreatePodSandBox  34m                    kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = failed to create pod network sandbox k8s_httpd-example-6bf9c787d7-d4mnd_testproj_f468e5fe-b4e9-45b6-b6f6-856426565406_0(6e1a579fdb65e06338b7e1e68f09866369562a4888142d363fb377c770b9bdf2): error adding pod testproj_httpd-example-6bf9c787d7-d4mnd to CNI network "ovn-kubernetes": plugin type="ovn-k8s-cni-overlay" name="ovn-kubernetes" failed (add): failed to send CNI request: Post "http://dummy/": dial unix /var/run/ovn-kubernetes/cni//ovn-cni-server.sock: connect: no such file or directory

So probably we will need to check the state for ovn as well

@adrianriobo is this still relevant?


Update @vue/unit-mocha install instructions

Similar to this comment addressing vue-jest install issues, the install instructions for @vue/unit-mocha have also changed and need to be updated too.
Before change:
Instructions currently read, and produce:
vue add unit-mocha

📦  Installing vue-cli-plugin-unit-mocha...

npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/vue-cli-plugin-unit-mocha - Not found
npm ERR! 404

After change
Instructions that current work, and should read/produce:
vue add @vue/unit-mocha

📦  Installing @vue/cli-plugin-unit-mocha...

+<EMAIL_ADDRESS>added 70 packages from 505 contributors and audited 29226 packages in 17.477s

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[x] Docs
[ ] Underlying tools
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

Other information:

I also realised this also happens in the  instructions for @vue/unit-jest installation instructions.
Tried to install via the current instructions which read vue add unit-jest and it threw the same error as above.


IDK where's my free t shirt

how can i actually help? someone tell me how i can help, i think i just need to learn more

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

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

Hi @mcloota,
The easiest way to start contributing would be starting out with our guides!
Please read this guide on how to get started and feel free to ask any questions!


Please make kubernetes.io available over HTTPS

Hello,
Please consider making the Kubernetes documentation (http://kubernetes.io) available over HTTPS. Better yet, consider applying Strict-Transport-Security, Upgrade-Insecure-Requests, and related security headers to the web site as well.
I apologize if this is a duplicate - I did a quick search but didn't find anything else that looked relevant in this repository. The web is better with HTTPS :)
Jonathan

Note: GitHub just added support for this:
https://konklone.com/post/github-pages-now-sorta-supports-https-so-use-it

It's still a "sorta" level of support.  We're working on a way to do this using Netlify.


Add design dialog to the generated component

In WCM Core, components all have a design dialog that at least includes the styles tab.  We should add the same to the component generator.

Closed by PR #15


Question about loss_functions.py

Hi, thank you for great implementation. I appreciate your work as well as your generosity for opening it.
As mentioned in title, I have a question about line 35 of loss_functions.py, as given below
https://github.com/cvqluu/Angular-Penalty-Softmax-Losses-Pytorch/blob/c41d599622b6a6ba7e5be6faf3a01da1202024ea/loss_functions.py#L35
To my understanding, I think it would initialize new fully connected layer in each epoch of training.
I don't understand how this layer can be optimized via backpropagation, as it would be re-initialized each time.
It would be a great help if anyone can teach me why such inference is wrong.

Hi @junikkoma , nn.Linear is used to initialize the weights of fc layer, and during model training, forward() function is called. Loss only get initialized once here: https://github.com/cvqluu/Angular-Penalty-Softmax-Losses-Pytorch/blob/c41d599622b6a6ba7e5be6faf3a01da1202024ea/models.py#L24


RHEL 7.7 support

This page suggests cloud-init for RHEL 7.7 is available in preview.  However if I try to create VM with RHEL 7.7 through the Azure Portal, cloud-init is shown as unavailable.  Does this mean it's not available and the table is wrong or is there something I've missed?

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

ID: 67fb132b-0dc3-d493-69bc-dc28b5bfd5e1
Version Independent ID: ba5cb476-3e32-911f-0395-0cc8087d1ff3
Content: Overview of cloud-init support for Linux virtual machines in Azure
Content Source: articles/virtual-machines/linux/using-cloud-init.md
Service: virtual-machines-linux
GitHub Login: @danielsollondon
Microsoft Alias: danis

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

Hi, as the RHEL 7.7 cloud-init support is in preview, the default RHEL images in the portal do not use cloud-init to provision, we expect GA cloud-init support for RHEL\CentOS 7.7 very soon, and the images will be updated early next year. Apologies for the confusion, I will update the page.

Thanks @danielsollondon for sharing the insights. :)
I will assign the issue to you to update the documentation.

#reassign: @MicahMcKittrick-MSFT

Page was updated.


Rewrite style props

Checklist
Please update this checklist as you complete each item:

[x] Tests have been included for all bug fixes or added functionality.
[x] The changelog.rst has been updated with any significant changes.
[ ] GitHub Issues which may be closed by this Pull Request have been linked.

Do we want to tackle the custom_vdom_constructor pylint issues?

So custom_vdom_constructor actually does implement an overload. My suspicion is that pylint assumes decorators don't modify function interfaces - hence the error messages. IMO, if you're using MyPy you don't need PyLint. If you want to continue using it, I'd recommend just turning off this check.

Pylint catches a lot of things that I've seen mypy miss. Typically that's a fairly useful error to retain.
It seems like a bug within Pylint, and the generally accepted solution is to directly call the decorator, rather than using the @ decorator operation.
https://stackoverflow.com/questions/40135129/pylint-getting-it-to-understand-decorators


Remed-exam

It is for alitadinugraha

https://github.com/dudung/sk6093-01-2023-2/issues/9
https://github.com/dudung/sk6093-01-2023-2/issues/8

Submit the tasks before ~29 Jun 2024, 2100~ 01 Jul 2024, 1800.

Through this message I would like to say thank you for the opportunity, given to carry out my remedial exams. Below I will tag a link to the remedial assignment that I have done. thank you for your attention.

https://github.com/dudung/sk6093-01-2023-2/issues/9#issuecomment-2199354654
https://github.com/dudung/sk6093-01-2023-2/issues/8#issuecomment-2199121985


Add support for example for use in openapi documentation

https://swagger.io/docs/specification/adding-examples/
Example:
let Simple = sequelize.define('simple', {
  title: {
    type: Sequelize.STRING,
    example: '10 Shocking Things That Go Viral'
  },
  articleId: {
    type: Sequelize.INTEGER,
    example: 42
  },
  tags: {
    type: Sequelize.ARRAY(Sequelize.STRING),
    example: ['clickbait', 'viral', 'article']
  }
});

let def = definition(Simple);
/*
{
  "type": "object",
  "required": ["title", "articleId", "tags"],
  "properties": {
    "title": {
      "type": "string",
      "example": "10 Shocking Things That Go Viral"
    },
    "articleId": {
      "type": "integer",
      "example": 25
    },
    "tags": {
      "type": "array",
      "items": {
        "items": "string"
      },
      "example": ["title", "articleId", "tags"]
    }
  }
}
*/

Closing for a several reasons that mostly boil down to, "Needs better specification".
On the Sequelize side, it's not clear where we should pull examples from.  I'm not seeing any official API documentation for example, examples, or description field.  (A description` field is mentioned here, but not in actual API docs).
On the JSON Schema side, example isn't a valid field either.  examples is (as of  draft 6), but it' has to be a valid JSON schema Object.


Theme Matched Footer

Issue Type:

[x] Feature Request

Describe the bug
Will make a footer which is responsive and theme matched.

@Abh1shekSingh i have some another ideas for it will discuss later


[8.x] PHP 8.1 support

This PR adds some additional fixes for PHP 8.1 support. Thanks to @GrahamCampbell for his help with this and third party library support.
Todo

[x] Check into PDO::ATTR_STRINGIFY_FETCHES behavior: https://github.com/laravel/framework/issues/36813#issuecomment-852418759
[x] Mockery fix: https://github.com/mockery/mockery/pull/1144
[ ] predis/predis PHP 8.1 compatibility
[ ] ramsey/uuid: https://github.com/ramsey/uuid/pull/388
[ ] aws/aws-php-sdk serializable methods: https://github.com/aws/aws-sdk-php/pull/2316
[ ] aws/aws-php-sdk DateTimeResult::fromTimestamp issue: https://github.com/laravel/framework/runs/3599702797#step:8:166
[ ] Argon2 hashing issue: https://github.com/laravel/framework/runs/3599702797#step:8:126
[ ] Fix all tests
[x] Enable build failures

@kylekatarnls  why have you approved a PR in a broken state?

I would like to see ReturnTypeWillChange merged which should unlock the composer install for Laravel. And I think we could have 1 other PR first to add all the #[ReturnTypeWillChange] then the rest in a second step with conflicts fixed when ready.

It’s not ready yet. It’ll probably be another few weeks, at the earliest. Even if the framework code is ready, it’s dependencies are not all ready, which means it still won’t work in practice.

I know, I just released a new fix today for Carbon compatibility with PHP 8.1 interfaces that added new return types.

I would like to see ReturnTypeWillChange merged which should unlock the composer install for Laravel

I can install Laravel v9 just fine right now. Is there anything custom you're doing maybe that's preventing you to do that?

Check into PDO::ATTR_STRINGIFY_FETCHES behavio

I think this is covered now. :)

@taylorotwell I've skipped the tests that are currently failing in PHP 8.1, so we can make this pull request in a mergeble state.


competitive_hackerrank

hacktoberfest2k20
Hey there!, Thanks

@vishwajeet009  This problem already exists as Array Manipulation in the Hackerrank_problems folder.
Please add your solution inside that folder in the format solution_number.cpp


PiCamera not worjking

Software Versions

Python:
OS:
Kivy:
Kivy installation method:

Describe the bug
A clear and concise description of what the bug is.
The issue tracker is a tool to address bugs only (search known bugs: https://git.io/vM1iE).
Please use the #support Discord channel at https://chat.kivy.org/ or Stack Overflow for
support questions, more information at https://git.io/vM1yQ.
Expected behavior
A clear and concise description of what you expected to happen.
To Reproduce
A short, runnable example that reproduces the issue with latest kivy master.
Code and Logs and screenshots
// REPLACE ME: Paste your code and logs here, a template can be found at https://git.io/vM1id.

Additional context
Add any other context about the problem here.

https://github.com/kivy/kivy/issues/7346.


⚠ Bofrost.be has degraded performance

In e6c04e6, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1284 ms

Resolved: Bofrost.be performance has improved in ddd2aef after 6 minutes.


Update managing_transitive_dependencies.adoc

Typo fix
Signed-off-by: Andrey Bekman<EMAIL_ADDRESS>

@abekman Sorry for the broken link in https://github.com/gradle/gradle/pull/6136#issuecomment-409241693. The correct one is:
https://github.com/gradle/gradle/blob/master/CONTRIBUTING.md#signing-off-commits-after-submitting-a-pull-request
Did you add the footer manually? Because now the format is broken, it needs to be Signed-off-by: Andrey Bekman<EMAIL_ADDRESS>There's no need to create another PR, just sign off your commit and force push the branch.

Hi @marcphilipp , I have updated commit comment

@abekman Thanks! The DCO check still wasn't happy because the commit's author name did not match the name in the signed-offf footer. To avoid this in the future, please set your Git user.name (see https://help.github.com/articles/setting-your-username-in-git/) to your real name. I changed the author name manually and merged this to master.


MINOR Moved ApiMessageAndVersion and AbstractApiMessageAndVersionSerde to clients module.

MINOR Moved ApiMessageAndVersion and AbstractApiMessageAndVersionSerde to clients module.
Existing unit tests would be sufficient as this change is more about moving classes into client module.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@junrao  This PR is based on https://github.com/apache/kafka/pull/10271. So, this should be merged only after https://github.com/apache/kafka/pull/10271 is merged.
Review can be done starting with the commit https://github.com/apache/kafka/pull/10578/commits/223ce42ea2a9f18798b5300f386df1cc15d60c31


Highligh Rule SymbolizerEditorWindow

FEATUREREQUEST
Description:

When working with many rules, one might get confused to which rule the currently opened SymbolizerEditorWindow belongs. This problem could be counteracted via highlighting the rule that is currently being edited.

See PR #1073


Update state.py

Updating line which gets "num_devices" to get the backend in general (vs "gpu" specifically)
This is more of a question/confirmation than a necessary PR. I made this general to have it work with a cpu, but feel free to leave it as is if there's a deeper dependency.

Thanks for catching this! The backend should definitely not be hardcoded there


Google App Engine deployment example

[UPDATE] : see the more recent comments below for how simple this is today and all routing working great etc out of box
I deployed to google app engine and routing is working great once I started using the routes configuration. Before that was going in circle for routing in production. Was trying to update handlers in app.yaml and looking for something like that familiar _redirects step for netlify deployments etc.
Anyhow i have a basic example linked below showing the necessary code changes for google app engine deployment. I can list out more steps re: app engine setup if that helps for an example in the docs. It's pretty easy (compared to other cloud providers), but still some stuff missing that could be listed in a clear 1,2,3 steps format for the docs.
https://github.com/NickFoden/starter-express-app-engine/pull/1
MUCHAS GRACIAS!
Between Remix launching and testing out subscriptions/streaming data in faunaDB this week is 11/10.

Hey @NickFoden I know it's been a while, but I'd love to get this in our examples directory. Have you tried running with more recent versions by chance?

@chaance I have not, but have been meaning to update a project. Will circle back here in near future after doing a deployment and see what is required. I think it's going to be much simpler today.

@chaance as expected this is way simpler today and confirmed works great when selecting the Remix App Server I may try again with express. But at moment this is very cool news.
https://github.com/NickFoden/remix-app-server-gae/pull/1

see the Notes.md file, I will clean up with proper bullets and formatting with the gcloud cli commands and push a commit after do some things today.

Deployed here:

https://remix-app-server.ue.r.appspot.com/
routing working well and hitting a 3rd party api in the loader etc

@NickFoden It would be awesome to have a PR that adds an example to our repo.

@NickFoden It would be awesome to have a PR that adds an example to our repo.

Sure thing. Will add this week. Have few urgent items first.

Closing the issue but feel free to add a comment here when/if you're able to get the PR in!


Query Workbench crashes

Executing any query or explain causes a fatal issue in the Query Workbench when connected to a Couchbase Server Community Edition cluster, but works with a Capella connection.
It seems to fail when trying to render results as syntax error are being displayed correctly:

Queries in the SQL++ Notebook works as expected.
Tested with multiple prior versions of the extension and the behaviour is the same.
No exceptions, logs or any other information is displayed
VSCode Version: Version: 1.94.2
CB Server: 7.6.2 Community Edition

Thanks for the feedback, we will work on a fix asap

Hey,
Thank you for reporting this bug. We realised that this exists only in the community edition.  We are also aware that for some reason the Plan tab is empty after the query is successful, and we will take a look at this.
We have released a new version of the extension( v2.1.8) which renders the results as expected and the queries should work as expected. Please let us know if you face any difficulty after updating to new version.
Thanks!

Awesome, thank you very much. It's working great!


HOSTEDCP-1284: Bumps k8s.io/pod-security-admission to v0.28.3

What this PR does / why we need it:

Bumps k8s.io/pod-security-admission to v0.28.3

Which issue(s) this PR fixes:
Fixes HOSTEDCP-1284
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test images
/test unit
/test verify
/test e2e-aws

/test images
/test unit
/test verify
/test e2e-aws

/area ci-tooling

/test e2e-aws

/test e2e-aws

/test e2e-aws

/approve

/lgtm


P2P Stability

Was tendermint/tendermint#9055
Tracking issue for Q3 work on stabilizing the P2P layer.

[ ] #19

[ ] Make a list of potential risks based on the spec work - capture as many of these as possible in issues

[x] tendermint/tendermint#9123
[ ] Stretch goal: go through tendermint/tendermint#5670 to see whether any of the raised issues are applicable to v0.34 and could be addressed in this quarter

I suggest not tracking #19 at this issue, so that it becomes a tracking issue by itself.
This would also allow closing this issue, since #19 is currently its only document dependency.

@cason OK for me


$http legacy promise methods have been deprecated

More information here https://docs.angularjs.org/api/ng/service/$http#deprecation-notice

Thanks for the heads up!

Released as v2.1.1.

Awesome. Thanks. :+1:


Questions re: anyOf/allOf/oneOf, and one more thing

Question
I'm sure there are details I'm unaware of that impact these things, so forgive me if this has been discussed before. I have a few questions/thoughts regarding anyOf/allOf/oneOf:

Could oneOf be translated to an enum with associated values of the possible options?

For anyOf/oneOf where the only options are "null" and some $ref,  could it be translated to an optional instance of that type? Because as of now, it's this:

struct itemPayload: Codable, Hashable, Sendable {
    var value1: OpenAPIRuntime.OpenAPIValueContainer?
    var value2: Components.Schemas.NamedAPIResource?
}

from this:
"item": {
    "anyOf": [
        {
            "type": "null"
        },
        {
            "$ref": "./NamedAPIResource.json"
        }
    ]
}

when it would ideally just be:
typealias itemPayload: Components.Schemas.NamedAPIResource?

Separately, is it possible to set the configuration so the generated types from components/schemas are public, but the initializers and underlying stuff is internal? Or would I just have to do 2 separate targets that generate the types and clients separately, then I combine them for public-facing stuff in a 3rd target?

Hi @edonv,
yeah this is a large topic, so let me answer the individual points inline.
Regarding nullability, check out this maintainer doc (https://swiftpackageindex.com/apple/swift-openapi-generator/1.2.1/documentation/swift-openapi-generator/handling-nullable-schemas), which goes into detail of some of the decisions we made and hopefully answers your question.
At a higher level - we try to apply the smallest amount of post-processing to the OpenAPI doc, we don't flatten various schemas, because we want to stay faithful to the doc (rationale explained here: https://swiftpackageindex.com/apple/swift-openapi-generator/1.2.1/documentation/swift-openapi-generator/project-scope-and-goals).
That means that if you write something like:
Foo:
  type: object
  properties:
    bar:
      anyOf:
        - type: null
        - type: string

we will generate the complicated structure, even though you could have spelled the same thing as:
Foo:
  type: object
  properties:
    bar:
      type: string

The idea is that the OpenAPI document is the source of truth, and we follow that as closely as possible. If your OpenAPI document is written in an inefficient or verbose way, so will your generated Swift code.

Could oneOf be translated to an enum with associated values of the possible options?

That's already the case today. If you're not seeing that, please provide an example of the input OpenAPI snippet and generated code.

For anyOf/oneOf where the only options are "null" and some $ref, could it be translated to an optional instance of that type?

Not really, for the reasons explained earlier - we don't try to optimize your OpenAPI doc, that's up to the OpenAPI doc author. We just generate the most faithful Swift representation of the OpenAPI doc, without first pre-processing it.
If you're looking for ways to find and fix inefficiencies, there are many OSS tools that take an OpenAPI document as input and suggest improvements. But that functionality is out of scope of our Swift code generator, we try to be good OpenAPI community citizens and not needlessly include too many features that are already covered by other tools, as it both increases complexity of the generator and removes flexibility from our adopters.

Thanks for the detailed answer. I'm still relatively new to OpenAPI as it is, so it's likely just that the doc I'm working on isn't as efficient as it could be, leading the generated Swift code to be the same.
But in that vein, in OpenAPI, what would you say would be the best way to represent a property in an object that must be present but can be null? It sounds like there should be a better way than what I example above.

And any suggestions regarding my last question?

Separately, is it possible to set the configuration so the generated types from components/schemas are public, but the initializers and underlying stuff is internal? Or would I just have to do 2 separate targets that generate the types and clients separately, then I combine them for public-facing stuff in a 3rd target?

And any suggestions regarding my last question?

Separately, is it possible to set the configuration so the generated types from components/schemas are public, but the initializers and underlying stuff is internal? Or would I just have to do 2 separate targets that generate the types and clients separately, then I combine them for public-facing stuff in a 3rd target?

That's currently the only supported way, yes. Our integration test package has an example of using the generator in multiple targets to separate the generation of types, client, and server, and sharing the generated types in the other targets: https://github.com/apple/swift-openapi-generator/tree/main/IntegrationTest.

But in that vein, in OpenAPI, what would you say would be the best way to represent a property in an object that must be present but can be null?

That's not easy to do for the reasons outlined in: https://github.com/apple/swift-openapi-generator/issues/513#issuecomment-1911980259
I'd recommend to try to avoid needing to make a distinction between a present null value and an absent value, if at all possible.


It's a w ssome

It's awesome that codon has similar syntax like python and powerful performance.
I have a question that whether codon supports some common python  features or not? For example coroutine relevant, like asyncio IO, or the usage of yield?
Of course codon can integrate python code  just like the exaloop doc shows, but how does this work and will this influence performance?

Hi @drivenow -- you can check the docs under "language" to see the kinds of the things that are supported. yield is supported; async I/O not yet but we plan to incorporate it in a future version.
As for using external Python code/libraries: these will be executed by the regular Python interpreter so will have the same performance. Please let me know if you have any other questions. :)


updated from original

updated from original

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.engchina seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update sap-hana-high-availability-scale-out-hsr-rhel.md

Virtual IP always can fail-over to Master node. No need to add worker node in backend pool

@dipakmadhavi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rdeltcheva Can you take a look at this change and let me know if we can close the pull request? This has a merge conflict, and I'm unclear if the change is still relevant.

#please-close Closing this because it now has a merge conflict and I believe the topic was recently updated. Thank you!


Create a route in frontend to display OpenApI spec

We can show OpenAPI schema in the frontend like swagger ui

It's not necessary at this moment.


Error 403

https://imgur.com/a/OBwUcc7
I have already tried several rpg and I always have the same error, it does not translate anything.

This is because the old translation system is down. Refer to https://github.com/RPG-Maker-Translator/RPG-Maker-Translator/issues/2 for a workaround. (specifically Tranquwil's comment)


[FEATURE] Better Brand specific components

Comparing official Boostrap website and Boosted, it is not very clear which components are specific to Orange brand and default boostrap. For example stepbar.
The naming are not very well defined : Navbar Orange, Orange megamenu, Orange footer
May be another category in left menu could be fine ?

Maybe not another category, but prefix all custom component by Orange? to Group them in left navigation. A complete list of custom component is availabe

Contrary to Boosted v4 where the naming was inconsistent, we chose in Boosted v5 to avoid using "Orange" for the components' naming; except for "Orange navbar" to avoid the collision with the "Navbar" component from Bootstrap.
In our opinion, it's easier to search, to find in the side menu to avoid having 5 "Orange something" components not easily foundable when ordered alphabetically. On top of that, our objective is to have a multi-brand approach at some point, so the naming/the doc should probably tend to be brand-agnostic.
It's a bit hidden in the documentation, but there's a complete list in the About section of Boosted-specific components (IMO "custom" is not the right word) as mentioned by Loïc.
Closing it for now but we can revisit later if needed.


Report if ansible-lint is broken

As ansible-lint have very special requirements for running, we need to be sure that we run ansible-lint --version and report any potential error in a visible way in the UI.
This should make it easier for users to debug broken or incomplete installs of ansible-lint tool.
Related: https://github.com/ansible-community/ansible-lint/issues/1507

I was going to work on this, but there's already a little error popup.
If not installed:
Command failed: ansible-lint  --offline --nocolor -f codeclimate "/tmp/vsc/playbook.yml"
/bin/sh: 1: ansible-lint: not found

If pointing to the ansible-lint in a venv without pointing the rest of the extension to the venv:
Command failed: ./.venv/bin/python /tmp/vsc/.venv/bin/ansible-lint  --offline --nocolor -f codeclimate "/tmp/vsc/playbook.yml"
Failed to guess project directory using git:
ERROR    No module named 'ansible'
FATAL: ansible-lint requires a version of Ansible package >= 2.9, but none was found. Please install a compatible version using the same python interpreter. See https://docs.ansible.com/ansible/latest/installation_guide/intro_installation.html#installing-ansible-with-pip

I think this issue already done?

@ssbarnea, @lilatomic  This PR: https://github.com/ansible/ansible-language-server/pull/5 takes care of the following issue. If there is some problem in installation or configuration of ansible-lint, it reports the user and falls back to --syntax-check.

The Ansible extension 0.7.1 still doesn't have ansible-lint integration working, as @yoshiotobe demonstrated above.
ansible-lint 5.3.2 using ansible 2.12.1


Event property type is not used consistently

In src/events.js's documentation, we claim that all events have a property called type which contains the string that identifies the event.  However, in src/object.js, we store the event identifier in the event object's event property.  We also use the type property on a handful of events from map.js: layerAdd, layerRemove, resize, draw, drawEnd.
I think we should remove the type property, but that would be a breaking change.  Alternately, we could assign the event identifier to both the event and type properties.

I am in favor for removing the type as type and event could confuse the end-user. Since we do not have 1.0 I think it I would be okay with the breaking change. Also event is more connecting than type.

I had already made the decision to remove most of the type references.  PR #718 finishes the job.


Propose a feasible way to add/remove plugins

Problem Statement
From the roadmap

Plugin versions
Dynamically add or remove plugins at build time (to reduce binary size)

Proposed Solution
Instead of packing plugins to binary, making per plugin per image, refer to kuberhealthy's daemonset-check. The plugin manager in Agent will also need some adjustments.
Alternatives Considered
Additional Context

Yeah, I think the design decision not to create new pods for every test is deliberate. This is because when there are 1000s of tests, it puts a lot of load on the apiserver. So we wanted each test run and plugin to have relatively small costs in terms of compute and memory. But there could be a compromise where you deploy a new plugin as a new pod, which gets invoked by pluginmanager at set intervals. Good suggestion!


feat: add material animations

This PR adds the ability to animate and export our custom material properties.
Summary of changes:

Refactor of how material properties are defined

Properties can be defined with 2 "sub-properties": value, animated
To get the value of these properties from now on, we need to access value from the prop.

Allows export of material animations
Export all UV options
Change in how the material panel renders props (use self.draw_prop() now)

Remaining items:

[x] Cleanup code
[x] Wiper animations
[x] Channels being empty
[x] Remove only having animations on certain material types?

The functionality in this PR only works in Blender 3.1 or newer.

The result is wrong, with my test.
there is no material animation, and the glTF exported contains the ASOBO_material_UV_options even if no UV option has been exported
BlenderTests.zip
I created an action for the material animation and 2 to move the object
the 2 action the affects the transform are exported correctly , the material one is missing

Converting to draft until #38 is merged


Bug

I get "Invalid procedure call or argument" in the following code:
in the code "syncDir = collSyncIDToDir(.syncID)" below:
#If Mac Then
    If collSyncIDToDir.Count > 0 Then 'Replace sandbox paths
        For i = 1 To m_providers.pCount
            With m_providers.arr(i)
                Dim syncDir As String: syncDir = collSyncIDToDir(.syncID)
                .mountPoint = Replace(.mountPoint, .baseMount, syncDir)
            End With
        Next i
    End If
#End If

MAC latest OS. Works on one mac, but not the other.
Reset OneDrive.

Thanks @dcircleinc for raising this and the other issues. I will try to check this tomorrow on my Mac.

A note that may help, I signed out of OneDrive within Excel. No change.
I used the mac tool to reset credentials (tool can be found in the Resources folder of the onedrive app’s contents). No change.
I used the reset onedrive tool within the app’s contents, and it didn’t work. I did this again, and the second time, when I opened a file (opening the file from onedrive.com http://onedrive.com/ “in app” or “Open in Excel”), this time it prompted to access to folders (Grant permissions warnings). After that, it worked.
It asked for Grant permissions to some cloud folders and then the actual onedrive folders where the  PDF file being generated is to be saved.
It may be related to Onedrive on Mac (or Onedrive account linked withing excel)  somehow changing or losing permissions to those folders, and then the script not being able to access the folders to iterate and save.
I tried loading a local version of the excel file directy, and it initially did the same thing. I confirmed that the the save-to folder requested with the script does exist locally on the user’s Mac. Just in case it requres that instead of just the "Cloud Folder" that OneDrive may create if the file folder to save the PDF to isn’t set to be synced locally using the OneDrive App.
Thank you,
Ryan Taliercio
@.***

On Sep 23, 2024, at 1:14 PM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc for raising this and the other issues. I will try to check this tomorrow on my Mac.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2368888973, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF4SW65HO2TT4JFYQ23ZYBD53AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRYHA4DQOJXGM.
You are receiving this because you were mentioned.

Thanks @dcircleinc for the detailed info!
I updated to the latest OneDrive version for Mac (the preview one) and I cannot recreate the issue. Can you please test Guido's version of GetLocalPath and see if that has the same issue on the other Mac?

I was unable to reproduce the issue once I ran the reset onedrive tool the second time on that single Mac. I have the latest non-preview version, which was already installed when the error began. I think it has something to do with permissions as after resetting onedrive the second time, the excel file asked to Grant Permissions multiple times to different folders (though I only requested access to one folder), and then it didn’t produce the error. The error just started happening suddenly, nothing changed in the code and OneDrive was already on the same version. The only thing that did change was in onedrive online (web), one of the shared folders was moved to a new location, but the the error occurred on the other workbooks were opened in folders that hadn’t been changed. Maybe there’s a way to force excel to require asking for permissions again. I think this can be done in VB by requesting a folder listing “behind the scenes” using a DIR function, not sure.
Should I update to the new version you pushed anyway?
Ryan Taliercio
@.***

On Sep 25, 2024, at 6:54 AM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc for the detailed info!
I updated to the latest OneDrive version for Mac (the preview one) and I cannot recreate the issue. Can you please test Guido's version of GetLocalPath https://gist.github.com/guwidoe/038398b6be1b16c458365716a921814d and see if that has the same issue on the other Mac?
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2373748122, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF56XIJ4Y64FPZKES5TZYKI47AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZTG42DQMJSGI.
You are receiving this because you were mentioned.

Another update… Now it’s happening on My computer (Mac) out of the blue today. I’m not getting any message, but the LocalPath in my Debug.Print when calling GetLocalPath using the workbook full path from excel, is empty. I reset OneDrive, no change. I signed out of Excel, and back in (to reconnect the OneDrive account within Excel), no change.
I restarted the Mac, and then it worked. The above line then showed the full path properly. It’s the same thing that happened on the other computer. I though resetting OneDrive resolved it, but it was actually restarting the Mac (which explains why when I did the reset OneDrive hing Twice on the other computer,  it worked the second time - I think I restarted the computer after I ran it the second time).
Ps this was the newer code you updated.
Ryan Taliercio
@.***

On Sep 25, 2024, at 6:54 AM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc for the detailed info!
I updated to the latest OneDrive version for Mac (the preview one) and I cannot recreate the issue. Can you please test Guido's version of GetLocalPath https://gist.github.com/guwidoe/038398b6be1b16c458365716a921814d and see if that has the same issue on the other Mac?
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2373748122, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF56XIJ4Y64FPZKES5TZYKI47AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZTG42DQMJSGI.
You are receiving this because you were mentioned.

Maybe there’s a way to force excel to require asking for permissions again

I don't know how to do this but I don't think it's necessary. Once permission is granted the file can be accessed and if permission is denied then next time the macro tries to accesss the file then the Grant Access dialog pops-up anyway.

I though resetting OneDrive resolved it, but it was actually restarting the Mac

I moved a folder that was synced via a shortcut. Nothing I did updated the ini file under OneDrive settings. Until I restarted. This indeed seems to be a OneDrive issue and it makes sense why the code is not working - because it's reading obsolete information. And this was on Windows. On Mac is the same.
It almost feels that we would need to find a way to force update the settings files that OneDrive is using. Maybe there is a way to automate this but I would have to read the Microsoft documentation, if I can find the correct one and if even possible from VBA.
Thanks for now! I wiill leave this open until I find the time to delve into it properly.

An error is appearing now on windows, 2 computers, could be more but don’t know.
ERROR HERE: Runtime Error 9, subscript out of range:
ReDim Preserve outdirs.arrDirs(1 To outdirs.dirCount)
in the below:
Loop Until lastRecord > size
ReDim Preserve outdirs.arrDirs(1 To outdirs.dirCount)
CloseFile:
Close #fileNumber
End Sub
Ryan Taliercio
@.***

On Sep 25, 2024, at 6:54 AM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc for the detailed info!
I updated to the latest OneDrive version for Mac (the preview one) and I cannot recreate the issue. Can you please test Guido's version of GetLocalPath https://gist.github.com/guwidoe/038398b6be1b16c458365716a921814d and see if that has the same issue on the other Mac?
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2373748122, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF56XIJ4Y64FPZKES5TZYKI47AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZTG42DQMJSGI.
You are receiving this because you were mentioned.

Thanks @dcircleinc
I just pushed a new version. Can you please test it? If it still fails can you please provide me with the version of OneDrive where the code fails?

I get this on Mac, invalid procedure call or argument (error 5)
error here: syncDir = collSyncIDToDir(.syncID)
#If Mac Then
If collSyncIDToDir.Count > 0 Then 'Replace sandbox paths
For i = 1 To m_providers.pCount
With m_providers.arr(i)
Dim syncDir As String: syncDir = collSyncIDToDir(.syncID)
.mountPoint = Replace(.mountPoint, .baseMount, syncDir)
End With
Next i
End If
#End If
Ryan Taliercio
@.***

On Oct 1, 2024, at 6:59 AM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc
I just pushed a new version. Can you please test it? If it still fails can you please provide me with the version of OneDrive where the code fails?
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2385472214, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF6J6A7RY42ZXWAWUJTZZJ6CNAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBVGQ3TEMRRGQ.
You are receiving this because you were mentioned.

I think the error may be in the line:
Dim syncDir As String: syncDir = collSyncIDToDir(.syncID)
I don’t think the “.” should be there, not sure but I think it should be:
Dim syncDir As String: syncDir = collSyncIDToDir(syncID)
??
Ryan Taliercio
@.***

On Oct 1, 2024, at 6:59 AM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc
I just pushed a new version. Can you please test it? If it still fails can you please provide me with the version of OneDrive where the code fails?
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2385472214, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF6J6A7RY42ZXWAWUJTZZJ6CNAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBVGQ3TEMRRGQ.
You are receiving this because you were mentioned.

MAC Version on OneDrive:
23.214.1015.0001
But I replaced .syncID with syncID and it works, I didn’t see a WITH so I assume it should be syncID, unless I missed it.
But, this makes it work, which is weird since this same code has been working with this line in it with .syncID.
I wonder if OneDrive changed so now it’s executring this part of the code, whereas before it didn’t execute this part so the error wasn’t encountered. Though I think VBA would want me when I saved it that it wouldn’t compile.
Ryan Taliercio
@.***

On Oct 1, 2024, at 6:59 AM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc
I just pushed a new version. Can you please test it? If it still fails can you please provide me with the version of OneDrive where the code fails?
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2385472214, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF6J6A7RY42ZXWAWUJTZZJ6CNAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBVGQ3TEMRRGQ.
You are receiving this because you were mentioned.

@dcircleinc

I don’t think the “.” should be there

Yes, it does. It refers to the m_providers.arr(i).syncID from the enclosing With block.
Could you please try the latest code? Just pushed a minor change.

Thank you, tried newest code. I get no error on the MAC but it returns an empty path when calling GetRemotePath(ThisWorkbook.FullName)
ThisWorkbook.fullName returns something like https://d.docs.live.net/asdlkjaskldjkal/aslkdalkdjakl/adslkadaskl/ActualFileName, so it seems it’s passing the right value to the function.
When I reverted and removed the .syncID and set it to synciD ,it returns the path properly, but since that’s supposed to be there, I’m not sure how or why it works.
Ryan Taliercio
@.***

On Oct 1, 2024, at 11:22 AM, Cristian Buse @.***> wrote:
@dcircleinc https://github.com/dcircleinc
I don’t think the “.” should be there
Yes, it does. It refers to the m_providers.arr(i).syncID from the enclosing With block.
Could you please try the latest code? Just pushed a minor change.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2386302752, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF35GD4GA6ED6WUC2U3ZZK42HAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGMYDENZVGI.
You are receiving this because you were mentioned.

When I reverted and removed the .syncID and set it to synciD ,it returns the path properly, but since that’s supposed to be there, I’m not sure how or why it works.

This is very interesting. It should not work. I need to update the CreateODDiagnosticsFile method with the extra information for sandboxing that is only relevant to Mac. I will let you know when I'm done so you can run it.
Thanks for all the feedback!

@dcircleinc
Could you please update the latest version and run the CreateODDiagnosticsFile method? This will create a file with some useful information that can guide me in the right direction. Can you then share the file contents?
Many thanks!

@dcircleinc
Thanks for the diagnostics data. I deleted the data because it was in issue #29.
It seems, the code cannot read the following:

the main mount location - I would expect this to be something like /Users/ryan/Library/CloudStorage/OneDrive-Personal/
the sync paths

Your configuration must be different than mine. I will look into this and let you know.

Hope it helps, it’s so frustrating that it works for a few weeks and then something changes and it stops working (mostly on Mac).
You’re doing an amazing job.
This is what I get when I used “syncID" and spit back the local path returned from:
GetLocalPath(thisworkbook.FullName):
"/Users/ryan/Library/CloudStorage/OneDrive-Personal/subdirectory/subdirectory2” and this worked.
In order to get this to work on the Mac, I had to add this, and for whatever reason, it returns the correct local path with syncID and not .syncID.
It seems .syncID worked sometimes.
On Error Resume Next
.syncDir = collSyncIDToDir(.syncID)
                    If (Err.Number <> 0) Then
                     .syncDir = collSyncIDToDir(syncID)
                    End If

                .mountPoint = Replace(.mountPoint, .baseMount, .syncDir)
                On Error GoTo 0

Ryan Taliercio
@.***

On Oct 1, 2024, at 4:53 PM, Cristian Buse @.***> wrote:
@dcircleinc https://github.com/dcircleinc
Thanks for the diagnostics data. I deleted the data because it was in issue #29 https://github.com/cristianbuse/VBA-FileTools/issues/29.
It seems, the code cannot read the following:
the main mount location - I would expect this to be something like /Users/ryan/Library/CloudStorage/OneDrive-Personal/
the sync paths
Your configuration must be different than mine. I will look into this and let you know.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2387053611, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDFZT4JII56IE72E5R7TZZMDS3AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBXGA2TGNRRGE.
You are receiving this because you were mentioned.

Thanks again for all the help so far. I still need some more info if possible.
In the 95bbe8e999b9a6af.ini file, the first line should be like library = .... I would need that line to see why the mount point is not being read. This file can be found at: /Users/ryan/Library/Application Support/OneDrive/settings/.
As for the sync IDs, there should be a file called .849C9593-D756-4E56-8D6E-42412F2A707B at: /Users/ryan/Library/CloudStorage/OneDrive-Personal/. You might need to CTRL+Shift+. from keyboard (Command+Shift+Dot). The contents of the file should look like this: {"guid" : "564ad8ab-4e36-4f3a-9eff-0ae1bb13a24d","version" : 1}. Is the file there and what is the content?

Ryan Taliercio
@.***

On Oct 1, 2024, at 5:32 PM, Cristian Buse @.***> wrote:
Thanks again for all the help so far. I still need some more info if possible.
In the 95bbe8e999b9a6af.ini file, the first line should be like library = .... I would need that line to see why the mount point is not being read. This file can be found at: /Users/ryan/Library/Application Support/OneDrive/settings/.

I can’t find this directory nor the .ini file. I searched for the .ini as well, no result

As for the sync IDs, there should be a file called .849C9593-D756-4E56-8D6E-42412F2A707B at: /Users/ryan/Library/CloudStorage/OneDrive-Personal/. You might need to CTRL+Shift+. from keyboard (Command+Shift+Dot). The contents of the file should look like this: {"guid" : "564ad8ab-4e36-4f3a-9eff-0ae1bb13a24d","version" : 1}. Is the file there and what is the content?

Content of that file is:
{"guid" : "60f16864-3de4-4936-a61f-2be2e8e7777a","version" : 1}

—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2387114255, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDFY7LFDZFNNHEKZZWHDZZMIIXAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBXGEYTIMRVGU.
You are receiving this because you were mentioned.

I can’t find this directory nor the .ini file. I searched for the .ini as well, no result

Maybe here: /Users/ryan/Library/Containers/com.microsoft.OneDrive-mac/Data/Library/Application Support/OneDrive/settings/

On Oct 1, 2024, at 5:56 PM, Cristian Buse @.***> wrote:
/Users/ryan/Library/Containers/com.microsoft.OneDrive-mac/Data/Library/Application Support/OneDrive/settings/

Found it in Users/ryan/Library/Containers/com.microsoft.OneDrive-mac/Data/Library/Application Support/OneDrive/settings/Personal - but it will not let me open it. I tried copying it and it says the file is “damaged”. I closed OneDrive and tried, same thing.
Then a strange thing, when I opened onedrive it made me sign in again.
Now I opened the other file and the guid is different:
{"guid" : "d4634849-bed4-43dc-b6e8-bf6b226358a5","version" : 1}

Try opening with TextEditor app. If it still does not open then please send me the file at<EMAIL_ADDRESS>Weird that the guid updated. The one starting with 6 looked wrong anyway.

Sorry, I don’t know what happened now. When I tried to open that file, even in textedit. it wouldn’t open.
Then onedrive had me sign in again, and it keeps saying now I have 2 of the same onedrive accounts running, and i’m in a loop.
I try to delete one, and then the same thing happens when I sign in, so I’m stuck.
Here was the ini file from previous that I was unable to open.
￼
Ryan Taliercio
@.***

On Oct 1, 2024, at 6:39 PM, Cristian Buse @.***> wrote:
Try opening with TextEditor app. If it still does not open then please send me the file at @.*** @.***>
Weird that the guid updated. The one starting with 6 looked wrong anyway.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2387208394, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF6DPBOF4IAWUUHUZNTZZMQB7AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBXGIYDQMZZGQ.
You are receiving this because you were mentioned.

I was able to open it in windows notepad. Textedit on mac wouldn’t do it:
library = 1 4 95BBE8E999B9A6AF!103 327456003 "SkyDrive" Me personal "/Users/ryan/Library/Group Containers/UBF8T346G9.OneDriveSyncClientSuite/OneDrive.noindex/OneDrive" 1 564ad8ab-4e36-4f3a-9eff-0ae1bb13a24d  - 197948186 16777229 D2A0343F-60C2-4785-B278-FE4D7F8DEAAB
installID = 1
originatorID = 5fc35e4b-4ba5-4d18-bfe2-01e439c50250
The path is to this 849C9593-D756-4E56-8D6E-42412F2A707B file
is now: /Users/ryan/Library/CloudStorage/OneDrive
The GUID in the new file now that I had to reinstall again is:
{"guid" : "eaaa01d4-9a8d-43cb-879c-f364c86c2151","version" : 1}
Ryan Taliercio
@.***

On Oct 1, 2024, at 6:39 PM, Cristian Buse @.***> wrote:
Try opening with TextEditor app. If it still does not open then please send me the file at @.*** @.***>
Weird that the guid updated. The one starting with 6 looked wrong anyway.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2387208394, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF6DPBOF4IAWUUHUZNTZZMQB7AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBXGIYDQMZZGQ.
You are receiving this because you were mentioned.

If this helps, I opened up a different file (which is using the same methods) after reinstalling OneDrive.
I got an error in:
If i > 0 Then
relPath = Mid$(lineText, i + 8)
folderID = Left$(lineText, i - 1)
If oDirs.dirCount = 0 Then ReadODDirs aInfo, oDirs
With m_providers.arr(AddProvider())
.webPath = mainURL & cID & "/" & relPath & "/"
relPath = oDirs.arrDirs(oDirs.idToIndex(folderID)).dirName
.mountPoint = BuildPath(mainMount, relPath & "/")
.baseMount = mainMount
.syncID = syncID
End With
End If
I left the file open, quit onedrive, opened it again, and the erorr disappeared and the proper path was returned.
I haven’t tried again to exit onedrive, start again, and then open Excel and see what happens.
Ryan Taliercio
@.***

On Oct 1, 2024, at 6:39 PM, Cristian Buse @.***> wrote:
Try opening with TextEditor app. If it still does not open then please send me the file at @.*** @.***>
Weird that the guid updated. The one starting with 6 looked wrong anyway.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2387208394, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF6DPBOF4IAWUUHUZNTZZMQB7AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBXGIYDQMZZGQ.
You are receiving this because you were mentioned.

Then onedrive had me sign in again, and it keeps saying now I have 2 of the same onedrive accounts running, and i’m in a loop.

OneDrive has probably updated on your computer(s) but it kept files in different places and now it's all "confused" about it.
Could you please use the latest version of this repo (did some changes last night) and re-run the ``CreateODDiagnosticsFile``` method? Thanks!

I replaced with new verision, attached is the file.
Local path appears to be returning incorrect intermittently, but I get no error (I only see an error in my code because I have a check that creates a subfolder, and it can’t find the path the function returns, and spits back an error. When I check the path its as below, and somewhere on the root that doesn’t exist.
Folder Path Initial: https://d.docs.live.net/d60675b8af8da0f7/DesignCenter Cash Sheets/Cash_Sheet_10_2024_Design Center.xlsm
Folder Path Remote to Local (GetLocalPath) : /Folder1/filename.xlsm
When I close Excel and restart, it works and returns the actual local long “cloud” path. It seems to be random.
When I get that incorrect path returned, If I put in the If statmenet for the .syncID to return syncID instead, it returns the full cloud path correctly.
Ryan Taliercio
@.***
:￼

On Oct 2, 2024, at 6:09 AM, Cristian Buse @.***> wrote:
Then onedrive had me sign in again, and it keeps saying now I have 2 of the same onedrive accounts running, and i’m in a loop.
OneDrive has probably updated on your computer(s) but it kept files in different places and now it's all "confused" about it.
Could you please use the latest version of this repo (did some changes last night) and re-run the ``CreateODDiagnosticsFile``` method? Thanks!
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2388241790, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF4JO52SDDTHRLEAXLLZZPA6DAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBYGI2DCNZZGA.
You are receiving this because you were mentioned.

Mac VBA7 x64

Accounts found: 2
Name: Personal
ID: 95bbe8e999b9a6af
Has DAT: False
Is Valid: True
Name: Personal
ID: 95bbe8e999b9a6af
Has DAT: True
Is Valid: False

Providers found: 8
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/95bbe8e999b9a6af/
Sync Dir:
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /_Seigermans CASH SHEETS/
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/de05781aea838628/Documents/Seigermans CASH SHEETS/
Sync Dir:
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /__CASHSHEET TEMPLATES/
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/d60675b8af8da0f7/CASHSHEET TEMPLATES/
Sync Dir:
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /_Natuzzi Cash Sheets/
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/8cc293ffa7f430b9/Natuzzi Cash Sheets/
Sync Dir:
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /_110 CASH SHEETS/
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/51207bab5645f67e/CASH SHEETS IN HERE/
Sync Dir:
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /_SHOWPLACE CASHSHEETS/
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/39e895601e7750c9/Desktop/CASHSHEETS/
Sync Dir:
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /_DesignCenter Cash Sheets/
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/d60675b8af8da0f7/DesignCenter Cash Sheets/
Sync Dir:
Base Mount: $Group.Container$/OneDrive.noindex/OneDrive
Is Business: False
Is Main: False
Mount Point: /_Sofabed Cashsheets/
Sync ID: d641d3be-9319-40bf-bff0-a1e0f977e0ba
Web Path: https://d.docs.live.net/d60675b8af8da0f7/Sofabed Cashsheets/
Sync Dir:

Ok, so the problem still lies with the Sync IDs and apparently for some reason the Main Mount is blank until you restart.
Hopefully last thing I ask, can you please put the below code into a new module, run the PrintSyncIDs method and let me know what got printed to the Immediate window?
Option Explicit

Sub PrintSyncIDs()
    Const syncIDFileName As String = ".849C9593-D756-4E56-8D6E-42412F2A707B"
    Dim collCloudDirs As Collection: Set collCloudDirs = GetODCloudDirs()
    Dim collFiles As New Collection
    Dim odCloudDir As Variant
    Dim arrPaths() As String
    Dim syncID As String
    Dim fileName As String
    Dim folderPath As String
    Dim i As Long
    '
    For Each odCloudDir In collCloudDirs
        collFiles.Add odCloudDir
        collFiles.Add odCloudDir & "/" & syncIDFileName
    Next odCloudDir
    arrPaths = CollectionToStrings(collFiles)
    If Not GrantAccessToMultipleFiles(arrPaths) Then Exit Sub
    '
    Set collFiles = New Collection
    For Each odCloudDir In collCloudDirs
        syncID = ReadSyncID(odCloudDir & "/" & syncIDFileName)
        If LenB(syncID) > 0 Then
            Debug.Print odCloudDir, syncID
        Else
            fileName = Dir(odCloudDir & "/", vbDirectory)
            Do While LenB(fileName) > 0
                folderPath = odCloudDir & "/" & fileName
                collFiles.Add folderPath
                collFiles.Add folderPath & "/" & syncIDFileName
                fileName = Dir
            Loop
        End If
    Next odCloudDir
    If collFiles.Count > 0 Then
        arrPaths = CollectionToStrings(collFiles)
        If Not GrantAccessToMultipleFiles(arrPaths) Then Exit Sub
        '
        For i = LBound(arrPaths) To UBound(arrPaths) Step 2
            syncID = ReadSyncID(arrPaths(i + 1))
            If LenB(syncID) > 0 Then Debug.Print arrPaths(i), syncID
        Next i
    End If
End Sub

Private Function GetODCloudDirs() As Collection
    Dim coll As New Collection
    Dim cloudPath As String:  cloudPath = GetCloudPath()
    Dim folderName As String: folderName = Dir(cloudPath, vbDirectory)
    '
    Do While LenB(folderName) > 0
        If folderName Like "OneDrive*" Then
            coll.Add BuildPath(cloudPath, folderName)
        End If
        folderName = Dir
    Loop
    Set GetODCloudDirs = coll
End Function
Private Function GetCloudPath() As String
    GetCloudPath = GetUserPath() & "Library/CloudStorage/"
End Function
Private Function GetUserPath() As String
    GetUserPath = "/Users/" & Environ$("USER") & "/"
End Function
Private Function CollectionToStrings(ByVal coll As Collection) As String()
    If coll.Count = 0 Then
        CollectionToStrings = Split(vbNullString)
        Exit Function
    End If
    '
    Dim res() As String: ReDim res(0 To coll.Count - 1)
    Dim i As Long
    Dim v As Variant
    '
    For Each v In coll
        res(i) = v
        i = i + 1
    Next v
    CollectionToStrings = res
End Function
Private Function ReadSyncID(ByRef syncFilePath As String) As String
    Dim b() As Byte:       ReadBytes syncFilePath, b
    Dim parts() As String: parts = Split(StrConv(b, vbUnicode), """guid"" : """)
    '
    If UBound(parts) < 1 Then Exit Function
    ReadSyncID = Left$(parts(1), InStr(1, parts(1), """") - 1)
End Function

I forgot to mention, the code above must be placed in the same project as the LibFileTools module as it uses a few methods from it.

It was /Users/ryan/Library/CloudStorage/OneDrive-Personal  3b7c602a-3493-42c7-8064-7f39a156338d
It seems to be working at the moment, but the other day I had the same problem suddenly so I had to comment the .synciID again, but this time it worked without the error trapped and .syncID replaced with syncID value.
Just in case, I’m writing an alternate method using DIR and similar functions on Mac. I take the currently open file and search the user’s onedrive for the file. I get the onedrive path from ENVIRON(“HOME”) and append “CloudStorage” (removing the com.microsoft.excel path that it returns). Then I search that directory for the file name, and then use that as a possible starting point to set the local directory. In my case this work since the filename is unique for the user. I also sort the results of the DIR loop by newest first - since they have the document open and it saves just before running, it should be the first on the list if there are matches. I have this working in both windows and Mac so far.
Ryan Taliercio
@.***

On Oct 4, 2024, at 11:23 AM, Cristian Buse @.***> wrote:
I forgot to mention, the code above must be placed in the same project as the LibFileTools module as it uses a few methods from it.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2393952033, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDFYZ2BYTKRZUNQUOBODZZ2XF5AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJTHE2TEMBTGM.
You are receiving this because you were mentioned.

Still, It makes absolutely no sense that it works with syncID instead of .syncID. The former is the result of the last syncID = ReadSyncID(... that happened in the same method and should be already added to the collSyncIDToDir collection as a key.
I wish I was able to debug this myself. it's driving me nuts as I cannot replicate.

Let me try to recreate the scenario, disable the error catching,  and I’ll then run the PrintSyncID tool.
It also happened on a windows computer when I removed a shared folder for a user, and readded them with a new OneDrive account. When I opene the file in excel, I got an error in the same part of the code. When I then exited onedrive, without restarting excel, the script worked. When I started onedrive again, it was fine. Please note this new Onedrive account was added to Excel as well.
I had to add a new account for this user as that particular computer got that OneDrive worldwide bug where shared folders do not appear in the onedrive app, and appear as shortcuts only in their folder. When clicking on these, they take you to the web version of the page. There are no local folders associated with the shared folder. Microsoft hasn’t been able to fix it. The account was fine then suddenly the folders were replaced with shortcuts. I have no solution other than to add new dummy accounts.
Ryan Taliercio
@.***

On Oct 4, 2024, at 12:27 PM, Cristian Buse @.***> wrote:
Still, It makes absolutely no sense that it works with syncID instead of .syncID. The former is the result of the last syncID = ReadSyncID(... that happened in the same method and should be already added to the collSyncIDToDir collection as a key.
I wish I was able to debug this myself. It's driving me nuts as I cannot replicate.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2394062201, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF3E6WQI6J5E4R5UCNLZZ26WFAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUGA3DEMRQGE.
You are receiving this because you were mentioned.

Hi Ryan,
Any updates on this? I've been a bit swamped in the last 2-3 weeks and had no chance to revisit. Thanks!

Was I supposed to send something else ?  It sure if I missed something .I still have that line commented out but I removed it and it’s working on my Mac at the moment. It seems so random but may to be tied to when OneDrive updates or is reinstalled/reset. I wrote a brute force method which  saves the current file so it has the newest modified date, looks for the “OneDrive” folder name (with wildcards). it starts with a likely default location on Mac and uses the environment variable in windows. It then parses those to find the NEWEST file with the currently open file name just in case there are duplicates, and then uses that as the path to OneDrive (I can strip out the subdirectories if needed) but in my case this is the folder I need. For Mac i have a Macscript method for director parsing and “save as” dialog, but I also added an AppleScripttask and made a little Automator app which automatically copies the proper .scpt I need to execute (to do the parsing/searching) into the Application Scripts / com.Microsoft.excel folder on the user’s Mac. I’m Not sure what functions of Macscript will stop working in the future (or any day) so anything that uses it I’m Afraid might stop working, and I do see the vbafiletools uses Macscript. This is a problem on the Mac when promoting a dialog box with the save as folder and file name pre-selected as Macscript is needed, but the AppleScriptTask  version seems to work (slower but it’s just a starting point). This also works on windows and uses the shell “find” commands. Sent from my iPhoneOn Oct 21, 2024, at 4:52 AM, Cristian Buse @.***> wrote:
Hi Ryan,
Any updates on this? I've been a bit swamped in the last 2-3 weeks and had no chance to revisit. Thanks!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Was I supposed to send something else ?

No. But you mentioned that you are writing an alternative method and was wondering if while doing it, you stumbled across the answer of how the issue is happening.

I wrote a brute force method ....

Ok. I think I understand what you've done. Thanks!
I will think about this in the next few days and will let you know if any idea comes to mind.

Here’s another update from testing, where I received an error in this code below in this line, with error “Runtime error 9, Subscript out of range”). In this case there is an “old” onedrive folder, and we connected to the “new” one.
ReDim Preserve outdirs.arrDirs(1 To outdirs.dirCount)
This happens when I added a new OneDrive account to the systtem, add it to Excel Accounts (onedrive access within Excel), and then open the file and attempt to run the library.
If I reboot, the error disappears, but it seems to happen when we just add a new onedrive signin and then attempt to access the library without rebooting.
Private Sub ReadDirsFromDB(ByRef filePath As String _
, ByVal isPersonal As Boolean _
, ByRef outdirs As DirsInfo)
If Not IsFile(filePath) Then Exit Sub
Dim fileNumber As Long: fileNumber = FreeFile()
'
Open filePath For Binary Access Read As #fileNumber
Dim size As Long: size = LOF(fileNumber)
If size = 0 Then GoTo CloseFile
'
Const chunkSize As Long = &H100000 '1MB
Const minName As Long = 15
Const maxSigByte As Byte = 9
Const maxHeader As Long = 21
Const minIDSize As Long = 12
Const maxIDSize As Long = 48
Const minThreeIDSizes As Long = minIDSize * 3
Const maxThreeIDSizes As Long = maxIDSize * 3
Const leadingBuff As Long = maxHeader + maxThreeIDSizes
Const headBytesOffset As Long = 15
Const bangCode As Long = 33 'Asc("!")
Dim curlyStart As String: curlyStart = ChrW$(&H7B22) '"{
Dim quoteB As String:     quoteB = ChrB$(&H22)       '"
Dim bangB As String:      bangB = ChrB$(bangCode)    '!
Dim sig As String
Dim b(1 To chunkSize) As Byte
Dim s As String
Dim lastRecord As Long
Dim i As Long
Dim j As Long
Dim k As Long
Dim idSize(1 To 4) As Long
Dim nameSize As Long
Dim dirID As String
Dim parentID As String
Dim dirName As String
Dim nameEnd As Long
Dim nameStart As Long
Dim isASCII As Boolean
Dim mustAdd As Boolean
Dim idPattern As String
'
idPattern = Replace(Space$(12), " ", "[a-fA-F0-9]")
If isPersonal Then
sig = bangB
idPattern = "" & idPattern & "![a-fA-F0-9]"
Else
sig = curlyStart
idPattern = idPattern & "*"
End If
Do
Dim currFileChange As Date: currFileChange = FileDateTime(filePath)
Dim lastFileChange As Date
'
i = 0
If currFileChange > lastFileChange Then
With outdirs
Set .idToIndex = New Collection
.dirCount = 0
.dirUBound = 256
ReDim .arrDirs(1 To .dirUBound)
End With
lastFileChange = currFileChange
lastRecord = 1
End If
Get fileNumber, lastRecord, b
s = b
i = InStrB(1, s, sig)
Do While i > 0
If isPersonal Then
For j = i - 1 To i - maxIDSize Step -1
If j = 0 Then GoTo NextSig
If b(j) < bangCode Then Exit For
Next j
If (j < maxHeader) Or (i - j < minIDSize) Then GoTo NextSig
Else
j = InStrB(i + 2, s, quoteB)
If j = 0 Then Exit Do 'Next chunk
idSize(4) = j - i + 1
If idSize(4) > maxIDSize Then GoTo NextSig
For j = i - 1 To i - maxThreeIDSizes Step -1
If j = 0 Then GoTo NextSig
If b(j) < bangCode Then Exit For
Next j
If j < maxHeader Then GoTo NextSig
idSize(1) = i - j - 1 'ID 1+2+3
If idSize(1) < minThreeIDSizes Then GoTo NextSig
End If
'
k = j + 1 'ID1 Start
For j = j To j - headBytesOffset + 1 Step -1
If b(j) > maxSigByte Then GoTo NextSig
Next j
If (b(j) <= maxSigByte) And (b(j - 1) < &H80) Then j = j - 1
If b(j) < minName Then j = j - 1
'
nameSize = b(j)
If nameSize Mod 2 = 0 Then GoTo NextSig
nameSize = (nameSize - 13) / 2
If b(j - 1) > &H7F Then
nameSize = (b(j - 1) - &H80) * &H40 + nameSize
j = j - 1
End If
If (nameSize < 1) Or (b(j - 4) = 0) Then GoTo NextSig
'
If isPersonal Then
idSize(4) = (b(j - 1) - 13) / 2
idSize(3) = (b(j - 2) - 13) / 2
idSize(2) = (b(j - 3) - 13) / 2
idSize(1) = (b(j - 4) - 13) / 2
nameStart = k + idSize(1) + idSize(2) + idSize(3) + idSize(4)
Else
If b(j - 1) <> idSize(4) * 2 + 13 Then GoTo NextSig
idSize(3) = (b(j - 2) - 13) / 2
idSize(2) = (b(j - 3) - 13) / 2
idSize(1) = idSize(1) - idSize(2) - idSize(3)
nameStart = i + idSize(4)
End If
For j = 1 To 4
If (idSize(j) < minIDSize) _
Or (idSize(j) > maxIDSize) Then GoTo NextSig
Next j
'
nameEnd = nameStart + nameSize - 1
If nameEnd > chunkSize Then Exit Do 'Next chunk
'
dirID = StrConv(MidB$(s, k, idSize(1)), vbUnicode)
If Not dirID Like idPattern Then GoTo NextSig
'
k = k + idSize(1)
parentID = StrConv(MidB$(s, k, idSize(2)), vbUnicode)
If Not parentID Like idPattern Then GoTo NextSig
'
If isPersonal Then
k = k + idSize(2)
If Not StrConv(MidB$(s, k, idSize(3)), vbUnicode) _
Like idPattern Then GoTo NextSig
If Not StrConv(MidB$(s, k + idSize(3), idSize(4)), vbUnicode) _
Like idPattern Then GoTo NextSig
End If
'
On Error Resume Next
j = outdirs.idToIndex(dirID)
mustAdd = (Err.Number <> 0)
On Error GoTo 0
'
If mustAdd Then
With outdirs
.dirCount = .dirCount + 1
If .dirCount > .dirUBound Then
.dirUBound = .dirUBound * 2
ReDim Preserve .arrDirs(1 To .dirUBound)
End If
.idToIndex.Add .dirCount, dirID
j = .dirCount
End With
With outdirs.arrDirs(j)
.dirName = MidB$(s, nameStart, nameSize)
.isNameASCII = True
For k = nameStart To nameEnd
If b(k) > &H7F Then
.isNameASCII = False
Exit For
End If
Next k
If .isNameASCII Then
.dirName = StrConv(.dirName, vbUnicode)
Else
.dirName = ConvertText(.dirName, codeUTF16LE, codeUTF8)
End If
.dirID = dirID
.parentID = parentID
End With
Else
With outdirs.arrDirs(j)
If (Not .isNameASCII) Or (Len(.dirName) < nameSize) Then
dirName = MidB$(s, nameStart, nameSize)
isASCII = True
For k = nameStart To nameEnd
If b(k) > &H7F Then
isASCII = False
Exit For
End If
Next k
If isASCII Then
.dirName = StrConv(dirName, vbUnicode)
Else
.dirName = ConvertText(dirName, codeUTF16LE, codeUTF8)
End If
.isNameASCII = isASCII
End If
End With
End If
i = nameEnd
NextSig:
i = InStrB(i + 1, s, sig)
Loop
If i = 0 Then
lastRecord = lastRecord + chunkSize - leadingBuff
ElseIf i > leadingBuff Then
lastRecord = lastRecord + i - leadingBuff
Else
lastRecord = lastRecord + i
End If
Loop Until lastRecord > size
ReDim Preserve outdirs.arrDirs(1 To outdirs.dirCount)
CloseFile:
Close #fileNumber
End Sub
Ryan Taliercio
@.***

On Oct 21, 2024, at 8:59 AM, Cristian Buse @.***> wrote:
Was I supposed to send something else ?
No. But you mentioned that you are writing an alternative method and was wondering if while doing it, you stumbled across the answer of how the issue is happening.
I wrote a brute force method ....
Ok. I think I understand what you've done. Thanks!
I will think about this in the next few days and will let you know if any idea comes to mind.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2426606350, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDFZERBXVFGCE2FWX52LZ4T3CRAVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRWGYYDMMZVGA.
You are receiving this because you were mentioned.

I’m sorry I didn’t make clear, this was happening on 2 PC’s (not mac). This was when I added a new account to onedrive, connected it within Excel. Then the script giave this error. When I restarted OneDrive manually, or rebooted, the error disappeared.
Ryan Taliercio
@.***

On Sep 23, 2024, at 1:14 PM, Cristian Buse @.***> wrote:
Thanks @dcircleinc https://github.com/dcircleinc for raising this and the other issues. I will try to check this tomorrow on my Mac.
—
Reply to this email directly, view it on GitHub https://github.com/cristianbuse/VBA-FileTools/issues/30#issuecomment-2368888973, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHBDF4SW65HO2TT4JFYQ23ZYBD53AVCNFSM6AAAAABOWLBU3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRYHA4DQOJXGM.
You are receiving this because you were mentioned.

Thanks for the info Ryan!
I could look into forcefully terminate the OneDrive process via an API call and possibly restart it. I've done this many years ago but not sure how reliable it would be nowadays or if it's even allowed by modern antivirus software.
Before I even go down that rabbit hole, I assume you've already tried to set the rebuildCache argument to True when calling GetLocalPath / GetRemotePath? Thanks!


Add kubeProxy stanza to kops manifest

This allows Prometheus to scrape the kubeProxy endpoint as outlined in https://github.com/ministryofjustice/cloud-platform/issues/1636

@jasonBirchall now that we've done the kops upgrade, should this be merged?


AF2_advanced crashed upon processing sequence with repeat

Hi Yoshitaka-san,
I tried running AF2_Advanced on these two files separately:
>HHAWx4
HHAWHHAWHHAWHHAW

And
>KKAWx4
KKAWKKAWKKAWKKAW

It gave me the following error message:
running mmseqs2
  0%|          | 0/150 [elapsed: 00:00 remaining: ?]
Traceback (most recent call last):
  File "/home/ubuntu/storage1/colabfold/runner_af2advanced.py", line 163, in <module>
    hhfilter_loc="colabfold-conda/bin/hhfilter", precomputed=precomputed, TMP_DIR=output_dir)
  File "/home/ubuntu/storage1/colabfold/colabfold_alphafold.py", line 292, in prep_msa
    A3M_LINES = cf.run_mmseqs2(I["seqs"], prefix, use_filter=True, host_url=mmseqs_host_url)
  File "/home/ubuntu/storage1/colabfold/colabfold.py", line 150, in run_mmseqs2
    raise Exception(f'MMseqs2 API is giving errors. Please confirm your input is a valid protein sequence. If error persists, please try again an hour later.')
Exception: MMseqs2 API is giving errors. Please confirm your input is a valid protein sequence. If error persists, please try again an hour later.

How can I resolve the issue?
G.V.

Please confirm your input is a valid protein sequence.

This is not an issue of localcolabfold, but mmseqs2. The two sequences you showed gave the same error in my localcolabfold environment. Did you test them on GoogleColab-AlphaFold2_advanced.ipynb? I guess it will cause an error.
Please test it on the notebook before you report here.

Please confirm your input is a valid protein sequence.

This is not an issue of localcolabfold, but mmseqs2. The two sequences you showed gave the same error in my localcolabfold environment. Did you test them on GoogleColab-AlphaFold2_advanced.ipynb? I guess it will cause an error. Please test it on the notebook before you report here.

Yoshitaka-san,
You're right. I tried on GoogleColab-AlphaFold2_advanced.ipynb.
It gives the same error message. I'll check with them.
G.V.