Compile broken in UE4.14.0?

Thanks for the awesome ping plugin! I'm going to use it with the Cardinal Menu System (free multiplayer game menu) https://forums.unrealengine.com/showthread.php?121203-Community-Project-Blueprints-Cardinal-Menu-System-Steam-and-LAN-Server-Browser-w-Friends-List to fetch the pings from listen servers on Steam. I have it working in UE4.13 without issues on our in house project but it won't compile on the new engine version. I tried the launcher and source versions of the engine. I'd take a look at it myself but I figured you all have a lot better idea of what is going on with the code.
Thanks!
Parvan / Thaddeus

We're updating our project to 4.14 right now, so I'll have to update PingPlugin to 4.14 for that.  Hopefully will have it updated soon and then I'll push the changes to this repo.

We recently updated to 4.14, and as I recall, all you need to do to fix the compiler errors is remove the includes for Object.h (and probably also Engine.h) from the WinPingThread.h and MacLinuxPingThread.h.
However, we're using it from code and not buueprints, and we also had a problem with the fact that it was a plugin at all, so we had to convert it to a module.

@swolter - I have not looked into it myself yet, but is it possible for the plugin to provide the module to use in code?

Thanks for the help. No worries if it's a lot of work. I talked to someone who thinks that exposing WAN IP for Steam servers on a session property key pair is a bad idea as it makes DDOS attacks on hosted games easier? Maybe I will rethink the easy solution I'm thinking of. If I get some extra time soon, I will take a look at the plugin.

Unless Steam proxies TCP and UDP data to/from game clients and the game server, the game client has to already have the game server's IP address so it can communicate with it - so no addition information would be given that the client already doesn't know.
For a game service with permanently available but dynamically created game servers/VMs, one can run a small VM without a game server in the same zone/location/data center as the game servers and ping that machine as the representative of all the game servers in that same location.

Hi... In 4.14 version my ping is 9999, with this ping plugin solve the problem? is possible recompile ping plugin for this version ?

crazyandre192, are you asking if it will fix the online sessions listen server's reporting a ping of 9999 out of the box? It won't fix it without implementing it to do so. This plugin strictly pings an IP. But, you can have the session share information about the session, such as IP, and then have the clients ping the IP. That's what we did.

how can i ping on the ip? I'm using advanced session plugin and online session reporting ping 9999

They have a screenshot of how to set up the blueprint nodes. https://raw.githubusercontent.com/DescendentStudios/PingPlugin/master/Example.png
You can use VaRest  and https://www.ipify.org/ to get the external IP of your host in json. Then have the server add the IP to the session information. Use this plugin for the clients to ping each listen server.

Thanks... Is there a sample project available?

We might release pings in cardinal menu in the future. It's on the agenda. Pings are working in our internal project. Honestly I thought epic would have implemented this by now but I think they are leaving out some crucial things to slow asset flips and other things like that. https://forums.unrealengine.com/community/community-content-tools-and-tutorials/93565-community-project-blueprints-cardinal-menu-system-steam-and-lan-server-browser-w-friends-list

I understand,thanks ... I have to release my multiplayer fps in October and I do not have ping in the server list ... I think for Epic not a good figure for developers, ping is important for server search and not you may ignore something so important to a multiplayer ...

Build break should be fixed with lastest pull request merges.


xdg-ninja: change jq json source

There was an issue caused by inconsistencies on the implementation of echo through different shells. Some, like bash, did not interpret \n while others, like dash, did. This caused issue when sending the information to jq to parse as it as JSON specifications forbids multi-line strings.
A quick fix was applied by giving jq the path to the file to analyze rather than grabbing it from stdin.
Fixes #22

Thank you very much!


mWeb - Chat - Pay button isn't responding in chat conversation

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to https://staging.expensify.cash/

Log in with any account (lets call Account A)

Click on Plus button and request money

Select amount and hit next ( Account B)

Select user and click Request

Log out from Account A

Log in with Account B

Click on user who requested the money

Click on Pay button

Expected Result:
Verify that the expanded view of the IOU opens
Actual Result:
Pay button isn't responding in chat conversation
Workaround:
Unknown ✔
Platform:
Where is this issue occurring?
Web
iOS
Android
Desktop App
Mobile Web ✔
Version Number: 1.0.75-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/124006617-7bd19d00-d9a8-11eb-82a7-49cf730357aa.mp4
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @tylerkaraszewski (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Not produced on the latest main branch on local

This was a regression which is just fixed.

Just verified this is working now in Android / mWeb. Closing the issue


[dump] CBS3 generic water leak sensor

Please see attached dump.  Thank you for your work on this!
tuha-tech-leak-detector.bin.zip

Thanks, added as Tuya Generic L31 Water Leak Water Level Sensor


Item Info not working

alt-c does not work at all, be it normal magic rare unique weapon.

@SoM6666 What did you bind alt+c to? In the default config there is no alt+c

just re-downloaded lunaris and it's in the settings, look
On Sat, Feb 29, 2020 at 6:48 AM Michael van Tricht<EMAIL_ADDRESS>wrote:

@SoM6666 https://github.com/SoM6666 What did you bind alt+c to? In the
default config there is no alt+c
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mtricht/lunaris/issues/68?email_source=notifications&email_token=AOV767S37HZOJBKEXLC2763RFD2YDA5CNFSM4K57CG4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENLYBVA#issuecomment-592937172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOV767TTY4Z5TGSTZA526K3RFD2YDANCNFSM4K57CG4A
.

look where?

attached jpeg of lunaris settings
On Sat, Feb 29, 2020 at 2:41 PM Michael van Tricht<EMAIL_ADDRESS>wrote:

look where?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mtricht/lunaris/issues/68?email_source=notifications&email_token=AOV767QQTSHEIJ425ML6I6LRFFSH7A5CNFSM4K57CG4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENMEFQA#issuecomment-592986816,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOV767R7WA7WH5ICWM6ANVTRFFSH7ANCNFSM4K57CG4A
.

Ah, currently alt+c only works for maps. The screenshot for weapon DPS hasn't been released yet, it'll be included in a release I'll be doing this weekend to prepare for the upcoming patch.


modbus-flex-getter does not appear to read address 0

I can use modbus-read to read holding register 0, but modbus-flex-getter shows undefined when I try to read address 0.

there it is - should be fixed in next release 0.9.+

Tested successfully. Many Thanks.


longSeason nonworking?

I have some hourly data where there is a daily seasonality which I am modeling with a 24 hour seasonality component, but there is also a weekly seasonality that I would like to model. I believe that longSeason is the correct choice here, with a period of 7 and stay of 24. Here is my model:
import pydlm as dlm
the_dlm   = dlm.dlm(oc1819_tr.total)
the_ar    = dlm.autoReg(degree=5, discount=0.99, name='ar_5', w=1e7)
the_trend = dlm.trend  (degree=1, discount=0.995, name='trend1', w=1e7)
the_daily = dlm.seasonality(period=24, discount=0.99, name='seasonal_daily', w=1e7)
the_weekly = dlm.longSeason(data=oc1819_tr.total, period=7, stay=24, name='seasonal_weekly', w=1e7, discount=0.99 )
dlm_model = the_dlm + the_ar + the_daily + the_trend + the_weekly

However, this causes a TypeError in the fit function which does not seem to be a configuration error:
TypeError                                 Traceback (most recent call last)
<ipython-input-148-c0cf1bf753dd> in <module>
      1 import timeit
      2 i = timeit.default_timer()
----> 3 dlm_model.fit()
      4 timeit.default_timer()-i

~/anaconda3/lib/python3.7/site-packages/pydlm/dlm.py in fit(self)
    281
    282         """
--> 283         self.fitForwardFilter()
    284         self.fitBackwardSmoother()
    285

~/anaconda3/lib/python3.7/site-packages/pydlm/dlm.py in fitForwardFilter(self, useRollingWindow, windowLength)
    183         # see if the model has been initialized
    184         if not self.initialized:
--> 185             self._initialize()
    186
    187         if self._printInfo:

~/anaconda3/lib/python3.7/site-packages/pydlm/func/_dlm.py in _initialize(self)
    148         """
    149         self._autoNoise()
--> 150         self.builder.initialize(noise=self.options.noise, data=self.padded_data)
    151         self.Filter = kalmanFilter(discount=self.builder.discount,
    152                                    updateInnovation=self.options.innovationType,

~/anaconda3/lib/python3.7/site-packages/pydlm/modeler/builder.py in initialize(self, data, noise)
    275             for i in self.automaticComponents:
    276                 comp = self.automaticComponents[i]
--> 277                 comp.updateEvaluation(0, data)
    278                 transition = mt.matrixAddInDiag(transition, comp.transition)
    279                 evaluation = mt.matrixAddByCol(evaluation,

TypeError: updateEvaluation() takes 2 positional arguments but 3 were given

Could this be related to the fact that all components except longSeason no longer take a data argument?

i guess nobody's home

Try to do a clone (As you would get the latest version) of the repository rather than PyPi installation, and then run longSeason.


Add log_audio, TBAudio, tests, examples

Do we really need to ship a 1 Mb .wav file only for testing?
Can't we just generate some random noisy data and be done with it?

No problem with noisy data. I will remove the test file now

Codecov Report

Merging #27 into master will increase coverage by 1.58%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   79.31%   80.89%   +1.58%
==========================================
  Files          24       25       +1
  Lines         290      314      +24
==========================================
+ Hits          230      254      +24
  Misses         60       60

Impacted Files
Coverage Δ

src/TensorBoardLogger.jl
100% <ø> (ø)
:arrow_up:

src/Loggers/LogAudio.jl
100% <100%> (ø)

src/Loggers/LogText.jl
96.15% <100%> (+0.32%)
:arrow_up:

src/logger_dispatch_overrides.jl
72.91% <66.66%> (+3.68%)
:arrow_up:

src/protojl/summary_pb.jl
90% <0%> (+10%)
: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 4dd51d4...3c0993f. Read the comment docs.

@shashikdm a bit off-topic, but you might want to learn to use git rebase.
Rather than repeatedly mergeing in master, rebaseing on top of master,
gives a much cleaner git history and makes reviewing PRs much easier.

@oxinabox I will take care from next time.

You should be able to do the same FileIO thing that was done for Images,
and that will let us also do the same thing to avoid direct dependencies
as in
https://github.com/PhilipVinc/TensorBoardLogger.jl/pull/28

@oxinabox I will take care from next time. Is there any DataType in Julia for sound type variables (like Colorant for Images) that you know of? I looked into https://github.com/JuliaAudio/ but could not find any. Currently log_audio supports AbstractArray of samples. if there's a particular datatype then it will help in automatic dispatch.

I'm not sure, perhaps @ssfrr will be able to answer?

yes I had dropped him a dm in slack a few days ago. He has not been online I guess.

Yeah, I've been on vacation and not on slack. The JuliaAudio stack stores audio as the SampleBuf type (from SampledSignals.jl, which is basically a wrapper around an AbstractArray that also stores a samplerate.
Let me know if there's any more info you need.

Great! thank you very much

So we'll have to add a dependency to SampledSignals.jl. Then we can implement Logger despatch for SampleBuf type variables.

SampledSignals does not have a huge set of dependencies.  though it has more than is ideal
How about we open an issue and think about that later.
See if anyone ends up using TensorBoard logger with this this type.
(and if so, compatibility can come from either package, or from the user's script (though that is technically piracy))

Otherwise, we could create a submodule so that to log audio files you must first do using TensorBoardLogger.Audio. We will still need to depend on SampledSignals, but we will only load it and it's dependency if needed.
Though this somewhat defeats the purpose of a general logging framework, because you need to declare in advance that you will be logging audio files...

Lets save working that out for #29 and when it comes up in practice.
Lets get this PR going for now

Agreed.
@shashikdm  If we leave out the dependency on SampledSignals, what's the status?

@shashikdm If we leave out the dependency on SampledSignals, what's the status?

log_audios and  load_audio is usable for explicit interface.
TBAudios and TBAudio can be used as wrappers for logger interface.
However, there is no automatic dispatch to audio_summary.

That's okay for now.
Can I merge?

Yes, you can merge


Ciphersuite format

2 sugestions for the Ciphersuite format:
A. Points serialization:
The point serialization we use is curve agnostic. No reason to be defined in the Ciphersuite. We could simplify things and define them as global parameters.
B. expand_len definition:
After #243 is merged, we can define expand_len in the ciphersuite and impose requirements for its value in a way that expand_message will never fail. We can then remove all the if uniform_bytes is INVALID, return INVALID steps (and potentially even make hash_to_scalar to never fail if we remove the check for 0s).

Discussed on WG on 6th of Mar. Will revisit for draft 03 after we simplify Sign

Discussed on WG call on 13th of Mar. The current point serialization cannot be curve agnostic. Will update the test vectors to use un-compressed format and re visit this after.

Discussed on WG call on 27th of Mar. Consensus is to define ciphersuite specific encodings with the addition to define PKs as points in the operation's input

Discussed on the WG on the 22nd of May. Will adress after PR #257 is merged


Improve documentation on Views

Various issues that were logged in the akkaserverless-docs repo:
By @mckeeh3
In Implementing Views, use a complete protobuf file example including the file names for the example value entity protobuf, event-sourced entity, and from a topic.The reason for this request is that there are references from the view protobuf files to the source protobuf files, i.e. domain.CustomerState, and it must be clear how the view to source names are determined. It is not clear why the reference domain.CustomerState has the domain. prefix.
By @mckeeh3
In the Creating Views section, there are examples of queries, but there are no examples of code implementations that reference queries. For example, define an API that performs a query on a view and returns a stream and another example that returns a single row. What seems to be missing is a section that explains how views are used.
By @mckeeh3
In Implementing Views in both JavaScript and Java, add a more detailed description about streams.
View queries that return streams contain the data available at the time the query is performed. We have already had questions about these streams from users (Jeremy). It was not clear what data is returned in these streams.
By @mckeeh3
In Creating a View from a Value Entity (JS, Java), change the first sentence:
Using an example of a customer registry, you can define a Customer Value Entity in Protobuf as:
to
Using an example of a customer Value Entity, it defines a domain Protobuf as:
Phrase the revised sentence as needed. The point is to make it clear that the view is using a domain Protobuf that has been defined for a value entity. The value entity and a view for the value entity share a common domain Protobuf.
By @mckeeh3
In Implementing Views for both JS and Java it should be clear that Update Transformation is possible for Value Entity views, Event Sourced Entity Views, and Views from topics.
The "Update transformation class" (Java, JS) subsection is included within the Creating a View from an Event Sourced Entity, which implies that transformation is not allowed with Value Entity Views and Topic Views.
There should b a single "Update transformation functions" subsection referenced from each of the three View from Value Entity, View from Event Sourced Entity, and Topic Views subsections.

Parent epic: https://github.com/lightbend/akkaserverless-docs/issues/761

All child tasks completed.


cstor block device  Inactive

Description
helm install  3.3.1
kubectl get bd -n openebs
NAME                                           NODENAME       SIZE            CLAIMSTATE   STATUS     AGE
blockdevice-102253af77d45d264eaac2e2d5ba230b   A<PHONE_NUMBER>528   Unclaimed    Active     28h
blockdevice-8dd07a04e59af42d034d81f4a20516ab   B<PHONE_NUMBER>528   Unclaimed    Inactive   50s

in B
lsblk
NAME   MAJ:MIN RM  SIZE RO TYPE MOUNTPOINT
sda      8:0    0    4G  0 disk
sr0     11:0    1  492K  0 rom
vda    253:0    0  200G  0 disk
└─vda1 253:1    0  200G  0 part /
vdb    253:16   0    2T  0 disk
└─vdb1 253:17   0    2T  0 part

Follow this https://openebs.io/docs/troubleshooting/ndm  I have wipe the disk, restart ndm pod on Node B
wipefs -fa /dev/vdb1
wipefs -fa /dev/vdb

but the block device still Inactive
kubectl get bd -n openebs
NAME                                           NODENAME       SIZE            CLAIMSTATE   STATUS     AGE
blockdevice-102253af77d45d264eaac2e2d5ba230b   A<PHONE_NUMBER>528   Unclaimed    Active     29h
blockdevice-a2368ea3ef3a21d2b67d18109f41c6cb  B<PHONE_NUMBER>528   Unclaimed    Inactive   107s

I compare the pod log on between A and B
 eventcode=ndm.blockdevice.create.success msg=Created blockdevice object in etcd rname=xxxxx

On A  log stop after success but B log will   ndm.blockdevice.deactivate.success   or  ndm.blockdevice.update.success constantly
openebs-ndm-vl694.log.zip
Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce

Screenshots

Your Environment

kubectl get nodes:
kubectl get pods --all-namespaces:
kubectl get services:
kubectl get sc:
kubectl get pv:
kubectl get pvc:
OS (from /etc/os-release):
Kernel (from uname -a):
Install tools:
Others:

me too

watch kubectl get bd -A -o wide
You will see that the blockdevice status value is constantly changing between active and inactive.


Feature/orders with products returns parts

Hi,
thanks for contributing, but your reports are not working for our spree app out of the box. Different errors here and there while loading the reports. I tried to make the gem and the two initial reports rock solid to avoid most errors in most scenarios, and since we are not in need for your new reports, I will not fix them.
Anyway, glad you can use the gem for project :)
Best
Mike


Add context menu in project explorer

Right-clicking on a product or sub-project in the project explorer should give a context menu with:

Clean
Build
Run

for the respective product. That would allow the user to selectively build/rebuild a product.

I'm not sure about the Run. Because:

The current product can be not-runnable.
What to run if a user selected the 'sub-project'? (there are can be multiple runnable products)

Doesn't matter. In that case Qbs will simply fail I guess. But the question is whether this is the correct way to do it. Vscode rather uses launch.json files.

Right, I'm case if sub projects there would be no run command. Only for products.

Doesn't matter. In that case Qbs will simply fail I guess. But the question is whether this is the correct way to do it. Vscode rather uses launch.json files.

If we want to see the application output in the terminal, then we can't use the QBS to run the product. In this case creates the VSCode terminal using the createTerminal() VSCode API where are passes the product executable path && run env., ant the VSCode terminal homself starts the application.

@rweickelt , main question is: do we need to use the VSCode terminal for the product Run action, or not?

It would be nice to use the built-in terminal, but if that is very difficult, then maybe skip it for now.

In this case creates the VSCode terminal using the createTerminal() VSCode API where are passes the product executable path && run env., ant the VSCode terminal homself starts the application.

I don't understand. I can only see that the createTerminal() API sets up a new terminal, but not that it starts any application. Where do you pass the executable?

Here is the snippet:
        const escaped = QbsUtils.escapeShell(executable);
        const terminal = vscode.window.createTerminal({
            name: 'QBS Run',
            env: env,
            cwd: path.dirname(executable)
        });
        if (process.platform === 'darwin') {
            // workaround for macOS system integrity protection
            const specialEnvs: string[] = ['DYLD_LIBRARY_PATH', 'DYLD_FRAMEWORK_PATH'];
            for (const specialEnv of specialEnvs) {
                if (env[specialEnv]) {
                    terminal.sendText('export ' + specialEnv + '=' + QbsUtils.escapeShell(env[specialEnv]));
                }
            }
        }
        terminal.sendText(escaped);
        terminal.show();

The terminal.sendText(escaped) is what do you need )).. And yes, it is crasy, but it is so (I'm too was confused..). ))

OK. And why is it not possible to send "qbs run -f PRODUCT -d BUILD_DIRECTORY" instead to the terminal? That way you could use the same terminal for all products (except on darwin where you need to set these variables, but that should be fixed in the Darwin cpp module), shouln't it.

send "qbs run -f PRODUCT -d BUILD_DIRECTORY" instead to the terminal?

WOW,  I have not thought about it .. need  to check it, thanks.

Fixed in v2.0.0


Problem with using with mvim

I installed this on OSX Yosemite (10.10.5) and every time I enter the full-screen mode with mvim I have a problem that occurs when scrolling down and then back up to the top of the file. The display gets stuck on the top-most line and repeats it in the window. It doesn't actually write to the file, but it is enough to make the application unusable. Does anyone know what the cause is?

This isn't related to my vimrc, but your vim installation :)
Try to update to El Captain and latest MacVim.


Presenter mode is janky

Difficult to startup
Unobvious to control

Cannot further define "jank". Closing issue.


Create dataset loader for IndoNLU NERGrit

https://indonlp.github.io/nusa-catalogue/card.html?nergrit

#self-assign

Hi @cahya-wirawan, are you still working on this? I will assume inactivity if there's no reply and will free the assignees. Thanks!

Hi @bryanwilie, I am working on it.

Noted and thank you pak @cahya-wirawan!


Adding multiple arrows

What about the addition of multiple arrows for analysis and square colouring?

This would be a chessboard.js feature - you may want to bring it up with @oakmac.


Add a common header to all pages in the documentation

On other sites we've adopted this shared header:

              ROS Resources:
            Documentation
            |
            Support
            |
            Discussion Forum
            |
            Service Status
            |
            Q&A answers.ros.org

This sort of thing would be valuable to have as we deploy the standalone site. We're working on a more complete one with better theming, but for now this should get people a consistent navigation experience.

In general, I agree with the banner idea.  However, I don't think we should put the links to http://wiki.ros.org (that would be the Documentation and Support links).  We are explicitly moving away from those in ROS 2, so adding additional links to it seems counterproductive.

This was done in #1096, so closing.


Istio Status Conditions

Description
Changes proposed in this pull request:

Istio Status Conditions

Related issue(s)

https://github.com/kyma-project/istio/issues/468

/test pull-istio-operator-integration-gcp


Select only one best node for allocate

What this PR does / why we need it:
Select only one best node for allocate
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 #699
Special notes for your reviewer:
Release note:

/lgtm


I am getting this error.

null:278 Uncaught TypeError: Cannot set properties of null (setting 'current')
at null:9809:837
at commitAttachRef (null:16581:28)
at commitLayoutEffectOnFiber (null:16469:17)
at commitLayoutMountEffects_complete (null:17275:17)
at commitLayoutEffects_begin (null:17264:15)
at commitLayoutEffects (null:17220:11)
at commitRootImpl (null:18620:13)
at commitRoot (null:18544:13)
at performSyncWorkOnRoot (null:18163:11)
at flushSyncCallbacks (null:8652:30)

After going through source file. I had to just pass a ref to SummerNote. I think it will be best to provide a full react exmple.


Add ability to set default step for queries

When scrape interval is common(eg. 15s), default step  does not make much sense.
Flag to set default step would be nice.
There was similar proposal for Thanos (Issue: #3651, PR: #3740). It would be nice to have this flag in Prometheus as well.

Thank you for your issue.
It seems that in my experience with Prometheus, I have pretty much always been happy with the default behaviour of the Prometheus UI. I even think that it works quite well.
The fact that by default, we have an adaptive step is something really appreciated, because the UI can quickly make you request years of data, by pressing a few times the + button.
Once a step is set in the UI, it is not by changed anymore. Such a scenario would mean that you would put a very high load on a prometheus server while de-zooming.
I would like to ask what is the problem that this would solve. Is it really needed to have a fixed default step, or should we adapt the graph panel to compute a step by default, which would be better than the current automatic step ?

Sorry for being unclear in describing issue.
This step should only be used when step is not explicitly set in UI. This way we can have both adaptive step and ability to set default step for all queries.
This flag would be helpful when scrape interval is relatively big (eg. 1 hr). In such cases, if user query without specifying step then Prometheus will use 1s as default value which is too low for no good reason. WDYT ?
PS: @bwplotka Please correct me if my understanding is wrong regarding this. 🤗

I think that there is a misunderstanding between the default step and the loopback delta. In general, scraping less often than 2 minutes will lead to pretty bad results with Prometheus default settings. Users who have lower scrape intervals can find convenient to adjust the --query.lookback-delta= to be a multiple of their scrape interval, but that is not related to the step interval.

It seems that in my experience with Prometheus, I have pretty much always been happy with the default behavior of the Prometheus UI. I even think that it works quite well.

Great that it worked for you, but some customization is necessary for a smooth experience when scrape interval is customized. While it's more often seen in Cortex/Thanos use cases, I saw this in Prometheus too. Especially if you KNOW as Prometheus admin that you don't allow different scrape intervals than e.g 30s, having 1s is just not needed and inefficient.

I think that there is a misunderstanding between the default step and the loopback delta. In general, scraping less often than 2 minutes will lead to pretty bad results with Prometheus default settings. Users who have lower scrape intervals can find it convenient to adjust the --query.lookback-delta= to be a multiple of their scrape interval, but that is not related to the step interval.

I think there is no misunderstanding. For such an example system, I have seen users were already adjusting lookback delta, so step is the only missing piece causing worse then needed UX. (:
We already fixed this in Thanos, but would be nice to upstream this for vanilla Prometheus users too! 🤗

I tend to disagree, the problem is that with 30s, you can have samples that are up to 29s off.

The point is that if you have many targets, you need to have a higher step by default in the UI to see when each target is changing. Having a mower interval seems like mainly a workaround for thanos being slow  (as per https://github.com/thanos-io/thanos/issues/3651#issue-771357436 )

I would also like to know what problem is fixed with this by thanos:
default step is de facto "14s" for range queries. The code that was merged in thanos only matter when you zoom below 250 ms, which is strange?

We could have it but I don't think it is low hanging fruit.

What is good: we currently already display the step in the reacti UI
I would like to have this handled by the react ui only. I appreciate the fact that you have to pass a step to the API. It is not the API to decide which step to use.
I would like to keep the default as is it now (I have not checked what default we have).

Thanks!

I have updated the title to reflect that the goal of the issue is to be able to pass something else than '1' in https://github.com/prometheus/prometheus/blob/8c8de46003d1800c9d40121b4a5e5de8582ef6e1/web/ui/react-app/src/pages/graph/Panel.tsx#L116 via a flag.


Update env.json syntax with split out pub/priv ipv4

@MaximilianMeister yea, I left "ipv4" there because other tools/repos are still using it.. It should be removed eventually (or... just removed as part of depreciating the terraform repo)

@MaximilianMeister the goal here is for supporting openstack style environments, where not all VMs have a floating IP.
So - https://github.com/kubic-project/automation/pull/53 picks up from here, and generates a SSH config from env.json which can include a proxycommand for a bastion host etc.

My bad :)


Retain AbstractBinaryMemcacheDecoder.currentMessage when sending it o…

…ut and release it when it's not used.
Motivation:
AbstractBinaryMemcacheDecoder.currentMessage is not retained after sending it out. Hence, if a message contains extras, io.netty.util.IllegalReferenceCountException will be thrown in channelInactive.
Modifications:
Retain AbstractBinaryMemcacheDecoder.currentMessage After putting it to out and release it when it's not used.
Result:
No IllegalReferenceCountException or leak.

@windie LGTM..
@daschl ok ?

@normanmaurer @windie in the catch block we are passing the invalid chunk up the pipeline with out.add, right? So if you release it here it won't be accessible up the pipeline?

@daschl link ?

@normanmaurer here for example: https://github.com/windie/netty/commit/b43d57f8c76c62571f934d401e0ab44e9194c97c#diff-a5317e7fab57c10c246e483c669307ecR159 .. to me it seems it is released but next line its passed up the the chain as invalid chunks.

@daschl I may be missing something but doesn't invalidChunk(...) return a new object and so its ok to release the currentMessage ?

aah right! then LGTM :)

@windie thanks again!
Cherry-picked into 4.1 as<PHONE_NUMBER>f72a620aa29fb7b446cd8e2d3ae5fc


why do we need to access /vivo uri ?

In the recent changes the access to the UI needs to be done through /vivo... please document the reasons.
If there are strong reasons, please make a redirect from / to /vivo so it don't fail when trying to access the service with a 404 status code

We can configure vivo url/path, but I'm guessing this is related to kubernetes ingress configuration. @patrick-stephens might know more

The goal here was to provide a configurable sub-path for Vivo (and our other services) similar to other tools, e.g. Grafana and Prometheus both support specifying an optional sub-path to serve things on.
The intent was to make it optional and configurable with @tarruda trying a few ways to achieve this but ultimately it ended up being hardcoded unless we refactor the current implementation to support it. I'm not sure you can support a redirect here @tarruda as I think the problem was that it is all created at build time but maybe it could be built in directly?
This came about during deployment of our on-prem solution which highlighted a few issues with services unable to support sub-paths so complicating a lot of the data segregation for common things like Ingress support (which is a reasonable and common ask).
With a sub-path we can easily segregate traffic intended for a specific service which is particularly important when you are running multiple services in the cluster, or even multiple Vivo instances.
You can see the issue by looking at the optional Ingress configuration for our on-prem deployment: https://github.com/calyptia/charts/blob/297dde672a3b32e76deab407e343b46fb98637a9/charts/calyptia-cloud-standalone/values.yaml#L258-L272
Vivo traffic is easy to redirect, we just match the sub-path.
For the others we have to come up with mutually exclusive regexes to ensure traffic for one service is not sent to the other. This is brittle and requires in-depth knowledge of the service itself, there is no guarantee we would not have an overlap (e.g. what if a new endpoint is added /vivo from one of the other services?) in the future.
The issue is around the Vivo internal links it provides: we can easily segregate traffic at a cluster level but the user is accessing this from the browser on their host so if they click a link it needs to ensure it provides the same sub-path prefix to forward the traffic rather than assuming it can use the root / path always.

got it, I care about using it also in standalone mode, just make sure it works right out of the box


[WIP] Integrate CloudSlang with StackStorm

We are looking to integrate CloudSlang (http://www.cloudslang.io/) workflow engine with StackStorm.
Adding @orius123 from CloudSlang team...
This is an early WIP, to make sure we are doing it the right way..
The idea is to add a new Runner (just added here a copy of LocalShellRunner, so ignore the new file content) , that will trigger CloudSlang (as a script, using simple shell cmd).
This is for first phase...
Any thoughts ? comments?

That sounds great - new community contributed runners are definitely more than welcome.
As far as the approach goes - yes, adding a new runner is the way two go. Currently we have two workflow runners (action-chain and mistral-v2) which you can use as an example / starting point.
Action-chain one is bundled in, so the one you should really be interested in is Mistral since you will probably follow similar patterns when adding a CloudSlang runner.
With an external runner there are multiple things you need to keep in mind and handle:

How do you talk to the runner (I assume CloudSlang has some kind of REST API).
Triggering a work-flow (in your case, it sounds like you will do that via shell script).
Retrieving the results and periodically querying the work-flow engine for the workflow status. We currently have a concept of "sync" and "async" runner. Mistral one is async and it looks like CloudSlang one should be async as well. In this case, async runner means that we asynchronously schedule a work-flow run and then periodically poll the workflow engine for the status and results. In comparison to the sync approach, this approach is more efficient and allows us to make the system HA - you can restart the StackStorm components and on restart we will simply continue polling workflow engine for status and result. Currently we have "querier" abstraction for handling periodic querying of the workflow engine (see st2actions/st2actions/query/* for an example of how we do that for Mistral).

Can you elaborate on what is "talk to the runner"?
Isn't this either triggering or Retrieving the results? which more func is needed?

@meirwah Yeah, with that I really just meant the following:

Triggering a workflow run
Periodically polling workflow engine for the workflow run status and result
Canceling a workflow run (optional, but nice to have)

In Mistral case, we handle all of that by talking to the Mistral REST API.
As far as 2) goes - workflow consists of multiple "tasks" (that's the term we use to refer to an "atomic" unit of action inside a workflow) which means we want to periodically poll the engine, retrieve the intermediate results and incrementally update the workflow result inside the StackStorm database.
This allows us to return "partial" results to the user as soon as they are available. This is important since a lot of workflows are long running and it's important to see the workflow progress and tasks results as soon as they are available.
A less ideal alternative would be to only update and retrieve the full result when the workflow completes.

Sure, makes perfect sense...
I will be happy if we can do this in phases ...
Phase 1: (less ideal alternative) only trigger using a shall cmd and retrieve the full result when the workflow completes.
Phase 2: move to work with REST API to periodically poll the CloudSlang  engine.
That way we can move forward in a more iterative/agile way...
And we can understand well what API are needed to make CloudSlang a first class citizen in StackStorm..
WDYT?

@meirwah I'm fine with the iterative approach.
From what you said it sounds like you will use synchronous approach?
Because if you use asynchronous approach + talking to the CloudSlang REST API it's shouldn't be much additional work to incrementally update the results vs only updating it when a workflow completes.

Added a WIP label so no one accidentally merges this PR.

@kami I meant a synchronous approach for phase 1.
I agree that we will be easier to upgrade it if we go with ASYNC approach, but since our REST API is quite new(and might by missing some of the func needed), I would like get better understanding of the needs (by phase 1), so we can understand which REST API will be needed (added or modified) for StackStorm integration.
Does it makes sense?

Sure.
To get this pull request to an experimental MVP phase (and something which is also applicable and useful to other people) which we can merge into master, it needs to contain at least the following things:

Initial runner implementation which works end to end. This means user can declare workflow definition in the metadata file, trigger this workflow execution and retrieve the result.
For a first MVP, I'm OK with using sync (CLI) approach (you can re-use subprocess utility functions which are also used in the local runner). You do need to keep in mind though that the synchronous approach means restarting the action runner service will kill all the workflow executions which are currently in progress. Well, it actually depends on how the CLI is implemented. Killing it might not cancel the execution, but it will make it impossible for us to retrieve the result if we don't implement the "querier" pattern also used in the Mistral runner.
Tests for basic functionality.
Basic documentation on how to configure and use the runner.

I will also try to look into the CLI myself some time in the next couple of days to get a better understanding of what is possible and not possible using the CLI.
In the meantime, you should also have a look at how Mistral is implemented and let us know if you have any questions.
P.S. This will also be a great learning experience for us since all of the runners so far have been implemented by us. I hope that the minimum we get out of this (in addition to a new runner) is some lessons learned and documentation on how to write a new runner which will hopefully make it easier for other users in the future who are implementing a new runner :)

:+1:

@Kami ,
@orius123 has added basic capability of triggering CloudSlang from StackStorm, it seems to work well from both UI and CLI.
We plan to add tests + documentation next week...
Feel free to review and comment (or if you prefer, wait for us to add tests+docs...)
Open question we have :
Currently CloudSlang is not installed by default with ST, it assumes that CloudSlang is under a certain path in the file system (look at st2.conf.sample), and triggers it from there ...
What we can do, is add to the "st2_deploy.sh", the setup of CloudSlang  (just download + unzip , that's it... ) , so one step less for user to use CloudSlang in StackStorm, WDYT ?

@meirwah Sorry for the delay. I was traveling when you originally added a comment and I forgot to check it out when I got back online.
In any case, I will have a look today.

:+1:
Let me know about the "st2_deploy.sh" part...

What we can do, is add to the "st2_deploy.sh", the setup of CloudSlang (just download + unzip , that's it... ) , so one step less for user to use CloudSlang in StackStorm, WDYT ?

I believe @DoriftoShoes already answered this on IRC, but yes, I think it's reasonable to install and configure CloudSlang in st2_deploy.
We should make it optional and opt-in though (INSTALL_CLOUD_SLANG=${INSTALL_CLOUD_SLANG:-0}) - similar as how you can currently specify if you want to install Mistral, but in this case, we should default to False. Also keep in mind that dependencies need to be updated (java, ...) if user selected to install CloudSlang.

@meirwah How are things going - I believe documentation and tests are only things which are left to be done (and changing the parameter type to object from string)?

@orius123 is working on tests.
Regrading the param type to object, I thought we are waiting to @enykeev response on the thread...
@SamMarkowitz will start soon to work on documentation part...

Apparently, I was missing you're waiting for me. Sorry for that :)
Right now, st2web doesn't support type: object params. We have plans to eventually unwrap type: object fields in the same manner it works for action parameters (a from inside a form), but I have no ETA on that feature.
Temporarily, we could probably make a simple version of that field that allows user to define an object using json notation or key-value list, though, at the end of the day, it would hardly make any difference to the user as he would have to enter the same string to the same field anyway.

@Kami please note that @orius123 just added tests to the PR,
please review...

I've added some more comments and the tests so far look good :+1:
As far as inputs type goes, I would still like to change it to object since this is the right thing to do. Using string really means we are just compensating for a lack of feature in the WebUI which is not the right thing to do. WebUI is just one of the API consumers and we are trying to make API consumer agnostic and make sure it's useful to all the consumers (webui, python client, cli, etc.).

Hope I fixed all of the comments.
And @SamMarkowitz is done with docs as far as we see.
Any other comments?

Thanks. I will have a look.

I'm testing the changes locally (I've also created a new local branch with some minor style changes and improvements).
If everything goes well, it should be merged some time in the next couple of days.

Also, in the future it will be easier for both of us if you keep your changes in a separate branch - this way you won't have conflicts when you pull down latest master.

Closing in favor of new PR - #1622.


MVP of Sysapps

On AP-ALB we have the concept of 'App' somewhat like a "OpenResty conf proxy rule related to one end user application". But there are some applications that we could use the very own AP-ALB App rules to proxy for internal administrative functions.
The idea with this MVP is for now somewhat copy the structure of what is done with Apps, but use different paths, folders, etc and eventually even allow implement some ACLs more specific to All sysapps than have to change each item of app to specify itself as system.

Implemented on v0.8+


Follower resync on restart BTS-829

Currently, when a follower is restarted and comes back so quickly such
that the leader has not noticed that the follower was gone, we only
resync the follower on the first write to the leader (lazy resync) and
not right away.
In this way, we later get a potentially expensive dropped follower and
shard resync when the server restart is already a long time ago.
This PR makes it so that we immediately resync a follower shard if the
server has been restarted.

Always resync a follower shard on restart.
CHANGELOG.

Scope & Purpose

[*] :hankey: Bugfix

Checklist

[*] Tests

[*] integration tests (covered by existing)
[*] resilience tests (covered by existing)

[*] :book: CHANGELOG entry made

[*] Backports:

This is the backport to 3.9, original for devel: https://github.com/arangodb/arangodb/pull/15974

[*] GitHub issue / Jira ticket: https://arangodb.atlassian.net/browse/BTS-829

Wrong base.


Alternative W/O Plugin

I know this is an old plugin but I still had a need for it. I couldn't get it to work but, there is a built in command line rhythmbox-client --seek=-20 which doesn't work (although it should http://unix.stackexchange.com/questions/75677/start-a-song-from-a-specific-time-in-rhythmbox-using-command-line)
But, as that stackexchange shows, you can run dbus-send --print-reply --dest=org.mpris.MediaPlayer2.rhythmbox /org/mpris/MediaPlayer2 org.mpris.MediaPlayer2.Player.Seek int64:-15000000 which does. With ubuntu, it's easy to setup the keyboard shortcut with the built in keyboard manager
If anyone else was struggling to find a solution...

Thanks for that @JZL003 .. will leave this open for a while in case anyone else runs in to the same issue.


dry-data -> dry-types

dry-data has been renamed to dry-types but the readme for dry-logic still references old name (resulting in Github's 404)

thank you :)


Challenge Submit 1

@mobencm thank your for your submission, the team will start reviewing your PR in the short term and get back to you with some feedback, comments and questions. Please address them accordingly.

@mobencm thank you for your submission, we are closing this PR as we move to the next stage in the process.


Investigate reduction performance difference between XeTLA and Triton

Looking into our FlashAttention-2 benchmark, we see that reduction codegen for XeTLA is better than Triton.
Investigate reasons and whether the kernel code or any pass in our pipeline has to be tuned.

Initial investigation. For perf_attn:06-fused-attention.forward.py.
Environment:
Commit: 4313cfee5ad7a4ff0b267be8c0775937227c25ec
Device: Intel(R) Level-Zero, Intel(R) Data Center GPU Max 1100
L0 version: 1.3.29735
IGC version: igc-1.0.16900.23

Results:
     Z     H   N_CTX  D_HEAD     Triton     oneDNN
0  4.0  48.0  1024.0    64.0  20.311264  40.937938

If we look into the generated assembly, we see:
--- Triton
+++ XeTLA
-//.spill size 10368
-//.spill GRF est. ref count 1935
+//.spill size 7040
+//.spill GRF est. ref count 343

Triton spills way more variables than XeTLA.
Also:
--- Triton
+++ XeTLA
+      slm_size:        25600

XeTLA's kernel is using local memory to cache data whereas our kernel's only using global and private memory. This may explain worse spilling on our side.

Memory access analyses:
XeTLA:

Triton:

Note how the Triton kernel performs way more memory accesses. This might be caused by worse spilling.

Utilization analysis (Note different Y axis scale):
XeTLA:

Triton:

This translates to worse utilization in our kernel. Note Triton stall % is about as double and utilization, about as half.

Looking at a stall event chart for these two kernels:
XeTLA:

Triton:

The purple line corresponds with SBID (score board dependency) stalls. These stall events are related to memory accesses. We see how XeTLA's are more scattered and a handful of instructions dominate the graph. However, for Triton, we see way more purple all around the graph. This would be yet another hint at spilling hurting our FlashAttention-2 performance.

Using the options provided in collect.sh, we get the following data when running FlashAttention-2:
First of all, it's worth noting that increasing the GRF size fixes the stalling problem and no stalling is taking place now. This makes our kernel "better behaved" w.r.t. memory accesses:

Stalling-wise, we have also improved a lot, getting <40 % stalled threads. It's also true our occupancy is around 50 %. This is the case as we're increasing the GRF size and our GPU cannot host as much threads.

If we take a look at the stalling events:

We see the effects of getting rid of spilling. However, we see a couple of instructions dominating the stalling. Pending investigation. Also, we see more "other stall" events.

Looking into reduction differences and whether reduced occupancy may be detrimental for us.

Our reduction code is similar to (pseudocode):
%input0 : tensor<8x16xf32>
%input1 : tensor<8x16xf32>
; 8 slices for each tensor -> 16 slices
%slice_0 = extract %input0[0] : tensor<8x16xf32> -> tensor<16xf32>
%max_0 = reduce { max } (%slice_0) -> f32
...
%slice_7 = extract %input0[7] : tensor<8x16xf32> -> tensor<16xf32>
%max_7 = reduce { max } (%slice_7) -> f32
...
%slice_8 = extract %input1[0] : tensor<8x16xf32> -> tensor<16xf32>
%max_8 = reduce { max } (%slice_8) -> f32
%red_res = glue %max_0 %max_1 ... : tensor<16xf32>

In LLVM, this would be:
%input0 : <8 x float>
%input1 : <8 x float>
; 8 slices for each tensor
%el_0 = extractelement <8 x float> %input0, i64 0
%max_0 = call float @llvm.GenISA.WaveAll.f32(float %el_0, i8 12, i32 0)
; ...
%red_res = ; insertelement x 16

The issue with WaveAll is that it compiles to (SIMD16, pseudocode):
; Split input 16-element vector in two. '.8' means 8 elements offset
mov (8|M0) rhs lhs.8
; Element-wise max, store in lhs
sel (8|M0) ge lhs lhs rhs
; Do the same again and again
mov (4|M0) rhs lhs.4
sel (4|M0) lhs lhs rhs
mov (2|M0) rhs lhs.2
sel (2|M0) lhs lhs rhs
mov (1|M0) lhs rhs.1
sel (1|M0) lhs lhs rhs

This results in 16 reductions of 8 operations each, meaning 2^7 operations. This is the case because we are not exploiting SIMD parallelism here, we're just reducing over the subgroup, underutilizing SIMD lines.
Ideally, we would like to do something like:
; Glued and transposed input matrices
%input : tensor<16x16xf32>
; 16 slices
%slice_0 = extract %input[0] : tensor<16x16xf32> -> tensor<16xf32>
...
; 15 ops to reduce each slice
%max_0 = max %slice_0, %slice_1
%max_1 = max %max_0, %slice_2
...
%max_t = glue %max_0, %max_1 -> tensor<16x1xf32>

Which, in LLVM, would be as simple as:
; Transpose original input
%input: <16 x float>
%el_0 = extractelement %input[0]
%el_1 = extractelement %input[1]
%max_0 = max %el_0, %el_1
; Just 15 max operations required!
; ...
%max = max %max_n1 %max_n2
; Transpose output

Here we could perform the same reduction in just 15 operations. Now, why does this "matrix transpose" work? Where is even the matrix? How can we transpose it?
Let's step back for a moment. Conceptually, we have a 16x16 matrix here distributed as follows across the subgroup:
+--------------------+---------+
| sub_group_local_id | indices |
+--------------------+---------+
|                  0 | 0-15    |
|                  1 | 16-31   |
|                  2 | 32-47   |
|                ... | ...     |
+--------------------+---------+

In this matrix, reducing across the second dimension is very expensive, as we need to perform subgroup reduction. Doing a matrix transpose:
+--------------------+---------------------+
| sub_group_local_id |       indices       |
+--------------------+---------------------+
|                  0 | 0,16,...,i*16,...   |
|                  1 | 1,17,...,i*16+1,... |
|                  2 | 2,18,...,i*16+2,... |
|                ... | ...                 |
+--------------------+---------------------+

Now we can reduce over the first dimension utilizing the SIMD lanes as we saw above.
We've seen the matrix and how nice it is to transpose it. However, how can we do that? This is yet to be explored. I'll build a POC for this. Possible solutions include going through local memory (probably fastest) or shufflevector + sub_group_shuffle to do everything in registers.

By transposing it has extra overhead.
The IGC supports to cooperation with SIMD and SIMT if there is limitation in SIMT expression.
The invoke SIMD and inline VISA can help to generate micro operation with the SIMD extension.
And in this case, I think the invoke SIMD is more suitable.
https://github.com/intel/llvm/blob/3c2c938769914eedd935c4fff097591f340ee0de/sycl/doc/extensions/experimental/sycl_ext_oneapi_invoke_simd.asciidoc
And the SIMD 2D extension allows us to defines the strides of the operands of the SIMD binary operation. It can allow us to use the best papalism of the SIMD.
https://github.com/intel/vc-intrinsics/blob/master/GenXIntrinsics/docs/GenXLangRef.rst#2d-region

The GENX intrinsics may be a nice option. However, our kernels model work-item behavior as of now, not EU behavior, so this would mean changing pretty much all we have as of now, right? I'd rather go with the local memory approach for now, get some numbers and see if that improves performance.

Yes, make sense.
Let's check out what we can get with pure SIMT paradigm.

Prototype with SLM transpose taking shape. Overall structure finalized, missing last bits of lowering.

Pushed prototype. Shaping into what we want.

Couldn't work on POC much this week. DPAS output input to reduction has expected size now to exploit matrix transpose approach

Lots of progress in POC. Hitting one workaround. Will try to work around it.

PoC ready to review. Will push latest perf report. Minor fixes to be pushed.

PR with latest needed code in llvm-target ready for review and with one approval. Feature will be available behind an environment variable.


Add support for External Siren sensor type

Before submitting this form

[X] I have checked that this sensor type was not suggested for addition before; I know that it could
be in open or closed issues, and if there was another issue mentioning this sensor type I could
simply comment in it instead

[X] I have updated the <sensor type> mention in the title of the issue, but left the issue title
as it was otherwise

[X] I understand that if I want to suggest the addition of more than a single sensor type, I need
to file that form for each separate type

Sensor type
Qolsys Z Wave Siren
Home Assistant device class
Siren
Does the panel send sensor status updates?

[ ] Yes, I can see the Open/Close messages (I have been able to capture ZONE_ACTIVE messages)
[X] No, it seems to only tell us the sensor exists, but never update its state

Sensor data from an INFO message
'id': 'ZW58',
'type': 'Z-Wave Siren',
'name': 'External Siren58',
'group': 'Siren',
'status': 'Closed',
'state': '0',
'zone_id': 20,
'zone_physical_type': 1,
'zone_alarm_type': 3,
'zone_type': 120,
'partition_id': 0
Sensor data from a ZONE_EVENT's ZONE_UPDATE message
cant trigger atm... sleeping wife would kill me
Sensor data from a ZONE_EVENT's ZONE_ACTIVE message
see previous message :)
Additional context
if you need the trigger ill try and get when no one is home.
item: https://qolsys.com/iq-siren/
I should mention it shows up in the other z-wave devices, not the normal z-wave window, on the iq4

The type seems to be Z-Wave Siren from your pasted INFO message, is that accurate ?
I do need to know also if it translates status or if it's just a dummy device when using through Control4 !

I dont even know how to look up what your asking.
TRhis is what comes across MQTT AppDaemon log.
2023-04-27 17:21:38.362700 WARNING qolsys_panel: sensor of unknown type: {'id': 'ZW58', 'type': 'Z-Wave Siren', 'name': 'External Siren58', 'group': 'Siren', 'status': 'Closed', 'state': '0', 'zone_id': 20, 'zone_physical_type': 1, 'zone_alarm_type': 3, 'zone_type': 120, 'partition_id': 0}
On Thursday, April 27, 2023 at 09:29:30 AM CDT, Raphaël Beamonte<EMAIL_ADDRESS>wrote:

The type seems to be Z-Wave Siren from your pasted INFO message, is that accurate ?
I do need to know also if it translates status or if it's just a dummy device when using through Control4 !
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Have you been able to trigger and confirm that there's ZONE_ACTIVE messages appearing for the Siren ?


Generate DDL: settings aren't working

System information:

Operating system (distribution) and version win 10x64
DBeaver version 6.1.4

Describe the problem you're observing:
"Use fully qualified names" and "Compact SQL" options has no effect on Generated DDL
Steps to reproduce, if exist:

Choose some table
Generate SQL - DDL for the table
Check-uncheck "Use fully qualified names" and "Compact SQL" options

Issue isn't reproducible for other generated SQL like SELECT, UPDATE, INSERT, etc

Fixed

Verified.
"Use fully qualified names" works as expected


Issue with instructions of challenge: Scientific Computing with Python Projects - Budget App

Describe your problem and how to reproduce it:
In the README, the instructions for the check_funds method ask for a method that returns False when the amount-argument is less than the balance and returns True otherwise.
Meanwhile the test expects the method to work the opposite way round, expecting a return of False when the amount is 20 and the balance is 10
Add a Link to the page with the problem:
https://repl.it/@MakuraGaeshi/boilerplate-budget-app-5
Tell us about your browser and operating system:

Browser Name: Google Chrome
Browser Version: 86.0.4240.111
Operating System: Windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

Hello there,
Thank you, for taking the time to report this. We are aware of the issue here:
Duplicate of https://github.com/freeCodeCamp/freeCodeCamp/issues/39351


Fix Symbol.to_networkx()

Closes https://github.com/dwavesystems/dwave-optimization/issues/18
Closes #52

The image in the docs looks like


Questions on ParMesh

Hello!
While implementing a parallel 4d mesh generator from a parallel 3d mesh, I'm trying to create internal ParMesh structure following the general lines from existing ParMesh constructor. Now I got several questions about the ParMesh class.
What I understand (correct me if I am wrong):

The groups (which are then converted to the gtopo) contain all the groups of process indices which have something shared with the local process.
group_sfaces is a table which for each group of processes gives indices of shared faces (as numbered from 0 and then incremented while looking for the shared faces) for this group of processes.

For example, if there are two processes, then there are two groups = {0} and {0,1}.
3) Why then group_sfaces has only one row? Is it because we just don't consider for group_sfaces group number 0 which is the local process itself?
4) Is the ordering of process numbers in a single group important? For example, for the case of two processors groups (as table, which I got from gtopo for a 3d mesh), the second group is {0,1} for process 0 and {1,0} for process 1.
5) what is the meshgen variable, will it be used somewhere if the mesh is already created or is it just some kind of info parameter?
I would be grateful for any comments.
Thanks,
Kirill

Hi Kiril,
You are correct about 1)-3).
Regarding 4), group 0 is always the "local" group that doesn't have any shared entities, see http://mfem.github.io/doxygen/html/communication_8hpp_source.html#l00052.
Regarding 5): meshgen describes the type of the mesh, see http://mfem.github.io/doxygen/html/classmfem_1_1Mesh.html#ae840fd4a3a6da40bbeaa4e33ce70345f. This is used in several Mesh and ParMesh functions (grep for it).
Hope this helps,
Tzanio

Thanks, @tzanio,
Now I am looking at ParMesh contructor in MFEM. At line 353, http://mfem.github.io/doxygen/html/pmesh_8cpp_source.html, MyRank is added as one of the processor groups to local variable "groups". Then in the end "groups" are converted into a ParMesh gtopo internal structure via gtopo.Create().
But when I print out group_lproc from a ParMesh object as obtained by
    Table group_lproc;
    group_lproc.MakeI(pmesh.gtopo.NGroups());
    for ( int row = 0; row < pmesh.gtopo.NGroups(); ++row )
    {
        group_lproc.AddColumnsInRow(row, pmesh.gtopo.GetGroupSize(row));
    }
    group_lproc.MakeJ();

The first group is 0 for all ranks. Is this very first element of groups (the "local" group) changed from MyRank to 0 somewhere in gtopo.Create()? Are there any other changes to the groups there?
My aim is to take given groups from already existing ParMesh and copy it to the ParMesh which I am creating. So I thought that group_lproc would be the same as local variable "groups" in ParMesh constructor.
Could you please clarify this moment?
Thanks,
Kirill

Thanks, @tzanio!
It does help.
I've got another question - on how vertices are locally numbered in a parmesh object.
What I want to know is:
if, say, two processes share vertices A and B which have local indices i0 and j0 for process 0 and i0 > j0, then is it true that i1 > j1 where i1 and j1 are local indices of the same vertices A and B for process 1, or not?
Thanks,
Kirill

if, say, two processes share vertices A and B which have local indices i0 and j0 for process 0 and i0 > j0, then is it true that i1 > j1 where i1 and j1 are local indices of the same vertices A and B for process 1, or not?

Nope.
The only thing you are guaranteed is that the vertices will be numbered in the same order in the row of group_svert that corresponds to the group for these two processors. The actual local numbering in each processor can be very different.

Thanks, @tzanio!

Could anyone please explain to me what should be stored in group_ldof table in pfespace->gcomm if pfespace is a ParFiniteElementSpace object? The linear finite element collection is used so that ldofs are the mesh vertices.
As a background, I am looking for a bug in my mesh generator which finally leads to wrong indices appearing in pspace->ldof_ltdof (same global index for different vertices = ldofs) for pspace built for the created ParMesh and linear finite element collection. So I am trying to understand which part of the pmesh is wrong.
Thanks,
Kirill

Could anyone give me in idea what is the purpose of renumeration of local vertices in parMesh constructor: http://mfem.github.io/doxygen/html/pmesh_8cpp_source.html#l00175?
Lines:
  175    // re-enumerate the local vertices to preserve the global ordering
  176    for (i = vert_counter = 0; i < vert_global_local.Size(); i++)
  177       if (vert_global_local[i] >= 0)
  178       {
  179          vert_global_local[i] = vert_counter++;
  180       }

Is it a necessary step? I mean, can I have an arbitrary global indices for local vertices on each process (of course, consistent for the shared vertices)?
In my mesh generator I don't do any local vertex renumeration and I see that in comparison with MFEM generator for the same serial mesh and partioning, I have different local vertex ordering. And this finally lead to a slightly different computation results.
So, I wonder whether I have generated invalid ParMesh because of the ordering.
Thanks,
Kirill

@tzanio,
Another question about your comment - is it necessary to have the same order in the rows of group_s for the corresponding groups of the processors when I am generating these tables?

The only thing you are guaranteed is that the vertices will be numbered in the same order in the row of group_svert that corresponds to the group for these two processors. The actual local numbering in each processor can be very different.

Hi Kirill,
Yes, the shared entities should be ordered the same way in their group, on each of the processors that forms that group. This way you can communicate the data directly in the group.
Hope this helps,
Tzanio

Hi Kirill,
To answer your question about the re-enumeration at pmesh.cpp:175: there are at least two reasons we do that: 1) when running on one processor, we want to preserve the vertex ordering from the serial mesh; and 2) this ordering is important in the method ParMesh::ReorientTetMesh(). There may be other parts in the code where this local vertex ordering is important but I am not sure.
The method ParMesh::ReorientTetMesh() needs to be called when using Nedelec elements of order >= 2 on tetrahedral meshes - see this description for some details.
Veselin

Thank you very much for the answers, @tzanio and @v-dobrev !

Hello again!
Could someone please explain what is exactly happening in the lines 550-573 in ParMesh constructor?
  550             switch (shared_faces[sface_counter]->GetType())
  551             {
  552                case Element::TRIANGLE:
  553                   sface_lface[sface_counter] = (*faces_tbl)(v[0], v[1], v[2]);
  554                   // mark the shared face for refinement by reorienting
  555                   // it according to the refinement flag in the tetrahedron
  556                   // to which this shared face belongs to.
  557                   {
  558                      int lface = sface_lface[sface_counter];
  559                      Tetrahedron *tet =
  560                         (Tetrahedron *)(elements[faces_info[lface].Elem1No]);
  561                      tet->GetMarkedFace(faces_info[lface].Elem1Inf/64, v);
  562                      // flip the shared face in the processor that owns the
  563                      // second element (in 'mesh')
  564                      {
  565                         int gl_el1, gl_el2;
  566                         mesh.GetFaceElements(i, &gl_el1, &gl_el2);
  567                         if (MyRank == partitioning[gl_el2])
  568                         {
  569                            std::swap(v[0], v[1]);
  570                         }
  571                      }
  572                   }
  573                   break;

At the moment I am implementing construction of 3D mesh from a 2D mesh, and actually I am able to create elements, boundary elements and vertices, as well as all shared entities with proper ordering. But the ParMesh object is still incorrect for the case of several mpi processes and I think this is because I don't do anything with refinement flags.

How should I set the refinement related structures in the ParMesh for a tetrahedral case?
What should I look at when considering parallel case? With that, the serial mesh generation works fine. So the error resides somewhere in parallel part of the structure but likely not in the shared entities.

I would appreciate any help!
Thanks,
Kirill


Fix PROMETHEUS_CONTAINER_PORT for Python3

Fixes #7

Looks good to me. Thanks for fixing this!

Thanks for merging!


[DOCFIX] Fix alluxio presto sandbox docs

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/Alluxio-Pull-Request-Builder/8544/
Test PASSed.

please help merge and cherry pick to 2.2 branch @ZacBlanco

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.2 please

@ZacBlanco can i borrow your powers again to cherry pick this to branch 2.1?

alluxio-bot, cherry-pick this to branch-2.1 please


Implemented more complex seach capabilities using Mango (MongoDB like) query syntax

Issue #16 asks for more complex queries.
The solution
Using the solution in this PR, complex queries is written as a PouchDB selector, which is the same as CouchDB selector, which is a Mango query language, which is inspired by MongoDB query language.
Query parsing and rows matching are performed by using pouchdb-selector-core. All comparison operators ($eq, $ne, $gt, $lt, etc.) and logical operators ($and, $or, etc.) have been implemented by pouchdb-selector-core.
The reasonings
Reasonings of using MongoDB query language and pouchdb-selector-core:

Current way to write search query still works. Performing search with ?search={column1:value1,column2:value2} still works, and it is the same as ?search={column1:{$eq:value1},column2:{$eq:value2}}. It means it retains backward compatibility.
MongoDB is popular, thus it's query language is familiar. In my opinion, for Stein case, it is better to write the query like this than in other query language like SQL or defining our own custom query format.
pouchdb-selector-core is used internally by PouchDB and it is well maintained. PouchDB is a mature library and I have not yet met any problems using PouchDB selector. There are another MongoDB query matcher called mongo-parse, but it has security problem (script eval injection).

The implementation
I don't use existing match function in pouchdb-selector-core, which is matchesSelector, because it only perform match for one row. I think it is not efficient to process the query every row. The query doesn't change so it is better to only process the query once and then use it to filter the dataset. This is why I use massageSelector and filterInMemoryFields in the implementation because these two functions is performed matchesSelector.

One thing to note for users, should always be a string or null since data from Stein are always string or null even though it is a number. For example, ?search={"year":{"$eq":2020}} would not work because the value (2020) should be a string ?search={"year":{"$eq":"2020"}}.
Performing to string conversion for all field values is possible but difficult because using logical operators the query object can be deep.

One thing to note for users, should always be a string or null since data from Stein are always string or null even though it is a number. For example, ?search={"year":{"$eq":2020}} would not work because the value (2020) should be a string ?search={"year":{"$eq":"2020"}}.
Performing to string conversion for all field values is possible but difficult because using logical operators the query object can be deep.


Update Documentation “test-doc”

Automatically generated by Netlify CMS

👷 Deploy Preview for elastic-yonath-a8509d processing.
🔨 Explore the source changes: 8b428daf82faff6c685ba7c196836dfd6c783b8a
🔍 Inspect the deploy log: https://app.netlify.com/sites/elastic-yonath-a8509d/deploys/618ccc7300712a0007a43a72


[FEATURE] Change text color of the profile views bubble to white

Description
I think the text color of the bubbles would be seen more clearly if it was white. I have attached a screenshot of both side by side for comparison
Screenshots

Additional information
No response

I would like to work on this one.

@JunDevHarsh Great but you should've discussed it first before opening a PR.
no worries, just a suggestion from me.

Hello @HarshDeep61034,
I noticed that you had requested to work on Issue #3423 and I apologize for not reaching out to you before creating the pull request. I had taken the initiative to work on the issue as I saw that it was unassigned and I wanted to help move the project forward.
However, I understand that this might have caused confusion and I apologize for any inconvenience caused. I would like to request you to review the pull request and let me know if there's anything that needs to be fixed or if you have any concerns.

@JunDevHarsh  No problem 😊 👍
I also make a lot of mistakes daily.

Please check that the color contrast is more than 4.5:1 with white text on the orange background before making this change.

@EmmaDawsonDev Please take a look at this. Is this what you were referring to? I agree with you that the white text does not contrast well enough with the orange background.

@EmmaDawsonDev Please take a look at this. Is this what you were referring to? I agree with you that the white text does not contrast well enough with the orange background.

Yes, that’s what I was referring to.

@EmmaDawsonDev Please take a look at this. Is this what you were referring to? I agree with you that the white text does not contrast well enough with the orange background.

Yes, that’s what I was referring to.

I think this issue can be closed since the black text has a good contract ratio


Runtimes api endpoint 404

The API docs mention a runtimes api endpoint (for apps) yet we get a HTTP 404 when hitting it. A quick scan of the code suggests it has not been implemented.
Is the goal to use the other endpoints for apps, given that metrics are generic enough that an application/services split may not be necessary? Having looked at what cf-abacus does (and I like it a lot, I think we can get huge benefit from it) , making things just a little more abstract and not having a hard apps/services split in the model (and perhaps also not forcing the model to mirror CF so tightly) would make it more flexible as we try to build a cloud platform featuring not just CF, but other runtimes also (much like Bluemix)

The API docs mention a runtimes api endpoint (for apps) yet we get a HTTP 404 when hitting it.  A quick scan of the code suggests it has not been implemented.

Yes, the code doesn't yet implement all the APIs listed in the doc. The idea here is to evolve the APIs and implementation together with the community over time.

Is the goal to use the other endpoints for apps, given that metrics are generic enough that an application/services split may not be necessary

One goal is to get that kind of feedback and ideas from the community, so this is good feedback :smiley:
We had that applications/services split in our initial implementation for Bluemix, and that's one of the main reasons why we still have it in the doc. Then, as the service usage APIs evolved over time to support more service providers, they became more generic and you're right that they could easily be used for runtimes too now.
For example -- just brainstorming here -- if we renamed 'service_id' to a more abstract 'resource_id' or made it a service_id | runtime_id choice in the schemas, and with a few other minor adjustments (we'd have to work out the details) we could probably make the same API work for runtimes as well.

Check the comments in eede09f which describe how we've now combined service usage and runtime usage into a single (simpler) API. Let us know if this does what you had in mind or if you have any more ideas on this.


Warning about broswer field in modules when taregting server

I get lots of the following warnings when bundling a nodejs app and I don't know how to silence them:
WARNING Library "mongoose" contains "browser" field. Set .target("browser") to avoid problems with your browser build!

{warnings : false} ?

Fair enough, but why is it warning in the first place if I already specify .target('server')? And won't I disable other potentially very useful warnings with {warnings: false}?

@mika-fischer you are right! we need to disable those for server builds


Add ParlAI to list of dialogue systems.

Add ParlAI to the list of dialogue systems.
Disclaimer: I am the tech lead on ParlAI.

Interesting, adding it. Thanks for your contribution.


Can't install with pip under python 3.12 but it works with manual installation (spoiler in description)

pygame-ce 2.4.0 (SDL 2.28.5, Python 3.12.2)
{'pyvidplayer2': '0.9.12', 'ffmpeg': '6.1.1-essentials_build-www.gyan.dev', 'pygame': '2.4.0'}
If I try pip install pyvidplayer2, I got this in the numpy install :
ModuleNotFoundError: No module named 'distutils'
as distutils have been removed from python, it's normal.
I have tried to change the requirements, like this (With the last version of numpy and with pygame-ce) :
numpy
opencv_python
pygame-ce
pysubs2
PyAudio
and I can run all the examples with pyvidplayer installed from sources.
pip install this packages versions :
Package       Version

numpy         1.26.4
opencv-python <IP_ADDRESS>
pip           24.0
PyAudio       0.2.14
pygame-ce     2.4.0
pysubs2       1.6.1
Very good library, thanks !

Hi,
Sorry for the super late reply. You are correct, the numpy version does cause an install error. The next patch will fix this. Thanks for telling me!

Version 0.9.14 fixes this problem.


Fixed bull dashboard and use .env for redis config

This fix the issue reported in APPS-53
Here some additional instruction for the release note:

Environment variable: Configure the REDIS URI
Run npm install

@duboisp : The commit is not associated with any Jira number, is this what you want to do?

Also, I think we need to specify environment variable in the jira

Also, I think we need to specify what value for environment variable in the jira

That depend on the environment.
In the process of preparing the release note, we would add the instruction of which environment variable need to be set and we will prepare a config sample for our production environment.


make fails trying to parse docbook.xsl in another directory

I'm trying to build freerdp master with
cmake -DCMAKE_INSTALL_PREFIX=/usr \
            -DCMAKE_INSTALL_LIBDIR=lib \
            -DWITH_PULSE=ON \
            -DWITH_SERVER=ON \
            .
    make

from directory ~/Sources/AUR/freerdp-git. It fails with
[ 95%] Generating xfreerdp.1
warning: failed to load external entity "/home/miguel/Sources/julia/manpages/docbook.xsl"
cannot parse /home/miguel/Sources/julia/manpages/docbook.xsl

Why is make trying to parse files that are outside its own repository?

docbook.xsl is the stylesheet used  to create the manage which is usually part of a package called docbook-xsl. It looks like it is tried to build the manage even if docbook.xsl wasn't found.

I have docbook.xsl 1.78.1 installed, and a bunch of docbook.xsl files under
/usr/share/xml/docbook/.
On Thu, Jan 29, 2015 at 3:15 PM, Bernhard Miklautz<EMAIL_ADDRESS>
wrote:

docbook.xsl is the stylesheet used to create the manage which is usually
part of a package called docbook-xsl. It looks like it is tried to build
the manage even if docbook.xsl wasn't found.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeRDP/FreeRDP/issues/2341#issuecomment-72095900.

@mbaz could you add the part of your cmake output related to manages?
And you do have a "manpages/docbook.xsl" so the docbooks style for manages?

I'm sorry, I don't understand what exactly do you mean by 'the part of the
cmake output related to manpages'. Is it a file, or the output of the cmake
command?
I do have a manpages/docbook.xsl:
$ ls -l /usr/share/xml/docbook/xsl-stylesheets-1.78.1/manpages/docbook.xsl
-rw-r--r-- 1 root root 15230 Apr 17  2013
/usr/share/xml/docbook/xsl-stylesheets-1.78.1/manpages/docbook.xsl
On Fri, Jan 30, 2015 at 10:44 AM, Bernhard Miklautz <
<EMAIL_ADDRESS>wrote:

@mbaz https://github.com/mbaz could you add the part of your cmake
output related to manages?
And you do have a "manpages/docbook.xsl" so the docbooks style for manages?
—
Reply to this email directly or view it on GitHub
https://github.com/FreeRDP/FreeRDP/issues/2341#issuecomment-72220467.

Closing due to lack of feedback.


Tests folder structure should be traversed correctly

If you create a folder structure under <project>/tests/<some directory>/<testfile.js>  , the test cases are not recognized unless you change the conf/nightwatch.json with that specific directory. This will not be feasible at large scale when teams will like to create folders for their tracks.  We need to ensure that the folders are ignored and just .js files are recognized as valid test cases at any level under tests/

  srcFolders.forEach(function (folder) {
             ^
TypeError: Cannot call method 'forEach' of undefined
    at module.exports (/Users/agupt13/Documents/workspace/testarmada_magellan/magellan-performance/node_modules/testarmada-magellan/src/interop/magellan-nightwatch/get_tests.js:21:14)```


Unexport Panics testing helper. It was exported unnecessarily.

Fixes  #246.

Can you please also add yourself to A+C.

Done.

Thank you

Sure! Glad to help!


Reactive-gRPC 0.10.0-RC1

Hello. We've released Reactive-gRPC 0.10.0-RC1. Would you be willing to try it out and report any findings?
https://github.com/salesforce/reactive-grpc/releases/tag/v0.10.0-RC1

updated to 1.0 :)


Create the github pages for portal website

Create the github pages for portal website

Describing the project
Providing screen shots and features available
Features coming up

@AshanFernando @chamathpali @rehrumesh @charithsoori  I would like to work on this issue

@AshanFernando
Are there screenshots available currently? Do you want to use the same color scheme for the sign in page?

@ajkendal For the moment, lets use the same color scheme.


"Get-AzureADAuditSignInLogs" returns no more than 1040 entries, cant go more granular than a day

Running AzureADPreview v<IP_ADDRESS> on PowerShell 5.1 on Windows 10 20H1 19041.329.
I'm trying to get all signin logs for a customer of ours with 1000++ users in Azure AD. I'm trying to do "paging" by taking one day at the time, using -EndDate and -StartDate parameters, but even when only using a day at a time I get no more than 1040 entries.
# Get all signin logs
## Assets
$Now            = [datetime]::Now
$DaysBack       = [byte] 32
$Intervals      = [byte] 1
$AadSignInLogs  = [array]$()
$AadCurrentPage = [array]$()
## Get
while ($DaysBack -gt 0) {
    # Info
    Write-Information -MessageData ('Days back: {0}, interval (in days): {1}.' -f ($DaysBack,$Intervals))
    # Get logs
    $AadCurrentPage = [array](
        Get-AzureADAuditSignInLogs -All:$true -Filter (
            "createdDateTime gt {0} and createdDateTime lt {1}" -f (
                $Now.AddDays(-$DaysBack).ToString('yyyy-MM-dd'),
                $Now.AddDays(-$DaysBack+$Intervals).ToString('yyyy-MM-dd')
            )
        )
    )
    # Add to total
    $AadSignInLogs += [array]($AadCurrentPage)
    # Info
    Write-Information -MessageData ('Found {0} entr{1}.' -f ($AadCurrentPage.'Count'.ToString(),$(if($AadCurrentPage.'Count' -ne 1){'ies'}else{'y'})))
    # Calculate how many days back to check next time
    $DaysBack = $(if($DaysBack -lt $Intervals){0}else{$DaysBack-$Intervals})
}
## Remove unused variable
$null = Remove-Variable -Name 'AadCurrentPage' -Force
## Remove duplicates and sort by time
$AadSignInLogs = [array]($AadSignInLogs | Sort-Object -Property 'Id' -Unique | Sort-Object -Property 'createdDateTime' -Descending)

Gives me this output
Days back: 32, interval (in days): 1.
Found 0 entries.
Days back: 31, interval (in days): 1.
Found 0 entries.
Days back: 30, interval (in days): 1.
Found 1040 entries.
Days back: 29, interval (in days): 1.
Found 1040 entries.
Days back: 28, interval (in days): 1.
Found 1040 entries.
Days back: 27, interval (in days): 1.
Found 1040 entries.
Days back: 26, interval (in days): 1.
Found 1040 entries.
Days back: 25, interval (in days): 1.
Found 1040 entries.
Days back: 24, interval (in days): 1.
Found 1040 entries.
Days back: 23, interval (in days): 1.
Found 1040 entries.
Days back: 22, interval (in days): 1.
Found 1040 entries.
Days back: 21, interval (in days): 1.
Found 1040 entries.
Days back: 20, interval (in days): 1.
Found 1040 entries.
Days back: 19, interval (in days): 1.
Found 1040 entries.
Days back: 18, interval (in days): 1.
Found 1040 entries.
Days back: 17, interval (in days): 1.
Found 1040 entries.
Days back: 16, interval (in days): 1.
Found 1040 entries.
Days back: 15, interval (in days): 1.
Found 1040 entries.
Days back: 14, interval (in days): 1.
Found 1040 entries.
Days back: 13, interval (in days): 1.
Found 1040 entries.
Days back: 12, interval (in days): 1.
Found 1040 entries.
Days back: 11, interval (in days): 1.
Found 1040 entries.
Days back: 10, interval (in days): 1.
Found 1040 entries.
Days back: 9, interval (in days): 1.
Found 1040 entries.
Days back: 8, interval (in days): 1.
Found 1040 entries.
Days back: 7, interval (in days): 1.
Found 1040 entries.
Days back: 6, interval (in days): 1.
Found 1040 entries.
Days back: 5, interval (in days): 1.
Found 1040 entries.
Days back: 4, interval (in days): 1.
Found 1040 entries.
Days back: 3, interval (in days): 1.
Found 1040 entries.
Days back: 2, interval (in days): 1.
Found 1040 entries.
Days back: 1, interval (in days): 1.
Found 1040 entries.

Clearly I'm not getting all signin logs.
I've tried specifying time as well with the -EndDate and -StartDate parameters, but they don't seem to accept it.
Can you please add a way of either paging or returning more than 1040 objects with Get-AzureADAuditSignInLogs?

Found out that I can do time as well, like this for instance:
"createdDateTime gt 2020-06-29T12:00:00.000000Z and createdDateTime lt 2020-06-29T12:05:00.000000Z"

Think I'll use Graph API directly instead.

auditLogs/signIns

But please make it possible to get all results with the AzureAD / AzureADPreview module. Shouldn't be too much to ask.

Off course I had to try to solve this with the limitations/ design flaw that currently present. The customer I'm working on now requires intervals no greater than 10 minutes (!), unless I'll reach the 1040 default page size on Microsoft Graph API.
# Get all signin logs
## Assets
$DaysBack          = [byte] 3
$MinutesInterval   = [byte] 10
$AadSignInLogs     = [array]$()
$AadCurrentPage    = [array]$()
## Calculate
$ToDate            = [datetime] [datetime]::Now
$FromDate          = [datetime] $Now.AddDays(-$DaysBack)
$DaysBackInMinutes = [uint32] $([timespan]($ToDate - $FromDate)).'TotalMinutes'.ToString('0')
## Get
while ($DaysBackInMinutes -gt 0) {
    # Info
    Write-Information -MessageData ('Minutes back: {0} ({1}), interval (in minutes): {2}.' -f ($DaysBackInMinutes,$ToDate.AddMinutes(-$DaysBackInMinutes).ToString('yyyy-MM-ddTHH:mm:ss'),$MinutesInterval))
    # Get logs
    $AadCurrentPage = [array](
        Get-AzureADAuditSignInLogs -All:$true -Filter (
            "createdDateTime gt {0} and createdDateTime lt {1}" -f (
                $ToDate.AddMinutes(-$DaysBackInMinutes).ToString('yyyy-MM-ddTHH:mm:ss.000000Z'),
                $ToDate.AddMinutes(-$DaysBackInMinutes+$MinutesInterval).ToString('yyyy-MM-ddTHH:mm:ss.000000Z')
            )
        )
    )
    # Add to total
    $AadSignInLogs += [array]($AadCurrentPage)
    # Info
    Write-Information -MessageData ('Found {0} entr{1}.' -f ($AadCurrentPage.'Count'.ToString(),$(if($AadCurrentPage.'Count' -ne 1){'ies'}else{'y'})))
    # Calculate how many days back to check next time
    $DaysBackInMinutes = $(if($DaysBackInMinutes -lt $MinutesInterval){0}else{$DaysBackInMinutes-$MinutesInterval})
}
## Remove unused variable
$null = Remove-Variable -Name 'AadCurrentPage' -Force
## Remove duplicates and sort by time
$AadSignInLogs = [array]($AadSignInLogs | Sort-Object -Property 'Id' -Unique | Sort-Object -Property 'createdDateTime' -Descending)

I also ran into this issue but was limited to 1000 results.
A fix or at least a documentation update would be appreciated.
Test Environment Info
PSVersion
$PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.17763.1007
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0, 5.0, 5.1.17763.1007}
BuildVersion                   10.0.17763.1007
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Commandlet version
Get-Command Get-AzureADAuditSignInLogs |select CommandType,Name,Version,Source|fl

CommandType : Cmdlet
Name        : Get-AzureADAuditSignInLogs
Version     : <IP_ADDRESS>
Source      : AzureADPreview

System Version
Get-ItemProperty -Path "HKLM:\SOFTWARE\Microsoft\Windows NT\CurrentVersion"|select ProductName,ReleaseId,@{name="OS Build";e={$_.CurrentBuildNumber+"."+$_.UBR}}

ProductName           ReleaseId OS Build
-----------           --------- --------
Windows 10 Enterprise 1809      17763.1098

Issue Example
Create time value
$aMonthAgo=[DateTime]::Today.AddMonths(-1).AddHours(-4).ToUniversalTime().ToString("yyyy-MM-ddTHH:mm:ssK")

echo $aMonthAgo
2020-05-29T00:00:00Z

Create second time value
$rightNow=[DateTime]::Now.ToUniversalTime().ToString("yyyy-MM-ddTHH:mm:ssK")

echo $rightNow
2020-06-30T01:09:39Z

Run the query
$logs = Get-AzureADAuditSignInLogs -Filter "createdDateTime gt $aMonthAgo and createdDateTime lt $rightNow" -all $True

$logs | Measure-Object
Count    : 1000
Average  :
Sum      :
Maximum  :
Minimum  :
Property :

Results of report created via gui
$test = Import-Csv C:\Users\jared\Downloads\SignIns_2020-05-30_2020-06-29.csv

$test | Measure-Object
Count    : 250000
Average  :
Sum      :
Maximum  :
Minimum  :
Property :

Note:
I've also tested using -all $True without any filter, using -Top 1500 with a filter, using -Top 1500 without a filter, and using the command just on it's own.
Everything returns only 1000 results.

Hi @o-l-a-v
Thank you for your issue submission.  I have created a work item to track this issue and will be sending it to engineering.  I need to see how they want to address this.  It is possible we will need to create a conceptual document to address the use of this cmdlt.  At this point, I am going to go ahead and close this issue.  I will post an update as soon as I have any additional information.
Thank you!
Bill

Can we please leave the unresolved ongoing issue open until it's actually resolved?
Any way I approach this I cannot understand why starting an investigate into a problem would justify marking it as resolved.

@Jared-Harrington-Gibbs Unfortunately no.  This is not a documentation issue, this is an issue with the cmdlt.  The issues for this github repo are specific to documentation errors, misspellings, etc.  Support issues are not handled here.

That makes sense.
Thank you for the clarification on reasoning.

Just a heads up: This seems to work with latest version of AzureADPreview, v<IP_ADDRESS>.
Did following:
#Requires -Module AzureADPreview

# Assets
$DateTimeFormatInRequest = [string] 'yyyy-MM-ddTHH:mm:ss.000000Z'
$DaysBackInTime          = [byte] 10

# Get
$SignIns = [array](
    Get-AzureADAuditSignInLogs -All:$true -Filter (
        "createdDateTime gt {0} and createdDateTime lt {1}" -f (
            [datetime]::Now.AddDays(-$DaysBackInTime).ToString($DateTimeFormatInRequest),
            [datetime]::Now.ToString($DateTimeFormatInRequest)
        )
    )
)

# Stats
$SignIns.'Count'
$SignIns.'Status'.'ErrorCode' | Group-Object -NoElement | Sort-Object -Descending -Property 'Count'

Got 14 778 signins.


rechtspraak archive link seems to be dead

Hi
The rechtspraak archive link (http://static.rechtspraak.nl/PI/OpenDataUitspraken.zip) seems to be dead. Is there any alternative to get the data?

@ELotfi I don't think that's supported anymore. We have moved to downloading on the API made available which can be accessed through a python package developed here - https://github.com/maastrichtlawtech/extraction_libraries/tree/rechtspraak
The API documentation is available here - https://www.rechtspraak.nl/SiteCollectionDocuments/Technische-documentatie-Open-Data-van-de-Rechtspraak.pdf

Closing the issue as its related to Rechtspraak and not case law explorer.


vscode app bar in windows is not black anymore.

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:
Version: 1.68.1 (user setup)
Commit: 30d9c6cd9483b2cc586687151bcbcd635f373630
Date: 2022-06-14T12:48:58.283Z
Electron: 17.4.7
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044

OS Version: Win10 21H2

The issue:
the app bar was black before updating to the latest version 1.68.1, and now it's ugly.
Steps to Reproduce:

update to the latest version 1.68.1

@DarkHeros09 what part is the app bar? Can you clarify which part of the picture has changed?

in comparison before

:

it looks like you have enabled the native titlebar, please update your setting for window.titleBarStyle to custom

window.titleBarStyle

YES thank you, FYI i have two laptops and both of them had the same issue after updating vscode, and i didn't change any settings. it happened right after updating.
again thank you.

@DarkHeros09 that's very strange, i'll be keeping an eye for this