Feature request (or, please point me in the right direction)

Hey folks! I really love what you've done, and have used it for it's simplicity of code. I've begun learning Angular for a new project, and would like to use what you've built along with it, but it seems to be incompatible with the [UI-Bootstrap project] (https://github.com/angular-ui/bootstrap)
Are there any plans to port/integrate this project with Angular? If not, where could I start with that?
Thanks!

Hello, I don't know if it suits your needs but I personally use this project:
https://github.com/joaoneto/angular-bootstrap-select

FTR
Unfortunately, joaoneto's repository is not maintained anymore.
Creating my own directives was easier for me.
With repeat:
<select class="selectpicker" select-picker-bind-change-directive>
   <option ng-repeat="item in items" repeat-select-picker-directive>
      {{item}}
   </option>
</select>

I use a directive like this with $timeout to trigger the digest cycle:
  .directive('repeatSelectPickerDirective',
  ['$timeout',
    function ($timeout) {
      return function (scope, element, attrs) {
        if (scope.$last) {
          $timeout(function renderAfterPopulate() {
            $(element).parent().selectpicker('render')
          })
        }
      }
    }
  ])

If I bind to an event, I use a directive on the select tag:
.directive('selectPickerBindChangeDirective',
  ['$timeout',
    function ($timeout) {
      return function (scope, element, attrs) {
        $timeout(function bindOnChange() {
          element.on('changed.bs.select', function (event, index) {
            $timeout(function () {
              ...
            })
          })
        })
      }
    }
  ])

I forked angular-bootstrap-select and created my own version that I actively maintain: https://github.com/caseyjhol/ng-bootstrap-select. I'm open to pull requests. The main advantage is the addition of support for option attributes (data-icon, data-content, title, etc.). The examples need to be updated as well.


python3Packages.pythonegardia: add patch for search path

Motivation for this change
Fix build (https://hydra.nixos.org/build/159224435)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 147423 run on x86_64-linux 1

2 packages built:

python38Packages.pythonegardia
python39Packages.pythonegardia


Fails to load sites with a self-signed certificate

Hi!
I'm trying to load a page served on my local machine, but I keep getting a setURL exception https://localhost:8443 message in the dev console, and a blank page with a loading indicator.
I wonder if this has something to do with having a self-signed certificate on that domain? I get similar behavior on other sites with self-signed certificates.
I'd be completely fine with manually importing certs or some such, but I don't know how to do that either.
Any thoughts?

I'd be completely fine with manually importing certs or some such, but I
don't know how to do that either.

I think that you set it up in the desktop chrome browser and the atom one
will use the same settings.  You should test it in the desktop browser
first.  If something doesn't load on the desktop it won't load in Atom
either.  Let me know if this helps.
In any case, there shouldn't be exceptions.  Can you give me an example URL
so I can handle the error better?
On Tue, Mar 10, 2015 at 1:11 PM, David Alan Hjelle<EMAIL_ADDRESS>
wrote:

Hi!
I'm trying to load a page served on my local machine, but I keep getting a setURL
exception https://localhost:8443 message in the dev console, and a blank
page with a loading indicator.
I wonder if this has something to do with having a self-signed certificate
on that domain? I get similar behavior on other sites with self-signed
certificates.
I'd be completely fine with manually importing certs or some such, but I
don't know how to do that either.
Any thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/mark-hahn/web-browser/issues/42.

Oh wait.  Are you using a file:// protocol?  There is a bug in the
webview tag that has been fixed but not landed in Atom yet.  This may be
the cause of your problem.
On Tue, Mar 10, 2015 at 5:32 PM, Mark Hahn<EMAIL_ADDRESS>wrote:

I'd be completely fine with manually importing certs or some such, but
I don't know how to do that either.

I think that you set it up in the desktop chrome browser and the atom one
will use the same settings.  You should test it in the desktop browser
first.  If something doesn't load on the desktop it won't load in Atom
either.  Let me know if this helps.
In any case, there shouldn't be exceptions.  Can you give me an example
URL so I can handle the error better?
On Tue, Mar 10, 2015 at 1:11 PM, David Alan Hjelle <
<EMAIL_ADDRESS>wrote:

Hi!
I'm trying to load a page served on my local machine, but I keep getting
a setURL exception https://localhost:8443 message in the dev console,
and a blank page with a loading indicator.
I wonder if this has something to do with having a self-signed
certificate on that domain? I get similar behavior on other sites with
self-signed certificates.
I'd be completely fine with manually importing certs or some such, but I
don't know how to do that either.
Any thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/mark-hahn/web-browser/issues/42.

Thanks! Yes, after I imported the certificate (Chrome on OS X uses the built-in Keychain.app for certificates, apparently) it loads fine. I should have thought of that earlier!
As far as the exception, you can see it by going to https://webmail.thehjellejar.com. This page has a certificate->URL mismatch that triggers a setURL exception https://webmail.thehjellejar.com exception.

This web-browser package is no longer supported.  Use the atom package browser-plus instead.


AppsLanguageSwitch: Changes more settings than it should?

Prerequisites

[X] Verify that this is not a Windows issue;
[X] Refer to the Before running.
[X] Refer to the How to use.
[X] If you issue concerns the Wrapper, please mention @BenchTweakGaming in issue

Steps to reproduce
Hi there,
It doesn't matter if I invoked the script with AppsLanguageSwitch -Enable or AppsLanguageSwitch -Disable, but this tweaks changes both of these options:

But it should only change the first one, right?
This is a screenshot from Windows 10 (where I'm posting this), but I'm reporting this issue from a Windows 11 perspective, where I've actually used and experienced this issue while setting up a new machine with Windows 11.

Hm... 🌵

Couldn't reproduce: all work correctly. The function uses this cmdlet Set-WinLanguageBarOption -UseLegacySwitchMode. Works as it was intended to be.
It doesn't check Use the desktop language bar when it\s available.
https://docs.microsoft.com/en-us/powershell/module/international/set-winlanguagebaroption

@farag2 It does for me, every time, with -Enable or -Disable. Anything I can do to help debug this?

I'd take a look remotely. Really it's so strange... As I mentioned, we may do it via AnyDesk. PM in Telegram: https://t.me/sanctuary_d


uwp version?

with broadfileaccess now available on uwp platform of windows 10 can we have a FileManager version on uwp as well?

Why UWP? Feel the retro smell of this program and enjoy the "old" win32 experience.

It is not a goal of this project to support anything besides a native (x86 / x64) desktop app.


Return 0 to avoid errors with setup-cfg-fmt in Makefile

Hi!
I am using setup-cfg-fmt in a makefile. When I run make format it returns an error.
The makefile command
format:
	setup-cfg-fmt setup.cfg

Output from terminal when I run make format
setup-cfg-fmt setup.cfg
Rewriting setup.cfg
make: *** [format] Error 1

The script works as expected and updates setup.cfg.
I cannot see any error in your script, but maybe it is because the script returns the variable retv set to 1 if it completes successfully, and Make expects programs return 0 if there aren't any errors.
Can you make a small change to the script and set the variable retv to 0 if it runs successfully? Another option is to not return retv if it isn't used.

the nonzero exit code is to indicate changes have occurred. you're free to ignore the exit code


fix: remove duplicate step in getting started guide

Why:
Confusing for user to read.
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):
GitHub copilot installation guide for Neovim.

Check off the following:

[X] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[X] For content changes, I have completed the self-review checklist.

Why:
Confusing for user to read.
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):
GitHub copilot installation guide for Neovim.

Check off the following:

[X] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[X] For content changes, I have completed the self-review checklist.

😜

😉👌🙄🤫
On Sun, 9 Apr 2023, 8:16 pm Ismaelkadir, @.***> wrote:

Why:
Confusing for user to read.
Closes:
What's being changed (if available, include any code snippets,
screenshots, or gifs):
GitHub copilot installation guide for Neovim.
Check off the following:

I have reviewed my changes in staging (look for the "Automatically
generated comment" and click the links in the "Preview" column to view your
latest changes).
For content changes, I have completed the self-review checklist
https://github.com/github/docs/blob/main/contributing/self-review.md#self-review
.

😜
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/24953#issuecomment-1501115262, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A6TDZ4D2ZKASKXBHDI6WKRTXAKSCVANCNFSM6AAAAAAWYCGPDI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Does breadcrumb have a solution?

@alexchopin  @Atinux @pi0 @clarkdo  there are a lot of nonsensical issues like this being raised lately. I guess they take a lot of your time too. Can we do something to help to clear this up. Maybe assign some moderators from community, otherwise some very feedback and issues are escaping attention under this rubble.

I wanna be able to close issues :pizza: do me do me
Also, lets funnel people into the Vue Land #nuxt Discord channel from:

nuxtjs.org
README.md
The header of https://gitter.im/nuxt/nuxt.js

@husayt I like your logic :pizza:

See a nonsensical issue with No description provided.
Become concerned the admins' time is being wasted
Tag all the admins to waste their time
:call_me_hand:

Thanks @qm3ster

@Atinux pls merge #2783 and gib me permission to close issues :ok_hand: hue hue

Done, great work BTW 🙌

@qm3ster your sarcasm is noted. You lost the point however. There are over
300 open issues. The point was to see what can be done to help admins on
this. Project has grown immensely and needs more helping hands.
On 27 Feb 2018 14:02, "Mihal Malostanidis"<EMAIL_ADDRESS>wrote:

@husayt https://github.com/husayt I like your logic 🍕

See a nonsensical issue with No description provided.

Reply to this email directly, view it on GitHub
https://github.com/nuxt/nuxt.js/issues/2899#issuecomment-368887518, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABHXrAx5In-gW57X2O8y9Ou8Qk_yfnwCks5tZAr9gaJpZM4SStgI
.

@husayt I agree also with you, we have more and more issues and a lot of them are non-sense. You guys are helping us a lot and I am more than happy to have you on board to help us make some issue triage and assign us critical issues that needs to be resolve and close the others where an answer is given.

Thanks @Atinux. Can you also add me to docs website. That also needs a bit of cleanup, I noticed quite a few broken anchors and etc.

@Atinux I'd be willing to help out here too!


Decode Cas SLO response

Sometimes the SLO response from Cas is in URIEncoded format. Decoding it so that ST can be extracted.

Coverage remained the same at 90.285% when pulling 4ccc4111b6cee5b4463627b8a1fdea5ef2ab1ee7 on soumyart:master into 12a37bef12f28b601df6a331576b03afd9ad31bc on TencentWSRD:master.


Documents added

Make these three necessary changes. 1. Add registration numbers next to your names. 2. Add the website of data source link 3. in github repo link it should be of main repo and not of any file
Thank you!

Done


Installer: PHP7.2 + Vagrant VM

I've created a Vagrant VM and added a working Vagrantfile.
For more informations about the VM, use our Documentation: https://github.com/CircleBilling/CircleBilling/wiki/Vagrant-Test-VM
Also I've changed the mysql_* functions to PDO with clean transactions and rollbacks for the database import.

@gOOvER can't find any license stuff on install.phtml (https://github.com/CircleBilling/CircleBilling/commit/33e6c1fabcdcda7941cbf5d14c8800f6cc79490f#diff-b996fc136f541cb2810088c68ebdd784)


os.Mkdir set setting suid and/or guid bits

What version of Go are you using (go version)?

$ go version
go version go1.14.5 linux/amd64

Does this issue reproduce with the latest release?
Yes, I also tested with go 1.14.6 linux/amd64
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/theo/.cache/go-build"
GOENV="/home/theo/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/theo/dev/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/theo/dev/repos/ubsu/go.mod"
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-build371835464=/tmp/go-build -gno-record-gcc-switches"

What did you do?

I tried using os.Mkdir with an os.FileMode that should result in the suid bit being set for the directory.
package main

import "os"

func main() {
	os.Mkdir("testdir", 0755 | os.ModeSetuid)
	// os.Chmod("testdir", 0755 | os.ModeSetuid)
}

I expected the resulting directory to have the setuid bit set when running ls -l in my shell, but to no avail.
Note that when uncommenting the last line of the program, os.Chmod properly sets the setuid bit for the directory.
Am I missing something here; perhaps this is a dumb question? What's with this behavior?

Forgot to mention in the body: the same thing happens with os.ModeSetgid and the setgid bit.

The mkdir syscall cannot set the S_ISUID bit.
Here is the relevant sentence from the Linux mkdir manual:

It is modified by the process's umask in the  usual  way:  in the  absence  of  a  default  ACL, the
mode of the created directory is (mode & ~umask & 0777).

The mask 0777 removes the S_ISUID bit.

Mask 0777 excludes the S_IGUID as well. I suggest to extend the documentation of os.Mkdir.

Ok, thanks for the clarification.


update_cp_org_libraries.py runs veryyyy slow

Results of my initial tests of the functions that get run.
Total time per repo: 3.5s
validate_actions_state: 0.459673s
validate_contents: 0.433544s
validate_core_driver_page: 0.003178s
validate_default_branch: 0.000001s
validate_in_pypi: 0.040591s
validate_labels: 0.117371s
validate_passes_linting: 0.639910s
validate_readthedocs: 1.361857s
validate_release_state: 0.333909s
validate_repo_state: 0.134701s
So the real big one is validate_readthedocs. That one is definitely one of the more complicated ones but I think we could for sure simplify it.
validate_passes_linting also takes a while but it actually clones the repository so that make sense.

We are now often seeing the job exceed the 6 hour job runtime limit. For example: https://github.com/adafruit/circuitpython-org/actions/runs/6809744073/job/18516615845.
There is some GitHub API rate limiting going on too (see in that job), but it would be really nice to make this run faster.

Thinking about your local test in #361, are we doing the queries without credentials? If we did them with credentials, would we avoid the rate limit?

I'm not sure about authorization. I filed #348 thinking that this process was using Classic PATs (https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/managing-your-personal-access-tokens#creating-a-personal-access-token-classic) which are deprecated or discouraged; I forget the exact context that led to me filing that issue but it seems to be related to this note in the docs:

Note: Organization owners can restrict the access of personal access token (classic) to their organization. If you try to use a personal access token (classic) to access resources in an organization that has disabled personal access token (classic) access, your request will fail with a 403 response. Instead, you must use a GitHub App, OAuth app, or fine-grained personal access token.

Using PATs of some kind seems to be the Proper Way (TM) to do this, but "fine-grained" PATs are the preferred way now, if we don't want to deal with having a GitHub App: https://docs.github.com/en/actions/security-guides/automatic-token-authentication#granting-additional-permissions
I am pretty sure SOME kind of authentication/token is at play here, otherwise the unauthenticated limit is something like 20 API calls an hour, far too few.

My local test was without authorization, but that was kinda-nice since it let me easily hit the timeout case. For the run in circuitpython-org that's troubling us, we appear to be using a token via ADABOT_GITHUB_USER and _ACCESS_TOKEN:
Thu, 09 Nov 2023 09:20:13 GMT     ADABOT_GITHUB_USER: ***
Thu, 09 Nov 2023 09:20:13 GMT     ADABOT_GITHUB_ACCESS_TOKEN: ***

since the value of repository secrets can't be inspected, I don't know what user & token is in play.

A successful run looks like this:
Sat, 25 Nov 2023 09:19:00 GMT Run Date: 25 November 2023, 09:19AM
Sat, 25 Nov 2023 09:19:00 GMT  - Report output will be saved to: /home/runner/work/circuitpython-org/circuitpython-org/bin/adabot/libraries.v2.json
Sat, 25 Nov 2023 10:02:45 GMT GitHub API Rate Limit reached. Pausing until Rate Limit reset.
Sat, 25 Nov 2023 10:02:45 GMT Rate Limit will reset at: 2023-11-25 10:19:13
Sat, 25 Nov 2023 10:19:13 GMT GitHub API Rate Limit reached. Pausing until Rate Limit reset.
Sat, 25 Nov 2023 10:20:59 GMT {
Sat, 25 Nov 2023 10:20:59 GMT   "updated_at": "2023-11-25T09:19:00Z",
[rest of json snipped]

so we have:

runs for ~40 minutes before hitting rate limit
sleeps for ~20 minutes after printing "rate limit reached" message once
encounters rate limit message again, but apparently doesn't sleep this time
finishes about 2 minutes later

A failed run:
Fri, 24 Nov 2023 09:20:27 GMT  - Report output will be saved to: /home/runner/work/circuitpython-org/circuitpython-org/bin/adabot/libraries.v2.json
Fri, 24 Nov 2023 10:06:30 GMT GitHub API Rate Limit reached. Pausing until Rate Limit reset.
Fri, 24 Nov 2023 10:20:39 GMT Rate Limit will reset at: 2023-11-24 10:20:39
Fri, 24 Nov 2023 10:20:39 GMT GitHub API Rate Limit reached. Pausing until Rate Limit reset.
Fri, 24 Nov 2023 10:20:39 GMT Rate Limit will reset at: 2023-11-24 10:20:39
Fri, 24 Nov 2023 15:20:09 GMT Error: The operation was canceled.

here,

runs for about 45 minutes
sleeps for about 15 minutes
apparently goes to sleep again for a long time, or never hits rate limit again
gets timed out 5 hours later

Besides adding debugging another idea is to prepend the command with timeout so that we can hopefully get a Python traceback from where the process is stuck, something like timeout -s INT 18000 [adabot command]. this would be a change in the circuitpython-org repo, not here, I think.

also we could sleep +60 seconds after we think the rate limit will reset, not +1 second; since we hit the rate limit after 40-45 minutes and it resets after 1 hour, this doesn't really lower our throughput much.

I think I found the real problem :tada:

Fixed by #362.

I saw another failed 6-hour run, despite circuitpython-org being updated to include #362. :confused:
https://github.com/adafruit/circuitpython-org/actions/runs/7030445965
Reopening
@jepler

oh drat

I have thought about moving part of the CI into libraries themselves somehow.  The basic idea is to have repos trigger when updates to push a record of the information gathered either to a central repo (like a folder in this repository, for example) or possibly an S3 bucket.  This has the advantage of spreading out or eliminating API calls, as well as distributing the burden over multiple CI runs across the libraries.  This would allow this CI to just collect them all.


v1.0.0

Coverage remained the same at 91.781% when pulling 2561f9e5b04ea34626e8a6da1278857c5c8f40ea on dev into c20d4956c95d431e58af22a2131d054c0f6ec3c1 on master.


avoid calling daemon.Info()

just use pkg/platform (which is cross platform)
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

@jfrazelle I don't know how to update vendors :confused:

gvt! hang on can get a link in a bit I'm on my phone
On Mon, Jun 13, 2016 at 3:08 PM -0400, "Antonio Murdaca"<EMAIL_ADDRESS>wrote:
@jfrazelle I don't know how to update vendors 😕
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

https://github.com/FiloSottile/gvt
using gvt you'll want to gvt fetch it and it should update the manifest

thanks @damm
@jfrazelle good to go :)

Thankssss


Official Docker Image

What are your thoughts on adding docker support? Both Unbound and NSD would be great to have as officially maintained docker images and some of your other projects already have docker support. I have an example repository over at docker-nsd. I am more than happy to create a pull request and help to continue to maintain the image. (I would prefer to contribute here rather than have a separate fork.)

Hi @nbish11! Thank you for bringing this to our attention. We'd love to add a reference in contrib for this, but prefer not to ship an official Docker image at this point. The reason for that is that there are too many configuration parameters and there's no one recommended way that is guaranteed to work for every user. Adding it to contrib is a good solution in situations like this. Issuing a PR against contrib/Docker (or similar) is probably the best way forward? Just adding a link to contrib/README.md is another possibility.

Thanks k0ekk0ek. I will open a PR against contrib a little later to close this issue.
Your not wrong about the config options. I've actually 'borrowed' your docker magic build technique from the krill library to allow NSD to be built for cpu or memory optimisation; as even for me, NSD needs to be built differently, depending on the project.

Closing this issue.


test: replace cli with go-ceph for snapshot namespace PR

This PR is to test changes of https://github.com/ceph/ceph-csi/pull/1446 and to avoid panic in E2E tests

This is just a testing PR
cc @agarwal-mudit

Thanks @Madhu-1

/test ci/centos/mini-e2e-helm/k8s-1.19


The example code in the provided link needs to be updated.

Sandcastle example:
Browser:
Operating System:

https://cesium.com/learn/cesiumjs-learn/cesiumjs-3d-tiles-styling/
The example code in the provided link needs to be updated.
链接中的示例代码需要更新了

Thanks @danchaotaiyang! I just fixed this on the website. The update should be published shortly.


Crop box can't be move or resized with base64 image in angular modal on Mobile

Hello,
I'm using the latest 2.2.1 version of your jQuery plugin with an angular app running on Mobile. (both cordova and Mobile browser).
I've an issue with the crop box. I can load the picture, and the crop box, but i just can't move or resize the box when i am on Mobile (it's working like a charm on Desktop Browsers).
I have seen your known issues with iOS, and i'm resizing my pictures to fetch 400px max (square) so no heavy picture anymore.
As i said, I can load the picture, the cropper and previews, but i can't move/resize the crop box on mobile.
Also, i'm using a base64 image as src.
There is my init function, called after the picture is completely loaded :
$('#cropper').load(function() {
  $('#cropper').cropper({
    aspectRatio: 1 / 1,
    preview: '.preview',
    strict: true,
    autoCrop: true,
    movable: false,
    cropBoxMovable: true,
    cropBoxResizable: true,
    zoomOnTouch: false,
    zoomOnWheel: false
  });
});

Do you have any ideas of what's happening there ?
Thanks.

I'm sorry, I have not idea about this, you may need to check it by yourself. By the way,  the strict option had been removed since v2.0.0.

Thank you for the tip with the strict option. I've removed it, but nothing change...
That's weird then, this plugin is supposed to work with mobile isn't it ? Do i need a special option to enable the plugin on Mobile Phone or best practices to do ?
Anyone has already tested this plugin with cordova ?
Thanks for the Help :-).


As user i want optionally remove local feature brach after finishing it.

I hope I did not bring you the inconvenience with all my issues :)

On the contrary! I like all the good ideas and testing TGit is now getting :D

Added to the 3.7 release :)


Reuse computed module info

Fix to re-use the already computed module info rather than re-running it. We'll avoid walking the frame stack and constructing the set twice

Thanks. Pinging Rob for a review on this.


Update command syntax

Changed COMMAND_WORD in each command class, as well as prefix syntax in CliSyntax
Updated LogicManagerTest to reflect new syntax

This PR can be closed as it is entirely covered by updates in #34


NoMethodError: undefined method `timestamp_sort_order' for Rails 4.x.x

I was able to integrate paper trail with rails 4.x.x my implementation was based on this stackoverflow.
My problem though is that my solution was creating a Module PaperTrail::Version inside the initializer. Now I want to refactor my code and move it to a model app/models/paper_trail/version.rb but doing this seems to cause errors. Any help?
ruby: 2.1.2
rails: 4.0.1
paper_trail: 4.0.0
Gist here!

Per the contributing guide we prefer that you ask usage questions only on Stack Overflow, so I'll close this to encourage participation there. Thanks!
If you think you've found a bug in PaperTrail, please use our bug report template.


[AutoPR apimanagement/resource-manager] Revert not support formats xml-link and rawxml-link

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5763

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3106


fix: escape headers and unordered lists in markdown utils

Summary
Expanded markdown support in remove markdown and escape markdown methods. This does not currently support ordered lists, unfortunately.
Checklist

[x] If code changes were made, then they have been tested

[ ] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running pdm lint
[x] I have type-checked the code by running pdm pyright

[ ] This PR fixes an issue
[ ] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Due to more quirks with the client this entire implementation is wrong.


Allow projectDir to be configurable

Description
I want to use this project in a monorepo setup, which requires me to have the project setup from the root (our tooling depends on running from the root). Therefore I'd like to be able to pass in an option to configure the procjectDir.
I'm going to fork the project and try to implement this myself, unless this is something that you absolutely do not want in the project. Mostly just looking for feedback here since I'm going to invest time in this myself.
Expected Behavior
Running npm run start -- --projectDir=src/js/mobile or something similar would allow me to run react-scripts onto that directory instead of always depending on process.cwd(). This would of course be backwards compatible and fall back to process.cwd() if the parameter is not set.

I found the entryPoint configuration for expo that allows me to do what I want here. Although it doesn't allow me to have multiple app projects in the same monorepo, it's sufficient for now at least. I'll deal with multiple projects when if I get to that point.


Store MudTable state in query string

Feature request type
Enhance component
Component name
MudTable
Is your feature request related to a problem?

There is a MudTable with filters, sorting and paging on the page
I click on the row and go to other page
I click back
MudTable's state is not preserved

Describe the solution you'd like
Starting from .NET 8 Blazor supports [SupplyParameterFromQuery] in a non-routable components, it may be utilized to preserve state. We can start from something easy like page number, for example.
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

+1


✨ Builder: Navigation Menu

I have a lot of the groundwork in #378, if someone wants to take it over let me know and I'll move it out of draft, if not, will leave it until an alternative is ~the same level of development.

This component would be a great addition (and much appreciated)


cannot see any of the docker containers installed by Deplorarr

I have 55 containers and over 192 images, but I am not seeing anyone of the ones installed.  I can see where i can install them, but not the existing running containers.
I am using Proxy Socker, so that might be the problem, does this support Proxy Socket?

Right now the app is configured to connect to the Docker socket: '/var/run/docker.sock'.
With the socket proxy, you would be connecting with something like 'tcp://localhost:2375'?

@moltra Could you give me some more information about  the Proxy Socket or a link to the project?

@moltra  Deployarr uses Socket-Proxy.
@lllllllillllllillll I came to post a similar issue/request - support for Docker Socket Proxy. I am the author of Deployarr. DweebUI looks awesome. I see you have a patreon page, do you have an opencollective page? I would love to sponsor the project.

@anandslab yes, I know I was at one time very active on your discord.  I still review it at time to time.  I will probably get active on your board once I get my server back up.

@anandslab That's an interesting project. My next release won't have the docker.socket requirement, so it shouldn't have any issue with socket-proxy.
I'll have to take a look at creating an Open Collective page.

Awesome, i already added DweebUI to the next Deployarr release, with docker socket. I will update it once you add socket proxy support.
Opening an opencollective page is very easy. Be sure to add a separate organization page for DweebUI so you can keep it separate from from your individual account. I had many issues with Patreon. Plus it appears they take a higher commission.


[CDCSDK] Add check to ensure received OpID always more than older checkpoint

Added a check so that if the existing checkpoint value is greater than one which we receive, we don't update the checkpoint.

Also, please write comment in detail describing the problem and solution.


Inventory script empty space

Inventory scripts aren't properly recognizing empty space [] in the slots section.
Current work around is to fill empty space with air [i@air] however in accorance to the inventory script container... it should automatically fill empty slots with air.

This makes no sense. The leading spaces clearly indicate that [] is working. Are there errors in the console?

It's actually an issue posting for someone else on the IRC, I've tested it and encountered the same issue, and it's quite common amongst IRC newbies. It appears to be ignoring the ending spaces, however registers the previous ones.

Re-tested just then.
http://puu.sh/mHkiR.png - Image of inventory opened
Debug: http://mcmonkey.org/paste/29003
Script:
Inv:
  type: inventory
  size: 27
  definitions:
    x: i@barrier
  slots:
  - "[] [] [x] [] [] [] [] [] []"
  - "[] [] [x] [] [] [] [] [] []"
  - "[] [] [x] [] [] [] [] [] []"

mcmonkey4eva/Denizen@cb2d22a

If you look at the Top Picture, you can grasp that the Code is somehow ignoring all the []'s after the Item, and skips to the next row. You can fix this by specifying that the Last [] in each row be air. So somehow for some reason, Brackets only get turned into air if they are before a valid item, and are ignored if they aren't... Strange

@Devotedworker we already fixed it, test latest dev build


Any trained weights offered?

Can you provide a weight file that has been trained for inferencing a single image?
Thank you a lot.

Hi, this repository supports inference for this method, but if you need training, you can use this repo: https://github.com/uyzhang/CLIP-RC.


Update to alexa-remote2 3.8.1

Fixes #22
Awesome solution from @bbindreiter https://github.com/cakebake/node-red-contrib-alexa-remote-cakebaked/issues/22#issuecomment-853162235
Fixed in upstream alexa-remote2 https://github.com/Apollon77/alexa-remote/releases/tag/v3.8.1

Thanks for the PR.
I refer again to the discussion https://github.com/cakebake/node-red-contrib-alexa-remote-cakebaked/issues/22#issuecomment-855463636 that I am happy to add developers to the project to enable faster reaction for end users.

GitHub, NPM and Node-RED in sync - new tag: 4.0.3

Thank you so much @cakebake !


Update Array of Type Mixed

Hi,
i'm currently trying to save an array of Informations inside a Log entry, but i just cant get it to work i want to lock the max Infomation saved to 20 rows, but it jsut doesnt work it just saves the new one... help please xD
here is my model:
var log = new mongoose.Schema({
    Info: Array,
    Type: {
        type:      String,
        "default": "undefined"
    },

    Class: {
        type:      String,
        "default": "undefined"
    },

    Message: {
        type:      String,
        "default": "undefined"
    },

    Counter: {
        type:      Number,
        "default": 0
    },

    Last: {
        type:      Date
    }
});

here is my code where i update:
logModel.findOne({
    Message: newEntry.Message
}, function (err, doc) {
    if ( doc ) {
        // This should lock the max length to 20 ...
        if(doc.Info.length > 20) {
            while(doc.Info.length >= 20) {
                doc.Info.shift();
            }
        }
        // add the new one...
        doc.Info.push(newEntry.Info[0]);
        doc.Counter++;
        doc.Last = newEntry.Info[0].Date;

        // mark modified
        doc.markModified("Info");
        doc.save(function (err, newdoc) {
            console.log(doc.Info.length, newdoc.Info.length);
            console.timeEnd('Processed Log Entry ' + currentNumber + '/' + data.length);
            callback();
        });
    } else {
        newEntry.Last = newEntry.Info[0].Date;
        newEntry.save(function (err) {
            console.timeEnd('Processed Log Entry ' + currentNumber + '/' + data.length);
            callback();
        });
    }
});

kinda tricky how we have our update builder working but we can get around it by using doc.Info.nonAtomicPush(...) instead of doc.Info.push(...) for now.

nonAtomicPush doesn't utilize the $push atomic method which is what is causing the update builder to forget about the original shifted items.


IOS App 2.2.1 doesn't connect to private Zulip server

I installed Zulip server with a self signed SSL Cert.
Could that be the reason?
Error message of App:
Uh oh, something happened that we didn't expect.
Unable to login. Please try again.

@sabotrax, docs clearly point out that self signed certificates are not supported (yet). Ok they just mention the desktop app but i assume the same behaviour for ios.
But i have a related problem even using a properly signed cert (start.com). Desktop app and browser connect without any problem but the IOS app just refuses to do so. Same error message as reported by sabotrax.
The strange thing is that the app did connect once (right after installation) but after a day or so it suddenly started to fail for no obvious reason (no changes were made to the server configuration).
Killing the app in IOS does not help, uninstall + reinstall does not help either.
The only thing special with my setup is that i'm using port 7777 instead of standard 443, thus i enter "https://myserver.foo:7777" as domain.
I'm running Zulip server 1.3.6 on a Ubuntu 14.04 vm. The app runs on IOS 9.0.2.
Using tcpdump i can say that the app is trying to connect since i see network traffic on the server box.

Ya, I did read it somewhere but wasn't sure about it. Thanks for pointing it out.

I have to correct myself: there're actual no network packages arriving at my server box (maybe my tcpdump captured some traffic related to a desktop app connection) which means the IOS app does not even try to connect. Sorry for my probably misleading prior post...
As i mentioned before i have a installed an "official" SSL certificate signed by startssl.com. I reckon the app has some problem with the certificate though. I manually imported the startssl CA and intermediate certificate to the iphone but that did not change anything. Maybe the app includes the custom port 7777 when it comes to certificate CN verification? Very little to guess at this place since there's no way for me to debug.

Ok, my mistake. I added the protocol part in front of the server name "https://myserver.foo:7777". Omitting the "https://" everything works.
Sorry for that.

We should probably patch the iOS app to add the https:// if it wasn't there

I have a installed an SSL certificate.  Desktop app run in window & mac have no problem. visit in browser have no problem too, even in iOS.
but, in iOS app,error message:
Uh oh, something happened that we didn't expect.
Unable to login. Please try again.

I am sure that I have configure server, email and pwd correctly.
server: https://myserver:10443
email<EMAIL_ADDRESS>password: xxxx

@wq163, do you prefix myserver:10443 with https:// in the Domain field when trying to connect? If you do so ommit the https:// and try again (see my post above).

@virtuaris ,yes. I prefixed with https://, didn't connect to server

@wq163, sry if i did not express myself clearly enough. Please try without the https:// prefix in the Domain field.

@virtuaris, sorry, I did not look carefully.  thank you.

Updated the description to cover the actual problem here, which is if you include https:// in the private server URL , it doesn't work.


[gtkmm] build failure

Host Environment

OS: Linux pop-os 5.4.0-7642-generic #46~1598628707~20.04~040157c-Ubuntu SMP Fri Aug 28 18:02:16 UTC  x86_64 x86_64 x86_64 GNU/Linux

To Reproduce
Steps to reproduce the behavior:
./vcpkg install gtkmm
Failure logs
yingshaoxo@pop-os:~/Documents/vcpkg$ vcpkg install gtkmm
Computing installation plan...
The following packages will be built and installed:
  * atkmm[core]:x64-linux
  * brotli[core]:x64-linux
  * bzip2[core]:x64-linux
  * cairo[core,gobject]:x64-linux
  * cairomm[core]:x64-linux
  * dirent[core]:x64-linux
  * expat[core]:x64-linux
  * fontconfig[core]:x64-linux
  * freetype[bzip2,core,png]:x64-linux
  * gdk-pixbuf[core]:x64-linux
  * gtk[core]:x64-linux
    gtkmm[core]:x64-linux
  * harfbuzz[core,glib]:x64-linux
  * json-c[core]:x64-linux
  * libepoxy[core]:x64-linux
  * libpng[core]:x64-linux
  * libuuid[core]:x64-linux
  * pango[core]:x64-linux
  * pangomm[core]:x64-linux
  * pixman[core]:x64-linux
  * pthread[core]:x64-linux
  * pthreads[core]:x64-linux
  * ragel[core]:x64-linux
  * tool-meson[core]:x64-linux
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-linux...

CMake Error at scripts/cmake/vcpkg_fail_port_install.cmake:94 (message):
  Target 'Linux' not supported by atkmm!

Call Stack (most recent call first):
  ports/atkmm/portfile.cmake:1 (vcpkg_fail_port_install)
  scripts/ports.cmake:136 (include)

Error: Building package atkmm:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: atkmm:x64-linux
  Vcpkg version: 2020.11.12-unknownhash

Additionally, attach any relevant sections from the log files above.
``

Hi @yingshaoxo
Thanks for posting this issue.
atkmm only supports windows platform now.

Hi @yingshaoxo
Thanks for posting this issue.
atkmm only supports windows platform now.

But how can i use gtkmm under Linux if I can't build atkmm?
I just want to install gtkmm, then do some development with it under Linux.
Of course I love vcpkg, but how can I make it to work with this purpose?

@yingshaoxo
Since atkmm is one of required dependencies for gtkmm.  Without atkmm , gtkmm cannot be built on Linux in vcpkg.
I'm sorry, there are no ways to make it work on non Windows now.

@NancyLi1013
Thanks, I have figured it out.
This is a bug that just happened with vcpkg: https://github.com/microsoft/vcpkg/issues/9564
atkmm can be compiled independently.

Thanks for your update. It's so great that you have found another way to use gtkmm on Linux. Also thanks for your help to pointing this out. It seems that you use gtkmm that is not provided by vcpkg.
The root cause of this issue is the same as that in #9564. So let's track this issue there and I'm closing this for now.


suggestion: create release

In that project, if we add releases after changes it would be great.
Or if no just close this release option 😢
What about you?

Hello @priyankarpal , we will think of it.


feat: Add support for DateInterval conversion and verification

Implemented a new DateIntervalConverter to handle DateInterval serialization with context-aware scrubbing. Updated the verification initialization to include the new converter and added tests to validate DateInterval serialization.

@dotnet-policy-service agree company="KeepFlying Pvt Ltd"

Fixes #286

thanks. will deploy now


updating the required torch version

Hi, thanks for the great stuff!
Is there any plan to update the torch version (from ==1.10 to anything newer), or relax it?

Hi @keunwoochoi ,
Thanks for your interest.
Torch >1.10 is yet fully tested. And as observed by ourselves and other users (https://github.com/salesforce/LAVIS/issues/40), some newest torch version introduces incompatible changes.
For better reproduce our results as reported, you may want to stick with torch 1.10. If you'd like to leverage new torch versions, you may try manual updates
Thanks.

This is now relaxed to torch>=1.10 in https://github.com/salesforce/LAVIS/commit/cd4db98c93d87abe926c9c0e21093f4c56f856de.
However, please be aware the for best reproducing the training for ALBEF and BLIP models, torch=1.10 is recommended.


haskell.packages.ghc921: jailbreak splitmix

Motivation for this change
splitmix already has a commit that fixes the version bounds for GHC 9.2.1 but has not had a release since then.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Nice! I think this accounts for 90% of the problems I've seen so far…

Nice! I think this accounts for 90% of the problems I've seen so far…

You'll see random accounts for 90% of the problems now ;)


arte.tv: "Die Wege des Herrn" - only one language

I successfully downloaded the following URL:
https://www.arte.tv/de/videos/062202-001-A/die-wege-des-herrn-staffel-1-1-10/
Content is offered with various languages and subtitles. Couldn't find out how to download the german synchronized version.

Hi, please try this version:
mediathek.host.zip
Don't forget to unzip it first. It should prefer the german source.

Thanks :), download with german track confirmed. Genreal question would be, how to let the user select?

Currently the only way to change the language is to change the url from de to fr or using the language chooser on the arte homepage.
The new version v0.4.1 now contains a fix for your problem.


SRT does not work

Hello
I use this tool when start RE2,but it doesn't work.All data  never update.
What shoud I do?

Is this with the Steam version, non-pirated?

It's Steam version.

The 1.4 version is work.


How can I remove "? Dir =" from the URL

How can I remove "? Dir =" from the URL
For example: https://domain.com?Dir=A/B/C
Display as: https://domain.com/A/B/C
The latter will show 403 Forbidden
For example: https://demo.directorylister.com/tests/Bootstrap/

You may be able to accomplish this via your web server's configuration. That is by using Apache mod_rewrite or NGINX Rewrite Rules. This is not something Directory Lister can do out of the box.

Need this feature? Check out my modification. Its  very easy.
flatblowfish/modify-directory-lister


Add blob support for GitHub API

Adding Git Blob support.
Also adds a configuration to automatically strip the authentication tokens from cassettes when they're created.

Coverage increased (+0.08%) to 91.102% when pulling bbaf8e30a33f5888c9eeecc1a90dedf33932608c on azizshamim:git-data-wrapper into ff1c4d58515ccf5527cb87a00fb461d0815c8dfc on edgurgel:master.

Coverage increased (+0.08%) to 91.102% when pulling bbaf8e30a33f5888c9eeecc1a90dedf33932608c on azizshamim:git-data-wrapper into ff1c4d58515ccf5527cb87a00fb461d0815c8dfc on edgurgel:master.

thanks @azizshamim


Incomplete documentation for native retargeting

Take a look at this page.
https://docs.microsoft.com/en-us/cpp/porting/use-native-multi-targeting
It ends with "When these products are installed, the" which is incomplete.

Good catch. Closing as fixed with PR #134.


auto submitted in  when changing date

When I use this Calendar component in <form> and select a date, it is auto submitted.
I looked at the html and found the date is <button> and no type speicified.
The default behavior of <button> is submit in HTML5, so it seems to happen.
Please put the type in <button> tag.
This should be good enough for this issue.
<button type="button">

Thanks for reporting the issue. I will take a look at this during this weekend.

Hey @hackcomj We've pushed a fixed addressing your issue.
It'd be great if you could give it a try. The latest version is 0.0.0-beta.16.
Thanks.

@tusharf5 It's working well. Thank you very much.
If you change in npm as well, it will help many people. (It is still 0.0.0-beta.8)
Thank you again.


Delete endpoint implementation with delete_tokens configuration

Description
Add a DELETE endpoint guarded by a separate auth token array.
Motivation and Context
Closes #127
How Has This Been Tested?
Wrote a test case and tested manually.
Changelog Entry
### Added

- Change `Config.get_tokens` to `Config.get_auth_tokens` and update invocations.
- Add delete_tokens array to config and `get_delete_tokens` method to Config for retrieving delete tokens.
- Add DELETE endpoint that will remove files if a delete token is set and authorized by the request.
- Test for new DELETE endpoint
- Add DELETE_TOKEN_ENV for setting the delete tokens array from an ENV var.

-->
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[ ] Refactor (refactoring production code)
[ ] Other

Checklist:

[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.
[x] I have formatted the code with rustfmt.
[x] I checked the lints with clippy.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report
Patch coverage is 92.30% of modified lines.
:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.

Files Changed
Coverage

src/config.rs
83.33%

src/server.rs
95.00%

:loudspeaker: Thoughts on this report? Let us know!.

I was thinking a bit more. Maybe we should create an enum for token_type: auth, delete. And then use just one function get_tokens(&self, enum). I have no idea, whether this is better code or me just trying to optimize without any gain.
For me thia PR is perfecctly fine. Let's see what @orhun has to say.

@tessus I don't know what would be the benefit of adding an enum but we can go with that for the sake of having a single function for retrieving the tokens. @ajbdev can you also make that change?

I came up with this "brilliant" idea, thus I'll take care of it. ;-)

@orhun all done!

@orhun It seems that gh had a connection issue during 2 tests. Any chance you can rerun the action?
If I run the tests locally, everything is fine.


replace all todo! macros in rpc crate with rpc error responses

Describe the feature
Currently all missing rpc handlers will panic if being called:
https://github.com/paradigmxyz/reth/blob/c5bc272057b473337191de50e7f86d07cf93c5a1/crates/net/rpc/src/trace.rs#L19-L26
we will implement them gradually, and don't want the node to panic if an umimplemented handler is called. Therefore the todo!s should be replaced by an Err(InternalServerError("unimplemented")) instead.
Additional context
No response

I'd like to take this one. And I'm interested in implementing RPC step by step. :)


Fully featured crawler

Uses regex to respect the robots.txt file and uses a random interval between 1 and 5 seconds to ensure DoS protection services don't kick in.

@someshkar add a gitignore to your thing and exclude the env directory.
https://github.com/github/gitignore/blob/master/Python.gitignore
Use that

Otherwise, I think the code is pretty legit :+1:
If you want more bonus points then I'd recommend keeping a track of visited domains and adding the sleep only if the same domain has been hit > DOMAIN_THRESHOLD times
Be careful of the distinction b/w domain and webpages

Alright, I've added that .gitignore file and implemented the domain hit threshold.

@someshkar couple of things:
Do git rm -r env and commit
Also, your crawler fails on my website http://palashtaneja.com even though other crawlers get the URLs on it.
Traceback (most recent call last):
File "main.py", line 8, in
title, links_and_data, current_time, main_source = c.crawl()
File "/Users/palashtaneja/Github/OSMarathon/open-source-marathon/basic/crawler.py", line 45, in crawl
domain = str(re.findall(r'://(.*?)/', link)[0])
IndexError: list index out of range

I removed the env, as for the error, @sayamkanwar was running into a similar issue on his machine for whatever reason with my basic parser, while it worked flawlessly on my machine.

Turns out this error started showing up on my machine too, and that was because my regex wouldn't match the root domain (https://techcrunch.com) and would throw an index out of range error. I've fixed it now.


Cognito

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.

Codecov Report

Merging #848 (a4e7943) into main (2dc66ee) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #848   +/-   ##
=======================================
  Coverage   91.69%   91.69%
=======================================
  Files         102      102
  Lines        4128     4128
  Branches     1174     1174
=======================================
  Hits         3785     3785
  Misses        324      324
  Partials       19       19

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 2dc66ee...a4e7943. Read the comment docs.


Support for Objecttype: LIMU/ DOCU  "Documentation"

Hey,
as most of know, we can extend custom classes and programs with a documentation (Transaction se61).
Unfortunately, this type of object is not (yet?) supported. Is there any specific reason why? Do you think it could technically be implemented?
I would love to git these objects as well.
Thanks,
fischmir

abapGit bases it objects on R3TR level, the LIMU objects are included in the R3TR
which R3TR object type are you using?

Not sure if I got your question right. This is what I see in an transport request:

creating a general text via SE61 is a R3TR DOCT object(which is supported), which object type does the LIMU DOCU listed above belong to?

I am guessing it's this:

I did a quick test and also wasn't able to get the documents serialized. The logic for it retrieves the lines and then discards them because the DOKSTATE isn't R (which seems to be saved but not actived) but A (saved and activated) for some reason.

Since we only serialize active versions this needs to be changed to 'A' I think:
https://github.com/abapGit/abapGit/blob/ce677e449ff4258dd00da720d6658fbfd65a5903/src/objects/oo/zcl_abapgit_oo_base.clas.abap#L169

That looks promising. Would love to test it soon.

Works perfectly, thank you!


How to combine the nenural net log-softmax outputs and fsa

Hello, I am reading train.py and decode.py. For me, It is difficult to know how to combine the nenural net log-softmax outputs and fsa. Could you provide some papers or description about that to help me understand, thanks. Here is the codes what I don't understand:
dense_fsa_vec = k2.DenseFsaVec(nnet_output, supervision_segments)

You can find some descriptions about it by visiting the following two links:

https://github.com/k2-fsa/k2/blob/2dbb3e09b152fcf98354c946baa271e5b57c8321/k2/csrc/fsa.h#L114

/*
  Vector of FSAs that actually will come from neural net log-softmax outputs (or
  similar).
  Conceptually this is a 3-dimensional tensor of log-probs with the second
  dimension ragged, i.e.  the shape would be [ num_fsas, None, num_symbols+1 ],
  e.g. if this were a TF ragged tensor.  The indexing would be
  [fsa_idx,t,symbol+1], where the "+1" after the symbol is so that we have
  somewhere to put the output for symbol == -1 (remember, -1 is kFinalSymbol,
  used on the last frame).
  Also, if a particular FSA has T frames of neural net output, we actually
  have T+1 potential indexes, 0 through T, so there is space for the terminating
  final-symbol on frame T.  (On the last frame, the final symbol has
  logprob=0, the others have logprob=-inf).
 */

https://github.com/k2-fsa/k2/blob/2dbb3e09b152fcf98354c946baa271e5b57c8321/k2/python/k2/dense_fsa_vec.py#L15

class DenseFsaVec(object):

    def __init__(self, log_probs: torch.Tensor,
                 supervision_segments: torch.Tensor) -> None:
        '''Construct a DenseFsaVec from neural net log-softmax outputs.
        Args:
          log_probs:
            A 3-D tensor of dtype ``torch.float32`` with shape ``(N, T, C)``,
            where ``N`` is the number of sequences, ``T`` the maximum input
            length, and ``C`` the number of output classes.
          supervision_segments:
            A 2-D **CPU** tensor of dtype ``torch.int32`` with 3 columns.
            Each row contains information for a supervision segment. Column 0
            is the ``sequence_index`` indicating which sequence this segment
            comes from; column 1 specifies the ``start_frame`` of this segment
            within the sequence; column 2 contains the ``duration`` of this
            segment.
            Note:
              - ``0 < start_frame + duration <= T``
              - ``0 <= start_frame < T``
              - ``duration > 0``
        '''

Mm, thanks , I have seen these two material. But it is too little for me. Could you provide other material?

I am writing tutorials for k2. Please just wait for a few days.

@Curisan just add some notes in case you are eager to learn about this before fangjun's documentation.
When we train or decode, usually we feed data into nnet model batch by batch, we prepare batch data with K2SpeechRecognitionIterableDataset in lhotse
https://github.com/lhotse-speech/lhotse/blob/08c31c3bd2711d4b6c614d64a1d3c26abb892a37/lhotse/dataset/speech_recognition.py#L86-L94
You can see that a batch is a few of  Cuts and each Cut may have multiple supervisions, so the question we have now is: after feeding feature (N, T, C_feature) into nnet and getting nnet_output (N, T, C_nnet_output), we need to know which part of nnet_output corresponds to each supervision, right? This is exactly what k2.DenseFsaVec(nnet_output, supervision_segments) does.  As supervision_segments gives the info of seq_idx (corresponds to N in nnet_output), start_frames and num_frames (corresponds to T in nnet_output), then we can easily get the part of nnet_output for each supervision with those information in DenseFsaVec (of course if we do subsampling in model like tdnn, we need to do the same subsampling for start_frame and num_frames as well.)
Then in DenseFsaVec, for each supervision (with the corresponding part of nnet_ouput), we'll create an DenseFsa (Hopefully you have understood the format of DenseFsa with the documentation in fsa.h, but you can also view it as a normal Fsa, they are equivalent from the perpective of Fsa concept). So next step we'll call intersect_(pruned) to intersect the DenseFsa with the decoding_grah to get the lattice, then get the tot_scores or best_path for training or decoding.
You may want to check test code in k2/python/tests or test code in lhotse to get know well about the data format. Feel free to ping us if there's any question.

Thank you very much.

@Curisan
There is some documentation about dense fsa vector available at
https://k3.readthedocs.io/en/latest/core_concepts/index.html
Please let us know whether it is clear or need more clarification.

Great!

Could you provide some papers or description about that to help me understand

Here is a paper I just found that is relevant about it:

Generating exact lattices in the WFST framework, https://www.danielpovey.com/files/2012_icassp_lattices.pdf

Figure 1 from the paper shows what DenseFsaVec looks like. It is called "the search graph of the utterance" in the paper.

Could you provide some papers or description about that to help me understand

Here is a paper I just found that is relevant about it:

Generating exact lattices in the WFST framework, https://www.danielpovey.com/files/2012_icassp_lattices.pdf

Figure 1 from the paper shows what DenseFsaVec looks like. It is called "the search graph of the utterance" in the paper.

In that paper the DenseFsaVec would be " Acceptor U describing the acoustic
scores of an utterance"
In k2, so far we are dealing only with state-level lattices, not
determinized lattices.
The "search graph of the utterance" (S = U o HCLG) is the result of calling
IntersectDensePruned().
On Sun, Jan 24, 2021 at 10:20 PM Fangjun Kuang<EMAIL_ADDRESS>wrote:

Could you provide some papers or description about that to help me
understand
Here is a paper I just found that is relevant about it:

Generating exact lattices in the WFST framework,
https://www.danielpovey.com/files/2012_icassp_lattices.pdf

Figure 1 from the paper shows what DenseFsaVec looks like. It is called
"the search graph of the utterance" in the paper.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/issues/44#issuecomment-766355840, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO6C4TZGSIMM6HDWVODS3QUD3ANCNFSM4U5LJLYA
.

In that paper the DenseFsaVec would be " Acceptor U describing the acoustic
scores of an utterance"
In k2, so far we are dealing only with state-level lattices, not
determinized lattices.
The "search graph of the utterance" (S = U o HCLG) is the result of calling
IntersectDensePruned().
On Sun, Jan 24, 2021 at 10:20 PM Fangjun Kuang<EMAIL_ADDRESS>wrote:

Could you provide some papers or description about that to help me
understand
Here is a paper I just found that is relevant about it:

Generating exact lattices in the WFST framework,
https://www.danielpovey.com/files/2012_icassp_lattices.pdf

Figure 1 from the paper shows what DenseFsaVec looks like. It is called
"the search graph of the utterance" in the paper.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/snowfall/issues/44#issuecomment-766355840, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO6C4TZGSIMM6HDWVODS3QUD3ANCNFSM4U5LJLYA
.

I see. Thanks.

I see. Thanks.


Docker compose fails to build the project end to end on "Rounding Decimals" error

Describe the bug
The docker compose workflow described in the quickstart doesn't work.
To Reproduce
Steps to reproduce the behavior:
$ docker-compose up --build
j40_data_pipeline_1 | 2022-01-06 08:47:25,574 [data_pipeline.etl.score.etl_score_post] INFO     Rounding Decimals
j40_data_pipeline_1 | Traceback (most recent call last):
j40_data_pipeline_1 |   File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
j40_data_pipeline_1 |     return _run_code(code, main_globals, None,
j40_data_pipeline_1 |   File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
j40_data_pipeline_1 |     exec(code, run_globals)
j40_data_pipeline_1 |   File "/data-pipeline/data_pipeline/application.py", line 283, in <module>
j40_data_pipeline_1 |     cli()
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1128, in __call__
j40_data_pipeline_1 |     return self.main(*args, **kwargs)
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1053, in main
j40_data_pipeline_1 |     rv = self.invoke(ctx)
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1659, in invoke
j40_data_pipeline_1 |     return _process_result(sub_ctx.command.invoke(sub_ctx))
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 1395, in invoke
j40_data_pipeline_1 |     return ctx.invoke(self.callback, **ctx.params)
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/click/core.py", line 754, in invoke
j40_data_pipeline_1 |     return __callback(*args, **kwargs)
j40_data_pipeline_1 |   File "/data-pipeline/data_pipeline/application.py", line 266, in data_full_run
j40_data_pipeline_1 |     score_post(data_source)
j40_data_pipeline_1 |   File "/data-pipeline/data_pipeline/etl/runner.py", line 102, in score_post
j40_data_pipeline_1 |     score_post.transform()
j40_data_pipeline_1 |   File "/data-pipeline/data_pipeline/etl/score/etl_score_post.py", line 258, in transform
j40_data_pipeline_1 |     self.output_score_tiles_df = self._create_tile_data(
j40_data_pipeline_1 |   File "/data-pipeline/data_pipeline/etl/score/etl_score_post.py", line 211, in _create_tile_data
j40_data_pipeline_1 |     score_tiles[constants.TILES_SCORE_FLOAT_COLUMNS] = score_tiles[
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/pandas/core/frame.py", line 3600, in __setitem__
j40_data_pipeline_1 |     self._setitem_array(key, value)
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/pandas/core/frame.py", line 3641, in _setitem_array
j40_data_pipeline_1 |     self[k1] = value[k2]
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/pandas/core/frame.py", line 3612, in __setitem__
j40_data_pipeline_1 |     self._set_item(key, value)
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/pandas/core/frame.py", line 3797, in _set_item
j40_data_pipeline_1 |     self._set_item_mgr(key, value)
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/pandas/core/frame.py", line 3762, in _set_item_mgr
j40_data_pipeline_1 |     self._check_setitem_copy(stacklevel=5)
j40_data_pipeline_1 |   File "/usr/local/lib/python3.8/dist-packages/pandas/core/generic.py", line 3933, in _check_setitem_copy
j40_data_pipeline_1 |     raise com.SettingWithCopyError(t)
j40_data_pipeline_1 | pandas.core.common.SettingWithCopyError:
j40_data_pipeline_1 | A value is trying to be set on a copy of a slice from a DataFrame.
j40_data_pipeline_1 | Try using .loc[row_indexer,col_indexer] = value instead
j40_data_pipeline_1 |
j40_data_pipeline_1 | See the caveats in the documentation: https://pandas.pydata.org/pandas-docs/stable/user_guide/indexing.html#returning-a-view-versus-a-copy
j40_data_pipeline_1 exited with code 1

Expected behavior
The quickstart can create and run the application.

@saran-ahluwalia Assigned you because I think this commit introduced the bug.
@esfoobar-usds I'm assuming that our github actions resources aren't enough to run this end to end test, but maybe there's something that could be done to catch cases like this. Do you think this is worth looking into? I can file a follow up issue if so.

Based on independently running score-post and score-run locally does not trigger this exception. @esfoobar-usds was able to corroborate this result. @esfoobar-usds did eliminate the SettingWithCopyWarning by creating a new reference pointer to the series object further up in the call chain.

Just ran this against my branch and it runs successfully.

I think this is still broken in main, maybe you accidentally hit "comment and close".

This was unrelated to @saran-ahluwalia's change. The current theory is that this was a pandas warning that turned into an error when we updated dependencies (because that's the way pandas gradually phases out usage).
@esfoobar-usds has a fix for the underlying issue on his branch (we've confirmed that works), so once his branch is merged in, we can close this. The branch is: https://github.com/usds/justice40-tool/tree/esfoobar-usds/1062-implement-changes-export-files


No notifications in other WMs

There are no notifications in the notification window. If i manually run scripts/logger.py subscribe I only see the quote.
Dunst works fine and displays messages.

Found out the issue to be that the logger.py scripts needs to be initialized. So i have to run this line on startup:
$XDG_CONFIG_HOME/eww/scripts/logger.py init on startup, which in i3wm looks like this:
exec --no-startup-id $XDG_CONFIG_HOME/eww/scripts/logger.py init

Yes


there should be a list of different tags in the read.me, as well as a standardised way of ordering them

Technically folks can add any tags they want at the moment.
However the website, app and cli will only show these tags
https://github.com/studiorack/studiorack-core/blob/main/src/config-defaults.ts#L276
I could make that more explicit though. So agree with this request.

This commit updates the README with a list of tags which should be used:
https://github.com/studiorack/studiorack-registry/commit/32c90809f11e6c8d7ecf3bc7d54982bb9a856e55


not working

I type as shown html-  but instead I am getting:
(<)html-(>) (</) html- (>)
I am not getting the Bootstrap snippet.
html-min:
html-oi
is not working either. I do have Emmet so I don't know if this is causing the problem.
Please help. Thanks

Me too. When I begin my index page I want to make Bootstrap 3 skeleton with typing html- in atom then I hit tab, but nothing happened.

I have not received any updated fixes for this yet, I am hoping someone can explain.

i'm entering in html- and i see a pop-up but before i can select anything - it goes away.  same for the rest of the tags.  i can't even click on them either- the pop-up just goes away.

same ^

I have the same problem -- but it was solved by quitting Atom (and restarting it).

Had the same problems as above people, it seems  just like sublime all you need to to is restart and it works fine

same problem

This is what worked for me, having Emmet installed along with atom-bootstrap3:

Restart Atom after installing atom-bootstrap3;
Type html-;
Press RETURN instead of TAB.


crash at end of workout

crash on end of workout.

fixed; was attempting to set identical keys on an ExpandoObject due to lazy copy and paste; also forgot to set CurrentWorkoutActivity to null on completion of a Workout.

fixed; was attempting to set identical keys on an ExpandoObject due to lazy copy and paste; also forgot to set CurrentWorkoutActivity to null on completion of a Workout.


Game description under readme section

Describe the bug ✍
Hii adding the game description can enhance the readme section and also allow users to get an idea on how the game works
Provide step by step information reproduce the bug 📄
I would like to add step by step procedure on enjoying the game and wish to to contribute to this project under gssoc'24
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created seprate chennals for all projects...
Thank you :)

I'm incredibly enthusiastic about tackling the "Game description under readme section." issue for GSoC 2024! My skills in HTML , CSS , JS perfectly align with this project's needs. Is there a possibility of me being assigned this issue?

Describe the bug ✍
Hii adding the game description can enhance the readme section and also allow users to get an idea on how the game works
Provide step by step information reproduce the bug 📄
I wish to to contribute to this project under gssoc'24
Select program in which you are contributing
GSSoC24
Code of Conduct
I follow CONTRIBUTING GUIDELINE of this project.


Create HDF5 Example Dataset w/ example notebook

@tonyfast created an example dataset that is ~764MB zipped/1.3GB unzipped.
It is available here:
https://quansight.nyc3.digitaloceanspaces.com/datasets/tracts.zip


Switch Role for one profile

I have many projects with multiple roles: PM, mobile, web, data/AI but I only want to use one Github profile-linked account.
How can I do that, I don't want to write web functions but a snippet of data/AI is just coming in the python file :)

I am not sure if I understand your request. Can you please elaborate what do you mean by profile and roles and where do you see them in VS Code?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[PRODUCT]: Soil Moisture - NLDAS and SMAP

Links from @FHanbali
SMAP

https://nsidc.org/data/smap/smap-data.html
https://nsidc.org/data/SPL3SMA/versions/3
https://smap.jpl.nasa.gov/

NLDAS

https://disc.sci.gsfc.nasa.gov/datasets?keywords=NLDAS
ftp://ldas.ncep.noaa.gov/

Soil Moisture Data Sets: Overview & Comparison Tables

https://climatedataguide.ucar.edu/climate-data/soil-moisture-data-sets-overview-comparison-tables

Hello @FHanbali .  I think we have a blocker 🚫 on this task related to data access. Can you help unstick this?

Too many links and products to sift through.  I'm leaning towards the SMAP products, but I need more eyes on this decision.
https://smap.jpl.nasa.gov/data/

Thanks @adamscarberry, looping-in @FHanbali.  I think we need additional information on which specific products are to be added to Cumulus (specific links, products, bands, etc.). Stay tuned. I'm setting the 🛑Blocked tag pending additional information.

@FHanbali We need to get more resolution on what actually we want to pull from this dataset, essentially we need to figure out who has the need, what that need is, and get it a little more fleshed out.

Looked into the SMAP data more.  There are 4 levels.  Level 2 has data from a single pass.  Level three has combined data from a single day and level 4 incorporates modeling to fill in gaps in the data.  Issues with the data sets.
Level 3 data
https://nsidc.org/data/spl3smp_e/versions/5
Level 3 data has large gaps as it only includes data from the satellite pass that day.   It does have two runs an AM and PM but combined there are still data gaps.  Data is available for previous day.  So on 8/25/2023 data is available for 8/24/2023

Level 4 data
https://nsidc.org/data/spl4smgp/versions/7
Level 4 data is available 3 days in the past so on 8/25/2023 data is avaiable for 8/22/2023.  but it provides an entire coverage

I believe that Level 4 data would be most useful and will start to gather that information.


MAX_HTTP_OUTPUT_BUFFER leads to invalid JSON when exceeded

MAX_HTTP_OUTPUT_BUFFER can cut off the JSON string and lead to invalid parsing even if the request is valid.

Fixed by 6847069 by getting content header length. Now it just returns NULL when the malloc fails and doesn't return half a JSON so cJSON shouldn't ever fail parsing it.


Error missing Jenkins config

Hi, Can somebody point me, what  i am missing here. Thanks in advance.

Here is my yaml file
name: trigger jenkins job
on:
  pull_request:
    branches: [ master ]

jobs:
  build:
    name: Build
    runs-on: ubuntu-latest
    steps:
    - name: trigger single Job
      uses: appleboy/jenkins-action@master
      with:
        url: "http://<IP_ADDRESS>:8080"
        user: "jenkins"
        token: ${{ secrets.jenkins }}
        job: "abc"

make sure secrets.jenkins is available for your repo. Check Github secret configuration.


fix: strip trailing slashes in BaseService.set_service_url

This commit modifies the BaseService.set_service_url()
function so that it removes any trailing slashes.
Previously, a service URL containing a trailing slash
(e.g. "https://myserver.com/api/v1/") would result in a
request URL that contains two slashes between the service URL
and the operation path (e.g. "https://myserver.com/api/v1//myoperationpath"),
and some server implementations might fail to process such
a request properly.
With the changes in this commit, a request URL of
"https://myserver.com/api/v1/myoperationpath" will be used instead.

Reminder to myself:  before we merge this, we need to test this version of the core with existing generated unit tests.

I also did a test with the platform-services python SDK where I ran the global-search integration test with and without the python core fix.
The service URL I used in each case was: https://api.global-search-tagging.cloud.ibm.com/.
In the "before" scenario (without the python core fix), I see this in the http debug messages:
send: b'POST //v3/resources/search?limit=1 HTTP/1.1\r\nHost: api.global-search-tagging.cloud.ibm.com

so the path value passed in the HTTP request message does in fact have a leading double-slash (the global-search server still processes this request just fine, but not all servers behave this way).
In the "after" scenario (with the python core fix), I see this in the http debug messages:
send: b'POST /v3/resources/search?limit=1 HTTP/1.1\r\nHost: api.global-search-tagging.cloud.ibm.com

so the path value now has just a single slash.

:tada: This PR is included in version 3.13.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


"Breaking Change (Cross-Version)" only reports single line for "incompatible properties"

Should have been fixed by:
https://github.com/Azure/openapi-diff/pull/316
https://dev.azure.com/devdiv/DevDiv/_git/openapi-alps/pullrequest/541412
However we are still only getting the short error:
https://github.com/Azure/azure-rest-api-specs-pr/pull/17989/checks?check_run_id=24170830169

Fix: https://dev.azure.com/devdiv/DevDiv/_git/openapi-alps/pullrequest/546153


WIP: Update documentation for scoped labels

update documentation to include an overview of the yaml label template that is part of PR #22976
Signed-off-by: Jon Roadley-Battin<EMAIL_ADDRESS>

I've converted it to draft now as long as the other PR hasn't been merged.
Feel free to de-WIP this PR once the other one has been merged.

is this still needed considering https://github.com/go-gitea/gitea/pull/23309

is this still needed considering #23309

Yes it is as https://docs.gitea.io/en-us/labels/ is more for as an overview on labels not how to customize

I was just about to respond, but yes I agree this PR is still helpful and covers customization (and documenting the legacy version)

:trumpet: :robot:


Promote Gateway for Mesh (GAMMA) to Standard

What type of PR is this?
/kind gep
What this PR does / why we need it:
This PR raises Gateway API for Mesh to "Standard" channel.
What is included:

GEP-1426 (Routes binding to service) and GEP-1686 (conformance tests for mesh)
Updated various docs to declare stable.

Followup needed:

Blog post announcement

What is not included:

Changes to any tests; we already have existing conformance for mesh, and the feature is already called MeshCoreFeatures so I don't think we need to change anything there. In the future, we may add experimental features to Mesh, at which point we can make a new set of features like MeshExperimentalFeatures or similar.
Changes to any Go/CRD APIs beyond docs. The Gateway for Mesh work did not introduce any API changes. Instead it just introduced new ways to use the existing APIs with corresponding documentation changes to remove the 'Experimental' signifier.

Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

The Gateway API for Mesh ("GAMMA") support has been promoted to the Standard Channel

I think it's time!
/approve

Hit it! 🙂
/approve

Although I definitely still want to see continued evolution and growth of this pattern, I agree that it's time to formally graduate this pattern to GA to reflect the widespread implementation and overall stability of this pattern. Thanks to everyone that helped make this a reality!
/approve

/lgtm

/hold cancel
looks like this was added on the pr creation and not intended

/retest-required

We usually add a hold for GEP or similar scope changes now, just to avoid accidental merges (since if someone with approver makes a PR, it's automatically approved). This happened a few times before 😨 .
Looks like this needs a make generate before the tests will pass.

Looks like this needs a make generate before the tests will pass.

Should to be good this time!

/hold cancel

Woo!! :tada: :tada: :tada: :tada: :tada: :tada: :tada:
Thanks everyone for the years of hard work on getting this passed the finish line!

Thanks for writing the PR on this one, @howardjohn! 🙂


Add model compression doc

PR types

New features
PR changes

Docs
Describe

Add model compression doc
Reviewers could preview the doc by this link

@LiuChiachi How about add Mengsi to review your doc together?

@LiuChiachi How about add Mengsi to review your doc together?

Thanks. I invited her just now.

Could @chenxiaozeng  review my doc? Thanks


feat: improve semantic-release setup

Improve the semantic-release setup to make it easier to maintain.

Dropped node 8 support for node 10.
Switched from CircleCI to GitHub Actions for CI.

TODO: add node 12 to CI for test.

Can you consider adding to package.json.
"engines": {
    "node": "^10 || ^11"
  }

Can you consider adding to package.json?
"engines": {
    "node": "^10 || ^11"
  }

Done
https://github.com/prisma-labs/graphql-request/pull/164/commits/a2cd0083ad3888ca1dcd52e79b81818896ad5ce4
Also added a node matrix to the github action on push and pull_requests.
https://github.com/prisma-labs/graphql-request/pull/164/commits/b614631d4e0450795541be7c8f729c1aae8a0cd8
Not sure yet what's the best branch to get people to do pull request on?
Semantic Release has default branches for running the releases on. But ideally all the merged pull requests goes into a branch that doesn't run this release script. So perhaps a branch called develop or staging? Or is it fine to keep releasing on a pre-release e.g beta branch every pull request that is merged?

@andykenward :+1:  You and I have grate time extending this thing, but I don't think prisma team is thinking of merging anything :D. I'm not sure is the question you're asking for me, but I don't know.

@ultrox It was a general question to whoever finds this pull request.
Yeah, I am not sure if the maintainers will do anything, since I've seen issues closed #160 #122 in relation to maintaining this package. But as it is used by 72.1K projects on Github I thought it is worth a try. Plus I get to learn some different tooling.

CLOSES #128 #161

Thanks for this @andykenward but we'll be using https://github.com/prisma-labs/dripip. Feel free to take a shot at setting it up. I'll close this PR soon unless you express interest in repurposing to get dripip going. Let me know.

Hey @andykenward, actually, seeing as semantic-release is already in use on this project and seeing as I also want to go away from CircleCI which this PR does, I think we should just get this done, then I'll make the conversion to dripip later.
Please ping me when this is ready for review.

@jasonkuhrt FYI about GITHUB_TOKEN & semantic-release, read this note

Note: Automatically populated GITHUB_TOKEN cannot be used if branch protection is enabled for the target branch. It is not advised to mitigate this limitation by overriding an automatically populated GITHUB_TOKEN variable with a Personal Access Tokens, as it poses a security risk. Since Secret Variables are available for Workflows triggered by any branch, it becomes a potential vector of attack, where a Workflow triggered from a non-protected branch can expose and use a token with elevated permissions, yielding branch protection insignificant. One can use Personal Access Tokens in trusted environments, where all developers should have the ability to perform administrative actions in the given repository and branch protection is enabled solely for convenience purposes, to remind about required reviews or CI checks.

Thanks @andykenward!


Support for DE collector

It seems that this region isn't supported.

Sumologic recently released their own, officially supported provider. As such, I am not actively maintaining this one. Their efforts can be found on their github pages: https://github.com/SumoLogic/sumologic-terraform-provider


export-name throwing an error

Bug Report

tslint-microsoft-contrib version: latest
TSLint version: latest
TypeScript version: latest
Running TSLint via: (pick one) CLI

TypeScript code being linted
export * from "./MyClass";

with tslint.json configuration:
{
  "rules": {
    "export-name": true
  }
}

Actual behavior

➜  tinka-client git:(master) ✗ npm run lint

> @crazy-factory/ts-module-boilerplate@ lint /mnt/c/Users/ngautam/git/tinka-client
> tslint -t stylish --project tsconfig.json './src/**/*.ts'

The 'export-name' rule threw an error in '/mnt/c/Users/ngautam/git/tinka-client/src/index.ts':
TypeError: Cannot read property 'elements' of undefined
    at getExportsFromStatement (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint-microsoft-contrib/exportNameRule.js:97:26)
    at ExportNameWalker.visitSourceFile (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint-microsoft-contrib/exportNameRule.js:112:35)
    at ExportNameWalker.SyntaxWalker.visitNode (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint/lib/language/walker/syntaxWalker.js:482:22)
    at ExportNameWalker.SyntaxWalker.walk (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint/lib/language/walker/syntaxWalker.js:24:14)
    at Rule.AbstractRule.applyWithWalker (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint/lib/language/rule/abstractRule.js:31:16)
    at Rule.apply (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint-microsoft-contrib/exportNameRule.js:27:21)
    at Linter.applyRule (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint/lib/linter.js:209:29)
    at /mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint/lib/linter.js:150:85
    at Object.flatMap (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint/lib/utils.js:160:29)
    at Linter.getAllFailures (/mnt/c/Users/ngautam/git/tinka-client/node_modules/tslint/lib/linter.js:150:32)

Expected behavior
tslint to pass
Did I do something wrong? My project is almost empty, I'm starting from scratch, and it used to be okay before.
I'm sure it's because of export-name because it says so on error message, and if I set the rule to false, it just passes.

here's my state of project, it's basically empty:

Looks like duplicate of #664.

woops, sorry about that 😄


v0: allow identifiers to start with a digit.

See https://github.com/rust-lang/rfcs/pull/2705.

Thanks!


[feature]Visualization sensor_msgs/NavSatFix message in rviz

New Feature
add OverlayGpsDiaplay and vislalize sensor_msgs/NavSatFix message in rviz by using Google Static Map API.

@hakuturu583 do you plan to add this feature or would you like that someone from Autoware team adds it? If the latter - can you provide some pointers to the documentation or how to do it?
I will for now close the issue, please re-open if you can provide more details.

@dejanpan Sorry for my late reply. I choose latter and I will add detailed documentation about my Pull Request. Please wait.
Thank you.

Please feel free to open this again @hakuturu583 if you plan to add this feature.


capability to compute dynamic stats given a query

should we leave open to do some refactor or close and reopen another issue?
This issue is basically done.


feat: occlusion spot add insert almost zero velocity option

Description
This pr does not change previous logic until prameters are not changed.
see this  for stable behavior

add occlusion spot insert almost zero velocity setting
this parameter is going to insert almost zero velocity which is defined at min_allowed_velocity
with this setting ego keep slow down until occlusion pass ego detection area

      insert_stop_point: true(false = insert safe velocity consider emergency brake )
      min_allowed_velocity: 1.0           # [m/s] minimum velocity allowed(not to stuck)

because ego velocity and acceleration can be wavy and it's a bit hard to define "not effective velocity area" stably I add parameters for pass judge option

      consider_pass_judge: true             # [-] whether to consider pass judge for occlusion spot(detection area length)

Related links

Tests performed

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@taikitanaka3 Could you update the status of this PR? :pray:

@kenji-miyake
I can close this PR for this change was merge at another PR


Downgrade Elasticsearch client to 2.x

While we are still using Elasticsearch 2.x

Codecov Report

Merging #192 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #192   +/-   ##
========================================
  Coverage    95.66%   95.66%
========================================
  Files           41       41
  Lines         1384     1384
  Branches        85       85
========================================
  Hits          1324     1324
  Misses          47       47
  Partials        13       13

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 9989a82...a926076. Read the comment docs.


Upgrade GCC support

Codecov Report

Merging #6 (ed6bc6a) into main (a89507d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   74.05%   74.05%
=======================================
  Files          40       40
  Lines        7519     7519
=======================================
  Hits         5568     5568
  Misses       1951     1951

Flag
Coverage Δ

projmgr-cov
86.90% <ø> (ø)

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

Impacted Files
Coverage Δ

tools/projmgr/src/ProjMgrWorker.cpp
87.50% <ø> (ø)

Codecov Report

Merging #6 (ed6bc6a) into main (a89507d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   74.05%   74.05%
=======================================
  Files          40       40
  Lines        7519     7519
=======================================
  Hits         5568     5568
  Misses       1951     1951

Flag
Coverage Δ

buildmgr-cov
86.05% <ø> (ø)

projmgr-cov
86.90% <ø> (ø)

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

Impacted Files
Coverage Δ

tools/projmgr/src/ProjMgrWorker.cpp
87.50% <ø> (ø)


Restructuring of the basis module

Hi everyone!
Here a couple of notes on the PR
Summary of the content
I was able to start the PR for the basis module restructuring just in time.
The PR contents are quite a lot:

`neurostatslib.basis' module have been completely restructured, while it contains the same basis function that are in main (RaisedCosineBasisLog/Linear, OrthExponentialBasis, MSplineBasis)
testing the basis module in test/test_basis.py (basis class instantiation, expected input/output, and checked that exception raised when the input did not conform the method requirements, test that all the non-abstract classes implement the abstract methods).
Added Developer Notes to the documentation with an introduction page and a note on the basis module
Added a couple of example scripts in 'docs/examples' that show the user how to define, combine and plot the basis functions
Added a plugin that renders Latex in markdown (I wrote a couple of equation in the example scripts.

Additional comments
It looks like mkdocstrings does not fully support the Numpy docstrings format yet, see here. I don't like the rendering of the documentation as it is now. I suggest to change to one of the supported style now that the number of docstrings is relatively small.

Broader point about the memory warning in the basis function that I pulled out of a smaller comment: in general, like I think I've mentioned, I'm uncomfortable with the basis checking the approximate memory usage against some limit and raising an exception if it goes above it. I'm not aware of another scientific python package that implements this check? I.e., you can easily create a torch tensor, numpy array, or xarray that's too big for memory, and I'm assuming there's a good reason for that. My guess is that this step could cause it to break in weird ways, and it's not really doing what the user wants --- they'd want to make sure their whole analysis pipeline (or at least the GLM portion that's written in our package) doesn't exceed memory (rather than just the basis matrix), but there's no way we can guarantee that.
It's ultimately up to you, but I think we should take the fact that this doesn't exist in other scientific packages, despite being easy to approximate and relevant, as a strong warning sign (happy to be proven wrong if this does exist somewhere!)

* for your tests, not really sure what using the `self.cls` syntax gets you -- why not just explicitly use the name of the basis class?

self.cls is used in the meta-test to make sure that every concrete class has its own Test class implemented.

I'm still seeing check_input_number in basis.py, is that what you intended?

Check input number check that an N-inputs are passed to N-dim basis, so that's a check we must provide


Term(d::AbstractDecapode) should avoid expanding operators

That way we don't mangle nice decapodes by converting them to SSA.

We should delete line 6 from src/colanguage.jl, which expands all operators.
Then, in the logic handling op1s, we should check if the function f is a vector of symbols, and use AppCirc1 if so.
This could be handled by checking the type explicitly, or refactoring to use multiple dispatch.

@quffaro, we need to bump this priority up.


ix unbalanced nb_metachannel changes.

Otherwise DefaultTransforms on inverse pass generates different list than on meta pass.

I think this is not needed: https://github.com/libjxl/libjxl/pull/373 does the same thing.


automatically decide if .sock file is abandoned; do not depend on user to delete it

On Linux hosts, UNIX socket paths provide for communication between the CLI and hardshare daemons.
The command hardshare terminate will cleanly delete the file. In practice, processes can be interrupted, resulting in abandoned .sock files, as a consequence of otherwise common activities, e.g., hardshare ad -d, then the owner reboots the host.
This issue is to automatically decide if the .sock file is abandoned and can be deleted. To help, it might be best to introduce more tracking files, e.g., a .pid file that stores the PID of the corresponding daemon.

This has not occurred recently, following a variety of improvements since this issue was opened. However, it still can occur in some (rare?) cases.
Also, this issue will be obsolete because I am working to support daemon control via TCP on localhost.

new hardshare client will be released later today. closing this issue because this issue is obsolete.


[macOS] upgrade php version to 8.1

Description
php-8.1 is considered  stable we must upgrade too, as composer now requires 8.1 strictly
Related issue: https://github.com/actions/virtual-environments-internal/issues/3030
Check list

[X] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run macos1015, macos11, macos12


Fix Gulp error

Steps to fix:

npm install --save-dev gulp
npm install node-sass --save-dev

https://github.com/sass/node-sass/issues/1579

This was fixed by updating npm


Add TKG Maven version check

This it a following issue after https://github.com/adoptium/TKG/issues/503 about check in docker and podman, we should check the maven version as well.

@sophiaxu0424 for future reference, please use keyword in PR to close the issue
see https://github.com/adoptium/TKG/pull/565#issuecomment-2158886707