Add support for overriding default keyboard actions

The keyboard navigation implementation in react-aria is excellently designed, but it lacks customization options. This limitation makes it challenging for me to utilize the existing logic of react-aria in my keyboard-first application. Moreover, the keyboard logic is embedded deeply within the hooks, making alterations difficult.
In my pull request, I have introduced a KeyboardEventHandler that allows users to choose whether to override the default keyboard operation logic. It provides two levels of customization:

Adding only navigation triggering keys while still utilizing the default navigation logic, such as returning the built-in KeyboardAction: 'nav-down'.
Full customization of the keyboard logic, for example, by returning an empty string ''.

This second level of customization has proven to be incredibly useful in scenarios where I use react-window for virtual scrolling. In such cases, the pagination logic, among other things, deviates from the default, rendering it unusable without extensive customization.

This is a interesting use case, I had originally thought you could create a new keyboard delegate but you want to customize the keyboard actions themselves. Personally, I think this could be fine if the actions were supplementary to the original actions AND didn't heavily deviate/conflict with the keyboard actions mentioned by the APG. Could you expand on your customizations that you mentioned in the latter part of your PR description? I wasn't sure if the stories you added were illustrative of what you are trying to accomplish in your own app or not.

@LFDanLu You are correct; this PR indeed supplements the original operations without affecting the original behavior. This PR stems from my own needs as I am building a keyboard-first application, and I have a personal preference for Vim-style navigation. Therefore, I wanted to add additional navigation operations besides the arrow keys. For example, Ctrl+J would act as ArrowDown, and Ctrl+K as ArrowUp. As for the "custom operation," I now believe it should return "noop" instead of an empty string. The action "noop" is not in the original list of actions, so it indicates that the default keyboard operation should not execute. The reason for introducing "noop" is because I am using react-window for virtual scrolling, and since the ScrollToItem operation is internal logic of react-window, the original PageUp operation does not work.
Furthermore, returning "noop" can actually be used to cancel the default keyboard operation. This may conflict with APG, but I believe developers should be aware of what they are doing, so it's up to the developers to decide whether they want to do this.

@zzzze I see, thank you for the additional details. I'll have to discuss this further with the team, especially the last bit regarding allowing the default keyboard operations to be overridden which I feel will be the only part of contention.

I have a different use case but could potentially benefit from this change as well. I am currently trying to integrate the GridList component with the ariakit's composite component. Essentially, GridList elements would function as results from an input field above (similar to a combobox). The current keyboard behavior being so embedded in the hooks does not allow me to easily connect to an external virtual focus manager. I'd love to be able to define custom onKeyUp behavior that plays nice with third party libraries.

@zzzze Hey, sorry about the delay but the team talked about this today. We aren't opposed to the overall overriding and extending of the keyboard actions though we'll want to be quite careful in how we document this since it is an advanced use case. We did however want to explore a different api than what you've added here, that being adding/modifying the keyboard delegates so that they can accept a similar kind of keyboard handler override that you've done in useSelectableCollection and then refactoring useSelectableCollection so it would call that delegate keyboard handler and pass whatever key it received from it to navigateKey. There will need to be some experimentation done there since it might cause some issues in Virtualizer/other places that have also use keyboardDelegate, but this approach would ideally allow users to create their own keyboard delegates to override the keyboard actions and/or extend from the base level keyboard delegates if they only wanted to add/override a single action.
If you are interested, definitely feel free to try this out!

There's a number of conflicts now, I'm going to close this and we can continue the discussion in the Issue I've opened for this https://github.com/adobe/react-spectrum/issues/6839 and in other PRs, or we can reopen this one if you're still interested.
Thanks for starting this!


Huge requirements footprint

If I don't have Cython, scipy, or spark installed, trying to install bat will attempt to install those as well.  All I want to do is pull a brolog into a dictionary.  Can we reduce the requirements footprint such that most dependencies are optional, and attempting to use or import a feature that requires one of them causes an exception explaining the need for it?

Also, I don't believe bat should require bat.  That's recursive.  If I bumped the version on a fork, some versions of pip would try to install your version as a requirement of mine until I took that out.

@petiepooo Thanks for the feedback and bug finding. :)

Having bat in the setup.py is just a silly bug, thanks for finding that.
I agree the large set of requirements is suboptimal, in general I wanted folks to be to just install bat and then run all the stuff, but as you suggested perhaps we can minimize the libraries that get installed for the 'base' install 'pip install bat' and then push most of them to 'pip install bat[all]'.

I'll fix both of these issues right away and put up another PyPI release, thanks again.

You rock!  I've forked for now so I can get going on my setup..

@petiepooo so after some experimenting with this. I'm unsure I want to remove all the dependencies. The point of the project is to provide 'bridges' to Pandas, Scikit-Learn, Spark, etc. Without those dependencies basically nothing works and for most folks that will be a terrible 'out of box' experience.
I investigated a 'pip install bro[min]' option but there's no way to remove dependencies like that.. you can only add.
I've already removed the errant 'bat' dependency, and I've made sure that the wheel is pushed to pypi. So.. shrug.. you're probably savvy enough to just pull the few files you need and set your PYTHONPATH to those files... sorry to not get a great solution here.

I'd urge you to reconsider. People that want a bridge to scikit know to install scikit, or would the moment they get an import error trying to load the bat module that uses it. Same with spark or cython.  By making a bridge to all those advanced components, then including them in the install dependencies, you're telling users they must have everything installed even if they only need bat for one small thing. That's not what install dependencies were meant for.

The point is well taken, I'll definitely give it further consideration. Thanks for the suggestion, I'll keep this issue open. Perhaps others might chime in...

Just to add my £0.02...
I agree with @brifordwylie in this question. If I am to run Bat I sort of expect to get these dependencies installed. My understanding is that is was made for heavier data wrangling and ML from the outset.
However, the idea of a BtD  (Bro to Dictionary) utility is quite interesting for a more lightweight approach. In extremely pseudo code I could see it work something like this:

Open the log file
Parse the headers to figure out fields
Iterate through the lines to populate the dictionary

That should in theory work, but could become quite memory intensive as well but it all depends on how much data your Bro host(s) generate.

@swedishmike yeah I was thinking the same thing.. make a no dependency version that just has the log reader/tailer... and that's about it. I made this as an experiment https://pypi.python.org/pypi/bat_min... so at a minimum (see what I did there?) I'll finish this up and then if folks just want the core log reader then can just do
$ pip install bat-min

It looks like I can also use "pip install --no-deps bat", but then have to manually install the dependencies I'd actually use, like watchdog, requests, etc.  On the plus side, when I am missing a module, the ImportError exception makes it very clear what is missing.
Your monkey; your circus; your decision.  I will leave this discussion with a link to the python.org's recommendation for how to handle dependencies:
https://packaging.python.org/discussions/install-requires-vs-requirements/
This seems to indicate that the install_requires keyword should be a minimal set of packages, and the full set of modules as described by @swedishmike might be better placed in a requirements.txt file.
Thanks for writing and releasing this package!  It's quite useful.

@petiepooo sounds like you have a good solution of just running pip with the --no-deps option. As you mentioned people should be able to install BAT without all the dependencies, so we should support basically two use cases.

install no packages, people can run Bro log reader/tailer and then install additional dependencies when/if they need them.
install a bunch of packages and all the scripts/tests/example/notebooks just work

There are two approaches to supporting both of these use cases.
pip install --no-deps bat (to support 1) and  pip install bat (to support 2)
OR
We rip out all the packages in setup.py and put them into requirements.txt (as you suggest)
pip install bat (to support 1) and pip install -r bat (to support 2)
Using the --no-deps option seems good to me, it's explicit. Also if we rip out all the package most 'non expert' python folks they will simply do a 'pip install bat' and then wonder why the tests don't run, and none of the examples run, and they will be like '..this project is broken...' and quit. Where as a 'python expert' (like yourself) will say hey I don't want all the dependencies.. I'll just run the --no-deps option.
So by keeping things as they are, you (the expert) can easily figure out how to do what you want and then non-expert just does the 'pip install bat' and everything works as expected.


Coping with custom data of arbitary shape

Hello!
I am wondering if your bs prepare works when there are multiple input files in the same root dir?
Does it create several round_{num} directories per dataset?
Instead, if I want one model to ingest all the input files - what should I do differently?
Also, are there any data preprocessing stages in prepare such that every dataset has the same size/shape/normalisation?
Best,
Samia

Hello!
By input files do you mean volumes (zarr containers with at least a raw_dataset)? If yes, bs prepare first prepares the volumes (dumped to round_{num}/volumes.yaml), which you can prepare multiple of (by specifying the number of volumes when prompted) in a round. It does not create round directories for each volume. In a round, all volumes that have a labels_dataset will be used for ingestion (training), and all volumes (since all must have an raw_dataset) are target volumes for inference and post-processing.
You can also prepare volumes ahead of preparing configs with bs prepare volumes and loading the dumped yaml when preparing round configs.
Datasets can have arbitrary size/shape. Normalization is not included in prepare at the moment and would have to be checked manually / adjusted as needed in the gunpowder pipelines (in root/round_dir/setup_dir/{train,predict}.py)
prepare does have some general prompts while preparing volumes (such as asking if you want to do bounding box / create a mask) -- the bs utils commands also has clahe normalization at the moment. bs utils convert (which is called when preparing volumes if input file is .tif or a directory of .tifs) handles some basic normalization.
Let me know if that works -- I will be revamping the README, examples, and documentation in the coming days !
Thank you for your interest!
Best,
Vijay

I plan to include support for HDFlike arrays. For now, I just pushed commit 7f8ae269ecbf575c8e50a6b1adce36796e640b94 which removes the .tif conditions since tifffile.imread can read more than just .tifs (not yet tested on non-tifs) -- let me know how it works for your other dataset!
uint8 for storing image arrays is advisable (especially if the volumes can get large). uint32 or uint64 for labels. If you can view your zarrs arrays in neuroglancer, and if your zarr arrays have "offset", "voxel_size", "axis_names", and "units" as attributes -- then I would expect things to run smoothly : )

Many thanks Vijay! I shall check and report back if I find issues.


Added predict_item to RepresentablePredictor and adapted subclasses.

Addresses #238.
Also added a way to create a MeanPredictor based on an existing Predictor.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-240/1 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-240/1/index.html

Job PR-240/2 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-240/2/index.html

Codecov Report

Merging #240 into master will decrease coverage by <.01%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##           master     #240      +/-   ##
==========================================
- Coverage   81.24%   81.24%   -0.01%
==========================================
  Files         133      133
  Lines        7429     7432       +3
==========================================
+ Hits         6036     6038       +2
- Misses       1393     1394       +1

Impacted Files
Coverage Δ

src/gluonts/model/trivial/identity.py
100% <100%> (ø)
:arrow_up:

src/gluonts/model/predictor.py
82.35% <100%> (+0.18%)
:arrow_up:

src/gluonts/model/trivial/constant.py
100% <100%> (ø)
:arrow_up:

src/gluonts/model/trivial/mean.py
94.87% <84.61%> (-2.43%)
:arrow_down:


https://owncloud.org/support/ links only to core bugtracker

Hi,
https://owncloud.org/support/ currently only links to https://github.com/owncloud/core/issues.
Maybe its better to link https://doc.owncloud.org/server/8.2/developer_manual/bugtracker/index.html instead?
As an alternative it could be also linked https://github.com/owncloud/core/blob/master/CONTRIBUTING.md#submitting-issues

PR in https://github.com/owncloud/owncloud.org/pull/860


Update svm to 22.0.0

Updates org.graalvm.nativeimage:svm from 21.3.1 to 22.0.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.graalvm.nativeimage", artifactId = "svm" } ]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Codecov Report

Merging #265 (e99d9d5) into main (12a7c70) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #265   +/-   ##
=======================================
  Coverage   88.76%   88.76%
=======================================
  Files          91       91
  Lines        2776     2776
  Branches       68       68
=======================================
  Hits         2464     2464
  Misses        312      312

Continue to review full report at Codecov.

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


Remove unused Node.js packages

The goal is to keep the package.json file lean and speed up yarn install run by git hooks, e.g. when checking out a different branch.

Before removing unused packages
Running yarn install --frozen-lockfile remains fast when no packages have changed:
$ hyperfine --runs 10 'yarn install --frozen-lockfile'
Benchmark 1: yarn install --frozen-lockfile
  Time (mean ± σ):      1.424 s ±  0.031 s    [User: 1.840 s, System: 0.182 s]
  Range (min … max):    1.379 s …  1.472 s    10 runs

The folder node_modules is removed and all packages are reinstalled by the post-merge hook. This operation takes more time:

Note
I still don't know why the author of the post I found is removing node_modules in post-merge hooks. I have had issue recently when trying to skip this step. I need to think more about this as avoid removing node_modules  would say us time.

After removing packages
TODO

Babel is using 400M but it's via peer dependencies.
vscode@c52922a7bd17:/workspaces/challenge-registry/node_modules$ du -h --max-depth=1 --threshold=50M
409M    ./@babel
98M     ./@angular-devkit
110M    ./@angular
137M    ./@nrwl
65M     ./typescript
166M    ./canvas
61M     ./@nxrocks
1.9G    .


How to prevent automatic sleep?

Describe the bug
After upgrading to version 1.7, the orb automatically enters sleep mode. This causes my installed debian10 system to stop working, and other machines are also unable to connect to my debian10 system installed in the orb via SSH.
To Reproduce

Install the debian10 machine
Install the ssh-server on the debian10 system
Wait for the orb to enter sleep mode

Attempt to remotely connect to the debian10 system via SSH fails
After waking up the debian10 system, attempt to remotely connect via SSH still fails
Trying to restart the ssh-server with sudo systemctl restart ssh gets stuck

Expected behavior
No response
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.7.0
Commit: c5439c5612cdb5febbbc69169a4f3b11cdada6c2 (v1.7.0)
System info:
macOS: 14.6.1 (23G93)
CPU: amd64, 16 cores
CPU model: Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz
Model: iMac19,1
Memory: 64 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-08-26T10-51-05.417265Z.zip

Screenshots and additional context (optional)
No response

There's no sleep mode. Please run killall -USR1 'OrbStack Helper' while it's stuck and share a new diagnostic report.

There's no sleep mode. Please run killall -USR1 'OrbStack Helper' while it's stuck and share a new diagnostic report.

Since the restart, the issue has not occurred again. I will provide further feedback here if the problem recurs.

Should be fixed in the next version. If it happens again, please add a comment and we'll reopen this.

Released in v1.7.1.


Code Does not transpiles

TypeError: Cannot read property '244' of undefined
at getDeclarationSpaces (/usr/local/lib/node_modules/typescript/lib/tsc.js:35312:52)
at /usr/local/lib/node_modules/typescript/lib/tsc.js:35304:84
at Object.forEach (/usr/local/lib/node_modules/typescript/lib/tsc.js:260:30)
at getDeclarationSpaces (/usr/local/lib/node_modules/typescript/lib/tsc.js:35304:28)
at checkExportsOnMergedDeclarations (/usr/local/lib/node_modules/typescript/lib/tsc.js:35256:41)
at checkInterfaceDeclaration (/usr/local/lib/node_modules/typescript/lib/tsc.js:37101:17)
at checkSourceElement (/usr/local/lib/node_modules/typescript/lib/tsc.js:37865:28)
at Object.forEach (/usr/local/lib/node_modules/typescript/lib/tsc.js:260:30)
at checkSourceFileWorker (/usr/local/lib/node_modules/typescript/lib/tsc.js:37975:20)
at checkSourceFile (/usr/local/lib/node_modules/typescript/lib/tsc.js:37959:13)

TypeScript Version:  2.7.0-dev.201xxxxx

Search Terms:
Cannot read property '244' of undefined
Code
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.

Expected behavior:
Actual behavior:
Cannot read property '244' of undefined. The Code Does not transpiles.
Playground Link:
Related Issues:

Compiles fine here on Windows with version 2.7.2. May I ask why you're still using a dev version of a 2.7.x, when 2.7.2 was released over 2 months ago already?

Hey hi,
I am new to typescript and trying to learn some new skills.
Was not aware of the new release I do update vs code on a regular basis.
I did update to 2.9.0-dev.
It turns out I had a few botched up imports fixing the same solved the
issue.
On Mon 16 Apr, 2018, 8:53 PM Martin Johns<EMAIL_ADDRESS>wrote:

Compiles fine here on Windows with version 2.7.2. May I ask why you're
still using a dev version of a 2.7.x, when 2.7.2 was released over 2 months
ago already?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TypeScript/issues/23426#issuecomment-381643414,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVMSFVOH9IQ1QomHDMmp-EIWD4FhnXnWks5tpLdlgaJpZM4TWtTW
.

Duplicate #21928 and others
Pro tip - search for the function name at the top of the call stack as this is more likely to find related issues

Sure, thanks guys appreciate it.
On Tue 17 Apr, 2018, 12:38 AM Ryan Cavanaugh<EMAIL_ADDRESS>wrote:

Closed #23426 https://github.com/Microsoft/TypeScript/issues/23426.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TypeScript/issues/23426#event-1577069166,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVMSFTclqsvByQjae2M4-kSrqYs8T-szks5tpOw2gaJpZM4TWtTW
.


custom port on service  ingress gateway is deleted after some time

Hello,
I have to expose Redis to outside with istio gateway, and I have added the custom port for redis on ingress gateway service manually as below.
kubectl edit svc istio-ingressgateway -n istio-system
- name: redis-port
  nodePort: 31402
  port: 11337
  protocol: TCP
  targetPort: 6379

It works well, the redis is accessible from the outside throug ingressgateway IP
.\redis-cli.exe -h ip -p 11337
However, after some time (maybe a few days), the custom port is deleted automatically on the service ingress, and redis is not accessible from outside.
So, why the custom port (the block with 5 lines) is deleted automatically pls?
Thanks

This does not seem to be an istio issue. Better check your k8s audit log and see who touches the resource.

This does not seem to be an istio issue. Better check your k8s audit log and see who touches the resource.

It is changed automatically serveral times, it is istio 1.7.3 in AKS 1.20.5.
Is it related to the nodePort?  I use the nodePort by chance, i choose this numero. We may be need to fix the nodePort of this custom port or verify the numero before using it?

name: redis-port
nodePort: 31402
port: 11337
protocol: TCP
targetPort: 6379

Thanks

Resource can be updated without recreating so age will be the same. Generation will be different though. Audit log should be provided by your k8s provider: https://kubernetes.io/docs/tasks/debug-application-cluster/audit/.

Resource can be updated without recreating so age will be the same. Generation will be different though. Audit log should be provided by your k8s provider: https://kubernetes.io/docs/tasks/debug-application-cluster/audit/.

thanks for you reply. I think the pb is that the svc ingressgateway is pending
I have installed istio 1.7.3 in AKS 1.20.5.
And I have deleted and recreated istio on AKS, this time the svc is always on pending status :
istio-ingressgateway        LoadBalancer   <IP_ADDRESS>   <pending>     15021:31101/TCP,80:30322/TCP,443:30411/TCP,15443:32016/TCP   9m53s
describe svc ingressgateway
Warning  SyncLoadBalancerFailed  22s (x8 over 10m)  service-controller  Error syncing load balancer: failed to ensure load balancer: instance not found
describe pod
Warning  Unhealthy  11m (x8 over 11m)  kubelet, aks-pool3-13468513-vmss000000  Readiness probe failed: Get "http://<IP_ADDRESS>:15021/healthz/ready": dial tcp <IP_ADDRESS>:15021: connect: connection refused
why it does not work any more?

We had the same error in our rec environment today, the istio gateway does not work after some days.
I think it my be related to the autoscaling of aks nodes. Here is the error:
kubectl.exe describe svc istio-ingressgateway -n istio-system
Warning UpdateLoadBalancerFailed 72m (x3 over 73m) service-controller Error updating load balancer with new hosts map[aks-nodepoolspot-37892142-vmss00002a:{} aks-pool1-37892142-vmss000000:{} aks-pool2-37892142-vmss000000:{} aks-pool2-37892142-vmss00001v:{} aks-pool2-37892142-vmss00001w:{} aks-pool3-37892142-vmss000000:{}]: instance not found
Normal UpdatedLoadBalancer 71m (x3 over 94m) service-controller Updated load balancer with new hosts
So we can't create custom port in service istio-ingressgateway? Because it does not work with nodepool autoscaling?
Where do you think the error is pls


React Native app shows 'index.android.bundle' error when setting targetSdkVersion to 28

🐛 Bug Report
After I updated my targetSdkVersion from 27 to 28, I ran react-native run-android connected to an emulator and was presented with a screen stating "Unable to load script from assets 'index.android.bundle'. Make sure your bundle packaged correctly or you're running a package server."
To Reproduce
From a new or existing (functioning) React Native app, change the targetSdkVersion in android/build.gradle file to 28.
Expected Behavior
The app loads as it did before.
Environment
  React Native Environment Info:
    System:
      OS: Linux 4.4 Ubuntu 16.04.5 LTS (Xenial Xerus)
      CPU: (8) x64 Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz
      Memory: 17.71 GB / 31.87 GB
      Shell: 4.3.48 - /bin/bash
    Binaries:
      Node: 8.15.0 - ~/.nvm/versions/node/v8.15.0/bin/node
      Yarn: 1.12.3 - /usr/bin/yarn
      npm: 6.4.1 - ~/.nvm/versions/node/v8.15.0/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      Android SDK:
        API Levels: 23, 24, 25, 26, 28
        Build Tools: 25.0.0, 28.0.3
        System Images: android-28 | Intel x86 Atom_64
    npmPackages:
      react: 16.6.3 => 16.6.3
      react-native: 0.58.3 => 0.58.3
    npmGlobalPackages:
      react-native-cli: 2.0.1

+1

I think that It could be related to this issue: https://github.com/facebook/react-native/issues/23349#event-2130706843

Same problem after upgrading from RN 57.7 to 58.5

i had a same problme
and fixed by this link

I ran into the same type of error with the bundle not being found. My issue was fixed by changing the gradle version I was using. I made a comment about this here https://github.com/facebook/react-native/issues/23739#issuecomment-469417259. Hopefully, the solution I used will work for someone else 😄

我升级28以后，在Android9.0系统上 ，text多行显示时，最后一行显示一半，好像整个text不计算行间距似的，这是怎么回事

I ran into the same problem with a project I upgraded to 0.59, so I thought the upgrading went wrong.
After creating a new project with react-native init I ran into the same problem when trying to run it on android simulator or a physical device.
I followed the instructions mentioned here https://stackoverflow.com/questions/44446523/unable-to-load-script-from-assets-index-android-bundle-on-windows/44476757#44476757 (as @mr-niri posted) and this resolved my issue.
Output of react-native info
info
  React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
      Memory: 276.07 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 8.11.3 - ~/.nvm/versions/node/v8.11.3/bin/node
      Yarn: 1.12.3 - /usr/local/bin/yarn
      npm: 5.6.0 - ~/.nvm/versions/node/v8.11.3/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 27, 28
        Build Tools: 27.0.3, 28.0.3
        System Images: android-28 | Google Play Intel x86 Atom
    IDEs:
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.1 => 0.59.1

Duplicate of #22024.

Starting with Android 9.0 (API level 28), cleartext support is disabled by default.

AndroidManifest.xml file change as following
<?xml version="1.0" encoding="utf-8"?>
<manifest ...>
    <uses-permission android:name="android.permission.INTERNET" />
    <application
        ...
        android:usesCleartextTraffic="true"
        ...>
        ...
    </application>
</manifest>


#269

WRT escaping quotes that appear in HTML
Implements solution suggested in https://github.com/chjj/marked/issues/269, also breaks tests.

@jguffey: Thanks for the PR!
 I'm not familiar with the library enough to understand what would be the optimum solution, the test will help me, and others, understand what's going on.
Thanks!


Views are rendered twice on route transition

My team has recently faced an issue with controllers being instantiated twice on route transition. While trying to find the reason for this, I've recognized that even the corresponding views got rendered twice, which was rather... unlikely.
This problem was detected in v0.2.7 - https://github.com/angular-ui/ui-router/commit/20fcbf8384090a092c7c91a26443e3c818438120 thus I considered moving to v0.2.8 - https://github.com/angular-ui/ui-router/commit/f9ce4dddbe579d215ba84ee9e9ec29249b23ec05 should solve this. But it did not.
Information
Angular-version: 1.2.11
The relevant routes look like this (I cannot post everything, since it's part of a closed project):
$stateProvider
        .state('1ColumnLayout', {
            'abstract': true,
            url: "/1",
            templateUrl: "1ColumnLayout.html"
        })
        .state('login', {
          parent: "1ColumnLayout",
          url: "/login",
          views: {
            "screen-above": {template: ""},
            "screen": { templateUrl: "login.html", controller: "loginController"}
          }
        })
       .state('plantDetails', {
           parent: "2ColumnLayout",
           url: "/plantDetails?plantId",
           views: {
               "screen-above": { templateUrl: "userInfoAndNavigation.html"},
               "screen-left": { templateUrl: "details.left.html", controller: "detailsLeftController"},
                    "screen-right": { templateUrl: "details.right.html", controller: "detailsRightController"}
                },
                resolve: {
                    currentId: getId(false)
                }
            })

While using this template in 1ColumnLayout.html...

...and this one in 2ColumnLayout.html

In a more-or-less long debugging session, I was able to drag down the issue to the following line in viewDirective.js (in v0.2.7):
initialView = transclude(scope);

The call stack mentioned that by calling transclude(...), the compile function is called again, building up the entire view, including controller instantiation. Afterwards, the function went ahead as expected, but destroyed the previously built view completely, building it up once more.
In v0.2.8, this line of code has changed to
anchor    = angular.element($document[0].createComment(' ui-view-anchor ')),

which seems more intuitive. However, moving to v0.2.8 did not fix the issue - seems to be broken on another part.
The curious thing...
... is primarily related to the sometimes in the topic: After performing all potential route transitions at least once, all views got rendered only once - as expected.
Workaround
Anyway, just for testing, I've changed
initialView = transclude(scope);

in v0.2.7 to
 initialView = angular.element($document[0].createComment(' ui-view-anchor '));

(picked from v0.2.8), and injected $document as well. Result: Everything worked as expected (I've added my diff so that the whole change is visible), which is... interesting.
diff --git a/public/lib/angular-ui-router/src/viewDirective.js b/public/lib/angular-ui-router/src/viewDirective.js
index efc8930..5e0a9ed 100644
--- a/public/lib/angular-ui-router/src/viewDirective.js
+++ b/public/lib/angular-ui-router/src/viewDirective.js
@@ -1,6 +1,6 @@

-$ViewDirective.$inject = ['$state', '$compile', '$controller', '$injector', '$anchorScroll'];
-function $ViewDirective(   $state,   $compile,   $controller,   $injector,   $anchorScroll) {
+$ViewDirective.$inject = ['$state', '$compile', '$controller', '$injector', '$anchorScroll', '$document'];
+function $ViewDirective($state, $compile, $controller, $injector, $anchorScroll, $document) {
   var $animator = $injector.has('$animator') ? $injector.get('$animator') : false;
   var viewIsUpdating = false;

@@ -15,7 +15,8 @@ function $ViewDirective(   $state,   $compile,   $controller,   $injector,   $an
             name = attr[directive.name] || attr.name || '',
             onloadExp = attr.onload || '',
             animate = $animator && $animator(scope, attr),
-            initialView = transclude(scope);
+            initialView = angular.element($document[0].createComment(' ui-view-anchor '));
+

         // Returns a set of DOM manipulation functions based on whether animation
         // should be performed

Even worse, I was not able to reproduce this issue using v0.2.8 with a more simple example, and I'm not sure why it still occurs - wasn't that easy to drag down to a particular line as in v0.2.7 ;)
Maybe some of you have a clue what may cause this, or may check this in a more complex project - rendering views twice, including controller instantiation, is rather unlikely when it comes to performance, or possible side-effects that come along with init-functions.
For the moment, this appears to be a blocking issue in our case which prevents us from moving to v0.2.8 and, as a result, from other fixes / improvements.

I am using 0.2.13 and this is still happening.

@ikhos What version of Angular?

@nateabele I'm using 1.3.15. Since I am using multiple modules I am using angular.bootstrap().
'use strict';

angular.module(ApplicationConfiguration.applicationModuleName, ApplicationConfiguration.applicationModuleVendorDependencies);

angular.module(ApplicationConfiguration.applicationModuleName).config(['$locationProvider',
	function($locationProvider) {
		$locationProvider.hashPrefix('!');
	}
]);

angular.element(document).ready(function() {
	if (window.location.hash === '#_=_') window.location.hash = '#!';

	angular.bootstrap(document, [ApplicationConfiguration.applicationModuleName]);
});

This is the bootstraping config:
'use strict';

var ApplicationConfiguration = (function() {
	var applicationModuleName = 'myApp';
  var applicationModuleVendorDependencies = [
    'ui.router',
    'ui.bootstrap',
    'ngAnimate',
    'ngSanitize',
    'ngTouch',
    'ajoslin.promise-tracker',
    'angular-google-analytics',
    'angular-inview',
    'LocalStorageModule',
  ];

	var registerModule = function(moduleName, dependencies) {
		angular.module(moduleName, dependencies || []);

		angular.module(applicationModuleName).requires.push(moduleName);
	};

	return {
		applicationModuleName: applicationModuleName,
		applicationModuleVendorDependencies: applicationModuleVendorDependencies,
		registerModule: registerModule
	};
})();

I just found out that if I comment the angular.bootstrap(...) line, the ui-view renders just once. Any hints on this behaviour will be greatly appreciated.

I feel really dumb now. I was using ng-app="" as well as initialising the app programmatically. I just removed the ng-app directive from my view and everything works perfectly now. Thanks.

:+1: You might feel dumb for having wasted time on it, but those are the best kinds of problems to have. Seriously.

I almost missed @edrpls 's answer, just because @nateabele 's bad joke. :smirk:
thank you all.
Maybe silly, but remind again. hey, @edrpls 's answer helps.

This occurs for me with the latest ui-router. I need to use ng-app="app" and angular.bootstrap because I have window.name = "NG_DEFER_BOOTSTRAP!";  .The app is only instantiated once, so why does this occur for me still?

Hi there,
I'm using Angular 1.3.15 and latest UI-Router (0.2.18) and my root UI-VIEW is always duplicating.
I'm bootstrapping the application by myself and not like @edrpls, I'm not using ng-app :)
How can I fix that? Where are the suspicious places that might make this issue?

Hi there,
I'm facing the same issue. Using Angular 1.4.9 and angular-ui-router 0.2.18.
The application is bootstrap using only the directive ng-app.
On load, no problem, the ui-view directive behave as expected and routes work well. But at some point, I use
$state.go('defaultRoute', {}, {reload: true});
This line reload the page as expected, but during the refresh the ui-view is duplicated for a moment.
This duplication prevent one of my controllers to initialise scroll event on window as it should.
Do someone have any clue of what causes this ?

Hi there,
the problem still exists with angular 1.5.3 and angular-ui-router 0.2.18. The directives on my view are initialized a second time when I route to the next view.
Is there a bug fix available to avoid this problem??

Why is this Closed?
This happens to me as well when using Chartist.js (charting library).
Once I add a Pie chart, the root <ui-view> is duplicated, the second one is empty.
If I navigate to this state from another state, I can see my previous state at the bottom <ui-view> and my current state at the top one.
I'm bootstraping with angular.bootstrap, no ng-app anywhere.
This hack fixes it:
$rootScope.$on('$stateChangeStart', () => {
  $animate.enabled(false);
});

But I'd really like it to be resolved, especially if I'd want to add state change animations later on.
Versions:
"angular": "1.5.0",
"angular-animate": "1.5.0",
"angular-ui-router": "0.2.18",

This is still a problem for me too. I'm using ng-animate along with ui-router to animate transitions. I started noticing that my view content was appearing twice on the screen, the bottom of the two look like styled empty templates while the one above is rendered properly. I'm using ng-app, not bootsrapping the app in js.
These are my versions:
"angular": "1.5.5",
"angular-animate": "1.5.5",
"angular-ui-router": "0.2.18",

For me the solution was to completely remove angular-animate. And it is extremely bad solution I know.

It's also an issue for me. Is there any acceptable workaround yet?

+1

Me too have this problem using the following versions:
"angular": "^1.4.0",   "angular-animate": "^1.4.0",
To solve this issue I disable animation for the view that has effects.
The code for this workaround is the following:

and in CSS:
.no-animate {    -webkit-transition: none !important;    transition: none !important; }

+1
I removed instances of reload:true to get around this on some pages, but I still have other pages that are rendering multiple views for a split second during state transitions.  The rootscope listener that @alexilyaev posted gets around the issue, but I also would like a better solution.  I was hoping I could get around it with absolute view naming, but that doesn't seem to help.


Trendelenburg Gait

Hello @Rahmyyy,
I am a master student at the University of Applied Sciences Upper Austria (Campus Hagenberg). Currently I am working on my masters thesis, where I want to perform gait analysis using computer vision. During my research i stumbeled over your repository, which is great! However, I would need a collection of videos of trendelenburg gait for my research. Do you know any dataset containing trendelenburg gait videos?
Kind Regards,
Georg

Hi Georg, unfortunately it is challenging getting a hold of patient videos with trendelenburg gait patterns, hence why GAVD was created. There are a few videos linked to GAVD with the sublabel "Pathological - Myopathic" gait that have the trendelenburg gait pattern.  Your project sounds very interesting, I hope this data can help you out.


Updates to TEX to handle new parameter and security requirements

Retrieve userId from SecurityContextHolder, pass userId to getMetadataCollectionId and set repositoryName on repositoryConnector
Signed-off-by: Graham Wallis<EMAIL_ADDRESS>

Ths PR has fixed TEX in k8s & compose. Thanks!


Request to migrate k8s client to dynamic client in operator

Request to migrate k8s client to dynamic client in chaos-operator
Motivation: To be generic for both openshift and k8s resource.
Currently, deploymentconfig is using dynamic client.

We've begun implementing dynamic clients wherever necessary.


New dataset

switching to the transfusion dataset made a few features for years and quarters for the dataset

@johnkarlen Why closed?


Use console.log instead of deprecated sys.puts

This prevents unnecessary warning coming from the deprecation of sys/util puts (See nodejs.org/api/util.html#util_util_puts)

I should be including this in the [https://github.com/driftyco/ionic-cli/blob/eccba8557ce3a3b9d873e39f2101cf9064464dba/lib/hooks/after_platform_add/010_install_plugins.js] latest hook-fixes branch.


Perform login with Slack user, create user if needed

Using the authenticated Slack user, get the Slack user
email address and check the Cloudant DB for that user.
If user exists, log them in and get their Customer info.
If user does not exist, create one.
Context will be set using this Customer info.
Code is removed to handle login and adding a user.

Needs some work...How to you WIP this stuff?


Remove unsafe code in gateway

Remove unsafe code in the gateway crate. Its use of unsafe was to accommodate simd-json, but can be removed by instead continuously working with the mutable buffer rather than going from bytes -> str -> bytes again.
This also adds #![deny(unsafe_code)] to all crates that weren't already marked with it.

Shouldn't this be #![forbid(unsafe_code)]?

Shouldn't this be #![forbid(unsafe_code)]?

It doesn't really matter either way. The difference is that with deny you can allow usage on particular items, while forbid won't allow you to allow usage anywhere. Since we have a PR process, if there's ever unsafe code that's submitted then we can review whether it's necessary or not. For consistency and simplicity I think it's easier to just combine it into the denial list.


Minor tweaks to job descriptions

About the PR
 A lot of jobs had outdated or misleading information, like doctors talking about viruses and cloning. Now they shouldn't be
Why / Balance
 Outdated and wrong
Technical details
None
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

no cl no fun

I have literally no idea why the test failed, it is literally just description changes. Maybe because Brigmeds gone?

test fail erroneous


TitleElement.TitleWidth why not default to "auto"?

The default value is not suitable for every case, and it is troublesome to change every time.

good idea


Accessing settings as strings via #[]

Describe the bug
Before the update to 0.11, a setting could be referenced via a string through config.[]. Now it must be a symbol, otherwise the setting is not found.
I'd like to know if it's desired behavior. If not, I can send a patch myself.
To Reproduce
class A
  extend Dry::Configurable

  setting :foo, :bar
end

A.config[:foo]
# => :bar
A.config["foo"]
# => ArgumentError (+foo+ is not a setting name)

Expected behavior
A.config["foo"]
# => :bar

Your environment
Affects two gems I maintain:
https://github.com/waiting-for-dev/warden-jwt_auth/issues/21
https://github.com/waiting-for-dev/devise-jwt/issues/159

I can send a patch myself.

Please do. My intention was to maintain public API behavior.


Clickhouse Bad Address

Summary
On a fresh deploy of Plausible, the main docker image side throws an error nc: bad address 'clickhouse.railway.internal You will be able to log in to the dashboard and it will receive data about visitors, but the main page will get stuck.

Recreation
Deploy a brand new Plausible
Logs
Clickhouse
/entrypoint.sh: create new user 'default' instead 'default'

Processing configuration file '/etc/clickhouse-server/config.xml'.

Merging configuration file '/etc/clickhouse-server/config.d/docker_related_config.xml'.

Merging configuration file '/etc/clickhouse-server/config.d/main-config.xml'.

Logging trace to /var/log/clickhouse-server/clickhouse-server.log

Logging errors to /var/log/clickhouse-server/clickhouse-server.err.log

/entrypoint.sh: create database 'plausible_events_db'

Processing configuration file '/etc/clickhouse-server/config.xml'.

Merging configuration file '/etc/clickhouse-server/config.d/docker_related_config.xml'.

Merging configuration file '/etc/clickhouse-server/config.d/main-config.xml'.

Logging trace to /var/log/clickhouse-server/clickhouse-server.log

Logging errors to /var/log/clickhouse-server/clickhouse-server.err.log

/entrypoint.sh: create new user 'default' instead 'default'

ClickHouse Database directory appears to contain a database; Skipping initialization

Processing configuration file '/etc/clickhouse-server/config.xml'.

Merging configuration file '/etc/clickhouse-server/config.d/docker_related_config.xml'.

Merging configuration file '/etc/clickhouse-server/config.d/main-config.xml'.

Logging trace to /var/log/clickhouse-server/clickhouse-server.log

Logging errors to /var/log/clickhouse-server/clickhouse-server.err.log

Hey @ColeTownsend, thanks for creating this ticket.
This seems to be a known issue in a recent release of Plausible, they have a workaround and a fix will be coming in soon.
I'll implement the workaround now, it may just require you to make some changes to your clickhouse config in the repo forked as part of the railway template deploy. Once I have a fix in place I'll send you instructions.

Okay, @ColeTownsend  I've confirmed that the fix in #8 got it working again on a new deploy of Plausible.
All you need to do to get your instance working is either
Option 1: copy over the clickhouse config file into your forked repo
When you deployed this template, railway created a fork of this repo for you on your GitHub account called something like "Plausible". You just need to find that repo and merge the changes in this repo (the upstream) back down into your fork.
If you're not sure how to do that, you can also just replace the https://github.com/railwayapp-templates/plausible/blob/main/packages/clickhouse/ch-user-config.xml in your plausible repo with the file in this plausible repo via a new commit.
Let me know if you have any issues with that.
Option 2: Delete your current project and redeploy via the template
This will delete all your data and is definitely the less optimal option but the quickest. I would only recomend this if you don't have any data in plausible yet.
Enjoy the rest of your week!


Lesson 6    -  IndexError: list index out of range

Discussed in https://github.com/smartcontractkit/full-blockchain-solidity-course-py/discussions/687

Originally posted by mdshaik07 January  4, 2022
When i try to deploy script(deploy.py) after creating a new network locally ganache-local. I'm ending up with ::
"" PS C:\Users\TV317DB\demos\brownie_fund_me> brownie run scripts/deploy.py --network ganache-local
INFO: Could not find files for the given pattern(s).
Brownie v1.17.2 - Python development framework for Ethereum
BrownieFundMeProject is the active project.
Running 'scripts\deploy.py::main'...
File "C:\Users\TV317DB.local\pipx\venvs\eth-brownie\lib\site-packages\brownie_cli\run.py", line 50, in main
return_value, frame = run(
File "C:\Users\TV317DB.local\pipx\venvs\eth-brownie\lib\site-packages\brownie\project\scripts.py", line 103, in run
return_value = f_locals[method_name](*args, **kwargs)
File ".\scripts\deploy.py", line 23, in main
deploy_fund_me()
File ".\scripts\deploy.py", line 6, in deploy_fund_me
account = get_account()
File ".\scripts\helpful.py", line 16, in get_account
return accounts[0]
File "C:\Users\TV317DB.local\pipx\venvs\eth-brownie\lib\site-packages\brownie\network\account.py", line 115, in getitem
return self._accounts[key]
IndexError: list index out of range. ""
Any resolution ?

Duplicate of Discussion Lesson 6: IndexError @ 5:42:02 timestamp #345


Update maven-wrapper to 3.5.4 and add maven-wrapper.jar to .gitignore

The MavenWrapperDownloader downloads the JAR if it is missing locally.
Fixes WW-4980

Coverage decreased (-0.01%) to 46.431% when pulling 1b879701d7f948330d30813fa88fe0c7122e7d20 on sepe81:update-maven-to-3.5.4 into af3ee0747410ffb64448e933888ff4a15d72fe10 on apache:master.

Could you create a JIRA ticket to cover this change? We use that to automatically generate change log when preparing a release and announcement. Thanks in advance!

LGTM 👍


Issues when finetuning pretrained lora checkpoint for downstream tasks.

Hi Lucas,
I was trying to run the MHR codebase to finetune on a downstream task after loading the pretrained lora checkpoint provided here. I am not able to load the pretrained lora checkpoints into the model and I getting some shape mismatch errors. I am pretty sure I am messing up something with the config files and hyperparameters setting. Would it be possible for you to provide a command that can be used to FT lora checkpoint on any downstream task say RTE?
I was able to load the poly checkpoint and finetune that further using the provided scripts here. It would be great if you can add scripts to finetune lora, ia3, and mhr pretrained checkpoints.
Thanks,
Prateek

Hi Prateek,
Sorry for the late reply; did you manage to solve this issue ?


bad POD, C<=>>

We have many cases in doc/Language/subscripts.pod of
C<=>>

Which should probably be
C<< => >>

Otherwise the trailing > isn't code-ified. Opening a ticket so someone can review the instances and make sure my take is correct.

Yes, C<< => >> is the way. IIRC C«=>» should also work.


how to received attached images

i have used this plugin and its working fine for android and ios.but,
how to get attached images ?, is this functionality available in this plugin?

+1

FCM message can transfer a payload of up to 4KB to a client app... so, send files need some creativity of you. Maybe upload image to server and send a FCM message with ID of image...


[?] 0.9.5b8: Opening a minimised sub-window

Double click on e.g, an image element in the tree to open the properties page.
Minimise the properties window.
Double click on the image element again. Properties window remains minimised.

Proposal: When you would open a sub-window if it is not open, but instead it is open and minimised, then do a restore.

I know that you have more wx skills than I do, however in case it helps...
After a quick look at the code and wx docs, it looks like in meerk40t/meerk40t/gui /mwindow.py either immediately before or immediately after a self.Raise() you need to check self.GetClientSize() and if [0,0] then self.Iconize(false).


Fix ThemeProvider HMR

Closes #97
@dmitweb would you mind verifying that this fixes your issue? Thanks!

Thanks!


Update referrer-policy.json

Chrome: implementation bumped to v. 61 per this comment: https://bugs.chromium.org/p/chromium/issues/detail?id=627968#c15 (and confirmed with tests).

Thanks!


RPC Server Connection Issue

When the RPC flag is set to active, an IllegalArgumentException is immediately thrown:
Exception in thread "main" java.lang.IllegalArgumentException
at java.base/java.util.concurrent.ThreadPoolExecutor.(ThreadPoolExecutor.java:1332)
at java.base/java.util.concurrent.ThreadPoolExecutor.(ThreadPoolExecutor.java:1255)
at org.aion.api.server.rpc.RpcProcessor.(Unknown Source)
at org.aion.api.server.http.HttpServer.(Unknown Source)
at org.aion.Aion.main(Unknown Source)

fixed in https://github.com/aionnetwork/aion/pull/380

fixed in https://github.com/aionnetwork/aion/pull/380


Error running NER tagger example without embeddings

The README for tag_char_rnn.py indicates it can be run without word vectors by omitting the --embed option. However, when I try to run it that way I get this error:
python tag_char_rnn.py --rnntype blstm --optim sgd --wsz 30 --eta 0.01 \
    --lower 1 \
    --epochs 50 --batchsz 10 --hsz 200 \
    --train ../data/oct27.train \
    --valid ../data/oct27.dev \
    --test ../data/oct27.test \
    --cfiltsz 1 2 3 4 5 7
Lower-case word tokens
Reading CONLL sequence file corpus
Max sentence length 38
Max word length 45
Traceback (most recent call last):
  File "tag_char_rnn.py", line 100, in <module>
    ts, _ = reader.load(args.train, word2index, args.batchsz, shuffle=True)
  File "/mnt/pool/fiddle/baseline/python/baseline/reader.py", line 219, in load
    words_vocab["<UNK>"] = 1
TypeError: 'NoneType' object does not support item assignment

Looks like it's trying to use embeddings even when not available - maybe the note about running it without them should be removed?

You really shouldnt run a tagger without pre-trained word vectors.  You simply will not be able to match performance obtainable with pre-trained embeddings.  Can you point out where I said this, and I will fix it?

The comment in the docs is here

If you want to use only the convolutional filter word vectors (and no word embeddings), just remove the -embed line above.

I understand that not using vectors has no chance of matching the performance of running with vectors, I just figured I'd give it a shot to check that everything else was OK while I was downloading the Glove vectors.
Besides that this is a lovely project, thanks for making it!

Thank you!


20s delay for autorization after process restart

Current Behavior
When restarting the openvpn-auth-oauth2 service, the next authorization will take a 20s nap:
time=2024-02-07T13:19:27.701+01:00 level=INFO msg="initialize authorization via oauth2" cid=4 kid=1
time=2024-02-07T13:19:47.884+01:00 level=INFO msg="successful authorization via oauth2" cid=4 kid=1

The client side browser will get a timeout (504 from the reverse proxy in front of openvpn-auth-oauth2), while accessing oauth2/callback. Still, after 20+ seconds, the connection is established successfully.
With log.level=DEBUG, nothing is logged in between.
After the first delayed auth, all others will succeed within fractions of a second as expected.
Expected Behavior
No response
Steps To Reproduce
No response
Environment
No response
openvpn-auth-oauth2 logs
No response
openvpn server logs
No response
Anything else?
No response

Do you have ip6 enabled on the server?

ipv6 is not disabled -> yes ;-)

Do you have experience in tcpdump?
A tcpdump could identify some system level behaviors which hard to identify from remote. Maybe a service endpoint not rechable but due automatic retries in go internals it within 20 seconds.
The binary is not linked against libc and may behavior differently

Tcpdump with all known traffic filtered out gives zero packets during that 20s pause.
Apparently, the outh2 manager will pause after the userinfo endpoint was called.

Could you please enable the debug listener via --debug.pprof?
Then, if initialize authorization via oauth2 comes up, please open the url http://localhost:9001/debug/pprof/goroutine?debug=2 and save the output to a file. Please do this 3-4 times within the 20 seconds. It should give hints where the code is hanging.

In can see this one:
goroutine 45 [select]:
net/http.(*http2ClientConn).RoundTrip(0xc00035c000, 0xc000158d00)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/h2_bundle.go:8381 +0x44f
net/http.(*http2Transport).RoundTripOpt(0xc00012ef30, 0xc000158d00, {0x60?})
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/h2_bundle.go:7649 +0x1b1
net/http.(*http2Transport).RoundTrip(0xd37160?, 0xc000360200?)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/h2_bundle.go:7598 +0x15
net/http.(*Transport).roundTrip(0xd37160, 0xc000158c00)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/transport.go:602 +0x78b
net/http.(*Transport).RoundTrip(0x0?, 0xc0000d2960?)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/roundtrip.go:17 +0x13
github.com/jkroepke/openvpn-auth-oauth2/internal/utils.(*UserAgentTransport).RoundTrip(0xc0001e3370, 0xc000158c00)
	/home/runner/work/openvpn-auth-oauth2/openvpn-auth-oauth2/internal/utils/http.go:20 +0xf7
net/http.send(0xc000158b00, {0xa22760, 0xc0001e3370}, {0x1?, 0x48e314?, 0xd3e1a0?})
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/client.go:260 +0x606
net/http.(*Client).send(0xc0001a3950, 0xc000158b00, {0x3f?, 0x0?, 0xd3e1a0?})
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/client.go:181 +0x98
net/http.(*Client).do(0xc0001a3950, 0xc000158b00)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/client.go:724 +0x912
net/http.(*Client).Do(...)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/client.go:590
golang.org/x/oauth2/internal.doTokenRoundTrip({0xa25aa0, 0xc0000d2900}, 0xc000158a00)
	/home/runner/go/pkg/mod/golang.org/x/oauth2@v0.16.0/internal/token.go:256 +0x112
golang.org/x/oauth2/internal.RetrieveToken({0xa25aa0, 0xc0000d2900}, {0xc000156600, 0x40}, {0xc000156780, 0x40}, {0xc0002c8000, 0x2e}, 0x957f2f?, 0x0, ...)
	/home/runner/go/pkg/mod/golang.org/x/oauth2@v0.16.0/internal/token.go:226 +0x112
golang.org/x/oauth2.retrieveToken({0xa25aa0, 0xc0000d2900}, 0xc00012ed80, 0xc?)
	/home/runner/go/pkg/mod/golang.org/x/oauth2@v0.16.0/token.go:167 +0x85
golang.org/x/oauth2.(*Config).Exchange(0xc00012ed80, {0xa25aa0, 0xc0000d2900}, {0xc000336775, 0x80}, {0xc0000c6260, 0x1, 0x0?})
	/home/runner/go/pkg/mod/golang.org/x/oauth2@v0.16.0/oauth2.go:234 +0x26f
github.com/zitadel/oidc/v3/pkg/client/rp.CodeExchange[...]({0xa25aa0, 0xc0000d2360?}, {0xc000336775?, 0x80}, {0xa28c40, 0xc0001d6b40}, {0xc000042450, 0x1, 0x1})
	/home/runner/go/pkg/mod/github.com/zitadel/oidc/v3@v3.10.2/pkg/client/rp/relying_party.go:434 +0x254
github.com/zitadel/oidc/v3/pkg/client/rp.CodeExchangeHandler[...].func1(0xc000158600)
	/home/runner/go/pkg/mod/github.com/zitadel/oidc/v3@v3.10.2/pkg/client/rp/relying_party.go:503 +0x6f7
net/http.HandlerFunc.ServeHTTP(0xa216f0?, {0xa24fc0?, 0xc000304000?}, 0xc0001d6b40?)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/server.go:2136 +0x29
github.com/jkroepke/openvpn-auth-oauth2/internal/oauth2.(*Provider).Handler.(*Provider).oauth2Callback.func3({0xa24fc0, 0xc000304000}, 0xc000158000)
	/home/runner/work/openvpn-auth-oauth2/openvpn-auth-oauth2/internal/oauth2/handler.go:210 +0x5c6
net/http.HandlerFunc.ServeHTTP(0x4459c0?, {0xa24fc0?, 0xc000304000?}, 0x6e8c3a?)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/server.go:2136 +0x29
net/http.(*ServeMux).ServeHTTP(0xd6d8e0?, {0xa24fc0, 0xc000304000}, 0xc000158000)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/server.go:2514 +0x142
net/http.serverHandler.ServeHTTP({0xc0002c41e0?}, {0xa24fc0?, 0xc000304000?}, 0x6?)
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/server.go:2938 +0x8e
net/http.(*conn).serve(0xc00009e480, {0xa25aa0, 0xc0000d2480})
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/server.go:2009 +0x5f4
created by net/http.(*Server).Serve in goroutine 33
	/opt/hostedtoolcache/go/1.21.5/x64/src/net/http/server.go:3086 +0x5cb

It seems like openvpn-auth-oauth2 hangs at CodeExchangeHandler which current do a HTTP request against the token endpoint.
Sometimes, it could be an unidentified http2 issue. As I know, you could test the environment variable GODEBUG=h2client=0 which should disable http2 and use http1.1 for connection.

if you run curl -XPOST https://idp/index.php/apps/oidc/token, without adding any creds, does it work without any issues?

I get an immediate 400 {"error":"invalid_grant","error_description":"Invalid grant_type provided. Must be authorization_code or refresh_token."} as expected.

I get an immediate 400 {"error":"invalid_grant","error_description":"Invalid grant_type provided. Must be authorization_code or refresh_token."} as expected.

Still a different from the 401 answer.
Which IDP your are using?

It's Nextcloud. Since I curl-posted neither auth-code nor rfsh-token, 400 appears precise to me.
Nextcloud will log "Login failed: ". If this would happen all the time, I'd point towards Nextcloud, but it only happens for the first auth attempt after openvpn-auth-oauth2 was restarted. All following auths are performed fast and flawlessly.

Could you please re-do this

Could you please enable the debug listener via --debug.pprof?
Then, if initialize authorization via oauth2 comes up, please open the url http://localhost:9001/debug/pprof/goroutine?debug=2 and save the output to a file. Please do this 3-4 times within the 20 seconds. It should give hints where the code is hanging.

with GODEBUG=http2client=0 in hope, that the http function for http11 is more readable? http2 is to complex to debug

In the meantime, I had the machine slightly reconfigured, config file cleanup, other VLAN. The error didn't occur any more... I moved the machine back to the initial VLAN, no 20s delay. Moving back to the final VLAN again (only VLAN/IPaddr changed), and there it was again, the 20s delay after service restart....
So here's the logs: log.0 before connect, log.x after success.
pprof2.tgz

Based on the dump, I still can see that go is waiting for an action. Until a timeout of 20 seconds hits.
https://github.com/jkroepke/openvpn-auth-oauth2/blob/acee037fb615611b345d6a3205a8c963f424a73c/internal/oauth2/provider.go#L152
The 20 seconds are configured in openvpn-auth-oauth2 to prevent any deadlocks.
Anyways I have no idea how I could debug this from remote. Its somewhere deeper in the go language or in the network topology, if it could be fixed an VLAN change.

Since the machine is virtualized, it doesn't know about the VLAN change, just the IP addr change... I guess it was just the reboot that re-enabled the problem. Maybe some RNG stuff?

Here is my current guess for the issue:
openvpn-auth-oauth2 uses and library for oauth2 which uses the go standard lib.
While OIDC exists as spec, many providers my not implement all variants.
Since we have the token endpoint in scope, a token can be received in 2 different "auth styles", if client ID and client secret is in use: client_secret_basic and client_secret_post
Ref: https://cloudentity.com/developers/basics/oauth-client-authentication/client-secret-authentication/
A full generic OAUTH2 doesn't know, whats supported and there is no spec to advertise the auth method. Based that, clients may require multiple try to detect what is working.
https://github.com/golang/oauth2/blob/ebe81ad83719fe3426335b22e40a1e3a76fa45c0/internal/token.go#L212-L253
If nothing is known, the client tries to use client_secret_basic first where the credentials are put in the Authorization header. It seems like Nextcloud may have an issue here or the Authorization header my not pass through nginx.
Then client_secret_post is used, where the credentials are put inside a POST body. Since this worked, the oauth2 client cache the information which method has been used unless the service gets an restart.
Thats the reason why 2 requests are visible for first time.
To mitigate the issue from  openvpn-auth-oauth2 side, I have to make the auth style parameter configurable, but the underlaying library currently does not allow this.

@andreas-p Are you able to the the binary here, if the behavior still present?
https://github.com/jkroepke/openvpn-auth-oauth2/actions/runs/8024800554/artifacts/1270912217
It should work, w/o changes any options.


Performance of GPU version

I cannot reproduce the performance claims of the software.
I ran some performance test on a single Nvidia Pascal Titan X GPU and the CPU version on a single core of a Intel(R) Core(TM) i7-6850K CPU @ 3.60GHz CPU. I first ran the following two tests:
# Classification CPU
time ./mdtwObj -t CLASSIFICATION -i CPU 3 1 \
    -f data/classification/rm_1/X_MAT data/classification/rm_1/Y_MAT data/classification/rm_1/Z_MAT \
    -k 10 -o 1000 152 -m 0 DTW 2>&1 | tee classification_CPU.txt

# Classification GPU
time ./mdtwObj -t CLASSIFICATION -i GPU 3 512 1 \
    -f data/classification/rm_1/X_MAT data/classification/rm_1/Y_MAT data/classification/rm_1/Z_MAT \
    -k 10 -o 1000 152 -m 0 DTW -d 0 2>&1 | tee classification_GPU.txt

Results are here: classification_CPU.txt, classification_GPU.txt
The GPU version is about a factor of 27 times faster then the single threaded CPU version which seems more realistic than the quoted three orders of magnitude. Also this is only a single core (this particular CPU has 6 cores/12 threads) so the code would have to be parallelized for a fair comparison.
Simply compiling the code with -O3 makes the CPU version only about 6 times slower than the GPU version (results are here: ). The optimisation flags don’t make the GPU code run any faster. Given that this could be made even 6 times faster with parallelisation the claim that the GPU version is faster than the CPU version does not hold even without comparing to other implementations.
classification_CPU_O3.txt, classification_GPU_03.txt
Part of a review at: https://github.com/openjournals/joss-reviews/issues/1049

I understand your concerns about the performance and I'll try to give you the best explanation for that issue. The reason why the GPU version is just roughly 6 time faster than the CPU is mainly due to the dimension of the dataset. In fact, the software is intended to work on bigger batches of data rather than just those used here as example tests. In addition, to be honest, I didn't take into account the case of comparing my software neither with a parallel version software or an optimized compiled version.
Although this comment might not be the best answer to your question, I've also found here the following issue with compiling a CUDA programma with the -03 option  but i don't know whether this is the case or not.
Sadly, I cannot make my GPU version faster for this task since this version for the Multi Dimensional Dynamic Time Warping is the best i could write. Anyhow, whoever will use the software will always benefit from it especially when the size of the dataset to handle is made of more than one thousand of time series.

I'll try the other performance flags for the CUDA compilation once you deem the code ready for another review round! Can you provide a large enough data set to properly test the software by then?
I do think that the GPU version should be quite a bit faster (especially given that the machine I tested on has a very fast GPU and only a mid-range CPU). If that is not the case: Maybe there are some easy to figure out bottlenecks? Did you try this: https://developer.nvidia.com/nvidia-visual-profiler It's really very powerful.

I've uploaded two big dataset at the following link. The information about these datasets are the summarized down here:

Name
# variables
# classes
Train set size
Test set size
Dataset size
Time series length

UCI_CHAR
9
6
7352
2947
10299
561

Motor imagery in ECoG   recordings
64
2
278
100
378
3000

You can run the software on both CPU and GPU by using the following commands:
CPU:
./mdtwObj -t CLASSIFICATION -i CPU 3 0 -f <path_dataset>/DATA <path_dataset>/LABEL -k 10 0 -o 10299 561 -m 0 DTW -d 0
GPU :
./mdtwObj -t CLASSIFICATION -i GPU 3 <num_threads> 0 -f <path_dataset>/DATA <path_dataset>/LABEL -k 10 0 -o 10299 561 -m 0 DTW -d 0
I do know the NVIDIA Visual Profiler tool and I will try to use it to understand whether there's a bottleneck on my code. At first glance, i might say to you that a likely bottleneck might lie in transferring the data between the host and the device.

I encounter similar results (and thus conclusions) as @krischer . In my case, the example referenced in the first comment runs in 120 seconds on the CPU (Xeon E3-1240 v3) and 9 seconds on the GPU (GTX 760).
Keeping in mind that the CPU version

only uses a single core
is not as highly tuned as the GPU kernels (just looking at short_dtw_c() shows inefficient multi-dimensional arrays with mallocs and frees that could definitely be optimized further)
has no fundamental hardware characteristic that warrants a GPU speedup of more than a factor of ~10

the claims of three orders of magnitude performance gains in the paper cannot be supported.


Can't seem to get logformat to match what I have

I can't seem to get logformat to match what I have.
Output:
<IP_ADDRESS> [105-224-241-57.south.dsl.telkomsa.net] - - [26/Jun/2017:20:34:46 +0000] "GET /favicon.ico HTTP/1.1" 200 2238 "https://www.tnet.com/home/files/config.txt" "Mozilla/5.0 (X11; Linux armv7l) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.84 Safari/537.36"
Which is defined in my configuration as:
CustomLog       /www/logs/vhost.combined "%a [%h] %l %u %t \"%r\" %>s %b \"%{Referer}i\" \"%{User-agent}i\""

Please try:
goaccess access.log --log-format='%h %] %^[%d:%t %^] "%r" %s %b "%R" "%u"' --date-format=%d/%b/%Y --time-format=%T

Bingo... thanks!


Allow setting pass specific flags within check-tests

With PR #2319 , the following option to compiler_flags doesnt seem to work.
compiler_flags = ["-iree-codegen-convert-to-gpu=\"use-legacy-conv-lowering=false\""],

I get the following error
 Unknown command line argument '-iree-codegen-convert-to-gpu=legacy-conv-lowering=false'.

The command line works as expected with iree-opt (see test added with PR here)

So from what we discussed previously, I think a flag like this doesn't work with iree-translate either and the check test rules are just invoking iree-translate. I haven't played a lot with pass options, but my understanding is that you have to set these flags while building up a pass pipeline and they are specific to an individual instance of the pass (https://mlir.llvm.org/docs/PassManagement/#instance-specific-pass-options). iree-translate creates its main pass pipelines programmatically or you can invoke command-line specified pipelines. So you can run iree-translate -iree-mlir-to-vm-bytecode-module --iree-hal-target-backends=vulkan-spirv to use a predefined pipeline or iree-translate --iree-hal-target-backends=vulkan-spirv --pass-pipeline=canonicalize,iree-codegen-convert-to-gpu{legacy-conv-lowering=false} (something like that) to build up your own custom pipeline that includes an individual instance of iree-codegen-convert-to-gpu with some specific options, but I don't think you can just iree-translate -iree-mlir-to-vm-bytecode-module --iree-hal-target-backends=vulkan-spirv -iree-codegen-convert-to-gpu=legacy-conv-lowering=false and get it to pass that option to pre-existing instances of iree-codegen-convert-to-gpu=legacy-conv-lowering. I believe something like general global pass options for all instances of a pass would require changes to MLIR infra. For this specific pass, you could probably thread through a flag to iree-mlir-to-vm-bytecode-module that controls how it creates instances of this pass.

Yeah can't change pass pipeline options here like that. You could create new targets or use global flags, but I think a more holistic solution for controlling pipelines is better. I'll be exposing the ability for pipeline control as part of #3817 that should allow this to be plumbed through. Closing this issue now as it's 5 months old and we've survived without it in the current infra.

Thanks for closing this. @antiagainst  refactored the Vulkan/SPIR-V target to not use pass instance specific options. We just have vanilla LLVM command-line options. Just better overall to have these flags at all levels of the infra (pass, pipeline and binary).


When the prop "altitude" is not set the MapView does not work on Android

Summary
If you skip the prop "pitch" on the MapView, the following error occurs on opening the app on Android (The app opens and close. I got this error on LogCat of the Android studio):
FATAL EXCEPTION: main            Process: com.<package>, PID: 29312            com.facebook.react.bridge.NoSuchKeyException: pitch
The same problem occurs if the prop "heading" is skipped. This should raise an Exception or an Error on the build.
Reproducible sample code
import MapView, { PROVIDER_GOOGLE } from 'react-native-maps'

<MapView
    showsCompass={false}
    toolbarEnabled={false}
    showsUserLocation
    provider={PROVIDER_GOOGLE}
    maxZoomLevel={16}
    minZoomLevel={12}
    style={Styles.map}
    ref={mapRef}
    initialCamera={{
        center: {
            latitude: userCoordinate[0],
            longitude: userCoordinate[1],
        },
        pitch: 85, // <- if I remove this prop, the error occurs on the Android
        heading: 30,
        zoom: 14,
    }}
    onRegionChange={handleOnStartChangeArea}
    onRegionChangeComplete={handleChangeArea}
>
    {markers}
</MapView>

Steps to reproduce
1 - Import the MapView component
2 - Set the prop "initialCamera" and remove/skip de camera prop "pitch"
3 - Run the command "yarn android"
4 - The app will open and close in sequence without any error
Expected result
The expected result should be the map appears on the coordinates specified on the prop "initialCamera.center".
Actual result
The app opens and close as soon as the MapView Component is called!
React Native Maps Version
1.7.1
What platforms are you seeing the problem on?
Android
React Native Version
0.72.3
What version of Expo are you using?
Not using Expo
Device(s)
Motorola moto g(30)
Additional information
No response

@felipehidra Did you find any solution?

@haiderTkxel No. If the prop "pitch" is required, this should throw an error, no?

@felipehidra  not an issue ... please checkout typing (specifically MapView.types.ts / Camera type where pitch is explicitly required


fix(docker): bump uid for pptruser

Closes #13229

Would it be possible to save UID in a variable, so it can be used in USER command?
The same way as they do it in .Net Dockerfile, they save it in APP_UID variable ?
We are getting the same error as described in this stackowerflow question and the obvious way to fix it is to hard-code UID in our deployment, but if we'd have in in a variable it would be much easier and won't break if UID is changed again.

@olegd-superoffice Please open a bug to track this. Also happy to review a PR that implements this, and maybe even allow user to control it from the outside.


Explicitly define the file directory

Osquery was not working with Fleet after deployment due to wrong directories.
The following command installs the osqueryd service with --flagfile=\ProgramData\osquery\osquery.flags" however osquery.flags found at "C:\Program Files\osquery\osquery.flags"
"c:\Program Files\osquery\osqueryd\osqueryd.exe" -ArgumentList "--install" -Wait

The original osquery.flags defines the certfile.crt to be in "C:\programdata\osquery", it should be in  "c:\Program Files\osquery" instead.

Thanks for fixing this!


Overhaul

Brings the codebase into the current year by

migrating from async.js/callbacks to promises
adding typescript
adding Jest with a few unit tests for some of the most important pieces of code
replacing all but one usage of alks-node with alks.js since alks-node is deprecated
removing some unnecessary dependencies like async.js and left-pad
updating dependency versions to latest versions
breaking the code up from arbitrary groupings of functions into a bag-of-functions style layout
adding husky, prettier, & lint-staged for code autoformatting and automatic building of the dist folder on commit
fixing a couple small long-standing bugs such as being asked for a password and then a network password when running alks developer configure for the first time
updating the logger to print file/line number info in the prefix instead of an arbitrary string

This is a very large PR with a lot of changes, however the functionality from a user perspective should be identical save for the few bug fixes made. At least half of the files changed are files in the dist folder which are compiled automatically by typescript from the src folder, so you don't really need to look through them since they are autogenerated unless you just really don't trust typescript and/or rollup.
If you need help reviewing this PR or have any questions about the reasoning behind any of the changes please feel free to reach out

Awesome to see that this tool is continuing to grow. Thank you guys so much for taking over and continuing to improve the tool!


Segmentation fault when libdds.so is unload

I compiled libdds.so on Linux with the default parameter and I'm trying to use it with ctypes in Python. I noticed, when I unload the library at the end of a Python script, I get a segmentation fault.
Here is a toy example of Python script:
from ctypes import *
dll = cdll.LoadLibrary("libdds.so")

When I run it:
$ python libdds.py
Segmentation fault (core dumped)

According to the debugger, the issue come from FreeMemory function:
$ gdb
(gdb) file python
Reading symbols from python...done.
(gdb) run libdds.py
Starting program: /home/pierre/anaconda3/bin/python libdds.py
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff5e59454 in Memory::ReturnThread(unsigned int) ()
   from /home/pierre/Documents/python/bridge/python-dds-master_copy/libdds.so
(gdb) bt
#0  0x00007ffff5e59454 in Memory::ReturnThread(unsigned int) ()
   from /home/pierre/Documents/python/bridge/python-dds-master_copy/libdds.so
#1  0x00007ffff5e5cb31 in FreeMemory () from /home/pierre/Documents/python/bridge/python-dds-master_copy/libdds.so
#2  0x00007ffff7de7de7 in _dl_fini () at dl-fini.c:235
#3  0x00007ffff6a0cff8 in __run_exit_handlers (status=0, listp=0x7ffff6d975f8 <__exit_funcs>,
    run_list_atexit=run_list_atexit@entry=true) at exit.c:82
#4  0x00007ffff6a0d045 in __GI_exit (status=<optimized out>) at exit.c:104
#5  0x00007ffff69f3837 in __libc_start_main (main=0x400ab0 <main>, argc=2, argv=0x7fffffffde68, init=<optimized out>,
    fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7fffffffde58) at ../csu/libc-start.c:325
#6  0x00000000004009e9 in _start ()

An idea about how to correct that issue?

The same script works for me on Ubuntu 18.04.
Are you sure you are running with master from my fork of the repository. I am not a contributor here - this project has been in silence for a few months now.
The stack dump suggests FreeMemory is called from the dll destructor, which I have removed.

The backtrace mentioned above was generated with the code of the official repository, not yours. But, as I saw you proposed some fixes, I tried to compile libdds.so based on the code on your master branch (cf. my answer to your pull request). I still got an error but from a different part of the code:
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".

Breakpoint 1, Memory::GetPtr (this=0x7ffff67c0520 <memory>, thrId=0) at Memory.cpp:113
113	    cout << "Memory::GetPtr: " << thrId << " vs. " << nThreads << " vs. " << memory.size() << endl;
(gdb) bt
#0  Memory::GetPtr (this=0x7ffff67c0520 <memory>, thrId=0) at Memory.cpp:113
#1  0x00007ffff654e2ce in CloseDebugFiles () at Init.cpp:398
#2  0x00007ffff6557985 in Memory::~Memory (this=0x7ffff67c0520 <memory>, __in_chrg=<optimized out>) at Memory.cpp:23
#3  0x00007ffff7829ff8 in __run_exit_handlers (status=0, listp=0x7ffff7bb45f8 <__exit_funcs>, run_list_atexit=run_list_atexit@entry=true) at exit.c:82
#4  0x00007ffff782a045 in __GI_exit (status=<optimized out>) at exit.c:104
#5  0x00007ffff7810837 in __libc_start_main (main=0x555555636f40 <main>, argc=2, argv=0x7fffffffddf8, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7fffffffdde8)
    at ../csu/libc-start.c:325
#6  0x0000555555717e0e in _start () at ../sysdeps/x86_64/elf/start.S:103
(gdb) n
Memory::GetPtr: 0 vs. 0 vs. 0
114	    exit(1);

This time, the error was located in the destructor of the Memory class.
I made some changes to bypass the problem. You can find what I did in my fork of your work. In my opinion, it's not clean but it works: Python can release the lib without any error. I guess there is a problem because the memory is clean twice (but, to be honest, I don't know the code of this project, so it's just a supposition).

Thanks!  I'll add rimmington's fixes to the next release.

I experienced the same problem although with a different setup:
From within a Java based application I called the libdds.so external library via JNA. Similar behaviour: on closing the application a SIGSEGV error was produced.
Following the above suggestions from rimmington I changed two files: Init.cpp and Memory.cpp and recompiled the libdds.so library. Rechecking the above setup did not produce any error.
Please find attached the two changed files (Init.cpp and M
changedFiles.zip
emory.cpp from version 2.9 of the library.
Greetings
Klaus


feedbacks

Hi @tarleb,
Just used this template for the first time (https://github.com/InseeFrLab/pandoc-filter-fr-nbsp).
First of all, thanks a lot! It is really easy to use.
I have two suggestions for the documentation:

Explain how to update the documentation (despite this is quite easy) with make docs
Precise that the docs/CNAME file must be removed

Will do a PR if you are OK with these proposals, let me know.

Thank you, a PR would be most welcome. I should probably have removed docs/CNAME from this template, maybe its better to do that instead of documenting it.


Support for markdown fences without extra config

This is a cosmetic change to the actual implementation of the vim plugin, but it allows it to play nicely with the plasticboy/vim-markdown plugin for fenced code blocks inside Markdown documents without extra configuration.

How did you test this?
How are you using vim?
I'm asking because I know you can add a vim extension to IntelliJ, and probably to VisualStudio Code.
IntelliJ already has a Markdown plugin that interacts with the K syntax highlighter.
Trying to make sure we're not going to have conflicts or something break for other tools.

Tested in Vim & Neovim - hadn't realised people might be loading this into other editors as there seems to be an IntelliJ plugin already here, but I don't know how up-to-date that is.
I'll test this against whatever Vim emulation is available for IntelliJ and VSCode and get back to you.

@SchmErik you use vim in IntelliJ, can you make sure this still works?


better offline reconnect behavior

When internet connectivity is lost, webtorrent (websocket) tracker connections are lost and no reconnect happens. We should detect this condition and improve how it's handled.
This module might help: https://github.com/feross/connectivity

Reconnect behavior could still be improved. I don't see https://instant.io reconnecting to the websocket tracker after my laptop battery has died, or I've shut the lid.

The socket's readyState is 1, which is WebSocket.OPEN. Maybe we need to send heartbeats or something? Why isn't the 'close' event firing?

This is fixed now. You can go offline, come back online, and the download should resume. Also, remote peers who may be waiting for peers (because they have 0), will be able to connect to you.
There were many bugs that I squashed to close this issue:
simple-websocket

emit 'close' event correctly when we go offline https://github.com/feross/simple-websocket/commit/3ff3010b8743844d5279daf954c5663b515ed323

bittorrent-tracker

announce on reconnect https://github.com/feross/bittorrent-tracker/commit/6ef2d98b07e22b7258f9387925e7970c2280bf1b
remove socket from pool on destroy https://github.com/feross/bittorrent-tracker/commit/4128604301391bf61f9db070c0005d3b63b62ea2
https://github.com/feross/bittorrent-tracker/commit/6ef2d98b07e22b7258f9387925e7970c2280bf1b

This fix is live on https://instant.io, and in webtorrent if you run npm update.


Update selected state of sidebar

Please make it look like so:

For the hover effect, I suggest doing the same but with a lower opacity

Addressed in c6efcc53d1321944dd9ba5fdaf6243fcb0b1d10e


Make sure TaskSpec step container names aren't too long as well

Changes
PR #491 did the trick for generated resource containers, but not for TaskSpec-defined step container names. This fixes that.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Release Notes
n/a

/assign @pivotal-nader-ziada
/assign @dwnusbaum

/test pull-knative-build-pipeline-integration-tests

/lgtm
/approve

/test pull-knative-build-pipeline-integration-tests

@dwnusbaum : I am working on a PR to fix the flakiness you ran into.


more err tolerant build-pyspark.sh

the changes to build-script.sh from the other branch. Use lockfiles to avoid repeating steps in the event of an early exit

btw, I think I can add a script to set up pre-commit hooks for the lints / formatting from the docs. the creation of the pre-commit hook needs to be scripted because client side hooks can't be setup automatically

btw, I think I can add a script to set up pre-commit hooks for the lints / formatting from the docs. the creation of the pre-commit hook needs to be scripted because client side hooks can't be setup automatically

Thanks for the suggestion! I've created an issue to track this: #233.


TestBetaGeo setup_class is too slow

The setup_class of TestBetaGeo is too slow as it performs MCMC sampling on a synthetic dataset. We should make this a fixture of tests that actually need it (if there's more than one), and possibly mark them with @pytest.mark.slow so that they are not run locally by default.
https://github.com/pymc-labs/pymc-marketing/blob/7ebc19465ad6daf62fbcb56d1b91fc40d09da7a0/tests/clv/models/test_beta_geo.py#L50
For other tests I have faked posterior draws by doing prior sampling with narrow priors around the "expected" values, and setting that as the posterior dataset. This allows us to check the summary methods work as expected without doing slow mcmc sampling.
CC @larryshamalama

I'm working on the ParetoNBD PR right now, and have added pytest fixtures for CDNOW_sample.csv and CDNOW_master.csv which could potentially resolve this issue. These CSVs contain 2,357 and 23,570 rows respectively, and any tests using them should probably be marked as @pytest.mark.slow.
My opinion on using CDNOW for testing has flip-flopped in recent weeks because even CDNOW_master is much smaller than many datasets encountered in practice, but these are real-world benchmarks used in many research papers, and will be useful for testing against lifetimes MLE convergence and reproducing research results.

Resolved in https://github.com/pymc-labs/pymc-marketing/pull/709


Xaumi Redmi Note 4

In gmail always apears the following message:
this app won't work properly unless you allow google play services request to access the contacts

@Dolguim go to Settings, Apps/Applications, choose Gmail, then Permissions and finally change CheckBox state on Contacts item.

Obsolete after the 2018 version rewrite.


Mod Header is not working in latest chromium browser (ModHeader - Modify HTTP headers" was turned off)

While running and uploading crx file using selenium java with chromium browser with latest verion we are getting this. But i need to manually click  Manage exenstion and i need to enable it
Can we please help us to reslove this. This is blocker for our testing with latest chromium version
This is working in Chromium 128.0.6576.0 and not working in later version  129.0.6631.0

Did you manage to solve this? I'm having a similar issue with 129 and later versions of chromedriver. When ran locally on 129 or 130 everything seems to be fine, but any runs we have setup with github actions are just throwing an error about the extension not loading:
Request failed with status 500 due to session not created: session not created: failed to wait for extension background page to load: chrome-extension://jciacimpfapbkckkegdfngcnbdpabaoi/_generated_background_page.html
We're using wdio v8.


Use the camp platforms instead of MemorySpace

https://github.com/LLNL/camp/blob/master/include/camp/resource/platform.hpp
https://github.com/LLNL/RAJA/blob/develop/include/RAJA/internal/get_platform.hpp
https://github.com/LLNL/RAJA/blob/develop/include/RAJA/util/camp_aliases.hpp

Move bufferManipulation::HasMemberFunction_move into typeManipulation.hpp.


Make data pathing flexible

See #41

This looks good to me! One small change: I think we should have the env vars in all caps to follow convention


Proofreading pass

@rotycenh

OPS Build status updates of commit 5462f93:
:warning: Validation status: warnings

File
Status
Preview URL
Details

building-blocks/extending-templates/toc.json
:warning:Warning

Details

bread/toc.json
:warning:Warning

Details

toc.json
:warning:Warning

Details

docs/cloud-adoption/governance/resource-management/azure-resource-access.md
:white_check_mark:Succeeded
View

building-blocks/extending-templates/toc.json

[Warning] Error happen when converting building-blocks/extending-templates/toc.json to Pdf. Details: Could not find a part of the path 'T:\bc1d\building-blocks\extending-templates\toc.json'.

bread/toc.json

[Warning] Error happen when converting bread/toc.json to Pdf. Details: Could not find a part of the path 'T:\bc1d\bread\toc.json'.

toc.json

[Warning] Error happen when converting toc.json to Pdf. Details: Could not find file 'T:\bc1d\toc.json'.

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


Epic 1115 - Subscriptions

[ ] NCI-Agency/anet#1344: maintain per position a list of UUIDs to which he/she has subscibed
[ ] add a follow/unfollow button to any item with an UUID
[ ] add a "my feeds" page. It should contain

[ ] a chronological list (most recent on top) of

[ ] any note that references an item to which a user has subscribed
[ ] any report that references such items
[ ] at a later stage, system messages related to change of state of tracked object (i.e. report is approved)

Proposal of how should "events" be constructed at the first phase of implementation:
all lists are chronological, newest first
Person: list combining:
- all published (including cancelled) reports of which the person has been the author or attendee
- all notes posted on that person
Position: list combining:
- all published (including cancelled) reports of which the person in that position has been the author or attendee
- all notes posted on that pesition
Organization: list combining:
- all published (including cancelled) reports of this org
- all notes posted on that org
Task: list combining:
- all published (including cancelled) reports citing this task
- all notes posted on that task
Report: not sure yet
Authorization group : not sure yet
Location
- all published (including cancelled) reports with this location
- all notes posted on that location
My feed:
- a list of all event I have subscribed to
- with ability to filter by type (People / Locations / ...)