feat: enhance logging

We can now run the program and specify log level via arg or env :

ENV LOG=debug go run . start
go run . -log=debug start

3 levels available : trace, debug and info

trace & debug for dev purpose with slow but clean logs
info for prod with fast json encoded logs

Also, this PR will try to replace every fmt.print based logs with the zerolog lib, and will log all the silenced errors, at the cost that during the tests for exemple, too many lines might be displayed.

@gallexis btw any zerolog should be used strictly for logging. So CLI command feedback/messages should stay with fmt.Printf yeah?

@gallexis btw any zerolog should be used strictly for logging. So CLI command feedback/messages should stay with fmt.Printf yeah?

Yeah it sounds good like that

Is there a way to log the line where the log call is situated? Hard to know where it's coming from otherwise

Is there a way to log the line where the log call is situated? Hard to know where it's coming from otherwise

I added it thanks to a hook, but only for debug/trace level because it slows down the logging

@gallexis any idea what is going on with the commit history here?

Yeah idk what happened, I pulled the main branch then rebase with this branch

I'll look monday how to get rid of these commit

Actually I just tried the v3 to see if there was any breaking change. Seems there is not so why not using the v3?

Word, just curious

I'm getting more and more confused about his commit history hahaha

ahah don't worry I think I fixed it by created a new branch a rebasing it without only my commits


Symmetric matrix predicate doesn't work when Aij == Aji

The issue is reported by Jean-Francois Lecomte. Here is the attached patch:
private static class SymmetricMatrixPredicate
    implements AdvancedMatrixPredicate {

        @Override
        public boolean test(Matrix matrix) {
            if (matrix.rows() != matrix.columns()) {
                return false;
            }

            double matrix_max = matrix.max();
            for (int i = 0; i < matrix.rows(); i++) {
                for (int j = i + 1; j < matrix.columns(); j++) {
                    double a = matrix.get(i, j);
                    double b = matrix.get(j, i);
                    double diff = Math.abs(a - b);

                    if (diff / Math.abs(matrix_max) > 1e-12) {
                        return false;
                    }
                }
            }

            return true;
        }
    }

Is this still not working properly? I checked the code and it is different from the snippet you provided.


Move pin af definitions to a universal pin_mappings file

This makes the peripheral files really peripheral specific and moves the glue into a separate file. I'd like to compare it to all the datasheets to make sure there is no typo in there.
I'm not sure if we should keep the analog mappings in the adc.rs file.

Nice idea!

Looks awesome!

hmm, I have a potential thought for this. Would it be preferable to just have a single definition of all the pins for each micro variant? It will result in some duplication, but it will also make it much clearer what the total mapping for any given micro is.

hmm, I have a potential thought for this. Would it be preferable to just have a single definition of all the pins for each micro variant? It will result in some duplication, but it will also make it much clearer what the total mapping for any given micro is.

I was thinking the same as this would make things much easier to change and verify.

Just my $0.02, I like @HarkonenBade 's idea of splitting out the pin definitions by variant; as mentioned by @therealprof it would simplify adding/modifying/verifying things, IMO. In regards to potential duplication, we may be able to refactor out some common configuration as we find it.

Would it be preferable to just have a single definition of all the pins for each micro variant?
I split it up so its (everything) -> (family) -> (specific micro). I think it's the best way.

Yeah, I'm just wondering if a flat single macro for each supported chip feature would be easier to keep track of. Also I'd be down to make a python script that takes the pin mapping table from a datasheet in a sane way and outputs the macro def.

f. Also I'd be down to make a python script that takes the pin mapping table from a datasheet in a sane way and outputs the macro def.

Pdf parsing sounds somewhat broken and insane, but you could maybe see what stm32cubemx uses internally. It's probably something xml-y.

f. Also I'd be down to make a python script that takes the pin mapping table from a datasheet in a sane way and outputs the macro def.

Pdf parsing sounds somewhat broken and insane, but you could maybe see what stm32cubemx uses internally. It's probably something xml-y.

That is fair, currently pulling some of their MCU defs to take a look at what the file format is like.

@HarkonenBade PDF parsing is indeed tricky. Most likely you'll only find all the fancy tipos and minor changes in typography in all the different data sheets.

Well, I've managed to hack together something that can read the cubemx database files and spit out a pins! macro definition for a particular mcu.

@HarkonenBade Cool stuff, potentially even more accurate than the SVD files or datasheets/documentation.

https://github.com/HarkonenBade/cube-parse
There is the current state, the code is super messy and potentially fragile, but it seems to function. I'll try and grind some of the rough edges off if it seems useful.

Looks good. Does anyone know what the license of these files is? I liked having a guaranteed subset, but it's way better and less error prone this way.

Looks good. Does anyone know what the license of these files is? I liked having a guaranteed subset, but it's way better and less error prone this way.

I believe that it is under the same licence applied to all of cubeMX which is this: https://www.st.com/resource/en/license_agreement/dm00218346.pdf . I'm not 100% certain if derivative use would be permitted, but I also am not an expert at reading licence agreements.

use, reproduction or redistribution of this software package partially or totally may be done in any manner that would subject this software package to any Open Source Terms (as defined below)
is software package or any part thereof, including modifications and/or derivative works of this software package, must be used and execute solely and exclusively on or in combination with a microcontroller or a microprocessor devices manufactured by or for STMicroelectronics.

Doesn't seem very compatible. We should probably store them in a separate repo akin to stm32-rs and just use the end result. I wouldn't depend on them not completely changing under us. Having a separate crate doesn't seem possible/easy, so i think we just need to copy-paste.

hmm, I'm not sure that would be directly a terrible restriction, as is the stm32f0xx-hal really going to be used on anything other than ST devices? Also if we were copy pasting then we would still fall foul of the licence I suspect.

Nah, we're only using the technical data, that's not copyright, while the original files may be.
According to Wikipedia

Copyright does not cover ideas and information themselves, only the form or manner in which they are expressed

So we're probably safe, but I'm no lawyer

@david-sawatzke

Doesn't seem very compatible.

What do you mean by that? Using the result should be no problem. Redistributing parts of CubeMX is of course a bad idea but it's likely they're part of the family packs and in that case it might be even possible to retrieve them on the fly.

I wouldn't depend on them not completely changing under us.

I would not change anything from how we're working today: if we need the pins, get the files, run the tool put the output into this crate and done. We'll not notice if they change anyway but unless they're buggy I'm also not expecting any major changes.

@david-sawatzke

Doesn't seem very compatible.

What do you mean by that? Using the result should be no problem. Redistributing parts of CubeMX is of course a bad idea but it's likely they're part of the family packs and in that case it might be even possible to retrieve them on the fly.

They are sadly only distributed as part of the actual cubemx application. The family packs just contain hal sources and templates/examples. The issue is if the pin defs count as 'derived works' from the cubemx software package.

Using the result seems to be no problem

Yes, but i personally like the way stm32-rs does it, generating final files as needed and using original files as the source of truth. But we can't do it this way in the repo

Yes, but i personally like the way stm32-rs does it, generating final files as needed and using original files as the source of truth. But we can't do it this way in the repo

There're two obvious ways how to achieve the same (and a few more non-obvious ones):

Ask for permission to include the source files in the GH repo
Download them on the fly

Yes, but i personally like the way stm32-rs does it, generating final files as needed and using original files as the source of truth. But we can't do it this way in the repo

There're two obvious ways how to achieve the same (and a few more non-obvious ones):

Ask for permission to include the source files in the GH repo
Download them on the fly

The latter will require working out how to extract them from the STM32CubeMX installer without running it however.

Getting permission would be great, but i wouldn't download them on the fly, since we could experience random breakage with newer versions and stm32cubemx is pretty large.

Slow progress on autogeneration still being made in https://github.com/HarkonenBade/cube-parse, the latest output looks something like https://gist.github.com/HarkonenBade/5ae17d4f85e5f83faeb05407fd5a653c

I'm currently working on generating STM32 symbols/components for the LibrePCB base library. For that we also needed the pinout information. I found an alternative to parsing the XML files directly: STM32CubeMX can be scripted to export pinout mappings in CSV format. The result can be found here: https://github.com/LibrePCB/stm32-pinout
That functionality is also available directly through the GUI. You can export the pinout CSV files manually through a menu, both with and without AF mappings. You can find an example file here, it basically matches the table in the datasheet.
Unfortunately the CSV files exported through scripting do not contain AF mappings. However, the AF mappings are printed to the terminal during export, so the information is there.
I reverse-engineered the CubeMX JAR a bit and found the function that handles the command line CSV export:
CommandLineManager.addCommand("csv pinout", "csv pinout <file>", 0, new CommandLineListener() {
  public boolean onCommandEntered(String[] args) {
    return MxSystem.getMxMcu().csvDump(args[0], false);
  }
});

The false parameter to csvDump specifies whether AF mappings are exported as well or not. Unfortunately it's hardcoded to false.
I opened a support ticket with ST to ask whether that functionality could be added or not. I don't have high hopes that this will happen though, usually requests like that end up in /dev/null.
Another option would be to try patching the JAR so we could export pinout files with AF mappings using the official software. From those CSV files we could generate grouped AF pin mappings for various STM32 HALs. I'm not sure how easy patching JARs is, IIRC they have some kind of checksum. All we'd need to do is changing that false to true though.
Regarding licensing, I think that those exported CSV files only contain technical information from the datasheet and probably do not fall under copyright. IANAL though.

By patching a single byte in the JAR bytecode, the exported CSV files now include the AF mapping information :tada:
https://github.com/LibrePCB/stm32-pinout/
See the data directory for the current export.
@david-sawatzke @HarkonenBade is this useful to you, or does the database parser already handle all information you need? The reason why I think using the CSV export functionality might be useful, is that ST appears to do some post-processing in their code on top of the data available in the database files. For example:
    if (paramName.equals("GPIO_Speed") && currentIp != null && currentIp.getIPGroup().equalsIgnoreCase("Connectivity")) {
      String MxDBVersion = this.mcu.getProject().getMxDataBaseVersion();
      if (CodeGenerator.isDBversionOlderThan("DB.4.0.140"))
        return "GPIO_SPEED_HIGH";
      if (getMcu().getFamily().equals("STM32L1") || getMcu().getFamily().equals("STM32L0") ||
        getMcu().getFamily().equals("STM32L4") ||
        getMcu().getFamily().equals("STM32F2") ||
        getMcu().getFamily().equals("STM32F4") ||
        getMcu().getFamily().equals("STM32F7"))
        return "GPIO_SPEED_FREQ_VERY_HIGH";
      if (getMcu().getFamily().equals("STM32F0") || getMcu().getFamily().equals("STM32F3") ||
        getMcu().getFamily().equals("STM32F1"))
        return "GPIO_SPEED_FREQ_HIGH";
    }

However, I don't know whether this affects AFs as well.

Quick update, after checking @HarkonenBade's approach with cube-parse again a bit in more depth, this definitely looks like the cleaner approach than the hack I did above. We had a lengthy discussion tonight in the rust-embedded Matrix channel with @adamgreig and @therealprof about grouping of features and ended up with the conclusion that using the "GPIO IP Versions" as features is probably the best approach.
I've forked cube-parse for now and added a lot of documentation on how the STM32CubeMX database is structured, and why grouping features by "GPIO IP Version" probably makes sense.
In the case of the STM32F0, the following five GPIO versions...
STM32F031_gpio_v1_0_Modes.xml
STM32F042_gpio_v1_0_Modes.xml
STM32F051_gpio_v1_0_Modes.xml
STM32F052_gpio_v1_0_Modes.xml
STM32F091_gpio_v1_0_Modes.xml

...would result in the following features:

io-STM32F031
io-STM32F042
io-STM32F051
io-STM32F052
io-STM32F091

The feature names could still be bikeshedded. A possible alternative would be removing the STM32 prefix so that the feature name is io-F031. I think some kind of prefix (like io) makes sense though, to indicate that these are not necessarily prefixes of your MCU (see below).
Note that sometimes those GPIO peripheral versions don't match the name. For example the STM32F091_gpio_v1_0 version is shared among the following MCUs:
#[cfg(feature = "io-STM32F091")]
// STM32F030CCTx
// STM32F030RCTx
// STM32F091C(B-C)Tx
// STM32F091C(B-C)Tx
// STM32F091C(B-C)Ux
// STM32F091C(B-C)Ux
// STM32F091R(B-C)Tx
// STM32F091R(B-C)Tx
// STM32F091RCHx
// STM32F091RCYx
// STM32F091V(B-C)Tx
// STM32F091V(B-C)Tx
// STM32F091VCHx
// STM32F098CCTx
// STM32F098CCUx
// STM32F098RCHx
// STM32F098RCTx
// STM32F098RCYx
// STM32F098VCHx
// STM32F098VCTx

This includes some F098xxxx MCUs as well as some STM32F030xxxx. However, other STM32F030xxxx MCUs are using other GPIO versions (e.g. STM32F030C6Tx -> io-STM32F031).
We'll probably have to solve this issue with documentation (and maybe a lookup table from MCU to feature).
The alternative would be adding a feature per MCU and then adding a dependency on the proper GPIO version, but @therealprof objected to having many features :slightly_smiling_face: I'm still unsure whether having a feature per MCU would be a good idea or not.
(The MCU feature dependencies could look something like this: STM32F098CCUx = ["io-STM32F091"].)

I'm still unsure whether having a feature per MCU would be a good idea or not.

I think the feature per IO IP version makes more sense (if properly documented) since which IO IP the MCU has is a feature of it.
On the other hand as soon as we need to differentiate between USART, I2C, etc. IP versions, it would maybe be easier to just select the MCU and all the features get selected correctly. But that could also happen via some code generation, i.e. creating from a template for a given MCU.

@dbrgn I haven't looked at this issue & repo in quite a while, but I think, as long as it's all generated, it's fine to have a feature for each cpu.
I prefer it over the approach with the IP features, since having the human do the lookup is quite error-prone & takes more effort to check, especially when "only" switching the variant. I also can't see many disadvantages to this approach, since it just consists of adding something generated to the cargo.toml, especially since this is gonna have to be done either in the cargo.toml or in the documentation. I've skimmed the conversation in matrix and based on my understanding, the major objection here is the need to test each & every feature.
But a complete test "just" needs to test all the combinations that are used, not each stm32f42 part if they're all the same. This doesn't differ from what needs to be done if this lookup table is just documentation, since all combinations that exist as a part still need to be tested. (I'm also not sure if we can't just rely on testing all the combinations when updating the generated files and otherwise ignoring multiple gpio mappings)
For the "unnecessary" variants: It's also more obvious to the user, if each feature they need is named in a somewhat consistent way, not the thing we currently do, where the "granularity" of the features differ based on which family you're using.
TLDR: The difference is whether the lookup table is in the documentation or exists as features. Removing the human element seems like a good idea.
(BTW: I have very little skin in the game, feel free to ignore this)

I tend to agree with you, especially with regard to only having to test one mcu-feature per gpio version. However, I still do see value in mainly using gpio version based features for cfg guards in the code because it allows using newer MCUs with older HALs without having to re-generate all the pin mappings first, as long as the gpio version of the new MCU is already covered.
@therealprof would you also object to per-mcu features if they're purely aliases for a gpio version feature?
mcu-STM32F098CCUx = ["io-STM32F091"]

As aliases, they would not need to be tested in CI.

I might be missing something, but it seems as though this mapping is not found by cube-parse.
#[cfg(any(feature = "stm32f030xc", feature = "stm32f042", feature = "stm32f048"))]
i2c_pins! {
    I2C1 => {
        scl => [gpiob::PB13<Alternate<AF5>>],
        sda => [gpiob::PB14<Alternate<AF5>>],
    }
}

stm32f042c4.pdf does have this mapping and it does show up in CubeMX.
However GPIO-STM32F042_gpio_v1_0_Modes.xml (or any other F0 file for that matter) does not include it.
Not sure what's going on there or how CubeMX still knows about the mapping

Hm, this is weird indeed. These MCUs all use the STM32F042_gpio_v1_0 definitions:
STM32F042T6Yx.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>
STM32F042K(4-6)Ux.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>
STM32F042G(4-6)Ux.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>
STM32F042K(4-6)Tx.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>
STM32F042F6Px.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>
STM32F042F4Px.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>
STM32F042C(4-6)Ux.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>
STM32F042C(4-6)Tx.xml:	<IP ConfigFile="GPIO-STM32F0xx" InstanceName="GPIO" Name="GPIO" Version="STM32F042_gpio_v1_0"/>

However, that IP file only seems to contain GPIO_AF5_I2C1 mappings for PA11/PA12.
At the same time I can confirm that PB13 / PB14 have the I2C1 functionality available in STM32CubeMX... :confused:
Maybe some "common" functionality that's inherited from another file?


Refactor test setup, add interface to write DIDs on ledger

Codecov Report

Merging #921 (79f948a) into purge/vdrtools-zmq (e6d4889) will decrease coverage by 0.09%.
The diff coverage is 44.44%.

@@                  Coverage Diff                   @@
##           purge/vdrtools-zmq     #921      +/-   ##
======================================================
- Coverage               44.26%   44.18%   -0.09%
======================================================
  Files                     415      415
  Lines                   29139    29147       +8
  Branches                 6233     6233
======================================================
- Hits                    12899    12879      -20
- Misses                  12433    12460      +27
- Partials                 3807     3808       +1

Flag
Coverage Δ

unittests-aries-vcx
44.18% <44.44%> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

aries_vcx/src/common/ledger/transactions.rs
42.39% <0.00%> (-0.60%)
:arrow_down:

aries_vcx/src/utils/constants.rs
0.00% <ø> (ø)

...ries_vcx/src/utils/mockdata/profile/mock_ledger.rs
0.00% <0.00%> (ø)

aries_vcx/tests/test_pool.rs
83.69% <ø> (ø)

aries_vcx_core/src/ledger/base_ledger.rs
0.00% <0.00%> (ø)

aries_vcx_core/src/ledger/indy_vdr_ledger.rs
30.00% <0.00%> (-1.75%)
:arrow_down:

aries_vcx_core/src/wallet/indy/wallet.rs
40.60% <0.00%> (-1.74%)
:arrow_down:

aries_vcx/src/utils/devsetup.rs
48.44% <63.49%> (-0.92%)
:arrow_down:

aries_vcx/tests/utils/devsetup_alice.rs
73.15% <66.66%> (-1.22%)
:arrow_down:

aries_vcx/tests/utils/devsetup_faber.rs
81.57% <66.66%> (-0.78%)
:arrow_down:

... and 1 file with indirect coverage changes


Wage error

Wages are not consistently displayed correctly. This could be an issue with the HTTP response, or with the asynchronous nature of js.

Fixed!


Unbound variable return type

the create_wallet method in wallet.py has an unbound return value if no payload is specified.
if wallet_payload:
        wallet_response = await aries_controller.multitenant.create_subwallet(
            wallet_payload
        )
 return wallet_response

should be
return await aries_controller.multitenant.create_subwallet(wallet_payload)

and the default None for wallet_payload should be removed to make it required.

Will fix this. Good catch!


[Proposal] Implement Css Modules

I have added Css Modules, with a working example. I have seen a few request for it. I also verified that with commit it will work with react-toolbox no changes necessary.
@coryhouse, I am on the fence on actually including merging this. I would be fine leaving it as a how-to implement.

Coverage remained the same at 94.928% when pulling d993be3b830c3b734e31643c6a98ceb71f57eb90 on kwelch:feature/css-modules into 49fb922032f3366daffbd8badab636642c7b7b4c on coryhouse:master.

Since this is a React starter, I'd vote to leave this out. I added Sass-loader initially because it's very popular but in retrospect I arguably shouldn't have taken an opinion on styling at all. There are literally over a dozen ways to handle CSS and I don't see a clear winner at this point.

I agree. I am a fan of it because it put styling at the component level, but I agree I would not call it the clear winner yet. I am interested in seeing how libraries like aphrodite land, because it so easily translate to what you expect on the React-Native side.
I will close.


Merge Pool

How long would the button be disabled for and would the user need to have the button in the end product? Couldn't it just be disabled without the button?

What are you talking about?

How long would the Merge Pull Request Button be disabled for?
https://docs.gitlab.com/ce/user/project/merge_requests/work_in_progress_merge_requests.html

I think you meant to just comment on https://github.com/sindresorhus/refined-github/issues/620 or open this issue on https://github.com/sanemat/do-not-merge-wip-for-github.

Yeah i was commenting on @kshvmdn post about the wip block


Override the nginx image properly in tinkerbell stack chart

Issue #, if available:
Description of changes:
Nginx images were not getting overridden properly in the values file. This makes sure we are overriding the nginx image in the tinkerbell stack chart with the one in our bundles.
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.20


Return Person details in Booking endpoint responses

Replace the previous mocked Person properties of the Bookings responses with actual data.

 Add person details to API responses

LGTM - Just one question - a3a7da1 seems to be adding config, and 75e5cc1 removes it. Is this needed?

Not needed any more, it was from when I was getting the client_credentials JWT manually rather than letting Spring do it


Fixing #201 by treating status code 1223 as a success code.

Can your restore the comment and add to it why this code is needed?

This fix is available in 0.9.1


Configure mesh coordinate system

Description

Implements #2322
Add a new combo box and button to camera settings for mesh viewer
Add a new dialog window for custom axis mappings
Implement the axis mappings in gl driver (other drivers to come)

This looks really good now! Having tried it, I think the behaviour of immediately popping up the axis dialog on selecting 'Custom' feels right, would you agree?
The only thing is that since you've removed the explicit button to edit the mapping, there's no way for the user to go back into fix a mistake if they selected the wrong thing without selecting a different axis mapping and then back to custom. That's also the only way currently to view what the custom mapping is - which I think is fine, but it needs a way to go back to that dialog. Even if we allowed them to "re-select" custom to pop up the dialog, that's not very intuitive. I think it would be good to keep the button you had previously, but still preserve this behaviour where it automatically pops up when selecting custom.
One other minor note - similar to how the matrix parameter inputs are disabled when looking at the VS input (because they're not used), we should probably disable the axis mapping drop-down/button when looking at the VS output. It doesn't do much harm but it could confuse people if they expect it to work and don't understand why it has no effect.
The CI failure is a check to make sure all python structs/functions are listed in the documentation - not very critical for structs like this but it's better to have it listed. If you look in docs/python_api/renderdoc/outputs.rst I think the AxisMapping struct would fit best next to the items at the bottom under the Mesh View heading.

Also, do you want us to work on applying the custom mappings in the other drivers now and add them to this PR or tackle those in a follow up PR?

Sorry I meant to respond to this too - that's up to you really, I don't require strict feature parity in PRs like this where it's not a significant burden to handle it myself, so after the above issues are resolved I'm happy to merge this as-is. If this were a large scale feature that I would be unlikely to implement myself on other APIs, I wouldn't expect support for all APIs in a single PR as it's fine to add things piece-wise but I would expect follow up PRs for the other APIs at some stage.
I would certainly very much appreciate you adding support in the other drivers, as I'd want that before I close the linked issue, but whether you want to keep working on that in this PR or do a follow-up, or even do neither, any of those options are fine. Just let me know what you plan on doing so I can extend the support myself if needed :smile:.

@baldurk We've made the changes to address your latest review comments. We've also decided to handle custom mappings for the other drivers in a follow up PR.

Thanks, I've merged now. Just a heads up - I'd appreciate a PR for the other APIs within the next 2 weeks or so, otherwise I'll likely look at implementing it myself. The next stable version will be coming out at the end of this month and I think it makes more sense to ensure the other API implementations are complete - rather than either having confusingly non-functional UI or setting up something to make it conditionally disabled/enabled depending on the underlying API.


VSCode setup as an option for non devcontainer setup

Many people like vscode, many dont. I personally would like an option to include some vscode setup like in the container base, mainly for the debugging options.
Setup debugging is hard and we should make it as simple as possible for people.
Though in light of not overbloading #95  the generating process what you guys think?

Not a full solution, but since we are adding a Wokwi standalone prompt that adds support for Wokwi Vs Code extension, it will include a .vscode/launch.json with the required settings to debug

one potential problem, depending on what debugging extension we are using, it need to be installed by the user. So using cppdbg would need the microsoft C/C++ extension, using corged-m-debug would Cortex-Debug extension etc. And iirc VSCode is not smart enough to figoure it out. Have to look into it though, maybe we can find something. We dont have this problem in the container where we can setup the vscode extension by ourself.


zone_lockdown: add Priority to ZoneLockdownCreateParams and ZoneLockdownUpdateParams

Codecov Report

Merging #1052 (27dc22c) into master (6c5ea4a) will increase coverage by 0.67%.
The diff coverage is 62.94%.

@@            Coverage Diff             @@
##           master    #1052      +/-   ##
==========================================
+ Coverage   49.06%   49.73%   +0.67%
==========================================
  Files         108      115       +7
  Lines       10428    10965     +537
==========================================
+ Hits         5116     5453     +337
- Misses       4200     4347     +147
- Partials     1112     1165      +53

Impacted Files
Coverage Δ

access_application.go
76.52% <ø> (ø)

access_bookmark.go
72.44% <ø> (ø)

workers_account_settings.go
35.71% <35.71%> (ø)

rulesets.go
35.09% <47.36%> (+7.43%)
:arrow_up:

workers_tail.go
52.00% <52.00%> (ø)

workers_subdomain.go
57.14% <57.14%> (ø)

email_routing_settings.go
60.00% <60.00%> (ø)

email_routing_rules.go
65.64% <65.64%> (ø)

lockdown.go
57.69% <66.00%> (+6.94%)
:arrow_up:

email_routing_destination.go
66.25% <66.25%> (ø)

... and 12 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


ThrottleForScheduledScanOnly parameter is missing in Set-MpPreference

ThrottleForScheduledScanOnly parameter is available in Windows 11 build 22621.1778 Stable channel in Defender module's Set-MpPreference cmdlet but isn't listed in the document page.

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

ID: 44f43117-c0f9-84e2-9a2a-0335f516f7d0
Version Independent ID: cc8d93f2-8837-4169-0fc7-167d0ffca777
Content: Set-MpPreference (Defender)
Content Source: docset/winserver2022-ps/defender/Set-MpPreference.md
Product: w10
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

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


test for process_forwarded_payload fails because ws can't connect

When I run tests like described here I get:
 testing process_forwarded_payload()
[2020-02-08T21:02:47Z INFO  substratee_worker::tests::commons] got rsa shielding key form enclave
[2020-02-08T21:02:47Z INFO  substratee_worker::tests::commons] deserialized rsa key
[2020-02-08T21:02:47Z INFO  ws] Queuing connection to ws://<IP_ADDRESS>/
[2020-02-08T21:02:47Z ERROR ws::io] Unable to establish connection to ws://<IP_ADDRESS>/: WS Error <Internal>: Unable to obtain any socket address for ws://<IP_ADDRESS>/

this doesn't make sense, the substraTEE-node is running and listening. Where does the strange IP come from?

@brenzi is this still an issue? I haven't heard of process_forwarded_payload


Small cleanup for named lazy member loading

Now that we never fail out of the fast path, we can simplify some logic in the surrounding code. Follow-up to https://github.com/apple/swift/pull/26975.

@swift-ci Please smoke test

@swift-ci Please test source compatibility

Can one of the admins verify this patch?

@xymus Do you mind taking a look at the serialization side of this PR?

@swift-ci Please smoke test

@swift-ci Please test source compatibility


Existing Cluster Constructs

Is the blueprints.ClusterAddOn usable on existing cdk eks.Cluster Constructs / ICluster interfaces? It would be awesome to be able to ship ClusterAddOns to existing clusters deployed using aws-cdk. I'm not sure if that is the point of the blueprints, but it would extend very nicely on top of people who code their own Clusters or have them already running.

I will add an example of importing an existing cluster and applying add-ons to them.

@shapirov103 Is it also possible to provide an example in the documentation on how to use blueprints in combination with existing eks.Cluster constructs?
e.g.
import { Cluster } from 'aws-cdk-lib/aws-eks';
<...>

export class InfrastructureStack extends Stack {
    private cluster: Cluster;

    constructor(scope: Construct, id: string, uniqueName: string, owner: nnOwner, featureBranchToggle: boolean, props?: StackProps) {
        super(scope, id, props);

        this.cluster = new eks.Cluster(this, 'eks', {...});
        <...use blueprint 'Construct' ClusterAddOn to build on top...>

Will add an item to show leveraging existing cluster and add add-ons / teams.


IndexOutOfRangeExceptionBridgeVs.Grapple.Grapple.BinaryGrapple in Grab

IndexOutOfRangeException: Index was outside the bounds of the array.
Module "BridgeVs.Grapple.Grapple.BinaryGrapple", line 0, col 0, in Grab
System.Tuple`2[System.Type,System.Byte[]] GrabT
Module "BridgeVs.Grapple.Truck", line 16, col 0, in LoadCargo
Void LoadCargoT
Module "BridgeVs.DynamicVisualizers.DynamicObjectSource", line 355, col 0, in BroadCastData
Void BroadCastData(System.Object, System.IO.Stream)

fixed in the next release


NugetAuthenticate and PowershellGet Nuget provider

Type: Question
Enter Task Name: NuGetAuthenticate
Environment

Server - on-premises - version Dev17.M153.5
Agent - Private - version 2.153.1
Powershell version 5.1.17763.1007
OS - Windows Server 2019 Standard, build 17763.1217

Issue Description
We have started to use Azure Artifacts and needed a way to authenticate with the feeds.
I have built NuGetAuthenticate locally and installed it on our on-premises instance. It works perfectly in all places where we call nuget.exe directly or use provided nuget tasks.
But we also have a custom task that installs powershell modules. The concept is the same that exists for installing build tools. We create a temporary directory, add it to PsModulePath, call Register-PsRepository if it has not been registered yet and then we use Save-Module to place the module in that directory. The details of the logic do not matter that much, it's been working for a couple of years now with the preauthorized feed url.
I hoped that Powershell nuget provider would work with nuget credential provider which is installed by NuGetAuthenticate, but when I try to install modules after running NuGetAuthenticate I am getting an error that says that I need to provide credential to access the feed. It means that it does not work for installing PsModules.
My question: do you see a way of making NuGetAuthenticate task to install credential provider in such a way that PowershellGet nuget provider can use it? Or maybe there is a way for me to create System.Management.Automation.PSCredential object when we run under the build by extracting username and PAT from the service connection?
Error logs
This is the error that I get from Save-Module cmdlet:
System.Exception: Cannot access 'https://pkgs.dev.azure.com/...snip.../nuget/v3/index.json'. Are you missing 'Credential' parameter in the cmdlet?

Hi @dpurge ,
This feature does not currently exist. Could you create a suggestion ticket in this repo?

Hi @apluche, thank you for your response. Apologies for the delay in my answer, I was trying to find instructions how to create a "suggestion ticket" in this repo, could not find either instructions or an example and later was a bit busy :)
Should I just create a new ticket and explain what I would expect and why I think it's useful?
It would be a copy of this question - so should I do that?
FYI - since the functionality did not exist, I have already rewritten our task. Now I simply call nuget.exe to download the package, so that we can use credential providers installed by NuGetAuthenticate. It works, therefore I no longer have that much interest in this functionality, but still think Azure Artifacts not working for hosting powershell modules "out of the box" is not ideal :)
Since you have answered my question, I think it's right to close the ticket.

Hi @dpurge , a suggestion ticket is the same as an issue but they type that is assigned to it is different. I'm glad to hear that you were able to unblock yourself. If you would like Azure Artifacts to consider this feature, please make sure to create a ticket since we keep track of those in our backlog.

Created suggestion for a new feature: #13215


Fix AUTO_ALIGNOF copy/paste typo

I will merge this separately, but note that this change is also present in #666.


[17.03] firefox: 54.02 -> 55.0.2

Motivation for this change
Firefox update
Things done

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

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Note: I'm unable to test the build on my local system due to low battery issues, so I have to rely on the CI check.

Doesn't build yet:
DEBUG: configure:10890:23: error: #error "Insufficient libpng version (10629 required)."

Added libpng update required by firefox (minimum version 1.6.29)

Yep... firefox also needs a newer sqlite...

Is the commit order so that the required library updates come before the Firefox update?

@bjornfor no, its in the order of firefox's "i need newer versions!" complaints.
During the night I confirmed that the build works. Haven't had the time to verify that it actually runs, though.

@tokudan: Please reorder the commits to not break "git bisect". (PRs shouldn't introduce breakage, even temporary, if it's easy to avoid.)

@bjornfor is there is a way to do that through the github website? If not, i probably won't be able to do that before tomorrow evening.

@tokudan: I don't think so. Better use the git rebase -i ... on the command line. (No rush.)

Let's have another go at this... I have reordered the commits and am trying to build this right now.

Ok... the build is finally done and firefox 55.0.2 actually runs.
I've updated the checks in the first box. I believe this PR can be merged now.
Please note: I have NOT checked all packages that depend on libpng and sqlite... that would take weeks.


Update README.md

the change of nightshadetvn -> rudy3333, is because it's my dead name.

Thanks and welcome back :)


[BUG] Button Issue in Android and iOS

Button and List Message Not working on Android ios

Buttons and List  are deprecated

#56

I tested the buttons and the list again, they worked again on IOS.

Remove button from docs, please.


setunits() output has bad mass line

julia> setunits()
mass:	 "eV/c^2"  	 where 1.0 eV/c^2               = 1 amu    <---- This is wrong conversion between eV/c^2 to amu !!
length:	 "m"       	 where 1.0 m                    = 1 m
time:	 "s"       	 where 1.0 s                    = 1 s
energy:	 "eV"      	 where 1.0 eV                   = 1 eV
charge:	 "e"       	 where 1.0 e                    = 1 e

I think this is just a typo in printing the mass unit at the end of that line.
This is fixed in #35 .


How does run-workers work?

What are you trying to achieve?
I'm tryin to execute command:

npx codeceptjs run-workers 2 [4, 8]

and I expect to get as much chrome browsers as workers count
What do you get instead?
1-2 chrome opens and that's it, no more browser windows (cpu and mem are enough atm)
Details

CodeceptJS version: 2.3.1
NodeJS Version: 12.10.0
Operating System: MacOS Mojave
TestCafe: 1.5.0
Configuration file:

exports.config = {
	tests: './*_test.js',
	output: './output',
	helpers: {
		TestCafe: {
			url: 'myUrl',
			browser: 'chrome',
			show: true
		},
		MailSlurp: {
			require: '@codeceptjs/mailslurp-helper',
			apiKey: 'validApiKey',
			timeout: 60000
		},
	},
	include: {
		"mainPage": "../pages/mainPage.js",
		"osagoPage": "../pages/osagoPage.js",
		"userClicksForgotNumber": "./steps/userClicksForgotNumber.js",
		"userSavesStateByEmail": "./steps/userSavesStateByEmail.js"
	},
	bootstrap: null,
	mocha: {},
	name: 'productName',
	translation: 'ru-RU',
	timeout: 10000
};

UPD 1.
When I've executed command:

npx codeceptjs run-workers 2 --verbose

There were spawned multiple errors all containing this:

[1] Error | Error: The specified 1341 port is already in use by another program.

UPD 2.
I've digged into code, first codeceptjs's, then testCafe's and, boy, what do I see:
async function getValidPort(port) {
    if (port) {
        const isFree = await endpointUtils.isFreePort(port);
        if (!isFree)
            throw new runtime_1.GeneralError(types_1.RUNTIME_ERRORS.portIsNotFree, port);
    }
    else
        port = await endpointUtils.getFreePort();
    return port;
}

TestCafe can manage ports by its own, so, why do we need this kind of management:
// in ctor
this.iteration = 1;
...
// and then
async _startBrowser() {
    this.dummyTestcafeFile = createTestFile(global.output_dir); // create a dummy test file to get hold of the test controller

    this.iteration += 2; // Use different ports for each test run
    // @ts-ignore
    this.testcafe = await createTestCafe('localhost', 1338 + this.iteration, 1339 + this.iteration);

Why can't we just pass null to a ports in previous line?
UPD 3.
I've made a fork and applied my suggestions and that works

Thank you for investigation!


Tutorial - Extension Package

Description Of Changes
Adds a brief tutorial for creating an extension package.
Motivation and Context
We were requested to add short tutorials for this purpose.
Testing

[x] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

~[ ] Minor documentation fix (typos etc.).~
~[ ] Major documentation change (refactoring, reformatting or adding documentation to existing page).~
[x] New documentation page added.
~[ ] The change I have made should have a video added, and I have raised an issue for this.~

Change Checklist

~[ ] Requires a change to menu structure (top or left-hand side)/~
~[ ] Menu structure has been updated~

Related Issue
Fixes #914

Can you move the fixes to the other pages to a separate PR?


Overwrite Content-Type

We got a server that only accepts Content-Type: application/json and not Content-Type: application/json; charset=utf-8;
OkHttp keeps generating application/json; charset=utf-8; when we do the following json post
MediaType type = MediaType.parse("application/json");
String json = "{"APIRequests"}";
Request request = new Request.Builder()
.url(apiURL)
.post(RequestBody.create(type, json))
.build();
Response response = client.newCall(request).execute();
String html = response.body().string();
and returns an error. Is there any posibility to overwrite the content-type and not use the generated one from MediaType.parse?
Server it's a public one and i do have no control over it

First, fix your server.
Second, work around the problem by manually converting the string to UTF-8 bytes with getBytes() and use the byte[] to create the response body.

I said server it's not mine, so i need to send proper content-type

Got it. Please report a bug against the server.

Ok thank you
On Tue, Dec 15, 2015 at 3:25 AM, Jesse Wilson<EMAIL_ADDRESS>wrote:

Got it. Please report a bug against the server.
—
Reply to this email directly or view it on GitHub
https://github.com/square/okhttp/issues/2099#issuecomment-164612015.

First, fix your server.
Got it. Please report a bug against the server.

We should have an option to disable/override this behavior. I'm sending custom Content-Type and Accept headers to the server (in my company) as a way to implement api versioning. This is currently messing with our endpoint calls.
Glad to help if you need.

The ResponseBody API that accepts a String is the one that’s opinionated about charsets on content types. If you want to do your own thing, just convert your string to bytes first and use that API.

The issue is old and closed, but I'll just point out that this was really unexpected when I set the Content-Type to the specific value the Layer API expects (a popular chat service for applications) and it got overwritten.
Though thankfully the OkHttp network interceptors exist and can override anything!

If this is troubling you, you don’t need to have OkHttp pick a charset for you. Just do this:
    String requestBody = ...
    Charset charset = ...
    byte[] bytes = requestBody.getBytes(charset);
    return RequestBody.create(contentType, bytes);

I'm faced with a similar issue when using okhttp in combination with Elasticsearch 6.1.2
The type MediaType.parse("application/x-ndjson"); is rejected by the server due to the included encoding.  Is there an RFC which requires the encoding? The workaround by @swankjesse works fine.

I have the following code:
`
String xmlRequest = xmlMapper.writeValueAsString(updateLoginIdRequest);
byte[] bytes = xmlWithNamespace.getBytes(StandardCharsets.UTF_8);
RequestBody requestBody = RequestBody.create(bytes);
Request request = new Request.Builder()
.url(urlBuilder.toString())
.header("Content-Type", "application/x-wamupdateuserloginid+xml;version=1")
.put(requestBody)
.build();
`
The server is rejecting the request because the custom content type header is not correct. What am I doing wrong?


Register-PSRepository code snippet is unhelpful

The following code snippets should be revised:
Register-PSRepository -Name "PowershellAzureDevopsServices" -SourceLocation "https://<org_name>.pkgs.visualstudio.com/<project_name>/_packaging/<feed_name>/nuget/v2" -PublishLocation "https://<org_name>.pkgs.visualstudio.com/<project_name>/_packaging/<feed_name>/nuget/v2" -InstallationPolicy Trusted -Credential $credsAzureDevopsServices
And
Find-Module -Repository PowershellAzureDevopsServices
The first code snippet uses the -Credential parameter on Register-PSRepository which was never implemented and doesn't do anything. In fact, it also appears to have been removed and credential persistence is being re-evaluated according to this issue:
https://github.com/PowerShell/PowerShellGet/issues/102
Instead, the page should show that you need to register the repository without the credentials and instead, Find-Module should be passed the -Credential parameter instead.

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

ID: 35b3cd51-b365-433b-9149-697582d64e88
Version Independent ID: 1cb1f4a3-5928-223d-592f-f3462fef4e4c
Content: Use Azure Artifacts as a private PowerShell repository - Azure Artifacts
Content Source: docs/artifacts/tutorials/private-powershell-library.md
Product: devops
Technology: devops-artifacts
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@catbusstop Thank you for the feedback.
This issue has been previously reported under https://github.com/MicrosoftDocs/azure-devops-docs/issues/8145, please refer there for further updates.


Support FORCE_COLOR and NO_COLOR

https://force-color.org and https://no-color.org are "community standards" supported by many tools, such as ruff, pip, pytest, rich, nox, tox.
It would be nice to set the FORCE_COLOR: 1 env var in GitHub Actions so I can have coloured output in the CI. It makes logs more readable, especially when there's a failure.
uv does have a --color CLI option, but more and more tools are using uv behind the scenes (such as https://github.com/hynek/build-and-inspect-python-package) so I don't have direct access to the uv invocation.

Yeah good call -- I think we might respect one of these... NO_COLOR, at least? Via https://docs.rs/anstream/latest/anstream/struct.AutoStream.html. I'll look into it.

Yeah we do support NO_COLOR, but not FORCE_COLOR=1.

It does seem like anstream also supports CLICOLOR_FORCE which seems to do the same (in premise) as FORCE_COLOR, so maybe a doc update?

We might need to validate the precedence here too with the command-line arguments.

I already have FORCE_COLOR on the CI for other tools. I'd prefer if uv supported FORCE_COLOR, so I don't need to add an extra env var just for uv.
I forgot to mention it earlier, CPython 3.13 also supports FORCE_COLOR:

https://docs.python.org/3.13/using/cmdline.html#controlling-color

Yeah we should support it -- we also support it in Ruff for this reason.

Thanks for looking into it in anstream @samypr100, you rock.

Should we add CLICOLOR_FORCE as well so that it's respected over the color CLI argument?

FYI anstream respects NO_COLOR and CLICOLOR / CLICOLOR_FORCE.  Until recently, FORCE_COLOR has been very ad-hoc with people copying patterns of it and extending it in bespoke ways without any coordination.  Looks like someone is now trying to coordinate that effort though I think I want to be cautious and see how well their variant takes off vs any other.  I'm also concerned that both CLICOLOR and FORCE_COLOR exist without specifying their interaction.
see also my comments on rust-cli/anstyle#192

Thanks @epage. For some reason FORCE_COLOR has become very popular in the Python ecosystem -- you can see all the listed projects here: https://github.com/astral-sh/ruff/issues/5499#issue-1787106134

That raises even more questions for me as none of the implement https://force-color.org/ (though most are likely "close enough").

Non-empty FORCE_COLOR

https://force-color.org/

Checks for presence of FORCE_COLOR rather than checking of FORCE_COLOR is non-empty

https://github.com/Textualize/rich/blob/master/rich/console.py#L952-L956
https://github.com/wntrblm/nox/blob/main/nox/_options.py#L605
https://github.com/pytest-dev/pytest/blob/97ed533f63d5780a05702a711555cb6744247a37/src/_pytest/_io/terminalwriter.py#L26-L37
https://github.com/pypa/build/blob/main/src/build/__main__.py#L46-L53

treats it as a bool-ish

https://github.com/tox-dev/tox/blob/main/src/tox/config/cli/parser.py#L352-L361
(rust) https://github.com/zkat/supports-color/blob/d5c906d1fcd91cd1bf92d22f3d0cca452de8ca9f/src/lib.rs#L38-L43 (I believe this was modeled off of a JS package)

Note that higher numbers express certain capabilities of the terminal

treats it as an integer-bool

https://github.com/python/mypy/blob/master/mypy/util.py#L552-L557

So what approach is "right"?  https://force-color.org/ is just a website that some person put up within the last 9 months.  That doesn't make it definitive.

I've raised the discrepancies in donatj/force-color.org#30


Kubernetes-workload test migrated to go

Signed-off-by: Ritikaa96<EMAIL_ADDRESS>Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added

Fixes #3239

Relates to #2737

@tomkerkhove @JorTurFer PTAL

/run-e2e workload

Could you split the tests in different methods for testing the scaleUp and scaleDown? It's the approach for other scalers, and even it's only organizational, it makes the tests consistent. For the rest, looking good heart

sure! @JorTurFer

/run-e2e workload

this shows its successful however the check shows it didnt , cant see what is causing this error here. @JorTurFer Any idea?

The comments with the command /run-e2e are updated by the process with the execution url, you can see the logs there.
BTW, the command shows also that is has failed 👎

this shows its successful however the check shows it didnt , cant see what is causing this error here. @JorTurFer Any idea?

The comments with the command /run-e2e are updated by the process with the execution url, you can see the logs there. BTW, the command shows also that is has failed -1

Hm.. its strange its working fine in my environment . might recheck for anything causing the e2e to fail in the check.

Hi @JorTurFer PTAL

/run-e2e kubernetes_workload*

/run-e2e kubernetes_workload

Thanks for this contribution! ❤

This change was missed again - https://github.com/kedacore/keda/blob/9fe478d0bc6ec18bfc66b9af87db16e303ca1134/tests/run-arm-smoke-tests.sh#L7-L12

f**k me, and I have done the activemq scaler and I have forgotten it again, thanks a ton for the heads up


Meteor <IP_ADDRESS> user's collection no longer getting helpers?

Hi, I'm getting a regression with my user collection's helpers.
The user collection is defined essentially like this:
Schema.User = new SimpleSchema({
  // So many fields
});

Meteor.users.attachSchema(Schema.User);
Meteor.users.attachSchema(Schema.AutoTimestamps);

var userHelpers = {
  /* All the helpers */

  checkReputation: function(access) {
    check(access, String);
    if (!Meteor.userId()) throw new Meteor.Error('accessDenied', 'access denied');

    // Check to see if the user has enough reputation
    switch (access) {
      case 'alwaysHappy':
        return true;
      default:
        return false;
    }
  },
};
Meteor.users.helpers(userHelpers);

Meteor.users.before.update(function (userId, doc, fieldNames, modifier, options) {
  /* Update some stuff */
});

Meteor.users.after.update(function (userId, doc, fieldNames, modifier, options) {
  /* update some stuff */
});

And its used in a method like this;
  clientCreateOpportunity: function(op) {
    check(op, { /* details */});

    if (!Meteor.userId()) throw new Meteor.error('accessDenied', 'access denied');
    console.log(Meteor.userId());
    if (!Meteor.user()) throw new Meteor.error('accessDenied', 'access denied');
    console.log(Meteor.user());
    if (!Meteor.user().checkReputation('alwaysHappy')) throw new Meteor.error('accessDenied', 'access denied');
    /* Do more things */
  }

Meteor.user() used to have the helper function, and now it doesn't - in the last week or so.
I'm using Meteor <IP_ADDRESS> and these packages (summary):
aldeed:autoform                 5.8.1  Easily create forms with automatic insert and update, and automatic re...
aldeed:collection2              2.9.1  Automatic validation of insert and update operations on the client and...
aldeed:template-extension       4.0.0  Adds template features currently missing from the templating package
dburles:collection-helpers      1.0.4  Transform your collections with helpers that you define
iron:router                     1.0.12  Routing specifically designed for Meteor
matb33:collection-hooks         0.8.1  Extends Mongo.Collection with before/after hooks for insert/update/rem...
meteorhacks:cluster             1.6.9  Clustering solution for Meteor with load balancing and service discovery.
meteorhacks:kadira              2.28.5  Performance Monitoring for Meteor
meteorhacks:kadira-profiler     1.2.1  CPU Profiler for Meteor (used with Kadira)
ongoworks:security              2.0.1  Logical security for client-originated MongoDB collection operations
reywood:publish-composite       1.4.2  Publish a set of related documents from multiple collections with a re...
zimme:collection-behaviours     1.1.3  Define and attach behaviours to collections
zimme:collection-softremovable  1.0.5  Add soft remove to collections

What changed?  Did I miss an update?
Workaround (doesn't work):
As a workaround, I did this:
    if (!Meteor.userId()) throw new Meteor.error('accessDenied', 'access denied');
    if (!Meteor.user()) throw new Meteor.error('accessDenied', 'access denied');
    var u = Meteor.users._transform(Meteor.user());
    if (!u.checkReputation('alwaysHappy')) throw new Meteor.error('accessDenied', 'access denied');
    ...

I'm still getting this error with the work around.

Hey thanks, looks like an internal change in Meteor has caused this, I'll look into it as soon as possible

Hey @MichaelJCole I've had a look into it and everything works just fine, so I think the issue may be with how you're handling things at your end

If you do feel there is a bug however, if you can reproduce the issue in an isolated example app I'm happy to take a look

Hey @dburles,
The problem was caused by Meteor.error() instead of Meteor.Error().  It was the error that was undefined.  Dammit.
Sorry for the static and if I wasted your time.  Really appreciate this module, it's so useful, I barely notice it.


Coverage % could be more precise

On large projects, 1% of coverage can represent a lot of lines/files.
Having a float (ex: 73.34%) instead of an int (73%) for the global coverage would be really useful to see the coverage impact of a commit/branch/other

Hello @TiteiKo
A decimal format for the global rate is a very good idea :+1:
Let me know if you would like to implement it :wink:

I'm on it, just one question... do you think it should be configurable via an option?
As most people use it on their CI with parsers to get the coverage, breaking the format wouldn't be very nice...
So I was thinking an option (cover_rounding?) by default at 0, but I'm currently fighting against the Configuration that needs to be passed on to the Summary.... And now it's a mess... I'm almost thinking of making the Configuration available from anywhere..

Just be simple, please ;) I think you're right that 73.34 is better than 73.
So we should use 73.34 as the default, isn't it?
With this hypothesis, don't bother with an option.
And it's not yet version 1.0 ;)
What do you think?

Hell yeah!


statistics of datasets

Hi, thank you for your excellent work! I found that /data/statistic/ did not have the .txt split files of other datasets. Is there any way to access these files?

I've found them, I'm going to close this issue.

@TungWg Hi, I just upload the split files hhh.

Where did you find them?

I found them in detection_results.tar.


Fix a mis-namespaced TokenStream ref

This should fix the broken build.
I was trying to figure out how this ever worked, and why it suddenly broke. Looks like proc-macro-error introduced a "breaking" (IMO this never should have worked) change from 1.0.2 to 1.0.3, as everything is green if you pin proc-macro-error to =1.0.2. I'm an absolute novice here, but this jumped out.

@wafflespeanut can you make a patch release for this?

Yes! I'm planning to do it tonight!


Retrieving elements from ordered set that were updated in given time interval

We use an ordered set for registering heartbeats of devices connected to our backend via websockets. Discovering that Kredis stores scores as float timestamps, I realized I could use ZRANGE to retrieve device IDs that have either recent or stale heartbeat:
heartbeats = Kredis.ordered_set("devices:heartbeats")
heartbeats.zrange("-inf", 1.minute.ago.to_f, byscore: true) # returns device IDs with stale heartbeat
heartbeats.zrange(1.minute.ago.to_f, "+inf", byscore: true) # returns device IDs with recent heartbeat

I recently created a helper method that accepts a time range to make this easier:
def heartbeat_ids(range)
  from = range.begin&.to_f || "-inf"
  to = range.end&.to_f || "+inf"

  heartbeats = Kredis.ordered_set("devices:heartbeats")
  heartbeats.zrange(from, to, byscore: true)
end

heartbeat_ids(...1.minute.ago) # stale heartbeats
heartbeat_ids(1.minute.ago..) # recent heartbeats

I was wondering whether there is interest in adding a similar method to Kredis::OrderedSet, or make #elements accept an optional time range.

@janko Looks convenient and appropriate for this type in my view.
Half-relatedly, I wonder if Basecamp is still using Redis and Kredis internally, and whether this gem still makes sense for their team to maintain? 🤔


eventRender function in options

I was trying to use the function eventRender which is passed through in the CalendarOptions as below:
eventRender: function (event, element) {
console.log('Rendering event', event);
return true;
},
From the fullcalendar documentation: https://fullcalendar.io/docs/event_rendering/eventRender/
Is this possible or am I implementing it incorrectly ?
The issue is that the function does not appear to run.

Output 8 EventEmitter (eventRender)="eventRenderFunction($event.detail)", version 1.2.0

Update to version 1.2.0


merge client into main application

As the client seems small and simple, it might be worth integrating it into the main application. We can introduce the following flags --server and --client.
What do you think @bozerkins?

Sounds like a promising idea.
This way we could also build a single docker image for both server and client, instead of multiple docker images.
This change would be easier to do after we merge #122


Implement external potentials on a grid

In Monte-Carlo simulations, when working with a fixed framework a nice way to compute the energy is to pre-compute the potential (electrostatic or non-bonded) on a grid, and then interpolate the energy for each particle given its position.
For electrostatic potential, the idea to compute the electrostatic potential U on the grid, and then get the energy as q U. For Lennard-Jones potential, we need one grid for each atom type of the moving molecule, and we can precompute the energy by placing a particle on each point of the grid and using the standard energy computing method.
Then, we can save the pre-computed grids to the disk, and reload them in each simulation using this grid. This is very useful for adsorption in a fixed framework, where you can reuse the same pre-computed grid in multiple simulations, and you don't need to simulate the framework particles.
Implementation strategy
We can have two steps here: use a new kind of Simulation which compute the grid and output it to the disk; and add a new kind of GlobalPotential that read the cube file and can be used in the [potential] section of the input files.
I think the Gaussian cube file format will be nice to use here, as it represent volumetric data, and can be visualized with standard tools. The only downside is that we need more that one file for a simulation.

Interesting. For adsorption, you would still need the grid positions to compute the interactions with the adsorbent, right?
A simple alternative for adsorption (if we'd have particle insertion) would be to add the grid/solid structure, but don't add any moves and then add specific moves for the adsorbed species. Would the "grid method" be more useful in that case?

For adsorption, you would still need the grid positions to compute the interactions with the adsorbent, right?

I am not sure what you mean here. You don't need the positions of the atoms in the framework, because they are already accounted for in the pre-computed grid.
The point here is that once the grid is computed, computing the interaction of a particle with the grid (so with the rigid structure) is a O(1) operation, while computing the interaction with all the atoms in the structure is still an O(n) operation.

Ah, I see. You would take the adsorbing molecule and move it through the whole grid beforehand. Wouldn't it be necessary to also sample multiple intra molecular configurations as well as orientations?  I guess the initial computation could be very time consuming then.

Yes, the initial step is pretty long, that why you run it as a separated simulation, and you save the result to the disk.
You don't need to sample orientation, because this work at the particle level, not at the molecule level. Which means you need one grid for each particle type.

Note: There is an explanation of the method and references to other papers for this algorithm in this paper.


Inconsistency: Unused local in compiler but not services

TypeScript Version:  3.0.0-dev.20180609
Code
for (const [_] of []) {
    const x = 0;
}

Compiling with tsc (with --noUnusedLocals set): there are two unused-local errors.
But in the editor, there is only an error at x.

This is working in the master branch now. Probably fixed by #24958. (typescript@next hasn't been updated in a week...)


install fail on win10,ubuntu (by vmware and google cloud )

i think this code game is funny ,so i try to install this for teaching my students.
but i try  install code  in  three way  by  the   doc https://github.com/codecombat/codecombat/wiki/Dev-Setup%3A-General-Information ...
and i fail all...
so i just doubt  this doc can use or not ......
if you guys have free time ,can you make a video  about install this code ..
thanks for all.......

"i just doubt this doc can use or not..."
i think so too!!!!

"i just doubt this doc can use or not..."
i think so too!!!!

haha ...i doubt you are my  laoxiang...

make sure node version 8.x.x

make sure node version 8.x.x
sure i do

homepage can open, but others can't open

"i just doubt this doc can use or not..."
i think so too!!!!

haha ...i doubt you are my laoxiang...

haha, xiangqinmen......

"i just doubt this doc can use or not..."
i think so too!!!!

haha ...i doubt you are my laoxiang...

haha, xiangqinmen......

你还没成功么？我在网上找了个docker版的 可用。不过版本很老，有需要可以留言。。

me too

Hi everyone - I recommend giving the new docker guide a try. It makes it a lot easier to get started, and you don't have to worry about the different installations or dependencies.
Would love to hear if that solves your problems!


Error installing hstr on Ubuntu 20.04.5 LTS

Hi,
I am having trouble installing hstr on Ubuntu 20.04.5 LTS. Please see screenshot below.

I added 2.6.0 release for Ubuntu 20.04 (Focal Fossa) https://launchpad.net/~ultradvorka/+archive/ubuntu/ppa/+packages


Error printedFullSerialNumber unknown / Connection lost to service or device - no error

Since October i often lost connection. But that's a nother problem i hope. The Adapter often is not connectet to service or device. Restart helps. I lost lot of date, couse i get no error in log. Can you fix it, too?
`

discovergy.0
2023-11-19 11:11:28.954
info
All meters initialized, polling data every 30 seconds

discovergy.0
2023-11-19 11:11:28.894
info
All meters associated to your account discovered, initialise meters

discovergy.0
2023-11-19 11:11:28.694
error
State  type : printedFullSerialNumber unknown, send this information to the  developer ==> printedFullSerialNumber : "1ESY1162041493"

discovergy.0
2023-11-19 11:11:28.551
info
Discovergy Adapter startet, trying to discover meters associated with your account

discovergy.0
2023-11-19 11:11:28.507
info
starting. Version 0.5.8 in /opt/iobroker/node_modules/iobroker.discovergy, node: v18.16.1, js-controller: 5.0.12

discovergy.0
2023-11-19 11:11:28.551
info
Discovergy Adapter startet, trying to discover meters associated with your account

discovergy.0
2023-11-19 11:11:28.507
info
starting. Version 0.5.8 in /opt/iobroker/node_modules/iobroker.discovergy, node: v18.16.1, js-controller: 5.0.12

`

Van you please Tex Version 0.5.13? That should solche this issue

The Error is fixed in V0.5.14 Thanks. I hope this fix the other error (Connection lost), too.
`

discovergy.0
2023-11-19 13:01:56.800
info
All meters initialized, polling data every 30 seconds

discovergy.0
2023-11-19 13:01:56.739
info
All meters associated to your account discovered, initialise meters

discovergy.0
2023-11-19 13:01:56.322
info
Discovergy Adapter startet, trying to discover meters associated with your account

discovergy.0
2023-11-19 13:01:56.303
info
starting.  Version 0.5.14 (non-npm:  DrozmotiX/ioBroker.discovergy#1bf8b04000a644e761e806744bb552eaf6eb943a)  in /opt/iobroker/node_modules/iobroker.discovergy, node: v18.16.1,  js-controller: 5.0.12

discovergy.0
2023-11-19 13:01:52.557
info
terminating

`

The Error is fixed in V0.5.14 Thanks. I hope this fix the other error (Connection lost), too.
`

discovergy.0 | 2023-11-19 13:01:56.800 | info | All meters initialized, polling data every 30 seconds
-- | -- | -- | --
discovergy.0 | 2023-11-19 13:01:56.739 | info | All meters associated to your account discovered, initialise meters
discovergy.0 | 2023-11-19 13:01:56.322 | info | Discovergy Adapter startet, trying to discover meters associated with your account
discovergy.0 | 2023-11-19 13:01:56.303 | info | starting.  Version 0.5.14 (non-npm:  DrozmotiX/ioBroker.discovergy#1bf8b04000a644e761e806744bb552eaf6eb943a)  in /opt/iobroker/node_modules/iobroker.discovergy, node: v18.16.1,  js-controller: 5.0.12
discovergy.0 | 2023-11-19 13:01:52.557 | info | terminating

`

Thank you conforming, I will review that version and push to stable this week


Cannot ping an AWS cell using go v1.15

I built kip locally with go v1.15 and I'm mounting it as a volume, as described here: https://github.com/elotl/kip-minikube/
It starts normally, but when I tried to run nginx on virtual node, it seems that node_controller cannot (first few tries result in connection refused, but I guess it's expected as EC2 instance is booting) ping the cell:
W0914 14:20:52.134617       7 node_controller.go:509] Heartbeat error from node d222e754-34f0-4667-b117-2d52e1f1e2b6: Get "https://<IP_ADDRESS>:6421/rest/v1/ping": x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0

It's not an issue for me right now, as I can always use v1.13, but I guess we need to tackle it sooner or later.
Found this discussion: https://github.com/golang/go/issues/39568

FYI: after 3 days trying to get the minikube cluster work I wasn’t able to reproduce the issue. I’m pretty sure the patch fixes it.
3 days is way too much time to spend on this, so I just merged the PR. I don’t think there is much value to be created with me spending another day confirming the bug is fixed.

FYI: after 3 days trying to get the minikube cluster work I wasn’t able to reproduce the issue. I’m pretty sure the patch fixes it.
3 days is way too much time to spend on this, so I just merged the PR. I don’t think there is much value to be created with me spending another day confirming the bug is fixed.


Find references even when TypeScript sub-projects not yet open

From https://github.com/Microsoft/vscode/issues/65102...
It seems like with a multiple-tsconfig.json setup, VS Code can't find all references. (Even though technically all of them should be "joined" via a root tsconfig.json that uses project references.)
Here's a minimal repro of my setup: https://github.com/dcecile/typescript-references-test
common/
  message.ts
  tsconfig.json
server/
  hello.ts
  tsconfig.json
client/
  sharedMain.ts
  tsconfig.json

Both client and server reference the common project
"Find references" fails to find the references to the common function
"Rename symbol" also fails to rename the references
After loading files from the client and server projects, find references works

Confirmed with TypeScript 3.5.0-dev.20190407

This is currently known issue in which if the file is not open or declaration maps are not present it cannot find all references in sibling projects. #28261 is wip for this but needs more work before can be merged in since we need to be smart about when to lookup symbol in other projects(as loading projects in most common scenarios is costly and can result in delay when someone does first time find all references)

@RyanCavanaugh I believe the Duplicate label is a mistake here.

This blocks us from splitting our code into multiple smaller projects as (consistent!) renaming across the entire codebase is very important to us. We would love to have this feature!

#28261 is wip for this but needs more work before can be merged in since we need to be smart about when to lookup symbol in other projects(as loading projects in most common scenarios is costly and can result in delay when someone does first time find all references)

Personally, I will happily trade off performance for "Find all references" and "Rename symbol" that actually works.
In https://github.com/strongloop/loopback-next, we have configured TypeScript to treat the entire monorepo as a single project, so that we can use "Find all references" and "Rename symbol".
We also don't use declaration maps, therefore cross-project imports are resolved against the original TypeScript sources, not via .d.ts and .d.ts.map files.
This setup works pretty well for us and I am not aware of any performance or memory limitation when working in VS Code. If the performance of #28261 is similar, then I would happily accept it.
Why we want to migrate from our single-project setup to project references when VS Code experience is good now? The problem is that our build times are super slow. We are building each monorepo package independently, because each package has its own outDir, e.g. packages/{name}/dist. Because cross-project imports are resolved against the original TypeScript sources, I suspect that the compiler is repeatedly parsing each TypeScript source file again and again for each top-level package depending on it.
Anyhow. Is there a way how to finish #28261 sooner, possibly without performance optimizations and perhaps behind a new feature flag?

I currently have a code base that uses composite projects and has 34 sub-projects.
Having to open all 34 sub-projects to "Find all references" is a little tiring =/

Hi, any updates on this? We've recently migrated a very large frontend codebase to sub-projects, and opening files from each before finding references isn't feasible.

I have a similar problem in a case implementing scenario described in the project references documentation.
There is a root tsconfig.json, referencing packages which are a set of composite projects (implementation and tests). In case there is no tsconfig.json with non-empty files in a package, VS Code won't pick up the config for the files in the package.
https://github.com/pablobirukov/vs-core-project-references

For what it's worth, in order to get this to work, we had to add a "solution" tsconfig.json at the root of our repo that references all projects, as recommended in the Typescript handbook:

Another good practice is to have a “solution” tsconfig.json file that simply has references to all of your leaf-node projects and sets files to an empty array (otherwise the solution file will cause double compilation of files). Note that starting with 3.0, it is no longer an error to have an empty files array if you have at least one reference in a tsconfig.json file.

We were able to automate generating this list using meta-updater.  See our meta-updater config for more details

I did all, i created a root tsconfig with references, I set tsconifg in all project with composite: true  and still I need to open project to use find references.
I found a hack, I added "files": ["./project1/tsconfig.json", "./project2/tsconfig.json"] to root tsconfig and it seems it force vscode to load all tsconfig and its working very well.


Support for accessing camera!

To support QR code scanning it would be great if we could access the camera. I am not sure if this is a Clay limiation or a Pebble limitation on the configuration pages. #feature

Sounds like a job for a custom component. You can do it with the following now though.
config item:
{
  "type": "input",
  "id": "photo",
  "label": "Tap here to choose a photo of the QR code",
  "attributes": {
    "type": "file"
  }
}

In the AFTER_BUILD handler your custom function:
clayConfig.getItemById('photo').on('change', function(event) {
  // Get a reference to the taken picture or chosen file
  var files = event.target.files;
  var file;
  if (files && files.length > 0) {
    file = files[0];
  }

  // do something with `file`
});

taken from: https://developer.mozilla.org/en-US/docs/Mozilla/B2G_OS/API/Camera_API/Introduction
I haven't actually tried this. It is very possible that the webviews are sandboxed to not allow file input boxes. If thats the case then I do not think there is much you can do without changes in the mobile apps.


ERROR: Could not find a version that satisfies the requirement tf_nightly==2.9.0.dev20220401 (from scenic) (from versions: none)

When I tried to install scenic
I run  pip install . and get this error.
ERROR: Could not find a version that satisfies the requirement tf-nightly (from versions: none)
ERROR: No matching distribution found for tf-nightly
when I tried to run !pip install tf-nightly
here is the result:
!pip install tf-nightly
pip install . install tf-nightly
Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple, https://pypi.ngc.nvidia.com
Processing /misc/export3/tansihan/scenic
Preparing metadata (setup.py) ... done
Collecting install
Downloading https://pypi.tuna.tsinghua.edu.cn/packages/4d/c8/8cbca135f9e167810756ea2bc34b028501936675fcbd7dadccf752fa4622/install-1.3.5-py3-none-any.whl (3.2 kB)
ERROR: Could not find a version that satisfies the requirement tf-nightly (from versions: none)
ERROR: No matching distribution found for tf-nightly
My python version is 3.9.0
I wonder why this happens and how to solve it.

pip install . -i http://pypi.douban.com/simple/ --trusted-host pypi.douban.com
I run it in this way, seems successfully

Seconding that @Claire874 's solution worked for me -- that said, it would be nice to get this dependency fixed.

Should be fixed by f84823a.


Create support model for the DDS

Jeff-Chew created the following on Aug 01:
User Story

As a [user role below]:
DDS support team

I need to:
effectively triage all incoming support tickets

so that I can:
best manage the team's time and allocation on support

Additional Information

DDS Support process
Northstar Support activities

Original issue: https://github.ibm.com/webstandards/digital-design/issues/1337

On Aug 01, Jeff-Chew commented:
FYI, I created the label "to be triaged", which I think we should automatically add to a bug when it comes in. We can filter by this label during triage meetings then remove once we have triaged it.

On Aug 01, linda-carotenuto commented:
Is this v18 only?

On Aug 01, Jeff-Chew commented:

Is this v18 only?

I think this model can serve for both honestly, but we can discuss.


Add option to configure metadata table name

Hi,
I've implemented option to set custom metadata table name. Let me know if you would like to accept this change or if there is any issue to be addressed beforehand.

Perfect, this is exactly what I had in mind. Thank you for the contribution :)
I've pushed schemamama_postgres 0.2.3 to crates.io.


Implement working Task that deploys to k8s using helm

Expected Behavior
Once we have implemented a simple hellworld Task (#59) a user should be able to define a Task that can deploy at a k8s resource using helm.
This would mean that:

Templating is in place that allows for the Task to use the image and registry values specified in the input image resource in the steps of the Task as arguments to the helm commands
Any required service account needed for accessing to the cluster is available to the Task
Any logic needed to support the cluster type (see #60) is implemented

To try this out manually, you'll need a k8s cluster has been setup that you can deploy to.
Should include:

Expanding the docs about how to write a Task that uses an k8s cluster resource, and how to set up the service account correctly
Add docs on how to debug this when it doesn't work
Test coverage, unit and also integration if possible (see #16)

Actual Behavior
We have no logic currently for even running a task, but #59 will add basic logic. #62 will expand on this but shouldn't block this task.
Steps to Reproduce the Problem

Deploy the Pipeline CRD + related CRDs to your cluster (note: should be docs on how to do this post kubebuilder)
Create a Resource like this (you can use a public image such as the kritis one in this example or you can use any other image):

apiVersion: pipeline.knative.dev/v1beta1
kind: Resource
metadata:
  name: helm-resources
  namespace: default
spec:
  resources:
    - name: kritis-app-github-resource
      type: github
      params:
      - name: url
        value: https://github.com/grafeas/kritis
      - name: revision
        value: master
    - name: kritis-server
      type: image
      params:
        - name: url
          value: gcr.io/kritis-project

Create a Task like this:

apiVersion: pipeline.knative.dev/v1beta1
kind: Task
metadata:
  name: helm
  namespace: default
spec:
    inputs:
        resources:
            - name: workspace
              type: github
            - name: imageToDeploy
              type: image
        params:
            - name: pathToHelmCharts
              value: string
            - name: helmArgs
              value: string
        cluster:
            - name: clusterToDeployTo
    buildSpec:
        steps:
        - name: deploy
          image: kubernetes-helm
          args: ['deploy', '--path=${pathToHelmChart}', '--set image=${imageToDeploy.Url}.${imageToDeploy.Name}', '${helmArgs}', '--cluster=${clusterToDeployTo.Endpoint}']

(Note I am pretty much making up the helm command here, please replace it with something that actually works and makes sense!)

Create a TaskRun like this:

apiVersion: pipeline.knative.dev/v1beta1
kind: TaskRun
metadata:
  name: helm-run-0
spec:
    taskRef:
        name: helm
    resourceRef:
       name: helm-resources
    trigger:
        triggerRef:
            type: manual

Look at the logs from the running container (Note the best docs we have on this at the moment are some of the steps in the Build quick start) to see that the helm command was successful (or not successful)
Observe that the status of the TaskRun has been updated appropriately, e.g. see this example which contains:

Outcomes of individual steps
Started, Completed, and Successful conditions

Look at the cluster itself and verify that the image has been deployed to it

Additional Info

Blocked by #59 (helloworld task), #58 (GitHub resource).
Whether or not you want the cluster to actually be managed by helm (e.g. as in this tutorial) or just use helm to deploy is optional.
Kritis helm charts, docs are here

Should we consider this as a resource since it is the output of a Task?

Should we consider this as a resource since it is the output of a Task?

By "this" do you mean the cluster than gets deployed to? Definitely!
Or do you mean the deployment itself? b/c that's an interesting idea that I don't think we'd thought of!

I meant the deployment of the chart, a helm resource that knows how to deploy similar to how the image resource knows how to upload the image to a registry

I meant the deployment of the chart, a helm resource that knows how to deploy similar to how the image resource knows how to upload the image to a registry

Oooo neat! I think that's a cool idea, and makes a lot of sense since deployments will probably be a big part of what I imagine folks would want to use Pipelines for. Would you be interested in throwing together an issue to look into designing a Deployment resource?

Sure, I'll write up an issue for it

/assign @pivotal-nader-ziada

If it helps, Helm's founder contributed and maintains a container image that runs helm, with GKE integration, here.

/close
addressed by PR: #160
Will address the cluster selection by another issue and pr

Okay I created #177 to add support for clusters, closing this issue.


无法在 Openeuler 24.03 LTS 上安装使用

Description
尽管显示成功安装：
[jyu@localhost ddns-go_6.6.8]$ sudo ./ddns-go -s install
2024/08/19 10:15:14 Install ddns-go service successfully! Please open the browser and configure it

但是局域网内连接电脑的9876端口显示连接被拒绝。
检查日志信息，似乎服务有故障。
[jyu@localhost ddns-go_6.6.8]$ service ddns-go status
Redirecting to /bin/systemctl status ddns-go.service
● ddns-go.service - Simple and easy to use DDNS. Automatically update domain name resolution to public IP (Support Aliy>
     Loaded: loaded (/etc/systemd/system/ddns-go.service; enabled; preset: disabled)
     Active: activating (auto-restart) (Result: exit-code) since Mon 2024-08-19 10:15:14 CST; 1min 24s ago
    Process: 9745 ExecStart=/home/jyu/sources/ddns-go_6.6.8/ddns-go -l :9876 -f 300 -cacheTimes 5 -c /root/.ddns_go_con>
   Main PID: 9745 (code=exited, status=203/EXEC)

[jyu@localhost ddns-go_6.6.8]$ service ddns-go status
Redirecting to /bin/systemctl status ddns-go.service
● ddns-go.service - Simple and easy to use DDNS. Automatically update domain name resolution to public IP (Support Aliy>
     Loaded: loaded (/etc/systemd/system/ddns-go.service; enabled; preset: disabled)
     Active: activating (auto-restart) (Result: exit-code) since Mon 2024-08-19 10:15:14 CST; 1min 33s ago
    Process: 9745 ExecStart=/home/jyu/sources/ddns-go_6.6.8/ddns-go -l :9876 -f 300 -cacheTimes 5 -c /root/.ddns_go_con>
   Main PID: 9745 (code=exited, status=203/EXEC)

系统日志如下：
Aug 19 10:09:26 localhost dbus-daemon[1097]: [system] Activating via systemd: service name='org.freedesktop.hostname1' unit='dbus-org.freedesktop.hostname1.service' requested by ':1.13' (uid=1000 pid=9356 comm="/usr/bin/hostnamectl --transient" label="unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023")
Aug 19 10:09:26 localhost systemd[1]: Starting Hostname Service...
Aug 19 10:09:26 localhost dbus-daemon[1097]: [system] Successfully activated service 'org.freedesktop.hostname1'
Aug 19 10:09:26 localhost systemd[1]: Started Hostname Service.
Aug 19 10:09:26 localhost systemd[1]: libvirtd.service: Deactivated successfully.
Aug 19 10:09:26 localhost systemd[1]: libvirtd.service: Unit process 2925 (dnsmasq) remains running after unit stopped.
Aug 19 10:09:26 localhost systemd[1]: libvirtd.service: Unit process 2926 (dnsmasq) remains running after unit stopped.
Aug 19 10:09:56 localhost systemd[1]: systemd-hostnamed.service: Deactivated successfully.
Aug 19 10:11:17 localhost systemctl[9486]: [systemctl enable ddns-go.service] called by PID 9478 (./ddns-go -s install)
Aug 19 10:11:17 localhost systemd[1]: Reloading requested from client PID 9486 ('systemctl') (unit sshd.service)...
Aug 19 10:11:17 localhost systemd[1]: Reloading...
Aug 19 10:11:17 localhost systemd-rc-local-generator[9514]: /etc/rc.d/rc.local is not marked executable, skipping.
Aug 19 10:11:17 localhost systemd[1]: /usr/lib/systemd/system/dbus.socket:5: ListenStream= references a path below legacy directory /var/run/, updating /var/run/dbus/system_bus_socket → /run/dbus/system_bus_socket; please update the unit file accordingly.
Aug 19 10:11:17 localhost systemd[1]: Reloading finished in 311 ms.
Aug 19 10:11:17 localhost systemctl[9520]: [systemctl daemon-reload] called by PID 9478 (./ddns-go -s install)
Aug 19 10:11:17 localhost systemd[1]: Reloading requested from client PID 9520 ('systemctl') (unit sshd.service)...
Aug 19 10:11:17 localhost systemd[1]: Reloading...
Aug 19 10:11:17 localhost systemd-rc-local-generator[9549]: /etc/rc.d/rc.local is not marked executable, skipping.
Aug 19 10:11:17 localhost systemd[1]: /usr/lib/systemd/system/dbus.socket:5: ListenStream= references a path below legacy directory /var/run/, updating /var/run/dbus/system_bus_socket → /run/dbus/system_bus_socket; please update the unit file accordingly.
Aug 19 10:11:17 localhost systemd[1]: Reloading finished in 304 ms.
Aug 19 10:11:17 localhost systemctl[9554]: [systemctl start ddns-go.service] called by PID 9478 (./ddns-go -s install)
Aug 19 10:11:17 localhost systemd[1]: Started Simple and easy to use DDNS. Automatically update domain name resolution to public IP (Support Aliyun, Tencent Cloud, Dnspod, Cloudflare, Callback, Huawei Cloud, Baidu Cloud, Porkbun, GoDaddy...).
Aug 19 10:11:17 localhost systemd[1]: ddns-go.service: Main process exited, code=exited, status=203/EXEC
Aug 19 10:11:17 localhost systemd[1]: ddns-go.service: Failed with result 'exit-code'.
Aug 19 10:13:18 localhost systemd[1]: ddns-go.service: Scheduled restart job, restart counter is at 1.
Aug 19 10:13:18 localhost systemd[1]: Started Simple and easy to use DDNS. Automatically update domain name resolution to public IP (Support Aliyun, Tencent Cloud, Dnspod, Cloudflare, Callback, Huawei Cloud, Baidu Cloud, Porkbun, GoDaddy...).
Aug 19 10:13:18 localhost systemd[1]: ddns-go.service: Main process exited, code=exited, status=203/EXEC
Aug 19 10:13:18 localhost systemd[1]: ddns-go.service: Failed with result 'exit-code'.

DNS Provider
华为云
Did you search for similar issues before submitting this one?
No, I didn't
Operating System
Linux
Architecture
x86_64
Version
v6.6.8，v6.6.7
How are you running ddns-go?
Service
Any other information
Openeuler 24.03 LTS

指定配置文件路径试下

我没有使用配置文件，请问这文件路径应该填什么？

参考首页， 指定到/home/yourname
现在看到默认到/root下了

还是不可以
[jyu@localhost ddns-go_6.6.8]$ systemctl status ddns-go.service
● ddns-go.service - Simple and easy to use DDNS. Automatically update domain name resolution to public IP (Support Aliy>
     Loaded: loaded (/etc/systemd/system/ddns-go.service; enabled; preset: disabled)
     Active: activating (auto-restart) (Result: exit-code) since Mon 2024-08-19 12:29:17 CST; 7s ago
    Process: 14048 ExecStart=/home/jyu/sources/ddns-go_6.6.8/ddns-go -l :9876 -f 300 -cacheTimes 5 -c /home/jyu/.ddns_g>
   Main PID: 14048 (code=exited, status=203/EXEC)

问题似乎在 Openeuler 默认开启 SElinux上，把他关闭就可以安装成功了。
# selinux on
[jyu@localhost ddns-go_6.6.8]$ sudo setenforce 1
[jyu@localhost ddns-go_6.6.8]$ sudo ./ddns-go -s install -c /home/jyu/.ddns_go_config.yaml
2024/08/19 12:31:44 Install ddns-go service successfully! Please open the browser and configure it
[jyu@localhost ddns-go_6.6.8]$ systemctl status ddns-go.service
● ddns-go.service - Simple and easy to use DDNS. Automatically update domain name resolution to public IP (Support Aliy>
     Loaded: loaded (/etc/systemd/system/ddns-go.service; enabled; preset: disabled)
     Active: activating (auto-restart) (Result: exit-code) since Mon 2024-08-19 12:31:44 CST; 4s ago
    Process: 14368 ExecStart=/home/jyu/sources/ddns-go_6.6.8/ddns-go -l :9876 -f 300 -cacheTimes 5 -c /home/jyu/.ddns_g>
   Main PID: 14368 (code=exited, status=203/EXEC)
[jyu@localhost ddns-go_6.6.8]$ sudo ./ddns-go -s uninstall
2024/08/19 12:32:02 ddns-go service uninstalled successfully
# selinux off
[jyu@localhost ddns-go_6.6.8]$ sudo setenforce 0
[jyu@localhost ddns-go_6.6.8]$ sudo ./ddns-go -s install -c /home/jyu/.ddns_go_config.yaml
2024/08/19 12:32:23 Install ddns-go service successfully! Please open the browser and configure it
[jyu@localhost ddns-go_6.6.8]$ systemctl status ddns-go.service
● ddns-go.service - Simple and easy to use DDNS. Automatically update domain name resolution to public IP (Support Aliy>
     Loaded: loaded (/etc/systemd/system/ddns-go.service; enabled; preset: disabled)
     Active: active (running) since Mon 2024-08-19 12:32:23 CST; 23s ago
   Main PID: 14505 (ddns-go)
      Tasks: 11 (limit: 98790)
     Memory: 5.9M ()
     CGroup: /system.slice/ddns-go.service
             └─14505 /home/jyu/sources/ddns-go_6.6.8/ddns-go -l :9876 -f 300 -cacheTimes 5 -c /home/jyu/.ddns_go_config>

8月 19 12:32:23 localhost.localdomain systemd[1]: Started Simple and easy to use DDNS. Automatically update domain name>
8月 19 12:32:23 localhost.localdomain ddns-go[14505]: 2024/08/19 12:32:23 Listen on :9876
8月 19 12:32:23 localhost.localdomain ddns-go[14505]: Success to open the browser, please configure in the web page

现在 webui 也能打开了。我再配置看看有没有其他问题。

已经配置成功了。感谢你们的这个项目，真的帮了很大的忙 ^__^


Feature request: package filter searchbox

When using multiple packages it becomes harder to find the needed package in the list. Having some simple filter box with fuzzy searching would be great.

And If you want to use the filter, you can press the button in the blue box.

You can press the vertical scroll bar here to enter focus mode.
And then you can enter your keywords directly.

Up to you :).
Still can't do that even when pressing that sidebar, probably there is some conflicting option on my side.

The focused feature is done, you can enter ctrl+shift+' or press the search button to focus gomod.

First, this looking glass icon is now in File Editor too and clicking on it switches to gomod-explorer. I think this should be removed.

Second, in go-mod-explorer clicking it doesn't make possible to search either, but I've started a fresh instance of VSCode and confirmed that this is my environment problem, fresh instance works correctly. I'm going to look up what's up with my setup.

yea, i forgot to add a restriction for other explorer, try the new version.
maybe, I try it on linux, it works.


"Attempted an unsupported operation" when trying to open Settings

Everytime I try to open Settings or About Gluxle, an alert pops up saying "Attempted an unsupported operation", then the software closes. I can play games, and open Help, About this game and Scroll Buffer without problems.

The problem is that the translation DLLs didn't get rebuilt properly. I will do a new build shortly that will fix this. In the meantime, I've attached an updated Spanish translation DLL: if you unzip it and overwrite the existing one with this, all the dialogs should work for you.
GlkEspañol.zip

It works, thank you so much! Question, one of the reasons I wanted to get to Settings was so I could change the language to English, but I'm not seeing an option. Is it possible? Either way, everything works properly now, so thank you again! :)

You can't change the language from Settings, it is just picked up from the Windows settings. But if you want English, remove the translation DLLs (GlkEspañol.dll etc.). Without those DLLs present the UI will just be in English.

Perfect, thank you!

A new build of Windows Glulxe, 0.60.153, has been released that fixes this problem, see https://github.com/DavidKinder/Windows-Glulxe/releases/tag/<IP_ADDRESS>


[deploy] Do not fail Maven Central Rules sources or javadocs jar missing for an empty jar.

JReleaser should do the following:

inspect the JAR looking for *.class entries. If found then -sources and -javadoc is required.
If not found then look for other entries (besides META-INF/MANIFEST.MF and META-INF/maven/*). If found then -sources is required.

Long discussion: https://github.com/jreleaser/jreleaser/discussions/1287

Workaround:
              <maven>
                <nexus2>
                  <maven-central>
                    <applyMavenCentralRules>false</applyMavenCentralRules>
                    ...

Please reopen this issue.
This issue isn't fixed entirely. While the checksum doesn't fail, it now fails later with 1.6.0-SNAPSHOT:
org.jreleaser.model.JReleaserException: Rules for publishing to Maven Central were not met
	at org.jreleaser.sdk.commons.AbstractMavenDeployer.collectDeployables(AbstractMavenDeployer.java:148)

[ERROR]     [nexus2] timefold-solver-core-0.8.38-sources.jar is missing
[ERROR]     [nexus2] timefold-solver-core-0.8.38-javadoc.jar is missing
[ERROR]     [nexus2] timefold-solver-spring-boot-starter-0.8.38-sources.jar is missing
[ERROR]     [nexus2] timefold-solver-spring-boot-starter-0.8.38-javadoc.jar is missing
[ERROR]     [nexus2] timefold-solver-ide-config-0.8.38-javadoc.jar is missing

Reproducer (after running set version to 0.8.38 instead of SNAPSHOT on it): https://github.com/TimefoldAI/timefold-solver/tree/0.8.x

I don't want to spam this thread, so I opened a separate dicussion thread to figure out where the issue lies: https://github.com/jreleaser/jreleaser/discussions/1355

🎉 This issue has been resolved in v1.6.0 (Release Notes)


Cannot start job on workers in some situations

Reproduction Steps

Manager A started
Worker A started and registered with Manager A
Worker B and Manager B started, killing Worker A and Manager A
Worker B registered with the old Manager A, but db was not updated somehow
Manager B picked up the old db and don't know the ip of Worker B

Error Log

Environment

CDK CLI Version:
Framework Version:
Node.js Version:
OS               :

Other

This is :bug: Bug Report

I can reproduce in on Cloud easily, but not locally.


Missing <Match_type, Match_key> in Action function call.

In addition, please fix the default function ptr order.

Fixed. Please check.


feature request can we have outline or toc of blog

whether in the top of blog content or left side

Hi @wanghaisheng! You can use the insights template as your blog, which already has a toc on the left side, in addition you can move it to the right or top, and add additional components from the blog page.

can you share a url of insights

Here it is: https://screwfast.uk/insights/en/insight-1/

i  manage to find this under same level of blog ,and url seems weired ,does this should be /en/xxxx/xxx
and if you go to blog in fr and click any blog ,you will found url  is https://screwfast.uk/fr/blog/fr/post-2/
fr will be duplicate

That's correct, the default language in this case is English, so we don't prefix it, however we prefix French. And then each blog or insights content collections also have their own language prefixes, so it's blog/en and blog/fr.