structure sensor on jetson tx1 (aarch64)

Hi, i'm trying to use the structure sensor, using openni2 on the nvidia jetson tx1 board.
i'm having trouble running the samples.
I had to make small change to get it to compile on aarch64 (arm-64) but i mostly copied and simplified the existing makefiles for arm.
can't find much info on using this specific camera on the tx1 though i can find some on the previous version of this board, the tk1.
Does anybody can help me out with this? Or better, would anyone with a similar setup would be interested in trying to make it work on their side? Thanks for your time guys.
here is the output from SimpleRead sample with verbose level = 0
ubuntu@tegra-ubuntu:~/OpenNI2/Bin/aarch64-Release$ ./SimpleRead
1374 INFO       New log started on 2016-09-13 15:04:59
1794 INFO       --- Filter Info --- Minimum Severity: VERBOSE
2234 VERBOSE    No override device in configuration file
3179 VERBOSE    Configuration has been read from './OpenNI.ini'
3377 VERBOSE    OpenNI 2.2.0 (Build 33)-Linux-Arm (Sep 13 2016 15:02:36)
3520 VERBOSE    Using './OpenNI2/Drivers' as driver path
3702 VERBOSE    Looking for drivers in drivers repository './OpenNI2/Drivers'
20744 INFO       New log started on 2016-09-13 15:04:59
20816 INFO       --- Filter Info --- Minimum Severity: VERBOSE
20879 VERBOSE    Initializing USB...
23969 INFO       USB is initialized.
24111 INFO       Device connected: PrimeSense PS1080 (1d27/0600@1/3)
26713 INFO       New log started on 2016-09-13 15:04:59
26781 INFO       --- Filter Info --- Minimum Severity: VERBOSE
26845 VERBOSE    Initializing USB...
29669 INFO       USB is initialized.
29821 VERBOSE    Trying to open device by URI '(NULL)'
30416 VERBOSE    Initializing device sensor...
30603 VERBOSE    Configuring module 'Device' from section 'Device' in file '/home/ubuntu/OpenNI2/Bin/aarch64-Release/OpenNI2/Drivers/PS1080.ini'...
30769 INFO       Module 'Device' configuration was loaded from file.
30831 VERBOSE    Connecting to USB device...
30877 VERBOSE    Trying to open sensor<EMAIL_ADDRESS>5276333 VERBOSE    Starting libusb asynch thread...
5278864 WARNING    Failed to set thread priority (1) - (1)
5279489 WARNING    USB events thread: Failed to set thread priority to critical. This might cause loss of data...
Warning: USB events thread - failed to set priority. This might cause loss of data...
5280557 VERBOSE    Trying to open endpoint 0x4 for control out (for old firmwares)...
5282246 INFO       Connected to USB device
5319104 VERBOSE    Getting hardware versions...
30833501 ERROR      Get version failed: USB transfer timeout!
32834538 VERBOSE    Getting hardware versions...
58346917 ERROR      Get version failed: USB transfer timeout!
58347317 VERBOSE    Getting hardware versions...
83860843 ERROR      Get version failed: USB transfer timeout!
83861230 VERBOSE    Shutting down USB depth read thread...
83861422 VERBOSE    Shutting down USB image read thread...
83863406 VERBOSE    Shutting down USB events thread...
83865091 VERBOSE    Device closed successfully
83865824 VERBOSE    Shutting down Scheduler thread...
Couldn't open device
Could not open "1d27/0600@1/3": USB transfer timeout!
using SUDO, the program is able to correctly set the thread priority :
ubuntu@tegra-ubuntu:~/OpenNI2/Bin/aarch64-Release$ sudo ./SimpleRead
344 INFO       New log started on 2016-09-13 15:10:09
451 INFO       --- Filter Info --- Minimum Severity: VERBOSE
552 VERBOSE    No override device in configuration file
627 VERBOSE    Configuration has been read from './OpenNI.ini'
841 VERBOSE    OpenNI 2.2.0 (Build 33)-Linux-Arm (Sep 13 2016 15:02:36)
883 VERBOSE    Using './OpenNI2/Drivers' as driver path
930 VERBOSE    Looking for drivers in drivers repository './OpenNI2/Drivers'
5266 INFO       New log started on 2016-09-13 15:10:09
5336 INFO       --- Filter Info --- Minimum Severity: VERBOSE
5393 VERBOSE    Initializing USB...
8890 INFO       USB is initialized.
9019 INFO       Device connected: PrimeSense PS1080 (1d27/0600@1/3)
10039 INFO       New log started on 2016-09-13 15:10:09
10104 INFO       --- Filter Info --- Minimum Severity: VERBOSE
10161 VERBOSE    Initializing USB...
16149 INFO       USB is initialized.
16260 VERBOSE    Trying to open device by URI '(NULL)'
17001 VERBOSE    Initializing device sensor...
17127 VERBOSE    Configuring module 'Device' from section 'Device' in file '/home/ubuntu/OpenNI2/Bin/aarch64-Release/OpenNI2/Drivers/PS1080.ini'...
18021 INFO       Module 'Device' configuration was loaded from file.
18059 VERBOSE    Connecting to USB device...
18387 VERBOSE    Trying to open sensor<EMAIL_ADDRESS>5272756 VERBOSE    Starting libusb asynch thread...
5274641 VERBOSE    Trying to open endpoint 0x4 for control out (for old firmwares)...
5278283 INFO       Connected to USB device
5337926 VERBOSE    Getting hardware versions...
30850496 ERROR      Get version failed: USB transfer timeout!
32854179 VERBOSE    Getting hardware versions...
58368169 ERROR      Get version failed: USB transfer timeout!
58368939 VERBOSE    Getting hardware versions...
83883152 ERROR      Get version failed: USB transfer timeout!
83883856 VERBOSE    Shutting down USB depth read thread...
83884670 VERBOSE    Shutting down USB image read thread...
83886246 VERBOSE    Shutting down USB events thread...
83886761 VERBOSE    Device closed successfully
83887274 VERBOSE    Shutting down Scheduler thread...
Couldn't open device
Could not open "1d27/0600@1/3": USB transfer timeout!
some info on the board/os :
uname -a
Linux tegra-ubuntu 3.10.67-g458d45c #2 SMP PREEMPT Wed Aug 24 17:25:02 UTC 2016 aarch64 aarch64 aarch64 GNU/Linux
lsusb
Bus 002 Device 002: ID 0955:09ff NVidia Corp.  Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub Bus 001 Device 003: ID 1d27:0600 ASUS  Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub

i don't remember changing anything for the verbose level.
Though i realised that the verbose level is read from OpenNI2.ini file (from the log output)
ubuntu@tegra-ubuntu:~/OpenNI2$ locate OpenNI.ini
/etc/openni2/OpenNI.ini
/home/ubuntu/OpenNI2/Config/OpenNI.ini

ubuntu@tegra-ubuntu:~/OpenNI2$ more /etc/openni2/OpenNI.ini
[Log]
; 0 - Verbose; 1 - Info; 2 - Warning; 3 - Error. Default - None
Verbosity=3
LogToConsole=0
LogToFile=0

[Device]
;Override=""

[Drivers]
; Location of the drivers specified by a relative path based on OpenNI's shared library or an absolute path.
; Path separator "/" can be used to be portable for any platforms.
; Default - OpenNI2/Drivers
;Repository=OpenNI2/Drivers

ubuntu@tegra-ubuntu:~/OpenNI2$ more Config/OpenNI.ini
[Log]
; 0 - Verbose; 1 - Info; 2 - Warning; 3 - Error. Default - None
Verbosity=3
LogToConsole=0
LogToFile=0

[Device]
;Override=""

[Drivers]
; Location of the drivers specified by a relative path based on OpenNI's shared library or an absolute path.
; Path separator "/" can be used to be portable for any platforms.
; Default - OpenNI2/Drivers
;Repository=OpenNI2/Drivers

FYI : here are my changes in the source
diff --git a/ThirdParty/PSCommon/BuildSystem/CommonDefs.mak b/ThirdParty/PSCommon/BuildSystem/CommonDefs.mak
index dd88b04..9953349 100644
--- a/ThirdParty/PSCommon/BuildSystem/CommonDefs.mak
+++ b/ThirdParty/PSCommon/BuildSystem/CommonDefs.mak
@@ -18,6 +18,8 @@ else ifneq (,$(findstring i386,$(MACHINE)))
        HOST_PLATFORM = x86
 else ifneq (,$(findstring arm,$(MACHINE)))
        HOST_PLATFORM = Arm
+else ifneq (,$(findstring aarch64,$(MACHINE)))
+       HOST_PLATFORM = Arm
 else
        DUMMY:=$(error Can't determine host platform)
 endif
diff --git a/ThirdParty/PSCommon/BuildSystem/Platform.Arm b/ThirdParty/PSCommon/BuildSystem/Platform.Arm
index c4a112d..4b78b1d 100644
--- a/ThirdParty/PSCommon/BuildSystem/Platform.Arm
+++ b/ThirdParty/PSCommon/BuildSystem/Platform.Arm
@@ -1,7 +1,7 @@
 ifeq "$(CFG)" "Release"

     # Hardware specifying flags
-    CFLAGS += -march=armv7-a -mtune=cortex-a9 -mfpu=neon -mfloat-abi=softfp #-mcpu=cortex-a8
+    #CFLAGS += -march=armv7-a -mtune=cortex-a9 -mfpu=neon -mfloat-abi=softfp #-mcpu=cortex-a8

     # Optimization level, minus currently buggy optimizing methods (which break bit-exact)
     CFLAGS += -O3 -fno-tree-pre -fno-strict-aliasing
@@ -9,6 +9,6 @@ ifeq "$(CFG)" "Release"
     # More optimization flags
     CFLAGS += -ftree-vectorize -ffast-math -funsafe-math-optimizations #-fsingle-precision-constant

-    DEFINES += XN_NEON
+    #DEFINES += XN_NEON
     CFLAGS += -flax-vector-conversions
 endif

I see a lot of people are having issues compiling for aarch64. To make it easy for everyone, I have uploaded a copy of my OpenNI2 source files that have been modified to work on aarch64, cortex-A57 on my github: https://github.com/mikeh9/OpenNI2-TX1.git
Also included are samples in binary format if you want to just run them.
Thanks to jetsonhacks.com for providing the instructions on how to compile the source for TK1.

Sorry, I just checked and I don't have the archive anymore.


package iceplume - reopening

What changes does this PR introduce?
Introduce a new package "iceplume" to parameterize melt plumes at glacier fronts as well as code in main branch that propagate the properties computed from the pkg to state variables.
What is the current behaviour?
There is no current behavior. This commit submit a new pkg.
What is the new behaviour
A new package to parameterize melt plumes at glacier fronts
Does this PR introduce a breaking change?
It should not, because it is a new package.
Other information:
Reference: Cowton, T., D. Slater, A. Sole, D. Goldberg, and P. Nienow (2015), Modeling the impact of glacial runoff on fjord circulation and submarine melt rate using a new subgrid-scale parameterization for glacial plumes, J. Geophys. Res. Oceans, 120, 796–812, doi:10.1002/2014JC010324.
Suggested addition to tag-index
(To avoid unnecessary merge conflicts, please don't update tag-index. One of the admins will do that when merging your pull request.)

i created a new PR in verification_other, https://github.com/MITgcm/verification_other/pull/58 . Once this set up is copied into verification, or put into MITgcm/verification_other/ level , it should compile and run successfully with this PR #(was 652, now) 682.

@antnguyen13 I run the small test experiment you provided (https://github.com/MITgcm/verification_other/pull/58 ) and this is nice to have an example using pkg/iceplume.
But when I tried (with gfortran) with "-devel", it stops quickly because of out-of-bounds indices in iceplume_calc.F, e.g., in line 190.

@antnguyen13 Any update on this ?

But when I tried (with gfortran) with "-devel", it stops quickly because of out-of-bounds indices in iceplume_calc.F, e.g., in line 190.

I tried again after merging master branch in (including pr 666) and got the same error.

@jm-c i've now fixed the out-of-bound bug, along with a few more bugs (that also crashes the code once you get through the bug you have reporeted) related to overlap and initialization issues reported by a couple of other users (M.Wood and K.Schulz).

I am also seeing some unexpected changes for diagnostic "IPmasspl" when using more than 1 tile per proc (i.e, nSx > 1): I run the iceplume_test experiment but compiled with "sNx=10, nSx=2) and got non zero value at k=2 for this diagnostic output (at iter=1) but was all zeros when I was using only one tile.

@jmc I've made a first attempt to switch to flux form for calculations of tendency.   The important points are:

the INITIAL subglacial discharge is a point source with initial volume flux Q_sg [m3/s], T_sg_0 = 1e-3 degC, S_sg_0 = 1e-3 g/kg.  But within the parameterization, we get a profile of entrained mass with its associated profiles of T,S. So any "added [heat,salt]flux" to the entire system is just from Q_sg of T_sg_0 and S_sg_0, but the output of the parameterization is additional vertical profile of displaced (entrained) addmass and associated T,S profiles, from which we begin to compute the [heat,salt] fluxes to give to apply_forcing.F.
the background submarine melt is due to ocean losing heat to melt it, so there are two fluxes involved I think: the loss of ocean heat to melt the addmass of submarine melt, THEN should be the heatflux associated with this addmass into the ocean at its own T,S?

While doing so, i got also a bit confused on why
for the subglacial discharge (plume) we
a. compute tendency using _hFacC (in apply_forcing.F),
b. add the additional term addmass*theta (similar to empmr i think?, in apply_forcing.F)
c. have heat,salt fluxes are of same sign as addmassplume (in iceplume_calc.F)
while for the submarine background melt (bg) we
a. did not apply _hFacC during calc of tendency (in iceplume_plume_tendency.F),
b. did not add term addmass*theta to tendency (in iceplume_plume_tendency.F)
c. have the heat,salt fluxes are of OPPOSITE sign to addmassbg (in iceplume_calc.F)
I think the current switch is incomplete. I kept it such that old results are duplicated to within machine precision.  But once I have more inputs from you as to whether what i'm doing is making sense or not, i can proceed with deleting some un-needed code.

@antnguyen13 I have few preliminary comments:

I wanted to merge latest master branch in this PR but as I tried to compile the iceplume_test experiment from https://github.com/MITgcm/verification_other/pull/58 it failed. Could you check before bringing latest master in ?
the resetting of temp_addMass and salt_addMass to UNSET_RL in ini_parms.F (lines 1553,1554) is still there so I am not sure what the lastest set of changes means.
few things could make the reviewing easier, e.g., returning to original version of external_fields_load.F since all changes are commented out ; the readme in pkg/iceplume could be removed, and we don't really need commented lines starting with "catn", a "C" for text and "c" for commented out code is preferable. Also not clear why the bi,bj loop range are not the usual ones, i.e., myBxLo(myThid), myBxHi(myThid), and myByLo(myThid), myByHi(myThid) in iceplume_check.F and iceplume_init_fixed.F.


Error: Cannot find module 'vscode' when use vscode in server

VSCode Version:1.14.2
OS Version:win10 X64

Steps to Reproduce:
1.clone the official example for language server from https://github.com/Microsoft/vscode-languageserver-node-example
2.then follow these steps:

cd client
npm install
code .
cd ../server
npm install
append "post install": "node ./node_modules/vscode/bin/install" to scripts in package.json
npm install vscode -save
code .

open server.ts which located in server/src folder, then add a line import * as vscode from 'vscode' at the beginning  of server.ts.
press ctrl+shift+B to build task.
after build finished press f5 under client folder
error message is shown below:
Error: Cannot find module 'vscode'
at Function.Module._resolveFilename (module.js:470:15)
at Function.Module._load (module.js:418:25)
at Module.require (module.js:498:17)
at require (internal/module.js:20:19)
at Object.

Reproduces without extensions: Yes/No

Getting exactly the same issue, and struggling with the various sources of documentation / examples.

@tempit In fact at first I asked this question under vscode repo, but was closed and advised to open an issue under this repo. But you know this repo was not stared by many people, it's a demo so the there will less probability to be resolved. Eventually have you resolved this question? i'm not.

@hechaochao, yes I did. My issue was that I had stuff from 'vscode' module imported in the server, which should be independent.
I moved to only importing stuff from 'vscode-languageserver' and everything was resolved.
Hope this helps.

@tempit Yes, but what I want to know is the reason why 'vscode' module can't be used in the server.

The vscode module can only be used on the client side since it needs the VS Code extension host API which is not available on the server.


Make logging output location more programmatically configurable

Overview
This PR includes various improvements to make InfluxDB's logging output location more programmatically configurable in general. This feature is only important for users who embed InfluxDB into their Go programs, and was already half-supported. This change just improves the existing feature.
For an in-depth example of why this change is important, please see the description of https://github.com/sourcegraph/appdash/pull/131
Required for all non-trivial PRs

[x] Rebased/mergable
[ ] Tests pass
[ ] CHANGELOG.md updated
[x] Sign CLA (if not already signed)

The CI failure is now listed as:
--- FAIL: TestServer_BackupAndRestore (2.05s)
	backup_restore_test.go:97: query results wrong:
			exp: {"results":[{"series":[{"name":"myseries","columns":["time","host","value"],"values":[["1970-01-01T00:00:00.001Z","A",23]]}]}]}
			got: {"results":[{}]}

But unless I am crazy wrong, I don't see how this change could cause it exactly? Is it perhaps a flaky test?

@slimsag might be worth repushing your change - it seems the CI failure affected more than just your test and I wasn't able to reproduce it locally.
I am intrigued by your use case. Are you intending to imbed a full influxdb server in instrumented apps rather than just an influx client? Or is influxdb just an example of an app that you could instrument with your solution?

@slimsag that's a flaky test. I'll run a rebuild of your branch.

@slimsag I like the idea of making it easier to make the logging output more configurable.
Since I doubt anyone would want to dynamically update the logging locations once influx is running, some notes about calling SetLogOutput on each service, before calling Open on the server would be useful.

@slimsag needs a rebase. Happy to look at this again when you've taken a look through my comments.

Thanks for the CI help! I apologize for the delays in updating this PR -- really sorry about that :)
I've rebased against master and will address all comments now.

@jonseymour

I am intrigued by your use case. Are you intending to imbed a full influxdb server in instrumented apps rather than just an influx client? Or is influxdb just an example of an app that you could instrument with your solution?

So Appdash, as you may or may not know, is an open-source project for tracing the performance of applications (both standalone and distributed ones). Basically we measure real-time performance of operations for e.g. everything that happens in serving an HTTP request, and then we need a place to store those numbers for later viewing by the user (typically a developer, but sometimes even the end user of a product). In turn, we have a massive number of writes (~1KB of data per HTTP request), and not very many reads (maybe 5% of data inserted is actually read).
In the past, we used some rather simple but primitive storage mechanisms. Like literally just a Go map[int64][]byte in memory. As the project has grown, we want to answer more important queries like "what are the slowest HTTP endpoints in the entire system as a whole?", so we wrote some more primitive storage mechanisms to handle this sort of data-aggregation, but it quickly became very complex and very non-performant. Of course, I know of one other open-source time series database written in Go and so @chris-ramon has been working hard over the past few months to see how compatible it is with Appdash, and the results are looking very promising.
One major constraint for Appdash, however, is that it is typically no-setup-required. For example, Appdash should ideally scale to multi-node distributed setups like what you might observe inside high-throughput environments like Google, Facebook, etc. But very importantly, it also needs to be literally zero-setup for use within small web servers, CLI tools, and in the case of Sourcegraph, the company I work at, easy to setup on-premises and local dev machine deployments (because when users see our app going slow, asking them to set up tracing software + database software is not a nice response compared to "just copy+paste the JSON trace" or something like that =) ).
This last no-setup-required case, is exactly where Appdash needs to be able to embed InfluxDB within the process itself. To do this we're using github.com/influxdata/influxdb/cmd/influxd/run.NewServer to spin up a new InfluxDB server in-process; and this works surprisingly well! The only issue is that InfluxDB logs pollute our app's logs, and thus PR will let us direct them to a log file and make debugging issues much easier.
Hopefully this clarifies everything and, while I know it was very verbose, I hope this wasn't an unwarranted response! Hopefully I didn't come across as trying to pitch anything, and if this type of comment is right here just let me know and I'll be glad to remove.

CHANGELOG.md updated

It's not 100% clear to me if I should update the changelog for this PR; and would I update the Features or Bug Fixes section?

The AppVeyor failure looks fishy:
=== RUN TestTimer_Elapsed
--- FAIL: TestTimer_Elapsed (2.00s)
	stress_test.go:58: expected around 2s got 1.9998922s

Is this an open issue somewhere else?

@slimsag you can update CHANGELOG to say you added this as a feature. Yes the flaky Windows test is known to us. We will sort it next week.

@slimsag I just need to get another reviewer to look at this before it can be accepted.

Got it; I've updated the changelog. Please squash as you see fit :)

👍

@slimsag we're ready to merge this. You may want to diff with master and see if anything has landed in the last two weeks that involves logging stuff you might want to make more customisable.
When you're ready ping me and I'll merge this (or any updates you add).

Hi @e-dard, and thanks for the help on this! I'm sorry to leave a larger change for later, I hope it's okay, but I've made a somewhat verbose change that I think is better than having SetLogger methods.
At this point I can easily call Server.SetLogOutput(somefile) to direct all my InfluxDB server output to a log file; so I am 100% content with this PR as-is.
I've rebase, so PTAL at my new commits starting with commit 8c43c75 (tsdb/engine/tsm1: add and use CacheLoader.SetLogOutput).

I'm thinking this is unrelated, let me know if not:
--- FAIL: TestServer_BackupAndRestore (2.03s)
	backup_restore_test.go:97: query results wrong:
			exp: {"results":[{"series":[{"name":"myseries","columns":["time","host","value"],"values":[["1970-01-01T00:00:00.001Z","A",23]]}]}]}
			got: {"results":[{}]}


[WIP] Move the Flexvolume provisioner into the driver code

https://github.com/rancher/longhorn/issues/106

Other than that shutdown issue, LGTM. We can merge after it got fixed.

We also need to update the yaml file, add manager-url to driver-deployer's command.
      containers:
        - name: longhorn-driver-deployer
          image: <new_image>
          imagePullPolicy: Always
          command:
          - longhorn-manager
          - -d
          - deploy-driver
          - --manager-image
          - <new_image>
          - --manager-url
          - http://longhorn-backend:9500/v1

Of course this needs to be done after you build a new image.

Merged, thanks.


New Ore Glow

Changes ore glow so it looks different
also adds ore glow to iron and copper

technically you already gave me approval

I'll probably have time to look this over in a few days. One thing I notice, though, is that some things you're doing operations with are integers when they should be floats. It mostly doesn't cause issues but I feel like it's best practice to stick with floats when working with floats.
Also, in some of your checks you check if frx_fragColor is some value over one. This isn't possible because before the material shader runs, it should be between 0.0 and 1.0.
In any case, thank you for sorting this out - a few days ago I was experimenting with iron checks but couldn't quite get it done right.

Okay, ignore the second comment. I took a closer look and you aren't actually doing what I thought you were.

I just changed ints to floats. I thought it wouldn’t really matter since it would be converted to float anyways, but if you would like it this way that’s fine with me. Do you need me to add parentheses so it’s more clear what I’m trying to accomplish?

No, this looks good. I'll test it when I can and hopefully merge it. Thank you.

It looks good, merging.


Adding prometheus e2e container code

Signed-off-by: Travis Bickford<EMAIL_ADDRESS>

Thanks @tbickford!


Misc changelog and POM comment updates in Core and Serialization

API change check
API changes are not detected in this pull request.

/check-enforcer override


Expose OnDeployExecutor Interface and store exception message on result node

Expose OnDeployExecutor Interface: so scripts can be triggered externally
Store exception message on result node: so it can be read

Pull Request Test Coverage Report for Build 4688

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 53.498%

Totals

Change from base Build 4687:
0.004%

Covered Lines:
13850

Relevant Lines:
25889

💛  - Coveralls

Will try to get to this next week.

Can you please help me understand the use case for Expose OnDeployExecutor Interface: so scripts can be triggered externally?

@HitmanInWis I believe @geertsmichael discussed the use-case here: https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/1956

I'm not really seeing the use case, just the request to be able to call the OnDeployExecutor from other code.  i.e. I see "what" is being requested (ability to re-trigger a script from code), but don't understand "why" it's being requested especially since the JMX console already allows re-running a script I believe AND a script can always be re-run by re-deploying the bundle (simply need to delete the status node if you are re-running a previously successful script).
OnDeployScripts were always kinda meant to be one-time things, but I imagine folks have come up with other use cases that I'm not thinking of.  Seeking to understand.

The use-case is about a tool that I wrote, which will visualize the script execution (success, running, fail). This tool allows to do a re-run through my Touch UI tool.

One quick question remains about CHANGELOG.md - other than that this looks good and I'll test it out later today.

Everything tests out, merged.  Thanks @michael-geerts for the contribution and @shsteimer for the recommended approach.


Cannot connect because unable to refresh token: Too many requests

Got the following error after the most recent reboot of HA:

2024-12-12 19:00:42.595 WARNING (MainThread) [hass_nabucasa.iot] Cannot connect because unable to refresh token: Too many requests
2024-12-12 19:00:45.473 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Tualatin for kwikset
Traceback (most recent call last):
File "/usr/local/lib/python3.13/site-packages/aiokwikset/api.py", line 207, in async_renew_access_token
await self.loop.run_in_executor(None, cognito.renew_access_token)
File "/usr/local/lib/python3.13/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/local/lib/python3.13/site-packages/pycognito/init.py", line 699, in renew_access_token
refresh_response = self.client.initiate_auth(
ClientId=self.client_id,
AuthFlow="REFRESH_TOKEN_AUTH",
AuthParameters=auth_params,
)
File "/usr/local/lib/python3.13/site-packages/botocore/client.py", line 569, in _api_call
return self._make_api_call(operation_name, kwargs)
~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.13/site-packages/botocore/client.py", line 1023, in _make_api_call
raise error_class(parsed_response, operation_name)
botocore.errorfactory.TooManyRequestsException: An error occurred (TooManyRequestsException) when calling the InitiateAuth operation: Too many requests
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 640, in __async_setup_with_context
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/kwikset/init.py", line 41, in async_setup_entry
await client.async_renew_access_token(entry.data[CONF_ACCESS_TOKEN], entry.data[CONF_REFRESH_TOKEN])
File "/usr/local/lib/python3.13/site-packages/aiokwikset/api.py", line 211, in async_renew_access_token
raise _map_aws_exception(err) from err
aiokwikset.api.UnknownError: Too many requests

It is worth noting that I have rebooted may HA instance SEVERAL times in the last few days, so it's possible I've triggered something on the kwikset side of things, but I figured this was still worth recording because obviously I'm unable to access my locks right now via HA as a result. Is this a situation where I am just going to need to wait and then try reloading or ???

FYI.. Waiting and then reloading resolved the issue.
Still worth investigating as it may be a good idea to build in some retry/resiliency/caching so that this error doesn't come up for others.


[FLINK-6073] - Support for SQL inner queries for proctime

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x ] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[ ] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

@fhueske @twalthr @sunjincheng121 @shijinkui @stefanobortoli @hongyuhong
I have made a first implementation draft for supporting inner queries mainly when operating on processing time. I would highly appreciate some feedback from you to further enhance the approach.
The idea of the implementation is described in https://issues.apache.org/jira/browse/FLINK-6073?filter=-2

FLINK-9714 has been added and will be included in the next release.
Closing this PR.


Add Ayomide to interested technical writers for 2024 GSOD

Hi! Please remove the   from your entry, they're not needed. Thank you!

closing this as it's been a week with no response. feel free to reopen if you have an update!

Hello, i'm not sure what you asked me to remove from my entry. Please check your previous comment.

Please remove the quote marks at the beginning and end of your information, thank you.

I already made another PR and it has been added. Thank you.


Can't step into code that is defined in a lib (virtual environment)

From @thiagorabelo on September 13, 2017 21:28
Environment data
VS Code version:  1.15.1
Python Extension version:  0.7.0
Python Version:   3.5.2 (in a virtual environment)
OS and version:    Kubuntu 16.04.3 LTS
Actual behavior
Can't step into line when this line is in a lib (at least in Django).

In this example instead of debugger enter in code definition of "get_object_or_404" in file "django/shortcut.py", in folder "site-package" of virtual environment, it jumps to "_pq_fetch_tuples" method on line 796 in Cursor class in file "psycopg2cffi/_impl/cursor.py" in "site-package" folder of virtual environment.
In same way, instead step into "render" function in "django/shotcuts.py", jumps to function "urlparse" in "/usr/lib/python3.5/urllib/parse.py" at line 301.
Expected  behavior
Enter the appropriate function.

Steps to reproduce:

While debugging some block of code, step into a function or method defined in some lib.

Logs
Output from Python output panel
Empty
Output from Console window (Help->Developer Tools menu)
Empty
Copied from original issue: DonJayamanne/pythonVSCode#1210

Nice! I'm only waiting this issue be solved to move permanently to VSCode.
Thanks!

Lol, I'm no longer able to replicate this issue.
@thiagorabelo Please could you try debugging using the experimental debugger.
All you need to do is:

Open launch.json
Change "type":"python" to "type":"pythonExperimental"
Add the setting "django": true

Here you go:
        {
            "name": "Django",
            "type": "pythonExperimental",
            "request": "launch",
            "stopOnEntry": false,
            "pythonPath": "${config:python.pythonPath}",
            "program": "${workspaceRoot}/manage.py",
            "args": [
                "runserver",
                "--noreload"
            ],
            "django": true
        },

Hi, @DonJayamanne
I tried using the default vscode-python debugger and I got the same problems from earlier.
Then, I tried use the "python experimental" as you specified above and every thing worked well. However, I noticed these traceback on terminal:
Traceback (most recent call last):   File "/home/thiago/.vscode/extensions/ms-python.python-2018.4.0/pythonFiles/experimental/ptvsd/ptvsd/wrapper.py", line 796, in done     fut.result()   File "/home/thiago/.vscode/extensions/ms-python.python-2018.4.0/pythonFiles/experimental/ptvsd/ptvsd/futures.py", line 40, inresult     reraise(self._exc_info)   File "/home/thiago/.vscode/extensions/ms-python.python-2018.4.0/pythonFiles/experimental/ptvsd/ptvsd/reraise3.py", line 8, inreraise     raise exc_info[1].with_traceback(exc_info[2])   File "/home/thiago/.vscode/extensions/ms-python.python-2018.4.0/pythonFiles/experimental/ptvsd/ptvsd/futures.py", line 149, in callback     x = next(it)   File "/home/thiago/.vscode/extensions/ms-python.python-2018.4.0/pythonFiles/experimental/ptvsd/ptvsd/wrapper.py", line 1229, in on_variables     pyd_var = self.var_map.to_pydevd(vsc_var)   File "/home/thiago/.vscode/extensions/ms-python.python-2018.4.0/pythonFiles/experimental/ptvsd/ptvsd/wrapper.py", line 215, in to_pydevd     return self._vscode_to_pydevd[vscode_id] KeyError: 125

@thiagorabelo That's great. please could you provide some instructions (or simple program) that we could use to replicate this issue (with the experimental debugger).

@DonJayamanne , I have to apologize to you.
I don't know what I did! I can't reproduce this issue any more. 😅

@thiagorabelo
We've identified the other issue you came across and have resolved it in the debugger.

@DonJayamanne
I've been trying to reproduce the issue in the same block of python code to report to you with more details. But I don't know why I can't do it again.
Any way, this is a great news.
Thank you.


Ignore global maps when processing selector keys

Process key maps without waiting for longer global mappings that
start with the same keys.

Thanks! Merged it.

Oh actually, we've moved this functionality into the maktaba library and plan to eventually decommission this standalone repo. You probably want to make this same change at https://github.com/google/vim-maktaba/blob/f60443777137a0f91e489b1757c6602c63fb3731/autoload/maktaba/ui/selector.vim#L375 and switch whatever usages you have over to maktaba#ui#selector instead.
@Kashomon FYI

You probably want to make this same change at https://github.com/google/vim-maktaba

Done - https://github.com/google/vim-maktaba/pull/236.
I tracked this from the relatedfiles plugin which uses this version of selector.  The c key in selector window hanged because it overlapped with cs, cS maps from vim-surround.

Excellent, thanks for the pointer and fixes! We'll be converting our other plugins over to use the maktaba copy after a little time to let the new maktaba version circulate around.


Login Button Remains Visible After Google OAuth2.0 Login

Issue: Login Button Remains Visible After Google OAuth2.0 Login
Description
When using Google OAuth2.0 for authentication with NextAuth, the login button continues to display even after a successful login. Instead of showing the user's profile picture and other details, the UI remains unchanged with the login button still visible.
Steps to Reproduce

Navigate to the login page of the application.
Click on the "Login with Google" button.
Complete the Google OAuth2.0 authentication process.
Observe that the login button is still visible on the page.

Expected Behavior
After a successful login with Google OAuth2.0, the application should:

Hide the login button.
Display the user's profile picture, email, and any other relevant details in place of the login button.

Actual Behavior

The login button remains visible even after a successful login.
The user's profile picture and details are not displayed.

Screenshots
Before Login

After Login

That is because session is not setting, i am on it


not able use colbert embeddings with llamaindex

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
how to use fastembed lateinteraction embedding module in llamaindex as colbert is available inside it only?

@dosu
it says colbert is not in model list.

ValueError: Model colbert-ir/colbertv2.0 is not supported in TextEmbedding.Please check the supported models using `TextEmbedding.list_supported_models()

Can you provide a solution in which i can use colbert using fastembed these solution are not working

you have not used fastembed i want to use colbert with fastembed

@jerryjliu can you help here?

colbert embeddings are not supported at the moment in llama-index


Emphasis the HP screen box

Closes https://github.com/scylladb/sphinx-scylladb-theme/issues/508
Preview

Changes

Added a subtle shadow to the home page search box.

Removed the background gradient from the hero box component. It was not looking right in pages with the sidebar and the hero box component. For example:

Before:

Now:

LGTM


Autocomplete-plus behaviour is overridden

In other languages within Atom, I experience autocomplete-plus showing a list of options as I type. I can select an option by arrowing to it and hitting ENTER or hide the list with ESCAPE or ignore altogether.
With Omnisharp-atom running, this behaviour mutates slightly (but frustratingly): SPACEBAR now accepts the option.  This causes wrong code to be thrown in simply by typing a lot of common lines of code.
For example, if I naturally begin typing out:
for (int i
The minute i shows up, I get an autocomplete list (which I'm about to ignore). But once I hit space to continue, it inserts:
 for (int if (Condition) {
Not sure if this is intended. I don't see anywhere describing that Omnisharp is overriding Autocomplete's behaviour.   How can this be disabled?

I think the problem is in this line https://github.com/OmniSharp/omnisharp-atom/blob/master/lib/features/intellisense.ts#L15. At least there should be an option to choose space as a confirm key.

You can temporarily disable "Intellisense" in OmniSharp options. But this will also disable confirming on ";" and "." keys.

I know this is an issue that has come up before, after I finish the work
I'm doing right now I think I'll see what I can do about this.
On Mon, Jan 11, 2016 at 3:18 PM, Noderino<EMAIL_ADDRESS>wrote:

You can temporarily disable "Intellisense" in OmniSharp options. But this
will also disable confirming on ";" and "." keys.
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-atom/issues/609#issuecomment-170677357
.

Thanks, I'll temporarily disable it.   I appreciate the prompt response. Please let me know if there's anything I can do to help.

with you guys could fix this soon. this really killing omni sharp usage for me

I tried just removing event.text === " "  from the if statement and everything seems to work, without being annoying. Does this break something else? Otherwise I vote for just removing spacebar as a confirming action.


[FLINK-25198][docs] Add doc about name and description of operator

What is the purpose of the change
this is part of This PR is part of https://cwiki.apache.org/confluence/display/FLINK/FLIP-195%3A+Improve+the+name+and+structure+of+vertex+and+operator+name+for+job , aims at to add doc about name and description.
Brief change log
add docs about name and description in the section of operators.
Verifying this change
This change is a trivial work without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@gaoyunhaii  thanks for the detailed review, I have addressed the comments


Make API calls from custom validation functions

Guidelines

Please include a demo of the issue/behaviour/question you have

Make API calls during the validation event

Please try to be as detailed as possible

Would be good if we can make API calls during the custom validation function execution and wait for the API's response esp. when your validation needs to take place against the persistent DB values

You may fork one of the following starter templates if you would like:

CodeSandbox

StackBlitz

Is your feature request related to a problem? Please describe.
Currently in order for API calls to work as a part of the custom validation function, one need to make the call async as API calls are promises. If you mark your custom validation function as async, nothing gets validated. So, it appears that you can't have promises in custom validation routine
Describe the solution you'd like
Have an ability to make API calls within custom validation functions.
Describe alternatives you've considered
None
Additional context
N/A

Yes, promises are currently not supported, but would be an interesting use case. Do you have an idea on how to achieve this?

Hey @Domino987 I managed to get this working though not under the validation rule but under the onRowUpdate and onBulkUpdate events.

Thats works as well. Very good. Ill close this then.


Issue with timedTransitions using PlatformIO

I was looking for a FSM with best user interface, and I really like yours.
After some tests with your examples, by compiling with PlatformIO, I encountered an issue:
(To be clear: everything works well with Arduino-IDE! I don't understand this different behaviour with PlatformIO)
Usually, the code line fsm.add(timedTransitions, num_timed); lets crash my esp32:

Sometimes, the code is running fine until first hardware reset. After first reset, each program start ends up with same crash.
There is one exception about the crash: Adding timedTransitions after adding standard transitions works well. Working with standard transitions works always.
This crashs my esp32
  fsm.add(timedTransitions, num_timed);
  fsm.add(transitions, num_transitions);

This works well
  fsm.add(transitions, num_transitions);
  fsm.add(timedTransitions, num_timed);

Do you have any idea?
I am working with PlatformIO on different projects, usually it doesn't matter if compiling with PIO or Arduino-IDE. I want to use your FSM in my next project, therefore, I would like to know about this strange behaviour.
Thank you.

Hey, thanks for reaching out. And thank you for pointing out that the version is not on platform.io. That's strange. Will check it out.
And gals that you like the lib. Wasn't also happy with most others. That's why I did mine back then. 😃

still v1.0.0 on platformIO

Hey, thanks for pointing it out.
Just published the new recent version to PlatformIO.


Pydantic field with const should not be editable

When declaring a Pydantic field as constant, the Prefect UI shows the default value, yet it still allows for editing it.
Tested with Prefect 2.14.10.
Expected behaviour
Prefect UI should render the field as non-editable.
Minimal example
from typing import Union
from prefect import flow
from pydantic import BaseModel, Extra, Field

class CustomConfig(BaseModel, extra=Extra.forbid):
    value: float

class DefaultConfig(BaseModel, extra=Extra.forbid):
    value: float = Field(
        default=0.1,
        const=True,
    )

class FlowConfig(BaseModel, extra=Extra.forbid):
	config: CustomConfig | DefaultConfig

@flow()
async def my_flow(
    configuration: FlowConfig,
) -> None:
    # do something
	...

@NicholasFiorentini from my testing this has been fixed in our enhanced deployment parameters ui which is available in cloud and will be coming to open source soon.

Closing this issue now that enhanced deployment parameters are available in oss and cloud. Thank you for reporting!


os: regression in os.Expand treatment of '${}'

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
tip
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/iand/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/iand"
GORACE=""
GOROOT="/opt/go"
GOTMPDIR=""
GOTOOLDIR="/opt/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build689803627=/tmp/go-build -gno-record-gcc-switches"
What did you do?
package main

import (
	"fmt"
	"os"
)

func main() {
	fmt.Printf("%q", substituteEnv("${}"))
}

func substituteEnv(s string) string {
	return os.Expand(s, func(key string) string {
		if val, ok := os.LookupEnv(key); ok {
			return val
		}
		return key
	})
}

What did you expect to see?
An empty string printed.  The Go 1.10 behaviour is that os.Expand returns an empty string.
Can be see on playground which is currently running Go 1.10.2
What did you see instead?
The $ is printed without the braces.
The CL that introduced the new behaviour: https://github.com/golang/go/commit/7bf631e1fc55839a6bff404a91597326e1addc10 derived from https://github.com/golang/go/issues/24345

Well, that was intentional. Not sure if something can be done about it. /cc @ianlancetaylor

I think I'd expect it to either output ${} or an empty string. $ on it's own seems wrong.

None of the added test cases in that CL test for ${}, nor do they make any characters disappear, so I think this was a mistake. Will look into a fix.

Thanks @mvdan. I was afraid we would be adding checks for more edge cases. But I guess it's better not to break existing behavior.

@agnivade indeed. I'm more concerned about consistency than existing behavior, but both are important. See my comment on the CL for an alternative approach that doesn't keep existing behavior.


Update runc to v1.0.0-rc91 and pickup latest containerd/cri

https://github.com/opencontainers/runc/releases/tag/v1.0.0-rc91
There were changes in containerd/cri that needed to be picked up as well for v1.0.0-rc91 runc to work.
Signed-off-by: Davanum Srinivas<EMAIL_ADDRESS>

Needs https://github.com/containerd/cri/pull/1521

Need to rebase because of https://github.com/containerd/containerd/pull/4356

https://github.com/containerd/containerd/pull/4239 was merged, so you can drop the urfave, and gomd2man bumps from this PR
I also opened https://github.com/opencontainers/runtime-spec/issues/1052 to ask the runtime-spec maintainers to consider a new release (not a blocker for this PR)

linter fails with the following: We need @AkihiroSuda 's PR to land first in https://github.com/containerd/cri/pull/1521
level=warning msg="[runner] Can't run linter unused: buildssa: analysis skipped: errors in package: [/home/runner/work/containerd/containerd/src/github.com/containerd/containerd/cmd/containerd/builtins_cri.go:21:10: could not import github.com/containerd/cri (/home/runner/work/containerd/containerd/src/github.com/containerd/containerd/vendor/github.com/containerd/cri/cri.go:45:2: could not import github.com/containerd/cri/pkg/server (/home/runner/work/containerd/containerd/src/github.com/containerd/containerd/vendor/github.com/containerd/cri/pkg/server/container_create.go:36:13: could not import github.com/containerd/cri/pkg/containerd/opts (/home/runner/work/containerd/containerd/src/github.com/containerd/containerd/vendor/github.com/containerd/cri/pkg/containerd/opts/spec_unix.go:327:13: cannot use dev.Permissions (variable of type configs.DevicePermissions) as string value in struct literal)))]"
level=warning msg="[runner] Can't run linter goanalysis_metalinter: S1005: failed prerequisites<EMAIL_ADDRESS><EMAIL_ADDRESS>

need to look into that spurious critest pipe failure.. anyhow restarted tests and it's fine


DSpace Cannot send email to TLSv1.2+ only servers

Describe the bug
DSpace 6.x and lower cannot send email to mail server that only support TLSv1.2 and higher.
This is due to DSpace 6.x using JavaMail 1.4 which do not support TLSv1.2. Google search say that upgrade to JavaMail 1.6 will fix this problem.
To Reproduce
Steps to reproduce the behavior:

Set mail.server to SMTP server that only support TLSv1.2 and up.
Try dspace test-email.
Get "Error: javax.mail.SendFailedException: Send failure" exception.

Expected behavior
Able to send email via TLSv1.2 server

Verified that currently we are running JavaMail (javax.mail) version 1.4.7 on both 6.x and 7.x. See parent POM in both which has this:
<dependency>
      <groupId>javax.mail</groupId>
      <artifactId>mail</artifactId>
      <version>1.4.7</version>
</dependency>

Upgrading may be as simple as just changing this dependency to the latest version...haven't tried it yet myself though

Fixed by #9321 as we'll be updating to jakarta.mail v2

Closing, realized this is already fixed on dspace-7_x and main: https://github.com/DSpace/DSpace/blob/dspace-7_x/pom.xml#L1535


debug - errors in console

#5644

debugged a C# app that uses Tasks
found exception in the dev tools console

//Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:117
source: property 'sourceReference' is missing: Error: source: property 'sourceReference' is missing
    at file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:79:17181
    at Object.p [as _notify] (file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:23:22802)
    at Object.__dirname.undefined.F.enter (file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:23:26130)
    at __dirname.undefined.t.Class.define._run (file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:23:27451)
    at __dirname.undefined.t.Class.define._error (file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:23:26968)
    at file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:79:12410
    at t.e.dispatch (file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:79:13482)
    at t.e.handleData (file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:79:13006)
    at Socket.<anonymous> (file:////Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:79:12276)
    at emitOne (events.js:90:13)e.onUnexpectedError @ //Applications/Visual Studio Code - Alpha.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:117
winjs.base.js:47WARNING: Promise with no error callback:undefined

This error is coming from the c# debug adapter, and you are seeing it due to issue #5828.
If you have repro steps you could file it agains the c# debug adapter

Filed https://github.com/OmniSharp/omnisharp-vscode/issues/248

@isidorn this doesn't sound like a C# extension issue, so I am closing our side.

@gregg-miskelly ok, then I will reopen on our side to investigate

Did not see this again. Closing until I hear more complaints with steps


This does not work for Ionic 3

Here ionic info
`cli packages: (/Users/wilfredonoyola/workspace/statusfy/sf-ionic-1/node_modules)
@ionic/cli-utils  : 1.19.2
ionic (Ionic CLI) : 3.20.0

global packages:
cordova (Cordova CLI) : 8.0.0

local packages:
@ionic/app-scripts : 3.1.9
Cordova Platforms  : android 6.3.0 ios 4.5.4
Ionic Framework    : ionic-angular 3.9.2

System:
Android SDK Tools : 26.1.1
ios-deploy        : 1.9.2
Node              : v6.11.3
npm               : 3.10.10
OS                : macOS High Sierra
Xcode             : Xcode 9.3 Build version 9E145

Environment Variables:
ANDROID_HOME : /Users/wilfredonoyola/Library/Android/sdk/

Misc:
backend : pro`

it did not work for me too

for ionic3 ,it works well at Android4.4.2, but it did not wok at Android6.0.0.

Feel free to implement the fix, verify it carefully, update the documentation and then make the pull request to the master branch.

I use the launcher APP Instead of cordova-plugin-autostart，
@luckxiang How do you use https://www.npmjs.com/package/@ionic-native/app-launcher to autostart an app? Is it possible to catch the event when app is going to close?

@MInesGomes  I'm sorry,  the time is too long. I went to C development now.


SVG Native Spec

Is your feature request related to a problem? Please describe.
Adobe & Apple are developing a SVG Native subset of SVG. In addition, there appears to be a private SVG library on iOS. Putting these two together, you might conclude Apple may eventually support SVG Native images in iOS without a webview. In addition, Adobe already has a SVG Native renderer for iOS
It would be great if the SVG output also conformed to this subset. It looks like it's already mostly compliant. Currently I can see the following issues:

Use of svg inside the main svg [1], [2], [3]
em units - I can't quite work out where these are used, but I do remember seeing them
Text - I think this is just a fallback - maybe just a way to turn this off and throw an exception would suffice

And I noticed these too - but not certain they would actually show up if someone was to make an iOS library

foreignObject

Describe the solution you'd like
^^
Describe alternatives you've considered
Nil
Additional context
Nil

Thank you for your suggestion.  We will keep it in mind, but it may not be a high-priority item.  The nested SVG is actually a significant feature in MathJax, so removing could be problematic.  But it could probably be done if one were willing to accept a loss of some functionality.

Would you accept PRs for the feature?
It would be useful to know what features you’re relying on for the bested SVGs. I know it’s clipping - but not really sure what else it does

We will always look at PRs, though there is no guarantee that they will be accepted.
The main use of the nested SVGs is to handle equation numbers so that they properly float at the right (or left) margin while the display equation stays centered (or left- or right-aligned) when the container size changes.  That might not be a concern for your use case, but that is the crucial case that I was thinking of.  The clipping could probably be handled in other ways.  The text uses for characters not in the MathJax fonts.  That is currently important, but will be less so with version 4.0, which has much greater character coverage.  The foreignObject usage is for adding the content of semantic annotations (or in v4.0, the new HTML-in-MathML or HTML-in-TeX features).  It is something you may be willing to lose.  There may be some scaling done in em units in order to handle the foreign objects and text fallbacks.  The main layout should be unitless or px units.  There is also some CSS styling that would have to be moved to inline attributes.
There are significant internal updates that are part of the upcoming v4.0 release, so if you plan to make these changes, I would suggest you start with the v4.0.0-alpha branch rather than the develop branch, as that will make your changes more applicable to the future codebase.

@dpvc do you have an example of this resizing behaviour?
For clipping, I was thinking we need to be careful that multiple SVG IDs don't conflict with each-other when inline in the HTML source. I think we currently have that case for text characters, but it doesn't really matter, since the content would be the same. One idea would be to take a string-hash of the input source, then suffix it with an incremental ID for each <clipPath> in SVG


Lack of documentation from migrating from v1 to v2.

Problem here is that we're using the v1 docs for Jointable Foreign Key https://v1.gorm.io/docs/many_to_many.html#Jointable-ForeignKey and there does not seem to be a clear path to have this supported in V2.
https://gorm.io/docs/many_to_many.html#Override-Foreign-Key
Can we have a clear description about these terminologies: foreignKey, references, joinForeignKey, joinReferences

Like https://gorm.io/docs/associations.html#tags ?

Like https://gorm.io/docs/associations.html#tags ?


GH action: add SBOM + scanner for dockerfile and docker images

GH action: add SBOM for linux docker images+ scanner for dockerfile
- use trivy to scan all Dockerfile.release.full
- use syft to generate sbom from images we build
- upload to codeql for trivy scan result from image
- windows is not working for sbom and trivy yet
sbom result artifacts: https://github.com/zdtsw/containers/actions/runs/2882299165
scan result: https://github.com/zdtsw/containers/actions/runs/2882299167

looks like  Specify at least 1 USER command in Dockerfile with non-root user as argument is the worst thing in all our dockerfile
some others might need to be addressed too
https://avd.aquasec.com/nvd/2022/cve-2022-37434/
https://avd.aquasec.com/nvd/2020/cve-2020-16156/
https://avd.aquasec.com/nvd/2016/cve-2016-2781/

This PR has no related issue for it.
After discussion in the PMC call today, I will create an issue so we can discuss a best path forward, thinking about the design and benefits and maintenance implications of such a change and to talk about the scanning that occurs upstream already, in order to compare what is being checked / scanned to avoid duplication.

@tianon / @yosifkit do either of you two have any thoughts about this? It would be good to understand what scanning the official images already go through prior to publishing. I wonder if this is the most appropriate place to have this code (perhaps it's better in the upstream GitHub action that is maintained by your team?

Created https://github.com/adoptium/containers/issues/267 for discussing best approach and location for such scanning and work to take place.

Thanks for the ping, @gdams :heart:
I would say that at the Dockerfile source level is probably not the best place to check for issues with the final built images. :sweat_smile:
That being said, this is definitely a priority for Docker - there's a lot of work in BuildKit to generate SBOMs directly at build-time (see especially https://github.com/moby/buildkit/issues/2773 and linked PRs), and that's the direction we're planning to take. :+1:

@zdtsw can this be closed now given the comments above?


feat(common): remove independent packages

It's found that the following packages in "universe.common" are not referred by others, so we are wondering if we can remove them during porting.
glog_component: this package provides the glog (google logging library) feature as a ros2 component library.
goal_distance_calculator: this node publishes deviation of self-pose from goal pose.
path_distance_calculator: this node publishes a distance from the closest path point from the self-position to the end point of the path.
tier4_debug_tools: this package publishers tf2 pose and lateral error.
time_utils: provided by APEX, transform ROS2 time to standard C+++ time.

Note: Confirm the contribution guidelines before submitting a pull request.
Click the Preview tab and select a PR template:

Standard change
Small change

These are optional packages and are not included in the default dependencies, but they should be keep available.

However, some packages may not really be used, so we should check it.

However, some packages may not really be used, so we should check it.

Thanks for the quick reply, glad for further information

@JianKangEgon
The path_distance_calculator is used in https://github.com/autowarefoundation/autoware.universe/blob/main/launch/tier4_autoware_api_launch/launch/deprecated_api.launch.xml#L18
and
https://github.com/autowarefoundation/autoware.universe/blob/main/launch/tier4_autoware_api_launch/package.xml#L21 .

Thank you for the PR! In my understanding,

glog_component: used in some packages for debugging
goal_distance_calculator and path_distance_calculator: this is a debug tool. It may not be used in runtime, but it is used for debugging.
tier4_debug_tools: this is a debug tool. It does not have a dependency but is used for debugging.
timer_utils: this is a static library. If this does not depend on anything, maybe we can remove it.

Oh thanks for updating, I used the older version of Universe in September. Sure, some useful packages are supposed to be kept.


15841 added new Dev LD key to sample config file

Issue #: bcgov/entity#15841
Description of changes:

added common LD project key (Dev) to example env file
app version = 4.2.10

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

I usually go find the key in the network tab by searching for the client ID. This is a great change. Thanks Sev.

Yes, that works. You can also go to the Sources tab, open the app JS bundle, and search for the key name.


doc: describe Hashicorp Vault with a ServiceAccount per Tenant

In addition to the single ServiceAccount KMS support for Hashicorp
Vault, Ceph-CSI can now use a ServiceAccount per Tenant as well. This
adds the user-documentation with references to the example deployment
files.
Closes: #2222

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)
/retest all: run this in case the CentOS CI failed to start/report any test
progress or results

@mergifyio rebase
@humblec it seemt Mergify dropped the approvals again, could you re-add yours and maybe add the [ready-to-merge] label?


Please don't pull in wasm dependencies on non-wasm platforms

async-h1 depends on async-std with the unstable feature; that in turn pulls in wasm dependencies via futures-timer then gloo-timers. Please consider adjusting the feature flag dependencies to avoid pulling gloo-timers and its pile of wasm-specific dependencies on non-wasm platforms.

https://github.com/async-rs/async-std/issues/823

Closing this issue because there's nothing we can do to resolve it here or in the other http-rs crates that have this problem, but please add your support to async-rs/async-std#823

Does async-h1 need the "unstable" feature of async-std? Could it have a finer-grained dependency?

That's a great point. Async-h1 itself does not

Unfortunately, this does not really fix the problem — async-h1 depends on http-types, which uses async_std::sync::channel, which is gated behind unstable and async-std does not have a more granular set of features to enable: https://github.com/async-rs/async-std/blob/master/src/sync/channel.rs#L61


Installing with angular-cli/webpack

Hi, I am trying to install the timepicker with an angular-cli/webpack project so am unable to edit the system.config.js directly as in the instructions:

map[‘ng2-datetime-picker'] = 'node_modules/ng2-datetime-picker/dist';
packages[‘ng2-datetime-picker'] = { main: 'ng2-datetime-picker.umd.js', defaultExtension: 'js’ }

I have implemented the other steps but am unable to get the timepicker appearing. Could you kindly advise whether I would need to take alternative steps to map in this environment? I would be grateful for any help you may be able to give. Many thanks!

Hmm, I thought this line can make webpack compatible.
https://github.com/ng2-ui/ng2-datetime-picker/blob/master/package.json#L7

I have tested with webpack and it seems working fine for me. Please verify the change.

Hi, my apologies, it seems the issue was coming from one of the dependencies and a JQuery version it required, which was conflicting with anther package. However, this now seems to work fine. Thank you for your help.


Idea to make Vtor simpler

how about make it simpler by:
Max matches MaxConstraint and Foo matches FooConstraint,  linked by classFullName
@Constraint(XxxConstraint.class) is not necessary.

Nice idea!!! Definitely on todo!

Sorry for taking too long :(


Resize pane

Requirements
Atom    : 1.23.1
Electron: 1.6.15
Chrome  : 56.0.2924.87
Node    : 7.4.0
Description of the Change
Atom gets into a perplexing state for some people (for example) whether by some combination of keys or package actions where there is only one pane, but it does not take up the full width of the editor window. This changes in this Pull request prevent atom to get into this state.
Alternate Designs
NA
Why Should This Be In Core?
Fixes a bad behavior in atom UI
Benefits
Added test for the new code
Better experience in atom UI
Possible Drawbacks
None that I can think of.
Applicable Issues
It's applicable to #11673

@50Wliu Thanks, I'll correct them

Thanks!  You'll also need to remove the package-lock.json files you checked in as Atom won't build if they're present.

Got it, I'll delete them

Looks great, nice work! ⚡


translate ng-bind which is inside angular directives template

I have one directive which is creating dynamic control.Directive's template is as below
"<div class=\"form-group\">\n <input type='text' ng-model=\"value|translate\" class='form-control'/>\n </div>"

where in my directive scope i have :
scope.value = 'VALUE';

and in config section i have :
.config(['$translateProvider', function ($translateProvider) {
    $translateProvider.translations('en', {
        'VALUE' : 'Some value'
    });

    $translateProvider.translations('ar', {
        'VALUE': 'بعض القيمة',
    });

    $translateProvider.preferredLanguage('en');
}])

but it gives nonassign error : Error: [ngModel:nonassign]
how ever by applying filter on scope.value in directive and removing translate filter from ng-model solves problem.
$filter('translate')(scope.value);

and changing ng-model='value|translate' to ng-model=value
But when user changes language it will not translate this pre-filled value of textbox because its assigned it by translating it as value and filter is not applied on it.
here is fiddle : http://jsfiddle.net/jugnu_pathak/X2fsw/47/

What is your problem? ng-model='value|translate'? Well, that's not valid AJS. A model cannot contain a filter. Even it would be, how should that work? A model reflection by translation?
You are looking for a change of the value of value but not the reflection itself, I guess.
And I think you are looking for the HTML input's placeholder attribute?!
Additionally, your demo code was a little bit confusing. See result here http://jsfiddle.net/mk88745o/1/

You are still using an outdated AJS 1.2, please update.
You should not use minified code in demos, that's bad for reading error messages.
You had written a directive a) in a very uncommon (and at least in this case not needed) way and b) skipping the AJS template binding (other than you described here!). There is no need for jQuery, at least a self made template injection will obviously broke the binding feature. Too much code, too less functionality.

Hope that helped you. I'm closing in forward, but feel free to write.


Since 0.13.7 + manipulating layout in a function doesn't update actual layout

I have a ReactGridLayout looks like this:
<ReactGridLayout ... onLayoutChange={this.layoutChange} className="layout" layouts={layouts} rowHeight={this.rowHeight} width={this.state.pageWidth}>
I then have a function that manipulates layouts by setState. In 0.13.6 the "actual" layout is changed and works fine. But in 0.13.7 + the state will change and will be but as a prop in  but the "actual" layout, the layout shown on the screen is NOT changed.

can you post the function code?

Sure, here's the code:
    changeBoxSize: function (index, size) {
        var layout = this.state.wideLayout;
        var w = 0;
        var h = 0;
        var changeState = false;
        if (size.height !== undefined) {
            while (this.getHeightFromPixels(h) < size.height) {
                h++;
            }
            if (this.getHeightFromPixels(h) - size.height > this.columns.lg / 2) {
                h--;
            }
            if (layout[index].h !== h) {
                layout[index].h = h;
                changeState = true;
            }
        }
        if (size.width !== undefined) {
            while (this.getWidthFromPixels(w) < size.width) {
                w++;
            }
            if (layout[index].w !== w) {
                layout[index].w = w;
                changeState = true;
            }
        }
        if (changeState) {
            this.setState({
                wideLayout: layout
            });
        }
    },

whats triggering the logic for changing the layout?   i didnt have an issue with layout but i was having an issue with ES6 and binding my click functions to this. layout changes appear to work in this version for me but they are API triggered, so thats why i thought maybe syntax

Its send from a component living inside a grid-item. It has specific width/heights and therefore I have to send info to the layout how big it is. Im able to change the layout, thats no problem, the thing is the actual layout I see on the screen isnt the same as the layout Ive set.

Can you reproduce this in http://www.webpackbin.com/VymTE3zWG?

Here you go: http://www.webpackbin.com/Nk5cKsEPM

Would you care to explain how to replicate it within that example?

@STRML Sorry. I've updated it to be a little more clear (http://www.webpackbin.com/Nk5cKsEPM). Let me explain what happens.

The grid renders it's items which are <MyComponent /> elements and passes them a didUpdate callback.
The item checks its internal height and then runs the didUpdate callback.
The didUpdate callback finds that the component is taller than its assigned height, and so updates the layout to give it an appropriate height and then uses setState to set the layout to the updated version.
The grid re-renders, but somehow still uses the original layout. You can verify this by looking at the contents of the item. The text will show the height that the item is supposed to be while the actual height of the item will be too short.

@STRML @FerusAndBeyond Turns out it was how the layout was being cloned that was the issue. I was doing this:
let layouts = _.clone(this.state.layouts);
layouts.lg[i].h = newHeight
this.setState({layouts: layouts});

But since _.clone clones the outer object but keeps the references within the object, when the componentWillReceiveProps function onResponsiveReactGridLayout does an _.isEqual call, it found that they were actually equal since I had already modified the reference.
What I ended up doing was using https://www.npmjs.com/package/clone to do a deep clone instead of only cloning the outer object, so my final solution looks like:
const clone = require('clone');
let layouts = clone(this.state.layouts);
layouts.lg[i].h = newHeight
this.setState({layouts: layouts});

That's right, it does work (I use $.extend instead of lodash)


MFA Azure Sync Account

This section: Under Exclude, select Users and groups and choose your organization's emergency access or break-glass accounts.
should include a reference or a note to the Azure AD Connect sync account per the note above as it could be frequently missed following these steps.
Thanks!!!

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

ID: ca2fb784-f7e9-6fad-9889-2dc6639bc0e1
Version Independent ID: 8b90e35e-6918-1073-85f6-ad47786033c6
Content: Conditional Access - Require MFA for administrators - Azure Active Directory
Content Source: articles/active-directory/conditional-access/howto-conditional-access-policy-admin-mfa.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Dupo24
Thanks for your feedback! We will investigate and update as appropriate.

Thanks. Like you said it is noted at the beginning of the article to exclude service accounts such as the Azure AD Connect Sync account, but I made a pull request to mention these in the "Exclude" section too. https://github.com/MicrosoftDocs/azure-docs-pr/pull/200157


add WKT geometry parser

in https://github.com/developmentseed/geojson-pydantic/pull/64 we added .wkt property to return geometry as WKT. I know it's not the main purpose of this library but I think it will be nice to have the ability to go the other way around!

well there are already nice tools to do that:

https://github.com/larsbutler/geomet
https://github.com/cleder/pygeoif
...

I'm going to close this issue, if we were to add this feature we would mostly copy/paste code from ☝ so it doesn't make sense


Hooks causing re-rendering of items

Hi, im currently working on an infinite scroll of items (prolly more than thousands), where each item is attached to useDraggable and useDroppable hooks to allow them to be draggable/droppable among each other. However, i am having problem of the hooks triggering the re-render of the thousands of items whenever im trying to drag a single item into another one. I tried setting the disabled to false for some of the items, but it will still cause them to re-render. Hence, would like to know does anyone face this kind of use case before and is there any way to handle this.
Thank you.

Have you considered using a virtualized infinite list? It's generally not a good idea to try to add drag and drop functionality to a list of thousands of items, it will result in severe performance concerns.

Hi, thanks for the reply. I am now using react-virtualized so that not all items are re-rendered on drag. But, i realised that for example when i am at item 570 - 600, even though the 'draggableNodes' in the context are only these 30 nodes, but the preview of the custom drag overlay is much more laggier than if i am at item 1 - 30. Do u know what could be the reason? Thanks

I'm not sure, it shouldn't make a difference in theory. Can you share a codesandbox replicating the issue?


"bugdown" is a confusing name

At the sprints today, a newer contributor thought "bugdown" was a means of finding bugs in the Markdown processor. Perhaps we should find a better name.

@zulipbot claim

@zulipbot  Work In progress: I'm setting up the vagrant dev environment on Linux OS to explore zerver/lib/bugdown/ and understand how it works so I best decide what name suits it and possibly suggest more features!

@Ijebusoma welcome!
If you have any problems setting up the dev environment or with this issue, you can ask here or over at http://chat.zulip.org.

@tommyip  Suggested name: Zintax

I think probably zerver.lib.markdown would be a reasonable name.  Maybe we should try going for that?

We have fixed this by renaming bugdown to markdown. See https://chat.zulip.org/#narrow/stream/3-backend/topic/zerver.2Elib.2Ebugdown for more details.

Don’t forget to leave links to the relevant PRs. This was #15569,  #15606, and #15670.


Early returning from on_connect results in panic

Hi there, great work on this crate
I'm using an on_connect implementation such as the following to have the server kill a connection if it doesn't pass some checks (specifically, using axum to extract headers and validate data passed within them)
async fn on_connect(&mut self, socket: Socket, address: SocketAddr, args: <Self::Session as SessionExt>::Args) -> Result<ezsockets::Session<<Self::Session as SessionExt>::ID, <Self::Session as SessionExt>::Call>, Error> {
        info!("New connection from {}, {:?}", &address, &args);
        return Err(anyhow!("Not implemented!").into());
}

which results in the following panic (removed full backtrace, will attach if relevant)
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: ()', /Users/roee/.cargo/registry/src/github.com-1ecc6299db9ec823/ezsockets-0.5.1/src/server.rs:248:14

which leads me to the following snippet in server.rs
pub async fn accept(
        &self,
        socket: Socket,
        address: SocketAddr,
        args: <E::Session as SessionExt>::Args,
    ) -> <E::Session as SessionExt>::ID {
        let (sender, receiver) = oneshot::channel();
        self.connections
            .send(NewConnection {
                socket,
                address,
                args,
                respond_to: sender,
            })
            .map_err(|_| ())
            .unwrap();
        receiver.await.unwrap()
    }

A potential solution would be to eliminate the unwrap from both expressions, or modify the return type of the 'accept' function to an enumeration that includes either a session or an exit code.

Hey,
if you want to make some checks on HTTP headers, it's best to do it in your own axum HTTP handler, instead of in the ServerExt::on_connect implementation.
I made an example of this, check it out here
https://github.com/gbaranski/ezsockets/blob/master/examples/chat-server-axum/src/main.rs#L139-L154


fix(ui): disable touch for v-tabs-items

closes #399

Pull Request Test Coverage Report for Build 528078403

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 21.419%

Totals

Change from base Build 527981517:
0.0%

Covered Lines:
2049

Relevant Lines:
9795

💛  - Coveralls

@ahochsteger This it's a workaround, IMO the values should be displaied in a single column instead

Check my other PR, I'm fixing this there

@robertsLando ok, thanks for letting me know.


Patched synapses added

Patched synapses added

Patched synapses added, please review.

please accept the last rq only
current rq is pull 2

pull 3

Last one if anything has changed. Also please take care of the RateCell display_name

Ive sent the pic to William's discord to check


Failures on [vm/compiler] Fix Mac builds and disassembler test.

There are new test failures on [vm/compiler] Ensure we emit catch entry move for receiver when it is captured.
The tests
standalone_2/dwarf_stack_trace_obfuscate_test RuntimeError (expected Pass)
standalone_2/dwarf_stack_trace_test RuntimeError (expected Pass)
vm/dart/use_save_debugging_info_flag_test RuntimeError (expected Pass)

are failing on configurations
dartkp-linux-debug-x64
dartkp-linux-product-x64
dartkp-linux-release-simarm64
dartkp-linux-release-x64
dartkp-mac-release-simarm64
dartkp-no-bare-linux-release-simarm64
dartkp-no-bare-linux-release-x64
dartkp-obfuscate-linux-release-x64
dartkp-win-release-x64

@sstrickl Teagan, could you look at these failures. I don't think my change should have affected these tests, so maybe there is a bug in the DWARF parser?

Am checking into it. Initial finding is that it looks like we're generating a DW_LNE_set_address opcode with size 5 (32-bit pointer following extended opcode) but writing a 64-bit pointer after it (so should be size 9). Given that, it's not the DWARF reader, but the DWARF writer that's at fault here.
... and looking at the DWARF writer, the problem is that we're writing sizeof(compiler::target::kWordSize), or sizeof(int), instead of just the value of compiler::target::kWordSize. Adding a fix now.


Incorrect step in "Retrieve an item using the SDK"

Step 3A for "Retrieve an item using the SDK" should not be there. It looks like it was left in by mistake from "Add items to a container using the SDK". Please remove it

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

ID: af90dbc6-bff5-0f9a-7969-ac94b0352987
Version Independent ID: ec5de32b-ad0a-be2f-144f-6376ca55ccd9
Content: Tutorial: Develop a .NET console application with Azure Cosmos DB for NoSQL

Content Source: articles/cosmos-db/nosql/tutorial-dotnet-console-app.md
Service: cosmos-db
Sub-service: nosql
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

@perktime
Thanks for your feedback! We will investigate and update as appropriate.

Hi @seesharprun,
Could you please look into this request, thanks.

@perktime, it looks like this was already fixed.
#please-close


Use random.choices when selecting a task instead of random.choice

Prerequisites

[X] I am using the latest version of Locust
[X] I am suggesting a new feature, not asking a question

Description
According to current implementation/documentation, if weights are given to tasks, tasks are stored in the list duplicated by the amount given as weight, and then the task is selected from the list by using Python's random.choice.
This, however, has two shortcomings:

Inserting big weights can lead to unreasonably big datastructures
Only integers can be weights

It should be fairly easy to rewrite the gen_tasks_from_base_classes function and how it's output is handled in TaskSet class to store the weights separately and pass them together with the list of tasks to random.choices in gen_next_task.
This will allow to store arbitrarily large weights without duplicating tasks in integer or float as well as potentially enable the functionality to modify task weights at runtime by modifying the list storing the weights.
I could try to implement this with a PR if this change is desired.

Sounds great!

@bakhtos Any chance you’ll have time to take a look?

@cyberw Yes I actually finally got around to working on this two days ago!
I think I already changed the list to the dict in the class itself, now I need to figure out which external things are affected, like helper functions and tests.


Performance issues and memory leak when loading textures and rendering via QuadRenderer -> RenderWindow

Also had to annotate the splicer to be able to connect it to the group.
Attached patch (_StridePerf.vl) demonstrates the issue. Also included a beta patch to create dummy textures.
Perftest.zip

2021.4.9 preview 971 and it seems no longer to be an issue.


XML properties in webdav response not properly encoded

On the xml webdav responses some properties are not properly encoded
for example

filename
expected value (Value returned by oc10)
Actual value

/folder &2.txt
webdav/file%20%262.txt
webdav/file%20&2.txt

/C++ folder
webdav/C%2b%2b%20folder
webdav/C++%20folder/

@dpakach is this still an issue?

@dpakach is this still a problem?
I don't see this issue mentioned in expected-failures.

Just checked on the latest master of ocis. Can confirm that it has been fixed.
  <d:response>
    <d:href>/remote.php/dav/files/admin/C%2b%2b%20folder/</d:href>
  </d:response>
  <d:response>
    <d:href>/remote.php/dav/files/admin/folder%20%262.txt/</d:href>
  </d:response>


Revert "Refactor and fix style for some specs."

Reverts Homebrew/brew#4068
RE: https://github.com/Homebrew/brew/pull/4068#issuecomment-383358335

CC @reitermarkus


How to set showsMicrophoneButton property using showRPSystemBroadcastPickerView

Native Code:
RCT_EXPORT_METHOD(showRPSystemBroadcastPickerView) {
    if (@available(iOS 12.0, *)) {
        dispatch_async(dispatch_get_main_queue(), ^{
            NSURL *url = [[NSBundle mainBundle] URLForResource:nil withExtension:@"appex" subdirectory:@"PlugIns"];
            NSBundle *bundle = [NSBundle bundleWithURL:url];
            if (bundle) {
                RPSystemBroadcastPickerView *picker = [[RPSystemBroadcastPickerView alloc] initWithFrame:CGRectMake(0, 0, 100, 200)];
                picker.showsMicrophoneButton = YES;
                picker.preferredExtension = bundle.bundleIdentifier;
                for (UIView *view in [picker subviews]) {
                    if ([view isKindOfClass:UIButton.class]) {
                        [((UIButton*)view) sendActionsForControlEvents:UIControlEventAllTouchEvents];
                    }
                }
            }
        });
    }
}

I found picker.showsMicrophoneButton = YES default in native code.
How we set it dynamically from react native code?

you can create a new export method in your project.

It will export as a param at v4.2.0


WIP: Add a custom resource for reloading systctl data

Previously we managed procps and reloaded it if an attribute was set. We did that through some funky notifications. The problem is systemd based systems have their own systemd way of reloading sysctl data. This is a super simple resource that just decides the best thing to do and does it. We notify that resource instead of the ruby block / service restart stuff before.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

@rshade so I couldn't actually get the CD-ROM stuff to work in virtualbox locally, on any of the above platforms.
I've look at using something from ipv4. instead.

Pulled this into #103


Adding solution for queue class

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Hi, this is my first ever open source pull request. Please let me know if I missed anything.
Thanks

We're waiting for #36278 to land
cc @RandellDawson

Congrats @prashanthashok on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝

@thecodingaviator @RandellDawson  Thanks for reviewing my PR and thanks @freecodecamp for the opportunity! :)


Possibility to list/display outdated lib

I wondered if it was possible to show the lib that is referred to make it possible to assess how urgent a restart is (sure you can guess some things from pacman updates, but maybe there were a lot of lib updates).
Besides my rather cosmetic request: very good replacement of overdue, I like it!

Actually, this is already implemented: longoverdue info <regex> (the regex is passed to pgrep).
I can probably add a verbose switch to the main command as well.

@IdleGandalf is this still something you want or can I close this?

Hm a flag or something to show it in main out would be pretty awesome, alternative adding/modifying info would be an option too. I tested info and it requires to specify the program, so info is not really suited to get an overview after an update in the current state.

That's what I thought. I'll see what I can do.

Here you go. I hope you like what I came up with.

Looks good. Thanks!


Attaching images to a work item will make unsaved added text outdated, and forced to discard

This bug seems to have been introduced sometime in the last 6 months or so. I haven’t reported it because I haven’t been irritated enough to actually sit down and take the time to capture everything I needed to report it. Here are the replication steps:

Create a work item (I don’t think it matters what type it is, but my screenshots are of a story)

Add a title and a few paragraphs of important details into the body of the work item

Attach an image from your computer

Click on the “Save and close” button

The result: The story is identified as being “changed by someone else since you opened it.”… but that’s not true. I have simply attached images and it has overwritten the save.

This is extremely frustrating when you have added a bunch of details into a story and committed SERIOUS time into filling out these work items in a BUNCH of different area’s of the work item to only find out that after you attached your image it killed everything. I have taken the time to copy and paste all of the different text area’s into a text editor in order to re-copy it back over, but it takes a ton more time to do that.

Me and my colleagues has also noticed this bug.


Change install include files

Install only C/C++ headers.
Also fixes missing headers (location.hh, etc.) when including in own files.

I tested again and seems to work without flags, did you use any?
Can you test again? It was probably the CMAKE_CURRENT_BINARY_DIR variable messing up

OK, now the C directory is getting created, but the generated headers (e.g., c/GraphQLAst.h) aren't included:
$ find
.
./include
./include/graphqlparser
./include/graphqlparser/AstNode.h
./include/graphqlparser/c
./include/graphqlparser/c/GraphQLAstNode.h
./include/graphqlparser/c/GraphQLAstVisitor.h
./include/graphqlparser/c/GraphQLParser.h
./include/graphqlparser/GraphQLParser.h
./include/graphqlparser/JsonVisitor.h
./include/graphqlparser/lexer.h
./include/graphqlparser/location.hh
./include/graphqlparser/position.hh
./include/graphqlparser/stack.hh
./include/graphqlparser/syntaxdefs.h
./lib
./lib/libgraphqlparser.dylib

I tested using an out-of-tree build: mkdir build && cd build && cmake .. -Dtest=1 -DCMAKE_INSTALL_PREFIX:PATH=/Users/swolchok/libgraphqlparser/install && cmake --build .

I think the problem is that the GLOB method can't really support out-of-tree builds. I'll send out a change to at least include the .hh files.

I just pushed https://github.com/graphql/libgraphqlparser/commit/3675c74bf6f4249bf6d32708073c0f401b92c32e to install the missing headers. Feel free to rebase and get this working with out-of-source builds!

closing because this has gone stale; feel free to open another PR


Isolate network objects from business logic objects

Our localization "mumbo jumbo" is to large extent caused by the fact that we incorrectly reuse objects between various software layers. In ideal world, we have (simplified):
// Business level

class OperationBO {
   private String title;
}

OperationBO operationBO = new OperationBO();
operationBO.setTitle("payment.localization.key")

/// ...

// Network level
class OperationVO {
    private String title;
}
OperationVO operationVO = new OperationVO();
operationVO.setTitle(i18nService.getKey(operationBO.getTitle(), locale));

I agree, but we should do this change with future OperationFormData redesign in mind. See also related issue:
https://github.com/lime-company/powerauth-webflow/issues/221


Copilot solution window not appearing

I signed up to access Copilot, and for about five minutes, I had access to a solutions menu ( Ctrl + Enter). I am still enjoying the autofill feature (alt + ), but I've spent the last couple of hours trying to troubleshoot this, and now I'm pretty disappointed. I don't know if anyone else has had this issue, but I would be happy to try anything if someone can shed some light on the subject.

/extCopilot

It looks like this is caused by the Copilot extension. Please file the issue in the Copilot Discussion Forum. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Falcon::Adapters::Input#gets doesn't comply with the Rack specification

The Rack specification says that the #gets method should behave like IO#gets, but currently it just retrieves the whole request body. I don't know who uses that method in the first place (I always use #read), but I thought it would be good to make Falcon fully Rack-compliant.

That makes sense.

We'd need to copy something like https://github.com/socketry/async-io/blob/c39a819204c18864f66f354f1e1034dab31e2874/lib/async/io/stream.rb#L74-L85

The specification for gets is completely insane:
gets(sep=$/ [, getline_args]) → string or nil click to toggle source
gets(limit [, getline_args]) → string or nil
gets(sep, limit [, getline_args]) → string or nil

Reads the next “line'' from the I/O stream; lines are separated by sep. A separator of nil reads the entire contents, and a zero-length separator reads the input a paragraph at a time (two successive newlines in the input separate paragraphs). The stream must be opened for reading or an IOError will be raised. The line read in will be returned and also assigned to $_. Returns nil if called at end of file. If the first argument is an integer, or optional second argument is given, the returning string would not be longer than the given value in bytes.

I started implementing this. It's a huge abomination.
Then, I wondered, what does the rack spec actually say?
https://www.rubydoc.info/github/rack/rack/master/file/SPEC#The_Input_Stream

The input stream is an IO-like object which contains the raw HTTP POST data. When applicable, its external encoding must be “ASCII-8BIT” and it must be opened in binary mode, for Ruby 1.9 compatibility. The input stream must respond to gets, each, read and rewind.

And specifically:

gets must be called without arguments and return a string, or nil on EOF.

It seems like a conforming implementation just needs to call #read, which is exactly what we are doing already.
Please discuss this if you think this interpretation is incorrect, because as I said, it would be good to get this right. But, implementing IO#gets is really nasty.


Problem getting Gamertag via Xuid

Hello, first off let me say that thank you for all the hard work I know has been put into this API it has literally given me so many possibilities.
I have built a wrapper for Laravel 5.0 that I will use to access the API. It implements GuzzleHttp Requests to get the info, sending api key, etc but I am having an issue with the /v2/gamertag/{xuid} part of your API.
When I visit the url via my browser I get my gamertag no problem. But when I get it via Guzzle and then I receive a Guzzle Stream as the body, and if I attempt $res->json() as I do with most of the other responses in order to build a PHP collection I get an error about it being malformed json even tho this same method works just fine on the /v2/xuid/{gamertag} endpoint which also only prints plain text response.
I know this isn't really your problem I guess, but I'm looking for any pointers. I can work without the ID > Gamertag function but I'd like to have it.

Both endpoints /v2/xuid/{gamertag} and /v2/gamertag/{xuid} return plain text, not JSON. This is where you will be getting issues.

I understand that the that's where '->getBody()' is supposed to be used but
I only get the Guzzle Stream object was wondering if had any idea why.
On Jan 27, 2015 8:21 AM, "Alan Wynn"<EMAIL_ADDRESS>wrote:

Both endpoints /v2/xuid/{gamertag} and /v2/gamertag/{xuid} return plain
text, not JSON. This is where you will be getting issues.
—
Reply to this email directly or view it on GitHub.

I unfortunately don't understand why you are getting the issue. If you can provide me some example code I may be able to help further.

Yeah I think I figured it out. Probably was a easier way since its just
plain text lol
On Jan 28, 2015 8:23 AM, "Alan Wynn"<EMAIL_ADDRESS>wrote:

I unfortunately don't understand why you are getting the issue. If you can
provide me some example code I may be able to help further.
—
Reply to this email directly or view it on GitHub
https://github.com/xboxapi/XboxAPI-Example-Code/issues/3#issuecomment-71841820
.


Fix inventory status

Fix display name duplication.
Add required option to uriParameters.

Thanks for the pull request and bringing this to our attention. I've made a similar change, but with slightly different displayNames. I also added required for the hostId parameter for both the inventoryStatusHost (URI parameter) and inventoryStatusHosts (query parameter) methods.
/2.0/vdn/inventory/hosts/connection/status
original: inventoryStatusHost
your suggestion: inventoryStatusHostList ->
changed to: inventoryStatusHosts (plural)
GET /2.0/vdn/inventory/hosts/connection/status
original: inventoryStatusHostList
your suggestion: inventoryStatusHostListRead
changed to: inventoryStatusHostsList (plural)
This matches our stated style guide for displayNames:
NSX RAML readme
List - Read a collection of objects. HTTP GET
Read - Read a single object. HTTP GET


Update and improve documentation for reviews

Some passed proposal have not been reflected in our review documentation:

No merge tickets anymore
No merge order on features anymore
No translation key changes to release branches anymore

With the new CI infrastructure, it also does not make too much sense to explicitly ask reviewer to run tests locally.
Also add some special considerations for ensuring that the mockup data for the Admin UI does not get out of date

Hi @gregorydlogan and @lkiesow
As you both are Opencast QAs, please indicate whether you are fine with those changes by approving the PR.

Looks good. I think a few points don't necessarily need to be listed at all anymore but this definitely reflects our process than the old version and we can always improve things later.


lanuch.json format happen to some errors

VSCode Version: newest version
OS Version: OSX newest version

Steps to Reproduce:

create new debug configation

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

What exactly is the problem?

This is a luanch.json formatted question


Allow setting core affinity for the whole process

On Windows, you can set the affinity for the whole process which is then inherited by child processes. Unfortunately, this is not the behavior for thread affinities.
Could it be possible to also add a set_for_current_process function ?

Implemented my own version @ https://crates.io/crates/affinity which fixes most of the standing issue on this crate (including this issue).
If you ever read these comments (some issues have been opened since Feb 2019), feel free to reach out to me or copy paste my crate's code into yours so we can reduce ecosystem fragmentation.