Collabora

Is it possible to setup this ? i found some config files für nginx but dont know how to port this.
for Example:
server {
    listen       443 ssl;
    server_name  office.example.com;

    ssl_certificate /path/to/certficate;
    ssl_certificate_key /path/to/key;

    # static files
    location ^~ /loleaflet {
        proxy_pass https://localhost:9980;
        proxy_set_header Host $http_host;
    }

    # WOPI discovery URL
    location ^~ /hosting/discovery {
        proxy_pass https://localhost:9980;
        proxy_set_header Host $http_host;
    }

   # main websocket
   location ~ ^/lool/(.*)/ws$ {
       proxy_pass https://localhost:9980;
       proxy_set_header Upgrade $http_upgrade;
       proxy_set_header Connection "Upgrade";
       proxy_set_header Host $http_host;
       proxy_read_timeout 36000s;
   }

   # download, presentation and image upload
   location ~ ^/lool {
       proxy_pass https://localhost:9980;
       proxy_set_header Host $http_host;
   }

   # Admin Console websocket
   location ^~ /lool/adminws {
       proxy_pass https://localhost:9980;
       proxy_set_header Upgrade $http_upgrade;
       proxy_set_header Connection "Upgrade";
       proxy_set_header Host $http_host;
       proxy_read_timeout 36000s;
   }
}

Sure can. I didn't see a default /' location there so I did a google and other commabora examples have one.

Create your new Proxy Host, specify https protocol and the server host/ip and port 9980. Also enable Websockets Support.
On the SSL tab select a new certificate, enable HTTP/2.
On the Advanced tab, enter:

# static files
location ^~ /loleaflet {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
}

# WOPI discovery URL
location ^~ /hosting/discovery {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
}

# main websocket
location ~ ^/lool/(.*)/ws$ {
  proxy_set_header Upgrade $http_upgrade;
  proxy_set_header Connection "upgrade";
  proxy_http_version 1.1;
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
  proxy_read_timeout 36000s;
}

# download, presentation and image upload
location ~ ^/lool {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
}

# Admin Console websocket
location ^~ /lool/adminws {
  proxy_set_header Upgrade $http_upgrade;
  proxy_set_header Connection "upgrade";
  proxy_http_version 1.1;
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
  proxy_read_timeout 36000s;
}

Save and test.

worked perfectly :) so thank for that.
would it be an idea to add "presets" for most common services ? so you can add a preset that fills out as many as possible.

Nice idea. Feel free to create a new issue for that :)
Glad it works for you!

worked perfectly :) so thank for that.
would it be an idea to add "presets" for most common services ? so you can add a preset that fills out as many as possible.

Hey could you share how you got it to work? How did you set yours up please? I had mine working before and now it does not work. Any input is greatly appreciated. Is there a config file in nextcloud i have to put that domain of the collabora?

On a related note, what would the advanced section be like when using the nextcloud built-in CODE collabora editor?
https://www.collaboraoffice.com/online/connecting-collabora-online-built-in-code-server-with-nginx/
They refer to the following lines in the NC nginx config:
    location ~ ^\/(?:index|remote|public|cron|core\/ajax\/update|status|ocs\/v[12]|updater\/.+|oc[ms]-provider\/.+|.+\/richdocumentscode\/proxy)\.php(?:$|\/) {
        fastcgi_split_path_info ^(.+?\.php)(\/.*|)$;
        set $path_info $fastcgi_path_info;
        try_files $fastcgi_script_name =404;
        include fastcgi_params;
        fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
        fastcgi_param PATH_INFO $path_info;
        fastcgi_param HTTPS on;
        # Avoid sending the security headers twice
        fastcgi_param modHeadersAvailable true;
        # Enable pretty urls
        fastcgi_param front_controller_active true;
        fastcgi_pass php-handler;
        fastcgi_intercept_errors on;
        fastcgi_request_buffering off;
    }

If this makes any difference: I am using NPM to reverse-proxy into a nextcloudpi installation, which itself uses apache (https://ownyourbits.com/2017/06/12/why-nextcloudpi-uses-apache-and-not-nginx/).

So it turns out that the Collabora team has changed some folders and names and the nginx config from above does not seem to work anymore. After puzzling together the info from various forums, I came up with a working config. However, since I don't really understand what I'm doing, somebody more knowledgeable than me should maybe verify this info.
You may also have to clear your browser cache for this to work.
`# static files
location ^~ /browser {
proxy_pass $forward_scheme://$server:$port;
proxy_set_header Host $http_host;
}
WOPI discovery URL
location ^~ /hosting/discovery {
proxy_pass $forward_scheme://$server:$port;
proxy_set_header Host $http_host;
}
Capabilities
location ^~ /hosting/capabilities {
proxy_pass $forward_scheme://$server:$port;
proxy_set_header Host $http_host;
}
main websocket
location ~ ^/cool/(.*)/ws$ {
proxy_pass $forward_scheme://$server:$port;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection "Upgrade";
proxy_set_header Host $http_host;
proxy_read_timeout 36000s;
}
download, presentation and image upload
location ~ ^/(c|l)ool {
proxy_pass $forward_scheme://$server:$port;
proxy_set_header Host $http_host;
}
Admin Console websocket
location ^~ /cool/adminws {
proxy_pass $forward_scheme://$server:$port;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection "Upgrade";
proxy_set_header Host $http_host;
proxy_read_timeout 36000s;
}`

So it turns out the Collabora team has changed some folders and names and the nginx config from above does not seem to work anymore. After puzzling together the info from various forums, I came up with a working config. However, since I don't really understand what I'm doing, somebody more knowledgeable than me should maybe verify this info.
You may also have to clear your browser cache for this to work.
# static files
location ^~ /browser {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
}

# WOPI discovery URL
location ^~ /hosting/discovery {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
}

# Capabilities
location ^~ /hosting/capabilities {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;

 }
# main websocket
location ~ ^/cool/(.*)/ws$ {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Upgrade $http_upgrade;
  proxy_set_header Connection "Upgrade";
  proxy_set_header Host $http_host;
  proxy_read_timeout 36000s;
}

# download, presentation and image upload
location ~ ^/(c|l)ool {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Host $http_host;
}

# Admin Console websocket
location ^~ /cool/adminws {
  proxy_pass $forward_scheme://$server:$port;
  proxy_set_header Upgrade $http_upgrade;
  proxy_set_header Connection "Upgrade";
  proxy_set_header Host $http_host;
  proxy_read_timeout 36000s;
}

Some more info here:
https://help.nextcloud.com/t/updated-collabora-nextcloud-and-now-get-404/129873/2
https://help.nextcloud.com/t/nextcloud-office-document-loading-failed-this-request-has-been-blocked-the-content-must-be-served-over-https/130439/3
https://sdk.collaboraonline.com/docs/installation/Proxy_settings.html#reverse-proxy-with-nginx-webserver

I'm using this as of June 5, 2023, and it doesn't work. Is anyone able to get this configuration with Collabora in Nginx Proxy Manager?

@DCCInterstellar sorry, I moved on to Nextcloud AIO a month or two ago, where I did not have to tinker with this anymore. But until then, I believe my above solution worked.

@mitohund No worries! I've tried it over and over but for some reason, Collabora kept giving me issues. I decided to let it go and install OnlyOfficeDocumentServer. So far that's been working really well Nextcloud.

@DCCInterstellar I just noticed that in the above code block there is an unnecessary empty line and empty space, towards the end of the # Capabilities block, just before }. Don't know if this could cause any issues?

@DCCInterstellar I just noticed that in the above code block there is an unnecessary empty line and empty space, towards the end of the # Capabilities block, just before }. Don't know if this could cause any issues?

same here. I am trying to get this to work since weeks:
https://github.com/CollaboraOnline/online/issues/7063
if anyone has an idea?

i'll pile on - i can't get this thing to work - i been pullin my hair out - its complaining about unautorized:
wsd-00001-00062 2023-08-29 08:20:56.797140 +0000 [ docbroker_003 ] ERR  Unauthorized Request while starting session on https%3A%2F%2Fdkloud.douno.it%3A443%2Findex.php%2Fapps%2Frichdocuments%2Fwopi%2Ffiles%2F9415_ocbn5c6s32h1 for socket #24. Terminating connection. Error: No acceptable WOPI hosts found matching the target host [cloud.fqdn.com] in config| wsd/COOLWSD.cpp:5047
wsd-00001-00062 2023-08-29 08:20:56.812968 +0000 [ docbroker_003 ] ERR  #18: Read failed, have 0 buffered bytes (ECONNRESET: Connection reset by peer)| net/Socket.hpp:1122
wsd-00001-00062 2023-08-29 08:20:56.813091 +0000 [ docbroker_003 ] WRN  #18: Unassociated Kit (55) disconnected unexpectedly| wsd/COOLWSD.cpp:3497


fix to spelling error and add period

[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] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Out of curiosity, how did you even notice this from the site?

Out of curiosity, how did you even notice this from the site?

I noticed the spelling error on the Curriculum menu, then noticed the missing period as I was editing the file. Spelling errors tend to jump out at me.

Advanced Node and Express - Serialization of a User Object
hi my dear can you help me something about programming on I can not go beyond this step:
https://learn.freecodecamp.org/information-security-and-quality-assurance/advanced-node-and-express/serialization-of-a-user-object
please help me i wait for your answer please.

@abdibogor You should ask your question on our forum.  The GithHub site is only for reporting issues with the challenges (i.e bugs) and/or creating PRs to fix them.
Thank you.


Playwright tests

ADR: https://github.com/boxwise/boxtribute/blob/3fc1483a3b3051206e900fbd1cb02520b3158594/docs/adr/adr_frontend_tests.md
Updated FE README:  https://github.com/boxwise/boxtribute/blob/3fc1483a3b3051206e900fbd1cb02520b3158594/front/README.md#testing

@fhenrich33 can you post a link in the description with a fully rendered file, like what @pylipp does with his?

Good ADR @fhenrich33. There are some missing sentences in the context section that should be cleaned up, and I'd like that section on MSW to clarify that you're referring to the current production or development environment (or a specific part of it) rather than just saying "we".
Otherwise, nice work, please request reviews whenever you're ready.


"Don't Recrawl" resolver

Hi there,
We're trying to implement a crawl strategy which essentially never re-crawls a URL, unless under some specific conditions. The basis for this functionality is a IRecrawlableResolver which trivially returns false.
When exploring this approach we found that on subsequent crawls the crawler would put referenced child URLs from the cached HttpCrawlData back on to the queue for processing. This happens at the last bit of beforeFinalizeDocumentProcessing() in HttpCrawler.  We expected the IRecrawlableResolver which always returns false to never put anything on the queue.
Is this expected behaviour of how recrawls should be treated?
Many thanks,
Simon

Hello, they are put in the queue, but are they actually processed/committed, or are they all rejected since "premature"?
Depending on your answer, it could be expected behavior  See, if you crawl a website and you say a page is not ready to be recrawled, the crawler does not download it and thus, cannot extract and follow its links.  It then relies on what is in the cache, to make sure a "premature" page does not prevent its children from being re-crawled (in case those are not premature).
So when done first-pass crawling, all "orphans" (those not encountered due to being premature or else) are put back in the queue for processing to make sure they are not missed.
You could change the <orphanStrategy> to IGNORE, but they may fall off the cache (to be confirmed) and will be reattempted on a subsequent crawl, so the default PROCESS may be best.
Does that answer?

Hi Pascal,
Thanks for your response. To explain out situation briefly: we identify particular pages as "articles", other page we could just call "nodes" which are used for links. Articles are either REJECTed or ACCEPTed based on content and we never want to re-crawl them. However, we do want to re-crawl "node" such as landing and listing pages. We found that articles REJECTed articles are not considered for re-crawling since they are not PROCESSed as 'valid' by the data store, and therefor not restored as having cached crawl data.
I believe we have found a work around by ACCEPTing all articles, and instead tagging them with booleans of corresponding to the content filter criteria for the Commiter to decide whether to commit, and REJECTing nodes to ensure they are always crawled. I guess this somewhat skews ACCEPT / REJECT semantics in our situation, which is fine unless there are more fine grained status we could use?
Many thanks,
Simon

Hello Simon. I am glad you found a way that works for you.
The only alternate option that comes to mind right now is to rely on the collector.is-crawl-new which you can try to filter on using a metadata filter.  Basically, it will be false if coming from the cache (i.e., previously encountered).

OK, thanks Pascal - I'll close the issue since this is all expected behaviour. Thanks for your input nonetheless!
All the best,
Simon


newly opened child channels receive channelActive before the parent channel

creating a child channel in the childChannelInitializer
        func requestStreamInitializer(channel: Channel, streamID: HTTP2StreamID) -> EventLoopFuture<Void> {
            return channel.pipeline.addHandlers([HTTP2ToHTTP1ClientCodec(streamID: streamID, httpProtocol: .https),
                                                 SendAGETRequestHandler(responseReceivedPromise: self.responseReceivedPromise)],
                                                first: false)
        }

        self.multiplexer.createStreamChannel(promise: nil, requestStreamInitializer)

will have SendAGETRequestHandler see channelActive when ctx.channel.parent!.isActive is still false. That's a problem because when sending a request in the child's channelActive the program will crash as the HTTP2Parser's session: NGHTTP2Session! is still nil, ie. we'll crash.

Does this mean that trying to use an HTTP2 client based on this library would always fail right now, or does this only affect some special case? (I'm not in a hurry, just curious.)

Does this mean that trying to use an HTTP2 client based on this library would always fail right now, or does this only affect some special case? (I'm not in a hurry, just curious.)

 You'll need to wait for both the parent & the child channel to receive channelActive. That's why in https://github.com/apple/swift-nio-examples/pull/6 I haveCreateRequestStreamHandler which opens the new stream (aka the child channel) only when the parent channel receives channelActive.

Yup, good catch: that just calls activate way too early.


Append behavior is prepend?

Hey,
I would expect that a.Append(b) will return b appended after a, same as a = append(a, b...). I think that the current implementation is doing the other way around.
But maybe this is only me :smile:

I agree the docs are lacking (there are no docs), I did name the parameter dst, not a or b.
e.g. https://golang.org/pkg/strconv/#AppendBool & friends

I had considered this as well, and thought that a more useful name might be AppendTo.
In the case of all of the strconv.AppendXY cases, there is an additional second argument providing the source. The semantics of dst = src.Append(dst) ends up with the src and dst in an order that is swapped from what we normally would expect.
But using an analogous form to src.WriteTo(dst) from io.WriterTo, the order and behavior of the function is now more apparent in the function name.

We could just move Append to be a package-level func instead of a method.
Thoughts, @puellanivis @posener?

Package-level also works.

I don't have a strong opinion.
I agree that with the current implementation the code without the documentation is not clear.

FWIW it's since been documented from when I originally closed this bug (https://godoc.org/go4.org/mem#RO.Append)

I meant that when you read a code that uses the function and not digging into the doc, it is not clear. Reading a.Append(b) doesn't say if a is appended to b or vice versa.

Changed. Yay v0.


[Jetsnack] - Start using version catalog for dependency management

First of all, thank you guys for such a awesome collection of compose samples! I really like it! 🤩
In this PR I have updated the JetSnack project to start using version catalog for dependency management.
I encountered one issue with regards to using the plugin part in the app gradle file.  From what I understand there is an open issue on Providing a way to use plugin definition from version catalog without version. I managed to get around this by converting the gradle file to Kotlin DSL. Would really like some extra feedback on that if someone has time 😄

name: [Jetsnack] - Start using version catalog for dependency management
about: Updating the JetSnack project to start using version catalog
label: 'triage me'
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Ensure the tests and linter pass
[x] Appropriate docs were updated (if necessary)

Thank you very much for your contribution, it looks really good! Unfortunately I won't be able to merge it because it would disturb how we manage updating all of our samples. In the future, I would encourage you to open an issue before working on something like this to ensure we could support it.

Thank you very much for your contribution, it looks really good! Unfortunately I won't be able to merge it because it would disturb how we manage updating all of our samples. In the future, I would encourage you to open an issue before working on something like this to ensure we could support it.

No worries at all! I didn't think about that! I will open an issue next time😊 I learned a lot for doing this anyways, so it was totally worth it 😊


[incubator/kafka] Add ssl-secrets

What this PR does / why we need it:
In a scenario where you need to deploy multiple Kafka clusters into the same Kubernetes cluster dynamically without touching anything in this chart you will need to take special care of the secrets (e.g. certificates, passwords). Since this chart only receives a name pointing to a secret, we cannot automate dynamic deployments without redefining the secrets ref. This PR add the possibility of dynamically deploying a secrets resource among with your Kafka cluster.
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Signed-off-by: Andre Pontes<EMAIL_ADDRESS>

Hello @andrembpontes ,
Do you have any document to guide how to implement ssl for kafka cluster using this helm chart?
I really appreciate all the work you have done.
Tks in advance,

I have configure all of these variable like this PR and this docs:
https://kafka.apache.org/documentation/#security_ssl
But the docker images from CP still ask the exist of these env variable:
KAFKA_SSL_KEY_CREDENTIALS
KAFKA_SSL_KEYSTORE_CREDENTIALS

2 above env variable does not have description in apache kafka docs.
Has any body set up SSL for kafka cluster using this helm chart successfully? Can you give me some recommends?


Render unknown Roblox datatypes in tables differently

Right now, if I log a table with, say, a CFrame inside it, and make it print to the Roblox output, it renders like so:
Log.Info("Some table: {Value}", { myKey: new CFrame() });

[INFO] Some table: {"myKey":null}

The table output is okay. However, myKey's value is interpreted as null, and this makes reading this output pretty confusing! Could Roblox data types be interpreted like the below?
{"myKey":<CFrame>}

or maybe even with the CFrame's components? (I doubt people would really be interested in this method though.)
{"myKey":CFrame(0, 0, 0, 1, 0, 0, 0, 1, 0, 0, 0, 1)}

The same thing happens with stuff like Instances - they get printed as null.

help me pls


Wrong Card Appears After Clicking the "Learn More" Button at Project Page

Description
When clicking the "Learn More" button on the project page, the wrong project card appears in certain instances. This leads to confusion for users, as the displayed content does not match the selected project.
Steps to Reproduce

Navigate to the project page.
Click on the "Learn More" button for any project.
Observe that the wrong project card is displayed in some cases.

Expected Behavior
No response
Actual Behavior
No response
Screenshots
https://github.com/user-attachments/assets/b6aa8912-b689-46ff-beed-eabf85e104fa
Additional Information
No response
What browser are you seeing the problem on?

[X] Firefox
[X] Chrome
[X] Safari
[X] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@IkkiOcean please complete the issue that i have assigned you before, then i will assign you this.

@IkkiOcean please complete the issue that i have assigned you before, then i will assign you this.

fixing it first will be better as adding more projects to the page will make it more worse. I can fix it rn

@IkkiOcean can you fix it now.

@IkkiOcean can you fix it now.

Hi Vivek, do want those cards as I see no reason keep them there. I can make a button like open project which redirects to the project. it's your wish if you want those cards. I can add more content to them to make them look more appealing and useful

@IkkiOcean yes you can make a button like open project which redirects to the project.

@IkkiOcean yes you can make a button like open project which redirects to the project.

Okay and for projects which aren't implemented there will be learn more button with some more description. Is that fine?

yes.


Inference speed so slow on T4

I tried running Nemo-12b 4-bit model on one T4 GPU, but the inference speed is very slow. Additionally, the 'forward' function takes much longer than 'generate'.
Is there a speedup benchmark for the T4? I'm wondering if I'm doing in the right way.

Are you using FastLanguageModel.for_inference(model) for inference?


chore: refine types and onChange handlers

Description of changes:

Change handlers and types to account for differences across component types.
Verified by creating a form. E2E test would be useful to prevent regressions here. That will follow in another PR

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

Codecov Report

Merging #643 (d712557) into tagged-release/forms-and-views (b780992) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@                        Coverage Diff                         @@
##           tagged-release/forms-and-views     #643      +/-   ##
==================================================================
+ Coverage                           90.26%   90.37%   +0.11%
==================================================================
  Files                                  96       96
  Lines                                3573     3596      +23
  Branches                              960      968       +8
==================================================================
+ Hits                                 3225     3250      +25
+ Misses                                323      321       -2
  Partials                               25       25

Impacted Files
Coverage Δ

packages/codegen-ui/lib/types/form/index.ts
100.00% <ø> (ø)

...es/codegen-ui/lib/utils/form-component-metadata.ts
77.77% <ø> (ø)

...ckages/codegen-ui-react/lib/forms/event-targets.ts
100.00% <100.00%> (ø)

...codegen-ui-react/lib/forms/form-renderer-helper.ts
94.26% <100.00%> (+0.40%)
:arrow_up:

packages/codegen-ui-react/lib/forms/form-state.ts
96.15% <100.00%> (+0.32%)
:arrow_up:

packages/codegen-ui-react/lib/forms/type-helper.ts
100.00% <100.00%> (ø)

.../codegen-ui-react/lib/forms/typescript-type-map.ts
100.00% <100.00%> (ø)

.../lib/amplify-ui-renderers/amplify-form-renderer.ts
37.16% <0.00%> (+1.76%)
:arrow_up:

Continue to review full report at Codecov.

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


Deadlock always occurs while delete job.

org.quartz.JobPersistenceException: Couldn't remove job: Deadlock found when trying to get lock; try restarting transaction
[See nested exception: com.mysql.jdbc.exceptions.jdbc4.MySQLTransactionRollbackException: Deadlock found when trying to get lock; try restarting transaction]
I deploy a cluster with 2 nodes, would it because i did'n set the acquireTriggersWithinLock property?

@zhanjiejun Hello, can you share your solution?

set properties below
org.quartz.jobStore.lockHandler.class=org.quartz.impl.jdbcjobstore.UpdateLockRowSemaphore
org.quartz.scheduler.batchTriggerAcquisitionMaxCount=50
org.quartz.jobStore.acquireTriggersWithinLock=true


[BuildSystem] Add support for directory nodes.

This adapts the directory tree signature so that it can finally be used in a
build file to express dependencies for commands which consider their input
paths recursively.

rdar://problem/22640399 [llbuild] Add support for dependencies on directory contents

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

👏

This PR works a lot better without the version that creates a signature for the entire root file system of the bot. 😳

Do you know how long that take ?😂

macOS bot took it like a champ... Linux bot ran out of memory ;)

👏
should the bot even have permission to go outside its directory 🤔


Create release notes for 0.1.1

Document changes in references/changelog.md before we cut a patch release

@pmeier - Shall we generate these automatically with GitHub: https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes?

Here is automatically generated version for the pre-release: https://github.com/Quansight/ragna/releases/tag/v0.1.1rc0


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1214 ms

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


KEDA Redis Scaler dbIndexFromEnv proposal

Proposal
Providing dbIndex value directly from env as host and port, ex. yaml element dbIndexFromEnv
Use-Case
Now i set host and port in single config file for whole project, but for dbIndex i need to duplicate it in every scaler.
Anything else?
No response

This sounds totally legic!
Are you willing to contribute with it?


Update modifiers to call public view functions

Fixes #1179.
Requires #1268, #1269, #1270.
🚀 Description
It should be possible to query the condition that modifiers require.

[x] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[x] ✅ I've added tests where applicable to test my new functionality.
[x] 📖 I've made sure that my contracts are well-documented.
[x] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:all:fix).

@frangio yes, I've checked them all.
The first thing I noticed is that we don't have as many as I though.
And the second is that this is the most common pattern:
  modifier onlyRole(string _role)
  {
    checkRole(msg.sender, _role);
    _;
  }

I think I would like to remove of all the check functions. Call the require(hasRole) directly on the modifiers. But this is different from what we discussed, and needs more thought.

Rebased on master.

Yup, from what I recall check was only present in Roles, and I removed it for the reasons described above.

Thanks @elopio!


Ubisoft Connect doesn't support non-default install Ubisoft Connect locations for not-installed games.

My Ubisoft connect is installed to %userprofile%\scoop\apps\ubisoftconnect\current and it's never detected (I keep getting the  "Failed to import games from Ubisoft Connect. Ubisoft Connect client was not initialized, please start the client at least once to generate user library data." message.
Installed games (In non-default install location D:\Games\Ubisoft) are detected fine, but uninstalled games are not.
I also use your Epic Store extension and my Epic store is installed to a similar location, but that one works fine.

The path is taken from registry keys that Ubisoft Connect creates during installation. This simply means that scoop doesn't install the client correctly.


Cannot Disable Gaurdian on new Build

As far as I am aware, on the new Quest 3s build, I have not been able to find a way to permenently disable gaurdian. The only workaround I've found is running this adb command, but it doesn't persist between restarts.
adb shell setprop debug.oculus.guardian_pause 1

Ok, after more looking, there is an option under advanced options to disable it, but again, it does not persist

@SeanErn be sure to review the One-Time setup instructions. https://github.com/juchong/QuestNav?tab=readme-ov-file#one-time-headset-setup-required-for-each-headset
These settings should persist.

Disable the guardian for development purposes Settings > Advanced > Experimental Settings > Enable Custom Settings and TURN OFF Physical Space Features, MTP Notification, and Link Auto Connect

These settings might also be located in Settings > Developer > Experimental Settings > Enable Custom Settings on some older OS builds

Thats so weird! On my headset after I set it in the GUI and restarted via adb, the option turned off.

I am going to check the software version of my headset tomorrow. Maybe factory reset it to start with a clean slate. Thanks again!

Can confirm, after resetting my headset the GUI setting persists.


Crash on start

Windows 8.1 64-bit
Broken at: Process Hacker 3.0.3557 Commit: 8818aa1c4bda3abdc6bff1ffae9e5e379c9d7e38
Process Hacker crashed when starting. It does not crash when started as Admin.
0xC0000005 (Invalid access to memory location.)

It starts normally with plugins disabled.
Dump:
ProcessHacker_KEEJBCYXGZNPHJJ_DumpFile.zip

Thanks I've fixed the crash in the latest nightly build 👍
You can delete \plugins\ExtendedTools.dll temporarily and preform the update via the Help menu.


Using TAPAS on domain specific data, training large tables of domain specific data for TAPAS

Dear developers, I have the following questions:

I want to use TAPAS on the MIMIC dataset, the columns of which contains specific terms like 'CPT_CD, CPT_NUMBER, CPT_SUFFIX'. To enable fine-tuning on a dataset like this, how should I proceed?
Additionally, the tables in MIMiC are large. Is there a provision to incorporate this?

Thanks!

Hi @sbhttchryy , thanks for your interest

I suggest you use one of the notebooks have examples on how to convert the data into the required format in tf examples. Alternatively you could follow the original format for one of the datasets we use, like TabFact, WikiSQL, etc and run the scripts from there.

There are multiple possible approaches and it's hard to tell which is the best in advance. One is to split the table into multiple chunks and then join the results using a heuristic. Another is to try heuristic ways of trimming the table content to restrict it to whatever is relevant. We tried something like this for https://aclanthology.org/2020.findings-emnlp.27/ but mostly on a column level, but perhaps in your case the other approaches makes more sense. We will also be realeasing the code for https://aclanthology.org/2021.findings-acl.289/ very soon,.


Can't handle message of type "any"

I wanna write one of my handlers to get specific messages from users. But their type may be any! And going through the docs didn't help me at all, it's misserable!
Even tho I decided to go to see how Bot.Handle works, and if there any wildcard implementation out there, but found only this function:
func (b *Bot) Handle(endpoint interface{}, h HandlerFunc, m ...MiddlewareFunc) {
	if len(b.group.middleware) > 0 {
		m = append(b.group.middleware, m...)
	}

	handler := func(c Context) error {
		return applyMiddleware(h, m...)(c)
	}

	switch end := endpoint.(type) {
	case string:
		b.handlers[end] = handler
	case CallbackEndpoint:
		b.handlers[end.CallbackUnique()] = handler
	default:
		panic("telebot: unsupported endpoint")
	}
}

As I can see, there's no any way to implement it.
Hope I'm wrong, cause it's really the place where there's the problem of any amateur framework with so bad architecture. The problem is that a framework cuts off some low level functional from a developer by making so high-level abstraction

Please elaborate on what are you trying to achieve...


Printing variable in Python causes segfault when accessing it in C++

I'm not able to create a minimum reproducible example (can attach a large project with instructions), but I'm experiencing a problem when wrapping C++ code in which if I initialize a cdefed variable inside a function in a .pyx file, print it there right after giving it some value, and then pass it to a C++ function, as soon as the C++ function tries to do something with this variable, it will segfault.
Example:
.pyx file:
cdef extern from "cppfile.cpp":
    void a_cpp_function(double a, double b)
def a_py_function(double p):
    cdef double problematic_var = 10
    print("will produce segfault with this: ", problematic_var)
    a_cpp_function(problematic_var, p)

.cpp file:
#include <iostreams>
double sum_vals(double a, double b)
{
    std::cout << "next statement is segfault" << std::endl;
    return a + b;
}
void a_cpp_function(double a, double b)
{
    double temp;
    temp = sum_vals(double a, double b);
    std::cout << "result is: " << temp << "\n";
}

(I think this small example will not end up segfaulting, but in a larger function it does)
Removing the python print statement will eliminate the problem. This seems to only affect variables of type double, and these are just lone numbers, not arrays or anything that could cause an out-of-bounds error.

Just realized now I was summing it to another variable that was a field of a struct that got declared but not initialized. Initializing this field in the struct solved the problem.
Still find it very funny that removing the Python print statement would also solve the problem :).


[Event Request] Table 1003, OnBeforeValidateQtyToTransferToInvoice

Requesting a new publisher event in table 1003 inside OnValidate() on Qty. to Transfer to Invoice

LOCAL [IntegrationEvent] OnBeforeValidateQtyToTransferToInvoice(VAR JobPlanningLine : Record "Job Planning Line";VAR xJobPlanningLine : Record "Job Planning Line";VAR IsHandled : Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


wip

Codecov Report
Attention: 286 lines in your changes are missing coverage. Please review.

Comparison is base (feb2fef) 16.49% compared to head (bb0b07f) 16.17%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #56      +/-   ##
==========================================
- Coverage   16.49%   16.17%   -0.33%
==========================================
  Files          63       64       +1
  Lines        8170     8334     +164
==========================================
  Hits         1348     1348
- Misses       6822     6986     +164

Files
Coverage Δ

...and-godot-lib/src/scene_runner/deleted_entities.rs
0.00% <0.00%> (ø)

...t/decentraland-godot-lib/src/scene_runner/scene.rs
25.00% <4.16%> (-16.31%)
:arrow_down:

...raland-godot-lib/src/scene_runner/scene_manager.rs
0.33% <0.00%> (-0.02%)
:arrow_down:

...traland-godot-lib/src/scene_runner/update_scene.rs
0.00% <0.00%> (ø)

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


WIP: new structure, add linters

Fixes #89 .
Changes proposed in this pull request:

new structure of files in flutter directory
linters for flutter and common

TODO: excluded in common doesn't work (exclude generated files from lint)


Type for datazoom event

Version
5.4.3
Link to Minimal Reproduction
No response
Steps to Reproduce
Create datazoom event
Try to type
Current Behavior
For example, we have the following code:
`myChart.on("dataZoom", event  => {
if (event.batch) {
return dataZoom.value = { start: event.batch[0].start, end: event.batch[0].end };
}
			dataZoom.value = { start: event.start, end: event.end };
		});`

For the event TypeScript gives an error:
 Argument of type (this: EChartsType, e: unknown) => {     start: any;     end: any; } | undefined is not assignable to parameter of type WithThisType<(...args: unknown[]) => boolean | void, EChartsType>
How to type an event?
Expected Behavior

Environment
- OS: macOS Sonoma
- Browser: Chrome 120.0.6099.129
- Framework: Vue@3

Any additional comments?
No response

this code has no errors:
myChart.on('dataZoom', (event: any) => {
  var dz = { start: event.start, end: event.end };
  if (event.batch)
    dz = { start: event.batch[0].start, end: event.batch[0].end };
  console.log(dz);
});

📌 please close issue if problem solved.

Same problem, event has no type defined.


Replace "rejected" with "declined"

Describe the task
A clear and concise description of what the task is.
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

Related to #421

fairly sure this was part of the text changes PRs


Some questions about hyperparameters

Hi! First of all I love the work you've with the TransFuser project, it's really quite interesting and I've had fun diving into autonomous driving research with CARLA thanks to you and your team, so thank you for that!
Currently I am trying to replicate your results and also tune the hyperparameters of the transfuser model so I have a few questions with respect to that.

I see that in all of the train.py scripts for all of the models, the --epochs parameter is set to a default value of 101 except for CILRS which is set to 201, however in the supplementary material you mention that all of the models were trained for 101 epochs. Was the CILRS model really trained at 201 epochs or why is it the only model set to that default value?

In the README for each model you give an example of how to train that model, for example,
CUDA_VISIBLE_DEVICES=<gpu_id> python3 train.py --id transfuser --batch_size 56 and
CUDA_VISIBLE_DEVICES=<gpu_id> python3 train.py --id late_fusion --batch_size 128
I see that for different models you provide different values for --batch_size in those examples, however the default values are all set to 24 if I'm not mistaken. What batch size did you use for producing the results found in your main paper? Did you use the same values for all of the models or, like in the examples provided, use different values for different models?

During the training process, the model validates itself every 5 epochs, is this so that it can autostop if there's no improvement after a certain number of epochs or what is the reason for this validation? If this is the case, then there wouldn't be much of a need to tune the epochs parameters, correct?

From what I can tell, the main tunable parameters when calling the train.py script are --epochs, --lr (learning rate), and --batch_size. Do you have any suggestions/recommendations as to how I can go about trying to tune these? Also, are there any other parameters that I didn't mention that could also be fun to experiment with to try to optimize the transfuser model?

After how many epochs does the training terminate if the validation loss has not improved? And in what part of the code do you define this termination value? I can't seem to find it.

In the config.py you mention, there is a learning rate parameter set to 1e-4. Is this parameter equivalent to the one that we define when running, for example, CUDA_VISIBLE_DEVICES=<gpu_id> python3 train.py --lr 0.0001 or is it for something else? Also, why does the CILRS model not have a learning rate parameter as a training option, but it does have a learning rate parameter in its config.py file?

I currently have access to a cluster with a few 40GB GPUs so I think I will try to implement a grid search over some of these transformer architecture parameters to take advantage of these resources. This leads me to 2 more questions.

I understand that wandb is a good tool that can be used for performing a grid search, tho I have never used it nor have I performed a grid search before. Do you have any insight as to how I could go about using this tool and what files I would have to modify/create to perform a grid search of these transformer parameters on a cluster?

I know that the transfuser model uses a GPT encoder in its architecture and that those parameters are found in config.py, however I am still fairly new to the topic of attention and I'm not quite sure how all of those parameters affect the overall architecture. Do you happen to have any articles you can recommend for me to read and get a better understanding of how I could tune these transformer parameters?

Again, thank you so much for your insight, I've gained a lot from reading all of the issues posted here and I know they'll also be of use to others!

This is currently not implemented in the code. I had tensorboard open in the browser and I would just monitor the validation loss manually. But I agree that it should be implemented in the code itself. In our case, validation was done after every 5 epochs and I terminated the training if the validation loss did not improve for 25 epochs.

Both the learning rate parameters are the same, you can define them either as a command-line argument or in the config.py file.

I'm not that familiar with wandb so I don't know how to run grid search using wandb.

For tuning the transformer architecture, you can start with n_embd, n_layer, n_head parameters. I am not aware of any article which specifically focuses on transformer tuning but I'm certain that you should be able to find some resources online.

Thank you for you quick replies. you've been a great help, I will try out some configurations this week and I'll let you know if I find anything interesting!

Hello again, so I've tried training a few models by varying the n_layer and n_head parameter values in config.py with {4,8,16} and from what I can tell they all trained successfully, the only notable difference being that the training time increased as n_layer increased.
However, when I tried evaluating these models, only the ones that were trained on the default value of n_layer=8 are evaluating correctly. When I evaluate the models with a different n_layer value it takes about a minute to complete and I get the following error for each scenario that it tries to run. Any idea as to what could be the issue here?

One more question, the n_layer parameter refers to the number of encoder layers within the transformer architecture, correct?

Update: I changed a line in leaderboard/team_code/transfuser_agent.py by adding the strict=False argument to the function load_state_dict(path_file, strict=False). It took longer than usual for the evaluation to actually start but as far as I can tell it seems to have fixed the issue without any noticeable error as it appears to be evaluating correctly now.
When testing different values of n_layer and n_head I had trouble implementing a tool to automatically do a grid search so I did it by hand by making various copies of the transfuser model directory and directly modifying the transformer parameters in each respective config.py file. My best guess is that when evaluating the modified transfuser models the original config.py file with the original parameters gets called at some point leading to the error shown above. My only concern is that I believe this argument only hides the underlying problem rather than actually fixing it, but as long as it continues evaluating correctly I guess it shouldn't be a problem.

The error means that you are loading the wrong checkpoint for the model, just verify that the model and checkpoint are the same.
n_layer corresponds to the number of self-attention layers in the transformer.
If you set strict=False when loading the checkpoint, then checkpoint loading related errors will not be shown. You should use strict=True and make sure that the model and checkpoint are same (verify the parameters in the config file).

Sorry for the late reply, it took a while to do a decent amount of testing. You were in fact right, by setting strict=False it appears to have loaded the wrong checkpoint resulting in some incorrect and faulty evaluations. I believe this issue was caused because I tried to simultaneously evaluate various transfuser models trained with different n_layer and n_head parameters. The quick solution I found was to make copies of the directory and modify each config file to match the parameters with which it was trained, that way I could run various evaluations simultaneously.
The only thing that still puzzles me is why the n_layer and n_head parameters in the config file need to match the values with which each model was trained in order to evaluate that model correctly since I thought those parameters only affected the training process.

These parameters are related to the model architecture. So whenever you are loading a checkpoint (which contains the stored weights of the model), you need to match the configuration.

That makes sense, thank you. So after varying the n_layer and n_head parameters I've noticed that all resulting transfuser models have a similar trend where there is a large gap between the training loss and validation loss (like the ones seen in the image below). Do you think it's because I trained the models on the 14_weathers dataset while all the evaluations take place on a clear noon weather, or is there another reason? Is there a way to reduce this gap?

Validation loss is generally always higher than training loss since the model will eventually start overfitting to training data and get low training loss. The trend is ok, the main thing you need to check is the performance of the model on the evaluation routes.

Ok, thank you. I have one final question before closing this issue.
From what I understand, the PID controller does not affect the training process, only the evaluation of the model. Therefore, I would only need to tune the PID controller hyper-parameters in the config.py file for running an evaluation, not for training a model, correct?

that's right

Thank you, I highly appreciate all the help you've given me. I will now close this issue, keep up the good research!


Obtain MessageDigest via FastThreadLocal

Motivation:
Creating a new MessageDigest every time is wasteful, we should store them in FastThreadLocal.
Modifications:
Change WebSocketUtil to store MD5 and SHA1 MessageDigest in FastThreadLocal and use these.
Result:
Less overhead and less GC.

@Scottmitch @trustin PTAL

How about just writing a utility class dedicated to producing md5/sha1 hash? Given that what you really want is a single-array-in-single-array-out digest, the utility class can be completely stateless and thread-safe without any lock. i.e. no reason to use thread local.

Sure... Let me give this a try once I have a bit time.

Am 19.12.2015 um 02:03 schrieb Trustin Lee<EMAIL_ADDRESS>How about just writing a utility class dedicated to producing md5/sha1 hash? Given that what you really want is a single-array-in-single-array-out digest, the utility class can be completely stateless and thread-safe without any lock. i.e. no reason to use thread local.
—
Reply to this email directly or view it on GitHub.

@trustin can we do this as a follow up pr as this will take some more time and this PR already gives us an improvement. WDYT ?


Validate inputs to charge

Fake stripe should validate the inputs to charge similarly to Stripe and should raise the same errors if appropriate.
For instance, raise if there is no token. Raise if amount is not a positive integer. Raise if there's no currency, etc.

Closing this because fake_stripe has been deprecated: https://github.com/thoughtbot/fake_stripe/pull/154


Fix to locate carla python package in performance_benchmark.py

Description

Fixes #1382
Where has this been tested?

Platform(s): Linux 64-bit
Python version(s): 2.7, 3.4
Unreal Engine version(s): 4.21

Possible Drawbacks
None AFAIK

This change is

Thanks!


[Bug]: Patternfly CSS is being overridden

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Rigth now several style's components are being overridden from the original Patternfly version.
We should not have this behavior in the dashboard.
Expected Behavior
All the components in the dashboard should use the default style of Patternfly.

This is a pretty bad infrastructure problem. Overriding css from PF is a VERY bad idea.
cc @DaoDaoNoCode when we get a moment, I'd like you to work to undo this damage to our CSS classes so we can stay in sync with PF updates without constant problems.

To clarify the intent behind this ticket, let us think on what is happening, how to fix it, and more importantly, how to prevent it going forward.

.pf-... class references exist heavily in a lot of our CSS -- these are usually nested under a custom class name, so it is usually limited to a given usage. This serves to modify the PF library which can hurt us when upstream changes implementations on us and our styles are no longer compatible... also it can prevent us from seamlessly adopting feature fixes and new implementations -- this will be annoying to track down.
The goal should be to ONLY use PF components and very very rarely use custom styles to achieve a custom implementation around PF (never within' PF classes unless we are hotfixing a bug) -- but more ideally, this should be conversations with UX to change our design to meet PF components and just avoid custom styles
To prevent this we probably should look at adding a css linter rule to prevent any usage of .pf- -- how to do this will require some googling, but I suspect this isn't the first needed implementation of this usecase and something in npm will already be set to do this for us

This is an ongoing effort with each PF Upgrade that @DaoDaoNoCode goes through.
From a quick look, the current offenders are:

App.scss
OdhCard.scss
OdhListItem.scss
EnableModal.scss
LearningCenter.scss
LearningCenterToolbar.scss

There are two use-cases that are waiting on https://github.com/patternfly/patternfly-react/issues/8028 -- needs a followup. @DaoDaoNoCode can you review their response and see if you have an action that is their React components or if we got it for free? If not, I'll need to carve out time to follow up with it.
There is also one more use-case -- NotebookController.scss -- this is deprecated. Let us not worry about this. This all should go away when we do the Notebook Namespace decommission (Milestone)

Moved to story in Jira:  https://issues.redhat.com/browse/RHOAIENG-2408


Examples and swagger for APIs related to managed server operation

All examples and swaggers for new get, list and cancel APIs for management operations

/azp run automation - sdk

Be free to merge changes since API changes are checked in.

/azp run automation - sdk

@allenjzhang what does this tag WaiitngForArmFeedback means? Who should I ping to take a look at this?

@allenjzhang what does this tag WaiitngForArmFeedback means? Who should I ping to take a look at this?

Hi @toki95, if your PR fall in below 4 category, it will need ARM by adding the lable:
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.
Normally it will be addressed in your first comments of your PR. but not sure why your PR not include it.  you can refer to first comment in other PR e.g. https://github.com/Azure/azure-rest-api-specs/pull/8433
For track why your PR no this, Can you tell us how did you create this PR?

@allenjzhang what does this tag WaiitngForArmFeedback means? Who should I ping to take a look at this?

Hi @toki95, if your PR fall in below 4 category, it will need ARM by adding the lable:
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.
Normally it will be addressed in your first comments of your PR. but not sure why your PR not include it.  you can refer to first comment in other PR e.g. https://github.com/Azure/azure-rest-api-specs/pull/8433
For track why your PR no this, Can you tell us how did you create this PR?

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk


Modified explanation of initial domain

Users can misunderstand the initial domain '.onmicrosoft.com' can be customized, edited, and deleted.

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

Note that publishing in this repository is suspended until Tuesday, May 25. Merging will resume Tuesday. Authors can still review and sign off on PRs in the interim. Thanks.

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


[Proposal] Remove blog section and prefer Medium

Remove blog and keep footer link that redirect to Medium.

I will concur to that. Would be nice to tack edits and use git, being a DEV oriented blog, rather than a marketing one

Should I point to https://vulpem.medium.com/ in the meantime?


New file addition.

I would like to write the LEARN.md file for this project. Please assign this task to me under GSSoC'24 with appropriate labels. @VanshKing30

Okay go ahead!


chore(main): release graalvm 0.0.3

:robot: I have created a release beep boop
0.0.3 (2023-03-20)
Features

graalvm: add graalvm image (f7b5cce)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/plantuml/docker/releases/tag/graalvm-v0.0.3 :sunflower:


Added armstrong number and updated readme.md

Have you read the Contributing Guidelines on Pull Requests?
yes
(Write your answer here.)
Description
Added armstrong number and updated readme.md
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixed #4725
(Write your answer here.)

@HarshCasper please review this pr. Thank you!!


[SPARK-5239][CORE]JdbcRDD throws "java.lang.AbstractMethodError: oracle.jdbc.driver.xxxxxx.isClosed()Z"

rs.isClosed() will throws:
Exception in thread "main" java.lang.AbstractMethodError: oracle.jdbc.driver.OracleResultSetImpl.isClosed()Z

Similarly,stmt.isClosed() will throws
Exception in thread "main" java.lang.AbstractMethodError: oracle.jdbc.driver.T4CPreparedStatement.isClosed()Z

Above exception is throwing because "isClosed()" is not implemented in the "OracleResultSetImpl" class which implements the ResultSet Interface in Ojdbc14.jar, I think use below condition to check whether resultset is null or not is enough
if (null != rs) {
	rs.close()
}

Can one of the admins verify this patch?

See comment in JIRA; this should not be hacked around because the JDBC driver is unsuitable.

I take it back (see https://issues.apache.org/jira/browse/SPARK-5481) Yes, there's evidence of JDBC 3 drivers in the wild, still, and because I confirmed that close()ing these three objects is required to be a no-op if already closed, so, this really can't hurt.
Similar changes should be made in JDBCRDD.scala in Spark SQL. Could you update the PR to address that too?

@luogankun do you want to augment this PR to include JDBCRDD.scala too? I can do so if you're not available but wanted to give you the opportunity.

ok to test

Test build #27040 has started for   PR 4033 at commit d8c61d6.

This patch merges cleanly.

Test build #27040 has finished for   PR 4033 at commit d8c61d6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27040/
Test PASSed.

Close the PR.

@luogankun are you going to make a new PR? or are you asking me to do so? you can just reopen and add to this PR.

@srowen To do other things, forgot to resubmit a new PR after close the PR. Thanks for your help.

NP I will assign to you @luogankun in any event for the credit. Right now the new PR fails Hive tests, twice, which is suspicious, though I can't see how it would affect behavior. I'm waiting to see if it's just flakiness or a real problem.

The stacktrace message of all failure testcase are: sbt.ForkMain$ForkError: Futures timed out after [1 minute], whether related Jenkins?


Display embargo dates in Restrictions Note

User story
When an object has an embargo date it needs to be displayed in the restrictions note display
Acceptance criteria

[ ] I see the value of the embargo_date_display field in the restrictions_note_display field.
[ ] I can successfully use this feature with Voice Over in Safari.
[ ] I have checked this feature using one of the following browser extensions and confirmed that it does not add any level A or level AA issues:

axe dev tools
Lighthouse
Wave

Concrete example
https://catalog-staging.princeton.edu/catalog/99129112208406421 is an example record. "embargo_date_display" is populated however "restriction_note_display" is missing this data point. Currently the value is stored as:
This dissertation is under embargo until A digital copy is available for viewing in the Mudd Manuscript Library reading room during the embargo period. If you are interested in this service, please fill out the Ask Us form using the following link.
This should read
This dissertation is under embargo until 05/31/2024. A digital copy is available for viewing in the Mudd Manuscript Library reading room during the embargo period. If you are interested in this service, please fill out the Ask Us form using the following link.
Implementation notes, if any
The contents of embargo_date_display need to be injected into restriction_note_display. See https://catalog-staging.princeton.edu/catalog/99129112208406421/raw for example data.

@rladdusaw @jrgriffiniii I'm a bit confused on this one. Could we add an example of a record where this "works" in the ticket created last week https://github.com/pulibrary/orangetheses/issues/77. We had worked out the parsing of the embargo dates in this class earlier in the year? https://github.com/pulibrary/bibdata/blob/main/marc_to_solr/lib/embargo_date_extractor.rb. The date extractor should produce a human readable version of the date for display in the note.

Working example: https://bibdata-staging.princeton.edu/bibliographic/99127127233306421
Example with error: https://bibdata-staging.princeton.edu/bibliographic/99129112208406421

@mzelesky the latter record is formed correctly per the discussion we had about this embargo dates, is my memory correct there? The data looks like:
<subfield code="a">This dissertation is under embargo until</subfield>
<subfield code="g">20240531.</subfield>
<subfield code="c">
A digital copy is available for viewing in the Mudd Manuscript Library reading room during the embargo period. If you are interested in this service, please fill out the Ask Us form using the following link.
</subfield>

Our code was supposed to turn the subfield g into to something was human readable when we indexed this data.

I discussed this with @kelynch and we are going to expect that the human formatted date will be part of subfield A as in this example: https://bibdata-staging.princeton.edu/bibliographic/99127127233306421.
See https://github.com/pulibrary/etd_transformer/issues/175 for the change that will introduce this into the dissertation data that Alma will import.

Index these four mms IDs in Alma Sandbox
99129114193106421
99129114193006421
99129114192906421
99129114192806421

The records are indexed and can be viewed in catalog staging.


Unexpected "error: tape memory out of bounds (overrun)"

I've run into some issue when playing with bf using this interpreter. Below is the simplest test case I was able to make, that show the problem:
>
><<

$ echo ">\n><<" | brainfuck
error: tape memory out of bounds (overrun)
exceeded the tape size of 30000 cells
$ echo $?
1

If we add single space between > and < then problem disappears:
>
> <<

$ echo ">\n> <<" | brainfuck
$ echo $?
0

Thank you for reporting the issue. I have confirmed that this is indeed an issue.
167b2de should resolve the issue.

Thanks for resolving the issue so quickly. I will check it in a free time.

I confirm that problem is resolved. Are you going to make a new release soon?

@radarek Thanks for reminding me! I have just published a new release: https://github.com/fabianishere/brainfuck/releases/tag/2.7.2
Let's see how quick downstream can catch up :)


xkb syntax "section" appears twice AND "section" as §'s name is colored

two minor issues related to each other:
on /runtime/syntax/xkb.vim file, line 44 (defining keyword xkbIdentifier),
the word section appears twice, is there a particular reason for that?
section is also the xkb name of this symbol: § (U0047),
so if I have a line like key <AE04>{[ quotedbl, section, U230b, U21d2, F9, U1F4AC]}; in a xkb file,
I will see the section word in Keyword color, but it shouldn't.
A possible solution is to change line 34 to:
syn region xkbBracket start='\[' end='\]' contains=ALLBUT,xkbBracketError,xkbSpecial,xkbTodo,xkbIdentifier transparent
(I simply added xkbIdentifier in contains=ALLBUT).
(Note: I reported the same issue on NeoVim: https://github.com/neovim/neovim/issues/12990 )

Have you tried contacting the maintainer?

https://github.com/vim/vim/blob/80361a5f2b134c88597d60b3d363b52084e712a1/runtime/syntax/xkb.vim#L1-L6
FWIW: The URL is not valid anymore. @ewen-goisot can you contact the maintainer via mail please?

@chrisbra
yes, I contacted the maintainer recently


[Feature Request] Allow new keymaps to be ignored by whichkey

Small feature request to allow forcing all keymap descriptions to which_key_ignore.
This is for hiding all langmapper registered keys from whichkey.
I've tested using hack_keymap = false and just modifying the auto.lua to just set desc to which_key_ignore and seems to work. Will be great to have it as an option.
[Side question] What is the difference between:
hack_keymap = true with automapping({ buffer = false })
vs
hack_keymap = false with automapping({ buffer = true })

I'll consider adding an option to add custom desc for keymaps. While you can filter mappings for which-key by 'LM (' prefix.

What is the difference between

These settings are used for different plugin use cases, more details in the readme: https://github.com/Wansmer/langmapper.nvim?tab=readme-ov-file#usage

With #38  you can use the custom_desc method to set your own description for each translated mapping.

Works perfectly! Thanks.


Got to ask about game**.tgz

Hi,
Would it be much work to make an option like -g to perhaps drop pulling the game**.tgz package?  Don't see the use pulling this when all I want is httpd :-)
Thanks for the script though!  Appreciated :-D

Hi, you can just easily remove game entry from sets near:
set -A sets 'comp' 'game' 'man' 'base'
in 16th line of the script.
Regards
Kamil Monticolo
11 paź 2015 01:34 "Birger J. Nordølum"<EMAIL_ADDRESS>napisał(a):

Hi,
Would it be much work to make an option like -g to perhaps drop pulling
the game**.tgz package? Don't see the use pulling this when all I want is
httpd :-)
Thanks for the script though! Appreciated :-D
—
Reply to this email directly or view it on GitHub
https://github.com/qbit/snap/issues/30.

A better option might be to allow custom lists of set files via cli or in ~/.snaprc

I'll try that next time, thank! :-)


Added notebook with several examples using the same model

Issue #66 , if available: Notebook with examples for the landing page.
Description of changes: This is a notebook to drive the standardization of the use (API) of different loops. The examples here can be used in the blogs for the methods of the  landing page.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I tried to run the notebook. I think you are missing some imports e.g., Sequential


Got an error trying to use code-group

Describe the bug
I would like to use code-group feature on debian Bookworm with node v18.19.0.
Minimal reproduction
Steps to reproduce the behavior:

git clone slidev
virtualenv env
source env/bin/activate
npm install -g @slidev/cli
cd docs/features/
npm exec slidev --- --port 5050 "frontmatter-merging.md"

It fails with:
8:33:16 AM [vite] Pre-transform error: Invalid end tag.
8:33:16 AM [vite] Pre-transform error: Invalid end tag. (x2)
8:33:17 AM [vite] Internal server error: Invalid end tag.
  Plugin: vite:vue
  File: /home/LNCMI-G/christophe.trophime/github/slidev/docs/features/frontmatter-merging.md__slidev_1.md:22:1
  20 |
  21 |  Cover Page

  22 |  </CodeBlockWrapper>
     |   ^
  23 |
:::

  24 |
They will end up being equivalent to the following page:

      at createCompilerError (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-core/dist/compiler-core.cjs.js:1363:17)
      at emitError (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-core/dist/compiler-core.cjs.js:2944:5)
      at Object.onclosetag (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-core/dist/compiler-core.cjs.js:2316:9)
      at Tokenizer.stateInClosingTagName (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-core/dist/compiler-core.cjs.js:784:16)
      at Tokenizer.parse (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-core/dist/compiler-core.cjs.js:1142:16)
      at Object.baseParse (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-core/dist/compiler-core.cjs.js:2983:13)
      at Object.parse (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-dom/dist/compiler-dom.cjs.js:898:23)
      at Object.parse$2 [as parse] (/home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vue/compiler-sfc/dist/compiler-sfc.cjs.js:1790:24)
      at createDescriptor (file:///home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vitejs/plugin-vue/dist/index.mjs:64:43)
      at transformMain (file:///home/LNCMI-G/christophe.trophime/github/slidev/slidev-env/lib/node_modules/@slidev/cli/node_modules/@vitejs/plugin-vue/dist/index.mjs:2396:34)

Environment

Slidev version: v0.49.29
Browser: Chromium 128.0.6613.137 built on Debian GNU/Linux 12 (bookworm)
OS: DB12

Sorry, Slidev doesn't support code-group out of the box. You can install a corresponding MarkdownIt plugin manually (https://sli.dev/custom/config-vite)

By the way, are there others slidev features that require a custom setting?

Sorry, I didn't explain very clearly. frontmatter-merging.md is part of the docs, and the code-group syntax in it is processed by VitePress - which is not a Slidev feature. All features described in https://sli.dev/features/ works out of the box unless the doc says some options should be set.

Use https://github.com/hunghg255/markdown-it-code-group:
vite.config.ts:
import { defineConfig } from 'vite'
import CodeGroup from 'markdown-it-code-group'

export default defineConfig({
  slidev: {
    markdown: {
      /* markdown-it options */
      markdownItSetup(md) {
        /* custom markdown-it plugins */
        md.use(CodeGroup)
      },
    },
    /* options for other plugins */
  },
})

styles/index.ts:
import 'markdown-it-code-group/styles/code-group-colors-light.css'
import 'markdown-it-code-group/styles/code-group-colors-dark-media.css'
import 'markdown-it-code-group/styles/code-group-base.css'

Just followed your suggestion:
npm install --save markdown-it-code-group
Then add vite.config.ts and styles/index.ts
but I still have the same error when trying with frontmatter-merging.md

This problem is kind of tricky😥

Ok but how do you get this working in the slidev website?
Which configuration do you actually use for the ci:publish?

Ok but how do you get this working in the slidev website? Which configuration do you actually use for the ci:publish?

The website is built with VitePress, where this feature can be used directly. Slidev does some transformations to the Markdown before passing it to MarkdownIt, and that's why markdown-it-code-group is currently not working.

I'm quite confused.

The syntax from markdown-it-code-group.md comes from VitePress. That's why it's not working with markdown-it-code-group ?

with your proposed setup for allowing use of markdown-it-code-group, running slidev against the test case from markdown-it-code-group should work out the box? Which is not the case...

12:40:19 PM [vite] Pre-transform error: Cannot set properties of undefined (setting 'type')
12:40:19 PM [vite] Pre-transform error: Cannot set properties of undefined (setting 'type') (x2)
12:40:19 PM [vite] Internal server error: Cannot set properties of undefined (setting 'type')
Plugin: unplugin-vue-markdown
File: /workspaces/course-prepost-slides/1.basic.md__slidev_1.md
at Array. (file:///workspaces/course-prepost-slides/node_modules/markdown-it-code-group/dist/index.mjs:87:20)
at Core.process (file:///workspaces/course-prepost-slides/node_modules/markdown-it/lib/parser_core.mjs:56:13)
at MarkdownIt.parse (file:///workspaces/course-prepost-slides/node_modules/markdown-it/lib/index.mjs:511:13)
at MarkdownItVDrag.md.parse (file:///workspaces/course-prepost-slides/node_modules/@slidev/cli/dist/chunk-YP37OZJY.js:2138:19)
at MarkdownIt.render (file:///workspaces/course-prepost-slides/node_modules/markdown-it/lib/index.mjs:530:36)
at file:///workspaces/course-prepost-slides/node_modules/unplugin-vue-markdown/dist/chunk-XC4FHL6S.js:156:25
at TransformPluginContext.transform (file:///workspaces/course-prepost-slides/node_modules/unplugin-vue-markdown/dist/chunk-XC4FHL6S.js:327:37)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async PluginContainer.transform (file:///workspaces/course-prepost-slides/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:49033:18)
at async loadAndTransform (file:///workspaces/course-prepost-slides/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:51856:27)

If I use pnpm instead of npm to install slidev, I would endup with VitePress installed. I shouldn't have problem with
code-group feature? But how to run slidev then: pnpm exec slidev -- --port 3030 "1.basic.md" ?

VitePress and Slidev are diffrent softwares. Installing VitePress as a dependency doesn't affect how Slidev runs. Code group is a built-in feature in VitePress, but this feature is currently incompatible with Slidev.

Sorry to be "rude" but then why is "code-group" feature in the doc?
Hopefully, it will be in slidev, once we can properly add markdown-it-code-group plugin right?

The "code-group" feature is in the doc because it visually looks better than just putting two code blocks in a column 😇. Other project's docs like Vue also uses this VitePress's feature: https://vuejs.org/guide/quick-start.html#creating-a-vue-application.

Hopefully, it will be in slidev, once we can properly add markdown-it-code-group plugin right?

Yes. By the way, I personally don't think it is a good idea to put code groups into slides 😇.

I wonder if it could work using:
import pluginMdc from 'markdown-it-mdc'

const md = new MarkdownIt()
  .use(pluginMdc)

with a syntax like: https://github.com/nuxt/content/blob/b429b1810ba90dad557aa6392c18c0c9b2cc51a3/docs/content/2.usage/2.markdown.md?plain=1#L127C1-L149C3

I think the “code-group” is awesome feature compared to PPT.
Hope it can be Integrated into slidev.


create basic editor page, load data from config given UUID

Closes #226 and #228
Changes in this PR

adds a basic editor page to the app
if a UUID is provided via the URL or UUID field on the page, the configuration file will attempt to load based on the language provided in the URL
modifies the existing language switch code to support this new editor page
adds unofficial French translations for the editor (from Google Translate)

Testing this PR
To access this new page locally, you can visit one of the following URLs:
With a UUID provided: http://localhost:8080/index-ca-en.html#/en/editor/00000000-0000-0000-0000-000000000000
With no UUID provided: http://localhost:8080/index-ca-en.html#/en/editor/

This change is

Should this be a completely separate app? It doesn't seem like something we want to deploy alongside the regular site.


Not working in newer Electron versions

Apologies for the quick and dirty report, I spend way too long trying to figure out why after updating my frame did not work anymore. But hopefully this might save someone else some time
Long story short (here is the URL that enlightened me: https://github.com/electron/electron-quick-start/issues/463)
With Electron 12 and above 'contextIsolation' is set to true per default. So, when running the app the error 'uncaught reference require is not defined' (visible in the Developer Tools) occurs. As a result, when running the app, it is stuck on the splash screen and unresponsive to input.
Fix:
Add 'contextIsolation: false' in main.js in line 42, i.e.
  win = new BrowserWindow({
    width: 1024,
    height: 600,
    webPreferences: {
      nodeIntegration: true,
      enableRemoteModule: true,
      contextIsolation: false
    },
  });

Disclaimer: I am not an expert in nodejs / electron whatsoever, and I don't know whether there is any security implications from doing this (my assumption is not since it all runs local).

Just found #188 (same problem and same conclusion as here), hence closing this issue.


non-substantial speedups of sparse OPS

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System:
Compiler:
Package used (Python/R/Scala/Julia): Python
Or if installed from source:
MXNet commit hash (git rev-parse HEAD): -b sparse
profile_output.json.zip
I am using the sparse ops to train a 2-layer fully connected NN. When I compare with my original dense implementation, the speedup is not significant. (on;y about 33%). When I run profiler, it seems most of the time is taken in the dense, fully connected layer backward computations (json attached).
The custom OPS that were built for the specific loss function being used are reasonably fast.
The code to build the network is below:
depth = 2
widths = 2048
nonlins = "relu"
def make_network(depth,widths,nonlins,out_dim,dropouts,beta,rho,dim_in,label='softmax_label'):
    data = mx.sym.Variable('data', stype='csr')
    labs = mx.sym.Variable(label)
    norm = mx.initializer.Normal(sigma=0.01)
    v = mx.symbol.Variable("v", shape=(dim_in, widths[0]), init=norm,stype='row_sparse')
    net = mx.sym.dot(data,v)
    net = mx.sym.Activation(data=net,name='ac1',act_type=nonlins[0])
    net = mx.sym.IdentityAttachKLSparseReg(data=net,sparseness_target=rho,penalty=beta,name='kl1')
    net = mx.sym.Dropout(data=net,name='dl1',p=dropouts[0])
	# remaining hidden layers
    for layer in xrange(2,depth+1):
    	net = mx.sym.FullyConnected(data=net,name='fc'+str(layer),num_hidden=widths[layer-1])
        net = mx.sym.Activation(data=net,name='ac'+str(layer),act_type=nonlins[layer-1])
        net = mx.sym.IdentityAttachKLSparseReg(data=net,sparseness_target=rho,penalty=beta,name='kl'+str(layer))
        net = mx.sym.Dropout(data=net,name='dl'+str(layer),p=dropouts[layer-1])
    # output layer
    net = mx.sym.FullyConnected(data=net,name='fcout',num_hidden=out_dim)
    net = mx.sym.Activation(data=net,name='acout',act_type='sigmoid')
    net = mx.sym.Custom(data=net, label=labs, name='ce',op_type='CrossEntropyLoss')
    return net

well, we cannot do anything about the FC layers since the first layer produces a dense matrix. What's the expectation?


Graph view only shows commits on remote default branch, not local

Testing #229359
Maybe this is by design? Following the first half of the steps in the TPI Cherry Pick section
My local default branch (branch2) has these 5 commits on them. Only the first 3 are pushed to origin/branch2

When I switch to the topic branch (branch3) it shows origin/branch2, but not the local branch2

I am assuming that you have created branch3 from branch2. If that is the case then the behaviour that you are seeing is expected. At the moment the logic that computes the base branch queries for remote branches only. The thinking is that eventually the remote branch is the one that you will use as the base branch when you create a pull request.


Semantic highlight intradoclinks in documentation

Our intradoclink resolution certainly needs some improvement, most links I tried making didn't seem to work, here noop is a macro in the same file so it should be highlight but it isn't.

bors r+


Fix typo - spelling

Ouch!

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


Lower linux cmake minimum version

The current stable Flutter SDK bundles CMake 3.10.2 and it doesn't look like this library is using any new CMake features.
Launching lib/main.dart on Linux in debug mode...
Building Linux application...
CMake Error at flutter/ephemeral/.plugin_symlinks/sqlite3_flutter_libs/linux/CMakeLists.txt:1 (cmake_minimum_required):
  CMake 3.14 or higher is required.  You are running version 3.10.2

Exception: Unable to generate build files

We use FetchContent_MakeAvailable which the docs say has been added in 3.14. This was the main motivation for using 3.14 as a required lower version, I'm not sure if it's safe to accept older versions too?

Whoops, I missed it. No, it's not safe to accept older versions.

Do you think there's anything we could do about this? I didn't know that Flutter bundles their own cmake, it's using the one from the system for me.

I think it bundles CMake if you use the snap version, as recommended by the Flutter "getting started" tutorial for GNU/Linux.
An option could be to use FetchContent_GetProperties as described (and not recommended) in the overview https://cmake.org/cmake/help/latest/module/FetchContent.html#id2
I eventually pinned the previous version of this library for my project.

I don't know much about snaps, but judging from this it looks like they just depend on the cmake package from the distribution? Ubuntu 21.10 should ship cmake 3.18, so I'm still not sure where you got the 3.10 from.


Remove call of Twig_Template::getEnvironment

It deprecated since 1.20 (to be removed in 2.0)

+1

ping @asm89


New to Analytics does

Its not a bug just a information required as I am new to Google Analytics. Google Analytics provide a

you don't have to add anything, you only need to pass your domain ID and the plugin takes care of the rest
// example
Vue.use(VueGtag, {
  config: { id: "UA-1234567-1" }
});

if you also want to track page switch automatically, you can pass the router instance as third parameter
// example
const router = new VueRouter({ ... })

Vue.use(VueGtag, {
  config: { id: "UA-1234567-1" }
}, router);

for other things you can look at the documentation

vue-gtag documentation
gtag.js official documentation


Update Getting Started experience for Python

The change includes the following changes:

Improve Getting Started experience: Use the standard confetti-filled welcome page
Use updated balena example projects: Some links were wrong
Update flask & Dockerfile: Upgraded to Flask 2.0.0

@balena-ci retest

@balena-ci I self-certify!

@balena-ci retest


bug: Cody keeps reverting code back to previous code

Version
v1.32.2
Describe the bug
In the last day or so, Cody seems to ignore any context I place in the message box, and defers to previous conversation context instead.  If it has Items [a, b, c] in a list, for instance, and that order has appeared in the conversation previously, and then I manually change it to [c, b, a] and save the file, and then ask Cody to add line d, it will generate code with [a, b, c, d] instead of [c, b, a, d], using the old order rather than building off of the current code that I included in context.  This is with Claude 3.5 Sonnet.
Also at the same time, clicking on the downvote button no longer does anything, it stays hollow black:

Expected behavior
If I include a file in context using Alt+L, it should base its modifications on that file as it currently exists, not on the previous conversation.
Additional context
Did prompts change in the latest update or something that would cause this?
No response

For example:

Despite me explicitly highlighting the new abbreviated labels and sending them to the chat context:

Every time I try, it insists on its own previous output rather than the code I am showing to it, unless I copy and paste it myself:

I highlight this line and try again:

but it gives a different line:

I highlight this line and try again:

But it gives a line from a completely different file:

If I start a new chat, it says it doesn't see anything at all, which is likely the underlying problem?

I reverted to v1.30.3 but it's still happening.  😣

Thanks @endolith for flagging this bug. I believe this is either than indexing bug with our local context fetching or just a plan bug on our end. Will address

It may just be an issue with that particular conversation?  Maybe because it's very long?

Yeah I don't think it's a length issue, I've been able to repro this bug on my own end with a short conversation

It seems like once cody has context, it doesn't get updated any more, but if you give it new context it works fine.
E.g. starting a new chat with context always works, and if I start a chat and give it only A.tsx, and then later B.tsx as context it has accurate knowledge of B.tsx.
However, If I give it directly A.tsx & B.tsx, then later change something in either and provide it again with the files, it doesn't have the updated knowledge.
I haven't structurally tried to proof that this happens all the time, but it feels like it does.

Yeah, context doesn't seem to change mid convo, even if I delete embeddings folder and it rebuilds it

This is still a frequent problem :/

Still happening.  I am going to try your competitors now.  :(

Yeah this apparently isn't just a bug, but is how things work. If you want Cody to use the newest code you MUST paste the latest code wholesale into the chat window. Otherwise it will used cached code that you may have changed 20 times on the screen. This is a horrible problem. The way around it, as far as I can tell, is to open a new conversation.
This bug really really really needs fixing - it will reintroduce bugs you have fixed and you may not even realize that Cody changed the code back to before the bug you fixed in the background.
I have less serious complaints about this product, but this is by far the most serious issue in this product that I have experienced at this moment.

@davidbien
I switched to Cursor... 💀
It's more expensive though, so Cody could still win me back by fixing things, maybe. 😬

Cursor doesn't integrate in vscode with a plugin - this is key cuz I'm already using Xcode and vscode to develop my iOS app, and I just think that Cursor's decision to supply an editor is so weak that I didn't even try it - like I want to be dependent on another editor.

Cursor is VScode, though, so it runs all the same plugins, but with less clunky integration of the AI.

Yeah I tried Cody for a bit. It lost the plot on the chat we were having after like 2-3 prompts - suddenly it wondered what we were talking about - what problem I was having. Cody works tons better than that - like night and day. It might be a problem that it maintains cached code - but it isn't a problem that it maintains an incredible chat context that you can even go back hours later and ask it to restore the comments on a method it generated where it stomped on my comments and it did it perfectly. That's pretty sick.
One thing is though: even when I post the new code straight into the chat window it may even still use a cached version of that code when generating. I have to then tell it to please use the code I just pasted and it will. The persistence of old code from long ago in the conversation is impressive honestly - but also sucks as we understand.

The key, really, when working with Cody's impl of Claude 3.5 Sonnet, is: if you are working on a class in the chat window - which is my primary modus operandi, make any changes to the code using Cody. Even if you want to make a small change to the class that the AI didn't notice - which happens all the time - use the chat window to make that change. This keeps the AI's idea of the code in its context the same as what is in the editor. If you go and fix a problem on your own and don't tell Cody you are going to do it then Claude's context diverges from what is in the actual source file.

Yeah, I've seen that happen in both.  (This bug is a different problem, though, where the AI is literally being fed the wrong lines of code.)