Server does not start after fresh import from Strava: KeyError: 'consider_for_achievements'

After a fresh import of Strava exported data, the server does not start.
The relevant cache migration seems to be applied before import of activities, and after import, it crashes.
It seems, the relevant key is not set during import.
I am running version 0.24.2
Full log below:
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Apply cache migration if needed …
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Applying cache migration delete_activities_per_tile …
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Applying cache migration reset_time_series_embellishment …
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Applying cache migration delete_tile_visits …
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Applying cache migration delete_heatmap_cache …
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Applying cache migration delete_activity_metadata …
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Applying cache migration delete_activity_metadata …
2024-08-05 16:28:04 geo_activity_playground.core.cache_migrations INFO Applying cache migration convert_distances_to_km …

Import from Strava export:   0%|          | 0/1 [00:00<?, ?it/s]2024-08-05 16:28:0
Import from Strava export: 100%|██████████| 1/1 [00:00<00:00,  2.83it/s]
Embellish time series data: 100%|██████████| 1/1 [00:00<00:00, 339.56it/s]
Extract explorer tile visits:   0%|          | 0/1 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/pandas/core/indexes/base.py", line 3805, in get_loc
    return self._engine.get_loc(casted_key)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "index.pyx", line 167, in pandas._libs.index.IndexEngine.get_loc
  File "index.pyx", line 196, in pandas._libs.index.IndexEngine.get_loc
  File "pandas/_libs/hashtable_class_helper.pxi", line 7081, in pandas._libs.hashtable.PyObjectHashTable.get_item
  File "pandas/_libs/hashtable_class_helper.pxi", line 7089, in pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: 'consider_for_achievements'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/app/geo_activity_playground/__main__.py", line 118, in <module>
    main()
  File "/app/geo_activity_playground/__main__.py", line 93, in main
    options.func(options)
  File "/app/geo_activity_playground/__main__.py", line 67, in <lambda>
    *make_activity_repository(options.basedir, options.skip_strava),
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/geo_activity_playground/__main__.py", line 112, in make_activity_repository
    scan_for_activities(repository, tile_visit_accessor, config, skip_strava)
  File "/app/geo_activity_playground/webui/upload/controller.py", line 114, in scan_for_activities
    compute_tile_visits(repository, tile_visit_accessor)
  File "/app/geo_activity_playground/explorer/tile_visits.py", line 83, in compute_tile_visits
    if activity["consider_for_achievements"]:
       ~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pandas/core/series.py", line 1121, in __getitem__
    return self._get_value(key)
           ^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pandas/core/series.py", line 1237, in _get_value
    loc = self.index.get_loc(label)
          ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pandas/core/indexes/base.py", line 3812, in get_loc
    raise KeyError(key) from err
KeyError: 'consider_for_achievements'

Thanks for the pull request! Unfortunately I have refactored the code and haven't pushed it. There is now a separate layer between the raw extracted activities and the enriched ones. I now apply default values to all columns such that this shouldn't happen again.

Your latest change from a41e508 leads to a new error:
  File "/app/geo_activity_playground/importers/strava_checkout.py", line 164, in <listcomp>
    if not (activity_extracted_meta_dir() / f"{activity_id.pickle}").exists()
                                               ^^^^^^^^^^^^^^^^^^
AttributeError: 'int' object has no attribute 'pickle'

Should probably be  f"{activity_id}.pickle"?


ct-server stability during election

Hello,
Similar to issue #811, we are seeing a failure of the ct-server on random instances about once a week.
The following entry was in the ct-server.FATAL log:
Log file created at: 2017/09/05 16:43:39
Running on machine: ip-172-31-11-120
Log line format: [IWEF]mmdd hh:mm:ss.uuuuuu threadid file:line] msg
F0905 16:43:39.573050  1252 masterelection.cc:362] Check failed: task->status().ok() /root/election/7a1ff88f-7cfb-4105-a657-38f2b1f5df6b: FAILED_PRECONDITION: Invalid JSON: Couldn't find 'node'

However, the etcd cluster appears to be completely healthy:
./etcdctl cluster-health
member 14f4d580c02ab5c5 is healthy: got healthy result from http://<IP_ADDRESS>:4001
member 40b2ec55845b6dec is healthy: got healthy result from http://<IP_ADDRESS>:4001
member dcbb94f558f01d98 is healthy: got healthy result from http://<IP_ADDRESS>:4001
cluster is healthy

Below is some more information regarding version(s)
etcd & ct-server instance OS & version:
Ubuntu 16.04 LTS (AWS)
etcd version info:
2017-09-05 18:37:18.177416 I | etcdmain: etcd Version: 3.2.2
2017-09-05 18:37:18.177428 I | etcdmain: Git SHA: cb2a496
2017-09-05 18:37:18.177439 I | etcdmain: Go Version: go1.8.3
2017-09-05 18:37:18.177451 I | etcdmain: Go OS/Arch: linux/amd64

ct-server version info:
ct-server version 66796fee67cc7785ea653bda4713496b968c24a4
Debug build (NDEBUG not #defined)

Like issue #811, is the best approach to extend the etcd timeout(s)?

Here are the logs.  WARNING and INFO logs truncated to the last 10k lines of each file.
ct-server_logs_truncated_20170905.tar.gz

I wanted to follow-up, and close this issue, as we have been stable for a few months.  We updated our ct-server instances using #1412, and merged #1417.


Typo Explicitely -> Explicitly

I have just signed the CLA, not sure why the CLA check is failing

@vicentealencar, probably just some kind of race condition, the check is now passing.

Thanks for the patch, @vicentealencar, merged!


.gitattributes - TTF files need to be listed as binaries

One annoying problem I have with this repository is that ttf (True Type Fonts) files from the nasm directory and another file.  I've touched non of these files but they always listed as "modified"
    modified:   nasm/Arial-Bold.ttf
    modified:   nasm/Arial-BoldItalic.ttf
    modified:   nasm/Arial-Italic.ttf
    modified:   nasm/Arial.ttf
    modified:   perl/perl-binary-module-dependency-1.template

This is getting in the way of anything I'm doing including switching branches with SourceFree ( https://www.sourcetreeapp.com/ )

I never ever had similar problems. Could be the problem of SourceFree. Have you tried plain git status command line from the repository?

It could be.  But still the files seem to change in the GUI with the MSYS2 repository.  But some thing may have to do with line-endings.

I have the same problem.


Directions Missing or Unclear

The title of this doc is "Change your email address...", but there are no actual instructions on this page on how to change any email address.
The doc goes into adding a custom domain, which is already in the Setup section. The doc then states the Add Custom Domain wizard will guide you, with no further instruction.
If the Add Domain setup wizard is supposed to also change your Admin email from .onmicrosoft.com to your custom domain, that should be included in the Setup doc, Add Domain section. The Add Domain doc should then link here, where there should be instruction on how to manually update the default .onmicrosoft email account to the custom domain for the admin user, and then secondary users. For myself, the default admin account did not appear to change from the .onmicrosoft account, and no additional emails or aliases were added after adding a custom domain. I came here to find out how to do it manually.

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

ID: d1394eba-4e78-2fd6-84f3-4ee64aedab51
Version Independent ID: 0ba9ff8c-20a8-58a9-dce7-a3bb0627c134
Content: Change your email address to use your custom domain - Microsoft 365 admin
Content Source: microsoft-365/admin/email/change-email-address.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

Here are some instructions that may help fill out this section. This is what I had to do to fix the my issue. Pretty simple, but should be included in the help docs.
Resolved Issue: After adding custom domain, your Username/E-Mail is still @.onmicrosoft.com
How to change your email address to a custom domain
You must of completed the Add Domain Setup to update your email address to custom domain

Enter the Microsoft 365 Admin Center
Open Users > Active Users
Click on the Name of the user you wish you modify.
A side panel will open with the details of the user account.
Under Username/Email, click on Manage Username
Click the Dropdown to the right of the @ sign, containing the .onmicrosoft account address, and change it to your custom domain.
Save Changes
If you were logged in under the account you changed, Microsoft 365 will now log you out, and you can log in with your new Username/Email.

@ApeironThanatos Thank you for your feedback. We have made changes to the doc via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/1767
Thanks
Sri

Well, there is only the *.onmicrosoft.com address to select. I wanted to change to my real address, but I do not want to have microsoft host my email. I have my own email server. Problem now, is that all my contacts on teams believe I have this onmicrosoft.com address. This article gave me hope there was a solution, but turns out it did not give any solution to this.

@ApeironThanatos @lefodnes - According to the author @pebaum, this article is being decommissioned and incorporated into another as indicated on this https://github.com/MicrosoftDocs/microsoft-365-docs/pull/1767#issuecomment-631616048.
We will close this issue now. Thank you for your contribution to make the docs better! Much appreciated!


Updates to CHANGELOG for 6.3

Changelog updates based on taking all the commits in 6.3 since 6.2 and bowling them down to human readable descriptions of the changes.  All PRs where previously shared in a xls with the team to categorize.  Please review for grammar

I'll create a separate PR later for putting this in the format used in the release notes, but it's good for now.

I merged your changes here already


Chrome may suspend the Web Audio context absent a user gesture

An AudioContext must be created or resumed after the document received a user gesture to enable audio playback.

I get this warning in Chrome Canary (M65) on, eg, http://phaser.io/examples/v2/audio/fade-in (iframe), and on my localhost (no iframe) when I open a game tab in the background and then focus it. But I don't get it when doing the same on https://samme.github.io/phaser-examples-mirror/audio/fade in.html. Not sure why.
The audio context is created but with a suspended state. Playback appears to go normally but there's no audio output.
A workaround would be
if (game.sound.usingWebAudio &&
    game.sound.context.state === 'suspended')
{
  game.input.onTap.addOnce(game.sound.context.resume, game.sound.context);
}

The AudioContext was not allowed to start. It must be [resumed] (or created) after a user gesture on the page. https://goo.gl/7K7WLu

https://samme.github.io/phaser-examples-mirror/audio/fade in.html

Phaser can try to resume the audio context via mouseout, which is wrong and will fail.


Getting a AssertionError: "undefined" === "function" when pushing

Hi all,
I am a little new to meteor and heroku so please excuse me if this is a simple issue.
I am running one of the newer versions of meteor (1.4.1) and its node.js version is 4.5.
After pushing the heroku app I get the following error:

2016-09-12T07:56:42.786286+00:00 app[web.1]: assert.js:93
2016-09-12T07:56:42.786521+00:00 app[web.1]:   throw new assert.AssertionError({
2016-09-12T07:56:42.786524+00:00 app[web.1]:         ^
2016-09-12T07:56:42.788035+00:00 app[web.1]: AssertionError: "undefined" === "function"
2016-09-12T07:56:42.788037+00:00 app[web.1]:     at wrapPathFunction (/app/build/bundle/programs/server/mini-files.js:77:10)
2016-09-12T07:56:42.788037+00:00 app[web.1]:     at Object.<anonymous> (/app/build/bundle/programs/server/mini-files.js:108:24)
2016-09-12T07:56:42.788038+00:00 app[web.1]:     at Module._compile (module.js:456:26)
2016-09-12T07:56:42.788038+00:00 app[web.1]:     at Object.Module._extensions..js (module.js:474:10)
2016-09-12T07:56:42.788039+00:00 app[web.1]:     at Module.load (module.js:356:32)
2016-09-12T07:56:42.788039+00:00 app[web.1]:     at Function.Module._load (module.js:312:12)
2016-09-12T07:56:42.788040+00:00 app[web.1]:     at Module.require (module.js:364:17)
2016-09-12T07:56:42.788041+00:00 app[web.1]:     at Object.<anonymous> (/app/build/bundle/programs/server/boot.js:9:13)
2016-09-12T07:56:42.788040+00:00 app[web.1]:     at require (module.js:380:17)
2016-09-12T07:56:42.788041+00:00 app[web.1]:     at Module._compile (module.js:456:26)
2016-09-12T07:56:42.868321+00:00 heroku[web.1]: State changed from starting to crashed
2016-09-12T13:39:46.392377+00:00 heroku[router]: at=error code=H10 desc="App crashed" method=GET path="/" host=jonpon.herokuapp.com request_id=bad357bd-e1e3-43c5-9e51-fb46f73161a3 fwd="<IP_ADDRESS>" dyno= connect= service= status=503 bytes=
2016-09-12T13:39:46.635477+00:00 heroku[router]: at=error code=H10 desc="App crashed" method=GET path="/favicon.ico" host=jonpon.herokuapp.com request_id=d4bbbe6b-dd5f-4cf8-9f33-3bb023ff5082 fwd="<IP_ADDRESS>" dyno= connect= service= status=503 bytes=

After doing some research I came across this https://github.com/meteor/meteor/issues/7475 thread which says it is some sort of issue with mup. I dont recall using mup so  I have no idea what they are referring to.
Is there any solution to this issue?
I was using the other meteor buildpack and switched to this one only to get the same issue...

That particular error is one that is typically thrown when you are attempting to run a Meteor 1.4+ app with node v0.10.x.  It looks like there might be something funny going on with setting the node version.
Could you share the contents of the .meteor/release file from your project?
What is the output of heroku buildpacks within your project?
Can you run heroku config:set BUILDPACK_VERBOSE=1, push to heroku, and share the output of the build operation?

.meteor/release<EMAIL_ADDRESS>heroku buildpacks : === jonpon Buildpack URL https://github.com/AdmitHub/meteor-buildpack-horse.git
heroku config:set ... when i pushed it had the standard "everything up-to-date", the most recent error in the logs is :
2016-09-13T03:50:10.605560+00:00 heroku[web.1]: Process exited with status 8
2016-09-13T03:50:10.542283+00:00 app[web.1]: assert.js:93
2016-09-13T03:50:10.541886+00:00 app[web.1]:
2016-09-13T03:50:10.542530+00:00 app[web.1]:   throw new assert.AssertionError({
2016-09-13T03:50:10.542534+00:00 app[web.1]:         ^
2016-09-13T03:50:10.544447+00:00 app[web.1]: AssertionError: "undefined" === "function"
2016-09-13T03:50:10.544448+00:00 app[web.1]:     at wrapPathFunction (/app/build/bundle/programs/server/mini-files.js:77:10)
2016-09-13T03:50:10.544449+00:00 app[web.1]:     at Object.<anonymous> (/app/build/bundle/programs/server/mini-files.js:108:24)
2016-09-13T03:50:10.544450+00:00 app[web.1]:     at Module._compile (module.js:456:26)
2016-09-13T03:50:10.544450+00:00 app[web.1]:     at Object.Module._extensions..js (module.js:474:10)
2016-09-13T03:50:10.544451+00:00 app[web.1]:     at Module.load (module.js:356:32)
2016-09-13T03:50:10.544452+00:00 app[web.1]:     at Function.Module._load (module.js:312:12)
2016-09-13T03:50:10.544452+00:00 app[web.1]:     at Module.require (module.js:364:17)
2016-09-13T03:50:10.544453+00:00 app[web.1]:     at require (module.js:380:17)
2016-09-13T03:50:10.544453+00:00 app[web.1]:     at Object.<anonymous> (/app/build/bundle/programs/server/boot.js:9:13)
2016-09-13T03:50:10.544454+00:00 app[web.1]:     at Module._compile (module.js:456:26)
2016-09-13T03:50:10.626393+00:00 heroku[web.1]: State changed from starting to crashed
2016-09-13T03:50:10.627381+00:00 heroku[web.1]: State changed from crashed to starting

Sorry if I wasn't clear -- the log that would help me is the one of the build process itself (e.g. the output when you run git push heroku), not the heroku logs after the slug has been built.

Oh no worries, my bad.
It just returned the everything up-to-date
Here is a screenshot of the commands

To trigger a rebuild (which displays the build process), you'll need to make a new commit.  Heroku will only rebuild the slug if there are new commits since the last build.

Oh wow, it works now.
To review, basically what happened was that I needed to commit again after I changed the buildpack?
Thanks

No, the buildpack was the same.  But the commit triggered a rebuild.
The buildpack is a set of instructions for Heroku to rebuild your app.  It only executes those instructions if it thinks your app has changed; which you indicate by having new commits.
Glad it works!


Create modal

Hi :wave:
For this PR, I added a modal to choose an inventory to create a shopping list with.

Good job! Time to merge!


Cannot find the module after compiling

Hello,
This is what I did :
I first compiled libmodsecurity in /usr/local/httpd-2.4.33 and edited the configure to find libmodsecurity
Here is my script
export LIBRARY_PATH=/usr/local/httpd-2.4.33/lib/
cd /somepath/ModSecurity-apache-master
./autogen.sh
./configure --with-apxs=/usr/local/httpd-2.4.33/bin/apxs --with-libmodsecurity=/usr/local/httpd-2.4.33
make -j4
make install
So now I can't find the module and I don't know how to install it with apache
Can you help please?

@Lud3rik Sorry for the long delay :(
Can you check if the issue persists by applying one of the patches below:
#29
#39
#20
Thanks for letting us know! :)

make sure that your platform is configured to read the library from the LIBRARY_PATH. Apache env or the platform, should be configured for that.


[Bug]: CDPPage doesn't exist for Target with page type

Minimal, reproducible example
const puppteer = require('puppeteer');

puppteer.launch({
    headless: false,
    channel: 'chrome',
}).then(async browser => {
    const page = (await browser.pages())[0];
    await page.goto('https://www.example.com');
    const iterations = 10;
    const pagesPerIteration = 10;
    const urls = ['https://www.example.com', 'https://www.google.com', 'https://www.bing.com']; // more reproducable if open different pages
    for (let i = 0; i < iterations ; i++) {
        for (let j = 1; j <= pagesPerIteration; j++) {
            await page.evaluate((_urls) => {
                window.open(_urls[Math.floor(Math.random() * _urls.length)]);
            },urls);
        }
        await new Promise(resolve => setTimeout(resolve, 1000)); // to avoid any race conditions

        const pages = await browser.pages();
        const targets = browser.targets().filter(target => target.type() === 'page');

        if(targets.length !== pages.length){
            throw new Error('target without page');
        }

        while((await browser.pages()).length > 1){ // close all pages except the first one to avoid memory leak
            await (await browser.pages())[1].close();
        }
    }
})

Error string
target without page
Bug behavior

[X] Flaky
[ ] PDF

Background
I use automation for pages, that are opened by the user.
After upgrading to Puppeteer 20, I notice sometimes I can't perform it.
All targets are still available, but puppeteer CDPPage doesn't exist for some of them. It occurs randomly.
if(target.type() === 'page'){
  await target.page() // will return null
}

it doesn't like it is a race. I have an infinite loop, and a CDPPage is never created for these targets.
Expectation
const pages = await browser.pages();
const targets = browser.targets().filter(target => target.type() === 'page');
page.length === targets.length

page should exist for all targets wich type is page
Reality
sometimes target with type page exists but the page is not
Puppeteer configuration file (if used)
No response
Puppeteer version
20.9.0
Node version
18.16.0
Package manager
npm
Package manager version
9.5.1
Operating system
Windows

I think the problem is in the line:
await new Promise(resolve => setTimeout(resolve, 1000)); // to avoid any race conditions

the pages call is asynchronous and it will wait for the newly created pages to initialize, that is if the newly created page has been detected. 1000ms does not seem to be enough for that. You need to use a higher timeout or explicitly wait for the right number of targets before calling the pages.

I can increase this to hours, still will not be available. I've added this only for repro.
In my app, I use waitForTarget  in an infinite loop to track targets.

import * as puppeteer from 'puppeteer';
async function main() {
const browser = await puppeteer.launch({
headless: false,
channel: 'chrome',
});
const page = (await browser.pages())[0];
await page.goto('https://www.example.com');
const iterations = 20;
const pagesPerIteration = 10;
const urls = ['https://www.example.com', 'https://www.google.com', 'https://www.bing.com']; // more reproducable if open different pages
for (let i = 0; i < iterations ; i++) {
    for (let j = 1; j <= pagesPerIteration; j++) {
        await page.evaluate((_urls) => {
            window.open(_urls[Math.floor(Math.random() * _urls.length)]);
        },urls);
    }
    await new Promise(resolve => setTimeout(resolve, 1000)); // to avoid any race conditions

    const pages = await browser.pages();
    const targets = browser.targets().filter(target => target.type() === 'page');

    if(targets.length !== pages.length){
        throw new Error('target without page');
    }

    while((await browser.pages()).length > 1){ // close all pages except the first one to avoid memory leak
        await (await browser.pages())[1].close();
    }
}
await browser.close();

}
main();
typesciprt code

non-flaky version:
const puppteer = require('puppeteer');

puppteer.launch({
  headless: false,
}).then(async browser => {
  const page = (await browser.pages())[0];
  await page.goto('https://www.example.com');
  const iterations = 10;
  const pagesPerIteration = 10;
  const urls = ['https://www.example.com', 'https://www.google.com', 'https://www.bing.com']; // more reproducable if open different pages
  for (let i = 0; i < iterations; i++) {
    for (let j = 1; j <= pagesPerIteration; j++) {
      await page.evaluate((_urls) => {
        window.open(_urls[Math.floor(Math.random() * _urls.length)]);
      }, urls);
    }
    await browser.waitForTarget(() => browser.targets().filter(target => target.type() === 'page').length === pagesPerIteration + 1);
    const pages = await browser.pages();
    const targets = browser.targets().filter(target => target.type() === 'page');

    if (targets.length !== pages.length) {
      throw new Error('target without page');
    }

    for (const p of (await browser.pages())) {
      if (p !== page) {
        await p.close();
      }
    }
  }
})

@OrKoN Thanks a lot, for pointing to this.
I used something like this.
const foundTargets = new Set();
for(let j = 1; j<= pagesPerIteration; j++){
        const target = await browser.waitForTarget(target => foundTargets.has(target._targetId) === false);
        foundTargets.add(target._targetId);
}

and it's not equal to
await browser.waitForTarget(() => browser.targets().filter(target => target.type() === 'page').length === pagesPerIteration + 1);

thank you very much, sorry for disturbing


Initial Pose

Is there any way we can provide an initial pose to the method?
Thank you in advance for the help!

Hi Shan,
When I tried changing transformSum[] in the code, map building became worse.
Should I change other things more in the code?


[APMSqliteStore ensureTableWithName:elementaryColumnInfo:addOnColumnInfo:error:] crash

Description
[APMSqliteStore ensureTableWithName:elementaryColumnInfo:addOnColumnInfo:error:] crashes within 3 seconds of launching. And all the crashes are coming from iOS 16.
We hope to get it resolved as soon as possible. Thanks a lot!

The crash stack trace is in the crash.txt file. crash.txt
Reproducing the issue
No response
Firebase SDK Version
10.4.0
Xcode Version
14.3
Installation Method
CocoaPods
Firebase Product(s)
AB Testing, Analytics, Crashlytics, In-App Messaging, Performance, Remote Config, Storage
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

Thanks for reaching out, @linziyiwj. May I know if the issue is reproducible locally? If so, could you provide a sample app that reproduces the issue?

Thanks for reaching out, @linziyiwj. May I know if the issue is reproducible locally? If so, could you provide a sample app that reproduces the issue?

@rizafran Sorry, this crash problem cannot be reproduced locally, it is currently a crash problem collected through firebase


CUDA fails on torchcodec==0.1.1+cu124 but is fine on 0.1.1+cu121

🐛 Describe the bug
When trying to use the VideoDecoder with a random video with the standard pip install torchcodec version (and either CUDA version of torch 2.5.1), I get this:
Traceback (most recent call last):
  File "/home/ubuntu/.local/share/uv/python/cpython-3.10.16-linux-x86_64-gnu/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/home/ubuntu/.local/share/uv/python/cpython-3.10.16-linux-x86_64-gnu/lib/python3.10/threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "/home/ubuntu/si/jupe/decode_test.py", line 31, in worker
    video = VideoDecoder(video_file, device="cuda")
  File "/home/ubuntu/si/.venv/lib/python3.10/site-packages/torchcodec/decoders/_video_decoder.py", line 96, in __init__
    core.add_video_stream(
  File "/home/ubuntu/si/.venv/lib/python3.10/site-packages/torch/_ops.py", line 716, in __call__
    return self._op(*args, **kwargs)
RuntimeError: Unsupported device: cuda

However, if I simply swap out torchcodec to pip install torchcodec --extra-index-url https://download.pytorch.org/whl/cu121, it does not have this error, either when I am using torch==2.5.1+cu121 or torch==2.5.1+cu124
Happy to put more effort into making a minimum repro but I suspect this is pervasive enough to easily reproduce.
Versions
invariant:
python3.10.16
CUDA 12.6.85
torch2.5.1+cu121/cu124 (does not change behaviour)
FAILS ON
torchcodec==0.1.1+cu124
SUCCEEDS ON
torchcodec==0.1.1+cu121
Version info script fails with
python collect_env.py
Collecting environment information...
Traceback (most recent call last):
  File "/home/ubuntu/si/collect_env.py", line 692, in <module>
    main()
  File "/home/ubuntu/si/collect_env.py", line 675, in main
    output = get_pretty_env_info()
  File "/home/ubuntu/si/collect_env.py", line 670, in get_pretty_env_info
    return pretty_str(get_env_info())
  File "/home/ubuntu/si/collect_env.py", line 495, in get_env_info
    pip_version, pip_list_output = get_pip_packages(run_lambda)
  File "/home/ubuntu/si/collect_env.py", line 450, in get_pip_packages
    for line in out.splitlines()
AttributeError: 'NoneType' object has no attribute 'splitlines'

Hi, @devanshrpandey! CUDA support is only available through the PyTorch index URL specified in our install instructions: https://github.com/pytorch/torchcodec?tab=readme-ov-file#installing-cuda-enabled-torchcodec
Reading this instructions, I now realize that fact is implied, but not explicitly stated. I've filed #445 to track making that change.

Ah, that's actually on me. I assumed that it would have the same behavior as torch/torchaudio/torchvision. Apologies!

@devanshrpandey, no, that's on us, as we mistakenly thought the pypi version should be CPU only. It was only when writing up #445 that I realized we were mistaken. We should change that for the next release. Regardless, we should be explicit in the README until then.


Could we recommend a different scheduled build time?

Hi there, the readme in this repository shows a cron of 0 0 * * 0 which is 00:00 on Sunday. We get a large spike of runs at the top of every hour because lots of people have copied this value into their Actions. We cope with this by smoothing out the load at that time.
So this possibly the worst time for us and our customers to run this build. Could we avoid this time and come up with a different one please?

In the workflows generated by people clicking "set up this workflow" (see screenshot below) the build time is randomly generated within some bounds. So for workflows set up using that method they'll already be moderately distributed.

For this repo we can't do that and I think it would be quite odd to pick a hard-coded random time. We could add a comment into the example workflow asking people to pick a random time to help ease load. How would you explain that?

So glad to hear that using Set up this workflow uses a random time. The README  says "To get code scanning results from CodeQL analysis on your repo you can use the following workflow as a template:", so the proposed https://github.com/github/codeql-action/pull/259 ☝🏾 just moves it back by an hour and half to not be at our peak. That way if anyone does copy the workflow instead of using Set up this workflow they'll be better off.


[mimalloc] Fix build error on x64-android

/home/vzhli17/jim/vcpkg/buildtrees/mimalloc/src/v2.1.7-d58a1dd7b9/src/arena.c:798:3: error: no matching function for call to 'atomic_store_explicit'
  mi_atomic_store_release(&segment->thread_id, 0);
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Refer to upstream PR 694 for modification.

[X] Changes comply with the maintainer guide.
[ ] ~SHA512s are updated for each updated download.~
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version.~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[ ] Any patches that are no longer applied are deleted from the port's directory.
[ ] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[ ] Only one version is added to each modified port's versions file.

Compile test pass with following triplets:

x64-windows
x64-android

An issue 950 was raised upstream for tracking.

Can you submit an upstream PR?

Can you submit an upstream PR?

Submitted PR 954 upstream.

Can you submit an upstream PR?

Submitted PR 954 upstream.

The fix PR has been merged upstream.


Feature/tri 370 implement endpoint get a list of jobs with processing state

@ds-ext-ooyeneye please answer or fix review comments!

Hi @ds-jhartmann @SaberIstos , can I ask for your opinion here?
Dapo and I had an open discussion this morning, if we should do requested changes for JobStatusResult class or not.
For me changing from String's to specific type properties (UUID, JobStates) gives:

cleaner solution,
easier to maintaine and underestand API model,
open–closed principle for further modifications/adjustments,
no need to parse to string few times - spring's json parser does this anyway.

So the question is: what you think - should we stay with String, or do the change?

Hi @ds-ext-kmassalski, to understand you correctly: You want to change 'String jobId' to 'UUID jobId' and 'String status' to 'JobState status'? Or do you have something different in mind?

"You want to change 'String jobId' to 'UUID jobId' and 'String status' to 'JobState status'?" - correct

In this case I think we should do that. As you mentioned, it adds to maintainability, is cleaner to understand and provides precisely what the consumer of our API can expect as a result.

After analyzing the changes in the above feature branch, I guess that using the benefit of the spring json parser is the common way instead of parsing the jobId as a string for several times. So the usage of UUID as a data type is better from my point of view.


setting enumerations from strings

I am migrating from a 'smart enum' library based on what used to be 'BOOST_ENUM'. I noticed that the _from_string function is static, and recall having been bitten by this before in that library.
My problem is that it is possible to call this as a member function on an existing enumeration with rather unpleasant consequences.
BETTER_ENUM(Enum, int, A, B, C)
Enum X = Enum::_from_string("A")
X._from_string("B"); // legal, but no change of X

I would much prefer the function to be non-static, in order to make sure that at the end X == Enum::B holds, and (for example) having a factory function akin to what is present in the standard library, such as:
X = make_enum<Enum>("A");
X = make_enum<Enum>(2);

Also, I am missing constructors that allow for example:
Enum X("A")
Is there a good reason to not allow this?

I think it's ultimately a matter of preference. I think something close to what you are proposing is also discussed in Should Better Enums provide enum "objects" or traits types? Depending on what exactly you'd like to do, there may be some small technical limitations. I think you're proposing something between the current "object" interpretation and a full-on traits-only interpretation, so to answer your question usefully, I'd (we'd) have to look in total at what would be implemented where and how.
You're welcome to experiment. You can use that question as a list of things to watch out for – I think it's all I know. You can also glance in the branch that the question mentions. Though  the branch is quite outdated, it's good enough to get an idea everything that needs to go into a partial- or full-traits implementation.
In particular, the make_enum factory would probably conflict with declaring enums inside namespaces, and would need some kind of workaround. It's probably solvable with a combination of private data/static methods in objects, and maybe some kind of friend traits class.
Regarding the specific choices:

I (as a matter of preference) decided that fully-explicit, named conversions are "safer" than implicit or explicit casts. I also personally tend to find them much easier to read. Hence the named from_string function, rather than converting constructors.
I also prefer to think of most of my small "objects" as mostly immutable, with mutation explicitly opted into by writing a visible assignment operator. Hence, I would not want from_string to conflate conversion with mutation. Better Enums are supposed to mimic language enums and other basic language types. This actually makes me want to go back and make even more functions static.
from_string should work even if there is no object yet. from_string is logically the enum constructor that converts from strings. It's just not a C++ in-language constructor, but an explicitly-named one obtained by abusing a static function.
Given that there case-sensitive and case-insensitive string conversion functions, it immediately seems odd to me to prefer one to the other for implementing a converting constructor.

It's pretty unfortunate that C++ allows calling from_string as if it were a non-static function. I'd love to disallow that somehow, given the above interpretation.
The stuff above is suited to how I, and some proportion of C++ programmers, think about code. If it hurts you, you're welcome to modify the library. If you think it will be definitively better, a discussion or, even PR, would be quite welcome :)

Thanks for that, I think I understand your arguments. Indeed, it would not be much of an issue if the compiler could prevent the static function being called on objects somehow. Just for my understanding, what is the reason for having case sensitive and case insensitive versions of the _from_string factory function? Isn't this dangerous as the _enumeration is in fact case sensitive and could have both "A" and "a" as valid values?

The strongest justification (not that strong) is that the wider project that spawned Better Enums needed both kinds of string conversion. Beyond that reason, it's only that it might be useful for someone else. I suppose at this point, case-insensitive conversion could be removed, if there is suitably large advantage in doing that.
Case-insensitive conversion is slightly dangerous, but I don't think it's particularly dangerous relative to what else can happen with enums. For example, converting from an integer is approximately as dangerous, because two enum constants might have the same numeric value.

Well, it's a different kind of danger. The result of conversion will be correct for the ambiguous from_integer conversion, but potentially incorrect for the ambiguous from_string_nocase conversion. Round-trips can be broken in both cases.


What are some possible uses cases?

This seems like an efficient way to represent the DOM.    What is an example real life use case that you thinks represents the sweet spot usage for this lib?

I've started using it for testing as a replacement for JSDOM. its speed is amazin in comparison but JSDOM's scope is huge so there are some cases where Linkedom won't work because it's missing some JSDOM properties. Nevertheless the speed difference is worth it as you can alway's drop back to JSDOM if needed.
Some people use it for SSR too.

thanks @haikyuu, curious to know what's missing, beside agreeing JSDOM scope is huge, and it's not this library purpose to implement everything JSDOM has. A list of great to have missing features would help me out eventually prioritizing, so if you ever get a chance to write down one as issue, I'd very much appreciated that 👋

Sure. I'll do that as I am trying to write my new tests using LinkeDOM.


cors error

I want to connect client web page for socketIO and admin-ui to socketIO server.
But the error appears.
When I use this code below, the client web disconnects with cors error.

const io = new Server(httpServer, {
cors: {
origin: "https://admin.socket.io",
},
});
instrument(io, {
auth: false,
});

Of course, I did like this, and "xhr poll error" happens on admin-ui.

const io = new Server(httpServer, {
cors: {
origin: "", // and I did ["https://admin.socket.io",""] instead of this
},
});
instrument(io, {
auth: false,
});

does admin-ui supports only local environment?

It seems you are missing the credentials: true part:
const { createServer } = require("http");
const { Server } = require("socket.io");
const { instrument } = require("@socket.io/admin-ui");

const httpServer = createServer();

const io = new Server(httpServer, {
  cors: {
    origin: ["https://admin.socket.io"],
    credentials: true
  }
});

instrument(io, {
  auth: false
});

httpServer.listen(3000);

Reference: https://socket.io/docs/v4/admin-ui/#server-side
Could you please check?

It seems you are missing the credentials: true part:
const { createServer } = require("http");
const { Server } = require("socket.io");
const { instrument } = require("@socket.io/admin-ui");

const httpServer = createServer();

const io = new Server(httpServer, {
  cors: {
    origin: ["https://admin.socket.io"],
    credentials: true
  }
});

instrument(io, {
  auth: false
});

httpServer.listen(3000);

Reference: https://socket.io/docs/v4/admin-ui/#server-side
Could you please check?

My mistake. The line "credentials: true" is not there, but I wrote it already.

any updates on this?

@MehdiMamas unfortunatly I am not able to reproduce the issue.
From my understanding there are two possible explanations:

either the cors configuration is incorrect (which does not seem to be the case here)
or the UI is not able to reach the Socket.IO server

Could you please share what gets printed to the console?

did you find the answer

Getting error as well.

anyone found a solution for this?

same error shows for me even after using * as origin please help.
const { instrument } = require ('@socket.io/admin-ui');
const express = require('express');
const app = express();
const http = require('http');
const { Server } = require('socket.io')
const cors = require('cors');

app.use(cors());

const server = http.createServer(app);

const io = new Server(server, {
    cors: {
        origin:"*",
        methods: ['GET', 'POST'],
        credentials: true
    }
});

server.listen(3000, () => {
    console.log('listening on *:3000');
})

instrument(io, { auth: false})

I was getting this error earlier but I found out this is due to the ad-blocking extension so try disabling the ad-block or change browser if you are using a browser like brave.

same issue here.
even with origin: "*" im getting GET http://localhost:8080/socket.io/?EIO=4&transport=polling&t=Oyga8kN net::ERR_FAILED 404 (Not Found)

I have also tried everything listen in this thread, to no avail.

Same issue


POST fails if Preceded by PUT

What is the expected behavior?
Expect the POST to always set the correct Content-Length header
What is the actual behavior?
The Post call uses the existing (LAST) set Content-Length from the previous call, in my case the code has a loop around a set of calls like (WHERE some_data is a string)-
while(something_or_other)
{
  curl->post(some_url, some_data);
  curl->put(some_other_url, some_params);
}

First time through the loop, works like a charm. The second time the Content-Length header is reporting the length from the previous PUT call.
What steps will reproduce the problem?
Create a sequence of calls like the above example, make sure content lengths are DIFFERENT and it should fail.
You can see the issue if you turn on curl verbose() which will show what it things the content length is and what is is sending Like this debug snip-
Content-Length: 45

* upload completely sent off: 45 out of 45 bytes

Above output that is OK, it fails with output looking like below
Content-Length: 45

* upload completely sent off: 25 out of 25 bytes

At this point the server on the other end times out waiting for data.
To verify the fix I set the content length explicitly before the post call and all is well. I did it like this -
$this->curl->setHeader('Content-Length', strlen($this->curl->buildPostData($card)));

I tried a hack and a PR but didn't realize that the post data could also be an array. I think the problem only shows up when the post data is a string type not an array.
Sorry for the hack PR :)

Hi. Similarly to #472, I believe this issue also should be fixed. Regards

Zach -
Just ran all the code on the new version and worked well.
Again thanks for an easy to use well thought out package!
Sandy

@sganz You're very welcome


ALL TEN PERCENT STOCK BOARDS

IF FALL PREVENTION OR FAAA ASK ABOUT LOGSTICS T CAN ONLY BE DONE FOUR DIFFERENT AGENCY WAYS?

BUY ANTIVIRUS PROGRAM.


⚠ Bofrost.lu has degraded performance

In 935a00e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1659 ms

Resolved: Bofrost.lu performance has improved in 43d7cff after 19 minutes.


SessionOption no longer valid

Connect-MicrosoftTeams : A parameter cannot be found that matches parameter name 'SessionOption'.

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

ID: d2abd383-7815-9404-41ba-72bbedb5208a
Version Independent ID: 88342584-33e2-47c6-baf1-81644d5cc7a5
Content: New-CsOnlineSession (SkypeForBusiness)
Content Source: skype/skype-ps/skype/New-CsOnlineSession.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

Hello @SharePointBlackBelt Vincent, how are you?
You opened the issue for the article New-CsOnlineSession but the error references Connect-MicrosoftTeams.
The article for Connect-MicrosoftTeams does not list SessionOption and New-CsOnlineSession is no longer included.
Can you please clarify?
Thank you

"Skype for Business Online Connector [e.g., New-CsOnlineSession] is currently part of the latest Teams PowerShell module [e.g., Connect-MicrosoftTeams] and we are no longer required to connect Skype for Business Online".
Ref: https://answers.microsoft.com/en-us/msteams/forum/all/new-module-no-longer-contains-the-new/d50d30f1-9068-4016-b09b-60b39918aeef
So if the S4BO commands are rolled into Teams PS module, then SessionOption should be an option, right?

@SharePointBlackBelt
S4BO commands are rolled into Teams PS module however as the note says this one is deprecated.
SessionOption is not part of Connect-MicrosoftTeams.

Ahh, OK, sorry. Closing


Pen does not work correctly in Bounds mode on Screen Ruler

Microsoft PowerToys version
0.74.1
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
Screen ruler
Steps to reproduce
I am using the Surface Slim Pen on the Surface Studio Laptop.
When using the bounds mode, the measurement area is never drawn or shown.  If I use the touchpad instead, it works as expected.
The other modes work properly with pen.
✔ Expected Behavior
Bounds mode should draw a box as the pen is tapped and dragged.
❌ Actual Behavior
A single circle appears (showing the tap was detected) and nothing else appears.
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/20504


Added my implemention of the bubblesort visualizer

I created this with P5.js in like 15 minutes, so ist not very advanced but it fulfills ist purpose :)

Your YAML was valid but we decided to not allow trailing spaces just to keep everything clean. That's why it was rejected. Maybe we should put that somewhere on the wiki page... 🤔


Zoom Icons render differently on browsers

Leaflet 1.2.0
Safari Version 11.0 (126<IP_ADDRESS>.7)
Chrome Version 61.0.3163.100 (Offizieller Build) (64-Bit)
Firefox 55.0.3 (64-Bit)
Opera 47.0.2631.83
Microsoft Edge 38.14393.0.0
This screenshot is taken from the homepage http://leafletjs.com

Please do not ignore Issue Template.
It's not clear what behavior you expect.
Every browser renders fonts / borders / shadows / etc differently. So it's okay, that those controls look different in different browser.

I expect that the icons look the same in every browser. I might add custom icons below the zooming icons and I want to be sure that the style of my icons match the style of the built in zooming icons (same height and width).

Styles are slightly different depending on Browser, because in case of Browser.touch leaflet-touch class is added to map containers, which overrides some styles.
You can easily override default Leaflet styles if you want to.

Hi @googol7, @cherniavskii describes the situation well.
The major difference is whether Leaflet detects touch or not, which affects the size of the controls (slightly larger for touch to make them easier to hit), as well as the shadow/border, presumably since some lower end devices had/have problems with box-shadow.
I'm closing this issue, since this is not a bug, but rather works as designed as far as I can tell.

See also #5266

Is there an option to avoid the leaflet-touch from being added?
If not, how can it be overridden easily?

Is there an option to avoid the leaflet-touch from being added?

Use L_NO_TOUCH switch.
https://leafletjs.com/reference-1.7.1.html#global-switches


number of Images is problem display

Hi all,
i had a problem display number of Images is Zero ?
review attachment

Hi there,
Yes this is what is displayed when NumberOfStudyRelatedInstances is not returned from the server. We should default it to something other than 0.

This has now been switched to undefined, so I'll close this.


Hail exploration

Exploration of HGDP + 1KG data using Hail

Ah, and also add an nbviewer link to the jupyter notebook on the main branch in a separate PR (after you're happy and this one is merged).


Run e2e testing nightly with dev versions of rstudio owned repos

Ex: htmltools

Closing in favor of #417. cc @wch


ERROR: (gcloud.app.deploy) INTERNAL: Internal error encountered.

Fresh install and walk through of the standard wordpress install instructions.
cd to my-wordpress-project and run gcloud app deploy app.yaml cron.yaml
First error is:
ERROR: (gcloud.app.deploy) The [application_readable] field is specified in file [/Users/coreysnyder/php-docs-samples/appengine/php72/wordpress/my-wordpress-project/app.yaml]. This field is not configurable with runtime [php72] since static files are always readable by the application. It can safely be removed.
So, I opened app.yaml and deleted each line of application_readable: true Saved and re-ran
gcloud app deploy app.yaml cron.yaml
Then I get ERROR: (gcloud.app.deploy) INTERNAL: Internal error encountered.
Not sure what to do now as the error doesn't give me a lot of info.

ok, so I think part of this may have been because of the docs again. I'm guessing here that I was in the wrong project in my shell. I learned this because I decided to try a fresh install from step one again, and got an error about "the project not existing". So I gcloud init which is not mentioned in the tutorial and initialized a new config.
That being said, I still got the error about the application_readable no longer needed. Did the steps of just opening app.yaml and deleting all the references to it save and re-running gcloud app deploy app.yaml cron.yaml
I did get a successful install. Hope that last little bit helps someone.


Clarified statement development stopping

This comment is based on Shougo's comment on reddit:

NeoBundle development is done. I have replaced to dein.vim. Dein.vim is faster and well tested. And it implements neovim asynchronous installer.

Thank you!


edsl.md: support for uint256

@MrChico looks good to me. thanks!
BTW, it's support for int256 not uint256, right?

Yeah, should be int256, that's a typo
On Wed, Jun 13, 2018 at 11:11 PM, Daejun Park<EMAIL_ADDRESS>wrote:

@MrChico https://github.com/MrChico looks good to me. thanks!
BTW, it's support for int256 not uint256, right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kframework/evm-semantics/pull/227#issuecomment-397164819,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIhE_0fQCq5aq9W6smu409xo8xKJbuTiks5t8eKEgaJpZM4Um9ax
.


downloaded program doesn't work when disconnected

Try this program with either the bar or ring.  Download the program to the micro:bit.
https://makecode.microbit.org/_6t8MkcT00EM8
Notice that the program runs fine when the micro:bit is connected to MakeCode.

Now, get rid of MakeCode and plug and unplug the micro:bit, so it is not connected to WebUSB, or power it by battery.
Note that the program doesn't work now.

The code is super simple and being used an example in doc:
input.onButtonPressed(Button.A, () => {
    modules.ledPixel1.setAll(0)
    let n = modules.ledPixel1.numPixels()
    for(let i = 0; i <= n - 1; i++) {
        modules.ledPixel1.setPixel(i, 0x00ff00)
        basic.pause(200)
    }
})

input.onButtonPressed(Button.B, () => {
    modules.ledPixel1.setAll(0)
    let n = modules.ledPixel1.numPixels()
    for (let i = 0; i <= n - 1; i++) {
        modules.ledPixel1.setPixel(i, 0x0000FF)
        basic.pause(200)
    }
})

Another need for reliable commands...

@mmoskal it seems that get num_pixels is always ignored in the disconnected case

Add ?passive=1 to prevent the web site to send any packet.


Please open source a "Computer" WASM keyword

Is there any way I can convince Picovoice to release an open source "computer" keyword under the Apache 2.0 license? (wasm version)
I bet a lot of people worldwide have been wanting to say "computer" and have their computers obey their commands like on Star Trek.  And your codebase can do it.   I've been working the latest version of Porcupine for a little while now and I think it's really awesome.  You guys have something really amazing here.  It's like a foundational level technology.
I don't know if you've been keeping tabs on what I've been working on but I've got Porcupine working as a wake word for DeepSpeech, I think the two systems working together is really amazing:
https://github.com/jaxcore/deepspeech-plugin/tree/master/examples/node-wakeword
The WASM keywords you've open sourced so far (like "bumblebee", "hey edison') are sufficient, but I think most people would probably prefer to say "computer" like on Star Trek.  I think it would be the #1 requested keyword.
So why not just open source "Computer"?
I understand you guys are trying to create a business around your keywords and need people to pay.  But what I'd like to try to convince you of, is if you open source a "computer' hotword it could become a sort of defacto open source alternative to "Alexa".  And the more people who use Porcupine the more opportunity there will be to sell them custom alternatives to the open sourced "Computer" keyword.

Hi there, I appreciate the comment. Currently, we are focused on servicing enterprise clients. The sole purpose of the code we have published on GitHub is to allow our customers to freely evaluate the performance and ease-of-integration before signing up for added services and development tools. From our standpoint, we'd rather people say "hey picovoice" to promote our brand than promoting a movie.


CanvasItem nodes anchored to any "wide" setting double in size in the editor when compiling from another scene

Tested versions

Reproducible in Godot v4.2.2-stable-win64

System information
Windows 10 - Godot v4.2.2 - using Mobile Renderer
Issue description
As the title suggests, CanvasItems keep doubling in size (in the EDITOR) when I run the project outside of the respective CanvasItem scene. Not necessarily run another scene in itself, but simply run the project with another scene opened.
Is there any way I can quickly fix this? I need to deliver a prototype till Saturday and I might need to switch engines.
Steps to reproduce

Run the project with the "home" scene opened, everything should be fine.
Go to the "button" scene, then run the project again.
Switch to the home scene again.
Suddenly, the panel in the "home" scene has doubled in size!

https://github.com/godotengine/godot/assets/82157205/3218b67e-6d21-43ee-803c-d2ccdf842f97
Minimal reproduction project (MRP)
BUG.zip

Thank you for reporting, consolidating in:

https://github.com/godotengine/godot/issues/90079

See there for more details, if you think something was missed about this and it's not the same issue, please comment here and it can be reopened


Make BufferingHelper public

This class is currently pub-internal. An implementation is necessary when reading the request body multiple times. For example, ASP.NET Core WebHooks includes resource filters that calculate hashes of body content. They use the following helper to ensure this is safe:
https://github.com/aspnet/WebHooks/blob/be8571953a0124c0877f777e8fb5bb17329204ab/src/Microsoft.AspNetCore.WebHooks.Receivers/Filters/WebHookVerifyBodyContentFilter.cs#L94-L106
For reference: https://github.com/aspnet/HttpAbstractions/blob/3e3772eecd4cc57399c28a3f899e6b0406ef2e1b/src/Microsoft.AspNetCore.Http/Internal/BufferingHelper.cs

Hmm, MVC used this class in a few places long before the ASP.NET Core WebHooks implementation was a thing.


Feature: border around spotlight

A border is made around the spotlight, which may help in accentuating the content.
A border is further useful for implementing pointer and zoom mode as they have borders around them as per #14 .
The current implementation is limited in the sense it do not work with Custom QQuickItem element if they do not have border properties defined.

Hi @mayanksuman, thanks for the pull request. Did you create a branch of the zoom feature branch? Because there are two commits within your pull request (from @rj-jesus) that should for now stay on that feature branch. Could you create a pull request without those please?
For the rest of the commits - I will look through it tonight or tomorrow.

@jahnf Yes, I created the branch from zoom feature branch by mistake. I have tried to rebase it now.

Code looks good, but if the border is not supported by all shapes, this will end with issue reports like "border does not work..." , so my proposal is one of the two (or both):

analogue to the allowSpotRotation for shapes there could be sth like supportsBorder and if a shape is selected that does not support the border the whole border group box will be disabled.
implement the border for all the current custom items.

The first one will be definitely the quicker one :)

Thanks @mayanksuman , will have another look tomorrow and probably merge, going offline now.
For the shapes that don't support a border (for) now - and the automatically disabling of the border group box - do you want to implement that? (I can also do that maybe tomorrow)

I am currently targeting to implement border on all custom shapes. Please feel free to implement the disabling of border groupbox if the shape do not have a border property.

After thinking about it the shapes don't need a border property at all if we would use a second opacity mask to create a border. This would support all custom shapes. I'll post an example tomorrow or Wednesday.. so you don't need to implement a border for the custom shapes.

That will be great. I will stop working on this and will focus on other issues.


Liberty server tasks blow away jvm.options

https://github.com/OpenLiberty/ci.gradle/pull/513 causes a delete of jvm.options and other configs during liberty tasks. What I guess we're supposed to do is to set up jvmOptions or jvmOptionsFile in the liberty.server block in build.gradle, or create a jvm.options in the config folder. This a problem for me because if you don't have these options set, then your server's jvm.options just gets deleted when running any liberty server gradle tasks. I'd rather just have the server's jvm.options persist instead.
Is this intended? If so, can we add an option to not delete the deployed jvm.options?

@five-iron If you mean the jvm.options in the build folder where the Liberty server got created gets deleted, then the answer is yes it is intended. The jvm.options should come from either the build.gradle configuration or a config file under source config folder. If you mean something else, please respond with more details.

Yeah I think you've captured what I'm saying. My setup right now doesn't use a build folder for creating a server, I'm pointing to a preinstalled location, so the path looks like this: C:\liberty_install_loc\wlp\usr\servers\defaultServer\jvm.options.
How hard would it be to prevent gradle from touching this file altogether (or at least when those jvmOption-related configs are not set, so as to not blow away the deployed file)?

@five-iron I see. So I can understand why you wouldn't want it deleted from a pre-installed Liberty, but we have nothing built into the code to prevent that from getting overwritten. The problem is our plugin will not know if that file was pre-existing or came from our plugin configuration on a previous run. Perhaps we can have some special logic if the user is pointing to a pre-installed Liberty, then leave the files as is unless plugin configuration is specified currently. I will discuss with my team.

@cherylking Do you know when this could get released / when the next regular deployment cycle might be?

@five-iron We typically wait for more fixes to go in, but it is not a problem for me to do a minor fix release if this is important for your forward development/work. Let me see if I can fit this in my schedule tomorrow or early next week.

Don't sweat it if you can't fit in, it's not a blocker for us just a minor annoyance.

Ok, thanks. I went ahead and create a snapshot if you want to give it a try. Are you familiar with how to pull in snapshots from Sonatype?

Hm no I am not familiar with that. I tried searching maven.org but can't find snapshots anywhere for liberty-gradle-plugin.

@five-iron The [readme](Liberty Tools helps you quickly a) has a small bit on using snapshots. The version would be 3.5.2-SNAPSHOT.

Thanks I was able to download the snapshot and verify that it works as intended.


Toogle labels option

Description
Added toggles to the SettingsMenu with the options to toggle the labels of Nodes and Edges.
Motivation and Context
This PR addresses:  #431
These changes will allow a user to toggle the labels and make it easier to hide potential sensitive data before taking a screenshot.
How Has This Been Tested?
Locally tested manually and using existing tests.
Screenshots (optional):

Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

[x] I have met the contributing prerequisites

Assigned myself to this PR
Added the appropriate labels
Associated an issue: https://github.com/SpecterOps/BloodHound/issues/672
Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing

[ ] I have ensured that related documentation is up-to-date

Open API docs
Code comments (GoDocs / JSDocs)

[x] I have followed proper test practices

Added/updated tests to cover my changes
All new and existing tests passed

Good morning, @Palt, and thank you for another contribution! I have some functional feedback on your PR before I ask our Engineering team to review the code.

The "Explore" portion of the product is one subset of BloodHound's functionality. As such, changes that impact only that portion of the application must live within the page and not at a global level. Our UX designer has provided the following mockup for the expected behavior of this function within the product within the graph context specifically (specifically the menu in the bottom-left corner):

The graph elements should maintain a static position regardless of whether labels are displayed or hidden; the view should not reset when toggling the label elements on and off.
We would like to include options to show or hide all labels quickly.

I appreciate the effort you've put into this, and I recognize this is a decent bit of rework on the changes you have already made. We do have some upcoming work planned for our team, where they can help take this across the line if you'd rather not rework!
On a related note, I would love to send you a swag package to show our gratitude for your contributions to the product. If you can send me an email at shinck (AT) specterops (DOT) io with your shipping address, I'll get that on its way!

Hi @StephenHinck, and thank you for great feedback!
I’m happy to be able to contribute when I can as this is an awesome tool that I use quite a lot.
Reading through what you’ve written it is as you say, this is not the way to implement it. I’ll blame it on my lack of understanding of the project and will continue digging a bit deeper into it to get a better grasp on it.
I’ll see what I can do to see if I can implement the needed changes to get it to a state where you can agree with it.
Regarding Swag: Thank you! I’ve sent you an email.

I've re-done the PR and would say that it now aligns with what you described @StephenHinck. It is now a smaller change and far less complex from previous solution.
Hope it is what you are looking for.

@StephenHinck are the updates from @Palt what you are looking for? If so we can start getting this code reviewed.

@StephenHinck are the updates from @Palt what you are looking for? If so we can start getting this code reviewed.

From a functional perspective, this matches our AC. The ticket is up at BED-4613 internally!

Hey @Palt! We have refactored how these GraphButtons work after the time you originally opened this PR. I've taken a moment to push updates to your branch. Would you kindly take a moment to review and validate the changes on your end?
@Holocraft could you please take a look as well?

Code looks good but when node is selected it doesn't hide the label for it. This doesn't happen for the selected edge.
@StephenHinck is this ok? ( considering they can just not select one for the screenshot ) or would we want to hide it as well when selected? ( could mean having to deselect and right panel resets but haven't really dug deep into it, the panel resetting doesn't happen for edge so maybe its possible )
I bring it up because initial motivation from contributor was to hide all sensitive info in case of screenshot.

Because of https://github.com/SpecterOps/BloodHound/issues/466, I think it makes sense to make it consistent that if you hide labels, they are all hidden.

It looks good and functions @elikmiller  however on top of what @specter-flq mentioned about nodes not hiding their labels, I noticed that if all labels are hidden, or just edge labels hidden, and you select a node, the edge-labels appears? See example below. I've not had the time to look in to why that is but it looks like edges re-render for some reason every time you are selecting a node?

@Palt you are correct - great catch. I pushed a new commit that should fix the issue. The problem was that we manually toggle edge labels on and off when a user drags a node. This behavior was overriding the settings you created.

I made an update that brings the UX of these new buttons inline with what we have implemented in BloodHound Enterprise. With that, I'll go ahead and merge this PR. Thank you for your contributions @Palt and we appreciate your patience!


Testing slack integration

Disregard, I am just spamming the channel for the sake of testing.

Disregard, I am just spamming the channel for the sake of testing.


clean up executable installation

The spack package for fabtsuite (https://github.com/mochi-hpc/mochi-spack-packages/tree/main/packages/fabtsuite) places executables in subdirectories of the install prefix that are difficult to find, because spack packages are intentionally installed to unique locations.
It would be simpler if the commands that users should run were placed in a bin subdirectory (either in the spack package, or maybe more straightforward to do generically in the main repository).
This would also be a good time to think about the names of the executables.  It might make sense to prepend a name to the commands that associates them clearly with this test suite ("fput" and "fget" are kind of generic).

@carns , we want to rename fput/fget. Here are a few candidates that we came up with:

fabtget / fabtput
ftget / ftput
fi_get / fi_put

The option 3. follows the convention of bin/ programs of libfabric [1].
Which option do you like most?
[1]  https://github.com/ofiwg/libfabric/blob/main/Makefile.am

Thanks @hyoklee .  I would vote against 3, because that seems problematic for it to sound like part of the upstream libfabric package.  The others are fine.  Thanks!

OK. I'll use 1 then.


Add discovery method for asset that can be discovered

The workflow would be:

Call new OPC UA method DiscoverAsset(providing a string in the format Protocol://IPAddress:Port) on the OPC UA server interface from the connectivity software. The connectivity software will discover the asset and auto-generate a WoT Thing Description file.
Read auto-generated WoT Thing Description file from OPC UA File interface on the connectivity software.
Add the OPC UA Companion Spec mapping information to the WoT Thing Description using an external tool like UA Cloud Viewer.
Write the modified WoT Thing Description back to the connectivity software (e.g. UA Edge Translator).

This has been added to the spec v1.1.


[external-dns] gandi provider present but no catalog entry ?

noticed there is a gandi provider in the external-dns repo, but no instructions on how to use it and no catalog entry seems to be present on try.rancher.com (used to test quickly if it wasn't a new feature).
Any idea how it should be used ?

Seems there is no catalog item indeed. Maybe @janeczku can tell something about the status of this provider.
If you want to use it, you can create the catalog item. The base is in another provider, i.e. https://github.com/rancher/community-catalog/blob/master/templates/dnsimple/2/docker-compose.yml
ROOT_DOMAIN, NAME_TEMPLATE and TTL are generic, you want to remove DNSIMPLE_* and add GANDI_APIKEY as described here: https://github.com/rancher/external-dns/blob/master/providers/gandi/gandi.go#L37
I can help you if you want to test it, let me know.

Thanks for adding this


Release mode opens a new window when the notification is clicked.

As title
Release mode opens a new window when the notification is clicked.

I think it might have something to do with WinToast.

@ollydixon Where you able to get this issue resolved ? this has been an issue with two other notifications libraries that I had tried and I am wondering if local_notifier has this issue resolved or or if a workaround is possible ?

@ollydixon that's great! can you share the plugin setting change once you have access to the code ? I have been stuck on this issue for a while and it would greatly help, Thanks.
Based on what change fixes it we can then also see if a proper fix can be integrated into this and other notification libraries.

@lijy91 is there any update on this ? In case its confirmed as an issue with WinToast then we would have to raise an issue there I guess.


How to completely uninstall?

Thank you for making this amazing tool. I would like to know how to uninstall F5-TTS and its dependencies. Thank you very much!

Maybe you could notice that there exists just a same issue #275 at first place before you start the issue?
Also
https://github.com/SWivid/F5-TTS/blob/e963929b8e6961a37f8321b788ca1969ad9ac4e0/README.md?plain=1#L103


Fixes typo in README.md

Oh crap, thanks for spotting that! :grinning:

@lucidstack :+1: I only noticed it because it tripped me up when I lazily copypasta'd it. Loving this plugin, thanks for making it!


Error in worker

in order to detect the browser, you need to take into account that the code can be executed in the worker and there is no window, but there is self
root = typeof window !== "undefined" ? window : null;

should be rewritten as
root = self instanceof Window || self instanceof ServiceWorkerGlobalScope || self instanceof Worker ? self : null;

Hey, if anyone else finds this issue like I did and is using cloudflare workers, here's what worked for me:
import {monotonicFactory} from 'ulid'
// or import {factory} from 'ulid'

const prng = () => {
  const buffer = new Uint8Array(1)
  crypto.getRandomValues(buffer)
  return buffer[0] / 0xff
}
export const ulid = monotonicFactory(prng) // or factory(prng)

This will bypass the code that checks for browser crypto and allow you to set your own. The PRNG function is the same as used internally with a different global reference.
I think this package may be abandoned, I'm working on potentially forking it.

Friendly reminder that we now have worker support over on ulidx, which is maintained unlike this library.


Adds byte to the type of cases in the JsonFormatter

Rapidash discovered an issue where TEXT and other blob columns where not being sent over to SQS, which makes a lot of the work that we are doing towards Subscribed Events kind of hard since a lot of our fields within our app are TEXT fields and tracking their changes seems relevant.
Through some digging, we figured out the problem is originated not in Changestream but within one of our dependencies, mysql-binlog-connector, as it deserializes BLOB columns into byte arrays.
The way the JsonFormatterActor checks for the type of the value, made it so that it didn't recognize the byte array and would just ignore that column as an unsupported value.
The changes done adds a case for byte arrays so that we can support it and forces it's casting into a string.

Notes
Personally, I have some worries regarding how this could impact our performance since now we would be doing ALL BLOBS, which could put us in a position where a huge BLOB gets transformed and sent as is.
The other concern is this case would capture all byte arrays, not necessarily sure how that could impact, or break, other BLOB fields

This is an interesting problem case ...  what if we just synthesized a *textfld*_dirty field that was pre-computed before getting sent over SNS?
What sort of versioning support do we have on record-types that are audited?
If we allowed users to revert-to-version: X or diff between version X Y, we would only need the track who changed what field, and not necessarily what was changed in the given field value.

Digging into the tests a bit on this I see there's a blob_that_should_be_ignored field on the fixtures table (src/test/scala/changestream/helpers/Fixtures.scala), implying that the omission was intentional. I do share your performance concerns @cpalafox. I also wonder if there's a size constraint on what we send to SQS, and what the implications would be of piping BLOBs through that.
At the moment we are only running changestream as a single process (well, two, but each for a distinct use case), and it has started to sweat a bit when we put it through really heavy loads, so I'd be wary of doing anything that slows it down. I think we need to put some work into this and see if we can get it into kubernetes and make it scale to multiple processes. My suspicion is that's doable, but I haven't seen the way clear through the woods completely. LMK if you guys would like to collab on that.
I was looking into getting it into kube last week, got a little stuck on some weirdness with duplicate actors being created, but once I figure that out I think it'll be GTG.
I also saw there's are some benchmark tests in the specs for changestream, so maybe adding one of those would be a way to quantify potential blob impact?

This is an interesting problem case ... what if we just synthesized a *textfld*_dirty field that was pre-computed before getting sent over SNS?
What sort of versioning support do we have on record-types that are audited?
If we allowed users to revert-to-version: X or diff between version X Y, we would only need the track who changed what field, and not necessarily what was changed in the given field value.

We do not plan on allowing people to revert to a previous version, but the idea is that we can tell people what was the field before and after the change. And maybe there's exceptions to consider. Are notes for a line_item relevant if they changed? Maybe not?, but could a Workspace.description change be relevant for Audit Logs purposes? probably?
I don't know if just having a field_dirty would be useful in all scenarios?

Digging into the tests a bit on this I see there's a blob_that_should_be_ignored field on the fixtures table (src/test/scala/changestream/helpers/Fixtures.scala), implying that the omission was intentional. I do share your performance concerns @cpalafox. I also wonder if there's a size constraint on what we send to SQS, and what the implications would be of piping BLOBs through that.
At the moment we are only running changestream as a single process (well, two, but each for a distinct use case), and it has started to sweat a bit when we put it through really heavy loads, so I'd be wary of doing anything that slows it down. I think we need to put some work into this and see if we can get it into kubernetes and make it scale to multiple processes. My suspicion is that's doable, but I haven't seen the way clear through the woods completely. LMK if you guys would like to collab on that.
I was looking into getting it into kube last week, got a little stuck on some weirdness with duplicate actors being created, but once I figure that out I think it'll be GTG.
I also saw there's are some benchmark tests in the specs for changestream, so maybe adding one of those would be a way to quantify potential blob impact?

@jacksenechal I'd be down to collab on getting it into kubes, as it would also allow us to run changestream on team sandboxes rather than depending on staging01 deploys. I don't know how much effort it'd take or how to do it, but definitely available to help out with the right guidance + pairing.
Good call on the benchmark test, this is definitely something we could do.

@cpalafox this PR is ready to go for getting changestream onto team sandboxes. I've got it running now in the master sandbox, with the values specified in charts/changestream/values-dev-example.yaml there. I'm getting ready to roll the chart out to staging and prod.
Note that I haven't used the parent-child chart pattern with this one. It's just intended to be released manually via helm specifying the values file to use with -f. Feel free to make a copy of values-dev-example.yaml for your team and check that in if you want to keep it around.

@jacksenechal appreciate the update on this one, this will allow us to make some progress and do some debugging!

We need to confirm what version of the AWS SDK we have linked in the build... It looks like before AWS SDK 1.12 SNS/SQS messages are limited in size to 256kb ... there is an additional set of libraries to add extended support for upto 2GB
https://docs.aws.amazon.com/sns/latest/dg/large-message-payloads.html
https://github.com/awslabs/amazon-sns-java-extended-client-lib/
https://github.com/dwhjames/aws-wrap/issues/58
Is it possible for the blobs in our database to be bigger than 256kb?

I think I could've explained better what's happening with the latest changes:

Take BLOB byte array, no matter how big it is
Compresses it using ZLib
3- Encodes it with Base64
4- Sends message with blob field as a string

So, for example:
Scenario A: A very long text
  blob = "VERY LONG" * 2000

---
string size: 18000
raw byte array size: 18000
compressed byte array size: 73
Encoded string size: 100

Scenario B: Decently sized long text
blob = "VERY LONG" * 100

----
string size: 900
raw byte array size: 900
compressed byte array size: 26
Encoded string size: 36

Scenario C: A Short description
blob = "dinner with client x, they accepted to renew our things. Good Soup"

-----
string size: 66
raw byte array size: 66
compressed byte array size: 70
Encoded string size: 96

Scenario D: Emoji galore
blob =  "🙄🙄🙄🙄🙄🙄🙄🙄🙄🙄🙄" * 50

string size: 1100
raw byte array size: 2200
compressed byte array size: 28
Encoded string size: 40

Our current message size is about 2.5KB. With this compression + encoding, I'm not worried about performance impact or getting close to the 250KB limit

Dicussions and different efforts landed on going with truncation at the moment, those changes can be found on : https://github.com/mavenlink/changestream/pull/95


Include services in solr search

https://github.com/ufal/lindat-repository/issues/27

@vidiecan umm, what?

@kosarko search box in https://lindat.mff.cuni.cz/en but I think we can close this one


docs: Fix broken links and remove invalid content on v0.38 docs

closes: #1375
PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[X] Updated relevant documentation (docs/ or spec/) and code comments

This is more a question: what is the effect of setting this? The document was already rendered without this change.

It sets the relative order of pages in this folder for displaying on the left side menu.

@cason this should be almost ready to go. Except of a couple of links in the QA section that I need to double check with @lasarojc I think most of the issues were addressed. If you run the check for broken links, there are a few reported but those should be false positives, they are indeed valid links.


Support binary file operations

The ARM Semihosting specification differentiates between binary and text streams. These enums are defined already, e.g. https://github.com/taiki-e/semihosting/blob/6ebb824221291afda0d64d0afe2e430b6662fccf/src/sys/arm_compat/mod.rs#L80
There doesn't seem to be a way to open a stream as "binary".
Rust opens all files as binary, and as such doesn't provide an option to open files in text mode. The gdb protocol doesn't differentiate between "binary" and "text", but other debuggers such as J-Link may make the distinction.
There are two options I can think of for semihosting to support binary mode:

Add an additional OpenOption to specify whether a file is opened in binary mode or not
Open all files in "binary" mode, which is the same behaviour as std::fs::File in Rust

Thanks for the report! I think we should always use that as well as std::fs::File (I'm not sure why I did not use binary mode...).

Fixed in v0.1.13.


feat: allow inspecting inner fields of nw.Struct

in https://github.com/narwhals-dev/narwhals/pull/1104 we allow inspecting the inner dtypes of nw.List
We should do the same for nw.Struct

this would be awesome, thanks @DeaMariaLeon !

Still working on this.


DbtDag fails even though Airflow connection and dbt works fine

I have a simple dbt model that creates a view in a PostgreSQL db that's hosted on docker. The dag renders but it fails without any reason. No error logs, nothing. I checked if the dbt project I am trying to schedule works on it's own and it does. The connection in Airflow is also fine, but regardless the task fails.
Below are the results of reach test. I don't really know how to debug this.
The Dag
DBT_PROFILE = ProfileConfig(
    profile_name = "dbt_profile",
    target_name = "dev",
    profile_mapping = PostgresUserPasswordProfileMapping(
        conn_id = "demo_dbt_postgres",
        profile_args = {
		    "dbname": "data_db",
		    "schema" : "public"
        }
    )
)

dbt_dag = DbtDag(
    dag_id = "dbt_dag",
    project_config = ProjectConfig(DBT_PROJECT_PATH),
    profile_config = DBT_PROFILE,
    execution_config = DBT_EXE_CONFIG,
    start_date = datetime(2024, 11, 3),
    schedule = "@daily",
    catchup=False
)

Airflow connection test results
astro@83d49cd73fbd:/usr/local/airflow$ airflow connections test demo_dbt_postgres
[2024-11-03T17:22:05.359+0000] {configuration.py:1058} WARNING - section/key [openlineage/namespace] not found in config
[2024-11-03T17:22:06.801+0000] {plugin.py:32} WARNING - Astro managed secrets backend is disabled
Retrieving connection: 'demo_dbt_postgres'

Testing...

Connection success!

dbt debug test
...

17:27:17  Connection:
17:27:17    host: host.docker.internal
17:27:17    port: 5433
17:27:17    user: postgres
17:27:17    database: data_db
17:27:17    schema: public
17:27:17    connect_timeout: 10
17:27:17    role: None
17:27:17    search_path: None
17:27:17    keepalives_idle: 0
17:27:17    sslmode: None
17:27:17    sslcert: None
17:27:17    sslkey: None
17:27:17    sslrootcert: None
17:27:17    application_name: dbt
17:27:17    retries: 1
17:27:17  Registered adapter: postgres=1.8.2
17:27:18    Connection test: [OK connection ok]

17:27:18  All checks passed!

Dag failed

hi @RsFaruque were you able to resolve the issue? Could you please share what was the change needed?


Increase timeout on restart in JS/elixir tests to 30s

Overview
Currently, when a JS or Elixir test restarts a node, it waits 15 or 10 seconds respectively for it to come back.
CI systems have shown this is not enough. Most recent failure is here:
https://builds.apache.org/blue/organizations/jenkins/CouchDB/detail/master/448/pipeline
but it's been happening semi-regularly since we reworked the restart code.
This PR bumps the timeout to 30s in both places.

thanks @rnewson


Rewording of Export-VM Description

I would suggest a rewrite for this paragraph:
The Export-VM cmdlet exports a virtual machine to disk. This cmdlet creates a folder at a specified location having three subfolders: Snapshots, Virtual Hard Disks, and Virtual Machines. The Snapshots and Virtual Hard Disks folders contain the snapshots of and virtual hard disks of the specified virtual machine respectively. The Virtual Machines folder contains the configuration XML of the specified virtual machine.
Changing the third sentence to read more easily:
Each of these folders contains the associated files.  The Snapshots folder contains the associated Snapshots, and the Virtual Hard Disk folder contains the specified virtual machine's virtual disks.

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

ID: 406f9b72-70b6-a452-e079-e1173fca02d4
Version Independent ID: ba568860-97d9-41a7-dd41-9898665b1948
Content: Export-VM (Hyper-V)
Content Source: docset/winserver2022-ps/hyper-v/Export-VM.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

@jonjunell Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/windows-powershell-docs/pull/3527
Once the author approves, the changes will be live.
Thanks
Sri


Make Gitlab's container-registry to own its cert and key files

Solves #354275
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@phozzy I advise renaming this PR to include gitlab because this appears to be for gitlab.

It seems the whole setup in the current state is messed up. It's easier to fix its documentation.


implement network-tap show available nics

using a multi-stage build image to get nics inside of the container, so this relies on this issue being solved on Docker Hub first: https://github.com/docker/hub-feedback/issues/1039


Ingesters get network errors when writing dynamo data during shutdown

time="2016-10-10T13:17:29Z" level=error msg="Failed to flush chunks for series: RequestError: send request failed\ncaused by: Put https://weaveworks-prod-chunks.s3.amazonaws.com/2/17142991289174007630%3A1476104540889%3A1476105230889: dial tcp: lookup weaveworks-prod-chunks.s3.amazonaws.com on <IP_ADDRESS>:53: dial udp <IP_ADDRESS>:53: connect: network is unreachable" source="cortex.go:440"

This might be a cause of data loss.

Waiting on loggly to come back before we close this.

K8s 1.4 upgrade seems to have fixed this (no occurrences in the log)


Rename Dema Therese to Surya Hardiansyah

I renamed the author's name inside ps3_pair.qmd into my name, Surya Hardiansyah

Seems fine


Bugfix/unique test search term report

PR Overview
This PR will address the following Issue/Feature:
https://github.com/fivetran/dbt_apple_search_ads/issues/16
This PR will result in the following new package version:

v0.2.2
Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:
Adds new fields to the unique test for the apple_search_ads__search_term_report model:
        - keyword_id
        - ad_group_id
        - campaign_id
        - organization_id
        - match_type

In addition to the existing search_term_text and date_day
PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:

[x] dbt compile
[x] dbt run –full-refresh
[x] dbt run
[x] dbt test
[ n/a] dbt run –vars (if applicable)

Before marking this PR as "ready for review" the following have been applied:

[x] The appropriate issue has been linked and tagged
[x] You are assigned to the corresponding issue and this PR
[x] BuildKite integration tests are passing

Detailed Validation
Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review":

[x] You have validated these changes and assure this PR will address the respective Issue/Feature.
[x] You are reasonably confident these changes will not impact any other components of this package or any dependent packages.
[x] You have provided details below around the validation steps performed to gain confidence in these changes.

dbt test passes. Again our internal data used for development is light and therefore didn't catch this issue before, but the customer who contributed this PR has confirmed that the fields they added now allows the unique test to pass.
Standard Updates
Please acknowledge that your PR contains the following standard updates:

Package versioning has been appropriately indexed in the following locations:

[x] indexed within dbt_project.yml
[x] indexed within integration_tests/dbt_project.yml

[x] CHANGELOG has individual entries for each respective change in this PR

[ n/a] README updates have been applied (if applicable)

[n/a ] DECISIONLOG updates have been updated (if applicable)
[n/a ] Appropriate yml documentation has been added (if applicable)

dbt Docs
Please acknowledge that after the above were all completed the below were applied to your branch:

[] docs were regenerated (unless this PR does not include any code or yml updates)

If you had to summarize this PR in an emoji, which would it be?

:dancer:

Thank you @yuna-tang for contributing this PR! We will be reviewing this and including this update to the package.
Additional changes:

upgraded integration test version
updated CHANGELOG
Also added match_type and organization_id to the unique test to mirror what we did in the ad reporting rollup package.

Updated PR and will wait to regen docs when everything is finalized.
@fivetran-joemarkiewicz ready for re-review

@fivetran-reneeli , I noticed the release version for dbt_apple_search_ads will be v0.2.1. Do you know which version of fivetran/ad_reporting will include this change?

Hi @yuna-tang , it should automatically pull this latest version given the parameters  currently are [">=0.2.0", "<0.3.0"]!
Note that we moved this to v0.2.2 because I made an accidental release on v0.2.1, but same effect should apply! This has been merged to main so will be closing this out


⚠ Samsung has degraded performance

In a54814d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2751 ms

Resolved: Samsung performance has improved in 179745d.


changes in order to keep "custom keywords" contained (together)

Pull Request Test Coverage Report for Build 362

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 53.296%

Files with Coverage Reduction
New Missed Lines
%

goodman_pipeline/core/tests/test_check_version.py
1
85.71%

goodman_pipeline/core/check_version.py
4
95.83%

Totals

Change from base Build 360:
-0.1%

Covered Lines:
2062

Relevant Lines:
3525

💛  - Coveralls


WFLY-19782 Upgrade SmallRye Fault Tolerance to 6.4.1 + WFLY-19785 test case

Resolves
https://issues.redhat.com/browse/WFLY-19782
https://issues.redhat.com/browse/WFLY-19747
https://issues.redhat.com/browse/WFLY-19785

Thanks @rhusar


'options.minContainerHeight' test fails

Hi!
I tried to build Cropper with grunt, and 'options.minContainerHeight' has failed. I checked properties of container and figured, that in
assert.ok(Math.round(container.height) === minContainerHeight);

container.height equals to 221.
I think checking as 'greater-or-equal' will be more correct. Some thing like this (I'm not an expert of QUnit):
assert.ok(Math.round(container.height) >= minContainerHeight);

The same thing in 'options.minContainerWidth' test.
Thank you for your attention.

It's my fault. Thanks!

Oops, I was wrong in my suggestion.. Sorry. I figured the reason of test only now xD
Good work!


Turn 3 type flags into properties

This is needed for object spread and rest, which will each need a type
flag.
There are 4-5 other likely targets for removal, and I may remove those
later.

@ahejlsberg and @mhegazy can you take a look at this?

I put Instantiated back in and removed ObjectLiteral instead. This keeps 3 flags open for use with Spread, Rest and Subset.

Oops, I didn't see your new comments, @ahejlsberg. I put ObjectLiteral back as well.

Why didn't you just introduce a new flags2 ?


.NET Core support

It will support .Net Core ?

If someone need - sure. There was one fork with .NET Core support with very little changes. I can try update it to master branch.

https://github.com/RevenantX/LiteNetLib/commit/7643b7f2c127987de2ac94bab61f9ae97eb71434


Instance trend for the current month vs. the last month

User Story
As user, I want to see my current number of instances and a trend of instances for the current month in comparison to the last month so that understand my instance usage month to month.
Impacts
API, UI, Docs
Role
User
Assumptions

The user can only view instance data for accounts that are visible to them via the AWS organization.

UI Details
UI will a value for the current number of instances and provide a chart/table with the trend of instances for the current month vs. the last month.
Acceptance Criteria

[ ] The user is able to see the instances data but only for the accounts visible via AWS organization for the current month vs. the last month.

@dmiller9911 This was reviewed during the refinement meeting. If you are ok with the definition of the story please move it into the Backlog in Zenhub.

I believe this has been implemented per @nlcwong 's mocks. Can this be closed?


TypeError: Cannot read properties of undefined (reading 'version')

I'm getting this error when trying to use in my Vue3 project. Anybody encountered this before?

TypeError: Cannot read properties of undefined (reading 'version')
at eval (index.mjs?4c81:15)
at<EMAIL_ADDRESS>(home.js:59)
at webpack_require (app.js:854)
at fn (app.js:151)
at eval (index.js?af75:1)
at<EMAIL_ADDRESS>(home.js:35)
at webpack_require (app.js:854)
at fn (app.js:151)
at eval (cjs.js?!./node_modules/babel-loader/lib/index.js!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader-v16/dist/index.js?!./src/views/Game.vue?vue&type=script&lang=js:3)
at Module../node_modules/cache-loader/dist/cjs.js?!./node_modules/babel-loader/lib/index.js!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader-v16/dist/index.js?!./src/views/Game.vue?vue&type=script&lang=js (home.js:131)

Basically, what I'm trying to achieve is to play the audio when user clicks 'yes' on the <Modal /> component which emits startGame
Here is the code:
<template>
  <Modal @startGame="startGame" />
</template>

<script>
import { useSound } from '@vueuse/sound'
import bgAudio from<EMAIL_ADDRESS>
export default {
  setup() {
      const { play } = useSound(bgAudio)

     const startGame = () => {
        /* other codes here */
        play()
    }
    return {
      startGame
    }
  }
}
</script>

Having the same issue. I think it is caused by some misconfiguration in vue-demi. I am using Vue ^3.0.0, and having similar error. In Eslint I am getting:
warning  in<EMAIL_ADDRESS>"export 'default' (imported as 'Vue') was not found in 'vue'
warning  in<EMAIL_ADDRESS>"export 'default' (reexported as 'Vue') was not found in 'vue'

Hey, could you let me know if v2 has fixed this issue for you?


[Engine][Task] Move SourceSplitEnumeratorTask to worker node

Purpose of this pull request
Move SourceSplitEnumeratorTask to worker node

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

LGTM @EricJoy2048


New fallback sink on failed exports

#155 - Adds a new file system fallback using the Serilog.Sinks.File sink

Converted to draft as this can be extended to be much more flexible.

Thanks for all the additional feedback and exploration!
Adding a direct dependency on a fallback sink (such as File) is probably a non-starter at this point because of past experiences maintaining and supporting the same concept (e.g. in the Seq and Elastic sinks, among others).
If we move forward with support for fallback sinks via the Serilog failure listener feature, the implementation will be unlikely to bear any similarity to this, because the functionality will be cross-cutting and not per-target-sink. It might be best to close this PR and keep discussion on the linked ticket.
Thanks again!


.netcore support

Any support planned for .netcore ?

Good idea. Thanks for the suggestion.
It was quite easy to get the library compiling for .netcore. I made a PR (#5) with the work in progress.
However, I am ignorant of how .netcore .dll's work and interact with the non-.netcore world, so I documented (on #5) a number of open questions I need to research before merging/releasing any of these changes.

Basic .netcore support is now implemented in the latest release:
https://www.nuget.org/packages/MlkPwgen/0.3.0
I've only tested the .netcore support lightly, so if anyone runs into any issues, please let me know by submitting a new issue.


MSQ: Use task context flag useConcurrentLocks to determine task lock type

Changes

Determine task lock type in MSQControllerTask using both the task context and the query context
Add method taskLockType() to ControllerContext so that ControllerImpl
may use the task lock type that is determined in MsqControllerTask
Clean up MSQControllerTaskTest and add test for lock type

Thanks for the quick review, @cryptoe !