Sudden OSError: [Errno 28] No space left on device upon start up

Hi there,
upon trying to start composer-local today, I suddenly ran into the following error:
2023-02-16T09:48:07.877041427Z + sudo chown airflow:airflow airflow
2023-02-16T09:48:08.071326885Z + [ -f /var/local/setup_python_command.sh ]
2023-02-16T09:48:08.071863885Z + pip3 install --upgrade -r composer_requirements.txt
2023-02-16T09:48:10.541523761Z Defaulting to user installation because normal site-packages is not writeable
2023-02-16T09:48:11.546277762Z ERROR: Exception:
2023-02-16T09:48:11.546309178Z Traceback (most recent call last):
2023-02-16T09:48:11.546316387Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 228, in _main
2023-02-16T09:48:11.546321012Z     status = self.run(options, args)
2023-02-16T09:48:11.546323095Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cli/req_command.py", line 182, in wrapper
2023-02-16T09:48:11.546325095Z     return func(self, options, args)
2023-02-16T09:48:11.546330178Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/commands/install.py", line 282, in run
2023-02-16T09:48:11.546334845Z     wheel_cache = WheelCache(options.cache_dir, options.format_control)
2023-02-16T09:48:11.546341137Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cache.py", line 293, in __init__
2023-02-16T09:48:11.546346262Z     self._ephem_cache = EphemWheelCache(format_control)
2023-02-16T09:48:11.546352928Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cache.py", line 260, in __init__
2023-02-16T09:48:11.546359803Z     self._temp_dir = TempDirectory(
2023-02-16T09:48:11.546362678Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/utils/temp_dir.py", line 138, in __init__
2023-02-16T09:48:11.546364720Z     path = self._create(kind)
2023-02-16T09:48:11.546366428Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/utils/temp_dir.py", line 186, in _create
2023-02-16T09:48:11.546368303Z     tempfile.mkdtemp(prefix="pip-{}-".format(kind))
2023-02-16T09:48:11.546370220Z   File "/opt/python3.8/lib/python3.8/tempfile.py", line 358, in mkdtemp
2023-02-16T09:48:11.546372012Z     _os.mkdir(file, 0o700)
2023-02-16T09:48:11.546373845Z OSError: [Errno 28] No space left on device: '/tmp/pip-ephem-wheel-cache-d9vlgzcw'
2023-02-16T09:48:22.974223461Z + sudo chown airflow:airflow airflow
2023-02-16T09:48:23.134888378Z + [ -f /var/local/setup_python_command.sh ]
2023-02-16T09:48:23.135352545Z + pip3 install --upgrade -r composer_requirements.txt
2023-02-16T09:48:25.787094171Z Defaulting to user installation because normal site-packages is not writeable
2023-02-16T09:48:26.922569046Z ERROR: Exception:
2023-02-16T09:48:26.922626255Z Traceback (most recent call last):
2023-02-16T09:48:26.922631796Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cli/base_command.py", line 228, in _main
2023-02-16T09:48:26.922634713Z     status = self.run(options, args)
2023-02-16T09:48:26.922637963Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cli/req_command.py", line 182, in wrapper
2023-02-16T09:48:26.922648255Z     return func(self, options, args)
2023-02-16T09:48:26.922650130Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/commands/install.py", line 282, in run
2023-02-16T09:48:26.922652130Z     wheel_cache = WheelCache(options.cache_dir, options.format_control)
2023-02-16T09:48:26.922654088Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cache.py", line 293, in __init__
2023-02-16T09:48:26.922655880Z     self._ephem_cache = EphemWheelCache(format_control)
2023-02-16T09:48:26.922657588Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/cache.py", line 260, in __init__
2023-02-16T09:48:26.922659421Z     self._temp_dir = TempDirectory(
2023-02-16T09:48:26.922661046Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/utils/temp_dir.py", line 138, in __init__
2023-02-16T09:48:26.922662963Z     path = self._create(kind)
2023-02-16T09:48:26.922664713Z   File "/opt/python3.8/lib/python3.8/site-packages/pip/_internal/utils/temp_dir.py", line 186, in _create
2023-02-16T09:48:26.922666630Z     tempfile.mkdtemp(prefix="pip-{}-".format(kind))
2023-02-16T09:48:26.922668463Z   File "/opt/python3.8/lib/python3.8/tempfile.py", line 358, in mkdtemp
2023-02-16T09:48:26.922670213Z     _os.mkdir(file, 0o700)
2023-02-16T09:48:26.922671838Z OSError: [Errno 28] No space left on device: '/tmp/pip-ephem-wheel-cache-dsp1l3wy'
╭─ Error ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ Environment failed to start.                                                                                │
│                                                                                │
│ To print debug messages please add --debug flag.                                                                               │
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

No config change on my end, worked flawlessly before.
Not sure where the the disk size limit would suddenly come from.
Device disk definitely has enough space.

Do you have enough inodes on your disk? df -i

Maybe it's worth playing with: dm.basesize https://docs.docker.com/engine/reference/commandline/dockerd/
Could you list your requirements list? (I think this shows it breaks during packages installation).
Please check if limiting the dependencies list helps as well.

Closing as inactive


Improved security and error handling

@dufoli Some small modifications here to improve the security and error handling ...

@dufoli Please hold off on merging this ... I need to fix a couple more things.

And squash all commits too

Squashed the commits, ready for review and merge.  I was mistakenly trying to address one issue, but realized the behavior is already exactly as it's supposed to be.  If the check for @RolesAllowed fails, an HTTP status code of 500 is returned with the SoapFault without any meaningful fault message .. this can probably be handled better by the end users with an interceptor.

@shumonsharif maybe we need exception mapper as done on resteasy extension...

@shumonsharif I wait some feedback this week then I will provide the release because support of interceptor and security are huge step forward!

Yes, agree @dufoli I think ExceptionMapper route may be a good option, will take a look!


Different Tree Structures

Hello,
I am facing a weird issue.
On the Development PC, I have this in FlaUInspect:

On the Testing PC, I have this in FlaUInspect:

These 2 tree structures are very different. Moreover, seems on the Testing PC, I lost the FrameworkID property.
I know that on the Test PC, chromium is started using force-rendering-accessibility.
Any ideas if there are any other settings that I need to force onto Chromium, in order to be ok?
On does anyone if I need a setting change in Windows, so I do not need to force that setting into chromium?

Are the versions of Chromium the same? Also is the Windows Version the same?

The first one war run on Windows 10 Profesional. The second, was run on Windows 10 Embedded, with Forced Accessibility parameter on the browser (Chromium).


Missing ISO 3166-1 codes in NE 1:10m Admin1

name
iso31661a2
status

Bonaire
BQ

Bouvet Island
BV

Christmas Island
CX

Coco Islands
CC

French Guiana
GF
Fixed

Guadeloupe
GP

Martinique
MQ

Mayotte
YT

Reunion
RE

Svalbard and Jan Mayen
SJ

Tokelau
TK

Other:

name
(commonly used)
status

Kosovo
XK

I think this is because these islands are treated as part of France, Australia, New Zealand, Norway etc.


upgrade-zulip-from-git: Support specifying tag or commit ID for refname.

Fixes #10706.
Issue: Before this commit, the refname positional argument to
upgrade-zulip-from-git script would run successfully for a branch
name on the given remote, but the script would fail if it was
provided with a tag or commit ID.
Solution: git clone -q -b refname LOCAL_GIT_CACHE_DIR deploy_path
would be split into two commands:
1.) git clone -q LOCAL_GIT_CACHE_DIR deploy_path
2.) git checkout -b refname refname which makes a new branch
having same name as the refname.
In case of an invalid refname, following will be the output:
root@shubham-padia:~# /home/zulip/deployments/current/scripts/upgrade-zulip-from-git 57w348953
2018-10-22 22:32:13,426 upgrade-zulip-from-git: Fetching the latest commits
fatal: Cannot update paths and switch to branch '57w348953' at the same time.
Did you intend to checkout '57w348953' which can not be resolved as commit?
Traceback (most recent call last):
  File "/home/zulip/deployments/current/scripts/upgrade-zulip-from-git", line 73, in <module>
    subprocess.check_call(["git", "checkout", "-b", refname, refname], preexec_fn=su_to_zulip)
  File "/usr/lib/python3.5/subprocess.py", line 581, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['git', 'checkout', '-b', '57w348953', '57w348953']' returned non-zero exit status 128

@shubham-padia I think refname as the name of the branch isn't great; e.g. if you did master^, I'm not sure master^ is a valid branch name.  I think we want to instead just use a name like deploy-$(UNIXTIMESTAMP) for the branch; can you try that?  That's probably pretty safe against the branch name being present in the remote repo.

@timabbott I've updated the PR with the requested changes.
The branch name uses the same timestamp as the deploy_path for consistency.
e.g if the deploy_path is /home/zulip/deployments/2018-10-23-17-54-26, the branch name will be deploy-2018-10-23-17-54-26.

Merged, after changing that bit of logic you commentd on above to os.path.basename(), since that's the cleanest way to do string-parsing on paths.  Thanks @shubham-padia!


Support for Canvas plugin

It would be great if ExcaliBrain supported notes from the core Canvas plugin. I often use canvases for overview notes, so it's pretty important to document the relationships within canvas files.
The canvas files seem like a fairly straightforward JSON format. Excalibrain would need to extract "file" links as children, as well as parse markdown from any embedded notes for links and dataview directives.

Bumping this...


Can this approach work for Image Classification of Pretrained Model

Great article. I did this and got it to work. I tried to adapt this approach to my Image Classification model - which is named dnn_model.zip. For some reason I keep getting an error when I try to load the model. Below is my controller. Please provide recommendations. Thanks.
    [HttpPost]
    public ActionResult<string> Post([FromBody] ImageData input)
    {
        try
        {
            if (!ModelState.IsValid)
            {
                // return BadRequest();
                return Content("Invalid model.");
            }

            var context = new MLContext();

            ImagePrediction prediction = _predictionEnginePool.Predict(modelName: "ImageDnnModel", example: input);

            string imginfo = "Prediction value is: " + prediction.PredictedLabel.ToString() + ", Score is: " + prediction.Score.ToString();

            return Ok(imginfo);
        }
        catch (Exception e)
        {
            string errmsg = "Opps, an error happened: " + e.ToString();
            return Content(errmsg);
        }
    }

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

ID: 1dce49db-3548-7dad-bdb1-812efde7ac86
Version Independent ID: 390a9c7a-2537-25ca-c041-0284c253acff
Content: Deploy a model in an ASP.NET Core Web API - ML.NET
Content Source: docs/machine-learning/how-to-guides/serve-model-web-api-ml-net.md
Product: dotnet-ml
GitHub Login: @luisquintanilla
Microsoft Alias: luquinta

Hi @rossnoe
It should work. I believe the issue you're facing is you're using the schema from ImageData which if you follow the image classification tutorial is meant for images stored on disk. The model expects image to be a byte[] so you can do one of two things. Either create a pipeline that includes the LoadRawImageBytes transform or convert the image to byte[] and use that as input. You can see this sample for reference.
https://github.com/dotnet/machinelearning-samples/blob/master/samples/csharp/getting-started/DeepLearning_ImageClassification_Training/WebApp.Predict/Controllers/ImageClassificationController.cs

Thanks very much Luis! I will implement the sample tomorrow. It's great to see the improving capabilities of ML.net.


update buildah's organization

TravisCI [#1764]:

Issues :-(

Links

[L376] 301 https://github.com/projectatomic/buildah  →  https://github.com/containers/buildah

Dupes
None ✓

[#1764] https://travis-ci.org/veggiemonk/awesome-docker/builds/431568778

@agebhar1 thanks a lot!


Vertical + images[] starts at image 7

When using 'vertical:true' with  images array  the reel starts at frame (image) number 7 and continue past the end of the array with image url "undefined".

Thanks for the report. Will look into this issue.


unsupported operand type(s) for +=: 'int' and 'NoneType'

I got the error "unsupported operand type(s) for +=: 'int' and 'NoneType'" and temporarily fixed it
with vmtools.py
<             vm_size += disk.size
---
>             if disk.size is not None:
>                 vm_size += disk.size

Unfortunately I could not dig deeper into the error and do not know when it happens.

Thanks for sharing, added to the code


ModuleNotFoundError: No module named 'python-tableau-parser'

@pragdave, I was looking for an extension for colspan cells in tables and found your awesome extension, tried to install it
Building wheels for collected packages: mdx-tableau
  Building wheel for mdx-tableau (setup.py) ... done
  Created wheel for mdx-tableau: filename=mdx_tableau-1.0.1-py3-none-any.whl size=7479 sha256=20d01995c428515c96cf4a9e58d1b1aec37e81b303837b1272591bad6e660cdd
  Stored in directory: C:\Users\USERNAME\AppData\Local\Temp\pip-ephem-wheel-cache-eej_tt4r\wheels\34\52\4b\36693b3788c9af67f470f7220f5121a8e4b8ddef194094f2e0
Successfully built mdx-tableau
Installing collected packages: mdx-tableau
Successfully installed mdx-tableau-1.0.1

But could not run it for mkdocs:
ERROR    -  Config value 'markdown_extensions': Failed to load extension 'python-tableau-parser'.
            ModuleNotFoundError: No module named 'python-tableau-parser'
Aborted with 1 Configuration Errors!

I don't know how to fix it...
Could you post it in pypi, please 🙏

So, I installed string-scanner and import extension as
markdown_extensions:
    - mdx_tableau

And now it is works!

@reregaga any chance you could post a working example of a table source from a .md file mkdocs is properly building with mdx_tableau? I'm gotten to the point where I'm not getting any errors, but nothing I've tried in my markdown sources ever gets me a table that is processed by this extension.


Consider replacing Webpack with Vite

Feature Description
It might be a bit early, but I think it's worth investigating switching the build process to pure esbuild. It supports experimental code splitting and has a plugin ecosystem as well. It should build supremely fast.
Vite was suggested previously as a Webpack replacment but I think it's a dead-end because it relies on legacy systems like Rollup and Babel and seems to be generally header into a similar bloated direction like Webpack. Also, in terms of speed, Vite is barely faster than Webpack in my testing.
Screenshots
No response

https://github.com/evanw/esbuild/issues/1441 could be one potential blocker for us, given that we modify the public path prefix at runtime.

Can't we just build with public path {{PUBLIC_PATH}}/ and then just replace it in gitea server code either on gitea startup or on first usage before serving?

Yes there are ways. Probably just using the existing window.config mechanism would be the easiest.

Also, in terms of speed, Vite is barely faster than Webpack in my testing.

I thing this is mainly coming from the fact how the code is currently structured. Vites power is code-splitting and being fast on dynamically importing code on demand. In case of the Gitea code it seems to me that it loads the features a monolithic bundle and "only" uses dynamic imports for libraries atm, so Vite probably can't optimize that to well.
If Vue is a thing for Gitea, Vite should be preferred over esbuild from my exp considering the setup complexity for vue SFC building with native esbuild. Maybe https://vitejs.dev/guide/why.html#why-not-bundle-with-esbuild has some helping hints as well.

Maybe https://vitejs.dev/guide/why.html#why-not-bundle-with-esbuild has some helping hints as well.

That page looks a bit outdated to me, esbuild can now do CSS minification and code splitting (althought beta-level). Vue SFCs will probably require a plugin.

I guess Vite has matured enough to be a possible Webpack replacement. It will require quite a bit of configuration and probably some plugins, but I think it could be made to work.
I see Vite currently a possibly more "complete" version of esbuild, which still lacks code splitting.

Actually, there are at least two blockers I think:

Vite can't do multiple CSS entrypoints. The entrypoint config is provided by rollup and I think rollup can only do JS entrypoints.
license.txt generation will need to be rewritten or a suitable plugin to be found.

I don't see a compelling reason to switch to vite currently for Gitea. Our webpack setup is supremely fast currently at around 20s to build, and that includes heavyweights like monaco. Vite just does not seem to be made for complex projects that need multiple entry points and fine-grained configurability.

Let's reopen. Vite is still a goal and I think with some Rollup plugins to load CSS, we may be able to achieve what Webpack currently does.

Likely use https://github.com/mjeanroy/rollup-plugin-license for license.txt generation with vite.

Attempt to migrate to Vite was done in https://github.com/go-gitea/gitea/pull/25874, but it did not work out:

Vite, or more specifically Rollup is 2-3 times slower to bundle our dependencies, likely needs https://github.com/evanw/esbuild/issues/16 and adoption of esbuild bundling in vite to really speed it up
Vite is heavily geared towards SPAs, or at least apps where it can serve HTML. This will never be the case for gitea, so it's a bad fit and would require a ton of workarounds to get working.

Vite should be re-attempted again after they release a version based on rolldown, which should provide a level of speed suitable to us.

Maybe https://github.com/farm-fe/farm is worth a try in the future. It promises to alleviate all problems I had with vite (rollup slowness, code splitting and slow dev server because of nonexistant lazy-compilation).


Decentraland: Sell orders listing

Issue link
Refs: #59
Types of changes
New feature (non-breaking change that adds functionality)
Meta (a change that affects future changes to this repo)
Technical debt (a code change that doesn't fix a bug or add a feature but makes something clearer for devs)
Notes

Which is the better place to load config? HomeScreen.js, App.js or another place?

I've used a babel config to remove all console.* from code. The app is faster without those.
Refs: https://docs.expo.io/versions/latest/react-native/performance/#using-consolelog-statements
That config is recommended for production since console.* can be useful on development. But, since ethers.js prints a lot of warns it's making the development process extremely slow (e.g.: Listing screen without console.* is taking about 2s, with console.* it's taking >10s!).
Question: Since our SDK is also prints warnings. Should we silent only ethers.js?

I've changed ListView component to FlatList because performance too.
Refs: https://docs.expo.io/versions/latest/react-native/performance/#listview-initial-rendering-is-too-slow-or

Should I rename the word Land to SellOrder on components names?

How to get usd mana price? Static value or fetch from a market rate api?

I'm not sure about the better place to keep contract addresses and objects, should we have a global assignment for that kind of objects?

And about integration tests? Should we write some? I'm missing a test script since I've tested listing fetch on my mobile.

Thanks!

Oh oops, I just approved part of the diff and accidentally approved the whole PR

Which is the better place to load config? HomeScreen.js, App.js or another place?

App.js sounds good for now. Until that file gets too big, and then we can split it out.

I've used a babel config to remove all console.* from code. The app is faster without those.
Refs: https://docs.expo.io/versions/latest/react-native/performance/#using-consolelog-statements
That config is recommended for production since console.* can be useful on development. But, since ethers.js prints a lot of warns it's making the development process extremely slow (e.g.: Listing screen without console.* is taking about 2s, with console.* it's taking >10s!).
Question: Since our SDK is also prints warnings. Should we silent only ethers.js?

That's worth passing on as feedback to ethers.js, potentially.
Adding that Babel config is a great move.
Wow, that difference is dramatic. Yes, I think silencing ethers.js is the move in the short run. Or maybe not silencing it, but only passing through higher severity than console.warn

I've changed ListView component to FlatList because performance too.
Refs: https://docs.expo.io/versions/latest/react-native/performance/#listview-initial-rendering-is-too-slow-or

Great, thanks. And thanks for explaining this in the notes.

Should I rename the word Land to SellOrder on components names?

Hmm, I'd say it depends. If the things we're showing is mostly just the land as opposed to details about the order, then no. But if the component's level of abstraction is that of a piece of land plus details about the sale, then yes.

How to get usd mana price? Static value or fetch from a market rate api?

Good question. I'm inclined to punt on this for now and remove USD from the views until 0.2.0. But put a TODO in the code to add it back.

I'm not sure about the better place to keep contract addresses and objects, should we have a global assignment for that kind of objects?

Can you include some pseudocode to clarify what you mean? My initial reaction is that yes, we should have a global assignment for this.

And about integration tests? Should we write some? I'm missing a test script since I've tested listing fetch on my mobile.

You mean like clicking with Enzyme? Or something involving interacting with ganache?
I think we should add integration tests soon-ish, but I don't think it's the best use of time until 0.2.0 or 0.3.0. What do you think?

@marcelomorgado FYI there's a merge conflict after merging other PRs. Can you please take a look when you get a chance? Thanks!

@marcelomorgado Why not get the ABI from tasit-contracts?

I'm not sure about the better place to keep contract addresses and objects, should we have a global assignment for that kind of objects?

Can you include some pseudocode to clarify what you mean? My initial reaction is that yes, we should have a global assignment for this.

Yes, sure:
import { abi as estateABI } from "../abi/EstateRegistry.json";
import { abi as markplaceABI } from "../abi/Marketplace.json";
const { Contract } = Action;

const estateAddress = "0x41b598a2c618b59b74540ac3afffb32f7971b37a";
const marketplaceAddress = "0x07c0e972064e5c05f7b3596d81de1afd35457eae";
...
const estateContract = new Contract(estateAddress, estateABI);
const marketplaceContract = new Contract(marketplaceAddress, markplaceABI);

As we'll call these contracts from others components, I've thought to put them on an isolated class. For now they are living inside of ListLandsScreen.js. If so, which is the best practice, pattern for that?

And about integration tests? Should we write some? I'm missing a test script since I've tested listing fetch on my mobile.

You mean like clicking with Enzyme? Or something involving interacting with ganache?
I think we should add integration tests soon-ish, but I don't think it's the best use of time until 0.2.0 or 0.3.0. What do you think?

Please, tell me if I'm wrong but, I think that the unit tests from React apps, is to be only for test if rendering is working well, mocks and fakes should be used instead of a real blockchain/api/http call.

I've changed ListView component to FlatList because performance too.
Refs: https://docs.expo.io/versions/latest/react-native/performance/#listview-initial-rendering-is-too-slow-or

Great, thanks. And thanks for explaining this in the notes.

Should I rename the word Land to SellOrder on components names?

Hmm, I'd say it depends. If the things we're showing is mostly just the land as opposed to details about the order, then no. But if the component's level of abstraction is that of a piece of land plus details about the sale, then yes.

Hmm, got it. Said that, the ListLandsScreen is listing "estates sell orders" and the LandClaimScreen  and Land actually show price (a sell order info).

Open threads:

Rename components (Land vs. SellOrder);
Silent console.log & open ethers.js issue;
USD Removal.
Integration test bootstrap.
Could I handle with these on next PR to merge that sooner?

@marcelomorgado Yep, let’s handle those in a future PR

Open threads:
* Rename components (Land vs. SellOrder);

* Silent console.log & open `ethers.js` issue;

* USD Removal.

* Integration test bootstrap.
  Could I handle with these on next PR to merge that sooner?

@marcelomorgado I forget, do we have an issue for these open threads yet?

Not yet, I'll create, I got it.

https://github.com/tasitlabs/tasit/issues/84


Check if studytracks are updated correctly (for example LSI)

When dealing with grappa, got a message saying current Life Science Informatics master studytracks are:

Bioinformatics and Systems Medicine
Biomathematics
Biostatistics
Ecological Informatics
Systems biology and medicine

But oodikone differs from these a bit

These are probably ok. The ones with 0 stats are filtered out from the selector. Assume everything's fine until someone says it isnt


Allow method chaining for setters

Given a message like
message Foo {
    double bar = 1;
    bool baz = 2;
}

right now we have to write
let mut foo = Foo::new();
foo.set_bar(100.0);
foo.set_baz(true);

while it would be nicer to write
let mut foo = Foo::new();
foo.set_bar(100.0).set_baz(true);

Coupled with #177, this would allow adding an element in the example message
message Foo {
    bool initialized = 1;
}
message FooBar {
    repeated Foo foo = 1;
}

to go from
// msg: &mut FooBar is in scope
let foo = Foo::new();
foo.set_initialized(true);
msg.mut_foo().push(foo);

to the much shorter
// msg: &mut FooBar is in scope
msg.add_foo().set_initialized(true);

The API change would be to make all set_foo methods return &mut self.

I personally think chained setters are wrong. However, because this request is popular, I think rust-protobuf should probably have an option to generate chained setters.
Also, note that single-liners are possible for your last use case:
msg.mut_foo().push_default().set_initialized(true);

My ancient PR #101 tried to add this back in April 2015, but @stepancheg  had strong opinions on chaining back then too :P .

Is addingmsg.with_foo(true).with_bar(10.0) , where with_* consumes self alright?

I personally think that chained setters is not a very good think, it's probably OK to have a codegen option to generate chained setters.

Does this mean that you would accept a PR that adds an option to enable with_XY methods that consume self?

It it will be option which is off by default, and if pr is with test, then definitely yes.
I think that chained setters should probably be called with_xx.

Okay, good to hear! I will try to implement that then :)

Unfortunately, I will have to withdraw my offer for implementing that, sorry.

dfdcff

Closing due to old age.


Globally rename Assessor Portal to Assessor Workbench

🐛 Summary
The cisagov/assessor-portal-packer repo was renamed to cisagov/assessor-workbench-packer, and the name of the resulting AMI was changed in cisagov/assessor-workbench-packer#31 from assessor-portal-hvm-${local.timestamp}-x86_64-ebs to assessor-workbench-hvm-${local.timestamp}-x86_64-ebs.  The new name more closely matches what folks are calling the tool.  The AMI name was unfortunately not updated in this repository, so the refreshed AMIs with the newer name are not being used.
The AMI name should be updated, and all resource names, etc. that reference the old "Assessor Portal" language should be updated to instead reference "Assessor Workbench".  Note that the name of the instance type corresponding to the Assessor Workbench would need to be updated in the inbound_ports_allowed and operations_instance_counts input variables, so this is a breaking change.

Should be closed by this PR:
https://github.com/cisagov/cool-assessment-terraform/pull/216


The prediction is not working properly

Describe the bug
After the training is done, the prediction of next word (using python test.py) doesn't work at all, the prediction results are always the same for any input sentence
To Reproduce
Steps to reproduce the behavior:

python test.py

Expected behavior
The result of prediction should make sense
Screenshots
$ python test.py
...
Enter atleast 3 words:
i am hungry
Word  hungry  does not exist
indexes of last 3 words : [68, 1632, 1]
prediction indexes      : [2, 0, 0, 0, 0, 0, 0, 0, 0, 0]
1 - i am hungry .
2 - i am hungry the
3 - i am hungry the
4 - i am hungry the
5 - i am hungry the
6 - i am hungry the
7 - i am hungry the
8 - i am hungry the
9 - i am hungry the
10 - i am hungry the
Enter atleast 3 words:
medicine has existed for thousands of
indexes of last 3 words : [11, 1563, 4]
prediction indexes      : [2, 0, 0, 0, 0, 0, 0, 0, 0, 0]
1 - medicine has existed for thousands of .
2 - medicine has existed for thousands of the
3 - medicine has existed for thousands of the
4 - medicine has existed for thousands of the
5 - medicine has existed for thousands of the
6 - medicine has existed for thousands of the
7 - medicine has existed for thousands of the
8 - medicine has existed for thousands of the
9 - medicine has existed for thousands of the
10 - medicine has existed for thousands of the
Enter atleast 3 words:
the world is not enough
indexes of last 3 words : [13, 63, 543]
prediction indexes      : [2, 0, 0, 0, 0, 0, 0, 0, 0, 0]
1 - the world is not enough .
2 - the world is not enough the
3 - the world is not enough the
4 - the world is not enough the
5 - the world is not enough the
6 - the world is not enough the
7 - the world is not enough the
8 - the world is not enough the
9 - the world is not enough the
10 - the world is not enough the

More info:
I use Keras 2.1.6 and tensorflow-gpu 1.10.0. I changed also the batch_size in config file from 20 to 100 to speed up the training time.

@cahya-wirawan
14 epochs are not good for the model, it was just done for testing purpose.
So, I have now also changed the CONFIG.py.
Also updated the test.py, the same word issue is fixed now.
Please let me know is there are any further issues.
Thanks for filing an issue.


Norm dotnet in azure

Using Azure to quickly host apps

:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:warning:Warning

Details

msteams-platform/get-started/get-started-dotnet-in-azure.md
:warning:Warning

Details

msteams-platform/get-started/get-started-dotnet.md
:warning:Warning

Details

msteams-platform/get-started/get-started-nodejs.md
:warning:Warning

Details

msteams-platform/assets/images/get-started/PublishToAzure1.png
:white_check_mark:Succeeded

msteams-platform/assets/images/get-started/PublishToAzure2.png
:white_check_mark:Succeeded

msteams-platform/assets/images/get-started/PublishToAzure3.png
:white_check_mark:Succeeded

msteams-platform/assets/images/get-started/PublishToAzure4.png
:white_check_mark:Succeeded

msteams-platform/assets/images/get-started/PublishToAzure5.png
:white_check_mark:Succeeded

msteams-platform/assets/images/get-started/azure-cloud-shell.png
:white_check_mark:Succeeded

msteams-platform/TOC.yml

[Warning] Invalid file link:(~/resources/dev-preview/developer-preview-intro).
[Warning] Invalid file link:(~/resources/design/framework/search).
[Warning] Invalid file link:(~/resources/resource-overview).
[Warning] Invalid file link:(~/concepts/bots/bots-test).
[Warning] Invalid file link:(~/resources/general/cross-domain).
[Warning] Invalid file link:(~/concepts/activity-feed).
[Warning] Invalid file link:(~/concepts/bots/bots-overview).
[Warning] Invalid file link:(~/concepts/bots/bots-conversations).
[Warning] Invalid file link:(~/concepts/tabs/tabs-context).
[Warning] Invalid file link:(~/concepts/tabs/tabs-content).
[Warning] Invalid file link:(~/resources/design/components/color).
[Warning] Invalid file link:(~/concepts/cards-actions).
[Warning] Invalid file link:(~/scenarios/scenarios).
[Warning] Invalid file link:(~/resources/design/framework/apps).
[Warning] Invalid file link:(~/concepts/outgoingwebhook).
[Warning] Invalid file link:(~/concepts/connectors).
[Warning] Invalid file link:(~/resources/design/resources).
[Warning] Invalid file link:(~/resources/design/framework/submitting).
[Warning] Invalid file link:(~/resources/design/framework/basics).
[Warning] Invalid file link:(~/concepts/tabs/tabs-configuration).
[Warning] Invalid file link:(~/samples/code-samples).
[Warning] Invalid file link:(~/concepts/bots/bots-notifications).
[Warning] Invalid file link:(~/resources/schema/manifest-schema).
[Warning] Invalid file link:(~/concepts/apps/apps-package).
[Warning] Invalid file link:(~/get-started/get-started-tenant).
[Warning] Invalid file link:(~/resources/design/overview).
[Warning] Invalid file link:(~/resources/design/framework/cards).
[Warning] Invalid file link:(~/resources/design/framework/tabs).
[Warning] Invalid file link:(~/resources/dev-preview/developer-preview).
[Warning] Invalid file link:(~/resources/design/framework/bots).
[Warning] Invalid file link:(~/concepts/concepts-overview).
[Warning] Invalid file link:(~/troubleshoot/faq).
[Warning] Invalid file link:(~/scenarios/bots-personal-conversations).
[Warning] Invalid file link:(~/concepts/tabs/tabs-update-remove).
[Warning] Invalid file link:(~/concepts/bots/bots-menus).
[Warning] Invalid file link:(~/get-started/get-started).
[Warning] Invalid file link:(~/concepts/bots/bots-create).
[Warning] Invalid file link:(~/get-started/get-started-nodejs).
[Warning] Invalid file link:(~/resources/design/components/fields).
[Warning] Invalid file link:(~/resources/design/framework/activity-feed).
[Warning] Invalid file link:(~/resources/design/framework/messaging-extensions).
[Warning] Invalid file link:(~/scenarios/bots-channel-conversations).
[Warning] Invalid file link:(~/publishing/apps-publish).
[Warning] Invalid file link:(~/resources/dev-preview/developer-preview-features).
[Warning] Invalid file link:(~/resources/general/rate-limit).
[Warning] Invalid file link:(~/resources/design/components/typography).
[Warning] Invalid file link:(~/resources/microsoft-graph).
[Warning] Invalid file link:(~/publishing/office-store-guidance).
[Warning] Invalid file link:(~/concepts/apps/apps-overview).
[Warning] Invalid file link:(~/concepts/deep-links).
[Warning] Invalid file link:(~/resources/design/components/buttons).
[Warning] Invalid file link:(~/resources/design/components/dialogs).
[Warning] Invalid file link:(~/resources/general/requirements).
[Warning] Invalid file link:(~/concepts/authentication).
[Warning] Invalid file link:(~/publishing/office-store-checklist).
[Warning] Invalid file link:(~/concepts/messaging-extensions).
[Warning] Invalid file link:(~/.general/general-overview).
[Warning] Invalid file link:(~/overview).
[Warning] Invalid file link:(~/resources/design/components/themes).
[Warning] Invalid file link:(~/resources/general/debug).
[Warning] Invalid file link:(~/get-started/get-started-app-studio).
[Warning] Invalid file link:(~/get-started/get-started-dotnet).
[Warning] Invalid file link:(~/feedback).
[Warning] Invalid file link:(~/troubleshoot/troubleshoot).
[Warning] Invalid file link:(~/get-started/get-started-dotnet-in-azure).
[Warning] Invalid file link:(~/concepts/bots/bots-cards).
[Warning] Invalid file link:(~/concepts/tabs/tabs-static).
[Warning] Invalid file link:(~/concepts/tabs/tabs-overview).
[Warning] Invalid file link:(~/concepts/apps/apps-upload).
[Warning] Invalid file link:(~/concepts/bots/bots-context).

msteams-platform/get-started/get-started-dotnet-in-azure.md

Line 25: [Warning] Invalid file link:(~/get-started/get-started-dotnet).

msteams-platform/get-started/get-started-dotnet.md

Line 10: [Warning] Invalid file link:(~/get-started/get-started-dotnet).
Line 11: [Warning] Invalid file link:(~/get-started/get-started-nodejs).
Line 20: [Warning] Invalid file link:(~/get-started/get-started-tenant).
Line 119: [Warning] Invalid file link:(~/get-started/get-started-dotnet-in-azure).
Line 131: [Warning] Invalid file link:(~/concepts/apps/apps-overview).
Line 139: [Warning] Invalid file link:(~/concepts/bots/bots-create).
Line 165: [Warning] Invalid file link:(~/concepts/apps/apps-upload).

msteams-platform/get-started/get-started-nodejs.md

Line 10: [Warning] Invalid file link:(~/get-started/get-started-dotnet).
Line 11: [Warning] Invalid file link:(~/get-started/get-started-nodejs).
Line 20: [Warning] Invalid file link:(~/get-started/get-started-tenant).
Line 178: [Warning] Invalid file link:(~/concepts/apps/apps-overview).
Line 186: [Warning] Invalid file link:(~/concepts/bots/bots-create).
Line 218: [Warning] Invalid file link:(~/concepts/apps/apps-upload).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Recurring conda error with clean install

I get the following error when attempting a clean install of bcbio (as I was encountering issues with the multiarray module from numpy related to libquadmath.so.0 in my previous install that I could not resolve):
UnsatisfiableError: The following specifications were found to be in conflict:

bcbio-nextgen 1.0.8
libcurl
Use "conda info " to see the dependencies for each package.

Traceback (most recent call last):
File "./bcbio_nextgen_install.py", line 260, in
main(parser.parse_args(), sys.argv[1:])
File "./bcbio_nextgen_install.py", line 38, in main
bcbio = install_conda_pkgs(anaconda, args)
File "./bcbio_nextgen_install.py", line 81, in install_conda_pkgs
"--file", os.path.basename(REMOTES["requirements"])])
File "/home/aadam/miniconda3/lib/python3.6/subprocess.py", line 291, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/home/aadam/bcbio/anaconda/bin/conda', 'install', '--yes', '-c', 'conda-forge', '-c', 'bioconda', '--file', 'requirements-conts-conda.txt']' returned non-zero exit status 1.
Running 'conda info' on the respective packages as instructed results in 'NoPackagesFoundError: Package missing in current linux-64 channels'.  However, when searching 'libcurl' in my bcbio directory, I find an entire libcurl directory under 'anaconda/pkgs'.   Not sure how to continue with installation.

Sorry for the issue and thanks for the report. Can you provide more details about how you're installing bcbio? It looks from the traceback like you're trying to manually install bcbio inside of a python3 conda environment. bcbio only supports python2 and we recommend using the automated installer:
https://bcbio-nextgen.readthedocs.io/en/latest/contents/installation.html#automated
which will setup the correct conda environment. Does that help explain the issues you're seeing?

Thanks for your reply.  I commented out miniconda3 from my .bashrc profile and loaded a python 2.7.9 environment module on a new cluster node,  I used the following command:
python ./bcbio_nextgen_install.py ~/bcbio --tooldir=~/bcbio/
And still get the following error, though without the conda environment error.
UnsatisfiableError: The following specifications were found to be in conflict:

bcbio-nextgen 1.0.8
libcurl
Use "conda info " to see the dependencies for each package.

Traceback (most recent call last):
File "./bcbio_nextgen_install.py", line 260, in
main(parser.parse_args(), sys.argv[1:])
File "./bcbio_nextgen_install.py", line 38, in main
bcbio = install_conda_pkgs(anaconda, args)
File "./bcbio_nextgen_install.py", line 81, in install_conda_pkgs
"--file", os.path.basename(REMOTES["requirements"])])
File "/opt/software/python/2.7.9/lib/python2.7/subprocess.py", line 540, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/home/aadam/bcbio/anaconda/bin/conda', 'install', '--yes', '-c', 'conda-forge', '-c', 'bioconda', '--file', 'requirements-conda.txt']' returned non-zero exit status 1

Thanks much for following up and the help debugging. I tried to replicate this on a clean machine and can't seem to replicate the issue. My guess is that an environmental variable set in your bashrc or modules is injecting a different conda environment with an incompatible libcurl. Unfortunately the error message doesn't give much idea where this might come from. You could try exploring with conda env list which might provide a clue about overlapping environments:
https://conda.io/docs/user-guide/tasks/manage-environments.html#activating-an-environment
Hope this helps some to help identify the issue. Thanks again.

I am having the exact same issue on CentOS citrus machine from our private cloud, after clean install, and trying to run:
bcbio_nextgen.py upgrade -u stable --tools --data
UnsatisfiableError: The following specifications were found to be in conflict:

bcbio-nextgen 1.0.8
libcurl
Use "conda info " to see the dependencies for each package.

Traceback (most recent call last):
File "/scratch/bcbio/anaconda/bin/bcbio_nextgen.py", line 222, in
install.upgrade_bcbio(kwargs["args"])
File "/scratch/bcbio/anaconda/lib/python2.7/site-packages/bcbio/install.py", line 66, in upgrade_bcbio
_update_conda_packages()
File "/scratch/bcbio/anaconda/lib/python2.7/site-packages/bcbio/install.py", line 241, in _update_conda_packages
"-c", "bioconda", "-c", "conda-forge", "--file", req_file])
File "/scratch/bcbio/anaconda/lib/python2.7/subprocess.py", line 186, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/scratch/bcbio/anaconda/bin/conda', 'install', '--quiet', '--yes', '-c', 'bioconda', '-c', 'conda-forge', '--file', 'bcbio-update-requirements.txt']' returned non-zero exit status 1
I hope you can help

Thanks for the additional report and sorry about these issues. I'm confused as to what is going on and not sure why I can't replicate this, but I just pushed a new 1.0.9 release with some conda resolution fixes that might avoid the issue. Would you be able to retry and see if that helps avoid the problem? If not, it would be useful to know exactly how you installed the initial version so I can see if I'm missing something in my attempt to reproduce? Thanks much for the help debugging.

Hi chapmanb,
Thanks for providing us bcbio pipeline. I'm trying to re-install it after removing my previous installation, but I got this error as well.
nohup.txt

version 1.0.9 solved the conda issue for me. Thanks for addressing this.


Change skins in demo.

Hello, first of all thanks a lot for your work I really appreciate it.
its working all good in matches but i was wonder is there any way to make it work on demo/gotv (my gameplays)  if there is then please help me to make it work.
Thanks again.

You need to change to hook all players instead of one, and apply skins on players based on xuid, since you are no longer the local player.
I considered implementing this, but haven't got around yet

I have no idea what you're talking about since i dont know how this work i only know how to build it to get the .dll file
Oh man if you cant do it than its pretty much impossible for me to do such since i got no knowledge about this stuff :/
i was hoping it would be as simple as changing some lines in the visual studio ... :/

in order to do that i must have skins so its a no go :/
man there has to be a simpler way to make this awesome skin changer work in demos :/

He explained how to do it...
"You need to change to hook all players instead of one, and apply skins on players based on xuid, since you are no longer the local player."

Thats the thing idk how to hook all players and whats xuid so im still searching around and not getting anything on this :/

We actually made a fork a while ago, that is outdated (aka not working anymore) now, that added an XUID feature:
https://github.com/advancedfx/nSkinz/tree/advancedfx
I would have made a pull request, but I think the quality of the code I added doesn't live up to your standards.
If the owner of this repository wants, I can try to reapply it and maybe improve the quality of my own code in there, but not sure if it will be good enough.


Adjust progress bar to reflect limited song length

This may not be possible with Amplitude.js - unless we trick it by drawing bar wider and hiding some of it,

closed by pull request #60


file explorer

Type: Feature Request
i cant create a new folder, also can't view my side bar where i can find source code and open a new folder
VS Code version: Code 1.80.1 (74f6148eb9ea00507ec113ec51c489d6ffb4b771, 2023-07-12T17:22:07.651Z)
OS version: Windows_NT x64 10.0.19045
Modes:

I suggest you press F1 to get the Command Palette, then run View: Reset View Locations

I press f1 but directed me to browser direct me to browser not showing the command palette

Try CTRL + SHIFT + P

Closing for lack of update


Checkmarx AST – Scan Summary & Details – fd09945d-b47d-480f-98e6-33782763a0ab
No New Or Fixed Issues Found


18.04 Fresh Install - Links on page are absolute, not relative causing issue with HTTPS

For Bug Reports

BookStack Version (Found in settings, Please don't put 'latest'): 0.22
PHP Version: 7.2.5
MySQL Version: 10.1.29-MariaDB-6

Expected Behavior
Running HAProxy in front of bookstack to handle SSL offloading.
When going to https://bookstack.mywebsite.com the site should work as it. The underlying HTML should point to all files either relative or absolute to include https in the address. This would cause the browser to accept the as it is not trying to load data from a non https location.
Current Behavior
Go to https://bookstack.mywebiste.com and browser gives error for mixed content. This causes an SSH error.
This can be verified by checking the source of the page. Example source for style sheet looks like this:
<link rel="stylesheet" href="http://bookstack.mywebsite.com/dist/styles.css?version=v0.22.0">

In 0.21 using the docker image, this worked fine and the style sheet code looked like this:
<link rel="stylesheet" href="/dist/styles.css?version=v0.21.0">

Steps to Reproduce
Use 18.04 installer for 0.22, but modified to remove mysql and add entries for the current mysql db.
HAProxy is setup to use the backend server on port 80. The URL rewrite is working correctly.
Navigate to new frontend defined in HAProxy to handle SSL on port 443.

Hi @rohitpaul, Sorry to hear you are having problems.
Can you try setting APP_URL=https://bookstack.mywebiste.com in the .env file or as an environment variable?
This should force all URL's written by BookStack to start with this base domain.

@ssddanbrown thanks that works!
Can I get some help with added sed command. Not sure how to update the command.
sed -i.bak 's/# APP_URL=.*$/APP_URL=https://bookstack.mywebsite.com/' .env

Thanks for your help!


INT-B-20777 TIO Queue Management

Agility ticket
Summary
Adds Queue Management to the TIO queue.
Contains B-20531
How to test
FF Off

Access the Office App
Login as TIO
There should be no Assigned column

FF On

Access the Office App
Login as TIO
There should be an Assigned column
Click the dropdown in the assigned column and see the office user you're logged in as
Click your name
refresh the page
Assigned column now has a value

Sort and filter on the assigned column are not working for me.

This is an issue.  I have a move assigned to my counseling office.  I am a TIO supervisor.  I first went in as someone not in that counseling office and assigned myself.  Then logged in as the supervisor.  My assigned dropdown list should have that assigned person (Sarah Smith), and my list of people in my office.
1st assignment:

Logging in as supervisor:

Test coverage on client failed.
Last run: Tue 22 Oct 2024 03:14:20 AM UTC
Refer to this report to see coverage details for the client app.
Documentation for test coverage can be found at this link.

Sorting on the Assigned column is still breaking for me.

Test coverage on client failed.
Last run: Thu 24 Oct 2024 12:57:00 PM UTC
Refer to this report to see coverage details for the client app.
Documentation for test coverage can be found at this link.


Is it possible to upload videos?

I realized that with .uploadPhoto() you can only upload photos, not videos. As of right now, there's no way to upload videos, right? Can someone confirm this?

my video is raw from smartphones but i can't upload video directly to instagram.

WTF...


Convert ems to px for mobile size buttons

Changes

Convert ems to px for mobile size buttons
Convert ems to px for search and cta padding to fix vertical alignment.

Testing

pull branch and gulp clean && gulp build
Visit homepage and resize down to mobile.
Expand mega menu hamburger menu and search menu and see that they are the same height.
Go into Settings > Customize fonts and change the font size and re-check the mobile size and see that the menu and search buttons are the same size. Compare to production to see that in production that are virtually unusably large.

Screenshots
Before:

After:

Also fixes global search and cta being misaligned:
Before:

After:

@wpears @ascott1 verbally approved this.


Simple change to mkDerivation to make it hard to accidentally get a huge closure

I was thinking of adding a simple flag to the generic mkDerivation called allowCcDependency (better names welcome), to prevent accidental dependencies on gcc/clang. The idea would be to have it add stdenv.cc to disallowedReferences by default, unless allowCcDependency is turned on for that particular package.
The reasoning is that very few packages actually need to retain a runtime dependency on the compiler that produced them, and we've had a few instances in the past where we accidentally ended up with huge closures because of that. If people support this idea, I'd submit a PR with the flag and set the flag in all packages that appear to need it.
I'm fine with treating the compiler specially because first, it's an "implicit" dependency you get through the stdenv, and second because it tends to be huge compared to other dependencies. Going forward, I'd like to think of other ways to prevent accidental dependencies on other undesired packages.
I'm tagging with closure-size because it's related to the need for smaller closures, but doesn't actually depend on anything in that branch.
cc @vcunat
https://github.com/NixOS/nixpkgs/issues/5895

Such a single special case? On some platforms libstdc++ and such libs aren't even split away from cc. It might be easier to create closure-size "tests", e.g. impose a hard limit on the size of certain (interesting) closures. Hydra nicely graphs sizes over time for quite a long now, but I suppose hardly anyone checks those.
A creeping problem with (dis)allowed references: we don't allow to have per-output restrictions, only conditions on all outputs at once. (Same for hydra's graphs, actually.)

Which platforms don't split them, and why does that affect this? I don't see any other way to impose a hard "you don't want this" limit today. The basic wisdom I want to encode is that I think we all have a reasonable expectation that most packages shouldn't depend on their compiler. It wouldn't be very hard to tell the computer about that.

I assumed the "impure" platforms don't have it split, even on the closure-size branch. Maybe they are rather broken in there, I don't know ATM, and it's no strong point anyway.

Ah, that might be true. I haven't looked at any of those in a while. I could easily make this possible to disable on a per-stdenv basis.

This is an instance of #15004

:+1:

Here's an example of adding cc to disallowedRequisites: https://github.com/NixOS/nixpkgs/commit/c03326445b067dca37ea323d998ffa3d520adb6d


Kebechet update

Update manager ran @09/16/2022, 11:28:31 , Pull Request: https://github.com/shreekarSS/aicoe-osc-demo/pull/8. Management complete.


[Console] SAML SSO Authenticator summary details are incorrect

Describe the issue:
Additional Query Parameters details are incorrect in the summary page of the authenticator.

How to reproduce:

Log in to the Console and click on Develop tab
Click on "Identity Providers"
Click on "New Identity Provider" and select "Google"
Provide the required details and create an IdP
Click on Update icon in the created IdP
Go to the Authentication tab and click "New Authenticator"
Select "SAML SSO"
Add required details including Additional Query Parameters
Click Next

Environment information

Product Version: 5.12.0-M1
OS: Mac
Database: MySQL 5.7
Browser: Safari - 14v

Since this is related to query parameters, the ideal way of showing them is in a comma separated way so that the user has the ability to easily identify what the query parameters are. If wi tend to show them in such a way as the custom tags are shown, it'll mislead the users into thinking that query parameters are custom tags as well. Hence will not fix this issue and will be closed.


How to implement "expand all children"?

in a tree view I would like to make it so that a normal click opens the item, and alt+click open the item and all its descendants. – how would I go about implementing this?

apparently there is an ADD_BY_ID_RECURSIVELY action type, which sounds interesting; but how do I dispatch it in my code?

https://github.com/table-library/react-table-library/releases/tag/v4.0.11 implements this feature. On https://react-table-library.com/ check out the Stories:

Features / Tree / Toggle All
Features / Tree / Toggle Recursively

In general you will find all the available functions on the returned object from the feature hook. E.g. const tree = useTree(...) and then console.log(tree.fns).
I hope this helps!

@rwieruch great, thank you!
I had seen tree.fns but was probably thrown off by the naming: on[...] made me think it's an event handler of sorts. should have looked more closely!

You may be right her, thanks! What would be a good naming?

I think simply dropping the on prefix might already be enough: onRemoveByIdRecursively → removeByIdRecursively
what do you think?

https://github.com/table-library/react-table-library/releases/tag/v4.0.11 implements this feature (install 4.0.11).
On https://react-table-library.com/ check out the Stories:

Features / Tree / Toggle All
Features / Tree / Toggle Recursively

In general you will find all the available functions on the returned object from the feature hook. E.g. const tree = useTree(...) and then console.log(tree.fns).
I hope this helps!

hi,
    tree.fns.onToggleAll({
      isCarryForward: true,
      isPartialToAll: true,
    });

call twin will disable expand all,
so, how to keep all tree expand at refresh current tree ??


grep 3.1 regression

After msys2 update i686 mingw gcc build tests began to fall with error:
terminate called after throwing an instance of 'int'
terminate called recursively

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.

I've managed to track down that grep update from 3.0 to 3.1 causes this problem.
On the other hand, after comparison of build logs, i found out than for some gcc libs libtool generates wrong configs:
grep 3.0:
# Dependencies to place before and after the objects being linked to
# create a shared library.
predep_objects="C:/msys64/mingw32/i686-w64-mingw32/lib/dllcrt2.o C:/temp/i686-8b-posix-dwarf-rt_v5/build/gcc-8-branch/gcc/crtbegin.o"
postdep_objects="C:/temp/i686-8b-posix-dwarf-rt_v5/build/gcc-8-branch/gcc/crtend.o"
predeps=""
postdeps="-lmingw32 -lgcc_s -lgcc -lmoldname -lmingwex -lmsvcrt -lpthread -ladvapi32 -lshell32 -luser32 -lkernel32 -liconv -lmingw32 -lgcc_s -lgcc -lmoldname -lmingwex -lmsvcrt"

grep 3.1
# Dependencies to place before and after the objects being linked to
# create a shared library.
predep_objects="C:/msys64/mingw32/i686-w64-mingw32/lib/dllcrt2.o C:/temp/i686-8b-posix-dwarf-rt_v5/build/gcc-8-branch/gcc/crtbegin.o"
postdep_objects=""
predeps=""
postdeps="-lmingw32 -lgcc_s -lgcc -lmoldname -lmingwex -lmsvcrt -lpthread -ladvapi32 -lshell32 -luser32 -lkernel32 -liconv -lmingw32 -lgcc_s -lgcc -lmoldname -lmingwex -lmsvcrt"

Looks like real cause of the problem is one of this changes in grep 3.1:
The following changes affect only MS-Windows platforms.  First, the
  --binary (-U) option now governs whether binary I/O is used, instead
  of a heuristic that was sometimes incorrect.  Second, the
  --unix-byte-offsets (-u) option now has no effect on MS-Windows too.

This is upstream grep changes:
http://git.savannah.gnu.org/cgit/grep.git/commit/?id=5ac3d87a69122d45a837840914c2f9e66b5bb8d4

Some new info about this bug.
I wrote script that dumps grep args, stdin and stdout:
#!/bin/bash
indata=`timeout 15 cat`
grep3binary=grep3.exe
grep31binary=grep31.exe
grep3outfile=/a/grep3.txt
grep31outfile=/a/grep31.txt
echo args: "$@" >> "$grep3outfile"
[[ -n "$indata" ]] && echo stdin: "$indata" >> "$grep3outfile"
echo output: >> "$grep3outfile"
[[ -n "$indata" ]] && ( echo "$indata" | "$grep3binary" "$@" >> "$grep3outfile" 2>> "$grep3outfile" ) || ( "$grep3binary" "$@" >> "$grep3outfile" 2>> "$grep3outfile" )
echo args: "$@" >> "$grep31outfile"
[[ -n "$indata" ]] && echo stdin: "$indata" >> "$grep31outfile"
echo output: >> "$grep31outfile"
[[ -n "$indata" ]] && ( echo "$indata" | "$grep31binary" "$@" >> "$grep31outfile" 2>> "$grep31outfile" ) || ( "$grep31binary" "$@" >> "$grep31outfile" 2>> "$grep31outfile" )

[[ -n "$indata" ]] && ( echo "$indata" | "$grep3binary" "$@" ) || ( "$grep3binary" "$@" )

Dumped all info and all info in grep 3 and grep 3.1 is the same(except grep version numbers), but when i switch binary to 3.1 i'm getting bug again. Tested on libstdc++-v3, the only difference is config.status and libtool files.
grep3.log
grep31.log
libstdc++-v3_grep3.zip
libstdc++-v3_grep31.zip
I don't know how it's possible to get same dumps, but different configs for different grep versions. Maybe i should dump some other info?

@Furniel no need any info. I send you link for changes. I dont plan to look at this issue at near time. Feel free to create patch and sent pull requrest

grep downgraded in repo to 3.0


[TextField] red error text fails a11y contrast ratio requirement of 4.5:1

Even on a white background, the error text for [TextField] is flagged at < 4.5:1 ratio, unacceptable for accessibility

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

The default red error text is too light against the best-case white background
Expected Behavior 🤔
It should be at least 4.5:1 on a white background;  Default behavior on hover often darkens the background, requiring an even darker red to pass

Steps to Reproduce 🕹

Steps:

https://next.material-ui.com/components/text-fields/
Go to Validation section where there's demo of red error text
I'll attach a pic where I open dev tools and use Lighthouse element inspector to inspect the text and it flags the contrast violation

Context 🔦
a11y testing; conformance required by customers

Your Environment 🌎
Your demo page (see url above, step 1); also happens on latest 4.x

Here's a work-around I will do:  Add to createMuiTheme:
MuiFormLabel: {
            root: {
                "&$error": {
                    color: COLOR_DM_BG_LIGHTEST_RED,
                }
            }
        },
        MuiFormHelperText: {
            root: {
                "&$error": {
                    color: COLOR_DM_BG_LIGHTEST_RED,
                }
            }
        },

where:
export const COLOR_DM_BG_LIGHTEST_RED = "#DE3421";


greensun_hybrid.yaml  Please add this new inverter definition

New Solar Inverter definition I made and use for my own inverter
greensun_hybrid.yaml
greensun_hybrid.yaml.zip

Thanks a lot.
How did you do it? Did you already know the registers before making it?
It would be wonderful if there were any step by step on how to do it...
I would like to do it for my Sofar inverter.

Hi,
I bugged the company until they sent me a spreadsheet with some chinese and
register information.  I still had to decode a lot and I only mapped the
data that I was needing.
There is still some info i'm not sure of like alarm codes and I did not add
info for a second battery since I don't have one yet.
Attached is the version of the spreadsheet I made more readable but still
not easy to figure out... The original was worse.... but this gives you an
idea of what I started with.
I also found some python scripts to probe the registers... Some worked and
some did not.  Creating my own inverter definition with ideas from the
registers in the spreadsheet and seeing what numbers I got back eventually
got me something useful.    I would ask the company for such a spreadsheet
and data.
Ben
On Sat, Oct 28, 2023 at 5:22 PM josemonino @.***> wrote:

Thanks a lot.
How did you do it? Did you already know the registers before making it?
It would be wonderful if there were any step by step on how to do it...
I would like to do it for my Sofar inverter.
—
Reply to this email directly, view it on GitHub
https://github.com/StephanJoubert/home_assistant_solarman/issues/436#issuecomment-1783941235,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIACEPET3QHMJVKL3FT3HJTYBWHTJAVCNFSM6AAAAAA6PSGO5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTHE2DCMRTGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

This is an agreement with megarevo. Greensun’s inverter is also manufactured by megarevo.

What agreement where. I don't see it or am I missing something?
On Thu, Mar 28, 2024, 7:23 PM cyh0928 @.***> wrote:

This is an agreement with megarevo. Greensun’s inverter is also
manufactured by megarevo.
—
Reply to this email directly, view it on GitHub
https://github.com/StephanJoubert/home_assistant_solarman/issues/436#issuecomment-2026433933,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIACEPBSWY75WBFN72UIBZ3Y2S7BJAVCNFSM6AAAAAA6PSGO5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRWGQZTGOJTGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

哪里有什么协议。我没有看到它还是我错过了什么？
……
On Thu, Mar 28, 2024, 7:23 PM cyh0928 @.> wrote: This is an agreement with megarevo. Greensun’s inverter is also manufactured by megarevo. — Reply to this email directly, view it on GitHub <#436 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIACEPBSWY75WBFN72UIBZ3Y2S7BJAVCNFSM6AAAAAA6PSGO5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRWGQZTGOJTGM . You are receiving this because you were mentioned.Message ID: @.>

Your greensun inverter is manufactured by megarevo, so this agreement is the same as that of the invt inverter.


Bubble Particle Text project [gssoc21]

Related Issue

I have added a 'Bubble Particle Text' project using canvas and vanilla JS. In this project the text is in bubble form and bubbles move on moving mouse over it and then text re-gains its form automatically

Closes: #481
Describe the changes you've made
Added the 'Bubble Particle Text' Project.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
NA
Checklist:

[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] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.

@991rajat ,can you pls review this PR.

Thanks for this awesome contribution. :star_struck:
Keep Contributing.

@991rajat, can you pls add the gssoc21 and level tag on this PR.


"Non-mandatory-script-verify-flag error: Invalid Schnorr signature" occurred with two different inputs (P2TR + P2WSH or P2SH) Signet.

Hi
Signet also experiences the same issue when attempting to spend two inputs (P2TR + P2SH or P2WPKH).
I have prepared two scripts for this transaction type, and both of them are encountering errors. If possible, please review both scripts to identify potential issues. Thank you.
spend_p2tr_p2wpkh_to_p2pkh.py
spend_p2tr_p2sh_to_p2pkh.py

Hi Mohsen,
I had a look at the first one.
Since it is a segwit address you should use get_segwit_address in line 21.
Also, while taproot (segwit v1) requires the scriptPubKey when signing, segwit v0 requires the script code; i.e. a template that corresponds to the non-segwit equivalent. For example, for p2wpkh you would use as script code the p2pkh scriptPubKey.
So when signing the segwit input you shouldn't use the scriptPubKey but the corresponding p2pkh template for that address. Example https://github.com/karask/python-bitcoin-utils/blob/master/examples/spend_p2wpkh_transaction.py demonstrates this.
I haven't tried it out the above, but will do with the first opportunity.
PS. In the meantime, if you try it out yourself please use testmempoolaccept to test tx validity instead of just sending the tx to a node for processing. That will test tx validity without spending the outputs. If there is still an issue I can then test using the same UTXOs.

Just tried it and it works. Closing the issue but let me know if you need further help.

Thanks, it will take me a while to graduate to Bitcoin. :D

This was a tricky one. It seems to me that you are doing fine!

This was a tricky one. It seems to me that you are doing fine!

Your package is truly exceptional.
I've successfully sent this transaction using the Dart package. https://mempool.space/testnet/tx/ffb96b60303eb8e76654d320204a2727dec57ca00cf947a50c2be40ff084e35e
I've created a Dart script that automatically detects transactions and generates the output.
I plan to translate it to Python at the earliest opportunity. Perhaps you'd be interested in including it in your package
Thanks

Thank you for your kind words. I have deliberately not included a lot of conveniences like that. The reason being that I primarily use the library for teaching purposes and want the students to have to deal with some of the low level stuff.
Having said that, I would be happy to consider such automations if they are well-defined, in their own files (or maybe inside utils.py) and do not influence the existing codebase (i.e. are optional).
If you find the time you can add an issue with the suggested methods/changes at a high level and discuss them or of course you can do a PR and we can take it from there.


Update refined to 0.9.28

Updates eu.timepit:refined from 0.9.14 to 0.9.28.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "eu.timepit", artifactId = "refined" } ]

labels: test-library-update, early-semver-minor, semver-spec-patch

Superseded by #455.


[BUG] skip_annotations: true still posts annotations

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
When setting skip_annotations to true, annotations are still posted into the PR
To Reproduce
Try the following config for this action
      - name: ESLint changed files with Reviewdog
        uses: tj-actions/eslint-changed-files@v21
        with:
          reporter: github-pr-review
          skip_annotations: true
          filter_mode: file
          config_path: ".eslintrc.json"
          ignore_path: ".eslintignore"
          extra_args: "--rulesdir eslint-rules"
          fail_on_error: true

Create a file that fails linting
let not_a_number: string = 12
let not_a_string: number = "sup"

Create a PR and annotations are posted.
What OS are you seeing the problem on?
ubuntu-latest or ubuntu-22.04
Expected behavior?
eslint runs and otherwise fails if relevant but does not post any annotations.
Relevant log output
N/A

Has all relevant logs been included?

[X] I've included all relevant logs

Anything else?

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Closing this issue as there has been no response


Incompatible library version (server version: 7, library version: 6)

Today, I upgraded my FR3 to version 5.6.0, but upon trying to use libfranka, I was faced with the error
RuntimeError: libfranka: Incompatible library version (server version: 7, library version: 6). Please check https://frankaemika.github.io for Panda system updates or choose a libfranka version that uses the server version 7 from the table at https://frankaemika.github.io/docs/compatibility.html .

However, the link https://frankaemika.github.io/docs/compatibility.html only lists libfranka versions compatible up to robot server version 6.

Closed - I didn't see the 0.13.3 release, which resolves my issue.


[CI/CD]: setup github actions

Summary
NA
Answer following questions?

Is your feature related to FrontEnd? No
Is your feature related to Backend? No
Is your feature related to API Integration? No
Is your feature realted to CI/CD? Yes

Resources

While running
nx format:check

I am getting following error Cannot find module '/home/subham/data/project/lms-genie/node_modules/prettier/bin-prettier'
Full Error Details:-
subham@subham-Lenovo-ideapad-330-15IKB:~/data/project/lms-genie$ nx format:check
nx format:check

Check for un-formatted files

Run command using --base=[SHA1] (affected by the committed, uncommitted and untracked changes):
  --base  Base of the current branch (usually main)                                                                                [string]

or using --base=[SHA1] --head=[SHA2] (affected by the committed changes):
  --base  Base of the current branch (usually main)                                                                                [string]
  --head  Latest commit of the current branch (usually HEAD)                                                                                [string]

or using:
  --files        Change the way Nx is calculating the affected command by providing directly changed files, list of files delimited by commas or
                 spaces                                                                                [string]
  --uncommitted  Uncommitted changes                                                                                [boolean]
  --untracked    Untracked changes                                                                                [boolean]

Options:
  --help           Show help                                                                                [boolean]
  --version        Show version number                                                                                [boolean]
  --exclude        Exclude certain projects from being processed                                                                            [string]
  --libs-and-apps  Format only libraries and applications files.                                                                           [boolean]
  --projects       Projects to format (comma/space delimited)                                                                               [string]
  --all            Format all projects                                                                                [boolean]

Find more information and examples at https://nx.dev/nx/format-check

Error: Cannot find module '/home/subham/data/project/lms-genie/node_modules/prettier/bin-prettier'
    at createEsmNotFoundErr (node:internal/modules/cjs/loader:1096:15)
    at finalizeEsmResolution (node:internal/modules/cjs/loader:1089:15)
    at resolveExports (node:internal/modules/cjs/loader:565:14)
    at Module._findPath (node:internal/modules/cjs/loader:634:31)
    at Module._resolveFilename (node:internal/modules/cjs/loader:1061:27)
    at Function.resolve (/home/subham/data/project/lms-genie/node_modules/.pnpm/v8-compile-cache@2.3.0/node_modules/v8-compile-cache/v8-compile-cache.js:164:23)
    at Object.<anonymous> (/home/subham/data/project/lms-genie/node_modules/.pnpm/nx@16.7.3_@swc+core@1.3.51/node_modules/nx/src/command-line/format/format.js:18:31)
    at Module._compile (/home/subham/data/project/lms-genie/node_modules/.pnpm/v8-compile-cache@2.3.0/node_modules/v8-compile-cache/v8-compile-cache.js:192:30)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32) {
  code: 'MODULE_NOT_FOUND',
  path: '/home/subham/data/project/lms-genie/node_modules/prettier/package.json'
}

@shivamvijaywargi,
Is this command nx format:check working fine at you end?
If not, do you have any idea how to fix it?

I will check in some time.

Please use pnpm format to check and pnpm format:fix to apply prettier fixes


to consider: what would it take to run Aegis on a non-kubernetes environment (at a reduced capability set)

There are several things to consider:

how to ensure the identity of components like aegis-sentinel (probably using SPIRE and linux kernel attestators)
how to secure authentication to aegis-sentinel (probably through an external OIDC provider)
how to dispatch secrets (possibly using a variation of the current API/SDK)

^ both of these are doable; but they are not high on the priority list for now.

A little bit more details about this.
To adapt Aegiso a non-Kubernetes environment while still using SPIRE, we can follow these steps:
Set up SPIRE Server and SPIRE Agent
Deploy SPIRE Server and SPIRE Agent on your Linux machines. SPIRE Server will be responsible for managing the issuance of SVIDs, while SPIRE Agents will be running on each machine to fetch the SVIDs for workloads.
Establish node attestation
Configure the SPIRE Server and SPIRE Agent to use a suitable node attestation plugin that fits our environment. We can choose from a variety of plugins provided by SPIRE, such as "sshpop" for SSH-based attestation, "aws_iid" for AWS EC2 instances, or "gcp_iit" for Google Cloud Platform instances. This will ensure that only valid machines can request SVIDs from the SPIRE Server.
Configure workload attestation
Configure the SPIRE Agents to use a suitable workload attestation plugin based on your requirements. Some common workload attestation plugins include "unix" (for Unix-based systems) and "docker" (for Docker containers). You can also create your own custom attestation plugin if needed.
Modify Aegis Safe
Replace the Kubernetes primitives with equivalent Linux components. For example, instead of using Kubernetes service accounts, roles, rolebindings, and namespaces, you can use Linux user accounts, groups, and file permissions to manage access control for the Secrets Store.
Modify Sentinel
Similar to the Secrets Store, adapt the Sentinel to work with Linux components instead of Kubernetes primitives. Ensure that the Sentinel can establish mTLS with the Secrets Store using the SVIDs issued by the SPIRE Server.
Modify workload registration and secret retrieval
Update the workloads to request SVIDs from the local SPIRE Agent rather than using Kubernetes primitives. When a workload needs to access a secret, it should present its SVID to the Secrets Store and establish mTLS using that SVID.

By following these steps, we can create a similar architecture to your Kubernetes-based solution, but for Linux machines without Kubernetes or Docker. The key is to replace the Kubernetes-specific components with equivalent Linux components, while keeping SPIRE as the core component for issuing and managing SVIDs.

By that logic, we can create a non-Kubernetes, docker-only solution, or a vm-only solution etc. too.


Terrain world for multiplayer

Hello i have a server and this old code is still working great with a few updates.
However i would like to implement terrain generation (currently everything is in a flat world)
I saw a function for creating terrain worlds but i dont really understand how to use it, simply replacing world.createFlatWorld with world.createFromString does not appear to work. Could someone help me?

Cleaning out my open issues, closing this as resolved as im assuming the above solution works. However i currently have no ability to test it myself.

can you put it in a code box? its the button to the left of the 1 key ``` three times before and after your code (someone had to tell me too) but this looks cool iv messed with this project a few times over the years this time im looking to really do some work on it because of another project im working on and these snippets are really helpful


Continue to add support for git urls as dependency repositories

Continue the PR https://github.com/helm/helm/pull/6734 work.
related issue: https://github.com/helm/helm/issues/9461
If applicable:

[x] this PR contains documentation
[x] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

ok, I will do this work as soon as I can. Thanks for your reply. @jdolitsky

Hi @yxxhero , is there anything i can help you with?
And one more question: is it also possible to give a contextDir with a chart in the Chart.yaml? For the usecase when the chart is not directly in the root of the Git repository (see example below)
# Chart.yaml
dependencies:
- name: nginx
  version: "1.2.3"
  repository: "ssh://git@git.host:7999/helm-dep/helm-dep.git"
  contextDir: "helm/chart"

Hi @yxxhero , is there anything i can help you with?
And one more question: is it also possible to give a contextDir with a chart in the Chart.yaml? For the usecase when the chart is not directly in the root of the Git repository (see example below)
# Chart.yaml
dependencies:
- name: nginx
  version: "1.2.3"
  repository: "ssh://git@git.host:7999/helm-dep/helm-dep.git"
  contextDir: "helm/chart"

I'm busy recently. I'll try to finish it Thanks very much.

@jdolitsky ping

@technosophos  @mattfarina @bacongobbler  @jdolitsky Look forward to your reply.  Thanks very much.

Hey @yxxhero great work!
From the inline docs i get that it is not possible to use subdirectories (contextdir) with this Git functionality:
* The helm chart and 'Chart.yaml' must be in the root of the git repo.
The chart cannot be loaded from a subdirectory.

Is this a design choice? Or is it functionality what is possible to add? For example we (my team) have more files and directories in our helm-chart-repo's. Files needed for CICD pipelines for example.
So our repo would be:
\chart-cicd\helm-chart
           \cicd-files

So when referencing the helm chart in the dependencies we would like to reference the subdir helm-chart from the repo chart-cicd

Hey @yxxhero great work!
From the inline docs i get that it is not possible to use subdirectories (contextdir) with this Git functionality:
* The helm chart and 'Chart.yaml' must be in the root of the git repo.
The chart cannot be loaded from a subdirectory.

Is this a design choice? Or is it functionality what is possible to add? For example we (my team) have more files and directories in our helm-chart-repo's. Files needed for CICD pipelines for example.
So our repo would be:
\chart-cicd\helm-chart
           \cicd-files

So when referencing the helm chart in the dependencies we would like to reference the subdir helm-chart from the repo chart-cicd

The current design is like this. Of course you can change it and work together to do it.

I'm going to add  contextDir features in my own repo. If it is done. You can test it. Thanks very much. @BlueCog

@bacongobbler @jdolitsky  Does this feature require design documentation? Thanks very much.

@yxxhero
FYI: i've build a Helm client with your PR and tested the functionality (private and public repo's). It works without any issue.
Greets

@jdolitsky  ping

I built the last commit from this PR and am getting the following error when trying to helmy dep up
» helmy version
version.BuildInfo{Version:"v3.6.1-yxx-1+unreleased", GitCommit:"eef224055ea6457c5a628a76815b0de5b0efe2c1", GitTreeState:"clean", GoVersion:"go1.16.5"}

» helmy dep up
Hang tight while we grab the latest from your chart repositories...
[... trim ...]
Update Complete. ⎈Happy Helming!⎈
Saving 2 charts
Downloading helm-example from repo git://git@bitbucket.org:example/helm-example.git
Save error occurred:  could not download git://git@bitbucket.org:example/helm-example.git: mkdir /tmp/helm1319646873: not a directory
Deleting newly downloaded charts, restoring pre-update state
Error: could not download git://git@bitbucket.org:example/helm-example.git: mkdir /tmp/helm1319646873: not a directory

Another issue I'm getting is, if the name of the dependency doesn't match the name of the repo, I get this error:
dependencies:
  - name: library
    version: 1.2.3
    repository: git://git@bitbucket.org:example/helm-library.git

» helmy dep up
Hang tight while we grab the latest from your chart repositories...
[... trim ...]
Update Complete. ⎈Happy Helming!⎈
Error: The name of dependency git://git@bitbucket.org:example/helm-library.git should be helm-library

In this case, the name of the chart is library but the repo is named helm-library.
The name field in the Chart.yaml is also library.

I built the last commit from this PR and am getting the following error when trying to helmy dep up
» helmy version
version.BuildInfo{Version:"v3.6.1-yxx-1+unreleased", GitCommit:"eef224055ea6457c5a628a76815b0de5b0efe2c1", GitTreeState:"clean", GoVersion:"go1.16.5"}

» helmy dep up
Hang tight while we grab the latest from your chart repositories...
[... trim ...]
Update Complete. ⎈Happy Helming!⎈
Saving 2 charts
Downloading helm-example from repo git://git@bitbucket.org:example/helm-example.git
Save error occurred:  could not download git://git@bitbucket.org:example/helm-example.git: mkdir /tmp/helm1319646873: not a directory
Deleting newly downloaded charts, restoring pre-update state
Error: could not download git://git@bitbucket.org:example/helm-example.git: mkdir /tmp/helm1319646873: not a directory

Another issue I'm getting is, if the name of the dependency doesn't match the name of the repo, I get this error:
dependencies:
  - name: library
    version: 1.2.3
    repository: git://git@bitbucket.org:example/helm-library.git

» helmy dep up
Hang tight while we grab the latest from your chart repositories...
[... trim ...]
Update Complete. ⎈Happy Helming!⎈
Error: The name of dependency git://git@bitbucket.org:example/helm-library.git should be helm-library

In this case, the name of the chart is library but the repo is named helm-library.
The name field in the Chart.yaml is also library.

mkdir /tmp/helm1319646873: not a directory fix. I used os.CreateTemp by mistake.
you shuld change like below:

dependencies:
  - name: helm-library
    version: 1.2.3
    repository: git://git@bitbucket.org:example/helm-library.git

Great, rebuilt the latest commit (416efce) and changed the name of the dependency to helm-library to match the name of the repo.
» helmy version
version.BuildInfo{Version:"v3.6.1-yxx-1+unreleased", GitCommit:"416efce3c2859d182ceaa6f739018bd46ab13259", GitTreeState:"clean", GoVersion:"go1.16.5"}

» helmy dep up
Hang tight while we grab the latest from your chart repositories...
[... trim ...]
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading helm-library from repo<EMAIL_ADDRESS>Deleting outdated charts

One question, does the name of the dependency have to match the name of the repo?
I ask this because, in my org, we use the helm- prefix to distinguish between applications and helm charts that deploy said applications (or libraries).
We named our internal library chart libs, but the repo helm-libs.
Just like scala-libs or jenkins-libs
I've noticed in umbrella/master charts if the name/version of a dependency doesn't match the name/version of the downloaded chart, helm values don't propagate downwards.
I'll craft a reproducible example and post it here a little later, maybe this issue is outside of the scope of this PR.

Great, rebuilt the latest commit (416efce) and changed the name of the dependency to helm-library to match the name of the repo.
» helmy version
version.BuildInfo{Version:"v3.6.1-yxx-1+unreleased", GitCommit:"416efce3c2859d182ceaa6f739018bd46ab13259", GitTreeState:"clean", GoVersion:"go1.16.5"}

» helmy dep up
Hang tight while we grab the latest from your chart repositories...
[... trim ...]
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading helm-library from repo git://git@bitbucket.org:example/helm-library.git
Deleting outdated charts

One question, does the name of the dependency have to match the name of the repo?
I ask this because, in my org, we use the helm- prefix to distinguish between applications and helm charts that deploy said applications (or libraries).
We named our internal library chart libs, but the repo helm-libs.
Just like scala-libs or jenkins-libs
I've noticed in umbrella/master charts if the name/version of a dependency doesn't match the name/version of the downloaded chart, helm values don't propagate downwards.
I'll craft a reproducible example and post it here a little later, maybe this issue is outside of the scope of this PR.

@rblaine95  I will find a way to remove this restriction. wait for your example. I will use your example to test it.

I think the issue I have experienced before - name/version of dependency doesn't match name/version specified in the sub-chart's Chart.yaml causes values to not propagate - was due to a build of the old git PR, it appears that PR names the sub-charts in charts/ as chart-name-chart-version.tgz instead of dep-name-dep-version.tgz
However, I'm experiencing this issue where the name of the repo doesn't match the name field in the sub-charts Chart.yaml
» helmy version
version.BuildInfo{Version:"v3.6.1-yxx-1+unreleased", GitCommit:"416efce3c2859d182ceaa6f739018bd46ab13259", GitTreeState:"clean", GoVersion:"go1.16.5"}

» helmy create nginx
Creating nginx

» cd nginx

» code .

» cat Chart.yaml
apiVersion: v2
name: nginx
description: A Helm chart for Kubernetes

[... trim ...]

dependencies:
  - name: helm-library
    version: 0.7.6
    repository: git://git@bitbucket.org:example/helm-library.git

» helmy dep up --skip-refresh
Saving 1 charts
Downloading helm-library from repo git://git@bitbucket.org:example/helm-library.git
Deleting outdated charts

» helmy template nginx .
Error: found in Chart.yaml, but missing in charts/ directory: helm-library

» ls -l ./charts
total 72
-rw-r--r-- 1 robert robert 71634 Jun 24 10:41 helm-library-1.2.3.tgz

» cd charts

» tar zxf helm-library-1.2.3.tgz

» cat ./helm-library/Chart.yaml
apiVersion: v2
name: library
description: Helm library chart
type: library

version: 1.2.3

I think the issue I have experienced before - name/version of dependency doesn't match name/version specified in the sub-chart's Chart.yaml causes values to not propagate - was due to a build of the old git PR, it appears that PR names the sub-charts in charts/ as chart-name-chart-version.tgz instead of dep-name-dep-version.tgz
However, I'm experiencing this issue where the name of the repo doesn't match the name field in the sub-charts Chart.yaml
» helmy version
version.BuildInfo{Version:"v3.6.1-yxx-1+unreleased", GitCommit:"416efce3c2859d182ceaa6f739018bd46ab13259", GitTreeState:"clean", GoVersion:"go1.16.5"}

» helmy create nginx
Creating nginx

» cd nginx

» code .

» cat Chart.yaml
apiVersion: v2
name: nginx
description: A Helm chart for Kubernetes

[... trim ...]

dependencies:
  - name: helm-library
    version: 0.7.6
    repository: git://git@bitbucket.org:example/helm-library.git

» helmy dep up --skip-refresh
Saving 1 charts
Downloading helm-library from repo git://git@bitbucket.org:example/helm-library.git
Deleting outdated charts

» helmy template nginx .
Error: found in Chart.yaml, but missing in charts/ directory: helm-library

» ls -l ./charts
total 72
-rw-r--r-- 1 robert robert 71634 Jun 24 10:41 helm-library-1.2.3.tgz

» cd charts

» tar zxf helm-library-1.2.3.tgz

» cat ./helm-library/Chart.yaml
apiVersion: v2
name: library
description: Helm library chart
type: library

version: 1.2.3

I will see it.

I will fix it in the weekend. @rblaine95

@rblaine95   All is ok. Please enjoy it.

@jdolitsky ping

@yxxhero just built it, working perfectly!

@rblaine95 ok， if you have other quesions, contact me!

@jdolitsky  Does git need to be replaced with github.com/Masterminds/vcs?

@jdolitsky Does git need to be replaced with github.com/Masterminds/vcs? This library also require git binary.

cc @mattfarina @technosophos . Opinion on this?

ping

@jdolitsky @technosophos ping

@rblaine95 cool, you are right.

@jdolitsky @technosophos

@jdolitsky @technosophos ping

In the tendency to GitOps it's sad to see this PR being closed. The maintains don't seem to appreciate the community effort at all.

@jdolitsky oh. sorry. It's my fault. I will create a PR. Thanks very much.


feat: allow setting loglevel

This PR lets users set a loglevel from the command line.
I am a little hesitant in adding this option as environment variable, too as it increases the api surface. For now i think it's enough to have just the cli flags.

/approve

/merge


Support for H.264 streams?

Hello,
is it possible to alter ffmpeg options to generate H264 stream instead of MJPEG? I.e. use libx264 codec?

Hi,
HA only support mjpeg streams in UI. That is the reason why I do that in this way.

Is the limitation of mjpeg only streams an actual technology limitation, or just a matter of no one having bothered doing the work yet?


libass: 0.15.0 -> 0.15.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/libass/libass/releases.
meta.description for libass is: "Portable ASS/SSA subtitle renderer"
meta.homepage for libass is: "https://github.com/libass/libass"
meta.changelog for libass is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.15.1 with grep in /nix/store/j51fsvgrflqphfdhmym5h5anjhkxxw74-libass-0.15.1

found 0.15.1 in filename of file in /nix/store/j51fsvgrflqphfdhmym5h5anjhkxxw74-libass-0.15.1

directory tree listing: https://gist.github.com/488669347599c3a16431c534315bb9bc

du listing: https://gist.github.com/4761adecec9eacbc9872ef89a96e8281

Rebuild report (if merged into master) (click to expand)

4396 total rebuild path(s)

1784 package rebuild(s)

1504 x86_64-linux rebuild(s)
1271 i686-linux rebuild(s)
370 x86_64-darwin rebuild(s)
1251 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
DisnixWebService
MIDIVisualizer
acoustidFingerprinter
adapta-gtk-theme
adoptopenjdk-icedtea-web
aegisub
agi
ahoviewer
airtame
alass
alchemy
alda
aldor
alephone
alephone-durandal
alephone-eternal
alephone-evil
alephone-infinity
alephone-marathon
alephone-pathways-into-darkness
alephone-pheonix
alephone-red
alephone-rubicon-x
almanah
amarok
amarok-kf5
ammonite
ammonite_2_12
ammonite_2_13
anki
ankisyncd
apache-directory-studio
apache-flex-sdk
apktool
apostrophe
aravis
archi
ardour
arduino
arduino-core
areca
asciidoc-full-with-plugins
asciidoctorj
aspectj
astroid
astrolabe-generator
atlassian-cli
attract-mode
audacious
audaciousQt5

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j51fsvgrflqphfdhmym5h5anjhkxxw74-libass-0.15.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libass https://github.com/r-ryantm/nixpkgs/archive/c9b192392c2e6202d371152829f0a641c411775e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j51fsvgrflqphfdhmym5h5anjhkxxw74-libass-0.15.1
ls -la /nix/store/j51fsvgrflqphfdhmym5h5anjhkxxw74-libass-0.15.1/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @codyopel for testing.

Result of nixpkgs-review pr 121562 at c9b19239 run on x86_64-linux 1

100 packages marked as broken and skipped:

aldor
avxsynth
foo-yc20
gnome3.gnome-books
gnome3.gnome-documents
grass
hydrus
jboss
junit
junixsocket
...

1248 packages skipped due to time constraints:

DisnixWebService
MIDIVisualizer
acoustidFingerprinter
adapta-gtk-theme
adoptopenjdk-icedtea-web
aegisub
agi
ahoviewer
airtame
alass
...

14 packages built successfully:

chromaprint
ffmpeg (ffmpeg_4)
ffmpeg_3 (ffmpeg_3_4)
ffmpegthumbnailer
gst_all_1.gst-libav
harvid
jellyfin
libass
navidrome
python38Packages.pyacoustid
youtube-dl (python38Packages.youtube-dl)
roon-server
unpaper
yle-dl


WordPress Theme to Streamline Installation

There are a few core things needed to make this work with WordPress, maybe we can figure out a way to make a WordPress theme to easily get that done
Plugins

WP GraphQL
WP GraphiQL

Settings

Metadata: Twitter handle for twitter:site meta tag

I'll start this and get a v0.1 ready.

I know I've fallen behind on this. Good news is that GraphiQL is now built into WPGraphQL, so actually this is currently one module and one piece of meta-data.

I know I've fallen behind on this. Good news is that GraphiQL is now built into WPGraphQL, so actually this is currently one module and one piece of meta-data.


SHA256 showing error, but compiles ok

Using sha256() showing as error in editor, but if you'll compile the code, FUNC code generates great and have no compile errors.
[{
"message": "/workspace/proj-name/contracts/contract.tact:129:42: Static function "sha256" does not exist
}]
Code:
let A: Int = sha256("abcd");

fixed on version 1.0.3


How to emit data on client side and listen on server side?

I'd like to emit data on client side, and listen this data on server side. How may i do this?
I'd like to listen on server side this data and start uploading big files. Then i'd like to broadcast this to regarding user.
I'd be so glad if you help me through this.

You can interact with this library in the same way as you would with Pusher, so you can use the documentation over there.
However, if you look in the documentation of Laravel itself, broadcasting is not really made to send data from client to the server, like you would with other websocket libraries like Socket.io. Better practice is to use Http requests for that.


Make a bash script which install every npm needed

Create an installation bash script for for those who want one-click install

discarded


Update ciris to 2.2.0

Updates is.cir:ciris from 2.1.1 to 2.2.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.1.1).
You might want to review and update them manually.
build.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "is.cir", artifactId = "ciris" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by #774.


Update README.md

Hi, I add a paper about egocentric query-focused video summarization published in CVPR 17'.

@QinghongLin Thank you for your interest in the list and adding the paper from CVPR 2017.
I see you have added the paper under video summarisation title. Can you please add it to CVPR (https://github.com/Sid2697/awesome-egocentric-vision#CVPR) too? Once you are done, I'll merge it!
Regards,
Siddhant Bansal

Sure, will do now.
Thanks!


Added compat for ResearchPowl

Sadly, it seems like this fork of ResearchPal lost the MP compat

Shame on them, I made it super easy to maintain and fully commented.

To be honest, I basically looked at the compat in 1.3 and applied it to 1.4 fork.


Samples still use removed assemblies folder

Before splitting up the PnP repos the repo contained an Assemblies folder. With the splitting of the repositories it was moved to the PnP-Site-Core repository. But there are still samples in this repository which contain references to this assemblies folder (eg. Core.UserProfiles.Sync.
I can clean this up in a PR but I don't know what the idea is regarding this references and where they should reference too.
Maybe it would be a good idea to bring the Assemblies folder also back to this repository.

@VesaJuvonen Any opinion about this?

PRs for the updates are fine. We absolutely should update samples and solutions in the PnP repository to use Core component with Nuget package, not with project references. There should not be that many of these, but seems that at least few got through after re-organization.

@VesaJuvonen But for example Core.UserProfiles.Sync requires Microsoft.Online.SharePoint.Client.Tenant.dll and this couldn't be resolved with using Core NuGet package. Or am I missing something?

This is now fixed in the dev branch with the #1289 submission. Tenant object is included in the latest Nuget package (http://www.nuget.org/packages/Microsoft.SharePointOnline.CSOM), so that everything works with them - no reason for direct assembly references. Thx for reporting and please do report also if you find other old reference issues. You'll be noted already based on this in the release notes, since without your input, this sample would have stayed broken.