load_relation not behaving with a different table

With my incremental macro setup like this
{% materialization incremental, adapter='sqlserver' -%}

  {% set unique_key = config.get('unique_key') %}

  {% set target_relation = this.incorporate(type='table') %}
  {% set existing_relation = load_relation(this) %}
  {% set tmp_relation = make_temp_relation(this) %}

  {{ run_hooks(pre_hooks, inside_transaction=False) }}

  -- `BEGIN` happens here:
  {{ run_hooks(pre_hooks, inside_transaction=True) }}

  {% set to_drop = [] %}
  {% if existing_relation is none %}
      {{ log("In incremental, if existing relation is none.This:" ~this ~ ". Target relation:"~ target_relation)}}
      {% set build_sql = create_table_as(False, target_relation, sql) %}
  {% elif existing_relation.is_view or should_full_refresh() %}
      {#-- Make sure the backup doesn't exist so we don't encounter issues with the rename below #}
      {% set backup_identifier = existing_relation.identifier ~ "__dbt_backup" %}
      {% set backup_relation = existing_relation.incorporate(path={"identifier": backup_identifier}) %}
      {% do adapter.drop_relation(backup_relation) %}

      {% do adapter.rename_relation(target_relation, backup_relation) %}
      {% set build_sql = create_table_as(False, target_relation, sql) %}
      {% do to_drop.append(backup_relation) %}
  {% else %}
      {% set tmp_relation = make_temp_relation(target_relation) %}
      {% do run_query(create_table_as(True, tmp_relation, sql)) %}
      --TODO: This config override is hacky
      {% do adapter.expand_target_column_types(
             from_relation=tmp_relation,
             to_relation=target_relation) %}
      {% set build_sql = incremental_upsert(tmp_relation, target_relation, unique_key=unique_key) %}
  {% endif %}

  {% call statement("main") %}
      {{ build_sql }}
  {% endcall %}

  {% do persist_docs(target_relation, model) %}

  {{ run_hooks(post_hooks, inside_transaction=True) }}

  -- `COMMIT` happens here
  {% do adapter.commit() %}

  {% for rel in to_drop %}
      {% do adapter.drop_relation(rel) %}
  {% endfor %}

  {{ run_hooks(post_hooks, inside_transaction=False) }}

  {{ return({'relations': [target_relation]}) }}

{%- endmaterialization %}

Log returns this
2021-06-17 13:33:12.042538 (Thread-1): In incremental, if existing relation is none.This:"DatabaseOutSideOfDBTProfile"."schemaname"."tablename". Target relation:"DatabaseOutSideOfDBTProfile"."schemaname"."tablename"
The issue for me is that the Table "DatabaseOutSideOfDBTProfile"."schemaname"."tablename" Does exist, so something is happening in load_relation (Line 6 above) that's not behaving as I'd expect.
https://github.com/fishtown-analytics/dbt/blob/c0d757ab1926d8d5560d857c5f776d3358618f42/core/dbt/include/global_project/macros/materializations/helpers.sql#L60 Which calls adapter.get_relation . Something is up

is the reason you're trying to make a new incremental materialization because it isn't working for you? IIRC, incremental should be supported currently....

is the reason you're trying to make a new incremental materialization because it isn't working for you? IIRC, incremental should be supported currently....

I need a merge strategy for the incremental materialization that doesn't exist today!

Not related at all, this was from my own Macros that are changing the names of things.


Date

It's me again ... I created a fiddle -https://jsfiddle.net/Misterfluff/8k0crvc9/
This works great. The only problem is that the date slips by one day. The first value should be 40 for April 1. The value represented is for April 2 (56). I know there is a function to fix this but I do not understand how to incorporate it. The function looks like this:
afterLoadData: function (timestamps) {
var offset = new Date().getTimezoneOffset() * 60;
var results = {};
for (var timestamp in timestamps) {
var commitCount = timestamps[timestamp];
timestamp = parseInt(timestamp, 10);
console.log(timestamp)
results[timestamp + offset] = commitCount;
};
return results;
}
Since I already have an afterLoadData statement, how would I incorporate this?
Sorry for so many questions ... still learning :-)

Man, you saved my (few) days with that timestamp acrobatic! :D

I do see the 40 for April 1.
I guess it's a timezone issue, so related to #105
v4 will use moment-timezone to handle all the dates, and add support for timezone


Fix the test cafe Login dropdown

Please do work for this task in a branch called issue-019.

[ ] Either remove this line or edit it so it matches our current login button


fix: correct environment handling in client_options_test

Correct a typo in the environment-variable name from #3353.
Add a missing environment-variable TearDown() from #3338.

This change is

do you mind rebasing to make sure the builds pass and merge after that?

Done.  Passing.  Merging.


Jax process hangs instead of failing on NCCL internal error

Description
I'm using JAX for large-scale multi-node on training and sometimes (in rare cases, maybe once in 3-5 days of non-stop computing) during training step I'm getting following error on some workers
worker X:
2024-01-11 05:43:11.726254: E external/xla/xla/service/gpu/nccl_utils.cc:211] Aborting communicator: 0x7f3e427e55b0 due to async NCCL error: remote process exited or there was a network error
2024-01-11 05:43:11.726330: E external/xla/xla/service/gpu/nccl_utils.cc:219] INTERNAL: external/xla/xla/service/gpu/nccl_utils.cc:216: NCCL operation async_err failed: remote process exited or there was a network error

worker Y:
2024-01-11 05:42:58.876471: E external/xla/xla/service/gpu/nccl_utils.cc:211] Aborting communicator: 0x7f45267e31a0 due to async NCCL error: remote process exited or there was a network error
2024-01-11 05:42:58.876681: E external/xla/xla/service/gpu/nccl_utils.cc:219] INTERNAL: external/xla/xla/service/gpu/nccl_utils.cc:216: NCCL operation async_err failed: remote process exited or there was a network error

after which training hangs forever (or least for 6h+, i've never waited more).
I don't think that it is jax bug, most likely just random fail with hardware as i have lots of GPUs.
The problem is that I have checkpointing and automatic restoration mechanism is case of fail of any worker but it doesn't work in this case as after this error training process completely hangs instead of failing.
What jax/jaxlib version are you using?
jax, jaxlib 0.4.20
Which accelerator(s) are you using?
GPU
Additional system info?
linux
NVIDIA GPU info
32x8 H100 GPUs, infiniband network, RDMA

Can you try upgrading to jax and jaxlib 0.4.23 and see if this goes away?
If not, dump the HLO? I am assuming that this ia a XLA:GPU issue.


Fix extensionless image loading panic

Objective
Avoid panic when using path without extension, but providing format in the asset meta.
Solution
Reorganize the code.
Changelog
Fix extensionless image loading panic.

@VictorBulba can you please remake this targeting main?


Seal the Spanned trait

This trait should have been sealed. https://docs.rs/syn/0.14/syn/spanned/trait.Spanned.html

Fixed in 6315c6555a104cac5e94920c60aee7cedb249aa7, will be released in 0.15.


Proposal:  hz supports generating handy RPC-style client code based on IDL

Proposal
hz supports generating hertz client code. All opinions on the user side are worth considering❤.
Current Template(Opinions are all welcome)🤗

The initial client template that has been designed so far: client template.
Generate client code based on IDL and auto-populate http request parameters based on IDL's annotations.
If you have any idea or suggestion, you may reply below or mention pr to the branch example/hz_client_template.

Quickly experience🤓
You may have a try on this feature with this example.
Description🧐

Writing an IDL, use hz for the code generation(coming soon):

namespace go cloudwego.hertz.template

struct Req {
  1: optional string QueryString(api.query="q1");
  2: optional string PathString(api.path="p1");
  3: optional string HeaderString(api.header="h1");
  4: optional string MixString(api.query="q2", api.header="h2", api.path="p2");
  5: optional string BodyString(api.body="sad");
}

struct Resp {
  1: optional string RespString(api.header="h1");
}

service Hertz {
    Resp BizMethod2(1: Req req)(api.post = '/life/client1');
    Resp BizMethod3(1: Req req)(api.post = '/life/client2');
    Resp BizMethod4(1: Req req)(api.post = '/life/client3');
}

Call with generated client:

	str := "hello, hertz"
	req := template.Req{
		QueryString:  &str,
		PathString:   &str,
		HeaderString: &str,
	}

        // Case1
	// Initiate interface calls directly, domain can be specified by IDL
	respStruct, rawResp, _ = hertz.BizMethod2(context.Background(), &req)

        // Also provide 3 ways of customization bellow:

	// Case2
        // Pass the configuration parameters to create the client and initiate the http call
	cli, _ := hertz.NewClient("http://<IP_ADDRESS>:8899", hertz.WithHertzClientOption(client.WithKeepAlive(true)))
	respStruct, rawResp, _ := cli.BizMethod2(context.Background(), &req, config.WithSD(false))

	// Case3
	// Create a client by passing a hertz client that has already been initialized
	customizedClient, _ := client.NewClient()
	cli2, _ := hertz.NewClient("http://<IP_ADDRESS>:8899", hertz.WithHertzClient(customizedClient))
	respStruct, rawResp, _ = cli2.BizMethod2(context.Background(), &req)

        // Case4
	// Injecting client middleware
	cli3, _ := hertz.NewClient("http://<IP_ADDRESS>:8899", hertz.WithHertzClientMiddleware())
	respStruct, rawResp, _ = cli3.BizMethod2(context.Background(), &req)

I think we should also support not using the Go-Moudle‘s case.

Is it possible to parse it into an AST that meets the needs of the http client, and let the community or users themselves use different generators to support code generation in different languages or different client frameworks?
It is difficult for us to decide whether users should use the hertz client or even the go language.

Is it possible to parse it into an AST that meets the needs of the http client, and let the community or users themselves use different generators to support code generation in different languages or different client frameworks? It is difficult for us to decide whether users should use the hertz client or even the go language.

IDL is already implementation-neutral, what hz does here, is generate the code according to IDL.

possible to parse it into an AST that meets the needs of the http client, and let the community or users themselves use different generators to support code generation in different languages or different client frameworks?
It is difficult for us to decide whether users should use the hertz client or even the go lang

Currently, this solution is used to generate the hertz client. If you need an http client for another language or another framework, you can parse the AST and generate the code yourself


Update the last project-load InitializeOnLoad handler to only run on project startup

Per, https://github.com/microsoft/MixedRealityToolkit-Unity/issues/8188, we're working on making it so that InitializeOnLoad isn't as costly for ongoing compile time/play mode entry times.
This particular handler is fairly pricey but only needs to be run on Editor launch.

/azp run


Add HKDF to bitcoin_hashes

@rustaceanrob and I have been working on a Rust-based BIP324 implementation over at https://github.com/rustaceanrob/bip324. We have been attempting to keep the code pretty clean in hopes of a future "soft landing" in rust-bitcoin. I figured the HKDF implementation is a small, self-contained chunk that might allow us to learn the ropes here first.
There was a mention in the discussion thread on BIP324 that the hashes interface may be changing in the near future. I am not sure the effect that would have on this implementation, but happy to work through any issues.
Closes #2551

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

82 of 85 (96.47%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 83.278%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

hashes/src/hkdf.rs
82
85
96.47%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
19149

Relevant Lines:
22994

💛  - Coveralls

@nyonson just FYI we are in a feature freeze at the moment. This only touches hashes and that has been released already so this could go in but it might be easier to just wait a week or so before we consider merging this - @apoelstra will likely work this out, but just wanted to mention it.

Hey @apoelstra bump please.

ACK 7d9b21ca3bd8eb7e04b0f8f16ac2ec360a73b307 but could you squash all these commits (or most of them) since they are largely part of the same logical change? Also some of these commits don't compile by themselves.

Woops, I probably should have noticed that. My bad.

ACK 7d9b21c but could you squash all these commits (or most of them) since they are largely part of the same logical change? Also some of these commits don't compile by themselves.

My bad. I crunched down the test ones since they weren't all that interesting.

Hey all! Any technical hold up on this one? There wasn't a ton of demand to get it merged before, but the payjoin guys would like to make use of it now.

Sorry, I think it just fell between the cracks.

Needs rebase since #2746 (I think) which removed the index traits from Hmac.

The payjoin guys would like to make use of it now

Here to confirm this HKDF is useful to implement Hybrid Public Key Encryption based on secp256k1 & bitcoin dependencies for p2p Authenticated Encryption with Additional Data & Oblivious HTTP, both used in Payjoin V2.

This diff will get you building:
diff --git a/hashes/src/hkdf.rs b/hashes/src/hkdf.rs
index eb4fe372..e11815a8 100644
--- a/hashes/src/hkdf.rs
+++ b/hashes/src/hkdf.rs
@@ -62,7 +62,7 @@ impl<T: Hash> Hkdf<T> {
         let total_blocks = (okm.len() + T::LEN - 1) / T::LEN;

         while counter <= total_blocks as u8 {
-            let mut hmac_engine: HmacEngine<T> = HmacEngine::new(&self.prk[..]);
+            let mut hmac_engine: HmacEngine<T> = HmacEngine::new(&self.prk.as_ref());

             // First block does not have a previous block,
             // all other blocks include last block in the HMAC input.
@@ -80,7 +80,7 @@ impl<T: Hash> Hkdf<T> {
             let end_index =
                 if counter == (total_blocks as u8) { okm.len() } else { counter as usize * T::LEN };

-            okm[start_index..end_index].copy_from_slice(&t[0..(end_index - start_index)]);
+            okm[start_index..end_index].copy_from_slice(&t.as_ref()[0..(end_index - start_index)]);

             counter += 1;
         }

@apoelstra FTR, once CI passes, please merge this before the epic hashes re-write.

This diff will get you building:
diff --git a/hashes/src/hkdf.rs b/hashes/src/hkdf.rs
index eb4fe372..e11815a8 100644
--- a/hashes/src/hkdf.rs
+++ b/hashes/src/hkdf.rs
@@ -62,7 +62,7 @@ impl<T: Hash> Hkdf<T> {
         let total_blocks = (okm.len() + T::LEN - 1) / T::LEN;

         while counter <= total_blocks as u8 {
-            let mut hmac_engine: HmacEngine<T> = HmacEngine::new(&self.prk[..]);
+            let mut hmac_engine: HmacEngine<T> = HmacEngine::new(&self.prk.as_ref());

             // First block does not have a previous block,
             // all other blocks include last block in the HMAC input.
@@ -80,7 +80,7 @@ impl<T: Hash> Hkdf<T> {
             let end_index =
                 if counter == (total_blocks as u8) { okm.len() } else { counter as usize * T::LEN };

-            okm[start_index..end_index].copy_from_slice(&t[0..(end_index - start_index)]);
+            okm[start_index..end_index].copy_from_slice(&t.as_ref()[0..(end_index - start_index)]);

             counter += 1;
         }

Ah, I see thanks. Totally missed my index usage there.

Working on the API updates


Deploy old demo (from Feb 2020) to BioSimulations.org

With demo models and simulations to we can illustrate what the tables of resources and views of individual resources will look like

I have deployed the feb 2020 version, to biosimulations.org, but this depended on the old API, which is not running. I can look into older demos that contained the example data within the app itself, but those will not have much of the front page changes

I was thinking of going back further when the models and simulations were hard-coded. Maybe this commit from December 2019 c68385794258d092ef3b74d6adfcde596dc592c1. I forgot that this did use the API for user account information.
I don't want to spend a lot of time setting this up.  Alternatively, run this commit locally, comment out any code for the API or auth0, and save a few screenshots to show during a presentation.


DeskReservation -

Sample
DeskReservation
Author(s)
@aprildunnam
Question
hanks for providing this! I've never touched Power Apps before, so a bit of struggle. I completed the "deployment" instructions per the GitHub page, but could use some guidance after that, if anyone has tips.

I added a desk to the Desks list in SP, but it doesn't show up in the app when attempting to make a booking ("Uh oh, no openings found!"). It was already connected to SP lists via the "Data" tab on the left.

Lots of small red "x" symbols. Seems related to the gallery groups? Not sure where to begin to fix this.

Any help is appreciated!

I have the same issue. I tried amending the advanced settings on my SharePoint lists, to no avail.

Morning, same for me, followed instructions, deleted existing data sources, replaced with my sharepoint URL, same with the red "x" symbols, but it all looks right - any advice greatly appreciated!

Same here. A lot of "Name isn't valid. This identifier isn't recognized." formula errors. I have set up my lists exactly as described in the instructions. I even deleted the lists and started from scratch.

Closing this issue. We'll revisit this sample, fix issues, and post an updated version at a later time.


Feature request: data tags for objects

Is your feature request related to a problem? Please describe.
Trying to create an interactive panel with popups to control various advanced properties for multiple entities shown on the page.
For example, cover control has 3 buttons: UP, STOP and DOWN. Now if you have 3 windows in a room you'll have 3 covers that's 9 buttons - that's still OK.
I'd like to include a few shortcut position buttons for these covers, like 1/3, 3/4 and "small holes", and a slider for free positioning. Now having all these for all 3 covers on the same page would not only very heavy but also hard to control.
Since these properties are used more rarely, I'd debloat the screen by generating dynamically an obj from HA with the buttons and the slider popping up on long-pressing the stop button on the page.
Now the problem is that I can do that effectively only for a single cover with the CC. To do it for each cover I'd have to duplicate the entire config of the popups using new ids for each cover. Painful especially because these are only drawn for a short while and then deleted (could also be shown/hidden)
Describe the solution you'd like
Have an universal tag property (re-writable) for all objects, supporting strings up to, say 100 characters. If tag is set ( != empty string) for an object, always return it together with the event.
If this would be possible, I would only have to create the objects uniquely with the tags set to the entity_id of the objects in HA, and processing the events of these objects later can be done universally!
Additional context

# this is a STOP button for COVER1 on the main page, long-press for the popup
      - obj: "p1b6"
        properties:
          "text_color": "{{ '#FFFF00' if is_state('cover.my_room_1', 'closing') else '#FFFFFF' }}"
          "text_opa": "{{ '80' if is_state_attr('cover.my_room_1','current_position', 0) else '255' }}"
        event:
          "down":
            - service: cover.close_cover
              target:
                entity_id: "cover.my_room_1"
          "hold":
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: jsonl
                parameters: >-
                  {"page":1,"id":200,"obj":"obj","x":5,"y":55,"w":230,"h":190,"shadow_opa":255,"shadow_color":"black","shadow_width":20,"shadow_spread":0}
                  {"page":1,"id":201,"obj":"btn","parentid":200,"x":10,"y":40,"w":150,"h":60,"toggle":false,"text_font":24,"text":"Small holes","tag":"cover.my_room_1"}
                  {"page":1,"id":202,"obj":"btn","parentid":200,"x":10,"y":110,"w":70,"h":60,"toggle":false,"text_font":24,"text":"1/3","tag":"cover.my_room_1"}
                  {"page":1,"id":203,"obj":"btn","parentid":200,"x":90,"y":110,"w":70,"h":60,"toggle":false,"text_font":24,"text":"3/4","tag":"cover.my_room_1"}
                  {"page":1,"id":204,"obj":"slider","parentid":200,"x":180,"y":40,"w":30,"h":130,"min":1,"max":100,"tag":"cover.my_room_1","val":
                  {{- state_attr('cover.my_room_1','current_position') -}}
                  }
                  {"page":1,"id":205,"obj":"label","parentid":200,"x":10,"y":10,"w":210,"h":20,"text":"Quick positions for COVER 1","align":"center"}

# this is a STOP button for COVER2 on the main page, long-press for the popup
      - obj: "p1b6"
        properties:
          "text_color": "{{ '#FFFF00' if is_state('cover.my_room_2', 'closing') else '#FFFFFF' }}"
          "text_opa": "{{ '80' if is_state_attr('cover.my_room_2','current_position', 0) else '255' }}"
        event:
          "down":
            - service: cover.close_cover
              target:
                entity_id: "cover.my_room_2"
          "hold":
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: jsonl
                parameters: >-
                  {"page":1,"id":200,"obj":"obj","x":5,"y":55,"w":230,"h":190,"shadow_opa":255,"shadow_color":"black","shadow_width":20,"shadow_spread":0}
                  {"page":1,"id":201,"obj":"btn","parentid":200,"x":10,"y":40,"w":150,"h":60,"toggle":false,"text_font":24,"text":"Small holes","tag":"cover.my_room_2"}
                  {"page":1,"id":202,"obj":"btn","parentid":200,"x":10,"y":110,"w":70,"h":60,"toggle":false,"text_font":24,"text":"1/3","tag":"cover.my_room_2"}
                  {"page":1,"id":203,"obj":"btn","parentid":200,"x":90,"y":110,"w":70,"h":60,"toggle":false,"text_font":24,"text":"3/4","tag":"cover.my_room_2"}
                  {"page":1,"id":204,"obj":"slider","parentid":200,"x":180,"y":40,"w":30,"h":130,"min":1,"max":100,"tag":"cover.my_room_2","val":
                  {{- state_attr('cover.my_room_2','current_position') -}}
                  }
                  {"page":1,"id":205,"obj":"label","parentid":200,"x":10,"y":10,"w":210,"h":20,"text":"Quick positions for COVER 2","align":"center"}

# this is a STOP button for COVER3 on the main page, long-press for the popup
      - obj: "p1b6"
        properties:
          "text_color": "{{ '#FFFF00' if is_state('cover.my_room_1', 'closing') else '#FFFFFF' }}"
          "text_opa": "{{ '80' if is_state_attr('cover.my_room_1','current_position', 0) else '255' }}"
        event:
          "down":
            - service: cover.close_cover
              target:
                entity_id: "cover.my_room_3"
          "hold":
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: jsonl
                parameters: >-
                  {"page":1,"id":200,"obj":"obj","x":5,"y":55,"w":230,"h":190,"shadow_opa":255,"shadow_color":"black","shadow_width":20,"shadow_spread":0}
                  {"page":1,"id":201,"obj":"btn","parentid":200,"x":10,"y":40,"w":150,"h":60,"toggle":false,"text_font":24,"text":"Small holes","tag":"cover.my_room_3"}
                  {"page":1,"id":202,"obj":"btn","parentid":200,"x":10,"y":110,"w":70,"h":60,"toggle":false,"text_font":24,"text":"1/3","tag":"cover.my_room_3"}
                  {"page":1,"id":203,"obj":"btn","parentid":200,"x":90,"y":110,"w":70,"h":60,"toggle":false,"text_font":24,"text":"3/4","tag":"cover.my_room_3"}
                  {"page":1,"id":204,"obj":"slider","parentid":200,"x":180,"y":40,"w":30,"h":130,"min":1,"max":100,"tag":"cover.my_room_3","val":
                  {{- state_attr('cover.my_room_3,'current_position') -}}
                  }
                  {"page":1,"id":205,"obj":"label","parentid":200,"x":10,"y":10,"w":210,"h":20,"text":"Quick positions for COVER 3","align":"center"}

# below the processor objects for the cover
      - obj: "p1b201" # Small holes
        event:
          "down":
            - service: cover.set_cover_position
              data:
                entity_id: "{{ tag }}"
                position: 19
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: p1b200.delete

      - obj: "p1b202" # 1/3
        event:
          "down":
            - service: cover.set_cover_position
              data:
                entity_id: "{{ tag }}"
                position: 48
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: p1b200.delete

      - obj: "p1b203" # 3/4
        event:
          "down":
            - service: cover.set_cover_position
              data:
                entity_id: "{{ tag }}"
                position: 90
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: p1b200.delete

      - obj: "p1b204" # slider
        event:
          "up":
            - service: cover.set_cover_position
              data:
                entity_id: "{{ tag }}"
                position: "{{ val | int }}"
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: p1b200.delete

      - obj: "p1b201" # Just close popup
        event:
          "down":
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_4
              data:
                keyword: p1b200.delete

Don't know if it's worth considering multiple tags. Like tag1, tag2, ... tagN...

I'm not sure about adding arbitrary attribute data, as this is going outside of the lvgl object model...

Would be very useful when integration with HA becomes more and more complex.

Another interesting note is that I have 5 plates operating on the same HA instance, with custom component. Each plate in a different room, showing 5-8 pages depending on location. 4 of them have weather, all of them have climate control + various switches, 3 of them have media player, some of them have convers control (total 10 covers, and don't have all of them in OpenHASP yet).
Load of the custom component with this config in HA takes 25.62 seconds. I have only implemented the above for one single cover but If I do it for all of them, that will increase boot time significantly...
Side note, my HA instance is running in a VM on a HP DL380 server with 16 Xeon cores and 128GB RAM...

It would be interesting to know why the components takes this long. It seems that there is a synchronous call, delay or time-out... I don't know.

I think it's due to the amount of mqtt messages being sent (1 per line)
A recent commit will provide a better performance

I think it's a bit of a hack/workaround, but what about moving most of the logic around controlling the covers into a script.
Then, you have something like this:
      - obj: "p1b6"
        properties:
          "text_color": "{{ '#FFFF00' if is_state('cover.my_room_1', 'closing') else '#FFFFFF' }}"
          "text_opa": "{{ '80' if is_state_attr('cover.my_room_1','current_position', 0) else '255' }}"
        event:
          "down":
            - service: script.position_all_covers
              data:
                oHaspEvent: "down"

Then the position_all_covers script is called with the argument "down". In the script you would look up the position(s) of each cover and then tell HA to adjust it's state +/- whatever a down event should increment by.

The goal is not to position all covers, just one.

The goal is not to position all covers, just one. You'd have the same quick position control, usable separately for each one.

I think i misunderstood? You want to dynamically create a few 'bookmarks' for fixed positions for a single cover?  You want to have a single set of the bookmarks but be able to choose which cover the bookmarks are for?
It's another hack but perhaps a dropdown listing all the covers. When you select a cover, the CC will adjust the 'selected' value of some input_select.  When a bookmark button is pressed, look up the current value of the input_select and use that to derive the entity ID that you feed into the entity_id for cover.close...

I guess the event can be appended with a tag... The data format depends on the schema validation in the CC
The tag could be a fixed tag key in the event with custom payload or custom key + value ...

+1 for fixed tag with payload as string, max 100 chars. One could put in a string whatever, json or alike.

The data format depends on the schema validation in the CC The tag could be a fixed tag key in the event with custom payload or custom key + value ...

it is trivial to disable validation and keep it treated as a 'raw' string rather than try to parse it as json/int...etc.
I keep going back and forth on weather or not it should be a small JSON object as this can make parsing inside of HA/Jinja2 templates a lot easier or just a freeform string which would require almost no changes to the CC but would increase the complexity of each HA/J2 template.

In the nightly build you can find that all objects have received a tag property.
It can contain custom JSON (or text) that will be passed along with each event.

MQTT payload is truncated:
[15:06:20.239][65524/98224 33][18980/19180  2] MQTT PUB: p1b201 => {"event":"down","tag":"cover.nappali_sz
[15:06:20.352][65524/97984 33][17296/19172 10] MQTT PUB: p1b201 => {"event":"up","tag":"cover.nappali_szom

also
2022-01-27 21:06:20 ERROR (MainThread) [custom_components.openhasp] Error decoding received JSON message: {"event":"down","tag":"cover.nappali_sz on hasp/plate_nappali/state/p2b201
2022-01-27 21:06:20 ERROR (MainThread) [custom_components.openhasp] Error decoding received JSON message: {"event":"up","tag":"cover.nappali_szom on hasp/plate_nappali/state/p2b201

The principle described in the OP works as long as the tags are short.

Thanks for the report @nagyrobi !
I found a few instances where the event buffer was indeed very short (40 or 100 characters total). It explains the 39 character limit you encountered for generic objects. This has now been adjusted to 512 characters for the whole event message (including the tag) across all event handlers.
I think that's a reasonable amount. Maybe we need to limit the tag length, so it will always fit.
But for now, go ahead and test again. Any tag upto 400 characters should be fine.
NB: you can set the tag to a JSON object like p1b7.tag={"mykey":"myvalue","test":true}
MQTT PUB: p1b7 => {"event":"down","val":1,"tag":{"mykey":"myvalue","test":true}}
MQTT PUB: p1b7 => {"event":"up","val":0,"tag":{"mykey":"myvalue","test":true}}

Thank you very much I think this will be enough for most use cases.
One suggestion:
Missing tags could be skipped from the payloads:
[06:30:13.507][65524/98096 33][16844/19188 13] MQTT PUB: p2b200 => {"event":"down","tag":null}
[06:30:23.964][65524/97868 33][18836/19076  2] MQTT PUB: p2b8 => {"event":"release","tag":null}
I think it's enough to only send the tag key when it contains data.
One could test against a valod tag key with a template like {% if value_json.tag is defined %}.

NB: you can set the tag to a JSON object like p1b7.tag={"mykey":"myvalue","test":true}

Tested with tags containing JSON objects, it's perfectly compatible with the CC out of the box (@dgomes):
      - obj: "p2b52"
        event:
          "hold":
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_teszt
              data:
                keyword: jsonl
                parameters: >-
                  {"page":2,"id":200,"obj":"obj","x":5,"y":55,"w":230,"h":190,"shadow_opa":255,"shadow_color":"black","shadow_width":20,"shadow_spread":0}
                  {"page":2,"id":201,"obj":"btn","parentid":200,"x":10,"y":40,"w":150,"h":60,"toggle":false,"text_font":24,"text":"Lyukacsos","tag":{"cover":"cover.konyha_v","position":"19"}}

      - obj: "p2b201" # redőny gyorspopup Lyukacsos
        event:
          "down":
            - service: cover.set_cover_position
              data:
                entity_id: "{{ tag.cover }}"
                position: "{{ tag.position }}"
            - service: openhasp.command
              target:
                entity_id: openhasp.plate_teszt
              data:
                keyword: p2b200.delete

This is ready to be documented ;) I'm running it in production.

 press color label >
 long-press Stop >

Made 2 examples for documentation:
 press color label >
 long-press Stop >

Ohh i really like the light toggle/color/long-press... that is a lot smarter than the interface that i was designing..!  please open a PR against the docs repo so I can borrow that implementation :D

As soon as @fvanroie decides to make a new dev brach on the docs repo, as this is not (yet) in the release.

There is a 0.6.3-dev docs branch. tag can go in 0.6.3 release.

Done:
https://haswitchplate.github.io/openHASP-docs/0.6.3/integrations/home-assistant/sampl_conf/#using-tags


Use default port for URLs

Adds support for database urls not containing an explicit port: postgres://user:password@localhost/database (#183).

These changes are now available in 2.0.0-rc.1.2

Thank you very much for the fast and good work!


When using AnimationNodeStateMachine as the root of AnimationTree, BlendTree and Animation nodes do not appear in AnimationTree Parameters

Godot version
4.0 alpha14
System information
Windows 10
Issue description

I have been trying to use a State Machine as the root node of my AnimationTree.
However, I noticed that after adding Animation and BlendTree nodes to my State Machine, they do not appear as Parameters - I am therefore unsure how to access their properties with code, since using the AnimationPlayer methods directly does not work when using the AnimationTree.
Steps to reproduce

Create new scene with AnimationPlayer and AnimationTree nodes as children of a node that does have animations (necessary to add animations to the State Machine)
Configure target of the AnimationTree to be the AnimationPlayer as usual.
Add one of each object (Blendspace1D, Blendspace2D, State Machine, BlendTree, Animation) to the State Machine root.
Note that neither BlendTree nor AnimationTree are present in the parameters.

Minimal reproduction project
AnimationTreeStateMachineRoot.zip

The included project did not contain any nodes in the BlendTree which had parameters, so it is expected behavior to have no parameters in the inspector for that BlendTree.
If I add a TimeScale node into the BlendTree, the properties indeed show up in the inspector:

Let me know if there is something I missed.

Pasting some discussion from Discord:

I was just looking for any means to modify Animation parameters from code when using the StateMachine root
Like playback speed and direction

Playback speed is controlled with a TimeScale node as I show in my screenshot
direction may be configurable with a negative timescale.

so I have to use a BlendTree?
Like if you have a StateMachine that just has the AnimationNodes, you cannot control playback speed?
They have to be within BlendTrees?

That's a good question. I believe yes, you need to make a simple BlendTree with a TimeScale node and the Animation node for each animation with a custom speed


Domain color-themes.com doesn't work anymore

I do not maintain it anymore, sorry. You can download an archive with all themes from the repository.


Stiky headers

Can headers become stiky ?

You can use this:
https://github.com/bgogetap/StickyHeaders

That would be out of the scope of this library, I'd recommend looking at other libraries to combine like @abramov73 suggested. Not sure which work well with this library.


perf: only attach lualine on LSP setup

Note that we are using lspsaga with lualine. This means it will be a bit more
expensive to load lspsaga with BufReadPost. We probably only need to use lualine
once LspAttach is called.
This makes it around 10ms faster startup time.
Signed-off-by: Aaron Pham<EMAIL_ADDRESS>

Either we can remove winbar from lualine but the windows will jitter a bit when LspStart, or keep it in current settings and a bit delay on startup.
Personally I prefer the later.

One downside is statusline won't be enabled until the user opens a filetype that has pre-configured server(s). (e.g., statusline won't be enabled in :enew, *.txt, *.log)

You are correct. I just feel like attaching lspsaga at startup is relatively heavy.

We can utilize exclude array to prevent lspsaga required by lualine.
https://github.com/ayamir/nvimdots/blob/78ab09e7121bdef3ae76212faa2b380a6cf17fec/lua/modules/configs/ui/lualine.lua#L16-L22
LspAttach is triggerd after lsp client attached to the buffer. So maybe we can set exclude dynamically by checking which lsp servers were installed. But there are no ways to get lsp servers from mason. We can just get all of packages including stylua which are not lsp servers.

require("mason-registry").get_installed_package_names()

Have a bit of time thinking about this. This means we are explicitly depends on mason. idk if we should do that. If you want to do I can go ahead and implement the logic.

require("mason-registry").get_installed_package_names()

Have a bit of time thinking about this. This means we are explicitly depends on mason. idk if we should do that. If you want to do I can go ahead and implement the logic.

IMO no need to do this.


minify_html = true causes Major page formatting issues to pages with numbered code blocks in Zola.

Bug Report
minify_html = true causes Major page formatting issues to pages with numbered code blocks.
This was first reported to the abridge theme: https://github.com/Jieiku/abridge/issues/84
I then posted this to the zola issue tracker: https://github.com/getzola/zola/issues/1942 (I am now posting it all here for completeness sake)
I implemented this in the after-dark theme and seen the same effect (pay particular attention to the footer at the bottom of the pages)

with minify_html = false it looks normal:

Environment
Zola version: 0.16.0
I also tested 0.15.3 with the same result.
Expected Behavior
The page should look the same with or without minify_html = true
Current Behavior
No Errors, just errors in page formatting when using minify_html = true
Step to reproduce
git clone https://github.com/Jieiku/zola-minify-test
cd zola-minify-test
zola serve

open this page: http://<IP_ADDRESS>:1111/overview-code-blocks/
Notice the page section headings and page footer are all broken following the numbered code blocks.
To further confirm the issue you can removed the line numbered code blocks, then minify_html = true will no longer cause an issue.

The generate html contains several instances of:

This attempts to close a span tag that was never opened. So html being fed into the minimizer is broken. Whatever the case for minify-html, that should be corrected.
Eliding all unneccessary details, a minimal test case looks something like this.

minify-html turns it into:

The rest of the page ends up being inside a pre tag which causes the weirdness.

So is this an issue with zola then?

Thanks for looking into this @winstonewert. It's not possible for minify-html to understand invalid HTML syntax by definition (nor is it its responsibility); @Jieiku if the HTML was generated, I'd suggest raising an issue with whatever tool generated the HTML.
With regards to the

 test case, the behaviour was changed in version 0.10.0 (not yet released) to simply drop the invalid closing tag instead of reinterpreting it as an opening tag, as it avoids an explosion in the tree if there are many (e.g. broken HTML generator/template). For reference, how minify-html handles invalid input is described in notes/Parsing.md as mentioned in the README.

Thanks @wilsonzlin , There is a ticket open at zola, hopefully somebody can figure out the cause.


Coverity Scan

You should use it to help find and fix defects in this driver. It might even help fix the BSOD that I have been having. Its website is at https://scan.coverity.com/.

got it. I will try it.
Thank you.


Implement BUIP055 anti replay feature

As per title and proposed spec. Can be modified is the spec ends up being different than what's proposed.

Added an empty commit for travis to kick in.

tx_clone.py test is failing
https://travis-ci.org/ftrader-bitcoinunlimited/BitcoinUnlimited/jobs/240065272#L7747


MPDD

Where can I download the MPDD dataset? Thank you.

Please visit the official website of MVTec with https://www.mvtec.com/company/research/datasets/mvtec-ad
or directly use this link
https://www.mydrive.ch/shares/38536/3830184030e49fe74747669442f0f282/download/420938113-1629952094/mvtec_anomaly_detection.tar.xz
to download the MVTec.

Please use this link to download the MPDD dataset
https://vutbr-my.sharepoint.com/:f:/g/personal/xjezek16_vutbr_cz/EhHS_ufVigxDo3MC6Lweau0BVMuoCmhMZj6ddamiQ7-FnA?e=oHKCxI

Thx!


bootstrap bash script proposal

Hello,
Please find attached a script to get from source and compile
the required dependencies so that odb.ml can be used.
Regards,
F.
--- bootstrap_odb.sh
#!/bin/bash
#set -x # debug
TODO
- odb.ml should have an md5 check also, for the moment I rewind to
a given tag, it seems that github forbids git archive to create
a .tar.gz of a given revision
command -v curl >/dev/null
if [ $? -eq 1 ]; then
echo "curl must be installed"
exit 1
fi
command -v git >/dev/null
if [ $? -eq 1 ]; then
echo "git must be installed"
exit 1
fi
mkdir -p bootstrap_odb
cd bootstrap_odb
here=pwd
get OCaml from sources, install it in ~/.odb
curl http://caml.inria.fr/pub/distrib/ocaml-3.12/ocaml-3.12.1.tar.bz2
-o ocaml-3.12.1.tar.bz2
echo "227a3daaedb150bf5037a3db01f5bf42  ocaml-3.12.1.tar.bz2" > md5sum
md5sum -c md5sum
if [ $? -eq 1 ] ; then
echo "problem with the downloaded ocaml"
exit 1
fi
tar xjf ocaml-3.12.1.tar.bz2
cd ocaml-3.12.1
./configure -prefix ~/.odb
make opt.opt && make install
cd $here
get odb and source its bashrc file for PATH and all
git clone --depth 1 git://github.com/thelema/odb.git
cd odb
git checkout 050def074a
source odb.bashrc
cd $here
get findlib, install it in ~/.odb
curl http://download.camlcity.org/download/findlib-1.2.7.tar.gz
-o findlib-1.2.7.tar.gz
echo "000bff723e8d3d727a7edd5b5901b540  findlib-1.2.7.tar.gz" > md5sum
md5sum -c md5sum
if [ $? -eq 1 ] ; then
echo "problem with the downloaded findlib"
exit 1
fi
tar xzf findlib-1.2.7.tar.gz
cd findlib-1.2.7
./configure -bindir  ~/.odb/bin
-mandir  ~/.odb/man
-sitelib ~/.odb/lib/ocaml/site-lib
-config  ~/.odb/etc/findlib.conf
make all && make opt && make -k install
cp src/findlib/topfind ~/.odb/lib/ocaml/
cd $here
get batteries using odb.ml (FINAL CHECK)
ocaml odb/odb.ml batteries

I thought I would be able to attach a file in the issue.
The script is here:
https://github.com/HappyCrow/bootstrap_odb

Maybe this could be combined with ocamlbrew https://github.com/hcarty/ocamlbrew


WIP: Remove default actix-web features

By default actix-web pulls in cookie which pulls in Ring which makes it unbuildable for non x86 or ARM architectures.
Since this package doesn't seem to have any need of any features beyond the Actix middleware interface I figured I could toss this upstream.
I'd still like to test it with code using the various default features to see if I'm accidentally breaking anything.

Great! Let me know when you are finished

Hey, @jkilpatr , how are we going? It is safe to merge it?

@svartalf  hey sorry for the delay, prod caught on fire and I got distracted with hot patches.
So I modified the .7 branch of https://github.com/actix/examples/tree/0.7/cookie-session
like so
`
//! Example of cookie based session
//! Session data is stored in cookie, it is limited to 4kb
//!
//! Redis session example
//!
//! User guide
extern crate actix;
extern crate actix_web;
extern crate actix_web_httpauth;
extern crate env_logger;
extern crate futures;
use actix_web::middleware::session::{self, RequestSession};
use actix_web::middleware::{Middleware, Started};
use actix_web::FromRequest;
use actix_web::{middleware, server, App, HttpRequest, Result};
use actix_web_httpauth::extractors::basic::{BasicAuth, Config};
use actix_web_httpauth::extractors::AuthenticationError;
use std::env;
struct Auth;
impl Middleware for Auth {
fn start(&self, req: &HttpRequest) -> Result {
let mut config = Config::default();
config.realm("WallyWorld");
let auth = BasicAuth::from_request(&req, &config)?;
    if auth.username() == "Aladdin" && auth.password() == Some("open sesame") {
        Ok(Started::Done)
    } else {
        Err(AuthenticationError::from(config).into())
    }
}

}
/// simple index handler with session
fn index(req: &HttpRequest) -> Result<&'static str> {
println!("{:?}", req);
// RequestSession trait is used for session access
let mut counter = 1;
if let Some(count) = req.session().get::<i32>("counter")? {
    println!("SESSION value: {}", count);
    counter = count + 1;
    req.session().set("counter", counter)?;
} else {
    req.session().set("counter", counter)?;
}

Ok("welcome!")

}
fn main() {
env::set_var("RUST_LOG", "actix_web=info");
env_logger::init();
let sys = actix::System::new("session-example");
server::new(|| {
    App::new()
        // enable logger
        .middleware(middleware::Logger::default())
        // cookie session middleware
        .middleware(session::SessionStorage::new(
            session::CookieSessionBackend::signed(&[0; 32]).secure(false),
        ))
        .middleware(Auth)
        .resource("/", |r| r.f(index))
})
.bind("<IP_ADDRESS>:8080")
.expect("Can not bind to <IP_ADDRESS>:8080")
.start();

println!("Starting http server: <IP_ADDRESS>:8080");
let _ = sys.run();

}
`
And it works just fine after changing the default imports.

@jkilpatr great to hear that! I suppose it would be hard to break anything with your patch, therefore I'll try to merge and publish it tomorrow (kinda busy today too :) )

Quick follow-up: 0.2.0 version


Support for Nuxt v2?

Description
Are you planning on adding support for Nuxt v2 or is there a chance to get it working on a legacy project?
I take that the underlying technologies changed significantly, just wondering if someone got to get it working.
Suggested solution
This is just a question.
Alternative
No response
Additional context
No response
Validations

[X] I agree to follow this project's Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Hi @doebi interesting quesiton. I would say if you use compatibility mode with composition API it should work. I haven't test it tho


Promote PodDisruptionBudget Version  to v1 for Adapter Kubernetes Version 1.25+

Since v1beta1/PodDisruptionBudget won't be supported in kubeVersion 1.25+, we need update the dependency version.
Scheduler Error Log:
[root@m-master1 ~]# kubectl logs po/hwameistor-scheduler-f4b5cb8dd-fb8st -n hwameistor
I1030 07:54:12.744726       1 shared_informer.go:230] Caches are synced for client-ca::kube-system::extension-apiserver-authentication::client-ca-file
I1030 07:54:12.744758       1 shared_informer.go:230] Caches are synced for client-ca::kube-system::extension-apiserver-authentication::requestheader-client-ca-file
I1030 07:54:12.745521       1 tlsconfig.go:178] loaded client CA [0/"client-ca::kube-system::extension-apiserver-authentication::client-ca-file,client-ca::kube-system::extension-apiserver-authentication::requestheader-client-ca-file"]: "kubernetes" [] validServingFor=[kubernetes] issuer="<self>" (2022-10-30 00:03:15 +0000 UTC to 2032-10-27 00:03:15 +0000 UTC (now=2022-10-30 07:54:12.745500531 +0000 UTC))
I1030 07:54:12.745559       1 tlsconfig.go:178] loaded client CA [1/"client-ca::kube-system::extension-apiserver-authentication::client-ca-file,client-ca::kube-system::extension-apiserver-authentication::requestheader-client-ca-file"]: "front-proxy-ca" [] validServingFor=[front-proxy-ca] issuer="<self>" (2022-10-30 00:03:16 +0000 UTC to 2032-10-27 00:03:16 +0000 UTC (now=2022-10-30 07:54:12.74555216 +0000 UTC))
I1030 07:54:12.745715       1 tlsconfig.go:200] loaded serving cert ["Generated self signed cert"]: "localhost@1667116449" [serving] validServingFor=[<IP_ADDRESS>,localhost,localhost] issuer="localhost-ca@1667116449" (2022-10-30 06:54:09 +0000 UTC to 2023-10-30 06:54:09 +0000 UTC (now=2022-10-30 07:54:12.745706543 +0000 UTC))
I1030 07:54:12.745826       1 named_certificates.go:53] loaded SNI cert [0/"self-signed loopback"]: "apiserver-loopback-client@1667116450" [serving] validServingFor=[apiserver-loopback-client] issuer="apiserver-loopback-client-ca@1667116450" (2022-10-30 06:54:09 +0000 UTC to 2023-10-30 06:54:09 +0000 UTC (now=2022-10-30 07:54:12.745817114 +0000 UTC))
E1030 07:54:13.678813       1 reflector.go:178] k8s.io/client-go/informers/factory.go:135: Failed to list *v1beta1.PodDisruptionBudget: the server could not find the requested resource
E1030 07:54:15.586564       1 reflector.go:178] k8s.io/client-go/informers/factory.go:135: Failed to list *v1beta1.PodDisruptionBudget: the server could not find the requested resource
E1030 07:54:21.469303       1 reflector.go:178] k8s.io/client-go/informers/factory.go:135: Failed to list *v1beta1.PodDisruptionBudget: the server could not find the requested

More info about PodDisruptionBudget, see https://github.com/kubernetes/kubernetes/pull/99290

can hwameiStor merge it with new version released soon ?


Fix: Fixed crash when launching from CMD with "Files"

Resolved / Related Issues

[X] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #13019

Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[X] Are there any other steps that were used to validate these changes?
Launch the app from CMD with "files C:". You will see the app launches successfully.

Great work


[ALERT]: Alert gets clipped on smaller screens

Description
The alert component does not resize appropriately to address smaller screens. I would be happy to help by opening a PR for this fix.
Component(s) impacted
Alert component

Browser
Chrome
Vivid Web Components version
v3.43.0
Severity
Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.
Application/website
Personal project
CodeSandbox example
https://codepen.io/christiansama/pen/xxBKQrV
Steps to reproduce the issue (if applicable)

Use the alert component with a long enough text and an action item.
Reduce the width of the viewport until the component gets clipped.

Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

Hi @ChristianSama , you are more then welcome to do so.
You can fine help here: CONTRIBUTING.md
Thanks you!

I noticed the --alert-min-inline-size css variable takes care of this. Setting it to auto makes the component resize appropriately. Quick question though, would it make sense to make auto the default value? Currently the default is 420px, but by setting it to auto it would responsive by default. WDYT?

basically it make sense, but there are 2 issues with it:

people who already use the component are expecting it to be 420px - changing it may break the design.
if there's a long text - the auto causes the alert to push the margin:

I think the fix here needs to be by setting style for mobile using media-query for `max-width: 768px;

Hi @ChristianSama we are very happy for your contribution!
However - opening a PR like you did is not the proper way.
We kindly ask you to follow our contribution page:
https://github.com/Vonage/vivid-3/blob/main/.github/CONTRIBUTING.md
We will close your PR, waiting to see your new PR :)
Thanks!

Hi, @rachelbt thank you for the comment! I believe I followed the steps in the contribution page. Could you please point out what is missing in my PR?

@ChristianSama
https://github.com/Vonage/vivid-3/pull/1529#issuecomment-1941272072

merged your PR to our repo.
Thanks for your contribution.


Feature Request: Foxy Gestures mousegesture support.

I left a request on Firefox's addon site for FoxyGesture support, which you kindly responded to inquire about API support from the FoxyGetures Dev.
I contacted the FG dev on their Git support page and they responded with the following

@somokol Can you bring me into your original conversation with the dev of temp containers? I don't think its necessary to have a whole system to register events. I think all that is necessary is for temp containers to add a listener using runtime.onMessageExternal. Then FG can invoke temp containers commands via a trivial user script.

Just thought I'd post that up here to see if this is feasible.

As mentioned in the FG Issue it's enough to implement a message-handler and write a small FG UserScript, no API needed. I'll look into it ASAP.

Just published version 0.39 with support for other WebExtensions. UserScripts that you could use in FoxyGestures are:
Open New Temporary Container Tab
const url = data.element && data.element.linkHref;
if (url) {
    browser.runtime.sendMessage('{c607c8df-14a7-4f28-894f-29e8722976af}', {
        method: 'createTabInTempContainer'
    });
}

Open Link in New Temporary Container Tab
const url = data.element && data.element.linkHref;
if (url) {
    browser.runtime.sendMessage('{c607c8df-14a7-4f28-894f-29e8722976af}', {
        method: 'createTabInTempContainer',
        active: true,
        url
    });
}

Open Link in New Background Temporary Container Tab
const url = data.element && data.element.linkHref;
if (url) {
    browser.runtime.sendMessage('{c607c8df-14a7-4f28-894f-29e8722976af}', {
        method: 'createTabInTempContainer',
        active: false,
        url
    });
}

Hope you like it. Let me know if something doesn't work as expected.

Wow! Thank You! So far, so good. It's working great on my FF 58.0.2.
Thanks again. This is such a perfect addition, to a perfect addon.


[Docs] COM-RPC tags

Move the tags documentation from ThunderTools repo into main documentation portal
Update to include latest tags

@sebaszm Made some minor tweaks to address comments.
If you could clarify the difference between the collapsed/extended compliant tags that would be appreciated.

@sebaszm Made some minor tweaks to address comments.
If you could clarify the difference between the collapsed/extended compliant tags that would be appreciated.

Thanks!
The snag is in the way how calls with a single parameter are defined. The JSON-RPC compliant way is to enclose them in an object, like params: { "name":  "abcd" }. Historically there was also possible to collapse it, like params: "abcd", since there is only one parameter it doesn't need a name to be accessed. So the collapsed format means that methods, notifications and properties omit the enclosing object when there is only one parameter to pass. The exteneded format is a mixture where the collapse is only used for properties, since they enforce one parameter anyway, but not for methods and notifications. And compliant format is that all calls including properties use the enclosing object. Collapsed and extended are not recommended for new interfaces.


Mscorlib not found

When starting server mscorlib not found exception is reported see the attached screenshot.

Which version of Windows / .net are you running?
Take a look here https://github.com/OmniSharp/omnisharp-server/blob/6785380ca55c4645b48b66d1c1b3466a0a5fcd5e/OmniSharp/Solution/AssemblySearch.cs#L8-L32

ver reports Microsoft Windows [Version 6.3.9600], it is Windows 8.1 Enterprise 32 bit.
Release in HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\NET Framework Setup\NDP\v4\Full has value 379893 meaning .NET Framework 4.5.2.
Thanks for pointing to the list, it is clear now. My mscorlib.dll is in C:\Windows\Microsoft.NET\Framework\v4.0.30319 probably because I do not have either SDK or Visual Studio installed. I assume it is a valid location. Will change, test and submit PR if it works. Very cool, thank you!

Works for me now (manually copied the bin content to C:\Users\david.podhola\.atom\packages\omnisharp-atom\node_modules\omnisharp-server-binaries\lib\server)

Not sure if this is related, but I'm getting a lot of issues on mscorlib with nuget packages.

You must add a reference to assembly 'mscorlib'

Specifics:
Class library project, .NET Framework 4.5, on a Win 7 Enterprise 64bit.
Atom 1.0.2, Omnisharp 0.20.
The folder suggested by @davidpodhola (.atom\packages\omnisharp-atom\node_modules\omnisharp-server-binaries\lib\server) does not seem to exist on my machine.
Any ideas?


Unit Tests for Eigen version changes

Hi !
I am wondering if you have a suite of unit tests for this library? Specifically, I am working with a codebase that uses different version of Eigen, and I would like to ensure that all functions continue to work as expected - and also to ensure compatibility as Eigen continues to change.

Hi, yes we have tests. They are found here: https://github.com/vioshyvo/RP-test
To use different version of Eigen in tests, you can tweak paths in the Makefile (https://github.com/vioshyvo/RP-test/blob/master/test/Makefile). It seems that the Makefile assumes Eigen is located at cpp/lib of the mrpt directory.
Please let us know if it works!

Fantastic, thanks for responding so quickly.
It does work.


Network viz properties for selected target

For currently selected target make the node bigger.

on track for completion today 11/14

Implemented.


Fixes and documentation to deploy the whole application in K8s

WIP, Open to run CI.
TO DO before the review :

[X] Apply changes from #15
[X] Pass the C.I

About this P.R :

[X] Remove hardcoded ips for services
[X] Clean Dockerfiles
[X] Add external connection string to MongoDB in dealer using Env var.
[X] Add feed database script for MongoDB
[X] Add PVC for the Cassandra Chart
[X] Add API Gateway Chart
[X] Set Always for pullpolicy
[X] Modify prometheus Chart to make it work with the app
[X] Add Script to deploy and run all the services on Azure/K8s

Feel free to review / merge if you want to.

@dtzar ready to be reviewed.
I focused this PR on the individual helm folder.
There is still improvement to do but at least it is a good start.
Thanks

@dtzar review applied.
Thanks


copied additional file for hive tables

@ibuenros
please review when you have time

Two small comments, LGTM otherwise.

resolved the conflict and merged the pull request


WIP - Add narrative documentation

Addresses #24

[x] Landing page
[x] Installation
[x] Tutorial - Ragna 101 (Python)
[x] Set config file
[ ] What is RAG? - WIP
[ ] Contribute - WIP
[x] FAQ
[x] And "Coming soon" message to other pages in the pipeline

Huh - I'm NOT closing these 😕
I think the GitHub is no longer recognizing my fork since we made the repo public.
I think this will keep happening - I'll open a fresh with a new branch in this repo.


Motion events not sent to MQTT for AD110

I have 2 AD110 doorbells that work great. They detect motion events via BlueIris and record perfectly. But for some reason, the motion event isn't sent to MQTT and I don't even have a "motion" topic in my MQTT broker:

I did see that when there is a motion trigger event, the "event" topic displays the following (in this order):
{"Code": "VideoMotion", "action": "Start"}
{"Code": "AlarmLocal", "action": "Start"}
{"Code": "AlarmLocal", "action": "Stop"}
{"Code": "VideoMotion", "action": "Stop"}

As far as I can tell, we can just keep an eye out for the VideoMotion key and use that to signal any motion. I think this line in the python script directly contradicts that theory, hence not sending the motion signal to the MQTT server: if (is_ad110 and code == "ProfileAlarmTransmit") or (code == "VideoMotion" and not is_ad110):.  I think we should check for both the keys ProfileAlarmTransmit OR VideoMotion and then send the "motion" signal to the MQTT server in order to solve issue #13 as well . Although I don't understand why the devices send different payloads when motion is detected, maybe my doorbell is setup in a different configuration?

So the AD110 has some weird behaviour compared to the other devices. It sends a VideoMoion event for any motion and a ProfileAlarmTransmit for motion inside your configured zones. Therefore we listen for the ProfileAlarmTransit event as that's what you'd generally expect to be considered 'motion'  Do you have motion zones set up in the app?
By the way all events are posted to the amcrest2mqtt/X/event topic so you can use that potentially if you need custom behaviour.

Oh, that is very interesting. No, I do not have specific motion zones setup in the app, I just use the entire frame as one motion zone.
I will have to take a look at that topic and potentially create a custom sensor in Home Assistant to mitigate this.
Thanks for the guidance!


Fer2013.csv dependency not found

When trying to run the examples/face_classification/pipelines.py or train.py, program cannot find fer2013.csv. After investigating, I have found that paz/datasets/ferplus relies on having a path to a dataset that is not contained within the repository. As such, I have started to look online for the ferplus2013 database, I have found the database online, but am unaware of how to allow PAZ to see the csv in question.

I am aware this is more requesting aid than reporting an issue; I apologize for the inconvenience.


works wrong for specific clickhouse type  LowCardinality(String)

2020-04-22 13:57:31,804:ERROR:root:No field class for LowCardinality(String)
Traceback (most recent call last):
File "/root/superset/venv/lib/python2.7/site-packages/superset/sql_lab.py", line 179, in execute_sql
**db_engine_spec.cursor_execute_kwargs)
File "/root/superset/venv/lib/python2.7/site-packages/sqlalchemy_clickhouse/connector.py", line 210, in execute
self._process_response(response)
File "/root/superset/venv/lib/python2.7/site-packages/sqlalchemy_clickhouse/connector.py", line 341, in _process_response
for r in response:
File "/root/superset/venv/lib/python2.7/site-packages/infi/clickhouse_orm/database.py", line 244, in select
model_class = model_class or ModelBase.create_ad_hoc_model(zip(field_names, field_types))
File "/root/superset/venv/lib/python2.7/site-packages/infi/clickhouse_orm/models.py", line 59, in create_ad_hoc_model
attrs[name] = cls.create_ad_hoc_field(db_type)
File "/root/superset/venv/lib/python2.7/site-packages/sqlalchemy_clickhouse/connector.py", line 75, in create_ad_hoc_field
inner_field = cls.create_ad_hoc_field(db_type[6 : -1])
File "/root/superset/venv/lib/python2.7/site-packages/sqlalchemy_clickhouse/connector.py", line 91, in create_ad_hoc_field
raise NotImplementedError('No field class for %s' % db_type)

@vavrusa

用最新版，直接下载master分支，python setup.py install
作者一直没有往pypi更新，pip安装的不是最新的

yup it worked for me also, copying in the latest master branch and replace the files in the pip package did the trick.. Thanks jackpgao.
@cloudflare, can you please update the pip package..

Sorry, I've been busy. I'll wait a bit for https://github.com/cloudflare/sqlalchemy-clickhouse/pull/47 and update pip package.

seems the #47 is done. Could we update the pip package?

For everyone waiting for pip package: pip install git+https://github.com/cloudflare/sqlalchemy-clickhouse.git  will work.

LowCardinality can have FixedString(N) too. Would it be safe to say if db_type contains FixedString then String?

For everyone waiting for pip package: pip install git+https://github.com/cloudflare/sqlalchemy-clickhouse.git  will work.

I have put git+https://github.com/cloudflare/sqlalchemy-clickhouse.git in my requirements.txt file but still getting this exception.
Can you help me?

I can see that the problem still exists for columns of type LowCardinality(Nullable(String)) using the code on mater right now.


Direnv extension loads /.envrc by default when VScode is opened without opening any folder.

Direnv extension loads /.envrc by default when VScode is opened without opening any folder. But the path is read-only in macOS. It should load the ~/.envrc by default? This issue doesn't affect the integrated terminal.

The extension uses the extension process's working directory by default, which happens to be $HOME on Linux but apparently not on macOS.
Could you try the build at https://github.com/direnv/direnv-vscode/actions/runs/6172954558?pr=606 see if that fixes it?
Thanks!

The extension uses the extension process's working directory by default, which happens to be $HOME on Linux but apparently not on macOS.
Could you try the build at https://github.com/direnv/direnv-vscode/actions/runs/6172954558?pr=606 see if that fixes it?
Thanks!

I think It solved my problem.

I seem to have screwed up the release, should be fixed in 0.15.2
Thanks for the notice!


[To rel/0.13] [IOTDB-2843] remove unused jna dependency in cli

Coverage increased (+0.04%) to 67.466% when pulling 1810d74a2838398891ad73e0478679ef9cae5da0 on remove_jna into ca5fb172c3bc710bdc35c6f0558c045c74005c35 on rel/0.13.


Misleading docs - oclazyload with Systemjs not working (maybe just a bug?)

Hi,
According to the docs  https://oclazyload.readme.io/docs/systemjs, Systemjs should works without any issues already but it looks like oclazyload is not loading new modules into angular structure. There is no error in console so Im not sure what happening. I created this repo as an example https://github.com/kresli/systemjs-oclazyload-seed-example. Even I can see that the modules went through the oclazyload, the run function in third module is not evaluated.
After cloned repo please run this commands:
npm install
./node_modules/jspm/jspm.js install
./node_modules/serve/bin/serve

Looks like this is issue with oclazyload version. As @scottmcnab  described here https://github.com/ocombe/ocLazyLoad/issues/221, degrading oclazyload to lower version from 1.0.4 to 1.0.3 cause config successfully initialised but run is still not invoked. However, degrading oclazyload to 1.0.1 version, everything works ok.

Same here, reverting to 1.0.1 works for me as well. However, this might not be related to SystemJS, see #221

Yes it's related to #221, closing this one as it is in fact a duplicate. Thanks for the report though


pull request for e2e test at 11/16/2016 03:45:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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


QPM gives different results from git clone

~/$ qpm install fr.grecko.sortfilterproxymodel
INFO: Package fr.grecko.sortfilterproxymodel has a different license (MIT) than it's dependant (NONE).
Installing<EMAIL_ADDRESS>~/$ ls vendor/fr/grecko/sortfilterproxymodel/
LICENSE				qqmlsortfilterproxymodel.cpp
SortFilterProxyModel.pri	qqmlsortfilterproxymodel.h
qpm.json

I suspect because this https://github.com/oKcerG/SortFilterProxyModel/blob/99d89d8cfb407e2ccf29f72871b1a20d5fd41b13/qpm.json#L13 is set to v.0.1.0, but I'm unfamiliar with qpm so this is just a guess.
P.S. Why am I just now finding this project? It's awesome, and I implemented a far inferior version for some auto-complete text boxes I made. Thanks a million!

Same observing today December 20, 2018
qpm version is 0.1.0, however all the information and git version is 0.2
Please update version in qpm


{Misc.} Fix #24749: Remove outdated argument from dev documentation

closed https://github.com/Azure/azure-cli/issues/24749
Related command
Developer Documentation
Description
Removal of outdated argument from developer documentation.
Testing Guide
none
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Misc

Thanks for the update :)


ref doc gen broken due to deprecation work

-Ref doc gen was broken by: https://github.com/Azure/azure-cli/pull/6472
-extension script uses latest released version of CLI, hence why this wasn't picked up until now.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/williexu/azure-cli-extensions.git@fixrefdoc#subdirectory=src/$EXT&egg=$EXT"

@tjprescott please take a look

Merging this to unblock CI, as this seems to fix the issue.

@tjprescott I believe the reason we have them separate is the two scripts are a little different and we don't ship the doc scripts with the CLI.
I'll open an issue to look into this.
We should also add a test to attempt generating refdocs from the latest version of dev in the CLI so we find instances like this before the CLI release. I'll open an issue for this as well.

I think the CLI does verify refdocs. That script was updated. Unless you mean also generating the extension docs.

Yep:
Issues I've opened:
https://github.com/Azure/azure-cli-extensions/issues/241
https://github.com/Azure/azure-cli-extensions/issues/242


Ramen end-to-end script fails to deploy open-cluster-management-hub components.

Invoking the Ramen end-to-end script on a bare metal machine fails to deploy open-cluster-management-hub components.
# Invoke the end-to-end script
$ hack/ocm-minikube-ramen.sh
:
+ kubectl --context hub apply -k /tmp/veera/open-cluster-management/registration/cluster-manager/hub
clustermanager.operator.open-cluster-management.io/cluster-manager created
+ date
Wed Mar 16 21:05:51 PDT 2022
+ kubectl --context hub -n open-cluster-management wait deployments/cluster-manager --for condition=available
deployment.apps/cluster-manager condition met
+ date
Wed Mar 16 21:06:05 PDT 2022
+ until_true_or_n 90 kubectl --context hub -n open-cluster-management-hub wait deployments --all --for condition=available --timeout 0
Wed Mar 16 21:06:05 PDT 2022
error: no matching resources found
error: no matching resources found
:
+ kubectl --context hub -n open-cluster-management-hub wait deployments --all --for condition=available --timeout 0
error: no matching resources found
+ set -- 1
+ trap - EXIT

# Listing namespaces on the hub shows that open-cluster-management-hub namespace doesn't exist.
$ kubectl --context hub get ns
NAME                      STATUS   AGE
default                   Active   39m
kube-node-lease           Active   39m
kube-public               Active   39m
kube-system               Active   39m
open-cluster-management   Active   39m

# But the cluster-manager deployment is ready in the open-cluster-management namespace
$ kubectl --context hub get deploy -nopen-cluster-management
NAME              READY   UP-TO-DATE   AVAILABLE   AGE
cluster-manager   1/1     1            1           9m25s

# However, cluster manager runs into errors shown below.
$ kubectl logs --context hub  -nopen-cluster-management deploy/cluster-manager
W0317 04:06:04.009450       1 cmd.go:213] Using insecure, self-signed certificates
I0317 04:06:04.236035       1 observer_polling.go:159] Starting file observer
W0317 04:06:04.262826       1 builder.go:230] unable to get owner reference (falling back to namespace): pods is forbidden: User "system:serviceaccount:open-cluster-management:cluster-manager" cannot list resource "pods" in API group "" in the namespace "open-cluster-management"
I0317 04:06:04.263793       1 builder.go:262] clustermanager version v0.0.0-unknown-7aa41cd
W0317 04:06:04.744866       1 secure_serving.go:69] Use of insecure cipher 'TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA256' detected.
W0317 04:06:04.744891       1 secure_serving.go:69] Use of insecure cipher 'TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256' detected.
W0317 04:06:04.745409       1 builder.go:321] unable to get cluster infrastructure status, using HA cluster values for leader election: infrastructures.config.openshift.io "cluster" is forbidden: User "system:serviceaccount:open-cluster-management:cluster-manager" cannot get resource "infrastructures" in API group "config.openshift.io" at the cluster scope
I0317 04:06:04.745820       1 leaderelection.go:248] attempting to acquire leader lease open-cluster-management/clustermanager-lock...
I0317 04:06:04.745963       1 event.go:285] Event(v1.ObjectReference{Kind:"Namespace", Namespace:"open-cluster-management", Name:"open-cluster-management", UID:"", APIVersion:"v1", ResourceVersion:"", FieldPath:""}): type: 'Warning' reason: 'ClusterInfrastructureStatus' unable to get cluster infrastructure status, using HA cluster values for leader election: infrastructures.config.openshift.io "cluster" is forbidden: User "system:serviceaccount:open-cluster-management:cluster-manager" cannot get resource "infrastructures" in API group "config.openshift.io" at the cluster scope
I0317 04:06:04.807824       1 requestheader_controller.go:169] Starting RequestHeaderAuthRequestController
I0317 04:06:04.807902       1 shared_informer.go:240] Waiting for caches to sync for RequestHeaderAuthRequestController
I0317 04:06:04.808151       1 configmap_cafile_content.go:201] "Starting controller" name="client-ca::kube-system::extension-apiserver-authentication::client-ca-file"
I0317 04:06:04.808189       1 shared_informer.go:240] Waiting for caches to sync for client-ca::kube-system::extension-apiserver-authentication::client-ca-file
I0317 04:06:04.808619       1 configmap_cafile_content.go:201] "Starting controller" name="client-ca::kube-system::extension-apiserver-authentication::requestheader-client-ca-file"
I0317 04:06:04.808805       1 shared_informer.go:240] Waiting for caches to sync for client-ca::kube-system::extension-apiserver-authentication::requestheader-client-ca-file
I0317 04:06:04.809423       1 dynamic_serving_content.go:131] "Starting controller" name="serving-cert::/tmp/serving-cert-3282788192/tls.crt::/tmp/serving-cert-3282788192/tls.key"
I0317 04:06:04.810881       1 secure_serving.go:266] Serving securely on [::]:8443
I0317 04:06:04.811390       1 tlsconfig.go:240] "Starting DynamicServingCertificateController"
E0317 04:06:04.814462       1 leaderelection.go:334] error initially creating leader election record: leases.coordination.k8s.io is forbidden: User "system:serviceaccount:open-cluster-management:cluster-manager" cannot create resource "leases" in API group "coordination.k8s.io" in the namespace "open-cluster-management"
I0317 04:06:04.909062       1 shared_informer.go:247] Caches are synced for client-ca::kube-system::extension-apiserver-authentication::requestheader-client-ca-file
I0317 04:06:04.909247       1 shared_informer.go:247] Caches are synced for client-ca::kube-system::extension-apiserver-authentication::client-ca-file
I0317 04:06:04.909354       1 shared_informer.go:247] Caches are synced for RequestHeaderAuthRequestController
E0317 04:06:46.367604       1 leaderelection.go:330] error retrieving resource lock open-cluster-management/clustermanager-lock: leases.coordination.k8s.io "clustermanager-lock" is forbidden: User "system:serviceaccount:open-cluster-management:cluster-manager" cannot get resource "leases" in API group "coordination.k8s.io" in the namespace "open-cluster-management"
E0317 04:07:35.887935       1 leaderelection.go:330] error retrieving resource lock open-cluster-management/clustermanager-lock: leases.coordination.k8s.io "clustermanager-lock" is forbidden: User "system:serviceaccount:open-cluster-management:cluster-manager" cannot get resource "leases" in API group "coordination.k8s.io" in the namespace "open-cluster-management"

# I'm not sure if these errors have anything to do with the failure to deploy open-cluster-management-hub components.

I'm addressing this by using the registration-operator image that matches the manifests.  Please retest with fix that closes this issue.

I reproduced this error using latest registration operator image, deleted the cluster-manager deployment, then redeployed it  using the image matching the manifests and verified that the open-cluster-management-hub deployments were all available:
+ until_true_or_n 90 kubectl --context hub -n open-cluster-management-hub wait deployments --all --for condition=available --timeout 0
Thu Mar 17 16:35:23 PDT 2022
...
deployment.apps/cluster-manager-placement-controller condition met
deployment.apps/cluster-manager-registration-controller condition met
deployment.apps/cluster-manager-registration-webhook condition met
deployment.apps/cluster-manager-work-webhook condition met
Thu Mar 17 16:36:06 PDT 2022

I am also facing the same problem while deploying ramen.
Tue Mar 22 18:16:46 IST 2022

until_true_or_n 90 kubectl --context hub -n open-cluster-management-hub
wait deployments --all --for condition=available --timeout 0
Tue Mar 22 18:16:46 IST 2022
error: no matching resources found
error: no matching resources found
error: no matching resources found
error: no matching resources found
error: no matching resources found
Could you please respond if there is a workaround (change of few lines ) to
fix in the script locally  and I can move ahead. I saw that there are a lot
of changes done in 408. Should we wait for 408 to merge?

Regards
Jolly
On Fri, Mar 18, 2022 at 9:25 PM hatfieldbrian @.***>
wrote:

I reproduced this error using latest registration operator image, deleted
the cluster-manager deployment, then redeployed it using the image
matching the manifests and verified that the open-cluster-management-hub
deployments were all available:

until_true_or_n 90 kubectl --context hub -n open-cluster-management-hub wait deployments --all --for condition=available --timeout 0
Thu Mar 17 16:35:23 PDT 2022
...
deployment.apps/cluster-manager-placement-controller condition met
deployment.apps/cluster-manager-registration-controller condition met
deployment.apps/cluster-manager-registration-webhook condition met
deployment.apps/cluster-manager-work-webhook condition met
Thu Mar 17 16:36:06 PDT 2022

—
Reply to this email directly, view it on GitHub
https://github.com/RamenDR/ramen/issues/407#issuecomment-1072545988, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AV2OSQBTV7BPIIOVF6K3E4DVASROBANCNFSM5Q52RGHQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Could you please respond if there is a workaround (change of few lines ) to fix in the script locally and I can move ahead. I saw that there are a lot of changes done in 408. Should we wait for 408 to merge?

To fix this issue, the registration-operator image's tag needs to be changed from:
quay.io/open-cluster-management/registration-operator:latest to
quay.io/open-cluster-management/registration-operator:0.1.0-c723e190c454110797d89e67bcb33250b35e1fd7
The registration_operator_kubectl routine does this by producing two kustomization.yaml files, one for hub and one for spokes.
Hub:
resources:
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/cluster-manager/config/crds/0000_01_operator.open-cluster-management.io_clustermanagers.crd.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/cluster-manager/config/rbac/cluster_role.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/cluster-manager/config/rbac/cluster_role_binding.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/cluster-manager/config/operator/namespace.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/cluster-manager/config/operator/service_account.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/cluster-manager/config/operator/operator.yaml
patchesJson6902:
  - target:
      group: apps
      version: v1
      kind: Deployment
      name: cluster-manager
      namespace: open-cluster-management
    patch: |-
      - op: test
        path: /spec/replicas
        value: 3
      - op: replace
        path: /spec/replicas
        value: 1
      - op: test
        path: /spec/template/spec/containers/0/image
        value: quay.io/open-cluster-management/registration-operator:latest
      - op: replace
        path: /spec/template/spec/containers/0/image
        value: quay.io/open-cluster-management/registration-operator:0.1.0-c723e190c454110797d89e67bcb33250b35e1fd7

Spoke:
resources:
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/klusterlet/config/crds/0000_00_operator.open-cluster-management.io_klusterlets.crd.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/klusterlet/config/rbac/cluster_role.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/klusterlet/config/rbac/cluster_role_binding.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/klusterlet/config/operator/namespace.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/klusterlet/config/operator/service_account.yaml
  - https://raw.githubusercontent.com/stolostron/registration-operator/c723e190c454110797d89e67bcb33250b35e1fd7/deploy/klusterlet/config/operator/operator.yaml
patchesJson6902:
  - target:
      group: apps
      version: v1
      kind: Deployment
      name: klusterlet
      namespace: open-cluster-management
    patch: |-
      - op: test
        path: /spec/replicas
        value: 3
      - op: replace
        path: /spec/replicas
        value: 1
      - op: test
        path: /spec/template/spec/containers/0/image
        value: quay.io/open-cluster-management/registration-operator:latest
      - op: replace
        path: /spec/template/spec/containers/0/image
        value: quay.io/open-cluster-management/registration-operator:0.1.0-c723e190c454110797d89e67bcb33250b35e1fd7

To apply one of the above kustomizations:
kubectl --context <cluster name of hub or spoke>  apply -k <directory pathname of kustomization.yaml file>

By default there is one hub cluster named hub and two spoke clusters named hub and cluster1.
You could do this manually, but I recommend instead using the ocm-minikube.sh file contained in this pull request.  It may not be merged, but it has been tested.

I found that deleting the cluster-manager deployment was needed for the different container image to be deployed:

I reproduced this error using latest registration operator image, deleted the cluster-manager deployment, then redeployed it using the image matching the manifests and verified that the open-cluster-management-hub deployments were all available

I did this by issuing the following command:
kubectl --context hub -nopen-cluster-management delete deploy/cluster-manager

For example:
$ kubectl --context hub -nopen-cluster-management delete deploy/cluster-manager
deployment.apps "cluster-manager" deleted

@jmishrarh if you can still reproduce this issue feel free to report here any issues or success with fixing it

@jmishrarh

Based on the failure point, the registration-operator deployment that was previously unavailable became available meaning this issue tracked here has been fixed.
The issue encountered is that the hub cluster, which is also a managed cluster, waited 1 minute for its managed cluster subscription deployment, multicluster-operators-subscription-mc, to become available and it never did.  Have you tried running the script again or even just the command that failed to see if it just needed to wait longer?  E.g.:

$ kubectl --context hub -nmulticluster-operators wait --for condition=available --timeout 0 deploy --all
deployment.apps/multicluster-operators-application condition met
deployment.apps/multicluster-operators-subscription condition met
deployment.apps/multicluster-operators-subscription-mc condition met

The above command worked today.  I am facing another issue. Initially I got an error that osd0-cluster1 already exist. So I manually deleted the image. After that when I ran the script I found this error.

virsh vol-info --pool minikube osd0-cluster1
error: failed to get vol 'osd0-cluster1'
error: Storage volume not found: no storage vol with matching path 'osd0-cluster1'

virsh vol-create-as --pool minikube --name osd0-cluster1 --capacity 32G --format qcow2
Vol osd0-cluster1 created

virsh domblkinfo --domain cluster1 --device vdb
error: failed to get domain 'cluster1'

sudo virsh attach-disk --domain cluster1 --source /home/jmishra-vm/.minikube/images/osd0-cluster1 --target vdb --persistent --driver qemu --subdriver qcow2 --targetbus virtio
error: Failed to attach disk
error: Requested operation is not valid: target vdb already exists

@jmishrarh This new issue is tracked by #414.  I wonder how the previous subscription operator deployment error was resolved.  Did it succeed after more time elapsed or was something else done?

Thanks Brian. It did succeed after multiple re-try over a period of time.


fix issue #49

https://github.com/groovyfx-project/groovyfx/issues/49

Fixed with https://github.com/groovyfx-project/groovyfx/pull/56


UndoLogParser change to SPI.

Ⅰ. Describe what this PR did

UndoLogParser change to SPI mode and make fastjson as default implement.
Add config transaction.undo.log.serialization
Fix one typo.
Add test cases.

Ⅱ. Does this pull request fix one issue?
Fix #1010

Codecov Report

Merging #1099 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             develop    #1099      +/-   ##
=============================================
+ Coverage      41.84%   41.84%   +<.01%
  Complexity      1361     1361
=============================================
  Files            243      243
  Lines          10111    10113       +2
  Branches        1319     1319
=============================================
+ Hits            4231     4232       +1
  Misses          5334     5334
- Partials         546      547       +1

Impacted Files
Coverage Δ
Complexity Δ

...ata/rm/datasource/undo/JSONBasedUndoLogParser.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...ava/io/seata/core/constants/ConfigurationKeys.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...seata/rm/datasource/undo/UndoLogParserFactory.java
80% <100%> (+13.33%)
1 <0> (ø)
:arrow_down:

...seata/rm/datasource/undo/AbstractUndoExecutor.java
83.15% <100%> (ø)
20 <0> (ø)
:arrow_down:

...server/store/file/FileTransactionStoreManager.java
45.99% <0%> (-0.35%)
19% <0%> (ø)

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 abe4dec...458681c. Read the comment docs.


Month value deviates by -1

(Zotero 7 API)
{{ date.month }} seems to be zero-based, meaning the correct value is yielded only via {{ date.month + 1 }}. The day and year values are not zero-based.

@dstillman is this a bug or should we account for this behavior in Zotero 7 local API client applications?

What API are you referring to? JS Date month is 0-based.

You are right, sorry for disturbance, issue is in this code.
Should be fixed in 1.6.4


Revert working patterns functionality as it is broken on Production.

Migrate working pattern script breaks on production due to incorrect format of starts_on date.
This was introduced alongside the Working patterns changes in #926.
By reverting the working patterns functionality, we intend to bring production to a working state.
We recreated the broken working patterns column locally, and this branch fixes the issue.
Next steps:

[ ]  Run the reindex vacancies rake task

Update
We noticed an issue where any vacancies created between deploy time and now would cause further errors down the line.
We decided to fix the record in production with the incorrect year.
Fortunately this was the only record affected, and the migrate working patterns completed successfully.


Operator () and Original do not work with non-positional T2-T3

Currently these functions only check T3 == T in their constraint, instead of checking T3 == T | T2 == T

Resolved with https://github.com/SubstituteR/MoarPtr/commit/76f2d873ac69176a9c8bc451d6195a19d879da95


dacfx on Windows Server Core 2019?

Is it possible to install dacfx on Windows Server Core 2019?  If so, how?

@rschiefer -- Robb, thanks for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please let us know which topic you’re referencing, or provide more context around your general task.

https://docs.microsoft.com/en-us/sql/tools/release-notes-sqlpackage?view=sql-server-ver15

@rschiefer -- Robb, thank you for that information.  You may find answers here:

DBA Stack Exchange
Stack Overflow

@pensivebrian -- Brian, please look into this issue #4249.
cc @stevestein as a writer

I was able to finally install dacfx on windows server core 2019. I was apparently calling msiexec incorrectly.
It would be nice if the page I linked to advove listed server 2019 core as a supported os.
Thanks!
Get Outlook for Androidhttps://aka.ms/ghei36

From: William Anton Rohm<EMAIL_ADDRESS>Sent: Wednesday, March 4, 2020 12:10:03 PM
To: MicrosoftDocs/sql-docs<EMAIL_ADDRESS>Cc: Robb Schiefer<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/sql-docs] dacfx on Windows Server Core 2019? (#4249)
@rschieferhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frschiefer&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057522056&sdata=QXPwGDSa90eazr5PgcAegkn8NX%2BzFTVCsiG%2B8iyfeqQ%3D&reserved=0 -- Robb, thank you for that information. You may find answers here:

DBA Stack Exchangehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdba.stackexchange.com%2Fquestions%2Ftagged%2Fsql-server&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057522056&sdata=l4LQtX4pDDMoja7NLQuoIAtKdoRpMEbAJbrCMeHiQL0%3D&reserved=0
Stack Overflowhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fstackoverflow.com%2Fquestions%2Ftagged%2Fsql-server&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057532069&sdata=sxM2SWUrmQnGguWQwGaDOQszYdrBrmkBlIqJGqfNj0g%3D&reserved=0

@pensivebrianhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpensivebrian&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057542077&sdata=%2FjQ3AiknaNaPoffWoVTlXwllnnHdDs%2BYYHOPHFOugKA%3D&reserved=0 -- Brian, please look into this issue #4249https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsql-docs%2Fissues%2F4249&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057542077&sdata=1gmgbMi5DEby2y6vwZVy%2BWU4u9OkIAO%2B0UGRo0ewblE%3D&reserved=0.
cc @stevesteinhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fstevestein&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057552086&sdata=vjQ%2FdK3EXXTMtWYunYouj1KVWwAekSmPL16j52%2F94S4%3D&reserved=0 as a writer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsql-docs%2Fissues%2F4249%3Femail_source%3Dnotifications%26email_token%3DAAIFVEJCO5LWXJ4J37FZ353RF2KPXA5CNFSM4K6AC63KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENZKTZY%23issuecomment-594717159&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057552086&sdata=M4HzIBTb7a5LqYdO5kF2rWIOFLR0Mdrp2S74mqlbkqo%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAIFVEJRIMTFZWNX2UZ4QE3RF2KPXANCNFSM4K6AC63A&data=02|01||97c0dd9228134b1a1a9408d7c0674416|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637189422057562090&sdata=81nzBBVq5tMSU%2BxSIHR4iws43rPkBidg18ii5%2BJiOA0%3D&reserved=0.

@rschiefer - thanks for the feedback.
Was installing any different than installing on full Windows? If there are no differences then we typically don't call this out.
Thanks

It was different in that it you had to use the commandline (no wizard) and with all the limitations of Server Core it was unclear if it would still work correctly.

@rschiefer - thanks for the details. We added Server Core to the list of supported OSs in the article.
Thanks!
#please-close

thx!


GeoIP2 support

Would you provide support for GeoIP2?
I'm using maxmind GeoIP2 (mmdb) and geoip doesn't support it.

Enabled on master. If you have both installed it will use GeoIp2 by default.
I need help to fill it out:
        return [
            'latitude' => $this->geoIpData->location->latitude,
            'longitude' => $this->geoIpData->location->longitude,
            'country_code' => $this->geoIpData->country->isoCode,
            'country_code3' => null,
            'country_name' => $this->geoIpData->country->name,
            'region' => $this->geoIpData->continent->code,
            'city' => $this->geoIpData->city->name,
            'postal_code' => $this->geoIpData->postal->code,
            'area_code' => null,
            'dma_code' => null,
            'metro_code' => $this->geoIpData->location->metroCode,
            'continent_code' => $this->geoIpData->continent->code,
        ];

And we might need to change the database structure to add some more.


fix: fix volume size smaller than bi size issue

ref: https://github.com/longhorn/longhorn/issues/10017
we added a new webhook rule which rejects creating a volume which size is smaller than the backing image
update the volume size with larger size.

Summary by CodeRabbit

Bug Fixes

Updated volume size constants for integration tests, affecting volume creation and management.
Adjusted volume expansion sizes in recurring job tests, impacting snapshot creation and retention.

Tests

Enhanced test suite for recurring jobs to reflect updated volume sizes, ensuring accurate validation of snapshot behaviors.

Test

-vs test_backing_image.py: https://ci.longhorn.io/job/private/job/longhorn-tests-regression/7848/console
whole: https://ci.longhorn.io/job/private/job/longhorn-tests-regression/7847/console

@mergify backport v1.8.x


[Question] Async upload and additional metadata

Hello guys,
First of all congratulations for the amazing tool that you managed to create!
I have the following situation:
My web site has an add/update item form which contains few text fields and I'm using Filepond for images which can be attached to that item. I'm using the async upload of images and everything is perfect. Also I'm implementing a scenario where the user can select an image and mark it as a default/primary image (for now when the user clicks on image I'm adding a red border to that image). When I submit the form (with POST) the images ids are send to the back-end and I can process them,in my MVC controller, if I need to. But I also need to know which image was selected as primary. Currently the back-end receives just the images ids:

Is it possible to embrace the metadata somehow or to use some mix between the async and file encode plugin (without having the entire file encoded in base64)?
And vice versa when I load preloaded files from the server as local (when the user wants to update an item) how can I return some metadata (e.g. the file is marked as primary/default) to the front-end, so I can mark this image with a red border?

Technology used: Asp.Net MVC Core 3.1
Thank you in advance!

Thanks!
FilePond will upload both the file and the metadata to the server.process end point.
I suggest uploading the selected value separately.
You can set metadata to the each file in the files prop.
https://pqina.nl/filepond/docs/patterns/api/filepond-object/#setting-initial-files

@rikschennink you are right that FilePond will upload them during the server.process but in my case the server.process uploads the files to the server and the server saves them as temp files, after this the server returns an ID as a response for each file (i.e. some-guid). The form is still not submitted. After this the user may decide to select one of the images to be primary/default (clicks on some image) and when submit the form I don't know how to pass this information - the selected image metadata back to the server in addition to my other data (form fields), because in my backend I'm receiving only the hidden inputs (image ids) but no any metadata.

@KrisPetkov Add another field where you store the selection?

@rikschennink currently nowhere that's why I was wondering if it is possible to store the selection inside the concrete file's metadata and to pass the file's metadata to the server on form submit instead just the file ID. But if it is not possible I guess I will store the selection in a  separate field

That's currently not possible, you'd have to sync the values yourself using onupdatefiles

Thank you for the information! I will close the issue.