STM32 (Nucleo L476RG) SPI pins floating

Not sure whether that's a bug or simply me holding it wrong, but since there's so little documentation available on this I cannot figure out what's going on. I'm using the following code to test:
static const struct spi_config spi_cfg = {
    .frequency = 4000000,
    .operation = SPI_OP_MODE_MASTER | SPI_WORD_SET(8) | SPI_TRANSFER_MSB | SPI_MODE_CPOL,
    // .slave = 0,
};

struct device * spi_dev;

static bool spi_init(void)
{
    spi_dev = device_get_binding(DT_SPI_1_NAME);

    if (spi_dev == NULL) {
        printk("Could not get %s device\n", DT_SPI_1_NAME);
        k_sleep(2000);
        return false;
    }

    return true;
}

uint8_t spi_test_send(uint8_t cmd)
{
    int err;
    static u8_t tx_buffer[1];
    static u8_t rx_buffer[1];

    tx_buffer[0] = cmd;

    const struct spi_buf tx_buf = {
        .buf = tx_buffer,
        .len = sizeof(tx_buffer)
    };
    const struct spi_buf_set tx = {
        .buffers = &tx_buf,
        .count = 1
    };

    struct spi_buf rx_buf = {
        .buf = rx_buffer,
        .len = sizeof(rx_buffer),
    };

    const struct spi_buf_set rx = {
        .buffers = &rx_buf,
        .count = 1
    };

    err = spi_transceive(spi_dev, &spi_cfg, &tx, &rx);
    if (err) {
        printk("SPI error: %d\n", err);
    } else {
        printk("TX sent: %x\n", tx_buffer[0]);
        printk("RX recv: %x\n", rx_buffer[0]);
        return rx_buffer[0];
        // tx_buffer[0]++;
    }
}

uint8_t myData[] = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00};

int main(void)
{
    bool spiInitialized = false;

    while(!spiInitialized) {
        spiInitialized = spi_init();
    }

    spi_test_send(1);

    for(int i=0; i<10; i++) {
        k_sleep(15);
        myData[i] = spi_test_send(0);
    }

     k_sleep(10000);
}

My prj.conf looks like this:
CONFIG_SERIAL=y

CONFIG_CPLUSPLUS=y
CONFIG_PRINTK=y
CONFIG_STDOUT_CONSOLE=n
CONFIG_I2C=y
CONFIG_SENSOR=y
CONFIG_MULTITHREADING=y
CONFIG_GPIO=y

CONFIG_PINMUX=y
CONFIG_SPI=y
CONFIG_SPI_1=y

After deploying this on the L476RG, I'm measuring the voltage on CN8/A2 (-> PA_4), as this is documented to be the default CS pin used by Zephyr. The pin is floating at ~1.8V instead of being at 0V and 3.3V.
I've made a test using STM32-Arduino and deploying the following sketch on my board:
void setup() {
  Serial.begin(115200);
  pinMode(A2, OUTPUT);
}

void loop() {
  digitalWrite(A2, LOW);
  delay(5000);
  digitalWrite(A2, HIGH);
  delay(5000);
}

Here I could successfully measure the same pin alternating between 0V and 3.3V.
Is there something missing in order to get this working correctly or is it simply a bug?

OS: Gentoo Linux
Toolchain: Zephyr SDK, latest release

Hello,
not sure what your SPI configuration, but actually the PA4 is SPI1_NSS pin when Alternate function (AF5) is choosen.
Did you check the refMan of th STM32L476  about the "42.4.5 Slave select (NSS) pin management" to configure the SPI bus and

@mrusme, did you manage to solve your issue?
I had a look to general NSS pin set up and indeed, there might be an issue with _NSS pins definitions. Changing definition will make it alternate.
#define STM32L4X_PINMUX_FUNC_PA4_SPI1_NSS                                      \
	(STM32_PINMUX_ALT_FUNC_5 | STM32_PUSHPULL_PULLUP)

I'll check the reason why it was initially set to STM32_PUSHPULL_NOPULL before raising an issue. I suspect NSS pins (output enable) are not often used, explaining why this was not seen earlier (either CS pin or software mgmt would be preferred).
Then please note that toggling NSS pin is not equivalent to Arduino GPIO toggling.
NSS toggling is done by SPI IP while Arduino GPIO toggling is done by GPIO controller. Then same could be done in Zephyr like in blinky sample.

@erwango thanks for getting back. To be honest, I gave up on Zephyr after encountering more issues down the road which I couldn't figure out myself within a reasonable amount of time. Tried getting help on the IRC channel, which also didn't lead to much success back then.
Since I diverted to another stack this has become irrelevant to me, hence I can close this issue.

@mrusme, sorry to hear that. Btw, IRC is not much in use these days since we switched to slack some months ago(https://zephyrproject.slack.com/).

@erwango unfortunately that's of not much use for anyone else but you guys:

Pointed this out over a month ago to your somewhat active Twitter account, haven't heard back about it, which gave me the impression this channel is not for the community but rather only the companies working directly with you.

@mrusme, well that's strange, for sure there are people from the open community in there. So not only people from companies. Wonder why access is/looks blocked.

@mrusme :
Actually you need to access via the Slack invite link first : https://tinyurl.com/y5glwylp.
Maybe this need to be documented better indeed.

@erwango maybe change the link on the Zephyr contact page as it still is invalid? The one you just wrote seems to be valid, without actually going through the whole steps of registration.

@mrusme, this would make sense. Will submit a change.

Reopenning the issue, as it reveals a real issue that should be fixed.


 Just have a look at our pool's block list: https://poolmining.org/pool/dash. The possibility of this happening naturally is astronomically low.

@UdjinM6 @codablock @schinzelh I don't believe this is an issue anymore, correct? Are we able to close this?

Hasn't happened for a quite a while.


3.2 On Hackage

Is there a reason you are keeping version 3 off hackage. I would love to not have to pull it in with git!

Uploaded.


add HTML Elements (from line 67 to  114)

I have added description regarding commonly used HTML elements .

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Looks good to me, and congratulations on your first Pull Request!  🎉🎉🎉
We look forward to future contributions!


fix: pass support email as var to dashboard AB#11841

@arsforza I strongly think that this was the issue.


Adopt 'src/' layout

Fixes #7.
If the ansys/ directory is at the top-level, the protoc compilation will fail if there is already a build/ directory containing .proto files. This can be avoided by adopting a src/ layout instead.
In the ansys-tools-protoc-helper, there isn't really a great way to avoid this, since the directory above ansys/ needs to be passed to protoc.

Ping @akaszynski for review.


feat: Implement subset of crypto.subtle.importKey which can import a JSONWebKey using RSASSA-PKCS1-v1_5

@JakeChampion Thanks for landing this PR in 1.7.0.
I was hoping this would solve our problem (3rd party dependencies relying on subtle.crypto e.g. for signing AWS requests. However, we're now seeing different errors, such as Supplied algorithm is not yet supported.
Here's a code fragment from https://github.com/mhart/aws4fetch which seems to trigger this error:
async function hmac(key2, string3) {
  const cryptoKey = await crypto.subtle.importKey(
    "raw",
    typeof key2 === "string" ? encoder2.encode(key2) : key2,
    { name: "HMAC", hash: { name: "SHA-256" } },
    false,
    ["sign"]
  );
  return crypto.subtle.sign("HMAC", cryptoKey, encoder2.encode(string3));
}

It seems like the lack of HMAC support is causing the error.
Would this PR solve our problem? https://github.com/fastly/js-compute-runtime/pull/440
Can you give us an ETA when this PR lands in production?

@stefan-guggisberg - #440 does solve that issue 👍  - I am breaking that PR up into smaller more manageable PR for the team to review. If you subscribe to #440, when it is closed you should be able to run the code you've posted without any issues 👍


32 describe consumergroup show topic

fixes https://github.com/real-digital/esque/issues/32
setting consumer-id argument to required seemed sensible, tell me if there's a use case i didn't see

Oh, just noticed the tests. Will update my PR later.

You still have merge conflict. :(

Could it be that this is actually outdated by now? I think #53 will improve the describe consumergroup command already.

Fixed in #53


Vue strips methods from sub classes of Arrays.

Version
2.6.10
Reproduction link
https://jsfiddle.net/mpqx6y8r/2/
Steps to reproduce
Run the JSFiddle with the browser console open.
A created hook on the Vue component will attempt to log to the console the return value of this.Pet.getSpecies() and this.WaitingList.frontPerson().
However, Vue will fail the second call since this array method (class ExtendArray extends Array) has been stripped during data registration. The same does not happen with plain Objects (class ExtendObject extends Object).
What is expected?
Console output:
'Pet: Dog',
'WaitingList: Jack'
What is actually happening?
Console output:
'Pet: Dog',
'TypeError: this.WaitingList.frontPerson is not a function'

Duplicate of https://github.com/vuejs/vue/issues/9259


CesiumGeoreferenceComponent "Snap to East South Up" also resets scale

Drop a Cone in the level somewhere.
Set its Scale too (100,100,100)
Add a CesiumGeoreferenceComponent to it.
Click the "Snap to East South Up" button on the component.

The cone will shrink back down to a (1,1,1) scale.

Ok I'm closing this because it sounds like it's by design.


Voters info popover

Add popover to vote numbers that shows top 10 voter when mouse is hovering over it and add more... text that will open current Voters info page.
Example:

Implemented @feruzm


[libdom] Allow Response / Request to be generics

Similar to the FormData request by @wesbos in https://github.com/microsoft/TypeScript/issues/43797, it would be great if the Request and Response globals could also accept generic type parameters.
This will become more relevant as more frameworks continue to increase their usage of these standard globals, also in Node.js.
Consider the following example of an App Route handler in Next.js (handling a POST request to an API route):
export async function POST(request: Request): Response {
  const body = await request.json();
     // ^? const body: any ❌
  return new Response(JSON.stringify({
    abc: 123, // ❌ how can the POST function return type check this object?
  }))
}

What would be amazing would be something similar to this:
type RequestBody = { zzz: number };
type ResponseBody = { abc: number };

export async function POST(request: Request<RequestBody>): Response<ResponseBody> {
  const body = await request.json();
     // ^? const body: { zzz: number } ✅
  return new Response(JSON.stringify({
    abc: 123, // ✅ type checked
  }))
}

Workaround
Response
Wasn't able to find a nice workaround for typing the body of a Response yet.
Request
Using Type Assertion / casting via as:
type RequestBody = { zzz: string };

export async function POST(request: Request) {
  const body = await request.json() as RequestBody;

Caveats

Arguably, the Request is unknown until runtime, so maybe this should be handled instead with Zod or some other runtime validation

Oops, somehow reported this in the wrong repo, moved over here: https://github.com/microsoft/TypeScript/issues/52777


NotSubtype thrown while typechecking an ArrayStore opcode

Even without the --dump option, the benchmark below crashes for Jreduce
Here is the app+lib.jar file
https://drive.google.com/file/d/1ZJuoszSs6eDEsoimyyQMCMWBR4ppnqGO/view?usp=sharing
Here is the command used and the output

jreduce --work-folder _jreduce/workfolder --jre /usr/lib/jvm/java-8-openjdk-amd64/jre -S items+logic --stdlib ../../stdlib.bin -o app+lib.after.jar --timelimit 1800 app+lib.jar -- ls
17:37:16 ├ JReduce
17:37:16 │ ├ Started JReduce.
17:37:16 │ ├ Reading inputs
17:37:16 │ │ └ 0.002s
17:37:16 │ ├ Calculating Initial Problem
17:37:16 │ │ ├ setup
17:37:16 │ │ │ └ 0.050s
17:37:16 │ │ ├ run
17:37:16 │ │ │ └ 0.009s
17:37:16 │ │ └ 0.061s
17:37:16 │ ├ Refining Problem
17:37:16 │ │ └ 0.127s
17:37:16 │ ├ Initializing key function
17:37:16 │ │ ├ Calculating the hierarchy
17:37:16 │ │ │ ├ Load stdlib stubs
17:37:23 │ │ │ │ └ 6.516s
17:37:23 │ │ │ ├ Load project stubs
17:37:30 │ │ │ │ └ 7.024s
17:37:30 │ │ │ ├ Compute hierarchy
17:37:32 │ │ │ │ └ 2.242s
17:37:32 │ │ │ └ 15.783s
17:37:32 │ │ ├ Found 43862 items.
17:37:32 │ │ ├ Found 43844 facts.
17:37:33 │ │ ├ The core is 0 of them.
17:37:33 │ │ └ 16.359s
17:37:33 │ ├ Compute CNF
17:37:33 │ │ └ 0.289s
jreduce: "io/reactivex/internal/operators/observable/ObservableAmb.subscribeActual:(Lio/reactivex/Observer;)V" Just (ByteCodeInst {offset = 104, opcode = ArrayStore ARef}) NotSubtype (TRef ["Lio/reactivex/ObservableSource;"]) (TRef ["Lio/reactivex/Observable;","Lio/reactivex/ObservableSource;"])
CallStack (from HasCallStack):
error, called at src/JReduce/Logic.hs:971:31 in jreduce-0.1.0-Abs4ufFiDadAKpDliDdD1j:JReduce.Logic
CallStack (from -prof):
JReduce.Logic.logic.typeCheckStates (src/JReduce/Logic.hs:(969,7)-(977,29))
JReduce.Logic.logic (src/JReduce/Logic.hs:(624,1)-(1014,35))
JReduce.Logic.initializeKeyFunction (src/JReduce/Logic.hs:(277,1)-(373,69))
JReduce.Logic.computeCNF (src/JReduce/Logic.hs:(382,1)-(389,12))
JReduce.Logic.describeLogicProblem.\ (src/JReduce/Logic.hs:(481,91)-(556,5))
Control.Reduce.Problem.refineProblemA' (src/Control/Reduce/Problem.hs:(495,1)-(503,5))
JReduce.Logic.describeLogicProblem (src/JReduce/Logic.hs:(481,1)-(556,5))
JReduce.run.\ (src/JReduce.hs:(101,52)-(193,44))
Control.Reduce.Util.OptParse.withWorkFolder (src/Control/Reduce/Util/OptParse.hs:(107,1)-(116,41))
JReduce.run (src/JReduce.hs:(96,1)-(224,40))
Control.Reduce.Util.Logger.timeIO (src/Control/Reduce/Util/Logger.hs:(249,1)-(253,48))
Control.Reduce.Util.Logger.timedPhase'.runMa (src/Control/Reduce/Util/Logger.hs:163:7-84)
Control.Reduce.Util.Logger.timedPhase' (src/Control/Reduce/Util/Logger.hs:(161,1)-(181,11))
JReduce.main (src/JReduce.hs:(69,1)-(81,48))

The main error that you encounter is while type checking the method:
io/reactivex/internal/operators/observable/ObservableAmb.subscribeActual:(Lio/reactivex/Observer;)V
and at the opcode:
Just (ByteCodeInst {offset = 104, opcode = ArrayStore ARef})
the typechecker throw:
NotSubtype (TRef ["Lio/reactivex/ObservableSource;"]) (TRef ["Lio/reactivex/Observable;","Lio/reactivex/ObservableSource;"])
I think that the error code should be improved, to me more readable, but that will not fix the problem you are facing.
Inspecting the app+lib.jar, here ObservableSource is a super interface of Observable. Which means that correctly
that ObservableSource is not a subtype of Observable and ObservableSource.
It looks like the problem might be due to a typing problem, while storing in an array,  in Java: https://en.wikipedia.org/wiki/Covariance_and_contravariance_(computer_science)#Covariant_arrays_in_Java_and_C.23
And currently, Jreduce, sadly, does not support covariant arrays.
Inspecting the decompiled code, my hypothesis  seems to be correct, but if you have access to the source code, can you verify that the code in io/reactivex/internal/operators/observable/ObservableAmb.subscribeActual uses arrays covariance.

No there is no source code available. That class is from the library code.
Okay, I shall skip this benchmark in that case.


Debug identity test

Who is this for?
https://wellcome.slack.com/archives/C3TQSF63C/p1683627306387799
the ChangeEmail test has been failing intermittently for a few weeks, seems to have to do with it being wrapper in act as recommended, and therefore acting closer to a browser. I think it might also have to do with native validation stopping other alerts from showing.
What is it doing for them?
As the behaviour seems different locally than in a PR's tests, I want to see what logs.

The error message we should be looking for is "Enter an email address", not "Enter a valid email address", as seen in the code of the page.
That message never shows anyway as browsers' native validation catches it beforehand. I think it used to work because before being wrapped in act, it wasn't behaving as in a browser?
It's strange because when running locally, if you run only that test, it errors. If you run them all, it passes. This seems intermittent on Buildkite.
Apparently the clear action doesn't work consistently, I found a thread regarding it and it says doing a click on the input fixes it, which it does seem to do.
I'm not sure this is what should be tested or if the code should be refactored. I'm thinking we're happy with native validation and don't want to get rid of it just so we can test this use case?

https://github.com/wellcomecollection/wellcomecollection.org/assets/110461050/07fcdc89-f069-45a1-b1ae-58a220649989

Closed for #9775


Bad normals when exporting

Game
Resident Evil 5
Example game file names
Any
Blender version
2.78
Albam version
0.2.0
Observed behavior
All normals are having the same value, making the models look weird in any lighting condition
Expected behavior
Normals are exported fine.
Steps to reproduce
Import and export a model without touching it, see them in game or in the Figures section.

https://wiki.blender.org/index.php/User:Mont29/Foundation/Split_Vertex_Normals
Solving this issue doesn't seem straight-forward. There are many questions I have and so far I couldn't get the save per-vertex normal into blender nor calculate them to export them back. A quick test shows that exporting models with no normals is much better than incorrect normals.

After https://github.com/Brachi/albam/pull/30, normals are not being exported anymore.


[Tracking] Triage Meeting Statics

This issue serves as a tracker for the number of new issues each week and their basic type:
You can add a comment each week to update this.
The basic format should be:

Feature Requests: #
Enhancements: #
Bugs: #
RFC: #
Non-issue: #
Autocuts: #

6/17/2024
Feature Requests: 1
Enhancements: 3
Bugs: 3
RFC: 1
Non-issue: 1
Autocuts: 4


Support for all kinds of containers

Use a little bit of type_traits magic to make debugging of all container types possible.
For example, use std::enable_if to use the container-specialized overload if a begin, end (and size) function exists.

I recommand is_detected for that.
#include <type_traits>

// -- begin is_detected
//    https://en.cppreference.com/w/cpp/experimental/is_detected

struct nonesuch {
    ~nonesuch() = delete;
    nonesuch(nonesuch const&) = delete;
    void operator=(nonesuch const&) = delete;
};

namespace detail {
template <class Default, class AlwaysVoid,
          template<class...> class Op, class... Args>
struct detector {
  using value_t = std::false_type;
  using type = Default;
};

template <class Default, template<class...> class Op, class... Args>
struct detector<Default, std::void_t<Op<Args...>>, Op, Args...> {
  // Note that std::void_t is a C++17 feature
  using value_t = std::true_type;
  using type = Op<Args...>;
};

} // namespace detail

template <template<class...> class Op, class... Args>
using is_detected = typename detail::detector<nonesuch, void, Op, Args...>::value_t;

template <template<class...> class Op, class... Args>
using detected_t = typename detail::detector<nonesuch, void, Op, Args...>::type;

template <class Default, template<class...> class Op, class... Args>
using detected_or = detail::detector<Default, void, Op, Args...>;

// -- end is_detected

template <typename T>
using detect_begin_t = decltype(begin(std::declval<T>()));

template <typename T>
using detect_end_t = decltype(end(std::declval<T>()));

template <typename T>
using detect_size_t = decltype(std::declval<T>().size());

template <typename T>
struct has_begin_end_size {
    static constexpr bool value = is_detected<detect_begin_t, T>::value
                                  && is_detected<detect_end_t, T>::value
                                  && is_detected<detect_size_t, T>::value;
};

template <typename Container>
typename std::enable_if<has_begin_end_size<Container>::value, bool>::type
prettyPrint(std::ostream &stream, Container const &value) {
    // ...
}

Thank you very much!
I didn't know is_detected. That seems like a very useful "wrapper" around std::void_t.
Ideally, I would like to keep dbg.h C++11 compatible, so we would have to include an implementation of void_t as well.
(a technical thing: the content on cppreference.com seems to be CC-BY-SA licensed. I'm not really familiar with the details, but I don't think it's compatible with MIT)

The code on cppreference.com is just copied from the corresponding C++ Standard proposal, but the code itself is actually much older than that and has been floating around on the internet for years.  I don't even know who the original author is and would just assume that the code is public domain.
BTW, the CC-BY-SA licensing of cppreference.com is only a protection against someone just copying the whole wiki and filling it up with ads to make money from it.

Just to give you some reassurance, libstdc++ has the exact same code in the standard library, just with the variables renamed: https://github.com/gcc-mirror/gcc/blob/b05c7e439a823d99781e4aaa47a9782fa6693cb5/libstdc%2B%2B-v3/include/std/type_traits#L2582-L2598


fix(eds): remove txID field check from EVM ConfirmDeposit request

Description
Removing this check allows EDS to be compatible with v0.14.0 (required for current testnet version)
Todos

[ ] Unit tests
[x] Manual tests
[ ] Documentation
[x] Connect epics/issues
[x] Tag type of change
[ ] Upgrade handler

Steps to Test
Expected Behaviour
Other Notes

You're merging this on the v0.13 branch?

@jack0son I know we discussed this earlier. But we might have to rebuild a binary at some later point from this branch for people catching up and I don't like the idea that we are breaking this release. Could we create a releases/0.13.x* branch or something?

Sure but I don't think we are breaking this release. The field in question was not used in v0.13.x either - previously we were passing randomised data for this field.

but you are also removing the block height check which does break the release, no?

That change was made in v0.13.7 https://github.com/axelarnetwork/axelar-core/pull/1293
Unsure why it's showing up in this PR

no longer needed


Agent should synchronize existing resources with principal upon connect

The agent should synchronize all resources (apps, appprojects, etc) with the principal on the time it connects to ensure information on the principal is up-to-date as soon as possible.
Right now, the trigger is informer events on either side. While information will be eventually synchronized, it can take appResync time to do so.
When the agent connects in managed mode, it should request updates for all applications from the principal and submit back status information.
When the agent connects in autonomous mode, it should send updates for all applications to the principal.

Jayendra is available and taking a look at this one.


Screencast broken on docs

Describe the bug
A clear and concise description of what the bug is.
Readthedocs home page looks like this:

To Reproduce
go to: https://bqplot.readthedocs.io/en/latest/introduction.html
Expected behavior
A rad image convincing me to use bqplot shows up

Relatedly:
The top level link: https://bqplot.readthedocs.io/en/latest/
Leads to the table of with no explanatory text instead of a friendly intro paragraph. It would nice if that link led to the content of the Intro page


[Cdn] Use a singular name for an enumeration

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Cdn


Rename QGB to Blobstream in About

Problem
The About section of this repo still references QGB
Screenshot

Proposal
Someone with permission to edit the About should update it to say Blobstream.

Done

Great catch btw :+1: :+1:


QGIS Help pages not accessible

I have installed QGIS Desktop, but when I click Help to go to the documentation on the website I get a "file not found" error.
The URL it is trying to link to is:
https://docs.qgis.org/2.18/en/docs/user_manual/
This is a screenshot of the error:

@ableymcabley It should work now. Thanks for reporting.

@DelazJ wow you are fast :-)

I had this page opened when your confirmation of fix arrived :-)


Add snapshot tests for IdCompressor

Description
At the moment if you made a breaking change to IdCompressor's summary there aren't any tests to detect that happening. This adds a test suite that would fail if the summary format changed. The helpers to write to files are copy-pasted from tree2's similar suite rather than pulling them out to a shared package.
Reviewer Guidance
The review process is outlined on this wiki page.

Did you dry-run making a persisted change? If so, I'm happy with this.

Yeah I changed the ordering of some of the serialization and it correctly fails the tests.


please use docker compose v2 and stop being ignorant about EOL technologies

docker compose v1 is EOL since june 2023 https://docs.docker.com/compose/migrate/

mmh, are you sure this unpolite path is what you want to follow?

let's give you another chance

@u238 pinging you just because this looks like an atomic friction and after PR's title rename yeah, let's make it an atomic royal rumble

(which was public from its opening, so let's thunderdome: Two men enter, one man leave! :D

so an humble title like "use docker compose v2" does not work because reasons, let's hope for a more aggressive one like "please use docker compose v2 and stop being ignorant about EOL" because my feeling is a lot of people base their stability on this software stability


feat/fix: new project folder structure

This PR introduces a new project folder structure.
Due to the recent update, the main ui.lua file reaches 1998 lines of code, and the tendency is to increase more and more.
So, I decided to fragment ui.lua into a new folder with new files, at first for the actions and for the windows.
This was the structure of the project
├── LICENSE
├── lua
│   └── dooing
│       ├── calendar.lua
│       ├── config.lua
│       ├── init.lua
│       ├── state.lua
│       └── ui.lua
├── plugin
│   └── dooing.vim
└── README.md

The new project structure
├── LICENSE
├── lua
│   └── dooing
│       ├── config.lua
│       ├── init.lua
│       ├── state.lua
│       ├── ui
│       │   ├── actions
│       │   │   ├── io.lua
│       │   │   └── todo.lua
│       │   ├── render.lua
│       │   └── windows
│       │       ├── calendar.lua
│       │       ├── help.lua
│       │       ├── main.lua
│       │       ├── search.lua
│       │       └── tag.lua
│       └── ui.lua
├── plugin
│   └── dooing.vim
└── README.md

can you please review? @clementpoiret

Sure, I'll do that asap👌

seriously thinking about moving on from this, and instead of tearing everything apart, keeping things as they are. I don't have much time these days. And every release I'm gonna try to do small improvementes in quality of life updates, what you think?

That's also a good approach, but in that case you should define contribution guidelines, so that everyone's PR can help you gradually refactor things :)


Rethink penalty management

Is your feature request related to a problem? Please describe
We currently have two types of penalties, for load and time window violations. Some of the open issues and extensions plan to introduce more penalty types, which requires lots of new code in the current system.
Describe the solution you'd like
A way in which adding new penalty terms/situations is less code intensive. This should be easier than it is today.
Describe alternatives you have considered
None.
Additional context
None.

I don't think this is all that relevant anymore after a few refactorings, so I am closing the issue.


problem with loop index in a template ref

 Issue by sethladd
Originally opened as dart-lang/sdk#12664

From https://github.com/dart-lang/fancy-syntax/issues/19
I am very new to all this, but I think the following demonstrates a bug in fancy-sytax... or a bug in my understanding!
works
        <template id="article-div" bind attributes="obj">

{{ Content }}

Written By {{ Author }} -- (Id: {{ Id }})

        </template>
                <template id='article-page' bind syntax="fancy">
                    <template repeat="{{ allArticles }}">
                        <template ref="article-div" obj="{{}}" bind></template>
                    </template>
                </template>
doesn't work
                <template id='article-page' bind syntax="fancy">
                    <template repeat="{{ article in allArticles }}">
                        <template ref="article-div" obj="{{article}}" bind></template>
                    </template>
                </template>
This latter one, where the loop index is bound to "obj" in the call to the other template generates a backtrace in the console. It's at the bottom of this message.
I hope this helps. I need to do some work on something else right now and I have workaround so I'm sorry I've not sent a PR with this bug report.
ian
Uncaught Error: EvalException: variable not found: Content in 115065902 undefined:1
Stack Trace:
#0 Scope.
#1 IdentifierObserver._updateSelf (package:fancy_syntax/eval.dart:401:19)
#2 ExpressionObserver._observe (package:fancy_syntax/eval.dart:247:16)
#3 Updater.visitExpression (package:fancy_syntax/eval.dart:263:15)
#4 RecursiveVisitor.visitIdentifier (package:fancy_syntax/visitor.dart:59:51)
#5 IdentifierObserver.accept (package:fancy_syntax/eval.dart:415:41)
#6 Visitor.visit (package:fancy_syntax/visitor.dart:11:25)
#7 observe (package:fancy_syntax/eval.dart:56:27)
#8 _Binding._Binding (package:fancy_syntax/syntax.dart:71:24)
#9 FancySyntax.getBinding (package:fancy_syntax/syntax.dart:49:16)
#10 _bindOrDelegate (package:mdv/src/bindings.dart:162:46)
#11 _parseAndBind (package:mdv/src/bindings.dart:125:20)
#12 _addBindings (package:mdv/src/bindings.dart:96:18)
#13 _addBindings (package:mdv/src/bindings.dart:100:17)
#14 _addBindings (package:mdv/src/bindings.dart:100:17)
#15 _addBindings (package:mdv/src/bindings.dart:100:17)
#16 _TemplateExtension.createInstance (package:mdv/src/template.dart:62:17)
#17 Element.createInstance (file:///Volumes/data/b/build/slave/dartium-mac-full-trunk/build/src/out/Release/gen/webkit/bindings/dart/dart/html/Element.dart:289:59)
#18 _TemplateIterator.getInstanceNodes (package:mdv/src/bindings.dart:349:51)
#19 _TemplateIterator._handleChanges (package:mdv/src/bindings.dart:390:45)
#20 valueChanged (package:mdv/src/bindings.dart:290:19)
#21 _BufferingStreamSubscription._sendData. (dart:async/stream_impl.dart:184:55)
#22 _ZoneBase._runInZone (dart:async/zone.dart:82:17)
#23 _ZoneBase._runGuarded (dart:async/zone.dart:99:22)
#24 _ZoneBase.executePeriodicCallbackGuarded (dart:async/zone.dart:68:21)
#25 _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:184:41)
#26 _BufferingStreamSubscription._add (dart:async/stream_impl.dart:135:16)
#27 _SyncBroadcastStreamController._sendData. (dart:async/broadcast_stream_controller.dart:199:24)
#28 _BroadcastStreamController._forEachListener (dart:async/broadcast_stream_controller.dart:169:15)
#29 _SyncBroadcastStreamController._sendData (dart:async/broadcast_stream_controller.dart:198:21)
#30 _BroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:118:14)
#31 _deliverValues (package:observe/src/path_observer.dart:124:18)
#32 deliverChangeRecords (package:observe/observe.dart:227:14)
#33 _asyncRunCallback (dart:async/event_loop.dart:9:15)
#34 _createTimer. (dart:async-patch/timer_patch.dart:8:13)
#35 _Timer._Timer. (file:///Volumes/data/b/build/slave/dartium-mac-full-trunk/build/src/dart/tools/dom/src/native_DOMImplementation.dart:223:17)

 Comment by sigmundch

Added this to the M8 milestone.

 Comment by sigmundch

Removed this from the Later milestone.
Added this to the M9 milestone.

 Comment by sigmundch

Removed this from the 1.1 milestone.
Added this to the 1.2 milestone.

 Comment by jmesserly

Removed Area-Polymer label.
Added Area-Pkg, Pkg-PolymerExpressions labels.

 Comment by sigmundch

seems like it might be related to issue dart-lang/sdk#12742

Removed this from the M8 milestone.
Added this to the Later milestone.
Marked this as being blocked by dart-lang/sdk#12742.

 Comment by clayberg

Removed this from the M9 milestone.
Added this to the 1.1 milestone.

 Comment by kasperl

Removed this from the 1.2 milestone.
Added this to the 1.6 milestone.

 Comment by jmesserly

Removed Library-PolymerExpressions label.

 Comment by sigmundch

I believe this should be fixed now. Let us know if you still are running into issues.

Added Fixed label.


I use Detr to train my data set and get map = 0

I use Detr to train my data set and get map = 0
I use version 2.7 using Detr_ r50_ 8x2_ 150e_ Coco model training their own data sets (coco format), samples_ per_ The GPU parameter is set to 4, and the rest of the parameters are all by default. Why is the AP and AR on the verification set 0 after iterating to generation 70? How can I solve this problem
`
Running per image evaluation...
Evaluate annotation type bbox
DONE (t=2.70s).
Accumulating evaluation results...
DONE (t=1.15s).
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=1000 ] = 0.000
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=1000 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=300 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=1000 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.000
`

There are some hyper parameters like learning rate or some thing else need to be specifically tuned for your dataset. We have no idea about that.

I once had this problem! AP and recall are close to 0, but after I modify the learning rate, recall is 0.33 and AP is 0.0015 in the eighth epoch. I don't know whether my parameters still need to be modified, or whether the convergence of the model is so slow。Have you solved the problem? We can talk about it

 I've tried different optimizers, I've tried different learning rates (from 0.1 to 0.0000001), but I've got 0 map
------------------ 原始邮件 ------------------
发件人:                                                                                "open-mmlab/mmdetection"                                                                                @.>;
发送时间: 2021年4月16日(星期五) 下午4:20
@.>;
<EMAIL_ADDRESS>主题: Re: [open-mmlab/mmdetection] I use Detr to train my data set and get map = 0 (#4929)
I once had this problem! AP and recall are close to 0, but after I modify the learning rate, recall is 0.33 and AP is 0.0015 in the eighth epoch. I don't know whether my parameters still need to be modified, or whether the convergence of the model is so slow。Have you solved the problem? We can talk about it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi,
I am facing a similar problem with DETR using a custom dataset (custom_db). The mAP is always 0. I tried different param configurations.
Observations:

The loss is decreasing during training.
When I try to infer using DETR model weight (custom_db), it is predicting 100 bbox with a very low score. The 100 bbox coordinates and scores are exactly the same for different images.

Could you please take a look?
Thanks,
Neel

solution1: There is a formula for adjusting the learning rate. lr = 0.00125*batch_size
solution2: You can load pretrained weight.I remembered this file is trained by cocoDataset . So when I used ,I find a code from internet to change the weight file to match my dataset
All my solution.
Good luck!
| |
zhouxiangjun5450
|
|
@.***
|
签名由网易邮箱大师定制
On 4/23/2021 @.***> wrote：
 I've tried different optimizers, I've tried different learning rates (from 0.1 to 0.0000001), but I've got 0 map
------------------ 原始邮件 ------------------
发件人: "open-mmlab/mmdetection" @.>;
发送时间: 2021年4月16日(星期五) 下午4:20
@.>;
<EMAIL_ADDRESS>主题: Re: [open-mmlab/mmdetection] I use Detr to train my data set and get map = 0 (#4929)
I once had this problem! AP and recall are close to 0, but after I modify the learning rate, recall is 0.33 and AP is 0.0015 in the eighth epoch. I don't know whether my parameters still need to be modified, or whether the convergence of the model is so slow。Have you solved the problem? We can talk about it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sorry, I just saw the message.
My solution:

change learning rate. lr = batchsize * 0.00125
2.use pretrained weight. I remembered the weight was trained by cocodataSet. So when I use it , I find a code from internet to change the file  to match my dataSet
Hope my solution can give you  help

| |
zhouxiangjun5450
|
|
@.***
|
签名由网易邮箱大师定制
On 6/4/2021 22:50，Neelanjan @.***> wrote：
Hi,
I am facing a similar problem with DETR using a custom dataset (custom_db). The mAP is always 0. I tried different param configurations.
Observations:
The loss is decreasing during training.
When I try to infer using DETR model weight (custom_db), it is predicting 100 bbox with a very low score. The 100 bbox coordinates and scores are exactly the same for different images.
Could you please take a look?
Thanks,
Neel
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Support for AlpineJS v3

Are there any plans to support Alpine JS v3?

I've tried to upgrade it but it turned out more complicated than I thought since the plugins API changed.. I went with trying to create a new one that uses new features but I wasn't able to do so, a lot of things happened including my PC's disk failing.
I can recommend page.js as an alternative that's framework-independent and works fine with Alpine.js
If anyone would like to take this I'd be happy to accept push requests and sorry for the inconvenience.
Meanwhile I'll try once again

please support for Alpine v3 @rehhouari  ... really appreciated

please support for Alpine v3 @rehhouari ... really appreciated

the disk on my computer failed/broke, i can't work on it:(
I actually tried a few times but the plugin system changed; i might create a new lite version for alpine 3 but not promising anything because of the issues i'm having

Any updates on this?

I've updated it to Alpine v3, done some testing, it's almost backwards compatible except you no longer need x-router and the settings are set differently now.
sorry for taking long, even after I was able to work on it I didn't get through with it cause I was burnt out. Just pulled an all-nighter on this so I may have missed some edge-cases but both the examples are working so I'll call it a day.

That's great news, thank you.

Thanks so much! I'll try to test it and get feedback.


Security scan

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


Exact types - [WIP]

An initial attempt at Exact Types (#12936). This is not meant be a candidate for merging, but a chance to play around and experiment. The implementation is most likely buggy (re: parser + semicolons), and the errors could be better. Please do experiment with it though!
Semantics:
I write |T| to denote an exact version of T.

An object literal O is related to an exact type |T| if all properties in O are in T and O is related to T.
An exact type |T| is related to exact object type |U| if all properties in T are in U and T is related to U. (Normal typing handles the converse property check of U in T, I think).
An exact type |T| is related to an exact generic mapped type |U| if T is a generic mapped type and T and U have identical constraints.
Question: I don't think an exact but non-generic mapped type can be related to an exact generic mapped type. You never know the lower bound on the properties in the generic constraint, so you can never safely say that no properties will be lost by assigning to the exact generic mapped type. Is this right?
Excess property checking. There is no excess property checking for exact types as this is (partly) replaced by exactness. Excess property checking is disabled for non-exact types in an exact context. An exact context is within a branch of an union or intersection with at least on exact type. For example.

const contextTest: { | x: unknown; | } & { x: { a: string } } = { x: { a: "foo", b: "bar" } };  // ok

Property b is not reported as excess because the object literal { x: { a: string } } appears in an exact context. The choice came about as a hack. Freshness is turned off for EPC in nested intersection and union, but freshness in needed for exactness. To avoid duplicating essentially the same object flag I chose to keep freshness but turn off EPC in certain cases. I'm not sure if this is the best thing in general, but there are some nice things about it.
Examples:
type Exact<T> = {| [K in keyof T]: T[K]; |};

interface A {
    field: string;
}

interface B {
    field2: string;
    field3?: string;
}

type AorB = Exact<A> | Exact<B>;

const fixture: AorB[] = [
    {
        field: 'sfasdf', // error
        field3: 'asd'
    },
];

function foo<T>(x: Exact<T>): T {
    return x;
}

function foobar<T>(x: Exact<T>): Exact<T> {
    return { ...x };
}

foo({x: 1, y: 2}); // ok
foobar({x: 1, y: 2}); // ok

const dict: Record<string, number> = {};

foo(dict); // not ok
foobar(dict); // not ok

type ExactUnion = {| x: {z: string}; |} | {| y: boolean; |};
const a: ExactUnion = { y: true }; // ok
const b: ExactUnion = { x: {z: "hello"} }; // ok
const c: ExactUnion = { x: {z: "hello"}, y: false }; // not ok
const d: ExactUnion = { x: {z: "hello", other: 'hello'} }; // ok -- no excess checks in objects within exact (unless exact)

// to fix use deep exact
type DeepExactUnion = {| x: {| z: string; |}; |} | {| y: boolean; |};
const e: DeepExactUnion = { x: {z: "hello", other: 'hello'} }; // not ok

delete a.y; // error, can't delete on exact type

// Exact Constraints
type AllowedFields = "x" | "y";
type CorrectObject = {[field in AllowedFields]?: number | string};
type HasX = { x: number };

function exactTypes() {
  const checkType = <T>() => <U extends Exact<T>>(value: U) => value;
    const o = checkType<CorrectObject>()({
        x: 1,
        y: "y",
        z: "z", // error, not ok!
    }); // o has type Exact<CorrectObject>

  type HasX = { x: number };
  const oAsHasX: HasX = o; // succeeds
}

const x: {| x: number; y: boolean; |} = {x: 4, y: true};
const y: {| x: number; |} = x; // error

const x1: Exact<{ x: number; y: boolean;}> = {x: 4, y: true};
const y1: {| x: number; |} = x1; // error

const x2: {| x: number; y: boolean; |} = {x: 4, y: true};
const y2: Exact<{x: number; }> = x2; // error

For anyone watching: I'm waiting on some fixes for mapped type relations before picking this up again. Main thing to add is decent error messages.

@jack-williams are you still working on this?

No. Please feel free to take my changes and start your own experiments if you'd like.
For context: The reason why I stopped persuing this is because the method of object composition in TypeScript (intersection) is essentially meaningless for exact types. I think the correct solution is row types + row polymorphism, but that is a significantly bigger challenge.

@jack-williams that's why Flow has object spread

@jack-williams I'm not really familiar with TS codebase, is there some docs?

The spread type has been discussed here: #10727. The problem is that you end up with lots of similar concepts that are subtly different and it's not clear when to use what.
I don't think there are any good docs for the compiler itself, or if there are, I don't know where they are located. Sorry I can't be more helpful in this regard.

This experiment is pretty old, so I'm going to close it to reduce the number of open PRs.


/nft page mvp

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

Looks pretty! Found some lorem ipsum:


Error reinstalling a module instead of ignoring it

I don't think this should be changed. If it errors on install when there's nothing to do, then you can't write reusable scripts (for the button concept or otherwise) because there's no easy way to know if the database already has the required modules or not.

So... https://github.com/iasql/iasql-engine/blob/main/src/services/iasql.ts#L794-L797
We already do that? We need to figure out why that failed for you. Was that in a staging DB or a production DB?


Use 'auto', range-'for', and structued bindings to improve some loops

Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[x] Has this change been discussed on the forum or in an issue before?
[x] Does the code follow the SFML Code Style Guide?
[ ] Have you provided some example/test code for your changes?
[ ] If you have additional steps which need to be performed list them as tasks!

Description
Refactoring change. Should not affect behavior at all. The auto, range-for, and structued bindings features have been used to improve the readability of some loops and usages of iterators.
Tasks

[ ] Tested on Linux
[x] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?
CI.

SFMLBuildMaster: build this please.

@Bromeon: I addressed your comments, please take a look :)

I found a few more places to use ranged-for:
src/SFML/Window/Unix/WindowImplX11.cpp line 369
src/SFML/Window/Win32/JoystickImpl.cpp lines 468 and 1052

I found a few more places to use ranged-for:
src/SFML/Window/Unix/WindowImplX11.cpp line 369 src/SFML/Window/Win32/JoystickImpl.cpp lines 468 and 1052

Thanks! I will handle those in a separate PR (or feel free to do that yourself), I'd like this one to be merged since CI is happy with it :)

Sorry, I was looking at the master branch, not your branch when I referenced those files! I did find one loop you missed though at src/SFML/Network/Packet.cpp:578. I might submit a PR after yours get merged. Looks like a good first contribution for me.

@Bromeon: I think this can be merged if you are happy with the latest changes.


docs: small typo in static_pages.rst

Description
Small typo in static_pages.rst file URL-Will show table
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

@obozdag Thank you!


[BUG] Separator \n only outputs first changed file

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
When using separator: "\n", only the first line is kept in the step output.
To Reproduce

Use separator: "\n" with multiple changed files (more than one).
Echo the output from the next step.
In the latest step, you will only see the first changed file.

What OS are you seeing the problem on?
ubuntu-latest or ubuntu-20.04
Expected behavior?
Using separator: "\n" should set output correctly and retrieve all changed files, not only the first one.
Relevant log output
# -- STEP 1 --
> changed-files
...
  All changed files: .terraform.lock.hcl
  testfolder with spaces/.terraform.lock.hcl
  testfolder1/.terraform.lock.hcl
  ...

# -- STEP 2 --
> echo "$CHANGED_FILES"
.terraform.lock.hcl

Anything else?
I think this can solve the problem: https://github.community/t/set-output-truncates-multiline-strings/16852/3
Code of Conduct

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

@atorrescogollo I’m a little skeptical about using url encode values to replace the separator as not all downstream usage would decode the output.

@atorrescogollo This should be available in the latest release.


I could use a way to read the state of the emulated CPU/memory without holding a mutable reference

Hi,
thanks for this project! I am planning to use this emulator for an upcoming rewrite of my project, strop.
But to do this I would need to be able to do something like
pub struct Z80 {
    pub mach: iz80::PlainMachine,
    pub cpu: iz80::Cpu,
}

fn get_accumulator(&self, e: &Z80) -> u8 {
    e.cpu.registers().a()
}

This code fails to compile because
error[E0596]: cannot borrow `e.cpu` as mutable, as `e` is not declared as mutable
  --> src/z80/mod.rs:19:9
   |
18 | fn get(&self, e: &Z80) -> u8 {
   |               - help: consider changing this to be mutable: `mut e`
19 |     e.cpu.registers().a()
   |     ^^^^^^^^^^^^^^^^^ cannot borrow as mutable

I feel like the best way forward here would be to add a a method to &Cpu that returns an immutable reference to the Registers and then use its existing methods a, get8, pc etc.
Would you accept a pull request to do this or do you have a preferred way?

Yes, go ahead please.

Closed with #5, which implements the necessary method,


touch-action note for Firefox 52+

Added a note about not all Firefox platforms supporting pointer events or touch events, rendering touch-action support moot (eg Windows without APZ, Mac OS).

Thanks!


Catalog Android - popTo fail on first link

The first example of popTo in the Android catalog (that goes back to index.html/default controller) is not working.
Later use of popTo in the sample is OK. Maybe just a misconfiguration somewhere.
Everything looks good at @ggendre first sight.
find why and fix this on develop branch.

Fix on develop


Config filter sample update

Renamed config sample to make it clearer what it does.
Added logic to look at destination addresses, and if they match a pattern, treat those as redirects to environment variables.

Missing a using?
samples/ReverseProxy.ConfigFilter.Sample/CustomConfigFilter.cs(27,64): error CS0103: (NETCORE_ENGINEERING_TELEMETRY=Build) The name 'StringComparer' does not exist in the current context

Looks like this is ready to merge.

Missing a using?
samples/ReverseProxy.ConfigFilter.Sample/CustomConfigFilter.cs(27,64): error CS0103: (NETCORE_ENGINEERING_TELEMETRY=Build) The name 'StringComparer' does not exist in the current context

Yep, you suggested the change that added the comparer, that I accepted without merging locally 😺

#762


[Avatar] prevent Unknown Prop Warnings when Avatar is an iconButtonEl…

[ ] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

…ement on an IconMenu

Why are those property passed to the Avatar in the first place?

An example use case, which works (but generates unknown props warnings):
<IconMenu iconButtonElement={<Avatar src={current_user.avatar} />}>
  <MenuItem primaryText={'option 1'} />
  <MenuItem primaryText={'option 2'} />
</IconMenu>

Maybe a more appropriate implementation should be the following?
<IconMenu iconButtonElement={<IconButton style={{padding: 0}}><Avatar src={current_user.avatar} /></IconButton>}>
  <MenuItem primaryText={'option 1'} />
  <MenuItem primaryText={'option 2'} />
</IconMenu>

Yes, I think that the appropriate what to do it the second one has you get correct better keyboard accessibility.

I think that it would be great to add a warning when it's not true.

@oliviertassinari putting Avatar inside IconButton seems to result in a div inside a button element which is invalid HTML.


Beacon

Changed beacond images to bw cloud Rocky 9 image

Can one of the admins verify this patch?

I hope that image (c6906a58-1e05-4be0-8f20-41f24c8320b5) is available in the main tenant. I do not have access to check it.

It's there, I will merge it

Cool, please go ahead.


fix: same footer in all pages

Title and Issue number

Title :
Issue No. : 989
Code Stack :  HTML CSS JS
Close #989

Description

Footer aligned
Added same footer to the pages that had the footer
Space between counters fixed

Video/Screenshots (mandatory)

Type of PR

[x] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[x] I have performed a self-review of my code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have gone through the  contributing.md file before contributing

Additional context:

##Are you contributing under any Open-source programme?

[x] I am contributing under GSSOC'24
[ ] I am contributing under VSOC'24

@itsekta @apu52 @karishmaaa101 please check this


Allow table to be horizontally scrollable with full-width columns

There should be a way to disallow columns being truncated like this, and instead allow the table to stretch horizontally offscreen, which would then be able to be scrolled left and right with the arrow keys. There's an example output below for a table with many columns where truncation makes the table pretty much useless.

Yep, that's a useless table!
I was starting to type up that the easiest solution would be to render the table in a horizontally scrollable view component, but I'm looking at the footer and realizing that won't work as the footer should remain static even as the rest of the table scrolls.
This is a reasonable feature request, let me think about how this might work.

I've actually implemented a pretty simple solution for it here: https://github.com/influxdata/influx-cli/pull/393/files#diff-69a5ed4eca0dfe3e0e3d0a4f8a6552f9eafe6fd6fddcac1c1199176fab421578
Left and right arrow keys scroll between the visible columns.
Hopefully that helps with a little inspiration, and maybe you can lower some of that code into the library.

Thanks for the link!  I've actually already started working on this with a similar idea of column offsets, but with a bit more visual notice that scrolling is occurring.  I'm going to use shift+left and shift+right for scrolling as left/right are the default keys for paging, but that can be set through the keymap.

I've finished the first pass of horizontal scrolling, but I want to add column freezing as well - as in your example where you always have the index column.  This issue should remain open until that's done at least.

I've added horizontal freezing, so that you can freeze the first N columns, which should cover your use case if I understood your linked code above.  Let me know if this is reasonable or if there's other things that need to be added for this to be useful!  This has been released as v0.13.0.
It looks like Github automatically closed this issue due to one of the links I made, but I'd rather you close this yourself if you confirm that the new version solves the issue.  If it doesn't, please let me know what else you think may need to be done.

Awesome work! I do have a couple thoughts:

For the static footer, I think its width should be the full width of the rows. If I scroll to the right, the static footer still stays full width of the screen which makes the table look a little strange. Is there an easy way to do keep the row and footer size in sync?
Is there a method to get the width of the table?

I also think this behavior is a little strange. I like the small caret column on the left, but when it's on the right side, it is much bigger and visually seems off.

The footer thing is definitely a bug, I was focusing on all the cases where the table was larger, and not smaller... this should be fixed in #90 , will release after addressing the other issue that you posted as I was typing this as well.
For getting the total width, there are some internal calculations that are done to try and track the width for caching purposes, but it's not terribly nice for exposing to the outside world.  What are you trying to do?  A hacky way would be to render the table and run lipgloss.Width() on it to get the actual rendered width, if that would be useful.

For the right side carat, an easy fix is to force the alignment to be to the right.  A trickier question is whether or not the column should always just be one single character wide, because this may have unexpected impacts on the footer width.  Open to discussion on this one.

I think in the name of visual and scrolling consistency, it makes sense for the caret to be one character on either side.
It'd be nice that if you continue to scroll right, the caret column won't keep changing sizes, which is distracting from the actual data we want to show.

I changed my mind on releasing and released v0.13.1 with the footer fix as I need to crash to sleep for the night, and want to play with the carat width a bit more tomorrow.  Let me know if the footer works better for you with that.
I'll give this some thought of how to make the single character work while also preserving a potentially long footer as in your screenshot.

Also, another footer width edge case to handle:
When you scroll right and static footer wraps to create a new line, it means that the line count to clear changes from what the original calculation was. This makes it leaves residual top borders that stack every time a resize or scroll happens.

I think I want to optimize for the table width being constant.  This would add blank space at the end of the table if there are still > carets to display, and we could right-align the carets so that they don't jump around.  This would be similar to how it is now, just right-aligned.  The upside is the footer is safe.  The downside is the extra padding might look a bit odd, as you pointed out above.  Do you think this is a reasonable tradeoff?  I'm open to ideas/discussion here for sure.
In the meantime I'm going to go right-align the carets anyway, as I don't think there's a world where weirdly centered carets are a thing anyone wants.  We can adjust later, maybe with some configuration options.  I'm also going to adjust the scrolling so that it only scrolls until there's nothing left to the right, and stops.  This won't fix all footer resizing issues but will at least give some help there.

I've implemented the two things as discussed above and released as v0.13.2.  Give it a try and let me know if it helps, still open to discussing longer term formatting issues regardless.

I think that's a good call to optimize for consistency...Although I've tried out this new version, and my keybindings for left and right aren't doing anything anymore. Not sure if I'm missing something...

Were you using shift? The default keybind is shift plus arrows, but you can override that by changing the keymap.

Shift arrows did not work, and my overrides for left and right didn't work after that either.

Is the table set to be focused? As in myTable.Focused(true) is used when created?

Yeah, so I was able to page up and down, but not scroll left and right.

That’s a bit terrifying. Let me double check some things when I get some time. Are you able to run the scrolling example in this repo?

Another question that came to mind, what version of Bubble Tea are you using?  I did have to upgrade the Bubble Tea dependency in order to properly use shift+left/right, I'm wondering if there's a conflict there.

Ok, I've figured out that it's actually the remapping that is causing problems. If I have no WithKeyMap, the shift+left/right commands work great. But if I have WithKeyMap and pass in the default keybindings, or I try to override the ScrollLeft/Right keybindings, then I can't scroll left & right.

Oof.  I had trouble making that particular function work smoothly, and I thought I had fixed the edge cases but it's likely you're running into that.  Could you give me the dimensions of the table you're using so I can test locally?  As in, column widths and which are frozen?

I've fixed the keymap behavior to properly run all actions instead of short-circuiting, so your left/right rebinds should now work and are verified as unit tests with this change.

I finally got a good excuse to try out fuzzing, and it helped me find the edge cases you were likely running into.  I was doing an incorrect calculation regarding the frozen first column, which has been removed and now the fuzzer seems quite happy with all kinds of permutations.  Try v0.13.7 to see if that works better for you regarding scrolling all the way to the right again.

Glad to hear! I'll try it out a little later today and let you know.

Alright, I've come back and figured out that it still doesn't work...but I've discovered the exact line where things are going wrong:
	case tea.WindowSizeMsg:
		m.totalWidth = msg.Width
		m.simpleTable = m.simpleTable.WithPageSize(msg.Height - 7).
			WithMaxTotalWidth(msg.Width)
	}

For my use case, I want the table to fill the whole screen, and on resize it should still fill the whole screen.
When this resize line is commented out, I can scroll, but when it is present, I'm unable to scroll (but the table fills the screen line I want).


How to run hermit-os on raspberry pi?

Sorry for not answering earlier. Are you trying to run a Hermit VM on the Pi or are you trying to run bare-metal?
The former should be possible with QEMU, although I don't know if the Raspberry Pis support KVM or not. The latter would require some porting of the Hermit loader.


test: group poa admin

Add Group Proposal tests for when POA is a group address (group admin is the group policy itself).
Test

Software upgrade plan scheduling and canceling
Manifest module parameters update

All parameters
Inflation == nil
Empty parameters

Manifest stakeholders payout (mint) and burn (relates #65)
POA parameters update

All parameters
Empty admins

Bank send

Valid sender
Invalid sender

Token factory

Token create
Token modify metadata
Token mint and mint to
Token burn and burn from
Token force transfer
Token admin change

This PR is becoming huge so I want to get those merged. We can add more tests later.
Fixes #61

I'll re-run CI tomorrow, the test is failing because we reached the maximum number of Docker requests.


[Indigo] [Cypress] Make all components cypress-friendly.

Description
The Cypress requires some selection works and interaction works. Some of our customized component might not optimal for doing Cypress testing. We are going to make them friendly to the Cypress before assigning test-writing tasks out.

All changes for this issue should go into branch indigo-102-cy-friendly.


[BUG] Throwing error in salesforce-webhook

Describe the bug
If/when the event emitter hits an error it interrupts the event steam and has to be manually reset.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
No error-handling behavior has been implemented to this point, but we would want the
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Browse: Arc - Browse: Arc
Version [e.g. 22]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Test comment

Test comment

test comment

Test comment

test comment

test test

Describe the bug
If/when the event emitter hits an error it interrupts the event steam and has to be manually reset.
To Reproduce Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior No error-handling behavior has been implemented to this point, but we would want the
Screenshots If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Browse: Arc - Browse: Arc
Version [e.g. 22]
Version [e.g. 22]

Additional context Add any other context about the problem here.

UPDATE


COMP: Adds ComputeTangentAndNormals() function

A function name was changed to correct grammar and improve clarity, but
it broke backwards compatibility.   This request adds a function
with the old name to restore backward compatibility.

MacOS error:
-- Configuring done
CMake Error: install(EXPORT "ITKTargets" ...) includes target "gdcmjpeg8" more than once in the export set.
-- Generating done
CMake Generate step failed.  Build files cannot be regenerated correctly.


Create Sequence Diagram

[x] LucidChart the data flow
Questions:
[x] Where does the data start
[x] What Service does the data go through before it gets to the user?
[x] How is the data manipulated?
[x] Where is it stored?

SFGov-ERD-Sequence-Flow.pdf

The sequence diagram needs a revision. It looks like this is the same document as the ERD doc, but they should be different and have different purposes. I'm attaching a screen shot of what a sequence diagram looks like.

For ERD, the "Sections of Available Dates"'s contents will probably be change depending on what the API spits out. Same with "15 minute selected section"s. I have doubts about it being Dates/Array of Dates.
Besides that, everything looks straightforward and there's nothing much to change. There doesn't appear to be anything missing.


Admonition exclamation mark not displayed

Hi,
the admonition exclamation mark is not displayed properly.  Also not when including Lato and Roboto fonts:

Is there maybe an other font missing?

Please disregard this issue.  This seems to be part of "Font Awesome" that we somehow have to include.  I'll see how I can achieve that


compact:debug meta files are never deleted

Thanos, Prometheus and Golang version used:
Thanos: quay.io/thanos/thanos:v0.15.0
Prometheus: quay.io/prometheus/prometheus:v2.22.0

Object Storage Provider: Dell ECS
What happened:
We have a preview thanos deployment that we use with our preview clusters, I was looking at one of the S3 buckets that the sidecars upload to and found out that the $(bucket)/debug folder has a lot of files, much more than the number of objects in the s3 bucket, the retention in the compactor for this bucket is set to 3 months for the 3 retention flags, but looking at some of the meta files they are for blocks that are more than 1 year old now, for example:
"ulid": "01DPJDWX5QWGN99VA2CJMRRAR1",
	"minTime":<PHONE_NUMBER>000,
	"maxTime":<PHONE_NUMBER>000,
	"stats": {
		"numSamples": 11395469,
		"numSeries": 16123,
		"numChunks": 95231
	},
	"compaction": {
		"level": 38571,
		"sources": [
			"01DKH9RY2JTXYRNGRZ2TW4H4SW",
			"01DKHGMN9DP36NF94JCX31KAA3",
			"01DKHQGCJ1AY7K7VQQX3AB316M"
		],
		"parents": [
			{
				"ulid": "01DPJDW8ETX65KVVXHANAG0KNQ",
				"minTime":<PHONE_NUMBER>000,
				"maxTime":<PHONE_NUMBER>000
			}
		]
	}

that is from  Friday, August 30, 2019, which lead me to believe that the debug meta files might not get deleted, I looked a little bit at the code and I can see the files get created on upload:
https://github.com/thanos-io/thanos/blob/master/pkg/block/block.go#L94
but only the meta file in the block directory is deleted, DebugMetas not used anywhere in the Delete function inside that package:
https://github.com/thanos-io/thanos/blob/master/pkg/block/block.go#L167
What you expected to happen:
Debug meta file is deleted when the block is outside of retention window
How to reproduce it (as minimally and precisely as possible):
Set compactor retention to a value smaller than the current value, after the block object gets deleted the debug meta file will still be there.

That's correct, what do you propose? (:
This is really for debug, but was not proven to be very useful. Let's discuss and decide what to do :thinking:

I think the deletion call can mirror the Upload call and delete the file after deleting the meta file that is part of the block directory.

Maybe let's bring this up on the next community call? (:

Hi @GiedriusS, I don't know if the question is directed at me or someone else, I'm fine with that :) its a small issue and in worst case the files can be cleaned up offline by a separate tool if they become an issue.

Still reproducible

Still reproducible

We don't upload them anymore hence closing. https://github.com/thanos-io/thanos/pull/5110


Update bookinfo

This updates bookinfo. I thought it would be easy but the python ecosystem is.. interesting, and it felt easier to switch to OTEL than make it work with deprecated packages.
Once this PR is approved, I will push to the real repos and update the yamls
Fixes https://github.com/istio/istio/issues/49205

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
9ede9ce051c1b56b70e9c3f04a02f1a76a89ea69
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
9ede9ce051c1b56b70e9c3f04a02f1a76a89ea69
link
true
/test lint

gencheck_istio
9ede9ce051c1b56b70e9c3f04a02f1a76a89ea69
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

This updates bookinfo. I thought it would be easy but the python ecosystem is.. interesting, and it felt easier to switch to OTEL than make it work with deprecated packages.
Once this PR is approved, I will push to the real repos and update the yamls
Fixes #49205

Just want to know, which images(except bookinfo and distroless) are to be manually built and pushed, who can do that?

This updates bookinfo. I thought it would be easy but the python ecosystem is.. interesting, and it felt easier to switch to OTEL than make it work with deprecated packages.
Once this PR is approved, I will push to the real repos and update the yamls
Fixes #49205

Just want to know, which images(except bookinfo and distroless) are to be manually built and pushed, who can do that?

Only frank I can. It's everything under samples/book info/platform

This updates bookinfo. I thought it would be easy but the python ecosystem is.. interesting, and it felt easier to switch to OTEL than make it work with deprecated packages.
Once this PR is approved, I will push to the real repos and update the yamls
Fixes #49205

Just want to know, which images(except bookinfo and distroless) are to be manually built and pushed, who can do that?

Only frank I can. It's everything under samples/book info/platform

does header_injector build/push manually?

does header_injector build/push manually?

Bookinfo is done manually by https://github.com/istio/istio/blob/master/samples/bookinfo/src/build-services.sh. Looks like header_injector has its own makefile that can be run manually.

and push manually

@ericvn lmk if this looks good and I will build+push+update the pr

@howardjohn Sorry it took so long to verify things. Not everything went smoothly, but I was able to build, and run bookinfo with the new-images and verify things worked OK. I think this is good to go.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
9ede9ce051c1b56b70e9c3f04a02f1a76a89ea69
link
true
/test gencheck

lint_istio
ebb4332b4a4b4440477c89434c85ee5be7100b4d
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
ebb4332b4a4b4440477c89434c85ee5be7100b4d
link
true
/test lint

gencheck_istio
ebb4332b4a4b4440477c89434c85ee5be7100b4d
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
ebb4332b4a4b4440477c89434c85ee5be7100b4d
link
true
/test gencheck

lint_istio
2cd403f7414a6251e0e1a9dc04e6b76266d26f67
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
2cd403f7414a6251e0e1a9dc04e6b76266d26f67
link
true
/test lint

gencheck_istio
2cd403f7414a6251e0e1a9dc04e6b76266d26f67
link
true
/test gencheck

unit-tests_istio
2cd403f7414a6251e0e1a9dc04e6b76266d26f67
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Added Actinium (ACM) support

Hi,
This PR adds atomic swaps for Actinium (ACM).
A few resources regarding our project:
HP: https://actinium.org
GitHub: https://github.com/Actinium-project
ANN: https://bitcointalk.org/index.php?topic=4300440
Explorer: https://explorer.actinium.org
Explorer2: https://explorer2.actinium.org
Testnet Explorer: https://test-explorer.actinium.org
I have tested the functionality by using acmatomicswaps and btcatomicswaps.
Here the results in chronological order:
BTC initiate txn: https://test-insight.bitpay.com/tx/970876d6bbafc6bcb6e2af76c48af390bdf7dc0ef3fd73e40e4303d923c6c305
ACM participate txn: https://test-explorer.actinium.org/tx/8b5af3b2e6bc7c2f7d8635384ca9d7afc16659a7378b8c76ab80a5c8802be21d
BTC redeem txn: https://test-insight.bitpay.com/tx/f0be32fb27d5ff8b49936337d18b9321d735ecc03b5db4fc6417abcbffb34ede
ACM redeem txn: https://test-explorer.actinium.org/tx/e0a6a53069aaef3bd7c4577bf11e4d9ca7b39e85524e65cd61453bac16ede4a8
One hint: the examples in your README seem to have been written for an older variant of atomicswaps because the redeem & extractsecret calls are missing the secret hash respective secret value parameters.
Kind regards,

@brakmic please rebase  now that go 1.11+ is required.

@brakmic please rebase now that go 1.11+ is required.

Rebase done. Removed Gopkg.lock.

@brakmic need to add go.mod and go.sum in cmd/acmatomicswap/
https://github.com/golang/go/wiki/Modules

@brakmic need to add go.mod and go.sum in cmd/acmatomicswap/
https://github.com/golang/go/wiki/Modules

Have moved them to acmatomicswap.

@brakmic travis-ci has you flagged for abuse.  You can email<EMAIL_ADDRESS>and ask to have the ban removed.

@brakmic travis-ci has you flagged for abuse. You can email<EMAIL_ADDRESS>and ask to have the ban removed.

Weird. Will do it.

@dajohi
I got the email from travis. They can't find any misuse flag. Could your please, give me a screenshot of the misuse flag.
Here the content of their email:
 Juz (Travis CI)

Feb 21, 11:27 EST

Hi there Harris,

Thanks for getting in touch and I'm sorry for the frustration!

I"m having some difficulty finding any misuse flag for either decred or brakmic.
Could you send a screenshot of the misuse flag?

Many thanks and looking forward to hearing from you!
Cheers,
Juz

@dajohi
I got another email from Travis CI team:
Hi Harris, I'll close this ticket for now but please don't hesitate to follow up here if there are any issues. As my colleague confirmed, we don't see any misuse flag for account brakmic. We'd be happy to assist further if you have any questions, concerns or require further assistance.

They see no abuse in their systems, so I don't understand where the problem is. I have also logged on in my Travis-account today and couldn't see any warning there.

 # master 555117a
6 days ago
added go.mod & go.sum to acmatomicswap
-
Abuse detected

point them to https://github.com/decred/atomicswap/commit/555117a perhaps.

Ok, will do it.

@dajohi
I got a question from Travis team. It's about one of your team's projects:
Hello Brakmic,

Thanks for providing additional information, we are working to get this resolved as soon as possible.
By any chance, can you (or one of the Decred team members) shed some light on the

https://github.com/decred/dcrseeder project?

Our systems appear to be flagging this organization due to some code in this repo.

I look forward to hear from you and we will take it from there.

Regards

I sent an email to travis-ci.  will post once I receive a response.

@dajohi
The Travis team has unflagged my account. I have sent two trivial changes just to activate the build procedures and it worked. Here their email:
Hello Harris,

I see your builds are now successful. I'll be marking this as SOLVED.

If you run into any more issues, feel free to reach out to us.

Thanks and Happy Building!


refactor to use as a library

Refactored the repo to be able to install suql as a library.
Other changes are just formatting changes enforced by black formatter.

Abandoning for this: https://github.com/stanford-oval/suql/tree/2c733c8d4acae6733b6845e31c5bf59aabbd87e7


Add .NET docs

Proposed changes
.NET landing page from dotnetDocs branch to implement language chooser
Unreleased product version (optional)
1.5
Related issues (optional)

Didn't notice any serious issues at all :)

Woot!


Support slash as separator in tags

In my project I use a slash as a separator in the tags, for example:

myapp/v1.0.0
myapp-test/v1.2.3

This has some nice properties like being nicely grouped in some Git GUIs like Sublime Merge:

This doesn't seem to be supported by cargo-dist. Are there any plans to implement such support?
I'd be willing to work on that and submit as a PR myself.
From what I can tell, this is the place where the validation/parsing takes place: https://github.com/axodotdev/cargo-dist/blob/2c0f30bc3c628f285542d44cef778f0c8e17e19f/cargo-dist/src/tasks.rs#L1892-L1906
I couldn't find any other place in the code that seems to do it, as far as I can tell. Is it correct that this would be the one place that needs a change to support slashes as well?

Seems reasonable!


Create NAiAN

A nova aliança

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

This is an empty PR without a description. Closing it. Please feel free to re-open if / when there is some content.


meli: 0.8.9 -> 0.8.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/meli/meli/releases.
meta.description for meli is: Terminal e-mail client and e-mail client library
meta.homepage for meli is: https://meli.delivery
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.8.10 with grep in /nix/store/f33cywagwrjyahkx643rmmwz8v462xn9-meli-0.8.10
found 0.8.10 in filename of file in /nix/store/f33cywagwrjyahkx643rmmwz8v462xn9-meli-0.8.10

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

meli

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/787e9af16340b35d656f18f5be23ffcb290c7eee#meli

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

meli

Maintainer pings
cc @0x4A6F @matthiasbeyer for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


Launch script multislice

Things I'm going to add to launch.py

Migrate some commands API from tpu-vm to queued-resources to work better with multislice training
Delete the instance if command fail (regardless of autodelete) so that it won't eat up all the quota after many retries

@dlwh @rjpower  FYI

For (1) SGTM, let me know if I can help. Something simple that just runs queued-resources create and then polls until the instance is started seems like a good first step.
For (2) don't we normally want to reuse the instance? That's much faster for me in practice than delete/restart. (Or is this for batch job, in which case, sure, makes sense as an option).

In my experience when an instance is preempted (suspended), then you cannot create another one with the same name. The suspended instance is going to stay there and take up the quota.

I think you all are taking about slightly different things. If the node is
still up we want to reuse. Otherwise we have to delete the node and queued
resource and recreate
(Russell this is one of the main reasons i do not like queued resources…)
On Wed, Jun 12, 2024 at 4:50 PM Jason Wang @.***> wrote:

In my experience when an instance is preempted (suspended), then you
cannot create another one with the same name. The suspended instance is
going to stay there and take up the quota.
—
Reply to this email directly, view it on GitHub
https://github.com/stanford-crfm/levanter/issues/629#issuecomment-2164088339,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAACLIK7VD5ZXBJVFF6F5JDZHDNCXAVCNFSM6AAAAABJHJHEZ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRUGA4DQMZTHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@rjpower is there a way to specify which branch to run?

Not specifically. The script packages whatever is in your current
directory, so you can just checkout the branch you want to run.
On Thu, Jun 13, 2024, 2:33 PM Jason Wang @.***> wrote:

@rjpower https://github.com/rjpower is there a way to specify which
branch to run?
—
Reply to this email directly, view it on GitHub
https://github.com/stanford-crfm/levanter/issues/629#issuecomment-2166831747,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEUHZ57IXFWILWAU4CD6WLZHIF33AVCNFSM6AAAAABJHJHEZ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRWHAZTCNZUG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


If an element already has value, initializing inputmask on that set the value incorrectly.

If a page has an input element with prepopulated values, setting the inputmask on it with change the value to the masked value.

@pushkar1701 did you solve this? I am getting the same issue with a date field, I am working with a yyyy-mm-dd format in my backend, and masking the input as mm/dd/yyyy, but the initial prepopulated value of the field (in yyyy-mm-dd format) is breaking the mask

@pushkar1701 did you solve this? I am getting the same issue with a date field, I am working with a yyyy-mm-dd format in my backend, and masking the input as mm/dd/yyyy, but the initial prepopulated value of the field (in yyyy-mm-dd format) is breaking the mask

same here.. did you solve it? thanks


Uknown operationId

Instructions:
Run pnpm test. As soon as u see "broadcasted", CTRL+C and then run pnpm test again. The tx u submit now should have a stale issue. Then just let it run, keep retrying until you see something like:
Uknown operationId on {
  event: 'operationCallDone',
  operationId: '21',
  output: '0x00db0100000000000000049014495d6197bf209b8e3403ae52ca793d17aede3992af1157c286ebb96b853c243a000000370000000000000001'
}

Repro: https://github.com/ryanleecode/papi-test-2
Logs: https://gist.github.com/ryanleecode/b100bdae429b64571c736e4e8b0e18bf
related: https://github.com/polkadot-api/polkadot-api/issues/634

The tx u submit now should have a stale issue

No, it shouldn't. Because by default, before broadcasting the transaction, the transaction is validated against the currently finalized block, so against that block the transaction is valid.
You do realize that you are purposely creating a race-condition in which just one of the 2 transactions will be included, right? Also, you do realize that 99.9% of the time that would be the first transaction?
Are we on the same page thus far?

this is just steps to repro not actually how its being used.
we have an web server that submits usernames but it associated with one account. did some testing with mutliple concurrent requests and saw this log warning showed up. discussed with @voliva to just open an issue so u guys can take a look at it...
anyways im already aware this approach isnt viable. were going to run a cron job that runs a batch call instead

In the logs I can clearly see that everything is working as expected. I'm afraid that is not a bug, but rather a misunderstanding and/or a misusage of the library.
The "Uknown operationId on" message is to be expected. What's happening is that there is an ongoing operation got canceled, but we received its started event anyways. The log is there because in other cases that could be something problematic, but not in this case. In the logs we can clearly see that's the case, because these 2 things happened at the exact same time:
1-2024-08-15T21:02:06.544Z-<<-{"jsonrpc":"2.0","result":{"result":"started","operationId":"21"},"id":"1-32"}
1-2024-08-15T21:02:06.544Z->>-{"jsonrpc":"2.0","id":"1-34","method":"chainHead_v1_stopOperation","params":["fRO6LmRycl4DEYL8","21"]}

So, IMO everything is fine here.
I'm closing this issue. However, if there is anything that you think I missed, please feel free to open it again.


Keyed Redis component doesn't work correctly

If I add 2 Redis server instances and try to use "keyed" services to talk to each, only the first connection is used for both. The 2nd connection doesn't get used.
var builder = DistributedApplication.CreateBuilder(args);

var basketCache = builder.AddRedis("basketcache");
var basketCache2 = builder.AddRedis("basketcache2");

var basketService = builder.AddProject("basketservice", @"..\BasketService\BasketService.csproj")
                           .WithReference(basketCache)
                           .WithReference(basketCache2);

And then in BasketService's Program.cs:
var builder = WebApplication.CreateBuilder(args);

builder.AddKeyedRedisClient("basketcache");
builder.AddKeyedRedisClient("basketcache2");

var app = builder.Build();

var m = app.Services.GetRequiredKeyedService<IConnectionMultiplexer>("basketcache");
var m2 = app.Services.GetRequiredKeyedService<IConnectionMultiplexer>("basketcache2");

Console.WriteLine(m.Configuration);
Console.WriteLine(m2.Configuration);

Trying to debug it locally, it looks like a bug on this line:
https://github.com/dotnet/aspire/blob/01c625a518ad2b3a438a3607e122873c6395a50e/src/Components/Aspire.StackExchange.Redis/AspireRedisExtensions.cs#L83-L89
This needs to respect the "serviceKey". It is adding an OptionsFactory<ConfigurationOptions> without the key, which means it will be global.


fix any dict substitution

Codecov Report

Merging #4 (9613c41) into master (64034db) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   97.73%   97.73%
=======================================
  Files           5        5
  Lines         221      221
  Branches       60       60
=======================================
  Hits          216      216
  Misses          2        2
  Partials        3        3

Impacted Files
Coverage Δ

revolt/_validator.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Lm32 leds

This implements a Python module called litex, and a module within that called LED that can control the LEDs.

I suggest we force-push this to lm32 and use it as the basis for our development.
Ideally we would have started from an upstream tag, but we can do that later.

Couple of random thoughts;

There is no generic "leds" module that multiple boards implement?
We should rename the "lm32" directory to "litex". It's not really tied to the lm32 architecture right?
We should figure out a way to not commit the generated directory.


Google TTS does not working when SSL is enabled

Could someone possibly explain in plain English why this doesn't work in Hass.io when the DuckDNS add-on is configured? Media player turns on but nothing is spoken.
base_url is set as https://mydomain.duckdns.org:9123 and 9123 is port forwarded to 8123 internally in my router. Remote access works just fine.
Hass.io 0.65.6
Hass.io supervisor 0.98
Host system 1.3
Pi3

/move home-assistant/home-assistant


Trigger value set download (EXPOSUREAPP-10903)

Moves the trigger to download value sets to the repository as the overview screen may be skipped using the universal scanner
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-10903
@mlenkeit Can you confirm that this is fine with you as it is a change from the tech spec that requested the check to be on the certificate overview?

@chiljamgossow yes, I think that's fine 👍


:rotating_light: ACTION REQUIRED: Confirm Repository Status for Migration by Dec 20 :rotating_light:

Hi ,
The Developer Experience (DevX) team is moving code from the current BCDevOps GitHub organization to the new bcgov GitHub Enterprise organization, which is protected by single sign-on (SSO). To prepare for this migration, we need you to confirm the status of your repositories.
Here’s what you need to do:
Respond to this issue by December 20, 2024, to confirm if each of your repositories is:

Active and ready for migration to bcgov, OR
No longer needed and can be archived.

:bell: ACTION REQUIRED: Leave a comment on this issue with your repository status.
If we don’t hear back by December 20, we’ll proceed with archiving your repositories.
Questions? Email<EMAIL_ADDRESS>for help.
Thank you for your cooperation!
The Developer Experience Team, Digital Office

Hi ,
This is a reminder to confirm the status of your repositories by December 20, 2024, as part of the migration to the bcgov GitHub Enterprise organization.
Please leave a comment on this issue to let us know if your repositories are:

Active and ready for migration, OR
No longer needed and can be archived.

If we don’t receive a response by the deadline, we’ll proceed with archiving your repositories.
Need help? Email<EMAIL_ADDRESS>Thank you for your attention!
The Developer Experience Team, Digital Office

Hi ,
:rotating_light: Final call! This is your last chance to confirm the status of your repositories as part of the migration to the bcgov GitHub Enterprise organization.
Leave a comment on this issue by December 20, 2024, to confirm whether your repositories are:

Active and ready for migration, OR
No longer needed and can be archived.

If we don’t hear from you by the deadline, we’ll proceed with archiving your repositories.
Questions? Email<EMAIL_ADDRESS>Thank you for your prompt response!
The Developer Experience Team, Digital Office


Demo not working

The demo fails to return any results, showing a 503 (Service Unavailable) error.

Running on localhost throws "Request failed with status code 400".

I added http:// to the URL in localhost and it worked (no more 400), the online demo is still failing.

should be fixed now with the recent commits, at least there shouldnt be any more crashes and the project should run fine  - but I do think the stealth package we use here https://www.npmjs.com/package/puppeteer-extra-plugin-stealth is now outdated so performance of getting link previews not as good any more

@santiagopuentep feel free to look at #7 if you'd like to improve the results!


Quotes Around Variable in WHERE Clause

Attempting to use a variable in the WHERE clause as:
WHERE column_name = $variable/string
or
WHERE column_name = [[variable]]/string
For variable contents: "foo" this should expand to 'foo/string'.
Instead it expands to 'foo'/string.
Similar behavior seems to have been discussed for the MySQL and PostgreSQL (https://github.com/grafana/grafana/pull/9611) but this still seems to be a problem for the Crate datasource.

On further investigation, I did find that this works as a workaround:
WHERE column_name = $variable || '/string'
This seems to be contrary to the Grafana documentation on how this should work, however.

Hello,
I had the same sort of issue trying to use a variable in a WHERE clause in the metrics interface
WHERE column_name like $variable
or
WHERE column_name like [[variable]]
is translated in the query as :
WHERE column_name like ''<variable value>''   the 2 single quotes surround,
results in an sql error...
I manage to workaround the problem by adding '' ||  before $variable to have:
WHERE column_name like
'' || $variable
in the metrics interface avoid the issue in the query:
SELECT ... WHERE ... AND column_name like '' || '<variable value>' GROUP...
which success and behave as wanted too ;O)
Hope it will help, before the issue can be fixed.

Hi hi :)
Thanks a lot for your input! With our latest effort, the PostgreSQL implementation of Grafana works fine with CrateDB. Hence, we skip the Grafana Datasource Plugin for CrateDB and move over to the PostgreSQL data source. In case you wanna read more about how to pair CrateDB and Grafana, please take a look at this blog post of ours.
Best, Stephanie