Vulnerability count doesn't account for suppressed findings

Hi, I've installed the prebuilt WAR under apache-tomcat-8.0.33 on Windows 10 and everything looks okay, but when I try to add a dependency to my project or add a component I don't get any entries in the dropdowns.

When adding a component, the dropdown is also a text input field, so you can start typing. Once one component is added, the vendor, component name, and version fields will be populated. As you add more components, there will be more choices in the dropdown.

Ah, okay. Thanks!
So are the false positives that are displayed for, say, OpenSSL 1.0.2g caused by dependency-check?
Is there any way to filter them out?

Yes, this is caused by dependency-check and can be filtered out using a suppression file. Refer to https://jeremylong.github.io/DependencyCheck/general/suppression.html
Suppressions are global in Dependency-Track, meaning that it's currently not possible to suppress a finding for one application and not another one. The benefit is that you can suppress a finding and all applications with that dependency will now inherent that suppression.
it's been a while since I've worked on the 1.x codebase (the master branch is all 2.x and quite different), but I believe the suppression file goes into ~/.dependency-track directory.

Just to follow up for the purposes of anyone else reading this thread - on Windows the file to create is C:\Users\username\dependency-track\data\suppressions.xml.
I can see in the Tomcat output (and in dependency-check-report.html) that the suppressions are being correctly applied and the total as shown by the graph on the Dashboard is correctly excluding the suppressions, but the numbers shown next to the applications and as listed on the Vulnerabilities page do not exclude the suppressions.
Is there an easy way to fix this or do I need to edit the database by hand?
(Should I just be using the master branch and building it myself?!)

v2.0.0 (in the master branch) is currently in heavy development. I'll make sure this gets fixed in this version.

FWIW if I remove the component, restart Tomcat and add it again I get the correct number in the Applications pane, but sadly the Vunerabilities page is still not filtered.

(I also couldn't open the database using java -cp ../repo/com/h2database/h2/1.3.176/h2-1.3.176.jar org.h2.tools.Server -tcp -web as I get a "Unique index or primary key violation" error.)

FYI, I have branched the project to make the source tree more clear.

The only stable version (which can also be downloaded in a ready-to-deploy war) is 1.0.x.
The master and 1.0-stable branches are in sync and should provide an easy way to compile your own war resulting in a working application.
The 2.0-dev-springboot branch is unstable and has many issues. I do not intend to ever release this version. Just too many issues overall. This work was previously performed in the master branch but has since been reverted and separated to its own branch.
All new development is being done in the 3.0-dev branch and is a complete rewrite of Dependency-Track from the ground up using modern technologies and an API-first design. It also doesn't rely on Spring.


Duplicate longitude wrap

test_vcs_animate_projected_boxfill_polar.png is rotated because the new code sets the lon_0 (central meridian) to (-180 + 175) / 2 which is different than 0. I think the other polar plots are rotated because of the same reason.

Similarly test_vcs_animate_projected_meshfill_mollweide.png also appears to be affected by setting of lon_0.

@aashish24 @chaosphere2112 meshfill dataset varies from -40 to 361. It is not clear to me why the portion from -40 to 0 is missing from the plot. This may be a bug in proj4. It would be interesting to test this with the new proj4. Aashish, how is upgrading to the new proj4 going?

@aashish24 @doutriaux1 Any of you know why for sampleCurveGrid4.nc bounds_lat and bounds_lon have an additional parameter nvert = 4? I would have expected 2 bounds not 4.

I believe its because its a curvelinear grid and each cell requires four vertices.. but I could be wrong about it.

bounds_lat has 4 points as well. I would have expected 2 for lon and 2 for lat.

@aashish24 @doutriaux1 boxfill changes we see are because data range varies from -182.5 to 177.5 which results in an extra row on left and also the central meridian being moved from 0 to -2.5

@danlipsa nvert is the number of vertice per cell. So 4 makes sense here. Curvilinear grid.

@doutriaux1 Oh, I see. So each point can have a different lat/lon.

yes it can, it could be any shape too, like stars or anything example: https://www.myroms.org/links/newbad.png

@danlipsa you can check the values of lon and lat in https://github.com/UV-CDAT/uvcdat/issues/1465#issuecomment-133473257 as another example. And this is a data file where unfortunately the cell boundary data is missing (but basemap can still make a decent plot)

@aashish24 @jypeter that's another issue. Probably post release

@aashish24 @doutriaux1 Regarding the meshfill plots, it seems that sampleCurveGrid4.nc has very few points/cells below longitude -10 and all seem to be at high latitude.
This matches the projection we see with most values between lon -40 and 0 empty.
A side note is that it would be a worthwhile endeavor to upgrade the nc reader in ParaView to be able to read bounds. Then it would make it much easier/faster to debug uvdat on any customer's data.
I added a breakpoint in vcs2vtk.genGrid and printed a slice from data2 (which seems to be the bounds data). data2.shape is (1536,2,4)
p data2.data[numpy.any(data2[:,1] < -10, 1).data,:,:]
array([[[  71.20559511,   73.804111  ,   76.34111219,   73.65918693],
[  -2.0933775 ,    5.54460936,   -9.12835156,  -14.18136947]],
   [[  78.98636229,   81.57617148,   82.41927318,   79.95552707],
    [  -3.28731527,    4.14292241,  -25.08277799,  -24.86542329]],

   [[  81.57617148,   84.06203095,   84.82266859,   82.41927318],
    [   4.14292241,   15.22514943,  -28.10941667,  -25.08277799]],

   [[  84.06203095,   86.26519018,   87.07516919,   84.82266859],
    [  15.22514943,   36.57806873,  -40.359316  ,  -28.10941667]],

   [[  86.26519018,   87.37999805,   88.42000205,   87.07516919],
    [  36.57806873,   85.0000674 ,<PHONE_NUMBER> ,  -40.359316  ]]])

@aashish24 @doutriaux1 This finding matched the baseline test_vcs_animate_projected_meshfill_lambert.png


There was an error during login operation into Alpha ESS monitoring portal

I see, there exists an update. I installed it and it works. Fantastic.
Thanks a lot!
Since today I do not get any values.
I use node-red in ioBroker.
Works fine without any error over weeks. Thanks to the programmer(s).
Message:
There was an error during login operation into Alpha ESS monitoring portal: SyntaxError: Unexpected token  in JSON at position 0
We got the following unprocessable body:

I see, there exists an update. I installed it and it works. Fantastic.
Thanks a lot!

You're welcome. Alpha ESS changed some kind of compression settings in their backend today causing this issue.

This is related to #4.


libxml2: add icu support

Motivation for this change
Fixes #22623
Causes a massive rebuild.
Things done

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

This seems to more than double the closure size of both the bin and out outputs. Maybe it should be build without icu by default?

Good point @FRidh. So what exactly is the policy about what dependencies should be included?  Any sort of best practice?

Well, the Nixpkgs manual only mentions multiple outputs, but that won't help us here.
I think the best practice is here, put in only what you need, and everything else is optional.
If someone needs xmllint with ICU support they can build a version of libxml2 that has it. Therefore I suggest you make it an optional feature.
Please rebase your work to staging.

I think the best practice is here, put in only what you need, and
everything else is optional.

It's always a trade-off between size and functionality. If we could do
something like a list of libraries that should always be included by
other packages, that would remove some ambiguity.

If someone needs xmllint with ICU support they can build a version of
libxml2 that has it. Therefore I suggest you make it an optional feature.

Sure.

Please rebase your work to staging.

Sure, why?

Sure, why?

Shouldn't actually be necessary, but that was just to be sure we won't get a mass-rebuild even when ICU is off by default.

Nice catch @FRidh!
Indeed, the changed order of the buildinputs was enough to cause a massive rebuild.
I managed to massage it into submission (at least according to nox-review) so it should be OK now.

Do we have some idea about the use cases for ICU in there? /cc @fubarbaz.

@vcunat, as I understand it, some libraries will use icu while others use libiconv and to avoid ambiguity, you would want to ensure a uniform unicode conversion library all across.
But @FRidh is absolutely correct that it should be disabled by default (and now is).

Yes, at least default for use in nixpkgs, from what I know so far.  The ICU data are rather large.
For installing the preference might be different.  I wonder whether to provide at least an explicit attribute, to allow easier usage and provide binaries.

@vcunat we could add a libxml-icu in all-packages.nix, but, considering this is the first time it came up I think we shouldn't (yet).


elbv2: LBTargetGroup not created

What happened?

When healthCheck is specified, the resource was not created with the following message:

cannot run refresh: refresh failed: Too many health_check blocks: No more than 1 "health_check" blocks are allowed

When healthCheck is not specified, the resource was created but there is still a warning:

cannot run plan: plan failed: arn:aws:elasticloadbalancing:us-east-1:474756349353:targetgroup/client1-k8s-cluster-ingress/db514af6374ff608: health_check.healthy_threshold 5 and health_check.unhealthy_threshold 2 must be the same for target_groups with TCP protocol:

... Despite the doc indicates 3 as default value for both healthyThreshold and unhealthyThreshold  (https://marketplace.upbound.io/providers/upbound/provider-aws/v0.22.0/resources/elbv2.aws.upbound.io/LBTargetGroup/v1beta1#doc:spec-forProvider-healthCheck-healthyThreshold, https://marketplace.upbound.io/providers/upbound/provider-aws/v0.22.0/resources/elbv2.aws.upbound.io/LBTargetGroup/v1beta1#doc:spec-forProvider-healthCheck-unhealthyThreshold)
This warning blocks the deletion of this resource.
How can we reproduce it?

- name: ingress-lb-targetgroup
    base:
      apiVersion: elbv2.aws.upbound.io/v1beta1
      kind: LBTargetGroup
      spec:
        forProvider:
          healthCheck:
            - healthyThreshold: 3
            - unhealthyThreshold: 3
          port: 32080
          protocol: TCP
          region: us-east-1
    patches:
    - type: PatchSet
      patchSetName: providerconfigref
    - fromFieldPath: spec.id
      toFieldPath: metadata.name
      transforms:
      - type: string
        string:
          fmt: '%s-ingress-lb-targetgroup'
    - fromFieldPath: spec.id
      toFieldPath: spec.forProvider.name
      transforms:
      - type: string
        string:
          fmt: '%s-ingress'
    - fromFieldPath: status.VPCID
      toFieldPath: spec.forProvider.vpcId
      policy:
        fromFieldPath: Required

What environment did it happen in?

Universal Crossplane Version: 1.10.1-up.1
Provider Version: v0.22.0

Hi, @fcastarede thanks for raising this issue. I could not reproduce this issue.

cannot run refresh: refresh failed: Too many health_check blocks: No more than 1 "health_check" blocks are allowed

To fix the above error, you should use the healthCheck argument as follows.
    healthCheck:
    - healthyThreshold: 3
      unhealthyThreshold: 3

I did not encounter any errors when I created the example below.
apiVersion: elbv2.aws.upbound.io/v1beta1
kind: LBTargetGroup
metadata:
  labels:
    testing.upbound.io/example-name: elbv2
  name: test
spec:
  forProvider:
    name: example-lb-tg
    port: 32080
    protocol: TCP
    region: us-east-1
    healthCheck:
    - healthyThreshold: 3
      unhealthyThreshold: 3
    vpcIdSelector:
      matchLabels:
        testing.upbound.io/example-name: elbv2
---
apiVersion: ec2.aws.upbound.io/v1beta1
kind: VPC
metadata:
  labels:
    testing.upbound.io/example-name: elbv2
  name: sample-vpc
spec:
  forProvider:
    region: us-east-1
    cidrBlock: <IP_ADDRESS>/16
    tags:
      Name: DemoVpc


Yet another Mastodon explanation

Maybe focus on ActivityPub itself rather than Mastodon as a platform

Eh, not interested for now


Adding GitHub Actions, deploy steps in deploy-AKS.sh

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd spring-openai-bot
git checkout [branch-name]
./mvnw test

Test the code

What to Check
Verify that the following are valid

...

Other Information

These all refer to temporary services, I usually delete the target, and leave the original values as examples.


a few questions about the 'MBR' decoding strategy.

Hi, your code is incredibly useful. However, I have a few questions regarding the 'MBR' decoding strategy.
In your paper, you mentioned, "We first generate a set of candidate samples S from different random seeds of DIFFUSEQ and select the best output sequence that achieves the minimum expected risk under a meaningful loss function."
As I'm new to the 'MBR' decoding strategy, I might be misunderstanding something. My understanding is that you calculate the BLEU score for each candidate and then select the one with the highest score as the final output. Subsequently, you perform an evaluation on the selected output.
I've noticed that if we evaluate these candidates individually, we get 10 similar results, which aren't impressive. However, after using evaluation metrics to select the output, the result is significantly better. I'm unsure if this is an accurate method, or could it be considered "cheating"? (I apologize if this comes off as offensive.) This is because by using evaluation metrics to select the output, we will naturally obtain better metric results.
Additionally, I have read your code (https://github.com/Shark-NLP/DiffuSeq/blob/8bfafcbb26df218073b8117234afb9de9dfcbec9/scripts/eval_seq2seq.py#L16C1-L26C1), and it appears that you didn't calculate the BLEU score between the recovered sentences and the reference sentences. Instead, you only calculated the BLEU score between the recovered sentences and the other candidate recovered sentences.

Hi! Not an author, but I've looked at this code too (eval_seq2seq). I believe MBR here computes the BLEU score between each pair of the S candidates (not looking at the reference!), and selects the candidate with the highest BLEU score to other candidates. The BLEU score is then computed between the chosen candidate and the actual reference. It is not cheating as the candidate is chosen without using the reference at all.
Lines 168-177 in eval_seq2seq.py show this fairly clearly - we select the 'best' candidates (by comparing them to one another - not the reference) and then compare these selected candidates to the reference.

Oh,I got it. Thank you


✨ Handle kebab-case

Adds the --dashes option (subject to change for v1.0) to supports emitting camelCase class names in place of kebab-case ones. Changes the default to drop invalid (e.g. kebab-case names) from the generated TypeScript.
Note: Although the generated default has changed, releasing under 0.2 (no breaking change) because the existing emitted TS was invalid.
Adds css to Prettier config.

FYI @rdparker - I’m thinking of merging this implementation instead of yours. (I’ll merge Monday (2023-12-25) to give you some time to comment any opinions.
If you see this after merge, still free to comment. My overall goal is to finalize some of this config over this next week and look to v1.0 by mid-to-end of Jan 2024.

@rdparker - Again, feel free to comment or open a separate issue if this does not satisfy your needs!! Thanks in advance.


Latest volar LSP version crashes on neovim. Installed with mason

neovim version: 0.9.5
volar version: 1.8.27
OS: ubuntu 20.04
LspLog error:
[ERROR][2024-01-20 14:14:40] .../vim/lsp/rpc.lua:734	"rpc"	"/home/yunus/.local/share/nvim/mason/bin/vue-language-server"	"stderr"	"/home/yunus/.local/share/nvim/mason/packages/vue-language-server/node_modules/@vue/language-server/node_modules/@volar/language-server/node.js:63\n                    return fs.readFileSync((0, exports.uriToFileName)(uri), { encoding: encoding ?? 'utf-8' });\n                                                                                ^\n\nSyntaxError: Unexpected token '?'\n    at wrapSafe (internal/modules/cjs/loader.js:915:16)\n    at Module._compile (internal/modules/cjs/loader.js:963:27)\n    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)\n    at Module.load (internal/modules/cjs/loader.js:863:32)\n    at Function.Module._load (internal/modules/cjs/loader.js:708:14)\n    at Module.require (internal/modules/cjs/loader.js:887:19)\n    at require (internal/modules/cjs/helpers.js:74:18)\n    at Object.<anonymous> (/home/yunus/.local/share/nvim/mason/packages/vue-language-server/node_modules/@vue/language-server/out/nodeServer.js:3:16)\n    at Module._compile (internal/modules/cjs/loader.js:999:30)\n    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)\n
I saw similar issue at https://github.com/vuejs/language-tools/issues/1978 . but I did not see any solution there.

I'm not familiar with neovim. Does mason.nvim use node.js? What's your node.js version?

node version is 21.5.0, mason uses npm to install some packages(volar also).

I'm not sure about what happened, but it seemed that your node.js doesn't understand ?? token. Can mason select which node.js to use?
/cc @yaegassy

Ok, turned out mason used apt's nodejs(v12) library instead of node v21.5.0 installed with nvm. Updated node manually, that fixed the issue. Ty


Remove experimental disable resolution controller flag

Changes
In commits 5681ce99232cffebc42b4a097514bc0a3e14a6e5 and
5fa4348bdbb7f28280bcfd518f6025c94670e420 we introduced a controller flag to
disable resolution of taskRefs and pipelineRefs. This was intended to support an
experimental controller that could externalize resolution of these refs. Tests
and documentation for this flag were added as well.
The experimental flag ultimately never realized its purpose and the
experimental controller was written to only support PipelineRuns started in
PipelineRunPending state. This commit removes the code, tests and docs
surrounding this flag and its behaviour.
Submitter Checklist

[x] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
An experimental controller flag was added in Pipelines v0.28.0 to disable resolution of taskrefs and pipelinerefs. This flag has now been removed.

/kind misc

/test pull-tekton-pipeline-alpha-integration-tests

/lgtm

still getting used to permissions on this repo 😅 anyway, looks good

thanks @sbwsg!
/lgtm

test/v1alpha1 TestPipelineTaskTimeout 1m5s
/test pull-tekton-pipeline-alpha-integration-tests


nixos/dokuwiki: drop SSL forcing and document incompatibility

Motivation for this change
Dropping configuration which was forcing user to use Let's encrypt and SSL as default. It was non-intuitive and causing problems in some configurations.
Moreover, I've documented breaking changes introduced in https://github.com/NixOS/nixpkgs/pull/83769.
Following up on https://github.com/NixOS/nixpkgs/pull/94551.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg test dokuwiki


Support for N24 and M24 values for LMOTS_ALGORITHM_TYPE and LMS_ALGORITHM_TYPE.

As mentioned in the rfc, there is no implementation for N24 and M24 values.

Issue is resolved: https://github.com/pmvr/python-hsslms/pull/3.


Fix update reason field error

:+1:

!test

[Current coverage][cc-pull] is 64.67%

Merging [#184][cc-pull] into [master][cc-base-branch] will increase coverage by 0.60%

File vmpool/clone.py (not in diff) was modified. more

Misses -18
Partials +2
Hits +16

File .../auth/custom_auth.py (not in diff) was modified. more

Misses -1
Partials 0
Hits +1

@@             master       #184   diff @@
==========================================
  Files            41         41
  Lines          2805       2805
  Methods           0          0
  Messages          0          0
  Branches        325        325
==========================================
+ Hits           1797       1814    +17
+ Misses          941        922    -19
- Partials         67         69     +2

Powered by Codecov. Last updated by [15fb197...4d6e464][cc-compare]
[cc-base-branch]: https://codecov.io/gh/2gis/vmmaster/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/2gis/vmmaster/compare/15fb197b2304468e8fad41962e9f6268a40848fd...4d6e464e9e86d603c8d961363073cdc5051581ea
[cc-pull]: https://codecov.io/gh/2gis/vmmaster/pull/184?src=pr


Auth()->user() -> magic vars not available inside scope class

Laravel Version: 9.10.1
PHP Version: 8.1.0
Database Driver & Version: (un related)

Description:
I'm creating a multitenant (single DB) system -> I want to add scoped queries to the ORM models (some of them, like Users, Accounts, invoices, etc). When implementing this, there are two problems:

The auth()->user()->tenant_id is not available during the page refresh (user is logged in)
First point crashes the app -> and does not return any errors -> 502 Bad Gateway, nginx/1.18.0 (Ubuntu) -> Homestead v13.2.1

I found out the problem (the missing information inside the identity object) via debugger.
Steps To Reproduce:

Create an Scope class (in my case) TenantScope
Add:

/**
     * Apply the scope to a given Eloquent query builder.
     *
     * @param  \Illuminate\Database\Eloquent\Builder  $builder
     * @param  \Illuminate\Database\Eloquent\Model  $model
     * @return void
     */
    public function apply(Builder $builder, Model $model)
    {
        $builder->where('tenant_id', auth()->user()->tenant_id);
    }

Use the Scope using Booted function:

protected static function booted()
    {
        static::addGlobalScope(new TenantScope());
    }

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Nit

In https://github.com/GoogleChrome/sample-media-pwa/blob/2efbd1e25d1919c78eecb997318bfaaaf5e8ae7e/src/client/scripts/video-player/video-player.js#L175, did you mean SUPPORTS_REMOTE_PLAYBACK instead of SUPPORTS_MEDIA_SESSION?
@paullewis

I really truly did.

Fixed


IndexError: Cannot choose from an empty sequence, fake.py line 136

Should be fixed by now: pip install --upgrade fake-useragent


support for minicpm3.0

I am a staff member of Model Best. We have recently open sourced minicpm3.0. This is a model address that uses gtpq to quantify minicpm3.0. It has achieved good results. We look forward to your merger.
https://huggingface.co/openbmb/MiniCPM3-4B-GPTQ-Int4

https://huggingface.co/openbmb/MiniCPM3-4B-GPTQ-Int4
This is a model we made using Autogptq. It has been tested and has very few drops.

@LDLINGLINGLING you have a mistake in your MR:
https://github.com/AutoGPTQ/AutoGPTQ/pull/731/files#diff-907c9bd5e643df0d841efb08ec5d019013a6394be0608323af050ae7b6d51183R54
it tries to use MiniCPM3ForCausalLM but the correct name for the class is MiniCPM3GPTQForCausalLM. I think currently main is broken because of that.

Sorry, I mistakenly entered the class name when submitting the PR. I just downloaded the source code and found that it has been modified. I want to confirm with you that this bug has indeed been fixed.

The current autogptq repo has no ci to speak of which means any bug during PR has to be hand tested for regressions which is a mess. Coupled with all the new models, I really don't have the time to manually check.
Moving forward, as far I am concerned, I will not spend time to merge new models here in AutoGPTQ. Will still merge bug fixes as soon as they are reported due to backward compat with transformers integration. But new gptq models and new features, I will trigger/request ci tests for new models in https://githubcom/modelcloud/GPTQModel instead.

@LDLINGLINGLING yes, I think that the bug is fixed now in https://github.com/AutoGPTQ/AutoGPTQ/pull/734 and merged, at least the models I tried today (mistral-small v.2409 GPTQ 4bit and Qwen-2.5-32B GPTQ 4bit) work.
@Qubitium interesting, thanks for sharing. I didn't know about this new opinionated fork, I will try it out!

@avoroshilov hello，can you merge this pr https://github.com/AutoGPTQ/AutoGPTQ/pull/735，
This PR is my second submission. It is the Autogptq adaptation of minicpm2.0. I submitted our gptq model address in the PR，The Gptqmodel fork has merge。

Hey! I don't have write access to this repo, as I am not a collaborator. I think you meant to tag Qubitium in that other #735, probably.

ok，Thank you。

@LDLINGLINGLING As I have said earlier, I will no longer merge any new models due to lack of time and I have no access to this repo's perms to make large refractor (due to near impossible backward compat requirements for api) and ci to ensure this doesn't happen again.
Add on to the fact you submitted two PRs, one to GPTQModel that didnt even work (copy-pasted autogtpq code which we had to fix) and latest one to AutoGPTQ that broke everything for everyone. I really can't trust anything you submit in the future without actually running unit tests. You need to raise your standards when it comes to QA. Just don't have time for this. For autogptq at the moment, stability and not crashing is core goal at the moment.

sorry，I did modify the Gptqmodel based on the AutoGptq code that I copied. This is because in the previous version, MiniCPM2.0 directly merged the AutoGptq code, which led me to believe that AutoGptq and Gptqmodel were interchangeable. However, for the MiniCPM3.0 code, we indeed used this code for quantization. Although there was an error when I submitted the PR, I fixed it the next day. Is it certain that the code crash was due to the issue in my submitted PR? Below is the commit from MiniCPM3.0 where I made the fix.

Sorry, I will improve the quality of the review code in the future. gptqmodel is indeed copied from autogptq, because when MiniCPM2.0 was submitted to Autogptq, you directly merged it into gptqmodel, which made me mistakenly think that the code of Autogptq and the code of gptqmodel are shared. of


updated css files

please could you merge my PR?

@Code1705 please share a preview of changes.

changes that happened in the website after you modified the code.

above given is the screenshot of the changes, now please could you merge my pr?

please could you merge my pr?

I have committed changes.. please could you merge my PR?


installer should add logrotate config at installation time

When installing the code on POSIX systems it should drop a valid logrotate config into /etc/logrotate.d. Presumably the filename should be influxd.
Here's a proposal for the file contents:
/var/log/influxdb/influxd.log {
	daily
	rotate 7
	missingok
	dateext
	copytruncate
	compress
}

This will rotate logs daily, keeping a week's worth and compressing all non-active logs. It will date-stamp the compressed log files. The copytruncate command means the original log filehandle never changes. If the startup scripts are altered we may be able to eliminate that command. It does carry a small chance of missed log events.

A logrotate file will be part of the 0.9.4 package.


tensorflow grpc++ symbol problem

when I load tensorflow with clion plugin, I found int the .cc files, grpc++ cannot be found by the plugin.

and I may want to ask, can this project be handled by itself?

How did you set up tensorflow with clion+bazel plugin?
Did you run the ./configure script like in https://www.tensorflow.org/install/install_sources?
What is an example .cc file + line number that is having problems with grpc++ headers and symbols?
They work for me after I run ./configure, and then just import with the bazel plugin. On Linux, with .bazelproject configured to:
directories:
.
targets:
//...:all
For file tensorflow/core/distributed_runtime/rpc/grpc_channel.cc, CLion can resolve the reference to ::grpc::CreateCustomChannel on line 69 just fine (see attached image).
(https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/distributed_runtime/rpc/grpc_channel.cc#L69)
The other thing to keep in mind is that the .cc has to be reachable from a BUILD file target. For example, tensorflow/core/distributed_runtime/master_test.cc has a reference to grpc++.h that is unresolved, but it is commented out in the BUILD file (so not actually reachable as a target).

Thank you very much for you reply. sorry for the late reply, I'm fixing my network these days, it seems some urls has been blocked, and now It's OK.

Okay, closing issue then, thanks!


Add filtering/sorting to favorite setups component picker

Auto arrange current location setup to the top and others at the bottom, with a separator
Add ability to fast travel to the fav setup location


[WIP] Read the index file to get row count when query for 1 partition without other filters

Why is this PR needed?
Read the index file direcctly when query count(*) for 1 partition without other filters, currentlly, it comes into scan prune logic, we can optimize it.
What changes were proposed in this PR?
Read the index file to get row count when query count(*) for 1 partition without other filters
Does this PR introduce any user interface change?

No

Is any new testcase added?

No

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2675/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/963/

retest this please

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/970/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2682/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/975/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2687/


How I can use svgr with rollup with query parameters for import

Hello!
With webpack I can use imports for svg as https://react-svgr.com/docs/webpack/#use-svgr-and-asset-svg-in-the-same-project
How I can use query params for import with rollup for svg imports?

Just got this issue too.
I write a plugin to solve my problem rollup-plugin-resource-query.
import url from '@rollup/plugin-url'
import svgr from '@svgr/rollup'
import resourceQuery from 'rollup-plugin-resouce-query'

export default {
  plugins: [resourceQuery({ resourceQuery: 'url' }), url(), svgr()],
  input: 'src/main.js',
  output: {
    file: 'bundle.js',
    format: 'cjs',
  },
}

It solve issue in my project at least.
hope someone came across this post can benefit from it. 😄


Add bottom padding to Sidebar

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on modifying the padding class in the Sidebar.tsx component to enhance the layout.
Detailed summary

Changed the class from pt-7 to py-7 in the div element, which adjusts the vertical padding (top and bottom) instead of just the top padding.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#4993  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MananTank and the rest of your teammates on  Graphite


[Bug] context_recall metric failing when using custom dataset

Describe the bug
Hi, I am trying to setup evaluation on my custom dataset using Ragas but getting the following error when trying to evaluate for context_recall metric:
evaluating with [context_recall]
 17%|████████▊                                            | 1/6 [00:24<02:02, 24.42s/it]
Traceback (most recent call last):

  File ~/Projects/embedchain/.venv/lib/python3.11/site-packages/IPython/core/interactiveshell.py:3550 in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)

  Cell In[50], line 3
    result2 = evaluate(

  File ~/Projects/embedchain/.venv/lib/python3.11/site-packages/ragas/evaluation.py:105 in evaluate
    scores.append(metric.score(dataset).select_columns(metric.name))

  File ~/Projects/embedchain/.venv/lib/python3.11/site-packages/ragas/metrics/base.py:76 in score
    score = self._score_batch(dataset.select(batch), callbacks=group)

  File ~/Projects/embedchain/.venv/lib/python3.11/site-packages/ragas/metrics/_context_recall.py:120 in _score_batch

  File <string>:13
    "statement_3": "Load, split, and embed your documents: loader = TextLoader('state_of_the_union.txt'), documents = loader.load(), text_splitter = CharacterTextSplitter(chunk_size=1000, chunk_overlap=0, separator='
                   ^
SyntaxError: unterminated string literal (detected at line 13)

Ragas version: 0.0.21
Python version: 3.11
Code to Reproduce
Use the following code to reproduce. Unfortunately I cannot share the dataset here.
from ragas import evaluate

result2 = evaluate(
    dataset,
    metrics=[
        context_recall,
    ],
)

Error trace
Shared above.
Expected behavior
Expect to see he context_recall score.

Thanks for sharing @deshraj , Can you retry after installing ragas from source pip install git+https://github.com/explodinggradients/ragas.git

User's warning with invalid json
 0%|          | 0/2 [00:00<?, ?it/s]/home/ec2-user/anaconda3/envs/python3/lib/python3.10/site-packages/ragas/utils.py:29: UserWarning: Invalid json: Expecting value: line 1 column 2 (char 1)
  warnings.warn(f"Invalid json: {e}")
/home/ec2-user/anaconda3/envs/python3/lib/python3.10/site-packages/ragas/utils.py:29: UserWarning: Invalid json: Expecting value: line 1 column 2 (char 1)
  warnings.warn(f"Invalid json: {e}")
 50%|█████     | 1/2 [00:19<00:19, 19.78s/it]/home/ec2-user/anaconda3/envs/python3/lib/python3.10/site-packages/ragas/utils.py:29: UserWarning: Invalid json: Expecting value: line 1 column 2 (char 1)
  warnings.warn(f"Invalid json: {e}")
/home/ec2-user/anaconda3/envs/python3/lib/python3.10/site-packages/ragas/utils.py:29: UserWarning: Invalid json: Expecting value: line 1 column 2 (char 1)
  warnings.warn(f"Invalid json: {e}")

Have you fixed it already ? I have the same problem.

@arm-diaz we need to custumize the prompt for bedrock actually. there is #298 which has the correct prompts but we need to fix that
@RuiNov1st are you facing the issue with bedrock?


Way to identify existing files

We'd like to come up with a way to check if a file exists within a given folder. We talked about a couple ways of doing this, including:

updating datadex to include the files within each folder recursively
keeping a list of all unique file names across any folders
others

@hbsmith I'm trying to decide if I still think this is necessary. Can you give me some idea of what the use-case might be?

The case I was thinking of would be if you wanted to run analyses across
many different networks, and in order to run the analyses you need to know
if a file with the data exists or not (for example, the
transfer_entropy.txt file). In that case, it would be useful to identify
all the networks in which the analysis is currently possible.
If you at least know what the name of the file containing the transfer
entropy data is, maybe you could do this without knowing which directories
have the files directly? I'm not sure.
On Sun, Oct 29, 2017 at 4:18 PM, Douglas G. Moore<EMAIL_ADDRESS>wrote:

@hbsmith https://github.com/hbsmith I'm trying to decide if I still
think this is necessary. Can you give me some idea of what the use-case
might be?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ELIFE-ASU/datadex/issues/5#issuecomment-340311387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI5NdO9J2LLdX3f7dIL09LA8BGFTMFYBks5sxQe4gaJpZM4QJmUp
.


support for standard library sum()

Not sure if i am missing something, but it seems the sum(iterable) function is not supported (cannot find it in the supported list). That seems odd. Any chance of adding it?
I get the error:
Untyped global name 'sum': Cannot determine Numba type of <class 'builtin_function_or_method'>
the code is:
cost -= late_cost * sum([s for s in spares if s < 0])
(all floats)

Feature request

Thanks for opening this feature request. As this is a duplicate of https://github.com/numba/numba/issues/5088 I'm going to close it with view of keeping the number of open issues in the issue tracker to a minimum. I suggest commenting on and/or subscribing to #5088 to check for progress. Thanks again!


building the exe file

My problem is that I cant find the place which i need so i can make the exe file and i have already changed the webhook. i know i have to compile it but the my question is what exactly should i compile cuz there are like 20 files.

https://youtu.be/LYHPZXgqSPQ


Add recipe on HaikuPorts

@bach5000 recipe just merged, https://github.com/haikuports/haikuports/pull/1862
Thanks to @bach5000


Test job

Purpose

Remarks

PR introduces (a) breaking change(s): <yes/no>
PR introduces (a) deprecation(s): <yes/no>

Thanks for implementing this processor!
I have two minor suggestions for improvement:

Could we extend it to allow the user to specify how many events should be included in the sum? For example, if the user enters "5," the processor would calculate the sum of the last five events.
Also, what do you think about renaming the processor to something like SumProcessor or SummationProcessor to better reflect its function?

Let me know your thoughts!


Pagination for odata v4

I don't have the right to review it (i'll ask the write rights later, team is at the offsite).
But LGTM : plugin works flawlessly and code is clean ✅   @alexbourret you can merge !

Thanks Kevin !


Building from sourse on visual studio windows

Hi,
I would like to build build code from source.
I do not need any interface, but the algoritm.
In that case do I need to install additional libraries like Eigen? Or it is part of code already?
Is there any step by step manual how to build the code from source on windows?

Hi @petrasvestartas, you can find instructions on this page: https://github.com/GeometryCollective/boundary-first-flattening. Most of the necessary dependencies are included in the project, but you will need to install Cmake and Suitesparse on your machine.

Dear @rohan-sawhney
Is it possible to have console application example in C++ while using built libraries (without actually building the full project and its dependencies)? Or it is not so simple as in .NET languages?

I solved the same problem, you can refer to my article: https://blog.csdn.net/weixin_38980073/article/details/121541937


Support for more than 1 project

Only works for first entry in treeview

#4 This support this feature


BLC_04: X-Axis & Y-Axis labels are missing in Axes menu

SEVERITY: Major
MODULES: Bubble Chart, Bar Chart
OS / BROWSER: Windows 7 / Chrome 45.0
STEPS TO REPRODUCE:

Navigate to URL: http://static.gapminderdev.org/vizabi/release/v0.5.1/preview/bubblechart.html
Bubble chart page will be loaded
Click on Axes menu
Note that x-axis & y-axis labels are missing

ACTUAL RESULTS:
X-axis & y-axis labels are missing in Axes menu
EXPECTED RESULTS:
Both axes should be labelled as x-axis & y-axis to avoid confusion
SCREENSHOTS:

Can't reproduce this issue on develop branch.

@IncoCode
According to your screenshot attached in your above comment. This issue is fixed now as you can see about issue "summary". As now x-axis & y-axis labels are there under "Axes"

This was indeed fixed in Vizabi v0.6


Forward all args after the first two to python

The first two args are already used for cython and the compiler, respectively. The rest should be forwarded so that the following commands do the same thing:
python hello.py arg1 arg2
runcython3 hello.pyx '' '' arg1 arg2

Cool, great idea!


add /add command

add /add slash command, which takes in name, date, hours and minutes and adds to the database, sends a confirmation on completion, check for valid datetime format. datetime stored in database should be int (unix timestamp)

I would like to contribute


Possible Integer overflow in OperationChecking

Per https://lgtm.com/projects/g/rhuss/jolokia/snapshot/5af274c465d19828fdbb4f5787b92d5d4cad7539/files/it/core/src/main/java/org/jolokia/it/OperationChecking.java?sort=name&dir=ASC&mode=heatmap#xfd12e45d383a8229:1

thanks for the finding ! Sorry for the delay, I really work on Jolokia from time to time (where time is in the order of months right now).
Apologies. Closing this issue as the correspond PR is merged.


TO end cardinatlity must always be set to One

Tabular editor allows me the freedom to create a model that does not open in VS:

This is just one of many things that are allowed in Tabular Editor (or rather, it's allowed in the TOM), even though Analysis Services won't have it. It would be a lot of work to build in validation for everything. Instead, I'm relying on Tabular developers to know what they're doing :-)
Of course, it would be a minor thing to add validation that the TO-end cardinality should always be "One", but then in Compatibility Level 1500, you can actually have relationships where both ends have a cardinality of Many.
My design philosophy, at least for now, is to keep Tabular Editor as lightweight as possible in terms of what it lets you do to the TOM, and then rely on Analysis Services to validate the model.

Hi @otykier ok, I see your point. I am just trying to totally migrate away from Visual Studio as it is pretty much unusable on very large models.
Might be a good idea to have this as a rule in the Best Practice Analyzer so people catch it before actually deploying or opening in VS...


Integrate function cli into pulsar-admin cli

This change is based on #1329
701df16 is the commit for review in this PR.

rename pulsar-client-tools-shaded to pulsar-client-admin-shaded-for-functions, because this module is used by functions only to avoid protobuf conflicts
move protobuf3 references to Utils, so it won't be referenced out side of pulsar-functions
integrate function cli into pulsar-admin cli
701df16

/cc @srkukarni @jerrypeng

retest this please


Fix cupyx.seterr() when linalg not supplied

Rel #4138.
Currently, cupyx.seterr() unintentionally set None to _config.linalg when linalg argument is not supplied. This PR fixes it to leave as it is in such a case.

pfnCI, test this please.

Jenkins CI test (for commit adaeb6ba6a4172be6d4f374a855c67920dfe9527, target branch master) succeeded!

LGTM.


loophole unable to retrieve SSL certificate

failed to retrieve certificate with following errors:
`Registering your domain... Success!
Starting local proxy server...  Success!
Initializing secure tunnel...  Success!
Forwarding https://**.loophole.site -> http://<IP_ADDRESS>:80
Press CTRL + C to stop the service
Logs:
8:08AM INF Awaiting connections...
8:08AM INF Succeeded to accept connection over HTTPS
8:08AM INF Succeeded to accept connection over HTTPS
8:08AM INF Succeeded to accept connection over HTTPS
8:08AM INF Succeeded to accept connection over HTTPS
8:08AM ??? http: TLS handshake error from <IP_ADDRESS>:51160: acme/autocert: certificate is not valid yet
8:08AM ERR TLS Certificate failed to provision. Will be obtained with first request made by any client, therefore first execution may be slower
8:09AM INF Succeeded to accept connection over HTTPS
8:09AM ??? http: TLS handshake error from <IP_ADDRESS>:36596: acme/autocert: missing certificate
8:09AM INF Succeeded to accept connection over HTTPS
8:09AM ??? http: TLS handshake error from <IP_ADDRESS>:36600: acme/autocert: missing certificate
8:09AM INF Succeeded to accept connection over HTTPS
8:09AM ??? http: TLS handshake error from <IP_ADDRESS>:42700: acme/autocert: missing certificate
8:09AM INF Succeeded to accept connection over HTTPS
8:09AM ??? http: TLS handshake error from <IP_ADDRESS>:44898: acme/autocert: certificate is not valid yet
8:10AM INF Succeeded to accept connection over HTTPS
8:10AM ??? http: TLS handshake error from <IP_ADDRESS>:57038: acme/autocert: missing certificate
8:10AM INF Succeeded to accept connection over HTTPS
8:10AM ??? http: TLS handshake error from <IP_ADDRESS>:57054: acme/autocert: missing certificate
8:10AM INF Succeeded to accept connection over HTTPS
8:10AM ??? http: TLS handshake error from <IP_ADDRESS>:57826: acme/autocert: missing certificate
8:10AM INF Succeeded to accept connection over HTTPS
8:10AM ??? http: TLS handshake error from <IP_ADDRESS>:57836: acme/autocert: missing certificate
8:10AM INF Succeeded to accept connection over HTTPS
8:10AM ??? http: TLS handshake error from <IP_ADDRESS>:60290: acme/autocert: missing certificate
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:40408: acme/autocert: certificate is not valid yet
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:54828: acme/autocert: missing certificate
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:54832: acme/autocert: missing certificate
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:54834: acme/autocert: missing certificate
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:54838: acme/autocert: missing certificate
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:54848: acme/autocert: missing certificate
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:34456: acme/autocert: missing certificate
8:11AM INF Succeeded to accept connection over HTTPS
8:11AM ??? http: TLS handshake error from <IP_ADDRESS>:34468: acme/autocert: missing certificate
`

Please restart the tunnel and try again.
Certificate obtaining is fully client-side so we don't have the logs, but I did some cleanup in the backend which could help if the tunnel was not open correctly.

I'm seeing the same problem
Restarting the tunnel, changing hostname etc nothing fixes it

I get the same error

I got the same error, too. Restart terminal is ok.


Detects indentation based on comments

I have a lua file here that starts with a few comments that are indented with 3 spaces while the remaining file (actual code) is indented with 4 spaces. Your package reports 3 spaces as indentation of the file, is there a way the detection could be changed to ignore comments?

Can you update to version 0.5.0 and see if you still have this issue? I think I found a solution to re-checking after tokenization

Hrm nope, still detects it as 3 spaces, here's the top of the file as an example:
--[[ Transmisson.lua
This file contains all transmission related functionality, e.g. import/export and chat links.
For that it hooks into the chat frame and addon message channels.

This file adds the following API to the WeakAuras object:

DisplayToString(id, forChat)
   Converts the display id to a plain text string

DisplayToTableString(id)
   Converts the display id to a formatted table

ShowDisplayTooltip(data, children, icon, icons, import, compressed, alterdesc)
   Shows a tooltip frame for an aura, which allows for importing if import is true

ImportString(str)
   Imports an aura from a string

]]--

-- Lua APIs
local tinsert, tconcat, tremove = table.insert, table.concat, table.remove
local fmt, tostring, string_char, strsplit = string.format, tostring, string.char, strsplit
local select, pairs, next, type, unpack = select, pairs, next, type, unpack
local loadstring, assert, error = loadstring, assert, error
local setmetatable, getmetatable, rawset, rawget = setmetatable, getmetatable, rawset, rawget
local bit_band, bit_lshift, bit_rshift = bit.band, bit.lshift, bit.rshift
local coroutine = coroutine

local WeakAuras = WeakAuras;
local L = WeakAuras.L;

local version = 1421;
local versionString = WeakAuras.versionString;

local regionOptions = WeakAuras.regionOptions;
local regionTypes = WeakAuras.regionTypes;
local event_types = WeakAuras.event_types;
local status_types = WeakAuras.status_types;

-- local functions
local encodeB64, decodeB64, tableAdd, tableSubtract, DisplayStub, removeSpellNames
local CompressDisplay, DecompressDisplay, ShowTooltip, TableToString, StringToTable
local RequestDisplay, TransmitError, TransmitDisplay

-- GLOBALS: WeakAurasOptionsSaved WeakAurasSaved UIParent

local bytetoB64 = {
    [0]="a","b","c","d","e","f","g","h",
    "i","j","k","l","m","n","o","p",
    "q","r","s","t","u","v","w","x",
    "y","z","A","B","C","D","E","F",
    "G","H","I","J","K","L","M","N",
    "O","P","Q","R","S","T","U","V",
    "W","X","Y","Z","0","1","2","3",
    "4","5","6","7","8","9","(",")"
}

@Stanzilla The 0.5.0 version fixes your issue when I try loading your example file. You have the language-lua plugin installed right?

I do, yes. See http://i.imgur.com/a26XiFO.png though

It's always going to show 3 spaces on the lines that are three spaces. But what happens if you press tab on one of the blank lines. How many spaces does it insert?

It now displays 4 spaces even on the lines with 3 spaces in the comments so I guess it is fixed. All I did is wait and update another (unrelated) extension. Maybe I was missing an event trigger so it updates? Anyway, thanks for the fix!

Was probably this https://github.com/atom/atom/pull/11101


2021 Halloween lighting automation

Heard Halloween in the area I moved in is very active. Might be a good motivation for the project to aim doing some sort of automation by a week before 2021/10/30.

Replaced by, and done in https://github.com/130s/hut_10sqft/issues/401


Running some tests on nasm failures

Fixes #460
Description of the changes:

Will update when we figure out what's wrong

Before submitting a Pull Request:

[ ] I reviewed CONTRIBUTING.md
[ ] I built my changes locally
[ ] I ran the unit tests
[ ] I ran the functional tests with a device
[ ] I ran the performance tests with a device

I tested changes on:

[ ] Windows
[ ] Linux

It looks like the networking issue is resolved with no interaction needed from us.


add "additional ping targets" to hosts

Should be able to add additional ping targets for hosts.
To allow hosts such as routers to be easily configured to ping devices on the local network to ensure constant connectivity, such as security cameras.
Then any wireless jamming would be logged.

@emrahtavsan please modify the RouterOS agent so devices only ping the server they are associated with. Please remove the other ping targets.

We need to only ping the server for now.


Fix small timers only triggering once per frame

Fixes #52687
Timers with very small values have less accuracy if FPS are not high enough (for example, a 0.01s timer needs at least 100FPS for full accuracy). This helps to combat that issue by triggering multiple times on the same frame if needed.
Also related to this issue, the delta time was lower than expected when target FPS is much lower than physics FPS (eg: at 1 target FPS and 60 physics FPS, delta was being set to 0.13 when it should have been close to 1.00). That came from this line which I removed:
process_step -= (advance.physics_steps - max_physics_steps) * physics_step;

I couldn't understand the logic behind this since it's modifying process time when dealing with physics time, and after some simple tests I didn't find any issue with it's removal.
I wasn't able to test this properly on a full test/demo for 4.0 though, so in the meantime I'll apply this on 3.x to find out.
EDIT: Reverted this last change, see comments below

That line was added in https://github.com/godotengine/godot/commit/d5abd4eb750387c95d1040e7613f98bd816c58bb (PR https://github.com/godotengine/godot/pull/17353) by @zmanuel, maybe they're still around and can give some insights :)

I couldn't understand the logic behind this since it's modifying process time when dealing with physics time, and after some simple tests I didn't find any issue with it's removal.

This is presumably to keep the frame delta linked to the physics steps that have been issued. The physics steps are capped to prevent the physics spiral of death. Without this, the two will become out of sync in this situation, which may or may not be desirable, but it will be compatibility breaking.

Yeah, this looks like it could bring more trouble than it's worth, so I reverted it

I couldn't understand the logic behind this since it's modifying process time when dealing with physics time, and after some simple tests I didn't find any issue with it's removal.

This is presumably to keep the frame delta linked to the physics steps that have been issued. The physics steps are capped to prevent the physics spiral of death. Without this, the two will become out of sync in this situation, which may or may not be desirable, but it will be compatibility breaking.

That is correct; the line is there to prevent the two observable game clocks from getting out of sync. Now, really, introducing the line was a compatibility break for those who wanted the animation clock to keep in sync with the real world clock, but AFAIK there were no complaints about that :)

Another thing is that having a cap on the delta in that circumstance may be a good thing - if a machine suspends for e.g. 2 days then resumes, you don't want it processing 2 days worth of delta, or pausing while loading. There may be a protection against this earlier in the timing, I can't remember.
This also affects the main change in the PR, in some situations, you might not want it firing a load of timers if there is a pause. I am also left wondering whether the issue reported is actually a bug, or rather intended behaviour. There are no comments in the code so it is difficult to know. This should probably be established by asking reduz, who looks to be the original author (the commit is from 8 years ago). It is entirely possible that it is intended behaviour, in order to limit the processing (this technique is often used).
Also this is potentially compatibility breaking, so a little bit of care has to be taken.
Some options:

Try it in master, but not in 3.x (as too long into the life cycle)
If used in 3.x we could consider a switch with the old behaviour, with perhaps the old behaviour as default.

This is a common problem, anything one person sees as a 'bug', fixing it can annoy other users because it breaks their game (this happened with 2d camera changes for example). Generally having an optional legacy path can be the safer way of making such changes.
It is also something that can be worked around manually with a few lines of gdscript (which is a threshold for making core changes via a proposal), so given that we aren't having huge numbers of people reporting this, it is probably worth establishing whether this is actually a genuine oversight or a misunderstanding. If this is not a bug, then it is in effect a proposal.
As an aside, generally I think that people who are using Timer nodes at high frequency are probably using the wrong tool for the job. They are very rough, and unlikely to be reliable at higher frequencies than about 5 times per second. There is also a common misconception that physics ticks run in real time, asynchronously to the frame updates, with people trying to use them for e.g. sound. This is most definitely not the case. If there is a large frame gap, you will also get a large number of physics tick updates running in close succession - the delta passed is not a representation of wall clock time.

It is also something that can be worked around manually with a few lines of gdscript (which is a threshold for making core changes via a proposal), so given that we aren't having huge numbers of people reporting this, it is probably worth establishing whether this is actually a genuine oversight or a misunderstanding. If this is not a bug, then it is in effect a proposal.

You've given a really good point: this is an edge use case that can be implemented with a custom script, and so this looks more like a proposal than a bugfix.
Plus, it's not clear what the use case is from the issue, or if there's actually no use case and the user just assumed this is a bug. Assuming it's the latter, I agree with lawnjelly that the Timer node is not meant to be used as an accurate measure of time, and the scenarios where the framerate dips below 10FPS for example (which by that point affects timers lower than 0.1s) are mostly temporary situations; and having the Timer trigger more times there could make the situation even worse.
@Nukiloco, if you believe this is an important engine feature however, you should write a proposal at godot-proposals to describe and justify your use case. As lawnjelly said, this is potentially compatibility breaking, and can be easily worked around, so if you have a strong reason for this to be implemented, please describe in more detail why that is.
In the meantime, I'll close this PR until a potential proposal goes forward.


(FACT-3171) Fix IPv6 link-local unicast address filtering

RFC4291 define the Link-local unicast range as fe80::/10, not fe80::/16 (what we compared against because of the String comparison). As a consequence, some Link-local unicast addresses where not correctly filtered, e.g. fe87::.
While here, add a "working" example with an actual global IPv6 address from the 2001:0DB8::/32 range reserved for documentation (RFC3849).

Can one of the admins verify this patch?

@smortex we just merged some changes to address the test failures. If you rebase onto d3447d7a08703231c68cfef8c06c41133b38dff9 I think your PR will pass

Awesome!  Thanks for the head-up!


"Can not match a node to this assertion" error?

Started getting this error message on my redux-render-prop project from $ExpectType assertions. Any ideas what's going on?
Ex. from here https://github.com/epeli/redux-render-prop/blob/5ff7179445755930163ae9bbd1177274109af20d/dtslint/redux-render-prop.dtslint.tsx#L17
Here's how to reproduce this
git clone --single-branch -b dtslint-bug https://github.com/epeli/redux-render-prop.git
cd  redux-render-prop
npm ci
./node_modules/.bin/tslint --project tsconfig.dtslint.json

Used versions
$ node -v
v8.12.0
$ npm -v
6.4.1
$ tsc -v
Version 3.1.6
$ tslint -v
5.11.0
$ jq .version node_modules/dtslint/package.json
"0.3.0"

Not quite sure when this started to happen.
Here's the tslint config
https://github.com/epeli/redux-render-prop/blob/5ff7179445755930163ae9bbd1177274109af20d/tslint.json

I ran into the same issue just now. For me, it seemed to be related to the version of typescript that dtslint pulls in. dtslint defines  "typescript": "next", which at the moment of writing resolves to 3.5.0-dev.20190420. If I manually edit the package-lock and set it to 3.4.4 (latest stable), then everything works fine.

I can confirm. Unfortunately I'm seeing the same. As mentioned by @TiddoLangerak, workaround for<EMAIL_ADDRESS>is to edit package-lock.json and set  typescript dependency to v 3.7.2.
    "dtslint": {
      "version": "2.0.1",
      "resolved": "https://registry.npmjs.org/dtslint/-/dtslint-2.0.1.tgz",
      "integrity": "sha512-AiTv/GAn3Gg/WNBFFOfRn26sXpRpX/ZaTES+bcOaPPz6lnv/xh9L8PkpgcWfNyhQqEhsPi1JAoS24sCX3MYZ0g==",
      "dev": true,
      "requires": {
        "definitelytyped-header-parser": "3.8.1",
        "dts-critic": "^2.2.2",
        "fs-extra": "^6.0.1",
        "request": "^2.88.0",
        "strip-json-comments": "^2.0.1",
        "tslint": "5.14.0",
        "typescript": "=3.7.2"
      },
      "dependencies": {
        "typescript": {
          "version": "3.7.2",
          "resolved": "https://registry.npmjs.org/typescript/-/typescript-3.7.2.tgz",
          "integrity": "sha512-ml7V7JfiN2Xwvcer+XAf2csGO1bPBdRbFCkYBczNZggrBZ9c7G3riSUeJmqEU5uOtXNPMhE3n+R4FA/3YOAWOQ==",
          "dev": true
        }
      }
    },

I can confirm. Unfortunately I'm seeing the same. As mentioned by @TiddoLangerak, workaround for<EMAIL_ADDRESS>is to edit package-lock.json and set  typescript dependency to v 3.7.2.
    "dtslint": {
      "version": "2.0.1",
      "resolved": "https://registry.npmjs.org/dtslint/-/dtslint-2.0.1.tgz",
      "integrity": "sha512-AiTv/GAn3Gg/WNBFFOfRn26sXpRpX/ZaTES+bcOaPPz6lnv/xh9L8PkpgcWfNyhQqEhsPi1JAoS24sCX3MYZ0g==",
      "dev": true,
      "requires": {
        "definitelytyped-header-parser": "3.8.1",
        "dts-critic": "^2.2.2",
        "fs-extra": "^6.0.1",
        "request": "^2.88.0",
        "strip-json-comments": "^2.0.1",
        "tslint": "5.14.0",
        "typescript": "=3.7.4"
      },
      "dependencies": {
        "typescript": {
          "version": "3.7.4",
          "resolved": "https://registry.npmjs.org/typescript/-/typescript-3.7.4.tgz",
          "integrity": "sha512-A25xv5XCtarLwXpcDNZzCGvW2D1S3/bACratYBx2sax8PefsFhlYmkQicKHvpYflFS8if4zne5zT5kpJ7pzuvw==",
          "dev": true
        }
      }
    },


Support Resumability for nodejs environment

A number of use cases where this feature is desired seem to have come up (uppy and uppy-server being an example).  So maybe this is a great time to have this feature implemented? cc @Acconut @kvz

There's what I propose:
We add a urlStorage property to the options which must be null or an instance with following layout:
class UrlStorage {
  setItem(fingerprint, url, callback)
  getItem(fingerprint, callback)
  deleteItem(fingerprint, callback)
}

Notice how the API designed with callbacks to allow an asynchronous usage (which is required for Redis etc). However, we should also consider using Promises as this is probably going to be the standard in new browser APIs.
Furthermore, if the urlStorage options is null (it's default value) and we are in the Node environment, no resumability should be enabled. However, if we are in a browser, it should be set to an UrlStorage instance which wraps the localStorage API. This ensure backwards-compatibility.
What do you think?

I like this proposal, I do think it's nice that on the server, it would be cool if you could enable reusability with just configuration, so that not everybody has to write the same adapter for local file storage. I'm thinking something like this on the server:
const fsAdapter = require('tus-js-client/storage-adapters/fs')
const opts = {
   urlStorage: {
     adapter: fsAdapter,
     conf: {
        dir: '/tmp/tus-resumability'
     }
   }
}

In the browser, if we detect the default null as urlStorage, we substitute with this internally under the hood, much like you said, Marius:
if (opts.urlStorage === null && env === 'web') {
  const webAdapter = require('./storage-adapters/browserLocalStorage')
  opts.urlStorage = {
     adapter: webAdapter,
  }
}

so that not everybody has to write the same adapter for local file storage

Sure, that's what I also thought. We should provide a basic file-based url storage.
Regarding your code snippet I would prefer if the option interface looked more like:
// my-server-app.js
const FileUrlStorage = require('tus-js-client/storages/file')
const opts = {
   urlStorage: new FileUrlStorage({
      dir: '/tmp/tus-resumability'
   })
}

For me it's easier to see what's going on if you pass an object instead of a constructor and the corresponding arguments for the constructor.

💯% agreed!

👌

So I started to work on this over the weekend.

I can't figure out if "resumability" has been added to nodes or not?
The demo doesn't seem to work though and uploads always restart at 0%.

tus-js-client can automatically resume from network interruptions or other connection problems but it is currently not able to resume an upload after the process has stopped on its own. You can achieve it manually by saving the upload.url and pass it to a new tus.Upload as the uploadUrl option.

@Acconut I can close this issue right?

Yes, I think this is resolved. Thanks for reminding, Ife!


What does it all mean Bazel?

Please specify what version to Bazel this needs to be built with?  Also please make sure you can actually compile that version for aarch64 on the Jetpacks Nvidia provides for its products.  If can't be compiled on aarch64 please fix this.  Nvidia should really do a better job of making sure its stuff is able to be compiled on its Jetpacks.  It would be nice if Nvidia would not use build systems which cannot be easily installed on their provided Jetpacks.  There is not simple command to install bazel.

You can compile bazel on Jetson AGX using following commands:
curl -L https://github.com/bazelbuild/bazel/releases/download/3.2.0/bazel-3.2.0-dist.zip -o bazel-3.2.0-dist.zip
mkdir -p temp && cd temp && unzip ../bazel-3.2.0-dist.zip
./compile.sh

Please refer https://github.com/NVIDIA/TRTorch#platform-support for Platform support updates.

Refer to this issue for current instructions on aarch64 compilation #70, a more turn key solution is under development (#36)

Reopen this issue if there are still problems running on aarch64


Investigate why CI miri jobs are failing recently

The miri CI jobs are failing roughly since this PR: (2024-09-13)
https://github.com/wasmi-labs/wasmi/pull/1173#issuecomment-2349281132
CI logs stating the failure: https://github.com/wasmi-labs/wasmi/actions/runs/10852181007/job/30117620745
The PR merged before this linked PR and where the miri CI jobs still worked was this: https://github.com/wasmi-labs/wasmi/pull/1172 (2024-09-05)
Commits on the miri repository between 2024-09-05 and 2024-09-13 are:
https://github.com/rust-lang/miri/commits/master/?since=2024-09-05&until=2024-09-12
Unfortunately due to roll-ups it is more difficult to disentangle the changes to miri.
The problem is with this abtraction to (unsafely) convert between Wasmi reference types and UntypedVal:
https://github.com/wasmi-labs/wasmi/blob/main/crates/wasmi/src/reftype.rs
And in particular its usage here: https://github.com/wasmi-labs/wasmi/blob/main/crates/wasmi/src/func/funcref.rs#L55
What the code in question does is a fancy transmute as stated by the Rust reference:
https://doc.rust-lang.org/reference/items/unions.html#reading-and-writing-union-fields
The only missing part is that the Transposer union is missing a #[repr(C)] attribute which we should add. Local tests confirmed though that adding #[repr(C)] does not fix the miri warning.
This PR makes the implicit transmute more explicit and adds the missing #[repr(C)], though miri is still unhappy about it:
https://github.com/wasmi-labs/wasmi/tree/rf-fix-miri-errors

I crafted a minimal test case that pretty much does the same as Wasmi does, however, at least for the Rust playground miri accepts that code for reaons that I have not yet understood.
use core::mem;
use core::num::NonZeroU32;

#[derive(Debug, Copy, Clone, PartialEq, Eq)]
#[repr(transparent)]
pub struct FuncRef {
    funcref: Option<Func>,
}

impl FuncRef {
    pub fn null() -> Self {
        Self { funcref: None }
    }

    pub fn is_null(self) -> bool {
        self.funcref.is_none()
    }

    fn canonicalize(self) -> Self {
        if self.is_null() {
            // Safety: This is safe since `0u64` can be bit
            //         interpreted as a valid `FuncRef` value.
            return u64_to_funcref(0_u64)
        }
        self
    }
}

#[derive(Debug, Copy, Clone, PartialEq, Eq)]
#[repr(C)]
pub struct Func {
    store: u32,
    index: NonZeroU32,
}

fn funcref_to_u64(funcref: FuncRef) -> u64 {
    unsafe { mem::transmute::<FuncRef, u64>(funcref.canonicalize()) }
}

fn u64_to_funcref(value: u64) -> FuncRef {
    unsafe { mem::transmute::<u64, FuncRef>(value) }.canonicalize()
}

fn main() {
    let funcref = FuncRef { funcref: Some(Func { store: 1, index: NonZeroU32::new(2).unwrap() }) };
    assert_eq!(funcref, u64_to_funcref(funcref_to_u64(funcref)));
}

Based on the above minimal test case I was able to identify the location of the bug: It is in the canonicalize function. If we shift the work out of it we get the following piece of code that now is accepted by miri.
use core::mem;
use core::num::NonZeroU32;

#[derive(Debug, Copy, Clone, PartialEq, Eq)]
#[repr(transparent)]
pub struct FuncRef {
    funcref: Option<Func>,
}

impl FuncRef {
    pub fn null() -> Self {
        Self { funcref: None }
    }

    pub fn is_null(self) -> bool {
        self.funcref.is_none()
    }
}

#[derive(Debug, Copy, Clone, PartialEq, Eq)]
#[repr(C)]
pub struct Func {
    store: u32,
    index: NonZeroU32,
}

fn funcref_to_u64(funcref: FuncRef) -> u64 {
    if funcref.is_null() {
        return 0_u64
    }
    unsafe { mem::transmute::<FuncRef, u64>(funcref) }
}

fn u64_to_funcref(value: u64) -> FuncRef {
    if value == 0 {
        return FuncRef::null()
    }
    unsafe { mem::transmute::<u64, FuncRef>(value) }
}

#[test]
fn test_null_to_zero() {
    assert_eq!(funcref_to_u64(FuncRef::null()), 0_u64);
    assert!(u64_to_funcref(0_u64).is_null());
}

#[test]
fn test_funcref_to_u64() {
    let funcref = FuncRef { funcref: Some(Func { store: 1, index: NonZeroU32::new(2).unwrap() }) };
    assert_eq!(funcref, u64_to_funcref(funcref_to_u64(funcref)));
}


path completion: completing a folder should retrigger code completion

Testing #44460
Version 1.21.0-insider
Commit 9a04587cc6b42c332609a08bfd7e2dbeac592c21
Date 2018-03-01T06:47:39.037Z

create foo.html, bar.html in a folder 'a/b/c'
in `foo.html' have

<a href="../../../"

and do code completion after /.

pick a
this should result in <a href="../a/"  (slash at the end) and reopen code completion again so that the user can continue completing

I was modeling after TS's path complete, but I guess for html it's a bit different since you can import from folder and it defaults to index.js, and there is no such behavior in html.

A folder path is also valid in HTML (once navigated there, it's up to the web server to decide what to return), but I'd think it's the less common case and it users would rather continue to enter a path.

Think of the many keystokes necessary to enter a long path like aaa/bbb/foo.html
a, ctrl+space (code assist), enter to select aaa, /, b, ctrl+space, enter to select bbb, /, f, ctrl+space, enter to select foo.html
That's 11 keys to enter a path.
With the suggested change:

a, ctrl+space (code assist) select aaa, ctrl+space (code assist) select bbb, ctrl+space (code assist) select foo.html

To stick with a folder path:

a, ctrl+space (code assist) select aaa, ctrl+space (code assist) select bbb, escape

Let's be better than the extensions.


authorization in .net core

hi.
i get these errors that is attached in my images here.when trying to test authorization in ,net core in microsoft document.any advice?thanks.
https://learn.microsoft.com/en-us/aspnet/core/security/authorization/secure-data?view=aspnetcore-8.0#create-the-starter-app

Hello @Majidazadi12 ... Please re-open this issue using the Open a documentation issue link/feedback form at the bottom of the article. Use of the form adds metadata to your GitHub issue that cross-links the topic and pings the author automatically for a faster response.


Don't apply shorten-links to links in code blocks

Fixes #4758
Follow-up of #4759
Test URLs

Highlighted and non-highlighted code blocks in comment
Code block in readme
Review suggestion

Screenshot
Before

After

It should accept a Document | Element root

What should we use as testing subjects? I see two options:

A copy of the markup of a GitHub page, stored in a file. Problem: it will become obsolete as soon as GitHub changes its DOM.
Th URL of a GitHub page (via JSDOM.fromURL()). Problem: the snapshots will become obsolete when GitHub changes its DOM and the tests will fail.

What should we use as testing subjects? I see two options:

I think a barebones inline HTML snippet is enough, we just want to make sure that we don't keep causing regressions on this piece of code. The snippet can often be something like:

			<pre class="code-blob"
				https://google.com


Initialize metadata and tag indexing encoding for RediSearch

It close #2065.
The current encoding design:
search data type metadata:

key (index name) -> flag | expire | version | size | on_data_type (HASH or JSON)

prefixes encoding:

key (index name) | PREFIXES -> prefix1 prefix2 ...

tag field metadata encoding:

key (index name) | TAG_FIELD_META | field name -> separator | case sensitive

tag field index encoding:

key (index name) | TAG_FIELD | field name | tag | key -> (nil)

Refs:
https://redis.io/docs/interact/search-and-query/
https://redis.io/docs/interact/search-and-query/advanced-concepts/tags/

How can we list all indexes efficiently? 🤔

How can we list all indexes efficiently? 🤔 RediSearch seems would tracking the records for insert and maintaining index on that, should we support some global tracking structure?

Like RediSearch, we will track all HASH and JSON commands and adjust our indexes accordingly.

Like RediSearch, we will track all HASH and JSON commands and adjust our indexes accordingly.

I mean, when start the process, we should "LIST" all indexes

Like RediSearch, we will track all HASH and JSON commands and adjust our indexes accordingly.

I mean, when start the process, we should "LIST" all indexes

Ahh I got your point.
I think we can add a common prefix for all index key, or put them to a new CF. Currently I prefer the latter.

Like RediSearch, we will track all HASH and JSON commands and adjust our indexes accordingly.

I mean, when start the process, we should "LIST" all indexes

Ahh I got your point.
I think we can add a common prefix for all index key, or put them to a new CF. Currently I prefer the latter.

This skeleton code looks good to me. I also prefer the latter one if we want to keep the common prefix since the current column families are not for this purpose. But we need to do a little backward compatibility work while adding a new column family.

So only "tag" is support currently?

So only "tag" is support currently?

Yeah, I also plan to add the support of numeric indexes.
But for text and vector fields, it is currently not in the plan.


feat(console): add make:migration command

This PR adds the make:migration command
This checks a TODO on #759
The command allows you to create a raw migration file in SQL with the following command :
tempest make:migration create-book-table raw
You can create a classic migration file in PHP using :
tempest make:migration CreateBookTable
Or you can even create a migration file for a specific existing model with :
tempest make:migration CreateBookTable model
Tempest will scan the main namespace and let you chose in all classes that implements DatabaseModel  the Model you want to create the migration for
The final generated file and statements are then slightly differents

Note that this PR should not be merged without fixing #866  and #867 as those issues have been discovered while working on this PR.


Update chromedriver to 89

What is the context of this PR?
Updates chromedriver to 89 so it will match the future version of Chrome installed in the github actions. It'll prevent pull requests  from failing when the chromedriver version and the installed version of Chrome are different in future.
How to review

Run functional tests locally
Check that the functional tests run successfully in this pull request

 Update JS dependencies


Add make to Ubuntu/Debian dependencies

On a fresh Ubuntu 20.04 machine, I saw build failures for the substrate node template. Eventually I found this StackExchange question, which showed that make was not installed. Apparently it doesn't come preinstalled, at least in Ubuntu 20.04.

Added in #1081


Arkime Docker install

Hey, I'm trying to install with 5.5.0 docker and unfortunately there is no full deployment documentation to support me in installing. I have an ES8.8.1 cluster but don't know how to initialize a container after the docker image is created
If I can, I'll provide a screenshot of the error

By the way, my OS version is Debian 10 and currently only see the version supported by Debian 12

These two links might help http://arkime.com/questions and http://arkime.com/docker

I confirm that I've read all the official documentation, including these two：
http://arkime.com/questions and http://arkime.com/docker
Unfortunately, the documentation didn't help me build an Arkime Container from scratch. Let me briefly talk about my operation and the problems I encountered.
First, I have ES-8.8.1 cluster, health and green:

I downloaded the official image: ghcr.io/arkime/arkime/arkime:v5-latest

Then I looked at the documentation：http://arkime.com/docker  ,and configured docker-compose.yaml
version: '3'

services:

  capture:
    image: ghcr.io/arkime/arkime/arkime:v5-latest
    network_mode: "host"
    command: /opt/arkime/bin/docker.sh capture --update-geo
    volumes:
      - /home/wzj/arkime/pcap:/opt/arkime/raw
      - /home/wzj/arkime/etc:/opt/arkime/etc
    restart: always
    container_name: arkime-capture
  viewer:
    image: ghcr.io/arkime/arkime/arkime:v5-latest
    network_mode: "host"
    command: /opt/arkime/bin/docker.sh viewer
    volumes:
      - /home/wzj/arkime/pcap:/opt/arkime/raw
      - /home/wzj/arkime/etc:/opt/arkime/etc
    restart: always
    container_name: arkime-viewer

I found that I needed to config.ini configuration, so I got the default configuration "release/config.ini.sample" in the 5.5.0 branch and modified it:
[default]
elasticsearch=https://user:pass@host:port
rotateIndex=daily

passwordSecret=ARKIME_PASSWORD
httpRealm=Moloch
interface=ARKIME_INTERFACE
pcapDir=ARKIME_INSTALL_DIR/raw
maxFileSizeG=12
tcpTimeout=600
tcpSaveTimeout=720
udpTimeout=30
icmpTimeout=10
maxStreams=1000000
maxPackets=10000
freeSpaceG=5%

rirFile=ARKIME_INSTALL_DIR/etc/ipv4-address-space.csv
ouiFile=ARKIME_INSTALL_DIR/etc/oui.txt

dropUser=nobody
dropGroup=daemon
parseSMTP=true
parseSMB=true
parseQSValue=false
supportSha256=false
maxReqBody=64

reqBodyOnlyUtf8=true

smtpIpHeaders=X-Originating-IP:;X-Barracuda-Apparent-Source-IP:

parsersDir=ARKIME_INSTALL_DIR/parsers
pluginsDir=ARKIME_INSTALL_DIR/plugins

spiDataMaxIndices=4

packetThreads=2
pcapWriteMethod=simple
pcapWriteSize=262143
maxESConns=30
maxESRequests=500
packetsPerPoll=50000

logEveryXPackets=100000
logUnknownProtocols=false
logESRequests=true
logFileCreation=true

[headers-http-request]
referer=type:string;count:true;unique:true
authorization=type:string;count:true
content-type=type:string;count:true
origin=type:string

[headers-http-response]
location=type:string
server=type:string
content-type=type:string;count:true

[headers-email]
x-priority=type:integer
authorization=type:string

The directory structure is as follows：

Finally, used "docker-compose up -d" to create a container：

capture's error message:

viewer's error message:

@awick
I've never built Arkime before, either binary or docker version, I thought initializing operations was automatic in docker versions, but apparently that's not the case. Is there a problem with my operation and what do I need to do next?

So like http://arkime.com/questions says, please come to slack

You'll have to run init yourself, I'll continue to update http://arkime.com/docker with more info
You need to fix your config file and set all the ARKIME_ with real values
All those IMPORTANT errors are telling you what is wrong. So it can't find or open the config file. I would guess the permissions of /home/wzj do not allow docker to read the config file maybe?

Thanks for the reply. The config.ini was modified to real values, the question in this issue has been resolved, and the new issue I'm currently having is raised in #3020

Please come to slack. If you are asking about init, I answer it above


fft.waveform --> amplitude.waveform

As @rev3rend pointed out, .waveform() (oscilloscope) should not be a function of the FFT object.

This is an old comment, but a good point. The waveform should probably be attached to the main audio input signal/buffer somehow?


Documentation and APIs

I am creating a prototype using OrientDB.
I have realized that the documentation is very inconsistent for starting development for new comers. For example, when discussing Document DB, it suddenly jumps to Graph DB concepts. Same for Object DB concepts. Also, some APIs are deprecated in the examples.
Explicitly specifying different storage models -  Graph DB, Object DB & Document DB and using simple data model with CRUD operations to describe different operations will be a good addition to the documentation.
Also, there are inconsistencies in APIs.
My mental model thinks when acquiring a txn,  I would acquire either one of

OGraphDatabaseTx
OObjectDatabaseTx
ODocumentDatabaseTx not ODatabaseDocumentTx

When creating an instance from a pool, OGraphDatabasePool etc.
OObjectDatabasePool is deprecated in 2.1.2, but the replacement OPartitionedDatabasedPool does not return a OObjectDatabaseTx
Happy to discuss other thoughts.

I noticed the following points:

Multithreading

Examples on how to use orientdb Java API in a multithreading environment would helped me a lot. It was unclear to me that i can't reuse OrientVertex instances across multiple threads.
I guess it would not hurt to add javadoc within OrientVertex that informs you that the OrientVertex class is not theadsafe.

Warnings

I'm currently creating OrientEdgeType's on the fly and thus i see a lot of warnings in my log. I did not realize that i should create the OrientEdgeType's when i initialize my graph or in a dedicated notx transaction. Instead i stumbled upon OrientBaseGraph.executeOutsideTx() which has no JavaDoc & no documentation. I think it would be good to adapt the warning message or to create a dedicated documentation page which explains the warning and shows ways to avoid those warnings.
WARNING: {db=tinkerpop} Requested command 'create edge type 'HAS_ITEM' as subclass of 'E'' must be executed outside active transaction: the transaction will be committed and reopen right after it. To avoid this behavior execute it outside a transaction

embedding / remote connection

The http://orientdb.com/docs/2.1/Embedded-Server.html documentation explains how to setup the OServer but it does not explain how to use the OServer in combination with a plocal database in the same JVM. After some research and experimentation i realized that the OServer will automatically add found databases to the server when it detects it in the database folder and that the default login is admin/admin. A section that explains this would be very nice. Additionally a section that also explains how to enable replication in this kind of environment would be nice.

Indices

Examples on how to setup and use Verticle and Edge Indices via Java would be very helpful. I was able to setup the edge indices when i took a look at the performance test mentioned in #4105.
I did not manage to create an index using the IndexableGraph.createIndex() method. Instead i used the OrientEdgeType.createIndex() method. In general it is still a bit confusing to me that there are multiple ways to create an index. Furthermore some methods seem to utilize the index and some are not. For example  graph.getVertices("item.name", "somename" is not working for me but graph.getVertices("item", new String[] { "name" }, new Object[] { "somename" })is.
Initially i got the impression that orientdb would automatically create edge indices and make use of them. For example when invoking: OrientVertex.getEdges(final OrientVertex iDestination, final Direction iDirection, final String... iLabels) but this is afaik not the case.

Edge Schema/Type

I'm creating verticle types using graph.createVertexType("item", "V"); but i needed to use the label of my edges to create the edge type graph.createEdgeType("HAS_ITEM");

Documentation versions

https://www.google.com/search?q=oriendb+index+java returns various versions of documentation pages. I think it would be good to put a info text at the top of each 2.0 and older documentation pages which tells you that there is a newer version accessible.

I'd like to add something as I just wanted to create an issue but then I saw that there's already this one.

Navigation
As a Chrome user, the documentation is very painful to use as a click at the tree of topics on the left makes this tree disappear. Using the browser "back" sometimes doesn't get me back where I came from, I've got to reload the whole page. Also, the search function is broken. It doesn't return the filtered tree nor does it return any result. The workaround is to use Firefox, but I just don't want to install it just to use it with a single page(...any longer).

Doc content
Looking at the content, I'd say that there could be a bit more stuff about handling stuff via the Java API. Adding some more best practises could reduce redundant questions in the google group or on Stackoverflow and lighten up the customer experience.

My personal favourite "db-docs" are the ones of PostgreSQL. They've got a nice, clean, working interface that supports several versions of the product, as well:
http://www.postgresql.org/docs/9.3/interactive/rules-views.html#RULES-SELECT
Nevertheless, they don't have such a tree-of-topics representation at each site which I like about the OrientDB docs.

I also use Chrome and I don't have any issue with the menu on the left disappearing. I also have no problem with navigating back with the back button,
Scott

Documentation has already been considerably revamped recently and will continue to be improved.


Upgrade to Daffodil 3.5.0

Fixes #680.

Sorry, PEBKAC (my error).
On Thu, Aug 10, 2023 at 9:00 AM Steve Lawrence @.***>
wrote:

@.**** commented on this pull request.
In package.json
https://github.com/apache/daffodil-vscode/pull/749#discussion_r1290357695
:

@@ -3,7 +3,7 @@
"displayName": "Apache Daffodil™ Extension for Visual Studio Code",
"description": "Apache Daffodil™ Extension for Visual Studio Code providing DFDL syntax highlighting, DFDL code completion, DFDL schema debugging, and data editor",
"version": "1.3.0",

"daffodilVersion": "3.4.0",

"daffodilVersion": "3.5.0",

Did you include daffodil-cli/bin.NOTICE in the diff? THis is the diff I
see for that file:
index 755b6a29d..dab451ff9 100644--- a/daffodil-cli/bin.NOTICE+++ b/daffodil-cli/bin.NOTICE@@ -19,6 +19,10 @@ The following NOTICE information applies to binary components distributed with t
been in development since 2007.
It is currently developed by a community of developers.

Copyright++  Copyright 2007-, Tatu Saloranta @.***)+

Licensing
Jackson 2.x core and extension components are licensed under Apache License 2.0@@ -30,6 +34,20 @@ The following NOTICE information applies to binary components distributed with t
in some artifacts (usually source distributions); but is always available
from the source code management (SCM) system project uses.

FastDoubleParser++  jackson-core bundles a shaded copy of FastDoubleParser https://github.com/wrandelshofer/FastDoubleParser.+  That code is available under an MIT license https://github.com/wrandelshofer/FastDoubleParser/blob/main/LICENSE+  under the following copyright.++  Copyright © 2023 Werner Randelshofer, Switzerland. MIT License.++  See FastDoubleParser-NOTICE for details of other source code included in FastDoubleParser+  and the licenses and copyrights that apply to that code.++- com.typesafe.scala-logging.scala-logging_.jar+  Copyright 2014-2021 Lightbend, Inc.+

commons-codec.commons-codec-.jar
Apache Commons Codec
Copyright 2002-2020 The Apache Software Foundation@@ -87,25 +105,6 @@ The following NOTICE information applies to binary components distributed with t
Apache HttpComponents Core HTTP/2
Copyright 2005-2020 The Apache Software Foundation
-- org.apache.logging.log4j.log4j-api-.jar-  Apache Log4j API-  Copyright 1999-2022 The Apache Software Foundation--- org.apache.logging.log4j.log4j-api-scala_.jar-  Apache Log4j Scala API-  Copyright 2016-2022 Apache Software Foundation--  AsciidocPlugin.scala-  Copyright (c) 2008, 2009, 2010, 2011 Josh Suereth, Steven Blundy, Josh Cough,-  Mark Harrah, Stuart Roebuck, Tony Sloane, Vesa Vilhonen, Jason Zaugg--- org.apache.logging.log4j.log4j-core-.jar-  Apache Log4j Core-  Copyright 1999-2012 Apache Software Foundation--  ResolverUtil.java-  Copyright 2005-2006 Tim Fennell-

org.jdom.jdom2-.jar
Copyright (C) 2000-2012 Jason Hunter & Brett McLaughlin.
All rights reserved.@@ -134,8 +133,8 @@ The following NOTICE information applies to binary components distributed with t

org.scala-lang.scala-library-.jar

org.scala-lang.scala-reflect-.jar
Scala-  Copyright (c) 2002-2022 EPFL-  Copyright (c) 2011-2022 Lightbend, Inc.+  Copyright (c) 2002-2023 EPFL+  Copyright (c) 2011-2023 Lightbend, Inc.
Scala includes software developed at
LAMP/EPFL (https://lamp.epfl.ch/) and

And here's the diff for daffodil-cli/bin.LICENSE
diff --git a/daffodil-cli/bin.LICENSE b/daffodil-cli/bin.LICENSE
index e9ac9701a..e0bae9ece 100644--- a/daffodil-cli/bin.LICENSE+++ b/daffodil-cli/bin.LICENSE@@ -228,7 +228,7 @@ is subject to the terms and conditions of the following licenses.
 COPYRIGHT AND PERMISSION NOTICE

Copyright © 1991-2022 Unicode, Inc. All rights reserved.+    Copyright © 1991-2023 Unicode, Inc. All rights reserved.
Distributed under the Terms of Use in https://www.unicode.org/copyright.html.

 Permission is hereby granted, free of charge, to any person obtaining@@ -651,6 +651,32 @@ is subject to the terms and conditions of the following licenses.
 (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

+- com.fasterxml.jackson.core.jackson-core-2.13.4.jar+  This product bundles 'Jackson JSON processor' which bundles a shaded copy of FastDoubleParser https://github.com/wrandelshofer/FastDoubleParser.+  That code is available under the MIT license:++    MIT License++    Copyright (c) 2023 Werner Randelshofer, Switzerland.++    Permission is hereby granted, free of charge, to any person obtaining a copy+    of this software and associated documentation files (the "Software"), to deal+    in the Software without restriction, including without limitation the rights+    to use, copy, modify, merge, publish, distribute, sublicense, and/or sell+    copies of the Software, and to permit persons to whom the Software is+    furnished to do so, subject to the following conditions:++    The above copyright notice and this permission notice shall be included in all+    copies or substantial portions of the Software.++    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR+    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,+    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE+    AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER+    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,+    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE+    SOFTWARE.+

com.lihaoyi.geny_.jar
com.lihaoyi.os-lib_.jar
This product bundles 'os-lib' from the above files.@@ -1782,7 +1808,7 @@ is subject to the terms and conditions of the following licenses.
This product bundles 'SLF4J API' from the above files.
These files are available under the MIT license:
Copyright (c) 2004-2017 QOS.ch+    Copyright (c) 2004-2022 QOS.ch Sarl (Switzerland)
All rights reserved.

 Permission is hereby granted, free  of charge, to any person obtaining

—
Reply to this email directly, view it on GitHub
https://github.com/apache/daffodil-vscode/pull/749#discussion_r1290357695,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAHTO6543B7PXCPPV4UKYLXUUARVANCNFSM6AAAAAA3I6PN6Q
.
You are receiving this because you were assigned.Message ID:
@.***>


Update ActiveSupport/Railties

Due to https://github.com/ruby/gem_rbs_collection/issues/177, we're bringing in our own definitions for ActiveSupport (and Railties, which requires it) until that is fixed

What did you run to generate this?

rbs prototype rb path/to/gem-source-location/gem-name-1.2/lib/**/*.rb > path/to/this/repo/gems/gem-name/1.2/gem-name-generated.rbs
with a bunch of iteration until it passes, and looking at the existing type definitions when i ran into issues. Documentation to generate new types is on the readme at https://github.com/Shelf-Life/gem-rbs


Feature - Anchor ID's

One continuous issue we had with gitbooks was anchor ID's in HTML (looked terrible and would break IIRC). Especially for content that becomes technical, like keepers guides, it's helpful to point to and review ideas/concepts within one file. We had to remove anchor ID's altogether but getting the convo going on how we would use them as a component, or the best decision(s) to evaluate.
I assume it would be straightforward to make a component for these that is easier to use.

@twblack88 Anchor IDs would be be similar to HOCs (Higher Order Components) that wrap all existing components. It would be possible to have CCs provide a prop like 'anchorID' that would have the page scroll to the point of interest.

Tagging as nice to have as it's leaning outside of the scope of what's needed to complete the v1.

@twblack88 I've gained back some time so I will try to get this in for v1 launch as well.

@twblack88 I tried to do this the easy way, but it will require a little bit more work on my end before it's solid. I'm going to say this is a v2 feature. It's entirely possible to do by hand, but it will take up valuable time I'd like to spend testing and prepping the site for launch.
For any future dev:
I tried using gatsby-remark-autolink-headers. But the link didn't work, it broke the title rule (cause it builds into the MD), and there was no good way to determine what it should be as an id. I recommend instead extending the header shortcodes to support this feature. It's just an anchor link, so it should be simple enough.

Thanks for taking a stab! No worries at all, you can add a hash to the headings so apparently there is some functionality that seems to work.


Feature/card state persistence

Summary

The card states were being reset on every app launch (full restart not resuming)
Changelog

[General] [Add] - Card states data persistence
[General][Add] - Card order data persistence
Test Plan

Do the following while logged out first. Then try it while logged in.

Install app
Hide the a card
Force-close the app
Re-launch the app
The hidden card should not appear

Run this test on android and iOS.

[x] iOS Test pass
[x] android Test pass

Note: Required by #726 for Remote Card Activation Control


Force or schedule update for VMware tools

I am able to get information about a VM using vm.guest.toolsVersionStatus2 (and others) without any issues.  I am just wondering if there is a way to force or schedule an update when the result is guestToolsSupportedOld.

Great thanks.  I'll try it and post back.

That worked well.  Thank you for the help. Again.  Thank you for the links too.


Model references not loading

Description
Got a model for showing with  but src only works with this link of the Astronaut.glb "https://cdn.glitch.com/36cb8393-65c6-408d-a538-055ada20431b%2FAstronaut.glb?v=1542147958948".
I downloaded the model to use it locally but it doesnt load.

Browser Affected

[x] Chrome
[x] Edge
[x] Firefox
[ ] Helios
[ ] IE
[ ] Safari

OS

[ ] Android
[ ] iOS
[ ] Linux
[ ] MacOS
[x] Windows

Versions

model-viewer: v0.6.0
three.js: rXX

Hey @J97cp thanks for the report. Can you reload the page with your developer tools open to the network tab and paste a screenshot of what you see there?


Implement smoltcp with VirtIO NIC

This PR provides the needed types for smoltcp using the VirtIO NIC over PCI. An example echo server is provided in src/bin/virtio/main.rs. A few notes:

The current QEMU arguments only have port 5555 open. To use any other port, we need to modify the NIC section of the QEMU arguments. Connect to localhost:5555 for testing.
Only one process can create a functional device in the current code. There are no checks as it stands to ensure that only one device is created. Both of these issues can be fixed, but I wanted to get the PR review process started.

Will merge once the checks finish


Introduce FdSlice to mitigate potential unaligned access

As suggested in #9

Thanks!
I hadn't thought an iterator type would be necessary. but it's nice. :smile:

Released as 0.2.8 and 0.3.0.


Add acceptance test skeleton.

Adds the acceptance skeleton from the puppet-module-skeleton.
Adds initial class_spec tests for Splunk and Splunkforwarder classes.
Includes the appropriate module dependencies for testing.
This is just the foundation for adding more tests.

+1

+1


Find a way to purge local storage when docs are updated

I used Kontra from version 7. Kontra caches the docs to localStorage when you visit a page. After version 8 was released, I still see the old docs (because of the cache). After deleting the localStorage under https://straker.github.io/ I was able to get the latest docs.
I happen to notice this after bindKeys didn't work anymore.

That's a good point. The docs have a service worker that fetches the latest content and will update for you after you navigate to it and refresh/close the page. But there is that first time that means you have out-of-date docs. I'll try to implement a "There's a newer version of this page" type of notification so you're informed of the changes.