Dimmers not responding well

It looks like the dimmers are not set properly in the last release. See image below. When I ask Siri to set them to 25% it's looks like that is done but after that it's set back to 80%. When I do in manually slow is works fine.

@hcoppen

It works well for me on two different HC (HC 3 Lite and HC 2 Lite).
How do you set the poller period in config?
Is the problem related to one device or all dimmers?

It's the HC2 the poller is set to default and yes it seems de problem is with the most of the dimmers. But I have the idee that it sometimes does work. I didn't had this issue before I moved to de latest version. The lights don't even change to 25% en back to 80%. They just stay on 25% I think. So I think the command is not reaching the HC2

@hcoppen
It's hard for me to reproduce the error that occurs for you. I checked at home on various dimmers, both: manually controlling in Apple Home and by voice via Siri and it works without the slightest problem. I don't see a problem in the code either.
As for your settings, I have similar, I do not only have the add room name to device name option - I mention it because the fixes in the last version mainly concerned this option.

@hcoppen on 1.5.0 set brighness to some value e.g 50% and after that turn off this dimmer (turn off, not set value to 0) - is it working for you?

No that's right when I set it to 50% and I turn it of it stays on.

I checked with poller off and set to 2 s. I checked by controlling manually and using Siri. In any case, it works for me without a problem.
The only fact is that Siri sends two commands each time (turnOn / turnOff + value) and the manually control (in Apple Home) sends the turnOn / turnOff command only when you change the value from 0 to more than 0 or from more than 0 to 0. When you disable lamp by clicking on icon it will send command turnOff, but when you slide down in detail view it will send value 0. But still works for me without a problem.

What version of dimmers are you using maybe that's the problem

I also have, among others, 3.5 and they work without a problem.
And do you have the latest versions in iPhone / HomePod etc.?
Can you try with other poller period settings? 0 (disabled) or 10 sec.?

commando's are send but they go to 100% and then back to 25% in HC2 see image

If I turn poller off it works when going to 100% but when I go to 25% after it go's off

Yes I think it's with polling. When I set it to 10s it works the first time. But the second time it doesn't. But after 10s it works again.

I understand well that all commands are sent correctly, i.e. they go to the Fibaro HC? The problem is that a moment later it is sent as if a second command that reverses this value?

But I am still wondering why version 1.5.0 worked well with Siri. The only issue was the 50% and turning off. So there must be something else

https://github.com/ilcato/homebridge-fibaro-home-center/pull/189
This has changed between 1.5.0 and 1.5.1. The old code had a mistake - so the light cannot be turn off after changing the value. (What did you also say you couldn't do).

But why are there going 2 ‘on’ commands in 1.6.0 and only 1 ‘on’ command in 1.5.0.

But why are there going 2 ‘on’ commands in 1.6.0 / 1.51 and only 1 ‘on’ command in 1.5.0.

I don't think it's a problem at all. For me, it is also the case that when I control using Siri, it sends two commands and when manually in Apple Home, it is one and so in both cases it works well.

In your case, the commands are sent, right? And they reach HC? Only, moments later, the value is kind of reversed? Is the problem only about Siri? Does turning on and off also have this problem, or problem is only when setting the value?

Does setting the poller period to 0 (disabled) change something, helps?

I will get back to your question tomorrow. I found out that turn on and off by hand doesn’t work well every time. Sometimes it jumps to 100% but the lights are 25% then after 2 seconds it shows 25%. Other times it jumps to 100% and stays at 100% and also the lights burn 100%. And other times it works as it should work.
I will get back to you tomorrow. Thanks for now

@hcoppen
I did a PR that should:

solve the problem of blinking dimmers when setting values
solve the problem of double commands when controlling Siri

If the author of the plugin merge this PR and releases a new version (probably at the beginning beta) then please test.

Okay I will test as soon as PR is approved and beta is available. Thanks for now!

@hcoppen published 1.6.1-beta.0
@mkz212 see my comments on PR

I will test it this evening and give you feedback. Thanks

Hi changing the level works now with app and Siri. But I can't turn off the light anymore. Not by app and not by Siri. You can only turn it off by moving the level to 0. Als turning on the light will always move the level to 100%

@hcoppen Thank you. Indeed it is. I did PR to fix it. The next beta should be ok.
@ilcato Please merge and release next beta.

Ok I will test again

@hcoppen
Thank you.
When you will test it, please pay attention:

when setting the light brightness value (moving the slider) one command or several are sent? The light smoothly passes e.g. from 20% to 80% or, for example, it is set to 35% then 47%, etc. (it blinks) and only at the end it is 80%?
similarly with roller / blinds - moving the slider, for example from 20% to 80%, one command or several are sent? The roller / blinds smoothly changes positions or stops along the way?

With the current beta version the command is only sent when I release the slider after moving. So I go from 30% to 60% slowly it's only send when I release the slider at 60% 'Set 60%'. After 1 or 2 seconds it sends 'on' and then '60%' that's what your log says.

@ilcato Please merge and release next beta.

@mkz212, the new PR only changes the variable name.

@ilcato No. It add clearTimeout.

@hcoppen @mkz212 published beta.1

For me now it is working as expected.
There is a slight delay in setting the value but there is no blinking (in the case of dimmers) and jamming (in the case of blinds).
Unfortunately, Apple Home works in such a way that it sends commands when you move the slider, not only when you raise your finger.

It's is turning off now but when I turn it off and back on it always go's to 100%. So turning is off while it's 30% will go to 100% when I turn it back on

@hcoppen In version 1.6.0 it also happens like this.

@ilcato So far, I have come to the conclusion that when the dimmer is turned off, the value is set to 0%. And since it's 0%, it's like there's no value so it turns on 100%.

@ilcato  Okay, but is it not an option to use only 'turnOff' or 'turnOn' option on Fibaro when turning on or off and the 'setValue' option when moving the slider in homekit. POST /api/devices/{diviceID}/action/turnOn and turnOff should work. So by using this command it will remember its state. When using the slider in homeKit and the value is > 0 and < 99/100 u should use the setValue command.

@ilcato Okay, but is it not an option to use only 'turnOff' or 'turnOn' option on Fibaro when turning on or off and the 'setValue' option when moving the slider in homekit. POST /api/devices/{diviceID}/action/turnOn and turnOff should work. So by using this command it will remember its state. When using the slider in homeKit and the value is > 0 and < 99/100 u should use the setValue command.

This is not the point at all. The commands in HomeKit and in this plugin are correct.
The problem is that Apple stores two values: On (true/false) and Value (0-100%). If the Value is set to 0, it will set 100% at startup. Fibaro, on the other hand, complicated the case and Value uses only when the dimmer is on. If the Value is 0, it means that it is turned off. And the last value is stored elsewhere.
Example, when we turn of dimmer with value 73%:

in HomeKit, On will be false, Value will be 73%.
in Fibaro, Value will be 0. And somewhere else they store last used value to use when you turn on dimmer.

@hcoppen
You can check. Get API response for device:
Open in browser: http://fibaro-ip/api/devices/DEVICE-ID (replace FIBARO-IP with your Home Center IP and DEVICE-ID with device ID) and login.
Next turn on / off dimmer. You will see that it does not matter if you turn off through Homekit or Fibaro, the value will be 0.

I have HC 3 Lite and the latest firmware 5.150.18 and it happens like this. I will check on HC2 Lite. And you @hcoppen what HC and firmware do you have?

@ilcato Okay, but is it not an option to use only 'turnOff' or 'turnOn' option on Fibaro when turning on or off and the 'setValue' option when moving the slider in homekit. POST /api/devices/{diviceID}/action/turnOn and turnOff should work. So by using this command it will remember its state. When using the slider in homeKit and the value is > 0 and < 99/100 u should use the setValue command.

This is not the point at all. The commands in HomeKit and in this plugin are correct.
The problem is that Apple stores two values: On (true/false) and Value (0-100%). If the Value is set to 0, it will set 100% at startup. Fibaro, on the other hand, complicated the case and Value uses only when the dimmer is on. If the Value is 0, it means that it is turned off. And the last value is stored elsewhere.
Example, when we turn of dimmer with value 73%:

in HomeKit, On will be false, Value will be 73%.
in Fibaro, Value will be 0. And somewhere else they store last used value to use when you turn on dimmer.

Okay, clear. So after turning off the poller takes the value from Fibaro and that will be 0. Is it possible to know if only turnoff or turnon button is clicked in homekit? If so you could only use the /action/turnOn command then read the level from Fibaro and set it in HomeKit.

I have HC 3 Lite and the latest firmware 5.150.18 and it happens like this. I will check on HC2 Lite. And you @hcoppen what HC and firmware do you have?

latest 4.6.30

@hcoppen

We still don't understand each other.
Poller has nothing to do with it (it happens wit on and off).
The command also does not. But to make you know, the commands are currently being distributed well - recognise on, off, set value.
The problem is that at the start the value is always 0, and when you want to turn on with the value 0, it is logical that you do not turn on the value 0 so it set 100.
The solution is such that you have to find where Fibaro stores the value for the DISABLED (turned off) dimmer and use this value as the dimmer is turned off.

@hcoppen
We still don't understand each other:

Poller has nothing to do with it (it happens with turned on and off).
The command also does not. But to make you know, the commands are currently being distributed well - recognise on, off, set value.
The problem is that at the start the value is always 0, and when you want to turn on with the value 0, it is logical that you do not turn on the value 0 so it set 100.
The solution is such that you have to find where Fibaro stores the value for the DISABLED (turned off) dimmer and use this value as the dimmer is turned off.

regarding to your second point are you turning on the device by setting the value (POST  /api/devices/{diviceID}/action/setValue)? If so then it's clear you can't set 0 for turning on. Or do you use the POST /api/devices/{diviceID}/action/turnOn call?

@hcoppen
We still don't understand each other:

Poller has nothing to do with it (it happens with turned on and off).
The command also does not. But to make you know, the commands are currently being distributed well - recognise on, off, set value.
The problem is that at the start the value is always 0, and when you want to turn on with the value 0, it is logical that you do not turn on the value 0 so it set 100.
The solution is such that you have to find where Fibaro stores the value for the DISABLED (turned off) dimmer and use this value as the dimmer is turned off.

regarding to your second point are you turning on the device by setting the value (POST /api/devices/{diviceID}/action/setValue)? If so then it's clear you can't set 0 for turning on. Or do you use the POST /api/devices/{diviceID}/action/turnOn call?

It doesn't matter. If you make a setValue with a value of 0 it will be 100. If you make a turnOn with a value of 0, the effect will be the same. The problem is NOT in setting the value but in reading it!
Before this plugin set value it will first read value. For turned off dimmer it will always read Value = 0. This is because Fibaro in API, for turned off (only for turned off) dimmers, store last used value somewhere else - not in Value field. It use field Value only for enabled dimmers.

action/turnOn

But that's what I mean you have to POST /api/devices/{diviceID}/action/turnOn  with empty params {"args":[]} This will restore the last value of the dimmer in Fibaro

@hcoppen
The setValue command must be sent, otherwise when you change the brightness with the slider in Apple Home, the new value would not go to Fibaro. I don't know if it can be distinguished somehow, but it's certainly easier and more reasonable to just read the correct value from the API.

@hcoppen
The setValue command must be sent, otherwise when you change the brightness with the slider in Apple Home, the new value would not go to Fibaro. I don't know if it can be distinguished somehow, but it's certainly easier and more reasonable to just read the correct value from the API.

Yes, that's why I asked if you can distinguish between switching on (click on button) and the slider. When the slider is used you can use the setValue if you only click on button, to turn dimmer on, you could use the turnOn command

@hcoppen
The setValue command must be sent, otherwise when you change the brightness with the slider in Apple Home, the new value would not go to Fibaro. I don't know if it can be distinguished somehow, but it's certainly easier and more reasonable to just read the correct value from the API.

Yes, that's why I asked if you can distinguish between switching on (click on button) and the slider. When the slider is used you can use the setValue if you only click on button, to turn dimmer on, you could use the turnOn command

I don't know if it possible. And even if it is:

different way Apple Home sends command when you click on icon, different when you use slider, and different when use Siri.
in a situation when you change the value in the Fibaro application or using the button, the Home application will not retrieve the new value correctly.

@hcoppen If you would like to help, find where in the API, Fibaro stores the value but for the DISABLED Dimmer.

@mkz212, @hcoppen,  I didn't follow all the discussion but I think the problem is in the poller mechanism: when the dimmer is turned off the poller API update both the status of the On and Brightness characteristics --> Off and 0%.
The solution for me is to ignore the brightness update when the Off update is received (I'm assuming the Off update comes first). Let me check ...

@mkz212, @hcoppen, I didn't follow all the discussion but I think the problem is in the poller mechanism: when the dimmer is turned off the poller API update both the status of the On and Brightness characteristics --> Off and 0%. The solution for me is to ignore the brightness update when the Off update is received (I'm assuming the Off update comes first). Let me check ...

with poller disabled it also get wrong value

@mkz212, @hcoppen, I didn't follow all the discussion but I think the problem is in the poller mechanism: when the dimmer is turned off the poller API update both the status of the On and Brightness characteristics --> Off and 0%. The solution for me is to ignore the brightness update when the Off update is received (I'm assuming the Off update comes first). Let me check ...

with poller disabled it also get wrong value

For some reason, disabling poller is not working.

@mkz212, @hcoppen, I didn't follow all the discussion but I think the problem is in the poller mechanism: when the dimmer is turned off the poller API update both the status of the On and Brightness characteristics --> Off and 0%. The solution for me is to ignore the brightness update when the Off update is received (I'm assuming the Off update comes first). Let me check ...

with poller disabled it also get wrong value

For some reason, disabling poller is not working.

Maybe every time before plugin send set command it first get value?
I believe that best option is in getFunctions and in poller do something like: if dimmer is on get value from Value, else get from other field (that where fibaro store last used value)

Try beta.2, it seems to work

Beta 2 with disabled poller works.

I have poller enabled

Ok with enabled and disabled poller it works great!!
I don't sleep here half a night and I think how to solve it ... the boss comes and probably one hand, drinking coffee, wrote the code and works 😉👍

@mkz212, I opened an issue on the poller disabling mechanism not working. Probably it is something relating to the way you implemented the check on theHome Center availability.
I will also reduce the timeout for the brightness from 500 to 300ms.

Great work guys It also works for me! Just tested it!

@ilcato

I did PR - should fix disabling poller.
I tested with 300 ms for dimmers and for me it was to less - multiple values was send - so maybe 400 ms?
You can test less value for roller shutters / blind, now it is 1500 ms.

Great work guys It also works for me! Just tested it!

@hcoppen
Thanks for reporting bugs and for suggestions and for testing! Great job! 👍

I tested with 300 ms for dimmers and for me it was to less - multiple values was send - so maybe 400 ms?

@mkz212 is it still necessary the whole mechanism ? Isn't the initial issue caused by the poller? What about instead of ignoring multiple setValues in sequence blocking the pollerUpdate and reenabling it after the last change ? You can reuse the same timeout structure.

I tested with 300 ms for dimmers and for me it was to less - multiple values was send - so maybe 400 ms?

@mkz212 is it still necessary the whole mechanism ? Isn't the initial issue caused by the poller? What about instead of ignoring multiple setValues in sequence blocking the pollerUpdate and reenabling it after the last change ? You can reuse the same timeout structure.

I think it's not poller's fault. It's just that Apple sends too many commands when moving the brightness slider - it should only one at the end of the move, once you raise your finger, but it is not.

I think that it is correct to send update for each slider movement, after all this is the behaviour of a physical slider. And the Fibaro API is able to cope with multiple update. The problem is in the asynchronous and delayed feedback coming from the poller. If we disable poller updates while a slider is moving I think the problem disappear.

I think that it is correct to send update for each slider movement, after all this is the behaviour of a physical slider. And the Fibaro API is able to cope with multiple update. The problem is in the asynchronous and delayed feedback coming from the poller. If we disable poller updates while a slider is moving I think the problem disappear.

We need to test.

@ilcato

I tested and when I control directly through Fibaro it sends only one command - when I let go of the slider.
Sending several commands (when using Home slider) causes a blink.
I don't think that sending a few commands is ok.

That's why I would leave it as it is now. If Apple changes the action - so that the command is sent after lifting the finger - then we can eliminate timeout.

@ilcato
I personally definitely prefer to have moments of delay, but so that it does not blink and jam. And now when you fixed that it's not always set to 100%, it's already great!

I think it works great now guys! Even the RGBW controller works great now. Thanks for your quick support!

@ilcato
I personally definitely prefer to have moments of delay, but so that it does not blink and jam. And now when you fixed that it's not always set to 100%, it's already great!

The logic I proposed was already present in method setCharacteristicValue of FibaroAccessory class. I slightly modified it. Please test the new beta. Will close this issue after your feedback.

@ilcato 1.7.0-beta.0 seems to be ok for me! 👍 I did PR to quiet warnings.

@ilcato 1.7.0-beta.0 works for me too! 👍


Pprof

What this PR does / why we need it:
See #4540 for context

Defaults webhook's pprof to run on localhost if enabled, allows to disable/configure profiler for webhook (ensures that if webhook is run on a public address, pprof is not)
Adds an option to run pprof for cainjector
Allows to configure pprof's address for controller
All three components now have two flags to configure pprof - --enable-profiling (defaults to false), --profiler-address (defaults to localhost:6060)

Special notes for your reviewer:

The default address change for profilers and the fact that it no longer is enabled by default for webhook would be a breaking change for anyone currently running automation such as parka against the profiling endpoints. I think this is still worth doing- for webhook to avoid any risks associated with potentially exposing the pprof on a public address, for controller so as to make the configuration of pprof the same for all components

To try this out for any component:

enable profiling by passing --enable-profiling=true

kubectl port-forward deploy/ -n  6060:6060

observe profiling info at localhost:6060/debug/pprof

I've looked at how some other projects do this, here is an example of Grafana folks mentioning the same issue of wanting to avoid exposing pprof https://github.com/grafana/grafana/pull/36580

We should now be able to check https://github.com/jetstack/cert-manager/issues/4368 easier

Release note:

Breaking change: pprof now runs by default on localhost:6060 for webhook and controller, but only if explicitly enabled. Pprof can now be enabled also for cainjector. All three components have `--enable-profiling`, `--profiler-address` CLI flags to configure profiling. Thanks to @bitscuit for help with this!

Fixes #4542
Fixes #4540
Fixes  #4541

Just spotted this PR when rebasing my own - quick note, here you used Breaking change as part of the release note. However, if you use Action required as a prefix, then this will be automatically surfaced/highlighted at the top of the release notes so users don't have to dig through all the other items to see this one :)

Also, not too sure why this PR didn't get a milestone applied? 🤔 should have been handled by https://github.com/jetstack/testing/blob/226fb73b33ab23b2efeb389222fa8f8ed3ea303b/config/plugins.yaml#L69-L87 - does this go into v1.7?

Aha, I think this may have landed in v1.6, but there was no corresponding milestone for that release - the one that was created didn't have the v prefix: https://github.com/jetstack/cert-manager/milestone/27


refactor!: Rename command line flags for the sake of consistency

BREAKING CHANGE: renamed -c/--confdir to -cd/--configDirand -f/--file to -cf/--configFile
Signed-off-by: Leonard Goodell<EMAIL_ADDRESS>

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-rfid-llrp-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[x] I have opened a PR for the related docs change (if not, why?)
https://github.com/edgexfoundry/edgex-docs/pull/916

Testing Instructions
Build service
run device-rfid-llrp -h
verify flags in help have new names
run device-rfid-llrp --confdir something
verify error due to --confdir unknown
run device-rfid-llrp --configDir something
verify error due to file something not found
New Dependency Instructions (If applicable)

Codecov Report

Merging #130 (e79d3e1) into main (90ff275) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
- Coverage   41.06%   41.00%   -0.06%
==========================================
  Files          20       20
  Lines        9006     9006
==========================================
- Hits         3698     3693       -5
- Misses       4140     4143       +3
- Partials     1168     1170       +2

Impacted Files
Coverage Δ

internal/llrp/reader.go
40.13% <0.00%> (-1.11%)
:arrow_down:

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


Scrutinizer

Codecov Report

Merging #17 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #17   +/-   ##
=========================================
  Coverage     92.25%   92.25%
  Complexity      169      169
=========================================
  Files            27       27
  Lines           452      452
=========================================
  Hits            417      417
  Misses           35       35

Impacted Files
Coverage Δ
Complexity Δ

src/Business/Repository/ElasticsearchMigration.php
100% <100%> (ø)
14 <0> (ø)
:arrow_down:

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 65c982b...6f2b233. Read the comment docs.

Pull Request Test Coverage Report for Build 236

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 92.647%

Totals

Change from base Build 226:
0.07%

Covered Lines:
378

Relevant Lines:
408

💛  - Coveralls

Pull Request Test Coverage Report for Build 236

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 92.647%

Totals

Change from base Build 226:
0.07%

Covered Lines:
378

Relevant Lines:
408

💛  - Coveralls


GEOMESA-1767 Remote Transforms for HBase

This pull request implements Remote Transforms for HBase.
All tests for the HBase Datastore have passed.
Signed-off-by: Spandan Agrawal<EMAIL_ADDRESS>

All requested changes have been made!

merged, thanks!


Streams

Hello, I recently created a video about your package.
State and Navigation Management with GetX
I was wondering if you could critique it to let me know if I am understanding it right.
I also received a question. Do we need to close the stream in the state? Doesn’t StreamBuilder take care of disposing streams?
Great work! Thanks!

Hey, i'm not jonatas but i want to talk about it.
Thank you for making this video! it helps so much for people to start using it, and you showed the best part of get, the route management, that's the seller of this package heheh
Hope you try a little more the state management part! it is very good, and i loved it!

Hello, I recently created a video about your package.
State and Navigation Management with GetX
I was wondering if you could critique it to let me know if I am understanding it right.
I also received a question. Do we need to close the stream in the state? Doesn’t StreamBuilder take care of disposing streams?
Great work! Thanks!

Hi @tadaspetra, Initially, congratulations on the content.
I just saw your video and left a comment there. I just made a point in the Streams section.
Since GetX is entirely based on Streams, you can merge variables, add other Streams to a variable, transform Streams into a variable and do anything else that reactive programming allows you to do. In your example you created a StreamBuilder, and this is not necessary with Get. You could have eliminated many lines of code in this example.
Let's say the firebase is a given by a Stream, either from the firestore, or something else. You want to receive the amount and change it on the screen. Say you get a number from him, as you did in the example, you can just do this:
RxInt number = 0.obs;
number.bindStream(doc);
When the data arrives from Firebase, it will automatically enter a sink in GetX, and it will check the widget using number.value, and change it automatically. You could eliminate this StreamBuilder and this StreamController from your code. This is one of the points that GetX makes life easier, you can have dozens of data sources, and you will not need a StreamBuilder for each one, much less a StreamController. If "name '," number "," list "or anything from the server arrives, you don't need to nest dozens of StreamBuilders to change multiple data.
Remembering that since Controllers have lifecycles, you can sign this change in "onInit()"
Then take a look at the Workers too, they are amazing.

Awesome, thank you for the feedback. The first video was more to introduce people to the package, people seemed to really enjoy it, and want some more.
I'll make sure to look into all your points, and then do some more digging from your documentation, but in case I run into a roadblock is there a better way I can contact you other than creating an issue on GitHub 😂?

I'm not sure which media you use most (Facebook, Telegram, Twitter, Discord, Slack), but I just saw a post from you in a Facebook group and added you there.
I will create GetX's communication channels too, but you can contact me more directly over there, either by email (jonataborges01@gmail.com), or by any other means you prefer.
You can text me if you have any questions about the lib, or about using it.

@tadaspetra can i put a link to your video in documentation?

@tadaspetra can i put a link to your video in documentation?

Of course! I’m planning to create some more videos on GetX as well 😊

@Nipodemos  Do you also have any direct contact? You are one of lib's collaborators and we only talk for issues hahaha

Is there a slack/discord for GetX discussions? I like the package, reduces so much boilerplate but seems there is a lot of hidden functionality.

@jonataslaw i have Discord, @alan duarte#6930
My email is<EMAIL_ADDRESS>

Well, I think I can close this here

checkout this method for the stream with firebase link


fix:icon align when  datatype is of image

Description
Bug Link
I have raised this PR In-order to fix alignment of the icon in input component when datatype is selected as number  and position is selected as right
screenshot
Before issue is resolved , icon dissapears

After the issue is resolved

Cypress Testing

Cypress Testing video
https://github.com/user-attachments/assets/3898144e-e84e-48b8-9ff1-9f449007cf41

@appsmithorg/query-js-pod , Please review this pr if you have enough bandwidth.
Thanks in advance.

@saiprabhu-dandanayak this issue doesn't come under the purview of QueryJs Pod, tagging @rahulbarwal to take this forward!

Hey @saiprabhu-dandanayak, Sorry for the delay on this PR.
It has some lint issues, please check and fix:
https://github.com/appsmithorg/appsmith/actions/runs/10988694185/job/30505541681?pr=36473#step:9:4455
Run yarn run lint:ci to confirm if issues have been fixed.

Hi @rahulbarwal , fixed liniting issue and updated the pr , could you pls check this.
Thank you.

@saiprabhu-dandanayak client-lint is failing for latest changes: https://github.com/appsmithorg/appsmith/actions/runs/11032799313/job/30642535328?pr=36473#step:9:4550
Please fix and run yarn run lint:ci to confirm if your fix is working.


bring Auth0 back into app

Currently it is set up..code is there and ready to go but need to integrate it into the pages

done via #15


Great extension but missing all the form snippets!

Kinda weird because all of the other snippets has the form_for but lacking in everything else

@hridoy I'm reporting a bug the form works only with .rb and not with .erb extensions


Hide records from grid based on a field value

Hi;
I would like to hide records from user on a condition that a specific field has a specified value like 1 for example. This is in addition to multi-tenancy but it maybe possible through multi-tenancy as well.  But i want the records to be viewed by admin users.
Thank you

in xyz.Repository
 private class MyListHandler : ListRequestHandler<MyRow> {
            protected override void ApplyFilters(SqlQuery query)
            {
                base.ApplyFilters(query);

                if (!Authorization.HasPermission("Admin"))
                {
                    query.Where(fld.FildName == 1);
                }
            }
        }

thank you @Zahar661


Error linking during bundle on MacOS

ld: unknown option: --start-group
clang: error: linker command failed with exit code 1
Apparently macos treats unknown command line options as errors. I don't know if this is a libv8 issue or mini_racer.
Isolating which revision this was introduced now.

looks like 1481e928bf9d0bb22397051e49bf7ebf6b942f17 works
and 7a6f6d3c92178855e5f2840ab79be821bcf979f3 does not.
Not sure why that commit would cause an error in the linker, but there it is-- and looks like it's not tied to a libv8 rev bump

Also, it only occurs when using mini_racer through bundler. rake clean compile test runs fine on latest when walking through the revisions.
However-- rake clean compile test fails when updating the gemspec to 6.0 beta -- so I don't know what's up there. :|
update: looks like it's related to https://github.com/cowboyd/libv8/commit/99774366a4c369e8383e208c907e52c740156c0f still don't know how it's showing up in 5.9 though.

Closing this, bundler was picking up the later versions of libv8 (as the dependency is not explicit, but a minimum and bundler did not activate the 5.9 and below versions)
this is an issue with 6.0beta libv8 and will be tracked in #65


Bluetooth: ISO: Remove bt_conn_unref for ISO deferred work

Removed the bt_conn_unref from the deferred_work function.
For ISO, the conn unref for the peripheral will happen in
the bt_iso_disconnected function. For the central, the
unref shall only happen when the CIG is terminated.
Signed-off-by: Emil Gydesen<EMAIL_ADDRESS>

This fixes an issue with the peripheral and central unref'ing one too many times, and thus either hits the __ASSERT in bt_conn_unref, or simply sets an invalid (-1) reference counter.

@jhedberg - are you able to ack?


<-- React warning -->

Hi I use react 15.3.1 with redux-devtools to build a universal web app. There's a warning from react says :
warning.js?8a56:36Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
 (client) v data-reactid="2"><div data-reactid="3"
 (server) v data-reactid="2"><!-- react-empty: 3 -

It seems because of the DevTools.js doesn't be wrapped, does anyone know how to handle this?
Here's my DevTools.js :
import React from 'react';
import { createDevTools } from 'redux-devtools';
import LogMonitor from 'redux-devtools-log-monitor';
import DockMonitor from 'redux-devtools-dock-monitor';

// createDevTools takes a monitor and produces a DevTools component
export default createDevTools(
  // Monitors are individually adjustable with props.
  // Consult their repositories to learn about those props.
  // Here, we put LogMonitor inside a DockMonitor.
  // Note: DockMonitor is visible by default.
  <DockMonitor toggleVisibilityKey="ctrl-H" changePositionKey="ctrl-Q">
    <LogMonitor theme="tomorrow" />
  </DockMonitor>
);

Is it because the react-dev-tools don't render on the server?
I came across this also and after finding a related issue I was able to come up with this to get it working:
https://github.com/peter-mouland/react-lego/compare/redux...redux-devtools

DevTools should be used under a flag, only on client and only on development.
In case you're using the extension, you can include it just by checking typeof window === 'object' && typeof window.devToolsExtension !== 'undefined'.


update FROM to centos7-s2i-nodejs

The centos7-nodejs repo is no longer supported.

@burrsutter see my email, but this is not the solution we are looking for. Sorry!


metadata_cache: fix LMDB stale reader issue

if a process crashed or was stopped abruptly, stale readers remain. So
we clean stale readers just after we open the database.

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Replace lookupdev with findalldevs

Turns out there's already an example of using findalldevs in pcap. It was in the function below list.
However, by removing lookupdev I realised that the case of no default device was not handled at all. I've currently handled this by returning a Results<Option<Device>, Error> as opposed to a Result<Device, Error>.
A possible alternative would be to translate the null option to an error, but that would be semantically different from what libpcap does which does not consider the null case to be an error.
Currently a draft PR, because I want to have a think if there's a slightly prettier solution.
Closes #158

I think the Result<Options<Device>, Error> return type makes more sense to me, especially since it matches what libpcap does. I'm not sure about a prettier way to do it, but the end result type works for me.

In this PR I remove wstr_to_string as the need for it came from a bug in the now obsolete pcap_lookupdev:

In WinPcap and Npcap, this function may return a UTF-16 string rather than an ASCII or UTF-8 string.
From https://npcap.com/guide/wpcap/pcap_lookupdev.html

This does not seem to apply to pcap_findalldevs on Windows: https://npcap.com/guide/wpcap/pcap_findalldevs.html
Therefore, this also closes #165.


DataFeeder to read serialized numpy arrays

Numpy arrays can be serialized to disk and it's possible to do random seeks into them.
Implementing a DataFeeder for such data format will remove requirement to have full dataset in the memory and still do random seeks for sampling of batches.

FYI, I am working on this.


This is not issue

Hi thanks for your opensource, is this cpi supports VMware vSphere 6.5, if not please let me know up to  which version of VMware vSphere it supports thanks in advance.

@abhilash07 Yes, the latest vSphere CPI supports vSphere 5.5, 6.0 and 6.5.
You can ask questions in the official bosh-cpi channel here: https://cloudfoundry.slack.com/messages/C094QMKB2


not installable on sf2 master

composer-yaml requires "symfony/yaml": "2.0.*"
looks like sf2 master ships a higer version

Is there any updates on this? I can not get the phar to work


post_measurements not working

solcast version: 1.0.4
Python version: 3.8
Operating System: Ubuntu

Description
RooftopSite.post_measurements() doesn't appear to work. Thanks to some kind help on the solcast forum, following three issues were identified:

url needs authentication
I didn't get it working with passing a dict to requests.post - rather, I had to pass a json string. Maybe there are ways to do that, but I didn't succeed.
headers={ ‘Content-Type’:‘application/json’ } is needed

What I Did
see forum thread for my trial and the code which eventually resolved it.
That said: Thanks for your providing the library in the first place. I'm new to requests, so it gave me a great starting point and woking get requests in a breeze.

@StefaE Thanks for catching this!
Per the request docs, changing the post in base.py L52 post(url, json=data)  should resolve it. Can you give that a try? PRs are welcome if you get a chance.

@StefaE Thanks for catching this!
Per the request docs, changing the post in base.py L52 post(url, json=data)  should resolve it. Can you give that a try? PRs are welcome if you get a chance.

nope - you need add authentication to the url (you do that for get, but not for post) and you need header type.
I'll make a code proposal later this week.

nope - you need add authentication to the url (you do that for get, but not for post) and you need header type.
I'll make a code proposal later this week.

I see the auth missing now. Please try out the branch I just created.

I see the auth missing now. Please try out the branch I just created.

After much trouble, v1.0.7 has been published to PyPI.

After much trouble, v1.0.7 has been published to PyPI.


Documentation

This is awesome! We'd really like to get it working.
Might it be possible to supply some documentation or explanation as to how to get this going?

I've setup a conf and iglu, but unsure what port and interface to use on Bitrise?
Read from configuration: interface: eth0, port: 9090
Starting server...
Waiting for the server to become available at eth0:9090...
nc: getaddrinfo: nodename nor servname provided, or not known


Setting fontSize in theme breaks Text color in iOS

Steps to recreate:

Create a new app
Update the theme with:

Text:
  bodyMedium:
    fontSize: 17

The text widget appears fine on web but is not visible on iOS. Setting color prop in theme fixes the issue.

@snehmehta this is a blocker, could you please take a look

Same as #609


Give option to turn off bell when focusing next terminal

I'm on Mac OS and using the integrated terminal. When I focus the next terminal using the keystroke workbench.action.terminal.focusNext or focusPrevious, the bell rings and I would like to disable it. I've tried removing the Enable Bell checkbox in the integrated terminal settings but it doesn't remove it.

If the bell is ringing and enableBell is off, it's something in your shell that's doing it.

Thanks, I'll look into it


Integrate llvm/llvm-project@989879f8fded

Reset third_party/llvm-project: 989879f8fded41c732db93864461b3a67b9f1501 (2023-06-29 11:55:36 +0000): [Clang] Allow C++11 style initialisation of SVE types.
Updated to openxla/stablehlo@ce932dd
Fixed LLVMFuncOp creation to support comdat
Fixed llvm.zext type mismatch verification
Fixed transform::PadOp copy back support after llvm/llvm-project@431c49d6

Thank you. I was thinking about doing one tomorrow but you beat me to it. I can probably take one next week if it helps.

Thank you. I was thinking about doing one tomorrow but you beat me to it. I can probably take one next week if it helps.

Thanks Stella! This week seems to be a smooth one for integrations. :)

Regression cases are noise to me.

Thank you. I was thinking about doing one tomorrow but you beat me to it. I can probably take one next week if it helps.

I'm the integrator next week. I'll start one on Wed. :)


IE8 support

Tried to get this working (#2), seems to be mostly issues with incompatibilities in the testing libraries themselves, but maybe I could at least make a good faith effort to get it working.

fixed by https://github.com/nolanlawson/pseudo-worker/pull/4


fix(docker): fix streaming of combined stdout/stderr

This PR fixes #2261. Multireader concactinates, not combines, the two
streams, this meant that we blocked on stdin if stderr was not
completely buffered.
In addition, this implementation ensures that we call Wait() on the
command, and any error is return by Close(). Godoc for
exec.Cmd.Std(out|err)Pipe, confirms that we do not need to close those
pipes.
This PR also ensures that we do not leak go routines in the even that
cmd.Start() fails.
Checklist:

[ ] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[ ] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[ ] I have written unit and/or e2e tests for my change. PRs without these are unlike to be merged.
[ ] Optional. I've added My organization is added to the README.
[ ] I've signed the CLA and required builds are green.

tested with:
kind: Workflow
metadata:
  name:  wftestthing.3
  namespace: argo
spec:
  entrypoint: build
  templates:
  - container:
      args:
      - "for i in $(seq 100000); do date > /dev/stderr;  done "
      command:
      - sh
      - -c
      image: alpine:latest
    name: build```


Symfony Bundle - Cache without APCu puts it in system temp dir instead of Symfony's cache dir

I just encountered a problem where I had two different repositories utilising similar types cause a weird error - the applciation crashed (even just bin/console) because it couldn't find a service that doesn't exist. That service existed in the other repository.
No matter how many times I cleared the Symfony cache in var/cache and searched for this missing service in the second repository, I couldn't figure it out.
I finally stepped through the code in xdebug and found that GraphqliteCompilerPass function getPsr16Cache() has this tidbit
            if (function_exists('apcu_fetch')) {
                $this->cache = new Psr16Cache(new ApcuAdapter('graphqlite_bundle'));
            } else {
                $this->cache = new Psr16Cache(new PhpFilesAdapter('graphqlite_bundle'));
            }

If the APCu extension isn't setup, it uses the PhpFilesAdapter which uses the FilesystemCommonTrait, that sets the cache dir with
        if (!isset($directory[0])) {
            $directory = sys_get_temp_dir().\DIRECTORY_SEPARATOR.'symfony-cache';

Tl;DR: The cache gets set to /tmp/symfony-cache which is not configurable from what I can tell. This bundle should be setting the cache to the default Symfony cache directory.
Is there something I'm missing?

This would be a really simple fix I think?
diff --git a/start/server/vendor/thecodingmachine/graphqlite-bundle/DependencyInjection/GraphqliteCompilerPass.php b/start/server/vendor/thecodingmachine/graphqlite-bundle/DependencyInjection/GraphqliteCompilerPass.php
index 3d03c26..261b199 100644
--- a/start/server/vendor/thecodingmachine/graphqlite-bundle/DependencyInjection/GraphqliteCompilerPass.php
+++ b/start/server/vendor/thecodingmachine/graphqlite-bundle/DependencyInjection/GraphqliteCompilerPass.php
@@ -91,12 +91,18 @@ class GraphqliteCompilerPass implements CompilerPassInterface
      */
     private $annotationReader;

+    /**
+     * @var string
+     */
+    private $cacheDir;
+
     /**
      * You can modify the container here before it is dumped to PHP code.
      */
     public function process(ContainerBuilder $container): void
     {
         $reader = $this->getAnnotationReader();
+        $cacheDir = $container->getParameter('kernel.cache_dir');
         //$inputTypeUtils = new InputTypeUtils($reader, $namingStrategy);

         // Let's scan the whole container and tag the services that belong to the namespace we want to inspect.
@@ -442,7 +448,7 @@ class GraphqliteCompilerPass implements CompilerPassInterface
             if (function_exists('apcu_fetch')) {
                 $this->cache = new Psr16Cache(new ApcuAdapter('graphqlite_bundle'));
             } else {
-                $this->cache = new Psr16Cache(new PhpFilesAdapter('graphqlite_bundle'));
+                $this->cache = new Psr16Cache(new PhpFilesAdapter('graphqlite_bundle', 0, $this->cacheDir));
             }
         }
         return $this->cache;


Create travis file

This is a basis for travis integration, currently it just does ./gradlew build test.
It however fails on codestyle.

@KilianSteenman , @NielsMasdorp told me to ask you for ideas on what commands to run.


Update list of repos

@terrajobst This is ready for another look.

@scottaddie Some repos are inconsistent with the previous announcement

I'm going through some stale PRs here. @scottaddie can you review the merge conflict here?

@scottaddie friendly ping 😉

@mairaw Merge conflict resolved.
@terrajobst Can you please take another look and merge if it still looks good?

Pinging @terrajobst for a review


Use instance fields for calculations

Can I use the instance fields to calculate variables for a given type? And if so, how?
So far I only see event data being used ($event.[value].double), but what if I need a static value?
Something like $instance.[fieldname].Double would help but it doesn't seem to work like that...

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

ID: 721c4d80-9266-8804-5b67-d13863912067
Version Independent ID: 66ca3e80-3ec4-9448-432a-b8be9488563a
Content: Time Series Model - Azure Time Series Insights
Content Source: articles/time-series-insights/time-series-insights-update-tsm.md
Service: time-series-insights
GitHub Login: @deepakpalled
Microsoft Alias: dpalled

@mwlootens Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Hey @mwlootens ,
Currently, we don't allow the use of static fields on the instance as part of calculations on the variable. We've heard this feedback from other users as well and we will definitely consider it as part of our roadmap!
Can I ask what your use case is for this feature? What kinds of calculations are you performing and what information would you typically store as a static value in instance fields?

I could for example give each time series a static maximum value for a parameter and would use this to calculate the percentage for each event.

@mwlootens thanks for the reply. Would you mainly use the instance fields to store static values for calculations (i.e. in variables)?

@mwlootens thanks for the reply and your feedback! Like I said above, we'll consider this as part of our roadmap 👍

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@mwlootens I have created Uservoice entry: https://feedback.azure.com/forums/906859-azure-time-series-insights/suggestions/40082614-use-instance-fields-for-calculations for this request. please provide your feedback and upvote for the same. It will also enable other users to upvote and provide feedback.


Readme update

The readme currently recommends Akephalos as the best HtmlUnit driver, which is probably not accurate any more given how long that project has been idle.

I'm a little sad to be merging this, but you're right :(

@jslag Check this out https://github.com/Nerian/akephalos2


Remove eslint-loader, for performance

Build times are insane right now.  Mostly due to eslint-loader:
With: webpack built in 38492ms
Without: webpack built in 13282ms
:scream: 65% reduction in build times without it!

Let it be so.
👻

🐎


Exclude dist from repository

We should include output folder dist on the repository.
Add to .gitignore and .npmignore
Spawned off #294

We originally included it to make sure the minified build is distributed, but there's no need to include in the repo and rather have it ship with the npm package.

Looks like dist is already on .gitignore (it still got committed at some point in the past) and .npmignore excludes dist/*.map.
Was the comment meant to exclude just the minified build from npm, or the dist folder as a whole, @haadcode ?

Thank you @ricardojmendez for following up on this!
To clarify:

We should exclude dist/from the git repo (as these are the files produced by npm run build)
We should include dist/orbitdb.min.js in npm package as it's the "browser distributable"
We should exclude dist/*.map and dist/orbitdb.js (sourcemaps and debug build respectively) as they're fairly large and only needed for development purposes

I think that's it, hope I didn't miss anything.

Ok, changed the npmignore to exclude the debug build here https://github.com/orbitdb/orbit-db/commit/f1b2f2252fa028a7da64e5713210fb2f8e2062d1 along with a note about the ES5 build.

Cool, I was just about to submit a PR for that.

Sorry, I didn't see the update :/

No problem. :) Just pushed it but was still running my tests locally. I noticed you excluded orbitdb.debug.js on master and wanted to check if that was built anywhere, since I hadn't found where.

@ricardojmendez can you PR it nevertheless? I didn't remove the es5 build yet.

Done!

Thank you ❤


need an example using concat primitive in the model

I could not find an example of concat primitive, the documents are not clear as well.

Hi @lzhengchun, thank you for the question.
The closest thing to what you are looking for will be concat gtest. If you are running in specific issues with concat we would be glad to look at the code.

Hi @lzhengchun,
You can also take a look at simple examples I made some time ago.
Unfortunately, they are based on older Intel MKL-DNN v0.x, while the current version is v1.1. Version 1.x brought some incompatible changes (here is the transition guide), but it should be relatively easy to tweak the examples to make them work with the latest version of DNNL / Intel MKL-DNN.
The example was written for #382 issue that you might also find helpful.

It would be nice to actually have these in the main repo and the developer guide. @louie-tsai, is this something that you may be interested in doing?

Hi @lzhengchun, thank you for the question.
The closest thing to what you are looking for will be concat gtest. If you are running in specific issues with concat we would be glad to look at the code.

Thanks @vpirogov that gtest is helpful. I have succeeded!


compiling error

Hello, can you help me please with this issue? During the compiling I see this error message.
In file included from C:\Users\xxx\Documents\Arduino\sketch_jun13a\sketch_jun13a.ino:36:
c:\Users\xxx\Documents\Arduino\libraries\NeoPixel-Nudge-main/embHTML.h:20:12: error: redefinition of 'const char PAGE_INDEX []'
20 | const char PAGE_INDEX[] PROGMEM = R"=====(
|            ^~~~~~~~~~
In file included from C:........\Documents\Arduino\sketch_jun13a\sketch_jun13a.ino:17:
c:....\Documents\Arduino\libraries\NeoPixel-Nudge-main/embHTML.h:20:12: note: 'const char PAGE_INDEX [7974]' previously defined here
20 | const char PAGE_INDEX[] PROGMEM = R"=====(
|            ^~~~~~~~~~
exit status 1
Compilation error: exit status 1
Thank you for your help!

Hi @ondras76
well, I guess it is an issue with multiple definitions of the same; the only thing that happens is that it places the code in FLASH see https://www.arduino.cc/reference/en/language/variables/utilities/progmem/ . So feel free to use any other char* variable, readjust the code there where it references it, and you should be able to move forward.
Regards,
R.

Thank you,I was finish the project in esphome, is much more easy for me.Ondrej.3. 7. 2023 v 16:31, Radim Keseg @.**>:
Hi @ondras76
well, I guess it is an issue with multiple definitions of the same; the only thing that happens is that it places the code in FLASH see https://www.arduino.cc/reference/en/language/variables/utilities/progmem/ . So feel free to use any other char variable, readjust the code there where it references it, and you should be able to move forward.
Regards,
R.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


Geolocator.getCurrentPosition() is not working on Web as stablished on docs

I'm trying get the current position of the client when the app is running on a web browser. The issues are:

checkPermission() is failing on web, returning an error with the following message: UnsupportedError (Unsupported operation: Platform._operatingSystem): "Platform._operatingSystem". This causes the application to throw (or not doing anything at all, it just breaks out of the current flow) instead of returning LocationPermission.denied as docs suggests.
getCurrentPosition() is having the same behavior described on the past bullet, but could not mange to get the any error message.

I'm wondering if someone has a working example of this working on Web platform. Could this be a bug? Thanks for the help.
Code I'm using:
Future<String> getCountryCode() async {
    Geolocator geo = Geolocator();
    if (!kIsWeb) {
     //Mobile...
      var status = await geo.checkGeolocationPermissionStatus();
      if (status.value == 2) {
        Position position = await Geolocator()
            .getCurrentPosition(desiredAccuracy: LocationAccuracy.high);
        final coordinates = Coordinates(position.latitude, position.longitude);
        var addresses =
            await Geocoder.local.findAddressesFromCoordinates(coordinates);
        var first = addresses.first;
        return first.countryCode; // this will return country code
      } else {
        return 'CR';
      }
    } else {
    //Web...
      Position position =
          await geo.getCurrentPosition(desiredAccuracy: LocationAccuracy.high); // <- Application throws here
      // var status = await geo.checkGeolocationPermissionStatus(); <-It also throws when asking for the permissions
      final coordinates = Coordinates(position.latitude, position.longitude);
      var addresses =
          await Geocoder.local.findAddressesFromCoordinates(coordinates);
      var first = addresses.first;
      return first.countryCode;
    }
  }

We are also having an issue with version 9.0.2 where the positionFuture in getCurrentPosition seems to never resolve in this line of code:
https://github.com/Baseflow/flutter-geolocator/blob/08c1f285b0ed44fa0cd580e9ff9697df9407b31c/geolocator_android/lib/src/geolocator_android.dart#L121

@bmonson that file seems to be related to the android handler for geolocator. Is it happening on Android/Mobile devices as well? For me it's working on mobile (tested on Android, not iOS), the portion of code I provided after the //Mobile... comment works fine.

i'm also having the same issue as @Retr0sec7 , for me it works on mobile/android but in web it breaks when calling  Geolocator.getCurrentPosition()

@Retr0sec7, the method checkGeolocationPermissionStatus() is not a method supported by the current versions of the geolocator package. This method was removed in version 6.0.0 of the Geolocator plugin. If you are still using this I recommend updating to the latest version (9.0.2). Since web support is only added in version 6.2.0 of the geolocator plugin this would explain the errors you receive.
Please add some additional information to your question:

The version of the geolocator plugin used.
Provide the output of the flutter doctor -v command.


Adds new families

I need some more of families and units. Maybe you can merge this into master if i didn't do so much bullshit :smiley:

@nidup @nono-akeneo any feedback here?

Hey @fitn,
thanks for the comments.
the symbols came from our provider attribute provider or google :smile:

@fitn any update?


dev environment

As a developer, I want my local development environment setup to be easy and automatic, so that I don’t have to spend the entire day following a list of instructions. The fewer the commands, the better.

As a developer, my local development environment should run the same exact OS configuration as stage and prod environments, so that we don’t run into “works on my machine” scenario’s.

As a developer, I want the ability to log into the local dev server / container, so that I can debug things if necessary.

As a developer, I want to work on files local to my host filesystem, so that the IDE I am working in is as fast as possible.

As a developer, I want the files that I change on my localhost to automatically sync to the guest filesystem that is running my development environment, so that I do not have to manually push or pull files to the local server.

I believe 1-3 is covered by docker-compose up, if you think otherwise let
me know.
4-5 can be achieved on Linux and with the newer docker beta on Mac and
Windows too, I'll work on it next week and update the instructions if
necessary.
On Thursday, 16 June 2016, purificant<EMAIL_ADDRESS>wrote:

As a developer, I want my local development environment setup to be
easy and automatic, so that I don’t have to spend the entire day following
a list of instructions. The fewer the commands, the better.
2.
As a developer, my local development environment should run the same
exact OS configuration as stage and prod environments, so that we don’t run
into “works on my machine” scenario’s.
3.
As a developer, I want the ability to log into the local dev server /
container, so that I can debug things if necessary.
4.
As a developer, I want to work on files local to my host filesystem,
so that the IDE I am working in is as fast as possible.
5.
As a developer, I want the files that I change on my localhost to
automatically sync to the guest filesystem that is running my development
environment, so that I do not have to manually push or pull files to the
local server.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openh2labs/elastic-erga/issues/25, or mute the thread
https://github.com/notifications/unsubscribe/AF7V-X6iMk9lBi4tMIlcPcmm1_tFDpjSks5qMU_EgaJpZM4I3YsP
.

--
Konstantinos Konstantinidis
Ianos Ltd - Company No: 08121271 - Registered Office: 145-157 St John
Street, London EC1V 4PW


FullLink Scala.js-specific minifier

Since modules are now supported and work great with scala.js, but don't work great with the Closure compiler, and apparently won't ever work with it.  I think it would be great to have some sort of simple property name minifier.  Current, good javascript minifiers do well with ES imports and are able to tree shake most of the unused stuff. They do well with class name minification as well.  However, where they fail is property minification.
In scala.js we have access to a lot more semantic information so it might not be too tough to create a simple minification of property names, I think it would drastically reduce the output javascript size.  Today the code is pretty compressable but it could be a lot better.
Another option could be some guidance on how to pass the right parameters to something like Terser if there were some conventions that were universally applicable. https://terser.org/docs/api-reference.html#mangle-properties-options

Explanation of how mangling works: in terser: https://terser.org/docs/cli-usage#cli-mangling-property-names-mangle-props

I tried this and got a nice size reduction. However (I think this is expected) it doesn't work with dynamic loading. I think it won't know what names to call on the other js files if they are mangled independently

@cquiroz if you are talking about js.dynamicImport it worked for me with webpack 5.  This is bundler specific, because it's highly dependent on how the bundler processes files, and nameCache being the "big" feature that makes it work.

@cquiroz if you are talking about js.dynamicImport it worked for me with webpack 5. This is bundler specific, because it's highly dependent on how the bundler processes files, and nameCache being the "big" feature that makes it work.

It works but not ideal, still need a solution for mangle properties reserved and regex.

I've had a look at the terser options, and it seems we should be able to make it work with Scala.js:
Doc: https://terser.org/docs/api-reference#mangle-properties-options

It supports closure style property renaming based on quoting (mangle.properties.keep_quoted: "strict")
It supports rewriting an existing source map.
It supports not renaming top-level definitions.
It supports reserved identifiers (which we might need for global scope idents in modules).
It supports reserved properties (which we might need to avoid name clashes with exports).
It supports multiple files as input.

Unclear to me is how to best integrate. To integrate it externally (based on the linking report only), we need to expose quite a bit of the guts of the linker:

optimizeBracketSelects = false
trackAllGlobalRefs = true (and add to report)
assemble all required exported properties (unclear whether these need to be all to avoid clashes, or only the ones for the corejslib).

On the flipside, we wouldn't have to do anything for sourcemaps.

Actually, multiple modules may also work with Clojure Compiler, there is --chunk option for some time already (former is --module), more details are in this stackoverflow answer. Basic idea is to specify multiple output files (modules) along with their dependencies:
--js commonfunctions.js
--chunk common:1
--js page1functions.js
--js page1events.js
--chunk page1:2:common
--js page2function.js
--chunk page2:1:common

and Scala.js has knowledge about inter-module dependencies, right?

That's not the problem. We've known about that for a while.
The problem is mainly that GCC refuses to consider and preserve import statements to "the outside world" that is not under its control.

Is this one of the issues? https://github.com/google/closure-compiler/issues/3707

All the details are already in #3893.

@sjrd I was thinking of this limitation even when module kind is CommonJS.


Psalm crashes: Could not get class storage for

Describe the bug
When running psalm, an error occurs with the message:  Could not get class storage for. This is happening in the ClassLikeStorageProvider, here: https://github.com/vimeo/psalm/blob/master/src/Psalm/Internal/Provider/ClassLikeStorageProvider.php#L41-L48. This method is getting called from here: https://github.com/vimeo/psalm/blob/master/src/Psalm/Internal/Analyzer/Statements/Expression/Call/Method/AtomicMethodCallAnalyzer.php#L339-L341
It seems that the $fq_classlike_name parameter of the get method is null. I've dumped $class_storage and it represents \Illuminate\Database\Eloquent\Builder, that's why I'm creating the issue here and not in the main psalm repository.
When running vendor/bin/psalm --debug-by-line this piece of code, especially the query bit pops up:
return Cart::query()
  ->where('user_uuid', $this->user->getUuid())
  ->where('uuid', $this->session->get('cart_uuid'))
  ->exists();

Cart is just a regular Laravel model nothin special there.
Impacted Versions
barryvdh/laravel-debugbar             v3.4.1             PHP Debugbar int...
barryvdh/laravel-ide-helper           v2.8.0             Laravel IDE Help...
beyondcode/laravel-dump-server        1.4.0              Symfony Var-Dump...
fruitcake/laravel-cors                v1.0.6             Adds CORS (Cross...
inertiajs/inertia-laravel             v0.2.5             The Laravel adap...
laravel/framework                     v7.25.0            The Laravel Fram...
laravel/horizon                       v4.3.3             Dashboard and co...
laravel/scout                         v8.2.1             Laravel Scout pr...
laravel/tinker                        v2.4.2             Powerful REPL fo...
propaganistas/laravel-phone           4.2.4              Adds phone numbe...
psalm/plugin-laravel                  dev-master 95761f9 A Laravel plugin...
spatie/laravel-backup                 6.11.1             A Laravel packag...
spatie/laravel-db-snapshots           1.6.1              Quickly dump and...
spatie/laravel-event-sourcing         4.2.0              The easiest way ...
spatie/laravel-log-dumper             1.3.1              A function to du...
spatie/laravel-model-states           1.6.3
spatie/laravel-multitenancy           1.6.4              Make your Larave...
spatie/laravel-navigation             dev-master 90dc440 Manage menus, br...
spatie/laravel-query-builder          2.8.2              Easily build Elo...
spatie/laravel-schemaless-attributes  1.7.1              Add schemaless a...
spatie/laravel-settings               dev-master 1725356 Store your appli...
spatie/laravel-sluggable              2.5.0              Generate slugs w...
spatie/laravel-stubs                  1.1.0              Opinionated Lara...
spatie/laravel-tail                   4.2.1              Easily tail appl...
spatie/laravel-typescript-transformer dev-master 7a73107 Transform your P...
spatie/laravel-view-models            1.3.0              View models in L...
vimeo/psalm                           dev-master 9822043 A static analysi...
Any idea what the issue could be?

I notice you're on dev-master, it could be https://github.com/vimeo/psalm/issues/4013 .
Could you try updating psalm to the latest dev-master?

Hi @mr-feek,
That seems to fix it, although another error popped up, maybe we should stop using the master version 😬


Bug: Usage with middy httpJsonBodyParser

Expected Behavior
If you see the code snippet below, if you place the makeHandlerIdempotent call above the parser the middleware will fail if there is an error as when the middleware is going back up the chain on the other side the hash is different as httpJsonBodyParser has mutated the request.
So the expected behaviour is that makeHandlerIdempotent retains its own reference to the request when it receives it, so that when it creates the response other middleware do not affect it.
Current Behavior
If you use httpJsonBodyParser the makeHandlerIdempotent will not function if placed before the parser in the chain it will result in an error.
Code snippet
export const handle = middy(Sentry.wrapHandler(handler))
  .use(errorHandlerMiddleware())
  .use(httpHeaderNormalizer())
  .use(httpJsonBodyParser())
  .use(makeHandlerIdempotent({ persistenceStore, config: idempotencyConfig }))
  .use(schemaValidatorMiddleware({ schema: RequestSchema }))

Steps to Reproduce

Setup a lambda with makeHandlerIdempotent before httpJsonBodyParser
Call the lambda with the same request at the same time to trigger either a 409 or 500 from makeHandlerIdempotent
Observe error as it can't find the entry in the table to remove as the hash differs

Possible Solution
The middleware retains a reference to the hash created for the request so it isn't reliant on the request when the response is created.
Powertools for AWS Lambda (TypeScript) version
latest
AWS Lambda function runtime
22.x
Packaging format used
npm
Execution logs

Hi @andrewdavidcostello, thanks for opening this issue.
By looking at the code snippet you shared, I see you're using a number of middlewares that are not part of this library.
Could you share a bit more about what they do and what do you expect to happen?
Regarding this:

So the expected behaviour is that makeHandlerIdempotent retains its own reference to the request when it receives it, so that when it creates the response other middleware do not affect it.

I might be misunderstanding what's happening in the middleware stack (see above), but if any of the middlewares above mutate the event, then there's nothing we can do other than get the mutated version.
This is how Middy.js works.

Thanks for the reply, apologies the code snippet could be reduced to:
export const handle = middy(handler)
  .use(httpJsonBodyParser())
  .use(makeHandlerIdempotent({ persistenceStore, config: idempotencyConfig }))

Your assumption is correct the httpJsonBodyParser parses the body of the request mutating it.
As you currently for example allow the use of a local cache could the local cache (or similar) not also be used to store the hash of the incoming request (or via the key specified) for reference in the deletion, rather than rehashing the request that has now mutated?
I hope that makes sense.

Thank you for clarifying, it makes sense.
Unfortunately the local cache won't help in this scenario since its logic will run always after the payload has already been mutated, at least with the current middleware order.
To move forward, at least based on my current understanding, you have a couple options which you might want to consider based on why you're using the httpJsonBodyParser Middy.js middleware in the first place.
If you're using it so that you can then specify the idempotency key based on fields within the body of the request, then you can already do this without any additional 3rd party middleware, for example, assuming you have a request that looks something like this:
{
  "version": "2.0",
  "routeKey": "ANY /createpayment",
  "rawPath": "/createpayment",
  "rawQueryString": "",
  "headers": {
    "Header1": "value1",
    "X-Idempotency-Key": "abcdefg"
  },
  "requestContext": {
    "accountId": "123456789012",
    "apiId": "api-id",
    "domainName": "id.execute-api.us-east-1.amazonaws.com",
    "domainPrefix": "id",
    "http": {
      "method": "POST",
      "path": "/createpayment",
      "protocol": "HTTP/1.1",
      "sourceIp": "ip",
      "userAgent": "agent"
    },
    "requestId": "id",
    "routeKey": "ANY /createpayment",
    "stage": "$default",
    "time": "10/Feb/2021:13:40:43 +0000",
    "timeEpoch":<PHONE_NUMBER>723
  },
  "body": "{\"user\":\"xyz\",\"productId\":\"123456789\"}",
  "isBase64Encoded": false
}

You can use one of the JMESPath custom functions in our package to have the Idempotency utility pre-parse the body and allow you to reach into the parsed object:
const idempotencyConfig = new IdempotencyConfig({
  eventKeyJmesPath: 'powertools_json(body).["user", "productId"]',
});

export const handle = middy(handler)
  .use(makeHandlerIdempotent({ persistenceStore, config: idempotencyConfig }))

This is the example we have here in the docs, if you want to use the entire body I'd still recommend using the powertools_json() JMESPath function, so that the order of fields in the stringified body doesn't mess with your request like so:
const idempotencyConfig = new IdempotencyConfig({
  eventKeyJmesPath: 'powertools_json(body)',
});

If instead you're using the httpJsonBodyParser middleware so that your handler function has access to the parsed body, I'd just swap the order of the two middlewares and let the Idempotency one work with the non-mutated payload. Alternatively, I'd simply call JSON.parse(event) on the first line of the handler and still avoid having a dependency just for this (but this is just preference).
Depending on your use case, you might want to pick one or mix both the methods.

Again thanks for the reply and the detail!

If instead you're using the httpJsonBodyParser middleware so that your handler function has access to the parsed body, I'd just swap the order of the two middlewares and let the Idempotency one work with the non-mutated payload.

This is exactly the use case as the middleware removes a lot of boilerplate for each handler that requires it.
We could as you say approach this in a different way but I may not have explained well here (to see if Idempotency middleware could behave differently). So if we do switch the middleware and the Idempotency middleware runs against the non-mutated body this will lead to an error if the key in the idempotency table needs to be removed as the request object coming back out of middy to create the response has already been mutated and the hash generated is then different.
This was also using the powertools call in the event key as you have supplied but I will need to double check that one!
Maybe I just need to accept the two middleware's won't play nice 🙂

So if we do switch the middleware and the Idempotency middleware runs against the non-mutated body this will lead to an error if the key in the idempotency table needs to be removed as the request object coming back out of middy to create the response has already been mutated and the hash generated is then different.

Could you please elaborate more on what you mean by this? Possibly with a sample of the request / response?
I'm not sure this is a limitation of this middleware.

Sure might not be a quick reply on this one but will try to get something over within the week hopefully.


New package: SQLStrings v0.1.0

Registering package: SQLStrings
Repository: https://github.com/JuliaComputing/SQLStrings.jl
Created by: @c42f
Version: v0.1.0
Commit: 5be4600d09ac53b7a82bd8dc5f73aba59eca909d
Reviewed by: @c42f
Reference: https://github.com/JuliaComputing/SQLStrings.jl/commit/5be4600d09ac53b7a82bd8dc5f73aba59eca909d#commitcomment-46862046

Perhaps call it SQLStrings? Just a suggestion, but that seems to be the capitalization that is used by other packages:
$ rg -i sql ~/.julia/registries/General/Registry.toml
233:0aa819cd-b072-5ff4-a722-6bc24af294d9 = { name = "SQLite", path = "S/SQLite" }
388:1297d576-4826-11ea-035e-9995c721bad1 = { name = "SearchLightMySQL", path = "S/SearchLightMySQL" }
699:21a827c4-482a-11ea-3a19-4d2243a4a2c5 = { name = "SearchLightSQLite", path = "S/SearchLightSQLite" }
1175:39abe10b-433b-5dbd-92d4-e302a9df00cd = { name = "MySQL", path = "M/MySQL" }
1370:4327cdd6-4902-11ea-0272-430cea0431bd = { name = "SearchLightPostgreSQL", path = "S/SearchLightPostgreSQL" }
1778:57f923c0-76be-11e9-095d-c942eaffc94b = { name = "QuerySQLite", path = "Q/QuerySQLite" }
2384:76ed43ae-9a5d-5a62-8c75-30186b810ce8 = { name = "SQLite_jll", path = "S/SQLite_jll" }

Ok I changed the name for consistency, perhaps it's slightly better this way.
It's delightful that Julia Registrator was able to keep this same PR open and update it accordingly.  Thanks to whoever implemented that feature :-)

Could someone merge this? The waiting period is over, but our comments have blocked the automerge (I assume).


Invalid Syntax error in navgen.py

I'm attempting to run the navgen.py, but keep getting an error after I input the Cb Response URL.
Below is the syntax I get.  I have replaced the URL in this extract to the default example URL, but I get the exact same error when I add in our true URL:
[!] To find your API key, login to your Cb Response UI and navigate to the profile section.
[!] Here, enter the full url of your Cb Response instance. Example: https://bugcrowd.my.carbonblack.io
[] > https://bugcrowd.my.carbonblack.io
Traceback (most recent call last):
File "navgen.py", line 499, in
main()
File "navgen.py", line 456, in main
api_key, url = get_auth()
File "navgen.py", line 12, in get_auth
full_url = input("[!] Here, enter the full url of your Cb Response instance. Example: https://bugcrowd.my.carbonblack.io\n[] > ")
File "", line 1
https://bugcrowd.my.carbonblack.io
^
SyntaxError: invalid syntax

Looks like the script requires Python3.
My error was caused by running the NavGen script in Python2.7.

File "navgen.py", line 503, in
main()
File "navgen.py", line 469, in main
tid_dict = generate_tid_dict(data_from_cbr)
File "navgen.py", line 355, in generate_tid_dict
if "windows" in threat_report['tags'] or "linux" in threat_report['tags'] or "macos" in threat_report['tags']:
KeyError: 'tags'


Add a view to simplify proposal answer access in SQL queries

Description

Adds a new view for accessing a proposals answers.
Sample output of the SELECT statement on our dev db with some sorting applied:

For common cases; plain text inputs, radio buttons & number inputs where we only want the number, the first_value column pulls out the answer in an easy access text format. For anything more complex, json_value returns a json object (slightly extracted from) the answer table for processing.
Skipped patch number 146 as I saw that in use in #483
Motivation and Context

We've got a tool full of SQL queries to allow STFC staff to access select data from our various systems. Often this needs to pull in answers from the proposal system, and if you're starting with the proposal's ID then it's several joins to get from that to the proposals answers and the key of the question they're for. With this view it'd only need a single join. Applied over all our queries it'll add up to a lot of lines saved and simpler to read queries.
How Has This Been Tested

I've run the view's SELECT statement against our databases, and applied the patch to my local environment without issue.
Fixes

Closes https://github.com/UserOfficeProject/user-office-project-issue-tracker/issues/994
Changes

Depends on

Tests included/Docs Updated?

[ ] I have added tests to cover my changes.
[ ] All relevant doc has been updated

Turns out I had no idea how generic templates work, so missing that wasn't intentional, and the joins weren't quite right.

I've added the generic template title, as well as their natural key. The joins I'd been using were reporting non-template answers as part of each template, so I've replaced that with a UNION over 2 selects.

I've renamed the json column to avoid clashing with the name of a built in function in our trino SQL environment. That can be solved by adding quotes around the column name, but since this is a new view for convenience with no existing users to affect, lets just rename the column to be as easy to use as possible.


More stronger random tmp filenames.

It's better to have more stronger random tmp filenames, so duplication chance will be decreased.

Thanks for the PR. Have you encountered duplication in practice?

Thanks for the PR. Have you encountered duplication in practice?

You're welcome. Nope, not really.

I think there's a risk we're causing "too long file name" or broken cleanup-regexps for our users. I however don't think our users have any issue with duplicate names currently. Hmm...

Is there any test to check the aforementioned risk?

I would prefer we don't make this change if nothing currently known as broken, due to the risk @Tarrasch mentioned. And it is probably not worth the effort trying to mitigate the risk.

I would prefer we don't make this change if nothing currently known as broken, due to the risk @Tarrasch mentioned. And it is probably not worth the effort trying to mitigate the risk.

ok


[Feature] ExpirePartitionsAction support in flink

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
ExpirePartitionsAction support in flink
Solution
Add a action for ExpirePartitionsAction
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

done


Make use of "contacts" column in "schedules" table, it must contain the USER's CONTACT from the "users" table

When attempted to submit a new reservtion from STO user.... and yes successful store...
but checking the DB... i noticed column "contacts" is not utilized.

its working.... issue finished


linux/gnu: add utmpname()

This adds utmpname(3) on Linux with GNU libc.
Ref: https://refspecs.linuxfoundation.org/LSB_3.0.0/LSB-Core-generic/LSB-Core-generic/baselib-utmpname-3.html

r? @gnzlbg
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 8a480ed9d885ea1252191161cb457f8a494a64db has been approved by gnzlbg

:hourglass: Testing commit 8a480ed9d885ea1252191161cb457f8a494a64db with merge e61ec827bf1ac0c70033df9820f76ff02764062b...

:sunny: Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, status-azure
Approved by: gnzlbg
Pushing e61ec827bf1ac0c70033df9820f76ff02764062b to master...


Memory Leak？？？

I found a problem with Java wrapper class，such as ...
java code:
import java.util.ArrayList;
import java.util.List;
public class Runner {
public List testListInteger(){
List temp = new ArrayList<>();
for (int i = 0; i < 100000; i++) {
temp.add(i);
}
return temp;
}
}
python code:
from xxx import Runner
def test_run():
runner = Runner()
for num in runner.testListInt():
a = num
if name == 'main':
for i in range(1000000):
test_run()
I use the command “top -p pid” to observe the process memory usage, memory usage has been increasing. Only the String will be stable at a certain value. What should I do with it?

How is the memory usage if you run this in java only? So call testListInteger() recursively a million times in a java function. Paste both measurements (with jpype and with java only).
What do you mean with "only string will be stable"? And please paste code in a code block so it's easier to read.


It is no longer working with .cmd files

Environment
wt "# Development 3 dev.cmd"; split-pane -V "# Development 2 wds.cmd"; split-pane -H "# Development 1 ts-watch.cmd"

And 3 .cmd files:

Steps to reproduce
A few weeks ago I spent a lot of time trying to run commands ready to split windows, I tried hard to put the commands in the WT command itself, but the only way I could do it was by calling external command files, and now neither method is working
Expected behavior
I had taken a print when it worked, it looked like this:

It was making my life a lot easier, I was full of open terminals before.
Actual behavior

Translation of errors:
'"# Development 3 dev.cmd"' is not recognized as an internal command
or external, an operable program or a batch file.
[case closed with code 1]

It looks like the starting directory is not being set to the folder that contains the batch files. It would be best to make sure that the batch files can be accessed without an absolute path, or provide the path in the script.
The script is fragile and will break when the environment changes slightly without absolute paths.

But they are all in the same folder, I tried to add ". ", "" At the beginning but it still doesn't work, how can i correctly call the files to run?


filelight-night: Add version 1262

Relates to Extras/#9305
Based on okular-nightly.json

[x] I have read the Contributing Guide.

bucket/filelight-night.json -> bucket/filelight-nightly.json


Missing test coverage for arguments of BsModal

Tests added for the following arguments of BsModal:

size
position
scrollable
keyboard
backdropClose

TODO later we can add tests for transitionDuration and backdropTransitionDuration.
Fixes #1742

TODO later we can add tests for transitionDuration and backdropTransitionDuration.

Would be great if you could create an additional issue targeting that two. I think I had created similar tests for a component in Ember Bootstrap. But not fully sure anymore.

TODO later we can add tests for transitionDuration and backdropTransitionDuration.

Would be great if you could create an additional issue targeting that two. I think I had created similar tests for a component in Ember Bootstrap. But not fully sure anymore.

I think I recalled these tests: https://github.com/kaliber5/ember-bootstrap/blob/cf57ff67b775a8f12f3f46bf21f438c5c46af06d/tests/integration/components/bs-modal-test.js#L269-L333 Not covering the transition duration yet. But might be a starting point to write tests for it.


Plugin requires Read/Write access to Property API

Property bulk search API is a POST request, which requires Read/Write access to the API. Therefore to efficiently search properties you need much higher rights than it should.
Options are:

Rewrite plugin to only use GET operations
Get Akamai engineering to fix the API permissions

Closed by this PR: https://github.com/nodestream-proj/nodestream-plugin-akamai/pull/47


[Exploratory Testing] Cannot add authz for apis in a single app.

Currently if there is a thick client which targets the same service as the browser, with a custom authorization middleware,  even in case of authorization failure, the client receives a 200.(instead of a 401 or 403).
The reason is that there is no way of specifying the authentication type within the authz service. Since to enable browser based authz, cookie middleware has to be added, even if a custom middleware is added, the response gets redirected to a login page in case of a 401, which results in a 200 response getting sent back to an api based client (windows phone app).
The workaround today is either :

Have apis and browser based app as separate end points.
Call the cookie middleware within your custom middleware.

cc: @Tratcher , @HaoK /

When authorization fails, are you calling Challenge(authtypes) with the bearer authentication type? If you do so then only the named types will participate in the response, and the cookie middleware will not redirect.
Having the authtypes on the policy connected to an authorize attribute will have the same effect.

I tried adding the auth type to the policy and that did not work.
  services.Configure<BasicAuthOptions>(options =>
            {
                options.AuthenticationMode = AuthenticationMode.Passive;
                options.AuthenticationType = "BasicAuth";
            });

options.AddPolicy("Api", policyBuilder => policyBuilder.AddAuthenticationTypes("BasicAuth")
                                                                            .RequireAuthenticatedUser());

I think the issue is that AuthorizeAttribute does not call challenge today.

This should be working now


How to navigate desired URL after Azure AD Login

Core Library
MSAL.js v1 (@azure/msal or msal)
Core Library Version
1.4.4
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
None
Description
Hi Team,
We are using Azure AD Authentication to validate users after successful verification it is navigating to configured redirect url
For example :
redirecturi : https://www.testurl.com (Configured in App registration under Auzre portal)
Here it will navigate azure ad login page after successful login it will navigate to above redirecturi.
Problem :  Now If user is trying to access specific document or specific url as mentioned below  (If user is not authenticated)
https://www.testurl.com/data/100
User should navigate to Azure AD login page >> after successful login >> it should navigate to user requested URL for example above URL (https://www.testurl.com/data/100), But it always going to configured redirecturi i.e. https://www.testurl.com.
How to handle this type of scenario to navigate to desired URL?
MSAL Configuration
const MsalObj = new UserAgentApplication({
    auth: {
        clientId: config.msalConfig?.clientId,
        authority: config.msalConfig?.authority,
        postLogoutRedirectUri: config.msalConfig?.postLogoutRedirectUri,
        redirectUri: config.msalConfig?.redirectUri
    },
    cache: {
        cacheLocation: 'sessionStorage',
        storeAuthStateInCookie: false
    }
});

Relevant Code Snippets
No response
Identity Provider
Azure AD / MSA
Source
Internal (Microsoft)

Hi Team,
Please provide some information on this question

@raiajithkumarr Are you able to provide verbose logs for this behavior?

Hi @jo-arroyo
You want verbose logs from deployed version means application insight ? or locally ?
Currently locally it is working fine

After doing analysis of this behavior observed that, if we try to navigate child URL other than Origin(http://testurl.com)  for example  http://testurl.com/data/1 or http://testurl.com/report/1 or anything like this it will navigate to AAD login page ( https://login.microsoftonline.com//oauth2/v2.0/authorize?response_type=id_token&scope) twice
How to handle this behavior.
please find attachment for verbose logs (PII is disabled logs )
300620222033_MSAL.log

Hi Team,
Any updates on this issue.
It is going two times for AAD login (I think first time it is not getting token through msal.js library).
Please suggest here

Hi Ranjith, Have you deployed this application in Azure portal?

@raiajithkumarr Can you confirm that you are using the correct redirectUri in your deployed application?
Also, is the navigation to AAD only happening on the deployed version or on your local app as well?

Due to some other hooks it is calling authorization very fast before we get the response from MSAL Library.
Now we applied some delay with this it is working fine.
So I am closing this request.


Import and process TPS CSV extract file

Context
We need to import workforce data from Capita so we can expose employment history on the APIs.
Changes proposed in this pull request
Using the sample file from Capita, build a job that imports CSVs from a blob storage container. The job should update a table in the DB that contains employment history. When completed, the file should be moved into an archive container.
Note that we will get a file/several files each month and these will contain records that have been changed in the previous 6 months. From this we need to derive the employment history.
Guidance to review
DB + service + job + tests
Checklist

[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Import workforce data CSV


[Usbdev] Sequences

This PR contains approved sequences:
random length out transaction
minimum length out transaction
maximum length out transaction
out stall
out trans nak

@jdonjdon kindly merge this PR so that we can further submit more PRs for other sequences. Thanks


Use I18n.with_locale to not overwrite I18n.locale during html generation

Hey @damoiser
Thanks for creating this gem. I'm currently implementing it and noticed that https://github.com/damoiser/qr-bills/blob/master/lib/qr-bills/qr-html-layout.rb#L11 overwrites the I18n.locale.
This PR fixes that by using I18n.with_locale by only using the provided locale params[:bill_params][:language] within the wrapped code.
This should also fix #16

thanks for the PR @danielpuglisi ! will check your PRs in the following days and revert to you in case will have feedback

thx - it looks good!


Don't define dist/transparency.js as main

Addresses #138

Thanks for the pull request! Everything seems to be working fine now with browserify. I can't really remember why I initially added the main field, but I'm guessing it had something to do with how browserify worked back then.


[Synonym] add synonyms to "mammary alveolar cell"

CL term: mammary alveolar cell (CL:0002325)
Synonym to be added:
"mammary alveolar epithelial cell"
"mammary gland alveolar epithelial cell"
broad/exact/narrow/related ? exact

Hi, @paulacalipho. These synonyms were added in PR #1436.

Many thanks
From: bvarner-ebi @.>
Sent: Friday, January 28, 2022 7:00 PM
To: obophenotype/cell-ontology @.>
Cc: Paula Duek @.>; Mention @.>
Subject: Re: [obophenotype/cell-ontology] [Synonym] add synonyms to "mammary alveolar cell" (Issue #1420)
Hi, @paulacaliphohttps://github.com/paulacalipho. These synonyms were added in PR #1436https://github.com/obophenotype/cell-ontology/pull/1436.
—
Reply to this email directly, view it on GitHubhttps://github.com/obophenotype/cell-ontology/issues/1420#issuecomment-1024474973, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA7GPF4YA2EABMOOWJR3P7DUYLKR3ANCNFSM5MMFWAMQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>


Pretrained weights & training time

Hi,
Can you upload the trained weights you used to generate the README images.
Also how much time did it take to train on FFHQ-128
I'm traying to adapt your code and train on a 64x64 or 32x32 FFHQ versions
Thanks for the help.

Hey, there is a link to the trained weights in the README. It took 9 epochs and that was a few days I guess. Good luck and I hope I could help you!


Support SQL Server 2008

We are migrating our old ERP to Orchard Core, and we need to store and access data at SQL server 2008 servers that is what many of our customer owns.
Problem is we found yes sql doesn't support sql server 2008 sintax for pagination.
I can provide a PR for supporting it with a new SqlServerDialect that inherits from current SqlServerDialect and a few changes at SqlBuilder for allowing dialect to produce nested queries like the one SqlServer needs for pagination.
This other dialect will uncomment current commented lines
            // These are not necessary since SQL Server 2008
            //Methods.Add("day", new TemplateFunction("datepart(day, {0})"));
            //Methods.Add("month", new TemplateFunction("datepart(month, {0})"));
            //Methods.Add("year", new TemplateFunction("datepart(year, {0})"));

Is there any other thing you think YesSql is not compatible with Sql Server 2008 that I have to take into account for my PR?
Are you interested on merging a PR like that?

Please create a new dialect and an extension method to set it on the configuration. I made a change recently that should make it easy, just assign the dialect to the configuration instance.
Please read the latest comments here: https://github.com/sebastienros/yessql/pull/64

So actually nothing to do but to merge the PR ... working on it

Mmh I didn't know the syntax used at that PR was supported at sql server 2008.
https://github.com/sebastienros/yessql/pull/64/files#r533631723
RowNum keyword is not supported as far as I know
Most similar thing is ROW_NUMBER() function but is has to be used at select with order as far as I know. So you need to use a subquery

The pr finally was not accepted cause the complexity required for supporting it will make maintenance hard


Update to Provided Access devices

The text replaced reflect the new secure workstation modality that will be published on December 18th 2020 - Link is unaffected

@simorjay : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @simorjay - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!

Yes, please let's propose this on the -pr side.


Flow plugin throws syntax error for conditional expression

Input Code

FAIL
a ? (b => c) :  d => e

SUCCESS
a ?  b => c  :  d => e
a ?  b => c  : (d => e)
a ? (b => c) : (d => e)

Babylon/Babel Configuration

{
  "plugin": [ "flow" ]
}

Expected Behavior

Flow-parser(https://www.npmjs.com/package/flow-parser) output
ExpressionStatement (
  ConditionalExpressoin (
    test: Identifier('a'),
    consequent: ArrowFunctionExpression('b => c'),
    alternate: ArrowFunctionExpression('d => e')
  )
)

Current Behavior

SyntaxError: test.js: Invalid left-hand side in arrow function parameters (1:5)
> 1 | a ? (b => c) : d => e
    |      ^

Your Environment

software
version

Babylon
6.16.1

node
6.10.2

npm
3.10.10

Hey @jj-choi! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.

Thanks for reporting, this is a duplicate of #58


Please allow export/query of Exposure Score over Time

Related Feature: https://github.com/MicrosoftDocs/microsoft-365-docs/blob/public/microsoft-365/security/defender-endpoint/tvm-exposure-score.md
Can we have the feature to download the "Exposure Score over time" data - either directly or via KQL? This is a useful report for demonstrating at a high level the impact being made as new vulnerabilities appear.

@sebhurleyhte Thank you for your feedback. Since you are requesting a new feature, I'd recommend you provide this feedback directly on our Main feedback portal - https://feedbackportal.microsoft.com/, So that someone will be able to contact you to take this forward.
Thanks
Sri

@sebhurleyhte Thank you very much for your kind feedback. Considering your valuable feedback, please log this issue as advised as here we are limited to documentation fixes and improvements only. Concerning team will evaluate and take the appropriate action, as per their schedule. We proceed here to close it. Thanks again for taking out some time and sharing your feedback.