Getting data from redis getting slowly when redis-server disconnect

Redis-server disconnect when Redis-client not send any data during 300s
Redis-py-cluster get data raise ConnectionError, setting try_random_node is True.
continue while, get node is possibly not key slot node, raise MovedError,
setting refresh_table_asap = True.
next, get data to need refresh table asap time-consuming and
node-connect the only release is not disconnected
I think that when raising ConnectionError:

disconnect
get other connection from the current node
if rasing ConnectionError when getting connection is new connection(make_connection), else ConnectionError

@clusterdown_wrapper
def execute_command(self, *args, **kwargs):
    """
    Send a command to a node in the cluster
    """
    if not args:
        raise RedisClusterException("Unable to determine command to use")

    command = args[0]

    # If set externally we must update it before calling any commands
    if self.refresh_table_asap:
        self.connection_pool.nodes.initialize()
        self.refresh_table_asap = False

    node = self.determine_node(*args, **kwargs)
    if node:
        return self._execute_command_on_nodes(node, *args, **kwargs)

    redirect_addr = None
    asking = False

    try_random_node = False
    slot = self._determine_slot(*args)
    ttl = int(self.RedisClusterRequestTTL)

    while ttl > 0:
        ttl -= 1

        if asking:
            node = self.connection_pool.nodes.nodes[redirect_addr]
            r = self.connection_pool.get_connection_by_node(node)
        elif try_random_node:
            r = self.connection_pool.get_random_connection()
            try_random_node = False
        else:
            if self.refresh_table_asap:
                # MOVED
                node = self.connection_pool.get_master_node_by_slot(slot)
            else:
                node = self.connection_pool.get_node_by_slot(slot)
            r = self.connection_pool.get_connection_by_node(node)

        try:
            if asking:
                r.send_command('ASKING')
                self.parse_response(r, "ASKING", **kwargs)
                asking = False

            r.send_command(*args)
            return self.parse_response(r, command, **kwargs)
        except (RedisClusterException, BusyLoadingError):
            raise
        except (ConnectionError, TimeoutError):
            try_random_node = True

            if ttl < self.RedisClusterRequestTTL / 2:
                time.sleep(0.1)
        except ClusterDownError as e:
            self.connection_pool.disconnect()
            self.connection_pool.reset()
            self.refresh_table_asap = True

            raise e
        except MovedError as e:
            # Reinitialize on ever x number of MovedError.
            # This counter will increase faster when the same client object
            # is shared between multiple threads. To reduce the frequency you
            # can set the variable 'reinitialize_steps' in the constructor.
            self.refresh_table_asap = True
            self.connection_pool.nodes.increment_reinitialize_counter()

            node = self.connection_pool.nodes.set_node(e.host, e.port, server_type='master')
            self.connection_pool.nodes.slots[e.slot_id][0] = node
        except TryAgainError as e:
            if ttl < self.RedisClusterRequestTTL / 2:
                time.sleep(0.05)
        except AskError as e:
            redirect_addr, asking = "{0}:{1}".format(e.host, e.port), True
        finally:
            self.connection_pool.release(r)

    raise ClusterError('TTL exhausted.')

@yangjiaronga I do not understand your bug report.  You need to re-format this issue with more exact instructions on how to reproduce this error you are seeing becuase you are just quoting most of the code without any steps for me to reproduce the error. I need to see your client code and what scenario of the cluster you are using/building etc.

Thanks a lot for the reply.
Redis-server config
<IP_ADDRESS>:6379> config get timeout
1) "timeout"
2) "300"

If there are more than 300 seconds between two requests, a second request is triggered ConnectionError
I do not understand setting try_random_node  = True when raising  ConnectionError
except (ConnectionError, TimeoutError):
    try_random_node = True

    if ttl < self.RedisClusterRequestTTL / 2:
        time.sleep(0.1)
except ClusterDownError as e:
    self.connection_pool.disconnect()
    self.connection_pool.reset()
    self.refresh_table_asap = True

    raise e

I think the code should look like this
except ConnectionError:
    # try_random_node = True
    r.disconnect()
except TimeoutError:
    # try_random_node = True
    if ttl < self.RedisClusterRequestTTL / 2:
        time.sleep(0.1)

because set try_random_node = True，exclude r = self.connection_pool.get_random_connection()
Might trigger MovedError
if asking:
    node = self.connection_pool.nodes.nodes[redirect_addr]
    r = self.connection_pool.get_connection_by_node(node)
elif try_random_node:
    r = self.connection_pool.get_random_connection()
    try_random_node = False

finally, I don't want a ConnectionError to cause a MovedError

One other ting. You say that you do not want a MovedError to be triggered on a ConnectionError pathway, can you shoow with a example where you would get a MovedError raised up to your side of the code? or would this only be internally within the client lib that a MovedError would be raised? becuase most likley that MovedError would recorrect itself anyway in the next loop run of the main attempt loop.

I agree with trying a random node that this case and exception kinda assumes that whats happen is that one node just fallen off for some reason and that your cluster basically lost one node.
There are two possible reasons for triggering a connection error, lost one node(master node down) and node initiative disconnect.  But you've unified the two cases, I think it's wrong.
in my test script:
if run sleep(1) and then print(rc.get('foo')) again and time that call,  i get 0.001 time response.
I have no problem closing the connection when there is a connection error, But a TTL error is thrown, we be thinking about how to solve the problem of TTL error, I think you can avoid TTL errors by handling the two cases of connection errors
this only be internally within the client lib that a MovedError would be raised
This is how I found the problem
We switched from redis singleton to redis cluster in production environment
Peak redis cluster performance is 10% slower than redis single instance performance，about qps = 5W/s
Redis clusters are 5-10 times slower than redis singletons during low peaks, about qps = 30/s，Sometimes very quickly
Then I looked at the redis cluster source code
I found the reason for the slowness，Whenever running this code, It takes between 50 and 300 milliseconds
if self.refresh_table_asap:
    self.connection_pool.nodes.initialize()
    self.refresh_table_asap = False

And then I look up where setting self.refresh_table_asap = True
ClusterDownError and MovedError
I am sure my cluster is not down and no exceptions were caught, so is MovedError
except ClusterDownError as e:
    self.connection_pool.disconnect()
    self.connection_pool.reset()
    self.refresh_table_asap = True
    raise e
except MovedError as e:
    self.refresh_table_asap = True
    self.connection_pool.nodes.increment_reinitialize_counter()
    node = self.connection_pool.nodes.set_node(e.host, e.port, server_type='master')
    self.connection_pool.nodes.slots[e.slot_id][0] = node

So what do I find that causes a MovedError
This code can get the correct node, self.connection_pool.get_node_by_slot(slot)
This code will get a random node, which may trigger a MovedError self.connection_pool.get_random_connection()
Because I found the ConnectionError, because it will set try_random_node = True
<IP_ADDRESS>:6379> config get timeout
1) "timeout"
2) "300"

Too few requests during the low peak period will result in no requests within 300s, leading to active disconnection of redis-server

@yangjiaronga I think this has now been fixed in the following commit c62012c7bfaf3ffe4f199de39085fe355fcc5737 where the first proposed change was implemented. If it is possible, could you try the latest version of master or a commit that is after the mentioned commit to see if you still see the pattern or the problem now after this?

@yangjiaronga I did attempt to run your example script that has the timeout set on the node to 20 seconds in my case and this is the following result that i got from running the script
(redis) ➜  redis-py-cluster git:(master) ✗ python a.py
0.00032639503479003906
bar
0.002065896987915039

Yes there is a slowdown when running the second pass of x10, but that is more in line of an acceptable performance compared to the numbers you had where you had almost an x1000 increase in latency/lagg when the cluster was disconnected.
Right now i will consider this issue resolved after the tests i have done for now. If you see that this error still remains in some way/shape/form then please either comment here and we can discuss it (please do not reopen it), or please open a new issue if there is other details or other scenarios where this happens.


RST documentation throwing a warning

Review: By commit
Merge strategy: Merge (no squash)

Description
The RST documentation is starting up with an error about two virtual methods not being defined.
Instead of declaring them as empty methods in the *.h file, this is now done in the *.cpp file where method documentation strings can be added.
Verification
Did a clean build and all tests passed.  Documentation clean build completed without any warnings now.
Documentation
None
Future work
None

These documentation warnings are getting fixed through #273 , so we might not need to merge this PR.

Additionally, this might be a bit of bikeshedding, but I would say that it's preferable to keep documentation to header files. These represent the "outward" facing side of our code, so I think it makes sense to write user-oriented information (the documentation) where users naturally go to learn about how to use a file (declarations in header files). Comments / documentation in .cpp files should, imo, be reserved for implementation details that users don't need to know. Idk if @patkenneally would agree with me on this.

I had those two virtual functions with comments in the *.h file but doxygen was still throwing error warnings.  Not sure why.  Having them documented in the .cpp solved the problem.
Regarding documenting in *.h or *.cpp, this is a matter I leave up to the module developer.  I don't see a need to impose one coding preference here.

thanks for the feedback Pat.  But, I'm not about to modify the oxygen and sphinx documentation tools.  We've been working around some odd behaviors for years.  There are only so many battles I can fight in a week :-)


[Date] onChange event is not working properly

When I use the onChange event with direct keyboard input in the text field the onChange event is called with the new date. When I choose a date with the date picker, the event parameter is pushed with the old value. I have to choose again the same date to have the new value.

Which version are you using ?
For your issue description, please follow the guidelines pushed by @Tommass to make it easier for us to treat it.

I think your issue is a duplicate, and has been fixed here and delivered there.
I'm waiting for you confirmation before closing it.

Yep you're right :metal:


No extension description in found in archive error

Hi,
I keep getting this message when trying to add your extension to slicer3d. Could you please tell me how to proceed?

For reference I am on MacOS Monterrey, using an M1 chip and Slicer 4.11
Many thanks

Thanks for the report.
Based on the screenshot, it seems that you downloaded the source by clicking on "Download Zip" directly available in the GitHub interface.
Since the extension is expected to be available through the extension manager, could you download a recent version of Slicer available at https://download.slicer.org and try again.
If you still have issues, consider opening a new  issue.


observe the authority address balance on block submission

This PR removes one of the custom bots from devops
Overview
On block submission, this process checks the balance of the authority address, logs and send a telemetry event.
Changes

add a process with async balance check
test that simple process

Testing
mix test

Coverage decreased (-11.5%) to 75.35% when pulling ccbaa4cc71c989f0a75a50faf9d3672b19a0e60c on inomurko/observe_balance into 047d9d327047a95242d3b3b6fea0d0aa6ab1c323 on master.


Dave allured format compat on reopen

This is @Dave-Allured 's PR #2176 but with the recently-added OSX checks in place, as a proof of concept.

Sorting out why the OSX-specific github actions aren't running.


Notebook not working

Thanks for your work!
I find this file freq_model can not work properly, will you be so kind to fix it?
It seems that some scripts or imports are missing!

It's seems that you need to provide some detials
for example, which line can not work well.


Printing Payload value in the Log generated

Can you please help with how we can enable printing payload value in the events sent/received when generating the error trace.
This would be helpful to know by just looking at the log what messages are going around.
Currently, you print the Event name, instead, can you call the toString() function which by default returns the name of the event but we can then override this function if we want to print something else along with event name.
Thanks!

Hey @ankushdesai, we actually do not automatically print a payload value for an event, because the event can contain any arbitrary custom user data. We let the user to customize the log (besides the stuff that we automatically log already) in whatever way they want.
For example, one thing you could do to print the event payloads, is to instrument your own calls (in the generated machines) to the runtime logger (which is a property of the machine, this.Logger or this.Runtime.Logger), and call WriteLine on it, and then whatever you log there will become part of the trace, if that makes sense? You could either do that in the machine actions, or could even use one of our machine callbacks for this purpose (e.g. by overriding protected virtual Task OnEventDequeueAsync(Event e), which is called whenever an event is dequeued). Hope this helps!


sendEvent() does not check for error responses

When a request is made to the Pagerduty API with the Client.prototype.events.sendEvent method, and the request fails, at least in some cases, the method still returns a promise that fulfills, seeming to indicate success. In one instance, I sent a badly formed event and pagerduty sent back a 400 response indicating the error. In my opinion, the sendEvent method should have given me a Promise that rejected with some kind of error indicating the response, so I would know that my event had not been accepted.
There is a workaround, but it requires the calling code to inspect every response on their own, which is undesirable, imo.
I don't know if this is pervasive through out the package, but I would imagine it is. I also don't know what other errors it is masking in a similar way (e.g., if 5xx series responses would be treated any differently).
As a possible solution, if you use a library such as axios to make the requests, it will treat an error-indicating status codes as errors and return a promise that rejects.

Any feedback on this @kmart2234 ?

I plan to replace the request module with axios in version 2 of the module.

I see. Thanks


improve component styles

[ ] For each component put styles created with styled in a separate file (style.ts)
[ ] Add common styles in MUI theme
[ ] Improve BasicModal component
[ ] Add block comments

Follow up in #1168


Generate correct code for parameter components

The current code for parameter components fails because it can't generate a source file path for embedding. Also, it results in duplicate schema generation if there are references to schemas from the parameters and can have name conflicts between these schemas.
Instead, generate parameter classes only in cases where a nested schema needs to be generated and nest the schema inside that class. This also addresses the source file name problem since there is a type name to use as the basis for the file name.
Relates to #239

/merge


Client WebSockets Should Use Channels

Operating System: all
Node Version: all
NPM Version: all
webpack Version: 4.6.0
webpack-hot-client Version: 2.2.2

[ ] This is a bug
[ ] This is a feature request
[x] This is a modification request

Code
  // webpack.config.js
n/a

Expected Behavior
n/a
Actual Behavior
n/a
For Bugs; How can we reproduce the behavior?
n/a
For Features; What is the motivation and/or use-case for the feature?
Client WebSockets that originate from hot-client should use channels to specify that they're intended for HMR purposes and are not third-party sockets. Log message should be updated to include the channel. Broadcast should be updated to be able to target a channel.

Taking a further look at this, there's really no need due to the message envelope type property.


Integration fails to load. IMS 0.1.22

Error #1:
Logger: homeassistant.components.sensor
Source: helpers/entity_platform.py:360
Integration: Sensor (documentation, issues)
First occurred: 20:38:59 (1 occurrences)
Last logged: 20:38:59
Error while setting up ims platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 360, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/ims/sensor.py", line 283, in async_setup_entry
for condition in conditions:
TypeError: 'NoneType' object is not iterable
Error #2:
This error originated from a custom integration.
Logger: custom_components.ims
Source: custom_components/ims/init.py:157
Integration: The Israel Meteorological Service integration (documentation, issues)
First occurred: 20:38:59 (1 occurrences)
Last logged: 20:38:59
Key monitored_conditions is missing from config_entry.data

HA OS
Core 2024.1.3
Supervisor 2023.12.1
Operating System 11.4
Frontend 20240104.0
IMS 0.1.22

I also have the same problem

I'll try to fix it quickly, but as a quick workaround, remove the component and re-add it (no need for installation)

If it helps to investigate the issue, I missed update 0.1.21, so I jumped from version 0.1.20 to 0.1.22.

Other ption is downgrade to 0.1.21 working version

Hi,
Was the problem resolved in 0.1.23?

@vadimnazaroff I believe so


Make sort key optional, support getItem, deleteItem and exists operations with partition key only

Description

Makes sort key name optional in TableIndex when specifying TableConfig. This supports table with only partition key configured.
getItem, deleteItem and exists now supports operations with only partition key or both keys.

Signature for getItem, deleteItem and exists has changed, now they accepts key: DocumentClient.Key instead of pk and sk as separate arguments.
async getItem<T>( key: DocumentClient.Key,  fields?: Array<keyof T>): Promise<T>

example:
client.getItem({ id: 'book-123' });
client.getItem({ pk: 'library#books', sk: 'book-123' }, ['id', 'isActive']);

Related Issues

Issue: #3
How Has This Been Tested

Unit tests
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #4 into master will increase coverage by 0.13%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   93.22%   93.36%   +0.13%
==========================================
  Files          13       13
  Lines         192      196       +4
  Branches       50       52       +2
==========================================
+ Hits          179      183       +4
  Misses         12       12
  Partials        1        1

Impacted Files
Coverage Δ

src/DynamoHelper.ts
0.00% <0.00%> (ø)

src/mutation/deleteItem.ts
100.00% <100.00%> (ø)

src/query/exists.ts
100.00% <100.00%> (ø)

src/query/getItem.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Still error: ‘class v8::Maybe’ has no member named ‘ToChecked’ with nan 2.14.1

I'm trying to make node-osmium work on node 12 and 14. That I'm somewhere with, but now the build breaks on node 6:
../src/buffer_wrap.cpp: In static member function ‘static Nan::NAN_METHOD_RETURN_TYPE node_osmium::BufferWrap::filter_point_in_time(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/buffer_wrap.cpp:138:146: error: ‘class v8::Maybe<double>’ has no member named ‘ToChecked’
  138 | tic_cast<int32_t>(v8::Date::Cast(*info[0])->NumberValue(info.GetIsolate()->GetCurrentContext()).ToChecked() / 1000));
      |                                                                                ^~~~~~~~~

buffer_wrap.cpp#L138:
    NAN_METHOD(BufferWrap::filter_point_in_time) {
        INSTANCE_CHECK(BufferWrap, "Buffer", "filter_point_in_time");
        if (info.Length() != 1) {
            Nan::ThrowTypeError(Nan::New("please provide a point in time as first and only argument").ToLocalChecked());
            return;
        }

        osmium::Timestamp point_in_time;
        if (info[0]->IsInt32()) {
            point_in_time = Nan::To<int32_t>(info[0]).ToChecked();
        } else if (info[0]->IsString()) {
            Nan::Utf8String time_string { info[0] };
            point_in_time = osmium::Timestamp(*time_string);
        } else if (info[0]->IsDate()) {
            point_in_time = osmium::Timestamp(static_cast<int32_t>(v8::Date::Cast(*info[0])->NumberValue(info.GetIsolate()->GetCurrentContext()).ToChecked() / 1000));
        }

I've upgraded to nan 2.14.1 to ensure #852 is applied, but I'm still getting similar errors to #851 when building with Node 6.

@matburnham This is happening because they use the V8 methods, not the nan ones, so there’s nothing we can do about it on nan’s side. Can you report this to the node-osmium project instead?

Oh, duh. Of course it isn't anything to do with nan.


x/website: double "instead"s in go1.21.html

In https://go.dev/doc/go1.21, under "Tools":

instead of attempting to build the code and instead printing errors about unresolved imports or syntax errors.

This would read better as "instead of attempting to build the code and then printing errors about unresolved imports or syntax errors."

Thanks, sent https://go.dev/cl/517217.


docs(sdk): Update README.md for the wallet-sdk-gomobile

During the installation of go, I was stuck a bit because gomobile didn't worked.
This issue helped me.
I think it's worth modifying the README.

DCO signed, ready to build again

title updated, ready to build again

Hi @raphaelguye, thanks for finding this issue and for making a pull request!
I'd like to suggest that instead of adding export PATH="$PATH:$HOME/go/bin" to the The gomobile tools section, instead what you can do is move the existing $GOPATH/bin added to your path bullet point above the The gomobile tools point.
The reasons why are because:

The line you added is specific to Unix - it wouldn't work as expected on a Windows machine.
While the GOPATH environment variable defaults to $HOME/go, it could be different on someone else's machine.
The export command is just one way of adding GOPATH to your path, and once the terminal window is closed it'll be lost. Rather than picking a specific method of adding GOPATH to the path, we can simply tell the user to add it to their path and leave it up to them to choose how they want to do that.

By placing $GOPATH/bin added to your path above the The gomobile tools section, that tells the user that it's required before they can run those commands, which should help people in the future not run into the same issue you had, and the wording is agnostic to the user's OS + GOPATH env. var.
Let me know what you think :)

Closing this as there is no response.


Added schema constructs in JSON

Added allOf, oneOf and anyOf schema constructs in sample.
Provided is the description and screenshots for the same
We are using OAS 3,0 for creating service app json and invoking the same in Invoke json app by overriding the service endpoints

duplicate  of https://github.com/TIBCOSoftware/tci-flogo/pull/206


⚠ Bofrost.be has degraded performance

In 31fadf7, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 905 ms

Resolved: Bofrost.be performance has improved in 5fefba3.


Custom validation UNIQ new admin code & error handling

From #615
Major:

Added validation to new admin code so it wouldn't accept an already existing one

Minor:

Wrapped the labels in a span.field-name so WET validation would throw a more meaningful error (using the name of the field in the error description)

Plz review & squash+merge


Minor Capital Battles - Map Markers Not Disappearing (or hiding other Citiies)

Fixed in https://github.com/schofida/CWORR/commit/9865b0237b4a9bce1c2b1c8960eedf64cb338657


Metric aggregation not working

ENV

ELK 7.9
ElastAlert : current latest version (install from git clone)

My rule
name: HIGH MEMORY USAGE
type: metric_aggregation
index: monitor-*
metric_agg_key: system.memory.used.pct
metric_agg_type: avg
query_key: agent.name
doc_type: _doc
bucket_interval:
  seconds: 300
sync_bucket_interval: true
min_threshold: 0.1
#max_threshold: 0.05
filter:
- term:
    metricset.name: memory
alert:
- debug
- telegram

alert_text_type: alert_text_only
alert_text: "
Memory Used : {0:.2%}\n
Host : {1}\n
Timestamp : {2}"
alert_text_args: ["system.memory.used.pct","agent.name","@timestamp"]

# from botfather
telegram_bot_token: "xxxx"
telegram_room_id: "xxxx"

My document sample
{
  "_index": "monitor-viettel-2020.09",
  "_type": "_doc",
  "_id": "F_PJR3QBEw-zIe_T12WU",
  "_version": 1,
  "_score": null,
  "_source": {
    "tags": [
      "beats_input_raw_event"
    ],
    "service": {
      "type": "system"
    },
    "event": {
      "module": "system",
      "duration": 1188334,
      "dataset": "system.memory"
    },
    "metricset": {
      "period": 10000,
      "name": "memory"
    },
    "type": "monitor",
    "agent": {
      "id": "de1fb802-6fcc-4195-a06e-439c072d6a4c",
      "type": "metricbeat",
      "hostname": "proxy01",
      "version": "7.7.1",
      "ephemeral_id": "1d78bb7e-ca25-4bbf-bbbd-167d651da51c"
    },
    "host": {
      "name": "proxy01"
    },
    "system": {
      "memory": {
        "hugepages": {
          "surplus": 0,
          "total": 0,
          "used": {
            "bytes": 0,
            "pct": 0
          },
          "swap": {
            "out": {
              "fallback": 0,
              "pages": 0
            }
          },
          "default_size": 2097152,
          "free": 0,
          "reserved": 0
        },
        "actual": {
          "used": {
            "bytes": 908177408,
            "pct": 0.0545
          },
          "free":<PHONE_NUMBER>6
        },
        "page_stats": {
          "pgscan_kswapd": {
            "pages": 0
          },
          "pgsteal_direct": {
            "pages": 0
          },
          "pgscan_direct": {
            "pages": 0
          },
          "pgsteal_kswapd": {
            "pages": 0
          },
          "pgfree": {
            "pages": 52353231
          }
        },
        "total":<PHONE_NUMBER>4,
        "used": {
          "bytes":<PHONE_NUMBER>,
          "pct": 0.0893
        },
        "swap": {
          "out": {
            "pages": 0
          },
          "in": {
            "pages": 0
          },
          "total":<PHONE_NUMBER>,
          "used": {
            "bytes": 0,
            "pct": 0
          },
          "free":<PHONE_NUMBER>,
          "readahead": {
            "cached": 0,
            "pages": 0
          }
        },
        "free":<PHONE_NUMBER>6
      }
    },
    "ecs": {
      "version": "1.5.0"
    },
    "@timestamp": "2020-09-01T03:52:27.884Z",
    "tenant": "viettel",
    "@version": "1"
  },
  "fields": {
    "@timestamp": [
      "2020-09-01T03:52:27.884Z"
    ]
  },
  "highlight": {
    "metricset.name": [
      "@kibana-highlighted-field@memory@/kibana-highlighted-field@"
    ]
  },
  "sort": [
<PHONE_NUMBER>884
  ]
}

Run test
Command
python3 -m elastalert.elastalert --verbose --rule rules/VGCA_MONITOR-High-Memory.py

Result:
INFO:elastalert:Starting up
INFO:elastalert:Disabled rules are: []
INFO:elastalert:Sleeping for 59.99992 seconds
/usr/local/lib/python3.6/site-packages/urllib3-1.25.10-py3.6.egg/urllib3/connectionpool.py:988: InsecureRequestWarning: Unverified HTTPS request is being made to host 'datalake04'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
  InsecureRequestWarning,
/usr/local/lib/python3.6/site-packages/urllib3-1.25.10-py3.6.egg/urllib3/connectionpool.py:988: InsecureRequestWarning: Unverified HTTPS request is being made to host 'datalake04'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
  InsecureRequestWarning,
INFO:elastalert:Ran HIGH MEMORY USAGE from 2020-09-01 10:25 +07 to 2020-09-01 10:33 +07: 0 query hits (0 already seen), 0 matches, 0 alerts sent

I try to use .keyword but it still not work.

agent.name is string
system.memory.used.pct is number float

Try with Rule type ANY, have query hit , matches and alert sent (worked)
Thank

Have anyone got same issue with me ? Have another way to work arround this issue?
Thank so much!

Have anyone got same issue with me ? Have another way to work arround this issue?
Thank so much!

seems like it got bugged here https://github.com/Yelp/elastalert/blob/master/elastalert/elastalert.py#L896-L905 , change your run_every to something bigger than buffer_time.


Fix Windows launcher

Update of #485
TLDR:

assembly-ed Mill version workssssss! as far as i can tell 🎉
dev.launcher works in batch mode! In -i mode I'm getting scala.reflect.internal.MissingRequirementError: object scala in compiler mirror not found...

NOTE 1 (this is relevant only for dev.launcher mode):
As we all know, Windows has command-line limit of ~8000 characters.
Essentially there are 2 parts of fix:

Pass long props via file.
Pass classpath via "pathing jar" (see here)

Most people still use Java 8, as far as I know (me included).
So we shouldn't rely on Windows people to use Java9+, as is the current case (prependShellScript adding -XX:VMOptionsFile, see #405 among others..).

NOTE 2: Once you do -i it stays during that shell session (SETLOCAL wasn't used), so if you run mill after that it will still use interactive mode... This PR fixes that also.
@lefou WDYT?

@sake92 Is this ready to review? If so I can take a look

@lihaoyi yes please!

@sake92 this looks good to me I think.
Before I merge, could you flesh out the PR description a bit to summarize (1) exactly which user-facing problems you are fixing, (2) what the root cause of those problems is, (3) how your PR solves those root causes and (4) how you tested it to verify your PR solves the things you think it solves? I'm still a bit fuzzy on the different problems with dev, assembly, dev.assembly, dev.launcher, etc. and which ones I should expect to behave differently due to this PR. This will also help future maintenance.
Once that's done I'm happy to merge!

@lihaoyi I updated the description, hope it's clearer now.
In summary:

prod version will see almost no changes, except it'll work on Windows now also
dev.launcher will use mill.properties instead of -DMILL_PROP properties.
And on Windows it'll use "pathingJar" instead of "normal classpath"

This looks good, thanks for the detailed PR description!


change folder name

Fix #11
Check carrefully that I didn't forgot something while updating the name. normally vscode does a good job at tracking this kind of modifications.

change the name of the folder
update the extention name in the readme (in the extention list)
update the name of the package in setup.cfg

Thank you! Shouldn't we also update the Sphinx configs in the tests folder to use sphinx_favicon?
If I see it correctly, this will be a breaking change because people already using this extension will need to update their conf file after this update, right?

If I see it correctly, this will be a breaking change because people already using this extension will need to update their conf file after this update, right?

yes, that's the major issue but there is no real workaround

I dropped 3.6 tests as it's not a supported version of Python in Github actions


i18n(fr): update ui.ts

Description (required)

Translates studioHeading.label
Related issues & labels (optional)

Closes N/A
Suggested label: i18n

Just saw #7316...


report error for TypeError: Object of type set is not JSON serializable

This happens in commands/report.py, line 343, json.dumps

Fix by wrapping cidrs[cidr] in line 654 of audit.py in list()


Thank you for signing the Anthem Opensource Software CLA.


Motor Town config

Seems there is a motor town server config on a fork, that could be pulled in.
https://github.com/catb0t/PowerShellGSM/commit/521f3da8dda9ec2825e3b620b601923d5d291a5c

I don't like the script calling a batch file instead of having the content of that batch being integrated with the powershell.


creating a drive item with random item id results in 201 response.

Describe the bug
Giving any random string ( other than an empty string) as a remote item ID for creating a drive item gives a 201 status code. The response will be empty though.
Steps to reproduce

login to ocis as user user.
go to shares and make API post request to create a drive item with some string.

curl -X POST 'https://host.docker.internal:9200/graph/v1beta1/drives/{share-Space-Id}/root/children'
-d '{
    "remoteItem": {
        "id":"'123"
    }
}' -uadmin:admin

The status code is 201.

It only checks for valid sharespace id and if remoteitem has empty string or not.
What should the expected behavior and status code be?

I think this should return a 400 (Bad Request)

Also, once the sharer send the request for creating drive item and if the shares is unmounted, API call to this endpoint change the @client.synchronize property of the driveItem  to true and the body returns drive item.  If multiple API call is done to same endpoint, the status code will still be 201 though the body remain empty. Is this the expected behavior?


Add Department

Add Department
User Story
AS A business owner
I WANT to be able to view and manage the departments, roles, and employees in my company
SO THAT I can organize and plan my business
Acceptance Criteria
GIVEN a command-line application that accepts user input
WHEN I choose to add a department
THEN I am prompted to enter the name of the department and that department is added to the database

This has been completed

This has been completed


Unable to deploy FIori app to SAP backend System - unable to get local issuer certificate

Hi,
I have created a Fiori app using the Visual Studio FIori extensions, the preview works fine. However while deploying, I get the certificate issue
ERR! builder:custom deploy-to-abap Deployment Failed.
ERR! builder:custom deploy-to-abap unable to get local issuer certificate
I have mentioned some params in the yaml files to ignore SSL certificate, but it does not work

ui5.yaml
configuration:
ignoreCertError: true # If set to true, certificate errors will be ignored. E.g. self-signed certificates will be accepted

ui5-deploy.yaml
configuration:
target:
url: <server_url>
client: "111"
auth: basic
useStrictSSL: false
allowSelfSigned: true

The SAP server is a development server. How do we get visual studio to ignore the certificate issue.
Thanks

Hi @theblock-dev
This extension is to edit ABAP code, has nothing to do with Fiori or UI5 extensions
I'm not sure what extension is responsible, as I do pretty much no fiori development myself.
Best regards
Marcello


cmd/go: build, vet, fmt, test print path format ./example.go:1:1:

create a file main.go
package main
k
func main(){}

Then try the following
% go build
./main.go:2:1: syntax error: non-declaration statement outside function body
% go vet
vet: ./main.go:2:1: expected declaration, found k
% go test
./main.go:2:1: syntax error: non-declaration statement outside function body

So far so good everything consistent except for the vet: thingy in the beginning of the vet output instead of ./main.go:2:1: vet: expected declaration, found k but I let that slight for now :) Most important for vscode and my terminal is path consistency namely every path has at least one / in it
Now do go test again but rename the file to main_test.go
% go test
main_test.go:2:1: expected declaration, found k

Here we break the internet because vscode and terminals can't figure out that it's a path because ./ is stripped so you can't click on it anymore to open the src file in that location.
(ref: https://groups.google.com/forum/#!topic/golang-dev/ZbTYwDSlGvQ)

I think this is a duplicate of #24344. Note that that was a while ago, probably before go test started running vet. Still, it seems like the same issue to me.

OP replied on that previous issue, so I'll assume he agrees this is a duplicate :)


Load cursor directly from file instead of using a temporary buffer

Hi there,
xcursor 0.3.7 was just released. It adds a new function that allows reading a cursor data directly from a Read + Seek. The previous API always required a byte slice.
No PR since I am too lazy to fork, but basically I am talking about:
diff --git a/wayrs-utils/Cargo.toml b/wayrs-utils/Cargo.toml
index db321f5..7695623 100644
--- a/wayrs-utils/Cargo.toml
+++ b/wayrs-utils/Cargo.toml
@@ -24,7 +24,7 @@ shmemfdrs2 = { version = "1.0", optional = true }
 thiserror = "1.0"
 wayrs-client = { version = "1.1", path = "../wayrs-client" }
 wayrs-protocols = { version = "0.14", path = "../wayrs-protocols", optional = true }
-xcursor = { version = "0.3", optional = true }
+xcursor = { version = "0.3.7", optional = true }
 xkbcommon = { version = "0.8", optional = true }

 [package.metadata.docs.rs]
diff --git a/wayrs-utils/src/cursor.rs b/wayrs-utils/src/cursor.rs
index fe65127..b4a465c 100644
--- a/wayrs-utils/src/cursor.rs
+++ b/wayrs-utils/src/cursor.rs
@@ -24,7 +24,7 @@ use wayrs_client::Connection;

 use crate::shm_alloc::{BufferSpec, ShmAlloc};

-use xcursor::parser::{parse_xcursor, Image};
+use xcursor::parser::{parse_xcursor_stream, Image};

 use wayrs_protocols::cursor_shape_v1::*;
 pub use wp_cursor_shape_device_v1::Shape as CursorShape;
@@ -120,8 +120,11 @@ impl CursorTheme {
                     .or_else(|| theme.load_icon("default"))
                     .ok_or(CursorError::DefaultCursorNotFound)?;

-                let mut imgs = parse_xcursor(&std::fs::read(theme_path)?)
-                    .ok_or(CursorError::ThemeParseError)?;
+                let mut imgs = parse_xcursor_stream(&mut std::io::BufReader::new(
+                    std::fs::File::open(theme_path)?,
+                ))
+                .ok()
+                .ok_or(CursorError::ThemeParseError)?;
                 if imgs.is_empty() {
                     return Err(CursorError::DefaultCursorNotFound);
                 }

Dunno if this change really has any benefits. Feel free to just close this ticket. Also, this new API returns a std::io::Result instead of the Option that parse_xcursor() returns. In the patch above I ignore this and use .ok() to get rid of any error information. Feel free to come up with something better.
Alternatively, it's also possible to do this:
diff --git a/wayrs-utils/src/cursor.rs b/wayrs-utils/src/cursor.rs
index fe65127..882be95 100644
--- a/wayrs-utils/src/cursor.rs
+++ b/wayrs-utils/src/cursor.rs
@@ -24,7 +24,7 @@ use wayrs_client::Connection;

 use crate::shm_alloc::{BufferSpec, ShmAlloc};

-use xcursor::parser::{parse_xcursor, Image};
+use xcursor::parser::{parse_xcursor_stream, Image};

 use wayrs_protocols::cursor_shape_v1::*;
 pub use wp_cursor_shape_device_v1::Shape as CursorShape;
@@ -33,8 +33,6 @@ pub use wp_cursor_shape_device_v1::Shape as CursorShape;
 pub enum CursorError {
     #[error("default cursor not found")]
     DefaultCursorNotFound,
-    #[error("theme could not be parsed")]
-    ThemeParseError,
     #[error(transparent)]
     ReadError(#[from] io::Error),
 }
@@ -120,8 +118,9 @@ impl CursorTheme {
                     .or_else(|| theme.load_icon("default"))
                     .ok_or(CursorError::DefaultCursorNotFound)?;

-                let mut imgs = parse_xcursor(&std::fs::read(theme_path)?)
-                    .ok_or(CursorError::ThemeParseError)?;
+                let mut imgs = parse_xcursor_stream(&mut std::io::BufReader::new(
+                    std::fs::File::open(theme_path)?,
+                ))?;
                 if imgs.is_empty() {
                     return Err(CursorError::DefaultCursorNotFound);
                 }

(FYI: The old None return from parse_xcursor() is turned into an IO-Error with ErrorKind::other. So if you want, you could also do some matching on that to prevent any externally visible changes in behaviour.)

Also not sure if this has any actual advantages. Memory mapping the file and treating it as a byte slice may be even better. Though again, most of the cursors on my system are 69k, and it really doesn't matter.

b622ef2


Add remove method

Codecov Report

Merging #15 (0329275) into master (96d8ea5) will increase coverage by 0.30%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   94.65%   94.96%   +0.30%
==========================================
  Files           1        1
  Lines         131      139       +8
==========================================
+ Hits          124      132       +8
  Misses          7        7

Flag
Coverage Δ

unittests
94.96% <100.00%> (+0.30%)
:arrow_up:

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

Impacted Files
Coverage Δ

basedir.lua
94.96% <100.00%> (+0.30%)
:arrow_up:

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 96d8ea5...0329275. Read the comment docs.


Is GetOptionsObject really required?

Sure, it's an unobservable operation and can be avoided in implementations, but it makes the spec more difficult and can be replaced with 1 undefined check in one method and 2 in another.

It's a method used in a number of other specs, including 402 and Temporal. It could be removed throughout, but I'm prioritizing consistency for the moment.
Incidentally if you're implementing this you may want to hold off. It's only stage 2 and the API is likely to change before stage 3.

Incidentally if you're implementing this you may want to hold off. It's only stage 2 and the API is likely to change before stage 3.

Yes, I'm implementing it and understand that it's on stage 2 and most likely will be changed, but it's a polyfill for familiarization and experiments with this future API, not for production. Now early stage APIs are significantly fenced off from stable in core-js and they will be fenced off even more in the future to prevent possible errors associated with this.

Consistency and avoiding future behavior drift is much more important than spec complexity imo.

Consistency

So, it's inconsistent with current ES spec cases - for example, Error options (with cause) case.

cause is a uniquely special case that is intentionally different from everything else, because you can have a nullish cause of an exception.

It is about options argument case and definitely not about cause.
Anyway, I don't care - GetOptionsObject looks like a good internal abstraction, but it seems similar spec cases should be refactored for consistency -)

I'm sure they will once the AO first lands in 262, but it's not there yet.

it seems similar spec cases should be refactored for consistency

Yeah, things are in a kind of strange state right now because 402, Temporal, and lots of other proposals use options bags, and have mostly copy-pasted that AO around, but the only thing in 262 which uses it is Error, and Error is weird for other reasons. The trouble with Error, if I recall correctly, is that some engines have a non-standard extension which allows specifying a line number or file name or something in the same position as the spec now uses for the options argument, so it would probably not be web-compatible to throw an error when the user passes a number or string in that position. So Error needs to silently ignore non-object values passed in that position.
But new things (e.g. this API) don't have that legacy constraint, so they can throw if you pass a non-object value instead of silently ignoring it, and so all new things use (or at least should use) GetOptionsObject.

This can be further refined when landing in 262, but I think we're going to want something like this across many proposals. Closing as answered.


"Unknown" user making annotations and tags in private project

My understanding this that my project should be private, and thus people can't see, annotate, or tag project data through the listen page. However, this morning I noticed under 'Recent Annotations' there were several annotations listed as ""(not tagged)" by (unknown)". I understand people can create annotations without tagging, but it baffles me how the annotator can be unknown, and this seems to be an issue with the system's privacy settings.
For screenshot, see https://gyazo.com/91e37606f47a7b93e82baca057b5636f

I went to every site in that project as normal user and Admin.
Could not reproduce. Did you change the data?
Checked multiple different sites.
Closing for now. Reopen this issue is you notice the issue again.

please provide exact URL
another screenshot if it happens again.

https://github.com/QutBioacoustics/baw-server/blob/develop/app/views/sites/show.html.haml#L84
That's the line that could cause the problem. I've checked the database though and no annotation has both creator and updater set to null.


Would like to style demo.html

I know its not really a problem, But i just wanted to make the demo page look alot better

I want to contribute into this...

Can I work on this issue? Or someone else is working on it?

Can I work on this issue? Or someone else is working on it?

Well I'm working

Yea. Me too
On Sat, 30 May 2020, 09:40 silenteyesoncode<EMAIL_ADDRESS>wrote:

Can I work on this issue? Or someone else is working on it?
Well I'm working
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vinitshahdeo/Water-Monitoring-System/issues/243#issuecomment-636299947,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN55ZDQLQKPQLUHHBOPHNUDRUDBBVANCNFSM4MYNL55Q
.

I want to contribute to this.

@vinitshahdeo Please assign me to the issue

I would like to contribute.

I want to contribute into this...

Hey @oskorp
This is no longer the UI
You can check the UI in the wireframes in the master branch.
If you are familiar with React and can start the development in React, you can tell me and I will assign you.


Value set using MappedDiagnosticsLogicalContext is getting cleared and not getting logged

Hi! Thanks for reporting this bug!
Please keep / fill in the relevant info from this template so that we can help you as best as possible.
For .NET Core users, please check the platform support: https://github.com/NLog/NLog/wiki/platform-support
NLog version: (e.g. 4.4.13)
Platform:  .NET Core 3.1
Current NLog config (xml or C#, if relevant)
<?xml version="1.0" encoding="utf-8" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      autoReload="true"
      internalLogLevel="Trace"
      internalLogFile="Logs/internallog.txt">
	<targets>
		<target name="logfile" xsi:type="File"
				fileName="Logs/${shortdate}_logfile.txt" archiveAboveSize="5000000"
				layout="${longdate} ${level:uppercase=true} ${mdlc:item=UserName} ${message}"/>
	</targets>
	<rules>
		<logger name="*" minlevel="Debug" writeTo="logfile" />
	</rules>
</nlog>

In ASP.Core 3.1 AuthorizationHandler, once user is authorized, I am setting the user name using MappedDiagnosticsLogicalContext like below:
MappedDiagnosticsLogicalContext.Set("UserName", userName);

And expecting UserName be available in all log messages, however UserName is getting only logged in the AuthorizationHandler but not in any other log messages logged in subsequent code.

What is the expected result?
UserName has to be logged along with all the log messages which will be in several asynchronous threads after setting MappedDiagnosticsLogicalContext for the given request.

Did you checked the Internal log? yes. it doesn't give any details about the issue.

Please post full exception details (message, stacktrace, inner exceptions)

Are there any workarounds? no

Is there a version in which it did work? no

Can you help us by writing an unit test? no

If you can make a unit-test that reproduces the case, then I will be happy to look at it.

Hi,
I guess there is another thread involved.
But as this is in asp.net core handler, I would recommend to use ${aspnet-item}
example:
httpContext.Items["myvariable"] = 123;

config:
${aspnet-item:variable=myvariable} - produces "123"

I used httpContext and it worked for me. Thank you for the response.


[SPARK-17269][SQL] Move finish analysis optimization stage into its own file

What changes were proposed in this pull request?
As part of breaking Optimizer.scala apart, this patch moves various finish analysis optimization stage rules into a single file. I'm submitting separate pull requests so we can more easily merge this in branch-2.0 to simplify optimizer backports.
How was this patch tested?
This should be covered by existing tests.

cc @srinathshankar

Test build #64509 has started for PR 14838 at commit 063ddfb.

Test build #64509 has finished for PR 14838 at commit 063ddfb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64509/
Test PASSed.

LGTM. Merging to master and branch 2.0.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64515/
Test FAILed.


V12 version please

hello, could you migrate to V12? I loved your module, but it doesn't work in the new version

Second this. I can see the inspiration probably from 5E's Epic Rolls so I personally love this :)

In theory this should be fixed. The latest version should be compatible with version 11 and 12.


Support out-of-tree storage drivers

Summary
The ability to support out-of-tree storage drivers is an oft-requested feature. This enhancement will make it easier for developers to add new support to libStorage without having to be a member of this repository of the libStorage CI pipeline.

Issue moved to codedellemc/rexray #969 via ZenHub


Feature/sc 40297/sdk api client support api deprecation header

Summary
Follow up of this PR : https://github.com/opendatasoft/ods-dataviz-sdk/pull/166#event-9378374403
The goal for this PR is to show a warning when using a deprecated api function
(Internal for Opendatasoft only) Associated Shortcut ticket: sc-40297.
Changes
Add a warning when using a deprecated API function
Add an option to disable those warning (so that when using the API in production, you can disable it)
Breaking Changes
Changelog
When using a deprecated API function,  warning appear in thje console. An option is avaible to disable those warning
Open discussion
To be tested
Review checklist

[ ] Description is complete
[ ] 2 reviewers (1 if trivial)
[ ] Tests coverage has improved
[ ] Code is ready for a release on NPM

As discussed offline after testing, for now the feature doesn't work cross-domain because Explore API lacks a Access-Control-Expose-Headers header, this will be addressed outside this PR (sc-41250).
@wmai do you want to have a last look or do we go ahead?


[5.9] Swift SDKs: refactor and extend SwiftSDKBundleTests

Cherry-pick of https://github.com/apple/swift-package-manager/pull/6588
SwiftSDKBundleTests should cover the list subcommand and verify that all installed Swift SDK bundles are listed.
New testList test function was added, while testInstall was refactored to share mock Swift SDK generation code with it.
rdar://107882144

@swift-ci smoke test

@swift-ci smoke test

@swift-ci test windows


Test Metrics: Missing dataprovider info in test

Missing data provider in test info.
for a end user, the tool should display the information of the parameters provided with the data provider.
Current observation : Tool displays the same test name for data provider tests.
Expected to be : Tool should display the parameters with the test.

@saurabhgupta94 Fixed at this pull request https://github.com/adiralashiva8/testng-metrics/pull/10


Label '//apko:extensions.bzl' is invalid because 'apko' is not a package; perhaps you meant to put the colon here

Bazel: 7.3.1
MacOS 15
Trying to replicate the most basic example of a base image and I am getting the following error:
:184:26: Label '//apko:extensions.bzl' is invalid because 'apko' is not a package; perhaps you meant to put the colon here: '//:apko/extensions.bzl'?: Label '//apko:extensions.bzl' is invalid because 'apko' is not a package; perhaps you meant to put the colon here: '//:apko/extensions.bzl'?
What I did:

Added rules_apko to MODUEL.bazel

# https://github.com/chainguard-dev/rules_apko/releases bazel_dep(name = "rules_apko", version = "1.4.0")

Registered toolchain:

Copy from this repo MODULE.bazel
toolchain = use_extension("//apko:extensions.bzl", "apko")
toolchain.toolchain(apko_version = "v0.19.2")
use_repo(toolchain, "apko_toolchains")
register_toolchains("@apko_toolchains//:all")

Added a apko.yaml

contents:
  keyring:
    - https://packages.wolfi.dev/os/wolfi-signing.rsa.pub
  repositories:
    - https://packages.wolfi.dev/os
  packages:
    - wolfi-base

cmd: /bin/sh -l

archs:
  - x86_64
  - aarch64

Generated lockfile

bazel run @rules_apko//apko lock images/base_image/apko.yaml
The lockfile generates

Added BUILD.bazel

load("@rules_apko//apko:defs.bzl", "apko_image")

apko_image(
    name = "base_image",
    config = "apko.yaml",
    # See MODULE.bazel for how apko.lock.json is translated to @base_image//:contents
    contents = "@base_image//:contents",
    tag = "base_image:latest",
)

Added lock to MODULE, again copy from this repo:

# locks
apk = use_extension(
    "//apko:extensions.bzl",
    "apko",
    dev_dependency = True,
)
apk.translate_lock(
    name = "base_image",
    lock = "//images/base_image/lock:apko.lock.json",
)
use_repo(apk, "base_image")

Trying to build:

bazel build //...

ERROR: Analysis of target '//images/base_image:base_image' failed; build aborted: Error loading '//apko:extensions.bzl' for module extensions, requested by /Users/marvin/RustroverProjects/quant-engine/MODULE.bazel:184:26: Label '//apko:extensions.bzl' is invalid because 'apko' is not a package; perhaps you meant to put the colon here: '//:apko/extensions.bzl'?: Label '//apko:extensions.bzl' is invalid because 'apko' is not a package; perhaps you meant to put the colon here: '//:apko/extensions.bzl'?
INFO: Elapsed time: 0.183s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
make: *** [build] Error 1

The error points at the toolchain registration, but this is a carbon copy from this repo.
Please advise.

Solved.
Had to replace
toolchain = use_extension("//apko:extensions.bzl", "apko")
with
toolchain = use_extension("@rules_apko//apko:extensions.bzl", "apko")


Allow using SVG images as project icon

Since exporters will save their own icon, the target platforms don't have to support SVG to display the icon correctly.
In the future, we could consider supporting SVG favicons in the HTML5 export (whose support is quite good). This would require two things:

If the project icon is in SVG format, modify the loading logic to copy the SVG icon from the project as favicon.svg and reference it in the HTML.
When a SVG project icon is used, ignore requests to change the project icon from Godot itself, as it can only set a rasterized version of the icon. Perhaps this should be placed behind a project setting.

This closes #23068.
Preview

Since exporters will save their own icon, the target platforms don't have to support SVG to display the icon correctly.

I think at least Android uses the project icon by default, so this should likely be tested to confirm that it works.

Thanks!

May I suggest to include this small tweak also into 3.2.4, if it's possible? It should be pretty easy to port(one small change into main.cpp) and shouldn't break anything.

Cherry-picked for 3.2.4.


[Docs][Material] Highlight brush changes when migrating from Material v1 (Material Design 2) to v2 (Material Design 3)

Add a NOTE/WARN to the Color Update section of the v1 Material migration guide to notify that the Brush resources are now placed in ThemeDictionaries and should be referenced using the ThemeResource binding

related: https://github.com/unoplatform/Uno.Themes/issues/1157

Hello @kazo0 i am not sure which one to do 1157 or 1212


Wanna try this but don't know what to do

I want to try this but i dont know nothing abou go lang o ffmpeg
could someone explain me a step by step way for making this work?
i would like to try this on windows but if necesary i could set up a linux virtual machine

You can install Go here and ffmpeg here. Both websites should have a basic description of how to install their products. Once you have those two installed, the README will direct you on how to download anirip and use it
Hope this helped!

I will try it, i hope it works.
Thanks!

I tried but dont know what to do with the ffmpeg .zip content.

When i try to use the Anirip command i get an error
"[anirip] Error: validating session: could not verify session"
even changing some details in the link.
like the s at the end of "http(s)://" or the "es" for spanish
https:// www .crunchyroll .com/ es /ANIME

BTW: When i first tried to use "go get s32x.com/anirip"
I got an error telling me to install Git from here so I downloaded and installed this.
Besides from that i haven't done anything else so i would be really thankfull if someone could help me with this by telling me what I'm doing wrong or by giving me some kind of a step by step tutorial.
We can do it in other platform (like Discord) if it's not allowed to do it here.

hey, i get the same error. i think it is outdated...

^ getting the same error


A dependency is missing on new installation?

I recently uninstalled all gems in my system. After running gem install parity and trying to use it in projects where it was working before, I get:
% development restore production
/Users/tutec/.rbenv/versions/2.3.0/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require': cannot load such file -- git (LoadError)
        from /Users/tutec/.rbenv/versions/2.3.0/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
        from /Users/tutec/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/parity-0.9.3/lib/parity.rb:7:in `<top (required)>'
        from /Users/tutec/.rbenv/versions/2.3.0/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
        from /Users/tutec/.rbenv/versions/2.3.0/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
        from /Users/tutec/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/parity-0.9.3/bin/development:3:in `<top (required)>'
        from /Users/tutec/.rbenv/versions/2.3.0/bin/development:22:in `load'
        from /Users/tutec/.rbenv/versions/2.3.0/bin/development:22:in `<main>'

Is there a missing dependency? Thank you!

After running gem install git it started working. I think we need to add git as a development dependency in parity.gemspec.

Blerg, @tute, I'm sorry about that.
I'm going to be removing the executable from the Rubygems-installed gem shortly and recommending that users install the package from Homebrew or releases in the next week or so.


Offer option to read URLs from file not just database

prototype working on dev branch

Solved with: https://github.com/HW-SWeL/BMUSE/blob/master/core/src/main/java/hwu/elixir/scrape/scraper/examples/FileScraper.java


Create player's basic demo HUD

new functions in gui_renderer to add player hub, it temporary templated to modify in future
new asset_loader dunction to load player hub face textures
turn on/off on F5

closes #12

original player's hub looks like this (to comparison)

removing things connected with HUD
new variable on settings.py to calculate HP% in bar

closes #12

I think something went wrong with the rebase and it is really hard to review the changes


Static lib and missing run_exports

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
This package contains libs and headers. Please add an appropriate run_export to the recipe and remove the static library file from the outputs.
Installed packages
n/a

Environment info
n/a

I added run_exports in #48. I propose to implement the rest of this proposal by introducing a split into libudunits2 and udunits2, where the former contains only the shared libraries and the latter the rest. Then we can migrate all downstream packages and then remove the static library from udunits2 as well.


Upserting a field of an existing Document with null does not update that field

Given a Document with a certain _id exists in the database and has someField set to a non-null value.
If I create a new Object of that Document class, set its _id to that of the existing Document but set someField to  null, persisting that object does not set someField to null but leaves it as it was.
Setting someField of the new Object to a non-null value will update that Document.
Thus, the update query excludes fields that are set to null. I would expect it to be possible to update an existing Document's field with null during an upsert operation.

Have you tried setting the nullable option on the field in question? That should fix your issue. That said, upserts should $unset non-nullable fields if they are null. Could you provide a failing test case for us? You can look at existing tests for guidance.

Thanks @alcaeus! Setting the nullable option on the field solved the problem.
However, upserts don't $unset non-nullable fields to null, as my tests shows.

That's good to hear. I'll take a look at the failing tests shortly and fix this issue in 1.2.1. Thank you for your help!

I really apologise for the long delay here, but I completely lost track of the issue. First of all, thank you for providing the test case. I looked at the issue in question and it is easily fixable, but I'm not sure if we want to do so.
To explain, here's a quick summary on how we treat null as field value:

When inserting documents, we skip all fields with a null value unless they have the nullable option set.
When updating documents, we $unset all fields with a null value that don't have the nullable option set. nullable fields get their value set to null

Upserting documents thus should somehow respect both behaviors, which technically means calling $unset on fields with a null value. However, we don't know if the field value is null because it should be unset from the document or whether it just wasn't filled out because its value is not known and the upsert operation shouldn't modify it. In my opinion, both are valid use cases, but we can't satisfy both. This is why I'm not sure if this is something we can fix in 1.2 or whether it has to wait for 2.0 because it would require mapping changes (e.g. a flag to indicate how to deal with null values).

However, we don't know if the field value is null because it should be unset from the document or whether it just wasn't filled out because its value is not known and the upsert operation shouldn't modify it.

I think for 1.x we should keep the current behaviour (otherwise it would be a BC break if people would start losing data because of nulls). Also cross-referencing with #1425 as this is something we should support.

If inserts and updates both result in null values being set for nullable-mapped fields, it sounds like they can both get the same behavior. For non-nullable fields that have null values, we exclude them from an insert document and use $unset for updates/upserts.
Note: MongoDB offers a $setOnInsert operator if we needed conditional logic for upserts resulting in an insert, but I don't think that's needed.

However, we don't know if the field value is null because it should be unset from the document or whether it just wasn't filled out because its value is not known and the upsert operation shouldn't modify it.

If I'm reading this correct, you're asking about a case where we might not want to update fields because ODM can't discern if they've changed since a new document object was instantiated. I'd argue that if we're doing an insert/upsert with a newly created document (i.e. no changeset) we should assume that the user wants the post-write document in MongoDB to reflect the state of the document object being persisted and flushed. To do otherwise would require us to differentiate between uninitialized properties and those that were explicitly assigned with null (I don't think that's possible).
I'm of the opinion that upserts should follow insert semantics. The end result of the write operation should be that the

@jmikola correct - differentiating between uninitialized properties and explicit null values is impossible. I agree with your opinion that the behavior should be the same as for insert, even though some might see it as a BC break.
@Spunc I prepared a fix for this issue this morning but need to clean up a test. I hope to push this to the 1.2.x branch over the weekend. We're still tracking a different issue for 1.2.2, but I'm hoping to (finally) release a fix for your issue within a week.

After managing to pick @jwage's brain for a little bit, turns out we all got it backwards. The original idea was to preserve data already in the database, so writing null values shouldn't happen in $upsert queries. Thus, the behavior for non-nullable fields is correct but incorrect for fields with a nullable attribute set. Thus, I'll change the tests and pull request to fix this accordingly in 1.2.2.
As for overriding null values, your safest bet is to fetch data and then update it in the database, even at the cost of a hydration cycle. I agree your test cases may seem valid, but you typically don't know what other fields may already have been written to the database, so it's a good idea not to touch them.
We may consider adding a switch to enable this behavior (either on a per-field or per-document basis) in 2.x somewhere, but for now this is something we can't (and won't) fix. We will however address the inconsistency between nullable and non-nullable fields.


Shared/Central parameters support

This issue is a split from #831
Ability to associate/import/refence multiple parameter files into one outcome.
A fair amount of parameters will be shared by customers as well, which could lead to duplication among the parameter files. However, the Bicep community is already working on parameter imports in Azure/bicep#12019.

A cascading style will be nice. So you can inherit from a more general bicepparameter file and still have the ability to overrule an imported parameter. Like the union() function currently works. The last written value survives.

Hello @Jefajers,
Extendable parameter files became available as a preview feature in Bicep some time ago. I wonder if we should begin discussing how AzOps could support this new Bicep feature?
For reference on my thoughts, we are using these AzOps settings:
"AzOps": {
  "Core.AllowMultipleTemplateParameterFiles": true,
  "Core.DeployAllMultipleTemplateParameterFiles": true,
  "Core.ParallelDeployMultipleTemplateParameterFiles": true,
  "Core.MultipleTemplateParameterFileSuffix": ".\\w+"
}

Some thoughts on my end:

If I try to use extendable parameter files today, AzOps attempts to locate a template file to associate with my generic parameter files, hence failing the deployment. Example:
customer.bicep
customer.one.bicepparam
customer.two.bicepparam
westeurope.01.bicepparam

Where customer.one.bicepparam and customer.two.bicepparam both extend from westeurope.01.bicepparam as such:
using 'customer.bicep'
extends 'westeurope.01.bicepparam'

param ...

This fails, rather expectedly, with:

Did NOT find template  for parameters root/mg (00000000-0000-0000-0000-000000000000)/sub (00000000-0000-0000-0000-000000000000)/westeurope.01.bicepparam

Currently, the association between bicepparam files and template files is based on naming of the files as well as the configuration Core.MultipleTemplateParameterFileSuffix. However, bicepparam files have a using statement that defines the association to the template. Perhaps it would be possible to utilize that instead? Then it would also support generic bicepparam files out of the box, since they specify using none, and hence they shouldn't be deployed.

In extension to this point, perhaps a first step could be to not locate template files for bicepparam files with using none.

And then onto the more complex stuff. Let's say that I have a number of bicepparam files, that extend each other in a chain:
// westeurope.bicepparam
using none
param location = 'westeurope'
...

// westeurope.01.bicepparam
using none
extends 'westeurope.bicepparam'
param instance = '01'
...

// customer.one.bicepparam
using 'customer.bicep'
extends 'westeurope.01.bicepparam'
param customerName = 'azops'
...

If I were to make a change to westeurope.bicepparam, I would actually expect AzOps to redeploy the customer.bicep template for every customer.x.bicepparam file that extends (transitively) from westeurope.bicepparam.
We could probably accept a solution that requires all bicepparam files and templates to be in the same directory, but a solution that is capable of letting bicepparam files extend from files in parent directories would be optimal. Extending from bicepparam files in sibling directories seems like a mistake in my opinion, which could lead to unexpected behaviour.

I have also heard talk from the Bicep community about a proposal for a special type of "deployment file" that can specify things like what template to deploy, which parameter file(s) to use, settings for deployment stacks, etc. Right before the community call next week, my team is going to have a chat with some of the Bicep maintainers about bicepparam files, so perhaps we can find some synergies between Bicep and AzOps as part of that.

@Xitric, lets sync up after your meetup with Bicep maintainers 👍 and take it from there as we aim to incorporate usage of Bicep features together with AzOps.


async/await on Requester ?

Hi !
I am trying to add async/await to get data from specific service. Here is my code where I implemented but seems to be wrong, I get the response after cb() have been send.
It this a correct way to use cote js  ( Account Service and Comments Service ) ?
commentList.map(async comment => {         await accountResquester.send({ type: 'account-profile', account: { _id: comment.owner } }, response => {           comment.owner = response;         });         return comment;       });
Thanks.

Regular array methods like .map doesn’t work out of the box with async / await. That might be your problem here.

Hope it helps:
(async () => {
  let index = 0;
  await Promise.all([1, 2, 3].map(async e => {
  	index++;
  }));
  console.log(index); // 3 in console
})();


Unable to delete data from gogetta fields

Steps to recreate:
I was working in this field

I deleted the location name on test.gogetta /team. It appeared to work fine
I went to test.yourpeer, refreshed the page and saw the location name had gone.
I went back to gogetta / team Within a few seconds the red footer appeared
I went back to test.yourpeer, refreshed, and saw that the location name had re-appeared.
I deleted the location name once again on test.gogetta /team, Again it appears to work fine. No red footer appeared
This time though, even though the location name field on test.gogetta remained blank, test.yourpeer retained the location name
Returning to test.gogetta /find, the red footer has returned and the “loading location data” wheel is spinning

@adambard1 It is not a problem anymore, although the red bar still exists, I refresh and ensure my deletion worked, it generally words for 1st or 2nd time in all fields


failed at the last step when I train nasnet

When I run "python train_nasnet_mobile.py", It failed as follows:
1250/1250 [==============================] - 1439s 1s/step - loss: 0.0667 - val_loss: 0.0653
Exception IndexError: IndexError('list index out of range',) in <generator object val_generator at 0x24cf7c30> ignored
Exception IndexError: 'list index out of range' in <generator object get_controller at 0xd611500> ignored
Exception IndexError: IndexError('list index out of range',) in <generator object train_generator at 0x24cf7be0> ignored
Exception IndexError: 'list index out of range' in <generator object get_controller at 0xd611730> ignored
1)I didnt find anywhere a list index out of range.
2)I didnt find any file contains the Intermediate results,did you save the model that trained every epoch?
thks very much

when i run "evaluate_mobilenet.py" , i meet problem:
Traceback (most recent call last):
File "H:/neural-image-assessment/evaluate_mobilenet.py", line 3, in
from path import Path
ImportError: No module named 'path'
anyone else meet this problem? the mode path where?

What version of python are you using? This requires python 3.5.

And as for the list index exception, are you trying a different dataset like TID2013? Then you will need to change the parameters of the data generators.

@titu1994  i am use "Python 3.5.5 |Anaconda, Inc.| (default, Mar 12 2018, 17:44:09)  Windows10+tensorflow1.4"

i have solve this problem : https://pypi.org/project/path.py/ ;
use this pip install path.py


Assert not all tests skipped

What this PR does / why we need it:
We look at the junit xml file to compare the number of tests
against the number of tests skipped and if the number of skipped tests
is not lesser then we fail.
This is to make visible situations like in this PR: #5835
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Add check that not all tests have been skipped

/test  pull-kubevirt-e2e-k8s-1.20-operator

/test pull-kubevirt-e2e-k8s-1.20-operator

/test pull-kubevirt-e2e-k8s-1.20-operator

@fgimenez FYI

Looks like it works as intended: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/6068/pull-kubevirt-e2e-k8s-1.20-operator/1417125080781033472#1:build-log.txt%3A4862

/hold
I think we should think on Jed's suggestion before merging this. His ideas seem to have merit. What do you think @dhiller?

This looks good.
However, when I ran make functest locally last Friday when all tests were getting skipped, this was the output:
Ran 0 of 885 Specs in 12.383 seconds
SUCCESS! -- 0 Passed | 0 Failed | 3 Pending | 882 Skipped

In that case, skipped was lower than total...
Also the last time this happened about a year ago, the tests were not getting skipped, but make functest just always ended up returning true.
Is there any chance we could parse the output of make functest and fail is "Passed" is 0 or if "Failed" is non-0? That would cover both cases.
Either way, this is mergable, thank you for addressing this so quickly!!

Good point, will look into that! Thanks!

I've looked into the xml files of two examples where there have been pending test cases:

https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/6039/pull-kubevirt-e2e-k8s-1.18-0.41/1417058011150553088
https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/6039/pull-kubevirt-e2e-k8s-1.18-0.41/1416852068211101696

The resulting junit xml files show no indication that the pending cases are taken into account as separate pending state cases. I figure this is because on the CI the tests are run in two phases (parallel - serial) and then the result is a merged file of the original junit test files. So I think for CI we should be good enough.
However, while I haven't looked at what happens when the file is generated locally without parallel execution, I have looked at the code here: https://github.com/onsi/ginkgo/blob/66d7597cc0e5eccd35590d61fdf27845aa492cd0/reporters/junit_reporter.go#L126
This seems to indicate that tests with state pending and skipped are treated equally as skipped.

/retest

/unhold


Separate generic GraphQL client code from flutter implementation

Is your feature request related to a problem? Please describe.
When trying to use Graphql-flutter in a framework-agnostic way, it is today possible to target directly non-Flutter specific parts (Client, ...) of this library, which is probably the best in Dart as of today.
For example, following the BLoC pattern, we use graphql-flutter inside our business logic components that (even if used today with Flutter) are intended to be framework-agnostic (and could be used with AngularDart for example ?).
The problem is that, making a separate Dart package for this business logic components, a dependency on graphql-flutter would bring a dependency on Flutter itself, even if we're not using Flutter-specific parts of the package.
Describe the solution you'd like
Separate this wonderfull lib in 2 packages : 1 framework-agnostic with generic GraphQL, and another one with Flutter implementation targetting the first one
Describe alternatives you've considered
If this is not intended, we would probably create a new library based on the "generic" GraphQL implementation from this library. This would probably bring some overhead in maintaining synchronization between the 2 projects contributions.

I've just come across the README and just saw this statement "With that being said, the project lives currently still inside one package. We plan to spilt up the project into multiple smaler packages in the near future, to follow Apollo's modules design"
So that's great, sorry I didn't read this before ;).
And, for your information, we're really interested in my company in helping for that as it matches our development needs ;)

@adelcasse Yes will be adding that to version 1.0.0

@HofmannZ Is it possible to access the GraphQL client now for testing with 1.0.0-alpha.4?
I tried, but getting an error:

RangeError (index): Invalid value: Valid value range is empty: 0

Here is a sample of the code, perhaps I am doing something wrong?
      HttpLink httpLink = HttpLink(
        uri: graphQlEndpoint + '/graphql',
      );

      GraphQLClient graphql = GraphQLClient(
        link: httpLink,
        cache: _cache,
      );

      QueryOptions q = QueryOptions()
       ..document = signedRequest.body
       ..context = <String, dynamic>{
        "headers": signedRequest.headers,
      };

      var result = await graphql.query(q);

@HofmannZ Thanks! Keep up the awesome work.

Perhaps what I'll experiment with first is to use my fork to trim out Flutter pieces and make it compatible with AngularDart and see what it looks like at that point. And then try to see if I can refactor this library to depend on the trimmed-down base GraphQL library.

@dustin-graham It's on the top of the todo list now, just having the rolling impediments:

The subscriptions are still not merged with the new client
No example setup to test subscriptions implementation

After those to impediments the refactoring should be minimal because I kept code splitting in mind when I did the initial refactoring.

Separation MR at #203

graphql/client.dart is now in beta, though reeeeeaaaaally lacking in docs
will leave this open until it is better documented and out of beta

graphql is not yet in pub right ? Does it change a lot from the one currently bundled into this one ?

@jaumard It doesn't change much at all. You can find the beta on pub here. There are some examples too, though more docs and examples are required.

Ho ok didn't pop on search ^^ thanks @mainawycliffe ! I'll try it, as I don't need the flutter part I prefer not having it in my deps :)

Why is it dependent on dart:io? As far as I know, it makes it incompatible with web platform

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

Hello,
I've been trying to use the package on web platform with graphql: ^1.0.1 for websockets and it seems that the file socket_client.dart depends on dart:io is it normal ?
I've been trying to implement it based on this thread : https://github.com/zino-app/graphql-flutter/issues/231
Thanks.

@kei42 @klavs  just made #295 to track the dart:io issue


GPS polling interval is very long

I think this is an amazing app. Micromapping mode fills a hole that I've been bothered by for awhile.
One use of apps like this for me is to stand at or near an object and then add the point on or near the location indicator. When it's in 'follow location' mode, the new point is automatically added right where you are, which is great! However, I noticed that it seems to take about 10 seconds for location to update. I can understand that this might be for battery saving purposes, but it's hard to tell whether I'm in the right place sometimes. Could this interval be shortened? One second might be too much, but maybe 3 seconds?

Thanks, pkoby! I'll look into it, no idea why this is the case for now.
Closing as a duplicate of #106.

Whoops, didn't search for the right terms... Thanks!


add lang

Hi, I have forked your project and I am going to customize it
In the first step, I wanted to add a new language to it
I put the new language file in the lang folder but the language is not added
Can you guide me how to do this?

[image: image.png]
On Wed, Sep 6, 2023 at 9:41 PM elnaz rahimi dostan - الناز رحیمی دوستان
@.***> wrote:

Hi, I have forked your project and I am going to customize it
In the first step, I wanted to add a new language to it
I put the new language file in the lang folder but the language is not
added
Can you guide me how to do this?
—
Reply to this email directly, view it on GitHub
https://github.com/WongSaang/chatgpt-ui/issues/246, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALAX6FAZ3AZL5KQOMXTQYITXZB4OZANCNFSM6AAAAAA4NLTMPY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

[image: image.png] On Wed, Sep 6, 2023 at 9:41 PM elnaz rahimi dostan - الناز رحیمی دوستان @.> wrote:
…
Hi, I have forked your project and I am going to customize it In the first step, I wanted to add a new language to it I put the new language file in the lang folder but the language is not added Can you guide me how to do this? — Reply to this email directly, view it on GitHub <#246>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALAX6FAZ3AZL5KQOMXTQYITXZB4OZANCNFSM6AAAAAA4NLTMPY . You are receiving this because you are subscribed to this thread.Message ID: @.>

sorry i con not see your answer


feat(dtfs2-7093): update readme

Introduction :pencil2:
remove changes to circumvent code duplication issue on 7093

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

Project coverage is 25.77%. Comparing base (7f2580c) to head (d722ca9).

Additional details and impacted files
@@               Coverage Diff               @@
##           release-1.5    #3008      +/-   ##
===============================================
- Coverage        32.05%   25.77%   -6.28%
===============================================
  Files             1334     1036     -298
  Lines            30631    23806    -6825
  Branches          4947     3773    -1174
===============================================
- Hits              9820     6137    -3683
+ Misses           18157    15471    -2686
+ Partials          2654     2198     -456

Flag
Coverage Δ

api-test
0.39% <ø> (ø)

unit-test
26.04% <ø> (-6.28%)
:arrow_down:

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

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


#0: optimize allgather for small tensor sizes

For smaller tensor sizes, there is only a single packet sent through the erisc data mover channel and that packet may be smaller in size than the channel buffer. For those cases, the datamover channel buffer is shrunk to be the same size as the packet. This can save a large amount of time. For LLama, there are smaller allgathers of size 32x1024 which are allgathered on dim=3. In those cases we can get up to a 2x improvement for bfp8 and slightly less for fp16.

FYI @cglagovich

post_commit: https://github.com/tenstorrent/tt-metal/actions/runs/9360289155


PopupMenuButton lacks sufficient text contrast

Ratio is currently 3.36, should be at least 4.50.

Foreground: #8C8C8C
Background: #FFFFFF

@jonahwilliams  I'd like to try this! WIP

Hi @raacker I'm sorry I completely missed your comment - this is for our Flutter Gallery specifically, so its more a matter of design choice. If you would like to contribute to any of the accessibility bugs we have though I would be happy to point you towards some good first issues.

@jonahwilliams  It's okay. Thanks for reply. I tried to look into this issue and I figured out ratio was fine! :) And sure! it would be my pleasure to start any issues  👍


Override webflow

I tried putting a new duo-authn-webflow.xml into ${idp.home}/conf/authn/Duo, but it doesn't override the default distribution webflow config.  How can I make it do that?

Currently, this is not possible. See [https://github.com/Unicon/shib-mfa-duo-auth/blob/master/src/main/groovy/net/unicon/iam/shibboleth/idp/authn/duo/webflow/Config.groovy#L39-L40].
I see a couple options for this, though:

We could make those values properties that you could set in duo.properties or the like. This would likely be the easiest, but might require some configuration trickery.
We could work out what you're doing with the duo flow, and perhaps integrate it somehow, or at least provide a facility for it. This would likely require more work, but might provide for better future expansion.

JJ, if you are following the Shib Users list (which maybe you are), you'd see that David is changing this so that he can force a new password authentication in addition to a new Duo authn if an SP sends ForceAuthn. Right now, with Password only invoked as an Initial Authn, the Password does not get forced again when the SP sends ForceAuthn, only the Duo does. So, by suggestion of Scott Cantor, David is adding an additional invocation of the Password flow, as a subflow from the Duo flow, if ForceAuthn is set.
Seems like an option we want to provide "out of the box" in an easy way, so institutions can choose which interpretation they want to take on ForceAuthn -- re-force all the forms of authn that happened, including the 'initial one', or just those that happened after the initial one.

On Aug 1, 2015, at 2:09 PM, Jj<EMAIL_ADDRESS>wrote:
Currently, this is not possible. See [https://github.com/Unicon/shib-mfa-duo-auth/blob/master/src/main/groovy/net/unicon/iam/shibboleth/idp/authn/duo/webflow/Config.groovy#L39-L40 https://github.com/Unicon/shib-mfa-duo-auth/blob/master/src/main/groovy/net/unicon/iam/shibboleth/idp/authn/duo/webflow/Config.groovy#L39-L40].
I see a couple options for this, though:
We could make those values properties that you could set in duo.properties or the like. This would likely be the easiest, but might require some configuration trickery.
We could work out what you're doing with the duo flow, and perhaps integrate it somehow, or at least provide a facility for it. This would likely require more work, but might provide for better future expansion.
—
Reply to this email directly or view it on GitHub https://github.com/Unicon/shib-mfa-duo-auth/issues/8#issuecomment-126946775.

--
Michael A. Grady
IAM Architect, Unicon, Inc.

I believe what we are doing presently with the Duo flow is the correct behavior.  My issue is purely local in that I have to replicate current user experience.  I like the idea of the options in the duo.properties file, but that doesn't seem to mesh with how the rest of the IdP works in this respect.  What about modifying setup() in https://github.com/Unicon/shib-mfa-duo-auth/blob/master/src/main/groovy/net/unicon/iam/shibboleth/idp/authn/duo/webflow/Config.groovy to test and see if idp.home/conf/auth/Duo/ exists & if so, not loading ours?

Could you check what I have over in [https://github.com/Unicon/shib-mfa-duo-auth/tree/%238] to see if that's what you're looking for? To use a local file, you'd either create ${idp.home}/flows/authn/Duo/duo-authn-flow.xml or ${idp.home}/flows/c14n/Duo/duo-c14n-flow.xml (likely won't want to do the c14n).

Yes, that's doing what I needed.
Thanks

handled by [https://github.com/Unicon/shib-mfa-duo-auth/pull/10


feat: bump minimum python version to 3.10

Closes #29

@akoen Looks like you need to update the python version in test.yml too

Already on it :)


Stupid documentation

Location : https://ei.docs.wso2.com/en/7.1.0/streaming-integrator/samples/PublishKafkaInAvroFormatUsingSchemaRegistry/
In the documentation saying that we need to add avro schema into schema registry via curl to some URl that is not mentioned earlier where it is located/ how it is installed in the first place.
Post the avro schema to schema registry using, curl -X POST -H "Content-Type: application/json" --data '{ "schema": "{ "type": "record", "name": "sweetProduction","namespace": "sweetProduction", "fields":[{ "name": "name", "type": "string" },{ "name": "amount", "type": "double" }]}"}' _http://localhost:8081/subjects/sweet-production/versions_

Did the following:

Added descriptions to explain how the Avro schema is applied together with a link to Avro extension documentation in Siddhi documentation.

Formatted the sample page to make it more usable.


Fixup the space pin visualizer.

Allow visualization of multiple Space Pin groups (AlignSubtrees). Fixes #90
Allow visualization of global Space Pin (WorldLockingManager's AlignmentManager).

NOTE: This is a breaking change. The previous behavior found the first AlignSubtree in the scene (indeterminate), and visualized that with a global visualizer instance. A SpacePinVisualizer without its Target Subtree field set will now visualize the global AlignmentManager's SpacePins, rather than picking an AlignSubtree from the scene.

/azp run

/azp run

/azp run


[MM-17961] adds icons to user details page

Summary
Adds new icons to the user management page as specified in the design

Email Outline Icon
At Icon
Shield Outline Icon

Ticket Link

https://github.com/mattermost/mattermost-server/issues/11929
https://mattermost.atlassian.net/browse/MM-17961

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3521.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

will remove the test server to do some maintenance in the cloud infra. will create a new test server after that

Looks like the build is stuck for some reason. Has been running for 63 hours... lol
Is there a way to re-trigger the build?

@cpanato can you please look at this when you have a chance? ^^

Fixed @wiersgallak

Another thing is, would be great if you add the new icons to the components/widgets/icons/icons.stories.js to show them in the storybook.

@jespino what is the "storybook"? Haven't come across that yet.

@bradjcoughlin is not yet very advanced, but if you run make storybook you'll see it in your browser. Probably during the next week it will be in much better shape than now.

New commit detected. SpinWick upgrade will occur after the build is successful.

Test server creation failed. See the logs for more information.


Add pyupgrade pre-commit hook

Codecov Report

Merging #65 (0f2adcc) into main (c5861a5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #65   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          167       167
  Branches        26        26
=========================================
  Hits           167       167

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 c5861a5...0f2adcc. Read the comment docs.


wrong URL's in cloned webpages

I cloned one Profil from Hub H1 with Hubzila Version 2.2 to Hub H2 with Version 2.3.2. The 'old' webpages from H1 are also cloned. But the in webpages contains images are on Hub H2 not displayed.
I see, that the image URL on Hub H2 is so:
[baseurl-hub-h2]/[image-path-from-hub-h1]. But this image isn't cloned and this image  exist not at Hub H2. I mean, that this URL at Hub H2 [baseurl-hub-h2]/[image-path-from-hub-h1] is in this context wrong.

duplicate of #769
Should be resolved for new content by commit ae89f1b3b3 which fixes the missing call to sync file contents.


Add an additional setting for extra options to the Flix compiler

close #118
A new box added.

Thanks!


net/http: Default Client / HTTP mangles requests with specific long urls causing remote servers to throw 400s on valid requests

1.)   OSX 10.9.5 / Ubuntu 12.04.2 LTS (GNU/Linux 3.2.0-24-generic x86_64)
3.)
When making a request with this specific a long url (it seems when the query_string is 266 characters), the go http client seems to mangle the request.
Here is a sample hits that hits the google homepage, making the query longer and longer until the error occurs: https://gist.github.com/nemothekid/501fb7e520999342560a
4.) For comparison, the python requests library doesn't have a problem with this:
>>> import requests
>>> r = requests.get("https://www.google.com?part=id&id=ZKfj0-ygXbg%2C8SINiaqgNIQ%2CxeQhXEJaHRo%2CU2VHAMOP5DY%2CGMr1JhKms5M%2CyKjYj5HMaNU%2CJgFAaxNc-tk%2CoMnDCxiZVMw%2Ck4VG725R4X0%2CU-s10nNV_N0%2CkN-p51dFYrk%2CmmIFr0NgRi8%2C4uuQ-qxjqbU%2CuRaePS8fjQE%2CJt_G6a4g9lw%2C5GaFxfjdfX8%2CoKoDg2aG-z4%2CrArN_38Xil8%2C -RuZqxSiFIw%2CNBvTI73iNDo%2CDaRvvkud9MQ%2CZm4y67kxpBk%2CG9MQIHE4pd0%2C2YwKlqgCdQw%2C6pc67X22kFo%2CYvxGDTawvAI%2CgpdFKiMGrDA%2CVEqtKfXYX6A%2ClqQCJ-xCEBE%2CmW8WwnqTmDA%2CZ0W_4kN8isA%2C3i8icj_wJGM%2C0i6E8FPecGc%2Cw8ISxSwSz-0%2CGx2ahhyKqEU%2CrFsJdFKsJDU%2C9ET_qEJYEew%2CVdXBiVM6hyI%2CefL0fPMZGCo%2CHH1OMx7dhhk%2ClvCd9zjXeZE%2CIJ0uQjwTc8o%2CMqoReZ6WJjU%2C0UkWtQGzdXc%2CIZjgPfWX8Jc%2CGvnULl2TRlw%2C6UNXAoOt-zk%2CQDaDKoGV-YA%2CX253vC7vvb8%2Cc-MWDL4C8PY&maxResults=50&")
>>> r.text
u'<!doctype html><meta content="Search the world\'s information, including webpages, images, videos and more. Google has many special features to help you find exactly what you\'re looking for." name="description"><meta content="noodp" name="robots"><meta content="/images/google_favicon_128.png" itemprop="image"><title>Google</title>


security fee navigation on an in-progress payment request

Expected Behavior
cancelling out of security fee should take you back to the confirmation screen for the incomplete payment
Current Behavior
cancelling out of security fee on a payment request takes you to send screen contact picker

On the current build of master, cancelling out of the security fee takes me back to the review screen. Closing as it seems this was fixed


Modify ZeroScanner based on FastScanner to scan more builds.

ZeroScanner may miss a lot of Travis builds.
An alternative may be to modify ZeroScanner based on the FastScanner design to scan more builds.

My bad, ZeroScanner already uses Thomas' trick


Entry_parser setting is no longer needed in playbook examples

Im trying to set up a simple example of deploying Promtail and running into a weird problem where the generated YAML scrape configuration isn't written out like my playbook describes.   I cant think of why this would happen, hoping for an idea or two on how to get around this.
Here's my playbook:

On my target machine, this is the resulting /etc/promtail/promtail.yml

Notice how Jobname and Entry Parser are reversed?  Promtail fails to start  with this error after running the playbook.   Labels and targets are generated backwards as well.

You error seems to be related to an invalid scrap_config block - I faintly recall that entry_parser  was removed from the promtail configuration. Please check if the config you are pushing is valid

Notice how Jobname and Entry Parser are reversed?

The order of keys do not matter in yaml - so this is unrelated to the error you are encountering

Quick search for entry_parser yielded the following remark - https://github.com/grafana/loki/blob/f3da16797742bc7135ebd5ab1e27187a8d9ddd2f/docs/sources/upgrading/_index.md#promtail-config-removed
So I suggest to drop it from your configuration
Please drop it from your configuration

Thanks @patrickjahns, don't know how I missed that in my searching.  It was a late night!   I submitted a PR to update your examples to drop that setting as well.

Thanks for updating the README - will close the issue here if thats alright :-)