[CLI]: wandb.log(..., commit = False) doesnt work and continues to log metrics to the server

Describe the bug

Im trying to use wandb.log(..., commit = False) because I do a lot of spot instance training, so when I reload the checkpoint, I need to make sure that my wandb resumes at the right step. Because I don't save checkpoints at every step, the flag only becomes True (e.g. wandb.log(..., commit = True) ) when I save the checkpoint. However, somehow wandb.log(..., commit = False) just doesnt work and continue to log to the server. This is my current implementation:

# I oversimplified my code and omitted details just for illustrative purposes
while iter_num < TRAIN_CONFIG.TRAIN_STEPS:
    if iterm_num % est_interval == 0:
        ...
        wandb.log(
            {
                "est_train_loss": train_loss,
                "est_val_loss": val_loss,
                "est_lr": lr,
            },
            step = iter_num,
            commit = True,
        )
        ...
        torch.save(
            checkpoint,
            (
                f"transformer_dropout/model_checkpoints/ckpt.pt"
                if args.is_local
                else os.path.join(checkpoint_path, "ckpt.pt")
            ),
        )
    ...
    logits, loss = model(X,Y)
    ...

    wandb.log(
        {
            "loss": running_loss,
            "dropout_entropy": running_entropy,
            "dropout_l1_norm": running_l1_norm,
            "time": float(f"{dt*1000:.2f}"),
        },
        step = iter_num,
        commit = False
    )
    ...

Im also open to other ways of ensuring continuity in my wandb run when I resume a training run from a checkpoint
Additional Files
No response
Environment
WandB version: 0.16.3
OS: 14.3.1 (23D60)
Python version: 3.10.1
Versions of relevant libraries:
Additional Context
No response

Hi @yiphei! Thank you for writing in!
To confirm, when you are adding commit = False, you are interested in seeing the metrics not get uploaded to the wandb.ai, but it gets recorded regardless of you having the flag?

@ArtsiomWB correct! commit=False is just a no-op for me

@ArtsiomWB gentle bump

@kptkin, @ArtsiomWB gentle bump

@kptkin @ArtsiomWB bump

@kptkin @ArtsiomWB bump

@mvmacfarlane could you please provide more details on your issue?
i provided the explanation of the current behavior with commit=False as well as an option to use define_metric which will avoid the need to pass a step to wandb completely and allow you to define your own custom x-axis.
if none of these help with your case a more detailed explanation of your specific use-case would be helpful to help us figure out how to best assist you.

Due to inactivity we are closing this request, please comment to reopen.


Unbound selective prefetch

It would be a nice addition to be able to use prefetch but narrow it down to only a specific set of zones. Sometimes we have certain zones that we want to serve always from cache but requests are not that frequent, so we would want to prefetch these zones but not everything else that is in the cache.

It seems prefetch did not work as I expected, we still need requests for that record.

It seems prefetch did not work as I expected, we still need requests for that record.


Using NodeJs instead of Go, for a full JS stack?

Have you looked at or considered using Puppeteer as a backend to serve the different HTML5 games?

I don't plan to do this. If anything, I'd rather rewrite the APIs in Python since most ML is done in Python anyway.


WIP: Simplify hosting configuration with some extension methods

Added WaitForShutdown method to IWebHost
Added Run and RunApplication to IWebHostBuilder to Build and
Start the application in a single method.

After a discussion about what could be done to further simplify some of the startup logic with the WebHostBuilder a few things came up. These were some of the methods that we could add to make the startup logic more approachable while not completely diverging from the original API.
Further simplification requires picking kestrel to to be the default server but that won't happen as part of this PR and requires further discussion.
/cc @glennc @Tratcher @DamianEdwards

Will revisit.

We're looking at doing something in the meta-package: aspnet/MetaPackages#24


Analyze higher education enrollment data to verify that the cut point is in the right place

The results of this analysis could also help us properly describe the characteristics of the census tracts that are getting excluded as a result of this threshold so that users can better understand the effects.

This is tricky to walk through because super-college-heavy college towns have a lot of students (like >80%), but the vast majority of census tracts (even those with college students) do not have many. 20% is above the 90th percentile for all census tracts, and for all census tracts with >2% college students.
However, a few things here give me pause:

We do not adjust low income to account for college students, which leads to some weirdness: The 200FPL share for our poverty indicator is around 39% (smallest share I found with low income bool). If a tract has 20% college students but is in the 97th percentile for low income (share: 77%), even if every single student were contributing to the 200FPL statistic, the remaining residents would still be living at higher than 65th percentile for low income. This is very different than if we had, say, 90% college students -- then, there wouldn't be many residents likely living below 200FPL for reasons other than temporarily being a student! --> This makes me think we might consider adjusting income in some way to account for college students.
The cutpoint might be too low: There are only about 4,000 colleges in the US. If we assume that about half of them have a high rate of resident students (e.g., not a commuter college -- and this would far exceed the schools USNW ranks, so this doesn't seem crazy), cutting the variable at a generally high cut-point might actually be low for what we are trying to measure. The 2,000th tract (assuming 1:1 tract:college), ranked by pct of college students, has 25% college students. I'd argue that's a conservative value to have as well! That's a lot of assumptions, but I think that we are less concerned about including some colleges than others (e.g., I think we probably don't want to include UC Berkeley, but we might be open to including a tract near another institution that has 20% college students and is in the 99th percentile for low income).

There are a lot of nuances here, and I think some of it also depends on how we update the higher ed threshold more generally. But I think potentially adjusting for income (e.g., there are at least 1,000 residents who are not college students AND those residents are at or above 65th percentile for 200FPL) could be fruitful, or moving the cutoff value.

Moved to review -- will discuss in meeting


chore(workspace): Update _deploy-deno.yml

Can we make the Deno deploy a manual action too? Could be useful to sometimes skip the CI and just deploy to get a shareable URL.

Speaking of which I forgot to [skip ci] ughhh.

Can we make the Deno deploy a manual action too? Could be useful to sometimes skip the CI and just deploy to get a shareable URL.

in what scenarios? maybe it's a deeper issue where we want to skip all checks on draft PRs? is that the case?

Can we make the Deno deploy a manual action too? Could be useful to sometimes skip the CI and just deploy to get a shareable URL.

in what scenarios? maybe it's a deeper issue where we want to skip all checks on draft PRs? is that the case?

we can force skip all workflow actions using [skip ci], but the Deno deployment is exactly a check per se.
the only use case I had was being able to share a reproducer URL, so admittedly not a very frequent occurrence indeed.


Add Transformers.js update + demo

Hi there! 🤗 This PR adds a bullet point for SlimSAM + transformers.js support (which was added today), allowing these models to be run directly in a browser, using a few lines of JS code. Demo video:
https://github.com/czg1225/SlimSAM/assets/26504141/1ad1840d-0304-49a1-a0fe-6f07070073c7
Demo link + source code: https://huggingface.co/spaces/Xenova/segment-anything-web
Release notes: https://github.com/xenova/transformers.js/releases/tag/2.14.0

Transformers.js uses a very similar API as the python library, as can be seen with the following example code:
Example: Perform mask generation w/ Xenova/slimsam-77-uniform.
import { SamModel, AutoProcessor, RawImage } from '@xenova/transformers';

const model = await SamModel.from_pretrained('Xenova/slimsam-77-uniform');
const processor = await AutoProcessor.from_pretrained('Xenova/slimsam-77-uniform');

const img_url = 'https://huggingface.co/datasets/Xenova/transformers.js-docs/resolve/main/corgi.jpg';
const raw_image = await RawImage.read(img_url);
const input_points = [[[340, 250]]] // 2D localization of a window

const inputs = await processor(raw_image, input_points);
const outputs = await model(inputs);

const masks = await processor.post_process_masks(outputs.pred_masks, inputs.original_sizes, inputs.reshaped_input_sizes);
console.log(masks);
// [
//   Tensor {
//     dims: [ 1, 3, 410, 614 ],
//     type: 'bool',
//     data: Uint8Array(755220) [ ... ],
//     size: 755220
//   }
// ]

Visualization of masks:

We have add the demo link in our readme, thanks!

Amazing! Will close this PR then 😇


Arguments passed in launch terminal

We want to implement in the CMake Tools VSCode extension arguments passing to the launch terminal. We realized that there are numerous situations of quoting and escaping that we would need to handle in order for the arguments to be passed properly in all the available types of terminals.
We were wondering if this is already handled centrally by VSCode, so that all the other extensions wouldn't have to worry all about the same thing. Is there some API we that can act as an intermediary between an array of arguments and any type of terminal supported in VSCode?
Or how do you suggest we best approach this on our side if there is nothing available from your side?

For sendText you can send whatever you want, for args quoting is done for you if you pass in a string[], some cases might not work there which is why we let you use string here if you want to pass in a pre-quoted args string:
https://github.com/microsoft/vscode/blob/84ac04e8155d473868f5301eb979ed0dd527e116/src/vs/vscode.d.ts#L8044-L8048
The built-in capability is only provided for args as string[], not for args as string or sendText.


How to compile a contract to the bytecode?

I'm experimenting with: https://www.ethereum.org/token
Initially I used default Ethereum client, as suggested in the article.
Here is my code: https://gist.github.com/stefek99/e73e3b6a381ac08bd1fe3f47e8d175a5
(just hardcoded some values)
Contract creation: https://etherscan.io/address/0x3c27a55b7fffa58cde786db5659a67b67ddd098b#code
Token: https://etherscan.io/token/0x3c27a55b7fffa58cde786db5659a67b67ddd098b
The issue I'm having is that I cannot verify the contract, I informed Etherscan guys: https://twitter.com/stefek99/status/871175876779675648
Instead I'm trying to follow strategy here: https://ethereum.stackexchange.com/a/3257/2524

If getting verified on EtherScan is important to you, then write your Solidity code where ever but compile it using Browser Solidity here: http://chriseth.github.io/browser-solidity/

Then use that bytecode to deploy. I have not tested it but the code it spits out for the sample token contract matches what EtherScan spits out. I assume that EtherScan is using that same browser based compiler to check and verify contracts.

Here is the error I'm getting:

So in a way I've asked a convoluted question - https://meta.stackexchange.com/questions/66377/what-is-the-xy-problem
But maybe you'll know - how to compile it to the bytecode?

you have this error because no transaction origin has been selected.
you are using the injected provider. (Metamask or Mist)?

Metamask installed as Chrome extension, running in the background.
Mist install on my computer, at the time of taking screenshot not running.
Will try again with selected transaction origin.

i close. please  reopen if necessary


Chain should support RFID Data

This is necessary to fully port DoppelLab over to Chain.
There are some interesting design choices to figure out here that will require some discussion.

This was implemented by @kkleidal. Awesome!


Porcelain client

Implement the porcelain client.

[x] Clean up issues with project before refactoring (e.g., duplicate files, missing registry.yaml, etc.).
[x] Rename client to dm to make it a brand identifiable executable.
[x] Finish implementations of #21 and #23.
[x] Update README.md.

cc @vaikas-google, @bmelville

I know it's usually in bad taste to self merge, but this PR is needed to support the a side-by-side comparison of dm and helm on the Google group for sig-config.

Fixes #21


Improved Footer

Issue that this pull request solves:  Improves the footer component.
Closes: #125
Proposed changes: Updated the icons, added Newsletter column and made the footer more attractive .
Brief description of what is fixed or changed: Made the footer component more attractive.
Types of changes : Representation improvements
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality) Newsletter added
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots

Other information

Yeah sure. I'll get back to you after making the changes.

Hey @Kajol-Kumari , I have made the required changes. Please have a look and let me know if any other changes are required.

Hey @Kajol-Kumari , kindly review the changes and let me know if any further changes required.

@Kajol-Kumari , I've fixed this issue for both mobile and tab view. Please let me know if any further changes are required.

Hey @Swastik1710 please resolve the conflict so that I can merge this :)

Done.

Done.

Seems like, something got messed up as i am getting blank space at smaller screen

All done. Please have a look.

After I resolved the conflicts , when I tried to do git push it showed an error telling me to do Git pull. And after doing git pull when I did git push these two errors showed up:-
Module not found: Cannot resolve 'styled-component'
Module not found: Cannot resolve 'semantic-ui-css'
So I did
npm i semantic-ui-react semantic-ui-css
npm i styled-components
which solved the errors and this only caused 4k+ lines of changes.
So what do you want me to do now ?

@Swastik1710 do this

Copy the relevant codes somewhere as I can see you have made the PR using your main branch
Reset your head with upstream head

git remote add upstream https://github.com/HITK-TECH-Community/Community-Website.git
git fetch upstream
git checkout main
git reset --hard upstream/main
git push origin main --force

This will make your main branch even with the main repository.
After this, make a new branch (you can google it, if you don't know) and then put your code in there, push the change sand then make a new PR.


Request completionItem/resolve failed.

Today suddenly appeared, what is this
https://user-images.githubusercontent.com/59215561/195031446-bddf901a-7885-4f87-996b-81d80969f0fa.mp4

Message: Request completionItem/resolve failed unexpectedly without providing any details.

I tried but it didn't reproduce the problem.
This is what happens when I import "some" components (don't know what the difference)
This information comes from volar, maybe it's related to this? (did not exist before)

Could you try can this problem reproduce in https://github.com/johnsoncodehk/volar-starter?

Could you try can this problem reproduce in https://github.com/johnsoncodehk/volar-starter?

I tried but it didn't reproduce the problem.
This is what happens when I import "some" components (don't know what the difference)
This information comes from volar, maybe it's related to this? (did not exist before)

Could you try can this problem reproduce in https://github.com/johnsoncodehk/volar-starter?

I tried but it didn't reproduce the problem.
This is what happens when I import "some" components (don't know what the difference)
This information comes from volar, maybe it's related to this? (did not exist before)

I'm having the same issue since a few days and the intellisense stopped working for imports.
I downgraded the @types/node package since that was on the known issues but it doesn't fix it.
Here I had suggestions for imports alongside their packages

Now I have to manually specify the source before I can get any suggestion which is pretty annoying

Same issue since the update on 10/14. The component gets imported properly but this error is annoying. Custom components color scheme changed as well, which I don't see mentioned anywhere on the changelog.

@Mondotosz This is different problem, you can open a new issue for it.
@vhud, @1059845833 I can't reproduce this problem, could you provide repro case?

@Mondotosz This is different problem, you can open a new issue for it.
@vhud, @1059845833 I can't reproduce this problem, could you provide repro case?
My latest project is in my repository (vue3-music), I have tested other projects (including vite and vue-cli) without reproducing the problem.
I re-clone this project and found no problem importing modules, the only difference with other projects is that the bottom tag will have a "1 known issue"(no response after clicking) message from volar , but my original project still have this problem.

@johnsoncodehk
Have you tried creating a small vue cli project with the prior version of the plugin, then upgrading the plugin and attempting to create a new component? I'll try to nail down solid reproduction steps when I have more time, but this is something to try in the meantime.


Use existing zip, or zip up provided dir

https://github.com/sindresorhus/refined-twitter/pull/35#discussion_r64515684

Does it really need an option? If you point at a directory, zip it, if it's a zip file use it.

Makes sense - we'll go with that.

What do you think makes sense for the option that gets passed as - maybe --target (-t alias)?

--target or --source

Getting close on this. Just fighting a bug right now where Yazl is populating the zip with 0 byte files.

Released in 1.0.0.


Exercise Submitted

Please review and provide your feedback

Hi, I'm getting the following compilation errors with your submission:

Please fix these errors and resubmit the PR.
Thanks!

Hi, I'm getting the following compilation errors with your submission:

Please fix these errors and resubmit the PR.
Thanks!

Hi, It was working  in my local. I think I missed some files to upload. I will look into it, then will resubmit the PR.

I'm now getting NullPointer Error while running tests on my IDE:

Hi .. The code is working in my local ,that might be issue to your local environment ?  I am little curious to know what are next steps in this process .

Hi .. The code is working in my local ,that might be issue to your local environment ? I am little curious to know what are next steps in this process .

I'm pretty sure this is not an issue with my local but with the submission itself. Usually we expect that as soon as we clone the project we are able to build it. Next we should be able to run tests without any change in code or configuration on my side. Then only we look at the code. So the next step is to test that the submission really work on a different computer.


Transaction costs

How do SWAN network fees work?
( how does the SWAN blockchain work in general - can we join as a evm blockchain node? )
On Ethereum Mainnet, fees are astronomical right now.
So this also relates to Swan Mainnet fees?
A friend just wanted to register a ECP node. It is barely possible as it would costs like 12 Dollars..
can we do anything or do we have to wait?
can we extend the ECP Early Participant Reward period by some days?
computing-provider account create ...
Error: deploy cp account contract failed, error: insufficient funds for gas * price + value: balance<PHONE_NUMBER>000000, tx cost<PHONE_NUMBER>711028, overshot<PHONE_NUMBER>11028

this browser can not show the gas, if you want to know the all tx fee, you can see this page: https://mainnet-explorer.swanchain.io/address/0x82AB7Bd300976fe5e294586aD491d6bBd1CAfb06?tab=coin_balance_history

this browser can not show the gas, if you want to know the all tx fee, you can see this page: https://mainnet-explorer.swanchain.io/address/0x82AB7Bd300976fe5e294586aD491d6bBd1CAfb06?tab=coin_balance_history

okay that confirmes that 0.00089455 are deducted. still roughly 3 Dollars. This seems really high for a layer2. can you explain that?

I think it has been solved, please confirm it @ThomasBlock


Strange error every now and then after adding a new dependency

Hello, I've come across this multiple times, it usually happens after I install a new dependency then try to run the development server:
$ yarn serve
yarn run v1.12.3
warning ../../package.json: No license field
$ gridsome develop
internal/modules/cjs/loader.js:736
  return process.dlopen(module, path.toNamespacedPath(filename));
                 ^

Error: libvips-cpp.so.42: cannot open shared object file: No such file or directory
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:736:18)
    at Module.load (internal/modules/cjs/loader.js:605:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:544:12)
    at Function.Module._load (internal/modules/cjs/loader.js:536:3)
    at Module.require (internal/modules/cjs/loader.js:643:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/home/yomero/tmp/my-site/node_modules/sharp/lib/constructor.js:10:15)
    at Module._compile (internal/modules/cjs/loader.js:707:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:718:10)
    at Module.load (internal/modules/cjs/loader.js:605:32)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

It'll keep erroring out until I rm -rf node_modules then do a yarn install.
Any ideas on what's going on?
Cheers,
VA

This seems related to adding packages while the server is running.
Will close for now, as there is a fix for the error state:
rm -rf node_modules
yarn install


KeyError: 'days' when upgrading

Before you submit a new bug report, please check that

[X] I have read frequently asked questions and limitations
[X] I'm running the newest version of Garbage Collection, or the next pre-release.
[X] I have enabled debug logging for my installation.

Describe the bug
I've just completed doing an upgrade to my HA instance from 2021.11 to the latest. I've managed to get everything working OK apart from this integration. I get this error in the HA logs:
2022-11-20 23:03:11.673 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up garbage_collection platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 289, in _async_setup_platform
await asyncio.gather(*pending)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/garbage_collection/sensor.py", line 144, in async_added_to_hass
self._days = state.attributes[const.ATTR_DAYS]
KeyError: 'days'

I tried removing all sensors and even removing the custom component from HACS but it still reoccurred when setting up from fresh.
Configuration
{
  "home_assistant": {
    "installation_type": "Home Assistant OS",
    "version": "2022.11.3",
    "dev": false,
    "hassio": true,
    "virtualenv": false,
    "python_version": "3.10.7",
    "docker": true,
    "arch": "x86_64",
    "timezone": "Europe/London",
    "os_name": "Linux",
    "os_version": "5.15.74",
    "supervisor": "2022.10.2",
    "host_os": "Home Assistant OS 9.3",
    "docker_version": "20.10.18",
    "chassis": "embedded",
    "run_as_root": true
  },
  "custom_components": {
    "garbage_collection": {
      "version": "4.8.3",
      "requirements": [
        "python-dateutil>=2.8.2"
      ]
    },
    "google_home": {
      "version": "1.9.15",
      "requirements": [
        "glocaltokens==0.6.8"
      ]
    },
    "sun2": {
      "version": "2.1.3",
      "requirements": []
    },
    "frigate": {
      "version": "3.0.0",
      "requirements": []
    },
    "lockuser": {
      "version": "0.1.0",
      "requirements": []
    },
    "reolink_dev": {
      "version": "0.57",
      "requirements": [
        "reolink==0.0.62"
      ]
    },
    "hacs": {
      "version": "1.28.3",
      "requirements": [
        "aiogithubapi>=22.2.4"
      ]
    },
    "unified_remote": {
      "version": "6",
      "requirements": []
    },
    "tapo_control": {
      "version": "4.1.3",
      "requirements": [
        "pytapo==2.5",
        "onvif-zeep-async==1.2.0"
      ]
    }
  },
  "integration_manifest": {
    "domain": "garbage_collection",
    "name": "Garbage Collection",
    "version": "4.8.3",
    "documentation": "https://github.com/bruxy70/Garbage-Collection/",
    "issue_tracker": "https://github.com/bruxy70/Garbage-Collection/issues",
    "iot_class": "calculated",
    "integration_type": "helper",
    "dependencies": [],
    "config_flow": true,
    "codeowners": [
      "@bruxy70"
    ],
    "requirements": [
      "python-dateutil>=2.8.2"
    ],
    "is_built_in": false
  },
  "data": {
    "entity_id": "sensor.waste_black_bin",
    "state": "unavailable",
    "attributes": {
      "days": null,
      "last_collection": null,
      "last_updated": null,
      "next_date": null,
      "device_class": "garbage_collection__schedule"
    },
    "config_entry": {
      "entry_id": "64b4bb2f178b11eba11a9b9819cccfc2",
      "version": 6,
      "domain": "garbage_collection",
      "title": "Waste Black Bin",
      "data": {
        "unique_id": "99f4d75c-5903-41e0-9aae-e1ad15de290b"
      },
      "options": {
        "frequency": "weekly",
        "icon_normal": "mdi:trash-can",
        "icon_today": "mdi:delete-restore",
        "icon_tomorrow": "mdi:delete-circle",
        "first_month": "jan",
        "last_month": "dec",
        "collection_days": [
          "tue"
        ],
        "verbose_format": "on {date}, in {days} days",
        "date_format": "%d-%b-%Y",
        "verbose_state": false,
        "hidden": false,
        "manual_update": false
      },
      "pref_disable_new_entities": false,
      "pref_disable_polling": false,
      "source": "user",
      "unique_id": null,
      "disabled_by": null
    }
  }
}

Debug logs
2022-11-20 23:15:34.111 DEBUG (MainThread) [custom_components.garbage_collection] Setting Waste Black Bin (weekly) from ConfigFlow
2022-11-20 23:15:34.111 DEBUG (MainThread) [custom_components.garbage_collection] Setting Waste Green Bin (every-n-weeks) from ConfigFlow
2022-11-20 23:15:34.112 DEBUG (MainThread) [custom_components.garbage_collection] Setting Waste Recycling (every-n-weeks) from ConfigFlow
2022-11-20 23:15:34.337 ERROR (MainThread) [homeassistant.components.sensor] Error adding entities for domain sensor with platform garbage_collection
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/garbage_collection/sensor.py", line 144, in async_added_to_hass
self._days = state.attributes[const.ATTR_DAYS]
KeyError: 'days'
2022-11-20 23:15:34.344 ERROR (MainThread) [homeassistant.components.sensor] Error adding entities for domain sensor with platform garbage_collection
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/garbage_collection/sensor.py", line 144, in async_added_to_hass
self._days = state.attributes[const.ATTR_DAYS]
KeyError: 'days'
2022-11-20 23:15:34.347 ERROR (MainThread) [homeassistant.components.sensor] Error adding entities for domain sensor with platform garbage_collection
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/garbage_collection/sensor.py", line 144, in async_added_to_hass
self._days = state.attributes[const.ATTR_DAYS]
KeyError: 'days'
2022-11-20 23:15:34.415 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up garbage_collection platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 289, in _async_setup_platform
await asyncio.gather(*pending)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/garbage_collection/sensor.py", line 144, in async_added_to_hass
self._days = state.attributes[const.ATTR_DAYS]
KeyError: 'days'
2022-11-20 23:15:34.422 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up garbage_collection platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 289, in _async_setup_platform
await asyncio.gather(*pending)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/garbage_collection/sensor.py", line 144, in async_added_to_hass
self._days = state.attributes[const.ATTR_DAYS]
KeyError: 'days'
2022-11-20 23:15:34.429 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up garbage_collection platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 289, in _async_setup_platform
await asyncio.gather(*pending)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 692, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 788, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/garbage_collection/sensor.py", line 144, in async_added_to_hass
self._days = state.attributes[const.ATTR_DAYS]
KeyError: 'days'

I've tried to reproduce it, but I do not get the error. It fails on restoring the state - it finds the stored state, but the state is missing the attribute. Not sure why. perhaps something related to the update and the condition before the update. Not sure. I suppose if you'd remove and re-create the helper, that error would disappear?

Anyhow, I am adding a code to check for the individual attributes in the stored state.
Said that, not sure how this could happen, so I might be just adding an extra code. Will see - it should not breath other things :)

Check the laatest release if it helps. If yes, please close the ticket.

I suppose if you'd remove and re-create the helper, that error would disappear?

I tried this, I even removed the HACS component but I kept getting the same error. Maybe I was missing something it's been a long hard slog dragging it into 2022.11 :laughing:

Check the latest release if it helps. If yes, please close the ticket.

I've tested and it's back working, thank you so much!


New function in LIMS that can assign/unassign artifacts to workflow/s…

…tage
this add the function to route artifacts to workflows or workflow stages

Merge it myself


[BUG] Server got stuck during load

What happened?

I just tried to load my server stack with dcintegration loaded, but it stucks around this lines and dont proceed further:

[20:22:29] [Server thread/INFO] []: Loading mod
[20:22:29] [Server thread/WARN] []: SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
[20:22:29] [Server thread/WARN] []: SLF4J: Defaulting to no-operation (NOP) logger implementation
[20:22:29] [Server thread/WARN] []: SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
[20:22:30] [Server thread/WARN] []: SLF4J: Failed to load class "org.slf4j.impl.StaticMDCBinder".
[20:22:30] [Server thread/WARN] []: SLF4J: Defaulting to no-operation MDCAdapter implementation.
[20:22:30] [Server thread/WARN] []: SLF4J: See http://www.slf4j.org/codes.html#no_static_mdc_binder for further details.
[20:22:30] [Server thread/WARN] []: [Server thread] INFO JDA - Login Successful!
[20:22:31] [JDA MainWS-ReadThread/WARN] []: [JDA MainWS-ReadThread] INFO WebSocketClient - Connected to WebSocket

It just freezes and i need to kill the process in Task Manager to end it;

What should happen?

The server should continue the loading and be able to have players connecting.

Steps to reproduce

Download Mohist #219 from https://mohistmc.com/download/, run with CustomJava (just try to run Mohist .jar with a JRE with a greater version, like 15.0);
Place dcintegration-1.1.18-1.12.jar in mods folder;
Configure with the config file i provided (just need the token and channel id) and try to run the mod.

Version

[X] 1.12
[ ] 1.16

Platform

[ ] Forge
[ ] Spigot
[X] Mohist (Forge + Spigot Hybrid)

Mod version: 1.1.18
Config file

https://gist.github.com/vico93/56499bb8701cc8b9d8cc52ed4ec0a880

Additional info

Might be fixed with an actually planned backport to 1.12 from the current state
(will start with it as soon as I think the 1.16 is stable enough for release state again)

Is it fixed? If not just re-open


[Cisco CustomLink] - Added new condition to redirect to externalLink …

…without checking manifest
We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.

Helpful Tips
Add screenshots and annotations

In OSX Mojave, you can capture screenshots and add annotations. https://support.apple.com/en-us/HT201361
Another app you can use is https://www.getcloudapp.com/.

Add a gif to show an action
GIFs are great to show an action, however they sometimes move too fast.
One way to improve them is to capture 3 pieces of content:

screenshot before an action
screenshot after the action
gif of the action
You can edit a gif using https://ezgif.com/

Put a border around a screenshot/gif
Add a border to help improve the readability.
<kbd>![](https://www.spinnaker.io/assets/images/spinnaker-logo-inline.svg)</kbd>

Add expandable sections

Add expandable sections to hide away large code blocks/images or reduce the motion of a gifs
const example = 'spinnaker spinnaker SPINNAKER';

https://devopsmx.atlassian.net/browse/OP-21250


replace UniRef100.uniref90 fields by m2m on self / intermediate table with extra field for uniref90 accession

the uniref90 field is now indexed and can be searched, the uniref100 record pages may get links to their uniref90 group in the future and only the index is needed for that, so closing this as no m2m relation is required.


Scrolling inertia transfers scrolling when mouse over new scrollable area

VSCode Version: 1.16.1
OS Version: Windows 10.0.15063

Steps to Reproduce:

Open 2 windows in vs code that are scrollable
Hover mouse over first window
Flick mouse scroll wheel to trigger inertia scrolling
Move mouse pointer to second window. Second window will start scrolling.

Reproduces without extensions: Yes
This may be an OS level feature, but it's really annoying when the second window is the tabs on top that scroll when the mouse gets moved to hover it during inertia scrolling.

Edit: Demo shown using Microsoft Arc Mouse on a Surface Book. Flicked the "scroll wheel" on the mouse once, then proceeded to move the mouse while different windows started scrolling (because of scrolling inertia). Expected result would be to have originating window scroll the whole time no mater where the mouse drifts off to or a new scroll flick is triggered.

fyi @bpasero If you have any ideas
I can also reproduce on OSX with a magic mouse.
It appears this is how our platform (Chromium) implements scrolling inertia. i.e. scroll events keep coming in even if the mouse is moved over some other UI element.

That sucks, have not seen or thought about it before.


[state sync] structured logging + some counters

Motivation
Convert to structured logging (+ some refactoring for better error catching), add some error counters that go with error logging. Will clean up counters in later PR with dashboards

/canary

/help

Bors help and documentation

Bors is a Github App used to manage merging of PRs in order to ensure that CI is always green. It does so by maintaining a Merge Queue. Once a PR reaches the head of the Merge Queue it is rebased on top of the latest version of the PR's base-branch (generally master) and then triggers CI. If CI comes back green the PR is then merged into the base-branch. Regardless of the outcome, the next PR is the queue is then processed.
General

This project's Merge Queue can be found here.
This project requires PRs to be Reviewed and Approved before they can be queued for merging.
Before PR's can be merged they must be configured with "Allow edits from maintainers" enabled. This is needed for Bors to be able to update PR's in-place so that Github can properly recognize and mark them as "merged" once they've been merged into the upstream branch.

Commands
Bors actions can be triggered by posting a comment which includes a line of the form /<action>.

Command
Action
Description

Land
land, merge
attempt to land or merge a PR

Canary
canary, try
canary a PR by performing all checks without merging

Cancel
cancel, stop
stop an in-progress land

Cherry Pick
cherry-pick <target>
cherry-pick a PR into <target> branch

Priority
priority
set the priority level for a PR (high, normal, low)

Help
help, h
show this help message

Options
Options for Pull Requests are configured through the application of labels.

          Option
Description

Indicates that the PR is high-priority. When queued the PR will be placed at the head of the merge queue.

Indicates that the PR is low-priority. When queued the PR will be placed at the back of the merge queue.

Before merging the PR will be squashed down to a single commit, only retaining the commit message of the first commit in the PR.

if you want to cancel the canary, feel free to

:sunny: Canary successful

/land


Native epoll transport should honor AUTO_READ flag

Expected behavior
Data should only be read when requested, if AUTO_READ option is set to false
Actual behavior
When using Epoll transport, netty calls handler's channelRead0 without requesting read, even if AUTO_READ option is set to false. NIO Transport on the other hand works as expected.
This could be a problem if you are acting as a proxy.
Steps to reproduce
This is reproducible using small sized data I could reproduce it starting from 1 byte till 150k. Client must close connection as soon as done writing. Here are some settings I used.

SO_RCVBUF option was not set, although setting it didn't help.
AUTO_READ false
TCP_NODELAY false
SO_REUSEADDR true
cat /proc/sys/net/ipv4/tcp_rmem
4096	87380	6291456
/proc/sys/net/ipv4/tcp_wmem
4096	16384	4194304

This could help generating test data quickly:
cp some_large_file 10k_file
truncate -s 10240 10k_file
nc host port < 10k_file
Netty version
4.1.6.Final
JVM version
java version "1.8.0_65"
Java(TM) SE Runtime Environment (build 1.8.0_65-b17)
Java HotSpot(TM) 64-Bit Server VM (build 25.65-b01, mixed mode)
OS version
Linux 4.8.15-300.fc25.x86_64
x86_64 x86_64 x86_64 GNU/Linux

may be you should use Epoll level trigger, default Epoll Mode is edge trigger,  see this

Setting LEVEL_TRIGGERED bootstrap option did not help.

Client must close connection as soon as done writing.

IIRC if we get a EPOLLRDHUP then we must read until we get a socket error for stream sockets [1] while in ET mode as we may not be notified of activity on that FD again. This may not be necessary in LT mode, and especially if NIO transport is still able to deliver all data. I will need to investigate more.
[1] https://github.com/netty/netty/blob/4.1/transport-native-epoll/src/main/java/io/netty/channel/epoll/EpollRecvByteAllocatorHandle.java#L57


Continuous Integration

I have set up a GitHub action for creating binaries when a new tag is pushed, but we can do more with automation. I would like to create a test suite for nasher and have tests automatically run on PRs and commits. This will help ensure no breaking changes are introduced and help to catch bugs and blind spots.

As in unit tests or functional tests or both?

Both, really, though I'm more concerned with functional tests.

We use github actions to deploy our module: https://github.com/Kaikas/mintarn/tree/master/.github/workflows. Maybe something like that fir functional tests?


Globbing support

npm-workspace works at the current directory.
Being able to specify an array of directories would be useful (as well as exclude specific patterns).
Use case:
We release a library, with a set of worked out example installations. We would like npm-workspace to install the example installations which are held in a sub-directory.

Just discussed this with a colleague and we felt that "npm-workspace install --recursive" which tells npm-workspace to recurse through all sub folders running npm install (obviously ignoring node_modules directories) would be "good enough".

Hi Adam, thanks for the contribution, it sounds like a reasonable new feature to me. If you have a PR ready I'll merge it for sure, otherwise you'll probably have to wait a bit until I can find the time to work on this.

I think I should be able to pick this up... expect a PR in a day or so :-)


Beta distribution Sample method can return NaN

Tested with Math.Net v. 4.15
The Beta.Sample() method returns NaN for some a and b parameters. It can be reproduced by using:
a = 4.2430007555736642E-06
b = 0.0012675539420686256
The problem is in this method:
internal static double SampleUnchecked(System.Random rnd, double a, double b)
{
  double num = Gamma.SampleUnchecked(rnd, a, 1.0);
  return num / (num + Gamma.SampleUnchecked(rnd, b, 1.0));
}

if num = 0 and  Gamma.SampleUnchecked(rnd, b, 1.0) is also 0 we get an NaN.
This could fix the issue:
internal static double SampleUnchecked(System.Random rnd, double a, double b)
{
  double num = Gamma.SampleUnchecked(rnd, a, 1.0);
  if(num == 0.0)
    {
       return 0d
    }
  return num / (num + Gamma.SampleUnchecked(rnd, b, 1.0));
}

This issue seems to be fixed in version 5.0


Missing "6" character in custom font

I use your awesome custom Matrix font with my minimalistic matrix rain effect (a demo with a custom font).
And I'm not sure if you're aware but the font is missing the character "6". I'm not sure if this is intentional but I just decided to create an issue so you're aware of it.

Thank you, fellow code rain person, for your interest and kind words! I like your project! And welcome— to that rarefied fraternity of Matrix fans who notice that the "font", until 2021, is missing a digit. 😄
To me the lack of a six helps to emphasize that in the movies, the script isn't a notation or a symbolic structure of any kind, it is a cipher; any meaning in the scrambled glyphs is intentionally hidden layers deep from those who seek knowledge and truth. Including me and you. That's my personal interpretation, of course.
The downside to this interpretation is, anyone who asks me to "put their text in the code" faces a direct obstacle that my glyphs are not text. Most projects such as yours avoid this problem entirely. 😃
That said, when the franchise revisited the glyph set for the fourth film, they asked the multifaceted Andrew Glass, Unicode extraordinaire / asian scripts megascholar / repeat Matrix franchise advisor, to expand it, and he took a more informed approach:

As you noted, the digit 6 seems to have been omitted from the trilogy set. It seemed to me that there should be a complete numbering system, so a six would have been required. Let's imagine one might pick from the new signs, sufficient symbols to complete a hexadecimal set. Also, punctuation and common symbols might be needed in this "language", so maybe basic ASCII punctuation should have been included as a source too.

To me this is very fitting in light of the advances to the world depicted in Resurrections: the simulation is more devious; human and synthetic life live side-by-side in IO; and surprisingly, Neo is depicted as voluntarily using the original Matrix code for software development.
You get used to it, though.  I bet he doesn't even see the code. All he sees is Vue, React, and Svelte.
Matrix has ya!
-Rezmason
PS— thank you for being a teacher! 📚


Hardware buttons plugin doesn't help disable home button

I want my app to not go on homescreen or lock fir security purpose...i tried using hardware_buttons plugin...but that doesn't help with my issue... does flutter have a plugin/ function for doing this or it's an issue with flutter?
Please suggest a way if anyone knows

hardware_buttons is a deprecated plugins
if you need help to write you application it's better to ask a question on StackOverflow
as indicated in the new issue template for I want help writing my application

@Ashutosh3011
Per above comment, your question seem to be better suited for StackOverflow. You may use the new issue template to ask your question or please see https://flutter.dev/community for resources and asking questions like this.
If you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Biometrics to open the email app

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.714
Problem you are trying to solve
I'd like to keep my email more secure.
I live in a country where stolen phones are really common. There is a trend of phones being stolen while the owner has them unlocked, and they try not to click on the button to keep the screen on so they can access the data.
Suggested solution
I'd like to propose that the email app has a security feature similar to the one on the desktop app, where there is a need for a password.
My actual request would be to have a password to lock the email app and possibly allow biometrics to unlock it.
There could be a timer to lock it again similar to what WhatsApp already has.
Screenshots / Drawings / Technical details
Options available on WhatsApp as a reference:

Aditional information
The last switch was already asked at #7529.

Trying to defend against an attacker that gets access to an unlocked device on an app level is unreasonably costly (in terms of development and maintenance cost) and still leaves you unprotected in case you had the email app open while the device was stolen.
I suggest you look into solutions that can automatically lock your device when it is stolen. E.g. lock the device when the Bluetooth connection to a device that you keep on your body is lost.

I'm aware that I would still be unprotected if the email was open, and this exact behaviour of locking the app can be mostly created as well with apps like Kaspersky which can make apps need a fingerprint or a code to open, but that is still not ideal because when you open the app, sometimes it takes some seconds for the Kaspersky app to lock it.
I would still look into those options you mentioned, of course, but I would really appreciate it if this was at least not discarded up-front. I intend to make some contributions to OSS in the future, but I'm not familiar with Android development specifically yet, so if this issue could stay open here, to someone else to open a PR or even myself in the future, I'd be very grateful.

Essential feature for an email app. Access to email on a stolen phone is catastrophic as email is often used to reset passwords. I also help if one lends the phone to a child or other person with permission and you do not want them to see your mail or fiddle with it. The Norton app lock is a pain as not sure it uses biometrics.

I can see the use case behind this, though agree with cketti it will be some effort to implement and maintain. Would you mind re-posting this at https://connect.mozilla.org/t5/ideas/idb-p/ideas/label-name/thunderbird android and closing this issue? Connect has a broader reach and is a better way to capture product ideas such as this one.

Here it is for anyone interested https://connect.mozilla.org/t5/ideas/biometrics-to-open-thunderbird-android/idi-p/74741#M40268.


Update command to install nacl in install-hub.sh

Update command to install nacl in install-hub.sh

@coesensbert It is OK to remove the pip steps as the steps above added the packages at the user level.

@coesensbert It is OK to remove the pip steps as the steps above added the packages at the user level.

ok, I should clarify better. What I mean was that while building this thing, I remember doing the opposite. So not installingvia apt, but via pip3 install ... to fix issues with nacl. So I fear it might be ubuntu version dependent since I tested on 24.04

OK I see. Do you think we should have more test then? Try apt vs pip3?

Testing on docker ubuntu 24:

Don't know what is best, if it's ok for Harby it's ok for me with the merged PR


Update README.md with correct order of arguments in reducer

I think I found an error in the readme where the order of arguments is still reflecting the old v3.

awesome, thanks!


Allow user to set the listening port for mleap serving as command line argument

I would like to have the possibility to set the listening port from the starting command (as a parameter) so I will be able to launch multiple instances of mleap serving on the same machine without using docker

@davidecoccia We will be overhauling MLeap serving as part of 1.0. Putting together a roadmap soon and posting it in the README. Will close this issue and point you there when it is ready.

@davidecoccia, I opened pull request https://github.com/combust/mleap/pull/366 that consequently helps with this request. Hopefully we can get it reviewed / discussed / merged soon. :)

Merged #366


[stable/nextcloud] Image stuck at Initializing NextCloud... when PVC is attached

Describe the bug
When the helm chart is bringing up NextCloud, the application does not get past the log message
Initializing Nextcloud 17.0.7...

Version of Helm and Kubernetes:

helm: v3.2.1
kubernetes: v1.18.4+k3s1
Which chart:
stable/nextcloud
What happened:
Namespace is created.
Helm creates persistent-volume-claim
Helm instantiates MariaDB using bitnami/mariadb chart
Helm instantiates Nextcloud container
Nextcloud container starts
Nextcloud container does not get past
Initializing Nextcloud 17.0.7...

What you expected to happen:
Nextcloud was supposed to finish initialization
Nextcloud files were supposed to be copied with correct permissions to the PVC
How to reproduce it (as minimally and precisely as possible):
Initialize helm with the following:
helm install stable/nfs-client-provisioner nfs --namespace=nas  \
--set nfs.server=x.x.x.x --set nfs.path=/mnt/external

helm install -f values.yaml stable/nextcloud files --namespace=nextcloud

values.yaml
ingress:
  enabled: true
  annotations:
    kubernetes.io/ingress.class: traefik
    cert-manager.io/cluster-issuer: cluster-issuer
    traefik.ingress.kubernetes.io/redirect-entry-point: https
    traefik.frontend.passHostHeader: "true"
  tls:
   - hosts:
     - files.haus.net
      secretName: nextcloud-app-tls
nextcloud.host: files.haus.net
nextcloud.username: admin
nextcloud.password: P@$$w0rd!
internalDatabase:
  enabled: false
mariadb:
  enabled: yes
  password: P@$$w0rd!
  user: nextcloud
  name: nextcloud
persistence:
  enabled: yes
  storageClass: nfs-client
  size: 1Ti

I have also been trying to get this install to work with a PV and PVC and no luck, If I do it without a PVC and PV it works, s soon as I enable the PV, it says nextcloud directory isnt found, so I make the directory. Then it says "Error: failed to create subPath directory for volumeMount "nextcloud-data" of container "nextcloud"". does anyone have any ideas about this?

I am having the same issue. I also use nfs-client as storageClass, which might cause this bug? IIRC I used a manual created PV some time back and it worked.
Have you figured out how to make this work?

Not sure if we are having the same issue, but I will detail my investigation so far on trying to use persistence.existingClaim, in case it helps people progress in their own investigations and/or if the context would help someone more knowledgeable provide some help as I have only worked with k8s for a year or so.
From what I could see, the container creation process errors out with:
Error: failed to start container "nextcloud": Error response from daemon: OCI runtime create failed: container_linux.go:349: starting container process caused "process_linux.go:449: container init caused \"rootfs_linux.go:58: mounting \\\"/var/lib/kubelet/pods/49c19090-14d6-4bee-b774-ca24b0ddd259/volume-subpaths/jun30third-nextcloud-data-pv/nextcloud/0\\\" to rootfs \\\"/var/lib/docker/overlay2/40dca10bcad3a57d61d35d40d0bd897f6d2322c3a5d9f615d2a90a38d7fe4cd5/merged\\\" at \\\"/var/lib/docker/overlay2/40dca10bcad3a57d61d35d40d0bd897f6d2322c3a5d9f615d2a90a38d7fe4cd5/merged/var/www\\\" caused \\\"no such file or directory\\\"\"": unknown

I've looked on the node during the time of the directory creation, and some things to note:

the source directory always exists and the path is correct
the destination directory up to (.*)/merged is created when the container is being spun up, but I could never see merged directory inside (although I didn't have the container ID beforehand so relied on watch commands and manually looking on the node, so I can't guarantee that it was never there, I just know I could never see it there)

The only lead I've found so far to why this might be happening is https://github.com/kubernetes/kubernetes/issues/61545#issuecomment-465887014 and the following comment links https://github.com/kubernetes/kubernetes/issues/61563#issuecomment-428364190. My guess is that this is related to the second issue in the last comment (i.e. https://github.com/kubernetes/kubernetes/issues/61545), given that the config mounts are nested inside the directory mount, however given that the error is on subpath /nextcloud/0 of the container (which I have verified is the root subpath), this might not be true but is my best lead so far.
I'm currently poking by manually changing specifications to see if any configuration works (i.e. trying different variations of the mountpaths nesting to see if I can get it to start up manually before figuring out how to correct the chart), but in the meantime if anyone else finds a solution and/or if it seems I'm going down the wrong trail, please let me know!

Okay I got it working! I am using a Open Media vault NFS share for all of my persistent volumes. I set them up with the following settings and it now works without any issues when using the regular helm install, no extra stuff required.
Settings for nfs share.

rw,no_root_squash,insecure,async,no_subtree_check,anonuid=1000,anongid=1000

It also works with nfs-server-provisioner with default values.

Okay I got it working! I am using a Open Media vault NFS share for all of my persistent volumes. I set them up with the following settings and it now works without any issues when using the regular helm install, no extra stuff required.
Settings for nfs share.

rw,no_root_squash,insecure,async,no_subtree_check,anonuid=1000,anongid=1000

Tried changing the line in my /etc/exports and it didn't fix the problem.

Using the following snippets:
nfs-client-provisioner.values.yaml
nfs:
  mountOptions:
    - nfsvers=4
  server: <IP_ADDRESS>
  nfs.path: /mnt/external

I updated my nextcloud values with the new value persistence.accessMode=ReadWriteMany.

Got the same problem. Tested with version 17.0.0-apache and 19.0.1-apache. Also seeing that the dirs are root:root.
When we deploy without PVC the installation works

Using nfs-client-provisioner works but the main problem is that the initial rsync takes arround 5 minutes to complete (at least in my tests). You can look at the entrypoint.sh file.
rsync -rlDog --chown www-data:root --delete --exclude-from=/upgrade.exclude /usr/src/nextcloud/ /var/www/html/
If you disable the readiness and the liveness in the values, it works.
❯ k logs nextcloud-5756597dbc-nhg5m Initializing nextcloud <IP_ADDRESS> ... Initializing finished New nextcloud instance Installing with PostgreSQL database starting nextcloud installation Nextcloud was successfully installed setting trusted domains… System config value trusted_domains => 1 set to string XXXXXXXX AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message [Tue Aug 11 08:54:50.097547 2020] [mpm_prefork:notice] [pid 1] AH00163: Apache/2.4.38 (Debian) PHP/7.3.21 configured -- resuming normal operations [Tue Aug 11 08:54:50.097621 2020] [core:notice] [pid 1] AH00094: Command line: 'apache2 -D FOREGROUND'
I've trying some alternatives to that rsync but since there are a lot of small files to copy i haven't found any improvement.
Any ideas?

Log looks like stuck at Initializing Nextcloud 17.0.7..., because the rsync process extremely slow (for my local nfs, it is about 1.5MB/s, show the progress with rsync --info=progress2).
As the workaround like jesussancheztellomm, I disable the liveness probe on first installation, and enable it after finishing the installation.
Maybe we can refer to nextcloud/docker#968
It will not solve the problem of slow nfs transmission speed (I still have no idea why ...), but stateless nextcloud may remove the rsync process.

@timtorChen i can confirm, when i disabled the LivinessProbe it took 11min to sync. Also tried it with an S3 Storage backend, it took just seconds to sync.
So i looked deeper in my NFS, and we are using SYNC instead of ASYNC because we want not lose any data. I didnt test it with an ASYNC connection.

The nextcloud chart has migrated to a new repo.  Can you please raise the issue over there?  https://github.com/nextcloud/helm

Opened an incident over on the new repo. Tried to summarize some of the info from this discussion.


Added better documentation for temperature calculation

Add documentation for temperature function

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b3bbb03) 91.89% compared to head (c7e7be0) 91.89%.
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   91.89%   91.89%
=======================================
  Files           6        6
  Lines          74       74
=======================================
  Hits           68       68
  Misses          6        6

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


Add check for full resources

If resources are full, the script will break while fighting Soren and Wrizz for example, as an extra pop-up info box appears letting the player know not all resources could have been collected.
Because of this, I ask everyone to keep an eye on their resources (don't forget guild coins as well, there's a max of 200K) and spend them whenever necessary, while this issue is not fixed.

A possible way to fix this is by checking for full items inside the bag whenever starting the script. Seems very difficult to do though.


Add customizable footer links to ticket shop

This PR fixes issue #204

Unfortunately this change lacked the migrations and a few other file changes. Please compare to another pull request here which solved it in the meantime: https://github.com/fossasia/eventyay-tickets/pull/225


Add AWS related mattr_accessors

Should take care of #93 by allowing users to set the AWS region, access key, and secret key using either environment variables or secrets.yml

Can we default region to east? Otherwise, looks good to me.

You don't have to assign anything to Storytime.aws_region, as CarrierWave defaults the region to 'us-east-1'.

Perfect. Merge away then.


Unable to setup uCaptcha on frontend

I am unable to get uCaptcha to work on the frontend.
I am already using Umbraco Forms without problems, and I have followed the setup guide, and the required theme changes.
Still, it seems like uCaptcha is expecting something to be there, that is not there:

We are using a custom theme, in the way that we are using a custom Form.cshtml file, but relying on the default theme for anything else. So I added the default theme as mentioned in the required theme changes, and reinstalled uCaptcha. That did nothing.

Am i misunderstanding something, or is this broken?

Hi @aguyfromdenmark
It looks like the file copy is now working, can you manually create this file:
https://github.com/UmbHost/Our.Umbraco.Forms.uCaptcha/blob/main/src/Views/Partials/Forms/Themes/default/Fieldtypes/FieldType.uCaptcha.cshtml

Thanks @AaronSadlerUK , that did the trick :)


[Image Modification] Add WebP support + Path Finder support

Description
This update to the Image Modification extension add supports for converting images to and from WebP format, as well as for executing SIPS commands (such as resize, rotate, etc.) on WebP files by utilizing a temporary PNG image. WebP conversion is made possible by embedding the cwebp and dwebp binaries into the extension. The update also adds a preference for preferred file manager, currently supporting Finder and Path Finder.
Issues Resolved

Resolves #5584
Resolves #5587

Screencast
ImageMod720.webm
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

AMAZING!!!! Thank you 🤗🤗🤗🤗

@SKaplanOfficial the WEBP conversion doesn't seem to be working most of the time. I'm simply using Cleanshot to take a PNG photo and then convert it and it just errors out:

Any idea what might be going wrong there?

@abass That's the result of a permissions issue with the version downloaded from the store — my latest PR contains a fix for it. In the meantime, you can use find ~/.config/raycast/extensions/ -name "cwebp" to locate the cwebp (and likewise dwebp) binary, which you can give execute permission with chmod +x. Sorry for the inconvenience!

Hugely appreciate it! Thanks so much!


NRF51_DK firmware in release package invalid

For the NRF51 DK board that I have, the firmware from the release package does not work (Daplink drive does not enumerate).
0241_sam3u2c_nrf51dk_0x5000.bin
However, if the following other binary works (also from 0241 release, but not in the separate binary release package)
sam3u2c_nrf51dk_if_crc_legacy_0x8000.bin

@gamnes, can you confirm that the nrf51dk expects the application to be loaded at 0x8000?

Depending on which bootloader you are running; the nrf51dk expects the application to be loaded at 0x8000 for the new DAPLink bootloader (MAINTANANCE), or if running with the old CMSIS-DAP bootloader (BOOTLOADER) it expects the application at 0x5000.

Hi @gamnes, are new boards being created with the DAPLink bootloader?

New boards are created with the (old) CMSIS DAP bootloader, not the DAPLink bootloader. This is to keep compatibility with the other firmware that can be used on our boards.

okay, so this issue is because the board I had was upgraded to the daplink bootloader, but most people will have the cmsis-dap bootloader out of the box.  Maybe it would still be good to have both binaries in the release package to cover both cases. @c1728p9 would you agree?

There isn't a user facing way to get or update the DAPLink bootloader.  To avoid confusion the release package contains only interface firmware for existing bootloaders. I would prefer to keep it this way until new bootloaders are publicly released.


nixos/gnome3: fix gnome-flashback

Motivation for this change
This session would fail to start because we didn't have it in systemd.packages
(as we've switched to systemd gnome-session).
Haven't tested custom sessions.

Also investigating a gnome-flashback test as a part of gnome3-xorg test and gnome-classic.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Noticed there was a gschema override we needed to accommodate https://github.com/NixOS/nixpkgs/pull/71210/commits/0dbbeabdaba6b6ac57c2450c68afd8cf06e9e3f8.


cannot import name 'reset' from 'pytorch_lightning.callbacks'

Im sorry to disturbe you. I am a novice. I follow the conda environment installation guide, but i still find it is hard to run the project. I also can not find any useful solution online.

Hi
I also experienced the same issue with a higher pytorch-lightning version.
Please downgrade pytorch-lightning lower than 1.7 at least.
Note that this code is built on pytorch-lightning==1.3.8
Thank you, have a good day! :slightly_smiling_face:
Best, Dahyun


Update build-dynamic-lean-configuration-manager.md

Updating images (in progress, do not commit yet)

Superseded by new PR


Release 1.1.0

Please approve the release of zhmccli 1.1.0.
Change log: https://github.com/zhmcclient/zhmccli/blob/release_1.1.0/docs/changes.rst

Coverage increased (+0.007%) to 32.651% when pulling fa58c11f06dec355ff6382484d3419124a917512 on release_1.1.0 into 1c782ba19b1c6fa789ba4ad849e4690074344262 on master.


Support large data inputs and DDP consumption

🚀 The feature
DataPipe support for very large datasets at source and sink levels:
Source level:
Support for large sharded datasets (think 100M protein sequences across 1000s of CSV files, 100M molecular graphs in 1000s of pickled binaries, ...)

allow for loading and batched pre-processing
some support for shuffling

Sink level:
Support for easy parallelization of dataloading when training with DistributedDataParallel

distribute the input data across GPUs and workers without custom code
provide options to deal with uneven data distribution at the dataset level

Motivation, pitch
Writing training pipelines for very large datasets is currently quite labor-intensive, but often uses several concepts that I think could be abstracted away.
1 - pre-processing steps are often vectorize-able, so loading the data like in the README example:
@functional_datapipe("parse_csv_files")
class CSVParserIterDataPipe(IterDataPipe):
    def __init__(self, dp, **fmtparams):
        self.dp = dp
        self.fmtparams = fmtparams

    def __iter__(self):
        for filename, stream in self.dp:
            reader = csv.reader(stream, **self.fmtparams)
            for row in reader:
                yield filename, row

would lead to inefficiencies. A proposed way would be some implementation that would allow batching and batch processing of datasets.
2 - when using DDP with datasets above, in the setting in which there are multiple GPUs and multiple workers processing and loading the data per GPU, I think the current approach is to:

shard the initial dataset (in n files, for example)
distribute the files amongst the workers*GPUs to ensure that every GPU gets a different slice of the dataset

This could be facilitated if there was a way to load a dataset in chunks and distribute the chunks across workers automatically using DataPipes.
Alternatives
Some in progress/parallel implementations of this I've come across:

An API was proposed and implemented in https://github.com/pytorch/pytorch/pull/26545
Petastorm project tries to address this problem specifically for parquet files: https://github.com/uber/petastorm - however found this to not work in practice because does not guarantee even shards: https://github.com/uber/petastorm/issues/508
YogaDL is an open source data sharding that adds a caching layer for initial dataset shuffling: https://github.com/determined-ai/yogadl

Additional context
No response

TorchData is introduced for loading large dataset with better syntax.
For you first pitch, I do not fully understand why it is not efficient. We provide batch for you to batch prior data to a list. Then, you can provide vectorized function to apply on a batch.
The example you point out is basically reading rows from a single csv file. I believe you want to yield multiple rows at the same time. But, in terms of the efficiency, I don't see any difference because it's not vectorization.
For the second pitch, we are working on dynamic sharding for multiprocessing and distributed training. Then, you don't need to manually provide hard-coded number of shards within the implementation of your dataset. We will let you know when the feature is ready.

Thank you! Let us know if we can help test or give feedback in any way!

Not sure if this is a similar thing, but I'm concerned about being able to cache certain points in a pipeline (maybe there already is something?).
My problem is simple: split a datapipe into train/valid datapipes.
For example, I want to split a datapipe into train/valid datapipes. I think this would be a combo of fork and filter. The problem is that per iter, the filter may have to iterate through 10000+ train elements get to a validation element and vice versa.
I've been messing around with a Cacher data pipe that could help with storing the train and valid. If this isn't the correct method of avoiding this, I'd appreciate an example to be pointed to.

@josiahls
You can use Demux https://github.com/pytorch/pytorch/blob/1d497114e7e13822df403082975d4b212d836207/torch/utils/data/datapipes/iter/combining.py#L215 for this use case.

@ejguan Is there any update on the dynamic sharding for multiprocessing and distributed training feature?


Making whole new paths in SP shop

Hello,
I was wondering if it was possible to create a higher path than Master/Wizard in the SP spell shop. I could not find anything about this in the documentation or configs since all they explain is the enchanting mode. So is this possible? And how do I do this if it is possible?

Oh okay, I'll try that, thanks.


Can't run the methylation detection script

Unable to run the mCaller.py -m GATC script with the following error thrown. Please assist.
/home/adas/mCaller-master/mCaller.py -m GATC -r Pv_wholegenome_Seq.fasta -d r94_model_NN_6_m6A -e mCal.eventalign.tsv -f barcode1_merged.fastq -b A
Traceback (most recent call last):
File "/home/adas/mCaller-master/mCaller.py", line 187, in
main()
File "/home/adas/mCaller-master/mCaller.py", line 157, in main
assert os.path.isfile(modelfile), 'model file not found at '+modelfile
AssertionError: model file not found at r94_model_NN_6_m6A

Hi, I looks like r94_model_NN_6_m6A is not in the folder you're running mCaller from. Check that the file exists by that name and its location. You may also be missing a file suffix (e.g. .pkl).

Hello, please let me know if this issue is not resolved. I will close it for now.


Organize CSS Selectors and Properties

WHEN I view the source code
THEN I find semantic HTML elements
WHEN I view the structure of the HTML elements
THEN I find that the elements follow a logical structure independent of styling and positioning

Changed HTML files to reflect proper semantic structure


Add autoindent rules

Add autoindent rules for maps {} and lists [] referring to language-json.
Fix #14.

@GiantToast what needs to be done for this PR to be merged?

Bug me about 😄 Sorry it's been so long, I've just been crushed with work lately.


fix(utils): Fix clamp return wrong value when value is smaller than min value

Overview
PR on issue #184
If the value of bound2 does not exist, the clamp should return bound1 if value is less than bound1.
So I fixed when bound2 is null and modify the test case.
function clamp(value: number, min: number): number;
function clamp(value: number, min: number, max: number): number;

According to the docs, bound1 corresponds to min value.
The existing code seems to be written assuming that bound1 is max value.
PR Checklist

[x] I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

I think clamp was designed to be used like:
function clamp(value: number, maximumValue: number);
function clamp(value: number, bound1: number, bound2: number);

So I think the current implementation is correct and we should keep it.

Then I'll fix docs and commit it. 👍

Yes I think fixing the docs would be sufficient.
Although I think it is very hard for us to predict what clamp(3, 5)  does — we would always have to check the docs to know if 5 is min or max.
Better for us to deprecate the API with two arguments in the next release…


helm package throws "no such file or directory" error

Output of helm version: 2.10.0
When I run the following command on Ubuntu 16.04 using an Azure Pipelines CI server:
helm package Foo --dependency-update --destination /home/vsts/work/1/s/Artefacts --version 1.0.15

I get the following error:

No requirements found in /home/vsts/work/1/s/Ummati/charts.
Successfully packaged chart and saved it to: /home/vsts/work/1/s/Artefacts/Ummati-1.0.153-master.tgz
Error: stat /home/vsts/.helm/repository/local: no such file or directory

Prior to your package command you should run:
$ helm init -c

This will initialize the Helm client (no tiller). -c is short for --client-only. The helm client needs to be initialized to do dependency handling.


improve Readme / Get started

In the Readme, Get started section, you should add a step : add rebase to the .cabal file...  It seems obvious, but newbies will forget it.

@pcarbonn Probably you meant _add relude to the .cabal file? I think this can be useful addition :+1:

Yes, that's what I meant.


support opentelemetry

When the runpod handle a request, I want to trace it, support opentelemetry will be helpful

Are you looking for more information than what we provide in the logs? Additionally, have you set your RUNPOD_LOG_LEVEL to DEBUG to see additional information?


Like to see which sx1301 modules are compatible

Since you can't order the nebra lora module anywhere it would be great to know which sx1301 modules are compatible.
This way people are able to repair their miners and not reliant on support

We only officially support the Nebra one.
If you have an issue with yours please contact<EMAIL_ADDRESS>


allow filters to be defined in a check

option to set filters in the check.
Some checks require variable levels of handling, and at the moment, a new handler need to be created for each filter variation, for example:
pagerduty_notify_instantly
pagerduty_notify_after_5_failed_attempts
pagerduty_notify_after_10_failed_attempts
Say I have a single check that i want to only notify after 5 failed attempts, but all others must notify instantly when critical.
this results in having handlers like:
pagerduty_notify_instantly
pagerduty_notify_after_5_failed_attempts
pagerduty_notify_after_10_failed_attempts
it would be simpler to have 1 handler, and simply define the filter inside the specific check that needs it.

Could anyone clue me in on what's happening here, and I can finish writing tests for this PR:
https://github.com/sensu/sensu/blob/master/spec/server/filter_spec.rb#L156
Why all the nesting?

We do not want to add "filter" or "filters" to check definitions, as Sensu uses handler definitions to define/configure the result and event processing pipeline. Multiple handler definitions can be used to the same effect, without introducing more ways to influence the pipeline, potentially causing confusion.


Scanner Aborts/Crashes on Character Scan

Hi my scan keeps crashing when scanning my characters. It stops when the scanner moves on to the second character.
This error message was in the log:
Object reference not set to an instance of an object.
at InventoryKamera.CharacterScraper.ScanCharacters() in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\CharacterScraper.cs:line 23
at InventoryKamera.InventoryKamera.GatherData(Boolean[] formats, Boolean[] checkbox) in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\InventoryKamera.cs:line 94
at InventoryKamera.Form1.<>c__DisplayClass21_0.<StartButton_Clicked>b__0() in B:\Projects\VisualStudio\InventoryKamera\InventoryKamera\InventoryKamera\Form1.cs:line 207

What is the character that it crashes on?

It has crashed on Hu Tao, Tartaglia and Xingqui, I thought it might have been character based so I changed my party.

I just ran a character only scan and it worked perfectly

Were the passing and failed scans done on different resolutions? It would be helpful to know if you changed anything between the fail and success.

All settings were the same. I only unticked the Weapons and Artifacts box in the scan settings.
I have since ran a successful weapon only scan.
The scanner crashes during the Artifact only scan. So I assume the issue was with artifacts all along.
The scanner has crashed in the same spot each time when running an artifact only scan. When the bottom left artifact counter is on 131/213. This should hit 196/213 when completed.
The scanning in game had finished by that point. The artifact menu had closed and the paimon menu had then reopens and there are no more inputs in to my Genshin Game. At this point the Inventory Scanner app was still checking through he artifacts. The section in the app that shows the screengrabs of each artefact was cycling through what it had seen.
Maybe a issue with my laptop being slow and not processing each artifact fast enough
I have gone back to version 1.0 and used that, it worked perfectly

Hi there, this issue is hopefully resolved in the v1.1.1 prerelease. If it has been, please let us know so we can close this issue.


ISPN-14074 integrationtests/server-integration are broken with JDK17

https://issues.redhat.com/browse/ISPN-14074
Needed to update the configuration. This update in the configuration was generated via a script.

Thanks @jabolina


Add a layer showing elevation using contours. #82

This serves a different purpose from the "steep streets" layer: intuitively see high/low spots in the city.

Smoke test: I can tell that northbound along Fremont Ave is quite a climb, and see the steeper/less steep sections. (Also, Queen Anne is ridiculous.)

I even solved a slight mystery about the flattest way for me to get to Greenwood -- dip below 85th a bit. I've found a slightly more direct route straight up 83rd, but this shows me I could avoid even more hill if I wanted.

Looks reasonable in night mode.
I tried to tune the colors so that you can zoom in and still kind of see the map underneath, but suggestions definitely welcome here!
CC @eldang, @Robinlovelace, @muzlee1113

https://twitter.com/CarlinoDustin/status/1375238532642398210
The GIF was too big for github, but it's here, showing the tooltip and zooming behavior

The GIF was too big for github, but it's here, showing the tooltip and zooming behavior

BTW - GH let you upload video files, which tend to have an order of magnitude better compression.
My workflow is to run the macos raw screen capture (which is a high resolution .mov) through:
ffmpeg -i "${INPUT_FILE}" -vcodec h264 -acodec mp2 -vf scale=800:-1 "${OUTPUT_FILE}"

Thanks for the mp4 tip! I had tried it before, but didn't specify the codec, and the result didn't seem to work. Trying again:
https://user-images.githubusercontent.com/1664407/112679500-77399d80-8e29-11eb-8c33-b50e77f66eba.mp4


feat: throw message when adding lambda function with same name

Description of changes
when adding a function with the same name as an existing function, the cli throws errors that are not helpful.

debug:
Error: test is present in amplify-meta.json
    at AmplifyToolkit.updateamplifyMetaAfterResourceAdd (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/extensions/amplify-helpers/update-amplify-meta.js:127:15)
    at createFunctionResources (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-function/lib/provider-utils/awscloudformation/utils/storeResources.js:47:21)
    at addFunctionResource (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-function/lib/provider-utils/awscloudformation/index.js:88:56)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async Object.executeAmplifyCommand (/snapshot/repo/build/node_modules/@aws-amplify/amplify-category-function/lib/index.js:277:5)
    at async executePluginModuleCommand (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:135:5)
    at async executeCommand (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:33:9)
    at async Object.run (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/index.js:117:5)
There was an error adding the function resource

The PR provides a warning rather than error out after
https://github.com/aws-amplify/amplify-cli/assets/87995712/d15a27f9-4eea-4044-a62f-1c03224cb93f
please do point to if i need to add any tests and if yes where?

Issue #, if available
https://github.com/aws-amplify/amplify-cli/issues/12554

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #12623 (4c425c8) into dev (ec9a2ba) will increase coverage by 47.57%.
The diff coverage is 47.13%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##              dev   #12623       +/-   ##
===========================================
+ Coverage    0.00%   47.57%   +47.57%
===========================================
  Files        1296      843      -453
  Lines      149743    38370   -111373
  Branches     1296     7847     +6551
===========================================
+ Hits            0    18256    +18256
+ Misses     148447    18482   -129965
- Partials     1296     1632      +336

Impacted Files
Coverage Δ

...ify-category-function/src/events/prePushHandler.ts
33.33% <0.00%> (+33.33%)
:arrow_up:

...ervice-walkthroughs/generalQuestionsWalkthrough.ts
25.00% <0.00%> (+25.00%)
:arrow_up:

...lify-category-hosting/lib/S3AndCloudFront/index.js
89.65% <ø> (+89.65%)
:arrow_up:

...s/amplify-cli-core/src/errors/amplify-exception.ts
82.35% <ø> (+82.35%)
:arrow_up:

packages/amplify-cli-core/src/types.ts
100.00% <ø> (+100.00%)
:arrow_up:

packages/amplify-cli/src/commands/console.ts
0.00% <0.00%> (ø)

...c/extensions/amplify-helpers/auth-notifications.ts
41.00% <4.34%> (+41.00%)
:arrow_up:

...service-walkthroughs/execPermissionsWalkthrough.ts
70.42% <11.11%> (+70.42%)
:arrow_up:

...s/awscloudformation/utils/cloudformationHelpers.ts
11.62% <11.11%> (+11.62%)
:arrow_up:

...udformation/src/utils/remove-dependent-function.ts
12.24% <11.11%> (+12.24%)
:arrow_up:

... and 16 more

... and 1280 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add support for api_token

With api_token you are able to generate access to the api which has limited capabilities.
Therefore we wanted to use it and implemented it in our environment.

@willbuckner, @sinkingpoint, can you approve the PR if you're happy with it?
@krombel, feel free squash when you have an approval.

Thanks for your review. As you approved the PR I now squashed the commits as mentioned

Thanks!


Use a weaker dependency on smallvec.

There's no reason to use 1.4, 1.0.0 works just as well.
1.4 causes a big perf regression in Firefox, see
https://github.com/servo/rust-smallvec/issues/243, so while we figure
that out we'd like to keep using 1.3.0.

r? @gwenn or someone else?
If a dot-release could be made with this change it'd be fairly amazing.

version = ">=1.3.0, <1.4.0" ?

This evening I can apply this and cut a release with it and no other changes from the prev.
(We have had a substantial breaking change so it couldn't be a patch release without that, and I don't want to cut the next major quite yet)

1.0 is fine if it compiles (or whichever got rid of the soundness holes by using MaybeUninit)

(We have had a substantial breaking change so it couldn't be a patch release without that, and I don't want to cut the next major quite yet)

Yeah, that sounds great, thanks!

1.0 is fine if it compiles (or whichever got rid of the soundness holes by using MaybeUninit)

Yeah, it does, I checked out this repo, cargo build, cargo update --precise 1.0.0 -p smallvec, then cargo build again.
Keeping 1.0 in the dependency gives us flexibility to update when the bug is fixed as well, so I'd prefer this to >=1.3.0, <1.4.0.
Thanks.

@emilio This is in 0.24.2, which I just released.


This PR documents a safe unshimming algorithm [skip ci]

It is a guide for developers and it should be further automated as the dist jar verification for correct placement of public classes
Current validation for bitwise-identity is just an approximation.
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build

build


refactor: split IDDataFilter into a StratifiedByOutcomeSplitter and a IDDataFilter?

# TODO: Split IDDataFilter into a StratifiedByOutcomeSplitter and a IDDataFilter?

https://github.com/Aarhus-Psychiatry-Research/psycop-common/blob/9499887a3c007e82bcc69c9d93eb874d4485e6ba/psycop/common/model_training_v2/trainer/data/data_filters/original_ids.py#L14
Seems the two concerns are somewhat unrelated? Would also markedly simplify the function signature. What do you think? @HLasse

I get a 404 on the link so not completely sure what you're referring

👍


Codespaces for https://online.visualstudio.com only shows one tenant during create flow online.

Apologies if this shuold be rerouted.
Codespaces currently is unable to instantiate for multiple tenants, only the first tenant is offered. The subscriptions listed should either be from all tenants or an additional dropdown created for tenant selection.

VSCode Version: 1.46.0
OS Version: Ubuntu 18.04

Steps to Reproduce:

Have an Azure account with multiple tenants (sometimes called directories)
Navigate with authenticated Azure Account to https://online.visualstudio.com/environments
Click "Create Codespace" Notice how only 1 tenant is shown (namely the first one returned from the management service).

Does this issue occur when all extensions are disabled?: Yes
Slightly unrelated: I get "resource validation failed" when creating a codespace via VSCode, so I have no workaround for instantiating a Codespace

Please share via feedback: https://docs.github.com/en/github/developing-online-with-codespaces/about-codespaces


Automatically bump package versions when necessary

PR Highlights:
Whenever the main Closure Compiler version is upgraded:

The @ampproject/google-closure-compiler-* package versions are updated for all platforms
A new tag for that version is created and pushed

With this, we can automate upgrades when a new closure version is available, and simplify the publishing of @ampproject/google-closure-compiler-* to NPM

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

/cc @kristoferbaxter @dvoytenko I'm going to test, merge, and retest this PR with the December Closure upgrade. This should further streamline our monthly workflow.

Yay, this worked! All native binaries were automatically generated and uploaded, versions were updated, and a new tag was uploaded. See https://github.com/ampproject/amp-closure-compiler/actions/runs/424605084


CTiglPointTranslator terminates with TIGL_MATH_ERROR

CTiglPointTranslator returns TIGL_MATH_ERROR in following example:
TEST(TiglPointTranslator, Bug5){
    CTiglPoint x1(4000.0000000000000, 0.00000000000000000, 3000.0000000000000);
    CTiglPoint x2(6893.7199999999993, 4250.0000000000000, 3000.0000000000000);
    CTiglPoint x3(7554.9955629914994, 0.00000000000000000, 3017.7749461533222);
    CTiglPoint x4(8693.7789747175666, 4250.0000000000000, 3006.3001935476932);

    CTiglPoint p(7060.6260619422628, 1062.5000000000005, 3011.1796935014363);

    double eta, xsi;

    CTiglPointTranslator trans(x1, x2, x3, x4);

    ASSERT_EQ ( TIGL_SUCCESS, trans.translate(p, &eta, &xsi) );
}

@RlanderRISCSW Thanks for the detailed report and test 😄
I am gonna put this into our test suite. I suppose the reason could be the scaled number (factor 1000) and a hard-coded abort criterium.

Issue is fixed. The reason was indeed due to the different scale by a factor of thousand. That lead to the problem that the gradient of the objective function was 1e+6 larger than usual and the abort criterium was wrong.
I fixed the issue, by first determining the scale of the problem and adapt the tolerance of the gradient with the probem's scale.


V0.10.x : Can't run xrdp-chansrv in console VNC session

xrdp version
0.10.x and later
Detailed xrdp version, build options
Pretty much any

Operating system & version
Any
Installation method
Doesn't matter
Which backend do you use?
x11vnc, typically
What desktop environment do you use?
various
Environment xrdp running on
Typically physical machine
What's your client?
Any
Area(s) with issue?
Clipboard, File transfer / drive redirection
Steps to reproduce

Log the console in using Xorg. Start x11vnc with something like:-

x11vnc -display $DISPLAY -forever -repeat

Make a note of the DISPLAY

Create a VNC-compatible stanza for xrdp in xrdp.ini, e.g.:-

[localvnc]
name=localvnc
lib=libvnc.so
ip=localhost
port=5900
pamusername=asktestuser
pampassword=ask
chansrvport=DISPLAY(1)
enable_dynamic_resizing=false

The above is for console display :1

Connect through from an RDP client

✔ Expected Behavior
Clipboard and drive redirection available
❌ Actual Behavior
Clipboard and drive redirection not available. Chansrv log contains lines like these:-
[2024-09-09T15:15:37.732+0100] [ERROR] [setup_api_listen(chansrv.c:1380)] setup_api_listen: trans_listen failed for port /var/run/xrdp/1001/xrdpapi_1
[2024-09-09T15:15:37.497+0100] [ERROR] [setup_listen(chansrv.c:1357)] setup_listen: trans_listen failed for port /var/run/xrdp/1001/xrdp_chansrv_socket_1

Anything else?
See #3237

The socket directory needs to be present when chansrv is started, or chansrv can't create its main listening socket.
One option might be to move the socket directory for configurations like this. This has a couple of disadvantages:-

The user has to create the directory in such a way that only the user and the xrdp process can access it. This is challenging to get right, and makes it hard for the xrdp team to re-design this directory.
If the directory moves, other components like the sound driver(s) need to be aware this has happened. Again,the onus to get this right is on the user, rather than the developer(s).

A better solution is for chansrv to ask sesman to create the socket directory via an additional SCP message. If chansrv finds the socket directory for the user to be missing, it can simply ask sesman to create it.
This does require sesman to be running. UDS authentication will be adequate for this, and no complicated run-time configuration is required by the user.
I'm going to draft a solution along these lines.